Build paragraph id on top of features

This change builds the paragraph id generation on top of the features infrastructure. This allows things to be more streamlined and allow the following features:

Automatically add paragraph ids to elements as they're inserted into the tree
Replace duplicated paragraph ids with a fresh unique one
Provide easy extensibility of system via features

As part of this, a new feature is added internally that allows tracking of elements being inserted/removed. This builds on top of the element context, which should be swapped as the element context incurs a lot of allocations while the new events use structus.
This change also moves it out of the core library and into the features library.

@ThomasBarnekow this should be getting you closer to the preferred behavior for the paragraph id generation. This keeps the feature co-located with each other and enables some scenarios I'd like as well when using it (ie automatically adding paragraph ids when added to a tree if it's not there).

I didn't see th epoint of keeping the method that fills existing items from a word document but could be convinced to add it back if you see value in that.

I didn't see the point of keeping the method that fills existing items from a word document but could be convinced to add it back if you see value in that.

I have a use case in which I want to ensure the uniqueness of the w14:paraId values across multiple Word documents. Therefore, the design has always allowed for explicitly providing a (possibly central) generator instance which would do just that.

As for the central generator, that's an interesting use case that makes sense. I'm wondering if it can be composed if we tweaked the current framework.  What about the following:

IParagraphIdFeature is split into IParagraphIdGeneratorFeature and IParagraphIdCollectionFeature
IParagraphIdGeneratorFeature would request the current IParagraphIdCollectionFeature from the document
This way, if the collection is updated to be a shared collection (with whatever implementation you want), it will be able to use the Contains call there, and then update its own tracking of per-document call.
This would allow more flexibility of shared logic for the generation, but a separate service that tracks the actually used ids.

Some pseudo-code for how this would be used:
WordprocessingDocument doc = GetSomeDocument();
var shared = doc.UseSharedParagraphIdFeature();

WordprocessingDocument doc2 = GetSomeOtherDocument();
doc2.UseSharedParagraphIdFeature(shared);

With a off-the-cuff implementation as:

public static IParagraphIdCollectionFeature UseSharedParagraphIdFeature(this WordprocessingDocument doc)
{
    var newFeature = doc.GetParagraphIdCollectionFeature();

    if (newFeature is SharedParagraphIdCollectionFeature other)
    {
        return other;
    }

    other = new();
    other.Add(newFeature);

    doc.Features.Set<IParagraphIdCollectionFeature>(other);

    return other;
}

public static IParagraphIdCollectionFeature UseSharedParagraphIdFeature(this WordprocessingDocument doc, IParagraphIdCollectionFeature shared)
{
    if (shared is not SharedParagraphIdCollectionFeature sharedCollection)
    {
        throw new InvalidOperationException();
    }

    var newFeature = doc.GetParagraphIdCollectionFeature();

    sharedCollection.Add(newFeature);
    doc.Features.Set<IParagraphIdCollectionFeature>(sharedCollection);

    return shared;
}

private class SharedParagraphIdCollectionFeature : IParagraphIdCollectionFeature
{
    private readonly List<IParagraphIdCollectionFeature> _others = new();

    public void Add(IParagraphIdCollectionFeature other)
    {
        _others.Add(other);
    }

    public IEnumerable<string> RegisteredParagraphIds => _others.SelectMany(o => o.RegisteredParagraphIds);

    public int Count => _others.Sum(o => o.Count);

    public bool Contains(string id)
        => _others.Any(o => o.Contains(id));
}


Site hangs

Going to http://project-repos.ipfs.io/ doesn't resolve, at the moment. @harlantwood, any ideas?

Thanks @RichardLitt ...
Strange, the underlying github pages version is not working either:
https://ipfs.github.io/project-repos
Even though when I push the exact same branch/code to my fork it works:
http://harlantwood.github.io/project-repos/
Have to run, will try to follow up more soon.

That's weird! Anything I can do to help diagnose?

My next step is to remove the gh-pages branch on github and recreate with
npm run publish
Feel free to try this if you get to it first

deleted the branch and republished, still no love...

Pinged GH support.

Good move. I'm not sure what the problem is, either.

Hm, really strange.
Jamie @ Github said:

Sorry for the trouble. It looks like your site is now loading at http://project-repos.ipfs.io. It's possible that the DNS records hadn't fully propagated yet.

I said:

Hm, it doesn't load for me, even though the DNS is fully propogated
worldwide (and hasn't changed in at least several months I believe)
https://www.whatsmydns.net/#A/project-repos.ipfs.io
You see http://project-repos.ipfs.io loading just fine? We are a
distributed team, and we don't see it from any of our locations...

Then @danayel replied and said:

I can see it also. (screenshot attached)
Jamie is in America and I am in Japan, so we are viewing from different places also.
What do you see when you access that url?
Could you please try clearing your cache and then reloading the page?

But I still see nothing (just hangs), even when going through a proxy like https://www.proxysite.com/ ...

Maybe this is a good time to move the site into the same system by which we host our other sites, instead of GH Pages.  @lgierth how could we make that so?

@harlantwood for now, post an issue with a hash to https://github.com/ipfs/ops-requests
How you come up with that hash is your thing :)

Great, created https://github.com/ipfs/ops-requests/issues/25

Live via IPFS!


Intermediate launch tutorial

This tutorial shows the way to use launch files for large projects. It also shows examples of using parameters, YAML files, remappings, namespaces, default arguments, and RViz configs in launch files.
Signed-off-by: Shyngyskhan Abilkassov<EMAIL_ADDRESS>

@mergify backport foxy galactic


Hiding all overloads also hides the main function

Search terms

@hidden, overload
Expected Behavior
#1142 adds the ability to apply @hidden to only some overloads without hiding all of them. I'd expect that I can apply @hidden to all of the overloads except for the original function and still be able to see the original, non-overloaded function.
Actual Behavior
Typedoc hides the function when all the overloads are hidden
Steps to reproduce the bug
MRE: https://tsplay.dev/w1An8w
Notice how all overloads are marked with @hidden, but the original function is not. Typedoc hides this entire function from the docs, when I'd expect it to still show the combined method.
Environment

Typedoc version: 0.23.22
TypeScript version: 4.8.4
Node.js version: 16.17.1
OS: macOS Ventura

I can't seem to reproduce this. I pasted your file into a basic project, ran typedoc, and got this output:

I suspect your code was actually:
/** @hidden */
function getSomething(all: true): string[];
/** @hidden */
function getSomething(all: false): string;

function getSomething(all: boolean): string | string[] {
 return []
}

This is different. When overloads are declared, the signature declared on the implementation is not visible outside of the function, and effectively does not exist (getSomething(Math.random() > 0.5) is a type error).  If you want the catch-all signature to be visible, you need to add it separately
/** @hidden */
function getSomething(all: true): string[];
/** @hidden */
function getSomething(all: false): string;
/** documented! */
function getSomething(all: boolean): string | string[];
function getSomething(all: boolean): string | string[] {
 return []
}

The declare isn't in my actual code, I just did it there so I wouldn't have to define the actual function for the example. Wasn't aware it would make a difference.

I think I tried adding the combined overload but it caused type errors. Will try again later and let you know.

Must've messed something up the first time, it works now. Thanks!


Error : Not Found google cloud storage

OS: Windows10
Node.js version: 8.4.0
npm version:5.3.0
google-cloud-node  #version:1.2.1
add_file:function(path){
    bucket.upload(path, function(err, file) {
        if (err) {
          console.error(err)
        }
        console.log(file)
    });
  },

thats my function to upload a file the file is a 34 mb .dem protobuf file the error trace im getting is.
Error: Not Found
    at Request._callback (C:\Users\anshu\Desktop\statseeker\node_modules\gcs-resumable-upload\index.js:283:25)
    at Request.self.callback (C:\Users\anshu\Desktop\statseeker\node_modules\request\request.js:188:22)
    at emitTwo (events.js:125:13)
    at Request.emit (events.js:213:7)
    at Request.<anonymous> (C:\Users\anshu\Desktop\statseeker\node_modules\request\request.js:1171:10)
    at emitOne (events.js:115:13)
    at Request.emit (events.js:210:7)
    at IncomingMessage.<anonymous> (C:\Users\anshu\Desktop\statseeker\node_modules\request\request.js:1091:12)
    at Object.onceWrapper (events.js:314:30)
    at emitNone (events.js:110:20)
    at IncomingMessage.emit (events.js:207:7)
    at endReadableNT (_stream_readable.js:1059:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)

Hello, @c00kie17, can you fill in the details about your environment?

OS:
Node.js version:
npm version:
google-cloud-node version:

Thanks!

Note: The OP edited it into his original post. (I almost missed that.)
With #2582, this is the second report in three days that are npm 5.3.0.

@c00kie17 would you mind going into the file, C:\Users\anshu\Desktop\statseeker\node_modules\gcs-resumable-upload\index.js, adding a console.log(authorizedReqOpts) on line 275, then pasting the response here? (You can remove the Authorization: 'Bearer y.2....' line from the output)

