In the Editor, Initialize Textures With a Reasonable Size

Summary
When creating a new Texture resource "from scratch" in the editor (without source Pixmap), it should be initialized with a reasonable default size, like 128x128.
Analysis

Initialize empty textures with a default size of 128x128.
Otherwise, it's quite easy to trip into errors thrown by other parts of the core, complaining about the zero-size texture.
The setup shouldn't be done in the default constructor, but in an EditorAction with a setup context, like this one. Instantiating new textures via source code should not imply allocating some arbitrary default space.
Also, consider raising the editor hint min value for the texture size property to one.

Done by @Barsonax


Phase 1: PDS-EN to provide cloud endpoint for SBN to integrate the endpoint into SBN Survey Image Service.

The SBN Survey Image Service currently uses the SBN-PSI S3 bucket to display images. PDS-EN will provide a cloud endpoint containing CSS data, enabling SBN to start using PDS-EN's CSS data in the cloud.

Provided 2 ways to access css cloud - both got access denied.  Need review with both dev and SA team to get permission reverted and set correctly again.

(1) https://pds-uat.mcp.nasa.gov/data/open-data-registry/   (AccessDenied Error)
(2) aws s3 ls --no-sign-request s3://pds-css-archive/           (AccessDenied Error)

S3 is now publicly accessible

Nevermind. This is not working again...


Adds input styles

Adds styles for checkboxes, selects, text and textarea inputs.
Along with other general review, a couple of things came up when pulling this together:

In DIMES, we've used a custom HTML element which is styled to look like a select, but is really a button with a ul. Do we want to use the HTML select element, or do we want to go with a more custom approach across the board? On library.rockarch.org we opted for the select element. The implication here is that it's not possible to style the select options.
These questions are also noted in the change tracking document, but there are some fundamental decisions we need to make about default HTML structure and display of inputs and labels:

Should inputs be wrapped in a div? This would allow us to treat a label and an input as a unit, but it does add another HTML element to the page.
Should inputs take up the full width of their parent element by default, and should input labels stack on top of inputs by default? This is related to the above question.

fixes #58
fixes #65
fixes #69

custom select: I was just reading this article on native v. custom select inputs by Sarah Higley, which is useful context and I think this may be an important guiding thought:

While pure aesthetic preference is a common reason to roll a custom select component, it is not a good reason. Accessibility and usability should drive aesthetic choices, not the reverse. So many accessibility tradeoffs are involved in a custom solution that there must be a correspondingly strong reason to need one; “it should look prettier” just ain’t it.

I would vote yes, let's standardize labels stacking on top of inputs. I'm not sure about taking up the full width of the parent element by default, though. The selects and checkboxes don't generally do this in our implementation, right?

One other thing:

I'm noticing it's difficult to quickly distinguish which checkbox goes with which label in the grouped checkboxes. can we add some space between the label and the checkbox to its right that is not associated?

Hold on I still need to do the thing with labels.

OK, so in 001f82b I have added a width of 100% to selects, inputs, and textareas. This means that the labels will sit on top of those inputs, BUT it also means that the inputs will expand to fill the full width of their parent element. I think this will likely require some HTML changes in existing sites, and may not be as flexible as we'd like (for example if we want inline forms), but a more flexible approach would (I think) require a wrapping div.

Not sure if other folks think this is an issue, but the text seems to be slightly lower than the checkboxes in the grouped checkboxes:

Grrr, the joys and perils of flexbox.
I'm struggling a little bit with this checkbox group because, as far as I know, we don't actually have this implemented anywhere, so it feels weird to make this "look right" in isolation of knowing what the actual application is.

Yeah. Actually, you prompted me to wonder if we should be using fieldset groupings for the dimes filters:

OK, so what I've done here is add a wrapper to checkboxes, which improves alignment of the label and the checkbox across multiple layouts. As per @HaSistrunk's suggestion, I've also styled the fieldset element so it more closely matches the facets in DIMES.

Awesome.
And regarding the width of 100% on selects, inputs, and textareas, I'm good with that although I admit I don't have a good sense yet of how much we'll have to change existing sites when we implement, and if the lack of flexibility is going to be a barrier.


Add workflow to publish snapshots via GHA

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
Add workflow to publish snapshots via GHA
Issues Resolved
Closes https://github.com/opensearch-project/spring-data-opensearch/issues/9
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

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.

https://aws.oss.sonatype.org/content/repositories/snapshots/org/opensearch/client/ :partying_face:


I would like support ".geojson" file extensions

Please describe your request. Is your feature request related to a problem?
Only ".json" file extensions are supported for GeoJSON plot files. The older extension is to use .geojson and that should be supported as well
Describe the solution you'd like
Support GeoJSON plot files with the .geojson extension
Describe alternatives you've considered
Renaming existing files to change the extension. This may not be an available option for users
Additional context
N/A

PR: https://github.com/AgPipeline/drone-makeflow/pull/14


chore: Add Canonical DID identifier for sidetree-mock

@sandrask I don't understand this comment for the edge-sandbox repo.

@sandrask I don't understand this comment for the edge-sandbox repo.

@troyronda @fqutishat Never mind my comment - I thought  this was in sidetree-mock project


dockerCall isn't going to work with subprocess.PIPE

There's no return here:
https://github.com/BD2KGenomics/toil/blob/bfb63c9c85ae311945d61be2e1fa6e7717f05522/src/toil/lib/docker.py#L55
So there's no way to get the subprocess process out of dockerCall, so even if you tell it to send output to subprocess.PIPE you won;t be able to get ahold of the pipe to read from it.

I think you're meant to use dockerCheckOutput in that situation rather than dockerCall. That returns the stdout as a string.
(But I don't know how you'd get stderr.)

I'm really looking for a pipe and not a string, because I'm planning to push many gigabytes of output through it.
I'd also need @cmarkello and @glennhickey 's wrappers that I'm using to allow you to start up a Docker call and not wait for it to run to completion, so maybe for now I will write to disk.


Start position 1 & one slide show 2 dots

This comes from #10 comment.
If the Carousel#withStartPosition-method is used (only tested it with a parameter = 1) with a carousel which holds only a single slide, there will be two dots displayed in the navigation instead of one dot.

I wasn't able to reproduce this behavior. I did a simple test based on our current master:
    Slide s1 = new Slide(createSlideContent("Slide 1", "green"));
    Carousel c = new Carousel(s1);
    c.setWidth("100%");
    c.setHeight("180px");
    c.setStartPosition(1);
    add(c);

and it shows the only slide that was added and only one dot, like shown in the picture:

In fact, it doesn't matter if you chose a greater number to set in setStartPosition, it just displays one dot as there's only one slide.

Hey,
I produced this issue in another setting:
member variable:
    Carousel c = new Carousel().withSlideDuration(4).withStartPosition(1);

constructor() {
    add(c);
}

