Configuring vsgvr inside vsg-dev/vsgFramework fails

On openSUSE Leap I did run the following commands:
mkdir ~/src && cd ~/src
git clone https://github.com/vsg-dev/vsgFramework.git
cd vsgFramework
cat << EOF >> CMakeLists.txt
##############################################################################################################
#
# vsgVr
#
OPTION(BUILD_vsgVR "Build vsgVR" ON)
if (${BUILD_vsgVR})
    FetchContent_Declare(vsgvr
        GIT_REPOSITORY "https://github.com/geefr/vsgvr.git"
        GIT_TAG "main"
        GIT_PROGRESS TRUE
    )
    FetchContent_MakeAvailable(vsgvr)
endif()
EOF

and got the following failure when running  cmake -B ../vsgFramework-build
...
CMake Error at /home/user/src/vsgFramework-build/components/vsgvr-src/CMakeLists.txt:17 (find_package):
  find_package for module vsg called with REQUIRED, but
  CMAKE_DISABLE_FIND_PACKAGE_vsg is enabled.  A REQUIRED package cannot be
  disabled.

Why is find_vsg being disabled?
That's a required package for vsgvr to configure/build..

I wrote that in the commit at https://github.com/geefr/vsgvr/pull/13

vsgFramework adds vsgvr as an additional subproject where the vsg project is already available as a cmake target and find_package is not usable.

Seems like a strange way to correct the issue. Adding external modules should not require modifications to the included project.
How does vsgExamples do this?
Why can't vsgFramework pass in the vsg location when configuring vsgvr? - vsg must be compiled before vsgvr, so should have produced a vsgconfig.cmake file as part of that.

They removed the REQUIRED statement, see https://github.com/vsg-dev/vsgExamples/commit/aede11f0a8db8e23073ecd59e5d8afd38b8a4e2b

CMAKE_DISABLE_FIND_PACKAGE_vsg

This cmake variable was introduced with https://github.com/vsg-dev/VulkanSceneGraph/commit/0abaf64108e86fc3db28671f6396ae751305152b

Why can't vsgFramework pass in the vsg location when configuring vsgvr? - vsg must be compiled before vsgvr, so should have produced a vsgconfig.cmake file as part of that.

The vsgFramework project has recently been changed from ExternalProject to FetchContent, where each project is added as a submodule and a large cmake project is created. vsgvr is built as part of the cmake --build process.
find_package needs additional files besides vsgConfig.cmake, which are only created during an installation and thus are not available during the build step.
user@host:~/src/vsgFramework-build> make install DESTDIR=../vsgFramework-install
user@host:~/src/vsgFramework-build> ls ../vsgFramework-install/usr/local/lib64/cmake/vsg -l
total 20
-rw-r--r-- 1 user users 767 12 Oct 16:29 FindVulkan.cmake
-rw-r--r-- 1 user users 490 12 Oct 16:29 vsgConfig.cmake
-rw-r--r-- 1 user users 1722 12 Oct 16:29 vsgConfigVersion.cmake
-rw-r--r-- 1 user users 3626 12 Oct 16:30 vsgTargets.cmake
-rw-r--r-- 1 user users 803 13 Oct 12:03 vsgTargets-release.cmake

FetchContent should be installing vsg to a directory then, as one would if they did a FetchContent(fmt) or other open source project.
Projects included this way really should not be modified because of the parent project's quirks - removing the REQUIRED in vsgvr for instance means it will then fail whenever vsg isn't available from say a package manager.
Apologies, I have very minimal time at the moment, and am in the middle of several crunch times :) I'd definitely say the issue is in vsgFramework, afraid I don't have a 'correct' example to share at the moment.

I spent days trying to avoid the problems with FetchContent, what is checked in is the best workaround I could come up with, it looks like a CMake problem.
I'm not personally going to waste any more time on the problem within the VSG project, if you want to invest your own time on the issue please feel free.  Personally I'd recommend just getting just removing the REQUIRED option as a workaround as there are so many more important real-world problems you can spend your time on.

@robertosfield Fair enough, folks are free to branch as needed for now, I'll take a look/merge when I can.
(I expect other vsg changes have broken vsgvr, since my integration was still fragile. No rush on this stuff from my perspective..)

Okay, thanks for the fix.
Note that vsgvr doesn't have cmake install targets yet - I've just been running from the build directory for now

Since I often use packages, I made that possible on https://build.opensuse.org/package/show/home:rhabacker:branches:games/vsgvr that with a temporary patch.


Improvements for SearchParamManager

Description

Removes the CosmosDbStatusRegistryInitializer and moves the initialization logic into SearchParamManager so it can be shared with the SQL implementation
Uses pre-computed support defined in FilebasedSearchParameterRegistry to speed up IsSupported checks

Closing this for now


add pandas as a defualt package

resolves #228
Description
Add pandas as the default packages for dbt-snowflake
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

@jtcohen6 As I looked into this more, I realized that the reason we don't run into this issue without specify pandas was because pandas is required(maybe optional) for snowflake-snowpark package. I can def do the checking pandas way, but also think it is okay to require it.
Product call and we will make it happen.

-

@ChenyuLInx Sounds good! Let's do the other approach then: check to see whether pandas is installed, and then proceed accordingly.
Only question: It's not totally clear to me if a user could still call .to_pandas() on a Snowpark data frame, even if pandas isn't loaded into the environment. I think this would fail:
def model(dbt, session):
    df = dbt.ref('some_model')
    return df.to_pandas()

And so the user would need to configure it as:
def model(dbt, session):
    dbt.config(packages = ['pandas'])
    df = dbt.ref('some_model')
    return df.to_pandas()

@jtcohen6 Looking though the Slack chat again of how this issue is created, here's my current theory.

For snowpark python run, you cannot use any third party library before accept Anaconda Terms, but you can still use snowflake-snowpark package, and when you specify it as a package, snowflake will install it without including the optional pandas dependency. Thus user would run into the pandas not available error
Once you accept the Anaconda term, when you specify snowflake-snowpark as a package, snowflake will automatically install it with the optional pandas dependency. And that's why we never run into an issue with it.
to_pandas probably will not work if you don't accept the Anaconda term

So the current solution of checking whether pandas is available should work just fine.

before accepting the term, your pure snowpark python code works fine
after accepting the term, you still don't need to add pandas to every environment, it will be automatically added.

Might worth confirm with the snowpark team about it since both our testing and production account have accepted the term and there's no way I can test it.


[PRM-2498] - Remove 'Let us know' from you cannot go back to appeal details page

English:

Welsh:

rebuild


Use 3 seconds as default timeout for waiting for Futures

it was 150ms before and there were a lot failures in code using this.
An alternative would be to use test duration scaling but I like this better.
(A lot of the other durations should not be scaled.)

:+1:


Add to F-Droid

It would be perfect to have this on F-Droid!

+1

is this still being considered? would really love it! @hazuki0x0


etcd service discovery

Hi
Would an etcd service discovery pull request be accepted?

No, this has been discussed in the past. There is no standard layout of
targets in etcd, we don't want to prescribe one, and a general solution
would have to be similar in complexity to confd and
file_sd, so the preferred way is to just use confd (or anything else) and
file sd.
On Wed, Jul 12, 2017, 22:48 Rene Zbinden<EMAIL_ADDRESS>wrote:

Hi
Would an etcd service discovery pull request be accepted?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/2940, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAICBrB0y7h6FIppa-38vqI4XVNUAJNwks5sNTEGgaJpZM4OWLtg
.


I use fuild + alluxio to cache S3 but it seem to slow to cache so slowly

Alluxio Version:
v2.8.1
Describe the bug
It takes about 10hours but only cache 30G+ data.
NAME                       UFS TOTAL SIZE   CACHED     CACHE CAPACITY   CACHED PERCENTAGE   PHASE   AGE
segmentation-large-model   888.18GiB        34.56GiB   9.77TiB          3.9%                Bound   7h18m
To Reproduce
fluid+alluixo
2 GPU Server
Expected behavior
alluxio can cache data more quilcy
Additional context
log is here:
alluxio-logs.zip

@andyzheung can you help get the async profiler of fuse, worker and master?


Is there a way to use API key as auth

Hey there,
I'm using Loki from the Scaleway Cockpit and I haven't a username + password but just an API key.
For example, i can send a log by CURL by this way:
curl --location 'https://logs.cockpit.fr-par.scw.cloud/loki/api/v1/push' \
--header 'Content-Type: application/json' \
--header 'Authorization: Bearer myApiKeyHere' \
--data '{
  "streams": [
    {
      "stream": {
        "label": "helloWorld"
      },
      "values": [
          [ "1706107899000000000", "This is a basic log :-)" ]
      ]
    }
  ]
}'

So, is there a way to use the API key as auth instead of credentials?

You should be able to use headers options : https://github.com/Julien-R44/pino-loki#headers
import type { LokiOptions } from '../src/types/index'

const transport = pino.transport<LokiOptions>({
  target: 'pino-loki',
  options: {
    headers: {
      'Authorization': `Bearer ${process.env.LOKI_TOKEN}`,
    },
  },
})

Yes, sorry because i wanted to delete this POST since i saw in the documentation that we can also put the API KEY directly in the URL..
Thank's for your help!


inline ListboxField appears as CheckboxSetField

A ListboxField should appear as a multi value dropdown field:
ListboxField::create('Test', 'Test', ['0' => 'test1', '1' => 'test2'])

However if the elemental item is inline, it appears as a CheckboxSetField:

Interestingly, this extension https://packagist.org/packages/silverstripe/tagfield allows a (different looking) multi field dropdown to work

This isn't actually to do with elemental specifically - this will be the case any time that field is rendered in a react context. The ListboxField doesn't have its own react component, so it falls back on the component defined for MultiSelectField which happens to be a set of checkboxes.
I'm marking this as a feature request. It's not likely to be worked on by the core team any time soon but we'd definitely welcome PRs for this one. Using the react-select library, which is used for TreeDropdownField and TreeMultiselectField would be my recommendation.


Incorrect identification of interface blocks in regex frontend

In the presence of interface blocks, the regex frontend incorrectly identifies the end subroutine statement inside the interface block as the end of the parent subroutine. This happens even in the presence of the subroutine name at the end of the interface block i.e. end subroutine name. Strangely, this only seems to happen once the raw source has been sanitized by FortranReader; the regex frontend correctly parses the unsanitized string correctly and the sanitized string incorrectly (the sanitization itself appears to be correct though). I've attached an example.
regex_frontend_bug.txt

