How to set the flutter sdk

I set the path ~/mypath/flutter/bin,but the atom tell me "No SDK at ~/mypath/flutter/bin". Please help me,thanks!

I use mac.

You should set it to the main flutter directory - the parent of bin/. Cheers!


Feature/error monitor

What's the advantage of having runAppM return a AppResult? Why cant the exception handler be in runWithLogging directly? Will this remove the need for the third argument to runWithLogging as well?


windows: exec.Cmd.Wait returns an error when the process exits without reading stdin

What version of Go are you using (go version)?
go 1.8.1
What operating system and processor architecture are you using (go env)?
windows/amd64
What did you do?
package main

import (
	"fmt"
	"os/exec"
	"runtime"
)

type infiniteReader struct{}

func (r infiniteReader) Read(b []byte) (int, error) {
	// If this is uncommented, it fails with ERROR_NO_DATA instead of ERROR_BROKEN_PIPE
	//time.Sleep(time.Second)
	for i := range b {
		b[i] = '\n'
	}
	return len(b), nil
}

func main() {
	var args []string
	if runtime.GOOS == "windows" {
		args = []string{"cmd", "/c", "echo test"}
	} else {
		args = []string{"sh", "-c", "true"}
	}
	c := exec.Command(args[0], args[1:]...)
	c.Stdin = infiniteReader{}

	err := c.Start()
	if err != nil {
		panic(err)
	}
	err = c.Wait()
	if err != nil {
		panic(fmt.Sprintf("%#v", err))
	}
}

What did you expect to see?
Program should exit without error.
What did you see instead?
On Linux, the program exits without error. On Windows, the program panics with:
panic: &os.PathError{Op:"write", Path:"|1", Err:0x6d}

goroutine 1 [running]:
main.main()
        C:/tmp/repro.go:36 +0x1d5
exit status 2

This occurs because the exec.skipStdinCopyError ignores EPIPE from Write, but for Windows, WriteFile, and thus os.File.Write, returns syscall.ERROR_BROKEN_PIPE instead of syscall.EPIPE when the other end of the pipe  is closed while the write was outstanding, and ERROR_NO_DATA (0xe8) when the other end of the pipe is already closed before the write was initiated.
I think the fix is just to specialize skipStdinCopyError for Windows to also consider these two extra error codes. The other alternative would be to change os.File.Write to translate ERROR_BROKEN_PIPE and ERROR_NO_DATA to EPIPE.

/cc @ianlancetaylor @alexbrainman

I think in an ideal world we would change the syscall package to set EPIPE to be ERROR_BROKEN_PIPE, but that would not be Go 1 compatible.  So I think we should change the os/exec package to check for ERROR_BROKEN_PIPE in skipStdinCopyError on Windows.

That makes sense. To reiterate, though, I think we need to check for both ERROR_BROKEN_PIPE and ERROR_NO_DATA, as Windows distinguishes between these two cases that probably should not be distinguished. In almost all cases, the specific error you get is timing-dependent and not under the caller's control.

We even have TestIgnorePipeErrorOnSuccess for this. But the test is skipped on windows.
I will try to fix this.
Alex


what paper method/ algorithm is used for sentiment analysis?

what paper method/ algorithm is used for sentiment analysis?
how good is it, what performance are?
Thanks

The default sentiment analyzer just delegates to pattern.en, which uses a dictionary-based algorithm: https://www.clips.uantwerpen.be/pages/pattern-en#sentiment . According to the docs, the accuracy is about 75% for movie reviews.

For info, the webpage does not exist anymore, but I guess it is about this: https://github.com/clips/pattern

For info, the webpage does not exist anymore, but I guess it is about this: https://github.com/clips/pattern


Not working

its not working with METEOR, don't know what's the issue is..

any further information like, what you have tried, how do you include it and so on? if not i will close this issue immediately. ;)

it was working earlier, but suddenly we don't know what happened its not working in our production environment, but working in our local

dude... more infos... which ngquill/quill version are you using local and which in production.
Maybe you are using quill 0.2.x locally and on production you installed 2.x or something like that.

using  2.1.0 at both the places

Maaaan give me some insights...
JS errors on production... if .. show me the error messages or something.
If it works locally .. you did something wrong on the production system... so not my fault...
I give you a last chance to provide some relevant data... if not: Good luck to fix your prod system ;)

15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:59 TypeError: this.observer.takeRecords is not a function
at t.optimize (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:13)
at t.y (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:19)
at new t (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:19)
at Object.n [as create] (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:13)
at new e (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:13)
at n.component.controller.$postLink (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:98)
at 15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:58
at i (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:57)
at h (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:58)
at a (15cf0f1b540434bac1f0a117b554c9eb7fd3140c.js:58)

This is the error i am getting in my production env

congratulation ... an error message of an bundled and hopefully minified js-file *applause
Maybe your buildprocess destroys something.
If you run the project locally you are using the not bundled/minified scripts?

I DONT HAVE ANY FURTHER INFORMATION REGARDING THIS

yeah then sorry... i do not know anything about your prod system or your buildprocess... and maybe someone should open issues ans ask question, who has those information...
bye

@KillerCodeMonkey We are using ng-quill 2.1.0 in our meteor app.
And the error we are getting is as below. This is the only error I can show you now.

In the UI we should see the text editor and that's not coming. Please see below screenshot.

Hope this information can help you figure out something about the issue and help here.

Could you paste the source code of the controller/component?

@KillerCodeMonkey
They have to answer from India, So it will become in all probability tomorrow morning.
I apologize for the way in how the issue was reported.
Thank you for your time

Hi @KillerCodeMonkey
I am opening a new thread with more details on the issue.
Thank You.


Overly big pre-built framework

I've noticed that the pre-built framework is about 50MB uncompressed, 19MB compressed, and it's because the multiple versions within are actual copies, not symbolic links.

yes you are right! So sorry. Its because of not being symbolically linked... I've fixed that in the prebuild files now. Thanks for the issue! Can you kindly confirm all works?

Yeah, it's correct now, thanks!

:+1:


Cobra Issue: On IOS there is a constant error "WARN  Sending frame_sent with no listeners registered." (BTW and on Android it works fine).

Have you checked the docs and existing issues?

[X] I have read all of the relevant Picovoice Cobra docs
[X] I have searched the existing issues for Cobra

SDK
iOS
Cobra package version
2.0.1
Framework version
EXPO CLI 0.17.8
Platform
iOS
OS/Browser version
N/A
Describe the bug
I use your example of Cobra for IOS . It works however I keep getting endless warning messages:
"WARN  Sending frame_sent with no listeners registered."
On android it works fine.
Steps To Reproduce
It is reproducible on my environment. On simulator and real devices. Just run the IOS example and watch terminal console for messages.
Expected Behavior
Not to receive the warning messages ""WARN  Sending frame_sent with no listeners registered." or a better message that explains where this is in the stack etc'

Could you give our iOS Demo a try and see if you get the same issue?

Can you send me exact instruction. The IOS demo starts with pod install - what do you want me to download before that?

Hi All - I managed to find my problem and solve it.
This happens when the app closes or crashes without properly stop Cobra and then I start Cobra again.
If I make sure I stop / delete everything than all works fine. Another alternative is to call stop Cobra before you start it to make sure it was not left open from another run.
Thanks,
Ofer


fix bug: read port information from the configuration file.

🔍 What type of PR is this?

👀 What this PR does / why we need it:

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

🅰 Which issue(s) this PR fixes:

Fixes #365
📝 Special notes for your reviewer:

🎯 Describe how to verify it

📑 Additional documentation e.g., RFC, notion, Google docs, usage docs, etc.:

I have read the CLA Document and I hereby sign the CLA


Is using Path() params in a Depends() ok?

I want to validate a token based on what path is being accessed. I don't see anything similar in the documentation. Below is a simplified working example of the situation. My question is: Is this a valid usage, or is it abusing the fastapi API?
from fastapi import Depends, FastAPI, Header, HTTPException, Path, status

app = FastAPI()

auth_table = {"me": "whee43", "you": "whaa32", "Hermoine": "SPEW4life"}

def user_validator(user: str = Path(...), x_token: str = Header(...)):
    if user in auth_table and auth_table[user] == x_token:
        return user
    raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED)

@app.get("/users/{user}")
async def get_user(valid_user=Depends(user_validator)):
    return {"user": valid_user}

Yes, it is valid usage especially if you want to use Path(...) for validation, documentation, etc.

Thanks for the help here @SnkSynthesis ! 👏  🙇
And yep, that's fine! :heavy_check_mark:
Thanks for reporting back and closing the issue @vdwees  👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


Development

Adds MicrosoftVdot
Adds parameter -UserAgent to Test-EvergreenApp
Updates Pester tests - splits tests into seperate files aligned to public and private functions; adds additional tests via review of code coverage reports: https://app.codecov.io/gh/aaronparker/evergreen

Codecov Report
Base: 80.14% // Head: 0.80% // Decreases project coverage by -79.33% :warning:

Coverage data is based on head (5993ed9) compared to base (e6d259b).
Patch coverage: 8.62% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #443       +/-   ##
==========================================
- Coverage   80.14%   0.80%   -79.34%
==========================================
  Files         282     284        +2
  Lines        4065    4079       +14
==========================================
- Hits         3258      33     -3225
- Misses        807    4046     +3239

Impacted Files
Coverage Δ

Evergreen/Apps/Get-MicrosoftVdot.ps1
0.00% <0.00%> (ø)

Evergreen/Apps/Get-NanaZip.ps1
0.00% <0.00%> (ø)

Evergreen/Private/Expand-GzipArchive.ps1
0.00% <0.00%> (-89.29%)
:arrow_down:

Evergreen/Private/Get-Platform.ps1
0.00% <0.00%> (-53.85%)
:arrow_down:

Evergreen/Private/Remove-ProxyEnv.ps1
0.00% <ø> (ø)

Evergreen/Public/Export-EvergreenManifest.ps1
0.00% <0.00%> (-53.85%)
:arrow_down:

Evergreen/Public/Find-EvergreenApp.ps1
0.00% <0.00%> (-79.32%)
:arrow_down:

Evergreen/Public/Save-EvergreenApp.ps1
0.00% <0.00%> (-73.78%)
:arrow_down:

Evergreen/Public/Test-EvergreenApp.ps1
0.00% <0.00%> (-80.77%)
:arrow_down:

Evergreen/Private/ConvertTo-Hashtable.ps1
92.30% <83.33%> (ø)

... and 262 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Uploaded pufvars.csv

This has probably been done 100 times, but maybe this will save someone else from creating version 101.
I created pufvars.csv with puf2011 variable names and the SOI descriptions, copied from the 2011 public use booklet. The file is in the data directory of synpuf.
If there is a better way to do things like this via github, please let me know (@MaxGhenis, @andersonfrailey).

This is at https://github.com/donboyd5/synpuf/blob/master/data/pufvars.csv
I also put it in Google Sheets here.


Add align center to figure class #80

