Ensure $DISPLAY exists for xsel or xclip + add cygwin /dev/clipboard …

…support.
xsel and xclip both exist for cygwin when cygwin/X is installed, but it is much less common to have the X server running.  Ensure that the $DISPLAY environment variable is set when determining which command to pick.
Additionally, cygwin has a built-in virtual device called /dev/clipboard that we can leverage instead of requiring putclip to be installed.  Added support for that.
If you would like the test for $DISPLAY to be handled in another manner, I am open to editing this PR to meet those needs, but this worked in both my linux and windows/cygwin environment after the changes as is.

The additions of $DISPLAY and cygwin support should two separate commits.

PR #58 implements tests for $DISPLAY.


Epoll and Kqueue shouldn't read by default

Motivation:
Epoll and Kqueue channels have internal state which forces
a single read operation after channel construction. This
violates the Channel#read() interface which indicates that
data shouldn't be delivered until this method is called.
The behavior is also inconsistent with the NIO transport.
Modifications:

Epoll and Kqueue shouldn't unconditionally read upon
initialization, and instead should rely upon Channel#read()
or auto_read.

Result:
Epoll and Kqueue are more consistent with NIO.

@Scottmitch good catch!


Parallel Aggregate Hash Tables

Battle plan: Each thread allocates separate blocks for pointers and group/payload blobs. Separate column for hashes. Blocks of all threads are merged on combine

Just a vote to prioritize this effort.   This represents a significant performance opportunity.
There seem to be significant advantages to models that include thread-local aggregation (with overflow behaviors at some condition) for performance with bounded group by cardinality use cases ( most BI/visualization tools, in my experience )
Slide 35 has a comparative graph that makes a strong argument for Adaptive model, but I suspect there are related solutions.
https://people.csail.mit.edu/jshun/6886-s19/lectures/lecture12-2.pdf
with examples:
SQL1:   select group_by_buckets , sum(col2) from table group by group_by_buckets
SQL2:  select sum(col2) from table where group_by_buckets>0 ;
Ideal product behaviors for consistent performance:

similar performance for SQL1 and SQL2 when number of group_by_buckets is small
difference in performance between SQL1 and SQL2 is independent of # of rows in table
smooth performance curve as number of group_by_buckets scales
similar linear scalability for SQL1 and SQL2 when number of group_by_buckets is small
efficient handling of skew: with 32 threads, 999 million tuples of bucket 1, then 1 million other buckets with 1 tuple.
ideal processing model (Adaptive?) would allow for thread-local handling of bucket 1, other path for most other values.

We are almost done implementing this.

This has been merged


Reading BG values from history & clean teardown of pump connection (2nd attempt)

