Fix AlluxioUtilsSuite build on Databricks [databricks]

Databricks has an overloaded signature on PartitionDirectory that takes an array of FileStatus instead of a sequence, and calling with a sequence makes the intention ambiguous with the other overload which also takes a sequence but for a different type.  Changing to an array instead of a sequence fixes the compile issue while still remaining compatible with the original signature in Apache Spark.

build


Update Y-sort position of the first item in the sorted subtree

Fixes #79451.
Also updated the docs as Y-sorting affects not only the child nodes, the CanvasItem with y_sort_enabled = true is being Y-sorted together with its children (always was, in a potentially buggy manner but still).

Thanks!


add onboarding screen

Make the frame bit smaller otherwise it's being cut
Here instead WE NEED the back button (its a pop on the navigator)

Fix conflicts @jigneshsolanki


Resampling audio from url to byte array or stream. Linux/.NET Core

Hello,
I can't seem to get an audio byte array resampled in a way that will work debugging locally using .NET Core on my windows machine as well as hosted on Linux.
I've tried a few different ways with different providers and can't seem to get things working at all. Either the output is unintelligble or it won't run on the Linux Server (using MediaFoundationResampler)
I'm now attempting to use the WdlResamplingSampleProvider which from my understanding should work anywhere.
Any tips?
Here's what I've currently got:
public Stream ConvertMp3ToRawAudioData(string mp3Url)
        {
            using (HttpClient httpClient = new HttpClient())
            {
                var mp3Data = httpClient.GetByteArrayAsync(mp3Url).Result;
                using (MemoryStream inMemoryFile = new MemoryStream(mp3Data))
                {
                    using (var reader = new Mp3FileReader(inMemoryFile))
                    {
                        var resampler = new WdlResamplingSampleProvider(reader.ToSampleProvider(), 16000);
                        var outMemoryFile = new MemoryStream();
                        WaveFileWriter.WriteWavFileToStream(outMemoryFile, resampler.ToWaveProvider());
                        return outMemoryFile;

                        //var wp = new SampleToWaveProvider(resampler);
                        //var outMemoryFile = new MemoryStream();
                        //var writer = new WaveFileWriter(outMemoryFile, wp.WaveFormat);
                        //byte[] b = new byte[wp.WaveFormat.AverageBytesPerSecond];
                        //while (true)
                        //{
                        //    int read = wp.Read(b, 0, b.Length);
                        //    if (read > 0)
                        //        writer.Write(b, 0, read);
                        //    else
                        //        break;
                        //}

                        //outMemoryFile.Position = 0;
                        //return outMemoryFile;
                    }
                }
            }
        }

I was just having a similar issue and have something that seems to work. You can ignore the Mp3FrameDecompressor piece; I'm using NLayer because I need to run on non-Windows platforms. Key thing that was breaking it for me for a while was not calling waveFileWriter.Flush(), which then fails to write the header. So the resulting file had all the write data but was only playable in some apps (VLC), but not others.
public byte[] Resample(byte[] data, int samplingRate)
{
    var builder = new Mp3FileReaderBase.FrameDecompressorBuilder(wf => new Mp3FrameDecompressor(wf));
    using var memoryStream = new MemoryStream(data);
    using var mp3Reader = new Mp3FileReaderBase(memoryStream, builder);
    var sampleProvider = mp3Reader.ToSampleProvider();
    var resampler = new WdlResamplingSampleProvider(sampleProvider, samplingRate).ToWaveProvider();

    var outputStream = new MemoryStream();
    using var waveFileWriter = new WaveFileWriter(outputStream, resampler.WaveFormat);
    byte[] buffer = new byte[resampler.WaveFormat.AverageBytesPerSecond];
    int read;
    while((read = resampler.Read(buffer, 0, buffer.Length)) > 0)
        waveFileWriter.Write(buffer, 0, read);
    waveFileWriter.Flush();
    return outputStream.ToArray();
}

Actually, just returning outputStream.GetBuffer() in my example, rather than ToArray, works as well, without needing to call Flush.


gdown: Cannot retrieve the public link of the file. You may need to change 	the permission to 'Anyone with the link', or have had many accesses.

Description
root@localhost:~/stable-dreamfusion/pretrained/omnidata# gdown '1Jrh-bRnJEjyMCS7f-WsaFlccfPjJPPHI&confirm=t'
Access denied with the following error:
Cannot retrieve the public link of the file. You may need to change
the permission to 'Anyone with the link', or have had many accesses.

You may still be able to access the file from the browser:
 https://drive.google.com/uc?id=1Jrh-bRnJEjyMCS7f-WsaFlccfPjJPPHI&confirm=t

Steps to Reproduce
Run:
gdown '1Jrh-bRnJEjyMCS7f-WsaFlccfPjJPPHI&confirm=t'
Expected Behavior
Downloads the file
Environment
Ubuntu 22.04.2 LTS

getting the same error message


feat: Add abality to sort by other Class

@btrd can you state the use case for this? Is it to allow sorting on an association's attribute?
In any case you should update README.md with an example usage.


Test for bit flips on faulting addresses if they don't map to valid memory.

This is a fairly naive test. Generally speaking, it could end up being fairly easy to get a "hit" on valid memory by trying increasingly far offsets (by flipping each bit). This could be improved to account for memory region read/write/execute permissions in combination with the exception type (e.g. if the exception was a memory write error, only consider memory regions which were writable as valid intended addresses).
Closes #429.

This is a fairly naive test. Generally speaking, it could end up being fairly easy to get a "hit" on valid memory by trying increasingly far offsets (by flipping each bit). This could be improved to account for memory region read/write/execute permissions in combination with the exception type (e.g. if the exception was a memory write error, only consider memory regions which were writable as valid intended addresses).

Yes, that would help. There's also another case which we encounter in the wild and it's a NULL pointer with a bit flipped. We often see crashes that look like this, the original code is:
if (foo_pointer) {
  do_something_with(*foo_pointer);
}

And the crash address is exactly NULL plus one flipped bit (e.g. 0x0000000000400000). Note that this analysis would need to be done after the other address adjustment operations because we wouldn't want to catch NULL-plus-power-of-two-offset accesses which are also common.
Like you mention in the PR doing the bit-flip detection should also happen after having retrieved the "true" address of a non-canonical access. That's a very common occurrence.

