Implement targetting nested Organizational Units

*Issue #19 : *
Description of changes:
AWS supports Organizational Units to a depth of 5. Organizational Units may be specified as the target for deployment of SCPs, but only at the level directly beneath the root of an Organization.
A delimiter character is mentioned in one test file, but the feature is not implemented.
Example specifying nested OU in manifest.yaml file:
organization_policies:
  - name: nested_scp
    policy_file: policies/policy-targetting-nested-ou.json
    #Apply to the following OU(s)
    apply_to_accounts_in_ou:
      - Parent-OU:Child-OU

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

Thanks for reaching out to us. We have added this to our backlog.

Any thoughts on when this feature might be merged and released in CT?

Any thoughts on when this feature might be merged and released in CT?

+1

Thanks for the pull request. We have added this to our backlog and will review it for the next release.

With Control-Tower now supporting nested OUs this would be a huge help 👍

Fixed in v2.2


Fix numpy to xtensor cheatsheet

there is a missing () after the ellipsis :/

darn, missed it. 😄

thought you were all going to cheer at me merging promptly while on vacation :)

I'll fix it when I push the negative slice stuff :)
I would also like to sit on a :beach_umbrella: for merging ;)


Add SiteToc to MobileNav

This PR adds the SiteToc to the MobileNav. It also fixes double type declarations in MobileNav by importing the interfaces from Nav. Adding SiteToc was then just a matter of uncommenting code that was there already.
https://github.com/datopian/portaljs/assets/105868642/75921b39-d61d-46f6-a364-5a8fcda4e7ad

LGTM so merging. @marcchehab Thank-you for the contribution 🙏


Possible break on Godot 3.2.4.rc3?

First of all, thanks for the plugin. A life saver!
Since the latest Godot build 3.2.4.rc3 the plugin-refresher can't find any my plugins.

I tried to fix the plugin-refresher myself but I can't really find where it is going wrong.
I'm completely lost so I'm creating this issue just in case you know what might have changed :(

oh, huh. No, this is news to me. I suppose I'll have to look into it further. Thanks for the heads-up!

@coppolaemilio I think I found the issue in the engine code. If you wouldn't mind, do you think you could open a bug report in godotengine/godot since you're the one actually experiencing it?
The problem appears to be (to me) that an attempt was made to simplify the logic related to identifying what a plugin inherently was (shifting away from a dependency on the .cfg file), but in the process they messed up the logic for identifying the existence of the addon. You can see in 3.2-stable, here where they fetch the config file and get its base dir:
res://addons/godot-plugin-refresher/plugin.cfg -> res://adons/godot-plugin-refresher

...but then in the latest 3.2 here, they remove the config file path and use just the addon's subdirectory and try to get its base dir:
godot-plugin-refresher -> ""

This then results in the addon never being found in any conceivable case, afaict.

Yeah, I'll create an issue on the Godot repo! Thanks for your looking into it!

Then, since this isn't a problem with the addon's code, I'm gonna close this. Cheers!


Accessibility: Lines read twice when navigating with up and down arrow in decompiler view

Describe the bug
If I attempt to up/down arrow in the decompiler view (i.e. the text field containing the code of the current function), the line I focus gets read twice.
To Reproduce

Open Ghidra and start NVDA.
Load an executable, and fully analyze it.
Control+tab to the decompiler view.
Arrow up and down, and observe what the screen reader says.

Expected behavior
The line should be read only once per arrow press. Left and right arrow already do this. I assume this is because Ghidra tries to announce it to the screen reader when a line changes, but the screen reader already does this.
Environment (please complete the following information):

OS: Windows 10 22H2 (AMD64) build 19045.3803
Java Version: 21
Ghidra Version: 10.4 20230928
Ghidra Origin: Scoop

Yes, you are correct. I had it returning the line's text when ask for the current text. The interface that I had to implement also has the notion of returning a description of the thing your are on. For the description, I was returning the line number, followed by the line's text. Apparently, when you move up a line, the screen reader reads both the text and the description, causing it to repeat the line's text twice. I have changed the description to just report the line number. So now, when you move up or down a line in the decompiler, it will first read the text on that line, followed by the line number.

Awesome, thanks! I think this can be considered fixed then, provided it's been tested and works. I'm not sure how to test if it works, or I would.

Once this has been merged and pushed to github, this ticket will be closed. This will change will then be available at that point for those that use the bleeding edge of the master branch in github. Otherwise, it will be available in the next patch release.


Comment message name

There should be github-actions name or the person's name when a person is invited to an organization
Here you can find a demo message

@vj-abishek did you mean to close this?

@vj-abishek did you mean to close this?

Sorry that issue was solved 😅. I asked " the reply for organization invitation should be bot message or real human"

Sorry that issue was solved 😅. I asked " the reply for organization invitation should be bot message or real human"

ah ok, no problem 👍

ah ok, no problem 👍


docs: fixed typo in error.md

Hi, I added params to pass the path parameters of apiClient.getUser because they were missing.

Thanks


Solved issue #181. added nav bar to login and signup pages

Pull Request Format
PR Title
Issue#181 : [BUG] Add nav bar and footer to sign up and login pages solved
Type of PR

[X] Bug fix

Description
Added nav bar to login and signup pages

Add X in the box to specify.
[X] I have performed a self-review of my code.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Thank you for reviewing my pull request!

Hey @swaraj-das! Pls check and merge

@swaraj-das pls review

@manikdamle The login page looks decent, but there’s room for improvement. The sign-up page needs more attention as it requires significant enhancements. Additionally, both the footer and the navbar don’t look quite right and could use redesigning to improve the overall user experience.

@swaraj-das Ok will do so.


🐛 Bug: when activated with the keyboard, emojis on the site originate from top left corner

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Expected
When I press the Click Me button with the keyboard, I'd expect emojis blast from the button's position (prolly centre of the button) 🤩💥💥

Actual
Emojis blast from the top left corner of the page and this ain't so exciting when most of them end up beyond the visible area 😔⛈

Additional Info
Lies, lies everywhere 😡

The first two checkboxes in checklist aren't applicable but they're required: I didn't pull the repo, nor opened IDE 😏🐸

Ha, nice catch! Thanks for reporting. I honestly just threw that button together quickly last minute.
https://github.com/JoshuaKGoldberg/emoji-blast/blob/43e280beb46fb1b3f684ef180d9190fd10688c1c/packages/emoji-blast-site/src/components/hero/hero.tsx#L29-L34
I think a good fix would be that if the event doesn't have clientX and clientY, then to not pass a position at all. That way the blast will default to some random position on the page.
A probably-better alternative would be to have the emoji blast originate from the element itself. That might be a bit more work? The Title Explosion demo makes me think it's a bit more work.
Either way I'm happy for a PR!

Haha, shoutout @danielroe for reminding me of getBoundingClientRect(). Let's use that as a default if the event doesn't have coordinates.


No of Child Buttons

How can I change the number of child buttons?
I want 5 but am having difficulty in doing so.

Supposing you have already installed Grunt and successfully run:
npm install

the steps you need to follow are:

Add the appropriate number of child buttons in your html.

Open the file _customise.scss and specify the number of buttons you need, in your case 5:

$number-of-child-buttons: 5;

Save and run grunt sass or grunt watch-css. If you're not using the provided grunt tasks you can also process the SCSS on your own in you preferred way, obviously.
This way you should be good to go.
Let me know if you need any more help. Cheers.

Hey Nobita,
Thanks for your reply.
Actually I have never worked with grunt or node js which is why i am having problem in customizing.
I tried it with nodejs after watching some tutorial but I could not make it.
Is there a simple way or I have to learn grunt for this?

Actually in this case it's not a matter of learning Grunt, but just to manage to get it installed, as in this project everything is already configured to work out-of-the-box just by running a simple command.
Anyways, you can certainly do without. Do you have a way to compile files form SCSS to CSS or you work straight from CSS?

I work straight from css

Uhm.. let's see if this can help.
Head over to this link.
On the the left you have the current SCSS full source, on the right the resulting compiled CSS. It's already configured for 5 buttons. If that's ok you can simply copy the CSS, otherwise tweak the variables in the SCSS and it will be compiled instantly.
Mind that this is the current source, so if you forked/cloned before the today's last commit you'll want to pull in the changes via git or fork/clone a fresh copy as the API has been changed today.

Thanks for the Help :) :+1:

Haha... cheers mate!


Fails with nn.ModuleList

I have a model defined with a sequence of layers stored in a nn.ModuleList. For example I find it useful for defining a fully connected neural network (MLP) where you want to pass the number of layers and neurons per layer as input to the constructor, like this:
class Net(nn.Module):

    def __init__(self, inputSize, numLayers, nodesPerLayer, activationType):
        super(Net, self).__init__()
        if activationType.lower() == 'sigmoid':
            self.activation = nn.Sigmoid()
        elif activationType.lower() == 'tanh':
            self.activation = nn.Tanh()
        else:
            self.activation = nn.ReLU()
        self.hidden = nn.ModuleList()
        self.hidden.append(nn.Linear(inputSize, nodesPerLayer))
        for i in range(numLayers-1):
            self.hidden.append(nn.Linear(nodesPerLayer, nodesPerLayer))
        self.finalFC = nn.Linear(nodesPerLayer, 1)

    def forward(self, x):
        for layer in self.hidden:
            x = self.activation(layer(x))
        x = self.finalFC(x)
        return x

