Consumer poll causing high CPU utilization

Description
Hello,
Im working on implementing kafka consumer and observed that the consumer caused high CPU utilization when hosted on a app service.
I have a consumer subscribed to a single topic and have one instance of consumer only.
Im running confluent.kafka 1.1.0 and libradkafka 1.1.0
How to reproduce
Below is the code
var consumerConfig = new ConsumerConfig
{
BootstrapServers = kafkaUri,
EnableAutoCommit = false,
GroupId = consumerGroup,
AutoOffsetReset = AutoOffsetReset.Earliest
};
        using (var c = new ConsumerBuilder<Ignore, string>(consumerConfig).Build())
        {
            c.Subscribe(kafkaTopic);
            try
            {
                while (true)
                {
                    try
                    {
                        Console.WriteLine("Consumer running, waiting for messages");
                        var msg = c.Consume(TimeSpan.FromSeconds(1));
                        Console.WriteLine($"Consumed message '{msg.Value}' at: '{msg.TopicPartitionOffset}'.");
                        c.Commit();
                    }
                    catch (ConsumeException e)
                    {
                        Console.WriteLine($"Error occured: {e.Error.Reason}");
                        Log.Error(e.Error.Reason);
                    }
                }
            }
            catch (Exception ex)
            {
                // Ensure the consumer leaves the group cleanly and final offsets are committed.
                c.Close();
                Log.Information(ex.Message);
            }
        }

Checklist
Please provide the following information:

[ ] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[X] Confluent.Kafka nuget version.
[ ] Apache Kafka version.
[ ] Client configuration.
[ ] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[x] Critical issue.

hi,
how did you solved this issue? if you remmeber... currently with the same problem.


on start project i have error: chat id invalid

help me please

the master branch was a working version that had not yet been released to npm
but It was a bit confusing, I moved it to develop branch, so try the following example in the master now
const YouTube = require('youtube-live-chat');

const yt = new YouTube('CHANNEL_ID_IS_HERE', 'APIKEY_IS_HERE');

yt.on('ready', () => {
  console.log('ready!')
  yt.listen(1000)
})

yt.on('message', data => {
  console.log(data.snippet.displayMessage)
})

yt.on('error', error => {
  console.error(error)
})

i clone file index.js,
i changed file server.js
const YouTube = require('./index.js')
//const YouTube = require('youtube-live-chat');

const yt = new YouTube('blabla', 'blabla');

yt.on('ready', () => {
    console.log('ready!')
    yt.listen(1000)
})

yt.on('message', data => {
    console.log(data.snippet.displayMessage)
})

yt.on('error', error => {
    console.error(error)
})

start project in visual studio 2019, and project was closed...
you can help me? :)

i test develop version and test random yt channel and have this mesage
node server.js
No live stream found for channel UCgd9xnJbT5ZaKe8TzVKcG9Q
No live chat found
--- stopped listening ---
--- restartted listening ---
--- stopped listening ---

but channel is online, and have a chat

Hmm... I couldn't get the live stream for the channel even if I tried it with API Explorer
But maybe this is a channel dependent issue as I can get it on other channels (e.g. UCgd9xnJbT5ZaKe8TzVKcG9Q) without any problem..

Apparently, you need to use some other methods, since this is not working...
If I find a solution, I will let you know


Manually close the tree for issue #88531

Manually closing the tree for https://github.com/flutter/flutter/issues/88531

Landing TBR before presubmit checks have completed because this commit is intended to close the tree.

In the future, can we avoid breaking the hot reload tests? The Dart team is using those for HHH performance tests
https://github.com/flutter/flutter/issues/74529


[JBEAP-27509] extending history information

Add current manifest versions to the git commit so that they can be displayed in history operations

@TomasHofman could you review this when you have a moment?

To double check if this was intended, when I run the new history command against an old installation, I think it delegates to the old impl and shows something like this:
[thofman@rama prospero]$ ./prospero history --dir wf
[d0efe823] 2024-07-12T14:37:36Z - update [file:/home/thofman/Projects/prospero/manifest.yaml::a4ff586c9fce21976f8596c53d6247e200d22cc6]
[71420b8d] 2024-07-12T14:24:54Z - install [org.wildfly.channels:wildfly::33.0.0.Beta1]

It shows "install [G:A::version]" at the first record.
Running the command against a newly created installation, it doesn't show the GA anymore, just the version:
[thofman@rama prospero]$ ./prospero history --dir wf3
[7822a809] 2024-10-16T11:27:47Z - install [34.0.0.Beta1]

High level question: since the original impl of the history command also shows versions of manifests, what is the intended improvement here?

Thanks @TomasHofman I updated the description to explain the changes. Let me know if that clarifies it.

Running the command against a newly created installation, it doesn't show the GA anymore, just the version:
That's my mistake, I'll fix that thanks

To double check if this was intended, when I run the new history command against an old installation, I think it delegates to the old impl and shows something like this
Correct, the old history states (made before this change) should be displayed as before

OK, thanks for the explanation, I noticed the new schema version by accident but didn't know the details. Now it makes sense to me :+1: .

Thanks @TomasHofman 👍


some svgo & attribute flow optimizations

reduce apk size
yes everything is the same, i took a diff

too late, not merged in time for the 9136 pr

You can ping me on GitHub if it's better to merge a PR right away.


What is LearnableSpatialTransformWrapper layer?

Hello guys,
Thanks for the code and project ! I am trying to understand your model and I found that you are using this layer after the FFC Resblock ? Can you explain what are you trying to do with this layer ? I can not find any details in the paper.
https://github.com/saic-mdal/lama/blob/0449ee2ced46f13539b16c5fe8103eb46f56c16d/saicinpainting/training/modules/spatial_transform.py#L7

Hello!
Thank you!
We actually don't use this module, it's a leftover from old abandoned experiments.


Metadata Handling

metadata handling could (should?) be separated from the pixel data, as in some formats (e.g. CellVoyager) the metadata are already completely separate (.mlf etc.) from the pixel data (e.g. .tif without spatial calibration).
Originally posted by @imagejan in https://github.com/fmi-faim/faim-hcs/issues/42#issuecomment-1739095300

Solved with #59.


Update colour contrast of commented code

Update colour contrast of commented code
The current contrast is 3.49:1, which is below the minimum 4.5 ratio needed to meet the WCAG AA guidelines. The proposed change increases the contrast to 4.52:1
Example of how new contract looks

I'm sympathetic to increasing the contrast, but I'd like to keep the hue close to what it is now. Could you try to just darken the current color, instead of making it so gray?

Turns out there was CSS overwriting the brown to make it grey. The original brown passes colour contrast checkers so I will close this pr :)


Python 3.8 builds failing

The Python 3.8 jobs are seeing failures like this job. Snippet below:
 In file included from src/source/libscalene.cpp:93:
  /home/conda/feedstock_root/build_artifacts/scalene_1713294069178/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/include/python3.8/Python.h:44:10: fatal error: crypt.h: No such file or directory
     44 | #include <crypt.h>
        |          ^~~~~~~~~
  compilation terminated.

