Change DXB Token Address

Updated DXB SPL Token Address.
I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address: 8mgeCL7k2cB2KTN8NhD5biqvcbkfrmBogDpYoHmn5cKQ
Token Name: DefiXBet
Token Symbol: DXB
Logo:

Link to the official homepage of token: DefiXbet.com

our Token Address has been updated. It can be verified that this request is from our team by checking SPL Token address on DefiXBet Whitepaper slide 23

hello please also update our Coingecko link
https://www.coingecko.com/en/coins/defixbet

context: previous pr: https://github.com/solana-labs/token-list/pull/360

(on merging this, i confirmed this mint address change request is truthfully orignated from the team member behind DXB).


[BUG] Cannot compile on Windows due to unix-only deps

Run cmake --build D:\a\server\server/build --config Release
MSBuild version 17.12.6+db5f6012c for .NET Framework

  1>Checking Build System
  Building Custom Rule D:/a/server/server/src/CMakeLists.txt
  color.cpp
  logger.cpp
  Generating Code...
  logger.vcxproj -> D:\a\server\server\build\src\Release\logger.dll
  Building Custom Rule D:/a/server/server/src/CMakeLists.txt
  main.cpp
D:\a\server\server\src\main.cpp(19,10): error C1083: Cannot open include file: 'getopt.h': No such file or directory [D:\a\server\server\build\src\minechad-server.vcxproj]
Error: Process completed with exit code 1.

A solution would be to simply use MSYS2/MinGW along CMake.

I believe that we might want to replace such libraries with some cross-platform ones like boost. We won't be able to always get away with simulating one environment in another. It would be better to have a code that works natively on all platforms without creating unix-like environment for it.

Hmm.. thought MinGW was compiling stuff natively.
Does boost have an getopt alternative with both long and short opts?

I believe that we might want to replace such libraries with some cross-platform ones like boost. We won't be able to always get away with simulating one environment in another. It would be better to have a code that works natively on all platforms without creating unix-like environment for it.

Does boost have an getopt alternative with both long and short opts?

https://www.boost.org/doc/libs/1_86_0/doc/html/program_options.html


Fix noDebug argument not being passed

fixes #99576
The problem was that we started storing full configurations in storage for debug dynamic configurations.
This broke the noDebug flag which could be stored for launch config "Launch Program" and the next time that launch config is started the stored noDebug would be respected.
This commit simply always overwrites the noDebug flag to be allowed with what is passed and thus what the user requested.

Thanks for the review. I have moved back to the previous fix. Merging this.


[slack] Select channel using CLI option

An option to change channel is useful for testing at least. Proposition: --slack-channel or -s for short.

ApplicationInterface must introduce a new optional argument (an array) to output method. The value of the argument will be merged with values read from the configuration.


Fix grep date test

Amends GrepDateTest to

Negate the exit code such that a match returns exit 1 (test fails) and no match returns exit 0 (test passes)
Removes -w option so that terms such as <AcquisitionDate> where the risk term is not a isolated word now match
Adds -a option to explicitly treat binary files as text
Adds -q option to suppress output as is not needed by the test

Note that for TIFF files this only searches within values of Tags where the content is a string, such as ImageDescription
We may wish to add an additional test that searches the DateTime tag itself.

More discussions here: https://github.com/Sage-Bionetworks-Workflows/py-dcqc/pull/30#issuecomment-1506766484


Fix for garbled callsite info after await-calls #1805

The current approach to the problem of garbled callsite info in async methods (skipping a stackframe in LoggerImpl) does not work for logging calls after an await-call in an async method.
This fix exploits the fact that the compiler-generated state machine is nested under the user type. It detects the state machine in the current stack frame in AppendClassName and AppendMethodName in CallSiteLayoutRenderer, then fetches class and method info from the DeclaringType property.
A property, CleanNamesOfAsyncContinuations, has been added to toggle the behaviour, following the example of CleanNamesOfAnonymousDelegates.
This fix might possibly replace the "skip-frame" approach in LoggerImpl, if desired. The callstack will not be fixed, but it isn't really fixed now anyway.

Thanks!

oops, merge was not working well. Feel free to remove the commit and force push

mmm, that won't worked well (the push)

replaced with https://github.com/NLog/NLog/pull/2292

superseded by https://github.com/NLog/NLog/pull/2292


DCOS-19882: fix(PlacementConstraintsSchemaField): remove escaping from output

https://jira.mesosphere.com/browse/DCOS-19882

@juliangieseke please let me know if I'm looking at the wrong place but the UI doesn't display the expected result (screenshot below). For that reason, I'm requesting changes.

@weblancaster I couldn't reproduce there issue you've encountered.


cmd/gofmt: incorrect alignment of map keys when Arabic characters are used

gofmt formats this like so:
var foo = map[string]string{
	"foo":        "bar",
	"foobar":     "foobar",
	"ﺎﻠﻋَﺮَﺒِﻳﺓ": "foo",
	"foo11":    "bar",
	"foobar11": "foobar",
}

However, the expected format would be:
var foo = map[string]string{
	"foo":        "bar",
	"foobar":     "foobar",
	"ﺎﻠﻋَﺮَﺒِﻳﺓ":    "foo",
	"foo11":      "bar",
	"foobar11":   "foobar",
}

It looks like it formats by byte count, rather than character count (did not investigate yet).
Go 1.6.2 on Linux

Character count is also insufficient, of course, since characters can have different widths. Still, it seems like we ought to be able to do better.

gofmt doesn't know about fonts. Without that, it can't do better, and even with fonts knowledge, there's not much it can do since it doesn't control how you view a file (what editor you are using).
gofmt is using text/tabwriter to achieve alignment. This is essentially a duplicate of #8273. There's also the related #12073.

PS: The issue is not just present for the wider range of Unicode chars, it is really a font issue. There are plenty of people that (would like to) use a proportional font for programming (me included) - after all proportional fonts are used because they increase readability - but gofmt cannot handle those either, even if it's all English.
Since gofmt operates on text files which know nothing about character width on the screen, it must assume that all characters have the same width. At best, it could know which characters are an (integer) multiple of the "unit" width, and then introduce spaces as necessary. However, that would not solve the general problem and unless an editor follows the same formatting, would look just as ugly.
Unfortunate.

gofmt doesn't know about fonts. Without that, it can't do better, and even with fonts knowledge, there's not much it can do since it doesn't control how you view a file (what editor you are using).

Hm, I would expect the Arabic text in xterm to be monospaced? I don't read Arabic and it's difficult to see, but I have the same problem with Cyrillic characters, which certainly seem monospaced:
var foo = map[string]string{
    "foo": "bar",
    "сииииияяяяя": "z",
    "f1": "foobar",
}

@Carpetsmoker This is a separate issue. Note the formatting of:
var foo = map[string]string{
	"foo": "bar",
	"сииииияяяяя": "z",
	"f123": "foobar",
}

but:
var foo = map[string]string{
	"foo": "bar",
	"сииииияяяяя": "z",
	"f1234":       "foobar",
}

There is explicit code in gofmt that computes the length ratio between keys on separate lines. If the ratio between two key lengths becomes too large (or too small, depending which way you divide), the alignment of columns is broken. Here the threshold is crossed between f123 and f1234.
The thinking is that we don't want very long such tables where a single long key (possibly far down and not even on the screen when you look at the source) moves all values to the right. It's a heuristic. Your mileage may vary.

@Carpetsmoker PS: You're right, when using a proportional font, alignment as computed by gofmt is useless. At that point, gofmt only standardizes white space at the beginning of a line. Still better than nothing.

This is a separate issue.

Ah, okay; thanks. It's not a huge issue anyway, but I've encountered this a few times and sometimes it's annoying that you can't manually correct these little things due to gofmt's "all or nothing approach"...

@Carpetsmoker And now that you mentioned it, your original example is following the same heuristics. Note that:
var foo = map[string]string{
	"foo":        "bar",
	"foobar":     "foobar",
	"ﺎﻠﻋَﺮَﺒِﻳﺓ": "foo",
	"foo11":    "bar",
	"foobar11": "foobar",
}

but:
var foo = map[string]string{
	"foo":        "bar",
	"foobar":     "foobar",
	"ﺎﻠﻋَﺮَﺒِﻳﺓ": "foo",
	"foo111":     "bar",
	"foobar11":   "foobar",
}