addSlide(Sting title) {
    Slide s1 = new Slide(createSlideContent(title));
    c.setSlides(new Slide[]{s1};
}

The second dot occures at start and nothing is displayed:

If you use the slide-navigation the second dot disappears:

Hi @CoderGarm, sorry for the delay on this. Is this still an issue for you? In your latest example, no slides are being added to the carousel. I tried again as in https://github.com/FlowingCode/CarouselAddon/issues/19#issuecomment-885228049 but not able to reproduce the double dot display.

Hey,
it's not an issue. I've switched my application from the vaadin frontend to angular completely a few month ago.


Upgrade WalletConnect with latest 0.4.0 version

https://github.com/WalletConnect/WalletConnectSwiftV2

@colourfreak Can we make an icon of AlphaWallet and upload it somewhere kind of iconassets. We  need an icon, for Wallet Connect session. For now we are using the next url: https://avatars.githubusercontent.com/u/36216689?s=88&v=4. The size, actually i don't know maybe some kind of 100*100

@oa-s maybe in alphawallet.com. Probably better to do it with aw.app, but seems less reliable there

No, I mean if you get one, let’s host it on AlphaWallet.com

Oh, i see. That not a problem.
url: https://avatars.githubusercontent.com/u/36216689?s=88&v=4. I only shown where i got current icon


Xcode should be listed as a  MacOS prereq on install pages

On the installation page https://appsody.dev/docs/getting-started/installation The MacOS section starts that the preqs must be installed, and only lists Docker however proceeds to describe an error if Xcode cli has not been installed.  Surely that should be listed as a prerequisite too if required to install ?

Docs updated


Update duplicating-a-repository.md

Why:
Closes #16018

What's being changed:
When I pull a --bare clone of a repository, the name of the folder that gets created is old-repository.git whereas the name specified in the docs is just old-repository.
Updated the same.

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@deepansh96
Thanks so much for opening a PR! I'll get this triaged for review :zap:


Fix exception causes in vq.py

I recently went over Matplotlib, Pandas and NumPy, fixing a small mistake in the way that Python 3's exception chaining is used. If you're interested, I can do it here too. I've done it on just one file right now.
The mistake is this: In some parts of the code, an exception is being caught and replaced with a more user-friendly error. In these cases the syntax raise new_error from old_error needs to be used.
Python 3's exception chaining means it shows not only the traceback of the current exception, but that of the original exception (and possibly more.) This is regardless of raise from. The usage of raise from tells Python to put a more accurate message between the tracebacks. Instead of this:
During handling of the above exception, another exception occurred:

You'll get this:
The above exception was the direct cause of the following exception:

The first is inaccurate, because it signifies a bug in the exception-handling code itself, which is a separate situation than wrapping an exception.
Let me know what you think!

Looks like the test failure here is not related to my change, since I see it happening in other unrelated PRs.


Can't retrieve select() after insert()

Describe the bug
As per documentation (of the other official APIs), we should be calling .select() after an insert() to retrieve the inserted row's id.  Unfortunately, .select() does not exist on SyncQueryRequestBuilder, and simply calling insert does not return the inserted row.
To Reproduce
supabase.table("Test").insert(vars(test)).select('id').execute()
AttributeError: 'SyncQueryRequestBuilder' object has no attribute 'select'
Expected behavior
The id is returned.
Desktop:

Python 3.9
Supabase-py 1.0.2

Data is returned anyway from execute. Data is based on the returning param of insert, but defaults to full representation.
resp = supabase.table("table").insert(data, upsert=True).execute()
obj = resp.data[0] # full json object of row

You can change returning to "minimal," though I haven't tested it so not sure if it gives you only a primary key or not.

Tried setting returning to "minimal" and it just returned an empty list, even though the rows get inserted.

The documentation has since been updated. You don't call .select() after .insert() for the Python library. We will probably change to follow the JavaScript library in the next major version release.


Remember cookie

This gives the option to not ask for a code every time on a device but to trust it for a certain amount of time.

Coverage remained the same at 100.0% when pulling c058d7c2894efc79116f3616d0a32d11136fffba on jorisvaesen:master into 4bbfb6ec3f7b741cb88a7fec9a635135c31cbfbc on andrej-griniuk:master.

Coverage remained the same at 100.0% when pulling a75b55f6a9beb383ba961d0d479e1cedd120e54b on jorisvaesen:master into 4bbfb6ec3f7b741cb88a7fec9a635135c31cbfbc on andrej-griniuk:master.

Coverage remained the same at 100.0% when pulling ffa73f39556be1cc3f4d64b32081a9d4c849745b on jorisvaesen:master into 4bbfb6ec3f7b741cb88a7fec9a635135c31cbfbc on andrej-griniuk:master.

Coverage remained the same at 100.0% when pulling 527c7aa66522c8bf414b509919696c7ae53d8dfb on jorisvaesen:master into 4bbfb6ec3f7b741cb88a7fec9a635135c31cbfbc on andrej-griniuk:master.

Thank you for the contribution @jorisvaesen! Well done, really useful!

Hi @jorisvaesen,
I made few changes to make this feature configurable and disabled by default, not to surprise existing users, and tagged a new version.


fix: fix deno usage of protobuf and jintr

Replace these lines at build time for deno:
import { Jinter } from 'jintr';

to:
import { Jinter } from<EMAIL_ADDRESS>
and
import { BinaryReader, BinaryWriter } from "@bufbuild/protobuf/wire";

to:
import { BinaryReader, BinaryWriter } from "https://esm.sh/<EMAIL_ADDRESS>
Everytime one of these two lib is updated in the package.json, these should also be updated in the "build:deno" command: https://github.com/LuanRT/YouTube.js/blob/5ff30e12ded29afff5a758cbe96ac3418e6d20aa/package.json#L84

Also fixes this warning (Deno V1) and now an error in Deno V2:
error: Import assertions are deprecated. Use `with` keyword, instead of 'assert' keyword.

import package_json from '../../package.json' assert { type: 'json' };

Closes #767
Closes #766

deno compile is failing though :/. will see what I can do about it


Running test in Windows docker or inside locked system

Is there any solution currently available for running test inside the windows docker with Winappdriver? Or if test can be ran when the system is locked?

Someone pls guide me?

Hi @SanjeevKumarmn,
Can you clarify the scenario you are testing?

I have a Windows app that we run the tests in windows 10 machine and if the tests are run from Jenkins, I need to turn off the screensaver. Also we need to keep the system unlocked for the test that are scheduled overnight for unmanned. For security reason, it's not safe to keep the system unlocked and also its a big process to get this security turn off locks in certain companies. So how we run the test with system locked in more secured way? Can we achieve something similar to selenium docker(refer hub.docker.com) for running Windows app inside the docker / headless mode? This would let us effectively use the unutilized CPU capacity for parallel windows app testing and completeing the tests much faster.


ASAN heap-use-after-free in QUICStreamFrame::store

==2077==ERROR: AddressSanitizer: heap-use-after-free on address 0x62500002b1f5 at pc 0x00000055c058 bp 0x000004212df0 sp 0x0000042125a0
READ of size 1212 at 0x62500002b1f5 thread T2
    #0 0x55c057 in __asan_memcpy (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x51057)
    #1 0x10090f6a4 in QUICStreamFrame::store(unsigned char*, unsigned long*, bool) const QUICFrame.cc:156
    #2 0x10090ec99 in QUICStreamFrame::store(unsigned char*, unsigned long*) const QUICFrame.cc:103
    #3 0x1008ce31b in QUICNetVConnection::_store_frame(std::__1::unique_ptr<unsigned char, void (*)(void*)>&, unsigned long&, bool&, QUICPacketType&, std::__1::unique_ptr<QUICFrame, void (*)(QUICFrame*)>) QUICNetVConnection.cc:801
    #4 0x1008c98a2 in QUICNetVConnection::_packetize_frames() QUICNetVConnection.cc:838
    #5 0x1008b954c in QUICNetVConnection::_state_common_send_packet() QUICNetVConnection.cc:738
    #6 0x1008bcc57 in QUICNetVConnection::state_connection_established(int, Event*) QUICNetVConnection.cc:475
    #7 0x10005691a in Continuation::handleEvent(int, void*) I_Continuation.h:153
    #8 0x100a274b9 in EThread::process_event(Event*, int) UnixEThread.cc:122
    #9 0x100a2856d in EThread::execute() UnixEThread.cc:188
    #10 0x100a2537e in spawn_thread_internal(void*) Thread.cc:91
    #11 0x7fff7df9c6c0 in _pthread_body (libsystem_pthread.dylib:x86_64+0x36c0)
    #12 0x7fff7df9c56c in _pthread_start (libsystem_pthread.dylib:x86_64+0x356c)
    #13 0x7fff7df9bc5c in thread_start (libsystem_pthread.dylib:x86_64+0x2c5c)

0x62500002b1f5 is located 501 bytes inside of 4096-byte region [0x62500002b000,0x62500002c000)
freed by thread T3 here:
    #0 0x5643e6 in wrap_free (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x593e6)
    #1 0x8067d in ats_memalign_free ink_memory.cc:144
    #2 0x86ab5 in malloc_free(_InkFreeList*, void*) ink_queue.cc:323
    #3 0x823fe in ink_freelist_free ink_queue.cc:277
    #4 0x100008b4d in Allocator::free_void(void*) Allocator.h:70
    #5 0x1000021df in IOBufferData::dealloc() P_IOBuffer.h:318
    #6 0x100001c78 in IOBufferData::free() P_IOBuffer.h:331
    #7 0x100008fa8 in Ptr<IOBufferData>::operator=(IOBufferData*) Ptr.h:236
    #8 0x100008c36 in IOBufferBlock::clear() P_IOBuffer.h:419
    #9 0x100002aa4 in IOBufferBlock::dealloc() P_IOBuffer.h:463
    #10 0x100002878 in IOBufferBlock::free() P_IOBuffer.h:469
    #11 0x100016b48 in Ptr<IOBufferBlock>::operator=(IOBufferBlock*) Ptr.h:236
    #12 0x100060575 in Ptr<IOBufferBlock>::operator=(Ptr<IOBufferBlock> const&) Ptr.h:257
    #13 0x10005a9bb in IOBufferReader::consume(long long) P_IOBuffer.h:678
    #14 0x10097275a in QUICStream::_send() QUICStream.cc:383
    #15 0x10096eac3 in QUICStream::main_event_handler(int, void*) QUICStream.cc:100
    #16 0x10005691a in Continuation::handleEvent(int, void*) I_Continuation.h:153
    #17 0x100a274b9 in EThread::process_event(Event*, int) UnixEThread.cc:122
    #18 0x100a2856d in EThread::execute() UnixEThread.cc:188
    #19 0x100a2537e in spawn_thread_internal(void*) Thread.cc:91
    #20 0x7fff7df9c6c0 in _pthread_body (libsystem_pthread.dylib:x86_64+0x36c0)
    #21 0x7fff7df9c56c in _pthread_start (libsystem_pthread.dylib:x86_64+0x356c)
    #22 0x7fff7df9bc5c in thread_start (libsystem_pthread.dylib:x86_64+0x2c5c)

previously allocated by thread T3 here:
    #0 0x564bb0 in wrap_posix_memalign (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x59bb0)
    #1 0x80424 in ats_memalign ink_memory.cc:103
    #2 0x86726 in malloc_new(_InkFreeList*) ink_queue.cc:261
    #3 0x82206 in ink_freelist_new ink_queue.cc:184
    #4 0x100061b25 in Allocator::alloc_void() Allocator.h:63
    #5 0x1000618ad in IOBufferData::alloc(long long, AllocType) P_IOBuffer.h:293
    #6 0x1000612c5 in new_IOBufferData_internal(char const*, long long, AllocType) P_IOBuffer.h:264
    #7 0x100060e7f in IOBufferBlock::alloc(long long) P_IOBuffer.h:409
    #8 0x1000d8bba in MIOBuffer::append_block(long long) P_IOBuffer.h:958
    #9 0x1002916da in HttpSM::setup_server_transfer() HttpSM.cc:6492
    #10 0x10028b6b8 in HttpSM::handle_api_return() HttpSM.cc:1617
    #11 0x1002e27f7 in HttpSM::set_next_state() HttpSM.cc:7369
    #12 0x100272e26 in HttpSM::call_transact_and_set_next_state(void (*)(HttpTransact::State*)) HttpSM.cc:7122
    #13 0x10028a0b3 in HttpSM::handle_api_return() HttpSM.cc:1553
    #14 0x10026959c in HttpSM::do_api_callout() HttpSM.cc:355
    #15 0x10029d18f in HttpSM::state_read_server_response_header(int, void*) HttpSM.cc:1921
    #16 0x1002683d6 in HttpSM::main_handler(int, void*) HttpSM.cc:2535
    #17 0x10005691a in Continuation::handleEvent(int, void*) I_Continuation.h:153
    #18 0x1008794cc in read_signal_and_update(int, UnixNetVConnection*) UnixNetVConnection.cc:113
    #19 0x10087926a in UnixNetVConnection::readSignalAndUpdate(int) UnixNetVConnection.cc:1068
    #20 0x100803841 in SSLNetVConnection::net_read_io(NetHandler*, EThread*) SSLNetVConnection.cc:597
    #21 0x10084a8a1 in NetHandler::process_ready_list() UnixNet.cc:392
    #22 0x100848ef9 in NetHandler::mainNetEvent(int, Event*) UnixNet.cc:512
    #23 0x10005691a in Continuation::handleEvent(int, void*) I_Continuation.h:153
    #24 0x100a274b9 in EThread::process_event(Event*, int) UnixEThread.cc:122
    #25 0x100a2905f in EThread::execute() UnixEThread.cc:266
    #26 0x100a2537e in spawn_thread_internal(void*) Thread.cc:91
    #27 0x7fff7df9c6c0 in _pthread_body (libsystem_pthread.dylib:x86_64+0x36c0)
    #28 0x7fff7df9c56c in _pthread_start (libsystem_pthread.dylib:x86_64+0x356c)
    #29 0x7fff7df9bc5c in thread_start (libsystem_pthread.dylib:x86_64+0x2c5c)

Thread T2 created by T0 here:
    #0 0x55b9f6 in wrap_pthread_create (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x509f6)
    #1 0x100a24fc9 in ink_thread_create(void* (*)(void*), void*, int, unsigned long, void*) ink_thread.h:152
    #2 0x100a24bb9 in Thread::start(char const*, void*, unsigned long, std::__1::function<void ()> const&) Thread.cc:111
    #3 0x100a2da5e in EventProcessor::spawn_event_threads(int, int, unsigned long) UnixEventProcessor.cc:336
    #4 0x100a2e81f in EventProcessor::start(int, unsigned long) UnixEventProcessor.cc:385
    #5 0x1000e7201 in main Main.cc:1778
    #6 0x7fff7dd12144 in start (libdyld.dylib:x86_64+0x1144)

Thread T3 created by T0 here:
    #0 0x55b9f6 in wrap_pthread_create (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x509f6)
    #1 0x100a24fc9 in ink_thread_create(void* (*)(void*), void*, int, unsigned long, void*) ink_thread.h:152
    #2 0x100a24bb9 in Thread::start(char const*, void*, unsigned long, std::__1::function<void ()> const&) Thread.cc:111
    #3 0x100a2da5e in EventProcessor::spawn_event_threads(int, int, unsigned long) UnixEventProcessor.cc:336
    #4 0x100a2e81f in EventProcessor::start(int, unsigned long) UnixEventProcessor.cc:385
    #5 0x1000e7201 in main Main.cc:1778
    #6 0x7fff7dd12144 in start (libdyld.dylib:x86_64+0x1144)

SUMMARY: AddressSanitizer: heap-use-after-free (libclang_rt.asan_osx_dynamic.dylib:x86_64h+0x51057) in __asan_memcpy
Shadow bytes around the buggy address:
  0x1c4a000055e0: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x1c4a000055f0: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x1c4a00005600: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
  0x1c4a00005610: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
  0x1c4a00005620: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
=>0x1c4a00005630: fd fd fd fd fd fd fd fd fd fd fd fd fd fd[fd]fd
  0x1c4a00005640: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
  0x1c4a00005650: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
  0x1c4a00005660: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
  0x1c4a00005670: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
  0x1c4a00005680: fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd fd
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb

https://github.com/apache/trafficserver/commit/88b5d7c71e1288400356adb6103f4ec18794652b fix this.


[Bug]: Null DataTable links break build

Describe the bug
I have an issue with the DataTable component. I want to have the contentType=link type of table. For some data I have null, which are correctly handled in the dev config. But when I want to build for prod, I have an error:
"error: TypeError: Cannot read properties of null (reading 'startsWith')".
Steps to Reproduce

datatable with contentType=link
at least one null in link column
attempt to build

Severity
serious, but I can work around it
Additional Information, or Workarounds
remove nulls or replace with strings

https://github.com/evidence-dev/evidence/blob/c835f1d2c6062229c07d3021be659760e4657fce/packages/ui/core-components/src/lib/unsorted/viz/table/TableRow.svelte#L149
seems to be from this line, might be good to just return null in addBasePath if the input is null? would prob preserve prior behaviour

https://github.com/evidence-dev/evidence/blob/c835f1d2c6062229c07d3021be659760e4657fce/packages/lib/sdk/src/utils/svelte/addBasePath.js#L13
@zachstence seems like it's already halfway there, I'm assuming it's fine to switch to a == undefined?

@csjh How about if (typeof path !== 'string') return path;

Does that have any chance of clashing with weird proxy stuff we might do? I think all that might've been killed with the input DAG PR but might be forgetting something
If addBasePath only interacts with row values that would be good though


Added JWST imagery up to CasA, Fomalhaut

Still need the following with manual alignment:

[ ] Uranus Wide Field
[x] Fomalhaut
[ ] Dwarf Galaxy WLM
[ ] PEARLS
[ ] Jupiter (if possible)

Also need to build search data, @pkgw can you do that please?

Thanks! I've updated the search data too now.


Charts: Monitoring: PrometheusRules: 3/4 annotation checkboxes are unresponsive until 'Add Annotation' button is clicked

Setup

Rancher version: 2.10
Rancher UI Extensions:
Browser type & version: Chrome

Describe the bug
Under PrometheusRules > Alterting Rules section, 3/4 annotation checkboxes are unresponsive until 'Add Annotation' button is clicked.

To Reproduce

install monitoring
go to PrometheusRules
click Create
click Add Alert
under Annotations click the checkboxes

Result
Under PrometheusRules > Alterting Rules section, 3/4 annotation checkboxes are unresponsive until 'Add Annotation' button is clicked.
Expected Result
Under PrometheusRules > Alterting Rules section, 4/4 annotation checkboxes are responsive.

Screenshots
https://github.com/user-attachments/assets/9abe6dd6-90c9-4fd2-a8fe-5bba64c73e25

Additional context

Found while testing https://github.com/rancher/dashboard/issues/12436

Under PrometheusRules > Alterting Rules section, all annotation checkboxes are responsive to click.
Tested in:

Rancher v2.10-212d8b6e92992235d791d8f2aaea8436ab4f6b77-head
Dashboard master https://github.com/rancher/dashboard/commit/c321e42bde6f043d9b13272e548455eb90315b07


feat: Nullifier read requests in public kernel

Please read contributing guidelines and remove this line.

Benchmark results
Metrics with a significant change:

tx_sequencer_processing_time_ms (1): 606 (+27%)
l2_block_public_tx_process_time_in_ms (8): 4,980 (+29%)
l2_block_public_tx_process_time_in_ms (32): 19,451 (+27%)
l2_block_public_tx_process_time_in_ms (64): 38,854 (+27%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 87081317 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,700
18,884
36,452

l1_rollup_calldata_gas
66,456
239,000
469,544

l1_rollup_execution_gas
194,404
500,306
908,802

l2_block_processing_time_in_ms
1,158 (-3%)
4,445 (-2%)
8,874 (-1%)

note_successful_decrypting_time_in_ms
194 (-3%)
542 (-1%)
992

note_trial_decrypting_time_in_ms
103 (-4%)
62.9 (-3%)
126

l2_block_building_time_in_ms
17,519 (+7%)
69,566 (+7%)
139,365 (+7%)

l2_block_rollup_simulation_time_in_ms
12,467
50,037 (+1%)
100,355 (+1%)

l2_block_public_tx_process_time_in_ms
:warning: 4,980 (+29%)
:warning: 19,451 (+27%)
:warning: 38,854 (+27%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,708 (+2%)
27,803 (+5%)

note_history_successful_decrypting_time_in_ms
1,423 (+14%)
2,405 (+1%)

note_history_trial_decrypting_time_in_ms
90.4 (-7%)
141 (+7%)

node_database_size_in_bytes
18,780,240
35,536,976

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
252 (+1%)
44,736
28,001

private-kernel-ordering
192
52,625
14,627

base-rollup
1,327
177,932
997

root-rollup
72.7
4,320
825

private-kernel-inner
319 (-1%)
73,715
28,001

public-kernel-app-logic
205 (+5%)
34,918 (+8%)
27,971 (+10%)

public-kernel-tail
104
34,506
27,971

merge-rollup
8.65
2,840
997

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts

tx_size_in_bytes
19,179

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
2,669
1,372

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.0290 (-9%)
:warning: 606 (+27%)


Null operation produces an error

Reported by Andrew Wittenberg on 5 Jul 2004 19:19 UTC
This arises in a script that has an optional string argument, corresponding
to an optional command the script can execute if the user desires.
The script is supposed to do nothing if the argument is absent or blank.
Instead, it gives an error due to the following:
    NOAA/PMEL TMAP
    FERRET v5.70
    Linux(g77) 2.4.18 - 06/30/04
     5-Jul-04 15:09

yes? def sym a " "
yes? ($a)
!-> DEFINE VARIABLE
**ERROR: invalid command: DEFINE what name?
Why is the ($a) not simply treated as a blank line?
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/919

Comment by<EMAIL_ADDRESS>on 6 Jul 2004 21:31 UTC
This is essentially a parsing bug. A simple space on the
command line is parsed as "no command".  Because it is
first translated, ($a) makes it farther into the parsing
process, and a call is made to alias_command which sees
if the command buffer contains text that matches any of
the aliases. The zero-length string (blanks are removed)
is mistakenly matched with the first alias "LET". A simple
check should fix this: require the length of the string
(non-blanks) to be nonzero, in ALIS_COMMAND, before the
call to ALIAS_ID.
BUT, note that it now behaves as if you had entered " "
at the command line, which is a syntax error, as it is
a non-zero-length string, and the command does not start
with a letter...

Comment by<EMAIL_ADDRESS>on 8 Jul 2004 17:36 UTC
Changed match4.F to fix the error-message portion of this:
it no longer will come back with the spurious
!-> DEFINE VARIABLE
**ERROR: invalid command: DEFINE what name?
but, the command
yes " "
whether it is issued with a symbol or directly on
the command line, will still come back with:
**ERROR: command syntax.
Leaving the bug open, it'd be nice to change the parsing
so that yes? " "  is merely skipped.

Comment by<EMAIL_ADDRESS>on 13 Jul 2004 02:23 UTC
I agree, it would be better to have blank commands ignored altogether.
(That way one has full generality in creating/executing a command
string, including the "null" command.)
Could this be done by trimming strings of leading and trailing blanks
before trying to executing them?  Then the null command would be
interpreted as "" (a zero-length string) instead of " ".

Comment by<EMAIL_ADDRESS>on 15 May 2007 22:01 UTC
Fixed in parse_command.F. At statement number 500, the code is going to go ahead
and see what command is to be executed. It's found some number of "words" in the
command (single entities that make up the command; the first will be the command
name). Simply exit if word_num is 0. continuation lines have already been
addressed at this point.


How to set initial value?

If I want to pass in a key and have that be the selected one from my data as an initial value, then upon edit I just erase it, is it possible with this component?

You can set an initial value using the value prop which will be used to search and display any matching results.
<Typeahead value="initial value" {data} {extract} />

it sometimes shows the dropdown result, I'm ignoring it for now,


Config error - No module named 'click'

I'm using the add-on Check Home Assistant configuration for testing my configuration before updating the core.
Unfortunately the add-on points out the following error:
Platform error vacuum.xiaomi_vacuum - No module named 'click'
Updating works fine and I don't see this error in my HA log.
Can you please have a look and fix this?

What add on? You really did not provide any info on what exactly is the problem. Just that some addon you use for something else has a problem with Click.
Click is used by Miio for their command line stuff not really needed here IIANM.
I made a branch without it called "no_click" you can try that one. I currently do not use this library until I get another vacuum, so you have to test for yourself.

The add-on Check Home Assistant configuration checks whether your configuration files are valid against the new version of Home Assistant before you actually update your Home Assistant installation. This add-on will help you avoid errors due to breaking changes, resulting in a smooth update.
Here is the full output of the add-on:
`[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[22:05:50] INFO: Don't worry, this temporary installation is not overwriting your current one.
[22:05:50] INFO: Installing Home Assistant: stable...
[22:05:50] INFO: Please be patient, this might take a few minutes...
[22:06:18] INFO: Installed Home Assistant 2022.2.2
[22:06:18] INFO: Making a copy of your configuration for checking...
[22:06:18] INFO: Checking your configuration against this version...
[22:07:13] ERROR: The configuration check did not pass!
[22:07:13] ERROR: See the output below for more details.
Testing configuration at /tmp/config
Failed config
General Errors:
- Platform error vacuum.xiaomi_vacuum - No module named 'click'
Successful config (partial)
[22:07:13] INFO: The full output has been written to /share/check_config.txt
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.`
Thanks for your fast response and the new version. I tested it but I get the following error:
`Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht
Logger: homeassistant.loader
Source: custom_components/xiaomi_vacuum/miio/device.py:124
Integration: Xiaomi Vacuum (documentation)
First occurred: 22:15:56 (1 occurrences)
Last logged: 22:15:56
Unexpected exception importing platform custom_components.xiaomi_vacuum.vacuum
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 566, in get_platform
cache[full_name] = self._import_platform(platform_name)
File "/usr/src/homeassistant/homeassistant/loader.py", line 583, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
File "/usr/local/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 850, in exec_module
File "", line 228, in _call_with_frames_removed
File "/config/custom_components/xiaomi_vacuum/vacuum.py", line 7, in
from .const import *
File "/config/custom_components/xiaomi_vacuum/const.py", line 2, in
from .miio.dreame_const import (
File "/config/custom_components/xiaomi_vacuum/miio/init.py", line 1, in
from .device import Device
File "/config/custom_components/xiaomi_vacuum/miio/device.py", line 124, in
class Device(metaclass=DeviceGroupMeta):
NameError: name 'DeviceGroupMeta' is not defined
`

That is an interesting Idea. thanks for sharing the addon.
Whoops, I forgot to delete some of the references. Do you mind trying the new beta release? v1.3b2_No_Click

You're welcome.
I tested the new version and the first error disappeared. But now this error occurs:

[22:09:37] ERROR: The configuration check did not pass!
[22:09:37] ERROR: See the output below for more details.
Testing configuration at /tmp/config
Failed config
General Errors:
- Platform error vacuum.xiaomi_vacuum - No module named 'construct'
Successful config (partial)

Since the real update does not show any errors I'm not sure if this is worth fixing. Perhaps you can test the add-on and find out why these kind of errors appears.

Hmm :/ these are all python libraries used in python-miio. it is really odd that the addon is complaining about them.
Unfortunately I do not have a physical vacuum device right now to test it as my D9 died months ago. I just try to keep this thing running for people who currently use it, at least until I get another device and start working on it again.
Click was not really needed in this integration, as it is used for python-miio's command line stuff but I fear construct is.
I'm not a python expert but I beleive the config addon should not throw errors about these libraries.

Thanks for your effort.
I saw a few other users which have also problems with the Check Home Assistant configuration add-on. For example:

It also still includes the piece to run the core_check_config addon so the notification can include the status of that output although truth be told I haven’t been able to trust that addon for a while. I keep getting false error messages about bad python dependencies which I then just have to ignore since they don’t make any sense. They’re always about packages that if they were actually missing HA’s CI never would’ve passed in the first place. For now I’m leaving it in but jury is still out on the usefulness of that check these days.

So I guess everything is fine and we can ignore these errors.


Replace usage of cfoundry which is broken with latest CF

See https://github.com/pivotal-cf/broker-registrar/issues/4
Suggestion includes using https://github.com/cloudfoundry/cf-mysql-release/blob/develop/jobs/broker-registrar/templates/errand.sh.erb

https://github.com/cloudfoundry-community/broker-registrar-boshrelease/pull/2


Fix cfitsio include path on mingw

Still no joy even after that fix:
D:\a\_temp\msys\msys64\mingw64\bin/x86_64-w64-mingw32-gcc.exe -shared -Wl,--enable-auto-image-base -pipe -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high -pipe -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high -pipe -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high -march=x86-64 -mtune=generic -O2 -pipe -D__USE_MINGW_ANSI_STDIO=1 -s build/temp.mingw-3.8/imagecodecs/_rcomp.o build/temp.mingw-3.8/imagecodecs/_rcomp-cpython-38.def -LD:/a/_temp/msys/msys64/mingw64/lib/python3.8/config-3.8 -LD:/a/_temp/msys/msys64/mingw64/lib -lcfitsio -lz -lpython3.8 -lm -lversion -lshlwapi -o build/lib.mingw-3.8/imagecodecs/_rcomp-cpython-38.dll
  D:/a/_temp/msys/msys64/mingw64/lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: build/temp.mingw-3.8/imagecodecs/_rcomp.o:_rcomp.c:(.text+0x14079): undefined reference to `fits_rcomp_short'
  D:/a/_temp/msys/msys64/mingw64/lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: build/temp.mingw-3.8/imagecodecs/_rcomp.o:_rcomp.c:(.text+0x14131): undefined reference to `fits_rcomp_byte'
  D:/a/_temp/msys/msys64/mingw64/lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: build/temp.mingw-3.8/imagecodecs/_rcomp.o:_rcomp.c:(.text+0x14241): undefined reference to `fits_rcomp'
  D:/a/_temp/msys/msys64/mingw64/lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: build/temp.mingw-3.8/imagecodecs/_rcomp.o:_rcomp.c:(.text+0x1857e): undefined reference to `fits_rdecomp'
  D:/a/_temp/msys/msys64/mingw64/lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: build/temp.mingw-3.8/imagecodecs/_rcomp.o:_rcomp.c:(.text+0x18acf): undefined reference to `fits_rdecomp_byte'
  D:/a/_temp/msys/msys64/mingw64/lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: build/temp.mingw-3.8/imagecodecs/_rcomp.o:_rcomp.c:(.text+0x18bee): undefined reference to `fits_rdecomp_short'
  collect2.exe: error: ld returned 1 exit status
  error: command 'D:\\a\\_temp\\msys\\msys64\\mingw64\\bin/x86_64-w64-mingw32-gcc.exe' failed with exit status 1

mingw currently ships cfitsio 3.48 which does include these in ricecomp.c, but they are for some reason not exported by the dll, so investigating the build and packaging over there first...

Ah, none of the symbols in fitsio2.h are marked as exportable/public with CFITS_API...
Is this a bug upstream, or are these really private? If the latter, I guess imagecodecs shouldn't be relying on them? @cgohlke

On Windows, link with the static cfitsio.lib or exclude the rcomp extension.

Thanks. I think we'll be disabling it on mingw until these become part of the public cfitsio API.

v2021.6.8 contains a vendored cfitsio/ricecomp.c file that is compiled with the _rcomp extension on Windows.

Brilliant, builds ootb now, thanks!


CI ONLY 1.35.0 release

Babbage and advertise that we know about the Babbage era
(cherry picked from commit 6d33fa3)

I think we can close this historical PR.


Refactor: Asset names and symbols

The major changes are:

Using asset names on the left side of the transaction lists instead of the asset symbols
Using full asset names instead of just symbols in the top of the asset details page.
Consolidation of 2 separate transaction composables into one transactionData composable.
Major tweaks on how we pass the asset name and symbol to some components. For example some components were able to receive the tokens as property (ext-tokens) and also establish the tokens on its own. Now they are single purpose: so the parent needs to establish and pass the asset and the child purpose is to display them.
The logic that was establishing the transaction asset contractId was moved to middleware composable, so the transaction returned by the mdw is already "resolved" with correct data.
Some naming changes. Mostly because we use "token" as a reference to fungible tokens and "coin" as reference to the protocol main asset. If something can be both token and cain we need to call it an "asset".
Simplifying the AE coin symbol naming - instead of using both "Æ" and "AE" we are now using only "AE". So every protocol now has only one coin symbol without any quirky exceptions here and there.
Instead of displaying "AE Coin" label  for tokens we don't have any data we are now displaying "Unrecognized".

Actual Ae swap to WAE:

AE is marked as Unrecognized
Swap route is broken

Actual Ae swap to WAE:

AE is marked as Unrecognized
Swap route is broken

Fixed. Can you please take a look again?

Fixed. Can you please take a look again?

Fixed. Can you please take a look again?

it seems that for me this issue isn't reproducible:

it seems that for me this issue isn't reproducible:

My gif is based on wallet. superhero.com
.

The line that is coming from first token is broken.

 The line that is coming from first token is broken.

Fixed. Can you please retest?

For me the swap route looks good:

I have a question: is aeternity contract displayed properly?

also when I'm clicking to see the details of ae contact the redirection will be wrong: https://aescan.io/accounts/aeternity
Tx details: https://aescan.io/transactions/th_2fMGXFcCfyisc5Re4tRq6XrS2CLfzEfeskRpKxfuaP7DjyThWn

For me the swap route looks good:

Well first token suppose to Be AE Coin, so no, it is still not good. @Liubov-crypto can you please compare to a master branch behaviour, for swaps and pool movements.

For me the swap route looks good:
Well first token suppose to Be AE Coin, so no, it is still not good. @Liubov-crypto can you please compare to a master branch behaviour, for swaps and pool movements.

Please try now @CedrikNikita @Liubov-crypto

Please try now @CedrikNikita @Liubov-crypto

Besides the issue that Nikita described I got the following:

AE symbol is displaying as WAE in Tx details:

AE contract is still broken, also check AE symbol near sum (here it is as Token):

@peronczyk please check it.

@CedrikNikita
Regarding the look of the swap route I had a conversation with Tsvetan and we agreed to change the design as follows:

@Liubov-crypto
I've pushed new fixes for the mentioned bugs. I also removed the contract ID for the AE Coin for the pool transaction because the coin has no contract (it's not a token).

LGTM.
The issue with displaying WAE as AE I will log separately.


Update vulkano version to 0.12.

Let me know if there's anything else I've missed!

Yeah you missed some.
rg 0.11 should reveal everytime the old version is used. (or with your search tool of choice)

There's also the actual Cargo.toml used to run the vulkano-www examples.

Whoops looks like we forgot about this, resolved by https://github.com/vulkano-rs/vulkano-www/pull/64


Need a tutorial on how to use this Application.

Help me

Help me

You can join our discord server where everyone can help you. If you're having problems you can always open a ticket and I or the support team will sort it out for you.
But yes, I should have some guide here on GitHub for quick guides.

Guide


Ability to handle multiple accounts at once without user signing out and signing back in again

We have successfully implemented idsvr4 and it's awesome. Thanks for all the hard work!.
The users of our system could have multiple accounts and right now the only way for them to switch accounts is to logout and re-login. Is there any way that idsvr can handle multiple login sessions so that users can seamlessly switch between different accounts?
Thanks!

"multiple logon sessions" is tricky. You can append prompt=login to re-show the login page instead.

I thought of this as well and also your solution for it @leastprivilege. But will the silent renewal of access tokens in JS clients still work then? In client nr1 after the second login.

If you login to a second client app with a different identity at IdentityServer, then it's a new session and a user subject id, so the silent renew will no longer work at the first client app.

Yeah. What is the solution for that? Multiple sessions?

We only support a single authentication cookie as of now in IdentityServer.

Alright, and because of that there is no possibility to have multiple "sessions" right now?

As for now - yes.

As for now - yes.

Has this situation changed by now, or are is it in the roadmap to support this scenario? My customer is using IdSvr4 as the corporate OIDC Authority for internal (employee) and customer facing apps,  which normally rely on different IdPs. For employees, it is problematic that they need to sign-in/sign-out with the coorporate (ADFS) credentials and the local IdP credentials when switching apps.

no update. sorry.


Add flag to tweak parallelisation

Currently, batch mode creates a thread per core. On a Pi 4 this results in 100% CPU. This is a sensible default, but a flag should be added to allow this to be overridden.

Added in v0.2.1


Update fractal tasks for 0.10.0 task version

Codecov Report
Patch and project coverage have no change.

Comparison is base (4fbba0f) 40.07% compared to head (503f181) 40.07%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #19   +/-   ##
=======================================
  Coverage   40.07%   40.07%
=======================================
  Files           4        4
  Lines         277      277
=======================================
  Hits          111      111
  Misses        166      166

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


dealing with spam

What do you all think about turning on interaction limits?

If we suggest that users just submit a signed PR or request to join, it might help out with the spam.

good idea, I will do that!


Inserting ReCaptcha 3 token in FormData before submit

Firstly, thank you for making this awesome form validation library. Evaluated many and finally settled on Conform as it works really well with Remix.
My question pertains to implementing ReCaptcha 3 along with a Form. I need to send a Captcha token with FormData which can be validated server-side in the action. The token must be generated and inserted into the FormData before the request is sent.
How should I go about doing this? Is there an obvious way to do this that I am missing? I read the documentation thoroughly but couldn't find anything that could help me in this.
Thank you!

Providing relevant ReCaptcha 3 documentation for reference: the captcha challenge is invoked programmatically.

Hi @shripadk, thanks for giving conform a try!
I will suggest you to take a look of this article from sergiodxa.
i.e.
import { captcha } from '~/services/captcha.client';
import { Form, useSubmit } from '@remix-run/react';
import { useForm } from '@conform-to/react';

export default function Login() {
  const submit = useSubmit();
  const form = useForm({
    async onSubmit(event, { formData }) {
      // Prevent default form submission
      event.preventDefault();

      // grab the form element
      const $form = event.currentTarget;

      // Set the captcha token
      formData.set("captcha", await captcha());

      // Submit the form data and reuse the action from the form
      submit(formData, {
        method: 'post',
        action: $form.getAttribute("action") ?? $form.action,
      });
    },
  });

  return (
    <Form method="post" {...form.props}>
      Email address
      <input type="email" name="email" />

      Password
      <input type="password" name="password" />

      <button type="submit">Log In</button>
    </Form>
  );
}

@edmundhung Thanks a lot!


I think I found a mistake in the book

here you are saying if the slice is defined without a value its value will be nil
but when I tested it in go playground I got this

this is in chapter 3 I mistakenly made it here

my bad I thought the Println function will print the value  of the y variable but it is not
y == nil is true
but when you print the y variable it will print [] not the value nil


feat(linter): eslint/no-import-assign (partially done, in nursery category)

Implement ESLint's no-import-assign rule.
To get started, run
just new-rule no-import-assign

hi, i want to continue this rule, but i saw that it's implemented, what can i do to conclude ?

hi, i want to continue this rule, but i saw that it's implemented, what can i do to conclude ?

There are some cases that are not supported yet, we can try to pass more!

should i open a else block to deal with a Read end ReadWrite flag in no_import_assign.rs?

what approach do you suggest in this case?

should i open a else block to deal with a Read end ReadWrite flag in no_import_assign.rs?

You may give it a try 😁

To improve this should improve all its references like @Boshen said here #2164

I think we can port this exactly https://github.com/eslint/eslint/blob/972ef155a94ad2cc85db7d209ad869869222c14c/lib/rules/no-import-assign.js#L204-L206


Registry hack for Windows - using id #1 may break other allowed installations

Body
In the reg file, I'd recommend adding bypasses for HKCU and using the number 9 or something instead of 1. If there are other extensions installed they may be overwritten by using the same id number (like Anti-Captcha for example).
Version
0.0.0
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
No response

Please install the unpacked version.


Roles checkboxes aren't aligned with labels

Describe the bug
The "Roles"
To Reproduce
Go to edit a user's "Roles" -- the checkbox labels are not aligned with the checkboxes.
Expected behavior
The labels should appear next to checkboxes.
Priority

Low priority (annoying, would be nice to not see)

Affected roles

Staff

Screenshots

Desktop (please complete the following information):

OS: MacOS 12.4
Browser: Firefox

Additional context
In order to fix this in Hypha we need to add a hook to wagtail admin to load custom CSS on admin pages.
From there I want to note that simply adding float left to the checkbox doesn't do the full trick because of the way Wagtail checkboxes are rendered in Fun and Exciting ways.  In particular they used a -5px top which messes with vertical alignment.  That has since been changed though there is still a 2px top which might still cause a pixel of net awkwardness.
I'll have a PR for this soon.

This how it looks in Wagtail 3 so it will be fixed automatically as soon as we upgrade.

Much better -- though I think we'll still want to address the vertical centering of those labels (they're zippered right now instead of aligned).
Should I hold off on the patch in Wagtail 2 and wait until 3?

Yes, we will try to get in Wagtail 3 in a couple of weeks.

See https://github.com/HyphaApp/hypha/issues/2880

The wagtail 3 feature is merged and also this particular issue with vertical alignment has been addressed in it. Closing the issue now.


Running puppet or bash script during provisioning of images

As a DEVOPS engineer I would like to have possibility to upload the script, when I configure subscription, just after "get it now",  which will be executed when machine is provisioned in the cloud to be able to configure it automatically at the startup. It can be evolved to providing list of available services to installation (postgresql, mongodb, glassfish or even the name of docker image to be downloaded and started on the machine) on the node during configuration step of subscribing.

This is a valid request and is a candidate for this year's releases, but not yet scheduled.


Scrolling tabline

environment

NVIM 0.2.0-dev
vim-airline: 4a47cd70cd (almost master)
OS: Windows, but on WSL (so ubuntu 16.04 I guess ?)
 AirlineInit()
            let g:airline_section_b = airline#section#create(['branch'])
            let g:airline_section_c = airline#section#create(['%<', 'file', ' ', 'readonly', 'hunks'])
        endfunction

        autocmd VimEnter * call AirlineInit()

if you are using terminal:

Windows command + cygwin (... I think ?)
$TERM variable: xterm-256color
color configuration (:set t_Co?): not defined ?
if you are using Neovim:
does it happen in Vim: yes

actual behavior
When you have a certain number of tabs, they are not all displayed (that's fine), but the current one is not visible if we are on a left tab that is hidden by the overflow.
It was already reported in https://github.com/vim-airline/vim-airline/issues/488#issuecomment-251005533, but as the ticket was closed, I don't think it had the visibility it deserves.
some screens to show what is going on (tested on neovim and vim) :
Tab selected on the limits of the range of the last tab (name not complete) - neovim screen :

Tab selected not in the range of the last tabs (Tabs way on the left) - vim screen :

expected behavior
See **https://github.com/vim-airline/vim-airline/issues/488#issuecomment-251005533 ; when selecting a tab that is too much on the "left", the tabline should be redrawn to at least print the selected tab (and eventually its neighbors ?)

I've pushed PR #1684 to try and improve the situation; I'd be very grateful if anybody would like to test it!


getting started button format update, voting app services topic intro

Proposed Changes

updated CSS to implement white buttons for links to docs (vs. download installer links), based on Josh's new secondary-btn CSS with some tweaks to get it to work with current documentaton.css

updated anatomy of voting app topic title and intro

@johndmulhausen @joaofnfernandes @mstanleyjones @Raphaelbouchard
Signed-off-by: Victoria Bialas<EMAIL_ADDRESS>

What do we think, guys?
https://deploy-preview-1369--docker-docs.netlify.com/

I'm cool with it but I defer to @Raphaelbouchard

I agree, I like the distinction between CTAs that take you to more docs, downloads or tutorials

👍

@johndmulhausen @Raphaelbouchard re: button formatThanks, just to clarify re: button formats, I used Josh's button, but currently in order to get it to work, you have to override some things in documentation.css (e.g., white text is defined as a default color on an #DocumentationText a.button which is a little weird. So I had to get rid of that. It doesn't affect anything, because we are using because we already explicitly define the text color for the buttons we are using, e.g.,
#DocumentationText .darkblue-btn {
    color: #ffffff;
    margin-top: 15px;
}

Hopefully, Josh's new CSS will not have these problems. I'll check in with him today.
@johndmulhausen @ManoMarks @mstanleyjones @thaJeztah  Based on a good conversation I had with @joaofnfernandes re: tutorials yesterday, I added a little more context and explanation about artifacts like like Dockerfiles and  Compose files, and concepts like services, containers, and images so that users have a reference point for what they learn in beginner Getting Started with Docker to Define and deploy your app.  See what you think ...
This is just a start and making these tutorials easier to follow. I'd like to use this approach more thoroughly as we re-do the tutorials and choose new sample apps for them; perhaps a palette of several tutorials that build on each other, or if one big one that uses the same sample, then bite-sized topics within it, each that build on the previous.

LGTM, waiting for Jenkins migration to be done, then I'll merge this


timeAgo pipe does not correctly render future dates

What is the expected behavior?
new Date(new Date().getTime() + 19 * 3600 * 1000) | timeAgo should render in 19 hours rather than 1 second ago
What is the motivation / use case for changing the behavior?
not everything happens in the past.
Which version of Angular and Material, and which browser and OS does this issue affect?
All

I have a fix for this but can't seem to create a PR on the repo. Here it is on my fork.
https://github.com/jstandish/covalent/pull/1/commits/acb3b4345a9749c610de925defe189bd2bf6668e
https://github.com/jstandish/covalent/pull/1/commits/87768b99fc46ed9f02d83651daaff9b6e25e86af

Added a new timeUntil pipe to handle this, see PR


az stack --template-uri doesn't work for bicep template

Describe the bug
--template-uri  parameter on deployment stack command only allow to use json arm file.
To Reproduce
Steps to reproduce the behavior:

Try to deploy any bicep file from remote uri using az cli stack commands for resource group, subscription and mg:

az stack group validate \
    --name stack-armor \
    --resource-group rg-experiments-01 \
    --action-on-unmanage deleteResources \
    --deny-settings-mode none \
    --template-uri https://raw.githubusercontent.com/alefred/armor-greave/ft/stacks/demo-1/deploygroup.bicep

Error received:

Failed to parse 'https://raw.githubusercontent.com/alefred/armor-greave/ft/stacks/demo-1/deploygroup.bicep', please check whether it is a valid JSON format

Expected behavior
Template uri support bicep file deployment as ARM json file deployment
Screenshots

Repro Environment
Host OS: Ubuntu 22.04.3 LTS, Linux <IP_ADDRESS>-microsoft-standard-WSL2
Azure-cli version: 2.63.0

Thanks @dantedallag  indeed  a message indicating Bicep template are unsupported for now , could be the best.


Lazyload with intersection api with gatsby latest

Changes:

...

Type of change

[ ] Bug fix
[ ] New feature
[ ] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

Hi @ashraf-deriv

DP2P section and Sign up for your demo account switched places
Also please pull from the latest master, some of the content and design changed.
Found the alt of the images for Trade types are missing. Looks like it's production issue as well.
Instead of using dummy in the alt of of the dummy images, we can remove the word dummy

Hi @habib-deriv , Incorporated your suggestions.

in the test link there's a horizontal scroll bar on initial render that needs to be fixed

in the test link there's a horizontal scroll bar on initial render that needs to be fixed

@ali-hosseini-fs It is fixed.


app init and app use should write to the selection cache

This is new in aio-cli-plugin-console.
Once that is merged in, the config import should also write to the selection cache.

Fixed by #214


Add shaded version of Dropwizard and dependencies

Should be complete; undergoing final testing before merging.

Stalled by logging problem. Closing this for now.


The Search Function is the Worst I Have Ever Encountered

Problem
Okay, so if I want to search for a video, I am screwed. Let me just show you a comparison:
Youtube

Odysee

Primary Solution
I think that the "Exact match" option is a big step in the right direction. The obvious solution is to make the search algorithm better. But that is easier said than done.
Secondary Solution
Now I am no authority when it comes to the technical process of how searches are executed, but I will do my best 'cause the feature request template asks me to. So I am thinking something more along the lines of searching with tags.  Maybe some options like search a certain creator or make certain keywords exact match only like you can with some search engines by using quotes
Conclusion
Anyways, I think that this is the main thing driving users away from Odysee sending them crawling back to Youtube or at least it is for me. I think that this is where a much larger amount of development effort should be allocated to and prioritized above other issues due to the extremity of this issue.

Thanks for the suggestion. There's a separate private repo for the search features and improvements are on the Todo list.


Remove the configured ActionMailer preview path

Because it's the default configured path since RSpec 3.2.0, and RSpec warn about it:
WARNING: Action Mailer preview_path is not the RSpec default.
I think RSpec should honor a configured config.action_mailer.preview_path and don't warn about it when it's explicitly set.
https://github.com/rspec/rspec-rails/pull/1236

:+1:

:+1:, cool if it's now by default.


fix: Wait for seek to complete

Description
Attempt to fix flaky tests on Linux for positionEvents with TimerPositionUpdater.

[ ] need to validate if the tests are passing multiple times after each other
[x] test what the Playbin GstState is after seeking

Checklist

[x] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, refactor:,
docs:, chore:, test:, ci: etc).
[x] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation and added dartdoc comments with ///, where necessary.
[x] I have updated/added relevant examples in example.

