TypeError: Cannot read property 'rtp' of undefined on setRemoteDescription

I'm currently trying to make a weird media server thing as a little experiment, though I keep getting the following error on setRemoteDescription:
    at RTCPeerConnection.remoteRtp (/home/MYUSERNAME/code/JS/project/chat/chat/node_modules/werift/lib/webrtc/src/peerConnection.js:382:26)
    at /home/MYUSERNAME/code/JS/project/chat/chat/node_modules/werift/lib/webrtc/src/peerConnection.js:533:37
    at Array.forEach (<anonymous>)
    at RTCPeerConnection.setRemoteDescription (/home/MYUSERNAME/code/JS/project/chat/chat/node_modules/werift/lib/webrtc/src/peerConnection.js:530:27)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async GatewayRTCConnection.answer (/home/MYUSERNAME/code/JS/project/chat/chat/api/v2/gateway/index.js:24:9)
    at async WebSocket.<anonymous> (/home/MYUSERNAME/code/JS/project/chat/chat/api/v2/gateway/index.js:169:37)

Here is the serverside handler for getting an offer from the client.
let isNewConnection = true;
if (ws.rtc) {
    isNewConnection = false;
} else {
    ws.rtc = new GatewayRTCConnection(ws, this);
}

if (isNewConnection) {
    ws.rtc.connection.onTransceiver.subscribe(async (transceiver) => {
        const [track] = await transceiver.onTrack.asPromise();
        this.inChannel(channel._id, (otherWs) => {
            if (!this.clientRTCReady(otherWs)) return;
            if (otherWs.session.sessionId === ws.session.sessionId) return; // Don't perform the actions below on ourselves

            otherWs.rtc.addTrack(track);
            otherWs.rtc.renegotiate();
        });
    });
}

ws.send(this.packet("EVENT_VOICE_CONNECTION_ANSWER", {
    answer: await ws.rtc.answer(offer)
}));

if (isNewConnection) {
    this.inChannel(channel._id, (otherWs) => {
        if (!this.clientRTCReady(otherWs)) return;
        if (otherWs.session.sessionId === ws.session.sessionId) return; // Don't perform the actions below on ourselves

        const otherReceivers = otherWs.rtc.connection.getReceivers();
        for (let i = 0; i < otherReceivers.length; i++) {
            const otherRecevier = otherReceivers[i];
            ws.rtc.addTrack(otherRecevier.tracks[0]);
        }
    });
    ws.rtc.renegotiate();
}

This is the small GatewayRTCConnection wrapper I made to simplify the code a little:
class GatewayRTCConnection {
    constructor(ws, server) {
        this.ws = ws;
        this.server = server;

        this.connection = new werift.RTCPeerConnection();
        this.connection.onnegotiationneeded.subscribe(() => this.renegotiate);
    }

    async answer(offer) {
        await this.connection.setRemoteDescription(offer);
        await this.connection.setLocalDescription(await this.connection.createAnswer());
        return this.connection.localDescription;
    }

    async addTrack(track) {
        if (!track) return;
        this.connection.addTrack(track);
    }

    async renegotiate() {
        this.ws.send(this.server.packet("EVENT_RENEGOTIATE_REQUIRED", {}));
    }
}

I inspected it a little and found that one of the transceivers just does not have an mLineIndex, which is used in remoteRtp. I assume it's because I'm adding tracks myself to the connection, but I'm not sure what to do.
Note that it works fine with 2 users, as soon as the new user tries to connect, it generates this error.

Probably due to the inconsistency between the SDP of the remote Offerer and the Transceiver inside the PeerConnection. Please let me know the information of the remote SDP and Transceivers when the error occurs.

Thank you for your response.
The full, latest source for the server can be found here: https://git.hippoz.xyz/hippoz/brainlet
while the full source of the client can be found here: https://git.hippoz.xyz/hippoz/brainlet-react
Let me know if you would like me to mirror this to github though.
Also, I'm not quite sure what you mean by information of sdp and transceiver.

The cause seems to be an inconsistency between the number of Transceivers in the PeerConnection on the browser side and the number of Transceivers in the PeerConnection on the server side.
In other words, it is caused by the code on the client application side of the browser.
The reason for this problem is that on the server side, addTrack is performed multiple times (as many times as the number of users) to a single PeerConnection, which means that there are as many Transceivers as there are users.
Since brainlet-react is on the Offer side, it is necessary to use addTransceiver to match the number of Transceivers with the server side. (Currently, there is only one Transceiver since client is only doing addTrack once).


Migrate Worldwide organisations to a finder

Trello: https://trello.com/c/zC2hVLAe/323-migrate-world-organisations-to-a-finder

[ ] run rake task finders:publish

Before

After

Could you explain a little more in the commit message?

Thanks @tijmenb, I hope I have addressed your comments.


Doesn't work

|master ✓| → npm start

<EMAIL_ADDRESS>start xxxxx
node ./src/index.js

xxxx/api/src/config/mongoose.js:34
throw new Error(Unable to connect to database: ${mongoUri})
^
Error: Unable to connect to database: localhost
at NativeConnection. (xxx/api/src/config/mongoose.js:34:9)
at NativeConnection.emit (events.js:314:20)
at xxxx/api/node_modules/mongoose/lib/connection.js:809:37
at processTicksAndRejections (internal/process/task_queues.js:75:11)

following the instructions, you should have copied the .env file which will contain the settings for your database. You have to replace the string with your connection string in order to get mongoose to connect.


Set up unit tests to run locally

It would be handy for development if tests could be run locally using Jest or something like that (at least a subset of tests if not all). Right now instead you have to upload and trigger tests through Apps Script which isn't a nice tight feedback loop.

Fixed in #25

Awesome! Hooray for developer workflow! 🎉


Add new test cases

Codecov Report

Merging #270 (d6ad882) into master (d263d5c) will increase coverage by 1.97%.
The diff coverage is 23.07%.

@@             Coverage Diff              @@
##             master     #270      +/-   ##
============================================
+ Coverage     61.27%   63.25%   +1.97%
- Complexity      977     1003      +26
============================================
  Files           148      148
  Lines          3525     3535      +10
  Branches        423      432       +9
============================================
+ Hits           2160     2236      +76
+ Misses         1171     1092      -79
- Partials        194      207      +13

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/seedu/address/model/ModelManager.java
64.82% <10.00%> (-2.82%)
44.00 <0.00> (+1.00)
:arrow_down:

...address/logic/commands/events/AddEventCommand.java
73.97% <50.00%> (+21.19%)
11.00 <0.00> (+1.00)

...ess/logic/parser/events/AddEventCommandParser.java
84.21% <100.00%> (+36.84%)
6.00 <0.00> (+3.00)

...c/main/java/seedu/address/model/RemindersImpl.java
29.62% <0.00%> (ø)
9.00% <0.00%> (+1.00%)

...du/address/model/reminder/UniqueRemindersList.java
21.73% <0.00%> (ø)
6.00% <0.00%> (+1.00%)

...du/address/model/event/association/FauxPerson.java
26.08% <0.00%> (+4.34%)
3.00% <0.00%> (+2.00%)

...in/java/seedu/address/logic/parser/ParserUtil.java
97.40% <0.00%> (+5.19%)
30.00% <0.00%> (+3.00%)

...in/java/seedu/address/model/event/Description.java
80.00% <0.00%> (+10.00%)
7.00% <0.00%> (+1.00%)

src/main/java/seedu/address/model/event/Event.java
52.94% <0.00%> (+13.72%)
12.00% <0.00%> (+2.00%)

... and 6 more

Continue to review full report at Codecov.

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

Draft PR closed. For the actual PR please check out #277.


HDDS-9975. Add static import for assertions and mocks in hdds-server-framework

What changes were proposed in this pull request?
The goal of this task is to add import static for all assertions and interactions with mocks. Replacing:
Assertions.assert<X>(...) -> assert<X>(...)
Assertions.fail(...) -> fail(...)

and
Mockito.mock(...) -> mock(...)
Mockito.verify(...) -> verify(...)
Mockito.when(...) -> when(...)

and
ArgumentMatchers.any() -> any()
ArgumentMatchers.eq() -> eq()
ArgumentMatchers.matches() -> matches()
// also:
// Matchers.<...>()

makes the code more readable (shorter lines, less wrapping).
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9975
CI
https://github.com/wzhallright/ozone/actions/runs/7286857379

Can you please also include: hadoop-hdds/framework/src/test/java/org/apache/hadoop/hdds/utils/db/TestTypedRDBTableStore.java?

Thanks for you detailed review @adoroszlai .I missed it, it was my fault, and include it now. Thanks again.

Just two more items here:
https://github.com/apache/ozone/blob/d044aaa5426c4eab42ea5d1dbe049945b4c8707e/hadoop-hdds/framework/src/test/java/org/apache/hadoop/hdds/utils/db/TestTypedRDBTableStore.java#L258-L259

Just two more items here:
https://github.com/apache/ozone/blob/d044aaa5426c4eab42ea5d1dbe049945b4c8707e/hadoop-hdds/framework/src/test/java/org/apache/hadoop/hdds/utils/db/TestTypedRDBTableStore.java#L258-L259

@adoroszlai fixed comments,thanks for review.


Named categorical

It would be nice to have a
categorical distribution whose support is an arbitrary set of values, rather than integers 1 through N.  As @sharlaon points out, this distribution may (we think?) allow us to store deterministic values in the trace without any extra design overhead, by tracing a named categorical draw with singleton support.
The code below is an implementation pasted from Cora:
struct NamedCategorical <: Gen.Distribution{Symbol} end

# TODO(https://github.com/probcomp/Gen/issues/110): Remove this once Gen's
# dependency on Distributions.jl is updated
function Gen.Distributions.isprobvec(x::AbstractArray{T}) where T <: Real
  Gen.Distributions.isprobvec(collect(x))
end

"""
    named_categorical(probs::SVector{N, T},
                      labels::SVector{N, Symbol}) where {N, T<:Real}
Sample an element from `labels` according to the probability masses in
`probs`.  Requires `all(probs .>= 0)` and `sum(probs) == 1.0`.
"""
const named_categorical = NamedCategorical()

function Gen.logpdf(::NamedCategorical,
                    label::Symbol,
                    probs::SVector{N, T},
                    labels::SVector{N, Symbol}) where {N, T<:Real}
    d = Gen.Distributions.Categorical(probs)
    i = findfirst(==(label), labels)
    Gen.Distributions.logpdf(d, i)
end

function Gen.logpdf_grad(::NamedCategorical,
                         label::Symbol,
                         probs::SVector{N, T},
                         labels::SVector{N, Symbol}) where {N, T<:Real}
    (nothing, nothing, nothing)
end

function Gen.random(::NamedCategorical,
                    probs::SVector{N, T},
                    labels::SVector{N, Symbol}) where {N, T<:Real}
  labels[rand(Gen.Distributions.Categorical(probs))]
end

(::NamedCategorical)(probs, labels) = Gen.random(NamedCategorical(),
                                                 probs, labels)

Gen.has_output_grad(::NamedCategorical) = false
Gen.has_argument_grads(::NamedCategorical) = (false, false)

# export named_categorical

I guess there would have to be a type parameter, a in NamedCategorical{T} <: Distribution{T} (?), whereas the above code fixes T to be Symbol.

I've found this useful in PClean, and in Metaprob.
A couple comments:

The @dist DSL sitting in a PR now enables this as a one-liner: @dist named_categorical(names, probs) = names[categorical(probs)].
There's another version (called choose_proportionally) that I've implemented for PClean, with gradients.
An important consideration: you don't necessarily want to use find_first in logpdf, because there may be repeat names, and the logpdf should take them all into account.
Users of such a distribution should be careful that if the names ever change, so must the address: choices with different supports should have different addresses for Gen's inference to be correct.

this distribution may (we think?) allow us to store deterministic values in the trace without any extra design overhead, by tracing a named categorical draw with singleton support.

Note that this is not quite the case, due to restriction (4) above. Is there a use case you have in mind for tracing deterministic values?

Yeah, after seeing your comment I was trying to see if there was some
workaround possible given the design constraint in 4.  Didn't come up with
anything clean.
We do have use cases, and Vikash/Marco/I had some discussion last week that
we'll definitely want to get your input on as well.
On Wed, Sep 18, 2019, 1:45 PM Alex Lew<EMAIL_ADDRESS>wrote:

this distribution may (we think?) allow us to store deterministic values
in the trace without any extra design overhead, by tracing a named
categorical draw with singleton support.
Note that this is not quite the case, due to restriction (4) above. Is
there a use case you have in mind for tracing deterministic values?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/probcomp/Gen/issues/125?email_source=notifications&email_token=ABA7VVZN732LGR2HRVDA2D3QKKHM7A5CNFSM4IYCH4CKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7BMPQI#issuecomment-532858817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABA7VVYIW6TOF2YOJP26PP3QKKHM7ANCNFSM4IYCH4CA
.

Sounds good :-) I’ve been thinking quite a bit about deterministic values in traces (for MetaGen), so would be curious to discuss when you’re ready.

This was addressed by https://github.com/probcomp/Gen/pull/103


pronouncing.phones_for_word("a") returns ['AH0', 'EY1', 'EY1 F AO1 R T UW1 W AH1 N T UW1 EY1 T']

The third result seems incorrect.

It does!
Looks like some oddness in the CMU dictionary, which begins:

A  AH0
A(1)  EY1
A'S  EY1 Z
A.  EY1
A.'S  EY1 Z
A.S  EY1 Z
A42128  EY1 F AO1 R T UW1 W AH1 N T UW1 EY1 T
AA  EY2 EY1

The problem is here:
https://github.com/aparrish/pronouncingpy/blob/master/pronouncing/__init__.py#L30
Stripping the brackets and numbers means the word A42128 is interpreted as the word A.

See PR https://github.com/aparrish/pronouncingpy/pull/36 for a fix with tests.

weird, I wonder why "A42128" is even in there in the first place.

I've no idea, but it is quite a nice shade of red.

This may be relevant:
https://www.youtube.com/watch?v=2t6JJL2mEdM
How to Pronounce A42128 / How to Say A42128
From: Hugo<EMAIL_ADDRESS>Sent: Monday, October 23, 2017 11:55 AM
To: aparrish/pronouncingpy<EMAIL_ADDRESS>Cc: Fitzpatrick, Denis F<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [aparrish/pronouncingpy] pronouncing.phones_for_word("a") returns ['AH0', 'EY1', 'EY1 F AO1 R T UW1 W AH1 N T UW1 EY1 T'] (#34)
I've no idea, but it is quite a nice shade of redhttp://www.color-hex.com/color/A42128.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/aparrish/pronouncingpy/issues/34#issuecomment-338724800, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABZDtGdc95b3cuGYaBBrYjLFo7poh2Moks5svMTsgaJpZM4QCTMj.

Ha, that video is just generated from the same CMU Pronouncing Dictionary!
Maybe it's a Mountweazel. https://en.m.wikipedia.org/wiki/Fictitious_entry


Order explore page filter dropdown items alphabetically

See:
Geophysical Concept

Season

(all except Focus Area)

Closed by #649


CHILDESCorpusReader: multiple stems correspond to their source token; keep suffix PoS info

Fixes for _get_words in the class CHILDESCorpusReader
After some standard code like the following is run:
>>> import nltk
>>> corpus_root = nltk.data.find('corpora/childes/data-xml/Eng-NA-MOR/')
>>> from nltk.corpus.reader import CHILDESCorpusReader
>>> valian = CHILDESCorpusReader(corpus_root, "Valian/.*.xml")

The corpus reader does not seem to handle word tokens with multiple stems properly (note "Lastname's" below):
>>> valian.words('Valian/01a.xml')[:5]
['at', 'Parent', "Lastname's", 'house', 'with']
>>> valian.words('Valian/01a.xml', stem=True)[:5]
['at', 'Parent', 'Lastname', 's', 'house']
>>> valian.tagged_words('Valian/01a.xml')[:5]
[('at', 'prep'), ('Parent', 'n:prop'), ("Lastname's", 'n:prop'), ('house', 'n'), ('with', 'prep')]
>>> valian.tagged_words('Valian/01a.xml', stem=True)[:5]
[('at', 'prep'), ('Parent', 'n:prop'), ('Lastname', 'n:prop'), 's', ('house', 'n')]

Issues:

When stem is True, stems are all treated as distinct strings, making it difficult or impossible to align which stems correspond to which words.
If the stem is treated as a suffix in the XML data, then its PoS tag info is lost.

Results after the fixes are implemented:
>>> valian.words('Valian/01a.xml')[:5]
['at', 'Parent', "Lastname's", 'house', 'with']
>>> valian.words('Valian/01a.xml', stem=True)[:5]
['at', 'Parent', 'Lastname~s', 'house', 'with']
>>> valian.tagged_words('Valian/01a.xml')[:5]
[('at', 'prep'), ('Parent', 'n:prop'), ("Lastname's", 'n:prop~poss'), ('house', 'n'), ('with', 'prep')]
>>> valian.tagged_words('Valian/01a.xml', stem=True)[:5]
[('at', 'prep'), ('Parent', 'n:prop'), ('Lastname~s', 'n:prop~poss'), ('house', 'n'), ('with', 'prep')]

(~ is used as the separator for multiple "stems" of a word, following the CHILDES convention.)

@tomonori-nagano or @alexisdimi – would you please review this?

These changes appear reasonable, and in the absence of any objection from @tomonori-nagano or @alexisdimi I'm merging them


Add generic locator

Coverage decreased (-11.9%) to 87.775% when pulling 230b22f23a43590e1ecbb21d0dd6c1a80ca35a31 on add-generic-locator into 965b0548fdb60ad8d560c0d7bb205042ee5c25fc on master.

Coverage increased (+0.03%) to 99.756% when pulling 4c28af0e875ed1ac4f5e4d14607a76a187dfd004 on add-generic-locator into 965b0548fdb60ad8d560c0d7bb205042ee5c25fc on master.


[MINOR][SQL] Remove unnecessary extend from BroadcastHashJoinExec

What changes were proposed in this pull request?

This is just a minor fix. HashJoin already extends JoinCodegenSupport. So we don't need CodegenSupport here for BroadcastHashJoinExec. Submitted separately as a PR here per https://github.com/apache/spark/pull/31802#discussion_r592066686 .
Why are the changes needed?

Clean up code.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Existing unit tests.

cc @dongjoon-hyun could you help take a look when you have time, thanks.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40548/

Test build #135964 has started for PR 31805 at commit b7437da.

Thank you @dongjoon-hyun for review!

Test build #135964 has finished for PR 31805 at commit b7437da.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135964/


Fairy Race Character Import

The parser is failing to understand Fairy Magic properly.  Faerie Fire AND Enlarge Reduce should be set as innates AND leveled spells available as always prepared.  Faerie Fire is missing from leveled spells.

Same issue as #338 and partly due to the way these spells appear in DDB. Closing as tracking elsewhere


MLP::getParameters() is not correct

MLP::getParameters() is not correct
The weights and biases cannot return anything useful as they are not passed in by reference, so the method sets local versions of these only. Also, the extents need to match, so apparently (according to @weefuzzy this may need to be done with reset().

Not sure I see the issue here, the weights and biases are passed as FluidTensorView instances, this is not a container itself but manages a pointer to the actual storage. The assignment operator then copies the data to the FluidTensor allocated by the caller, which may be a bit confusing, but is a pattern used generally in the code base. The sizes are obtained from the MLP instance itself, this method is used in FluidJSON to store the MLP parameters as JSON.

I understand what you are saying, but this is currently not at all obvious from reading the code. Looking at asFluid() it is very hard to disentangle.
If you are confident that this is correct (and indeed the usage in FluidJSON.hpp and my own experience would seem to confirm that) then I'm happy for this to be closed.
As a side issue there's definitely something to think about should you wish the code to be navigable and useable by mere mortals - even if that's just a comment on the methods that use this return pattern, as it's very unlikely (IMHO) even an experienced coder would understand what is going on.

To. be more constructive - the simplest solution to the documentation issue (but one that likely @weefuzzy might rightly not be happy about) would be to rename types - if "Views" were "Ptrs" or something like that then this behaviour would become much less surprising from the viewpoint of the method, although the code within the method would still be non-obvious, as setting the value of a pointer type (the operation in the method is = ) generally doesn't result in changing the underlying data of the original pointer.

Here obviously isn't the place for thrashing through alternative designs, but I will not that this source of confusion is more my fault than @g-roma's.
We all agree that having operator= do a deep copy is confusing (although this is also what Eigen::Map does, so at least we're consistently confusing). Our experience so far has been pretty clear that having some sort of operator to do a multi-dimensionally range-checked deep copy is tremendously handy, both for coding quickly and for catching all sorts of bounds errors early. Perhaps a different operator would be clearer, whilst retaining the convenience.
Name-wise, C++20 uses span for a pointer + size combination, and this class satisfies a concept called View (which FluidTensorView doesn't, due to the deep copying). My slightly more radical idea than simply renaming would be to look at the feasibility of getting rid of FluidTensorView altogether and instead using the reference implementation of a multi-dimensional span looking likely to be n C++23. It does almost everything we can currently do (and better, obvs). Even if that turns out not to be a goer, I'd be open to renaming and decoupling from FluidTensor (or even ditching the latter).
Anyway, I don't think there is a bug here? So, suggest we close and all can feel welcome to start new issues with framework design ideas.

There is no bug. I agree that the operator= usage is the most confusing aspect of the reading the code and would support a name change for that operation. The other matters are likely for those more consistently involved, so I think this can be closed, and hopefully there is some useful contribution to possible areas of confusion, but I'll leave that to the main team to pick up - if input is helpful of course, please ask.


Include -native pkgs in list of OpenEmbedded platform dependencies (#230)

ROS_SUPERFLORE_GENERATED_PLATFORM_PACKAGE_DEPENDENCIES is a list of
package names, not a list of recipes => add -native dependencies to it.

An OE build target with a -native suffix means it's a "build tool", ie, it's to be built for the native instruction set of the build machine and will be executed on it during the build. A recipe can be "extended" to build a native version by appending native to BBCLASSEXTEND.

@herb-kuta-lge Alright, good to know. I think I'll just merge this then, since it's passing CI.

Looks good to me also, thank you both!


Feat: Timeline v9 (SOFIE-2514)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Updates TSR to use the new Timeline version 9, which is a Big breaking change (see PR: https://github.com/SuperFlyTV/supertimeline/pull/95 )

Codecov Report
Patch coverage: 100.00% and project coverage change: +62.73% :tada:

Comparison is base (8c174b7) 37.26% compared to head (8cd6131) 100.00%.

Additional details and impacted files
@@               Coverage Diff               @@
##           release51      #270       +/-   ##
===============================================
+ Coverage      37.26%   100.00%   +62.73%
===============================================
  Files             99        44       -55
  Lines           9975       656     -9319
  Branches        2460        76     -2384
===============================================
- Hits            3717       656     -3061
+ Misses          5696         0     -5696
+ Partials         562         0      -562

Files Changed
Coverage Δ

...ackages/timeline-state-resolver-types/src/index.ts
100.00% <ø> (ø)

...-resolver-types/src/superfly-timeline/api/index.ts
100.00% <100.00%> (ø)

...ypes/src/superfly-timeline/api/resolvedTimeline.ts
100.00% <100.00%> (ø)

...tate-resolver-types/src/superfly-timeline/index.ts
100.00% <100.00%> (ø)

... and 57 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I suppose that now that https://github.com/SuperFlyTV/supertimeline/pull/95 is merged, we can move forward on this?


BlackListed-Permissions function not working

I tested using the example from the wiki: https://docs.crazycrew.us/crazycrates/crates/examples/csgo
    3:
      DisplayName: '&e&l$1,000'
      DisplayItem: 'SUNFLOWER'
      DisplayAmount: 1
      Lore:
        - '&7Make it rain Money.'
        - '&6&lChance: &c&l20%'
      MaxRange: 100
      Chance: 20
      Firework: true
      Commands:
        - 'eco give %player% 1000'
        - 'lp user %player% permission set crazycrates.blacklist.basic.3'
      # Messages:
      #   - '&7You just won &r%reward%.'
      BlackListed-Permissions:
        - 'crazycrates.blacklist.basic.3'
      Alternative-Prize:
        Toggle: true
        Messages:
          - '&r &8[&bCrazyCrates&8]: &7You have already won that prize, so enjoy some gold nuggets.'
        Commands:
          - 'give %player% gold_nugget 16'

lp user ThiagoROX permission check crazycrates.blacklist.basic.3

[22:36:42 INFO]: [LP] Permission information for crazycrates.blacklist.basic.3:
[22:36:42 INFO]: [LP] - thiagorox has crazycrates.blacklist.basic.3 set to true in context global.
[22:36:42 INFO]: [LP] - thiagorox does not inherit crazycrates.blacklist.basic.3.
[22:36:42 INFO]: [LP]
[22:36:42 INFO]: [LP] Permission check for crazycrates.blacklist.basic.3:
[22:36:42 INFO]: [LP]     Result: true
[22:36:42 INFO]: [LP]     Processor: common.DirectProcessor
[22:36:42 INFO]: [LP]     Cause: 3098723f-0cac-3b7c-9a14-ef6aaec9c26f has crazycrates.blacklist.basic.3 set to true in context global
[22:36:42 INFO]: [LP]     Context: (dimension-type=overworld) (essentials:afk=false) (essentials:jailed=false) (essentials:muted=false) (essentials:vanished=false) (gamemode=creative) (server=genesis) (world=world)

Make sure that you are not OPed.

Thanks 😳


How to prevent BetterBibLaTeX from escaping braces in author names (institution name)

Zotero version:
<IP_ADDRESS>
BBT version:
5.6.8
Support log ID:
2EWDGA7X-refs-euc
Exporter used:
BetterBibLaTeX
Expected behavior:
Author with braces without escaping
Actual behavior:
Braces are escaped.
My author is an institution and and I put it in braces. When I export with BetterBibLaTeX the braces around the author are escaped. What settings need to be made to prevent BetterBibLaTeX from escaping. Thank you very much for your help!
@online{DeutscherLandkreistag_2021_AufgabenDerLandkreise, title = {Aufgaben der {{Landkreise}}}, author = {\{Deutscher Landkreistag\}} }

If you change the name to single-field mode by clicking the white bar to the left of the name (if you hover you will see "Switch to single field") and remove the braces, I think you will get the output you want.

Sorry, I overlooked this bar. Works perfectly. Thanks for answering so swiftly!


Merge docs and website folders from cloak branch into main

The cloak branch included a separate set of content and docs website configuration. When cloak was merged into main, those docs/configuration were preserved by moving them to the docs-0.3 and website-0.3 directories (#3371). These docs/configuration should be merged into the existing docs and website directories and the above temporary directories should be deleted.

The temporary directories added in https://github.com/dagger/dagger/pull/3371 look to have been removed at some point - so we can close this one out.


@Suppress with Ruleset name does not work

The docs at https://detekt.dev/docs/introduction/suppressing-rules/ say that it should be possible to suppress a warning like this @Suppress("complexity.LongParameterList")
So, this should not trigger this warning:
@Suppress("complexity.LongParameterList")
private fun foo (a: Int, b: Int, c: Int, d: Int, e: Int, f: Int, g: Int) {
}

but it currently does.
detekt 1.22.0 and gradle 8.1-rc-3

Does it work if you just @Suppress("LongParameterList")?

Yes, it works without the rule set name.
So it is not that urgent, but I'd like to use the full name so that it is easier to find the rule in the config file.

I would say that this is an issue in the documentation instead a bad behavior.
We already check for "rulename" and "detekt.rulename". I don't think adding even more options here is a good idea.

And there specifically is a test case for this
@Test
fun `suppresses by combination of rule set and rule id`() {
    assertCodeIsSuppressed("""@Suppress("complexity.LongParameterList")$code""", config)
}

I will investigate.

The reason for this behavior is not so obvious. Within the Rule#report function we check whether to suppress the finding or not. To determine the name of the ruleset for this purpose, we rely on Config#parentPath. The problem is that this is implemented only in the YamlConfig and all other Config implementations return null. Therefor the suppression mechanism fails when the suppression is configured with the rule set id.
As the rule - as it stands right now - is unaware of the ruleset it belongs to, the only way to resolve this is to implement the getter correctly in all Config implementations.   Any idea if that will cause any side effects? I will prepare a pr.


Add function to create configuration for actions in pkg

I'm playing around with the v3 beta, and trying to refactor the terraform provider. There doesn't seem to be an easy way to create a configuration, and I feel like I am duplicating a bunch of logic from initActionConfig in cmd/helm. It would be super helpful if there some methods added to build the configuration object in pkg

Some other issues. There is a wierd dependency issue. The code wants me to import "k8s.io/helm/pkg/action"
When i run go get<EMAIL_ADDRESS>I get this error go build helm.sh/helm/pkg/chart: no Go files in
That leads me into dependency helm. Hoping that https://github.com/helm/helm/pull/5498
will address the dependency problems.

If you have suggestions on how we could improve he experience around creating Configuration objects, we'd happy review PRs.

Closing as #6341 merged


Flush the stdout buffer after logging to console

Normally if the stdout is connected to a PTY, the stdout buffer is flushed automatically after each \n.
However if I run SRS under Docker (or other container services), the stdout is not connected to a PTY (unless I ask it to), causing the last log line to be incomplete (showing only half).
By adding fflush, we could make sure each log line is completely shown, whenever the stdout is connected to a PTY or not.
P.S. I am working on an SRS Docker image, which allows "one-key installing" SRS on a huge cluster of servers. It is available at https://hub.docker.com/r/m13253/srs/, with build script available at https://github.com/m13253/docker-srs.

👍


Chip: Improve styling and behavior

We need to revise the chip component.

Should it have more visible checkbox/radios?
Should fit better with the rest of the component-API

New design suggestion:

This should make it clearer which chips that could be selected as single (radio) and witch could be selected multiple (checkbox)

Summary of our discussions:
Initially, we considered simplifying the Chip component to only be "Removable" and keeping it as an internal component for use in the Combobox. However, we've noticed some developers have already used it in other contexts, so we need to evaluate that.
We considered to remove the Chip Checkbox, and update Checkbox with a new variant with a chip design. This chip variant of the checkbox will be used when placed horizontally. Similarly, the Chip Radio was considered to be removed, and Radio updated with a new variant with a chip design to be used when placed horizontally. For ToggleGroup, we were also considering a chip design variant.
After further discussion with developers, we considered keeping the Chip as a standalone component with all variants, as it would be easier to locate in the documentation.  If we find a better way in the future, we can always deprecate some of it.

New design is finished in Figma.
### Tasks
- [ ] Add all states
- [ ] Add relevant examples
- [ ] Old Chip is moved to workspace
- [ ] Combobox is updated with new chip


associate user with drop & allow them to edit

we already have a user_id column on the drop, and we need to fill this with something!

[x] rename user_id to soundcloud_user_id
[x] when a user creates a drop, store the client.get('/me').id in the drop.user
store the user's oauth token in the cookie (check the soundcloud gem on specifics)
[x] use it to match the user & the drop & then show them an edit/delete button

closed w/ https://github.com/sounddrop/sounddrop/issues/56


feat: update studio to add storage UI

uses the SUPABASE_PUBLIC_URL env var, else the default api endpoint is http://localhost:8080

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 57.077%

Totals

Change from base Build<PHONE_NUMBER>:
0.007%

Covered Lines:
3492

Relevant Lines:
6118

💛  - Coveralls


Vspec fixes

The PR contains multiple fixes.
In x86/x86_model.py

In X86UnicornVspecOps reset curr_mem_store for each input
When we load from memory, check if the address the instruction load from overlaps with any tainted address in the range [address, address+size]
In X86UnicornVspecAllMemoryFaults,  Do not call X86UnicornVspecAll.speculate_instruction twice on the faulty instruction.

In x86/x86_generator.py:
When we create instrumentation, do not re-use the same operand instance. Rather, copy the operand and set the dest property properly.

would you mind editing the history (git rebase --interactive) to make each fix as a separate commit?

@OleksiiOleksenko Is it better now ? Let me know if I should split the commits further

Yep, much better. Could you also prefix the commit messages with the module you're changing (see the commit history for examples)? It simplifies rebasing this commits in the future.

@van-ema there are a lot of conflicts after changing the target branch. Don't worry about it - I'll take care of them

See #67


fast-autocomplete separates words with a dash ('-') symbol.

Hi, i've been using fast autocomplete for my project and so far I'm quite happy with it, but I found something very annonying:
some of the words of my dictionary have a dash, like for example 'ASB-E'. When I search for 'ASB-E', I don't get tthe exact math of 'ASB-E',  autocomplete seems to assume that the dash is a word separator and  I get this result:

Is there any fix to it? I would like to make autocomplete treat the dash as just another letter. Thank you.

Hi, thanks.
Off the top of my head, you should add dash as a character that it accepts. Examples are here: https://github.com/seperman/fast-autocomplete#unicode
Sep Dehpour

On Mar 24, 2021, at 12:46 PM, Werner German Busch @.***> wrote:

Hi, i've been using fast autocomplete for my project and so far I'm quite happy with it, but I found something very annonying:
some of the words of my dictionary have a dash, like for example 'ASB-E'. When I search for 'ASB-E', I don't get tthe exact math of 'ASB-E', autocomplete seems to assume that the dash is a word separator and I get this result:
Is there any fix to it? I would like to make autocomplete treat the dash as just another letter. Thank you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @wgbusch
Seems like that must have solved the problem for you.


Update Post “how-is-a-termite-inspection-done”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
9a07fbe91a7ff7afa2b3fe48824eea0e0a9692b7

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/65484397898b7300088953b3


added images for the WIS2-SWIM interoperability section

These images are non-editable. Will need to remedy this before final publication.

superseded by #89


Can this project use the QUIC communication protocol and support streaming?

Problem description
I want to use this program to perform 3D modeling remotely.
Proposal
I want to use this program to perform 3D modeling remotely.So, can this project use the QUIC communication protocol and support streaming?
Additional context
No response

if it does not work dm crystal.999 on dc for help


Kaolin installation problem (Colab/Cloud)

I did:
!git clone https://github.com/NVIDIAGameWorks/kaolin.git and then
%cd kaolin
!python setup.py install
It said: Finished processing dependencies for kaolin==0.1.0, so I thought it's installed correctly.
However, when I try to import:
import kaolin as kal
print(kal.__version__)
it says that kal has no version, and also I can't use anything from kaolin.
On the other hand, another error is when line import kaolin as kal is not recognized because of the metaclass=KaolinDatasetMeta error.
Did anyone manage to start kaolin from Google Colab/Cloud? A few days ago everything was working...

Were you able to resolve this issue?

Were you able to resolve this issue?

Unfortunately, I wasn't. But a reformat is planned in December, so it will be fixed then.

Ohh. I've  tried installing a previous commit (from August 26) and that it didn't work too.  Currently trying out this solution (https://github.com/NVIDIAGameWorks/kaolin/issues/321)

Ohh. I've tried installing a previous commit (from August 26) and that it didn't work too. Currently trying out this solution (#321)

I've already tried that and it didn't work.

Changing line 56 in kaolin/datasets/base.py from
class KaolinDataset(Dataset, metaclass=KaolinDatasetMeta):
to
class KaolinDataset(Dataset):
This worked for me

Changing line 56 in kaolin/datasets/base.py from
class KaolinDataset(Dataset, metaclass=KaolinDatasetMeta):
to
class KaolinDataset(Dataset):
This worked for me

With the new update that happened few moments ago, everything is now working on Cloud/Colab :+1:
Great!!!
I will close issue right now.


feature: send out deployment markers

https://github.com/omisego/elixir-omg/issues/908
Overview
Send manual events and entries to Datadog and Sentry but also notify Slack
Changes

Add Datadog, Sentry and Slack integration in Circle CI

Testing
yolo

Coverage remained the same at ?% when pulling 0c3f7ff505d3f625db51df7a48fb345b2da24c97 on 908-dd-sentry-deployment-markers into f11cd8c25c312617d08a71fd61aa4ff800c97bb0 on master.

This lovely PR happened because @achiurizo paired with me. Tnx bubu.


New Navigation Taxonomy

OB has requested different taxonomy for navigation of the site. This is to mimic:

Home
Alma Mater

History of the College
Founders
Moto, Vision & Mission
Coat of Arms
College Anthem
College Houses
Rectors

Administration

Managing Committee
Rectors
Vice Rector
Deputy Principals
Staff

Academic

Achievements
News

Co-Curriculum

Clubs
Sports

Students

College Sections
Prefects

Facilities
Gallery

This is the new navbar for desktop design

In hover state

Prototype: https://codesandbox.io/p/devbox/dropdown-menu-tailwind-alpine-86pm7d

[ ] make close button high contrast
[ ] fill in whitespace under mobile nav

waiting for content from writers...


fix: Wrong conditional for master_user_options

Description
Found an error in master_user_options conditional creation
Old behavior (master_user_name     = try(master_user_options.value.master_user_name, null) == null ? try(master_user_options.value.master_user_name, null) : null):
if master_user_options.value.master_user_name == null
then
    try to put master_user_options.value.master_user_name (which is null due to the previous statement)
else (if master_user_options.value.master_user_name is not null)
   put null

Should be:
if master_user_options.value.master_user_name != null
then
    try to put master_user_options.value.master_user_name (which is not null due to the previous statement)
else (if master_user_options.value.master_user_name is null)
   put null

This issue has been resolved in version 1.1.1 :tada:


rio: 0.0.39 -> 0.1.0

Description of changes
Changelog: https://github.com/raphamorim/rio/blob/v0.1.0/CHANGELOG.md
cc @TornaxO7 @otavio @oluceps

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Neverthless, thank you for your contribution :)

@GaetanLepage There's already an accepted PR. We just need to wait until it's merged #318862

Oh, I haven't noticed that. Sorry for the useless ping then !

No problem :+1:


Updated Dashboard (slightly) for better readability.

While trying to debug some data uri problem I had, I decided to update the Dashboard a wee bit more.
I've mainly just added in some Unordered lists for various things.
The biggest change was adding in a Failure class so that the ProcessQueue and ReducingQueue can now have a wee bit more information. I also loop through child exceptions that have been caught and wrapped.
:)

Thanks for this! Very nice to have the exceptions in the dashboard.


Added code of conduct

@gaushikmr Thanks for the PR. I will review and merge asap. This was needed.

ok


feat: exclude detectors from lookup

It would be very handy to have the ability to exclude lookups for specific detectors.
In my project lookup for language in navigator might lead to undesirable results, unfortunately I haven't found an option to disable lookup in navigator.
Could you checkout my pr and consider to merge it?
Or maybe I am missing something and you can suggest how I can avoid lookup in window.navigator?

you can override the options.order -> it specifies which lookups to use and in which order (no need for an exclude)

Thank you! You did a great work!


Create DiikandSolution

Welcome to our website

Hello @augustintshibang,
Thank you for creating a pull request to contribute to the project. Before we can review, we ask that you sign the cla/google agreement. Furthermore, we don't take any pull requests into the release branch, only the "main" branch.

In addition, it appears that your commit does anything but add an empty file to the project. So I will close this PR anyway.


Emit a remark when rebuilding the standard library interface

Although users should usually use a prebuilt standard library, in those rare configurations where one needs to be built, the compiler appears to hang for several minutes, even on a trivial compilation. This PR adds a remark that's emitted when this happens, explaining that the standard library is being rebuilt and it will take a few minutes.

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 0ffe2a3a0a88caef606f86045933b21cc12d1ebf

Build failed
Swift Test OS X Platform
Git Sha - 0ffe2a3a0a88caef606f86045933b21cc12d1ebf

Build failed
Swift Test Linux Platform
Git Sha - c7f1c0368914f654f1795714141dd53f3bcad54d

Build failed
Swift Test OS X Platform
Git Sha - c7f1c0368914f654f1795714141dd53f3bcad54d

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - def55c081a44e0bb09d0d77234d1ff1bee32a616

Build failed
Swift Test OS X Platform
Git Sha - def55c081a44e0bb09d0d77234d1ff1bee32a616

@swift-ci please test and merge

@swift-ci please test and merge


[Core] Currency.compareTo

The following test fails for the last two lines.  compareTo returns
btc.compareTo(Currency.LTC) --> -10,
xbt.compareTo(Currency.LTC) --> 12
To fix, I think we need to add look at Currency.compareTo, subtracting hashcodes looks like it should work.  CurrencyPair.compareTo looks ok too.  so wondering how to fix.
Changing this may have a big impact, so any ideas?
  @Test
  public void testCompareTo() {
    Currency btc = Currency.BTC;
    Currency xbt = Currency.XBT;

    assertThat(btc.equals(btc));
    assertThat(btc.equals(xbt));
    assertThat(xbt.equals(btc));
    assertThat(xbt.equals(xbt));

    assertEquals(btc.compareTo(xbt), 0);
    assertEquals(xbt.compareTo(btc), 0);

    assertEquals(btc.compareTo(Currency.LTC), xbt.compareTo(Currency.LTC));
    assertEquals(btc.compareTo(Currency.ETH), xbt.compareTo(Currency.ETH));
  }

My temprary solution seems to be holding out
  ....
  @Override
  public int compareTo(Currency o) {
    return attributes.equals(o.attributes) ? 0 : toString().compareTo(o.toString());
  }
  .....
  @Override
  public int compareTo(CurrencyPair o) {
    return equals(o) ? 0 : toString().compareTo(o.toString());
  }
  ....

https://github.com/knowm/XChange/issues/2886
https://github.com/knowm/XChange/issues/2720
https://github.com/knowm/XChange/issues/2535
According to LiveCoin XBT ->
{ name: "Bricktox", symbol: "XBT", walletStatus: "normal", withdrawFee: 1488, difficulty: null, minDepositAmount: 0, minWithdrawAmount: 0.01, minOrderAmount: 583.3334 },
however Bricktox last post is March 2018 https://twitter.com/bricktox?lang=en (the only month it was active, for it's ICO).  In April 2018, Bricktox crashed to almost 0, and it now lives on a zombie coin.


Provide an executable that calls titlecase

It would be nice to be able to run titlecase some title to be capitalized in the shell rather than having to jump into ghci.

https://github.com/nkaretnikov/titlecase/pull/5 solves this.


Activate scripts from turbo streams

Hi! I was building my new Rails app with Turbo Frames and Turbo Streams and bumped into the issue of script tags not executed after being appended via Turbo Streams. Found the same issue(#186) here.
There I found the @xtr3me's PR https://github.com/hotwired/turbo/pull/192/ with an approach for turbo frames, and decided to make the same-ish thing for streams.
So I've come up with this PR. Could you please take a look?
It works well for content with

I'm also having this problem. Specifically, I want to use turbo streams to replace one of the frames with a chartkick chart. The script tag generated by the rails helpers doesn't execute.

Hi @geronimo, hi have your same issue and I have opted with an override of the chartkick helper and a stimulus controller for  render chart.js
https://gist.github.com/retsef/d6dffcc0b792bce66bd9038bd8646388
In our case we reload a turbo-frame containing charts on a date range picker.

Would be happy to see this finished. Let's try to reuse the logic we use from frames to trigger script tags. Then we need tests.

@dhh Happy to see your approval, it gave me extra motivation :)
Added a test to stream_tests.ts, which adds a blank div.message and then fills it with text and assigns a css class to it via a

Thank you @elShiaLabeouf for opening this original pull request.
I've created https://github.com/hotwired/turbo/pull/660 to catch this changeset up to the latest commits. I've added you as a co-author.

Hi @seanpdoyle! Thanks, I appreciate it!
I'm just happy this code has made it to the main!
Sorry for "abandoning" this, been buried under the bloody work at my workplace lately.

@seanpdoyle  The


replace the std package in runtime/fundamentals/standard_library with other package which is compatible with nodejs

Because std/fs package is not compatible with nodejs, so it is not proper to use std/fs package here.

this line https://github.com/denoland/docs/blob/34bab84e855dbf959cd1180b64e83b13622a5976/runtime/fundamentals/standard_library.md?plain=1#L67


Screenshots not matching the theme

Motivation
The theme looks quite different from the 3-years-old screenshots in the docs, at least in my browser:

Note the alignment of the navigation (left-aligned in screenshots) and the post headings (underlined and different size and color than in screenshots).
Suggestion
The screenshots should be updated. It would also be great to know how to obtain something closer to the old look (which is better in my opinion).

Feel free to submit pull requests with updated screenshots. The demo site's source files are all in /docs.

@mmistakes, what's the quickest and/or best way to remove the horizontal line that appears under headings?

By editing/overriding the CSS.


Steamworks ane not working on Mac

Hi,
When running a build on Mac with fresteamworks, the build does not install. It says file is damage or corrupt. But when I removed the ane from my build, then it installs fine. I am attaching error image from my Mac below. These are the logs on Mac:-
Mar 17 13:05:59 Mac6 Google Chrome Helper[1597]: A Parser-blocking, cross-origin script, http://adobe.tt.omtrdc.net/m2/adobe/mbox/ajax?mboxHost=www.adobe.com&mboxPage=1489736159530-948368&screenHeight=1080&screenWidth=1920&browserWidth=1302&browserHeight=833&browserTimeOffset=330&colorDepth=24&mboxSession=1489736159530-948368&mboxXDomain=enabled&mboxCount=1&mboxTime=1489755959536&mbox=adobe-global-mbox&mboxId=0&mboxURL=http%3A%2F%2Fwww.adobe.com%2Fdevnet%2Fair%2Farticles%2Fbuilding-ane-ios-android-pt5.html&mboxReferrer=&mboxVersion=57, is invoked via document.write. This may be blocked by the browser if the device has poor network connectivity. See https://www.chromestatus.com/feature/5718547946799104 for more details.
Mar 17 13:06:42 Mac6 Google Chrome Helper[1597]: A Parser-blocking, cross-origin script, http://adobe.tt.omtrdc.net/m2/adobe/mbox/standard?mboxHost=www.adobe.com&mboxPage=1489736159530-948368&screenHeight=1080&screenWidth=1920&browserWidth=1302&browserHeight=833&browserTimeOffset=330&colorDepth=24&mboxSession=1489736159530-948368&mboxXDomain=enabled&mboxCount=2&mboxTime=1489756002393&mboxPC=1489736159530-948368.22_6&mbox=defaultmboxname&mboxId=0&mboxURL=http%3A%2F%2Fwww.adobe.com%2Fdevnet%2Fair%2Farticles%2Fbuilding-ane-ios-android-pt5.html&mboxReferrer=&mboxVersion=57, is invoked via document.write. This may be blocked by the browser if the device has poor network connectivity. See https://www.chromestatus.com/feature/5718547946799104 for more details.
Mar 17 13:20:16 Mac6 Adobe AIR Application Installer[1639]: Application Installer begin with version <IP_ADDRESS> on Mac OS 10.11.6 x86
Mar 17 13:20:16 Mac6 Adobe AIR Application Installer[1639]: Commandline is: /Users/mac6/Downloads/Game.air
Mar 17 13:20:16 Mac6 Adobe AIR Application Installer[1639]: Installed runtime (<IP_ADDRESS>) located at /Library/Frameworks/Adobe AIR.framework
Mar 17 13:20:19 Mac6 Adobe AIR Application Installer[1639]: Unpackaging file:///Users/mac6/Downloads/Game.air to /var/folders/7g/m5sgxm5s07j5s2m78zbwnt040000gn/T/FlashTmp.Rt3HjY
Mar 17 13:20:19 Mac6 Adobe AIR Application Installer[1639]: Got an unexpected fatal error while unpackaging: [ErrorEvent type="error" bubbles=false cancelable=false eventPhase=2 text="not an AIR file" errorID=0]
Mar 17 13:20:42 Mac6 Adobe AIR Application Installer[1639]: Application Installer end with exit code 7
Mar 17 13:20:43 Mac6 com.apple.xpc.launchd[1] (com.adobe.air.ApplicationInstaller.287712[1639]): Service exited with abnormal code: 7
So, do you have any idea how to fix this. Thanks in advance.

Just from the error message, it's hard to tell what the problem is. Is there any chance you could send me the app bundle you've built? My email adress is in my profile, if the file's not too big feel free to just send it directly as an attachment - otherwise a Dropbox or Google Drive link or something like that would also work.
If that's not possible, can you share any details on how you're building the app?

I making build from flashdevelop.
My Packager.bat file looks like this:-
@echo off
:: Set working dir
cd %~dp0 & cd ..
if not exist %CERT_FILE% goto certificate
:: AIR output
if not exist %AIR_PATH% md %AIR_PATH%
set OUTPUT=%AIR_PATH%%AIR_NAME%%AIR_TARGET%.air
:: Package
echo.
echo Packaging %AIR_NAME%%AIR_TARGET%.air using certificate %CERT_FILE%...
call adt -package %OPTIONS% %SIGNING_OPTIONS% -target native %OUTPUT% %APP_XML% %FILE_OR_DIR% -extdir lib/
if errorlevel 1 goto failed
goto end
:certificate
echo.
echo Certificate not found: %CERT_FILE%
echo.
echo Troubleshooting:
echo - generate a default certificate using 'bat\CreateCertificate.bat'
echo.
if %PAUSE_ERRORS%==1 pause
exit
:failed
echo AIR setup creation FAILED.
echo.
echo Troubleshooting:
echo - verify AIR SDK target version in %APP_XML%
echo.
if %PAUSE_ERRORS%==1 pause
exit
:end
echo.
I am using AIR version 20. And also tell what else detail you want. Thanks

Can you try the new build available at http://dump.ventero.de/FRESteamWorks/v0.7/ please?

I built macOS version my game today with v0.7 with AIR 29. It works fine.


Add support for library crates with multiple binaries

When users have a library crate with multiple binary files, they are not able to specify metadata for a single binary, only (I believe) for all of them at once. This PR adds an optional argument binary: Option<&str> to compile() which can be used to select the binary by its' name.
An example project showing how to use it has been added in multi_binary_example and it produces the following result:

This would require a minor version bump, as it changes how the .compile() function is called.
I have only tested my changes with MSVC on Windows, GNU compiler will require additional testing.
This has been discussed in https://github.com/mxre/winres/issues/32 with mxre saying there is nothing winres can do (at that moment). You can also not have a per-binary build script (https://github.com/rust-lang/cargo/issues/1430), so something like this change is required to add metadata to library+binary combo crates.

Thanks for contributing!
I didn't have time to really look at the PR (and the repo wasn't on my watchlist so i didn't get a notification), but what do you think about adding a new method like compile_for() so we don't have to make it a breaking change?
This way we could also remove the Option from the public api if we wanted to.

Sure, compile_for() sounds good, I'll do that.

Sorry for the long delay! Writing this so you don't think we're ghosting you, there are 2 small things i wanna note already:

The gnu part does indeed not work, it just fails to compile. I meant to look into it (which is why i didn't straight up post the error :/ ) but had to postpone it to focus on some Tauri 1.3 stuff.
We're thinking about dropping this fork again, ref: https://github.com/tauri-apps/tauri/issues/6746 / https://github.com/tauri-apps/tauri/pull/6760 but not sure yet and it may take a while and we're not 100% sure yet anyway.

Anywayyy, i'll try to get back to you about the gnu stuff this week, sorry again.

Okay soooo, I ended up replacing the resource compiler with the embed-resource crate since it has gnu/msvc and cross-platform handling, and to push it to Tauri users without a new Tauri release asap (because we released NSIS support in 1.3 and this made the experimental cross-platform compilation guide a tiny bit nicer).
This change also added support for multi-binary handling, but as far as i understand it, embed-resource is not that good at handling .dll files and focuses on .exe binaries, which again is perfect for Tauri, but may not be that nice for more general uses.
Therefore i'd suggest reaching out to the winresource sister-fork and see if they are interest, though judging by https://github.com/BenjaminRi/winresource/issues/8 it looks like they are. I'd also be interested to help you figure out the gnu toolchain issues along the way if i find the time :)

Sorry for making conflicting changes "behind your back" but it was really a last-minute thing we wanted to tackle :/ Thank you so much for contributing though! ❤


Add missing WindowOperator#isBlocked method

WindowOperator can block when spilling,
therefore it needs to explicitly support isBlocked
method.

WindowOperator can block when spilling,
therefore it needs to explicitly support isBlocked
method.

https://github.com/prestosql/presto/blob/c74b775e103a7a239aafef1b7ca15fe6370495fe/presto-main/src/main/java/io/prestosql/operator/Operator.java#L64-L66

@findepi spill can be triggered by window operator itself when flipping memory without explicit revoke signal


Save chart as image

Hello
Any method possible to save chart as image? I want use chart on pdf export ( use TCPDF ) and I can add chart as image on it
Or any method to draw chart on PDF output
Thanks

toDataURL() will give you the base64 encoded PNG of the canvas.
You could use this as the SRC for the image in the pdf.

Does Chartist even use the canvas object? I thought, it utilizes SVG.

Hi,
Have you had any luck with this?, I've been having the same problem and couldn't find a solution so far.
Thanks

I've been struggling with this for a few days now....hopefully someone will be able to figure out some way to do this!

Take a look at PhantomJS. https://github.com/ariya/phantomjs/blob/master/examples/rasterize.js

Thank you very much, I try work whit it tonight
Best

@voltan did you have any luck with this?  I won't be able to try it out until this coming weekend :(

https://developer.mozilla.org/en-US/docs/Web/API/Canvas_API/Drawing_DOM_objects_into_a_canvas
That article details drawing svg to a canvas, from which you can create a PNG.

Hi Any luck on the above issue ?? . I am using html2canvas to convert my page to PNG, the whole page gets saved, BUT chart created by using chartist.

I guess it would be possible with any of the above approaches. However, this will be radically simplified with work done on #810, #813 and #814

if it can helps with people struggling with saving Chartist plots to start something, here's how I've done it :
https://gist.github.com/cyrilmesvayn/981767e80ee6fa23fc5611697426ef8c
Working example as of 06-07-2017 in Chrome 59 :
https://jsfiddle.net/qvbekmvy/33/
DISCLAMER : This is by no means bulletproof or complete. It works in Chrome, as in my use case I doesn't need to support anything else. It's meant as a starting point for someone figuring out how the heck to do this.


create RS irrespective of PV mount status.

Fix for [2319824] [RDR] Replication via Volsync requires PVC to be mounted before PVC is synchronized
Create ReplicationSource irrespective of PV being mounted to running pod.

How this is tested:
(ramen) [root@ramen1 samples]# kubectl scale deployment/busybox --replicas=0 -n deployment-cephfs --context dr1
deployment.apps/busybox scaled
(ramen) [root@ramen1 samples]# kubectl get replicationsource -A --context dr1
NAMESPACE           NAME          SOURCE        LAST SYNC              DURATION        NEXT SYNC
deployment-cephfs   busybox-pvc   busybox-pvc   2024-12-12T16:54:40Z   40.696555939s   2024-12-12T16:55:00Z
(ramen) [root@ramen1 samples]# kubectl delete replicationsource busybox-pvc -n deployment-cephfs  --context dr1
replicationsource.volsync.backube "busybox-pvc" deleted
(ramen) [root@ramen1 samples]# kubectl get po -n deployment-cephfs --context dr1
NAME                                      READY   STATUS    RESTARTS   AGE
volsync-rsync-tls-src-busybox-pvc-5kb8n   0/1     Pending   0          1s
(ramen) [root@ramen1 samples]# kubectl get po -n deployment-cephfs --context dr1
No resources found in deployment-cephfs namespace.
(ramen) [root@ramen1 samples]# kubectl get replicationsource -A --context dr1
NAMESPACE           NAME          SOURCE        LAST SYNC              DURATION        NEXT SYNC
deployment-cephfs   busybox-pvc   busybox-pvc   2024-12-12T16:57:34Z   34.517990544s   2024-12-12T16:58:00Z

ramen-dr-cluster-operator pod log:
[1] 588806
(ramen) [root@ramen1 ramen]# grep "PVC is not in use by ready pod, yet creating RS ..." /root/DR/samples/dr1-ramen.log
2024-12-12T16:55:11.923Z	INFO	controllers.VolumeReplicationGroup	volsync/vshandler.go:399	PVC is not in use by ready pod, yet creating RS ...	{"VolumeReplicationGroup": {"name":"deployment-cephfs-drpc","namespace":"deployment-cephfs"}, "rid": "14afb717-e7a4-4367-a85c-b31b207693a6", "rsSpec": {"protectedPVC":{"namespace":"deployment-cephfs","name":"busybox-pvc","protectedByVolSync":true,"storageID":{"id":""},"replicationID":{"id":""},"storageClassName":"rook-cephfs","annotations":{"apps.open-cluster-management.io/hosting-subscription":"deployment-cephfs/subscription","apps.open-cluster-management.io/reconcile-option":"merge"},"labels":{"app":"deployment-cephfs","app.kubernetes.io/part-of":"deployment-cephfs","appname":"busybox"},"accessModes":["ReadWriteMany"],"resources":{"requests":{"storage":"1Gi"}},"conditions":[{"type":"ReplicationSourceSetup","status":"True","observedGeneration":1,"lastTransitionTime":"2024-12-12T16:37:47Z","reason":"SourceInitialized","message":"Ready"}],"volumeMode":"Filesystem"}}, "runFinalSync": false}


certain env. in WSLENV are overwritten by WSLg (such as DISPLAY, WAYLAND_DISPLAY, XDG_RUNTIME_DIR)

The normal way of setting environment variables on linux should be to respect user settings.  However, WSLg will just overwrite your DISPLAY if you've set its value somehow.  In my case, I have some extensive scripts that find the correct IP on startup and registers that in a DISPLAY environment variable on the windows side with WSLENV making it propagate to the WSL side.  This failed miserably with WSLg overwriting it with :0.  (I'd obviously prefer using the WSLg over a third party X server, but just a few quick experiments made it clear to me that it is not nearly robust enough for actual use.)
After a whole bunch of fumbling around, I resorted to guiApplications=false in .wslconfig, but this is hard to find, it disables wslg for all distros, and it also disables the pulseaudio server.
Environment
Windows build number: Microsoft Windows [Version 10.0.22000.100]
Your Distribution version: 20.04
Your WSLg version: 1.0.24

Steps to reproduce
In your Windows environment variables:

Set a DISPLAY variable with whatever
Set WSLENV to include it, eg, Path/ul:DISPLAY

Expected behavior
DISPLAY should have the value set on the windows side.
Actual behavior
DISPLAY gets overwritten by :0.

@elibarzilay, thanks for reporting the issue. If WSLg is enabled, it assumes user is going to use WSLg as it's X server (at least currently, since WSLg doesn't support to co-exist with other X server solution), thus it automatically sets DISPLAY to :0 where WSLg's X server is listening. So would you please clarify your usage why you wants have custom DISPLAY path while WSLg is enabled ? thanks for trying out WSLg!

@hideyukn88, such arguments of "you installed X and therefore we're going to change your environment to use it" are, in my experience, never a good idea.  To be more specific:

My problem started from just update my WSL installation.  The process went as usual, no warnings about changing anything for me.  The thing is that WSLg is included in the update and therefore it got turned on and broke my working environment, which is not something that I wanted.  And my environment is "showing" what it wants by the fact that I do have DISPLAY set, so if only  WSLg would set it only if it's not already set then I could save a day of figuring out how to avoid it.

I have it disabled now so I can work, but I would like it if I could still use the pulseaudio server.  The two are tied together, and since I must disable all of WSLg globally, I cannot use the audio.

I would have probably set a second linux distribution just to try out the GUI functionality from time to time, but again, this is impossible since I only get a global on-off switch.

Even more likely, I'd just run WSLg with my usual third-party X server, and try random things with the WSLg server (ie, I'd run an occasional DISPLAY=:0 xterm).  Again, I can't do that since it can only be disabled globally.

There are other legitimate uses of multiple $DISPLAY settings, for example redirecting to a different (possibly remote) display, working with a frame-buffer-based server (like Xvnc) and more.  Ignoring what users set in their environment is extremely bad for all of this.

I agree with @elibarzilay, independently of the use case if WSLg sees the DISPLAY already set it should not override it.
The same rule applies in the opposite, if inside the distro there is a script that sets DISPLAY for xserver it should always first check if it is already set (in this case by WSLg)

@elibarzilay, @crramirez , thank you very much for detail and valuable feedback. Current WSLg behavior (override DISPLAY) is indented. thus I removed "bug" tag and replace it with "enhancement". We will have internal discussion with team (and of course monitor "upvote" of this issue), thank you very much for trying out WSLg!

@hideyukn88, I hope that the title change would not derail the discussion -- since the main point is not just to coexist with another X server or partial features, but rather respect for (intentional) user settings, whatever they may be.

@elibarzilay, I understand that and, while I personally agree with you, but as a product we will need to maintain certain integrity as a whole solution.
As you might see, several X11 socket issue has been posted there and we have FAQ for that (https://github.com/microsoft/wslg/wiki/Diagnosing-"cannot-open-display"-type-issues-with-WSLg). Mainly this is because user (intentionally or unintentionally) customize it, or left over from other solutions cause conflicts, thus honor the user settings might not always give "just work" experience.
And current design of WSLg is to provide whole packaged solution (Wayland, X11, audio input/output and etc) or otherwise disable entirely. Thus I changed the title to something more reasonable to express the usage scenario for "why we might want to do this (= respects user settings)".
Thank you very much, we always welcome feedback!

IMHO considering we are talking about something already as techy as a Linux distro and issue, worrying about adhering to "it just works" might not be the right idea. Chances are people are at least savvy enough to know Google-fu or to know if they have mucked with envvars

@Masamune3210, Just to be sure, this only apply to WSLENV. Any env. can be still override by bash or .profile. WSLg don't override what configured in user-distro side.

And update the issue title to reflect the precise feedback from issue opener.

+1 to @Masamune3210's point -- WSL should generally be a linux experience, so following linux conventions is IMO more important than any notion of "just works".
Re WSLENV -- I think that the assumption that this is the only issue here is wrong, even if in practice that's the only way to do it currently.  I was actually surprised that wsl env goes first through a shell rather than running env directly, and therefore I won't be surprised if this is changed in the future to not go through your shell.
And as a sidenote, another reason to avoid relying on .profile is that different shells behave differently.  And in my case of zsh setup it uses different initialization files.

(I forgot to include earlier the existence of wsl --exec which won't use the shell for initialization.)

Hi @elibarzilay, thank you very much for your feedback. We've made a change to honor environment variables specified by WSLENV. The change needs to be made outside of WSLg project, thus I can't say for sure when it will be available, but it is coming. Thanks again for trying out WSLg!

@hideyukn88, thanks!

Since this issue has overwritten XDG_RUNTIME_DIR discussion in it, I have a question: Is it intended that XDG_RUNTIME_DIR is overwritten by WSLg? I ran this command
> echo $XDG_RUNTIME_DIR
/mnt/wslg/runtime-dir

and it seems the variable is set by WSLg. Is this needed for WSLg to work?

This is fixed in WSL from aka.ms/wslstorepage, thanks!

@mangkoran, yes, that has to be specified for Wayland compositor to work, thanks!


More error checking

[x] Domain and bounds checking
[x] Enforce that all agents have to belong to the cast

we haves some basic domain checking now


GetEmbeddedTemplateEditUrl with skipSignerRoles and skipSubjectMessage true

Hi,
HelloSignClient#getEmbeddedTemplateEditUrl(String templateId, boolean skipSignerRoles, boolean skipSubjectMessage); with both skipSignerRoles and skipSubjectMessage set to true doesn't seem to work. Using version 3.5.6. Should there be an & in the query parameters if both are true?
Currently running this as workaround:
public String getTemplateEditUrl(String templateId) throws HelloSignException {
EmbeddedResponse ep = client.getEmbeddedTemplateEditUrl(templateId);
return ep.getEditUrl().replace("&force_signer_input=1&force_edit_subject_message=1",
"&skip_signer_roles=1&skip_subject_message=1");
}
Thanks,
Matt

@mashleyttu Sounds like a bug! I'll schedule this for patching in the next update.

Hi @mashleyttu , apologies for the delay on this. We've identified the root cause of this bug and will be releasing a fix for it in an upcoming software update in the next few weeks.
In the meantime, you can tweak your work-around to simply remove the force_* URL parameters. The presence of those forces the signer and message prompts to display. Removing them will prevent them from being shown.

I'm going to close this ticket, as this is an issue on the HelloSign API side of things.


fix binskim bug for HighEntropyVACheck issue

bug title: [ VS-DotNet-ApiPort ][ BinSkim ] - Defect : HighEntropyVACheck, Component : \a\2.6.0-alpha.10\ApiPort\netcoreapp2.1\ApiPort.dll (1 issue)
Same issue for offline version as well.
Binskim error:
error BA2015: 'ApiPort.dll' does not declare itself as high entropy ASLR compatible. High entropy makes Address Space Layout Randomization more effective in mitigating memory corruption vulnerabilities. To resolve this issue, configure your tools to mark the program high entropy compatible; e.g. by supplying /HIGHENTROPYVA to the C or C++ linker command line. (This image was determined to have been properly compiled as /LARGEADDRESSAWARE.)

@marklio @twsouthwick bubble up the CR request.

I was wandering if it was enabled by default. However, the binskim bug auto opened by the build system. I also searched some other repos and found some with this setting specifically enabled.

Never had to use this, so not sure what the default is. But you'd want to put this in the Directory.Build.props so it's picked up by all projects

Never had to use this, so not sure what the default is. But you'd want to put this in the Directory.Build.props so it's picked up by all projects
Good point. Fixed

I know csc doesn't do it by default, but the targets should be setting it if you are targeting a supported framework: https://github.com/microsoft/msbuild/blob/a78f69dae3d599544e52b69b15e5bd71e27cd3b3/src/Tasks/Microsoft.NETFramework.CurrentVersion.props#L108

I'm looking into why this behavior isn't working.

Looks like this tried to get fixed with https://github.com/microsoft/msbuild/issues/2912, but the fix wasn't correct. It is now being tracked by https://github.com/microsoft/msbuild/issues/4566


tools: add check typedocs command

Edit: Correct references to jsdoc
Generates detailed and summary report of tsdoc and type issues related to tsdocs in Javascript files.  Output is to the console.
Running the following from the root of the project will produce a report.
(run with npx at beginning if typescript is not installed globally)
tsc src/index.js --allowJs --checkJs --noEmit --target ES6

This PR wraps this into a command in the package.json.  npx is not required here, it will run against the version of typescript installed by this project.
{
  "scripts": {
    "check:tsdocs": "tsc src/index.js --allowJs --checkJs --noEmit --target ES6",
  },
}

The same command can be added as a step in the github checks in the future.
(see ./github/workflows/ci.yml)
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

So, a few things:

This isn't so much a JSDoc thing but a mechanism to dump all problems reported by the TypeScript compiler to the command line.
We'll be dropping JSDoc in the (hopefully) not too distant future in preference for Typedoc output.
I'm not sure how we'd be able to hook this up to GitHub Actions since there's just so many errors at the moment.

There might some confusion there.  Title is supposed to be TypeDoc.
TSC combines the TSDoc/ JSDoc comments above functions and variables, with any typescript issues.  There's no typescript and this is only checking JS files so, all the issues are TSDoc / JSDoc issues.
The TSDoc reporter and vscode language server (autocomplete engine) are just running this in the background.

Corrected the name of the command. Old habits.

So, actually, running the typedocs script already output warnings/errors that affect the docs building process. Have you tried that out?

That's only showing 47 warnings.  It's focused on the final docs.  Missing issues in tsdoc headers.
Are those the priority ?  Want those addressed ?

Yeah, those are the ones that affect the actual doc output. For example, we need to figure out how to get callbacks represented correctly in the docs. There's a few other issues too. An issue is that fixing some of these things for Typedoc might break them for JSDoc.

Sounds good. I'll start there. Need the practice.


Fixes #120 - Inject empty JsonWebToken.

Ok, added a few more tests and rewrote the spec part a bit.


Fix misleading "pending" terminology

The "pending" term has been misleading people for several years, let's fix this once and for all, shall we?
GIF comparison:

Relevant forums thread: https://forum.nano.org/t/replacing-pending-terminology-once-and-for-all/329/25

Requires #62 or an alternative solution described in #62 that is quite more complex

all I can say is: yes, pull this.


Subscription not expiring

I'm subscribing through Iron Router's waitOn. I initiated an instance as:
App.Subs = new SubsManager(
  cacheLimit: 20
  expireIn: 1
)

Then did App.Subs.subscribe 'inventory' within waitOn but after I change the route and wait for 1 minute, documents are still there if I query them using the client console.
Any ideas? Thank you!

+1. Subscription still seems to be active in Meteor.default_connection._subscriptions too

@arunoda What are we missing here?

Can I have a sample case? may be a simple app?

+1 not working

+1
I experienced the same issue. I can give an example.
I have a list with a pagination:
---- Item 1 ----
---- Item 2 ----
---- Item 3 ----

< | 1 | 2 | 3 | >

Initially (being on page 1) I subscribe to the list collection with skip and limit param to get only results for the first page. Navigating to page 2 subscribes to the same publication but with different skip value.
Now let's count the documents (the result is as expected):
MyCollection.find().count() // => 6

Let's wait until the subscriptions are expired (after expireIn) and count again:
MyCollection.find().count() // => 6

So even when the subscriptions are expired all documents are still cached.
Going back to page 1 count() would still return 6.
Let's wait again until the subscription is expired. Now we navigate to page 3.
What happens?
MyCollection.find().count() // => 3

So only when subscribing with different parameters the old subscriptions are actually removed!
I hope that helps.
For the moment I need to increase expireIn to something high for not getting trouble when paging through the list.


Multiple LANGUAGE_ONLY tags

Multiple LANGUAGE_ONLY tags can be added to a cell.  If any match, the cell is included.
Seperately, the course tool is updated such that if FORCE_EDITOR is set in the config, then the config's editor will take precedence over the environment variable.

@bmc Ready for code review and pull.

Hey @bmc, ping on this PR.  It needs to be merged before I can commit my changes to courses, as the courses use this new feature.

@joshuacook I think Brian is really swamped, can you take a look at this pull request?  I think it's a pretty straight forward one, but I still want at least one other person on the tools project to see it first.  If you approve, I can merge it, but then I'll need your help to push out a docker update with it.  (My PR for courseware changes is blocked on first approving these build tool changes.)

How do we know it doesn't break anything?
On Fri, Nov 8, 2019 at 4:29 PM Doug Bateman<EMAIL_ADDRESS>wrote:

@joshuacook https://github.com/joshuacook I think Brian is really
swamped, can you take a look at this pull request? I think it's a pretty
straight forward one, but I still want at least one other person on the
tools project to see it first. If you approve, I can merge it, but then
I'll need your help to push out a docker update with it. (My PR for
courseware changes is blocked on first approving these build tool changes.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/databricks-edu/build-tooling/pull/1?email_source=notifications&email_token=AAM47U6HINT32BL5GJS5XBDQSX775A5CNFSM4JGAFEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDTYCVY#issuecomment-552042839,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM47U3UGJSETX23ZFLXQFDQSX775ANCNFSM4JGAFELQ
.

--
Joshua Cook
<EMAIL_ADDRESS>310-433-1646

Also worth noting: I tested things carefully.

Can you clarify?
On Fri, Nov 8, 2019 at 4:31 PM Doug Bateman<EMAIL_ADDRESS>wrote:

Also worth noting: I tested things carefully.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/databricks-edu/build-tooling/pull/1?email_source=notifications&email_token=AAM47U3YON3VWAMRA3FNIRTQSYAFRA5CNFSM4JGAFEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDTYEZY#issuecomment-552043111,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM47U6GSYOJS74B5VBCDZLQSYAFRANCNFSM4JGAFELQ
.

--
Joshua Cook
<EMAIL_ADDRESS>310-433-1646

I tested by building the Spark-ILT course and then diffing the built artifacts.  Plus the Spark-SQL ILT course I created for Amgen.

Merged. @dougbateman, can you update the Wiki (the one associated with this repo) with appropriate docs?
I'll make a note here when the docker images are update.


GSSoC'21: Add contact section in home page

Referring to the nav-links, the contact section is not mentioned and present on the homepage of the website. I request the creators to assign me the issue.

Can I solve this issue?

@himanshu007-creator please assign this issue to me. I would like to work on this.

https://himanshu007-creator.github.io/SFC-foundations/pages/Contact-Us/contact.html
already there


wrong nodeplugin tag version in ceph-csi-rbd chart 2.0.0

Describe the bug
Wrong tag 2.0.0 in the chart
https://github.com/ceph/ceph-csi/blob/v2.0.0/charts/ceph-csi-rbd/values.yaml#L107
It should be v2.0.0
https://quay.io/repository/cephcsi/cephcsi?tag=latest&tab=tags
Environment details

helm chart version
2.0.0
Kubernetes cluster version
1.16
Logs
Failed to pull image "quay.io/cephcsi/cephcsi:2.0.0": rpc error: code = Unknown desc = Error response from daemon: manifest for quay.io/cephcsi/cephcsi:2.0.0 not found: manifest unknown: manifest unknown

Steps to reproduce
Install the chart by default, the pods can't start because of the ImagePullBackOff error
Actual results
The pods can't start because of the ImagePullBackOff error
Expected behavior
The pods should start by default without chart customization

@topikachu this will be fixed in next release, can you please set the image tag to v2.0.0 while installing helm charts

@topikachu this will be fixed in next release, can you please set the image tag to v2.0.0 while installing helm charts

no pr. I install the plugin by a customized values.yaml.

fixed in https://github.com/ceph/ceph-csi/pull/869


UUID is not assigned when emitting a message to multiple nodes

When a message is sent to a single node, it is assigned a UUID. From red/nodes/Node.js#99:
if (!msg._messageUuid) {
   msg._messageUuid = constructUniqueIdentifier();
}

However, when there are multiple output wires, line#144:
if (msgs[k]._messageUuid === null) {
    msgs[k]._messageUuid = constructUniqueIdentifier();
}

Unless I am mistaken, the condition msgs[k]._messageUuid === null is never true. Should this be !msg._messageUuid instead?
This bug causes the message to be logged with msguuid == undefined.
The message is assigned a uuid upon receive, Node.js#168:
if (!msg._messageUuid) {
    msg._messageUuid = constructUniqueIdentifier();
}

However, at this point, the message has already been logged.
I have raised a pull request with a fix and testcase: #552 .

Hi,
thanks for raising this.
This code is under active development at the moment (literally, I'm working on it right now) and subject to change.
As per our contribution guidelines, we cannot accept pull requests without a signed CLA in place.
These two facts combined means, I'll make sure this gets fixed as part of the work I'm doing at the moment, but I won't be able to accept your PR.
Thanks,
Nick

I believe I've put a fix for this in the latest pull request for #546.


[SPARK-40993][SPARK-41705][CONNECT] Move Spark Connect documentation and script to dev/ and Python documentation

What changes were proposed in this pull request?
This PR takes over https://github.com/apache/spark/pull/39211 and https://github.com/apache/spark/pull/38477 that proposes:

Move connector/connect/dev/generate_protos.sh → dev/generate_protos.sh to be consistent with other places
Move Python-specific development guides into python/docs/source/development/testing.rst

Why are the changes needed?
To keep the project structure and documentation consistent.
Does this PR introduce any user-facing change?
Python-specific development guides for Spark Connect will be added in https://spark.apache.org/docs/latest/api/python/development/testing.html.
How was this patch tested?
I manually tested:
./dev/generate_protos.sh
./dev/check-codegen-python.py

I also manually verified the Python documentation.

cc @grundprinzip @amaliujia @zhengruifeng @tedyu FYI

Maybe at least prefix the scripts with connect so that it's clearer where they belong to and not just a random code dump?

SGTM. let me rename them

Merged to master.


fix: set_checkbox() checks unchecked checkbox

Description
Fixes #7814
set_checkbox() with default true checks unchecked checkbox when redirected back with input.
Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Hoping to hear from @wrestlingpotatoes

Hoping to hear from @wrestlingpotato

The fix works great!


Show response time in milliseconds per request and total

This PR shows the response time in milliseconds per request and total:

I added a PR to use axios interceptors: https://github.com/usebruno/bruno/pull/397


Keyboard focus issue

Steps to reproduce
1- click in any input field
2- click outside of input filed without click done in keyboard
3- click in avatar for showModalBottomSheet and click outside will keyboard open and focus last field why?
Expected results
Keyboard display auto for click any inkwell or button if i don't click done in keyboard
Actual results
keyboard not open if i don't back to click any field
Code sample
import 'package:flutter/material.dart';
void main() {
runApp(const MyApp());
}
class MyApp extends StatelessWidget {
const MyApp({super.key});
// This widget is the root of your application.
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Flutter Demo',
theme: ThemeData(
colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
useMaterial3: true,
),
home: const MyHomePage(title: 'Flutter Demo Home Page'),
);
}
}
class MyHomePage extends StatefulWidget {
const MyHomePage({super.key, required this.title});
final String title;
@override
State createState() => _MyHomePageState();
}
class _MyHomePageState extends State {
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
backgroundColor: Theme.of(context).colorScheme.inversePrimary,
title: Text(widget.title),
),
body: Center(
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: [
InkWell(
onTap: () {
showSelectMediaBottomSheet(context);
},
child: const CircleAvatar(
radius: 50.0,
backgroundColor: Colors.grey,
),
),
const SizedBox(
height: 20.0,
),
Form(
child: Column(
children: [
TextField(
decoration: const InputDecoration(
border: OutlineInputBorder(),
labelText: 'Email',
),
onTapOutside: (event) => FocusScope.of(context).unfocus(),
),
TextField(
decoration: const InputDecoration(
border: OutlineInputBorder(),
labelText: 'Password',
),
onTapOutside: (event) => FocusScope.of(context).unfocus(),
),
],
))
],
),
),
);
}
}
void showSelectMediaBottomSheet(BuildContext context) {
showModalBottomSheet(
context: context,
shape: const RoundedRectangleBorder(
borderRadius: BorderRadius.vertical(top: Radius.circular(16)),
),
builder: (BuildContext context) {
return SizedBox(
height: 150, // Adjust the height as needed
child: Column(
children: [
ListTile(
leading: const Icon(Icons.camera_alt),
title: const Text('Camera'),
onTap: () {
// Handle camera selection
Navigator.pop(context);
// Call the function to open the camera
},
),
ListTile(
leading: const Icon(Icons.photo),
title: const Text('Gallery'),
onTap: () {
// Handle gallery selection
Navigator.pop(context);
// Call the function to open the gallery
},
),
],
),
);
},
);
}
Screenshots or Video
https://github.com/user-attachments/assets/522527bf-b5b1-4c78-8b4a-55baac15f5cd
Logs
No response
Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.3, on macOS 14.6.1 23G93 darwin-arm64, locale en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.2)
[✓] VS Code (version 1.94.2)
[✓] Connected device (4 available)
[✓] Network resources
• No issues found!

