Ombi installer broken

Ombi was recently updated from version 2 to version 3. The updater and installer in the toolkit are now broken. If you watch the update/install progress it’s trying to download and install the ARM version and it fails. I’ve tried completely uninstalling and reinstalling but it fails in the same way.

Thanks logging the issue.
Its going to need to have the install\update process changed to fix this. I will try to get around to doing it this week.

Any update on this? Really want to get it reinstalled since I uninstalled my copy when trying to troubleshoot the issue.

I'm working on it right now ;-)

FYI were still waiting on this to merge. Once that's accepted we should be in better shape to get ombi v3 into atk.

Following

--->Downloading files...
--2018-03-16 13:13:01--  https://github.com/tidusjar/Ombi/releases/download/v3.0.3030/linux-arm.tar.gz arm version?
then..
--->Extracting files...
unzip:  cannot find or open /tmp/ombidownload/.zip, /tmp/ombidownload/.zip.zip or /tmp/ombidownload/*.zip.ZIP.
No zipfiles found.
mv: cannot stat '/tmp/ombidownload/extract/Release/*': No such file or directory
--->Starting Ombi...
ombi.service is now: activating
ls /opt/ombi/
Ombi.sqlite
... this is wrong.. :) just wanna show you what´s wrong, if it might work in your computer.
Greets Postmanpaul.

Looks like the merge was committed 2 days ago. Any update on this?

The merge wasn't ready so it was reversed, however this isn't far away from being ready now :-)
I've got round all the issues I was facing with the their update scripts and it seems to be working correctly in my Dev branch.
I just need to do a couple of final tests (Arm device being one of them) and I'll push the new installer up.
couple of points to note when I do:

Currently it won't work if you have rtorrent\rutorrent installed as well. The new Ombi uses port 5000 which is the same port rtorrent uses behind the scenes. I think we might be able to get away with changing the rtorrent port to 5001. I'll submit a separate

You cannot migrate your old data over from the previous install. This is a limitation of Ombi not ATK

I also go on holiday for a week tomorrow, so  I want to push this today if I can ; -)

Please can people post if it works for them once its pushed up and if poss what they have it working on!

Thanks
Tom

Just let me know when i can test, i want to help as much as i can :)

OK its committed! Fingers crossed then, as im going away i've decided to push now and see how the community reacts!
@postmanpaul and @mrowlinson and anyone else who installs this please can you report back with how you get on.
Please note the reverse proxy isnt working in some scenarios.
Please also make sure you set an APIKEY in Ombi settings in order to check for the latest version correctly to save downloading every time you try and update. There will be improvements going forward im sure. However I think you've all waited long enough and this should be a good start to get it up and running :-)
thanks for your patience, This has been quite a tough one to get over the line.
Regards
Tom

If https://github.com/tidusjar/Ombi/issues/2070 gets solved (@tidusjar seemed open to this request) then an API key will be set automatically when it's found to be blank or missing. That's not in yet to my knowledge though. I just tested the Manual Update and it seemed to work fine (I went from Ombi v3 develop branch to Ombi v3 master by running that, upgrades from Ombi v2 should go smooth I think.)

Working for me now as well. Thanks for the update!

Damm! i like this!! thx alot!! ;D

So does the installer work now? Or just the updater?

Both should be working fine now. Are you having problems?

I don't know, I wanted to be sure before installing! :-) I'll give it a try as soon as I get back home from work, thanks for the good work!

No problem. Can u let us know how you get on? I'll probably close this issue then.

Of course, I'll let you know in 3h maximum! Thanks!

@TommyE123 It works!! Thank you so much!

Thanks for letting us know. There was a slight error in the update scripts which have now been fixed. I would suggest updating ATK if your using Ombi.

I'm having issues with the reverse proxy. Calling Ombi via internal IP works as expected, but it seems no matter what .conf settings I use I get text label issues identical to this

@Digmarx, are you ok now?
I see you didn't need the other issue you raised?

It's because you are blocking json files in your config

@tidusjar this repo is? Can you point out where? I didn't realize this. I've got a branch with an updated nginx config in the works and I've not had any issues with the config, but I'd like to make sure it's not actually causing issues.

No, this happens when users nginx config is denying access to json files, emby and ombi will break. Ombi relies on it for the translations

Yes, this was the case. On the Organizr wiki there was a bit about authentication that included blocking json/db/log files. I had configured that before installing emby and ombi and then forgotten about it. It appears that the Organizr wiki has been edited to remove that bit. So all's well, no harm to anything other than my ego :)

Thanks for letting us know.
I'm going to close this now as I think this is now working ok.
There will be some security improvements coming for reverse proxy for Ombi over the next few weeks.


RetNet : Check consistency of each forward mode

Hello authors,
I'm really happy to see this great work!
I have one question or request about the consistency of output from each forward mode.
I have been comparing three outputs by using below simple code.
import torch
from ret_net import MultiScaleRetention, RetNetRelPos

def test_msr():
    seq_len = 16
    dim = 16
    B = 3
    n_heads = 4
    chunk_size = 4
    x = torch.rand(B, seq_len, dim)
    x = torch.arange(B*seq_len*dim).view(B, seq_len, dim)
    x = x/x.max()

    xpos = RetNetRelPos(dim, n_heads, chunk_size)
    layer = MultiScaleRetention(dim, n_heads)

    # parallel
    output_p, _ = layer(x, xpos(seq_len, False, False), False, None)

    # recurrent
    output_r = []
    incremental_state = {}
    for idx in range(seq_len):
        rpos = xpos(idx+1, True, False)
        xi = x[:, idx, :].unsqueeze(1)
        out_r, incremental_state = layer(xi, rpos, False, incremental_state)
        output_r.append(out_r)
    output_r = torch.concat(output_r, dim=1)

    # chunkwise
    output_c, _ = layer(x, xpos(seq_len, False, True), True, None)

    check_diff('parallel  - recurrent', output_p, output_r)
    check_diff('parallel  - chunkwise', output_p, output_c)
    check_diff('recurrent - chunkwise', output_r, output_c)

def check_diff(name, A, B, eps=1e-6):
    D = A - B
    C = A/B
    print(name, torch.sum(torch.abs(D)))
    idx = torch.abs(D) < eps
    print(idx[0, :, 0])
    print()

And I got below result.
parallel  - recurrent tensor(6.4814e-07, grad_fn=<SumBackward0>)
tensor([True, True, True, True, True, True, True, True, True, True, True, True,
        True, True, True, True])

parallel  - chunkwise tensor(5.6081, grad_fn=<SumBackward0>)
tensor([ True,  True,  True,  True, False, False, False, False, False, False,
        False, False, False, False, False, False])

recurrent - chunkwise tensor(5.6081, grad_fn=<SumBackward0>)
tensor([ True,  True,  True,  True, False, False, False, False, False, False,
        False, False, False, False, False, False])

As you might see good agreement between parallel and recurrent results.
But the chunkwise output doesn't agree with both parallel and recurrent, after the 2nd chunk.
Could you give me a hint to understand this?
(I have already pulled the latest main branch)
Thanks a lot,
Masahiro Morinaga

Note that:
To simplify the problem, I changed the original code(MSR) below
class MultiScaleRetention(nn.Module):
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    def forward(
        self,
        x,
        rel_pos,
        chunkwise_recurrent=False,
        incremental_state=None
    ):
        bsz, tgt_len, _ = x.size()
        (sin, cos), inner_mask = rel_pos

        q = self.q_proj(x)
        k = self.k_proj(x)
        v = self.v_proj(x)
        g = self.g_proj(x)

        k *= self.scaling
        q = q.view(bsz, tgt_len, self.num_heads, self.key_dim).transpose(1, 2)
        k = k.view(bsz, tgt_len, self.num_heads, self.key_dim).transpose(1, 2)

        qr = theta_shift(q, sin, cos)
        kr = theta_shift(k, sin, cos)

        if incremental_state is not None:
            output, incremental_state = self.recurrent_forward(qr, kr, v, inner_mask, incremental_state)
        elif chunkwise_recurrent:
            output = self.chunk_recurrent_forward(qr, kr, v, inner_mask)
        else:
            output = self.parallel_forward(qr, kr, v, inner_mask)

        #output = self.group_norm(output)
        output = output.reshape(bsz, tgt_len, self.head_dim * self.num_heads)
        # output = self.gate_fn(g) * output
        # output = self.out_proj(output)
        return output, incremental_state

I notice that you comment on the following line:
#output = self.group_norm(output)

It is essential to keep identity among different forms. Besides, to check the consistency, we usually set eps=0 in group_norm. The inconsistency comes from small weight values and eps, where the small initialization will be fixed after training.

Thanks a lot for your comment.
I see. Now I understand what's going on.

Hi,
I confirmed three outputs are exactly the same after the group_norm.

Thanks a lot for your comment. I see. Now I understand what's going on.

Hi. Did you check the consistency again? I ran your code and fixed the group_norm thing. There is still a large diff between parallel forward and chunkwise forward.

You should use self.group_norm = MultiwayWrapper(args, RMSNorm(self.head_dim, eps=0, elementwise_affine=False)) # Check consistancy  instead of self.group_norm = MultiwayWrapper(args, RMSNorm(self.head_dim, eps=args.layernorm_eps, elementwise_affine=False)). Then all the output will be True

Same question. When eps=0, the consistency will be great, but the training stability will be a problem. Luckily, RMSNorm is much more stable than LayerNorm, then the eps could be much smaller.

In the paper, it said GroupNorm replaces LayerNorm, but in the code, it uses RMSNorm?

In the paper, it said GroupNorm replaces LayerNorm, but in the code, it uses RMSNorm. Looks like the goal is to normalize each head separately? Could you clarify on that part?

@Dao007forever We used LN in the experiments of our arxiv paper. In our latest experiments, we found that RMSNorm is more stable, especially to the LN.eps. So we switch the default one to RMSNorm.


Replace obsolete 'compile' with 'implementation'

Latest Gradle requires dependencies to use 'implementation'.

@Kishanjvaghela do you have a plan work on this?

#59 Fixed


fix: umbrella bonuses are doubled by offhand remarkable

As reported in https://kolmafia.us/threads/august-scepter.29095/#post-173259.

Not sure if I'm reading right but can we expand this to check for all offhand modeables or something?

I don't think there are any other ones at the moment?
Also I don't think we'll know whether new modeables will be doubled until they come out.

The way that modeables are implemented in KoL leads me to believe that it makes more sense to assume that they will be supported.


Set max length to 1

#27
Set TextInput max length as 1

@vinitkadam Hi :) Does this work with copying? Because I've had an issue with that one and that's why I've left it untouched.

what exactly do you mean by copying? It works as expected for me.

@vinitkadam Copy-pasting otp code, is working as expected?

Yes. Copy-pasting otp code is working as expected.


Improve preserveCertOrder description

Improve preserveCertOrder field descriptions based on feedback from API review board.

The LintDiff issues are known and expected.


Add Idiomatic Claude XML template

Add exemplar template to demonstrate working with Claude's xml instruction tuning.

This is a good template. Thank you for contributing! ❤ 🙌


make maxAngle = - minAngle, let gmapping work properly

In this question, someone reported the issue about gmapping
http://answers.ros.org/question/211165/gmappping-problem-no-map-received/
The issue seems solved after this fix.

Now we can work well using gmapping.  I will close this PR, if you have any question, i will reopen it.


I think you should provide some hints for the questions?

I got stuck in this question, "Quantifiers", but there is no hint on how to solve it.

I got stuck on a part too, so I just looked into the source code by using Chrome Developer Tools to see the javascript code for the answers that he was using. In my particular case, I found that my regex was correct but that I shouldn't use the answer() call. I read the question wrong.
I don't know if I'm allowed to post the link, so I won't. If you know HTML, then you can (eventually) find it.

The showAnswer() function will show you the answer :)


Display data from app JSON files in a user friendly view

During start_release we will now auto-generate readme.md by converting the readme.html document to markdown, and then combining the output with data (eg, configuration variables, action details) from the app's JSON in a user friendly view.
Testing:

Automated tests
Manually inspected auto-generated content https://github.com/jdemelo/ttt/blob/main/qradar-readme.md
Manually tested in autofocus repo
Forked gitlab repo and ran start_release successfully
Manually inspected the PR generated by start_release. The readme.md file looks pretty good.

It might be confusing introducing author_notes.md while readme.html is still in commission - assuming we convert author_notes.md to readme.html, what are your thoughts on introducing this change after we automate the author_notes.md --> readme.html conversion?
Could you also update https://github.com/splunk-soar-connectors/.github/blob/main/.github/CONTRIBUTING.md and https://github.com/splunk-soar-connectors/.github/blob/main/.github/pull_request_template.md to reflect the new documentation process? I can see people accidentally updating readme.md instead author_notes.md going forward

We had a good offline discussion about this comment and more. This PR has been updated following the decisions made, and is ready for re-review.


Update AppImage build process

Use Python 3.11
Build using python -m build

Contributor Checklist:

[ ] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed
[ ] No import of GPL code from MIT code

/azp run libertem.libertem-data

data_tests Python39 fails in test_scatter_update[pipelined] - looks like the same failure as #1458 - I'll investigate.

/azp run libertem.libertem-data


No Points Raw in Quick Start. Somebody help me. For frame [velodyne]: No transform to fixed frame [world].

For frame [velodyne]: No transform to fixed frame [world].  TF error: [Lookup would require extrapolation at time<PHONE_NUMBER>.666513920, but only time<PHONE_NUMBER>.152538880 is in the buffer, when looking up transform from frame [velodyne] to frame [world]]

TF

Yeah. I know it's TF error. But I do follow the Quick Start guide exactly. How can this error happen and how can I fix it. Can you help?

Please try to click "Reset" button on RViz. Sometimes, TF error is fixed by clicking "Reset" button.

Thank you for your help, but it does not work. The problem is still there. When I use roswtf, it says the ndet matching died. What's radiculous, I do the same processures on two different laptops; the former one works well, but the latter one comeup with this problem.

Still not working? Please check https://github.com/CPFL/Autoware/wiki/Demo and try again.


More accurate determination of used etcd image repo:tag

I very much appreciate your PR. Sorry for the very very late reply. I did not get the time to test it and then forgot :(
Thanks to 1.14, the entire headache with etcd has been closed in an elegant way: it's done by kubeadm.

Thanks for the answer anyway - I absolutely agree with that :)


metrics(ticdc): add resolved ts and add changefeed to dataflow

What problem does this PR solve?

fix etcd worker txn size heatmap
add owner resolved ts metrics
add changefeed to dataflow
add PD uptime
add PD leader history
add etcd metrics, eg, mvcc db size, etc.

Check List
Tests

Manual test (add detailed scripts or steps below)

Import to grafana.
Related changes

Need to cherry-pick to the release branch

Release note
None

Codecov Report

Merging #4038 (f345fea) into master (3873d39) will decrease coverage by 1.9268%.
The diff coverage is 64.0553%.

Flag
Coverage Δ

cdc
58.1101% <64.0553%> (-0.1265%)
:arrow_down:

dm
52.6427% <ø> (-3.3919%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4038        +/-   ##
================================================
- Coverage   57.0741%   55.1472%   -1.9269%
================================================
  Files           478        484         +6
  Lines         56551      59002      +2451
================================================
+ Hits          32276      32538       +262
- Misses        20978      23165      +2187
- Partials       3297       3299         +2

/merge

/run-integration-test
/run-dm-integration-test

/run-leak-test
/run-verify

/run-dm-integration-test


Short solution needed: "Boston dataset" (python-scikit-learn)

Please help us write most modern and shortest code solution for this issue:
Boston dataset (technology: python-scikit-learn)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to load Boston housing dataset


Refinements to Properties

Is your feature request related to a problem? Please describe.
I suggest these refinements to the properties especially useful when comparing folders/files.
Currently I just can see the size of one file but not for folders and or when selecting more than on folder/file.
Describe the solution you'd like
I would like to see the overall size of a folder and/or files (and the count of folders and files inside this folder).
Describe alternatives you've considered
Also when choosing more than one file and/or folder, I would like to see the count of files and the overall size.
Additional context
This is how it is handled in file explorer:

This proposal seems to include all the information we need so I will go ahead and approve this for development.

Would be really nice, if you would also add an option to the right-click menu to calculate the size of current selected folders directly inside the list view (per folder) , so that you are afterwards able to sort all files and folders based on their size.
If you change something inside this folder, or navigating away from the current path, I imagine that it gets resetted.

Have the points brought up in this issue been resolved in the recent updates? If not please let me know which things still need to be addressed, thank you 🙂.

@yaichenbaum All points are addressed for single files but they don't work for a selection of files/folders which is still very important imo.
Also the size of a folder is missing from this dialog (when showing the properties of only one folder).

@szaimen The size for folders is a different issue, #645 that we will work on in the future. I will keep this issue open since you mentioned you still need it to support showing the properties with the total size of all the selected items.

This issue has been resolved with recent pull requests.


Redundant: KSubsetPartitionType and PyCObject for SubsetPartition?

We have khmer_KSubsetPartitionType, but we also use PyCObject in some places to pass SubsetPartition around. Should we keep just one for consistency?

I vote for keeping khmer_KSubsetPartitionType in favor of PyCObject

Absolutely agree w/@mr-c.

[ ] Need CPython type for pre_partition_info

Fixed in #785.


Blog writing

Create your own blogs on Amazon alexa and how to create alexa skills and put them in the Blogs folder.
They must be an MD file with your name and credentials at the bottom.

Hi Garima, I would like to work on this issue.

Hi Garima, I would like to work on this issue.


Transpiler is unable to map circuit with delay with V2 backend

Environment

Qiskit Terra version: main
Python version: 3.9.1
Operating system: OSX

What is happening?
Attempting to transpile a circuit with a delay fails
How can we reproduce the issue?
hub = ""
group = ""
project = ""
backend_name = "ibm_peekskill"

from qiskit_ibm_runtime import QiskitRuntimeService

service = QiskitRuntimeService()

backend = service.backend(backend_name, instance=f"{hub}/{group}/{project}")

qc = QuantumCircuit(1,1)
qc.delay(1000, 0, "dt")
transpile(qc, backend, initial_layout=[0])

Gives "Unable to map source basis {('delay', 1)} to target basis {'snapshot', 'x12', 'barrier', 'xprs', 'x', 'id', 'cx', 'sx', 'measure', 'rz', 'reset'} over library <qiskit.circuit.equivalence.EquivalenceLibrary object at 0x7f9b889027c0>."
What should happen?
Transpilation should succeed.
Any suggestions?
No response

delay isn't in your Target for that backend. The transpiler is unable to convert a delay to anything since it's intrinsic operation so if the backend doesn't say it supports delay the transpiler will error because it has an instruction in the circuit it can't convert via an equivalence rule and the backend doesn't support. The backend implementation needs to add delay to the target for this to work. I did this in the qiskit-ibm-provider, I'm guessing the qiskit-ibm-runtime doesn't have delay in the target it builds

This is how we fixed this in qiskit-ibm-provider: https://github.com/Qiskit/qiskit-ibm-provider/blob/bd9ec5180be8db51dc08d8314446e89d2c5f4cfa/qiskit_ibm_provider/utils/backend_converter.py#L135-L138 I assume qiskit-ibm-runtime is missing this


BuildQueue

This is primarily an internal refactoring to support multiple scale out methods:

The first scale out method is basically what we're doing now: you run conveyor server and all builds are performed in goroutines talking to a single docker daemon. Scaling out can be achieved using something like docker swarm to scale out the docker API.
The second scale out method is via a BuildQueue. This will allow you to provide a queue implementation (sqs will probably be the first), allowing you to run a separate conveyor worker subcommand. The idea here is that conveyor server pushes build requests into a queue, and then individual workers pull jobs off and run the build with their local docker daemon.

The first method is probably the easiest, but I think using an actual queue and multiple worker nodes might be more robust in the long run. This at least gives users a couple options.

:+1:


Infinite loops with react-async-component

I've been trying to deal with infinite loops and other issues on react-async-component, which is also referenced on their issue list: https://github.com/ctrlplusb/react-async-component/issues/42 (another one is here: https://github.com/styled-components/styled-components/issues/762).
They are very difficult to diagnose, but I wanted to share one issue that might be common on react-universally. If you reference a config value inside an async component and that value references a process.env.variable, you will get an infinite loop. I'm not a 100% if this applies to all cases, since I had to run to fix this and redeploy to production.
In my case, this does not show up during development, but does show on deployment to a Dokku container. Kinda tricky to deal with.

I'm having the same issue.
Sometimes it happens in my machine, but with a reload is resolves. But it's still a problem!
Haven't found any solution yet.

New versions are being released. Please let me know if it is still an issue 👍


Add HMDB51 and UCF101 datasets

This PR add support for some more video datasets.
I was about to add downloading code for both of them until I realized that they use .rar files, for which there is no standard Python implementation available, so I dropped the idea.
This PR needs documentation and tests, but together with the changes in https://github.com/pytorch/vision/pull/1155 everything seems to work fine.
cc  @bjuncek

Codecov Report

Merging #1156 into master will decrease coverage by 0.56%.
The diff coverage is 31.32%.

@@            Coverage Diff             @@
##           master    #1156      +/-   ##
==========================================
- Coverage   65.65%   65.09%   -0.57%
==========================================
  Files          70       72       +2
  Lines        5527     5610      +83
  Branches      853      864      +11
==========================================
+ Hits         3629     3652      +23
- Misses       1639     1698      +59
- Partials      259      260       +1

Impacted Files
Coverage Δ

torchvision/datasets/__init__.py
100% <100%> (ø)
:arrow_up:

torchvision/datasets/ucf101.py
28.2% <28.2%> (ø)

torchvision/datasets/hmdb51.py
30.95% <30.95%> (ø)

torchvision/transforms/transforms.py
80.94% <0%> (-0.59%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks good to me (needs docs), but I am starting to wonder (and this goes for kinetics dataset as well) if we should include cacheing of the VideoClips object on the dataset level, or should we handle that in the training code?

@bjuncek I'm not yet sure where the caching should be handled.
I'll tackle this in a follow-up PR, where I'll be adding reference training scripts, and this will become a must to have.
I'll be merging this as is for now, and tests + docs (together with the transforms) will be added later.


[6.0] Deprecate namespace trait

Replaces https://github.com/laravel/framework/pull/29528
This trait isn't used so it doesn't makes sense to ship with it out of the box. Deprecating now and removing it in v7.0

Seems odd to have a new semver major release already with deprecated code in.

@GrahamCampbell taylor wanted to give people some time before it was removed: https://github.com/laravel/framework/pull/29528


Bug: wrong parsing for math

marked version: 0.3.6
My markdown code:
$B_{p_{1}}+B_{p_{2}}$
But when I try to parse into HTML using
$('#content').html(marked(mdString));
It returns wrong HTML code. Unexpected  occurred:

$B{p{1}}+B{p{2}}$

See also #936

You shouldn't put LaTeX math formula in markdown, since they are not supported and are instead interpreted as markdown.
If you want to keep the latex code as is, wrap it inside a `code` element:
`$B_{p_{1}}+B_{p_{2}}$`

Adding this to the 0.5.0 miestone as math is something that comes up quite a bit, but does not seem to specifically addressed in the specifications we are focusin on (see #956). Closing.

It's rude but solved my problem in a way...
function replace(content, regx) {
    content = content.replace(regx,
        function(expression) {
            var max_formula_length = 200;
            if (expression.length > max_formula_length || has_chinese(expression)) return result;
            expression = expression.replace(/\\\\/g, '\\\\\\\\');
            expression = expression.replace(/\\ /g, '\\\\ ');
            expression = expression.replace(/\\%/g, '\\\\%');
            expression = expression.replace(/\\{/g, '\\\\{');
            expression = expression.replace(/\\}/g, '\\\\}');
            expression = expression.replace(/\\#/g, '\\\\#');
            expression = expression.replace(/\\~/g, '\\\\~');
            expression = expression.replace(/\\_/g, '\\\\_');
            expression = expression.replace(/\\&/g, '\\\\&');
            expression = expression.replace(/\\\$/g, '\\\\$');
            expression = expression.replace(/\\\^/g, '\\\\^');
            expression = expression.replace(/\\\|/g, '\\\\|');
            expression = expression.replace(/\_/g, '\\_');
            return expression;
        });
    return content;
}

var data = replace(raw_data, /\$\$([\s\S]*?)\$\$/gm);
data = replace(data, /\$([\s\S]*?)\$/gm);
data = marked(data);

For example:
var raw_data = '$$\\mathcal{L}_C=\\frac{1}{2} \\sum_{i=1}^{m} \\| x_i-c_{y_i} \\|_2^2$$';
var data = replace(raw_data, /\$\$([\s\S]*?)\$\$/gm);
data = replace(data, /\$([\s\S]*?)\$/gm);
data = marked(data);
console.log(data);

output:

$$\mathcal{L}_C=\frac{1}{2} \sum_{i=1}^{m} \| x_i-c_{y_i} \|_2^2$$


Not able to handle the Invalid Domain - DomainLock

Hi,
I am using Domain Lock feature, and when the domain is incorrect if throws this error:
Uncaught ReferenceError: data is not defined     at out.js:1     at _0xd895e (out.js:1)     at out.js:1
Is there any way to handle this? I need to show popup which says "Invalid Domain". Is there any way to achieve this?
Thanks in advance.

Uncaught ReferenceError: data is not defined at out.js:1 at _0xd895e (out.js:1) at out.js:1

It's by design. This error should confuse "hacker". No, currently no way to change this behaviour.

If you desperately needs this feature, you can always see how it's originally implemented, copy and adapt to show your alert and paste it inside your original code. Then, you obfuscate normally without this option.
Keep in mind that, as @sanex3339 said, if you place an alert or something like that it will become easier to figure out where the protection is happening and to remove it.

Good point guys, i didn't think of this.
Thanks for investing time.


Lameduck mode does not work in 'no_sandbox' mode.

Observed behavior
hi,
After running node as a lameduck
There is no way to run a new nex node up & (background mode)
[INF] 2024-07-01 15:44:57 - Publishing node lame duck entered event

lameduck mode starts and doesn't end.
the only way is to forcefully terminate the nex process.
Expected behavior
The node should shut down gracefully in no_sandbox mode.
Nex and NATS version
nex --version
v0.2.5 [27f46b431de7cde09282049d86f64874ebcbd1e5] | Built-on: 2024-06-14T20:34:24Z
Host environment
No response
Steps to reproduce
No response

Hi @NoahHahm.  The command behaves this way on purpose in order to allow you to move workloads from one node to another.  We have talked about shutting down the node once no workloads were detected, but we haven't implemented that yet.  However, to your point, in order to make it match the behavior of nats-server, I expect we will eventually make the behaviors mirror one another.  I will leave this issue open as a tracker...let us refine some of the "workload migration" documentation, then we can revisit this feature at that time


an id must be provided if version type or value are set;

hi there,
I am new in elastalert. I got this error after making my first alert. anybody can help?
ERROR:root:Error writing alert info to Elasticsearch: TransportError(400, u'action_request_validation_exception', u'Validation Failed: 1: an id must be provided if version type or value are set;') Traceback (most recent call last): File "/opt/elastalert/elastalert/elastalert.py", line 1018, in writeback doc_type=doc_type, body=body) File "build/bdist.linux-x86_64/egg/elasticsearch/client/utils.py", line 69, in _wrapped return func(*args, params=params, **kwargs) File "build/bdist.linux-x86_64/egg/elasticsearch/client/init.py", line 248, in create return self.index(index, doc_type, body, id=id, params=params, op_type='create') File "build/bdist.linux-x86_64/egg/elasticsearch/client/utils.py", line 69, in _wrapped return func(*args, params=params, **kwargs) File "build/bdist.linux-x86_64/egg/elasticsearch/client/init.py", line 279, in index _make_path(index, doc_type, id), params=params, body=body) File "build/bdist.linux-x86_64/egg/elasticsearch/transport.py", line 327, in perform_request status, headers, data = connection.perform_request(method, url, params, body, ignore=ignore, timeout=timeout) File "build/bdist.linux-x86_64/egg/elasticsearch/connection/http_requests.py", line 84, in perform_request self._raise_error(response.status_code, raw_data) File "build/bdist.linux-x86_64/egg/elasticsearch/connection/base.py", line 113, in _raise_error raise HTTP_EXCEPTIONS.get(status_code, TransportError)(status_code, error_message, additional_info) RequestError: TransportError(400, u'action_request_validation_exception', u'Validation Failed: 1: an id must be provided if version type or value are set;') INFO:elastalert:Ran Example rule from 2016-11-06 11:32 IRST to 2016-11-06 11:47 IRST: 0 query hits, 0 matches, 0 alerts sen
and this is my example_frequency.yaml file :
es_host: localhost
(Optional)
Elasticsearch port
es_port: 9200
(OptionaL) Connect with SSL to Elasticsearch
#use_ssl: True
(Optional) basic-auth username and password for Elasticsearch
es_username: *****
es_password: *****
(Required)
Rule name, must be unique
name: Example rule
(Required)
Type of alert.
the frequency rule type alerts when num_events events occur with timeframe time
type: frequency
(Required)
Index to search, wildcard supported
index: metricbeat-*
(Required, frequency specific)
Alert when this many documents matching the query occur within a timeframe
num_events: 50
(Required, frequency specific)
num_events must occur within this amount of time to trigger an alert
timeframe:
hours: 1
filter:
term:
#some_field: "some_value"
host: 'something'
#count: 1
(Required)
The alert is use when a match is found
alert:
"email"
(required, email specific)
a list of email addresses to send alerts to
email:
<EMAIL_ADDRESS>

Are you using ES 5????

Looks related to https://github.com/elastic/elasticsearch/issues/21535 if you are using version 5.

thanks Ryan.
but would you please say me a solution? I used elasticsearch 5 but I
downgrade its version to 2.* . I will be so happy if you help me.
On Nov 15, 2016 8:59 PM, "Ryan Kyser"<EMAIL_ADDRESS>wrote:

Looks related to elastic/elasticsearch#21535
https://github.com/elastic/elasticsearch/issues/21535 if you are using
version 5.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yelp/elastalert/issues/797#issuecomment-260708699,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATqEr7bvljSaq3hr64PNcyDkQAyCsfbJks5q-ev2gaJpZM4Kqg4W
.

yes I am. what can I do to resolve this issue? thank ls a lot.
On Nov 11, 2016 1:27 PM, "ravi45722"<EMAIL_ADDRESS>wrote:

Are you using ES 5????
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yelp/elastalert/issues/797#issuecomment-259922606,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATqEr4zMsB9uSFISlIiq_He5rerC9XrVks5q9DwHgaJpZM4Kqg4W
.

same here!

I hit the same issue today.  I like to use ES 5.0 with elastalert, I wonder how can I workaround of this?

Hi there,
I find that client.create() actually requires an id now, but if you use
client.index() instead it will auto-generate the id.
so if you change this line
line 123 : es.indices.create(index)
in this file create_index.py it will be ok.
I change this line  to

es.indices.index(index)*
and then I run this command : python setup.py install
but I got this error :
Traceback (most recent call last):
File "/usr/local/bin/elastalert-create-index", line 9, in
load_entry_point('elastalert==0.1.3', 'console_scripts',
'elastalert-create-index')()
File "/usr/local/lib/python2.7/dist-packages/elastalert-0.1.
3-py2.7.egg/elastalert/create_index.py", line 124, in main
es.indices.index(index)
AttributeError: 'IndicesClient' object has no attribute 'index'

could you please help me?
On Tue, Nov 22, 2016 at 4:41 AM, Qingsong Yao<EMAIL_ADDRESS>wrote:

I hit the same issue today. I like to use ES 5.0 with elastalert, I wonder
how can I workaround of this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yelp/elastalert/issues/797#issuecomment-262118189,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATqErwvZbigjHcLfWS3mxbUeswSnNyN1ks5rAkE0gaJpZM4Kqg4W
.

--
best regards
Ehsan Bayandor

This branch contains es5 support. https://github.com/Yelp/elastalert/pull/820

Ah, i got the same problem using stroongloop, figuring out how to do the 'index' thing

In ES 5.x, use index not create,  the _id will auto generate

@kwdfmzhu thanks


I just added an example of multi-agent orchestration @MonolithFoundation @whisper-bye!

          I just added an [example of multi-agent orchestration](https://github.com/huggingface/smolagents/blob/main/docs/source/examples/multiagents.md) @MonolithFoundation @whisper-bye!

Tell me what you think!
Originally posted by @aymeric-roucher in https://github.com/huggingface/smolagents/issues/3#issuecomment-2566327823
I could not find it, it gives a 404 error

The link has been updated to https://github.com/huggingface/smolagents/blob/main/docs/source/en/examples/multiagents.md


http://<IP_ADDRESS>/arkos is not accessible from China

I'm trying to do OTA from china, but the http://<IP_ADDRESS>/arkos is not accessible. Is this normal?

Yes as that address no longer exist.


Max limit connections

Hi,
is it possible to set the software to run max 2 connections?
thanks

Use Google to find the solution, the issue section is for technical issues, not for support.

Sorry, this is a support question. I just need to know if .ini file can be modified to limit only 2 simultaneous connections.

Just try it, I didn't tested this though.

But I don't know what settings I have to edit to make it work.
On 9 February 2018 at 14:03, Stanislav Motylkov<EMAIL_ADDRESS>wrote:

Just try it, I didn't tested this though.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stascorp/rdpwrap/issues/408#issuecomment-364441855,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aij_chc1_O9AGc4hAwAeF3nXBaX2aM3Qks5tTFA5gaJpZM4R9K3F
.


⚠ PrivateHD has degraded performance

In 9e3bcc6, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1577 ms

Resolved: PrivateHD performance has improved in 3cb95b0.


Support Notation image signing

What is the problem you're trying to solve
Notation is a tool for signing and verifying signatures based on Notary V2 standard. Nerdctl already supports cosign for signing feature. I’d like to see nerdctl also supporting notation so users have one more option to use for signing and signature verification.
Keyless is specific to cosign today so we will skip discussing keyless by default in the following discussion.
Describe the solution you'd like
The current cosign signing and verification interfaces of nerdctl are like these:
nerdctl push --sign=cosign --cosign-key cosign.key devopps/hello-world
nerdctl pull --verify=cosign --cosign-key cosign.pub devopps/hello-world-bad

Making it generic, the interfaces will be:
nerdctl push --sign=<signing-tool-enum> TOOL_SPECIFIC_ARGS IMAGE
nerdctl pull --verify=<verifying-tool-enum> TOOL_SPECIFIC_ARGS IMAGE

The interfaces of cosign are
cosign sign --key cosign.key $IMAGE_URI_DIGEST
cosign verify --key cosign.pub $IMAGE_URI

So it is intuitive to add --cosign-key as TOOL_SPECIFIC_ARGS in existing cosign support.
However, the interface of notation is much different.
In notation, the private key must be added by notation key command first to be used by notation sign, and the private key can only be added with plugins. Based on notation’s response, the only supported plugin today is Azure Key Vault. The notation key command actually manages signing-keys.json file in notation store. The users can also put local keys in signing-keys.json manually. The local keys are just not supported in command yet. (Issue)
notation key add --plugin <plugin_name> [flags] <key_name>
notation sign --key <key_name> <reference>

To validate the signature, instead of using public key, users need to add the cert to the trust policy. notation verify will directly use the cert in trust policy to verity the image.
cat <<EOF > $HOME/.config/notation/trustpolicy.json
{
    "version": "1.0",
    "trustPolicies": [
        {
            "name": "wabbit-networks-images",
            "registryScopes": [ "$REGISTRY/$REPO" ],
            "signatureVerification": {
                "level" : "strict"
            },
            "trustStores": [ "$STORE_TYPE:$STORE_NAME" ],
            "trustedIdentities": [
                "x509.subject: $CERT_SUBJECT"
            ]
        }
    ]
}
EOF

notation verify $IMAGE

Signing
For signing, there are multiple options.
Recommended solution
Users will do any operations before notation sign by themselves. nerdctl push only integrates notation sign.
The experience will be
notation key add --plugin <plugin_name> [flags] <key_name> # or change the signing-keys.json
nerdctl push --sign=notation --notation-key <key_name> devopps/hello-world

The concern is that adding any non-nerdctl steps to be part of workflow is not ideal. However, considering the operation is actually managing signing-keys.json, it is consistent with managing trust policy in verification. Notation users will also expect the additional steps required.
Alternative thoughts

Users will do any operations before notation key by themselves. nerdctl push integrates notation key and notation sign.

The experience will be
nerdctl push --sign=notation --notation-plugin <plugin_name> PLUGIN_ARGS devopps/hello-world

The concern is there will be some logic to clean up the keys, which will cause race conditions for concurrent pushes.

Similar to recommended option, but add a nerdctl command to manage key.

nerdctl <notation-key> add --plugin <plugin_name> [flags] <key_name>
nerdctl push --sign=notation --notation-key <key_name> devopps/hello-world

It will an additional command outside regular container development command set. And it will be only useful for one signing option of signing feature (cosign doesn’t need it).

Integrate plugin calls to nerdctl command. There may be multiple plugins in the future, so it is good to keep plugin calls flexible and managed by users.

Signature verification
For signature verification, the only option is, users will do any operations before notation verify by themselves. nerdctl pull only integrates notation verify. It is not possible to change the trust policy for them because we need to support users running multiple nerdctl pull in parallel. Changing the trust policy for them will cause race conditions if they are pulling images with different public keys/certs.
# Configure trust policy first

nerdctl pull --verify=notation devopps/hello-world-bad

Can we close this? @ningziwen @djdongjin

@ningziwen, this can be closed, right?


Maintenance status of this project

Is this project still maintained by the guild? Couldn't get any answer for a month and also the PR's aren't merged or at least commented.
In general I really like the approach of loonajs but due to the lack of human interaction in this project I am not sure if it is already abandoned because of another approach.
Thanks for clarification.
I am not sure who of the guild are the maintainers so I mention @Urigo because of NgAtlanta and @kamilkisiela because he hosts the project.

I have the same question. I consider loona to be a really good project and want to start using it, but not sure if it is still maintained :(
Hope @kamilkisiela will give an answer soon!

I'm interested as well. I'm working in this space now with a different framework, and loona seems to be a perfect extension, however without active maintenance it will be hard to bring that in.
I'm happy to help with maintenance/contribution side.

I'm also interested to help with this project, this is a promising project, I'm really wanting to replace my current state management tool that is @ngxs/store to use this project to unify the way I fetch data from my Apollo Graphql server and also using this tool to manage my local state and also fetch data from Firebase/Firestore using @loona/angular.
@kamilkisiela and @Urigo, is there a way we can help you guys with this project? I have some available time and I would love to help you guys to maintain and contribute to this project.
Or you guys are working on some other project/idea to solve this same problem in another way that we are not aware?

Hi everyone and sorry for not replying earlier.
The current state is that we are working on a big tutorial in order to demonstrate all of our libraries, where and why should you use them.
Once that will be done we will address all issues and demonstrate it there.
The best way to help is to actually try the library in your projects and report real issues you have.
Instead of talking about general PRs or general issues and about general maintainability, show us what you are actually doing and needing from us.
We don't want to merge some random updates just to show movement before the tutorial is done, but we will merge PRs and respond to issues from actual usage, so those would be very helpful.
When you will actually build something with Loona and get stuck, you can even reach out to me directly, my email is on my public Github profile.
Thank you for your support and help!

@Urigo that's great to hear, thx for the update. In fact there are already PR's (https://github.com/kamilkisiela/loona/issues/525) where contributors are start living with a fork and closing the issue just because no one is merging the PR's here. I guess this is a movement we should all try to avoid.
I am looking forward to the big tutorial with loonajs.


Recovering from CameraOpenException

Hi Pedro,
I am currently working on allowing users to set their parameters to be used with the prepareVideo function. I ran into the following when the parameters are not entirely correct:
2019-03-14 18:00:20.755 29513-29513/nl.tno.flame.prosumer.debug E/ProducerFragment: com.pedro.encoder.input.video.CameraOpenException: This camera resolution cant be opened
        at com.pedro.encoder.input.video.Camera1ApiManager.start(Camera1ApiManager.java:121)
        at com.pedro.encoder.input.video.Camera1ApiManager.start(Camera1ApiManager.java:109)
        at com.pedro.encoder.input.video.Camera1ApiManager.start(Camera1ApiManager.java:116)
        at com.pedro.rtplibrary.base.Camera1Base.startEncoders(Camera1Base.java:386)
        at com.pedro.rtplibrary.base.Camera1Base.startStream(Camera1Base.java:372)

What I try do to is catch the exception and continue as usual. However, after the CameraOpenException, the preview remains black and does not return to normal.
What I've tried is to call the startPreview and prepareVideo functions again, but the openglview remains black. Do you have any ideas ?

You should use this methods to know resolutions supported by your device.
Back camera:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/base/Camera1Base.java#L451
Front camera:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/base/Camera1Base.java#L460

Thanks for the (quick) input pedro, very much appreciated. My interest came from the perspective if this event is recoverable. If it is not, I will go for the direction you just posted.

No, you cant handle this error.

Thanks! In that case I will close this issue.


bin path doesn't come from gem env

Shouldn't it? it's hardcoded to:
RBENV_COMMAND_PATH="${RBENV_ROOT}/versions/${RBENV_VERSION}/bin/${RBENV_COMMAND}"

but if gem env has a different EXECUTABLE DIRECTORY rbenv will not be able to find the executable. Is the user expected to configure this properly at build time for their ruby?

rbenv works best if all the executables are within the Ruby version's bin directory, as you noticed. If they are in another location, they can't be discovered without the use of a plugin.
I was working on built-in support for executables in user's home install location (~/.gem), but it's not yet merged: #639.
In your case, where is the EXECUTABLE_DIRECTORY located?

it is /usr/local/src/rubinius/gems/bin. The fix I have right now is /usr/local/src/rubinius/gems/bin is a symlink to /usr/local/src/rubinius/binwhich works fine. Also, ~/.rbenv/versions/rbx is a symlink to /usr/local/src/rubinius.

When installed by ruby-build, all rbx installations will have their gems/bin symlinked simply as bin https://github.com/sstephenson/ruby-build/blob/a5d0479af7ca59e9b31f52156d6b38887f68bf24/bin/ruby-build#L681-693

ah. so this is a known thing. Thank you! it still seems like it would be smart to use the gem env, but maybe that is a bad idea / not possible for reasons I don't understand. thanks for your help either way.

It's possible, it would just make rbenv code more complex and I've decided not to tackle that problem in rbenv core because I'm not a fan of Rubinius having gem executables separate in gems/bin.


Fix Quorum for legacy proposals to be original amount

https://dao.effect.network/proposals/22
The new p22 quorum is displayed for legacy proposals, fix to prevent future confusion to new people in future

Good catch. This is a bug in the frontend; the quorum should be fetched from the cycle table that the proposal belongs to, not from the config table

Fixed in https://github.com/effectai/effect-dashboard/pull/91


Add extra feature to /MapHack, create new plugin

I've added an extra feature to /MapHack, /MapHack stop - this stops map motd bypassing without needing to reload the map.
I've also created a new plugin based on KickJini which kicks anyone who connects with a client which does not contain "ClassiCube " in its name.

Thanks


Craft control dashboard - jsonapi detection methods

So i just bought craft control on my ipad to monitor my server and it uses vault protocolib and jsonapi. The dashboard is empty.
Anyone have custom detection methods i could add to json api?
My host is Wulf Hosting and my server is running on Spigot 1.11.2
Please note that everything in Craft Control works perfectly except the dashboard

Screenshot https://www.dropbox.com/s/ckiu89ufk7uk5jx/File 19-06-2017%2C 19 44 01.png?dl=0

Any errors in the server log?


Add support for cross-platform building.

A user should be able to build a ModuleLoader image on a specific arch and run it on other archs.
This is especially required for the hub&spoke topology as a user may build his ModuleLoader in a x86 cluster but run in on an ARM cluster.

One possible solution to explore would be to use kaniko to build for a specific arch (regardless of the host) using https://github.com/GoogleContainerTools/kaniko#flag---customplatform.

The doc for kaniko's --customPlatform says:

This is not virtualization and cannot help to build an architecture not natively supported by the build host. This is used to build i386 on an amd64 Host for example, or arm32 on an arm64 host.

It would not help building a kmod for amd64 on an arm64 machine.

Bummer. We need another solution then.


An administrator can search material collections by project [#94104142]

This adds a generic "where" hash to the SearchableModel#search method parameters that takes the following form:
{col => [op, value]}
where col is the column name, op is the operation and value is the right handle value of the operation.  In this commit the search is limited by project_id by passing the following from the materials collection controller:
{:project_id => ["=", params[:project_id]]}

This seems a bit dangerous because of the "=". Someone could put in ") [delete lots of stuff here]".
Active Record has a 'hash condition' approach which you can probably use instead, and it would be safer. It will be a little less flexible, but that should be ok in this case.
Because of the way paginate is setup and the way Active record query method chains work I think you can do this without modifying searchable_model:
MaterialsCollection.where({:project_id => params[:project_id]}).search(params[:search], params[:page], nil, nil, where)
So the full thing would look like:
filtered = params[:project_id].to_s.length > 0 ? MaterialsCollection.where({:project_id => params[:project_id]}) : MaterialsCollection

@materials_collections = filtered.search(params[:search], params[:page], nil, nil, where)

Here are some more details:
The searchable model is using will_paginate:
https://github.com/mislav/will_paginate/wiki
Which lets you call paginate at the end of an Active Record query chain.
Here is the rails 3.2 guide on hash conditions:
http://guides.rubyonrails.org/v3.2.21/active_record_querying.html#hash-conditions


Firefox  in repeat

Hi,
This works in chrome/IE/edge but not in FF.
The compose isn't rendered in FF although the activate() of the component in the compose is executed.
<template if.bind="(global && global.placeholders && global.placeholders[name] && global.placeholders[name].length > 0)">
    <template repeat.for="item of global.placeholders[name]">

            <template if.bind="($parent.escape.indexOf($index) == -1)">

                <compose model.bind="item" view-model="components/${item.contentType}/index">
                </compose>

            </template>

    </template>
</template>

So i put this as test  (with only the "first item" of the repeat)
<template if.bind="(global && global.placeholders && global.placeholders[name] && global.placeholders[name].length > 0)">
       <compose model.bind="item" view-model="components/${item.contentType}/index">
       </compose>
 </template>

before the above code
the compose in repeat would suddenly render on FF.
but that "first item" compose would not render in FF, but render ok in all other browsers..
If i remove the repeat, that first compose will render on FF.
The element that's composed just contains simple text for test.
Thanks for help.
Greetings
Felix

I think after putting
& oneTime & signal:'refresh-placeholder'
in the repeat, it began working in FF


XFA Rich Text not rendering correctly

Attach (recommended) or Link to PDF file here:
ESS ExpressVote Stencils.pdf
Configuration:

Web browser and its version: 92.0a1 (2021-07-27) (64-bit)
Operating system and its version: Windows 10

Steps to reproduce the problem:

enableXfa = true
Open form

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)
Missing leaders, barcodes, text out of alignment

I've also noticed this with another PDF that has dynamic pages: https://gitlab.freedesktop.org/poppler/poppler/-/issues/1107
dclUnica_2021_15052021.pdf

The documents contains few barcode elements and for now they aren't implemented.
About the dashed lines, there are few:
<p style="text-align:right;tab-interval:0.5in;xfa-tab-stops:after leader (rule (dashed 1.5pt) page) 7.85in...

and the tab-stops stuff isn't implemented too (see page 61 in xfa v3.3 specs).


edited Dutch national flag problem

Checklist

[x ] I've read the contribution guidelines.
[ x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
#fixes #874
(Write your answer here.)
Fill up the DCP form
If you are a part of DCP 21, fill up this form after the PR is merged.
The links provided in the form will be checked thoroughly, if someone tries fooling around, strict actions will be taken against the individ

edited the problem

Edited the problem you can check it.
And put it on the general questions folder .
On Thu, Aug 12, 2021, 9:01 PM Vaidhyanathan S M @.***>
wrote:

@.**** requested changes on this pull request.
Why are there 2 files ? Remove one of them. Also, place the file in
appropriate folder. If not sure, place it in "General Questions".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/smv1999/CompetitiveProgrammingQuestionBank/pull/909#pullrequestreview-728774490,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQXEOHOZFGN5D5V4HMDZFDLT4PSNJANCNFSM5B7JH5HQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Disable weekends?

Hey man, i really like your package!  Is there a way to disable weekends (saturday and sunday) so the user can't  pick those days? If i use pause slots for every weeknd that's a bit tedious lol :)

Hi!
Yeah, great idea, will put this in my pipeline!

I'm looking forward to it! cheers! :)

@kristijanbarabas https://pub.dev/packages/booking_calendar/versions/1.1.6
And check out the docs:
https://pub.dev/documentation/booking_calendar/1.1.6/booking_calendar/BookingCalendar/disabledDays.html
let me know

Awesome man! It does the job!  Thanks for letting me know. :)
Much better than my workaround function that created DateTimeRanges for every weekend! XD
If i have anymore ideas I will let you know. :D

How about hours disabled?

@SerdarYildiz06 https://pub.dev/documentation/booking_calendar/latest/booking_calendar/BookingCalendar/pauseSlots.html
Not what you are looking for?

Closing for no update


Failed to load the linter-less package

[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.19
System: Unknown Windows Version
Thrown From: linter-less package, v2.2.0
Stack Trace
Failed to load the linter-less package
At Cannot find module 'atom-package-deps'

Error: Cannot find module 'atom-package-deps'
    at Module._resolveFilename (module.js:334:15)
    at Function.Module._resolveFilename (C:\Users\Сергей\AppData\Local\atom\app-1.0.19\resources\app.asar\src\module-cache.js:383:52)
    at Function.Module._load (module.js:284:25)
    at Module.require (module.js:363:17)
    at require (module.js:382:17)
    at Object.<anonymous> (C:\Users\Сергей\.atom\packages\linter-less\lib\init.coffee:5:15)
    at Object.<anonymous> (C:\Users\Сергей\.atom\packages\linter-less\lib\init.coffee:1:1)
    at Module._compile (module.js:428:26)
    at Object.defineProperty.value [as .coffee] (C:\Users\Сергей\AppData\Local\atom\app-1.0.19\resources\app.asar\src\compile-cache.js:169:21)
    at Module.load (module.js:353:32)
    at Function.Module._load (module.js:308:12)
    at Module.require (module.js:363:17)
    at require (module.js:382:17)
    at Package.module.exports.Package.requireMainModule (C:\Users\Сергей\AppData\Local\atom\app-1.0.19\resources\app.asar\src\package.js:761:34)
    at C:\Users\Сергей\AppData\Local\atom\app-1.0.19\resources\app.asar\src\package.js:192:28
    at Package.module.exports.Package.measure (C:\Users\Сергей\AppData\Local\atom\app-1.0.19\resources\app.asar\src\package.js:169:15)

Commands
     -2:25.6.0 tree-view:toggle (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -2:17.7.0 settings-view:check-for-package-updates (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui)
     -1:41 core:paste (atom-text-editor.editor.mini.is-focused)
     -1:26 core:backspace (atom-text-editor.editor.mini.is-focused)
     -1:24 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:51 editor:newline (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "autoHideMenuBar": true
  }
}

Installed Packages
# User
color-picker, v2.0.13
file-icons, v1.6.11
linter-less, v2.2.0

# Dev
No dev packages

@Horn-Main How did you install the package?

This is almost certainly another instance of https://github.com/atom/atom/issues/8891 due to having non-ascii characters in your user profile path. There unfortunately is nothing that can be done until Atom releases the next version.

@Horn-Main
Atom v1.2.0-beta0 has been released as a beta, and it includes the updated Electron version that reportedly fixes this issue. If you don't want to wait until v1.2.0 is officially released you can install this preview now.


Move colums and double tree panel

What's the Problem?
Hello,
A feature I would like to see in the app:

Ability to rearrange the columns to customize the display. In my case, I usually have the "Size" column after the "Name" column, then "Type" and finally "Date modified".

Solution/Idea
Be able to re arrange the colums
Alternatives
Stay with Windows file explorer
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
<IP_ADDRESS>
Windows Version
Win10 21H2
Comments
No response

Tracked in #4643


Can't send null over channel

A channel reader cannot receive a null value when used in a foreach, nor can you send one:
https://github.com/phasync/phasync/blob/5b5a81ea3da329ccebad2180341e1a86c52bd90b/src/Internal/ReadChannel.php#L38-L42
It might be a good idea to allow sending a null, but serialize to as a SpecialValue::Null enum type or something, then deserialize to null when reading from the channel.
Use case:
reading csvs which sometimes have a null in a column.

If I enable sending null, it has implications for the design of for example publisher/subscriber - making it slower. Also, I'm not sure how I would serialize null for example via a stream. A null value strictly speaking has a type (it's a pointer to the memory address 0, but it still has a type). In most cases, you would send an array of the entire CSV line which can contain null.

You can implement it like this via a dedicated type for it:
enum SpecialValue { case NullValue; }

Then in the reader:

public function getIterator(): Traversable {
     while (null !== ($message = $this->read())) {
         yield $message === SpecialValue::NullValue ? null : $message;
     }
 }

And in your writer:
$value ??= SpecialValue::NullValue; // only set to null value if null already

The problem I have is that for every message, the PHP engine will have to check if the value is that special null representative. I will look into if I can implement this by actually sending the null value. If it has no performance impact, and the tests pass then that's fine.

Technically I am actually using the null value to close the channel. I carefully optimized performance so that I achieved 1 million messages per second. Supporting null significantly reduces that number, but sure an argument can be made that it is not neccesary. However, many things rely on channels, and often in many levels for a single operation and I spent time to get the performance to be absolute top. I will see if I can get the same performance with another approach.

In particular, to close the publisher type channels that is.


Will this tutorial continue to be updated?

No description provided.

Yes. Due to some affairs, some papers have not been updated until the middle of April.

No description provided.

Since energy is limited, I welcome readers to update this tutorial by providing papers I missed.
Thanks very much！


Refactor Batch Count Settings

The behavior of batch count and batch size under normal conditions makes sense:
Batch Size -> The number of images to create per batch
Batch Count -> The number of batches to create
When using dynamic prompts, what I expect Batch Size to indicate the number of images per combination instance. E.g. {red|blue|green} flower, I want 4 red flower images, 4 blue flower, and 4 green flower.
Then, batch count, multiple that N number of times.
It doesn't really make sense to add combinatorial batches, and then make me set the batch count equal to the number of possible combinations available. Just calculate the combinations.

That's not how DP interprets batch size and batch count. Those settings are division of labour, ie number of images to generate at a time * number of iterations. Combinatorial batches is used to indicate how many images you want per combination.

I expected to be able to set them all to 1, and given a combination of N, get N images, unfortunately, I only get 1 image. I have to change the batch count to match the number of combinations.

I'm happy to refactor and open to suggestions. Combinatorial batches in particular is confusing. Here are some constraints/considerations.

I can't change the semantics of  batch size. Batch size specifically means, how many images to generate simultaneously. I don't think your earlier suggestion that batch size should mean number of images per prompt instance would work. Say I want 10 images, nearly all GPUs would fall over with such a large batch. Currently combinatorial batches plays that role.

Combinatorial explosion can be problematic so I think you need a safeguard preventing the creation of an unexpectedly large number of images.

I expected to be able to set them all to 1, and given a combination of N, get N images, unfortunately, I only get 1 image. I have to change the batch count to match the number of combinations.

One possibility would be to replace combinatorial batches with max images. Then num batches  could be used to set the number of complete sets of unique prompt instances to generate. I don't think Batch size can change.  The problem with that approach is that it breaks the metaphor.  If we define a batch as a complete set of prompt instances, then batch size becomes non-sensical.

One possibility would be to replace combinatorial batches with max images. Then num batches could be used to set the number of complete sets of unique prompt instances to generate. I don't think Batch size can change. The problem with that approach is that it breaks the metaphor. If we define a batch as a complete set of prompt instances, then batch size becomes non-sensical.

Yes please!

But the issue with overloading the meaning of batch sounds as confusing as the current situation.

Let's take a step back. Without dynamic prompt enabled, batch size == number of images to generate, each being seed + 1 from the last, and batch count is the # of times to do that.
With dynamic prompts, batch count is ignored, because this extension produces a certain number of images per batch based on the combination matrix. This would be easy to explain and not surprising to users.
What you could introduce then is combination max/limit within the settings, which is set to 0 (unlimited) by default. This setting could be explained in a couple different ways. It's like head in linux, as in, it takes the first N combination prompts, and does only that, regardless of how many combinations are present in the current prompt. It could be explained as a limiter, so that you can create a prompt with a lot of combinations, but if you need to iterate quickly on parts of the prompt unrelated to the dynamic variables, you can use combination max/limit to produce only a few combinations, until you get the desired output, then increase the max all the way, or to 0 (meaning unlimited).

OK, I can go with that. But just to be clear, that only applies to
combinatorial mode. Standard / random mode remains unchanged. ie total
number of images is equal to batch count * batch size.
In combinatorial mode we ignore batch count completely and introduce a max
images option. The meaning of batch size doesn't change however.
Effectively batch count becomes min(num_combinations, limit) / batch size
On Fri, 20 Jan 2023, 06:53 Wes McNamee, @.***> wrote:

Let's take a step back. Without dynamic prompt enabled, batch size ==
number of images to generate, each being seed + 1 from the last, and batch
count is the # of times to do that.
With dynamic prompts, batch count is ignored, because this extension
produces a certain number of images per batch based on the combination
matrix. This would be easy to explain and not surprising to users.
What you could introduce then is combination max/limit within the
settings, which is set to 0 (unlimited) by default. This setting could be
explained in a couple different ways. It's like head in linux, as in, it
takes the first N combination prompts, and does only that, regardless of
how many combinations are present in the current prompt. It could be
explained as a limiter, so that you can create a prompt with a lot of
combinations, but if you need to iterate quickly on parts of the prompt
unrelated to the dynamic variables, you can use combination max/limit to
produce only a few combinations, until you get the desired output, then
increase the max all the way, or to 0 (meaning unlimited).
—
Reply to this email directly, view it on GitHub
https://github.com/adieyal/sd-dynamic-prompts/issues/191#issuecomment-1397920777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADGXMUH7MCX4DVE7YDYVKTWTIK4JANCNFSM6AAAAAAT5NIBK4
.
You are receiving this because you commented.Message ID:
@.***>

Have a look at v2.3.7 - I've added the max generation slider as discussed. Thanks for the suggestion.

I think there's some text left behind (I just updated to latest version, seeing the Max generations)
Generate all possible prompts up to a maximum of Batch count * Batch size)

oh, and it looks like the readme needs to be updated too:
https://github.com/adieyal/sd-dynamic-prompts#increasing-the-maximum-number-of-generations


Check for two paths for each file

This is because HTMLDir can generate a file from two different places:

foo.rst
foo/index.rst

Both lead to foo/index.html
fixes https://github.com/rtfd/readthedocs.org/issues/5368

Yea, the footer should have FS access, but we're talking about having that go away here soon too :/

but we're talking about having that go away here soon too

Do you mean the footer api? but the version selector is great!

No, the filesystem access on the webs :)

Works locally:

You should check the api response, because when we don't have results, we fallback to sphinx. But it should work with this logic.


Make test dependencies test only

This way people will not have to install mock when they are installing the package from PyPi.

Thanks


Add an option to accept configmap of schedulerConf

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind feature
What happened:
What you expected to happen:
kube-batch can load the schedulerConf from the specific configMap or the scheduerConf file, just make it easy to use for the end-users.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/feature

/cc @asifdxtreme

@yylin1 talks with me and he has same requirement. I can help on this task

/assign Jeffwan
thanks very much for your help :)

@jiaxuanzhou @yylin1
I'd like to understand more requirements. Right now, there's a scheduler-conf option to read from configuration path. Configmap can mount to controller container.  So technically, we can change the way we deploy kube-batch in helm to mount an user-defined configmap and don't need to change kube-batch.
Another case is users want to dynamically change configurations without restarting kube-batch pod, which requires a configmap listener and reload plugins once the target configmap changes.
Could you leave more comments on the usage?

@Jeffwan  it is great if the kube-batch could support the fsnotify of the configurations 😃

@Jeffwan
According to the operating situation, dynamically change configurations are more suitable. Using Configmap  to reload plugins is better.
I think it could add more details illustration about default configuration to changes actions and plugin combination.

fsnotify seems better; if conf changed, reload it in next scheduling cycle.

@Jeffwan any update?
we can also provide one web-hook, then combine the ability of configmap-reloader

@jiaxuanzhou Sorry for late response. I was busy with some other areas and didn't get a chance to work on it in past few months. I will get some time to make it in next two weeks. Feel free to take it if you get time.


Check that indel sequence is correct

https://github.com/oicr-gsi/debarcer/blob/dd0c93b5a73f8a8b9fd24f7c8678e8459d6e66a4/debarcer/src/generate_consensus.py#L166-L170
Indel sequence is retrieved from slicing the query sequence.
It needs to be clarified whether the +1 adjustment  is needed, because this is legacy code and there was once a 1-off error due to mixing 0-based and 1-based coordinates.

correct


feat: Request Queue v2

Closes #1365

@drobnikj can you please take a look as well
@vladfrangu I also see the puppeteer e2e tests failed on platform due to a chrome version mismatch, any idea why is that? maybe its just flakyness (since the scheduled run passed just fine), but it feels pretty weird

We will also surely need some unit tests for this, I don't want you to rewrite all existing tests to support v2, but with e2e tests we won't have such a confidence (as they won't run on each PR).

Ready for another pair of reviews! Still need to add some tests for this, but I'm more confident it should just work

FYI for the PR toolkit to pass, every PR needs to have an estimate and a correct label (or be connected to an issue that has those).


REST API to retrieve resourcemap xml document

@pkdash @dtarb @danames @shawncrawley
Believe it or not, this seems to have been already implemented, though I don't know if it works properly. The REST endpoint to use is:
/hsapi/resource/{resource-id}/data/resourcemap.xml
Please see if this works!
To understand this code, see:

hs_core/resourcemap_urls.py for the endpoint.
views/resource_rest_api.py for the implementation.
hs_core/views/init.py for the private routines used in the implementation.

If this doesn't work, I have another simpler implementation. I discovered this implementation after hacking it up.

@dtarb @danames @shawncrawley @pkdash
There is a lot more stuff in hs_core/resourcemap_urls.py that might not work.
See the url
resource/{server-key}/home/{proxy-user-key}/{resource-key}/data/resourcemap.xml

(at least, this is the best interpretation I can make of it; not sure exactly what it is doing). This is implemented via:
url(r'^resource/([A-z0-9]+)/home/{}/(?P<shortkey>[A-z0-9]+)/data/([^/]+)/$'.format(settings.HS_LOCAL_PROXY_USER_IN_FED_ZONE),
        views.file_download_url_mapper, name='get_resourcemap_or_metadata_file'),

There is a name conflict with the prior version so I think this is the one that gets looked up when you reverse-lookup get_resourcemap_or_metadata_file.

@dtarb @danames @shawncrawley @pkdash The former code is in the develop branch.
There seems to be an inconsistent implementation of iRODS federation, in the sense that some links use it properly and others just use local links. It looks like the first URL I mailed you is broken, because it does not use federation, but it was neither deleted nor updated.
One question for you: I know how to put this link into the resource description list so that apps will have it available easily. Should I do this?

@pkdash @hyi Can you shed any light on the URLS in hs_core/resource_map_urls.py?
One involves federation and I don't know how to interpret it. The other doesn't. Are both active?
Or is anyone using either one?

@pkdash @hyi (cc @dtarb) Here is a general question: when advertising URLS to apps, should I use the federated form, or should I shorten the form if the resource is local? Since this is shared only with the app, and not with the user, I think I should be consistently advertising the longer federated form of the address.... any objections? .

@alvacouch I won't be able to comment on the federation side. I think @hyi  can. The urls in resourcemap_urls.py are created to make the links in resourcemap.xml xml active (no broken links) and those are all mapped to single view function in views/int.py. These are not part of the REST API though. That's why I did not realize there is this end point for downloading the respurcemap.xml document. There is no implementation in resource_rest_api.py for downloading resouremap.xml document.

@pkdash That this is implemented in resourcemap_urls.py and not in resource_rest_api.py confused me, too!

@hyi @pkdash (cc @dtarb) It would seem to me on second thought that embedding the federation information into the URL is actually overkill, because -- in principle -- this url is being interpreted by the core engine, which knows about federation and can construct the federated URL to feed to iRODS from its knowledge of the actual location.....

@hyi @pkdash (cc @dtarb) Perhaps I should explain further. We have two uses of a federated URL: one for an invocation to iRODS REST, and another for a request to hs_core. It would seem to me that the proper thing to do is to advertise the direct iRODS link for federated data to apps, but to respond -- as hs_core -- to the unfederated link. This simplifies hs_core interaction and -- at the same time -- allows apps to intelligently download things. Thus, hs_core never responds to a federated URL, but apps know the iRODS URL and can use them.

@alvacouch I am not sure we are using iRODS REST at all. As far as I know every file access call needs to go through hs_core.

@alvacouch @pkdash The URL mapping in this resourcemap_urls.py is used to map the urls stored in resourcemap.xml file to the django_irods download urls so that these urls stored in resourcemap.xml are resolvable to the real download of the corresponding file stored in irods, by default, in hydroshareZone. However, if the resource is stored in a federated zone, either hydroshareuserZone or a third party irods zone that is federated with hydroshareZone, this federation zone name has to be in the url in order for the download view function to know where (i.e., which iRODS zone) to download file from, which is why I added that url mapping related to federation zone that allows to download file directly from a federated zone. Without this info, download function does not know which iRODS zone to download file from.

@hyi Is the zone name then part of  the url in resourcemap.xml for a file that is on a federated zone?

@pkdash Yes, otherwise, download view function does not know which irods zone to download file from.

@hyi The problem is how to get the correct url for the resouremap.xml.

@pkdash @alvacouch The urls encoded in resourcemap.xml are generated automatically when resource is created, however, to retrieve resorucemap.xml itself, a REST API needs to be added to check resource object to retrieve federation zone info, then form url properly to allow download of this resourcemap.xml file itself, which I think is part of your work that adds REST API end points for federation zone work.

@hyi Agreed, We need a new REST endpoint to download resourcemap.xml which should be similar to what we have for resourcemetadata.xml.

See branch 1034-file-metadata for a clone of the /resource/{id}/scimeta/ pattern to implement
/resource/{id}/resmap/. Make a quick check that this makes sense.

(This relates to #1034 as well)

@alvacouch It looks OK to me addressing this issue( #1425).

@alvacouch This would require changes to rest client which I can do after this work gets merged.

@alvacouch I cannot see the changes you made to 1034-file-metadata related to /resource/{id}/scimeta/ pattern. Could you let me know where the changes are?

I copied the scimeta routines and created resmap routines from them. The new routines include

resource_rest_api.ResourceMapRetrieve (class)
hs_core.AbstractResource.resmap_url

and an entry in hs_core.urls...

@hyi @pkdash Oops. Wrong branch. See branch 1034-expose-file-types-in-REST....!

@alvacouch It looks good to me as well. @pkdash When implementing hs rest client, when retrieving resourcemap.xml, just remember to pass in a path string that includes federated zone name if the resource is hosted in a federated zone rather than the default iRODS hydroshareZone.

@hyi @pkdash Darn. Changes are in the other branch, along with lotsa other confusing changes. I'll move the files.

@hyi

When implementing hs rest client, when retrieving resourcemap.xml, just remember to pass in a path string that includes federated zone name if the resource is hosted in a federated zone rather than the default iRODS hydroshareZone

I assume the rest client already takes care of the federated zone param for getting the resourcemetadata.xml. Otherwise, we have do this for some other similar apis in the client too.

@hyi @pkdash Lots of changes in 1034-expose-file-metadata-in-REST are part of another subproject, and there is a conflict with other changes to resource_rest_api.py .... will rebaseline and get back to you ...

@pkdash I am not sure whether rest client needs any changes to take care of federated zone parameter, it will be good to check this when you work on the issue for adding REST support for federation zones https://github.com/hydroshare/hydroshare/issues/1276

@hyi @pkdash Sorted out. Current code is in branch 1425-REST-API.... matching this issue.
@pkdash we have a bit of a chicken and egg problem. We now need to test this, which requires the Python REST API...!  Could you create the new method? Or should I?

@alvacouch If you can add the unit test similar to what we have for science metadata (https://github.com/hydroshare/hydroshare/blob/develop/hs_core/tests/api/rest/test_scimeta.py), I will make the necessary changes in the hs_restclient.

@alvacouch @pkdash @dtarb @horsburgh @rayi113 @mjstealey  I was thinking about this during our team call, and think there may be a way to remove federation zone info from internal url pattern for downloading files from a federated zone. I will have to look into this later after I get folders work done. @alvacouch You may hold off on this until I get a chance to look into it and see if I can just remove federation zone info from any url patterns, internal or external.

@alvacouch Any update on this.

@pkdash @hyi (cc @dtarb)
I have nearly got tests working, but I am concerned by some of the behaviors I am seeing.
Part of the reason for me seeing this behavior is that I am working with the actual REST calls rather than just the routines. So, in essence, I'm seeing what python-restclient sees rather than just testing the code. I am not satisfied with just testing the routines because so much of this depends upon how the built-in Django REST reacts to my code.
But -- for better or worse -- I'm using the Django REST client rather than the python rest client, and that is leading to some confusion for me:

The /hsapi/resmap/{id} call for grabbing resourcemap.xml gets redirected to an iRODS call (which I expect), but then
The iRODS REST call returns a Django ResponseFile (a file stream, in the content_stream object) rather than a file (which I don't expect).

My guess is that this is partly a side effect of using the Django REST client rather than a raw REST client.
@hyi Should the redirection calls to iRODS generate a streaming file object? Not absolutely sure I'm not messing up somewhere.
The buggy code is in branch 1425-*

@alvacouch Yes, iRODS returns a streaming file response as shown here: https://github.com/hydroshare/django_irods/blob/master/views.py#L125..L129. I expect you can just save this FileResponse object to file, but have never really done so. Perhaps @pkdash knows more on how hydroshare rest client does when downloading a file.

@hyi OK, that makes me more confident. I know how to save it to a file.

See PR #1478

@pkdash does this work? Has the python restclient been updated to access it ? When that is true, feel free to close it.

This has been implemented. Closing.


Avoid pulling plugins from the Terraform online repositry

We'd like to test what we ship, so we should not allow terraform
to pull from the online repository additional plugins, as that would
hide issues in the state that we're trying to test in the CI.
(cherry picked from commit 5d40e56d09a2ab976c0b654f1b6bfcd630eb991c)
Why is this PR needed?
Does it fix an issue? addresses a business case?
add a description and link to the issue if one exists.
Fixes #
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.
What does this PR do?
please include a brief "management" technical overview (details are in the code)
Anything else a reviewer needs to know?
Special test cases, manual steps, links to resources or anything else that could be helpful to the reviewer.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
How can we reproduce the issue? How can we see this issue? Please provide the steps and the prove
this issue is not fixed.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
If docs need to be updated, please add a link to a PR to https://github.com/SUSE/doc-caasp.
At the time of creating the issue, this PR can be work in progress (set its title to [WIP]),
but the documentation needs to be finalized before the PR can be merged.
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

Let's merge this for 5.0.0-GMC.
@dirkmueller I would appreciate if you could create an issue about this (or point me to the one if it already exists), so that it appears here https://github.com/SUSE/avant-garde/issues?q=is%3Aissue+is%3Aopen+label%3A5.0.0-GMC .
This makes it easy for us to track what it is into the release.
thanks


⚠ Bofrost.be has degraded performance

In dd0990d, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1221 ms

Resolved: Bofrost.be performance has improved in 3ae36c5 after 6 minutes.


Software version numbers

We discussed this, and probably it's best to remove the version numbers from the environment.yaml file, so that users will not be stuck with old versions. Any user can always specify the versions they want (to make their own analyses reproducible), and we can provide a list of the versions that we have tested the workflow with. In the Wiki, we can also show how to export an environment file from a specific conda environment.

One possibility is to have a git_install file as I sent in the slack to install R packages. In that file, it will check whether  the list packages are already installed in the environment. If not, it will download automatically. Of course, the packages are downloaded from CRAN, Bioconductor or github, so it would be the newest version. If users would like to use a specific R packages version, they could specify it in the environment.yaml file and update the environment. When they run the pipeline, the git_install will check the packages again and install those that are not downloaded yet. This has the advantage that users are free to choose the versions they like and less likely to get the complicated conflict issue. The way we are doing now could work at this moment and fail at another moment. This would be painful for users to figure out how to solve the issue by themselves and probably also difficult for the future maintenance. Also, I guess the R packages in bioconductor or CRAN are more likely to be continously maintained than those in Bioconda. Now, we have the latest R version 3.5.1 in the conda. However, we can't build the environment using it with any R packages, even the popular package, such as ggplot2 or edgeR.

Thinking about this again, perhaps we want to keep the version numbers in the environment file, to represent a setup that has been tested.


WIP: Command Pallete

That maps the existing shortcuts and common actions and provides a description. Includes
c - Create event
j - Go to previous week
k - Go to next week
[ - Toggle  left sidebar
] - Toggle right sidebar
z - Logout
w - Add to week list
m - Add to month list
Context
About command paletes Resource page (mangy)
Possible libraries:

react-cmdk

This is feature will be great, but I'm prioritizing the issues in the Cleanup Project for now: https://github.com/orgs/SwitchbackTech/projects/2/views/1
I'll be doing more research and documenting the findings in this issue leading up to when it's ready

Hey Azhar, thanks for reaching out! I decided to hold off on the command
pallete tasks until the issues in the cleanup project
https://github.com/orgs/SwitchbackTech/projects/2 are completed first.
Are you interested in helping on either of those issues in that project?
On Sat, Dec 09, 2023 at 3:10 AM, Azhar Ahmed @.***>
wrote:

Heey , I would like to work with you in this issue and use this app for my
tasks , can I join??
—
Reply to this email directly, view it on GitHub
https://github.com/SwitchbackTech/compass/issues/14#issuecomment-1848337992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHGEA34TTAON65BJAO3PX6LYIQTJXAVCNFSM6AAAAAA53CUZQKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBYGMZTOOJZGI
.
You are receiving this because you were assigned.Message ID:
@.***>

FYI that I unassigned myself due to my schedule, so this one is up-for-grabs again


fixed broken xrefs

to attribute :rhoardoc-jira-workflow: was added missing ending _context
the attribute :using-github-labels_contributing: doesn't exist,therefore all referring information was deleted
the name of attribute :repo-docs-name: was changed to :name-repo-docs:

Can one of the admins verify this patch?

[test]

[test]


Allow For HTTP Basic Auth for ES6 Http Client

This change is

@justinsoong thanks!

Good job @justinsoong. Can you also change the ES5 connector? I think this is a good resource for it.

@caiooliveiraeti sure thing


Cards change

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #5687
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

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

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot 📸

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

Copilot Summary

🤖 Generated by Copilot at 5f95f5d
Refactored the styles and components of various cards in the UI and the gallery to use a new card-border-color variable and to remove unnecessary animations. This improves the performance, design consistency, and code maintainability of the project.

🤖 Generated by Copilot at 5f95f5d

CardArticle style
shared across components now
less code, more spring

Effected cards:
Landing Page

All Carousel cards (Spotlight, Newest List, Latest Sales)
Top Collections
Featured Articles

Explore / Item Grid

items tab cards
collections tab cards

looks good to me!

Duplicate theme variable code  :/
This libs/ui is starting to be less sustainable.

True

I don't really like adding opacity on hover, normally I would have seen the opposite.
Duplicate theme variable code  :/
This libs/ui is starting to be less sustainable.
otherwise lgtm ✅

What do you mean by seeing the opposite? Would you remove the hover opacity completely? Thanks for feedback!

What do you mean by seeing the opposite?

Opacity by default, remove opacity on hover. I find it strange to add opacity on something that should stand out.

Would you remove the hover opacity completely? Thanks for feedback!

I would reintroduce a slight zoom on hover effect

Opacity by default, remove opacity on hover. I find it strange to add opacity on something that should stand out.

the opacity is mainly for future cart button over the img and other functionalities
that means it will stands out

I would reintroduce a slight zoom on hover effect

yup, that sounds good, I would leave it to another version tho

I don't really like adding opacity on hover, normally I would have seen the opposite.

Inspiration was coming from https://www.tensor.trade/trade/oogy_pods this is way how we implemented it in now in visual way, as current animation when you are hovering with cursor over are quite slow nad missing, also removing that sharp borders was intrusive

I would reintroduce a slight zoom on hover effect

yeah we had this zoom, which I was thinking, it's possible option. I would like to keep it as basic as possible those animation tbh to not disturb user. Other experiments we can try in fandoms shops for example, let's see, there is room accommodate lot of approaches I guess per collection.

pay 60 usd
quick delivery on first try! like it!

😍 Perfect, I’ve sent the payout
💵 $60 @ 37.46 USD/KSM ~ 1.602 $KSM
🧗 EfmnRhHaQqfT3phm4cUCHCU3gFVDoSPR1U9WXzMRQBMqZ4L
🔗 0x6e7450ef76cd641d5490c0bebf65d058409409124e2baa0912f369a56fc5c5a0
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Crash when running YOLOv3 testing

Failed copying input tensor from /job:localhost/replica:0/task:0/device:CPU:0 to /job:localhost/replica:0/task:0/device:GPU:0 in order to run LogicalAnd: Can't copy 4 bytes of a tensor into another with 1 bytes buffer.

Failed copying input tensor from /job:localhost/replica:0/task:0/device:CPU:0 to /job:localhost/replica:0/task:0/device:GPU:0 in order to run LogicalAnd: Can't copy 4 bytes of a tensor into another with 1 bytes buffer.


[feat]: Add contribution guidelines

closes #2

Hey @JunDevHarsh , finished up on the guidelines, let me know what you think

Hello @grand-rick001, I was going through your PR and saw that you have put more than I expected. But there are some reasons that I am not able to merge this PR to dev branch because you have made an extra commit after making a PR. Please avoid updating your branch after making a PR to any project. Or you can update your branch first and make a commit and raise a PR after that.

Please reply if you want to work on this issue again. If yes then I am closing this PR and will expect to create a PR after making changes to the code.

Okay, noted. Just to be clear, you want to close the PR so I can make changes, then re-open it?

Okay, I am closing this PR right now. Then you have to  make another PR reference to this issue.

No problem


Add Gitlab LDAP EE only vars

Linked to issue #4

I tested this for you, but the variables did not show up with gitlab_edition: gitlab-ee. It seems like they only show with a gitlab_version: gitlab-ee. Setting gitlab_version to the aforementioned value causes the "Install GitLab" step to error out with "No package gitlab-ce-gitlab-ee available."

This seems to work after changing gitlab_version to gitlab_edition. (The variables are in the gitlab.rb, but I still am not seeing any administrative section appear with the admin group) I was getting timeouts during the install, but I see you have a handy gitlab_install_timeout for this. My only other concern is the varying templating of booleans. It seems sometimes they need to be strings in order for the casing to be correct. Would it be worth adding a "to_json" {{ gitlab_XXXXXX | to_json }} to the variables? This would allow boolean variables and keep the correct casing on the template. I have done this for a few PHP config files and it seems to work. Only a few weird strings like passwords needed to be manually wrapped in single quotes.
Example:

nginx['redirect_http_to_https'] = {{ gitlab_redirect_http_to_https | lower }}
active_directory: {{ values.active_directory | default(gitlab_ldap_active_directory) }}
Both are booleans, but handled in different ways. I think to_json could be used on almost everything to keep it uniform.

Up to you, just trying to think of improvements.

Would it be worth adding a "to_json" {{ gitlab_XXXXXX | to_json }} to the variables? This would allow boolean variables and keep the correct casing on the template. I have done this for a few PHP config files and it seems to work. Only a few weird strings like passwords needed to be manually wrapped in single quotes.
Example:
* `nginx['redirect_http_to_https'] = {{ gitlab_redirect_http_to_https | lower }}`

* `active_directory: {{ values.active_directory | default(gitlab_ldap_active_directory) }}`
  Both are booleans, but handled in different ways. I think `to_json` could be used on almost everything to keep it uniform.

Up to you, just trying to think of improvements.

This might be something good to try.
But lets do that in another PR to keep it simple.


chore(main): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2024-07-26)
Bug Fixes

upgrade to rovercom + roverlib (f6a4d07)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.1
:sunflower:


Custom theme style enhancements

Standardize some parts of the site for easier maintenance in case chatGPT is updated, and request all themes that use these classes/id

It is added in 1.2.1


Fixes #98 by comparing min/max param values

ok. thanks
I would propose using Tuple instead of Seq/List
val (_min, _max) = if(min.value < max.value) min -> max else max -> min
...

Nice.  I like that solution.

Updated with your suggestion.