Always call decoders

This reverts part of #62 to always process decoder interfaces for struct fields. This removes a breaking change where certain built-in types like net/url.URL have a custom unmarshaling function for the empty string.
The other rules for "overwrite" still apply.
Part of https://github.com/sethvargo/go-envconfig/issues/64

@williamgcampbell @twmb PTAL at this and let me know what you think. I added tests specifically from https://github.com/sethvargo/go-envconfig/issues/61 to show that the Zap use case is still preserved even after reverting this part of 62.

I'm not sure why a struct field should be treated differently from a typed non-struct field -- this preserves the zapcore.Level case, but only inadvertently. If zapcore.Level were defined as
type Level struct {
    v uint8
}

then the problem would emerge. From a consistency perspective, I'd expect decoders to either always be called, or never be called -- this seems to special case structs. In may case, I'm not adding ,default= to fields. For an example of if zapcore structured their code a bit differently:
diff --git a/envconfig_test.go b/envconfig_test.go
index 015250e..aff07d5 100644
--- a/envconfig_test.go
+++ b/envconfig_test.go
@@ -43,12 +43,14 @@ func (c *CustomDecoderType) EnvDecode(val string) error {
 }

 // Level mirrors Zap's level marshalling to reproduce an issue for tests.
-type Level int8
+type Level struct {
+       V int8
+}