Related Issues

Is it ok to merge? I added quite a lot commits since your review @spydon :)

If the AudioPlayer is in a stopped state, the seek function will never complete, and the TimeoutException will fire 30 seconds later.
From the perspective of a user of the library, the application will hang 30 seconds before proceeding.
Sample code:
    var player = AudioPlayer();
    await player.setSource(DeviceFileSource(SomeAudioFile));
    await player.resume();
    sleep(Duration(seconds:1));
    await player.stop();
    print('-SEEK-');
    await player.seek(Duration(seconds:15));      // --- HANGS FOR DURATION OF TIMEOUT HERE
    print('-SEEK FINISH-');


Improve adapter code

Purpose
Fixes https://github.com/wso2/apk/issues/1971
Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f07876c) 26.30% compared to head (2706e38) 24.43%.
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1972      +/-   ##
==========================================
- Coverage   26.30%   24.43%   -1.88%
==========================================
  Files         124       43      -81
  Lines       14652     8776    -5876
  Branches     2575        0    -2575
==========================================
- Hits         3854     2144    -1710
+ Misses      10674     6508    -4166
  Partials      124      124

Flag
Coverage Δ

adapter
24.43% <ø> (+0.20%)
:arrow_up:

common-controller
?

config-deployer-service
?

idp-domain-service
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Cloudflare EdgeDNS support

