Convert LaTeX to markdown and generate printable versions (PDF)

The LaTeX documentation isn't very contributor-friendly and isn't rendered by GitHub so switching to markdown would be better. This would mean quite a bit of effort, but can be broken up into chunks. Once the markdown is generated, we can set up hooks to convert those files into PDFs for convenient printing.

For converting from markdown to pdf, pandoc looks pretty good, though you might have to tinker with it.

install pandoc from deb here, to get the latest: https://github.com/jgm/pandoc/releases/tag/2.10
pandoc README.md  -t html5 -o out.pdf

Have to install the .deb from https://wkhtmltopdf.org/downloads.html first

On setting up hooks...
There's some useful info about git commit hooks here: https://www.atlassian.com/git/tutorials/git-hooks
I was thinking that would be a good way to go to automatically produce the pdf docs from markdown upon commit (probably with a pre-commit hook), but I think it's overkill for now. The problem: you want to look at the pdf-rendered-from-markdown before you commit it, to make sure that the doc still looks good given whatever changes you've made. I don't think hooks can be used completely before you start a commit. At that point, all you're really doing is running a bash script anyway, it's not really a "hook" anymore.
I do think, though, that if we're treating the markdown as the Truth document, we could have a hook that checks to make sure there's a corresponding update to any .pdf whose .md has changed - just to make sure the pdfs do get updated (manually).
Probably best just to start with making the markdown, and then worry about automation later.

I'm willing to help take some of it (any section works for me). Should we try to divide the work?

Has the team looked into asciidoc? It was a markup language specifically designed for technical writing and documentation. It supports LaTeX and can also output to PDF with asciidoctor , using the asciidoctor-pdf  extension. GitHub also renders GitHub flavored asciidoc. We're currently using asciidoctor for maintaining documentation in the form of a static site that also generates pdf documents as part of its CI build on the SoCal Linux Expo A/V  repo.

Thanks for the input!
@SConaway we will start dividing up the work soon, thank you for offering to help!
@capsulecorplab the main reason for switching to markdown is to make the repo contributor and especially beginning contributor friendly. There are several builders who are using git and Linux for the first time. Would you say that asciidoctor fits that description?

I'd say that asciidoc would be a good fit, especially as it works with LaTeX.

@capsulecorplab Cool, I didn't know that was a thing. Hopefully it generates better looking pdfs than pandoc (at least, the default output from pandoc).
I don't want to pour cold water on it, but it would be good to know the pros/cons vs github-flavored markdown. The biggest benefit of markdown would be that it's pretty common and people don't have to adopt a new tool. Is there much of the documentation that we're afraid wouldn't render well with markdown?
Note that at this point I'm less gungho on the need to output pdfs automatically (could always just print the github webpage to pdf in chrome)

If you're writing documentation excusively for the GitHub UI, then the choice between GitHub flavored markdown (GFM) vs. GitHub flavored asciidoc (GFA) is for the most part a matter of personal preference. In terms of pure markup languages, Asciidoc has been argued to have a lighter syntax and consistency compared to markdown (see Asciidoc vs. Markdown). If it's within the roadmap to produce a static site and/or generate other doc formats (s.a., PDF, DocBook, EPUB), then asciidoc(tor) is perhaps the better suited option with its rich set of features and extensions. Asciidoc is also supported on several static site generators.
Hope this helps.

So far I'm not fully sold on this. I've switched to a different markup language in the past Achllle/dual_quaternions: reStructuredText and found it to be a true PITA to have to refer to documentation the whole time. It looks like AsciiDoc is much closer to MarkDown when it comes to basic functionality though.
Let's take a look at what's in the LaTeX docs right now aside from the most basic markup functionality:

tables
images
images in tables
side-by-side images

And what's missing in the generated PDFs that we would like:

internal links and hyperlinks don't work in GitHub
links to other documents within the repo or other way to easily navigate between pages

Let me know if I'm missing anything. AFAIK GFM can do all of those things.
To reiterate the main concerns:

documentation has to be up-to-date
documentation has to be complete, detailed, and beginner-friendly
documentation has to be contributor-friendly

We hit 2) already because of the great work of @ejunkins. We don't hit 1) but we're slowly chipping away at that and the hope is that that will get accelerated when we hit 3).
I know this is contradicting the original issue, but to hit 1), we shouldn't be pushing builders to print the PDFs (we can't really keep those up-to-date 😉), so that would remove the need to convert to PDF.
So if PDF generation becomes less of a concern, the question is: Is GFM good enough to warrant decent navigation between pages or does asciidoc work better for that since we have the ability to generate a static website? With the criteria above, I'd be tempted to say that GFM is probably good enough.
Other considerations:

once we switch to GFM, moving to asciidoc doesn't seem too much of a stretch. The bigger move is LaTeX -> GFM/GFA
GFM has much more support: plenty of questions asked on StackOverflow, extensions for every IDE out there, things like a markdown table generator.
note that the main concerns don't require us to use GitHub/git. We could consider readme.io or community editable wikis.

This is a typical tooling conversation and engineers love to go all in on their favorite tool. I think either solution would be good. "Who merges the PR first makes the decision". So if you're reading this and convinced we should go one way, please open a PR! 🥇

To aid in the process, pandoc can be used for the initial migration, with humans checking and editing. It supports exporting LaTeX to Markdown or AsciiDoc.
Links to other pages work just like regular links: [title](folder/page.md)

Did a test to reassure myself on things. I'm honestly not as familiar with GFM as I should be :sweat_smile: . But I'm pretty happy with the image capabilities in GFM now.
Honestly GFM seems fine for the needs of the documentation in these repos. I feel like it'll maximize approachability and ease of keeping up to date.
Note that I count publishing a static site as a non-requirement here. But if that does become one in the future, then agreed, it wouldn't be that bad of a transition from GFM to something else. It's getting it in GFM in the first place that's big deal.

Note I've been working on updating the README in osr-rover-code, but I'm up for taking a build doc in the main repo too.

Work has officially started on the conversion process!
The first piece to be converted will be the Wheel Assembly instructions as they need to be overhauled anyways for the upcoming update to the drive motor assembly.
You can follow progress on this branch. I expect to be done in a few weeks at most (not a trivial conversion, trying to do two things at once here)
Instead of then converting markdown to PDF, it would be much easier to recommend folks to an existing 'save/print' extension, such as this one for example in chrome which lets you pretty print README pages.
Once that PR is well underway or merged, we will start the process of dividing up the remaining work. We will use project 1 to track progress.

Anybody landing here, definitely check out the project page before grabbing any of this work. We're Command and Conquering over there, not here.

@SConaway I'd like to assign https://github.com/nasa-jpl/open-source-rover/issues/212 to you, but for some reason I'm not able to find your username in the assignees drop down. Not sure if I need to add you to the repo somehow.

Hmmm. I might need to be in the project for that. If you don't want to do that, you always could put something like (assigned to SConaway).

I tried to assign him to it as well, it wouldn't let me @ him or add to assign i dont know why...

You can only assign issues to issue commenters (source). So @SConaway simply comment on it and one of the maintainers will be able to assign it to you.

@Achllle done there.
Besides the SW Controls, what else should I do?

@SConaway can you comment on https://github.com/nasa-jpl/open-source-rover/issues/217 too, so I can assign you? Probably enough for now.
This is annoying, github.

Btw @SConaway, definitely join the new slack: https://www.tapatalk.com/groups/jpl_opensource_rover/let-s-give-slack-a-try-t224.html

Also: made a new channel in the slack, "docs-overhaul" for discussion. Let's try keeping the chatter there instead of here.

Closing this issue since we've moved efforts to Project 1 and discussion to Slack.


TFKUBE-454: Restore EBS snapshot from index recovery file

Pull request description
Restore EBS snapshot from index recovery file with no need to full re-index. The snapshots for RDS and EBS should be match and EBS snapshot should contain index recovery files.
Also user should pass the date of EBS snapshot creation. This will be used to calculate the age of the snapshot to extend the lifecycle to avoid out of date error (by default the max valid age of snapshot is 2 days).
Checklist

[ ] I have successful end to end tests run (with & without domain)
[ ] I have added unit tests (if applicable)
[ ] I have user documentation (if applicable)

TODO: Documentation for the introduced variable (confluence_shared_home_snapshot_creation_date)

As long as all of main, change and edge index are in place and in sync with the records in database, Confluence will be recovered from index snapshot stored in shared home. The age of snapshot is not a problem here, since index recovery is not actually checking it during startup.
In terms of the three properties regarding reindex thread counts, they either already have the default value 4, or already doing some calculation to dynamically decide the value based on workload and available cpus. Seems like we're re-enforcing Confluence features. https://confluence.atlassian.com/conf713/recognized-system-properties-1077914597.html
Will close this PR for now, we can re-visit it later on if necessary.


Communication

Opening an issue for communication as requested

Should we open a Slack channel for collaborators?
This way all communication will be in one central place and will enable us all to see it.
It will also enable us to communicate in a more streamline kind of way.
Let me know what you all think :)

I'm on the fence personally. Isn't this issue board already a central place for communication? I am not sure I see the benefit of adding slack as a communication channel, when we'll also need to check these issues in addition to slack.
What if we give issues a little bit more of a chance to show they can do the job? If we still have people wanting slack, we could try it out.
Anyone feel differently?

@neurocollective I hear ya.
I thought it might allow quicker more stream lined communication like a chat, if anyone needs help with something specific..
But I have no special attachments for slack, to be honest, Im not a fan of it at all :)
So yeah, lets see if the need arises in the future :)

@KateDK thank you for floating the idea

RE: original post
Sorry, I was not clear:
https://github.com/neurocollective/juniordevframework/issues/20

RE: original post
Sorry, I was not clear:
#20

Understood! Thanks for the clarification.


Very nice docker images (x11vnc & octave desktop)

I like your docker images.
One small bug in this image. With latest software update python 2.7 you need
https://bootstrap.pypa.io/pip/2.7/get-pip.py instead of https://bootstrap.pypa.io/get-pip.py
Btw. your images work basically without changes also with Ubuntu 20.04

