Update syntax file for .cabal

Incorporate changes in https://github.com/sdiehl/haskell-vim-proto/blob/master/vim/syntax/cabal.vim, notably:

field names are case-insensitive
allow whitespace between the field name and the colon

Codecov Report

Merging #1777 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1777      +/-   ##
==========================================
- Coverage   75.11%    75.1%   -0.01%
==========================================
  Files          76       76
  Lines      124994   125034      +40
==========================================
+ Hits        93886    93913      +27
- Misses      31108    31121      +13

Impacted Files
Coverage Δ

src/if_xcmdsrv.c
83.33% <0%> (-2.04%)
:arrow_down:

src/os_unix.c
58.34% <0%> (-0.49%)
:arrow_down:

src/gui.c
45.53% <0%> (-0.11%)
:arrow_down:

src/gui_gtk_x11.c
47.39% <0%> (-0.11%)
:arrow_down:

src/window.c
81.25% <0%> (-0.1%)
:arrow_down:

src/channel.c
83.8% <0%> (ø)
:arrow_up:

src/message.c
68.52% <0%> (+0.05%)
:arrow_up:

src/if_py_both.h
76.01% <0%> (+0.7%)
:arrow_up:

src/gui_beval.c
44.89% <0%> (+0.92%)
:arrow_up:

Continue to review full report at Codecov.

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

did you contact the maintainer of the file?

@Twinside What's your opinion on making this change upstreamed (vim/vim) ?

please note, that runtime file changes should be evaluated and processed by their respective maintainer. If they approve the change, they will send an update to Bram for inclusion with Vim. We don't generally process changes here, as long as the runtime file maintainer is not missing.

Instead of using ":syn match" with \c in the pattern, use ":syn case ignore" and ":syn keyword". It's a lot faster.

Yeah go ahead, I don't really have time to track the update to the cabal files

If you agree I can become the new maintainer of this file.


New version: Ultimaker.Cura version 5.1.1

Result: Installation Successful

Manifest
Add/Remove Programs

Name
Ultimaker Cura

Version
5.1.1

Publisher
Ultimaker B.V.

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 3032
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Home Assistant Quick Bar via Hotkey

Description
Use a hotkey (https://www.electronjs.org/docs/api/global-shortcut) to open the home assistant quick bar menu (https://www.home-assistant.io/docs/tools/quick-bar/). Similar to what Launchy (https://www.launchy.net/) does with executables. I was looking for a way to implement this, but frankly, I am just too new to coding as a hobby to figure it out.
Suggested Actions
User hits assigned hotkeys -> quick bar menu appears -> user types entity -> [enter] -> entity appears in small box allowing for user to interact with the entity without ever having to open Home Assistant in a browser.
Additional
It does not have to be specifically the home assistant quick bar menu, but something that functions the same way. Specifically with the entity filtering with "bits and pieces". I understand if this is out of the scope of this project.

This request does not seem to fit the scope of this project. Sorry for the erroneous request.


Task 'spotlessApply' not found in project ':api'.

It doesn’t work for me, giving the error: Task 'spotlessApply' not found in project ':api'. This error is correct because I set Spotless only for the root module, and it works with Gradle. I’m not sure why it requires adding the task to all modules.

Hello @fokkoru!
Did you apply the plugin for subprojects?
subprojects {
    apply(plugin = "com.diffplug.spotless")
}

What exact task you are running: for the file or entire project?

Hey @lipiridi
Nope, I applied the Spotless plugin only to the root project, so when I run ./gradlew spotlessApply, it formats the code for all projects. I’m also using the IntelliJ IDEA Spotless Gradle plugin and it works as well.
So, the question is: do we really need to apply com.diffplug.spotless to all subprojects, or is applying it just to the root project enough?

@fokkoru thanks for the answer.
The thing is, submodules can have different format configurations.
If a project has submodules, when you run the 'Reformat Project' command, it asks which module the task should be executed on. There’s also an option for the 'Root project.' So your case ideally should be covered. Do you see that modal window?

Yes, I see the window when I use 'Reformat Project,' but I need the option to reformat a single file. I want to use Action on Save, but in that case, it throws an error.

Now I see your point. It's fair.
So, we need to check the plugin presence in the submodule build file because I don't really want to add a new flag to the settings.
I'll try to do it soon.

@fokkoru Hi again! I was working on the new flow, and while testing, I realized that Spotless from the root project isn't applying changes to the submodules. If it's not included in the submodule plugins, it doesn't work - neither by the plugin nor by manually running the task. Could you share your Gradle build snippets?

Hmm, maybe it’s because I’m using target.
Here’s how my root project looks:
plugins {
    kotlin("jvm") version "2.0.20" apply false
    kotlin("kapt") version "2.0.20" apply false
    id("com.diffplug.spotless") version "6.25.0"
}

spotless {
    kotlin {
        target("**/src/**/*.kt", "**/src/**/*.kts")
        ktfmt().dropboxStyle().configure { opt -> opt.setMaxWidth(150) }
    }
    kotlinGradle {
        target("*.gradle.kts", "**/*.gradle.kts")
        ktfmt().dropboxStyle().configure { opt -> opt.setMaxWidth(150) }
    }
    java {
        target("**/src/**/*.java")
        removeUnusedImports()
        palantirJavaFormat()
    }
}

Yeah, this is it. I had a default target("src/*/java/**/*.java")
So I'm going to prepare the PR

Version 1.1.3 was released. Will be available in the marketplace during 2 business days.


[WIP] AppVeyor: Support MSVC

I removed the mingw64 system upgrade since it's brittle and the AppVeyor base image should be good enough for our needs
Releases are still built using mingw64
@nunoplopes There are still some compiler errors, am I doing something wrong? https://ci.appveyor.com/project/Kha/lean/build/1.0.202/job/hyhj945xsmtq1e6c#L4205

@Kha Lean compiles fine with VS 2017. It seems AppVeyor is using the previous version (VS 2015 -- 14.0)?
Can you select a more recent version? (VS 2017 aka VS 15.x).

@nunoplopes Thank you, now I'm down to two errors:
Error copying file "C:/projects/lean/build/shell/lean.exe" to "C:/projects/lean/src/../bin/".
'C:\projects\lean\src\..\bin\lean' is not recognized as an internal or external command, operable program or batch file.

I'm not sure how to debug this further remotely. Have you ever tried building Lean directly from cmake?

no, I always run it from VS directly and the copy works fine.
I managed to find the problem:
The binary is created at C:\projects\lean\build\shell\Debug\lean.exe
and then the copy cmd tries to copy from C:/projects/lean/build/shell/lean.exe
What I don't know is why lean.exe ended up in a Debug sub-directory. It's not the behavior I see from VS..

@nunoplopes  Thank you again for investigating. It looks like this is the expected behavior when using MSBuild with cmake: https://cmake.org/cmake/help/v3.0/variable/CMAKE_CFG_INTDIR.html
So I tried using NMake instead. Now it fails to run bin\lean.exe, apparently because the post-build event wasn't actually executed at all this time: https://ci.appveyor.com/project/leodemoura/lean/build/1.0.2285#L800

Correction: The post-build step presumably was executed, just not logged (it's a dedicated target now). Apparently some DLL is missing:
STATUS_DLL_NOT_FOUND             ((NTSTATUS)0xC0000135L)
Windows helpfully shows such missing DLLs in an information dialog, which of course is not quite as helpful on AppVeyor.

I was able to reproduce the problem!
The problem is that we copy lean.exe to bin but not mpir.dll. I copied it by hand and the build continued just fine.
(FYI: VS uses ninja with cmake not msbuild)

The patch in https://github.com/leanprover/lean/pull/1941 should fix the test issue in MSVC.

Thanks again, Nuno! The Ninja generator didn't seem available from the AppVeyor Windows cmake, so I'm sticking with NMake, which seems to work fine.
I see one last issue besides the test you fixed: Linking the tests with MSVC seems to be reallly slow. We're very close to AppVeyor's 60 min time limit.

I've set up the build to do link-time optimization, that's why it's so slow :)  (incrementally is fast, but we don't have incrementally here..)
You can build in debug mode instead; should be a bit faster.

Not sure it makes sense to do that for every single test executable :) . Well, I took the opportunity to add a debug build for mingw too.

Not sure it makes sense to do that for every single test executable :)

Well, definitely not for the unit tests. But if you do it for the main binary, you need to do it for all..

Okay, debug build is too slow using either compiler. Can't we use set_target_properties(lean PROPERTIES LINK_FLAGS "/LTCG:INCREMENTAL") (or leanstatic) or something to optimize just the Lean executable?

Nop, you can't just do link-time optimization of a single binary.
The reason is that once you compile a .c file with /GL, then the .o is not binary code but rather compiler IR. So it's a all or nothing thing. We can add a flag to enable/disable LTCG if that helps..

Looks good :tada:


OnEnter event only called on page load using react router

Hi - I know this issue has been flagged up a few times already, but none of the solutions (or the latest version) solve this for me.
I am having the exact same problem with Waypoint. Im using the latest version, however when switching pages via react router, the onEnter function doesn't get called when reaching the waypoint. On a fresh page reload it works as expected however.
Additionally, my react app has a few different pages (via react router), all with waypoints on, and on for some reason the waypoints work once, and then never again. My app is pulling data from a web service on each onEnter event, however these events are not being fired consistently, either only on page load, or just once and never again.
Thanks!
Justin

Is your page being remounted as a result of the route change?
If you add the debug prop to the waypoint, what's the output after a page transition?

Where does the component output to? The browser console?

Where does the component output to? The browser console?

That’s where it should log things.
Can you share some of your code? It might be easier to track this down then.

`@inject('uiStore', 'journalStore', 'authStore', 'paperStore')
@observer
class SingleJournalPage extends Component {
componentWillMount() {
console.log("SINGLE JOURNAL");
this.journalId = this.props.match.params.id;
this.props.paperStore.setFilter((e) => parseInt(e.journalId, 10) === parseInt(this.journalId, 10));
this.loadInitial();
}
loadInitial () {
if (!this.props.paperStore.isLoading) {
console.log("loading initial");
let journalId = this.journalId;
this.props.paperStore.getPapersForJournal(journalId);
}
}
loadMore (page) {
console.log("load more");
if (!this.props.paperStore.isLoading) {
let journalId = this.journalId;
this.props.paperStore.getNextPageForJournal(journalId);
}
}
render () {
const journal = this.props.journalStore.data.get(this.journalId);
return (

{journal && journal.name}

{/* <ReactPlaceholder type='media' rows={10}
style={{marginTop: '5em'}}
ready={!this.props.paperStore.isLoading}
showLoadingAnimation={true}> */}
{ this.props.paperStore.filteredView.map((element, index) => {
return (

)
})
}

<ReactPlaceholder type='media' rows={10}
style={{marginTop: '5em'}}
ready={!this.props.paperStore.isLoading}
showLoadingAnimation={true}>

);
};
}`
the loadMore function basically calls a function in paperStore which is a external api call to get more data. With my console log statements I can see that loadMore is only ever called once when reaching the waypoint.
Thanks,
Justin

Okay, and how many times do you see SINGLE JOURNAL logged?

Just once. Followed by a single load more call. Which then is never logged again.

What's the debug output from the waypoint (you can try scrolling a little after the first loadMore so that things end up in the log)?

the loadMore function only gets called once on page load so by the time the page has rendered, the initial load is there, but there is no output whatsoever from the waypoint in chrome console.