I intentionally ignored the NULL case (which was described in #429) because I was confused as to the usefulness of checking for NULL. Specifically, I was thinking that the purpose of this bit flip check was to determine whether an invalid memory access was the result of a bit flip of a valid one, whereas the NULL case is an invalid access resulting from a bit flip of an invalid (NULL) one. However, the example you provided is illuminating, and it would certainly be useful to suggest that incorrect logic may have resulted from a bit flip. So I'll consider a NULL address as a potential target of bit flips errors as #429 suggested.

Like you mention in the PR doing the bit-flip detection should also happen after having retrieved the "true" address of a non-canonical access. That's a very common occurrence.

Could we perhaps change the JSON output to have adjusted_address as an array rather than a single value? Which consumers rely on this JSON schema, and is there a process for making breaking changes?

Socorro (the Mozilla crash ingestion pipeline) relies on the JSON output for the rust-minidump stackwalker. We have no process for making breaking changes that I'm aware of.
However, Socorro is using 0.15.0 from 2022-11-30 and that predates  adjusted_address, so Socorro doesn't know anything about that field, yet.

Ah, I didn't realize it was so recent. That makes breaking it less worrisome :) But it's still good to verify whether this is an appropriate change, or whether we think this particular case (divining potentially multiple adjusted addresses) will only be useful for bit-flip checking. If that is the case, it may be better to implement the bit-flip checking as a separate field rather than an AdjustedAddress.

Yes, we can add a crash ingestion processor rule that looks at the array and performs whatever heuristics we want to determine if the crash report is likely from a bit flip and then stores that determination result somewhere. I think that should work out fine.

Great idea! Do we only support amd64 right now (i.e., I don't need to check that it's amd64 prior to adding this change, for now)?

@gabrielesvelto is there anything else you think should be added now?


http_client does not work with OpenBSD.

Hi, I built lsquic on OpenBSD (6.7-beta, current with diff from https://framagit.org/ledeuns/openbsd/-/raw/master/lsquic.diff)  but it does not work.
When I launch "./http_client -L debug -s www.google.com -p /", I get "out of room in packets_in" (same problem with option -4 and I tested with multiple servers, google.com, litespeedtech.com)
I checked the code but could not find any obvious issue. Can someone give a pointer to fix this please ?
Thank you in advance,
Logfile :

14:59:32.683 [DEBUG] host: www.google.com; port: 443
14:59:32.750 [DEBUG] tokgen: TOKGEN1 does not exist: generate
14:59:32.751 [INFO] tokgen: inserted TOKGEN1 of size 110
14:59:32.751 [DEBUG] tokgen: initialized
14:59:32.751 [INFO] engine: instantiated engine
14:59:32.751 [INFO] socket buffer size: 41600 bytes; max # packets is set to 0
14:59:32.752 [DEBUG] engine: grew heaps to 4 elements
14:59:32.752 [DEBUG] conn: generated and assigned SCID EECACF0199C735DE
14:59:32.752 [DEBUG] [QUIC:] rechist: instantiated received packet history
14:59:32.752 [DEBUG] [QUIC:] rechist: instantiated received packet history
14:59:32.752 [DEBUG] [QUIC:] rechist: instantiated received packet history
14:59:32.752 [DEBUG] [QUIC:] cubic: lsquic_cubic_init(cubic, $conn)
14:59:32.752 [INFO] [QUIC:] cubic: initialized
14:59:32.752 [DEBUG] [QUIC:] cfcw: recv_off changed: read_off: 0; recv_off: 15728640
14:59:32.752 [DEBUG] [QUIC:] conn: spin bit randomly disabled; random spin bit value is 0
14:59:32.752 [DEBUG] [QUIC:EECACF0199C735DE] handshake: created client, DCID: D4CDE846BE2BDA94
14:59:32.752 [DEBUG] [QUIC:EECACF0199C735DE] handshake: client handshake secret: D79BC7853D4F19E168F1EC5465577B5F53F44
393D8F6B7FD5330DB7E400B6796
14:59:32.752 [DEBUG] [QUIC:EECACF0199C735DE] handshake: server handshake secret: 16BA81205F7A4879398ABB8086814E302D45B
C8B39EE7908504A9E3E10873900
14:59:32.753 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] di: initialized
14:59:32.753 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] sfcw: recv_off changed: read_off: 0; recv_off: 16384
14:59:32.753 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] stream: created crypto stream
14:59:32.753 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] stream: calling on_new_stream
14:59:32.753 [DEBUG] [QUIC:EECACF0199C735DE] handshake: Create new SSL_CTX
14:59:32.889 [DEBUG] [QUIC:EECACF0199C735DE] handshake: generated transport parameters buffer of 43 bytes
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] handshake: initialized client enc session
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] handshake: handshake stream created successfully
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: negotiating version FF00001B
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: logging using client SCID
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] event: connection created
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] qlog: [15271142099,"CONNECTIVITY","NEW_CONNECTION","LINE",{"ip_version":"
6","srcip":"::","dstip":"2a00:1450:4001:81d::2004","srcport":"23273","dstport":"443"}]
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] event: version negotiation: proposed version FF00001B
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] qlog: [15271142110,"CONNECTIVITY","VERNEG","LINE",{"proposed_version":"FF
00001B"}]
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: record path 0: (:::23273 - 2a00:1450:4001:81d::2004:443)
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: record new path ID 0
14:59:32.890 [DEBUG] engine: incref conn EECACF0199C735DE, 'H' -> 'HT'
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: delayed stream creation.  Backlog size: 1
14:59:32.890 [DEBUG] engine: decref conn EECACF0199C735DE, 'HT' -> 'H'
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] pacer: lsquic_pacer_can_schedule: 1
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] stream: buffered 512 bytes; 512 bytes are now in buf
fer
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] handshake: wrote 512 bytes to stream at encryption level 0
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] stream: will flush up to offset 512
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] sendctl: lsquic_send_ctl_can_send: n_out: 0 (unacked_all: 0); cwnd: 46720
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] pacer: lsquic_pacer_can_schedule: 1
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] sendctl: created packet 0
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] event: created packet 0; flags: version=1, nonce=0, conn_id=1
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] pacer: lsquic_pacer_packet_scheduled: replenish tokens: 10
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] pacer: lsquic_pacer_packet_scheduled: tokens: 9
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] event: generated CRYPTO frame: offset: 0, size: 512
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] stream: flushed to or past required offset 512
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] handshake: retry read
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] handshake: enc level clear after on_write: want read
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] handshake: Exit shake_stream
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: sit-0 streams: max count: 0; created streams: 0
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: created 0 delayed streams
14:59:32.890 [DEBUG] engine: incref conn EECACF0199C735DE, 'H' -> 'HO'
14:59:32.890 [DEBUG] engine: incref conn EECACF0199C735DE, 'HO' -> 'HOK'
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] sendctl: cum_size: 559; limit: 1200
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] sendctl: Added 641 bytes of PADDING to packet 0
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] handshake: apply header protection using mask D4F3D251A7
14:59:32.890 [DEBUG] engine: batched packet 0 for connection EECACF0199C735DE
14:59:32.890 [DEBUG] engine: batched all outgoing packets for full conn EECACF0199C735DE
14:59:32.890 [DEBUG] engine: packets out returned 1 (out of 1)
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] event: sent packet 0, type Initial, crypto: clear, size 1200, frame types
: PADDING CRYPTO, ecn: 0, spin: 0; kp: 0, path: 0, flags: 1147658
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] sendctl: packet 0 has been sent (frame types: PADDING CRYPTO)
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] sendctl: set retx alarm to<PHONE_NUMBER>0, which is 150000 usec from now, mo
de RETX_MODE_HANDSHAKE
14:59:32.890 [DEBUG] engine: decref conn EECACF0199C735DE, 'HOK' -> 'HK'
14:59:32.890 [DEBUG] engine: send_packets_out: sent 1 packet
14:59:32.890 [DEBUG] engine: decref conn EECACF0199C735DE, 'HK' -> 'H'
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE-18446744073709551615] di: get_frame: no frame at read offset 0
14:59:32.890 [DEBUG] [QUIC:EECACF0199C735DE] conn: not tickable
14:59:32.890 [DEBUG] engine: incref conn EECACF0199C735DE, 'H' -> 'HA'
14:59:32.890 [DEBUG] engine: next advisory tick is 149864 usec away: conn EECACF0199C735DE: RETX_INIT
14:59:32.890 [DEBUG] entering event loop
14:59:32.921 [DEBUG] out of room in packets_in
14:59:32.921 [DEBUG] out of room in packets_in
14:59:32.921 [DEBUG] out of room in packets_in
14:59:32.921 [DEBUG] out of room in packets_in
14:59:32.921 [DEBUG] out of room in packets_in
[...]

This is the key log message:

14:59:32.751 [INFO] socket buffer size: 41600 bytes; max # packets is set to 0

The corresponding code is
    socklen_t opt_len;
    int recvsz;

    opt_len = sizeof(recvsz);
    if (0 != getsockopt(fd, SOL_SOCKET, SO_RCVBUF, (void*)&recvsz, &opt_len))
    {
        LSQ_ERROR("getsockopt failed: %s", strerror(errno));
        return NULL;
    }

    n_alloc = (unsigned) recvsz / MAX_PACKET_SZ * 2;
    LSQ_INFO("socket buffer size: %d bytes; max # packets is set to %u",
        recvsz, n_alloc);

Because MAX_PACKET_SZ is 0xFFFF, the maximum number of packets is calculated to be zero...  Admittedly, this is not the best logic in the world.
To fix this, you can just change MAX_PACKET_SZ to some other value.

Nice catch! If I set MAX_PACKET_SZ to 1500, it works :)
Thank you very much Dmitri.

I am glad to hear it works -- I will add OpenBSD to the list of supported platforms.  (After deciding on what the proper fix should be...)

Oh, and I missed that diff above!  Thanks.  If you want to open a PR, we always like that.  If not, no bid deal: I'll merge the diff myself.

The above diff is not enough, OpenBSD does not have STAILQ_.
I did not know how to add it to this diff so I added STAILQ_ manually to sys/queue.h on my system.

Oh. There is a sys/queue.h under wincompat directory that gets included for Windows build. You could do something similar for OpenBSD.


For beginners: Change TEST_MSG_MAXSIZE into a run-time limit.

Currently, there is a compile-time constant limiting length of helper messages output via TEST_MSG macro. It would be good to change it so that people do not have to re-compile of they need to see longer message.
Because Acutest should not output too long things by default, it still should have a limit, but it should be changed into run-time one. I'd suggest that TEST_MSG_MAXSIZE will be kept but will only determine the default length and user could specify another one by a new command line option, say --max-msg=NUMBER.

Hi @mity. Thank you for acutest, I've used if for years as my goto for any C I write.
I'm working on this issue now as my intro to open source contributions.  I understand it's a pretty simple feature, but I'm comprehensively documenting the process as practice and to add to the pull request.  I'm new to working on open source projects, so any feedback will be appreciated.
I see the equivalent issue for TEST_DUMP_MAXSIZE, #34.
Once this one is done, would you like that to also be implemented for consistency, or leave it for another beginner?


CreateExternalTable DDL supports table_partition_cols

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Assume we have a data lake stores as
table/year=2022/month=03/day=20/log.parquet
table/year=2022/month=03/day=21/log.parquet

Currently, CreateExternalTable supports defining columns and location (e.g. table/)
https://github.com/apache/arrow-datafusion/blob/5936edc2a94d5fb20702a41eab2b80695961b9dc/datafusion/src/sql/parser.rs#L70-L81
a sql query of select * from table where year = '2022' and month = '03' and day = '20' seems to scan all files under table/.
Describe the solution you'd like
CREATE EXTERNAL TABLE test (
    c1  VARCHAR NOT NULL,
)
STORED AS CSV
WITH HEADER ROW
PARTITIONED BY (p1, p2)
LOCATION '/path/to/';

same as existing ListingOption, PARTITIONED BY only supports String
https://github.com/apache/arrow-datafusion/blob/5936edc2a94d5fb20702a41eab2b80695961b9dc/datafusion/src/datasource/listing/table.rs#L178
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
partitioned by is also used in Trino and AWS Athena
https://trino.io/episodes/5.html
https://docs.aws.amazon.com/athena/latest/ug/create-table.html
I notice that ListingOptions supports table_partition_cols and also partition pruning, but just CreateExternalTable does not accept such input and pass through
https://github.com/apache/arrow-datafusion/blob/5936edc2a94d5fb20702a41eab2b80695961b9dc/datafusion/src/datasource/listing/table.rs#L165-L186
https://github.com/apache/arrow-datafusion/blob/5936edc2a94d5fb20702a41eab2b80695961b9dc/datafusion/src/datasource/listing/table.rs#L358-L365

sounds like a good enhancement to me


Numeric keyboard on mobile

Hi, there's any way to make the plugin behaves a little bit more like the type=number on mobile opening the numeric keyboard by default? When you start to edit a type=number field on mobile the numeric keyboard opens by default and it helps a lot on the form usability.
Thanks for your attention.

@vlad-ignatov is it simply not compatible?

https://developer.mozilla.org/en/docs/Web/HTML/Element/input inputmode="numeric" This should work

You should be able to make that using combination of pattern and inputmode props. For example pattern="[0-9]*" inputmode="numeric" - https://stackoverflow.com/questions/6178556/phone-numeric-keyboard-for-text-input.
This component supports more than just numbers thus it relies on input[type="text"] but at least you are able to invoke the numeric keyboard if you only work with integers.


handle securecookie: the value is too long: 5004

Hi, I am looking for alterntives to the cookie store due to cookie size limitations. So I am trying this one, but it gives me the same error. Is there a reason for this limitation?
securecookie: the value is too long: 5004

Ok using the another storage method solves the problem. I am using gob now.  https://github.com/bradleypeabody/gorilla-sessions-memcache?tab=readme-ov-file#storage-methods


Longitudinal classification handling

Dear Authors,
fantastic initiative for the healthcare community!
Understanding that you speak about "(1) reducing the requirements for evaluating longitudinal data and (2) offering a baseline set of method implementations for the community" in your paper - Would you please guide me toward your Longitudinal Machine Learning Classifiers for Longitudinal Classification, such as Adapted Random Forest or Similar Methods? Is there a future strategy there in your plans? What are your estimators in this regard?
We would like to analyse some of our longitudinal data at the hospital where I work, perhaps using your estimators, but I was unable to locate any in the unfinished documentation.
best regards,

Dear Authors, you appears busy. I am closing this now, happy to discuss it further when you become a little more available. Cheers.


[Pagination] [React] Keyboard accessible even when pagination is disabled, and style changes on hover

Description
Keyboard accessible even when pagination is disabled, and style changes on hover