Thanks for the bug report! I have incorporated the fix in the master branch and will also update the master branch to 20.04.


Branching/PR process in readme

@karbassi this is just a draft but I would like to get your feedback on including something like this in our readme since a few people have asked me now directly for similar guidance

Ugh crap, I assumed these other changes were already included, maybe just cherry pick the last two commits related to the branching/pr content in readme (70d3441c8326a8f43c39a031bf3ac9ad4520e6d8 and c11eee1f77b529d1a883dd1cfdcd853bf074b143)


Bittrex sockets bug when subscribing too quickly - InvalidOperationException: Data cannot be sent because the connection is in the disconnected state.

I think this happens because the socket isn't fully connected before the subscribe message is sent. Not sure if the bug is in GetOrderBookWebSocket or in the Signalrmanager handling code.
2018-08-13 23:02:29,576 [12] INFO  Worker.Exchange - Populating markets for Bittrex
2018-08-13 23:02:31.3940|ERROR|ExchangeSharp.Logger|Error invoking hub proxy SubscribeToExchangeDeltas: System.InvalidOperationException: Data cannot be sent because the connection is in the disconnected state. Call start before sending any data.
at Microsoft.AspNet.SignalR.Client.Connection.Send(String data)
at Microsoft.AspNet.SignalR.Client.Hubs.HubProxy.Invoke[TResult,TProgress](String method, Action1 onProgress, Object[] args)    at Microsoft.AspNet.SignalR.Client.Hubs.HubProxy.Invoke[T](String method, Object[] args)    at ExchangeSharp.SignalrManager.SignalrSocketConnection.<OpenAsync>d__10.MoveNext() in D:\Git\CryptoArbs\ExchangeSharp\ExchangeSharp\Utility\SignalrManager.cs:line 92: System.InvalidOperationException: Data cannot be sent because the connection is in the disconnected state. Call start before sending any data.    at Microsoft.AspNet.SignalR.Client.Connection.Send(String data)    at Microsoft.AspNet.SignalR.Client.Hubs.HubProxy.Invoke[TResult,TProgress](String method, Action1 onProgress, Object[] args)
at Microsoft.AspNet.SignalR.Client.Hubs.HubProxy.Invoke[T](String method, Object[] args)
at ExchangeSharp.SignalrManager.SignalrSocketConnection.d__10.MoveNext() in ExchangeSharp\ExchangeSharp\Utility\SignalrManager.cs:line 92

Should already be connected at that point. AddListener calls ReconnectLoop which ensures the connection. But I might be missing something clearly.

Add more logging and we can see if things are happening out of order
On Tue, Aug 14, 2018, 08:25 Jeff Johnson<EMAIL_ADDRESS>wrote:

Should already be connected at that point. AddListener calls ReconnectLoop
which ensures the connection. But I might be missing something clearly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jjxtra/ExchangeSharp/issues/241#issuecomment-412911842,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI8Did1SLZwJzowifHd6keih9fMaUXxIks5uQuv1gaJpZM4V72FA
.

Checked in some code. To reproduce, run
websocket-orderbook exchangeName=Bittrex symbols=*
and press space bar after one second.
It takes a few seconds to send the several hundred market symbols to Bittrex to request order books for. During the loop, if the manager connection is closed / disposed, this exception will be thrown. The new code will restart the loop if it's simply a matter of a connection issue. If the connection or manager are disposed, the loop will end.


Missing comparison with nmon

http://nmon.sourceforge.net/pmwiki.php
nmon comes from AIX originally but is available for Linux.

We wouldn't mind taking a PR for more comparisons but we probably won't do the work ourselves.


Wiki for API

Trying to learn the new api, and migrate old admin code from 1.x branch to new api. A reference documentation as WIKI for api would be really helpful. It can be just one line description and sample usage, not a lot.
e.g.
I am unable to find what API i have to use to show navbar in layout
function LayoutView(vm, layout) {
	layout.nav = new Navbar({title: 'Admin'});

	return function() {
		return el("div", [
			layout.nav.view,            /// ??????
			el(".container-fluid")
		]);
	}
}

yeah, sorry. v2 docs are non-existent. it's basically "read the demos"
all vnodes now have factories:
var el = domvm.defineElement,
    tx = domvm.defineText,
    cm = domvm.defineComment,
    vw = domvm.defineView,
    iv = domvm.injectView,
    ie = domvm.injectElement;

you'd want to use vw or iv depending on if you're injecting an existing vm or simply declaring a view.
defineText is not strictly necessary, since the pre-processor will convert numbers/strings into text nodes for you [1].
[1] https://github.com/leeoniya/domvm/blob/2.x-dev/src/view/preProc.js#L48

How do the views update/redraw in https://github.com/leeoniya/domvm/blob/2.x-dev/demos/streams.html without calling vm.redraw ?

after any redraw, if preprocessor detects a stream-esque value returned in the new template, it automatically chains a single-fire redraw onto the stream. put another way, it creates a dependent stream that will only invoke once and automatically unsubscribe.
this means that the dependent streams are destroyed and re-created on every redraw. this wouldn't fare well for any synthetic benchmarks, but it works quite fast in real life.

That makes sense. Is there a way to disable it? How does diff work and when/why should it be used?

That makes sense. Is there a way to disable it?

You can use (or generate) a build [0] that doesn't include streamFlyd.
You can just use the stream's values rather than the actual streams into your templates.
For any builds that have the capability of stream handling (mini+) [4], you can call domvm.streamCfg to override the is to always return false so nothing is ever detected as a stream.

The last option seems moot, since including a raw stream object directly ina  template will stringify to [Object object] or whatever if toString is not explicitly implemented in your stream flavor.
diff is explained here [1], you can see it used in [2] (commented/optimized impls) and [3].
[0] https://github.com/leeoniya/domvm/blob/2.x-dev/dist/README.md
[1] https://github.com/leeoniya/domvm/issues/111#issuecomment-275710310
[2] https://github.com/leeoniya/domvm/blob/2.x-dev/demos/bench/dbmonster/app.js
[3] https://github.com/leeoniya/domvm/blob/2.x-dev/demos/immutable.html
[4] https://github.com/leeoniya/domvm/blob/2.x-dev/src/builds/mini.js#L8

@iamjohnlong out of curiosity, are you secretly working on some docs? ;)
if so, i'll hold off to iterate on your work, otherwise i'll slap some super dense api docs together for the time being. i don't wanna duplicate or throw away any valuable work/contributions.

I'm not, but I'd be happy to add to something you cook up, examples and such. I was looking at the immutable demo you have. That turned me on to freezerjs, which I didn't know of. Then I was looking at his React demo with freezer and your immutable demo made more sense of the reasoning behind it. I didn't know vm.update existed.
I haven't built anything with React/Redux, mainly because Redux looks like you have a lot of boilerplate code and could get messy with large apps. Looking at your immutable demo w/ freezer almost seems too easy. Same with React/Freezer [1]. I'm wonder how well it'd scale with large routed apps. I'm going to build the todomvc with domvm/freezer [2] this weekend if I have some free time away from the wife and kids :)
[1] https://medium.com/@arqex/react-the-simple-way-cabdf1f42f12
[2] https://github.com/arqex/freezer-todomvc

the caveat to vm.update is that obviously it can never update a model you're closured. so...
function MyView(vm, model) {    // closured model; not affected by vm.update(newModel)
  return function(vm, model) {    // passed model at each redraw; is affected
    return el("div", model.name);
  }
}

you can choose which to use on a case-by case basis. this is why render()'s sig is identical to the closure's sig. it allows you to make pure components, if needed.

@leeoniya https://rawgit.com/iamjohnlong/domvm-freezer-todo/master/index.html
a work in progress, I have a few questions though. I'd like to pass in the freezer State.trigger somewhere other then the model. Would the opts be a good place for it?

yeah that's the most obvious place to put it if you insist on dep-injecting.
IMO it's easier to designate a global const STORE = new Freezer({...}); and then use STORE.trigger() wherever you need.

Passing around all needed dependencies becomes a major pain

Yes indeed. My approach is to separate reusable modular stuff (at a logical level, not necessarily actually used in another app), from the code which is entirely relevant only to the app. App code utilizes shared state from the encapsulating app context (in my can usually the application closure), which the other modules create their own state and are inject with the things they need to interact with the application, including effort expended in defining a logically coherent and suitably abstract API.
That's been working for me pretty well.

I guess for application structure, I'm planning on implementing this with required modules/es6. More or less like https://github.com/arqex/freezer-todomvc
I was following the convention of having a "reactions" file [1] that handles the setting the state
[1] https://github.com/arqex/freezer-todomvc/blob/master/src/todoReactions.js

that implementation seems needlessly complex with a lot of indirection, probably due to React's semantics.
if you're forced to call State.trigger('todo:clearCompleted');, then you may as well use rootVm.redraw(). the whole point [besides optimized sub-view sCU] is to avoid triggering these things by virtue of a global update event emitted by a central store.
i would recommend working off of @yosbelms 1.x implementation [1] to integrate Freezer and remove the need for explicit redraw() calls. You don't really need a pub/sub event system for this at all.
[1] https://github.com/leeoniya/domvm/tree/1.x-dev/demos/TodoMVC

shrug, it doesn't seem very complex to me. it's just the approach I took after looking at the freezer todo example already mentioned and https://github.com/mciparelli/inferno-freezer-todomvc

shrug, it doesn't seem very complex to me.

that's fine, but an implementation with an immutable store that still requires you to explicitly use a pub/sub system is somewhat puzzling to me in terms of benefits. on top of that, any TodoMVC that doesnt contain 10k todo items will benefit little from any form of optimized redraw, given domvm's perf profile.
while creating a distributed, fault-tolerant, microservice AWS-hosted todolist app with SSR rendering might be interesting technically, it's a pretty shitty use case to necessitate 7 levels of indirection and left-pad micro-dependencies. enterprise-level complexity should require enterprise-level problems. / $0.02

I get your points, it's less about the todo app itself and more thinking about how I would structure a large application using freezer and domvm. If I were writing a todo app for what it's worth I would write it in vanillajs, or the most optimized solution would be a pencil and paper.

