don't fail build if primary tag exists in registry

Signed-off-by: Robert Cerven<EMAIL_ADDRESS>Maintainers will complete the following section:

[ ] Commit messages are descriptive enough
[x] "Signed-off-by:" line is present in each commit
[x] Code coverage from testing does not decrease and new code is covered
[x] JSON/YAML configuration changes are updated in the relevant schema
[x] Changes to metadata also update the documentation for the metadata
[x] Pull request includes link to an osbs-docs PR for user documentation updates

Commit message could explain why we're reverting this: because failures during pulp_sync, pulp_publish, or koji_import would leave the tag in the registry but no build in Koji, meaning future builds will fail in the same way.

commit updated

rn updated: don't fail build if primary tag exists in registry


Http2 Protocol Support in libscope

Early production feedback has shown that extraction of HTTP/S metrics from connection scoped applications is very important for getting first level of observability into an application. libscope is currently able to extract HTTP metrics/events on connections where HTTP1 protocol is being used. This ticket is to extend the same metric and event extraction to HTTP2 connections.

Update - tl;tr progress but not solved yet

nghttp2 looks promising given their approach is to separate I/O from their parser. See the discussion here. Looks like we can simply feed payloads to their nghttp2_session_mem_recv() function.
Simple to add it as a contrib. Supports building in a separate cacheable folder.
I have it building and linked in and am feeding our payloads to a pair of their parsers - one for the TX side and another for the RX side of the socket.
Things initially looked good. I was able to feed it the outbound payloads from scoped curl connecting via HTTP/2. Callbacks for various events are being called and we can see things like method, url, headers, etc.
Ran into trouble when scoped curl sent the SETTINGS frame. The parser for the TX side saw it without issue. Later, when the webserver responded, the RX side parser's first input is a SETTINGS ACK that it doesn't expect. At that point, the parser is in a error state and stops generating events.
While nghttp2 expects I/O to be handled externally,it still maintains a state machine and balks when frames come in out of sequence. I can't just feed a parser payloads from one side of the conversation.
First thought was to see about disabling the state machine; just parse what I give you and generate events. Won't work because of the way previous state is referenced in later frames. Gotta have state.
Wondering now if there's a way to cross feed the two parsers with each other's outputs. Thinking...

Putting this here in case others have any interest. This is an excellent article explaining HPACK which HTTP/2 uses for header compression.

Update - tl;dr gave up on nghttp2

Came to the conclusion early yesterday that nghttp2 isn't going to work for our needs.
While it seemed promising that it wasn't doing the I/O, it is still maintaining state required for HTTP/2 header compression logic.
We can't feed their parser only one side of a conversation as it will be missing header state updates.
We can't feed both sides to one parser as it will get what it expected to have sent as inputs.
Moving on...

Update - investigating ls-hpack

After looking more at existing HTTP/2 parser libraries, still not seeing any that would fit well.
The actual parsing of HTTP/2 frames is simple. Frames have a 9-byte header with length, type, flags, and streamId fields followed by zero or more content bytes (the length header field specifies how many). We'll need TX and RX buffers in the  net_into.http struct so we can tolerate partial frames but that seems to be the only complexity of framing.
We'll need something to parse the content of the HEADERS frames. This is hopefully where ls-hpack comes in; i.e. a linked-list of lshpack_dec decoders indexed by streamId.
We'll need to sum the length of the DATA frames to produce content-length.
We don't need to parse any of the other frame types.

Update - ls-hpack works as expected

I've rolled my own HTTP/2 frame parser and am using the lshpack_dec_decode() to decode the HEADERS frame.
Scoping curl -v, I'm getting debug output from AppScope with headers and other details that match the console output so I think I have the parsing working as we need.

Update - adjusting the design a bit

With HTTP/2 header parsing working, we stepped back and looked at the overall design.
The initial plan was to mimic the way the current HTTP/1.x processing is working which is:

Scan payloads looking for the start and end of the header block.
Send that buffer to the reporting side
Skip over the body of the message

Using the same approach for HTTP/2:

Only process the HEADERS frames
Extract them as name/value string pairs
Build an HTTP/1.x-style header
send to the other side

We decided we don't like the added processing in the process' own thread and we don't like the re-parsing of the header. So, we're adjusting this a little:
We're now sending each HTTP/2 HEADERS frame to the reporting side through the circular buffer as a new EVT_H2HDR sub-type of protocol event. We'll crack them over there instead.

Need to circle back to this later...

Handle HTTP/2 frames in the same payload as the HTTP/1.1 response during upgrades.

Update - now parsing on the reporting side

I've adjusted the logic on the data side to queue HTTP/2 HEADERS frames and process them on the reporting side now. Overhead in the data side is now minimal - <10 usecs per request/response which is on par if not less than what's done on the data side for HTTP/1.
Started working on generating the http-req and http-resp events and ran into a bump. The logic for HTTP/1 saves pointers to the header values and expects the buffers they point to to remain valid until after the event is serialized to JSON. The HPACK decoder is unpacking headers one at a time and the string pointers aren't valid once the next header is unpacked. Need to copy them off somewhere.

ToDo Items

[ ] Send message from data side to let the reporting side know it can cleanup the HTTP/2 state when a channel closes.

Update - Initial Request and Response Events for HTTP/2

I have the code on the reporting side generating events now. Scoping curl https://nghttp2.org/robots.txt yields the request and response in the output. The fields in the response events are incomplete as they don't yet include the carry-over fields from the previous request.

<IP_ADDRESS>:56564 {"type":"evt","id":"pdugas-laptop-curl-2 https://nghttp2.org/robots.txt","_channel":"366285305382144","body":{"sourcetype":"http","_time":1631989648.3583419,"source":"http-req","host":"pdugas-laptop","proc":"curl","cmd":"curl -Lso /dev/null --http2 https://nghttp2.org/robots.txt","pid":1145974,"data":{"http_flavor":"2","http_method":"GET","http_target":"/robots.txt","http_scheme":"https","http_host":"nghttp2.org","http_user_agent":"curl/7.74.0","net_transport":"IP.TCP","net_peer_ip":"::2400:8902:0:0:f03c:91ff","net_host_ip":"::2601:cf:4500:f481:7657:cff2","net_peer_port":"23260","net_host_port":"56027"}}}
<IP_ADDRESS>:56564 {"type":"evt","id":"pdugas-laptop-curl-2 https://nghttp2.org/robots.txt","_channel":"366285305382144","body":{"sourcetype":"http","_time":1631989648.5326769,"source":"http-resp","host":"pdugas-laptop","proc":"curl","cmd":"curl -Lso /dev/null --http2 https://nghttp2.org/robots.txt","pid":1145974,"data":{"http_flavor":"2","http_status_code":"200","http_content_len":"62","net_transport":"IP.TCP","net_peer_ip":"::2400:8902:0:0:f03c:91ff","net_host_ip":"::2601:cf:4500:f481:7657:cff2","net_peer_port":"14953","net_host_port":"47720"}}}

Scoping curl --http2 http://nghttp2.org/robots.txt shows what we get from a protocol upgrade. That's the HTTP/1 request followed by the HTTP1 101 response. That's followed by the generated http-metrics event and finally the HTTP/2 response.

<IP_ADDRESS>:56536 {"type":"evt","id":"pdugas-laptop-curl-p2 http://nghttp2.org/robots.txt","_channel":"366170608958403","body":{"sourcetype":"http","_time":1631989533.4474449,"source":"http-req","host":"pdugas-laptop","proc":"curl","cmd":"curl -Lso /dev/null --http2 http://nghttp2.org/robots.txt","pid":1143324,"data":{"http_method":"GET","http_target":"/robots.txt","http_flavor":"1.1","http_scheme":"http","http_host":"nghttp2.org","http_user_agent":"curl/7.74.0","net_transport":"IP.TCP","net_peer_ip":"2400:8902::f03c:91ff:fe69:a454","net_peer_port":"80","net_host_ip":"2601:cf:4500:f481:7657:cff2:8287:3379","net_host_port":"33814"}}}
<IP_ADDRESS>:56536 {"type":"evt","id":"pdugas-laptop-curl-p2 http://nghttp2.org/robots.txt","_channel":"366170608958403","body":{"sourcetype":"http","_time":1631989533.621573,"source":"http-resp","host":"pdugas-laptop","proc":"curl","cmd":"curl -Lso /dev/null --http2 http://nghttp2.org/robots.txt","pid":1143324,"data":{"http_method":"GET","http_target":"/robots.txt","http_scheme":"http","http_flavor":"1.1","http_status_code":101,"http_status_text":"Switching Protocols","http_server_duration":174,"http_host":"nghttp2.org","http_user_agent":"curl/7.74.0","net_transport":"IP.TCP","net_peer_ip":"2400:8902::f03c:91ff:fe69:a454","net_peer_port":"80","net_host_ip":"2601:cf:4500:f481:7657:cff2:8287:3379","net_host_port":"33814"}}}
<IP_ADDRESS>:56536 {"type":"evt","id":"pdugas-laptop-curl-p2 http://nghttp2.org/robots.txt","_channel":"366170608958403","body":{"sourcetype":"http","_time":1631989533.6217721,"source":"http-metrics","host":"pdugas-laptop","proc":"curl","cmd":"curl -Lso /dev/null --http2 http://nghttp2.org/robots.txt","pid":1143324,"data":{"duration":174,"req_per_sec":2,"http_status":101,"proc":"curl","fd":6,"pid":1143324,"unit":"byte"}}}
<IP_ADDRESS>:56536 {"type":"evt","id":"pdugas-laptop-curl-p2 http://nghttp2.org/robots.txt","_channel":"366170608958403","body":{"sourcetype":"http","_time":1631989533.623549,"source":"http-resp","host":"pdugas-laptop","proc":"curl","cmd":"curl -Lso /dev/null --http2 http://nghttp2.org/robots.txt","pid":1143324,"data":{"http_flavor":"2","http_status_code":"200","http_content_len":"62","net_transport":"IP.TCP","net_peer_ip":"::2400:8902:0:0:f03c:91ff","net_host_ip":"::2601:cf:4500:f481:7657:cff2","net_peer_port":"14894","net_host_port":"47661"}}}

ToDo

[ ] Add req/resp tracking and metrics for HTTP/2.

Update - HTTP/2 Request and Responses

We're getting http-req and http-resp events from HTTP/2 now and they have the same fields as the HTTP/1 events.
Not yet generating http-metrics events or the http.* metrics.

ToDo

[ ] Generate http.* metrics for HTTP/2.

Update - Bugs Using a Browser

Started testing using Firefox instead of cURL to get more real-world traffic and ran into issues.
I read the spec wrong. HPACK state is supposed to be shared by all streams in a channel. That fixed the first issue where I was getting decoder failures when parsing a request/response on a different channel.
Next, I was missing RX frames. I found that the webserver was sending the frames back split into multiple packets and my stash logic was busted. Fixed that and we get RX frames.
Last, some RX frames were failing to decode. Turned out they include a dynamic table size update and the HPACK decoder was rejecting the request as it was too large. Change the max table size and I'm getting requests and responses using Firefox.

UPDATE - Scoping  Nginx

I setup scoped Nginx to handle HTTP/2 requests unencrypted on port 80 and with SSL on port 443 and I was able to hit http://localhost in a browser and get the stock welcome page. (config below) I see the HTTP/2 request and response events from AppScope as expected.
That config has a location for /push.html to test HTTP/2 Server Push. When I hit that URL in a browser, I see in the Network pane of the DevTools the initial request for push.html followed by the pushed style.css and image.jpg.

I see the initial request and response events followed by the other two responses.

Update - Missing Socket Descriptors

During integration testing, found that some scoped programs are using SSL/TLS in a way that prevents us from knowing the file descriptor for the socket so we can't find a corresponding net_info for the channel and have no saved state.
As a result, we don't have the results of earlier TLS detection, protocol detection, HTTP version detection, partial messages, etc.
The id passed to doProtocol() in these cases is the address of the SSL/TLS handle so it should be unique so we're exploring keeping another list of net_info objects to use.
One drawback so far, we don't have the actual socket info so we can't populate the net_* fields in the events.

OpenSSL and File Descriptors
In order to get the file descriptor associated with a given ssl context, OpenSSL provides several functions. In several cases, not all, including some instances of node.js, the file descriptor is not available. In these cases all of the OpenSSL functions capable of retrieving an fd return -1; no fd available.
After much time with gdb and poking around with OpenSSL source, disassembled node and copious symbol searches in node executables an answer emerges. Node never supplies an fd to OpenSSL. Node creates a socket, inits SSL, but no fd is ever exchanged. Rather, node uses the functions BIO_meth_set_read() and BIO_meth_set_write() to define callbacks to OpenSSL. When OpenSSL performs a send or receive, rather than using its' own BIO layers, including fds, it calls the method provided by node. Then, node does a mapping from the BIO object to a socket and libuv does the actual send or receive operation.
Given this model, we aren't going to map an ssl id to an fd without application specific knowledge. We could choose to interpret what node does and get the fd. That would not represent a generalized OpenSSL solution. We'd do it because, well because, it's node.
There is a potential to map fd and ssl id at runtime, so to speak. The description of such an approach is better documented elsewhere. Such an approach is likely to prove workable in a lot of scenarios, but surely not all.
At this stage of development the missing piece, due to lack of fd mapping to ssl id, is the socket connection detail; local ip:port and remote ip:port. We'll need PM feedback and design discussion to move this along.

Appreciate the effort here, @iapaddler.


Create new repos

engine-ui -> yaktor-ui-angular1
engine-red -> yaktor-node-red
(From https://github.com/SciSpike/cengine/issues/61)

yaktor-ui-angular1


Add more details to README

This PR widens the README.md with more details on the project structure and links to other resources.
The objective of the resulting document is to get the newcomer developers a kick-start on the Spine development.

@meizabel, PTAL.
@alexander-yevsyukov, @armiol, PTAL as well.

@alexander-yevsyukov, PTAL again.

@alexander-yevsyukov, PTAL again.


from what I can tell install.py requires importlib_metadata but pip is installing importlib_metadata by the name importlib-metadata and so it can't be found? also is my cpu being treated as a secondary cuda choice and that is ruining the code? is there anyway to tell this extension to ignore my cpu's built in graphics functionalities that I never use?

from what I can tell install.py requires importlib_metadata but pip is installing importlib_metadata by the name importlib-metadata and so it can't be found?
That's just my guess as to what is wrong. Below is the literal console output I'm basing my guess on. Please help.
WARNING:xformers:A matching Triton is not available, some optimizations will not be enabled.
Error caught was: No module named 'triton'
V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\venv\lib\site-packages\pytorch_lightning\utilities\distributed.py:258: LightningDeprecationWarning: pytorch_lightning.utilities.distributed.rank_zero_only has been deprecated in v1.8.1 and will be removed in v2.0.0. You can import it from pytorch_lightning.utilities instead.
rank_zero_deprecation(
*** Error running install.py for extension V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT.
*** Command: "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\venv\Scripts\python.exe" "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\install.py"
*** Error code: 1
*** stderr: Traceback (most recent call last):
***   File "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\install.py", line 2, in
***     from importlib_metadata import version
*** ModuleNotFoundError: No module named 'importlib_metadata'
Launching Web UI with arguments: --medvram --no-half-vae --censoredthispart --xformers --ckpt-dir V:\AI images stuff\checkpoint
*** Error loading script: trt.py
Traceback (most recent call last):
File "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\modules\scripts.py", line 382, in load_scripts
script_module = script_loading.load_module(scriptfile.path)
File "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\modules\script_loading.py", line 10, in load_module
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\scripts\trt.py", line 10, in
import ui_trt
File "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\ui_trt.py", line 10, in
from exporter import export_onnx, export_trt
File "V:\AI images stuff\A1111 Web UI Autoinstaller\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\exporter.py", line 3, in
import onnx
ModuleNotFoundError: No module named 'onnx'
So, I just edited install.py so it doesn't import version and doesn't use version, ran it, it "worked" or at least there were no errors, but when I went to the tab and clicked the orange export button I got this in the console
No ONNX file found. Exporting ONNX...

Am I even describing the problem correctly? I don't have a clue!
if it matters this is my pip show torch
pip show torch
Name: torch
Version: 1.13.1
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Location: c:\python3.10.0\lib\site-packages
Requires: typing-extensions
Required-by: accelerate, fairscale, timm, torchvision, xformers
TLDR: There might be many things wrong, I should maybe give up. But who knows, maybe one of you can save me?

I deleted these from my command arguments
--xformers --medvram --no-half-vae
and the export rt engine worked, not sure which one was ruining it, nor am I sure why my own manual install was getting some random cuda error message popups.....

it's not perfect though the console has these non-fatal errors:
[W] 'colored' module is not installed, will not use colors when logging. To enable colors, please install the 'colored' module: python3 -m pip install colored
[W] CUDA lazy loading is not enabled. Enabling it can significantly reduce device memory usage and speed up TensorRT initialization. See "Lazy Loading" section of CUDA documentation https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#lazy-loading
W] UNSUPPORTED_STATESkipping tactic 0 due to insufficient memory on requested size of<PHONE_NUMBER>8 detected for tactic 0x0000000000000000
W] Cache result detected as invalid for node: /input_blocks.4/input_blocks.4.0/skip_connection/Conv, LayerImpl: CaskGemmConvolution, tactic: 0x0000000204040190
Have I set something up wrong still? and if so what should I change?

TLDR: --medvram is what is f'ing with things. The reason I've had so much trouble nailing down this error though is that it seems like running with --medvram is so painful to this extension that removing --medvram and running it again won't fix the issue unless I delete my engine profiles and rebuild them...

Okay, I got it working now.

clean install of automatic1111 entirely

build profiles

generate images
all the above done with --medvram off

I turn --medvram back on

generate image, fine

export new profile fine

My only guess as to what I f'd up last time around that inspired my ranting ....
is that --medvram breaks exporting to Unet-onnx
but --medvram works fine exporting to Unet-trt
Which if true is great, it would mean adjusting the profiles can be done with medvram  flag on. As long as the initial profile was made once with medvram off.
But this is vague and mysterious and I'm guessing wildly. So, I won't close the issue, despite it working for me now.
The steps I took and problems I ran into, can't be right. I must have overlooked a detail.
I'm still paranoid I'll accidentally break it again and have to reinstall for a 4th time.
TLDR: The only sentence to read in this entire issue, if --medvram needs to be off to write the first profile for each model, that should be written down in the instructions somewhere, if I'm wrong, please figure out how I am wrong so I can know.
Thanks.

I think I figured out what was wrong. In my testing I didn't keep the prompts used perfectly consistent. Many of my errors about invalid profiles may have been due to text token lengths that I wasn't considering carefully enough. Although I think maybe the text token settings are a bit misleading.
From what I can tell min-text is not relevant at all, but max-text will error out and refuse to generate.
So there is optimal-text and max-text, and I'm not sure if optimal-text matters either? But apparently max-text matters and is important.

Okay, now I clearly don't know what I'm doing wrong. I can't seem to get it to work at all anymore, my only current guess is that it's related to my constant changing of --medvram and --no-half-vae settings and the profiles made on one flag aren't valid on another flag.

okay so the optimal text prompt length is just that optimal I guess... and the reasons I was getting no valid profile repeatedly were unrelated somehow...
Things are workingly perfectly now, but they've broken entirely so many times, and every time the only error I get tends to be no valid profile found. Despite the profile existing and being listed.
My only remaining theory is that closing automatic1111 and reopening it breaks every profile, or at least every custom profile.... which would be very odd if true. But I am almost afraid to test it given how long the profiles take to make, and how little strain leaving it endlessly open puts on my machine. But I have an obsessive need to close background programs so I'll end up testing this theory eventually either way.


App crashes with "Object has been destroyed" error

I believe it is coming from here due to win being destroyed:
https://github.com/jsonnull/electron-trpc/blob/8c2de955b2256e460d827d2103727b03807d00cf/packages/electron-trpc/src/main/createIPCHandler.ts#L57
Here's the full error:

Hey @thecodrr, can you provide more information about when the error occurs?
I'm confident I have a fix, but I want to be sure I understand the root cause before I patch it.

@jsonnull this occurs on closing the window.

Did you figure out a solution?

For us, this was happening because our app explicitly calls destroy on a BrowserWindow.
Reproduction
You can reproduce it by calling win.destroy() in the ready event handler in the example here: https://github.com/jsonnull/electron-trpc/blob/cfdb2e58126deb5867d57e21b5aac1d96ef49b44/examples/basic-react/electron/index.ts#L19
For example:
  setTimeout(() => {
    win.destroy()
  }, 5000)