Adds an align center style to all figure elements. Adding align-left or align-right will override default behaviour.

thanks @DrDrij let's give this a go. I'll merge this now.

@DrDrij this is the example deployment.
https://6041b9a1970aba6b9b9f50df--hungry-lovelace-7d0512.netlify.app
must not have run on your PR as it came from a fork (permissions are all a bit different for github actions)


Add support for SQL parsers

An SQL parser is too complex to be a part of this crate.  However, we should create an extension point for external SQL parsers (such as the one in datafusion) and add SQL parsing to the API.

DataFusion uses sqlparser-rs, one could probably that an AST from this crate as base


dont reload UI when socket reconnects

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[ ] Yes
[ ] No, because:

Have you updated all relevant documentation?

[ ] Yes
[ ] No

Description

do not reload UI when socket connects, in the event of back-end instability this makes for a very jarring UX
added an action for socket connection error for commercial analytics

Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings

Merge Plan

Added/updated tests?

[ ] Yes
[ ] No : please replace this line with details on why tests
have not been included

[optional] Are there any post deployment tasks we need to perform?

superseded by #5451


PushbackInputStream.Position is wrong

The Position property of the PushbackInputStream class always returns the position of the underlying stream. This is wrong: the value of the underlying stream's position should be reduced with the number of pushed-back bytes.

This is not actually a bug, for the following two reasons.

If the underlying stream supports seeking then the pushback buffer is never used, and the current position is guaranteed to be the same as the underlying stream.
If the underlying stream does not support seeking then the documented behavior for Stream.Position is that it should throw System.NotSupportedException.

The only chance that this code could break is when the underlying stream does not correctly throw NotSupportException as documented. This would be a bug in the underlying stream class, not the PushbackInputStream class.

It might be a good idea for PushbackInputStream.Position to implement the documented behavior instead of relying on the implementation of the underlying stream.


OpenJDK11 / AIX PPC64 : String.indexOf generates incorrect result after some time

Platform:
AIX 7.1
Architecture:
POWERPC
Please describe the issue that you are having with our binaries and if possible how to recreate it.
The test StringIndexOfBug listed below fails the String.indexOf after some time on ppc64 JDK11.
Excluding java.lang.String.indexOf from compilation makes the test successful.
public class StringIndexOfBug {

   public static final int MAX_ITERATION = 1000000;
   public static final String NON_ASCII_CHAR = "µ";
   public static final String TEST_STRING = "a string with a non-ascii character : " + NON_ASCII_CHAR;

   public static final void main(String[] args) {

      for(int i=0; i< MAX_ITERATION ; i++) {
         if(TEST_STRING.indexOf(NON_ASCII_CHAR) == -1) {
            System.out.println("ERROR: Unexpected -1 result for String.indexOf() at iteration #" +  i);
            System.exit(1);
         }
      }
      System.out.println("SUCCESS: No error found for " + MAX_ITERATION + " iterations");
   }
}

Here is the following results with default JIT compilation :
    $ java StringIndexOfBug
    ERROR: Unexpected -1 result for String.indexOf() at iteration #277904
    $ java StringIndexOfBug
    ERROR: Unexpected -1 result for String.indexOf() at iteration #334846
    $ java StringIndexOfBug
    ERROR: Unexpected -1 result for String.indexOf() at iteration #195645

With forced JIT compilation :
    $ java -Xcomp StringIndexOfBug
    ERROR: Unexpected -1 result for String.indexOf() at iteration #41982

Without JIT compilation for String.indexOf() :
    $ java -XX:CompileCommand=include,java/lang/String.indexOf StringIndexOfBug
    CompileCommand: exclude java/lang/String.indexOf
    SUCCESS: No error found for 1000000 iterations

Can you give us the output of java --version?

$ java --version
openjdk 11.0.3 2019-04-16
OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.3+7)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.3+7, mixed mode)

@regisamis Can you try the 11.0.4 build?  We may possibly have the 11.0.5 build out as well.

We have the same behavior with 11.0.4 version :
$ java --version
openjdk 11.0.4 2019-07-16
OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.4+11)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 11.0.4+11, mixed mode)

$ java StringIndexOfBug
ERROR: Unexpected -1 result for String.indexOf() at iteration #192993

@regisamis We still need to get 11.0.5 out, but hopefully that will fix this issue, stay tuned!

@regisamis Can you try with one of our later binaries and see if the issue still occurs?


Remove WebActionCaller

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
✔

Tests pass?
✔

Did new commit because of merge conflicts...


Reference page not correctly displaying article title.

The reference page http://amigo.geneontology.org/amigo/reference/PMID:30352852 displays the article title as '[object Object]' rather than 'Evolutionarily conserved Tbx5- Wnt2/2b pathway orchestrates cardiopulmonary development'.

@malcolmfisher103 Huh, nice catch. Let me look into that.

It seems like it might be due to the embedded markup in the title.
Doesn't work:
<ArticleTitle>Evolutionarily conserved Tbx5-Wnt2/2b pathway orchestrates cardiopulmonary development.</ArticleTitle>
Works:
<ArticleTitle>Adrenocortical function in 4-APP treated rats: a coupled stereological and biochemical study.</ArticleTitle>
Digging into the JS a little:
jxon = require('jxon');
str = '<PubmedArticleSet><PubmedArticle><MedlineCitation Status="MEDLINE" Owner="NLM"><PMID Version="1">30352852</PMID><DateCompleted><Year>2019</Year><Month>05</Month><Day>22</Day></DateCompleted><DateRevised><Year>2019</Year><Month>05</Month><Day>22</Day></DateRevised>
<Journal><ISSN IssnType="Electronic">1091-6490</ISSN><JournalIssue CitedMedium="Internet"><Volume>115</Volume><Issue>45</Issue><PubDate><Year>2018</Year><Month>11</Month><Day>06</Day></PubDate></JournalIssue><Title>Proceedings of the National Academy of Sciences of the United States of America</Title><ISOAbbreviation>Proc Natl Acad Sci U S A</ISOAbbreviation></Journal><ArticleTitle>Evolutionarily conserved Tbx5-Wnt2/2b pathway orchestrates cardiopulmonary development.</ArticleTitle></MedlineCitation></PubmedArticle></PubmedArticleSet>';
j = jxon.stringToJs(str);
j.PubmedArticleSet.PubmedArticle.MedlineCitation.Article.ArticleTitle

Returns:
{ i: [ 'Tbx5', 'Wnt2/2b' ],
  _: 'Evolutionarily conserved-pathway orchestrates cardiopulmonary development.' }

from string: Evolutionarily conserved Tbx5-Wnt2/2b pathway orchestrates cardiopulmonary development.
This means that the jxon transformation gives no way to recover the correct title. I wonder if the embedded HTML is intended or not and if this might be an upstream issue?
Either way, I suspect this is limited to titles with embedded features and if maybe just the _ object portion could be used as a "good enough" for situations like this. Alternatively, it may be possible to just string scan and grep out the title from the original string in the error case that an object is returned.

This seems intentional , see screenshot

Can we not skip these html elements?

@pgaudet Correct, from the examples above. They can be removed, but the JSON, as it stands, does not allow for a correct title to be extracted. We'd have to crack open the hood and have a fall-through case (or otherwise switch) to scanning the returned XML some other way.
The main point here is that it is not completely trivial to fix, but fairly easy.

