[top] AST integration

This is a large PR that performs first pass integration of ast, sensor_ctrl and ast_wrapper
Sorry for the giant PR everyone...but there was no clear way for me to do parts of this and not have CI break.
The various commits are described below

The first 3 commits in the commit series are borrowed from #2986 , please don't bother reviewing them here.

The 4th commit adds sensor_ctrl only.  Per @imphil suggestion, sensor_ctrl is added as a ip encapsulated under hw/top_earlgrey.
@imphil @eunchan @msfschaffner can you have a look?

The 5th commit adds ast_wrapper only.
@msfschaffner @eunchan @arnonsha can you have a look?

The 6th commit reverses the clk/rst connection for a pinmux alert.
@msfschaffner please have a look.  Without this change the ast integrated design dies at 0ps.

The 7th commit updates the rstmgr to ast connection, without which chip sims will not run.
@msfschaffner @eunchan can you have look?

The 8th commit updates topgen to support non-templated top-specific modules.
@eunchan please take a look.  I rebased with your latest change..and I hope I didn't break something.

The 9th commit integrates everything together

The 10th commit makes specific changes to ast to allow DV to proceed (all single clock) and to align naming, and fixes the CI issues regarding style and commit signature
@arnonsha please take a look.  I think the best thing would be for you to cherry-pick this commit into your PR.  I can then rebase on yours.

The 11th commit makes minimal updates for fpga (fpga synth will break otherwise)

The 12th commit makes minimal updates for verilator (verilator sims will break otherwise)

The 13th commit updates the DV environment to accommodate ast clocks and resets.
@sriyerg can you have a look? The fixes aren't the cleanest, but they worked for a couple of tests I tried.

The last commit is just auto-generated files.

Hi Arnon,
The temp aon_osc change is because top_earlgrey is not yet setup (or rather
tested) for complete async operation.  I'm pretty sure there will be things
we need to fix to get it fully working.
So for now, I want to keep all the clocks in-sync.  To this end, I'm
driving all the clocks with clk_aon, since that one is needed to even power
up the chip.
To ensure the processor does not execute too slow, I'm also bumping this up
to the speed that our previous environment used.  Once AST is fully
integrated via this PR (and your other PR), I will start enabling full
async operations along with clock dividers, then I will push a separate PR
to fix other remaining items.
On Thu, Aug 6, 2020 at 3:30 AM arnonsha<EMAIL_ADDRESS>wrote:

@arnonsha commented on this pull request.
In hw/top_earlgrey/ip/sensor_ctrl/data/sensor_ctrl.hjson
https://github.com/lowRISC/opentitan/pull/3068#discussion_r466318676:

+# SENSOR_CTRL register template
+#

@tjaychen https://github.com/tjaychen
ast_wrapper looks good.
please explain the temp change in aon_osc freq.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/3068#discussion_r466318676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSU2VWZVVUWWTOK6IKDR7KA3LANCNFSM4PWAKJ6Q
.

i've addressed the comments, please have a look!


release 0.9.0 build failure

As title:

This seems to be the same as this issue: https://github.com/rust-lang/rust/issues/56398 which has been fixed in more recent Rust releases. Could you update your Rust version and double check?


Make API Guardian an optional / provided dependency.

Reverted with c8779d06f, see https://github.com/junit-team/junit5/issues/1105


BUG: Introduction to pandas lecture

In Exercise 3: Typo in 'Replace all instances of NaN using the forward fill method.'. You need use 'backward' instead of 'forward' for exercises and answers sections.

In Exercise 5b: Answer for 'Rename the columns to Good Numbers and Bad Numbers' using column names 'Useful Numbers' and 'Not Useful Numbers'.


add an example for DecodeCustom

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.049%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6957

Relevant Lines:
9148

💛  - Coveralls


Buggy dashboard menu

Whenever I'm not logged in, on forums.hummingbird.me the dashboard menupoint does nothing. After changing to another menue point, the dashboard works.

We might want to finally update the forum header, now that Discourse is more or less stable.
Thoughts?

Probably a good idea, but definitely low priority. Forum activity is restricted to a fairly small subset of the community and I have a feeling that number will decrease further with the release of group functionality.


Better error message for invalid expires field

Hello,
Currently when the user passes an "expire" option that's not a Date, they receive the message option expires is invalid.
Could you please enhance this error message so that it says something like "expire" must be a Date object? I think this would be helpful :)
Thanks.

Totally agree. I tried passing  date.toUTCString(); and received that error... the expires is not invalid, the value that is being passed is.

Totally agree. I tried passing  date.toUTCString(); and received that error... the expires is not invalid, the value that is being passed is.

how to set expire on this library? i tried to set Date but still the cookies wasn't set

You have to use a Date object, it's part of the documentation. Make sure you are using expires and not expire, which would not work.


Comparison of ZZarchiveEntry.fileMode and S_IFDIF not working on Swift

Initial problem

"if (entry.fileMode & S_IFDIR)" isn't accepted by the Swift compiler, get "Ambiguous reference to member &".

Tried to use "if entry.fileMode == S_IFDIR".

It's accepted by the compiler but, in Swift, "fileMode" returns "16877" when it's a directory. Printing S_IFDIR in debug console returns "16877".

Try this instead: if (entry.fileMode & S_IFDIR) != 0
In Swift, integers like fileMode don't automatically convert to booleans for the if statement. Hence you need the additional != 0 to convert it to boolean.


vendor/rename protobuf package

This is a copy of what we use internally, but with our extensions (that just rename fields) stripped out and with a new package name to avoid protoreflect name collisions.

Also ran the online/authenticated/paid tests with this change, and everything still passes :tada:

So I would merge, but...
The fmt/gen jobs don't actually test anything :confused: Either no files are being updated or the git diff --exit-code isn't seeing them for some reason.

I'm gonna go ahead and merge this and file an issue for the bad diff checks, since the problem predates this change. We can fix both at the same time in a follow-up.


About data migration

Can it support the automatic setting of consumer groups when kafka engine tables are migrated?

The version 1.6.0 used, the kafka engine table needs to be manually modified during data migration


Test Issue

[ Task 1]

change the decorator
@app.get('/users')

[ Task 2]

Invoke this class
db = new Database()

[Task 3]
do something else..

solved


GitHub actions

Peer Review Information
Added two workflows that will both trigger on release when published.
publish-docker-hub.yml builds and pushes the docker file to docker hub.
publish-pyp.yml installs poetry and builds the package and pushes it to pypi
publishing to the package was tested on testpypi.

A few minor things: 1. Since there are already a CI file called main.yaml, should we have a file called release.yaml for all tag release github actions? Maybe we should rename main.yaml to linting-and-unittests.yaml to make it consistent with publish-docker-hub.yml and publish-pypi.yml? 2. (this is really minor) Maybe we should use .yaml suffix to make it consistent with all yaml files in this repo? I know both .yml and .yaml are valid extensions.


No module named 'azure.storage'

Hello, I have tried the instruction to install, reinstall the azure package.
However, all the time I am still getting the ModuleNotFoundError: No module named 'azure.storage'.
following are the results of: pip freeze
azure==4.0.0
azure-applicationinsights==0.1.0
azure-batch==4.1.3
azure-common==1.1.16
azure-cosmosdb-nspkg==2.0.2
azure-cosmosdb-table==1.0.5
azure-datalake-store==0.0.34
azure-eventgrid==1.2.0
azure-graphrbac==0.40.0
azure-keyvault==1.1.0
azure-loganalytics==0.1.0
azure-mgmt==4.0.0
azure-mgmt-advisor==1.0.1
azure-mgmt-applicationinsights==0.1.1
azure-mgmt-authorization==0.50.0
azure-mgmt-batch==5.0.1
azure-mgmt-batchai==2.0.0
azure-mgmt-billing==0.2.0
azure-mgmt-cdn==3.0.0
azure-mgmt-cognitiveservices==3.0.0
azure-mgmt-commerce==1.0.1
azure-mgmt-compute==4.3.1
azure-mgmt-consumption==2.0.0
azure-mgmt-containerinstance==1.2.0
azure-mgmt-containerregistry==2.2.0
azure-mgmt-containerservice==4.2.2
azure-mgmt-cosmosdb==0.4.1
azure-mgmt-datafactory==0.6.0
azure-mgmt-datalake-analytics==0.6.0
azure-mgmt-datalake-nspkg==2.0.0
azure-mgmt-datalake-store==0.5.0
azure-mgmt-datamigration==1.0.0
azure-mgmt-devspaces==0.1.0
azure-mgmt-devtestlabs==2.2.0
azure-mgmt-dns==2.1.0
azure-mgmt-eventgrid==1.0.0
azure-mgmt-eventhub==2.1.0
azure-mgmt-hanaonazure==0.1.1
azure-mgmt-iotcentral==0.1.0
azure-mgmt-iothub==0.5.0
azure-mgmt-iothubprovisioningservices==0.2.0
azure-mgmt-keyvault==1.1.0
azure-mgmt-loganalytics==0.2.0
azure-mgmt-logic==3.0.0
azure-mgmt-machinelearningcompute==0.4.1
azure-mgmt-managementgroups==0.1.0
azure-mgmt-managementpartner==0.1.0
azure-mgmt-maps==0.1.0
azure-mgmt-marketplaceordering==0.1.0
azure-mgmt-media==1.0.0
azure-mgmt-monitor==0.5.2
azure-mgmt-msi==0.2.0
azure-mgmt-network==2.2.1
azure-mgmt-notificationhubs==2.0.0
azure-mgmt-nspkg==3.0.2
azure-mgmt-policyinsights==0.1.0
azure-mgmt-powerbiembedded==2.0.0
azure-mgmt-rdbms==1.4.0
azure-mgmt-recoveryservices==0.3.0
azure-mgmt-recoveryservicesbackup==0.3.0
azure-mgmt-redis==5.0.0
azure-mgmt-relay==0.1.0
azure-mgmt-reservations==0.2.1
azure-mgmt-resource==2.0.0
azure-mgmt-scheduler==2.0.0
azure-mgmt-search==2.0.0
azure-mgmt-servicebus==0.5.2
azure-mgmt-servicefabric==0.2.0
azure-mgmt-signalr==0.1.1
azure-mgmt-sql==0.9.1
azure-mgmt-storage==2.0.0
azure-mgmt-subscription==0.2.0
azure-mgmt-trafficmanager==0.50.0
azure-mgmt-web==0.35.0
azure-nspkg==3.0.2
azure-servicebus==0.21.1
azure-servicefabric==<IP_ADDRESS>
azure-servicemanagement-legacy==0.20.6
azure-storage==0.33.0
azure-storage-blob==1.3.1
azure-storage-common==1.3.0
azure-storage-file==1.3.1
azure-storage-nspkg==3.0.0
azure-storage-queue==1.3.0
Could  anyone help on this?
Thanks a lot!