Is cloudflare supported as one of k8gb EdgeDNS servers ?

We didn't test it before but looking at https://github.com/kubernetes-sigs/external-dns/blob/master/docs/tutorials/cloudflare.md it looks promising.
We need to double check NS record creation support in Cloudflare.
@der-ali are you willing to test the setup assuming we provide you the config? Non of the maintainers has Cloudlfare access AFAIK, so testing on your side would be extremely valueable

Hi @ytsarev, gladly!
We are currently planning to build the second-site. I will inform as soon as we are ready for testing.

@der-ali amazing, thanks! I will try to prepare associated configs in the meantime

@der-ali we discussed this issue at today's k8gb community meeting. We are trying to evaluate if the associated external-dns cloudflare provider correctly supports "NS" record creation. Is possible that you can test Cloudflare external-dns part in isolation following https://github.com/kubernetes-sigs/external-dns/blob/master/docs/tutorials/cloudflare.md and creating NS record as expected result? Thank you so much in advance

Fixed in https://github.com/k8gb-io/k8gb/pull/1278


CMS 5.1 compatibility

Is there an ETA when this will become updated to work with CMS 5.1 recipe? Thanks.
Also should the default branch be changed to 3?

Official support for this recipe was dropped with 5.0.0 as noted in the changelog, so the CMS Squad won't be making any changes to it from here on.

