Add "type" support to IdsQuery

Filters documents that only have the provided ids. Note, this query uses the _uid field.
{
    "ids" : {
        "type" : "my_type",
        "values" : ["1", "4", "100"]
    }
}

The type is optional and can be omitted, and can also accept an array of values. If no type is specified, all types defined in the index mapping are tried.

@saimaz do you really think we should introduce a separate attribute for type? The query already uses ParametersTrait, type can be passed as a parameter

You are right @einorler. The library does not support any custom fields for any query. You can add whatever you want as a parameter.


KZL-665 -  Update CPP snippet for getMyRights

What does this PR do?
Update CPP snippet for auth:getMyRights.
Boyscout

Update description for getMyRights in every languages

 Snippets tests reports   SDK Version Tests overview      v6        v5                                   :heavy_minus_sign:                    v1        v1        v1


Revert changes on the TreeProcessor (#6006 and #5885)

The TreeProcessor refactor introduced a change on the behavior of beforeEach inside it. While this is an edge case, Facebook (and potentially anyone using inline requires) is relying in it.
Some fixes have been suggested, but for the moment we'll just revert the change. cc / @niieani

@mjesun this will break webpack. It had just merged the huge PR migrating its tests to jest.
I have identified the line which was causing the issue and fixed it here: https://github.com/facebook/jest/pull/6100. Was this fix not enough or is there a different reason for doing the revert?

+1 for #6100 instead. The order was undocumented and unintuitive before the changes. An option seems like an acceptable workaround to me, until FB is able to fix their tests

and potentially anyone using inline requires

How does inline requires impact this? Jest itself uses inline requires with no issues before or after this change

Our internal transformer uses inline requires over test code. Some tests are using a helper utility file, which contains:
beforeEach(() => {
  jest.resetModules();
});

The require for this file is at the top level, but because of inline-requiring, the first time the file is loaded it happens inside an it. Think about the following scenario:
const myHelper = require('./my-helper');
const foo = require('foo');

beforeEach(() => {
  jest.mock('foo', () => {});
});

it('does magic', () => {
  myHelper.createAnotherStuff();
  foo.bar();
});

This test case passes before the change to the tree processor, but breaks now. The reason is that before the change, the order of the beforeEachs was:

First the jest.resetModules() inside the beforeEach of the helper function.
Then the jest.mock inside the beforeEach of the test.

With the change to the tree processor, the new order is:

First the jest.mock inside the beforeEach of the test.
Then the jest.resetModules() inside the beforeEach of the helper function.

Which essentially means that the later resetModules kills all pre-initialization. This will affect anyone using inline requires with helper files.


Abandon CircleCI in favor of github actions

Motivation
All the falcosecurity org moved to github actions, retiring CircleCI.
Feature
Move the repo away from CircleCi.
Alternatives
None
Additional context
None

@cpanato you were willing to help if i remember correctly :)
/assign @cpanato @FedeDP

yes, working on that :)

Thank you! :)

I guess most of the work is done here. @cpanato we just need to update some docs, then I guess we can close this :partying_face:


Updated Badge design system file

I have  updated the badge design system file.

Add more details on what colors are used and what is the convention of that. Convention means like the color of bg is always used a shade of 100 and text when its dark is always used a shade 800 like that.
Add details about text that what text sizes are used like sm, md ... etc and where are they used.
And also what are the default paddings and margins.
Except that everything looks awesome. @itz-deepak

Tag me for review when you add all the changes. @itz-deepak

Please review I have made the changes

Looks perfect. Great work @itz-deepak
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow these steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/)
Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.
That’s it. Follow the leaderboard!
And don’t forget to share your achievement with your friends.


Swap Nodes in Pairs using Java

Write the code for the Swap Nodes in Pairs in Java.
Follow the steps:
go to src -> Algorithm -> create a folder named "Recursion" -> then create another folder for the problem (e.g Swap_Nodes_in_Pairs ) -> there add the java solution and create a README file there add the link of the question.
Make sure to make a folder for every question you solve and the folder should consist of the java file and the README file
Problem Link : Swap Nodes in Pairs

I would like to work on this issue! @aritra-tech


Replace new logo and add folder art

I have replace new logo, add banner to README and add folder art. I have changed it back in the art folder.

👍 awesome


update to stellarnet for new amount parsing consolidation

integrate https://github.com/keybase/stellarnet/pull/26, a new, more strict, more consolidated API for converting stellar amounts between strings and int64s and xdr.Int64s. And I'll revendor once that stellarnet PR is merged.
Also, it looks like stellarnet hasn't been revendored in a while, so there are some other changes in here too, but they look pretty additive / safe to me.

could you add a test for the exponential parsing?

@patrickxb done


#73 - update readme

Updated readme for #73.

This pull request #266 is assigned to @vatavuk/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @g4s8/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

@olegmoz thanks, updated, please, take a look

@Vatavuk thanks, corrected, take a look please

Codecov Report

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

@@            Coverage Diff            @@
##             master     #266   +/-   ##
=========================================
  Coverage     86.85%   86.85%
  Complexity      268      268
=========================================
  Files            40       40
  Lines          1255     1255
  Branches         54       54
=========================================
  Hits           1090     1090
  Misses          155      155
  Partials         10       10

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 f32256c...c2cea45. Read the comment docs.

Job was finished in 25 hours, bonus for fast delivery is possible (see §36)

There is an unrecoverable failure on my side. Please, submit it here:
PID: 4@54bbf007-af81-47f4-889b-85871a7a9b4d, thread: PQ-CT2E6TK9B
com.amazonaws.http.AmazonHttpClient$RequestExecutor[1359] com.amazonaws.ResetException: Failed to reset the request input stream;  If the request involves an input stream, the maximum stream buffer size can be configured via request.getRequestClientOptions().setReadLimit(int)
java.io.BufferedInputStream[170] java.io.IOException: Stream closed

0.53.15: CID: f1588d81-c929-4276-aedf-7e14d83b0ed9, Type: "Order was finished"

@sereshqua/z please review this job completed by @vatavuk/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat quality good


PNG and Null error

@51lv3str1
Great work!!
I'm wondering if you have an example where it gets exported to PNG rather than SVG?
Also on my beta.app.edublocks.org for some reason Blockly.Css.CONTENT is null :(

¡Hi Joshua!
Regarding to PNG export, I've just added that to this script. In the bottom of the script you can choose if to exportSVG() or exportPNG() (uncomment the line you want to use)
Regarding to Blockly.Css.CONTENT, I've just run this script (blockly-to-svg) in your page beta.app.edublocks.com (great page, by the way) and it exports the SVG fine. That variable is not null.
Can you tell us more about your problem?

Thanks so much for this. Amazing work.
I’ll try it out later today.
Yes, that bug is now fixed. There was a line I had to comment out on blockly. In newer versions the css contents in that function get wiped

Great work @asanzo I have tried it today but have a few questions.
It seems to limit the size here and cut things off, do you know why?
One last question is, how do I get the dropdown arrows to show on the final image?
Thanks once again!

Hi, again Joshua!!
In the script, you have to tweak canvas size:
Where it says:
var canvas = document.createElement('canvas');

you can add:
var canvas = document.createElement('canvas');
canvas.width = 800;
canvas.height = 600;

So you can choose canvas size. I have added that to the code.
Regarding to the dropdown arrows, sorry but I don't have a clue why in that page they don't show.
I've opened an issue to deal with it:
https://github.com/Program-AR/blockly-to-svg/issues/3
We don't know when we'll be able to correct it as we have lots of work :)
If you're ok with it, we can close this issue.


Security: address TLS/FS issues

This addresses the following issues:

Insecure TLS connections allowed:

removed the code and added a way to specify a certificate for self-signed / local setup
updated https-agent-proxy package to enable additional options support

Insecure fs module usage:

added code to check against root. As we rely on temporary files location for form submitted files and on user-specified certificates, the folder can be located anywhere. Using '' as the root allows to resolve both relative and absolute paths properly.

Note that the change restricts using non-trusted by NodeJS built-in certificates HTTPS connections, unless the certificate is provided either via NODE_EXTRA_CA_CERTS or HTTPS_PROXY_CA. Alternatively users can disable the check globally by specifying NODE_TLS_REJECT_UNAUTHORIZED=0 which is not recommended.
#minor

coverage: 67.523% (-0.05%) from 67.572%
when pulling 227e102b12df25a29420d120a2d7f294b7bf4438 on OEvgeny:chore/security-fixes
into 61414edfe0b88eb9bc40ce8c50b9f058ea2ef020 on microsoft:main.


TF golang client memory leak

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 1 and tf 2
Custom Code
Yes
OS Platform and Distribution
ubuntu20.04
Mobile device
No response
Python version
3.8
Bazel version
No response
GCC/Compiler version
9.3.0
CUDA/cuDNN version
11.4
GPU model and memory
No response
Current Behaviour?
Action1: new a tensor and use the only one tensor to do session run will not result in memory raise.
Action2: new a tensor every time before seesion run will lead to memory rasie.

Conclusion: NewTensor api lead to memory leak

Suggestion: Release tensor obvisly instead of just set finializer

Standalone code to reproduce the issue
package main

import (
	"flag"
	"fmt"
	"io/ioutil"
	"log"

	tf "github.com/tensorflow/tensorflow/tensorflow/go"
)