Hi @irisava
Could you confirm version of Python, version of pip, platform (Windws, Ubuntu, etc.) and exact command used to install.
Thanks you

Hello @lmazuel
Thank you for the reply!
Following is the info of my working environment:
Python 3.6.5
pip 18.1 from ...\appdata\local\programs\python\python36-32\lib\site-packages\pip (python 3.6)
Windows 10
Last command used in cmd: pip install azure-storage

azure-storage and azure-storage-blob/file/queue are incompatible and cannot work together. azure-storage is actually the deprecated old version of the three packages azure-storage-blob/file/queue
Please juste use azure-storage-blob/file/queue or just use azure-storage but not both. azure-storage-blob/file/queue is recommended if you don't have existing code base.
Thank you,

Closing for inactivity, since I believe I addressed the initial question. If this is still a problem, feel to open a new issue in the storage repo:
https://github.com/Azure/azure-storage-python
Thanks,


Deadlock when using fsnotify on Windows

Hi, I'm new to Go and have written a very simple program to watch some logs (which rotate). However although the program runs fine for some time, eventually it deadlocks with the following message:
fatal error: all goroutines are asleep - deadlock!

The code I'm using looks like this:
t, err := tail.TailFile("path/to/some/log", tail.Config{
    Follow: true,
    ReOpen: true})
if err != nil {
    log.Fatal(err)
}

for line := range t.Lines {
    // Some action based on a log line
}

I can provide a stacktrace if necessary
BTW: I'm on Windows.

What is the full code (the .go file)? Example program can be found here: https://github.com/ActiveState/tail/blob/master/cmd/gotail/gotail.go

Here the full code, I think it has something to do with fsnotify for windows. With poll = true the problem does not occur. As you see it is much simpeler than your example, I think I implemented it correctly. Below the code also the stacktrace for the go routines.
package main

import (
	"github.com/ActiveState/tail"
	"log"
	"regexp"
)

func main() {
	t, err := tail.TailFile("path/to/log/file", tail.Config{
		Follow: true,
		ReOpen: true,
		Poll: true, // With poll = true appended the problem does NOT occur, so suspect it has something to do with fsnotify
	})
	if err != nil {
		log.Fatal(err)
	}

	r, err := regexp.Compile(`regex`)
	if err != nil {
		log.Println("There is problem with the regexp.")
		return
	}

	for line := range t.Lines {
		if r.MatchString(line.Text) == true {
			// action
		}
	}
}

Stacktrace:
fatal error: all goroutines are asleep - deadlock!

goroutine 1 [chan receive]:
main.main()
        src/bitbucket.org/rubenhazelaar/zwinq/
dcwatcher/main.go:26 +0x2e5

goroutine 5 [select]:
github.com/ActiveState/tail.(*Tail).waitForChanges(0xc08204e000, 0x0, 0x0)
        src/github.com/ActiveState/tail/tail.g
o:298 +0x922
github.com/ActiveState/tail.(*Tail).tailFileSync(0xc08204e000)
        src/github.com/ActiveState/tail/tail.g
o:265 +0xa5f
created by github.com/ActiveState/tail.TailFile
        src/github.com/ActiveState/tail/tail.g
o:113 +0x471

goroutine 12 [chan send, locked to thread]:
gopkg.in/fsnotify%2ev0.(*Watcher).readEvents(0xc082086f00)
        src/gopkg.in/fsnotify.v0/fsnotify_wind
ows.go:479 +0xbfe
created by gopkg.in/fsnotify%2ev0.NewWatcher
        src/gopkg.in/fsnotify.v0/fsnotify_wind
ows.go:148 +0x452

goroutine 13 [chan receive]:
gopkg.in/fsnotify%2ev0.(*Watcher).purgeEvents(0xc082086f00)
        src/gopkg.in/fsnotify.v0/fsnotify.go:2
1 +0x5c
created by gopkg.in/fsnotify%2ev0.NewWatcher
        src/gopkg.in/fsnotify.v0/fsnotify_wind
ows.go:149 +0x46c

goroutine 14 [select]:
github.com/ActiveState/tail/watch.func┬À001()
        src/github.com/ActiveState/tail/watch/
inotify.go:88 +0x502
created by github.com/ActiveState/tail/watch.(*InotifyFileWatcher).ChangeEvents
        src/github.com/ActiveState/tail/watch/
inotify.go:125 +0x54a

Hi @rubenhazelaar, I have upgraded the version of fsnotify from v0 to v1. Have you tried the newer version of tail, to see if you are still seeing this issue?

I'm not seeing a deadlock error but I can confirm that with a log file that is not frequently updated (maybe once per second) I am seeing tail stop sending lines after a certain period of time when using Poll: false. By setting poll to true, everything works fine.
Note that this is on Windows. On OS X everything appears to work fine with poll false.

Hi @mitchellh, thanks for pointing it out. Can you confirm if you have tried this on the latest master, with fsnotify v1?
It is a known issue that I'm currently looking into it. I was hoping updating fsnotify to v1 would have rectified it as it originally appeared to be from fsnotify.

This seems not to be resolved yet. I have just hit the same issue and currently watching the poll options to see if it is happy.


Projects page lazy loading

Realization of "lazy loading" of project cards.
updated package: cvat-ui
The current approach to navigating project card pages loads data every time the page is opened, causing lots of React renders, API queries and increasing time to interactivity state.
Preview old behavior.
The project card script has been redesigned to "infinite scrolling".  Which allows us to reduce the number of requests to the API by at least half, significantly reduce the number of React re-renderers and reduce the time to the state of interactivity.
New behavior preview.
Checklist

[ ] ~I submit my changes into the develop branch~
[ ] ~I have added a description of my changes into CHANGELOG file~
[ ] ~I have updated the documentation accordingly~
[ ] ~I have added tests to cover my changes~
[ ] ~I have linked related issues (read github docs)~
[ ] ~I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)~

License