In this case, calling self.model.modules() returns the list of nn.Linear layers but also the nn.ModuleList as a separate module. Then the get_output_sizes function tried to run an input through this "module" and fails.
Additionally I think the get_parameter_sizes function calculates the parameter size incorrectly because it takes into account each layer twice.
I think for this case this issue can be solved by checking in both get_parameter_sizes and get_output_sizes functions whether a module is of type nn.ModuleList (using isinstance(m, nn.ModuleList), then do nothing.
What do you think?

Why not just iterate over model.parameters()?


Lower IOPS When First Test

I ran DiskSPD to test my SSD in SEQ Write QD64T1. The result of the first time test was lower than other time.
Ｉtry to repeat this result in other benchmark tool(like iometer), but this phenomenon not happen in other tool.
My work load as followed:

Create file 1GB test filet to SSD
delay 5s
ran diskspd.exe -o64 -t1 -Z64K -b64K -ag -S -w100 -W1 > D:TestFile.raw (first)
delay 5s
repeat step 3 and 4 for 5 times

Test results:

1599.4 MiB/s
1606 MiB/s
1604 MiB/s
1605 MiB/s
1608 MiB/s

To avoid SSD in power mode, i set -W1 to pre-write it. Is OS and SSD behavior to affect test result? thanks

This could be coming from anywhere, That aside, these results are within a half percentage point.
Handful of ideas/observations:
The default warmup (-W) is five seconds, and a 10 second write test on an SSD is nowhere near enough to reliably get to steady state operation. It makes you very vulnerable to uncontrolled behavior (inside the SSD or otherwise).
If IOMETER has no warmup interval it may be including behavior which DISKSPD is excluding with its warmup (1 or 5 seconds, or whatever you may choose next). If you want steady state, my suggestion would be to warm up for at least minute (60s) and then measure for 10s: -W60 -d10. Industry practice would be closer to five minutes warmup.
Do the same with IOMETER and the variance should be small & results repeatable.

Closing due to no response. Please reopen if you have more questions.


onPush() function not getting hit after push from controller using ZMQ

Config.yml
gos_web_socket:
    topics:
      - @app.real_time.subscribe
      - @app.real_time.push
    server:
        port: 8000        #The port the socket server will listen on
        host: localhost   #The host ip to bind to
        router:
            resources:
                - @AppBundle/Resources/config/pubsub/routing.yml
    client:
        firewall: main #can be an array of firewalls
        session_handler: @session.handler.pdo
    pushers:
        zmq:
            default: true
            host: <IP_ADDRESS>
            port: 8888
            persistent: true
            protocol: tcp

pubsub/routing.yml
real_time_push:
    channel: all/user
    handler:
        callback: 'app.real_time.push' #related to the getName() of your topic

services.yml
app.real_time.subscribe:
        class: AppBundle\RealTime\Subscribe
        arguments: [ @gos_web_socket.websocket.client_manipulator ]
app.real_time.push:
        class: AppBundle\RealTime\Push
        arguments: [ @gos_web_socket.websocket.client_manipulator ]

The following function is getting hit when client subscribes on this topic. I can see the text in the console.
Subscribe Class onSubscribe() : Subscribe.php
public function onSubscribe(ConnectionInterface $connection, Topic $topic, WampRequest $request)
    {
        var_dump("This user is subscribed");
    }

But when pushed via zmq from controller the onPush() function never got hit. I have already implemented TopicInterdace and PushableTopicInterface in Push.php.
Push Class onPush() : Push.php
public function onPush(Topic $topic, WampRequest $request, $data, $provider)
    {
        var_dump("Helloooooooooooooooooo0000000000");
    }

Please can any one tell me where did I do wrong. Thanks in advance. Here is the controller
$pusher = $this->container->get('gos_web_socket.zmq.pusher');
//push(data, route_name, route_arguments)
$pusher->push(['my_data' => 'data'], 'real_time_push', ['username' => 'sujit']);

@ProPheT777 Please have a look. I am stuck for 2 days now.

Why do you segregate these services?
      - @app.real_time.subscribe
      - @app.real_time.push

subscribe and push are part of the same class, I think the problem is here.

@ProPheT777 Sorry for late response. I just modified to this configuration but same problem.Subscribing is fine but onPush() is not working.
config.yml
gos_web_socket:
    topics:
      - @app.real_time.subscribe_and_push
      #  - @app.real_time.push
    server:
        port: 8000        #The port the socket server will listen on
        host: localhost   #The host ip to bind to
        router:
            resources:
                - @AppBundle/Resources/config/pubsub/routing.yml
    client:
        firewall: main #can be an array of firewalls
        session_handler: @session.handler.pdo
    pushers:
        zmq:
            default: true
            host: <IP_ADDRESS>
            port: 8888
            persistent: true
            protocol: tcp

pubsub/routing.yml
real_time_subscribe_push:
    channel: all/user
    handler:
        callback: 'app.real_time.subscribe_and_push' #related to the getName() of your topic

services.yml
app.real_time.subscribe_and_push:
        class: AppBundle\RealTime\SubscribeAndPush
        arguments: [ @gos_web_socket.websocket.client_manipulator ]

SubscribeAndPush Class onSubscribe() : SubscribeAndPush.php
public function onSubscribe(ConnectionInterface $connection, Topic $topic, WampRequest $request)
    {
        var_dump("Subscribed..............."); //This is working like a charm
        // TODO: Implement onSubscribe() method.
    }

SubscribeAndPush Class onPush() : SubscribeAndPush.php
public function onPush(Topic $topic, WampRequest $request, $data, $provider)
    {
        var_dump("PUSHED................."); //Never got hit
        // TODO: Implement onPush() method.
    }

Controller
$pusher = $this->container->get('gos_web_socket.zmq.pusher');
//push(data, route_name, route_arguments)

$pusher->push(['my_data' => 'data'], 'real_time_subscribe_push', ['username' => 'sujit']);

and after hitting the controller, on the profiler the GOS/Websocket bundle icon appears with push message counts but still no print in console or client side. @ProPheT777

There is another thing. According to the docs-
Pusher event
When pusher send message or fail to send it, we dispatch event to allow you to plug your own logic.

Success : gos_web_socket.push_success
Fail : gos_web_socket.push_fail

Will give an Gos\Bundle\WebSocketBundle\Event\PushHandlerEvent where can access to the message and to the handler.
But on the profiler events these events never fired.

I added eventListeners. But there is no response. @ProPheT777 please suggest what should I do.

Have you try with websocket pusher? (He use directly onPublish instead of onPush take care) Maybe windows is the problem with zmq

Can you tell me which is recommended? I thought websocket pusher is not recommended due to latency. Can you send me the link of proper zmq library. I have downloaded from This Link. thanks @ProPheT777
I am using php 5.5.12

Is there any way that this bundle support Wamp v2 ? thanks. @ProPheT777

I actually have the same probleme from the original poster.
OnPush() is not called from a controller, therefor the Push event in controller look ok.
I'm on Win10, using Wamp server, as the poster, so i guess ZeroMQ have issue with windows ?


CUDA Runtime Error at: monarch_cuda/monarch_cuda_interface_fwd.cu:665

I've tried running the below example code after building the packages from source from the README instructions.
import os
import torch

device = torch.device("cuda")
from flashfftconv import FlashFFTConv
flash_fft_conv = FlashFFTConv(2**14, dtype=torch.float16).to(device)
# B is batch size, H is model dimension, L is sequence length
B = 16
H = 768

# input can be smaller than FFT size, but needs to be divisible by 2
L = 16384

# the input, B H L
x = torch.randn(B, H, L, dtype=torch.float16).to(device) # same type as the input
k = torch.randn(H, L, dtype=torch.float32).to(device) # kernel needs to be fp32 for now

out = flash_fft_conv(x, k)
print(out)

I get the following error and then an output of all zeros printed to the console
ERROR: CUDA RT call "cudaFuncSetAttribute(&monarch_conv_cuda_16_32_32_kernel<32, 8, 16384, 1, 2, 16, false, 8, 8, 8>, cudaFuncAttributeMaxDynamicSharedMemorySize, 102400)" in line 535 of file monarch_cuda/monarch_cuda_interface_fwd.cu failed with the provided PTX was compiled with an unsupported toolchain. (222).
CUDA Runtime Error at: monarch_cuda/monarch_cuda_interface_fwd.cu:665
the provided PTX was compiled with an unsupported toolchain.

I have the following tools and packages installed
torch==2.1.0+cu121

nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2023 NVIDIA Corporation
Built on Mon_Apr__3_17:16:06_PDT_2023
Cuda compilation tools, release 12.1, V12.1.105
Build cuda_12.1.r12.1/compiler.32688072_0

and
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 525.85.05    Driver Version: 525.85.05    CUDA Version: 12.0     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA A100-SXM...  On   | 00000000:CB:00.0 Off |                    0 |
| N/A   31C    P0    60W / 400W |      0MiB / 81920MiB |      0%      Default |
|                               |                      |             Disabled |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+

The error message sounds like my driver version is incompatible but I'm fairly certain that with a major version (i.e. CUDA 12), the driver should be sufficient to run all cuda toolkit versions. Which drivers has this code been known to work with?

Installed cuda-toolkit-11-8 and torch==2.1+cu118 and things seem to be working okay now


Add support for self-signed certificates

Was trying to scan a semi-private site which is using a self-signed certificate.
It would be awesome to have a flag (e.g. --no-check-certificate) to circumvent this:
Error: unable to verify the first certificate

A bit of googling led me to this answer: http://stackoverflow.com/questions/10888610/ignore-invalid-self-signed-ssl-certificate-in-node-js-with-https-request
tl;dr setting NODE_TLS_REJECT_UNAUTHORIZED=0 in my environment fixed it.

Thanks @ajb - I've been struggling with this!

NODE_TLS_REJECT_UNAUTHORIZED=0 isn't working for me. it attempts to start crawling the site and then doesn't report finding any links.

NODE_TLS_REJECT_UNAUTHORIZED=0 isn't working for me. it attempts to start crawling the site and then doesn't report finding any links.

I have the exact issue and it's so frustrating. Nothing gets crawled and it just exits without any links crawled or error messages. I believe this is due to using an SSL certificate on the site but the SSL doesn't have an alternate name for the current domain that you are trying to crawl. I believe this environment variable should have catered for this error too.

Do you have the same issue with the v0.8.0 branch? Much has changed there.

With version 0.8.0 I get a crash:
`internal/modules/cjs/loader.js:638
throw err;
^
Error: Cannot find module '../lib-cjs/cli'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
at Function.Module._load (internal/modules/cjs/loader.js:562:25)
at Module.require (internal/modules/cjs/loader.js:690:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object. (C:\Users\adoobary\AppData\Roaming\npm\node_modules\broken-link-checker\bin\blc:3:1)
at Module._compile (internal/modules/cjs/loader.js:776:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)`
I tried on both Windows 10 and Linux, I get the same error.

You first have to
npm run build

Closing for now, as I think that this is fixed (on master branch).


fix: Update searxng settings

related: https://github.com/miurla/morphic/pull/309#issuecomment-2294810551
Since the configuration of the search section has been removed in this PR, I will restore it: https://github.com/miurla/morphic/pull/312

Since the changes are included in #320, this PR will be closed.


Replace data URL in SAMSum dataset and support streaming

This PR replaces data URL in SAMSum dataset:

original host (arxiv.org) does not allow HTTP Range requests
we have hosted the data on the Hub (license: CC BY-NC-ND 4.0)

Moreover, it implements support for streaming.
Fix #4146.
Related to: #4236.
CC: @severo

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Update about-azure-maps.md

@jingjing-z : Thanks for your contribution! The author, @kgremban, has been notified to review your proposed change.

#sign-off


Github Copilot Chat not working and panel is not showing up

VS Code Version: 1.83.1
OS Version: macOS Ventura 13.5.2

Problems:

GitHub Copilot Chat is not appear in the sidebar.
When I click copilot icon -> Github Copilot Chat, nothing is happening.
When I tried cmd + . at the code, there are also no code action such as using Fix with Copilot or Explain with Copilot appear.

What I've tried

Uninstall and reinstall copilot extensions
Signout and re-signin account

Steps to Reproduce:

Click on the copilot icon at the bottom right, then click Github Copilot Chat
Press cmd + . button at the code for code action and

Open the command prompt and type copilot logs.
Do you see something like this?
2023-11-01T20:24:41.101Z [INFO] [auth] Logged in as austenstone
2023-11-01T20:24:41.555Z [INFO] [chat] copilot token chat_enabled: false
2023-11-01T20:24:41.555Z [INFO] [auth] Got Copilot token for austenstone

Here is my log
2023-11-02T02:01:03.168Z [INFO] [auth] Logged in as chanon-mike
2023-11-02T02:01:03.765Z [INFO] [chat] copilot token chat_enabled: false
2023-11-02T02:01:03.765Z [INFO] [auth] Got Copilot token for chanon-mike

Do the in editor ghost text completions work for you? Based on your logs you do not have copilot access.

Yes, ghost texting work. Only chat is not working. I have GitHub student which include copilot.
It just remind me that I just joined the organization, when this happened. Does it related?

Please try some of the steps in https://github.com/microsoft/vscode-copilot-release/issues/436 such as logging out and back in within VS Code

Thank you for your response! I've tried all of those but nothing helped. I looked into Github Copilot settings and here is my settings.
I have my copilot on my own, but it seems like copilot from the organization overwritten my settings...
Are there anyway to prioritize my own settings over organization one?

Thank you for your answer!!


refactor(diagnose): rename more funcs

Signed-off-by: Xavier Coulon<EMAIL_ADDRESS>

Codecov Report

Merging #55 (eaf4472) into main (e72682a) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #55   +/-   ##
=======================================
  Coverage   75.55%   75.55%
=======================================
  Files          13       13
  Lines         765      765
=======================================
  Hits          578      578
  Misses        129      129
  Partials       58       58

Impacted Files
Coverage Δ

pkg/diagnose/diagnose.go
86.95% <100.00%> (ø)

pkg/diagnose/diagnose_deployment.go
71.42% <100.00%> (ø)

pkg/diagnose/diagnose_pod.go
72.72% <100.00%> (ø)

pkg/diagnose/diagnose_replicaset.go
58.82% <100.00%> (ø)

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


Feature request: Make an Item reference sampled data in a second table

I have a quite specific usage case where i need to annotate data from a table which i just sampled. I need a second table which then contains the sampled data annotated with additional data. An obvious but inefficient solution is to store a copy of the sampled data in the  second table together with the annotation.
The reverb documentation clearly states, that an item can reference different chunks. Therefor i guess that this is technically possible and maybe i just don't know how to implement it.
Here is a code example of the inefficient implementation:
import tensorflow as tf
import reverb

rt = reverb.Table(
        't1',
        reverb.selectors.Uniform(),
        reverb.selectors.Fifo(),
        1000,
        reverb.rate_limiters.MinSize(1),
        signature=tf.TensorSpec(1)
        )

rt2 = reverb.Table(
        't2',
        reverb.selectors.Uniform(),
        reverb.selectors.Fifo(),
        1000,
        reverb.rate_limiters.MinSize(1),
        signature=tf.TensorSpec(2)
        )

server = reverb.Server(
        [rt,rt2]
        )

client = reverb.Client(f'localhost:{server.port}')

with client.trajectory_writer(1) as w:
    w.append(tf.random.uniform([1]))
    w.create_item('t1',1,w.history[0])

tds = reverb.TrajectoryDataset.from_table_signature(
        f'localhost:{server.port}',
        't1',
        10,
        )

for s in tds.take(1):
    #sample data
    print(s.data)
    with client.trajectory_writer(1) as w:
	#annotate sampled data and save the sampled data in a new
	#table together with the annotation
	#a copy of the sampled data is saved, making this inefficient
        w.append(tf.concat([s.data,tf.random.uniform([1])],0))
        w.create_item('t2',1,w.history[0])

Item can reference other items in case for instance when items are N-step transitions. In such case, to avoid storing the same step N times, Reverb client tracks re-usage of the steps. In your case, however, when you sample an item and add it to the second table (with annotations) it is not possible (trivial to do). Hence 2x storage is the easiest/best way to go.


Auto scroll when displaying a new message.

https://www.npmjs.com/package/react-scroll-to-bottom

Need to review behaviour and params. Library change default scroll view

Probably better to exclude this library


resolve not launching on wayland because of missing Xauthority file

I got this error when running resolve.sh
$XAUTHORITY was not set.  Defaulting to /run/user/1000/gdm/Xauthority
Error: statfs /run/user/1000/gdm/Xauthority: no such file or directory

I'm on Cachyos with hyprland.
I was able to fix the problem by modifying this part in resolve.sh
# quick sanity check.
if [ -z "${XAUTHORITY}" ]; then
   if [ -f "${HOME}/.Xauthority" ]; then
      export XAUTHORITY="${HOME}/.Xauthority"

   elif [ $XDG_SESSION_TYPE = "wayland" ]; then
      export XAUTHORITY="${RESOLVE_CONFIGS}"

   else
      export XAUTHORITY="/run/user/`id -u`/gdm/Xauthority"
   fi
   echo "\$XAUTHORITY was not set.  Defaulting to ${XAUTHORITY}"
fi

Not sure if it's the best way tho fix this.

Hmm.  I tried wayland in Ubuntu 24.04 LTS and it works alright as-is with the 560.35.03 nVideia driver without incident.  CachyOS is arch-based and hyprland (which I wasn't familiar with) is a compositor for wayland instead of mutter...
It seems that your system doesn't set up a /gdm/Xauthority directory, leading to the crash. Wayland doesn't use Xauthority, so this makes sense.
Perhaps it might make more sense to check for "wayland" and then if it's running bypass the whole XAUTHORITY definition at all (or set it to /dev/null or something that doesn't matter)
[FIVE MINUTES LATER...]
I just did a test and setting it to /dev/null does seem to work, though I should note that by default there is an .mutter-Xwaylandauth value set for backwards compatibility in Ubuntu.  I'm guessing this is not the case in a wayland-only system, so having a fallback makes sense.
Request:  Do you want to submit a pull-request similar to your change above, but set XAUTHORITY to /dev/null instead for wayland-only distros that don't include X11 compatibility? It should probably still otherwise fall back as-is for X11-users.
I'm also happy to do this too.  Thanks!
Also, I see Resolve [19.0.3 just came out...]
(https://forum.blackmagicdesign.com/viewtopic.php?f=21&t=210182&sid=22a415d1797a922298adbd9d883ce950)  Exciting.

I've made a pull request #61.

It looks good to me!  Many thanks!


Font audit script

We have an issue where it is hard to see what fonts are included in the templates.json from Figma. It would be helpful to have a script that runs through all the element objects in the templates and returns a list of the the different font-families and weights used. Keep in mind that fonts are their font-weights + font-families.

could this be used for preloading? or are we still anti-preloading?  we can preload with fontfaceset

the only reason why i suggest preloading is because we don't really care if it renders fast since the user isn't ultimately viewing this on web, right?  preloading would allow us to not worry about any font stuff during the drawing process (so theres potentially better flow).  otherwise, the font detection has to happen in the middle of each drawing function (we would have to add in identical lines for font detection/loading in each draw function).
im picturing this:

during convert, each page is given a new attribute called fonts = [array of font-families/weights]
before drawing the page, load the fonts (w/ error detection)
draw

i also suggest this because if we loaded the font in each draw text method, then we'd be asking to load the same font potentially multiple times (although not sure if the load would be executed multiple times or skipped over after the first time)
tldr: if we want to render PC on a webpage vs. PDF eventually, we shouldn't preload but if thats not in the realm of possibilities, preload could maybe make this lib cleaner

@g-a-v-i-n let me know your thoughts or if you see some flaws in my logic here!


Release Service Bus Track 2 Library Preview 3 for Python

This is a parent issue tracking the work planned for Service Bus to release the second preview in June following the Python Azure SDK guidelines
Code complete date: June 5th, 2020
API Design

[x]  Auto-complete
[x] Dead Letter Queue support
[ ] (Stretch) Alternate UAMQP Data Type support

Features

[x] CRUD & list operations using connection string via the ATOM apis
[x] (Stretch) Revisit double-with call pattern
[ ] (Stretch) Connection Sharing

The followup questions for autocomplete and double-with have been included in our UX study agenda.  Any actions following from that will come in during the July milestone.


Add hex2patq and patq2hex functions

Like it says on the can.

(Also I'm aware these and others should really be 'vatq' now, but that's a change for another day..)


WIP: converting to TypeScript

Description

Motivation and Context

issue
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Author's Checklist:

[ ] My code focuses on the main motivation and avoids scope creep.
[ ] My code passes current tests and adds new tests where possible.
[ ] My code is SOLID and DRY.
[ ] I have updated the documentation as needed.

Reviewer's Checklist:

[ ] I kept my comments to the author positive, specific, and productive.
[ ] I tested the code and didn't find any new problems.
[ ] I think the motivation is good for the project.
[ ] I think the code works to satisfies the motivation.

WIP pull request.
I figure we can divide and conquer. I can take the files from the bottom to the top of the layer folder unless you have another suggestion.


Added Twisted Python Projects

Hi,
Stjepan here from Manning. I thought this title might be a good match for your list. Thank you for considering it.
Best,

This branch can't be merged, kindly resolve the conflicts and push the commit again.

Really appriciate your contribution. But this branch can't be merged, kindly resolve the conflicts and push the commit again.

Thank you. I have resolved the conflicts and pushed the commit again.

Superb! Thanks for your valuable contribution to make this repo wonderful. @stjepanjurekovic

Thank you, Chintan.
Best,
On Wed, Apr 26, 2023 at 2:04 PM Chintan Rathod @.***>
wrote:

Merged #392
https://github.com/GDGAhmedabad/Awesome-Learning-Resources/pull/392
into master.
—
Reply to this email directly, view it on GitHub
https://github.com/GDGAhmedabad/Awesome-Learning-Resources/pull/392#event-9102769475,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDIEMLVZR2ICNCTJ7WWDYDXDEFORANCNFSM6AAAAAAXMJAZGI
.
You are receiving this because you were mentioned.Message ID:
<GDGAhmedabad/Awesome-Learning-Resources/pull/392/issue_event/9102769475@
github.com>

--
Stjepan Jureković
Manning Publications - Marketing
@.***


Three-stage prediction

An app for analyzing vehicle damages is definitely very useful.
I think that it will be a good idea to undertake the prediction in three stages.

Does the image contain a vehicle?
If yes, does the image contain a part for which reference images are provided?
If yes, is it positive?

Closing this feature request as it is out of scope for this project.


clusterroles.rbac.authorization.k8s.io "kube-backup-reader" is forbidden

I trying to enable kube-backup for one of the clusters that is running on GCP.
cronjob-ssh.yaml has been deployed
kubectl describe job kube-state-backup-1545735000 -n kube-system returns:
Error creating: pods "kube-state-backup-1545736800-" is forbidden: error looking up service account kube-system/kube-backup: serviceaccount "kube-backup" not found
When the rbac.yaml is deployed, the following issue occurs:
kubectl create -f https://raw.githubusercontent.com/pieterlange/kube-backup/master/rbac.yaml --namespace=kube-system
clusterrolebinding.rbac.authorization.k8s.io/kube-backup created
Error from server (Forbidden): error when creating
"https://raw.githubusercontent.com/pieterlange/kube-backup/master/rbac.yaml":
clusterroles.rbac.authorization.k8s.io "kube-backup-reader" is forbidden:
attempt to grant extra privileges:

Based on this I tried:
kubectl create clusterrolebinding kube-backup-reader --clusterrole=cluster-admin --user=<email-address>
I ran the commands that are defined in this post https://stackoverflow.com/a/47332612/2777965, but the issue persists.
Now I wonder whether the "Tiller approach" should be used (https://github.com/fnproject/fn-helm/issues/21#issue-312627792).

Based on this I tried:
kubectl create clusterrolebinding kube-backup-reader --clusterrole=cluster-admin --user=<email-address>

Hey Ben! Merry xmas.
the serviceaccount should've been created when you deployed cronjob-ssh.yaml, but the error (attempt to grant extra privileges) you're getting when creating the RBAC rules is related to YOU not having sufficient privileges in the cluster to create the new rolebindings. So, instead of assigning kube-backup-reader cluster-admin privileges. bind yourself to cluster-admin and then create the RBAC rules. Good luck :) I'll check back later.

Issue one:
@pieterlange the feedback helped! The other issue was that my email address contained two Uppercase chars and the role I created contained only lower case characters.
Issue two:
running:
kubectl create --edit -f https://raw.githubusercontent.com/pieterlange/kube-backup/master/cronjob-ssh.yaml --namespace=kube-system
skipped the creation of the serviceaccount. Running it as a separate file worked and when a helm chart would be used then this issue would not occur at all.

Ah, it's a combined YAML file and with --edit it probably picked only one out of the 2 resources. Obscure flags in kubectl can be tricky. :)

The git-crypt seems to work as well now. I get encrypted secrets in my git repository. :)

I wonder whether the RBAC snippet is still required as without changing the current RBAC the secrets were sent to the repository.
rules:
- apiGroups: ["*"]
  resources: [
    "configmaps",
    "secrets",

I had to add pvc to the RESOURCETYPES as pvc yamls were removed from my git repository


First Login: can't get past GPG passphrase error

I have an existing Keybase account. After installing the client, I am able to enter my Keybase login, password and enter a device name.
I selected the Terminal option for entering my GPG passphrase. There was a problem (fat fingers?) and now it won't let me retry entering my passphrase.  Closing and re-opening the client results in the same issue - I can enter username, Keybase pwd, device name but then I am not given the option to enter my GPG passphrase.
"GPG error: exit status 2"
Model Name:	MacBook Pro
Model Identifier:	MacBookPro10,1
Processor Name:	Intel Core i7
Processor Speed:	2.3 GHz
Number of Processors:	1
Total Number of Cores:	4
L2 Cache (per Core):	256 KB
L3 Cache:	6 MB
Memory:	8 GB
Boot ROM Version:	MBP101.00EE.B0B
SMC Version (system):	2.3f36
Keybase Version 1.0.18-20170208165022+dc66461 (1.0.18-20170208165022+dc66461)
my log id: c153336045672ef1d54f501c

By signing in with my Keybase username (not email) I was able to select Keybase client authentication, which allowed me to progress

you have a bug in your local GPG configuration
657 2017-02-09T07:34:55.322308 ▶ [DEBU keybase gpg_cli.go:332] 283 gpg: malformed GPG_AGENT_INFO environment variable


How to test the pretrained modal on my own video?

I want to infer on my videos (only several sequences) and visualize them. Should I follow test_tapvid?

Yes, instead of using the loaded data, you can provide your video to run_crw() and use the visualization from here.


How to use Iterator to differentiate between array with single element vs a single element

How do I differentiate between array with single element vs a single element from the Iterator of a query ?
package main

import (
	"fmt"
	"log"

	"github.com/itchyny/gojq"
)

func main() {
	query, err := gojq.Parse(".tags[].name")
	if err != nil {
		log.Fatalln(err)
	}
	input := map[string]interface{}{
		"kind": "do#droplets",
		"href": "/droplets/123",
		"tags": []interface{}{
			map[string]interface{}{
				"name": "hello",
			},
			map[string]interface{}{
				"name": "goodbye",
			},
		},
	}
	iter := query.Run(input)
	var result []interface{}
	for {
		v, ok := iter.Next()
		if !ok {
			break
		}
		if err, ok := v.(error); ok {
			log.Fatalln(err)
		}
		if v != nil {
			result = append(result, v)
		}
	}
	fmt.Println(result)
}

Output: [hello goodbye]
Now, if I modify the input,
package main

import (
	"fmt"
	"log"

	"github.com/itchyny/gojq"
)

func main() {
	query, err := gojq.Parse(".tags[].name")
	if err != nil {
		log.Fatalln(err)
	}
	input := map[string]interface{}{
		"kind": "do#droplets",
		"href": "/droplets/123",
		"tags": []interface{}{
			map[string]interface{}{
				"name": "hello",
			},
		},
	}
	iter := query.Run(input)
	var result []interface{}
	for {
		v, ok := iter.Next()
		if !ok {
			break
		}
		if err, ok := v.(error); ok {
			log.Fatalln(err)
		}
		if v != nil {
			result = append(result, v)
		}
	}
	fmt.Println(result)
}

Output : [hello]
Output in both cases is an array. Now, if I change the path and set it to .href, I still get an array:
package main

import (
	"fmt"
	"log"

	"github.com/itchyny/gojq"
)

func main() {
	query, err := gojq.Parse(".href")
	if err != nil {
		log.Fatalln(err)
	}
	input := map[string]interface{}{
		"kind": "do#droplets",
		"href": "/droplets/123",
		"tags": []interface{}{
			map[string]interface{}{
				"name": "hello",
			},
		},
	}
	iter := query.Run(input)
	var result []interface{}
	for {
		v, ok := iter.Next()
		if !ok {
			break
		}
		if err, ok := v.(error); ok {
			log.Fatalln(err)
		}
		if v != nil {
			result = append(result, v)
		}
	}
	fmt.Println(result)
}

Output: [/droplets/123]
I want to be able to differentiate between Iterators which is an array of one element vs an Iterator that should be giving me a single value. Is there a good way of handling this using this library ?

These cases are indistinguishable in principle because every single value in jq semantics is an iterator emitting one value. You get an array because you're appending to the result, but you can just break the for loop after getting one value to get a single element. So correcting the title, between a single element from an iterator vs a single element and the answer is, indistinguishable.

but you can just break the for loop after getting one value to get a single element

yeah, I can do that. But, How do I decide when to break ?
The problem is that the Iterator does not give us any information of the underlying data format. In the above examples, we know that the value out of .href is supposed to be a single element as opposed to .tags[].name which could return an array.

I think that given a single input document, you can reason about whether a query would return an iterator that will always give a single value, an iterator that could return multiple values, or if it would be ambiguous:

.href is a simple query that would always give a single value iterator for a given document
.[].href is a simple query that could return multiple values
Given more complex queries it might require reasoning about multiple branches, but even in that case, if one branch can return a multi value iterator, then perhaps we should say that the whole branch statement can as well

Yeah, please don't be confused, .tags[].name does not return an array. It may return multiple values, a single element, or nothing. So you're saying the way of distinguishing an iterator which always emits an element and an iterator which emits unpredictable number of elements. if .foo then .href else .[].href can be both.

Note that the interpreter does no distinguish them on runtime. Considering everything an iterator simplifies the interpretation of jq semantics. For example, in a query [ .foo, 10, .[], .. ], considering .foo, 10, .[], .. are all iterators, [ ] is an operator which constructs an array from multiple iterators.

I think this is statically determined and was thinking of a static type system in jq language, but it'll require lots of work. I don't think I will implement this so closing.


Harmony: FormEditor

PR includes

[x] Bug Fix
[x] Feature

Related issues
Resolve #847
QA Instructions, Screenshots, Recordings

FormControlEditor

/canary

/canary

/canary

/canary

/canary

/canary

/canary


[Bug] Light mode not using proper color scheme when system is in dark mode

Describe the bug
When  light mode is selected in browser and android system is in dark mode then light mode does not work properly.

System theme: Light, App theme: Light
System theme: Dark, App theme: Light
System theme: Dark, App theme: Dark

How to Reproduce

Select light mode in browser
Put system in dark mode
observe the app

Expected behavior
Light mode should have same color scheme regardless of system theme mode.
Environment
- DDG App Version: 5.99.0
- Device: Xiaomi Redmi Note 8 Pro
- OS: Android 11

@malmstein assigning this issue to you. Not sure if there's something we want to address here.

This was fixed but never closed.


Add port to camera and thumbnail URL

Hi @marcolivierarsenault,
thanks for creating this nice Home Assistant integration! I noticed that I had to add the moonraker port to both camera URLs (camera and thumbnail).
I didn't fully work my way through the tests. Would a change to the tests be required for this change?

Hey 1 thanks a lot for the contribution.
Have you tried to put a url that includes a port in the original setup?
exemple:
http://<IP_ADDRESS>:123

Hi @marcolivierarsenault,
When adding a new printer to the integration, the setup screen looks like this:

I, as a user, don't get the idea to add a URL, because there are two fields asking for host and port, separately.
Or do you refer to the integration's "configure" screen show below? Yes, there I had to add the full URL including port. With this PR's change applied, the camera works without explicit configuration.

Oh sorry, @marcolivierarsenault. Now, I newly re-added the printer to the integration again to be fully sure. In my case, the camera streams runs on a different port than the moonraker API (see screenshots above). So in my case, I have to set the configuration entry manually.
When I tested this change, I was fooled by Homeassistant as it seems to have properly cached/saved the manual configuration somehow, while I thought I was starting with a clean device integration. Sorry for this!
I guess the PR might still be valid, but not 100% sure as I can't test it in my setup :/ If you think that change is not required, feel free to just close this PR.


Inputs to the decode_WEBNLG

I run the codes of the WEBNLG dataset, however, it generates empty test text files, and eventually, running the decode_WEBNLG.sh, it also generates a empty output text. Is that right?

I have the same problem. Any one can help?

I don't know where datasets are. Any one can help ? Thank you!

Hi, I solved the problem by adding the following code into the front of here.
if d == 'test':     datapoint, cats, tripes = get_data_dev_test(filename, train_cat)     all_cats.update(cats)     datapoints.extend(datapoint)     triples[d].extend(tripes)
With this modification, I also have confirmed that the test set is handled correctly.

The codes always returns
File "D:\kg2text-master\graph2text\onmt\bin\preprocess.py", line 241, in count_features
first_tok = f.readline().split(None, 1)[0]
It turns out that the file "train-surfaces.txt" and "train-surfaces-bpe.txt" is empty.
I fount it maybe caused by line 155 : l = l.firstChild.nodeValue.strip().lower()
It always return <DOM Text node "'\n\t\t\t\t'">
I mean there is nothing in there .Is that right?


New Async Server

Implements streamlined async server for introspection, fixes #3033, fixes #3041.
Uses a single port for communication that is managed by zmq.
Uses socket notification rather than a thread in the client.
Adds a heartbeat to keep the server alive.
The main classes will be moved elsewhere and used for code analysis background tasks after beta3.

@blink1073, thanks a lot for working on this. I'd like to know what you mean by

partially address #3033

What's left to be done to solve that issue?

Also, please add pyzmq to our setup.py and conda recipe and README file, to reflect that now it is a first class dependency (instead of being inherited through qtconsole).

@ccordoba12, I'll close the other issue with this and make a new one specific to analysis tasks for clarity.

Ok, nice! Please remember that this PR also fixes #3041 :-)

And sorry for the confusion, I thought #3033 == #3041, so I was a bit worried that something was missing to fix #3041 (which is a serious issue).

@ccordoba12, done on all counts.

I verified this works on Windows 7.

This is working very nice, thanks @blink1073!
I'm just getting this error in the internal console after a completion request is performed and succeeds:
Traceback (most recent call last):
  File "/home/carlos/miniconda/envs/test-async/lib/python2.7/site-packages/spyderlib/utils/introspection/manager.py", line 125, in handle_response
    self._finalize(response)
  File "/home/carlos/miniconda/envs/test-async/lib/python2.7/site-packages/spyderlib/utils/introspection/manager.py", line 137, in _finalize
    % (self.info.name, response['name'],
AttributeError: 'NoneType' object has no attribute 'name'

This happens on Linux, with PyQ4 and PyQt5, and Python 2.7

Does the latest push address the error?

Yes, it does. But now there's a problem when I start Spyder with SPYDER_DEBUG=3: it frozes after the main window is created. Before your last change, it didn't happen that.
Could you verify it in your side?

Couldn't verify, but tried another fix, care to try again?

Thanks, your last commit fixes the freeze I was seeing. So, as soon as Travis and AppVeyor are green, I'll merge ;-)


TF2 store

I enjoy your hud it's very helpful and clean but the store part of it I can't manage to get anywhere else except for three categories can you please fix it

@ADHDEMO hi, would you mind explaining what the issue is in more detail, possibly with screenshots of what you mean by categories? i am having trouble understanding it. thank you :)

The Mann co store menu does not look like that on my end. Try to reinstall the hud and make sure it is the only hud in your custom folder. If you need additional help with this issue, pls message me on discord @skaideer
That screen is supposed to look like this:


densenet example: avoid a float64 cast

It appears that this particular mean() operation causes a float64 cast unless you tell it not to. I have warn_float64=raise in my .theanorc and so it raises an exception about it. (I'm using recent master of lasagne+theano, on ubuntu.)
I have not tested whether this change affects convergence.

I have not tested whether this change affects convergence.

It cannot, it's just to monitor performance on the validation set. It also doesn't matter whether it's float64 or float32. You should use warn_float64 during development to make sure there are no unintentional casts, but then you can disable it. I'm still fine with your change if you make it conform to pep8 (the line is too long, I'd suggest a new line with a double indent for the new mean() argument). Please amend your commit instead of adding a second one (let me know if you need help with this). Thanks!

Like this? (Formatting)
True that warn_float64 is can be disabled, but IMHO this repository's code shouldn't require users to do things like that when not necessary. Lucky the fix is easy.

Like this? (Formatting)

I was thinking about a double indent (8 chars). A hanging indent has to have at least 4 characters, but anything greater is fine (and makes it easier to distinguish from a code-level indent). I opted for a more creative hanging indent now to avoid the large horizontal gap between mean( and its argument. Merging, thanks again!


⚠ Fortnite has degraded performance

In dcfa60b, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1578 ms

Resolved: Fortnite performance has improved in b09cd63.


False positives/negatives when using disallow/requireSpaceAfterCodeOperator

Even with the recent regex improvments there are still a few cases that slip through:
e.g. a(href='/#{year}/', rel='directory')!=year reports the first attribute as an error

@xdissent: made a further change to the regex to handle examples like above. Released in v1.2.10


fits ansible separation into separate hosts

GitHub Issue: (link)

Other Relevant Links (Google Groups discussion, related pull requests,
Release pull requests, etc.)

https://github.com/Islandora/documentation/issues/1429
What does this Pull Request do?
Currently, the ansible-role-fits is written assuming a single server deployment.  Specifically, it deploys the Islandora Module (Webserver), Microservice (Crayfish?), and the FITS service (Tomcat) into Tomcat server (https://github.com/Islandora-Devops/islandora-playbook/blob/dev/tomcat.yml#L24).
This causes issues for two server/multi server deployments, and will cause an issue for ISLE 8 development.
This PR separates the ansible-role-fits into Webserver, Crayfish and Tomcat host tasks.
What's new?
As noted above, separation of above noted  ansible-role-fits role into separate target hosts.
How should this be tested?
This PR is intended for testing.  The PRs to roles will need to be merged separately, then this PR updated before it can be merged.
Single Server Testing
vagrant up.
Multi Server Testing
(One option)
server 1: webserver + database + crayfish
server 2: karaf + database + tomcat + solr
Additional Notes:
I am still testing this PR in a multi server setup.  Running into some issues :(
Interested parties
Tag (@ mention) interested parties or, if unsure, @Islandora-Devops/committers

I have tested the single server deployment (Ubuntu) for this PR and it appears to function as expected.  I added a JPEG media (as an Original, Preservation Master File) to the repository item I created and a corresponding FITS XML file was created.

@ajstanley
When we have the Crayfits on one server and Tomcat/Fedora/Karaf on another server, getting the following error.
It is sending to the right server.  The end points seems to be running as expected: "The Fits microservice is up and running.".   Thus not sure where it is failing!  No error longs for fits.
2020-02-10 15:37:43,090 | DEBUG | -connector-fits] | SendProcessor                    | 57 - org.apache.camel.camel-core - 2.20.4 | >>>> http://webserver.ca:8000/crayfits?connectionClose=true Exchange[ID-dsu-repo-1581358642031-11-1]
2020-02-10 15:37:43,090 | DEBUG | -connector-fits] | HttpProducer                     | 110 - org.apache.camel.camel-http4 - 2.20.4 | Executing http GET method: http://webserver.ca:8000/crayfits
2020-02-10 15:37:43,104 | DEBUG | iplestore-index] | HttpProducer                     | 110 - org.apache.camel.camel-http4 - 2.20.4 | Http responseCode: 404
2020-02-10 15:37:43,105 | DEBUG | iplestore-index] | BlueprintPropertiesParser        | 57 - org.apache.camel.camel-core - 2.20.4 | Blueprint parsed property key: error.maxRedeliveries as value: 10
2020-02-10 15:37:43,105 | DEBUG | iplestore-index] | BlueprintPropertiesParser        | 57 - org.apache.camel.camel-core - 2.20.4 | Delegate property parser parsed the property key: error.maxRedeliveries as value: 10
2020-02-10 15:37:43,105 | DEBUG | iplestore-index] | BlueprintCamelContext            | 57 - org.apache.camel.camel-core - 2.20.4 | Resolved text: {{error.maxRedeliveries}} -> 10
2020-02-10 15:37:43,105 | DEBUG | iplestore-index] | DefaultErrorHandler              | 57 - org.apache.camel.camel-core - 2.20.4 | Failed delivery for (MessageId: ID:dsu-repo-39667-1581358267669-3:12:-1:1:2 on ExchangeId: ID-dsu-repo-1581358642031-6-92). On delivery attempt: 0 caught: org.apache.camel.http.common.HttpOperationFailedException: HTTP operation failed invoking http://webserver.ca:8000/media/1?_format=jsonld with statusCode: 404

