Adding a separate client-server connection for different components

I want to collect some of the env observations during RL at a different frequency compared to the actor. I plan to make it a separate process that connects to the actor via Agentlace and zmq, and passes in the latest observation (not a previously queued up obs). I am using the TrainerServer and TrainerClient setup from SERL, such as in this file.
What is the recommended way to achieve this? Is it best to use ActionServer as an EnvServer? Should this be on top of the TrainerClient that's already in the actor?

Hi @gautams3
if i understand correctly, you would like to send custom data from the client (actor) --> server (learner) besides using the queued datastore/replaybuffer. In this case, you can use this client.request() from the actor, and a request_callback fn from the server to receive the msg.
Let me know if this works

This can be implemented as a separate thread, periodic callback timer etc, then use the request() api to send the data to the learner node.


slang-test: retry failed test at the end.

We just retry it one more time, after all other tests are run, in serial.


Issue about model_config in mmdet

Hi, thank you for your great work!
I want to konw which model_config file in which version of mmdet used here when evaluating GenEval? Considering mmdet==2.27.0 needs to be installed, and 'configs/mask2former/mask2former_swin-s-p4-w7-224_lsj_8x2_50e_coco.py' is specified in code, I download the configs folder from mmdet v2.27.0, and set  the configs/mask2former/mask2former_swin-s-p4-w7-224_lsj_8x2_50e_coco.py as  args.model_config.
But too many weight download from download_models.sh can not match the model initialized based on the args.model_config, and all the final metrics are 0. I think this is because I don't use the correct model_config?
Best,
Fan Lu

Hopefully the problem is fixed by the solution proposed in #8 (the PR is also merged, so pulling the latest commit and re-installing should work as well).


Debugger not working for files loaded over ssh in the extension development host

Hi,
when debugging a VS Code Extension and launching the Extension Development Host for a file that is loaded over ssh, the debugger is not working.

VS Code Version: 1.76.0
OS Version: Darwin arm64

Steps to Reproduce:

Open a sample jasmine extension for development purposes such as the sample one: https://code.visualstudio.com/api/get-started/your-first-extension#:~:text=Debugging the extension,to check a variable's value.
Start debugging your extension
VS Code starts in a new window over the Extension Development Host
Go to remote explorer
Open any file over ssh
Run the extension

Actual

breakpoints are ignored

Expected

I can debug an extension for files loaded over ssh

The issue is VS Code Version independent.

What is a "sample jasmine extension"

To debug an extension running on the remote, you'll want to have the debugger running on the remote as well. That is, clone and open your extension in a directory on the remote, and then F5 from there. The new window will be connected to the remote, and you can run and debug it at that point.

To debug an extension running on the remote, you'll want to have the debugger running on the remote as well. That is, clone and open your extension in a directory on the remote, and then F5 from there. The new window will be connected to the remote, and you can run and debug it at that point.

That sounds like the missing part, I will try it, thanks!

Solution confirmed.


Upgrade openapi-generator version in python client-gen script

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #3148
Type of changes
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[x] Test A

What's the InferenceService yaml:
[To help us debug please run kubectl get isvc $name -n $namespace -oyaml and paste the output]
Commands Run :

Complete Output :
apiVersion: serving.kserve.io/v1beta1
kind: InferenceService
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"serving.kserve.io/v1beta1","kind":"InferenceService","metadata":{"annotations":{},"name":"sklearn-iris-example","namespace":"test-ns"},"spec":{"predictor":{"minReplicas":1,"sklearn":{"name":"sklearn-iris-predictor","protocolVersion":"v1","resources":{"limits":{"cpu":"100m","memory":"512Mi"},"requests":{"cpu":"100m","memory":"512Mi"}}}}}}  creationTimestamp: "2023-10-26T19:19:02Z"
  finalizers:
  - inferenceservice.finalizers
  generation: 1
  name: sklearn-iris-example
  namespace: test-ns
  resourceVersion: "1335223"
  uid: 1b4fdb36-639b-464f-bcb9-f92a7e5d8efd
spec:
  predictor:
    minReplicas: 1
    model:
      modelFormat:
        name: sklearn
      name: sklearn-iris-predictor
      protocolVersion: v1
      resources:
        limits:
          cpu: 100m
          memory: 512Mi
        requests:
          cpu: 100m
          memory: 512Mi