In the example of the official Pagination document, I think it is correct that you cannot access the tab when Previous is disabled.
Also, the style changing on mouse hover is also not correct!
Link to Reproduction
https://ark-ui.com/docs/react/components/pagination
Steps to reproduce

Go to Link

Ark UI Version
recent
Framework

[X] React
[ ] Solid
[ ] Vue

Browser
Google Chrome 112.0.5615.138
Additional Information
Window 11

That one is a bit tricky. By definition a pagination can be a button or a.
While a button can be disabled a link can not.


vNext: Gizmo prefabs "no meshrenderer" errors

The models that were imported for the Gizmo prefabs have a mesh renderer down in the hierarchy. Unity is throwing a MissingComponentException when the visualizer tries to fetch it.
MissingComponentException: There is no 'MeshRenderer' attached to the "GizmoLeft(Clone)" game object, but a script is trying to access it.
You probably need to add a MeshRenderer to the game object "GizmoLeft(Clone)". Or your script needs to check if the component is attached before using it.
UnityEngine.Renderer.get_material () (at C:/buildslave/unity/build/Runtime/Export/GraphicsRenderers.bindings.cs:123)
Microsoft.MixedReality.Toolkit.Tests.ControllerVisualizer.UpdateControllerVisuals () (at Assets/MixedRealityToolkit-Examples/Test/ControllerVisualizer.cs:65)
Microsoft.MixedReality.Toolkit.Tests.ControllerVisualizer.Update () (at Assets/MixedRealityToolkit-Examples/Test/ControllerVisualizer.cs:134)

We either need to modify the model (and learn how) or have the script ask for renderers on children as well.

Why don't we just use the default position handle gizmo in the editor?
It's not like we're going to have ppl use these arrows in a final build

this stems from calls in the test scene (from #2478) that is not querying deep enough for the renderer. I thought it might be part of the perf issue, but it is not.
I would still like to understand how best to fix this going forward as we may encounter models with this issue in the wild.

I don't think there's anything to fix here, since it was a test script that hasn't been (and won't be?) checked into the Toolkit was throwing the error, right?


Add README for each package.

@krisselden any specific information you want, or just the generic NPM readme with the package details?

Ideally we say what the conceptual point of the package is. Possibly even an arch diagram.
Since we are publishing them all separately, this README is also displayed when the package is viewed in NPM.

@locks I can pick this one, can you please assign it to me?

Done!

I think the first step here is to submit a PR with a proposal for the boilerplate README that we can add to all of the packages. After we decide on that we can create all of the READMEs and then fill them out with details as we go along.


Site does not deploy to Netlify?

Describe the bug
Cannot deploy after 5 attempts
Command failed
npm run dev
Command failed
npm run build
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-cnrnnby6
Steps to reproduce
Press the deploy button
Expected behavior
Deploy to Netlify seamlessly
Screen Recording / Screenshot
https://app.screencast.com/MD3jXixhUZ4kk
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 82418471,
  "usedJSHeapSize": 78698787,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = andaydvice
Chat ID = 64a30902a835

Additional context
No response

Bolt tried to fix problems 5 times but failed. To prevent spending more tokens, you might want to try to fix them manually or rollback to a previous checkpoint instead.

Yes I did roll it back and another 5 attempts failed wasting yet more credits because of errors

This happened to me as well. Also it seems like Bolt will keep trying the same fix over and over again. Also, Also, there seems to a lot of type errors.
Here my current workflow, Copy and paste the error into Claude or ChatGPT, Copy and paste the AI suggested fix back into Bolt. Claude and ChatGPT always seem to find different fixes than Bolt.


SAK-30952 Show instructions for the hidden sites preference

Introduces a new hidden_instructions I18N key that will need
localization.

:+1:

After merging #2342 now you have a conflict in this one, please solve it.

Just fixed, thanks :)

Cool !! Thanks !!


add a more recent EBL model

A more recent EBL model has been published by Saldana-Lopez et al. (2021), MNRAS, 507, 5144
Supporting Material can be downloaded here

added!


.github/workflows: add action to update tapioca definitions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

This is a test of the changes of #8233, but raised from a branch on the main repo, not from a fork, so that the Actions run has access to the secrets. Let's see if it works!
We're expecting to see a PR raised by BrewTestBot.

Generating split RBIs into sorbet/rbi/hidden-definitions/
fatal: ambiguous argument 'Library/Homebrew/sorbet': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
Switched to a new branch 'tapioca-update-2020-08-10'
fatal: could not read Username for 'https://github.com': No such device or address
##[error]Process completed with exit code 128.

Is this related to GITHUB_TOKEN?

Still the same problem where we can't push even from a not-forked PR. It's trying to push over HTTPS, which requires a GitHub username and a personal access token. We've got the token, which should work fine (tho I can't see what scopes it has). I wonder if there's a way to specify the username. Or we could git remote set-url<EMAIL_ADDRESS>to force SSH. 🤔

Oh good, at least this is a different error!

OK, so we're getting further, it's trying to push but can't push:
Switched to a new branch 'tapioca-update-2020-08-10'
fatal: refs/remotes/origin/HEAD cannot be resolved to branch

🤯

You can't push to BrewTestBot's repository using GITHUB_TOKEN.

This existing workflow already works, so you should be able to copy what it's doing without much modification: https://github.com/Homebrew/brew/blob/master/.github/workflows/spdx.yml

@jonchang We're trying to push to Homebrew/brew branch tapioca-.... We've basically copied the License workflow (the only changes are the Sorbet lines, now I've reset to what @vidusheeamoli had originally). So I'm confused as to how that works but this doesn't.

From @Bo98 in Slack:

I see your issue - you are running the action in the context of a pull request. You cannot push to origin that way - that would be a security exploit. You must test the action another way (e.g. wait for the scheduled event).

We'll have to merge this (or preferably #8233) as-is and wait three days - which shouldn't be a problem given it's the same as the License Action and we know the commands work.

Because this new workflow is triggered on the push event, it will run on any pull requests affecting that file. You still can't push to origin from a pull request due to security reasons. So the workflow needs to be broken up into two separate steps:

Do the type checking and updating of sorbet files
Run git commit, git push

Step 2 needs to be restricted where the GITHUB_REF is master, so it won't try to push when running against a pull request.

I just noticed this: also need to use actions/checkout since that will automatically set the right permissions to push to origin, using the default GITHUB_TOKEN.

actions/checkout won't work well with Homebrew/actions/setup-homebrew. I would propose using Homebrew/actions/git-try-push for pushing the branch.

Step 2 needs to be restricted where the GITHUB_REF is master, so it won't try to push when running against a pull request.

This isn't quite right. It's fine to run on push on Homebrew/brew and push events aren't triggered from fork pull requests.

After much head scratching (thanks so much to Dawid and Jon!), this finally generated a PR! 🎉 Here it is: https://github.com/Homebrew/brew/pull/8361
I had to remove the date specifier for the branch as I couldn't set it dynamically to the current date in the git-try-push params, so we should set "automatically delete HEAD branches" in this repo's settings if it's not already on (I don't have enough powers to see or set it). Then every three days it'll use a clean branch to run the Tapioca stuff and raise a PR from. I think we're good at merging things fast enough to get the PR dealt with within three days!
I'll leave this open for a bit so people can check it's not doing anything weird, or if people have ideas for improvements.

One thing I notice is CI doesn't appear to have triggered on the above test PR.

Yeah, I don't think CI is triggered by @github-actions. We had the same problem here: https://github.com/Homebrew/homebrew-cask-fonts/pull/2253#issuecomment-673288778

I had to remove the date specifier for the branch as I couldn't set it dynamically to the current date in the git-try-push params, so we should set "automatically delete HEAD branches" in this repo's settings if it's not already on (I don't have enough powers to see or set it).

Does setting an output variable and using it as a substitution like ${{ steps.id.output.branch }} work?

To summarise the current status of this:

[x] Pushing works with GITHUB_TOKEN.
[x] Opening a PR works with GITHUB_TOKEN.
[ ] PRs raised by github-actions[bot] don't trigger CI, so the PR isn't mergeable. Using HOMEBREW_GITHUB_API_TOKEN to (I assume) raise the PR as BrewTestBot doesn't authenticate properly.
[ ] Need to test if we can get date-specific branch names with the workaround that Jon suggested.

I'm not sure HOMEBREW_GITHUB_API_TOKEN secret is available in this repo. Had the same issue in Homebrew/actions. Someone with higher access level would need to check and enable it for this repo if it isn't. IIRC this secret is available in org settings, but I might be wrong.

Ohh. I thought I pulled that as an example from the License action, but it turns out that I read the line wrong and it's the other way around.
Either way, we want to raise PRs as BrewTestBot, I think, so we'll need some other token than the Actions built-in one.

I had to remove the date specifier for the branch as I couldn't set it dynamically to the current date in the git-try-push params, so we should set "automatically delete HEAD branches" in this repo's settings if it's not already on (I don't have enough powers to see or set it). Then every three days it'll use a clean branch to run the Tapioca stuff and raise a PR from. I think we're good at merging things fast enough to get the PR dealt with within three days!

I think it would be nice to use some sort of unique branch identifier OR to force-push to the existing branch name if there's changes rather than failing the workflow.

I'm not sure HOMEBREW_GITHUB_API_TOKEN secret is available in this repo.

It is not. If someone can let me know what rights it needs: I can add it.

[HOMEBREW_GITHUB_API_TOKEN] is not [set]. If someone can let me know what rights it needs: I can add it.

We need this Action to be able to use a BrewTestBot PAT to push commits to a branch of this repo and raise PRs. That is, public_repo.

PRs raised by github-actions[bot] don't trigger CI

Why is this the case?

@Bo98, I think it's to avoid recursive actions. You cannot trigger actions with the default GITHUB_TOKEN.

@vidusheeamoli @issyl0 Check out https://github.com/Homebrew/formulae.brew.sh/blob/master/.github/workflows/scheduled.yml and https://github.com/Homebrew/rubydoc.brew.sh/blob/master/.github/workflows/scheduled.yml. I've updated them recently to use Homebrew's actions as much as possible and they should just need use the @BrewTestBot token (as mentioned above).


[kube-prometheus] Specify version of k8s and add backwards compatibility