Looks like you have the web service on the Karaf box, so you'll need to change .env on the microservice to point to that instance.  Be default the microservice assumes it can find the web service at localhost:8080
See https://github.com/roblib/CrayFits/blob/master/.env#L29

@ajstanley that was the issue.
I am going to update the https://github.com/Natkeeran/ansible-role-crayfish PR to try to make that a variable.

@ajstanley
Templated it out the env config file.  Seems to work.  For review.  Thanks.

What's the status of this PR at the moment? It is of interest to the Multi-Tenancy Interest Group.

This all looks good (and it works on aa local build) but it will soon be largely redundant.  The plan is to move the generation of the fits xml from a web service to an executable held within the microservice.
I hope to be able to get to that in the next few weeks.


Handling of Bulk Messages and Rate Limits

We recently implemented a workaround that solves a possible rate limiting issue for Slack by collecting a bunch of rapid fire small messages and pushing them as one large message.
https://github.com/hubot-scripts/hubot-circleci/pull/12
It is unclear if we actually hit a rate limit, or if something else is wonky, but we wanted to get your input on this.

Do you perform error handling when a rate limit error message is returned via the API?
How would we catch that exception in the example I linked above?
Do you have any buffer or queue implemented that would gather up these many small messages, and resend them as a bulk message if the rate limit is hit?

