no response from bot(test in web chat ) though have completed all step successfuly

-I have checked and tested  my knowledgebase is working fine on https://www.qnamaker.ai

I have completed successfully step 5 and all other step also.
have sync the configuration app service also and it deployed successfully again ,I have checked the log

Command: deploy.cmd
Handling .NET Web Application deployment.
Using Msbuild from 'D:\Program Files (x86)\MSBuild-<IP_ADDRESS>4\MSBuild\MSBuild\15.0\Bin'.
All packages listed in packages.config are already installed.
Committing restore...
Assets file has not changed. Skipping assets file writing. Path: D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus.Common\obj\project.assets.json
Restore completed in 369.77 ms for D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus.Common\Microsoft.Teams.Apps.FAQPlusPlus.Common.csproj.
Committing restore...
Assets file has not changed. Skipping assets file writing. Path: D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus.AzureFunction\obj\project.assets.json
Restore completed in 424.7 ms for D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus.AzureFunction\Microsoft.Teams.Apps.FAQPlusPlus.AzureFunction.csproj.
Committing restore...
Assets file has not changed. Skipping assets file writing. Path: D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus\obj\project.assets.json
Restore completed in 362.21 ms for D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus\Microsoft.Teams.Apps.FAQPlusPlus.csproj.
NuGet Config files used:
C:\DWASFiles\Sites#1rbeibot-config\AppData\NuGet\NuGet.Config
Feeds used:
https://api.nuget.org/v3/index.json
Restore operation started...
Restoring library<EMAIL_ADDRESS>Restoring library<EMAIL_ADDRESS>Restoring library<EMAIL_ADDRESS>Restoring library<EMAIL_ADDRESS>Restoring library<EMAIL_ADDRESS>Restoring library<EMAIL_ADDRESS>6 libraries restored in 5.62 seconds
CSC : warning SA0001: XML comment analysis is disabled due to project configuration [D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus.Common\Microsoft.Teams.Apps.FAQPlusPlus.Common.csproj]
Microsoft.Teams.Apps.FAQPlusPlus.Common -> D:\home\site\repository\Source\Microsoft.Teams.Apps.FAQPlusPlus.Common\bin\Release\netstandard2.0\Microsoft.Teams.Apps.FAQPlusPlus.Common.dll
CSC : warning SA0001: XML comment analysis is disabled due to project configuration [D:\home\site\repository\Source\Microsoft.Teams.Apps.FaqPlusPlus.Configuration\Microsoft.Teams.Apps.FaqPlusPlus.Configuration.csproj]
Microsoft.Teams.Apps.FaqPlusPlus.Configuration -> D:\home\site\repository\Source\Microsoft.Teams.Apps.FaqPlusPlus.Configuration\bin\Microsoft.Teams.Apps.FAQPlusPlus.Configuration.dll
Microsoft.ApplicationInsights.Web: Copying ApplicationInsights.config to bin
Transformed Web.config using D:\home\site\repository\Source\Microsoft.Teams.Apps.FaqPlusPlus.Configuration\Web.Release.config into obj\Release\TransformWebConfig\transformed\Web.config.
Copying all files to temporary location below for package/publish:
D:\local\Temp\8d838fc27d9210e.
Creating app_offline.htm
KuduSync.NET from: 'D:\local\Temp\8d838fc27d9210e' to: 'D:\home\site\wwwroot'
Copying file: 'bin\Microsoft.Teams.Apps.FAQPlusPlus.Common.dll'
Copying file: 'bin\Microsoft.Teams.Apps.FAQPlusPlus.Common.pdb'
Copying file: 'bin\Microsoft.Teams.Apps.FAQPlusPlus.Configuration.dll'
Copying file: 'bin\Microsoft.Teams.Apps.FAQPlusPlus.Configuration.pdb'
Deleting app_offline.htm
Finished successfully.
-But while testing my bot in web chat I am not getting any response .
-When I open https://mybotname.azurewebsites.net/   it says your bot is ready.
-Even my  messaging endpoint is not up.as it is giving
HTTP 404 error
That’s odd... Microsoft Edge can’t find this page

Hi @sargamtiwari - can you please try to create the Teams application package https://github.com/OfficeDev/microsoft-teams-apps-faqplus/wiki/Deployment-Guide#step-6-create-the-teams-app-packages .
The  https://mybotname.azurewebsites.net/  is not supposed to render anything, as the app uses /api/messages endpoint to serve the incoming bot command requests.
Please let us know if the bot after installation in Teams worked for you or not.

Hi abbodh,
Thanx for your reply,
I have followed step 6 and created the teams package and deployed to teams  but still not getting any reply

Thank you for reporting this. Seems like there would be some deployment issue. Please validate tenant Id in App Service configuration in Azure portal. It should be tenant Id of tenant where manifest is installed.
Please review other configuration values and manifest values as well.
If issue still persists, try scaling up App Service plan to S3 and sync App Service from Deployment Center in Azure portal.


memory increase fast when database disconnected

Hi
I shutdown my oracle instance when the program running, and the memory increase fast.
More than 100M per second.
My demo code is as below
func dbRoutine() {
	for {
		var temp int
		//gdb.RLock()
		gdb.db.QueryRow("select popid from devicetable where devid=7008").Scan(&temp)
		if rand.Int()%10 == 0 {
			time.Sleep(50 * time.Millisecond)
		}
		//gdb.RUnlock()
	}
}

func main() {
	log.Println("starting", driver)
	startDB()

	for i := 0; i < 40; i++ {
		go dbRoutine()
	}
	//go checkStat()
	//go switchRoutine()
	select {}
}

dup https://github.com/mattn/go-oci8/issues/120


Ignore volumes marked as io.conplicity..ignore=true

Requires https://github.com/fsouza/go-dockerclient/pull/504

LGTM


Show button loading animation when waiting for MakeCode to open

Without this animation, it's not clear if the user's button click has registered.
A component rename to match its file name is also included in this PR.

Maybe we can use this for "Save" (dialog + toolbar button, depending on help step, so some complexity) rather than the indeterminate progress dialog which feels odd.

Maybe we can use this for "Save" (dialog + toolbar button, depending on help step, so some complexity) rather than the indeterminate progress dialog which feels odd.

Yes it's possible. I'll make a ticket to explore this as an option.


[Bug]

There is an issue with registration confirmation. I get the following error after clicking the link in my email: 403 Invalid signature.
I have tried several times and each time same error.

Hello. Sorry, my reply is very late. I'm going to close this because so much time passed, but if you still have the problem please create a new issue, I will surely look into it.
Sorry again.


Compile errors in Android project

Hello,
I'm getting the following error message:
Unhandled Exception:
Java.Lang.RuntimeException: Unable to start activity ComponentInfo{com.microsoft.smarthotel/md52696540e2fcaa81fffea6d2d44407794.SplashActivity}: java.lang.IllegalStateException: You need to use a Theme.AppCompat theme (or descendant) with this activity.

@angelSwp We tried reproducing this to no avail. Could you you provide us more information about your configuration? Are you still seeing this? We've not heard back in a week so I'm closing this, but please reach out if you're still seeing these issues.


New article-Synchronize-multiple-controls

This reference (old version) document for this article is - (link)
Preview - link

@adegeo Acrolinx score for this article - 100.


Change checkout fangorn icon so that it changes color properly

JIRA ticket number for this @hmoco ?

No JIRA ticket, this is a fix for the release

:+1:


how can I conver  query result DateTimePath to NumberPath？

The type in the database is DateTimestamp type,
I want to query in Java that converted to Long type， what can I do？

i try to use milliSecond()， but got
java.lang.IllegalArgumentException: Unsupported expression millisecond

milliSecond() is supported for dialects that support it. It returns the milliseconds portion of the timestamp however, not the time since Linux epoch.
What is the dialect you're using?

i am using JPA with mysql database

JPQL, the query language of JPA, does not define any date extract functions. Hibernates proprietairy HQL declares a bunch of date extract functions, but millSeconds nor epoch isn't one of them. As such, this is not a QueryDSL limitation, but a limitation of your query language.
What you need to do is define a custom function for your ORM (presumably Hibernate), and then define a custom template in QueryDSL. See also my answer on this Stackoverflow question: https://stackoverflow.com/questions/40744385/querydsl-4-with-rownumber-window-function/62200160#62200160 .


[FEATURE REQUEST] Allow --force-update for helm version >= 3.3

Is this a request for help?:
yes
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST

Environment (plugin version, maven version, OS, ...):
helm-maven-plugin 5.9 + helm v3.5.3
And as checked, for helm <= v3.2, the --repository-cache can not work. and for helm >= v3.3, repository cache works, but if repo already exist and no authentication provided, the exit value would be 1.
What happened:
"Unable to add repo" when repo already exist. and repository cache cannot work as expected.
What you expected to happen:
Allow to force update repo in helm-maven-plugin.
How to reproduce it (as minimally and precisely as possible):
Consider to use helm-maven-plugin 5.9 and helm v3.5.3, and try to helm init.
Anything else we need to know:

@axdotl  Alex, would you mind help for this enhancement?  Thanks very much!

Hi @NancyChunlan ,
as I'm no longer an active maintainer for this project maybe @jekkel or @manu11th can jump in.


button.html

I have created three types of button
1.Color oriented
2.Size oriented
3.Border radius
just click the button.html to see code

done

you need to create a directory with your username... then put button code in index.html

@imdhanish  I'll mark this PR as Invalid if you'll not correct things asap!

wait man

done

@imdhanish  Delete button.html and then create PR


Removed scalariform.sbt

The reason to remove this file is that this produces published versions of scalismo that are not contained in the git-history of commits. As we now want to use single commit based artifacts, this is no longer an option.

I disagree.
The purpose of this is to reformat before committing, to ensure a consistent style for all commits.
If everybody did that, we would see way less noise in the commit diffs (mostly whitespace changes), cluttering and effectively hiding where actual logic was changed.
If scalariform were used the way it's meant to ("sbt compile; git commit" from the command-line), we wouldn't have that problem at all. But it requires the discipline to do so.
If you remove this file, I predict that the issue will get even worse, because then everybody will be pushing things with their own formatting, and the noise would just increase.