Thanks. Best bet for anyone who comes across this (as mentioned in another similar issue) is to inline the recipe into their own project.


MSSQL Stored Procedure Metadata Ingestion causes Arithmetic Overflow

Affected module
ingestion/src/metadata/ingestion/source/database/mssql/queries.py
Describe the bug
The MSSQL_GET_STORED_PROCEDURE_QUERIES  sql code is using the wrong system table to compute stored procedure duration and the wrong unit conversion to seconds (millisec s/b microsec).
The query is using sys.dm_exec_procedure_stats "total_elapsed_time" column to reflect stored proc duration and using it in a DATEADD function to compute end-time.
In addition, it is dividing by 1000 to convert to seconds, but should divide by 1000000.
This column ("total_elapsed_time") is the cumulative time the stored procedure has a accrued over ALL of it's executions, not just the last one.  For a MSSQL server with a long running and frequently executed Stored Procedure, the "total_elapsed_time" value is large enough to cause an Arithmetic Overflow error in the DATEADD function.
Suggest the column that should be used for this is the "last_worker_time" column.
To Reproduce
Using an extracted subset of the MSSQL_GET_STORED_PROCEDURE_QUERIES sql:
        select
          s.last_execution_time  start_time,
          s.total_elapsed_time  elapsed_time,
          DATEADD(s, s.total_elapsed_time/1000, s.last_execution_time) end_time,
          OBJECT_NAME(s.object_id, s.database_id) as procedure_name
          from
            sys.dm_exec_procedure_stats s
          where OBJECT_NAME(s.object_id, s.database_id) IS NOT NULL
          and s.database_id = db_id()
          order by elapsed_time desc
        ;

        -- RESULT
        -- SQL Error [8115] [S0002]: An error occurred during the current command (Done status 0).
        -- Arithmetic overflow error converting expression to data type int.