whether you agree with the implementation I may have found a bug when using keys in defineView with diff set [1]. The browser freezes when deleting todos and some other odd things.
[1] https://github.com/iamjohnlong/domvm-freezer-todo/blob/master/app.js#L115

can't seem to reproduce. can you provide specific steps and browser you're seeing this in?

I uncommented the line https://github.com/iamjohnlong/domvm-freezer-todo/blob/master/app.js#L116
I'm in chrome  56.0.2924.87 (64-bit) on osx
open https://rawgit.com/iamjohnlong/domvm-freezer-todo/master/index.html#/ and hard refresh => add some todos => refresh => delete the first todo => browser freezes

confirmed, thanks. the irony of a browser-freeze bug in a Freezer demo is not lost on me.
so far i've narrowed it down to the reconciliation algo never hitting an exit condition out of this while loop [1]. should have a fix hopefully soon.
[1] https://github.com/leeoniya/domvm/blob/2.x-dev/src/view/syncChildren.js#L160

@iamjohnlong would you have time to try to get it down to a minimal test case? there's a lot of app to strip down here :/

http://pastebin.com/gHMXxFCW that's your immutable.html tweaked

i'm unable to reproduce it in the tweaked immutable.html

I'm away from my computer ATM. Be back in a couple hours

i've isolated a test case [1]
it doesnt look like keys cause this, but diff() is definitely involved. the weirdest thing is that the bug only manifests with > 4 sub-views where it causes one error in the reconciler and causes an infinite loop in the reconciler for > 5 sub-views - very odd.
[1] https://gist.github.com/leeoniya/82558c8a3861d715bf451017f0ab746f
[2] https://github.com/leeoniya/domvm/issues/113

issue is fixed.

the discussion no longer corresponds to the title, so closing this.
gonna use some bits from here while working on the docs.


[Bugs] Some problems of kinematic models. (face direction and kinematic chain modeling)

Thanks for your great work! When studying your wonderful motion representation, I found two small problem which might influence your effects. And they are all in common/skeleton.py.
1. Face Direction
This problem is already mentioned here: #112 #107. The direction of the across vector might be wrong.
https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/common/skeleton.py#L58-L62
Which is different from your code in motion_representation.ipynb.
https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/motion_representation.ipynb#L85-L89
This may lead to some errors, but thanks to the redundant 263-dimensional information, this can be recovered to a certain extent
2. Kinematic Chain
This problem may not be as serious as the above.
https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/common/skeleton.py#L81-L99
Here, each joints' position(except roots') will be converted to the rotation about it's parent along kinematic chain. However, not every chain start from root, which means line 85 is not very suitable especially for the arms' chains.
I thought it will be more suitable if it's like:
R = quat_params[:, chain[0]]

Again, thanks for your great work!

Hi, thanks for pointing this out. Now it would be too late to change the code, since many people have done experiments on the obtained dataset. However, I will highlight your issue, so people will see your comments.

Hi, thanks for pointing this out. Now it would be too late to change the code, since many people have done experiments on the obtained dataset. However, I will highlight your issue, so people will see your comments.

Thanks!

Thanks for your great work! When studying your wonderful motion representation, I found two small problem which might influence your effects. And they are all in common/skeleton.py.
1. Face Direction
This problem is already mentioned here: #112 #107. The direction of the across vector might be wrong.
https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/common/skeleton.py#L58-L62
Which is different from your code in motion_representation.ipynb.
https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/motion_representation.ipynb#L85-L89
This may lead to some errors, but thanks to the redundant 263-dimensional information, this can be recovered to a certain extent
2. Kinematic Chain
This problem may not be as serious as the above.
https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/common/skeleton.py#L81-L99
Here, each joints' position(except roots') will be converted to the rotation about it's parent along kinematic chain. However, not every chain start from root, which means line 85 is not very suitable especially for the arms' chains.
I thought it will be more suitable if it's like:
R = quat_params[:, chain[0]]

Again, thanks for your great work!

Hi,
I noticed that some processed motions are really wired when I change R = root_quat to R = quat_params[:, chain[0]] in Line 85 in HumanML3D/common/skeleton.py. I present the motion animation with ID 000004.
https://github.com/user-attachments/assets/48cf1716-fa0f-44ab-8941-704d2ef52ec5
It seems that the left arm and right arm swap after the first frame. But I can not figure out the reason.

https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/common/skeleton.py#L126C5-L140C38

Thanks, I will try.

@Silverster98 Here also needs to be modified synchronously，line 140: https://github.com/EricGuo5513/HumanML3D/blob/6b72fcda6c4fb90be69efd784942bf08be2d51e1/common/skeleton.py#L126C5-L140C38
Before
R = quat_params[:, 0] # Using global root rotation
After
R = quat_params[:, chain[0]] # Using the rotation of the current chain's starting joint

After thinking for a while, I realized it needs more modification than that.
For left and right arms, the chain's starting joint is 9 : spine3, so the rotation there should be accumulated all the way from root joint: (0: hip -> 3: spine1 -> 6: spine2 -> 9: spine3).
A quick hack is as follows
def forward_kinematics(self, quat_params, root_pos, skel_joints=None, do_root_R=True):
        # quat_params (batch_size, joints_num, 4)
        # joints (batch_size, joints_num, 3)
        # root_pos (batch_size, 3)
        if skel_joints is not None:
            offsets = self.get_offsets_joints_batch(skel_joints)
        if len(self._offset.shape) == 2:
            offsets = self._offset.expand(quat_params.shape[0], -1, -1)
        joints = torch.zeros(quat_params.shape[:-1] + (3,)).to(self.device)
        joints[:, 0] = root_pos
        for chain in self._kinematic_tree:
            if do_root_R:
                if chain[0] == 9:
                    R = global_R
                else:
                    R = quat_params[:, 0]
            else:
                R = torch.tensor([[1.0, 0.0, 0.0, 0.0]]).expand(len(quat_params), -1).detach().to(self.device)
            for i in range(1, len(chain)):
                R = qmul(R, quat_params[:, chain[i]])
                if chain[i] == 9:
                    global_R = R.copy()
                offset_vec = offsets[:, chain[i]]
                joints[:, chain[i]] = qrot(R, offset_vec) + joints[:, chain[i-1]]
        return joints


📃: Add javascript easy,medium,difficult level challenegs.

🔴 Title :📃: Add javascript easy,medium,difficult level challenegs.
🔴 Tech stack :
🔴 Objective :📃: Add javascript easy,medium,difficult level challenegs.
🔴 Summary :
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name :
Bhuvaneswari Kapuluru Gssoc extd

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

same issue closing it


Create solution to debug entitlement / get decision

Users are having issues determining how an entity is entitled to data. We've been adding logs to the platform, but this is causing production issues and isn't very usable since the log is extensive.
Acceptance Criteria

develop a POV for debugging entitlement and get decision
remove the log in https://github.com/opentdf/platform/issues/1503

consider the casbin roles that are currently set up and permissions on this command


Harden order resolution logic and pseudocode

This just adds better language about order resolution logic, as well as a solve for very niche paradoxes for convoys.

Looks good. I will pull it, but then I want to open an issue and start a discussion on how we are going to start implementing your logic into the current framework.


modules.services.xscreensaver: Add user service command

Users may want to define their own command to start the service.
Personally, i cannot use the nix's xscreensaver program on my debian box (once
locked, no password prompt and plenty of pam errors). With the exact same home
on a nixos box, no issue whatsoever. I have searched and tried to fix it to no
avail so far. I think it's related to pam (/etc/pam.d/xscreensaver is clearly
different between debian and NixOS). That's another issue altogether (other lockers
face the same issue either way ;)
Anyway, right now, i'm working around it using an xscreensaver-wrapper script.
It fallbacks to use /usr/bin/xscreensaver (if found) over the one installed in
my nix profile (the debian one has no issue either).
But right now, to do so, i'm defining my own systemd service. As the need could
be shared, i thought of opening this command option here.
Note:
Even though i read the docs, looked through issues, and was hinted at home-manager -f command on irc (thanks again), i did not find the right command stanza to try and build locally. So i rely on the ci to tell me if i messed up or not.
Cheers,

Thanks for the contribution! I'm not certain how frequently it would be to set an explicit command. If it's not something people would do often I would suggest simply letting the user override the command explicitly in their own configuration:
systemd.user.services.xscreensaver.Service.ExecStart =
  lib.mkForce "${config.home.homeDirectory}/bin/wrapped-xscreensaver";

systemd.user.services.xscreensaver.Service.ExecStart =
lib.mkForce "${config.home.homeDirectory}/bin/wrapped-xscreensaver";

Thanks.
I did not realize i could do that ;)
I guess i can close this PR then?

Yeah, I think we can close it. If I get more requests for this option then I'd be happy to resurrect this PR 🙂

I would suggest simply letting the user override the command explicitly in their own configuration:

Success! Thanks.
For people interested, the end result looks like:
...
  config = mkIf cfg.enable {  # optional dependency on xsession
    # override the service's exec start to use a wrapper defined below
    systemd.user.services.xscreensaver.Service.ExecStart =
      lib.mkForce "${config.home.homeDirectory}/bin/xscreensaver-wrapper";
    services.xscreensaver.enable = true;

    home = {
      packages = [ pkgs.xscreensaver ];
      file = {
        "bin/xscreensaver-wrapper" = {
          executable = true;
          text = with pkgs; ''
#!${stdenv.shell} -xe

if [ -f /usr/bin/xscreensaver ]; then
  # on debian, the nix xscreensaver does not work it refuses to display the
  # password prompt my best guess is an issue with pam but i cannot entangle it
  # yet so right now using debian's packaged xscreensaver over the nix one
  locker=/usr/bin/xscreensaver
else
  locker=${xscreensaver}/bin/xscreensaver
fi

$locker -no-splash
'';
        };
...

If you are generating the script then you can shove it directly into the service, which is a bit simpler. Might not be so useful if you want to be able to run the wrapper manually, though.
config = mkIf cfg.enable {  # optional dependency on xsession
  services.xscreensaver.enable = true;
  # override the service's exec start to use a wrapper defined below
  systemd.user.services.xscreensaver.Service.ExecStart =
      lib.mkForce (toString (pkgs.writeShellScript "xscreensaver-wrapper" ''
        if [ -f /usr/bin/xscreensaver ]; then
          # on debian, the nix xscreensaver does not work it refuses to display the
          # password prompt my best guess is an issue with pam but i cannot entangle it
          # yet so right now using debian's packaged xscreensaver over the nix one
          locker=/usr/bin/xscreensaver
        else
          locker=${pkgs.xscreensaver}/bin/xscreensaver
        fi

        $locker -no-splash
      ''));
};

If you are generating the script then you can shove it directly into the service, which is a bit simpler

oh, right, neat, thanks!

If you are generating the script then you can shove it directly into the service, which is a bit simpler

oh, right, neat, thanks!

It works fine :D
https://gitlab.com/ardumont/home/-/blob/master/xsession/session-lock.nix#L8-20


51 24 0:32 / /sys/fs/cgroup/perf_event rw,relatime - cgroup cgroup rw,perf_event,release_agent=/run/cgmanager/agents/cgm-release-agent.perf_event
52 24 0:30 / /sys/fs/cgroup/hugetlb rw,relatime - cgroup cgroup rw,hugetlb,release_agent=/run/cgmanager/agents/cgm-release-agent.hugetlb
34 21 0:35 / /run/lxcfs/controllers rw,relatime - tmpfs tmpfs rw,size=100k,mode=700
35 34 0:33 / /run/lxcfs/controllers/name=systemd rw,relatime - cgroup name=systemd rw,release_agent=/run/cgmanager/agents/cgm-release-agent.systemd,name=systemd
36 34 0:32 / /run/lxcfs/controllers/perf_event rw,relatime - cgroup perf_event rw,perf_event,release_agent=/run/cgmanager/agents/cgm-release-agent.perf_event
37 34 0:31 / /run/lxcfs/controllers/memory rw,relatime - cgroup memory rw,memory,release_agent=/run/cgmanager/agents/cgm-release-agent.memory
38 34 0:30 / /run/lxcfs/controllers/hugetlb rw,relatime - cgroup hugetlb rw,hugetlb,release_agent=/run/cgmanager/agents/cgm-release-agent.hugetlb
39 34 0:29 / /run/lxcfs/controllers/freezer rw,relatime - cgroup freezer rw,freezer,release_agent=/run/cgmanager/agents/cgm-release-agent.freezer
40 34 0:28 / /run/lxcfs/controllers/devices rw,relatime - cgroup devices rw,devices,release_agent=/run/cgmanager/agents/cgm-release-agent.devices
41 34 0:27 / /run/lxcfs/controllers/cpuset rw,relatime - cgroup cpuset rw,cpuset,release_agent=/run/cgmanager/agents/cgm-release-agent.cpuset,clone_children
42 34 0:26 / /run/lxcfs/controllers/cpuacct rw,relatime - cgroup cpuacct rw,cpuacct,release_agent=/run/cgmanager/agents/cgm-release-agent.cpuacct
43 34 0:25 / /run/lxcfs/controllers/cpu rw,relatime - cgroup cpu rw,cpu,release_agent=/run/cgmanager/agents/cgm-release-agent.cpu
44 34 0:24 / /run/lxcfs/controllers/blkio rw,relatime - cgroup blkio rw,blkio,release_agent=/run/cgmanager/agents/cgm-release-agent.blkio
53 22 0:36 / /var/lib/lxcfs rw,nosuid,nodev,relatime - fuse.lxcfs lxcfs rw,user_id=0,group_id=0,allow_other
55 33 0:37 / /run/cgmanager/fs rw,relatime - tmpfs cgmfs rw,size=100k,mode=755
54 22 8:1 /var/lib/lxd/shmounts /var/lib/lxd/shmounts rw,relatime shared:1 - ext4 /dev/disk/by-uuid/12beabfe-aca6-4475-bcff-b3c4d0e8bb3f rw,errors=remount-ro,data=ordered

I do see '/run/cgmanager' here, but I am not running cgmanager (anymore, I think i was uninstalled with the upgrade to beta4). Maybe that is it?
I have not yet tried to reboot this server.

This output looks correct to me...
Would be interesting to see if a reboot helps.

A reboot fixed this.
2 things I noted:

kernel is now 3.13.0-79 while it was 3.13.0-77 before reboot
lxcfs is not started at boot up, the container booted with and without lxcfs though

Anyway, it is now working again. Thanks @stgraber !

my /proc/self/mountinfo now looks differently btw:
17 22 0:15 / /sys rw,nosuid,nodev,noexec,relatime - sysfs sysfs rw
18 22 0:3 / /proc rw,nosuid,nodev,noexec,relatime - proc proc rw
19 22 0:5 / /dev rw,relatime - devtmpfs udev rw,size=4053076k,nr_inodes=1013269,mode=755
20 19 0:12 / /dev/pts rw,nosuid,noexec,relatime - devpts devpts rw,gid=5,mode=620,ptmxmode=000
21 22 0:16 / /run rw,nosuid,noexec,relatime - tmpfs tmpfs rw,size=817740k,mode=755
22 1 8:1 / / rw,relatime - ext4 /dev/disk/by-uuid/12beabfe-aca6-4475-bcff-b3c4d0e8bb3f rw,errors=remount-ro,data=ordered
24 17 0:17 / /sys/fs/cgroup rw,relatime - tmpfs none rw,size=4k,mode=755
25 17 0:18 / /sys/fs/fuse/connections rw,relatime - fusectl none rw
26 17 0:6 / /sys/kernel/debug rw,relatime - debugfs none rw
27 17 0:10 / /sys/kernel/security rw,relatime - securityfs none rw
28 21 0:19 / /run/lock rw,nosuid,nodev,noexec,relatime - tmpfs none rw,size=5120k
29 21 0:20 / /run/shm rw,nosuid,nodev,relatime - tmpfs none rw
30 21 0:21 / /run/user rw,nosuid,nodev,noexec,relatime - tmpfs none rw,size=102400k,mode=755
31 17 0:22 / /sys/fs/pstore rw,relatime - pstore none rw
32 24 0:23 / /sys/fs/cgroup/cpuset rw,relatime - cgroup cgroup rw,cpuset,clone_children
33 24 0:24 / /sys/fs/cgroup/cpu rw,relatime - cgroup cgroup rw,cpu
34 24 0:25 / /sys/fs/cgroup/cpuacct rw,relatime - cgroup cgroup rw,cpuacct
35 24 0:26 / /sys/fs/cgroup/memory rw,relatime - cgroup cgroup rw,memory
36 24 0:27 / /sys/fs/cgroup/devices rw,relatime - cgroup cgroup rw,devices
37 24 0:28 / /sys/fs/cgroup/freezer rw,relatime - cgroup cgroup rw,freezer
38 24 0:29 / /sys/fs/cgroup/blkio rw,relatime - cgroup cgroup rw,blkio
39 24 0:30 / /sys/fs/cgroup/perf_event rw,relatime - cgroup cgroup rw,perf_event
40 24 0:31 / /sys/fs/cgroup/hugetlb rw,relatime - cgroup cgroup rw,hugetlb
41 22 8:1 /var/lib/lxd/shmounts /var/lib/lxd/shmounts rw,relatime shared:1 - ext4 /dev/disk/by-uuid/12beabfe-aca6-4475-bcff-b3c4d0e8bb3f rw,errors=remount-ro,data=ordered
23 21 0:32 / /run/lxcfs/controllers rw,relatime - tmpfs tmpfs rw,size=100k,mode=700
42 23 0:31 / /run/lxcfs/controllers/hugetlb rw,relatime - cgroup hugetlb rw,hugetlb
43 23 0:30 / /run/lxcfs/controllers/perf_event rw,relatime - cgroup perf_event rw,perf_event
44 23 0:29 / /run/lxcfs/controllers/blkio rw,relatime - cgroup blkio rw,blkio
45 23 0:28 / /run/lxcfs/controllers/freezer rw,relatime - cgroup freezer rw,freezer
46 23 0:27 / /run/lxcfs/controllers/devices rw,relatime - cgroup devices rw,devices
47 23 0:26 / /run/lxcfs/controllers/memory rw,relatime - cgroup memory rw,memory
48 23 0:25 / /run/lxcfs/controllers/cpuacct rw,relatime - cgroup cpuacct rw,cpuacct
49 23 0:24 / /run/lxcfs/controllers/cpu rw,relatime - cgroup cpu rw,cpu
50 23 0:23 / /run/lxcfs/controllers/cpuset rw,relatime - cgroup cpuset rw,cpuset,clone_children
51 22 0:33 / /var/lib/lxcfs rw,nosuid,nodev,relatime - fuse.lxcfs lxcfs rw,user_id=0,group_id=0,allow_other

No mention of cgmanager anymore.

Good to hear that things are working again. I've pushed a fix for a tiny mistake in the lxcfs upstart job but I'm not sure why it didn't start for you anyway.
I checked on my 14.04 boxes and rebooting them does start lxcfs.

I think that fix is exactly why. I no longer have cgmanager installed, so the target cgmanager ready is invalid.
I also have LXC_AUTO set to false in /etc/defaults/lxc. So the upstart job never got started.

Ah, that makes sense, great.
@hallyn any chance you can tag a new lxcfs or just cherry-pick the fix I pushed yesterday?

@stgraber Has cgmanager been removed from the list of dependencies for LXD ? After the upgrade to LXD Beta4 cgmanager is being listed as unneeded package on Xubuntu 15.10 and is also missing in the list of dependencies for LXD when I try to install it in a nested container.

yep, it has. LXD never directly depended on cgmanager but liblxc1 which LXD uses did.
Between new kernel improvements and the work done on lxcfs, there's no more reason for cgmanager to be used so we're phasing it out with Ubuntu 16.04.


[CH-27791] Add check for "No reads after MakeConsensus"

Tickets: https://app.clubhouse.io/idseq/story/27791/bug-on-computestats-step-of-consensus-genome-pipeline

Description

This should address the issues noted in the bug. We just need to check if the consensus.fa file has any actual reads.

Tests

Tested directly on an EC2-IA instance in production. I tested 3 of the samples noted in the ticket and successfully got the error message.

As a sanity check, I also ran a "normal sample" and it was successful.

I'm going to do a local run after merging/releasing, then staging

Worked as expected in local run


Package names are wrong...

The packages are org.bdgenomics.adam

OK, so, since we pointed at the 0.16.0 release, this repository is a bit scoobied. I'll take more of a deep dive into this tomorrow.

Hmmmm?  What should the package names be?  I thought I was working off of master in adam.

@tdanford they're the correct names for packages in ADAM ;). I'm suggesting s/org.bdgenomics.adam/org.bdgenomics.qc/g.

I'll have a PR tomorrow AM for this. I think the PR will clarify.


fix: unify css collecting order

Fixes #11662

Description

This PR unify the CSS collecting order whether cssCodeSplit is true or false, aligning the latter with the former.
Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

I'm experiencing the css issues as well. I noticed this PR while searching and I'm wondering when it will be merged and if there are existing workarounds


Update the README in Protoc plugin

In this PR we update an outdated README file in tools/protoc-plugin in order to reflect the current "under the hood" details of the plugin's workflow.
Since the last update, we've stopped generating launcher scripts, as the Protobuf Gradle plugin now supports launching executable JARs.

@armiol, PTAL.


Feat: Want to enhance the page not found page

Describe the feature
I want to enhance the page not found page like when i opening the any other page then it is showing only a container with white background so i want to enhance it
Add ScreenShots

like that

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!!


Add count method to complement the search method

closes #5

Thank you for your PR. Could you please add a test in the OdooTest class.

Things I notice:
Odoo Request classes like Search, SearchRead expect a Domain even though the wrapper methods for them.
Seems like there's more tests to add, outside of this one, even though Count can perfectly be done with the Search Request class on its own, as long as a default/empty Domain is provided instead of null.
By this logic, I also adjusted the ObjectEndpoint class, so the following methods also null coalesce a default empty Domain object, like the model method does, and their Request classes and the Odoo API respectively expect:

count
search
searchRead

I renamed 2 tests that are actually testing the Model way, to include "Model"

testCount -> testModelCount
testCountWhere -> testModelCountWhere
testSearchRead -> testModelSearchRead
testSearchReadFields -> testModelSearchReadFields

and added:

testDirectCount
testDirectCountWhere
testDirectSearchRead
testDirectSearchReadFields

These however may need to be split to separate files, as the layer of their use is different (Direct calls vs Eloquent-like).

Hi @xewl,
i'm sorry, i have been very busy and had no time to review.
Your PR looks very good!
Good point with splitting the tests, we should consider this in a new PR.
Thank you for your contribution!

Happy NY @obuchmann
No worries, glad you found the time eventually :D
Thanks for the merge & version bump!


System.ArgumentException: Value does not fall within the expected range

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
I had installed updates for powertoys from preview page.
After installing updates invoked powertoys run from shortcut Alt + space
2024-07-08.txt
2024-07-10.txt
✔ Expected Behavior
Powertoys run should be invoked without any errors.
❌ Actual Behavior

Other Software
No response

/dup #33504


CSS Foundations Exercise 05-descendant-combinator: Add full solution

Complete the following REQUIRED checkboxes:

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this issue follows the location for request: brief description of request format, e.g. Foundations exercises: Add exercise for XYZ

The following checkbox is OPTIONAL:

[x] I would like to be assigned this issue to work on it

1. Description of the Feature Request:

Add full descendant combinator solution for this exercise to let us see the possibilities which we may have never thought of.
File Path: css-exercises>foundations>05-descendant-combinator>solution>solutions.css
2. Acceptance Criteria:

[ ] Add to solutions.css file either by css comment or by code.

/* Other possible combinations of selectors

div p
div .text
.container p

*/

or
div p {
  background-color: yellow;
  color: red;
  font-size: 20px;
  text-align: center;
}

div .text {
  background-color: yellow;
  color: red;
  font-size: 20px;
  text-align: center;
}

.container p {
  background-color: yellow;
  color: red;
  font-size: 20px;
  text-align: center;
}

This I believe could be good to add in as a commented out part, though the first option would be much better than needlessly repeating all of the styles.
I've assigned you. Just an FYI, you shouldn't complete the checkboxes for the Acceptance Criteria ahead of time; those can be completed as each item is completed.

This I believe could be good to add in as a commented out part, though the first option would be much better than needlessly repeating all of the styles.
I've assigned you. Just an FYI, you shouldn't complete the checkboxes for the Acceptance Criteria ahead of time; those can be completed as each item is completed.

Gotcha


Demarcation between common indicators and unique indicators

Can we have some indication that the top ones are common indicators and the bottom ones are unique (scheme specific indicators)? Any suggestions on how to implement this? @samwhocodes @gggodhwani


Use native implementation of getUserMedia on iOS 11+

In iOS 11 Apple has included a native implementation of getUserMedia. We should detect that the OS already has an implementation and use it instead of our implementation.
We should be able to use something like:
cordova.require('cordova/modulemapper').getOriginalSymbol(window, 'navigator.mediaDevices');

to detect if navigator.mediaDevices existed before we clobber it with our implementation.
Once we have a working implementation on iOS 11 we should test ImageCapture and MediaRecorder.

navigator.getMediaDevices is not available for WKWebView and UIWebView in iOS 11. It is only available for Safari.

This radar entry may be relevant: http://www.openradar.me/33571214

I have tested on iOS 12, and it seems that WKWebView still does not ship navigator.mediaDevices.


feature/CLS2-182-include-is-global-hq-field

Description of change
Currently this field is calculated inside the node controller and used by nunjucks. This PR moves that logic from the front end to the api
Checklist

[ ] Has this branch been rebased on top of the current main branch?

Explanation
The branch should not be stale or have conflicts at the time reviews are requested.

[ ] Is the CircleCI build passing?

General points

Other things to check

Make sure fixtures/test_data.yaml is maintained when updating models
Consider the admin site when making changes to models
Use select-/prefetch-related field lists in views and search apps, and update them when fields are added
Make sure the README is updated e.g. when adding new environment variables

See docs/CONTRIBUTING.md for more guidelines.

Fix broken test


'date.h' file not found while using the library

My operating system is Mac OS X 10.11, and I uses CMake as my project's build system.
I've already compiled sqlpp11 with these cmake flags :
cmake -DHinnantDate_ROOT_DIR=/Users/balthild/src/hinnantdate/ -DHinnantDate_INCLUDE_DIR=/Users/balthild/src/hinnantdate/

and installed sqlpp11 to /usr/local/ .
But when I tried including sqlpp11's header file in my project, it showed the error that can't find the header file of HinnantDate :
/usr/local/include/sqlpp11/chrono.h:30:10: fatal error: 'date.h' file not found
#include <date.h>

Hinnant's library doesn't provide any building system config so it's impossible to install the header file it into /usr/local/include (Copy-Paste probably wont work, since there's a tz.cpp which need compiling).

Hi Balthild,
date.h is a stand-alone header file. You can copy it anywhere you want. You then need to tell CMake to add the respective directory to the include paths.
Hope that helps.
Best,
Roland

I see. Thanks.


WIP PR for SRVLOGIC-122: [DOC] Modularize the "Core" (top-level) section.

User story for "Core" (top-level) section: SRVLOGIC-122

Incorporating the following mod doc JIRAs in the PR:

SRVLOGIC-39
SRVLOGIC-40
SRVLOGIC-123
SRVLOGIC-124
SRVLOGIC-125
SRVLOGIC-126
SRVLOGIC-127

@radtriste, @MarianMacik, @ricardozanini, and @krisv Can you please take a look and verify the content?

@radtriste, @MarianMacik, @ricardozanini, and @krisv Can you please take a look and verify the content?

Closing as stale, please reopen if still relevant.


place.sock file missing on MAC

I just hit an Issue with the socket creation file...
I set the CL_HOME to a different place and now the server panics with:
clace --config-file /Users/apple/workspace/clace/clace.toml server start
Error starting server: error removing socket file /Users/apple/workspace/clace/clace/.data/run/clace.sock : remove /Users/apple/workspace/go/clace/clace/.data/run/clace.sock: no such file or directory

It's a race condition perhaps, because on the previous "server start" I was not using CL_HOME, and so the socket file was in a different folder - the one I was kicking off Clase from.

What do you consider Best Practice ?
Have the Config file in HOME/clase/clase.toml
Have the CL_HOME point to  a place that would be a Volume that can be reused between server restarts  ?

I am now stuck and cant run clace :)
It really wants that old sockets file...

 CL_HOME should point to the home folder, not to the config file

ok, git pull, go rebuild and here we go:

mkdir -p /Users/apple/workspace/clace/clace/.data
cp clace.toml /Users/apple/workspace/clace/clace/.data/clace.toml
clace server start -c -l trace
4:22PM INF clace/internal/types/logger.go:52 > Logger initialized  backups=10 loglevel=trace maxSizeMB=50
4:22PM INF clace/internal/metadata/metadata.go:47 > Connecting to DB dbPath=/Users/apple/workspace/clace/clace/.data/clace.db
4:22PM INF clace/internal/metadata/metadata.go:91 > DB version is current
4:22PM TRC clace/internal/server/server.go:170 > Container management command cmd=docker
4:22PM WRN clace/internal/server/router.go:126 > Admin API access over TCP is enabled, enable 2FA for admin user account
4:22PM DBG clace/internal/server/server.go:238 > Error creating socket file, trying to dial socket file /Users/apple/workspace/clace/clace/.data/run/clace.sock error="listen unix /Users/apple/workspace/clace/clace/.data/run/clace.sock: bind: invalid argument"
4:22PM DBG clace/internal/server/server.go:241 > Error dialling UDS, trying to remove socket file error="dial unix /Users/apple/workspace/clace/clace/.data/run/clace.sock: connect: invalid argument"
Error starting server: error removing socket file /Users/apple/workspace/clace/clace/.data/run/clace.sock : remove /Users/apple/workspace/clace/clace/.data/run/clace.sock: no such file or directory. Original error listen unix /Users/apple/workspace/clace/clace/.data/run/clace.sock: bind: invalid argument

wil do..
here is there config btw

# The default configuration used by the Clace server and client

server_uri = "$CL_HOME/run/clace.sock" # the unix domain socket (UDS) path or URL to the Clace server
admin_user = "admin"

# Properties use by the Clace client to connect to the server
[client]
admin_password = "password"     # the password for the admin user. Required only if admin over TCP is enabled
skip_cert_check = false

# HTTP port binding related Config
[http]
host = "<IP_ADDRESS>" # bind to localhost by default for HTTP
port = 25222       # default port for HTTP

# HTTPS port binding related Config
[https]
host = "<IP_ADDRESS>"                               # bind to all interfaces (if port is >= 0)
port = 25223                                   # port for HTTPS
enable_cert_lookup = true                      # enable looking for certificate files on disk before using Let's Encrypt
service_email = ""                             # email address for registering with Let's Encrypt. Set a value to enable automatic certs
use_staging = true                             # use Let's Encrypt staging server
cert_location = "$CL_HOME/config/certificates" # where to look for existing certificate files
storage_location = "$CL_HOME/run/certmagic"    # where to cache dynamically created certificates

[security]
admin_over_tcp = true            # enable admin API's over TCP (HTTP/HTTPS). Admin is over UDS only by default
admin_password_bcrypt = ""       # the password bcrypt value
app_default_auth_type = "system"   # default auth type for new apps, "system" or "none" or custom auth
session_secret = ""              # the secret key for session cookie. Auto generated on server startup if not set
session_max_age = 86400          # session max age in seconds
session_https_only = true        # session cookie is HTTPS only

# Logging related Config
[logging]
level = "INFO"
max_backups = 10
max_size_mb = 50
console = false
file = true
access_logging = true

# Metadata Storage Config
[metadata]
db_connection = "sqlite:$CL_HOME/clace.db"
auto_upgrade = true

[system]
tailwindcss_command = "tailwindcss"
file_watcher_debounce_millis = 300
node_path = ""                      # node module lookup paths https://esbuild.github.io/api/#node-paths
container_command = "auto"          # "auto" or "docker" or "podman"

[plugin."store.in"]
db_connection = "sqlite:$CL_HOME/clace_app.db"

nc -U -l /Users/apple/workspace/clace/clace/.data/run/clace.sock

nc: File name too long

This is the same error as https://github.com/moby/moby/pull/13408 and https://github.com/golang/go/issues/6895, not sure why it is happening at length around 55 instead of 100 chars
Changing your CL_HOME to be at a shorter path would be a workaround.
I will look at changing the UDS to be in /tmp, but that would need some changes for Windows support. Also, currently running multiple Clace servers on one machine is possible, using a /tmp path will make that difficult

thanks @akclace
I moved the code to a shorter path and yes that fixed it for now. thanks.

I also pushed a fix, that changed the UDS path to be relative to CL_HOME. That way it works without special handling required for Windows.

I also pulled the latest fix and it now works on longer file paths ..
Fix https://github.com/claceio/clace/issues/24: Changed UDS path reference to be relative to avoid length issue
https://github.com/claceio/clace/commit/561f31fae0c0509c18ed50fc4c0cc4be4a567aba
So all good :)

