correct medium-and-up and large-and-up media queries

I might be mistaken here, but it seems like the media query breakpoints $large-and-up and $medium-and-up are using the wrong breakpoints.
They're currently defined in _variables.scss as:
$medium-and-up: "only screen and (min-width : #{$small-screen-up})" !default;
$large-and-up: "only screen and (min-width : #{$medium-screen-up})" !default;

but I believe they should be:
$medium-and-up: "only screen and (min-width : #{$medium-screen-up})" !default;
$large-and-up: "only screen and (min-width : #{$large-screen-up})" !default;

It is right as is, your code would make medium and up non-inclusive of itself

Sorry, I had misinterpreted the breakpoints as minimums, not maximums (e.g., medium = 601-992, not 992-1200). In this case, would it make sense to add another media query variable for extra-large screens:
$extra-large-and-up: "only screen and (min-width : #{$large-screen-up})" !default;

since currently there is no media query defined that uses the $large-screen-up breakpoint? The main use case is limiting the size of things on the screen so that they don't get super wide and short. Thanks!


[Docs][Question] Use latest Gemini Model Version through VertexAI Plugin

Hi team, I'm curious if there's example and documentation around how to use the latest version of Gemini models (e.g. gemini-1.5-pro-002) through VertexAI Plugin. When I define the model name in .prompt file, generate function always default to use 001 version.
"---
model: vertexai/gemini-1.5-pro
config:
temperature: 0.3
---"
Thank you!

After upgrading to genkit 0.9.1, we resolved this by noticing a version config was added. We successfully set the version by doing the following:

model: vertexai/gemini-1.5-pro
config:
version: gemini-1.5-pro-002
temperature: 0.1
...


Copy - JPB Skills selection still have placeholder text

Description
There's still placeholder text under each skill category - I suggest removing the text until the category description is ready.

Changes
Enter the approved copy here:

English
French

Content
Content

[ ] Copy provided in English and French.

For now, I'm adding the text from this doc, but we still require translations.

@rositak For now, I removed the text like you originally suggested. Can you confirm that this file has the copy you want? We'll handle the translation, if you're happy with the text.

@tristan-orourke i've made some changes to the text but i think we can use this copy for now - there'll be changes once the recommendation feature is available


Nintendo eShop: The method or operation is not implemented

The bcmdl file fails to open, attached the file and log.
log.txt
DownLoad.bcmdl

Will open now. https://github.com/MapStudioProject/CTR-Studio/commit/680b1b4f91b35f8d24e4bb6382e5ee7048fc2f60 . Will however not save properly atm, scene data is not loaded/saved yet.

Fixed https://github.com/MapStudioProject/CTR-Studio/commit/57cf3c4df71fd976881d393359d666a02f281c3a


Hi.

Hi, I have a question about Booksim2.0.
Is there a way to determine source node and destination node?
For example, starting from (x1,y1) and arriving at (x2,y2).
If not, which file should be referenced and modified?
Thanks

got solution ??

i have bit idea of it and i.e i have written script to list the of src, intermediate node, dest, packet no and clock cycle in entire simulation.

i have bit idea of it and i.e i have written script to list the of src, intermediate node, dest, packet no and clock cycle in entire simulation.

Hi, how did you write the script? Could you please share the solution?
Thank you soooo much!!!@bheemhh

i have bit idea of it and i.e i have written script to list the of src, intermediate node, dest, packet no and clock cycle in entire simulation.

Hello, @bheemhh  can you please share the script or your approach towards finding the source and destination nodes ?
Hoping for a positive response.  Thanks in advance !!!

i have bit idea of it and i.e i have written script to list the of src, intermediate node, dest, packet no and clock cycle in entire simulation.

Hi, how did you write the script? Could you please share the solution? Thank you soooo much!!!@bheemhh

Hii @diziww have you found any solution? Please let me know.

i have bit idea of it and i.e i have written script to list the of src, intermediate node, dest, packet no and clock cycle in entire simulation.

Hi, can you please share the script? I can pay for that. Thanks in advance!!


Link role in accessibility tree lacks value attribute

I am a screen reader user and developer who values the accessibility tree JSON that can be obtained through the Puppeteer API.  It seems like the "link" role has a name but not value populated.  I think the value should contain the href of the link, but it seems to be empty.

@jamalmazrui given this script example:
const puppeteer = require('puppeteer');

(async () => {
  const browser = await puppeteer.launch();
  const page = await browser.newPage();
  await page.goto('https://example.com');
  const snapshot = await page.accessibility.snapshot();
  console.log(snapshot);

  await browser.close();
})();

We have 1 link that as you mentioned correctly only has a role and name property. The accessibility information from Accessibility.getFullAXTree are as follows:
    {
        "nodeId": "18",
        "ignored": false,
        "role": {
            "type": "role",
            "value": "link"
        },
        "name": {
            "type": "computedString",
            "value": "More information...",
            "sources": [
                {
                    "type": "relatedElement",
                    "attribute": "aria-labelledby"
                },
                {
                    "type": "attribute",
                    "attribute": "aria-label"
                },
                {
                    "type": "contents",
                    "value": {
                        "type": "computedString",
                        "value": "More information..."
                    }
                },
                {
                    "type": "attribute",
                    "attribute": "title",
                    "superseded": true
                }
            ]
        },
        "properties": [
            {
                "name": "focusable",
                "value": {
                    "type": "booleanOrUndefined",
                    "value": true
                }
            }
        ],
        "childIds": [
            "19"
        ],
        "backendDOMNodeId": 12
    }

What would you suggest the page.accessibility.snapshot() should return for such a link?

With multiple browsers, I get a security warning when attempting to open
the URL https://example.com, so I'm reluctant to examine it directly.  I
should also mention that I have been using the "interestingly-only"
option rather than getting the full accessibility tree, in case that
makes a difference.
In general, I would expect the role of "link" in an accessibility tree
to have the the clickable text as the "name" attribute and the
underlying URL as the "value" attribute.  For example, this is how the
NVDA screen reader (free, open source on Windows) represents a link in
its accessibility tree.
n 8/6/2020 7:49 AM, Christian Bromann wrote:

@jamalmazrui https://github.com/jamalmazrui given this script example:
const  puppeteer  =  require('puppeteer');
(async  ()  =>  {
const  browser  =  await  puppeteer.launch();
const  page  =  await  browser.newPage();
await  page.goto('https://example.com');
const  snapshot  =  await  page.accessibility.snapshot();
console.log(snapshot);
await  browser.close();
})();
We have 1 link that as you mentioned correctly only has a |role| and
|name| property. The accessibility information from
|Accessibility.getFullAXTree| are as follows:
 {
     "nodeId":"18",
     "ignored":false,
     "role": {
         "type":"role",
         "value":"link"
     },
     "name": {
         "type":"computedString",
         "value":"More information...",
         "sources": [
             {
                 "type":"relatedElement",
                 "attribute":"aria-labelledby"
             },
             {
                 "type":"attribute",
                 "attribute":"aria-label"
             },
             {
                 "type":"contents",
                 "value": {
                     "type":"computedString",
                     "value":"More information..."
                 }
             },
             {
                 "type":"attribute",
                 "attribute":"title",
                 "superseded":true
             }
         ]
     },
     "properties": [
         {
             "name":"focusable",
             "value": {
                 "type":"booleanOrUndefined",
                 "value":true
             }
         }
     ],
     "childIds": [
         "19"
     ],
     "backendDOMNodeId":12
 }

What would you suggest the |page.accessibility.snapshot()| should
return for such a link?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/puppeteer/puppeteer/issues/6311#issuecomment-669975182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACNKGAYTTZMBJ7BIRPKFUDR7K7GBANCNFSM4PWCEUWQ.

so I'm reluctant to examine it directly

Interesting, I don't get this warning. This is how the DOM looks like:

So if I understand you correctly you suggest that the accessibility tree item would look as follows:
{
    "role": "link",
    "name": "More information..."
    "value": "https://www.iana.org/domains/example"
}

If this is ok for the Puppeteer team, I would be happy to raise a PR.

Yes, I support that proposal, thanks.
On 8/6/2020 9:55 AM, Christian Bromann wrote:

so I'm reluctant to examine it directly

Interesting, I don't get this warning. This is how the DOM looks like:
Screenshot 2020-08-06 at 18 52 54
https://user-images.githubusercontent.com/731337/89559581-25022880-d816-11ea-9289-e3c232deb170.png
So if I understand you correctly you suggest that the accessibility
tree item would look as follows:
{
"role":"link",
"name":"More information...",
"value":"https://www.iana.org/domains/example"
}
If this is ok for the Puppeteer team, I would be happy to raise a PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/puppeteer/puppeteer/issues/6311#issuecomment-670050651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACNKGDEYWKZQASTQMLO4ALR7LOAJANCNFSM4PWCEUWQ.

Not sure if this is related but I had to activate accessibility first to have the full accessibility resolved by accessibility.snapshot():
const page = await browser.newPage();
await page._client().send('Accessibility.enable');

Just to chime in since this issue is old but important.
The accessibility tree item for a link still doesn't contain its URL value:
await page.accessibility.snapshot({ interestingOnly: false });

..it might return a link as..
{
    "role": "link",
    "name": "More information...",
}

..when it should also include the link URL as value for completeness and usability..
{
    "role": "link",
    "name": "More information...",
    "value": "https://www.iana.org/domains/example"
}

Also encountering this problem w/ checkbox and input names not populating, e.g.
  {
    nodeId: '1245',
    ignored: false,
    role: { type: 'role', value: 'checkbox' },
    chromeRole: { type: 'internalRole', value: 14 },
    name: { type: 'computedString', value: '', sources: [Array] },
    properties: [ [Object], [Object], [Object] ],
    parentId: '1242',
    childIds: [],
    backendDOMNodeId: 1424
  }