Hi,
Based on @pazaan implementation in tidepool-org/chrome-uploader I ported first implementation of download of the pump history, right now supporting only BG Values
Also made a bit of refactoring in the main routine, so that connection build-up and safe teardown is placed in downloadPumpSession(), while custom functionality is passed as a reference (pumpDownload).
Added also all the commands for clean exit and close of pump connection.
Also added a check if 0x81 responses are really these expected (sometimes they aren't!).

@pazaan, did you have time to review this PR ?

Hey @paulokow. Just got back for a long OS trip without access to devices. Will take a look this weekend.

@paulokow Thanks for this incredible contribution! Would you be interested in being a direct contributor to this repo?

Hi @pazaan , I'll be honoured to be a direct contributor.


Only attach on initial view of a tabs content.

Building on #6832, tabs should allow for lazy loading to only occur for initial load, then reuse already created view.
This will be important for holding the current state of something like a form, as well as no requiring reinitialization of more costly views.

Really this should be stated as no detaching the view when the tab leaves the view, instead just hiding it.  This does not directly necessitate building off of #6832, but will likely be added after its merged.

@jelbourn Is this something you still think we should do?  I know that we have tests that will be effected by this, though its functionality is not breaking.

I think we need to measure what kind of on impact it would actually have


[DOCFIX] Add a bootstrap-conf command snippet in the Chinese version of docs/en/Configuring-Alluxio-with-secure-HDFS.md

This PR fixes issue with JIRA ticket ALLUXIO-1983
(Note: The fixed file is different from that in #3459 which fixed Configuring-Alluxio-with-HDFS.md, while this PR fixes Configuring-Alluxio-with-secure-HDFS.md. I also left some comment under #3466)
In docs/en/Configuring-Alluxio-with-secure-HDFS.md of the English version there is a bootstrap-conf command snippet added in the Configuring Alluxio section. This PR translate it into Chinese and add it in the corresponding page of the Chinese version.

Can one of the admins verify this patch?

LGTM, thanks!


Banana Pi: Active Event on GPIO makes it impossible to read value

Event change GPIO mode to EINT. With EINT its not possible to read memory mapped value. reding /sys/class/gpio/gpioXXX/value would work. this would be a possible a workaround.

it's labeled as bug but its some strange behavior of H3 SoC .

workaround was easy to implement


Added helper function for 1D noise (using 2D noise with a static value for one dimension)

Project file:
Testing-1D-Noise.zip

1d noise 😱😳, is it possible to make Randomly generated worlds with this?


Horizontal and vertical scrollbar always showing when I use Angular material to display a mat-table inside mat-dialog

🔎 Usage Questions
Before you ask a question, please make sure you've:

Looked for prior or closed issues
Checked the README
Looked for/asked questions on stack overflow: https://stackoverflow.com/search?q=simplebar

✨ Feature Requests
SimpleBar's mission statement is to keep things simple, however if you have a suggestion that falls within this approach that hasn't already been proposed, we'd love to hear it!
🐛 Bugs
Please complete the following template:
Current Behavior
I'm trying to use this scrollbar directive with Angular Material mat-table inside a mat-dialog.  When the mat-table is inside the dialog horizontal and vertical scrollbar both show regardless if there is overflow.  I've used this scrollbar with a mat-list inside a mat-dialog and all works fine.  Only happens with mat-table.  I'm using chrome 72.0.3626.119.
Screenshot

Expected behavior
Scrollbar should only show if there is an overflow.
Reproducible example

Suggested solution(s)

Additional context

Software
Version(s)

SimpleBar
3.1.3

Browser
n/a

npm/Yarn
6.4.1

Operating System
Windows 10

Angular
7.2.1

Closing this as it's pretty old. Please re-open a new issue related to the latest version if needed! Thanks.


Is the websocket function necessary? And if not how can I close it?

Hi @liudaac ,
OmniDB 3 beta is now live, and the websocket has been replaced with a different strategy using long polling.
2.x had some well-known issues related to the websocket and those are gone.
I will now close this issue, thanks for your contribution.
Best regards


Use sub-tests for restricted join tests

The sub-tests make it clearer which parts of the test are failing and which are succeeding.
This also adds MatchSuccess and MatchFailure for wider checks of whether something had failed or not (it's either in the HTTP 200 range or it isn't), rather than checking specific error codes.

Maybe this is a misunderstanding on my part, but don't sub-tests run in parallel? That's generally why I haven't used them in complement tests.

Maybe this is a misunderstanding on my part, but don't sub-tests run in parallel?

They only run in parallel if you call t.Parallel() at the beginning of the subtest.

--- FAIL: TestRestrictedRoomsRemoteJoinFailOver (15.34s) still though.


Color in UI: DL and Carb are mirrored page designs

the issues we would create for this page still apply because they are the same for Carbon.
For mobile there are specific mobile friendly images. The “Digital accessibility” section for example should be like current DL
https://www.ibm.com/design/language/elements/color-ui
Expected:

Staging:

transferring this to the Carbon website repo so we can make image updates there


implement ComplexesConstraint

It should work similarly to StrandPairsConstraint, for example, passing in many complexes in a single call.

The ComplexesConstraint class is there, but it is not yet being stored in Design or checked in search_for_dna_sequences.

Now that #93 has been implemented, ComplexesConstraint is automatically checked along with all other BulkConstraint's, so this has been closed.


Fixed issue

What this PR does / why we need it:
This issue replaces a command with the correct one.
Which issue(s) this PR fixes:
Fixes #167
Special notes for your reviewer:
Release note:

NONE

/review @Kristian-ZH @JordanJordanov


Today Button click doesn't set current date when autoclose: True

Today button has a strange behaviour. When autoclose parameter is set to True, after clicking "Today" on calendar widget widget is closing without setting current date. Don't know how to fix it.
My javascript:
 $(document).ready(function () {
    $('#datepicker').datepicker({
        todayHighlight: true,
        todayBtn: true,
        keyboardNavigation:true,
        autoclose: true,
      startView:3
    })
  })

thanks


Collection layout

Setup routes and layout for the home collection page.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chromium: Update upstream versions

chromium-binary-plugins is broken on master because the upstream versions went away.
Building and testing locally right now... will check off testing tasks below when complete.
Things done

[ ] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@bradediger Could you be so kind to take care of google-chrome as well? That would fix #17480

@NeQuissimus Are you sure it isn't covered by this one? Looks like google-chrome depends on chromium.upstream-info, which I updated here.

Cannot write: No space left on device

Sorry travis-ci :unamused:

@bradediger Oh, my bad. That is very much possible.

cc @aszlig

It would seem that @benley did this in #17500


Remove spring boot libraries

In the past, I have always tried to make it as easy for developers as possible. One pitfall I wanted to avoid was that users built their report in JasperSoftStudio, deployed it onto the JRI-Integration, and it would break.
Thus, I tried to include all libraries that might be required. Sometimes, libraries get added that are just there for running some tests/samples, e.g. the Spring Boot libraries.
I never wanted to go through the hassle of understanding all dependencies, thus I included most of the libraries.
After the log4shell issue I am more reluctant to do so.
I have just come across the security warning here: https://github.com/TIBCOSoftware/jasperreports/issues/241
In the upcoming release (and forward) I will remove all references to spring boot. If you should need them, you can add those libraries back yourself.

The gradle task jriDownloadJasperLibraries will first download the libraries, then copy them to the local lib directory. In this step, the spring libraries will be excluded:
  // https://github.com/TIBCOSoftware/jasperreports/issues/241
  // CVE-2021-22096 - Medium Severity Vulnerability,  Vulnerable Library - spring-core-5.1.4.RELEASE.jar
  // spring boot framework should not be required anyway
  exclude "spring-core-*"
  exclude "spring-beans-*"

the spring framework is required for recompiling the .jrxml file into .jasper.
Thus, I will remove the vulnerable version from the jasper-lib download and always upgrade to the latest one through the gradle build. Currently, I have included: spring-core-5.3.15.jar, spring-beans-5.3.15.jar


Allow Int#chr to accept an optional encoding, much like String.new(bytes : Bytes, encoding : String)

Discussion
Allow Int#chr to accept an optional encoding for converting a byte value to the encodings defined character for said byte value. String.new(bytes : Bytes, encoding : String) already allows specifying a custom encoding, so I feel like Int#chr should as well.

The idea does not sound very convincing. It seems like a very rare use case. String.new should be reasonably easy to use. Or you can implement a custom single character encoding method if you need to. But I don't see this as a common problem.
Can you explain why you think this would be generally useful as a standard lib feature?

Ths two methods are not related; String.new relates to UTF-8 byte sequences, whereas Int#chr deals with UTF-8 codepoints. Not all encodings define their characters in terms of codepoints, so such a generalization isn't possible here.

Right. It seems you can pass an encoding to Ruby's chr, but in my mind that doesn't make much sense. Maybe it only works for one byte, but more bytes aren't supported? But I don't see that as very useful.
Maybe a method Char.new(byte, encoding) and Char.new(bytes, encoding) would make sense, where a single char is decoded from a byte or a byte sequence?


Added references to Google C++ interface

Added links to new C++ interface by Google @mlubin @gbanjac.

Coverage remained the same at 91.249% when pulling 47938a7f761f11c054fc0ac0dfa4a541bf09c1e9 on osqp_cpp_interface_docs into 38429297b8c48c3189e2ca7eac97e7e821275bab on master.


Building 'PvQv_C'?

I've been looking for python implementations of tension splines for years, so I'm keen to try out your code. But I am having some problems finding the right commands to build PvQv_C.
I tried:

pip install . (which seems to run without error, but doesn't fix the No module named 'PvQv_C' error)
python setup.py build_ext --inplace followed by python setup.py install (which also seems to run without error, but then I get an import error on spatial_interpolators instead of PvQv_C)
importing and installing pyximport first (which seems to do nothing useful)

I'm on Python 3.8.8.

Hi @nwilliams-kobold
sorry about that.  I was reworking the cython import last time I worked on this project and needed to do a couple of extra changes.  I think I have things fixed for you in PR #2 .  I also added a quick flake8 CI to try to catch things like this.
Cheers,
Tyler

Perfect thanks. To confirm, this now installs fine just with pip install .


homework for week one sql

your name: Salem-Alalem
city: Glasgow
slack name: Sslim123
module: sql 2
week: 2


Add cookie support

It's resolve this issue https://github.com/Mashape/unirest-php/issues/35

great :+1: if you can look at my notes and update that'll be great, otherwise, I can check in again and take care of that myself.
thank you!

this needs rebasing, i might get to it later this weekend.

Please, could you merge this, I need this fix badly!
Thanks guys!

@cristianp6 i just made the fix, waiting for tests then releasing a new version

@cristianp6 @daydiff this is now available in v2.4.0

Thanks @ahmadnassri
I've made a pull request https://github.com/Mashape/unirest-php/pull/77
hope it helps

ah, that's what happens when attempting to fix bad merges late at night (and when no unit-tests are written!) totally my bad! thanks @cristianp6 for fixing!


Questions about initialize function

The initialize function is designed to be used once and then never used again.
Are the values of superchainConfig and systemConfig not going to change in the future?
https://github.com/tokamak-network/tokamak-thanos/blob/5ad9baac98217a0c1533969b00076d9a4443edba/packages/contracts-bedrock/src/L1/L1StandardBridge.sol#L87-L93
And since the permission for initialize is public, I wonder when the initialize is performed after the contract is distributed.

Hi @zzooppii
I think the modifier initializer make sure the initialize will be called one time

And since the permission for initialize is public, I wonder when the initialize is performed after the contract is deployed.

This function must be called when setting implementation by using upgradeAndCall. So it is safe

okay thank you


pdfpc, libgee: Add darwin support

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I think the platforms for one of the dependencies should also be updated.
Package ‘libgee-0.18.0’ in ‘/nix/store/8fzb0a1y40fc61ar3585ajp1vxkb3x30-git-export/pkgs/desktops/gnome-3/3.22/core/libgee/default.nix:21’ is not supported on ‘x86_64-darwin’, refusing to evaluate.

Sorry about the many name changes. Autocorrect got me twice. Thanks @LnL7.

It looks like it's actually gnome3.libgee-0.18.0, not sure why that isn't defined top-level.

I found it in all-packages.nix as libgee_0_8.

Yes, but that's actually another file https://search.nix.gsc.io/?q=libgee-&i=nope&files=&repos=

Ohh my bad. I see that I've update the wrong file. I'll get rid of that commit.


Add PaymentRequest and Payment messages

do not merge til erc20 payment spec, please
cc: @volodymyr-basiuk


AI Coding Assistant

Context and Problem Statement
Using an AI powered coding assistant can increase productivity. Which, if any, should we use?
Considered Options

Google Codey
GitHub Copilot Your AI pair programmer
Amazon CodeWhisperer Build applications faster and more securely with your AI coding companion
ChatGPT
Tabnine AI assistant that speeds up delivery and keeps your code safe
None (considering bans by Samsung, Apple, etc on staff using them)

Have been experimenting with using TabNine (with the Mistral setting) to write draft docstrings & unit tests in Python packages - have found that this improves clarity & would recommend. Do we have an update on policy in this area? May depend to some degree on how 'novel' we think the codebase being edited is.


Low FPS than report in Jetson nano

Hi,
I highly appreciate your repo. But I test your model in Jetson nano Tegra X1 it just got about 8FPS lower than 22 FPS in your report. I just follow your guide except I did not use USB camera, I used OpenCV to read image instead of. I want to know whether you use other kind of Jetson nano, which version of Pytorch or I made a mistake. Many tks.

I have the same issue.

@twmht I guess the author calculate with small cmap_window and quite small max_num_objects.

I have the same issue, I think. With resnet I reached only 6 FPS. I used the notebook given so it's quite strange that I can't get the same results of authors using the same environment.


Snapshot sequence number is not increased when snapshoting

Hello. I have a project that for testing uses the akka-persistence-inmemory. The problem is that the snapshot sequence number is not increased. I cannot publish the code since is private but I just create a simple PersistentActor and call the saveSnapshot(...) 3 times. I have another actor recovering with the same persistence ID and it gets the latest snapshot but the sequence number is 0  and it should be 3. I checked the log and I do get 3 SaveSnapshotSuccess(---) messages but they all have the same sequence number. Anyone know why?

Thanks, will go to the backlog.

Hello Dennis,
About this issue. I do not think it is a bug. It is not well documented.
The persistence number is increased as soon as you persist in something
before. So, if you snapshot without having persisted you end up having
snapshot a seqnr 0. If you do another snapshot then again that snapshot is
at 0. Only persistence increases the number. If this is the intended
behaviour then everything is fine and you can close the bug report but
maybe add the documentation. If not, I plan to add a pull request to Akka
itself and explain the behaviour in detail since I feel more than one will
have trouble getting their head around this persistence seqnr.
Best regards,
Vlad Filip
On Thu, 30 Aug 2018 at 07:54, Dennis Vriend<EMAIL_ADDRESS>wrote:

Thanks, will go to the backlog.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dnvriend/akka-persistence-inmemory/issues/51#issuecomment-417198580,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGUe4ECfikylwIqBwfaaigf2wHgRad1Qks5uV34egaJpZM4U50f0
.


Remove Admin Login

security vulnerability

removing
then will implement database for user log in, sign up and user profile

done


Added timeout to embedded replica sync.

closes issue-:#1224
Let's add a timeout to embedded replica sync so that we don't hang a sync() call if there's a network issue.

Thanks you for your contribution @xonx4l but this adds a timeout only to a test. It won't help at all with #1224.

@haaawk thanks for the feedback could you please provide me more info on the issue  ??

Closing the PR due to inactivity


update amit barletz

I will try to make a new PR later. I realized that I cannot run build locally for some reason


Issue: Photo not properly scaled

Issue description:
I notice that a lot of photo that is being showed in the gallery view isn't properly scaled. But if you view the thumbnail on the landing page then it's scaled properly.
Update:
The issue seems to only occur on photo taken vertical and not horizontal.
Update2:
Seems like photo taken with iOS portrait-mode isn't effected by this issue, but all taken with vertical mode is. Seems like the photo-tool doesn't know that it's taken vertical and scales it incorrectly.
Screenshots:

landing page (image 1 is being scaled properly here)

gallery (image 1 is not scaled properly, but image 2 is)

Environment:

OS: Server, Ubuntu/Docker:latest - Client, Windows 10 20H2
Browser: Firefox 86.0 64bit
Photo: Happens with both iPhone X and iPhone 11 (native camera app)

Other:
If you have any questions or want me to do anything else for testing please let me know. (Haven't tested the nightly build, because it didn't seemed that any commit fixed this issue)

This is a duplicate of #212 and can be fixed by correcting the exif rotation data. A tool that can help is https://github.com/Matthias-Wandel/jhead You can run jhead -autorot img.jpg

Is there nothing that could be done in PiGallery? When displaying the full photo its not laying down. Couldn't the PiGallery automatically solve this when displaying the thumbnail?

I working on #58. Once the advanced search is stable I will this and  #212.
In the meantime, feel free to jump in.
Most likely the issue is with this function: https://github.com/bpatrik/pigallery2/blob/ddf5b256f9ea4ce37ec40292959624b4b49d38e6/src/backend/model/threading/MetadataLoader.ts#L88
You can test it with:
in a "test.ts":
 import {MetadataLoader} from './src/backend/model/threading/MetadataLoader';
 MetadataLoader
   .loadPhotoMetadata('<a photo path>')
   .then(console.log).catch(console.error);

(of course you need nodejs and typescript compilation for this to work.)
If someone manages to fix it, please add a test here: https://github.com/bpatrik/pigallery2/blob/ddf5b256f9ea4ce37ec40292959624b4b49d38e6/test/backend/unit/model/threading/MetaDataLoader.spec.ts
I will merge the PR.

Lets keep the conversation in #212. Closing this as it a duplicate.


Optimization on local storage

This is a list to keep track of optimization todos related to local storage:

[ ] In-place sliding for var list column chunk. For var list, now we scan the whole column chunk from storage, and allocate a new one to copy from scanned column chunk and local vectors. This is quite inefficient as it bloats memory usage, and may trigger unnecessary read. Ideally, we should only read the whole chunk when we figure out the data column chunks needs to be resized (thus moved to another place), and sliding should happen in-place.
[x] When insert, we first append a new column chunk when inserting new nodes out of the max bound of existing node groups, and then perform updates. This should be rewritten as in place inserting a new column chunk through local storage.

Close this issue as it is stale now. For VAR LIST, we should change to a more write friendly storage layout similar to STRING.


philadelphia-core: Add 'FIXConnection#reprepare()'

Replace the existing updateCompID() method with one that feels more at home with the existing prepare() and update() methods.

Superseded by #209.


HDR10+ Profile changed with data parsed with version 0.2.5

Metadata extracted with 0.2.5..
Movie: X-Men Dark Phoenix... will upload a sample, if needed...
Source
HDR format                               : SMPTE ST 2094 App 4, Version 1, HDR10+ Profile A compatible
Encoded with x265 3.2+20
HDR format                               : SMPTE ST 2094 App 4, Version 1, HDR10+ Profile B compatible
When using metadata parsed with 0.2.4, everything is OK, and HDR10+ Profile is the same as source.

I really have no idea what the profiles mean.
The only difference is that 0.2.4 does not keep empty metadata fields in the JSON, so technically there should not be a real difference.
It was the fix for #10

Closing this as it seems like a non issue and no response.


Vulnerability scanning for the dotnet app

Check the vulnerability scanning method for the new dotnet app. We can always manually check via dotnet list package --vulnerable but we should verify to see if our existing workflow regularly kicks this off for us and if not, find a way to make that happen. Some options:

Github may notify us in the event that there is a vulnerability for a package version that we depend on: https://docs.github.com/en/code-security/secure-coding/automatically-scanning-your-code-for-vulnerabilities-and-errors/about-code-scanning. This may need some repo config tweaks
dotnet build or dotnet publish may present a warning if we are depending on a vulnerable package version.

Added the dotnet app root folder to the list of scanned apps: https://github.com/stripe/vscode-stripe/pull/297


gyp ERR! UNCAUGHT EXCEPTION

Getting the following error when trying to install Testem:
gyp ERR! UNCAUGHT EXCEPTION

I need to figure out what is causing this and fix it.

Currently fixed this issue by using an older version of Node and NPM. This is a problem with Node/NPM and not this environment setup, so I'm closing the issue.


Errors when running file : list index out of range

First off, the shebang is missing from the top, after adding
#usr/bin/env python

and running I've been getting an error.
I've tried running your script a few times to test the output, but haven't been able to get it to run properly.
Here is my output of running the script:
Enter your NCBI gene bank file name: ctg1.cluster001.gbk
2

Parsing started
1
Traceback (most recent call last):
  File "/home/seq/gbknuctest.py", line 128, in <module>
    ntgenbank()
  File "/home/seq/gbknuctest.py", line 50, in ntgenbank
    nm_version = (nm_and_version.split('.')[1]).strip('\n')
IndexError: list index out of range

The output file is blank besides the headers, but the .txt file generated is complete.
What i'm trying to get out of this ultimately is to have a csv containing every CDS and it's corresponding nucleotide sequence to add to an annotations summary file. While your script does much more than this, I haven't been able to find one that pulls the nucleotide sequence for every CDS in the file.
Not sure if this is due to an update in python or not, I'm currently running on python 2.7.15
Thanks in advance.

are you able to run with the test example file? and if its possible you can provide me how your file looks, and I will look at it.

Hi, the tool only writes info for the first sequence in the result file.

Hi, the tool only writes info for the first sequence in the result file.

are you testing on the example provided?, if so, there is only one entry on it


GitHub Sponsors?

Hi Florian,
Thanks a mill for this tool, it's very useful.
Do you have a personal GitHub sponsors button ?
I might try go and convince companies into sponsoring if, of course, you're considering it :)

Hi Jordi,
Thanks a lot, I'm really glad that this tool helps you :)
Right now I don't have a donation / sponsor button, as I did not want to get involved with the whole tax trouble that might come with it.
Do you have any experiences with that?
Kind regards,
Florian

Hi Florian,
Good question.
No idea how that works in Germany,  to be honest. In Spain you have several options to declare it in your tax returns, for example you could put it as a donation in your tax returns, otherwise it would be better to be setup as self-employed / company, like I do here in Spain.
It would advise consulting with an accountant there in Germany.
I didn't get almost any sponsors myself over several years, that's why I was asking, because that should change :)


consolidate ServiceUtils and ServiceLoaderUtils

@tsiq-karold was trying to remember myself why one version was set. My best guess is helped to prevent duplicates.


Formatting and var types specified

In relation to suggestions from @Maks3w
This is a separate PR for develop with the formatting and style changes.

Coveralls decreased possibly because of added space.

Thanks, please await for updates on other PR.  It will be for develop.


EVP_MAC_CTX_dup() only works if the ctx has been inited

According to this comment in #17064 calling EVP_MAC_CTX_dup() only works if EVP_MAC_init has been called on the ctx. This sounds like a bug.

Fixed by #17118


job-dsl extension for trigger and status publisher

Interesting, how generic push trigger will look in future.
And really interesting to see dsl because all code in this plugin was written with 'groovy based configuration' in mind.

In general feel free to merge


CH340 Compatibility

Will this be compatible with the chinese clone nano's with the CH340G chip instead of the 16u2?
Thanks

As far as I know, that chip is not programmable so it's not possible. CH34x chips directly connects the board to serial port.


HA 0.87  broke my alexa

sense updated my homeassitant to 0.87 version cant set mediaplayer alexa
2019-02-07 22:55:38 DEBUG (SyncWorker_3) [custom_components.media_player.alexa] Trying cookie from file /config/alexa_media.pickle
2019-02-07 22:55:38 DEBUG (SyncWorker_3) [custom_components.media_player.alexa] Error loading pickled cookie from /config/alexa_media.pickle: An exception of type FileNotFoundError occurred. Arguments:
(2, 'No such file or directory')
2019-02-07 22:55:38 DEBUG (SyncWorker_3) [custom_components.media_player.alexa] No valid cookies for log in; using credentials
2019-02-07 22:55:40 DEBUG (SyncWorker_3) [custom_components.media_player.alexa] Get to https://alexa.amazon.com/api/devices-v2/device was redirected to https://www.amazon.com/ap/signin?showRmrMe=1&openid.return_to=https%3A%2F%2Falexa.amazon.com%2Fapi%2Fdevices-v2%2Fdevice&openid.identity=http%3A%2F%2Fspecs.openid.net%2Fauth%2F2.0%2Fidentifier_select&openid.assoc_handle=amzn_dp_project_dee&openid.mode=checkid_setup&openid.claimed_id=http%3A%2F%2Fspecs.openid.net%2Fauth%2F2.0%2Fidentifier_
2019-02-07 22:55:40 DEBUG (SyncWorker_3) [custom_components.media_player.alexa] Preparing post to https://www.amazon.com/ap/signin/143-6557016-2207107 Captcha: None SecurityCode: None Claimsoption: None VerificationCode: None
2019-02-07 22:55:41 DEBUG (SyncWorker_3) [custom_components.media_player.alexa] Error message: Important Message!
            To better protect your account, please re-enter your password and then enter the characters as they are shown in the image below.

i have also deleted alexa_media.pickle   file to see if help and nothing

Do you see the configurator in your notifications? The logs are saying Alexa is asking for a captcha.

Yes its there, but nothing

I don't understand what you're seeing. What happens if you enter the captcha in the configurator? Are you seeing what you enter in your logs?
For example, from what you posted it's reporting Captcha: None SecurityCode: None Claimsoption: None VerificationCode: None, which means at this stage no data is being collected by the configurator.  As you enter data in the configurator, what you enter should appear in that line.

(Just chiming in to say that upgrading to 0.87 did not affect Alexa mediaplayer for me.)

i enter the captcha  press and screen goes away and just stills in the configurator to be set up, will not generate new entity

I have the same problem I see the notification that asks me to select enter the new captcha I enter the code it seems to accept it, however none of my Alexa devices are shown under the entires

If you keep seeing configurators that means Amazon is rejecting the input and not letting you log in. To understand why will require reviewing the logs and perhaps advanced debugging info (which should be done in a private gist and after scrubbing for emails/passwords).

@alandtse it seems that was Amazon  rejecting the password and not letting me log in.
thanks alot mate


Error in v3 test: Errorf format %q has arg out of wrong type

v3.0.5 on Go 1.12.5 Fedora Rawhide x86_64:
Testing    in: /builddir/build/BUILD/rethinkdb-go-3.0.5/_build/src
         PATH: /builddir/build/BUILD/rethinkdb-go-3.0.5/_build/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/sbin
       GOPATH: /builddir/build/BUILD/rethinkdb-go-3.0.5/_build:/usr/share/gocode
  GO111MODULE: off
      command: go test -buildmode pie -compiler gc -ldflags "-X gopkg.in/gorethink/gorethink.v3/version=3.0.5 -extldflags '-Wl,-z,relro -Wl,--as-needed  -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld '"
      testing: gopkg.in/gorethink/gorethink.v3
gopkg.in/gorethink/gorethink.v3/encoding
# gopkg.in/gorethink/gorethink.v3/encoding
./decoder_test.go:435:3: Errorf format %q has arg out of wrong type *gopkg.in/gorethink/gorethink.v3/encoding.UnmarshalerPointer
FAIL    gopkg.in/gorethink/gorethink.v3/encoding [build failed]

Probably affects master to since that code hasn't change since 3.0.5

It's golang changed behaviour. In go1.8 (latest go for gorethink.v3) it wasn't an error.
I recommend to update to v5, it has some major bug fixes.

Sadly github.com/theupdateframework/notary is still using v3 in their latest release.

It was already fixed in develop branch. I believe soon it will be merged to master, but it can be v6.


Running from s3

I would like to run from s3 in order to minimize deployment package sizes. But I am not sure what's the recommended way to do so. Should I unpack it all? Thanks for all recommendations.

@nonyy You should look into this https://github.com/alixaxel/chrome-aws-lambda#aws-lambda-layer


cigien: Watch rhythmtravel2000.com

cigien requests the watch of the watch_keyword rhythmtravel2000\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

rhythmtravel2000\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


SharedArrayBuffer feature request

Currently SharedArrayBuffer is disabled in Launcher, due to the current cross origin policies. This is a feature request to enable its support.
Additional info: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/SharedArrayBuffer#security_requirements

Let me take a look at this and have a chat with the team. I'm keen to have this feature but just need to run through any implications.

@LeXXik Just wondering, have you got any specific use case in mind for this?

I have physics running in a webworker. The state of the physics world is currently not shared with the main thread, which makes some operations, like syncing entities with bodies, more expensive than I'd like them to be.

Same here, using webassembly with threads require coop + coep for sharedarray buffer

Can this be optional in project settings in the Editor?

@isumygin-sc I think it's only if those 3rd party resources need to use SharedArrayBuffer. Which they shouldn't as it'll be unavailable for them currently.
I think they do need to have CORS set though

Can this be optional in project settings in the Editor?

You read my mind

Any idea when this work will be undertaken yet?

We hope to get to this within the next few weeks. Does that work for you?

We hope to get to this within the next few weeks. Does that work for you?

That will work for us. Thank you for the update.

the feature is implemented, will be deployed soon

it's deployed to PROD, please test it there. Don't forget to enable the feature in project settings first.

Not sure where to find that setting.

Worked perfectly, thank you!


Make port allocator not depend on order of ports

Currently the port allocator depends on the order of the ports in the
Spec and the Endpoint to be the same to determine if it is allocated or
while reconciling ports. This results in unnecessary false negatives
while testing for allocated and go through unnecessary
allocations. Fixed this by treating the list of ports as a set and check
if the same set of ports are available in the Endpoint as in Spec.
Signed-off-by: Jana Radhakrishnan<EMAIL_ADDRESS>

Do the unnecessary allocations only happen if the ports in the spec get reordered somehow, or is this a problem in normal operation (i.e. task Endpoint ends up with ports in a different order, so allocation repeats until the order happens to match in a sort of bogosort)?

Current coverage is 56.10% (diff: 85.71%)

Merging #1719 into master will increase coverage by 0.14%

@@             master      #1719   diff @@
==========================================
  Files            93         93
  Lines         14775      14802    +27
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           8268       8304    +36
+ Misses         5424       5409    -15
- Partials       1083       1089     +6

Powered by Codecov. Last update 9bee31d...e1557e5

Do the unnecessary allocations only happen if the ports in the spec get reordered somehow, or is this a problem in normal operation (i.e. task Endpoint ends up with ports in a different order, so allocation repeats until the order happens to match in a sort of bogosort)?

This happens only when spec gets reordered because Endpoint will always follow the spec order. This especially happens from cli which during inexplicably sorts the ports in the spec during a service update while  it does not during service create. Nevertheless this is something that can happen if someone is using the api directly and decides to update the spec with a different order in ports.

LGTM

LGTM


tsd install --save mocha doesn't actually get mocha

It says "written zero files" and the "installed" value in tsd.json doesn't get updated.
What's wrong?

Seems it expects tsd install mocha --save, with the --save flag notably at the end.


Cannot resolve Identity from identityId. Dispatching as Identity.SIGNED_OUT

Needed Version 1.2.2 of the YouTube Player API.
Videos are not playing in Android.
Please update the react-native-youtube npm package.
Urgent....!!!!!!

now using 1.2.2 4d65914


Add hal::set_exception_allocator(Callable&)

Such a callable may have a signature such as this:
void* memory_handler(size_t p_object_size, bool p_allocate);

Bool parameters are not advised but this sells the point. If allocating, the void* is returned out from __cxa_allocate_exception.  If deallocating, then the return value is ignored. Allocating exception memory should set the data to all zeros before returning it. Memory CAN be memset to zero for deallocating the memory, but isn't required. This is the base idea. This can change as implementation begins.

I don't remember the rationale or motivation for this. I think using PMR is the right choice even though its a bit annoying to deal with the fact that PMR provides you the size as well.

Replaced by https://github.com/libhal/libhal-exceptions/issues/20


numeric_state automation platform

Can you add some tests? The automation component is fully test covered

One style issue which is an unused import:
homeassistant/components/automation/numeric_state.py:10:1: F401 'MATCH_ALL' imported but unused

I'll add some unit-tests and update the pull.

You can run the tests locally easily by running scripts/run_tests which takes only a couple of seconds :-)

Awesome, thanks!

I have just added if-conditions in #364, you should now be able to use the automation rules you wanted :+1:

Yay! Good work, thanks!


[BUG] Missing locale

@ouranos @BrunoChauvet Updates missing locale for app settings modal. Please review.

👍


Remove function

When I remove a function and re-deploy, the old functions continue to exist on Twilio, and there doesn't seem to be a UI in the Twilio dashboard to remove a function deployed via the new functions API.
How does one remove a deployed function?

You are correct that when you redeploy with the Function file removed that the Function Resource and previous versions will stick around. It won't be accessible anymore and it's not deployed anymore. But previous versions are still stored by Twilio (not active though). That's on purpose because it's hard to tell for us whether you actually want it gone. The way you can delete it is by deleting it through the API directly or via other Twilio CLI commands.
There might be a couple of steps that you have to do though.
You can only delete a Function resource if the there is no Function Version that's currently active in any previous build. And Builds can be active on multiple Environments.
As a result we don't end up deleting Function resources for you because we could only do that if it wasn't active on any other environment that you have and that behavior might become to unpredictable. Plus it wouldn't allow you to rollback to a previous version anymore in case the things you did were wrong.
The docs I linked you to show the different API commands including how to do them in the Twilio CLI.
The steps you have to do are:

get a list of all Environments
get the list of all currently active Builds
check the Function Versions that are bundled in any of those Builds
check if any of the Functions is associated with the Function resource you are trying to delete
change any environments that might have to have new Builds associated that don't contain the Function
then you should be able to delete the Function resource.

I hope this helps.


Fix typo in CHANGES.md file

Lastest version was indicating v1.9.0 in CHANGES.md, but v1.8.0 is the current latest tag.

Hey, thanks for the PR. However, this was no typo, the 1.9.0 entry is a preparation for the upcoming release. The 1.8.0 release mainly contained some internal infrastructural changes, not really interesting for consumers of the library:


test(ACC): use deterministic test props

The random prop generator sometimes generated duplicate props.  There are tests that assert certain props were not present.  Since there are sometimes duplicates, the check for not present would fail since only one copy of the prop was removed while its duplicate remained.
This PR replaces makeProps with a static props object.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1309   +/-   ##
=======================================
  Coverage   99.75%   99.75%
=======================================
  Files         140      140
  Lines        2401     2401
=======================================
  Hits         2395     2395
  Misses          6        6

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 42ed567...d6f6fd4. Read the comment docs.


clojure upgrade tests

Determine if there are any compilation/test issues when upgrading to the latest clojure beta release (1.10.0-beta3).

Wrong repository.  Closing...


Add Python documentation

Currently, only the file handler documentation is missing. Is this wrapped yet, in webui-python?

Should the myWindow variable used throughout be changed to snake-case (myWindow -> my_window)?

Let me make additional changes, I intend to make the variable names snake cased

Ready to be merged now 👍🏾

hm, odd

@jinzhongjia can you try approving this pr?


Replace react-photo-gallery

Hi @skogsmaskin,
I came across this repo while searching for active projects using react-photo-gallery library. react-photo-gallery hasn't been maintained for almost 3 years and still has a hard peer dependency on React 16. At this point, it's safe to assume its author abandoned the library for good.
I'm the maintainer of react-photo-album library, a successor to react-photo-gallery. react-photo-album is very similar to the original library and requires just some minor tweaks to migrate.
Please let me know if you are interested, and I'll be happy to assist with PR.

Hey @igordanchenko!
This is great, thank you for succeeding the library. We are very interested with your assistance.

@skogsmaskin, the PR is ready.

@igordanchenko - sorry for the late reply due to covid. Thank you for this!

@igordanchenko - a dedicated team at our company has taken over the responsibility for our plugins and such (yes we have been growing a lot lately).  I have notified them to follow up on your PR!

@igordanchenko - great work :) I've tested the changes and upgrade-path, it all looks good. Off to merging and releasing a new version. Thank you!