Randomly dove down this rabbit hole and thought I should post what I found. Pubmed's database+api looks to be a little less feature rich than Pubmed Central's and so there's some interesting data storage formats here and some response types which aren't fully supported such as a JSON response that contains the AbstractText data (which is also affected in this particular article's case). There's not necessarily a quick fix available off the shelf but there are a couple of options:

Switch to requesting response in the rettype=medline format. This will provide the results in a text file which is formatted in a way where all of the currently used fields can be parsed and wherein there are no embedded html tags in the response. The response for this specific problematic article can be viewed here: https://eutils.ncbi.nlm.nih.gov/entrez/eutils/efetch.fcgi?db=pubmed&id=30352852&rettype=medline
The major drawback I can see of this response type is the difference in author name formatting where this return type provides author names as "Last, First (Middle)" format as opposed to the two separate fields currently parsed from the XML. Not a huge issue on its own as the string can be split, results reversed, and rejoined but this does expose the result to potential unforeseen corner cases.

Pull title and abstract text from the medline format above, and the other formats from the xml format currently requested. This produces potentially the least amount of churn as far as results goes, but doubles up all API requests. From a responsible API user perspective it might be nice to avoid doubling up requests if possible for load on pubmed's servers as well as latency on the page load.

Switch from using JQuery to create the text nodes and instead first creating a vanilla JS textnode via document.createTextNode(title). This will parse the title as a plain text even when this textnode is then appended via JQuery. As a result the html tags will appear as plaintext in the resulting title. Not the prettiest result but the easiest to implement and presents the fewest potential bugs.

Iterate through the xml results for both title and abstract, searching for each closing tag and then removing each corresponding opening tag. Assuming there's no additional attributes on the html tags provided then this is not going to be super bug prone, but even in that best case scenario there's likely to eventually end up being some corner case that gets mangled. If there are attributes on the included tags then this becomes even less fool proof to work 100% of the time.

Got any opinion on one of these options or an alternative? Want assistance?

Thank you @AlexanderNull for taking a bit of a look at this.
Druthers would not to have custom medline parsing (1 and 2). The same for 4.
I suspect the right answer is in the neighborhood of your "3": dig out the area around title by some method, maybe jxon still, and then convert it to a string without jxon's (frankly) incorrect method. I think the hack here was trying to do this with old in-browser libs (as jxon is rather out-of-date now).

Even without changing the rest client we could take the raw results and instead of passing them to jxon, pass them to the built in DOMParser (the library that caused all JXON support to be dropped from official specs). Using something like this around https://github.com/geneontology/amigo/blob/386c405edabbdb23a77adf1932344128e15e733f/javascript/web/ReferenceDetails.js#L279:
var parser = new DOMParser();
var xmlDoc = parser.parseFromString(xml);

// omitting various if checks for example brevity
var op = xmlDoc.querySelector('PubmedArticleSet PubmedArticle MedlineCitation Article');

// skipping over some more stuff
var title = op.querySelector('ArticleTitle')?.textContent || 'n/a';
var abstract = op.querySelector('Abstract')?.textContent || 'n/a';

document.querySelector has support all the way back to Chrome V1 so that shouldn't present any issues. This approach actually provides a double benefit as the document object contents can either be accessed via the innerHTML prop or as done in my example the textContent prop, the later of which automatically strips the embedded html tags from the results!
What do you think about this approach? I don't have all of AmiGO running locally yet but it seems like a pretty straightforward switch and the added sanitization is nice.

@AlexanderNull That sounds like a good strategy: bypassing the faulty jxon after the REST call. If you can give me a little more context for the code or a mock PR, I can get it in and test if NP from my end. AmiGO can be tricky to build and work with these days (although some fixes there imminent).


Migrate tests to use bluesky RunEngineSimulator

Once bluesky/bluesky#1638 is merged, we should migrate the existing unit tests away from the old version in our conftest and remove it.

bluesky alpha release has been made so this should be possible now
fairly mechanical

we say medium


Fix multiple options for latexmk

The comments on #3 made me realise that user_options still wasn't working as expected on latexmk. Basically with multiple options is was getting a command line like:
latexmk -cd -e -f -pdf -interaction=nonstopmode -synctex=1 -latexoption="first_option",-latexoption="second_option" file.tex
While the options showed up, they weren't being honoured. Digging into it, it appeared that options was being set to ["-cd", "-e", "-f", "-#{user_program}", "-interaction=nonstopmode", "-synctex=1", ["-latexmk=\"first_option\"", "-latexmk=\"second_option\""]], hence the bad behaviour. This PR fixes that.

Good catch, thanks!


To compile with qt5.15

Hello,
The qt5.15 is the current default release you get with brew.
But it has compilation issue with vtk.
The sol. could be to patch VTK. (one header to add to 2 files). (tested it works)
vtkQtLabelRenderStrategy
vtkQtStringToImage

The other solution is to use official Qt installer.
It's tested by @juliencastelneau

Julien has choiced Qt installer because he got the compile issues and can not choose an older qt release via brew so he took the qt installer.
What I propose is to solve the compile issues to be able to use the last  lts qt release.  the stepping stone towards qt 6. https://www.qt.io/blog/qt-5.15-released.
the way dep. are installed is not changed : via brew. (the apt for mac os).
I do not wait for VTK to do the job and so let them choice which qt release I can use.
Patch are good temp sol (other softwares has solved the compile issues with the same sol)

Agree with Nordine.
Maybe we could fork instead of patch VTK ?

Le 20 juil. 2020 à 11:54, nebatmusic<EMAIL_ADDRESS>a écrit :
Julien has choiced Qt installer because he got the compile issues and can not choose an older qt release via brew so he took the qt installer.
What I propose is to solve the compile issues to be able to use the last lts qt release. the stepping stone towards qt 6. https://www.qt.io/blog/qt-5.15-released https://www.qt.io/blog/qt-5.15-released.
the way dep. are installed is not changed : via brew. (the apt for mac os).
I do not wait for VTK to do the job and so let them choice which qt release I can use.
Patch are good temp sol (other softwares has solved the compile issues with the same sol)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/medInria/medInria-public/issues/756#issuecomment-660927784, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF2G3XFBPM2RXHI3H33EQ2DR4QH4DANCNFSM4O2MP5AQ.

We used to patch VTK. and since it is really a minor change it is not a good reason to  "fork" @juliencastelneau (maybe you have other reasons)
The pb is solved on master and maybe vtk 9.0 (https://gitlab.kitware.com/vtk/vtk/-/issues/17927, https://gitlab.kitware.com/vtk/vtk/-/issues/17921).
We are know using v8.1.2.

We have a way to apply our custom patches to external repos we use. But this can be a bit painful sometimes. And VTK 9 probably has the patch already included so I would rather go for moving to VTK 9 once we are sure there is no other problem coming with it in visualization

This patch is minor (to include one qt header into 2 files) and it will allow users to work with the last qt.
VTK.patch.txt


Add a test for PreStop hooks.

Add an e2e test for PreStop hooks.
Closes #7091

oops, this was trivially passing... needs some more work.  will re-open when ready...

Ok, this is now fixed, and passing because it's actually testing something, instead of just passing no matter what.
This is ready for review.
Thanks!
--brendan


Content changes for Vietnamese marriage outcomes

This supersedes #2721
Trello card
Description
This PR updates the content and links for Vietnamese opposite and same sex
marriages:

Removes the links for same sex marriages
Provide further options for proving legal residence and also provision of useful links to other services (same sex marriages).
Update Content and links for Opposite marriages in Vietnam

Factcheck Same sex marriage
Preview link
GOV.UK
Expected changes

Update content for same sex marriages except where partner is Vietnamese National

Before

After

Factcheck Opposite sex
Preview link
GOV.UK
Expected changes

Update Content and links for Opposite and same sex marriages in Vietnam

Before

After

LGTM


Does .backward() contain grads for both sequences?

Hi, first of all: Awesome repo! :) Thanks.
To the point: I would like to train an audio-phoneme aligner. I have two encoders.

Spectrogram encoder - returns as many embeddings as there are feature frames
Phoneme encoder - returns as many embeddings as there are phonemes

I want to align these two sequences with DTW and optimize both the SpecEncoder and PhonemeEncoder.
Is this possible out of the box? I looked in the code but I am not sure if the loss returns grads w.r.t to both input sequences.
Update: I just checked with a dummy code if the backward can be used to modify both sequences and it seems that both input sequences are changing:
import torch
from soft_dtw_cuda import SoftDTW

criterion = SoftDTW(use_cuda=False, gamma=0.1)
x = torch.randn(1, 5, 80).permute(0,2,1)
y = torch.randn(1, 5, 120).permute(0,2,1)
x.requires_grad = True
y.requires_grad = True

opt = torch.optim.Adam([x, y], lr=0.001)

for i in range(10):

    opt.zero_grad()
    loss = criterion(x, y)
    loss.backward()
    opt.step()
    print(y[0, 0])

The question remains. Is the gradient calculated correctly for both input sequences? Also do you think that this loss could be used to solve forced alignment tasks such as the one I described above? I will update you with how it is going later this week :)

Hi @janvainer, thanks for your interest!

I looked in the code but I am not sure if the loss returns grads w.r.t to both input sequences.

Yes, the code can compute the gradients w.r.t both inputs. Let me explain.
Take a look at _SoftDTWCUDA. This class takes the cost matrix D as the input to forward(), and computes the sDTW dissimilarity using that input. This class can also compute the gradient of the sDTW function, w.r.t D and returns it in backward().
Let's say D = dist_func(x, y). By calling sDTW on D, we are effectively computing the composite function loss = sDTW( dist_func (x, y)). Using the chain rule in derivatives, one can compute the derivative of loss w.r.t to both its inputs. This explains what you observed in your code snippet above.
In this code, we're relying on PyTorch's pipeline to apply the chain rule correctly, so overall the computation should be correct.

Also do you think that this loss could be used to solve forced alignment tasks such as the one I described above?

I'm a bit rusty when it comes to NLP, but if I understood your description correctly, what you intend to do should be possible.
Hope this helps!

Hi @janvainer, thanks for your interest!

I looked in the code but I am not sure if the loss returns grads w.r.t to both input sequences.

Yes, the code can compute the gradients w.r.t both inputs. Let me explain.
Take a look at _SoftDTWCUDA. This class takes the cost matrix D as the input to forward(), and computes the sDTW dissimilarity using that input. This class can also compute the gradient of the sDTW function, w.r.t D and returns it in backward().
Let's say D = dist_func(x, y). By calling sDTW on D, we are effectively computing the composite function loss = sDTW( dist_func (x, y)). Using the chain rule in derivatives, one can compute the derivative of loss w.r.t to both its inputs. This explains what you observed in your code snippet above.
In this code, we're relying on PyTorch's pipeline to apply the chain rule correctly, so overall the computation should be correct.

Also do you think that this loss could be used to solve forced alignment tasks such as the one I described above?

I'm a bit rusty when it comes to NLP, but if I understood your description correctly, what you intend to do should be possible.
Hope this helps!

Thanks! helped a lot :)


Webpack error related to mui on typescript project

I got the following error when installing 3.0.1 and using it on react 18.2.0 and typescript 4.7.4.
ERROR in<EMAIL_ADDRESS>3:0-40
Module not found: Error: Can't resolve '@emotion/react' in 'node_modules\@mui\styled-engine\GlobalStyles'

ERROR in<EMAIL_ADDRESS>3:0-47
Module not found: Error: Can't resolve '@emotion/react' in 'node_modules\@mui\styled-engine\StyledEngineProvider'

ERROR in<EMAIL_ADDRESS>6:0-39
Module not found: Error: Can't resolve '@emotion/styled' in 'node_modules\@mui\styled-engine'

ERROR in<EMAIL_ADDRESS>30:0-62
Module not found: Error: Can't resolve '@emotion/react' in 'node_modules\@mui\styled-engine'

ERROR in<EMAIL_ADDRESS>11:27-60
export 'ThemeContext' (imported as 'StyledEngineThemeContext') was not found in
'@mui/styled-engine' (possible exports: GlobalStyles, StyledEngineProvider, default)

ERROR in<EMAIL_ADDRESS>1:0-88
export 'css' (reexported as 'css') was not found in '@mui/styled-engine' (possible exports: GlobalStyles, StyledEngineProvider, default)

ERROR in<EMAIL_ADDRESS>1:0-88
export 'keyframes' (reexported as 'keyframes') was not found in '@mui/styled-engine' (possible exports: GlobalStyles, StyledEngineProvider, default)

ERROR in ./node_modules/react-collapse-pane/dist/react-collapse-pane.esm.js 2:0-48
Module not found: Error: Can't resolve 'styled-components' in 'node_modules\react-collapse-pane\dist'

webpack compiled with 8 errors and 1 warning

Installing the following fixed the errors:

npm i @emotion/react
npm i @emotion/styled
npm i styled-components


How was openAI 5M dataset generated ?

I have been using the OpenAI 5M dataset. I need the raw source dataset for it and came across https://huggingface.co/datasets/allenai/c4 as mentioned in the readme. However none of the datasets in allenai/c4 have 5M rows in the train dataset, and I found it hard to point the raw source data. Could you please help me with how the OpenAI 5M dataset was generated ? My best guess so far is it is a combination of en and en-multi train and validation datasets.
Thank you.

@zhuwenxing could you help explain the process we used to generate the OpenAI dataset?

We sampled 5 million data points from the dataset, but at the time, we only kept the embeddings and didn't preserve the correspondence between the text and embeddings, so we may not be able to provide you with the original text and corresponding embeddings.

Thanks @zhuwenxing
The hf dataset has many subsets mostly based on language or blocked words etc. Do you remember

Which data subset was used for sampling ? was it -en, -en.multi or something else ?
How was the query set created ? I am assuming it was also chosen from either train or validation data subset, but which ones ?
What kind of sampling was done ?

I apologize that I can't provide more definitive answers to these questions. When creating this dataset, I only worked with the embedding files, and the specifics of the data preparation process were not recorded or are not readily available to me now.
Your questions highlight valuable aspects of data provenance that we should capture in future work.


Web Support

Is your feature request related to a problem?
Support for web
Describe the solution you'd like
Describe alternatives you've considered
Camera Web Package
Additional context
Perhaps if needed, you can dumb things down to make it work on web?. What's holding it back?

Using the existing methods, the only thing you'll need to do is to update how the captured asset is generated. You can be more specific if you're running into any particular blockings.

@AlexV525 i missed your last comment. i'm uncertain how to update thow the asset is generated. you mean to modify the package? web support could be a great addition if it's not too much work. any tips? thx