could the core of the problem be that Accessibility.enable returns too quickly w/out fully completing?
an easy repro is looking at the checkboxes on booking.com

So the link value are not being sent by Chrome. I see that there are only two situations One covers range values and the other gets value for controls. And it looks like the link is not considered a control.

Hi Jamal! Having the value attribute filled with the URL is a legacy windows MSAA thing, we do that in the downstream windows layer, similar to how the MSAA value of a document is the url. The value in our cross-platform Blink layer only represents user input in a form control. However, there is a more complete answer to this question, since it seems reasonable to want to get the url field for a link, which Blink puts in a property called url.
I don't know much about Puppeteer, but it sounds like it uses the CDP API here: https://chromedevtools.github.io/devtools-protocol/tot/Accessibility/#method-getFullAXTree
Unfortunately, CDP does not seem to map all of the useful properties provided by Blink in an AXNode (property fields are listed at ax_enums.mojom in the Chromium source tree), and does not map the url property afaict.

I have added the URL property in https://chromium-review.googlesource.com/c/chromium/src/+/5631015 Once it is in stable, we can update Puppeteer and add a test.


I tried to get the mime headers passing an email header
	tr := textproto.NewReader(bufio.NewReader(bytes.NewReader(b)))
	headers, err := tr.ReadMIMEHeader()

What did you see happen?
I got this error:
malformed MIME header line: X-Hdr-a/b
What did you expect to see?
I expect to see this mime header along with the others in the headers object.
The issue happens because the function validHeaderFieldByte is missing the / character in the list of valid ones
https://datatracker.ietf.org/doc/html/rfc2822#section-3.6.8
optional-field  =       field-name ":" unstructured CRLF

field-name      =       1*ftext

ftext           =       %d33-57 /               ; Any character except
                        %d59-126                ;  controls, SP, and
                                                ;  ":".

The validation was already missing the character in version 1.19, but the canonicalMIMEHeaderKey function was not returning the boolean value to indicate as a !ok to the readMIMEHeader function.
// validHeaderFieldByte reports whether b is a valid byte in a header
// field name. RFC 7230 says:
//
//	header-field   = field-name ":" OWS field-value OWS
//	field-name     = token
//	tchar = "!" / "#" / "$" / "%" / "&" / "'" / "*" / "+" / "-" / "." /
//	        "^" / "_" / "`" / "|" / "~" / DIGIT / ALPHA
//	token = 1*tchar
func validHeaderFieldByte(b byte) bool {
	return int(b) < len(isTokenTable) && isTokenTable[b]
}

// canonicalMIMEHeaderKey is like CanonicalMIMEHeaderKey but is
// allowed to mutate the provided byte slice before returning the
// string.
//
// For invalid inputs (if a contains spaces or non-token bytes), a
// is unchanged and a string copy is returned.
func canonicalMIMEHeaderKey(a []byte) string {
	// See if a looks like a header key. If not, return it unchanged.
	for _, c := range a {
		if validHeaderFieldByte(c) {
			continue
		}
		// Don't canonicalize.
		return string(a)
	}
  ...
}

Related Issues and Documentation

net/mail: characters allowed in RFC 5322 are invalid while parsing email header #58862 (closed)
net/textproto: can write binary header it can't read #60674
net/textproto: fails when headers contain invalid headers #53855 (closed)
net/textproto: ability to skip conversion for canonical format #29965 (closed)
net/textproto: missing/incorrect arg in error string. #43408
net/textproto: mime header Get expects canonical header form #6252 (closed)
net/textproto: CanonicalMIMEHeaderKey panic on strings with spaces #6712 (closed)
net/textproto: NewConn fails to initialize commonHeader prior to use #46363 (closed)
net/textproto: missing argument to printf formatting in error string #37467 (closed)
mime: WordDecoder#DecodeHeader accept space as a part of 'encoded-word' #19417 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Seeing as the textproto package docs state it should work for the likes of SMTP, it seems like headers from those relevant RFCs should be supported.

Package textproto implements generic support for text-based request/response protocols in the style of HTTP, NNTP, and SMTP.

For #53188 we restricted the set of permitted characters in the textproto package to conform to RFC 7230.  We aren't going to go back on that.  For the specific case of mail messages we added CL 504416 to permit more characters for mail messages, as part of #58862 and #60332.
I don't think there is anything to do here.  Sorry.

The docs on textproto state it's intended for use in SMTP. If that's not the case, should those docs be updated to reflect that?


Huggingface models do not work

I wish to use other faster models than the default whisper large v3 turbo.
Ive triad downloading 50+ huggingsface models, but the app crashes or fails to download.
is there a way we can either download with our huggingface token or bypass huggingface security restriction?
maybe downloading manually and loading manually into the app?
alternative, is there a way to add groq distil whisper api to the http section instead of openai api?
many questions, would love any feedback (i am not a coder, so please be gentle)
p.s. another question is there a way to connect to the app via local http similar to whisper-asr-server?
p.p.s i love the folder watch feature, and thats why i use the app, but my computer is to slow to get any optimal workflow using the default whisper large v3 turbo models.
Best regards

@MartinMayday Let's see where the problem may be.
Can you give an example of some model you tried and that failed? Or try this Latvian model it definitely works  RaivisDejus/whisper-tiny-lv. Also huggingface models will only work with Huggingface whisper type, so select that in the whisper type selection box.
Most models are open and do not require any tokens. If you need to specify the token see this https://huggingface.co/docs/huggingface_hub/quick-start  as Buzz uses the huggingface_hub under the hood to download models. The huggingface-cli login may be what you need or set the HF_TOKEN environment variable to pass the token to the download scripts.
Downloading models manually is a bit tricky due to the necessary internal structure of the download scripts, but if you get errors downloading maybe something is broken in the caches. See this https://chidiwilliams.github.io/buzz/docs/faq#7-can-i-use-buzz-on-a-computer-without-internet to find the cache folder and maybe delete the old caches.
Buzz supports all OpenAI compatible APIs, also the Groq. See this discussion on more information on how to configure the Groq https://github.com/chidiwilliams/buzz/discussions/827 This will also let you connect Buzz to any local server that supports OpenAI API format.
If you have more questions or something is still confusing, let me know, we will figure this out. Even if all works I would love to hear on sections that are not easy to understand, maybe we can find some areas of the documentation to improve.

I just want to have access to a local (fast) model via http
or alternative through the buzz app with folder watch.
Hardware:
Intel 6 core 32GB ram
Radeon RX Vega 56 vram 8GB
Test results (used 84 min audio):
distil-whisper/distil-large-v3 = 42 min
whisper large-v3-turbo = 66 min
RaivisDejus/whisper-tiny-lv = 51 min
Models triad to get working (FAILED):
nyrahealth/CrisperWhisper (Gated repos - status: ACCEPTED)
Systran/faster-whisper-large-v3
deepdml/whisper-large-v3-turbo
deepdml/faster-whisper-large-v3-turbo-ct2
Brainiac77/whisper-base-bn-s2t
pedrogarcias/whisper-small-v2-jax
distil-whisper/distil-large-v3-ct2
what is working:
RaivisDejus/whisper-tiny-lv
distil-whisper/distil-large-v3
Other thing triad (FAILED):
https://github.com/SYSTRAN/faster-whisper
https://github.com/sanchit-gandhi/whisper-jax.git
https://github.com/Vaibhavs10/insanely-fast-whisper.git
https://github.com/shashikg/WhisperS2T.git
https://github.com/m-bain/whisperX.git
i am sure if i knew how to use python and docker i could figure it out better.
Other things triad (working):
https://github.com/ahmetoner/whisper-asr-webservice.git
Thanks in advance
p.s. i will look if i can get the cli working and try to clear cashe
Sent with Spark
On 26 Dec 2024 at 11.28 +0100, Raivis Dejus @.***>, wrote:

@MartinMayday Let's see where the problem may be.
Can you give an example of some model you tried and that failed? Or try this Latvian model it definitely works RaivisDejus/whisper-tiny-lv. Also huggingface models will only work with Huggingface whisper type, so select that in the whisper type selection box.
Most models are open and do not require any tokens. If you need to specify the token see this https://huggingface.co/docs/huggingface_hub/quick-start as Buzz uses the huggingface_hub under the hood to download models. The huggingface-cli login may be what you need or set the HF_TOKEN environment variable to pass the token to the download scripts.
Downloading models manually is a bit tricky due to the necessary internal structure of the download scripts, but if you get errors downloading maybe something is broken in the caches. See this https://chidiwilliams.github.io/buzz/docs/faq#7-can-i-use-buzz-on-a-computer-without-internet to find the cache folder and maybe delete the old caches.
Buzz supports all OpenAI compatible APIs, also the Groq. See this discussion on more information on how to configure the Groq #827 This will also let you connect Buzz to any local server that supports OpenAI API format.
If you have more questions or something is still confusing, let me know, we will figure this out. Even if all works I would love to hear on sections that are not easy to understand, maybe we can find some areas of the documentation to improve.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I really had high hopes getting the ChrisperWhisper working after seeing it on the huggingface transcribe leaderboard. but its gated and without any option to add username and token i cant figure out how to workaround this.
would be awesome if i just could to something like @.***/CrisperWhisper"
attaching image to illustrate:
Sent with Spark
On 26 Dec 2024 at 11.28 +0100, Raivis Dejus @.***>, wrote:

@MartinMayday Let's see where the problem may be.
Can you give an example of some model you tried and that failed? Or try this Latvian model it definitely works RaivisDejus/whisper-tiny-lv. Also huggingface models will only work with Huggingface whisper type, so select that in the whisper type selection box.
Most models are open and do not require any tokens. If you need to specify the token see this https://huggingface.co/docs/huggingface_hub/quick-start as Buzz uses the huggingface_hub under the hood to download models. The huggingface-cli login may be what you need or set the HF_TOKEN environment variable to pass the token to the download scripts.
Downloading models manually is a bit tricky due to the necessary internal structure of the download scripts, but if you get errors downloading maybe something is broken in the caches. See this https://chidiwilliams.github.io/buzz/docs/faq#7-can-i-use-buzz-on-a-computer-without-internet to find the cache folder and maybe delete the old caches.
Buzz supports all OpenAI compatible APIs, also the Groq. See this discussion on more information on how to configure the Groq #827 This will also let you connect Buzz to any local server that supports OpenAI API format.
If you have more questions or something is still confusing, let me know, we will figure this out. Even if all works I would love to hear on sections that are not easy to understand, maybe we can find some areas of the documentation to improve.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

CrisperWhisper looks really interesting. Unfortunately I do not see an easy or quick way to integrate it into the Buzz. It uses forked version of the transformers library, that may or may not be compatible with all the other models in the Buzz. Also the license of the CrisperWhisper restricts it to non-commercial uses. Easiest way to integrate would be to look for some API library that can use CrisperWhisper and then use that to integrate with the Buzz.
Note on other whisper types.

faster-whisper this works if you select Faster whisper as whisper type. Default model sizes are configured in the Buzz. To use custom Ctranslate2 compatible mode you can download it in Help -> Preferences -> Models and then select Faster whisper and  then paste the Huggingface id of a custom model.
whisper-jax this is not supported by the Buzz, but my tests show that it is not super much faster than the Faster whisper. I think the big speedup of jax approach comes on the specialized TPU hardware that most regular people do not have.
insanely-fast-whisper this is also not supported by the Buzz, but from their github repo it seems this whisper type also focuses and gets it's main speedup on powerful data center GPU setups. If someone does some testing on more regular consumer hardware and it shows this whisper type gets better speed, I would be happy to add it to the Buzz.
WhisperS2T and whisperX seem to use other whisper types under the hood, so they do similar things to the Buzz but with a command line interface. Speaker diarization is something I would like to see in some future Buzz version. Both of these are not currently supported.

Support for the Turbo models is built in the Buzz, you can select them in the dropdown for Whisper and Faster whisper.


Updates based on<EMAIL_ADDRESS>

Codecov Report

Merging #107 (678a273) into main (489f48a) will decrease coverage by 13.57%.
The diff coverage is 46.66%.

@@             Coverage Diff             @@
##             main     #107       +/-   ##
===========================================
- Coverage   89.53%   75.96%   -13.58%
===========================================
  Files           1        1
  Lines          86      104       +18
===========================================
+ Hits           77       79        +2
- Misses          5       17       +12
- Partials        4        8        +4

Impacted Files
Coverage Δ

boot.go
75.96% <46.66%> (-13.58%)
:arrow_down:

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 489f48a...678a273. Read the comment docs.


SOP generate-key --profile=rfc4880 generates single RSA key instead of primary+subkey

When implementing support for profiles in PGPainless' SOP implementation, I took the shortcut of reusing the old simpleRSAKeyRing() template.
This results in an OpenPGP key consisting of a single RSA key which is used for both certifying/signing as well as encryption.
It would probably be a good idea to generate a primary+subkey combo with two or more different (sub)keys instead. That way, the primary key could be long-lived, with rotating signing/encryption subkeys.
It is also a good idea to implement this as a more advanced template method in KeyRingTemplates.

This is now implemented.


Trouble on Stellarium with Synscan

Thanks a lot for your work !
I install it on a raspberry linked to a Skywatcher BD 200/1000 on and NEQ5 with a SynScan v3 handler and I use Stellarium on a remote PC. Everything works well except that  my scope moves sometimes, sometines not and he doesn't move in the right direction sometimes.
When linked directly on stellarium (without node-telescope-server) everything is Ok.
So my question is the following: do u think ur code works on a SynScan handler ?
If not which parameters i have to change to build it for synscan ?
Thanks

I received some patches that maybe fix your problems about random coordinates. Waiting for https://github.com/fcsonline/node-telescope-server/issues/7 to review the pull request.


Added Scoala Mihai Eminescu

Domain: @scoala-mihaieminescu.ro
Website: https://www.scoala-mihaieminescu.ro

@Andu98 First stage of verification passed, official domain is scoala-mihaieminescu.ro as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Andu98 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Works Fine but !!

I can draw graph thanks to your libraries but when i try to import the png into my program it misses the bottom part of it :

What i find strange its the fact that when i close my app it finish to generate the png :

Which library are you using to load the PNG?

I use GTK 2 to load the png

Are you using this function?
GtkWidget *image;
image = gtk_image_new_from_file ("myfile.png");

If so, did you make sure the file was completely written to the file before loading it?

I tried GTK 2, and it worked. Could you paste your code here?

`void graph_print(char input_f[], int argc, char* argv[], GtkWidget* widget, gpointer data)
{
fonction = gtk_label_new(input_f);

gtk_init(&argc, &argv);

gtk_container_remove(GTK_CONTAINER(window), vbox); // Remove previous window

hbox3 = gtk_hbox_new(TRUE, 10);
vbox = gtk_vbox_new(FALSE, 30);
vbox1 = gtk_vbox_new(FALSE, 30);

gtk_box_pack_start(GTK_BOX(hbox3), fonction, TRUE, FALSE, 10);

graph_draw();

image = gtk_image_new_from_file ("src/graph.png");

gtk_box_pack_start(GTK_BOX(vbox), hbox3, FALSE, FALSE, 5);
gtk_box_pack_start(GTK_BOX(vbox), image, TRUE, FALSE, 10);
gtk_container_add(GTK_CONTAINER(window), vbox);

gtk_widget_show_all(window);

gtk_main();

g_signal_connect(window, "destroy", G_CALLBACK(destroy), NULL);

return 0;

}
void graph_draw()
{
//récupération des points de la fonction pour les placer sur le graph
double xs [] = {-10, -1, 0, 1, 10};
double ys [] = {6, -2, -2, -1, 2};

//génération des axes
ScatterPlotSeries *series = GetDefaultScatterPlotSeriesSettings();
series->xs = xs;
series->xsLength = 5;
series->ys = ys;
series->ysLength = 5;
series->linearInterpolation = true;
series->lineType = L"dashed";
series->lineTypeLength = wcslen(series->lineType);
series->lineThickness = 2;
series->color = GetGray(0.3);

ScatterPlotSettings *settings = GetDefaultScatterPlotSettings();
settings->width = 600;
settings->height = 400;
settings->autoBoundaries = true;
settings->autoPadding = true;
settings->xLabel = L"X axis";
settings->xLabelLength = wcslen(settings->xLabel);
settings->yLabel = L"Y axis";
settings->yLabelLength = wcslen(settings->yLabel);
ScatterPlotSeries *s [] = {series};
settings->scatterPlotSeries = s;
settings->scatterPlotSeriesLength = 1;

//Génération d'un .PNG qui contient la représentation graphique de la fonction
RGBABitmapImageReference *canvasReference = CreateRGBABitmapImageReference();
DrawScatterPlot(canvasReference, 600, 400, xs, 5, ys, 5);

size_t length;
double *pngdata = ConvertToPNG(&length, canvasReference->image);
WriteToFile(pngdata, length, "src/graph.png");
DeleteImage(canvasReference->image);

return 0;

}
`
I put the function i use to generate the plot and the one i use to call it.
Thanks for your replies by the way

I had forgotten to close the file in the WriteFile function. Download the latest version of supportLib.c and things should work nicely.

Thanks a lot


[BACKPORT][v1.2.x][BUG] Replica stuck in buggy state with status.currentState is error and the spec.desireState is running

Ref: https://github.com/longhorn/longhorn/issues/4197

Verified passed on v1.2.x-head (longhorn-manager 5917653) by following https://github.com/longhorn/longhorn/issues/4197#issuecomment-1193023991.


Optimised setMod() function and other formatting

Hi, @Jihanger-997,
So after reading all the code, and functions I found a function setMod() in abstract class Character.java, that contains a series of if-else that can easily be replaced by a single if statement (line number = 60).
I haven't changed any of the game logic or any property just indented it better.
And for the print statement part, when you are going to use JFrame for UI?
as I can then help with these print statements in JFrame.

Posted some single comments, Character.java line 60 requires a different approach. Everything else will be merged.

Yes changing it now...

Posted some single comments, Character.java line 60 requires a different approach. Everything else will be merged.

@Jihanger-997, I have changed the required, please check.


dbclient: Relax named parameter identifier rules and align with JPA 2.0 spec to fix #2922

Currently, named parameter identifiers following the : are parsed in a stricter fashion than in JPA implementations offering similar functionality. Users coming from the JPA world will be inconvenienced while attempting to declare certain named parameters, as in Issue #2922.
This change relaxes the character requirements to include characters like underscores and dollar signs, aligning with the JPA 2.0 specification.
Rules for named parameters on page 123, rules for the identifiers following the : of a named parameter on page 136:
https://download.oracle.com/otn-pub/jcp/persistence-2.0-fr-eval-oth-JSpec/persistence-2_0-final-spec.pdf

Not sure why the OCA check failed. I signed it and successfully contributed before?

Wow, thanks a ton for doing this (and so quick) @amannm !

Well, first of all, DB Client is not JPA. :) This is more JDBC level library so I did not care about JPA when implementing rules for identifiers.
On the other hand relaxing identifier rules won't break anything so it sounds like a good idea if it helps people. Your changes seem to be OK. I verified state machine transitions and it shall work the same way.
We just need OCA check to pass.