What did you do?
Running contrib/kube-prometheus/hack/cluster-monitoring/deploy does not seem to work on k8s 1.5.x.
What did you expect to see?
Ideally the readme for kube-prometheus would indicate which version(s) of the k8s it is targeting.  If possible, would ideally have the script dynamically work with different versions of k8s either using a flag or automatically detecting the api version.
What did you see instead? Under which circumstances?
unable to decode "manifests/prometheus-operator/prometheus-operator-cluster-role-binding.yaml": no kind "ClusterRoleBinding" is registered for version "rbac.authorization.k8s.io/v1beta1"
unable to decode "manifests/prometheus-operator/prometheus-operator-cluster-role.yaml": no kind "ClusterRole" is registered for version "rbac.authorization.k8s.io/v1beta1"

Environment

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.4", GitCommit:"7243c69eb523aa4377bce883e7c0dd76b84709a1", GitTreeState:"clean", BuildDate:"2017-03-07T23:53:09Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"5+", GitVersion:"v1.5.4-rancher1", GitCommit:"6ed2b64b2e1df9637661077d877a0483c58a6ae5", GitTreeState:"clean", BuildDate:"2017-03-17T16:58:04Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:

Created with Rancher

Manifests:

See contrib/kube-prometheus/hack/cluster-monitoring/deploy

Yes we should definitely write that in there. @mgoodness can we stop helm from applying if a minimum cluster version is not met? I feel that would be the safest thing to do with the points of friction.

@brancz I was trying to make the current scripts backwards compatible with 1.5 by removing the rbac related stuff, but I'm still seeing errors when running.  Any ideas?  Is there a specific commit hash of kube-prometheus that should work with 1.5?
Output:
slkc:kube-prometheus ryan$ ./hack/cluster-monitoring/deploy
Error from server (AlreadyExists): namespaces "monitoring" already exists
serviceaccount "prometheus-operator" created
deployment "prometheus-operator" created
Waiting for Operator to register third party objects...done!
daemonset "node-exporter" created
service "node-exporter" created
deployment "kube-state-metrics" created
serviceaccount "kube-state-metrics" created
service "kube-state-metrics" created
secret "grafana-credentials" created
secret "grafana-credentials" configured
configmap "grafana-dashboards" created
deployment "grafana" created
service "grafana" created
configmap "prometheus-k8s-rules" created
serviceaccount "prometheus-k8s" created
service "prometheus-k8s" created
unable to decode "manifests/prometheus/prometheus-k8s-service-monitor-alertmanager.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
unable to decode "manifests/prometheus/prometheus-k8s-service-monitor-apiserver.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
unable to decode "manifests/prometheus/prometheus-k8s-service-monitor-k8s-apps-http.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
unable to decode "manifests/prometheus/prometheus-k8s-service-monitor-kube-state-metrics.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
unable to decode "manifests/prometheus/prometheus-k8s-service-monitor-kubelet.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
unable to decode "manifests/prometheus/prometheus-k8s-service-monitor-node-exporter.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
unable to decode "manifests/prometheus/prometheus-k8s-service-monitor-prometheus.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
unable to decode "manifests/prometheus/prometheus-k8s.yaml": no kind "Prometheus" is registered for version "monitoring.coreos.com/v1alpha1"
secret "alertmanager-main" created
service "alertmanager-main" created
error: unable to decode "manifests/alertmanager/alertmanager.yaml": no kind "Alertmanager" is registered for version "monitoring.coreos.com/v1alpha1"

@brancz I think we could build that logic into the charts, sure.

@ryanwalls I've been using 13a0dfbfc44f6c025363e612b469786a3cca091b on k8s 1.5 - right before the 1.6 changes...

@ryanwalls those errors are just because kubectl was not able to use apply for TPRs pre 1.6. To fix those errors you're having, just replace any kubectl apply call to TPRs in the deploy scripts to  kubectl create.

@dmcnaught When I use that commit 13a0dfbfc44f6c025363e612b469786a3cca091b, it has errors spinning up the alertmanager statefulset.
MountVolume.SetUp failed for volume "kubernetes.io/configmap/4c656171-3913-11e7-b6ca-0285ffacd72b-config-volume" (spec.Name: "config-volume") pod "4c656171-3913-11e7-b6ca-0285ffacd72b" (UID: "4c656171-3913-11e7-b6ca-0285ffacd72b") with: configmaps "main" not found
Error syncing pod, skipping: timeout expired waiting for volumes to attach/mount for pod "monitoring"/"main-0". list of unattached/unmounted volumes=[config-volume]

It seems to be looking for a ConfigMap named main that doesn't seem to be created.

In the v0.7.0 release we replaced all uses of ConfigMaps for configurations to Secrets, as the Alertmanager and Prometheus configs may contain secrets.
https://github.com/coreos/prometheus-operator/releases/tag/v0.7.0

@brancz The only thing I can think of is that somehow the tagged v0.7.0 prometheus operator has the wrong version of the alertmanager thirdpartyresource somehow.   Because I can definitely see it should be trying to load the secret instead of the configmap from https://github.com/coreos/prometheus-operator/blob/43fdeb95df2003c216b63a2cc486c052033bd4c7/pkg/alertmanager/statefulset.go#L234-L243, but the prometheus operation v0.7.0 is registering a tpr for alertmanager that is trying to load a configmap.

The TPR is just a mean of storing arbitrary data, the registered TPR has not changed since inception. The only difference is how the Operator handles the content of a TPR differently. You're referencing different commits of code and usage. Can you elaborate which versions you are using and if the Alertmanager pod stays this way, maybe you can post the, pod yaml so we can investigate further. Thanks! :slightly_smiling_face:

@ryanwalls I didn't see/notice that error, but looking at my notes I did see problems with data not appearing in Grafana.
My clusters are actually running
https://github.com/coreos/kube-prometheus.git at 333bd23434a8da6ed8bf4b6e57e72e71f75dbc40

@brancz Yeah, I'm not entirely clear on how the versions inter-operate with respect to TPRs.
Using the commit the @dmcnaught mentioned, when I run the /hack/cluster-monitoring/deploy script, I get two stateful sets created.  One called alertmanager-main and one just called main.  This seems odd as previous versions created only one stateful set for alertmanager.

Both of the stateful sets seem to be looking for ConfigMaps, not secrets.
Here is the stateful set definition for main:
apiVersion: apps/v1beta1
kind: StatefulSet
metadata:
  creationTimestamp: 2017-05-15T19:11:55Z
  generation: 5
  labels:
    alertmanager: main
    app: alertmanager
  name: main
  namespace: monitoring
  resourceVersion: "13287298"
  selfLink: /apis/apps/v1beta1/namespaces/monitoring/statefulsets/main
  uid: 5c607c7e-39a2-11e7-b6ca-0285ffacd72b
spec:
  replicas: 3
  selector:
    matchLabels:
      alertmanager: main
      app: alertmanager
  serviceName: alertmanager
  template:
    metadata:
      creationTimestamp: null
      labels:
        alertmanager: main
        app: alertmanager
    spec:
      containers:
      - command:
        - /bin/alertmanager
        - -config.file=/etc/alertmanager/config/alertmanager.yaml
        - -web.listen-address=:9093
        - -mesh.listen-address=:6783
        - -storage.path=/etc/alertmanager/data
        - -mesh.peer=main-0.alertmanager.monitoring.svc
        - -mesh.peer=main-1.alertmanager.monitoring.svc
        - -mesh.peer=main-2.alertmanager.monitoring.svc
        image: quay.io/prometheus/alertmanager:v0.5.1
        imagePullPolicy: IfNotPresent
        name: main
        ports:
        - containerPort: 9093
          name: web
          protocol: TCP
        - containerPort: 6783
          name: mesh
          protocol: TCP
        resources: {}
        terminationMessagePath: /dev/termination-log
        volumeMounts:
        - mountPath: /etc/alertmanager/config
          name: config-volume
        - mountPath: /var/alertmanager/data
          name: main-db
      - args:
        - -webhook-url=http://localhost:9093/-/reload
        - -volume-dir=/etc/alertmanager/config
        image: jimmidyson/configmap-reload
        imagePullPolicy: Always
        name: config-reloader
        resources:
          limits:
            cpu: 5m
            memory: 10Mi
        terminationMessagePath: /dev/termination-log
        volumeMounts:
        - mountPath: /etc/alertmanager/config
          name: config-volume
          readOnly: true
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      securityContext: {}
      terminationGracePeriodSeconds: 0
      volumes:
      - configMap:
          defaultMode: 420
          name: main
        name: config-volume
      - emptyDir: {}
        name: main-db
status:
  replicas: 1

And here is the one for alertmanager-main:
apiVersion: apps/v1beta1
kind: StatefulSet
metadata:
  creationTimestamp: 2017-05-15T19:19:58Z
  generation: 1
  labels:
    alertmanager: main
  name: alertmanager-main
  namespace: monitoring
  resourceVersion: "13287694"
  selfLink: /apis/apps/v1beta1/namespaces/monitoring/statefulsets/alertmanager-main
  uid: 7c38a8dc-39a3-11e7-b6ca-0285ffacd72b
spec:
  replicas: 3
  selector:
    matchLabels:
      alertmanager: main
      app: alertmanager
  serviceName: alertmanager-operated
  template:
    metadata:
      creationTimestamp: null
      labels:
        alertmanager: main
        app: alertmanager
    spec:
      containers:
      - command:
        - /bin/alertmanager
        - -config.file=/etc/alertmanager/config/alertmanager.yaml
        - -web.listen-address=:9093
        - -mesh.listen-address=:6783
        - -storage.path=/etc/alertmanager/data
        - -mesh.peer=alertmanager-main-0.alertmanager.monitoring.svc
        - -mesh.peer=alertmanager-main-1.alertmanager.monitoring.svc
        - -mesh.peer=alertmanager-main-2.alertmanager.monitoring.svc
        image: quay.io/prometheus/alertmanager:v0.5.1
        imagePullPolicy: IfNotPresent
        name: alertmanager
        ports:
        - containerPort: 9093
          name: web
          protocol: TCP
        - containerPort: 6783
          name: mesh
          protocol: TCP
        readinessProbe:
          failureThreshold: 10
          httpGet:
            path: /api/v1/status
            port: web
            scheme: HTTP
          initialDelaySeconds: 3
          periodSeconds: 5
          successThreshold: 1
          timeoutSeconds: 3
        resources: {}
        terminationMessagePath: /dev/termination-log
        volumeMounts:
        - mountPath: /etc/alertmanager/config
          name: config-volume
        - mountPath: /var/alertmanager/data
          name: alertmanager-main-db
      - args:
        - -webhook-url=http://localhost:9093/-/reload
        - -volume-dir=/etc/alertmanager/config
        image: quay.io/coreos/configmap-reload:latest
        imagePullPolicy: Always
        name: config-reloader
        resources:
          limits:
            cpu: 5m
            memory: 10Mi
        terminationMessagePath: /dev/termination-log
        volumeMounts:
        - mountPath: /etc/alertmanager/config
          name: config-volume
          readOnly: true
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      securityContext: {}
      terminationGracePeriodSeconds: 0
      volumes:
      - configMap:
          defaultMode: 420
          name: alertmanager-main
        name: config-volume
      - emptyDir: {}
        name: alertmanager-main-db
