Refactored Client Interfaces & Enhanced Tests

This patch refactors the Client interfaces such that ./api/client now has the Client interface which is the base for the ./client/Client and ./api/client/APIClient interfaces.
The ./api/client/APIClient interface is what the ./api/client/Client interface used to be. The ./client/Client interface is a superset of the ./api/client/Client functions with the addition of helper functions. However, the ./client/Client object returned by the NewClient function can be cast to ./api/client/APIClient when it's necessary to access underlying, advanced API functions.
Finally the test harness has been enhanced to provided canned test routines for testing executors, getting the instance ID, and validating common API routes.

Current coverage is 34.43%

Merging #34 into master will not affect coverage as of 043470e

@@            master     #34   diff @@
======================================
  Files           12      12
  Stmts          697     697
  Branches        15      15
  Methods          0       0
======================================
  Hit            240     240
  Partial         15      15
  Missed         442     442

Review entire Coverage Diff as of 043470e

Powered by Codecov. Updated on successful CI builds.


Use bullseye branch instead of master

Master branch has been dropped and bullseye became the main branch of
the project.
This is not the cherry-pick of 3912f5b5 because mentioned commit brings
firmware version changes and updates the revision so not a candidate for
dunfell
Related: https://github.com/RPi-Distro/firmware-nonfree/issues/20
Signed-off-by: Pavel Zhukov<EMAIL_ADDRESS>

Can one of the admins verify this patch?

@agherzan-ci ok to test


Error Installing pcf8574 & pcf8575 modules

When I install the modules pcf8574 and pcf8575 inside the virtual environment I got the error in the end:
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-qGDNFX/smbus-cffi/
I got from the log:
Collecting pcf8574
  Using cached https://files.pythonhosted.org/packages/5f/ab/c8955e73d866977c221fafc167a2fa6bdab5f07b3569fb2dec1d8a7a8836/pcf8574-0.0.5.tar.gz