[ x ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Hi @chepuhasasha
Thank you for your proposal. After a quick review I have some comments about it:

If we have less than 4 projects:

If I try to apply a filter o sorting, UI fails:

And the reason here not only because something is broken on client, these settings are applied server-side during extracting data from the database, so, they don't work at all now

After creating a new project we can not see it without updating the whole application

The same issue, if somebody else created a project from another client

Similar issue if some projects have been updated from another client, we will not see this update without reloading a page

General comment is that we actually have had infinite scrolling in the past, but we have received a feedback from our customers: it was inconvenient and didn't work well always, so, we decided to use pagination

I will close the PR because of the problems I mentioned.
Anyway, thank you.


Definitions for entities in realities/qualities/classifications

As already discussed in #1510 we have many quantity values without an entity in reality (but not every quantity value needs an entity in reality).
About Quantity values:
“There are always four separate things:

(1) the entity in reality (some sort of specifically dependent continuant, perhaps, or a process)

(2) a number

(3) a unit. The entity then has_value some

(4) quantity value entity, that has_unit the unit.
(And may have a specified value with a data property). The relationship between the quantity value entity (4) and the entity in the world (1) is is_about.”
Issue #434, comment by @jannahastings
It could be helpful for us to recognise concepts of types of quantities in order to classify quantity values, define qualities such as density, size or length and thus also determine entities in reality (rather in the form of concepts than individuals)
https://docs.google.com/spreadsheets/d/1iqttYN3ng9hSmbQNvOpwPi7dg7__dNIjx2GNFUsWRkc/edit#gid=0 (here previous relationships between quantity values and entities in reality are collected).
Ideas of solution
suggested definitions:
Density: "The amount of something per unit size." (http://purl.obolibrary.org/obo/NCIT_C45781)
(suggested Axiom: areal energy density is_about density. )
Length: "A 1-D extent quality which is equal to the distance between two points" (http://purl.obolibrary.org/obo/NCIT_C25334.) —>  underlying concept of hub height, rotor diameter, .. (each have axiom: has_quantity value length value)
(suggested Axiom: length value is_about length )
Size: "A morphology quality inhering in a bearer by virtue of the bearer's physical magnitude" (imported in many ontologies)
(Axiom: area value is_about size)
another definition of size:
"The magnitude or dimensions of a thing. More specifically, geometrical size (or spatial size) can refer to linear dimensions (length, width, height, diameter, perimeter), area, or volume. Size can also be measured in terms of mass, especially when assuming a density range." —> less specific, but perhaps too unspecific because it is difficult to distinguish from density and length.

We have spent the last few weeks looking in detail at the VIM (International vocabulary of metrology) and how the oeo relates to it.
Property in the VIM would therefore include both qualities and process attributes, because in the VIM there are quantities (measurable properties) and nominal properties (not measurable).

An implementation of the VIM in the oeo would be possible according to the schema, but would lead to a splitting that would involve a lot of effort.
Therefore, the proposal would be to introduce the Quantity class under Quality. The quantity values then refer to these. According to the VIM, it should ultimately also be possible for Quantity Values to refer to Process Attributes.
However, the first step would be to introduce Quantities.
We also need to tidy up our existing quantity values because labels and definitions are often contradictory. (Entities are often Quantities according to the label, but Quantity Values according to the definition)
I started sorting this out in the following table: https://docs.google.com/spreadsheets/d/1iqttYN3ng9hSmbQNvOpwPi7dg7__dNIjx2GNFUsWRkc/edit#gid=297800768

You are onto something important that I also recently noticed. Please read: https://github.com/OpenEnergyPlatform/ontology/issues/205 The whole idea of process attributes seems to me to be a little bit out of place. And I don't completely understand why was process profilenot used.
It think it is important to indicate that the definition of property in VIM is more in line with what we usually refer as ObjectProperties, so relations. I think the definition of quantityin VIM is pretty much the same as quantity value

Quantity: "property (x, owl:thing) of a phenomenon (y, bfo:occurrent), body, or substance (y, bfo: material entity), where the property (x) has a magnitude that can be expressed as a number (z) and a reference (k)"

I understand it as:
(forall (x)
    (if (Quantity x)
        (and (exists (y,z,k)
        	 (and (or (MaterialEntity y) (Occurrent y))
                  (and (and (Number z) (Reference k))
                       (and (has_quantity y x)
                            (and (has_magnitude x z)
                                 (has_reference x k)
                          )
                     )
                )
             )
        )
    )

Quantity value: "A quantity value (x) is an information content entity(x) defined by a numeral (z) together with a unit of measurement (k) to quantify an entity(y)."@en

Ours looks as something like, so the only difference is the domain where VIM uses Occurents or material entities, we let the property has_quantity the domain Entity which is pretty much every thing:
(forall (x)
    (if (QuantityValue x)
        (and (exists (y,z,k)
        	 (and (Entity(y))
                  (and (and (Numeral z) (Unit k))
                       (and (has_quantity_value y x)
                            (and (has_number x z)
                                 (has_unit x k)
                          )
                     )
                )
             )
        )
    )

But I don't think they are incompatible with each other so no need to implement Quantity, maybe addd an alternative label? or am I missing something @fabianneuhaus? :-).


Integration Test - Release - 2019/02/13 19:11:10

Auto create for integration test. IssueCommandFlow

@adunndevster : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Add support for IAM roles in shared credentials file

Is your feature request related to a problem? Please describe.
Add support for IAM role when loading credentials from file (see role_arn and source_profile under Supported config file settings.

Using an IAM role in the AWS CLI
AWS Secure Token Service - AssumeRole
Boto3 - Assume Role Provider

Describe the solution you'd like
Ideally, Soto would automatically handle retrieval and caching of temporary credentials for IAM roles, similar to how Boto3 does.
Describe alternatives you've considered
Looking at this response in a similar feature request, it seems that stsAssumeRole credentials provider could be used. However, I cannot find anywhere in the code where role_arn and source_profile are loaded from ~/.aws/credentials file, nor any references where ~/.aws/config is loaded.
Additional context
The following are examples of shared configuration files where an IAM role profile is configured. This profile foo uses credentials from default profile (as referenced by source_profile to retrieve temporary credentials that will be used to access AWS resources.
Example ~/.aws/config file:
cat ~/.aws/config
[profile default]
region = us-west-2

[profile foo]
region = us-west-2
role_session_name =<EMAIL_ADDRESS>
Example ~/.aws/credentials file:
[default]
aws_account_id           = fooaccount
aws_access_key_id        = FOOKEYID
aws_secret_access_key    = FOOSECRETKEY
aws_session_token        = IQoJb3JEXAMPLE=
aws_security_token       = IQoJb3JEXAMPLE=
x_principal_arn          = arn:aws:sts::...
x_security_token_expires = 2020-11-14T09:11:17-08:00

[foo]
role_arn              = arn:aws:iam::...
source_profile        = default

Additional example configuration files are available in AWS documentation and Boto3 documentation.

There is a slight issues with implementing this as it generates a circular dependency where SotoCore is dependent on SotoSTS which is dependent on SotoCore.

To solve that, we can consider these options:

Option A: Have SotoCore do this service request once, without using SotoSTS, and cache the response credentials.
Option B: Implement this feature in SotoSTS.

I would prefer Option A, since all the logic for loading configuration files is already present in SotoCore. While it will require manually implementing a client for consuming the STS endpoint, the implementation details can be hidden and not exposed to consumers of the library.
In both cases, we would want to update the logic to load and process shared configuration files (both ~/.aws/credentials and ~/.aws/config, with overridable paths), looking for role_arn, source_profile and other configuration settings needed.
Option B would require the logic in SotoCore for loading credentials from file to be exposed, somehow, so SotoSTS can process the settings accordingly. Choosing this option would also mean SotoSTS would be required for processing shared configuration files, which might not be ideal.

I'm not a great fan of A but is better than B. I can bring across the credential provider for assume role. Hopefully it won't drag across too much code.
One thing we won't know is what region to run the STS assume role call. We'll just have to use useast1 which is what all the IAM code is run from.
There are two other sections to the code.

the extra parsing of the credential config file
running an additional credential provider when we have all the details from the config file.

Given we will have all this we might as well support the "credential_source" tag as well.

@eneko do you think you can do the file parsing?

I'd be happy to give it a try, probably later this week.

#406 thats the STS AssumeRole credential provider

@eneko I have also change the ConfigFileCredentialProvider to have an internal Credential Provider. All you need to do is return the correct credential provider from getSharedCredentialsFromDisk.

Draft PR #408 is in progress


error

I run the command:
'query = psrqpy.QueryATNF()'
and get the error:
ValueError                                Traceback (most recent call last)
File ~/pyenv/base/lib/python3.10/site-packages/psrqpy/utils.py:163, in get_catalogue(path_to_db, cache, update, pandas, version)
162 try:
--> 163     float(val)
164 except ValueError:
ValueError: could not convert string to float: 'HAVE'
During handling of the above exception, another exception occurred:
ValueError                                Traceback (most recent call last)
File ~/pyenv/base/lib/python3.10/site-packages/psrqpy/search.py:351, in QueryATNF.get_catalogue(self, path_to_db, cache, update, overwrite, version)
350 try:
--> 351     dbtable = get_catalogue(path_to_db=path_to_db, cache=cache,
352                             update=update, pandas=True,
353                             version=version)
354 except Exception as e:
File ~/pyenv/base/lib/python3.10/site-packages/psrqpy/utils.py:165, in get_catalogue(path_to_db, cache, update, pandas, version)
164 except ValueError:
--> 165     raise ValueError("Value with error is not convertable to a float")
167 if dataline[2][0] == "-" or "." in dataline[2]:
168     # negative errors or those with decimal points are absolute values
ValueError: Value with error is not convertable to a float
During handling of the above exception, another exception occurred:
RuntimeError                              Traceback (most recent call last)
Cell In[11], line 1
----> 1 query = psrqpy.QueryATNF()
File ~/pyenv/base/lib/python3.10/site-packages/psrqpy/search.py:216, in QueryATNF.init(self, params, condition, psrtype, assoc, bincomp, exactmatch, sort_attr, sort_order, psrs, include_errs, include_refs, adsref, loadfromfile, loadquery, loadfromdb, cache, checkupdate, circular_boundary, coord1, coord2, radius, frompandas, fromtable, version)
214 # download and cache (if requested) the database file
215 try:
--> 216     _ = self.get_catalogue(path_to_db=loadfromdb, cache=cache,
217                            update=checkupdate, version=version)
218 except IOError:
219     raise IOError("Could not get catalogue database file")
File ~/pyenv/base/lib/python3.10/site-packages/psrqpy/search.py:355, in QueryATNF.get_catalogue(self, path_to_db, cache, update, overwrite, version)
351     dbtable = get_catalogue(path_to_db=path_to_db, cache=cache,
352                             update=update, pandas=True,
353                             version=version)
354 except Exception as e:
--> 355     raise RuntimeError("Problem getting catalogue: {}".format(str(e)))
357 if not cache:
358     if not overwrite:
RuntimeError: Problem getting catalogue: Value with error is not convertable to a float
How solve it?

Thanks for reporting this. I think there must be an error in the current pulsar catalogue file. Could you try using an older catalogue version, e.g.,
query = psrqpy.QueryATNF(version="2.2.0")

and let me know if that still works for you?

It seems that the latest version of the psrcat.db file that psrqpy reads the information from contains a few instances of lines with:
WARNING: HAVE 0 ERROR

These lines break the parsing of the file. I've let the ATNF catalogue maintainers know about this issue. They'll either be able to remove the offending lines, or I'll make a new psrqpy release that ignores lines that start with "WARNING".

Hello;
thanks for the rapid response. yes, the problem is from the catalog's latest version because it works with the older version as you mentioned.

I've created a new psrqpy release (v1.2.10), which is now available on PyPI for pip installation. This release will work with the current catalogue version.


Wrong test name in Teleportation.ipynb

To get Teleportation.ipynb to work, change ‘%kata T14_StandardTeleport_Test’ to ‘%kata T14_Teleport_Test’

Good catch! I missed that one when I tested this Notebook.
A better fix would probably be renaming the test to T14_StandardTeleport_Test in Tests.qs - it will fit the usual test naming scheme of "test number - task name - Test suffix" better. Do you want to do this fix? If not, I'll get to it at some point.

This has been fixed in #133.


Convo.say not working with Event API

Slack related:
So I spent 4 hours figuring why my convo.say was not triggered while I was trying to transition from RTM to Events API.
If using the Event API only, you need to call:
controller.startTicking();
Could this be added in the doc?
https://github.com/howdyai/botkit/blob/master/readme-slack.md#events-api

@airlikidh2 care to submit the PR?

Thanks for this, I was experiencing the exact same issue.
Went ahead and submitted a documentation PR (#802) for you.

I spent several hours too trying to make convo.say() work. It was only after I wrote an issue and I was typing its title that I saw this one. Now it works.
I see nowhere in the documentation either of bot.startConversation() or convo.say() an instruction to use controller.startTicking() first. Even the section about startTicking() doesn't say where to put it. I think the docs should be clearer about this.

@lpedromachado thanks for the feedback, i agree that startTicking is poorly documented.


refactor(storage): refactor storage into a single ImageStore

refactor(gc): drop umoci dependency, implemented internal gc

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

PR #1610 has been merged, pls rebase.

When an image is deleted, and a "reference" is dangling, then we should gc that reference (opinionated implementation!)
A <- R1, R2, R3 ... if A is deleted, gc R1, R2, R3 ...
Add this test in blackbox tests also.

added stress test with and without referrers, also tested the nightly builds, should be ok to be merged.


Config: Store role names instead of IDs

Currently, the Discord role IDs have to be hard-coded in the config file. To make the same configuration usable on multiple servers (e.g. test, prod), let's instead hard-code the role names.

I think last year we used the IDs because of their immutability (roles names can change, you can create a new one with the same name, and then different roles will be added to the user).
cc @Kislovskiy who might remember more details from last year about this.
Can I suggest that if the goal is to make it reusable between servers (incl. test/prod) to make it a configurable setting depending on the environment?

Thank you for the context. I have the impression that (at least in this project) role names are far more stable than role IDs.


OPTIONAL messes up the query planning

The query https://qlever.cs.uni-freiburg.de/wikidata/gOjCOE fails, but without the OPTIONAL it works. The error message is BAD QUERY (Could not find a suitable execution tree ....
When moving the OPTIONAL to the end, the full rdfs:label predicate is scanned (instead of @de@rdfs:label because of the language FILTER) and the query fails with an OOM error.

This slightly minimized query shows the same behavior.

https://qlever.cs.uni-freiburg.de/wikidata/284gzJ


pretrain the discriminator

I found that the validation step in discriminator training code is commented. Does that mean we don't do validation after training in each epoch? In addition, in the paper, you mentioned that the best performance achieved when pretrain the discriminator to get the accuracy of 0.82. Does this accuracy mean the training accuracy? Thanks!

Thanks! So the dev set is a (randomly selected ) subset of the training data, NOT an unseen reserved set?

Maybe I did not explain it clearly. The training data I mean, is the whole set we can get for each training (it contains the positive and negative data). The whole set will be divided into two parts, one is the evaluation set(200 positive samples and 200 negative samples) which is randomly selected from the whole set, and the rest is used for training. Hence, the dev set is an unseen reserved set.

OK. It's clear now. Thanks!

I re-implement you discriminator in PyTorch, I construct the following Conv2d layers:
self.conv2d = nn.ModuleList([             nn.Sequential(                 Conv2d(in_channels=1,                        out_channels=num_filters[i],                        kernel_size=(kernel_sizes[i], args.decoder_embed_dim),                        stride=1,                        padding=0),                 nn.BatchNorm2d(num_filters[i]),                 nn.ReLU(),                 nn.MaxPool2d(kernel_size=(self.max_len - kernel_sizes[i] + 1, 1))             )             for i in range(len(self.kernel_sizes))         ])
I think this is similar to your discriminator CNN, use various numbers of kernels with different windows sizes to extract different features. Then concat them on out_channel dimension, and then go through a Highway Network, a FC layer to yield the scores.
However, my discriminator always predicts by chance (around 0.5 accuracies even after many epochs). It seems you only train the discriminator 2 epochs in your code...
Is there any "tricks" I should notice?
My pre-trained generator(Luong's architecture) achieved about 28 BLEU on IWLST14 (DE-EN).
Do you force your generator to make translation within 50 words? Currently, I force my generator to finish translation within 50 words. Do you think I should I remove this constrains?
Thanks!!!

Yes. I compared my implementation with Facebook's fairseq-py system (which yield 28.14). And I also removed the sentences that exceed 50 words. And in each batch, I sampled both positive and negative samples (each batch contains these two class at the same time). It seems your implementation also did that. I don't know why I cannot pre-train the discriminator... If you can release your dataset I think it may be helpful.
BTW, I would like to confirm that the "image" we send to conv2d has the size (seqlen--height, embedSize--width, 1--in_channel), right?

Yes, you are right about the input image.  I did not think your problem is closely related with the training data and I am so sorry that I can not release our training data. My suggestion is that you should recheck your input to the discriminator, like, the sentences padded to the same length? The tag for positive and negative sample is right? ....

OK. Thanks for your suggestions, I will check that. In addition, the embedding layer in the discriminator is trained from scratch? Not loaded from pre-trained generator?

@wangyirui The embedding layer in the discriminator is trained from scratch.

Thanks! I think I might figure out the problem... When train the discriminator, the positive sample means the human translation of sentence A, and the negative sample means the machine translation of sentence A, right?

Yes, you are right!

@wangyirui - was the issue simply reversing the positive/negative data?  I have the same issue of the discriminator predicting at chance, even after many epochs.  Wondering how you were able to make training succeed, as I wouldn't expect simply switching positive/negative data to do so.  (I've tried the same, with no success)

@kellymarchisio It turns out that my translation has some problems, which cause the discriminator to predict almost 1 for all sentence. Carefully check your input may be helpful.

@wangyirui
Would you like to tell me what are the exact problems with your translation? I have the same problem as you, I can not pretrain the discriminator which always get about 50% accuracy on development set.


Fix flaky e2e tests

Signed-off-by: Asif Sohail Mohammed<EMAIL_ADDRESS>Description
Fixes flakiness of e2e tests(not completely).
Issues Resolved
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

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

Codecov Report

Merging #1382 (54626e3) into main (d0ccedd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1382   +/-   ##
=========================================
  Coverage     94.08%   94.08%
  Complexity     1156     1156
=========================================
  Files           162      162
  Lines          3248     3248
  Branches        263      263
=========================================
  Hits           3056     3056
  Misses          138      138
  Partials         54       54

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 d0ccedd...54626e3. Read the comment docs.


New test added fails

@alanshaw I completely missed to see the status on CI :( (shame on me). Seems that the test you added detected a problem. Does it run for you?
https://circleci.com/gh/ipfs/js-ipfs-unixfs-engine/567
 1) IPFS UnixFS Engine builder allows multihash hash algorithm to be specified:
     Uncaught TypeError: Argument must be a Buffer or Uint8Array
      at Buffer.equals (buffer.js:635:11)
      at ipldResolver.get (test/test-builder.js:48:28)
      at _get (node_modules/ipld-resolver/src/index.js:123:9)
      at node_modules/async/internal/once.js:12:16
      at next (node_modules/async/waterfall.js:21:29)
      at node_modules/async/internal/onlyOnce.js:12:16
      at r.util.deserialize (node_modules/ipld-resolver/src/index.js:361:13)
      at multihashing (node_modules/ipld-dag-pb/src/dag-node/create.js:61:7)
      at Multihashing.Multihashing.digest (node_modules/multihashing-async/src/index.js:33:5)
      at setImmediate (node_modules/multihashing-async/src/utils.js:8:7)
      at Immediate._onImmediate (node_modules/async/internal/setImmediate.js:27:16)

That's my fault, I'll fix

@diasdavid fixed here #182


Propagating CGE-predicted phenotypes from Resfinder to the summary outputs

Propagates the Resfinder CGE-predicted phenotypes into the summary and detailed summary.

As per request, the changelog has been updated.


Add HotKeys for switching chat tabs

Assign an accesskey property to the a tag, such as a number, and drop in the KeyTips jQuery plugin (written by @DamianEdwards himself), to get a nice UI experience.
Then it's just a case of using the browser's built in hotkey supports

IE: ALT + AccessKey

So, to switch to the first tab, ALT+1, and so on
BTW, I'll be happy to do this. :)

Looks like I'll need to use a button instead of an anchor because IE doesn't interpret accesskeys the same way that other browsers do: http://formattc.wordpress.com/2010/11/03/browsers-and-the-html-accesskey/
In IE, ALT+Key on an anchor is a focus event. If I use a button instead, it's a click on every browser.


Add the missing composer-cli to package.json

Hello "fabric-composer"s,
I'm pretty new to this project. As a matter of fact, saw it yesterday for the first time.
I have just downloaded and tried it locally, and I believe that we may need to add an extra dependency to the package.json. Though I may be wrong.
Please could you kindly take a look?
Thanks, Jonathan

Hello - welcome!  Many thanks for taking the time to take a look.
Our working assumption is that the 'composer-cli' tools would have been installed globaly - so it doesn't need to be added as a dependency.  Though reading back just now on the docs we seem to have lost the important comment about doing this!
If we install this locally - some of the scripts might need to modified. .... but getting a simple getting started would be good.  Let me think on it a bit!

+1 on this – and I personally don't think it should be installed globally because getting-started projects should be a simple as possible (ie, as few deps as possible) – just locally should be fine as all invocation appears to be from npm scripts which will set the PATH correctly

We are going to stick with composer-cli installed globally for now -- it makes it easier for people to manage their diverse networks, using the composer network list commands etc.

@dselman ok, well just make sure to include it in the documentation – at the moment it's not. I had to figure out which composer the error was looking for – initially thinking it was looking for https://getcomposer.org/
(not worth leaving this issue open until it's documented?)

Thanks everybody. Will be happy to start and try from a fresh/clean set-up, so as to confirm that if one is "doing the right thing" (following the step-by-step instruction) - it just works.
Of course I could "hack it" and get it to work somehow (especially as Simon was here in SF last week) - just want(ed) to make sure that this also works for others out of the box.
Sure, either way (cli installed globally or otherwise) - please let me know when we have this documented, and I'll be happy to give it a go. HTH, J


Added Palette support, placeholder color, fadeIn boolean property

Added color palette support
Passing to onLoad function through the event object all the available Swatches with their values rgba, hsl, titleTextColor, bodyTextColor
according to Google's docs
Added two more properties to image loading,

The user is now able to load image with a fade in transition using the fadeIn property
The user is now able to pass placeholderColor #hex as a property

Echo

I don't think this functionality needs to go in this library.
I would be interested in a fade property that made images use the normal fade animation on Android and on iOS the same sort of animation. Thank you!

I don't think this functionality needs to go in this library.
I would be interested in a fade property that made images use the normal fade animation on Android and on iOS the same sort of animation. Thank you!

I agree, but... (to give another point of view regarding this subject)... it should be a way to get a reference to the loaded image if this functionality should be somewhere else (like a third-party library or custom code). I tried to find something related to it, but it seems that today is not possible to get anything about the loaded resource.
That being said, this contribution seems to be useful for some use cases (like defining dynamic backgrounds from image colors, which is what I'm looking for).


Experience points not reduced for skills while using the number field

Describe the bug
I am using DSA 5 Core, Compendium, Magic I.
I am creating a new character and increase the entry for a skill. I am not using the + or - buttons. I am entering directly the new value e.g. "body control" from 0 to 10. In my version the number of free exprience points are not reduced for that!
To Reproduce
Steps to reproduce the behavior:
-create new charater

add human
add one point to COU (to gain the window for starting experience points -> choose 1100 EP)
now you have used 15 EP from 1100
change to skills tab
set value for e.g. climbing to 18 ab choose tab to leave the value field (do NOT use the +- buttons)
switch back to attributes
you have still 15 / 1100 EP used!
when you are trying to increase the value using the +- buttons you will not be able to  increase the value more then 11

Expected behavior
Needed experience points will be reduced as definied in DSA5
Screenshots

Desktop (please complete the following information):

OS: Windows 10
using the administrators console

Your attributes need to be high enough to allow +- to go to higher levels.
Manually editing the input field and automatically updating the ap is neither intended nor will this be implemented.
If you edit the value manually, you have to calculate by yourself.
This is intended to overcome the rules, quickly edit sth or circumvent possible bugs.

see also https://github.com/Plushtoast/dsa5-foundryVTT/wiki/de-FAQ

Hi, thank you so much for your fast answer!


Views outside of $viewPaths are not loaded properly

This can be resolved in src/lib/Sage/Template/FileViewFinder. The most straightforward way would probably be to overwrite the findInPaths() method to check if the path is absolute or relative. Issues might arise if it's intended to be a relative path, but it has a leading slash, as in the case of custom templates. My most recent commits to modify BladeProvider::normalizeViewPath() to account for custom templates probably exacerbates this issue. So that would have to be reworked as well. We might consider checking that if it's an absolute path, that the path exists within the document root or something.
The Symfony Filesystem class has methods we can use to fix it (isAbsolutePath() and possibly makePathRelative()).
The illuminate Filesystem class does not extend the Symfony one, to be clear.
We could merge the two and make it part of the container with something like...
<?php

namespace Roots\Sage;

class Filesystem extends \Illuminate\Filesystem\Filesystem
{
    public function __call($method, $parameters)
    {
        if (method_exists($symfonyFilesystem = new \Symfony\Component\Filesystem\Filesystem(), $method)) {
            return call_user_func_array([$symfonyFilesystem, $method], $parameters);
        }
        return parent::__call($method, $parameters);
    }
}

Or we could just poach the method from the Symfony class.
Our implementation should still keep everything decoupled, possibly write a new contract if necessary.
(If this hasn't been resolved by next weekend, I'll tackle it myself.)

I'm experiencing this same issue with a fresh install. Can't load the index, as the view cannot be
found.
However when using Bedrock, this issue doesn't occur.

Fatal error: Uncaught InvalidArgumentException: 'View [home.vagrant.Code.dev.pops.wp-content.themes.pops.index] not found.' in /home/vagrant/Code/dev/pops/wp-content/themes/pops/vendor/illuminate/view/FileViewFinder.php:137

Is there a fix to get this working without Bedrock currently?

Hi,
I just had the same issue, cannot load index with fresh bedrock then sage install.
The simplest way I found is to create uploads/cache/compiled folder then set right permissions.


Web UI not working

System Operating System (OS)
Windows
Python Version
3.10 (CPython)
Install Source
GitHub
Install version / commit hash
v4.0.0
Expected Behavior vs Actual Behavior
I wanna open the web UI but it doesn't work

Steps to reproduce

Open the web server
See the web page

Traceback
There is no error lol

Other details
No response

Unable to reproduce. Can you try again on the latest updates, and provide further info so we can help?

I updated and it still not work for me.
I tried to open the web UI in Firefox, Edge and Chrome and I've the same error in every browser.
It's a problem of MIME type, the web server send a text/plain type instead of a application/javascript so the web browser block the script for security reasons
Here we can see the wrong MIME type

How to patch the server:
https://github.com/encode/uvicorn/issues/576
I tried on a Win7 VM and the Web UI works, so idk why the web server didn't work on my computer.
And what you want as infos ?
I'm using Windows 11 22H2, Firefox 101.0.1, Edge 102.0.1245.44, Chrome 102.0.5005.115 (Chrome have all default settings, so it's not a problem from the settings of my web browser) and the last version of SpotDL I just downloaded.
My antivirus/firewall is Windows Security with all default settings.
I also tried with the released binary of SpotDL and with Python 3.9 instead of Python 3.10 and it still didn't work.

I'm getting the same error. Windows 11 21H2, Edge 102.0.1245.44, Python 3.9 & spotDL binary v4.0.0rc1.

Me too. I'm on Windows 11 21H2 using latest stable versions of Firefox and Chromium

Is there anything on the python console output where spotdl web was ran?

Is there anything on the python console output where spotdl web was ran?

I'm running the release candidate standalone with no arguments although running it with web has identical results.

I was able to reproduce this by modifying my registry.
Quickfix
What the registry should be:

Computer\HKEY_CLASSES_ROOT\.css > Content Type: text/css
Computer\HKEY_CLASSES_ROOT\.js > Content Type: text/javascript

It looks like your registry might have been "broken" (commonly after installing Visual Studio or another application, or just Win11 altogether?)
Fix
Override library mimetype references.
See https://github.com/spotDL/spotify-downloader/commit/de31601550e5b6b243f7a00b2bc82300f43f2d9d

I can confirm changing my registry to the above mentioned fix solved the issue for me.

Changing Computer\HKEY_CLASSES_ROOT\.js solved the issue for me as well, I did not have to modify my .css as it already had the correct value.
I do have both Visual Studio Community 2022 v17.0.0 & 2019 v16.11.6 installed.

Like when I try to start the web UI it downloads a Song lol. What am I doing wrong?
mathias Downloads % spotdl web
Searching Spotify for song named "web"...
Found YouTube URL for "070 Shake - Web" : https://www.youtube.com/watch?v=6Vp3ucu2GUw

070 Shake - Web                                                         Done               ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00
mathias Downloads %

@mathiasklenk outdated spotdl
py -m pip install -U pip
pip install -U --force spotdl==4.0.0rc2

@Silverarmor that fixed it.


Duration dashboards should be a single value - not a series

Hello,
Two questions regarding Grafana dashboards:

Why does "DAG ran duration" dashboard is showing graphs of 10-15 min series and not a single value? As far as I understand a DAG completed run should be translated to 1 single point in time. I see also that the raw data is series of 10-15 min data and not a single value in time.

In "Failed DAG run duration" dashboard I see weird values: 2.9 days for DAG? This does not align with my airflow real metrics.

Both of the issues above I see at your dashboard image example, and also in my own dashboard, so I guess this is not a bug, but an explenetion to the reasons would be great :)
I tried to transform the "duration" airflow metrics from "summary" to "gauge" type, but without success.
Thanks!

Regarding 1: I understand now that in prometheus, single histogram or summary create a multitude of time series. Is it possible to transform a summary into a gauge? I tried to change "match_metric_type" from observer to gauge but nothing happened.
Regarding 2: I think you have a bug:
in "Failed DAG run duration" and "DAG run duration", dashboards units are in "s" but data is in "ms" ("expr": ..... * 1000")

Im wondering if its possible to show paused dags? Thanks


fix: missing Gyro type and data

Some Gyro pools were added without a poolType and Gyro data. This PR assigns a proper type and upserts missing data on a pool syncing action.

I opened a new PR #616 to work on subgraph mapping extraction.  I'm going to close this one, since all the pools are already synced properly.


end-to-end visual demo

I think this project would benefit from an end-to-end demo that allows someone to pull up, on their local machine, a visual tool such as visdom, try to kick off a training run on a toy problem, and watch that training run in their browser.
Ideally, this example would be copy-paste code (either pure script calls or some Dockerized app call) that could serve as a "Quickstart" example.

@mairuoliu, all yours!

Thank you for #48!


Do not allow infinite recursion in repetition delimiter.

Fixes #357.

I liked your proposed changes a lot.  Merging this now, then cutting a release.


[REQ] Auto unpause the match after its paused

Hello, I think there should have a time limit on each pause. Like .tac has 30 seconds time limit. Other type of pause like .pause or .tech should also have a time limit, when a pause time is run out, it should be automatically unpause. A countdown timer display in the hint message that shows the time left of this pause should be good.
It's better if the time limit can be configurable.
Thanks!

Hey!
Yes I'll add a configurable pause-time limit in future
Thanks for the suggestion!

Did you get a chance to implement pause-timer feature.

Not yet :P
I'll pick it up soon


Added Dexalin bottle description

About the PR
Title
Fixes discord issue

Copy paste got me again Thanks for the PR!

it is not a copy paste problem, there`s literally no dexalin bottle description.

That'd do if too


Migrate Pods to Carthage function missing

New Issue Checklist

[X] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation or CocoaDocs
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
9.0

CocoaLumberjack Version
3.4

Integration Method
Carthage

Xcode Version
Xcode 9.4.1

Issue Description and Steps
I migrate from Pods to Carthage but since I added the CocoaLumberjack.framework and CocoaLumberjackSwift.framework I got an error here :
public func verbose(_ string: String, function: StaticString = #function, file: StaticString = #file, line: UInt = #line) {
    #if DEBUG
        _DDLogMessage(string, level: defaultDebugLevel, flag: .verbose, context: 0, file: file, function: function, line: line, tag: nil, asynchronous: true, ddlog: DDLog.sharedInstance)
    #endif
}

Both _DDLogMessage and defaultDebugLevel are not recognized.
To add the frameworks I added to the Linked Frameworks and Libraries of the project.
Then I added this script in Build Phase :
/usr/local/bin/carthage copy-frameworks

And this to the Input Files :
$(SRCROOT)/Carthage/Build/iOS/CocoaLumberjack.framework
$(SRCROOT)/Carthage/Build/iOS/CocoaLumberjackSwift.framework

Then add frameworks in Copy Files.
What did I do wrong ?
Thanks guys !

@Max0u
Thanks for reporting.
Try to confirm followings.

Set SWIT_VERSION to 4.0 before integrating app witch Carthage.
Use latest CocoaLumberjack (3.4.2).
Add not import CocoaLumberjack but import CocoaLumberjackSwift.

Actually I had no error with CocoaLumberjack 3.4.2 and Xcode 9.4.1.
Thanks.

Closing per latest comment. Feel free to reopen if you still have an issue.


Currently no indicated sidebar scroll position

There is currently no indicated sidebar scroll position (see below).

Fixed. This was in place but regressed.


Changes in URL encoding intended? / How to pass an encoded URL in an URL

Hi there,
I noticed a change in the URL encoding from version 1.4.6 upwards (it's also mentioned in the 1.4.7 changelog: https://github.com/loopj/android-async-http/blob/1.4.7/CHANGELOG.md):
What I want to to is pass an URL as a parameter within an URL.
The URL I want to pass looks like:
 mycompany://app/list?geoid=10809162000&etype=1&esr=2&roomi=2&rooma=2

I encode it with:
 url = URLEncoder.encode(url, "utf-8");

making it look like:
mycompany%3A%2F%2Fapp%2Flist%3Fgeoid%3D10809162000%26etype%3D1%26esr%3D2%26roomi%3D2%26rooma%3D2

and ultimately making a GET request with the encoded URL like:
https://api.mycompany.de/filters/mapped?webUri=mycompany%3A%2F%2Fapp%2Flist%3Fgeoid%3D10809162000%26etype%3D1%26esr%3D2%26roomi%3D2%26rooma%3D2

This works fine with 1.4.6 and breaks when upgrading to 1.4.7 or 1.4.8, obviously because of a change in the way the library handles URL encoding.
Passing the URL unencoded didn't help.
Unfortunately I don't have access to the server to check how the passed URL argument looks when arriving there.
So when using >= 1.4.7, how can I pass my encoded URL?

have you tried disabling library provided URL encoding feature? Use the setURLEncodingEnabled(boolean enabled)

Aaah, that was exactly what I was looking for, thanks very much!


update cccl to v2.3.2

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

The versions in cccl-config-version.cmake in the cuda_cccl v12.4.127 tarball for linux shows a version of 2.3.2

@conda-forge-admin, please rerender

Thanks Dan! 🙏
Is there a test that we could add to make sure we catch this in the future?

The release notes are still claiming 2.3.1.

This shouldn't be changed here. This change has already been made in https://github.com/conda-forge/cccl-feedstock/pull/13. The cuda-cccl feedstock is meant to track the last CCCL released as a part of the CUDA Toolkit. While 2.3.2 is out, it is not yet shipped in a CTK release.

@jrhemstad do you know what version of CCCL we should expect here?
Is cccl-config-version.cmake the best way to validate that?

Something like this?

This seems to be working, and the PR also includes upstreaming of a windows test we put into our recipe

Great teamwork here -- I appreciate the help. I think we should be good to merge once CI passes.

Agreed. Thanks all! 🙏

Indeed, thanks a lot for the refinement!

I opened a similar PR to add a test in cccl-feedstock here: https://github.com/conda-forge/cccl-feedstock/pull/16


heidihakuun-ArnavRavinder

[✅ ] I have read the steps to getting a blot

[ ✅] I am submitting art that...

[ ✅] is algorithmically generated (will change each time the program is run)
[ ✅] is drawable on a blot (fits in the work area & doesn't overlap too much)
[ ✅] is original (not copied from somewhere else)

[✅ ] Optional, if you used a tutorial or based your art on something else, please include the link here: Based it on Hakuun! https://ca.slack-edge.com/T0266FRGM-U078FB76K5F-13cf9fcd6959-512

[ ] Optional, if you remixed this from something else, mention it here: N/A

This is a cool drawing, but its not drawn programmatically, and its also not generative! Maybe make it so it makes random facial expressions or poses?

@arnavravinder this is super cute! Any updates on making it generative?


Bump next-auth from 3.29.0 to 4.0.6

This project is associated with step-by-step article and it can't be reproducible because next-auth v4 is GA now.
The change includes all migration efforts from v3 to v4 based on official migration document.
One thing to be mentioned in the article is to generate a new secret, which is mandated in next-auth v4.

@fkiller Thank you so much for this. I will review and test the changes.

@fkiller Can you please rebase the pull request with the latest master branch? I will merge it once the rebasing is done.


Setting verbose=False issues with threads

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[x] I carefully followed the README.md.
[x] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[x] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Observed Behavior
Setting verbose = False when creating a Llama instance causes all program output to disappear while the model is running. This includes in particular logging output from other threads. In addition, if I create several Llama instances and use them in parallel from multiple threads, I sometimes receive some (but not all) output from one of the instances (I assume that's because another instance finished running first and restored the output for everyone). And sometimes output is not restored when all instances have finished (probably because the last instance to finish restored output to a blocked state).
Expected Behaviour
Setting verbose = False when creating a Llama instance should cause output from that instance to disappear. Other program output should still be visible (this includes logging output and output from other Llama instances).

As a workaround for logging on Linux, you can duplicate the stderr file descriptor and log to that instead of the true stderr. E.g.:
logging.basicConfig (
    level = LOG_LEVEL, format = LOG_FORMAT,
    handlers = [
        logging.StreamHandler (os.fdopen (
            os.dup (sys.stderr.fileno()), 'w'))
    ])

This allows logging to proceed even while output has been blocked by llama.cpp.


Contributing: setup instruction update, and more

Updated dev setup instructions. Also link to Gitpod.
Replace "mirroring" section by current info.

Preview:

https://github.com/chalin/opentelemetry.io/blob/chalin-dev-doc-updates-2021-11-17/CONTRIBUTING.md
https://github.com/chalin/opentelemetry.io/blob/chalin-dev-doc-updates-2021-11-17/README.md

/easycla


Fix 'detected` option init in cygwin installer

Codecov Report

Merging #1156 into master will decrease coverage by 0.24%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1156      +/-   ##
=========================================
- Coverage   93.75%   93.5%   -0.25%
=========================================
  Files          39      39
  Lines        4260    4327      +67
  Branches      466     736     +270
=========================================
+ Hits         3994    4046      +52
+ Misses        266     265       -1
- Partials        0      16      +16

Impacted Files
Coverage Δ

browser/model/cygwin.js
100% <100%> (ø)
:arrow_up:

browser/model/devstudio-autoinstall.js
97.29% <0%> (-2.71%)
:arrow_down:

browser/model/helpers/downloader.js
96.13% <0%> (-1.66%)
:arrow_down:

browser/pages/account/controller.js
84.76% <0%> (-1.33%)
:arrow_down:

browser/model/jbossfusekaraf.js
95.09% <0%> (-0.83%)
:arrow_down:

browser/pages/confirm/controller.js
87.59% <0%> (-0.78%)
:arrow_down:

browser/model/cdk.js
99.28% <0%> (-0.72%)
:arrow_down:

browser/pages/install/controller.js
87.79% <0%> (-0.59%)
:arrow_down:

browser/services/data.js
82.24% <0%> (-0.52%)
:arrow_down:

browser/model/installable-item.js
93.81% <0%> (-0.35%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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


Link Translation Methods 1 and 2

"Option 3: Use the link translation setting" specifically calls out the inability of link translation to translate within Javascript and only within HTML / CSS. Is this also the case with Options 1 and 2, or do they improve this capability?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 15155736-f2c4-f743-1d69-a55e4d317661
Version Independent ID: c1af8414-1f69-4358-2734-e7839dba27ec
Content: Translate links and URLs Azure AD App Proxy
Content Source: articles/active-directory/manage-apps/application-proxy-configure-hard-coded-link-translation.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @msmimart
Microsoft Alias: mimart

Hi @MikeParker365 Thank you for your feedback! We will review and provide an update as appropriate.

@msmimart Could you please have this verified with engineering so that the page can be updated as per customer feedback.

@MikeParker365 Link translation is not supported for Javascript for all options.


Simplify installing with terraform init

Let's explore exposing this provider through the terraform registry so it can be more seemlessly installed for users with terraform init.

As discussed this will likely not happen for now as this provider is not popular enough for Hashicorp to officially maintain it. It does show up under the community providers though: https://www.terraform.io/docs/providers/type/community-index.html


yt-dl no longer downloading from http://www.arte.tv/fr

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.07.02. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.07.02

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl --verbose -F http://www.arte.tv/fr/videos/041012-003-A/la-valse-des-continents
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'-F', u'http://www.arte.tv/fr/videos/041012-003-A/la-valse-des-continents']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.07.02
[debug] Python version 2.7.12 - Linux-4.8.0-56-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 2.8.11-0ubuntu<IP_ADDRESS>, avprobe 2.8.11-0ubuntu<IP_ADDRESS>, ffmpeg 3.3-static, ffprobe 2.8.11-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[arte.tv:+7] la-valse-des-continents: Downloading webpage
[arte.tv:+7] la-valse-des-continents: Downloading JSON metadata
ERROR: An extractor error has occurred. (caused by KeyError(u'VID',)); 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.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/arte.py", line 203, in _real_extract
    return self._extract_from_webpage(webpage, video_id, lang)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/arte.py", line 246, in _extract_from_webpage
    return self._extract_from_json_url(json_url, video_id, lang, title=title)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/arte.py", line 86, in _extract_from_json_url
    title = (player_info.get('VTI') or title or player_info['VID']).strip()
KeyError: u'VID'
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 762, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 446, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
ExtractorError: An extractor error has occurred. (caused by KeyError(u'VID',)); 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.

azed@azed-H270N:~$

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
yt-dl no longer downloading from http://www.arte.tv/fr
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

Post the output with --dump-pages.

azed@azed-H270N:~$ youtube-dl --verbose --dump-pages -F http://www.arte.tv/fr/videos/041012-003-A/la-valse-des-continents
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'--dump-pages', u'-F', u'http://www.arte.tv/fr/videos/041012-003-A/la-valse-des-continents']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.07.02
[debug] Python version 2.7.12 - Linux-4.8.0-56-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 2.8.11-0ubuntu<IP_ADDRESS>, avprobe 2.8.11-0ubuntu<IP_ADDRESS>, ffmpeg 3.3-static, ffprobe 2.8.11-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[arte.tv:+7] la-valse-des-continents: Downloading webpage
[arte.tv:+7] Dumping request to http://www.arte.tv/fr/videos/041012-003-A/la-valse-des-continents
PCFkb2N0eXBlIGh0bWw+CgoKICAgICAgICAgICAgICAgICAgICAKPGh0bWwgY2xhc3M9Im5vLWpzIiBsYW5nPSJmciIKICAgIGRhdGEtYmFzZS11cmw9IiIKICAgIGRhdGEtZW52PSJwcm9kIgogICAgZGF0YS1oZWFkZXItYmFzZS11cmw9Imh0dHBzOi8vc3RhdGljLWNkbi5hcnRlLnR2L3N0YXRpYy9zdHlsZWd1aWRlLzIuNy4xL3NyYy8iCiAgICBk...

As clearly seen from the log it worked fine last time.

Sorry to bother you with this. Yt-dl is indeed working fine. Great piece of kit by the way.


add gil-refs-migration feature

Closes #3958
This PR implements the gil-refs-migration feature as proposed in that issue, and moves the FromPyObjectBound implementations to be gated on that feature.
This way users can migrate the rest of the Bound API deprecations without immediately being broken by the extract changes.
I also added open-by-default details tags to the 0.21 migration guide as per #3961.

After https://github.com/PyO3/pyo3/pull/3959, this would only affect trait qualified syntax (FromPyObject::extract) which extract one of the borrowed types, right?

I think it's a little more than just the syntax, it's about delaying the breakages caused by the lifetime change for the FromPyObject -> FromPyObjectBound for &str and such to be the last thing that needs fixing rather than the first. I think that's easier for users to deal with: fix all the deprecations first and then adjust these snippets as a final step.

One disadvantage I could see is that something like this [...] I guess that's the trade-off if we want to keep compile errors minimal while still enabling deprecation warnings.

Yes, though I was hoping it's an advantage of this feature to allow the rest of the code to migrate first, rather than a disadvantage :)

breakages caused by the lifetime change for the FromPyObject -> FromPyObjectBound for &str and such

But these breakages do only occur once users actively migrating the code, right? Just turning off gil-refs does not have that effect immediately I think. By allowing this intermediate state, users have to touch each place twice, once for the initial migration and then once again to fix the lifetime issues after turning off gil-refs-migration, whereas they could probably do it in one go most of the time. But maybe that's not too bad and if this helps for migrating edge cases I'm fine either way.

Just turning off gil-refs does not have that effect immediately I think.

Right I see, because now that #3959 is merged, not only does it allow &str extraction but because of the nature of the gil ref lifetime the extracted &str from &'py PyAny continues to have &'py str lifetime which avoids the temporaries 👍. I had missed that point previously, sorry 🤦
So the only break which this feature would now help avoid is users working with abi3 before 3.10, where the FromPyObjectBound for &str implementation cannot exist. But you're right it adds two touch points to the code.
So maybe this feature is indeed an overcomplication.
I a few other possible things I was wondering for this feature:

At the moment we have not deprecated any methods like PyAny::get_item, which can also return GIL Refs. I was wondering if we should deprecate some (all?) of those methods, and I was wondering if we should mark those deprecations with not(feature = "gil-refs") or not(feature = "gil-refs-migration").
We made breaking changes to intern! to return &Bound<'py, PyStr>. I was wondering if we should partially revert that and only break on not(feature = "gil-refs-migration").

I would say while those ideas might help users, they are not necessarily obvious or intuitive, so perhaps for the sake of simplicity we should back out from this.

I had missed that point previously, sorry

No worries 😄, I could have also just overlooked something as well.

At the moment we have not deprecated any methods like PyAny::get_item, which can also return GIL Refs.  I was wondering if we should deprecate some (all?) of those methods,

I while back I also though about if we should deprecate the inherent methods. Deprecating only the ones that return gil-refs feels a bit arbitrary to me, because they all operate on gil-refs and fundamentally are all deprecated. But actually marking all of them as deprecated will cause a lot of warning without immediate action, since resolving them requires the migration of the receiver and not the method itself. We have deprecated all the constructors and most of the macros (I think #[derive(FromPyObject)]s from_py_with is the only one that we currently don't have an idea for). If all of the ways to receive a gil-ref are deprecated that's probably enough.
Another option could maybe be to deprecate them under a rustc #cfg(...) flag, such that after fully migrating users can double check that there are really no gil-ref APIs lurking around somewhere. That way we don't have another feature, that could potentially left turned on by users (accidentally) and then break once we remove those features, and also don't flood everyones console with even more warnings by default.

We made breaking changes to intern! to return &Bound<'py, PyStr>. I was wondering if we should partially revert that and only break on not(feature = "gil-refs-migration")

Is there a case that we reasonably could expect that to break anything? It looks like every method that I think makes sense to use it in conjunction with (...attr, ...item, call_method..., ...) is already generic in a way to be compatible either way. And for the (rare) case where it does break something, we could document in the migration guide to use .as_gil_ref() temporarily.

Yes, I agree with all of the above. I think the case for this feature is not strong enough.
A #[cfg] just to check the gil-refs is an interesting idea. We don't need to block 0.21 final on that either as it'd be just an additive command line flag.


As a head nurse, I can select language preferences and interpreter contact information

... so that effective communication with patients who have limited English proficiency can be achieved

We will not proceed with this feature.


Wrong Demo link

Wrong Live Demo Link in README file
Hi @CarlosIgreda, It seems that your Live Demo Link in your README file redirects to another project, not to your portfolio. 🔗

@cuauh-cabrera Thanks for pointing that out buddy 🤓. Now it is fixed.


datadog-agent: 7.50.3 -> 7.56.2

Updates datadog-related packages for Go 1.22 compatibility, and fixes a few issues. This should be backported to 24.11, where datadog-agent is currently marked as broken.

Updated packages to the minimum version that supports Go 1.22
Stripped out windows-only integration that showed up as broken in datadog UI (agentcrashdetect)
Stripped out random test integration whose config is somehow included in the default configs
Compiled packages with docker support by default. This will save a somewhat heavy build step for users who need docker support, while only adding 2.5% to the bin folder size (4MB).
Update deprecated options for the process agent
Set dd_agent_bin argument for the process agent to communicate with the main agent.

Superset of #356308
Resolves #351119
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[x] 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/)
25.05 Release Notes (or backporting 24.11 and 25.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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 358927

x86_64-linux

:fast_forward: 1 package blacklisted:

nixos-install-tools

:white_check_mark: 2 packages built:

datadog-agent
datadog-process-agent

@ofborg eval

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 358927

x86_64-linux

:fast_forward: 2 packages blacklisted:

nixos-install-tools
tests.nixos-functions.nixos-test

:white_check_mark: 2 packages built:

datadog-agent
datadog-process-agent

aarch64-darwin

:white_check_mark: 2 packages built:

datadog-agent
datadog-process-agent

@ofborg build datadog-agent datadog-process-agent


Remove BoundPod type from source code

This PR is completely removing BoundPod type.
Since this is built on top of #5433, please do NOT merge until that PR is merged too.

cc @vmarmol @erictune @dchen1107

Looks good @wojtek-t, just needs a rebase.

I've run e2e tests (I need to patch out #5205 which is known to cause problems) and all tests passed:
Ran 26 of 28 Specs in 1299.213 seconds
SUCCESS! -- 26 Passed | 0 Failed | 2 Pending | 0 Skipped I0317 10:24:02.136998   31094 driver.go:89] All tests pass

This is a great cleanup.  Thanks @wojtek-t
On Tue, Mar 17, 2015 at 2:28 AM, Wojciech Tyczynski <
<EMAIL_ADDRESS>wrote:

I've run e2e tests (I need to patch out #5205
https://github.com/GoogleCloudPlatform/kubernetes/pull/5205 which is
known to cause problems) and all tests passed:
Ran 26 of 28 Specs in 1299.213 seconds
SUCCESS! -- 26 Passed | 0 Failed | 2 Pending | 0 Skipped I0317
10:24:02.136998 31094 driver.go:89] All tests pass
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/5502#issuecomment-82237735
.

LGTM, thanks @wojtek-t!


How to compress the params file?

Is there solution to compress the params file?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


More thorough review of literature

I'm sure I'm missing other work.

http://link.springer.com.sci-hub.cc/article/10.1023/A:1020192616580


Create dataset iarpa_babel_swahili_language_pack

uid: iarpa_babel_swahili_language_pack
type: processed
description:

name: IARPA Babel Swahili Language Pack
description: Swahili ASR Dataset. Official description says "IARPA Babel Swahili Language Pack IARPA-babel202b-v1.0d was developed by Appen for the IARPA (Intelligence Advanced Research Projects Activity) Babel program. It contains approximately 350 hours of Swahili conversational and scripted telephone speech collected from 2012-2014 along with corresponding transcripts."
homepage: https://doi.org/10.35111/afrp-a637
validated: True

languages:

language_names:

Niger-Congo
Swahili

language_comments:
language_locations:

Eastern Africa
Kenya

validated: False

custodian:

name:
in_catalogue: linguistic_data_consortium_ldc
type:
location:
contact_name:
contact_email:
contact_submitter: False
additional:
validated: False

availability:

procurement:

for_download: Yes - after signing a user agreement
download_url: https://doi.org/10.35111/afrp-a637
download_email:

licensing:

has_licenses: Yes
license_text: Multiple licenses:

For-profit use: https://catalog.ldc.upenn.edu/license/iarpa-babel-swahili-agreement-for-profit.pdf
Non-member: https://catalog.ldc.upenn.edu/license/iarpa-babel-swahili-agreement-non-member.pdf
Not-for-profit: https://catalog.ldc.upenn.edu/license/iarpa-babel-swahili-agreement-not-for-profit.pdf

license_properties:

multiple licenses
research use
non-commercial use
copyright - all rights reserved

license_list:

other: Other license

pii:

has_pii: Unclear
generic_pii_likely:
generic_pii_list:
numeric_pii_likely:
numeric_pii_list:
sensitive_pii_likely:
sensitive_pii_list:
no_pii_justification_class: general knowledge not written by or referring to private persons
no_pii_justification_text:

validated: False

processed_from_primary:

from_primary: Taken from primary source
primary_availability: No - the dataset curators kept the source data secret
primary_license:
primary_types:
validated: False

media:

category:

audiovisual

text_format:
audiovisual_format:

.WAV

image_format:
database_format:
text_is_transcribed:
instance_type: 350 hours of telephone conversations, no clue how long each file is. Estimating 15 minutes each, and 350 words per minute?
instance_count: 10K<n<100K
instance_size: 100<n<10,000
validated: False

fname: iarpa_babel_swahili_language_pack.json

Needs paying a fee (or becoming member of LDC): https://www.ldc.upenn.edu/language-resources/data/obtaining


Migrate to nextjs app router

Updates the project starter to use the latest version of Nextjs with app router and no src directory. Removes the js project option. Closes
https://github.com/o1-labs/zkapp-cli/issues/583
https://github.com/o1-labs/zkapp-cli/issues/400

The point of using app router is to enable server-side rendering right? Do users need to be aware of which routes and components use SSR and avoid o1js in them?

One of the reasons Next moved to app router was to better support React's server components. Info explaining usage with server components may be well suited to a Next.js specific section in the docs where we also call out things like loading the SDK in useEffect. The user will not be able to call useEffect in a server component but could attempt to load the SDK as a top level import and run into issues if we don't guide them on best practices there.

The point of using app router is to enable server-side rendering right? Do users need to be aware of which routes and components use SSR and avoid o1js in them?

Vercel is also pushing the app router as the new standard for NextJS. A user can chose only to use client side components with the app router if they want.


[zh-cn]Update content/zh-cn/docs/concepts/workloads/_index.md

Update Pods to Pod
content/zh-cn/docs/concepts/workloads/_index.md

The preview is good. Thanks.
/lgtm


Missing cursor in initial fetch response

Hi, we used in TikTokLive 4.5.2 package and  when it starts to  capture event  data we are facing error Missing cursor in initial fetch response .

I have the same problem.

I have the same problem + 1

I have the same problem + 1

Fixed.


Reminder to contact a friend

My idea is simple: allow reminders about specific friends. For example, if I'm talking to John Doe and he has a big event on December 11, I can enter: "friends remind John Doe December 11 2020 big meeting with boss about his promotion". This could create a one-time cron job that pops up, then delete it afterwards? Not sure how it would work.
I normally enter things like this on Google Calendar, but using friends would make it much easier.
I will be donating in the coming months. This is a wonderful tool. Thanks creating and developing it.

Thanks for the suggestion, @ethanmorse! This is a really cool idea.
I'm a bit wary to have friends creating and deleting cron jobs because that can get into the complexities of how different users and OSes manage their crontab (though I'm not an expert here—please let me know if there's a standard interface for this sort of thing that makes it less brittle).
How would you feel about some sort of friends today command (very open to other names!) that gives you things like:

upcoming friend birthdays (#176)
upcoming friend reminders (i.e. your suggestion)
the output of friends suggest

And then the README could instruct folks to have cron run friends today every day (as I do with friends suggest). Does that seem like a good approach to you?

I'm not sure of a standard interface.
friends today and the cron approach looks perfect. Excellent solution.

Great! I'm happy to work on that (likely this weekend) but don't want to step on your toes if you were hoping to take a shot at it (and I'd be happy to provide guidance). Either way works for me; just let me know!

I think it'd be much better if you were to work on it. I have zero experience with Ruby and am not in a position with career nor life to work on it properly. I've donated $50 in the meantime.

Thanks so much for the donation, @ethanmorse. It really means a lot. Some family things came up this weekend that slowed this down a bit but I am working on it. I'll likely tackle #176 first as I think that will help clarify how the user experience might work.
As I think about this I have a few questions on my mind that I'd love your input on:

If I do friends remind and add a reminder, should the reminder:

auto-delete itself when it's overdue? Or only delete itself when the user says so—if so, what's the API for that?
be able to be rescheduled? If so, what's the API for that?
be able to automatically recur? If so, what's the API for that?

Are birthdays really just a special case of reminders, or should they be separate (i.e. friends set birthday)?

Great questions. Excuse my lack of programming knowledge—I'm really only familiar with basic Python and HTML.
1i. Auto-delete should be fine if it can be rescheduled.
1ii. Yes, it should be able to be rescheduled. Not sure the best way to do this. Could each friends remind be given a tag/id that can be referenced for rescheduling? For example, friends remind Call Jacob about new friends features on October 1 is the first reminder, so it would be given id r1 (or similar). If I wanted to reschedule our call, friends remind reschedule r1 October 3. Reminders can be referenced through a simple friends list reminders.
1iii. Yes, I like the recurrences. friends remind recur r1 7. The number after the reminder id is numbers of days until it repeats again. Alternatively, putting a single date like friends remind recur r1 October 2 will set that reminder for that date every year thereafter.

I think birthdays can be a special case. See 1iii for example. I suppose if it's not too difficult separating them may be better and simpler (for the user, not necessarily for you :) ).

Thanks for the input, @ethanmorse! I want to stew on this a bit more—I'm a little wary of building something overly complex in the first version, but I'll see what I can come up with!


pull request for e2e test at 12/28/2016 02:45:06

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Loveneesh] Add support to remove non connected edges in P2P filter

Add support to remove all leaf node in the graph to see only connected cases. This only works in P2P filter and is hidden in other filters.
As City and State is always a connected graph I think it would not require this this filter.
This is a reference to #24 and #75

just fixing travel filter, will then add this feature. Thanks!

sure, Thanks!

Hey @lovewithmind, really sorry on the delay in merging this, but can you resolve the merge conflicts and I'll merge it ASAP? This looks like the only viable solution to improving cluster performance at the moment, and I'd love to merge it very soon.
Moreover, it looks like it only works on the P2P filter? Can you make it work for the others too? For now though, just fix the merge conflicts and I'll merge this

Hi @someshkar , I have resolved the merge conflict on the branch. You can merge it to master now.


add convert lowercase to uppercase button

Subject of the issue
Describe your issue here.
Your environment

version of angular-translate
version of angular
which browser and its version

Steps to reproduce
Tell us how to reproduce this issue. Please provide a working demo, you can use this template as a base.
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead

plz assign me this issue

Thank you @Mridul-2003 . But people with dyslexia may find it hard to read capital letters. You can instead change this to the opposite one i.e. the caps to lowercase. You can refer to this article: https://www.bdadyslexia.org.uk/advice/employers/creating-a-dyslexia-friendly-workplace/dyslexia-friendly-style-guide#:~:text=Avoid using all capital letters,letters are easier to read.

ok i add button for lowercase.Thanks for assign this to me


Fix ForwardConfig in elbv2 response templates

Update syntax in CREATE_RULE_TEMPLATE and SET_RULE_PRIORITIES_TEMPLATE so ForwardConfig is included in create_rule() and set_rule_priorities() response objects.

Codecov Report

Merging #5046 (16471f5) into master (61a5d5c) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5046      +/-   ##
==========================================
- Coverage   95.80%   95.79%   -0.02%
==========================================
  Files         663      663
  Lines       66468    66468
==========================================
- Hits        63680    63672       -8
- Misses       2788     2796       +8

Flag
Coverage Δ

servertests
36.57% <ø> (ø)

unittests
95.73% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/elbv2/responses.py
99.43% <ø> (ø)

moto/resourcegroupstaggingapi/models.py
83.18% <0.00%> (-0.59%)
:arrow_down:

moto/route53/responses.py
96.69% <0.00%> (-0.42%)
:arrow_down:

moto/sns/models.py
96.83% <0.00%> (-0.40%)
:arrow_down:

moto/sqs/models.py
96.76% <0.00%> (-0.18%)
:arrow_down:

moto/config/models.py
99.22% <0.00%> (-0.13%)
:arrow_down:

moto/ssm/models.py
94.42% <0.00%> (-0.11%)
:arrow_down:

Continue to review full report at Codecov.

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


Order of operation when creating constraint

// Context
let view1 = UIView()
let view2 = UIView()
let window = UIWindow()
window.addSubview(view1)
window.addSubview(view2)
view1.widthAnchor == 50
view2.widthAnchor == (view1.widthAnchor + 150) / 2

// Expectation: (50 + 150) / 2
view2.frame.width == 100

// Actual: (50 / 2) + 150
// <NSLayoutConstraint: UIView.height == 0.5 * UIView.height + 150   (active)>
view2.frame.width == 175

Pattern matching could be used to prevent orders that aren't supported or the math could be inversed within the Anchorage operator to achieve the desired effect using auto layout's native evaluation order.

This stems from the fact that the operators are an interface into building a LayoutExpression, which contains a constant and a multiplier, and these are always assumed to be of the form:
y = multiplier * x + constant

In the sample that @BrianSemiglia posted, the original expression looked like this:
y = (x + constant) * multiplier

The user of this code would probably expect that the multiplier would distribute across the expression, resulting in this:
y = x * multiplier + constant * multiplier

However, order of operations is not expected. No matter the order, the constant and multiplier are both just set on the expression directly.
What should we do about it?
We could fix this by introducing more intermediate types to capture the order of operations, but that would be a breaking change: anyone who currently has some code that uses the current behavior is technically using it wrong, but if their code is working for them, correcting the multiplicative distribution would change their layout.
Instead, I think it might be a good idea to build those intermediate types to capture the order of operations, but then use availability/deprecation annotations to show the user a warning that they are using Anchorage incorrectly (i.e. "holding it wrong"). In my opinion, an Anchorage expression should look like the underlying expression it represents, which always takes the form y = mx + b.

I agree with @ZevEisenberg that the expression format should be enforced. I don't think there's a strong use case for the distributive property when building layout constraints, except in contrived examples.
Perhaps adding a deprecation/warning to the * and / overloads that take a LayoutExpression on the LHS could help? However, that would preclude valid expressions like
view2.widthAnchor == 50 + view1.widthAnchor / 2

We could be strict about the ordering as well: anchor + constant * multiplier. Not sure if people are using other styles much. I'd be happy to run a patched branch against our internal projects and see if it turns up any issues.
Good call on the overrides. We may not need any new intermediate types; just stricter annotations on existing ones.

To clarify: I think it sense to accept any expressions that are unambiguously, algebraically equivalent to y = mx + b, and reject any that would be different if you apply distribution.