status:
  replicas: 0

@ryanwalls - yes - that version is pre secret replacing configmaps for config...

@dmcnaught Thanks.  Yeah I see that now.  I think the earlier commit you mentioned https://github.com/coreos/prometheus-operator/commit/13a0dfbfc44f6c025363e612b469786a3cca091b was creating a Secret but the Alertmanager was looking for a ConfigMap.  I think I have enough to get it working now.
If I get a working example with 1.5, then I'd be happy to create a PR with a README that has instructions.  Ideally, it would be good to have a tag and/or release that is specific to 1.5.

@dmcnaught @brancz Thanks for all the help, but keep hitting more issues...  Rules for alerting on k8s don't seem to be mounted in the prometheus pod.  So there are no alerts in prometheus.
Also, the targets to the kubelets and kubernetes endpoints are down.

I think I'm going to wave the white flag until we upgrade to 1.6 or a documented release for 1.5 is available.   I'd definitely recommend upgrading to 1.6 as soon as you can. Other than that there are only minor differences that would be necessary for current master to work on a 1.5 cluster. My guess would be that you have to change all kubectl apply calls in the hack/ scripts to kubectl create as there were issues in 1.5 that did not allow kubectl apply on TPRs. And in addition to that all RBAC resources would have to change the apiVersion from v1beta1 back to v1alpha1. These are changes that we could in theory solve with a helm chart, but we are currently relying for those on external contributions.

@brancz If I was to contribute a helm chart, what's your vision for where it would live and what it would encompass?  I see there's already a chart for prometheus-operator here: https://github.com/coreos/prometheus-operator/tree/master/helm.
So would you want to add to that existing chart to include more of what kube-prometheus does?  Or would you envision a separate chart that is put in https://github.com/coreos/prometheus-operator/tree/master/contrib/kube-prometheus?

Sorry @ryanwalls, I lost sight of this. @mgoodness is maintaining the helm charts, I'm sure he can point you at something that needs some work. All contributions welcome! :slightly_smiling_face:

I'm running into this issue with kubernetes v1.6.6 and prometheus-operator 0.11.1.  Is there a workaround or a fix?
Error:
unable to decode "prometheus-k8s-service-monitor-alertmanager.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"

Hi,
I'm running into this issue with Kubernetes v1.6.6 and prometheus-operator 0.11.1.  Is there a fix or a workaround for this error below?
Error:
unable to decode "prometheus-k8s-service-monitor-alertmanager.yaml": no kind "ServiceMonitor" is registered for version "monitoring.coreos.com/v1alpha1"
Env Info:
Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.6", GitCommit:"7fa1c1756d8bc963f1a389f4a6937dc71f08ada2", GitTreeState:"clean", BuildDate:"2017-06-16T18:34:20Z", GoVersion:"go1.7.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.6", GitCommit:"7fa1c1756d8bc963f1a389f4a6937dc71f08ada2", GitTreeState:"clean", BuildDate:"2017-06-16T18:21:54Z", GoVersion:"go1.7.6", Compiler:"gc", Platform:"linux/amd64"}
Thanks.

@bamb00 that looks like a different problem. Is that error from when you try to apply the manifest with kubectl or an excerpt from the Prometheus Operator logs?
Could you give us the output of
kubectl get thirdpartyresources

As it seems they have not been registered, which could be due to an RBAC issue.

Here is the output from TRPs,
kubectl get thirdpartyresources
NAME                                    DESCRIPTION                           VERSION(S)
alertmanager.monitoring.coreos.com      Managed Alertmanager cluster          v1alpha1
prometheus.monitoring.coreos.com        Managed Prometheus server             v1alpha1
service-monitor.monitoring.coreos.com   Prometheus monitoring for a service   v1alpha1

This part of the script did not get execute manually after applying prometheus-operator,
Wait for TPRs to be ready.
printf "Waiting for Operator to register third party objects..."
until kctl get servicemonitor > /dev/null 2>&1; do sleep 1; printf "."; done
until kctl get prometheus > /dev/null 2>&1; do sleep 1; printf "."; done
until kctl get alertmanager > /dev/null 2>&1; do sleep 1; printf "."; done
echo "done!"

@bamb00 You might try deleting your TPRs and rerunning the script from scratch.

I'm running each yaml individually.  I need a workaround to get pass registering the TPRs.

I see the problem.
The objects did not get create for each of the TRPs.  How do I do that?

@bamb00 it seems you are trying to apply the manifests currently on master of this repository, which will not work with v0.11.1 of the Prometheus Operator, as we've promoted the objects from v1alpha1 to v1. You should check out this commit 2747557, which was the last commit, where the manifests were compatiable, with v0.11.1 of the Prometheus Operator. Note that the Prometheus Operator v0.12.0 requires a Kubernetes v1.7.x cluster, so you can only upgrade to v0.12.0 of the Prometheus Operator once you have upgraded your Kubernetes cluster to a v1.7.x or higher version. The Prometheus Operator will then migrate your objects automatically when you upgrade it from v0.11.1 to v0.12.0.

Closing this as outdated


feat(module-hooks): sort module hook data

What this PR does / why we need it
This PR adds sorting to module hooks by using hashstructure, falling back to fmt.Sprintf("%v"). This makes re-runs of stencil more reproducible, since module hook insertion order is not guaranteed because file execution order is not.

Jira ID
DT-3484

Notes for your reviewers

Tests passed, failed due to codecov upload. I will fix in a future PR.


Fix subtraction underflow when sorting string arrays with many nulls

Rationale for this change
String arrays, boolean arrays, arrays of lists, and more follow a different code path than primitive arrays so updating to match https://github.com/apache/arrow-rs/pull/236
Are there any user-facing changes?
No.

Done @medwards @jhorstmann


no tty present and no askpass program specified

Hey,
I'm trying to use the new Passenger restart method (I used to use touch.txt).
I am using Ubuntu, RVM, Nginx + Passenger module.
When I go with the 100% vanilla setup of capistrano/passenger, I get this:
SSHKit::Runner::ExecuteError: Exception while executing as deploy@xxxxxxxxxx: passenger-config exit status: 1
passenger-config stdout: Nothing written
passenger-config stderr: *** ERROR: You are not authorized to query the status for this Phusion Passenger instance. Please try again with 'rvmsudo'.

I set this option: set :passenger_restart_with_sudo, true
And then I get this:
SSHKit::Runner::ExecuteError: Exception while executing as deploy@xxxxxxxxxx: sudo exit status: 1
sudo stdout: Nothing written
sudo stderr: sudo: no tty present and no askpass program specified

Is there anything I can do without messing with the sudoers file? I have a lot of servers and don't really have the energy right now to change all that, if possible.
Capfile:
require 'capistrano/setup'
require 'capistrano/deploy'

require 'rvm/capistrano'

require 'capistrano/bundler'
require 'capistrano/rails/assets'
require 'capistrano/rails/migrations'
require 'capistrano/passenger'
require 'capistrano/sitemap_generator'

require 'whenever/capistrano'
require 'airbrake/capistrano3'

# Loads custom tasks from `lib/capistrano/tasks' if you have any defined.
Dir.glob('lib/capistrano/tasks/*.rake').each {|r| import r}

deploy.rb
lock '3.2.1'

set :application, 'XXXXXXXXX'
set :repo_url,    'git@github.com:XXXXXXXXXX/XXXXXXXXX.git'

set :rvm_type, :system
set :rvm_path, '/usr/local/lib/rvm'

set :passenger_restart_with_sudo, true

set :whenever_identifier, ->{ "#{fetch(:application)}_#{fetch(:stage)}" }
set :whenever_roles, [:app, :primary_production_worker]

set :linked_files, %w{config/database.local.yml config/secrets.local.yml}
set :linked_dirs, %w{log}

after 'deploy:published', 'foreman:export'
after 'deploy:published', 'foreman:restart'
after 'deploy:finished',  'airbrake:deploy'

What version of capistrano is running on your server?  If you can upgrade to 5.0.10 or higher, that would be the quickest fix

lol, it took me about 10 minutes to realize you meant passenger and not capistrano. I was like hmm.... lots of versions must have been released.
I see, my passenger is out of date. thanks!

My mistake.  Glad you figured out what I meant.


Upgrade aesh to 0.66.3 to fix WFCORE-1277

Upgrade aesh to 0.66.3
https://issues.jboss.org/browse/WFCORE-1277

Can you backport this to wildfly-core-eap? Until we release WildFly 10.0.0.Final we are not putting stuff into master without a wildfly-core-eap PR.
Thanks for your help on this!

sure, thought it needs productized version. I have opened https://github.com/jbossas/wildfly-core-eap/pull/136

Thanks. I don't know about the process for CPs, but for work on majors and minors we normally upgrade to the community version first and then the productization team updates to the productized version later. That avoids delays in getting things into CI testing, blocking other fixes, etc.

Yes, this makes sense to me.
In CP release,  almost every component upgrade has an upstream issue, which has been tested there first to reveal any possible regression. Thus, we upgrade directly with prod version.


Document aws.max_retries to fix throttling

Document that for the AWS CPI, the option aws.max_retries can be used to retry
in case of throttling.
This feature is added in the  https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/pull/23, to cover https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/issues/19. The PR in bosh-aws-cpi-release should be merged and released before merging this PR.

@cppforlife thx for merging, but shouldn't that wait until https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/pull/23 was merged?


💭 App upadates Rejecting| External Storage Permission restricated due to latest policy.

Question
Due to the latest google play policy the use of External Storage permission is restricted for financial apps like loan etc..
Because of which our app update is getting rejected.
Is there any workaround where we can use the clicked photo from vision camera on RN view without storing the temp file in external storage and by pass the storage permission.
What I tried
No response
VisionCamera Version
3.7.1
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

I don't think VisionCamera forces external storage.
I store my files in a File.createTempFile(..., context.cacheDir): https://github.com/mrousavy/react-native-vision-camera/blob/9a0620427b81e513665a2034db8742de64a115d1/package/android/src/main/java/com/mrousavy/camera/core/utils/FileUtils.kt#L12-L15
How is this not following the Google Play policy?