I fully agreee that PR (#170) is the better solution. I recommend that every developer should locally install in the repository git-hooks the following two line:
sbt scalariformFormat
sbt test:scalariformFormat


Push change a single time for X.java and X.class change

[kind/Enhancement]
Which functionality do you think we should update/improve?
Currently, when using odo watch on a Java project, when modifying one java file in an IDE having auto-compilation like VS Code with Java Extension pack, there are 2 push and 2 rebuild of the component for a single modification.
C:\Users\Aurelien Pupier\AppData\Local\Programs\Microsoft VS Code>odo watch with-java2 --app demo --project myproject
Waiting for something to change in c:\ws\workingDirectoryForTest\openshift
File c:\ws\workingDirectoryForTest\openshift\src\main\java\com\redhat\fuse\boosters\rest\http\GreetingsServiceImpl.java changed
Pushing files...
 ?[32m✓?[0m  Copying files to component
 ?[32m✓?[0m  Building component
Waiting for something to change in c:\ws\workingDirectoryForTest\openshiftFile c:\ws\workingDirectoryForTest\openshift\target\classes\com\redhat\fuse\boosters\rest\http\GreetingsServiceImpl.class changed
Pushing files... ?[32m✓?[0m  Waiting for component to start
 ?[32m✓?[0m  Copying files to component
 ?[32m✓?[0m  Building component
Waiting for something to change in c:\ws\workingDirectoryForTest\openshift

Why is this needed?
it will improve performance and avoid to start testing and having the application shutting down while just starting

hum sometimes it is working:
Waiting for something to change in c:\ws\workingDirectoryForTest\openshiftFile c:\ws\workingDirectoryForTest\openshift\target\classes\com\redhat\fuse\boosters\rest\http\GreetingsServiceImpl.class changed
Pushing files... ?[32m✓?[0m  Waiting for component to start
 ?[32m✓?[0m  Copying files to component
 ?[32m✓?[0m  Building component
Waiting for something to change in c:\ws\workingDirectoryForTest\openshift
File c:\ws\workingDirectoryForTest\openshift\src\main\java\com\redhat\fuse\boosters\rest\http\GreetingsServiceImpl.java changedFile c:\ws\workingDirectoryForTest\openshift\target\classes\com\redhat\fuse\boosters\rest\http\GreetingsServiceImpl.class changed
Pushing files...
 ?[32m✓?[0m  Waiting for component to start
 ?[32m✓?[0m  Copying files to component
 ?[32m✓?[0m  Building component
Waiting for something to change in c:\ws\workingDirectoryForTest\openshift

you can ignore files for odo watch
you should be able to do this: odo watch --ignore "*.class"
if you have .gitignore file in your root folder odo should by default ignore those even if you don't specify anything using --ignore flag.
If this is not working can you please post odo watch -v 4 output?

ok, I remember now that it needs to be handled at VS Code level https://github.com/redhat-developer/vscode-openshift-tools/issues/391


run  with craft.onnx

Hi,
thanks for this amazing work,
i'm running craft in 111 ms  on windows, but i want to get maximum  speed up  , so i want to use onnx and i'm new to that,
but i'm confused that which scripts should i use to test?
( i want to use craft.onnx model so i dont need to produce it )
in below code line it is .pth version of craft.
parser.add_argument('--trained_model', default='../models/craft_mlt_25k.pth', type=str, help='pretrained model')

Hey @sabrabano0  You can use this Notebook to convert and do inference with ONNX model. If you just want to do inference just use the last block of the Notebook

net.load_state_dict(copyStateDict(torch.load('craft_mlt_25k.pth', map_location='cpu')))
as above line you first load pth model and convert it to onnx after this line .
also your code is using craft in cpu mode  that is so slow.
your didnt use pycuda and tensorrt to run onnx?  am i right?

@sabrabano0 ONNX is a unified model format. It doesn't matter whether I load the PyTorch model in CPU or GPU while converting to ONNX. If I am loading the PyTorch model in GPU then I need to use inputs in GPU similarly for CPU. If you need to use Tensorrt then you should convert ONNX model to Tensorrt again

thank you for reply,
im new to onnx ,
i want a script that work on test image and get result with onnx model ,  in my search:  it needs tensorrt and on windows tensorrt didnt have any pythonAPI, so i cant use it
but i want to know can i use onnx without tensorrt? dose it have good exe time for an image?
i ran craft on cuda and my avg time is 111ms
i want better exe time
craft_inference block that you mention in first reply is not what i mean.,
any suggestion is  appreciated
thank you.

I think you misunderstood about ONNX. For running the ONNX model you don't need Tensorrt.
To decrease the latency you can first try converting the ONNX model to Tensorrt and then you need to do inference with the Tensorrt model on Nvidia GPU(Tensorrt is only optimized for NVIDIA GPUs). I don't know the support of Tensorrt on Windows machine. You can try using Google colab if you want to do inference with Tensorrt easily

we can run onnx  models on  various ML frameworks , converting the models is not the important subject here, when you decide to use tensorrt you convert onnx to trt ,  i know about that, base subject is about  finding best framework  or  info about exe time of running this project!
i want to run it on tensorrt to get best performance, i asked about time you get by running this on your selected framework, in your code you imported tensorflow backend but didnt use that , i think your repo is incomplete or you didnt run craft and just converting models.

This repository aims to convert PyTorch pre-trained model to TFLite. I think you did not seen the Repository files completely. CRAFT_TFLITE.ipynb contains end to end code for conversion and inference of TFLite model. If you have any queries or issues running TFLite model please go through other issues as some queries are already addressed. Regarding timing of the model please go through the README as it is already mentioned. Please go through the README and complete project and let me k


Remove TCC chinese comments

#732  Remove TCC chinese comments

Codecov Report

Merging #734 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #734   +/-   ##
==========================================
  Coverage      32.78%   32.78%
  Complexity       927      927
==========================================
  Files            229      229
  Lines           8949     8949
  Branches        1090     1090
==========================================
  Hits            2934     2934
  Misses          5667     5667
  Partials         348      348

Impacted Files
Coverage Δ
Complexity Δ

.../rm/tcc/remoting/parser/DefaultRemotingParser.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...m/alibaba/fescar/spring/util/SpringProxyUtils.java
5.66% <ø> (ø)
3 <0> (ø)
:arrow_down:

...m/alibaba/fescar/rm/tcc/remoting/RemotingDesc.java
65.21% <ø> (ø)
9 <0> (ø)
:arrow_down:

...ibaba/fescar/rm/tcc/api/BusinessActionContext.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...a/fescar/rm/tcc/interceptor/ActionContextUtil.java
52.77% <ø> (ø)
5 <0> (ø)
:arrow_down:

...alibaba/fescar/spring/util/TCCBeanParserUtils.java
44.68% <ø> (ø)
8 <0> (ø)
:arrow_down:

...aba/fescar/rm/tcc/api/BusinessActivityContext.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...in/java/com/alibaba/fescar/rm/tcc/TCCResource.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...r/rm/tcc/interceptor/ActionInterceptorHandler.java
25% <ø> (ø)
7 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Download ImageNet images automatically

Modify 2_one-pixel-attack-imagenet.ipynb to download a sample subset of ImageNet files with appropriate labels. The files should be preprocessed as to be ready for the attack.

Hi,I also download a sample subset of ImageNet files by the given file 'imagenet_urls.txt', but some problems to be found:

some pics' urls has been invalid
the model cannot recognize the image correctly even without attack...
Thanks for your attention!

imagenet_urls.txt is just a sample file with some example URLs and is by no means finished or ideal for testing.

About the URLs being invalid: yes, this is a very common occurrence in the ImageNet files, and there should be additional logic that simply ignores URLs that fail to download.
Make sure you are processing the images similar to what is shown in the existing Python code. Also, note that the current model (keras.applications.MobileNet) is not particularly accurate, so you may get inaccurate results on a small sample image set.

It would also be nice if we had more URLs. This can be accomplished by downloading the URL files from the ImageNet database and sampling them randomly. Something under 5 MB of URLs could suffice. The labels for each of the images would also need to be stored somehow.

I am very glad to get your reply so soon!
Coincidently,I have the same idea to refine the valid URLs by adding some logic and it works,but it comes out a new problem that some label(s)(e.g."n09337253") are not within the ImageNet1000.As the result,I finally just  download very few images successfully.

Finding a good set of URLs with valid labels would be necessary to complete this. Send a pull request if you are able.


add define

  (1 .. 10) |> define(
    a = it + 1
    b = a +2
  )
  .map(b)
  .toseq()

it's really hard to implement, I think saving into tuple next to the iterator variable is good solution
for example instead of above code, you can write:
  (1 .. 10) |> map((it, it+1, it+1+2))
  .map((it, a, b) => b)
  .toseq()


chore(rust): improve logging of ockam_command's session keep-alive code

Requesed by @mrinalwadhwa -- only logical change is that I noticed we retry immediately on failure to start session. I added a small sleep, so that we don't behave badly in unexpected cases (we don't want to spam logs, saturate the network, etc).

Looks great 👍


fix issue with slycot balred change in state

This PR fixes the CI errors that we are getting with slycot <IP_ADDRESS> by checking on some different transformations to switch around states.

Looks good, thanks!
The X DISPLAY error is back on one of the runs.


rest.ino: "EL-Client sync failed!"

Hi! So I'm trying to get SLIP working and have been playing with the rest.ino example. Unfortunately, the only thing that comes out of it is:
��␁␀␀␀�␄␀␀�6�EL-Client sync failed!

My serial speed is set to 57600, which is also the speed set at /console.html and the one I have to use for uploading (for some reason, 115200 isn't working. As that sketch explicitly mentions that the serial speed is critical to making SLIP work: is there another setting somewhere that need to be changed? If not, what else could I being doing wrong, given that both uploading sketches as well as the remote serial console are working fine and just SLIP is not?
As per the recommendation on the releases page, my ESP-Link version is 3.0.14; my EL-Client is HEAD of this repository's master branch.

Did you enable SLIP on the services page?

Yes, I wouldn't have written an Issue if I hadn't. By now I've forgotten what project this was for, though, and it'd probably be best to try and reproduce with the current master and then just close this Issue. It'll be a few weeks before I will make it back to the bench, so feel free to just close it now.


fix(version): update fabric8-stack-analysis-ui to 0.0.13

There were the following issues with your Pull Request

Commit: e4392f592090fca7ff5dde96bf6266e4b9871e3c
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


blog: rss/atom feeds include markdown

Here is the Atom feed - the RSS has the same issue.
curl https://golang.github.io/dep/blog/atom.xml

This gives (partial included here):
        <summary type="html"><![CDATA[v0.4.1 of dep has been released - and along with it, this site for documentation and announcements about dep! And, being that it's been nearly six months since [the last dep status update](https://sdboyer.io/dep-status/2017-08-17/) (which are now off]]></summary>

Note that there is markdown in the summary.

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


FormForBuilderDirective find button bug

bug html demo:
<form form-for-builder validation-rules="schema" submit-with="submit()">
  <submit-button label="Button"></submit-button>
</form>

FormForBuilderDirective:
if ($element.find('input[type=button], button').length === 0) {
    htmlString += "<submit-button label=\"Submit\"></submit-button>";
}

edit:
find submit-button
if ($element.find('input[type=button], button, submit-button').length === 0) {
    htmlString += "<submit-button label=\"Submit\"></submit-button>";
}

Haha, that's kind of funny, although I apologize for not catching it.
Thanks for reporting this, @ddesign84. New release 3.0.3 should fix this for you.


Drop down options field with assist edit not showing correct value

Error when there is a Option drop down field with an assist edit on a list. After selecting a value via the option drop down, the value does not display correctly.
Possibly linked to #3786
Example:

Go to Change Log Setup.
Click Setup > Tables.
Change the Log Insertion value on any row.
Select another row and then go back to the previous row and it will be blanked out.
Also, try to select blank option in 'Log Insertion' and it will not let you.

Expected behavior
Value should be displayed when you select the row.
Should be able to select blank option for row.

Versions:

AL Language: Vsix 2.0.56865
Business Central: W1 14.0 (26140)

Hi @AddEleven ! Thank you for reporting the issue and for the instructions on how to reproduce. This is what I get:

Are you experiencing the same?

Hey @atoader,
Yes that is the issue. As well as not being able to select blank Log Insertion value after already inserting a different value

This issue has been fixed in the Wave 2 2019 release.


ci: separate stages for unit tests and flaky tests

Following #4606, we'll have more than one type of test task. We'd like CI to run all the tests. When checking a PR it should require all the well-behaved tests to pass. A failure among the less-reliable tests may mark the build as "unhealthy" but not failed.
If we want to do this for the engine, the place to do it is Jenkinsfile.
The Jenkinsfile for modules lives in https://github.com/MovingBlocks/ModuleJteConfig/blob/master/Jenkinsfile
Open Questions

[ ] Do we want to do this for engine tests? It does have some TteTest cases, but are they flaky enough to demote to lesser importance?

Things to Watch Out For

Make sure all the test results are recorded in Jenkins. i.e. splitting them up in to separate tasks doesn't accidentally make it so only the results of one or the other of those tasks are saved.
People have built tools to do elaborate monitoring and treatment of flaky tests. Resist going down that rabbit hole! 🕳
We don't want to make it too easy to accept test flakiness too quickly. Save your time for other things, like fixing flaky tests!

 Separate Jenkins test stages

you can see what all check will run by doing gradlew check --dry-run
It looks like gradlew check --exclude-task test might work if we want to have a "all the checks except tests" build step without explicitly naming each of those checks.

Also, to have the reference here (I need to make answering stuff preferably on GitHub a habit):
since https://github.com/MovingBlocks/Terasology/pull/4611 got merged, we now do have three different gradle tasks:

unitTest running only unit tests
integrationTest running only integration tests (MTE, TTE, etc.)
test running both, unit and integration tests - so basically all tests

See also changes done previously in https://github.com/MovingBlocks/ModuleJteConfig/commit/e64657b9d83284a6c272dcce265821a4d5024e16


findRelatedTests option doesn't use a =

Summary
The documentation incorrectly stated that the --findRelatedTests cli option expected an = before the list of files.  It does not, and this corrects that documentation
Test plan
No testing necessary, documentation only.  However, it would likely be good to provide a usage example for this somewhere in the docs.  It took me a while to figure out what the usage was between the following:
jest --findRelatedTests='foo.bar, foo.baz'
jest --findRelatedTests='foo.bar foo.baz'
jest --findRelatedTests 'foo.bar, foo.baz'
jest --findRelatedTests 'foo.bar foo.baz'
jest --findRelatedTests foo.bar foo.baz

Admittedly, that was mostly due to the misleading doc.

Codecov Report

Merging #3009 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3009   +/-   ##
=======================================
  Coverage   67.86%   67.86%
=======================================
  Files         147      147
  Lines        5349     5349
=======================================
  Hits         3630     3630
  Misses       1719     1719

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 445663b...431dbe6. Read the comment docs.

Thank you @benmccormick


One of the hyperlinks in Documentation Home not working

Describe the bug
Hyperlink for "The BioCompute Object working group" showing "404 page not found"
To Reproduce

Go to https://docs.biocomputeobject.org/
Find and click on 'The BioCOmpute Object working group' hyperlink
See error

221ac9e


[BUG] PE wrappers missing Address Space Layout Randomization (ASLR) and Data Execution Prevention (DEP)

setuptools version
60.5.0
Python version
Version independent
OS
Windows x64
Additional environment information
No response
Description
The PE file wrappers (cli-32.exe, cli.exe, gui-32.exe
gui.exe) for launcher.c were built with Visual Studio 2008, and the current msvc-build-launcher.cmd build script still targets this compiler release. These old executables (presumably built when Python 2.7 support was required) do not enable ASLR or DEP, increasing their attack surface.
Locally, I was able to update the compiler to VS2017 and recompile the executables with default settings which enabled DYNAMICBASE and NXCOMPAT without further steps. I can work on a patch, but wanted to check with the setuptools developers what the expected VS toolchain should be, and if there are any other considerations in making a patch for this issue.
Expected behavior
ASLR and DEP are enabled for the shipped executables.
How to Reproduce
On a Windows machine with Visual Studio installed:

Start a Visual Studio Native Tools Command Prompt session and navigate to setuptools directory in source or built distribution
Run dumpbin /headers gui-64.exe, and note the DLL characteristics under the OPTIONAL HEADER VALUES for the files:

            8000 DLL characteristics
                   Terminal Server Aware

An executable which has been built against a modern toolchain will instead display:
            8160 DLL characteristics
                   High Entropy Virtual Addresses
                   Dynamic base
                   NX compatible
                   Terminal Server Aware

Output

Code of Conduct

[X] I agree to follow the PSF Code of Conduct

I'm happy to update the script wrappers, although they're deprecated. I'd like to move to relying on distlib for script wrappers. There isn't any formal tool chain for building the wrappers. I do insist that I or one of the maintainers do the actual building, so what we need from you is (a) a PR with an update to the build scripts and (b) instructions or a Docker image that can build them.

This might be relevant: https://bitbucket.org/vinay.sajip/simple_launcher/issues/9/ci-and-meson-support

This has been completed with the work done on https://github.com/pypa/setuptools/pull/3760 and the subsequent executable rebuilds. Thanks!


Update Environment.yml to fix OpenSSL bug

New version of OpenSSL breaks Snowflake connector.
https://community.snowflake.com/s/question/0D5Do000010NAVzKAO/hi-snowflake-teami-am-facing-issues-while-connecting-to-snowflake-using-snowflake-connector-with-oscryptoerrorslibrarynotfounderror-error-detecting-the-version-of-libcrypto

Published to main repo


switched epel from 7-6 to 7-7

Change current EPEL release which is hard coded in 3 locations.

This will cause a bug in the next epel release.  Advise option of "latest."

Fixed by https://github.com/redhat-openstack/openshift-on-openstack/pull/161


:technologist: 添加dependabot

Codecov Report

Merging #7 (09c4bc5) into main (a0ebaa2) will decrease coverage by 16.71%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##              main       #7       +/-   ##
============================================
- Coverage   100.00%   83.29%   -16.71%
============================================
  Files            4       13        +9
  Lines          145      437      +292
============================================
+ Hits           145      364      +219
- Misses           0       73       +73

see 14 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Makefile: allow LIBRE_MK to be set outside

E.g. for cross compilation it may be necessary to specify the location of
re.mk where make is called.
@sreimers Same solution in baresip/Makefile. What I have found by reading Makefile doc and by small test Makefiles is that the ":=" always overwrites the variable. Setting LIBRE_MK from outside  was not possible. So it seems to be the correct solution?

Good catch, merged :-)

thanks!


Support for start-of-activation log sentinel

Description

Add support for filtering out an optional start-of-activation log sentinel
line that will contain the activation id and namespace in addition to the
sentinel marker.  For runtimes that inject this sentinel line into their
outputs, it becomes quite simple to do an asynchronous stream processing
operation on their log streams and forward log lines enriched with activation ids
and associated with the appropriate namespace to a platform logging service.
Appended is an example from an action that writes one line to stdout and no lines to stderr:
stdout stream:
XXX_THE_START_OF_A_WHISK_ACTIVATION_XXX with id cafca5b74be94eb8bca5b74be9beb80f for namespace guest
Here's a friendly message
XXX_THE_END_OF_A_WHISK_ACTIVATION_XXX

stderr stream:
XXX_THE_START_OF_A_WHISK_ACTIVATION_XXX with id cafca5b74be94eb8bca5b74be9beb80f for namespace guest
XXX_THE_END_OF_A_WHISK_ACTIVATION_XXX

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ x] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ x] I updated the documentation where necessary.

A proof of concept implementation of the runtime changes for nodejs is in https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/81

Codecov Report

Merging #3974 into master will decrease coverage by 4.69%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #3974     +/-   ##
=========================================
- Coverage   85.29%   80.59%   -4.7%
=========================================
  Files         146      146
  Lines        7057     7061      +4
  Branches      413      417      +4
=========================================
- Hits         6019     5691    -328
- Misses       1038     1370    +332

Impacted Files
Coverage Δ

...containerpool/kubernetes/KubernetesContainer.scala
92.3% <100%> (+0.64%)
:arrow_up:

...sk/core/containerpool/docker/DockerContainer.scala
97.53% <100%> (+0.06%)
:arrow_up:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.1%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-81.82%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
0% <0%> (-58.83%)
:arrow_down:

...la/whisk/core/database/cosmosdb/CosmosDBUtil.scala
92% <0%> (-4%)
:arrow_down:

...rc/main/scala/whisk/common/ForcibleSemaphore.scala
88.46% <0%> (-3.85%)
:arrow_down:

Continue to review full report at Codecov.

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

rebased to resolve conflicts with merged PR #3909

This has been sitting in WIP, got stale and is inactive since 2 months. Any update?

Not much has happened.   I do still think having a start sentinel is potentially useful, but there were some additional wrinkles I hadn't considered that came up in discussion of this PR.

We can't trust the namespace information in the start sentinel (malicious users could write actions that generate many spurious start sentinels to their action's logs targeting another namespace).  So we still need an additional mechanism to associate the namespace for the log.
Depending on implementation details of the underlying container engine, we may need a way to block deletion of the container until log processing has completed (or we could lose user logs).  This means there could still need to be some coupling between log processing and the invoker's container management state machine, which reduces the potential simplification of having purely streaming logs.

since it was easy, I rebased to resolve conflicts & squashed back to a single commit.


Allow to change renderer to WebgpuRenderer

Starting with Chrome version 113, webgpu is basically available without flags.
As a result, three.js' webgpurender is also actively being developed.
(https://github.com/mrdoob/three.js/commits/dev/examples/jsm/renderers/webgpu)
I think it would be good to include an option to change renderer to webgpurender in the Canvas component.

Hey @seo-rii,
with about 15% of global availability we're unfortunately still not close to a wide adoption. There's no definitive threshold for me, but it's not exactly a priority. There will be a transitional period where we want to provide the best possible DX in terms of choosing the renderer. If you have any plans and ideas how we should do that, feel free to propose changes or start a discussion on our Discord.
It is of course possible to easily change the renderer with the hook useRender in Threlte 6:
const renderer = new WebGPURenderer()
useRender(({ camera, scene }) => {
  renderer.render(scene, camera.current)
})

@grischaerbe I definitely understand your opinion. I just created this issue because there wasn't any issue about it.

Yep and I appreciate it! I'm leaving it open to be aware of it 👍

In the six months that have passed since June, it looks like we're up to 28.5% of all users, and 66.61% of desktop users!
I still think there's a while to go before this becomes too important. Maybe when either Safari or Firefox support it by default, it'll be time to revisit this?

https://stackblitz.com/edit/vitejs-vite-rmq
@MattFerraro Here's a stackblitz showing a simple working WebGPU + Threlte setup. It's still a bit clunky when replacing the default renderer and the need for a custom resizer, but it's possible. There's definitely still more that we can do before the WebGPURenderer becomes the default to make swapping renderers more seamless.

Let me know when there is a way to do this in Threlte so I can add Threlte to https://github.com/verekia/three-gpu-ecosystem-tests

Definitely anytime after Threlte 8 is released. Currently you can using our @next release.
Also, 72.46% global webgpu!! :tada:


button_handleClick setTimeout not triggered for Enzyme test

downshift version: 1.31.12
node version: 8.x
npm (or yarn) version: 6.x

Relevant code or config
What you did:
We have unit test that simulates a click and test the dropdown populates the right options.
enzymeWrapper.find('#randomButtonID').simulate('click');

What happened:
Our getToggleButtonProps passed in onClick function is working as designed but the button_handleClick fuction is not working right and isOpen state is never updated. https://github.com/paypal/downshift/blob/master/src/downshift.js#L548
onClick: composeEventHandlers(onClick, this.button_handleClick)

I debugged the latest few releases the found that the setTimeout function added 12 days ago is never triggered.  If I take the setTimeout out, the test will immediately pass. https://github.com/paypal/downshift/blob/master/src/downshift.js#L574
    setTimeout(() =>
      this.toggleMenu({type: Downshift.stateChangeTypes.clickButton}),
    )

Reproduction repository:

Problem description:
Since toggleMenu is never called, the isOpen state is always false and no drop-downs are populated in our unit test.  This has worked in v1.31.7 but fails after v1.31.9.
Suggested solution:
I did manage to bypass the test with mock timer setup, something like
    jest.useFakeTimers();
    enzymeWrapper.find('#randomButtonID').simulate('click');
    jest.runAllTimers();
    enzymeWrapper.update();

but is the setTimeout necessary? just to unit test a simple click do we have to always mock the timer?
I'm not familiar with ios blur or not sure why a setTimeout is needed in this case, maybe another solution https://codereview.stackexchange.com/questions/47889/alternative-to-setinterval-and-settimeout?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa

Yeah, it's unfortunate that this was necessary and I'd love to come up with a different solution if possible. Do you know if requestAnimationFrame will really solve your problem?
If not, perhaps we could add an if check to not use the timeout if process.env.NODE_ENV === 'test' or something 🤔

@kentcdodds Thank you for your reply.  I have not personally tried requestAnimationFrame as I'm not doing any ios development, wonder if someone here more knowledgable would know.
The if check might be a good idea.  Or if we can have a check to call only for ios/mobile? does process.env.BUILD_REACT_NATIVE do that job?

I'm pretty certain that the original bug was ios mobile web, so BUILD_REACT_NATIVE wont work for that case. But I do feel like the NODE_ENV check would be acceptable. I'm 99% certain that I'd accept a PR for that :)

Ok I will draft up a PR tomorrow, unless someone figure out an alternative in the next couple hours :)

I'll have a look around today and see if I can come up with a better solution.
If we can avoid this focus (in the click handler) triggering the blur event in mobile, then we can avoid the setTimeout. Just will require a bit of testing on different devices. Will let you know.
    if (
      this.props.environment.document.activeElement ===
      this.props.environment.document.body
    ) {
      event.target.focus()
    }

@aMollusk, if you can get that to work that'd be swell. @nicoxxie made a PR which I just merged to resolve this using the NODE_ENV check. We can move to something better if that comes along :) Thanks!


instance: remove manage.py script

Remove manage.py script from templates and rely only on installed cli.
--
Further reading http://click.pocoo.org/5/setuptools/

:+1:


Add holes support for Geometry.decompose_polygon_in_convex

Resolves #35676 as an alternative method to solve the use case described by @fian46 which can also be done via script bypassing NavigationPolygon.make_polygons_from_outlines which mostly does the same thing under the hood, yet this also:

helps godotengine/godot-proposals#200 as the exposed method allows to make proper convex shapes to be added to arbitrary collision objects using the results from polygon clipping, as clipping may produce polygons which lie inside other polygons (holes);

see https://github.com/godotengine/godot-proposals/issues/199#issuecomment-549429967 for the polygon "donut" where the limitation is worked around by hacking out a keyholed representation of a polygon;

see #30251 for a similar use case.

A new polygons_inner parameter is added to Geometry.decompose_polygon_in_convex which allows to pass holes to the polygon decomposer, so this just makes the method more flexible and shouldn't break anything. The user has to separate the input into outer and inner polygons themselves (can be checked with Geometry.is_polygon_clockwise):
var clip = Geometry.clip_polygons_2d(a, b) # `a` may completely enclose `b`: clip.size() == 2
var outer = []
var inner = []
for poly in clip:
	if Geometry.is_polygon_clockwise(poly):
		inner.push_back(poly)
	else:
		outer.push_back(poly)

for outer_poly in outer:
	var convex = Geometry.decompose_polygon_in_convex(outer_poly, inner)

As an alternative, the implementation could also be reused similarly to how #26614 has done to CollisionPolygon2D::_decompose_in_convex(). It means that a similar method could be moved from NavigationPolygon.make_polygons_from_outlines to Geometry.make_polygons_from_outlines or similar, which would also alleviate the requirement for distinguishing between inner and outer polygons, and multiple outer polygons could also be fed, which allows to simplify the above script down to:
var clip = Geometry.clip_polygons_2d(a, b)
var convex = Geometry.make_convex_from_outlines(clip)

I'm not sure if there's anything about #34776 which would throw away existing navigation code, so yeah it might make sense to salvage this for other use cases. If you think it makes sense to do this instead of the proposed enhancement, I could close this PR and do necessary steps to make this happen, else this PR can be seen as feature-complete. This could also be done in the next PR too, as two methods can co-exist nicely, depending on what kind of input you have.
Test project
geometry-decomp.zip

I've been working on the GeometyTools module and since found a better way to distinguish between outer and inner polygons using the same underlying PolyPartition library:
https://github.com/Xrayez/godot-geomtools/blob/7287d22f667bc23f8f6e44d23d03295f05f1bd38/2d/polytools/decomp/polypartition/poly_decomp_polypartition.cpp
So I'm closing this particular implementation being inferior in comparison, unless needed.
Of course similar can be made within Godot, but I don't want to compromise stability of the engine regarding this feature, and there's really not so much use cases for this to be available as part of core.


Verify default repo config is updated

When tests are updated, the default config should match.

Coverage increased (+0.007%) to 37.942% when pulling d608b106152d17f656be259d9de65aacfe6917ac on viraptor:verify_default_config into<PHONE_NUMBER>71d81d0e4f6b136341c880adfc1bf4 on HewlettPackard:master.

Coverage increased (+0.007%) to 37.942% when pulling d608b106152d17f656be259d9de65aacfe6917ac on viraptor:verify_default_config into<PHONE_NUMBER>71d81d0e4f6b136341c880adfc1bf4 on HewlettPackard:master.

Coverage increased (+0.02%) to 37.951% when pulling 066c737536a0e08e9e0198420d88bcafc358716a on viraptor:verify_default_config into<PHONE_NUMBER>71d81d0e4f6b136341c880adfc1bf4 on HewlettPackard:master.


feat: persist rh, rap, nl

Hi,
I am working on fixing Aedes bridge connection where nl flag is not respected:
https://github.com/moscajs/aedes/issues/736
For that I need to update aedes-persistence to persist nl, rap and rh flags.
The tests are still failing because qlobber also needs to be updated to work with nl, rap and rh flags:
https://github.com/davedoesdev/qlobber/pull/24
So this pull request needs to wait until qlobber is updated. 👍

Changes looks good. I think there would need some changes also to (in order):

https://github.com/moscajs/aedes-cached-persistence
https://github.com/moscajs/aedes-persistence-mongodb
https://github.com/moscajs/aedes-persistence-redis
https://github.com/moscajs/aedes-persistence-level

cc @mcollina @gnought @seriousme

Changes looks good. I think there would need some changes also to (in order):

https://github.com/moscajs/aedes-cached-persistence
https://github.com/moscajs/aedes-persistence-mongodb
https://github.com/moscajs/aedes-persistence-redis
https://github.com/moscajs/aedes-persistence-level

cc @mcollina @gnought @seriousme

FYI: I'm working to add this feature to https://github.com/moscajs/aedes-persistence-level
Kind regards,
Hans

I have updated qlobber dependency to 7.0.0 and fixed the remaining failing unit tests so I think that the PR is ready 👍.

9.1.0 is out now. Thanks @oldrich-s

Dont know if it has any influence but aedes-packet dependency could probably be updated to v3.0.0 before releasing new version

Argg missed it, let me create a quick patch

Done


py.path has been renamed? (ModuleNotFoundError: No module named 'py')

My CI pipelines using tox-conda don't seem to work anymore as of recently, as I get the following error:
  File "/home/runner/work/xugrid/xugrid/.tox/.tox/lib/python3.10/site-packages/tox_conda/plugin.py", line 10, in <module>
    import py.path
ModuleNotFoundError: No module named 'py'

I have a suspicion this is because py.path has been renamed to path?
See: https://pypi.org/project/path.py/
and: https://pypi.org/project/path/

After looking a little more: it seems like py.path was being used before, but I don't see it mentioned in setup.cfg.
Maybe it was present due to an other dependency, but not anymore?

py.path is part of the py package that is a dependency of tox version 3 and tox-conda only supports this version for now.

Ah right, that explains it -- it would've taken quite some time to figure it out.
Maybe it's worthwhile to mention this in the README? Currently, I guess I could've figured it by checking the setup.cfg, but that's not quite obvious.
I'll install tox < 4. Thanks!

I'm surprised the current tox-conda could be installed with tox 4 because tox-conda has a constraint on the version of tox.
Do you know why this happens?

My ci.yml looks like this:
  build:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2

      - name: install mamba
        uses: mamba-org/provision-with-micromamba@main

      - name: Test and build with tox
        run: |
          pip install tox
          tox -e build
      - uses: codecov/codecov-action@v1

The start of my tox.ini:
[tox]
requires = tox-conda
isolated_build = True
setenv =
    CONDA_EXE=mamba

I think I could've avoided it by using pip install tox-conda instead of pip install tox.
In this case, it must be tox installing tox-conda. I'm not sure why I set it up this way, most likely because I started with a standard tox configuration, and then found a way to use tox-conda.

It turns out that when tox creates the provisional env to install tox-conda, pip find an old version (0.8.1) of tox-conda that has no constraint on the version of tox.
So it seems the only solution to this is what you already did, i.e. install tox<4


Ensure ProjectConstraints in manifests given to solver have a Constraint

The solver generally assumes it's impossible to have a nil constraint; that should, instead, be treated as an anyConstraint{}. This can cause panics if the implementing tool doesn't follow the rules, though, so we need to defend against constraint-less ProjectProperties.

This should also include a new badOptsFailure if the problem exists in the root manifest. ...I think.


Monorepo using NativeScript 7 (rather than NativeScript 8) causing type conflicts in NativeScript 8 demo apps

This is not a blocker, but does cause a lot of typings complaints in the demo app and around parts of the YogaLayout plugin.
e.g. since NativeScript 7:

the FontWeight enum in NativeScript Core has moved to a new file.
certain class properties in NativeScript Core have changed into ES5 accessors

I understand that there are a large number of plugins to maintain in this repo, so solving this problem might actually have detrimental consequences (i.e. migrating to NativeScript 8 may break compatibility with all of them).
Perhaps a middle ground would be for at least the YogaLayout component to specify NativeScript 8 in its package.json. Or we just keep this on the radar for the long-term.

The repo will be upgraded soon, I'm waiting on an issue to be resolved with how the packages are linked 😀


[DataPipe] Fix a word in WebDS DataPipe

Stack from ghstack:

-> #1156

Fixes #1144
Note that the description is already correct in the docstring of the DataPipe. This merely fixes the docstring of the helper function.

@nivekt has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Restock notification function

Hello,
Love vue storefront a lot and learning how to build my own theme for plugging vs to my magento system.
Would like to know if magento's 'Restock notification' function will be activated in vs later too?
Thanks a lot!

@filrak it will be hard for me to take it (due to time constraints) I'll leave it unassigned maybe somebody can take care sooner

@michhy this feature isn't yet planned on our roadmap, would be great if someone could take care of it

thank you piotr, just a nice to have function only :)


Add files via upload

I just wrote an small documentation for this app. Can you please do short review ? Also, I would like to know how should I start with programming with std. schema ?

Hi, first of all, I think it is better to use the Wiki for such documentations, because Word files need to be downloaded to view and Github itself offers a Wiki feature.

Feedback
Requirements: The requirements are lacking information about which informations a task should contain (e.g. Title, description, due date, status, image?). Requirements should not limit the implementation in any way, so telling that the web page should use Django or flask is limiting your possibilities. This should rather just say something like: "Later the task scheduler should have a GUI e.g. web frontend".
Workflow documentation: The bullet points in my opinion are also requirements and should be listed in the requirements section. The workflow diagram looks fine, but there is no info what will happen, when the due date is reached, like some kind of notification.
Software architecture: You mention that web GUI is added later. What is used before that? Here I'm missing a command line as the output for the first version.
Software Component Design: For simplicity I would omit the upload in the first version of this diagram. First, there is not really an upload happening, and second, we can also add this GUI part at a later point and think about it in an isolated manner.
I am missing an entity like "TaskList". That is responsible for containing all tasks. Also, I would add methods to the Task class for storing, updating and deleting the task.
It seems like you don't really know what to configure, when I look at Config. So why don't we just drop that for now?
Maybe for this app the logging is also not that important or let's say at the moment. If we want to log somethign we can also just print it to the console. No need for an actual logging.


feature request: Multiple audit log storage

Please describe what you would like to see in ContainerSSH
Support multiple audit log storage in one containerssh instance.
Please describe your use case
I expect to keep the audit log in s3 and the local file system.

Because I may need to have a local copy of the audit log, to analyze, and a cold copy on S3.
Maybe it is better to have a webhook for audit log too, just as config and auth did.

Actually, I want to know which machines are idle (e.g. there is no ssh connection for 30 mins) from the audit logs in sshproxy mode.
I am not sure if web UI works. But the library for analyzing the logs is helpful.

This would best be done by analyzing the data from the Metrics endpoint. If data is missing there, we need to add it.

I did not notice it. Thanks for the info.

Should we keep this issue or close it?

This would best be done by analyzing the data from the Metrics endpoint. If data is missing there, we need to add it.

Metrics looks like a summary. I expect to get the machines' IP. It may not work.


HAWQ-1292. Change GUC enable_ranger(bool) to a text GUC(hawq_acl_type…

…), which can allow other kinds of ACL.

@zhangh43 @ictmalili  @xunzhang    please help to review. Thanks!

+1

Merged into master. Close this PR.


Remove version: Jackett.Jackett version 0.20.3939

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.3939/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


NuDB with SHAMAPV2 enabled

I deployed a chain on linux with SHAMAPV2 enabled and the node_db type is NuDB. Ripple version is 0.80.1.
But there is always a storage problem: when I open a new node to synchronize data，the ledger cannot be fetched if there are transactions on it. It reports:"nodeobject codec v2: short inner node size: " .
I don't know why ,is SHAMAPV2 not compatible with NuDB ？

Are you deploying your own alternate net?

You can think that I was deploying an alternate net, but that's not the key. The problem is I don't know the compatibility of SHAMAPV2, is this feature conflicts with NuDB ? Is there problem if I use RocksDB and SHAMAPV2？

Well, I don't know, but SHAMAPV2 isn't deployed on the main net :)

I see, thanks @sublimator
I launched a node to connect the main net, and use "feature" command to look up the status of new feature, I found that none of the new features are enabled, I wonder why.

No idea, many on the rippled team are quite conservative, with decades of
experience. Probably not rushing into anything with all that money at stake.
Move fast and ... break things? :)

   case 5: // compressed v2 inner node     {         auto const hs =             field<std::uint16_t>::size; // Mask size         if (in_size < hs + 65)             Throw<std::runtime_error> (                 "nodeobject codec v2: short inner node size: "                 + std::string("size = ") + std::to_string(in_size)                 + " hs = " + std::to_string(hs));
It seems the condition is not right, it assumes there are at least two txs, if there is only one tx in this ledger,it will throw an exception, and process dumped because there is no try-catch. If I change 65 to 33,the code runs well ,and I can get the ledger successfully.

That's ok, thanks @sublimator .


about page draft complete

Please pretty please accept my pull request

I have a couple of bonus formatting suggestions, but otherwise :shipit:


Create function in single or multiple .py files

Should we create our functions in the same .py file since all of our functions are independent of each other?

Hi Manvir. We can do that but I remember when we did IA1, the authors created different py scripts for different functions under the same folder. I think we can do the same because if we all work on the same py script, we might  encounter merging conflicts quite often if we want to make a minor change.
├── src
│   └── pycounts
│       ├── init.py
│       ├── plotting.py  <--------
│       └── pycounts.py
Although pycounts has three functions, the final one depends on the first two. Since all of our four functions are independent, it is not necessary to put them in the same script. We can put them in the same script in our final version of the project as no more changes are needed.

Added my function to a separate file


python38Packages.pysoundfile: 0.10.2 -> 0.10.3.post1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.8-soundfile/versions.
meta.description for python38Packages.pysoundfile is: '"An audio library based on libsndfile, CFFI and NumPy"'.
meta.homepage for python38Packages.pysoundfile is: '"https://github.com/bastibe/PySoundFile"

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.10.3.post1 with grep in /nix/store/6q6g4n71742p8p06qmj67ah236d5ry1w-python3.8-soundfile-0.10.3.post1

directory tree listing: https://gist.github.com/0dcf86e1215e6a6138303b6230a3f19a

du listing: https://gist.github.com/b8d4ea6c4da072f54fb0b87093d0fad5

Rebuild report (if merged into master) (click to expand)

27 total rebuild path(s)
9 package rebuild(s)
9 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
9 x86_64-darwin rebuild(s)
9 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
python27Packages.librosa
python27Packages.pysoundfile
python27Packages.soundfile
python37Packages.librosa
python37Packages.pysoundfile
python37Packages.soundfile
python38Packages.librosa
python38Packages.pysoundfile
python38Packages.soundfile

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6q6g4n71742p8p06qmj67ah236d5ry1w-python3.8-soundfile-0.10.3.post1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python38Packages.pysoundfile https://github.com/r-ryantm/nixpkgs/archive/60e76c93c275cdf31aed586b4fbb0cc01089b84f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6q6g4n71742p8p06qmj67ah236d5ry1w-python3.8-soundfile-0.10.3.post1
ls -la /nix/store/6q6g4n71742p8p06qmj67ah236d5ry1w-python3.8-soundfile-0.10.3.post1/bin

cc @fridh for testing.

@GrahamcOfBorg build python27Packages.librosa python27Packages.soundfile python37Packages.librosa python37Packages.soundfile python38Packages.librosa python38Packages.soundfile


Add bindings for consistent_with, middle_names, middle_initials and first_and_last_initials

Hey @amokan ,
My colleague @rogerbooth added more bindings to the library. He's off for vacation, so I'm submitting this PR on his behalf.
Please let us know if you're happy to accept those changes upstream.
Best,
Yevhenii

hey @amokan ,
would you have any time soon to look into this PR?

@lessless thank you for the reminder. I've been meaning to respond to this.
I will do my best to look this over and get a new release cut (assuming all checks out) over the weekend. If I cannot make that happen before Monday, I'll drop a note in here.
Thank you for the contribution!

@lessless - your PR has been merged in and I've cut a new release of the hex package (v0.6.0).
I did make some adjustments for the sake of consistency by changing your middle_initials to middle_initial (which also aligns with the rust package).
I also added tests for the new functions and flagged the initials/1 function as depreciated to avoid a hard removal of that right now.
Your work is also cited in the changelog.
Thanks again for the contributions!


Add someone to GitHub

GitHub ID to add: @alexandratran (myself)
Project to be added to: Hyperledger Besu docs
Team to be added to: Besu docs maintainers

I had some technical issues and had to disable and reset 2FA for my GitHub account, which kicked me out of the Hyperledger org. Looking to regain access to the org and the Besu docs maintainer team.

Invite sent!


[DUOS-2185][risk=no] Use custom Gson adapters

Partially Addresses
https://broadworkbench.atlassian.net/browse/DUOS-2185
This is only one component of 2185 - for the ticket, custom Gson adapters needed to be added. This PR just has the Gson adapters and updates the Gson object of all of the places (I believe) that broke due to the addition of the FileStorageObject. This needs to be merged first before the second half of the ticket can be merged.

Have you read CONTRIBUTING.md lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

Nice work Connor!  I would recommend test coverage on the GsonUtil as well.  A couple of tests ought to do it (one for Instant, one for BlobId, one for something with both of those elements and one for something without those elements).


As a user, I want to easily display the products objects I am getting

Checked for duplicates
No - I haven't checked
🧑🔬 User Persona(s)
Data user
💪 Motivation
...so that I can quickly see what it is about before doing my specific processing on them.
📖 Additional Details
No response
Acceptance Criteria
Given a product returned from the peppi iterator
When I perform print the returned object in a jupyter notebook
Then I expect to have a quickllok of the metadata associated with the product, possibly quicklook of the data as well
⚙ Engineering Details
We should use this protocol: https://ipython.readthedocs.io/en/stable/config/integrating.html#rich-display
🎉 I&T
No response

@tloubrieu-jpl can we add some specifics here? what types of products are we referring to? and is this just for displaying metadata or are these quicklooks of the data products as well? if it is the latter, we will want to break this up into several requirements based upon the product type.

@jordanpadams I did not have final ideas on that yet.
But yes we could break it down:

generic metadata display (for any type of product)
a specific collection display with statistics possibly on the data (number of products, time ranges...)
specific observational products display with specific statistics (ranges) or quicklooks

We could just start with the first option to experiment with the concept.
2 or 3 could be something interesting to get contribution from the community.


⚠ Bofrost.at has degraded performance

In 593c8fb, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 3555 ms

Resolved: Bofrost.at performance has improved in b5a9242 after 54 minutes.


forbid-prop-types doesn't work outside Component.proptypes

if you have something like
const propTypes = {
  foo: any,
  bar: object,
  baz: array,
}

...

MyComponent.propTypes = propTypes

forbid-prop-types won't error out because it checks only for proptypes directly declared on MyComponent.propTypes

It should definitely follow in-file propType declarations like that.
Can you try using PropTypes.object etc, rather than destructuring things off of PropTypes, to confirm whether that's the issue instead or not?

yeah it's the import destructuring:
import React, { PropTypes } from 'react'

const { string, any } = PropTypes // doesn't work
const { string, any } = React.PropTypes // works
...

Is there something new about this issue?

This is already fixed. https://github.com/yannickcr/eslint-plugin-react/blob/e835aaf52019db06f16342f44fffb12d75e79856/tests/lib/rules/forbid-prop-types.js#L784-L790


[WIP] Improved Validations

Pull Request Checklist

Description

This improves several validations being performed.
First verify that role_ref for the role binding types is just a String. Those are the properties that don't match exactly to Sensu Go specs so this validations ensures we're getting just the role name and not a Hash.
The second validation is executed once the catalog is compiled and will check the catalog to ensure the defined namespace is in the catalog.
Related Issue

This addresses some issues brought up in #1131

I marked WIP because I want to see if it's possible to check both catalog and sensuctl for available namespaces.

Namespace validation will check existing namespaces (defined outside Puppet) and ones defined in Puppet, one of those must exist to pass the validation

Reverted commit that forced role_ref as string because that's going to have to be changed in upcoming pull request.


Added Functionality

Added Definable Endpoints to getReferralLink() defaults to 'register?referral=' - see config/referral.php
Added getRefererName() Does what is says - Returns the users referers name
Added getReferredUsersCount() Returns the number of times a users affilate code has been used

There's no valid reason for this change, and now there is a new branch: 2.0


Type-checker only runs over the current file.

Currently when the type-checker runs on save it only runs on the current file. If the types in other files don't match up anymore because of this you do not see that there are errors. It also works the other way around. When you change a file that fixes type errors in other files, the old type errors are still shown.

In the image above I have fixed the error however it is still shown until I close and reopen the file.
Steps to reproduce:

have two reason files, A.re and B.re. A.re calls a function in B.re.
open A.re and B.re so that they are both open.
change the function in B.re to make it return a different type which won't work where A.re is using it's output.
go back to the A.re file but don't see any errors.
close A.re and open it again and now see the errors

This is an unfortunate limitation due to the way that Merlin works, which is the tool we use to gather the diagnostics such as type errors. Merlin does not have a global view of the project at this point in time and there isn't an efficient way for the editor to work around that. It is possible to use bsb (from Bucklescript) instead and it should take into account changes in other files. That means you will need to be using Bucklescript though (or switch to bsb-native). I don't use those configurations myself so someone else could perhaps provide more detail. cc @jchavarri

I am using Bucklescript so If anyone has anymore information on how to set this up that would be great.

@anisjonischkeit You can enable bsb as mentioned under the "Configurations" section in the README. For other configuration issues, if you join the discord channel (see here) there are usually people around that can help.


Tabs text cut off on small devices

The layout as such seems to be very well responsive, including the navigation. But this seems not to be true for the tabs. If the content of the tab navigation is wider than the screen, the text of each tab just gets cut off a bit. See secreenshot below (in contrast, the desired behaviour as it is the case for the navigation, would be that the tabs are scrollable, as it is the case in Angular Material):
Example:
https://de.webcams.travel/api/#api

I cannot reproduce this under the current version of your website.  I loaded it in chrome, set the screen to 300x400 and it responds as it should - it truncates the right side and allows scrolling to view the other tabs.

The second tab navigation is cut off. There it should say "ÜBERSICHT", "VERSION", "FEATURES", ...


Windows compilation fixes and Visual Studio project files

This PR adds Visual Studio project files to Vina to be able to compile a Windows version. It expects the current Boost 1.77 version installed in C:\local\boost_1_77_0 (default for the precompiled binary version) for either x86 or x64.
Additionally, some small fixes where needed for VS to successfully link everything (struct and class were sometimes used interchangeably) and for "non-traditional whitespace" (\t, \v, \r) to not spoil reading pdbqt files on Windows.
I tested our examples successfully on Windows and macOS with this version.

Please also add windows to ci. Thank you.

Thank you for the windows ci.
I have small suggestion. You can use cmd instead of PowerShell.
In ci there is windows 10, it have a curl.exe to use there.

Let's delete "forcefield.h" and "forcefield.cpp" because they are not needed :)
Related to #23, #27, and globing "*.h" *".cpp" in https://github.com/microsoft/vcpkg/pull/19645

Acceptable solution. Will be from next release.

@diogomart Agreed and done.

Can you restart windows ci?

Thank you.

This PR still needs some work on my side (python wheels for Windows), I am temporarily convert it to draft.

My vcpkg PR was merge. Please merge this PR for consistency.
Thank you.

@jeeberhardt Merging - thank you for our quick off-channel talk ;-)


Bug #457

deactivate invite only streams when last user removed from the stream.

Automated message from Dropbox CLA bot
@devstorm21, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Hi, @devstorm21 -- thank you for the pull request! I clicked through to see the "Details" of why the Travis CI build failed, and https://travis-ci.org/zulip/zulip/jobs/165986463#L1965 tells me that you have accidentally deleted the newline at the end of the file. I suggest you make a new commit on your Bug457 branch that adds a newline, run the linter and other tests locally, and then rebase your branch and force-push it to your GitHub fork, which will then update this pull request.
@christi3k is working on some git workflow help documentation that may also help you, and feel free to ask for development help at https://zulip.tabbott.net/#narrow/stream/development.20help !
Thanks again.

Automated message from Dropbox CLA bot
@devstorm21, thanks for signing the CLA!

Updated in #1954.


get full template's role when getting or finding

Installation information

jumpscale version: master
operating system: Ubuntu 16.04

Detailed description
I have two templates storagecluster.block and storagecluster.object while listing services under storagecluster it gets me both of the services under the same role also the same behaviour for getService which what I want. But when I inspected service.role all I can find is storagecluster, i even checked service.path and it was services/storagecluster/{servicename}. I want a way to get the child/specific role type from the service I got.

Filter on role and actorname when list services.
show actorname in the service view

@rkhamis I found out that serviceFind filter on actor name if I passed role = role.actorname so I think that part is already done

@khaledkbadr serviceFind is internal to the ays lib. API-wise you wouldn't be able to access it.


Move basic pkcs11 tests from src to tests

[ ] Move botanmain.
[ ] Move pkcs11_test

Done in the SystemTest branch

You can use this https://github.com/blog/1506-closing-issues-via-pull-requests to close the issue when you send the pull request.

#53 is still under review, this isn't fixed.


Example for GPT-3 Slack chatbot

Hi! I was looking for a way to build a GPT-3 Slack chatbot. I'd like the bot to work like this: in a Slack channel, respond to @mention. I heard good things about Modal and would like to try. Would you consider adding an example?
Related example: https://autocode.com/openai/threads/build-your-own-chatgpt-discord-bot-using-autocode-and-openai-84403434/

I think so, I'll try to adapt it, thanks!


Crash when looking for the usage of the item

http://pastebin.com/raw/RzLtKJXz
This is a crash report.
The game crashes when looking for the usage of uranium ingots from Adv Solar Panels.

Hi,
I am also having some issues with the usage button, however I don't think NEI is the culprit. I am getting a freeze for a full minute or two whenever I use this feature, and when I attempted  this on a local server my client actually exited to windows with no crash report.
Is there a better way to determine which mod is causing this issue other than removing them one by one? I can't seem to pinpoint which mod is causing problems here and its been around for a couple months now, any suggestions would be greatly appreciated! =)


Adding team tenets to the repo

Draft tenets document
Description
[Describe what this change achieves]

Category: Documentation
Creating new documentation around the security teams tenets

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks for the work on these, @shanilpa! Agree with @peternied 's comment, I think CONTRIBUTING.md looks like a better place for these as this developer guide is more about the mechanics of how to "do the coding thingy". I'm ok approving this once it moves there, and we can continue iterating on these tenets as we go along.

@davidlago and @peternied moved the content over to the CONTRIBUTING.md file.

Codecov Report

Merging #2593 (f9b3e3d) into main (30c967e) will increase coverage by 0.14%.
The diff coverage is n/a.

:exclamation: Current head f9b3e3d differs from pull request most recent head 6ef48bd. Consider uploading reports for the commit 6ef48bd to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #2593      +/-   ##
============================================
+ Coverage     61.25%   61.39%   +0.14%
- Complexity     3330     3382      +52
============================================
  Files           260      269       +9
  Lines         18509    18668     +159
  Branches       3269     3279      +10
============================================
+ Hits          11337    11461     +124
- Misses         5576     5609      +33
- Partials       1596     1598       +2

see 26 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thanks @shanilpa. Should you switch from draft to full PR? Changest LGTM as a first iteration of these in our repo.

Hi @shanilpa, to move this forward here are the steps you can follow:

Go to your repositories -> open your fork of the security repo -> find this branch "patch-1" -> click the "sync fork button" -> click "update" or whatever it prompts you to do
Go to your local IDE where you made these edits -> use git fetch -> git pull (this grabs the new updates) -> go to the CONTRIBUTING.md file and add an empty line to the bottom of it
Push your changes & after everything updates the lint error should be fixed and we can get this merged

Thank you for all your work and apologies if you already knew this. I know GitHub can be tricky with all the runners we have so I wanted to provide as much detail as I could.

Closing in favor of #2762


added mania hit number/text/dot

new element has been added to my list.
my delta4 skin has no mania hit (number/dot) and i think i should added.
some element has been test before end this
screenshot (running with macOS)
argon

delta 4

done
i added 2 type for mania-hit
Text and number


Fibonacci Search.c

Time Complexity analysis:
The worst case will occur when we have our target in the larger (2/3) fraction of the array, as we proceed to find it. In other words, we are eliminating the smaller (1/3) fraction of the array every time. We call once for n, then for(2/3) n, then for (4/9) n and henceforth.

Directory C is not for CPP programs. Please have a look. Also, add this comment in the file as a comment.

probelm solved plz accept


Fixing #1143

Resubmit

Build is currently failing because dev-next is jacked: https://github.com/driftyco/ng-cordova/pull/1188
Personally I'd just accept this -- a quick glance at the commit shows it's a trivial bugfix.

@Meekohi  Trivial bug fixes are probably what broke the build. If you'd like it PR'd faster; resolve the build issue? otherwise it'll wait until I've had a chance to check out the build and figure out why it's broken.

We had to upgrade our Jasmine instance. Thanks to #1174; it was fixed. Try again? And please target dev-next.
Sorry about that.


mFailureImageClickListener will call the mOnClickListener

I use BigImageView.setOnClickListener(final OnClickListener listener) to finish the Activity when user just tap the BigImageView, everything works fine but it become interesting if the failure image show up, check the source code i found this:
if (mOnClickListener != null) {
    mOnClickListener.onClick(v);
}

inside the mFailureImageClickListener statement block
https://github.com/Piasy/BigImageViewer/blob/df7453dd2531758d51611dc3905d78c9a15c40a0/BigImageViewer/src/main/java/com/github/piasy/biv/view/BigImageView.java#L122
so, the question is, why must check and invoke the mOnClickListener here? Shouldn't we separate their logic?

That's definitely a flaw of design.
Now I'm thinking, if we enabled tap-to-retry, should we only fire click event when SSIV is clicked, what do you think?

Now I'm thinking, if we enabled tap-to-retry, should we only fire click event when SSIV is clicked, what do you think?

I don't think so, what about animated image like gif and webp? as we know they are not displayed by SSIV

You are right, actually the click listener is set to thumbnail and main image, do you think it's a good idea to just bypass the failure image?

yes, when tap-to-retry is enabled, use mFailureImageClickListener, mOnClickListener otherwise.

Okay, I'll cut a release for that.

v1.6.1 is released, check it out :)


Fails on shebang

Currently bublé will throw an error when it encounters a shebang. Adding the acorn option allowHashBang to the existing transform options fixes that.
https://github.com/Rich-Harris/buble/blob/77cf57d743077108587f091d5f24487359fd252f/src/index.js#L56

Yeah, I'd be willing to merge a PR implementing this change.


Image does not save in android mobile

When I run this on mobile it does not export the image. What should I update in the code to do that?

What errors are you getting please?


Issues with multi_temporal_crop_classification.py config

I created the dataset using the instructions in https://github.com/NASA-IMPACT/hls-foundation-os/issues/46
I have in [multi_temporal_crop_classification.py](https://github.com/NASA-IMPACT/hls-foundation-os/blob/main/configs/multi_temporal_crop_classification.py):
# TO BE DEFINED BY USER: data directory
data_root = "data/multi-temporal-crop-classification/"

splits = dict(
    train="data/multi-temporal-crop-classification/training_data.txt",
    val="data/multi-temporal-crop-classification/validation_data.txt",
    test="data/multi-temporal-crop-classification/validation_data.txt",
)

However on running training I get:
rasterio.errors.RasterioIOError: data/multi-temporal-crop-classification/training_chips/chip_059_117_merged.tif: No such file or directory

Indeed chip_059_117 is defined in training_data.txt but is NOT present in training_chips.
On further searching I find the splits are defined in this repo, in which case I try:
splits = dict(
    train="data_splits/multi_temporal_crop_classification/training_data.txt",
    val="data_splits/multi_temporal_crop_classification/validation_data.txt",
    test="data_splits/multi_temporal_crop_classification/validation_data.txt",
)

However this results in:
FileNotFoundError: [Errno 2] No such file or directory: 'data/multi-temporal-crop-classification/hls-foundation-os/data_splits/multi_temporal_crop_classification/training_data.txt'

So I gather the text files must be copied to the location of the images. I do this and now have:
# TO BE DEFINED BY USER: data directory
data_root = "/teamspace/studios/this_studio/data/multi-temporal-crop-classification/"

splits = dict(
    train="training_data.txt",
    val="validation_data.txt",
    test="validation_data.txt",
)

Hi @robmarkcole . Thanks for taking the time to document this.
Indeed, there seems to be some discrepancy in the split files. I will look into this.
In the meantime, since we already explicitly split the data into a train and a val directory, we shouldn't actually require these splits. I will push this change.

Removed splits from the crop config file


Timestamp

Is there any way to get timestamp,not only the start time of word,but also duration

For CTC, yes, it is possible to get the start time and end time of a token (You have to infer the start and end time for a word from the corresponding tokens).
For transducer-based models, we can only get the start time, not the end time (so it is not possible to get the duration).

thank you,I will try in CTC

We have already some recipes that supports getting timestamps (only the start time of word currently). E.g.,

For conformer-ctc model, please refer to https://github.com/k2-fsa/icefall/blob/master/egs/librispeech/ASR/conformer_ctc3/decode.py.
For conformer-transducer model, please refer to https://github.com/k2-fsa/icefall/blob/master/egs/librispeech/ASR/pruned_transducer_stateless4/decode.py


Add TCK smoke test for coexistence of portable extensions and build c…

…ompatible extensions.
Fixes #332

On the meeting we agreed that we'll merge this so that we're able to stage another CR with this change included.
CC @starksm64


Sub dependency installation in docker fails

Be sure to check the existing issues (both open and closed!), and make sure you are running the latest version of Pipenv.

$ pipenv --version
pipenv, version 2021.5.29

Check the diagnose documentation for common issues before posting! We may close your issue if it is very similar to one of them. Please be considerate, or be on your way.

It might be related to "mixing Pipenv with system packages", but I don't know of a good way to avoid this in my case. I install it via pip install pipenv in a docker container and use it to generate a virtual environment with the required dependencies and then remove it.
Issue description
I try to install a sub dependency from a Pipfile.lock but it is not installed.
Pipfile.lock:
  "typing-extensions": {
              "hashes": [
                  "sha256:0ac0f89795dd19de6b97debb0c6af1c70987fd80a2d62d1958f7e56fcc31b497",
                  "sha256:50b6f157849174217d0656f99dc82fe932884fb250826c18350e159ec6cdf342",
                  "sha256:779383f6086d90c99ae41cf0ff39aac8a7937a9283ce0a414e5dd782f4c94a84"
              ],
              "markers": "python_version < '3.10'",
              "version": "==<IP_ADDRESS>"
          },

After the installation I see it in pipenv graph, but the module is not installed (importing it fails).
  $ pipenv graph|grep typing-extensions
        - typing-extensions [required: >=3.6.4, installed: ?]
        - typing-extensions [required: >=3.6.4, installed: ?]
      - typing-extensions [required: >=3.6.4, installed: ?]
        - typing-extensions [required: >=3.6.4, installed: ?]
        - typing-extensions [required: >=3.6.4, installed: ?]

  $ python
  Python 3.7.2 (default, Mar 27 2019, 08:44:46)
  [GCC 6.3.0 20170516] on linux
  Type "help", "copyright", "credits" or "license" for more information.
   >>> import typing_extension
  Traceback (most recent call last):
    File "<stdin>", line 1, in <module>
  ModuleNotFoundError: No module named 'typing_extension'

It works fine if I add the module as a primary dependency.
Expected result
The module is installed and can be imported.
Actual result
The module is not installed.
Steps to replicate
Work in progress. I am trying to come up with a smallest reproducible example.

@joe42 There is not enough information here to reproduce the issue, no Pipfile and the lock file is just a fragment of one.  At this point the pipenv version is no longer latest, so I am going to close this issue for now and suggest if you can reproduce a problem with the latest version of pipenv to supply a new bug report with steps to reproduce and output from running pipenv --support


Decrease Nav width

logo in nav dropped to 60px max width
Layout width for nav dropped to 250px
Check responsive size changes

Fixed with this commit


Analyse how a Graph collection could be modeled

In mathematics there are different types of graphs:

Undirected graph
Directed graph (& oriented graph)
Multigraph
Weighted graph
...

I actually though about this for some time. The problem with graph modelling is that they inherently contain back references (e.g. cycles), something that complicates persistent data structures.
I think a sparse Vector or primitive-keyed map would do most of the trick, but we should investigate how others have done it.

However, the first questions should probably be:

Is there a need for this?
Do graphs even need to be persistent?
Should we reinvent everything or wrap existing ones.
 But than again, maybe we should be the ones doing that. TBD.
Do we care about scalability, i.e. should it connect to http://titan.thinkaurelius.com (Titan is a scalable graph database optimized for storing and querying graphs containing hundreds of billions of vertices and edges distributed across a multi-machine cluster.), like Clojure's http://titanium.clojurewerkz.org/articles/getting_started.html does?

In Coursera's Algorithms II Sedgewick does a good job of introducing different graph concepts - though obviously in non-functional way.
Skienna has lots of graph theory as well (though I don't like his style very much).

Clojure and Scala don't have build-in graph support, but are supplemented by lots of libraries - though no mature library seems purely functional (citation needed):

Clojure: https://github.com/Engelberg/ubergraph (Ubergraph goes beyond Loom's protocols, allowing a mixture of directed and undirected edges within a single graph, multiple "parallel" edges between a given pair of nodes, multiple weights per edge, and changeable weights.)
Scala: https://github.com/scala-graph/scala-graph (doesn't seem mature enough)

Haskell has some purely functional graphs:

http://hackage.haskell.org/package/fgl

See also: Fully Persistent Graphs – Which One To Choose? - 2006 - https://pdfs.semanticscholar.org/522a/7d1cb5654bb474e5de3741eef36bc0a86f49.pdf

I think to start a Strawmans implementation myself to get a feeling for the topic. This first prototype should not be integrated into the Traversable hierarchy in a first step.
Currently I see no problem with back-references resp. cycles. Let's think about it mathematically:
A graph G := (V, E) is a tuple of a set V of vertices/nodes and a set of edges E ⊆ V x V.

For an undirected graph the following holds: ∀ x, y ∈ V : (x, y) ∈ E ⇔ (y, x) ∈ E. (Idea: I would add an isDirected flag to the constructor. If isDirected both edges are added to the set when one of them is added etc.)
We may create a weighted graph G := (V, E), with weights W and a total ordering ≤ ⊆ W x W, such that E ⊆ V x V x W.

Keeping this abstraction will make it easier to implement existing algorithm, e.g. calculating the minimal spanning tree (MST), find the shortest path (Knapsack problem) etc.
Using Vector under the hood is an implementation detail. We should not do that at this design stage. (When all that we have is a hammer everything starts to look like a nail.)
The strawmans Graph looks like this - we just think in interfaces for now:
interface Graph<T> {

    // vertices = HashSet.sequence(edges).apply((v1, v2) -> v1.union(v2))
    static Graph<T> of(Iterable<? extends Tuple2<? extends T, ? extends T>> edges) {
        ...
    }

    // need to think about behavior when there are edges using vertices not present in the set...
    static Graph<T> of(Set<? extends T> vertices, Set<? extends Tuple2<? extends T,  extends T>> edges) {
        ...
    }

    boolean isDirected();

    boolean isWeighted();

    Set<T> getVertices();

    Set<Tuple2<T, T>> getEdges();

}

Or should an undirected graph have a method Set<Set<T>> getEdges()... ?
interface WeightedGraph<T, W> extends Graph<T> {

    ...

}

Before looking into other libraries (myself), I want to model it myself. Then I will take a look at other definitions and learn.

Cool :)
The backreferences might be a problem, because you cannot easily change a node, without changing the whole cycle it belongs to (e.g. imagine a simple Javaslang List, that wraps around (i.e. last points to first). In that case you cannot add a new element to it in a persistent way in constant time - only by ruining the original or copying everything).

boolean isDirected();
boolean isWeighted();

An unweighted graph is simply one where all the weights are equal.
An undirected graph is simply a symmetric directed one.
I know in general algorithms this is how they are stored, but can't we reuse abstractions somehow (e.g. in the same wasy as a Set is a Map, with a dummy `value)?
We could have a general graph that could store positive/negative weights (e.g. see Floyd-Warshall) that is directed and can contain parallel edges.
Special cases (e.g. undirected/unweighted nodes) could be optimized for every node/edge in particular, so that the user doesn't have to know (i.e. special interfaces could give access to a single subset of the method, providing a undirected/unweighted view, having the advantage of a single implementation).
I know you strive for simplicity and not for one-ring-to-rule-them-all solution, and I think my abstraction has potential in this area.

FYI, Guava have recently released a graph API of their own (see com.google.common.graph.* in their 20.0-rc1 javadocs).
It's neither a persistent graph API nor does it seem to borrow from functional principles, but it might prove to be a source of inspiration.

@jbduncan nice, we will take it into account! thx

Graphs are, like Trees, multi purpose. There exist many libraries for specific use cases. Therefore it isn't prio 1 for us to develop the n-th Graph type.

Everytime I have put some time into this effort I try to make it fluent and don't inherently have a given algorithm in mind. What's most import is that it's searchable and fluently create able. By making them interfaces, we can defer the search functionality to a Lucene implementation or, if someone wants to try their hands, they can try to parallelize/distribute this with a different implementation.
I always modeled it something like:
  interface Graph extends Traversable<Node> {

    Node addNode(String name);
    Node addNode(String name, Map<String,Object> properties);

    /**
     * Maybe represent a subgraph?
     * @param query The query to apply. This might be too highlevel
     * @return a Graph representing all nodes found.
     */
    Graph findNode(Predicate<Node> query);

    // because these searches are by nature stateful, you want a new instance every time
    Supplier<UnaryOperator<Node>> SHORTEST_PATH = n -> ...// implement dykstra
    Supplier<UnaryOperator<Node>> DFS = n -> ...
    Graph traverse(UnaryOperator<Node> traverseFunction);

  }

  interface Node {

    /**
     * add a property to 'this'
     * @param name obvious
     * @param value anything you want
     * @return this
     */
    Node withProperty(String name, Object value);

    /**
     * Add a relationship between 'this' and other
     * @param other the thing to relate to
     * @return the new relationship.
     */
    Relationship relatesTo(Node other);

    /**
     *
     * @return the graph it is represented in. Assumes a node instance may only exist in 1 graph
     * which gives pause to representing a graph as a collection of subgraphs
     */
    Graph andThen();

  }

  interface Relationship {

    /**
     * Add a property to the relationship with a weight
     * @param name obvious
     * @param value obvious
     * @return this
     */
    Relationship withProperty(String name, Object value);

  }

Totally incomplete, but the most crucial thing is it's searchable.

I have had a in-progress implementation of this but as I have been so busy I don't know when I will get around to it. Should I create a PR to begin a discussion on it? It's total WIP.

I can push what I have and you can see what I have done and commented. I already started.
As for directed and not, I actually eliminated that as a dedicated thing and just made it so when you added an edge between nodes with directed it created the relationship between nodes. There was no specific flag identifying the relationship as bi-di. The reason I did this is, if you wanted to remove one of those directions you would have to also have to update the property on the reltionship. A "bi-di" relationship is just 2 relationships and not some special one.

Because I'm heavily biased (:wink:), I strongly suggest having a look at Guava's Graph API if it hasn't been looked at by the Vavr team already. (https://javadoc.io/doc/com.google.guava/guava/23.1-jre)
IMO, it has some interesting design decisions which should at least be taken into consideration before this issue is closed.

I assigned myself, I will create a suggestion.