func main() {
	// An example for using the TensorFlow Go API for image recognition
	// using a pre-trained inception model (http://arxiv.org/abs/1512.00567).
	//
	// Sample usage: <program> -dir=/tmp/modeldir -image=/path/to/some/jpeg
	//
	// The pre-trained model takes input in the form of a 4-dimensional
	// tensor with shape [ BATCH_SIZE, IMAGE_HEIGHT, IMAGE_WIDTH, 3 ],
	// where:
	// - BATCH_SIZE allows for inference of multiple images in one pass through the graph
	// - IMAGE_HEIGHT is the height of the images on which the model was trained
	// - IMAGE_WIDTH is the width of the images on which the model was trained
	// - 3 is the (R, G, B) values of the pixel colors represented as a float.
	//
	// And produces as output a vector with shape [ NUM_LABELS ].
	// output[i] is the probability that the input image was recognized as
	// having the i-th label.
	//
	// A separate file contains a list of string labels corresponding to the
	// integer indices of the output.
	//
	// This example:
	// - Loads the serialized representation of the pre-trained model into a Graph
	// - Creates a Session to execute operations on the Graph
	// - Converts an image file to a Tensor to provide as input to a Session run
	// - Executes the Session and prints out the label with the highest probability
	//
	// To convert an image file to a Tensor suitable for input to the Inception model,
	// this example:
	// - Constructs another TensorFlow graph to normalize the image into a
	//   form suitable for the model (for example, resizing the image)
	// - Creates and executes a Session to obtain a Tensor in this normalized form.
	modeldir := flag.String("dir", "", "Directory containing the trained model files. The directory will be created and the model downloaded into it if necessary")
	//imagefile := flag.String("image", "", "Path of a JPEG-image to extract labels for")
	flag.Parse()
	if *modeldir == "" {
		flag.Usage()
		return
	}
	// Load the serialized GraphDef from a file.
	// modelfile, labelsfile, err := modelFiles(*modeldir)
	//if err != nil {
	//	log.Fatal(err)
	//}
	model, err := ioutil.ReadFile(*modeldir)
	if err != nil {
		log.Fatal(err)
	}

	// Construct an in-memory graph from the serialized form.
	graph := tf.NewGraph()
	if err := graph.Import(model, ""); err != nil {
		log.Fatal(err)
	}

	// Create a session for inference over graph.
	session, err := tf.NewSession(graph, nil)
	if err != nil {
		log.Fatal(err)
	}
	defer session.Close()

	// Run inference on *imageFile.
	// For multiple images, session.Run() can be called in a loop (and
	// concurrently). Alternatively, images can be batched since the model
	// accepts batches of image data as input.
	//tensor, err := makeTensorFromImage(*imagefile)
	//if err != nil {
	//	log.Fatal(err)
	//}

	operation := graph.Operation("init_all_tables")
	_, err = session.Run(
		map[tf.Output]*tf.Tensor{},
		[]tf.Output{},
		[]*tf.Operation{operation})
	if err != nil {
		fmt.Printf("init err %v", err)
	}

	inputs := make([]string, 0)
	for i := 0; i< 8;i++ {
		inputs = append(inputs, "测试文本输入")
	}

	tensor, err := tf.NewTensor(inputs)
	if err != nil {
		log.Fatal(err)
	}
	for i := 0; i < 1000000; i++ {
		output, err := session.Run(
			map[tf.Output]*tf.Tensor{
				graph.Operation("input_tensor_name").Output(0): tensor,
			},
			[]tf.Output{
				graph.Operation("output_tensor_name").Output(0),
			},
			nil)
		if err != nil {
			log.Fatal(err)
		}
		probabilities := output[0].Value().([][]float32)[0]
		if i % 1000 == 0 {
			fmt.Println("repeat ", probabilities)
		}
	}
   	// output[0].Value() is a vector containing probabilities of
	// labels for each image in the "batch". The batch size was 1.
	// Find the most probably label index.
	//printBestLabel(probabilities, labelsfile)
}

Relevant log output
No response

Action1：right red box

@iammeizu Could you please confirm if this issue is replicating in TF v2.9.0 ?
Thank you!

If TF < 2.5, there was indeed a memory leak associated with deallocs of string tensors in, fixed by #49925.


Improvment proposal: Add possibility to specify multiple DBC files

Could potentially be useful.
Rough possible approach:

Keep one argument, but allow it to contain multiple files, like dbcfile=file1.dbc,file2.dbc
If more than one, add more with https://cantools.readthedocs.io/en/latest/#cantools.database.can.Database.add_dbc_file
If we do not have a real test case, we could possibly refactor the existing DBC file into multiple ones

Will be fixed by #113


v0.6.1 severe malfunction.

We upgraded alertmanager today, and experienced severe problems in the GUI. For example, it seems the previous version had some javascripting cached so the rendering was way off. After testing in incognito, it was somewhat better, but we ran into some weird bugs. For example, when clicking "hide silences", the actual view updated and showed the silences. The exact opposite behavior that was expected. Also, the silence icon was gone so we had no idea of knowing what alarm was silenced or not. We eventually downgraded as we were having so many issues that we didn't dare running this release in production.
Has anyone else experienced the same issues?

ime="2017-05-04T13:35:37+02:00" level=warning msg="Could not get file: AssetInfo ui/lib/angular-sanitize.min.js.map not found" source="web.go:31"
time="2017-05-04T13:35:37+02:00" level=warning msg="Could not get file: AssetInfo ui/lib/jquery.min.map not found" source="web.go:31"
time="2017-05-04T13:35:37+02:00" level=warning msg="Could not get file: AssetInfo ui/lib/angular.min.js.map not found" source="web.go:31"
time="2017-05-04T13:35:37+02:00" level=warning msg="Could not get file: AssetInfo ui/lib/angular-route.min.js.map not found" source="web.go:31"
time="2017-05-04T13:35:37+02:00" level=warning msg="Could not get file: AssetInfo ui/lib/angular-resource.min.js.map not found" source="web.go:31"
time="2017-05-04T13:35:37+02:00" level=warning msg="Could not get file: AssetInfo ui/lib/angular-moment.min.js.map not found" source="web.go:31"
time="2017-05-04T13:35:40+02:00" level=warning msg="Could not get file: AssetInfo ui/lib/jquery.min.map not found" source="web.go:31"

The logging of .map files not being found is normal. That usually happens when you open the dev console in your browser, which than tries to get a .js.map file for every .js file that was loaded.
Otherwise, I cannot report similar behavior. I've been switching between different versions multiple times.
On the silencing issue, @stuartnelson3 @mxinden can you check again that https://github.com/prometheus/alertmanager/commit/6a909abf17d56a09ade3ad6753b5eb844fe5b244 this was all correct.
It honestly sounds more like Internet Explorer (assuming based on the screenshot) to behave very differently in terms of JS caching and even loading cached stuff in incognito mode maybe? Something we should address (and do in the new UI afaik), but wouldn't classify it as severe malfunction.

Testing was done on both chrome, firefox and IE. We tested with windows, linux, and mac. Gui was all over the place.

Mh, interesting. Thanks for the update. I tried multiple browsers and switching various AM versions now as well but couldn't reproduce yet.
What distribution of the version are you running? Docker, Tarballs, compiled yourself?

We're using the tar balls, and bundling them into rpms. From what you're saying, it leads me to think that something went wrong in our end. I'll do some more checking over the weekend.

Will try again with tarballs rather than compiling it myself. I don't think your RPM packaging could be a cause. The javascripts and HTML files are compiled directly into the binary.

@fabxc @raypettersen I am not able to reproduce the issues either with a self compiled version of 6a909abf17d56a09ade3ad6753b5eb844fe5b244.

Thanks for the input. We're running behind nat and using external path option. Could that be the issue?

There are many ways how this could cause problems. However, returning to the previous version resolves the problem, right?

Reverting to 0.4.2 resolves everything.

Can't reproduce the issue of JS not loading properly at all but some of the silence-related ones. Trying to track it down.

Seems like this issue is under control. Will we see another release soon? We're holding off upgrading at the time being.

@raypettersen You can find the new release here. Thanks for the help!

Superb. I'll test the release tomorrow. Thanks!

Closing here. Please feel free to reach out to us again.


Collection Res Type -- Phase 2

In phase 2 the following features will be added to collection res type:

versioning (re: #1102)
publishing
collection holds other collections

tested on www


Remove 'delete-frame-functions hook after invocation to avoid unexpected behavior on kill-emacs

Done


ci(gcb): add cmake-super build on ubuntu-bionic

The build yaml files are included in this PR, but they won't be enabled until after this PR is merged.

This change is

Fixed the copyright thing.

FYI: I do not think the super build runs any integration tests, you could skip a few steps here.

Correct, but we (will) use this image for other builds too. For example, from this matrix of our current builds, I should change our current integration build to use this docker image, which I plan to do in a follow up PR.


Fix early termination in repair rule.

Created by: nc6
We should only terminate after looping over, not within the loop.

Closing as an obsolete


Error refactor

Addresses #153 config issues and #154, #158 error displaying

@Brechtpd @smtmfft I believe this is ready for merging, the changes in this PR don't change any code flow, but do throw errors that are handled by the API instead of panicking in the host. The CI changes are only impacting the maintainability of the CI code itself


Improve regression detection with continous testing

It seems the youtube-dl project has abandoned the idea of using tests with travis-ci to keep all IEs always working (last passed build was 2 years ago, #2571). Not that it's a bad idea, since it's hard to keep up with all those sites. But a side effect of that is that it's harder to detect test fails that are due to a new commit, and not a website change.
What's I'd like to propose, is to keep the current testing infrastructure as a dashboard for working/non-working IEs, and add another "testsuite" that would test each commit for regressions: if a test is failing, the previous commit would be tested as well, and if it fails, it's not considered a regression.
I've made a simple proof of concept https://gist.github.com/anisse/6093f8b5814ab3ce7140 in bash. This could be improved/rewritten, in order to be integrated with travis-ci.
What do you guys think ?

I've been working on a python version integrated with travis here:
https://github.com/anisse/youtube-dl
It is not ready for a pull request yet, since this is a bit more complex than i'd have wished, but already does the job nicely:

it does the original test suite, but marks it as allowed to fail in travis
it runs all tests on the pull-request/pushed commit and if they fail, then runs them on the original commit to see if they are a regression. It tries them a few times to be sure this is not a temporary error.

Warnings are treated as errors for now, since it could mess up with regression detections, but I'm not sure what to do about this yet. Any idea ?
Last but not least, this splits the test suite in multiple part, because we might otherwise run it to the travis time limit. It contains a generic solution to do that with nosetests. I originally wanted to do this work separately, but the travis 2 hours time limit forced my hand.

For parallel tests, see #7267. It uses nose's built-in parallel feature.

Looks nice. I had trouble using it nose processes (did not see all tests, wasn't deterministic), I should test your branch.
Note that both solutions can be used in parallel (pun unintended), since my solution splits the work at the travis level, allowing you to have different jobs (running on different VMs), while the nose-based solution splits the work inside a job/VM.
The nose solution could be made generic though if travis integrated automatic nose like it does with RSpec, Cucumber and Minitest.

This is copy of the message in #9235 :
The problem is that some tests are so unreliable, that it's very difficult to run them twice and get the same result.
I understand that due to the nature of the project, tests depend on a third party (the website), and that website may well be quite flaky. But I want to at least eliminate the most unreliable tests.
For example, I ran the full test suite (with regression detection) 75 times. Out of those, a few tests have more than 2 detected regressions (false positives):
      2 test.test_download:TestDownload.test_WeiqiTV ERROR
      2 test.test_download:TestDownload.test_YesJapan ERROR
      2 test.test_subtitles:TestYoutubeSubtitles.test_youtube_translated_subtitles ERROR
      3 test.test_download:TestDownload.test_DouyuTV_2 ERROR
      3 test.test_download:TestDownload.test_KuwoMv FAIL
      3 test.test_download:TestDownload.test_NRKTV ERROR
      3 test.test_download:TestDownload.test_Sohu_3 ERROR
      3 test.test_download:TestDownload.test_Vodlocker ERROR
      3 test.test_subtitles:TestNPOSubtitles.test_allsubtitles ERROR
      4 test.test_download:TestDownload.test_Chaturbate ERROR
      4 test.test_download:TestDownload.test_ElPais ERROR
      4 test.test_download:TestDownload.test_Jpopsuki ERROR
      4 test.test_download:TestDownload.test_Telegraaf ERROR
      4 test.test_download:TestDownload.test_TudouPlaylist FAIL
      4 test.test_download:TestDownload.test_Viddler FAIL
      5 test.test_download:TestDownload.test_NPO_4 ERROR
      5 test.test_download:TestDownload.test_NRKTV_1 ERROR
      6 test.test_download:TestDownload.test_C56_1 ERROR
      6 test.test_download:TestDownload.test_Smotri_5 ERROR
      7 test.test_download:TestDownload.test_AdobeTVVideo ERROR
      7 test.test_download:TestDownload.test_LePlaylist FAIL
      7 test.test_download:TestDownload.test_MiTele FAIL
      7 test.test_download:TestDownload.test_NPO_5 ERROR
      9 test.test_download:TestDownload.test_Yahoo_7 FAIL
     14 test.test_download:TestDownload.test_GodTube FAIL
     15 test.test_download:TestDownload.test_Mwave FAIL
     16 test.test_download:TestDownload.test_LivestreamOriginal_1 ERROR
     16 test.test_download:TestDownload.test_Sexu ERROR
     19 test.test_download:TestDownload.test_ACast ERROR
     19 test.test_download:TestDownload.test_StreetVoice ERROR

Some have as many as 19(!).
ERROR means that there was an error running the test. Maybe we should ignore those. FAIL, means that a test passed, then failed on the same code revision. You can see how this translates in regression detection here:
StreetVoice: https://travis-ci.org/anisse/youtube-dl/jobs/123862153 https://travis-ci.org/anisse/youtube-dl/jobs/123289890
MWave: https://travis-ci.org/anisse/youtube-dl/jobs/123289900
GodTube: https://travis-ci.org/anisse/youtube-dl/jobs/123289914
It even generates user issues, for example: Streetvoice #9219 .

Well I run test_ACast for 75 times:
for i in $(seq 1 75) ; do
    python3.6 -Werror test/test_download.py TestDownload.test_ACast || break
done

And there are no errors. Could you give an example of error messages?

Oh I found one: https://travis-ci.org/anisse/youtube-dl/jobs/123862117#L388
ERROR: test_ACast (test.test_download.TestDownload)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/anisse/youtube-dl/test/test_download.py", line 141, in test_template
    force_generic_extractor=params.get('force_generic_extractor', False))
  File "/home/travis/build/anisse/youtube-dl/youtube_dl/YoutubeDL.py", line 683, in extract_info
    self.report_error(compat_str(e), e.format_traceback())
  File "/home/travis/build/anisse/youtube-dl/youtube_dl/YoutubeDL.py", line 543, in report_error
    self.trouble(error_message, tb)
  File "/home/travis/build/anisse/youtube-dl/youtube_dl/YoutubeDL.py", line 513, in trouble
    raise DownloadError(message, exc_info)
DownloadError: ERROR: Unable to download webpage: HTTP Error 503: Success (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
-------------------- >> begin captured stdout << ---------------------
[acast] -where-are-you-taipei-101-taiwan: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 503: Success (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/home/travis/build/anisse/youtube-dl/youtube_dl/extractor/common.py", line 365, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/home/travis/build/anisse/youtube-dl/youtube_dl/YoutubeDL.py", line 1929, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/opt/python/2.6.9/lib/python2.6/urllib2.py", line 397, in open
    response = meth(req, response)
  File "/opt/python/2.6.9/lib/python2.6/urllib2.py", line 510, in http_response
    'http', request, response, code, msg, hdrs)
  File "/opt/python/2.6.9/lib/python2.6/urllib2.py", line 435, in error
    return self._call_chain(*args)
  File "/opt/python/2.6.9/lib/python2.6/urllib2.py", line 369, in _call_chain
    result = func(*args)
  File "/opt/python/2.6.9/lib/python2.6/urllib2.py", line 518, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

503 is indeed quite strange.

streetvoice.py updated with the new API in 4dccea8. Hopefully fixes 403 errors. Please leave comments if the problem is still.

I've improved the script in the regdetect branch:
https://github.com/anisse/youtube-dl/tree/regdetect
I've collapsed the commits and it's getting closer to ready for a merge.
It now tests for reliability by going back and forth to before/after push, to hopefully reduce the number of false positives. It also does automatic regression bisecting, so we don't need to look at the various commits in push to guess which one introduced a regression.
As you have seen, I have put in place automatic merging in my tree, and it already found a few regressions (#9991, #10018, #10030, #10048, #10064, #10096 for example).
I'm hoping the last changes will make it even easier to use since it will automatically pinpoint the bad commits, have less false positives, and tell us which tests/websites are not reliable.


"&quot" are included in the developer portal

Steps :

Log in to the developer -portal
Access Applications > New Application
Select SAML web application, & create SAML web application with required fields.
Edit SAML web application,
Navigate to Access Tab Expand SAML protocol,

This issue is not reproduced in the 5.11.0-beta4 pack. Therefore closing this issue. @NilukaSripalim Please reopen this if you are able to reproduce this in the latest pack.


Create learn

tKS ^^

[ y] I have read the contributing guidelines and I agree with the Code of Conduct
[y] Have you checked that there aren't other open pull requests for the same changes?
[y ] Have you tested that the committed code can be executed without errors?

DO NOT publish garbage PRs to farm Crypto AirDrops. Any user suspected of this action will get banned. Submitting broken code wastes the time of the contributors, who have to spend their free time reviewing, fixing and testing code that does not even compile, breaks other features or does not introduce any changes.

Closes #XXXX

Relates to #XXXX

What does this do?


Deserialize JSON using KotlinX Serialization instead of Jackson

This PR should in the end close #224.
Description
In this PR, I replace GSON as the JSON (de-)serialiser of choice with KotlinX’ Serialization library.
I know that #224 mentioned Jackson as one of the options but KotlinX Serialization seemed like a better fit to me with it being a KotlinX library and thus hopefully even better support and more “kotlin-y” APIs.
So far I could not see any reason not to use it for this project.
State of this PR
This PR is not yet entirely ready for review. It’s just a first work in progress that I’d like your feedback on. @andregasser

Is this a good way forward?
Is my change in the test JSON data from Int to String for id-related properties okay? (see e.g. Status API doc which mentions “Type: String (cast from an integer but not guaranteed to be a number)”
Are the code-style related changes okay?

Code style
When I applied code style settings from this project in IntelliJ IDEA, auto-formatting changed loads of lines so I assume there’s no style added to the project? I went for the Kotlin defaults then which seemed to mostly not change too much. I did not merge auto-formatting changes unrelated to this PR’s main changes though.
Usually, I use argument names heavily but this project doesn’t seem to do that. Was that a conscious choice? If so, I’m glad to remove the few additions again. If not, I’ll keep them. 😊
Same for var/val types and fun return types. I usually explicitly add them as well but also couldn’t see that everywhere here.
Still open
I needed to litter the JSON config in a few different files and for finishing up I’d like to find a better approach here that would still ensure that we’re using the same config everywhere without adding too much coupling. I already don’t like the JSON serializer being part of MastododonClient but I wanted to keep this PR as close to the original GSON usage as possible for now.

Hello @PattaFeuFeu
First, thanks for your work so far. Of course, your approach regarding KotlinX is totally fine. Honestly, I am not a Kotlin expert and thus was not aware of the existence of this functionality. But after checking out the docs a bit. I also think this is the right way to go. As you say, it seems better supported / integrated and also more lightweight I guess.  :-)
Regarding code style: Detekt for Kotlin code and PMD plus checkstyle for Java code should run once you run gradle check.
Regarding your other questions, I will need to have a look into the code a bit more. Hopefully I can do this tomorrow.

I am in complete agreement with @PattaFeuFeu. Kotlinx is much better and fits very good for the purposes of this library. By the way, terrific PR

Is my change in the test JSON data from Int to String for id-related properties okay? (see e.g. Status API doc which mentions “Type: String (cast from an integer but not guaranteed to be a number)”
This is totally fine, even better! Quickly checked Mastodon API docs and that's actually how the data is returned from the server. So all good!! :-)

Is this a good way forward?

Definitively. Go ahead! Looking forward to your changes!

I have gone through your code changes and must say, that this is looking very good so far. Looking forward to some test runs, once it is ready 🙂

@andregasser I think I’m mostly done. I will update the description soon and will add the pull request template to the bottom of the PR description with additional info.
How do we want to handle the decrease in code coverage? As all tests are still passing and I didn’t really add new functionality and rather replaced a dependency, I would personally ignore it (which also is the reason why I’m a bit sceptical of code coverage tools as “false positives” like this can happen easily).
If you’d like, I could potentially add some additional tests for edge cases. I introduced nullability in a few places where previously potential nullability was just ignored or covered up in a generic exception.

How do we want to handle the decrease in code coverage? As all tests are still passing and I didn’t really add new functionality and rather replaced a dependency, I would personally ignore it (which also is the reason why I’m a bit sceptical of code coverage tools as “false positives” like this can happen easily).

If you’d like, I could potentially add some additional tests for edge cases. I introduced nullability in a few places where previously potential nullability was just ignored or covered up in a generic exception.

Do not worry about the test coverage tool output too much, as I am also not 100% happy with it yet. Because of this, I also did not set codecov as a mandatory check to be passed before we can merge changes. It is more of informational nature for the moment. Once I find more time, I'll have a look at it and try to improve / replace it.
I will do some test runs based on the changes in this PR very soon.
Thanks for the update 🙂

This looks good to me! Regarding some of the mostly code style questions brought up earlier:

changing id-related properties in test data to string: this is correct. We've had to change all id types to strings earlier this year, the test data is probably mostly left over from before that change.
using argument names: I don't think it has ever been a conscious choice not to use them. I prefer them for clarity as well, so as far as I am concerned, we should keep it this way and use argument names throughout.

Fine for me as well. Let's use argument names in a consistent way.


[UI] Improve frontend plotting

Related to https://github.com/InstituteforDiseaseModeling/covasim/issues/51, we need to think about how to fix plotting. Here's how plotting looks on the BE:

At minimum we need to plot data, but other changes would be good too. Options:

Keep Plotly and expand it
Move to Altair
Switch back to mpld3, so we can exactly reproduce backend figures

@RomeshA @deviozc still can't assign you guys for some reason argh

I believe this issue is a duplicate of https://github.com/InstituteforDiseaseModeling/covasim/issues/51
Perhaps close #51, @inc0 ?

@hamelsmu related but not a duplicate, this is closed by #85 but altair would still be cool longer term (not MVP)

Closed by #85


Adding e2e test.

fixes #5.
With this CL, we

uploads a newly built image to gcr.io/php-mvm-a/php-nginx:testing if the functional tests passed
run end to end tests against an app deployed with gcloud, delete that version
if all the functional tests and end to end tests passed, and if it's a push on the master branch, it will uploads the newly built image to gcr.io/php-mvm-a/php-nginx:latest

If all works well, we can deploy the new image automatically :)

Alright, the e2e test started working :)
@sjlangley Can you taka a look?

@sjlangley Thanks! I believe cloud sdk installation is now rock solid. PTAL

I forgot to parameterize  the project id in the end to end test and just fixed it. @sjlangley PTAL

lgtm++

Thanks! Let's see if it can upload the image automatically :)

FYI, the image is successfully and automatically uploaded to gcr.io/php-mvm-a/php-nginx:latest which is great. Only thing is that the layers are entirely replaced because travis build environments are ephemeral. Ideally we can cache the layers as much as possible, so eventually I want to skip e2e tests in travis, and run e2e tests on somewhere else, where we can have persistent workspace.


Fuzzy search

I was trying to find the heavy checkmark emoji by searching "heacheck".
Unfortunately fuzzy search doesn't seem to be a thing here, so I was wondering if I could get some directions maybe implementing it?
More than glad to help to ship this feature. 👍

No?
'kay 😅


ZH: Update links to finished Android components

We have some links missing in Spark zeroheight to Android documentation for finished components:
Badge github and dokka
checkboxes, radios and switches github and dokka
Chip github and dokka
Icon dokka and github
Input/Textfield github and dokka
Modal is experimental dokka and github
Spinner github and dokka
Tabs github and dokka
Tag github and dokka link is missing an image

Done


Container never boots, iframe blocked by content policy.

Describe the bug
Just spins forever. The container never boots. I tried deleting all Application storage and uninstalling the service worker and starting from scratch. No change. The terminal never starts. The web container never boots.
The Network tab shows that an iframe and two other resources were blocked by the content policy (see screenshots)
Link to the blitz that caused the error
https://stackblitz.com/edit/vitejs-vite-r9gysk?file=src%2Fmain.ts&terminal=dev
Steps to reproduce
Just open the project.
Expected behavior
It should boot and execute the JavaScript.
Parity with Local

[ ] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 98469004,
  "usedJSHeapSize": 95587328,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 34c588ed

Additional context
No response

FWIW, I don't have any issues in Microsoft Edge on the same machine.

Hey @benlesh !
I think there might be something else going wrong here. That screenshot shows that an iframe for googletagmanager gets blocked and I get the same here.
WebContainer booting or not does not depend on this content loading or not.
Could you share what you see in Sources in the dev tools? This is what you should expect to see:

On the network tab, if you filter with staticblitz.com, you should at least see: blitz.**.js, builtins.*.js, full_bin_index.** and some wasm resources.

@Nemikolh your image isn't displaying for some reason.
When I filter by stackblitz in the network tab, I see a lot of New Relic URLS. I suspect that it's just my company's filtering and security blocking everything for unknown reasons. It works on Edge on the same machine.

Well, okay, it seems like it's just something related to how security folks configured browser monitoring for my specific instance of Chrome. It's working on Edge, and I'm unblocked.
I'll close this. Hopefully it's not a common issue for anyone else, but at least you know this can be a problem. Apologies for the noise.

No worries! All good! 🙌 And thank you for the report 😃
We should definitely handle this situation better as this is not the first occurrence of having WebContainer broken because of one of our domain being blocked or timing out.


"Open the root of a solution?"

The readme says to "Open the root of your solution within Atom"
What, exactly, does that mean?
How do I "open the root" of a solution?
What If I don't have a solution but just some random "sample.cs" file that I want to open and edit?

Hi @indyHarcourt
It means use open folder to open the root folder of your application, typically the folder that has your sln file or project.json.
You can open a sample.cs file but you cannot start the omnisharp-server on it (using ctrl+alt+o).


Update aws-core, http-client-spi, ... to 2.17.133

Updates

software.amazon.awssdk:aws-core
software.amazon.awssdk:http-client-spi
software.amazon.awssdk:netty-nio-client

from 2.17.130 to 2.17.133.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #720.


Importing stdlib functions

Is there a reason why only some of the files from https://github.com/python/cpython/tree/master/Lib have been copied to the thirdparty/stdlib folder? I was looking at the feature map from https://github.com/google/grumpy/wiki/Missing-features, which makes it seem like a lot of the features would be covered in duplicating most of the CPython Lib files.

There are a couple reasons. One is that not all language features and builtins are implemented so some files don't work. The other is that many libraries depend on modules that are written as C extensions in CPython. Grumpy does not support C extensions, so those modules need to be rewritten from scratch or brought in from PyPy. That work is still ongoing.
That said, a lot of progress has been made over the last couple months, so there's good reason to believe many libraries can be integrated cleanly now.
I'm going to close this issue. Please open issues for specific features or libraries that need to be implemented.


[Docs] Add diagrams for filter to DG

Updated use cases as well

Codecov Report
Base: 70.18% // Head: 70.18% // No change to project coverage :thumbsup:

Coverage data is based on head (b1947fe) compared to base (660b1c7).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #91   +/-   ##
=========================================
  Coverage     70.18%   70.18%
  Complexity      703      703
=========================================
  Files           107      107
  Lines          2163     2163
  Branches        249      249
=========================================
  Hits           1518     1518
  Misses          570      570
  Partials         75       75

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

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


Update private-cloud-cluster.md

Added the content to Add cluster manager and show activity logs  on cluster and namespace levels

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@Nidhi251289
There is a new button name needed in this image for Add Details - please update the file:

You added the following image links to the text but did not add the image files to this PR - please add these image files to your branch; otherwise, these will break the build and I cannot publish this PR:

@MarkvanMents please review upon return

@MarkvanMents passing this back to you, they will let you know when to publish. Screenshots need editing/cropping/borders upon merge.


[BSI][20210308] SafetyNet Attestation on Rooted Device

Rating: informational
Description:
The CWA app on Android implements Googles SafetyNet Attestation API 2 to verify the integrity of a user device. With
publicly available tools used for rooting Android devices (e.g. Magisk 3 ), it is possible to force a successful attestation
check, even if the device is rooted or modified. Whether it is possible to circumvent the attestation check or not generally depends on whether the device allows hardware-backed attestation or not. If hardware-backed attestation is
unavailable, the check will default to basic attestation, which should be considered as broken. For devices defaulting to
hardware-backed attestation, there currently exists a workaround, which forces the device to basic attestation anyway,
hiding the fact that hardware-backed attestation is actually available.
Even though this workaround will likely get patched in future system updates, it currently should not be expected that
SafetyNet Attestation guarantees a successful detection of rooted devices, or devices running customized firmware.
Proof of concept:
The following screenshots show a successful attestation on a rooted device:

Recommended controls:
It should be taken into account, that a motivated attacker is able to circumvent SafetyNet Attestation even on the most
recent devices, which allow for hardware-backed attestation. It should be noted though that, even as it is currently
possible to circumvent SafetyNet Attestation, this feature still raises the bar for potential attackers and makes attacks
more difficult and thus less likely.

Internal Tracking ID EXPOSUREAPP-5594

I have just checked the linked Jira issue.
Jira Ticket is flagged as:
Resolution: Wont Fix
Status: Obsolete
Unfortunately, no additional information was added to this Jira ticket.


When property "strutsConsole.enabled" is true, I still cannot open GUI 1.2

Describe the bug
When I try to select GUI 1.2 in the main menu of the FF!, I get the error message shown at the screenshots section. That text asks to set property "strutsConsole.enabled" to true. But when I do this, the same error shows up again when I want to open GUI 1.2.
Reporter
Martijn Dirkse
Category

Frank Console

To Reproduce
Steps to reproduce the behavior:

Start the Frank!Framework with some configuration. Ensure that property "strutsConsole.enabled" is set to true.
In the main menu, select "GUI 1.2"
You see the error.

Expected behavior
With the mentioned property set to true, you should see GUI 1.2 when you select it in the main menu.
Screenshots

Environment:

Apache Tomcat
Microsoft Edge
FF! version 7.6-20200619.155110

Additional context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

Hi Martijn,
I'm assuming you've restarted your server after setting the propery. Did you also reload your browser?

I successfully opened GUI 1.2. I did the following:

I had Microsoft Edge open. There was no tab with the Frank!Framework.

I started the Frank!Runner with the following command:
start.bat -DstrutsConsole.enabled=true -Djdbc.migrator.active=true

The Frank!Runner downloaded version 7.6-20200624.105213 of the Frank!Framework and installed it.

In the main menu, I selected "GUI 1.2". That GUI appeared.

I think this is a duplicate of #892, can you confirm this?

Yes, it is a duplicate.

Okey, then I'll close this issue.

I still have to refresh my browser before it works.

I remember again: When I restart the Frank!Runner, the GUI looses the connection with the server. It cannot be expected to work without a browser refresh. I have thus verified this issue.


Use HTTPS for the official websites

chainer.org currently does not use HTTPS, while it may have a negative effect on web search ranking (e.g., Google uses it as a ranking signal). We should consider making the website support/force HTTPS even though we do not have any POST requests. This would also be applied to the official documentation hosted via readthedocs.

What's the blocking factor of this issue?

AFAIK, there is no critical blocking factor, and there is only a matter of human resources.


Why request dependency pinned to <=2.18.4

the request library is already at 2.19.1 https://pypi.org/project/requests/2.19.1/
Why is the library requiring version <=2.18.4 is there a known problem?
I wan to install latest version 2.19.1 and use this version in IBM Cloud Functions runtime

installing latest version I get
ibm-cos-sdk-core 2.3.0 has requirement requests<=2.18.4,>=2.12.0, but you'll have requests 2.19.1 which is incompatible.

Hi @csantanapr  - I will take a look....

Hi  Barry,
Looking at this fellas queries on github:

We would need to use the latest setup.py from aws (includes support
for 3.7)

remove support for python 2.6

customise for ibm

Needs to tested against 3.7 and 2.7 but could be put in next weeks patch
release.
Regards
Connell
On 28/09/2018 01:16, Carlos Santana wrote:

installing latest version I get
|ibm-cos-sdk-core 2.3.0 has requirement requests<=2.18.4,>=2.12.0, but
you'll have requests 2.19.1 which is incompatible. |
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IBM/ibm-cos-sdk-python-core/issues/4#issuecomment-425281456,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgsN5NxkalV2dX-6nfrGnInu70tl7Axiks5ufWpdgaJpZM4W9px_.

Thanks @cpcunningham 👍

hi @csantanapr  - There is a monthly release scheduled for the 12th October that will remove this restriction.


errors casting json values don't provide location

It would be much more helpful if errors gave a path to the error location in the json data. Ideally this information would be both obvious to humans reading the error message and available programmatically.
>>> con.query("SELECT <tuple<a: int64, b: str>><json>$0", '{"a": 1}')
InvalidValueError: invalid null value in cast

>>> con.query("SELECT <tuple<a: int64, b: array<str>>><json>$0", '{"a": 1, "b": [2]}')
InvalidValueError: expected json string or null; got json number

EdgeDB Version: package:1.0b2.dev5500+g2f67186c8-2021030200~focal.nightly
OS Version: Linux 5.8.0-44-generic #50~20.04.1-Ubuntu SMP Wed Feb 10 21:07:30 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

This should be possible but it will touch a wide range of things.
We'll want to report paths in the json something like:

."a"
".b"[0]`
for the examples above.

Our big composite casts from json work by breaking down the operation into a bunch of smaller casts, so we'll need to arrange to pass down extra information for error messages into those casts, and create a way for that to be passed into the cast functions.
It's possible that actually including the indexes in array casts will make it too expensive, though, in which case we'd need to be a bit less ambitious.


TriggerDagRunOperator - How do you pass state to the Python Callable

Does anyone have an example how to use the payload with TriggerDagRunOperator?
How can I get the payload on triggered dag?
Can someone provide and example?
I was looking at #1010, and prefer to open a new issue.
I imagine it could better separate the issues.

Hi!
Good question and the answer is that it is not exposed currently in any useful way to allow setting it : https://github.com/airbnb/airflow/blob/master/airflow/operators/dagrun_operator.py#L46
I don't it is needed for most needs.
from airflow import DAG, utils
from airflow.operators import *
from datetime import date, datetime, time, timedelta

def foo(context, dag_run_obj):
    print "INSIDE FOO = {}".format(context['params']['passed_by_me'])
    if True:
        return dag_run_obj

dag = DAG(dag_id='test_trigger_dag_run_for_Sid_1',
          default_args={"owner" : "me",
                        "start_date":datetime.now()},
          schedule_interval='*/1 * * * *')

trigger = TriggerDagRunOperator(task_id='test_trigger_dagrun',
                                trigger_dag_id="simple_dummy_dag_v1",
                                python_callable=foo,
                                params={'passed_by_me':1},
                                dag=dag)

If you want to pass some state to the python callable via the TriggerDagRunOperator, just follow the example above.

Excuse me if I am being presumptuous about your needs. I will close this for now, but if you need something else, you can ping me gitter.

@r39132 Thanks for the response.
I dont think I made me clear. Imagine that I have a DAGS that trigger other DAG. So DAG1, DAG2 and DAG3 can trigger DAG4 using TriggerDagRunOperator.
Every DAG that triggers DAG4 will pass some parameters and I would like to get this parameters on DAG4.
Am I clear?
Thansk again!!!

Read https://www.linkedin.com/pulse/airflow-lesson-1-triggerdagrunoperator-siddharth-anand?trk=pulse_spock-articles
And look at the examples i checked in yesterday.
https://github.com/airbnb/airflow/blob/master/airflow/example_dags/example_trigger_target_dag.py
kwargs['dag_run'].conf['message']

If this does not make sense, then ping me https://gitter.im/airbnb/airflow

Currently, you need to use a python operator to access this data, but then you can use xcom to push that data around within the dag.

Since it's in the context, you can also reference it within a template {{ dag_run.conf['foo'] }}, I'm adding that to the example.

https://github.com/airbnb/airflow/commit/2a3e526a30146abc3bb3af36a04ab8b24b48eb76

A related use-case is to trigger a dag-run from the command line, with a particular configuration.  We have achieved this by copying the creation of a DagRun object of the TriggerDagRunOpeartor, into a stand-alone python script.
Would it make sense to expose the creation of DagRun's in some method?
The code I have had to copy is:
def CreateDagRun(args):

    dict_args = vars(args)
    session = settings.Session()
    dr = DagRun(
        dag_id=args.dag_id,
        run_id=args.run_id,
        conf=dict_args,
        external_trigger=True)
    logging.info("Created DagRun {} with parameters: {}".format(dr, dict_args))
    session.add(dr)
    session.commit()
    session.close()

where args is an argpase derived set of objects.  Some thought should be given to how the arguments are exposed.

Great question and a great use-case!

@BradHolmes This is now supported in the CLI itself thanks to @mistercrunch
This is also documented on https://www.linkedin.com/pulse/airflow-lesson-1-triggerdagrunoperator-siddharth-anand

Super!  Thanks for the fix, and for the excellent write-up.


Do not automatically delete com.apple.quarantine attribute

The Homebrew installation script automatically removes quarantine as way to make it easier to install and run the application, as stated in the README:

Homebrew installation script is configured to automatically delete com.apple.quarantine attribute, that's why the app should work out of the box, without any warnings that "Apple cannot check AeroSpace for malicious software"

However, while I believe this is a. non-issue for most cases, I also believe it is a bad practice and could be seen as a security risk in some cases (e.g. company approved software).
A much better option would be to explicitly ask the user, at the end of Homebrew installation, if they want to delete com.apple.quarantine attribute, or provide instructions to do it manually.

and could be seen as a security risk in some cases (e.g. company approved software).

Is it your real case? (Your company didn't approve the software) Or are you speaking purely hypothetical?

This is exactly my case.
And to tell the truth, I don't think new software will be approved at all here. 🤷
But I still think it is a better user experience to let the user know what the install script is doing.

Hi @nikitabobko ,
I don't know if anything changed on this matter, but AeroSpace is now approved on my company. It install and works without issues now. 🤷
I'm happy with that and replaced Amethyst with it. :)
I hope I can contribute with the project, if not with Code at least with feedback and maybe some scripts.
Thanks for the great work. <3


Unable to link repository (Webhook API error)

URL to the linked Repo/Org: https://github.com/eclipsesource/jsonforms-tooling
Steps to reproduce the problem:

Login to CLA Assistant
Setting up CLA for a new repository (organization)

What is the expected result?
CLA Assistant integration successful:

Webhook is created
CLA Assistant shows repository in list of linked repositories
CLA Assistant reacts to PRs

What happens instead?
CLA Assistant indicates in UI that linking was successful, however after reloading the page the repository is not shown anymore.
The console shows the following errors:

Additional information

It doesn't matter if choosing a predefined CLA or a gist, both don't work
The UI popup indicates linking was successful
The new repository shows up an in the list of linked repositories, however only the checkmark for gist is given but not for webhook. After reloading the page, the repository no longer shows up in the list of linked repositories.
Retrying after log out / login in both Github and CLA assistant doesn't work
I have the required rights to the repository.
I tried setting up the webhook in the repository manually (copied the settings from another repository which was linked to CLA assistant before) and indeed the checkmark for webhook is then also shown when linking the repository. However CLA assistant still doesn't react to PRs and when reloading the page, the repository again vanishes from the list of linked repositories.

@sdirix your repository is stored in the DB with wrong repository id (I don't know how this could happen). We crosscheck the data stored in our DB with 'real' data from GH, that's why cla assistant can't find it and list it as linked one.
I've removed the DB entry manually. Could you try to link the repo again, please?

Thanks for looking into this issue! Yes, the linking now worked. Thanks for the efforts!
One more thing I don't understand, maybe this is related to the database issue so I will mentioned it here. If it's an own issue I'll open another issue if you prefer.
We share the same gist for multiple repos, however it seems that a different number of people are actually linked.
For privacy reasons I will use A,B,C for numbers where A is the largest and C the lowest.

https://github.com/eclipsesource/jsonforms is signed by A people
https://github.com/eclipsesource/jsonforms-ui-schema-editor and four other repositories are signed by B people
https://github.com/eclipsesource/jsonforms-typescript-seed is signed by C people.
When linking the new repository https://github.com/eclipsesource/jsonforms-tooling the number of signed people is shown as C.

All of these repositories are linked to the same gist and all of them have shared set to true. Also when looking at the signers, they form a subset relation: signers of C is a subset of signers of B which is a subset of signers of A. Is there a way to just sync them all to the list of A signers?

This is possible in case the repo jsonforms and jsonforms-ui-schema-editor haven't had shared flag previously and had some contributors before you changed it to shared.
The only workaround to bring them to the same set of contributers is to export the set A and import it again for the repo jsonforms-tooling e.g.

Thanks for the advice. I'll close this as the original issue was resolved. Thanks again!


{Packaging} Fix #17939: Relax PyJWT version

Description
Fix #17939
ADAL already supports PyJWT > 2.0 (https://github.com/AzureAD/azure-activedirectory-library-for-python/pull/246), it is safe for Azure CLI to use PyJWT 2.1.0.

Packaging


⚠ captana.io | API has degraded performance

In ccd2333, captana.io | API ($CAPTANAIO_API_BACKEND_CHECK) experienced degraded performance:

HTTP code: 200
Response time: 3510 ms

Resolved: captana.io | API performance has improved in fa01e19.


TypeError: Expected int32, got list containing Tensors of type '_Message' instead.

My code is
model = Sequential()

model.add(Convolution2D(nb_filters, kernel_size[0], kernel_size[1],
                        border_mode='valid',
                        input_shape=input_shape))
model.add(Activation('relu'))
model.add(Convolution2D(nb_filters, kernel_size[0], kernel_size[1]))
model.add(Activation('relu'))
model.add(MaxPooling2D(pool_size=pool_size))
model.add(Dropout(0.25))

model.add(SpatialPyramidPooling([1, 2, 4]))
model.add(Dense(128))
model.add(Activation('relu'))
model.add(Dropout(0.5))
model.add(Dense(nb_classes))
model.add(Activation('softmax'))

But it can`t work, here is the error
Traceback (most recent call last):
  File "/Users/liyuan/Desktop/keras-spp/test_spp.py", line 20, in <module>
    model.add(SpatialPyramidPooling(pooling_regions, input_shape=input_shape))
  File "/Library/Python/2.7/site-packages/keras/models.py", line 299, in add
    layer.create_input_layer(batch_input_shape, input_dtype)
  File "/Library/Python/2.7/site-packages/keras/engine/topology.py", line 401, in create_input_layer
    self(x)
  File "/Library/Python/2.7/site-packages/keras/engine/topology.py", line 572, in __call__
    self.add_inbound_node(inbound_layers, node_indices, tensor_indices)
  File "/Library/Python/2.7/site-packages/keras/engine/topology.py", line 635, in add_inbound_node
    Node.create_node(self, inbound_layers, node_indices, tensor_indices)
  File "/Library/Python/2.7/site-packages/keras/engine/topology.py", line 166, in create_node
    output_tensors = to_list(outbound_layer.call(input_tensors[0], mask=input_masks[0]))
  File "/Users/liyuan/Desktop/keras-spp/SpatialPyramidPooling.py", line 110, in call
    outputs = K.concatenate(outputs,axis = 0)
  File "/Library/Python/2.7/site-packages/keras/backend/tensorflow_backend.py", line 1427, in concatenate
    return tf.concat(axis, [to_dense(x) for x in tensors])
  File "/Library/Python/2.7/site-packages/tensorflow/python/ops/array_ops.py", line 1029, in concat
    dtype=dtypes.int32).get_shape(
  File "/Library/Python/2.7/site-packages/tensorflow/python/framework/ops.py", line 637, in convert_to_tensor
    as_ref=False)
  File "/Library/Python/2.7/site-packages/tensorflow/python/framework/ops.py", line 702, in internal_convert_to_tensor
    ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
  File "/Library/Python/2.7/site-packages/tensorflow/python/framework/constant_op.py", line 110, in _constant_tensor_conversion_function
    return constant(v, dtype=dtype, name=name)
  File "/Library/Python/2.7/site-packages/tensorflow/python/framework/constant_op.py", line 99, in constant
    tensor_util.make_tensor_proto(value, dtype=dtype, shape=shape, verify_shape=verify_shape))
  File "/Library/Python/2.7/site-packages/tensorflow/python/framework/tensor_util.py", line 367, in make_tensor_proto
    _AssertCompatible(values, dtype)
  File "/Library/Python/2.7/site-packages/tensorflow/python/framework/tensor_util.py", line 302, in _AssertCompatible
    (dtype.name, repr(mismatch), type(mismatch).__name__))