marketplace

Issue Type: Feature Request
hello , I get this error when I see the market
I get this error when I see the market
we cannot connect to the extensions marketplace at this time please try again later

Got similar issue. On Console log says ->
{timestamp: "2020-06-13T00:53:21.571+0000", status: 500, error: "Internal Server Error",…}
error: "Internal Server Error"
message: "sortOrder parameter must be either 'asc' or 'desc'."
path: "/vscode/gallery/extensionquery"
status: 500
timestamp: "2020-06-13T00:53:21.571+0000"

Got similar issue. On Console log says ->
{timestamp: "2020-06-13T00:53:21.571+0000", status: 500, error: "Internal Server Error",…}
error: "Internal Server Error"
message: "sortOrder parameter must be either 'asc' or 'desc'."
path: "/vscode/gallery/extensionquery"
status: 500
timestamp: "2020-06-13T00:53:21.571+0000"

I have the same Issue this is the request payload
{"filters":[{"criteria":[{"filterType":8,"value":"Microsoft.VisualStudio.Code"},{"filterType":12,"value":"4096"}],"pageNumber":1,"pageSize":50,"sortBy":4,"sortOrder":0}],"assetTypes":[],"flags":946}

any solution?

Nevermind! it's related to vscodium https://github.com/VSCodium/vscodium/issues/418

Please follow these steps

Help -> Toggle Developer Tools -> Network
Then check for any network failed requests while browsing and installing the extensions if found all requests fails then its probably a proxy or DNS issue at your end.
To confirm its a DNS issue check if you can access this page https://ms-python.gallerycdn.vsassets.io/extensions/ms-python/python/2019.8.29288/1565127378610/Microsoft.VisualStudio.Services.Icons.Default, if you cannot please try configuring your DNS like pointing to <IP_ADDRESS> as suggested here? https://<IP_ADDRESS>/dns/. Otherwise please contact your network administrator.
If it is not a network issue, please share the logs from the following locations

Help -> Toggle Developer Tools -> Console
View -> Command Pallette... -> Toggle Shared Process -> Console
View -> Open View… -> Log (Shared)
View -> Open View… -> Log (Window)

I made the test of the page the photo appeared to me, when entering the console all these problems appear

Seems proxy issues

@feelipecl Please start VSCode with --log-net-log=netlog.json, try searching the marketplace and then upload the resulting netlog.json here.

@feelipecl When you start with --log-net-log=netlog.json there should be a netlog.json created. Could you upload that? (Maybe you have the http.proxy setting in VS Code set or one of the HTTP_PROXY or HTTPS_PROXY environment variables.)


Dynamic modulesNamespace definition

In addition to the helm configured modulesNamespace we should provide an option that Fybrik user will define in which namespace the modules will be deployed.
The possible solution is to create the module helm releases in the namespace specified by the app.fybrik.io/modules-namespace label.
The namespace is not managed by fybrik and should exist before.

fybrik-blueprints-role.yaml is currently been applied on the module namespace so with the approach above the role should be created not by fybrik unless the user will provide a list of possible namespaces on fybrik deployment time that he will for the modulesNamespace .

fybrik-blueprints-role.yaml is currently been applied on the module namespace so with the approach above the role should not be created by fybrik unless the user will provide a list of possible namespaces on fybrik deployment time that will be use for the modulesNamespace .

Only if not managed by Fybrik.

The idea is that namespaces are not managed by fybrik and it is not fybrik responsibility to provide relevant RBAC configurations, it should be done externally.

We have a managed flag in the values.yaml. When set to false, no roles should be deployed. When set to true, it is responsibility of Fybrik to provide RBAC.

@shlomitk1 fybrik cannot do it, it is the egg and chicken problem, in order to deploy something into these ns, manager needs permission to do it.

@shlomitk1 fybrik cannot do it, it is the egg and chicken problem, in order to deploy something into these ns, manager needs permission to do it.

Which ns? If it is managed by Fybrik then Fybrik also creates the namespace and the appropriate roles.
This issue is about managed=false. For managed=true, the current behavior is ok.

close it as duplicated by #793


Utils: Extract size for all duplicatePackages

[ ] utils: add total size to warnings.duplicatePackages[]
[ ] ui: show totalSize on DuplicatePackagesWarning

Not need it anymore - https://github.com/relative-ci/bundle-stats/issues/1235


Subscriptions List - Tags not provided for ServicePrincipal

Using a ServicePrincipal:
'GET https://management.azure.com/subscriptions?api-version=2020-01-01' (docs) does NOT return Subscription tags
whilst
'GET https://management.azure.com/subscriptions/{subscriptionId}?api-version=2020-01-01' (docs) does return Subscription tags
Using user context both APIs return the Subscription tags.
why?
Powershell:
Invoke-WebRequest -Uri $uri -Method get -Headers @{"Content-Type" = "application/json"; "Authorization" = "Bearer $bearerAccessToken" }
Invoke-AzRestmethod -path $path -method GET

any insights for this?

Removing Subscriptions from this. PM is following up with the owning team on this tag issue.

is there any news?

@rthorn17 - FYI.

ok, this seems more RBAC related. I can tell that with 'Reader' the API (GET https://management.azure.com/subscriptions?api-version=2020-01-01) does not return the subscription´s tags, whilst with 'Owner' it does.
GET https://management.azure.com/subscriptions/{subscriptionId}?api-version=2020-01-01 however works with 'Reader'.
Can you please shed light?

Any update on this?

Hi,
Unfortunately, this is still an issue. According to the List Subscriptions endpoint documentation, the sample response includes tag while the original response does not.
Sample response from documentation
{
  "value": [
    {
      "id": "/subscriptions/291bba3f-e0a5-47bc-a099-3bdcb2a50a05",
      "subscriptionId": "291bba3f-e0a5-47bc-a099-3bdcb2a50a05",
      "tenantId": "31c75423-32d6-4322-88b7-c478bdde4858",
      "displayName": "Example Subscription",
      "state": "Enabled",
      "subscriptionPolicies": {
        "locationPlacementId": "Internal_2014-09-01",
        "quotaId": "Internal_2014-09-01",
        "spendingLimit": "Off"
      },
      "authorizationSource": "RoleBased",
      "managedByTenants": [
        {
          "tenantId": "8f70baf1-1f6e-46a2-a1ff-238dac1ebfb7"
        }
      ],
      "tags": {
        "tagKey1": "tagValue1",
        "tagKey2": "tagValue2"
      }
    },
    {
      "id": "/subscriptions/72ac930a-f34e-42d8-b06d-dc2a9e12ed71",
      "subscriptionId": "72ac930a-f34e-42d8-b06d-dc2a9e12ed71",
      "tenantId": "2a0ff0de-96b2-4859-bb7c-a430d07a3e0c",
      "displayName": "Example Subscription2",
      "state": "Enabled",
      "subscriptionPolicies": {
        "locationPlacementId": "Internal_2014-09-01",
        "quotaId": "Internal_2014-09-01",
        "spendingLimit": "Off"
      },
      "authorizationSource": "RoleBased",
      "managedByTenants": [
        {
          "tenantId": "8f70baf1-1f6e-46a2-a1ff-238dac1ebfb7"
        },
        {
          "tenantId": "f7fb6af2-321d-47c8-9c0f-b0239eaad39a"
        }
      ],
      "tags": {
        "tagKey1": "tagValue1",
        "tagKey2": "tagValue2"
      }
    }
  ],
  "nextLink": "..."
}

Any update regarding this issue? :-)