Likely a result of upstream changes ( https://github.com/conda-forge/linux-sysroot-feedstock/issues/52 )
Fixing probably involves updating the include search paths
That said, Python 3.8 will be EOL'd October 2024. So maybe it is easier to just drop it now

Thanks - I was wondering what was going wrong. I've removed it from the build.

Ok after some discussion with a couple colleagues and a closer look at the build system, think we may want to make some tweaks to how scalene builds. This is independent of whether Python 3.8 is supported as it is relevant for all builds. Have raised an upstream issue ( https://github.com/plasma-umass/scalene/issues/812 ) to discuss how we might want to approach this


How to add search_kwargs when using CustomComponent to create a vectorstore

I want to add a custom vectorstore, like in src/backend/langflow/components/vectorstores/Chroma.py did.
build method return type is Union[VectorStore, BaseRetriever], but the actual code is return a Chroma instance.
return Chroma(persist_directory=persist_directory, client_settings=chroma_settings)

If the next component need a Retrieval, .as_retrieval() function is auto called when building the graph.
If I want to control the retrieval policy, like k or score, I need to add a search_kwargs parameter to as_retrieval.
I want to config k or score parameter to my custom vectorstore, but I find it difficult to do it.

VectaraSelfQueryRetriver.py is something like a converter to convert an exist VectorStore to Retrieval, it is like the earlier Langflow version did, every vectorstore has a corresponding
retrieval component.
But today, I think almost every vectorstore component can act as a VectorStore or BaseRetriever. And I want to control the policy of BaseRetrieval, but I can not pass search_kwargs to a component which actually a VectorStore instance.

Hi @wodadehencou  ,
We hope you're doing well. Just a friendly reminder that if we do not hear back from you within the next 3 days, we will close this issue. If you need more time or further assistance, please let us know.
Thank you for your understanding!

Thank you for your contribution! This issue will be closed. If you have any questions or encounter another problem, please open a new issue and we will be ready to assist you.


⚠ Bofrost.at has degraded performance

In 647370e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 960 ms

Resolved: Bofrost.at performance has improved in 4cba0c1 after 6 minutes.


False positives with annotations

 Environment data

Language Server version: 2021.2.3
OS and version: darwin x64
Python: 3.9.1
python.analysis.typeCheckingMode: strict

 Expected behaviour
The module variable __annotations__ should be recognized correctly.
 Actual behaviour
See screenshot below
 Logs
--
 Code Snippet / Additional information

var: int = 42

print(__annotations__)

The python output in 3.9.1 is:
{'var': <class 'int'>}

Where is this defined in the Python docs or PEP? I know we already support a few of these that are documented: https://github.com/microsoft/pyright/blob/f2ff105380c71d17b729934da867e258a1ed6228/packages/pyright-internal/src/analyzer/binder.ts#L225-L235

Since you updated the title to mention __builtins__, see #954.

Where is this defined in the Python docs or PEP? I know we already support a few of these that are documented: https://github.com/microsoft/pyright/blob/f2ff105380c71d17b729934da867e258a1ed6228/packages/pyright-internal/src/analyzer/binder.ts#L225-L235

I noticed it while investigating #964 and looking at the output of locals() since I wanted to see if any of them worked.
I only found PEP 3107 that mentions it in the context of function annotations. Could be an implementation detail for modules although it seems to work just as expected.

Since you updated the title to mention builtins, see #954.

Removed the reference. Should have checked beforehand.

I see. I don't think we'd want to special case more of these if they aren't properly documented as being available on every module. I did find that PEP, but it only mentioned functions (as you said).

I see. I don't think we'd want to special case more of these if they aren't properly documented as being available on every module. I did find that PEP, but it only mentioned functions (as you said).

I'm fine with that


buildAll: Add vioinput SDV target

Signed-off-by: Kostiantyn Kostiuk<EMAIL_ADDRESS>

@kostyanf14 Are there errors on DVL log?

@YanVugenfirer Yes, there are. I already create a PR with fixes https://github.com/virtio-win/kvm-guest-drivers-windows/pull/643


Fix Edit Dialog target

Use currentTarget instead of target.
Fixes Issue #229

did you confirm that that worked?

Yup works for me.

Okay, I merged it. I'm in the middle of some car electrical issues. I'll create a release when I get home.

Thanks!


Build classpaths using BFS

Previously, Fastpass would compute a project's classpath using DFS,
whereas Pants uses BFS for this.
I added some benchmarks for RuntimeBFS and CompileBFS (because it's so easy to get a huge slowdown there). It looks like there's a small improvement with this PR (against a 22MB export, 6067 exports):
[info] BloopPantsBenchmark.compileDependenciesPlusOne          ss    5  2.446 ± 0.350   s/op
[info] BloopPantsBenchmark.compileDependenciesPlusOneBefore    ss    5  7.879 ± 0.353   s/op
[info] BloopPantsBenchmark.compileDependenciesStrict           ss    5  0.782 ± 0.263   s/op
[info] BloopPantsBenchmark.compileDependenciesStrictBefore     ss    5  2.161 ± 0.297   s/op
[info] BloopPantsBenchmark.runtimeDependencies                 ss    5  0.940 ± 0.364   s/op
[info] BloopPantsBenchmark.runtimeDependenciesBefore           ss    5  5.410 ± 0.189   s/op

It took me a while to understand the BFS logic. I am more used to the following structure
val queue = new Queue[T]
while (queue.nonEmpty) {
  val current = queue.pop()
  queue ++= getNext()
}

Yes, me too... But then I couldn't find a way to cache the result of a traversal, so that it can be re-used from other traversals.


[stable/zeppelin] Zeppelin uses latest api versions

Is this a new chart
It is not.
What this PR does / why we need it:
The beta Kubernetes API that this chart uses has been removed from newer versions of Kubernetes. This updates the Deployment API to the stable one and includes a new required property.
Which issue this PR fixes

fixes #20057

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Adding @danisla as he is the chart owner.

/assign @jlegrone

/ok-to-test

Hi @zanhsieh How do I get this reviewed?

@scottrigby
@mattfarina
@davidkarlsen
@paulczar
@cpanato
@jlegrone
@maorfr
Could you guys look at this chart please? Thank you.

@danisla As the maintainer, do you have any feedback on this PR?

@danisla  @davidkarlsen can we please have this merged ? I think it is needed.

This seems sane to reopen. It was stale only because of no maintainer feedback

/lgtm

Also re #21103 see https://github.com/helm/charts/issues/23685


Updatin readme md file

What

Changes

Tested on

Related issues

Programmer is a person who fix a problem that you don't know you have, in a way you don't understand. Hybrid Mobile application developer with React & Ionic.


I want to know how Establishing NSMachBootstrapServer communication？

hello，Dear author! I want to push stream to virtual camera. Thank to your mac-obs-virtual-camera project.   But when I run the follow code:
   self.port = [[NSMachBootstrapServer sharedInstance] servicePortWithName:@MACH_SERVICE_NAME];
I don't get a valid port. Is there any special configuration?  I can't find any information about NSMachBootstrapServer.  Can you give me some tips please!  God bless you

I'm not the author, and this is not a real answer to this question, however: CMIO DAL plugins like this are deprecated by Apple and no longer function since macOS 14. I’d currently advise against using them.

Thanks for your advise! I want compatibility with versions prior to macOS 12.3. So I found the DAL plugin ins. Aftre macos 12.3, I will use camera extension.


[ic-alert] default - allow icon to be changed

Summary of the bug
A prop needs to be passed in for the icon. An example of a use case is attached below.
🧐 Expected behaviour
The Icon in default Alert should be changeable, so that users can amend it to whatever they like. The other alert types (info, success, warning, and error) all use specific icons to relate to the status of the alert, but the default one should be customisable by the user so that it is flexible in many different scenarios.
📸 Screenshots or code

Additional info
AC
Given that a user is implementing a default type alert, when setting its props; they then should be able to specify a different icon and they should be able to specify no icon.

Add a slot to replicate design behaviour, to only be allowed to be done on "default", not other states. Include guidance to only include on default and add a code example on the site


feat: pass event object based on listeners signature

Pull Request Type

[X] Feature addition
[X] Bugfix
[ ] Documentation update
[ ] Code refactor
[ ] Tests improvement
[ ] CI/CD pipeline enhancement
[ ] Other: [Replace with a description]

Description & changes
Event object is now passed (or not) based on whether there are any parameters in listener
Test Scenarios
Tested on my bot and with test script - both as standalone function listeners and as extension methods
Python Compatibility

[X] I've ensured my code works on Python 3.10.x
[ ] I've ensured my code works on Python 3.11.x

Checklist

[X] I've run the pre-commit code linter over all edited files
[X] I've tested my changes on supported Python versions
[ ] I've added tests for my code, if applicable
[ ] I've updated / added  documentation, where applicable

Purely because I'm certain we are going to get people saying "hOw Do I gEt ThE EvEnT" can you make the listener throw a warning if an event that would normally have a Param (ie any apart from startup and ready) don't on instantiation?

Also, would it be worth moving the inspect call to the listeners instantiation too, as inspect is very expensive, and this is a high traffic part of the lib


multi dimensional array or dictionary binding doesn't work

From @mhamri on January 2, 2018 6:13
I was able to use following code in asp.net mvc but it's not working in asp.net core and i can't see any other solution rather than writing a custom model binder. i tried whatever solution was out there on the internet but look like it's not working properly.
I am using asp.net core 2.0
ScoresViewModel.cs
public class ScoresViewModel
{
    public List<List<int>> Inputs { get; set; }
}

index.cshtml
<from id="question" >
   <input type="text" name="Inputs[0][0]" value="10" />
   <input type="text" name="Inputs[0][1]" value="11" />
   <button type="submit" id="submit" value="submit"/>
</from>

$("#submit").submit(function () {
    var getData = function () {
        return $("#question").serialize();
    };
    $.ajax({
        url: "/Api/ScoreCalculator/Calculate",
        method: "POST",
        data: getData(),
        success: function (response) {
           $("#panel-report").html(response);
        },
        error: function (errResponse) {
            debugger;
        }
    });
    return false;
});

I can see in my browser that the request is post and mime type is x-www-form-urlencode
"postData": {
   "mimeType": "application/x-www-form-urlencoded",
   "text": "Inputs[0][0]=10&Inputs[0][0]=11",
   "params": [
       {
       "name": "Inputs[0][0]",
       "value": "10"
       },
       {
       "name": "Inputs[0][1]",
       "value": "11"
       }
   ]
}

and in the controller i am getting the content
public class ScoreCalculatorController{

     public IActionResult Calculate(ScoresViewModel scores){
             return OK(scores)
     }
}

i tried [FromBody] and [FromForm] but none is working, also if i just try to model bind a string it works fine so i believe i don't need to use [FromBody] and [FromForm] in here.
this code was working fine in mvc 5. look like asp.net core model binder is not supporting complex types correctly. any hint on this?
Expected behavvior
by passing Inputs[0][0] the model binder, bind the value to a multi diemnsional array (int[][], list<int[]>, list<list>) or dictionary (dictionary<int, list>, dictionary<int, int[]> , ...) based on the type.
why it should works in this way?

@Html.TextBoxFor(m => m.Inputs[0][0]) is generating <input data-val="true" data-val-required="The Int32 field is required." id="Inputs_0__0_" name="Inputs[0][0]" type="text" value="1"> so passing the Inputs[0][0] should bind back to the same model
single dimensional array works the same way. simply passing something like Inputs[0] will do the binding to public List<int> Inputs{get; set;}
aps.net mvc works in this way

actual behavior
ignoring the data
Copied from original issue: aspnet/Home#2735

@mhamri please provide additional information to help us reproduce your issue. For example, which version of ASP.NET Core MVC are you using?

We remain unable to reproduce this issue. For example, I duplicated your ScoresViewModel and used a simple view containing
@Html.TextBoxFor(model => model.Inputs[0][0], htmlAttributes: new { @class = "form-control" })
...
@Html.TextBoxFor(model => model.Inputs[0][1], htmlAttributes: new { @class = "form-control" })

My action (further below) receives the expected ScoresViewModel instance.

[HttpPost]
public ActionResult Scores(ScoresViewModel scores)
{
    if (!ModelState.IsValid)
    {
        return new HttpStatusCodeResult(HttpStatusCode.BadRequest);
    }

    return RedirectToAction(nameof(Scores));
}

Clearing the investigate label until we have more information.

Thank you for your feedback. We're closing this issue as no updates have been provided in a timely manner and we have been unable to reproduce the issue. If you have more details and are encountering this issue please add a new reply and re-open the issue.


calendar entries with MS 'Custom Time Zone'  create erroneous error messages

I found a bug in MagicMirror
Platform: Place your platform here... give us your web browser/Electron version and your hardware (Raspberry Pi 2/3/4, Windows, Mac, Linux, System V UNIX).
pi, linux
Node Version: Make sure it's version 12 or later (recommended is 14).
14
MagicMirror Version: Please let us know which version of MagicMirror you are running. It can be found in the package.json file.
2.17.1
Description: Provide a detailed description about the issue and include specific details to help us understand the problem. Adding screenshots will help describing the problem.
Steps to Reproduce: List the step by step process to reproduce the issue.
use the attached ics on calendar for 2.17.1 and see the message
Expected Results: Describe what you expected to see.
Actual Results: Describe what you actually saw.
user sees error in MM output
Moment Timezone has no data for Customized Time Zone
every time the calendar ICS is scanned.
Configuration: What does the used config.js file look like? Don't forget to remove any sensitive information!
Additional Notes: Provide any other relevant notes not previously mentioned. This is optional.
testical.ics.txt

actually emitted by the luxon library..  nothing we can do, its in the console log, and not harmful.


Index is missing

Hi,
just to note, that generated chm files, miss indexes. Hopefully indexes exist.

Klonuo,
Thank you for your inquiry. We will look into this issue and get back to you shortly.
Office Scale Team

Klonuo,
Thank you for your feedback. Please note that we provide the CHM files as a snapshot of the documentation. We don’t plan to maintain or update them moving forward. For the latest content, please view the files in GitHub or on MSDN.
Office Scale Team


PHOENIX-6943 Add MetadataCache on each region server.

Changes done in this PR:

Added DDLTimestamp maintainers as scan and mutation attribute for get and put mutations requests.
Deserialize DDLTimestamp maintainers on the regionserver.
For every DDL Timestamp maintainer, make a request to SYSCAT regionserver to get LAST_DDL_TIMESTAMP.
Cache the LAST_DDL_TIMESTAMP on the regionserver so that next request can be served from the cache.

In bunch of tests I have replaced PHOENIX_JDBC_URL with getUrl() method.
PHOENIX_JDBC_URL is evaluated to jdbc:phoenix:localhost;test=true which doesn't have zookeeper port number in the URL. It defaults to 2181 which is the default port number. Before the patch in PhoenixTestDriver, we create CQSI object once, cache it in PhoenixTestDriver and all the other calls uses the same CQSI object. But after the patch, we maintain a cache of ConnectionInfo -> ConnectionQueryServices within PhoenixTestDriver. To construct the right ConnectionInfo we need the actual zookeeper port number. Thats why we can't use PHOENIX_JDBC_URL anymore.

@jpisaac  @tkhurana  Can you review this please?


License Choice

Guys, lets discuss the choice of our license here.
Guys, lets discuss the choice of our license here.
Here are some of the popular OSI-approved licenses Source

Apache License 2.0
BSD 3-Clause "New" or "Revised" license
BSD 2-Clause "Simplified" or "FreeBSD" license
GNU General Public License (GPL)
GNU Library or "Lesser" General Public License (LGPL)
MIT license
Mozilla Public License 2.0
Common Development and Distribution License
Eclipse Public License version 2.0

I believe MIT license is best suited for us as it is very simple, to the point and lets the user do almost anything with our project.

I agree, I'll also just elaborate on that slightly by saying that we, in fact, actively encourage modification and reproduction of our code since this package is meant for learning purposes, and so the MIT license allows for that flexibility, without holding us liable to any subsequent modifications.

Also we can add, since our package is not novel work but just a reproduction of a pre-existing package, we are not adding some extra functionality to python. This package is useful for learning purposes only and well suited for beginners who are starting to build their package for python. That is why we are allowing others to distribute, modify or improve our code without our permission.


Do you have plan to add SAM as a visual encoder?

SAM can be used with Siglip/CLIP
For example, Vary uses SAM+CLIP, and Deepseek-VL uses Siglip+SAM.
Would you like to try them with this codebase?

We absolutely can; just to confirm, is this the model you'd want us to try adding: https://huggingface.co/timm/samvit_base_patch16.sa1b?
CC @ashwin-balakrishna96 to add to our internal run list!

Hello @siddk @ashwin-balakrishna96
Yes! Please try the SAM-base!
Here are some experience from my colleague:

If you want to concatenate the SAM output with the Siglip output, you may need to add 2 convolution layers after the SAM-base, to change the output size from [64, 64, 256] to [256, 1024]. You can check this figure or the Vary paper.

It's possible to use the SAM-base as the only visual encoder. But you need pretraining to align SAM-base to LLM embedding space using a small language model (e.g., OPT-125M). You may need multiple epochs in this phase

Best,
StarCycle

I guess the training pipeline is the same for Dinov2.

We cannot just use Dinov2 as the visual encoder since it is not aligned with the LLM embedding space.
We cannot align Dinov2 with contrastive learning like CLIP, which may let Dinov2 to forget many things...
But we can combine Dinov2 and a small language model, and let them predict the next token. Such generative pretraining may be very helpful. Please note that in your another work Voltron-robotics, you took exactly the approach!

Please let me know if you find anything interesting!
Best,
StarCycle

You can just start with this:
from torch import nn
from urllib.request import urlopen
from PIL import Image
import timm

class DownSampledSAMVit(nn.Module):
  def __init__(self, name, downsample_channels=(512,1024)):
    super().__init__()
    self.SAMViT = timm.create_model(
      name,
      pretrained=True,
      num_classes=0,  # remove classifier nn.Linear
    )
    data_config = timm.data.resolve_model_data_config(self.SAMViT)
    self.transforms = timm.data.create_transform(**data_config, is_training=False)

    in_channels = self.SAMViT.neck[-1].weight.shape[0]
    downsamples = []
    for i in range(len(downsample_channels)):
      out_channels = downsample_channels[i]
      downsamples.append(
        nn.Conv2d(
          in_channels,
          out_channels,
          kernel_size=3,
          stride=2,
          padding=1,
          bias=False,
        )
      )
      in_channels = out_channels
    self.downsamples = nn.Sequential(*downsamples)

  def forward(self, rgb):
    out = self.SAMViT.forward_features(rgb)
    out = self.downsamples(out)
    return out

img = Image.open(urlopen(
    'https://huggingface.co/datasets/huggingface/documentation-images/resolve/main/beignets-task-guide.png'
))

model = DownSampledSAMVit('samvit_base_patch16.sa1b').cuda().eval()
transforms = model.transforms
output = model(transforms(img).unsqueeze(0).cuda())

@StarCycle thanks a bunch for the suggestion. We can try integrating the SAM baseline in a week or so, but if you have cycles and would be interested in opening up a PR to integrate it in the meanwhile (especially because it seems like you've already been thinking about how the code should look), we would also be very happy to review it and integrate it into Prismatic :)


Using terminal mode with 90/270 degree rotation leads to incorrect output and panic

Version of ssd1306 in use (if applicable): 0.8.1
MCU/other hardware in use: Seeeduino XIAO RP2040 with expansion board
Display resolution and interface: I2C, 128x64

Description of the problem/feature request/other
When TerminalMode and Rotation90 or Rotation270 is used, writing a new line of text results in the first character being written to the first line of the display, with the remainder of the text for the same line displayed on a later, but expected, line. After writing the eighth line, attempts to write the ninth line trigger a panic, possibly because the first character of the ninth line would (incorrectly) need to be written off screen.
Test case (if applicable)
A sample reproducer using rp2040-hal is provided in plm/ssd1306-rotation-panic, with the relevant lines below.
let i2c = ... // omitted for brevity
let mut display = I2CDisplayInterface::new(i2c),
    DisplaySize128x64,
    DisplayRotation::Rotate270,
)
.into_terminal_mode();

if display.init().is_ok() && display.clear().is_ok() {
   let _ = writeln!(display, "INIT OK");
   loop {
       for count in 0..u8::MAX {
           let _ = writeln!(display, "{}TEST", count); // panics on count == 7
       }
   }
}

Expected display output, which should repeat indefinitely:
INIT OK
0TEST
1TEST
2TEST
3TEST
4TEST
5TEST
6TEST
7TEST

Actual display output, which panics when writing the eighth test line:
I0123456
 TEST
 TEST
 TEST
 TEST
 TEST
 TEST
 TEST

Hey, thanks for the detailed bug report! It made it easy to reproduce your issue. I've opened PR #197 to fix this bug.


endent, dinophilus

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[d,e]

/trunk merge


create 'Get-RemoteFileContent' with the ability to wait for non-existent files

use IO.FileSystemWatcher to wait for file creation, otherwise a simple Get-Content via remote session should be sufficient.

initial impl closed with #453ac3c


Locals map interpolation problem.

Thank you for opening an issue.
In this template paragraph text could be removed, however please retain headers.
Prerequisites

[X] Ensure you have latest version of plugin installed
[X] Search for possible issue duplicates

Installation details
GoLand 2019.3.1
Build #GO-193.5662.65, built on December 23, 2019
Licensed to Vsevolod Poliakov
Subscription is active until February 7, 2020
Runtime version: 11.0.5+10-b520.17 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.15.3
GC: ParNew, ConcurrentMarkSweep
Memory: 1979M
Cores: 16
Registry: run.processes.with.pty=TRUE, ide.completion.variant.limit=500, suggest.all.run.configurations.from.context=true, folding.check.collapse.state.before.placeholder.text=TRUE, ide.balloon.shadow.size=0, ideFeaturesTrainer.welcomeScreen.tutorialsTree=TRUE
Non-Bundled Plugins: BashSupport, IdeaVIM, Key Promoter X, PythonCore, SnakeYAML plugin, com.chrisrm.idea.MaterialThemeUI, com.daylerees.rainglow, com.intellij.tasks, izhangzhihao.rainbow.brackets, name.kropp.intellij.makefile, org.intellij.plugins.hcl, org.toml.lang

[x] intellij-hcl plugin version (Settings->Plugins)
0.7.7
[ ] Terraform version (terraform -v)
Terraform v0.12.18

Terraform Configuration Files
locals {
  current = local.workspace[terraform.workspace]
  workspace = {
    production = {
      dns_name = "some_name"
    }
  }
}

Expected Behavior
When I write local.current or local.workspace[terraform.workspace] or local.workspace["production"] I expected to see dns_name in autocomplete result and no errors.
Actual Behavior
I see the production field and this is not valid. In any of these three cases, I should see dns_name. Also, now I see an error 'unresolved references' but terraform works. So, probably it's some kind map syntax interpolation problem.
Steps to Reproduce
Create an empty tf file and put here:
locals {
  workspace = {
    production = {
      dns_name = "some_name"
    }
  }
}

locals {
  current = local.workspace["production"].dns_name
}

You will see an error.

Implementing some Terraform code as demonstrated in the for_each documentation errors out in the editor as well. It's pretty close to what @eriksw mentions above.
locals {
  # setproduct works with sets and lists, but our variables are both maps
  # so we'll need to convert them first.
  networks = [
    for key, network in var.networks : {
      key        = key
      cidr_block = network.cidr_block
    }
  ]
  subnets = [
    for key, subnet in var.subnets : {
      key    = key
      number = subnet.number
    }
  ]

  network_subnets = [
    # in pair, element zero is a network and element one is a subnet,
    # in all unique combinations.
    for pair in setproduct(local.networks, local.subnets) : {
      network_key = pair[0].key
      subnet_key  = pair[1].key
      network_id  = aws_vpc.example[pair[0].key].id

      # The cidr_block is derived from the corresponding network. See the
      # cidrsubnet function for more information on how this calculation works.
      cidr_block = cidrsubnet(pair[0].cidr_block, 4, pair[1].number)
    }
  ]
}

resource "aws_subnet" "example" {
  # local.network_subnets is a list, so we must now project it into a map
  # where each key is unique. We'll combine the network and subnet keys to
  # produce a single unique key per instance.
  for_each = {
    for subnet in local.network_subnets : "${subnet.network_key}.${subnet.subnet_key}" => subnet
  }

  vpc_id            = each.value.network_id
  availability_zone = each.value.subnet_key
  cidr_block        = each.value_cidr_block
}

When you get to the for_each statement in the resource block, .network_key and .subnet_key both are highlighted as unresolved references. I only add the comment in the event seeing another example helps. I certainly don't intend to add noise and am very appreciative of the effort and thoughtfulness put into this plugin for working with HCL.


Added back SymbolTable for exposing actual offsets of symbols in index.

Changes:

Reverted back old method that used to be there and was removed due to fact that TSDB and Prometheus was not using those.
However, since TSDB is a library, we should watch for other users, e.g Thanos (:
This is needed for Thanos index cache: https://github.com/improbable-eng/thanos/blob/943f49c2fd7575b3a15bcc27f99e9d1ce7c167f0/pkg/block/index.go#L50 This blocks bit upgrade of TSDB on Thanos side. ):
Added test for Symbol method
Initializes map for Symbols with right lengh
Made Decoder LookupSymbol method public, otherwise there is no point in having Decoder struct public.

Follow up questions:

Wonder if we should add SymbolTable method to IndexReader interface? I think not, as they are not used by consumers of this interface (e.g Compaction).
Thanos uses index.Reader struct directly.
Why Symbols returns error? I guess just for good pattern, as other implementation could want to have it? Since SymbolTable is not in interface I did not return error in method syntax.
How to ensure important methods are not removed from TSDB codebase? Is it worth to maintain list of users of this library? e.g Prometheus and Thanos, Cortex etc? That will allow to
double check in future or at least notify owners?

Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

Circle CI does not look right... I guees it is some migration period to CircleCI and it is being set up, right?

That was an internal method that should never have existed (it was a layering violation), nor are there any public methods that take offsets. I don't think this makes sense as part of the API.

Made Decoder LookupSymbol method public, otherwise there is no point in having Decoder struct public.

Decoder should probably be private.

Thanks for taking a look @brian-brazil ! (:  I can see totally your concerns with limiting API, but we have IMO valid use case: Thanos as consumer of TSDB library wants to be able use helpers for decoding index bytes:

index.Decoder
index.Reader.symbolsSlice (or symbols depends on index version)

This is to be able to implement index cache, so storing part of index (version, symbol, values, posting index) in different place than rest (actual postings and series). Which allows us to use S3 API to get_range bytes we need from index.
The truth is that TSDB format (including index format) is an API itself, so why not exposing useful helper methods for accessing it?
I agree that on IndexReader interface you don't want to expose offsets (layering violation), that's why I did not add this method to interface. However, exposing helpers for decoding index bytes on lower level for the reuse purpose is healthy both for Thanos and TSDB (we basically maintain same piece of code!). (:
If you don't agree with reusability pattern, do you have any suggestion how in Thanos we should overcome lack of such helpers?

Sound Good, will give it a try.

PTAL @brian-brazil . Updated description with summary of changes.


Explorer Compact Folders config setting no longer applies to top level folders in workspace

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.61.0
OS Version: Windows_NT x64 10.0.19043 (using WSL1)

Steps to Reproduce:

Open an existing workspace with multiple top level folders, or create one
Notice the Explorer now opens all top level folders

If this is a desirable feature, perhaps a separate config setting for applying Compact Folders setting to the top level folders, too.

Same here. This started happening yesterday. It's probably a new issue in this month's release.
"Collapse Folders in Explorer" does not work if the cursor is on a file. You have to select a top-level folder in the Explorer and then it works.

Yes, this does appear to be an exact duplicate of #134665 Sorry I missed that one.


Update plan-for-government-gcc-high.md

Planned feature: https://domoreexp.visualstudio.com/MSTeams/_workitems/edit/460108

@David-Rosenthal - please review

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@cowitkow I'm closing these 3 PRs without merging, per @daro: "We'll be shortly redirecting that section to the Teams Service Description."


flux-helm-operator status.go/annotator doesn't recover from transient TLS error to Kubernetes API

Describe the bug
The call to retrieve a list of namespaces here failed with a transient TLS error. The for loop bailed.
https://github.com/weaveworks/flux/blob/af9d3be07a240970cf1a935e4e88e0728052aad4/integrations/helm/status/status.go#L63

ts=2019-06-21T11:13:42.171831046Z caller=status.go:101 component=annotator loop=stopping err="Get https://example.hcp.westus2.azmk8s.io:443/api/v1/namespaces: net/http: TLS handshake timeout"

No further retry was made. The flux-helm-operator process didn't exit. http://<IP_ADDRESS>:3030/healthz continued to produce 200/OK
No further helm charts upgrades were started, despite updates to HelmChart resources.
To Reproduce
Steps to reproduce the behaviour:

Install flux-helm-operator container on kubernetes
Have a flaky network between flux-helm-operator pods and the api master
Wait a bit, observe TLS error
See flux-helm-operator not retry, sit stuck forever
kubectl -n flux port-forward pod/flux-helm-operator-588cb46cb8-sk9fw 3030 + curl -v http://<IP_ADDRESS>:3030/healthz = 200/OK
Delete the flux-helm-operator pod, let it restart.

Expected behavior
I expected flux-helm-operator to retry connectivity to Kubernetes API endpoint, or to exit, or to report unhealthy and a livenessProbe restart the pod.
Logs
$ kubectl -n flux logs deploy/flux-helm-operator --tail=90 -f
W0620 07:14:56.890850       7 client_config.go:549] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
ts=2019-06-20T07:15:02.019263142Z caller=helm.go:84 component=helm warning="unable to connect to Tiller" err="error getting tiller version: context deadline exceeded" host=tiller-deploy.kube-system:44134 options="{Host: Port: Namespace:kube-system TLSVerify:true TLSEnable:true TLSKey:/etc/fluxd/helm/tls.key TLSCert:/etc/fluxd/helm/tls.crt TLSCACert:/etc/fluxd/helm-ca/ca.crt TLSHostname:tiller-server}"
ts=2019-06-20T07:15:22.105590267Z caller=helm.go:88 component=helm info="connected to Tiller" version="sem_ver:\"v2.10.0\" git_commit:\"9ad53aac42165a5fadc6c87be0dea6b115f93090\" git_tree_state:\"clean\" " host=tiller-deploy.kube-system:44134 options="{Host: Port: Namespace:kube-system TLSVerify:true TLSEnable:true TLSKey:/etc/fluxd/helm/tls.key TLSCert:/etc/fluxd/helm/tls.crt TLSCACert:/etc/fluxd/helm-ca/ca.crt TLSHostname:tiller-server}"
ts=2019-06-20T07:15:22.105914964Z caller=chartsync.go:152 component=chartsync info="starting git chart sync loop"
ts=2019-06-20T07:15:22.106196363Z caller=operator.go:95 component=operator info="setting up event handlers"
ts=2019-06-20T07:15:22.106301262Z caller=operator.go:115 component=operator info="event handlers set up"
ts=2019-06-20T07:15:22.106411861Z caller=operator.go:128 component=operator info="starting operator"
ts=2019-06-20T07:15:22.106475261Z caller=operator.go:130 component=operator info="waiting for informer caches to sync"
ts=2019-06-20T07:15:22.107728953Z caller=server.go:41 component=daemonhttp info="starting HTTP server on :3030"
ts=2019-06-20T07:15:22.797249327Z caller=checkpoint.go:21 component=checkpoint msg="update available" latest=0.9.2 URL=https://github.com/weaveworks/flux/releases/tag/helm-0.9.2
ts=2019-06-20T14:24:44.63291148Z caller=checkpoint.go:21 component=checkpoint msg="update available" latest=0.9.2 URL=https://github.com/weaveworks/flux/releases/tag/helm-0.9.2
ts=2019-06-20T20:48:04.612503758Z caller=checkpoint.go:21 component=checkpoint msg="update available" latest=0.9.2 URL=https://github.com/weaveworks/flux/releases/tag/helm-0.9.2
ts=2019-06-21T01:35:21.114551968Z caller=checkpoint.go:21 component=checkpoint msg="update available" latest=0.9.2 URL=https://github.com/weaveworks/flux/releases/tag/helm-0.9.2
ts=2019-06-21T09:03:50.508971132Z caller=checkpoint.go:21 component=checkpoint msg="update available" latest=0.9.2 URL=https://github.com/weaveworks/flux/releases/tag/helm-0.9.2
ts=2019-06-21T11:13:42.171831046Z caller=status.go:101 component=annotator loop=stopping err="Get https://example.hcp.westus2.azmk8s.io:443/api/v1/namespaces: net/http: TLS handshake timeout"
ts=2019-06-21T14:18:18.163120018Z caller=checkpoint.go:21 component=checkpoint msg="update available" latest=0.9.2 URL=https://github.com/weaveworks/flux/releases/tag/helm-0.9.2

Additional context
Add any other context about the problem here, e.g

Flux version: docker.io/weaveworks/flux:1.12.3
Helm Operator version: docker.io/weaveworks/helm-operator:0.9.1
Kubernetes version: v1.12.8 in AKS
Git provider: Bitbucket
Container registry provider: Azure Container Registry

I suspect some refactor work in #2006 will actually resolve this as this no longer depends on the core API but instead reuses the shared informer.

Since the PR linked in my previous comment has landed in version 0.10.0 of the Helm operator, kubectl is no longer used in the Helm operator to request or list any resources (but it is still used to annotate resources from a release), and I expect this issue to be resolved.
If you do however run into the same (or new) symptoms, feel free to open up a new PR.


Added option to specify sid field name

This adds in the ability to change the name of the sid field by providing a sidfieldname: 'id' option.
Useful in projects where all id fields in system have the same name and the sessions table is to be used even outside of the module (for, say, reading session data for analytics).

Thanks!!


Add support for debug rendering obbs

Add optional matrix so debug AABB rendering can be oriented.

We discussed this here: https://github.com/playcanvas/engine/discussions/3073#discussioncomment-590090
I was keen to add it back then, but it was agreed to keep those outside of the code engine. Perhaps this can be still outside?

Lets add it, and create a ticket to move all these non-pubic rendering helpers to extras for now?

Created an issue: https://github.com/playcanvas/engine/issues/6024


[Feature request]: Pass config file as a commandline argument

Please describe your feature request
Having this feature would make it more convenient to run it as a system-wide service.

Just to add that in order to achieve running it as a system-wide service, more changes would be needed, e.g. wluma stores training data in $XDG_DATA_HOME.
https://github.com/maximbaz/wluma/blob/8df0b8e2c04c9e7bf0a5f560d59e29786cc5be8e/src/predictor/data.rs#L58-L59
May I ask what is your motivation for running it as a system-wide service?

Not planned for now, as I'm not really seeing a benefit, but any new arguments in favor of this request are very welcome, here or in a new issue.


garage: 0.6.1 -> 0.7.0

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This seems to be a semi-breaking update need some manual interventions: https://garagehq.deuxfleurs.fr/documentation/working-documents/migration-07


[Stepper] Remove Paper

Stepper
The root component (Paper) was replaced with a div. Stepper no longer has elevation, nor inherits Paper's props.
-<Stepper elevation={2}>
-  <Step>
-    <StepLabel>Hello world</StepLabel>
-  </Step>
-</Stepper>
+<Paper square elevation={2}>
+  <Stepper>
+    <Step>
+      <StepLabel>Hello world</StepLabel>
+    </Step>
+  </Stepper>
+<Paper>

Closes #18423

@oliviertassinari I've tweaked the Button layout and variant for the first demo to better match the original component:
Spec:
https://material.io/archive/guidelines/components/steppers.html#steppers-types-of-steppers

Before:

After:

Once it's good, I'll do the rest.

@mbrookes Looks better 👍

Option 2 – add a white background so you can distinguish the component boundaries from the demo container padding, add a divider:

@mbrookes I prefer option 1. I don't think that the stepper should have any padding.

I had to override the padding in the store:

And if you are ShutterStock, you would need to:

What about we kill this padding?

I don't think that the stepper should have any padding.

I agree. I just didn't want to be too radical!
Mk 3:

I agree. I just didn't want to be too radical!

Oh right, one breaking change at the time, removal of padding could be the very next one 👼.

Mk 3:

A couple of thoughts:

Having the buttons closer to each other makes it easier to use the playground. Otherwise, you need the mouse to make more movement to go back.
Maybe we should have one static demos of the Stepper at the top of the page with an inline preview. The demos are quite long and can be hard to navigate.
For the customized stepper, I think that we can drop the buttons, make is static, and have step 2 as active: https://deploy-preview-22564--material-ui.netlify.app/components/steppers/#customized-stepper. It would cover all the use cases and simplify the demo.
Do we need the labels like "This is the bit I really care about!", What if we already displayed the active step? e.g. "Step 2"
Remove the built-in padding of the Stepper would allow to simply the demos, as would allow to remove the padding we add on the controls.

We had a power cut, so I've been offline, but have been working on simplifying the demos. It seems like I've already hit on several if the things you suggested. Take a look.


update build.gradle "compiling with a JDK newer than" warning for jdk11

We had a talk about jdk versions today and I think we agreed it is fine to use jdk 11 for development.
Note this is different than saying Terasology engine will start using Java 11 language features; for the time being source compatibility remains at Java 8. This is just so the build environment won't scold people for using the current LTS release of OpenJDK.
it would have been faster to write a pull request than type up this issue if not for two things:

it needs documenting anyway ;)
the way it's currently written, it checks to see if the value is exactly JavaVersion.VERSION_1_8, and we want the new check to allow anything 8 – 11, and that would have taken me several moments to look up.

This came up several times on Discord by now, thus making a comment here to bump the importance 🤓


One on one skill SVG and background blobs for cards

For this section the middle image does not get imported. It seems to be an issue with how it is made in Figma. It is different from the other two images on each card. Trying the other two images leads to the correct result so the conclusion is that it is to do with the image itself and being rendered as an SVG but not optimised as one.
The other issue is that the background blobs are being imported as white and mesh with the card background.

The middle card svg is fixed by changing the svg vector from figma . i.e. Design made edits to the type of group it was in figma.
The issue with the bacgkground not appearing still exists and it does not have to do with the colour of the svg


Remove duplicate upload_file logic from Admin Ingest page.

Purpose
While looking at setting up tests for the admin app pages under #736, I realized that we can refactor the pages to have a bare minimum of business logic in them, with the logic coming from the backend utilities package.
In this PR, I've removed the dupliocate upload_file to blob storage functionality and used the functionality from the AzureBlobStorageHelper.py file instead. I've also added a missing unit test to cover this functionality, achieving the overall goal of testing the admin app pages better.
Required by #736
Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test
Deploy the admin app and test whether the following are working:

Ingestion of files
Ingestion of URLs
I've tested the above with and without integrated vectorization, and for txt and pdf files.

Other Information

Was this tested with Integrated Vectorization enabled for URLs?


Release/12.3.1

[sc-157787]
How to reproduce

Train & deploy a classical model

Create a notebook using MEA template, use the plugin codeenv for the kernel

Launch it -> 💥  (at first call of get_model_handler)

Missing an arg in the __init__ method of PredictionModelInformationHandler since folder context update

It works ! I just have 2 quick questions prior approving :)
* Is it ok if we support only saved models in the notebook ?

* I had to create a dedicate code env with the requirements from the plugin since I couldn't select the plugin managed code env when creating the notebook and therefore the `graphviz` package was missing. Am I just doing sth wrong here or is it expected ?

It was always the case since the plugin was released (and as far as I know, there were no requests on this from customers), so I guess for the fix at least we can keep as is. Perhaps a card could be opened in the backlog to see if we want to allow to use training models as well!
Normally you should be able to use the plugin code env. Are you sure the plugin code env appears in the code env list (aka was not deleted)? I guess you could try to recreate one, it should work


ged: support new reflection system

In progress.

https://github.com/tweakoz/orkid/issues/140
https://github.com/tweakoz/orkid/issues/141
https://github.com/tweakoz/orkid/issues/142
https://github.com/tweakoz/orkid/issues/144
https://github.com/tweakoz/orkid/issues/145
https://github.com/tweakoz/orkid/issues/146
https://github.com/tweakoz/orkid/issues/147


CM-90: Hashtable in C#

One other thing that would be nice is benchmarks showing why Dictionary<TKey, TValue> is preferred over HashTable since .Net Framework 2.0. See this note: https://github.com/dotnet/platform-compat/blob/master/docs/DE0006.md


Green Gem in HandInteractionExample will immediately scale the mesh down on interaction

Describe the bug
Transform Scale Handler is using the wrong scale values
To reproduce
Open HandInteractionExample scene
Try to far manipulate the green gem on the left side
Observe it snaps to a lower scale

Expected behavior
Gem should keep it's size until it gets scaled.
Gem setup should define max / min bounds that include the initial setup
Your Setup (please complete the following information)
MRTK stabilization 2.0.0
tested with WMR controller far interaction

@thalbern  I found that the slate with MRTK logo has the same symptom

the problem with the gem is that the bounding box is referring to the gem gameobject itself and the manipulationhandler points to the parent gameobject including the buttons - AND that in combination with the Transform Scale Handler simply doesn't work because it will cause a conflict.
Transform Scale Handler will apply the min max scaling relative to the transform of the gameobject it's referring to. however if both of those components (manipulationhandler and boundingbox) are attached and point to different objects they will fight against each other.
A possible solution is to move manipulation handler to the parent gameobject and have another Transform scale handler there - then it's well defined in both cases which max / min to take and also makes more sense to do it that way bc we want to include the buttons in the manipulationhandler case

@MenelvagorMilsom fyi

@thalbern Is this a side effect of https://github.com/microsoft/MixedRealityToolkit-Unity/pull/5544 or has it been this way?

@thalbern Confirmed, simply assigning the same Platonic to the ManipulationHandler fixes the problem. The MRTK logo slate's ManipulationHandler was missing transform. same solution works.

fixed in stabilization. will be in mrtk_development after next bulk merge

thanks @cre8ivepark !


An exception is returned after user permission assignment

Scenario/Intent
The SDK should be able to assign a permission to a user.
Environment Details

HPE Docker Image : hewlettpackardenterprise/hpe-oneview-sdk-for-ansible:v5.10.0-OV5.6
OneView 5.40A

Steps to Reproduce
You will find below a playbook to reproduce the issue:
- hosts: localhost
  vars:
    config: "oneview.json"
    ansible_python_interpreter: "/usr/local/bin/python3.6"
    scopeName: "ScopeTest"
    userName: admin-test
    password: "admin12345"
    roleName: "Infrastructure administrator"
    emailAddress<EMAIL_ADDRESS>    permissions: [ ]

  tasks:

    - name: "Create a scope"
      oneview_scope:
        config: "{{ config }}"
        state: present
        data:
          name: "{{ scopeName }}"

    - name: "Get scope"
      oneview_scope_facts:
        config: "{{ config }}"
        name: "{{ scopeName }}"

    - debug: var=scopes[0].uri

    - name: "Get User"
      oneview_user_facts:
        config: "{{ config }}"

    - name: "Fact"
      set_fact:
        __permissions: [ {
            roleName: "{{ roleName }}",
            scopeUri: "{{ scopes[0].uri }}"
            }
          ]

    - name: Update Permissions
      oneview_user:
        config: "{{ config }}"
        state: present
        data:
          userName: "{{ userName }}"
          password: "{{ password }}"
          emailAddress: "{{ emailAddress }}"
          enabled: true
          description: null
          mobilePhone: ""
          officePhone: ""
          fullName: "admin User"
          permissions: "{{ __permissions }}"

The user is created but the following exception is returned :
root@289e12e75de2:/playbooks# ansible-playbook scope-user-perm.yml
 [WARNING]: Found both group and host with same name: localhost

PLAY [localhost] ********************************************************************************************************************************

TASK [Gathering Facts] **************************************************************************************************************************
ok: [localhost]

TASK [Create a scope] ***************************************************************************************************************************
ok: [localhost]

TASK [Get scope] ********************************************************************************************************************************
ok: [localhost]

TASK [debug] ************************************************************************************************************************************
ok: [localhost] => {
    "scopes[0].uri": "/rest/scopes/f2fe231b-c130-46c6-8dc2-034907ea1204"
}

TASK [Get User] *********************************************************************************************************************************
ok: [localhost]

TASK [Fact] *************************************************************************************************************************************
ok: [localhost]

TASK [Update Permissions] ***********************************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: TypeError: Value of unknown type: <class 'hpeOneView.resources.security.users.Users'>, <hpeOneView.resources.security.users.Users object at 0x7fd5b57bba20>
fatal: [localhost]: FAILED! => {"changed": false, "module_stderr": "Traceback (most recent call last):\n  File \"/root/.ansible/tmp/ansible-tmp-1618822111.4147062-272120607855560/AnsiballZ_oneview_user.py\", line 113, in <module>\n    _ansiballz_main()\n  File \"/root/.ansible/tmp/ansible-tmp-1618822111.4147062-272120607855560/AnsiballZ_oneview_user.py\", line 105, in _ansiballz_main\n    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)\n  File \"/root/.ansible/tmp/ansible-tmp-1618822111.4147062-272120607855560/AnsiballZ_oneview_user.py\", line 48, in invoke_module\n    imp.load_module('__main__', mod, module, MOD_DESC)\n  File \"/usr/local/lib/python3.6/imp.py\", line 235, in load_module\n    return load_source(name, filename, file)\n  File \"/usr/local/lib/python3.6/imp.py\", line 170, in load_source\n    module = _exec(spec, sys.modules[name])\n  File \"<frozen importlib._bootstrap>\", line 618, in _exec\n  File \"<frozen importlib._bootstrap_external>\", line 678, in exec_module\n  File \"<frozen importlib._bootstrap>\", line 219, in _call_with_frames_removed\n  File \"/tmp/ansible_oneview_user_payload_wcyx0dof/__main__.py\", line 182, in <module>\n  File \"/tmp/ansible_oneview_user_payload_wcyx0dof/__main__.py\", line 178, in main\n  File \"/tmp/ansible_oneview_user_payload_wcyx0dof/ansible_oneview_user_payload.zip/ansible/module_utils/oneview.py\", line 776, in run\n  File \"/tmp/ansible_oneview_user_payload_wcyx0dof/ansible_oneview_user_payload.zip/ansible/module_utils/basic.py\", line 2348, in exit_json\n  File \"/tmp/ansible_oneview_user_payload_wcyx0dof/ansible_oneview_user_payload.zip/ansible/module_utils/basic.py\", line 2341, in _return_formatted\n  File \"/tmp/ansible_oneview_user_payload_wcyx0dof/ansible_oneview_user_payload.zip/ansible/module_utils/basic.py\", line 522, in remove_values\n  File \"/tmp/ansible_oneview_user_payload_wcyx0dof/ansible_oneview_user_payload.zip/ansible/module_utils/basic.py\", line 505, in _remove_values_conditions\nTypeError: Value of unknown type: <class 'hpeOneView.resources.security.users.Users'>, <hpeOneView.resources.security.users.Users object at 0x7fd5b57bba20>\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}
        to retry, use: --limit @/playbooks/scope-user-perm.retry

PLAY RECAP **************************************************************************************************************************************
localhost                  : ok=6    changed=0    unreachable=0    failed=1

Expected Result
Permissions correctly handled by the SDK.
Actual Result
An exception is returned by the SDK
Best Regards,
Nicolas

Hi @bigmagic currently we are not supporting users resource. we will take this as an enhancement request and will extend our support to this resource also.

Perfect. I thank you.

Going forward, enhancement requests will be addressed in the NEW HPE OneView Ansible Collection. Please consider migration to the Ansible Collection. Migration support material can be found in the Migration Support folder.

Hi @bigmagic the support for User resource has been added and also updated examples are merged into master branch. Please feel free to reopen if further help needed.


Remove deleted notes from the queue

Hi! This plugin is amazing, thank you a lot for the effort you put in.
Would it be possible to automatically remove deleted notes from the queue? Otherwise the plugin creates empty notes with the header once they come up in the queue.

Yeah that shouldn't be too difficult to add in. I'll add that into the next version.

Sweet, thank you!


Don’t fetch relations if sync is turned off

tl;dr This adds a check that Scout’s sync setting is enabled before performing the expensive queries to fetch all of an element’s relations as it’s being deleted.

I have a Craft site using Scout with 1,000+ very large entries (60-200 matrix blocks each, ~20,000 words of text content each, etc.). The publishing workflow of the site involves programmatically creating a new draft of each entry at different points once a year and updating that draft with content generated outside of Craft.
Simplified example:
$article = Entry::findOne(123);
$draft = Craft::$app->getDrafts()->createDraft($article, $currentUserId);
$newContent = MyModule::$instance->myService->getUpdatedFieldsArray();
$draft->setFieldValues($newContent);
Craft::$app->getElements()->saveElement($draft);

This process was extremely slow; on the production server it was taking 2-5 minutes per entry. In the process of debugging, I found that disabling Scout cut the time down to ~3 seconds per entry. Since this particular process should never result in any public changes that would be reflected in Algolia, I added some code to disable and re-enable Scout’s sync setting around my draft creation process.
Simplified example:
Scout::$plugin->setSettings(['sync' => false]);

// Create the new draft, update the content, and save

Scout::$plugin->setSettings(['sync' => true]);

While this did help a little, and prevented Scout from pushing any updates to Algolia, the process was still taking 2+ minutes per entry. I narrowed the issue down further to Scout’s Elements::EVENT_BEFORE_DELETE_ELEMENT listener, which was being fired dozens of times.
From what I can tell, when Craft creates the new draft of an entry, it also duplicates all of the associated matrix blocks. Then because I'm immediately overwriting those blocks with updated content, Craft was deleting each of those original duplicates. Each deleted block then hit Scout’s Elements::EVENT_BEFORE_DELETE_ELEMENT listener, which calls ->getRelatedElements(). Fetching those related elements was taking ~3-6 seconds per block. Scout then loops through the returned collection of relations in the Elements::EVENT_AFTER_DELETE_ELEMENT listener and calls ->searchable() on each to make any necessary index updates. But since sync is off, searchable() just bails early, so these queries essentially aren’t used when sync is off.
This adds a check before fetching those relations to make sure sync is actually enabled and the queries are worth executing.

@bencarr walked me through this today and this seemed like the most sensible fix.
I can't imagine a scenario where sync was off and one would want to fetch the relations…


update the codec for big decimal (#663)

Refer:
import java.math.{BigDecimal => JBigDecimal, BigInteger

implicit val javaBigDecimal: BsonCodec[JBigDecimal] =
   BsonCodec(BsonEncoder.javaBigDecimal, BsonDecoder.java

Code Link
/claim #663
Closes #663

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Looks great, please add a test that fails without the change! 🙏

Looks great, please add a test that fails without the change! 🙏

@jdegoes, I have added the test.


React capstone project - Metrics webapp

-Fetched data using API
-setup routers
-Stored the data in redux store
-displayed data from redux store
-styled the components
-deployed

INVALID ❌ ❌ ❌
Hi @karanJ2212   👋
While you made a great effort in this project, I can't go ahead and review your code.
Invalid Code Review Request ❌
You have submitted a project that is not checking for Linter Errors 😨, Kindly configure Linters and submit again once you are done 🙏
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Kindly create  a new PR that follows the Gitflow approach
Cheers, and Happy coding!👏👏👏
✅ Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.
⚠ Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Added calculator favicon.ico

hey @hemor thanks for the PR but went with a different solution!
Went with the other PR's because they solved individual issues in separate pr's :)
Appreciate the help!

I actually made the PR after fixing the favicon issue then I saw the second one and I did it but I thought PR will only point to the latest commit at the time of creating the PR and I'll have to create another PR for the second issue but it doesn't work that way.
Anyway, that's something I learnt from hacktoberfest.
Thanks.


Add function to dynamically turn specific errors into HTTPErrorResponse.

Added a test, also for https://github.com/metal-stack/metal-go/pull/44.

needs metal-go update


Chain service tweaks for testing infrastructure

These are a collection of changes that I've made to the chain service while working on testing infrastructure.
Changes

deploy now be passed an rpc endpoint (nice for scripting)
chain service now reads env vars when constructed (nice for scripting)
remove some obsolete methods from the chain service interface

I thought the wallet no longer read from env variables?

I don't think that's true for contract deployment. We're still reading the env vars for the contract addresses.

I thought the wallet no longer read from env variables?

I don't think that's true for contract deployment. We're still reading the env vars for the contract addresses.

Should we read contract env vars from the config instead? It seems bad to mix configuration strategies.


How to add a particular style for a row?

does it possible to add a custom style just a single row?
My goal is add a highlight effect for new rows.
Thanks in advance.

Hi @robertop87
You can use the beforeRowBind() to apply a custom class to your new rows.


Fail to build Gurobi 8.1.0 in JuliaPro<IP_ADDRESS>

I have installed Gurobi and set the license correctly. I have run Pkg.add("Gurobi"). I got an error when I run Pkg.build("Gurobi")
Building Gurobi → `~/.juliapro/packages/Gurobi/kaIvm/deps/build.log`
┌ Error: Error building `Gurobi`:
│ ERROR: LoadError: Unable to locate Gurobi installation. Note that this must be downloaded separately from gurobi.com
│ Stacktrace:
│  [1] error(::String) at /Applications/JuliaPro-<IP_ADDRESS>.app/Contents/Resources/julia/Contents/Resources/julia/lib/julia/sys.dylib:?
│  [2] top-level scope at /Users/smartfm/.juliapro/packages/Gurobi/kaIvm/deps/build.jl:50
│  [3] include_relative(::Module, ::String) at /Applications/JuliaPro-<IP_ADDRESS>.app/Contents/Resources/julia/Contents/Resources/julia/lib/julia/sys.dylib:?
│  [4] include(::Module, ::String) at /Applications/JuliaPro-<IP_ADDRESS>.app/Contents/Resources/julia/Contents/Resources/julia/lib/julia/sys.dylib:? (repeats 2 times)
│  [5] top-level scope at none:0
│ in expression starting at /Users/smartfm/.juliapro/packages/Gurobi/kaIvm/deps/build.jl:49
└ @ Pkg.Operations /Users/vagrant/worker/juliapro-release-osx1011-0_6/build/tmp_julia/Julia-1.0.app/Contents/Resources/julia/share/julia/stdlib/v1.0/Pkg/src/Operations.jl:1119

I checked my files there is no Gurobi810 file in Library.
Can anyone help me out?  Thanks a lot.

Please follow the installation instructions in the README: https://github.com/JuliaOpt/Gurobi.jl#installation

Note that this must be downloaded separately from gurobi.com

Have you installed Gurobi from http://www.gurobi.com/?

Yes, I installed Gurobi from http://www.gurobi.com/. Where should I set GUROBI_HOME environmental variable. I tried to run it in JuliaPro, I got:
julia> export GUROBI_HOME=/Library/gurobi810/mac64
ERROR: syntax: "/" is not a unary operator
I run it in the terminal, I just got an empty command line:
username-MacBook-Pro:~ username$ export GUROBI_HOME=/Library/gurobi810/mac64
username-MacBook-Pro:~ username$
Thanks.

julia> ENV["GUROBI_HOME"] = "/Library/gurobi810/mac64"

julia> import Pkg

julia> Pkg.build("Gurobi")

I checked my files there is no Gurobi810 file in Library.

If this is true, then you should replace "/Library/gurobi810/mac64" with the correct path.

Problem solved. Thanks a lot.


Force srlinux image to latest version in quickstart

Description
Add step to the quickstart documentation that forces the srlinux image to the latest version.
Self Check:

[ ] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[x] End user documentation is included or an issue is created for end-user documentation

Processing this pull request

Merged into branches iso6, master in c98e5027589aefc51791d3be56755c6d31707544

Processing #5315.


Add esi plugin and tests to CMake build

This compiles the esi plugin and registers all its tests with CTest. The functionality of the installed plugin cannot be easily verified because autest does not work with the CMake install yet.

The segfault was caused by the asserts being omitted from the build in Release mode. Only this test was heavily impacted, but all the esi tests will need to be updated, in a separate PR.


Adding functionality  to user model

I think when we add functionality to a model we need also to add tests.
Maybe you can add at the issue description that you gonna add some test to this model.

@Shalevro2 Yes with test_user.py file you meant right?

@Shalevro2 Yes with test_user.py file you meant right? Otherwise django-pytest CI won't pass

Yes, think so.


NPM Package not found

NPM Package not found :(

Yes this package is not available on NPM. Twitter does not support Oauth 2  for login at the moment. I originally wrote this to see if we could get it to work. Hope to complete it when OAuth 2 for login is fully supported by Twitter.
Oauth 1 strategies already exist for Twitter and Passport


Consider another color scheme for the search bar

when I search for a functionapp in the main search bar, and open it, the functionapp search bar is also populated with the same app name.
The main search bar color scheme doesn't change, but the function app blade colar scheme changes to red, which makes me think this is an error.
I instinctively click the x button and other functionapps  come up.

Dupe of https://github.com/projectkudu/AzureFunctionsPortal/issues/1041


Duplicate body_shape_entered reported on CollisionShapes not close to the collision (even when disabled)

So I've searched through the issues, and I've seen some people complain about something similar, but I feel like I have boiled it down into the most fundamental here.
Please note that I'm new to Godot, so unless I'm doing something wrong in my body_shape_entered receiver in this particular line, then you can probably disregard this as a bug report (and instead view it as a documentation issue):
func _body_hit(body_id, body, body_shape, local_shape):
    shape_owner_get_owner(shape_find_owner(local_shape))

This also seems to apply to body_shape_exited.
Godot version:
3.2.3.stable.mono.official
OS/device including version:
Windows 10. Graphics not relevant.
Issue description:
It seems that sometimes two body_shape_entered-signals are emitted when a RigidBodyhas turned on contact monitor and set the contacts reported appropriately. One for the actual collision that occurred and sometimes one for the first CollisionShape in the RigidBody. This seems to occur more often with bodies moving quickly at the RigidBody in question.
This seems like an unintended effect as it is not consistent and can happen with CollisionShapes that are disabled.
Steps to reproduce:

Create a setup that involves a RigidBodywith multiple CollisionShapes. Disable the first CollisionShape.
Create a spawner that fires a RigidBody over and over at this one (preferably ones that disappear after a second, with spawn rate set to one second).
Connect body_shape_entered and observe the passed values to that function.
Let it fire a couple of times, this bug is inconsistent.

Minimal reproduction project:
bodyshapeenteredbug.zip
Sample output:
ALERT - DISABLED ENTRY:
[RigidBody:1812]
1812
0
0
0
@@2
disabled: True
--------

Correct entry:
[RigidBody:1812]
1812
0
5
5
@@12
disabled: False
--------

My demo adds a bunch of CollisionShapes and disables all but one, hence colliding with two different shapes should be impossible, yet this shows we collided with @@12 (expected) and @@2 (should be impossible as it is disabled). If you go into Spawner.gd and change -100 to 0 on line 20, this stops happening.

Can you check if it's happening in 3.2.2? If not, then it's maybe it's caused by same change that did #44632 for me.

Ok it's unrelated, just tried and your issue reproduces in 3.2.2 too.

Poked a bit more since I was curious, it's already broken in 3.2 alpha0, but it's fine in 3.1. Unfortunately the build is broken a lot there so it's hard to bisect, but the issue is not there as of commit ee2c26a7b3, which is there 3.2 version starts according to the changelog, so it got broken somewhere between ee2c26a7b3 and 3.2 alpha0.

As identified by @snaar, this is related to CCD, which is enabled by default even when it shouldn't be (fixed with #43801, but not backported to 3.2). This is also related to #39767, i.e. reporting the wrong local collision shape, which is fixed by #42881, but hasn't been merged yet. However, this issue is not fixed with #42881, because this happens when Bullet CCD detects a collision with a RigidBody with multiple CollisionShapes. When a CCD collision is detected, it creates a collision manifold and an estimated contact point, which doesn't contain the shape data. When there is no shape data, Godot uses the first shape. This is the incorrect signal being emitted.


Database objects are still deleted after disconnection

Database objects are still deleted after disconnection。
Infomation of data change still lost between disconnection time.

Hi @dongfo.
the database objects are deleted intentionally, so the queues don't grow "infinitelly"(constrained by resources). SqlTableDependency doesn't allow to reuse previous database objects.
After reconnection you should refresh your data, in the same way as you did at the beginning with initial data, but clear the old data before it or synchronize them. You can subscribe to WhenStatusChanges and do it based on enum value TableDependencyStatus.Started.
I can provide an example later. Let me know what do you think.
Thx, Tomas

Hi,
version 2.0 has been just released. There are 2 new lifetime scopes. I hope you will enjoy them.
Tomas Fabian


dropdown -> dropup if its child of .fixed-bottom

Need additional css definitions for .dropdown-menu, in the case a parent element has the class fixed-bottom. For example a navbar on the bottom.
[File: _dropdown.scss]:
.dropup, .fixed-bottom { //add: ", .fixed-bottom"
  .dropdown-menu {
    top: auto;
    bottom: 100%;
    margin-top: 0;
    margin-bottom: $dropdown-spacer;
  }

  .dropdown-toggle {
    @include caret(up);
  }
}

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via CodePen/JS Bin or Stackblitz and report back with your link, Bootstrap version, and specific browser and OS details.
This is an automated reply

Example on CodePen:
https://codepen.io/YVEICE/pen/NOeWog/

Thoughts @twbs/css-review?

Adding the .dropup class solves this, imo it's not worth fixing this in css.


telemetry: Add suspended column

Projects, the old name for instances, might be suspended. These are not consuming any resources, and might not provide any value for the end user. It would be great if FlowForge understood how many of the instances are suspended when telemetry is provided to FlowForge.

I have raised https://github.com/flowforge/flowforge/issues/2073 to get the telemetry data updated to include this new field.


Hide split editor pane

I'd like the ability to hide split editor panes.
Sometimes when working on a feature request I get called to a really important bug fix. What I'd like to ask is that when I create a split new editor pane, there could be a button to "hide to the side" similar to what happens with the Activity Bar.
So this way I could open the new relevant tabs/files in a new split editor pane and hide the others while they're not being used but still keeping every relevant file/tab opened for that feature request.

Does https://code.visualstudio.com/updates/v1_84#_maximize-editor-group help you?

Almost there. It doesnt allow me to select one pane to be hidden. Id does the opposite, it allows one pane to appear.
I, for one, have an Ultra-Wide monitor, and I like to sometimes have two panes opened. But I'd still like for a third pane (with the tabs for other things) to be hidden temporarily.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Use Type 1 Euler fonts from amsfonts

@ylemkimon It would be nice to get rid of the eu*.mf files if possible.  Did this change adversely affect the generated fonts?


Implement parameters to pass custom ignore file

Handling of the .dockerignore and .containerignore files is implemented in #1205, however, these files require people to prepare their projects with some knowledge of how they work and thus knowledge of docker/podman. The command parameters added in this PR allows the person responsible to build the image to provide an ignore file so that classic projects do not need one.
This can be useful for example on BinderHub. The administrator can provide sensible defaults if there is no ignore file present in the repository being built or force the use of a standard one.
The implementation provide three strategies:

Merge
Ours
Theirs

Where merge, as the name suggest will put together the content of the two files, ours will force the use of the file passed in parameter and theirs will use the file present in the repository if present otherwise the one passed to the command.
Fixes #1323
Follow up off #1205

This means these files behave differently from all other configuration files. Do you think these are important enough that repo2docker should support special overrides and merging behaviour? Should something similar be considered for other files e.g. so an admin can set the default python/R version if it's not specified in a repo, or install some extra default packages like UI extensions or other utilities for local use? Should we add some callable hooks instead that allow full modification of the build context before building (and perhaps similar hooks at other stages)?

You are bringing very good questions.
In my original idea I had essentially BinderHub in mind and the optimization of the image building process. For example, avoid generating a big context with files considered useless and improve cache use (in the case of git cloning, no two clones are the same because the content of .git differs).
Your other suggestions sound interesting as well and the hooks would likely be a better fit or to keep things simple maybe a pre-build script ?
There's one thing though, the .dockerignore/.containerignore files influence what goes inside the build context while other configuration file would influence what is deployed within the image so they do not work at the same level in the build process.

The hook would be a pre-build hook called whilst constructing the build context (maybe at the start, just before creating the tar?). It would there have the full ability to add/remove/change files, whether that deleting unwanted files, adding a default version if not set, or modifying environment.yml.
Since the hook is a callable it would be defined in a repo2docker config file rather than a repo2docker command line arg, but in this particular case you need to pass the custom ignore file anyway that's not a problem.

I am not sure to follow you properly.
Do you have something like this in mind ?
repo2docker.conf:
pre-build:
  hook: /path/to/extra_ignore_hook.py
  name: extra_ignore
  args: /path/to/extra_ignore_file

?


Limitations of the Github search api

The following page shows that the API has some limitations.
https://docs.github.com/en/free-pro-team@latest/rest/reference/search#rate-limit
To solve this problem, I have been using a script that generates HTML for every few items, waits a minute when the limit is reached, and finally combines them into one HTML with my own python script.
Can you make it so that I don't have to do this and the module doesn't hit the limit?
It would make it much easier to use!

I am having this error
Run ethomson/issue-dashboard@v1
Error: Validation Failed: {"message":"The listed users and repositories cannot be searched either because the resources do not exist or you do not have permission to view them.","resource":"Search","field":"q","code":"invalid"}

is it related?


undefined is not an object (evaluating 'validAttributes[propKey]')

Facing this on 6.0.0-rc12:
undefined is not an object (evaluating 'validAttributes[propKey]')
Cannot read property 'children' of undefined

react-native 0.48.2

Same error, I found that it happens using LinearGradient, RadialGradient, Ellipse.

Upgrading to react native 0.50.1+ seems to fix it for me. @pietro909 can you confirm?

Upgrading is not an option, for now, we switched back to 5.4.2 and it works.
I have the feeling that it would fix it, though.

@serratus did we introduce this with #486? :(

@brunolemos I guess the fallback is not working properly for everyone using RN < 0.50. Do you have a different idea how to tackle that?

Checkin it

By bad, fixed on https://github.com/react-native-community/react-native-svg/pull/499


[PE-D][Tester B] Class field: Allows spaces

Was able to add a class '1 2 3 4' with spaces in between, even though the UG says that only alphanumeric characters are allowed.

User guide notes does not specify allowing spaces:

Labels: type.DocumentationBug severity.Medium
original: kimberlytmq/ped#7

Update UG so that we allow spaces (and actually any characters, I don't think we have validation)


Have rate limits for routes created by space-cloud

The problem faced currently?
Not having rate limiting on routes can be disastrous if someone tries to abuse it. It would be great to have rate limiting on routes created by space-cloud that can be enabled to protect the service. Having a meaningful default with an ability to change based on requirements would work well.
How can we solve it?
We can have simple(default) sliding window rate limiting on the routes and serve a 429 Too Many Requests with an optional Retry-After header if the rate limits are being hit.

If you want this feature to be implemented, give it a thumbs up reaction, so that we can determine which features are important to you.
👍

We can use Istio to implement this in K8s while use the serviceRouter piece in runner for docker

What options/knobs are we looking at? @prakashmishra1598 @YourTechBud
Ability to associate rate limiting on a user/role level would be amazing!

Rate limiting based on roles and users sounds interesting. We can inspect the claims and use the keys the developer has configured

We could have a default rate limits on all routes in the first place and leave it to the user to modify them based on needs. That was what I was thinking about. Rate limiting based on roles and users can be an additional feature.
Also, the implementation should ideally expose metrics related to rate limits being hit that can help alert the user of a need to increase the limits and/or scale up or of a potential abuse

This sounds like an enterprise feature


Add 0.15.1 blog post

This will add the new 0.15.1 blog post by @johnSchnake
Signed-off-by: jonasrosland<EMAIL_ADDRESS>

Codecov Report

Merging #828 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #828      +/-   ##
==========================================
+ Coverage   44.55%   44.61%   +0.06%
==========================================
  Files          76       76
  Lines        4624     4624
==========================================
+ Hits         2060     2063       +3
+ Misses       2431     2430       -1
+ Partials      133      131       -2

Impacted Files
Coverage Δ

cmd/sonobuoy/app/status.go
57.14% <0%> (-2.2%)
:arrow_down:

pkg/plugin/aggregation/aggregator.go
83.2% <0%> (+4%)
:arrow_up:

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 aead26d...5a2f275. Read the comment docs.


Logout to Login results in an error

Describe the bug
When you have an account in Payload and you login with Google, it works. But if you click the logout button and try to login again by clicking the Login with Google button you will get an error on the oauth callback.
http://localhost:3000/api/oauth/callback/google?code=4%2F....
{"errors":[{"message":"Something went wrong."}]}
You will get the following error in your terminal:
[13:58:57] ERROR: ID Token "nonce" claim missing
    err: {
      "type": "OperationProcessingError",
      "message": "ID Token \"nonce\" claim missing",
      "stack":
          OperationProcessingError: ID Token "nonce" claim missing
              at Module.processAuthorizationCodeOpenIDResponse (webpack-internal:///(rsc)/./node_modules/oauth4webapi/build/index.js:1248:23)
              at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
              at async OIDCCallback (webpack-internal:///(rsc)/./node_modules/payload-auth-plugin/dist/index.js:128:24)
              at async handleCustomEndpoints (webpack-internal:///(rsc)/./node_modules/@payloadcms/next/dist/routes/rest/index.js:194:19)
              at async eval (webpack-internal:///(rsc)/./node_modules/@payloadcms/next/dist/routes/rest/index.js:416:36)
              at async AppRouteRouteModule.do

To Reproduce
Steps to reproduce the behavior:
You have to make sure the gmail address you are using already exists in the users-collection in Payload, as it does not create a user for you on first login. Just make a default admin user and add the gmail account to the users collection.
Add a GoogleAuthProvider to the adminAuthPlugin:
 adminAuthPlugin({
      providers: [
        KeycloakAuthProvider(),
        GoogleAuthProvider({
          client_id: process.env.CLIENT_ID,
          client_secret: process.env.CLIENT_SECRET
        })
      ],
    }),

Add a login button
      <form
        action={async () => {
          'use server'
          signin('google')
        }}
        className="w-full"
      >
        <Button type="submit" className="w-full !my-0">
          Sign in with Google
        </Button>
      </form>

Login with Google, Success!
Now logout and try to log back in, it will show: {"errors":[{"message":"Something went wrong."}]}
It will give the following error in the console:
GET /api/oauth/authorization/google 302 in 13ms
[13:40:22] ERROR: ID Token "nonce" claim missing
    err: {
      "type": "OperationProcessingError",
      "message": "ID Token \"nonce\" claim missing",
      "stack":
          OperationProcessingError: ID Token "nonce" claim missing
              at Module.processAuthorizationCodeOpenIDResponse (webpack-internal:///(rsc)/./node_modules/oauth4webapi/build/index.js:1248:23)
              at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
              at async OIDCCallback (webpack-internal:///(rsc)/./node_modules/payload-auth-plugin/dist/index.js:128:24)
              at async handleCustomEndpoints (webpack-internal:///(rsc)/./node_modules/@payloadcms/next/dist/routes/rest/index.js:194:19)
              at async eval (webpack-internal:///(rsc)/./node_modules/@payloadcms/next/dist/routes/rest/index.js:416:36)

...

Expected behavior
I expected to be able to login and back in again, over and over.
Desktop (please complete the following information):

OS: MacOS Sequoia 15.1.1
Browser Chrome
Version Latest

Thanks for creating the issue, I am tracking this

We're facing the exact same error unfortunately

Please update the plugin version to 0.4.0, and also read the doc to first for any breaking changes.