[Question] Search not working after using setTransformer

Summary of problem or feature request
Hello, I'm relatively new to laravel and laravel-datatables. I'm using setTransformer to edit my data. But the search function wont work at all.
Code snippet of problem
Here is my ajax code
public function fetchCasesAjax($steam64){
        $cases = CaseModel::where('staffSteam64', $steam64);

        return Datatables::of($cases)
        ->setTransformer(function($case){
            return [
                'id' => $case->id,
                'timestamp' => $case->timestamp,
                'staffStam64' => $case->staffSteam64,
                'overwatchSteam64' => $case->overwatchSteam64,
                'playerSteam64' => $case->playerSteam64,
                'overwatchName' => $this->getStaffNames($case->overwatchSteam64),
                'playerName' => $this->getPlayerNames($case->playerSteam64),
                'type' => $case->type,
                'notes' => $case->notes,
            ];
        })
        ->make();
    }

overwatchName and playerName do not exist in the database table.  $this->getStaffNames() and $this->getPlayerNames() returns a string.
Here is my datatables ajax code
$(function() {
            $('#table').DataTable({
                processing: true,
                serverSide: true,
                ajax: '{{route("fetchCases", $steam64)}}',
                pageLength: 15,
                lengthMenu: [10, 15, 20, 50, 100],
                "language": {
                    "loadingRecords": "Loading...",
                    "processing": '

Loading

'
                },
                order: [[ 0, "desc" ]],
                deferRender: true,
                columns: [
                    {data: 'timestamp', name: 'timestamp'},
                    {data: 'playerName', name: 'playerName'},
                    {data: 'type', name: 'type'},
                    {data: 'notes', name: 'notes'},
                    {data: 'overwatchName', name: 'overwatchName'},
                    {data: 'id', name: 'edit', searchable: false, sortable: false, width: "184px",
                        "render": function(data, type, row, meta){
                            if(type === 'display'){
                                data = '<button class="ui inverted green button tiny">Edit</button><button class="ui inverted red button tiny">Delete</button>';
                            }
                            return data;
                        }
                    },
                    {data: 'playerSteam64', name: 'playerSteam64', visible: false},
                    {data: 'overwatchSteam64', name: 'overwatchSteam64', visible: false}
                ]
            });
        });

System details

Operating System => Windows 10
PHP Version => 7.4.3
Laravel Version => 7.22.0
Laravel-Datatables Version => ~9.0

You should disable search and sort on those columns since they are computed values. If you want them to be searchable / orderable, you would to implement a custom sql that will handle them. See https://yajrabox.com/docs/laravel-datatables/master/filter-column for creating your own custom filter.

So I tried doing filterColumn for both playerName and overwatchName.
It worked perfectly fine for playerName but when I tried doing it for overwatchName it threw me an error.
This is the error I got when playerName searchable was set to false and only had a filterColumn for overwatchName
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ') or LOWER(`cases`.`playerSteam64`) LIKE ? or LOWER(`cases`.`overwatchSteam64`) ' at line 1 (SQL: select count(*) as aggregate from (select '1' as `row_count` from `cases` where `staffSteam64` =<PHONE_NUMBER>4066675 and (LOWER(`cases`.`timestamp`) LIKE %1% or LOWER(`cases`.`type`) LIKE %1% or LOWER(`cases`.`notes`) LIKE %1% or () or LOWER(`cases`.`playerSteam64`) LIKE %1% or LOWER(`cases`.`overwatchSteam64`) LIKE %1%)) count_row_table)

Here is the filterColumn I added for overwatchName
        ->filterColumn('overwatchName', function($query, $keyword){
            $query->whereRaw($this->getOverwatchSteamIdQuery($keyword));
        })

Here is getOverwatchSteamIdQuery (getPlayerSteamIdQuery is identical)
    public function getOverwatchSteamIdQuery($name){
        $query = User::select(['steam64'])->where('name', 'like', '%' . $name . '%')->get();
        $sql = '';
        if($query == null){
            return 'cases.overwatchSteam64 like \'%' . $name . '%\'';
        }
        else{
            foreach($query as $key => $value){
                if(empty($sql)){
                    $sql .= 'cases.overwatchSteam64 like \'%' . $value->steam64 . '%\'';
                }
                else{
                    $sql .= ' OR cases.overwatchSteam64 like \'%' . $value->steam64 . '%\'';
                }
            }
        }
        return $sql;
    }

Thanks! Fixed the issue.


[Docs / Drilldown Menu] Option value shows rendered HTML

See https://foundation.zurb.com/sites/docs/drilldown-menu.html
Table "Plugin Options" | Row "data-back-button" | Column "Default"

Back

Expected Behavior
HTML source code should be displayed.

Back

Current Behavior
Rendered HTML source code is displayed.
Back

Fixed in https://github.com/zurb/foundation-docs/pull/36


Do you need help with maintenance?

It appears that this library contains some super-critical bugs such as #51, though PRs take a long time to get merged. Do you need help with maintaining this library?

Apologies, we've been very busy! @wpercy will be leading maintenance of vobject and we are starting to get these critical issues fixed right now!

👍

Thanks for the updates! Any chance of a new release with the recent set of fixes?

Yep, I'm hoping to be able to put out a new release late this week/early next week - trying to fix these pesky unicode bugs now.


MSC1849: Proposal for m.relates_to aggregations

Aggregations MSC for edits & reactions to replace #441
Rendered

There is some related prior art here. Mjark and I thought about how to do updates vs relations back in 2015 - https://github.com/matrix-org/matrix-doc/blob/5795e1cedaa001994847702193bd02e517e07c83/drafts/general_api.rst#general-client-changes
Some combination of the two may work for aggregating relations.

@Kegsay indeed - i had not forgotten it, and we should definitely take inspiration from it.
In other news: we should perhaps consider extending redaction API to say "redact this event" or "redact all related events".

Are we still calling this WIP, btw?

Some feedback which needs to be incorporated somewhere:

This doesn't give us an easy way to stop someone from being able to send reactions in a room. (You'd have to split the PL for sending default events from the one from sending m.reaction events, and then either promote or demote users appropriately, which is a major pain, unless we start saying something like "by default PL to speak is 0, and PL to send reactions is 10, but we now set users to PL 10 by default".  Having a distinction between normal users and 'trusted' normal users (a bit like +r on IRC) feels like it could be useful in the PL pantheon.
This doesn't let us distinguish power levels between users who are allowed to add new reactions versus users who are allowed to +1 existing reactions.  Apparently Discord provides this as an option, which is useful for mitigating abuse, and would also be needed for accurately bridging to Discord.

@jryans did you propose how to tell if an aggregation key is encrypted?
if not, how about:
"alg": "A256CTR" which then matches how we encrypt attachments for E2E.  Given the aggregation keys are kindasorta mini encrypted attachments, this doesn't seem unreasonable.
Someone needs to doublecheck whether CTR is the right mode, however, and if so what we do about IVs and hashes etc.

@jryans did you propose how to tell if an aggregation key is encrypted?

I added a comment on the MSC just to record that we want some kind of metadata, but no specific proposal. I'll copy your idea there to continue the thread.

Please bear in mind that I'm posting this comment with my community member hat on rather than my NV employee one
While I haven't seen it stated explicitly in the proposal (I might have missed it though), from what I heard in various exchanges, including https://github.com/vector-im/riot-web/issues/9793, the way edits are going to be implemented specifically allows admins and mods in rooms to edit anyone's messages in rooms, which I see as a massive red flag and a huge ethical concern in a protocol that's currently mainly used for IM.
I've already discussed my PoV on this in the linked issue, but thought I should also make my point clear in this timeline in order to keep the discussion around aggregations in one place.
I see this specific aspect of edits as a huge risk, as imho it's easy to abuse, and abuse would cause confusion in the best case scenario, impersonation with potentially harmful consequences for the victim in the worst case scenario. As a user, I feel like the risk it makes me take, in terms of potential disinformation and how I might act upon it without realising the fake, to even read messages as edited (rather than see the edits as individual messages) exceeds by a lot what the positive aspect of the feature. And I find it very intrusive to even allow other users to edit my own messages without my consent.
I do see the point that it might be a useful tool for moderation, but it really sounds to me like it brings little to the table, nor can I see the absolute need for it given we already have a redaction feature. I'm waaaay more comfortable with mods deleting my messages rather than putting something I don't think or agree with under my name.
On a UX/UI point of view, I'd tend to think that this aspect adds a hard requirement for a perfectly explicit UX that allows users to figure out from a glance, and without additional actions, who authored which part of a message as easily and as straightforwardly as figuring out who authored a message, and this on every client that has at least one user. This doesn't sound a realistic goal to me.
A small and brief text at the end of the message like the "(edited)" we have now is not enough, imho, as I myself often misses it when reading messages from others, and I'm almost sure that non-technical users would either miss it as well most of the time, or not know what it means and decide not to care (my dad would definitely be in either of those 100% of the time). Plus, it turns an informational metadata that you don't have to process if you don't care into an information that'd be presented as non-important but is in fact as important as the message and its context if you want to understand both.
As to the argument "some platforms such as some forums and github do it", I'd answer that it doesn't make it a good thing to have (and some chose not to have it, such as Reddit afaik). I already find it pretty intrusive and badly implemented there, where the use of such a feature is more justified than IM (e.g. GitHub's target audience is used to anything being editable by a bunch of people and processing changes histories).
Also, as a user, I really feel like that, with this rolled out everywhere, Matrix, something that got my interest because it gives me control over my communications, takes a big chunk of this control back without me being able to do anything about it, and it makes me sad. And will probably prevent me from making announcements, or even speaking, in rooms in which I don't know each mod and admin to some extent.
I'm currently chatting with (h)activists and journalists (and some other people with interest in tech and/or privacy) to try to get them interested in Matrix, but this element being added to Matrix would probably greatly hamper these efforts, because I can't genuinely promote a platform which allows disinformation and impersonation by design.
In summary, to me this aspect of message editing is both risky and unnecessary, and will likely keep me away from message editing as long as it's a thing, and that makes me sad (especially since I was waiting on such a feature for a long time). Someone abusing it would be able to make me say "I like nazis" in a way that would look very convincing and legitimate, all without my consent (and possibly without even me noticing), and most non-technical people would just think I like nazis and possibly hate me for it. I like a message editing feature in which I'm the only one that can edit my messages and noone else, and if a mod has an issue with it, they can talk to me or just delete my message.

We might want to include https://github.com/vector-im/riot-web/issues/10203 in the spec as well.

I think we might need a way to preserve the rel_type upon redacting an edit. As it stands we can't tell a redacted edit from a redacted message, causing riot to render redacted edits in the timeline. Not sure where we could put it though, perhaps redacted_rel_type in unsigned?

right, i've merged the rewrite branch into the main branch at last. hopefully it incorporates all the review from both the old and rewritten versions of the MSC.
@erikjohnston there are a bunch of XXXs remaining here which need your input before we can get this thing finally FCP'd.
@anoadragon453 there is also the question over whether https://github.com/matrix-org/synapse/pull/5626/files is actually the right solution to the problem (as per https://github.com/matrix-org/matrix-doc/pull/2154/#discussion_r308473720)
@bwindels i need you to explain to me why it's useful for the bundled version of an m.replace includes the origin_server_ts and sender of who edited it. Surely this is on the timestamp & sender of the replacement event itself...

@bwindels i need you to explain to me why it's useful for the bundled version of an m.replace includes the origin_server_ts and sender of who edited it. Surely this is on the timestamp & sender of the replacement event itself...

The origin_server_ts is useful to continue aggregate locally from sync on top of bundled m.replace relations. We also need it, and the sender, for showing the Edited at hh:ss (by sender) tooltip.

I've split this up into #2674 (general event shape), #2675 (serverside aggregations), #2676 (edits), and #2677 (reactions), and created #2678 as an open issue for encryption.

This one should probably be closed, then, and the original post updated?


🎉 release: v7.0.1

This is a hotfix PR.

:tada: This PR is included in version 7.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Docu update for SDKization

Bundle all docu update related to the SDKization refactoring and related (e.g., the removal of the explicit docker switcheroo) to after the corresponding code changes are done

[ ] ECC refactoring #69

[ ] remove switcheroo #97

Fix also Time to get the code section.
$ git clone https://github.com/hyperledger-labs/fabric-private-chaincode.git $GOPATH/src/hyperledger-labs/fabric-private-chaincode

$ git clone https://github.com/hyperledger-labs/fabric-private-chaincode.git $GOPATH/src/github.com/hyperledger-labs/fabric-private-chaincode

Consolidate READMEs. With the new build system we don't need extra README in each component folder that explains how to build the component. I suggest that we combine them in a single README and only provide extra information if needed.

content for new section FAQ:

sudo make solves issues with misconfigured docker environments but changes $GOPATH

https://docs.docker.com/install/linux/linux-postinstall/
https://stackoverflow.com/questions/55843277/cant-run-go-program-using-sudo-command

Maybe move "Behind a proxy" to FAQ section or closer to IAS section

Add:    sudo snap install --stable --classic protobuf
in Get NanoPB section

we should  also a simple software development section on the FPC programming model (shim.h et al ...)

we should also a simple software development section on the FPC programming model (shim.h et al ...)
We definitely also should document that folks have to use the wrapper scripts from fabric/bin and point out the utility scripts fabric/bin/ledger_init & fabric/bin/ledger_shutdown if users want to interactively play with FPC ..

Let's address this briefly in #129 and provide concise documentation on the programming model later.

addressed mostly in PR #142 (issue #129) and PR #137 (with increased docu inside shim.h)


Render offline buffer to a static file

I'm creating a composer and I'm currently investigating on offline rendering the result to a file.
I've tried using the recorder but it records the audio not in real time, but in offline time, making and ultra short and fast audio file.
I've also searched for this online but I'm yet to find a solution that works with offline toneJs.
I tried audioBufferToWav to make a blob of type: 'audio/wav' with the audioBuffer generated with Offline to pass createObjectURL but this generates a wav file with correct length but filled with silence.
I also tried to make a blob out of the ToneAudioBuffer but I haven't had any luck with this as well.
In the documentation of Offline and ToneAudioBuffer  there is no explanation of what to do with the context but to log it.
Some of my code:
Code where I create the buffer
async createMP3(notes: INote[]) {
const recorder = new Tone.Recorder();
const buffer = await Tone.Offline(({ transport }) => {
const instrument = this.currentSampler!;
instrument.connect(recorder);
recorder.start();
const song = new Tone.Part((time, note) => {
instrument.triggerAttackRelease(note, '1860i', time);
}, this.songToTonejs(notes)).start(0);
transport.schedule((time) => {
song;
}, 1);
// make sure to start the transport
transport.start(0);
}, this.getSongLenght(notes));
return buffer;
Code where I create the wav file
async exportMp3(){
const mp3 = await this.playerService.createMP3(this.composerState.currentVersion.notes)
const wav = audioBufferToWav(mp3.get()!)
const blob = new Blob([wav], {type: 'audio/wav'});
console.log(mp3,blob,wav)
const url = await URL.createObjectURL(blob);
const anchor = document.createElement("a");
anchor.download = "recording.wav";
anchor.href = url;
anchor.click();
The code above generates a valid .wav file with no sound.
Sorry about the styling, I'm not really sure how to make it look nice here on GH.
Does anybody has experiece with this issue?

Looks like you're mixing up Tone.Recorder and Tone.Offline. Recorder is for recording the realtime output of an online context. Offline is for silently rendering a precomposed piece of sound faster than realtime. It sounds like you want to do the latter, in which case the reason you're getting a silent wav is because you're connecting instrument to recorder (which is useless in an offline context) instead of to the offline context's own destination. Try instrument.toDestination() and remove the Recorder stuff and it should work.

You were correct. I could connect my PolySynth to the offline context and it worked correctly :)
I'm still working out making this work with the sampler, but those are my problems. Thank you for your help!


Syntax Highlighting for Editors

I'm using CommonMark.NET for a Windows based Markdown editor. (https://github.com/mike-ward/Markdown-Edit). Currently, I'm using regex's for syntax highlighting source (not rendered output, but source), which has lots of limitations due to the block nature of the language.
So..., I was thinking, it might be very cool to generate text segments with metadata for use with source highlighting. This could be used to correctly highlight source markdown. Is this something you would be interested in collaborating on?

This wold be very usefull.
I am trying to replace the custom parser(regex based) with CommonMark.Net in WebEssential 2013.
Unfortunately i am having exactly this problem.
If i have time and try to implements a test suite would you be interested in a pull request?
Thanks.

Yes, it would be really great if you could create these tests.
azu/markdown-to-ast might prove useful - they seem to have implemented this in JS. Though I don't see any ready tests in their repository that we could borrow.

I have taken a look at https://github.com/azu/markdown-to-ast.
It does seems they compute the position from the column and line.
I was thinking that probably a better solution would be to simply track the position during parsing (obviously it need to handle tabs correctly).
I will try to put together a test suite anyway.

I have started playing with this. First results (handling only block indentation) are promising.
One thing I added while doing that was the AsEnumerable() method that is now in master since it provides an easy way of testing the results.
Another issue is that I need to refactor at least MatchEmphasisStack and MatchTildeStack since the code there overlap significantly and they are the most complex from inline perspective to get the correct start/end positions.

Cool!
I am a little busy at the moment preparing the pull request for https://github.com/madskristensen/WebEssentials2013/issues/1732.
When i am done with that i will switch to this issue and see if i can help (but you seem to have tackled this issue very fast on your own :) )

Question - is it ok if the positions tracked will include the delimeters? E.g., for an inline code element, the substring represented by the positions will be
``var i = 1;``

so if you need to configure some syntax highlighting for the contents, you would need to do additional parsing to remove the wrapping backticks. The logic to do so is very simple though.
Although it would be no problem to store both positions, I don't really want to since that would increase memory used and thus probably impact performance.

If i read correctly the specification three cases are possible based on how many " ` " are used in the source markdown?
`code`
``code``
```code```

The general idea is that for the purpose of integrating the library in any text editor designed to keep memory pressure low (avoid string allocations) the only requirement is to be able to obtain a series of TextRange(RangeStart, RangeEnd | RangeLength) associated to each AST node.
Each node can be then mapped to a CodeArtifact that categorize the TextRange for some language or style.
The AST could even not contain any string content at all (it could be retrieved by splicing the source buffer based on the positional information in each TextRange. Visual studio expose the text editor in the extensibility model exactly this way)

No, the number of ` characters is not limited, what you would need to do is to reduce the length and increase the start index as long as the first and last characters of the block are `.

The strange thing is that I actually have managed to implement the source position mapping (in the sourcepos branch). For example, for input strings
\0\0   **foo**\r\n\t\0bar \n\n> *quoting **is nice***, `right`?
* animals\r\n* * *horses*\n* * dogs\n* persons\r* 1. john\n* 2. anna

All inline elements are parsed with correct position and length... Removal or \0, different newlines (single or double character), tabs, block indentation - all these are handled.
Currently implemented tests
What is left to do

add tracking to remaining inline element types (currently only the ones in the sample above are done).
check which cases do additional manipulation with Block.StringContent that might not be tracked.
currently one test with multiple tabs is failing.

Question: would you need this information also for block elements or is the StartLine/EndLine enough for practical purposes?

The current web essentials parser has the following classification

Bold
Italic
Header
Quote
Code

Fenced
Inline
Indented

    public static class MarkdownClassificationTypes
    {
        public const string MarkdownBold = "md_bold";
        public const string MarkdownItalic = "md_italic";
        public const string MarkdownHeader = "md_header";
        public const string MarkdownCode = "md_code";
        public const string MarkdownQuote = "md_quote";

        [Export, Name(MarkdownClassificationTypes.MarkdownBold)]
        public static ClassificationTypeDefinition MarkdownClassificationBold { get; set; }

        [Export, Name(MarkdownClassificationTypes.MarkdownItalic)]
        public static ClassificationTypeDefinition MarkdownClassificationItalic { get; set; }

        [Export, Name(MarkdownClassificationTypes.MarkdownHeader)]
        public static ClassificationTypeDefinition MarkdownClassificationHeader { get; set; }

        [Export, Name(MarkdownClassificationTypes.MarkdownCode)]
        public static ClassificationTypeDefinition MarkdownClassificationCode { get; set; }

        [Export, Name(MarkdownClassificationTypes.MarkdownQuote)]
        public static ClassificationTypeDefinition MarkdownClassificationQuote { get; set; }
    }

At the moment only this types are highlighted in the text editor when editing a markdown file.
To have feature parity with the current implementation, i suppose handling only this cases would do.
But i think that implementing this in a more general purpose would give more value to CommonMark.Net.

OK, I believe the inline source position tracking is now implemented.
I would be very grateful if someone could get the sourcepos branch and play around a bit before I merge it into master.

I will take a look at it tomorrow.
Thanks.

Sorry,
i don't think i will have the time to try this out until this weekend(too much boring work stuff :)).

No problem :)
All of my own test cases currently pass. I also ended up implementing additional performance optimizations so in the end sourcepos branch is even faster than master - the position tracking (not counting the adjusting for tabs/indents) is just a few addition and assignment operations so they by themselves don't do much damage to the performance.
As soon as you will give the green light (just so that someone else has tried it), I will merge the changes and also push a new nuget package.

I am a bit lazy and not sure if it will be useful so for now I will not add SourcePosition and SourceLength to the Block class. I did however, add them to FencedCodeData class since that is the specific feature required in Web Essentials.
Oh... And it also needs quotes... Well, I'll see what I can do... :smile:

I have looked at the code and done some testing.
All seem very good.
Having the SourcePosition and SourceLength in the block class could be useful to do a visual matching of block boundaries in the editor without knowing the code block type(fenced, inline, etc...), but i can try to implement it myself later and create a pull request if you don't have time (the same goes for quotes).
I can now try to implement the mapping between the AST and the model used by the visual studio editor and replace to the custom one implemented in web essential :).
I hope to have something ready to show next week.
Thanks,
you have done an outstanding job (and very fast too!).

Commit https://github.com/Knagis/CommonMark.NET/commit/cc0e1b3b05ce39b5122ef1b567e750666d1e7663 now includes tracking of SourcePosition/SourceLength for all block elements as well. I removed the properties from fenced code data object since the common properties should be used. The difference is though that the block properties will include the fences themselves - you would have to trim them yourself to get the boundaries of the contents.
By the way, the current result would allow you to enable syntax coloring for indented code blocks as well, if you allow the user to specify "default language" in the settings.
I also marked StartLine/StartColumn/EndLine as obsolete. I think that the new approach is much more flexible since the line number is usable only in very simple cases since even word wrapping could make it unusable.