Error: missing plugin for distribution 'ubuntu-20.04'.

By updating proot-distro this error gets occurred
How to get backup of this distro

Well, i found a way to install it because apparently you can't downgrade proot distro.
In modded-ubuntu, use command nano setup.sh and replace ubuntu-20.04 with ubuntu. It should work perfectly

The all installation & projects going to vanish
I cleaned my termux & installed ubuntu from proot-distro

The all installation & projects going to vanish
Really? Well, i guess i didn't notice it because i am starting with clean slate haha.


[nxp][examples][common][lock-app] Add DiagnosticLog and Update zap file

Add DiagnosticLogs in lock-app ZclCallbacks.cpp
Update zap file: enable door-lock attributes

Is the Build on Darwin (clang, simulated) test a known issue? I saw it in other PRs too.


chore(ci): switch to pyinstaller to build single binary application

Codecov Report

Merging #99 (a2d6177) into main (2e16230) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #99   +/-   ##
=======================================
  Coverage   93.71%   93.71%
=======================================
  Files          15       15
  Lines         668      668
=======================================
  Hits          626      626
  Misses         42       42

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 2e16230...a2d6177. Read the comment docs.


[EAV] Value::setAttribute add return this

:+1: I like to add a return $this, but I don't like test complicationю

Looks good now that the unit test hasn't changed. I'll merge it in. Thanks!