How many messages are you sending when this error happens (order of magnitude answer here would be fine)?  Do you happen to have any kind of error message/stack trace/etc that might give more info on exactly what happened?

We saw the errors when we sent more than 40+ messages within a second or two. Unfortunately, hubot crashed and didn't give us any meaningful error message. You can see the errors returned here: https://github.com/hubot-scripts/hubot-circleci/issues/11

@evansolomon any word on this?

@evansolomon ?

Slack has been experimenting with rate limiting for bot users lately. As far as I can tell, these are logged:
[Tue Sep 15 2015 16:08:07 GMT-0700 (PDT)] WARNING Received rate limiting error {"code":-1,"msg":"slow down, too many messages..."}
[Tue Sep 15 2015 16:08:07 GMT-0700 (PDT)] WARNING Received rate limiting error {"code":-1,"msg":"slow down, too many messages..."}
[Tue Sep 15 2015 16:08:07 GMT-0700 (PDT)] WARNING Received rate limiting error {"code":-1,"msg":"slow down, too many messages..."}
[Tue Sep 15 2015 16:08:07 GMT-0700 (PDT)] WARNING Received rate limiting error {"code":-1,"msg":"slow down, too many messages..."}
[Tue Sep 15 2015 16:08:07 GMT-0700 (PDT)] WARNING Received rate limiting error {"code":-1,"msg":"slow down, too many messages..."}