Finding the offending Stored Procedure:
        select
          s.last_execution_time  start_time,
          s.total_elapsed_time  elapsed_time,
          -- DATEADD(s, s.total_elapsed_time/1000, s.last_execution_time) end_time,
          OBJECT_NAME(s.object_id, s.database_id) as procedure_name
          from
            sys.dm_exec_procedure_stats s
          where OBJECT_NAME(s.object_id, s.database_id) not like 'sp_%'
          order by elapsed_time desc
        ;

        2024-12-08 20:00:01.047	**_3431915680349_**	replication_cis
        2024-12-08 20:30:00.497<PHONE_NUMBER>221	CIS_TASKS_LOAD
        2024-12-09 02:43:41.597<PHONE_NUMBER>52	replication_abs
        2024-12-08 21:59:01.030<PHONE_NUMBER>27	CIS_TASKS_PreProcesses
        2024-12-07 06:02:20.223<PHONE_NUMBER>48	DatabaseMaintenancePlan
        2024-12-07 05:05:01.013<PHONE_NUMBER>6	GetSnrStatsForSPU

Expected behavior
MSSQL Stored Procedure Metadata would be ingested into OpenMetadata instead of failing on an Arithmetic Overflow error.
Version:

OS: Ubuntu 20.04.6 LTS (GNU/Linux 5.4.0-198-generic x86_64)
Python version: 3.10.14
OpenMetadata version: 1.5.12
OpenMetadata Ingestion package version: docker.getcollate.io/openmetadata/ingestion:1.5.12