So it looks as though some styling has forced the span element that gets created as the waypoint to be sized to 0x0 so it never gets hit. Within the browser i have forced the waypoint to be 200x900 and black bg so I can see it, and when it is a forced size the load more function is called every time.

@JustinMoser any chance you can post the styles that were causing this? Sorry for the delayed response getting back to you.


New version / dependency issue

Hi there,
I've been using your code off-and-on for a few weeks. I'm hitting a new dependency error presumably due to changes in the underlying packages. Do you current get this error below? If not, could you send a complete list of the versions associated with the dependencies?
Traceback (most recent call last):
File "/content/drive/MyDrive/creative/Stable-Diffusion-Latent-Space-Explorer/run_sd_experiment.py", line 7, in
import models
File "/content/drive/MyDrive/creative/Stable-Diffusion-Latent-Space-Explorer/models.py", line 5, in
from torchvision import transforms
File "/usr/local/lib/python3.10/dist-packages/torchvision/init.py", line 6, in
from torchvision import _meta_registrations, datasets, io, models, ops, transforms, utils
File "/usr/local/lib/python3.10/dist-packages/torchvision/_meta_registrations.py", line 4, in
import torch._custom_ops
ModuleNotFoundError: No module named 'torch._custom_ops'
Thanks!

Hello,
what torch, torchvision and CUDA version are you using?
Back then when I worked on the code, I used torch 2.0.0, torchvision 0.15.1 and CUDA 11.8
If your CUDA version matches you can install the rest via:
pip install torch==2.0.0 torchvision==0.15.1 --index-url https://download.pytorch.org/whl/cu118


Making it so Powershell starts in the container.

This fixes a bug where the main application would create all processes inside a container.  This behavior is fixed.  The only process created in the container is

All powershell scripts.
The application that PSF launches.

I'm confused about why there's 17 commits - some from December.  Let's redo this PR and avoid the 'develop' branch.  Rather just merge a commit from dhoehna:fix to microsoft:master - preferably 1 squashed commit (or at least some small number of commits).


feat: Add transfer-route-owner command to cf cli

Co-authored-by: Shwetha Gururaj<EMAIL_ADDRESS>This is an intro PR for the upcoming changes made by the platform interfaces team. This PR will not be merged until sign off and successful run of integration test below once we have an environment with cf-api:edge.
Does this PR modify CLI v6, CLI v7, or CLI v8?
v8
Description of the Change
Adding the cf transfer-route-owner command to the cf cli
Why Is This PR Valuable?
When users share a route, ownership is maintained by the originating space. This new command allows transfer of ownership.
Why Should This Be In Core?
This command is needed when a route is shared and needs to be deleted or modified by the target space.
Applicable Issues
List any applicable Github Issues here
How Urgent Is The Change?
Not urgent, still working with the Platform Interface team on definition.
Other Relevant Parties
CF API transfer route owner endpoint is required

I may be behind the times, but I think we decided to change this command to move-route.
cc @MerricdeLauney

I may be behind the times, but I think we decided to change this command to move-route.
cc @MerricdeLauney

Based on this conversation i believe you are correct

thanks @Gerg & @MerricdeLauney . @gururajsh and I updated to move-route. Completely forgot about that thread.


Rename this library

Context
Data Packages, and Frictionless Data specifications, are essentially part of the protocol and inner workings of this package, but the package itself does not really require knowledge of these specs. Branding it as "Data Packages" is misleading, and potentially would confuse users into thinking that a knowledge of data packages is required to use this package.
I think we should rename the package to simply pipelines or pipeline.
What are your thoughts @akariv
Any opinion on this @brew @roll @danfowler @vitorbaptista @amercader @rufuspollock ?

Is it recognizable enough as a pipeline(s)?
Because it's just kinda a common word. Maybe data-pipeline(s)

I found I needed a passing understanding of data packages to understand how to write pipelines and processors.

I agree that pipeline(s) is probably too generic.

packaging-pipeline(s)
out there suggestion: conveyor-belt (conveyor belts can move packages from A to B)

Thinking it over, while it is true that @brew (and anyone else who wants to write their own processors) needs to know how Data Packages work in order to work with this tool, the most common user should be someone who (1) knows where her data is, (2) what it looks like, (3) where she wants to put it, and (4) how to write YAML.  Thinking back to the first sentence of the README:

datapackage-pipelines is a framework for declarative stream-processing of tabular data. It is built upon the concepts and tooling of the Frictionless Data project.

Maybe the name should reference the most important type of data it ingests and spews (tables): table-pipelines, tabular-pipelines, tabular-data-pipelines, table-factory, table-streams, tabulator-streams (probably worth emphasizing in the README the connection to tabulator)...

I've thought more and more and more like the factory concept (pipelines just suggesting moving water around - not processing it). Conveyor belts may be a bit passive but they have the factory sense. Maybe more of an assembly line ...

But pipeline is an established concept in programming

@roll good point i.e. unix pipes. However this is a bit more heavy duty than classic pipes but i think you're right.

@akariv @rufuspollock @jobarratt
I've come to think this is a crucial step to take, sooner rather than later.
Candidates:

Data Workflows: CLI dw
Data Pipelines: CLI dp
Data Factory: CLI df
Others?

Any of these will be better than the current name, and all address the above concerns that excluding "Data" (e.g.: pipelines) is confusing.
I'm happy to take a decision if needed, but I prefer to have @akariv take the call on this if he desires, as the author of the framework :).

Name for plugins are superlong (cc @brew). I was against pipelines because it's not specific enough in my mind. But it there other one word alternatives? datapipes?

I added another option to https://github.com/frictionlessdata/datapackage-pipelines/issues/69#issuecomment-327076707 after chatting with @rufuspollock yesterday: shortening "Data Workflows" to "Data Flows"

I also had in mind dataflow(s) as something short enough: dataflow-aws, dataflow-goodtables etc

i like Data Flows or Data Pipelines. Pipelines marginally more because it's more reminiscent of infrastructure

DPP isn't just moving data from one point to another, but also transforming, changing and filtering it. Not sure how that helps, but perhaps it's more of an assembly line, than a pipeline? Having said that, I like Data Flows and Data Pipelines. If I have to describe the package to someone, I say it's a data pipeline framework.

@brew if you like assembly lines we're probably a factory and our current pipelines are some combination of machines connected by conveyors ;-)


search: fix loans search permissions

Fix issue where it was possible to bypass permissions if a query was supplied to the loans search.
Closes #47

Build is failing, something broken maybe?


Cannot declare multiline function

I'm not sure what I'm missing from the readme, but I cannot seem to able to define multiline functions.
Given:
let bla = (one, two) ->
    console.log one
    console.log two

It compiles to:
let bla = function(one, two) {}
    console.log(one)
    console.log(two)

There is just a single line break after -> and the indentation is the same on the 2nd and 3rd line.
I'm using the VSC extensions, so the language server is already displaying me an error before compilation, saying
Cannot find name 'one'
Changing it to an arrow function:
let bla = (one, two) =>
    console.log one
    console.log two

Results in:
let bla = (one, two) =>
    console.log(one)
    console.log(two)

Here's the same with types:
let bla = (one: string, two: string) =>
    console.log one
    console.log two

It turns to:
let bla = (one: string, two: string) =>
    console.log(one)
    console.log(two)

Hmmm, I have VSC set to 4 spaces for indentation.
If I use 2 spaces, it works.
3 spaces creates this weird code:
let bla = (one: string, two: string) => {
  return  console.log(one)
  (console.log(two))
}

@zolomatok Currently the indentation in Civet is hard coded at two spaces and most things will work correctly with that.
I'm currently working on an update to allow for arbitrary/deeper nesting. I'll let you know when it's out.

Ah, great, thanks Daniel!
Is there an official or unofficial roadmap for Civet?
I'm very curious to get a glimpse of what state the language is in at the moment.
Seems like there's a ton of work that already been done, I mean barring a few hiccups, the language already seems to have feature parity with coffee.

@zolomatok The short term goal is to get to 97%-99% compatibility with Coffee2 using the "civet coffeeCompat" directive prologue. I'm running tests on the CoffeeScript codebase and fixing issues as I find them.
Then it's making sure there is a clear incremental migration path from some unfortunate Coffee inconsistencies (for in/of/from names are swapped, not precedence). Adding some extra features &blocks, increasing TypeScript support... and the big one of having a really nice VSCode LSP.
This section of the Readme is a pretty good place to check: https://github.com/DanielXMoore/Civet#things-removed-from-coffeescript especially the TODOs near the bottom.

Thanks Daniel, really helpful! Read the readme like 5 times but I missed that :D
+1 for optional assignment, I never understood why it's not a thing in typescript and it's always a big bummer
conditional assignment is also very nice!
Civet has a really, really bright future 🌈

Also possibly of interest is a Discord server which originally had the goal of adding TypeScript features to CoffeeScript but it has a #civet channel where Daniel announces new releases and features and such, and we discuss direction.

Works great, thank you!


[SPARK-21397][BUILD]Maven shade plugin adding dependency-reduced-pom.xml to …

…base directory
What changes were proposed in this pull request?

run ./build/mvn -DskipTests clean package and we get many files 'dependency-reduced-pom.xml'
under the source directory

run ./build/mvn clean, the files dependency-reduced-pom.xml can not be cleaned.

So i suggest to specify dependencyReducedPomLocation to 'target/' to store these files.
How was this patch tested?
manual tests

Can one of the admins verify this patch?

I like the idea, but see this note: https://maven.apache.org/plugins/maven-shade-plugin/shade-mojo.html#dependencyReducedPomLocation . I'm not clear whether this has side-effects.

@srowen Thanks for your time. I am not sure the open issue, which is mentioned in the note,  whether has bean resolved.  as the note said

setting a value for this parameter with a directory other than ${basedir} will change the value of ${basedir}
,but after running mvn checkstyle:check , i get the right path of ${basedir}/target/checkstyle-output.xml

That's not the problem this would create -- it changes basedir after execution of the shade plugin, so it might affect things like the release build. Is it really a problem? gitignore already ignores these files, even though they're a little annoying.

I just  use mvn checkstyle:check ,which contains output files with basedir, to check whether the basedir is correct with this PR.
Could you please specify the affect exactly as you said? what is your mean about release build? or does it the output package is not right?

Yes this PR is a trivial Improvement for me. i will closed it if this has side-effects exactly. Thanks.

Meaning I think you'd have to run the release process to verify the artifacts still work, because it's the process that uses shading


Ability to automatically derive traits on newtypes

 Issue by Seldaek
Tuesday Aug 06, 2013 at 21:15 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/8353
This issue was labelled with:  in the Rust repository

Disclaimer
Please bear with me, this contains long code samples but you can skip through most of them until the last one if you are only mildly interested, they're just there to illustrate my point.
Problem
While adding comments on rand::rng() I figured that it would be a lot better if the function would return a generic "interface" type for Rngs instead of returning the type of the particular struct it uses at the moment. However after discussing this with @kballard and @cmr on IRC, it seems that it is now pretty much impossible to achieve.
I'll go through the variants I researched, and then propose a solution (which may not be realistic, I have no clue about rustc internals, but I hope it is).
Failed attempt 1: Returning traits as a makeshift interface
This failed miserably because it's just not how the language works at all, and I got that now, but it's included for completeness.
use std::rand::{IsaacRng, XorShiftRng, RngUtil, Rng};

fn main() {
    let rng = rng();
    printfln!(rng.gen_bytes(10));
}

pub fn rng<T: Rng + RngUtil>() -> T {
    IsaacRng::new()
}