In terms of best practices, the way the install.sh script sets up the env is the clace.toml should be minimal. If you run clace password > $CL_HOME/clace.toml, that add a bcrypt password hash for admin user to clace.toml. Nothing else is required in the default config. If you want a simpler admin password, run clace password --prompt > $CL_HOME/clace.toml. That way you can type in the password.
With this minimal config, admin APIs are not enabled over HTTP(S), they are enabled over UDS only. To access apps, the admin password can be used. To create apps, you can use the client running locally which connects over UDS. That is a secure config.
If you need the client and server to be on different machines, that is when you need to enable admin_over_tcp and set the admin_password. See https://clace.io/docs/configuration/security/#admin-api-access for details

thanks for the tips !


Bot Conflict Error

Please tick all applicable boxes.

[x] I am using Python 3.5.3 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[ ] The latest master version (release-260419)
[ ] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
[INFO] launcher: Starting sanity checks
[INFO] launcher: Checking for Python 3.5+
[INFO] launcher: Checking console encoding
[INFO] launcher: Setting console encoding to UTF-8
[INFO] launcher: Ensuring we're in the right environment
[INFO] launcher: Adding local bins/ folder to path
[INFO] launcher: Required checks passed.
[INFO] launcher: Optional checks passed.
[INFO] launcher: Moving old musicbot log
[ERROR] launcher: Error starting bot
Traceback (most recent call last):
File "run.py", line 367, in main
from musicbot import MusicBot
File "C:\Users\Johann\Downloads\Music Bot\MusicBot\musicbot_init_.py", line
7, in
from .utils import _get_variable
File "C:\Users\Johann\Downloads\Music Bot\MusicBot\musicbot\utils.py", line 3,
in
import aiohttp
ModuleNotFoundError: No module named 'aiohttp'
[INFO] launcher: Attempting to install dependencies...
Collecting pynacl==1.2.1
Using cached PyNaCl-1.2.1-cp37-cp37m-win32.whl (158 kB)
Collecting discord.py[voice]==1.2.5
Using cached discord.py-1.2.5-py3-none-any.whl (655 kB)
Requirement already satisfied: pip in c:\users\johann\appdata\local\programs\pyt
hon\python37-32\lib\site-packages (from -r requirements.txt (line 3)) (21.0.1)
Collecting youtube_dl
Using cached youtube_dl-2021.3.3-py2.py3-none-any.whl (1.9 MB)
Collecting colorlog
Using cached colorlog-4.7.2-py2.py3-none-any.whl (10 kB)
Collecting cffi
Using cached cffi-1.14.5-cp37-cp37m-win32.whl (166 kB)
Collecting websockets<7.0,>=6.0
Using cached websockets-6.0-cp37-cp37m-win32.whl (83 kB)
Collecting aiohttp<3.6.0,>=3.3.0
Using cached aiohttp-3.5.4-cp37-cp37m-win32.whl (583 kB)
ERROR: Cannot install discord-py[voice]==1.2.5 and pynacl==1.2.1 because these p
ackage versions have conflicting dependencies.
The conflict is caused by:
The user requested pynacl==1.2.1
discord-py[voice] 1.2.5 depends on PyNaCl==1.3.0; extra == "voice"
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/user_g
uide/#fixing-conflicting-dependencies
[CRITICAL] launcher: You may need to run as admin to install dependencies.
[INFO] launcher: All done.
C:\Users\Johann\Downloads\Music Bot\MusicBot>
Steps to reproduce