The problem arises because the regex search pattern for class SubroutineFunctionPattern looks for the end subroutine | function phrase, but the match with the subroutine name is only optional (as it should be). That's why it matches incorrectly with the end subroutine statement inside the interface block.
Defining a grouping inside the regex search for the named-grouping <spec> fixes the problem; in the constructor for SubroutineFunctionPattern, the search pattern r'(?P<spec>.*?)' should be replaced by
r'(?P<spec>.*(^[ \t\w()]*interface.+^[ \t\w()]*end interface\b.*?)*.*?)'.
The interface blocks are then stored in the named-grouping 'spec' and can be retrieved later on if need be in a similar fashion to imports, variable declarations, calls etc (although I cannot foresee a scenario where we would need to retrieve interface blocks, given we already have enriched calls once we complete the scheduler parsing).


Add support for acceptFirstMouse on window

Hey man!
Stumbled upon this project yesterday and I think this is a great idea!  Figured I couldn't come empty handed, so here's my first attempt at a contribution. I immediately ran into some interesting stuff because acceptFirstMouse can only be passed as an option in the constructor of Window, which made me think that it only supports being 'uncontrolled'... I think...
Had some trouble testing this as well, so that's still missing. I didn't want to spend a lot of time on that before checking whether this prop is a good idea in the first place. Maybe you were planning on passing the initial options in a different way?
Anyway, love the idea, I'm definitely going to dive into how you got this to work.

Thanks for the PR!

I didn't want to spend a lot of time on that before checking whether this prop is a good idea in the first place. Maybe you were planning on passing the initial options in a different way?