{ method: 'POST',
uri: 'https://www.googleapis.com/upload/storage/v1/b/my_project_id/events/new_player/o',
qs:
{ name: 'file.csv',
uploadType: 'resumable',
predefinedAcl: 'publicRead' },
json: { contentType: 'image/jpeg', metadata: { custom: 'metadata' } },
headers:
{ 'X-Upload-Content-Type': 'image/jpeg',
Authorization: 'Bearer ... } }

OS: Windows 7
Node.js version: 6.9.5
npm version: 3.10.10

I believe the path should be: https://www.googleapis.com/upload/storage/v1/b/your_bucket_name
Can you show how you create the Bucket object?

The bucket object is already created im not creating it in my code. I created it manually from the google cloud console.

Sorry, I meant the @google-cloud/storage "Bucket", i.e.:
var storage = require('@google-cloud/storage')
var bucket = storage.bucket('...')
var file = bucket.file('...')

It seems like the argument passed to storage.bucket() and/or storage.file() might be incorrect. In a typical scenario, this should look like:
var bucket = storage.bucket('my-bucket-name')
var file = bucket.file('my-file-name')

If the file is in a folder, it would look like:
var bucket = storage.bucket('my-bucket-name')
var file = bucket.file('my-folder-name/my-file-name')

This issue was moved to googleapis/nodejs-storage#25


Deleted inactive/irrelevant links

What does this PR do?
Remove resource(s)
For resources
Description
It removes inactive links mostly
Exceptions:
*Chinese: third JS position is a blog post,
last one redirects somewhere
*D3.js - the same as above
Why is this valuable (or not)?
Some inactive/irrelevant links will be removed.
How do we know it's really free?
n/a
For book lists, is it a book? For course lists, is it a course? etc.
n/a
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

I pulled your main and honestly I don't see any changes.
I also thought I would keep this 'delete-links' branch and maybe find more to delete.


Save extra comment fields

Fixes https://github.com/eregs/notice-and-comment/issues/115

Would you mind adding tests around sanitize_fields? Unrelated, I'm a tad concerned that we're writing our own validation framework -- would it make sense to reuse Django's or find a more general purpose version in a later step?

I'll add tests, and not just for sanitize_fields - I wanted to make sure that we are OK with my approach.
Regarding the validation framework, I think Django's is tightly coupled with their models. Yes, it makes sense to find a more general purpose library if we are going to do much more of this.

Good stuff


Watching apps

Would allow users to click Watch button on app pages, and have them show up a new page.
Said page can also include last updates from all the games you own (up to 10), since /myapps is gone, this will be a good alternative.

This could also cleanup #244, bot account could just watch apps instead of having a separate table.

Would be interesting to make watched apps page public, so that users could share their own watch lists.


fig 1

the size of each panel in col 1 should be the same size as the panels in col 2 & 3.
and should be lined up perfectly vertically
(eg, top of panels and bottom of panels should align)

ok adjusted accordingly, better?

doesn't look right to me, let's discuss tomorrow.

ok vertically aligned up.
Due to text size, the horizontal length of col 1 is a little larger than other cols.
But at least one thing (length, width, text size, or text content) has to be sacrificed for all others to align with cols2.

better.   in the spiral, shouldn't 1 & 2 be on the same "arc" of the spiral to make our point?

the width of col 1 need not be the same as the other col's, so that's fine.

that is right...I will try to put them on same arc tmr morning.
On Thu, Dec 8, 2016 at 10:07 PM, joshua vogelstein<EMAIL_ADDRESS>
wrote:

the width of col 1 need not be the same as the other col's, so that's fine.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/163#issuecomment-265922041, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALX0y3FZ0VruM3pNBPoyfUlwP4oZvO2Yks5rGMX2gaJpZM4LHN0Q
.

ok updated to same arc now

ok, close :)
On Fri, Dec 9, 2016 at 7:28 AM, cshen6<EMAIL_ADDRESS>wrote:

ok updated to same arc now
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/163#issuecomment-266002669, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjcinNlEakRe4f4hK1CI2xA_qqhUE5ks5rGUlbgaJpZM4LHN0Q
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York


Add German Translation

Hi Jessica,
first of all - Thank you! I wanted to learn Git for a while now - Two weeks ago i learned it within one evening with your App. It was a pleasure!
As my first project i thought about giving back sth. to you and your app by doing the translation to German. In the end it was more to do than i thought in the beginning, but i nailed it in a first, rudimentary version. For native Germans it might still sound a bit strange from time to time as i just translated what i read from the english version, not extensively checking if it is perfect to the end.
One idea to further improve the translation, i included on the index and resources page, inviting git-it finishers in the german version to contribute to the translation. I linked it to my forked repository, where i could review changes and open pull-requests for a larger number of commits from time to time. Would you be fine with this idea?
The following commits now mainly translated the files in the german resources/contents-folder. Further ideas and issues i recognised (partially related to translations), that are not to be found within this folder i will try to handle in further commits and pull-requests.
The files package.json and package-lock.json were changed by npm on npm-update & npm-upgrade. Maybe it was not the best way to commit these changes. Do they matter to this pull-request? Can i somehow exclude them from the pull-request?
Greetings from Germany!
Jonas

Ok, seems like i found a way to remove the package-issue. :)

Hi @jotoeri! :) It seems there is nothing in your lib/locale-de-DE.json? Some months ago I wanted to start a translation but actually only got to making that one file.
Should I add it to the pull request? It’s the code block pasted below. I left the English words in parentheses because otherwise people would be very confused by the terminology they see everywhere in tutorials and on Github/Gitlab.
Thoughts on that, and also a general question to @jlord:
I think the "Add tooltip because of language barrier" https://github.com/jlord/git-it-electron/pull/216 by @vardrop is quite relevant because German developers mostly also use words like "committen" and "pushen". So for a learner it’s important to both have a less jargon word to know the concept (like "change" or "upload") as well as to know the real terms.
There is also a translation like https://github.com/danielauener/git-auf-deutsch where half of the words are nonsense or satire though (or at least seem like it). And in git - the simple guide, the German version simply uses the English words in German form.
{
  "n": {
    ".gitignore": "Ignorierdatei (.gitignore)",
    "repository": "Projektordner (repository)",
    "repositories": "Projektordner (repositories)",
    "branch": "Zweig (branch)",
    "branches": "Zweige (branches)",
    "fork": "Kopie (fork)",
    "forks": "Kopien (forks)",
    "request": "Anfrage (request)",
    "pull request": "Änderungsanfrage (pull request)",
    "pull requests": "Änderungsanfragen (pull requests)",
    "commit": "Änderungssatz (commit)",
    "commits": "Änderungssätze (commits)",
    "upstream": "ursprüngliches Projekt (upstream)",
    "feature": "Funktionalität (feature)",
    "status": "Status",
    "diff": "Unterschiede (diff)",
    "readme": "Hilfedatei (readme)",
    "collaborator": "Teammitglied",
    "collaborators": "Teammitglieder",
    "remote": "zentraler Projektordner (remote)",
    "remotes": "zentrale Projektordner (remotes)",
    "contributing": "mitmachen",
    "connections": "Verbindungen"
  },
  "v": {
    "add": "hinzufügen (add)",
    "fork": "kopieren (fork)",
    "forked": "kopiert (forked)",
    "clone": "Arbeitskopie erstellen (clone)",
    "push": "hochladen (push)",
    "close": "schließen",
    "pull": "herunterladen (pull)",
    "commit": "Änderung bestätigen (commit)",
    "merge": "zusammenführen (merge)",
    "merged": "zusammengeführt (merged)",
    "checkout": "wechseln (checkout)",
    "stage": "vorbereiten (stage)"
  },
  "adj": {
    "merged": "zusammengeführte(s/r) (merged",
    "forked": "kopierte(s/r) (forked)",
    "remote": "ferne(s/r) (remote)"
  }
}

Hey Jan,
yes, that is true, the locale-file is empty as i did not really know the effect it provides. Where are these translations used?
Your translations are quite well, would be nice to integrate it. Tank you!
Furthermore it would be nice to review the whole translation with your technical translations and your way to provide the original english words. And maybe make true german grammar sentences out of it. ;) If i find the time i will do so, but if you want you are invited to do so, too. :+1: :)
Cheers!
Jonas

Your translations are quite well, would be nice to integrate it. Tank you!

Cool! :) I think Github’s "Add a suggestion" feature only provides the ability to suggest single lines. Could you add me as collaborator on your fork, then I will add a commit to update lib/locale-de-DE.json.

the locale-file is empty as i did not really know the effect it provides. Where are these translations used?

I actually also don’t know that. I assumed it to be some sort of glossary, and started with it just to have clarity over what the terminology should be. :)

@jotoeri thanks! Added a commit with the glossary terms. Not sure if I will have time soon to proofread the whole thing though, that’s why I also didn’t continue with the translation back then. But I’ll see, and let me know if you need help on something specifically – e.g. which parts do you feel least comfortable with? :)

So i improved the translation now by myself. Removed the translation-contribution information, so now it should be a simple and usable translation to German.


test Wed Sep  9 07:46:30 2020

This is a test Wed Sep  9 07:46:30 2020

Adding another email.
--
Billy K. Poon
Research Scientist, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
1 Cyclotron Road, M/S 33R0345
Berkeley, CA 94720
Tel:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
Web: https://phenix-online.org
On Wed, Sep 9, 2020 at 7:46 AM Billy K. Poon<EMAIL_ADDRESS>wrote:

This is a test Wed Sep 9 07:46:30 2020
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/phenix-lbl/testing/issues/5, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGXF2N7JTZMERKX62C5Y4TSE6INRANCNFSM4RCOMKUA
.

This is from my other email address.
On Wed, Sep 9, 2020 at 8:16 AM Billy Poon<EMAIL_ADDRESS>wrote:

Adding another email.
--
Billy K. Poon
Research Scientist, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
1 Cyclotron Road
https://www.google.com/maps/search/1+Cyclotron+Road?entry=gmail&source=g,
M/S 33R0345
Berkeley, CA 94720
Tel:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
Web: https://phenix-online.org
On Wed, Sep 9, 2020 at 7:46 AM Billy K. Poon<EMAIL_ADDRESS>wrote:

This is a test Wed Sep 9 07:46:30 2020
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/phenix-lbl/testing/issues/5, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGXF2N7JTZMERKX62C5Y4TSE6INRANCNFSM4RCOMKUA
.

--

-Billy


Advertize node versions support

Not a critically necessary change, as by nature this library supported Node 18 from the start, but it's nice to have explicit support marked, so that we have a warning if we try to install this in an untested version (looking at Node 20 coming soon)

@NSeydoux I don't think we should make this change:

It will introduce an unnecessary warning for users trying to use this on odd versions of node
It introduces maintenance burden for us to immediately update the engines field for each new version of Node. I think we can be confident fetch will be available in Node 20 given that it is available in Node 19.

Odd versions of Node are active development versions, and as per our policy we don't support them, so warning our users about it is a matter of transparency. And I don't doubt that this will be compatible with Node 20, however we should test that it works properly and then mark our support, and not the other way around assuming things will work and them potentially breaking without a warning from our libraries.

I've added node 20 support in the engine as well now @NSeydoux


Large WARC files from storage tab are unreliable to download

With larger WARC files, approaching the 1GB mark, downloading from the storage tab via HTTP or HTTPS is error prone: downloads stall and fail regularly.

Should be obsolete now, this is from very old version.


Logging in to Google account crashes on Android 10

After enabling signature spoofing with smali patcher and installing microg through nanodroid I find I am unable to login to a google account, this was working normally on android 9, I'm not sure what the issue is but I think microg needs to be updated.
Here's my logs https://www61.zippyshare.com/v/fzyTKmYQ/file.html
, I can't seem to attach it in github for some reason.

The link gives a 404 for me.

I'm also having a crash on an Android 10 ROM with the latest Microg. Logs are attached.
From looking a bit it seems that maybe the root cause is this:
10-23 13:25:56.189  3840  4611 W Binder  : java.lang.SecurityException: no permission to read the sync settings: Neither user 10102 nor current process has android.permission.READ_SYNC_SETTINGS.
microg_crash.log

@mar-v-in I uploaded it with a new link, github doesn't let me attach it for some reason.

@mar-v-in I uploaded it with a new link, github doesn't let me attach it for some reason.

change it to zip or 7z


Added loading indicator to app wall

Added loading indicator to app wall when fetching application stats

@nwmac Is the app stat etentative === loading?

Going to close this as it's not really the right thing to do.


rename_node without event

Need option to rename node without triggering event. In case of DB name reject, need to revert it's name back.

`lion@tmn-net-mon-02:/var/www/html/lib$ diff jstree-modified/jstree.js vakata-jstree-a7f2242/dist/jstree.js
3926c3926
< 		rename_node : function (obj, val, no_trigger) {

  rename_node : function (obj, val) {

3951,3953c3951
< 			if(!no_trigger) {
< 				this.trigger('rename_node', { "node" : obj, "text" : val, "old" : old });
< 			};

  	this.trigger('rename_node', { "node" : obj, "text" : val, "old" : old });

`