run.bat

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.

Change the requirements.txt file so that it looks like this:
pynacl
discord.py[voice]
pip
youtube_dl
colorlog

Then run update.bat(if your on windows, if not chose the appropriate update file for your system OS) once it's all up to date, then do run.bat(same note as before). Make to to press n and then y that way it doesn't reset the requirements.txt file back to the original.
Why do I need to change the requirements.txt file?
Discord pushed an update that broke discord.py, it was fixed in a new version of discord.py however we haven't updated the requirements.txt file to target the latest version of discord.py with the fix.

You can also use review, but you'll need to turn on intents


empty rename not working

Please format your bug report according to the template. If you do not provide me with the needed information I cannot help you.

@ChristianChiarulli I cannot reproduce, please make sure you are on v2.0.0, you might not have updated renamer


Pull request for python-openssl

For travis-ci/travis-ci#4283.
Ran tests and found no setuid bits.
See https://travis-ci.org/travis-ci/apt-whitelist-checker/builds/72050199

Sorry, but this test will have to be re-run. Will open a new one later.


Code style formatting using black + isort

Added code style formatting using black and isort.

https://github.com/psf/black (The Uncompromising Code Formatter)
https://pycqa.github.io/isort (sorts imports alphabetically, and automatically separated into sections and by type)

Added flake8 to dev-dependencies (incl. flake8-black + flake8-isort) for future linting (and potential editor niceness / developer delightness).

Note: Since flake8 doesn't support PEP 518 (pyproject.toml) configuration yet, we'll have to deal with pesky tox.ini (or .flake8 / setup.cfg) files for the time being.