select @@servername, @@version;
-- SQLSERVER
-- Microsoft SQL Server 2008 R2 (SP3-GDR) (KB4057113) - 10.50.6560.0 (X64)
--	Dec 28 2017 15:03:48
--	Copyright (c) Microsoft Corporation
--	Standard Edition (64-bit) on Windows NT 6.1  (Build 7601: Service Pack 1) (Hypervisor)
Additional context
Suggested SQL correction:
MSSQL_GET_STORED_PROCEDURE_QUERIES = textwrap.dedent(
"""
WITH SP_HISTORY (start_time, end_time, procedure_name, query_text) AS (
select
s.last_execution_time start_time,
-- DATEADD(s, s.total_elapsed_time/1000, s.last_execution_time) end_time,
DATEADD(s, s.last_worker_time/1000000, s.last_execution_time) end_time,
OBJECT_NAME(object_id, database_id) as procedure_name,
text as query_text
from sys.dm_exec_procedure_stats s
CROSS APPLY sys.dm_exec_sql_text(s.plan_handle)
WHERE OBJECT_NAME(object_id, database_id) IS NOT NULL
AND s.last_execution_time > '{start_date}'
),
Q_HISTORY (database_name, query_text, start_time, end_time, duration,query_type, schema_name, user_name) AS (
select
db.NAME database_name,
t.text query_text,
s.last_execution_time start_time,
-- DATEADD(s, s.total_elapsed_time/1000, s.last_execution_time) end_time,
-- s.total_elapsed_time/1000 duration,
DATEADD(s, s.last_worker_time/1000000, s.last_execution_time) end_time,
s.last_worker_time/1000000 duration,
case
when t.text LIKE '%%MERGE%%' then 'MERGE'
when t.text LIKE '%%UPDATE%%' then 'UPDATE'
when t.text LIKE '%%SELECT%%INTO%%' then 'CREATE_TABLE_AS_SELECT'
when t.text LIKE '%%INSERT%%' then 'INSERT'
else 'UNKNOWN' end query_type,
NULL schema_name,
NULL user_name
FROM sys.dm_exec_cached_plans AS p
INNER JOIN sys.dm_exec_query_stats AS s
ON p.plan_handle = s.plan_handle
CROSS APPLY sys.dm_exec_sql_text(p.plan_handle) AS t
INNER JOIN sys.databases db
ON db.database_id = t.dbid
WHERE t.text NOT LIKE '/* {{"app": "OpenMetadata", %%}} /%%'
AND t.text NOT LIKE '/ {{"app": "dbt", %%}} */%%'
AND p.objtype NOT IN ('Prepared', 'Proc')
AND s.last_execution_time > '{start_date}'
)
select
Q.query_type AS QUERY_TYPE,
Q.database_name  AS QUERY_DATABASE_NAME,
Q.schema_name AS QUERY_SCHEMA_NAME,
Q.query_text AS QUERY_TEXT,
Q.user_name AS QUERY_USER_NAME,
Q.start_time AS QUERY_START_TIME,
Q.duration AS QUERY_DURATION,
SP.procedure_name AS PROCEDURE_NAME,
SP.query_text AS PROCEDURE_TEXT,
SP.start_time AS PROCEDURE_START_TIME,
SP.end_time AS PROCEDURE_END_TIME
from SP_HISTORY SP
JOIN Q_HISTORY Q
ON (
Q.start_time BETWEEN SP.start_time and SP.end_time
OR Q.end_time BETWEEN SP.start_time and SP.end_time
)
order by PROCEDURE_START_TIME desc
;
"""
)

p.s.   It appears that the MSSQL_SQL_STATEMENT query (in the same file) should also be corrected, as it is doing the same thing (using total_elapsed_time column and /1000).


Add Windows support for find

Windows support needed. Currently, PkgInfo is using find (Unix) command to collect all the package dependencies.

Thanks!
getLastModificationTime is not available at compile time:
cannot 'importc' variable at compile time; stat

Ah, yes that would cause some issues for definite.


Update for R5 RP API version 2019-07-01

https://github.com/Azure/azure-rest-api-specs/tree/master/specification/netapp/resource-manager/Microsoft.NetApp/stable/2019-07-01
https://github.com/Azure/azure-rest-api-specs/pull/7020
Change of export policy naming NFSv4 to NFSv4.1
Change of creation date naming for snapshot.

/azp run net - mgmt - ci

/azp run net - mgmt - ci


Add agenda auto numbering

@normanjaeckel I can't find a rest api function to call it.

See #1701

Done in #1701.


Update minimatch from 0.x to 3.x

I noticed a warning during the npm install of our app relating to minimatch:
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue

istanbul f.ex. depends on an old fileset version and there is a ticket already for replacing fileset entirely that I also contributed to: https://github.com/gotwarlost/istanbul/issues/638 - That being said I wonder if you would be willing to accept this PR and release a 0.2.1 version of fileset to get rid of the old minimatch version right away.

Hi @gilligan,
Please update your fileset dependency to 2.0.x, it has been fixed recently.

And let me know if it works correctly, and as expected. I didn't updated the documentation but the API surface remained the same.

@mklabs i think you did not get what I meant ;) if fileset was a direct dependency under my control I would obviously just update. However various packages depend on old fileset 0.2.x and i can't update those obviously ;)


Issue with regions

Hi! I have found an issue with regions.

Test2 must be in #R1 region on the map.
Class for test
public class Class1
{
    #region R1
    public void Test1()
    {

        #region R11
        int i1 = 0;
        #endregion

        #region R12
        int i2 = 0;
        #endregion

        #region R13
        int i3 = 0;
        #endregion

        #region R14
        int i4 = 0;
        #endregion
    }

    public void Test2()
    {

    }

    #endregion
}

My guess is that the region within a region is screwing with my parsing logic.
Thanks for the clear example and sample code, will look into it!

Should be fixed in the new release!

Great! Thank you!


Implementing missing elements/component

[x] Delete (https://bulma.io/documentation/elements/delete/)
[X] Modal (https://bulma.io/documentation/components/modal/)

solved.


Build with stable rust

This is a subset of #27 which is hopefully easier to review and possibly be integrated.

lgtm! thanks!


Send message to dev team if any QE run fails

Similar to: #1503
If there are failures in any of the QE nightly runs, send a message to the dev team.

@sebrandon1 are our QE jobs really mature and stable enough? Otherwise, we might end up having too many alerts about QE failures that we wouldn't have time/people to attend/fix them quickly. There's already a pending failing job from the lifecycle ts (7h ago). To be honest, I'm not sure that this chat alert msg is the best idea here.

@greyerof I'll close this for now and we can reopen when there's a more stable set of runs.


Add length validation to the postcode field

We have a database length limit of 11 characters on the postcode field but no validation to check it when submitting. Whilst unlikely this would cause an error in our app, so this adds a length validator to control the problem.
I was looking at production and hit this which raised a rollbar error, happy to not merge this one as collecting the address is temporary, but it's here if we want to.

I'm closing this for now, I'll add length validation to all the fields with a db limit.


Allow dynamic import of react libraries

I am in the process of wrapping a component. The readme states that it has to be imported dynamically in NextJS apps (probably because it tries to import every plugin of the library it itself wraps). I tried to do it the known way, but I'm getting the following error once the app loads :

Is there a way to allow for dynamic imports?

I think the code CodeBlock does a little of this check it out here under the _get_imports  https://github.com/pynecone-io/pynecone/blob/main/pynecone/components/datadisplay/code.py

Can we really achieve something like the snippet below like that?
import React from 'react';
import dynamic from "next/dynamic";
import 'suneditor/dist/css/suneditor.min.css'; // Import Sun Editor's CSS File

const SunEditor = dynamic(() => import("suneditor-react"), {
  ssr: false,
});

const MyComponent = props => {
  return (
    <div>
      <p> My Other Contents </p>
      <SunEditor />
    </div>
  );
};
export default MyComponent;

That's what the library recommends one does and I can't see how that would be supported right now. Or is this more of a workaround?

Yes I believe so the plotly component does something almost identical
class PlotlyLib(Component):
    """A component that wraps a plotly lib."""

    library = "react-plotly.js"

class Plotly(PlotlyLib):
    """Display a plotly graph."""

    tag = "Plot"

    # The figure to display. This can be a plotly figure or a plotly data json.
    data: Var[Figure]

    # The layout of the graph.
    layout: Var[Dict]

    # The width of the graph.
    width: Var[str]

    # The height of the graph.
    height: Var[str]

    # If true, the graph will resize when the window is resized.
    use_resize_handler: Var[bool]

    def _get_imports(self):
        return {}

    def _get_custom_code(self) -> str:
        return """import dynamic from 'next/dynamic'
const Plot = dynamic(() => import('react-plotly.js'), { ssr: false });
"""

    def _render(self) -> Tag:
        if (
            isinstance(self.data, Figure)
            and self.layout is None
            and self.width is not None
        ):
            layout = Var.create({"width": self.width, "height": self.height})
            assert layout is not None
            self.layout = layout

        return super()._render()

Nice, thanks!


Add new yarn test:colors and the generated files

Added new _tests directory, containing a script to copy fragments to dev/colors section
Create every fragment with every color inside dev/colors

Note: Files of note:

_tests/colors-generator.js
package.json

Other files are generated from fragments section. Only their weight and their background colors are different.

Great. But it merges all fragments and it's colors into one directory with weird ordering.
In my view it should keep the per fragment subdirectory and then in each fragment showcase the colors. In the same order.
What do you think?

Sorry but I don't understand what you mean. Would you please elaborate?

Instead of having all color examples unordered in /dev/colors/ the idea is to have it similar to /fragments/ such as /dev/colors/fragments/item/. So on this item page it would be easy to showcase the different colors ordered specifically.

Oh so you mean to add each fragment/color into it's own page? That would be fantastic for showcasing but what about testing and checking if a new color is breaking something?

Yes we can easily more that idea for showcasing afterwards.
testing is a bit harder, but also more organized. If one changes the item fragment one doesn't have to look through all other fragments. Only for color changes one needs to check every item.
So I would go with let's do it with subdirectories in dev. Test it for a release and then see how to move it to a showcase section. Does that make sense?

General sidenote: The content left alignment with not a lot of text looks a bit weird.
Some things:

[ ] buttons: need a subtitle
[ ] buttons: we should test at least all non outline colors
[ ] content: centered one needs subtitle
[ ] editor: broken (not showing up)
[ ] embed: would just use the iframe for simplicity
[ ] hero: remove header image
[ ] items: would just showcase icon and image both with title, subtitle and text
[ ] list: subtitle?
[ ] nav: add search and breadcrumb
[ ] react: broken (not showing up)
[ ] search: subtitle?

For all fragments, that have various options it might make sense to group them by type. Say for content left we generate all colors. Then for content center and then for content right.
That would make differences for each between different colors easier to catch.

@stp-ip I don't understand the issue with search fragment. I can see both title and subtitle:

I updated the dev/colors section with new faq fragment, updated examples. Didn't update item fragment since it's being copied from the fragments section so it would be hard to change. Can we keep that as is?

Yeah resolved. Didn't see the regenerated color fragments yet.

Updated the items still open.
Items feels too crowded and not necessary. Couldn't we selectively only copy over a specific fragment. Say copy over "items" and "logos", but not the others?
Same for embed to just copy over the iframe to simplify testing etc.
Hero is harder to test with the background image. We could cheat a bit and have the background images colocated with the page or fragment and just not copy over the background for the color section therefore it would 404, but that feels like a dirty hack.
IDs would need to be fixed, when we use it as actual color showcase. If we agree, that we can merge before. We should create an issue to track it so.

I don't understand what you mean by ID.
For items fragment and others like it, I can add an exception object in the script but that would need maintaining. Would that be fine?

Whitelists are easier to maintain, but if the code would get much more complex, let's go with an exception object instead.
collapse and uncollapse use the id= of the element. These are the same for all colors and therefore break the functionality.

Mehrnoosh had it fixed by adding fragment directory name prepended to the ID. She removed it in favor of easier linking. Should I add it back?

Usually one wouldn't have the same filename in the same directory so easier linking makes more sense, but in this case it breaks functionality. It's a special case so happy to open an issue and fix it, when we move colors to be showcased. Makes sense?

Fantastic.

The same thing happens for nav search as well. Should we create an issue for that as well?

Yeah mentioned it already in the list of items. Will mark both of them checked, when we have separate issues. :+1:

Fixed:

buttons now have outline
buttons now have all supported colors
items now only shows items and logos, with title and subtitle and content
embed now only shows iframe

Not sure how to remove background image from hero without too much customization in the script.
Also split the script a little for better readability. Not much readable however. Sorry :D

Hehe wasn't sure, if it's just me not knowing javascript that good :)
Ok. Let's merge and create the issues as discussed to move the last items to their own issues. We can leave the hero image for now, but would be great, when you create an issue for it. So I might take a look at a hacky non js solution.


os: file permissions from os.mkdir are incorrect

What version of Go are you using (go version)?

$ go version
go version go1.14.1 darwin/amd64

I've tried before with a 1.13 release with the same result
Does this issue reproduce with the latest release?
Updated because I thought there might be a fix to 1.14.1.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/..redacted../Library/Caches/go-build"
GOENV="/Users/..redacted../Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/..redacted../Documents/workspace/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.14.1/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.14.1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/hq/76l3k83s55v0sggvm462gztc0000gn/T/go-build951404932=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
 On the playground it works fine, on my mac it fails:

go run main.go
2020/04/06 12:12:49 &{.sample 64<PHONE_NUMBER> {472033283<PHONE_NUMBER>7 0x118cbc0} {16777220 16804 2 68774231 501 20 0 [0 0 0 0] {1586199947 472033283} {1586199947 472033283} {1586199947 472033283} {1586199947 472033283} 64 0 4096 0 0 0 [0 0]}}
2020/04/06 12:12:49 drw-r--r--
2020/04/06 12:12:49
2020/04/06 12:12:49
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0x10a509d]