Ok.
I have done some preliminary refactory on the WebEssential codebase to remove the current parser implementation and replace it with CommonMark.Net.
After i have something working i will update this issue or, if you want to close this one, open another to report any additional issue (if any).
Thanks


ExchangeOnlineMangement module authentication window opening under Windows PowerShell Terminal

Windows Terminal version
1.11.2921.0
Windows build number
10.0.22000.0
Other Software
Windows PowerShell - 5.1.22000.282
ExchangeOnlineManagement - 2.0.5
Steps to reproduce
Open Windows PowerShell tab in Windows Terminal
Run commands:
Import-Module -Name ExchangeOnlineManagement
Connect-ExchangeOnline

Wait a while before you realise the authentication prompt opened under Terminal, then switch to it and move on with your life
Expected Behavior
The authentication pop-up should, well, pop up.  It should open on top of Terminal, not under it.
Actual Behavior
It behaves like a pop-under, and it's easy to miss that it's already opened but underneath.

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #2988 for more discussion.
/dup #2988

Hi @zadjii-msft , sorry for the dupe, I did search for it but obviously my search-fu is not up to scratch.  I'll track that ticket.  Thanks!


Client - typo in gateway generate-certs message

version: development 1.13.0
There's typo in gateway generate-certs message
Steps:

Clone and install development branch Source or Docker
Start the Client
execute gateway generate-certs command
pay attention to the message

Actual:
pass phase
Expected:
pass phrase

Fixed by #6119


Change global variables for tests

As @goktugyil says in #225, it is better to have the variables inside tests instead of being global for the whole class

I dont agree necessarily. It violated DRY (Do not repeat yourself).
But yes if a value is used exactly once, it makes some sense.

Makes sense


The La La Land Video in README

How was the video of  La La Land conducted? directly apply DWpose on the video?

Yes. Just infer every frame with DWPose.


Fix doRequestOAuth to find proper client secrets JSON file

The environment variables GAMUSERCONFIGDIR and CLIENTSECRETS are
factored in to GC_Values[GC_CLIENT_SECRETS_JSON]. On a new installation, it will have the value
os.path.join(GM_Globals[GM_GAM_PATH], FN_CLIENT_SECRETS_JSON) but on an
existing installation the environment variables may have it located somewhere/named something else.

Thanks, I'd like to push a 4.02 prerelease very soon also since there are a number of issues in 4.01.
Do you see any more fixes? (cleanup can wait for now)

Jay,
doRequestOAuth() is called whenever a bad oauth2.txt file is encountered:
getClientAPIversionHttpService, doGetUserInfo, OAuthInfo
In these cases, login_hint = sys.argv[3] in doRequestOAuth is
inappropriate. It may make sense in those locations to issue a message and
bail.
"Your oauth2.txt file is invalid, issue 'gam delete oauth' followed by 'gam
oauth create'"
Ross
On Thu, Nov 3, 2016 at 10:50 AM, Jay Lee<EMAIL_ADDRESS>wrote:

Thanks, I'd like to push a 4.02 prerelease very soon also since there are
a number of issues in 4.01.
Do you see any more fixes? (cleanup can wait for now)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jay0lee/GAM/pull/313#issuecomment-258221284, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIU8LzpiJfF4p9flPv7G4iOSNXXhsQnHks5q6h7zgaJpZM4Kos5V
.

--
Ross Scroggs
<EMAIL_ADDRESS>

Good point. I moved the sys.argv[3] down into the main command parsing area so it's only checked for login_hint on "gam oauth create".


Quick Setup guide - don't work in some linux systems

Specifically in the openSuse and Debian system, the quick setup guide presents problems after the requirements install, i don't tested in more systems but the error occur when trying to do the command
python3 manage.py migrate

Yeah, just the command "python" won't wortk either on this systems, we need to standard the quick guide or to  avoid this messages, the complete error message is:
Traceback (most recent call last):   File "manage.py", line 22, in <module>     execute_from_command_line(sys.argv)   File "/home/vinicius/Vinicius/programs/Bucket/kamu/kamu/lib/python3.6/site-packages/django/core/management/__init__.py", line 381, in execute_from_command_line     utility.execute()   File "/home/vinicius/Vinicius/programs/Bucket/kamu/kamu/lib/python3.6/site-packages/django/core/management/__init__.py", line 357, in execute     django.setup()   File "/home/vinicius/Vinicius/programs/Bucket/kamu/kamu/lib/python3.6/site-packages/django/__init__.py", line 24, in setup     apps.populate(settings.INSTALLED_APPS)   File "/home/vinicius/Vinicius/programs/Bucket/kamu/kamu/lib/python3.6/site-packages/django/apps/registry.py", line 89, in populate     app_config = AppConfig.create(entry)   File "/home/vinicius/Vinicius/programs/Bucket/kamu/kamu/lib/python3.6/site-packages/django/apps/config.py", line 90, in create     module = import_module(entry)   File "/home/vinicius/Vinicius/programs/Bucket/kamu/kamu/lib64/python3.6/importlib/__init__.py", line 126, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "<frozen importlib._bootstrap>", line 994, in _gcd_import   File "<frozen importlib._bootstrap>", line 971, in _find_and_load   File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked ModuleNotFoundError: No module named 'rest_framework'
Any clue about what i have to do in that case?

Using the Python3.6 virtual environment as pointed on the README.md should set the Python 3 as default python. How are you using virtualenvs?

Yeah, i will try on python 3.6.5, currently i am using python 3.7, that could mislead the manage.py. After the test i will update this issue...

@vinimmelo This issue will be closed by the overdue action.


An UE Bug(?) when using emmet in *.vue

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

VS Code Version: 1.67.2
OS Version: Win10 21H2

Steps to Reproduce:
When I input css code in

me too, hope it can be fixed

I'm unable to reproduce this issue. What is your value for the emmet.includeLanguages setting?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


govendor update deletes all of the files in the package to be updated

I'm using govendor at 9de80febcf158338d5eb650b395315d8d4a72a6e. For some reason, when I run govendor update it updates the JSON file to the wrong revision, and blows away the actual file to be updated:
jacobsa:~/go/src/github.com/googlecloudplatform/gcsfuse ((detached from 818d87d))% go version
go version devel +a33b522 Sun Aug 2 23:47:00 2015 +0000 darwin/amd64

jacobsa:~/go/src/github.com/googlecloudplatform/gcsfuse ((detached from 818d87d))% go env | grep VENDOR
GO15VENDOREXPERIMENT="1"

jacobsa:~/go/src/github.com/googlecloudplatform/gcsfuse (master)% git checkout 818d87d98848695a1f30e1710b6d3f8182b7329e

jacobsa:~/go/src/github.com/googlecloudplatform/gcsfuse ((detached from 818d87d))% govendor list github.com/jacobsa/fuse
v github.com/googlecloudplatform/gcsfuse/vendor/github.com/jacobsa/fuse [github.com/jacobsa/fuse]

jacobsa:~/go/src/github.com/googlecloudplatform/gcsfuse ((detached from 818d87d))% govendor update github.com/jacobsa/fuse

jacobsa:~/go/src/github.com/googlecloudplatform/gcsfuse ((detached from 818d87d))% git status
HEAD detached at 818d87d
Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        modified:   vendor.json
        deleted:    vendor/github.com/jacobsa/fuse/LICENSE
        deleted:    vendor/github.com/jacobsa/fuse/README.md
        deleted:    vendor/github.com/jacobsa/fuse/connection.go
        deleted:    vendor/github.com/jacobsa/fuse/conversions.go
        deleted:    vendor/github.com/jacobsa/fuse/debug.go
        deleted:    vendor/github.com/jacobsa/fuse/doc.go
        deleted:    vendor/github.com/jacobsa/fuse/errors.go
        deleted:    vendor/github.com/jacobsa/fuse/freelists.go
        deleted:    vendor/github.com/jacobsa/fuse/mount_config.go
        deleted:    vendor/github.com/jacobsa/fuse/mount_darwin.go
        deleted:    vendor/github.com/jacobsa/fuse/mount_linux.go
        deleted:    vendor/github.com/jacobsa/fuse/mounted_file_system.go
        deleted:    vendor/github.com/jacobsa/fuse/ops.go
        deleted:    vendor/github.com/jacobsa/fuse/unmount.go
        deleted:    vendor/github.com/jacobsa/fuse/unmount_linux.go
        deleted:    vendor/github.com/jacobsa/fuse/unmount_std.go

no changes added to commit (use "git add" and/or "git commit -a")

jacobsa:~/go/src/github.com/googlecloudplatform/gcsfuse ((detached from 818d87d))% git diff vendor.json
diff --git a/vendor.json b/vendor.json
index 408f7c1..7624ced 100755
--- a/vendor.json
+++ b/vendor.json
@@ -12,8 +12,8 @@
                        "canonical": "github.com/jacobsa/fuse",
                        "comment": "",
                        "local": "vendor/github.com/jacobsa/fuse",