Actually need option to skip trigger and checks on some functions too. To revert some changes, denied by backend.

This has been discussed many times - manage this state in your code and simply do not react to the event depending on a condition you manage. As for reverting changes - this is a bad idea - just refresh the tree.


Multiple widgets used the same GlobalKey.

Also i got this in the logs constantly:
flutter: Another exception was thrown: Multiple widgets used the same GlobalKey.
It does not say where :)
We should sort this out early before it bits us.
anyone know how to turn on logging with stack trace so we can see where its coming from ?
maybe we should put in a stack trace helper that can be turned on and off via settings ?
also this: https://github.com/flutter/sentry
Sentry is easy to self run on our servers.

Sentry is a good option to track stacktraces. In this specific case it wouldn't have helped since the stacktrace on duplicate key errors isn't with much informations. I could track it down and the problem relied on a mixture between using flutter modular and responsive widgets using in the MaterialApp builder method, where the navigator is not yet accessible. I fixed it by using a UniqueKey for our Rail so when resizing the Rail it gets every time a new key, that may have (small) effects on the resizing performance. With new keys the view will not be recycled. It's not the finest solution but was the fastest. Since people are not resizing windows all day, i think it's good until we encounter performance issues. ( I tested in debug mode which is much slower then production and haven't had ANY issues on performance, while resizing)


Unrealistic Setup Time

I just checkout the code on a Raspberry Pi B Module (512MB of memory), and npm install has been running for the last 4 hours and still going. This needs to be tuned up to a more realistic time figure.
Any suggestions on how we could go about doing that?

Will forking these modules into the project reduce the setup time?

@mustafa96m not really you would still need to copy all that data into the pi
@AlKass the best approach I could think of is bundling and generating minified builds.
For example once the front end is built we dont really need to get all the dependencies needed for react which are the majority. So in theory, we could build everything on a beefy machine (dev machine), then use the generated build for the pi which will only have the components needed to run it, none of the dev deps and it will already be transpiled to run on node without babel and so on

i think we are doing the react part wrong...
we should not compile/babel/minify on the Pi.. we should provide clean dist files

I agree with @Ahmed-90 's comment, I just was still asleep when I replied 😄

two points
@AlKass which part took the most time ? npm install ? npm run build ?
@Ahmed-90 what if a user makes his own plugin ? does he get to rebuild or does he have to get a separate development package

@AlhasanIQ we should provide a clean API so when a user want to build his own he just connect to the interface i don't think its logical to have a full building process in the final product

@AlhasanIQ , I fell asleep waiting on npm install to finish. Haven't had the chance to run npm run build yet.
@Ahmed-90, we will continue to maintain the React part of the project in the project directory, so the user can always choose to build from scratch if he/she is willing to wait. That said, we still need to find ways to minimize the time required for the build process to run.

@AlKass well.. logically you shouldn't ship source with the controller.. not everyone is a developer and not everyone want to build plugins..
source should be provided in a different package... maybe an option during instillation or something

@Ahmed-90
In that case, we could maintain the source in a different repository and link it to PiTrol as a submodule. That way developers can add --recursive to git clone and get the source as part of the developer tool set.


.?p show @user doesn't work

📢 Bug reported by Dadi<PHONE_NUMBER>23161090)
What is the title of your bugreport?
.?p show @user doesn't work
How can you reproduce the bug?
.?p show @user
Normally you should get the banlog of a user but currently you get this message: "@Dadi, You forgot to mention a user or to enter a userID"
Which result would normally have to be?
As i said above, you should see the banlog
Comments:
❗✅ Dadi: important

Sieht behoben aus - v4c lol


fix direct navigation to claim via canonical urls from search

i.e. @lulz:a/test-live-sched:1
@lulz:a
This used to work in the past, not sure when it broke.  Should work with # also

Works with https://odysee.com/... Links too


