Bradley Dice

Results 467 comments of Bradley Dice

The only difference I see is that the `[source]` link was moved out of the box: ## Before ![image](https://github.com/rapidsai/cudf/assets/3943761/9d192fc1-0dc5-47e0-9048-1d0d12d9d8b5) ## After ![image](https://github.com/rapidsai/cudf/assets/3943761/96e2bc47-2417-4870-82c8-7b33862b2a8f)

Great! That's fine. It looks like the docs built so I'll merge this and we can try it with #15846.

I don't think #593 affects this issue.

This issue, or #146, would solve a problem raised by @ramanishsingh and @rsdefever at the @mosdef-hub all-hands meeting. They want to be able to put log files generated by PBS/SLURM...

Another example: naming is inconsistent between `_AggregatesStore` (plural `Aggregates`) and `_DefaultAggregateStore` (singular `Aggregate`).

@kidrahahjo Using `aggregates` in place of `jobs` would be API-breaking for the "most stable" public methods in signac-flow (it changes the name of many public keyword arguments). It would also...

> > > @bdice do you think we should rename the private `_JobOperation` class to `_JobsOperation` (or `_AggregateOperation)? @kidrahahjo You suggested a counterexample to my proposal before I even posted...

@vyasr Could you link to that discussion and copy the relevant parts into this issue for posterity? (Also sorry if I took credit for your idea, I don’t remember the...

I think the action items here are: 1. Make a single pass through `project.py` to ensure consistency in variable names after #464 is merged. 2. At a dev meeting, decide...