There's not really any other way to pass them in, truth be told, so I think this makes sense. Considering that there's no "controlled" analogue, I would just name the prop acceptFirstMouse and call it a day.
That said, could you add a case for this under WindowElement.commitUpdate which emits a warning if the user changed this prop?  I'd like them to know that <window acceptFirstMouse /> is how it's meant to be used.
As far as testing: I totally feel you. I'll tell you what- if you can add a file under the examples/ directory, I'll figure out the "proper" testing (don't feel bad, I still need to add tests for position and size, haha)

No problem, happy to help!
So I added an example, removed 'default', and put in probably the lamest warning ever. Totally wasn't sure about how you were planning to do that stuff so I thought I'd just throw it in and we could discuss it. Maybe it's a good idea to put those checks behind a gate so it doesn't run in production, but it seems a mechanism like that is not in there yet.
About the testing, I think I have found out why I had trouble with that, and might have some ideas on how to make it easier. If you're interested I could make a separate issue to share my ideas.

Did some more digging in the react codebase and added a stacktrace to the warning message. Went for checking process.env.NODE_ENV before logging, just to be sure.
I created an issue for the stuff I ran into when trying to test this.

 Out of curiosity, what does getFiberStackAddendum do?

I found it being used here (and in the files next to that one).
It returns a sort of stack trace of where we are for the current render. If I trigger the warning in the ionize-example-app you'll get this:

I'm not really sure it's perfect like this, but I figured it's better than nothing. I'm still planning on debugging react to see how they use this and what it does over there.


duplicate @nestjs/common declaration

"@nestjs/common": "^7.6.15",

Appears twice in package.json

I'm not quite sure about that.
One is a devDependency and the other one is for peerDependency.
I used this as a reference where they are also set as dev- and peerDependency

Ah yes it is peer dependency, my mistake. I will close.


SPMD for initializing model using nnx.jit

I would like to ask an example for initializing and updating a model using nnx.jit with SPMD.
Is there any relevant example?

nnx.jit's API is changing very soon (#3963) according to #4107.  Currently you could use nnx.iter_graph to traverse the entire model and use with jax.device_put to shard the arrays inside Variables.
with path, parent in nnx.iter_graph(model):
  if isinstance(parent, nnx.Module):
    for name, value in vars(parent).items():
      if isinstance(value, nnx.Variable):
        sharding = ...
        value.value = jax.device_put(value.value, sharding)

Hello,
I have some code examples for this purpose that would cause an error like ValueError: Mismatch custom node data: NodeDef(...
Could you please help me?
def _create_state():
    model = Model(...)
    tx = optax.adam(...)
    wrt = nnx.All(nnx.Param, nnx.Everything())
    optimizer = nnx.Optimizer(model, tx, wrt=wrt)
    graphdef, state = nnx.split((model, optimizer))
    return graphdef, state

with mesh:
    abst = nnx.eval_shape(_create_state)
    shardings = nnx.get_named_sharding(abst, mesh)
    graphdef, state = nnx.jit(_create_state, out_shardings=shardings)()


Version 0.10.2 property media_selection

If you assign more than 10 images in a media selection, they will be shown on the page, but if you want to delete or watch the assigned images in the media selection again you can't display more than 10 images.

Just tried, seem to be working now. I have assigned 12 images, and Sulu showed all of them to me. Can you try to reproduce it again, and close this issue, if you get the same result?


failing docs CI

The RTD builds are timing out again. With my own setup I get this error instead:
Traceback
AttributeError                            Traceback (most recent call last)
~/checkouts/readthedocs.org/user_builds/xarray-keewis/conda/latest/lib/python3.8/site-packages/IPython/core/formatters.py in __call__(self, obj)
    700                 type_pprinters=self.type_printers,
    701                 deferred_pprinters=self.deferred_printers)
--> 702             printer.pretty(obj)
    703             printer.flush()
    704             return stream.getvalue()

~/checkouts/readthedocs.org/user_builds/xarray-keewis/conda/latest/lib/python3.8/site-packages/IPython/lib/pretty.py in pretty(self, obj)
    392                         if cls is not object \
    393                                 and callable(cls.__dict__.get('__repr__')):
--> 394                             return _repr_pprint(obj, self, cycle)
    395
    396             return _default_pprint(obj, self, cycle)

~/checkouts/readthedocs.org/user_builds/xarray-keewis/conda/latest/lib/python3.8/site-packages/IPython/lib/pretty.py in _repr_pprint(obj, p, cycle)
    698     """A pprint that just redirects to the normal repr function."""
    699     # Find newlines and replace them with p.break_()
--> 700     output = repr(obj)
    701     lines = output.splitlines()
    702     with p.group():

~/checkouts/readthedocs.org/user_builds/xarray-keewis/checkouts/latest/xarray/core/rolling.py in __repr__(self)
     99         """provide a nice str repr of our rolling object"""
    100
--> 101         attrs = [
    102             "{k}->{v}".format(k=k, v=getattr(self, k))
    103             for k in list(self.dim) + self.window + self.center + [self.min_periods]

~/checkouts/readthedocs.org/user_builds/xarray-keewis/checkouts/latest/xarray/core/rolling.py in <listcomp>(.0)
    100
    101         attrs = [
--> 102             "{k}->{v}".format(k=k, v=getattr(self, k))
    103             for k in list(self.dim) + self.window + self.center + [self.min_periods]
    104         ]

AttributeError: 'DataArrayRolling' object has no attribute 'y'

I think that was introduced in #4219. cc @fujiisoup
Also, we should definitely ask support why those two behave differently.

Ah, this
attrs = [
--> 102             "{k}->{v}".format(k=k, v=getattr(self, k))
    103             for k in list(self.dim) + self.window + self.center + [self.min_periods]
    104         ]

should be "{k}->{v}".format(k=k, v=getattr(self.dims, k)) not "{k}->{v}".format(k=k, v=getattr(self, k))
I'll send a fix.


Show database path when unable to find the file

The "Cannot find the database file" item should show the database file path in the description

hi may can i do this ?
please give me address file  and function name

Sure thing! Take a look at main.py lines 46 and 177. Let me know if anything in the code is unclear or if you get stuck.

I have searching for files with suffix db the directory and write this on the description.But I don't know what database extensions to search for


added kwargs for softmax_cross_entropy_with_logits calls

This commit breaks the project for me. It looks like the correct name for the kwarg is, and always has been, 'labels' rather than 'targets'. https://www.tensorflow.org/api_docs/python/nn/classification#softmax_cross_entropy_with_logits


Cross-filtering does not work when using fold transformation

First of all, thanks for developing Deneb!
I had to create a custom "table" using Deneb like in this video: https://www.youtube.com/watch?v=2wZpdZ8hCik.
When I try to get a row filtered in this matrix, based on a selection on another visual, cross-filtering does not work. I see in the original unfolded dataset that the right data point is filtered. However, in the dataset_0 data layer, no data is available. I would have expected the filtered row from the original dataset to be unpivoted with the fold transformation.
When looking at the dataset using cross-highlighting instead of cross-filtering none of the rows appear as "on" in the selected column.
Is this expected?
Thanks!

Hi, and thanks for using it!
I'd need to know more about your spec and data to comment in detail, but I'll provide my suspicions and maybe you can fill in the blanks. A fold transform will mutate the dataset from its original grain, and the transformed data stream probably does not contain the correct __row__ identifier that is required to reconcile back to Power BI. You can confirm this by checking the resulting data_x dataset for a layer or mark in the debug pane.
If the granularity of your data stream matches the original dataset after transformation, as long as you can ensure that the __row__ identifier is present, cross-filtering should work as you expect. If you have transformed the data away from this, then Power BI will not know what to do as it can no longer reconcile the datum to anything it originally sent to the visual.

Thank you for such a quick response! I really appreciate it.
My transformation of the data set includes calculating a rank to be able to sort the elements of the dataset by one of my columns. Then I carry out the fold transform to unpivot a set of columns with different metrics. As far as I can see, the row identifier is kept when I do these transformations on the dataset. In fact, when using fold row is repeated across the rows that are being unpivoted, which is what I would expect. However, the cross-filtering or cross-highlighting behavior is not taking this into account apparently.
See for example,
. This element has been selected in another visual, cross-highlighting enabled in the interactions and in Deneb settings and I would expect the selected field to be "on" instead of neutral.
When I test interactions between these same visuals in the opposite direction it works fine, i.e., a selected element of the table cross-highlights the other visual correctly. This makes me wonder if it could be related to a one-to-many relationship between the row identifier across visuals being caused by the fold transform that PowerBi can't handle?

The relationship in your model shouldn't affect Deneb's internal treatment of the dataset. Custom visuals know nothing about the data model apart from the fields provided, and Power BI resolves everything on their behalf. We will need more information to get to the root cause of your problem and to understand if this is an issue with your specification or Deneb in this case.
Because I develop and maintain Deneb for free, I don't have the bandwidth to set up complex scenarios to find potential issues independently. Anything that folks can do to reduce my time spent on debugging will help me get to the root cause faster and either find a solution or a fix.
Please provide a minimally reproducible copy of your issue and supporting visuals in a Power BI workbook, preferably with anonymized data, and attach it here (rename the .pbix  to a .zip). I will try to find time to look into what the issue might be. To manage your expectations, I will shortly be taking some leave, so I will have even more limited capacity over the next couple of weeks, but I will try to find some time if you can provide something that helps me see the issue in context.
If you need more dedicated assistance, it may be better to engage with a community expert (refer to the Support page for more information on how to do this and how to help them help you faster.)

Thanks for your support!
When building a simplified version of the PBI to share here, I realized that the issue is not actually with the fold operation. It's a filter transformation that I applied to the data that is affecting the cross-filtering.
I generated two reports, one without the additional filter, where cross-filter is working as expected and another one with this additional transformation step, where cross-filtering is not working. Test Pareto.zip


Comments style guide and coverage

I'll keep this short as possible, but I believe our inline comments should be focused on 'quantity over quality' in way.
Unlike driver code like Wasatch.PY which can benefit from more prose-style long comments explaining histories and pontificating on specifications-- Enlighten is predominantly glue code, and so we want comments that help us quickly navigate the codebase and understand control flow.
We benefit more from simple comments like "don't do this" and "this happens when X" more than "here's everything about (concept)."
This issue is to amp up our comment coverage, addressing some seriously parched areas of our codebase, while also focusing on quick orienting comments, to address some of the overwatered areas.

For Qt related comments, we want 3-5 lines of explanation with a link the Qt documentation.
We will likely need to reference Python PySide6 documentation: https://doc.qt.io/qtforpython-6/PySide6/QtWidgets/QCheckBox.html  - or -
The C++ reference https://doc.qt.io/qt-6/qcheckbox.html


[12.x] Return unprocessable response when parameter is not a valid UniqueStringId

This changes the behavior of the HasUniqueStringIds::handleInvalidUniqueId() method so that it returns a semantically correct HTTP response that can help callers determine the root cause of a failing response.
If I pass an incorrectly shaped UUID (1234-th!s-is-not-A-uuid-#^@b) to a route that expects a UUID, returning a 404 is misleading. While it is true the model probably does not exist in the database and therefore is not found, the reality is that the request is failing validation. It is easy to assume that the response returned is the result of a DB query. (For instance, I tried to optimize routes by skipping route model binding because I didn't understand that the database was NOT being queried in this case).
When HasUniqueStringIds::isValidUniqueId() indicates the key invalid, we throw a new custom exception InvalidIdFormatException. It has been added to the Handler::$internalDontReport array, and when raised in the application during an HTTP request, gets converted to a 422 response.
I targeted master because I believe this behavior change will break tests for users and therefore is not appropriate for a patch release.

Alternatives

Modify the ModelNotFoundException. I'd be happy to pivot back to this if it makes more sense for the framework. It seems very muddy, but the new InvalidIdFormatException duplicates much of what exists in that class.
Throw a ValidationException instead. Something like ValidationException::withMessages([$field => '???']). I'm not sure what the message should be in that case.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!


Issue With New Area Code of ALABAMA - 659

Imported from Google Code issue #402 created by kratos.chaitanya on 2014-01-09T05:13:16.000Z:

What steps will reproduce the problem?
1.Enter a US - Alabama Phone Number 659-xxx-xxxx
2.Select Region as "US"
3.It shows returns that phone number is invalid and if you select region as "SG" it shows as valid Singapore Number.
What is the expected output? What do you see instead?
Valid for US case and invalid for Singapore Case
What version of the product are you using? On what operating system?
5.9
Please provide any additional information below.
please address this issue asap

Why is this issue closed? We're experiencing this bug in production.

Comment #6 (and previous comments) explain why this is closed. NANPA says 659 is currently not in use. (http://nanpa.com/enas/npa_query.do search for 659 and there's a key/value in the result for "Is this code in use:").

Yes, but it apparently is being used - we have customers complaining that their Alabama 659 number is not being accepted, and it's necessitating a workaround.

Did you call one of the numbers to verify that it connects? How many numbers are there?
Neither the NANPA site nor the government of Alabama's telephone prefix directory list 659:
http://www.nanpa.com/enas/npa_query.do
http://info.alabama.gov/PHONE_BOOK_pdfs/Prefixes.pdf
I called half a dozen +1 659 numbers I found online, and none of them connected.


UI/education

Educational Videos Page
:recycle: Current situation & Problem
We are in need of a view for displaying educational videos. Certain tasks associated with messages will be to watch a video, and "more information" buttons in the Medications view links to the videos as well. Before we can implement these features, we need videos to reference.
:gear: Release Notes

Implemented an environment-accessible VideoManager module that downloads the static videos data from Firestore on sign in.
Built out the Education tab with a sectioned-list style UI. Each section is expandable and features a header and a collection of video thumbnails.
The video thumbnails are linked to a view that has an embedded video and a description of the contents of the video.
Implemented the asynchronous components (fetching videos and thumbnails) with state/error handling and progress views.

The Education tab now looks like this:

Education Page
Video View
With Video Descriptions

:books: Documentation
Note: At the moment, the server does not contain descriptions for the videos, so no videos will feature descriptions.
:white_check_mark: Testing
Note: At the moment, testing decoding videos from Firestore is not possible until the emulator in the build-and-test job of the CI is seeded by the new seeding functionality in StanfordBDHG/ENGAGE-HF-Firebase. Testing the VideoManager's functionality will be complete with a Unit test once the server seeding data feature is integrated into the iOS CI (see #59). Until then, UI tests are done by skipping decoding the videos from the emulator.
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[X] I agree to follow the Code of Conduct and Contributing Guidelines.

Great job @nriedman! 🚀


Changes dask.array.meshgrid to use broadcast_to internally (#2981)

PR for Issue #2981. Changes dask.array.meshgrid to use broadcast_to internally.

LGTM. Made one minor style comment above. The other comment can be ignored.

LGTM. Thanks. Any thoughts @shoyer?

Toggling for CI.

Thanks for the fixes. LGTM.

Planning on merging tomorrow if there are no more comments.

Thanks @magonser.

FWIW am doing a bit of tidying in meshgrid while adding broadcast_arrays in PR ( https://github.com/dask/dask/pull/3217 ). Might want to give it a quick look if you have time.


cmd/go: improve error message for unparsable, too-new, go.mod

Split from #70949.
The tl;dr of that issue is:

Set GOTOOLCHAIN=local to prevent automatic toolchain upgrade
Use Go 1.23 toolchain
go.mod says 1.24rc1 and contains a tool block

The 1.23 toolchain does not upgrade because of GOTOOLCHAIN=local, so it attempts to parse the go.mod itself. It does not understand tool blocks, so it reports a parse error:
go: errors parsing go.mod:
go.mod:206: unknown block type: tool

We could have a better error message in this case. Perhaps one of:

Add a hint to the above error that GOTOOLCHAIN=local prevented and upgrade
Or (my preference), stop immediately after determining the module Go version without fully parsing the go.mod, since we won't build such a module anyway.

Related Issues

x/tools/gopls: improve error message when user needs newer go version #61673 (closed)
cmd/go: go get -u && go mod tidy behaves differently starting with Go 1.21.0, leading to unknown directive toolchain errors #62409 (closed)
cmd/go, x/mod: "invalid go version" message mentions old version format #61888 (closed)
cmd/go: `go env GOMOD` tries to download newer toolchain #61455 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Upon further thought, if there is no tool block, GOTOOLCHAIN=local go build says "go: go.mod requires go >= 1.24rc1 (running go 1.24; GOTOOLCHAIN=local)". IMO the error should be identical with the addition of a tool block.


Integration with Openai Advance Audio Mode

Could we look into this?

What's the use case?

Closing until you give more context


Handle negative values to reshape

(base on #1508 )
array.reshape() accept at most one negative value which indicates the size of that dimension should be inferred.

This PR now has conflicts with master.  Can you update so I can merge?


Constraint index

@joseph-gergaud introduced a specific type for integer ranges to avoid dispatch problems when the user does not enforce Real type for bounds. Check this:
julia> constraint!(ocp, :initial, 2:3, [ 0, 0 ])
julia> constraint!(ocp, :final, Index(1), 0)
julia> constraint!(ocp, :initial, 2:3, [ 0, 0 ], [1, 2])
julia> constraint!(ocp, :final, Index(1), 0, 2)
julia> constraint!(ocp, :control, Index(1), 0, 2)
julia> constraint!(ocp, :state, 2:3, [ 0, 0 ], [1, 2])

ok

@ocots quick review + MR please?


Path Component Accessible label

Adding accessible label to Path component
Note the test snapshots are auto-generated by adding the --updateSnapshot flag when running JS tests.

Epic screen example:
Screen Reader before change

After Change


Hikvision (DS-KD8003) ISAPI command

Can someone teach me how to add the function of open door/relay for hikvision doorbells?
I found this home assistant thread  discussing this and they had this command for open door
To open an electrical door (relay 1 or 2)
curl -i --digest -u admin:xxx -X PUT -d 'open' http://192.168.0.xx/ISAPI/AccessControl/RemoteControl/door/1
Not sure if this can be implemented into this plug-in or any other plugin.
I tried putting in the URL with the IP of the door station.
Also added username and password options.
I get this error in reuturn:
[08/07/2021, 17:20:23] [HTTPLock] Error getting status: connect ECONNREFUSED <IP_ADDRESS>:80
Note: I did not add the port in the url. It just appears in the logs
Apologies for the noob question, I'm not a coder, so I don't know how these things work exactly

Hi there, apologies for taking so long to get back to you about your issue. Unfortunately, as I mentioned in your other issue (#5), I would need more information on the specifics of how the digest authentication would need to be implemented.
Similarly, as I do not have the device you are referring to, it would be very difficult for me to add support without having one to test with.
Please feel free to fork this plugin and make the necessary changes to support your device.


Saving and restoring models from file

What is the current best practices for saving and restoring trained models?
I have initialized and optimized some models based on the following snippet:
m = gpf.models.SVGP(X, y.reshape((-1, 1)), kern=gpf.kernels.RBF(len(selected_features)), likelihood=gpf.likelihoods.Gaussian, Z=inductionPoints)
gpf.train.ScipyOptimizer().minimize(m, maxiter=1000)

And then pickled the optimized model:
with open('/path/to/file', 'wb') as fp:
    pickle.dump(m.read_trainables(), fp)

But then I want to use the model for prediction later on. I am using the following
with open('/path/to/file', 'rb') as fp:
    param_dict = pickle.load(fp)
m = gpf.models.SVGP(X, y.reshape((-1, 1)), kern=gpf.kernels.RBF(len(selected_features)), likelihood=gpf.likelihoods.Gaussian, Z=inductionPoints)
m.assign(param_dict)

Is there any way to use the assign method without specifying the arguments again? It seems kind of clunky that I have to have access to exactly the same variables as for the training, especially also if the induction points have been shuffled. Am I missing something, or is this not the way to do it?

With "without specifying the arguments again" you mean in the constructor of SVGP?
Unfortunately, you're right. The way you're doing is the best way. It's also the way that I do it whenever I need to store models. The problem is that we don't want to have to store the structure of all the objects in the SVGP instance when we store the parameters. This would be clunky, easy to break and wouldn't compatible with code changes.
Just out of interest though, how would you like to see it being done?

@markvdw Yes, I mean the arguments to the constructor, ie. I don't want to specify X, y, Z, etc. I really don't see why they should be relevant for further use, or maybe I am not understanding correctly what the GP model has to use for future predictions. It is my understanding that the hyperparameters would be sufficient?
So, the way I would prefer it is something akin to the API in scikit-learn: first, assign the constructor to an object, then fit/optimize the model using supplied datapoints, and then after that use the optimized model for prediction or save the model for later use:
# Initialize the model
GPmodel = gpf.models.SVGP(kern=kernel_fn, likelihood=likelihood_fn, kwargs)

# Fit/optimize over supplied datapoints
GPmodel.optimize(X_train, y_train, maxiter=maxiter, optimizer=optimization_fn)

# Now, internally, the model contains optimized hyperparameters and can be used to predict on new data
predictions = GPmodel.predict(X_test)

# We can also save the model
GPmodel.save_to_file('/path/to/save/file')

And in a separate session, we would be able to call the model from the save file and it would contain exactly the same hyperparameters, likelihoods, priors, etc. as before:
GPmodel = gpf.models.restore('/path/to/save/file')
predictions_restored_model = GPmodel.predict(X_test)

Does that make sense?

@markvdw A small update: I'm running into some errors with the above code. Whenever I try to assign model parameters, I get the following:
Traceback (most recent call last):
  File "/home/neergaard/anaconda3/envs/gpflow/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 2963, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-5-01aaaf39abc2>", line 2, in <module>
    likelihood=gpf.likelihoods.Gaussian, Z=X.copy())
  File "/home/neergaard/anaconda3/envs/gpflow/lib/python3.6/site-packages/gpflow-1.1.1-py3.6.egg/gpflow/core/compilable.py", line 90, in __init__
    self.build()
  File "/home/neergaard/anaconda3/envs/gpflow/lib/python3.6/site-packages/gpflow-1.1.1-py3.6.egg/gpflow/core/node.py", line 156, in build
    self._build()
  File "/home/neergaard/anaconda3/envs/gpflow/lib/python3.6/site-packages/gpflow-1.1.1-py3.6.egg/gpflow/models/model.py", line 79, in _build
    likelihood = self._build_likelihood()
  File "/home/neergaard/anaconda3/envs/gpflow/lib/python3.6/site-packages/gpflow-1.1.1-py3.6.egg/gpflow/decors.py", line 67, in tensor_mode_wrapper
    result = method(obj, *args, **kwargs)
  File "/home/neergaard/anaconda3/envs/gpflow/lib/python3.6/site-packages/gpflow-1.1.1-py3.6.egg/gpflow/models/svgp.py", line 118, in _build_likelihood
    var_exp = self.likelihood.variational_expectations(fmean, fvar, self.Y)
  File "/home/neergaard/anaconda3/envs/gpflow/lib/python3.6/site-packages/gpflow-1.1.1-py3.6.egg/gpflow/decors.py", line 64, in tensor_mode_wrapper
    'Tensor mode works only for Parameterized object.')
gpflow.core.errors.GPflowError: Tensor mode works only for Parameterized object.

Yes that totally makes sense. For predictions only the hyperparameters and variational parameters are needed (so Z definitely is needed for predictions, but not X and Y). The reason that we require to supply X and Y as inputs to construct a model is a hangover from the full GP. In the full GP (or GPR in GPflow), you need the hyperparameters and the data in order to make predictions. For sparse GPs this is no longer necessary, but we decided to keep a similar interface (other GP packages made the same choice).
Anyway, there are some vague plans to make something like a VariationalPosterior object, which only requires hyper and variational parameters for predictions, so you don't have to keep the data around (or pass in data). However we currently explicitly choose not to support something like:
GPmodel = gpf.models.restore('/path/to/save/file')
predictions_restored_model = GPmodel.predict(X_test)

For the reason that this restore() function would have to re-create all the kernel and likelihood objects. It is very hard to store the object hierarchy in a general way, especially as versions of the code change, and since we're moving to much more complex models.

Thinking about it a bit more, actually, perhaps there would be a neat way to do this, but it's not something we're actively developing at the moment. Obviously, we're happy to accept contributions, although for a feature this big and important, we would recommend having a few calls beforehand to discuss software design etc.

Hello @doktorneergaard, @markvdw
import gpflow
import numpy as np
import numpy.random as rnd
rnd.randn(100, 1)
Y = np.sin(X) + np.sin(1.5*X) + 0.3 * rnd.randn(*X.shape)
m = gpflow.models.SGPR(X, Y, gpflow.kernels.RBF(1), Z=X.copy())
gpflow.train.ScipyOptimizer().minimize(m, maxiter=10)
saver= gpflow.saver.Saver()
saver.save('/tmp/model', m)
m.read_trainables()['SGPR/kern/variance']
> array(2.64659385)

next session:
m =  gpflow.saver.Saver().load('/tmp/model')
m.read_trainables()['SGPR/kern/variance']
> 2.64659385

PS: @doktorneergaard, could you post next time MWE, so that we could copy&paste your code. Thank you!

What?! This is amazing! I didn't even know that we had this!

@markvdw No worries, thank you for your detailed responses!
@awav This is exactly what I needed, thank you!. Tested and works for my purpose. Sorry for not posting a proper MWE.
@markvdw, @markvdw This would be great to have in the documentation!


BUG: Local quotes no longer working

Local quotes no longer works on the latest obsidian insider build

Thank you for noticing. I have no free time and can't promise soon update, but it will be fixed

it works, thanks a lot!!


File Service - Files

I would like to trigger a "File created" event on Azure Files. Is this possible?

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

ID: b7ee5166-ff40-f4ed-ce07-5312f935e388
Version Independent ID: 68f01b43-e3b2-003b-24cb-446a08aa0c1b
Content: Create event-based triggers in Azure Data Factory
Content Source: articles/data-factory/how-to-create-event-trigger.md
Service: data-factory
GitHub Login: @douglaslMS
Microsoft Alias: douglasl

Hi @maecky Thank you for your feedback! We will investigate and get back to you with our findings.

@maecky only blob storage is supported with event-based triggers at this time.


Bottom popover arrow color

First of all, I want to thank the creator and the contributors for this great library.
About the issue, when a bottom popover is shown, its arrow should have the same color of the header, just like in the original popover.
Curiously, I could not found any Bootstrap adaptations (for Angular, Vue or React) that didn't have this issue, so I'm not sure if it's a common difficulty.
Also, if it's a complicated one, I'm willing to help, but I'm not sure where to start, so I'd be glad if someone could me give directions here, so I could try something out.
Thanks in advance!

This is an issue with bootstrap itself as you can tell from the latest v4 documentation: http://getbootstrap.com/docs/4.0/components/popovers/
The documentation you linked to is from bootstrap v4 alpha, which is now dated. bootstrap v4 beta suffer from the issue you are seeing.

I see... thanks for the clarification! :1st_place_medal:
I'm closing this since it's not related to Reactstrap itself.


[question] How to use from VueX store ?

I can't seem to be able to create notifications from the VueX store, how do you do that ?

nevermind, found out :
            this._vm.$toasted.show(params['data']['message'])


Update input streams of already running / existing stream

Use case: Hatchery
In relation with #23, the user can update the stream description on re-launch. We have to decide on behavior.

Consider as part of #47

As of current master, it is possible to edit the Stream/Call's Execution's parameters manually (i.e. directly through Ignite), however there is no code handling such changes to parameters and the new values won't be used until the agent container is restarted.
As Hatchery seems to be working alright at the moment without this feature and #47 was implemented, is it still needed?
One way something like this could be implemented without additional Perper functionality is by creating a blank stream of e.g. stream arrays in the caller and subscribing to all the streams from the last item in the receiver; likewise for other types of parameters.


502 error occurred while creating pull request

Description
Logs:
2023/07/19 10:29:00 ...es/pull/temp_repo.go:98:createTemporaryRepoForPR() [E] Unable to init tmpBasePath for <PullRequest [0]Jack/HBERP#0[dev...feature/rxorder]>: context canceled
2023/07/19 10:29:00 ...vices/pull/update.go:172:GetDiverging() [E] CreateTemporaryRepoForPR <PullRequest [0]Jack/HBERP#0[dev...feature/rxorder]>: context canceled
2023/07/19 10:29:00 ...ers/web/repo/pull.go:1297:CompareAndPullRequestPost() [E] NewPullRequest: context canceled
2023/07/19 10:33:10 ...es/pull/temp_repo.go:98:createTemporaryRepoForPR() [E] Unable to init tmpBasePath for <PullRequest [0]Jack/HBERP#0[dev...feature/rxorder]>: context canceled
2023/07/19 10:33:10 ...vices/pull/update.go:172:GetDiverging() [E] CreateTemporaryRepoForPR <PullRequest [0]Jack/HBERP#0[dev...feature/rxorder]>: context canceled
2023/07/19 10:33:10 ...ers/web/repo/pull.go:1297:CompareAndPullRequestPost() [E] NewPullRequest: context canceled
After upgrading from version 1.19.3 to version 1.20, it still cannot be resolved
Gitea Version
1.20.0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.39.0, Wire Protocol Version 2 Enabled
Operating System
Windows Server 2012R2
How are you running Gitea?
SYSTEM
Prod
windows service
Database
MSSQL

Could you paste more logs before these logs line?

Sorry, Although I haven't found the reason, I have already solved this problem.
I no longer tried to use pull request , but instead solved it with picking.
which may still be a branch issue because only this branch has this error.


issue #187: Make AWS storage provisioning configurable via values.yaml file

Covering both: Besu Node and GoQuorum Node

Hi @joshuafernandes ,
please let me know if you see any issues with this PR.
Thank you!
Sergi


[Bug] Can't set falsy value with hook

Describe the bug
Can't set falsy value (false, empty string, number 0) with hook
To Reproduce
Steps to reproduce the behavior:

Clone my test case: https://github.com/pnthach95/mmkvtest
Press Set true or Set string or Set 1 button
Press Set false or Set empty string or Set 0 button
See warning Trying to set a null value to hook for type (boolean | string | number) is not allowed, values ain't changed

Expected behavior
Result text should show False, blank, 0
Screenshots

Platform Information:

OS: Android
React Native Version 0.65.1
Library Version 0.6.3

Additional context
N/A

I will investigate this

I tried, bug isn't fixed. I try set 0, empty string, it shows same warning, pull my test case for new cases. And check my PR #164.

@pnthach95 Thanks for the fix. I actually did make those changes but forgot to commit which is why it didn't work for you. I have merged now.


[ISSUE #6555]Fix DefaultMQProducer#request message callback timeout bug

Make sure set the target branch to develop
What is the purpose of the change

fix #6556
Brief changelog
-Fix DefaultMQProducer#request message callback timeout bug
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Could you please provide more information about this bug? or try to write a unit test to cover this bug?

Could you please provide more information about this bug? or try to write a unit test to cover this bug?

link #6555  I reorganized the questions


Don't put the postgres data on a separate volume

Description
This allows us to snapshot database state readily.
Test plan
building the docker image...

how does this allows us to snapshot database state readily?

You can't snapshot a volume in docker afaik.  This is basically a port of the same functionality I built for covidhub.


range_width having no effect in FeatureMatcher?

Hi, I'm trying to stitch a sequence of images that I know are adjacent, and I thought I understood setting range_width to 1 when matching might improve results, but it doesn't seem to have any effect. Perhaps I have misunderstood how this parameter works, but it doesn't seem to be doing anything from what I can tell.
I tried taking a fresh copy of Stitching Tutorial.ipynb and changing the range_width parameter in the matching section, and I still get a full matrix of confidence values for every pair of images no matter what value I choose – I expected that setting it to 1 would force it to only consider adjacent pairs (e.g. pair 1-2 would have a confidence but pair 1-3 would not). That's what the C++ code seems to be doing, but I admit I haven't dug deeply enough into it.
Here's my results:

You can see on the final block that under the hood the type is being set to cv2.detail.BestOf2NearestRangeMatcher correctly.
Do you have any suggestions, or have I misunderstood how this is supposed to work? If I manually set the confidence of non-adjacent images to zero then I get a better stitching result in later stages, but I was hoping for a significant performance increase of comparing fewer images too. Thanks!

Could you share the Images you are using?

OK, I had a first brief look, but I havnt worked with the RangeMatcher yet and dont quite understand why there are no changes. If you dug deeper please let me know.

Thanks for looking. I've been trying to do more digging. Honestly, I don't think this is an issue with stitching. I finally bit the bullet and set up a build of the C++ source code, and tried running stitching_detailed.cpp with the range_width parameter, and the BestOf2NearestRangeMatcher still seems to give me a full list of matches between images with confidence values, even with the range set to 1. I'll report back if I work anything out.

Rusty as my C++ is, I noticed that BestOf2NearestRangeMatcher::operator () didn't seem to be called at all in my tests.
At least for stitching_detailed.cpp, I believe the issue is simply that the function isn't labelled as virtual in matchers.hpp, since it does the following:
Ptr<FeaturesMatcher> matcher;
if (matcher_type == "affine")
    matcher = makePtr<AffineBestOf2NearestMatcher>(false, try_cuda, match_conf);
else if (range_width==-1)
    matcher = makePtr<BestOf2NearestMatcher>(try_cuda, match_conf);
else
    matcher = makePtr<BestOf2NearestRangeMatcher>(range_width, try_cuda, match_conf);

After marking the function as virtual it works correctly and the matrix of confidence values comes back with all zeros for non-index-adjacent images when range_width is 1.
I'm not sure how the Python bindings work under the hood though, since this is a C++ inheritance related issue. It seems odd, since stitching really calls cv2.detail_BestOf2NearestRangeMatcher directly. A bit like how avoiding polymorphism in the C++ code also works:
Ptr<BestOf2NearestRangeMatcher> matcher = makePtr<BestOf2NearestRangeMatcher>(range_width, try_cuda, match_conf);
But I guess this is just to do with the Python -> C++ layer. I'll post an issue over on the OpenCV GitHub.

Nice, thanks for your effort. Please link the issue If you don't mind!

OK I saw it xD
https://github.com/opencv/opencv/issues/22315

FWIW if you want to emulate the behaviour of the range based matcher, for now I've realised you can do so manually by doing the image pairing part yourself, calling .apply rather than .apply2, e.g. something like the code below (although if you have any idea how to more nicely create a deep copy of an OpenCV object, please let me know...!).
matcher = cv.detail_BestOf2NearestMatcher()
matches = []

for i in range(len(features)):
    for j in range(len(features)):
        if 0 < j-i <= range_width:
            match = matcher.apply(features[i], features[j])
            match.src_img_idx = i
            match.dst_img_idx = j
            matches.append(match)
        elif 0 < i-j <= range_width:
            match_to_copy = matches[j * len(features) + i]

            match = cv.detail.MatchesInfo()

            # swap src and dst, invert homography
            match.src_img_idx = match_to_copy.dst_img_idx
            match.dst_img_idx = match_to_copy.src_img_idx
            match.H = np.linalg.inv(match_to_copy.H)

            match.inliers_mask = match_to_copy.inliers_mask
            match.num_inliers = match.num_inliers
            match.confidence = match_to_copy.confidence

            dmatches = []
            for dmatch_to_copy in match_to_copy.matches:
                dmatch = cv.DMatch()
                dmatch.distance = dmatch_to_copy.distance
                dmatch.imgIdx = dmatch_to_copy.imgIdx
                # swap queryIdx and trainIdx
                dmatch.queryIdx = dmatch_to_copy.trainIdx
                dmatch.trainIdx = dmatch_to_copy.queryIdx

                dmatches.append(dmatch)

            match.matches = dmatches

            matches.append(match)
        else:
            matches.append(cv.detail.MatchesInfo())

Nice thank you! Do you need the functionality in stitching asap or could we wait another week or two If the issue is solved within OpenCV?

Btw do you think https://github.com/opencv/opencv/issues/20945 is similar? I havnt been able to check if its working in c++ but if yes it could be a wrapper issue as well

Nice thank you! Do you need the functionality in stitching asap or could we wait another week or two If the issue is solved within OpenCV?

No not at all, the workaround above is working for me for now, thanks!

Btw do you think opencv/opencv#20945 is similar? I havnt been able to check if its working in c++ but if yes it could be a wrapper issue as well

I'll take a look at this later and see what happens in the C++ version too.

Btw do you think opencv/opencv#20945 is similar? I havnt been able to check if its working in c++ but if yes it could be a wrapper issue as well

In short, yes! The output argument masks should be marked with CV_IN_OUT in the header file, but it isn't.
Your issue has been open so long I decided to just get the Python build working from the source code. Marking that argument fixes the issue and now gc_color and gc_colorgrad work correctly within stitching if I write e.g.:
seam_finder = SeamFinder()
seam_finder.finder = cv.detail_GraphCutSeamFinder('COST_COLOR_GRAD')

I've made a pull request for both fixes https://github.com/opencv/opencv/pull/22329

Thank you! This is something that has been bugging me for some time. Unfortunately I have zero C++ experience and am somewhat dependent on the Python wrappers and docs.

(p.s. if you do run into any more issues with the C++ code or Python bindings feel free to message me, can't promise anything but I can take a look!)

Thanks! BTW for which use case are you using the library? I'm always interested whats brings people here.

I taught some of the maths behind photo stitching at my old university lecturing job, so I was familiar with the ideas, and now just trying to put together some proof of concepts for an idea at work, but it's just for lining up photos really. Your library just makes it much easier than dealing with the raw OpenCV bindings!
By the way, I'll make a pull request on this repo (or maybe more than one) at some point soon with some minor ideas from what I've found playing around with settings.

Your library just makes it much easier than dealing with the raw OpenCV bindings!

Thanks! I'm happy I published it since it seems to help a bunch of other people.

By the way, I'll make a pull request on this repo (or maybe more than one) at some point soon with some minor ideas from what I've found playing around with settings.

🚀

with the release of OpenCV 4.7 your PR should now be in production. I'll have a look in the next days and also add a test for this.

Great news, thanks :)

Hi @chinery
Thanks for great contribute and research
I know matches_graph as below:
A --> B
A --> D
B --> C
B --> D
D --> E

Could you share how to stitch panorama when  knowing are adjacent?


This code is for a clock that uses an Adafruit TLC5947 LED driver, a real-time clock (RTC), a TMRpcm audio library, and an SD card. It sets the current time on the clock using the RTC and displays the time on the LEDs using the TLC5947. It also plays audio files from the SD card at specified times and has alarm functionality. The code uses a number of global variables to keep track of the current time and other settings, and it has a number of functions including setup() and displayupdate(). It should be noted that the code is incomplete, it's missing a lot of the functions and some of the lines are not complete.

This code is for a clock that uses an Adafruit TLC5947 LED driver, a real-time clock (RTC), a TMRpcm audio library, and an SD card. It sets the current time on the clock using the RTC and displays the time on the LEDs using the TLC5947. It also plays audio files from the SD card at specified times and has alarm functionality. The code uses a number of global variables to keep track of the current time and other settings, and it has a number of functions including setup() and displayupdate().
It should be noted that the code is incomplete, it's missing a lot of the functions and some of the lines are not complete.

Sounds like a chat bot?


Possible bug with GbmDetectorCollection

I tried to convert TTE detector data to a PHAII format so that I can group detectors for spectral fitting using GbmDetectorCollection. I follow the examples in the GBM Data Tools and GDT documentation https://fermi.gsfc.nasa.gov/ssc/data/analysis/rmfit/gbm_data_tools/gdt-docs/notebooks/SpectralFitting.html but get errors  when trying to fit a background to a list of detectors. However, working through the error seems to show an empty list after using GbmDetectorCollection. The conversion beforehand from TTE to PHAII seems to work fine, but the collection element does not seem to populate a list after. I could be doing something wrong, but I am struggling to see the issue and would appreciate some feedback.  I am using a MacOS (Monterrey 12.6.5). Attached is a screenshot of the executed code.

I believe the bug is related to this TODO.  Essentially, the detector name is not copied over from the TTE when converting to a PHAII.  This can be confirmed by creating a collection with native CTIME or CSPEC data.

@AdamGoldstein-USRA Thanks for implementing a fix. I tested the fix-tte-to-phaii-detector-name branch and found it worked successfully. Consider the bug resolved. Thank you!


Most reliable way of executing a command forcing a timeout

I am using sh.sudo() under Debian to execute administrative commands. However, some of these may hang for a long time, hence I want to add a timeout protection to the execution.
Is the _timeout parameter reliable for this is use case? Even if the command creates additional (child) processes? What would we your suggested approach?

This issue went under my radar and I'm sure you've moved on past it by now, but in case anyone else is searching for a similar question, yes this is the intended use of _timeout


UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1

Hey,
i just got this error while running for some time
UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128) 2016-08-10 16:29:29,642 [sentry.errors.uncaught] [ERROR] [u'ConnectionError: (\'Connection aborted.\', BadStatusLine("\'\'",))', u'  File "pokecli.py", line 530, in <module>', u'  File "pokecli.py", line 96, in main', u'  File "pokemongo_bot/__init__.py", line 450, in tick', u'  File "pokemongo_bot/health_record/bot_event.py", line 65, in heartbeat', u'  File "pokemongo_bot/health_record/bot_event.py", line 78, in track_url', u'  File "requests/api.py", line 111, in post', u'  File "requests/api.py", line 57, in request', u'  File "requests/sessions.py", line 475, in request', u'  File "requests/sessions.py", line 585, in send', u'  File "requests/adapters.py", line 453, in send']

Are you running the latest dev version?

yes same just happened

My guess would be an API error, how should I go about reproducing the effect?

yes, latest dev branch.
where is the logs file? i can share it with you or anything else i can do?

Not sure of the bot outputs logs by default, truthfully. Id recommend setting "health_record": false, in your config, that may solve your issue

ok, i will try, thnx.
it's not urgent bug because after restart it keep on running just thought it might be good to have a record of it.

I get the same error. Sometimes the bot can go on for 10 hours before the bug occurs, some times it's just a matter of minutes.

@Lordedi attach your log

[17:28:34] Going to a new place, ignoring cached location.
[17:28:34] Login procedure started.Traceback (most recent call last):
File "pokecli.py", line 530, in
main()
File "pokecli.py", line 138, in main
report_summary(bot)
File "pokecli.py", line 147, in report_summary
metrics.capture_stats()
File "/home/raviss_e/PokemonGo-Bot/pokemongo_bot/metrics.py", line 76, in capture_stats
response_dict = request.call()
File "/home/raviss_e/PokemonGo-Bot/pokemongo_bot/api_wrapper.py", line 90, in call
if not self.can_call():
File "/home/raviss_e/PokemonGo-Bot/pokemongo_bot/api_wrapper.py", line 58, in can_call
raise NotLoggedInException()
pgoapi.exceptions.NotLoggedInException
(I changed health_record: false)

no one has solved this issue ?

NotLoggedIn is usually called when you've entered some part of your login information wrong, or the server is abnormally busy

My login is good maybe server is busy.
Thank you to reply me mister

Also, just to be sure. You've used "pip install -r requirements.txt --upgrade" when you run new versions of the bot right? The --upgrade tag is important

Yes I did.

This happens when the servers are busy and your bot disconnects. Try using my macOS script
PokeBot.zip. It's code taken from the Wiki, edited, and made to restart the bot after 5 seconds if it crashes. You can edit the script using TextEdit or Atom (basically, anything that can edit text/scripts. But ScriptEditor does not work). If you want one on Windows just copy/paste that into a .txt and save it as a .bat

ok thanks I will try this ;)

Still doesn't work

OP is due to a connection error, not related to the bot


Custom ObservableObject to fix animation

Do you guys mind testing this out. Got it working on my end. Not sure if it's a good idea though. There might be a better way to achieve the same thing as well.

@mbrandonw @stephencelis marking as draft for now. Will test this out more over the weekend.

@mbrandonw @stephencelis have you been able to get bindings working properly inside TextField. I can't seem to do things like state.text = String(state.text.prefix(5)). I know this is a bit of SwiftUI issue though... was just curious...

Hm, this was auto-closed when the upstream branch merged. Wanna reopen against main?

@mbrandonw @stephencelis have you been able to get bindings working properly inside TextField. I can't seem to do things like state.text = String(state.text.prefix(5)). I know this is a bit of SwiftUI issue though... was just curious...

Once upon a time doing WithViewStore(self.store, removeDuplicates: { _, _ in false }) { … } helped here, though I think even that doesn't work anymore. I think it's probably better to validate on de-focus these days.

There's no Composable Formatters in the pipeline is there?...
 As in the Binding State ends up out of sync with the store.

@iampatbrown Can you illustrate the problem? We're not sure it's the binding getting out of sync with the store so much as the TextField's state, which could be modeling itself internally in a different way. We haven't dug too deep into it.
As for composable formatters, you could say that ;) Bidirectional parsing is a formatter in disguise, so one could be passed to the formatter-based TextField initializer. That works differently, though, as it allows any input and only does the parsing/printing when the value is committed on de-focus.

Yeah it's definitely TextField's internal state. For example in 0159-safer-conciser-forms-pt2 there's
case .binding(\.$displayName):
    state.displayName = String(state.displayName.prefix(16))
    return .none

I've tweaked the code slightly to get it to work as .binding() at the end of the reducer is going to override this state change... but you end up in a situation like this
Profile
My Really Long Display Nam

And then I press e which changes the view to
Profile
My Really Long Display Name

and the debug output is
received action:
  SettingsAction.binding(
    BindingAction.set(
      WritableKeyPath<SettingsState, BindableState<String>>,
      "My Really Long Display Name"
    )
  )
(No state changes)

It just seems like a fairly common thing people might want to do. So I was thinking it would be good to have a recomended way of handling it.

Actually the following seems to work fine.
struct SettingsState: Equatable {
  @BindableState var displayName = ""
}

enum SettingsAction: BindableAction, Equatable {
  case binding(BindingAction<SettingsState>)
  case validate(PartialKeyPath<SettingsState>)
  case resetButtonTapped
}

let settingsReducer = Reducer<SettingsState, SettingsAction, Void> { state, action, _ in
  switch action {
  case .validate(\.displayName):
    state.displayName = String(state.displayName.prefix(6))
    return .none
  case .binding:
    return .none
  case .validate:
    return .none
  case .resetButtonTapped:
    state = .init()
    return .none
  }
}.binding()

struct ContentView: View {
  let store: Store<SettingsState, SettingsAction>

  var body: some View {
    WithViewStore(self.store) { viewStore in
      Form {
        Section(header: Text("Profile")) {
          TextField("Display name", text: viewStore.$displayName)
            .onChange(of: viewStore, keyPath: \.displayName, send: .validate(\.displayName))
        }

        Button("Reset") {
          viewStore.send(.resetButtonTapped)
        }
      }
      .navigationTitle("Settings")
    }
  }
}

struct OnViewStoreChangeModifier<State, Action, LocalState>: ViewModifier where LocalState: Equatable {
  let viewStore: ViewStore<State, Action>
  let toLocalState: (State) -> LocalState
  let action: Action

  func body(content: Content) -> some View {
    content.onChange(of: toLocalState(viewStore.state)) { _ in
      viewStore.send(action)
    }
  }
}

extension View {
  func onChange<State, Action, LocalState>(
    of viewStore: ViewStore<State, Action>,
    state toLocalState: @escaping (State) -> LocalState,
    send action: Action
  ) -> some View where LocalState: Equatable {
    self.modifier(
      OnViewStoreChangeModifier(
        viewStore: viewStore,
        toLocalState: toLocalState,
        action: action
      )
    )
  }

  func onChange<State, Action, LocalState>(
    of viewStore: ViewStore<State, Action>,
    keyPath: WritableKeyPath<State, LocalState>,
    send action: Action
  ) -> some View where LocalState: Equatable {
    self.modifier(
      OnViewStoreChangeModifier(
        viewStore: viewStore,
        toLocalState: { $0[keyPath: keyPath] },
        action: action
      )
    )
  }
}

Hadn't thought about using
case validate(PartialKeyPath<SettingsState>)

Might be better to pass the Store though...
TextField("Display name", text: viewStore.$displayName)
  .onChange(of: self.store.scope(state: \.displayName), send: .validate(\.displayName))

struct OnViewStoreChangeModifier<State, Action>: ViewModifier where State: Equatable {
  let store: Store<State, Action>
  let action: Action

  func body(content: Content) -> some View {
    WithViewStore(self.store) { viewStore in
      content.onChange(of: viewStore.state) { _ in
        viewStore.send(action)
      }
    }
  }
}

extension View {
  func onChange<State, Action>(
    of store: Store<State, Action>,
    send action: Action
  ) -> some View where State: Equatable {
    self.modifier(OnViewStoreChangeModifier(store: store, action: action))
  }
}

Actually.... I'd probably go for something like this...
WithCheckedBinding(viewStore.$displayName) {
  TextField("Display name", text: $0)
}

public struct WithCheckedBinding<Value, Content>: View where Value: Equatable, Content: View {
  @Binding var base: Value
  @State var expected: Value
  let content: (Binding<Value>) -> Content

  public init(_ base: Binding<Value>, @ViewBuilder content: @escaping (Binding<Value>) -> Content) {
    self._base = base
    self._expected = State(initialValue: base.wrappedValue)
    self.content = content
  }

  var checkedBinding: Binding<Value> {
    Binding { expected }
    set: { newValue, transaction in
      expected = newValue
      $base.transaction(transaction).wrappedValue = newValue
    }
  }

  var isValid: Bool { base == expected }

  public var body: some View {
    content(checkedBinding)
      .onChange(of: isValid) { isValid in
        if !isValid { expected = base }
      }
  }
}

That way the logic can stay inside the reducer.


Find a way to handle errors when doing multiple reductions

Current implementation of SPHERE.Dataset will step when an error occurs in one of the reductions

could this be avoided?
do we need to implement a success flag in the reductions?
if yes, at which point is a reduction considered a success?

No exception should be raised but simply the errors should be logged. At the end of the processing of a reduction object, the log should indicate if the reduction succeeded or failed

Implemented in 8f9e84b. Closing ticket


There is an extra scroll bar in the 'Upload' dialog for one emulator blob container

Storage Explorer Version: 1.24.0-dev
Build Number: 20220401.7
Branch: main
Platform/OS: Windows 10
Architecture: ia32
Azurite Version: 3.16.0
Language: Czech/German/Spanish/French /Italian/Japanese/Dutch Polish/Portuguese/Russian/Swedish
How Found: From running test cases
Regression From: Previous release (1.21.3)
Steps to Reproduce

Install and run Azurite 3.16.0 -> Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select 'Deutsch' -> Restart Storage Explorer.
Expand 'Local & Attached -> Storage Accounts -> (Emulator - Default Ports) (Key)' -> Blob Containers.
Create a blob container -> Click 'Upload -> Upload Folder/Files...'.
Check there is no extra scrollbar in the 'Upload' dialog.

Expected Experience
There is no extra scrollbar.

Actual Experience
There is an extra scrollbar.

We should have the upload dialog be the same width regardless if the container is emulator or not....

Verified this issue on the build main/20220610.4. Fixed.


Fix updated fields array example syntax

I got the following error with your actual example:
$ bundle exec cap production deploy
(Backtrace restricted to imported tasks)
cap aborted!
SyntaxError: config/deploy.rb:35: syntax error, unexpected ':', expecting =>
    "title": "Project",
            ^
config/deploy.rb:35: syntax error, unexpected ',', expecting end-of-input

Tasks: TOP => production
(See full trace by running task with --trace)

What version of Ruby are you running?

Doh. I misread this the other day. My bad. Will merge soon. Thanks!


tmux: reorder tmux.conf -> put extraConfig in the end

This allows overriding of every tmux.conf setting via extraConfig, even plugin settings.
See also #875.

@kalbasit Do you have time to have a quick look at this? I don't really know anything about tmux 🙂

I just pushed a new version where I

split the commit into the real change and the refactoring
rebased it  this onto #1195. This solves the merge conflict with master which had arisen.

Checklist
(does not cover 1fd475c, bcf8aff)

[x] Code formatted with ./format.

[ ] Code tested through nix-shell --pure tests -A run.all.

[x] Code tested through tmux related tests:
nix-shell --pure tests -A list \
| sed -nE 's/(.*tmux.*)/run.&/p' \
| xargs -i nix-shell --pure tests -A '{}'

[x] Test cases updated, not added.

[x] Commit messages are formatted like
{component}: {description}

{long description}

rebased onto master after #1195 was accepted
edited last note accordingly

Thanks! Rebased to master in a7cdfaa32585fce404cf8890bae099348e53e0ad.


Mutate and Create Fields When Sending to Algolia

Hello! We currently have some cloud functions that mutates our Algolia indices when relevant documents are created, updated, or deleted, and we are excited to discover that this Firebase extension can help remove some of this code. However, one of our indices change one field to a more search-friendly form, i.e. turning a serialized JSON of rich text into plaintext. We would like to move all of our function code to using this extension, but this is currently blocking us from doing so. Thanks!

hey Peter, I want to better understand your use case.  Are you looking to inject your logic to post process fields values before it is indexed into Algolia?

hi! thanks so much for getting back to me, that is indeed my use case.

I will have to do some R&D on how to call external functions.

hey @PeterCardenas, created a PR for the post processing.


Cow: Watch universalheatingsolutions.co.uk

Cow requests the watch of the watch_keyword universalheatingsolutions\.co\.uk. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

universalheatingsolutions\.co\.uk has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


(cleanup) Remove application from deployment steps. Fix #221

Fix #221
We are now move away from using application.

/lgtm


[Backport 1.15][FLINK-26028] Add pulsar sink documentation

This is a backport PR for https://github.com/apache/flink/pull/19056

@flinkbot run azure


Git clone fix to use basename

Fixes #59544
leverage path to get basename regardless of the string url parameter.
Fix for network drives as relative paths, and will also work with real URI's.

Thanks! 🍻


Ignore Warning

Not sure if this is a content or product feedback. Probably depends on the answer :) In traditional FXCop, you can exclude a specific instance of a warning and provide a reason why you did it. Is that possible with the AL analyzers? In particular, the "Possible overflow assigning Text to Text[250]" is logged even when you are explicitly using CopyStr to only assign the MaxStrLen of the field in question, so it's a handled warning. Over time, you project tends to fill up with dozens of these false positives making it harder to find the real issues. I do see that it's possible to ignore the overall rule using a ruleset, but that's not particularly ideal either. As in the example above, you would want to know if there actually was a possible overrun, but then ignore the warning once you're sure it isn't a problem.

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

ID: 4a5b68ec-d706-ab10-8cb9-22e8d98c3494
Version Independent ID: 3da84c14-159f-3ac7-ee94-5e0faff222d8
Content: Using the Code Analysis Tool - Business Central
Content Source: dev-itpro/developer/devenv-using-code-analysis-tool.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hi, thanks for your feedback. The issue that you mention has been improved with an update of the AL Extension, not sure if you have access to that update. With regards to support for excluding specific instances, I am sorry but this is not (yet) supported. Thanks.


Your browser sent a request that this server could not understand

I'm using client = Session(impersonate='chrome110')
This error raised when i'm doing POST request without data, while other request like GET and POST with data are working normally.

<TITLE>Bad Request</TITLE>

Bad Request

Your browser sent a request that this server could not understand.

Reference #7.4a1d770.1689773281.2509c458

Then i change to httpx, with client = Client(http2=True), it works.
But i want to keep using curl_cffi because some of time they block my httpx request. I've read on the internet that the error is because the headers content is too large. I can't give reproduction code because it needs a login. Is there a way to modify the headers using Session?

requests.post(url)

Any particular headers or cookies are you using? It would be very helpful if you could use Charles or Fiddler to capture the 400 request and successful request and post them here.

The response is HTTP 411 : Length Required.
I already fixed it by adding 'Content-Length': '0' to the header. I think  curl_cffi not generating Content-Length header in the POST request without data, while httpx does.
Thanks for the reply, i'll close this. :)

Thanks for reporting back your findings, I will add that header for empty bodies.


Add requester pays support

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

Linking to #148, since apparently titles don't get the same treatment as comments

Thanks for the style comments @dannycjones - I committed all of the changes you suggested!

Otherwise, this should be ready to merge? @monthonk @dannycjones

Rebase complete! @monthonk

Merged - thanks for the contribution!


AtFiles

LeoJS Milestone: Scripting and 'execute-script' command works! (With or without text selection, and/or with @others and children outline!)

@boltex Congratulations!


Add "noob-trap" to Glossary

Possible definition:

Features which exist in a game, but are anti-useful to use. Using them either only harms the player, or develops usage habits which are counter-productive.

We might want to provide alternative words for use.

Features that seem helpful or appealing to beginners, but actually aren't. Using them is less effective in almost every circumstance than an alternative, or develops bad habits.


Issue with seg_report when using --skull-strip-t1w skip

What version of fMRIPrep are you using?
20.1.0rc1
What were you trying to do?
run fMRIPrep on skull stripped images
exception calling callback for <Future at 0x7f6aa5488ef0 state=finished raised FileNotFoundError>
concurrent.futures.process._RemoteTraceback:
"""
Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/traits_extension.py", line 128, in validate
    value = Path(value)  # Use pathlib's validation
  File "/usr/local/miniconda/lib/python3.7/pathlib.py", line 994, in __new__
    self = cls._from_parts(args, init=False)
  File "/usr/local/miniconda/lib/python3.7/pathlib.py", line 649, in _from_parts
    drv, root, parts = self._parse_args(args)
  File "/usr/local/miniconda/lib/python3.7/pathlib.py", line 633, in _parse_args
    a = os.fspath(a)
TypeError: expected str, bytes or os.PathLike object, not list

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 486, in run
    self._get_hashval()
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 538, in _get_hashval
    self._get_inputs()
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 609, in _get_inputs
    self.set_input(key, deepcopy(output_value))
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 302, in set_input
    setattr(self.inputs, parameter, deepcopy(val))
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/traits_extension.py", line 329, in validate
    value = super(File, self).validate(objekt, name, value, return_pathlike=True)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/traits_extension.py", line 130, in validate
    self.error(objekt, name, str(value))
  File "/usr/local/miniconda/lib/python3.7/site-packages/traits/trait_handlers.py", line 172, in error
    value )
traits.trait_errors.TraitError: The 'in_file' trait of a _ROIsPlotInputSpecRPT instance must be a pathlike object or string representing an existing file, but a value of "['/scratch/fmriprep_wf/single_subject_RTC01_wf/anat_preproc_wf/n4_only_wf/inu_n4_final/mapflow/_inu_n4_final0/sub-RTC01_ses-func02_T1w_valid_corrected.nii.gz']" <class 'str'> was specified.

Thanks for the report (and using the release candidate)!
I think I see the problem, I'll push a second rc this week to see

@andrew-stier I've pushed a new pre-release 20.1.0rc2 that hopefully fixes the problem. Can you try that version and report back?

will do

@mgxd I got this error running with 20.1.0rc2:
200410-08:41:14,880 nipype.workflow CRITICAL:
fMRIPrep failed: Does not support two inline functions in series ('def _pop(in_files):
if isinstance(in_files, (list, tuple)):
return in_files[0]
return in_files
'  and 'def _pop(inlist):
if isinstance(inlist, (list, tuple)):
return inlist[0]
return inlist
'), found when connecting fmriprep_wf.single_subject_RTC01_wf.anat_preproc_wf.n4_only_wf.inu_n4_final to fmriprep_wf.single_subject_RTC01_wf.anat_preproc_wf.anat_norm_wf.inputnode. Please use a Function node.

thanks for following up on this - a (real) fix is in the works.

thanks for following up on this - a (real) fix is in the works.

Is this done?

it should be resolved - @andrew-stier when you get a chance, could you try with the master branch (poldracklab/fmriprep:unstable image)?

@andrew-stier, please reopen if this has not been resolved. Thanks for posting!


Update handlebars

Update handlebars library, freezedev/lyia-template needs to be updated as well

Oh yeah, this is done. Happened a few weeks ago.


Relax dependency versions?

Hi @bauerdavid ,
first of all congrats to this great plugin! Here in Dresden many people are using it. I would like to add it to a plugin-collection: devbio-napari to ship it together with other plugins to collaborators. However, there is a tiny issue with pinned dependencies in napari-nd-annotator, which limits who can install it.
Would it be possible to change napari == 0.4.15 to napari <= 0.4.15 in these two places?

https://github.com/bauerdavid/napari-nD-annotator/blob/a4caf6adfb325755ec00aabf4ed2abf5033d5b69/setup.cfg#L44
https://github.com/conda-forge/napari-nd-annotator-feedstock/blob/a2267b9002eb70df8d2876bd7fb647574966fafa/recipe/meta.yaml#L24

Furthermore, would it be possible to change vispy == 0.9.6 to vispy <= 0.9.6 in this place?

https://github.com/bauerdavid/napari-nD-annotator/blob/a4caf6adfb325755ec00aabf4ed2abf5033d5b69/setup.cfg#L45

Thanks for considering! Let me know what you think.
Best,
Robert

Hey @haesleinhuepf,
thank you very much, I'm glad you can put the plugin to use! :)
The napari version was fixed to 0.4.15 due to some problems arising when using 0.4.16. However, as napari will continue to evolve, this plugin should follow these changes as well. Also I already had some discussion about that version fix with some other users. So I will soon take a look at this issue and make it work with newer napari versions as well.
As for vispy, I decided to fix its version because of https://github.com/napari/napari/issues/4415 (I just realized that you were the one to post this issue! I was sure that I've seen your name somewhere 😄). I will check that also when the plugin is compatible with the new napari version.

Hi @bauerdavid ,
just one thing because I'm not sure if I was clear. I hope that your plugin supports older versions (napari<=0.4.15), because the devbio-napari distribution I mentioned also doesn't support 0.4.16. Thus, I don't care much about 0.4.16, I care more about 0.4.11-0.4.15 because they are quite common in the field. The problems in 0.4.16 will hopefully go away with 0.4.17 at some point ;-)
Thanks!
Robert

As for vispy, I decided to fix its version because of napari/napari#4415 (I just realized that you were the one to post this issue! I was sure that I've seen your name somewhere 😄). I will check that also when the plugin is compatible with the new napari version.

That's why I have this in devbio-napari:
https://github.com/haesleinhuepf/devbio-napari/blob/master/setup.cfg#L58-L59

Yup, you were clear, but of course I misread it... Unfortunately if I recall correctly, there were some functionalities which depend heavily on newer versions of napari. But I will check whether that's possible.

No pressure. Thanks for the feedback! 🌞

I reverted napari to 0.4.11, but didn't spend too much time with it, because I realized that supporting all these versions of napari might(!) require a huge amount of work, mostly because of the bounding box layer.
Between different releases napari sometimes changes its package structure, and (of course) built-in layers are changed accordingly, not to mention other changes in these layers. If we would want to make the BoundingBoxLayer compatible with every version of napari since 0.4.11, this might require a lot of version specific code (probably implemented as multiple-branch 'if-elif'-s). But note that I did not do a detailed analysis, this is rather a gut feeling.
I'm afraid I won't be able to make this happen. But if you or anyone else wanted to take a look at this and check if it's feasible, I would be more than happy to collaborate 🙂
As for vispy, I am thinking maybe the easiest solution would be to lift the version restriction, as it is not related to the plugin, but to napari, and having vispy<=0.9.6 would cause conflicts with later versions of napari.

I'm afraid I won't be able to make this happen.

No problem. I really didn't want to create any pressure.

and it actually did really take making some 'if-else'-s and version checking.

Just out of curiosity: did you upload that code to GitHub? I would like to take a look.
Thanks again for considering!

Sure! This is the commit: https://github.com/bauerdavid/napari-nD-annotator/commit/da82a5fb4d7a178996716a8b47ba52d8983fb07e

Hey @haesleinhuepf,
the new version of the plugin is out. As mentioned in my previous comment, it works with napari>=0.4.11, except for the bounding box functionality, which is available from 0.4.15. If you install the plugin as before (plugin manager or pip), than by default bounding boxes will not be available. Bounding box layer can be installed in multiple ways:

install the annotator as pip install napari-nD-annotator[all]
install the napari-bbox plugin either from pip or the plugin manager
Feel free to try it!

Best wishes,
David

Awesome, big thanks for working on this @bauerdavid !


Support use of output from synth step as input in later ShellSteps

I would expect this code to work:
    const pipeline = new GitHubWorkflow(app, 'Pipeline', {
      workflowPath: `${dir}/.github/workflows/deploy.yml`,
      synth: new ShellStep('Build', {
        installCommands: ['yarn'],
        commands: ['yarn build'],
      }),
      jobSettings: { if: 'check on Synthesize and Publish Assets' },
    });

    const stage = new Stage(app, 'MyStage', {
      env: { account: '111111111111', region: 'us-east-1' },
    });

    new Stack(stage, 'MyStack');

    pipeline.addStageWithGitHubOptions(stage, {
      pre: [
        new ShellStep('Extra-Shell-Step', {
          additionalInputs: {
            'cdk.out': pipeline.cloudAssemblyFileSet,
          },
          commands: ['ls -al cdk.out'],
        }),
      ],
    });

The resulting yaml file doesn't work, since the uploaded artifact is named "cdk.out" but the id used as the name from the pipeline.cloudAssemblyFileSet is `"Output":
      Build-Build:
        name: Synthesize
        if: check on Synthesize and Publish Assets
        permissions:
          contents: read
          id-token: none
        runs-on: ubuntu-latest
        needs: []
        env: {}
        steps:
          - name: Checkout
            uses: actions/checkout@v2
          - name: Install
            run: yarn
          - name: Build
            run: yarn build
          - name: Upload cdk.out
            uses<EMAIL_ADDRESS>            with:
              name: cdk.out # ← here ⚠
              path: cdk.out

And later:
      MyStage-Extra-Shell-Step:
        name: Extra-Shell-Step
        if: check on Synthesize and Publish Assets
        permissions:
          contents: read
        runs-on: ubuntu-latest
        needs:
          - Build-Build
        env: {}
        steps:
          - uses: actions/download-artifact@v2
            with:
              name: Output  # ← here ⚠
              path: cdk.out
          - run: ls -al cdk.out

The name Output is coming from @aws-cdk/pipelines/lib/blueprint/shell-step.ts:
    if (props.primaryOutputDirectory) {
      this._primaryOutputDirectory = props.primaryOutputDirectory;
      const fileSet = new FileSet('Output', this);  // ← here ⚠
      this.configurePrimaryOutput(fileSet);
      this.outputs.push({ directory: props.primaryOutputDirectory, fileSet });
    }

Perhaps I'm wrong, but it appears that by using this.cloudAssemblyFileSet.id instead of CDKOUT_ARTIFACT here and here this would work as expected.

I forgot to add that I already have a PR for this teed up as well, just waiting to hear feedback in case it needs adjusted or maybe there's a bigger plan that I'd be invalidating here.


SocialConnect ready for Cel2

SocialConnect should keep working almost transparently.
### Tasks
- [ ] https://github.com/celo-org/social-connect/issues/282
- [ ] https://github.com/celo-org/social-connect/issues/283
- [ ] https://github.com/celo-org/social-connect/issues/284
- [ ] https://github.com/celo-org/celo-blockchain-planning/issues/203
- [ ] https://github.com/celo-org/celo-blockchain-planning/issues/204
- [ ] https://github.com/celo-org/celo-blockchain-planning/issues/201
- [ ] https://github.com/celo-org/celo-blockchain-planning/issues/267
- [ ] https://github.com/celo-org/social-connect/issues/285

@alecps @soloseng
Can you please take a look at the tasks?
My understanding is that there's nothing to do for Alfajores L2. There's only 2 things i imagine make sense:

Do a quick test of using ODIS pointing to Danjo, to just check it works. Should basically be the same that checks alfajores but pointin to danjo RPC
Double check contracts to see if there's something to change. Or document.

What i see is too many tasks that makes it unclear what this really is.
Am i mistaken?
Also if there are thing that needs to be post-deployment, let's create another isseu for that

I agree with @mcortesi , I didn't write any of these tasks and don't understand the reasoning behind most of them. ODIS depends on Forno, so this is really just a task to make sure forno transitions endpoints cleanly

Ideally, services relying on forno should be able to keep the same URL, and forno will switch to the L2 network when the migration happens so ODIS will not need to do anything
State will be migrated to the L2, so we don't need to check that contracts are migrated
No one uses Alfajores ODIS, so we can test this during the alfajores migration

Nothing changes for ODIS as a result of the L2 migration except for the full node it talks to

@lvpeschke I've removed all the extraneuous tasks from this epic. I'm not sure about the contractkit one. @soloseng can you add context there?

Hey all, could the tests on Dango happen ASAP to uncover any potential issues ASAP and decide how to tackle them, if any?

@lvpeschke for the tests to happen we need to point Alfajores ODIS to Dango, faucet some Dango accounts (I can ask devops for this) and then just to run our existing e2e tests against that instance.
This will cover

https://github.com/celo-org/social-connect/issues/284
https://github.com/celo-org/social-connect/issues/286

There are some other tickets here that I don't understand

https://github.com/celo-org/celo-blockchain-planning/issues/204
https://github.com/celo-org/celo-blockchain-planning/issues/203
https://github.com/celo-org/celo-blockchain-planning/issues/267

@martinvol do you have more context about the above?


Application Exception: Forbidden - The request is understood, but it has been refused or access is not allowed. An accompanying error message will explain why. This code is used when requests are being denied due to update limits.

This is an issue created automatically by GrammarNazi when an exception was captured.
Date (UTC): 06/14/2023 00:59:51
Exception:
Reason : Forbidden
Details : {"errors":[{"message":"You currently have access to a subset of Twitter API v2 endpoints and limited v1.1 endpoints (e.g. media post, oauth) only. If you need access to this endpoint, you may need a different access level. You can learn more here: https://developer.twitter.com/en/portal/product","code":453}]}
(0)
Code : 403
Date : 06/14/2023 00:59:51 +00:00
URL : https://api.twitter.com/1.1/statuses/mentions_timeline.json?count=200&since_id=1664594625053724673&tweet_mode=extended
Twitter documentation description : Forbidden - The request is understood, but it has been refused or access is not allowed. An accompanying error message will explain why. This code is used when requests are being denied due to update limits.
Exception caught counter: 3282.

This is due to the new Twitter API pricing. We'll have to disable the Twitter bot :(


New BoundsError in tests

Not sure if this is spurious, or if it was introduced by one of your dependencies changing something? Ref http://pkg.julialang.org/detail/SimJulia.html

Hi
The error is introduced by a dependency. Compat or Distributions...
I will look at it this evening
Kind regards
Ben

On 18 Jun 2016, at 03:00, Tony Kelman<EMAIL_ADDRESS>wrote:
Not sure if this is spurious, or if it was introduced by one of your dependencies changing something? Ref http://pkg.julialang.org/detail/SimJulia.html
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

It is definitely a problem with Compat.
@compat get(env.active_proc) gives the error.
I isolated the problem with following small example:
`using Compat
type Test
a :: Float64
end
type TestCompat
t :: @compat Nullable{Test}
function TestCompat()
tc = new()
tc.t = @compat Nullable{Test}()
return tc
end
end
tc = TestCompat()
@compat get(tc.t)`
In Julia v0.4 without the @compat macro everything works fine (a NullException is thrown) but with the macro I get the following error message:
ERROR: LoadError: BoundsError: attempt to access 2-element Array{Any,1}:
:get
:(tc.t)
at index [3]
I will open an issue at JuliaLang/Compat.jl

I removed the @compat macro before get and this seems to work. A new version of SimJulia is tagged.


fix(deps): update icr.io/ibm-iac-charts/sysdig-deploy docker tag to latest

This PR contains the following updates:

Package
Type
Update
Change

icr.io/ibm-iac-charts/sysdig-deploy
helm_release
minor
1.61.10 -> 1.62.1

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This issue has been resolved in version 1.2.13 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: