reqwest 0.11 breaks tokio dependencies

The 0.4.7 release bumped the reqwest dependency to 0.11 which depends on tokio 1.x instead of 0.2.x
arangors can no longer be used with actix-web until they release the v4 (still in beta) based on tokio 1.x.
I think this version bump should be mentioned in the Changelog as a breaking change or rollbacked

Well, personally I don't regard a dependency bump a potential breaking API change. But it surely is a inconvinience. I think I can bump the current version to 0.5.0 and addtionally publish a 0.4.8 with all the same code except that reqwest remains at 0.10.
But the future changes will not be added in the 0.4 world where reqwest is fixed at 0.10. Else it can be really tiresome to maintain.
Sorry for the long wait and thank you so much for your feedback!

Tokio dependencies are quite tricky and a big issue on many repositories.
Cargo can't resolve it as it causes a runtime crash.
It think your solution is good. The 0.4.7 should be yanked afterwards I think

Now that the v0.4.8 is published, and I yanked the 0.4.7 to keep 0.4 consistently fixed at tokio v0.2.

Thanks !


Ubuntu Icon option seems to require a full path.

The contents of /usr/share/applications/code.desktop
Includes
[Desktop Entry]
Name=Visual Studio Code
Comment=Code Editing. Redefined.

But this is undone on the next update.

It pulls the icon from /usr/share/pixmaps/code.png which should have been installed via the deb package:  https://github.com/Microsoft/vscode/blob/23f80bae3594818416a469b29dd4d9d45207fb76/build/gulpfile.vscode.linux.js#L66


feat(kms-support): Adding support for KMS keys

Closed in favor of https://github.com/spanio/node-sshpk/pull/1/files


Missing files

I am unable to open the song creator as it displays two error messages:
"The code execution cannot proceed because MSVCP140.dll was not found. Reinstalling the program may fix this problem."
"The code execution cannot proceed because VCRUNTIME140.dll was not found. Reinstalling the program may fix this problem."
I have tried reinstalling, but these messages keep popping up. I did not download the game through Steam, so that might be part of the issue.

nvm i downloaded the things i needed


Calendar In C

Description
I have prepared a documentation on Calendar In C
This fascinating program prints a calendar of the year on the basis of :

Year
Month by Month in sequential order by inputting Year by the user
Day by inputting Date , Month and Year

Hope you find the Date, Date, Month of your choice! :)
Enjoy the code!
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Error loading sample data

When using a fresh install, I get an error on bootstrapping sample data.
Command:
bundle exec rake db:bootstrap
Output:
Load Sample Data? [y/n]: y
Loaded Payment Methods samples
Loaded Shipping Categories samples
rake aborted!
ActiveRecord::StatementInvalid: PG::UndefinedTable: ERROR:  relation "spree_shipping_methods_zones" does not exist
LINE 5:                WHERE a.attrelid = '"spree_shipping_methods_z...
^
:               SELECT a.attname, format_type(a.atttypid, a.atttypmod),
pg_get_expr(d.adbin, d.adrelid), a.attnotnull, a.atttypid, a.atttypmod
FROM pg_attribute a LEFT JOIN pg_attrdef d
ON a.attrelid = d.adrelid AND a.attnum = d.adnum
WHERE a.attrelid = '"spree_shipping_methods_zones"'::regclass
AND a.attnum > 0 AND NOT a.attisdropped
ORDER BY a.attnum
Most relevant stack trace:
/Users/brett/.rvm/gems/ruby-2.2.4@spree_react/gems/spree_sample-3.1.0/lib/spree/sample.rb:13:in load_sample' /Users/brett/.rvm/gems/ruby-2.2.4@spree_react/gems/spree_sample-3.1.0/lib/spree_sample.rb:12:in load_samples'
/Users/brett/.rvm/gems/ruby-2.2.4@spree_react/gems/spree_sample-3.1.0/lib/tasks/sample.rake:20:in block (2 levels) in <top (required)>' /Users/brett/.rvm/gems/ruby-2.2.4@spree_react/gems/spree_core-3.1.0/lib/tasks/core.rake:95:in block (2 levels) in <top (required)>'
Interestingly enough, the relation table spree_shipping_method_zones does exist.

As a temporary fix I added the migrations from the latest download of spree and it fixed the issue.

Fixed via https://github.com/spark-solutions/spark-starter-kit/pull/36


[MRG] Speedup topos

Now all the topo_plots use Eric's speedier unified axis.
EDIT:
Closes https://github.com/mne-tools/mne-python/issues/2953

Coverage decreased (-0.01%) to 90.752% when pulling 921cb0d74caf7548d61b4137d04b3049d64da4f9 on jaeilepp:speedup-topos into 2726e11582cd050331aee795b77bbde592ce0db7 on mne-tools:master.

Coverage decreased (-0.004%) to 90.759% when pulling 913cade364e4d75df3f04f179ecf4aa71512af3e on jaeilepp:speedup-topos into 2726e11582cd050331aee795b77bbde592ce0db7 on mne-tools:master.

Ready for merge?

The tfr topo is a bit more narrow here than on master for some reason.

I was going to check it out, but I can also do that on another PR. So you can merge if you're happy.

How is plot_epochs_spectra broken? It seems alright to me.

Ahh, I see... It's not plotting the topo part.

Coverage increased (+8.7%) to 90.766% when pulling 9c95aaa1bfc17b6ededa056c8482ad388b119d98 on jaeilepp:speedup-topos into f237e4072a31d061a76a3ff55522343cf5002a78 on mne-tools:master.

Coverage increased (+8.8%) to 90.794% when pulling 64d7430a992fb718fee46194989722770f03bfdd on jaeilepp:speedup-topos into f237e4072a31d061a76a3ff55522343cf5002a78 on mne-tools:master.

Does this close #2953?

I ran examples/visualization/plot_topo_channel_epochs_image.py and when clicking on an image I get:
  File "/home/larsoner/custombuilds/mne-python/mne/viz/topo.py", line 226, in _plot_topo_onpick
    show_func(plt, ch_idx)
  File "/home/larsoner/custombuilds/mne-python/mne/viz/topo.py", line 601, in _erfimage_imshow
    ax.set_xlabel(x_label)
AttributeError: 'module' object has no attribute 'set_xlabel'

Seems I missed epoch image topo yesterday. It also seems that it was broken on master too. I added a test for it.
I noticed that you can't do epochs.plot_topo_image I'll add a method for it.

Does this close #2953?

Indeed it does.

Coverage decreased (-0.09%) to 81.942% when pulling 6d642912c6da6242e49f4d08256546da783d6623 on jaeilepp:speedup-topos into f237e4072a31d061a76a3ff55522343cf5002a78 on mne-tools:master.

Coverage decreased (-4.3%) to 77.734% when pulling f397b662b94503aa1ac1dc8d074f9b5c7e9e4050 on jaeilepp:speedup-topos into f237e4072a31d061a76a3ff55522343cf5002a78 on mne-tools:master.

Coverage increased (+8.7%) to 90.696% when pulling 0c5e33cd0395ebd78235781f25cfcc0af09bf03b on jaeilepp:speedup-topos into f237e4072a31d061a76a3ff55522343cf5002a78 on mne-tools:master.

Coverage increased (+8.8%) to 90.796% when pulling ea1d5dd2b53c93f6cbeb76edfd528b6513e64154 on jaeilepp:speedup-topos into f237e4072a31d061a76a3ff55522343cf5002a78 on mne-tools:master.

@kingjr your PR on cwt broke examples ...
plot_time_frequency_sensors.py gives now
/Users/alex/work/src/mne-python/mne/time_frequency/tfr.py in _cwt(X, Ws, mode, decim, use_fft)
    243                 if use_fft:
    244                     ret = _centered(ret, n_times)
--> 245                 tfr[ii, :] = ret[::decim]
    246         yield tfr
    247

ValueError: could not broadcast input array from shape (401) into shape (400)

Coverage increased (+8.7%) to 90.691% when pulling bb7c692b61a92fc1963c314d3ed5eb6f91272b9f on jaeilepp:speedup-topos into 95df1e0921a7d28f971cb0a8c7f15c8a426284be on mne-tools:master.

The rebased version (with the fix) works for me for time_frequency/plot_time_frequency_sensors.py, visualization/plot_topo_channel_epochs_image.py (other than the pre-existing xlabel issue), plot_cwt_sensor_connectivity.py, and visualization/plot_topography.py. +1 for merge from me

I take that back, I didn't update properly. With your latest changes, in plot_topo_channel_epochs_image.py I actually get:
/home/larsoner/custombuilds/mne-python/mne/viz/topo.py in plot_topo_image_epochs(epochs, layout, sigma, vmin, vmax, colorbar, order, cmap, layout_scale, title, scalings, border, fig_facecolor, font_color, show)
    689     for idx in range(epochs.info['nchan']):
    690         ch_type = channel_type(epochs.info, idx)
--> 691         scale_coeffs.append(scalings[ch_type])
    692     for epoch_data in data:
    693         epoch_data *= np.asarray(scale_coeffs)[:, np.newaxis]

Can you confirm?

on this example I get:
/Users/alex/work/src/mne-python/mne/viz/topo.pyc in
plot_topo_image_epochs(epochs, layout, sigma, vmin, vmax, colorbar, order,
cmap, layout_scale, title, scalings, border, fig_facecolor, font_color,
show)
689     for idx in range(epochs.info['nchan']):
690         ch_type = channel_type(epochs.info, idx)
--> 691         scale_coeffs.append(scalings[ch_type])
692     for epoch_data in data:
693         epoch_data *= np.asarray(scale_coeffs)[:, np.newaxis]
KeyError: 'stim'

Coverage decreased (-4.2%) to 77.737% when pulling 5d708cd1258899e8c713baa14af21c478c45434b on jaeilepp:speedup-topos into 95df1e0921a7d28f971cb0a8c7f15c8a426284be on mne-tools:master.

Coverage increased (+8.9%) to 90.802% when pulling bf9b4e4147dc2c664f88bd509980039264d16132 on jaeilepp:speedup-topos into 95df1e0921a7d28f971cb0a8c7f15c8a426284be on mne-tools:master.

Coverage decreased (-0.1%) to 90.696% when pulling 22c20e64558b843675f9d9c7f6a16a61c4f1ae4a on jaeilepp:speedup-topos into 4ea477bf6e876d353b270445b0fd8ffb48841d89 on mne-tools:master.

Coverage decreased (-8.8%) to 81.97% when pulling 882f6f348ec05934fde88c5710e82445931766f1 on jaeilepp:speedup-topos into 4ea477bf6e876d353b270445b0fd8ffb48841d89 on mne-tools:master.

Ready for review.

perfect thanks @jaeilepp !


Use packages.drupal.org composer repository in example

Followup #699. Also opened a PR at drupal-project. https://github.com/drupal-composer/drupal-project/pull/159

Platform.sh just announced they're defaulting to the new repository. I think it's time we do the same :)


Fix the clinicalEnrichment() using pathways

The previous version of clinicalEnrichment can only perform the enrichment of TP53 and MYC because the pathway names are gene symbols. Now the pathway enrichment can work as expected.