Event Feed Grouping error on Acceptance

This issue was originally reported by @lancewf
Note: This issue is still in the product/ux phase and is not ready for engineering work, though, as always, comments are welcome.
Overview
The number of environment events is not correct on acceptance. Below is a gif showing the problem. The select tool displays that there are 126 environment events. The feed shows 100 grouped together. Then when loading the last 26 events it loads another 100 in a group. If you look in both groups you will see duplication of events.
My guess of what the problem is. The load more button should trigger a request that sends a before from the last item in the group. Either the grouped end_date is correct or the UI is not using it.
Screenshots

is this still a problem?

this work has been deprioritized.


Is there a way to avoid current patch comparison? Shout out all offences

Hey, well done with your plugin! but we're having an issue with rubocop offences due to you're compaing current patch added lines to offence lines currently pronto doesn't add any comments for the previous commits or any offences that are not related to the current patch. Would you please explain or is there  a way to avoid this? I tried to make PR with a ENV variable but I get duplicate results when I skip .select { |line| line.new_lineno == offence.line }
Thanks

Also it doesn't consider .rubocop.yml file maybe that's the issue, I'm not sure.

@azbshiri hey, thanks for reporting this. But I'm not sure if I understand the issue correctly. Could you give a specific example of when it doesn't work as you would expect?

