Flaky Test: DockerSuite.TestKillWithStopSignalWithDifferentSignalShouldKeepRestartPolicy

Seen failing on master, and z
 ----------------------------------------------------------------------
 FAIL: docker_cli_kill_test.go:77: DockerSuite.TestKillWithStopSignalWithDifferentSignalShouldKeepRestartPolicy

 docker_cli_kill_test.go:89:
     // Restart policy should still be in place, so it should be still running
     c.Assert(waitRun(cid), check.IsNil)
 ... value *errors.errorString = &errors.errorString{s:"condition \"\"false\" == \"true\"\" not true in time"} ("condition \"\"false\" == \"true\"\" not true in time")

 ----------------------------------------------------------------------

ping @michael-holzheu

@tophj-ibm Thanks for hint. ping @ddingel

It might be worth adding a zipped version of the test-integration-cli in bundles as an artifact for the jenkins build like is done on the other builds so then other people can look at them.
ping @mikedougherty, I don't want to change the jenkins scripts to upload things without your permission 😄

The testcase DockerSuite.TestKillWithStopSignalWithDifferentSignalShouldKeepRestartPolicy confuses me:
func (s *DockerSuite) TestKillWithStopSignalWithDifferentSignalShouldKeepRestartPolicy(c *check.C) {
	// Cannot port to Windows - does not support signals int the same way as Linux does
	testRequires(c, DaemonIsLinux)
	out, _ := dockerCmd(c, "run", "-d", "--stop-signal=CONT", "busybox", "top")
	cid := strings.TrimSpace(out)
	c.Assert(waitRun(cid), check.IsNil)

	// Let's docker send a TERM signal to the container
	// It will kill the process, but not disable the restart policy
	dockerCmd(c, "kill", "-s", "TERM", cid)

	// Restart policy should still be in place, so it should be still running
	c.Assert(waitRun(cid), check.IsNil)
}

As the comment correctly says the init process "busybox top" is killed by SIGTERM. Isn't the container automatically killed when its init process dies?
And I don't understand the comment "Restart policy should still be in place". As we don't explicitly set a restart policy and the default policy is "no", the container should not be restarted.
So to me the testcase looks wrong.
ping @vdemeester Perhaps I am missing something? Could you explain since you commited d2e64247e6dfd3257a7d015890aa2f168ff643a0?

After short discussion with @ddingel: Perhaps just a "restart=always" is missing for the testcase?

Seems logically, but if that's the case I'm not sure how the test would be passing currently on everything else.

@tophj-ibm if I understand waitRun correctly it will return immediately on success.
So from my understanding there could be a small race window between delivering the signal with docker kill actually terminating the container. If in that window we check waitRun everything will look good, right?
@vdemeester maybe waitRun isn't the best option, so before checking the status of the container the test should wait for some time?

@ddingel: good catch, I threw in a sleep statement between them and sure enough it failed without the restart policy active.
I can think of a couple of ways to fix this, so I'll open up a PR for this unless one of you wanted to.

@tophj-ibm so there is:

a fix so that this test case will fail reproducible
some fixes for other test cases which might have the same underlying problem, maybe it is worth to introduce a function like waitRunAtLeast?
a fix for this test case for restart policy

So depending on what you want to fix or take care of I would pick up the remaining. Also I have no problem if you decide to take care of all :).

It might be worth adding a zipped version of the test-integration-cli in bundles as an artifact for the jenkins build like is done on the other builds so then other people can look at them.

on the Docker-PRs job (aka the "janky" status context), bundles.tar.gz is currently populated with files returned from this find command:
find bundles -name '*.log' -o -name '*.prof' -o -name integration.test

is there more that should archived? i think the entire directory was pretty large and we didn't want to give up the storage for stuff that would be rarely used (binaries probably).

ping @mikedougherty, I don't want to change the jenkins scripts to upload things without your permission 😄

i don't care, they're not my scripts ;)

@mikedougherty oh this would be just adding that command and uploading the results to the master z job.

@ddingel yes I can get all of them.

@tophj-ibm oh i see. sorry, i didn't quite gather the entire context. that sounds reasonable and straightforward. IMO it would be appropriate if someone else took care of it (you, perhaps? not sure who's best) -- i'd rather empower people and not be a bottleneck. that being said, i can take the task if necessary.

@mikedougherty yeah I can update it, I just wanted to double check it was fine with you.

:scream_cat: you are right, I think a --restart=always is missing :scream_cat: How did I miss that :sweat_smile: This test is just plain wrong, my bad :scream:
@tophj-ibm @ddingel feel free to open a PR to fix it, thanks a lot 👼


Remove AMZN registry in hub-regionA.json firewall policy