TypeError: Expected int32, got list containing Tensors of type '_Message' instead.

I don`t know how to solve the error:
TypeError: Expected int32, got list containing Tensors of type '_Message' instead.

my keras version is 1.2.1, and my tensorflow version is 1.0.1

The currently supported keras version is 2.0. Could you try upgrading ?

After I upgrade keras, it does work. Thank a lot!


Modifying authenticated user and persistence of the flash messages

While working with Behat and the Laravel Extension I have encountered the following issues:

After we modify the authenticated user the changes are not visible until the user relogs. That’s probably due to requests, which are made using the same instance of the application. If the user is signed in, it's stored in the memory to the end of a scenario.

https://github.com/laravel/framework/blob/5.0/src/Illuminate/Auth/Guard.php#L134-L137

The flash messages are persistent. They should disappear after page’s reload but that doesn’t happen.

I have created a repository with exemplary tests, which show above mentioned issues.
https://github.com/tmalecki/l5-behat-test
Thanks.

+1, at the moment Mink reload() Method is a little confusing because of that (flash messages, filled in form fields etc). As a workaround a "true" reload can be implemented in a context class or a trait like this:
/**
 * Reload page and delete session data.
 *
 * @When I reload page and flush session
 */
public function reloadAndFlush()
{
    Session::flush();
    $this->reload();

}

@bcat-eu thanks!
going to close this feel free to reopen if needed.


Rename codepoint to gid

The use of codepoint here is confusing since it is actually a glyph id
(HarfBuzz calls it that way probably because the buffer holds actual
code points before shaping).

Thanks, that's annoyed me for a while but not enough to do anything about. :-)


[ECard Settings] Wallet - Clicking on Address does nothing after submitting empty phone number

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.24-7
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Log in to<EMAIL_ADDRESS>Go to Settings > Wallet > Expensify card.
Click Reveal details > Update address > Save.
Click Get physical card.
Enter invalid phone number > Next.
Click Get physical card.
Click Address. Note that you can open Address page.
Return to previous page.
Click Phone number > Clear phone number field > Next.
Return to previous page.
Click Address.

Expected Result:
Address page opens.
Actual Result:
Nothing happens when clicking Address.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/43996225/f3cd7235-644d-4fc6-9889-af68942d35a4

View all open jobs on GitHub

@grgia assigning you for initial review, then let's pass this off to a contributor that's been involved.

Going to see if the callstack will take this one

asked here

I am Pedro Guerreiro from Callstack - expert contributor group. I’d like to work on this job!

@grgia @kavimuru What's the password for the user<EMAIL_ADDRESS>?

Thanks for the patience @pac-guerreiro, assigning now!

<EMAIL_ADDRESS>
@pac-guerreiro you don't need access to this account (we don't give out expensifail emails), but you should be able to test the form flow bugs in DEV

@grgia Ok, but I'm not able to replicate the exact behaviour because I don't have a valid card, I can only mock it. Nonetheless, from my tests it seems to work!

PR ready, have fun! 😄

@kevinksullivan could you please attach payment summary 🙇

payment request here https://staging.new.expensify.com/r/2938310154194899

Awaiting payment summary from @kevinksullivan

Payment summary:

@rushatgabhane C+: $500

$500 approved for @rushatgabhane


Find bottleneck

There is a bottleneck in gemmlowp mode.

It appears that finding parameters for gemmlowp quantization is inside the image iteration loop. These parameters need to be determined and stored before run-time (perhaps do this outside the scope of the experiment routine and store them).

In the process of fictionalization of offline processes. After, must figure out where to call them outside of main loop and pass their return values to the glp function.

Left to do is to read in the min and max floating point multiplication result values from csv (in features/) and pass them to glp in gemmlowp mode.


weather widget showing errors after load frequently

I know there might be some change to Yahoo! 's side, but there are a lot of issues with the query not working after loading the page. Sometimes it won't load, then other times it will work.
is there a way to look into the fix for this?

I'm also seeing this issue.  I can't get it to show at all now.

+1 on this issue. Also getting an error message: "There was a problem retrieving the latest weather information."

Working now for me.

I have a Pen on CodePen using this plugin - it has been working.

Having sambe problem time to time it gives me the error message: "There was a problem retrieving the latest weather information."

Same. It's been happening intermittently (but often) over the past several weeks. What can we do?

The best alternative if Yahoo is proving too unstable is to use another weather API like https://darksky.net/dev/docs or https://openweathermap.org/api. Both require creating an account and using an API key which is why simpleWeather has never switched.


.git is being deployed from VScode

When creating a function via VScode, it generates a .functionignore where .git should be ignored via the first line ".git*" - This is not the case.
After deployment of a function to Azure .git is also deployed, when I change .functionignore from ".git*" to ".git" - .git is not being deployed.
The Zip package size is greatly reduced by changing .functionignore
Looking at the main branch It seems the ".git*" filter is still active.
Source:
src/commands/createNewProject/ProjectCreateStep/ScriptProjectCreateStep.ts
Azure Functions 1.13.1
VScode 1.83.0
Windows 11 Enterprise Version 10.0.22621 Build 22621
WSL2 - Ubuntu 22.04.3 LTS

QQ: Are you using .functionignore or .funcignore?  FYI: I've only seen the latter in documentation.

I just checked I'm using .funcignore, I think my my spellchecker "helped" me.

Hi, @iobear a fix for this has been released in v1.13.2 of the extension.


writePSQLFile: init

Adds a function called writePSQL which beautifies and validates the input SQL text, created at nix.camp 2022
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg eval

This seems like an elegant refinement.notich is needed in the way of testing, but if i understand right, ot could break a lot of legacy sqp (this is a meta situation).
Could someone at the nix.camp fire up tree and sbuild a pile of (if not all amd any which contain sql , either in the derivation or in the installer (tribler used to do this)) derivations with this check added
@zimbatm this seems like up your alley, could we trouble you for a thumbs up or amy other feedback? I cam give you a lush ride to/from the camp for your helpful critixism)

@zimbatm I was making a NixOS VM Test with https://github.com/NixOS/nixpkgs/pull/191201 and it would have been nice to use writePSQL to write the sql files for the initialScript in that test, for example.

Are there any other use cases you can think of?
I'm the grumpy old that doesn't want children to have fun :)
Usually, the best is to start with the use cases, and then extract the common functions. That way, we don't end up with a super large API surface that is unmaintained.

@zimbatm @SuperSandro2000 I've refactored this function in order to use psql and the postgresqlTestHook instead of some perl script (pgformatter). Please review this instead :D

Does postgresql not have a static check mode, where the file is just checked for whether the engine would accept it? I feel like loading it into a temporary database might not work for a lot of files (e.g. you can’t run any UPDATEs without already having a table set up).

@Profpatsch That is the intention of this PR, and if you know of such an option, that would be good to use.

Are there plans to use that helper function in nixpkgs itself?
Without actual usage, these types of functions tend to rot, so it's best to keep them out of nixpkgs. Otherwise, let's also add some usage of it to this PR.
Technically it looks valid

There cannot be actual usage until it is merged into nixpkgs, but I do understand that a lot of the writers are underutilized.


Misc. fixes on filtering, added commits-by-year

The start date was arbitrary at 2005, changed to actual first commit.
Filter by years in by-month and by-week was incorrectly used eg #135
Other greps also could match on [0-9]{4} in commit (hash or message)
Added commits-by-year histogram

Thanks for your contribution to the project! I am having trouble with this code, however:
Git commits by year:

        year    sum
./git-quick-stats: line 643: test: 1: unary operator expected
./git-quick-stats: line 643: test: 2: unary operator expected
./git-quick-stats: line 643: test: 3: unary operator expected
./git-quick-stats: line 643: test: 4: unary operator expected
./git-quick-stats: line 643: test: 5: unary operator expected
./git-quick-stats: line 643: test: 6: unary operator expected
./git-quick-stats: line 643: test: 7: unary operator expected

< sniped for brevity and to reduce spam >

./git-quick-stats: line 643: test: 2006: unary operator expected
./git-quick-stats: line 643: test: 2007: unary operator expected
./git-quick-stats: line 643: test: 2008: unary operator expected
./git-quick-stats: line 643: test: 2009: unary operator expected
./git-quick-stats: line 643: test: 2010: unary operator expected
./git-quick-stats: line 643: test: 2011: unary operator expected
./git-quick-stats: line 643: test: 2012: unary operator expected
./git-quick-stats: line 643: test: 2013: unary operator expected
./git-quick-stats: line 643: test: 2014: unary operator expected
./git-quick-stats: line 643: test: 2015: unary operator expected
./git-quick-stats: line 643: test: 2016: unary operator expected
        1       0       |
        2       0       |
        3       0       |
        4       0       |
        5       0       |
        6       0       |
        7       0       |
        8       0       |
        9       0       |
        10      0       |
        11      0       |
        12      0       |
        13      0       |
        14      0       |
        15      0       |
        16      0       |
        17      0       |
        18      0       |

< sniped for brevity and to reduce spam >

        2000    0       |
        2001    0       |
        2002    0       |
        2003    0       |
        2004    0       |
        2005    0       |
        2006    0       |
        2007    0       |
        2008    0       |
        2009    0       |
        2010    0       |
        2011    0       |
        2012    0       |
        2013    0       |
        2014    0       |
        2015    0       |
        2016    0       |
        2017    85      |████████████████████████████████████████████████████████████████████████████████

 Generate:
    1) Contribution stats (by author)
    2) Contribution stats (by author) on a specific branch
    3) Git changelogs (last 10 days)
    4) Git changelogs by author
    5) My daily status
    6) Output daily stats by branch in CSV format
    7) Save git log output in JSON format

 List:
    8) Branch tree view (last 10)
    9) All branches (sorted by most recent commit)
   10) All contributors (sorted by name)
   11) Git commits per author
   12) Git commits per date
   13) Git commits per month
   14) Git commits per year
   15) Git commits per weekday
   16) Git commits per hour
   17) Git commits per hour by author
   18) Git commits per timezone
   19) Git commits per timezone by author

 Suggest:
   20) Code reviewers (based on git history)

Please enter a menu option or press Enter to exit.
>

This occurs on Ubuntu 20.04 in WSL, Fedora 33, and RHEL 7 machines.

I had a feeling about that grep. I could reproduce it on another repository however, added a fix. Could you give it a test?
Do you mean the README.md? I could give it a go.

This is looking pretty good :) I have a couple of notes:

Try running shellcheck  on your portion of the code. If there are variables we can quote or legacy syntax we can modify, that would be a big help in robustness. Note that some variables can't easily be quoted because they are expecting to be split via whitespace. Existing code can be used to judge what needs to be changed. Off the top of my head, prefer $(..) to backticks, and some of your since/until variables can probably be quoted without issue
I noticed that on my Fedora 33 and WSL Ubuntu machine, the ordering for the years was correct. However, on my RHEL 7 machine, it put 2017 at the end. Double-checking and verifying the years are correctly sorted before awk prints them might be worth pursuing.
This is 100% stylistic, but I tend to prefer [ or [[ over test
It looks like the manpage still needs to be updated to reflect the new functionality

I have looked at the shell check issues in my changes and eliminated all notices.
The manual page is updated now.
I right now still looking where grep -E "($startYear|$endYear)" comes from, in what commit and why was it added.
For now just referenced tickets that encounter issues with it (but some also confuse commit vs author date, at least initially).

This is working great on all three Linux distros I checked. Good work! However, as you alluded to, it does indeed break when BSD grep is used. Specifically on macOS 11.6.2 with grep 2.5.1. We try to maintain compatibility with this flavor, too. It looks like the -P option may be the issue:
Git commits by year:

usage: grep [-abcDEFGHhIiJLlmnOoqRSsUVvwxZ] [-A num] [-B num] [-C[num]]
	[-e pattern] [-f file] [--binary-files=value] [--color=when]
	[--context[=num]] [--directories=action] [--label] [--line-buffered]
	[--null] [pattern] [file ...]

Okay. so that grep is does a lookahead/behind to avoid matching a +,-HHMM timezone spec, and return year from a date. I will have to explore some alternate solutions.

This is the first thing I can think of. Replaced grep -Po with sed -E.

Confirmed working as expected for the following platforms:

Ubuntu 20.04 LTS (via WSL)
macOS 11.6.2
CentOS 7
Fedora 33

Good work, and thanks for the contribution!

Attached are four screenshots for the README.md to display the new functionality. Could you update their respective URLs the images are pointing to with these new ones provided when you get a chance? Thanks!

Interesting way to add images. Done. However.
I noticed the $_pathspec argument / _GIT_PATSHPEC setting while I was doing some more hacking. That was not included in commitsByMonth (that I copied) but it is used in many other commands. Including commitsPerDay. Could be related to
#92.
I think this should be supported by all the per-date commands as well. What do you think? It is missing from a lot of commands that could easily support this. Another: csvOutput supports it but jsonOutput not.
I could add this to:

-j / jsonOutput
-b / branchTree
-Y / commitsByYear
-m / commitsByMonth
-w / commitsByWeekday
-o / commitsByHour

If I can ask one mroe question of you.
The headers 'Generate' and 'List' in the menu have me confused a bit.
I am really interested in showing the "actual" changes made to a project over a period and writing stats about that. (This is somewhat open to interpretation, but Git can measure this at least by 'chunks' added/removed, using diffstat. I'm also thinking about LoC). This requires some script to be performed to collect the stats for each revision (with certain filter settings). I have now:
--changes (commits sorted by chunks inserted/added)
--changes-by-author (idem. but grouped by author)
--changes-per-year (group on year)
If I would add this in another PR, where should these be under you think?

Quite a few things in here, so I'll try to address them in order.
Re: _GIT_PATHSPEC - This is correct, it is not used in every command. Every command it is used in:
detailedGitStats()
changelogs()
csvOutput()
contributors()
commitsPerDay()
commitsByTimezone()
suggestReviewers()

Every command it is not used:
myDailyStats()
jsonOutput()
branchTree()
branchesByDate()
commitsPerAuthor()
commitsByMonth()
commitsByWeekday()
commitsByHour()

Although double-check this to make sure it's true.
This does indeed look like an oversight on our part and should be fixed. It would be best if this were a separate PR, IMO.
Regarding #92, the user stated the following:
"Filtered to my directory of interest, generating "Contribution stats (by author)" gives me an empty list. However, the "My daily status" options appears to work correctly, and the list "Git commits per author" shows me stats for the entire repo."
Looking at where this variable is used, it would help explain the differences between what they are seeing. Interestingly enough, the one without _GIT_PATHSPEC is the one where they say the output is correct, so that would probably take some further investigation as to why that one works and the other one does not. You are correct that these seem to be related.
Re: JSON output issue - Could you report this as a bug with a breakdown of how it is failing for us so we can keep track of it?
Re: Generate vs List - These were made before I joined the project. To be honest, they are somewhat arbitrary in nature and could probably use a revamping. My personal opinion is that "Generate" should deal with anything that does any custom calculations based on the git log, and "List" is more for formatting git's log so it's easier to read like with bar graphs. Again, this is arbitrary and you can argue this isn't consistent. For example, "Branch tree" should probably be under "Generate" since it's generating an ASCII-style tree graph based on git commits. If you'd like to take a stab at adjusting them, you're welcome to in another PR.
For the specific ones you mentioned, are you referring to the third party tool called diffstat which is outlined here? https://linux.die.net/man/1/diffstat
If so, this kinda goes against the spirit of the tool by attempting to do everything with bash and standard built-in utilities across POSIX-compliant OSes like sed and awk. The reason I say that is because none of my machines have that utility installed on them, and it would be a package this one would be dependent on.

Also, please let me know when you're done with this so I can do a final check and merge it into the codebase :)

Copy on the separate PR's, I'll will remove the last commit from my master so the PR can get a final inspection move forward.
Also I will see about the two issues separately.
What I meant was git diff --stat. The utility you link is similar but not required from what I can see. But it is a histogram that shows the actual chunks removed and added per file (and the ratio) plus a summary line ::
$ git diff --stat
 Sys/box.inc   | 8 ++++++++
 Sys/check.inc | 3 +++
 Sys/info.inc  | 9 ++++++---
 3 files changed, 17 insertions(+), 3 deletions(-)


warning from clang when using ocaml

Describe the bug
I have opam installed on my system and manage ocaml compiler with it. Anytime I compile something I get this warning:
clang-16: warning: argument unused during compilation: '-idirafter /nix/store/...-glibc-2.38-44-dev/include' [-Wunused-command-line-argument]

Steps To Reproduce
Steps to reproduce the behavior:

install opam package
try to build something

Expected behavior
These warnings mess up compiler and dune outputs, making them hard to read. I would like to remove these warnings.
Additional context
Maybe this comes from something wrong on my side. In this case please let me know what I should do to fix that.
Notify maintainers
@henrytill @marsam
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
these 6 paths will be fetched (0.69 MiB download, 10.03 MiB unpacked):
  /nix/store/g0kc0z0wzs2s4kl377zssyxmmk5z9vim-file-5.45
  /nix/store/ihhhd1r1a2wb4ndm24rnm83rfnjw5n0z-gcc-wrapper-12.3.0
  /nix/store/7n1vikpxsl1gs5h8f0f8zqfj2mcdy1ii-gnu-config-2023-09-19
  /nix/store/5vyb54sgl2x1ihrdfvj46vi2w1acv9nn-gnumake-4.4.1
  /nix/store/10i1kjjq5szjn1gp6418x8bc1hswqc90-stdenv-linux
  /nix/store/lfldnv3apxsd59g1jl65kw6xfkmv5w27-update-autotools-gnu-config-scripts-hook
copying path '/nix/store/7n1vikpxsl1gs5h8f0f8zqfj2mcdy1ii-gnu-config-2023-09-19' from 'https://cache.nixos.org'...
copying path '/nix/store/g0kc0z0wzs2s4kl377zssyxmmk5z9vim-file-5.45' from 'https://cache.nixos.org'...
copying path '/nix/store/ihhhd1r1a2wb4ndm24rnm83rfnjw5n0z-gcc-wrapper-12.3.0' from 'https://cache.nixos.org'...
copying path '/nix/store/5vyb54sgl2x1ihrdfvj46vi2w1acv9nn-gnumake-4.4.1' from 'https://cache.nixos.org'...
copying path '/nix/store/lfldnv3apxsd59g1jl65kw6xfkmv5w27-update-autotools-gnu-config-scripts-hook' from 'https://cache.nixos.org'...
copying path '/nix/store/10i1kjjq5szjn1gp6418x8bc1hswqc90-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.82, NixOS, 23.11 (Tapir), 23.11.5648.44733514b72e`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(mik): `""`
 - channels(root): `"home-manager-23.11.tar.gz, nixos-23.11, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

I believe this warning doesn't come from Opam. IIUC it comes from cc-wrapper: https://github.com/NixOS/nixpkgs/blob/8e94a18639fecb4e8572298eb49dba72efae337d/pkgs/build-support/cc-wrapper/default.nix#L521-L522 My guess is that Opam is using your system's Clang as compiler, and it's showing those warnings.

umm ok . I have clang in my configuration, added it so opam can build the compiler.
is it possible to change this behavior or is it better to do without opam ? I realize ocaml compiler is managed by opam. So maybe I could use ocaml from nix packages and don't use opam at all using only packages from nix and use nix-shell for specific purpose environment. I am not sure about the behavior of dune in this case.
what do you think about this?


Is there a license for this repo?

Hi there -- I came across this repo, and I really like it -- I had been thinking I'd have to write something myself to parse the health data, and I'm glad to see someone has beat me to the punch. :grin:
Is there a license for this repo? I would really like to be able to be able to use this as a base for other things, but I don't want to step on any toes.
Thanks again for publishing this!

Hey @saintaardvark 👋 , thanks for asking! I've just committed 2dab351eef7f114c2555f1a09f8851f3da32f300 which includes the MIT license.

That's awesome -- thank you so much for sharing!


Google PaLM (Independant Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @vmanoharas  Thank You so much for your support in the submission process. There are 2 non functional updates to the recently published connector. Request your guidance in the changes I have to make from my end.

Can you please assist in adding a Co-Publisher in the connection page and documentation. He had helped in the last rounds of the development. It will be of great help as I missed to add his name in the publisher details. His name is Vidya Sagar Alti.

While creating the connection, could see for other connectors that the publisher name is being shown as below :

However for the current published connector(as shown below) , the publisher names are not visible. Can we please add both the Publisher names (Priyaranjan KS and Vidya Sagar Alti) here ?

Thank You once again for your support !
Regards,
Priyan

[[certify-connector]]


Response from remote is not json

Hi, I'm currently trying to build FMI4cpp with conan. But whenever I attempt to I get this error in the terminal:
"Response from remote is not json, but 'text/html; charset=utf-8'. [Remote: ais]"
And based off installion.md I do not have the wrong url.
"conan remote list
ais: https://ais.jfrog.io/artifactory/api/conan/ais-conan-local [Verify SSL: False]
conancenter: https://center.conan.io [Verify SSL: False]"
For reference I'm currently trying to use conan 1.6, and operating through a container in WSL.

Hi, try https://github.com/NTNU-IHB/FMI4cpp/pull/139

I was able to clone #139 run, "sudo apt-get install libzip-dev libpugixml-dev", and then able to configure and build. I have one last question though. I am quite familiar with FMPy, but this is my first time interacting with FMI4cpp are there any special steps I have to take to run the examples?

You need to pass '-DFMI4CPP_BUILD_EXAMPLES=ON' if you have not already.

Ok, thank you for your help it has been very useful. I ran that command to cmake and got no errors as well as
'-DFMI4CPP_BUILD_TESTS=ON.'
But this is my actual last question. I am trying to run fmu_test.cpp. But is there something else I am missing to run it? Because I tried using run and debug with a c++ compiler and ended up with an include error for,
'#include <fmi4cpp/fmi4cpp.hpp>'
Which I assume shouldn't be an issue since everything built correctly.

It might be that you are trying to run a .cpp file and not an actual CMake target. Check if there are executables in your build folder that you can invoke from the termnial. If they work you are good. And since you built the code, it should be ok.

There doesn't seem to be any executables in the build folder despite having no build issues.

Check the out or examples folder.

I was still unable to find any.

Check the .dir files or observe the output feom CMake to find the location. Anyhow, note that this has to do with how CMake is invoked, not fmi4cpp by itself.

These are my terminal outputs.

They the files are being written to FMI4cpp itself but I do not see them in the directory. The same  for out/build/configure. This is also what one of the insides of the .dir files looks like.

Check the CI workflow for how CMake ought to be invoked.

Sorry, I am unsure what the CI workflow is. But I edited the comment to include the build terminal image as well. Unless I have mistaken that output for the build step completing.

https://github.com/NTNU-IHB/FMI4cpp/blob/vcpkg/.github/workflows/config.yml

Thank for you help I was able to run the tests with no problems and run the test_fmu.


P2P Relay Mesh Network?

Hello,
Does Goose act as a P2P Relay Mesh network?
If you have "A" connected to "B" and "B" connected to "C" then can "A" message "C" passing messages through "B"?
A <=> B <=> C

Also, what type of scaling can Goose handle?
Thanks

Yes, Goose supports a P2P relay mesh network architecture with a maximum hop of 3. In your example of nodes A, B, and C, Goose enables messages from node A to be relayed through node B to reach node C, as long as the total hop count does not exceed 3 hops.
Goose has been successfully run with a network of 30+ nodes, and I believe it has the potential to scale even further. In fact, I'm confident that Goose can handle networks with hundreds of nodes without encountering significant scalability issues.


making dummy-test parallel

You can probably just merge this without worring about previous PR?
Having separated methods, htsohm.py and htsohm_hpc.py have been refactored so that "create_material" method contains individual select_parent, dummy_test, and write_child_definition_files steps. This is done serially in htsohm.py and in parallel for htsohm_hpc.py.

Flagged needs work because I just can't remember if this is the final version that we ran with or not. We can review when you're back in town.

There are still some kinks to work out, but might this be sufficient to merge the PR?


License duplicates

The AboutListTile class that provides list of all licenses used by an app contains a lot of duplicates.
The class is located here: https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/about.dart
Please remove the duplicating entries.

What are the duplicates?

I'm sorry I just checked again and there are no duplicates.
Basically it looks like the same license is present multiple times, but in fact there are some minor differences making it technically different documents.


Potential bug in testLeapfrogVerletIntegrator

I saw this error in the nosetest. Any idea? It happened after a rerun. So I think it is something statistically very unlikely ?
openpathsampling.tests.test_toy_dynamics.testLeapfrogVerletIntegrator.test_momentum_update ... Exception RuntimeError: 'maximum recursion depth exceeded' in <object repr() failed> ignored
Exception RuntimeError: 'maximum recursion depth exceeded' in <object repr() failed> ignored
Exception RuntimeError: 'maximum recursion depth exceeded' in <object repr() failed> ignored

I saw this happen once too, but I have no idea what's going on. There's nothing recursive in this, nor is there anything stochastic. It just tests that the momentum update on a linear PES gives the correct value (which is pretty much trivial).
Was there no further traceback? It's hard to tell what's going on without that. I suppose I could add a bunch of debug statements, but I hate to do that deep in our integrator.
My best guess is that there is some weirdness in state for nose; maybe if the tests run in some particular order, some variable ends up getting redefined in some recursive fashion?
It's definitely not a bug in the integrator itself; it is probably a bug in the testing procedure.

Good enough for me. I have never seen it happen before. We are moving away from nose anyway so let's keep it for now. If someone has the same issue he will find this post.