I haven't found a workaround other than to avoid calling destroy().
Root Cause
detachWindow is called in win.webContents.on('destroyed', () => {: https://github.com/jsonnull/electron-trpc/blob/cfdb2e58126deb5867d57e21b5aac1d96ef49b44/packages/electron-trpc/src/main/createIPCHandler.ts#L95
detachWindow then accesses win.webContents.id, which throws an error if the window is already destroyed: https://github.com/jsonnull/electron-trpc/blob/cfdb2e58126deb5867d57e21b5aac1d96ef49b44/packages/electron-trpc/src/main/createIPCHandler.ts#L62
It also accesses win.id, which used to throw an error, but now does not: https://github.com/electron/electron/issues/38170
Apparently the destroyed event doesn't guarantee that the window hasn't yet been destroyed.
Potential Fix
The easiest would probably be to capture win.webContents.id earlier in attachSubscriptionCleanupHandlers and pass it to detachWindow: https://github.com/jsonnull/electron-trpc/blob/cfdb2e58126deb5867d57e21b5aac1d96ef49b44/packages/electron-trpc/src/main/createIPCHandler.ts#L81-L98
I'll put up a PR with this fix, but I'm definitely open to alternatives!


Fix documentation of non-existent 'editor' config

Looks like this was erroneously added two years ago in commit 1827680beaf10c0710b3daed2cf1f6679c9a070a. I suspect the author misinterpreted the --editor command line option which overrides the newContentEditor configuration option.

Hello @callahad,
thank you for submitting this addition. editor is indeed not a correct config option.


feat(instrumentation-mongoose): Support v7 and v8

Resolves: https://github.com/open-telemetry/opentelemetry-js-contrib/issues/1606
Which problem is this PR solving?
Related: https://github.com/getsentry/sentry-javascript/issues/11499

This PR adds support for Mongoose versions 7 and 8 to the instrumentation.

Short description of the changes
Mongoose 7

The instrumentation already supports async Mongoose operations.
Moved tests using callbacks into a separate Mongoose v5/v6 test file
Skipped instrumentation for deprecated remove Mongoose Docs

Mongoose 8

Skipped instrumentation for deprecated count Mongoose Docs
Skipped instrumentation for deprecated findOneAndRemove Mongoose Docs

The current set of tests is passing without any issues, except findOneAndUpdate, which starts to create only one span, starting from version 7. I separated that test for v7/v8.
To avoid separating the package because of type mismatches, I have added a few // @ts-expect-errors inside tests with their reasons.

@onurtemizkan I haven't reviewed this PR, yet. A side issue: I notice that you are force-pushing. If you are able, please don't force-push to a PR branch, at least not after there have been any reviews on it. It makes it harder to do subsequent re-reviews because it isn't clear what has changed from a previous commit.

Thanks for looking into this @trentm, and yes I'll be careful not to force-push on reviewed code. FWIW these last couple were rebases on main.


Fix highlight behavior on DropdownButtonFormField

Fixes https://github.com/flutter/flutter/issues/94605
When tapping on DropdownButtonFormField it request Focus successfully but then instantly loses it when pushing the dropdown menu. That causes DropdownButtonFormField to lose highlight when menu opens and stays highlighted after menu closes.
This PR fix this issue by using bool _isMenuOpen for InputDecoration's isFocused parameter instead of Focus.of(context).hasFocus.

@AhmedLSayed9 Any chance you can add some tests for this? We won't be able to land it without tests.

We're going to close this PR for now to get it off of our backlog.  If you want to continue with it, please feel free to submit a new PR and we'll take a look.


Selecting text operation is cancelled by 'activate'

If I want to select the text of a registry value, I click-and-drag, and the text highlights.  Once I release the mouse, the field becomes editable and my selection is cancelled.  The selection should be preserved.

Editing a registry entry now requires a double click to activate, so I'm going to assume this problem is resolved. Please reopen the issue if it is still a problem.


fixes podspec reference to source files

Hi there,
The podspec is currently pointing to a broken reference to the source files. This PR just updates it.

Thank you @chourobin !


Missing peer dependencies

This package does not provide specific peer dependencies like the docs recommends https://eslint.org/docs/developer-guide/working-with-plugins#peer-dependency
@ChristophWurst

Fix it :)


Elisavet

close the .m files in the editor to avoid commiting files such as<EMAIL_ADDRESS>Can you fix it? just update the branch

ok done


Wait forever when running an unzip file command

we have this as the command.
run(bash: "unzip /User/myName/Desktop/myZipFile.zip -d /User/myName/Desktop/)

This is just supposed to unzip a rather large zip file (100Mb)
But the Swift shell freezes the app.
We traced it down to where control is frozen, it is in this file Command.swift
public func finish() throws {
   self.waitUntilExit() // <-- It freezes here
   guard self.terminationStatus == 0 else {
      throw CommandError.returnedErrorCode(command: commandAsString()!, errorcode: Int(self.terminationStatus))
   }
}

We are forced to downgrade back to 3.0.0Beta in the mean time. Thanks for your work. Awesome library.

Ok. i tried just unzipping  a small text file. Same thing.
Investigating further..

OK. So i have now tried the runAndPrint command
and it correctly unzips the file on terminal, but,
it is sad because i cannot get the output like below
let output = try SwiftShell.runAndPrint("unzip", "/Users/jt/Desktop/Bob/myZipFile.zip", "-d", "/Users/jt/Desktop/Bob")

runAndPrint doesn't return anything, so that is expected. Could you try
let output = try SwiftShell.run("unzip", "/Users/jt/Desktop/Bob/myZipFile.zip", "-d", "/Users/jt/Desktop/Bob")

to see if it is bash that never returns. If that doesn't work I'll take a look when I get back to my computer on Monday.

ok. I'll update this over the weekend. And give you the results.

I have tested this with runAndPrint using SwiftShell 3.0.1 and it is working fine unless unzip asks for input (if e.g. the files I am unzipping already exist in the destination) and I run it from the terminal. Then the application never finishes. If I run it in Xcode and provide the input in its console it's working fine. It doesn't matter if I build the executable with Xcode or with the Swift Package Manager.
This is very strange, I don't really have any theories at the moment. Have you found any way to work around it?

@kareman the workaround for now is to use runAndPrint
Unfortunately, i cannot get a return value because runAndPrint was not built for that.
But if i use the run command it fails. Note: the run command works with other terminal commands.. it just hangs when unzippng.
I remember way back in my school days something about flushing std buffers? But im not sure if this has anything to do with this. It was too long ago.

I think the problem only occurs when the “unzip” command asks for an answer to a question (like if it should overwrite already existing files). And it seems it only does this when standard input is a terminal. Some workarounds are
let output = run(bash: "null | unzip /User/myName/Desktop/myZipFile.zip -d /User/myName/Desktop/")

or
let output = "".run("unzip", "/Users/jt/Desktop/Bob/myZipFile.zip", "-d", "/Users/jt/Desktop/Bob")

Or maybe you can call the unzip command with the flags "-n  never overwrite existing files" or "-o  overwrite files WITHOUT prompting".
Please let me know if any of these suggestions solve your problem.

I am having the same issue here as @patchthecode. The bash command does not seem to ever return.
I can get it to unzip with this command:
run(bash: "unzip ./myfile.zip"
If I then put a flag in the command it no longer returns:
run(bash: "unzip -a ./myfile.zip"
or
run(bash: "unzip ./myfile.zip -d ./myfile2
Interestingly those same commands work fine when I use runAndPrint instead:
try! runAndPrint(bash: "unzip -a ./myfile.zip"
try! runAndPrint(bash: "unzip ./myfile.zip -d ./myfile2"
I am just trying to avoid the console output because its very verbose and doesn't add anything to my script.

What happens if you try let output = "".run(bash: "unzip -a ./myfile.zip")?

Yea you are correct it only happens when unzip asks for more input. I thought I was careful to make sure to delete the unzipped file after I was done with it. On a second run of the same script it fails. I was missing a delete of a __MACOSX directory that unzip creates. That was causing unzip to ask for input.
let output = "".run(bash: "unzip -a ./myfile.zip")
Clears the prompt and allows it to continue.

Excellent. I'll close this issue now.
@patchthecode feel free to reopen it if your problem was not resolved.

@kareman i'll try it today and let you know


Backend: Endpoint to retrieve user-orders

Description
This PR will provide order info to display on order summary.
Fixes #46
How to test?
Please provide a short summary how your changes can be tested?
when migartion and seeds available for Products table -> setup database npm run db:setup, then npm run dev
Checklist

[x] I have performed a self-review of my own code
[x] I have followed the name conventions for CSS Classnames and filenames, Components names and filenames, Style filenames, if you are in doubt check the the project README.MD and here https://github.com/HackYourFuture-CPH/curriculum/blob/master/review/review-checklist.md
[x] I have commented my code, particularly in hard-to-understand areas, if you code was simple enough mark the box anyway
[x] I have made corresponding changes to the documentation, if you code was simple enough mark the box anyway
[x] This PR is ready to be merged and not breaking any other functionality

@Sarulathaanbu in issue #46 you need to do endpoint GET api/orders?user={user_id}  but for now your router different rout it is returning api/orders/{order_id}


View Interviewer/Electee After clicking on event

Feature to be able to view who is interviewing you, or who you are interviewing upon clicking on the event that you are signed up to.

Covered by #154


http transport support

This will address https://github.com/open-traffic-generator/openapiart/issues/65
@ajbalogh I have changed related to HTTP transport. I feel we need to add response for specific golang object. Please review and modify accordingly.

This PR is missing a test case in the transport_test.go file.

@alakendu please mark it as draft since we have a long way to go here.

I have added unit test for go HTTP transport. @ajbalogh  could you please review it.


fix: provider actions

What it solves
Deploy issue on dev branch
How this PR fixes it
The PROVIDER_ACTIONS enum value was updated and the type fixed in a test.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3407

Relevant Lines:
8766

💛  - Coveralls


Dev hosting instructions

This PR adds instructions showing a way to run Highfive in the context of doing development. Until I delete the branch, you can see a live version of the README changes here.

Thanks, this looks good! It might be good to add something about the config file required to setup the user/token, although perhaps that is already mentioned in the README.


Modify sys.indexes view

Signed-off-by: Avneet Toor<EMAIL_ADDRESS>Description
Because part of the work of sys.indexes was changing column data types, extensive work was done adding sys.indexes to the babelfishpg_tsql--1.2.0--1.3.0.sql upgrade script. However, PostgreSQL does not allow column data types to be changed in views once they exist, so any addition of updated views to the upgrade script where data types are changed returned an error. The steps I took to mitigate this was:

in the upgrade script rename sys.indexes to sys.indexes_deprecated
add the updated definition of sys.indexes
add the definition of any views/functions that use sys.indexes as a dependency to rebuild them to use the updated sys.indexes definition.
if sys.indexes_deprecated has no other dependencies (by, say, a customer script) drop the view. Else, keep it for now.

To aid in the above process I added 2 new helper functions:

sys.has_dependencies : Returns True if an object is dependant upon by other objects. False otherwise.
sys.drop_view: If a view is not depended upon by other objects, drop the view. Else, leave it.

Issues Resolved
BABELFISH-343
BABEL-2712
Check List

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

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

One more comment: I absolutely do not think handling user objects that depend on catalog objects is a requirement. The Postgres community is very cautious about changing the definition of catalog objects, but when they feel it's warranted they simply change them and let the user deal with it.


Temp info getting

A useful link I used to get some info: https://github.com/lod/seek-thermal-documentation/wiki/Temperature-Values

Thanks for your contribution. I will merge your code after my holidays.

Hi Andrey,
I tried your contribution concerning temperature on Maarten Project. However, i still can't understand how you managed to get these informations. Can you please explain a bit more your method and eventually how you did to obtain your linear trend line on excel.
Thank you in advance for your help.

@AchrefDash I'm curious, was the information you got meaningful? Did it match real world temperatures?
The method is trivial: I have googled out that raw pixel values from camera are absolute. This means they contain actual temperature data, expressed in Kelvin value with some linear representation. Thus, I made some experiments to figure out linear law for these values, based on ice, boiling water and human body. The values for trend line are expessed in my source code comments.

Hi @undera  Thank you for your quick response.
So, the results I have are almost similar to the results obtained with Seek App with a coefficient in addition. After doing some tests on human body, the soldering iron, lamps, electrical cabinets and others,  I got an average coefficient of 0.8 to get almost the same Seek App tempetures values but this approach remains very approximate depending on the tests conditions.

I modified your version a little bit to show the max/min hot/cold points with colored circles in the image and show the temperature with one decimal point at that location. It's not very nice but it works as a proof of concept. My problem at this point is that the shown temperatures are  a bit too high. But i have to verify this with a known reference temperature.
Example image:

Source file:
seek_viewer.zip

The ways to visualize the point may vary.
Indeed, the biggest question is the formula of translating raw value to temperature. We need more data, probably, to calibrate it.

just as a side note,  Seek Thermal states that

All Seek imagers are generally accurate within 5 degrees C above or below the reading
(http://www.eevblog.com/forum/thermal-imaging/seek-compact-pro-temperature-readout-issue/msg1403982/#msg1403982)

I got a new CompactPro last week that had a offset of ~6-7°C after 5! minutes...
There is a huge variation in the readings from device to device. But I noticed once you know
when the readings has stabilized over time its quite constant and you could use an offset. That offset is what I really miss with the official Seek App...

@undera can you please explain how you came to this line of code?
return (0.0171156038 * x + 37) - 273;
Where do you got the 0.0171156038 and the 37 from?
Did you got it from Excel as curve function?

@pauledd Yes, I took it from excel function.

@undera thanks, just one more question:
you state

// 330C => 603K => 32768 raw (known upper limit, full 15 bits - 2^15)

in your seek_viewer.cpp. Can you pls explain where you got this 15bits from?
As I was reading through the eevblog I've only seen that the cam outputs 14bits,
then stretched to uint16... how does the 15 come into play?

I don't remember exactly. In my experiments, I used candle as maxed temp value, and probably got this 32768 raw number.

ah, okay. I will try that too :)

Added more things to this PR:

added device temp sensor data getting to investigate further topic of absolute values getting (#29)

max/min position pointers (thanks @rainer38, but had to modify code to support rotation and scaling)

made seekpro to have 15fps by default
What do you mean with this? Isn't the Seek Pro restricted to ~9fps?

@pauledd Discrard what I've wrote there, I found I was confused

Now I also have added gradient on the side and central point measurement, hoping we'll collect some more data on raw values.

Values from app when camera self-temp is ~26C:
rmin,rmax,central,devtemp: 15060 15708 15298 6577 / min-max-center: 30.3 43.4 35.1
Same setup, camera cooled down to ~18C:
rmin,rmax,central,devtemp: 16384 17002 16932 6723 / min-max-center: 57.0 69.4 68.0
And after 10minutes:
rmin,rmax,central,devtemp: 14654 15584 15147 6582 / min-max-center: 22.2 40.9 32.1
Which makes me conclude that camera self-temperature highly relates to values measured. I'll look for a way to use devtemp plus linear 14bit value to 330C hypothesis together...

@undera Great work! However, from my Seek Compact, I read measurements like
rmin,rmax,central,devtempsns: 15556 15828 15628 4252	min-max-center-device: 890.3 895.8 891.8 591.7

. Did you do your research with a CompactXR or CompactPRO? I suspect there could be different formulae for each model.

I have only one CompactPro for Android device.

@undera Good to know. So I want to do experiments with a Seek Compact and share my findings.

Id did a series of calibration experiments. In the process, I wrote a viewer which stores the raw data in 16bit images for later processing. Unfortunately, finding a function to infer the actual device temperature from the raw value leaves me stumped. @undera How did you find the NTC information? Did you open your camera? I gave up and resorted to a linear approximation which seems to be exact within ±1°C after the device has warmed up. I published the adjustments in the temp-info_compact in my fork.

During my experiments, I also noticed some vignetting. After the device warmed up, the edges of an approximately uniform temperature surface appear 1.6 °C warmer than the centre.
Example: (normalized)

This  shows, the vignette increases with device temperature, too.

vignetting is a known thing for cameras, it is known that Seek uses software way to compensate it in their apps.
The problem of simple linear approach is that device's temperature will affect measurements, and I perfectly see that in my experiments.
My approach of making sure sensor has certain temperature is simple: I turn camera off in a room with air conditioner set to certain temp, I let it quite of time to stabilize temp, then I quickly turn it on and record values. I also use trustworthy thermometer to tell what was the temp around it (and inside it).

Thank you for your input.

(…) I turn camera off in a room with air conditioner set to certain temp (…)

Unfortunately, I currently have no options to produce environments with a range of stable temperatures wide enough to explore the non-linear parts of the assumed NTC reading. For the non-PRO variant of the Seek Compact, we are left with a linear approximation best fitting for a device temperature of whatever raw value of 4236 is standing for. Which is better than nothing, I guess.

Thats not practical at all if you want a quick location-independent measurement (…)

Fortunately, I want to do a long-term fixed-location measurement (for quick measurements on the go, I'd actually use the official app).

Hello,
Any updates on this topic ?
I have the feeling that the sensors values changed. I tried your code but I get tempertatures around 80 degree.
How do you get the values used in this fct ?
double device_sensor_to_k(double sensor) {
    // formula from http://aterlux.ru/article/ntcresistor-en
    double ref_temp = 297.0; // 23C from table
    double ref_sensor = 6616.0; // ref value from table
    double beta = 200; // best beta coef we've found
    double part3 = log(sensor) - log(ref_sensor);
    double parte = part3 / beta + 1.0 / ref_temp;
    return 1.0 / parte;
}

Thank you in advance

Nothing has changed. My impression is that people get different temperature because my formula is somehow wrong. But I have just one camera, so I have no way to validate this. Nobody brought any reliable measurements to compare, still.

Until we somehow find a reliable formula, you can resort to static calibration:

Let your camera run for a couple of minutes until it has warmed up (takes about 15 minutes, see below for a graph from the Seek Compact camera).
Store an image showing an object with a reference temperature.
Hope your camera does not change temperature during measurements.

I have an example application at my branch which stores the raw frames so you can do corrective temperature processing after the measurements.

@hoehermann - In step 2 above, don’t you need an image with two different reference temperatures (two objects with known temperatures) to calculate the linear relationship?
Or how do you do it using only one reference temperature?

@cjnqt I measured a plane of known temperature. As the device temperature changes, the reading changes. I calculated the offset of "reading should be" and "actual reading". Then I did a linear fit along a · device_temperature + b = corrective_offset. Sampling points in the example given above are any frames after frame 7000. This gives not only two, but a whole bunch of points to interpolate. Now I have a linear function I can apply to the measurements: measurement_value + corrective _function(device temperature) = corrected_value.

Feel free to take from this PR whatever code you want and close it. I have no spare time now to spend on it, sorry.

Would you mind flipping on the reviewers can edit setting so I can make those changes and merge it up. That way your contributions will get attributed correctly. If that's not possible to change after starting a PR I'll fork your branch and merge it under a separate PR. Thanks for all the work you put in here by the way.

Feel free to take from this PR whatever code you want and close it. I have no spare time now to spend on it, sorry.

The converting flux values to temperatures is not so easy task. Each sensor is different, so you will need to do calibration for each piece of camera separately. The conversion model is also not linear, but it should be done according to Planck's law (or its RBFO approximation) including thermal parameters of the viewed scene, such as emissivity, background temperature or atmospheric transmissivity.
There is also an issue in this library, because it normalizes flat-field to value 0x4000. This can be correct only in the situation when the shutter temperature is constant (and is always the same as it was during the calibration). The correct way would be the normalization to mean value of FFC image, or better, to some value that corresponds to real flux emitted by the shutter. Because if shutter temperature is known (e.g. derived from camera temperature) it can be used for radiometric correction (flux viewed on shutter => shutter temperature). You just need to include some coefficient to correlate between shutter image and scene image to compensate for lens influence.

The conversion model is also not linear, but it should be done according to Planck's law (or its RBFO approximation)

Unfortunately, I am not a physicist and cannot implement this approximation without spending considerable time on educating myself first. :/

The correct way would be the normalization (…) some value that corresponds to real flux emitted by the shutter.

I just noticed that the data is actually available in m_flat_field_calibration_frame, but not used anywhere. I currently do not have a thermal camera at hand, so I cannot experiment with it. :(

It does not work on OpenCV v4
I changed CV_AA => LINE_AA in https://github.com/undera/libseek-thermal/blob/temp-info/examples/seek_viewer.cpp#L70

It does not work on OpenCV v4
I changed CV_AA => LINE_AA in https://github.com/undera/libseek-thermal/blob/temp-info/examples/seek_viewer.cpp#L70

Are you talking about just this branch or master?

It does not work on OpenCV v4
I changed CV_AA => LINE_AA in https://github.com/undera/libseek-thermal/blob/temp-info/examples/seek_viewer.cpp#L70

Are you talking about just this branch or master?

only this branch.
master branch works for me :-)

This branch reports values like rmin,rmax,central,devtempsns: 15687 15973 15751 7291	min-max-center-device: -31.4 -25.7 -30.2 -13.4 with my seek thermal pro  (after makeing the CV_AA->LINE_AA changes above.)  It's in a "normal" office space, about 22C or so, pointing at a board that might be 25-30 tops, but either way, nothing is negative here :)
So it seems more info is required?  Still, super neat that this all "just worked" for the most part :)

This forum and the corresponding Rpi2SeekDevA code suggest that the temperature profiles, calibration info etc can be sourced via specific pixels.  https://www.eevblog.com/forum/thermal-imaging/running-seek-thermal-cameras-(-others)-on-a-raspberry-pi/


Error: "Use of const in strict mode." when doing a release build

[X] I have reviewed the documentation
[X] I have searched existing issues
[X] I am using the latest React Native version

I followed the guide here:
https://facebook.github.io/react-native/docs/running-on-device.html
to prepare my app for the appstore (iOS)
When I create a build, it will fail with this message:
node /myapp-ios.git/node_modules/react-native/local-cli/cli.js bundle --entry-file index.js --platform ios --dev false --reset-cache --bundle-output /Users/myuser/Library/Developer/Xcode/DerivedData/myapp-dfraoaivwzsccuavfyudbsliqfwd/Build/Products/Release-iphoneos/myapp.app/main.jsbundle --assets-dest /Users/myuser/Library/Developer/Xcode/DerivedData/myapp-dfraoaivwzsccuavfyudbsliqfwd/Build/Products/Release-iphoneos/myapp.app

/myapp-ios.git/node_modules/metro-core/src/formatBanner/index.js:13
const formatBanner = require('./formatBanner');
^^^^^
SyntaxError: Use of const in strict mode.
    at Module._compile (module.js:439:25)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/Users/myuser/Documents/Development/myapp/myapp-ios.git/node_modules/react-native/local-cli/server/checkNodeVersion.js:10:20)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
+ [[ false != true ]]
+ [[ ! -f /Users/myuser/Library/Developer/Xcode/DerivedData/myapp-dfraoaivwzsccuavfyudbsliqfwd/Build/Products/Release-iphoneos/myapp.app/main.jsbundle ]]
+ echo 'error: File /Users/myuser/Library/Developer/Xcode/DerivedData/myapp-dfraoaivwzsccuavfyudbsliqfwd/Build/Products/Release-iphoneos/myapp.app/main.jsbundle does not exist. This must be a bug with'
error: File /Users/myuser/Library/Developer/Xcode/DerivedData/myapp-dfraoaivwzsccuavfyudbsliqfwd/Build/Products/Release-iphoneos/myapp.app/main.jsbundle does not exist. This must be a bug with
+ echo 'React Native, please report it here: https://github.com/facebook/react-native/issues'
React Native, please report it here: https://github.com/facebook/react-native/issues
+ exit 2

``

## Environment

Environment:
  OS: macOS High Sierra 10.13.4
  Node: 8.4.0
  Yarn: 1.5.1
  npm: 5.6.0
  Watchman: 4.9.0
  Xcode: Xcode 9.3 Build version 9E145
  Android Studio: 3.0 AI-171.4443003

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: 0.55.3 => 0.55.3

## Steps to Reproduce

Just building it with the settings above will cause this.

## Expected Behavior

It would build with the combined file.

## Actual Behavior

It does not build with the error message above.

This answer on Stackoverflow helped me:
https://stackoverflow.com/a/48431161/690771
I had to set another path to "node". When testing on CLI, the node version was correct; however it seems something inside the React environment would take the wrong node. Not sure where this is coming from.


I can't run this one in android

I am checking it on android platform now.
But unfortunately, this one doesn't seems to work well.
So Is it possible for me to use on android platform?
Thank you for your time.

same here, when i tap on the ModalPicker, there are no response at all, no warning/error

any update?

work around is https://github.com/d-a-n/react-native-modal-picker/issues/28


Add apache license

This PR adds the Apache 2 license, so that we register this package in order to smooth the ClimaCoupler workflow.

bors r+


Add zNotch Extension

Description
New extension znotch to manage macbook's notch visibility directly from Raycast. Allows explicitly to hide and show the notch, as well as toggle to an opposite state.
Screencast
Since macOS doesn't provide any option to record laptop's screen's notch, the overall effect looks this way:

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Not sure what's happened with this PR, but shows that every single file has changed. The review page wont even load.

Could you have a look? Thanks!

Not sure what's happened with this PR, but shows that every single file has changed. The review page wont even load.

It's because @zkondor is attempting to merge directly into main. You need to:

Stash your PR changes somewhere.
Reset your fork to https://github.com/raycast/extensions
Create a new branch for your PR
Apply the changes to the new branch.
Re-create the PR.

I don't think that's the real problem @nhojb - we should make PR to the main branch - it's correct.
The big problem is that on the fork @zkondor removed everything - all extensions and every single file that was in the repository - instead of just adding his extensions.
You can see it by checking his fork content.

I pushed the fix, but in case you need to do it in the future, here is how to do it easily:
//clone repository (it will connect to upstream automatically)
gh repo clone zkondor/raycast-extensions
//FINDER: copy extension folder before next steps
//fetch latest changes from our public repository
git fetch upstream
//reset fork to public repository state
git reset --hard upstream/main
// now you have the same state as our public repository, force push changes to fork as we overwrite history
git push origin main --force
// FINDER: re-add the extension
git add .
git commit -m "Add zNotch Extension"

I tried to fix the problem, but somehow I was able to force push to it the reset, but I'm not allowed to add the extension back to the repository 🤯


cilium-dbg: Reprint header line periodically with statedb

When watching a table for changes it's easy to get lost with what the columns meant. Add a periodic (every 100 lines) reprinting of the header line to make it easier to follow.
As this makes it easier to mix up the header line with the rows, add a '# ' to the front.
Output of cilium-dbg statedb health --watch=100ms looks like this now:
  ...
  agent.datapath.agent-liveness-updater            timer-job-agent-liveness-updater               OK         ...
  # Module                                         Component                                      Level      ...
  agent.datapath.agent-liveness-updater            timer-job-agent-liveness-updater               OK         ...
  ..

/test

/ci-integration


[BUG] - fvm 2.4.1 doesn't support Dart 2.10.5.

I am on MacOs 13.2.1
VSCode 1.77.0
Describtion
I have an old project which uses flutter 1.22.6
So far I got running by fvm flutter run in VSCode terminal. But recently I got the following error when I try to run any command starting with fvm in this project directory in the VSCode terminal.
 fvm --version
Can't load Kernel binary: Invalid kernel binary format version.
fvm 2.4.1 doesn't support Dart 2.10.5.

I run repair and reactivate fvm. In the other projects all fvm commands execute well as expected but in this old project throwing this error message.
I tried to run fvm flutter run out of the VSCode terminal it runs well.
What is wrong with fvm when running it in the VSCode terminal where opened project set with old flutter version? In my case Flutter 1.22.6.
Logs
Can't load Kernel binary: Invalid kernel binary format version.
fvm 2.4.1 doesn't support Dart 2.10.5.
Desktop (please complete the following information):

OS: [iOS 13.2.1]
FVM Version [2.4.1]

Additional context
Here my setings.json
"dart.flutterSdkPath": ".fvm/flutter_sdk",
"dart.flutterSdkPaths": ["/Users/vfiruz/fvm/versions"],
// Remove .fvm files from search
"search.exclude": {
 "**/.fvm": true
},
// Remove from file watching
"files.watcherExclude": {
 "**/.fvm": true
},

use an old version of fvm that is compatible

Take a look at the standalone installation to see if this solves the problem for you
https://fvm.app/docs/getting_started/installation#standalone

Hi, I am currently facing the same issue. I want to ask, what is the best approach to getting an FVM version that works with Flutter 2 projects? Thanks.

Hi, I am currently facing the same issue. I want to ask, what is the best approach to getting an FVM version that works with Flutter 2 projects? Thanks.您好，我目前面临同样的问题。我想问，获得适用于 Flutter 2 项目的 FVM 版本的最佳方法是什么？谢谢。

choco uninstall fvm
choco install fvm --version=2.3.1
flutter pub global activate fvm
fvm use 2.10.5


Installation of v2024.12.19 broken for macos x64

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
amd-emcee0:x-ui-tests-dmitriyg jenkins$ curl https://mise.run | sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  9054  100  9054    0     0  88684      0 --:--:-- --:--:-- --:--:-- 88764
mise: installing mise...
> curl -#fLo /var/folders/05/3j34cr554xl8c1rfv6r19k4r0000gq/T/tmp.lEzFjiVr7m/mise-v2024.12.19-macos-x64.tar.zst https://github.com/jdx/mise/releases/download/v2024.12.19/mise-v2024.12.19-macos-x64.tar.zst
######################################################################## 100.0%
mise-setup: tarball=/var/folders/05/3j34cr554xl8c1rfv6r19k4r0000gq/T/tmp.lEzFjiVr7m/mise-v2024.12.19-macos-x64.tar.zst
validating checksum
sh: line 147: warn: command not found
shasum: standard input: no properly formatted SHA checksum lines found

Expected behavior
Installation script is working well
Additional context
Seems like this commit broke installation. because there is no option for x64 arch and tar.zst extention

if [ "$(get_ext)" = "tar.zst" ]; then
...
elif [ "$os" = "macos" ]; then
        if [ "$arch" = "arm64" ]; then
          echo "$checksum_macos_arm64_zstd"
        else
          warn "no checksum for $os-$arch"
        fi

Is it expected behavior?
Here we have mise-v2024.12.19-macos-x64.tar.zst checksum.

Same issue here. Users installing mise via the installation script in Xcode Cloud environments are experiencing CI failures, but these can be prevented by using Homebrew for installation.


Initial config version, basic parameters handled

Can one of the admins verify this patch?

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build


Node Affinity NodeSelectorTerms description is wrong

If you specify multiple nodeSelectorTerms associated with nodeAffinity types, then the pod can be scheduled onto a node if one of the nodeSelectorTerms can be satisfied.

If you specify multiple matchExpressions associated with nodeSelectorTerms, then the pod can be scheduled onto a node *only if all matchExpressions is satisfied.

Should be,
If you specify multiple matchExpressions associated with nodeSelectorTerms, then the pod can be scheduled onto a node if one of matchExpressions can be satisfied.
If you specify multiple node label requirements associated with matchExpressions types, then the pod can be scheduled onto a node only if all requirements can be satisfied.

Hi @dancnfoo
What's the URL of the page that you don't think is correct?
/triage needs-information
/language en

@sftim i guess this https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/

https://v1-20.docs.kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#node-affinity

Can someone confirm this issue has been triaged?

/triage accepted

/remove-lifecycle stale

/assign

#37991 also tried to "fix" this. But I think there is nothing to fix.

Here is my attempt #38032, but @cortespao is suggesting that it's better to add samples. Is anyone willing to take this over?


Flutter does not refresh UI contents immediately when resize FlutterView

Steps to Reproduce

Add FlutterView as a subview of Activity's contentView
Continuously resize FlutterView
FlutterView will has some black space, and refresh contents for new size after some delay, then contents occupies all FlutterView size.

example code:
        View flutterView = Flutter.createView(
                MainActivity.this,
                getLifecycle(),
                "/"
        );
        this.mFrameLayout.addView(flutterView);
        SeekBar seekBar = findViewById(R.id.seekbar);
        seekBar.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {
            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
                ViewGroup.LayoutParams lp = mFrameLayout.getLayoutParams();
                lp.width = 8 * progress;
                lp.height = 16 * progress;
                mFrameLayout.setLayoutParams(lp);
            }

            @Override
            public void onStartTrackingTouch(SeekBar seekBar) {

            }

            @Override
            public void onStopTrackingTouch(SeekBar seekBar) {

            }
        });

Screen capture:

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.0.0, on Mac OS X 10.14.1 18B75, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
[!] iOS toolchain - develop for iOS devices (Xcode 10.1)
    ✗ libimobiledevice and ideviceinstaller are not installed. To install with Brew, run:
        brew update
        brew install --HEAD usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    ✗ ios-deploy not installed. To install with Brew:
        brew install ios-deploy
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
        For more info, see https://flutter.io/platform-plugins
      To install:
        brew install cocoapods
        pod setup
[✓] Android Studio (version 3.1)
[!] IntelliJ IDEA Community Edition (version 2018.1.1)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] VS Code (version 1.30.0)
[✓] Connected device (1 available)

After reading codes, I got the point.
Resize events are posted from main thread, handled at UI Task Runner (another thread for embedder), with help of message loop.
The delayed time is between posted timestamp and handled timestamp.
Still looking for some methods to handle or decrease the delay.

@loongliu
Can you create a github demo project which can help us reproduce this issue?

@kangwang1988 It is simple to reproduce this.
Follow this instruction Add Flutter to existing apps.
Create a demo android project contains flutter module.
Change the MainActivity with following code.
public class MainActivity extends AppCompatActivity {
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        FrameLayout view = new FrameLayout(this);
        ViewGroup.LayoutParams lp = new ViewGroup.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT);
        view.setLayoutParams(lp);

        final FrameLayout frameLayout = new FrameLayout(this);
        FrameLayout.LayoutParams flp = new FrameLayout.LayoutParams(400, 600);
        flp.gravity = Gravity.BOTTOM;
        frameLayout.setLayoutParams(flp);
        view.addView(frameLayout);

        View flutterView = Flutter.createView( MainActivity.this, getLifecycle(), "route1" );
        frameLayout.addView(flutterView);
        SeekBar seekBar = new SeekBar(this);
        seekBar.setLayoutParams(new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, 100));
        view.addView(seekBar);
        seekBar.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {
            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
                ViewGroup.LayoutParams lp = frameLayout.getLayoutParams();
                lp.width = 8 * progress;
                lp.height = 16 * progress;
                frameLayout.setLayoutParams(lp);
            }
            @Override
            public void onStartTrackingTouch(SeekBar seekBar) {   }
            @Override
            public void onStopTrackingTouch(SeekBar seekBar) {   }
        });

        setContentView(view);

    }
}