I'm not sure yet if all the messages are going through or not. I've been able to reproduce this pretty easily with our .help command because of the sheer number of commands.  For testing, I generated a bunch of bacon ipsum, and had a script like:
  robot.respond /bacon/i, (msg) ->
    msg.send """
<PASTE BACON HERE>
"""

It does seem that if you hit this too aggressively, you'll get disconnected. hubot will reconnect, but then you run in https://github.com/slackhq/hubot-slack/issues/210

I'm also running into rate limits - we're using hubot as a central tool for a bunch of things our people are doing and as the number of people and tasks scales up, the number of responses hubot is sending is breaking the rate limit. Looks to me like we might need to implement something in the adaptor that slows down responses...

I was running with this issue on hubot 2.17.0; and the bot wouldn't reconnect it just stalled; had to do the HUBOT_SLACK_EXIT_ON_DISCONNECT="true" env variable .. nasty but the reconnection never happened.

I'm experiencing this issue too. From the documentation https://api.slack.com/docs/rate-limits, I was thinking it would be possible to modify the adapter to check the slack callback answer for HTTP 429 Too Many Requests, and if so, falling back to the sendBulkMessage until the Retry-After header is reached and we can send the bulked message. What do you think ? I'd try it but I'm a coffee and js newbie.

Agree that this is something that should be dealt with. The RTM API has its own rate limit (approx 1 message / second), and it returns an event in case you hit the rate limiter (rather than a 429 with the web APIs), that should be heeded.


Partial search returning incorrect results

55 Pyrmont" includes "55 Pyrmont Bridge Road..." in the results, but "55 Pyrmont Bri" doesn't.

Thanks @tompahoward for looking into this


Clipboard doesn't retain cut text after source application has been closed

(System: TrueOS-Desktop-2016-10-28 installed in VirtualBox; Lumina Desktop).
This problem was observed with every application I examined so far; as an example I choose Firefox and Lumina Text Editor:
Successful operation:
In the address field of Firefox, I type the word crackle_crackle. Then I cut it out with shortcut Ctrl + X and I can paste it into Lumina Text Editor with shortcut Ctrl + V successfully.
Unsuccessful operation:
In the address field of Firefox, I type the word crackle_crackle. Then I cut it out with shortcut Ctrl + X and close Firefox.
Then I try to paste the word into Lumina Text Editor with shortcut Ctrl + V, but the word isn't pasted; thus the cut word is lost.
(Analogous behavior can be observed if the cut operation Ctrl + X is replaced by the copy operation Ctrl + C).

This is how application clipboards are designed to function on X11. If you want to preserve clipboard contents across all applications, then you need to startup a "clipboard manager" application which is designed to perform this exact functionality in addition to (typically) providing clipboard history and stuff as well.
Some apps like this which come to mind for TrueOS are:

copyq (my personal recommendation -  works really well with Lumina)
gnome-clipboard-daemon (if you are using the GNOME desktop)
xclipboard (old and ugly but works)
parcellite (GTK2, but might work)


add Cj and K to website

http://beakernotebook.com/getting-started
needs installation instructions for clojure and kdb.
http://beakernotebook.com/index
add to table
http://beakernotebook.com/features
add to first section

https://github.com/twosigma/beaker-notebook-www/commit/5b8c8f2


CAPA Kubernetes v1.25

Task
Update CAPA cluster to Kubernetes v1.25.x.

Closing in favor of https://github.com/giantswarm/roadmap/issues/1994 . The Kubernetes v1.25 release will be coordinated by team Turtles.


[BUG] ValueError: Could not find a benchmark class for any of the following potential names

🐛 Describe the bug
Installed following requirements.txt in fresh env
flash-attn==2.5.7
scandeval[generative]==12.6.1
torch==2.1.2
wheel==0.43.0

Running
scandeval --model microsoft/phi-2 --language nl --use-flash-attention --raise-errors --verbose