https://github.com/user-attachments/assets/e9fb3de6-1fd7-4d6c-9156-950502ec6139
video issue so uploaded new video if click done in keyboard all of things is correct if don't and click outside not work correctly

When you trigger showModalBottomSheet, it will open a new route. And it comes with isDismissible as true by default, which means the model is dismissed when tapping outside. This is actually calling Navigator.pop method. If you use Navigator.push to a new page instead when tapping on the avatar, you will also see the same issue.
This is a known issue at https://github.com/flutter/flutter/issues/145155 (or https://github.com/flutter/flutter/issues/48464). You can find a workaround at https://github.com/flutter/flutter/issues/145155#issuecomment-2203536461.
Closing this as a duplicate of these issue.


Remove the "simple" router

After migrating to IonRouter in my apps, everything seems to behave normally (There is doubling of animation after vue HMR, but that's a minor issue).
Based on that, I'm not sure we need the "simple" router at all anymore.

Yeah,.. maybe we can keep it for the more complex implementation with solid router? Like Ionic use react router. IDK if it's worthy.

Removed for now.


app.py gives error

I have cloned the entire repo, installed my librarires from the requirements.txt.
The article in high-level steps. Step number 4. Run app.py.
It throws an error as below
ValueError: No valid connection settings.

Okay. I have resolved this.
It's because in the article it says that I need to set env variable as HOST_NAME as opposed to SERVER_HOSTNAME as needed by the constant.py file.
Maybe make that correction either in the article or at the code.

Thanks for raising @dejokz! This should be fixed in the blog post now.


Update Function

How does this option in ./install really works? will it update whenever a new release has been committed?

Yes this will get the last version!

the database did not update. I updated the database manually.


Fixed error RC1107 when consumers compile their resource file.

Hit this error while trying to build:
2>RC : fatal error RC1107: invalid usage; use RC /? for Help
Caused by the argument /I"...\packages\Microsoft.AspNet.SignalR.Client.Cpp.v140.WinDesktop.1.0.0-beta1\build\native\include\" passed to rc.exe. It turned out that it doesn't like the trailing backslash.

Hi @tn0502, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

How are you trying to build the project? I have never seen this...

I'm building a NuGet package using an internal build system and the version of rc.exe being used here didn't like the trailing backslash.

You mean the part that says: "Solution: Add a slash to the last include path will do the trick. If your last include path already contain a slash at the end, delete it will also work."
It's completely contradictory. Also, I don't think paths should have a trailing backslash unless they're meant to be used as prefixes, e.g. $(SolutionDir).
My rc.exe is really simple as I have only one dependency, SignalR. I tried removing other parameters but only removing the trailing backslash worked.

Thanks!


⚠ Bofrost.es has degraded performance

In a0a7f70, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1583 ms

Resolved: Bofrost.es performance has improved in 198e505 after 6 minutes.


[Epic][Pdr bot] New smart contract, so user needs just one tx for 2-sided predictions across >1 feeds

Build a smart contract to be able to submit two sided predictions to multiple feeds in a single tx.
TODO:

[x] Write the smart contract
[x] Write tests for the smart contract
[x] Add PredictionManager module into pdr-backend
[x] Add a command into pdr-backend to deploy the manager contract.
[x] Add tests for PredictionManager module
[x] Update Predictoor PP to support "train_on", "feeds" structure for feeds.
[x] Implement PredictionManager into predictoor agent

Flow:

if doesn't exist, pdr-backend deploys the prediction manager contract using PRIVATE_KEY and saves the address of the deployed contract in a file.
pdr-backend interacts with the smart contract to submit predictions and claim payouts.

Appendix: Background, motivation, details
(From slack conversation between Berkay & Trent 20240229)
Berkay:
Some predictoors use smart contracts, submitting 20 predictions in a single tx.

Pros: no risk of losing stake due to failed tx, single tx to submit all, single predictoor private key.
Cons: predictoors gotta deploy & manage a smart contract

Trent: That's cool! Where did you hear about this?
Berkay: I inspected the top addresses in autobotocean
Trent: We could even consider making this easy for people to do, ie deploy a contract that does predictions across 20 feeds, two-sided
Berkay:
Probably yes! we can abstract the complexity. Users would have to not lose their private key and the deployed contract address, and input them to pdr-backend. However the predictoors I've seen were 50-50s, is it feasible to compute the prediction for 10 feeds in a single pdr-backend process?
Contract interface would look like:

submit([]contracts, []stakesup, []stakesdown, epoch): submit predictions for given epoch to contracts
payout([]contracts, []epochs): claim payout and send funds back to owner

Trent:
Yes, exactly!
Perhaps make a github issue and we can then decide when to do.
If you want to do it sooner rather than later, go for it
Related : if that existed then we could easily modify predictoor_agent.py to work across >1 feeds.
Berkay:
[Issue created!]
It's a huge friction remover and ensures stakes submitted on each side.
But there's a challenge, calculating prediction for multiple pairs before the epoch closes. We could leverage threading maybe to run the calculations in parallel. Do you think this could be a problem?
Should we ensure this is working before proceeding with 701?
Trent:
Re speed: It comes down to how quickly a model can  be built and predicted from. Right now they're very simple, so very fast. We might even be able to get away with a single thread.
A single thread would be the obvious place to start. Then we can optimize later, if we need.
Berkay:
Sounds good!
Current version of predictoor is not bad, I've been using it with our predictoors, caused some stake to be lost in the beginning but didn't have any problems since I set the cut off seconds to 40. But yes, setting up k8s is still a lot of friction.
I can start working on the smart contract solution if you approve, right after I'm done with the current tasks. or I can take another task on the ml side
Trent:
Sure do go ahead with the smart contract solution.
BTW I think should keep the K8S functionality too. Anyone doing fancier modeling will need it.

cc @jfdelgad

@trizin fyi Jaime built something like this back in December. He's now moving to work on it again. You guys should sync

Congrats!! Great work


Update UI

Repositioned Comment and Status

Not too sure about this colour scheme, looks a little weird... but I can change it back if anything.

Fixed inconsistent button sizes.
Do note that this is conflicting with @lawwm implementation at #103, so we need to make a decision as to whether we want the caching of comments
@fantablack @lawwm @junlee1991 @Btaykb
~Do give feedback regarding the above.~
Updated colours:

We'll be using the updated layout and removing the cache function. Thanks for the feedback.
Closes #110

I think the caching comments makes  it easy for it to be overlooked by users.

I feel that the blue color scheme for the AddWindow/EditWindow can be a little too clashing when the base color of the main page is still black/grey. We can probably look into changing the color scheme of the main page to make it more complementary to blue.

I feel that the blue color scheme for the AddWindow/EditWindow can be a little too clashing when the base color of the main page is still black/grey. We can probably look into changing the color scheme of the main page to make it more complementary to blue.

Good point. I am actually looking into those and I do intend to change the colour scheme for the main page!

Do note that this is conflicting with @lawwm implementation at #103, so we need to make a decision as to whether we want the caching of comments

For this matter, I understand @lawwm point of view that in the case of adding multiple similar comments, having a cache would allow for greater speed and convenience than manually typing out the same comment repeatedly. But I feel that this case may not be common enough to warrant another table for comments specifically.
Also users might be confused with the separate table and while they could just simply read up the UG to understand its purpose and value, I feel when faced with the decision of choosing between a design than can be more easily understood at first glance vs having to search it up in the UG, we should perhaps be looking into the former.

Will merge and close tomorrow if no other issues.

Will merge and close tomorrow if no other issues.

LGTM