Error in make scalapkg

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System:
Mac OS X 10.12.2
Compiler:
Package used (Python/R/Scala/Julia):
Scala
MXNet version:
0.9.1
Or if installed from source:
MXNet commit hash (git rev-parse HEAD):
d0728ca552d919fd40616149016ba4dd664c4e20
If you are using python package, please provide
Python version and distribution:
If you are using R package, please provide
R sessionInfo():
Error Message:
Please paste the full error message, including stack trace.
make scalapkg
(cd /myhome/Tools/mxnet/scala-package;
mvn clean package -Posx-x86_64-gpu -Dcxx="g++"
-Dcflags="-DMSHADOW_FORCE_STREAM -Wall -O3 -I/myhome/Tools/mxnet/mshadow/ -I/myhome/Tools/mxnet/dmlc-core/include -fPIC -I/myhome/Tools/mxnet/nnvm/include -Iinclude -funroll-loops -Wno-unused-parameter -Wno-unknown-pragmas -Wno-unused-local-typedefs -msse3 -I/usr/local/cuda/include -DMSHADOW_USE_CBLAS=1 -DMSHADOW_USE_MKL=0 -DMSHADOW_RABIT_PS=0 -DMSHADOW_DIST_PS=0 -DMSDHADOW_USE_PASCAL=0 -DMXNET_USE_OPENCV=1 -I/usr/local/Cellar/opencv/<IP_ADDRESS>/include/opencv -I/usr/local/Cellar/opencv/<IP_ADDRESS>/include -DMSHADOW_USE_CUDNN=1  -I/usr/local/opt/openblas/include -DMXNET_USE_DIST_KVSTORE -I/myhome/Tools/mxnet/ps-lite/include -I/myhome/Tools/mxnet/deps/include -DMXNET_USE_NVRTC=0" -Dldflags="-pthread -lm -lcudart -lcublas -lcurand -L/usr/local/cuda/lib64 -L/usr/local/cuda/lib -lopenblas  -L/usr/local/Cellar/opencv/<IP_ADDRESS>/lib -lopencv_calib3d -lopencv_contrib -lopencv_core -lopencv_features2d -lopencv_flann -lopencv_gpu -lopencv_highgui -lopencv_imgproc -lopencv_legacy -lopencv_ml -lopencv_nonfree -lopencv_objdetect -lopencv_ocl -lopencv_photo -lopencv_stitching -lopencv_superres -lopencv_ts -lopencv_video -lopencv_videostab -lcudnn  -L/usr/local/opt/openblas/lib -L/usr/local/lib/graphviz/ /myhome/Tools/mxnet/deps/lib/libprotobuf-lite.a /myhome/Tools/mxnet/deps/lib/libzmq.a -lcuda"
-Dlddeps="/myhome/Tools/mxnet/ps-lite/build/libps.a /myhome/Tools/mxnet/dmlc-core/libdmlc.a /myhome/Tools/mxnet/nnvm/lib/libnnvm.a")
[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[ERROR] 'dependencies.dependency.artifactId' for ml.dmlc.mxnet:libmxnet-init-scala-${platform}:${libtype} with value 'libmxnet-init-scala-${platform}' does not match a valid id pattern. @ line 77, column 19
@
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project ml.dmlc.mxnet:mxnet-macros_2.11:0.1.2-SNAPSHOT (/myhome/Tools/mxnet/scala-package/macros/pom.xml) has 1 error
[ERROR]     'dependencies.dependency.artifactId' for ml.dmlc.mxnet:libmxnet-init-scala-${platform}:${libtype} with value 'libmxnet-init-scala-${platform}' does not match a valid id pattern. @ line 77, column 19
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http:/cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
make: *** [scalapkg] Error 1
Minimum reproducible example
if you are using your own code, please provide a short script that reproduces the error.
make scalapkg
Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.
After building mxnet

make scalapkg

What have you tried to solve it?

Modify pom.xml

@javelinjs

@Bill-Q seems scalapkg hasn't support osx-x86_64-gpu yet,
see https://github.com/dmlc/mxnet/blob/master/scala-package/core/pom.xml
start at line 18

I see. Thanks for pointing it out. Do we have an ETA for supporting it?

By adding profile osx-x86_64-gpu in core/pom.xml, with a new sub module under native, I think it should not be hard to support gpu on osx. Are you interested in contributing this support?

@javelinjs what does adding the support for osx-x86_64-gpu entail -- is it just pom.xml massage or more heavy lifting?

It could be done by modifying pom.xml, while I'm not sure whether special compile options are needed.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!

Does it build on Mac OSX?  People try and abandon the project when it does not build on Mac.  I'd think the way to close it is to make it build, and the issue should not be closed until it does.

@javelinjs

As we've discussed, it builds on osx for cpu, but not gpu. I'm working on maven deploy currently. The gpu support for osx is under relatively low priority. I'll make a roadmap soon.

yes, when i change to cpu, that's work. it's mean scala+gpu+mac not work

does windows support mxnet along with scala , if so then how  ?


Bumped version of sharp

I guess your PR is getting ignored, if you look at README:
This Project Is No Longer Maintained

Because of other occupation, I can't maintain it anymore. If you find this project usefull, please fork it. Please note that new issues / pr messages will be ignored.


[iOS][Action.ShowCard] ShowCard not expanding the card

Platform
What platform is your issue or question related to? (Delete other platforms).

iOS

Author or host
Host
If you're an author, who are you sending cards to?
1.2
What version are you using? Ex: NuGet 1.0.2, or latest master, etc...
Details
ShowCard action does not expand the card on adaptiveCards v1.2, seems to be working fine on 1.3
Inline is the payload
{"body":[{"items":[{"size":"medium","weight":"bolder","type":"TextBlock","text":"🎉🎉  Ment.io was installed in Teams Platform!"}],"type":"Container"},{"items":[{"type":"TextBlock","text":"Now that you installed Ment.io, you can create questions, collect answers with reasoning, pin the discussion over time, and get useful discussion analytics","wrap":true}],"type":"Container"},{"url":"https://urlp.asm.skype.com/v1/url/content?url=https%3a%2f%2fteamsimages.blob.core.windows.net%2fteamsimage%2fwelcomeimage.jpg","type":"Image","height":"auto"},{"type":"TextBlock","text":"Learn How To:","weight":"bolder"}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json","selectAction":{"title":"Pin in Tab","card":{"type":"AdaptiveCard","$schema":"http://adaptivecards.io/schemas/adaptive-card.json"},"type":"Action.ShowCard"},"type":"AdaptiveCard","actions":[{"title":"Ask","card":{"body":[{"items":[{"size":"large","text":"🤔Ask a question ","weight":"bolder","type":"TextBlock","spacing":"None"}],"type":"Container"},{"items":[{"size":"medium","spacing":"Large","text":"Create a new discussion by asking a question. Just click on the Ment.io App from the chat bar and press ➕ to add a new question","type":"TextBlock","wrap":true}],"type":"Container"},{"spacing":"Large","type":"Image","url":"https://urlp.asm.skype.com/v1/url/content?url=https%3a%2f%2fteamsimages.blob.core.windows.net%2fteamsimage%2fexpl_ask_new.jpg","height":"auto"}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json","type":"AdaptiveCard","version":"1.0"},"type":"Action.ShowCard"},{"title":"Answer","card":{"body":[{"items":[{"size":"large","text":"🙋Answer the question at hand","weight":"bolder","type":"TextBlock","spacing":"None"}],"type":"Container"},{"items":[{"size":"medium","spacing":"Large","text":"Direct the discussion by suggesting one of the possible answers. Start a comment thread and got input on your take.","type":"TextBlock","wrap":true}],"type":"Container"},{"spacing":"Large","type":"Image","url":"https://urlp.asm.skype.com/v1/url/content?url=https%3a%2f%2fteamsimages.blob.core.windows.net%2fteamsimage%2fexpl_answer.jpg","height":"auto"}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json","type":"AdaptiveCard","version":"1.0"},"type":"Action.ShowCard"},{"title":"Pin Tab","card":{"body":[{"items":[{"size":"large","text":"📌Pin a discussion in a tab","weight":"bolder","type":"TextBlock","spacing":"None"}],"type":"Container"},{"items":[{"size":"medium","spacing":"Large","text":"Make your discussion last, add it as a tab and jump into the discussion at any time","type":"TextBlock","wrap":true}],"type":"Container"},{"spacing":"Large","type":"Image","url":"https://urlp.asm.skype.com/v1/url/content?url=https%3a%2f%2fteamsimages.blob.core.windows.net%2fteamsimage%2fexpl_pin.jpg","height":"auto"}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json","type":"AdaptiveCard","version":"1.0"},"type":"Action.ShowCard"},{"title":"Explore","card":{"body":[{"items":[{"size":"large","text":"🤓Explore discussion insights","weight":"bolder","type":"TextBlock","spacing":"None"}],"type":"Container"},{"items":[{"size":"medium","spacing":"Large","text":"Track the progression of the discussion in the discussion tab, with analytics views for Answers, Discussions, and even Team analytics.","type":"TextBlock","wrap":true}],"type":"Container"},{"spacing":"Large","type":"Image","url":"https://urlp.asm.skype.com/v1/url/content?url=https%3a%2f%2fteamsimages.blob.core.windows.net%2fteamsimage%2fexpl_analytics.jpg","height":"auto"}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json","type":"AdaptiveCard","version":"1.0"},"type":"Action.ShowCard"}],"version":"1.0"}

@Aditya0603
You said "seems to be working fine on 1.3" What's 1.3? Our next version will be 1.2.4

@nesalang assigning to you for investigation; @Aditya0603 is this also being tracked for 1.2.* upgrade on the teams side?

@shalinijoshi19 ,
@Aditya0603 confirmed that it's no issue on the latest; I will mark it with low priority, and come back later, once teams finishes 1.2.4 integration.

@Aditya0603 / @VikrantSingh01 can you please confirm where/what branch/what iOS package version you are observing this bug on so we know to decide on our next steps?

@VikrantSingh01 ping on this one

@shalinijoshi19 @nesalang  : Aditya is O.O.F
"@Aditya0603 : You said "seems to be working fine on 1.3" What's 1.3? Our next version will be 1.2.4"
Can you check this on AC 1.2.3. I guess Aditya was trying this version.

@nesalang can we test this issue on our outbound 1.2.5 bits see if we can repro it today? Thanks

Here are the payload to use for bug bash
samples/v1.0/Elements/
Action.ShowCard.Style.json
Action.ShowCard.json
samples/v1.2/Tests/
NestedShowCard.json

it's no repro with 1.2.5 package.


Add another column with columnOption dosent work

I added another column to oracle named Appname which is clob. i'm trying to add columnOption that will add to every log row the app name as deafalut. when I add columnOptions to the WithSetttings(). it dosent writes the logs to the db at all.
can you help me? I did exacly the same code as the readme file.
const string column = "APPNAME";
var columnOptions = new ColumnOptions
{
AdditionalDataColumns = new List
{
new DataColumn(column , typeof(string))
}
};
Log.Logger = new LoggerConfiguration()
//.Enrich.FromLogContext() /* uncomment this line if you want to store dynamic values and passing them by LogContext.PushProperty(name, value)... remember, this PushProperty is Disposable*/
.Enrich.WithProperty("column", "my-app-name")
.MinimumLevel.Verbose()
.WriteTo.Oracle(cfg =>
cfg.WithSettings(logConnectionString,CIBUS.LOG ,columnOptions: columnOptions)
.UseBurstBatch()
.CreateSink())
.CreateLogger();

@AmitayBohadana did you found a solution for this issue , Cause I have the same issue


feat: how to monitor cortex inbound requests?

Problem
I'm testing my copilot alternative setup, but I could not debug the outbound requests there. It would be great if I can debug inbound requests from cortex cli. For now, it just outputs the request path, no request body is logged.
Success Criteria
E.g.
1723389668 - Request /chat/completions
 {
   "messages": [
      "role": "user",
      "content": "hello"
   ],
   "stream": true,
   "model": "llama3.1"
}

Additional context
Add any other context or screenshots about the feature request here.

Merged into #1153


Update README.md

fixed: grammatical error in readme.md

Thanks @yashgupta97, I will use this sugestion #32


Installation doesn't work

No matter what project I add the addon/autogrid folder to, it won't let me enable the plugin, stating:
"Unable to load addon script from path: 'res://addons/AutoGrid/core.gd'. This might be due to a code error in that script.
Disabling the addon at 'res://addons/AutoGrid/plugin.cfg' to prevent further errors."
It finds an error in every script in the AutoGrid folder:

in all files except core, it only finds 2 errors, both of which are solved by removing "tool" at the top of the script (this is also 2 of the errors in core)
in core, it finds 10 errors. 2 are the same as the above, 6 of them seem to be about "yield" being removed from godot 4, and the last 2 seem two (lines 10 and 21) seem to only have the use of "setget" in common, so maybe that's causing it?

Here's the core.gd error messages:
Line 1:Unexpected "Identifier" in class body.
Line 2:Unexpected "extends" in class body.
Line 10:Expected end of statement after variable declaration, found "Identifier" instead.
Line 21:Expected end of statement after variable declaration, found "Identifier" instead.
Line 146:"yield" was removed in Godot 4. Use "await" instead.
Line 146:Expected statement, found "(" instead.
Line 157:"yield" was removed in Godot 4. Use "await" instead.
Line 157:Expected statement, found "(" instead.
Line 1261:"yield" was removed in Godot 4. Use "await" instead.
Line 1261:Expected statement, found "(" instead.

I could try fixing those seemingly simple errors myself, but I don't know if that would break things further, especially since I really have no idea what I'm doing. I'd rather not risk it.
The only exception to all this is when I load the demo project - then it works perfectly fine. But if I put it into my own project, or into a demo/learning project by Godot, then it doesn't work.
The version I'm using is 4.2.2.
What should I do?

This addon first made for Godot 3.x, I think you downloaded the Godot 3.x version, can you make sure to download from here?
Note: There is still some visuals missing in Godot 4 version, I am working on it.

I tried downloading from the embedded link, but it still gave me the exact same scripts. Is there a chance you accidentally uploaded the Godot 3 version?

I have published new release (v1.3) with some small improvements. Can you try that too? Can you share the errors or screenshot of the errors with me If it doesn't work?

I'll get back to you when I can, in a game jam rn

The addons folder isn't updated, only the demo project is

@AlbinoElk Thanks, I forgot to update that. I will probably remove that addons folder in the future to avoid confusion. Regardless, I recommend to download from releases.

Hello, I did download from release the godot 4.X, but the code in core.gd is very similar to the one in 3.X (no @ before tool, yield still there.
Did you upload files from 3.X into the 4.X by accident?

@Ryuggo Thanks for reporting the release version was wrong. I have updated the releases for Godot 4 version

Does it work for 4.2.2? The bitmap boxes don't show up.

@MTandi You have to create bitmask first. You can create it either with "AutoGrid" button which located on top menu or use the shortcut "Shift + T" while MeshInstance node is selected.

I did try to create it, no visual indication that it was successful.

@MTandi I fixed it, and updated the release file. I didn't get that error while testing. Thanks for reporting.


Pre-processing of files

There are times when a file type (I'm looking at you NPM CAS <.<) produces many many instances of a simple string. Something like an IP resolution/lookup saying IP <IP_ADDRESS> resolved to localhost.
These events in the logs are entirely useless but also incredibly insecure when dealing with private systems as it will give away both ips and URLs. There needs to be a way to deal with these strings without bloating out rules and making it longer to check if we already have a record of that IP.
Pre-processing of files to remove this type of line/information will speed up the processing as well. Each key we find increases the amount of time it takes to check the next key is new. when we have 100k addresses in a log file the time is increased dramatically and without many solutions.
This feature will go in after the new config file is implemented.

Rules in the config that look like this: "regex rule"=\delete now designate lines that should be removed entirely from the log.


SVN repository has no source code

From<EMAIL_ADDRESS>on March 04, 2007 11:21:51
What steps will reproduce the problem? 1. Perform SVN Checkout with TortoiseSVN on Windows. Repository URL: http://google-ctemplate.googlecode.com/svn/trunk What is the expected output? What do you see instead? I expected to receive the latest bleeding edge perftools source code, but
all I got were documentation files. What version of the product are you using? On what operating system? I am using TortoiseSVN 1.4.1 on Windows XP Professional Please provide any additional information below. The file attached is the list of files, including the SVN files, that I got
from the checkout.
Attachment: files.txt
Original issue: http://code.google.com/p/gperftools/issues/detail?id=6

From<EMAIL_ADDRESS>on March 21, 2007 22:14:40
Finally got some time to do this!  All old versions should be in there, as well as
the current one.
Status: Fixed


Tls server components

What was changed

Created more TLS assets for the frontend and internode configurations to enable encrypted communication channels.
Configuration changes necessary to enable encryption at the server components.
Why?

I need a local environment that is configured as close as possible with our production setup.
I hope others do too.
Checklist

Closes

How was this tested:

tctl can connect locally
tctl can connect from admin-tool container
local worker clients can connect over localhost with TLS
tui(temporal-ui) shows triggered workflows etc

Any docs updates needed?
doc updated here

Closing due to inactivity, feel free to re-open if you'd like to continue the process of getting this merged.


Choose a build system

As the project increases in size, it is sensible to use some kind of build system to do the build, rather than manually invoking the compiler. There are plenty to chose from, some common ones are:

Automake
Bazel
CMake
Meson

Which one you choose is not super important, but the sooner you do, the easier it will be to add things like CI builds, third-party libraries, automated tests and that sort of thing.
If one is chosen, I'd be up for implementing it in this project for you.

I have experience with CMake (and visual studio projects, but that's probably not a good way to go on anything that you hope might be cross-platform at any point!), so I could do that easily. Wouldn't mind investigating any others though.
Actually, I've been looking for an excuse to do something with https://build2.org/ at some point :)

Thanks a lot for the suggestion, I've been coding heavily for the core functionality. If you'd like to contribute based on your experience, please feel free to help, I'll merge proper PRs.
Currently I'm working on decouple the client side code and implement the long message function, so I may choose a build system when the code is better organized.
Many thanks.


EQUATION EXPLORER: Allow searching/displaying any equation

With Bruno's new fast algorithm for mapping equation to id and back, there's no reason the equation explorer can't handle arbitrary equation numbers or formulas, even if we don't have implications it could still be useful as a web-based find_equation_id.

This can be combined with #378.

It could be, though they're really two separate tasks.

Note that my python code is efficient in almost all cases, with the exception of equations with the same shape on the left-hand side and right-hand side: then it has to list all equations in order (basically).  This could probably be solved if there is appetite for it, but the combinatorics of how to count equations up to isomorphism in that case were a bit involved.


lxd recover: Instance "..." has a different instance type in its backup file ("")

Required information

Distribution: Ubuntu
Distribution version: 18.04 copying to 20.04

Original server:
  driver: lxc
  driver_version: 3.0.3
  kernel: Linux
  kernel_architecture: x86_64
  kernel_version: 4.15.0-171-generic
  server: lxd
  server_pid: 2805
  server_version: 3.0.3
  storage: zfs
  storage_version: 0.7.5-1ubuntu16.12
  server_clustered: false
  server_name: source_server
  project: ""

New server:
  driver: lxc | qemu
  driver_version: 4.0.12 | 6.1.1
  firewall: nftables
  kernel: Linux
  kernel_architecture: x86_64
  kernel_features:
    netnsid_getifaddrs: "true"
    seccomp_listener: "true"
    seccomp_listener_continue: "true"
    shiftfs: "false"
    uevent_injection: "true"
    unpriv_fscaps: "true"
  kernel_version: 5.4.0-109-generic
  lxc_features:
    cgroup2: "true"
    core_scheduling: "true"
    devpts_fd: "true"
    idmapped_mounts_v2: "true"
    mount_injection_file: "true"
    network_gateway_device_route: "true"
    network_ipvlan: "true"
    network_l2proxy: "true"
    network_phys_macvlan_mtu: "true"
    network_veth_router: "true"
    pidfd: "true"
    seccomp_allow_deny_syntax: "true"
    seccomp_notify: "true"
    seccomp_proxy_send_notify_fd: "true"
  os_name: Ubuntu
  os_version: "20.04"
  project: default
  server: lxd
  server_clustered: false
  server_name: new_server
  server_pid: 881
  server_version: 4.0.9
  storage: dir
  storage_version: "1"
  storage_supported_drivers:
  - name: zfs
    version: 0.8.3-1ubuntu12.13
    remote: false
  - name: ceph
    version: 15.2.14
    remote: true
  - name: btrfs
    version: 5.4.1
    remote: false
  - name: cephfs
    version: 15.2.14
    remote: true
  - name: dir
    version: "1"
    remote: false
  - name: lvm
    version: 2.03.07(2) (2019-11-30) / 1.02.167 (2019-11-30) / 4.41.0
    remote: false

Issue description
I copied a running LXD container on Ubuntu 18.04 (zfs storage) by tarring up the files in /var/lib/lxd/storage-pools/default/containers/mycontainer and copying to /var/snap/lxd/common/lxd/storage-pools/default/containers/mycontainer on a new server.
I also edited the backup.yaml file to change from a ZFS storage pool to a directory pool (I set up a new LXD container in the 'dir' pool and copied the pool: fragment from backup.yaml across).  I then ran lxd recover:
# lxd recover
This LXD server currently has the following storage pools:
 - default (backend="dir", source="/var/snap/lxd/common/lxd/storage-pools/default")
Would you like to recover another storage pool? (yes/no) [default=no]:
The recovery process will be scanning the following storage pools:
 - EXISTING: "default" (backend="dir", source="/var/snap/lxd/common/lxd/storage-pools/default")
Would you like to continue with scanning for lost volumes? (yes/no) [default=yes]:
Scanning for unknown volumes...
Error: Failed validation request: Failed checking volumes on pool "default": Instance "mycontainer" in project "default" has a different instance type in its backup file ("")

I eventually worked out how to fix this by adding:
  type: container

to the container: section in backup.yaml, and then the container imported.
This should probably be a default value so that older containers will import.

Yes agreed. We added something similar for the instance DB record recently, but should do the same for volume DB record type too.
https://github.com/lxc/lxd/pull/10209

Actually looking at the error report from https://github.com/lxc/lxd/pull/10209 (https://discuss.linuxcontainers.org/t/recovery-from-errant-lxd-init/13768?u=tomp) it seems like they were getting the same error as you, which means that https://github.com/lxc/lxd/pull/10209 will fix it. This should be in LXD 5.1 and LXD 5.0.1 when they are released.


Infinite reconciliation loop (rawstatuscollection)

What this PR does / why we need it:
Federated resources that have target resources without status field are constantly evaluated as "changed", if RawStatusCollection is Enabled. This triggers an infinite loop when multiple FederatedResources are created at the same time in the cluster.
Also, this PR applies to collectedStatuses the same transformation that is applied to the FederatedResource status (json.Marshal - json.Unmarshall), so to prevent avoidable changes (and reconciliations) to the FederatedResources.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1373
Special notes for your reviewer:

@cebernardi Did you manage to figure it out the issue with the tests ?

@cebernardi Did you manage to figure it out the issue with the tests ?

tests are failing because of
1.15.3: Pulling from library/golang
 library/golang
docker: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit.

I'm trying to retrigger the build..

@cebernardi I can do that for you.

@cebernardi I can do that for you.

yes please @hectorj2f ...
It just failed again for the same reason...

@cebernardi Done, we now need you to sign the cla.

@hectorj2f let me check with my company, they should have already signed it, and they might be in the process of returning it..

@hectorj2f the CLA is signed.. is there a way to trigger again the check?

@cebernardi No, I don't know it. Perhaps close-and-reopen the PR. I am not sure.

/check-cla

double checked again.. CLA was signed BUT not returned to Linux Foundation..

/check-cla

Too many commits. I think you need to merge the commits into one.

@cebernardi i tested your changes and they work. We need a proper commit log, and address your signing issues.

/check-cla

/lgtm


Prepare version 1.1.0

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-07-07T13:15:38.950+0000

Duration: 7 min 40 sec

Commit: 904c89740672d0e0fbcda830f0cba3af74074979

Test stats :test_tube:

Test
Results

Failed
0

Passed
171

Skipped
0

Total
171

Trends :test_tube:


Module 1.4 assignment

👋 Hi @mrchmielewski, in case you'd like to try it again, the goal for 1.4 is to go to http://git-school.github.io/visualizing-git/ and to re-create the image that Eric has created here: https://github.com/github-campus-advisors/Campus-Advisor-Training/blob/master/Module 1/Module 1.4 Assignment.md

Oh, that's actually easier. Here it is!


Error index.js not found

Error index.js not found
$ npm start

<EMAIL_ADDRESS>start
node .

node:internal/modules/cjs/loader:456
throw err;
^
Error: Cannot find module 'C:\laragon\www\laravel-whatsapp-server\backend\dist\index.js'. Please verify that the package.json has a valid "main" entry
at tryPackage (node:internal/modules/cjs/loader:448:19)
at Module._findPath (node:internal/modules/cjs/loader:692:18)
at resolveMainPath (node:internal/modules/run_main:15:25)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:24)
at node:internal/main/run_main_module:23:47 {
code: 'MODULE_NOT_FOUND',
path: 'C:\laragon\www\laravel-whatsapp-server\backend\package.json',
requestPath: 'C:\laragon\www\laravel-whatsapp-server\backend'
}

npm run build then npm start


Java 9 additions

Fixes parsing of the version number (if the version number is just 9 or 10 for the .0.0 releases).
Updates to GC-logging config for Java 9 and newer (applies to Java9 branch)

Looks good, but I'm going to do a very mean thing. Can you please add a test for the get_jdk_version method? That covers jdk8 and jdk9 syntaxes for the version. Thank you

Okay, added a unit test


⚠ Minecraft has degraded performance

In d37fd05, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1336 ms

Resolved: Minecraft performance has improved in 25c4631.


Use Flow for JS Type Checking

Flow http://flowtype.org/ is an intersting tool that does Javascript type checking. This can help find a lot of errors that are hard to spot initially. It does not require you to do anything different to your code than add a comment at the top.  Eventually it can support transitioning to a more explicit typing.

This looks really cool, but I don't think we want an active dependency on another programming language, or handle the grunt integration for a complex tool in this project.

Closing old issues that I do not expect to be tackled in the context of this project.


Part 7 of Async Compaction : Ensure Compaction Operation compacts the data file as defined in the workload

Part 7 of Async Compaction : Ensure Compaction Operation compacts the data file as defined in the workload

@n3nash @vinothchandar : Will be fixing the failing test (passes locally). But, ready otherwise for review

A refactor may be needed for the datafile argument, rest LGTM.

Committed as part of Consolidated diff


scipy.misc.imsave is depricated

https://github.com/udacity/deep-learning-v2-pytorch/blob/3a95d118f9df5a86826e1791c5c100817f0fd924/cycle-gan/helpers.py#L75
Currently, this piece of code throws an exception module 'scipy.misc' has no attribute 'imsave'
Possible cause scipy.misc.imsave is deprecated.
Possible fix: imageio.imwrite('filename.jpg', array)

This should work with the version of scipy recommended for this course. However, it would be great if you could add a PR for this. We will review and approve the changes.


Qemu processes left behind for older versions of docker

Description of problem
Started a container with docker 1.13.1
$ docker run -it debian
# exit

Expected result
With no other containers , ps aux | grep qemu should show no qemu processes around
Actual result
Qemu processes still left behind.
Upgraded to docker-ce 1.17.09. Problem goes away. Something seems to be inconsistent in the way kill/delete is handled. We need to fix this for older versions of docker. Have seen this with Docker 1.12 and 1.13

I see the same, and further, I see the containers in the stopped state when executing cc-runtime list:
# cc-runtime list
ID                                                                 PID         STATUS      BUNDLE                                                                                CREATED                          OWNER
2dca2fad5eeee801db2e30ccfb90932dcde8c3901da62c7889f4d0e21b84220a   231284      stopped     /run/docker/libcontainerd/2dca2fad5eeee801db2e30ccfb90932dcde8c3901da62c7889f4d0e21b84220a   2017-12-07T19:25:04.582622533Z   #0
91012f3e960d43347c46c52461a8624fe9d1146d13f31b4ffcd47f837286f862   231141      stopped     /run/docker/libcontainerd/91012f3e960d43347c46c52461a8624fe9d1146d13f31b4ffcd47f837286f862   2017-12-07T19:24:56.093629951Z   #0

I'm seeing something similar or maybe the same on docker 17.09.0-ce.
When running metrics tests in a loop or under the CI, every now and then the 'unexpected qemu' error triggers:
https://github.com/clearcontainers/tests/blob/master/metrics/lib/common.bash#L269
I've seen this a few times, but have not had time to sit down and see if it is reliably reproducible.
If I were to do that, I'd probably start by running this:
https://github.com/clearcontainers/tests/blob/master/integration/stability/soak_parallel_rm.sh
and see how that fairs - it has a pretty good track record of catching such things.


TOMEE-2583 - add translate in PT to examples/datasource-versioning

ticket of Jira = https://issues.apache.org/jira/browse/TOMEE-2583

Nice job @robinsonvs


run container as non-root

PR:

Sets user and group IDs as 1000, user is called netchecks
Installs poetry in ~/bin/, since /opt/ isn't writable to non-root users
Added missing --no-cache to the cli Dockerfile—should be even smaller now!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
216

Relevant Lines:
240

💛  - Coveralls


Updated Docker Compose and Docs

So I updated the docker compose to make it easier for people to just have to create a .env file with API_URL , DOCKER_HOST, and SWAGGER_HOST setup in it see below for my example:
API_URL=http://<IP_ADDRESS>:8000
DOCKER_HOST=<IP_ADDRESS>
SWAGGER_HOST=<IP_ADDRESS>

I also updated the docs to fix some typos and wrong ports and to update with the instructions to use this.
Let me know what you think

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

Yea I’ll fix all these when I get home tomorrow I didn’t realize I had some of this still not finished correctly so I’ll fix them, thanks

Thanks for the PR, agree it's a worthwhile change. Thanks for working through it with us.

Np sorry I'll get it cleaned up tomorrow when I'm home so I can test everything on my setup.

I'm testing out my changes I have updated now will push something up here shortly after I've got it all working and updated right

ok all changes have been updated and the system now works flawlessly on docker for windows which uses a <IP_ADDRESS> IP.  You can see the screenshot below, let me know if there is anything else we want to change.

I also updated the docs so they should look correct now as well let me know

DOCKER_HOST is reserved environment variable in some docker-machine on windows 7.
If you view the effective configuration with your setup with command docker-compose config
and the use the provided .env file, you will get this.
networks:
  alpha-nlu-network: {}
services:
  api:
    entrypoint:
    - node
    - start.js
    environment:
      DOCKER_HOST: tcp://<IP_ADDRESS>:2376
      DUCKLING_URL: http://duckling:8000
      MODEL_DIR: ./local-storage/rasa/nlu-model
      RASA_URL: http://rasa:5000
      REDIS_HOST: redis
      REDIS_PORT: '6379'
    healthcheck:
      test:
      - CMD
      - curl
      - -f
      - http://tcp://<IP_ADDRESS>:2376:8000/agent
    image: samtecspg/articulate-api:repo-head
    networks:
      alpha-nlu-network: null
    ports:
    - <IP_ADDRESS>:8000:8000/tcp
    volumes:
    - /c/GIT/articulate/local-storage/rasa/nlu-model:/app/projects:rw
  duckling:
    image: samtecspg/duckling:<IP_ADDRESS>
    networks:
      alpha-nlu-network: null
    ports:
    - <IP_ADDRESS>:8500:8000/tcp
  rasa:
    image: samtecspg/articulate-rasa:repo-head
    networks:
      alpha-nlu-network: null
    ports:
    - <IP_ADDRESS>:5000:5000/tcp
    volumes:
    - /c/GIT/articulate/local-storage/rasa/nlu-model:/app/projects:rw
    - /c/GIT/articulate/local-storage/rasa/rasa-config.json:/app/config.json:rw
  redis:
    command: redis-server --appendonly yes
    image: redis:4.0.6-alpine
    networks:
      alpha-nlu-network: null
    ports:
    - <IP_ADDRESS>:6379:6379/tcp
    volumes:
    - /c/GIT/articulate/local-storage/redis-data:/data:rw
  ui:
    environment:
      API_URL: http://<IP_ADDRESS>:8000:8000}
    image: samtecspg/articulate-ui:repo-head
    networks:
      alpha-nlu-network: null
    ports:
    - <IP_ADDRESS>:3000:3000/tcp
version: '3.0'

So docker-machine will override url <IP_ADDRESS> with tcp://<IP_ADDRESS>:2376 which creates incorrect http://tcp://<IP_ADDRESS>:2376:8000/agent. Maybe LOCAL_HOST variable should be used instead.

Odd cause I was using this on windows and it works fine for me

If I run command docker-machine env default I will get:
export DOCKER_TLS_VERIFY="1"
export DOCKER_HOST="tcp://<IP_ADDRESS>:2376"
export DOCKER_CERT_PATH="C:\Users\z0034zpz\.docker\machine\machines\default"
export DOCKER_MACHINE_NAME="default"
export COMPOSE_CONVERT_WINDOWS_PATHS="true"
# Run this command to configure your shell:
# eval $("C:\Program Files\Docker Toolbox\docker-machine.exe" env default)

So this environment variable is present on the virtualbox image if you use docker-machine in windows 7.
 And docker commands should work in local git bash.

@btotharye if you are using windows 7. how did you mount the volumes locally? I am struggling with that. I am mounting everything from virtual machine and it is very tedious.

I am running docker-toolbox for windows 7. That creates virtualbox(default vm with docker setup) and adds git bash for docker terminal. Docker for Windows works for Windows 10 Profesional.

thats probably the difference is I am using docker for windows on Windows 10, let me test it out on the other windows platform and see what I can work out and I'll update it.

ok I have updated this if you can try @Johnz86 using DOCKER_INSTANCE as the env

@btotharye I tried it with docker instance and it now works for me.
Just for people that use docker-toolbox and windows 7, you need to do following:

Add local-storage from this project as shared folder for default vm machine in docker toolbox
In docker terminal create volumes and mount them

docker create volume local-storage
docker-machine ssh default "sudo sh -c \"echo $(docker volume inspect local-storage -f '{{.Mountpoint}}') \""

Then create .env file in the root of the project with config:
API_URL=http://<IP_ADDRESS>:8000
DOCKER_INSTANCE=<IP_ADDRESS>
SWAGGER_HOST=<IP_ADDRESS>
MODEL_DIR=/mnt/sda1/var/lib/docker/volumes/local-storage/_data/rasa/nlu-model
RASA_CONFIG=/mnt/sda1/var/lib/docker/volumes/local-storage/_data/rasa/rasa-config.json
REDIS_DATA=/mnt/sda1/var/lib/docker/volumes/local-storage/_data/redis-data

In the docker terminal run
docker-compose up

Visit web page http://<IP_ADDRESS>:3000 to create the bot via UI.


No validation loss shown in tensorboard

While I was fine-tuning T5 model, I found that there was no validation loss being shown in tensorboard. So I have no idea when to stop fine-tuning. Can you tell me how to track validation loss along with train loss, and when to stop fine-tuning phase?
Thank you.

Hello,
I didn't use tensorboard to keep track of the losses. Instead, I have computed specifics metric each #K checkpoints (\ie early stopping). You could do the same.
If you want to use tensorboard, try to have a look at this: https://github.com/google-research/text-to-text-transfer-transformer/blob/main/notebooks/t5-trivia.ipynb


Create test1

Overview
TL;DR

Questions
<If relevant, write a list of questions that you would like to discuss related to the changes that you have made.>
Next Steps
<If incomplete, create a task list of items that are still being worked on within the Pull Request.>
Review
<If complete, or ready for :eyes:, use @mentions for quick questions, specific feedback, and progress updates.>

km

Hello 👋🏻 It looks like you may be practicing using GitHub. That's great! But, this repository is in active use, and this may be distracting those who maintain it. I'm going to close this, but if you'd like to suggest a bug fix or feature, please open a new issue with the context and information in the body. Thanks!


Forwarding Google OAuth user information through nginx

I'm running oauth2_proxy and nginx in two docker containers, and it's working great as an authentication layer. However, I don't know how to reliably forward google user information (full name, profile picture) through to the endpoint.
In my oauth2_proxy config file, I have:
pass_user_headers = true
pass_host_header = true
pass_access_token = true
set_xauthrequest = true

In nginx, I have:
auth_request_set $user $upstream_http_x_auth_request_user;
auth_request_set $email $upstream_http_x_auth_request_email;
proxy_set_header Host $host;
proxy_set_header X-Email $email;
proxy_set_header X-Real-IP $remote_addr;
auth_request_set $auth_cookie $upstream_http_set_cookie;
add_header Set-Cookie $auth_cookie;

The user headers help, but are not all the information I'm looking for.
I could GET the desired info with an access token, but the pass_access_token doesn't work as is. Likely because I'm not forwarding it through nginx, but I can't find any documentation on appending the X-Forwarded-Access-Token.
Occasionally, there's a user cookie that I can parse to get the information. But, it's not always present and I can't figure out when it is/isn't available.
What am I missing?
Thanks,
Pat

I spent some time looking at this, but haven't gotten anywhere yet. Following the pattern, this should work:
auth_request_set $token $upstream_http_x_forwarded_access_token;
add_header X-Access-Token $token;

However, it doesn't. The code that sets headers is here, and it looks like it sets differently than X-Auth-Request-*. Is there a reason for this?

There are two modes in which oauth2_proxy works: original "proxy" mode, and nginx "auth_request" mode. For "proxy" mode, any extra info headers need to be included in the request made to the application, not in the response to the user's browser. In "auth_request" mode, no further request is made, and any extra info headers need to be included in the response to nginx. The option --set-xauthrequest only works for "auth_request" mode while the other --pass-* options only work for "proxy" mode.
In the short term, you could use oauth2_proxy in the original proxy mode.
New options, or new meanings for the existing options, would be needed to pass more information in the response for "auth_request" mode. Perhaps if --set-xauthrequest is enabled, then --pass-access-token should add a response header too. (The other --pass-* appear to be either redundant or inapplicable.)

Thanks for getting back to me. Could you clarify what's involved with running in original proxy mode? If I have to move away from nginx, then it seems like a lot of config.
However, it seems that I could get around it by patching rw.Header().Set("X-Forwarded-Access-Token", session.AccessToken) here.
More generally, it looks like the proxy already fetches user info from google. Maybe I could grab that and store as a header or cookie?

"original proxy mode" is often used with nginx as well, and that's even shown in the first diagram: https://github.com/bitly/oauth2_proxy#architecture
These are like lego pieces. You can put either one in front or behind. It's easy.
You could certainly add whatever information you want to response headers. Those changes would just not be likely to be accepted upstream.

It's likely my own ignorance, but re-ordering the legos seemed like it would be a headache (SSL termination, multiple endpoints, etc.). At least for now, I went the patch route. See below.
Regarding getting accepted upstream, do you think the diff below would be a viable pull request? It's a non-breaking change, and (in my opinion) makes pass_access_token = true; set_xauthrequest = true config act as expected.
oauth_token.patch
diff --git a/oauthproxy.go b/oauthproxy.go
index dd2b58e..19ed0e3 100644
--- a/oauthproxy.go
+++ b/oauthproxy.go
@@ -680,6 +680,9 @@ func (p *OAuthProxy) Authenticate(rw http.ResponseWriter, req *http.Request) int
 		if session.Email != "" {
 			rw.Header().Set("X-Auth-Request-Email", session.Email)
 		}
+		if p.PassAccessToken && session.AccessToken != "" {
+			rw.Header().Set("X-Auth-Request-Access-Token", session.AccessToken)
+		}
 	}
 	if p.PassAccessToken && session.AccessToken != "" {
 		req.Header["X-Forwarded-Access-Token"] = []string{session.AccessToken}

Dockerfile
FROM golang:latest

# Download oauth2_proxy source
RUN go get -d github.com/bitly/oauth2_proxy
WORKDIR /go/src/github.com/bitly/oauth2_proxy

# Copy over config and patch files
RUN mkdir -p /opt/oauth2-proxy/etc
COPY oauth_token.patch /opt/oauth2-proxy/etc/
COPY oauth2_proxy.conf /opt/oauth2-proxy/etc/

# Patch and build oauth2_proxy
RUN git apply /opt/oauth2-proxy/etc/oauth_token.patch
RUN go install github.com/bitly/oauth2_proxy

# Run the server
CMD oauth2_proxy -config=/opt/oauth2-proxy/etc/oauth2_proxy.conf

nginx.conf
# One of these for each authenticated endpoint
# If you have >3 endpoints, I recommend a pre-processing script
location /$YOUR_ENDPOINT/ {
    # Forwards to authentication proxy
    auth_request /oauth2/auth;
    error_page 401 = /oauth2/sign_in;
    # Once authenticated, sets cookies and headers for upstream parsing
    auth_request_set $email $upstream_http_x_auth_request_email;
    auth_request_set $token $upstream_http_x_auth_request_access_token;
    add_header X-Email $email;
    proxy_set_header X-Access-Token $token;
    proxy_set_header Host $host;
    proxy_set_header X-Real-IP $remote_addr;
    auth_request_set $auth_cookie $upstream_http_set_cookie;
    add_header Set-Cookie $auth_cookie;
    # Websocket upgrading
    proxy_http_version 1.1;
    proxy_pass_request_headers on;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection "upgrade";
    proxy_pass http://$FORWARD_ADDRESS;
}

I think those three lines, to make --set-xauthrequest and --pass-access-token work together if both are enabled, are reasonable. But note that I am not a maintainer of oauth2_proxy.

I'll make the PR and let the maintainers figure it out. Thanks for the help!


feat: GridForm disableFieldsOnSubmit

Overview

Adds disableFieldsOnSubmit feature + implements PropsProvider + useFieldContext hook, which lays the groundwork for more Modular Forms work.
Also fixes a weird thing where aria-required was only being added to showRequired fields rather than all required fields.

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: GM-331
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.45-alpha.7f5b20.0@codecademy/gamut-labs@19.23.8-alpha.7f5b20.0@codecademy/gamut@36.7.2-alpha.7f5b20.0@codecademy/monaco@0.3.2-alpha.7f5b20.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://612ce8c56c68e41a42240b5b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.45-alpha.3b06fc.0@codecademy/gamut-labs@19.23.8-alpha.3b06fc.0@codecademy/gamut@36.7.2-alpha.3b06fc.0@codecademy/monaco@0.3.2-alpha.3b06fc.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://612d2421d9efcc18327c61c6--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.53-alpha.764288.0@codecademy/gamut-labs@19.23.16-alpha.764288.0@codecademy/gamut@36.8.4-alpha.764288.0@codecademy/monaco@0.3.7-alpha.764288.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://612e31989a7a291ba885a63b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.53-alpha.d5ec42.0@codecademy/gamut-labs@19.23.16-alpha.d5ec42.0@codecademy/gamut@36.8.4-alpha.d5ec42.0@codecademy/monaco@0.3.7-alpha.d5ec42.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://612e33da2f79341c2929563e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.53-alpha.68a90d.0@codecademy/gamut-labs@19.23.16-alpha.68a90d.0@codecademy/gamut@36.8.4-alpha.68a90d.0@codecademy/monaco@0.3.7-alpha.68a90d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://612e54ffda61db373d06317b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.53-alpha.534c1f.0@codecademy/gamut-labs@19.23.16-alpha.534c1f.0@codecademy/gamut@36.8.4-alpha.534c1f.0@codecademy/monaco@0.3.7-alpha.534c1f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://612e8447d0416d211da2d573--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://612eaa6f0375c348b65f9e56--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.56-alpha.1e10f9.0@codecademy/gamut-labs@19.23.19-alpha.1e10f9.0@codecademy/gamut@36.9.2-alpha.1e10f9.0@codecademy/monaco@0.3.10-alpha.1e10f9.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://61321aa5dcfbb43f36e3884d--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.56-alpha.9d534a.0@codecademy/gamut-labs@19.23.19-alpha.9d534a.0@codecademy/gamut@36.9.2-alpha.9d534a.0@codecademy/monaco@0.3.10-alpha.9d534a.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://613221a0fece252377f78b82--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.57-alpha.cd3463.0@codecademy/gamut-labs@19.24.1-alpha.cd3463.0@codecademy/gamut@36.9.2-alpha.cd3463.0@codecademy/monaco@0.3.10-alpha.cd3463.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6132431df3740755c7fd29d7--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.58-alpha.460c18.0@codecademy/gamut-labs@19.24.2-alpha.460c18.0@codecademy/gamut@36.9.3-alpha.460c18.0@codecademy/monaco@0.3.11-alpha.460c18.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://613246c8dcfbb471ede38cd3--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.59-alpha.db39d3.0@codecademy/gamut-labs@19.24.3-alpha.db39d3.0@codecademy/gamut@37.0.1-alpha.db39d3.0@codecademy/monaco@0.3.12-alpha.db39d3.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://61325f1bbac5dd6685af37f1--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.60-alpha.de08e4.0@codecademy/gamut-labs@19.24.4-alpha.de08e4.0@codecademy/gamut@37.0.1-alpha.de08e4.0@codecademy/monaco@0.3.12-alpha.de08e4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://613261dddf6ebf51d122fa13--gamut-preview.netlify.appDeploy Logs


OCR0065: Create repo on Hugging face for all the datasets we have of OCR

Description:
So currently we have a lot of OCR data that we have annotated and all of those images are on s3 with each image as a single object and also the cvs files are all in s3. So to make the datasets easily usable and easily accessible I will creating the zip files with all the images and upload to the hugging face repo with the transcriptions and all with the data split or data distribution that eric used.
Completion Criteria:
All the Tibetan OCR data uploaded to Openpecha hugging face.
Subtasks:

[x] Lhasa Kanjur
[x] Lithang Kanjur
[ ] Derge Tenjur
[x] Norbuketaka
[x] Google Books
[x] Betsug data
[x] Durtsa data
[x] update the script for the special case of google books and norbuketaka data

note:
for the Norbuketaka and Google books, we already have a hugging face repo but without the data distributions so I am using that hugging face repo to create the new hugging face repo on Openpecha hugging face with the data distribution but without the zipped image file
Card Reviewer:

[x] @10kalden

https://huggingface.co/datasets/openpecha/OCR-Lhasakanjur

@ta4tsering kindly reach out to @gangagyatso4364 regarding how to combine the uchen dataset in one hugging face dataset.

Kindly add url to the dataset

https://huggingface.co/datasets/openpecha/OCR-Durtsa
https://huggingface.co/datasets/openpecha/OCR-Betsug
https://huggingface.co/datasets/openpecha/OCR-Lithangkanjur

https://huggingface.co/datasets/openpecha/OCR-Norbuketaka
above is the norbuketaka hugging face repo and below is the missing images from the data.
norbuketaka_missing.txt
and below is the data distribution used by eric.
norbuketaka data distribution


Can't move label on Bubble Chart if playing

While playing the BubbleChart, it's not possible to drag the labels of selected countries

This was fixed by 8f17dd2f6f2fb23883966b4712b4179cf5ccf225


Enhance the test coverage part-1 : dubbo-cluster module

Add test case: dubbo-cluster module

Lets's add them to project when finished.

Lets's add them to project when finished.

Added

Claim

close via #1885


Airplane screen improvements

Primary goals

[x] Add all airplanes by model modal
[x] Add toggle to label airplane by condition
[x] Multiple airplanes purchase
[ ] Sell all unused airplane by model
[x] Swap airplane with another free airplane (retain schedule, config, base setup etc)

Extended goals

[x] Drag and drop for config and base

Did not implement sell all unused as selling airplane is not that bad right now


Display "Checkout with PayPal" button on checkout rather than "Buy with PayPal"

On the cart page when the option "Display shortcut in the cart" option is turned on, the module displays "Buy with PayPal".
(Display shortcut in the cart)

(Cart)

This makes sense in the context of a product, but on the cart page, it would make more sense to the customer if it displayed "Checkout with PayPal" or "Express checkout with PayPal".
E.g. the button at /views/img/ECShortcut/us/checkout.png:

Hello,
Thank you for your message.
We are working on the new version of the module integrating the customization of the shortcut buttons and we will take your idea into account.
This new version will be available in October 2020.
Best regards,

Hello,
The official release of the v5.3.0 is available on github. The personnalisation options have been added for shortcuts.
It will be also available on prestashop.addons soon.
Have a nice day !
Best regards,
Tetiana Vorgul


setting labels based on who responds

this comment should replace the current set of labels with Waiting on user feedback

jk it should work this time? :eyes:

This looks quite neat--eggxellent work!


Replicated Django WebAppWithTests folder

@vagisha-nidhi Is this folder same as python/django/webapp? Why do you want to replicate?

@bishal-pdMSFT Yes this is the same. I will add tests folder for Python webapp in this,

@bishal-pdMSFT I have pulled in from latest master and validated the folder contents for this PR.


samples: modules: lvgl: add keypad and encoder demo

Adds the LVGL keypad and encoder demo to sample build.

Nice! Didn't know about this sample :)
@faxe1008 it almost feels like we should really get the "basic" sample back to a minimal one like we used to have if demos allows folks to quickly try out more complex scenarios, inc. input, on their boards?  But then there's also things like #67508 that can be quite useful ...

@everedero thanks for your submission, indeed this sample was missing :^). Overall looks fine to me, but I was wondering if an overlay might make sense here because this sample does not work "out of the box" with the native_sim as the sample readme would suggest. Using zephyr,native-linux-evdev could be an elegant solution for this.
@kartben agree on this. After this PR is merged pretty much all indev are covered in other samples (except the button indev one) and could be removed, making the sample more basic once again 👍

@faxe1008 That would be nice indeed, I will try to add this.

Will be done in PR #68168


Agreement holder getting 'unexpected error' on attempting to sign

Describe the bug
A clear and concise description of what the bug is.
Expected Behaviour
A clear and concise description of what you expected to happen.
Actual Behaviour
A clear and concise description of what you expected to happen.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@micheal-w-wells Any more details for this one? ie. is this for amendments/initial RUPS? Or does it occur on all plans? Is this an issue you're seeing in test, locally, or in production? Steps to reproduce would be helpful, as I cannot seem to hit this issue locally.


Error from tutorial when setting the connection string on the IoT Edge device

When running the below command with my properties filled in it errors out.
az vm run-command invoke -g IoTEdgeResources -n EdgeVM --command-id RunShellScript --script '/etc/iotedge/configedge.sh "{device_connection_string}"'
I get the following error even though the VM I created was following the steps in the prerequisites section:
Deployment failed. Correlation ID: 8c259324-ce83-4433-bce2-5dec20c4897a. VM has reported a failure when processing extension 'RunCommandLinux'. Error message: "Enable failed: failed to execute command: command terminated with exit status=127
[stdout]

[stderr]
/var/lib/waagent/run-command/download/1/script.sh: 2: /var/lib/waagent/run-command/download/1/script.sh: /etc/iotedge/configedge.sh
HostName=erwelch-hub.azure-devices.net;DeviceId=myEdgeDevice;SharedAccessKey=<removed-for-security>: not found
".

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e2289383-6ce2-a036-c710-7d1fad6e1700
Version Independent ID: 3b902941-0876-17c3-3f0b-a4b7f7770531
Content: Quickstart create an Azure IoT Edge device on Linux
Content Source: articles/iot-edge/quickstart-linux.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@erwelch Thanks for the feedback! We are currently investigating and will update you shortly.

@erwelch Can you please re-register an IoT Edge device and follow the steps again?

@erwelch I retested this with a brand new VM and couldn't reproduct your error. Here's what a successful output looks like:
kelly@Azure:~$ az vm run-command invoke -g IoTEdgeResources -n EdgeVM --command-id RunShellScript --script '/etc/iotedge/configedge.sh "HostName=KellyHub.azure-devices.net;DeviceId=myEdgeDeviceTest;SharedAccessKey={key}"'
{
  "value": [
    {
      "code": "ProvisioningState/succeeded",
      "displayStatus": "Provisioning succeeded",
      "level": "Info",
      "message": "Enable succeeded: \n[stdout]\n Mon Mar 18 22:38:51 UTC 2019 Connection string set to HostName=KellyHub.azure-devices.net;DeviceId=myEdgeDeviceTest;SharedAccessKey={key}",
      "time": null
    }
  ]
}

Looking at the error message, it says that the script wasn't found. Can you check on your VM that the configedge.sh file exists in /etc/iotedge/? That file only exists in the IoT Edge on Ubuntu VM image. If you used a different image, then the script won't be found.

I deleted my resource group and started over following every step and still get the same error. How are you deploying the VM? I'm using the command from the file:
az vm create --resource-group IoTEdgeResources --name EdgeVM --image microsoft_iot_edge:iot_edge_vm_ubuntu:ubuntu_1604_edgeruntimeonly:latest --admin-username azureuser --generate-ssh-keys --size Standard_DS1_v2

I ssh'd into the VM and I do see the configedge.sh file. I also see the config.yaml and installedge.sh files in the same directory.

Yes, I just double double checked by copying all the commands directly from the article and only changing resource names.
When you SSH'd into the VM, did you try running the command directly on the machine? From the iotedge directory, run configedge.sh "<your_iothub_edge_connection_string>". This will help us narrow the troubleshooting down to either the VM/script or the RunShellScript connection.

@gregman-msft Did you ever see anything like this when you were creating the solution?

@kgremban - I was able to run the command when I ssh'd!

@kgremban I haven't seen this before, will need to do some offline investigation.

@erwelch What tool are you using to run these commands? PowerShell, Azure cloud shell, bash, etc?

@erwelch Following up to see if you're still experiencing this issue and want us to continue working to help you resolve it?

@AshokPeddakotla-MSFT can you #please-close this issue, and we can reopen it if @erwelch reports that they're still having trouble setting up the VM?


[DOC] Headless tutorial code causes error when trying to import icons

Is this issue related to an existing part of the documentation? If so, please provide a link
https://refine.dev/docs/tutorials/headless-tutorial/#adding-resources
Describe the thing to improve
After setting up tutorial as instructed (npx superplate-cli -o refine-headless tutorial), when trying to use icons as in the tutorial code
import { PostIcon } from "icons";

I am getting error:
Cannot find module 'icons' or its corresponding type declarations.ts(2307)
Please note I did not execute npm i @pankod/refine-simple-rest as I had used superplate to bootstrap the project.
My system info:
MacOS v12.6
node v16.17.0
nvm v8.15.0
From package.json:
"dependencies": {
    "@pankod/refine-core": "^3.18.0",
    "@pankod/refine-react-router-v6": "^3.18.0",
    "@pankod/refine-react-table": "^4.7.2",
    "@pankod/refine-simple-rest": "^3.18.0",
    "@testing-library/jest-dom": "^5.12.0",
    "@testing-library/react": "^11.2.6",
    "@testing-library/user-event": "^12.8.3",
    "@types/jest": "^26.0.23",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "react-scripts": "^5.0.0",
    "web-vitals": "^1.1.1"
  },
  "devDependencies": {
    "@types/node": "^12.20.10",
    "@types/react": "^18.0.15",
    "@types/react-dom": "^18.0.6",
    "autoprefixer": "^10.4.12",
    "postcss": "^8.4.16",
    "tailwindcss": "^3.1.8",
    "typescript": "^4.7.4"
  },

Describe the solution (optional)
Update tutorial with missing package, maybe?

Hey @vijayganeshpk,
The tutorial assumes your project is configured for absolute imports. So you should configure a tsconfig.json with baseUrl setting pointing to the /src directory in your project.
You may not be able to see the PostIcon in the document because it is collapsable. Actually, import { PostIcon } from "icons" under src points to a icons.ts file like the one below:
export const PostIcon = (
    <svg
        xmlns="http://www.w3.org/2000/svg"
        width="20"
        height="20"
        viewBox="0 0 24 24"
        fill="none"
        stroke="currentColor"
        strokeWidth="2"
        strokeLinecap="round"
        strokeLinejoin="round"
    >
        <polygon points="12 2 2 7 12 12 22 7 12 2"></polygon>
        <polyline points="2 17 12 22 22 17"></polyline>
        <polyline points="2 12 12 17 22 12"></polyline>
    </svg>
);

Hey @vijayganeshpk, thank you for your feedback. We're sorry to hear that you had a problem while following the tutorial. @salihozdemir shared the content for the file and npx superplate-cli -o refine-headless tutorial comes with basePath set to src already but I have to admit, Show PostIcon section is really easy to skip when you are following the tutorial so I added a bit more contrast and an explanation to avoid users missing out this section.
Opened a PR for this here

@salihozdemir / @aliemir thank you.
Sorry, my bad!
I missed it due to not looking properly at the Show PostIcon expansion box. Maybe I assumed it was a sample data from server, similar to the previous Show response section.


vendor: update errcheck

Reason for upgrade explained in #198.
I've added all files, including optional ones (tests, README) - these may be removed if you like.

Sorry to mess you around, but can you please use gvt to update the vendored code? It is largely automatic: gvt update <package>. This is my fault; I thought I had documented this in the README.

Sure, I'll update the PR now.

Done. But I'm not sure is it ok to not include https://github.com/kisielk/gotool/blob/master/LEGAL

Not ideal, but we can add it in if someone complains. Thanks!
PS. I've updated the CONTRIBUTING file with instructions for gvt.


DBAAS 366 Show empty state when there is no provider account

Jira: https://issues.redhat.com/browse/DBAAS-366

LGTM, my only question, do we want to show them any UI when there is an error in fetchInstancesFailed?

There is already UI for that

LGTM, my only question, do we want to show them any UI when there is an error in fetchInstancesFailed?

There is already UI for that
@yzhao583 I meant if we are ok to show the Provider Account field even when there is an error?

Yes, I think we need that to allow users to switch provider account

@yzhao583 PTAL at merge conflict following merging Tommy's PR? thanks!


[zh-cn] sync reference/node content

content/en/docs/reference/node/kubelet-checkpoint-api.md
content/en/docs/reference/node/topics-on-dockershim-and-cri-compatible-runtimes.md

/lgtm
/approve


Add support for disabled notif according to feature

Task/Issue URL: https://app.asana.com/0/488551667048375/1205090813634262/f
Description
See attached task description.
Steps to test this PR
See attached task description.

Current dependencies on/for this PR:

develop

PR #3372   👈

This comment was auto-generated by Graphite.


feat: Update service config behavior

This PR is mainly adds the behavior we specified for the per-service-config. However, I also refactored some parts along this:

Added per service configuration

Only read if path is provided
Always written if path is provided

Reduced some of the information stored in the openapi document
Reduced duplicate defaults (ParsedGeneratorOptions)

The defaults specified in the cli/index.ts where specified in many other places as well, there should be no duplication anymore.

Moved tsConfig to be read only once in the beginning, not for every service separately
Updated package.json behavior as well.
Removed sdk prefix from metadata
Fixed occurrences of metaData => metadata

It would be nicer to split the PR to small pieces.

why the test-services in core are removed?

I changed the configuration a bit, so that in the unit test there are no readme's or additional files. We don't need them there. The test services are still there, though.

It would be nicer to split the PR to small pieces.

File is not generated.

As discussed approved under the condition that the open points are addressed so that the demo tomorrow can run on canary. I will do a rereview on main tomorrow morning.

Thanks, I adjusted everything let me know if you have additional findings (you can also comment on the closed pr)


Let's Encrypt for wildcard installs

I am able to use free Let's Encrypt certificates in saltcorn to allow for wildcard domains (ie wimala.livedata.au and cott.livedata.au) etc
I have to do this manually by inserting the keys into the saltcorn install which is a pain.
Can this be made part of the default configuration?

So... Did I put this in wrong place?


Document redis installation

Please make sure that you have checked the boxes:

[x] Review the Quickstart guide
[x] Search for both open and closed issues regarding the problem you are experiencing
[x] For permissions issues (Access Denied and credential related errors), please refer to the requisite docs before submitting an issue:
AWS, GCP, OpenStack, GitHub

Description of issue:
It seems redis is now required for Security Monkey's scheduler as of v1.0.0.  I don't see this documented anywhere.  It should be documented how to set this up:
yum install redis
systemctl enable redis
systemctl start redis

Is the following documentation insufficient: https://github.com/Netflix/security_monkey/blob/develop/docs/autostarting.md#the-scheduler ?

That was how I identified that redis needed to be installed.  I think because the docs show how to setup the postgres server, they should also show setting up redis (https://github.com/Netflix/security_monkey/blob/develop/docs/installation/03-install-sm.md).  If the user wants to use an RDS instead of postgres, they can, and if they want to use ElastiCache instead of the local redis they can.

We added a blurb about this in the Quickstart guide.
Namely, the Redis part is more for production deployments vs. getting things initially up and running.


fix: calculate fiat amount from correct value

@lisabaut it has something to do with that.
The wrong fiatValue has been used there. fiatValue is the fiat value of the suggested budget and NOT the fiat value of the payment amount.  The suggested budget is 10 times the invoice amount.