Rationale
My take is that it may be good to apply these kinds of tooling on projects from the get-go as soon as possible to build some kind of understanding of how to align devs work, even if they're coming from different schools. I think black has done a great job at creating as few options as possible so that devs won't have to end up in endless discussions regarding "single quotes vs double quotes", "2 spaces vs 4 spaces", "trailing comma at end of arrays or not", "how many newlines should there be before a new function outside of a class", etc.
Using tools such as black and isort in new projects (I'm not full-on advocating to always reformat existing larger codebases though) should make us avoid potential discussions regarding what code styles people prefer. Also makes PR:s / code diffs cleaner.
Getting things like that in early is usually easier than later having to deal with politics around different code styles to choose later on, and also making a larger dent if having to deal with merge conflicts to a larger extent if code formatting ruling isn't applied very early on.

“Any color you like. Black.”

Is there anything worth adding to the README.md? I'm thinking, as a pycharm or vscode user, what should I do to avoid discrepancies in formatting on future PRs.
Or maybe this can be part of a more structured # Contributing section in a later PR

I think it makes sense to add that kinds of contributor info to a CONTRIBUTING.md.


Create initial list of Farcaster category submissions

What is it?
Make a JSON dump of the initial Farcaster frame and channel applicants to share with Farcaster team and see what kind of metrics we can get back

https://docs.google.com/spreadsheets/d/1TFGlY7l-rK8lN2xA6tV3-H3Fh_GsW5hjvdcCjmBsLqc/edit?usp=sharing


Parallex Web Design

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
A Parallex Web Design Efffect Made with help of HTML, CSS, JS.
Scrolling through site gives amazing effect.
Tech Stack
Web Development

closed with #282


Update NAN to 2.x for node 4.x and iojs 3.x support

Patch is almost done.

landed in https://github.com/mathiask88/node-snap7/commit/49105da9520b9b7a509fdb1767b387c6b15d2db4


[Help] When using Modal and Form components, the Form validation fails, but Modal is closed

基本信息
软件包名称和版本：<EMAIL_ADDRESS>框架版本： Vue 3.4
浏览器： Chrome 127.0.6533.101
Basic Info
Package Name And Version<EMAIL_ADDRESS>Framework version: Vue 3.4
Browser:  Chrome 127.0.6533.101
#最小复现代码 Mini Code

</emplate>
  <a-modal v-model:visible="customPropertiesModalVisible" auto-label-width @before-ok="handleCustomPropertiesFormSubmit"
           @cancel="$refs.customPropertiesFormRef.resetFields()"
           width="600" title-align="start" title="modal">
    <a-form :model="customPropertiesForm" :layout="'horizontal'" ref="customPropertiesFormRef"
            :rules="customPropertiesFormRules" @submit="handleCustomPropertiesFormSubmit" scroll-to-first-error>
      <a-form-item label="name" field="FN">
        <a-input v-model:model-value="customPropertiesForm.FN" />
      </a-form-item>
    </a-form>
  </a-modal>
</template>

<script setup lang="js">

const customPropertiesModalVisible = ref(false);
const customPropertiesFormRef = ref(null);
const customPropertiesForm = reactive({
....more
})

const handleCustomPropertiesFormSubmit =async ()=>{
  const valid = await customPropertiesFormRef.value.validate();
  if (valid) {
    for (const key in valid) {
      if (!valid.hasOwnProperty(key)) continue;
      const el = valid[key];
      const { field, isRequiredError, label, message, type, value } = el;
      if (!isRequiredError) continue;
      customPropertiesFormRef.value.setFields({
        [field]: {
          value: value,
          errors: [{ message: `${label} ${message}`, type: type }]
        }
      });
      customPropertiesFormRef.value.scrollToField(field);
      return false;
    }
  }

 other more...

  customPropertiesModalVisible.value = false;
  return true;
}

</script>

出现问题
当表单校验时, 出现了红色的"错误状态"提示,但是Modal依然被关闭.
切入断点发现 程序 进入了 if 分支,没有向下运行,但是 Modal依然被关闭了,我认为这是一个错误,我不知道怎么解决它.
[根据之前的Issue提示,我尝试了Modal的ok事件和 和 beforeOk事件,问题并没有被解决]
Something went wrong
When the form is validated, a red "Error Status" message appears, but Modal is still closed.
Cutting into the breakpoint found that the program went into the if branch and didn't run down, but Modal was still closed, I think it's a bug and I don't know how to fix it.
[I tried Modal's ok event and and beforeOk event according to the previous issue prompt, and the problem was not solved]
如果可以的话,各位能帮帮我嘛? 我不知道我错在哪里了
can you help me? I don't know what I did wrong

我看见了arco-design-react,中有一段描述
为什么无法调用 Modal 里嵌套的 Form 的方法？
例如以下使用方式：
const [form] = Form.useForm();
useEffect(() => {
  form.setFieldsValue({})
}, [])
return <div>
  <Modal visible={visible}>
    <Form form={form}>
      {/** ... */}
    </Form>
  </Modal>
</div>

this is because when the form method is called, the modal is not yet mounted, and the form has not been created. You can set mountOnEnter=false for Modal.
[Google Translate] 这是因为在调用 form 的方法时，Modal 还未挂载，Form 还未创建。 可以设置 Modal 的 mountOnEnter=false
[I don't understand why it wasn't stated in the Vue version, even if it was just copied or directly reproduced and referenced to the React version, it has tormented me for a while]
[我不明白为什么不在Vue版本中说明,哪怕直接复制一遍,或者直接说明参考React版本都可以,这折磨我有一段时间了]


cleanup: making the root as empty as possible

Moving files around to make the root as empty as possible. This is opinionated and not necessary but gets me into understanding some things :)

I like this! More of this please! But I am unsure @gicmo appreciates them ;)


Bug: Emery dep causes<EMAIL_ADDRESS>to fail w/ Astro 4

Error: Directory import '/Users/calebjasik-personal/Git/jasik-2.0/node_modules/.pnpm/@keystatic+core@0.3.19_react-dom<EMAIL_ADDRESS>is not supported resolving ES modules imported from /Users/calebjasik-personal/Git/jasik-2.0/node_modules/.pnpm/@keystatic+core@0.3.19_react-dom@18.2.0_react<EMAIL_ADDRESS>Did you mean to import<EMAIL_ADDRESS>    at new NodeError (node:internal/errors:406:5)
    at finalizeResolution (node:internal/modules/esm/resolve:227:11)
    at moduleResolve (node:internal/modules/esm/resolve:845:10)
    at defaultResolve (node:internal/modules/esm/resolve:1043:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:383:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:352:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:228:38)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:85:39)
    at link (node:internal/modules/esm/module_job:84:36)

I'm running into this issue when trying to upgrade to<EMAIL_ADDRESS>or later. Sorry for a lack of source, it's a private repo.

Could you try deleting your node_modules and lockfile and re-installing? This was fixed in a newer version of emery

@emmatown thx for the suggestion! Fixed things for me 👍


./test/integration/run.sh does not show the tests results.

When I following the steps in CONTRIBUTING.md to run the integration tests with command ./test/integration/run.sh after changing some source code, the results show as following:
# ./test/integration/run.sh
Using default tag: latest
latest: Pulling from dockerswarm/swarm-test-env
Digest: sha256:2ad61a065f185853d6c3e5ec98a8fc42f7cd4fab8f1ef575753fc874c184d2f0
Status: Image is up to date for dockerswarm/swarm-test-env:latest
++ time curl -L --silent https://github.com/docker/compose/releases/download/1.6.2/docker-compose-Linux-x86_64

Actually some tests execution failed，but it does not show the results and the detailed stacks. I will to enhance this doc to make it clearly. If some guys have some good suggestions, you can append them here.

To show results, print statements can be added to the failing tests.
For example
echo "PRINTING $output"


Allow referencing a subset of template files and or preprocessor

See #191
Basically JStachio will provide an SPI for the jstachio-apt module to allow different loading of templates at compile time which in theory would allow for preprocessing such as taking a subset of the template.
The impetus is the feature of wanting subset of template to be used as the template also known as "fragments".

See https://github.com/jstachio/jstachio/discussions/191


Fix broken annotation links

Several places link to /docs/syntax#annotations, but that anchor doesn't exist. Also the syntax page doesn't define or introduce annotations. After giving a rationale and example, it does hint that they are called annotations when it says "Annotation syntax also works with tags", but they should have been introduced before then.
This commit moves the annotations description out of Attributes by adding a new Annotations header (on the Syntax page), and tweaks the opening sentence to give a name to what it's describing. So now the links work and readers know what annotations are.

 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.

Thanks @pjungwir!


Meeting 7 Notes

Added the the page for notes.
If you have any issue number you want to add for your tasks, feel free to add them under the meeting notes.

It seems I forgot to close the issue.
Meeting notes can be accessed here: Meeting #7


Re-introducing parallel bgzip (pbgzip) during split align prep