According to Privacy policy issue MANAGE_EXTERNAL_STORAGE it sounds fine as this is only related to this one specific permission - I couldn't find any info about the new policy you are mentioning - can you please link that here?

Hello @mrousavy
My bad our team is using the external storage permission to get the temp file created by vision camera which is not needed as you said earlier.


BP Update And Builder Image Changes

BP Update and builder image changes

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

Project coverage is 38.76%. Comparing base (35e315a) to head (08729ea).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #262   +/-   ##
=======================================
  Coverage   38.76%   38.76%
=======================================
  Files          31       31
  Lines        1999     1999
=======================================
  Hits          775      775
  Misses       1136     1136
  Partials       88       88

/lgtm


Decomission farallon & remove kops support

Fixes https://github.com/2i2c-org/infrastructure/issues/1454

I'll clean out the AWS account manually.


Announcement: repo included in CNCF Submission!

Kubecost has submitted this project to the Cloud Native Computing Foundation (CNCF) for onboarding under the new “OpenCost” effort. We are currently in the queue for review, and we’re excited to share updates on this mission as they develop.
From the start, our goal has been to enable teams using Kubernetes to operate with a single model for measuring and managing cost across all of their environments. We consistently hear from users and our partners that standardization across cloud platforms, Kubernetes distributions, and teams is strongly needed. To most effectively drive a standard, we feel it is important that OpenCost is not driven by any single entity—and that the project continues to evolve and thrive within the broader Kubernetes community. We believe the CNCF is the best organization to help with this goal.
The community of cloud industry leaders supporting OpenCost intend to develop industry documentation that standardize methodologies and best practices for tracking Kubernetes costs. Founding companies of the OpenCost community are all heavy adopters of Kubernetes and have experienced firsthand the gap around Kubernetes cost standards. If you are interested in joining the OpenCost community or interested in learning more, please join the CNCF Slack community — we’d love to speak with you!

Proud to report that the OpenCost project has been approved by the CNCF TOC and we are now going through onboarding!

Let's close this once https://github.com/cncf/cncf.io/issues/610 has been closed

Upstream updated.


Side car Container not working

Describe the bug
Side car container with azure login --identity not working
Steps To Reproduce
Kubectl  apply -f -
kind: ServiceAccount
apiVersion: v1
metadata:
  name: quick-start
  namespace: default
  labels:
    azure.workload.identity/use: 'true'
  annotations:
    azure.workload.identity/client-id: ""

---
apiVersion: v1
kind: Pod
metadata:
  name: quick-start1
  namespace: default
  labels:
    azure.workload.identity/use: "true"
  annotations:
    azure.workload.identity/inject-proxy-sidecar: "true"
spec:
  serviceAccountName: quick-start
  containers:
    - image: mcr.microsoft.com/azure-cli
      name: oidc
      command: ["/bin/sh"]
      args: ["-c", "while true; do echo hello; sleep 1000;done"]
  nodeSelector:
    kubernetes.io/os: linux

Expected behavior
Successful login
Logs
Log from Side car

Log from Container

Environment