@mmozuras I tried to write a prettier plugin for pronto but it turns out I have to somehow get the lines that have issues and use them to return warnings/errors, etc' e.g. I just want to run prettier against a specific kind of files and return error if prettier exists with 1 or vice versa.


Design

The preliminary design stage is finished, the following - code.

I am pleased with myself !


Refactor connlib callbacks to diff / emit state when any of ip, dns_servers, routes change

In connlib, we emit various callbacks when one of the above mentioned network config elements change. Instead, we should emit a generic on_network_state_change including our entire state of these.
We should diff the old state with new state and only emit this callback when that state has changed, to avoid tracking this in each platform-side codebase.

Another way of phrasing this is to merge on_update_routes into on_set_interface_config. We should probably introduce a struct here to make it easier to pass this data around.

https://github.com/firezone/firezone/pull/6427 paves the way for doing this. We now just need to add the routes to that as well.


Split string according to delimiter and create multiple fields for the elements

Does this cover all record types?


Remove Attribute application@allowBackup value

It's for an application not a library ?
Error:Execution failed for task ':app:processDebugManifest'.
> Manifest merger failed : Attribute application@allowBackup value=(false) from AndroidManifest.xml:16:9-36
    is also present at [com.ernestoyaquello.stepperform:vertical-stepper-form:0.9.9] AndroidManifest.xml:12:9-35 value=(true).
``

Hi, I add the same problem and had to for the merge using tools:replace="" attribute (more info from Android-dev ).
I submitted the PR #28 which fix the problem, so you should wait for a new release :).

First of all, sorry for the immense delay in my response; after a long period of abandonment it is only recently that I have found the time to work on this library again.
Regarding this issue, this is no longer a problem in the new version of the library.


create a util function for checking keyCodes in core

Describe the bug
We have a couple of places where we are doing checks to see if a key pressed is a certain key (ESC, ENTER, etc). Let's refactor this into a reusable function.
Expected behavior
Instead of having checks like:
if (e.keyCode === ESC || e.key === 'Esc') {
  ...
}

where e is of type KeyboardEvent we could have a reusable function, e.g.:
if (keyEquals(e, Keys.Escape)) {
   ...
}

And the function would check against our dictionary of tuples to check the key in a cross-browser compatible way.

We need to get #4479 merged before we do this...


[BugFix] Fix ambiguous exception when type mismatch in fillnull command

Description
Fix ambiguous exception when type mismatch in fillnull command
Related Issues
Resolves https://github.com/opensearch-project/opensearch-spark/issues/959
Check List

[ ] Updated documentation (docs/ppl-lang/README.md)
[x] Implemented unit tests
[x] Implemented tests for combination with other commands
[ ] New added source code should include a copyright header
[x] Commits are signed per the DCO using --signoff

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

@qianheng-aws @LantaoJin What is the expected GA spark version we can assume this to be released with ?
should we add a notice in the release for 0.7 regarding this ?

@qianheng-aws @LantaoJin What is the expected GA spark version we can assume this to be released with (~3.5 ? ) ? Should we add a notice in the release for 0.7 regarding this ?

This patch should be released in Spark v3.5.4 and v4.0.0. I have no idea about the date of release. I think this PR resolves the ambiguous issue so we don't need a notice in release any more.


Include dependencies to compile OpenSC and fc2 in doc/create-CVN.md.

I'd rather create separate files for distribution/system specific build instructions and leave this one CVN centred and platform-agnostic.


Pass state between pages

I'm having trouble determining how I would share state between page components. Since they are top-level components, I don't believe there's a way to do a Flux or Redux style architecture where data comes in through a top-level component and gets passed to children via props. Am I missing something?
For example, I have a Profile "page" component where a user can log-in. I am setting userId state in that component, but I want that state available to other page components. Is this possible?

It's very easy to use Redux or Apollo with Next, here are some examples:
https://github.com/zeit/next.js/tree/master/examples/with-redux
https://github.com/zeit/next.js/tree/master/examples/with-apollo-and-redux
using getInitialProps you can create the store in both server and client, but in the client you can do something like:
let reduxStore = null

export const initStore = () => {
  if (process.browser && !reduxStore) {
    reduxStore = createStore()
  }
  return reduxStore
}

the example with-apollo-and-redux can help you a lot.

Thanks for pointing me in the right direction, @Goluis.

Thanks @Goluis.
Closing this since, we've an answer.


(Question) Why "where" clause was removed?

Hi there!
I am just curios why the "where" clause was removed. It seems to be quit useful, it allows to have the algorithm described first and the implementation details second, feels natural for me. Where there any logical reasons for removing it?

#659 582


Use GitHub action runner instead

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Yes this is the run from my own fork using ubuntu-latest runner https://github.com/sweatybridge/postgres/actions/runs/3856067392/jobs/6571831699. The run built the image in 30 mins but failed to push because I didn't give it the token.

I think with ccache, the runner doesn't need as much cpu as before.

https://github.com/sweatybridge/postgres/actions/runs/3860186275/jobs/6580353194 here's a successful run without push.
I suspect we are missing some configuration for our self hosted x86 runner. May be the docker credentials were not mounted as secrets?

I think with ccache, the runner doesn't need as much cpu as before.

Wouldn't that cause things to fail again if a PR changed enough things that the cache couldn't be re-used?
I'm happy to approve this if you'd like, but it doesn't sound like we understand the root cause for the failures, given things were building OK three weeks ago.

Thanks, I will check with @pcnc next week. May be the runner had an old version of ccache image in its docker cache.


Error playing sound

Can't get sound alerts to work, have tried .wav and .m4a files, both in .resources/sounds/ and .resources/sounds/true/
filenames are alert.wav and alert2.m4a
This is while testing notifications with NOTIFICATION_TEST set to true
Saw no more config information in any readme so asking here

Okay, I can look into this.
Thanks!

This is the error that I get btw:
"[2:07:56 PM] info :: ✔ playing sound
[2:07:56 PM] error :: error playing sound: 1"

Potentially solved in #91

Okay, I see the problem. Yes, #91 fixes this. I need to update the docs to say how to set this variable. Not very specific.


MINOR: Maybe decorate inner topics for SourceNode

When creating the source node in TopologyBuilder, we need to decorate its input topics if they are inner (i.e. repartition) topics with the prefix.
Also did some minor cleanup in the printing function for better visualization in debugging.

@dguy @mjsax @enothereska for reviews.

test this.

test this

please test this

Example output with two sub-topologies:
ProcessorTopology:
	KSTREAM-SOURCE-0000000000:
		topics:		[topic1]
		children:	[KSTREAM-MAP-0000000001]
	KSTREAM-MAP-0000000001:
		children:	[KSTREAM-FILTER-0000000004, KSTREAM-FILTER-0000000007]
	KSTREAM-FILTER-0000000004:
		children:	[KSTREAM-SINK-0000000003]
	KSTREAM-SINK-0000000003:
		topic:		X-Counts-repartition
	KSTREAM-FILTER-0000000007:
		children:	[KSTREAM-SINK-0000000006]
	KSTREAM-SINK-0000000006:
		topic:		X-KSTREAM-MAP-0000000001-repartition

ProcessorTopology:
	KSTREAM-SOURCE-0000000008:
		topics:		[X-KSTREAM-MAP-0000000001-repartition]
		children:	[KSTREAM-LEFTJOIN-0000000009]
	KSTREAM-LEFTJOIN-0000000009:
		states:		[Counts]
	KSTREAM-SOURCE-0000000005:
		topics:		[X-Counts-repartition]
		children:	[KSTREAM-AGGREGATE-0000000002]
	KSTREAM-AGGREGATE-0000000002:
		states:		[Counts]

Test failures look like they are legit. Same in each build:
org.apache.kafka.streams.processor.internals.StreamThreadTest.shouldCloseActiveTasksThatAreAssignedToThisStreamThreadButAssignmentHasChangedBeforeCreatingNewTasks

Addressed commends. Also fixed the issue when build() function is called before the topics metadata has been refreshed for pattern subscriptions (this should only happen when users explicitly call builder.build(), during debug / testing).

Merged to trunk.


properly support utf16le, add sanity check for categories

Hi there! I've picked up your (absolutely awesome, by the way) yet-another-factory-planner off the Satisfactory discord, and went ahead to look at it, and perhaps tinker and improve. But doing all the deployment steps, I encountered quite some issues on the npm run parse-docs step, and so went on to look at the parser first. This PR is basically the result of me finally pushing through the deployment steps for the planner, and having parser working without errors on fresh and unmodified Docs.json.
The changelog is as follows:

Buffer.toString() doesn't seem to handle BOM correctly (it can handle utf-8 without BOM, but anything else causes issues). TextDecoder does handle all that properly, so used it instead.
non-existing categories (color gun, ficsmas event stuff, etc) caused undefined values to appear in categoryClasses, and downstream code can't handle that, thus stopping with an error. Added a filter to remove any potential undefined values.

Hi, thanks for your contribution! And well done for navigating my spaghetti, lol. Tbh the site is still using the docs file from U4 currently so I have not tested what breaks for the new U5 docs file, that's something I'm gonna be working on in the next few days, as well as adding ficsmas to the calculator site. I'll go ahead and merge your changes in. That BOM thing was something that had stumped me early on so you can see my solution was to just snip it out, lol... Good tip about TextDecoder.
I will also be updating the readme for this tool until I have it properly working for update 5 so other people will know what's up.


Adding support to Brave browser.

Hi!
Really nice workflow, thanks for sharing.
Any plan on adding Brave support ?
I stopped using Google Chrome few months ago as I found it getting slower and slower and would love to ba able to use your workflow too.

I am sorry but I have no plan for minor browsers. If the browser supports Open Scripting Architecture, you could fork this repository and try your favorite browser by adding some codes in ease like #6. Chromium based browsers will work maybe.

I just deployed new version for supporting Chromium based browsers.
If you want to add your favorite browser, follow this steps.


__cpp_lib_three_way_comparison is not defined in clang 8 or 9

iterator_interface.hpp doesn't compile on my clang 8.
As a local kludge, I just renamed all references  to __cpp_lib_three_way_comparison with a legal #define variable set to 0
Great iterator interface, I love it!
Arthur

