Wildcards in XML path

It would be nice to have wildcards in 'xml_path':
- name: Generate Lint Annotations
  uses<EMAIL_ADDRESS>  with:
    xml_path: '**/build/reports/lint-results-*.xml'

Wildcards should be compatible with workflow syntax:

*: Matches zero or more characters, but does not match the / character.
**: Matches zero or more of any character.

Currently, it is necessary to provide a specific path, e.g.:
- name: Generate Lint Annotations
  uses<EMAIL_ADDRESS>  with:
    xml_path: ${{ github.workspace }}/app/build/reports/lint-results-debug.xml

Sorry for the README update stopping.
In v1.10, glob format path is supported.
Would you resolve by this?

Yes, sorry, version 1.1.0 works just as expected.
I just got a dependabot alert about this version.


modify activity event

What is this about? Pls create a JIRA issue about this. And notice the dangerjs comments.


Support parsing of indentation-based languages

I was using a bunch of indentation-based languages, like CoffeeScript, Jade, and want to create DSLs by myself.
I find some tricks maintaining indentations in pegjs by searching, was was wondering if there's a consistant solution:
http://stackoverflow.com/questions/11659095/parse-indentation-level-with-peg-js
http://stackoverflow.com/questions/4205442/peg-for-python-style-indentation
https://gist.github.com/jakubkulhan/3192844
https://groups.google.com/forum/#!searchin/pegjs/indent/pegjs/RkbAB4rPlfU/xxafrY5wGCEJ
But will pegjs support this feature?

I think this problem is best solved by allowing state in general, just like Pegasus does and as suggested in #285. Here is an idea (the following is Pegasus’ significant whitespace grammar translated to pegjs and with my syntax idea added):
{var indentation = 0}

program
  = s:statements eof { return s }

statements
  = line+

line
  = INDENTATION s:statement { return s }

statement
  = s:simpleStatement eol { return s }
  / "if" _ n:name _? ":" eol INDENT !"bar " s:statements UNDENT {
      return { condition: n, statements: s }
    }
  / "def" _ n:name _? ":" eol INDENT s:statements UNDENT {
      return { name: n, statements: s }
    }

simpleStatement
  = a:name _? "=" _? b:name { return { lValue: a, expression: b } }

name
  = [a-zA-Z] [a-zA-Z0-9]* { return text() }

_ = [ \t]+

eol = _? comment? ("\r\n" / "\n\r" / "\r" / "\n" / eof)

comment = "//" [^\r\n]*

eof = !.

INDENTATION
  = spaces:" "* &{ return spaces.length == indentation }

INDENT
  = #STATE{indentation}{ indentation += 4 }

UNDENT
  = #STATE{indentation}{ indentation -= 4 }

Note the #STATE{indentation} blocks near the bottom (obviously inspired by Pegasus). I call those state blocks. The idea is to allow a state block before actions. Here is a more complicated state block:
#STATE{a, b, arr: {arr.slice()}, obj: {shallowCopy(obj)}, c}

It is shorthand for:
#STATE{a: {a}, b: {b}, arr: {arr.slice()}, obj: {shallowCopy(obj)}, c: {c}}

In other words, after the shorthand expansion has been applied, the contents of a state block is a list of identifier ":" "{" code "}". Adding a state block before an action tells pegjs that this action will modify the identifiers listed, and if the rule is backtracked those identifiers should be reset to the code between the braces.
Here are the compiled functions for INDENT and UNDENT from the above grammar above, with resetting of the indentation variable added:
    function peg$parseINDENT() {
      var s0, s1, t0;

      s0 = peg$currPos;
      t0 = indentation;
      s1 = [];
      if (s1 !== peg$FAILED) {
        peg$reportedPos = s0;
        s1 = peg$c41();
      } else {
        indentation = t0;
      }
      s0 = s1;

      return s0;
    }

    function peg$parseUNDENT() {
      var s0, s1, t0;

      s0 = peg$currPos;
      t0 = indentation;
      s1 = [];
      if (s1 !== peg$FAILED) {
        peg$reportedPos = s0;
        s1 = peg$c42();
      } else {
        indentation = t0;
      }
      s0 = s1;

      return s0;
    }

And here’s a bit of how the “complicated state block” from above could be compiled:
s0 = peg$currPos;
t0 = a;
t1 = b;
t2 = arr.slice();
t3 = shallowCopy(obj);
t4 = c;
// ...
if (s1 !== peg$FAILED) {
  // ...
} else {
  peg$currPos = s0;
  a = t0;
  b = t1;
  arr = t2;
  obj = t3;
  c = t4;
}

What do you think about this idea of being able to:

Tell pegjs about which stateful variables will be modified by an action.
Supply the code needed to store those variables if they need to be reset. (Including shorthand syntax for the simple case where the variable is a primitive value.)

And what do you think about the syntax?

Hi guys,
Just to clarify, am I correct that it is better not to use PEG.js (with workarounds from the top of this issue) with indentation-based languages till this issue is closed?
Thanks.

@hoho I don't get you meaning.. But I later found another solution to parse indentations with parser combinator like solutions and it worked. And I think my original indention to parse indentations with PEG.js gone.

I mean there are workarounds to parse indentation, but the comments say that these workarounds will fail in some certain cases.

Let me clarify the situation: Parsing indentation-based languages in PEG.js is possible. There are various solutions mentioned above and I just created another one as I tried to get a “feel” for this (it’s a grammar of a simple language with two statements, one of which can contain indented sub-statements — similar to e.g. if in Python).
One thing common to all the solutions is that they need to track the indentation state manually (because PEG.js can’t do that). This means there are two limitations:

You can’t compile the grammar with caching safely (because the parser could use cached results instead of executing state-manipulating code).
You can’t backtrack across indentation levels (because there is currently no way to unroll the state when backtracking). In other words, you can’t parse a language where there are two valid constructs which can be disambiguated only after a newline and indentation level change.

Limitation 1 can cause performance issues in some cases, but I don’t think there are many languages for which limitation 2 would be a problem.
I’m OK with this state until 1.0.0 and I plan to circle back to this topic sometime afterwards. The first level of improvement could be getting rid of limitation 2 using more explicit state tracking (as suggested above) or by providing a backtracking hook (so that one can unroll the state correctly). The second level could be getting rid of the need to track indentation state manually by providing some declarative way to do so. This could help with limitation 1.

H, I wrote a (tiny, hacky) patch for PEG.js that supports proper backtracking, as I explained here: https://github.com/pegjs/pegjs/issues/45

sorry for the bump 😜
I was just looking into creating CSON and YAML parsers for a language I am designing, and while looking on ways to create a indention based parser with PEG.js, I came up with a simple method that:

doesn't rely on push/pop state's
asserting indention levels via code within actions

It had occurred to me that either of the above 2 solutions actually adds performance problems to the generated parsers. Additionally in my opinion:

relying on state's not only adds a ugly PEG.js syntax but also can affect what type of parsers that can be generated as they would need to support action based state handing.
sometimes adding some code in actions results in a language dependent rule, and for some developers that means they can't use plugin's to generate parsers for other languages like C or PHP without resorting to more plugin's to handle actions on rules, which just means a bigger build system just to support 1 or 2 changes.

After a while I started about creating my own variant of the PEG.js parser and thought: why not just use increment (“++”) and decrement (“--”) prefix operators (++ expression and -- expression) to handle the results of match expressions (expression * or expression +).
The following is a example grammar based on @dmajda's Simple intentation-based language, rewritten to use the new ++ expression and -- expression instead of & { predicate }:
Start
  = Statements

Statements
  = Statement*

Statement
  = Indent* statement:(S / I) { return statement; }

S
  = "S" EOS {
      return "S";
    }

I
  = "I" EOL ++Indent statements:Statements --Indent { return statements; }
  / "I" EOS { return []; }

Indent "indent"
  = "\t"
 / !__ "  "

__ "white space"
 = " \t"
 / " "

EOS
  = EOL
  / EOF

EOL
  = "\n"

EOF
  = !.

Much more pleasing to the eye, no? Easer to understand too, both for humans and software.
How does it work? simple:

Indent* tells the parser that we want 0 or more of what Indent is returning
++Indent tells the parser to increase the minimum amount of matches required for Indent
Now any time the parser is about to return the matches for Indent, it first expects it to be 1 more match then before, otherwise peg$SyntaxError gets thrown.
--Indent tells the parser to decrease the minimum amount of matches required for Indent
Now any time the parser looks for Indent and returns the matches it expects 1 less match then before, otherwise peg$SyntaxError gets thrown.

This solution is the best way to add support for 'Significant Whitespace Parsing' without adding an ugly syntax to PEG.js grammars or blocking 3rd party generators.
Here's the changed rules to add support for parsing this in src/parser.pegjs:
{
  const OPS_TO_PREFIXED_TYPES = {
    "$": "text",
    "&": "simple_and",
    "!": "simple_not",
    "++": "increment_match",
    "--": "decrement_match"
  };
}

PrefixedOperator
  = "$"
  / "&"
  / "!"
  / "++"
  / "--"

SuffixedOperator
  = "?"
  / "*"
  / "+" !"+"

Am I right to assume that to support it compiler/generator side we will have to:

add a compiler pass that ensures ++ expression or -- expression are only being used on expression * or expression +, where expression must be of types: choice, sequence or rule_ref
add a cache based check in the generated parser for expression * or expression + that asserts the minimum required match is met before returning the matches
optionally add a helper method for generated parsers to implement that returns number of matches required for a given rule, eg. nMatches( name: String ): Number

@futagoza, this is clean and clever. I like it. I am working on a parser that handles state, but the only state we really need is indentation levels. I may use this idea and give you credit for it. Tracking the indentation level still effectively requires pushing/popping state and so it may still prevent some optimizations but the semantics of this are very nice.
@kodyjking what do you think of this?

@lydell I tried the Pegasus inspired solution without luck in the Online PEG.js editor
Line 39, column 5: Expected "!", "$", "&", "(", ".", character class, comment, end of line, identifier, literal, or whitespace but "#" found.