Kubernetes version (use kubectl version): 1.22
Cloud provider or hardware configuration: AKS
OS (e.g: cat /etc/os-release):  cat /etc/os-release`
Kernel (e.g. uname -a): Linux quick-start1 5.4.0-1091-azure #96~18.04.1-Ubuntu SMP Tue Aug 30 19:15:32 UTC 2022 x86_64 Linux
Install tools:
Network plugin and version (if this is a network-related bug):
Others:

Additional context

@dbg-raghulkrishna

Did you try passing the clientID of the managed identity as part of the az login --identity command?
Have you set up the federated identity credential?
Exec into the sidecar proxy and check if there is network connectivity to https://login.microsoftonline.com/72f988bf-86f1-41af-91ab-2d7cd011db47/v2.0/.well-known/openid-configuration

can you give me the command to try that?
yes
az identity federated-credential create
--name "kubernetes-federated-credential"
--identity-name "test"
--resource-group "test"
--issuer ""
--subject "system:serviceaccount:default:quick-start"
cannot exec into sidecar proxy

@dbg-raghulkrishna With managed identity client id the command is az login --identity -u <client ID>.

az identity federated-credential create
--name "kubernetes-federated-credential"
--identity-name "test"
--resource-group "test"
--issuer ""
--subject "system:serviceaccount:default:quick-start"

The issuer here is empty, but I'm assuming you just redacted it while posting here? The issuer must be AKS OIDC Issuer URL if you're on AKS.

cannot exec into sidecar proxy

You need to pass the container name as part of kubectl exec -c azwi-proxy

@aramase

i tried az login --identity -u . still the same error
yes i redacted before posting here

@dbg-raghulkrishna Is this issue still relevant? Have you tried the latest release v0.15.0?

it is fixed now @aramase


Ethck's Downtime Tracking module compatibility

https://foundryvtt.com/packages/downtime-ethck
https://github.com/Ethck/Ethck-s-Downtime-Tracking

Related to #37

Module integration discussion here: https://github.com/Ethck/Ethck-s-Downtime-Tracking/issues/90

Compatibility attained. Closing as completed.


Indicate that it is the node public key displayed in the log

PR description
feat: add public in key pair loading for clarity purpose
Fixed Issue(s)

#548

Signed-off-by: Alexandre PARIS-VERGNE<EMAIL_ADDRESS>

I think @MadelineMurray or @macfarla may have logged the original issue (a looooong time ago).


Training iteration stops silently if not Debugging

Have I written custom code: Yes
OS Platform and Distribution: Windows server 2012 R2 Standard
TensorFlow installed from: Pip repositories
TensorFlow version: 1.7.0
Bazel version: NA
CUDA/cuDNN version: NA
GPU model and memory: NA
Exact command to reproduce NA
I'm attempting to train a tensorflow.contrib.keras.model.Sequencial on an image dataset. I have noticed however that when running the script normally, everything appears to work fine until the process does not exit. When I ran the scrip in debug mode, I received the message "StopIteration: Could not import PIL.Image. The use of 'array_to_img' requires PIL". The error is pretty self-explanatory, however is it intended behavior to fail silently like this?  I did not realize at first that the execution halted without exiting, until sometime passed.
This is my model initialization and training block:
    model = tf.contrib.keras.models.Sequential()
    model.add(tf.contrib.keras.layers.Conv2D(32, kernel_size=(3, 3),
                     activation='relu',
                     input_shape=data_train.image_shape))
    model.add(tf.contrib.keras.layers.Conv2D(64, (3, 3), activation='relu'))
    model.add(tf.contrib.keras.layers.MaxPooling2D(pool_size=(2, 2)))
    model.add(tf.contrib.keras.layers.Dropout(0.25))
    model.add(tf.contrib.keras.layers.Flatten())
    model.add(tf.contrib.keras.layers.Dense(128, activation='relu'))
    model.add(tf.contrib.keras.layers.Dropout(0.5))
    model.add(tf.contrib.keras.layers.Dense(4, activation='softmax'))

    model.compile(loss=tf.contrib.keras.losses.categorical_crossentropy,
                  optimizer=tf.contrib.keras.optimizers.Adadelta(),
                  metrics=['accuracy'])

    model.fit_generator(data_train,
                        steps_per_epoch=100,
                        epochs=5,
                        validation_data=data_test,
                        validation_steps= data_test.batch_size)

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

@fchollet Any comments on the intended behavior here?

Nagging Assignees @karmel, @fchollet: It has been 18 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @karmel, @fchollet: It has been 33 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @karmel, @fchollet: It has been 48 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @karmel, @fchollet: It has been 63 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @karmel, @fchollet: It has been 78 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@omalleyt12 -- is this something you can look into?

@karmel yep I'll take a look

@iustin94 is data_train a subclass of keras.Sequence? I'm guessing this has to do with multiprocessing

I can't answer this question as I don't have access to that repository currently. You might be right, I recall we were attempting to use multiprocessing however we couldn't see it in the process monitor so we suspected we were doing something wrong.

I tried to recreate the lack of error situation with the code below but this seems to be (correctly) throwing an error for me:
import numpy as np
import tensorflow as tf

keras = tf.keras

class MySequence(keras.utils.Sequence):
  def __getitem__(self, x):
    if x == 1:
      raise ValueError('Should raise an error.')
    return np.ones([10,100,100,3]), np.ones([10,4])

  def __len__(self):
    return 3

model = keras.models.Sequential()
model.add(keras.layers.Conv2D(32, kernel_size=(3, 3),
                              activation='relu',
                              input_shape=(100,100,3)))
model.add(keras.layers.Conv2D(64, (3, 3), activation='relu'))
model.add(keras.layers.MaxPooling2D(pool_size=(2, 2)))
model.add(keras.layers.Dropout(0.25))
model.add(keras.layers.Flatten())
model.add(keras.layers.Dense(128, activation='relu'))
model.add(keras.layers.Dropout(0.5))
model.add(keras.layers.Dense(4, activation='softmax'))

model.compile(loss=keras.losses.categorical_crossentropy,
              optimizer=keras.optimizers.Adadelta(),
              metrics=['accuracy'])

model.fit_generator(MySequence(),
                    steps_per_epoch=3,
                    epochs=1)

On the multiprocessing bit, I looked into it more and the code you had will use multithreading and not multiprocessing to grab the input data, which may explain why you weren't seeing anything in the process monitor. If you want Keras to use multiprocessing with a generator/Sequence, you can do:
model.fit_generator(MySequence(),
                    steps_per_epoch=3,
                    epochs=1,
                    use_multiprocessing=True)

Let me know if there's any more context you can provide, otherwise I'm not sure if I'll be able to recreate this issue

Closing for now, this issue may have been fixed since this bug was reported


[Question] Should NeedlessOperation be flagged for something that explicitly returns Unit

It's worth having something that only returns Unit explicitly as function, or should we support this as a valid operation? In https://github.com/microsoft/qsharp/pull/1687, I coverted ExplicitReturn to a function because lint rule was complaining. Prolly we can live with it, but wanted to get other folks opinion on this.
/// # Sample
/// Unit
///
/// # Description
/// The `Unit` type is the singleton type whose only value is ().
/// Functions implicitly return `Unit` if no explicit or implicit
/// return is specified.
namespace MyQuantumApp {

    @EntryPoint()
    function ExplicitReturn() : Unit {
        return ();
    }
    operation NoReturn() : Unit {
        // No return, thus implicitly returning `Unit`.
    }
}

We think that this is a non-issue, as explicitly returning Unit from an operation is uncommon. We disabled that lint on empty operation bodies primarily for the UX of katas and the samples, as well as when you're in the process of declaring an operation. Once there's a body to the operation, it makes sense for the lint to trigger.


events on  RegistrationController not working

Hi,
I tried events with this config code,but it does not work.Is it important to declare the type of events in code? or any other errors ? Thanks a lot!
Code below (in a valid config file):
'modules' => [         'user' => [             'class' => 'dektrium\user\Module',             'controllerMap' => [                 'resgitration' => [                      'class' => '\dektrium\user\controllers\RegistrationController',                     'on '. \dektrium\user\controllers\RegistrationController::EVENT_BEFORE_CONFIRM =>                          function($event){                             \Yii::$app->session->set('test before confirm', $event->getId());                         }                  ],            ]        ], ]

also I tried $event in type yii\base\Event ,yii\web\UserEvent, and dektrium\user\events\UserEvent ,but got the same silent blank. as the source of dektrium\user\controllers\RegistrationController pointed:
 /**      * Event is triggered before confirming user.      * Triggered with \dektrium\user\events\UserEvent.      */     const EVENT_BEFORE_CONFIRM = 'beforeConfirm';
is there a proper way to invoke the event ?

It is quite confusing ... I use another same template (Yii2-app-advanced) without installing yii2-user,and got events work. What wonders me most, is that only one controller class of this module has some problem, and I have to use this controller in production environments. :(

OOOOhhhh.....I get a solution. I wrote registration into resgitration  by mistake. Hope everyone write code that be really careful.
So the correct code should be:
'modules' => [ 'user' => [ 'class' => 'dektrium\user\Module', 'controllerMap' => [ 'registration' => [ 'class' => '\dektrium\user\controllers\RegistrationController', 'on '. \dektrium\user\controllers\RegistrationController::EVENT_BEFORE_CONFIRM => function($event){ \Yii::$app->session->set('test before confirm', $event->getId()); } ], ], ], ]


update opentelemetry dependencies to the latest version

I also had to use a str instead of SpanKind because its Display impl was removed. I checked on tracing-opentelemetry and they did the same.
I suppose we need to go to 0.5 to release it?

Thanks for the PR, yes after merge + some update (ci), I'll release as 0.5

After running the project with the same clippy arguments present on .mega-linter.yml, I found some problems regarding opentelemetry-jaeger. It looks like PipelineBuilder was split into CollectorPipeline and AgentPipeline here: https://github.com/open-telemetry/opentelemetry-rust/pull/748.
Are you able to update the PipelineBuilder? I'm not familiar with the opentelemetry-jaeger crate. If you're busy I could take a look later. :smile:

Thanks for the investigation, I'll take a look during the week


Contentful Appearance and Help Settings

Research the settings in Contentful as Titled.

John to make sure help text looks good. Clean up task.

Can't complete until data models settle in and client gives ok, probably post Sprint 4.


add remaining IRT case studies

Added the remaining IRT case studies

I changed the title to be more informative.  Thanks!  I'll write a blog post to announce them ASAP.

Thanks, Bob!
On Thu, Mar 17, 2016 at 4:41 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

I changed the title to be more informative. Thanks! I'll write a blog post
to announce them ASAP.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan-dev.github.io/pull/16#issuecomment-198133138


feature request: only show files when using -f

Only show files names when using -f (possibly by using another cmd line option).

Added in version 2.3.0.
https://github.com/jftuga/chars/commit/808169471d175d35e46abe90e78fc14c26653d68

Looks good! It might be interesting to show the full table though,
since if you pass multiple values to -f then without the table you
won't know which of the -f options the file has in it.
--
bye,
pabs
https://bonedaddy.net/pabs3/


fix: fix path to emulator

Summary
motivation: the path to emulator used in the orb is not always the right one, this should fix it
Test Plan
this change is used in https://github.com/react-native-community/datetimepicker/pull/155
after some fighting, you see tests are passing
What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅❌

Android
✅❌

Checklist

[x] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

related: https://github.com/react-native-community/react-native-circleci-orb/issues/53


cmocka_cloned configure error on Mac for latest master

macOS 11.6.1
AppleClang <IP_ADDRESS>00029
cmake 3.21.2
[ 21%] Performing build step for 'cmocka_cloned'
CMake Error: Invalid value used with --config
Usage: cmake --build <dir>             [options] [-- [native-options]]
       cmake --build --preset <preset> [options] [-- [native-options]]
Options:
  <dir>          = Project binary directory to be built.
  --preset <preset>, --preset=<preset>
                 = Specify a build preset.
  --list-presets
                 = List available build presets.
  --parallel [<jobs>], -j [<jobs>]
                 = Build in parallel using the given number of jobs.
                   If <jobs> is omitted the native build tool's
                   default number is used.
                   The CMAKE_BUILD_PARALLEL_LEVEL environment variable
                   specifies a default parallel level when this option
                   is not given.
  --target <tgt>..., -t <tgt>...
                 = Build <tgt> instead of default targets.
  --config <cfg> = For multi-configuration tools, choose <cfg>.
  --clean-first  = Build target 'clean' first, then build.
                   (To clean only, use --target 'clean'.)
  --verbose, -v  = Enable verbose output - if supported - including
                   the build commands to be executed.
  --             = Pass remaining options to the native tool.
make[2]: *** [tests/cmocka_cloned-prefix/src/cmocka_cloned-stamp/cmocka_cloned-build] Error 1
make[1]: *** [tests/CMakeFiles/cmocka_cloned.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

The master branch does not make use of cmocka.  I assume you meant the default develop branch.
I can reproduce this issue with CMake 3.21.2 but not with an earlier version (3.13.4).  It appears that CMAKE_BUILD_TYPE is not propagated when the external cmocka project is being built (see this line).  This issue is further discussed here.  Let us look into how to best resolve the issue.

The master branch does not make use of cmocka. I assume you meant the default develop branch.
I can reproduce this issue with CMake 3.21.2 but not with an earlier version (3.13.4). It appears that CMAKE_BUILD_TYPE is not propagated when the external cmocka project is being built (see this line). This issue is further discussed here. Let us look into how to best resolve the issue.

Yeah, the develop branch.

We need to spend some time on this to get it right, as it seems to be a nontrivial issue to solve in a portable manner.  In the meantime, some possible workarounds include:

Use an older version of CMake.
Modify src/CMakeLists.txt to use --config Release or exclude the --config setting on macOS.
Disable the tests via -DBUILD_TESTING=OFF.

Modify src/CMakeLists.txt to use --config Release or exclude the --config setting on macOS.

Sorry, that should read tests/CMakeLists.txt.

@lindstro Thanks. I can live with -DBUILD_TESTING=OFF for now.

Thanks,
This worked for me on mac M1 with macOS 12.0.1
cd zfp
mkdir build
cd build
cmake -DBUILD_TESTING=OFF ..
cmake --build . --config Release
ctest not gonna do anything
Well it built fine! I am hoping to put it to use soon. I needed 4D arrays.

This issue has hopefully been resolved on develop.  Please let us know if the fix does not work for you.


Scenario outline with examples in title

Regarding this: https://github.com/cucumber/cucumber-js/issues/267
Greetings!
Do we have anything similar in ruby?

marcfallows commented on Oct 23, 2014
Hello,
I would expect to be able to have my scenario outline title have more information by using the examples >within the title itself:
Scenario Outline: A <some> step is <result>
   When a <some> step
   Then I get <result>
Examples:
   | some    | result  |
   | passing | passed  |
   | failing | skipped |

Then my scenario titles end up very useful:
Scenario: A passing step is passed Scenario: A failing step is skipped
This doesn't seem to be working for me. Is this a supported feature? If not, is it something that I could >assist in having included? Or, is it not expected to work like this?

If not, it would be very useful to have something like this. It would increase the readability of reports with alot of example data.

I really like this idea too, and I also think it's good to try to have some consistency to where features available in one implementation of Cucumber should also be available in others. Would you be able to put together a PR for this, @rKrums? I'd be happy to help out or pair on this.

I'm sure we could do this in the compiler in cucumber-core, but I wonder if it will need to go into gherkin's pickles compiler in the long run.
cc @brasmusson @aslakhellesoy

The gherkin compilers do implement this already, see https://github.com/cucumber/gherkin/blob/master/testdata/good/example_tokens_everywhere.feature and https://github.com/cucumber/gherkin/blob/master/testdata/good/example_tokens_everywhere.feature.pickles.json

@danascheider I'm on a really tight schedule right now, and would not be able to participate. Thank you for offer. I understand, that it's possible to do that already, but have not checked.

Any plans of having this feature in the Ruby version?

@sharmarahul This is already possible.

@danascheider No, this is not possible in Cucumber-Ruby, as it only uses the Gherkin parser and implements its own (AST) compiler (instead of using Gherkins compiler - which did not exist when the compiler in Cucumber-Ruby-Core was implemented).

Ahhh I see. I was actually just wondering about that as I was poking around in that part of the codebase yesterday. That being the case, @sharmarahul, is this something you'd be able to help out with? If not I'll get to it eventually but I can't promise when.

Hi, do we have plan to include this feature in 3.0?
Thanks

+1, we would really like to see this

@xuanzhaopeng This is not slated for the 3.0 release. We may add it at a later time, but it will happen a lot sooner if you, @Githraine, or @sharmarahul are able to help us out with it. If you are, please let us know and we can point you in the right direction as far as implementation.

What would it take to add this in?

@Githraine somebody needs to focus in on it and implement it! It looks like it's not a priority for anyone on the contributor team, but we'd be happy to support and coach someone who wanted to do the work. Are you interested?

I have not done that before, but I think it would be useful.  Sure, I'll give it a shot.

@apetkova It's definitely still a feature I'd be interested in adding.

Actually, this feature has already been added to master, but is simply not released yet. It looks it was fixed with https://github.com/cucumber/cucumber-ruby/commit/7f43c9533691522899b87a22a700b22ae7ce20bb ; Since I don't see any tests for it, maybe it was a nice "side effect", but definitely works, so I guess this issue can be resolved?

@apetkova Sounds good. Feel free to reopen/ask us to reopen if it turns out not to be resolved or there's a problem.

We finally got upgraded to the latest Cuke.
This is NOT resolved.  Given this scenario outline:
@mm_calc_tvr_001 @wip
Scenario Outline: Traditional vs Roth Base tests -
Given I output ""
And I am on the "Traditional vs Roth Calculator" calculator page
When I populate the calculator from "" and "calculate"
Then the calculator results should match the "base" section of ""
Examples:
  | title                                                  | inputs                                         |
  | Single Higher Roth Now Matches                         | single_higher_now_roth_matches                 |
  | Married Higher Roth Now Matches                        | married_higher_now_roth_matches                |
  | Married Lower Roth Now Matches                         | married_lower_now_roth_matches                 |
  | Married Lower Roth Later Matches                       | married_lower_later_roth_matches               |
  | Married Lower 401k Later Matches                       | married_lower_later_401k_matches               |
  | Single Lower Roth Later Matches                        | single_lower_later_roth_matches                |
  | Married Lower 401k Later no Match                      | married_lower_later_401k_does_not_match        |
  | Single Lower Later 401k no match                       | single_lower_later_401k_does_not_match         |

the results are showing:
Feature: Calculators - Traditional Vs Roth, Scenario: Traditional vs Roth Base tests - , Examples (#1)

@Githraine somebody needs to focus in on it and implement it! It looks like it's not a priority for anyone on the contributor team, but we'd be happy to support and coach someone who wanted to do the work. Are you interested?

Hey seems to be this issue is not at resolved. I was able to implement the solution by overriding the builder class in my local repo. Can you please guide me how to make this change implemented here, so that it helps others.

So you need to clone the repo (after forking it to your own repo). And then push a branch up and make a Pull Request.
There are a few of us actively supporting the cucumber-ruby project, so it'll get reviewed reasonably quickly.


Dependency of gojek is pointing to gojektech

The code has reference to gojektech/heimdall which now should be moved to gojek/heimdall.
https://github.com/gojek/heimdall/search?q=gojektech

Resolved by #110


[Feature] Add CLI support for Kuberay

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Command Line Interface (CLI) support is desired to be added to Kuberay as it provides users with ability to interact with ray clusters in Kubernetes through command line.
Use case

Users can manage kuberay resources (cluster, compute templates etc) through command line.

Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Thanks for @wolfsniper2388's help on the KubeRay CLI! Since the initial version is working, let's close this PR.


Brightness problems with transparent images

Problem
I intalled this via dotnet add package WPF.ImageEffects --version 1.5.0 and I found that when I apply the Brightness property to a positive number, the transparent parts of the image get tinted white.
However, this does not happen when I apply the property to a negative value or zero.
Notes
I have a single instance of the BrightnessContrastEffect class, where that instance is applied to a Grid full of Images.
The instance is created programmatically:
public BrightnessContrastEffect BrightnessContrastEffect { get; } = new BrightnessContrastEffect();
...
BrightnessContrastEffect.Brightness = 0.5;

And the instance is attached this way:
SpriteGrid.Effect = BrightnessContrastEffect;
I am not sure if the problem is related to the fact that the shader also picks up other elements that are not part of the grid itself, but are rendered behind it, thus applying the shader to everything within the grid's boundaries.
Also, having to apply the effects to the images individually isnt a solution for me because it would cause the effect to "accumulate" in semi-transparent images

Thanks for your help.

The shader do not touch the alpha value of the pixels.
Maybe the control has a background color which is modified, too?

Nop, i created a brand new WPF.NET C# project for testing, just added an image, added the effect to it and the issue persists:

As shown in the image, i selected a random png asset i had, placed it in a gray background colored window (it can be any color, it was just to differenciate between the place where the buggy white tint takes place and the rest of the window), attached the effect, and that's the result

I'm sure your image doesn't contains full transparent pixels.
If your image contains partial transparent pixels, it will combine the color from the background. That's correct.
Here is an example. The blue rectangle is a full transparent area. (This grid has the blue background color.)
brightness = -1

brightness = 1

Please use the example demo from this respository and test again.


Support Centrifuge

UI

[ ] Icons
[ ] SNS preview

files
icons-centrifuge.zip
SNS-centrifuge.zip
preview

We will later integrate centrifuge para-chain directly.


APIv2: drop ExtensionRangeArray method?

Consider not generating the ExtensionRangeArray method on generated message structs. It's no longer used by the implementation, and isn't a useful user-facing API.

Strictly speaking, our compatibility agreement only reserves the right to add/remove methods with an XXX prefix. Even though this has little useful value to the user, isn't removal of it going against that document?

We're already marking it as deprecated in the v2 generator; remove six months after APIv2 is released per breaking change policy?

We're already marking it as deprecated in the v2 generator; remove six months after APIv2 is released per breaking change policy?

SGTM. The blocks-v2 label was confusing me since it seemed to suggest that we would remove this before we release v2.

More of a "decide if we're going to remove this before releasing". I'm going to leave the label on to make sure we include the appropriate deprecation notice in the v2 release.

Related to this. I'd like to see the following removed as well:

Enum.UnmarshalJSON for proto2.
Enum.EnumDescriptor; replaced by native protobuf reflection
Message.Descriptor; replaced by native protobuf reflection
protobuf struct field tags on messages
json struct field tags on messages

The justification for 1 and 5 is that they give the illusion that encoding/json works on generated types, when they actually do not. The jsonpb package even states this. Their existence cause greater harm when people mistakenly rely on them.
The justification for 2 and 3 is that these methods are superseded by protobuf reflection which is the flagship feature of APIv2.
The justification for 4 is that this tag should never be relied upon by users. We can ensure that proto.GetProperties continues to work even if they're missing. However, users should never have used Go reflection to manually look at this tag.

:+1: all of these are good arguments. Especially the json struct tag.

In our offline discussion @neild and I decided that we will announce (at v2 release time) the deprecation of ExtensionRangeArray and the items listed https://github.com/golang/protobuf/issues/1007#issuecomment-573263370 with the warning that we probably will remove them in 6 months. Whether we actually carry through with it is to be TBD (pending user feedback).

Dropping of ExtensionRangeArray is already announced in the v1.20.0 release. Do we still want to keep this open?

I think until we actually drop it, the issue should stay open?

I analyzed all modules in the Go proxy and found no direct calls to the ExtensionRangeArray method. It's probably say to treat it as equivalent to the XXX fields and methods and to remove it. I don't think we can remove the other stuff mentioned in https://github.com/golang/protobuf/issues/1007#issuecomment-573263370, though.

http://golang.org/cl/305574 removes ExtensionRangeArray.
For the other stuff I proposed in https://github.com/golang/protobuf/issues/1007#issuecomment-573263370, I don't think we can safely change those given that there are too many static or dynamic references to them.


Please help me I have been trying for WEEKS

Im trying to make permissions for my minecraft survival server with friends and I JUST CANNOT find out how to make it so people joining can ONLY modifly the world (break, build craft) without having OP! I have the essentials 'tpa' working, but I cannot get this to work on ANY permissions? Please help. I do not know the command

Make sure it's not spawn protection (set it to 0 in the server.properties).
Also, if you use WorldGuard make sure that the flag "passthrough" is allowed for the region _global_ (default)

LuckPerms command is /lp.
Make a group first by /lp creategroup
Now if you want a prefix on it: /lp group  meta setprefix 1 "<PREFIX W/COLORS> (colour of name)"
Before we start permissions, can you list down all of the plugins of your server? (/pl in game to view)

The plugins I currently have installed are:
dynmap, LuckPerms, WorldEdit, OmNom, AdminFun, Timber, BackpacksRemastered, Vault, WorldGuard, GUIShop, AreaShop, Essentials, GCore, Towny, Jobs, MCJukebox, GriefPrevention, CustomCommands, TownyChat, DiscordSRV, TownyNameUpdater, TheNewEconomy

World Modification
WorldGuard may already have its settings set, however, GriefPrevention is another plugin that is probably blocking every single modification to the world. GriefPrevention is like what it says, meant to block griefers, but for running a Survival server with claim blocks, it's better that you get a Survival plugin or a plugin that features claim blocks and buying of claim blocks using in-game economy (Essentials). If you really want to use GriefPrevention for setting up your server, you can follow the page on GriefPrevention Setup and Configuration which will help you to create admin areas that griefers cannot break and other things.
WorldGuard, like what Andre601 said, make sure that the flag passthrough is allowed for the region global . You can do this by using this command: /rg flag global passthrough allow. You may also use this command if you want players to be able to use items such as doors, trapdoors, jukeboxes, note blocks: /rg flag global use allow. This should be it for WorldGuard. Make sure to copy the same exact region: its  _ _ global _ _ not _ global _.
Permissions
Setting up Permissions on LuckPerms is quite easy. I've been on a server with over 60 plugins, it's hard to configure, but there's a convenient way to do it with LuckPerms. Go into your game and type /lp editor. This will generate a link that will open up a Permission Web Editor. This will allow you to view your groups that you added and the permissions inside each group.
Now, you aren't familiar with the plugins' permissions. Of course I'm not either, but the internet has lists that you can follow for each plugin's command you are looking for. For example, you want to find the Essentials Fix command. You can go online and search "Essentials permission nodes" and there will be websites with the list. Use CTRL + F on Google Chrome to find the command and its node quicker.
Back to LuckPerms Editor, the Editor is there, once you find the permission node, go back to the editor, find the group you want to apply the permission to, and you will probably be seeing a ton of different things and a '+' sign.

Permission: Put your permission node here and remove any unnecessary spaces
Expiry: If you only want the permission to last for a period of time, you may fill this up, otherwise, leave it blank
Server: Leave this blank
World: If you only want it to apply to a single world, fill this up, leave blank for all the worlds.
Contexts: This is unneeded, leave it blank
True - When pressed, it will turn to false. This is the same as adding a positive permission or negating a permission from a group. If its true and you add it, the group/user will be able to access this command. If its false, the player will not be able to.
When you have done all the configuring of the permission you want, press the + sign and it will be added to the list below the blank spaces.
When you are done adding permissions from the editor, press the floppy disk "SAVE" icon and it will load up a command. Copy that command by pressing on it or using CTRL + C, and paste the command in Minecraft using CTRL + V. And run the command, you will be spammed with a few things, but those are the changes that will be applied.
Reminder: Don't use the same editor everytime you are editing permissions, or else the permissions will mess up and may duplicate in some cases.
The more easier way but makes life tougher is using the ingame command to add permissions:
/lp group  permission set (permission node) (true/false)
The same applies for players, just change "group" to "user".
When you are ready, go ahead and give your players their ranks. Use
/lp user  parent set
and there you go.
This may be long, but its what's essential for LuckPerms and your problem, my explanations also
Good Luck handling your server!

Thanks! ill try right now

Did it work? You may mark this issue as closed when done.