Then, drag the seekbar.

@loongliu I tried to use to code sample to replicate this issue, but didn't see the seekbar you mentioned, so am not sure if this is still a valid issue using latest Flutter version. Can you confirm if this has been resolved or issue persists ?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Restangular tests not working

Hello,
I cloned the repository and then npm install && npm test. It has failed without launching a single test.
What could be the problem ?

Hi @charlouze, Thanks for the response. It's fixed now.


irmin: fix fold for non-persisted, cleared trees

I noticed that some calls to cached_hash and cached_value were missing too:
https://github.com/mirage/irmin/commit/c53cb671cd96f45d4a80f4e87146682db8f7b7af
Do we want to apply that diff too?

I made 2 commits to fix the issues I raised above. I can push those here or open a second PR.
https://github.com/Ngoguey42/irmin/commits/more-cached-calls


Dig error as command missing when installed and in PATH for user on Debian

Followup from https://github.com/ZmnSCPxj/clboss/pull/52
I'm running clboss master at 18d1a09e9ff52b31df79347cd42b96b3ed79891a on Debian GNU/Linux bullseye/sid and see the follwing in the logs at startup:
... UNUSUAL plugin-clboss: DnsSeed: Cannot seed by DNS: `dig` command is not installed, it is usually available in a `dnsutils` package for your OS.