Presently around line 762 the target FQDNs are specified:
"targetFqdns": [
  "*.docker.com",
  "*.docker.io",
  "docker.io",
  "ghcr.io",
  "github-production-container-registry.s3.amazonaws.com"

Can we remove the Amazon registry?

@joaquinrz - can you do this in the same PR as adding ghcr.io?


Implement Clone and Debug for HashTable's Iter struct

Found those implementations missing, which is preventing me from implementing the same in encapsulating iterators.

:umbrella: The latest upstream changes (presumably #549) made this pull request unmergeable. Please resolve the merge conflicts.

@bors r+

:pushpin: Commit e9fd7b44ef807b5357c3e8406cbdd033860f2ab9 has been approved by Amanieu
It is now in the queue for this repository.

:hourglass: Testing commit e9fd7b44ef807b5357c3e8406cbdd033860f2ab9 with merge cd9a9559f4fcdb31a051c5b21ecde429c1e354db...

:sunny: Test successful - checks-actions
Approved by: Amanieu
Pushing cd9a9559f4fcdb31a051c5b21ecde429c1e354db to master...


Wrong folder referenced

In my PR, I used the folder /assets/ instead of /img/ you may wish to fix that when using the icon for new window

TY! All clear ✅


Learning graph embeddings for the syntactic graph

Input file of tripes in the format verb subject object score, e.g.:

take#VB event   place   112493.0
see#VB  x       frame   56300.0
have#VB anyone  idea    56202.0
offer#VB        Expedia airfare 56180.0
provide#VB      invention       method  53138.0
have#VB anyone  experience      48864.0
receive#VB      family  friend  47644.0
do#VB   people  thing   47483.0
have#VB student opportunity     45680.0
take#VB meeting place   44599.0
use#VB  Gemvara material        42037.0
show#VB FIG.    view    41693.0
illustrate#VB   FIG.    embodiment      37872.0
have#VB people  problem 37540.0
take#VB change  place   37195.0

pruned: http://panchenko.me/data/joint/verbs/cc16malt/vso-1.3m-pruned-strict.csv.gz
full: http://panchenko.me/data/joint/verbs/cc16malt/vso-25m-pruned.csv.gz

Construct the graph of triples via triple co-occurrences. If two triples have the two common elements (e.g. subject and verb OR verb and object), then they are connected.
Alternatively, If two triples have only one common element (e.g. common subject) then the weight is one.

Use deepWalk, node2vec, and path2vec to learn vector representations of these triples. Use various meta-parameters for each approach, e.g. the number of dimensions and walks.

Evaluate the obtained triple representations on the task of frame induction by running the evaluation code available in this repository using the methodology described in this paper: https://arxiv.org/abs/1805.04715 (published at ACL'2018, https://acl2018.org/).

I've checked embeddings obtained with DeepWalk by loading to gensim and checking most similar nodes, it looks like:
similar to:  have#VB people time
have#VB people depression 0.7461773157119751
have#VB people address 0.7298743724822998
have#VB people monopoly 0.7279400825500488
have#VB people brain 0.7251932621002197
have#VB people score 0.7251231074333191
similar to:  provide#VB provider information
provide#VB image information 0.9872413873672485
provide#VB change information 0.987193763256073
provide#VB column information 0.9869943261146545
provide#VB publication information 0.9869118928909302
provide#VB booklet information 0.9863284230232239

Looks like we can continue with running more experiments on this graph

looks indeed reasonable - please provide a few more examples here to get a
better understanding (just a few more triples and a bit more nearest
neighbours per triple).
good to go in my mind - just proceed as planned.
On Sun, Jul 15, 2018 at 3:16 PM, Oleksiy<EMAIL_ADDRESS>wrote:

Looks like we can continue with running more experiments on this graph
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-405088072, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vgzvY7NrpXv2mQx-z4sET7fjAhiEks5uGzKEgaJpZM4UirHv
.

similar to:  highlight#VB report problem
highlight#VB report fact 0.9538995623588562
highlight#VB report importance 0.9202446341514587
highlight#VB report need 0.9141565561294556
highlight#VB study fact 0.8908078074455261
underscore#VB report need 0.8627906441688538
highlight#VB study need 0.8606923818588257
emphasize#VB report need 0.8565777540206909
highlight#VB study importance 0.854476273059845
stress#VB report need 0.8541354537010193
emphasize#VB report importance 0.8476148843765259
similar to:  answer#VB story question
answer#VB manager question 0.9966793060302734
answer#VB time question 0.9966354966163635
answer#VB candidate question 0.9964892864227295
answer#VB site question 0.9964324831962585
answer#VB scientist question 0.9964265823364258
answer#VB team question 0.9963186979293823
answer#VB Minister question 0.9963048696517944
answer#VB Paul question 0.9962888956069946
answer#VB research question 0.9962843656539917
answer#VB Jesus question 0.9962024092674255

similar to:  wear#VB woman burqa
wear#VB woman bikini 0.9982713460922241
wear#VB woman veil 0.9982062578201294
wear#VB woman pants 0.9981944561004639
wear#VB woman scarf 0.9980823397636414
wear#VB woman clothing 0.9980330467224121
wear#VB woman blouse 0.9979761838912964
wear#VB woman bra 0.9979567527770996
wear#VB woman outfit 0.9979463219642639
wear#VB woman hijab 0.9979447722434998
wear#VB woman coat 0.9979429244995117
similar to:  feature#VB knife blade
feature#VB Knife blade 0.9742035269737244
have#VB knife blade 0.9537492990493774
have#VB knife edge 0.7106331586837769
have#VB knife handle 0.6131154894828796
have#VB blade edge 0.554728627204895
have#VB voice edge 0.5477390885353088
have#VB Democrats edge 0.47338515520095825
have#VB plate edge 0.4549257159233093
have#VB Republicans edge 0.45163947343826294
have#VB Romney edge 0.4388987720012665

similar to:  address#VB comment issue
address#VB member issue 0.995779812335968
address#VB candidate issue 0.9954516291618347
address#VB discussion issue 0.9953857660293579
address#VB workshop issue 0.9953724145889282
address#VB legislation issue 0.9953103065490723
address#VB conference issue 0.9952685832977295
address#VB one issue 0.9952408671379089
address#VB Commission issue 0.9950791597366333
address#VB Circuit issue 0.9950239062309265
address#VB leader issue 0.9950180053710938
similar to:  catch#VB thing fire
catch#VB idea fire 0.9952883124351501
catch#VB car fire 0.9951303005218506
catch#VB plane fire 0.9951239228248596
catch#VB tree fire 0.9947503805160522
catch#VB house fire 0.994671642780304
catch#VB home fire 0.9944249987602234
catch#VB clothing fire 0.9942343831062317
catch#VB ship fire 0.994118332862854
catch#VB team fire 0.9940105676651001
catch#VB vehicle fire 0.9939444065093994

thanks - looks quite reasonable. please generate now various versions of
the embeddings and then we can perform their evaluation in the context of
the frame induction task as done in the referenced paper below (which
@dmitryustalov is currently presenting in Melbourne).

Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-405117504, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vpr8xl5OqA8Z47DinPN7f6IFsPHxks5uG6uSgaJpZM4UirHv
.

Please send links to the pre-computed word vectors (on the servers).

on ltcpu1: /home/6oliinyk/triframes_results

ok thanks

On 30 Jul 2018, at 10:54, Oleksiy<EMAIL_ADDRESS>wrote:
on ltcpu1: /home/6oliinyk/triframes_results
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/uhh-lt/triframes/issues/1#issuecomment-408794054, or mute the thread https://github.com/notifications/unsubscribe-auth/ABY6vgfIJ-bERHzKTQKyh31Kl6VTdCrHks5uLsmzgaJpZM4UirHv.

@dustalov Here is the updated version of the data which you can use.
The list of all embeddings for 100k triples: https://docs.google.com/spreadsheets/d/1CCwLjWetFuMwl3qgO8yQXVpbbxlRWDlIbqoNrVjB6B8/edit?usp=sharing
The data are stored here: http://ltdata1.informatik.uni-hamburg.de/frames/

I have access only to ltnode1d and farnsworth. Please release the rest of the data including the id to triple mappings for each graph.

I have access only to ltnode1d and ltcpu3. Please release the rest of the data including the id to triple mappings for each graph.

http://ltdata1.informatik.uni-hamburg.de/frames/

It may be also you do not need the pkl files - the embeddings should be self contained.

No. There are some integer identifiers instead of triples in the vector files. There is no reliable approach to map these identifiers to the actual triples.

@alexteua please generate embeddings which contain actual triples, not integer ids please.

@alexanderpanchenko @dustalov
In every directory with generated embeddings there are 2 subdirectories now: num_ids and triple_ids. First one contains embeddings which you saw, second one with triples instead of numerical ids.  I also committed script which can do such a replacement.
In triframes_results directory you can also find id_to_name_full.pkl  and id_to_name_pruned.pkl which are dictionaries for graphs based on full and pruned datasets accordingly.

Thanks.

@dustalov I've updated the files. Please check! http://ltdata1.informatik.uni-hamburg.de/frames/

I will upload more embeddings soon

Thanks. Please try implementing the graph generation procedure using a sparse adjacency matrix which will make it possible to store graphs from more than 100k triples.

On 20 Aug 2018, at 18:54, Oleksiy<EMAIL_ADDRESS>wrote:
I will upload more embeddings soon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/uhh-lt/triframes/issues/1#issuecomment-414387270, or mute the thread https://github.com/notifications/unsubscribe-auth/ABY6vjnNKteSbQKSLhItL8fW6YvAczEHks5uSunWgaJpZM4UirHv.

I've uploaded more embeddings done for:

deepwalk, full dataset, first 100K
deepwalk, pruned dataset, first 1M

Google doc is updated with configurations

node2vec implementation is going to out of memory error while generating embeddings for latest mentioned categories.
It accepts only edgelist as input format, which is more redundant than adjlist.

Please let me know if we need embeddings with some specific configurations to generate

Thanks! Out of memory is on which machine? Ltcpu1?
On Tue, Aug 28, 2018, 10:42 AM Oleksiy<EMAIL_ADDRESS>wrote:

node2vec implementation is going to out of memory error while generating
embeddings for latest mentioned categories.
It accepts only edgelist as input format, which is more redundant than
adjlist.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-416500946, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vowia4iJU4wnsOLFM_nViGyhGU1jks5uVQJ5gaJpZM4UirHv
.

Yes, Ltcpu1. I just see "Killed" output from running script

Could you provide please some nearest  neighbors for the two mentioned new
models?
On Tue, Aug 28, 2018, 10:38 AM Oleksiy<EMAIL_ADDRESS>wrote:

I've uploaded more embeddings done for:

deepwalk, full dataset, first 100K
deepwalk, pruned dataset, first 1M

Google doc is updated with configurations
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-416499736, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vi5bnUD1X86wSWEgDS9YnD2An2kIks5uVQGIgaJpZM4UirHv
.

Please run again and check the memory usage using htop.  The machine has
256 gb of ram.
On Tue, Aug 28, 2018, 10:46 AM Oleksiy<EMAIL_ADDRESS>wrote:

Yes, Ltcpu1. I just see "Killed" output from running script
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-416502079, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vje7RYosi2T8gcb4vC79ezi_npyfks5uVQNigaJpZM4UirHv
.

for one of the embeddings within "deepwalk, full dataset, first 100K" category
similar to:  eat#VB PERSON cookie
eat#VB PERSON cookie 0.961711049079895
eat#VB PERSON cereal 0.9581553339958191
eat#VB PERSON portion 0.9558544158935547
eat#VB PERSON plate 0.9556226134300232
eat#VB PERSON healthy 0.9554531574249268
eat#VB PERSON dessert 0.9551925659179688
eat#VB PERSON fruit 0.9550948739051819
eat#VB PERSON pizza 0.9548265933990479
eat#VB PERSON fruit 0.9545609951019287
eat#VB PERSON pizza 0.9542282223701477
similar to:  read#VB PERSON autobiography
read#VB PERSON verse 0.9585568904876709
read#VB PERSON comic 0.9577409625053406
read#VB PERSON manga 0.9567180275917053
read#VB PERSON poem 0.9564070105552673
read#VB PERSON excerpt 0.9562754034996033
read#VB PERSON novel 0.9559537768363953
read#VB PERSON newspaper 0.9558944702148438
read#VB PERSON trilogy 0.9556835293769836
read#VB PERSON translation 0.9554949998855591
read#VB PERSON ebook 0.955351710319519
similar to:  have#VB THING image
have#VB THING frame 0.9377962946891785
have#VB THING acid 0.936640739440918
have#VB THING field 0.9364444017410278
have#VB THING beach 0.9363809823989868
have#VB THING reference 0.9361435770988464
have#VB THING restaurant 0.9358288049697876
have#VB THING discussion 0.9358265399932861
have#VB THING inhabitant 0.9356974363327026
have#VB THING pad 0.9350844621658325
have#VB THING lock 0.9350626468658447

Could you please show 100 nearest neighbors as well for these triples?
On Tue, Aug 28, 2018, 11:05 AM Oleksiy<EMAIL_ADDRESS>wrote:

for one of the embeddings within "deepwalk, full dataset, first 100K"
category
similar to: eat#VB PERSON cookie
eat#VB PERSON cookie 0.961711049079895
eat#VB PERSON cereal 0.9581553339958191
eat#VB PERSON portion 0.9558544158935547
eat#VB PERSON plate 0.9556226134300232
eat#VB PERSON healthy 0.9554531574249268
eat#VB PERSON dessert 0.9551925659179688
eat#VB PERSON fruit 0.9550948739051819
eat#VB PERSON pizza 0.9548265933990479
eat#VB PERSON fruit 0.9545609951019287
eat#VB PERSON pizza 0.9542282223701477
similar to: read#VB PERSON autobiography
read#VB PERSON verse 0.9585568904876709
read#VB PERSON comic 0.9577409625053406
read#VB PERSON manga 0.9567180275917053
read#VB PERSON poem 0.9564070105552673
read#VB PERSON excerpt 0.9562754034996033
read#VB PERSON novel 0.9559537768363953
read#VB PERSON newspaper 0.9558944702148438
read#VB PERSON trilogy 0.9556835293769836
read#VB PERSON translation 0.9554949998855591
read#VB PERSON ebook 0.955351710319519
similar to: have#VB THING image
have#VB THING frame 0.9377962946891785
have#VB THING acid 0.936640739440918
have#VB THING field 0.9364444017410278
have#VB THING beach 0.9363809823989868
have#VB THING reference 0.9361435770988464
have#VB THING restaurant 0.9358288049697876
have#VB THING discussion 0.9358265399932861
have#VB THING inhabitant 0.9356974363327026
have#VB THING pad 0.9350844621658325
have#VB THING lock 0.9350626468658447
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-416508109, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vpV9o5KGmPNKvLJB49ULwyJVLeQIks5uVQfHgaJpZM4UirHv
.

eat#VB PERSON pizza 0.9548265933990479
eat#VB PERSON fruit 0.9545609951019287
eat#VB PERSON pizza 0.9542282223701477
Why there are duplicates?
On Tue, Aug 28, 2018, 11:08 AM Alexander Panchenko <
<EMAIL_ADDRESS>wrote:

Could you please show 100 nearest neighbors as well for these triples?
On Tue, Aug 28, 2018, 11:05 AM Oleksiy<EMAIL_ADDRESS>wrote:

for one of the embeddings within "deepwalk, full dataset, first 100K"
category
similar to: eat#VB PERSON cookie
eat#VB PERSON cookie 0.961711049079895
eat#VB PERSON cereal 0.9581553339958191
eat#VB PERSON portion 0.9558544158935547
eat#VB PERSON plate 0.9556226134300232
eat#VB PERSON healthy 0.9554531574249268
eat#VB PERSON dessert 0.9551925659179688
eat#VB PERSON fruit 0.9550948739051819
eat#VB PERSON pizza 0.9548265933990479
eat#VB PERSON fruit 0.9545609951019287
eat#VB PERSON pizza 0.9542282223701477
similar to: read#VB PERSON autobiography
read#VB PERSON verse 0.9585568904876709
read#VB PERSON comic 0.9577409625053406
read#VB PERSON manga 0.9567180275917053
read#VB PERSON poem 0.9564070105552673
read#VB PERSON excerpt 0.9562754034996033
read#VB PERSON novel 0.9559537768363953
read#VB PERSON newspaper 0.9558944702148438
read#VB PERSON trilogy 0.9556835293769836
read#VB PERSON translation 0.9554949998855591
read#VB PERSON ebook 0.955351710319519
similar to: have#VB THING image
have#VB THING frame 0.9377962946891785
have#VB THING acid 0.936640739440918
have#VB THING field 0.9364444017410278
have#VB THING beach 0.9363809823989868
have#VB THING reference 0.9361435770988464
have#VB THING restaurant 0.9358288049697876
have#VB THING discussion 0.9358265399932861
have#VB THING inhabitant 0.9356974363327026
have#VB THING pad 0.9350844621658325
have#VB THING lock 0.9350626468658447
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-416508109,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABY6vpV9o5KGmPNKvLJB49ULwyJVLeQIks5uVQfHgaJpZM4UirHv
.

ok, I will commit some examples for 100 nearest neighbors to the repo, to keep the discussion cleaner.
regarding duplicates currently I have no idea, will check the dataset if the issue is there.

Ok. Thanks
On Tue, Aug 28, 2018, 11:16 AM Oleksiy<EMAIL_ADDRESS>wrote:

ok, I will commit some examples for 100 nearest neighbors to the repo, to
keep the discussion cleaner.
regarding duplicates currently I have no idea, will check the dataset if
the issue is there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-416511634, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vmL8XJI8CwP2ETbvsqiBEyNal72Tks5uVQpngaJpZM4UirHv
.

You can upload to Google drive or Dropbox samples of nearest neighbors and
post the link here...
On Tue, Aug 28, 2018, 5:36 PM Alexander Panchenko <
<EMAIL_ADDRESS>wrote:

Ok. Thanks
On Tue, Aug 28, 2018, 11:16 AM Oleksiy<EMAIL_ADDRESS>wrote:

ok, I will commit some examples for 100 nearest neighbors to the repo, to
keep the discussion cleaner.
regarding duplicates currently I have no idea, will check the dataset if
the issue is there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-416511634,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABY6vmL8XJI8CwP2ETbvsqiBEyNal72Tks5uVQpngaJpZM4UirHv
.

here are some samples of nearest neighbors:
https://drive.google.com/drive/folders/1mqgxIyHswPFOKH9FzlTdfaUjO3z6vnP1?usp=sharing

Thanks, can you also generate the same for 'eat#VB PERSON pizza' and 'read#VB PERSON autobiography'?
What about the duplicates? Do you have any ideas why this happens?

I checked the dataset and duplicates are there.
eat#VB  PERSON  pizza  3131.0
eat#VB  PERSON  pizza  1593.0
eat#VB  PERSON  pizza  1117.0

Ok, I have to check that. I think this the result of some normalisation
(like lowercasing). Please then sum up the frequencies before the training
and release a file in the same format without duplicates.
On Fri, Sep 7, 2018, 7:27 PM Oleksiy<EMAIL_ADDRESS>wrote:

I checked the dataset and duplicates are there.
eat#VB PERSON pizza 3131.0
eat#VB PERSON pizza 1593.0
eat#VB PERSON pizza 1117.0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-419510154, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vggXClbq8YqxeSqakWlpB8SCPKBRks5uYqxvgaJpZM4UirHv
.

do you mean to retrain embeddings? There are various hyperparameter combinations, which one to pick up? I guess to retrain all of them will take a lot of time

We do not know yet which ones are the best. If this is just machine time
that's not an issue - just launch maybe a bash script with all the
combinations and let it run.
On Sat, Sep 8, 2018, 8:57 AM Oleksiy<EMAIL_ADDRESS>wrote:

do you mean to retrain embeddings? There are various hyperparameter
combinations, which one to pick up? I guess to retrain all of them will
take a lot of time
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/triframes/issues/1#issuecomment-419618652, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABY6vmoWqZfo04kY9XqifgcuhqCzYL9mks5uY2pcgaJpZM4UirHv
.

Just some information on duplicates in the input data:
pruned: http://panchenko.me/data/joint/verbs/cc16malt/vso-1.3m-pruned-strict.csv.gz
here is just about 1% of duplicates
full: http://panchenko.me/data/joint/verbs/cc16malt/vso-25m-pruned.csv.gz
here is about 40% of duplicates

interesting, thanks

On 17 Sep 2018, at 22:43, Oleksiy<EMAIL_ADDRESS>wrote:
Just some information on duplicates in the input data:
pruned: http://panchenko.me/data/joint/verbs/cc16malt/vso-1.3m-pruned-strict.csv.gz http://panchenko.me/data/joint/verbs/cc16malt/vso-1.3m-pruned-strict.csv.gz
here is just about 1% of duplicates
full: http://panchenko.me/data/joint/verbs/cc16malt/vso-25m-pruned.csv.gz http://panchenko.me/data/joint/verbs/cc16malt/vso-25m-pruned.csv.gz
here is about 40% of duplicates
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/uhh-lt/triframes/issues/1#issuecomment-422164186, or mute the thread https://github.com/notifications/unsubscribe-auth/ABY6vqXnqr6jdxthzbNT7y0giXGfc86Gks5ucAl_gaJpZM4UirHv.


audiobookshelf: 2.8.0 -> 2.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for audiobookshelf is: Self-hosted audiobook and podcast server
meta.homepage for audiobookshelf is: https://www.audiobookshelf.org/
meta.changelog for audiobookshelf is: https://github.com/advplyr/audiobookshelf/releases/tag/v2.8.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.8.1 with grep in /nix/store/d92pvj385wadh9xlxz86kkaixvynpf1d-audiobookshelf-2.8.1
found 2.8.1 in filename of file in /nix/store/d92pvj385wadh9xlxz86kkaixvynpf1d-audiobookshelf-2.8.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
audiobookshelf

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d92pvj385wadh9xlxz86kkaixvynpf1d-audiobookshelf-2.8.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A audiobookshelf https://github.com/r-ryantm/nixpkgs/archive/1aa289bcbff9a23c68cb9e83f6135600330a6c75.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1aa289bcbff9a23c68cb9e83f6135600330a6c75#audiobookshelf

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d92pvj385wadh9xlxz86kkaixvynpf1d-audiobookshelf-2.8.1
ls -la /nix/store/d92pvj385wadh9xlxz86kkaixvynpf1d-audiobookshelf-2.8.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

audiobookshelf

Maintainer pings
cc @jvanbruegge @adamcstephens for testing.

@NixOS/nixpkgs-merge-bot merge


[WIP] [iscsi-targetd] block volume support

Currently, we are trying to add block volume functionality to k8s storage toward v1.9.
This feature adds volumeMode parameter into PV Spec and PVC Spec to handle both filsystem type volume and block type volume. In order to support Block volume at external-provisioner, we need

Update API directory/files
Add support of volumeMode parameter to iscsi/targetd external-provisioner

This PR is still WIP because the API and plugin change hasn't merged yet.
Please confirm if this API change is proper.
These are related Proposal and API update PR.

Block volume proposal
https://github.com/kubernetes/community/pull/1265
API Change
https://github.com/kubernetes/kubernetes/pull/50457
Block Volume for iSCSI plugin
https://github.com/kubernetes/kubernetes/pull/54752

@wongma7 @jsafrane @raffaelespazzoli @rootfs

@mtanino just a little stylistic change: can you collapse the following lines:
	var volumeMode v1.PersistentVolumeMode
 	volumeMode = v1.PersistentVolumeFilesystem

like this:
       volumeMode := v1.PersistentVolumeFilesystem

everything else looks fine to me.

@raffaelespazzoli
yup. done

#425

/cc @raffaelespazzoli @rootfs
PR is updated. PTAL.

LGTM, thanks!

@raffaelespazzoli @rootfs
Now I'm ready to get review again. Thanks.

/lgtm
cc @childsb for merge

@mkimuram

@childsb Could you merge this PR? thanks.


Test Charm when it's updated

Currently, the newest release of charm is for 1.19.2.
Test it when 1.20.1 is released.
Curse Forge Page
Github Issue

Charm is dead and has not been touch since Sep of last year. It's been 11 or so months.


Calibration Factor Changes in TRME_RR

In TRME the variable cfac is defined as follows:
cfac = 1. / (2 * (1. - (1. + self.r0) * np.exp(-self.r0)))
In TRME_RR it is defined similarly but commented out Instead it is a simpler form:
 %cfac = 1./(2*(1.-(1.+ITER.r0)*exp(-ITER.r0) ));
if ITER.iterMax > 0
     notConverged = 1;
     cfac = 1./(1.-exp(-ITER.r0));
     %cfac = 1./(2*(1.-(1.+ITER.r0)*exp(-ITER.r0) ));

Attached is a plot of the two expressions for cfac (aka 1/beta in Egbert & Booker 1986 (EB86)).  In EB86 beta is quoted as 0.7784 for r0=1.5.  Thus we can see that the simpler expression from TRME_RR is the better approximation.

The cfac is .7784 for r0=1.5.  This is hard coded in the Fortran code.
cfac is the beta in Equation A3 in egbert and booker 1986.
Note that A3 has a typo and the opening square bracket should be on the other side of the 1/sqrt(2pi)
The psi' FWIW comes from (A2)

This can be made into a function or a look up table with interpolation.  The priority is currently low, but this issue is not closed.
TODOs:

Compute beta for a few values of r0.  You can use r0=1.5-->beta=0.7784 as a test to sanity check the calculation
Either make a program that computes beta(r0) from analytic expression (using a Taylor expansion?) or one that interpolates.  In either case, cfac is still not properly integrated into the program.

This issue was intended to resolve the problem of two definitions of "cfac", which we now know is 1/beta.
We have concluded that between the two expressions one is better:
cfac = 1./(1.-exp(-ITER.r0));

is the better choice for 1/beta.
A numeric solution following exactly Equation A3 in EB86 could be added, and I created issue #111 for this, but the issue with two calibration factors is closed.


subject("vutuv verification email") has to use gettext

lib/vutuv/emailer.ex

Closed in c41b43698c18689d551df63e71bdb84160688805


KUBERNETES_COLLECT_EVENTS send event but do not flag them with configured TAGS

**Output of the info page **
2017-08-18 09:42:51,589 | WARNING | dd.collector | utils.service_discovery.config(config.py:31) | No configuration backend provided for service discovery. Only auto config templates will be used.
====================
Collector (v 5.16.0)
====================

  Status date: 2017-08-18 09:42:46 (5s ago)
  Pid: 25
  Platform: Linux-4.4.41-k8s-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/collector.log

  Clocks
  ======

    NTP offset: 0.0018 s
    System UTC time: 2017-08-18 09:42:51.662215

  Paths
  =====

    conf.d: /etc/dd-agent/conf.d
    checks.d: /opt/datadog-agent/agent/checks.d

  Hostnames
  =========

    ec2-hostname: ip-172-20-85-135.eu-west-1.compute.internal
    local-ipv4: <IP_ADDRESS>
    local-hostname: ip-172-20-85-135.eu-west-1.compute.internal
    socket-hostname: kube-state-metrics-2169312636-jbxkl
    public-hostname: ec2-34-253-216-94.eu-west-1.compute.amazonaws.com
    hostname: i-02d0c299bf692272f
    instance-id: i-02d0c299bf692272f
    public-ipv4: <IP_ADDRESS>
    socket-fqdn: kube-state-metrics-2169312636-jbxkl

  Checks
  ======

    kube_dns (5.16.0)
    -----------------
      - instance #0 [OK]
      - Collected 81 metrics, 0 events & 0 service checks

    kubernetes_state (5.16.0)
    -------------------------
      - instance #0 [OK]
      - Collected 19091 metrics, 0 events & 1294 service checks

    kubernetes (5.16.0)
    -------------------
      - instance #0 [OK]
      - Collected 375 metrics, 0 events & 3 service checks

    ntp (5.16.0)
    ------------
      - Collected 0 metrics, 0 events & 0 service checks

    disk (5.16.0)
    -------------
      - instance #0 [OK]
      - Collected 32 metrics, 0 events & 0 service checks

    docker_daemon (5.16.0)
    ----------------------
      - instance #0 [OK]
      - Collected 368 metrics, 1 event & 1 service check

  Emitters
  ========

    - http_emitter [OK]

====================
Dogstatsd (v 5.16.0)
====================

  Status date: 2017-08-18 09:42:46 (5s ago)
  Pid: 19
  Platform: Linux-4.4.41-k8s-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/dogstatsd.log

  Flush count: 69
  Packet Count: 1173
  Packets per second: 1.7
  Metric count: 9
  Event count: 0
  Service check count: 0

====================
Forwarder (v 5.16.0)
====================

  Status date: 2017-08-18 09:42:50 (1s ago)
  Pid: 18
  Platform: Linux-4.4.41-k8s-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/forwarder.log

  Queue Size: 377933 bytes
  Queue Length: 2
  Flush Count: 216
  Transactions received: 137
  Transactions flushed: 135
  Transactions rejected: 0
  API Key Status: API Key is valid

======================
Trace Agent (v 5.16.0)
======================

  Pid: 17
  Uptime: 698 seconds
  Mem alloc: 2488032 bytes

  Hostname: kube-state-metrics-2169312636-jbxkl
  Receiver: <IP_ADDRESS>:8126
  API Endpoint: https://trace.agent.datadoghq.com

  Bytes received (1 min): 0
  Traces received (1 min): 0
  Spans received (1 min): 0

  Bytes sent (1 min): 0
  Traces sent (1 min): 0
  Stats sent (1 min): 0

Additional environment details (Operating System, Cloud provider, etc):
running in a kubernetes cluster on AWS.
Steps to reproduce the issue:

set environnement variable: KUBERNETES=true and KUBERNETES_COLLECT_EVENTS=true
set environnement variable: TAGS=env:staging
Run your dd-agent container on 1 of your k8s node

Describe the results you received:
Kuberenetes events are sent to datadog but they do not have the tag env:staging
Describe the results you expected:
Kuberenetes events should be flagged with the tag env:staging
Additional information you deem important (e.g. issue happens only occasionally):

I think it's related to https://github.com/DataDog/docker-dd-agent/issues/65 as well.
Tags seem to be essential to use kubernetes integration with multiple clusters.

Hello @julienbachmann,
Unfortunately I cannot reproduce this bug. Please note that host tags are resolved in our backend and will not appear in the output of /opt/datadog-agent/agent/agent.py check kubernetes.
The most common cause of missing host tags is running several agents per host. This is currently not supported as two agents will send conflicting host metadata payloads and the backend-side resolution will have issues. Did you make sure you are not running another "event-less" agent on this node?
FWI, 5.17.0 (to be out in a couple of days) introduces a new mechanism to allow for easier event collection, you might want to give it a try: https://github.com/DataDog/integrations-core/pull/687

It also doesn't work for me. I have a datadog agent with such configuration:
k8s-daemonset.yaml:
apiVersion: extensions/v1beta1
kind: DaemonSet
metadata:
  name: datadog-agent
spec:
  template:
    metadata:
      labels:
        app: datadog-agent
      name: datadog-agent
    spec:
      hostNetwork: true
      hostIPC: true
      dnsPolicy: ClusterFirstWithHostNet
      containers:
      - image: datadog/docker-dd-agent:latest-dogstatsd-alpine
        imagePullPolicy: Always
        name: datadog-agent
        ports:
        - containerPort: 8125
          name: dogstatsdport
          protocol: UDP
        env:
        - name: API_KEY
          value: secret
        - name: DD_LOGS_STDOUT
          value: 'yes'
        - name: DD_TAGS
          value: "DC:tier1-europe-west1"
        - name: DD_URL
          value: "http://my-host-to-sniff-dd-requests:8081"

datadog.conf
dd_url = http://my-host-to-sniff-dd-requests:8081
api_key = secret
gce_updated_hostname = yes
collector_log_file = /opt/datadog-agent/logs/collector.log
forwarder_log_file = /opt/datadog-agent/logs/forwarder.log
dogstatsd_log_file = /opt/datadog-agent/logs/dogstatsd.log
jmxfetch_log_file = /opt/datadog-agent/logs/jmxfetch.log
tags = DC:tier1-europe-west1
non_local_traffic = yes
log_to_syslog = no

And I have a host which sniff all datadog agent http requests
{
   "series":[
      {
         "tags":null,
         "metric":"datadog.dogstatsd.packet.count",
         "interval":10.0,
         "device_name":null,
         "host":"gke-preemptible2-1be1ade4-ms1r.c.project.internal",
         "points":[
            [
<PHONE_NUMBER>.0,
               1809
            ]
         ],
         "type":"gauge"
      },

As you can see metric datadog.dogstatsd.packet.count sent with "tags":null but I guess it should be "tags": "DC:tier1-europe-west1"

Hi @username1366
As @xvello said, tags in datadog.conf are resolved in our backend, they are sent separately as host tags. They are not applied to every metrics client side. So this behavior is expected, and the tag should be set in the app if gke-preemptible2-1be1ade4-ms1r.c.project.internal is also the host name or a host alias for this node and you have a full agent running on it.

Can we have the entrypoint push down the global tags to the kubernetes config for now?


Storage patching fails for fields on proxy models

It turns out that proxy models re-use field instances, which means that during the first call to setup_storage the wrapping gets messed up.
Suppose you have a field image that appears on a model that has a proxy of it. During setup_storage, the following might happen:

The field on the proxy model gets overridden. Default storage is stored in previous_storages for the field, and then LocMemStorage is set as the field's storage.
The field on the main model gets overridden. It's the same instance. LocMemStorage is stored in previous_storages for the field, and then LocMemStorage is set as the field's storage.

I think the solution here might be to ignore proxy models when overriding file fields, though it might also work to make filefields a set rather than a list.

No worries - send all the bugs you find through anytime you find them!
This should be fixed by 8f21cc37c72a591fa376c85c56a13e8ca15b4cc5. Let me know if it is still an issue for you. The new version is up on PyPI too.
Thanks again!

Yep, that works now! Thanks!


support auto release charts and host eg charts on GitHub Repo

Description:

support auto release eg charts
host Helm Charts via GitHub Pages and Releases

subtask of #1072.
Blocker:
chart-releaser-action needs to use github pages via branch not github action. Our site for now is using github action. We need to refactor our versioned docs to transfer into gh-pages branch, instead of github actions.
[optional Relevant Links:]
Use: https://github.com/helm/chart-releaser-action
Ref: https://github.com/helm/chart-releaser

hey @Xunzhuo can you highlight the pros and cons of hosting in GH vs something like Docker Hub which supports OCI Artifacts now

+1 for using OCI if possible.

I think using DockerHub is a good option too. I am very sure of the differences between them.
Let us hear some ideas from @AliceProxy.

+1 for DockerHub, after reading over the Docker doc above. 🙂

Adding to the +1 on DockerHub for charts.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/1031 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit cae1352:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-source' (branch: Release_Archive_main_2022-02-11-01-16-42) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add support for geometryFunction prop to RDraw

Expose geometryFunction prop to enable using createBox() function or any custom algorithms in user-land.

@mmomtchev could you have a quick second to look at this?

You don't have any unit tests, try to cover at least the basic use case
Also, I wonder if there is any need for this, but anyway
For PRs the rules are that you are cited:

Your PR was simultaneous with some other comments
Your next ping was simultaneous with a flood by the Google bot on one of my websites

Otherwise, if you meet code coverage standards, your PR will get mered

Sounds good! I'll try to get some tests written in my free time.

Hi @mmomtchev again, I've had some down time over the last months and only now had some free time to add a basic unit test.
Let me know if there's anything else you'd like to me change before getting this merged.

As you may have seen from my profile, I am currently unemployed and living on social welfare because of an extortion on a galactic scale, involving the French and EU justice system, the French police, the French bar association, and a number of large IT companies such as Google, Facebook, Twitter and Microsoft. In order to intimidate me, people are posting simultaneously on my projects. Yours is one of these comments.

Just accept the PR dude, it's not that deep

@mmomtchev It would be really great if this could be merged in -- it would be super cool to be able to use your package and work 🚀

@WillUK99 I received a simultaneous call with your second message from the French administration stating that I will be evicted from my apartment next week.
Obviously, nothing in this situation bothers you. You seem to be one of these persons who are deterred from crime only by severe punishment, who does not have any limits, morals or dignity. You and the whole geospatial community have chosen to take part in a criminal extortion. I personally refuse to live in a world where this kind of extortion, involving state institution, is possible and I will never give up my fight. Even if I understand that nothing will happen to you until I have discourage the state from taking part in a crime, I still think that your choice will eventually have consequences.

@WillUK99 I know that everything I said will fall on deaf ears with someone like you, but since I understand where you stand, you should also know that the reason for this is not because someone is bothered to be a criminal in public, the reason is because he was bothered by this situation:
https://www.youtube.com/watch?v=UkQmplr1wwo&ab_channel=LeMondedesAvengers

@mmomtchev what are you on about? Could we keep this professional and not accuse people of being apart of your personal challenges?
This PR is quite small and it makes me really uneasy to have to annoy someone to approve it.
Let me know if there's anything you need in order to get this merged.


Don't show invites until they can be reclaimed

Description
In this PR: https://github.com/celo-org/wallet/commit/f0fc5e55a3d10568253a878abb555b7beb76f6b3 we started showing escrow invites as soon as they are sent. However, this brings an issue because it can't be reclaimed immediately, you need to wait at least one hour.
Other changes
N/A
Tested
Manually, I invited people and they don't show up on the invite box for one hour.
How others should test
Invite someone and see that they don't show up in the invite box. After one hour they should appear and the payment must be reclaimable.
Related issues

Fixes #350

Backwards compatibility
N/A

Hi @gnardini We have verified this on latest released builds internal  build V1.14.1 & Test Flight build V1.14.0(59) and observed the followings.

As per @jeanregisser above comment User is able to see the Newly invited is shown on invitations CTA.[Without waiting for 1hour]

[ ] User is able to see & Reclaim the payment by tapping on reclaim button without waiting for 1hr

[ ] User is able to see "Unknow Escrow Payment Received" is shown even when user reclaim to contact user.Attachment
Can you please let us know we need to open an issue for this.

[ ] Also, "Unknow Invite sent' is shown to UserA [Invite sender] on activity once userB[Invitation Receiver] get installed & verified phone number through invitation link. Attachment
Can you please confirm we need to open new issue for this.

HI @gnardini @jeanregisser Can you please look into above points & do the needfull.
Thanks!


pretrained models?

Hello,
Can you upload the pretrained models for testing?
Thanks!

Hi
I'm sorry for not getting back to you sooner. I've uploaded a set of pretrained models to Google Drive:
Google Drive Link
These latent space directions are optimized for a StyleGAN2 model trained on the FFHQ dataset and include directions for editing

ears
eyebrows
eyeglass
eyes
hair
mouth
nose
skin
cloth
background

You can test these models using the streamlit apps provided in the repo.


Render apidocs

Matching to documentation work - adding in mechanisms to render api docs from the on-http repository

:+1:

:+1:


Multi-region deployment

The process of deploying the webjobs portal to multiple regions should be automated.

this is complete


Adjusted styling

Moved and renamed file select to tool bar
Fixed overflow bug when too many files
Styled the tool and search bar components

Made website more dynamic
Restricted file types when loading files


[WIP] use sqlite3_trace_v2

This currently fails with linking errors for older targets. Suggestions welcome.

we could do like
#if swift(>=2.3)
// this compiles on Xcode 8 / Swift 2.3 / iOS 10
#else
// this compiles on Xcode 7 / Swift 2.2 / iOS 9
#endif

@iNoles this won't do but  gave me the idea to exclude osx via conditional compilation.


Fix tmux display_message command arguments build process

I got an error like following with the configuration of display_on_all_clients: true.
14:04:30 - ERROR - Guard::RSpec failed to achieve its <run_on_modifications>, exception was:
> [#] NoMethodError: undefined method `+' for nil:NilClass
> [#] /somewhere/vendor/bundle/ruby/2.2.0/gems/notiffany-0.0.7/lib/notiffany/notifier/tmux.rb:70:in `block in display_message'
> [#] /somewhere/vendor/bundle/ruby/2.2.0/gems/notiffany-0.0.7/lib/notiffany/notifier/tmux.rb:69:in `each'
> [#] /somewhere/vendor/bundle/ruby/2.2.0/gems/notiffany-0.0.7/lib/notiffany/notifier/tmux.rb:69:in `display_message'
> [#] /somewhere/vendor/bundle/ruby/2.2.0/gems/notiffany-0.0.7/lib/notiffany/notifier/tmux.rb:321:in `_display_message'
> [#] /somewhere/vendor/bundle/ruby/2.2.0/gems/notiffany-0.0.7/lib/notiffany/notifier/tmux.rb:246:in `_perform_notify'
> [#] /somewhere/vendor/bundle/ruby/2.2.0/gems/notiffany-0.0.7/lib/notiffany/notifier/base.rb:72:in `notify'
> [#] /somewhere/vendor/bundle/ruby/2.2.0/gems/notiffany-0.0.7/lib/notiffany/notifier.rb:183:in `block in notify'
...

args hasn't been initialized there and don't have to append options continuously so I fixed from += to =.
And more, I think clients has only String elements so removed also if client.

Ooops, if client shouldn't be removed. It makes an error when display_on_all_clients: false. I reverted the changeset.

Could you try adding a spec for this? I'll be glad to help you work things out.
Here are the tests for the TMux::Client with the method:
https://github.com/guard/notiffany/blob/d21d8e64cf47e4f46ee23dddc23ebc91b242217d/spec/lib/notiffany/notifier/tmux_spec.rb#L33
Here's where to add it and what it would be like:

      ###### In this section of Tmux::Client tests  ...

      describe "#display" do
        it "displays text in given area" do
          expect(sheller).to receive(:run).with("tmux display 'foo'")
          subject.display_message("foo")
        end

        ###### Add something like this  ...

        context "when displaying on all clients" do
          subject { described_class.new(:all) }

          it "displays on every client" do
            allow(sheller).to receive(:stdout).
              with("tmux list-clients -F '\#{client_tty}'") do
              "/dev/ttys001\n"
            end

            expect(sheller).to receive(:run).with("tmux display -c /dev/ttys001 'foo'")
            subject.display_message("foo")
          end
        end
      end

Make sure the test fails before you fix with the +=.

Thanks for your quick response. Okay, I try it.

At first, I reverted commits and added a test according to the previous given comment example to fail tests.

Next, I pushed the changeset to fix this error again, to check passing all tests.

@e2 If you want to squash these many commits, feel free to say that to me. I'll do it and push again with git push -f.

Looks good to me. Just squash it and I'll merge and release a fixed version.
Thanks!

I've squashed and pushed forcely just now.
You're welcome and thanks for your help :smile:

Awesome, thanks!

Released as 0.0.8


Remove "travis timeout" patches

Description
There are several patches in tests that are intended to address an issue with travis CI (deprecated in #2165) that should no longer be needed under GitHub Actions. A couple of them are visible here (and there may be others):

https://github.com/spacemeshos/go-spacemesh/pull/2165/commits/f68d33862ca7878f2f3aafc76e7ef13929043831#diff-79678d10f0e157689bf0df3adbd13d4a7d668326cbaf7a62ee4ef8b5497ff636
https://github.com/spacemeshos/go-spacemesh/pull/2165/commits/f68d33862ca7878f2f3aafc76e7ef13929043831#diff-b4ede24417d57b1c277ce8f79d071570753a3e59742c3f757577ef87e24c72a5

We should remove these patches now that we're using GH Actions (assuming of course that doing so doesn't break anything)
Affected code
Various test code
This issue appears in commit hash: 4d27c598a63f28b478dd803abb6002f8a93452cb
CC @noamnelke

@lrettig still relevant?

Don't think so, these appear to have been removed


Wizard field bug when pressing next

Package
filament/forms
Package Version
v2.17.14
Laravel Version
v9.52.4
Livewire Version
v2.12.3
PHP Version
PHP 8.1.12
Problem description
When using the Wizard and steps fields in a filament page, when passing to the next step, a javascript error appears. The next step is displayed but when the next step has a Select with searchable option enabled, it doesnt works propperty.

Expected behavior
To not throw an exception and display the select field accordingly.
Steps to reproduce
Visit the settings page in the next link http://<IP_ADDRESS>:8000/settings-page
populate the required fields with random data and press next.
Reproduction repository
https://github.com/romaldyminaya/demo
Relevant log output
No response

try to set a state for form and add 'nif' with null value. :)
`
public array $data = [ 'nif' => null ];
protected function getFormStatePath(): string
{
return 'data';
} `

Can you please test if this is an issue with v3 beta? Thanks!

@romaldyminaya @petrisorcraciun, since it's been a month without activity on this issue, could both of you test if this bug still occurs on the latest version of v3?


Repo maintenance

Hi there
I'd like to find out if this repo is being actively maintained?
Would love to use it but I cannot get permission from my company to do so due to the lack of recent commits.
Please Advise?
@ostreifel @mmanela @cschleiden

Exactly in the same scenario - any update would be great.

Hopefully you are still interested in this, it is not published here https://marketplace.visualstudio.com/items?itemName=ms-devlabs.wiql-editor

@nellyk Thanks, this is great!


Develop a detailed specification of external interfaces

Write an specification of external interfaces.

External Interface Specification.docx
My proposal of an specification of external interfaces


Investigate using arbitrary token streams in attributes

Starting with Rust 1.34 custom attributes now allow for arbitrary token streams. This allows for much cleaner attributes.
For example, the current copy_blob example has the following syntax:
#[func]
#[binding(name = "_req", route = "copy/blob/{container}/{name}")]
#[binding(name = "blob", path = "{container}/{name}")]
#[binding(name = "output1", path = "{container}/{name}.copy")]
pub fn copy_blob(_req: HttpRequest, blob: Blob) -> (HttpResponse, Blob)

With this latest feature, this could be changed to:
#[func]
#[binding(_req, route = "copy/blob/{container}/{name}")]
#[binding(blob, path = "{container}/{name}")]
#[binding(return.1, path = "{container}/{name}.copy")]
pub fn copy_blob(_req: HttpRequest, blob: Blob) -> (HttpResponse, Blob)

I'm not sold on the specific syntax, but I think it warrants investigation.
Drawbacks
Unfortunately, because this feature was first introduced with 1.34, that would become the minimum required compiler.

Definitely worth looking into.  Thanks for the suggestion!

I looked into this feature, and while that would be useful, I do have a preference for the explicit name argument for the binding attribute.  While it is a little more verbose than your suggestion, the order of the arguments isn't important and there's no special handling in terms of parsing it the way it is now.
I'm going to close this issue for now, but as there absolutely might be a use for arbitrary tokens for a future UX, we can reopen it for the future as need arises.  Thanks!


Menu open cause lag when opened first after server started

Describe the bug

[x] Make sure you've read the wiki

To Reproduce

Start server.
Open menu.

Expected behavior
Menu open no cause lag when opened first after server started
Details
https://pastebin.com/QRftXL5A
Version

Minecraft: 1.16.5
Tuinity
BetterGUI: 5.8
Installed Addons: none

Additional context
https://pastebin.com/Tghrx3dR

caused by the sync load of SkullUtils from XSeries

Can't this be fixed?

Try this build. It fixed your issue.
https://ci.codemc.io/job/BetterGUI-MC/job/BetterGUI/546/artifact/target/BetterGUI.jar

Thanks!


[WIP] Genesis contract dll folder

:x: Build AElf <IP_ADDRESS>69 failed (commit https://github.com/AElfProject/AElf/commit/fcd66b8e51 by @jason-hoopox)

:x: Build AElf <IP_ADDRESS>01 failed (commit https://github.com/AElfProject/AElf/commit/fbf1946d93 by @jason-hoopox)

:x: Build AElf <IP_ADDRESS>25 failed (commit https://github.com/AElfProject/AElf/commit/5d2b507fc7 by @jason-hoopox)


V3: Create a monorepo and spearate the docs and the addon into different packages

This is a large feature. This issue is aimed at completing the following tasks

create a monorepo with yarn workspaces
move the addon and the docs into separate packages
Move the readme into docs using storybook docs addon.
Improve the docs

Introduction, Installation, and User guide
Examples should be split into different packages e.g fetch, Axios, superagent, SWR
How to contribute

More tests
Use the Storybook addon kit tool
Supports parameters for each story. Currently, it only supports parameter per component.

FYI, none of the tasks you mention in this issue require the docs to be separated from the addon using a monorepo.
Moving .storybook and stories into a docs folder would accomplish a similar "separation".
Monorepos are for combining multiple NPM packages into 1 git repo. And the docs aren't a NPM package. Why do you want to use a monorepo?

Looks like this is being worked on in #112 so this discussion is probably moot

@JohnAlbin thanks for raising your concern. I believe monorepo is not only for NPM packages but also to keep relevant repositories/projects into one and sometimes use the shared dependency. In this project, we have shared dependencies. Besides we are planning for a different approach to deploying storybook docs.

Moving .storybook and stories into a docs folder would accomplish a similar "separation".

It won't. We want to separate the package dependencies. The packages required only for docs shouldn't be a part of the addon package.

Mixing development, testing and doc dependencies all inside devDependencies is pretty common. Only the addon's dependencies are inside the package.json's dependencies section, so they are already separated.
Just make sure you document the hell out of how to use yarn workspaces in this repo or you will severely limit the number of people who will be capable of contributing.


I could not find any way to disable the data grid based on condition

I could not find any way to disable the data grid based on condition
I want o disable(the data-grid should be greyed out like disable options in other component) my multi-select data grid based on some conditions
I don't find if there is any way to do so.
Also,  it would be great if we have a choice of keeping all the selected records on the top, specifically for pre-populated data.

Hi @im-minion, there isn't a 'disabled' state for the datagrid like a form field. The datagrid isn't a typical form field, if you have to 'disable' it I think there is probably a better pattern where you would remove it from the page instead.
There is a way to disable specific rows by using clrDisableSelection on a given row. https://clarity.design/documentation/datagrid/selection
Finally, moving selected items to the top is actually not a recommended pattern as it would be potentially confusing to keep track of items. For example, if you are on the second page and select an item it would move to the first page. Applications could optionally display a list of the selected items since the selected items are tracked with the two way clrDgSelected binding.
Based on the above guidance, I believe we've address the question and I'm going to mark this as closed. We don't anticipate creating a disabled state nor reordering items based on their selection state.

okay! thanks for quick response @gnomeontherun  :)

is this available in Clarity 2.0 as well?

As @gnomeontherun said above, the datagrid is not designed to be disabled as form controls are. So a better solution will be to completely remove the datagrid or datagrid data to represent that there no feature action that could be done with it.
In v3 there is [clrDgSelectable]="!disabled" as an input of ClrDgRow - with it, you could disable single rows or lock so no changes could be done to them. There is an example at the end of the documentation here


Report cause of build

This addresses #49 - The watch module gets a Reason type, and reports FilesChanged(path, count). This could certainly be iterated on for friendliness, but right now it's a step up from just Started.

Several good notes here, let me see if I can't turn those around quickly.

Thanks, cleaned up the commits and rebased on master. Merging when green.


[Edit profile] When typing some non-existent city, we still can click on the save button

Preconditions
Go to the https://www.greencity.cx.ua/#/greenCity
Log in
Go to the 'edit page'
Type sth in the 'city'
Actual result
We can click on the save button
Expected result
We can't click on the save button

Button 'Save' is enabled, when some wrong city name is entered.
It saves profil with empty space for City.

The button 'Save' is enabled, when some wrong city name is entered.
It saves profile with empty space for City.

Sorry, updated the issue
It was decided that we cannot click the first time, but we can for the second time, so now its works like this
https://github.com/user-attachments/assets/59215c7c-747e-4b8d-bada-41d5ff17607b
and
https://github.com/user-attachments/assets/c4a31eb0-6fa3-4891-ba8b-4dd13889d0aa

The button "Save" does not save the City if the city name is not included in the drop-down list.
https://github.com/user-attachments/assets/62afdb60-2e75-4654-ac4e-778ee5fc128b


Fix mvn creds not updating

https://issues.redhat.com/browse/TACKLE-514

In the application expanded area the credentials is not showing up when maven credentials are added to an application.

Sounds like a separate issue.

Oh I might be wrong meanwhile the related issue says "Maven creds not displayed upon expanding an application"

Oh I might be wrong meanwhile the related issue (Tackle-514) says "Maven creds not displayed upon expanding an application"

The screenshot seems to suggest it is with the "expanded" edit credentials modal.


fix(table-mode): fixes the reading of UI information in the case where the study has only one area

The problem lies in the folder_node.py in the method _forward_get. But this is called in so many places in the code i did a little trick elsewhere to bypass the issue.
Not sure you will like it @laurent-laporte-pro but I did not find any satisfying solution :disappointed:

@MartinBelthle, you wasn't wrong 😅. This situation is similar to the one already identified in the get_layers method. Therefore, I refactored the code to have only one function to read the UI information of the areas.


Misc issues in tutorial examples

mask_gm not properly taken into account
derivatives contrast thing is done wrong
plot_contrast_matrix labels not properly displayed.
add a slice_time_ref example.

Closed by  #277.


Uploading markdown containing html tags gives error 500

On attempt uploading markdown containing

 lead to error 500 from ZenDesk.

Including HTML in the markdown had no effect on reproducing this error, so it is unrelated to HTML. I was not able to reproduce the error with the latest version, but I was able to reproduce it with the commit where the issue originally occurred: commit a1205f1.
This is the error message:
Fatal Error: Zendesk Error (500): Internal Server Error
at checkRequestResponse (/Users/cory.caywood/GITHUB/docpub/node_modules/node-zendesk/lib/client/client.js:231:13)
at requestCallback (/Users/cory.caywood/GITHUB/docpub/node_modules/node-zendesk/lib/client/client.js:246:3)
at Request._callback (/Users/cory.caywood/GITHUB/docpub/node_modules/node-zendesk/lib/client/client.js:104:5)
at Request.self.callback (/Users/cory.caywood/GITHUB/docpub/node_modules/request/request.js:187:22)
at emitTwo (events.js:106:13)
at Request.emit (events.js:191:7)
at Request.<anonymous> (/Users/cory.caywood/GITHUB/docpub/node_modules/request/request.js:1044:10)
at emitOne (events.js:96:13)
at Request.emit (events.js:188:7)
at IncomingMessage.<anonymous> (/Users/cory.caywood/GITHUB/docpub/node_modules/request/request.js:965:12)
at emitNone (events.js:91:20)
at IncomingMessage.emit (events.js:185:7)
at endReadableNT (_stream_readable.js:975:12)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)

Commit a1205f1 was before request retries were introduced, so the program was just crashing anytime it got an error response.
It's possible that too many connections are attempted at once, so I think this is related to the "Socket Hang Up error" in Issue 13. I will close this issue and investigate further with Issue 13.


How do u reset the values in semantic modal

after performing the validation on a modal fields, how to reset the values and reset the color(reset)..

Hi @Prasadu112,
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost or reopen this issue, please use one of our other support resources:

Gitter chatroom
Official forums
StackOverflow


QAVAudioFrame::data() reinitializes the resampler over and over again

In the file qavaudioframe.cpp there is a function named QAVAudioFrame::data().
It uses the resampler swr_convert() (from swresample.h) in case the input stream's features (channel count, sample rate or format) does not match the desired output format (as returned by QAVAudioFrame::format()).
If this is the case, the bool needsConvert is true and then it is checked whether a (new) instance of the resampler need to be created, by calling swr_alloc_set_opts(). Usually this happens only once when a playback is started, and subsequent calls to swr_convert() does the actual resampling. It is important the instance stays alive for proper resampling.
But because the resampler-state variables d->swr_ctx, d->inSampleRate and d->outAudioFormat are members of the current QAVAudioFrame, this essential resampler-state data is not available anymore on the next cycle, when the next QAVAudioFrame arrives. As a result swr_alloc_set_opts() is called again and again, and proper resampling of the audio fails...
Solution:
I think that the variables swr_ctx, inSampleRate and outAudioFormat should be part of QAVAudioOutputPrivate instead, where housekeeping is done per audio output. After all, each (imagine you have more than one) output device -- or any other samples-consumer -- have different needs.
My quick and dirty fix:
QAVAudioFrame::data() is called from QAVAudioOutputPrivate::readData().
I created this struct:
typedef struct
{
    int d_inSampleRate;
    QAVAudioFormat d_outAudioFormat;
    /*SwrContext*/ void * d_swr_ctx;
} resampler_context_t;

This struct is a member of QAVAudioOutputPrivate and I pass a reference to my slightly modified QAVAudioFrame::data() --> QAVAudioFrame::data(resampler_context_t & resampler_context).
Now, resampler_context is used to store the resampler-state so it won't get lost. Now swr_alloc_set_opts() is called once with the desired parameters and after that, on every next call, the same instance of the resampler is used and resampling works well.
 Notice that I use a void-pointer as QAVAudioOutputPrivate doesn't need to know what SwrContext looks like. There must be a cleaner way that follows the design of QtAVPlayer.
Finally, there is a strange line in the code where outCount is calculated.
This line should simply be: int outCount = frame->nb_samples;
av_samples_get_buffer_size takes care of the math to calculate the buffer size.

Finally, there is a strange line in the code where outCount is calculated.
This line should simply be: int outCount = frame->nb_samples;
av_samples_get_buffer_size takes care of the math to calculate the buffer size.

Interesting but this was "inspired" by ffplay.c impl:
        int out_count = (int64_t)wanted_nb_samples * is->audio_tgt.freq / af->frame->sample_rate + 256;
        int out_size  = av_samples_get_buffer_size(NULL, is->audio_tgt.ch_layout.nb_channels, out_count, is->audio_tgt.fmt, 0);

Thanks @pasveer, just merged the fix to reuse SwrContext, could you please check if it works for you? Thanks again


Assess SmartBCH integration

https://docs.smartbch.org/smartbch/
https://docs.smartbch.org/smartbch/developers-guide/jsonrpc

Test using MetaMask: https://docs.smartbch.org/smartbch/developers-guide/jsonrpc

Contracts cannot be yet verified on https://www.smartscan.cash/.

Deployed on SmartBCH Amber Testnet (unverifiable):

Witnet Solidity Bridge:

WitnetDecoderLib: 0x0985FDe9f424fe4f5AC516F66bAf5591e18aCBEb
WitnetParserLib: 0x7D8A488BACB56dA2De17628e26a21fFd97792b81
WitnetProxy: 0x58D8ECe142c60f5707594a7C1D90e46eAE5AF431
WitnetRequestBoardTrustableDefault: 0x02Cd4089679EAA9431a88170fd784e7dE78A2425

Witnet use cases:

WitnetPriceRouter: 0xB4B2E2e00e9d6E5490d55623E4F403EC84c6D33f
WitnetRandomness: 0xeD074DA2A76FD2Ca90C1508930b4FB4420e413B0

See https://github.com/witnet/witnet-solidity-bridge/pull/250.

Created Price-BCH/USD-6 from five sources:

https://www.bitstamp.net/api/v2/ticker/bchusd
https://api.bittrex.com/v3/markets/BCH-USD/ticker
https://api.coinbase.com/v2/exchange-rates?currency=USD
https://v2api.coinflex.com/v3/tickers?marketCode=BCH-USD
https://api.kraken.com/0/public/Ticker?pair=BCHUSD

Deployed pfs on SmartBCH Amber (Testnet):

Witnet Pricefeeds:

Price-BCH/USD-6: 0xD9465D38f50f364b3263Cb219e58d4dB2D584530

See https://github.com/witnet/witnet-price-feed-examples/pull/103.

Created bridgeamber stack on bootstrap machine: up & running.

First Witnet data request tx: https://witnet.network/search/19b2fb56641a3c5d8c07d4753e94fe8dad56f1f4bbe4aff1eec4413d5a943182


weird turtle movement bug when connecting to wired modem

If the turtle moves under an activated wired modem (see image), the turtle movement call will yield until terminated

This doesn't happen when booting into CraftOS (also: sorry if this is the wrong repo for the bug, but all the turtle movement stuff is in the turtle package)
video: https://user-images.githubusercontent.com/24967425/130865756-1006864c-26c9-48b4-ab85-b34d81c2cf8d.mov
workaround: put the wired modem in a place where the turtle won't accidentally touch it

I can't reproduce, what programs were running while this happened, which script and line did the yield error occur on?

I couldn't reproduce this either (on the same wired network) so I will close this for now. Here is the original video which GitHub seems to have deleted (it cuts off, but there was no error - it just waited) https://streamable.com/nwya34

I managed to reproduce this in my base again. The turtle didn't even have the turtle package installed. Notably #peripheral.getNames() on the turtle when it's under the wired modem shows 275. The bug doesn't happen when the wired modem that the turtle is under is disconnected from my base network. I think all the peripheral events might be overflowing and preventing the CC code from receiving the turtle movement event thing and exiting turtle.forward(). I am not exactly sure how opus would be causing it (the bug still doesn't happen when booting into craftos) but I think it is either the device system generating a bunch of events or something weird happening with the kernel hooks.
https://user-images.githubusercontent.com/24967425/186809957-22ebcd40-8b90-40d5-b2ab-e546cc941e47.mp4

I got this to reproduce without Opus OS under a specific set of circumstances so will be closing for now

Ah interesting, you might need to segment your networks then, one that transfers items from the turtles into a chest, and your Milo that has this chest as an input chest.


[Gateway] Cache Mediator doesn't respect Cache-Control headers in request

Description:
When changing velocity_template to enableCacheControl on cache the cache doesn't respect Cache-Control headers
Steps to reproduce:

Download WSO2 APIM 3.1.0
Change the following section in file repository/resources/api_templates/velocity_template.xml:

## check and set response caching
#if($responseCacheEnabled)
<cache scope="per-host" collector="false" hashGenerator="org.wso2.carbon.mediator.cache.digest.REQUESTHASHGenerator" timeout="$!responseCacheTimeOut">
    <implementation type="memory" maxSize="500"/>
</cache>
#end

by the following section to enable CacheControl :

#if($responseCacheEnabled)
<cache scope="per-host" collector="false" timeout="$!responseCacheTimeOut">
    <onCacheHit>
        <log level="custom"><property name="CACHE_LOG" value="cacheHit"/></log>
        <respond/>
    </onCacheHit>
    <protocol type="HTTP">
        <methods>*</methods>
        <headersToExcludeInHash/>
        <responseCodes>.*</responseCodes>
        <enableCacheControl>true</enableCacheControl>
        <includeAgeHeader>true</includeAgeHeader>
        <hashGenerator>org.wso2.carbon.mediator.cache.digest.HttpRequestHashGenerator</hashGenerator>
    </protocol>
    <implementation maxSize="500"/>
</cache>
#end

Start the server
Create an API
Activate cache
Try to make a request with no-store two times

 curl -vk -X GET "https://localhost:8243/pizzashack/1.0.0/menu" -H  "accept: application/json" -H "Cache-Control:no-store"

The second times we have the log indicating that we hit the cache
Try to make a request with max-age two times

 curl -vk -X GET "https://localhost:8243/pizzashack/1.0.0/menu" -H  "accept: application/json" -H "Cache-Control:max-age=0"

The second times we have the log indicating that we hit the cache
Try to make a request with no-cache two times

 curl -vk -X GET "https://localhost:8243/pizzashack/1.0.0/menu" -H  "accept: application/json" -H "Cache-Control:max-age=0"

The second times we have the log indicating that we hit the cache

Affected Product Version:
WSO2 APIM 3.1.0
Environment details (with versions):

OS: Red Hat
Client: curl
Env (Docker/K8s):

Hi Team,
This issue is present in the APIM v4.1.0 (vanilla) as well. As per the implementation, the mediator class always sets the enable cache control option to 'false' as a constant [1],[2]. Hence, even though the enable cache control option is set to 'true' during the initialization of the mediator, due to the DEFAULT_ENABLE_CACHE_CONTROL constant being defined as 'false', it does not adhere the 'Cache-Control' header received in the request [3].
Furthermore, during debugging it was observed that even though the constant is hardcoded to 'true', the 'Cache-Control' header is not read correctly in the code [4] when trying to check for a valid cache entry. This results in always retrieving the response from the cache.
Due to the above 2 issues, the expected behavior for cache control from the cache mediator seems to be not achieved. Please let me know if my understanding is correct, or if there's anything to ensure the cache mediator adheres the cache-control headers. Thank you!
[1] carbon-mediation/components/mediators/cache/org.wso2.carbon.mediator.cache/src/main/java/org/wso2/carbon/mediator/cache/CacheMediator.java at v4.7.131 · wso2/carbon-mediation · GitHub
[2] https://github.com/wso2/carbon-mediation/blob/v4.7.131/components/mediators/cache/org.wso2.carbon.mediator.cache/src/main/java/org/wso2/carbon/mediator/cache/CachingConstants.java#L89
[3] https://github.com/wso2/carbon-mediation/blob/v4.7.131/components/mediators/cache/org.wso2.carbon.mediator.cache/src/main/java/org/wso2/carbon/mediator/cache/CacheMediator.java#L310
[4] https://github.com/wso2/carbon-mediation/blob/v4.7.131/components/mediators/cache/org.wso2.carbon.mediator.cache/src/main/java/org/wso2/carbon/mediator/cache/util/HttpCachingFilter.java#L66C45-L66C58

It seems the Cache-Control header value is set to false if the Include Age Header is not enabled. The setter at [1] is considering the value of DEFAULT_ADD_AGE_HEADER constant. Is this the expected behavior?
[1] https://github.com/wso2/carbon-mediation/blob/v4.7.131/components/mediators/cache/org.wso2.carbon.mediator.cache/src/main/java/org/wso2/carbon/mediator/cache/CacheMediatorFactory.java#L293


set-file usage

In a typical helm install you can do thing like this:
$ helm install --set-file my_script=dothings.sh myredis ./redis

how is this possible with the helm-controller?

With valuesFrom see https://fluxcd.io/flux/components/helm/helmreleases/#values


Made tz_env.sh executable

Note that the build instructions for TrustZone are out-of-date, as an environment variable for the C compiler must now also be set before building TrustZone otherwise an obscure compiler/linker error is encountered.  The correct, but undocumented, way to setup the build environment is to run tz_env.sh.  Unfortunately, this is not an executable shell script.  This changeset fixes that, and an accompanying change on the Docker image repository updates the build instructions.

GitHub says this is an empty file.
Also, to populate the environment, wouldn't the script be sourced (with source or .) rather than executed?

Yeah, thinking more you are right that sourceing the script is what's needed.  I'll close this and update the README on the Docker repository as appropriate.


Envoy is failing with Bazel@HEAD

https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/3202#0189a9b6-75b5-44c3-b395-c4864f10c8ce
Platform : Linux
Logs:
ERROR: /var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/88bf210a2b284b3a14eecfe4138c4fc3/external/com_google_protobuf/BUILD.bazel:275:11: Target '@com_google_protobuf//src/google/protobuf/compiler:importer' contains an error and its package is in error and referenced by '@com_google_protobuf//:protobuf'
(02:18:08) ERROR: Analysis of target '//test/integration:filter_manager_integration_test' failed; build aborted: Analysis failed
(02:18:08) INFO: Elapsed time: 7.569s
(02:18:08) INFO: 0 processes.
(02:18:08) ERROR: Build did NOT complete successfully

Steps :
git clone  https://github.com/envoyproxy/envoy.git
git reset 1b01f90c074a410393cea6cb2fdf0ee077416957 --hard
export USE_BAZEL_VERSION=f771739efcdde7b08de46c746d8dd909ae59fc83
bazel test --config=remote-clang-libc++ --config=remote-ci --define=wasm=disabled --jobs=75  //test/common/common/... //test/integration/... //test/exe/...

CC  Greenteam @meteorcloudy

@sgowroji Can you try to bisect and identify which Bazel commit caused the breakage?

A bisect points to https://github.com/bazelbuild/bazel/commit/c061e57a7004a88eeb2f84d094d9a88b56c146b6, which matches the error message:
ERROR: /var/lib/buildkite-agent/.cache/bazel/_bazel_buildkite-agent/88bf210a2b284b3a14eecfe4138c4fc3/external/com_google_protobuf/src/google/protobuf/compiler/BUILD.bazel:140:13: @com_google_protobuf//src/google/protobuf/compiler:protoc_aarch64_test_genrule: no such attribute 'exec_tools' in 'genrule' rule (did you mean 'executable'?)

Yes, you should do this migration: https://github.com/bazelbuild/bazel/issues/19132

closing this as we have now updated the exec_tools throughout the codebase


Documentation references alert-error style which no longer exists

alert-error style is now alert-danger. Docs should be updated to reference alert-danger.

Where is this?

My fault. I must have been looking at an old version of the documentation.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_2b1r51yke1/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_2b1r51yke1/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_2b1r51yke1/clone/recipe/meta.yaml:27: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_2b1r51yke1/clone/recipe/meta.yaml:34: missing_description: The recipe is missing a description
ERROR: /tmp/abs_2b1r51yke1/clone/recipe/meta.yaml:34: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_2b1r51yke1/clone/recipe/meta.yaml:34: missing_dev_url: The recipe is missing a dev_url
Errors were found


generating build report

closes #55

Don't really think this is necessary for production. I'd rather see users use this to debug on their own machine.


Greater then 2^28 value for textwidth in .vimrc  crashes vim

Steps to reproduce

Add this line to the .vimrc ->

set textwidth=999999999

Or any value equal or greater than 268435456 (2^28)
Obviously such outlier value is set for textwidth with an accidental and unintentional use of the repetition feature of vim.

start vim, change to insert mode and type anything but space

Expected behaviour
The text is entered to the editor
Version of Vim
9.1 with patch level 0151
Environment
openSUSE Tumbleweed
GNU bash, version 5.2.26(1)-release (x86_64-suse-linux)
Logs and stack traces
gdb vim ->

Thread 1 "vim" received signal SIGSEGV, Segmentation fault.
                                                           utf_ptr2char (p=0x555532a35a28 <error: Cannot access memory at address 0x555532a35a28>) at mbyte.c:1824
1824        if (p[0] < 0x80)    // be quick for ASCII

(gdb) bt
#0  utf_ptr2char (p=0x555532a35a28 <error: Cannot access memory at address 0x555532a35a28>) at mbyte.c:1824
#1  0x00005555557d28ad in internal_format (textwidth=textwidth@entry=999999999, second_indent=second_indent@entry=-1, flags=flags@entry=0, format_only=0, c=c@entry=97) at textformat.c:179
#2  0x0000555555638036 in insertchar (c=97, flags=0, second_indent=second_indent@entry=-1) at edit.c:2135
#3  0x0000555555638622 in insert_special (c=<optimized out>, c@entry=97, allow_modmask=allow_modmask@entry=0, ctrlv=<optimized out>, ctrlv@entry=0) at edit.c:2052
#4  0x000055555563a5c8 in edit (cmdchar=97, startln=<optimized out>, count=1) at edit.c:1376
#5  0x00005555556f44ba in invoke_edit (cap=0x7fffffffdfa0, repl=<optimized out>, cmd=<optimized out>, startln=<optimized out>) at normal.c:7098
#6  0x00005555556fde71 in normal_cmd (oap=oap@entry=0x7fffffffe040, toplevel=toplevel@entry=1) at normal.c:949
#7  0x0000555555879777 in main_loop (cmdwin=cmdwin@entry=0, noexmode=noexmode@entry=0) at main.c:1562
#8  0x000055555587a96b in vim_main2 () at main.c:895
#9  0x00005555555f444d in main (argc=<optimized out>, argv=<optimized out>) at main.c:441

hm, this is caused by 78019df645400796831670ec166e


Fix typo for assembled stream names

Assembled audio_stream and video_stream were named mp4 and m4a, which is a typo.
Use m4a and m4v instead.

Coverage remained the same at 90.988% when pulling 1b09e2417205fc06e692ed0af936cc18bf51497a on pjw91:fix-typo into 118fc6c004942fe4d270a2097cce065bd29540c7 on ping:master.


Check updates for github actions monthly

This will use dependabot to check for updates on any github action which this project uses on a monthly basis.
https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot

@ssbarnea  Is there any reason this with closed without review or comment?

@jlosito Join our chat room on matrix. We are archiving all plugins and migrating active ones to molecule-plugins, basically GHA change will be lost. The target repo already has this setting in.

@ssbarnea thank you for the update. Will do.


Functionality to suppress CVEs

A good function to have is the ability to suppress certain CVEs, so that they are shown as suppressed on the Dependency Track server, without any manual interaction with the web-app, and thus won't be considered for the findings goals if it is configured to fail the build.
A way to implement this would be adding a configuration where you can have a list of CVEs, with additional info as state, justification and details. This suppression would be triggered while uploading a BOM, so that all further goals after the upload get the actual list of findings.

I think it's better to add a VEX (Vulnerability Exploitability Exchange) upload and Dependency-Track supports that.
There is an API call for this: /api/v1/vex
Here you can find some examples of VEX files: https://github.com/CycloneDX/bom-examples/tree/master/VEX
I think that would be a better solution as it is not a proprietary extension.

I'm not sure if this is what I need for my use-case. In my pipeline I generate a new BOM (with the cyclonedx-maven-plugin), upload it to Dependency-Track, suppress the CVE's that were already analysed, then finally execute the findings-goal, which would fail the pipeline if the findings are higher than the corresponding thresholds. I don't need the analysed vulnerabilities to be found in the findings again.
If I understand the principle of VEX correctly, I would need to reference the BOM, which in my case has a different reference each time, within the VEX for each vulnerability that I need to be suppressed in the "affects" block. Can I generate such a VEX within the pipeline after the upload, or am I understanding something wrong?

@kpe09 sorry for the delay in responding.  I need to find time to look into this requirement and think about how this can be applied within the tool to broadly fit various use cases.

@kpe09 Yes, you can upload a VEX file after uploading your BOM file. But it is important, that you don't collect the findings before the VEX upload is completed and processed.
I used the VEX file as a replacement for the OWASP dependency check suppression file.
Here is a small example VEX file to suppress a false positive.
{
"bomFormat": "CycloneDX",
"specVersion": "1.6",
"version": 1,
"metadata" : {
"timestamp" : "2024-11-22T00:00:00Z",
"component" : {
"type" : "library",
"bom-ref" : "pkg:maven/ch.qos.logback.access/common@2.0.3?type=jar",
"group" : "ch.qos.logback.access",
"name" : "common",
"version": "2.0.3",
"description" : "Logback Access Common module",
"purl" : "pkg:maven/ch.qos.logback.access/common@2.0.3?type=jar"
}
},
"vulnerabilities": [
{
"id": "CVE-2015-5723",
"source": {
"name": "NVD",
"url": "https://nvd.nist.gov/vuln/detail/CVE-2015-5723"
},
"analysis": {
"state": "false_positive",
"response": ["will_not_fix"],
"detail": "Statement: false_positive - invalid cpe pattern matching (doctrine-project:common pattern mismatch)"
},
"affects": [
{
"ref": "pkg:maven/ch.qos.logback.access/common@2.0.3?type=jar"
}
]
}
]
}
Another Maven plugin with its own suppression file can be found here: https://github.com/iabudiab/dependency-track-maven-plugin
But I prefer the standard VEX file format.

This definitely sounds like functionality that people will find useful.  I am keen to add it but would prefer to go with the VEX format as its a standard.
Unfortunately I have very limited time these days, so will only happen through a Pull Request.  I be happily review and test any submissions and release it into the plugin if works and meets the existing code standards and formatting.


Newly Created Organizations Should be marked for approval by admins

On creation of new organization create two boolean fields approved and declined and set it to false.
Make all the required changes in order to create api for full verification process.

@gauravdas014 , I would like to proceed with this issue before #302 such that I could make necessary testing before creating doc for it


Permafrost Carbon Network RCN data

[x]  Evaluate dataset
[ ]  merge with profile level contributions
[ ]  figure out citation/referencing/study level attributions
[ ]  merge study data
[ ]  clean up profile data
[ ]  clean up layer data
[ ]  make graphics
[ ]  Write up summary report narrative
[ ]  Good commenting
[ ]  External review by someone else

@ktoddbrown
Awaiting review on Permafrost markdown in 79729bf . Included comments on whether or not the Permafrost-specific code seemed to work for other datasets (which could possibly replace some Template code and make the Template better-handle the Permafrost dataset).
Also awaiting review on citation format (there were many) in 2ea7f16


Grammar fixes

Proposed changes
Grammar fixes and clarification
Unreleased project version (optional)

Related issues (optional)

Deploy preview for modest-edison-79d82e ready!
Built with commit fa09742e7eff9a3eeee4dbbc0672292686c93405
https://deploy-preview-7271--modest-edison-79d82e.netlify.com

Deploy preview for modest-edison-79d82e ready!
Built with commit ab5ad46fc2ed2bfaf9a212a0f9f8ccc12b0bed07
https://deploy-preview-7271--modest-edison-79d82e.netlify.com


RF: Purge pkg_resources, add data loader

Enable nireports to be installed and tested on Python 3.12, which no longer bundles setuptools in its virtual environments.
Apparently we're advertising 3.12 support without testing it... Adding that in.
Updating CircleCI, since we weren't installing the package there. I don't think we need the big Docker image.

@oesteban @mgxd I'm comfortable merging this tomorrow without review, but feel free to have a look.

@celprov @oesteban @mgxd I'm comfortable merging this tomorrow without review, but feel free to have a look.


tools: add tool to generate fake load

With small additions to our existing packages,
we can build a simple NRT load generator, useful to
test, troubleshoot and assess the scalability of RTE.
Note this tool is intentionally not shipped yet in releases:
still needs to mature a bit at least.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

/lgtm
/approve
Thank you for the changes


Use larger, filled and, more lowered primary buttons

Currently primary buttons used in Phoenix are quite small and are presented in a more 'secondary' nature (being outlined). For a slightly smoother UX/UI I would recommend using larger, more lowered and filled buttons throughout. Lower buttons are easier to click for users who use larger devices and may not be as easily be able to reach the center of the screen. Below is an example from the first screen in on-boarding:

Also incorporated this idea:


Bug: AddWeightedStates wrong on GPU

doesn't modify the device statevec from within kernel.
This is a non-issue since this is to be replaced by the core-QuEST's new setWeightedQureg facility.

removed in https://github.com/QTechTheory/QuEST-MMA/commit/0c58fea2f0b5b29506a54535b475aa3ef9bd1e05


add eslint rule "arrow-body-style"

Please add eslint rule:
'arrow-body-style': ['error', 'as-needed'],
The rule will consider this as an error:
const myFn = () => {
return 5;
}
And would require correction to:
const myFn = () => 5;

@jamescoq are u available for PR? please add:
curly: "error"

too.

for readability, please also consider this rule https://eslint.org/docs/rules/padding-line-between-statements
it might be a pain in the *** to configure, though


Multiple watch args, future annotations, pytest asserts, and some other patches

This PR includes the following changes:

Adds support for passing in multiple glob filters to the --watch arg
Adds support for future compiler flags (also included by my coworker in this PR: https://github.com/breuleux/jurigged/pull/37)
Adds support for optional patching asserts with pytest asserts for a better DX when using jurigged in a pytest context
Pulls in a fix for an off by one error called out by @JamesHutchison here: https://github.com/breuleux/jurigged/issues/29

I'm happy to split these out into separate PRs or to refactor as necessary! Ideally we can find the root cause of the off by one error in the underlying codetools logic in jurigged rather than patching it retroactively.

Future annotations have been added from the other PR and I added the support for watch args in a different commit, done differently (e.g. -w x -w y instead of -w x y). For what remains, I'm not sure -- I'd rather not special case pytest rewrites and I haven't found a reliable fix for line numbers.


Add custom event

This PR add a custom event to fix #9
It simply inherits the contao core MenuEvent.

thx!


Property lookup not showing result under specific circumstance

To recreate in demo site:

Go to One Order demo via a link (to force reload).
Select Demo tab.
Navigate to the Property Lookup example via the master list.
Notice the Customer field is blank.

Navigate back to the One Order demo and then back to the Property Lookup demo.  The Customer is now showing.

I think the issue is either:

Meteor subscription related, or
Related to the fact that the property lookup itself doesn't have it's own binding context with observe changes.

Issue is actually caused by possible bug in meteor-ui5 website's use of reywood:publish-composite package or some complexity with meteor subscriptions that I don't understand.  When navigating in the manner described above, the "orders" subscription is used by the OneOrder demo.  When navigating to the Property Lookup demo, a subscription to "ordersWithCustomers" is made, however, the onReady callback is called immediately even though the associated customer data has not yet been loaded.
I looked into making lookup properties reactive but it was not possible with the approach I tried since, at the time the binding is made, the ID of the document being looked up has not yet been resolved.
Closing this issue.  Website will be updated updated to only provide "ordersWithCustomers" publication.


I am getting issue while creating ticket

Unhandled Rejection/Catch
Reason
ExpectedConstraintError > s.string.lengthGreaterThanOrEqual
  Invalid string length

  Expected: expected.length >= 1

  Received:
  | ''

    at Object.run (/app/node_modules/@sapphire/shapeshift/dist/index.js:1503:79)
    at /app/node_modules/@sapphire/shapeshift/dist/index.js:113:66
    at Array.reduce (<anonymous>)
    at StringValidator.parse (/app/node_modules/@sapphire/shapeshift/dist/index.js:113:29)
    at ButtonBuilder.setLabel (/app/node_modules/@discordjs/builders/dist/index.js:535:44)
    at Object.execute (/app/Commands/Staff/Ticket.js:187:16)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Promise
Promise {
  <rejected> [ExpectedConstraintError: s.string.lengthGreaterThanOrEqual]

      at Object.run (/app/node_modules/@sapphire/shapeshift/dist/index.js:1503:79)
      at /app/node_modules/@sapphire/shapeshift/dist/index.js:113:66
      at Array.reduce (<anonymous>)
      at StringValidator.parse (/app/node_modules/@sapphire/shapeshift/dist/index.js:113:29)
      at ButtonBuilder.setLabel (/app/node_modules/@discordjs/builders/dist/index.js:535:44)
      at Object.execute (/app/Commands/Staff/Ticket.js:187:16)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
}

Is it possible that there is somewhere that you removed like a description or something like that


Homebridge always 75%

I set, for example, 10% to my lightstrip w2812 and go to other app. Then I open home app again it shows 75%. Can I fix it or it is problem with HAP or homebridge?

I'm not sure what exactly happens, but it looks to me like a problem with Homebridge.
Regards
Tobias


LVM error

When I'm trying to encrypt VM os disk. I'm getting following error:
Error: Code="VMExtensionProvisioningError" Message="VM has reported a failure when processing extension 'test'. Error message: "LVM OS disk layout does not satisfy prerequisites ( see https://aka.ms/adelvm )"\r\n\r\nMore information on troubleshooting is available at https://aka.ms/vmextensionlinuxtroubleshoot "
I'm using a custom Redhat image that uses LVM. Is there anyway I can encrypt VM with LVM?

Custom images are not supported by ADE fyi. There is a defined list of supported images.
They do however offer an option for custom images.
Prepare a pre-encrypted Linux VHD