Hi,
Thanks for the PR and tracking the issue. Appreciate it.


getting *** marks in pipeline tasks

I'm trying to run a pipeline but not sure why the pipeline consists of **** marks in results page, it is trying to hide the values and I need the values for understanding the error or issues.
the issue comes in all the tasks and not any specific one.

Hi, this looks like a more general issue with a 3rd party task/secrets, not an issue with a task in this repo. Please direct more general issues to https://developercommunity.visualstudio.com/spaces/21/index.html


Undo (Ctrl + Z) hangs the kernel

Issue Type: Bug

Write any piece of code in a cell.
Try to UNDO.

Expected Result:
Undo works just fine.
Actual Result:
Goes into an endless loop.

Can't be stopped
Kernel restart button doesn't work

I have to manually reopen the file (effectively restart the kernel)

As a visual representation, in the screenshot below, I have tried to perform undo action.

Note. It's been 5 mins now!
-- Using the Preview Notebook Editor
Extension version: 2020.10.302853766-dev
VS Code version: Code - Insiders 1.51.0-insider (605bc4ec1eaf8dc1acd89a0c3c5743926ed0dc09, 2020-10-12T10:52:10.028Z)
OS version: Windows_NT x64 10.0.19041
Remote OS version: Linux x64 4.4.0-19041-Microsoft

@vpankaj97 Does this issue repro for you locally or only when working over remote (I noticed that you had a remote OS listed).
Also, could you please change the setting python.logging.level in VS Code to debug, repro the issue, then post the output from the Console: Python tab in a file here? That has some diagnostic info that could help us to diagnose this.

@IanMatthewHuff Yes, I was able to repro this locally too.
On the WSL side, I'm attaching the WSL python log.
Steps inside log file:

Started an .ipynb file
wrote a print command
Pressed Undo - It went on a loop
hard closed the file

PythonLog.txt

@vpankaj97 Got it. I'm still not reproing this. But I don't currently have a WSL setup. I'll try to get someone one my team with a windows / WSL setup to check this out.

Okay.
Just FYI, I'm on WSLv1 over ubuntu 18.04.
Python Daily channel and Pylance as the language server.
[The normal Notebook in python extension works fine.]

@joyceerhl can you take a look?

I can repro this in a regular notebook

I'm having the same problem, but I don't think the kernel hangs - it just doesn't seem to run at all even though the timer keeps spinning. I can easily delete the cell and run something else with the kernel still being intact.

Also confirming this issue. As @Trollgeir described, the kernel doesn't seem to stop, it's just the timer that keeps spinning. I have to press "ctrl+z" twice or so to cancel the timer.
OS: Win10 (no WSL connected)
Jupyter kernel: local
VS: Insider's version 1.51 + 2020.10.0-rc (14 October 2020)


Support generating comments

This would be absolutely amazing, the generated bindings could have documentation with odoc and using them in an editor would be much easier with the documentation on hover.

I discussed about this with the odoc team, and we may support it somewhere in the OCaml Platform in the future, and there seems to be no reason for us to do this right away.


No such file or directory in "copies.rb"