I run lightningd as user bitcoin who has dig installed and available:
bitcoin in ~/.lightning at will-nuc ➜ echo $SHELL
/usr/bin/fish

bitcoin in ~/.lightning at will-nuc ➜ whoami
bitcoin

bitcoin in ~/.lightning at will-nuc ➜ which dig
/usr/bin/dig

bitcoin in ~/.lightning at will-nuc ➜ dig -v; echo $status
DiG 9.16.8-Debian
0

Everything seems to be working, but odd error message I can't seem to get rid of nonetheless...

Can you check which dig result?  Also, try something like:
`which dig` -v

Also:
`sudo -u bitcoin dig -v
 sudo -u bitcoin which dig

which dig actually causes some issues for me, as I'm using fish shell (which I think does not have the which command anymore, at least I have an abbreviation (like an alias) saved to command -v)...
command -v dig though (run as will, bitcoin or su) returns dig correctly, as does sudo -u bitcoin dig -v, so it's certainly in the same (dfefault) path for everyone:
/usr/bin/dig

Now I know this lookssuspiciously like my shell might be interfering, however we are never calling which dig ever AFAIK, so I don't think this should be an issue:
Ev::Io<std::string> can_get() {
	return Ev::runcmd("dig", {"-v"}).then([](std::string _) {
		return Ev::lift(std::string(""));
	}).catching<std::runtime_error>([](std::runtime_error const& _) {
		return Ev::lift(std::string(
			"`dig` command is not installed, it is usually "
			"available in a `dnsutils` package for your OS."
		));
	});
}

I applied the patch manually on top of master, which revealed the below, but did not raise the dig error subsequently:
2022-04-26T14:14:54.844Z INFO    plugin-clboss: PATH = /home/bitcoin/.pyenv/versions/lightningd-3.8.1/bin:$PATH
2022-04-26T14:15:05.044Z INFO    plugin-clboss: clboss 0.11E
2022-04-26T14:15:05.045Z INFO    plugin-clboss: PeerComplaintsDesk: Auto-close: enabled.
2022-04-26T14:15:05.045Z INFO    plugin-clboss: AmountSettingsHandler: Onchain reserve set by --clboss-min-onchain to 2000000 satoshis.
2022-04-26T14:15:05.045Z INFO    plugin-clboss: ChannelFeeManager: zerobasefee: allow
2022-04-26T14:15:05.138Z INFO    plugin-clboss: New block at 733645
2022-04-26T14:15:05.138Z INFO    plugin-clboss: Started.
2022-04-26T14:15:05.524Z INFO    plugin-clboss: InternetConnectionMonitor: online.
2022-04-26T14:15:05.526Z INFO    plugin-clboss: New block at 733646
2022-04-26T14:15:10.823Z INFO    plugin-clboss: PeerCompetitorFeeMonitor: Surveying 02113fba7b4a54068a335b2a042fd889138f6eb3c791eea6435e144cde90409d47 progress: 9/55 (0.163636)
...

Reverting the patch (back to master @ f4f0c80) instantly causes the error again:
2022-04-26T14:18:06.931Z UNUSUAL plugin-clboss: DnsSeed: Cannot seed by DNS: `dig` command is not installed, it is usually available in a `dnsutils` package for your OS.

Now I am even more confused :P

I applied the patch manually on top of master, which revealed the below, but did not raise the dig error subsequently:

wut.

2022-04-26T14:14:54.844Z INFO    plugin-clboss: PATH = /home/bitcoin/.pyenv/versions/lightningd-3.8.1/bin:$PATH

That is one helluva weird $PATH.  You expect only a "pure" PATH, one which has only :-separated directory names, and not a self-referential thing like that.  A $PATH here means something in the shell or in the invocation somewhere did not expand the previous $PATH fully, which might be the root cause of your problem.  Looks like shell shenanigans, maybe your shell startup script had too many escapes when it set the PATH environment variable?  With getenv("PATH") that broken, it looks like execvp is failing because of this?

I think you're correct. Perhaps fish shell is evaluating the path later, or something I don't understand. Appears it shall remain a mystery anyway...

Hmmm.  fish says it is an "interactive" shell.  Maybe you have a different login shell vs system shell, and some files you changed for fish may also be in use by the system shell, which got confused due to fish-specific syntax?  How do you start lightningd?  Do you manage it with crontab, systemd, or something or other?  What is your /bin/sh?  Is it a link to fish or is it some other binary?

Yep the issue was in systemd:
/etc/system/systemd/lightningd.service
[Service]
Environment="PATH=/home/bitcoin/.pyenv/versions/lightningd-3.8.1/bin:$PATH"
Environment="PYENV_VERSION=lightningd-3.8.1"
RuntimeDirectory="lightningd"
ExecStart=/usr/local/bin/lightningd --lightning-dir=/home/bitcoin/.lightning

Apparently Environment in a unit file does not expand env vars? Who knew (not me!)
Manually hardcoding the PATH seems to work, however I fixed my pyenv setup so I can remove the PATH environment line altogether, and now CLBOSS works flawlessly without that pesky Dig error!
Thanks for the help, even though it didn't turn out to stricly be related to CLBOSS :)

Okay, glad that was helpful!


feat: enable SSL for production

Description
Enable SSL configuration
the solution might be to be able to pass a SecurityContext to the serve function in serve.dart.
Requirements
I think that should be configurable in CLI commands for build production only.
dart_frog build --cert  --key
server port should be 443 automatically
Additional Context
the feature would securise communication between the reverseproxy and the dart_frog VM. (required by some companies policy)

Since the release of v0.3.5 you can now manage the security context yourself. Does this solve your use case?

Closing this issue as you can solve it with what I mentioned above, if you have any other questions please open a new issue <3


Readonly dashboard authors

as an enhancement, there would be great to limit user's access to sections entries,
per section or globally; as is implemented in django admin.

There is an extension which does take care of Author roles, I'm aware it might be due some love. If you're in urgent need for this ping me within the coming weeks, to see your exact needs and if the extension covers your needs as is.

I've updated Author Roles and there is a pull request in discussion at the moment to fix a little bug. I have it working on 2.6.4 locally.

I've updated Author Roles and there is a pull request in discussion at the moment to fix a little bug. I have it working on 2.6.4 locally.

That's great @andrewminton ! I'm about to update an older site that makes heavy use of this extension and wasn't sure if that would work. Great to know it does :)
On another note: I recently contacted Twisted Interactive regarding maintenance/ownership of one of their long unmaintained extensions (reflected upload field) and they told me they'd be ok with transferring ownership to me (though it hasn't happened yet - will have to ask again next week).
I also proposed to transfer ownership of their two most popular (and also unmaintained) extensions author roles and import csv to the symphonists-account - but maybe you want to jump in regarding author roles if you're actively using & maintaining it?

@twiro Maybe mention Import CSV on Gitter because me and @animaux were just using Remote DS and XML Importer.  I'm a bit behind on my job and didn't get everything working yet but I think the integration branch of XML Importer now does everything Import CSV did.  Maybe Import CSV only needs legacy support? @animaux might confirm better.

@cmnstmntmn Some nice people will open source a solution I've worked with them that will solve this.
We just need to release some new delegates and this could only be done for 2.7.x ... stay tune!

to the symphonists-account

@twiro I've ask them too, but never got an answer... But I think it would be better to move them under your name ;)

hah @nitriques looking forward to see what delegates you're adding I've
added a delegate before for membership management let me know when you add
the PR as I might have something different I'd like to add.
On Fri, 22 Jan 2016 at 19:26 Nicolas Brassard<EMAIL_ADDRESS>wrote:

@cmnstmntmn https://github.com/cmnstmntmn Some nice people will open
source a solution I've worked with them that will solve this.
We just need to release some new delegates
https://github.com/DeuxHuitHuit/symphony-2/tree/authors-del and this
could only be done for 2.7.x ... stay tune!
to the symphonists-account
@twiro https://github.com/twiro I've ask them too, but never got an
answer... But I think it would be better to move them under your name ;)
—
Reply to this email directly or view it on GitHub
https://github.com/symphonycms/symphony-2/issues/2542#issuecomment-174000417
.

@jonmifsud Perfect!

just great, great, great!
On Fri, Jan 22, 2016 at 8:31 PM, Nicolas Brassard<EMAIL_ADDRESS>wrote:

@jonmifsud https://github.com/jonmifsud Perfect!
—
Reply to this email directly or view it on GitHub
https://github.com/symphonycms/symphony-2/issues/2542#issuecomment-174003057
.

:D It's section based AND entries based (meaning some users only see and subset of all entries)

same here I had something like that... but interested to see what you came
up with :)
On Fri, 22 Jan 2016 at 22:15 Nicolas Brassard<EMAIL_ADDRESS>wrote:

:D It's section based AND entries based (meaning some users only see and
subset of all entries)
—
Reply to this email directly or view it on GitHub
https://github.com/symphonycms/symphony-2/issues/2542#issuecomment-174051352
.

This as been done in #2686


Parser does not read non ascii characters : ŞÜİĞ

Description
The parser does not read the following characters :
{
    "4bcfadb7-c345-474c-aaaf-65734b14f1e6": {
        "fps": 24,
        "text": {
            "4e50c16f-a944-4adc-9d78-74cca4ba1cb9": {
                "alpha": 1,
                "groups": "4e50c16f-a944-4adc-9d78-74cca4ba1cb9",
                "context": "A ŞĞÜİ B",

When I try to read the context field, the output is simply "A      B", and the rest of the characters seems as they never existed in the first place.
I am using the following command to read the json field :
text = new std::string(txt["context"].getstd::string());`
What shall I do in order to read the non ascii characters ?
Reproduction steps