Any update on this? Looks like an issue when I am using command Get-AzSubscription. Working without any issue on my regular account, having problem to get tags on Manage Identity :(

Yes, this issue is RBAC related, but it's because of where the RBAC assignment is which tells ARM where to source the call from.  If the Role Assignment is made directly to the resource, ARM sources the list call from ARM's resource tables where tags info is stored.  If the RBAC role is inherited to the resource, which most Service Principals have inherited rights to subscriptions, ARM sources the info from Management Groups RP based on the Hierarchy.  Management Groups doesn't store tags values of the subscriptions, so when the data is sourced from MG RP, it doesn't include the tag data.
Our teams are working on moving this LIST Subs call from going to Management Groups RP to have it sourced from ARG instead, which will resolve this issue.  Since ARG holds all the subscription data, it will be able to return tags data in the list call.  We are looking for this to be done summer next year.

Hi @rthorn17 we are still encountering this issue. Are there any plans to fix this please?


Contribution Guide

Are there any guides/guide lines on contributing? Specifically I was hoping to extend some features/syntax and I wanted to be able to add a test however it is non-obvious to me how I should execute the test suite in the first place. Are there any other pieces of information I should know before undertaking a pull-request?

Thank you for asking. I was always thinking I should add a contribution guide like Carthage has, a simple one.
Basically I follow the Swift coding guidelines of GitHub.
To run unit tests on Xcode with Command-U, you need run Carthage first.
carthage bootstrap --no-use-binaries --use-submodules

For now, because it fails with a tvOS issue, platforms should be specified to workaround.
carthage bootstrap --no-use-binaries --use-submodules --platform iOS,Mac,watchOS

Travis can test tvOS target even for now, so your pull request will be tested for all the targets before it is merged.

Another piece of information is that it requires Carthage greater than 0.8.0 (not sure which is required). Just know I had 0.8.0 and had to go to 0.9.4 and now that command succeeds.

Thanks for mentioning the version of Carthage. The latest version of Carthage is always preferable just because I do not test older versions. Currently Homebrew cannot install the latest version of Carthage. The installer of the latest version is available at the Carthage release page:
https://github.com/Carthage/Carthage/releases
0.9.4 is the latest at the moment.

Added a contribution guide.
https://github.com/Swinject/Swinject/blob/master/CONTRIBUTING.md


Add required tag

Adds missing required: true tag to the request connect widget
endpoint.

:shipit:


[Bug] Looks like longer responses are cut off

Context / Scenario
When I ask a question about a document where I expect a longer answer (like 2.000 characters), the answer is cut off. I tried with different documents and prompts and it looks like the max response is about 300 tokens?
What happened?
I'm using the MemoryServerless variant in a .Net 7 console app. I imported a PDF document "Building Your Own Product Copilot: Challenges, Opportunities, and Needs" (https://arxiv.org/pdf/2312.14231.pdf). And when I ask the following question "Give me a comprehensive list of all the findings in max 3000 characters". I get an answer that is cut off around 300 tokens?
I'm using Azure OpenAI service met GPT4 model version 1106-Preview and text-embedding-ada-002
Example:
            var memory = new KernelMemoryBuilder()
                 .WithAzureOpenAITextGeneration(new AzureOpenAIConfig
                 {
                    APIType = AzureOpenAIConfig.APITypes.ChatCompletion,
                    Auth = AzureOpenAIConfig.AuthTypes.APIKey,
                    APIKey = azureOpenAiKey,
                    Endpoint = azureOpenAiEndpoint,
                    Deployment = azureOpenAiChatDeployment
                 })
                 .WithAzureOpenAITextEmbeddingGeneration(new AzureOpenAIConfig
                 {
                    APIType = AzureOpenAIConfig.APITypes.EmbeddingGeneration,
                    Auth = AzureOpenAIConfig.AuthTypes.APIKey,
                    APIKey = azureOpenAiKey,
                    Endpoint = azureOpenAiEndpoint,
                    Deployment = azureOpenAiEmbeddingDeployment
                 })
                .Build<MemoryServerless>();

            filename = @"C:\temp\BuildingYourOwnProductCopilot.pdf";
            await memory.ImportDocumentAsync(filename, documentId: "copilot");

            var answer = await memory.AskAsync("Give me a comprehensive list of all the findings in max 3000 characters");
            Console.WriteLine(answer.Result);

An example answer is:
"The study on building product copilots, involving 26 professional software engineers, revealed several findings:

Trial and Error Process: Engineers found prompt engineering to be time-consuming, involving a lot of trial and error in ad-hoc environments.

Prompt Output Wrangling: Engineers struggled to systematically parse machine-readable outputs from prompts and had to iterate or change approaches frequently.

Context Balance: Balancing the right amount of context with token limitations was challenging, as was managing and tracking prompt assets.

Advanced Model Behavior: Engineers faced issues with models getting stuck in loops or going off-track and had difficulty in recognizing task completion.

Testing and Benchmarks: Traditional software testing methods were inadequate for LLMs, leading to challenges like flaky tests, creating benchmarks, and determining testing adequacy.

Automated Benchmark Creation: There was interest in systems that convert binary feedback into benchmarks, with a preference for straightforward percentage evaluations and actionable insights.

Metrics and Human Judgment: There was a disinterest in advanced machine learning metrics, with a focus on software engineering and business-centric metrics, and a belief in the irreplaceable role of human judgment.

Awareness and Visibility: Tool builders suggested mechanisms for alerting stakeholders of cost changes and recommended rigorous regression testing tools.

Ecosystem Support: Developers sought a unified "one-stop shop" for tool integration and expressed the need for templates and guidance in tool selection.

10"
It stops at 10... and I expect it to have some text after 10. (I tried this many times and obviously always with different result and it always looks cut off. Ending in the middle of a sentence)
Importance
I cannot use Kernel Memory
Platform, Language, Versions
I'm using Kernel Memory version 0.28.240212.1 (But also saw this issue with version 0.23.?)
Relevant log output
No response

You can set the maximum number of tokens for the answer when configuring Kernel Memory:
var kernelMemory = new KernelMemoryBuilder(builder.Services)
   //...
    .WithSearchClientConfig(new()
    {
        AnswerTokens = 800
    });

The default value for this property is 300.

Thank you! I tried some config changes, but was not aware of this one.
And it solves my problem!


Fix: packadd doesn't work when a new path partly matches with 'runtimepath'

Reported by @fw
Repro steps
Create /opt packages fo and foo:
mkdir -p ~/.vim/pack/test/opt/fo
mkdir -p ~/.vim/pack/test/opt/foo

Start Vim and do packadd:
vim --clean
:packadd foo
:packadd fo

Expected:
Both ~/.vim/pack/test/opt/foo and ~/.vim/pack/test/opt/foo are added to runtimepath
Actual:
~/.vim/pack/test/opt/foo is added to runtimepath but ~/.vim/pack/test/opt/fo is not
Cause and Solution
https://github.com/vim/vim/blob/master/src/ex_cmds2.c#L3824
It checkes a new path is in runtimepath using strstr(), but should check also the edge of the result of strstr().
Ozaki Kiichi

Codecov Report

Merging #2817 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2817      +/-   ##
==========================================
- Coverage    75.4%   75.39%   -0.02%
==========================================
  Files          92       92
  Lines      134592   134601       +9
==========================================
- Hits       101490   101483       -7
- Misses      33102    33118      +16

Impacted Files
Coverage Δ

src/ex_cmds2.c
78.67% <100%> (ø)
:arrow_up:

src/if_xcmdsrv.c
83.81% <0%> (-0.9%)
:arrow_down:

src/gui_gtk_x11.c
47.68% <0%> (-0.15%)
:arrow_down:

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

src/message.c
74.78% <0%> (-0.05%)
:arrow_down:

src/eval.c
81.2% <0%> (-0.05%)
:arrow_down:

src/syntax.c
78.27% <0%> (-0.03%)
:arrow_down:

src/screen.c
76.58% <0%> (-0.03%)
:arrow_down:

src/if_py_both.h
76.59% <0%> (-0.01%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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

Ozaki Kiichi wrote:

Reported by @fw
Repro steps
Create /opt packages fo and foo:
mkdir -p ~/.vim/pack/test/opt/fo
mkdir -p ~/.vim/pack/test/opt/foo

Start Vim and do packadd:
vim --clean
:packadd foo
:packadd fo

Expected:
Both ~/.vim/pack/test/opt/foo and ~/.vim/pack/test/opt/foo are added to runtimepath
Actual:
~/.vim/pack/test/opt/foo is added to runtimepath but ~/.vim/pack/test/opt/fo is not
Cause and Solution
https://github.com/vim/vim/blob/master/src/ex_cmds2.c#L3824
It checkes a new path is in runtimepath using strstr(), but should check also the edge of the result of strstr().

Thanks for looking into this and providing a solution.  I think we
better use copy_option_part().  And we should also use pathcmp() to
ignore differences in ways a path can be written.
--
From the classified section of a city newspaper:
Dog for sale: eats anything and is fond of children.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


When building a .NET Core 3.1 project that references a .NET Standard 2.1 project, I get Error NETSDK1073: The FrameworkReference 'Microsoft.WindowsDesktop.App' was not recognized (NETSDK1073)

/usr/local/share/dotnet/sdk/3.1.200/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.Sdk.FrameworkReferenceResolution.targets(5,5): Error NETSDK1073: The FrameworkReference 'Microsoft.WindowsDesktop.App' was not recognized (NETSDK1073)
I am building the solution on a Mac machine, so my question is why is Microsoft.WindowsDesktop.App even in my project's framework reference? When I expand the framework list on my project, I don't even see Microsoft.WindowsDesktop.App on the list.

However, when I use JetBrains Rider, I can see it, but with an error.

Any update or tip from someone?

Same here , did you solve this? @lewisgauss

@jadenrogers it stopped becoming a problem for me, after updating to a new .NET Core version some time ago. So I can't help you much here...

I had the same problem when building in azure pipeline (building locally on my Windows box had no problem). Changing the vm image to windows from ubuntu solved the problem. I guess the Windows related frameworks are simply not available on non-Windows systems.


Build on stable Rust

The main blocker for this is the use of #[fundamental] in the notification system. I haven't looked into it in too much detail, but I think this can be done by reducing the amount of typegineering going on. Thoughts?

Ping @Diggsey, @brson. Thoughts on this?
I'm happy to replace the notification infrastructure with something more like the emitter setup in rustc. That would get the facility without the ability to call the notifiers as though they were functions. My personal feeling is that the internal ergonomics win isn't worth requiring nightly.

Yes please! We should try to get on stable IMO.
On Feb 17, 2016 6:54 PM, "Kamal Marhubi"<EMAIL_ADDRESS>wrote:

Ping @Diggsey https://github.com/Diggsey, @brson
https://github.com/brson. Thoughts on this?
I'm happy to replace the notification infrastructure with something more
like the emitter setup in rustc. That would get the facility without the
ability to call the notifiers as though they were functions. My personal
feeling is that the internal ergonomics win isn't worth requiring nightly.
—
Reply to this email directly or view it on GitHub
https://github.com/Diggsey/multirust-rs/issues/58#issuecomment-185519212
.


Repo sync for protected CLA branch

The pull request is created from master637341488964012793 to master to fix git push error for protected CLA branch

@Taojunshen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


net/firewall: Gateway Groups not available

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
Plugin version used: 1.2
The os-firewall automation-rules do not allow gateway-groups to be selected (as gateway).
If one tries to set a gw-group via API there is no error response.
The gateway of the processed rule is set to 'none'.
To Reproduce
Steps to reproduce the behavior:

Add a gateway group in 'System - Gateways - Group'
Install the plugin os-firewall
Try selecting the before created gw-group as target gateway for a rule in 'Firewall - Automation - Filter'

Expected behavior
Being able to select gateway groups as target gateway for those rules as this is an important functionality.
Screenshots
Per example:

automation rules

default rules

Additional context
This is a very commonly used functionality and therefore it would be nice for it to be available using the API.
Lack of this functionality makes the API management unusable in many practical use-cases and environments. (WAN-/VPN-Failover and so on..)

Same issue on:
OPNsense 22.7.7_1-amd64
FreeBSD 13.1-RELEASE-p3
OpenSSL 1.1.1s 1 Nov 2022

@sbobber can't be an issue if it's a feature request, but feel free to open a PR

@AdSchellevis understood, would be great, thanks


Updating form validation docs with .field--error

i already did this because i'm stoopid and didn't see this PR

(corpsethumb)


Working with invitations.

There are invitations in the telegram. I  have an invitation to a group that I don't have. What should I do to receive this chat or what updates should I wait to see this chat (group) in order to process the invitation?

You can't receive the chat most of the time, you can only join it. All links are processed by calling getInternalLinkType -> getExternalLinkInfo -> getExternalLink. These links will have type internalLinkTypeChatInvite and should be processed as documented.

Thx) Here I have a link and I know that this is the internalLinkTypeChatInvite type. Can I use join Chat By Invite Link? Or do you need any additional actions?

Call checkChatInviteLink with the given invite link to process the link. If the link is valid and the user wants to join the chat, then call joinChatByInviteLink.

I called checkChatInviteLink. Why do I get this error when calling joinChatByInviteLink: [ 1][t 0][1705989755.632174015][DialogInviteLinkManager.cpp:84][!Td]	Receive wrong result for ImportChatInviteQuery: updates ?
When I call this method a second time, there is no such message.

Could you enable TDLib log with verbosity level 4 and send it as a file to https://t.me/tdlib_bot?

@levlam, sent

The mentioned error should be fixed in the latest TDLib version from Github.

I use this version 1.8.23

You need to update to the latest version from master.

@levlam Thx)


Used pandas.core.ops attribute does not appear to exist