-const (
-       DebugLevel Level = 0
-       InfoLevel  Level = 5
-       ErrorLevel Level = 100
+var (
+       DebugLevel Level = Level{0}
+       InfoLevel  Level = Level{5}
+       ErrorLevel Level = Level{100}
 )

 func (l *Level) UnmarshalText(text []byte) error {
@@ -2153,16 +2155,16 @@ func TestProcessWith(t *testing.T) {
                },
                {
                        // https://github.com/sethvargo/go-envconfig/issues/61
-                       name: "custom_decoder_overwrite_existing_value",
+                       name: "custom_decoder_not_overwrite_existing_value",
                        input: &struct {
-                               Level Level `env:"LEVEL,overwrite,default=error"`
+                               Level Level `env:"LEVEL,overwrite"`
                        }{
-                               Level: InfoLevel,
+                               Level: DebugLevel,
                        },
                        exp: &struct {
-                               Level Level `env:"LEVEL,overwrite,default=error"`
+                               Level Level `env:"LEVEL,overwrite"`
                        }{
-                               Level: InfoLevel,
+                               Level: DebugLevel,
                        },
                        lookuper: MapLookuper(nil),
                },

I get the failure:
$ go test
--- FAIL: TestProcessWith (0.00s)
    --- FAIL: TestProcessWith/custom_decoder_not_overwrite_existing_value (0.00s)
        envconfig_test.go:2272: mismatch (-want, +got):
              &struct{ Level envconfig.Level "env:\"LEVEL,overwrite\"" }{
            - 	Level: envconfig.Level{},
            + 	Level: envconfig.Level{V: 5},
              }
FAIL
exit status 1
FAIL	github.com/sethvargo/go-envconfig	0.280s

I think the main fix for #64 was #67 to propagate noinit. This reverts part of #62 so fix the breakage between 0.7.0 and 0.8.0, but I do still hold the opinion that non-existent env vars should not call custom decoders with empty values. This also makes it impossible to differentiate between an unset env var and an env var set to an empty string.

I do still hold the opinion that non-existent env vars should not call custom decoders with empty values.

I think I agree with you, but unfortunately users are depending on that current behavior and I don't want to introduce that subtly breaking change in a minor release. I've opened https://github.com/sethvargo/go-envconfig/issues/69 to track changing this in the 1.0 release.

I agree from the angle of the Hyrum's Law aspect, but another angle to it is that by keeping the current state, then more people will be dependent on the current behavior by v1, and this may make it even less compelling to change the behavior for v1. Changing the behavior now could be framed as a bugfix ;)


calculate_bending_strength results - coordinate axes

I find the way the results of "calculate_bending_strength" are presented strange. m_y and m_z follow the global axes defined in the section, but chi_y and chi_z follow the rotated 'theta' axes. Is this intentional? What is the reason for this? (See chi_z=0 for rotated n.a pi/6)
import math
from pprint import pprint

from shapely import Polygon

from structuralcodes.geometry import SurfaceGeometry
from structuralcodes.materials.concrete import ConcreteEC2_2004
from structuralcodes.materials.reinforcement import ReinforcementEC2_2023
from structuralcodes.sections._generic import GenericSection
from structuralcodes.sections._reinforcement import add_reinforcement_line

# Materials
concrete = ConcreteEC2_2004(25)
reinforcement = ReinforcementEC2_2023(
    fyk=500, Es=200000, density=7850, ftk=550, epsuk=0.07
)

# Create section
poly = Polygon(((0, 0), (350, 0), (350, 500), (0, 500)))
geo = SurfaceGeometry(poly, concrete)
geo = add_reinforcement_line(
    geo, (50, 450), (300, 450), 20, reinforcement, n=10
)
geo = add_reinforcement_line(geo, (50, 50), (300, 50), 20, reinforcement, n=10)
geo = geo.translate(-175, -250)
sec = GenericSection(geo)

# Draw & print
print('====== SECTION ======')
print('====== Theta = pi/6 ======')
pprint(sec.section_calculator.calculate_bending_strength(theta=math.pi / 6))

====== SECTION ======
====== Theta = pi/6 ======
UltimateBendingMomentResults(theta=0.5235987755982988,
n=-0.003881820593960583,
m_y=-537978185.0265349,
m_z=-64913391.04673183,
chi_y=-1.6762708318650637e-05,
chi_z=0,
eps_a=0.0015959697879269737)
Another topic that I found confusing is the sign convention for the moments. Normally, it is understood that a positive moment with a horizontal neutral axis compresses the lower block and causes tension in the upper block, but in this case, it doesn’t seem to be that way. I understand the reasoning, which is to be consistent with the coordinate axes (Y horizontal to the right and Z vertical upwards). I just think this should be explained in the documentation to avoid misunderstandings.
This leads me to think that when we use theta = 0 in this function, we get a negative moment (according to fib) and a positive one (according to the standard convention). In this way, theta = 0 leads to a negative m_y result (with an approximate phase shift of 180º between the resultant moments and the neutral fiber angle). Shouldn’t it be the case that when theta = 0, we get a positive My moment and an Mz close to zero? I’m not sure if I explained myself well or if what I’m saying makes sense to you. I hope I’m not causing confusion

What are your thoughts on the chi-values, @talledodiego?

 Thanks for having spotted this.
It was simply a bug, I returned the strain before rotating them back to the global crs.
Solved in PR #206


Fixes a SHA256 documentation error and adds details

Fixes #3495
Here is a first "draft" to fix this issue. The informations related to the SHA256 implementations have been taken from referencesource.microsoft.com and MSDN
Let me know if anything needs rework.

Closing and reopening to begin new build.

//cc @bartonjs

Closing and reopening to restart build.

@rpetrusha I pushed an update to the PR to fix the elements related to FIPS mode.

Thank you, @sputier, for making the additional corrections. We'll merge your PR now. Changes should appear on docs.microsoft.com in the next day or two.


Error failing to get MB artist ID

What version of Music Assistant has the issue?
2022.7b4
The problem
Still seeing this error occur.  https://github.com/music-assistant/hass-music-assistant/issues/565
Anthony Warlow does have a MB ID https://musicbrainz.org/artist/7ad0a863-e2e0-4038-be9b-02f4011ee348
How to reproduce
It only occurs on select artists.
Relevant log output
Logger: music_assistant.metadata
Source: /usr/local/lib/python3.10/site-packages/music_assistant/controllers/metadata/init.py:158
First occurred: 3:24:23 PM (3 occurrences)
Last logged: 4:01:21 PM
Unable to get musicbrainz ID for artist Anthony Warlow !
Unable to get musicbrainz ID for artist Genesis Lynea !
Unable to get musicbrainz ID for artist 12 Days !
Additional information
No response
What version of Home Assistant Core are your running
2022.7.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

While being logged as error, it is pretty harmless. It just means it couldn't grab the musicbrainz id at that moment with the combination of track/album is was processing at that time. The log level is a bit harsh atm but it helps me diagnosing how many times it is off. There's a good chance that it grabbed the musicbrainz id a few seconds later for another track.
Maybe you have a song by such an artist that has some incorrect tags. It sends the artist + album Nad/or track name to musicbrainz to get an id back.
I'll adjust the logging a bit so that will log at info level instead of warning and include some more info like what info it used to try to get the match. Maybe that helps in better finding out if there's an error in the code or in the tags.

Perfect sounds good

This one is fixed a few betas back.


readme updates

Updates to readme

New section for dependencies
More precise instructions on building petsc on systems with/without a batch queuing system

Dear Dave,
Congratulations! All your modifications to the readme passed the build tests.

Cool - it would have been sad if managed to break the build with my cosmetic changes :D

Just FYI: You can watch the build go down here: https://travis-ci.org/SalvusHub/salvus


Upgrade the version of sfuture.

It changes the order of result and isSuccess of onComplete callback.

LGTM

Sorry I changed PR again.
Now onComplete callback follows node convention.

Now it looks perfect. Ship it!


enhancement: Use cache feature in CI to avoid rebuilding dependencies

The Github Actions can be combined with a cache step to store folder and recover then in next build:
https://github.com/marketplace/actions/cache
As we are using submodules for external dependencies and Conan, it takes few minutes to build for every new CI push. It would be great if we could save that time by restoring cached binaries instead.

Good idea


Checkbox to Use CDN URLs for Media Library on Admin Pages

This is a complete rewrite of #336 (so it's not dependent on that PR) that moves away from using the wp_prepare_attachment_for_js() filter and attaches to wp_get_attachment_url and attachment_link filters instead.  It also includes a new checkbox allowing the user to specify if they want to use CDN links instead of local ones for Media Library content on admin pages.
The new checkbox is located within the CDN page under Advanced and looks like this:

This pull injects a performance penalty. A wordpress site, with a lot of files, this line (https://github.com/szepeviktor/w3-total-cache-fixed/pull/350/commits/9fa9e8ce6aaf3f89596db4088aa907802d6f3ed2#diff-b6574d852aff295ce887208a2c1d4d9dR774):
$allowed_files = $this->get_files();´
In our site, each page refresh is taking 6 seconds to complete, after using this commit (get_files enumerates all files)


Failed to upgrade from v1.8.0 to v1.9.0-rc

Describe the bug
https://buildkite.com/risingwave-test/upgrade/builds/41#018f659e-91e9-4f7f-8b6f-b4dcbb377421
After upgrade

Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
No response
The version of RisingWave
v1.9.0-rc
Additional context
No response

caused by #16090

postgresql backend is not supported to upgrade to v1.9.0


adding refs for navBar and footer

This way we can access to the navbar from an action using : this.props.navigator.refs.navBar

Still don't see how it could be accessed via name 'navBar'? Also why you need it?

Thank you for your answer.
By clicking on the "Next" button of the navbar, I want to retrieve the values of the switches of the page (see attached screenshot).
Since the navbar is created before the switches, I can not pass an onNext event containing the switches references when calling the action.
The only working way I found is to access this.props.navigator.refs.navBar inside the action.
Do you think I should use another way?

Your idea is not "Flux-like". All UI layer should do is just calling actions, UI should not have any logic.
You should define actions for each field and store which will listen them and change its state. "Next" will call some another action as well and that store will listen it and process it without problems - that store will have all values already, right?

Ok I understand now, thank you
I followed demo https://github.com/mrblueblue/react-native-alt-demo to store the values of the fields


Display seed used if specs are randomized

Jasmine allows to randomize tests with random and seed options. When seed is not provided it generates one. It can be useful to see what seed was used if spec suite fails and one wants to re-run it in the same order.
It shows exactly the same message as the default ConsoleReporter.

Great !

Available in 2.6.0.


Update dependencies to routerify v3

Hello,
This PR makes routerify-cors work with routerify v3. I also updated the "develop" branch, which was out of sync from master. Thanks a lot.

Hello @rousan, can you check the PR please? Thanks a lot.

Hi @aeyoll , I check it, everything is good. But the README.md file needs some changes. I commented in the PR.

Hi @aeyoll , I checked it, everything is good. But the README.md file needs some changes. I commented in the PR.

Hello @rousan, I don't see any other comment? I already updated the README.md file with the correct versions, what's missing?

Hi @aeyoll , Released v3.0.0.

Thanks a lot @rousan  :)


[Bug] lmdeploy卡住，不能接收任何请求

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.

Describe the bug
模型一直正常运行，但是一旦遇到某个图片报错OSError: image file is truncated，就会卡住，不能接收任何请求
Reproduction
No
Environment
sys.platform: linux
Python: 3.10.9 (main, Jan 11 2023, 15:21:40) [GCC 11.2.0]
CUDA available: True
MUSA available: False
numpy_random_seed:<PHONE_NUMBER>
GPU 0: NVIDIA A10
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 12.2, V12.2.140
GCC: gcc (GCC) 10.2.1 20200825 (Alibaba 10.2.1-3 2.17)
PyTorch: 2.1.1+cu121
PyTorch compiling details: PyTorch built with:
  - GCC 9.3
  - C++ Version: 201703
  - Intel(R) oneAPI Math Kernel Library Version 2022.2-Product Build 20220804 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v3.1.1 (Git Hash 64f6bcbcbab628e96f33a62c3e975f8535a7bde4)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: AVX512
  - CUDA Runtime 12.1
  - NVCC architecture flags: -gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_90,code=sm_90
  - CuDNN 8.9.7  (built against CUDA 12.2)
    - Built with CuDNN 8.9.2
  - Magma 2.6.1
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=12.1, CUDNN_VERSION=8.9.2, CXX_COMPILER=/opt/rh/devtoolset-9/root/usr/bin/c++, CXX_FLAGS= -D_GLIBCXX_USE_CXX11_ABI=0 -fabi-version=11 -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -DNDEBUG -DUSE_KINETO -DLIBKINETO_NOROCTRACER -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wall -Wextra -Werror=return-type -Werror=non-virtual-dtor -Werror=bool-operation -Wnarrowing -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=old-style-cast -Wno-invalid-partial-specialization -Wno-unused-private-field -Wno-aligned-allocation-unavailable -Wno-missing-braces -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Werror=cast-function-type -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_DISABLE_GPU_ASSERTS=ON, TORCH_VERSION=2.1.1, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=1, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

TorchVision: 0.16.1+cu121
LMDeploy: 0.5.3+
transformers: 4.44.0
gradio: Not Found
fastapi: 0.109.0
pydantic: 2.5.3
triton: 2.1.0
NVIDIA Topology:
  GPU0  CPU Affinity  NUMA Affinity GPU NUMA ID
GPU0   X        N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

Error traceback
Exception in callback _raise_exception_on_finish(<Future finis...s truncated')>) at /home/admin/miniconda3/lib/python3.10/site-packages/lmdeploy/vl/engine.py:19
handle: <Handle _raise_exception_on_finish(<Future finis...s truncated')>) at /home/admin/miniconda3/lib/python3.10/site-packages/lmdeploy/vl/engine.py:19>
Traceback (most recent call last):
  File "/home/admin/miniconda3/lib/python3.10/site-packages/PIL/ImageFile.py", line 271, in load
    s = read(self.decodermaxblock)
  File "/home/admin/miniconda3/lib/python3.10/site-packages/PIL/PngImagePlugin.py", line 932, in load_read
    cid, pos, length = self.png.read()
  File "/home/admin/miniconda3/lib/python3.10/site-packages/PIL/PngImagePlugin.py", line 167, in read
    length = i32(s)
  File "/home/admin/miniconda3/lib/python3.10/site-packages/PIL/_binary.py", line 85, in i32be
    return unpack_from(">I", c, o)[0]
struct.error: unpack_from requires a buffer of at least 4 bytes for unpacking 4 bytes at offset 0 (actual buffer size is 0)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/admin/miniconda3/lib/python3.10/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
  File "/home/admin/miniconda3/lib/python3.10/site-packages/lmdeploy/vl/engine.py", line 26, in _raise_exception_on_finish
    raise e
  File "/home/admin/miniconda3/lib/python3.10/site-packages/lmdeploy/vl/engine.py", line 22, in _raise_exception_on_finish
    task.result()
  File "/home/admin/miniconda3/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/admin/miniconda3/lib/python3.10/site-packages/lmdeploy/vl/engine.py", line 151, in forward
    outputs = self.model.forward(inputs)
  File "/home/admin/miniconda3/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/admin/miniconda3/lib/python3.10/site-packages/lmdeploy/vl/model/internvl.py", line 171, in forward
    images = [x.convert('RGB') for x in images]
  File "/home/admin/miniconda3/lib/python3.10/site-packages/lmdeploy/vl/model/internvl.py", line 171, in <listcomp>
    images = [x.convert('RGB') for x in images]
  File "/home/admin/miniconda3/lib/python3.10/site-packages/PIL/Image.py", line 922, in convert
    self.load()
  File "/home/admin/miniconda3/lib/python3.10/site-packages/PIL/ImageFile.py", line 278, in load
    raise OSError(msg) from e
OSError: image file is truncated

对于这块的异常处理不是很完善。出错的原因是图片损坏导致convert('RGB')抛异常了。
目前比较推荐能在传请求之前先对图片做一次convert检查一下图片的完整性。

@irexyc Can we make up a dummy image in this case? At least to avoid the "hung" issue.

请问这个问题什么时候解决，我也碰到了这个问题，我跑的图量比较大，先检查一遍url耗时会很久

着急的话，可以使用 #2312，按照 installation 文档源码编译 lmdeploy


Remove Deploy transaction from tests (where possible)

Please check if the PR fulfills these requirements

[x] The formatter, linter, and tests all run without an error
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

removes usage of Deploy where it is possible

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Yes
Other information

Can you try rebasing this branch so it doesn't include all commits from 0.10.1 branches?


Error try exec run.bat

Script should be run from project root, not from scripts directory.

it worked but it says, pytesseract.pytesseract.TesseractNotFoundError: C:\Users\User\AppData\Local\Programs\Tesseract-OCR\tesseract.exe is not installed or it's not in your PATH.
my tesseract is installed in C:\Users\jesrm\AppData\Local\Programs\Tesseract-OCR , for some reason it is not being detected

@jesramos Notice how your tesseract is installed under a different user? I left the location path as C:\Users\User because you are meant to change the value of User to be your user. This means you need the edit the config value at src/config/settings.json to be C:\Users\jesrm\AppData\Local\Programs\Tesseract-OCR\tesseract.exe.


Stuck on Tx status paid ether now whole thing gone

If you are reporting a problem with Token Wizard, please include the following information:
Idk which network.. tried to make sound coin. I'm dealing with many professional music artists.
Which network did you use? (Mainnet, Kovan, Rinkeby, etc.)
I just clicked new crowdsale
If you were able to create it, what is the URL of your crowdsale?
Website isn't up yet. That's why I set the crowdsale out for a month
Do you have screenshots showing the problem?
No.. I'm stick on Tx status screen.. I paid already.. it keeps asking for more money
Do you see errors in the dev console? If yes, please include a screenshot
To open the dev console in Google Chrome, press F12, or go to View -> Developer -> Developer Tools, and open the Console tab
If you see errors, please right click on them and "Save as..". Zip saved file and attach it to the Issue.
Idk

i went ahead and payed for a new crowdsale with your website token wizard.. now i have an error on that

@Gio300

i went ahead and payed for a new crowdsale with your website token wizard.. now i have an error on that

What error? in Chrome dev console? Could you attach log:

To open the dev console in Google Chrome, press F12, or go to View -> Developer -> Developer Tools, and open the Console tab

If you see errors, please right click on them and "Save as..". Zip saved file and attach it to the Issue.

Did you pass through the whole deployment process? Have you got the URL of your crowdsale?

Closed due no answers

As far as I know the crowdsale is good. Thank you
On Fri, Apr 20, 2018, 7:09 AM Victor Baranov<EMAIL_ADDRESS>wrote:

Closed due no answers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/poanetwork/token-wizard/issues/792#issuecomment-383107964,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkloxnAm8k36qyNDAeIbdf-YitHhWUe-ks5tqewGgaJpZM4TUvHg
.


"No Results" still triggered when results are returned

System Information
Windows 10 (64-bit)
Chrome 83.0.4103.97
Use Case
I have a field that allows users to add a comma-delimited list of tags. I'm implementing autoComplete.js to replace my current auto-complete system, which suggests canonical tags to append to the end of the field.

Code
setupAutosuggest: function (customTagInputId) {
    const resultsContainerId = customTagInputId + '-results';
    new autoComplete({
        data: {
            src: async () => {
                const customTags = document.getElementById(customTagInputId).value.trim();
                if (customTags === '') {
                    console.log('no input');
                    return [];
                }
                const query = split(customTags).pop();
                console.log('input: ' + query);
                const source = await fetch(`https://api.example.com/v1/tags/autocomplete?term=${query}`);
                const apiResponse = await source.json();
                const data = apiResponse.hasOwnProperty('data') ? apiResponse.data : null;
                if (!data) {
                    console.log('no data received from API');
                    return [];
                }
                let tagSuggestions = [];
                let tagName;
                for (let i = 0; i < data.length; i++) {
                    tagName = data[i].attributes.name;
                    tagSuggestions.push(tagName);
                }
                console.log('suggestions received from API: ' + tagSuggestions.join(', '));
                return tagSuggestions;
            },
            cache: false
        },
        selector: '#' + customTagInputId,    // Input field selector              | (Optional)
        threshold: 3,                        // Min. Chars length to start Engine | (Optional)
        debounce: 300,                       // Post duration for engine to start | (Optional)
        resultsList: {                       // Rendered results list object      | (Optional)
            render: true,
            container: source => {
                source.setAttribute('id', resultsContainerId);

                // Automatically hide results list upon click
                document.getElementById(customTagInputId).addEventListener('autoComplete', function (event) {
                    function hideSearchResults() {
                        const searchResults = document.getElementById(resultsContainerId);
                        while (searchResults.firstChild) {
                            searchResults.removeChild(searchResults.firstChild);
                        }
                        document.removeEventListener('click', hideSearchResults);
                    }
                    document.addEventListener('click', hideSearchResults);
                })
            },
            destination: document.getElementById(customTagInputId),
            position: 'afterend',
            element: 'ul'
        },
        maxResults: 6,                         // Max. number of rendered results | (Optional)
        highlight: true,                       // Highlight matching results      | (Optional)
        resultItem: {                          // Rendered result item            | (Optional)
            content: (data, source) => {
                source.innerHTML = data.match;
            },
            element: 'li'
        },
        noResults: () => {                     // Action script on noResults      | (Optional)
            const result = document.createElement('li');
            result.setAttribute('class', 'no_result autoComplete_result');
            result.setAttribute('tabindex', '1');
            result.innerHTML = 'No Results';
            document.getElementById(resultsContainerId).appendChild(result);
            console.log('no results (supposedly)');
        },
        onSelection: feedback => {             // Action script onSelection event | (Optional)
            const customTagsField = document.getElementById(customTagInputId);
            let customTags = split(customTagsField.value);
            customTags[customTags.length - 1] = feedback.selection.value;
            customTagsField.value = customTags.join(', ') + ', ';
            console.log('selected ' + feedback.selection.value);
        }
    });
};

Results
This works almost perfectly. For the first tag entered, it always works as expected.

Then

But
But for any tag after the first, it always shows "No Results" despite it actually having results.

There's nothing interfering with autoComplete.js receiving input correctly and there's nothing different about how the results are being provided to it, so there must be some other mechanism in the core that's incorrectly triggering the "no results" condition. I poked around for a little while and couldn't find anything obvious, so I'm hoping that someone else might be able to either find where the bug is in the core or point out something in my own code that I'm overlooking.

Ah, nevermind. I figured out the part of the library's configuration that I didn't understand. Since I was doing the result filtering on the server side, I needed to disable the filtering that autoComplete.js was doing on the client side by default in its searchEngine setting. I added this to make it stop filtering out any results, and now it works as expected:
searchEngine: function (query, record) {
    return record;
}

Without this, it was comparing the entire chain of tags (e.g. "foo, bar, bas") to the results returned for the incomplete string at the end (e.g. ["basket", "basil", "base"]) and not finding any matches.

Hello @PhantomWatson,
I'm glad you've found a solution to your issue!
That's correct. The default searchEngine compares the entire given string within the input field while using the searchEngine API lets you use a custom algorithm that bypasses the default.
Cheers, have a good day! :)

ho thanks you @PhantomWatson , 2 hour loose with no result hits !
IMHO this callback searchengine must be in set in default example ping @TarekRaafat

@PhantomWatson thank you so much, I had the exact same problem!
As @cbastienbaron said, this should definitely be a default search engine option instead of having to write boilerplate.
There could also be an how-to implement server-side autocompletion!

Ah, nevermind. I figured out the part of the library's configuration that I didn't understand. Since I was doing the result filtering on the server side, I needed to disable the filtering that autoComplete.js was doing on the client side by default in its searchEngine setting. I added this to make it stop filtering out any results, and now it works as expected:
searchEngine: function (query, record) {
    return record;
}

Without this, it was comparing the entire chain of tags (e.g. "foo, bar, bas") to the results returned for the incomplete string at the end (e.g. ["basket", "basil", "base"]) and not finding any matches.

after this, the highlight does not work. any suggestions?

solved it with highlights
searchEngine: function (query, record)
                    {
                        let capsQuery = self.capitalize(query);
                        if (query.split(' ').length > 1)
                        {
                            return record;
                        }
                        else
                        {
                            return record.replace(capsQuery, `<mark>${capsQuery}</mark>`);
                        }
                    },


New theme settings Integration

After the backend team sets up the API for new theme setting, we will need to integrate it with new theme setting panel.

Hey team! Please add your planning poker estimate with Zenhub @dhruvikn @jsartisan @KelvinOm


storage/kubernetes: allow arbitrary client IDs

Use a hash algorithm to match client IDs to Kubernetes object names.
Because cryptographic hash algorithms produce sums larger than a
Kubernetes name can fit, a non-cryptographic hash is used instead.
Hash collisions are checked and result in errors.
Closes #631

FYI this passes integration testing (since we don't test those on travis)
$ go test -v ./storage/kubernetes/
=== RUN   TestIDToName
--- PASS: TestIDToName (0.00s)
=== RUN   TestNamespaceFromServiceAccountJWT
--- PASS: TestNamespaceFromServiceAccountJWT (0.00s)
=== RUN   TestLoadClient
2016/10/27 16:39:06 third party resource already created "auth-code.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "auth-request.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "o-auth2-client.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "signing-key.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "refresh-token.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "password.oidc.coreos.com"
--- PASS: TestLoadClient (0.11s)
=== RUN   TestURLFor
--- PASS: TestURLFor (0.00s)
=== RUN   TestStorage
2016/10/27 16:39:06 third party resource already created "auth-code.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "auth-request.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "o-auth2-client.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "signing-key.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "refresh-token.oidc.coreos.com"
2016/10/27 16:39:06 third party resource already created "password.oidc.coreos.com"
=== RUN   TestStorage/AuthCodeCRUD
=== RUN   TestStorage/AuthRequestCRUD
=== RUN   TestStorage/ClientCRUD
=== RUN   TestStorage/RefreshTokenCRUD
=== RUN   TestStorage/PasswordCRUD
=== RUN   TestStorage/KeysCRUD
=== RUN   TestStorage/GarbageCollection
=== RUN   TestStorage/ClientConcurrentUpdate
--- PASS: TestStorage (0.55s)
    --- PASS: TestStorage/AuthCodeCRUD (0.07s)
    --- PASS: TestStorage/AuthRequestCRUD (0.05s)
    --- PASS: TestStorage/ClientCRUD (0.09s)
    --- PASS: TestStorage/RefreshTokenCRUD (0.03s)
    --- PASS: TestStorage/PasswordCRUD (0.06s)
    --- PASS: TestStorage/KeysCRUD (0.04s)
    --- PASS: TestStorage/GarbageCollection (0.12s)
    --- PASS: TestStorage/ClientConcurrentUpdate (0.04s)
    	transactions.go:48: update1: PUT https://<IP_ADDRESS>:443/apis/oidc.coreos.com/v1/namespaces/default/oauth2clients/ojqxonlkoi3gizztonzgm5tign2wczjwnb5hgzdrnxf7fhheqqrcgji Conflict: response from server "{"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"Operation cannot be fulfilled on thirdpartyresourcedatas.extensions \"ojqxonlkoi3gizztonzgm5tign2wczjwnb5hgzdrnxf7fhheqqrcgji\": the object has been modified; please apply your changes to the latest version and try again","reason":"Conflict","details":{"name":"ojqxonlkoi3gizztonzgm5tign2wczjwnb5hgzdrnxf7fhheqqrcgji","group":"extensions","kind":"thirdpartyresourcedatas"},"code":409}"
    	transactions.go:49: update2: <nil>
PASS
ok  	github.com/coreos/dex/storage/kubernetes	0.664s


Change how BluetoothAdapter identifies devices

Abstract identity in an interface independent of Mac Address
Change PoweredUpHost function to use generic state instead
of bluetooth address as ulong
Implement interface in existing adapters (simple transformation)
Change BluetoothKernel interface to accept device instead of
bluetooth address
Add formatting to the MAC Address DeviceInfo

Closes #154

Hi @tthiery,
unfortunately I had not enough time but today I took this changes and improved them, so that I could establish a working connection also with iOS devices.
I'm not done with the implementation but you can see the current code I made on your base here:
https://github.com/sharpbrick/powered-up/compare/issue-154-deviceinfo...Berdsen:issue-154-deviceinfo-adjustements?expand=1
So the iOS solution will get a GUID as DeviceIdentifier (NSUuid) whereas the Android DeviceIdentifier will be the MacAdress as string :)
Works at least for both implementations, so we can go for this for now.
As an example with ExampleBluetoothByKnownAdress I have to give the NSUuid as string and it will also work. I need to check how this will work ExmpleTwoHubsMotorControl I have to check, if it is also working, when I know the UID.


Android: Async Storage is not accesible, when a background task is being handled in index.android.js

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
Android: Async Storage is not accesible, when a background task is being handled in index.android.js
note* - no issue on messaging().setBackgroundMessageHandler, it brings out the remote message
snippet:
const retrieveEasDataMessage = async () => {
try {
const accName = await AsyncStorage.getItem("accountName");
const remoteMessage = await AsyncStorage.getItem(${dataMsgKey}-${accName});
if (remoteMessage !== null) {
console.log(launch retrieveEasDataMessage remoteMessage : ${JSON.stringify(remoteMessage)});
}
} catch (err) {
console.log(launch retrieveEasDataMessage err : ${JSON.stringify(err)});
}
};
//Quit state handler, once the app is quit, if the app receives an firebase data-message, below listener will be handling that
messaging().setBackgroundMessageHandler(async (remoteMessage) => {
try {
if (remoteMessage && remoteMessage !== null) {
await retrieveEasDataMessage();
console.log(setBackgroundMessageHandler remoteMessage : ${JSON.stringify(remoteMessage)});
await storeEasDataMessage(remoteMessage);
}
} catch (error) {
console.log(android setBackgroundMessageHandler error : ${JSON.stringify(error)});
}
}) : () => {/**/ }
======
The above code will be executed when a data message is received, once the app is killed / quit state, there
const accName = await AsyncStorage.getItem("accountName");
"accountName" is already set while foreground & background message handling but when quit state handling if we access it, this accName is always null
React Native version:
Run react-native info in your terminal and copy the results here.
react-native info
warn Your project is using deprecated "rnpm" config that will stop working from next release. Please use a "react-native.config.js" file to configure the React Native CLI. Migration guide: https://github.com/react-native-community/cli/blob/master/docs/configuration.md
warn The following packages use deprecated "rnpm" config that will stop working from next release:
react-native-extra-dimensions-android: https://npmjs.com/package/react-native-extra-dimensions-android
Please notify their maintainers about it. You can find more details at https://github.com/react-native-community/cli/blob/master/docs/configuration.md#migration-guide.
info Fetching system and libraries information...
System:
OS: macOS Mojave 10.14.5
CPU: (8) x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
Memory: 34.23 MB / 16.00 GB
Shell: 3.2.57 - /bin/sh
Binaries:
Node: 10.18.0 - /usr/local/opt/node@10/bin/node
Yarn: 1.16.0 - /usr/local/bin/yarn
npm: 6.13.4 - /usr/local/opt/node@10/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
Android SDK:
API Levels: 23, 24, 25, 26, 27, 28
Build Tools: 25.0.0, 25.0.1, 25.0.2, 26.0.1, 26.0.2, 27.0.3, 28.0.0, 28.0.3, 29.0.2
System Images: android-23 | Intel x86 Atom_64, android-23 | Google APIs ARM EABI v7a, android-23 | Google APIs Intel x86 Atom_64, android-26 | Android TV Intel x86 Atom, android-26 | Google APIs Intel x86 Atom, android-26 | Google APIs Intel x86 Atom_64, android-26 | Google Play Intel x86 Atom, android-28 | Android TV Intel x86 Atom, android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom, android-28 | Google Play Intel x86 Atom_64
IDEs:
Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
Xcode: 11.2.1/11B500 - /usr/bin/xcodebuild
npmPackages:
react: 16.9.0 => 16.9.0
react-native: 0.61.1 => 0.61.1
npmGlobalPackages:
react-native-cli: 2.0.1
react-native-log-ios: 1.5.0
Expected Results
React Native's AsyncStorage should be an accessible in the above scenario
Please share your idea to fix this
thanks folks

Hi @ravichandran18390 this would be more appropriate for the async storage repo. Could you file an issue over at https://github.com/react-native-community/async-storage?
I'll close this for now. If this is an issue with React Native core, feel free to discuss! Thank you


Export Smart Contract code

Please describe the problem that your request should resolve.

Describe the solution you'd like

Add a button to export Smart Contract code.
Is it already possible to achieve the same outcome in any other way? If so, how?

Additional context

part of https://github.com/aeternity/aescan/issues/495


Error under Python 3 in generate_encryption_key

Under python 3.5.2, I get the following when trying to generate an encryption key:
$ ./manage.py generate_encryption_key
Traceback (most recent call last):
File "./manage.py", line 23, in
execute_from_command_line(sys.argv)
File "/home/ssharkey/.pyenv/versions/conestoga/lib/python3.5/site-packages/django/core/management/init.py", line 367, in execute_from_command_line
utility.execute()
File "/home/ssharkey/.pyenv/versions/conestoga/lib/python3.5/site-packages/django/core/management/init.py", line 359, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/home/ssharkey/.pyenv/versions/conestoga/lib/python3.5/site-packages/django/core/management/base.py", line 294, in run_from_argv
self.execute(*args, **cmd_options)
File "/home/ssharkey/.pyenv/versions/conestoga/lib/python3.5/site-packages/django/core/management/base.py", line 345, in execute
output = self.handle(*args, **options)
File "/home/ssharkey/.pyenv/versions/conestoga/lib/python3.5/site-packages/cryptographic_fields/management/commands/generate_encryption_key.py", line 12, in handle
self.stdout.write(key)
File "/home/ssharkey/.pyenv/versions/conestoga/lib/python3.5/site-packages/django/core/management/base.py", line 107, in write
if ending and not msg.endswith(ending):
TypeError: endswith first arg must be bytes or a tuple of bytes, not str
The default ending in self.write() from django's management base.py is ending='\n'
IF I change line 11 of generate_encryption_key.py to:
self.stdout.write(key, ending=b'\n')

then it works.

Exact same error using Django 2.1, Py 3.6.4 but edit did not fix. used print() and grabbed in terminal before the exception.


Strategically Place Logs

We want to log when the user does anything, then we can look back and see some trends and even do some AB testing! I'd like to use this issue to make sure everyone has a voice on what they want tracked.
cc @petschekr @bunsenmcdubbs
What I'm thinking so far:

[x]  visiting
[x] creating an account
[x] starting registration
[x] completing registration

And what to log:

[ ] any demographic information available
[x] a unique identification when available
[x] time
[x] location

Anyone else have any other ideas? Assigning @kexin-zhang on this since she's the master of metrics!

I've got some basic logs printing out in c06e8970b2859d195414706d9bda5b2964bf13ab. Any thoughts on how to go about getting location? IP Address?

IP Address might be our best bet but that could be tricky. Oftentimes the web app is too high in the stack to (easily) get any information about IP address or it is spoofed/omitted/inconsistently represented.
Might be worth punting on location for now. @illegalprime thoughts?

If we can get the router / reverse proxy to pass along IP information in X-Forwarded-* headers, you can access it from Express by running app.enable('trust proxy'); (when in production) and getting it from request.ip.

Thanks @petschekr and @bunsenmcdubbs! I'm going to go ahead with request.ip

We also have this: https://developer.mozilla.org/en-US/docs/Web/API/Geolocation/Using_geolocation

now that #76 has been merged, is this issue closed?


improve merging performance

A generic greedy merging method was added in https://github.com/thunder-project/thunder-extraction/commit/aecd49348540e3b92def4e443f7d2551aa1dbb2a but the performance is quite poor in practice, sometimes taking longer than the initial fitting itself! We should find an implementation that doesn't require n^2 region comparisons.
One simple improvement would be to precompute the k nearest neighbors for each region based on region centers and then do comparisons only over those neighbors, under the (correct) assumption that valid merges will be local.
cc @jwittenbach @j-friedrich might be interested in working on it

fixed in https://github.com/thunder-project/thunder-extraction/pull/5


Mass assign users to groups

As a user I would like to mass select users and assign them to a specific group.

Good use case. Thanks. I'll try to do it for 2.0.

@larsboehnke Maybe you can test the current development version of OpenSlides master that the implemented feature is useable for you?

@emanuelschuetze I try it later, thanks :+1:


Unable to fetch preview for many url

Hi, I tried this demo but many url previews are unable to fetch. Like Zomato, swiggy etc.
What may be the problem?
Thanks
Binay

@binaykumarrana if you provide an example URL that doesn't provide a preview, I'll look into it.

this is the example url that doesn't work. getMetaTags() returns no image value.
https://www.msn.com/en-us/news/crime/operation-misafekid-recovers-123-missing-children-in-wayne-county/ar-BBO7I5N?ocid=ientp
Is it possible to provide the getter in SourceContent that we can get property=og:image meta content. Most people only wants a url preview of `og:image'.

@HarryMMR trying this url with the latest code does retrieve images (they aren't the best images, but there are 14 there).  If you don't want to build from source, you can grab a binary to try for yourself from my maven repo: https://github.com/jdonmoyer/mvn-repo

@HarryMMR You can provide your own ImagePickingStrategy to TextCrawler with the exact behavior that you want.


DevP2P expiration conformance

DevP2P packets expiration fields should be in seconds past the epoch,
not milliseconds past the epoch.
Signed-off-by: Danno Ferrin<EMAIL_ADDRESS>

Should we also do this in places like:
  public static PongPacketData create(final Endpoint to, final Bytes pingHash) {
    return new PongPacketData(
        to, pingHash, System.currentTimeMillis() + PacketData.DEFAULT_EXPIRATION_PERIOD_MS);
  }

If so, we should extract the function for getting now + expiration to PacketData and use that for all of them.

default expiration has been refactored.  And I think I got all the places as I renamed the constant, I'm letting the compiler keep me honest.


Add WhitelistDC, Blacklist and BalcklistDC policies

It is time saving when we want to whitelist a hole dc with many hosts.
We can blacklist a dc instead of whitelisting all other hosts of other dcs.
We can also blacklist some hosts.

Thanks for the load balancing policies. I'm currently working on a php-driver release. I'll attempt to review this next week. Sorry for the delay.

OK thanks !

Hello
Did you review my commit ?
Thanks

I will start review later today.

@waitamer looks great. Thanks for adding these (with tests). Sorry this took so long.

Thanks !


Send button stays disabled

Linux light client, current version: 2.0.160120.
I click on an account now in the Overview list, the i click on the TRansfer tab at the top, and the from field keeps defaulting to a different account. So, if I go in that field and re-type the account i want to send from, the fee field at the bottom disappears and the send button stays disabled. I have no way to send funds from my old delegate.kencode account to my kencode account. Both accounts reside in the same wallet.

I'll take a look. We are planning new release tomorrow, I'll include a fix.

I am now running the latest v...121, the from field is fixed.
the fee field at the bottom is still gone, and the send button stays disabled. I have no way to send funds from my old delegate.kencode account to my kencode account. Both accounts reside in the same wallet.

Is anybody else experiencing missing fee issue?

@kenCode-de, fee won't be shown if your account's balance is zero, make sure you have funds.

Well, the funds are there. I claimed them from vesting:
http://cryptofresh.com/b/2862177
The funds show in my BTS balance too:
http://cryptofresh.com/u/delegate.kencode
The wallet won't let me send those BTS to anyone though, the Send button remains disabled.

I found the fee field bug but can't reproduce the send button staying disabled.

Found it!

clicked on my delegate.kencode account in overview
click on transfer tab
from delegate.kencode to kencode (send button is still disabled)
1310bts balance that i can send, so i clicked on that link.
send button still disabled
i changed the amount to send to just 2bts (the send button lit up!)
i clicked the available balance link again, it changed my 2bts entry to 1310bts
i sent my delegate.kencode bts balance to kencode successfully:
http://cryptofresh.com/b/2910469

so i guess the disabled-send bug resides in the send button detecting the balance or something?

Should be ok now, please reopen if it still occurs after the next release.


Neg fallout - 2

Hey, thanks for the quick fix. However, it seems that your commit does not build with Rust 1.0.0-beta, at least on Windows 64. It works fine with the latest available nightly (083b8a404 2015-04-05)
   Compiling cgmath v0.1.3 (https://github.com/kvark/cgmath-rs.git?branch=neg2#498df017)
[...]\neg2\src\transform.rs:240:24: 240:25 error: parameter `S` is never used
[...]\neg2\src\transform.rs:240 pub trait ToComponents<S, V: Vector<S>, P: Point<S, V>, R: Rotation<S, V, P>> {
                                                                                ^
[...]\neg2\src\transform.rs:240:25: 240:25 help: consider removing `S` or using a marker such as `core::marker::PhantomFn`
[...]\neg2\src\transform.rs:240:41: 240:42 error: parameter `P` is never used
[...]\neg2\src\transform.rs:240 pub trait ToComponents<S, V: Vector<S>, P: Point<S, V>, R: Rotation<S, V, P>> {
                                                                                ^
[...]\neg2\src\transform.rs:240:42: 240:42 help: consider removing `P` or using a marker such as `core::marker::PhantomFn`
error: aborting due to 2 previous errors
Could not compile `cgmath`.

@va1erian We need a 1.0 branch here, either taken before my PR or after merging it (depending on what path is simpler, to me it looks like the latter). The master should continue following rust nightly, and that's what my PR is about.

@bjz ping

Ok, waiting gets boring rather quickly. Merging.


Update docs.md

Memorystore Typo in docs.md
External Contributor Checklist

🚨 Please review the guidelines for contributing to this repository.

[ ] Make sure that all commits follow the conventional commit specification for the Mender project.

The majority of our contributions are fixes, which means your commit should have
the form below:
fix: <SHORT DESCRIPTION OF FIX>

<OPTIONAL LONGER DESCRIPTION>

Changelog: <USER-FRIENDLY-CHANGE-DESCRIPTION> or <None>
Ticket: <TICKET NUMBER> or <None>

[ ] Make sure that all commits are signed with git --signoff. Also note that the signoff author must match the author of the commit.

Description
Please describe your pull request.
Thank you!

Thanks, resubmitted in #2448 and #2449.


merging authorization-page and master branch

@NEsfandiari
We refactored and cleaned up code

we made file structure to be consistent with molecules and atoms structure
modified filenames to be consistent
added exports to index.js files to make imports more efficient
refactored css to not include itallic and u tags
refactored to remove inline style for side navbar component
removed unnecessary comments

https://deploy-preview-23--determined-benz-390e9b.netlify.com/


Can't save data to MINIO

Hello,
I deploy Kubeflow using Vanilla deployment link
minio version: 7.1.12
I'm able to login to MINIO UI using defined credentials, however in first step:
from minio import Minio
import os

minio_client = Minio(
        "<IP_ADDRESS>:9000",
        access_key="minio",
        secret_key="minio123",
        secure=False
    )
minio_bucket = "mlpipeline"

import glob

def upload_local_directory_to_minio(local_path, bucket_name, minio_path):
    assert os.path.isdir(local_path)

    for local_file in glob.glob(local_path + '/**'):
        local_file = local_file.replace(os.sep, "/") # Replace \ with / on Windows
        if not os.path.isfile(local_file):
            upload_local_directory_to_minio(
                local_file, bucket_name, minio_path + "/" + os.path.basename(local_file))
        else:
            remote_path = os.path.join(
                minio_path, local_file[1 + len(local_path):])
            remote_path = remote_path.replace(
                os.sep, "/")  # Replace \ with / on Windows
            minio_client.fput_object(bucket_name, remote_path, local_file)

upload_local_directory_to_minio("models/detect-digits",minio_bucket,"/models/detect-digits")

I'm getting error:
HTTPConnectionPool(host='<IP_ADDRESS>', port=9000): Max retries exceeded with url: /mlpipeline?location= (Caused by ResponseError('too many 503 error responses'))
What the reasone could be in this case?

The reason was wrong host name in minio_client.
To check your Minio host name:
kubectl get pods -n kubeflow | grep minio
Copy your pod name
kubectl logs your-minio-pod-name -n kubeflow
You get:
Endpoint:  http://<IP_ADDRESS>:9000  http://<IP_ADDRESS>:9000
My host name is:
http://<IP_ADDRESS>:900
So after correction:
minio_client = Minio(
        "<IP_ADDRESS>:9000",
        access_key="minio",
        secret_key="minio123",
        secure=False
    )

You should also correct line:
upload_local_directory_to_minio("models/detect-digits",minio_bucket,"/models/detect-digits")
On (no slash):
upload_local_directory_to_minio("models/detect-digits",minio_bucket,"models/detect-digits")

Thanks @doxenix! Indeed, I did not write that the user need to set their minio IP. That can be different for every cluster.
Thanks for the feedback, will change it. I may have uploaded the wrong version from my cluster.


The RDF toolkit incorrectly revises IRIs for those with non-standard structure

When serializing IRIs such as those found in the Qlaira example for the IDMP project (i.e., any IRI that is associated with the SPOR repository managed by the European Medicines Agency), the serializer incorrectly re-writes some of the abbreviated IRIs and completely replaces others with the full string value of the IRI.  Certain SPOR product IRIs were changed from "spor-p;" to "spor-p;/products/", and similarly with "spor-s;" replaced with "spor-s;/substances/". The replacements tend to occur in the named class or individual rather than other references to them.
See the Qlaira Example ontology in IDMP/EXT/Examples.

Hi, Elisa!
It seems that the problem occurs because the IRI of the ontology used by some entities has the following format: https://spor.ema.europa.eu/smswi/#/products/. It seems to me that according to the IRI standard, the # char should be used to delimit the namespace and the fragment (an identifier) of an entity's IRI (see: https://datatracker.ietf.org/doc/html/rfc3987#section-2.2)
So, it is expected that the division should be like that:
Namespace: "https://spor.ema.europa.eu/smswi/#" Fragment: "/products/00006006"

and not like this (as it is currently in this ontology):
Namespace: "https://spor.ema.europa.eu/smswi/#/products/" Fragment: "00006006"

And rdf-toolkit tries to act according to this standard.
Shouldn't this be somehow addressed by the authors of this ontology, and not in the rdf-toolkit?

@trojczak Hi Rafal,
Although the IRIs are non-standard, if there is some way to address this for cases that include '/#/' in the name, maybe a speclal "switch", we need to be able to generate something consistently. I've tried it both ways, the way it is now and the way you've suggested, and it's not clear that it is consistent in either case. These IRIs are specific to the European Medicines Agency SPOR data, but it is a really important use case for our IDMP project.  and, what Protege and other tools do with it when they consume it is also inconsistent.
These are slash-style IRIs that include an embedded "#" unfortunately, so they do not follow any of the patterns that are specified in the 'Cool IRIs for the Semantic Web' or other W3C recommendations - the "#" style IRI should not have anything other than the local name after the "#" per the W3C, ad "#" is reserved in RDF for those IRIs that are "hash style", so there is an inconsistency in the way that EMA produces their data. If we can add a check for cases where there is a "slash style" IRI that includes an embedded "#", and in that case produce what the ontologist decide the proper prefix for the ontology is, which is in the header of the ontology (regardless of serialization in turtle or RDF/XML) that would be better.  I'm ok if this requires some kind of additional optional switch for the serializer, but we really need this for the IDMP project since it is prevalent in the EU public data. Doing this manually is time consuming and not the best use of my time, for example, and I was forced to upload the ontology bypassing the serializer, which is not ideal.


Refactoring with Bedrock Converse API

Issue #, if available:
close #347
Description of changes:

Introducing converse API, removed existing methods for anthropic Claude and Mistral
Both Claude and Mistral are integrated into converse API
Remove dependency for Anthropic SDK
Remove topK since converse API currently not supports it
Added types for bedrock invocation
Converse API accepts images / documents as byte, however, it still handle as:

images: as base64 encoded string
documents (textAttachments): as string

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The main change to compose argument to invoke bedrock: https://github.com/aws-samples/bedrock-claude-chat/pull/448/files#diff-7819170020dbd32228434729db857fd62a67137b2d50f72478285a06e04c3483R97-R171


chore: move build_abi logic to lib

chore: change extract_abi_entries arg type to &CompilationArtifact<DYLIB>
chore: move extract_abi_entries to dylib sub-module
chore: change arg type of CrateMetadata::resolve_output_dir to camino::Utf8PathBuf
chore: move CrateMetadata to cargo_near_build
chore: move types related to abi logic to lib
chore: move rite_to_file to cargo_near_build::near::abi
chore: move BUILD_RS_ABI_STEP_HINT_ENV_KEY to cargo_near_build::env_keys
chore: move generate_abi to cargo_near_build::near::abi::generate::procedure
chore: move cargo_near::commands::abi_command::abi::run to cargo_near_build::build_abi

@race-of-sloths ping

@dj8yfo Thank you for your contribution! Your pull request is now a part of the Race of Sloths!
We are launching a survey "Who is faster? You vs SpaceX"

Current status: waiting for scoring
We're waiting for maintainer to score this pull request with @race-of-sloths score [0,1,2,3,5,8,13] command. Alternatively, autoscoring [1,2] will be applied for this pull request
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Invite contributor @race-of-sloths invite to invite the contributor to participate in a race or include it, if it's already a runner.
Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called


google-cloud-aiplatform 1.72.0 is available but I don't see PR from regro-cf-autotick-bot

Comment:
google-cloud-aiplatform 1.72.0 is available  since Nov 12th
https://pypi.org/project/google-cloud-aiplatform/
but I don't see any PR from regro-cf-autotick-bot
should I report the issue to regro-cf-autotick-bot or should I do I create the PR manually ?

The issue is that the source URL is different for this version. The following file doesn't exist
https://pypi.org/packages/source/g/google-cloud-aiplatform/google-cloud-aiplatform-1.72.0.tar.gz
while the one from the previous versions exist
https://pypi.org/packages/source/g/google-cloud-aiplatform/google-cloud-aiplatform-1.71.1.tar.gz
I don't find the correct URL on this page:
https://pypi.org/project/google-cloud-aiplatform/1.72.0/#google_cloud_aiplatform-1.72.0.tar.gz

Presumably an issue of dashes vs. underscores?


Fix 7255 - wrap long texts on user profile info

wrap long texts on user profile info
fix go-gitea#7255
discussed in PR go-gitea#7254
close #7254
Signed-off-by: Michael Gnehr<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@17fa227). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #7333   +/-   ##
=========================================
  Coverage          ?   41.24%
=========================================
  Files             ?      466
  Lines             ?    63153
  Branches          ?        0
=========================================
  Hits              ?    26048
  Misses            ?    33700
  Partials          ?     3405

Continue to review full report at Codecov.

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


edit appointment implementation

will do merge conflicts directly in github

ignore this PR first, its PR-ing my bin files

ignore this PR first, its PR-ing my bin files

done, can check and merge now

checks failing here due to Person constructor not having the fields of age/gender/ethnicicity. Added it and merged into main so checks will (pass)/ fail due to other reasons after merging into main

checks failing here due to Person constructor not having the fields of age/gender/ethnicicity. Added it and merged into main so checks will (pass)/ fail due to other reasons after merging this PR into main

@Bryan-Goh


Support for DNN 8.x and 9.x

Does v5.7.1 support DNN 8.x and 9.x?

I just installed the most recent version (05.07.01) on a clean DNN 9.1.1 and it runs without problem.
So this version is good for DNN 8 & 9.


CHANGED PROPOSAL-  MF refactor: create new high level class: 'DNA structure modifier'

Hello,
GO:0097617 annealing activity and GO:0008301 DNA binding, bending are types of 'binding', but they should probably have a 'true' molecular activity parent (see also #16742). I propose  to create a new term ~nucleic acid topology modifier~  ->  'DNA structure modifier' The hierarchy would look like this:

GONEW:  'DNA structure modifier'
-- GO:0097617 annealing activity
--- GO:1990814 DNA/DNA annealing activity
--- GO:0097098 DNA/RNA hybrid annealing activity
--- GO:0033592 RNA strand annealing activity
-- GO:0008301 DNA binding, bending

We can also add other children such as

GO:0003916 DNA topoisomerase activity
GO:0034057 RNA strand exchange activity
GO:0004386 helicase activity

Is this OK ? @ValWood @thomaspd
Do you prefer 'nucleic acid topology modifier' or 'nucleic acid structure modifier'?
Should we add other grouping terms: DNA topology modifier and RNA topology modifier? I was wondering whether there are enough children to justify adding these.

Paul had proposed to add a lot more terms, but it seems to me it's beyond just 'topology modifier', see screenshots:

Thanks in advance for the feedback, Pascale

GONEW: nucleic acid topology modifier
-- GO:0097617 annealing activity
--- GO:1990814 DNA/DNA annealing activity
--- GO:0097098 DNA/RNA hybrid annealing activity
--- GO:0033592 RNA strand annealing activity
-- GO:0008301 DNA binding, bending

Is this OK ?

As drafted above, NO. It may be OK to create the "nucleic acid topology modifier" term, but it would not work as a parent of all suggested descendants. DNA bending does not alter DNA topology, and I'm not sure annealing necessarily does. DNA topology is only changed when the linking number changes, which requires breaking one or both DNA strands, i.e. what topoisomerases do.
It's not the most intuitive subject, but I found this helpful:
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC2643378/ (PMID:19225573)
As it stands, I'm not entirely sure anything besides topoisomerases could fit under "nucleic acid topology modifier", and if there isn't anything else, then the topoisomerase activity term itself does the necessary grouping.

Do you prefer 'nucleic acid topology modifier' or 'nucleic acid structure modifier'?

These do NOT mean the same thing. Topology is much narrower. If you use 'nucleic acid structure modifier', then you could place a lot more terms under it.

Agreed. I don't see a stong need for a grouping term for these. As I said in the previous ticket. Most have enzyme activity parent, AND DNA binding parent. This is fine
Annealing is an odd one. It seems to be used to annotate things which result in the removal of single strand binding proteins. I don't really know what the activity is but it definitely Isn't a topological change

Annealing is quite odd as a MF anyway. The term cites https://en.wikipedia.org/wiki/Nucleic_acid_thermodynamics#Annealing
"Annealing, in genetics, means for complementary sequences of single-stranded DNA or RNA to pair by hydrogen bonds to form a double-stranded polynucleotide. Before annealing can occur, one of the strands may need to be phosphorylated by an enzyme such as kinase to allow proper hydrogen bonding to occur. (...) "
Perhaps this should be a process?

Before annealing can occur, one of the strands may need to be phosphorylated by an enzyme such as kinase to allow proper hydrogen bonding to occur

really? we should tell the microarray people?
citation required !

New proposal discussed with @thomaspd

GONEW:  'DNA structure modifier'
-- GO:0097617 annealing activity
--- GO:1990814 DNA/DNA annealing activity
--- GO:0097098 DNA/RNA hybrid annealing activity
--- GO:0033592 RNA strand annealing activity
-- GO:0008301 DNA binding, bending
-- GO:0003916 DNA topoisomerase activity
-- GO:0034057 RNA strand exchange activity
-- GO:0004386 helicase activity

@ValWood @mah11 Does that look better ?

Do we need this term? All should have the parent
catalytic activity, acting on DNA
If you do add (I'm  a bit ambivalent)

GO:0004386 helicase activity would not go under here (RNA helicases)

How are you defining "structure modifier"? (presumably any geometric or topological change?), shouldn't things like
crossover junction endodeoxyribonuclease activity (GO:0008821)
recombinase activity (GO:0000150)
go under here too?

I recommend we get rid of
DNA topological change (GO:0006265)  and DNA geometric change (GO:0032392) from process? (they could point to the new proposed term?)

Is annealing a catalytic activity ? That would probably solve the issue.

I don't think so. A catalyst is something that lowers the activation energy of a reaction, thereby increasing its rate,  and is not consumed.

"DNA annealing may occur spontaneously, but it is promoted in vivo by certain classes of annealing proteins. " PMID:25646379. That seems to fit the definition of a catalyst ?

Unless the catalyst needs to consume energy - I dont think all annealing 'facilitators' use ATP. But if that's not a requirement then I dont see what is missing to call this reaction a catalytic reaction.

I think so yes.

"DNA annealing may occur spontaneously, but it is promoted in vivo by certain classes of annealing proteins." PMID:25646379. That seems to fit the definition of a catalyst ?
Unless the catalyst needs to consume energy

I haven't seen catalysts referred to as consuming energy or not consuming energy, but I'm fairly sure that's beside the point. Catalysts work by providing an alternative route for a reaction with a lower activation energy than the uncatalyzed version. I don't know the exact mechanism by which "facilitators" promote annealing, but it does sound like catalysis to me anyway.

On the more general question about the altered proposal:
I don't feel like there's a grouping term missing, i.e. I don't need a new grouping term.
I would very much need to see the definition of the proposed grouping term to understand why any deoxyribonuclease - not just the ones Val mentioned - would not also fit it. And any definition that could distinguish (topological + geometric) changes from backbone cleavage would probably cry out for a term name more specific than "structure modifier".

Lowering activation energy is different from consuming net energy. Protein hydrolysis is a spontaneous (if slow) reaction; trypsin et al. increase the rate and impose specificity by providing an active site that captures specific peptides and juxtaposes them to enzyme amino acid residues and cofactors precisely positioned to support hydrolysis. No energy is consumes; some should be released.

I am happy to move the terms under catalytic activity
-- GO:0097617 annealing activity
--- GO:1990814 DNA/DNA annealing activity
--- GO:0097098 DNA/RNA hybrid annealing activity
--- GO:0033592 RNA strand annealing activity
Pascale

Wont implement


contributor guide improvements

tries to make the contrib guide more approachable.

quick link to the rendered doc: https://38563-1301584-gh.circle-artifacts.com/0/dev/install/contributing.html

I  wonder if we can find a better icon to symbolize bug fixes? The current choice is so much larger than the rest 🤔

Looks great, @drammock !
While we are at it, would it be okay to nitpick on two small things in the existing text ... ?

While we are at it, would it be okay to nitpick on two small things in the existing text ... ?

I would say yes, even better if you just want to push a commit to @drammock's branch to fix them!

I wonder if we can find a better icon to symbolize bug fixes? The current choice is so much larger than the rest and not monochrome

We could make it monochrome quite easily. The size might be tricky but should be possible. I'm also open to using Unicode emoji instead of fontawesome, if there are better choices there

While we are at it, would it be okay to nitpick on two small things in the existing text?

Yes go right ahead

@hoechenberger is this better?
https://38648-1301584-gh.circle-artifacts.com/0/dev/install/contributing.html

I can add the changes tomorrow, did not get to it today. Just quickly from my notes, in case someone wants to beat me to it:
I suggest to change line 60
then `pushing`_ the local changes up to your fork),

to be more explicit:
then `pushing`_ the local changes up to your fork on GitHub),

I think this is something that is hard to grasp at the beginning, where which version of the code lives.

And then, under Running the test suite, the sidebar (pytest flags) creates a lot of awkward white space as it is longer than the text to its left (and constrained by the code bars). Maybe it could be placed at the end of the paragraph using the whole width?
But as I said - very much nitpicking :)

@hoechenberger is this better?
https://38648-1301584-gh.circle-artifacts.com/0/dev/install/contributing.html

Amazing! I like it!


Correcting export for serviceaccount

File: themes/default/content/registry/packages/gcp/api-docs/serviceAccount/account/_index.md
Currently, I'm getting an error because the format for serviceaccount from the official documentation seems off:

Making it serviceaccount all lowercase fixes the issue. May need to update the docs.

hi @robschn sry about the trouble, this issue is tracked here: https://github.com/pulumi/pulumi-gcp/issues/811


fix(versions): update Activiti/activiti-cloud-dependencies versions

UpdateBot pushed maven dependency: org.activiti.cloud.notifications.graphql:activiti-cloud-notifications-graphql-dependencies to: 7.0.13

/test this
/approve


Add max_output_tokens to WindowService

Also need max_generated_tokens_length - not sure if this is the same thing.

Also need max_generated_tokens_length - not sure if this is the same thing.


Data value schema disable validation

Adds:

--data-values-disable-validations flag to command-line interface
data_values_disable_validations keyword argument to the @ytt:library.instance.eval() function.

Even when validations are disabled, if they are malformed (i.e. there's a syntax error), it still results in a templating error. Is that okay?

I think that if an annotation is not valid starlark, then yes it should be reported as a syntax error. If there is an error about the order or number of arguments in a tuple, then no, I don't think there should be an error, which is consistent with how I am reading the code.
Currently half way through processing this PR and I will follow up tomorrow.

(see also conversation in #carvel about the appropriate scope of this flag: https://kubernetes.slack.com/archives/CH8KCCKA5/p1652295963803589)


Count aggregate fails on simple DataFrame

An equivalent example with just Pandas seems to work fine, so I'm not sure what I'm doing wrong here. Could this be an actual bug?
import pandas as pd
import dask.dataframe as dd
ddf_broken = dd.from_pandas(
    pd.DataFrame([1, 2]),
    npartitions=1,
)
ddf_broken.groupby(
    ddf_broken.index
).aggregate('count').compute()

...
  File "/Users/justusperlwitz/jupyter-notes/env/lib/python3.6/site-packages/pandas/core/generic.py", line 5162, in groupby
    **kwargs)
  File "/Users/justusperlwitz/jupyter-notes/env/lib/python3.6/site-packages/pandas/core/groupby.py", line 1759, in groupby
    return klass(obj, by, **kwds)
  File "/Users/justusperlwitz/jupyter-notes/env/lib/python3.6/site-packages/pandas/core/groupby.py", line 392, in __init__
    mutated=self.mutated)
  File "/Users/justusperlwitz/jupyter-notes/env/lib/python3.6/site-packages/pandas/core/groupby.py", line 2863, in _get_grouper
    if not isinstance(gpr, Grouping) else gpr
  File "/Users/justusperlwitz/jupyter-notes/env/lib/python3.6/site-packages/pandas/core/groupby.py", line 2538, in __init__
    self.grouper = _convert_grouper(index, grouper)
  File "/Users/justusperlwitz/jupyter-notes/env/lib/python3.6/site-packages/pandas/core/groupby.py", line 2891, in _convert_grouper
    raise ValueError('Grouper and axis must be same length')

$ pip freeze | grep -e dask -e pandas
dask==0.16.0
pandas==0.21.0

Thanks for the excellent issue report. This should be fixed in #3035. Note that this is only a problem for single-partition dataframes grouped by index when using agg (something for which dask will not benefit you, as you'd get no parallelism). Trying using npartitions > 1 works fine on the latest release. After #3035 this also works for npartitions == 1.
Latest release
In [1]: import pandas as pd

In [2]: import dask.dataframe as dd

In [3]: df = pd.DataFrame({'a': range(100)})

In [4]: ddf = dd.from_pandas(df, npartitions=5)  # Multiple partitions

In [5]: ddf.groupby(ddf.index).agg('count').compute().head()
Out[5]:
   a
0  1
1  1
2  1
3  1
4  1

After #3035
In [1]: import pandas as pd

In [2]: import dask.dataframe as dd

In [3]: df = pd.DataFrame({'a': range(100)})

In [4]: ddf = dd.from_pandas(df, npartitions=1)  # Single partition

In [5]: ddf.groupby(ddf.index).agg('count').compute().head()
Out[5]:
   a
0  1
1  1
2  1
3  1
4  1


remove draft from blog template

this removes the draft: true from our blog template.
since ive been here ive never seen anyone merge a blog set to draft on purpose.
but i have seen a lot of folks merge it without wanting it to be a draft.
and its slow to get out the fix to undraft it.
alternatively, i can leave draft in and set it to false if we think having it there so folks know about it would be helpful
@cnunciato do you have negative feelings about this? is keeping draft: true important to you?

@susanev Not at all, totally fine with removing it. There are a few other mentions of how draft works in the blogging README, though, so it'd be good to make the appropriate adjustments to that stuff as well; I'm happy to do that if you'd rather not, up to you.

@cnunciato k wasnt sure if that info was still helpful in case anyone does go and add draft on their own. i went ahead and removed the mentions, let me know if anything doesn't make sense. or feel free to push a commit.


Update _library.ts to support SDL2 installed by homebrew

to enable lib linking on Mac (x64), install sdl2 using Homebrew.
brew install sdl2 sdl2_image sdl2_ttf

Homebrew will install into /usr/local/lib by default.

LGTM but does it make sense to search "/usr/local/lib" first and then "/System/Volumes/Data/opt/homebrew/lib"? Switch the order I mean.

changed.


Improve celery monitoring.

Notify healtchecks.io periodically to make sure that celery/celerybeat is up
and running

@flamingspaz or @comzeradd r?

Coverage decreased (-0.07%) to 88.632% when pulling 0603edc1bc233010fa85a9e8458104a521550d0b on johngian:celery-healthcheck into 14d20609cf4af50a440dd923b5b2b1c1da05f74a on mozilla:master.


Can't add clinical data to maf file

Hi,
I can read and work with a maf file but when I try to add external clinical data to it, I get this error:
Error in gsub(pattern = " ", replacement = "_", x = colnames(sample.anno),  :
string de entrada 23 es inválida en este locale

What I am trying is this:
clinic <- '/Users/abelgd/Desktop/training_bioinfo/POSTPIPELINE/clinicopatologicas_ptes.txt'
c <- read.delim(clinica, sep="\t", header=T,check.names = F)
c <- as.data.frame(c)

Where c is finally a df containing different columns with clinical data of the samples. First column name of c is Tumor_Sample_Barcode with exactly the names of the tumor_sample_barcodes from maf file.
merged_maf2 <- read.maf(maf = '/Users/abelgd/Desktop/training_bioinfo/Maftools/maf_files_indels_filt/merged', clinicalData = c, removeDuplicatedVariants = F,
                         vc_nonSyn = c("Frame_Shift_Del", "Frame_Shift_Ins", "Splice_Site", "Translation_Start_Site","Nonsense_Mutation", "Nonstop_Mutation", "In_Frame_Del","In_Frame_Ins", "Missense_Mutation", "intron_variant",
                                       "3_prime_UTR_variant", "5_prime_UTR_variant", "synonymous_variant", "upstream_gene_variant",
                                       "intron_variant,splice_polypyrimidine_tract_variant", "splice_region_variant,synonymous_variant",
                                       "downstream_gene_variant","Intron","3'UTR", "5'UTR","5'Flank", "Silent", "Splice_Region", "3'Flank" ))

@eibol1 Hi, try reading with isTCGA = FALSE
read.maf(maf = laml.maf, clinicalData = laml.clin, isTCGA = FALSE)


Capture or select the full image from the photo library instead of square image

Hi, I would like to be able to capture or select the full image from the photo library instead of uploading a square image. What's the best way of doing that? Thank you.

Hi @zoufishanmehdi,
Did you check the option kTGCameraOptionUseOriginalAspect (described here).

Thanks for the reply. I have pod 'TGCameraViewController', ‘2.2.8’ installed. I don't see the optionkTGCameraOptionUseOriginalAspect in TGCamera.h.


Add file encryption in addition to string encryption to the integration

Currently, the integration works if you're trying to encrypt a strong and store it on ceramic and gate it using Lit.
Could we add the relevant functions that also encrypt a file and then store the encrypted file contents on ceramic?

I think you can do this by base64 encoding the file (to turn it into a string)

That's how I did it


Make seriesof put the best match of the tree in extra_info, not the equip

Followup from #1778, this is actually user-facing now


Aggregator runner

and documentation

Codecov Report

Merging #16 (08dffa9) into main (a9d8f9c) will decrease coverage by 0.22%.
The diff coverage is 89.43%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   91.83%   91.60%   -0.23%
==========================================
  Files          15       15
  Lines        1726     1751      +25
==========================================
+ Hits         1585     1604      +19
- Misses        141      147       +6

Impacted Files
Coverage Δ

src/hdx/scraper/utilities/region_lookup.py
100.00% <ø> (ø)

src/hdx/scraper/outputs/update_tabs.py
86.66% <78.94%> (-1.96%)
:arrow_down:

src/hdx/scraper/configurable/aggregator.py
89.94% <86.66%> (-0.83%)
:arrow_down:

src/hdx/scraper/runner.py
89.93% <89.79%> (-0.07%)
:arrow_down:

src/hdx/scraper/base_scraper.py
100.00% <100.00%> (+2.53%)
:arrow_up:

...rc/hdx/scraper/configurable/resource_downloader.py
100.00% <100.00%> (ø)

src/hdx/scraper/configurable/rowparser.py
94.39% <100.00%> (ø)

src/hdx/scraper/configurable/timeseries.py
82.97% <100.00%> (-1.34%)
:arrow_down:

src/hdx/scraper/utilities/reader.py
97.38% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.


IRC nicks containing forward slash (/) ignored

I know this isn't really a common thing, but my channels are linked using UIN, so nicknames of people on other networks show with a bit added to the end to show what network they're on, for example, someone on freenode in the linked channel would show as somenick/fn and someone on efnet would show as somenick/ef.
It seems the nicknames with / are ignored or not seen by the bot relay script for some reason

Seems this was my fault and isn't actually an issue. Apologies


Add play/pause functionality.

Rephrase the extension's functionality so that it not only plays all
audio elements or WebAudio contexts which were blocked, but can pause
and resume all of them again afterward.
Redo the icons as play and pause buttons. Remove the notification.

Jaume: I'm proposing these changes as a way of prototyping what the UI for this functionality might look like if it were built in to the browser.
I realize this is a large change – basically a rewrite – and it probably has bugs. I mainly tested it with https://www.shadertoy.com/view/Xds3Rr and it works there (tested on a build of Chromium which has the autoplay restrictions in place, though). I plan to continue to refine it, but there are some time constraints for getting this in other peoples' hands to look at. So if there are no strong objections to the general direction of these changes I'd appreciate being able to merge these changes in sooner rather than later.
Thanks for your consideration.

Thanks, Ken!
Seems to work more or less with the same sites (i've found a kongregate game that doesn't seem to work with this rewrite).
To be honest, developing and testing this extension is being a bit tricky, considering that the policy is being discussed and that Chrome and DevTools behave inconsistently across sessions, reloads and navigation.
As long as sites that don't work can be reported and improve the mechanism to unmute media, I think it's the best that we can do at the moment.

Thanks for merging this Jaume. Would you mind sending me the link to that Kongregate game?
I know that developing this is tricky – ran into those issues myself, especially that opening DevTools makes all of the audio suddenly start working. It's down to console.log() calls and looking at them after the fact. But this is a very useful talking point and proof point, so thanks for writing it and being open to having it be a testbed.


Feature/push viashhub

Changelog
Retag docker images for viash-hub and push them there
Issue ticket number and link
~Closes #xxxx (Replace xxxx with the GitHub issue number)~
Checklist before requesting a review

[x] I have performed a self-review of my code

[ ] Conforms to the Contributor's guide

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[ ] Minor changes
[ ] Bug fixes

[x] Proposed changes are described in the CHANGELOG.md

[x] CI tests succeed!

I wonder if we can no de-duplicate the main_build and main_build_viash_hub using reusable workflows. From what I can see in the example on github, it is possible to have a build step first which is followed by different deployment stages.


Enable docker build kit

Since a feature RUN <<EOF from the docker build kit is being used, it would be a good idea to enable it while running docker build, as the build may not work in some systems.

What was changed
Added DOCKER_BUILDKIT=1 in the run-tls script

Why?
The RUN <<EOF feature was not working in some systems. For some reason build kit had to be manually turned on for it to work in those cases

Checklist

Closes

How was this tested:

Any docs updates needed?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

tested on linux/manjaro


Consider using markdown preview font size for font size in notebooks

Markdown font sizes shouldn't necessarily correspond to the font size of output elsewhere in a notebook. It would be nice if the font of markdown could be modified independent of the output font.
See comments here:
https://marketplace.visualstudio.com/items?itemName=ms-toolsai.jupyter&ssr=false#review-details

Duplicate of #126294


Remove warnings, outdated templates, proxy commands to C3

What

Removes the outdated directory of templates in /templates. None of these are actually up-to-date. As-needed some of this content can be added to npm create cloudflare templates that live here.
wrangler generate and wrangler init both just kick you over to npm create cloudflare automatically now. wrangler generate template <repository_url> behavior still works, just now via create cloudflare.
Removes a lot of old code

Line numbers on errors that pnpm run build outputs are confusing to me, and this is weird, this part not modified.
wrangler:build: src/index.ts(339,3): error TS2769: No overload matches this call.
wrangler:build:   The last overload gave the following error.
wrangler:build:     Argument of type '(yargs: CommonYargsArgv) => Argv<CommonYargsOptions & { name: string | undefined; } & { type: string | undefined; } & { site: boolean | undefined; } & { ...; } & { ...; } & { ...; }>' is not assignable to parameter of type '{ [key: string]: Options; }'.
wrangler:build:       Index signature for type 'string' is missing in type '(yargs: CommonYargsArgv) => Argv<CommonYargsOptions & { name: string | undefined; } & { type: string | undefined; } & { site: boolean | undefined; } & { ...; } & { ...; } & { ...; }>'.
wrangler:build: src/index.ts(782,3): error TS2769: No overload matches this call.
wrangler:build:   The last overload gave the following error.
wrangler:build:     Argument of type 'boolean' is not assignable to parameter of type 'string'.

All of my examples at https://github.com/cloudflare/workers-sdk/issues/6459 seem to be using the old code-path for generate, so I don't think this is (generally) semver breaking anymore with old generate URLs, thanks all. I appreciate the attention given here to ensure this didn't introduce major breaking changes to existing command behaviour in v3.

Thanks @andyjessop for the explanation, and @Cherry for additional info and pointer to https://github.com/cloudflare/workers-sdk/issues/6459 - which I believe we can close as soon as this PR is merged, right?
I've reviewed this case and here is my decision in a few bullet points:

this is a breaking change, and at odds with semver, however...
we don't really understand the real world impact of this breaking change because we don't have analytics that would tell us how often wrangler generate is invoked with a template name option
my estimate based on some research that the impact is likely close to 0 (I couldn't find docs or other references where we would instruct users to use wrangler generate with a template name anywhere except for a few 3rd party blog posts)
@irvinebroque has initiated this breaking change and approved it from the product perspective, so we have no blockers there
it is however wrong and unexpected that if a user calls wrangler generate my-worker some-template we give them c3 flow, and ignore the template name — this is a bug that we should address it, however not today
it's Friday, we should get this change in because overall it's valuable and Monday is feature freeze for bday week

we already believe the impact of the breaking change is minimal, we can live with the bug described in the previous point for a few days

Next steps:

[ ] merge this now as is and release it
[ ] file a bug describing the issue above
[ ] fix the bug next week by throwing a descriptive error when the user invokes wrangler generate with the template name option, and tell them that this feature is not available and instead they can just use generic c3 flow. Or even prompt them if they'd like us to invoke for them instead.
[ ] escalate the analytics issue as a blocker for proper decision making which is causing us to spin in circles of endless discussions about what might or might not be the usage of various features. this is a major waste and productivity drag on all of us.

that's all.. does this all matter? in terms of user impact likely not, but if we claim that we want to change our culture and improve DX across the board, then this is the level of attention we need to pay to these changes, and we should implement this level of polish (the bug fix described above) in all the scenarios where we are unsure about the real world impact. Only in cases where we believe that (and have data to prove it) the impact is 0, we can skip the polish.

and I see that @penalosa already merged this. thank you!


chore: add typehint to RdKafkaConsumer#getQueue

Prevent the vscode intelephense error : "Undefined method 'getPartition'."

Superseded by https://github.com/php-enqueue/enqueue-dev/pull/1071


module 'reduced_nm' has no attribute 'StartHandler'

Hi
I am encountering an issue that I am a bit clueless about (on the noetic-devel branch).
I set up both machines according to this guide and everything of section 1 worked nicely. However, when firing up node_manager and start discovery on the remote, I encounter the following issue:
[WARN] [1644535863.463285]: ERROR while start 'master_discovery': b"module 'reduced_nm' has no attribute 'StartHandler'\n"
[WARN] [1644535863.464635]: start discovering on octopi failed:
	b"module 'reduced_nm' has no attribute 'StartHandler'\n"

The exception is raised here:
https://github.com/fkie/multimaster_fkie/blob/11c985a163969654e7bb68bea647cae9a6a25334/fkie_node_manager/src/fkie_node_manager/start_handler.py#L181
Any hint or pointer is greatly appreciated, thx!

Hi @tkazik
I mean, I've seen this error message before, but I can't remember the context...
What happens if you ssh on octopi and call:
rosrun fkie_node_manager remote_nm.py --package fkie_master_discovery --node_type master_discovery --node_name /master_discovery _mcast_port:=11511 _mcast_group:=<IP_ADDRESS> _robot_hosts:=[] _heartbeat_hz:=0.5 __name:=master_discovery

Hi @atiderko
Thx for the quick follow-up. I just tried your suggestion on octopi through ssh and unfortunately, I get the same response:
module 'reduced_nm' has no attribute 'StartHandler'

No further information before/afterward and the node terminates after this output. I don't really understand this behavior, as the module should be imported on the following line:
https://github.com/fkie/multimaster_fkie/blob/56f48c2a297017165144d05201b1bbc6fe5f336c/fkie_node_manager/src/fkie_node_manager/__init__.py#L58
Somehow, I suspect this to be related to a python2 vs python3 issue...any other ideas?

Hi @tkazik,
it may be that python3 is more restrictive. I changed the module 'reduced_nm'. Can you please try again now?

Sweet, works like a charm now!
Thank you so much for maintaining and improving this nice package...even during the weekend!


Add Docs Examples

Still missing Components examples

Coverage remained the same at 100.0% when pulling d09bdd0afe66125e738bb2cdfddffacb94c8810d on documentation-examples into e1844519ad9bf97f46c6018ab91bb9207e3d249f on dev.


Bestiary fixture data needs to be updated

Monsters are missing raw_hp, raw_def, and raw_atk fields and the data is also several patches out of date.

Closed with 0f9e51b686bd0b92ffd33ed838f3246a6d5cd2f2


Fix linter warnings

I'm afraid I can't merge this, because on my platform (macOS) the cast is necessary:
error[E0308]: mismatched types
   --> src/base.rs:751:46
    |
751 |             .filter_map(|i| get_group_by_gid(i))
    |                                              ^ expected u32, found i32


Unable to create silence in Karma Dashboard throws "Failed to fetch" error

Hi @prymitive
I am unable to create silence in Karma Dashboard throws "Failed to fetch" error.
Can you please help on this?
Thank you,
Jaise George

Check browser console "Developer Tools" on Chrome, it should have logs for the error causing this

@prymitive ,
Below is the error,
Access to fetch at '' from origin '' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'

HTTP responses have Access-Control-Allow-Origin: * header set which is not allowed by browsers.
See cors:credentials on https://github.com/prymitive/karma/blob/master/docs/CONFIGURATION.md#alertmanagers
and https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS for context.
Either get rid of that header (karma doesn't set it) or adjust your cors:credentials setting.

@prymitive,
Thank you for the help!!
I have set the below values in confgmap and issue got resolved.
cors:
credentials: omit


It should be possible to provide a custom JSON serializer

From @MovGP0 on Wednesday, October 25, 2017 2:50:15 AM
Patching does not work when the entity uses custom datatypes that are not serializeable by default.
In my case, I use NodaTime to provide time-dependent information in a save manner:
public sealed class MyEntity
{
    [JsonProperty("created")]
    public Instant Created { get; set; }
}

[
    { "op": "replace", "path": "/created", "value": "2017-10-25T12:34:56.000Z" }
]

Patching fails, since it cannot convert the ISO date string to an Instant, unless the global JSON serializer is set up to support those datatypes:
JsonConvert.DefaultSettings = () => new JsonSerializerSettings()
     .ConfigureForNodaTime(DateTimeZoneProviders.Tzdb);

However, one might need a different global configuration than I want to use for patching. Therefore, it should be possible to provide a custom JSON converter for patching, rather than the global one:
var settings = new JsonSerializerSettings().ConfigureForNodaTime(DateTimeZoneProviders.Tzdb);
var myJsonSerializer = JsonSerializer.Create(settings);

patch.ApplyTo(myEntity, myJsonSerializer);

Copied from original issue: aspnet/JsonPatch#120

Hi. Thanks for contacting us.
We're closing this issue as there was no much community interest in this ask for quite a while now.


Readme is out of date - document SSH agent support

Currently, the readme states that the local provisioning mode require private_key to be set while, in fact, SSH agent also works perfectly fine. Document SSH agent support.

Resolved in https://github.com/radekg/terraform-provisioner-ansible/pull/61/commits/13dba4ff1cd20a71a43bae9d5d785fac6f88fcb1#diff-04c6e90faac2675aa89e2176d2eec7d8R119.