Output
2024-04-19 13:48:40 ⋅ Benchmarking microsoft/phi-2 on the truncated version of the Dutch sentiment classification dataset Dutch Social
Traceback (most recent call last):
  File "/home/ampere/vanroy/scandeval/.venv_watt/bin/scandeval", line 8, in <module>
    sys.exit(benchmark())
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/scandeval/cli.py", line 331, in benchmark
    benchmarker(model=models)
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/scandeval/benchmarker.py", line 742, in __call__
    return self.benchmark(*args, **kwargs)
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/scandeval/benchmarker.py", line 589, in benchmark
    record = self._benchmark_single(
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/scandeval/benchmarker.py", line 693, in _benchmark_single
    dataset = self.dataset_factory.build_dataset(dataset_config)
  File "/home/ampere/vanroy/scandeval/.venv_watt/lib/python3.10/site-packages/scandeval/dataset_factory.py", line 57, in build_dataset
    raise ValueError(
ValueError: Could not find a benchmark class for any of the following potential names: dutch-social, sentiment-classification, sequence-classification.

I also tried the same for English, which lead to similar errors. I also downgraded to 12.6.0 which had the same error.
Operating System
Linux
Device
CUDA GPU
Python version
3.10.x
ScandEval version
12.6.1

Thanks for your bug report. Can you try installing from source (main branch) and seeing if the issue persists?

Yes, that does seem to work. Any idea why that is the case? Import-hell?

Thanks for the quick fix!


⚠ Adacta-fintech.com has degraded performance

In 8f346e3, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 502
Response time: 692 ms

Resolved: Adacta-fintech.com performance has improved in 3d3cc6b.


fail pipeline if git clone fails

Fixes #1357
This PR fails if can't clone the repository which enables the pipeline to give a valid error message for the user to debug in the pipeline output.

good catch! thanks for the PR.


Set up and run linting

Add Prettier plugin/config to ESLint
Set lint-staged to run both ESLint and Prettier as appropriate
Broaden lint-staged globs, because gitignored files won't be staged anyway
Lint ./tests
Sort package.json canonically

I have no idea what's up with the changes to README.md; Prettier did that.

Looks good. Maybe we should merge the changes minus eslint import until we figure out the absolute vs relative paths.

I'll make a new PR.


Seq

Add functions below:
newSeq, removeIf, getIf, get, repeat, at, compact, count

@liuyang1204 I have revised the docs, function names and some implementations. Please check.

Great!


Babel plugin master merge

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Include page no., page links, and totals in api response using pagination

Hi!!
I've read  #264 and I was looking for exactly the same thing this guy does not want.
I noticed that nowadays api generator applies LimitOffsetCriteria for paged queries, which uses limit and skip Model operations. This is is nice but it wont output pagination data, like: current page no. , total, link page, etc.
I would like to keep using LimitOffsetCriteria but include pagination data (the same output as Luciano's original message)
"success": true,   "data": {     "total": 10,     "per_page": 10,    ...    "data": [       {...},       {...},       {...},    ] }, "message": ""
Any ideas on how should I proceed? When trying to keep only data items they talked about ResponseUtil but I cannot see how it could help me now.
Note; If I change my generated api controllers to use paginate($limit ..) the response is as expected. But its not what I want.
Thanks in advance

duplicate of https://github.com/InfyOmLabs/laravel-generator/issues/264#issuecomment-444884727


doesn't work behind proxied environment

C:\streaker>streaker.exe --username booyaa
http://github.com/users/booyaa/contributions
2015/05/21 10:35:37 http: Get http://github.com/users/booyaa/contributions: dial tcp: GetAddrInfoW: No such host is known.

It does work if you have http_proxy set as an env variable. Quoting the docs for the ProxyFromEnvironment func in net/http

ProxyFromEnvironment returns the URL of the proxy to use for a given request, as indicated by the environment variables HTTP_PROXY, HTTPS_PROXY and NO_PROXY (or the lowercase versions thereof). HTTPS_PROXY takes precedence over HTTP_PROXY for https requests.


Unable to clone private GitHub repository using token

Describe the bug
GitLeaks is unable to clone a private GitHub repository using token when scanning a whole organization. It returns this error
INFO[2020-02-28T21:31:48+01:00] gathering github repos... progress: page 1 of 1
INFO[2020-02-28T21:31:48+01:00] cloning... https://mygitserver/XXXX/myrepo.git
WARN[2020-02-28T21:31:48+01:00] unable to clone via https and access token, attempting with ssh now
WARN[2020-02-28T21:31:48+01:00] unable to get ssh auth, skipping clone and audit for repo https://mygitserver/XXXX/myrepo.git: open C:\Users\user/.ssh/id_rsa: The system cannot find the file specified.
INFO[2020-02-28T21:31:48+01:00] cloning<EMAIL_ADDRESS>panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x10 pc=0x89d4c2]
I checked the github.go file but I cannot see it is passing any credential when cloning via https
To Reproduce

Create a private repository in a GitHub organization
Generate a token for one of the organization owners and give repo and read:org permissions to the token
Run gitleaks '--access-token=xxxxxxx' '--host=Github' '--baseurl=https://mygitserver/api/v3' '--org=XXXX'

Expected behavior
GitLeaks scan all the repositories in the organization
Screenshots
If applicable, add screenshots to help explain your problem.
Basic Info (please complete the following information):

OS: Windows and Linux
Gitleaks Version: 4.0.1

Additional context
Add any other context about the problem here.
cc @zricethezav

I tried to run a git clone with same token and it works.

@GolanTrevize10 mmm not sure what is going on with this one. It's passing the token to an http client right here: https://github.com/zricethezav/gitleaks/blob/master/hosts/github.go#L37-L44

But I think the http client is only used to call the github api. That part
works. But in the cloning I think it is not using token
El vie., 28 feb. 2020 22:09, Zachary Rice<EMAIL_ADDRESS>escribió:

@GolanTrevize10 https://github.com/GolanTrevize10 mmm not sure what is
going on with this one. It's passing the token to an http client right
here:
https://github.com/zricethezav/gitleaks/blob/master/hosts/github.go#L37-L44
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zricethezav/gitleaks/issues/349?email_source=notifications&email_token=AGABEMK7ACO22WVMXGHUZ6LRFF4Q3A5CNFSM4K54G6H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENKFKTQ#issuecomment-592729422,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGABEMOWFKVHYHYNPF7CT3DRFF4Q3ANCNFSM4K54G6HQ
.

https://github.com/zricethezav/gitleaks/pull/350 should fix it. Turns out I was not passing an auth method when cloning in that github loop

Yes, it works, thanks


[8.x] Allow index.blade.php views for anonymous components

For a hypothetical "accordion" anonymous Blade component system (a component with sub-components), I would love to use the following syntax:
<x-accordion>
    <x-accordion.item>
        ...

This PR allows me to use index.blade.php as a default view for an anonymous component pointing to a folder.
Here would be the folder structure

components/accordion

index.blade.php
item.blade.php

This expands on the convention of index.html/php/js inside folders.
Without this addition, I have to use redundant wrapper names like: <x-accordion.accordion>,  <x-accordion.group>, or have a file called accordion.blade.php at the same level as the accordion folder holding the sub-component views.
As far as breaking changes, because this addition is only run immediately before an error is thrown about not being able to find a view for a component, I don't see it causing any odd behavior for existing projects. I could have missed something though of course.
Thanks for your time and attention!

This is a great idea!
views
└── components
   ├── dropdown
   │   └── item.blade.php
   └── dropdown.blade.php

Better when all sub-components are in the same folder:
views
└── components
   └── dropdown
       ├── index.blade.php
       └── item.blade.php

I really like this approach and I’m really exited to use it in my projects.

This is gonna open the door for Laravel Components Market!

PSA: You may need to run php artisan view:clear for this to work 😃

@brentarcane it's a good idea to always do that any way when updating Laravel.


Add Declarative Support to Bulk Subscribe

In what area(s)?
/area runtime
/area docs
Describe the feature
RIght now, Bulk Subscribe is added only for Programmatic Way. The ask is to be able to use Bulk Subscribe in a Decalarative Way as well.
Release Note

RELEASE NOTE:

/assign

I like option 1 because it can allow Dapr to have sane defaults for maxBulkSubCount and maxBulkSubAwaitDurationMs

I like Yaron's proposal better.

Sane defaults are always a good practice to follow when possible.
Since the bulk functionality has multiple fields I think it makes sense to put it in it's own section. Can we allow something like this as a minimum?
apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
  name: orders-subscription-mqtt
spec:
  topic: orders
  route: /orders
  pubsubname: mqtt-pubsub
  bulkSubscribe:
    enabled: "true"
  metadata:
    rawPayload: "true"

In the example above, bulk subscription is enabled with sane defaults that users can override. I think this actually mimics most pub/sub message brokers today.

OK, so if I understand correctly, we will have sane defaults mentioned in dapr/dapr code and they can be overridden by common metadata, something like this?
apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
  name: orders-subscription-mqtt
spec:
  topic: orders
  route: /orders
  pubsubname: mqtt-pubsub
  bulkSubscribe:
    enabled: "true"
  metadata:
    rawPayload: "true"
    maxBulkSubCount: "4"
    maxBulkSubAwaitDurationMs: "10000"

In case of Yaron's proposal, it would move anything bulk related under bulkSubscribe:
apiVersion: dapr.io/v1alpha1
kind: Subscription
metadata:
  name: orders-subscription-mqtt
spec:
  topic: orders
  route: /orders
  pubsubname: mqtt-pubsub
  bulkSubscribe:
    enabled: "true"
    maxBulkSubCount: "4"
    maxBulkSubAwaitDurationMs: "10000"
  metadata:
    rawPayload: "true"

But yeah, there shouldn't be many changes though. We won't be changing param names ideally and there may be some additional parameters, which should be fine to add to proto.

But yeah, there shouldn't be many changes though. We won't be changing param names ideally and there may be some additional parameters, which should be fine to add to proto.

I think we need to optimize for user experience here, and if we think the YAML above is what users would expect, then as maintainers updating a proto is an implementation detail.


Allow 830 as time

goc s -t 830 "My task"
utils.go:95: unable to parse time: parsing time "2022-11-15 830 CET": hour out of range

Zeropad the -t value when the value is three digets might solve this? What do you think?

Implemented in latest commit: 8fa7cc0


Create dbt converter for Airbyte Recharge source connector

The goal of this task is to ensure the Airbyte Recharge connector output can be used to run Fivetran's dbt analytical model.
Definition of Done:

Create a pull request that adds new tables or views, enabling the execution of Fiventran's dbt with Airbyte output data.
Update the documentation inside the connector folder to explain how to configure the user dbt package to run the analytical model.

Resources:

Check example of how it was done for Zendesk Support connector

I would like to work on this.

All yours!

Hello, I did a 1st step. A few notes,

a few tables are just unnested arrays on Fivetran's side, while Airbyte keeps them nested in a parent table.
address_shipping_line on Fivetran's side seems to be a union (or other transformation) of shipping_lines_override and shipping_lines_conserved
Fivetran has order and orders for connectors created before or after 2024-06-18.
Subscription_history seems to be a model of subscription table, checked on API docs - there is no info regarding that table.

Fivetran
Airbyte

address
addresses

address_discounts
nested in addresses

address_shipping_line
nested in addresses as shipping_lines_override and shipping_lines_conserved

charge
charges

charge_discount
nested in charges as discounts

charge_line_item
nested in charges as line_items

charge_order_attribute
nested in charges as order_attributes

charge_shipping_line
nested in charges as shipping_lines

charge_tax_line
nested in charges as tax_lines

customer
customers

discount
discounts

one_time_product
onetimes

order (# applies to connectors created before 2024-06-18)
orders

orders (# applies to connectors created on or after 2024-06-18)
orders

order_line_item
nested in orders as line_items

subscription
subscriptions

subscription_history
subscriptions

Thanks for all the research @rytismi let's move to second phase. Probably the most complicated part will be the subscription and subscription_history. I'll try to get more info about them but you can start doing the transformation for the other tables.

Hey @marcosmarxm
Can I work on this issue? My PR for LinkedIn issue is already about to merge as you know...? 🙇
@rytismi seems to be inactive since three weeks , since no update/PR has been made!!

Hey @ombhardwajj, it's already merged. :D
https://github.com/airbytehq/airbyte-dbt-models/pull/56

@rytismi
Great work !!
Apologies , Missed that PR!!


When loading the contexts, an exception is raised

It works the same on two accounts, I get the following error.

The lines concerned seem to be
try:

	self.MyContexts = json.loads(re.search(r"var g_rgAppContextData = ([\s\"\\\/\-\w\d\{\}\[\]\:\.\,\']+);",text).group(1))

	self.PartnerContexts = json.loads(re.search(r"var g_rgPartnerAppContextData = ([\s\"\\\/\-\w\d\{\}\[\]\:\.\,\']+);",text).group(1))

	self.PartnerSteamID = long(re.search(r"UserThem\.SetSteamId\( \'(\d+)\' \);",text).group(1))

	self.OurEscrow = int(re.search(r"var g_daysMyEscrow = (\d+);",text).group(1))

	self.TheirEscrow = int(re.search(r"var g_daysTheirEscrow = (\d+);",text).group(1))

except:

	print tradeURLResponse.text

	raise

Could you give me the userID, app and context of this user? to check which context it has

I'm sorry everything is my fault it was a faulty parameter.

Well I reopen it since it happens again.
The App is 753, the context is 6 and the userID is Swappy<PHONE_NUMBER>7889055) (trade URL : https://steamcommunity.com/tradeoffer/new/?partner=147623327&token=jNOS2Euh)

Context guessing is based on a regular expression that catches the json string coded into the website response. This is quite error prone because I must tell the selector which characters are being expected on the json string.
I might have to change that in the future but for now It'll have to stay this way.
I've now pushed a fix for your case, alongside a better exception handler for error messages from valve


Add robots.txt setting.

@robgietema could you take a look at the suggestions and move this forward? Thanks!


[📑 Docs]: update AsyncAPI version on Parsers page

What Dev Docs changes are you proposing?
AsyncAPI has released version 2.6.0, but the page for parsers still shows 2.0.0

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@Arya-Gupta Thanks for pointing out the issue. Will you like to make a PR on this?

@akshatnema Sure! Will do.


[Dashboard] Remove default

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR primarily focuses on refactoring the layout of various components in the dashboard by replacing the Stack component with a div that uses Tailwind CSS classes for styling. This change aims to streamline the layout and improve consistency across the UI.
Detailed summary

Replaced Stack with <div className="flex flex-col gap-2"> in multiple components:

airdrop-tab.tsx
mint-form.tsx
lazy-mint-form.tsx
shared-metadata-form.tsx
update-metadata-form.tsx
transaction-timeline.tsx
NewsletterSection.tsx
HoverMenu.tsx
CancelSubscriptionModalContent.tsx
add-contract-subscription-button.tsx
permissions/add-keypair-button.tsx

Ensured consistent use of Tailwind CSS for layout adjustments across these components.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4828  👈
#4827
#4826
#4825
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kien-ngo and the rest of your teammates on  Graphite


avcc param sets with emulation prevention bytes don't parse

In the new AVCC module (thank you! super helpful), an SPS with an emulation prevention doesn't parse. e.g., this panics:
    #[test]
    fn hikvision() {
        // From a Hikvision 2CD2032-I.
        let avcc_data = hex!("014d401e ffe10017 674d401e 9a660a0f
                              ff350101 01400000 fa000003 01f40101
                              000468ee 3c80");
        let avcc = AvcDecoderConfigurationRecord::try_from(&avcc_data[..]).unwrap();
        let sps_data = avcc.sequence_parameter_sets().next().unwrap().unwrap();
        dbg!(sps_data);
        let ctx = avcc.create_context(()).unwrap();
        let sps = ctx.sps_by_id(ParamSetId::from_u32(0).unwrap())
            .expect("missing sps");
    }

I'm not sure how you'd like this to work. Is ParamSetIter supposed to yield an encoded NAL unit or RBSP? If the former, Avcc::create_context should pass it through the RBSP decoder before calling SeqParameterSet::from_bytes. If the latter, ParamSetIter should pass it through the RBSP (and I suppose needs to return a Vec or Cow<[u8]> or some such).

Seems I've also left some dbg!() in my haste to push code I worked on a few months back. :disappointed:
My preference is to have ParamSetIter yield data which is still encoded, and have the caller do the extra steps.
It's a shame that the RbspDecoder interface makes simple cases like this really complicated!  I can't see an easy way to produce Cow<[u8]> via this interface either, although that would be lovely.

Commit 7ef5c74 is a bit unsatisfactory, but gets this case working; I'll probably not have time in the short term to work out how to refactor RbspDecoder to support this better.  Sorry, it's a bit hairy!
I'd be happy to re-do these interfaces -- any suggestions would be welcome!

Thank you!

Seems I've also left some dbg!() in my haste to push code I worked on a few months back. 😞

Did you mean to fix that in 7ef5c74? It's still there.

I'd be happy to re-do these interfaces -- any suggestions would be welcome!

fwiw, the "simple cases" are all my software sees right now. It currently gets all its H.264 data from ffmpeg, which always gives it a complete "extra data" (sps + pps) or frame. So my own, not-terribly-robust NAL splitting code is super easy to use—give it some data, it calls a closure once per NALU, no dealing with state between calls. So I haven't thought through the other cases nearly as much as you.
I want to replace the usage of ffmpeg here with a (to-be-determined/written) Rust rtsp crate, which may behave differently. So I may end up caring about the complicated cases as well. But I haven't given it much thought yet. Here are some ideas off the top of my head:
I don't think it's a problem RAM-wise to buffer a whole NALU at once. I assume the concern is CPU?
How much does the naive implementation that always copies to accumulate a whole NALU in a (reused) Vec cost? Maybe you already started with this and discarded it, but I'm catching up.
If that's too expensive, how often does a single NALU get split into two push calls in practice? Maybe the mux/"outer" parsers (RTP / .mp4 / MPEG-TS / Annex-B) could have a single method that pushes some bytes with an end flag. If the whole thing is in one call, it can pass the buffer through as-is; otherwise it can copy to an internal buffer.
The RBSP layer could always expect a complete NAL. Perhaps (if the outer parser's buffer stores only one NALU), it could take a mutable buffer and shift over all the non-"emulation prevention" bytes, sort of like Vec::retain does.
I also wonder if when doing this with network protocols and tokio and such, it might be better to use the bytes crate rather than plain &[u8] / Cow<[u8]> / Vec<u8>. But I haven't actually worked with tokio yet, so I don't really know.
Speaking of the bytes crate, one of its selling points is to "facilitate zero-copy network programming by allowing multiple Bytes objects to point to the same underlying memory." So maybe my "if the outer parser's buffer stores only one NALU" caveat could be removed by using it as a convenient way of starting with some subset of the buffer, then shrinking it a bit as the emulation prevention bytes are removed.

Now removed the dbg!() in a follow-up commit :sweat_smile:
moonfire-nvr looks really cool, I will be taking a look!
My aim with the design (maybe not 100% realised yet!) was to avoid another portion of memory write+read workload at more-or-less the source video bitrate.  It's true that, depending on the size of the 'lump' that need to be accumulated, the writes+reads might not get all the way through the cache hierarchy down to RAM, but wanting to avoid cpu 'back-end stalls' (I think that's the right term?) was the motivator.  As you already know, those costs aren't a deal-breaker for lots of workloads.
I confess I've not really measured the result yet for this crate, but this is mirroring the approach in mpeg2ts-reader, which avoids accumulating the full PES-packets for the caller, and instead hands on the little fragments of PES packet from each TS-packet payload.  Benchmark results vs ffmpeg appear good (but you should never trust benchmarks!).
It all pales into insignificance if the video actually has to be decoded of course! :scream:
I sometimes use this code with higher-bitrate streams (40-50Mbps is common; 100Mbps is sometimes suggested).  My hope is that when the app is just proxying / extracting metadata, the network will be the limiting factor, rather than CPU / cache / RAM.
So in the end, the complexity arises from my desire to parse the header from slice_*_rbsp() without having to accumulate those whole NALUs.  For SPS/ PPS etc my code already accumulates the complete NALU -- I've no idea how I'd parse a partial one :)
It would be nice for this crate to support both a simple interface and a complex 'partial-input' one, with a common implementation.   Not sure how yet.
I have looked briefly at bytes a while back but couldn't quite work out if it would solve my problems.  I'll have to look again!  I took the impression that bytes adds the cost of an Rc on taking sub-slices, so I'd want to measure the cost to my precious inner-loops :smile:

I've raised #4 to track the desire to support better APIs for some of these things.

FWIW, I revised the fix for this problem in 2ce960b, since there was actually already a way mechanism to perform the required decode, but I'd forgotten about it!


Reinforcement learning

Is there any plan to have RL?
Paper:
https://arxiv.org/abs/1911.10641
Benchmark:
https://github.com/awslabs/or-rl-benchmarks

Maybe I'm reading it wrong but this seems benchmarked on instances with 2-3 vehicles, 5-10 orders?

Maybe, but state of the art in RL is around 50 cities pure TSP.
Not very useful.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le lun. 2 déc. 2019 à 13:26, CervEdin<EMAIL_ADDRESS>a écrit :

Maybe I'm reading it wrong but this seems benchmarked on instances with
2-3 vehicles, 5-10 orders?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1759?email_source=notifications&email_token=ACUPL3IOM62PXIA23US3723QWT5F5A5CNFSM4JTUYQE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFTKHUA#issuecomment-560374736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3LUCBSV5H5RXOWA76TQWT5F5ANCNFSM4JTUYQEQ
.

The largest VRP test in that Amazon team benchmark was 8 by 8 map, 5 orders
3 pick-up locations

There was also a 2018 paper comparing directly with OR-tools:
https://arxiv.org/abs/1802.04240v2

I do not trust the results. Especially, I do not trust they used the best
parameters. And I know it does not scale.
As I said before, we may incorporate RL, but for now, it is not useful.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le lun. 2 déc. 2019 à 13:44, bhack<EMAIL_ADDRESS>a écrit :

There was also a 2018 paper comparing directly with OR-tools:
https://arxiv.org/abs/1802.04240v2
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1759?email_source=notifications&email_token=ACUPL3PSFQZGAS23HJBMXVDQWT7L7A5CNFSM4JTUYQE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFTL2EI#issuecomment-560381201,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3OWJPM5DFJQKERLDLLQWT7L7ANCNFSM4JTUYQEQ
.


swift-demangle: don't crash on an old pre-swift-2 mangled name

rdar://problem/27248817

@swift-ci Please smoke test and merge


Update CIs to pull correct image based on branch

This commit/PR also serves as a test commit/PR for the newest image build of apache/age:latest

Everything looks good now.

Good job Taha!


Allow for debug and production builds of Monaco JS files

In Rust debug mode, debug build is used, thus minification and name mangling step is skipped.
In Rust release mode, JS files are minified allowing for maximum performance.
I'm not sure if this is the best approach. Maybe we should use some feature flag instead of basing on build mode?
Nevertheless we should allow for using builds with nonminified files and without names mangled, because it really helps debugging and hacking Monaco editor.

Done. And changed prod to release to be more consistent with Rust naming.

Done. And changed prod to release to be more consistent with Rust naming.

Great, I was gonna suggest that too but didn't want to be too pedantic :)


Uncaught TypeError: this.getRegexForProperty is not a function

This just suddenly happened after a morning of working on react code with no issues.
I reverted my code to an earlier state with git, tried closing and reopening atom, and restarting my computer. My react version is 15.0.1.
Here is the complete error:
/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:871
TypeError: this.getRegexForProperty is not a function
at LanguageMode.editor.languageMode.autoDecreaseIndentForBufferRow (/Users/klc34/.atom/packages/react/lib/atom-react.coffee:47:38)
at TextEditor.module.exports.TextEditor.autoDecreaseIndentForBufferRow (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:3558:32)
at Selection.module.exports.Selection.insertText (/Applications/Atom.app/Contents/Resources/app.asar/src/selection.js:499:21)
at /Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1063:29
at /Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1102:29
at TextBuffer.module.exports.TextBuffer.transact (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:866:18)
at TextEditor.module.exports.TextEditor.transact (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1519:26)
at /Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1096:24
at TextEditor.module.exports.TextEditor.mergeSelections (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:2497:18)
at TextEditor.module.exports.TextEditor.mergeIntersectingSelections (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:2459:35)
at TextEditor.module.exports.TextEditor.mutateSelectedText (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1094:19)
at TextEditor.module.exports.TextEditor.insertText (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1060:19)
at TextEditor.object.(anonymous function) [as insertText] (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/underscore-plus/lib/underscore-plus.js:77:27)
at TextEditorComponent.module.exports.TextEditorComponent.onTextInput (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor-component.js:482:26)
at HTMLDivElement. (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor-component.js:3:61)

Sorry, did not realize I need to update to 0.16.x

same issue with 0.16.1. reinstalled, same issue

Updated to 0.16.1 and still get the error, out of the blue...

Updates to 0.16.1 and getting this error too.

Updated to 0.16.1 .. after restart atom error went away.

I skirted around the problem by disabling the react package and installing language-babel 2.44.1

I've updated all the packages, restarted .. no change. Then I've restarted it agin but when noticed an anoying tab asking if I want to help to improove Atom which i previusly ignored; I clicked Yes and Crazy! - got it to work!!
Well not saying clicking on Yes fixed it.. maybe , maybe not! Just if anyone in the same situation try it.

for me, the:

Updated to 0.16.1 .. after restart atom error went away.

also worked. Thanks for the hints @chammonsvusion

I had the same issue. Started after a git reset --hard on a project listed in the tree view. To fix, I opened a new Atom window, closed the old window, and added the projects to the tree view.

Updating to 0.16.1 can fix the issue though, I'm very confused - why I need this update all of a sudden? Could still work yesterday and made no change since then..

Updating to 0.16.1 and restart Atom can fix the bug.

I had a smae issue and I have Updated to 0.16.1 and restart Atom. This has been fixed now.

Yes, you MUST restart Atom, even after upgrade to 0.16.1.

I have restarted atom and still have the issue

@fengerzh if your using react snippets, please update it to latest version. This also could be a reason why its not working.

Still having issues

😸 I was having issues as well, but went away after upgrading & close & restart atom. ✅

I have also encountered this  error message . when i upgrade rect package version to 0.16.1  . it's not reappear

After upgrading all packages and restarting the atom problem went away.

Another user confirming that updating to v 16.1 and restarting Atom solves this issue. You can close this one now :)

Same here the update to v16.1 + restart solved the issue.

Same here the update to v16.1 + restart solved the issue.

In case it wasn't obvious, from Atom, go to File > Settings and then click Packages and update react

Updating the React package to v0.16.1 and restarting the Atom editor solved for me.


Fix generation of BigQuery EDP simulator K8s configs.

This also updates pre-merge checks to ensure that the target can build successfully.

This change is


Update QuatExpMap.cpp

Replace M_PI with EIGEN_PI here too. Trying to fix a build.

Seems like a good idea; the code depends on Eigen in any case and this provides a standard way to get at PI.


chore: use dfinity/setup-dfx github action

Overview
This is to be compatible with dfxvm once it's released.
I also updated all of the workflows to use actions/checkout@v4.  This PR is split into two commits: the first updates to actions/checkout@v4, the second updates to use dfinity/setup-dfx.
Requirements
These tests should install dfx now, and after dfxvm is released.
Considered Solutions
Another option would have been to copy the path setup from https://github.com/dfinity/setup-dfx/blob/main/action.yml into the provision scripts.
# Linux
echo "$HOME/.local/share/dfx/bin" >> $GITHUB_PATH
# Macos
echo "$HOME/Library/Application Support/org.dfinity.dfx/bin" >> $GITHUB_PATH

Recommended Solution
There's a github action now, so it seems nice to use it.  Also the workflows used to use a couple different methods of overriding the dfx version.  Now there's just one:
      - uses: dfinity/setup-dfx@main
        with:
          dfx-version: 0.14.2

Considerations
This should reduce maintenance costs going forward, since https://github.com/dfinity/setup-dfx should be able to perform its part.

Closed because this change would mean one of two things:

every workflow that doesn't pin the dfx version would use the latest dfx version (no reproducibility)
every workflow would have to pin to the latest dfx version, so using a newer dfx version would mean touching every workfloow