status:
  components:
    predictor:
      latestCreatedRevision: sklearn-iris-example-predictor-00001
  conditions:
  - lastTransitionTime: "2023-10-26T19:19:02Z"
    reason: PredictorConfigurationReady not ready
    severity: Info
    status: Unknown
    type: LatestDeploymentReady
  - lastTransitionTime: "2023-10-26T19:19:03Z"
    severity: Info
    status: Unknown
    type: PredictorConfigurationReady
  - lastTransitionTime: "2023-10-26T19:19:03Z"
    message: Configuration "sklearn-iris-example-predictor" is waiting for a Revision
      to become ready.
    reason: RevisionMissing
    status: Unknown
    type: PredictorReady
  - lastTransitionTime: "2023-10-26T19:19:03Z"
    message: Configuration "sklearn-iris-example-predictor" is waiting for a Revision
      to become ready.
    reason: RevisionMissing
    severity: Info
    status: Unknown
    type: PredictorRouteReady
  - lastTransitionTime: "2023-10-26T19:19:03Z"
    message: Configuration "sklearn-iris-example-predictor" is waiting for a Revision
      to become ready.
    reason: RevisionMissing
    status: Unknown
    type: Ready
  - lastTransitionTime: "2023-10-26T19:19:02Z"
    reason: PredictorRouteReady not ready
    severity: Info
    status: Unknown
    type: RoutesReady
  modelStatus:
    states:
      activeModelState: ""
      targetModelState: Pending
    transitionStatus: InProgress
  observedGeneration: 1

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

@sivanantha321 can you please review and merge the PR?

Hi @sivanantha321 can you please merge this PR if looks good to you ?  It will help me in Hacktoberfest submission.

@nilakshi104 You have to run make generate and commit those generated files


Updated wink version for binary switch fix

This updates the version of python-wink to 0.6.1. This version fixes an issue discovered by @TDWhbp where generic z-wave switches wouldn't responded to state changes.
This version also includes adding the sensor type (Temp, humidity, ect.) to the name of each Wink Spotter sensor. This should make it easier to identify the sensors, before they were labeled sensor.spotter, sensor.spotter_1, ... now they should be displayed as sensor.spotter_temperature, sensor.spotter_humidity, ...

@balloob When I run requirements_all.py the below lines get removed and Tarvis fails saying something about importing unifi. So I manually added them back in. Now Tarvis is failing saying requirements all isn't up-to-date?
Also any time I have changed the python-wink version number and re-ran gen_requirements_all.py the python-wink version doesn't update. I have to manually update it in the file. Am I not changing the version everywhere I need to?
 -# homeassistant.components.device_tracker.unifi
-unifi==1.2.4
-
-# homeassistant.components.device_tracker.unifi
-urllib3
-


Editor Page -> Editor: Generic JSON

This is done.  The default editor is currently just a simple JSON monaco editor.


Backpack error

CoreGui.RobloxGui.Modules.BackpackScript:1012: attempt to call method 'Select' (a nil value) | CoreGui.RobloxGui.Modules.BackpackScript, line 1012 - global changeSlot CoreGui.RobloxGui.Modules.BackpackScript, line 455

@Partixel We already have a issue for this bug. However the bug is technically fixed, next time the core-scripts from here get released it will no longer have this issue. Reference to the original issue: https://github.com/ROBLOX/Core-Scripts/issues/163

@Partixel Thanks man, if you have any other bugs regarding the core scripts. Don't hesitate to post here. :)


[Desktop App] Update app

As a user of the desktop app I want to be able to update the desktop app right from within the app so that I don't have to manually download a new version.
Acceptance criteria

Update should always be optional but strongly encouraged
For the first version we don't have to be able to add release notes yet, but should be considered that we want at some point to list changes included in the newer version

Tested with a 1.0.0 Safe: 0x55C1a91291956Eab87227d4184A26e6F80A771d5
The notification that pops up in the bottom left corner always says "Update not available", but I still got the notification in the top right corner to update the safe

I tried to add a new owner but instead of a Tx to add a new owner, a Tx to upgrade the safe showed up instead.
Safe address: 0x55C1a91291956Eab87227d4184A26e6F80A771d5
Note: I waited but the Tx remains in pending for ever. Is not part of this ticket to make the upgrade possible, but is good to point out that the upgrade seems not to be working in the desktop app.