Thanks!  The pattern __foo is reserved by the standard, and in particular a macro that starts with that (or is it "__cpp" -- I forget) is supposed to evaluate to false if the compiler has never seen that macro.  This is to make checks for such macros well-formed under versions of C++ from before they were introduced.  This also passes the tests in Travis.  Are you possibly running an oddly=configured build?

On Tuesday, 17 December 2019 11:46:11 AM AEDT Zach Laine wrote:
Thanks! The pattern __foo is reserved by the standard, and in particular a macro that starts
with that (or is it "__cpp" -- I forget) is supposed to evaluate to false if the compiler has
never seen that macro. This is to make checks for such macros well-formed under versions
of C++ from before they were introduced. This also passes the tests in Travis. Are you
possibly running an oddly=configured build?
Yes, others have said I am odd...
../stl_interfaces/iterator_interface.hpp:13:16: error: '__cpp_lib_three_way_comparison' is
not defined, evaluates to 0 [-Werror,-Wundef]
So, I could add -Wno_undef to my compile line. But that error line perhaps implies a naked
__cpp_lib_three_way_comparison may be deprecated, or it could be a clang thing.
In my little world, I'm lucky the only macro I need is #if NDEBUG, so I have thankfully
forgotten all about them and where they fit in the evolving Standard.
Again, these are all tiny issues. I do love how you have factored out much of the drudgery
in making iterators. It is now permanently in my toolkit.
Many thanks,
Arthur

Ok, thanks for clarifying.  I'll probably just add a defined() around those so that users with those flags are not caught in pointless compilation errors.


Error Failed to get app runtime OS

My deployments started failing today with the following error.
Error: Deployment Failed with Error: Error: Failed to get app runtime OS
I'm running my WebApp in Linux containers and until today everything worked fine. I tried to update my publish profile but that didn't help. Actions workflow was autogenerated by Azure console when setting up deployments. But as I said, that exact config was working a couple of days ago.
There are not many logs in the output so it's hard to figure out what's going on. Any hint, please?

This is the debug output
##[debug]Could not parse response body.
##[debug]{}
##[debug]Encountered a retriable status code: 503. Message: 'Service Temporarily Unavailable'.
##[debug][GET] https://***.scm.azurewebsites.net:443/diagnostics/runtime
##[debug]Could not parse response body.
##[debug]{}
##[debug]Encountered a retriable status code: 503. Message: 'Service Temporarily Unavailable'.
##[debug][GET] https://***.scm.azurewebsites.net:443/diagnostics/runtime
##[debug]Could not parse response body.
##[debug]{}
##[debug]Encountered a retriable status code: 503. Message: 'Service Temporarily Unavailable'.
##[debug][GET] https://***.scm.azurewebsites.net:443/diagnostics/runtime
##[debug]Could not parse response body.
##[debug]{}
##[debug]Encountered a retriable status code: 503. Message: 'Service Temporarily Unavailable'.
##[debug][GET] https://***.scm.azurewebsites.net:443/diagnostics/runtime
##[debug]Could not parse response body.
##[debug]{}
##[debug]loaded affinity cookie ["ARRAffinity=f2e7c40719d65274f71e47ab410e8962551aca6eb1a4ecd4d8fee5af422a7912;Path=/;HttpOnly;Secure;Domain=***.scm.azurewebsites.net","ARRAffinitySameSite=f2e7c40719d65274f71e47ab410e8962551aca6eb1a4ecd4d8fee5af422a7912;Path=/;HttpOnly;SameSite=None;Secure;Domain=***.scm.azurewebsites.net"]
##[debug]getAppRuntime. Data: {"statusCode":503,"statusMessage":"Service Temporarily Unavailable","headers":{"content-length":"19","server":"nginx","set-cookie":["ARRAffinity=f2e7c40719d65274f71e47ab410e8962551aca6eb1a4ecd4d8fee5af422a7912;Path=/;HttpOnly;Secure;Domain=***.scm.azurewebsites.net","ARRAffinitySameSite=f2e7c40719d65274f71e47ab410e8962551aca6eb1a4ecd4d8fee5af422a7912;Path=/;HttpOnly;SameSite=None;Secure;Domain=***.scm.azurewebsites.net"],"date":"Sat, 12 Dec 2020 23:41:17 GMT","connection":"close"},"body":"Service Unavailable"}
##[debug]Failed to fetch Kudu App Runtime diagnostics.
##[debug]Service Temporarily Unavailable (CODE: 503)
Error: Deployment Failed with Error: Error: Failed to get app runtime OS

And I indeed had problems with accessing Kudu services - I was receiving status 503 when I tried to access Advanced Tools, Log Stream, SSH, or Deployment Center (logs).

I have the same problem.

I have the same problem.

Hello @Duchynko , can you share how you were able to resolve the issue?

how did someone close this issue without a single reply even? Also problems persists in 2024


Confusion about 2D data processing

Your work is very interesting! I have run into some problems.
When processing 2D data, I followed the instructions and adopted the Online3D processing method. However, the 2D data processed by Online3D did not have such information as"./2D/scene0145_00/pose"
I want to know how to get the directory structure of ScanNet-MV:
scannet-mv
├── 2D
│   ├── scenexxxx_xx
The error message is as follows

Hi,
You can try this link: https://cloud.tsinghua.edu.cn/d/f9dc0ae45f2f4666a209/. The downloaded data contains pose folder.


Make some column/row fixed

How to freeze row/column fixed in table?
By using current functionality we can freeze headers but can not freeze any row / columns.
I tried TableHeadFixer but, If I use this plugin then I am getting multiple errors in JExcel functionality.
So I can not merge both the functionality of TableHeadFixer and JExcel

This is already included in the roadmap. So, I am going close this one.

I would propose allowing sticky-rows... Here is a rough specification:

Sticky to bottom
This would need to then be bypassed by the COLUMN SORT function, and also the "ENTER" key that allows adding new rows... so if the row is sticky to bottom, any time you get to the bottom and hit enter to create a new row at the bottom, the new row will show up above the "sticky to bottom" row. This would also require the formula inside the cell for the bottom row to automatically update to include the additional row in the formulas like SUM, etc.

Sticky to top
This would need to then be bypassed by the COLUMN SORT function

Sticky to top/bottom of group of rows
This would need to then be incorporated into the COLUMN SORT function, to sort rows inside each group, but not to mix them together, and to maintain the sticky row at the top/bottom of the group.

OPTION TO ADD TO FUNCTIONALITY OF #3 ABOVE
I propose modifying the "ENTER" key function when you are at the bottom of a group (before the sticky row), to add a new row to the group, similar to "sticky to bottom" functionality, except specific to each group. Then, to navigate without adding a new row, the user would rather use the arrow keys.


Implement dpmpy tag. Fixes #69

Coverage increased (+0.1%) to 94.545% when pulling 071a0879958c13ca276c93709be5b0c267591ba6 on Fak3:f69_tag into 09ee3e8aa9382e07bc4741b89724ec601b3b8938 on frictionlessdata:master.


Timeout if client goes to sleep

If the client PC, tablet, whatever goes to sleep the idle timer goes to sleep too. When it wakes back up it seems that the application detects that as activity so the timeout doesn't happen, even if it was asleep longer than the timeout period. I need to set my app up so that it always goes to the login screen when the timeout expires. if the device goes to sleep after say 5 minutes the timer still tracks from the last activity. If the timeout is set to 30 minutes and they come back after 10 they will continue where they left off. However if they come back after an hour they should immediately be redirected to the login screen. Is this possible?

What version of ng-idle are you using? It should be doing that since v1. It stores a timestamp of the last activity, and upon interrupting, it checks to see if the idle interval was exceeded, and should time you out.

Thanks for the quick response. I am using the latest version. I put together a small test app and it seems to work OK when my tablet to goes to sleep. I'm looking into my app to see why it's not doing what I expect. I'll let you know if I need more help,


docs(components): add properties for group 5 web components

Description

Updated documentation for web components in group 5
What's included?

Components updated:

Icon button toggle
Treelist
Text area
Text field
Text lockup
Typography

Test Steps

[x] Run start task for knapsack in nx console
[x] Open the localhost url for knapsack
[x] Check the docs for above mentioned components

General Tests for Every PR

[x] npm run start still works.
[x] npm run lint passes.
[x] npm run stylelint passes.
[x] npm test passes and code coverage is not lower.
[x] npm run build still works.

Screenshots or link to StackBlitz/Plunker

:tada: This PR is included in version 8.12.1 :tada:
The release is available on:

v8.12.1
GitHub release

Your semantic-release bot :package::rocket:


After migration to .net8 SignalR stopped to work

Describe the bug
Recently we migrated our solution from .net6 to .net8. After that SignalR stopped to work. There were not code changes - only migration of the .net version + upgrade of all nuget packages.
To Reproduce
This is the SignalR configuration:
public class Startup
{
    public void ConfigureServices(IServiceCollection services)
    {
        //...
        services.AddChatEngineDependencies(_configuration);
        //...
    }

    public void Configure(IApplicationBuilder app, IHostEnvironment env)
    {
        //...
        app.UseChatEngine();
        //...
    }
}

public static class ServiceCollectionExtensions
{
    public static IServiceCollection AddChatEngineDependencies(this IServiceCollection services, IConfiguration configuration)
    {
        services.AddControllers();
        services.AddSignalR()
            .AddAzureSignalR(configuration.GetConnectionString("AzureSignalR"));
        //...
    }
}

public static class ApplicationBuilderExtensions
{
    public static IApplicationBuilder UseChatEngine(this IApplicationBuilder app)
    {
        app.UseEndpoints(endpoints =>
        {
            endpoints.MapHub<ChatHub>("/hub").RequireAuthorization();
        });
        return app;
    }
}