No such file or directory @ rb_sysopen - zold-node/zold-copies/0000000000000000/10
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/copies.rb:66:in `read'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/copies.rb:66:in `block in add'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/copies.rb:66:in `each'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/copies.rb:66:in `find'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/copies.rb:66:in `add'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/node/entrance.rb:74:in `push_unsafe'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/node/entrance.rb:53:in `block in push'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/node/entrance.rb:52:in `synchronize'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/node/entrance.rb:52:in `push'
	/var/lib/gems/2.3.0/gems/zold-0.9.11/lib/zold/node/front.rb:123:in `block in <class:Front>'

@rultor release, tag is 0.10.4

@rultor release, tag is 0.10.4

@yegor256 Done! FYI, the full log is here (took me 6min)

The job is not in WBS, won't close the order


[JAVA][DATASET][ver17.0.0] Support HDFS filesystem for Read/Write using Dataset API

Describe the bug, including details regarding any error messages, version, and platform.
I am using the Arrow Flight module to Version 17.0.0 , along with the
DataSet API to perform the read/write operations to persist the data
on the HDFS filesystem.
However I get below error message
Error message : "Got HDFS URI but Arrow compiled without HDFS support"
The per the documentation for V17.0.0 , its is mentioned HDFS support
is included Apache Arrow Java panache release
As per one of the post , I could see the Issue#37323 is never release
to any Arrow version
Please could you guide me if HDFS support is included , or I will java
to carry out any additional steps to get this working on my Hadoop
environment
Thanks
Atul S
Component(s)
Integration, Java

Apologies for typo
"The per the documentation for V17.0.0 , its is mentioned HDFS support
is included Apache Arrow Java package release "

It seems that it's not implemented yet: https://github.com/apache/arrow/issues/36703
Can we close this as a duplicated?
BTW, could you share the documentation URL you referred?

Thank you Kou for the response.
Please find below the documentation Link
https://arrow.apache.org/docs/java/dataset.html#read-data-from-hdfs
I understand this is duplicate and can be closed , just wanted to check if we see HDFS support to be enabled in upcoming release , if yes any timelines around the same ??
Appreciate your help on this

Thanks. I close this as a duplicated.
It seems that it was added by GH-10333. Nobody noticed that HDFS support isn't included yet...
The upcoming release will not include HDFS support because nobody works on this. Do you want to work on this?


Article tags field and serializer

Describe your changes :
I updated the articles tag field and added the tag serializer to the Article serializer

I worked on this as the final PR to solve #140

[ ] Bug fix
[x] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

[x] I have read the Code Of Conduct document.
[x]  I have read the CONTRIBUTING document.
[x]  I have performed a self-review of my own.
[x]  I have tagged my reviewers below.
[x]  I have commented my code, particularly in hard-to-understand areas.
[x]  My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x]  All new and existing tests passed.

Reviewers
Backend: @PraveenMalethia , @codyseibert

@MidouWebDev I can add or edit the project board


Fix add-grammar and convert-grammars

Since I want to replace grammar, I ran
./script/add-grammar --replace ruby-haml.tmbundle https://github.com/ezekg/language-haml.
As a result, I noticed that it was strange behavior.

Added new language-haml to grammars.yml, but there is also old ruby-haml.tmbundle in grammars.yml

The old ruby-haml.tmbundle line was deleted from vendor/README.md, but the new language-haml was not added to language-haml.

This PR fixes those behaviors by updating grammars.yml. This is why running script/convert-grammars after removing old one in add-grammar.
And the other commit makes script/convert-grammars working well with the path coded in grammars.yml didn't exist.

license files were not updated in #3355. I try to investigate it later.

I finally straighten it out.
add-grammar --replace does not update license files.
But it will do now with these commits.


Fix monorepo yarn maintenance

This PR fixes yarn.lock maintenance for monorepos, i.e. it's possible to have more than one yarn.lock file per repository to be maintained within the same branch/PR.
Closes #166

Codecov Report

Merging #167 into master will decrease coverage by 0.13%.
The diff coverage is 44.89%.

@@            Coverage Diff             @@
##           master     #167      +/-   ##
==========================================
- Coverage   68.72%   68.59%   -0.14%
==========================================
  Files          19       19
  Lines         937      952      +15
  Branches      156      161       +5
==========================================
+ Hits          644      653       +9
- Misses        293      299       +6

Impacted Files
Coverage Δ

lib/config/definitions.js
100% <ø> (ø)
:arrow_up:

lib/worker.js
69.34% <36.36%> (+6.42%)
:arrow_up:

lib/workers/branch.js
77.77% <47.36%> (-22.23%)
:arrow_down:

Continue to review full report at Codecov.

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

Awesome!


NTR: protein targeting to the basolateral/apical plasma membrane

Dear editors,
this paper PubMed=11802162, describes the function of AP4M1 in localization of several proteins to the basolateral membrane of epithelial cells. Moreover, it suggests it is signal-mediated. Could you please create the terms indicating the signal-mediated targeting of proteins to the apical and basolateral membrane, a child of GO:0006605 protein targeting.
Thanks,
Lionel

Hi Lionel,
I've taken a quick look at the paper you cite, and also the Mellman review cited therein that I quote below. I am inclined to agree with you that localization of proteins to the apical or basolateral membranes of epithelial cells is signal-mediated.
I see that we already have this term:
protein localization to basolateral plasma membrane (GO:1903361)
Def: A process in which a protein is transported to, or maintained in, a location within a basolateral plasma membrane.
is_a: 'protein localization'
This term does not specify that the localization is signal mediated, but after reading the Mellman review, I was wondering if this kind of localization occurs any other way. Do you know of any such localization or targeting that is NOT mediated by a signal?
thanks,
-Karen
Mellman I. Endocytosis and molecular sorting. Annu Rev Cell Dev Biol. 1996;12:575-625. Review. PubMed PMID: 8970738:

TARGETING SIGNALS CONTROL RECEPTOR RECYCLING IN POLARIZED EPITHELIAL CELLS
In polarized epithelial cells, receptors internalized at one plasma membrane domain typically recycle back to that domain. This has been clearly demonstrated in Madin-Darby canine kidney (MDCK) cells in which Tfn receptor and LDL receptor, both proteins restricted to the basolateral surface, are continuously internalized and recycled basolaterally with a high degree of fidelity (Mellman et al 1993b). Transport across the cell to the apical domain (transcytosis) is possible as well, although this normally occurs only in the case of specialized transcytotic receptors such as the polymeric Ig receptor (Mostov 1994). It is obvious from the study of epithelial cells that receptor recycling cannot be a default process, but must be governed by one or more targeting signals.

Hi Karen,
I do not know of such example but this does not preclude the existence of such mechanisms. In that case we clearly know it is signal-mediated but what should we do when there is no evidence of such targeting signals. That is why I thought to introduce such granularity would help. Moreover, the definition includes retention in the basolateral membrane of some proteins and it may include some non signal-mediated processes.
Hope it helps,
Lionel
Th

@pgaudet  has recently been overhauling so that 'targeting' terms were merged into localization because of the difficulties associated with 'targeting'
I think the plan was that targeting would be represented as
MF signal-sequence_binding involved_in   process,
which here might be:
MF signal-sequence_binding involved_in
GO:0099638    endosome to plasma membrane protein transport has_substrate (gene x)
if this is describing transport (active movement)  rather than merely 'localization' (which includes transport).

Based on the way the definition of the existing term is written, it looks to me like it is a "standard" definition that was written a long time ago, not necessarily based on much understanding of the biology.

protein localization to basolateral plasma membrane (GO:1903361)
Def: A process in which a protein is transported to, or maintained in, a location within a basolateral plasma membrane.

From the reviews I took a quick look at yesterday, it looks to me like the protein targeting to the basolateral/apical plasma membrane that occurs in epithelial cells is a specialized process that occurs in these polarized cells. Whichever parentage is appropriate, we definitely do need to have a parallel term for the apical targeting, which we don't have at all, and I think the definitions could be improved. So far, I'm not sure that we need a separate term for signal mediated since it seems that all of it is signal mediated. Perhaps I'll check with some of the review authors.
@ValWood - Thanks for tagging @pgaudet on this, I was planning to get her input. From what I've read so far, this is definitely an active targeting, both for the original localization to the appropriate membrane and also during endocytic recycling.

Hello,
@lbreuza Look like the paper you are annotating is from 2002; is there more recent data on this? That might help us create the terms correctly. AP4M1 is involved in clathrin-coated vesicles transport; don't you think this is the likely mechanism here ? Is there any data linking clathrin-coated vesicle transport to protein localization to basolateral plasma membrane?
We also have 'regulation' of these terms (for eg 'regulation of protein localization to basolateral plasma membrane'). I don't think this is optimal but it could be a solution for now. Otherwise it might be relevant to create 'signaling involved in protein localization ...' but without the molecular evidence that seems unjustified.
@krchristie
indeed the plan was something along what you write:
MF signal-sequence_binding involved_in
GO:0099638 endosome to plasma membrane protein transport has_substrate (gene x)
Thanks, Pascale

@pgaudet AP4M1  is involved in vesicle transport (but not clathrin-dependent); also, the paper clearly shows that AP4M1 is specifically involved in targeting of proteins to the basolateral plasma membrane, ie by recognition of a targeting motif on transported proteins.

@krchristie
after discussion with Lionel I suggest you create the term 'protein targeting to the basolateral/apical plasma membrane' for now (until we have time to fix the branch).
Thanks, Pascale

@pgaudet
Having scanned through a number of the papers used for annotation, as well as a few reviews, I think that there is only one process going on for sorting/recycling to apical or basolateral plasma membrane. Many of the annotated genes are vertebrate proteins with polarized epithelium, but even looking at the Aspergillus and Arabidopsis proteins annotated to 'apical protein localization', this seems like a conserved process for targeting to specific regions of membrane.
Considering that you are proposing to merge some of the existing basolateral terms in https://github.com/geneontology/go-ontology/issues/14233, I really don't see why we would want to create any more terms to represent processes we already represent. To me it seems best to use existing terms and fix them to more clearly represent what we are already using them for. These two seem to be the main terms:
GO:0045176 - apical protein localization (164 annotations)
GO:1903361 protein localization to basolateral plasma membrane (165 annotations)

I don't think there's anything left to be done here, so I'm going to close this.


WIP: use storage API

WIP!!!
This PR changes the codebase to use the storage API for writing and reading files. This is needed for portable installs, RO deployments etc.
Fixes #2585

@nws would be happy for a quick check

There is still something missing: secret key storage. I am not sure if it is safe to put that onto media as that might as well be S3

This was just PoC to start a discussion; without an active user for reviews / code submissions this will go nowhere


Hosting on Gandi Simple Hosting

Hello,
I'm trying to deploy my app made with your book support (very good book) on Gandi Simple Hosting
Gandi simple hosting needs a wsgi.py which point the app.
https://wiki.gandi.net/en/simple/instance/python
I don't understand how I will use the manage.py in this context.
Thanks!

You can create a simple wsgi.py file as follows:
from app import create_app
application = create_app(os.getenv('FLASK_CONFIG') or 'default')

But unfortunately, this platform does not seem to allow to run custom tasks, such as the manage.py deploy task that creates or upgrades the database. You need to ask them how that can be achieved.

Thank you, it works!
For custom tasks, there is an ssh access which permits me to create and upgrade a sqlite database via manage.py.
The here is also a postgresgl database on the virtual host... I'll come back to close my issue when I have succeeded in linking my app to this database.
Thanks

Finally, I think I'm going to deploy it on a vps which seems more configurable than this simple hosting...


bug: 'primary_key' is allowed by sink types other than 'upsert'

Describe the bug
create sink invalid_pk_sink from t with (connector = 'blackhole', primary_key = 'v1');

To Reproduce
As above.
Expected behavior
We should only allow primary_key when upsert is enabled.
Additional context
No response

As far as I can tell from digging into the actual code implementation, there are 4 possible states of type =  for a sink today:

upsert, which requires primary key.
Unspecified or specified to an unrecognized value. This is the case for the example statement above (blackhole), as well as elasticsearch sink. It will actually default the SinkType enum to upsert (src). This is likely a bug and we would require format xxx soon under the new syntax. We need to clarify the actual behavior, connector by connector, for example:

blackhole seems to act as if append-only, when unspecified. Then primary_key is unnecessary.
elasticsearch seems to act as if upsert, when unspecificed. Then primary_key should actually be required.

debezium, which also requires primary key (src). Due to the same logic above, its SinkType is still upsert.
append-only. In the current implementation, providing primary_key does make a difference:

Without primary_key, the all kafka messages would have key = {} (empty json object) and they would be sent to same kafka partition, leading to data skew.
With primary_key, the values of these columns would be included in kafka key, which is then hashed to decide which kafka partition a message should write to.
Practically, empty primary_key is allowed but is bad practice right now. We shall first fix the behavior to round-robin before disallowing primary_key here. Or do we still want to give user the ability to select partition based on some columns, even when the sink is append-only?

@neverchanje @tabVersion Could you help clarify what should we do in each cases?

Here are my thoughts.

connector = 'es' only works with format upsert encode native, and I think it always works in an upsert way so primary_key is required.
connector = 'blackhole' does nothing (works with format append_only encode native). I am ok with seeing it as append-only. So we should reject primary_key in this case, jusr for blackhole sink.
connector = mq and format debezium encode json/avro/protobuf is another case of upsert and primary_key is required.

Overall, we are going to require primary_key for upsert and debezium format but I want to make it optional for append-only format. I've seen requirements that user just wants to partition the data by some column and we can offer the  ability without effort.

Without primary_key, the all kafka messages would have key = {} (empty json object) and they would be sent to same kafka partition, leading to data skew.

Not true, Kafka will use Round Robin for these messages and I don't think there is data skew.

Not true, Kafka will use Round Robin for these messages and I don't think there is data skew.

Kafka would use Round Robin when key is unset. But our current implementation always sets key = {} (2 bytes). This is a minor implementation bug we need to fix.

Not true, Kafka will use Round Robin for these messages and I don't think there is data skew.

Kafka would use Round Robin when key is unset. But our current implementation always sets key = {} (2 bytes). This is a minor implementation bug we need to fix.

OMG, that's a bug indeed.

Seems to be a misunderstanding because #9768 was not done.

Not true, Kafka will use Round Robin for these messages and I don't think there is data skew.

Kafka would use Round Robin when key is unset. But our current implementation always sets key = {} (2 bytes). This is a minor implementation bug we need to fix.

OMG, that's a bug indeed.

Let's follow this?
https://github.com/apache/kafka/blob/f1e58a35d7aebbe72844faf3e5019d9aa7a85e4a/clients/src/main/java/org/apache/kafka/clients/producer/internals/DefaultPartitioner.java#L28-L32

Seems nothing else to do


[Question] Stuck between master and RC Beta branch

Hi,
I'm completely clueless on how to find a way to get to work with the MRTK and Hololens.
I finally got to work my dev env ( meaning VS17 or Unity + MRTK managin both to deploy and build the app).
BUT ! When I use the master branch with Unity 2017.1.1.f1 I'm stuck with
Error! RenderTexture.GenerateMips failed: render texture does not have mip maps (set useMipMap to true).
that's freaking annoying because it floods my console a lot and could even lower performances.
On the other hand, with Unity 2017.2.0f1 (RC) and beta branch, I'm being told my target device ( Hololens isn't up to date)  DEP3321: To deploy this application, your deployment target should be running Windows Universal Runtime version 10.0.16288.0 or higher. You currently are running version 10.0.14393.1715. Please update your OS, or change your deployment target to a device with the appropriate version.	GTS_Demo
I'm already running Insider on it and it's up to date :(
(Latest SDK and insider update installed on the dev computer)
And finally, master branch MRTK with a beta Unity doesn't work, as I understand API changed a lot.
I know this is not bug related but I'm starting to loose hope on finding a stable dev environment...  Plus Unity forums and  Microsoft Hololens forums never have shown very helpful
Thanks a lot in advance !

Hey Andre,
The generate mipmaps error is benign, you can ignore it, although like you mentioned, it fills the logs and is annoying.
The issue you are running into with 2017.2 is a pretty easy fix.  On the unity build settings window, you probably have the build target set to "latest installed" where as HoloLens is at version 15063. If you set that to match, it should solve that problem.

Holy cow, it's finally working ! thanks a lot @jessemcculloch !
I'm a bit confused, isn't the latest SDK (16 something) needed to run the beta MRTK ?

Yes and no. It needs to be installed for some of the API's for the Immersive Headsets and Motion Controllers to compile. But the OS on the HoloLens itself is only at 15063.

Noted !
But as I expected something else doesn't work.
I'm attempting to build&run the SpatialUnderstanding scene. But it crashed my Unity or throws an exception in VS or crashes on Hololens. Is it an known issue ? I can't get to work any spatial visualization since this morning ^^"

Did you grab the pre-release of Dev_Unity_2017.2 from the releases page, or did you just download the raw code from the branch this morning?

My bad, forgot to answer.
Yup I did :)
And there is more I never managed to get to work SpatialMapping from MRTK, even in the 'old' version.
I check drawVisualMeshes but never show a thing ( So I used the Unity vanilla component )


about the training details

Thank you for your work; I've found the results to be significantly better than those from GFPGAN 1.4. I am attempting to reproduce your results but have noticed some discrepancies.
In config.py, should self.crop_components = False actually be set to self.crop_components = True?
Regarding the instruction "train until you think it is okay," how can I be certain that it is indeed okay?

I am also looking for an answer @LeslieZhoa


90% Test coverage in ambry.cli

Expand the tests in test_warehouse to produce a minimum of 90% test coverage for each of the files in the directory ambry.cli. Develop these tests in tests/test_cli.py
NOTE: Some of this work will involve improving tests, some will involve removing code. Be sure to ask if it looks like code is old, useless or never used.

I'll take it

Ok, you can assign yourself to issues to claim them.


Strip trailing whitespace for scm fetch_revision

A recent change to SSHKit means that capture no longer strips trailing whitespace. This affects the behavior of fetch_revision, such that the returned revision now includes a newline.
This PR restores the previous behavior of fetch_revision by explicitly calling strip on the captured output.

Thanks @mattbrictson living dangerously on SSHKit master I see :)

@mattbrictson , the build failed but it looks like an internal error from Travis (segmentation fault whilst installing Gherkin) - https://travis-ci.org/capistrano/capistrano/jobs/60981262 I've re-triggered the build.

(Travis hiccough apparently)


SQLite Semantic Search Example?

Hi,
Is there an example of integrating semantic search into an SQLite database?
Thank you!

There are quite a few examples that do this - https://neuml.github.io/txtai/examples/

Thx!


[WEBSITE-836] Switch to data platform API

Added Accept header to request in get method to set them to application/n-triples.

Coverage remained the same at 100.0% when pulling c3dfbf0f7a5e4cba621cabdee4e4642e03cb6e2b on katylouise:katylouise/website-836_switch-api into 5b8f1c9a96db0f7be587be56e45d7ddc9a1ca0fd on ukparliament:master.

Coverage remained the same at 100.0% when pulling c3dfbf0f7a5e4cba621cabdee4e4642e03cb6e2b on katylouise:katylouise/website-836_switch-api into 5b8f1c9a96db0f7be587be56e45d7ddc9a1ca0fd on ukparliament:master.

Coverage remained the same at 100.0% when pulling a99c04b4e09fd231359156c1cabc0d91d8aadb15 on katylouise:katylouise/website-836_switch-api into 5b8f1c9a96db0f7be587be56e45d7ddc9a1ca0fd on ukparliament:master.

Coverage remained the same at 100.0% when pulling a99c04b4e09fd231359156c1cabc0d91d8aadb15 on katylouise:katylouise/website-836_switch-api into 5b8f1c9a96db0f7be587be56e45d7ddc9a1ca0fd on ukparliament:master.


Translation to Czech

Hello, is it possible to translate APP and missions to Czech language? I have tried to do it on Windows using the app, but my main OS is MacOS and I cannot find a way how to import translated files.

Hello! Translations are handled by the community, so if someone is able and willing to do it, we can absolutely integrate it into the app. Most of the time, people do a few translations at a time.  Just doing the UI is a good starting point, then maybe the tutorials.  This is a nice way of getting to know how to do it. When you're done with the files, just send them to me, and I will build them into the app.


UglifyCSS looses parts of the css when there's a comment ending with '.../*/'

Hi,
I had a strange issue, where uglifycss lost some parts of my css while uglifying. Through trial and error I was able to pin it down a bit.
In some of my CSS I have comments that are "marked" like this:

/*//////////////////////////////////////////////*/
/* Some interesting bla bla bla */

With the latest version of uglifycss (0.0.15) some parts of the css will be lost. It worked with an older version, I only update a few weeks ago.
I found out, that a slash following and asterik and another slash at the end of a comment => .../*/ or /*///   ///*/ causes the problems.
I think uglify should handle this better or throw an error indicating that it appears to be a nested comment. See also http://www.w3.org/TR/CSS21/syndata.html#comments
Cheers,
Dominik

UglifyCSS is not a parser so it hardly detects this kind of construct. I'm going to add a warning at the beginning of README.md about using only valid CSS as input.
Maybe you could use a tool like [csslint.net] (https://github.com/CSSLint/csslint) in your build process.
Thanks for your report.

Hi Franck,
thanx for your answer.
The point is having this ("///////") in the css IS valid according to the
w3c-link I posted in my issue report.
While this bug/issue persists in uglifycss, adding a comment to the readme
is surely a good idea.
Thanx for the tip with csslint. I'll take a look at it.
Cheers,
Dominik
Am 05.07.2015 15:32 schrieb "Franck Marcia"<EMAIL_ADDRESS>
UglifyCSS is not a parser so it hardly detects this kind of construct. I'm
going to add a warning at the beginning of README.md about using only valid
CSS as input.
Maybe you could use a tool like csslint.net
https://github.com/CSSLint/csslint in your build process.
Thanks for your report.
—
Reply to this email directly or view it on GitHub
https://github.com/fmarcia/UglifyCSS/issues/33#issuecomment-118619858.


Graphql error: Cannot read properties of undefined on DELETE sql command

When I try to run this statement:
const r = await app.platformatic.entities.pokemon.delete({ skipAuth: false, where: { id: { eq: 3 } } })

If there is a match, I get this error:
Cannot read properties of undefined (reading 'reply')
    reqId: "req-4"
    err: {
      "type": "GraphQLError",
      "message": "Cannot read properties of undefined (reading 'reply')",
      "stack":
          TypeError: Cannot read properties of undefined (reading 'reply')
              at getPublishTopic (/pokedex/node_modules/@platformatic/db-authorization/index.js:306:36)
              at /pokedex/node_modules/@platformatic/sql-events/index.js:64:38
              at Array.map (<anonymous>)
              at /pokedex/node_modules/@platformatic/sql-events/index.js:63:31
              at async generations (/pokedex/plugin-generations.js:21:21)
              at async Promise.all (index 0)
              at async Object.fastifyGraphQl [as graphql] (/pokedex/node_modules/mercurius/index.js:556:23)
      "path": [
        "generations"
      ],
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "extensions": {}
    }

Checking the code and adding a console.log I get:
        async getPublishTopic (original, opts) {
        console.log({ opts })

// output:
{
  opts: {
    action: 'delete',
    data: {
      id: '1',
      name: 'bulbasaur',
      height: 7,
      weight: 69,
      generation: 1,
      isBaby: 0,
      isLegendary: 0,
      isMythical: 0,
      colorId: 5,
      evolutionChainId: 1
    },
    ctx: undefined
  }
}

To replicate you need to checkout this PR: https://github.com/Eomm/pokedex/pull/2
and modify the plugin-generations.js by adding the statement await app.platformatic.entities.pokemon.delete({ skipAuth: false, where: { id: { eq: 3 } } })
I was testing that the app.platformatic.entities.pokemon.delete blocks the anonymous user from performing such actions as documented https://oss.platformatic.dev/docs/reference/db/authorization/rules

So this is expected but we should have a better error message / improve the API.
The code above is asking for skipAuth: false, i.e. to look at the current user from the reply to validate if it has permissions to perform the action, but you are not passing it in.
This would partially be solved by #669, so you get an API to generate the context to pass around.
However, I propose we remove skipAuth completely.
@marcopiraccini @ivan-tymoshenko wdyt?

Some history on this:

We added skipAuth  because before the behavior was to use authorizations automatically also with custom plugins. We added this explicit skip option.
Then we decided to remove auth if ctx is not set (so skipping auth automatically)

So it seems now skipAuth is useless indeed, because we can obtain the same effect not passing the context. However, I would like a more "explicit" way to control this (this is why we added an explicit skipAuth at the beginning).

However, I would like a more "explicit" way to control this (this is why we added an explicit skipAuth at the beginning).

What would that look like?

If a authorization is in place, I think we should always check it at the entities level UNLESS we specify to not to. This is why we added skipAuth, and I think this is the correct way (because explicit). So IMO we should:

Fix the error above, authorization can ONLY be disabled, not forced. In other words, no skipAuth: false, we might choose a different name or way to configure this, but must be set programmatically.
If we have a ctx, authorizations are applied, UNLESS explicitly skipped.

That's great.

ASSIGN TO ME.👍


write a program in python to print fabonacci series

assign this to me

Hello @chandantiwari1, I'd love to work on this.
Can you please assign this to me?


added NNX init_cache docstring example

added NNX init_cache docstring example

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (182afc0) 56.13% compared to head (c257e9e) 56.13%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3688   +/-   ##
=======================================
  Coverage   56.13%   56.13%
=======================================
  Files         102      102
  Lines       12220    12220
=======================================
  Hits         6860     6860
  Misses       5360     5360

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add the Data Collections Explorer

Add the Data Collections Explorer to the overview on the homepage. This takes care of one of the tasks mentioned in Issue #12.

Sorry, I haven't seen that I was picked for review somehow. Did you discuss with Rainer or Thomas if we plan to directly link to other pages? The other ones do have an extra page with documentation etc.


Post release automated changes for policy releases

Post release automated changes for azure-arm-policy

API change check
API changes are not detected in this pull request.


card size

How can I modify size of the card?

Hello @paritoshk
You can inspect the card, find the right class and add a css attribute to, it.
But the feature is not implemented for the moment in the library, feel free to fork it.


Correct Path Mapping Issue

So, I'm trying to map to the same directory that CWA is using (/mnt/user/media/books/import/) - see below for my compose.  Doesn't seem to be working correct.  Book downloads, is available, and I can actually grab it using the download link in CWA BD, but...it's not in /mnt/user/media/books/import (or imported by CWA and then deleted).
# If you change the FLASK_PORT, do not forget to change it in ports and healthcheck as well.
services:
  calibre-web-automated-book-downloader:
    container_name: CWA-Downloader
    image: ghcr.io/calibrain/calibre-web-automated-book-downloader:latest
    environment:
      FLASK_PORT: 9084
      FLASK_DEBUG: false
      CLOUDFLARE_PROXY_URL: http://cloudflarebypassforscraping:8000
      INGEST_DIR: /mnt/user/media/books/import
      BOOK_LANGUAGE: en
    ports:
      - 9084:9084
    healthcheck:
      test: ["CMD", "curl", "-f", "http://localhost:9084/request/api/status"]
      interval: 30s
      timeout: 30s
      retries: 3
      start_period: 5s
    restart: unless-stopped
    volumes:
    # This is where the books will be downloaded to, usually it would be
    # the same as whatever you gave in "calibre-web-automated"
      - /mnt/user/media/books/import:${INGEST_DIR:-/cwa-book-ingest}

  cloudflarebypassforscraping:
    container_name: CWA-Downloader-CFBypass
    image: ghcr.io/sarperavci/cloudflarebypassforscraping:latest
    restart: unless-stopped

Oh, this is my fault, I was mistaken in how docker compose works
Please keep the default value of Ingest dir and it should work
I will remove INGEST_DIR from the configuration on next release it does not make sense to be configurable, since its only used inside the docker and you can just mount it anywhere
Basically try this :
services:
  calibre-web-automated-book-downloader:
    container_name: CWA-Downloader
    image: ghcr.io/calibrain/calibre-web-automated-book-downloader:latest
    environment:
      FLASK_PORT: 9084
      FLASK_DEBUG: false
      CLOUDFLARE_PROXY_URL: http://cloudflarebypassforscraping:8000
      INGEST_DIR: /cwa-book-ingest
      BOOK_LANGUAGE: en
    ports:
      - 9084:9084
    healthcheck:
      test: ["CMD", "curl", "-f", "http://localhost:9084/request/api/status"]
      interval: 30s
      timeout: 30s
      retries: 3
      start_period: 5s
    restart: unless-stopped
    volumes:
    # This is where the books will be downloaded to, usually it would be
    # the same as whatever you gave in "calibre-web-automated"
      - /mnt/user/media/books/import:/cwa-book-ingest

  cloudflarebypassforscraping:
    container_name: CWA-Downloader-CFBypass
    image: ghcr.io/sarperavci/cloudflarebypassforscraping:latest
    restart: unless-stopped

That fixed it - figured it was something to do with the container mapping given that's what you modified for the ports! :)  Thanks!  Great work on this as well!


add compatibility with React 15.x

Using ReactDOM.unstable_renderIntoContainer (and losing the ability to connect to its parent?)

fixed in 1a8b1346d2879f471de6eed6796710b0e274a5ea


Querying vertex hangs

For one of the vertices in our DB, when we query for its edges, the DB server hangs:
edges = vertex.getEdges(Direction.IN, incomingProperty.getEdgeLabel());
for (Edge edge : edges) {
...
}
Here is the thread dump:
at sun.misc.Unsafe.arrayBaseOffset(Native Method)
at com.orientechnologies.common.directmemory.OUnsafeMemoryJava7.get(OUnsafeMemoryJava7.java:31)
at com.orientechnologies.common.directmemory.ODirectMemoryPointer.get(ODirectMemoryPointer.java:79)
at com.orientechnologies.orient.core.serialization.serializer.binary.impl.OLinkSerializer.deserializeFromDirectMemoryObject(OLinkSerializer.java:108)
at com.orientechnologies.orient.core.serialization.serializer.binary.impl.OLinkSerializer.deserializeFromDirectMemoryObject(OLinkSerializer.java:43)
at com.orientechnologies.orient.core.storage.impl.local.paginated.base.ODurablePage.deserializeFromDirectMemory(ODurablePage.java:127)
at com.orientechnologies.orient.core.index.sbtreebonsai.local.OSBTreeBonsaiBucket.getKey(OSBTreeBonsaiBucket.java:285)
at com.orientechnologies.orient.core.index.sbtreebonsai.local.OSBTreeBonsaiBucket.find(OSBTreeBonsaiBucket.java:203)
at com.orientechnologies.orient.core.index.sbtreebonsai.local.OSBTreeBonsaiLocal.findBucket(OSBTreeBonsaiLocal.java:1225)
at com.orientechnologies.orient.core.index.sbtreebonsai.local.OSBTreeBonsaiLocal.loadEntriesMajor(OSBTreeBonsaiLocal.java:729)
at com.orientechnologies.orient.core.db.record.ridbag.sbtree.OSBTreeRidBag$SBTreeMapEntryIterator.prefetchData(OSBTreeRidBag.java:458)
at com.orientechnologies.orient.core.db.record.ridbag.sbtree.OSBTreeRidBag$SBTreeMapEntryIterator.next(OSBTreeRidBag.java:440)
at com.orientechnologies.orient.core.db.record.ridbag.sbtree.OSBTreeRidBag$SBTreeMapEntryIterator.next(OSBTreeRidBag.java:420)
at com.orientechnologies.orient.core.db.record.ridbag.sbtree.OSBTreeRidBag.nextChangedNotRemovedSBTreeEntry(OSBTreeRidBag.java:1011)
at com.orientechnologies.orient.core.db.record.ridbag.sbtree.OSBTreeRidBag.access$200(OSBTreeRidBag.java:57)
at com.orientechnologies.orient.core.db.record.ridbag.sbtree.OSBTreeRidBag$RIDBagIterator.next(OSBTreeRidBag.java:329)
at com.orientechnologies.orient.core.db.record.ridbag.sbtree.OSBTreeRidBag$RIDBagIterator.next(OSBTreeRidBag.java:253)
at com.orientechnologies.orient.core.iterator.OLazyWrapperIterator.hasNext(OLazyWrapperIterator.java:89)
at com.orientechnologies.common.collection.OMultiCollectionIterator.hasNext(OMultiCollectionIterator.java:80)

The same happens when we query this vertex from the console:
select from 19:2274
I'll send the DB sample via e-mail.

Which version do you use ?

2.1-rc3

Hi @ylopotun i got your database and i'm investigating for the  root cause, but anyway checking the data of that record i saw that it didn't have any edged linking in, so i think you can workaround removing the property.
I'll send you by mail the specific detail.

Hi @ylopotun,
we finally found the main problem, created the relative tests and fixed, we are going to release a RC in the next hours with the fix, we created also and additional tool that check for eventual more case with ridbag problem and fix it, you can invoke it using "repair database --fix-ridbags" opening the database with the console in plocal.
closing


Add update interval for progress bar

Adds an optional update_interval for progress bars that allows the user to skip unnecessary renders for frequently updated iterators.

Fixes #676

Checklist:

[x] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[x] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[x] Add .. versionchanged:: entries in any relevant code docs.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

This is a good PR, but it's making me realize we're just slowly reimplementing tqdm without the design and focus of that library. I'll merge this, but I think I'm going to close any other progress bar feature requests.


Limited to database set in config file

Currently the database config file contains the name of the database name and the tool only works on this database meaning that a duplicate config file is needed when working on a database with a different name.
The cli should accept an optional database name that will take precidence over the database name set in the config file.

Resolved in cd33da6


How to set range value dynamically

My datepicker in the template is defined as follows:
<input matInput [placeholder]="'select date here"
                           [satDatepicker]="picker" readonly
                           [ngModel]="myParams.dateRange" #dateRange="ngModel" autocomplete="off"
                           (dateInput)="onDateFilterChange(dateRange.value)" (click)="picker.open()">

In the code I try to update the datepicker dynamically with:
            this.myParams.dateRange.begin = new Date(2018, 7, 5);
            this.myParams.dateRange.end = new Date(2018, 7, 25);

The datepicker only shows the dates, when clicked for the first time.
How can I debug this? Whats the problem here? Why is the range not shown?

See examples of angular material datepicker.  this.myParams.dateRange = {begin: date, end: date}

You mean this? https://stackblitz.com/edit/angular-4cfnyl
I did, but still no clue and isn't this: this.myParams.dateRange = {begin: date, end: date} just the same like
this.myParams.dateRange.begin = new Date(2018, 7, 5);
this.myParams.dateRange.end = new Date(2018, 7, 25);

Because you are editing the same object and angular thinks that value has not changed.

YES! Thats the issue! Thank you very much! Spend hours on that one...
Using:
this.myParams.dateRange = {begin: new Date(startMilliseconds), end: new Date(endMilliseconds)}
instead of:
this.myParams.dateRange.begin = new Date(startMilliseconds);
this.myParams.dateRange.end = new Date(endMilliseconds);

fixed it.


Full screen menus and timeline bar is too large.

can we change the full screen menus for zoom out 69%?
because the time line and stuff is too large for a 1080p screen
by dafault

by zooming out 67%

It's already see-through and it goes away after a short while.. Doesn't seem like a problem at all.
Also, #Linus4lyf

it is too big on the screen and annoying, that's the whole point. i wish it was not that big.


Updated to Ruby 2.6.6

A few version bumps in the Gemfile as well as the bump in ruby.  Some of
this work is to get ready for the eventual upgrade to Rails 6.0 as well
as keeping things up to date.
Also pegged the version of things in the stub/api/Gemfile as they were
now pull things for Rails 6 and as a result pre-commit was failing.
We should probably audit the version of things in the Gemfile.

Seems there still more to do here
https://github.com/alphagov/verify-infrastructure-config/blob/339dab2/pipelines/deployer/deploy-verify-hub.yml#L1002
https://github.com/alphagov/verify-infrastructure-config/blob/339dab2/pipelines/deployer/deploy-verify-hub.yml#L1107#


Support relative URLs in readme-reader.js and code-edit-view.html

Currently all URLs in exhibits need to be hard coded as absolute references. It would be expedient for future flexibility to support relative URLs in both readme-reader.js and code-edit-view.html.

@paulmasson

Currently all URLs in exhibits need to be hard coded as absolute references.

It might be more accurate to state: Currently all URLs in exhibits need to be hard coded as absolute references - if they are to operate correctly in iframes with readme-reader or code-edit-view as parents.

It would be expedient for future flexibility to support relative URLs in both readme-reader.js and code-edit-view.html.

Agree this would be nice, but it may be difficult to accomplish.
An interim measure might be to make scripts that are intended to be embedded as children in iframes to compensate for being embedded.
I have updated Star Spotter with an example of code that seems to work:
var pathSprites = ( window.self === window.top ) ?  './sprites/' : '../../exhibits/astronomy/star-spotter/sprites/';

https://github.com/exploratoria/exploratoria.github.io/blob/master/exhibits/astronomy/star-spotter/star-spotter.html#L19-21
The should work with both readme-reader abd code-edit-view as thay are both in the same relative URL hierarchy

That's an interesting approach, but it requires the coder to be aware of too much structure of the website. I was hoping to make the embedding process as transparent as possible.
One way to do this would be by creating an anchor and using its DOM properties to turn a relative reference into an absolute reference. This absolute reference could then be used to add a  tag to the document header before passing the document to the showdown library. Presumably showdown can handle relative references as long as the  tag is included.
It does occur to me though that if we have set up the site to allow download of code, then relative URLs could be a bad idea. While it makes some things simpler for us, the downloaded file would reference things that don't exist on the end user's computer. The end user still needs an absolute reference to classes and data for the downloaded page to function.
We should probably just keep absolute references after all. Thoughts?

While it makes some things simpler for us, the downloaded file would
reference things that don't exist on the end user's computer. The end user
still needs an absolute reference to classes and data for the downloaded
page to function.

Bingo.
There may well be issues with CORS and some files. But at the least, the
user can click on the link and save the data to their disk.

We should probably just keep absolute references after all. Thoughts?

How about a rule that says "absolute references are recommended for items
that would otherwise have issues with loading into Code-Edit-View"?
On Sun, Sep 20, 2015 at 3:08 PM, paulmasson<EMAIL_ADDRESS>wrote:

That's an interesting approach, but it requires the coder to be aware of
too much structure of the website. I was hoping to make the embedding
process as transparent as possible.
One way to do this would be by creating an anchor and using its DOM
properties to turn a relative reference into an absolute reference. This
absolute reference could then be used to add a tag to the document header
before passing the document to the showdown library. Presumably showdown
can handle relative references as long as the tag is included.
It does occur to me though that if we have set up the site to allow
download of code, then relative URLs could be a bad idea. While it makes
some things simpler for us, the downloaded file would reference things that
don't exist on the end user's computer. The end user still needs an
absolute reference to classes and data for the downloaded page to function.
We should probably just keep absolute references after all. Thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/exploratoria/exploratoria.github.io/issues/5#issuecomment-141840619
.

Sounds like a good idea. Added to notes.md file.
Would you like to revert the changes to Star Spotter or shall I?

I'm working on parametric equations for the next few hours. So go ahead
On Sun, Sep 20, 2015 at 4:16 PM, paulmasson<EMAIL_ADDRESS>wrote:

Sounds like a good idea. Added to notes.md file.
Would you like to revert the changes to Star Spotter or shall I?
—
Reply to this email directly or view it on GitHub
https://github.com/exploratoria/exploratoria.github.io/issues/5#issuecomment-141844480
.

Will do. Thanks!


Bring in the UI work

PR for UI work from @marksoper.
To build the UI code run
 INCLUDE_UI=true make

Merging this change in. From here on, UI changes should be incremental small PRs.

Fixes https://github.com/vmware/docker-volume-vsphere/issues/637


[TIKA-4309] Support MachO

Hm, clearly there's a conflict with
https://github.com/apache/tika/blob/30e110a81b52018654cf4b99777f91a38a771602/tika-core/src/main/resources/org/apache/tika/mime/tika-mimetypes.xml#L427-L438
and, by extent, with
https://github.com/apache/tika/blob/30e110a81b52018654cf4b99777f91a38a771602/tika-core/src/main/resources/org/apache/tika/mime/tika-mimetypes.xml#L417-L425
Also, I'm uncertain how to handle multi-arch executables, except for not returning archs at all.
And I'm at loss how to make the tests pass since the multi-package change. Need help

cc @Gagravarr as author of the related change

Apologies if you've already figured this out, but the way the above work is that if cafebabe matches on offset 0 and then there's a match at 4096, then tika identifies the file as x-java-jnilib. At a lower priority, if there's just a match on cafebabe, then the file is identified as java-vm.

While I did figure that part out, I didn't figure out how to resolve the conflict with ExecutableParser, so I still need help there @tballison :)

The magic for MachO is cefaedfe (32 bit) or cffaedfe (64 bit) as the first four bytes?
It looks like you've coded the magic for fat MachO as cafebabe. Is there something more distinctive for fat MachO? This might be useful: https://github.com/gabriel-vasile/mimetype/pull/53/files#diff-db84ad344d2bf91ea9533dbaa5974e81940b053644665471594fb2eedd34e7daR19 ?

 I'm not very familiar with this file type, and I'm happy for a "no!"

The magic for MachO is cefaedfe (32 bit) or cffaedfe (64 bit) as the first four bytes?

/* Constant for the magic field of the mach_header_64 (64-bit architectures) */
#define MH_MAGIC_64 0xfeedfacf /* the 64-bit mach magic number */
#define MH_CIGAM_64 0xcffaedfe /* NXSwapInt(MH_MAGIC_64) */

and
/* Constant for the magic field of the mach_header (32-bit architectures) */
#define	MH_MAGIC	0xfeedface	/* the mach magic number */
#define MH_CIGAM	0xcefaedfe	/* NXSwapInt(MH_MAGIC) */

Should we treat a fat machO file like a container file and parse its individual components as separate files? I'm not very familiar with this file type, and I'm happy for a "no!"

It's truly a container, and we can do that - a link to an example would be helpful :) In test files, it contains two separate "almost-mach-o" blobs

While I did figure that part out, I didn't figure out how to resolve the conflict with ExecutableParser, so I still need help there @tballison :)

Having not thought deeply about this, one option is to leave the mime file as is and add a magic for fat machO that's different from the other cafebabes in the mime type file -- we need to confirm the whatever makes fat machO distinct
...

we need to confirm the whatever makes fat machO distinct

I thought about that as well, we can read entire header and each arch header to confirm what we're looking at. Why I paused - even if I read, it makes no sense as we can return only one content type.

Y, that makes sense. If we treat it as a container, file though, we could make up/find a mime type for fatmachO (application/x-fat-mach-o I just made that up...is there an existing mime type?) and then the attachments/embedded files would have their own correct mime types.

This offers an example of how to parse attachments as separate files: https://github.com/apache/tika/blob/main/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-pkg-module/src/main/java/org/apache/tika/parser/pkg/PackageParser.java#L386

Doh, it looks like pronom had an entry for 32bit: https://www.nationalarchives.gov.uk/PRONOM/Format/proFormatSearch.aspx?status=detailReport&id=1491&strPageToDisplay=signatures

I don't see a fat machO in pronom, though. :(

Y, that makes sense. If we treat it as a container, file though, we could make up/find a mime type for fatmachO (application/x-fat-mach-o I just made that up...is there an existing mime type?) and then the attachments/embedded files would have their own correct mime types.

I've seen application/x-mach-binary and application/x-mach-o only, however I'd be more than happy to come up with more definite ones, alike the ELF-ones. Mach-O also can be shared lib, executable, etc. So technically we should've been doing application/x-mach-o-executable application/x-mach-o-sharedlib and so on. For Fat I've seen application/x-mach-universal but following the proposed structure it would've been application/x-mach-o-universal

If at all possible, we should try to use magic to distinguish fat machO from the other cafebabes.

The structure of fat Mach-O is quite vague (and this), only deep validation by code can help. So ideally I'd use ExecutableParser as priority and if it fails - try other magic-matching cafebabe's

It is tricky if you're new to Tika.

:) That I've noticed :)

I can try to help if you can create the skeleton for this file type

I'd gladly do so, the container is quite simple. It's 0xCAFEBABE + uint32t of number of headers and every header just contains cpu/arch/type flags

Adding more precision to the mimetypes makes sense to me. I'd def want @Gagravarr to weigh in. For some mime types, we use attributes to make the description more precise, e.g. text/plain; charset=UTF-8 but for others, we modify the actual mime type and avoid attributes.

So ideally I'd use ExecutableParser as priority and if it fails - try other magic-matching cafebabe's....

This is really unpleasant to do currently in Tika. Can we do something like in the gabriel vasile link above in mimetypes?

(source))
// Class matches a java class file.
func Class(in []byte) bool {
	return classOrMachO(in) && in[7] > 30
}

// MachO matches Mach-O binaries format
func MachO(in []byte) bool {
	return classOrMachO(in) && in[7] < 20
}

this approach relies on testing the 8th byte, or in other words for far Mach-O
struct fat_header {
	uint32_t	magic;		/* FAT_MAGIC */
	uint32_t	nfat_arch;	/* number of structs that follow */
};

the nfat_arch and I can't find thus far any reasonable explanation why counter is limited by 20. For Java Class, that's a lsb byte of uint16_t major version - also makes no sense why 30 is the threshold. Seem like weird empiric values to me.
To test for Mach-O universal, we could look for 0xCAFEBABE or 0xCAFEBABF, get offset of the first Mach-O from the first struct, and verify that it's a Mach-O. Does Tika's XML allow "read uint and read second it at first ints location"?

https://opensource.apple.com/source/file/file-80.40.2/file/magic/Magdir/cafebabe.auto.html

# Since Java bytecode and Mach-O universal binaries have the same magic number,
# the test must be performed in the same "magic" sequence to get both right.
# The long at offset 4 in a Mach-O universal binary tells the number of
# architectures; the short at offset 4 in a Java bytecode file is the JVM minor
# version and the short at offset 6 is the JVM major version.  Since there are only
# only 18 labeled Mach-O architectures at current, and the first released
# Java class format was version 43.0, we can safely choose any number
# between 18 and 39 to test the number of architectures against
# (and use as a hack). Let's not use 18, because the Mach-O people
# might add another one or two as time goes by...

this comment is priceless :)

So, I guess we have two routes:

Update XML parser to recognize Mach-O universal instead of JNI stuff.
Go with container route

And in any case improve the non-fat Mach-O parsing by extending collection of MIME types.

OMG, that is priceless! Thank you for finding that!

update the mime-types.xml (not, literally, our XMLParser), right?

I think we're basically in agreement? This is what I see:

update the mime-types.xml for non-fat mach-o -- include 32-bit -- figure out if we mark that as a new mime type, a mime-type with different attributes or a separate metadata item
update mime-types.xml for fat mach-o vs the other cafebabes -- create a new mime type for fat mach-o
Parse the fat mach-o as a package file and treat the embedded mach-os like we do other attachments in Tika.

update the mime-types.xml (not, literally, our XMLParser), right?

Yes, I've expressed myself poorly - you're correct.

I think we're basically in agreement?

Totally, we're on the same page. And I'd propose to do a PR per step. I'll do the step 1 with some extra stuff (I want to try parsing Mach-O type using XML) and submit a separate PR for that.  Like, read value and shift to that position?

Maybe it supports value ranges or value sets?

Dynamic offsets aren't currently implemented.
Value ranges can be implemented as a regex (less than ideal, but works for some cases).
Location ranges: yes definitely, as you probably noticed. See e.g. https://github.com/apache/tika/blob/main/tika-core/src/main/resources/org/apache/tika/mime/tika-mimetypes.xml#L539

@tballison I clearly ventured into some weeds with package support, the tests seem to pass on this (rewritten) PR, however I'm not sure if I'm testing what I should be testing.

Prob won't be able to review until tomorrow. Thank you!

@tballison as long as it does not get lost in the drawer :)

Y, it got lost in the drawer. Sorry! Many, many thanks!


SeekableStreams in BAMInputFormat.java aren't closed

In addProbabilisticSplits, a SeekableStream is opened and closed. However, in addBAISplits, three SeekableStreams are opened:
https://github.com/HadoopGenomics/Hadoop-BAM/blob/master/src/main/java/org/seqdoop/hadoop_bam/BAMInputFormat.java#L335 and
https://github.com/HadoopGenomics/Hadoop-BAM/blob/master/src/main/java/org/seqdoop/hadoop_bam/BAMInputFormat.java#L338 and
https://github.com/HadoopGenomics/Hadoop-BAM/blob/master/src/main/java/org/seqdoop/hadoop_bam/BAMInputFormat.java#L407
But are not closed. Would this cause any potential issues? We are having issues loading from s3a with connections not being closed. Trying to figure out whether this would be the cause.

@fnothaft could this be the cause of the s3a issue?

Thanks for finding those! They are definitely an issue, but do you have spark.hadoop.hadoopbam.bam.enable-bai-splitter set to true? If you have not set this config value, then this is not your problem.
Here is the referenced s3a issue, for anyone interested: https://github.com/bigdatagenomics/adam/issues/1906

Thanks @akmorrow13!


Enhancement: support --color=never, --color=auto, --color=always

With current ninja colored output exists only in the "smart" way. ninja deactivates the colors, when piped into something:
$ ninja       # colored output
$ ninja | cat # not colored output

A lot of programs (like ls and grep) have a command line flag to change this behaviour. It would be nice, if ninja supports this, too:
$ ninja --color=always       # colored output
$ ninja --color=always | cat # colored output
$ ninja --color=never        # not colored output
$ ninja --color=never | cat  # not colored output

Duplicate of #1475.


Updated "Scatterer" mod to source from Spacedock

Scatterer is now hosted on Spacedock, rather than Kerbal Stuff.

needs to be renamed to .netkan

LGTM, merging with my thanks


Change user from systemUser to core

`
[root@hpe-dl580gen9-01 test-infra]# ssh -i ssh_key/key<EMAIL_ADDRESS>Fedora CoreOS 31.20200319.dev.1
Tracker: https://github.com/coreos/fedora-coreos-tracker
Discuss: https://discussion.fedoraproject.org/c/server/coreos/
`

In case the user didn't pass ssh public key I will not create the user.
I can create the user and allow login with password but unsure we want to.
@ronniel1 , @avishayt , @filanov , @romfreiman @oshercc , did we have the user password here on purpose?

In case the user didn't pass ssh public key I will not create the user.
I can create the user and allow login with password but unsure we want to.
@ronniel1 , @avishayt , @filanov , @romfreiman @oshercc , did we have the user password here on purpose?

No, we don't want password-based login

is it console password? if so, we might want for now - in care the vm will
not get network
On Mon, May 25, 2020, 19:36 Avishay Traeger<EMAIL_ADDRESS>wrote:

In case the user didn't pass ssh public key I will not create the user.
I can create the user and allow login with password but unsure we want to.
@ronniel1 https://github.com/ronniel1 , @avishayt
https://github.com/avishayt , @filanov https://github.com/filanov ,
@romfreiman https://github.com/romfreiman @oshercc
https://github.com/oshercc , did we have the user password here on
purpose?
No, we don't want password-based login
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/filanov/bm-inventory/pull/163#issuecomment-633645782,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLPP46NMHILRJFOIJI6FITRTKM7BANCNFSM4NJRB2JA
.

OK,
Currently the status is this:
We have password based-login both for core user.
In addition if the user supply  ssh public key we are adding ssh & password based login for systemUser user.
After this PR is merged we will have:
In case the user provided ssh public key we will have ssh and password based login.
As Rom said we might want the password based login for debuging network issues.
Note that this is only for the ISO in live CD, once the install start and the host reboot this configuration is deleted.

The password-based login is via SSH, console, or both?  Is SSH configured to allow password auth?

ssh is configured not to allow password based login.
The password will work only from console

ssh is configured not to allow password based login.
The password will work only from console

OK excellent


Questions about the models supported by the SERAC method.

As far as I know, the serac method does not make changes to the target model, but acts as a plug-in to distinguish whether the query is related to the editing knowledge or not, why do some models such as Qwen not support the serac method?

Thank you for your interest. 😊 The SERAC method relies on a smaller model from the same series to assist with editing. When we integrated the Qwen model, we didn't have a suitable smaller model available in the same series, which is why SERAC wasn't supported at that time.  Your reminder is appreciated, and we will consider adding support for it in the future.

About The SERAC method relies on a smaller model from the same series to assist with editing.
This smaller model is needed to reply questions related to the edit memory?

Yes, more details can be found in the SERAC paper.


Update action_code_examples.yml

closes https://github.com/github/docs/issues/1411
noticed a minor typo when reviewing the code examples on this page. thank you all for this rad landing page!

Why:
https://github.com/github/docs/issues/1411
What's being changed:
Minor typo fix

Check off the following:

[ ] All of the tests are passing.
[ ] I have reviewed my changes in staging.
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

Thanks @becca! I'll update this branch and merge once tests are passing 🍏


Always provide -y   to apt-get install

Ref: workaround was attempted at https://github.com/neuronets/datalad-action/pull/1/files
Origin: trying to use datalad-installer on a pristine'ish Debian AMI on AWS.

[x] confirm that solves the issue for @hvgazula @aaronkanzer

note: next aspect is that while trying on that AMI -- it failed to install because there were no prior apt-get update run and some package information was outdated. But it is a separate issue

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (2196b1b) 65.57% compared to head (0fc1b1d) 62.11%.

Files
Patch %
Lines

src/datalad_installer.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #194      +/-   ##
==========================================
- Coverage   65.57%   62.11%   -3.46%
==========================================
  Files           1        1
  Lines        1644     1618      -26
  Branches      408      331      -77
==========================================
- Hits         1078     1005      -73
- Misses        466      510      +44
- Partials      100      103       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


test34

Description
Briefly describe the bug you are facing.
Steps to reproduce
Provide steps to replicate.
Observed result
Please provide command output with --debug flag set.
Expected result
Describe what you expected.
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS:
sam --version:

Add --debug flag to command you are running

test

test


Calling dll function with nodejs fails

I've some problems with calling functions in a special dll of software used in our company.
I made a vb.net file and compiled it to a dll.

Public Class TestDLL     Public Shared Function Add(ByVal a As Integer, ByVal b As Integer) As Integer         Return a + b     End Function End Class

In nodeJS i wrote the code:

 const testlib = koffi.load('BRCdll/testdll/testdll.dll'); const Add = testlib.func('int __stdcall Add(int, int)'); const result = Add(1, 2); console.log('result:', result);

Output:

const Add = testlib.func('int __stdcall Add(int, int)');
^
Error: Cannot find function 'Add' in shared library

Why does this go wrong?

AFAIK, C# and VB.net public methods don't use a pure C ABI. Please look into unmanaged exports.


NullReferenceException on right-clicking administration building

My KSP suddenly got very slow and crashed, right after i right-clicked the administration building.
Other times, the same error popped in the console, but the game didn't crash. Now apparently the game entered a loop and crashed.
I'm using 1.9.1

The error stack
[LOG 20:41:17.222] [MessageSystem] Save Messages
[LOG 20:41:17.583] Game State Saved to saves/default/persistent
[LOG 20:41:17.588] Administration (KSP.UI.UICanvasPrefab)
[LOG 20:41:17.589] [UIMasterController] RemoveCanvas:Administration

[ERR 20:41:22.454] Exception handling event onFacilityContextMenuSpawn in class StrategyNotifier:System.NullReferenceException
  at (wrapper managed-to-native) UnityEngine.MonoBehaviour.IsObjectMonoBehaviour(UnityEngine.Object)
  at UnityEngine.MonoBehaviour.StartCoroutine (System.Collections.IEnumerator routine) [0x00012] in <7d9ec060e791409ab3eb85c61e312ed6>:0
  at Strategia.StrategyNotifier.OnFacilityContextMenuSpawn (KSP.UI.Screens.KSCFacilityContextMenu menu) [0x00024] in <ad1f24c047af4716ace30351b02ea8a1>:0
  at EventData`1[T].Fire (T data) [0x000b0] in <ac04291822974a138f007b85091a10d0>:0