Create a non ascii character json field in your json structure.
Try to read the json field.
You can see that ascii characters are not read.

Expected vs. actual results
Expected :
The non ascii characters should be read.
Actual results:
The ascii characters seems to be omitted under all circumstances.
Minimal code example
text = new std::string(txt["context"].get<std::string>());

Error messages
There are no error messages.

The json parser just skips the non ascii characters as empty spaces.

Compiler and operating system
linux, debian, GCC 4.8 - 12.0
Library version
JSON for Modern C++ version 3.11.1
Validation

[ ] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

See https://json.nlohmann.me/home/faq/#parse-errors-reading-non-ascii-characters

Ok.
How about reading from a json ?
text = new std::wstring(j["context"].getstd::wstring());
Would this ever work ?
It says expected string but recevied an array of numbers.
What shall I do here ?

What does this exactly mean ?
std::u16string and std::u32string can be parsed, assuming UTF-16 and UTF-32 encoding, respectively. These encodings are not supported when reading from files or other input containers.
Does it mean that, we need to first read the context of a json file into json, then read from that source with u16string ?

@nlohmann ,
Any suggestions here ?
This is the code that I have been using :
std::u16string* text;
try{ text = new std::u16string(txt["context"].getstd::u16string()); }  catch(json::type_error& te){ VI_FATAL("{}, {}, {}", te.what(), FILE, LINE); exit(0); }
And this is the error that I have gotten :
[14:51:47] APP: [json.exception.type_error.302] type must be array, but is string, /app/src/coms/json_parser.cpp, 160
And here is the json :
{
"4bcfadb7-c345-474c-aaaf-65734b14f1e6": {
"fps": 24,
"text": {
"4e50c16f-a944-4adc-9d78-74cca4ba1cb9": {
"alpha": 1,
"groups": "4e50c16f-a944-4adc-9d78-74cca4ba1cb9",
"context": "A ŞĞÜİ B",
            ...

The library only supports UTF-8. Other encodings will likely be treated as errors by the parser. Can you provide a minimal working example?

This is the json :
{
"4bcfadb7-c345-474c-aaaf-65734b14f1e6": {
"fps": 24,
"text": {
"4e50c16f-a944-4adc-9d78-74cca4ba1cb9": {
"alpha": 1,
"groups": "4e50c16f-a944-4adc-9d78-74cca4ba1cb9",
"context": "A ŞĞÜİ B",
}
}
}
}
This is how I read the json file :
            std::ifstream project_json_input(manifest_file);
            Json = json::parse(project_json_input);

And this is how I read the field
try {
    text = new std::wstring(txt["context"].get<std::wstring>());
}  catch(json::type_error& te) {
    VI_FATAL("{}, {}, {}", te.what(), __FILE__, __LINE__); exit(0);
}

I have accepted the reality of  the library only supports UTF8 :
However, now I have found a solution as below :
The solution is to convert the wstring to utf8 number encoding, and then convert it back to wstring in backend.

Can you please add the JSON file as is - the encoding is important here.
Strings are stored as std::string internally. Assuming txt["context"] is valid and returns a basic_json&, then get<std::wstring>() is equivalent to a static_cast<std::wstring>(...) - that behavior (and whatever exceptions come out of this) is out of scope of this library.


Make extra changes from document

What does this PR do?

Make small UI changes from the feedback document

Description of Task to be completed?

Fix alignment in the about page
Fix the find out more button in Safari
Add descriptions to meet the team
Add links and hover to our partner's cards
Fix alignment in our partners

Looks good @puritybirir 👍. Lets get the below sorted though:

Make the whole partner card clickable,

Sort stories by date,

Adjust height so that date is visible for all stories,

Can we remove the white space below cards.

This is resolved


CSR Attributes / SET OF

RFC says SET OF, but our implementation uses SET. Needs fixing.
Also, please help my understand whether my implementation (the sorting of SET and SET OF) is actually spec compliant.

also the property needs to be renamed to attributes

also the property needs to be renamed to attributes

done

This is ridiculous. There won't ever be any sorting, because the tags are all the same and we definitely don't want to sort when we parse.


Update index.ipynb to create passable README

Passable!  Generated from index.ipynb.  Using the fast.ai one as a template.
Needs edits of course.

And credit for github kanban-style project! Thanks.
On Dec 24, 2020, at 3:48 PM, Charles Twardy<EMAIL_ADDRESS>wrote:

Closed #3https://urldefense.com/v3/__https://github.com/cmorris-jacobs/jacobs-vault/issues/3__;!!B5cixuoO7ltTeg!Upbkpk6pf4-2MhkRyy1QkqNY3XkIs4WrR4lugB2gJV5nNzg-K7wJvsGYA35p2HPNMfo$.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/cmorris-jacobs/jacobs-vault/issues/3*event-4148064358__;Iw!!B5cixuoO7ltTeg!Upbkpk6pf4-2MhkRyy1QkqNY3XkIs4WrR4lugB2gJV5nNzg-K7wJvsGYA35pPxhtyts$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ASFTCRYXKPZ25ODVWLRIQHLSWOZMBANCNFSM4VIMI4SQ__;!!B5cixuoO7ltTeg!Upbkpk6pf4-2MhkRyy1QkqNY3XkIs4WrR4lugB2gJV5nNzg-K7wJvsGYA35pzKqGRLw$.

NOTICE - This communication may contain confidential and privileged information that is for the sole use of the intended recipient. Any viewing, copying or distribution of, or reliance on this message by unintended recipients is strictly prohibited. If you have received this message in error, please notify us immediately by replying to the message and deleting it from your computer.


Exception not handled by on error event

If file watched deleted, node will exit with exception and can't be catched outside
fs.js:954
  return binding.stat(pathModule._makeLong(path));
                 ^

Error: ENOENT: no such file or directory, stat '/tmp/1234.csv'
    at Object.fs.statSync (fs.js:954:18)
    at Tail.watchEvent (/opt/JAICAS/node_modules/tail/lib/tail.js:123:18)
    at FSWatcher.<anonymous> (/opt/JAICAS/node_modules/tail/lib/tail.js:105:24)
    at emitTwo (events.js:106:13)
    at FSWatcher.emit (events.js:192:7)
    at FSEvent.FSWatcher._handle.onchange (fs.js:1367:12)

This is caused by
https://github.com/lucagrulla/node-tail/blob/master/src/tail.coffee#L62
you can simply fix it with try/cache or check file exist before fs.statSync(this.filename);

Thanks for finding this.
I'd say that between the 2 options you suggested check file existence is neater.
Are you planning to send a PR with the fix?

fixed with release 1.2.3


Add link to Quantum Computing Stack Exchange so questions may be dire…

…cted there instead of the issue tracker.

I'm okay with this but since OpenFermion core does not explicitly depend on cirq, perhaps we should not mention cirq here.


Suggestion: Setting for auto start

Would like an option, maybe even default, to autostart with the computer/windows.

That's not part of the tool, you can do this from your operating system. So I let MacOS start it automatically after login which should be possible to do on Windows too.

It's usually part of settings of the program on windows and sometimes an option in the installer.
Unfortunately there is no convenient way on windows (you have to manually locate the windows startup system folder and create a shortcut to the exe file there).
https://support.microsoft.com/en-us/windows/add-an-app-to-run-automatically-at-startup-in-windows-10-150da165-dcd9-7230-517b-cf3c295d89dd
Not that important though, but that is usually how apps using the app tray works.

Unfortunately there is no way to add that from within JavaFX


Figure out root component

[ ] Where should the root component be located?
[ ] How should the root component be named?
[ ] What does the root component render?

https://github.com/BuilderIO/qwik/commit/ab9c1eb651812be03bc7a052fe8124659565c41e


Retain original function signature to avoid breaking changes

Let's say you're a library maintainer. If you want to wrap a function with throws, this introduces a breaking change since the return type is changed to include the CatchEnforcer.
I think we should instead add a .try() to the original function. For example:
const someFunction = throws((arg: string) => {
  if (!arg) throw new SomeError();

  return `hello ${arg}`;
}, { SomeError });

someFunction
  .try('kyle')
  .catchSomeError(err => { /* ... */ });

This makes it so any existing consumers of someFunction don't receive breaking changes. They can opt into better error handling, making it easier for both maintainers and consumers

I've realized this goes against the original goal of adding enforced error checking. I do think something like this is very valuable in regards to adoption, but I also see this not being desirable in some use-cases.
Is there a way we can allow for an opt-in, as well as enforcing checking? For example:
const forced = throws((arg: string) => {
  if (!arg) throw new SomeError();

  return `hello ${arg}`;
}, { SomeError });

forced('kyle')
  .catchSomeError(err => { /* ... */ });

const notForced = throws.allowUnsafe((arg: string) => {
  if (!arg) throw new SomeError();

  return `hello ${arg}`;
}, { SomeError });

notForced
  .try('kyle')
  .catchSomeError(err => { /* ... */ });

// Allows skipping of error catching
notForced('kyle')

Now that I'm thinking about it, it seems this would be the ideal scenario:
import { throws, throwsUnsafe } from 'ts-throws';

// by default, `throws` does add `.try`
const func = throws((arg: string) => {
  if (!arg) throw new SomeError();

  return `hello ${arg}`;
}, { SomeError });

func
  .try('kyle')
  .catchSomeError(err => { /* ... */ });

// Allows skipping of error catching
func('kyle') // -> "hello kyle"

// `throwsUnsafe` means that the function is not safe to call directly
const funcWithForcedChecking = throwsUnsafe((arg: string) => {
  if (!arg) throw new SomeError();

  return `hello ${arg}`;
}, { SomeError });

funcWithForcedChecking
  .try('kyle')
  .catchSomeError(err => { /* ... */ });

// Does NOT skipping of error catching
funcWithForcedChecking('kyle') // TS error

The prominent use case is "I want to document my errors and make consuming them more convenient", but we still support "I have a dangerous function that needs to be handled properly"

Implemented in #12 at version 2.0.0


Please whitelist repository "llvm-toolchain-xenial" for Clang >=9 on Ubuntu xenial

Hi!
llvm-toolchain-trusty (without a number) for Clang 9 on Ubuntu trusty is there, but llvm-toolchain-xenial (without a number) for Clang 9 on Ubuntu xenial — as I read https://apt.llvm.org/ — is still disallowed.
Any chance it could be added?
Thanks!
Related: #392 #399

Any news?

There's a pull request for review at #408 now. Thank you!

Could you please also whitelist llvm-toolchain-xenial-9 ?

Almost time to whitelist the version 10! Why is this still pending?

It makes me sad that this trivial-to-fix issue is still open 10 months later. What does it take to get this fixed?

Closing, does not seem relevant any more


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

terminado .. Falta agregar mas coceptos de CSS


Cryptic "Invalid Value" response

This may be the wrong place to post this, but I'm having difficulty reaching anyone familiar with google apis on either android-developers or google-apis-explorer-users forums.
I'm having a problem deciphering the root cause of a cryptic "invalid value" response for the androidpublisher.inapppurchases.get api. OAuth is working fine, and I'm able to get the access_token as well as refresh_token.
I'm guessing something in the parameters are malformed or missing but cannot decipher where. The dev console project seems to be connected correctly because for an unauthorized user, querying on my packageName throws a permission denied error.
I also suspected the productId may be malformed, and I tried all sorts of combinations such as subs:{packageName}:pro_monthly_15, {packageName}:pro_monthly_15, etc. with all the same invalid response.
Can anyone help me shed some light on this?
oauth2Client.credentials = {
            access_token: '{removed}',
            refresh_token: '{removed}'
        };

        googleapis
            .discover('androidpublisher', 'v1.1')
            .execute(function(err, client) {

                console.log(['discovered', err, client.androidpublisher.apiMeta]);

            var params = {
                packageName: '{removed}',
                productId:   'pro_monthly_15',
                token:       'glhannifclifbhdgbpalegib.AO-J1OyeEpe0JagpGtG588_Jor3mtqjp_CRB-xGdq55kqMMWqGyGd2YlesHdazWPnOC4CoB0EVP-o_j1LT7taDJE8vUxg7UcjzeMPZ4WHi79aTYdv3FalrvqKAFTvWZqJqwjecdGaTpa'
            };
            client
                .androidpublisher.inapppurchases.get(params)
                .withAuthClient(oauth2Client)
                .execute(function (err, response) {
                    console.log([err, response]);

                    res.json({
                        response: response,
                        error: err
                    });
            });
        });

Cryptic response:
{
  "response": null,
  "error": {
    "errors": [
      {
        "domain": "global",
        "reason": "invalid",
        "message": "Invalid Value"
      }
    ],
    "code": 400,
    "message": "Invalid Value"
  }
}

created new CX value (new engine) it ran well.

Passing a wrong token to params will give you this error too. In my case I was passing a token which was just too short.


edge: handle addStream after setRemoteDescription

fixes https://github.com/webrtc/adapter/issues/514
The rtpSender was not created when the client code was doing this:
pc.setRemoteDescription(...)
.then(() => {
  pc.addStream(...);
  return pc.createAnswer();
})

(patch version after this please)

It seems that it is breaking a test on firefox stable.
(Ok, after this I will patch the version).

I think that is the effect of #516 -- let me close and reopen to trigger travis again?

Ok, thanks!

@fippo It seems to be that this fix got lost when moving https://github.com/otalk/rtcpeerconnection-shim out.

whoops? taking a look...

nope, seems to work. Can you show a jsfiddle where it doesn't?


Create CODE_OF_CONDUCT.md

A heartfelt thanks to you for your contribution to this project 🎉🎉
We are really grateful for your contribution.
Happy Learning 💚!!!

A heartfelt thanks to you for your contribution to this project 🎉🎉
We are really grateful for your contribution.
Happy Learning 💚!!!

Glad to help :)


XslTransformer.kt:47-48: This class is stateless set of...

The puzzle 29-67e5e248 from #29 has to be resolved:
https://github.com/objectionary/deog/blob/d77573f84a8d57723a3a54de2c846c5b4fef0aff/src/main/kotlin/org/objectionary/deog/sources/XslTransformer.kt#L47-L48
The puzzle was created by @c71n93 on 12-Oct-23.
Estimate: 90 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@c71n93 please pay attention to this new issue.


service upgrade API does not allow to adding new sidekick services / removing existing ones.

Server version - v0.43.0-rc2
Currently Service upgrade API does not allow to adding new sidekick services / removing existing ones.
It would be good to provide support for being able to add new sidekick services / removing existing ones from a service.

@sangeethah lets chat about the use cases to test once in the office. Only API support for now :)

Tested with rancher server version - v1.2.0-pre4-rc4 using rancher-compose:
All the below use cases work as expected, except 4:

Start a service with 2 sidekicks.Add another sidekick as part of upgrade. Confirm upgrade.
Start a service with 2 sidekicks.Add another sidekick as part of upgrade.Roll back.
Start a service with 2 sidekicks.Remove one of the existing sidekick as part of upgrade. Confirm upgrade.
Start a service with 2 sidekicks.Remove one of the existing sidekick as part of upgrade. Rollback -https://github.com/rancher/rancher/issues/6380

UI support for this feature is tracked in - https://github.com/rancher/rancher/issues/6381


Update to include new Enum functionality + min/max function updates for Elixir 1.4

Though 1.4 isn't officially released I figured it would be good to clarify about min/1 vs min/2 and max/1 vs max/2, as well as update for the deprecation of uniq/1 and include the new map_every/3 function that is included in the release, as it meshes well with the chunk and chunk_by pieces quite well.
We should probably wait to merge this until the official release as to not confuse anyone with features not yet available without installing from source. I added the fix label since uniq is being deprecated.

@brain-geek, updated. Thanks!

@ybur-yug one more place. :)

@brain-geek got it!

@ybur-yug last one! Otherwise, everything is OK to merge.

Ok if you wanna give it one final lookover I think thats all of them but I also have not excelled at noticing small details today it seems @brain-geek

@ybur-yug Feel free to merge.

@brain-geek I'll make an issue for translation, and once 1.4 is out I'll officially merge since these functionalities/changes are not present unless running it which requires a build from source on master right now.
Especially since this is in the basics section, I don't want to confuse anyone who comes in on a blank slate.

I've pushed my commit to this branch.
Just one more thing before merge: @ybur-yug , please squash first few commits into one (some of them don't have any semantic value) .

@ybur-yug one of the commits in the branch still has lol cmon bobby title. :)

@brain-geek I'll squash my stuff and keep the others here in a bit today. Sorry I just got to this, been busy the last week

Going to clean the history here then merge now that 1.4 is out

Awesome!  If we can get this out plus the staged new lessons, we could do a big ol release announcement to kick off 2017.

Elixir 1.4 has been released finally. Let's merge this after fixing history

@brain-geek I pushed up after squashing my crap commits, if it looks good to you feel free to merge.

Awesome. Thanks, @ybur-yug !
We can finally merge and close this one. :)


Voicing Support

Hi,
I know that this project is currently unmaintained, but I just wanted to voice my support for this project. Using pweave as a backend for making reports through Spyder is a truly awesome idea and I hope this gets funding soon!
Thank you for all the effort you've put into the project!

Thanks! We'll get there after we release Spyder 4.