Exceptions (if any)
vendor-957f9e95.js:508 [2024-05-06T10:25:40.633Z] Error: Server returned handshake error: SignalR Service is now in 'Default' service mode. Current mode works as a proxy that routes client traffic to the connected app servers. However app servers are not connected.
log @ vendor-957f9e95.js:508
_processHandshakeResponse @ vendor-957f9e95.js:508
_processIncomingData @ vendor-957f9e95.js:508
HubConnection.connection.onreceive @ vendor-957f9e95.js:508
it.onmessage @ vendor-957f9e95.js:508
vendor-957f9e95.js:508 [2024-05-06T10:25:40.634Z] Error: Connection disconnected with error 'Error: Server returned handshake error: SignalR Service is now in 'Default' service mode. Current mode works as a proxy that routes client traffic to the connected app servers. However app servers are not connected.'.
log @ vendor-957f9e95.js:508
_stopConnection @ vendor-957f9e95.js:508
transport.onclose @ vendor-957f9e95.js:508
_close @ vendor-957f9e95.js:508
it.onmessage @ vendor-957f9e95.js:508
vendor-957f9e95.js:508 Uncaught (in promise) Error: Server returned handshake error: SignalR Service is now in 'Default' service mode. Current mode works as a proxy that routes client traffic to the connected app servers. However app servers are not connected.
    at HubConnection._processHandshakeResponse (vendor-957f9e95.js:508:35675)
    at HubConnection._processIncomingData (vendor-957f9e95.js:508:34368)
    at HubConnection.connection.onreceive (vendor-957f9e95.js:508:28393)

Further technical details

Before the migration we were able to see:

[09:27:19 INF] Hub 'ChatHub' is now connected to '(Primary)https://test-communication.service.signalr.net/(hub=ChatHub)'.

After the migration it is not logged anymore and the exception above is logged in browser console.

Ok, I've found an issue myself. The problem was the fact that in Configure method UseEndpoints was called twice - from within Configure and from UseChatEngine. Once I moved the logic to single UseEndpoints it started to work again.


Add security to migration (Old 263)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:08
Add the starting security levels to the migration.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:13
Done in the V2 release.
ALawrence66 - 2015-02-24T23:27:13.133833+00:00


Adds f32.mul, f32.div for vcode backend for x64.

Adds support for lowering clif instructions Fdiv and Fmul
for new vcode backend.

@julian-seward1 @bnjbvr This push was not intended to be a new PR. Instead this was supposed to be an update to #1797. This push was intended to address @bnjbvr latest comments.


Using the Box component with the clone attribute has inconsistent styling in production vs dev

The resulting styles when using the Box component with the clone attribute are created differently in production than they are in development mode.
Ideally the Box component would be able to override styles in production mode like it does in development, but at the very least it should be the same in both modes.
With the current limbo of jss and possible conversion to a different css-in-js library (https://github.com/mui-org/material-ui/issues/16947) I have been trying to use the Box component as much as possible to reduce the amount of change needed if material-ui decides to change. So it would be best if the Box component was able to override original styles, and behave consistently between environments.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Seems related to https://github.com/mui-org/material-ui/issues/16609 , but that was already closed.
Current Behavior 😯
The resulting styles are created differently in production than they are in development mode, with the production mode not allowing the Box component using the clone attribute to override already defined styles like background-color.
Production: Card has white background
Development: Card has Purple Background

Expected Behavior 🤔

It should work the same in production as it does in develop, with the <Box clone bgcolor="primary.main"> overriding the child component <Card>'s (or any other material component's) original styles.
Production: Card has Purple Background
Development: Card has Purple Background
Steps to Reproduce 🕹

Steps:

Use the Box component with the clone attribute to override a style on a material-ui component like the Card.
Override a style like bgcolor which is already defined on the child component.
Check the development version where it works
Check the production version where is doesn't

Code:
// import condition matters!
import { Box, Card } from '@material-ui/core';

const ConnectedApp = () => {
  return (
    <Box clone bgcolor="primary.light">
      <Card>Foo</Card>
    </Box>
  );
};

Dev Output:

Foo

Prod Output:

Foo

Here you can see the dev build: https://codesandbox.io/s/quizzical-dew-fi3qw
Here you can see the prod build: http://codyschaaf.com/box-demo/
(prod repo: https://github.com/CodySchaaf/box-demo)
Context 🔦

I'm trying to rely more on the Box component instead of makeStyles because of the uncertain future of jss. Also I'm starting to prefer the ease of use that comes with the Box component and its helper shortcuts. But when trying to add styles to existing components it isn't always consistent in its behavior.
Your Environment 🌎

Tech
Version

Material-UI
v4.9.14

React
v16.13.0

Browser
Chrome v81.0.4044.138

TypeScript
3.9.2

Duplicate #18496


[BUG] Warkaround attr allocation.offset assertion failure

https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/10087337167

E2E CI testing: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/10087755509

It seems a workaround instead of bug fixing.
The scratch space in SLM per each function is pre-arranged on transforming the tt.func to llvm.func.
We need to align to the upstream code as much as possible in order to upstream the Intel backend in the future.
What is the failure case? And we need to find the root cause and fix it.

It seems a workaround instead of bug fixing.
The scratch space in SLM per each function is pre-arranged on transforming the tt.func to llvm.func. It would cause issue which is hard to debug if the Triton inlines or call the tt.func if the offset is missing. We need to align to the upstream code as much as possible in order to upstream the Intel backend in the future.
What is the failure case? And we need to find the root cause and fix it.

This failure may have occurred within 3 weeks https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/9770466839/job/26971711342,  but not a stable recurrence.
The changes in third-party is a workaround indeed, just to ensure that the offset does not depend on this Attribution.

The changes in third-party is a workaround indeed, just to ensure that the offset does not depend on this Attribution.

It is wrong if the offset is missing, it breaks the Triton calling convention. What is the log of the issue?

The changes in third-party is a workaround indeed, just to ensure that the offset does not depend on this Attribution.

It is wrong if the offset is missing, it breaks the Triton calling convention. What is the log of the issue?
log

I checked the log. But there is not much useful information found right now.
The changes in this PR is not correct. All the operations of the Triton dialect requiring the SLM as scratchpad may use the same location of the SLM in conflict with this change.
I'd suggest let us replace the assert with some vivid exception reporting error handling.
Narrow down the root cause of the issue first.

The conclusion of this PR after off line discussion:

This issue is only reproduceable randomly on CI tests. We need this changes to work around the issue and make the CI works properly at first.
We need to investigate the root cause of the issue and fix it and revert the work around in this PR.

The issue is tracked https://github.com/intel/intel-xpu-backend-for-triton/issues/1647


[WIP] Implement Organizer Collections endpoints

https://openscience.atlassian.net/browse/OSF-3088
Organizer Collections are what the project organizer uses to organize projects. The structure of Organizer Collections is much simpler than the v1 implementation and does not support back-end smart folders nor nesting of Collections. We call them Organizer Collections to disambiguate from API collections. All Organizer Collections do is point to nodes. By and large, Organizer Collections are Nodes with most of the functionality stripped out.
Collections are only available from the /collections/ endpoint, not the /nodes/ endpoints.

NOTE: Everything is set to be in DEV_MODE because we have an internal project that needs this immediately. Tests are not in yet, but will be put in hopefully while someone is reviewing basic implementation.

JIRA issue #?

I think I may need to ensure that node_links can't link to Organizer Collections.

Shouldn't /data/links/self from /v2/collections/ point to /v2/collections// instead of /v2/nodes//? Possibly outside the purview of this story, just asking.

Absolutely should point to collections. Just missed that while focusing on node_links

I can fix the import.

I'm happy. I tried to break it and couldn't. I hurt it's feelings a little bit by taking advantage of the thing @brianjgeiger fixed in 1512c83. On to CR2...

There appears to be a failing test from develop filtering registrations. I started to fix it, but it took more than a couple of hops, so it seems as though there's something not quite finished about the registrations implementation.

Those failing tests are not mine. They are because the Elastic index isn't being updated fast enough for the asserts. Probably related to licensePicker.

Those failing tests are also not mine.

Pass finished. Minor changes suggested.
@brianjgeiger What is the plan to meet the Dashboard's use case, where the client will want collections and their contents (the actual nodes, not the node links), including title and contributor info? What will the requests look like like?

Hmm, I see. We'll won't get the relationship's relationship in that case.
Well, Chris Wisecarver's caching mechanism is what will ultimately fix that. I don't see a way of avoiding multiple requests. That being said, if we put bulk gets in, we can make it much more efficient (that's the new feature, not the bulk request feature).

@brianjgeiger Can you clarify how the caching will meet the Dashboard use case? What requests will Caner need to send to get what he needs?

It depends on the ultimate implementation of the caching, so I cannot yet answer that question. With bulk gets he could reduce it down to 2 or 3. With caching, it could either be a single request or it could be many requests that are responded to quickly, depending.

Since the primary purpose for this PR is to support the new Dashboard work, I am hesitant to merge this in before we have the features necessary for basic Dashboard functionality. Maybe we hold on this, pending discussion with @cwisecarver as to how to meet Caner's needs?

OK on merge hold -- I think we can also do that in consultation w/ @caneruguz - Thanks

I had some consultations with various people. The recommended solution is to automatically embed detail-view relationships (with an option to not auto-embed on a link-by-link basis). Dawn is looking into this on the embed PR now, and Sam doesn't expect that this will be too difficult.

@brianjgeiger and I had a conversation. All dashboard needs is a list of collections, renaming and removing collections and a list of projects underneath that collection. Embedding projects into collection call would be useful but it's not critical since collection names will be a list that fetches its content when clicked.

There is a JS test failing that is not related to what I'm doing.

/links/self appears to be incorrect; it should be using /collections/ rather than /nodes/:
Response:
{
  "data": {
    "id": "2kx8j",
    "type": "collections",
    "attributes": {
      "title": "Created with API",
      "date_created": "2015-11-12T16:25:22.512675",
      "date_modified": "2015-11-12T16:25:22.512675"
    },
    "relationships": {
      "node_links": {
        "links": {
          "related": {
            "href": "http://localhost:8000/v2/collections/2kx8j/node_links/",
            "meta": {}
          }
        }
      },
      "linked_nodes": {
        "links": {
          "related": {
            "href": "http://localhost:8000/v2/collections/2kx8j/linked_nodes/",
            "meta": {}
          }
        }
      }
    },
    "links": {
      "self": "http://localhost:8000/v2/nodes/2kx8j/",
      "html": "http://localhost:5000/2kx8j/"
    }
  }
}

Visiting the link for /links/html raises a 500. Here's the traceback
Traceback (most recent call last):
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/Users/sloria/miniconda/envs/osf/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/Users/sloria/projects/cos-projects/osf/framework/routing/__init__.py", line 112, in wrapped
    data = fn(*args, **kwargs)
  File "/Users/sloria/projects/cos-projects/osf/website/views.py", line 362, in resolve_guid
    return proxy_url(url)
  File "/Users/sloria/projects/cos-projects/osf/framework/routing/__init__.py", line 268, in proxy_url
    response = app.view_functions[match[0]](**match[1])
  File "/Users/sloria/projects/cos-projects/osf/framework/routing/__init__.py", line 112, in wrapped
    data = fn(*args, **kwargs)
  File "/Users/sloria/projects/cos-projects/osf/website/project/decorators.py", line 91, in wrapped
    return func(*args, **kwargs)
  File "/Users/sloria/projects/cos-projects/osf/website/project/decorators.py", line 222, in wrapped
    return response or func(*args, **kwargs)
  File "/Users/sloria/projects/cos-projects/osf/website/tokens/__init__.py", line 80, in wrapper
    return func(*args, **kwargs)
  File "/Users/sloria/projects/cos-projects/osf/website/project/views/node.py", line 377, in view_project
    ret = _view_project(node, auth, primary=primary)
  File "/Users/sloria/projects/cos-projects/osf/website/project/views/node.py", line 724, in _view_project
    'category': node.category_display,
  File "/Users/sloria/projects/cos-projects/osf/website/project/model.py", line 774, in category_display
    return self.CATEGORY_MAP[self.category]
KeyError: None

We should remove that html link altogether.

Discussed in person: collections created through the API need is_folder set to True. We might want to have a model method for creating collections.

In addition to removing the html link as suggested in https://github.com/CenterForOpenScience/osf.io/pull/4503#issuecomment-156156401, I'm wondering if collection node's category should be set to '' in order to avoid the 500 error.

Pass finished.

Nnnnnnooooo. I mean, I can, but nobody should ever be going to a collection's html link. Though properly it should just give a 404 rather than a 500, I'd rather not make that work better. It shouldn't 500.

Everything should be right now, except the 500 if someone visits a collection through the web's node renderer.

When I try to create a collection with a title > 200 characters, the error object does not have a pointer to the title attribute.
Request body:
{
    "data": {
        "type": "collections",
        "attributes": {
            "title": "awoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjwawoeifjwoeifjw"
        }
    }
}

Response:
{
  "errors": [
    {
      "source": {},
      "detail": "Title cannot exceed 200 characters."
    }
  ]
}

But my guess is that this is unrelated to this PR and is a possible bug with our error formatting. It isn't a blocker to merging this.


Only 4 properties are returned by response.GetMappingFor for every index

Hi @bielu ,
While I was testing the beta I noticed that all indexes in the Examine dashboard only contained 4 fields, namely
Id, __IndexType, __NodeId, __NodeTypeAlias
Previously (v8) response.GetMappingFor(indexesMappedToAlias[0]).Properties; returned all fields in the index, but now it is only these 4. The "Runtime" property does contain a list of all fields, but is it wise to use these?
https://github.com/bielu/bielu.examine.providers/blob/83594edf4949c5cce9fe8a069e9bafa159fce7de/src/Bielu.Examine.ElasticSearch/Providers/ElasticsearchExamineSearcher.cs#L92

HI @mcl-sz,
that is deffo bug it should return all properties, I will have a look on the weekend :)

My assumption issue is related to new mapping in v8 and docs are not great yet so I was little blind here :) I saw content index and was happy 😂 so might need invesitgate get mapping for!