anything extra I need to do after submitting the OCA request?

I see you marked as contributor now. But I have no idea why the check is still failing. @romain-grecourt is our build master, maybe he knows more.

...but I still can't see you on the list of OCA Signatories

/oca-checked

/oca-checked

/trigger


Improve labels on LED columns for bluetooth pairing

At this point:

The labels are e.g. "Number of LEDs lit in column 5 on the micro:bit display"
This leads to a very verbose screen reader announcement "Number of LEDs lit in column 5 on the micro:bit display  spin button  required  editable  selected 4” (NVDA, default settings)
The most important information is which column you are in - can we just flip it to put the salient info up front?
e.g.  "Column 5 - number of LEDs lit on the micro:bit display"

Hello, I tried to solve the issue.
This is what I did:
Modified the FormattedMessage in the PatternColumnInput component to put the column number first in the label, improving screen reader accessibility. The new label format is 'Column {colNum} - number of LEDs lit on the micro:bit display'.
You can review changes in this commit: https://github.com/lattaai12/microbit-foundation-ml-trainer-520/commit/0094b6a0714204d229fd6ffa154d6aad947fecbe.

[!CAUTION]
Disclaimer: The commit was created by Latta AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

This issue was tried to solve for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Connect Function does not work properly with Toolbox Options

Version
5.0.0
Reproduction link
https://codepen.io/jinyjs/pen/ZEWRRBQ
Steps to reproduce
Click on the 'Bar' Option in 'chart1', the data of 'chart1' is being displayed in 'chart2' as well.
Also vice -versa
What is expected?
The data in the charts should not be transferred when using the toolbox Option in combination with connect Function.
What is actually happening?
Data from one chart is being displayed in the other charts.

At present, this effect is by design.
If intending to achieve the expected effect, needs to make connection manually.
That is, listen to each event from chart1 and dispatchAction to chart2.

Hallo,
thank you for the information. Is there any help example that I can refer to in order to understand how to use the dispatchAction function.


Show more registers data

https://trello.com/c/Cet7azEm/371-food-demo-should-show-all-the-data

looks good 👍


Develop

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 11 (0.0%)  changed or added relevant lines in 1 file are covered.
19 unchanged lines in 10 files lost coverage.
Overall coverage increased (+1.007%) to 62.028%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

catalyst_utils/management/commands/generate_mailing_lists.py
0
11
0.0%

Files with Coverage Reduction
New Missed Lines
%

catalyst_utils/context_processors.py
1
0%

catalyst_utils/cache.py
2
0%

catalyst_utils/management/commands/export_gradebooks.py
2
0%

catalyst_utils/management/commands/export_surveys.py
2
0%

catalyst_utils/management/commands/generate_mailing_lists.py
2
0%

catalyst_utils/management/commands/update_authz_groups.py
2
0%

catalyst_utils/management/commands/update_gradebook_attr.py
2
0%

catalyst_utils/management/commands/update_netid_admins.py
2
0%

catalyst_utils/management/commands/update_person_attr.py
2
0%

catalyst_utils/management/commands/update_survey_attr.py
2
0%

Totals

Change from base Build<PHONE_NUMBER>:
1.007%

Covered Lines:
526

Relevant Lines:
848

💛  - Coveralls


feat: Add MuQ, a thread-safe variant

MuQ is a thread-safe variant of Q
with a slightly different API.
Specifically,
it lacks a Do method because we don't want to lock the queue
while an arbitrary user-specified callback runs.
And it does not include panicking Pop and Peep variants
because it's impossible to write code that uses them safely
without additional external locking:
for !muq.Empty() {
  muq.Pop()
}

This will panic if run from two goroutines because there's a race
between Empty() and Pop().

Current dependencies on/for this PR:

main

PR #10   👈

This comment was auto-generated by Graphite.


Utilize Terraform to create or update Datadog dashboards

Value Statement
As VANotify developer
I want to utilize Terraform for creating dashboards
So that I have insights on my application metrics in Datadog
Acceptance Criteria
GIVEN I am creating a Datadog dashboard
WHEN I run Terraform
THEN the appropriate dashboard is created
Checklist

[ ] Make sure everything that was manually created and we want to keep is here.

Assumptions

Additional Info/Resources

The Terraform for Datadog documentation is at https://registry.terraform.io/providers/DataDog/datadog/latest/docs/resources/dashboard.
The fastest method is to use Datadog to export the dashboard and then create the terraform file using the json file.

Out of Scope

Open Questions

@ffafara-tw do we still think this is an issue? Just checking in as I go through the backlog with Qaysar.

Yes it is something we still want to do.
I believe Phil is doing some work around creating DataDog resources (graphs, alerts, dashboard) via terraform as part of his current story. So depending on how far he gets we might cancel this in the future.

Closing this because the work has been done -
Here is a list of Datadog dashboards created by Terraform

Closing this because the work has been done -
Here is a list of Datadog dashboards created by Terraform


yx: 1.0.0 -> 1.0.1

Description of changes

Add version test
Don't misuse pname
Fix wrong version number
Fix man page installation

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

$ nix-build -A yx
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1

$ find /nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1/bin
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1/bin/yx
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1/share
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1/share/man
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1/share/man/man1
/nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1/share/man/man1/yx.1.gz

$ nix-build --attr pkgs.yx.passthru.tests
/nix/store/ag5yj0yblwmq1sm0hgpiq6n8i06vfxb7-yx-1.0.1-test-version

$ /nix/store/87gqqqrf3cbpad5jf7mrc9fg43npl31p-yx-1.0.1/bin/yx --help
Usage: yx [<options>] [<path>]
<options> :-
  -h|--help         : show this help
  -d|--doc <n>      : parse <n>th YAML document, default: 1
  -f|--file <file>  : parse YAML from <file>, default: STDIN
  -v|--version      : display version
  --                : end of command options
<path> :-
  <key>|<index> ... : path to the data to extract
<key>               : map key value
<index>             : sequence <index> number, starts with 1

Add a :+1: reaction to pull requests you find important.

Superseded by #284353.


added assembly page

missing tests
fixes: #18 #17

@lcacollect/documentation needs to be bumped so it includes the Assembly entry in the sidemenu.

There is a typo in "Aseemblies"

Users shouldn't see the Meta Fields when creating a new assembly

Great work @EgiLuk !
You can remove the FILTERS from the Assembly Layers Table?
Could you add the information from the dialog (Category, Description, Life Time and Conversion Factor) in between the Assembly Name and the layers table to the user can see it?

And you need to bump the package version


plugin fails to establish connection to the API

Hi
plugin is failing to be loaded and I can see following in alerta.log:
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7fc86a8265c0>: Failed to establish a new connectio
n: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 638, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/usr/lib/python3/dist-packages/urllib3/util/retry.py", line 398, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /blackouts (Caused by
NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fc86a8265c0>: Failed to establish a new connection: [Errno 111] Conn
ection refused'))

This looks like it tries to connect to localhost port 8080, but there is no service listening on this port.

solved by adding configuration to nginx:
upstream backend {
        server localhost:8080 fail_timeout=0;
}


Attuned items dont stay attuned or have attuned attribute turned on

Items that require attunement or are currently attuned dont keep that attribute when imported

Please can you provide a character link, and the options you have selected when importing. (screenshot works best for this)

Please can you provide a character link, and the options you have selected when importing. (screenshot works best for this)

Thanks for the quick reply. Let me know if you need more info. (its happened on 3 characters)
In this character you can see Shatterspike and Ring of spell storing require attunement (the ring is a homebrew if that makes any difference but shatterspike is from sunless citadel - purchased)
I am also using The Forge if that also makes any difference?
As for the options i selected, I am not sure - I have updated that character a couple of times.  I will start a new character in another world and let you know how it goes.
https://www.dndbeyond.com/profile/Mushie101/characters/31459823
On Monday, 11 January 2021, 11:23:05 pm AEDT, Jack Holloway<EMAIL_ADDRESS>wrote:

Please can you provide a character link, and the options you have selected when importing. (screenshot works best for this)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks for the quick reply. Let me know if you need more info. (its happened on 3 characters)
In this character you can see Shatterspike and Ring of spell storing require attunement (the ring is a homebrew if that makes any difference but shatterspike is from sunless citadel - purchased)
I am also using The Forge if that also makes any difference?
As for the options i selected, I am not sure - I have updated that character a couple of times.  I will start a new character in another world and let you know how it goes.
https://www.dndbeyond.com/profile/Mushie101/characters/31459823
On Monday, 11 January 2021, 11:23:05 pm AEDT, Jack Holloway<EMAIL_ADDRESS>wrote:

Please can you provide a character link, and the options you have selected when importing. (screenshot works best for this)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

The character imports for me with correct attunement options. Please can you confirm the version of ddb-importer you're using, and a screen shot of the options you use when importing. Thanks.

The character imports for me with correct attunement options. Please can you confirm the version of ddb-importer you're using, and a screen shot of the options you use when importing. Thanks.

I just tried it on another character and it was all good, so I am not sure what happened to the other 2 characters. Thanks for checking, sorry to any hassle.  Do you want me to delete the comment in github?
Marcel.
On Tuesday, 12 January 2021, 4:26:29 am AEDT, Jack Holloway<EMAIL_ADDRESS>wrote:

The character imports for me with correct attunement options. Please can you confirm the version of ddb-importer you're using, and a screen shot of the options you use when importing. Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I just tried it on another character and it was all good, so I am not sure what happened to the other 2 characters. Thanks for checking, sorry to any hassle.  Do you want me to delete the comment in github?
Marcel.
On Tuesday, 12 January 2021, 4:26:29 am AEDT, Jack Holloway<EMAIL_ADDRESS>wrote:

The character imports for me with correct attunement options. Please can you confirm the version of ddb-importer you're using, and a screen shot of the options you use when importing. Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Please let me know if it re-occurs.

Please let me know if it re-occurs.


Training ChangeFormer from scratch.

I'm trying to train ChangeFormer via open-cd and as an argument I'm passing the file changeformer_mit-b0_256x256_40k_levircd.py , where there is a checkpoint variable with a link assigned to it. After removing the link, leaving the checkpoint contain an empty string, it still loads checkpoints from somewhere. How to train ChangeFormer from scratch then?

checkpoint=None


du: Update the GNU error message to our for tests/du/threshold.sh

We are doing a better job.
Make it FAIL => PASS

Warning: Congrats! The gnu test tests/du/threshold is no longer failing!


[c++] Faster Serializer::Field

The change adds overloads of Serializer::Field function that avoids generating calls to detail::omit_field in the cases where the result is known at compilation. This makes the compilation a bit faster (it depends on actual types being used, but the expected estimate would be 3-5%) and also likely to generate faster, smaller and more efficient runtime code when compiler fails to optimize away the above calls.

Is this faster at runtime or compile time? Approximately how much faster?

@chwarr, added a description.


python.pkgs.pygraphviz: add graphviz path

Motivation for this change
I noticed that pygraphviz is not really working unless I explicitly add graphviz to my environment. To fix that, I patch pygraphviz to always add graphviz to its path.
Also it depended on python.pkgs.graphviz instead of the C package graphviz.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python2.pkgs.pygraphviz python3.pkgs.pygraphviz

nix-review passes. Nothing actually depends on pygraphviz. I only know of a soft dependency from networkx, which is how I noticed the problem.

Thanks! Backport in #51502.


Singletons 3 support + misc fixes

This commit adds support for singletons-3.0 (for GHC 9.0), plus some miscellaneous fixes:

add some COMPLETE pragmas on some pattern synonyms,
remove the testing dependencies from the library component in the cabal file.

Thanks a lot for cleaning up some of my mess. I will modify my package.yaml accordingly and merge this.

I will modify my package.yaml accordingly and merge this.

Yes, sorry, I only realised later you were using package.yaml and not writing the cabal file yourself. I don't know how to replicate the behaviour of the cabal file in the package.yaml file.

Can you please allow changes from maintainers to this PR? Or, if you are ok with this, I can just apply your changes as a patch.

Can you please allow changes from maintainers to this PR? Or, if you are ok with this, I can just apply your changes as a patch.

Edits by maintainers have been allowed from the start, so you should be able to make changes.

Once again, thank you. I learned a couple new things about git, stack and ghc thanks to this PR.


SASL GSS-API/Kerberos

Can't find example configuration to login LDAP with Kerberos keytab without bind_dn and password.
Is it possible or planned?

@eugene-bright just to clarify when you say "login into LDAP" you mean the connection between the auth_server and LDAP, and not between docker client and auth_server?

@eugene-bright just to clarify when you say "login into LDAP" you mean the connection between the auth_server and LDAP, and not between docker client and auth_server?

I'd like the auth_server would connect to a LDAP server by using SASL GSSAPI without plaintext password by utilizing provided keytab. I suppose that current libs do not compatible with MIT Kerberos and do not respect KT5_KTNAME and KT5CCNAME environment variables.

I'd like the auth_server would connect to a LDAP server by using SASL GSSAPI without plaintext password by utilizing provided keytab. I suppose that current libs do not compatible with MIT Kerberos and do not respect KT5_KTNAME and KT5CCNAME environment variables.

Thanks for the clarification :)
Seems you are right about current libs don't support it (yet.. https://github.com/go-ldap/ldap/issues/115 ). I will keep this open, and keep an eye on the upstream ticket.

Thanks for the clarification :)
Seems you are right about current libs don't support it (yet.. https://github.com/go-ldap/ldap/issues/115 ). I will keep this open, and keep an eye on the upstream ticket.


confusing error message on simple inductive definition

Hello
i've a teeny inductive data type (part of trying to do a little curry/church lambda calculus)
inductive types (fv : nat)   : Type
    | tyvar : fin fv  -> types fv

i get two errors
on inductive i get invalid return type for types.tyvar
on the return expression for tyvar  i get the error function expected at types

When defining an inductive type, omit the parameters in the return type (in this case, fv):
inductive types (fv : nat)   : Type
    | tyvar : fin fv  -> types

that solves it for me, thanks!
... also explains why i wrote too many things as indices rather than parameters last time i wrote lean code.
a) should there be / could there be a better error message here?
b) is this clearly talked about in any of the basic docs or example developments? :)

By coincidence, I'm working on a set of changes that will expand the error message to
2:26: function expected at
  types
term has type
  Type

It still doesn't tell you what's going on with the inductive specifically, but do you think that might have made things slightly more clear?

I think that we should have a hint like (Lean does not write parameters in
the result type). This is a common error from Coq users.
On Tue, Jul 18, 2017 at 9:22 AM, Sebastian Ullrich<EMAIL_ADDRESS>
wrote:

By coincidence, I'm working on a set of changes that will expand the error
message to
2:26: function expected at
types
term has type
Type
It still doesn't tell you what's going on with the inductive specifically,
but do you think that might have made things slightly more clear?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/leanprover/lean/issues/1750#issuecomment-315993560,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1A7C42o76d9QMKb9RDUGTZ6uGQdryAks5sPGs_gaJpZM4OaXiT
.

@Kha  .... that or the current error message are only helpful if you already know / understand the lean difference in notation.
I agree with @digama0 ,

Unfortunately, I don't see a good way at all for how to bring inductive compiler and elaborator together to produce such a message. We could implement a crude but simple approximation by inspecting each intro rule preterm before elaboration.

@cartazio In response to your other question, this is mentioned in Theorem Proving in Lean, e.g. in Section 7.2: https://leanprover.github.io/theorem_proving_in_lean/#07_Inductive_Types.html. Maybe we can make it more prominent. We have also started working on a reference manual, which, I think, will help in situations like this. (At least, it will provide an obvious place to look when something goes wrong.)

i cant speak to the best path for discoverability :)
but YES, a reference manual that i can look at would be a better condensed discovery tool :)
hrmm, shame about the error message ux puzzle
should i leave this ticket open till those action items happen, or does it make sense to close it atm?

I think it makes sense to close it.


extract template rendering for variables into own repo

Template rendering for variables in the final version should be efficient and easy to maintain.
Currently we're using https://github.com/valyala/fasttemplate which still makes unnecessary transformations between []byte and strings. This also denies access to std go templating patterns like if conditions or iterating over a slice or map. A new repo for efficient templating on a byte slice would solve the problem.

Done


DRAFT: please_ack support PoC for the 0453-issue-credential-v2 protocol

This PR is PoC code for the PR#2540 (https://github.com/hyperledger/aries-cloudagent-python/pull/2540). It shows possible implementation of the please_ack decorator support ('option 2' in the document in the PR#2540)

Thanks!  Awesome.  Sorry for the silence on this — we’ve been at a conference this week.  We’ll be looking at this next week.  Would you be willing to present your design and implementation at the ACA-Pug meeting this coming Tuesday (2023-10-17, 8:00 Pacific / 17:00 Central Europe)?

Thanks! Awesome. Sorry for the silence on this — we’ve been at a conference this week. We’ll be looking at this next week. Would you be willing to present your design and implementation at the ACA-Pug meeting this coming Tuesday (2023-10-17, 8:00 Pacific / 17:00 Central Europe)?

Hello @swcurran . Thank you for the answer. I think it's a good idea. I will participate the meeting and share my thoughts/ideas.

Note the lint and failed PR test(s).


configure emitter parameters from filter rewriteTag

Ability to set emitter parameters from filter rewriteTag.
In addition to clusterfilter-only https://github.com/fluent/fluent-operator/pull/1069
None

apiVersion: fluentbit.fluent.io/v1alpha2
kind: Filter
metadata:
  labels:
    fluentbit.fluent.io/enabled: "true"
  name: filter
  namespace: namespace
spec:
  match: kube.*
  filters:
  - rewriteTag:
      emitterName: rewrite
      emitterStorageType: filesystem
      emitterMemBufLimit: 8MB
      rules:
      - $flow_tag msg $TAG[0].msg false

@chrono2002 you didn't use the make manifests to generate the CRDs, so your modifications will be deleted each time the make generate command will be played

@chrono2002 you didn't use the make manifests to generate the CRDs, so your modifications will be deleted each time the make manifests command will be played

Yeah, @chrono2002 you'll need to change the go struct definition first and the crd yamls are generated automatically by running:
make manifests
make generate
make fmt
make vet
make docs-update

Here're some examples:
https://github.com/fluent/fluent-operator/pull/1018
https://github.com/fluent/fluent-operator/pull/972


MCP23017 integration or RPI-GPIO with I2C?

Hello
I use MCP23017 integration for a long time ago.
My home alarm is very stable using this integration and a all wired network sensors and switches.
(In the alarm system I prefer wired sensors and switches)
As I understood, we will loose it.
Is it possible to have MCP23017 integration in HACS?
This RPI-GPIO could have I2C?
Or I need to think about change my hardware...
Thanks for your work and contributions

The MCP23017 is a different integration, it is not related to the RPI-GPIO integration.
For now I am only adding integrations I have the hardware to test my changes on as I don't believe in supporting an integration I can't test the code myself.
I will consider buying a MCP23017 and adding support for it.