(GitHub's issue tracker formats a bit differently, but in Sublime the latter is perfectly aligned.) That is, again, here the foo11 crosses the threshold.
I double-checked the implementation of text/tabwriter, and it uses actual rune (Unicode) count to determine the width of text: see updateWidth in https://golang.org/src/text/tabwriter/tabwriter.go?h=updateWidth#L380.
Thus, this is all working as intended.

Hah, thanks for the clarification @griesemer. As a point of interest, it also seems to be Vim's fault:

As you can see, Emacs and Sublime Text work as expected, but Vim doesn't in xterm, lx-terminal, or as gVim...


[10.0-pos_multi_session_restaurant] kitchen printer state does not sync on table floor screen

Steps to reproduce :

set up a kitchen printer in 2 pos belonging to the same multisession
add items on a table that need to be printed in kitchen but do not press the order button
Actual behavior :
on the other pos, on the table floor, the order stays black --> we need to go on other floor and come back to see it's red
Expected behavior :
on the other pos, on the table floor, the order must be red without having to go on other floor and come back

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Confirmed


Remove deprecated endpoints

Removes the deprecated endpoints from the api service, including/plugins and /plugins/excluded.
The changes can be released only after the internal user for those endpoints have migrated away from it.

I refactored both analytics pipelines that hit the hub API regularly (PyPI and GitHub) to use the new endpoints. Both ran successfully.


Blender 4.0 segfaults when selecting 'CUDA' option in preferences

Launching Blender 4.0 on my Fedora machine (Vega 64 + ROCm 5.7) with the following command works, however, when selecting the 'CUDA' option in preferences, there is a segfault and the application crashes.
LD_LIBRARY_PATH="/home/baf9900/Documents/zluda/:$LD_LIBRARY_PATH" blender
This is the crash log of Blender, not sure how helpful it may be:
# Blender 4.0.2, Commit date: 1970-01-01 00:00, Hash unknown

# backtrace
blender(+0xef5c9c) [0x555e4183ec9c]
blender(+0x9549eb) [0x555e4129d9eb]
/lib64/libc.so.6(+0x3e9a0) [0x7f59b625c9a0]

# Python backtrace
  File "/usr/share/blender/4.0/scripts/addons/cycles/properties.py", line 1540 in get_devices_for_type
  File "/usr/share/blender/4.0/scripts/addons/cycles/properties.py", line 1684 in draw_impl
  File "/usr/share/blender/4.0/scripts/startup/bl_ui/space_userpref.py", line 647 in draw_centered
  File "/usr/share/blender/4.0/scripts/startup/bl_ui/space_userpref.py", line 182 in draw

Is Vega 64 working on ROCm 5.7? Offically AMD says that only Radeon VII is supported.

I believe it should do, even if it lacks official support? rocminfo seems to recognise it at least:
ROCk module is loaded
=====================
HSA System Attributes
=====================
Runtime Version:         1.1
System Timestamp Freq.:  1000.000000MHz
Sig. Max Wait Duration:<PHONE_NUMBER>3709551615 (0xFFFFFFFFFFFFFFFF) (timestamp count)
Machine Model:           LARGE
System Endianness:       LITTLE
Mwaitx:                  DISABLED
DMAbuf Support:          YES

==========
HSA Agents
==========
*******
Agent 1
*******
  Name:                    Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz
  Uuid:                    CPU-XX
  Marketing Name:          Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz
  Vendor Name:             CPU
  Feature:                 None specified
  Profile:                 FULL_PROFILE
  Float Round Mode:        NEAR
  Max Queue Number:        0(0x0)
  Queue Min Size:          0(0x0)
  Queue Max Size:          0(0x0)
  Queue Type:              MULTI
  Node:                    0
  Device Type:             CPU
  Cache Info:
    L1:                      32768(0x8000) KB
  Chip ID:                 0(0x0)
  ASIC Revision:           0(0x0)
  Cacheline Size:          64(0x40)
  Max Clock Freq. (MHz):   5000
  BDFID:                   0
  Internal Node ID:        0
  Compute Unit:            16
  SIMDs per CU:            0
  Shader Engines:          0
  Shader Arrs. per Eng.:   0
  WatchPts on Addr. Ranges:1
  Features:                None
  Pool Info:
    Pool 1
      Segment:                 GLOBAL; FLAGS: FINE GRAINED
      Size:                    65697620(0x3ea7754) KB
      Allocatable:             TRUE
      Alloc Granule:           4KB
      Alloc Alignment:         4KB
      Accessible by all:       TRUE
    Pool 2
      Segment:                 GLOBAL; FLAGS: KERNARG, FINE GRAINED
      Size:                    65697620(0x3ea7754) KB
      Allocatable:             TRUE
      Alloc Granule:           4KB
      Alloc Alignment:         4KB
      Accessible by all:       TRUE
    Pool 3
      Segment:                 GLOBAL; FLAGS: COARSE GRAINED
      Size:                    65697620(0x3ea7754) KB
      Allocatable:             TRUE
      Alloc Granule:           4KB
      Alloc Alignment:         4KB
      Accessible by all:       TRUE
  ISA Info:
*******
Agent 2
*******
  Name:                    gfx900
  Uuid:                    GPU-0215219f78122104
  Marketing Name:          AMD Radeon RX Vega
  Vendor Name:             AMD
  Feature:                 KERNEL_DISPATCH
  Profile:                 BASE_PROFILE
  Float Round Mode:        NEAR
  Max Queue Number:        128(0x80)
  Queue Min Size:          64(0x40)
  Queue Max Size:          131072(0x20000)
  Queue Type:              MULTI
  Node:                    1
  Device Type:             GPU
  Cache Info:
    L1:                      16(0x10) KB
    L2:                      4096(0x1000) KB
  Chip ID:                 26751(0x687f)
  ASIC Revision:           1(0x1)
  Cacheline Size:          64(0x40)
  Max Clock Freq. (MHz):   1630
  BDFID:                   768
  Internal Node ID:        1
  Compute Unit:            64
  SIMDs per CU:            4
  Shader Engines:          4
  Shader Arrs. per Eng.:   1
  WatchPts on Addr. Ranges:4
  Features:                KERNEL_DISPATCH
  Fast F16 Operation:      TRUE
  Wavefront Size:          64(0x40)
  Workgroup Max Size:      1024(0x400)
  Workgroup Max Size per Dimension:
    x                        1024(0x400)
    y                        1024(0x400)
    z                        1024(0x400)
  Max Waves Per CU:        40(0x28)
  Max Work-item Per CU:    2560(0xa00)
  Grid Max Size:<PHONE_NUMBER>(0xffffffff)
  Grid Max Size per Dimension:
    x<PHONE_NUMBER>(0xffffffff)
    y<PHONE_NUMBER>(0xffffffff)
    z<PHONE_NUMBER>(0xffffffff)
  Max fbarriers/Workgrp:   32
  Packet Processor uCode:: 469
  SDMA engine uCode::      434
  IOMMU Support::          None
  Pool Info:
    Pool 1
      Segment:                 GLOBAL; FLAGS: COARSE GRAINED
      Size:                    8372224(0x7fc000) KB
      Allocatable:             TRUE
      Alloc Granule:           4KB
      Alloc Alignment:         4KB
      Accessible by all:       FALSE
    Pool 2
      Segment:                 GLOBAL; FLAGS:
      Size:                    8372224(0x7fc000) KB
      Allocatable:             TRUE
      Alloc Granule:           4KB
      Alloc Alignment:         4KB
      Accessible by all:       FALSE
    Pool 3
      Segment:                 GROUP
      Size:                    64(0x40) KB
      Allocatable:             FALSE
      Alloc Granule:           0KB
      Alloc Alignment:         0KB
      Accessible by all:       FALSE
  ISA Info:
    ISA 1
      Name:                    amdgcn-amd-amdhsa--gfx900:xnack-
      Machine Models:          HSA_MACHINE_MODEL_LARGE
      Profiles:                HSA_PROFILE_BASE
      Default Rounding Mode:   NEAR
      Default Rounding Mode:   NEAR
      Fast f16:                TRUE
      Workgroup Max Size:      1024(0x400)
      Workgroup Max Size per Dimension:
        x                        1024(0x400)
        y                        1024(0x400)
        z                        1024(0x400)
      Grid Max Size:<PHONE_NUMBER>(0xffffffff)
      Grid Max Size per Dimension:
        x<PHONE_NUMBER>(0xffffffff)
        y<PHONE_NUMBER>(0xffffffff)
        z<PHONE_NUMBER>(0xffffffff)
      FBarrier Max Size:       32
*** Done ***

It could also be my installation of ROCm that is at fault. I'm using the packages provided in the Fedora repos.

Can confirm it works on blender for windows on the 7900xt. it took A LOT of time to load the render kernels the first time, but then it was able to render a small benchmark scene. so it shouldn't be Blender the problem.
On Linux (ArchLinux) it couldn't find my gpu when selecting CUDA, but probably i was just doing something wrong, i'll try again later.
Just to be sure, are you able to render a scene in cycles using your gpu if you select HIP?

Just to be sure, are you able to render a scene in cycles using your gpu if you select HIP?

It has worked in the past, although, stability left a lot to be desired with it often crashing while part way rendering a scene. Recently, as soon as I select my GPU as a HIP rendering device Blender crashes :( That's why I was looking forward to giving this a shot.
Other ROCm packages such as rocm-opencl from the Fedora repos seem to work for me, with applications like Resolve recognising my Vega 64 as an OpenCL device.

Just to be sure, are you able to render a scene in cycles using your gpu if you select HIP?

It has worked in the past, although, stability left a lot to be desired with it often crashing while part way rendering a scene. Recently, as soon as I select my GPU as a HIP rendering device Blender crashes :( That's why I was looking forward to giving this a shot.
Other ROCm packages such as rocm-opencl from the Fedora repos seem to work for me, with applications like Resolve recognising my Vega 64 as an OpenCL device.

Then your issue is in rocm. Zluda uses rocm as backend, if i undestand it right,  so it's probably the same issue

Further digging points to an issue somewhat related to this perhaps - https://bugzilla.redhat.com/show_bug.cgi?id=2216594 ?
But yeah I'll close this because it doesn't seem to be an issue with ZLUDA specifically. Regardless, maybe it's time to retire the Vega 64.


catch subscribed NodeViewChange events

Coverage remained the same at 46.548% when pulling 2527d151d4f1f68617a63f8ca0a0895180e8cdb1 on sync into 54e6b9299a837da930104a276b0503726af1a0ae on master.

@catena2w ,
I was planning to fix this issue in a different way. I would not subscribe NodeViewSynchronizer to vault and state modifications.
My understanding, from discussions with @kushti in Slack, was that vault and state modification events were supposed to be received and handled not by NodeViewSynchronizer but by another actor of the application.
Should I still go ahead and refactor this fix, after this PR is accepted?
Another thing I wanted to do is to move event messages to the same file of the actor that emits them. Right now, event messages are located in the same file of the actor that is supposed to receive them, but this is problematic in cases such as vault and state modification events, where we do not know the actor that should receive them.

@ceilican you can fix it in your way


logging/#gelf-logger is missing JSON configuration example

The https://github.com/keboola/developers-docs/blob/master/extend/common-interface/logging.md#setting-up page is missing example of JSON configuration that is needed in Developer portal.
Something like
{
  "verbosity": {
    "100": "none",
    "200": "normal",
    "alert": "camouflage",
    "error": "verbose",
    "notice": "normal",
    "warning": "normal",
    "critical": "camouflage",
    "emergency": "camouflage"
  },
  "gelf_server_type": "tcp"
}

https://keboola.atlassian.net/browse/DOCS-46


Bump springboot from 2.7.9 to 2.7.10

Brief changelog
Bump springboot from 2.7.9 to 2.7.10. This PR also closes #4756
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.
[x] Update the CHANGES log.

It looks like the snakeyml version referenced in 2.7.10 is still 1.30.

@nobodyiam My apologizes, I will wait for next springboot release.

review again

Hello, I am a serverless review bot on flows.network. Here are my reviews of code commits in this PR.

Commit 1
The patch is about upgrading the version of Spring Boot from 2.7.9 to 2.7.10. There are no potential problems identified in this patch, and it appears to be a straightforward update.

@nobodyiam @Anilople Although this springboot version doesn't bump snakeyaml's version. But I think it's also worth to update it because it also solve other cves. Like CVE-2023-20863 etc.


An invalid response was received from the upstream server.

Now my application is available，I will add this api to kongserver

Kong Error
An invalid response was received from the upstream server.
Is this what the problem , what should I do

closing this, answer is on gitter


Some quick PEPification changes

Some quick PEP changes, mostly to help minimize line length. Some other behavior changes, let me know what you guys think @tsalo @mriedel56

Sorry for the delay. What behavior changes?

LGTM. Shouldn't change behavior and makes things easier to read.


Release new version for updated dependencies?

loader-utils are required afaict, but are not included in the current npm versions dependencies list.

You're right, the v0.9.0 tag doesn't declare loader-utils in package.json dependencies:
https://github.com/kaitai-io/kaitai_struct_loader/blob/ce34822849f0e735011d6d5ca8b9b1f9b968aab8/package.json#L23-L27
but uses it:
https://github.com/kaitai-io/kaitai_struct_loader/blob/ce34822849f0e735011d6d5ca8b9b1f9b968aab8/index.js#L2
Which is obviously an error. However, if I'm supposed to release a new version, I should really fix a few things - first, there is a growing queue of Dependabot pull requests that I should merge so that any known vulnerabilities get fixed in the dependencies. The only reason why I have been leaving them unmerged is that I got an impression from this issue https://github.com/kaitai-io/kaitai_struct/issues/729 that it's desirable for our users that we maintain compatibility with the lowest possible version of Node.js that is installed on some LTS Linux distribution (I don't like and understand it much, I'd rather support only Node.js versions that aren't EOL, which would mean Node.js 12 and 14 at the moment - that would make life so much easier. But I'm no expert on Node.js, I don't use it in any serious application, so I don't dare to make such bold decisions like that on my own.). That's why I was shooting for Node.js {4,8,10,12,14} at first, but then found that Node.js 4 doesn't work with yarn, so I raised it to Node.js 6. And I'm sure that some of the dependency updates suggested by Dependabot would break some of the versions (they would probably break compatibility at least with Node.js 4 and 8), so I was not merging them right away and I've been hoping to find some time to try locally if these dependencies break the compatibility or not.
Also, I don't know if I need to use a united old dependency version compatible with Node.js 6 for all Node.js versions (even for Node.js 14, which could run something much newer) or whether there is some way to declare different dependency versions for different Node.js versions (but I don't know if e.g. Webpack 5 won't use a different syntax that will make it incompatible with current codebase). Do you see that "lowest possible version" madness?
The second thing that I wanted to fix are relative imports. I thought that the PR https://github.com/kaitai-io/kaitai_struct_loader/pull/5 added them, but I found that it has never actually worked as soon as I introduced the unit testing in  the commit https://github.com/kaitai-io/kaitai_struct_loader/commit/684faf475cae6521bf3fa214a28bc5676e044e0d. I tried to fix it, but I essentially got stuck because I'm not that proficient in writing Webpack plugins. IIRC, the problem is that the Kaitai Struct compiler is handling the imports itself, which Webpack does not expect - when KSC sees a meta/imports declaration, it requests the appropriate .ksy spec from the filesystem, processes it and at the end returns a map with generated JS module filename as a key (e.g. HelloWorld.js) and the JS code as a value. However, Webpack does not work like that - when you write import DoomWAD from './game/doom_wad.ksy', Webpack reads contents of that file and gives it as a source code to the kaitai_struct_loader plugin. In return, it expects a single JavaScript module (generated purely from that source code that it provided), then processes it, searches for any import ... or require(...) statements (and the KS-generated modules with imports always contain require('./HelloWorld') for meta/imports: [hello_world]) and then it gets mad because there is no HelloWorld.js file there. I haven't yet figured out how to adapt the kaitai_struct_loader for this or how to change Webpack behavior.

Regarding the issue with the automatic CamelCasing of requires by the KSY compiler, it looks like if you stick to CamelCasing the actual KSY files, the requires will match the filenames and webpack will figure out things (you ofcourse also need to CamelCase the import statement, though the type you import will be based on the meta-id field from the KSY file, so it will have the "proper" ksy lowercase naming)
Thisi s really a gross hack, and IMHO the compiler should simply not CamelCase the requires, or at least offer an option to not do that.


MDC Toolbar

MDC Toolbar was added.
List of classes:

Toolbar
ToolbarSection
Content (for fixed toolbar functionality)

merged


Add property types and refactor how auto-generated classes are integrated

The majority of this pull request is adding or changing auto-generated files; I will comment in-line the places where you should pay careful attention.
This is very much an incremental pull request -- all tests pass, but old tests that are no longer relevant given the new architecture have been removed. Furthermore, the documentation is no longer accurate, and will need to be updated.  With this request, we are one step closer to 1.0.0-beta1 (but still a couple requests off!)

Many of these auto-generated classes are missing the license header; I'll add them in a later pull request once the architecture has been nailed down a little more

looking at it now!


Fix coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.319%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
5440

Relevant Lines:
6773

💛  - Coveralls


Allow Windows to use case insensitive path completion

Fix #382

@gwenn if you don't mind my asking, when is the next release? anxious to get this change in.


⚠ SceneTime has degraded performance

In aa72aaf, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 9416 ms

Resolved: SceneTime performance has improved in 348cebc after 7 minutes.


<epub:trigger handler does not stop propagation of event

This issue is a Bug
When viewing the document found here http://epubtest.org/epubs/epub30-test-0100-20160116.epub; one of the pages has a test for epub-trigger.  The first test validates if an external button can cause the audio playback to begin.
This external button is configured using <ebpub:trigger .....>
The button handler executes correctly, but the event itself is then bubbled up.  If the reader has another event handler wired to the dom tree, that event handler will also execute.
In my case, I have a click handler wired to the body element, which I use to control page navigation (click on the left side of the page, go left; click on the right side of the page, go right).  In my case, if the user clicks on the button, I want the button action to occur and not the navigation action.
Ideally, the code in trigger.js would return false, which would cause the event not to bubble up.
Using readium-js-viewer, dev branch; in a hybrid app.
(Latest Chrome, windows 7)

note that the epub-testsuite files are linked from the default cloud reader library:
https://readium.firebaseapp.com/?epubs=epub_content%2Fepub_testsuite.opds

here is a direct link to the trigger page:
https://readium.firebaseapp.com/?epub=https%3A%2F%2Fcdn.rawgit.com%2FIDPF%2Fepub-testsuite%2Fmaster%2Fcontent%2F30%2Fepub30-test-0100&goto={"idref"%3A"xhtml-multi-003"%2C"elementCfi"%3A"%2F4%2F2%2F2[trigger]%2F2%2C%2F1%3A0%2C%2F1%3A1"}&

code reference:
https://github.com/readium/readium-shared-js/blob/develop/js/models/trigger.js#L103

Fixed by https://github.com/readium/readium-shared-js/pull/346


1.1.1r Not Available in GitHub

I am trying to find the 1.1.1r release that just came out, but it does not seem to be in GitHub. Any pointers to the commit?

You will not find a download for that release (it is of course in github) as there was an issue and it was withdrawn.
https://www.openssl.org/news/newslog.html

12-Oct-2022 OpenSSL 3.0.6 and 1.1.1r are withdrawn. New releases will be created in due course.

1.1.1r release that just came out,

There was no new release....1.1.1r was released (and withdrawn) over a year ago. A spurious email seems to have gone out that was a year old.


Provider produced inconsistent final plan with automated logpush ownership challenge

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.5.1
on linux_amd64
provider registry.terraform.io/cloudflare/cloudflare v4.14.0
provider registry.terraform.io/hashicorp/google v4.81.0

Affected resource(s)

cloudflare_logpush_job
cloudflare_logpush_ownership_challenge

Terraform configuration files
resource "cloudflare_logpush_ownership_challenge" "ownership_challenge" {
  zone_id = module.cloudflare_id.zone_id[0]

  destination_conf = local.destination_conf
}
data "google_storage_bucket_object_content" "challenge_file" {
  bucket = "<bucket_name>"
  name   = cloudflare_logpush_ownership_challenge.ownership_challenge.ownership_challenge_filename
}
resource "cloudflare_logpush_job" "http_logs" {
  zone_id = module.cloudflare_id.zone_id[0]

  dataset                     = "http_requests"
  destination_conf            = local.destination_conf
  enabled                     = true
  filter                      = var.filter
  frequency                   = var.frequency
  logpull_options             = "fields=${local.http_fields}&timestamps=unixnano&CVE-2021-44228=${var.log4j}"
  max_upload_bytes            = var.max_upload_bytes
  max_upload_interval_seconds = var.max_upload_interval_seconds
  ownership_challenge         = data.google_storage_bucket_object_content.challenge_file.content
}

Link to debug output
https://gist.github.com/Arnall/5a5823566c96191a4cbb1ec5ae7f90b4
Panic output
No response
Expected output
no error during apply
Actual output
"cloudflare_logpush_job.http_logs" error: Provider produced inconsistent final plan
When expanding the plan for cloudflare_logpush_job.http_logs to include new
values learned so far during apply, provider
"registry.terraform.io/cloudflare/cloudflare" produced an invalid new value
for .ownership_challenge: was null, but now
cty.StringVal("[redacted]..[redacted]").
This is a bug in the provider, which should be reported in the provider's
own issue tracker.
Steps to reproduce
Just try to create cloudflare_logpush_job and cloudflare_logpush_ownership_challenge resources with one plan/apply.
Additional factoids
hi @jacobbednarz , I'm sorry to open this issue again, you answered my previous one: https://github.com/cloudflare/terraform-provider-cloudflare/issues/2752 (didn't find where i can reopen the issue and it seems you don't answer on comments in closed issues) but adding a dependency changes nothing... so i still think there is a problem... I add info here :
So far i tried:
resource "cloudflare_logpush_ownership_challenge" "ownership_challenge" {
  zone_id = module.cloudflare_id.zone_id[0]

  destination_conf = local.destination_conf
}
data "google_storage_bucket_object_content" "challenge_file" {
  depends_on = [resource.cloudflare_logpush_ownership_challenge.ownership_challenge]

  bucket = "[REDACTED]"
  name   = cloudflare_logpush_ownership_challenge.ownership_challenge.ownership_challenge_filename
}

or
data "google_storage_bucket_object_content" "challenge_file" {
  bucket = "[REDACTED]"
  name   = cloudflare_logpush_ownership_challenge.ownership_challenge.ownership_challenge_filename
}
resource "cloudflare_logpush_job" "http_logs" {
  depends_on = [data.google_storage_bucket_object_content.challenge_file]

  zone_id = module.cloudflare_id.zone_id[0]

  dataset                     = "http_requests"
  destination_conf            = local.destination_conf
  enabled                     = true
  filter                      = var.filter
  frequency                   = var.frequency
  logpull_options             = "fields=${local.http_fields}&timestamps=unixnano&CVE-2021-44228=${var.log4j}"
  max_upload_bytes            = var.max_upload_bytes
  max_upload_interval_seconds = var.max_upload_interval_seconds
  ownership_challenge         = data.google_storage_bucket_object_content.challenge_file.content
}

or
resource "cloudflare_logpush_ownership_challenge" "ownership_challenge" {
  zone_id = module.cloudflare_id.zone_id[0]

  destination_conf = local.destination_conf
}

resource "cloudflare_logpush_job" "http_logs" {
  depends_on = [resource.cloudflare_logpush_ownership_challenge.ownership_challenge]

  zone_id = module.cloudflare_id.zone_id[0]

  dataset                     = "http_requests"
  destination_conf            = local.destination_conf
  enabled                     = true
  filter                      = var.filter
  frequency                   = var.frequency
  logpull_options             = "fields=${local.http_fields}&timestamps=unixnano&CVE-2021-44228=${var.log4j}"
  max_upload_bytes            = var.max_upload_bytes
  max_upload_interval_seconds = var.max_upload_interval_seconds
  ownership_challenge         = data.google_storage_bucket_object_content.challenge_file.content
}

it always lead to the same plan :
  # data.google_storage_bucket_object_content.challenge_file will be read during apply
  # (config refers to values not yet known)
 <= data "google_storage_bucket_object_content" "challenge_file" {
      + bucket              = "[REDACTED]"
      + cache_control       = (known after apply)
      + content_disposition = (known after apply)
      + content_encoding    = (known after apply)
      + content_language    = (known after apply)
      + content_type        = (known after apply)
      + crc32c              = (known after apply)
      + customer_encryption = (known after apply)
      + detect_md5hash      = (known after apply)
      + event_based_hold    = (known after apply)
      + id                  = (known after apply)
      + kms_key_name        = (known after apply)
      + md5hash             = (known after apply)
      + media_link          = (known after apply)
      + metadata            = (known after apply)
      + name                = (known after apply)
      + output_name         = (known after apply)
      + self_link           = (known after apply)
      + source              = (known after apply)
      + storage_class       = (known after apply)
      + temporary_hold      = (known after apply)
    }

  # cloudflare_logpush_job.http_logs will be created
  + resource "cloudflare_logpush_job" "http_logs" {
      + dataset                     = "http_requests"
      + destination_conf            = "gs://[REDACTED]/test/test.com/{DATE}"
      + enabled                     = true
      + frequency                   = "low"
      + id                          = (known after apply)
      + logpull_options             = "fields=CacheCacheStatus,CacheResponseBytes,CacheTieredFill,ClientCountry,ClientIP,ClientIPClass,ClientRequestHost,ClientRequestMethod,ClientRequestProtocol,ClientRequestReferer,ClientRequestSource,ClientRequestURI,ClientRequestUserAgent,ClientSrcPort,EdgeColoCode,EdgeEndTimestamp,EdgePathingSrc,EdgePathingStatus,EdgeResponseBytes,EdgeResponseStatus,EdgeStartTimestamp,EdgeTimeToFirstByteMs,OriginIP,OriginResponseBytes,OriginResponseDurationMs,OriginResponseHeaderReceiveDurationMs,OriginResponseStatus,ParentRayID,RayID,SmartRouteColoID,UpperTierColoID,WorkerCPUTime,WorkerStatus&timestamps=unixnano&CVE-2021-44228=false"
      + max_upload_bytes            = 104857600
      + max_upload_interval_seconds = 300
      + zone_id                     = "[redacted]"
    }

  # cloudflare_logpush_ownership_challenge.ownership_challenge will be created
  + resource "cloudflare_logpush_ownership_challenge" "ownership_challenge" {
      + destination_conf             = "gs://[REDACTED]/test/test.com/{DATE}"
      + id                           = (known after apply)
      + ownership_challenge_filename = (known after apply)
      + zone_id                      = "[redacted]"
    }

Plan: 2 to add, 0 to change, 0 to destroy

As you can see in the plan the attribute ownership_challenge in cloudflare_logpush_job.http_logs does not appear (it's considered null) instead of appearing with (known after apply) as other attributes in cloudflare_logpush_ownership_challenge.ownership_challenge or data.google_storage_bucket_object_content.challenge_file.
If i do something wrong please tell me...
Thanks.
References
https://github.com/cloudflare/terraform-provider-cloudflare/issues/2752

please don't open duplicate issues for the same problems already discussed in other issues.
recommendations have already been provided in the other issue and seeing how you are still using dynamics you will unfortunately need to debug those yourself as they are an abstraction on top of the provider which we do not support. you can use terraform logging and built in internals to diagnose why you are getting the values you currently are.

The problem is, i followed your answer in the other issue... i do not understand what i'm doing wrong, i do not understand what you mean by "abstraction on top of the provider", i think i use the provider the right way and obviously it does not work the way it should...

same here... I still couldn't find a workaround

resource "cloudflare_logpush_ownership_challenge" "ownership_challenge" {
zone_id          = var.cloudflare_zone_id
destination_conf = var.gcp_bucket_destination_url
}
resource "null_resource" "fetch_gcp_bucket_file_content" {
depends_on = [cloudflare_logpush_ownership_challenge.ownership_challenge]
provisioner "local-exec" {
command = "gsutil cat ${var.gcp_bucket_destination_url}/${cloudflare_logpush_ownership_challenge.ownership_challenge.ownership_challenge_filename} > ${path.module}/${cloudflare_logpush_ownership_challenge.ownership_challenge.ownership_challenge_filename}"
}
}
data "local_file" "bucket_file" {
depends_on = [null_resource.fetch_gcp_bucket_file_content]
filename   = "${path.module}/${cloudflare_logpush_ownership_challenge.ownership_challenge.ownership_challenge_filename}"
}
resource "cloudflare_logpush_job" "job_http_requests" {
depends_on = [data.local_file.bucket_file]
enabled             = true
zone_id             = var.cloudflare_zone_id
name                = var.lougpush_name
destination_conf    = var.gcp_bucket_destination_url
logpull_options     = var.logpull_job_options
ownership_challenge = data.local_file.bucket_file.content
dataset             = "http_requests"
}


tintColor for ActionSheet

ActionSheetIOS supports tintColor. It'd be nice to be able to configure it for this library as well.

Would gladly accept a PR.

PR is open: https://github.com/react-native-community/react-native-image-picker/pull/898


Added change log for 1.5.2

Signed-off-by: Asif Sohail Mohammed<EMAIL_ADDRESS>Description
Adds change log for Data Prepper 1.5.2 release
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1977 (bac6657) into main (7a6f747) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1977      +/-   ##
============================================
- Coverage     93.93%   93.88%   -0.05%
- Complexity     1536     1537       +1
============================================
  Files           197      197
  Lines          4565     4565
  Branches        367      367
============================================
- Hits           4288     4286       -2
- Misses          194      196       +2
  Partials         83       83

Impacted Files
Coverage Δ

...rwarder/discovery/AwsCloudMapPeerListProvider.java
92.68% <0.00%> (-2.44%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Test Checks

What does this PR do?
Test

@fniephaus this reports are very agressive. Can we disable them somehow?

@fniephaus this reports are very agressive. Can we disable them somehow?

Yes, we can. Will file a PR next week. BTW: You can skip CI by adding [ci skip] to your commit message.


Minor - Fixing catalog id in delete col + update docs

Issue #, if available:
#975
Description of changes:

Fixed _catalog_id in get_table call of delete_colum
Moved SAR section to layers in docs

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: c596827e01c24a32c5c316b03542f1f9869384ac
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Remove CRDs failed while uninstalling hwameistor

$ kubectl get crd -o name
| grep hwameistor
| xargs -t kubectl delete
Cannot delete localvolumegroups.hwameistor.io

we will fix the uninstall steps at docs .


Do you have any plan about supporting reactor-netty,and spring cloud gateway can work with it.

Do you have any plan about supporting reactor-netty, and spring cloud gateway can work with it.

Thank you for your question.
I would like to support reactor-netty in future versions.

Is this the issue that tracks progress on netty/WebFlux support?


Rename framework.config to test-juggler.config

also in all references and readme.md
Keep in mind this will be breaking change, so only do with major version upgrade


⚠ Bofrost.at has degraded performance

In 33ecc08, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1156 ms

Resolved: Bofrost.at performance has improved in cafda9f after 13 minutes.


Update install.sh

On my system with Fedora 38 service won't start before you make script runnable

Thanks for your contribution!


[BUILD] Close stale PRs

Closes #12968
Closes #16215
Closes #16212
Closes #16086
Closes #15713
Closes #16413
Closes #16396

Test build #70775 has started for PR 16447 at commit ecaca37.

Test build #70775 has started for PR 16447 at commit ecaca37.

Test build #70775 has finished for PR 16447 at commit ecaca37.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #70775 has finished for PR 16447 at commit ecaca37.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70775/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70775/
Test FAILed.

Merged build finished. Test FAILed.

(Yeah I ignore Jenkins here because there are no changes whatsoever in this PR)

(Yeah I ignore Jenkins here because there are no changes whatsoever in this PR)


The non-developer "learn how to get involved" link on the homepage is broken

http://docs.opencivicdata.org/en/latest/anthropod/index.html page does not exist yet.

Link is gone.


XART token update failure

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[x] question about how to use this project

*Summary
I made made an update to my token to change the address which and token symbol but this pull request has not been merged for almost 4 weeks. Is anyone able to help me address this please, next phase of our project is to link the Token to our NFT project

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)
Pull request details below;
https://github.com/solana-labs/token-list/pull/12674

squashed and merged


Formal verification on Huff

Hello,
How can we help apply formal verification on the Huff language? Is the verification of the weierstrudel smart contract a good starting point? Would the huff-rs compiler itself benefit from a formal verification of its code? Thanks.
Guillaume Claret for https://formal.land/

My current approach to formally verifying Huff code bases such as METH is by leveraging hevm to verify a spec written in Solidity: https://github.com/Philogy/meth-weth/blob/main/test/fv/METH.fv.sol As of this writing hevm is missing some features (such as symbolic pranks) but dxo (hevm dev) has told me these features are coming soon™.


Version3.2

Pull Request Test Coverage Report for Build 101

225 of 250 (90.0%)  changed or added relevant lines in 12 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 92.278%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/AccelerationCard.class.st
9
12
75.0%

repository/IngSoft2-Model/NoCard.class.st
0
3
0.0%

repository/IngSoft2-Model/RandomDeck.class.st
0
6
0.0%

repository/IngSoft2-Model/SpeedCard.class.st
7
13
53.85%

repository/IngSoft2-Model/Game.class.st
109
116
93.97%

Totals

Change from base Build 83:
0.2%

Covered Lines:
478

Relevant Lines:
518

💛  - Coveralls

Pull Request Test Coverage Report for Build 101

225 of 250 (90.0%)  changed or added relevant lines in 12 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 92.278%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/AccelerationCard.class.st
9
12
75.0%

repository/IngSoft2-Model/NoCard.class.st
0
3
0.0%

repository/IngSoft2-Model/RandomDeck.class.st
0
6
0.0%

repository/IngSoft2-Model/SpeedCard.class.st
7
13
53.85%

repository/IngSoft2-Model/Game.class.st
109
116
93.97%

Totals

Change from base Build 83:
0.2%

Covered Lines:
478

Relevant Lines:
518

💛  - Coveralls


How to use JWT token user id as policy key to limit traffic to each user

Describe the question
How to use JWT token user id as policy key to limit traffic to each user
I have multiple user who are coming to my apigateway using JWT token and I would like to rate limit using JWT user id, how can I achieve that, any help would be highly appreciated.

Hi @pradeepkusingh. Unfortunately, as of now, this is not a supported feature. There's a draft similar to this one at #347, but it wasn't finished.

Thanks @marcosbarbero , do we have any plan to wrap up the #347 ?

Unfortunately, I'm out of time to do that. If you feel like creating a PR I will be happy to review it

Hi @pradeepkusingh, you should be able to do this by creating a custom RateLimitKeyGenerator, where the key is the JWT user id. Of course you will need to parse and validate your JWTs inside the generator implementation - but this approach should work.


Fix toPromise, firstToPromise, skipWhile, takeWhile in readme

First commit just strips trailing spaces from readme-src.coffee because Atom does that by default, and it doesn't affect how the output in README.md is rendered.

Second commit fixes the description of toPromise (it gives the last event, not the first) and adds firstToPromise to the readme.

Third commit addresses #559.

LGTM.
As we are here, could you add that toPromise and fromPromise take an optional Promise as a second argument?

Got it.

:+1: thanks

It doesn't look like http://baconjs.github.io/api.html was ever rebuilt to incorporate this change.

site rebuilt.


fix: refresh page index after changing search keyword

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #1655
[ ] <brief_description_of_what_I've_added>
The problem is that If you are on the 10th page of a search and search for something new, you go to the 10th page of the latest search result. Happens for collection as well.

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[x] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[x] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[] My fix has changed something on UI; a screenshot is best to understand changes for others.

I'm not a fan of using event bus, isn't there another solution to correct this problem?

Yes 🥺 please, avoid event bus for any costs

@roiLeo @yangwao EventBus is the quickest solution to this bug. Of course, there are other elegant ways, but it will take me a long time to process - -!

wait I'm confused about what this PR fixes

The problem is that If you are on the 10th page of a search and search for something new, you go to the 10th page of the latest search result. Happens for collection as well.

This isn't happening for me on the gallery component and the collection search might be fixed in #2511

There is still the same problem online. You can have a try according to this video
https://user-images.githubusercontent.com/31397967/157041990-c8c697ec-c795-4dbd-b59e-9a3677eea826.mov

There is still the problem online. You can have a try to search on the nav bar.
Screen.Recording.2022-03-07.at.9.19.04.PM.mov

Hi! Please see if this problem needs to be fixed ？ @yangwao

Oh, I see now. You are right. When I search for nft, go to the 10th page, then search for to I'm actually on 10th page but URL shows page 1.
Plus I've found another issue, when I share someone's URL https://beta.kodadot.xyz/rmrk/gallery/?search=nft&page=10 it actually pushes me on page 1, rewriting the 10th page to first. If you can fix it with this that would be superior! :)

If you @Jarsen136 join it with #2591 that would be superior! :)

I'm not a fan of using event bus, isn't there another solution to correct this problem?

Alright, I use another simple and reasonable solution to solve this bug instead of using event bus. 👋

wfm! waiting some approval if so!

yolo

pay 100 usd

😍 Perfect, I’ve sent the payout
💵 $100 @ 144.28 USD/KSM ~ 0.693 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0xa019e8664673f1a96ac01516dc224ab131a538f33f9469797fd46abe7869424a
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Implement remove function from list to Calendar Event

Connect the remove function that removes and item from the list to the calendar component.

use ID generated by Andrews generate UUID to add delete method in StaffAdd or StaffList to delete just that ID and it should render as removed in the list and calendar without having to refresh, still researching and tinkering

4 hours 4/4, and an hour 4/5, still not rendering properly, Andrew is continuing to work out the fix today


Calliope: wrong simulation warning for "switch LED matrix"

To Reproduce
Steps to reproduce the behavior:

Create a program with the "switch LED matrix ..." block
Click on "run on Calliope Mini"
See error

Expected behavior

No warning

Resolved by a GCI student.


Azure Domain Services join Azure VM with powershell

[Enter feedback here]
There does'nt seem to be a clear tutorial on how to join AZ AD DS using powershell for windwows server core running only on cmd

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

ID: 038b3b50-317c-c0c1-cbee-c69f3f828f84
Version Independent ID: 93810ce3-0817-c221-beb4-35de3f40d43e
Content: Azure Active Directory Domain Services: Join a Windows Server VM to a managed domain
Content Source: articles/active-directory-domain-services/join-windows-vm.md
Service: active-directory
Sub-service: domain-services
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@rexhinvorpsi Thanks for your feedback! We will investigate and update as appropriate.

@rexhinvorpsi Have you looked into the document Installing AD DS by Using Windows PowerShell ?

Hi ,
Will this work the same for an Azure VM. It is actually a kubernetes node created via aks-engine.

@rexhinvorpsi can you please help here if you know any documentation around the same or if the documentation can be updated with the PowerShell cmdlets for Azure VMs.

Hi again ,
I am really sorry because i am really new in the AKS Azure env and can't follow the documentation. I just wanted to domain join the Azure Vm with the azure active directory , i know there is a deployment file but that isn't working so i decided to do it myself via powershell.
Thank you very much in advance
Best Regards
Rexhin

I'm a little unsure what you're trying to do, @rexhinvorpsi. The AKS VM nodes aren't necessarily going to pass through any domain-joined benefits to the containers they run. There's also limited actions that you can perform on those VM nodes in a supported manner - https://docs.microsoft.com/en-us/azure/aks/support-policies
Windows Server containers themselves that run on the AKS nodes could also be domain-joined, but again, I'm not sure what is you're trying to accomplish here. As containers may be short-lived, you may not necessarily want to domain-join them as then you have to perform clean-up tasks as those containers are then terminated.
The basic PowerShell cmdlet for what you want to do is Add-Computer - https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.management/add-computer
A basic example is:
Add-Computer -DomainName CONTOSO -Restart

I'll update the doc to add this cmdlet.
Make sure that the virtual network your AKS cluster is deployed to is peered to the virtual network subnet where Azure AD DS is deployed.
And also remember that Windows Server support in AKS in preview. I don't think any of these things are necessarily specific to AKS, rather how you handle domain-joins in a Kubernetes cluster. You may be better served detailing your needs at https://stackoverflow.com/questions/tagged/azure-kubernetes to get design and implementation assistance.

Thank you for the great explanation . I see, i am trying to enable gsma in the AKS cluster via feature gates and referred to this article https://kubernetes.io/docs/tasks/configure-pod-container/configure-gmsa/ , given that the AKS service didn't support feature gates , i used the open source aks-engine to enable feature gates and deploy the cluster. Now the part fo creating the GSMA Service account was unclear for me at least, as i said i am new in AKS and Azure.
Thank you very much for your time

Yes, AKS doesn't currently support gMSA. Along with the kubernetes.io doc (which I think was written in conjunction with Microsoft), we have https://docs.microsoft.com/en-us/virtualization/windowscontainers/manage-containers/manage-serviceaccounts that explains a little more.
Is there any additional feedback specific to this Azure AD DS doc that we can address? It looks like you've created a question on Stackoverflow for Windows and Kubernetes, which is probably the best place for additional technical support - https://stackoverflow.com/questions/57321957/enabling-gsma-in-azure-kubernetes-service. Otherwise, @SaurabhSharma-MSFT can see if there's additional support that can be provided.

@rexhinvorpsi  I've updated the doc to include steps on using the Add-Computer cmdlet to join a VM to the Azure AD DS managed domain. This doc update should publish in the next few days.
If you're still having technical issues, please let @SaurabhSharma-MSFT know.
For now, #please-close

Thank you for the great help!!


MM-18911: Filtering search by channel should also show the channel name and not only its ID

Summary

Web app filter search in: to display channel name (channel id) and display name like mobile
https://user-images.githubusercontent.com/7275313/68829461-d300ed80-065d-11ea-9642-2fbfdadbf6a2.PNG
Ticket Link

( https://mattermost.atlassian.net/browse/MM-18911 )
Screenshots

Creating a new SpinWick test server using Mattermost Cloud.

New commit detected. SpinWick will upgrade if the updated docker image is available.

The test server shows a different character, it should be the tilde as displayed here: ~
The tilde should be within the parentheses (~some-channel)

Other than that looks good, adding Katie as this touches her area

Moved the tilde inside the parentheses.  @esethna   do you think it looks like a '-' because of the browser?  it looks like a tilde on mine.

Mattermost test server updated with git commit e62c559008dcf0072adeacbb2c3be13eff076184.
Access here: https://mattermost-webapp-pr-4217.test.mattermost.cloud

@allenlai18 this prevents searching, it should display as you did, but search functionality shouldn't break because of this change.

@allenlai18 this prevents searching, it should display as you did, but search functionality shouldn't break because of this change.

@allenlai18 this prevents searching, it should display as you did, but search functionality shouldn't break because of this change.

hmmm. What part of searching is breaking? It seems to be working for me

I am seeing a tilde in my testing on Mac and it is correctly placed inside of the parentheses.

I have added another channel "Katie Test (~apple).  The channel name is not being identified by autocomplete  in a search starting with "t"

New commit detected. SpinWick will upgrade if the updated docker image is available.

I am seeing a tilde in my testing on Mac and it is correctly placed inside of the parentheses.

I have added another channel "Katie Test (~apple). The channel name is not being identified by autocomplete in a search starting with "t"

I think that is the expected behavior. katieTest is camel cased. if it was seperated with dashes it works.  what @Willyfrog was talking about is the search is broken when you search for posts within a channel.

@allenlai18 this prevents searching, it should display as you did, but search functionality shouldn't break because of this change.

Fixed! Let me know what you think.

Mattermost test server updated with git commit d778a8657e7baf880ba509e29eef1e7eb54e45cb.
Access here: https://mattermost-webapp-pr-4217.test.mattermost.cloud

New commit detected. SpinWick will upgrade if the updated docker image is available.

Mattermost test server updated with git commit 956e0a538134e37f7b56b02f033e0178812b2717.
Access here: https://mattermost-webapp-pr-4217.test.mattermost.cloud

Test server destroyed

@Willyfrog   can you merge when you get a chance?


Settings | Importing settings from file ignores special providers

There seems to be a problem with “Importing“ settings from a file. Is it is not overriding special providers (NWI, CBC, RSA) settings - just ignoring the changes for this.
Note that it works fine for Context Menu items. Context Menu items are getting overridden from imported files.
Expected Behavior
It should override everything - Context Menu items, NWI, CBC and RSA.

This is probably related to the forceOverride parameter from the ConfigFile.updateSpecialProvider function.


View Profile: Bigger

I had to explain another person where they can find their profiles. Then he said that its not very intuitive. Please make it bit more visible.

did they specify if they got to the sidebar or not?
i am thinking about two solutions:

either make the button inside the sidebar bit more visible (right now the whole area is visible)
make the profile section bigger with more info in the navbar - that would mean adding a name and wrapping it around the profile picture

they opened the bar, but they didn't notice it even though its there. I think because its in gray it makes people not notice it imidietly. Im okay with options. maybe put in bald or just change colour would help.


Unable to install plugin for 5.0.0

Issue Description
I am trying to install the kaae-latest.tar.gz for the 5.0.0 stable release but with error as follows

TIA!

@sps90 this is a known issue atm still working around trying to get the installer to work.
KaaE 5.x is barely started - If you want to test the current codebase just manually copy the plugin directory to your Kibana 5  /plugins and let us know your findings - thanks!


[Activation] Block to chainedheaderblock

fixes #1813
A phenomenal amount of unit tests are breaking but cannot be related to this change so must be breaking in core-way-activation.
Obviously don't merge until that is resolved

OnBlockConnected also should signal chained header block instead of Block

The issue only mentioned block disconnected, but happy to do block connected too if a priority?

The issue only mentioned block disconnected

The issue doesn't specify it. There is basically no description except for the issue name and issue name suggests that block signaling in general should be changed to signal CHB,

but happy to do block connected too if a priority?

Yes, please do.

Ok this is now done.  I have run the unit tests and compared against  core-way-activation that the same number are still failing (that aren't relevant to this change).

Looks good to me. Just a few minor changes

Thanks @noescape00 These changes are now done.


Add debian lib64 path to elfLibDir

Hi,
I've recently been trying to packaging booster for my debian/ubuntu based distro and we ran into an issue with the elfLibDir not having the path debian uses. This PR fixes that.

Thank you!


Lack of LDAP/M$ AD Authentication

If LDAP/M$ AD authentication deployed, Avored will be more useful for mass adoption.
Is it possible to integrate/make this feature?

@gnu4cn Yes definately. I know it much important it is for any company to have an LDAP login as a feature to easy role/permission access based on groups and etc so yes for sure will implement the feature.


Create testing member account

User Story
As a modernisation platform engineer
I need a testing account
With an account type of member
So that I can run terratest for modules in a member account that I know will be clear to avoid false failures
Value
When creating module we have decided to use Terratest to test the module.  Whilst we can use local stack for some basic modules, most modules will have functionality that we want to ensure works with our member account structure and functionality that cannot be simulated locally.  Therefore if we have a clean test account, where resources are only created and destroyed with Terratest, we can test the modules.
Questions / Assumptions
New account details
Account name: testing
Account type: member
Environment: test
Github team slug: modernisation-platform
Access: developer
Networking:
Business unit: platforms
Subnet sets: general
Definition of done

[ ] new testing account created
[ ] readme has been updated
[ ] user docs have been updated
[ ] another team member has reviewed
[ ] tests are green

Reference
How to write good user stories

Completed Steps 1 to 6 on the the Runbook for creating the account / environment some bugs found and issues have been raised for this


driver: dac: add nxp mcxn10 dac support

add nxp mcxn10 dac support, the local test has passed

Could you please rephrase the commit message, it should point to the board rather than driver: dac.
E.g.
board: mcxn10: add nxp dac support

Could you please rephrase the commit message, it should point to the board rather than driver: dac. E.g. board: mcxn10: add nxp dac support

Hi @marcusfolkesson, thanks for your comment here, could you please give your reason here? And I would like to share some examples of what I've seen. They all used samples: drivers: xxx: xxx format.

From the Zephyr doc description, the area domain is used to indicate where the change was changed, so, I think using samples: drivers: xxx: xxx format is better than board: mcxn10: add nxp dac support format, because other users can easily know this changed code located at area samples/drivers/dac, and from the summary of change domain, the user can know this change is used to support frdm_mcxn947 dac example.

Could anyone help to merge this PR? thanks in advance.

Could anyone help to merge this PR? thanks in advance.

have to wait at least 33 more hours to allow reviews
https://zephyrproject-rtos.github.io/zephyr-merge-list/

To resolve conflicts, forced pushed just now.

conflict again, updated and force pushed.

Hi @martinjaeger, help to review, thanks, NXP frdm_mcxn947 board related files are too prone to conflicts.

Can anyone help merge this PR? Thanks.


User Story: As a watchdog, I need to view entity status and performance over time so that I can analyze performance

As a watchdog, I would like to know how many companies that have graduated from the SBA 8a business development program still benefit through 8a extension contracts that circumvent the 8a statutory limits (like 8a STARS that effectively grants 8a's another five years of 8a status beyond the 9 years they get while in the 8a program).  This same logic extends to all of the other socioeconomic set asides related to growth, such as small business, HUBZone, etc.  How many small business continue to compete against small businesses through GWACs/IDIQs/BPAs even after they have grown above the NAICS small business size standards.
Whatever reporting is done must be able to show status at a point in time compared to today. We would want to analyze the impact of a socio-economic program to a business, watching what happens to that business before the certification, during the certification and then post-certification. What happens to the officers and the company? Do they continue to grow? Fizzle out? Get acquired? What are the attributes of a company that graduates a business development program and continues to succeed?
What would be the impact to small (or other socioeconomically classified business) if contracting officers required all businesses to recertified their size or socioeconomic standard for every task order?
How much money is going to large businesses from contracts set aside for small business?

Have added a story here: https://federal-spending-transparency.atlassian.net/browse/PUB-76


Exception happens after refresh book details

I got exception when I open book detail url at the startup, and I got this book save in my collection.
main.05513be….bundle.js:1240 EXCEPTION: Uncaught (in promise): TypeError: Cannot read property 'id' of undefined

Can be fixed (?) by adding filter to getSelectedBook  selector
export function getSelectedBook(state$: Observable<State>) {
  return combineLatest<{ [id: string]: Book }, string>(
    state$.let(getBookEntities),
    state$.let(getSelectedBookId)
  )
  .map(([ entities, selectedBookId ]) => entities[selectedBookId])
  .filter(i => !!i);
}

Thanks, that also solved the issue when I removed "useHash:" from the RouterModule.

This issue is no longer present in the current example


error Command failed with exit code 1 when opening whitelist

Hi, i am trying to find a solution to an error i faced while trying to open up whitelist. Please advise.

Hi @Goanft,
A bug report must follow the given template and you have to provide all the requested information, otherwise it's invalid.
If you simply wanna ask a question, you can do that in the Discussions Section.
Thank you.


suppress authentication errors on profile update

this is specifically a fix for the work to store a user's timezone and device selection preferences in their profile to ensure that users with permissions to upload on behalf of another user are not presented with errors concerning permissions to also modify that user's profile data.

@gniezen - can you review these changes? This is some pretty basic error suppression to move forward with storing the user's preferences in their patient profile while not re-working our permissions system.

 [uploader] Suppress error on timezone change for users who don't have permission

LGTM - I also double-checked that the tests fail when the status code is changed in the code.
Shouldn't you maybe add some comments/TODOs so that this can be removed at a later stage when the permissions system is updated?


Add Data Serving Mechanism via datasette

DO NOT MERGE THIS PR WITHOUT ADDING ME AS AN ADMIN
I need to add the proper secret so Heroku can authenticate.
@gregce @DJedamski this will refresh the below URL anytime the data is updated.
https://covid-repo-data.herokuapp.com/

The above page is generated with datasette, a tool for exploring and publishing data. It helps people take data of any shape or size and publish that as an interactive, explorable website and accompanying API.

This is refreshed automatically with GitHub Actions.
cc: @T-Holland @neovintage @inc0 @awmatheson @aronchick

Note this only loads the most recent file, I could load all of them as separate tables.  I can do that after this PR


CB-8554 Updated source to pass Fortify scan.

From Fortify:
Access Control: SecurityManager Bypass (Security Features, Structural)
Poor Error Handling: Empty Catch Block (Errors, Structural)
Poor Style: Non-final Public Static Field (Encapsulation, Structural)
Poor Error Handling: Throw Inside Finally (Errors, Structural)
And from SonarQube:
Preserve Stack Trace
Reliance on default encoding

Changes look good I think, but I've sadly landed some code before looking at this that causes it to not merge cleanly. Would you be able to rebase?

bump


Update circe-core, circe-generic, ... to 0.11.2

Updates

io.circe:circe-core
io.circe:circe-generic
io.circe:circe-jawn

from 0.11.1 to 0.11.2.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.circe" } ]

labels: semver-patch

Codecov Report

Merging #1180 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1180   +/-   ##
=======================================
  Coverage   80.52%   80.52%
=======================================
  Files          54       54
  Lines        1022     1022
  Branches       45       45
=======================================
  Hits          823      823
  Misses        199      199

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 cda79cd...305ee7a. Read the comment docs.


extend multiple characters similar to '/' to trigger different dropdown plugins

I want to extend multiple characters similar to '/' to trigger different dropdown plugins, but after copying a set of code, I'm getting an error saying 'Adding different instances of a keyed plugin (suggestion$)'. I also tried adding a key in addOptions, but I'm still having issues. Can you help explain how to extend multiple characters?

https://tiptap.dev/docs/editor/api/utilities/suggestion#find-suggestion-match I think you need to write your custom function to do this


Resizing a Glutin-based window makes it gray

Resizing a Glutin-based window makes it gray.

Fixed by eliminating the use of FFICallbacks and instead return a vector of polled events on the Rust side. On the Pharo side we filter a returned collection of events by removing unnecessary "duplicated" events that are not required to be processed within one frame, for example window resized, window refreshed, window moved and cursor moved events.
This way only a few events gets polled and resizing is instant 👍


docs/guide-ja/tutorial-docker.md added [ci skip]

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
--

Fixed issues
--

Merged. Thank you!


Automate data pull from acumatica

Please make sure this error get fixed before we decide to deploy in this report to stakeholder

import datetime instead of having from datetime import datetime


If telnet password is set the timeout due to inactivity is always set to 1 min

Looking at the code below the telnet connection will be closed after 1 min of inactivity even when MAX_TIME_INACTIVE is set to different value.
`
#ifdef MAX_TIME_INACTIVE
	// Inactivity - close connection if not received commands from user in telnet
	// For reduce overheads

	uint32_t maxTime = MAX_TIME_INACTIVE; // Normal

	if (_password != "") { // Request password - 18/08/08
		maxTime = 60000; // One minute to password
	}

	if ((millis() - _lastTimeCommand) > maxTime) {
		TelnetClient.println("* Closing session by inactivity");
		TelnetClient.stop();
		_connected = false;
		_silence = false;
	}

#endif`

Hi @achuchev
Please download lastest version with this fixed
Thanks for your feedback

@JoaoLopesF Yes, it works!


[kythe] branch lexed as a (AMS) keyword, not identifier

Input test case
class riscv_jump_instr extends riscv_directed_instr_stream;
    riscv_instr          branch;
endclass

Describe what is wrong or missing
CST for this code:
Node @3 (tag: kDataDeclaration) {
        Node @1 (tag: kInstantiationBase) {
          Node @0 (tag: kInstantiationType) {
            Node @0 (tag: kReferenceCallBase) {
              Node @0 (tag: kReference) {
                Node @0 (tag: kLocalRoot) {
                  Node @0 (tag: kUnqualifiedId) {
                    Leaf @0 (#SymbolIdentifier @3323-3334: "riscv_instr")
                  }
                }
              }
            }
          }
          Node @1 (tag: kVariableDeclarationAssignmentList) {
            Node @0 (tag: kVariableDeclarationAssignment) {
              Leaf @0 (#"branch" @3344-3350: "branch")
              Node @1 (tag: kUnpackedDimensions) {
              }
            }
          }
        }
        Leaf @2 (#';' @3350-3351: ";")
      }

Expected CST:
Node @3 (tag: kDataDeclaration) {
        Node @1 (tag: kInstantiationBase) {
          Node @0 (tag: kInstantiationType) {
            Node @0 (tag: kReferenceCallBase) {
              Node @0 (tag: kReference) {
                Node @0 (tag: kLocalRoot) {
                  Node @0 (tag: kUnqualifiedId) {
                    Leaf @0 (#SymbolIdentifier @3323-3334: "riscv_instr")
                  }
                }
              }
            }
          }
          Node @1 (tag: kVariableDeclarationAssignmentList) {
            Node @0 (tag: kVariableDeclarationAssignment) {
              Leaf @0 (#SymbolIdentifier @3344-3350: "branch")
              Node @1 (tag: kUnpackedDimensions) {
              }
            }
          }
        }
        Leaf @2 (#';' @3350-3351: ";")
      }

In CST branch here should be SymbolIdentifier and not branch

Not a kythe issue so much as it is a lexer issue:
https://cs.opensource.google/verible/verible/+/master:verilog/parser/verilog.y;drc=cb7d6286758ebbb446cb9ccb87d5bfff350e94cf;l=533
https://cs.opensource.google/verible/verible/+/master:verilog/parser/verilog.y;drc=cb7d6286758ebbb446cb9ccb87d5bfff350e94cf;l=793
branch is a Verilog-AMS keyword, and conservatively, the lexer and parser interpret it as such.  I don't have dialect selection implemented yet.

Dialect-selection would mean a configurable post-lexer pass that downgrades select keywords into SymbolIdentifier before passing it onto the parser.

Status: Currently, this is worked around in Verible's kythe extractor, the branch keyword (when used as an identifier) is ignored safely.

Is fixed since https://github.com/chipsalliance/verible/commit/ef935c9036984a4583df5510f813f4e66db0799f in which branch is qualified as SymbolIdentifier as it is never used in the grammar.


Cherrypick gandalf changes

This PR cherrypicks all the changes necessary to have the smooth access request flow on Gandalf.
@graceguo-supercat  @john-bodley @michellethomas

Codecov Report

Merging #72 into airbnb-master will decrease coverage by 0.08%.
The diff coverage is 55%.

@@                Coverage Diff                @@
##           airbnb-master      #72      +/-   ##
=================================================
- Coverage           65.7%   65.61%   -0.09%
=================================================
  Files                346      346
  Lines              21830    21847      +17
  Branches            2419     2421       +2
=================================================
- Hits               14343    14336       -7
- Misses              7476     7500      +24
  Partials              11       11

Impacted Files
Coverage Δ

...perset/assets/src/components/StackTraceMessage.jsx
58.33% <ø> (ø)
:arrow_up:

...uperset/assets/src/SqlLab/components/SouthPane.jsx
88.57% <ø> (ø)
:arrow_up:

...uperset/assets/src/SqlLab/components/ResultSet.jsx
84.84% <0%> (-0.87%)
:arrow_down:

superset/assets/src/SqlLab/reducers.js
57.98% <0%> (ø)
:arrow_up:

superset/views/core.py
73.67% <16.66%> (-1.16%)
:arrow_down:

superset/sql_lab.py
74.05% <50%> (-0.41%)
:arrow_down:

superset/views/base.py
66.89% <50%> (ø)
:arrow_up:

superset/security.py
72.9% <60%> (-1.31%)
:arrow_down:

superset/assets/src/chart/Chart.jsx
66.1% <66.66%> (ø)
:arrow_up:

superset/assets/src/SqlLab/actions.js
72.72% <88.88%> (+0.16%)
:arrow_up:

... and 1 more

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 132edc2...d18b0c8. Read the comment docs.


Opening autocomplete results in new browser tabs

Like many people, I'm a heavy user of middle click or ctrl+click to open links in new browser tabs.
Very sadly, this doesn't work for Algolia autocomplete suggestions… (demo search boxes: https://community.algolia.com/instantsearch.js/ – https://laravel.com/)
Maybe related: #323, #481

Hi @vlakoff, can you provide us a list of steps to reproduce the behavior you are experiencing? Thanks

Go to one of the websites mentioned above.
In the search box at top of the page, type some characters. Suggestions should be displayed below the box.
Middle-click on a suggestion: nothing happens.
Ctrl+click on a suggestion: opens in the same browser tab.

Expected: both middle-click and ctrl+click should open links in new browser tabs.
Tested on Firefox and Chrome.

@vlakoff Thanks for reporting this! Could you open an issue on DocSearch meanwhile? This seems to be a DocSearch issue: https://github.com/algolia/docsearch


Example on Docu page gives different result

On the wiki page on decode a few examples are listed.
When I run the example SELECT decode(NULL, 6, 'Spark', NULL, 'SQL', 4, 'rocks') in my own environment it gives a different result.
See Attachments

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

ID: 2a77b343-4c33-13eb-7758-7a5414709324
Version Independent ID: a29c2bc2-56a8-e28a-f945-815ca8d4f32f
Content: decode function - Azure Databricks
Content Source: databricks/spark/latest/spark-sql/language-manual/functions/decode.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@mssaperla
Could you please check and add your comments on this, update as appropriate.

@joeldewaternn
Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.

Thanks for your dedication to our documentation. Unfortunately, we have been unable to review your issue and apologize for the delayed response. The requested updates have not yet been made. The timeline for resolution varies based on resourcing. We've created an internal work item (DOC-7194) to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close

#please-close


Improves item progress tab

Fixes #410

Also, you should rebase as #426 impacted (slightly) a component that you modified here.
In order to remember until where I reviewed, it was commits "Adds routing supporting for tabs on item progress page" and "Generates new translates".


modifications to backref made inside _wrap_with_default_query_class(fn, cls) get lost

def _wrap_with_default_query_class(fn, cls):
    @functools.wraps(fn)
    def newfn(*args, **kwargs):
        _set_default_query_class(kwargs, cls)
        if "backref" in kwargs:
            backref = kwargs['backref']
            if isinstance(backref, string_types):
                backref = (backref, {})
            _set_default_query_class(backref[1], cls)
        return fn(*args, **kwargs)
return newfn

In the code above backref is extracted from kwargs, modified if it's a string type, but the modifications are never reintroduced to kwargs.

fixed in #1087


fix message row in admin dashboard

http://localhost:3000/admin opens an admin dashboard on the chat section there needs a UI fix

Hi @alokVerma749
I can fix this, can you assign the issue to me please?

sure

Hi @alokVerma749
Your sign up flow doesn't work, can you provide a test account for me to check the error? Thanks

You can create an account on the signup page then you will be able to log in to the admin dashboard.

make sure to put your mongodburl in next.config.js before you signup

To reproduce the bug you can make some messages on the contact form

make sure to put long email address and message, so that you can produce that overflowing UI

Hey @chasingtherain are you working on the issue

Hey @chasingtherain, is there any progress on this issue.
I just wanted to see if you needed any help or if there was anything that I could do to support you.
I appreciate all of the work you've done on the project so far and I would be grateful for any updates you could provide.

Hi @alokVerma749
I will pass on this issue, thanks


DNS stuff for APPRunner

Currently it can't run on root domain.

https://github.com/hashicorp/terraform-provider-aws/issues/14447 is relevant to fixing double tf apply.


fix RPi compile error problem (and linux)

Exclude files which is not used for Ableton Link library itself.
Some of (#) comment out does not work because of indent.

see also https://github.com/comoc/ofxAbletonLink

@hiroMTB
thanks!


Improve logging of REST request adding MDC

References

Fixes #3244

Description
A custom filter has been introduced to add in the thread local a correlation id provided by the client (if any) and an unique request id to simplify the trouble-shooting of the request. This technique is common and well described in the literature https://logging.apache.org/log4j/2.x/manual/thread-context.html
the X-CORRELATION-ID seems to  be often used for this purpose.
An additional custom header X-REFERRER is used instead than the standard (mispelled) REFERER header. The choice of a custom header is due to the fact that many browsers are starting to apply very restrictive policy about the referer header, see https://developers.google.com/web/updates/2020/07/referrer-policy-new-chrome-default
https://web.dev/referrer-best-practices/
This mean that, to get the best from this logging mechanism, the Angular UI should be improved adding to all the requests these two new custom header

X-CORRELATION-ID with a random generated uuid or any hard to conflict string generated at the application load and keep stable for all the browsing session
X-REFERRER to note the page that has generated the REST request

Current limitation
As the solution is based on a filter that is set after the security filter any log produced by the security filter such as invalid csrf token, bad credentials, etc are not enriched with the extra details.
REST Contract
Once we are in agreement about the solution, specifically the name of the Header I will create a PR also for the REST Contract to note the purpose of this two custom headers
Instructions for Reviewers
Checkout the PR and look to the log file making some activity on the angular UI, the HAL browser or postman.
Checklist

[X] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[X] My PR passes Checkstyle validation based on the Code Style Guide.
[X] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[ ] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[n/a] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

@abollini : As #3301 was just merged, this PR will need to be rebased on main in order to run the newly updated CI build process.

@tdonohue done

hi @tdonohue thanks for looking to that.
The first two points are related to the current limitation, as noted in the description the login process happen outside the new filter. We can for sure investigate and try to hack a bit the security filter to make sure that the logging filter is invoked or the mdc is properly initialized. As we said that we want to focus on a good enough approch instead than try to get the perfect solution I have preferred to keep this PR simpler.
Point 3 we can for sure merge the before and referrer log together, personally I prefer to have on separate line but no problem to change. We can also easily mute the after but I found it useful as it states that the request has been fully processed and guarantee that we have no stale/zombie request pending.
The request id is the only id that is guaranteed to be present as it is generated by the server and allow us to track a single request. Try to perform an item creation or a community delete to see more logs. Of course if you are the only one browsing the app it doesn't seem very useful but it is a real service with simultanious users.
The request and correlation id are both required and useful when available ad angular itself is the main source of concurrency as the browser process many request in parallel.

@abollini : thanks for the quick response!
I had missed the "current limitation" note above about how this doesn't quite work for login/logout, invalid CSRF, etc.  That's completely reasonable as this just should be a "quick fix" as you noted.  One option we could consider is to change the log4j2.xml configuration slightly to:
$${ctx:correlationID:-unknown} $${ctx:requestID:-unknown}

This would result in lines without a correlationID or requestID to automatically have unknown unknown in the log line.  I admit though that this results in a large number of log lines with unknown unknown when DSpace first boots up.  But, it keeps the log format more consistent.  That said, if you prefer the current format, I don't feel strongly in requiring this change...just presenting it as an option.
In terms of the number of lines per request, I still prefer myself a shorter syntax (as noted in 3 above).  However, we could make this easily configurable via something like logging.server.include-after-request.  When set to true, you see that After request line in the logs.  When set to false, you just see a single line which says something like Request [GET /server/api/authn/status] originating from http://localhost:8080/server/.    I'd recommend false by default, but if others disagree, we could even default to true.
Regarding 4, I see your point now after playing with this more.  The correlationID and requestID do serve different purposes.  I'm OK with leaving both in for now...we can always improve more later.
(As a sidenote, in last week's meeting, @artlowel noted we may want to look closely at whether the correlationID aligns with GDPR / privacy restrictions.  If it is easy to map it back to a specific user/login (which it likely is as soon as you login), then it may not be an anonymous identifier.  This doesn't mean we need to change this PR, but it does mean we may need to determine whether we need a way to still "anonymize" these logs via a command-line script or similar... I'd leave that out of scope for 7.0 though, but something we may need to think about for 7.1.)
All in all, the direction of this PR looks good, as does the code.  The only reason I haven't given it a thumbs up, is that I'd like to ensure it is tested with the corresponding Angular PR (and that other PR is not working at this time).

After the suggestions I added a commit for:

change the log layout to automatically have unknown unknown
add the two headers to the allowed headers in the CORS configuration
merge the two before request log lines into one

Hi @tdonohue, thanks for pointing out the missing escape. I fixed the bug and also added the logging.server.include-after-request property to enable/disable log after request.

Merging as-is, since @artlowel approved the Angular side of things... This was just waiting on a general approval of the direction by someone else.
Thanks again @LucaGiamminonni and @abollini


OKD v4.4 on bare metal installation stuck in the fcos setup

we are trying install okd 4.4 cluster on Digital Ocean.
Following the https://docs.okd.io/latest/installing/installing_bare_metal/installing-bare-metal.html documentation
But we stuck into the step of bootstrap of fcos step. It seems that we setup the https server to download the raw image and verify the connectivity. The connection works fine. But after we provide the kernel edit steps mentioned in the docs and we noticed it was not downloading the fcos raw image. any idea on this ?
Note:- we are using latest fcos version .

• received below connection refused ERROR
[root@linux home]# openshift-install --dir=installation_directory/ wait-for bootstrap-complete --log-level=info
INFO Waiting up to 20m0s for the Kubernetes API at https://api.okd4.mydomain:6443...
ERROR Attempted to gather ClusterOperator status after wait failure: listing ClusterOperator objects: Get https://api.okd4.mydomain:6443/apis/config.openshift.io/v1/clusteroperators: dial tcp <IP_ADDRESS>:6443: connect: connection refused

We are trying this on digital ocean VMs. If anyone has prior experience add your feedback

See https://docs.okd.io/latest/installing/installing-troubleshooting.html

Hi @vrutkovs Thanks . I followed the troubleshooting guidance .I noticed below error messages .Any idea on this ?

output:-
[core@okd ~]$ ./openshift-install gather bootstrap --dir=installation_directory/ --bootstrap xxx.xxx.xxx.xx  --master xxx.xxx.xxx.xxx
INFO Pulling debug logs from the bootstrap machine
FATAL failed to run remote command: Process exited with status 127

output:-
No result showing
core@bootstrap ~]$ journalctl -b -f -u bootkube.service
-- Logs begin at Wed 2020-07-15 05:43:23 UTC. --

output:-

output:-

Please find the attach .openshift_install_state.json file .
openshift_install_state.zip

log bundle doesn't seem to be generated .we are getting below error while executing this command
[core@okd ~]$ ./openshift-install gather bootstrap --dir=installation_directory/ --bootstrap xxx.xxx.xxx.xxx --master xxx.xxx.xxx.xxx
INFO Pulling debug logs from the bootstrap machine
FATAL failed to run remote command: Process exited with status 127

Interesting, try https://docs.okd.io/latest/installing/installing-troubleshooting.html#installation-manually-gathering-logs-with-SSH_installing-troubleshooting?
My bad, I mistakenly requested .openshift_install.json while we needed .openshift_install.log - https://docs.okd.io/latest/installing/installing-troubleshooting.html#installing-getting-debug-information_installing-troubleshooting

Hi @vrutkovs . find the attach .openshift_install.log
openshift_install.log

I tried https://docs.okd.io/latest/installing/installing-troubleshooting.html#installation-manually-gathering-logs-with-SSH_installing-troubleshooting this .But not logs result displaying .
[core@bootstrap ~]$  journalctl -b -f -u bootkube.service
-- Logs begin at Wed 2020-07-15 05:43:23 UTC. --

You're using OpenShift Installer 4.4.0-0.okd-2020-04-21-163702-beta4, which is 3 month old. There's been multiple fixes releases since that
Please remove the install directory (including .openshift_install* files), get the installer from GA build and start from scratch.

Thanks @vrutkovs  . I'll check it

Hi @vrothberg  I tried with GA build but still getting below terraform error. I have attached .openshift_install.log file .

openshift_install.log

I think @vrutkovs can help :)

Could you also attach your install-config.yaml?

find the attach install-config.yaml file .
install-config.zip

Right. You shouldn't be using openshift-install create cluster, use create ignition-configs.
See https://docs.okd.io/latest/installing/installing_bare_metal/installing-bare-metal.html#installation-user-infra-generate-k8s-manifest-ignition_installing-bare-metal

I followed create ignition-configs way .

Right. You shouldn't be using openshift-install create cluster, use create ignition-configs.
See https://docs.okd.io/latest/installing/installing_bare_metal/installing-bare-metal.html#installation-user-infra-generate-k8s-manifest-ignition_installing-bare-metal

Looks like even after following that step, they are hititng the same error.

Issues are resolved .Followed the @dustymabe okd installation steps and installed the okd cluster.   https://dustymabe.com/2020/08/13/openshift-okd-on-fedora-coreos-on-digitalocean-part-1-deployment/  . Thanks @dustymabe for the support given. and thanks @vrutkovs for checking the issue.


can lager support user defined level name [JIRA: RIAK-1895]

Hi, @jaredmorrow
In my situation, I need to log some thing into a file, so such file can be analysed later. But I don't want to use the level defined by lager, I want to use a customized level name(etc. lager:my_log("some useful informaion")).  Like log4j in java, it allowed user to defined a "log name" for each Log object, so the "log name" can be used in application code to found the Log object.
I have read the source code, and found lager only support predefined level. Will lager support user-defined level or something like this in the future?

Hi we could look into supporting something like this in the future, but there'd have to be a way to map the "custom" log level to a "real" level.

In the 3.x branch we have multiple sink support which implements something similar to this without needing to define custom log levels. Instead you can define sink names which can output information to a backend using whatever controls you want to associate with them.
So I'm going to close this issue.  Please reopen if it doesn't address what use case you have in mind. Thanks.


Scanner window on initial display incorrect size

I have the scanner component inside the dialog box.  The dialog box has the wrong dimensions when there is no camera image in it and cuts off the scanner component.  When there is an image everything works fine.  This is the case with any React component such as Box, Card, etc

Were you able to solve this?

I fixed it with these styles:
            container: {
              width: "100%",
              height: "0", // hack to make the video not shift the layout
              paddingBottom: "100%", // hack to make the video not shift the layout
            },
            video: { width: "100%", height: "auto" }, // hack to make the video not shift the layout
          }}```