Another point I noticed, and which may also be related to this, is that the ExternalIndex PublishedValuesOnly is set to false.

@mcl-sz can you open separate ticket for it so I can track them separatelly, please? :)

This is fixed as well, it was related to me wrongly using mapping methods, now it works as should :)

This works like a charm!


Avoid using jobs in SeriesUpdatedEventHandler

When a series has their metadata or ACLs changed, the SeriesUpdatedEventHandler propagates those changes to all events of the series. The handler uses lots of Opencast jobs for this, making an update that should only require a couple milliseconds take a couple seconds instead.
This tries to make updating events fast by avoiding using jobs for updating metadata or ACLs, instead calling the code the jobs would run directly.
Your pull request should…

[ ] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits

Are there any plans to backport this in r12.x, or will this break anything?

As far as I recall, we simply didn't want to take the chance of it breaking anything and then put a lot of effort into fixing it for r/12.x. #4539 brought a part of this PR into r/12.x, and while it's not quite the speed increase this PR provided, it still helped while being very unlikely to actually break anything.
But in short, no, we are currently not planning on backporting this to r/12.x


Add in ML by Target and redo ML classes to be schema

Fixes #425

@jthorton @kaminow Could I grab a review here?
This is a big re-work of the way models are spec'd pulled and can be grabbed in a workflow all done with lots of schema.

The YAML file now has a schema MLModelRegistry made of subschema MLModelSpec which you can dynamically pull models from by date and target.
Some changes to the YAML files themselves
The new Mac1 model is ready deployed and will be auto selected in the e2e WF
The models have a schema for when they are pulled down LocalMLModelSpec, making them easy to instantiate from local files
The inference classes themselves are schema. I still need to work out how to serialize the mtenn pytorch models or possibly even drop them and then reconstruct but I think I want to push this to a seperate PR.

The inference classes themselves are schema. I still need to work out how to serialize the mtenn pytorch models or possibly even drop them and then reconstruct but I think I want to push this to a seperate PR.

regarding this, as long as the config used to construct the model is tracked along with the model weights, it should be trivial to reconstruct the mtenn models


a11y violations appearing on the IBM Dynamic Assessment Plugin (DAP)

I ran the Dynamic Assessment Plugin (DAP)
https://www-03.ibm.com/able/dynamic-assessment-plug-in.html
on multiple pages throughout http://carbondesignsystem.com
These are some of the violations it returned:
Each form control must have associated label.
Multiple WAI-ARIA 'search' landmarks must have unique labels specified with aria-label or aria-labelledby.
The WAI-ARIA widget with img role must have a nonempty alternate text or a label.
The WAI-ARIA role(s) and/or attribute(s) identifier are not valid for the element identifier. See rule help for guidance.
Minimum contrast of identifier with its background is not sufficient to meet WCAG AA requirements for text of size identifierpx and weight of identifier.
Multiple WAI-ARIA 'navigation' landmarks must have unique labels specified with aria-label or aria-labelledby.
Could these be addressed for Accessibility Verification Testing (AVT)?

@aagonzales @marijohannessen If you want to grab some time in my calendar, I can walk you through running the DAP on your example/doc pages. I recognize that many/most of the pages on http://carbondesignsystem.com are "just doc", however in order to truly claim that the Carbon Components are accessible, the doc needs to be accessible as well.  :)  (We could even beef up the Accessibility Guidelines page a bit as well - maybe mention landmarks, arrow/esc keys, dialogs, etc, etc).
In addition, every component on all example pages should be accessible, in order to show best practices when using Carbon Components. Ideally, we could point out the things that consumers of each component need to do in order to achieve accessibility (make sure there's a textual label, etc, etc).
If you have a bit of money in the budget, you could also contract an AVT team to run Accessibility Verification Testing for you - they will test using the DAP, the keyboard, and the JAWS screen reader on Windows/Firefox. They can also run VoiceOver on Safari/Mac OS X.  I can recommend a team for you - just ask.

@carmacleod @arbydev Could you please point specific cases/instances where things are failing. We have also made updates since this issue was created, are this issues still existing?
We always check for accessibility and it is a priority for us so blanket statements like this can be confusing for us without specifics. We also work closely and have regular meeting with Bo Campbell from IBM Accessibility.

We could even beef up that Accessibility Guidelines page a bit as well - maybe mention landmarks, arrow/esc keys, dialogs, etc, etc).

Yes, we would love to do this. We've had this in our backlog for awhile. We'd love any help on this.

Oh! Apologies, @aagonzales - I just happened to notice this open issue - I didn't actually run the tools to see for myself. Mea culpa.
So, running the DAP very quickly on one page (http://carbondesignsystem.com), I see the following:

the Search needs a label (either a visible label element, or at a minimum, an invisible aria-label="Search:"  (because placeholder does not accessibly replace a label)
the Getting Started, Style, Components, and Resources images have alt text that is redundant with the adjacent link text. These images are purely decorative, so they need to have alt="" (instead of, for example, alt="Getting started"). This is so that screen readers don't read the text out twice.
there are 2  elements. I would suggest that the one with id="root" should just be a div, in which case it wouldn't need the role="main" aria-label="Main website wrapper". The other one, with id="maincontent", currently has aria-labelledby="page-title" but there's no id="page-title" on the page. That should either point to a real id, or be removed; main elements don't really need a label because there's only supposed to be one (visible) main element.
If you remove the main wrapper to fix the "2 mains not allowed" problem, above, then you will be notified that the "Toggle Side Nav" button has been "orphaned", i.e. it is not inside a landmark region. To fix this, please move the toggle side nav button into the  somehow - maybe the

 could be the  instead, or maybe a new element could wrap the button and the current nav, and the current nav could be demoted to div. ;)
the search input does not need role="search" because its parent div (the one with class="bx--search bx--search--sm") already has role="search".
there are some SVG's with role="img" aria-label="Menu arrow icon" and aria-hidden="true". Please note that aria-hidden="true" is ignored when an element has a role and a label. So for these, I think you want to remove the aria-label and the aria-hidden, and just use role="presentation" (which removes them from the browser's accessibility tree) because I think these images are purely presentational. (Alternatively, you can use role="none", which is synonymous with role="presentation" but "none" only works on newer browsers).
similarly with the SVG's with aria-label="sidenav link icon" - just use role="presentation" (not "img") and remove the aria-label, because these arrows are purely presentational and they don't need to be exposed to AT.
the 4 section elements in the main need to either have a label (either with a visible heading pointed to by aria-labelledby, or with an invisible aria-label), or they need to be turned into divs. This is because
 elements are supposed to be able to be used in the document outline, and so they need a label. Also, screen readers can navigate to sections, but the user won't know which section they are navigating to unless it has a label. I think these 4 sections could be marked up as follows:

 ...  (i.e. change this one to a header (which is a banner landmark) instead of a section; then it doesn't need an aria-label because there's only one banner per page. It would be a tiny bit unexpected to have the banner landmark _inside_ the main landmark, so if possible, could the banner be moved outside the main? i.e. just in the markup - it would still look the same)

 ...

 ... (or maybe "Working with Carbon"? ... not quite sure what the label for this section would be, but not "gray" :)

 ...

In case it's helpful, here's a great explanation of HTML5 section elements and ARIA landmark regions. (Click on items in the left nav to read about them; click on show/hide landmarks/headings to see those).