Collecting smbus-cffi (from pcf8574)
  Using cached https://files.pythonhosted.org/packages/29/3d/a50bd997979c7554c9c571753d34e976eb88ebf41d3a66accb2468bd3c69/smbus-cffi-0.5.1.tar.gz
    Complete output from command python setup.py egg_info:
    Package libffi was not found in the pkg-config search path.
    Perhaps you should add the directory containing `libffi.pc'
    to the PKG_CONFIG_PATH environment variable
    No package 'libffi' found
    c/_cffi_backend.c:15:10: fatal error: ffi.h: No such file or directory
     #include <ffi.h>
              ^~~~~~~
    compilation terminated.
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-1KSHMT/smbus-cffi/setup.py", line 48, in <module>
        'Topic :: System :: Hardware',
      File "/usr/lib/python2.7/dist-packages/setuptools/__init__.py", line 144, in setup
        _install_setup_requires(attrs)
      File "/usr/lib/python2.7/dist-packages/setuptools/__init__.py", line 139, in _install_setup_requires
        dist.fetch_build_eggs(dist.setup_requires)
      File "/usr/lib/python2.7/dist-packages/setuptools/dist.py", line 724, in fetch_build_eggs
        replace_conflicting=True,
      File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 782, in resolve
        replace_conflicting=replace_conflicting
      File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 1065, in best_match
        return self.obtain(req, installer)
      File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 1077, in obtain
        return installer(requirement)
      File "/usr/lib/python2.7/dist-packages/setuptools/dist.py", line 791, in fetch_build_egg
        return cmd.easy_install(req)
      File "/usr/lib/python2.7/dist-packages/setuptools/command/easy_install.py", line 704, in easy_install
        return self.install_item(spec, dist.location, tmpdir, deps)
      File "/usr/lib/python2.7/dist-packages/setuptools/command/easy_install.py", line 730, in install_item
        dists = self.install_eggs(spec, download, tmpdir)
      File "/usr/lib/python2.7/dist-packages/setuptools/command/easy_install.py", line 915, in install_eggs
        return self.build_and_install(setup_script, setup_base)
      File "/usr/lib/python2.7/dist-packages/setuptools/command/easy_install.py", line 1183, in build_and_install
        self.run_setup(setup_script, setup_base, args)
      File "/usr/lib/python2.7/dist-packages/setuptools/command/easy_install.py", line 1171, in run_setup
        raise DistutilsError("Setup script exited with %s" % (v.args[0],))
    distutils.errors.DistutilsError: Setup script exited with error: command 'arm-linux-gnueabihf-gcc' failed with exit status 1

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-1KSHMT/smbus-cffi/
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/aerodolphin/ve/lib/python2.7/site-packages/pi_mqtt_gpio/server.py", line 887, in <module>
    main(args)
  File "/home/aerodolphin/ve/lib/python2.7/site-packages/pi_mqtt_gpio/server.py", line 759, in main
    GPIO_MODULES[gpio_config["name"]] = configure_gpio_module(gpio_config)
  File "/home/aerodolphin/ve/lib/python2.7/site-packages/pi_mqtt_gpio/server.py", line 436, in configure_gpio_module
    install_missing_requirements(gpio_module)
  File "/home/aerodolphin/ve/lib/python2.7/site-packages/pi_mqtt_gpio/server.py", line 254, in install_missing_requirements
    % (module, pkgs_required, err)
__main__.CannotInstallModuleRequirements: Unable to install packages for module <module 'pi_mqtt_gpio.modules.pcf8574' from '/home/aerodolphin/ve/lib/python2.7/site-packages/pi_mqtt_gpio/modules/pcf8574.pyc'> (['pcf8574']): Command '['/usr/bin/env', 'pip', 'install', 'pcf8574']' returned non-zero exit status 1
2020-10-16 10:31:49,991 mqtt_gpio (INFO): Startup
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple

How to solve it?

This solved the problem:
sudo apt-get update
sudo apt-get install libffi-dev


where's the x86_64 3.8.1 version?

seems only x86 version and others were bumped to 3.8.1?

I think it's there, it just gets bumped in a different way than the other files in the repository.
$ docker run alpine:3.8 cat /etc/os-release
Unable to find image 'alpine:3.8' locally
3.8: Pulling from library/alpine
4fe2ade4980c: Pull complete
Digest: sha256:621c2f39f8133acb8e64023a94dbdf0d5ca81896102b9e57c0dc184cadaf5528
Status: Downloaded newer image for alpine:3.8
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.8.1
PRETTY_NAME="Alpine Linux v3.8"
HOME_URL="http://alpinelinux.org"
BUG_REPORT_URL="http://bugs.alpinelinux.org"

ahh.  thanks!


Test result sending fails when Artemis is not running on port 443

Describe the bug
The job created in Jenkins contains the following URL as server URL in test result sending section:
https://rss-bbbutton.ddnss.org/api/programming-exercises/new-result
However, my server is running on a non-standard https port as configured in application-prod config file. It should read
https://rss-bbbutton.ddnss.org:4443/api/programming-exercises/new-result
I am not sure whether I can use
https://rss-bbbutton.ddnss.org:4443
as entry under server as then the port number would be given twice in two different attributes (port vs. url).
To Reproduce
Run Artemis on non-standard server port, create an programming exercise.
Expected behavior
s. above
Environment

 - OS: Linux server, Macos client
 - Browser Firefox
 - Version HEAD

Hi Steffen, I think this should work out that you add a port in the server.url attribute in the yml file.
Please try it out and let us know if it works.

Hi Stefan, indeed, adding the port to the server attribute works and then it creates the right URL in jenkins. Maybe a warning is helpful pointing out the workaround in the logs when the port property is different from 443 but the URL does not contain it.


[cxx-interop] Import CF_OPTIONS types such as Foundation.NSTextCheckingType correctly

This fixes linker errors which occurred when using Foundation extensions to Swift.String, such as func replacingOccurrences.
After the last rebranch, Clang started wrapping certain types in clang::ElaboratedType. This caused CF_OPTIONS types such as Foundation.NSTextCheckingType to be imported incorrectly in C++ language mode (NSTextCheckingType was imported as uint64_t without getting the special treatment of a CF type), which was causing deserialization errors when Swift tried reading Foundation.swiftmodule. Those deserialization errors were silenced by default. The IR for those functions was not emitted, which caused linker errors later.
rdar://109830032

@swift-ci please smoke test

@swift-ci please smoke test


'mat-sidenav' inside component and not directly in 'mat-sidenav-container' does not resize to 100%

Bug, feature request, or proposal:
When 'mat-sidenav' is wrapped in component under 'mat-sidenav-container' then height 100% is not calculated correctly anymore but If  'mat-sidenav' element is directly under 'mat-sidenav-container' it is ok.
Basically: following breaks height 100%
<mat-sidenav-container>
  <app-sidenav></app-sidenav>

    <router-outlet></router-outlet>

</mat-sidenav-container>

but: would work:
<mat-sidenav-container>
  <mat-sidenav mode="side" opened="true" class="sidenav">
    <mat-nav-list>
      <mat-list-item *ngFor="let link of links">
        {{ link }}
      </mat-list-item>
    </mat-nav-list>
  </mat-sidenav>

    <router-outlet></router-outlet>

</mat-sidenav-container>

What is the expected behavior?
height should be 100% for sidenav not be stuck with window initial size
What is the current behavior?
height is stuck to size of window

But if 'mat-sidenav' is in html directly under 'mat-sidenav-container' every thing is ok
What are the steps to reproduce?
Example project: https://github.com/aldas/angular-material2-sidenav-issue-example
Sidenav OK commit: https://github.com/aldas/angular-material2-sidenav-issue-example/commit/ced15fbff94f00099372000b00350d2279bab67b
What is the use-case or motivation for changing an existing behavior?
modularise application menu inside sidenav setup.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
    "@angular/core": "^5.0.0",
    "@angular/material": "^5.0.0-rc0",

...
    "typescript": "~2.4.2"
    "@angular/cli": "1.5.0",

Chrome latest, FF latest
Is there anything else we should know?
nope

Using mat-sidenav in a different component renders it within mat-sidenav-content, probably because that's how angular renders components.
Expected structure:
<mat-sidenav-container>
    <mat-sidenav-content>
    </mat-sidenav-content>
    <mat-sidenav>
    </mat-sidenav>
</mat-sidenav-container>

What happens when you put mat-sidenav in a component:
<mat-sidenav-container>
    <mat-sidenav-content>
        <mat-sidenav>
        </mat-sidenav>
    </mat-sidenav-content>
</mat-sidenav-container>

Atm in my real project I just pulled 'mat-sidenav' up to parent component and modularized just content of 'mat-sidenav' into its own component 'app-sidenav'
so: this works fine.
<mat-sidenav-container>
  <mat-sidenav mode="side" opened="true" class="sidenav">
    <app-sidenav></app-sidenav>
  </mat-sidenav>

    <router-outlet></router-outlet>

</mat-sidenav-container>

I'm quite new to Angular so I did not know to think about that (some) components/directives have strict parent-child relation and there can not be intermediate elements/components between.

<mat-sidenav> should be placed directly inside the <mat-sidenav-container>, so this is working as intended

Just wasted an hour on this, you might to mention in the doco page that  wouldn't work if it pushed down to a custom component.
https://material.angular.io/components/sidenav/overview


Fixes #22

This will allow useSoftDeletes to detect the trait up the inheritance tree, fixing the desired behavior.

A new inspection was created.

The inspection completed: No new issues


fix: update to eye v10.16.14

fix: update to eye v10.16.14

:tada: This PR is included in version 16.15.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v16.15.15

Your semantic-release bot :package::rocket:


Fix the pending_request_count to release the failed requests

Before
After sending a 136 requests that will fail in Enqueue:
curl -s localhost:8002/metrics | grep pending
# HELP nv_inference_pending_request_count Instantaneous number of pending requests awaiting execution per-model.
# TYPE nv_inference_pending_request_count gauge
nv_inference_pending_request_count{model="test_model",version="1"} 136

After fix
After sending a 136 requests that will fail in Enqueue:
curl -s localhost:8002/metrics | grep pending
# HELP nv_inference_pending_request_count Instantaneous number of pending requests awaiting execution per-model.
# TYPE nv_inference_pending_request_count gauge
nv_inference_pending_request_count{model="test_model",version="1"} 0

I couldn't find any tests for nv_inference_pending_request_count metrics. Need to add testing of these features to avoid regression. Will open another ticket to enhance the testing.

Also, existing tests can be found here: https://github.com/triton-inference-server/server/blob/main/qa/L0_metrics/metrics_queue_size_test.py
Feel free to rename test script from queue size to pending request count or something if it makes it easier to find


[1609] Migrate code to application terraform module

Context
Use the application terraform module instead of the separate terraform resources, to simplify the code and take advantage of future improvements in the module.
Changes proposed in this pull request

Use the application module
Remove the kubernetes module
Remove extra code
Remove references to paas

Guidance to review
Follow steps in https://hedgedoc.teacherservices.cloud/2_sbmGGhRvu3g2ua5aVDJQ?view
Link to Trello card
https://trello.com/c/kqnBW9IA

 Apply for TT: migrate code to terraform modules


chore: 🔧 Make angular demo run with standalone dependencies to prevent errors when required version of wrappers differ

Description
After multiple problems with e2e tests not working as expected, we traced the problem to an error while building our angular demo project. This tries to share the version of @angular/core and @angular/forms with the one provided in @synergy-design-system/angular. This leads to problems with the angular compiler, as the versions may differ. In this case (e.g. when @synergy-design-system/angular has a different version of one of this packages, the local module resolution of angulars build system does not work anymore. However, the server itself is still starting up. This leads to the problem that selectors created via Playwright will never run in a timeout, making the tests stay in sort of a limbo between death and live.

This is NOT a problem that users may likely face in the wild, as it is caused by the dedupe mechanism used in pnpm and the usage of a monorepo that locally links the packages together.

Rationale
We have to make sure updates of peerDependencies do not cause problems with the build of the demo to make tests less flaky.

This was tried in https://github.com/synergy-design-system/synergy-design-system/pull/502. However, it does not work correctly, because there are multiple issues with this.

During development of https://github.com/synergy-design-system/synergy-design-system/pull/508, we noticed that this is also happening in our react vite builds. However, in this case, only production builds seem to have the problem. This may be some internal problem when using esbuild.


Reduce the hbar value used in an example or tutorial

Problem
For the purposes of examples and tutorials, we do not need to use large values of hbar. For example, to show a how a TransferTransaction works we can specify one hbar vs 100 hbar.
Solution
Review examples and tutorials and if there are places we use over 5 hbar, reduce the value to 5.
Alternatives
No response

Resolved in 16d6e6c


Exports objects out of files and assemble in crafty.js

This helps to avoid globals and makes it more modular as the components can be loaded individual.
It is a step towards making it work with node.js.
I have not done it with the rest of the modules as I want to get feedback first :)

That's exactly what you suggested before. I like the idea very much, it's clean approach, however it's a lot of work.
Crafty already works on nodejs with minimal changes (see #902 ), even the tests run on nodejs successfully.
Lets also hear @starwed 's opinion on this.

It took me 20 minutes, so the rest is maybe 30 minutes :) So that is okay.
But you see that there is quite some different approaches to extending the Crafty object. Some extend the whole object, other attach to a property and then we have Components.

Nobody seems to comment against it, so merging and will make a new pull request with the rest of the work done.

Right now you plan to export one component / extension-object per file, right? See starwed's comment from before

I'd like to see only one component per file.

Hmm, I'm not so certain about this as a hard rule. Some components are very short, and depend heavily on other components. For instance, should "FourWay" be in a separate file from "MultiWay"? All the former component does is call multiway. Having to constantly hop between files hurts readability just as much as having to navigate a huge monolithic file.

There are scores of components that should be in their own files, though, so I agree in general. And I think everything in its own file, while not ideal, is certainly an improvement over the current situation.

I tend to agree that all *way components are easier to understand if they are in one file. So if we go down this route (to bundle multiple components into a file) then exporting objects gets messy, as you have to export an array of components and/or extension objects - it would be arguably worse than the way it is now.

@mucaho I understand what you're saying, and I think it can be solved by making a function in Crafty that takes an array of components and loops over them.
The problem that I am solving is not putting each component in each file, but removing the need for Crafty to be global variable (globals are bad).
But I agree that some components should stay together because they are so close together in functionality.


Allow Octave interface to compile with Octave v5++ installed

This PR is for users who want the Octave interface and have Octave version 5.x.x installed.
Synopsis:
Compiled Shogun with the Octave interface (only that one) in Ubuntu 20.04 LTS. I followed the build steps from the Install page in the Home website, but it failed when it didn't find the header file "octave/config.h"
My local Octave installation is version 5.2.0 and I found that the make templates didn't cater for any version higher than 4.x.x.
After adding the check for Octave version 5.x,x and pointing the compiler to the right header file (octave/octave-config.h) the compilation and installation went through with no issues.
I also tested several of the examples within my Octave installation and they all run fine.

Great, thank you very much for that!

Our CI for the interface is down atm. It might take a little while until we fix it and then merge the PR (xmas etc). But much appreciated!


Enable Application Insights crash logging

DO NOT MERGE.
Errors and exceptions  are reported to app insights.

[ ] update AI SDK to disable cookies and local storage.

Blocked by https://github.com/Microsoft/ApplicationInsights-JS/issues/335


git checkout Chinook_Sqlite.sqlite after tests

Ref https://github.com/JuliaDB/SQLite.jl/issues/109

Coverage remained the same at 0.0% when pulling ae175841112452d0cde61693474d3a30dcfe1b3d on felipenoris:fn/ckout into cc88dbd4949b5090208aab928848c40912df76e9 on JuliaDB:master.

no, you shouldn't assume git is installed in order for this to work.use a separate file, or gitignore it.


[DB Import] Add presubmit check

This PR is adding a few related things at the same time (Apologies for that.):

It introduces a new CLI command config which adds to utility functions for handling config files: cli.py config dump dumps the entire config after YAML processing and cli.py config list_pipelines returns a newline-separated list with all the defined pipelines.
It is adding a Docker image definition for running presubmit checks in a Docker container. The purpose of the Docker container is mainly to build the bigquery-emulator. (It's a golang-tool and needs to be built from source)
It is adding a GitHub Actions workflow that first builds the Docker image and then runs the unit and integration tests.
It is adding a dummy pipeline config which is needed to see whether the integration tests work.

But I think P0 is to provide users the digests to pull docker images from gcr. So if we don't solve it in this PR, makes sure we have an assigned issue to track the follow-up.

If it has such a high priority I think we should try to solve it immediately or at least have a plan before merging this.
If we wanted to use the "have the CI build the containers" approach that I'm proposing here I see the following options. (Of course it's also possible to just use the proven approach with prod_digests.txt):

Have also the user build their own docker images on demand. This would mean we have a script which calls docker buildx build instead of docker pull. Docker would then "build" the image but since we will have all the caching layers in the registry this will end up being the same image as if it was pulled. The only difference is if a layer is not in the cache - a docker pull would fail, but a docker buildx build would just build the layer. To me this is the preferred approach because the user will never have an outdated or non-matching docker image. docker buildx build will always fetch or build the proper version.
If we wanted to have tags the user can docker pull from, then one option would be to use commit hashes as image tags. In that case we would have a script that determines the commit hash from the git repo and looks up the corresponding docker image. This of course doesn't work if HEAD is not a commit on main, but we could fall back to the latest common ancestor commit between HEAD and origin/main.
Or we determine the latest commit that affected the docker image and use that. This has the disadvantage that now the script determining the tag to pull needs to know what could have affected the image - which is really not its job. But I see the biggest downside that the user needs to know when they should rebuild the Docker image because local non-committed changes won't affect the logic.

So what I'm basically asking is if solution 1 would be an option or if we need prebuilt images. :sweat_smile:
If we do then option 2 or 3 might be valid, but I would rather lean towards using prod_digests.txt for now and
maybe come back to the BuildKit based approach later.
WDYT?

But I think P0 is to provide users the digests to pull docker images from gcr. So if we don't solve it in this PR, makes sure we have an assigned issue to track the follow-up.

If it has such a high priority I think we should try to solve it immediately or at least have a plan before merging this.
If we wanted to use the "have the CI build the containers" approach that I'm proposing here I see the following options. (Of course it's also possible to just use the proven approach with prod_digests.txt):

Have also the user build their own docker images on demand. This would mean we have a script which calls docker buildx build instead of docker pull. Docker would then "build" the image but since we will have all the caching layers in the registry this will end up being the same image as if it was pulled. The only difference is if a layer is not in the cache - a docker pull would fail, but a docker buildx build would just build the layer. To me this is the preferred approach because the user will never have an outdated or non-matching docker image. docker buildx build will always fetch or build the proper version.
If we wanted to have tags the user can docker pull from, then one option would be to use commit hashes as image tags. In that case we would have a script that determines the commit hash from the git repo and looks up the corresponding docker image. This of course doesn't work if HEAD is not a commit on main, but we could fall back to the latest common ancestor commit between HEAD and origin/main.
Or we determine the latest commit that affected the docker image and use that. This has the disadvantage that now the script determining the tag to pull needs to know what could have affected the image - which is really not its job. But I see the biggest downside that the user needs to know when they should rebuild the Docker image because local non-committed changes won't affect the logic.

So what I'm basically asking is if solution 1 would be an option or if we need prebuilt images. sweat_smile If we do then option 2 or 3 might be valid, but I would rather lean towards using prod_digests.txt for now and maybe come back to the BuildKit based approach later.
WDYT?

But I think P0 is to provide users the digests to pull docker images from gcr. So if we don't solve it in this PR, makes sure we have an assigned issue to track the follow-up.

If it has such a high priority I think we should try to solve it immediately or at least have a plan before merging this.
If we wanted to use the "have the CI build the containers" approach that I'm proposing here I see the following options. (Of course it's also possible to just use the proven approach with prod_digests.txt):

Have also the user build their own docker images on demand. This would mean we have a script which calls docker buildx build instead of docker pull. Docker would then "build" the image but since we will have all the caching layers in the registry this will end up being the same image as if it was pulled. The only difference is if a layer is not in the cache - a docker pull would fail, but a docker buildx build would just build the layer. To me this is the preferred approach because the user will never have an outdated or non-matching docker image. docker buildx build will always fetch or build the proper version.
If we wanted to have tags the user can docker pull from, then one option would be to use commit hashes as image tags. In that case we would have a script that determines the commit hash from the git repo and looks up the corresponding docker image. This of course doesn't work if HEAD is not a commit on main, but we could fall back to the latest common ancestor commit between HEAD and origin/main.
Or we determine the latest commit that affected the docker image and use that. This has the disadvantage that now the script determining the tag to pull needs to know what could have affected the image - which is really not its job. But I see the biggest downside that the user needs to know when they should rebuild the Docker image because local non-committed changes won't affect the logic.

So what I'm basically asking is if solution 1 would be an option or if we need prebuilt images. sweat_smile If we do then option 2 or 3 might be valid, but I would rather lean towards using prod_digests.txt for now and maybe come back to the BuildKit based approach later.
WDYT?

I'm actually convinced solution 1 is a good idea, but there are a few things need to be done:

Create and use our own gcr as cache, as github action cache is limited
A wrapper script to help users run locally.
Figure out how to build dependency between dockerfiles (e.g. https://github.com/openxla/openxla-benchmark/blob/main/devtools/docker/dockerfiles/cuda11.8-cudnn8.9.Dockerfile#L9) if there is no digest reference from the gcr. It's anther major reason we introduced manage_images.py. Maybe https://www.docker.com/blog/dockerfiles-now-support-multiple-build-contexts/#:~:text=Create Build Pipelines by Linking bake Targets can be a solution.

So maybe let's use prod_digests.txt and we need more time to prepare for solution 1. (Also if you have time, could you also create an issue to switch to solution 1?)

LGTM, Thanks!


[Feature] Enables the use of ArrayType in synthesizer.

This PR adds ArrayType to PlaintextType, allowing it to be used in synthesizer and consequently in programs.
This PR also extends the cast operation to arrays.
Depends on #1778.

is mean i can use ArrayType in my leo program? if not when i can use?


Support for new bun.lock file

Build/Submit details page URL
https://expo.dev/accounts/mubin-memorang/projects/new-bun-lock/builds/9bdab95d-e384-4223-b82e-40ed9e6e7ffd
Summary
With Bun v1.1.39, Bun has introduced a new text based lockfile. This really helps our workflow during reviews to see what changed, and is also something that will be the new default with Bun 1.2 (this replaces the binary lockfile bun.lockb).
Right now, if you try to build the app with EAS using the new text based lockfile, EAS detects the project as yarn based, causing it to use yarn and create a lockfile from scratch.

This should not be the case, and it should also start detecting the bun.lock file as a bun project.
Managed or bare?
Managed
Environment
expo-env-info 1.2.1 environment info:
System:
OS: macOS 15.2
Shell: 5.9 - /bin/zsh
Binaries:
Node: 21.7.3 - ~/.nvm/versions/node/v21.7.3/bin/node
npm: 10.5.0 - ~/.nvm/versions/node/v21.7.3/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.16.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 24.2, iOS 18.2, macOS 15.2, tvOS 18.2, visionOS 2.2, watchOS 11.2
Android SDK:
API Levels: 34, 35
Build Tools: 33.0.1, 34.0.0, 35.0.0, 35.0.0
System Images: android-34 | Google APIs ARM 64 v8a, android-34 | Google Play ARM 64 v8a
IDEs:
Android Studio: 2024.1 AI-241.18034.62.2411.12071903
Xcode: 16.2/16C5032a - /usr/bin/xcodebuild
npmPackages:
expo: ~52.0.20 => 52.0.21
expo-router: ~4.0.14 => 4.0.14
react: 18.3.1 => 18.3.1
react-dom: 18.3.1 => 18.3.1
react-native: 0.76.5 => 0.76.5
react-native-web: ~0.19.13 => 0.19.13
npmGlobalPackages:
eas-cli: 14.2.0
Expo Workflow: managed
Enabled experimental React Native Directory checks. Unset the EXPO_DOCTOR_ENABLE_DIRECTORY_CHECK environment variable to disable this check.

✔ Check package.json for common issues
✔ Check Expo config for common issues
✔ Check native tooling versions
✔ Check if the project meets version requirements for submission to app stores
✔ Check for common project setup issues
✔ Check dependencies for packages that should not be installed directly
✔ Check for app config fields that may not be synced in a non-CNG project
✔ Check npm/ yarn versions
✔ Check for issues with Metro config
✔ Validate packages against React Native Directory package metadata
✔ Check Expo config (app.json/ app.config.js) schema
✔ Check that native modules do not use incompatible support packages
✔ Check for legacy global CLI installed locally
✔ Check that native modules use compatible support package versions for installed Expo SDK
✔ Check that packages match versions required by installed Expo SDK
Didn't find any issues with the project!
Error output
No response
Reproducible demo or steps to reproduce from a blank project

Clone https://github.com/shiroyasha9/expo-new-bun-lock (created using bun create expo-app and then migrated to use the new lockfile).
Change the projectId and owner in app.json to your credentials.
Run any of the build command e.g. eas build --platform ios --profile development-simulator.

Same issue here
Sample failed run: https://github.com/CodeWithShreyans/guider/actions/runs/12439445332


GPFS AFM Metrics.

Hi,
As agreed i have tried to come up with the metrics we need in order to get AFM in the grafana dashboard.
Im not sure which to choose excactly, maybe you have some information about that?
GPFSAFM, GPFSAFMFS, GPFSAFMFSET?
Which is best to use for monitoring AFM in grafana?
We usually use the command "mmafmctl  getstate" to see the state, que and filesets used for AFM. sometime we use mmfsadm dump afm to see more detailed information as it also displays split write, current files in que/inflight (per node) and much more.
My best guess would be these are required as a minimum, but not 100% sure:
gpfs_afm_queued
gpfs_afm_inflight
gpfs_afm_complete
gpfs_afm_errors
i think the above is a good pointer towards checking the health of AFM on the different filesets?
Here is my initial idea of metrics to use for basic performance monitoring:
gpfs_afm_bytes_read
gpfs_afm_bytes_written
gpfs_afm_ops_sync
gpfs_afm_ops_sent
gpfs_afm_bytes_pending
gpfs_afm_avg_time
gpfs_afm_tot_write_time
gpfs_afm_conn_broken
gpfs_afm_used_q_memory
gpfs_afm_num_queued_msgs
if you have some better ideas im open for it.. we just need to make sure that we can see the current state of AFM in grafana as well as monitor the que, performance etc..
/Andi Christiansen

Hi Helene,
Any updates on this? eta? anything? :)
Best Regards
Andi


Show recent task sequences on project level

With the started/finished events of task sequences - as proposed here - this information can be displayed in the project overview for each project.
Task:

Show the last 5 task sequences that were triggered for this project in the project tile

One sequence is identified by: icon, <service> in <stage>, sequence name + started/finished, x min/sec/hours/days ago the sequence was triggered.

Table with four columns: icon | service/stage | sequence | time
max five rows
ordered by the last column

The icon:

when the sequence is started but not finished -> show a loading icon
depending on the result, turn the icon color into red/green
be opinionated: support the icons for deployment (small box), evaluation (traffic light), remediation (escalation mark)

Build it as a re-usable component to be used in the environment screen (even though the environment screen is out of scope of this issue)

Definition of Done:

Currently we can't provide a "good" solution for this issue without touching the backend. In order to get the result finished, triggered etc. for each sequence we have to load all its traces.
Let's say we have 10 projects with 3 stages each. 10 x 5 sequences are loaded and then let's say with an average of 6 tasks (6 x 3 Traces) per stage we would load 10 x 5 x 3 x 6 x 3 = 2,700 traces and we would have 1 (get projects) + 10 (recent 5 sequences per project) + 10 x 5 (traces for each sequence) = 61 API-Calls.
This is going to lead to performance issues.
One possible solution would be, to save the result of a sequence in the sequence (e.g. a number determines loading, finished successfully, finished failed or waiting for approval). Currently the result is determined when we iterate through all traces of a sequence.


No longer PSR-0; updated Standards section

It is PSR-4 only now the Teapot directory has been removed.

There are some more friendly, published URLs for the PSRs, that it might be worth using, e.g. http://www.php-fig.org/psr/psr-4/

Ignore this - got into a bit of a mess :-(


#2752 Fix null value representation in ORC files

Description
Applying #2761  into 3.3.x branch.
Related Issue : #2752

run build
deploy to 5

run build
deploy to 5

Self test done.
These PRs will be merged in series, in a short period of time:
~~https://github.com/metatron-app/metatron-discovery/pull/2612~~
~~https://github.com/metatron-app/metatron-discovery/pull/2634~~
~~https://github.com/metatron-app/metatron-discovery/pull/2642~~
~~https://github.com/metatron-app/metatron-discovery/pull/2646~~
~~https://github.com/metatron-app/metatron-discovery/pull/2645~~
~~https://github.com/metatron-app/metatron-discovery/pull/2654~~
~~https://github.com/metatron-app/metatron-discovery/pull/2664~~
~~https://github.com/metatron-app/metatron-discovery/pull/2666~~
~~https://github.com/metatron-app/metatron-discovery/pull/2674~~
~~https://github.com/metatron-app/metatron-discovery/pull/2703~~
~~https://github.com/metatron-app/metatron-discovery/pull/2704~~
~~https://github.com/metatron-app/metatron-discovery/pull/2707~~
~~https://github.com/metatron-app/metatron-discovery/pull/2706~~
~~https://github.com/metatron-app/metatron-discovery/pull/2716~~
~~https://github.com/metatron-app/metatron-discovery/pull/2724~~
~~https://github.com/metatron-app/metatron-discovery/pull/2715~~
~~https://github.com/metatron-app/metatron-discovery/pull/2738~~
~~https://github.com/metatron-app/metatron-discovery/pull/2748~~
~~https://github.com/metatron-app/metatron-discovery/pull/2754~~
~~https://github.com/metatron-app/metatron-discovery/pull/2761~~


Testing tray-only (no window) app

How would I go about testing tray-only (no windows) app?
Every example I can find uses app.client.waitUntilWindowLoaded(), which is obviously no go with tray-only app. There is no such a thing as waitUntilTrayLoaded(), as well.
I am trying to make a simple test here: https://github.com/hovancik/strechly/pull/4/files but I am getting errors: https://travis-ci.org/hovancik/strechly/jobs/145827763 and no idea how to move.

I don't think it is possible at this time to test tray only apps using spectron.
Spectron is built on top of Chromedriver which tests renderer processes. Tray code runs in the main process and so it cannot be tested through Chromedriver unless a renderer process is available to run code in.

@kevinsawicki I see, thanks .
Anyway, meanwhile I was thinking about how to make a workaround and I made a small one:
I am showing startup window for 5 seconds and that way I am able to test the start of my app works here
I have a problem thought,I am not able to do:
 after(function () {
    if (app && app.isRunning()) {
       return app.stop()
     }
 })

As I am getting:
 strechly "after all" hook:
 RuntimeError: no such window: target window already closed
 from unknown error: web view not found
    at window() - application.js:206:19

Maybe you happen to know how to end app process? I wasn't able to find anything.

Calling app.stop() currently requires at least one created window (hidden or showing) since stopping the app routes through the renderer process.
This could probably be fixed though, I'll take a look.

It would be useful if this were mentioned in the docs.
app.start() never returns for apps that don't open a window on startup.


x axis labeling disappears when using twinx()

Code sample, a copy-pastable example if possible
A "Minimal, Complete and Verifiable Example" will make it much easier for maintainers to help you.
import proplot as plot
import numpy as np
import xarray as xr

times = np.arange(1990, 2000)
data = xr.DataArray(np.random.rand(len(times)), dims=['time'], coords=[times])

f, ax = plot.subplots(aspect=2)
ax.plot(data.time, data)

twinax = ax.twinx()

# twinax.format(xlocator=1)
ax.format(xticks=np.arange(1990, 2000, 2))

Actual result vs. expected result
Using twinx(), the tick labeling disappears on the twin axis. This happens for datetime and normal numbers. Using xticks/xlocator on the main axis or twinax instance changes the ticks but does not produce labels.

You can actually see this in the RC settings demo on the docs on the right subplot.
https://proplot.readthedocs.io/en/latest/basics.html#Changing-rc-settings

Which version are you using? Is this the "dev" release?

I'm using the 0.0.5 that's on pip now.

Actually this is intended behavior. Counterintuitively, the matplotlib twinx function generates two y axes with a shared "twin" x axes, and ProPlot ensures the shared axes ticks, tick labels, and axis labels are invisible so you don't accidentally draw multiple tick labels or labels from both "twins" on top of each other.
As you have demonstrated, this naming is really confusing for users, so proplot introduces the more intutively-named altx function for making an "alternate x axes" and alty for making an "alternate y axes".

@lukelbd I met the same problem today.
If I use the same example as @bradyrx showed above without twin, the xaxis tick labels are correct:

But, if I add the the twinx(), the tick labels disappeared:

This is the actual case I met: I want to plot two kinds of profiles with pressure, but I couldn't let the pressure ticks labels show up ...

Counterintuitively, the matplotlib twinx function generates two y axes with identical "twin" x axes, and ProPlot ensures the shared axes ticks, tick labels, and axis labels on the secondary x axis are hidden so you don't accidentally draw duplicate labels from both "twins" on top of each other. If you want to set the x axis properties, you should do so on the original axes.

Yes I understand this and agree with you that the naming is bad. I also agree that you shouldn't be able to modify the dual axis so you don't get out-of-phase ticks, clutter, etc.
The issue, as @zxdawn also mentioned, is that the original axis completely disappears. And in my example, you see that modifying the original axis does nothing to fix this (below with alty):
import proplot as plot
import numpy as np
import xarray as xr

times = np.arange(1990, 2000)
data = xr.DataArray(np.random.rand(len(times)), dims=['time'], coords=[times])

f, ax = plot.subplots(aspect=2)
ax.plot(data.time, data)

twinax = ax.alty()

# twinax.format(xlocator=1)
ax.format(xticks=np.arange(1990, 2000, 2))

I agree you shouldn't be able to modify the twin axis, but it shouldn't render the original axis unreadable. It's still important to see the ticks, time series labeling, etc.

My bad! I see now, this definitely is a major problem. I will address this before the end of the weekend.

Sorry for the wait guys; I had to step back from proplot for a while and knew this one would be tricky to fix. But it's finally done.

@lukelbd Thanks for the fix! Will test it for the O3 and Qv profile tomorrow.

This is great, thanks Luke!

Works well. Thank Luke!


Almost Completed Compile

Greetings @jake3991
First off, nice job here
Very impressed with your work

Was able to compile 14 out of 15 packages

It looks like the point cloud library (pcl) is giving me fits for catkin build
Running ros noetic / ubuntu 20.04
Followed the install of all your python3 and ROS dependencies using either apt-get or pip
Have you seen this before?
Could I have installed a dependency incorrectly?
here are a couple more pcl errors:

Thanks!
I have a few questions.

Can you post the error message at the top? The first one that occurs?
Did you clone these repos into the src folder?
clone https://github.com/ethz-asl/libnabo.git into your catkin workspace
clone https://github.com/ethz-asl/libpointmatcher.git into your catkin workspace
clone https://github.com/jake3991/Argonaut.git into your catkin workspace

Thanks for the quick response
So I did  clone all 4 github repos into the src folder
The error in the picture above is caused by missing logs of a missing logs directory
Was able to fix this in a cleaned catkin_ws by creating a directory called logs
"mkdir logs"
Still having a problem though
extended my terminal buffer alot to see the original error
Looks like the pcl library requires C++14
(See  screenshot below)
Any idea on how to fix this?

Oh interesting, maybe we can tweak the build a bit. Could you try these steps?

catkin clean
Change line 4 in this file to be as below https://github.com/jake3991/sonar-SLAM/blob/main/bruce_slam/CMakeLists.txt:

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++14 -O3")

Recompile

Changed the line 4 to reflect your suggestion above
No luck
Same error

I can't replicate the bug on my side but I did find a thread when searching for your error.
https://github.com/PointCloudLibrary/pcl/issues/2968
Let me know f you are able to fix it!

Thanks for the link and help above
will let you know what i find

In case it ends up being relevant, I'm using PCL version 1.10

Thanks for the info, might be useful
Didn't have any luck yesterday
Below is a little gem for you
It is my shell commands installing (almost) all the dependencies this repo requires
Let me know if I am installing any dependencies incorrectly
Feel free to put this in a bash file and attached it to this repo:

sudo apt -y update
sudo apt -y upgrade
sudo apt install -y python3-cv-bridge
sudo pip install -y gtsam
sudo pip install matplotlib
sudo apt install -y python3-message-filters
sudo apt install -y python3-numpy
sudo pip install opencv_python
sudo apt install -y ros-noetic-rosbag
sudo apt install -y ros-noetic-rospy
sudo pip install scikit_learn
sudo pip install scipy
sudo apt install -y ros-noetic-sensor-msgs
sudo pip install Shapely
sudo apt install -y ros-noetic-tf
sudo pip install tqdm
sudo pip install pyyaml
sudo apt install -y ros-noetic-pybind11-catkin
sudo pip install catkin-tools

I'm indeed using an NVIDIA graphics card. I also have an Intel processor. @pszenher have you seen this bug before?

I have the same issue and I solved this ,open CmakeLists.txt under bruce_slam,and add this
add_compile_options(-std=c++11 ) add_compile_options(-std=c++14 ) set(CMAKE_CXX_STANDARD 14)

Recommend to clean the build and devel folder first and then run catkin build


literal workaround

Since quotations '"table"."name"' is unconventional, the LLMs would hallucinate table names, i.e. drop quotations, and it would crash the Pydantic model, raising an invalid error. So, to workaround that, replaces everything except alphanumeric with _ which the LLM understands/copies well, then remaps it back to the original table name with all the special characters after the LLM chooses the table.

Superseded by https://github.com/holoviz/lumen/pull/799


Non-interactive mode for jfrog cli for use in Continuous Integration

We're using jfrog cli in the context a headless, non-interactive CI process.
Just recently we noticed that jfrog cli 1.33.1 had started to ask interactive questions, where it had not before. e.g.
$ jfrog rt config --url https://art.tools.company.us/artifactory --user <username> --password <password> artifactory
Is the Artifactory HTTP proxy configured to accept a client certificate? [no]:

We are working around this like so:
$ echo | jfrog rt config --url https://art.tools.company.us/artifactory --user <username> --password <password> artifactory
Is the Artifactory HTTP proxy configured to accept a client certificate? [no]:
$

Passing --interactive false as a parameter does not work on all permutations of the config command.  e.g.
$ jfrog rt config --url https://art.tools.company.us/artifactory --interactive false --user <username> --password <password> artifactory
...
...
[Error] Wrong number of arguments.

Observation:  It would be useful if setting the CI environment variable to "true" would cause jfrog cli to suppress user interaction and simply accept the default.

@tegataiprime and @jvalkeal,
@yahavi created https://github.com/jfrog/jfrog-cli/pull/591 to support a non-interactive mode for the various config commands.
We're trying to get this out quickly for you guys. Please follow https://github.com/jfrog/jfrog-cli/pull/591 for updates.

@tegataiprime and @jvalkeal,
Version 1.34.0 is released. With this release, all config commands can be used with flags (non-interactive mode).
Also, following your suggestion, when the CI environment variable is set to true, the CLI will run without prompts.
Looking forward to your feedback.


Initial implementation for the restore queue

Adds the estimated queue to the NAB restore object
Why the changes were made
To include estimated queue number for the restore object.
How to test the changes made
Create multiple NAR objects and observe if the queue is proper.

@mateusoliveira43 note the file where I've moved the common types. Let me know if that is fine, currently we were mixing backup types inside restore object: https://github.com/migtools/oadp-non-admin/pull/128/files#diff-bce3cc24a9a3be49f46a28f3a11baff2a1f07f5e735a1dbc683e535e1aaf625d

@mpryc it is fine
I would change name to something like common.go to avoid any possible conflict with kubebuilder in the future

Blocked by: https://github.com/openshift/oadp-operator/pull/1607

@mpryc avoid using make test and use make simulation-test, as described in our documentation https://github.com/migtools/oadp-non-admin/blob/master/docs/CONTRIBUTING.md#code-quality-and-standardization


fix: getError return type

This fixes the return type for the function since the supabase itself may return null and not a string

Hey @rafaelwendel do you think merging this one is a good idea?

Set on 0.0.8 version


Try also plotting cumulative bins starting from the poor-quality end -- this would tell us about the average metrics of bad variants being dropped

This would tell us about the average metrics of bad variants being dropped


condense to single line, since only one type of monomer is used

https://github.com/Fudenberg-Research-Group/dynamic_extrusion_boundaries/blob/f40351f44153c62f3bdd01f5f645a1654db18463/scripts/simulations/simulation_1d.py#L65-L66
typedict = {'A': 0}
monomer_types = np.zeros(monomers_per_replica, dtype=int)
site_types = np.repeat(monomer_types, sites_per_monomer)

could be:
site_types = np.zeros( monomers_per_replica * sites_per_monomer, dtype=int) # only considering one type of monomer

also can roll back from multiple types of CTCFs as we only use one type (https://github.com/Fudenberg-Research-Group/dynamic_extrusion_boundaries/blob/fc26672f12fec649d7084b4a6b73273a6c74e167/scripts/simulations/simulation_1d.py#L69)


Guests should still be able to try out the app without api

From what I recall, I believe guests are supposed to be able to use the app with a cap. If we decide we don't want to do this right now, we still need to have some kind of message that's presented in the case where a guest trys to submit a prompt without an api

Duplicate https://github.com/polywrap/evo.ninja/issues/573


return Page()

I think that the return Page(); which is being used if the model is not valid when creating or editing (a course or an instructor) is missing a key point at server side which is that the select menu and the checkboxes list will become empty when any of the OnPost actions is called.
Currently, the line return Page(); will result in a "NullReferenceException" because the populated fields (for example the DepartmentNameSL which is being populated by PopulateDepartmentsDropDownList at OnGet) are not longer populated.

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

ID: 140ce637-4d58-995f-c69e-b1de51f1dbb5
Version Independent ID: 9bc46ffa-60db-bdee-0984-fab183c8f503
Content: Razor Pages with EF Core in ASP.NET Core - Update Related Data - 7 of 8
Content Source: aspnetcore/data/ef-rp/update-related-data.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@eyadraschad  thanks, we'll fix this in the next doc update.

Is there a workaround for this issue ?

@edikaufmann yes, the workaround would be to call the methods that populate the select list and the list of checkboxes again OnPost (when the validation of the model fails) just like you did originally with OnGet. Because in this case this would be the same as making a new Get request.

The method is indeed already called, but the argument would seem to be incorrect. In the Create page, when saving to the DB fails, the code is:
PopulateAssignedCourseData(_context, newInstructor);
however, shouldn't newInstructor be replaced with Instructor, which is the model bound to the view? Similarly for Edit with instructorToUpdate, assuming my reasoning is correct.


Team 2 Blog 3

This is our blog for Domain Centered FM.

There are still some conflicts, can you please resolve them?


tab text is too small on 133

see screenshot above, this wasnt the case before

What operating system do you have?

What operating system do you have?

Manjaro Linux

I see. This is a forced measure. Some Linux OS use very large fonts. They do not fit into their boxes. The boxes themselves cannot be enlarged, since they do not fit into the tabs. There is nothing left to do but reduce the font size to the optimal size. If there were no media status panel, then the fonts would have enough space, but this way - no.

i see, could an option be made to disable this?

What do you think, maybe it’s easier to just disable the media panel in the tabs than to change the font size?

the media playing thing is useless IMO, the icon itself is enough, so thats ok by me

Okay, so be it.

Changes made. In the next version of the style, the font size will be set according to the operating system settings.

i tried this on nightly branch and it works, shape of the tabs was changed tho .__.

The next version of the style is quite different from the current one. You may have to reconfigure some keys.

ok, but it doesnt say that in the readme

Hi, what is the specific commit that caused this problem? I wasn't affected by the bug you mentioned and would prefer to have the old larger fonts. The workaround you provided in the other post didn't solve it to me.

Hi, what is the specific commit that caused this problem? I wasn't affected by the bug you mentioned and would prefer to have the old larger fonts. The workaround you provided in the other post didn't solve it to me. (edit: I just reverted to the 132 release. Still works fine for the most part)

The problem itself is not with the Firefox version, but with the Wavefox theme, as QNetITQ comments. I even noticed that in Firefox version 133 there was a new increase in the default size of the tabs. If you can, test with version WaveFox v1.6.128, in which I did not notice this problem with the font size but with the problem with the serifs as I pointed out in #216.

@ciro-mota I know, I was referring to the 132 release of the style, not of FF itself. 128 is also fine but it has some extra bugs not present in 132.

changing tab option to 11 gives me my beloved australis back @QNetITQ should i close the issue?

Let it remain open until the next version comes out with a fix.


T04b.02 - Fixed location is not showed in Google Maps app

changed "query(adressString)" to ".appendQueryParameter("q", addressString);"

Fixed.


Github Actions: logs visualization

The Github Actions plugin should be able to show logs for a specific workflow run.

Interested in helping on this one. Are there any specifics you had in mind? I see there being a call to the REST api to get the workflows and then assign them to a list of sorts ..... then when selected make a call with the associated ID to obtain the log for that run. Any concerns with that approach?

Thanks @NetPenguins -- I've assigned you. @Nek any further clarifications to provide?

@NetPenguins take a look at https://github.com/spotify/backstage/tree/master/plugins/circleci. There is a component for logs visualisation that can be a good fit for these logs too.

@NetPenguins I think starting simple is the best way to go.

@Nek Do you have a repo that has multiple jobs per run? It would also be helpful to have a job/run that has had a failure in it to test the styling around failures.

@Nek So I realized when I dug in that I had an very outdated version. So after upgrading I also noticed that there is now a Jobs element inside the WorkFlowRunDetails page. The only thing I can think to add that would create a complete log would be the steps per job. Is this what you were after in the initial request? If so I will add the nested steps under their respective jobs using the same formatted tree that displays the jobs.

@NetPenguins Actually, this sounds great! Please go forward with the changes.

@Nek I think this should be closed by #2172


Rk 162960214 tsp create sit request backend

Description
Adds end points for the Storage In Transit objects. This includes an Index (get a list based on the shipment), Get (retrieve a single SIT based on its ID), Create (create a SIT), and Patch (update a SIT). Doing this also required adding two functions to the model for SIT, one to retrieve a collection from the DB based on the shipment ID and another to retrieve a single SIT based on its own ID.
Setup
No special steps needed. Regular test suite can be used on these.
Code Review Verification Steps

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in
Querying the Database Safely
have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #dp3-engineering
[ ] Secure migrations have been tested using bin/run-prod-migrations

[ ] There are no aXe warnings for UI.
[ ] This works in IE.
Any new client dependencies (Google Analytics, hosted libraries, CDNs, etc) have been:

[ ] Communicated to @willowbl00
[ ] Added to the list of network dependencies

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Oops. I definitely forgot about the Delete Handler. Going to punch that up and some testing for it and push it in here asap.

Had a verbal code review with some folks and am working on the following points:

[ ] Adding in authorization to check for permissions on office/TSP use

[ ] Also adding in functionality to ensure the office is limited to its specific functional needs

Office user can update but not create or delete

[ ] Need to adjust the payload so that the Patch and Create end points can't create goofy or harmful statuses

A chunk of work for the first two points was included in the last push. Going to get the last one and some more testing set up and pushed out on the next one.


Unsuccessfully tried to modify server.ovpn to use ifconfig-pool and ifconfig-pool-persist to assign remote ip addresses

The standard server pivpn configuration file works.  I wanted to be revised the configuration file to define a specific set of IP addresses to be assigned to the client.
I made changes to the following to lines:
server
ifconfig-pool-persist <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>
I also tried
ifconfig-pool <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>
which replaced
server <IP_ADDRESS>  <IP_ADDRESS>
OpenVPN did not start up and had the following log
OpenVPN.log
Dec 31 10:33:53 EBJPi6 ovpn-server[274]: Options error: Unrecognized option or missing or extra parameter(s) in /etc/openvpn/server.conf:9: server (2.4.0)
Dec 31 10:33:53 EBJPi6 ovpn-server[274]: Use --help for more information.
server.ovpn  (MODIFIED)
dev tun
proto udp
port 1194
ca /etc/openvpn/easy-rsa/pki/ca.crt
cert /etc/openvpn/easy-rsa/pki/issued/server_VUtZIxxbdDMV4q9J.crt
key /etc/openvpn/easy-rsa/pki/private/server_VUtZIxxbdDMV4q9J.key
dh /etc/openvpn/easy-rsa/pki/dh2048.pem
topology subnet
server
ifconfig-pool-persist <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>
Set your primary domain name server address for clients
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
Prevent DNS leaks on Windows
push "block-outside-dns"
Override the Client default gateway by using <IP_ADDRESS>/1 and
<IP_ADDRESS>/1 rather than <IP_ADDRESS>/0. This has the benefit of
overriding but not wiping out the original default gateway.
push "redirect-gateway def1"
client-to-client
keepalive 1800 3600
remote-cert-tls client
tls-version-min 1.2
tls-auth /etc/openvpn/easy-rsa/pki/ta.key 0
cipher AES-256-CBC
auth SHA256
user nobody
group nogroup
persist-key
persist-tun
crl-verify /etc/openvpn/crl.pem
status /var/log/openvpn-status.log 20
status-version 3
syslog
verb 3
#DuplicateCNs allow access control on a less-granular, per user basis.
#Remove # if you will manage access by user instead of device.
#duplicate-cn
Generated for use by PiVPN.io
Have you searched for similar issues and solutions? Yes - nothing relevant was found
(yes/no / which issues?)

Console output of curl -L install.pivpn.io | bash
  Output Here

Console output of pivpn add or pivpn add nopass
  Output Here

we are moving all non-pivpn related issues over to pivpn subrredit at r/pivpn
if you still want some help, please reopen this there.
and please be more clear with what you are really trying to accomplish.
Kind Regards.


feat(dragula-model): events after model synchronisation

fix for issue #12

@luckylooke can you take a look at this?

Ok.. I will look at it soon.
On Sep 1, 2015 3:18 PM, "Nicolas Bevacqua"<EMAIL_ADDRESS>wrote:

@luckylooke https://github.com/luckylooke can you take a look at this?
—
Reply to this email directly or view it on GitHub
https://github.com/bevacqua/angular-dragula/pull/14#issuecomment-136716856
.

Thank you @DanielSchuech! :+1:


Add monthly post generation

Deployed and tested
Endpoint w/ Test Data

Maybe the one thing to consider changing before merging it in is figuring out the mojibake you noted, but also not really a blocker - can just file a GH issue to keep track of it.

I actually decided not to address it; some googling seems to suggest


[CAD-3723]  Add progress to initial chain selection

@nfrisby last commit (https://github.com/input-output-hk/ouroboros-network/pull/3518/commits/5c59ed7590da2a2f7df54f108a1eb0fe25f64132), is that what you were thinking of?

bors merge

Bah --- I just realized this was already Merged :)


Feature Request: Fold indicator options

Feature Requested
Thanks for the continuous work on this great theme.
I dug around in the style settings options and search but cannot find a way to turn off the (end of line) fold indicator.
It is rather distracting to me (used to be a "...", also I personally don't need it to be click-able. So please add an option to turn it off (or turn it back to "...")
A side request (on this topic) would be that the Collapse Arrow can have an option to only show when hover or in active line, even when folded. This is less important as I already have that in CSS.
Both of these request, I guess, are in the spirit of making the look very minimal while writing.
Thank you.
Relevant Screenshot
I want to turn this off.

Cannot find it

Relevance

[X] The feature would be useful to more users than just me.

Style Settings

[X] I have checked that the feature is not already available via the Style Settings plugin.

ah, how could I miss those 🤣
Hidden by default and setting to re-enable in latest release


WI creation: Cannot go back to title input field after leaving it empty and switching to description input field

Open the work item creation dialog and start typing something into the title field. Then remove all the text. (Alternatively, you can leave it empty altogether.)
Change to the description input field.
Notice, that you cannot go back to the title input field. (Not even with +)

See this screen capture for an example

@kwk , looks similar to :https://github.com/fabric8io/fabric8-planner/issues/740

@naina-verma Yes. This is a duplicate of #740 . Sorry for having searched the already opened issues.


Add identity property in managedCluster definition and add an example

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Is the CI broken? I run npx autorest --reset && npx autorest --validation --azure-validator --message-format=json specification/containerservice/resource-manager/readme.md locally and no error is reported.

Hi @xizhamsft , could you please help have a review?

Closing then re-open to fix CI.

The CI keeps failed, and the error message seems to be unrelated to this PR. Anything I can do to fix this?  @jhendrixMSFT

Close and reopen to fix CI.

@norshtein I think your branches are out-of-date.  Can you please rebase your PR on top of master and update the target branch as well?

@norshtein I think your branches are out-of-date. Can you please rebase your PR on top of master and update the target branch as well?

CC @xizhamsft . Could you please rebase branch Azure:dev-containerservice-Microsoft.ContainerService-2019-04-01 upon master? I can then rebase this PR upon dev branch.

/LGTM.
@norshtein, could you try rebase your fork and update the PR? I don't own the dev branch. It will be the same process of rebased and then PR. Let me know if that does not work. Thanks.

Target branch has been rebased.

Rebased upon updated target branch.


Setup continuous integration with https://travis-ci.org

https://travis-ci.org

The build status can be found on [https://travis-ci.org/crawler-commons/crawler-commons]. The alerts are sent by email to the projects mailing list.


[Future Request] Update to 1.19.2

I 2nd this request. This mod been very helpful since Tc3 changed a few things


Kusto Notebook - Changing current Cluster connection to a connection with database specified does not work.

Azure Data Studio Version:
Version: 1.23.0-rc1 (user setup)
Commit:<PHONE_NUMBER>af2e08d17a8928825815cffad94f17
Date: 2020-10-09T00:55:31.875Z
VS Code: 1.49.0
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Steps to Reproduce:

Create a new Kusto notebook, connecting to an ADX Cluster.
Click on Change Connection and connect to a Database.

Actual: it doesn't change the connection.

Expected: it changes connection to connect to a database. And the database should have been listed too.
Note:

This wasn't so much of a problem before because the default first db was selected.
Changing from ClusterA.DB1 to ClusterB.DB 2 works fine. :)

Looks like this is specific to notebooks. Issue does not happen for Queries.

After investigations on the Notebook side, we found that the connectionProfile gets updated with the new database as a result. The code cell itself should update to the new connection, this also does not happen with SQL and is not a regression. So I will assign @JustinMDotNet to understand if this fix should be done in SQLToolsService for Kusto default database edge case to work properly. (can be moved to December, but would like feedback from Livesite Tooling team).


DOJO(mentor/page): List mentor submitted exercises

This page will list all the mentor's submitted exercises, with the ability to edit and delete them.
Here's the page mock
The page's states:

When a new question is added
When a question is deleted

Here are the components that you'll have to make before building the page:

Mentor problem card
Filter by module dropdown (could be generic)

Here's the filter by module dropdown onClick event design

There are a few things missing from the Mentor problem card:

Top 3 wrong answers
Correct out of total ratio
The card design on Edit state

We already created a card that has these things (https://www.figma.com/file/f9xRyl1bJOjtBRd8zFORzg/c0d3?node-id=311%3A2314) but for some reason a new one that doesn't have them was created in the last design meeting. Feel free to discuss it with the engineers or the designer.
Notes:

@SlyBouhafs is working on #1975 which is a subpage for this page.

This is a good idea.

@JasirZaeem Hi, are you working on this?

Things to add:

Create a query to get all the mentor's created exercises based on the user ID
Modify the exercise card to not show ANSWERED or NOT ANSWERED states
Add a back button to the addExercise page

Make NavBar references DRY for the curriculum pages.

Add filtering box
Add edit/delete functionalities

[ ] Add filtering box
[ ] Add edit/delete functionalities


Improve Footer UI/UX: Enhance Social Media Icons, Section Alignment, and Button Design

The current footer has several areas that could be improved to enhance both visual appeal and user experience:
Social Media Icons:
Realign and adjust spacing between the social media icons.
Improve icon contrast to ensure they are more visible against the dark background.
Optionally, add hover effects for better interaction.
Section Spacing & Alignment:
Add more space between the "Explore," "Policy," "Address," and "Courses" sections for better readability.
Ensure proper alignment so that the footer looks balanced and structured.
"Back to Top" Button:
Redesign the “Back to Top” button to match the overall style of the footer.
Consider making the button less intrusive by changing its shape or placement.
This issue aims to improve the overall usability and aesthetic appeal of the footer to enhance user interaction and site cohesiveness.

@vishanurag please assign this issue


Release: v0.2.0 x/eventlog/v0.2.0 x/grpcerror/v0.2.0 x/htmx/v0.2.0 x/testlog/v0.2.0

#88  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jespert and the rest of your teammates on  Graphite

Merge activity

Jun 10, 3:16 PM EDT: @jespert merged this pull request with Graphite.


Update develop after patch release

Summary
This updates the develop branch with the changes merged to master for the patch release which were https://github.com/stan-dev/math/pull/2365 https://github.com/stan-dev/math/pull/2366 and https://github.com/stan-dev/math/pull/2371

@serban-nicusor-toptal did this as a PR just so we have everything triple checked.

@rok-cesnovar I've tested this and it looks good to me. One more thing: the version number, release notes, and tags need to be updated on https://github.com/stan-dev/stan/tree/develop after v2.26.1.

Oh, right, we need to do that as well. Thanks!

https://github.com/stan-dev/cmdstan/pull/985 and https://github.com/stan-dev/stan/pull/3007 will handle that.


Incorrect reference to ISS Webserver

[Enter feedback here]
Hi,
Just before "Clean up Resources" there is a paragraph: "To see the load balancer distribute traffic across all three VMs, you can customize the default page of each VM's IIS Web server and then force-refresh your web browser from the client machine."
This paragraph references to a IIS Web server. I am sure this is impossible as Ubuntu Linux VMs with a NGINX Webserver is deployed in this tutorial.
Perhaps a little check and update to take away any confusion for the readers.
Cheers,
Marco

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

ID: 86a6a27a-c9a2-e7da-52dc-d5b50f18e488
Version Independent ID: ee34b3c4-2860-fd58-b89c-dddc31d45a19
Content: Quickstart: Create an internal load balancer - Azure CLI - Azure Load Balancer
Content Source: articles/load-balancer/quickstart-load-balancer-standard-internal-cli.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

@MTijbout, Thanks for reaching out. We will investigate and update this thread.

@MTijbout, Thanks for reaching out. We will investigate and update this thread.

@MTijbout, the doc has been updated accordingly. It would take some time for the changes to get reflect over the doc.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

@MTijbout, the doc has been updated accordingly. It would take some time for the changes to get reflect over the doc.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


File response with webserver sending

https://www.yiiframework.com/doc/guide/2.0/en/runtime-responses#sending-files
https://www.yiiframework.com/doc/api/2.0/yii-web-response#xSendFile()-detail

Q
A

Version
3.0.x-dev

PHP version
-

Operating system
-

https://github.com/yiisoft/http#contentdispositionheader-usage


Add merge notice event

Pull Request Test Coverage Report for Build 23

13 of 13 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.8%) to 93.467%

Totals

Change from base Build 21:
0.8%

Covered Lines:
132

Relevant Lines:
143

💛  - Coveralls


Watch mesh config in ingress controller

Fixes: #24110
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @jronak. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jronak: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
6901cc227e3b11a611458d677d6f1379a9eb61ce
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-distroless-k8s-tests_istio


Examples: tlshelloworld.c, tls_read or recv?

Greetings!
I was going through the examples and I noticed that the tlshelloworld.c does not make use of the tls_read nor tls_write, should these be used to populate the application buffer or is the tls consume stream enough?
Thanks!

Hello!
tls_read and tls_write work on the TLS buffer itself. tls_consume_stream writes the protocol bytes to the internal TLS buffer (after deciphering, validating and authorizing). So, we should have: recv => tls_consume_stream and then tls_read. Same way we use tls_write to write actual payload data. Then use tls_get_write_buffer to get the actual TLS bytes to write on the socket (or any other medium).
It should be also safe to use tls_write while calling tls_cosnume_stream (reading).
SSL_read and SSL_write are performing both socket I/O and TLS protocol I/O using recv, tls_read, send and tls_write.
Thank you!


Starts from this year -- 2024 -- detect can never detect any alternative languages

The detect method for Azure Translator cannot detect more than one language starting from 2024, for any combination inputs, it can always detect only one language with no return for any alternatives in response body.
Before 2024, it can detect at max two languages, though it should detect all languages based on documentation.
We try to understand what should be expected outputs and what is the plan to fix the current issue.

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

ID: 43893926-c1b5-5794-d1d2-0c0e0a558664
Version Independent ID: 9546a20a-c5b2-6f01-5847-8adcabd14050
Content: Translator Detect Method - Azure AI services
Content Source: articles/ai-services/translator/reference/v3-0-detect.md
Service: azure-ai-translator
GitHub Login: @laujan
Microsoft Alias: lajanuar

@zhufamily
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!

@zhufamily
Could you please add a link to the documentation you are following for these steps?

It is right here
https://learn.microsoft.com/en-us/azure/ai-services/translator/reference/v3-0-detect
and here
https://learn.microsoft.com/en-us/rest/api/cognitiveservices/translator/translator/detect?view=rest-cognitiveservices-translator-v3.0&tabs=HTTP
You can change the input text to multiple languages, but it cannot detect any alternatives since year 2024.

I think you misunderstood the point, documentation is OK.  It must be some glitch in the new release, which leads to no longer detect the alternative languages, we were using alternative languages perfectly fine before 2024.  Then starting from this year, the detect service can no longer detect more than one language.

@zhufamily
Thank you for bringing this to our attention.
I've delegated this to content author @laujan, who will review it and offer their insightful opinions.

Hi @zhufamily, Thank you for your feedback.  The method was returning multiple detected languages, but not all resulted in a high confidence score. Now the service returns only the highest scored detected language, which seems to work well for most scenarios. Thanks again!
#please-close


Implementation

This section describes the structure of and rationale un- derlying Spanner’s implementation. It then describes the directory abstraction, which is used to manage replica- tion and locality, and is the unit of data movement. Fi- nally, it describes our data model, why Spanner looks like a relational database instead of a key-value store, and how applications can control data locality.
A Spanner deployment is called a universe. Given that Spanner manages data globally, there will be only a handful of running universes. We currently run a test/playground universe, a development/production uni- verse, and a production-only universe.
Spanner is organized as a set of zones, where each zone is the rough analog of a deployment of Bigtable servers [9]. Zones are the unit of administrative deploy- ment. The set of zones is also the set of locations across which data can be replicated. Zones can be added to or removed from a running system as new datacenters are brought into service and old ones are turned off, respec- tively. Zones are also the unit of physical isolation: there may be one or more zones in a datacenter, for example, if different applications’ data must be partitioned across different sets of servers in the same datacenter.

@ngaut


Fix Left join implementation is incorrect for 0 or multiple batches on the right side

Which issue does this PR close?
Closes #235
Closes #239
Closes #143
Rationale for this change
Fixes behavior of left join with regard to multiple batches on the right side and 0 right side batches.
This way it also also likely much faster as it avoids keeping generating / keeping / indexing into a HashSet for each right side batch.
What changes are included in this PR?

We add a Vec<bool> to keep track of left-side rows that didn't match with the right side. (number o left rows bytes extra memory usage for left joins, not that much compared to storing the left side data and hashmap + indices. It could be more memory-efficient by using a bitmap instead.
Generate rows for any item that is not marked as visited.
Add some tests for 0 / multiple batches on the right

Are there any user-facing changes?
Only correctness improvements

Codecov Report

Merging #238 (3d7a01b) into master (c945b03) will increase coverage by 0.04%.
The diff coverage is 98.41%.

@@            Coverage Diff             @@
##           master     #238      +/-   ##
==========================================
+ Coverage   76.46%   76.50%   +0.04%
==========================================
  Files         135      135
  Lines       23250    23303      +53
==========================================
+ Hits        17777    17829      +52
- Misses       5473     5474       +1

Impacted Files
Coverage Δ

datafusion/src/physical_plan/hash_utils.rs
100.00% <ø> (ø)

datafusion/tests/sql.rs
99.88% <ø> (ø)

datafusion/src/physical_plan/hash_join.rs
87.09% <98.41%> (+1.15%)
:arrow_up:

Continue to review full report at Codecov.

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

Merging this when it's green


feat: add invocation trace events.

Add invocation (actions and expectations) started & ended events, along with their stack trace.
As addressed in #3536.

@jonathanmos I made some changes, most of them as agreed f2f, please review 🙏🏼 🙂

Squashed commits with meaningful commit messages.


Select by day of week

It would be nice if the query language supported relative dates.
Example. How many data points have there been during this work week?
SELECT * FROM foo WHERE time > DAYOFWEEK(sunday)

or
SELECT * FROM foo WHERE time > DAYOFWEEK(1)

or
' SELECT * FROM foo WHERE time > DAYOFWEEK(1)'

I disagree, just because Sunday is not the first day of the week in every country in the world.
But I support an idea very similar to yours.
SELECT * FROM foo WHERE time = week(26)

Return every records from within the duration of the 26th week of the current year.

This implies regional settings somewhere (to take care of the first day of the week between regions). The server level is too high for that, it may be better at db or series level.
I'm looking forward to see the team include several functions related to dates in the new versions of InfluxDB.

Hi,
stumble upon a case where i wanted to group by day of week...
having a day of week if definitely a good idea has we try to work with time database :)
it can be super useful in cases like having a graph of values by day of weeks or filter by day of weeks to detect a pattern, ...:

[Monday: 12, Thuesday: 11 ....]
[Monday 05/10/2015,Monday 12/10/2015,.... ]

I recently had a requirement similar to this. I wanted to compare timeseries from different days, aligned by the same time of day. The workaround I ended up with was rebasing the data I wanted to a common base timestamp and then adding the original date as a tag. This then allowed me to visualise the different days in the same graph within grafana.

Related #3290

Closing as a duplicate of influxdata/influxdb#2071.


bump dependencies

New feature of Gradle Kotlin DSL (= operator for Property<T> assignment) has some IDE integration problems (link).

TODO: Fix RoboVM version being specified in like... 3 different places?


[WIP] First take on adding namespace support to vault. Aim to close #2806

Description
Please explain the changes you've made
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@tmacam are you planning to finishing this for 1.12? What else is needed here?

@tmacam are you planning to finishing this for 1.12? What else is needed here?

No. As I mentioned on my comment on https://github.com/dapr/components-contrib/issues/2806#issuecomment-1535460992 there is a way to use enginePath to provide the same functionality as namespaces with the advantage we can test enginePath with Vault OSS whereas namespace isn't. Let me drop this PR.


Fixed few typos

isRefres -> isRefresh
isRefesh -> isRefresh

Oh god. THanks!

Too many Ref's exception 😄


How to reformat my code to 80 character limit per line?

If I have understood correctly, I think you need a code formatter such as prettier rather than a linter such as StandardJS.  Unfortunately  prettier is not (yet) 100% compatible with StandardJS, though you could check out some of the hybrids such as prettier-standard ...

I don't think its a standard violation to have lines longer than 80.  There is --fix flag included with standard that will fix a lot of linting bugs for you.  I don't think --fix does line lengths though.  Prettier  will though, but you are venturing into multiple ruleset territory, which can be a nightmare to get working in general though.  I've heard good things about prettier though.  I can't testify to its use with standard though.

Closing.  Please let me know if there is anything undressed and we can re-open.


fix: revert "Avoid creating new node processes by leveraging processAsync"

Reverts mihar-22/svelte-jester#57

Should be major bump as it requires Jest >=27.


Preferences not present in admin panel after flush or multiple selenium tests

Hello,
I ran into a problem when running selenium tests. My preferences are gone when running more than one test case.
This happens with both pytest and python manage.py test. While the first test case test_preferences_present passes, the same exact test case test_same_thing_again fails, because the preferences are gone.
tests.py:
from django.contrib.auth.models import User

from django.contrib.staticfiles.testing import StaticLiveServerTestCase
from selenium.webdriver.firefox.webdriver import WebDriver

from dynamic_preferences.registries import global_preferences_registry

global_preferences = global_preferences_registry.manager()

class MySeleniumTests(StaticLiveServerTestCase):
    def setUp(self):
        self.selenium = WebDriver()
        self.selenium.implicitly_wait(10)
        self.user = User.objects.create_superuser("admin", "", "supersecurepass")

    def tearDown(self):
        self.selenium.quit()

    def test_preferences_present(self):
        assert global_preferences['my_preference'] is True
        self.selenium.get('%s%s' % (self.live_server_url, '/admin/'))
        username_input = self.selenium.find_element_by_id("id_username")
        username_input.send_keys('admin')
        password_input = self.selenium.find_element_by_id("id_password")
        password_input.send_keys('supersecurepass')
        self.selenium.find_element_by_xpath('//input[@value="Log in"]').click()
        self.selenium.find_element_by_link_text("Global preferences").click()
        self.selenium.find_element_by_link_text("my_preference")

    def test_same_thing_again(self):
        assert global_preferences['my_preference'] is True
        self.selenium.get('%s%s' % (self.live_server_url, '/admin/'))
        username_input = self.selenium.find_element_by_id("id_username")
        username_input.send_keys('admin')
        password_input = self.selenium.find_element_by_id("id_password")
        password_input.send_keys('supersecurepass')
        self.selenium.find_element_by_xpath('//input[@value="Log in"]').click()
        self.selenium.find_element_by_link_text("Global preferences").click()
        self.selenium.find_element_by_link_text("my_preference")

dynamic_preferences_registry.py:
from dynamic_preferences.registries import global_preferences_registry
from dynamic_preferences.types import BooleanPreference

@global_preferences_registry.register
class MyPreference(BooleanPreference):
    name = 'my_preference'
    default = True

(venv) C:\Users\NoneOfYourBusiness\workspace\django\bla\bla>python manage.py test
Creating test database for alias 'default'...
System check identified no issues (0 silenced).
.E
======================================================================
ERROR: test_same_thing_again (exampleapp.tests.MySeleniumTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Users\NoneOfYourBusiness\workspace\django\bla\bla\exampleapp\tests.py", line 40, in test_same_thing_again
    self.selenium.find_element_by_link_text("my_preference")
  File "C:\Users\NoneOfYourBusiness\workspace\django\bla\bla\venv\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 427, in find_element_by_link_text
    return self.find_element(by=By.LINK_TEXT, value=link_text)
  File "C:\Users\NoneOfYourBusiness\workspace\django\bla\bla\venv\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 966, in find_element
    'value': value})['value']
  File "C:\Users\NoneOfYourBusiness\workspace\django\bla\bla\venv\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 320, in execute
    self.error_handler.check_response(response)
  File "C:\Users\NoneOfYourBusiness\workspace\django\bla\bla\venv\lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: Unable to locate element: my_preference

----------------------------------------------------------------------
Ran 2 tests in 26.012s

FAILED (errors=1)
Destroying test database for alias 'default'...

Strangely the assertion
assert global_preferences['my_preference'] is True

passes in both cases, so the preference should be at least somewhere present.
Same problem occurs when you do the following:

do the things as described in the installation instructions
go to admin panel and click on "Global preferences"
my_preference is present
do: python manage.py flush
do same steps again from above again:

go to admin panel and click on "Global preferences"
my_preference is not present

When checking the db.sqlite3 file manually, then the dynamic_preferences_globalpreferencemodal is completely empty though.
I am not sure if I forgot to do something or if this is indeed a bug. If you need more information (like an example project) I am happy to provide these.

It's probably because the test runner flushes the database between each tests, and preferences are not recreated automatically until you actually access them. And preferences are cached as well, meaning assert global_preferences['my_preference'] is True will only query the cache.
I suggest you flush your cache as well as part of your test setup (cf https://github.com/EliotBerriot/django-dynamic-preferences/blob/develop/tests/test_manager.py#L16)
It should be good after that ;)

Another way would be to assign a value to the preference: global_preferences['my_preference'] = True would effectively create or update the pref in database, as well as cleaning the cache. But cleaning the cache between tests is good practice anyway ;)

Thank you for the quick response, clearing the cache did indeed fix the problem!
    def tearDown(self):
        caches['default'].clear()

Is this something that we should include in the documentation?


Package request: svlangserver

Project description
A language server for systemverilog that has been tested to work with coc.nvim, VSCode, Sublime Text 4, emacs, and Neovim

It is a language server for verilog and system verilog.
Metadata

homepage URL:  https://github.com/imc-trading/svlangserver
source URL: https://github.com/imc-trading/svlangserver
license: MIT
platforms: unix, linux, darwin, windows

Add a :+1: reaction to issues you find important.

If you (or anyone else) want, you can start a PR from my svlangserver Nix expression. I'm not sending the PR myself, because I don't work with Verilog often and I'm not sure, I'd be motivated enough to maintain it properly in Nixpkgs.

Also, for my (simple) use case, I found Veridian more useful than svlangserver. My Nix packaging of Veridian is available here.

I did try your svlangserver nix expression however it for some reason it didn't work with Verilator as linter on Helix editor. I am sure that's on me messing up my LSP config. Since then I have also switched to Veridian thanks to your config. It worked perfectly.
I am using Diagnostic Language Server for the Verilator Linter and Veridian for the rest.
I do want to contribute a couple of packages to Nix. However, I am still learning and I do not want to waste the time of reviewers by having them read my garbage code.


Need a service that reads the number of an inventory item

As a order managing specialist
I need to inspect if an item of a  requested order is available or not
So that this requested order can be filled
Details and Assumptions

use PostgreSQL to store item information, use generated UNIQUE item ID as KEY  for indexing.

Acceptance Criteria
Given [some context]
When [certain action is taken]
Then [the outcome of an action is observed]

Hi Jianze, is your story  related to the 'read a resource' function, if so I think rename the story as 'Read an inventory item' might be more clear. @benno0810

the number of an inventory and availability of inventory sometimes have different meanings and need to create 2 services to the requests respectively.
E.g specialists send an order containing:
{A:10, B:11}
we reply:
{A:9, B:11}
he will need to calculate by himself at the front end or on the client.
is it reasonable or possible to happen? @wsrong97

As stated in the homework, our instructor suggests at least what our microservices would be able to do.
The primary requirement for creating these microservices is that they must be RESTful
according to the guidelines that we will learn in class. Initially each microservices must
support the complete Create, Read, Update, & Delete (CRUD) lifecycle calls plus List (5
API calls in all).
So apart from these services, we can create more based on need.

@wsrong97 it comes to me that we might not know the details inside an order because we don't have access to the order database or order cache, in this case, a query to us should only contain some separate item IDs, so the status for an order is not in our hands so far.

@wsrong97 it comes to me that we might not know the details inside an order because we don't have access to the order database or order cache, in this case, a query to us should only contain some separate item IDs, so the status for an order is not in our hands so far (but the inventory information we must handle).

I guess so,  I don't know the entire design of this eCommerce services. I think in reality, different microservices must rely on each other to provide higher level functions. But how to interact with services that other squad groups are aimed to provide, is outside the aim of our squad.


Ability to change MQTT Topic.

Feature request

What problem does this feature solve?
The issue is that if I have multi HyperHDR server in my network connected to the same MQTT Server they will all share the same topic HyperHDR/JsonAPI, trigger an action will trigger all HyperHDR servers

More FR tho it's not so important, It will be even more useful I can specify the instance index through the topic,
HyperHDR/0/JsonAPI # Instance 0
HyperHDR/1/JsonAPI # Instance 1

What does the proposed API look like?
How should this be implemented in your opinion?
Add a textbox in MQTT Config tab to modify main topic
{var_mqtt_topic}/JsonAPI
{var_mqtt_topic}/JsonAPI/response

Are you willing to work on this yourself?
Haven't code c++ before but I can help if needed

PR implementing custom MQTT topic: https://github.com/awawa-dev/HyperHDR/pull/620
Temporarily installers are available here for a while:: https://github.com/awawa-dev/HyperHDR/actions/runs/5813337111
Please test it and let me know if it works OK. I think we leave the direct instance channel for HyperHDR v21 as there is an option to switch instances using JsonAPI.

I just tested it and it works perfectly good .
Thank you.


JTextField uneditable when JCEF present, until JFrame loses focus

This is the weirdest thing. The JFrame contains:

a JPanel with form fields (among which are some JTextFields)
and a JInternalFrame that contains the JCEF component

The JCEF component only fully initializes when the JFrame becomes visible.
And the JTextFields are then uneditable, unless the JFrame loses and regains focus (click outside of the JFrame and then inside again, or close and open seeing that I don't fully dispose them, they keep their state when I close them).
So we need to find a way to make sure the JTextFields are editable when opening a frame with a JCEF component for the first time. Maybe a callback on JCEF initialized that makes the JFrame lose and regain focus automatically?
I opened a stackoverflow post here: https://stackoverflow.com/q/64622736/394921

fixed in commit aaf0e5ea7f8451d901377696c61c75b8e7fb0565


False Positive - SSH Public Key

Pattern Name / Description

The pattern picked up itself.
Sample Snippet

ssh-rsa (.*|\n) [\S]*@?[\S]*

https://github.com/advanced-security/secret-scanning-custom-patterns/security/secret-scanning/423

[optional] Propose Solution

[suggestion]

#19


Ability to render pymd 'show' tables to Markdown .

Another "I don't know why this build is failing" issue:
https://travis-ci.org/locationtech/rasterframes/builds/571611437#L4212-L4215

Do we need to add pweave to test_requires because of the import on line 50 in setup.py

Another "I don't know why this build is failing" issue:
https://travis-ci.org/locationtech/rasterframes/builds/571611437#L4212-L4215

☝  trying to fix the build / unit tests

OK that oughta do it


Pointer:setActive does not work

As the title says: Pointer:setActive does not work. It does set the state within the Pointer, and notifies Scene of the change, but the state is completely ignored for the overlap check.

Fixed by 447a270


Can't call view functions in contracts

Environment information
latest main branch
no plugins

Python Version: 3.9.7
OS: macOS

What went wrong?
The command I am running:
ape run get_my_number --network ethereum:development:http

get_my_number.py:
from ape import accounts
from ape import project

def main():
    test_account = accounts.load("ganache_0")
    contract_type = project.MyNumber
    contract = test_account.deploy(contract_type)
    my_num = contract.number()
    print(my_num)

The error output:
ERROR: Transaction not signed!

How can it be fixed?
I don't know yet but am looking into and will put back what I find here.

The bug was that we didn't need to encode the txn when using eth_call.
Also, we needed to set the chain id on the txn in the contract.__call__ method.