@gugaspinto Also this might suit you (but I'm going to track this one :D)

Thanks...
Probably it solves my situation..
Very thanks for you attention


Fix for #524

Fixes #524

added one additional test
all existing tests are passing

@vishwacsena Build is failing. Looks like your new test.
(node:5696) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'urls' of undefined
at /home/travis/build/Microsoft/botbuilder-tools/packages/Ludown/lib/parseFileContents.js:32:31
at Array.forEach ()
at collateQnAFiles (/home/travis/build/Microsoft/botbuilder-tools/packages/Ludown/lib/parseFileContents.js:31:163)
at Context. (/home/travis/build/Microsoft/botbuilder-tools/packages/Ludown/test/ludown.qnafiles.test.suite.js:35:9)
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:5696) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 13)
(node:5696) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
npm ERR! code ELIFECYCLE
npm ERR! errno 10
npm ERR<EMAIL_ADDRESS>test: cd test/ && mocha --opts mocha.opts

@cleemullins - the offending test needed a fix. All tests are passing locally. Hope this round will go through clean with the build.

@cleemullins - the build is failing again with ludown translate tests. Are these using my personal translate key? If so, looks like Tom yanked out the key I had created in the PPE subscription. I ran into this issue today when testing Darren's ginormous lu file with over 15K lines of utterances. So

We should switch to a team key and not use my personal key
I do not know how/ where this is getting picked up from.

In any case, I created a new key off our PPE subscription and emailed you the key. Can we restart the tests with the updated key?

@tomlm has updated the build, so things should pass. kicking off a new build.

Pull Request Test Coverage Report for Build 1241

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 68.771%

Totals

Change from base Build 1234:
0.02%

Covered Lines:
3860

Relevant Lines:
5383

💛  - Coveralls


code testing and code coverage?

it seems there is no test case and code coverage?

+1

+1

Not only is it a good practice, but it provides examples on how to test websocks with express-ws in our own code.


cico_build_deploy.sh creates the "html" directory as root

When the cico_build_deploy.sh script is run, it builds the doc set inside the container with root privileges, therefore creating the $REPO_HOME/html directory owned by root:root. This makes it impossible for the regular user to build the doc set later using the build_guides.sh script as it tries to remove the root-owned directory.
Either the html directory should be removed after it serves its purpose in the cico_build_deploy.sh script, or its ownership should be set to the regular user who ran the script.

@tradej closing this due to #520


the apply_all_changes script contains logic to setup genkins configur…

…ations and revert_all_changes script alllows to revert all changes

groovy scripts for jenkins build automation


Fix broken link

This fixes the broken link to Rudi Grinberg's "Type Safe Routing - Baby Steps" blog post

cc @evancz


NSF credit missing in footer

This blurb needs to go into the footer: Copyright 2022 Environmental Data Initiative. This material is based upon work supported by the National Science Foundation under grants #1931143 and #1931174. Any opinions, findings, conclusions, or recommendations expressed in the material are those of the author(s) and do not necessarily reflect the views of the National Science Foundation. Please contact us with questions, comments, or for technical assistance regarding this web site or the Environmental Data Initiative. Please read our privacy policy to know what information we collect about you and to understand your privacy rights.

Thanks @cgries. The proposed text takes up quite a bit of real estate. Can we pair it down to something like This material is based upon work supported by the National Science Foundation under Award Numbers #1931143 and #1931174?
Should we also add the NSF logo to the list of affiliates in the footer?

sure that's fine for the footer. But the full text needs to be somewhere on the website as per NSF. Scroll down on this page: https://www.nsf.gov/pubs/gpg/nsf04_23/6.jsp to see the instructions and decide where they should go.

This issue is partially fixed by PASTAplus/content-x@a756852f035a2dd7add90b815d1e9fabcdf59c88

The remainder of this fix is now in main at 6bb06359c904066210531c32b2e4e1cf85b0c926.


API filter option for docs.rs

There are multiple APIs present under standard library and other crates listed under crates.io, having a filtering feature to segregate nightly APIs would be very helpful where there huge number of APIs present

Hi @artech-git, thank you for this idea!
Could you help me understand more about what you are trying to achieve?
Are you talking about standard-library docs? Or crate documentation?
And for the filter-feature, do you mean in normal module lists or in the search?
In general it is likely that this is actual an issue for rustdoc or the stdlib-docs, not for docs.rs, which is "just" hosting the docs generated by rustdoc.
I also remember seeing a rustdoc issue about rendering nightly-only apis differently in search-results? ( cc @GuillaumeGomez )

Well I was referring to this feature in general if it can be made available for both std docs and crates docs as well
for the filter feature maybe we can have a label button under the interface, or perhaps what will be more powerful is to embed this feature directly in the search bar where we can filter the APIs using some prefix

I don't remember such a feature request. If the issue doesn't already exist, can you open one on the rust-lang/rust repository please?

with pleasure 😊

I don't remember such a feature request. If the issue doesn't already exist, can you open one on the rust-lang/rust repository please?

it would be awesome if you could link the rustdoc issue after it's created, then we can close this one.
Thanks!


Send request replay to arbitrary connection

Problem Description
I'm using mitmproxy to make requests that did not originate from an actual client. The type of request I'm trying to make could never actually be made through the proxy. For example I want to connect to an arbitrary host like "<IP_ADDRESS>:1337" (locally for testing) with an arbitrary Host header e.g. "www.example.com". I'm creating a new instance of mitmproxy.http.Request and also an instance of mitmproxy.connection.Server. But the problem is that the replay ignores flow.server_conn and constructs a new connection based on Request, which breaks what I'm trying to do. It will actually connect to <IP_ADDRESS>:1337 (if I pass the host to the Request constructor) but the request ncat gets looks like this (which interestingly is missing the 1337 port):
GET http://<IP_ADDRESS>/ HTTP/1.1
Host: www.example.com

What I want is the following request made to <IP_ADDRESS>:1337
GET / HTTP/1.1
Host: www.example.com

Proposal
I think I found the right place?
https://github.com/mitmproxy/mitmproxy/blob/2d2a9f3c528166f04dba168f0adbe510eb74900f/mitmproxy/addons/clientplayback.py#L70-L76
But I wasn't able to patch it to make it work. Ideally we could check for server_conn is None and just clone it, but "those flows will still have a server_conn attribute with a timestamp_start set to None.".
So how can I make replay use my flow.server_conn while not breaking existing replay? I don't want replay to create a server_conn from flow.request, which is what's currently happening. I want to manually create a connection and have replay make the request to it.
Alternatives
A night of sleep, tomorrow probably brings the solution
Additional context
I'm tired.
I can provide a stripped down addon tomorrow, it goes like this (imagine there would be parameters :smirk:  )
request = mitmproxy.http.Request()
client_conn = mitmproxy.connection.Client()

# I pass an address here that is not related to Request above
server_conn = mitmproxy.connection.Server()

flow = mitmproxy.http.HTTPFlow(client_conn, server_conn)
flow.request = request

ctx.master.commands.call("replay.client", [flow])

WTF are you even doing?
E.g. fuzzing for other vhosts on a given IP and other fun misconfigurations. I just realized that this refs some of those "allow custom DNS in mitmproxy". This is a subset I guess?

Look into the distinction between .host and .pretty_host. Mitmproxy should forward requests where the host header does not match the .host attribute just fine. FWIW it would also be feasible to do for external clients by supplying a different target in the HTTP CONNECT.

A night of sleep, tomorrow probably brings the solution

Checks out.

Look into the distinction between .host and .pretty_host. Mitmproxy should forward requests where the host header does not match the .host attribute just fine. FWIW it would also be feasible to do for external clients by supplying a different target in the HTTP CONNECT.

Thanks a lot. I think it's working now as expected, I was misunderstanding the authority argument too.
This appears to do what I want:
import time

import mitmproxy
import mitmproxy.http
import mitmproxy.connection
import mitmproxy.net.http.headers

from mitmproxy import ctx

class MyAddon:
    def running(self):
        self.make_request()

    def make_request(self):
        headers = mitmproxy.http.Headers(
            [(b"Host", b"www.example.com"), (b"X-Foo", b"bar")]
        )

        request = mitmproxy.http.Request(
            host=b"<IP_ADDRESS>",
            port=1337,
            method=b"GET",
            scheme="http",
            authority=b"",
            path=b"/",
            http_version=b"HTTP/1.1",
            headers=headers,
            content=b"",
            trailers=None,
            timestamp_start=time.time(),
            timestamp_end=None,
        )

        client_conn = mitmproxy.connection.Client(
            ("", 0), ("", 0), request.timestamp_start - 0.0001
        )

        server_conn = mitmproxy.connection.Server(("", 0))

        flow = mitmproxy.http.HTTPFlow(client_conn, server_conn)
        flow.request = request

        ctx.master.commands.call("replay.client", [flow])

addons = [MyAddon()]

ncat -vkl 1337
mitmdump -s make-request.py

ncat gets
GET / HTTP/1.1
Host: www.example.com
X-Foo: bar

The only thing that's somewhat awkward is that the server_conn is not used at all and that it is instead created from the request. It's somewhat of a duplication, does this info really belong to the request object?
Turns out my patch to clientplayback.py was working.


Twitter weird response

Hi, is anyone getting some response like this :
Searching...

Twitter weird response. Try to see on browser: https://twitter.com/search?q=%20from%3Abarackobama&src=typd
Done. Output file generated "output_got.csv".

after running:
python Exporter.py --username "barackobama" --maxtweets 10 --toptweets

fixed this issue by adding:

import ssl
ssl._create_default_https_context = ssl._create_unverified_context

to the header of Exporter.py


Stop on BLE Receiver and Transmitter doesn't stop

Phone Receiver is still scanning, receiving callbacks, even though peripheral.disconnect() has been called.
Same for transmitting.
Makes using the new Demo UI difficult as it has multiple modes and needs an explicit start/stop.