@futagoza Do you have a fork/branch with the indentation patch enabled and a small sample grammar?

I'm working on this fork/branch indentation
@krinye "I recommend adding the @ prefix operator as well. It's purpose is to simply extract a single rule result out of a sequence"
Could one of you guy have a look and make and make a comment or PR with the fix. Thanks :)

I debugged... increment_match-not-found

What year can we expect 1.0 and indentation support? 2020?
Sorry...

Ah, didn't notice the caveat:
Am I right to assume that to support it compiler/generator side we will have to:

add a compiler pass that ensures ++ expression or -- expression are only being used on expression * or expression +, where expression must be of types: choice, sequence or rule_ref
add a cache based check in the generated parser for expression * or expression + that asserts the minimum required match is met before returning the matches
optionally add a helper method for generated parsers to implement that returns number of matches required for a given rule, eg. nMatches( name: String ): Number

Just for kicks, I tried adding this in visitor.js
      increment_match: visitExpression,
      decrement_match: visitExpression,

Now I get Invalid opcode: undefined.

@kristianmandrup concerning the @ operator to extract single values from sequences, I have a fork with just that feature added to PegJS available here:
https://github.com/krisnye/pegjs
It's a pretty simple addition.

@krisnye +1 for the grammar based implementation, nice and simple. If you don't mind, I will be adding this to my variant of PEG.js 😄
@kristianmandrup see you commit for my suggestion

@futagoza Please do.
I was discussing the indentation logic and I suggest the following syntactic
// increments the named state variable
identifier++
// decrements the named state variable
identifier--
// repeat constant amount or state variable (with default zero if variable not yet incremented)
rule{integer | identifier}
// repeat min/max using constants or state variables
rule{integer | identifier, integer | identifier}
The parser we are working on can handle arbitrary state, but honestly, the above is all that is needed for indentation parsing.

Thanks a lot guys! If its so easy to do, why not make a "dedicated" fork where the things you mention "just works" ;)
Cheers!

@krisnye

Using identifier++ can easily cause a messy error if the developer meant identifier+, that's why I chose to go with ++identifier and, for consistency, --identifier
As mentioned in a different issue about ranges, rule{ STATE_REPEAT / RANGE } can be confused with rule{ ACTION }, especially if you are building a syntax highlighter for PEG.js, so this approach has been denied by @dmajda

@kristianmandrup (OFF TOPIC) I am good at designing features, and sometimes making implementations, but horrible at testing and benchmarking them, so usually I make working variants (without any tests or benchmarks) in private non-repo directories on my pc, and then forget about them 🤣. For my current variant of PEG.js (named ePEG.js 😝, an extended rewrite of PEG.js) I am adding the stuff mentioned here, as well as other features (ranges, imports, templates, etc), so I am adding tests and benchmarks, but I am also currently working on a C++ project which is taking my time, so there's no ETA on that.

@futagoza Thanks mate :) Looking at the feature extensions, but doesn't mention indentation support. Is that included but undocumented or coming down the road?
Someone else from this list pointed me at other grammar/parser solutions that I might look into as well. Keep me posted! Cheers!

@kristianmandrup It's not included as far as I can tell, but @dmajda said 3 years ago that he will look into it after releasing PEG.js v1, but from what I can tell that wont be for another 2 years, unless he plans to release more minor versions of PEG.js v0 (0.12, 0.13, etc)

I meant if you had included indentation support in ePEG already or on the roadmap?

@kristianmandrup oh 😆, its on the roadmap

@futagoza agreed on the ++/-- as pre operations, and I forgot about action syntax, we changed it to [min,max]
So
++identifier
--identifier
rule[integer | identifier]
rule[integer | identifier, integer | identifier]

@krisnye [ ... ] is used for character classes, see https://github.com/pegjs/pegjs#characters
What I'm doing in ePEG.js is adding ranges (also on the roadmap) to achieve what I think your describing:
space = [ \t]*
rule = range|expression

expression can be any of these: ++space, --space or space
range can be min.., min..max, ..max or exact
min, max or exact can only be an unsigned integer
using a range with an expression (e.g. 2|expression) can allow us to set the total amount of expression required for rule to parse successfully.
using the exact range with ++expression or --expression (e.g. 3|++expression) can allow us to set the integer amount for ++ or --, which is by default 1
using either min or max range with ++expression or --expression throws a syntax error.

I'm not using state variables as that would just be confusing with rule identifiers.
Using a combination of range, ++, -- or @ I'm hoping to create PEG grammar files that rely less on a rules action for their result, which should increase the development time of whitespace (e.g. indentation-based, ASCII art, etc) languages as the language designers and/or implementers don't have to worry about trying to confirm if the correct amount of whitespace has been parsed.
This should also allow plugin developers to create parser generators that can optimise with less fear of what our action's language is (by default JavaScript, but with plugin's can change it to CoffeeScript, PHP, etc).

So it seems like it's still not possible today to parse Python with PEG.js out of the box, is that right?
If not, is this something that will be coming soon?  Is there a set of tasks required to make this work that people could contribute?
I have a project where I'd love to be able to get a Python AST in JS, modify it and then convert it back to source code with the same formatting, so I'd be interested in making this happen if there is a clear roadmap.

@mindjuice Not yet, it's planned for post 1.0.
If you cannot wait, my nephew and I created a parser written in TypeScript that uses the same syntax and handles indentation. It's not documented yet, but it's pretty simple. There is still work ongoing as we're using it as the parser for a new language design.
https://github.com/krisnye/pegs

You can also try another parser generator with support such as chevrotain

I believe it's very possible to parse python-like indentation with PEGjs.
The example below uses only four-space-based indentation, but it can be extended to cover arbitrarily-spaced tabs and other whitespace characters.
In fact, the language I'm working on has a more complicated indentation story than Python has and this grammar works well for it.
{
    let prevIndentCount = 0;
}