[EXC 20:41:23.401] NullReferenceException
	UnityEngine.MonoBehaviour.StartCoroutine (System.Collections.IEnumerator routine) (at <7d9ec060e791409ab3eb85c61e312ed6>:0)
	Strategia.StrategyNotifier.OnFacilityContextMenuSpawn (KSP.UI.Screens.KSCFacilityContextMenu menu) (at <ad1f24c047af4716ace30351b02ea8a1>:0)
	EventData`1[T].Fire (T data) (at <ac04291822974a138f007b85091a10d0>:0)
	UnityEngine.DebugLogHandler:LogException(Exception, Object)
	ModuleManager.UnityLogHandle.InterceptLogHandler:LogException(Exception, Object)
	UnityEngine.Debug:LogException(Exception)
	EventData`1:Fire(KSCFacilityContextMenu)
	KSP.UI.Screens.KSCFacilityContextMenu:Create(SpaceCenterBuilding, Callback`1)
	SpaceCenterBuilding:OnContextMenuSpawn()
	SpaceCenterBuilding:OnRightClick()
	<OnMouseTap>d__49:MoveNext()
	UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr)

[Many iterations of the same error later]
[ERR 20:41:34.576] Exception handling event onFacilityContextMenuSpawn in class StrategyNotifier:System.NullReferenceException
  at (wrapper managed-to-native) UnityEngine.MonoBehaviour.IsObjectMonoBehaviour(UnityEngine.Object)
  at UnityEngine.MonoBehaviour.StartCoroutine (System.Collections.IEnumerator routine) [0x00012] in <7d9ec060e791409ab3eb85c61e312ed6>:0
  at Strategia.StrategyNotifier.OnFacilityContextMenuSpawn (KSP.UI.Screens.KSCFacilityContextMenu menu) [0x00024] in <ad1f24c047af4716ace30351b02ea8a1>:0
  at EventData`1[T].Fire (T data) [0x000b0] in <ac04291822974a138f007b85091a10d0>:0

[EXC 20:41:35.618] NullReferenceException
	UnityEngine.MonoBehaviour.StartCoroutine (System.Collections.IEnumerator routine) (at <7d9ec060e791409ab3eb85c61e312ed6>:0)
	Strategia.StrategyNotifier.OnFacilityContextMenuSpawn (KSP.UI.Screens.KSCFacilityContextMenu menu) (at <ad1f24c047af4716ace30351b02ea8a1>:0)
	EventData`1[T].Fire (T data) (at <ac04291822974a138f007b85091a10d0>:0)
	UnityEngine.DebugLogHandler:LogException(Exception, Object)
	ModuleManager.UnityLogHandle.InterceptLogHandler:LogException(Exception, Object)
	UnityEngine.Debug:LogException(Exception)
	EventData`1:Fire(KSCFacilityContextMenu)
	KSP.UI.Screens.KSCFacilityContextMenu:Create(SpaceCenterBuilding, Callback`1)
	SpaceCenterBuilding:OnContextMenuSpawn()
	SpaceCenterBuilding:OnRightClick()
	<OnMouseTap>d__49:MoveNext()
	UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr)

I'm using various mods, if necessary i'll post the complete list here

I've seen this reported elsewhere, but I haven't been able to reproduce it.  Could you provide the mod list (or just a KSP.log file would probably be better)?

My modlist
B9 Aerospace Procedural Wings - Fork (B9-PWings-Fork 1:v0.93)
Breaking Ground (BreakingGround-DLC 1.4.1)
Chatterer (Chatterer 0.9.99)
Chatterer Extended (ChattererExtended 0.6.2)
ClickThrough Blocker (ClickThroughBlocker 1:<IP_ADDRESS>)
CommNet Antennas Consumptor (CommNetAntennasConsumptor 3.0.2)
CommNet Antennas Extension (CommNetAntennasExtension 2.1.3)
CommNet Relays (ContractConfigurator-CommNetRelays v2.1.0)
Community Parts Titles (CommunityPartsTitles 0.6.4)
Community Resource Pack (CommunityResourcePack <IP_ADDRESS>)
Contract Configurator (ContractConfigurator 1.29.0)
Contract Pack: Bases and Stations Reborn (ContractConfigurator-KerbinSpaceStation 2:<IP_ADDRESS>)
Contract Pack: Clever Sats (ContractConfigurator-CleverSats 1.4)
Contract Pack: Exploration Plus (ContractConfigurator-ExplorationPlus 2.0.1)
Contract Pack: Field Research (ContractConfigurator-FieldResearch 1.2.1)
Contract Pack: Kerbal Academy (ContractConfigurator-KerbalAcademy 1.1.10)
Contract Pack: Tourism Plus (ContractConfigurator-Tourism 1.5.2)
Crowd Sourced Science (CrowdSourcedScience v6.0)
Custom Barn Kit (CustomBarnKit <IP_ADDRESS>)
Distant Object Enhancement Continued (DistantObject v<IP_ADDRESS>)
Distant Object Enhancement Continued default config (DistantObject-default v<IP_ADDRESS>)
Kerbal Changelog (KerbalChangelog v1.3.0)
Kerbal Engineer Redux (KerbalEngineerRedux <IP_ADDRESS>)
Kerbalism (Kerbalism 3.11)
Kerbalism - Default Config (Kerbalism-Config-Default 3.11)
kOS: Scriptable Autopilot System (kOS 1:<IP_ADDRESS>)
Making History (MakingHistory-DLC 1.9.1)
Modular Rocket Systems (ModularRocketSystem 1.13.2)
Module Manager (ModuleManager 4.1.4)
Procedural Parts (ProceduralParts v2.1.0)
Real Plume (RealPlume 2:v13.3.1)
Real Plume - Stock Configs (RealPlume-StockConfigs v4.0.1)
Reentry Particle Effect Renewed (ReentryParticleEffect <IP_ADDRESS>)
RemoteTech Redev Antennas (RemoteTechRedevAntennas 0.1.1)
ReStock (ReStock 1.1.2)
ReStock Extra - Rigid Legs (RestockRigidLegs 1.1.2)
ReStock+ (ReStockPlus 1.1.2)
SmokeScreen - Extended FX Plugin (SmokeScreen <IP_ADDRESS>)
Strategia (Strategia 1.8.0)
Surface Mounted Lights (surfacelights 1.16)
Textures Unlimited (TexturesUnlimited <IP_ADDRESS>)
Toolbar Controller (ToolbarController 1:<IP_ADDRESS>)
Waypoint Manager (WaypointManager <IP_ADDRESS>)
Zero MiniAVC (ZeroMiniAVC 1:<IP_ADDRESS>)

Principia is also installed (Gallai release, i believe). Also, i was using Ubuntu 20.04.
The lagginess also happened in apparently random moments when flying rockets or airplanes, so i suspect other mods.
Actually i already gave up on trying to play KSP on Linux, cause i don't see any way to debug the random lagginess (i would appreciate if you know any tools besides the game debug console).
Almost every time i have to REISUB (can't even xkill).
I suspect some bizarre memory leak/lack of code optimization for Linux, but i don't know where to start. This Exception was the only big error shown (besides the many warnings). I could keep digging though.
I've been playing with a similar modlist on Win10 and encountered no problems with Strategia or KSP in general (despite the 15 minutes load times on my old computer).

My KSP.log. There were actually tons of errors, so i'm sorry if any of them are known.
I'll check later if those tons of errors also appear on Win10.
KSP.log


Stack items when sorting itembox

It should also sort the item type. So all the weapons are together, all the ammo is together, all the health items are together etc.

It should also sort the item type. So all the weapons are together, all the ammo is together, all the health items are together etc.

I did some changes to achieve that.
PTAL 😃


Getting error with basic first code

Hi, I tried only the first lines of code that were :
import aspect_based_sentiment_analysis as absa

nlp = absa.load()
text = ("We are great fans of Slack, but we wish the subscriptions "
        "were more accessible to small startups.")

slack, price = nlp(text, aspects=['slack', 'price'])
assert price.sentiment == absa.Sentiment.negative
assert slack.sentiment == absa.Sentiment.positive

and got the following error :
Traceback (most recent call last):
File "test_absa.py", line 3, in
nlp = absa.load()
File "C:\Users\theo.flaus\Anaconda3\envs\test\lib\site-packages\aspect_based_sentiment_analysis\loads.py", line 34, in load
model = BertABSClassifier.from_pretrained(name, config=config)
File "C:\Users\theo.flaus\Anaconda3\envs\test\lib\site-packages\transformers\modeling_tf_utils.py", line 2699, in from_pretrained
model(model.dummy_inputs)  # build the network with dummy inputs
File "C:\Users\theo.flaus\Anaconda3\envs\test\lib\site-packages\keras\utils\traceback_utils.py", line 67, in error_handler
raise e.with_traceback(filtered_tb) from None
File "C:\Users\theo.flaus\Anaconda3\envs\test\lib\site-packages\aspect_based_sentiment_analysis\models.py", line 141, in call
outputs = self.bert(
ValueError: Exception encountered when calling layer "bert_abs_classifier" (type BertABSClassifier).
The first argument to Layer.call must always be passed.
Call arguments received by layer "bert_abs_classifier" (type BertABSClassifier):
• token_ids={'input_ids': 'tf.Tensor(shape=(3, 5), dtype=int32)'}
• attention_mask=None
• token_type_ids=None
• training=False
• bert_kwargs=<class 'inspect._empty'>
Any idea how to solve this ?

Same here, any solutions?

I got the same error.  For me it was solved when I installed from the environment.yml. My problem in specific is that I had Python >3.7

A bit late, but can confirm (at least for me) changing Python version immediately fixed the issue.
To create a new virtual environment with Python version 3.6 in Jupyter Notebook, take a look at this.
This is what it look like for me:

I am unable to install python version 3.6 on jupyter notebook colab. I get an error like this

Any ideas why this might be the case?
@VedantNemane

for me error is :
RecursionError: maximum recursion depth exceeded in instancecheck


[ISSUE-3653] Max_time and last return inconsistent result

It's a order by time desc bug. DescReadWriteBatchData should be flipped before returned.

Coverage increased (+0.004%) to 68.287% when pulling d86d65431a38b29bcfec546bc7b49670352962eb on masterDescBug into 902dbfb6fdda03e5d608dc12f9ebb28177b4fed9 on master.


[RFE] Support TimescaleDB backend

Taking inspiration from zabbix monitoring software's recent advance(See R1), it will be great if miq can also delete metrics records over a set period of time(assumed one year old).
R1: https://blog.zabbix.com/zabbix-time-series-data-and-timescaledb/6642/

I considered this a long time ago to replace our metrics database entries.  While TimescaleDB itself it pretty awesome, it had some nuances to it that make it hard to use with our application.
I can't remember then offhand, but if I remember them, then I'll come back here and enumerate them.

it will be great if miq can also delete metrics records over a set period of time(assumed one year old).

This already happens via the purger


Chunk System

Create a independent chunk system for VoxelObjects.

I'm thinking of using surfaces to make a chunk system of sorts

chunk system is closely tied to #1 , any further discussion will be done there


Api causes long load time, so question does not appear even though it is posted.

The API-check spends a long time running, which means that the question is posted after the question list reloads. This means the question does not show up unless you refresh! TOP PRIORITY TO FIX!

Added a try/catch. It is still very slow, but at least itposts and loads.


stopwatch class on CPU always returns 0 on Linux

Worked fine on Mac, but appears to return 0.0 every time on Linux (dirac, CUDA
4.0). It's this line:

    cudpp_app::StopWatch timer;
            printf("CPU execution time = %f\n", timer.getTime());

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 8 Jul 2011 at 5:52

This was caused by calling start() when I meant stop() in testScan.cpp.  Fixed.

Original comment by<EMAIL_ADDRESS>on 22 Jul 2011 at 5:01

Changed state: Fixed


fix: Replace IE11 incompatible DOM features

Closes #360
This PR replaces Element.prototype.closest and Element.prototype.remove by their ponyfilled versions. These methods aren't automatically polyfilled by Babel on apps. So, with this, Reakit should work with IE11 by simply using Babel (production build).
Does this PR introduce a breaking change?
No

Codecov Report

Merging #443 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #443      +/-   ##
=========================================
+ Coverage   95.29%   95.3%   +<.01%
=========================================
  Files          94      94
  Lines        1424    1427       +3
  Branches      445     446       +1
=========================================
+ Hits         1357    1360       +3
  Misses         67      67

Impacted Files
Coverage Δ

packages/reakit/src/Menu/__utils/useShortcuts.ts
96.96% <100%> (ø)
:arrow_up:

packages/reakit/src/Dialog/__utils/usePortalRef.ts
100% <100%> (ø)
:arrow_up:

packages/reakit/src/Dialog/__utils/useFocusTrap.ts
97.01% <100%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

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


Porting favico-generator to GUI app

Due I was using Proton Native to build the GUI, it has some limitation for the look and feel, even though the functionality is the same and is lightweight
Please review it and let me know whether it needs any change.

Hi, thanks. I haven't been able to test it out but I bet it'll be nice.


Adding support for FIFO Queues

This plugin as it currently stands does not support FIFO Queues.
It is missing the following required fields:

Message Group Id
Message Deduplication Id

This PR aims to make it compatible, by adding those two fields to the send-message requests whenever the Queue name contains .fifo, which is the required suffix for FIFO Queues.
The Group Id is what ensures the FIFO behaviour of the Queue, but only if it's the same for every message, it's setting a default value for it, which will ensure the FIFO behaviour, but it can be overwritten if needed.
The Deduplication Id must be unique to every message if you want it to be sent, so unless otherwise specified, it's generating a uuid for it for every send request.

Hi @hobsons-feliperoque, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in your Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

Is this PR dead if @hobsons-feliperoque doesn't update his profile? The changes look like a nice addition to the official plugin.

Any update on this. This feature is a nice to have!


Donut-Maven-Plugin is not supporting Cucumber-Junit latest version

Donut-Maven-Plugin is not supporting Cucumber-Junit latest version.
Latest cucumber Junit version is 4.2.0.
Also donut should support the 'Rerun Failing Tests' capability which is introduced in the latest version of Junit.

donut-maven-plugin now supports the latest cucumber JSON output format for cucumber 4.2


Not able to start taskcat on windows

$ curl -s https://raw.githubusercontent.com/aws-quickstart/taskcat/master/installer/docker-install-master| python -E
[INITALIZING taskcat Installer!!]
Ready!! type 'taskcat' to start
nampatra@IBM144-PC0BY4WK MINGW64 ~
$ taskcat
bash: taskcat: command not found
nampatra@IBM144-PC0BY4WK MINGW64 ~
$ ./taskcat
bash: ./taskcat: No such file or directory
I am running on windows OS . Please help out.

@nampatra  Apologies for the late response.. we are working on windows support. As of now, it's very limited. If you can try installing via pip with the instructions here: https://aws-quickstart.github.io/taskcat/#installing-via-pip and let us know the results that would be very helpful.
You will need windows bash installed with python3 and pip configured properly.
If you have met those requirements. Please try the steps below:

pip install taskcat
Download the taskcat.py file from here: https://raw.githubusercontent.com/aws-quickstart/taskcat/master/taskcat.py
Set the execute bit on the file  chmod 755 taskcat.py

@nampatra any update on the workaround above?

Closing due to no activity. @nampatra  please reopen if you have further feedback


[Enhancement] Remove Video Section

Motivation / Problem
We need to remove the YT video from the landing page due to privacy implications.
Solution
For now, we should simply remove the video player and use a placeholder image. We can use the CtD section image.
Notes
Since this is the only place where we actually displayed any video, we should also remove all related dependencies.

Do we leave the link to the youtube video on the "Learn More" button or do we remove that as well?


Posix event engine tcp client

Implements the Posix event engine client Connect and CancelConnect methods

Includes a test for connection timeouts and cancellation
Tests successful connect and subsequent exchange of data using the event engine client test suite.

Fixed your comments. Also found a minor bug in the poller so updated the poller api contract in 7133e494404d111f7c7d328df07c6e95a90e4ac8


feat: enable pylint workflow.  suppress all errors (for now)

There are already pre-commit hooks for flake8. Any reason why you strongly prefer pylint?

no strong preference, but I saw a gazillion warnings/errors with pylint, so I thought it would make sense to start enabling these lints one by one over time.
if you think flake8 is better, feel free to ignore this.

I'll probably stick with flake8 for now. But will keep pylint in mind if I see a need for change.


Update service worker to perform caching activities in install event

The service worker event activate is not guaranteed to run to completion, for example if the browser closes. If it fails or ends early, then it will never be rerun.
The install event has stronger guarantees, and is a better solution for updating the caches.

activate could be updated to only perform cleanup

Congrats on the Flutter 2 PWA launch!
I was taking a look at the service worker for one of the sample apps, which I assume is representative of all Flutter PWAs.
Unfortunately, the activate handler is still being used for critical cache manipulation, not just cleanup. I've mentioned this elsewhere, but putting any data operations that require guaranteed completion inside of activate handler risks cache inconsistency, as we experienced in the Workbox project (https://github.com/GoogleChrome/workbox/issues/1793).
Apologies if I'm misreading your service worker, but based on the fact that this issue is still open, I'm assuming that I'm correct in that assessment.


Remove Sketch Toolbox entry (deprecated)

Project URL
https://github.com/buzzfeed/Sketch-Toolbox
Category
Grpahics
Description
Remove Sketch Toolbox because it has been deprecated.

          1 Warning

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

2993
404
https://raw.githubusercontent.com/sindresorhus/gifski-app/master/Stuff/screenshot.jpg

5591
301
https://github.com/jwilm/alacritty redirects tohttps://github.com/alacritty/alacritty

  Generated by :no_entry_sign: Danger