Fixed in recent PR - now has a flag in addition to just relying on Bluetooth device state. This allows for more fine grained controls in the app.


Change text on search button

Changes the text on the search button that is used in the matchmaker tab.
Apparently people regularly don't realize that they need to press this button to actually start searching, so the text is more explicit now.
Btw, do existing translations automatically get some sort of mark that the original text has changed and they need retranslation?

They do not that is why a new key should be used instead of reusing an old one.


Updated User-Agent header logic

Issue/Request:
The User-Agent header was not set when passed as a HTTP header param. Added the ability to set User-Agent via header and/or separate CLI parameter -U.

@rakyll is there any chance to get this merged? Setting the user-agent would allow Flagger load generator service to work for A/B testing scenarios when the test is based on browser/os versions.
Thanks

@rakyll Maybe, you should publish a release version. 😢

This update will append the original UA so you end up with something like new ua hey/0.0.1:
// set userAgent header if set
    if *userAgent != "" {
        ua = *userAgent + " " + heyUA
        header.Set("User-Agent", ua)
    }

I just very quickly threw up a fork that will completely replace the UA, with installation instructions in the readme:
https://github.com/lindsey-em/hey

Hey
Thanks for your help!
I tried installing it by following the readme instructions. While trying to
run the "go get..." I got this error:
unrecognized import path "net/http/httptrace" (import path does not begin
with hostname)
Is there anything else I should do?
Thank you
On Thu, Dec 8, 2022 at 2:30 AM lindsey-em @.***> wrote:

This update will append the original UA so you end up with something like new
ua hey/0.0.1:
// set userAgent header if set
if *userAgent != "" {
ua = *userAgent + " " + heyUA
header.Set("User-Agent", ua)
}
I just very quickly threw up a fork that will completely replace the UA,
with installation instructions in the readme:
https://github.com/lindsey-em/hey
—
Reply to this email directly, view it on GitHub
https://github.com/rakyll/hey/pull/87#issuecomment-1341800101, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXZ5357FRU4WR37GIL3XFVLWMET2VANCNFSM4ENOA7AQ
.
You are receiving this because you commented.Message ID:
@.***>

--
ZIV ELYASHIV
DDoS Expert
E: @.**
*M: *+972-(0)50-3213330
[image: Red Button]
[image: Under DDoS Attack? Click here for IMMEDIATE ASSISTANCE]
https://www.red-button.net/
www.red-button.net https://www.red-button.net/
[image: Blog] https://www.red-button.net/blog/
https://www.linkedin.com/companies/red-button-ddos-experts [image:
youtube] https://www.youtube.com/channel/UCNDr_afX7_e-ityDrjF5DBQ
This e-mail message may contain confidential or legally privileged
information and is intended only for the use of the intended recipient(s).
Any unauthorized disclosure, dissemination, distribution, copying or the
taking of any action in reliance on the information herein is prohibited.
E-mails are not secure and cannot be guaranteed to be error free as they
can be intercepted, amended, or contain viruses. Anyone who communicates
with us by e-mail is deemed to have accepted these risks. Red Button is not
responsible for errors or omissions in this message and denies any
responsibility for any damage arising from the use of e-mail. Any opinion
and other statement contained in this message and any attachment are solely
those of the author and do not necessarily represent those of the company.


Improved Vms fetching by using new API features, and divided fetching…

… according to different API versions
@mareklibra Please review

LGTM in general

@mareklibra Done, please review

@mareklibra @jniederm fixed

@jniederm , are we ready to merge?


[OSX] (Scene2D) UI events sometimes doesn't work

Issue details
I run the game, UI events don't work, i have to double click to a button to trigger the clicked event, the button don't event display the down state
It's totally random, on different game restart, the issue will disappear
Reproduction steps/code
Just create some UI controls
Version of LibGDX and/or relevant dependencies
1.9.3-SNAPSHOT
Lwjgl3 backend
Stacktrace
None
Please select the affected platforms

[ ] Android (not tested)
[ ] iOS (not tested)
[ ] HTML/GWT (not tested)
[ ] Windows (not tested)
[ ] Linux (not tested)
[x] MacOS

Please provide SSCCE, without code it is practically impossible to determine what is the problem, especially when it is nondeterministic.

https://github.com/libgdx/libgdx/wiki/Issue-Tracker


Issue #120: Byteorder mark removal in UncSource removes first left angle bracket '<' of xml document

This pull request fixes the issue of reading XML files due to BOM removal not working as described in issue #120. I added several test cases for reading differently encoded files:

UTF-8
UTF-8 with BOM
UTF-16 Big Endian
UTF-16 Little Endian
ISO-8859-1

Currently "foreign" (non-standard 7-bit) characters such as äöü are not supported directly if the file is encoded in an ANSI codepage format such as ISO-8859-1. I added a test case for that as well which currently fails.
In order to support ANSI encoded files some changes have to be made to the framework, i.e. the IUpdateSource::GetUpdatesFeed() method has got to return a Stream instead of a string for example. This would allow the Xml Reader for checking the XmlDescription and its encoding attribute. I might propose a fix for that as well some time in the future while I'm at it.

Hope this fix gets integrated asap, as it well known for more than 5 years now:
https://groups.google.com/forum/#!topic/nappupdate/Irr2cIch5bQ

Thank you @sonntam ! looks good, merging now!


Card with glowing border added

fixes #173

@neha030 your PR is  showing changes in water drop effect please check it


Add ExtractResponse method to Azure.Core's HttpMessage in Azure.Core/ClientModel integration branch

Add the work from https://github.com/Azure/azure-sdk-for-net/pull/41764 to the Azure.Core 2.0 feature branch.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core


feat(single-page): add xml to static files regexp

Hello!
I noticed that xml files are not recognized as static by the single-page construct.
Therefore they are redirected to index.html.
This PR fixes this issue.
Thanks a lot!

Thanks!


windowsservercore-1709 with aspnet 4.7.2 crashes when starting container

I just deployed an ApiRest and a Web into mcr.microsoft.com/dotnet/framework/aspnet   4.7.2-windowsservercore-1709 image and running it perfectly. When I stop the container and try to start it again it crashes and exits with code<PHONE_NUMBER>.

RUN Install-WindowsFeature -name Web-Server -IncludeManagementTools
RUN Dism /online /enable-feature /featurename:IIS-ManagementService /all
RUN New-ItemProperty -Path HKLM:\software\microsoft\WebManagement\Server -Name EnableRemoteManagement -Value 1 -Force
RUN net start wmsvc
EXPOSE 80
Docker version:
Client: Docker Engine - Community
Version:           18.09.0
API version:       1.39
Go version:        go1.10.4
Git commit:        4d60db4
Built:             Wed Nov  7 00:47:51 2018
OS/Arch:           windows/amd64
Experimental:      false
Server: Docker Engine - Community
Engine:
Version:          18.09.0
API version:      1.39 (minimum version 1.24)
Go version:       go1.10.4
Git commit:       4d60db4
Built:            Wed Nov  7 00:56:41 2018
OS/Arch:          windows/amd64
Experimental:     false

This is what I get when trying to exec a powershell: (cause the container stays up for a few seconds)
Service 'w3svc' has been stopped
APPCMD failed with error code 4312


CRUD Comments

Users can create comments on any post
Users can view all comments on all posts
Users can update their comments
Users can delete their comments

pushed to feature/routes


unable to implement in react native 0.67

Facing below error when i tried to implement it, Can anyone help us to sort it out?
error: RangeError: Maximum call stack size exceeded
at RegExp.exec ()
at tokenize (/Users/comp/work/mobile-app/react-mob-app/node_modules/@babel/highlight/lib/index.js:67:38)
at tokenize.next ()
at highlightTokens (/Users/comp/work/mobile-app/react-mob-app/node_modules/@babel/highlight/lib/index.js:84:3)
at highlight (/Users/comp/work/mobile-app/react-mob-app/node_modules/@babel/highlight/lib/index.js:112:12)
at codeFrameColumns (/Users/comp/work/mobile-app/react-mob-app/node_modules/@babel/code-frame/lib/index.js:103:65)
at UnableToResolveError.buildCodeFrameMessage (/Users/comp/work/mobile-app/react-mob-app/node_modules/@react-native-community/cli-plugin-metro/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:264:12)
at new UnableToResolveError (/Users/comp/work/mobile-app/react-mob-app/node_modules/@react-native-community/cli-plugin-metro/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:223:35)
at ModuleResolver.resolveDependency (/Users/comp/work/mobile-app/react-mob-app/node_modules/@react-native-community/cli-plugin-metro/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:129:15)
at DependencyGraph.resolveDependency (/Users/comp/work/mobile-app/react-mob-app/node_modules/@react-native-community/cli-plugin-metro/node_modules/metro/src/node-haste/DependencyGraph.js:288:43)

I get the same error. I've attempted a few things: none of which have helped.

switched from trying it in an Expo-based project to a normal ReactNative project.
Tried only importing the methods I need (starting with import {imread} from '@techstark/opencv-js';) and even just including that single function causes the same problem.

I couldn't resolve that, else we can integrate OpenCV 3.4.1 as a native plugin which causes more weightage of the app but no option.

error: RangeError: Maximum call stack size exceeded
I get this error too on react-native 0.71.8 and expo ~48.0.15.
Any solutions?

I have the same issue