pub fn weak_rng<T: Rng + RngUtil>() -> T {
    XorShiftRng::new()
}

test.rs:6:4: 7:1 error: mismatched types: expected `T` but found `std::rand::IsaacRng` (expected type parameter but found struct std::rand::IsaacRng)
test.rs:6     IsaacRng::new()
test.rs:7 }
test.rs:10:4: 11:1 error: mismatched types: expected `T` but found `std::rand::XorShiftRng` (expected type parameter but found struct std::rand::XorShiftRng)
test.rs:10     XorShiftRng::new()
test.rs:11 }
test.rs:15:14: 15:31 error: the type of this value must be known in this context
test.rs:15     printfln!(rng.gen_bytes(10));
                         ^~~~~~~~~~~~~~~~~

Failed attempt 2: Using newtypes to "hide" the real implementation
This would be an ok workaround, it's not perfect but it seems to be workable given the constraints of the language and isn't too crazy, however it fails because it is missing the proper implementations:
use std::rand::{IsaacRng, XorShiftRng, RngUtil, Rng};

fn main() {
    let rng = rng();
    printfln!(rng.gen_bytes(10));
}

struct StrongRng(IsaacRng);

pub fn rng() -> StrongRng {
    StrongRng(IsaacRng::new())
}

struct WeakRng(XorShiftRng);

pub fn weak_rng() -> WeakRng {
    WeakRng(XorShiftRng::new())
}

test.rs:19:14: 19:31 error: failed to find an implementation of trait std::rand::Rng for StrongRng
test.rs:19     printfln!(rng.gen_bytes(10));
                         ^~~~~~~~~~~~~~~~~

Failed attempt 3: Implementing a decorator newtype
This probably is workable unless I don't get how to work around the compiler errors that are left, but I assume it's not a dead end. The problem though is that it is extremely verbose and seriously painful to write all this boilerplate just to have a straight decorator wrapping the underlying struct.
use std::rand::{IsaacRng, XorShiftRng, RngUtil, Rng, Weighted, Rand};

fn main() {
    let rng = rng();
    printfln!(rng.gen_bytes(10));
}

struct StrongRng(IsaacRng);
impl Rng for StrongRng {
    pub fn next(&mut self) -> u32 {
        return (**self).next();
    }
}
impl RngUtil for StrongRng {
    fn gen<T:Rand>(&mut self) -> T {
        return (**self).gen();
    }
    fn gen_int_range(&mut self, start: int, end: int) -> int {
        return (**self).gen_int_range(start, end);
    }
    fn gen_uint_range(&mut self, start: uint, end: uint) -> uint {
        return (**self).gen_uint_range(start, end);
    }
    fn gen_char_from(&mut self, chars: &str) -> char {
        return (**self).gen_char_from(chars);
    }
    fn gen_weighted_bool(&mut self, n: uint) -> bool {
        return (**self).gen_weighted_bool(n);
    }
    fn gen_str(&mut self, len: uint) -> ~str {
        return (**self).gen_str(len);
    }
    fn gen_bytes(&mut self, len: uint) -> ~[u8] {
        return (**self).gen_bytes(len);
    }
    fn choose<T:Clone>(&mut self, values: &[T]) -> T {
        return (**self).choose(values);
    }
    fn choose_option<T:Clone>(&mut self, values: &[T]) -> Option<T> {
        return (**self).choose_option(values);
    }
    fn choose_weighted<T:Clone>(&mut self, v : &[Weighted<T>]) -> T {
        return (**self).choose_weighted(v);
    }
    fn choose_weighted_option<T:Clone>(&mut self, v: &[Weighted<T>]) -> Option<T> {
        return (**self).choose_weighted_option(v);
    }
    fn weighted_vec<T:Clone>(&mut self, v: &[Weighted<T>]) -> ~[T] {
        return (**self).weighted_vec(v);
    }
    fn shuffle<T:Clone>(&mut self, values: &[T]) -> ~[T] {
        return (**self).shuffle(values);
    }
    fn shuffle_mut<T>(&mut self, values: &mut [T]) {
        (**self).shuffle_mut(values);
    }
}

pub fn rng() -> StrongRng {
    StrongRng(IsaacRng::new())
}

struct WeakRng(XorShiftRng);
impl Rng for WeakRng {
    pub fn next(&mut self) -> u32 {
        return (**self).next();
    }
}
impl RngUtil for WeakRng {
    fn gen<T:Rand>(&mut self) -> T {
        return (**self).gen();
    }
    fn gen_int_range(&mut self, start: int, end: int) -> int {
        return (**self).gen_int_range(start, end);
    }
    fn gen_uint_range(&mut self, start: uint, end: uint) -> uint {
        return (**self).gen_uint_range(start, end);
    }
    fn gen_char_from(&mut self, chars: &str) -> char {
        return (**self).gen_char_from(chars);
    }
    fn gen_weighted_bool(&mut self, n: uint) -> bool {
        return (**self).gen_weighted_bool(n);
    }
    fn gen_str(&mut self, len: uint) -> ~str {
        return (**self).gen_str(len);
    }
    fn gen_bytes(&mut self, len: uint) -> ~[u8] {
        return (**self).gen_bytes(len);
    }
    fn choose<T:Clone>(&mut self, values: &[T]) -> T {
        return (**self).choose(values);
    }
    fn choose_option<T:Clone>(&mut self, values: &[T]) -> Option<T> {
        return (**self).choose_option(values);
    }
    fn choose_weighted<T:Clone>(&mut self, v : &[Weighted<T>]) -> T {
        return (**self).choose_weighted(v);
    }
    fn choose_weighted_option<T:Clone>(&mut self, v: &[Weighted<T>]) -> Option<T> {
        return (**self).choose_weighted_option(v);
    }
    fn weighted_vec<T:Clone>(&mut self, v: &[Weighted<T>]) -> ~[T] {
        return (**self).weighted_vec(v);
    }
    fn shuffle<T:Clone>(&mut self, values: &[T]) -> ~[T] {
        return (**self).shuffle(values);
    }
    fn shuffle_mut<T>(&mut self, values: &mut [T]) {
        (**self).shuffle_mut(values);
    }
}
pub fn weak_rng() -> WeakRng {
    WeakRng(XorShiftRng::new())
}

test.rs:70:14: 70:31 error: multiple applicable methods in scope
test.rs:70     printfln!(rng.gen_bytes(10));
                         ^~~~~~~~~~~~~~~~~
note: in expansion of fmt!
<std-macros>:226:20: 226:37 note: expansion site
<std-macros>:224:4: 231:5 note: in expansion of printfln!
test.rs:70:4: 70:33 note: expansion site
test.rs:98:4: 100:5 note: candidate #1 is `__extensions__::gen_bytes`
test.rs:98     fn gen_bytes(&mut self, len: uint) -> ~[u8] {
test.rs:99         return (**self).gen_bytes(len);
test.rs:100     }
test.rs:70:14: 70:31 note: candidate #2 is `std::rand::__extensions__::gen_bytes`
test.rs:70     printfln!(rng.gen_bytes(10));
                         ^~~~~~~~~~~~~~~~~