So I found the next bottleneck in our bcbio setup (there is an infinite sequence of these, so I'm not worrying too much about it). During sample/align preparation, bcbio converts gziped and BAM inputs into bgziped and tabix'd files for split alignment. However, the gzip to bgzip conversion is rather slow since it is running on only one core. Based on the sources, it seems as if bcbio once used a parallel pbgzip implementation. In our local installation, I reinstalled pbgzip from a samtools fork and it performs just fine, scaling up to 8 cores until the gzip input stream becomes the limiting factor. I was wondering is the segfault issues reported in the sources are still valid, and if it is possible or desirable for the official bcbio installation to move back to pbgzip at some point in time.

Sven-Eric;
It would be great if you'd like to investigate this. I'm agreed that bgzip is a big bottleneck and re-introducing parallel bgzip would be a nice improvement.
In my past experience, parallel bgzip would segfault on some samples without apparent cause. Or at least, I couldn't identify a single factor that caused it to fail in some cases and not in others, which didn't make it easy to produce reliable work/not work cases. I did do one pull request that seemed to improve things some:
https://github.com/nh13/samtools/pull/2
but it still failed after that and I kind of gave up as I don't really understand all of the magic in that branch. I was hoping someone would take this on and we'd have parallel bgzip but that didn't happen so we can re-examine.
I guess the next best step would be having a standard build of this and make it an option so we can start to find and categorize failure cases. Are you using the raw build from @nh13 implementation, or have adjusted at all? Thanks much.

I would be happy to help here too.
Thumb typed for added typos

On Aug 14, 2015, at 7:14 AM, Brad Chapman<EMAIL_ADDRESS>wrote:
Sven-Eric;
It would be great if you'd like to investigate this. I'm agreed that bgzip is a big bottleneck and re-introducing parallel bgzip would be a nice improvement.
In my past experience, parallel bgzip would segfault on some samples without apparent cause. Or at least, I couldn't identify a single factor that caused it to fail in some cases and not in others, which didn't make it easy to produce reliable work/not work cases. I did do one pull request that seemed to improve things some:
nh13/samtools#2
but it still failed after that and I kind of gave up as I don't really understand all of the magic in that branch. I was hoping someone would take this on and we'd have parallel bgzip but that didn't happen so we can re-examine.
I guess the next best step would be having a standard build of this and make it an option so we can start to find and categorize failure cases. Are you using the raw build from @nh13 implementation, or have adjusted at all? Thanks much.
—
Reply to this email directly or view it on GitHub.

Thanks for chiming in, @nh13. Any help would be greatly appreciated, especially since I am not fluent in C. Perhaps it would be a good start if you could split up your pbgzip implementation into a separate repository as @chapmanb suggested? Perhaps Brad then could include that into the bcbio installation routine and/or biolinux. We could then allow activating the parallel pbgzip on-demand (labeled as experimental) in the bcbio_system.yaml, so that other bcbio users are protected from the random segfaults that Brad used to observed. More performance-conscious users like myself could activate pbgzip and we wait for the bug reports to come in to narrow down on a minimal failed input that generates a segfault. Does that make sense?

Hi everyone,
Sorry for jumping into the discussion, but I think the idea of an standalone implementation of pbgzip is a good one.
@nh13 what do you think about that?
@schelhorn the last commit to pbgzip is from April 14th 2013, am I right? The discussion that Brad pointed out was in September that year, so I am not sure that the problem is solved. Unless Brad wasn't using the latest version at that time?
The problem is that, as Brad pointed out, if the segfault happens randomly, it may be quite hard to point where the potential bug is. @chapmanb , any clue of how could we test it? Do you know about any particular sample where it always failed, or was it always random?
If we can't determine where the bug is exactly, then as Sven-Eric suggests, to label the integration as experimental may be an option.
Thanks!

Yes, the problem Brad noticed likely isn't fixed, so I'd suggest that bcbio supports both binaries for now and allows the user to opt in for the parallel version. Then only people are affected who know what they're doing and are willing to help with bug reports.

Sounds good,
@schelhorn did you compile this version of pbgzip? I am having troubles compiling it both in Mac (expected) and Linix. This is the last error in CentOS 6.6.  Do you recall having this issue?
...
gcc -g -Wall -O2 -pthread -o pbgzip ../bgzf.o ../knetfile.o util.o block.o consumer.o queue.o reader.o writer.o pbgzf.o pbgzip.o   -lm -lz -lbz2
consumer.o: In function `consumer_run':
consumer.c:(.text+0x3f3): undefined reference to `consumer_init_header'
consumer.c:(.text+0x47c): undefined reference to `consumer_update_header'
collect2: error: ld returned 1 exit status
...

I'll keep trying.

I was able to find an old commit with some compiling code and pushed it up here: https://github.com/nh13/pbgzip.  It is untested and I already found a bug (see https://github.com/nh13/pbgzip/issues/1).  Nonetheless, it should be a basis for seeing if we can get it working.

Nice, thanks! Let's then move the discussions in there.

Excellent, thanks for all the contributions. If no one objects I will close this issue in two days. We can reopen it once we have bug reports for samples where pbgzip failed.

Thanks @schelhorn , I've been compressing and uncompressing some samples without seeing any errors, I guess that some special race condition needs to happen.


QuicksortAlgo.cpp

Related Issue
I have made PR on Quick sort Algorithm
Closes: #[issue number that will be closed through this PR]
Give a clear description what modifications you have made
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly whereever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.

Screenshots(optional)

Original
Updated

original screenshot
updated screenshot

@Dinesh0602 Hey can you please first create an issue and then create pr
You need to add file in such way that

sorting folder

Quicksort folder(create it)

I'm just closing this Pr now you can create new pr with required changes

Sir please remove  those labels. Due to those labels   I am not able to get a Hacktoberfest T-shirt


Consul Chart Improvements

Since we are running a Petset should we include notes about deleting storage when we delete the Petset?

I have confirmed that the default reclaim policy is Delete and that volumes are being cleaned up on AWS. This should also work for GCE and Cinder. http://kubernetes.io/docs/user-guide/persistent-volumes/#reclaiming.
I would be more than happy to update the README to call out that volume delete may not work if you're using something other than the above. Thoughts?

The PetSet docs have a create example with a timeout. Please update the readme in the chart

Could you please point me at the example you are referring to as I would really like to make sure I address this issue correctly. I'd also be happy to review a PR with the stated changes if you're happy to work on it.

http://kubernetes.io/docs/user-guide/petset/#deleting-a-petset
Look for the section that has a grace ENV variable. Those commands allow for the full delete of a PetSet and storage. Current alpha limitation for PetSet.

Btw thanks for the help!!!!!

Documented PV cleanup steps assuming that you've already removed the Helm release.


Calendar and Contacts

Westwork needs a backend to store calendar and contacts. Let's use this issue to search for and choose one that works.

For this, I personally use radicacle. It's basic but will likely provide enough functionality for our purposes.


[AdaptiveCards] Date input missing date picker

I'm using an Adaptive Card to allow users to edit some fields during a conversation. Some fields are date type and it is working fine in emulator (I can select a date from the date picker).
However, when trying the same exact code published in my webchat (through directline with secret) the date fields appear simply as text fields without any picker.
It should work according this page, no? http://adaptivecards.io/visualizer/index.html?hostApp=WebChat (Bot Framework)

Hi @AndreMantas, which browser are you using? Implementation of HTML5 input controls varies between browsers. The Emulator is based on Chrome.

Oh, my bad. I was testing randomly with firefox (and it was outdated). Works as expected now, thanks.

No, there is no validation provided by Web Chat. You should probably do this in your bot code.


Typo "Azure Blob storage"→"Azure Blob Storage"

https://docs.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-boot-starter-java-app-with-azure-storage
#PingMSFTdocs

@hyoshioka0128 - Thank you for contributing! But please do not open any more PRs to change capitalization of terms. Our teams will decide whether such changes are needed and make them.


Documentation & default branch change

I am trying to add Github pages to this project for Api documentation. It expects the HTML/CSS files to be in master\docs folder.
So we would need to use master as default branch instead of develop.

Done


Collection filtering

Hey @darrenwiens, what do you think about having a text box in the Common tab that filters collections by ID, to help navigate big STACs? Whenever the filter text is changed, the collection list would update to only include matching collections.
I threw together a quick working demo below, mostly to make sure it was possible with ipywidgets before I suggested it.
If you like the idea I'm happy to make a PR. If not, no worries!

I like the idea for this added functionality.
The other inputs (Collection(s), Limit, and ID(s)) all act on the query, but the new filter acts on Collection(s). Is there a way to group Collection(s) together with Filter to make it more clear from a UX perspective?

Good point, I played around with some changes to try and make it clearer that the filter doesn't affect the query.

Changed Filter to 🔍 so it feels less like a query input.
Added # selected so you can tell the filter input doesn't select anything.
Divided into Collections and Items sections.

# 3 might be overkill, but I figured I'd throw some options your way.

That looks awesome! Happy to review a PR.


Jstree v4

Hello,
Any news about jstree 4 ?
When do you plan to release it  ?
Kind regards.
Fred

I just published some dev code to:
https://github.com/vakata/jstree/tree/v.4.0
Some basic functionality is still missing, and the code works only in browsers that support generators, but that will change over the next few weeks.


Model compiler doesn't work properly with provided model

I was playing around with compiling new models, and I downloaded the models given in the vision kit tutorial at this link. Compiling the mobilenet_v1 works properly, however the mobilenet_ssd model uses operations during preprocessing that are unsupported in toco/tflite. This means that you can't feed the input placeholder. Rather you need to use the output of the preprocessing step. I.e.:
Working:
./bonnet_model_compiler.par \
--frozen_graph_path="./mobilenet_ssd_256res_0.125_person_cat_dog.pb" \
--input_tensor_name="Preprocessor/sub" \
--output_tensor_names="concat,concat_1" \
--input_tensor_size=256 \
--output_graph_path="./converted_graph.binaryproto"

Output:
VisionBonnet model binary ./converted_graph.binaryproto generated.

 This is largely a problem with tflite not supporting operations, but it would be nice to provide a model that doesn't use unsupported operations.

Closing in reference to #308. It still would be nice to have a model that does not require this change.


Calculation of memory component address space wrong?

I believe the address calculation for memories is done wrongly, ie. it is done as for registers which doesn't make sense physically. The standard is very vague here - maybe there are different interpretations?
The RDL example below is a trivial example of a 32-bit physical bus system, where there is a memory attached which is only 13 bits wide and 4 entries deep. This means that the physical 4deep x 13wide memory is connected to the bus system taking up 4 32-bit words address space (0x10 total address space), and within each of those 4x 32-bit words, the memory is connected to the 13 lowest bits in each word.
Even so, the tool calculates (wrongly in my opinion) that the memory only takes up 2 words (0x8 address space), which is physically impossible. A 4-entry memory (with any width from 1-32 bits in this case) will always take up 4 words (0x10 address space in this case).
If you believe that the address/size calculation done by the tool today is correct, can you point to the location in the standard where this is described?
default regwidth    = 32;
default accesswidth = 32;

addrmap memexample {
  external mem {
    memwidth   = 13;
    mementries = 4;
  } mymem;

  reg { field {} r8[8] = 0; } reg8;
};

The default assignments of regwidth and accesswidth properties only apply for reg components. These properties are not valid for mem components so they do not have any effect (see <IP_ADDRESS>).
The SystemRDL spec does not provide any explicit guidance on how a memory is mapped to an address space, so the only remaining interpretation is to allocate bytes by rounding up the memwidth to the next power-of-two bytes as is implied by 11.2-e.
The behavior of inferred address allocation is also not defined very clearly in the spec. For this reason the compiler has an optional warning you can enable that encourages users to explicitly assign addresses.

Thanks for the detailed answer.


Setting CSP headers while deploying

Hi I have been trying to set custom content-security-policy headers while deploying the cluster

LogScale have support for the environment variable STATIC_IMAGE_CONTENT_URL if that is what you're asking for. If that does not fit your needs, you can likely leverage features of whatever ingress controller you have in front of LogScale to override CSP headers.

Closing. Feel free to reopen with more details.


Fixes #74

This is propperly not the right way to do it. But lets talk about it.

That's not necessarily a bad approach. My only questions are 1) do we think we would need the same check in the other "ThisToThat" methods? 2) Whether we should streamline the following GetTargetDataTypeDefinition call to reuse the retrieved property data type to save potentially loading that twice?

I did some testing to see if it was necessary to do the same thing on the others "ThisToThat" methods.
But because, it has been done/set in DbToString, which writes to the umbraco cache, it shouldn't.


updated qlcolorcode (2.0.6b4)

We prefer to use stable releases, in this case, 2.0.5

2.0.5 should be fine, then?

2.0.5 has no binary release, looks like?
If there's no release within a few days after last commit, just open up a new issue, looks like the autor is pretty responsive.
https://github.com/anthonygelibert/QLColorCode/issues/2, for example


[SWARM-761] Upgrade to wildfly-camel-4.3.0

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you created a JIRA and used it in the commit message?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[ ] Have you built the project locally prior to submission with mvn clean install?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Add to whitelist