goroutine 1 [running]:
main.main()
        /Users/..redacted../Documents/workspace/private/dir-test/main.go:39 +0x47d
exit status 2

What did you expect to see?
The permissions on the directory to be the same as reported from the playground, as well as the file generation to be successful.
What did you see instead?
Wrong file permissions on the directory.

What is your umask value?  What does the command umask print?

Directories must be executable in order to see what’s inside them. Changing the perms when creating the .sample dir to 0777 (though I’d be careful giving world rwx perms to anything) fixes the issue.
The playground runs in its own sandbox with a fake filesystem. It may be that it doesn’t take this particular bit into account.

Changing the perms when creating the .sample dir to 0777 (though I’d be careful giving world rwx perms to anything) fixes the issue.

Changing it to os.Mkdir(configDir, 0777) still gives me wrong permissions, while it let's me create the file in the directory without an error.
2020/04/06 12:32:16 drwxr-xr-x

What is your umask value? What does the command umask print?

$ umask
022

It will fail on linux/amd64 with the same issue as well (altered to print confErr and bail if non-nil):
2020/04/06 15:41:34 directory doesn't exist, creating stat /home/andybons/.sample: no such file or directory
2020/04/06 15:41:34 &{.sample 4096<PHONE_NUMBER> {932390647<PHONE_NUMBER>4 0x58b100} {65026 2373912 2 16800 22334 89939 0 0 4096 4096 8 {1586202094 932390647} {1586202094 932390647} {1586202094 932390647} [0 0 0]}}
2020/04/06 15:41:34 drw-r-----
2020/04/06 15:41:34 <nil>
2020/04/06 15:41:34 stat /home/andybons/.sample/conf.yaml: permission denied

As far as I understand, you can’t stat a file within a directory without having executable permissions on the directory. @ianlancetaylor am I missing something?

Ah I see the difference in permissions. Sorry for the confusion. Was focused on the file creation.

Ah I see the difference in permissions. Sorry for the confusion. Was focused on the file creation.

No worries, was just about to point that out.

Changing it to os.Mkdir(configDir, 0777) still gives me wrong permissions, while it let's me create the file in the directory without an error.
2020/04/06 12:32:16 drwxr-xr-x

What's wrong about these permissions? That looks like what I'd expect from os.Mkdir(path, 0777) with umask 022.

What's wrong about these permissions? That looks like what I'd expect from os.Mkdir(path, 0777) with umask 022.

I would've expected 0777 to be the permissions, an error stating that with my umask setting I can't create a directory with the requested permissions or at least a notice that the permissions I requested were forcefully overridden by umask.

The purpose of umask is to be applied to the permissions of files created by a process. It's not an error to create a file or directory with permissions bits removed by the umask; the expected usage under most circumstances is for files to be created with maximal permissions and those permissions reduced to the desired set by the user's umask.
For example, the mkdir shell command executes a mkdir syscall with permissions 0777:
$ umask
0022
$ mkdir foo
$ ls -ld foo
drwxr-xr-x  2 dneil  wheel  64 Apr  6 13:02 foo
$ rmdir foo
$ umask 0002
$ mkdir foo
$ ls -ld foo
drwxrwxr-x  2 dneil  wheel  64 Apr  6 13:03 foo


Change include dir priority in CFLAGS

Move CFLAGS += -I/usr/local/include below BoringSSL, so that if OS have OpenSSL header files installed in /usr/local/include, it won't mess with vendored BoringSSL.
This should fix the issue with macOS CI job.

Checklist

[x] Change is covered by automated tests
[x] ~Benchmark results are attached (if applicable)~
[x] The coding guidelines are followed
[x] ~Public API has proper documentation~
[x] ~Example projects and code samples are up-to-date (in case of API changes)~
[x] ~Changelog is updated (in case of notable or breaking changes)~

@Lagovas thoughts?

I'm waiting green tests) We discussed this in slack and @iamnotacake mentioned that this change brought more failures and he will debug it


question about test with python3

Problem Summary

When I have ran the
leetcode test ./3.longest-substring-without-repeating-characters.py -i 'abc'
it shows:
✘ Runtime Error
✘ runtime: N/A
✘ answer:
✘ stdout: ''
✘ error: Line 66: ValueError: No JSON object could be decoded
How to reproduce

try any cpp/python with leetcode test ./[file] should reproduce the problem
Environment

leetcode-cli version:

OS version:ubuntu 16.04
Node version: 4.2.6
Npm version:3.5.2

-i is used to read testcase from stdin, for your case please try use -t instead?
leetcode test ./3.longest-substring-without-repeating-characters.py -t 'abc'

i suggest adding a flag -l for leetcode submit to specify the language of file we submit or make .lcconfig visible to leetcode submit.
p.s. leetcode-cli will set the python3 file with extention ".py3" which is pretty rare in python world. vim also doesn't regard ".py3" as a python file by default.

@shihan9 there is another python3 thread: #55


Why projects uses AUC to evaluate instead of ROC?

RT

AUC = a systematic metrics including different thresholds.


Delete /hub page

In the same spirit as #559, I'm putting forward the challenge: Can we live without this page? It's largely redundant of this documentation site. Can we lose it?

I'm on the fence about this one - I think there's value in having a simple landing page for the major sub-projects that can point people in the right direction. I imagine something like http://bokeh.org/ where there's just a splash page, high level use-case information, and pointers to next steps.
https://jupyterhub.readthedocs.io/en/stable/ does have that information, but also a lot of other information, and it's laid out more like a traditional verbose documentation page rather than having lots of UI elements that make it easy to parse.
So I guess what I'm thinking is that:

I agree that it's better for us to point to a jupyterhub docs site for this information, rather than putting the copy on jupyter.org
but, I don't know of a jupyterhub docs page that has the right high-level information / splash page right now :-/

Sooo...hmmm, thinking out loud a bit: maybe the right path forward is to think about what the end-goal should be, and then take baby steps towards that. I think the goal is to have something like a projects/ page with pointers to other project-specific places, so removing hub/ would be a step in that direction 🤔

I outlined my general feeling about this in https://github.com/jupyter/jupyter.github.io/issues/529#issuecomment-1001656348, but in case if this change was to go forward let me focus on technical aspect here:
The search results for "JupyterHub" in Google include /hub at the first position, so we probably should keep the /hub permalink working and redirect to the new target page (currently proposed to be JupyterHub documentation). This can be achieved by adding a small template as in: https://stackoverflow.com/questions/39657226/jekyll-redirect-link and redirect field.

I added some thoughts over in #529 since that's the more generic topic to discuss.

Following discussion here and in #529, I am withdrawing this proposal.


Use Core data above CBL

Core Data forces us to use an uppercase letter for the first letter of the *type" property.

We need to change from chatroom to Chatroom for it to work.

The hack in 34de87e610ec4aa13b47c6e0bf426ed64addc419 caused other problems.
The wrapper has many problems that I talk about in my documentation.
I'll keep the PR that way, but not merge it.


pc -v audits container

Describe the bug
I'd like to use the cli tool to export container audits
Expected behavior
I would expect  pc -v audits container to return back the audits from the container runtime.
Current behavior
But it is returning nothing, and when reviewing the source code, I see audits/firewall/app/container the following API being called which is for WAAS and not container runtime.
Possible solution
I would suggest we update this endpoint or create a new option for this result.
Steps to reproduce

Screenshots

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 59, node.js 5.4, python 3.7.3):
Operating System and version (desktop or mobile):
Link to your project:

PR #82 submitted yesterday

@dukekautington3rd Thanks for this. I guess we're waiting for approval now.

Closing this one as it has been fixed, thanks @dukekautington3rd and @sgordon46!