note: in expansion of fmt!
<std-macros>:226:20: 226:37 note: expansion site
<std-macros>:224:4: 231:5 note: in expansion of printfln!
test.rs:70:4: 70:33 note: expansion site
test.rs:79:0: 122:1 error: multiple applicable methods in scope
test.rs:79 impl RngUtil for StrongRng {
test.rs:80     fn gen<T:Rand>(&mut self) -> T {
test.rs:81         return (**self).gen();
test.rs:82     }
test.rs:83     fn gen_int_range(&mut self, start: int, end: int) -> int {
test.rs:84         return (**self).gen_int_range(start, end);
           ...
test.rs:134:0: 177:1 error: multiple applicable methods in scope
test.rs:134 impl RngUtil for WeakRng {
test.rs:135     fn gen<T:Rand>(&mut self) -> T {
test.rs:136         return (**self).gen();
test.rs:137     }
test.rs:138     fn gen_int_range(&mut self, start: int, end: int) -> int {
test.rs:139         return (**self).gen_int_range(start, end);
            ...
error: aborting due to 3 previous errors

Proposal
The ideal way would be to allow the newtype to derive traits, so that making a decorator is not a PITA anymore. As you see right now it fails because the derive can't find the right impl, but I think it should just create it on the fly instead of complaining. That way we get explicit "interface" definition by defining which traits are implemented on the newtype, and we can return that and avoid a BC fiasco when we need to change the underlying RNG (see #8349 for more info on that).
I'd be happy to try and help implement this feature because I strongly believe this is needed, but to be honest I have no clue where to even start so pointers would be nice if the consensus is that it is a good thing and that it is at all feasible :)
use std::rand::{IsaacRng, XorShiftRng, RngUtil, Rng};

fn main() {
    let rng = rng();
    printfln!(rng.gen_bytes(10));
}

#[derive(Rng, RngUtil)]
struct StrongRng(IsaacRng);

pub fn rng() -> StrongRng {
    StrongRng(IsaacRng::new())
}

#[derive(Rng, RngUtil)]
struct WeakRng(XorShiftRng);

pub fn weak_rng() -> WeakRng {
    WeakRng(XorShiftRng::new())
}

test.rs:70:14: 70:31 error: failed to find an implementation of trait std::rand::Rng for StrongRng
test.rs:70     printfln!(rng.gen_bytes(10));
                         ^~~~~~~~~~~~~~~~~
note: in expansion of fmt!

Closing in favor of https://github.com/rust-lang/rfcs/issues/261.


Accept and broadcast 0-fee transactions

How does one set Electrumx to accept and broadcast transactions at 0 sat/byte (no fee at all) ?

Wrong repository, this is Electrum Personal Server not ElectrumX.


fix: do not throw error when trying to prune missing directory

Motivation
Noticed the following warning
Nov-28 10:42:04.700[]                 warn: Error pruning invalid SSZ objects persistInvalidSszObjectsDir=/data/lodestar/beacon-data/invalidSszObjects - ENOENT: no such file or directory, scandir '/data/lodestar/beacon-data/invalidSszObjects'
Error: ENOENT: no such file or directory, scandir '/data/lodestar/beacon-data/invalidSszObjects'
    at Object.readdirSync (node:fs:1503:26)
    at pruneOldFilesInDir (file:///usr/app/packages/cli/src/util/pruneOldFilesInDir.ts:6:30)
    at Timeout._onTimeout (file:///usr/app/packages/cli/src/cmds/beacon/handler.ts:100:40)
    at listOnTimeout (node:internal/timers:581:17)
    at processTimers (node:internal/timers:519:7)

This happens because we don't check if the directory actually exists before pruning it which can happen in case of invalid ssz objects dir.
Description
Do not throw error when trying to prune missing directory

:tada: This PR is included in v1.24.0 :tada:


[Bug] consume or give the fee to the validator when a transaction fails

🐛 Bug Report
To prevent attacks of using invalid transactions, the node should consume the fee even when the transaction fails.
The fee should be variable to allow legitimate higher-paying transactions to win even in the presence of transaction "spam".
Add unit tests to exercise this scenario.

Agreed, this issue has now been address with the introduction of Accepted/Rejected transactions via consensus. The fee is now being consumed and a new Transaction::Fee type was introduced to prevent attacks. Closing as resolved.


⚠ Bofrost.es has degraded performance

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

HTTP code: 200
Response time: 1601 ms

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


Clean up, add comments, formatter, and config file

@nk-coding Any further suggestions for this PR?

@nk-coding Any further suggestions for this PR?

well, except from the open comment, I have no new ones


Protect against redefining MESA_EGL_NO_X11_HEADERS

When building wlroots for an embedded Linux (hobby) project with Wayland only, I run into errors where MESA_EGL_NO_X11_HEADERS is redefined several times.
This PR simply adds a check if it is already defined, before defining it.

Thanks!


Project Idea: Galaxy Flavors

Develop training and documentation for Galaxy Flavors. Good for advanced users.

Example of Docker with installation of workflows, data libraries and data managers: https://github.com/IFB-ElixirFr/ReproHackathon/blob/master/reprohackathon1/galaxy/Dockerfile

Documentation to create a new flavor from an existing galaxy docker image: https://github.com/vjmulat/gcc2017_dockerflavor_demo/blob/master/INSTALL.md

Working In Progress with

Vipul Patel
Deepak Tanwar (create a new Galaxy flavor with tools + workflow using kitematic on windows)
@vjmulat (create a new Galay flavor with tools using kitematic on MacOS) !

@yvanlebras I also did on MacOS

What's wrong with my Dockerfile ? I cannot see workflow and cannot create labels for tools in yaml file

Guide to add tools from webhook use: https://github.com/bgruening/docker-galaxy-stable#Extending-the-Docker-Image

Concerning workflow installation:

installing workflows
    workflow-install --workflow_path $GALAXY_HOME/workflows/ -g http://localhost:8080
        -u $GALAXY_DEFAULT_ADMIN_USER -p $GALAXY_DEFAULT_ADMIN_PASSWORD

Where all Galaxy workflows needs to be in one directory, here the $GALAXY_HOME/workflows/.

oh yes @dktanwar ! I think I got it! You don't give the $GALAXY_DEFAULT_ADMIN_USER and $GALAXY_DEFAULT_ADMIN_PASSWORD ;) You have to modify it giving the real info ;)

Oh shoot! I thought that it will automatically take the default<EMAIL_ADDRESS>anyways, updated it. I will update once it is done! Thanks a lot!

@yvanlebras :  workflow.pptx

@yvanlebras Please let me know if you would like me to update anything.

thanks @dktanwar ! I will check if possible to present it!


Fix PipelineRun pruning

This fixes a potential race in the pruning of PipelineRun's. Presently these are pruned
before the relevant status update is made on the owner object. Because of the owner reference
this deletion will also update the corresponding owner, making it likely that this update
might fail. Normally failure would result in a requeue but because the oject is question
has been deleted this will no longer work properly.
This fix changes the deletion to happen after the status update, and will also prune any
PipelineRun objects that exist after the object is in its expected state. This means that
even if the prune fails a subsequent reque should allow the prune to succeed.

First test runs was green, but I have added the logging from https://github.com/redhat-appstudio/jvm-build-service/pull/169

GitHub rate limiting
/retest

/retest

/retest

/retest

these changes seem on the right path with respect to this timing issue
that said, before we merge this, I'm going to add an e2e job in this repo (vs. adding the test back to e2e-tests) that runs with pruning on
So we we will have e2e's with pruning off (to facilitate debug of new features) and pruning on (to vet these timing issues with pruning moving forward, or if/when we evolve to the more advanced throttling @stuartwdouglas and I have discsussed)
then when see get both positive feedback with this PR, and subsequent days (maybe a weeks worth) of positive results in other PRs with the new in repo pruning e2e, we'll revisit adding the e2e, with pruning on, back to e2e-tests
@Michkov @brunoapimentel @psturc FYI ^^

/hold
until I get that new e2e defined in openshift/release

@gabemontero why do you want to hold this? The current state machine is broken with respect to failures due to pruning. I can't say for sure that it is the fix to the test issue, but we won't really know until it is merged.

@gabemontero why do you want to hold this? The current state machine is broken with respect to failures due to pruning. I can't say for sure that it is the fix to the test issue, but we won't really know until it is merged.

Well, as far as I know, the only "things" hitting the current pruning bug is when we try to activate our e2e in https://github.com/redhat-appstudio/e2e-tests .... no stand alone human users that I know of are complaining that they are hitting this issue
So the e2e has already been commented out (again) in https://github.com/redhat-appstudio/e2e-tests so we are no longer blocking the other teams' e2e's
Now, if I am uninformed about human users hitting this on the staging cluster, then sure, let's reset and merge this.
Otherwise, adding the new e2e shouldn't be hard.  I'll work on the PR in a couple and hopefully @psturc can tag it for merge before he leaves today.
Then, we can get some e2e based validation (via repeated runs since it is a timing issue) yay/nay on this change.
Ideally, I can test this against the new e2e a bunch of times this afternoon and merge this by EOB for me Tuesday.
WDYT @stuartwdouglas ?

new test defined in https://github.com/openshift/release/pull/31184 ^^

ok tracking the initial in repo pruning job run https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/redhat-appstudio_jvm-build-service/171/pull-ci-redhat-appstudio-jvm-build-service-main-jvm-build-service-in-repo-with-pruning-e2e/1557047676359413760
Happen to have Phillip's PNC fallback PR also running this for comparison.

ok the new pruning e2e is clean, with logs or our new pruning messages in the operator log .... let's see how consistent things are
run 2
/test jvm-build-service-in-repo-with-pruning-e2e

also our pruning off e2e had no pruning messages

appstudio bootstrap fail
/test jvm-build-service-in-repo-with-pruning-e2e

/retest
for the builder images

passed
/test jvm-build-service-in-repo-with-pruning-e2e

passed again
going to retest again, see if we can get PaC req processor clean for potential image update the merge either way
/retest

we are running into the known PaC bug
both the jdk11 builder is complete
jvm-build-hacbs-jdk11-builder-on-pull-request-hb5v5   True        Completed   106m        93m
as is the controller
jvm-build-hacbs-jdk11-builder-on-pull-request-hb5v5   True        Completed   106m        93m
merging


ship libFuzzer.a

both arm64 and x86_64 ship this in prebuilts/clang/host/linux-x86/clang-r475365/runtimes_ndk_cxx/

cc: @pirama-arumuga-nainar do we need to ship libFuzzer in RISC-V?

prebuilts/clang/host/linux-x86/clang-r498229b/runtimes_ndk_cxx/riscv64 lgtm, so i think this is done now?

yes https://android.googlesource.com/platform/prebuilts/clang/host/linux-x86/+/refs/heads/main/clang-r498229b/runtimes_ndk_cxx/riscv64/


To-Do List for Serenity Developer Guide

Some of tasks in this list are planned, while others are just ideas on what can be done.

[ ] Movie Tutorial Sample

[x] Master/Detail In Memory Editing for Cast
[x] Show movies an actor played in as a tab
[x] Person / movie main image upload ability
[x] Person / movie multiple image upload ability
[ ] Front page for viewing movies (/title/123213/lord-of-the-rings)
[ ] Front page for viewing persons (/name/5454/keanu-reeves)

[ ] URL Shortener Sample

[ ] Service to register short URLs
[ ] List / edit interface for short URLs
[ ] API Key / Registration for customers

[ ] Help Desk / Task Management Sample

[ ] Task
[ ] Issue (Ticket)
[ ] Work Item Linking
[ ] Teams
[ ] Automatic assignment with priority
[ ] Gantt Chart?
[ ] Workflow Management and Process Template?

[ ] Background Task Sample

[ ] Bulk Mail Server

[ ] Configuration Editor Sample

[ ] Declaring Settings
[ ] Editing Interface for Settings
[ ] Handling Different Setting Scopes

[ ] Dynamic Entities / Fields

[ ] Adding Custom Fields To Existing Entities by Enduser
[ ] User definable dynamic entity and UI for it (like Dynamics CRM, Sharepoint etc)

[ ] Reporting Sample

[x] Data Only Report
[ ] Sql Server Reports
[ ] Dynamic Reporting

[x] Cancellable Bulk Action with Progress Bar Sample
[ ] Organization Tree / Multi Level Organization Editor Sample
[ ] Check List Box Sample
[x] Quick Filter Bar Sample
[ ] Advanced Filter Editing Sample
[x] Excel Output Sample
[x] Script / CSS Bundling
[ ] Grid Row Ordering Sample
[x] Formatting in Lookup Editor
[x] Column Formatters
[ ] Simple and Advanced Editor Types
[x] Ajax Actions in Grid (Delete, Perform Operation etc)
[x] Inline Editing In Grid
[ ] News Sample
[ ] Meeting Management Sample and Meeting Request with Outlook etc.
[ ] Writing Database Tests
[ ] Content Management System?
[ ] Forum?
[x] Temporarily upload a file and parse it (e.g. excel)
[x] Add extention table editing sample (in tab / dialog / integrated form)

Amazing! Which formats will be  suitable: source code, documentation, video tutorial? Are we contribute with some source code?

This is my to-do list for developer guide, but i'm open to any contributions. You may send a pull request for code. Write some chapter for Serenity guide in markdown format so i can include it. Video for tutorial in Serenity guide would be cool, as my pronunciation is a bit on poor side :)

Where is the "Data Only Report" Sample found?

Excel output is data only report

hi volkanceylan,
I am new in Serenity, i download nerp application and facing following issue.
Error	13	The command ""D:\web\NERP-master\NERP\NERP.Web\tools\node\lessc.cmd" "D:\web\NERP-master\NERP\NERP.Web\Content\site\site.less" > "D:\web\NERP-master\NERP\NERP.Web\Content\site\site.css"" exited with code 9009.	D:\web\NERP-master\NERP\NERP.Web\NERP.Web.csproj	1840	5	NERP.Web
Please guide us
Thanks,

Please open a new issue with error details in output window. This is todo list.

@volkanceylan  can you please share whats status of  Background Task Sample
Bulk Mail Server, i'm interested to study this

Volcan the checked items on this list are already in the developer guide? e.g where can be found the "Ajax Actions in Grid (Delete, Perform Operation etc)", or "Inline Editing In Grid" or "Excel output"? Should we expect a new version of the developer guide soon?
By the way, each and every one of the tasks you have listed are amazing. I wish I could help, but I'm a rookie in both, c# and serene, so I can't.

Are there any plans to bring the documentation up to date?


Publish SNAPSHOT releases to oss.jfrog.com

Looks like this: http://oss.jfrog.org/repo/io/zipkin/zipkin-common/

I can't wait to use this!


https://bethesda.net/community/ blacklisted

Weird. Sorry about that.
Whitelisted.
Please allow up to 10mins for cache to reload.


Move change log to CHANGELOG.md

Move the change log from README.md to CHANGELOG.md and change the formatting.

Part of release 1.6.2


Bug: Distribution of Responses errors

Certain questions on the Beta server are throwing an error for the Distribution of Responses chart:
:HTTPDriver for http://clickhouse:8123 returned response code 400)
 Code: 26. DB::ParsingException: Cannot parse quoted string: expected opening quote ''', got '"': while executing 'FUNCTION if(startsWith(responses, '[') :: 4, CAST(responses, 'Array(String)') :: 1, array(responses) :: 3) -> if(startsWith(re
This may be triggered by:
Issue 1002 - The database returned an unexpected error.

This appears to be related to the parsing of multiple answers in fact_problem_responses.sql: https://github.com/openedx/tutor-contrib-aspects/blob/4ea65d69307b53e5da96cedb46296c785fe3a259/tutoraspects/templates/openedx-assets/queries/fact_problem_responses.sql#L19
Seems to be because the answers are sometimes single quoted, sometimes double quoted:
 ["Un emprunt à l'anglais ou anglicisme", 'Un type de demande', 'Une question structurée et précise']
 ['Un type de demande']                                                                                │
 ['Une question structurée et précise', 'Un type de poisson']

@SoryRawyer heads up in case you can get to looking at this before I can...

Gah, sorry for missing this. I found a workaround that involves escaping single quotes and then replacing double quotes with single quotes. If you think that would be an acceptable solution, I can open a PR with that change.

Ah nice, that seems better and probably faster than the regex I was working on. A PR for that would be great. If there's a way we could bump the escaping logic up into the MV so it's only got to be processed once that would be ideal.


feat(linter) eslint plugin unicorn: no array reduce

Current dependencies on/for this PR:

main

PR #1610   👈

This stack of pull requests is managed by Graphite.


yosys synth script inserts buffers after tristates

Description
We are using tristates in the design. We noticed while running STA that a buffer followed the tristate.
We already have the option turned off in the config to add output buffers:
set ::env(PL_RESIZER_BUFFER_OUTPUT_PORTS)
We tracked it down to the yosys synth script calling insbuf:
    opt_clean -purge
    insbuf -buf {*}$::env(SYNTH_MIN_BUF_PORT)

    set stat_libs ""

We are able to work around it with this patch: https://github.com/TinyTapeout/tinytapeout-03p5/blob/main/tt_mux.patch
And here's the tt_mux source: https://github.com/TinyTapeout/tt-multiplexer/blob/main/proto/tt_mux.v
Expected Behavior
openlane shouldn't insert buffers after tristates.
Environment report
Kernel: Linux v5.15.0-71-generic
Distribution: ubuntu 20.04
Python: v3.8.10 (OK)
Container Engine: docker v20.10.21 (OK)
OpenLane Git Version: 4cd0986b3ae550cdf7a6d0fba4e0657012f635d8
pip: INSTALLED
python-venv: INSTALLED
---
PDK Version Verification Status: OK
---
Git Log (Last 3 Commits)

4cd0986 2023-02-13T14:32:59+02:00 Bugfixes (#1658) - Kareem Farid -  (grafted, HEAD, tag: 2023.02.14)
---
Git Remotes

origin  https://github.com/The-OpenROAD-Project/OpenLane (fetch)
origin  https://github.com/The-OpenROAD-Project/OpenLane (push)

Reproduction material
n/a
Relevant log output
n/a

Please provide any steps to reproduce and show where was the buffer inserted.

the outputs should be  ->  ->
but instead they are  ->  ->  ->


chore: update Finsupp.fun₀ comment after resolution of issue

The issue has been resolved, the panic is no longer there, but the doc-string can still only be modified in the file where it is defined.

Thanks! maintainer merge

Let's see if the bot picks it up this time:
maintainer merge

Thanks!
bors merge


LabResult and LabTotal Class and edit test cases accordingly

Fixes #420
Fixes #340
Fixes #423
Fixes #331

Codecov Report

Merging #419 (503cf86) into master (a5f1eee) will increase coverage by 1.16%.
The diff coverage is 59.82%.

@@             Coverage Diff              @@
##             master     #419      +/-   ##
============================================
+ Coverage     59.88%   61.04%   +1.16%
- Complexity      541      572      +31
============================================
  Files           100      103       +3
  Lines          2064     2146      +82
  Branches        222      234      +12
============================================
+ Hits           1236     1310      +74
- Misses          736      743       +7
- Partials         92       93       +1

Impacted Files
Coverage Δ

...seedu/programmer/logic/commands/AddLabCommand.java
62.50% <0.00%> (ø)

...grammer/logic/commands/DashboardCommandResult.java
100.00% <ø> (+100.00%)
:arrow_up:

...u/programmer/logic/commands/ShowCommandResult.java
36.36% <ø> (+3.03%)
:arrow_up:

...edu/programmer/logic/parser/EditCommandParser.java
61.11% <0.00%> (-3.60%)
:arrow_down:

...du/programmer/model/student/UniqueStudentList.java
84.21% <ø> (ø)

src/main/java/seedu/programmer/ui/CommandBox.java
0.00% <0.00%> (ø)

.../main/java/seedu/programmer/ui/CommandHistory.java
78.57% <0.00%> (+3.57%)
:arrow_up:

...main/java/seedu/programmer/ui/DashboardWindow.java
0.00% <0.00%> (ø)

...c/main/java/seedu/programmer/ui/LabResultCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/programmer/ui/MainWindow.java
0.00% <0.00%> (ø)

... and 32 more

Continue to review full report at Codecov.

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


[EPD-465] Fix React Conversation hooks to work with Elevenlabs Streaming

While using import { useConversation } from "vocode" in a React frontend, I am seeing the following error when using Elevenlabs synthesizer with experimental_streaming=True:
Uncaught (in promise) DOMException: Failed to execute 'decodeAudioData' on 'BaseAudioContext': Unable to decode audio data
Likely a case of not properly handling the promise in the React hook, a quick fix for this will be very appreciated :)
EPD-465

What was the fix for this? Can you please share it @ajar98 ?
I'm running an older version of vocode codebase which has been heavily modified, would be really grateful to see the commit(s) which made the fixes


[stable/oauth2-proxy] Add @compleatang to owners

Now that I'm added to the organization; happy to help maintain this chart.

/assign @unguiculus
/assign @mattfarina

/lgtm

/approve


Font problem

Bomb timer , spectator font seems to appear in different sizes sometime?
is this a bug?

Using windows7 may cause this problem

I am on Win 10 .

I'm sorry I don't know. In Windows 7, I see fonts that suddenly get bigger or smaller. This does not happen in windows10

it happens rarely . quick restart game will fix it so... not big deal XD

Are you change your resolution of your game after inject osiris ?
Like from 16:9 to 4:3,

Nope. but its alright.


Add PXT_TARGET #define in codal

This is generic version of https://github.com/microsoft/pxt-microbit/pull/3597 for future use

Do I need to port this? Or is this just for future?

Just for future i think


mechops real

About the PR
Ports #32346 from Wizden.
Why / Balance

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.
[X] I was playing HEATS while porting this PR.

Changelog
:cl: Mocho, ThunderBear2006, NULL882

tweak: The H.O.N.K. has received an airtight cabin for honk operations in outer space.
add: Added the Ripley MK-II, a heavy, slow all-purpose mech, featuring a pressurized cabin for space operations.
add: Added the Clarke, A fast moving mech for space travel, with built in thrusters (not certain if they work properly though :trollface:)
add: Added the Gygax, a lightly armored and highly mobile mech with enough force to rip walls, or someone's head off.
add: Added the Durand, a slow but beefy combat suit that you dont want to fight in close quarters.
add: Added the Marauder, a specialized mech issued to ERT operatives.
add: Added the Seraph, a specialized combat suit issued to ??? operatives.
add: The syndicate has started issuing units under the codenames "Dark Gygax" and "Mauler" to syndicate agents at an introductory price.
add: The exosuit fabricator can now be emagged to reveal new recipes.
add: There are 4 new bounties cargo can fulfill for mechs. Feedback on the cost/reward is welcome!

no promises on tests passing, i on the other hand am certainly about to pass out

no promises on tests passing, i on the other hand am certainly about to pass out

YAML LINTER :goberserk:

no promises on tests passing, i on the other hand am certainly about to pass out

YAML LINTER :goberserk:

seems to be a property thats not even a datafield right now? idk im just gonna use a placeholder value for now

I'm not sure if it was ready on wizden. Better to wait when EE will finish them.

I'm not sure if it was ready on wizden. Better to wait when EE finish them.

does he know chat

ME WHEN MECH WARS

2 pirasmas test fails, and that one heisentest related to profiles, should be good to merge once the merge button havers get to play around with the PR

Cog is such a shit map, it caused checkfail for mechs


Print is truncated at 1 page

Repro

Go to https://docs.mattermost.com/overview/product.html and print

Expected

Print the entire document

Observed

Print is only for first page, see screenshot:

Possible fixes:

Propose opening this as a Help Wanted ticket

This might be helpful info.  If I print the document in Firefox and then choose the "simplify page" option at the top, it prints all 3 pages.  Otherwise, only the first.

thanks @toni517b super helpful, hopefully we can figure out how to default to full page print...

@jespino Apparently this issue is still relevant today. I tried cherry picking @icelander's fix but the fix did not work.
Therefore, I added some print medias in the PR shown above.
Before:


url encoding not working

I am trying jquery-address for the first time and have downloaded version 1.4.
I dropped the "crawling" sample folder into my apache2 server site root and browsed http://mysite.mydomain/crawling. The page loads fine onto the home page, but all the deep links produce a "404".
I noted the following:
The link for "learn more" address in the browser has taken the format - http://mysite.mydomain/crawling/#!/%2Flearn-more
If I manually change the encoding for "/%2F" to just "/" it works fine.
This principle is true for all the deep links.
Is this a bug or have I missed a fundamental step somewhere?

Related to #96

Have you considered that the row in crawling example:
$('.content').html($('.content', data).html()).parent().show();
is encoding all urls because $('.content', data).html();
and all ajax loaded links will be similar to href="#%21/learn-more" instead href="#!/learn-more"
This may solve the problem temporally:
$('.content').html(decodeURIComponent($('.content', data).html())).parent().show();


QuickTX collateral selection results in InsufficientCollateral

I'm trying to send a scriptTx using QuickTx but getting: "InsufficientCollateral (Coin (-12084272)) (Coin 307193)"
Collateral summary:
IN
5f7e7cae5ac68122fd07ae76977034802977c160e5d015cbeeb80db2e4b618d2 - index 0 - 9793445 lovelace
3e2bdf7f96ff13d552bd7a4979b04f710665fca59b98af8e1461a0fe31475c97 - index 1 - 2598020 lovelace
OUT collateralReturn:   12084272
Total collateral:              307193
Error:
InsufficientCollateral (Coin (-12084272)) (Coin 307193)
The first issue is that, even though an exact UTXO for the DEFAULT_COLLATERAL_AMT is present, the UTXO selection strategy doesn't always select this one.
Additionally, the calculated total collateral seems to be insufficient...

maybe good to use a dedicated "fixed value" UTXO selection strategy first with the aim of selecting an UTXO containing exact 5 ADA before using default UTXO selection strategy
it would be helpful to add functionality to QuickTx allowing users to specify collateral UTXO (hash + index) explicitly. When set, this would have priority over UTXO selection (and can be used as-is, i.e without changes based on CollateralPercentage)

Full Transaction (pre-prod):
{
  "body" : {
    "inputs" : [ {
      "transactionId" : "f02f73f6732020a2251d47360809a1475e4401f359ce8d7faed42479711b49a4",
      "index" : 0
    } ],
    "outputs" : [ {
      "address" : "addr_test1qrqhq0xp6ach453pm5cfjmhg3jg68tcvlgn80nv8zsfth956mxp5gzhjqj8xj9zdlrg6v6nr9jqwpzrvf0vmmrhuztdstmsl7l",
      "value" : {
        "coin" : 9795205,
        "multiAssets" : [ ]
      },
      "datumHash" : null,
      "inlineDatum" : null,
      "scriptRef" : null
    } ],
    "fee" : 204795,
    "ttl" : 37064850,
    "certs" : [ ],
    "withdrawals" : [ ],
    "update" : null,
    "auxiliaryDataHash" : "FrS36GUFCrqmie6wVQvEh9Grvzo82xluIuOeHzhOzmE=",
    "validityStartInterval" : 0,
    "mint" : [ ],
    "scriptDataHash" : "qYfkutoyIPZ678LGuPvvkjZFVSkqawGa0S08BnXOI8k=",
    "collateral" : [ {
      "transactionId" : "5f7e7cae5ac68122fd07ae76977034802977c160e5d015cbeeb80db2e4b618d2",
      "index" : 0
    }, {
      "transactionId" : "3e2bdf7f96ff13d552bd7a4979b04f710665fca59b98af8e1461a0fe31475c97",
      "index" : 1
    } ],
    "requiredSigners" : [ "wXA8wddxetIh3TCZbuiMkaOvDPomd82HFBK7lg==" ],
    "networkId" : null,
    "collateralReturn" : {
      "address" : "addr_test1qrqhq0xp6ach453pm5cfjmhg3jg68tcvlgn80nv8zsfth956mxp5gzhjqj8xj9zdlrg6v6nr9jqwpzrvf0vmmrhuztdstmsl7l",
      "value" : {
        "coin" : 12084272,
        "multiAssets" : [ ]
      },
      "datumHash" : null,
      "inlineDatum" : null,
      "scriptRef" : null
    },
    "totalCollateral" : 307193,
    "referenceInputs" : [ ]
  },
  "witnessSet" : {
    "vkeyWitnesses" : [ {
      "vkey" : "JUZu4RKSKZJIkr/BsL/JJtf6nvaHAtn1dwZSKWwuQEc=",
      "signature" : "zQKYkbJl1NFEA279yVow4hKXOZDYUe5zimgWFPQ5fuDy/ZtUy+HUSz0iz5wAzftuk89HqLGzHuQYK0ETrQeoDA=="
    } ],
    "nativeScripts" : null,
    "bootstrapWitnesses" : null,
    "plutusV1Scripts" : [ {
      "type" : "PlutusScriptV1",
      "description" : null,
      "cborHex" : "59014f59014c01000032323232323232322223232325333009300e30070021323233533300b3370e9000180480109118011bae30100031225001232533300d3300e22533301300114a02a66601e66ebcc04800400c5288980118070009bac3010300c300c300c300c300c300c300c007149858dd48008b18060009baa300c300b3754601860166ea80184ccccc0288894ccc04000440084c8c94ccc038cd4ccc038c04cc030008488c008dd718098018912800919b8f0014891ce1317b152faac13426e6a83e06ff88a4d62cce3c1634ab0a5ec133090014a0266008444a00226600a446004602600a601a00626600a008601a006601e0026ea8c03cc038dd5180798071baa300f300b300e3754601e00244a0026eb0c03000c92616300a001375400660106ea8c024c020dd5000aab9d5744ae688c8c0088cc0080080048c0088cc00800800555cf2ba15573e6e1d200201"
    } ],
    "plutusDataList" : [ {
      "constructor" : 0,
      "fields" : [ {
        "constructor" : 0,
        "fields" : [ {
          "constructor" : 0,
          "fields" : [ {
            "bytes" : "c1703cc1d7717ad221dd30996ee88c91a3af0cfa2677cd871412bb96"
          } ]
        }, {
          "constructor" : 0,
          "fields" : [ {
            "constructor" : 0,
            "fields" : [ {
              "constructor" : 0,
              "fields" : [ {
                "bytes" : "9ad983440af2048e69144df8d1a66a632c80e0886c4bd9bd8efc12db"
              } ]
            } ]
          } ]
        } ]
      }, {
        "constructor" : 0,
        "fields" : [ {
          "constructor" : 0,
          "fields" : [ {
            "bytes" : "c1703cc1d7717ad221dd30996ee88c91a3af0cfa2677cd871412bb96"
          } ]
        }, {
          "constructor" : 0,
          "fields" : [ {
            "constructor" : 0,
            "fields" : [ {
              "constructor" : 0,
              "fields" : [ {
                "bytes" : "9ad983440af2048e69144df8d1a66a632c80e0886c4bd9bd8efc12db"
              } ]
            } ]
          } ]
        } ]
      }, {
        "constructor" : 1,
        "fields" : [ ]
      }, {
        "constructor" : 0,
        "fields" : [ {
          "constructor" : 0,
          "fields" : [ {
            "bytes" : "e16c2dc8ae937e8d3790c7fd7168d7b994621ba14ca11415f39fed72"
          }, {
            "bytes" : "4d494e"
          } ]
        }, {
          "int" : 11604
        } ]
      }, {
        "int" : 2000000
      }, {
        "int" : 2000000
      } ]
    } ],
    "redeemers" : [ {
      "tag" : "Spend",
      "index" : 0,
      "data" : {
        "constructor" : 1,
        "fields" : [ ]
      },
      "exUnits" : {
        "mem" : 20499,
        "steps" : 8023468
      }
    } ],
    "plutusV2Scripts" : null
  },
  "auxiliaryData" : {
    "metadata" : {
      "json_metadata" : {
        "674" : {
          "msg" : [ "Jadex: cancel order" ]
        }
      },
      "cbor_metadata" : "a11902a2a1636d736781734a616465783a2063616e63656c206f72646572"
    },
    "nativeScripts" : [ ],
    "plutusV1Scripts" : [ ],
    "plutusV2Scripts" : [ ]
  },
  "valid" : true
}

Result{successful=false, response='{"error":"Bad Request","message":""transaction submit error ShelleyTxValidationError ShelleyBasedEraBabbage (ApplyTxError [UtxowFailure (UtxoFailure (FromAlonzoUtxoFail (BadInputsUTxO (fromList [TxIn (TxId {_unTxId = SafeHash \"3e2bdf7f96ff13d552bd7a4979b04f710665fca59b98af8e1461a0fe31475c97\"}) (TxIx 1),TxIn (TxId {_unTxId = SafeHash \"5f7e7cae5ac68122fd07ae76977034802977c160e5d015cbeeb80db2e4b618d2\"}) (TxIx 0)])))),UtxowFailure (UtxoFailure (FromAlonzoUtxoFail NoCollateralInputs)),UtxowFailure (UtxoFailure (IncorrectTotalCollateralField (Coin (-12084272)) (Coin 307193))),UtxowFailure (UtxoFailure (FromAlonzoUtxoFail (InsufficientCollateral (Coin (-12084272)) (Coin 307193))))])"","status_code":400}', code=400, value=null}

@stikkos  I have faced similar issue while testing a smart contract transaction with Blockfrost backend. I am not sure if those are related.
Here is my scenario.
In same method,

First lock some fund at a script address
Unlock fund

#2 fails with "IncorrectTotalCollateral" and "BadInputsUTxO". In my situation, it seems that the collateral inputs selected by the UtxoSelectionStrategy in step #2 have already been spent, and step #2 is retrieving outdated utxos from Blockfrost.
I'm not sure whether this is an issue with the BFUtxoService because we don't cache anything.
To validate my assumption, I modified my code to have different senders for steps 1 and 2. In this setup, sender1 locks the transaction, and sender2 unlocks it, all within the same method. This approach worked.
I'm curious if you've encountered a similar scenario.

It would be helpful to add functionality to QuickTx allowing users to specify collateral UTXO (hash + index) explicitly. When set, this would have priority over UTXO selection (and can be used as-is, i.e without changes based on CollateralPercentage)

This is a good idea. Let's add a new method collateralUtxos() to ScriptTx.

Hi Satya, that makes sense and yes I did notice some occasional failures due to IncorrectTotalCollateral or BadInputsUTxO.
I think it's more likely a race condition then a caching issue....
My assumption is that the collateral UTXO output is indeed spend by a different tx (submitted slightly earlier) but maybe this tx is not yet included in a block when the UTXO selection happens (only in mempool) but is included when the 2nd transaction is submitted? It's only an assumption though, I'm not very familiar with low-level tx validations...

Let's add a new method collateralUtxos() to ScriptTx.

Thanks, allowing to set explicit collateral in ScriptTx would be nice.

Thanks @stikkos. I will verify again the flow to see if it's present in the library's code base. I've never encountered this issue while testing with a Yaci DevKit instance (with a 1-sec block time).
By the way, I think we can also add another method to QuickTxBuilder, something like withExclusion(Set<Utxo>). This would exclude the specified utxos from UtxoSelection. It could be useful in scenarios other than collateral.


[FEATURE] NLog - Ability to add custom fields to the log event.

Hi,
Would be great if there was a way to dynamically serialize key-value pairs from the MDLC as top-level fields in the event body, without placing them under the metadata field.
For example:
Using the latest version of Elastic.CommonSchema.NLog, an MDLC holding the key-value pairs "foo":"bar" and "abc":123 would be serialized into the event like this:
{
    "metadata" : {
        "foo": "bar",
        "abc": 123
    }
}

I think an option should exist to have them serialized directly at the top level of the event like this:
{
    "foo": "bar",
    "abc": 123
}

On issues similar to this (#104 and #141) others have already suggested using a logstash or ingest pipeline to rename the fields. I find that for larger scale multi-tenant setups, it works out much better to let each app owner get their desired fields right in their app "at the edge".
This can help by simplifying the process ("what you write is what you get"), giving developers more freedom to make changes independently, and removing the additional overhead of having to maintain one or more pipelines.

See also https://github.com/NLog/NLog/issues/4982 for an example of customizing your own NLog JsonLayout that outs JSON in Ecs-Common-Schema-Format. Where you are free to customize it in whatever direction you like.

Thank you for raising this feature request.
With the new version known ECS properties will be picked up automatically. However unknown properties will always need to be mapped explicitly.
While with Serilog you can create generic sinks using your own EcsDocument subclasses utilizing its extension mechanisms: https://github.com/elastic/ecs-dotnet/tree/main/src/Elastic.CommonSchema?rgh-link-date=2023-02-07T13%3A10%3A15Z#extending-ecsdocument
I am not quite seeing a way to do the same our EcsLayout and continue to support the xml config as well: https://github.com/elastic/ecs-dotnet/tree/main/src/Elastic.CommonSchema.NLog#how-to-use-from-nlogconfig

Closing this since a customized NLlog JsonLayout is the only way forward here.


Mismatched dependencies in busted-2.0.rc12-0.rockspec

Please verify the base version dependencies before publishing the versioned (non-scm) rockspec - fails to run due to mis-matched lua_cliargs (must be 2.5-5 due to API changes) as well as (intentionally deprecated dependency on) luasocket with the internally set version, 2.0.rc11-0 (also mismatched to spec version).
busted/modules/cli.lua:111: attempt to call method 'add_flag' (a nil value)

This has broken our continuous integration overnight and I've had to peg these until this is updated.
Thanks - it's likely an honest mistake :)

I'm encountering the same issue this morning as well.

I'm also getting this (though with the rc11 rockspec). See https://travis-ci.org/daurnimator/lua-http/jobs/132782969

sigh, sorry, completely spaced out on that. I released 2.0.rc12-1 just now which fixes that. cliargs is correctly 3.0-1 now as of https://github.com/Olivine-Labs/busted/pull/458/files.

Get an issue with latest one
Installing of travis failed with
Error: Directory busted-2.0.rc12-1 not found inside archive v2.0.rc12-1.tar.gz
Lua 5.2.4
Luarocks 2.3.0

@alexsergeyev hmm, I see this:
luarocks install busted
Installing https://luarocks.org/busted-2.0.rc12-1.rockspec...
Using https://luarocks.org/busted-2.0.rc12-1.rockspec... switching to 'build' mode
Updating manifest for /usr/local/lib/luarocks/rocks-5.2
busted 2.0.rc12-1 is now built and installed in /usr/local (license: MIT <http://opensource.org/licenses/MIT>)

Lua 5.2.4
Luarocks 2.3.0

Maybe the rock cache hadn't updated yet? Try again.

@alexsergeyev did this start working on it's own or is it still broken?

Not sure if related, but I'm seeing the rockspec refer to a non-existent file (standalone_loader):
$ luarocks install --verbose --local https://raw.githubusercontent.com/Olivine-Labs/busted/master/busted-2.0.rc12-1.rockspec
Using https://raw.githubusercontent.com/Olivine-Labs/busted/master/busted-2.0.rc12-1.rockspec... switching to 'build' mode

os.execute:     gunzip -c v2.0.rc12-1.tar.gz|tar -xf -
Results: 1
  1 (number): 0

Error: Build error: Failed installing busted/modules/standalone_loader.lua in /home/vagrant/.luarocks/lib/luarocks/rocks-5.1/busted/2.0.rc12-1/lua/busted/modules/standalone_loader.lua

Our CI is also broken because of this. This seems like an easy bug that can be caught by busted's own CI. I remember something similar has happened couple months ago too. Given how critical busted is to the Lua community, is there any plan create a test case to catch similar issues in the future?

No plan as I can't reproduce the issue. Mind making a PR?

Sorry, I was referring to version 2.0.rc11-0.  I can reproduce with the following travis setup:
git clone https://github.com/torch/luajit-rocks.git
pushd luajit-rocks
    git checkout 6529891
    cmake . -DWITH_LUAJIT21=ON -DCMAKE_INSTALL_PREFIX=${TRAVIS_BUILD_DIR}/install
    make install
popd

mkdir $HOME/.luarocks
cp ${TRAVIS_BUILD_DIR}/install/etc/luarocks/config.lua $HOME/.luarocks/config.lua
echo "wrap_bin_scripts = false" >> $HOME/.luarocks/config.lua
luarocks --local install busted 2.0.rc11-0
luajit /home/travis/.luarocks/bin/busted

However, what I don't understand is 2.0.rc11-0 was working fine for us for a long time. It just started failing couple days ago. I am not familiar with how luarocks manages dependencies. Did lua_cliargs break the API between version >= 2.5-0, < 3.0.rc-1?

Just
luarocks install --local https://raw.githubusercontent.com/Olivine-Labs/busted/master/busted-2.0.rc12-1.rockspec

is enough for me to reproduce. Since the archive generated by GitHub doesn't change for the 2.0.rc12-1 tag (if the tag wasn't overwritten), maybe the rockspec was changed? It refers to busted/modules/standalone_loader.lua which is definitely not in the archive.

Yes, they did break the API and we had a problem with not pinning the dependency properly. It was resolved in the latest rc.


POC: Introduce Node.js 6+ compatibility mode for gulp@3

Refs: https://github.com/gulpjs/gulp/issues/1640#issuecomment-240683358
This is unfinished POC, it lacks the documentation and perhaps the tests.
It works though, and successfully unbreaks gulp@3 with Node.js 6+ if gulp was required using require('gulp/compat').
This is a pure opt-in, it does not change anything unless the user specifically enables it (except for installing one extra unused package).
If gulp is already loaded at the time of the opt-in, then it throws an error —  so that deps won't misuse it and change the behaviour of everything, — this makes this option available only to the top-level gulpfile itself.
If this would be accepted, I will gladly move vinyl-fs-03-compat to gulpjs both on GitHub and npm.
/cc @phated, @contra, @TheAlphaNerd, @jasnell
Thoughts?

Would it potentially make sense to wrap the require of graceful-fs in a try-catch. If the require fails then opt to using the compat version.
This could be a good way to allow for a transparent fix for those on affected versions.

I find this an interesting workaround. Let me think on it for awhile.

@phated Would it be ok to transfer vinyl-fs-03-compat now? Asking in advance because I don't want to spam you =).

@ChALkeR no worries.  If we decide to pull this workaround in, I'll create a branch/tag in vinyl-fs proper

@phated Ah, ok. That makes sense =).

just fyi, related to this: https://github.com/nodejs/node/pull/8166. It doesn't solve the issue but it's one small step.

Another note: this is not a separate module, but a mere separate entry point for these reasons:

That requires less changes for the users to integrate, it's a one-liner this way.
If any dependencies used in the gulpfile also do require('gulp') on their own for some reason — this top-level opt-in would propogate to those.
Introducing a separate module would likely raise more concerns and recelive less usage.
It would be harder to support if this was a separate package.

Ah, this approach has a drawback.
The propogation would only work if the gulp loaded by require('gulp') is the exact same package, not a separate copy in a deeper node_modules dir. This could be fixed by making the compat flag in vinyl-fs-wrap.js stored per-process (i.e. in some global storage), not per-gulp-module.
Thoughts?

The propogation would only work if the gulp loaded by require('gulp') is the exact same package, not a separate copy in a deeper node_modules dir.

Is this even supported by task managers like Gulp?

This could be fixed by making the compat flag in vinyl-fs-wrap.js stored per-process (e.g. in global), not per-gulp-module.

Had this been necessary, could we use a symbol to minimize the possibility of a collision?

@mgol

Is this even supported by task managers like Gulp?

I could definitely imagine people doing require('gulp') in some package out there for some reason. And I could imagine such package being required from the gulpfile, for some reason. It doesn't even matter if that code path that actually uses the require('gulp') result would be not taken in the dependency — require('gulp') would crash itself.
Note that about 2943 packages depend on gulp via the dependencies chain (even without the devDependencies). That's about 1% of all npm packages.

@mgol e.g. sifttt is both managed using a gulpfile with a top-level require('gulp') and does require('gulp') inside the package itself: markbirbeck/sifttt/blob/master/lib/beam/GulpSource.js#L3

Note<EMAIL_ADDRESS>was released by @isaacs and that version does not trigger the warning anymore, so this can be closed now.

Closed, will reopen if any issues arise with the new graceful-fs or if the change there gets reverted for some reason.

Works for me.  Thanks to everyone that contributed towards a solution!

For posterity and anyone else with gracefsul-fs@3: this was done with https://github.com/isaacs/natives/blob/master/index.js, which also uses a bunch of undocumented and internal things, relies on implementation details, and also has some other hacks (e.g. it's basically a noop on the buffer module, and that one was just the module that will definitely have issues with that approach, and other modules also could be affected, e.g. I suspect util is affected).
That perhaps shouldn't manifest itself in the near future in regards to graceful-fs@3, (hence it was ok to close this now and leave things as they are), but in the long term noone should ever user graceful-fs below v4 and noone should ever use that natives module in their own projects.


GUI source

Hey there! Is there no way to get gui part source code?

Hey! We are thinking about it, and, if it's not a secret, what would you do with it?
Do you have something specific in mind or just want to take a look?

I guess you are using Facebook's pop? I haven't any experience with animation and I'd like to see how do you got these. Also I made a simply analog for me, but with 7zippin' and aes encryption after all. It'll be quite nice for me, if I can get your awesome UI in my app.


This fixed: type number format be saved as a string by SetCellValue. …

PR Details
This fixed: type number format be saved as a string by SetCellValue.
eg:
type Number int
SetCellValue( s , A1, Number(10))
SetCellValue( s , A2, Number(10))
SetCellFormula( s, A3,  "SUM(A1,A2)")
result:
A1,A2  be save as a string:  '10
A3 is 0 if you open the file

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

type Number int
you can try it
package main

import (
	"fmt"

	"github.com/xuri/excelize/v2"
)
type Number int
func main() {
	f := excelize.NewFile()
	defer func() {
		// Close the spreadsheet.
		if err := f.Close(); err != nil {
			fmt.Println(err)
		}
	}()
	var value Number = 10
	if err := f.SetCellValue("Sheet1", "A1", value); err != nil {
		fmt.Println(err)
		return
	}
	if err := f.SetCellValue("Sheet1", "A2", value); err != nil {
		fmt.Println(err)
		return
	}
	if err := f.SetCellFormula("Sheet1", "A3", "SUM(A1,A2)"); err != nil {
		fmt.Println(err)
		return
	}
	if err := f.SaveAs("Book1.xlsx"); err != nil {
		fmt.Println(err)
	}
}

ok ,i understood.
maybe using generic to constrain inputs is a better to avoid irregular param

The issue #1546 talk about generic supports. I closed this PR, if you have any question, please let me know, and you can reopen this anytime.


If user skips Epoch, the application will return error

might be solved by PR #151 , but not sure, should observe

@AnthonyMadia could you help us to verify this one? if its fixed? thank you

Yea, looks like I do not have this error. In the console, I see that Epoch x Ended with Y {_hex: '0x1e', _isBigNumber: true}  after the ended message. Nothing about an error in the queue operation for me. @ChialiTsai

Thank you for verifying @AnthonyMadia
if looks good for you, I'll let u close the ticket, you are our gatekeeper :)


Error on react

Hi, I would like to use your library in my react project, but when I import it it gives me this error

Can you help me?

@AlessioGesuelliUnicam I have the same problem.
module is not defined which they have the same cause, using module.export and export default at the same time. 😢
https://github.com/bptlab/chor-js/issues/181

Any updates? 🥲
@AlessioGesuelliUnicam


[REQUEST] Add inbuilt dotenv-expand functionality

The dotenv-cli package comes with inbuilt dotenv-expand functionality.
This is appropriate to add to dotenv-flow-cli instead of dotenv-flow, because dotenv-expand can be used alongside dotenv-flow already, like so:
const dotenv = require('dotenv-flow');
const dotenvExpand = require('dotenv-expand');

dotenvExpand(dotenv.config());

However, dotenv-expand doesn't have inherent CLI support - which is why dotenv-cli has it built in.
It's for this reason that dotenv doesn't include dotenv-expand out of the box as well, since they can be used separately/together.
In the interim, I'm getting around this by using dotenv-cli and pointing it manually to my .env.local file like this:
dotenv -e .env.local <my commands>

It would be nicer/simpler to use dotenv-flow-cli if it supported variable expansion :)

Agree with that but dead project..


fix: custom err detection [APE-713]

What I did
Issue with custom errors.
fixes: #126
Fixes: APE-693
How I did it

Detect custom error message
Raise ContractLogicError instead of VirtualMachineError
Pass in Hex of error type. Going to make eth-tester use hex str instead of bytes for consistency next

Note this related issue which can further enhance these features: https://github.com/ApeWorX/ape/issues/690
Since we know the ABI, we should be able to do cool stuff here!
How to verify it
Checklist

[x] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[x] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

EthTester gives us: "execution reverted: b'\\x82\\xb4)\\x00'"
Is there an easy way to go from the string "b'\\x82\\xb4)\\x00'"" (note the inner-stringified bytes of the string) to what Hardhat gives us 0x82b42900` ?
So tldr, still not consistent because it is a little cleaner on Hardhat's side after this PR.


Only take port from pending_appservers when checking availability

The app engine controllers only keep track of their own pending instances. The first part of the instance key does not contain an IP address.

https://ocd.appscale.com:8080/job/Daily Build/3807/


Credit card CSV extra headers not auto stripped

Example headers:
Credit card transactions;;Customer number: xxxx
MasterCard xxxxx
Outstanding/current transactions till: 01/08/2017
Billing date: 01/25/2017
Voucher date;Date of receipt;Reason for payment;Foreign currency;Amount;Exchange rate;Amount;Currency

Expect all but the last row to be automatically removed on import.

Really? How will you figure out which row is the real header?
I just posted an issue in the parent project ( https://github.com/halloffame/ynab-csv/issues/23) asking for a variable where we can select the number of header rows. That allows for files with 0 (just show the first row example data to choose columns), and files with several. We know that n-1=header row.
automatic would be awesome, but I can't imagine how to do it without big assumptions.
I wanted to submit a patch, but the project doesn't build on Linux (unsupported platform for fsevents, apparently). Thanks for all your work!

@ohthehugemanatee I made one assumption: the CSV will not parse properly if an incorrect header row is used. Then, the trick is just defining what "properly" means and extracting that info from the parse result. Turns out, Papaparse does a nice job of recording any errors is encounters while parsing. For our purposes, I'm defining parse failure as the first row of data throwing a FieldMismatch error (too many or too few values). This could be too strict for some, but I've found DB always sets the right number of values for data rows. Extraneous header rows, on the other hand, have far fewer "fields", so the parsing will definitely fail according to my criteria. Then, it's just a matter or detecting the failure, stripping off the first row (the bad header) and trying again. This is how I fixed #3.
For this issue specifically, it appears my failure criteria are actually not strict enough. If 2 consecutive "extra" header rows have the same number of "fields", then it will actually register as a success and continue parsing. Rows 2 and 3 in my sample data each have just one field (no delimiters), which causes this to happen. I've just solved this by also imposing a rule that there must be at least 3 fields (date, amount, payee). There are still cases where a false positive success is possible (e.g. 2 "extra" header rows have 4 fields each), but this is sufficiently unlikely for me and does not appear in any of my imports in any case.
Perhaps this approach feels too fragile for you (or the parent maintainer), and I agree a user-specified header number is generally more robust. However, it works well for me and prevents an extra step from the user.
I hope that answers your question


What do you feel were the biggest challenges in creating the foundation?

This is a great question! We think everyone is going to have their own take on this so we want to address it in our next live broadcast.


chore: remove KUMA_GATEWAYS_UI feature flag [DO NOT MERGE]

chore(policies): remove MeshGateway
chore: remove KUMA_GATEWAYS_UI feature flag
Resolves #2185.
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>

Guessing the backend API no longer has MeshGateway in it either?

Not, yet, but seeing as nothing can break, this doesn’t have to be timed to go in together. The only consequence is that there will be a window where MeshGateway is still listed under policies. I’ll get this picked up now, too.


Queries for account info occasionally throw an error

Describe the bug
Queries for account info occasionally throw an error due to insufficient fee. This is due to a bug in the SDK which incorrectly compares the query cost (actual cost) with the maximum set for the client or query.
This issue will be closed when the SDK has been fixed in this respect.
Expected behavior
Queries do not throw an error

resolved with sdk 2.0.15

resolved with sdk 2.0.15


Polish translation

Hi, @trembacz
Thank for your contribution!
I'll notify you know when the version is updated. :)


news update- sc15

r? @philloooo

@philloooo More edits completed.  Can you review today?

so i made several edit on this branch. Can you pull the latest and check if
the pdf works as expected?
On Fri, Jan 22, 2016 at 10:15 AM htroyer<EMAIL_ADDRESS>wrote:

@philloooo https://github.com/philloooo More edits completed. Can you
review today?
—
Reply to this email directly or view it on GitHub
https://github.com/LabAdvComp/cdiswebsite/pull/3#issuecomment-173965501.

Ok, the rest of my changes are done.  Ready to merge @philloooo

looks good! Merged and pushed to amazon


Published crates include some junk files

I guess these are sitting ignored in Erick's clone? @erickt you should make a fresh clone for publishing.
We can close this after confirming that the next release does not include these.

The serde_macros crate also includes a ton of junk:
bar.rs
bench-606762c8cf051d2b
foo
foo.d
foo.dSYM/
foo.ll
foo.rs
gen1.rs
gen1.rs.bk
gen1.rust
gen2.rs

And serde_json contains a file called "old" which looks like an old Cargo.lock.

cargo has a feature where you can limit the included files in a published package that might be useful here. Documentation is here http://doc.crates.io/manifest.html (search for include = or exclude =.
I've found just using include = and being explicit about what files go in the published packages works well for my crates.

Thanks Cody. I will look into using include = and explicitly listing the files we want.

Ugh I forgot that cargo did that. I've gotten a clean checkout, and will look into using include = to keep this from happening again.


Apollo6.0 module Camera is not running

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04): Linux Ubuntu 18.04
Apollo installed from (source or binary): source
Apollo version (3.5, 5.0, 5.5, 6.0):  6.0
Output of apollo.sh config if on master branch:

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

./docker/scripts/dev_start.sh
./docker/scripts/dev_into.sh
./apollo.sh build_opt_gpu
./scripts/bootstrap_lgsvl.sh
./scripts/bridge.sh
Run the LGSVL simulator with Apollo 6.0 use Example in https://www.svlsimulator.com/docs/python-api/dreamview-api/

Supporting materials (screenshots, command lines, code/script snippets):

dv.set_hd_map('Borregas Ave')
dv.set_vehicle('Lincoln2017MKZ')

You can check Lincoln2017MKZ is Apollo 6.0 (modular testing) or Apollo 5.0 (full analysis), Apollo 6.0 (modular testing)  has no Camera.

In general, we can't fully control the timing, which is related to the system scheduling, but this may not have a great impact because the time is short


Add --exit-non-zero-on-fix

When enabled, we exit zero if any errors are fixed, even if the fixes result in there being no remaining lint violations.
Closes #2633.

When enabled, we exit zero if any errors are fixed, even if the fixes result in there being no remaining lint violations.

Presumably, "when enabled, we exit non-zero if any errors are fixed...".
Thanks for implementing this, @charliermarsh! :pray:


C# Fix common resource-name imports in snippets

Codecov Report

Merging #2007 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2007      +/-   ##
============================================
+ Coverage     85.19%   85.21%   +0.02%
- Complexity     4983     4985       +2
============================================
  Files           457      457
  Lines         20058    20092      +34
  Branches       2105     2105
============================================
+ Hits          17088    17122      +34
  Misses         2269     2269
  Partials        701      701

Impacted Files
Coverage Δ
Complexity Δ

...gle/api/codegen/transformer/SampleTransformer.java
98% <ø> (ø)
10 <0> (ø)
:arrow_down:

...oogle/api/codegen/util/csharp/CSharpTypeTable.java
87.71% <100%> (ø)
27 <1> (ø)
:arrow_down:

...sformer/csharp/CSharpGapicSnippetsTransformer.java
97.81% <100%> (+0.26%)
41 <3> (+2)
:arrow_up:

Continue to review full report at Codecov.

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


Resync semver versions for both edinburgh and master

Signed-off-by: Ernesto Ojeda<EMAIL_ADDRESS>

@jamesrgregg @MightyNerdEric @cloudxxx8 We need to re-sync the git-semver versions on this repository to fix the merge jobs.

@jpwhitemn @JPWKU we need some help to review this one and get it fixed for Edinburgh.  Could you please review and merge this one first before #36


Expose marked object

It would be neat if this plugin could expose the marked object it is using, so it can be modified. marked exposes (for example) the Renderer class which is useful to modify with custom rendering. This is not possible using only this plugin now, but requires only a single line of code to make possible.

:+1:
Btw, no need to open both a PR and issue. PRs are issues ;)

Sorry for opening both, realised afterwards that I could do it myself :laughing:


Consider filter null children

Children could be null when I use preact, change the toArrayChildren to something like this would solve this problem.
export function toArrayChildren(children) {
  const ret = [];
  React.Children.forEach(children, (child)=> {
    if(child != null) ret.push(child);
  });
  return ret;
}

https://github.com/react-component/animate/issues/7


Was able to fix it on Series 7 on watchOS 9

I have successfully edited the code to work on watch OS 9. I do need to fix a couple bugs then I'm going to do a pull request.

could i get the .ipa? so you can install it?
Thank you very much for the progress.


clean up random seeding

@ronuchit the change to update_config is not super ideal, but the alternatives -- updating all current and future unit tests to explicitly set seed, or keep things the way they are now -- seem less good. lmk what you think though.
cc @NishanthJKumar , who originally pointed out the seed(0) grossness.

oops this seems fundamentally broken, disregard until further notice...

okay unfortunately we can't seed the action space / options in the environment for the reason I specified in the new comment. things I tried that didn't work:

use __post_init__ -- doesn't work because environments aren't dataclasses, so this method is never called
change all environments to call super().__init__() at the end of their __init__() rather than the beginning, but this quickly got messy because some envs use rng inside of __init__().

okay np


ig-json-parser library problem

Hello,
butterknife is a default library to development, so I must use it! Congratulations!
But I would like to use json parser by Instagram (https://github.com/Instagram/ig-json-parser/).
And this part of gradle block butterknife:
compileJava {
  doFirst {
    // Directory should exists before compilation started.
    generatedSourcesDir.mkdirs()
  }
  options.compilerArgs += [
                       '-processor',
                       'com.instagram.common.json.annotation.processor.JsonAnnotationProcessor',
                       '-s',
                       generatedSourcesDir
  ]
}

What is the proper way to use both libraries together?
bg,
Larten

Use the android-apt plugin. This is a problem with their library and their very poor recommendation to use that technique.

Thank you!


mentions in logging plugins don't link correctly

they show as just the id within the mention syntax. this is most likely to do with the allowed mentions feature

presumed resolved by 266637480db0ddb1bf5bce2536b3f2947b21ede6


Text offsets, and centered text

Hi! I've been struggling with the new text system, and was getting really frustrated with the offset of the text in-game.
(It's probably worth noting i copy-pasted the existing code to put it in 2.9.4 and I'm not using IntPtr).
But I did some calculations and found exactly what works.
This is what I put in the InternalDraw function, at the very top.
float internalOffset = ScaledHeight /
                        (Font == GTA.Font.Pricedown || Font == GTA.Font.HouseScript
                            ? 2
                            : (Font == GTA.Font.Monospace
                                ? Screen.AspectRatio
                                : 1.5f)); // These are aspect ratios, assuming 1.5f is a perfect aspect ratio. and 2 being

float x = (Position.X + offset.Width) / screenWidth;
float y = (Position.Y + offset.Height - internalOffset) / screenHeight;
float w = WrapWidth / screenWidth;

[Centers the text to it's origin when doing width / 2, height / 2]
And here's some of the properties that I believe would be of great use to some:
// Get's raw height value 0-1
public float RawHeight
{
    get { return Function.Call<float>((Hash)0xDB88A37483346780, Scale, (int) Font); }
}

// Get's text pixel height scaled to 720p
public float ScaledHeight
{
    get { return RawHeight * Screen.Height; }
}

// Get's actual height based on game resolution
public float Height
{
    get { return RawHeight * Game.ScreenResolution.Height; }
}

[These actually work, it blows my mind. You can legit get the text's total pixel height!!]
Hope someone can implement this and do the internalOffset some mathematical justice if it's wrong.

No need to keep this open any longer IMO as we have implemented all the features except for the raw height.


Update required Windows version for WinHttpHandler to include WS2019

Ready for review, not ready for merge. A new release is required before merging the documentation.

Internal previews

📄 File
🔗 Preview link

aspnetcore/grpc/netstandard.md
Use gRPC client with .NET Standard 2.0

...not merging per instruction above.


sane: really fix udev error

#12874 tried to patch out the literal /bin/sh, but an additional space
sneaked in.
cc @aszlig

Thanks!