@lukasschor Until we have new version of the app we cannot update it. I think we can close this ticket and if there is an issue then we can rise a new issue. What do you think?

@matextrem Is this still in progress or can this ticket be closed or moved to QA/Review?

Added on hold label to reflect the fact that we need to have an initial version released and an update for it in order to do a 'real' QA


add FluentValidationAutoValidationActionFilter BindingSource.Path

Hi.  I would like to use your extension for my application but for one of my end endpoint (PUT with id from route) and data from the body payload the validator is not being invoked.
[HttpPut]
[Route("/mymodule/{identifier}")]
public override async Task<ActionResult> HandleAsync([FromRoute] UpdateMyModuleCommand request, CancellationToken cancellationToken = default)
public class UpdateMyModuleCommand
{
[FromRoute(Name = "identifier")] public string Identifier { get; set; }
[FromBody] public UpdateMyModuleCommandParams Params { get; set; }
    public class UpdateMyModuleCommandParams
    {
        public string Name { get; set; }
        .... etc
     }

if in the OnActionExecutionAsync of the FluentValidationAutoValidationActionFilter,
I add this condition then my validator is invoked properly.
Could you please add a confguration EnableRouteBindingSourceAutomaticValidation with the BindingSource.Path in the next version.
Thanks.
if (subject != null && parameterType.IsCustomType() &&
((autoValidationMvcConfiguration.EnableBodyBindingSourceAutomaticValidation && bindingSource == BindingSource.Body) ||
(autoValidationMvcConfiguration.EnableFormBindingSourceAutomaticValidation && bindingSource == BindingSource.Form) ||
(autoValidationMvcConfiguration.EnableQueryBindingSourceAutomaticValidation && bindingSource == BindingSource.Query) ||
(autoValidationMvcConfiguration.EnableCustomBindingSourceAutomaticValidation && bindingSource == BindingSource.Custom) ||
(bindingSource == BindingSource.Path)
)
)
{

@madostie I just created a PR for it, I would appreciate it if could you take a quick look at it.

Just did, looks good.  Thanks for this speedy addition.  When do you plan to release 1.3?

There is still 1 remaining issue on the 1.3 roadmap: #13. If that one gets resolved 1.3 will release.

@madostie v1.3.0 just got released


Add account manager

This PR is based on #75

Add account manager
Add passkey account credential id to env
Align network manager with account manager

Summary of this PR:
Fetch account data from storage, initialize SimpleAccount and PasskeyAccount. Later, in WalletBackgroundProvider, use getActive() to retrieve the active account and perform sign(), getCallData(), getDummySignature(), etc. methods.


mat-chip-remove button is not keyboard accessible

What is the expected behavior?
The button to remove a mat-chip should be:

keyboard operable
have a name and role (be a button)

What is the current behavior?
The button to remove a mat-chip is not:

keyboard operable
have a name and role (be a button)

What are the steps to reproduce?

Using either of the following screen readers

NVDA 2018.4.1 + FireFox 64

Open the mat-chip example
Press the tab key until you reach the 'Lemon' chip under the 'Chips Autocomplete' example.
Press the tab key again. Observe that the button to remove the chip did not receive focus and that the button to remove the chip does not have an accessible name or role.

Which versions of Angular, Material, OS, TypeScript, browsers are affected?

Angular Material Example 7.2.2 located at https://material.angular.io/components/
NVDA 2018.4.1 + FireFox 64
VoiceOver for IOS (12.1.3) + Safari (12.1.3)

Is there anything else we should know?
It is possible to remove a chip by pressing backspace. However, I have found that most users are not aware of that behavior. People who use a mouse have access to another (more obvious) way to remove a chip (by clicking the button to remove it), and keyboard-only users should have the same opportunity.

For the chips autocomplete example, the remove buttton isn't keyboard accessible for the MDC based chips and button components

Hello,
I believe this issue has been fixed. I can access the remove button with keyboard for the chips autocomplete example. When I tab to a chip, I can move to the remove button by pressing the right arrow key.

Macos 13.3.1 (a) (22E772610a)
Chrome Version 114.0.5735.45 (Official Build) (arm64)

-Zach


feature request: also return real file path

At the moment the plugin returns a path that looks like capacitor://path/to/file, which is fine to be used on the webview (do fetch to it, use it as src on images or videos), but can't be used by the Filesystem plugin to copy to a permanent location.
Capacitor camera plugin has
path ?: string;
webPath ?: string;

where path is the real file path in the filesystem and webPath is a capacitor:// path the webview understands.
Since you are already using path for the  capacitor:// path, I would't use the same names as Camera plugin as it will be a breaking change in your plugin, but maybe you can return it like filePath or filesystemPath.

Sounds good

Hi!
I was looking into download files using this plugin on devices but I can't find the file once downloaded and wondering if this has something to do with that and its related.
Also, if thats the case, this is going to be added to the plugin?

Hi @Fdom92 ,
I'm currently facing the same issue. Have you managed to solve this?
If so, would you mind sharing your code solution?
Thanks,
Kevin

closing since the plugin seem abandoned


Documentation implies Squirrel checks for updates regularly, but it doesn't

The Electron documentation mentions that "Squirrel will check for an update again at the interval you specify." This doesn't seem to be true—the auto-updater seems to check for updates at launch, and again if manually invoked, but the Electron source code never calls Squirrel's startAutomaticChecksWithInterval to enable periodic checking.
I think it'd make sense to expose an API on the auto-updater module that calls through to startAutomaticChecksWithInterval, or update the docs. I'd be happy to submit a pull request if this is already on the wish list!

The misunderstanding description is already gone.


Import error: The data couldn't be read because it isn't in the correct format.

Most of the stencils created after running convert.sh fail to import, with the error:
The data couldn't be read because it isn't in the correct format.

My environment is as such:
Omnigraffle: 7.10.2 (v195.13 r328142)
OS X: Mojave 10.14.5
Ghostscript: 9.27
Mactex: 2019.0508
xpdf: 4.0.01

The stencil groups which do load properly are:

Customer Engagement
Databases
Machine Learning
Management & Governance
Robotics

Thanks for raising (I think) I'll take a look at the stencils you have listed.

Ok, so to test this I remove the icons from Omnigraffle with a quick rm -rf to really make sure as well as doing the same for my git based working directory.
Then cloned the repo and used
open $HOME/Library/Containers/com.omnigroup.OmniGraffle7.MacAppStore/Data/Library/Application\ Support/The\ Omni\ Group/OmniGraffle/Stencils

to open the stencil directory in finder, made a new directory AWS and copied all of the stencil files into it.
Finally opened Omnigraffle and got creative copying various services from the stencil groups you listed above into a beautiful diagram.
So then I got creative and started going through all of the stencil groups and found the following to be problem children

AR & VR

That was the only stencil that didn't load for me

Omnigraffle 7.10.2 (v195.13 r328142)
OSX 10.14.6

Just pushed 23a122a which for me resolves the issue with the AR & VR Stencil.
I'd be interested in knowing if others are having the same issue reported or even with different stencils.


(2/4) Trigger v0.17.6 CLI Staging Pipeline

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/lgtm

/unhold
/retest

/approve


don't pull in nnpops on osx or windows

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch packages can't have selectors. If the selectors are necessary, please remove noarch: python.

@conda-forge-admin, please rerender

noarch packages can't have selectors. If the selectors are necessary, please remove noarch: python.

Well that answers my question!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Looks like 3.10 if failing for OSX and Linux because we don't have 3.10 builds for openmm, which is fine, so I will go ahead and merge this in.

The 8.0 beta does have 3.10 builds.  I think we need a proper beta release of this package that is built against the betas of OpenMM and OpenMM-Torch.  This also seems to have only built for x86, not ARM.

@peastman This package used to be noarch so I forgot to add the arm builds! I will fix that!
Also I misread the errro:
package openmm-torch-0.03-py36h110734a_0 requires python_abi 3.6.* *_cp36m, but none of the providers can be installed
It looks like openmm-torch is missing the python 3.10 builds with the main tag, but has them with the openmm_rc tag, so I will need to also rebuild this package where it pulls those in.


fixed matt's link by adding in the proper target and rel properties

closes #1

LGTM, thanks for the help!


Fixfix failed running alias function with an alias function in original function.

Proposed changes
Issue Number: close #xxx

failed to run sql:
create alias function f1(int) with parameter(n) as dayofweek(hours_add('2023-06-18', n))
create alias function f2(int) with parameter(n) as dayofweek(hours_add(makedate(year('2023-06-18'), f1(3)), n))

 select f2(f1(3))

it will rise an exception: f1 is not a builtin-function.
because f2's original function contains f1, and f1 is not a builtin-function, should be rewritten firstly.
we fix the bug by analyzing the original function every time when applying rewrite alias function.


Add homepage and about page content

Whats with the weird space at the top of the screen? Is there meant to be content there when in a room? (also I cant look at the home page while also being in a room from a layman's perspective)

I don't super see a need to have the homepage be tied to a room since we scrapped having rooms be attached to wordgroups as far as I am aware. The space at the top is just so it feels less bunched up at the top, can change it as well.


tf.global_variable_initializer is misspelled as tf.global_variable_initializers (extra 's' at end) in release notes

In the release notes for 0.12.0rc0, the note on the renaming of tf.initialize_all_variables to tf.global_variable_initializer has a typo, calling it tf.global_variable_initializers instead. It's a small typo, but I think it's worth fixing since this is such an important breaking change.

Hmm, looks like some of our documents are also wrong:
https://github.com/tensorflow/tensorflow/blob/r0.12/tensorflow/python/ops/variables.py#L90
and the actual method seems to be
tf.global_variables_initializer Will send out a PR as a fix.


Type Error Corrected in code comment

Type error corrected.

@aftab10662  Thank you for contributing this pull request! We're truly excited and honored to have you as a member of our community. We look forward to working together with you to further build the Aurelia platform.
We noticed that you haven't yet signed our CLA. We would appreciate it if you would consider doing that for us. Before signing, please take a few minutes to read our contributing guidelines. These guidelines will help get you up to speed on how our community likes to operate and how the pull request process works. Once you have read that, assuming you are in agreement, simply sign our online CLA.
After you've signed, please ping this issue to let us know and we'll get your improvements merged in.
Thank you again!
The Aurelia Team

Dear Rob,
I have signed CLA.
Request you to review and merge my PRs.
On Tue, Feb 23, 2016 at 4:19 AM, Rob Eisenberg<EMAIL_ADDRESS>wrote:

@aftab10662 https://github.com/aftab10662 Thank you for contributing
this pull request! We're truly excited and honored to have you as a
member of our community. We look forward to working together with you to
further build the Aurelia platform.
We noticed that you haven't yet signed our CLA. We would appreciate it if
you would consider doing that for us. Before signing, please take a few
minutes to read our contributing guidelines
https://github.com/DurandalProject/about/blob/master/CONTRIBUTING.md.
These guidelines will help get you up to speed on how our community likes
to operate and how the pull request process works. Once you have read that,
assuming you are in agreement, simply sign our online CLA
https://docs.google.com/forms/d/1Npd68d32UDUKUYuCDDrW0VN5Lv8p-a6Ziaa2JFKp6Cg/viewform?c=0&w=1
.
After you've signed, please ping this issue to let us know and we'll get
your improvements merged in.
Thank you again!
The Aurelia Team
—
Reply to this email directly or view it on GitHub
https://github.com/aurelia/templating/pull/266#issuecomment-187424003.

Thanks for the contribution! In the future, if you could use our commit message format, that would really help us with generating our reports on work done against the repo.

Sure Rob.
On Wed, Feb 24, 2016 at 3:15 AM, Rob Eisenberg<EMAIL_ADDRESS>wrote:

Thanks for the contribution! In the future, if you could use our commit
message format, that would really help us with generating our reports on
work done against the repo.
—
Reply to this email directly or view it on GitHub
https://github.com/aurelia/templating/pull/266#issuecomment-187928079.


Fix for  issue #745

Fix for get_trees() method of CatBoostTreeModelLoader tree explainer.
In addition to fixing the issue reported in #745, it also fixes the IndexError: list index out of range  error caused by the same line of code.

Thanks @ibuda! To make sure this says fixed could you also add a short unit test to test_tree.py that would fail before you added this?

@slundberg Please check if the test looks good to you. I had no access to CatBoostTreeModelLoader.num_trees to assert equals it against the model's tree_count_, so I just called the class initialization of TreeExplainer on the model that triggers the error.

Sorry for losing track of this! I really appreciate you sending this in. Staying on top of Github issues has been a challenge, but I didn't realize I had let any PRs get stuck.
Thanks for doing this @ibuda!


Update database

Good day.
Is it possible to make it so that when executing a PATCH query, the field is not overwritten in the database, but added to the end?
...

Can you give an example with what you trying to do so i can better
understand your question?
Em seg, 22 de out de 2018 às 11:11, Aleksandr<EMAIL_ADDRESS>escreveu:

Good day.
Is it possible to make it so that when executing a PATCH query, the field
is not overwritten in the database, but added to the end?
...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prest/prest/issues/339, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEpfGcTLjF1d_1vz_Fvsnih1QeZirR5Hks5undH3gaJpZM4XzdK_
.

--
Felipe Oliveira
https://www.felipeweb.net.br

For example, I have a table "test"
CREATE TABLE test (
test_id bigserial primary key,
test_name varchar(20) NOT NULL,
test_desc text NOT NULL,
date_added timestamp default NULL
);
Suppose in the field "test_desc" there is an entry "example", so here I want that when executing the query
curl -s -X PATCH -H "Content-type: application/json" -d '{"test_desc": "new text" }' 'http://<IP_ADDRESS>:8000/test_db/public/test?id=104'
in the "test_desc" field, the value of "example new text" has become
as does this sql query
UPDATE test SET test_desc = CONCAT(test_desc, '  ', 'some_new_str') WHERE id =104 ;

pREST does not support COUNT function yet
Em ter, 23 de out de 2018 às 06:18, Aleksandr<EMAIL_ADDRESS>escreveu:

For example, I have a table "test"
CREATE TABLE test (
test_id bigserial primary key,
test_name varchar(20) NOT NULL,
test_desc text NOT NULL,
date_added timestamp default NULL
);
Suppose in the field "test_desc" there is an entry "example", so here I
want that when executing the query
curl -s -X PATCH -H "Content-type: application/json" -d '{"test_desc":
"new text" }' 'http://<IP_ADDRESS>:8000/test_db/public/test?id=104'
in the "test_desc" field, the value of "example new text" has become
as does this sql query
UPDATE test SET test_desc = CONCAT(test_desc, ' ', 'some_new_str') WHERE
id =104 ;
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prest/prest/issues/339#issuecomment-432166952, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEpfGd3iKqt9E4rBsI37sBkpeq8BUhAJks5unt7egaJpZM4XzdK_
.

--
Felipe Oliveira
https://www.felipeweb.net.br

not COUNT    bat  CONCAT

sorry support COUNT, but not CONCAT

@AlexIskander you propose some REST signature for CONCAT?
COUNT example:
https://postgres.rest/query-statements/#select-get

Are you understand difference between COUNT and CONCAT?

@AlexIskander unfortunately we don't have the behavior you want, but you can use sql scripts instead. Look at https://postgres.rest/executing-sql-scripts/
Thinking more about your doubt I don't know if it's feasible to implement it because in your example you're consjuder just string datatypes, but what if we need to "concat" numeric, date, time, interval, or others datatypes ?? If so we'll need to implement a "concat" behavior for each kind of target datatype.


Optimisations to TIFF image stack creation in CLEM workflow

Partially resolves issue #313:

More comprehensive transfer of metadata to created image stacks
Improved functions used in array manipulation of when creating image stacks

adjust_contrast seems to only ever be "stretch". Could it possibly take any other values?

Not a problem, was just wondering why the switch from bool to str


Added Wise Banking to Connectivity Apps

Added Wise Banking to the connectivity apps feature. Created a new procedure 'RegisterWiseBanking' and call to that procedure from LoadBankingAppsData.

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

Options:

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

Contributor License Agreement

@microsoft-github-policy-service agree [company="Wise lausnir hf."]

@microsoft-github-policy-service agree company="Wise lausnir hf."

It's a minor thing, but I'd move the code to line 170, so it's easier to tell the order of when an app is added

It's a minor thing, but I'd move the code to line 170, so it's easier to tell the order of when an app is added

I have moved the code to line 170.

merged internally, just need one more vote

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

Availability update: We will publish a fix for this issue in the next update for release 21.
Build ID to track: 51135.