Indent 'indent'
    = i:("    "+) {
        let currentIndentCount = i.toString().replace(/,/g, "").length;
        if (currentIndentCount === prevIndentCount + 4) {
            // DEBUG //
            print("=== Indent ==="
            print("    current:"+currentIndentCount);
            print("    previous:"+prevIndentCount);
            print("    lineNumber:"+location().start.line);
            // DEBUG //
            prevIndentCount += 4;
            return "[indent]";
        }
        error("error: expected a 4-space indentation here!")
    } // 4 spaces

Samedent 'samedent'
    = s:("    "+ / "") &{
        let currentIndentCount = s.toString().replace(/,/g, "").length;
        if (currentIndentCount === prevIndentCount) {
            print("=== Samedent ===");
            return true;
        }
        return false;
    }

Dedent 'dedent'
    = d:("    "+ / "") {
        let currentIndentCount = d.toString().replace(/,/g, "").length;
        if (currentIndentCount < prevIndentCount) {
            // DEBUG //
            print("=== Dedent ===")
            print("    current:"+currentIndentCount);
            print("    previous:"+prevIndentCount);
            print("    lineNumber:"+location().start.line);
            // DEBUG //
            prevIndentCount -= 4;
            return "[dedent]";
        }
        error("error: expected a 4-space dedentation here!");

FunctionBody
    = Newline Indent FunctionSourceCode (Newline Samedent FunctionSourceCode)* Dedent


fix: Use ignored variable in WorkflowServiceCollectionExtensions.AddDaprWorkflow

Use ignored variable in WorkflowServiceCollectionExtensions.AddDaprWorkflow
Description
Use ignored variable in WorkflowServiceCollectionExtensions.AddDaprWorkflow to remove duplication. (biggest nit ever)
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #1055
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests (no impact)
[ ] Extended the documentation

@tomkerkhove - Looks like there is a merge conflict, can you take a look at that?

@halspang We are good to go!


Remove sub-request functionality

out of curiosity, are sub-requests being superseded by something 'better' or just considered a bad idea?
I am currently investigating using sub-requests to support multiplexed update requests via PATCH  - where the PATCH payload is actually a batch of partial updates to possibly multiple different records and which need to be unpacked and dispatched as if they were individual PUT / POST / DELETE requests. Kind of like https://tools.ietf.org/html/rfc6902 but for REST requests. This reduces the number of individual REST requests.
If not sub-requests, any suggestions on how this could still work?

Resolved.


How to configure this library in ios RN-Cli

I have added the already  permissions.
And i am try to setup the IOS manually than show this error in Xcode
#import "React/RCTBridgeModule.h" this file not found show error .
And  Adding the following under "Build Settings" -> "Header Search Paths".
$(SRCROOT)/../node_modules/react-native/React
But this error not solve..
And this code working  in android but i am trying  to record in iOS i am facing this error very time this--

ERROR  Error in micHandler: [Error: Error occurred during recording]
ERROR  Error in micHandler: [Error: Failed to stop recorder. It is already nil.]
ERROR  Error uploading audio: [TypeError: Network request failed]

---------------------------------- This code is android perfectly  working but IOS not working --------------------------------
const micHandler = async () => {
const baseDir = ${RNFS.DocumentDirectoryPath}/cinch-all-audio;
try {
  if (!isRecording) {
    await RNFS.mkdir(baseDir, {NSURLIsExcludedFromBackupKey: true});
    const fileName = `${Date.now()}.mp3`;
    const audioPath = `${baseDir}/${fileName}`;
    const audioSet = {
      AudioEncoderAndroid: AudioEncoderAndroidType.AAC,
      AudioSourceAndroid: 6,
    };

    // Start recording
    const result = await audioRecorderPlayer.startRecorder(
      audioPath,
      audioSet,
    );
    setIsRecording(true);
  } else {
    const result = await audioRecorderPlayer.stopRecorder();
    setIsRecording(false);

    const parts = result.split('/');
    const fileName = parts[parts.length - 1];
    const reference = ref(storage, `/${'recordedAudio'}/${fileName}`);
    const pathToFile = result;

    try {
      const fileData = await fetch(pathToFile).then(response =>
        response.blob(),
      );
      const uploadAudioFile = await uploadBytes(reference, fileData);

      if (uploadAudioFile) {
        const audioURL = await getDownloadURL(reference);
       console.log(audioURL);
      } else {
        console.error('File upload failed');
      }
    } catch (error) {
      console.error('Error uploading audio:', error);
    }
  }
} catch (error) {
  console.error('Error in micHandler:', error);
  alert(error);
}

};

i faced some issue, fixed by changing extension from .mp3 -> .m4a. I dont know why but on ios mp3 doesnt work. On android it also NO issues with m4a


Pip 6.0+ compatibility fixes

Use stdlib logger in newer versions of pip
See https://github.com/pypa/pip/pull/2008 for details

I did happen to eyeball it, but in the future, yes, your r+ is plenty good enough!

I did happen to eyeball it, but in the future, yes, your r+ is plenty good enough!

I see we don't have Travis running the full Cartesian product of pip and Python versions tox does. We should do something about that sometime.

I see we don't have Travis running the full Cartesian product of pip and Python versions tox does. We should do something about that sometime.

In case I was unclear, merge away. Thanks for the patch, @jgmize!

issue #67 covers fixing travis to run all the tests.
issue #68 covers updating tox.ini to cover pip 6.0, 6.0.1 and 6.0.2.
I think that's everything outstanding here. Merging!


Stats for competitive season 5 is not reset.

Issue
Stats was not reset to 0 in the new season. Example profile below.
Links that cause the issue
https://owapi.net/api/v3/u/Half-2673/stats

I think that behavior is expected. It will return previous season info until you calibrate in new season.

We only return what is on your public profile, you public profile returns your old season stats until you have new ones.

Yes, this is intended behaviour. My profile even contains my Season 1 competitive rank still.

I see... That explains it.. Guess i have to figure out a clever solution on my end. Thanks for the replies :-)


Remove classes from the Toolbar Component

In #1482 we removed the styles on the class .pf-c-toolbar__pagination, this class is not needed anymore as we just use .pf-c-pagination so we can remove it after the breaking change period.
You'll find it in toolbar-pagination-example and toolbar-expanded-example

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


start event not triggered

I'm trying to use the code:
    Pace.on('start', function(){
        console.log('hello');
    });

But don't works.
The events 'done, hide, stop', works like a charm.

My guess is you're binding to the event after it happens.  The start event should happen immediately when Pace is loaded, unless you disable startOnPageLoad using the paceOptions object before Pace is loaded onto the page.

    <script src="pace.min.js"></script>
    <script>Pace.on('start',function(){console.log('hello');});</script>

Thats is my code on
But with the option startOnPageLoad works now.

@zackbloom You were exactly right for my case. Mistakenly put my "start" callback in the $(document).ready(function() {}) block. Thanks!

Try this:
<script>

	window.paceOptions = {
	  startOnPageLoad: false
	}

</script>
<script src="http://cdn.rawgit.com/HubSpot/pace/master/pace.js"></script>
<script>

	Pace.on('start', function(e){
		console.log('Pace.start')
		console.log(e)
	})

	Pace.start()

</script>


Bump MDM/MDSD image versions

Which issue this PR addresses:

Fixes https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/14466762/
What this PR does / why we need it:

Addresses vulnerabilities in our docker images.
Test plan for issue:

Existing E2E
Is there any documentation that needs to be updated for this PR?

N/A

/azp run e2e


Input object can't have nested input object field

Setup
I have AInput that contains an inner BInput field that I want to setup like this:
@gqlInputObjectType()
export class AInput {
  @gqlField({
    class: "AInput",
    type: "BInput",
  })
  bInput: BInput;

  constructor(bInput: BInput) {
       this.bInput = bInput;
  }
}

@gqlInputObjectType()
export class BInput {
  @gqlField({
    class: "BInput",
    type: GraphQLID,
  })
  id: string;

  @gqlField({
    class: "BInput",
    type: GraphQLInt,
  })
  quantity: number;

  constructor(id: string, quantity: number) {
    this.id = id;
    this.quantity = quantity;
  }
}

But I can't regen the code due to this error:
field bInput references BInput which isn't a graphql object

I tried setting up BInput using @gqlObjectType instead, which seems to successfully regen, but the graphql type is incorrect as it needs to be an Input type not just a generic type, so the server just errors out.
Question
Should there be a separate @gqlInputField we can use? Or is there a setup I'm missing? Thanks!

Nvmind figured out part of it, need to use custom type instead of this setup

What was the custom type solution you used?
Looking at the example, it feels like it should work so may still be worth
fixing eventually.
On Tue, Feb 20, 2024 at 22:17 Ashley Xue @.***> wrote:

Nvmind figured out part of it, need to use custom type instead of this
setup
—
Reply to this email directly, view it on GitHub
https://github.com/lolopinto/ent/issues/1754#issuecomment-1955967014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACS2VN2PMBNNFFABLQTX2Y3YUWGOZAVCNFSM6AAAAABDM5QCUSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJVHE3DOMBRGQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Gonna reopen this as I think this is still worth updating as I ran into this again. Though please let me know if I'm just doing it wrong. Basically just trying to do:
@gqlInputObjectType()
export class CreateObjectInput {
@gqlField({
class: "CreateObjectInput",
type: "[CreateObjectBInput]", <- this type is generated through the schema graphql action codegen
})
objectBInputs: string;
}
which gives `Error: field objectBInputs references CreateObjectBInput which isn't a graphql object"

presumably, you're still using v0.1.x and not on the beta, right?
fixed in main but will need to backport a few things over for v0.1.x


Fixes ResolveAssemblyReference tests on mac

Changes instances of hardcoded paths such as:
c:\MyComponents\1.0, c:\MyLibraries\... etc
to use readonly fields which are constructed to be valid on both
Windows and Unix. Replace all occurrences of these paths, to use the
new fields.
PR-URL:

Hi @radical, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

LGTM

Looks good, thank you!


Specify key to limit on

Hi!
Is it possible for you to look into a way to specify what key to throttle it on?
At the moment i'm using Sidekiq-throttler which does it via:
sidekiq_options throttle: { threshold: 20, period: 1.minute, key: 'tvdb' }

There's a way to create "shared" strategies:
# config/initializers/sidekiq-throttled.rb
Sidekiq::Throttled::Registry.add(:tvdb, {
  :threshold => { :limit => 20, :period => 1.minute }
})

# app/jobs/latest_updates_worker.rb
class LatestUpdatesWorker
  include Sidekiq::Worker
  include Sidekiq::Throttled::Worker

  sidekiq_throttle_as :tvdb

  # ...
end

And ability to be a key lambda from job args

Dynamic configuration (key/limit/period) is not available ATM.
But I'm pretty open to add it.

I have created separate issue ( #4 ) to discuss dynamic options.
I dislike the way they are usually done, so wanna try to find the best approach.
Feel free to express your thoughts! :D

As it's already a way to specific keys. Then I guess my question got answered. Thanks!


Environment Variables

This PR adds

RENTERD_BUS_GATEWAY_ADDR
RENTERD_TRACING_SERVICE_INSTANCE_ID
RENTERD_SEED

I added RENTERD_BUS_GATEWAY_ADDR to make sure it's consistent with the other environment variables. I added RENTERD_TRACING_SERVICE_INSTANCE_ID because we were simply using the worker ID which no longer makes sense now that we have tracing in other components as well.
Most imporantly I renamed the wallet seed to simply RENTERD_SEED because i was configuring a worker instance and was getting error messages indicating my wallet seed was missing. I think that's a bit odd because a worker should not require a wallet seed to be instantiated... Adding another seed or key variable would be bad UX, so I think simply renaming RENTERD_WALLET_SEED to RENTER_SEED might solve it without making too many changes.

@n8maninger I want to change RENTERD_WALLET_SEED to RENTERD_SEED... since this affects the uploaders I wanted to run this by you. PR description mentions why I want to change it.

Looks good to me, I'll update the uploader once this is merged


User persistent volume

Provision each user with a 10GB persistent volume.

Every user gets a 10GB pv on the cluster. 10GB is the default value in the jupyterhub helm chart.
storage:
type: dynamic
extraVolumes: []
extraVolumeMounts: []
static:
pvcName:
subPath: '{username}'
capacity: 10Gi

N.B., need to think about restricting so don't end up with too many unused disks getting created.


Sorry your client is not allowed to use the grant type password

The security mechanism of the API changed today, and thus the library stopped working.
Here is another similar library with the same issue: https://github.com/qlaffont/fortnite-api/issues/192

Rewrite your Auth.php abit https://pastebin.com/PLxTn16y

just tested the api, it doesnt work anymore

I got it working on my fork of this library
https://github.com/TimVerheul/fortnite-php

I got it working on my fork of this library
https://github.com/TimVerheul/fortnite-php

Awesome, thanks a lot. I've merge the changes and it seems to be working properly!

Done, check it out.

Hi,
Still, get this error:
Fatal error: Uncaught Exception: Sorry your client is not allowed to use the grant type password in C:\xampp\htdocs\zzz\vendor\tustin\fortnite-php\src\Auth.php:86
Any help?

@ZeProf2Code Are the account details you provided to login valid?

It looks like Epic changed stuff again, and it stopped working nearly 4 hours ago :(


Proguard setting

Do I need add any proguard setting for this library?

Nope, no need for any ProGuard configuration.

Thanks.

No prob!


Fix version

0.1.4 was created without bumping the version

I would recommend you to wait for these to be merged :

#14
#19

That is my fault, pretty busy these days. I need some time to finish verbum and integrate it to CI, it will do all the work automatically.

This is a config file for a version management tool I developing https://github.com/chocolacula/verbum
This is an example of log it prints during CI workflow run. As you can see it determines which version component to up, change the version in source files, then make a commit and a tag.
Commits to analyse:

Merge pull request #[1](https://github.com/chocolacula/gdmd/actions/runs/12126806292/job/33809687315#step:5:1)4 from ccoveille-forks/refactoring
update the README files
ci: fix actions and update README
Merge pull request #19 from ccoveille-forks/add-badge
documentation: add godoc badge
ci: add golangci-lint and verbum
documentation: add godoc badge
update the README files
Merge pull request #11 from ccoveille-forks/refactoring
chore(code): refactoring
Merge pull request #[9](https://github.com/chocolacula/gdmd/actions/runs/12126806292/job/33809687315#step:5:10) from ccoveille-forks/fix-install
fix(documentation): installation instructions
chore(code): refactoring
Always returns an error, never panics...
fix(documentation): installation instructions
Merge pull request #[10](https://github.com/chocolacula/gdmd/actions/runs/12126806292/job/33809687315#step:5:11) from ccoveille-forks/typos
chore(typo): fix styles
chore(typo): fix styles
fix: make help message better

component to up: patch
version bumped to 0.1.4


gcc9: sprintf and strncat fixes in plugins

Failing builds on Fedora 30.  Used Docker and Fedora 30 on my Mac to make sure it Is building correctly.

Cherry picked to 8.0.x for 8.0.6 release


/block.validate endpoint

#1643
Overview
Creates an endpoint /block.validate that conducts stateless block validation. To be expanded to a stateful validation in future work.
The endpoint takes 3 parameters: transactions, hash and number.
Stateless validation consists of:

Verifying that the parameters are correctly formed.  ✅
Verifying that the block hash corresponds to the Merkle root hash.  ✅
Verifying that the transaction bytes correspond to correctly formed transactions. ✅

Changes

Additions to block_constraints.ex to validate parameters.
New module block_validator.ex to validate submitted block.
Requisite changes in controller and router file

Testing
mix test test/omg_watcher_rpc/web/controllers/block_test.exs
mix test test/omg_watcher/block_validator_test.exs
mix test test/omg_watcher_rpc/web/validators/block_constraints_test.exs
The following negative cases are tested:

Incorrectly formed hash parameter
Incorrectly formed number parameter
Incorrectly formed transactions parameter
Incorrectly formed elements (i.e not hexadecimal transaction bytes) in transactions array.
Invalid Merkle root
Incorrectly formed transactions

Props to @ripzery for the work here!

hey @unnawut thanks, makes sense. have move the block validation logic into OMG.Watcher.BlockValidator and parameter validation logic into existing block_constraints.ex file.


TypeError: 'undefined' is not a function (evaluating 'i(d)')

On Safari 5.1.7, from blank new angular-cli project:
polyfills.ts
...
/**
 * Required to support Web Animations `@angular/animation`.
 * Needed for: All but Chrome, Firefox and Opera. http://caniuse.com/#feat=web-animation
 **/
import 'web-animations-js';  // Run `npm install --save web-animations-js`.
...

See docs/support.md
The polyfill is supported on modern versions of all major browsers, including:

Safari (iOS) 7.1+
Safari (Mac) 9+

In particular, the polyfill requires requestAnimationFrame. If your browser does not have requestAnimationFrame, consider loading a requestAnimationFrame polyfill first (example).

When running ANY .html page in the decompressed file downloaded from github, all I get are blank pages.
Firebug shows all code deployed....no errors...


python3.pkgs.pandas: remove optional dependencies

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Might break some reverse dependencies if they did not specify their dependencies correctly.

tested several reverse dependencies as well and none broke

Bit more rebuilds but it is Python

Why did you throw a mass rebuild with potential breaking changes on master? That would have been a perfect candidate for staging.


Argument Type Error in HttpClient

Hello, I am trying to use this library but I am currently having an argument type error. I noticed in your code under the utils package  that your doGet function is as below:
 @protected
  Future<Response> doGet(String url, {Map<String, dynamic> headers}) {
    return httpClient.get(url, headers: headers);
  }

Shouldn't this be ?
 @protected
  Future<Response> doGet(String url, {Map<String, dynamic> headers}) {
    return httpClient.get(Uri.parse(url), headers: headers); //Uri.parse(url)?
  }

Is there a way to work around this as this is critical to my current project?
Thanks

@lejard-h Cool. That worked. Thanks.


Exposing larger number of ports in range (18000-20000)

Bug description
I would like to report error (500) while I am trying to espose more than 250 ports in a range. I am trying to run Freepbx image which requires 2000 UDP ports (in range 18 000 - 20 000 to be exposed and Portainer can't assign those ports. I am using image tiredofit/freepbx:latest
Expected behavior
Enter port range 10000-20000 with UDP selected. Click on Deploy the container.
Portainer Logs
Provide the logs of your Portainer container or Service.
You can see how here
Steps to reproduce the issue:

Go to container
Press Duplicate/Edit
Press +publish a new network port
enter port range 10000-20000 and select UDP
Click Deploy
The message box with show timeout error 500.

Technical details:

Portainer version: Community Edition
7.16.2
Docker version (managed by Portainer):
Kubernetes version (managed by Portainer):
Platform (windows/linux): Linux
Command used to start Portainer (docker run -p 9443:9443 portainer/portainer):
Browser: Chrome

Additional context
Look like it can't handle any larger range. I managed to make it work with 10000 - 10250  (only 250 ports) and can not pass that number.

@markosole
[1] I am able to create the tiredofit/freepbx:latest container and Duplicate/Edit to add the 10000-20000 UDP port:
https://user-images.githubusercontent.com/20426210/210023291-bb271cd7-7776-4f90-aaf1-686bb2a0c371.mov
[2] For an additional test, I created wordpress with the initial creation of the 10000-20000 UDP port without error:
https://user-images.githubusercontent.com/20426210/210023490-58cdc821-28af-4f31-973e-7d90dece648e.mov
[3] For an additional test, I created nginx container and Duplicate/Edit to add the 10000-20000 UDP port with the 500 error related to the ports being allocated and in use:
https://user-images.githubusercontent.com/20426210/210023767-2baafbfd-4107-4356-a530-12c8774754d5.mov
Can you try the following:
[1] Open the Web Developer Tools
[2] Deploy Container (after adding ports with Duplicate/Edit)
[3] Find the POST 500 error
[4] Select Response for error
[5] Create Container (with the ports as I did in [1])
Thanks!


Add tag(s) for proposals

@bonustrack Tags can be any array of strings? or there are any set of keywords?


Improve the createPatch process (if still in use?) for new Jenkins

Been ages since I've been fishing around in there, but I ended up making a Launcher job in our new Jenkins for :reasons: that really didn't have anything to do with making patches, but that createPatch Gradle task failed due to the URLs changing (and even pointing back to the old Jenkins while using the new temporary name thinking that job existed over there as well)
Direct code link at https://github.com/MovingBlocks/TerasologyLauncher/blob/develop/build.gradle#L414 and new Jenkins link http://cbc.<IP_ADDRESS>.xip.io/teams-terasology/job/Terasology/job/TerasologyLauncherOldStyle/14/console (I need to make new accounts manually for the moment - let me know if anybody is interested)
Snippet in case that job goes away / gets renamed:
> Task :createPatch FAILED

FAILURE: Build failed with an exception.

* Where:
Build file '/home/jenkins/workspace/Terasology/TerasologyLauncherOldStyle/build.gradle' line: 412

* What went wrong:
Execution failed for task ':createPatch'.
> java.io.FileNotFoundException: http://jenkins.terasology.org/job/Terasology/TerasologyLauncherOldStyle/lastSuccessfulBuild/buildNumber

I expect to recreate the new job in a multi-branch pipeline job which will still change the path, even if we eventually go back to using jenkins.terasology.org (and doubly so since the new Jenkins is actually a Cloudbees Core Jenkins I'm playing with for work)

I guess we'll update the patch computation anyways when further working #260. After JRE bundling is finally done we want to investigate google/archive-patcher or something similar as a replacement for the current solution (unmaintained dependency).


add Choice enum

I was under the impression that we couldn't use 0 for one of the choices as it was the default value. Is this not an issue here?

I initially thought this won't be a problem because we don't have a storage var for choice only, but it's better this way anyway just to be safe :)


Support for HUB75

Hey,
I just found this project and it would be a perfect match for cheap LED Panels I have laying around.
Unfortianatly they are HUB75 and not ws2812b.
Woult it be possible to get support for that?
Here is already a libary ready to go: https://github.com/pixelmatix/SmartMatrix/tree/master
My Matrix has 64x64 and an option to group pixels would also zoom in th pixel art and would not show the too small.
Thank you.

I think if your matrix is not 8x32, this is probably the wrong project for you.

Yes sorry @reptil1990 , unfortunately @rozza-m is right, to run this project with a different pixel count everything would have to be reworked.

Would it not be possible to group Pixels? So that a pixel in the original scale is lighting up 4 pixels in a bigger matrix?
That sould be doable?
You are the experts just a thought.

I have a experience with HUB75 Panels and

they needs at least esp32
(needs a lot more pins)
panels are more expensive
a lot of code must be rewritten

For me it make also no sense. Write your own code to display the PixelIt Stuff on such a display. You need a MQTT Client or REST API, a GFX Libary and you could still use NodeRED and the PixIt Flows . Changing the PixelIt code to support HUB75 it make IHMO absolute NO sense.


Fix an attributeerror on elasticsearch objects in some versions of th…

Fixes an error occurring with older versions of elasticsearch library introduced in #1006
self.current_es_addr = (self.current_es.host, self.current_es.port)
AttributeError: 'Elasticsearch' object has no attribute 'host'

These settings are applied to the global config before schema check, which is then used for the default in rules.

Sorry for the bug! thanks!


bug(ui): Dont show markdown style in sidebar

Issue:
Like title says dont show markdown styles in sidebar, so instead of it showing as bold it should show as " * * bold * *"

lol https://github.com/Satellite-im/Uplink/issues/946

we know @sdwoodbury but matt asked to remove it so we added a ticket for it to address that


failed to install via helm

What happened:
Installing kube.fed-v2 vi helm reports an error
helm install fedv2-charts/federation-v2 --version=0.0.9 --namespace federation-system
Error: apiVersion "core.federation.k8s.io/v1alpha1" in federation-v2/templates/federatedtypeconfig.yaml is not available

I add the helm repo for the fed-v2 before
 helm repo add fedv2-charts https://raw.githubusercontent.com/kubernetes-sigs/federation-v2/master/charts

helm repo list
NAME            URL
stable          https://kubernetes-charts.storage.googleapis.com
local           http://<IP_ADDRESS>:8879/charts
svc-cat         https://svc-catalog-charts.storage.googleapis.com
fedv2-charts    https://raw.githubusercontent.com/kubernetes-sigs/federation-v2/master/charts

helm search federation-v2
NAME                            CHART VERSION   APP VERSION     DESCRIPTION
fedv2-charts/federation-v2      0.0.9                           Kubernetes Federation V2 helm chart

What you expected to happen:
the fed-v2 control plane is installed without any errors
Environment:

Kubernetes version (use kubectl version)

Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.1", GitCommit:"eec55b9ba98609a46fee712359c7b5b365bdd920", GitTreeState:"clean", BuildDate:"2018-12-13T19:44:10Z", GoVersion:"go1.11.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.0", GitCommit:"641856db18352033a0d96dbc99153fa3b27298e5", GitTreeState:"clean", BuildDate:"2019-03-25T15:45:25Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}

Federation version: 0.0.9
Install method (helm vs script): helm
Install configuration (namespaced vs cluster-scoped): cluster-scoped
Others

/kind bug

I had the same issue but after removing the CRDs and other things (as explained in deleting federation) and run it a second time it worked.

@freegroup As per @HaraldGustafsson's comment, are you deploying to a clean cluster (where fedv2 has not been deployed previously or fedv2 has been removed as per the instructions)?

IT Is a fresh and Clean kubernetes Cluster.

@freegroup What version of helm are you using?  And have you granted the necessary permissions to the tiller (if rbac is enabled in the host cluster)?  I was not able to reproduce the issue you report (using kube 1.14 helm 2.13) so long as I granted the tiller the necessary permissions (see #817).

Clarify that I also installed to a cluster that not previously had fed V2 installed, the deletion was between the attempts. I did not delete the federation system namespace in between. But it worked the second time. The cluster is on VMs on openstack. Maybe a race condition?

I'm using helm in the version below:
helm version
Client: &version.Version{SemVer:"v2.8.1", GitCommit:"6af75a8fd72e2aa18a2b278cfe5c7a1c5feca7f2", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.8.1", GitCommit:"6af75a8fd72e2aa18a2b278cfe5c7a1c5feca7f2", GitTreeState:"clean"}

we have RBAC enabled in our cluster. Installation of helm is shown below
cat <<EOF | kubectl create -f -
apiVersion: v1
kind: ServiceAccount
metadata:
 name: helm
 namespace: kube-system
---
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRoleBinding
metadata:
 name: helm
roleRef:
 apiGroup: rbac.authorization.k8s.io
 kind: ClusterRole
 name: cluster-admin
subjects:
 - kind: ServiceAccount
   name: helm
   namespace: kube-system
EOF

helm init --service-account helm

The Kubernetes version is 1.14.0 and it is created via https://gardener.cloud/ . Gardener seems to pass the kubernetes conformance test with 100% (https://k8s-testgrid.appspot.com/conformance-gardener#Gardener, v1.14 AWS). I think nothing special or broken setup should be in the gardener cluster.
I created a new fresh cluster and I have the very same problem.

brew upgrade kubernetes-helm is the solution. Sorry for the noise.
 helm version
Client: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}

maybe it is a good idea to mention which are the required HELM version for the installation
Thanks for the support!

@freegroup The helm install instructions already mentions the dependency on helm > 2.10:
https://github.com/kubernetes-sigs/federation-v2/blob/master/charts/federation-v2/README.md#prerequisites

Thank you again for your patience. Sometimes I accidentally overlook the important things. :-/


Swap logos on the front page

Description
Logos to add:

Arduino (Requires approval from their side, please just create a PR at first)
Bosch (now we're a partner cc @MarianRaphael )

Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

cc @iskerrett @Yndira-FlowForge

Is this order ok?

@Yndira-FlowForge Arduino is somewhat small, Leclanche is the first one my eyes saw, so can we swamp Arduino and leclanche?

Arduino is somewhat small, Leclanche is the first one my eyes saw, so can we swamp Arduino and leclanche?
Sure, it would be like this


Develop a mechanism for targeted extended attribute warnings

It may be desirable to specifically call out particular extended attributes, such as [RuntimeEnabled=SomethingOrOther] in Blink called out here.

What I think we should do is to parse RuntimeEnabledFeatures.json5 and include things that are enabled for stable, and exclude everything else. Some way of validating things ahead of shipping would be useful of course.


Columns

Code contains:

info out of column campaign into separate columns
rest dict columns droped
unixstamp into normal date+time type
added a column with the creator-ids

Wowh! That looks awesome, would never have come to my mind to combine a for loop with the extraction 🙈
One thing we discussed yesterday during the column / data dict checkup was, that we might want to keep the information wether or not someone has set a profile picture + a link to the profile because kickstarter recommends this to ensure a successful project.


ws.end() not sending a close message

I want to send a message with some data before closing the connection, I thought this is what ws.end() is for but I'm not seeing a message on the client, the connection is closed properly but the message never gets to the client. This is how I'm currently trying to do this:
ws.end(undefined, JSON.stringify({ some: "data" }))

And as you can see there are no messages, the code above is indeed running because I made sure of it.

https://unetworking.github.io/uWebSockets.js/generated/interfaces/websocket.html#end
Undefined is not a valid close code.

I've just tried sending a code and it didn't work either, I tried most codes documented here: https://developer.mozilla.org/en-US/docs/Web/API/CloseEvent

As a workaround I'm calling ws.send() and then ws.end() which is doing what I want in case this is helpful for anyone else:
ws.send(JSON.stringify({ the: "error" }))
ws.end()

@gmencz you are not checking the reason correctly, its not a standard message its part of the close event, like this:
ws.onclose=e=>console.log(e.code, e.reason)


unclear if possible or how to add meta type info to a custom Aggregation

ValueError: Metadata inference failed in `_groupby_apply_funcs`. received when I tried my hand at custom Aggregation. It seems like it would be reasonable to add meta to the Aggregation object but that errors differently.
def percentile(n):
    from crick import TDigest
    # doc https://github.com/jcrist/crick/blob/master/crick/tdigest.pyx
    # doc https://docs.dask.org/en/stable/dataframe-api.html#dask.dataframe.groupby.Aggregation

    def percentile_chunck(grouped):
        g_res = []
        for items in grouped:
            t = TDigest()
            g_res.append(t.update(items))
        return(g_res)

    def percentile_agg(chunk_TDigests):
        tagg = TDigest()
        for t in chunk_TDigests:
            tagg.merge(t)
        return tagg

    def percentile_finalize(tagg):
        return tagg.quantile(n)

    return dd.Aggregation(
        name    = 'percentile_{:2.0f}'.format(n*100),
        chunk   = percentile_chunck,
        agg     = percentile_agg,
        finalize=percentile_finalize,
    )

Spike_day = Spike_data.groupby('modulus_min')['abs_filtered_count'].agg([
    np.sum,
    np.mean,
    np.std,
#    np.median,
    np.var,
    np.min,
    np.max,
    percentile(0.05),
    percentile(0.25),
    percentile(0.50),
    percentile(0.75),
    percentile(0.95)
])
display(Spike_day.head(10)) # prove the aggs work

If there is a better way to combine percentiles with other aggregations, I would like to hear that as well.  But, I was thinking if I could work through this I could answer several Aggregation related questions on StackOverflow even if there was a better approach.

Can you provide a minimal example? http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports
I'm not sure I fully understand, but add ing SeriesGroupBy / DataFrameGroupBy.quantile method is well within scope for dask.dataframe.

I am not sure I could condense the above code further without ending up at the min/max example in the docs.  It seems like there are a few open questions on how to do this kind of thing so I wanted to try my hand at a custom agg function. The only thing that should have a hard "must be a numeric" requirement is the finalize and it does.
The error I get is the following:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/utils.py in raise_on_meta_error(funcname, udf)
    168     try:
--> 169         yield
    170     except Exception as e:

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/core.py in _emulate(func, *args, **kwargs)
   4710     with raise_on_meta_error(funcname(func), udf=kwargs.pop("udf", False)):
-> 4711         return func(*_extract_meta(args, True), **_extract_meta(kwargs, True))
   4712

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/groupby.py in _groupby_apply_funcs(df, *index, **kwargs)
    838     for result_column, func, func_kwargs in funcs:
--> 839         r = func(grouped, **func_kwargs)
    840

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/groupby.py in _apply_func_to_column(df_like, column, func)
    875
--> 876     return func(df_like[column])
    877

<ipython-input-131-0d0e0460796b> in percentile_chunck(grouped)
     22             t = TDigest()
---> 23             g_res.append(t.update(items))
     24         return(g_res)

crick/tdigest.pyx in crick.tdigest.TDigest.update()

TypeError: x must be numeric

During handling of the above exception, another exception occurred:

ValueError                                Traceback (most recent call last)
<ipython-input-131-0d0e0460796b> in <module>
     52     percentile(0.50),
     53     percentile(0.75),
---> 54     percentile(0.95)
     55 ])
     56 display(Spike_day.head(10)) # prove the aggs work

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/groupby.py in agg(self, arg, split_every, split_out)
   1619     @derived_from(pd.core.groupby.SeriesGroupBy)
   1620     def agg(self, arg, split_every=None, split_out=1):
-> 1621         return self.aggregate(arg, split_every=split_every, split_out=split_out)

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/groupby.py in aggregate(self, arg, split_every, split_out)
   1607     def aggregate(self, arg, split_every=None, split_out=1):
   1608         result = super(SeriesGroupBy, self).aggregate(
-> 1609             arg, split_every=split_every, split_out=split_out
   1610         )
   1611         if self._slice:

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/groupby.py in aggregate(self, arg, split_every, split_out)
   1378             split_every=split_every,
   1379             split_out=split_out,
-> 1380             split_out_setup=split_out_on_index,
   1381         )
   1382

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/core.py in apply_concat_apply(args, chunk, aggregate, combine, meta, token, chunk_kwargs, aggregate_kwargs, combine_kwargs, split_every, split_out, split_out_setup, split_out_setup_kwargs, **kwargs)
   4664
   4665     if meta is no_default:
-> 4666         meta_chunk = _emulate(chunk, *args, udf=True, **chunk_kwargs)
   4667         meta = _emulate(aggregate, _concat([meta_chunk]), udf=True, **aggregate_kwargs)
   4668     meta = make_meta(

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/core.py in _emulate(func, *args, **kwargs)
   4709     """
   4710     with raise_on_meta_error(funcname(func), udf=kwargs.pop("udf", False)):
-> 4711         return func(*_extract_meta(args, True), **_extract_meta(kwargs, True))
   4712
   4713

/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/lib/python3.7/contextlib.py in __exit__(self, type, value, traceback)
    128                 value = type()
    129             try:
--> 130                 self.gen.throw(type, value, traceback)
    131             except StopIteration as exc:
    132                 # Suppress StopIteration *unless* it's the same exception that

~/develop/ConnectTheDots/.venv/lib/python3.7/site-packages/dask/dataframe/utils.py in raise_on_meta_error(funcname, udf)
    188         )
    189         msg = msg.format(" in `{0}`".format(funcname) if funcname else "", repr(e), tb)
--> 190         raise ValueError(msg)
    191
    192

ValueError: Metadata inference failed in `_groupby_apply_funcs`.

You have supplied a custom function and Dask is unable to
determine the type of output that that function returns.

To resolve this please provide a meta= keyword.
The docstring of the Dask function you ran should have more information.

Original error is below:
------------------------
TypeError('x must be numeric')

Ok in an attempt to simplify did the following:
def percentile(n):
    from crick import TDigest
    # doc https://github.com/jcrist/crick/blob/master/crick/tdigest.pyx
    # doc https://docs.dask.org/en/stable/dataframe-api.html#dask.dataframe.groupby.Aggregation

    def percentile_chunck(grouped):
        g_res = []
        for items in grouped:
            g_res.append(0.0)
        return(g_res)

    def percentile_agg(chunk_TDigests):
        return 0.0

    def percentile_finalize(tagg):
        return float('0.0')

    return dd.Aggregation(
        name    = 'percentile_{:2.0f}'.format(n*100),
        chunk   = percentile_chunck,
        agg     = percentile_agg,
        finalize=percentile_finalize,
    )

That worked as a aggragation and produces 0.0 as expected.
I think made the following change:
    def percentile_chunck(grouped):
        g_res = []
        for items in grouped:
            t = TDigest()
            g_res.append(t.update(items))
        return(g_res)

That produces the error.  The documentation says that chunk(grouped): can return tuples and but does not say that each value must be numeric.  My understanding is that these results are going to be passed into the agg function later.  In the case of percintile, we are keeping a line segment in memory and modifying it with each run.

For another example:
from hurry.filesize import size as filesize
human_sum = dd.Aggregation('human_sum', lambda s: s.sum(), lambda s0: s0.sum(), lambda s1: filesize(s1))

The input is int64. The output is object. Or should be.
This is intended to provide a human readable size of a sum of files in my dataset.
Pinging original dev for Aggregation: @chmp

@jhulten @nbmorgan do we have a runnable example here? I glanced through and didn't see anywhere a DataFrame was defined (apologies if I missed it).

Here is a runnable version slightly diffent error.  I am still trying to follow the custom agg rules.
import os
import io
import sys
import time
import datetime

import numpy as np
import pandas as pd
import zlib

import requests
import json
import pprint

import dask
import dask.dataframe as dd
from dask.distributed import Client, LocalCluster

client = Client(
    n_workers=6,
    threads_per_worker=4,
    memory_limit='8GB',
    host='localhost',
    processes=True,
)
client

# In[3]:

ddf = dask.datasets.timeseries()
ddf

# In[4]:

print(len(ddf))
display(ddf.head())
display(ddf.tail())

# In[10]:

def AddDerivedTimeFeature(src_df):
    #res_df = res_df = dd.from_pandas(src_df, 30)
    res_df = src_df
    res_df = res_df.reset_index() #Convert index col back to a normal col so we can manipulate it
    res_df['hour'] = res_df['timestamp'].apply(lambda x: x.hour)
    res_df['minute'] = res_df['timestamp'].apply(lambda x: x.minute)
    res_df['monthday'] = res_df['timestamp'].apply(lambda x: x.day)
    res_df['weekday'] = res_df['timestamp'].apply(lambda x: x.weekday)

    secperhour = 60 * 60
    secperday  = secperhour * 24
    secperweek = secperday * 7
    secperqtr  = int(secperweek * 13.0446428571)

    res_df['tzepochtime'] =  res_df['timestamp'].apply(lambda a: int(time.mktime(a.timetuple())))
    res_df['modulus_day'] = res_df['tzepochtime'].apply(lambda a: a % secperday)

    res_df = res_df.set_index('timestamp')

    return  res_df

# In[11]:

ddf2 = AddDerivedTimeFeature(ddf.compute())

# In[12]:

def percentile(n):
    from crick import TDigest
    # doc https://github.com/jcrist/crick/blob/master/crick/tdigest.pyx
    # doc https://docs.dask.org/en/stable/dataframe-api.html#dask.dataframe.groupby.Aggregation

    def percentile_chunck(grouped):
        g_res = []
        for items in grouped:
            t = TDigest()
            g_res.append(t.update(items))
        return(g_res)

    def percentile_agg(chunk_TDigests):
        tagg = TDigest()
        for t in chunk_TDigests:
            tagg.merge(t)
        return tagg

    def percentile_finalize(tagg):
        return tagg.quantile(n)

    return dd.Aggregation(
        name    = 'percentile_{:2.0f}'.format(n*100),
        chunk   = percentile_chunck,
        agg     = percentile_agg,
        finalize=percentile_finalize,
    )

# In[15]:

Spike_day = ddf2.groupby('modulus_day')['x'].agg([
    np.sum,
    np.mean,
    np.std,
    np.var,
    np.min,
    np.max,
    percentile(0.05),
    percentile(0.25),
    percentile(0.50),
    percentile(0.75),
    percentile(0.95)
])
display(Spike_day.head(10)) # prove the aggs work

@nbmorgan is ddf2 supposed to be a dask or pandas dataframe?
Also can you

remove unused imports? Are requests, zlib, etc used?
Simplify things: Do you need 5 calls to percentile or does 1 do? Do you need the other aggfuncs?
Minimize things: timeseries() makes a large dask DataFrame by default. Does it reproduce with a smaller one? freq=D?
Is AddDerivedTimeFeature necessary? Do we need to group by that, or does grouping by id already in the df work?

The feature request here is just to add the meta parameter to aggregate(). I'm not sure it matters exactly why OP's code is failing.

Completely forgot about this issue, as there never was a reply by OP. As future reference for myself, as I think I already spent some time a while back to figure this out: the error here is that crick is called with a Tuple[str, pd.Series]. The call would fail even outside dask. As it stands the issue has nothing to do with the meta data inference.
Re extending daks: since this ia a question for the maintainers, I have no opionion there. However, I guess an example would be helpful to see whether getting the meta inference is not already possible with the current implementation.


Support R_MIPS_GPREL16 relocations correctly

Discussed in discord.
synopsis: objdiff does not currently read the ri_gp_value field from the .reginfo header for MIPS objects, which is required to calculate R_MIPS_GPREL16 relocations for local symbols.
This PR attempts to correct that. Passing the file to implicit_addend is required to determine whether the symbol being relocated against is local or not.

Looks like you're using nightly for fmt. Hopefully the code style is fixed now.

Thanks a lot for the fix!


feat: port Data.Num.Bitwise

A few notes:

Fixed a repr bug in Data.Num.Basic. After the fix, #eval PosNum.one.bit1.bit0 prints 6.
~ could mean SNum.not or NzsNum.not. I gave a higher priority to SNum.not.

bors r+


Add Web Speech Synthesis API to Firefox status

We will turn it on Firefox 45 Desktop.  So could you add this status?
Bugzilla ... https://bugzilla.mozilla.org/show_bug.cgi?id=1003439
MDN ... https://developer.mozilla.org/en-US/docs/Web/API/Web_Speech_API
Spec .. https://dvcs.w3.org/hg/speech-api/raw-file/tip/webspeechapi.html
Edge status ... https://dev.windows.com/en-us/microsoft-edge/platform/status/webspeechapisynthesis
Chrome status ... shipped (http://www.chromestatus.com/features/4782875580825600)
Safari status ... shipped.

Could you do it? We'd love feedback on the contribution process!
Creating a feature is really simple: https://github.com/mozilla/platatus/blob/master/CONTRIBUTING.md
You've already done most of the work (finding the status of other browsers, the spec and MDN URLs and the bugzilla tracking bug).


update to v0.332

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


reset-counters don't work after customizing heading

Hi, I have a custom chapter heading which interferes with the reset counters and also the figure numbering. Can you please have a a look.
#set par(first-line-indent: 2em, justify: true)

#set text(
  font: "New Computer Modern",
  //font: "Times New Roman",
  size: 13pt
)

// Page settings (numbering, size, header, footer etc)
#set page(
  numbering: "1 / 1",
)

#import "@preview/i-figured:0.2.4"
// make sure you have some heading numbering set
//#set heading(numbering: "1.")
#let reset-counters(
  it,
  level: 1,
  extra-kinds: (),
  equations: true,
  return-orig-heading: true,
) = {  }

// apply the show rules (these can be customized)
#show heading: i-figured.reset-counters
#show figure: i-figured.show-figure
#show math.equation: i-figured.show-equation
#i-figured.outline()

#[
// #show heading.where(level: 1): set heading(supplement: [Chapter])
// Configuring chapter headings

#show heading.where(level: 1): set heading(supplement: [Chapter])
// Configuring chapter headings
#set heading(numbering: "1.1")
#show heading: it =>  block({
  let num-style = it.numbering
    let num = text(weight: "thin", numbering(num-style, ..counter(heading).at(here()))+[ \u{200b}])
    let x-offset = -1 * measure(num).width
    // Draw the subheading name
    pad(left: x-offset, par(hanging-indent: -1 * x-offset, text(size:14pt, fill: purple.lighten(25%), num) + [] +   text(fill: purple, size:14pt, it.body)))
})

#show heading.where(level: 1): it => pagebreak(weak: true, to: "odd") + block({
    set text(font: "New Computer Modern", weight: "black")
    v(3cm)
    block(text(48pt, fill:purple.lighten(25%))[Chapter #counter(heading).display()])
    v(1cm)
    block(text(48pt, fill:purple)[#it.body])
    line(length: 100%, stroke: 2pt + gradient.linear(..color.map.viridis))
    v(1cm)
})

= Related Works
#lorem(10)
#figure([b], caption: [This is another figure.]) <my-second-figure>
$ F_n = floor(1 / sqrt(5) phi.alt^n) $ <my-second-equation>

= Section Title
#lorem(50)
#figure([b], caption: [This is another figure.]) <my-second-figure>
$ F_n = floor(1 / sqrt(5) phi.alt^n) $ <my-second-equation>

] // End all includes here

I encountered the same problem, what I did as a workaround was to add the code from reset-counters to my own heading rule. In my case, adding these lines was sufficient:
if it.level <= 1 {
  for kind in (image, table, raw) {
    counter(figure.where(kind: "i-figured-" + repr(kind))).update(0)
  }
}

Since version 0.2.4 the reset-counters function has a return-orig-heading parameter which can be set to false for this exact purpose. See https://github.com/RubixDev/typst-i-figured/pull/5 for more info


Increase memory limit for mgc

Fixes #538

/lgtm
/approve
Is this worth landing now, or can it wait? cc @maleck13

/lgtm


Backport to older minecraft versions?

Describe the bug
It would be great if this could support older minecraft versions such as 1.19.2, 1.18, and possibly further back
Steps to reproduce

Search for a version below 1.20.1

Logs
No response
Additional context
No response

Backports of Connector are not planned, sorry. It would require backporting all of FFAPI, FFLoader, and also double the maintenance costs. Our focus is on future versions where running Connector will become easier over time.


Fix null input handling in Reduce function

Reduce currently used the BaseVector's addNulls to add nulls at the end
to the localResult. This API is very basic and does not handle the variety
of cases that Expr::addNulls is designed to handle. For eg, if the target
vector is smaller than the input rows, or has a different encoding or is
empty. This change moves the Expr::addNulls() API to EvalCtx to make
it easily accessible to vector functions like Reduce without pulling in the
complete Expr header file and finally employing it to add nulls.
Test Plan:
Added unit tests

cc: @yma11


add classic JeeNode section

To be culled from these sources:

https://jeelabs.net/projects/cafe/wiki
https://jeelabs.net/projects/hardware/wiki
https://jeelabs.org/pub/docs/jeelib/

Also needs to mention the other JN variants: JN Micro & JN USB.

Done, at least the basics.


Uncaught TypeError: Cannot set property 'className' of undefined

[Enter steps to reproduce below:]

...
...

Atom Version: 1.12.7
Electron Version: 1.3.13
System: Mac OS X 10.12.2
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot set property 'className' of undefined
At /Applications/Atom.app/Contents/Resources/app.asar/src/line-numbers-tile-component.js:176

TypeError: Cannot set property 'className' of undefined
    at LineNumbersTileComponent.module.exports.LineNumbersTileComponent.updateLineNumberNode (/Applications/Atom.app/Contents/Resources/app.asar/src/line-numbers-tile-component.js:176:24)
    at LineNumbersTileComponent.module.exports.LineNumbersTileComponent.updateLineNumbers (/Applications/Atom.app/Contents/Resources/app.asar/src/line-numbers-tile-component.js:98:16)
    at LineNumbersTileComponent.module.exports.LineNumbersTileComponent.updateSync (/Applications/Atom.app/Contents/Resources/app.asar/src/line-numbers-tile-component.js:78:19)
    at LineNumberGutterComponent.module.exports.TiledComponent.updateTileNodes (/Applications/Atom.app/Contents/Resources/app.asar/src/tiled-component.js:67:19)
    at LineNumberGutterComponent.module.exports.TiledComponent.updateSync (/Applications/Atom.app/Contents/Resources/app.asar/src/tiled-component.js:30:12)
    at GutterContainerComponent.module.exports.GutterContainerComponent.updateSync (/Applications/Atom.app/Contents/Resources/app.asar/src/gutter-container-component.js:79:25)
    at TextEditorComponent.module.exports.TextEditorComponent.updateSync (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor-component.js:232:39)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/text-editor-component.js:323:28
    at ViewRegistry.module.exports.ViewRegistry.performDocumentUpdate (/Applications/Atom.app/Contents/Resources/app.asar/src/view-registry.js:219:9)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/view-registry.js:3:59

Commands

Config
{
  "core": {
    "disabledPackages": [
      "language-javascript-jsx",
      "react",
      "linter-sass-lint"
    ],
    "telemetryConsent": "no"
  },
  "editor": {
    "fontSize": 11,
    "invisibles": {},
    "preferredLineLength": 100,
    "showInvisibles": true
  }
}

Installed Packages
# User
atom-beautify, v0.29.13 (inactive)
color-picker, v2.2.3 (active)
editorconfig, v2.0.5 (active)
emmet, v2.4.3 (active)
file-icons, v1.7.25 (active)
git-plus, v5.29.1 (active)
highlight-selected, v0.11.2 (active)
language-babel, v2.48.5 (active)
language-docker, v1.1.7 (active)
language-haml, v0.24.1 (active)
language-smarty, v1.7.0 (active)
linter, v1.11.18 (active)
linter-eslint, v8.0.0 (active)
linter-htmlhint, v1.3.1 (active)
linter-sass-lint, v1.7.4 (inactive)
linter-scss-lint, v3.0.4 (active)
minimap, v4.25.7 (active)
pigments, v0.37.0 (active)
project-manager, v3.3.3 (active)
react-snippets, v0.5.0 (active)
svg-preview, v0.10.0 (active)
atom-dark-syntax, v0.27.0 (inactive)
atom-dark-ui, v0.52.0 (inactive)
atom-light-syntax, v0.28.0 (inactive)
atom-light-ui, v0.45.0 (inactive)
base16-tomorrow-dark-theme, v1.3.0 (inactive)
base16-tomorrow-light-theme, v1.3.0 (inactive)
one-dark-ui, v1.6.2 (active)
one-light-ui, v1.6.2 (inactive)
one-dark-syntax, v1.5.0 (active)
one-light-syntax, v1.5.0 (inactive)
solarized-dark-syntax, v1.0.5 (inactive)
solarized-light-syntax, v1.0.5 (inactive)
about, v1.7.0 (active)
archive-view, v0.62.0 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.13.1 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.31.4 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.27.0 (inactive)
autosave, v0.23.1 (active)
background-tips, v0.26.1 (active)
bookmarks, v0.42.0 (active)
bracket-matcher, v0.82.2 (active)
command-palette, v0.39.0 (inactive)
deprecation-cop, v0.54.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.40.0 (active)
find-and-replace, v0.202.2 (inactive)
fuzzy-finder, v1.4.0 (active)
git-diff, v1.1.0 (active)
go-to-line, v0.31.0 (inactive)
grammar-selector, v0.48.2 (active)
image-view, v0.60.0 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.0 (active)
link, v0.31.2 (inactive)
markdown-preview, v0.158.8 (active)
metrics, v1.1.2 (active)
notifications, v0.65.1 (active)
open-on-github, v1.2.1 (inactive)
package-generator, v1.0.1 (inactive)
settings-view, v0.243.1 (active)
snippets, v1.0.4 (active)
spell-check, v0.68.4 (active)
status-bar, v1.4.1 (active)
styleguide, v0.47.2 (active)
symbols-view, v0.113.1 (inactive)
tabs, v0.103.0 (active)
timecop, v0.33.2 (active)
tree-view, v0.210.0 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.35.1 (active)
whitespace, v0.35.0 (active)
wrap-guide, v0.38.2 (active)
language-c, v0.54.0 (active)
language-clojure, v0.22.1 (active)
language-coffee-script, v0.48.0 (active)
language-csharp, v0.13.0 (active)
language-css, v0.40.1 (active)
language-gfm, v0.88.0 (active)
language-git, v0.15.0 (active)
language-go, v0.43.0 (active)
language-html, v0.47.1 (active)
language-hyperlink, v0.16.1 (active)
language-java, v0.24.0 (active)
language-javascript, v0.122.0 (active)
language-json, v0.18.3 (active)
language-less, v0.29.6 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.37.0 (active)
language-php, v0.37.3 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.1 (active)
language-ruby, v0.70.2 (active)
language-ruby-on-rails, v0.25.1 (active)
language-sass, v0.57.0 (active)
language-shellscript, v0.23.0 (active)
language-source, v0.9.0 (active)
language-sql, v0.25.0 (active)
language-text, v0.7.1 (active)
language-todo, v0.29.1 (active)
language-toml, v0.18.1 (active)
language-xml, v0.34.12 (active)
language-yaml, v0.27.1 (active)

# Dev
No dev packages

Hi @dontuse, what were you doing when this occurred?

@dontuse Any other information you can share with us?

Thanks for taking the time to contribute!
We haven't gotten a response to our questions in our comment https://github.com/atom/atom/issues/13464#issuecomment-267772117. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.

Hi there
I am getting same error.
Dossier.js:710 Uncaught TypeError: Cannot set property 'className' of undefined
at Dossier.checkRequire (Dossier.js:710)
at Dossier.checkMoi (Dossier.js:1254)
at Object.ReactErrorUtils.invokeGuardedCallback (ReactErrorUtils.js:69)
at executeDispatch (EventPluginUtils.js:85)
at Object.executeDispatchesInOrder (EventPluginUtils.js:108)
at executeDispatchesAndRelease (EventPluginHub.js:43)
at executeDispatchesAndReleaseTopLevel (EventPluginHub.js:54)
at Array.forEach ()
at forEachAccumulated (forEachAccumulated.js:24)
at Object.processEventQueue (EventPluginHub.js:254)
Could you help me?
Thanks


Why not use cabal to build and link the C-wrapper

Cabal is perfectly capable of building C/C++ files and can link them with the fltkhs library. Perhaps the custom Setup.hs isn't needed then.

The original goal was to release the c-bindings seperately. If you look in the c-examples directory, for instance, there are a number of pure C demos. It might be worth revisiting, let me think about it. Is this so users don't need to install autotools?

You can always have a separate build for the C-wrapper if you wanted to create a library, but for a haskell library, it is much easier to have it directly link the object files. Now the autoconf is looking for C compilers, but GHC knows those.

That's a great point. I think it's probably time to do that. I'm working on other things at the moment and since we already have a decent install story I'm inclined to put it on the back burner. Feel free send a PR if you need it immediately.
BTW I don't know if you noticed but I released a new version where the user can override the event handlers and drawing functions with Haskell functions of almost all the widgets. Before it was only a few.


Tfdbg does not work with Coordinator/QueueRunners

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Mint 18
TensorFlow installed from (source or binary): Binary (pip)
TensorFlow version (use command below): v1.2.0-rc2-21-g12f033d 1.2.0
Bazel version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: N/A

Describe the problem
The Tensorflow debugger does not seem to be working with Queues; data never seems to be fetched by the QueueRunner threads, be it from a file (using tf.TFRecordReader and tf.parse_single_example) or preloaded (using tf.train.slice_input_producer). Instead, the coordinator.should_stop() is True right away. This is only the case after wrapping the session in a tf.python.debug.LocalCLIDebugWrapperSession. The example should make things clearer.
Moreover, another error occurs at coordinator.join(threads).
I am aware of the FAQ entry on Threads, but that does not explain why the data fetching threads would not be working.
Source code / logs
To make it easiest to replicate, I simply took the example on working with preloaded data, and wrapped the session in there with the debugger. I uploaded the gist with two lines added to https://gist.github.com/rubenvereecken/079cdf1abc76866714ff6f752167481d#file-fully_connected_preloaded_debug-py-L92.
To reproduce, run the file. Once you drop in the debugger, run once. It then exits. The full output is below:
Extracting /tmp/data/train-images-idx3-ubyte.gz
Extracting /tmp/data/train-labels-idx1-ubyte.gz
Extracting /tmp/data/t10k-images-idx3-ubyte.gz
Extracting /tmp/data/t10k-labels-idx1-ubyte.gz
Traceback (most recent call last):
  File "ex.py", line 191, in <module>
    tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
  File "/home/ruben/anaconda3/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "ex.py", line 143, in main
    run_training()
  File "ex.py", line 138, in run_training
    coord.join(threads)
  File "/home/ruben/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/coordinator.py", line 389, in join
    six.reraise(*self._exc_info_to_raise)
  File "/home/ruben/anaconda3/lib/python3.6/site-packages/six.py", line 686, in reraise
    raise value
  File "/home/ruben/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/queue_runner_impl.py", line 233, in _run
    enqueue_callable = sess.make_callable(enqueue_op)
AttributeError: 'LocalCLIDebugWrapperSession' object has no attribute 'make_callable'

The stacktrace is about coord.join(threads), but this is only possible because coord.should_stop() never seems to be False, which would indicate there is data to load. Without the added debugger lines, the example simply works.

cc @mrry
@rubenvereecken Thanks for reporting this issue. We are aware of it and will push a fix to it soon.

@caisq thank you so much, I look forward to it.

@rubenvereecken While you wait for the fix, I want to ask you whether you are trying to debug the data input queues, or the training operation on the main thread. If the latter, there is a workaround for that.

@caisq Ah actually the former, but I'd be working my way towards the latter. Is there a way to debug these training ops while still using data fed from queues? Either way, could you point me at the workaround? Much appreciated!

@rubenvereecken The workaround is based on the assumption that the train op runs on the Python main thread, while the data queues run on the child threads, which should usually be the case.
You can just use the thread_name_filter kwarg of the wrapper's constructor to limit the debugging to the train op.
sess = tf_debug.LocalCLIDebugWrapperSession(sess, thread_name_filter="MainThread$")

This is talked about in the FAQ. Doing this doesn't change the source of the input data. They still come from the queues; they just don't break into the TFDBG UI when they run.

Oops. I may have given incomplete suggestion. In your code at https://gist.github.com/rubenvereecken/079cdf1abc76866714ff6f752167481d#file-fully_connected_preloaded_debug-py-L92
make sure that your wrapped Session object is used only to run the train_op. You can do something like this:
Move the line sess = tf_debug.LocalCLIDebugWrapperSession(sess) after line 100. That makes sure that when the data input queue ops are created, the make_call() method of the original session, not that of the wrapped session is called. The wrapped session doesn't have a make_call() method, which was recently added. This was the root cause of the issue you are seeing.

:tada:


[Backport release-22.05] redis: 7.0.0 -> 7.0.4

Description of changes
Fixes CVE-2022-31144
Backport of:

https://github.com/NixOS/nixpkgs/pull/182044
https://github.com/NixOS/nixpkgs/pull/181434
https://github.com/NixOS/nixpkgs/pull/177443

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 182060 run on x86_64-linux 1

2 packages marked as broken and skipped:

python310Packages.pyres
python39Packages.pyres

9 packages built:

discourse
discourseAllPlugins
netbox
python310Packages.django-redis
python310Packages.python-redis-lock
python39Packages.django-redis
python39Packages.python-redis-lock
redis
suricata


cmd/go: list command fails when directory is the target of a module replace directive

What version of Go are you using (go version)?

$ go version
go version go1.12beta1 darwin/amd64

Does this issue reproduce with the latest release?
Yes the issue is reproducible with the latest version of go (go1.12beta1)
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/subhash_sharma/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/subhash_sharma/go"
GOPROXY=""
GORACE=""
GOROOT="/Users/subhash_sharma/.goversions/go1.12beta1"
GOTMPDIR=""
GOTOOLDIR="/Users/subhash_sharma/.goversions/go1.12beta1/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/subhash_sharma/subash/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/w0/d6l19r7j3dbg7fq56_nn4l340000gp/T/go-build083506298=/tmp/go-build -gno-record-
gcc-switches -fno-common"

What did you do?
The following testscript reproduces the issue:

# listbug.txt

# prime the mod cache with a replaced module
go mod download<EMAIL_ADDRESS>go mod download

go list<EMAIL_ADDRESS>! go list<EMAIL_ADDRESS>
-- go.mod --
module mod

require example.com/fruit v1.0.0

replace example.com/fruit => example.com/replacedfruit v1.0.0

-- main.go --
package main

import (
    "fmt"
)

func main() {
    fmt.Println(fruit.Apple)
}

# mod/example.com_fruit_v1.0.0.txt

-- .mod --
module example.com/fruit

-- .info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}

-- go.mod --
module example.com/fruit

-- fruit.go --
package fruit

const Apple = "apple"

# mod/example.com_replacedfruit_v1.0.0.txt

-- .mod --
module example.com/fruit

-- .info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}

-- go.mod --
module example.com/fruit

-- fruit.go --
package fruit

const Apple = "apple"

What did you see?
go list<EMAIL_ADDRESS>passes with output
example.com/fruit

go list<EMAIL_ADDRESS>fails with output
 go: directory<EMAIL_ADDRESS>outside available modules

What did you expect to see?
go list<EMAIL_ADDRESS>should pass because that is the directory in which the replaced example.com/fruit is now located
go list<EMAIL_ADDRESS>should fail because that module is now located elsewhere due to the aforementioned replace in go.mod.
cc @bcmills @myitcv

I couldn't get the original test script to work, but I believe this one reproduces the same problem:
go mod download

! go list<EMAIL_ADDRESS>go list<EMAIL_ADDRESS>
-- go.mod --
module mod

require example.com/fruit v1.0.0

replace example.com/fruit => example.com/replacedfruit v1.0.0

-- main.go --
package main

import (
    "fmt"
    "example.com/fruit"
)

func main() {
    fmt.Println(fruit.Apple)
}

-- .gomodproxy/example.com_fruit_v1.0.0/.mod --
module example.com/fruit

-- .gomodproxy/example.com_fruit_v1.0.0/.info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}

-- .gomodproxy/example.com_fruit_v1.0.0/fruit.go --
package fruit

const Apple = "Apple"

-- .gomodproxy/example.com_replacedfruit_v1.0.0/.mod --
module example.com/replacedfruit

-- .gomodproxy/example.com_replacedfruit_v1.0.0/.info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}

-- .gomodproxy/example.com_replacedfruit_v1.0.0/fruit.go --
package fruit

const Apple = "Apple"

pathInModuleCache in internal/modload/load.go is the culprit. It needs to be made aware of replacements.


bumping sharp version to 0.28.3 to support arm64 on macOS

Fixes #150 by upgrading sharp to a version that support arm64 on macOS

This also fixes #149

But it looks that sharp api changed a lot, this PR needs more changes. It would be cool to write more tests to be sure gulp-responsive is compatible with sharp api.

That makes sense but I'm not enough into this project at the moment to write those tests.
I'm currently using it to create webp, png, and jpeg images in various sizes with renaming, etc. and it's so far without any errors though I understand that's not the same as test coverage ;)

i have problems with node14. can we please push forward the upgrade to 0.28.3?
cheers

i also want it to use with node >= 14


Corrects marshaling of source ip and port

Found this while trying to use pqsi and pqsp.

[approve ci]

Cherry-picked to 7.1.x


fix example bug of sampled_softmax_with_cross_entropy in cn_doc

fix example bug of sampled_softmax_with_cross_entropy in cn_doc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.root seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Deploy to heroku failed (node failure)

Do you have a log from the deploy. Otherwise it’s going to be tricky to help you.

all fixed! my mistake. sorry for bothering