-                       "revision": "13eb2958d3bd51261f6870f45adfc4168c98205d",
-                       "revisionTime": "2015-07-29T16:25:39+10:00"
+                       "revision": "818d87d98848695a1f30e1710b6d3f8182b7329e",
+                       "revisionTime": "2015-08-03T12:52:47+10:00"
                },
                {
                        "canonical": "github.com/jacobsa/fuse/fsutil",

Note the following:

818d87d98848695a1f30e1710b6d3f8182b7329e is the revision of the dependant repo, not of the dependency being updated. The old revision (13eb2958d3bd51261f6870f45adfc4168c98205d) is correct.

If I run only godep update, all of my dependencies are deleted.

What am I doing wrong?

Ping. Out of all of the bugs I've filed, this is the only one that's actually keeping me from using govendor, and I'd really like to switch away from godep, which is even buggier. But I can't find a workaround. :-/

Ack. I've made some time this morning. I should have a fix soon.
On Tue, Aug 4, 2015 at 11:15 PM Aaron Jacobs<EMAIL_ADDRESS>wrote:

Ping. Out of all of the bugs I've filed, this is the only one that's
actually keeping me from using govendor, and I'd really like to switch away
from godep, which is even buggier. But I can't find a workaround. :-/
—
Reply to this email directly or view it on GitHub
https://github.com/kardianos/govendor/issues/25#issuecomment-127884773.

@kardianos: Thanks for the fix! I can confirm that this mostly works for me now. It turns out that there were actually two bugs here, and only one has been fixed. I'll file the other one separately.


Remove MAX_CHARS traffic control

This is a revert of my first PR in opendevin: #1015 😢
Back then, we didn't have #2070, only MAX_ITERATIONS to control traffic. MAX_ITERATIONS is useful but not sufficient because it doesn't count the context length, so I introduced MAX_CHARS. Now I think it's time to remove it. Reasons:

Counting characters is inaccurate as most LLMs charge by tokens, not characters.
We already have #2070 that uses this nice litellm feature to compute costs. I checked litellm doc and it seems they cover most common scenarios: https://github.com/BerriAI/litellm/blob/main/model_prices_and_context_window.json. For custom or self-hosted models, litellm also allows you to define the cost model by yourself.
It's hard to set a empirical bound for MAX_CHARS. The current default limit is so high that either MAX_ITERATIONS or MAX_BUDGET_PER_TASK would be hit first, making it useless.
I personally never heard anyone leveraging MAX_CHARS.
Less is more.

I think you're right. Let's do it.

Just a sidenote: I can easily hit 10m chars within 100 steps trying to work with huge C# classes. 🤣
Glad to see this go! 👍


overlayClassName does not override default className

Summary:
If you try to set overlayClassName then overlay className will be not my-class-name but ReactModal__Overlay ReactModal__Overlay--after-open my-class-name.
Also ReactModal__Overlay--after-open is only added for the first time you open the modal. So if you open modal, close it and then open it again - the modal will be mounted with ReactModal__Overlay--after-open (this class should be added only after modal is opened to be able to use animations). You can see this bug here: https://codepen.io/JamesJGoodwin/pen/LYYVZGK

Also overlayClassName seems to be immutable since this code has no effect:
overlayClassName={`modal-overlay${this.state.afterOpen === true ? ' --after-open' : ''}`}
onAfterOpen={() => this.setState({ afterOpen: true })}

This way overlay className will be still modal-overlay
@diasbruno

Hi @JamesJGoodwin. Unfortunately, our documentation is not complete about the class names.
Using a string as the class name will result in appending the name to the default list of class names.
But you can give it an object specifying which names to use.
See https://github.com/reactjs/react-modal/blob/master/src/components/Modal.js#L39


JavaScript error on custom domain blade; cannot generate TXT to verify my domain

Describe the bug

I am following the "Set up a custom domain in Azure Static Web Apps" instruction set.  I'm trying to verify ownership of the domain, as instructed.  But, the code I'm supposed to copy for the TXT record is never generated, and JavaScript errors are dumped to the console.
To Reproduce
Steps to reproduce the behavior:

Under a static web app project, got to Custom Domains
Click Add, then choose 'Custom domain on other DNS'
Put in your domain name:  "domain.tech"  WITHOUT the subdomain, as per the instructions (my domain ends in a .tech; JS error dumped to console after pressing next)
Press the Generate Code button... you're supposed to wait, but mine is never generated.  Instead, JavaScript errors are dumped to the console.

Expected behavior
I had expected to see a code that I could copy and paste as a TXT record for my registrar.
Screenshots

Device info (if applicable):

I have used both Brave and Firefox.

Additional context
I have also tried this with a different static app, and got the same results.  I was only able to finish the first part of the instructions: adding the "www", and that part was verified by Azure.  I just need to get to part 2 & 3 of the instructions:  generated the TXT code, and setting up an Alias.

I have the exact same issue. Hopefully a temporary thing.

What kind of internship student is doing this awful work? Even my assistant can use ChatGPT to make a better one. Hold on... Maybe this is really a Chatpgt master piece.

This issue is also present when doing things in PowerShell, either using Azure CLI or Azure PowerShell Module.

Can confirm the exact same issue above - tried multiple domains and static apps. Same custom blade issue error in console.

Issue seems to be resolved.

I can confirm that I'm now able to generate the special code for my TXT record -- no errors dumped to browser console.  After I complete the entire process, I I'll close this out.


Error when piping deploy

If the output of deploy is piped, we throw an error:

In node the process.stdout process doesn't have the clearLine function we are using to rewrite the output.
Two approaches:

follow hardhat and do the ascii escape codes directly
switch to the verbose text UI

Both of these have ongoing maintenance implications we need to consider.

Closing in favour of #644


Error 400: Invalid Input: INVALID_OU_ID - invalid only on route Organizaztion Sub Orgs

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/google-apps-manager. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GAM release from http://git.io/gamreleases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/GAM/wiki
I am using a Google Apps for Work, Unlimited or EDU account (Google Apps Standard / Free is not supported)

Full steps to reproduce the issue:

gam update org kmbs.konicaminolta.us add file OrgUserMov
eList.txt

2.Generates the following: moving<EMAIL_ADDRESS>to /kmbs.konicaminolta.us (1/1)
Error 400: Invalid Input: INVALID_OU_ID - invalid

If I change the org to a sub org it works OK

Expected outcome (what are you trying to do?):move users from sub org to mail org
Actual outcome (what errors or bad behavior do you see instead?):Error 400: Invalid Input: INVALID_OU_ID - invalid

Try:
gam update org / add file OrgUserMoveList.txt
the admin console lies about the root OU being named after your domain,
it's really just /.
Jay
On Wed, Mar 2, 2016 at 10:46 AM DougBelina<EMAIL_ADDRESS>wrote:

The issue tracker is for reporting product deficiencies. How do I
questions should be posted to the discussion forum at
https://groups.google.com/group/google-apps-manager. When in doubt, start
at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GAM release from
http://git.io/gamreleases and I still have this issue.
I am typing the command as described in the GAM Wiki at
https://github.com/jay0lee/GAM/wiki
I am using a Google Apps for Work, Unlimited or EDU account (Google
Apps Standard / Free is not supported)

Full steps to reproduce the issue:

gam update org kmbs.konicaminolta.us add file OrgUserMov
eList.txt

2.Generates the following: moving<EMAIL_ADDRESS>to /
kmbs.konicaminolta.us (1/1)
Error 400: Invalid Input: INVALID_OU_ID - invalid

If I change the org to a sub org it works OK

Expected outcome (what are you trying to do?):move users from sub org to
mail org
Actual outcome (what errors or bad behavior do you see instead?):Error
400: Invalid Input: INVALID_OU_ID - invalid
—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/204.

--
Jay Lee

gam update org "/" add file OrgUserMoveList.txt worked like a charm.  Thank you for your quick response.


Less logging

Hi,
The plugin is logging every 10 seconds the total and current consumption. Can this be suppressed by setting a logging option or  so?
My Homebridge log is flooded by these messages.
Thanks,
Rien

There's no way in homebridge-p1 to suppress the log messages.
The log (and debug) messages are written to stdout; warnings and errors are written to stderr.  You could redirect stdout to /dev/null and only capture stderr in your log.

Is there a possibility already to change this? My meter has DSMR 5 so it gets data every second, and the RPI seems to stop parsing the data after 2 days or so, homebridge still runs and functions but theres no new data from p1 or even history. When I reboot homebridge it runs fine again...


Bump scipy dependency

Sparse arrays were introduced in scipy 1.8.0, since they are used in topicwizard, this has to be specified in the dependencies.

Done


Package name
torch/pytorch
Package version/tag
2.2.0
Build options
No cuda, cpu only for now
Installation timeframe
The sooner the better!
Installed on Hera, Orion and container. Intel would be nice, but I have not been successful in completing an intel build. GNU was straight forward (for the most part).
Other information
WCOSS2 support would be nice at some point.
WCOSS2

[x] Check this box if and only if your package should be installed on WCOSS2 Cactus and Dogwood (all spack-stack packages will be installed on Acorn). If not, you may disregard the rest of the items below and submit this request.

WCOSS2: General questions
No response
WCOSS2: Installation and testing
No response
WCOSS2: Technical & security review list

[x] The code is mature, stable, and production ready
[ ] The code is does not and cannot use the internet, and does not contain URLs (http, https, ftp, etc.) except in comments
[ ] The package does not contain prebuilt binary files that have not been approved by NCO security review
[ ] The code has no publicly disclosed cybersecurity vulnerabilities and exposures (search https://cve.mitre.org/cve/)
[x] The code is not prohibited by DHS, DOC, NOAA, or NWS
[ ] The code comes from a trusted source. Trusted sources include other NWS, NOAA, or DOC, agencies, or other Federal agencies that operate at a FISMA high or equivalent level. Additionally, trusted sources could be third-party agencies through which there is an existing SLA on file (such as RedHat).
[x] The code is actively maintained and supported (it continues to get updates, patches, etc.)
[x] The code is not maintained by a private entity operating in a foreign country (if it is, make a note below)
[x] There is sufficient documentation to support maintenance
[x] There are no known security vulnerabilities or weaknesses
[x] Installing and running the code does not require privileged processes/users
[ ] There are no software dependencies that are unapproved or have security concerns (if there are, make a note below)
[ ] There are no concerns related to SA, SI, and SC NIST control families

WCOSS2: Additional comments
No response

@AlexanderRichert-NOAA @Hang-Lei-NOAA Note the "WCOSS2 support would be nice at some point" in the above request. Not urgent, but you may want to start the request once we have it in spack-stack for the RDHPCS machines.
@AlexanderRichert-NOAA Do you want to take this on or should I? It's for EMC and JCSDA likewise.

I don't know if that's helpful, but these are the options I used on Orion/gnu:
cmake -DUSE_CUDA=OFF -DUSE_CUDNN=OFF -DUSE_NCCL=OFF -DBUILD_CAFFE2_OPS=OFF -DUSE_MKLDNN=OFF -DUSE_DISTRIBUTED=ON -DCMAKE_INSTALL_PREFIX=...

and the build / CMakeCache.txt:
/work/noaa/da/gvernier/ai/pytorch-v2.1.2/build/CMakeCache.txt

@guillaumevernieres to get things rolling for WCOSS2, can you please fill out the checklist to the best of your ability?
@Hang-Lei-NOAA FYI...

@guillaumevernieres to get things rolling for WCOSS2, can you please fill out the checklist to the best of your ability? @Hang-Lei-NOAA FYI...

Will do @AlexanderRichert-NOAA .

As far as creating a test installation on one platform is concerned: my plan is to work on this after my current PR #993 is merged.

As far as creating a test installation on one platform is concerned: my plan is to work on this after my current PR #993 is merged.

Sounds good. Thanks @climbfuji .

@guillaumevernieres Just to let you know that I started working on this yesterday. I've hit some problems with incompatible py-protobuf versions for py-torch and cylc-flow, and also build errors on Ubuntu with Intel. I'll keep you posted.

@guillaumevernieres Just to let you know that I started working on this yesterday. I've hit some problems with incompatible py-protobuf versions for py-torch and cylc-flow, and also build errors on Ubuntu with Intel. I'll keep you posted.

Thanks for the update @climbfuji . How about not building the python API (-DBUILD_PYTHON=OFF)? I don't know why cylc-flow is part of the build, but I'm pretty sure we wouldn't need this feature.

I assume you refer to not building the Python API for cylc? Wouldn't that preclude building the cylc UI, which uses py-jupyter-server. cylc is in spack-stack because several of JCSDA's partners are using it or are looking into using it, and JCSDA is also considering switching to cylc due to its superior handling of cross-cycle dependencies.
Regardless, there are build errors for the particular version of py-protobuf that py-torch wants, so even if we get the two packages cylc and py-torch to coexist (which I think is quite possible), then we still can't build py-torch.

No, I meant not building the python API at all.

@climbfuji I think what @guillaumevernieres is trying to say is: can we build torch without the pytorch python API bindings? I think we are only using the C interface for torch (is that right G?). Having the Python part of torch would be handy in the future, but is not essential right now

I'll look into that. There's also an option to use an internal/vendored version of protobuf/py-protobuf, which might be a better solution.

I'll look into that. There's also an option to use an internal/vendored version of protobuf/py-protobuf, which might be a better solution.

Not sure why we don't see @travissluka 's message, a glitch in the matrix!
But yes to what he said. We don't need the python binding, at least not at first.

I think I accidentally hit delete... I clearly shouldn't check GitHub before 6am

FYI here is the spack bug I filed for protobuf: https://github.com/spack/spack/issues/43054

Upon my inquiry of the libtorch/pytorch on wcoss2 with NCO. This has to be associated with the operational development. So, the frist step: we need a complete proof to show it is required for operational use, and get support from EMC federal side.  Then second, detailed code review can start.

@guillaumevernieres to get things rolling for WCOSS2, can you please fill out the checklist to the best of your ability? @Hang-Lei-NOAA FYI...

Any updates on this @AlexanderRichert-NOAA ?

@guillaumevernieres Upon our initial inquiry, this is not approved by NCO.
Therefore, we have to apply for the code review.
Before going to that step, please prepare following questions, and make sure that is required in our operational models:
(1) Demonstrate the necessity of the requested code.
The functionality provided by the software is irreplaceable for operations. It is used by operational models. There is currently no existing NCEP code that offers comparable capabilities. or Insufficient time to develop.
(2) Verify the software license and developer credentials.
The software is developed by U.S. or security approved agencies. License is required and acceptable.
(3) Carefully go through the source code to check if it meets the requirements. (EE2, FISMA, additional requirements from NCO).
A line by line check to see if any outreach code or unnecessary functions need to be removed.
(4) Validate the building script to ensure compatibility with the operational machine environment.

Since I used using py-torch on google's machines.
As I examined these issues, I feel very difficult to proceed.
(1) I did not see any our operational model or ones under development use this package.
(2) It is working under Anaconda environment, which was not approved by NCO. (We used to push these python env to NCO, was declined). Then we have to think about install without [Anaconda].
(3) License, and outreach in code may not meet the EE2 and FISMA requirements. It has to been explained in the code review.

The requirements for pytorch are seen easily in requirements.txt
Pytorch recommends Anaconda, but it is not required.  It can be installed from source with instructions here

@aerorahul thanks for sending those helpful details.
@Hang-Lei-NOAA pytorch/libtorch is almost certainly something that will be required for us to do AI/ML work. Even if the process is going to take some time to work through I think it's best we start now and chip away at it.

@aerorahul thanks for sending those helpful details. @Hang-Lei-NOAA pytorch/libtorch is almost certainly something that will be required for us to do AI/ML work. Even if the process is going to take some time to work through I think it's best we start now and chip away at it.

@JacobCarley-NOAA , the requirements that @aerorahul posted above are for python. In addition to that, we need to also look at the other problematic fractal of dependencies. It starts here:
https://github.com/pytorch/pytorch/blob/main/.gitmodules
Happy to help build a minimalist library with gcc if that can help.

@JacobCarley-NOAA @aerorahul Thanks, We can install that version. Our developers need to know what they can work and make sure that way will match their demands. This would be a unique solution to wcoss2 that differs from spack-stack on other RDHPCS. For a profound view, we may plan a long term python approach to follow. As I know in the past many years, NCO support python 3 series. However, our developers have requested many different python environment, like miniconda, Anaconda, etc. If our development work use a unified, e.g. python 3, that will be more consistent for future.
@guillaumevernieres Please prepare the settings for the environment that you need. We may based all on python 3. We can have a meeting next Monday for details. The code review will have to answer a series of questions and we have to find out a solution to insert into current operational environment. You participation is important. Thanks.