Was running a careless script when the following (abridged) error occurred:
  File "/net/holy-nfsisilon/ifs/rc_labs/hekstra_lab/people/rahewitt/opt/anaconda/envs/test/lib/python3.11/site-packages/reciprocalspaceship/dtypes/internals.py", line 398, in __array_ufunc__
    result = ops.maybe_dispatch_ufunc_to_dunder_op(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'pandas.core.ops' has no attribute 'maybe_dispatch_ufunc_to_dunder_op'

Perhaps a move in a new version of pandas.

Do you mind confirming the pandas and rs versions? That will help me track down what's going on here

The versions are as follows:
reciprocalspaceship: 1.0.1
pandas: 2.0.3
Note: upgrading to pandas==2.1.1 appears to have fixed the issue.

Sorry for the delay, but looking into this it does seem that pandas v2.0.3 has relevant method/namespace:
In [1]: import pandas as pd

In [2]: pd.__version__
Out[2]: '2.0.3'

In [3]: from pandas.core import ops

In [4]: ops.maybe_dispatch_ufunc_to_dunder_op?
Docstring:
Dispatch a ufunc to the equivalent dunder method.

Parameters
----------
self : ArrayLike
    The array whose dunder method we dispatch to
ufunc : Callable
    A NumPy ufunc
method : {'reduce', 'accumulate', 'reduceat', 'outer', 'at', '__call__'}
inputs : ArrayLike
    The input arrays.
kwargs : Any
    The additional keyword arguments, e.g. ``out``.

Returns
-------
result : Any
    The result of applying the ufunc
Type:      builtin_function_or_method

Do you mind including more details of the traceback and a minimal example? I don't seem to be able to reproduce this locally with rs v1.0.1 and pandas v2.0.3.

I'm going to close this issue because I can't reproduce it, but please reopen if you run into any more troubles that seem like this.


Hue Bridge and Conbee shown as 'not supported' accessories.  Any way to remove them from My Home screen?

Issue
I understand per #76 these serve a function and must be there, but is there a way to remove these as 'favourites' from My Home screen?  There is no Settings button for these if you press their icons so no way to turn off 'Include in favourites'.
Am I missing something obvious here?
Thank you

Log Messages

Debug Files

See https://github.com/ebaauw/homebridge-hue/issues/767.

Thanks. Removed fro My Home using an old iPad running 12.4.8


URGENT: es-abstract module breakup has broken latest version on fusebox bundler

https://github.com/es-shims/array-includes/commit/c7fd1a05e94bcb36c0c7ffa3e8651a68dd88f700
This commit has caused my bundler to freak out. Are you sure you're referencing es-abstract correctly? Their documentation states:
All abstract operations will also be available under an es5/es2015/es2016/es2017/es2018/es2019 entry point, and exported property, if you require a specific version.
Yet your code is using syntax like
var ToInteger = require('es-abstract/2018/ToInteger');
You're referencing a folder path that's not actually being exposed by the es-abstract module.
var ToInteger = require('es-abstract/es2018/ToInteger'); if I'm not mistaken.
if so please push a new version out asap, this will likely be breaking across a whole range of bundlers.

It’s definitely exposed by v1.17.0-next.0 and v1.17.0-next.1, both of which are valid in the semver range in the linked commit. Tests pass in many packages using these versions.
Are you doing any kind of dependency resolution override? Does npm ls exit successfully?

I have forgotten to update es-abstract’s readme; thanks for the reminder to do that.

I’m looking at the latest version on master. The entry point is index.js.
https://github.com/ljharb/es-abstract/blob/master/index.js
It doesn’t expose a property called 2018, and the fuse box bundler is breaking as a result. While the true folder path may exist, the index.js is used as the sole source of truth here

It's not supposed to. es-abstract/2018/ToInteger is a file path, not a property.

CJS modules are just a file system. node resolution looks up the "main" when requiring a directory, but every file on the filesystem is directly accessible.
I'm not familiar with "fusebox bundler", can you elaborate on what it is? (The primary bundlers used in the JS ecosystem are webpack and browserify, and others include rollup, parcel, pika, metro; i've never heard of any others)

Fusebox is a bundler like we pack etc, but it’s designed for minimal config and maximum performance
@nchanged i might be a little out of my depth here could you weight in

Specifically, es-abstract/2018/ToInteger maps - by node's module resolution algorithm which in this respect has been unchanged for a decade - to this file<EMAIL_ADDRESS>

@jpike88 I see https://github.com/fuse-box/fuse-box/pull/1642 has been merged into master for fusebox but not released; the implication is that https://github.com/fuse-box/fuse-box/issues/1641 might be the problem.

I’m sitting on next.155 version so I’d be maybe only a couple of weeks out of date with master

In that case, I'm not sure - but I'm convinced it's either a configuration issue on your end, or a bug in fusebox, since deep requires are a quite reliable pattern.

sorry guys, the latest commit broke the resolver, and unfortunately, that wasn't covered with tests.

@nchanged thanks, so to confirm, this was just a bug in fusebox?

@ljharb yeah, the latest commit pushed the parent node_modules to the array which is essentially wrong, resulted in picking up a different version of es-abstract located there. All good now! ;-)


Questions

Hello,
I have some questions:

Is there an Api doc for this gem like Rails API docs?
Do you have more examples of how to use this gem?
At the current example at https://github.com/heckpsi-lab/em-midori/blob/master/example/route.rb#L37 it seems the router is in charge of responding, is this a good practice?
What is the layer of the system responsible for representing business data and logic for this gem?

Thanks! :)

I'm working on enriching API docs these days, which could be used before v0.1.0 release.
This gem is still on the early development, examples are just showing the principles and APIs we want it to be. More practicable examples would be enriched after we stabilize the APIs.
Yes. The API we designed want to use route definition instead of writing API docs manually. https://github.com/heckpsi-lab/em-midori/blob/master/example/route.rb#L33 The route definition not only shows which Controller to use, but also how request and response are, all possible errors and how to handle them. With the help of middlewares, this definition could be further simplified.
controllers could be defined to deal with complex logic if you are building large projects. But if the logic is simple enough, you can also directly write it in router. em-midori provides the flexibility of which design pattern you want in your project.

API docs could be viewed at here. But only 14.91% code are documented for now.


chore: log retry

Closes #31
Adds function to log retries. A function is used instead of a decorator because it is not possible to use a decorator to access the while loop in retry_with_backoff where retries are made.
Also renames _decorators.py to _human_utils.py
In _human_utils.py, from unstructured_client.models import errors is moved to prevent a circular import error between the models and utils modules.
Testing

from your unstructured venv, pip uninstall unstructured-client
navigate to unstructured-python-client, check out to this branch, and pip install -e .
pip install requests_mock
open a python shell and run the following:

from unstructured.partition.api import partition_via_api
import requests_mock

with requests_mock.Mocker() as mock:
    mock.post("https://api.unstructured.io/general/v0/general", status_code=500)
    elements = partition_via_api(filename="example-docs/fake-email.msg")

Hey @Coniferish @awalker4 I propose to squash branch before merging to main, there should be checkbox to do that automatically.

I see other repos follow this, which gives better overview on main of what happened recently, without going into individual commit details:

In both cases we see 9 days worth of history :)

Thanks @hubert-rutkowski85. I was just used to those setting existing in the other repos. They aren't set up on this one because it is ai generated. I'll look into setting that up for us, though.


Segfault in 1.4.x istio-proxy with SD extension when starting up while receiving traffic

Setup:

From a GCE VM (in this case: ubuntu-1604-lts-drawfork-v20190930 running on n1-standard-4 (4 vCPUs, 15 GB memory).
~15 QPS of incoming gRPC traffic (to the productcatalogservice from the hipster shop on port 3550)
start envoy with metadata exchange filter and stackdriver filter (1.4.x versions)
observe consistent Segfault.

Exemplar Stacktrace:
[Envoy (Epoch 0)] [2020-02-26 04:14:34.352][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:83] Caught Segmentation fault, suspect faulting address 0x20
[Envoy (Epoch 0)] [2020-02-26 04:14:34.352][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:70] Backtrace (use tools/stack_decode.py to get line numbers):
[Envoy (Epoch 0)] [2020-02-26 04:14:34.352][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:71] Envoy version: 5f5d621c9fd0b68e12c8e1b95db849588fbdc475/1.12.0/Clean/RELEASE/BoringSSL
[Envoy (Epoch 0)] [2020-02-26 04:14:34.353][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #0: __restore_rt [0x7f07e3c45390]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.361][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #1: Extensions::Stackdriver::Log::Logger::exportLogEntry() [0x5638dcf0b6a2]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.369][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #2: Envoy::Extensions::Common::Wasm::Null::Plugin::Stackdriver::StackdriverRootContext::onTick() [0x5638dcf00a3d]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.377][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #3: std::__1::__function::__func<>::operator()() [0x5638dd029467]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.386][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #4: Envoy::Extensions::Common::Wasm::Wasm::tickHandler() [0x5638dd47337f]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.394][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #5: std::__1::__function::__func<>::operator()() [0x5638dd47ee31]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.402][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #6: event_process_active_single_queue [0x5638de299866]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.410][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #7: event_base_loop [0x5638de29821e]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.419][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #8: Envoy::Server::InstanceImpl::run() [0x5638ddf4fdd3]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.427][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #9: Envoy::MainCommonBase::run() [0x5638dd0d2018]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.435][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #10: main [0x5638dd0d0ff2]
[Envoy (Epoch 0)] [2020-02-26 04:14:34.435][4178][critical][backtrace] [bazel-out/k8-opt/bin/external/envoy/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #11: __libc_start_main [0x7f07e388a830]

If you turn off the incoming gRPC traffic, the Envoy starts and stays up. If you then turn the traffic back on, it continues to function properly without segfault.
Appears to be something around initialization.
I have not done extensive testing with 1.5, etc.
/cc @PiotrSikora

Just skimming through the extension's code, my guess is that once the traffic starts flowing, even if it's only internal (e.g. agent doing "are you up yet?" checks, or kubernetes health-checks), extension tries to send metrics/logs to Stackdriver, but Envoy didn't receive the CDS config yet, so it doesn't know about the target cluster and crashes instead of handling this gracefully.
Does this sound plausible, @bianpengyuan @jplevyak?

Saw this again with 1.4.x testing last night in a normal k8s setup.
However it was so pervasive that 3 out of 4 ingresses were crashlooping.

This could be related to excessive long start up time, which exaggerate the effect of this bug.

@bianpengyuan is https://github.com/istio/proxy/pull/2721. Fixed in 1.5 and master?


Add a C++ example to the documentation

This PR adds a long-form C++ example to the documentation, similar to the existing C and Fortran examples.
This fixes #101.

View the example in the docs at: https://babelizer.readthedocs.io/en/mdpiper-add-cxx-example/example-cxx.html.

In reading over all of the examples, there is a lot of duplication between the languages. Each example is divided into three parts:

Build an example library
Run the babelizer on that library
Add metadata so that it can be used in pymt

I wonder if we can simplify this.
Building a BMI library seems like it is outside the scope of the babelizer. Instead of describing how to do this, could we either point to the instructions for each example or use an example library that can be installed via conda?
Since the pymt stuff is the same for each language can we have just one page that shows how to do this and then point to that page from each of the examples? This step is also optional and a little beyond the scope of the babelizer. Maybe that could be made more clear?

In reading over all of the examples, there is a lot of duplication between the languages.

I agree, and you're correct--it would be good to simplify. However, I'm OK with duplication here because most people will care only about one language, and I want a complete, end-to-end, example for each. Full disclosure: the first few times I tried this workflow (model -> BMI -> babelizer -> pymt), I was confused until I made an end-to-end example. I'm hoping my experience can help inform others.

@mdpiper, I got this example to work on a Mac, using the global environment file at https://github.com/LimnoTech/babelizer-examples/blob/main/environment.yml.
I'll now circle back to try to get it to run on Windows, which didn't work the first time I tried. I'll share updates.


Combine static and dynamic libraries in a project

Hey,
VcPkg currently uses /MDx compiler flags for dynamic libraries and /MTx flags for the static libraries.
Since the static and dynamic runtime libraries are incompatible, how can I combine static and dynamic libraries?
I would like to use the static version of library A and use the dynamic version of library B.
It would be useful if static libraries would link to the dynamic c runtime too. (or configurable)

This is a scenario where the flexibility of triplets comes in useful.
You can modify an existing triplet or create a new one and do something like this:
set(VCPKG_TARGET_ARCHITECTURE x86)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)

if (PORT STREQUAL zlib)
	set(VCPKG_CRT_LINKAGE static)
	set(VCPKG_LIBRARY_LINKAGE static)
endif()

In this example, I set a default and then I specify exceptions. So, everything will be built dynamically except zlib which is specifically set to static.
Also, as you can see, there is a different setting for CRT and Library linkage, so it is configurable as well.

Thanks, that might solve it.
I guess I'll have to set VCPKG_LIBRARY_LINKAGE of zlib to dynamic?
In order to get the libraries to link.

@madebr
VCPKG_LIBRARY_LINKAGE refers to the build type of zlib itself.
So, in the example, you will get static libs and not dlls.
VCPKG_LIBRARY_LINKAGE should be set to static.
Furthermore, I made a mistake in the previous post (which I have now edited). You should not specify different CRT in a per port basis.
More information and further triplet customization can be found in the documentation.


Update deployment.md

wrong article used

I signed it

/assign @daminisatya

/cla-check

I signed it

@Arhell I think I had not finished the CLA sign up process. Let's see if it works now.

I signed it

Page preview:
https://deploy-preview-22445--kubernetes-io-master-staging.netlify.app/docs/concepts/workloads/controllers/deployment/#selector

Hello @piazera .
Thanks for opening this pull request.
/lgtm

Thanks!
/approve


Fix dup exception handling and print in 8686

Fix for PR #8686 where I forgot cleaning the file before putting it in a PR.
@ticoann please review

Sorry Alan, I merged that patch too early.


PICNIC-797 - Crypto refactor and use hooks

Removes operation/{name}.container.tsx
Uses operation prop to key all child components for useSelector so they can update independently
Adds some convenience functions to constants/crypto.tsx for getting input placeholder text and textType
Rewrites stories to use Sb.MockStore and some neat store updates with Container.produce for better, more organized, stories

cc @keybase/react-hackers @keybase/picnicsquad

bump @keybase/react-hackers


ClickHouseKeeperInstallation doesn't work on some kubernetes clusters

We have multiple clusters, the same ClickHouseKeeperInstallation can work on some clusters but doesn't work on other clusters. I'm using the ClickHouseKeeperInstallation from 0.24.0 branch.
On those clusters that don't work,
$ kubectl describe sts clickhouse-keeper
Name:               clickhouse-keeper
Namespace:          default
CreationTimestamp:  Wed, 18 Sep 2024 16:32:03 -0700
Selector:           app=clickhouse-keeper,uid=b00ef8b7-416a-4948-a10a-edc3d4186bc6
Labels:             app=clickhouse-keeper
                    uid=b00ef8b7-416a-4948-a10a-edc3d4186bc6
Annotations:        <none>
Replicas:           3 desired | 0 total
Update Strategy:    RollingUpdate
Pods Status:        0 Running / 0 Waiting / 0 Succeeded / 0 Failed
Pod Template:
  Labels:       app=clickhouse-keeper
                uid=b00ef8b7-416a-4948-a10a-edc3d4186bc6
  Annotations:  kubectl.kubernetes.io/restartedAt: 2024-09-18T23:32:03.895289135Z
  Containers: <none>
  Volumes:      <none>
Volume Claims:  <none>
Events:
  Type     Reason        Age                    From                    Message
  ----     ------        ----                   ----                    -------
  Warning  FailedCreate  5m36s (x26 over 144m)  statefulset-controller  create Pod clickhouse-keeper-0 in StatefulSet clickhouse-keeper failed error: Pod "clickhouse-keeper-0" is invalid: spec.containers: Required value

$kubectl get sts clickhouse-keeper  -oyaml
apiVersion: apps/v1
kind: StatefulSet
metadata:
  creationTimestamp: "2024-09-18T23:32:03Z"
  generation: 2
  labels:
    app: clickhouse-keeper
    uid: b00ef8b7-416a-4948-a10a-edc3d4186bc6
  name: clickhouse-keeper
  namespace: default
  ownerReferences:
  - apiVersion: clickhouse-keeper.altinity.com/v1
    blockOwnerDeletion: true
    controller: true
    kind: ClickHouseKeeperInstallation
    name: clickhouse-keeper
    uid: b00ef8b7-416a-4948-a10a-edc3d4186bc6
  resourceVersion: "136857642"
  uid: 51bc0c34-bd84-442d-97f2-6ac3a508454c
spec:
  persistentVolumeClaimRetentionPolicy:
    whenDeleted: Retain
    whenScaled: Retain
  podManagementPolicy: OrderedReady
  replicas: 3
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      app: clickhouse-keeper
      uid: b00ef8b7-416a-4948-a10a-edc3d4186bc6
  serviceName: ""
  template:
    metadata:
      annotations:
        kubectl.kubernetes.io/restartedAt: "2024-09-18T23:32:03.895289135Z"
      creationTimestamp: null
      generateName: clickhouse-keeper
      labels:
        app: clickhouse-keeper
        uid: b00ef8b7-416a-4948-a10a-edc3d4186bc6
    spec:
      containers: null
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext: {}
      terminationGracePeriodSeconds: 30
  updateStrategy:
    type: RollingUpdate
status:
  availableReplicas: 0
  collisionCount: 0
  currentRevision: clickhouse-keeper-6f77b954f8
  observedGeneration: 2
  replicas: 0
  updateRevision: clickhouse-keeper-676f994b45

Ok it turns the kubernetes version has to be at least 1.29. After upgrading it works now


edits/comments in draft paper

feel free not to merge this but to just consider this as feedback if you want...

I merged this in by hand. Thanks!


Improve versioning with Git tags / GH releases.

Hello,
thanks for the lib It works great! I faced with some limitations:

I would like to avoid downloading https://github.com/mxschmitt/playwright-go/blob/c9d1531e0354a7f267237b3ac55595a8f18170e4/run.go#L29 after every container start. I assumed that playwright is already installed on the official mcr.microsoft.com/playwright:bionic images.

It looks like that playwright is installed from an external source https://github.com/mxschmitt/playwright-go/blob/c9d1531e0354a7f267237b3ac55595a8f18170e4/run.go#L15 not the official ones.

How can I install a different playwright version? The current is 1.7

Hi @StarpTech,
thanks for reaching out.
Regarding the first issue, the language support on Playwright is based on a driver. This driver is then used like in the C#, Java or Python version to start the actual browsers. The communication is based on stdin/stdout. This driver is then installing the browsers here. And yes, the browsers are preinstalled in the official Docker image, but the driver isn't and unfortunately can't.
For the second one, yes it was until a few days where the driver was not yet hosted officially. There I migrated over to the official mirror, see here and #54.
The Go version and the driver version needs to be in sync. I will start later or in the next few days assigning logical version numbers to the repo as GitHub releases and tags, so it makes sense, as it's done in Python repository: https://github.com/microsoft/playwright-python/releases
Let me know if you have more questions and glad to hear that you like it. (There is a lot of potential in improving the option type handling and adding more tests, thats why I call it currently not yet production ready).

Regarding the first issue, the language support on Playwright is based on a driver. This driver is then used like in the C#, Java or Python version to start the actual browsers. The communication is based on stdin/stdout. This driver is then installing the browsers here. And yes, the browsers are preinstalled in the official Docker image, but the driver isn't and unfortunately can't.

Does this mean we even have to download the browsers twice? This is esentially what I see when running my container
2020/12/22 16:46:12 Downloading driver...
2020/12/22 16:46:27 Downloaded driver successfully
2020/12/22 16:46:27 Downloading browsers...
Failed to process descriptor at /tmp/pw/node_modules/playwright
2020/12/22 16:46:28 Downloaded browsers successfully

For the second one, yes it was until a few days where the driver was not yet hosted officially. There I migrated over to the official mirror, see here and #54.

The Go version and the driver version needs to be in sync. I will start later or in the next few days assigning logical version numbers to the repo as GitHub releases and tags, so it makes sense, as it's done in Python repository: microsoft/playwright-python/releases

Great :+1:

Does this mean we even have to download the browsers twice? This is esentially what I see when running my container (based on playwright base image)

If the browser version will match with the browser installed in the Docker image, then they won't be downloaded again. This is blocked by the versioning, so its more clearer in the future.
I renamed the issue and we keep that open until I have the clear versioning in place, if you have more suggestions, feel free to open new issues, thanks!

@mxschmitt during testing it gets even worse. It downloads .ms-playwright for every subdir.

@mxschmitt during testing it gets even worse. It populates .ms-playwright for every subdir.

Thats currently the dilemma with my current approach. The driver needs to be downloaded a single time when it runs for the first time, currently during runtime which hangs in the beginning for a few seconds and its stored in the $PWD/.mw-playwright.
So doing something like that would install the browsers in e.g. in Docker:
package main

import (
	"log"

	"github.com/mxschmitt/playwright-go"
)

func main() {
	if err := playwright.Install(); err != nil {
		log.Printf("could not install: %v", err)
	}
}

Does this work for you approach or answer your question?

I do not fully understand how it solves the issue because my $PWD is different for each test suite. I need a way to pass the location of the driver.

I do not fully understand how it will solve the issue because my $PWD is different for each test suite. I need a way to pass the location of the driver.

Not yet, we can add something like that. Let me come up with a proposal for it, will provide a PR for it soon.

@StarpTech I rolled to Playwright v1.7.1 #66 but won't publish a release yet because I want to add missing functionality like video recording etc.

Closed, I released version 0.171.0 which follows the same version schema as Playwright for Python etc. are using. It corresponds to version 1.7.1 of Playwright.


Add lessons scheme

Add lessons scheme for DB, increase safety

Done, waiting for @zuevval

Going to add tests here

So now all lessons and steps can be moved to database and after that alpha branch can be removed.

@zuevval feel free to merge afret approving.


Eager prealloc no inline

Always find the preallocated size

Pushing in spite of travis failure because cargo nightlies are broken


ON+Prgm shortcut uses up >65,000 bytes of RAM, but only in Classic mode

Steps to reproduce:

Set calculator to classic mode (mode, up, right, enter, clear)
Use Cesium shortcut (ON + Prgm)

lol

Also happens when using ON + Prgm while editing a program normally, regardless of mode.

Fixed with release 3.1.7.


Remove unnecessary namespace qualifiers in C# LINQ Samples program.cs files

Program.cs files should all be contained within a namespace and we should remove any unnecessary namespace qualifiers.

@matthewreily just a tip, if you reference an issue in the PR then it will close automatically when we merge. Put "Fix #nnn" in the description of your PR

Thanks for the tip!
Sent from my iPhone

On Nov 18, 2015, at 9:30 PM, Beth Massi<EMAIL_ADDRESS>wrote:
@matthewreily just a tip, if you reference an issue in the PR then it will close automatically when we merge. Put "Fix #nnn" in the description of your PR
—
Reply to this email directly or view it on GitHub.


i65: basic payload validation

If tracking request includes test:true the server will validate the payload and return

HTTP code 200 if the payload includes all required properties

{
  "ok": true
}

HTTP code 400 and the list of required properties that are missing

{
  "ok": false,
  "missing": [
    "application_name"
  ]
}

The tracking request is not persisted in the repository. This PR does not alter the current server behavior changed if the test property is not defined or not set to true.

Delivers first part of #65


[suggestion] emulate readlink using posix ls

stat(1) is not posix compatible.
emulated readlink using ls

Thanks for the PR! I think you're right, ls is probably a more reliable way to emulate readlink support.
Do you see any reason to keep around the stat readlink emulation? I'm inclined to get rid of it if there's ls readlink emulation.
Also, it might be nice to have an automated test or two around this change, but I'm not super worried about it since it's a bonus feature that's basically a big regex.
I can test this out and verify that it seems to work right sometime in the next few days. Or if someone else wants to try it out and vouch for the change, I'd consider merging it sooner.

the ls readlink emulation is just suggestion. so i didnt consider removing the stat readlink emulation.
i think the ls readlink emulation can cover the stat readlink emulation.
and i was careless about an automated test. it's my fault. i'll add automated test soon.
i didnt usually pay attention about automated test. if i may, can i ask you an advice while i add an automated test?


[V2] Links are broken

🐛 Bug Report
Links are broken, when opening a doc page directly, when navigating to the page from home page, links work.
related issue https://github.com/pnpm/pnpm.github.io/issues/94
Have you read the Contributing Guidelines on issues?
yes
To Reproduce

Open this page: https://pnpm.js.org/installation/#upgrading
click on the link in the tip

(or on this page click on the workspace link)
Expected behavior
This page is opened: https://pnpm.js.org/continuous-integration
Actual Behavior
A non-existen page is opened: https://pnpm.js.org/installation/continuous-integration
Your Environment

Public source code: https://github.com/pnpm/pnpm.github.io/edit/source/versioned_docs/version-5.x/installation.md
Public site url:
Docusaurus version used: v2
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0):
Operating system and version (desktop or mobile):

Reproducible Demo

Hi
This is a known issue due to using relative links and having your host provider adding a trailing slash
You can use absolute paths, relative paths to file (including md extension) or configure your host.
What is your host?
Duplicate of https://github.com/facebook/docusaurus/issues/3372

GitHub pages is our host


computation in component is much lower

We are facing a problem that, computation is much lower if executed in a CyberRT component.
We just write an image undistortion program using opencv (cpu mode) in docker. The computation takes less than 10ms only. However, when translating to a component, the undistortion process takes more than 100ms. We put the computation in the Proc function of the component, and use another component to send message only. Then, we config the scheduler of this component, results not change. Only the mentioned two components are launched in docker.
Addition, we change the processor_num in the scheduler_conf file, but changes not occure.
So, what's the problem?  and why the scheduler conf is not work?

I think maybe these are two problems. First，the conf file corresponds to the process name, the scheduler conf is not work, I guess it may not be the conf file for your process. If the processor does not have the corresponds conf,  it will use the default_proc_num in cyber.pb.conf, normally there's nothing wrong here. Then,  can you show the detail of your case ? Otherwise, it's not good to locate the problem

@zhangchaoltt   Thanks. The test codes are as follows:
TestComponent.cc  [dir: apollo/modules/test/]
bool TestComponent::Proc(const std::shared_ptr& msg){
double start_time = cyber::Time::Now().ToSecond();
undistort_one_image();
std::cout << "diff time: " << cyber::Time::Now().ToSecond() - start_time<< std::endl;
return true;
}
dag file    [dir: apollo/modules/test/]
module_config {
module_library : "/apollo/bazel-bin/modules/test/libtest_component.so"
components {
class_name : "TestComponent"
config {
name : "undist_img"
readers {
channel: "/apollo/test"
}
}
}
}
launch file    [dir: apollo/modules/test/]

test
<dag_conf>/apollo/modules/test/test.dag</dag_conf>
<process_name>test_undist</process_name>

scheduler conf file  [dir: apollo/cyber/conf/test_undist.conf]
scheduler_conf {
policy: "classic"
classic_conf {
groups: [
{
name: "undistortion_test"
processor_num: 8
affinity: "range"
cpuset: "0-7"
processor_policy: "SCHED_OTHER"
processor_prio: 0
tasks: [
{
name: "undist_img"
prio: 0
}
]
}
]
}
}

You can use cyber::Time::Now().ToNanosecond() instead of  cyber::Time::Now().ToSecond()  to calculate time-consuming

You can use cyber::Time::Now().ToNanosecond() instead of cyber::Time::Now().ToSecond() to calculate time-consuming

sorry, it doesn't work for me. Time consuming is the same as using cyber::Time::Now().ToSecond().
In addition, I modify the routine_num and default_proc_num in the cyber.pb.conf, but time consuming does not change also.

sorry, it doesn't work for me. Time consuming is the same as using cyber::Time::Now().ToSecond().
In addition, I modify the routine_num and default_proc_num in the cyber.pb.conf, but time consuming does not change also.

@jeff-zhang123 Try this command bazel build -c opt --copt=-fpic //cyber/...  to build CyberRT,  which mean enable compiler optimization with level 2, as same as -O2. then test your case.

@zhangchaoltt   I update cyberRT to the latest version, and build cyber using your command. it's ok now, I change the routine_num from 100 to 10, and cyber_launch my test component, time consuming reaches 7ms. Thanks for your reply.


Added small icons in contact us page in home page and about page

i have added icons in contact us page as shown!

Hope you like it!!

@BVPKARTHIKEYA add it to all html pages in the repo in footer section.

can you mention the names of html pages for collections and services i will make changes!!

and i would like to redesign the contact us page in a better manner!!
would you like me to do that also!!

and i would like to redesign the contact us page in a better manner!! would you like me to do that also!!

to all html-pages.

i have updated contact us icons to all html pages and i have redesigned the broken contact us page !!

Hope you Like it!!
please approve my pr with best level!!

@BVPKARTHIKEYA resolve merge conflicts

resolved!!
thanks for your feedback

@BVPKARTHIKEYA if redesign the contact us page , well then i will assign u , level 3.

can you specify me the changes mam!!
i have already made the broken contact us page into a modern one!!
do you like my changes of contact page
my changes are:
1.showing map in a beautiful manner
2.input fields are modified
3.made the column wise option changes to row wise changes
4.added icons to the contact page

can you specify me the changes mam!! i have already made the broken contact us page into a modern one!! do you like my changes of contact page my changes are: 1.showing map in a beautiful manner 2.input fields are modified 3.made the column wise option changes to row wise changes 4.added icons to the contact page

place contact us message in as usual page.

That's it.

so shall i remove the one present here mam!!

so shall i remove the one present here mam!!

Yes.u need to place it in as usual place in footer section.

ok mam i will do the required changes!!
Thanks for your feedback!

Added contact us form in the footer itself!
Hence the changes requested are done!! mam

@BVPKARTHIKEYA gap between footer of social icons.

is it ok mam??

 is it ok mam??

follow us and social should be included , same as index.html page.

ok mam!!

i have made the changes mam

ok mam!!  i have made the changes mam

but follow has different background color.

shall i change color mam!!

shall i change color mam!!

yes color same as footer.

now the color is same mam
is it ok??

now the color is same mam is it ok??

yes thank for these great changes 🥳

your words mean a lot to me!!
Thank you very much!!

your words mean a lot to me!! Thank you very much!!

Appreciate ur consistency work.

i hope you give me the best level mam!!

i hope you give me the best level mam!!

yes did.

we Shall connect on Linkedin!!
https://www.linkedin.com/in/boddeda-venkata-pavan-karthikeya-1a670b255/

we Shall connect on Linkedin!! https://www.linkedin.com/in/boddeda-venkata-pavan-karthikeya-1a670b255/

yes , sent u invitation too.

Messaged u @Anjaliavv51 !!


More Styling

The Current styling is Boring, and I want to change it.
Regards, Ireland aka PJ Satur
July 22, 2022

After 8 months am going to do this now

[ ] yes


[13pt] HydroVIS output file changes (s3.py)

Ryan Spies and Shawn Crowley have new requirements for which files and drop locations when we push files up to HydroVIS on production releases.
Our s3.py code will need to be changed, or more probable, make a new hydrovis_s3.py file as the complexity has increased.
The new file pattern in the HV bucket will be

(fim version) (ie.. fim_4_3_3_4)

hand_datasets

(each of the hucs folders filtered as before)

The sierra test scripts are going to be upgraded as part of a second issue and its outputs will go to:

(fim version)

qa_datasets

usgs_rating_curve.csv (from our inputs directory)
agg_nwm_recurr_flow_elev_stats_location_id.csv (probably from fim_performance/(huc version)/rating_curve_comparison folder.

This will need to be ready for the next production release to HV.

Aug 9, 2024: Completed somewhere along the line.


Migrate cosmos-sdk to v0.46.2

What is the purpose of the change

Upgrade cosmos-sdk to v0.46.2
Upgrade ibc-go to v5

Brief Changelog

Upgrade cosmos-sdk to v0.46.2
Upgrade ibc-go to v5

Testing and Verifying

This change is a trivial rework/code cleanup without any test coverage

Documentation and Release Note

Does this pull request introduces a new feature or user-facing behaviour changes? no
How is the feature or change documented? not applicable

How is this affected by the dragonberry patch?

I think we should not use 46 sdk

@levanhieuhust -- nice work on this.
There was -- a lot-- of miscommunication around the dragon and 46.3.
I've spoken with Marko and he says it is good to go.  I am going to update this branch.

@levanhieuhust -- nice work on this.
There was -- a lot-- of miscommunication around the dragon and 46.3.
I've spoken with Marko and he says it is good to go. I am going to update this branch.
Actually, @levanhieuhust would you mind doing it, once ibc-go is updated to v0.46.3?
Thank you.

Thank Sir ! I'm willing to keep doing it when ibc-go is updated to v0.46.3

I refreshed this, but I think we should close in favor of #347