Is this solved after 2 years? Getting the same error using

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I have the same issue and idea for the fix: can it be solved by exposing all the functions in separate files (eg. /src/*) to the package?
To have the ability to import them separately instead of 1 large file.


Turtle not drawing a line

Tried a simple piece of code in Jupyter hub. The turtle moves but does not draw a line. Any ideas?
`from ipyturtle import Turtle
Bill = Turtle()
Bill.pencolor("Red")
Bill.pendown()
Bill.forward(100)
Bill.right(90)
Bill.forward(100)
Bill
#print (Bill.isdown())`

First this 2 lines in one cell
Bill = Turtle()
Bill
than all other commands
After the first two lines the tutrtle exists and the communication between JavaScript and Python works


[SPARK-14069][SQL] Improve SparkStatusTracker to also track executor information

What changes were proposed in this pull request?
Track executor information like host and port, cache size, running tasks.
TODO: tests
How was this patch tested?
N/A

cc @rxin

Test build #53775 has started for PR 11888 at commit f76de44.

Test build #53775 has finished for PR 11888 at commit f76de44.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53775/
Test FAILed.

Merged build finished. Test FAILed.

Test build #53884 has started for PR 11888 at commit fe80390.

I found it's difficulty to write tests for it. As it just collects the informations which are already exposed by the system, is it worth to test them again? cc @rxin

It's probably ok.

Test build #53884 has finished for PR 11888 at commit fe80390.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53884/
Test FAILed.

retest this please.

Test build #53904 has started for PR 11888 at commit fe80390.

Test build #53904 has finished for PR 11888 at commit fe80390.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53904/
Test FAILed.

Merged build finished. Test FAILed.

retest this please.

Test build #53921 has started for PR 11888 at commit fe80390.

Test build #53921 has finished for PR 11888 at commit fe80390.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53921/
Test FAILed.

retest this please

Test build #53925 has started for PR 11888 at commit fe80390.

Test build #53925 has finished for PR 11888 at commit fe80390.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53925/
Test PASSed.

Test build #54526 has started for PR 11888 at commit 0b9400e.

Test build #54526 has finished for PR 11888 at commit 0b9400e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54526/
Test PASSed.

@rxin I don't understand the distinction between getExecutors and getExecutorList. Why not just be more explicit about what the strings represent?
The other problem is the keys to the other maps are also expected to be host:port but I would normally expect them to be executor IDs. Also host can be confusing because it can be in one of many different formats (e.g. IP addr, all the hostname formats in EC2).

My proposal:
def getExecutorIds(): Array[String]
def getExecutorInfo(executorId: String): Option[SparkExecutorInfo]

private class SparkExecutorInfoImpl(
    val id: String,
    val hostport: String, // host:port
    ...
    val cacheSize: Long,
    val numRunningTasks: Int)
 extends SparkExecutorInfo

This is more consistent with the existing status API, where we have things like
def getActiveJobIds(): Array[Int]
def getJobInfo(jobId: Int): Option[SparkJobInfo]

scala> df.count()
16/03/30 15:21:58 INFO SparkContext: Starting job: count at <console>:27
...
16/03/30 15:21:58 INFO SparkDeploySchedulerBackend: Launching task 7 on executor id: 3 hostname: <IP_ADDRESS>.
16/03/30 15:21:58 INFO SparkDeploySchedulerBackend: Launching task 8 on executor id: 0 hostname: <IP_ADDRESS>.
16/03/30 15:21:58 INFO SparkDeploySchedulerBackend: Launching task 9 on executor id: 5 hostname: <IP_ADDRESS>.
16/03/30 15:21:58 INFO SparkDeploySchedulerBackend: Launching task 10 on executor id: 1 hostname: <IP_ADDRESS>.
16/03/30 15:21:58 INFO SparkDeploySchedulerBackend: Launching task 11 on executor id: 4 hostname: <IP_ADDRESS>.
16/03/30 15:21:58 INFO SparkDeploySchedulerBackend: Launching task 12 on executor id: 2 hostname: <IP_ADDRESS>.

Yea that integer id is completely useless to users who want to figure out what to do with their clusters.

OK, @rxin and I discussed this more offline. Our proposal is:
def getExecutorInfos: Seq[SparkExecutorInfo]

private class SparkExecutorInfoImpl(
    val host: String,
    val port: Int,
    ...
    val cacheSize: Long,
    val numRunningTasks: Int)
 extends SparkExecutorInfo

Then we don't need to tie us down with the very specific host:port format, and we don't have to expose the executor IDs, which are just integers that don't mean much.

Test build #54577 has started for PR 11888 at commit 39dd0ee.

Test build #54577 has finished for PR 11888 at commit 39dd0ee.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54577/
Test FAILed.

Merged build finished. Test FAILed.

LGTM retest this please

Test build #54588 has started for PR 11888 at commit 39dd0ee.

Test build #54588 has finished for PR 11888 at commit 39dd0ee.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54588/
Test PASSed.

Merged into master thanks guys.

This looks somewhat dodgy to me from a thread-safety perspective since executorIdToTaskCount isn't thread-safe and thus we may iterate over it in SparkStatusTracker while also concurrently updating it while processing task updates. I'm going to fix this in a followup but wanted to point it out here since ideally this should not have slipped past code review.

Why did we merge this when the description says "N/A"?

@rxin Do you mean the N/A in "How was this patch tested?"  Some guy said that the lack of tests was ok.  https://github.com/apache/spark/pull/11888#issuecomment-200162987

Yea but "TODO: tests" and tests: N/A ...
We needed to update the description.


include git commit hash and date in version output (-V)

:pushpin: Commit ba33f3c has been approved by japaric

@homunkulus r+

:hourglass: Testing commit ba33f3c with merge ba33f3c...

:sunny: Test successful - status-appveyor, status-travis
Approved by: japaric
Pushing ba33f3c3b9fee3270315e1a249471ac407a3e866 to master...


Uncaught Error: Cannot call function 'emit' on missing remote object 27Error: Cannot call functi...

[Enter steps to reproduce:]

...
...

Atom: 1.33.0 x64
Electron: 2.0.11
OS: Microsoft Windows 10 Pro
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot call function 'emit' on missing remote object 27
Error: Cannot call function 'emit' on missing remote object 27
at throwRPCError (C:\Users\ashid\AppData\Local\atom\app-1.33.0\resources\electron.asar\browser\rpc-server.js:143:17)
at EventEmitter. (C:\Users\ashid\AppData\Local\atom\app-1.33.0\resources\electron.asar\browser\rpc-server.js:354:7)
at emitMany (events.js:147:13)
at EventEmitter.emit (events.js:224:7)
at WebContents. (C:\Users\ashid\AppData\Local\atom\app-1.33.0\resources\electron.asar\browser\api\web-contents.js:295:13)
at emitTwo (events.js:126:13)
at WebContents.emit (events.js:214:7)
At C:\Users\ashid\AppData\Local\atom\app-1.33.0\resources\electron.asar\renderer\api\remote.js:221

Error: Cannot call function 'emit' on missing remote object 27
Error: Cannot call function 'emit' on missing remote object 27
    at throwRPCError (~/AppData/Local/atom/app-1.33.0/resources/electron.asar/browser/rpc-server.js:143:17)
    at EventEmitter.<anonymous> (~/AppData/Local/atom/app-1.33.0/resources/electron.asar/browser/rpc-server.js:354:7)
    at emitMany (events.js:147:13)
    at EventEmitter.emit (events.js:224:7)
    at WebContents.<anonymous> (~/AppData/Local/atom/app-1.33.0/resources/electron.asar/browser/api/web-contents.js:295:13)
    at emitTwo (events.js:126:13)
    at WebContents.emit (events.js:214:7)

    at Object.exception (~/AppData/Local/atom/app-1.33.0/resources/electron.asar/renderer/api/remote.js:221:30)
    at metaToValue (~/AppData/Local/atom/app-1.33.0/resources/electron.asar/renderer/api/remote.js:225:28)
    at Object.remoteMemberFunction (~/AppData/Local/atom/app-1.33.0/resources/electron.asar/renderer/api/remote.js:125:16)
    at ContextMenuManager.t.exports.ContextMenuManager.showForEvent (~/AppData/Local/atom/app-1.33.0/resources/app/static/<embedded>:11:400841)
    at WindowEventHandler.handleDocumentContextmenu (~/AppData/Local/atom/app-1.33.0/resources/app/static/<embedded>:11:287614)

Commands
     -6:18.6.0 editor:move-to-end-of-screen-line (input.hidden-input)
  6x -6:13.9.0 core:move-left (input.hidden-input)
  5x -6:11 core:move-right (input.hidden-input)
     -5:42.9.0 editor:newline (input.hidden-input)
     -5:41.1.0 core:backspace (input.hidden-input)
     -5:32.4.0 script:run (input.hidden-input)
     -5:07.3.0 core:move-to-bottom (input.hidden-input)
     -4:55 script:close-view (atom-workspace.workspace.scrollbars-visible-always.theme-atom-dark-syntax.theme-atom-dark-ui)
     -4:36.8.0 editor:newline (input.hidden-input)
     -4:36.6.0 core:backspace (input.hidden-input)
  2x -4:35.6.0 editor:newline (input.hidden-input)
     -4:24.7.0 core:save (input.hidden-input)
     -4:23.7.0 script:run (input.hidden-input)
     -4:03.2.0 core:backspace (input.hidden-input)
     -3:34.7.0 script:run (input.hidden-input)
     -0:09.2.0 encoding-selector:show (atom-text-editor.editor.is-focused)

Non-Core Packages
atom-simplified-chinese-menu 0.6.5
git-plus 8.1.0
script 3.18.1
simplified-chinese-menu 5.3.5

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/16630 so you can subscribe there for updates if you'd like (we believe that this will be fixed in a future Electron upgrade).

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Introduce RoundDown helper to assist in buying max coins when precision is not available

Create the beginnings of a unit test project

Thanks for putting this together. Probably about time we had a unit test project! :)


testing bundler smoke tests

I think the caching is not right, testing with this Draft PR.

It's good!