VideoMAEv2  Feature of HACS

Dear author, I am very interested in your work, could you please disclose the VideoMAEv2  Feature of HACS.

Hi, the features of HACS have been released, please see the readme.md for more details.

Thank you!


[cpptoml] new port

New PR for issue #6087

Wrong commit ... discard this PR, I will create a new one


Kt

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

 Quality gate: Failed
2 Issues

 2 Code issues

Details


#13128: Fixed incorrect position of {pos} string in ColumnSchemaBuilder __toString

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
unknown

Fixed issues
#13128

Can this please be merged into master?

if the append string is not inserted at last, its not really an append string anymore right ;)

Yeah but still it makes sense to change it because DBMS syntax is this way. It may be useful to point that out in the docs though.

does this "append" typology match any mysql typology or is it chosen by us. Tried to find a clarifying reference.

Basically append is used to append to the column definition. The position indicator should always be last in the SQL statement.
MySql: http://dev.mysql.com/doc/refman/5.7/en/alter-table.html
| CHANGE [COLUMN] old_col_name new_col_name column_definition
      [FIRST|AFTER col_name]

Cubrid: http://www.cubrid.org/manual/91/en/sql/schema/table.html#change-modify-clauses
    CHANGE [COLUMN | CLASS ATTRIBUTE ] old_col_name new_col_name column_definition
             [FIRST | AFTER col_name]
  | MODIFY [COLUMN | CLASS ATTRIBUTE] col_name column_definition
             [FIRST | AFTER col_name]

Oracle: It seems it was a mistake to allow the position option. I have removed it from the oci __toString.

Not sure why these tests are failing... help much appreciated.

Tests are not failing, it's codeclimate, not travis.

Merged. Thank you!


Open Format Definition and a Conformant open formats page

I would like to link to a Open Format Definition page with a set of conformant open formats. This would be similar in concept to the Conformant Licenses page.
I would like to link to the Open Format Definition page from the ODI Open Data Certificate site. To earn an Open Data Certificate you are asked a number of questions including, "Is this data in a standard open format?".  The help text is shown in the image below and the "Read more..." link goes to a UK government site, https://www.gov.uk/government/uploads/system/uploads/attachment_data/file/183962/Open-Standards-Principles-FINAL.pdf
I think it makes sense to have a global definition that can be leveraged by a number of tools.
Are there plans to work further on the Open Format Definition page beyond updating section 1.3 Open Format in the Open Definition v2.1?

I don't think the OFD has been looked at in ages, but this seems like a great use of it. Are there other changes you feel necessary (thanks for the PR removing old links so far)?
Please send a note to od-discuss about your plans, or let me know to do so if you don't feel like joining yet another mailing list.

I'm happy to propose something to od-discuss. Stay tuned...

Sent!

@mlinksva would you like me to draft a new OFD page based on the od-discuss conversation? I would avoid the approving formats concept and just breakdown the meaning of the words and perhaps reference some external lists of open data formats.

@Stephen-Gates yes that'd be great.

Draft at http://stephen-gates.github.io/OpenDefinition/index.markdown
Please pardon my lack of GitHub, Markdown and HTML skills. Hopefully the content is useful.

I've converted the content to a google doc for easier reading https://docs.google.com/document/d/1f7tFyGyZG9FxQT3hK7HkBq-LCCB6DKgNqui4UnlfMe0/edit?usp=sharing

@Stephen-Gates FWIW I suspect the reason http://stephen-gates.github.io/OpenDefinition/ insn't readering is that the markdown file needs 'front matter' to be parsed, see http://jekyllrb.com/docs/frontmatter/
But in this case it'd be good to work from https://github.com/okfn/opendefinition/blob/gh-pages/ofd/index.markdown
Submit a PR against that, replacing most of it with your new material and we can focus on iterating from there.

I believe this was addressed by #101 so closing. @Stephen-Gates if not feel free to re-open, or open new issues for the OFD page as needed.


Assets Content file is broken

Assets in the Content folder are not showing up in UE5.1.1

UE5.2 as well

Try doing: git clone --recursive https://github.com/BLaZeKiLL/UE5VoxelTutorial.git
It worked for me


Update resequencing documentation

Description
This PR updates the resequencing documentation to better "flow" with the prior
"Building and Loading" and "Imputation" docs.
Type of change
What type of changes does your code introduce? Put an x in boxes that apply.

[ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
[ ] FEATURE (non-breaking change which adds functionality)
[ ] BUGFIX (non-breaking change which fixes an issue)
[ ] ENHANCEMENT (non-breaking change which improves existing functionality)
[x] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have updated relevant documentation

Changelog entry
Please add a one-line changelog entry below. This will be copied to the changelog file during the release process.

Clean up resequencing documentation

@btmonier Should we add a diagram to the documentation?  I think I had posted one, which I can look for if you don't have it.
Also, did you want to add any information about other variant callers, e.g. Octopus or Clair3?  I had the comparison document here: https://docs.google.com/document/d/181wfF55q7FyhQNfUNyEK3CqGNAeJ9zKZXE6qJk8Wjd0/edit?tab=t.0#heading=h.1718lhxeqy60

@btmonier Should we add a diagram to the documentation? I think I had posted one, which I can look for if you don't have it.
Also, did you want to add any information about other variant callers, e.g. Octopus or Clair3? I had the comparison document here: https://docs.google.com/document/d/181wfF55q7FyhQNfUNyEK3CqGNAeJ9zKZXE6qJk8Wjd0/edit?tab=t.0#heading=h.1718lhxeqy60

Since this update was taking longer than expected, I was going to hold off on adding images for now. This will probably be included in the next wave of updates.

Also, did you want to add any information about other variant callers, e.g. Octopus or Clair3? I had the comparison document here: https://docs.google.com/document/d/181wfF55q7FyhQNfUNyEK3CqGNAeJ9zKZXE6qJk8Wjd0/edit?tab=t.0#heading=h.1718lhxeqy60

Will add to next PR


Credentials provider option not to hardcode password into the code

Credentials provider from :

~/.devrant/credetials
env DEVRANT_PASSWORD
devrant.password system props


[FEATURE] Add member management for projects

In Rancher it all clusters and projects have a section of members.
This section allows to manage users and groups and their role binding.
In a standard rancher setup there are three predefined role bindings:

project-owner
project-member
read-only

You can manage more specific rights by using a custom setting of rights.

I'm interested this feature, how can i help you?


Fastlane Unable to repair provisioning profile with API_Keys and prompting for username and password

New Issue Checklist

[ ] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Fastlane Unable to repair provisioning profile with API_Keys and prompting for username and password  even though i'm not passing username and password in command
Command executed
fastlane sigh repair --force --api_key_path "AppleConnectAPI/XYZ_InHouse.json" --team_id ******
Complete output when running fastlane, including the stack trace and command used
[✔] 🚀
[06:37:51]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
[06:37:53]: To not be asked about this value, you can specify it using 'username'
[06:37:53]: Your Apple ID Username:

 [REPLACE THIS WITH YOUR INFORMATION]

Environment
[06:30:17]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
[06:30:19]: Generating fastlane environment output, this might take a few seconds...
✅ fastlane environment ✅
Stack

Key
Value

OS
10.15.5

Ruby
2.6.0

Bundler?
false

Git
git version 2.21.0

Installation Source
~/.rvm/gems/ruby-2.6.0/bin/fastlane

Host
Mac OS X 10.15.5 (19F101)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.6.0/lib

OpenSSL Version
OpenSSL 1.1.1b  26 Feb 2019

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
11.4

System Locale

Variable
Value

LANG

LC_ALL
en_US.UTF-8
✅

LANGUAGE

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.172.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

executable-hooks
1.6.0

bundler-unload
1.0.2

rubygems-bundler
1.4.5

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

terminal-table
1.8.0

multipart-post
2.0.0

word_wrap
1.0.0

artifactory
3.0.15

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

faraday-net_http
1.0.1

ruby2_keywords
0.0.2

faraday
1.3.0

faraday_middleware
1.0.0

gh_inspector
1.1.3

rubyzip
2.3.0

security
0.1.3

uber
0.1.0

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

httpclient
2.8.3

google-api-client
0.38.0

nanaimo
0.3.0

colored2
3.1.2

claide
1.0.3

CFPropertyList
3.0.3

atomos
0.1.3

xcodeproj
1.19.0

unicode-display_width
1.7.0

plist
3.6.0

public_suffix
4.0.6

addressable
2.7.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

babosa
1.0.4

excon
0.78.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.7

fastimage
2.2.1

json
2.2.0

mini_magick
4.11.0

xcpretty-travis-formatter
1.0.1

dotenv
2.7.6

bundler
2.0.1

naturally
2.2.0

simctl
1.6.8

jwt
2.2.2

declarative
0.0.20

mini_mime
1.0.2

multi_json
1.15.0

signet
0.14.0

os
1.1.1

memoist
0.16.2

googleauth
0.14.0

rake
12.3.2

digest-crc
0.6.3

google-cloud-errors
1.0.1

google-cloud-env
1.4.0

google-cloud-core
1.5.0

google-cloud-storage
1.29.2

emoji_regex
3.2.1

aws-eventstream
1.1.0

aws-sigv4
1.2.2

aws-partitions
1.416.0

jmespath
1.4.0

aws-sdk-core
3.111.0

aws-sdk-kms
1.41.0

aws-sdk-s3
1.87.0

forwardable
1.2.0

logger
1.3.0

date
2.0.0

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

fileutils
1.2.0

mini_portile2
2.4.0

nokogiri
1.10.2

rexml
3.2.0

psych
3.1.0

mutex_m
0.1.0

generated on: 2021-01-22

 [REPLACE THIS WITH YOUR INFORMATION]

There is no official statement from Apple for reparing profiles with App Store Connect API. ( download_all or creating new one would work, but not for 'repairing' profiles.)
Official Doc
https://developer.apple.com/documentation/appstoreconnectapi/profiles?changes=latest_major
Fastlane repair_provisioning_profile! method requires Spaceship.login
https://github.com/fastlane/fastlane/blob/48151291f2c4949c3b1b9919ba2cc81a7cc33293/sigh/lib/sigh/repair.rb#L9


⚠ ClamAV has degraded performance

In b706c2b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 439 ms

Resolved: ClamAV performance has improved in 2f6dddb.


Use jackson-dataformat-csv for csv/ssv/tsv handling

When using the connector we ran into a few limitations of the current csv, ssv and tsv parsing that caused the connector to fail for those cases.
The specific issue we encountered was with the separator occurring in fields or headers which where then quoted to make it "legal".
An example of this is:
header1,header2,header3
value1,"value, 2",value3

Since the current parsing logic splits on all occurrences of,this would break, as the second line looks like it contains an additional field.
Since the project is already using jackson, which these days also supports csv parsing, it might make sense to extend the code a bit, in order to properly handle csv,ssv and tsv quoting and similar shenaningans.
This PR adds a dependency on jackson-dataformat-csv to add csv parsing functionality and replaces the current '.split(separator)' way of handling csv/ssv/tsv with the jackson parser.
Open Questions

Trimming
One thing that I noticed when implementing this is that tests failed originally, because my first implementation didn't trim whitespaces.
Personally I'd consider not trimming whitespaces to be the better behavior, as they may be there for a reason and it shouldn't be the job of this connector to guess whether they were intentional or not.
Is there a historic reason for the trimming to be there and do people rely on this functionality, or is this something that might be changed?

Multi Character Separators
Jackson only supports single char separators, but we currently take a String for the separator, do we envision actually supporting multi-char separators at some point?
If not, would it make sense to change this to a char now, as this implementation currently ignores anything beyond the first character of the separator.

ping @nineinchnick & @ebyhr
I cannot request reviews directly, sorry.

Do you have any opinions on the two open questions I mentioned in the PR @nineinchnick ?

Ad 1. I agree, but let's preserve the current behavior.
Ad 2. We don't have to worry about multi-char separators.
At some point we might start a discussion on how to extend the table functions to make the readers more configurable, that is add some arguments to override stuff like separators, quotes, trimming etc.

At some point we might start a discussion on how to extend the table functions to make the readers more configurable, that is add some arguments to override stuff like separators, quotes, trimming etc.

I agree, but that day is not today I think :)

Changed the implementation a bit to take a char, so we don't give the impression of allowing multi-character separators.

Thanks! Please squash the three first commits, except Changed implementation of CsvPlugin to only allow single-char separators.

done

Sorry, one last thing - can you make the title of the first commit shorter? Something like "Support quotes values in CSV files"

done

@soenkeliebau I just released 432 with this change.

thanks!!


Cannot remap key Fn on a Mac

Microsoft PowerToys version
0.63.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
I'm using a win10 os which installed on a MacBook Pro (Model: A1398).
I do need to swap the Fn key (bottom left corner) with left-ctrol.
When I using PowerToys->Keyboard Manager to do the remap, I met issue:

open PowerToys;
click Keyboard Manager;
click 'Remap a key';
click '+';
click 'Input';

✔ Expected Behavior
I hope the Fn key can be listened, and remaped.
❌ Actual Behavior
I can see every key's name displayed on the pop up dialog, except the Fn key.
Seems Keyboard Manager can NOT hear anything when tap Fn key.
Other Software
N/A

The Fn button is usually handled inside the keyboard itself, meaning it sends different data to the computer. Therefore: either not possible or at least difficult.


running "akash query market lease get" asks for the provider

Running the command akash query market lease get results in the command asking for the provider. This doesn't make sense as one intended usage of this is to allow the user to figure out what provider has the specified lease
e.g.
 ../../akash --home ./cache --node=http://rpc-edgenet.akashdev.net:26657 query market lease get --owner akash1x063d5ta37lrcwmes4xylg52wgcvch5q8r709s --dseq 39122 --gseq 1 --oseq 1
Error: required flag(s) "provider" not set

the thing is, there could be more than one lease for an order.  perhaps it should default to the current, active lease.
This also goes for provider lease-status, provider send-manifest - it's even more clear there, it for the most part should only deal with the active lease.

What I was basically looking for here was a way to get the provider(s) from owner, dseq, gseq, & oseq. I did some poking around in the market code and figured out if you do
akash --home ./cache query market lease list --owner     "akash14dkxfa74fa2v7nmwvay0falv4s56aecuxwl4yg" --dseq 1 --gseq 1 --oseq 1

You'll get results like this
leases:
- lease_id:
    dseq: "1"
    gseq: 1
    oseq: 1
    owner: akash14dkxfa74fa2v7nmwvay0falv4s56aecuxwl4yg
    provider: akash1wztupg65uhzd0kjfkg9wnsdmuxn4zspv8zmx5g
  price:
    amount: "624"
    denom: uakt
  state: active
pagination:
  next_key: null
  total: "0"

That command gives you a way to go from the information you already know about the lease to figuring out which provider(s) have fulfilled the lease. So I don't think any changes are really necessary at the this time.
It seems like presently akash query market lease get is redundant however. You can get the same thing by just substituting akash query market lease list.


Pending contents for 20210906

https://github.com/pingcap/tidb/issues/18771

https://github.com/tikv/tikv/issues/5751

@tsionkun Here's what I found:
For ModIntUnsignedUnsigned, ModIntUnsignedSigned, ModIntSignedUnsigned, and ModIntSignedSigned, the tests may have to be edited as the first three are all grouped into one mod_int_unsigned test and the last one is just called mod_int in the test (https://github.com/tikv/tikv/blob/master/components/tidb_query_expr/src/impl_arithmetic.rs lines 751 and 719)
TruncateDecimal has been split into two functions, truncate_decimal_with_uint and truncate_decimal_with_int (https://github.com/tikv/tikv/blob/master/components/tidb_query_expr/src/impl_math.rs lines 494 and 507)
Other than this here's the list of outstanding functions:
GetParamString: missing
GetVar: missing
RowSig: missing
SetVar: missing
ValuesDecimal: missing
ValuesDuration: missing
ValuesInt: missing
ValuesJSON: missing
ValuesReal: missing
ValuesString: missing
ValuesTime: missing
AesDecrypt: missing
AesEncrypt: missing
AesDecryptIV: missing
AesEncryptIV: missing
Encode: missing
Decode: missing
All the TiDB functions in “tidb/builtin_info.go” (from Database down to RowCount) are missing as the respective “impl_info.rs” file does not exist at all.
Sleep: missing
Lock: missing
ReleaseLock: missing
JsonValidJsonSig: missing
JsonContainsSig: missing
JsonArrayAppendSig: missing
JsonArrayInsertSig: missing
JsonMergePathSig: missing
JsonMergePreserveSig: missing
JsonPrettySig: missing
JsonSearchSig: missing
JsonStorageSizeSig: missing
JsonValidStringSig: missing
JsonValidOthersSig: missing
DateLiteral: missing
TimeStringTimeDiff: missing
DurationStringTimeDiff: missing
DurationDurationTimeDiff
StringTimeTimeDiff: missing
StringDurationTimeDiff: missing
StringStringTimeDiff: missing
TimeTimeTimeDiff: missing
NowWithArg: missing
NowWithoutArg: missing
GetFormat: missing
SysDateWithFsp: missing
SysDateWithoutFsp: missing
CurrentDate: missing
CurrentTime: missing
CurrentTime0Arg: missing
CurrentTime1Arg: missing
Time: missing
TimeLiteral: missing
UTCDate: missing
UTCTimestampWithArg: missing
UTCTimestampWithoutArg: missing
AddTimeDateTimeNull: missing
AddStringAndString: missing
AddTimeStringNull: missing
AddTimeDurationNull: missing
AddDateAndDuration: missing
SubTimeDateTimeNull: missing
SubStringAndString: missing
SubTimeStringNull: missing
SubTimeDurationNull: missing
SubDateAndDuration: missing
SubDateAndString: missing
UnixTimestampCurrent: missing
UnixTimestampInt: missing
UnixTimestampDec: missing
ConvertTz: missing
Quarter: missing
SecToTime: missing
TimestampAdd: missing
Quarter: missing
SecToTime: missing
TimestampAdd: missing
ToSeconds
UTCTimeWithArg: missing
UTCTimeWithoutArg: missing
Timestamp1Arg: missing
Timestamp2Arg: missing
TimestampLiteral: missing
StrToDateDate: missing
StrToDateDatetime: missing
StrToDateDuration: missing
FromUnixTime1Arg: missing
FromUnixTime2Arg: missing
ExtractDatetime: missing
ExtractDuration: missing
AddDateStringString: missing
AddDateStringInt: missing
AddDateStringDecimal: missing
AddDateIntString: missing
AddDateIntInt: missing
AddDateDatetimeString: missing
AddDateDatetimeInt: missing
SubDateDatetimeString: missing
SubDateDatetimeInt: missing
TimeFormat: missing
TimestampDiff: missing
SubDateStringReal: missing
SubDateIntReal: missing
SubDateIntDecimal: missing
SubDateDatetimeReal: missing
SubDateDatetimeDecimal: missing
SubDateDurationString: missing
SubDateDurationInt: missing
SubDateDurationReal: missing
SubDateDurationDecimal: missing
AddDateStringReal: missing
AddDateIntReal: missing
AddDateIntDecimal: missing
AddDateDatetimeReal: missing
AddDateDatetimeDecimal: missing
AddDateDurationString: missing
AddDateDurationReal: missing
AddDateDurationDecimal: missing
Char: missing
Convert: missing
ExportSet3Arg: missing
ExportSet4Arg: missing
ExportSet5Arg: missing
Format: missing
FormatWithLocale: missing
InsertUTF8: missing
Substring2ArgsUTF8: missing
Substring3ArgsUTF8: missing
Everything else is transferred successfully.

@Adam-Kulju I think you can directly comment under tikv/tikv#5751?

@tsionkun do you mean in the comments? I can't edit the original list myself.

@Adam-Kulju Yes make a new comment. This is another repo instead of tikv/tikv


Transitions between tracks of different widths

I've noticed that there can be 'overshoot' when smoothing 2 tracks of different widths. For example make a vertical track that is 3mm wide, then add a horizontal track of 1.5mm that is connected to the first. (right-angle)
You'll see that there will be 3mm thick curves which overshoot the expected boundaries.
Happy to provide more information/help with beta testing if you would like.
All the best,
Dax.

I think I've fixed it. Attach some screenshots if it's still going wrong.

Hey Matt!
Yes, can confirm that your fix has solved the issue. Lines are now looking great!
Thanks again,
Dax.


Set letters lowercase when created flavors

when generated flavors.dart file show this warning from dart:
The constant name 'MYFLAVOR' isn't a lowerCamelCase identifier.
Try changing the name to follow the lowerCamelCase style.dartconstant_identifier_names

dart(constant_identifier_names)

Fixes https://github.com/AngeloAvv/flutter_flavorizr/issues/131


faster test bots

Now the bot test takes 30-60 minutes due to the restriction of prallel executions (AFAFIK). My paying for GitHub didn't solve the issue. So, probably using an external CI might resolve the issue. I don't have a concrete idea yet.

https://jcdan3.medium.com/4-ways-to-speed-up-your-github-action-workflows-a0b08067a6c6
I'll try caching dependencies later.

https://docs.github.com/en/actions/hosting-your-own-runners/about-self-hosted-runners
https://github.com/philips-labs/terraform-aws-github-runner


EntryNotFound error

I've used httpyac before, but with the latest version of VSCode Insiders I get the following error. I've tried to uninstall/re-install the extension but the error repeats.

Version: 1.74.0-insider (system setup)
Commit: ef415578d1b1019db585718968a303eeca4b8979
Date: 2022-11-11T05:19:14.666Z
Electron: 19.1.3
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: Yes

@asampal I could not reproduce your error on my setup. Can you provide an example setup, how to reproduce the error. The error message is too generic, to investigate further. What do you mean with sandboxed? dev container?

I'm not sure what sandboxed means - it is part of the details shown in VSCode. I'll try to figure out what is special about the workspace where this happens. In a brand new workspace the error doesn't occur.

@asampal I don't think sandboxed is the problem at all. Probably Electron sandboxed is meant. It might be helpful to have an http file to reproduce the error. Or which setup is used (use of wsl or remote developmemnt, use of special CLI parameters e.g. --user-data-dir).

Needs more information.


Add option to disable GPU acceleration

With our recent black screen issue we realized we have some clients where GPU acceleration doesn't work too well.  So we should add the option for the user to disable
Could be added in here: https://github.com/RocketChat/Rocket.Chat.Electron/blob/develop/app/app.js#L15
with flags looking like this:
app.commandLine.appendSwitch('--disable-2d-canvas-image-chromium');
app.commandLine.appendSwitch('--disable-accelerated-2d-canvas');
app.commandLine.appendSwitch('--disable-gpu');

Just need to check their settings on startup to decide to apply the flags or not.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

With different intel GPU I get this issue very often

Black screen came to me after 2.10.

How can we have this option persisted? I toggle the Disable GPU option, but it always reverts to use GPU after a restart of the app.

@iamhammer well that's kind of a drag...Thankfully I haven't been having performance issues lately due to that being enabled. Great tip tho! Thanks!

@bkraul we are adding a setting to persist disabling hardware acceleration. Next release it will be available.


wireshark: 3.4.3 -> 3.4.4

Motivation for this change
https://www.wireshark.org/docs/relnotes/wireshark-3.4.4.html
Things done

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

[x] NixOS

[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 116018 at ed2281f2 run on x86_64-linux 1

9 packages built successfully:

compactor
haka
ostinato
python38Packages.pyshark
python39Packages.pyshark
termshark
wifite2
wireshark (wireshark-qt)
wireshark-cli

3 suggestions:

warning: build-tools-in-build-inputs
makeWrapper is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/applications/networking/sniffers/wireshark/default.nix:40:3:
   |
40 |   buildInputs = [
   |   ^

warning: unclear-gpl
gpl2 is a deprecated license, check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/applications/networking/sniffers/wireshark/default.nix:99:5:
   |
99 |     license = licenses.gpl2;
   |     ^

warning: unused-argument
Unused argument: fetchpatch.
Near pkgs/applications/networking/sniffers/wireshark/default.nix:3:33:
  |
3 | , libssh, nghttp2, zlib, cmake, fetchpatch, makeWrapper
  |                                 ^

@TredwellGit Can you take a look at the suggestions? They seem very spot on.

Backported in 52abed1c7ce95bd02a53ca4dd06d34ab5e826f84.


Update Column Name in Database Diagram from Token to Secret

I suggest updating Column name in the Applications table from Token to Secret in the DB diagram

Very good point, thanks for noticing!


AttributeError: module 'importlib.resources' has no attribute 'files'

It looks like a recent change to nmigen dependencies is causing ModuleNotFoundError's
Traceback (most recent call last):
  File "/home/colepoirier/src/nmigen/nmigen/_yosys.py", line 17, in <module>
    importlib_resources.files # py3.9+ stdlib
AttributeError: module 'importlib.resources' has no attribute 'files'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/colepoirier/src/nmigen/nmigen/_yosys.py", line 19, in <module>
    import importlib_resources # py3.8- shim
ModuleNotFoundError: No module named 'importlib_resources'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "decoder/pseudo/pywriter.py", line 8, in <module>
    from soc.decoder.power_pseudo import convert_to_python
  File "/home/colepoirier/src/soc/src/soc/decoder/power_pseudo.py", line 18, in <module>
    from soc.decoder.power_decoder import create_pdecode
  File "/home/colepoirier/src/soc/src/soc/decoder/power_decoder.py", line 84, in <module>
    from nmigen.cli import rtlil
  File "/home/colepoirier/src/nmigen/nmigen/cli.py", line 4, in <module>
    from .back import rtlil, cxxrtl, verilog, pysim
  File "/home/colepoirier/src/nmigen/nmigen/back/cxxrtl.py", line 1, in <module>
    from .._yosys import *
  File "/home/colepoirier/src/nmigen/nmigen/_yosys.py", line 21, in <module>
    import importlib_resources # py3.6- shim
ModuleNotFoundError: No module named 'importlib_resources'

How are you installing nMigen?

python3 setup.py develop
It appeared in the past that it was only necessary to run this command once, and new commits pulled in after this would work fine. Now, it appears that one must run python3 setup.py develop every time a new commit is pulled in from nmigen/nmigen source repo.

Now, it appears that one must run python3 setup.py develop every time a new commit is pulled in from nmigen/nmigen source repo.

Are you sure? I think what happened in your case is that when you update, the files are automatically updated as well, but the dependencies are not; to get those updated you have to rerun setup.py.

That sounds accurate to what happens. The effect however is the same, python3 setup.py develop must be run every time new nmigen code is pulled in case there have been dependency changes. This is fine now that I know this is the assumption I must operate under. Is there a note to this effect on the README? If not, I think it is probably a necessary addition. Thanks for your help in clearing this up.

There isn't; I'm going to describe this in the upcoming documentation (see the doc branch) which already goes into more detail with regards to Python package management.

Ah thanks very much. Sorry I was looked in the docs older of the master branch, now having looked at the same under the 'doc' branch things are much clearer. I really appreciate you taking the time and effort to do this!

Sorry I was looked in the docs older of the master branch, now having looked at the same under the 'doc' branch things are much clearer.

Yeah the new docs aren't really finished or properly public yet, it's understandable you'll get confused.

This is now documented at https://nmigen.info/nmigen/latest/install.html#development-snapshot.


fix: use real url to deployment package and ARM template

Shortened aka.ms/ URLs don't work. In consequence connector can't get certificate.
Change(s):

Use url  https://github.com/Azure/Azure-Sentinel/raw/master/Solutions/WithSecureElementsViaFunction/Data%20Connectors/WithSecureElementsViaFunctionConn.zip to deploy package
use url https://raw.githubusercontent.com/Azure/Azure-Sentinel/master/Solutions/WithSecureElementsViaFunction/Data%20Connectors/azuredeploy_Connector_WithSecureElements_AzureFunction.json in Deploy to Azure button

Reason for Change(s):

Urls https://aka.ms/sentinel-WithSecureElementsViaFunction-azuredeploy and https://aka.ms/sentinel-WithSecureElementsViaFunction-functionapp don't work. In consequence connector can't get certificate.

Testing Completed:

Yes

@microsoft-github-policy-service agree [company="{WithSecure}"]

@microsoft-github-policy-service agree company="WithSecure"

Hello @ws-mpiglas, Please package the solution using V3 tool
https://github.com/Azure/Azure-Sentinel/blob/master/Tools/Create-Azure-Sentinel-Solution/V3/README.md

Thanks @v-prasadboke. I've created new solution package. Do you know why  https://aka.ms/sentinel-WithSecureElementsViaFunction-azuredeploy and https://aka.ms/sentinel-WithSecureElementsViaFunction-functionapp didn't work? Other solutions in this git repository are using short urls and they are working fine.

Thanks @v-prasadboke. I've created new solution package. Do you know why https://aka.ms/sentinel-WithSecureElementsViaFunction-azuredeploy and https://aka.ms/sentinel-WithSecureElementsViaFunction-functionapp didn't work? Other solutions in this git repository are using short urls and they are working fine.

Hello @ws-mpiglas, Sorry for the inconvenience but the shortlinks were not updated.
I've updated them now you can try the shortlinks. Please check and confirm on it once.
If working you can close this PR

Hello @ws-mpiglas, We are waiting for your update

Hi @v-prasadboke, certification was successful this time. Thanks for your support. I'm closing PR now


The app should offer to recompress video before sending

Yeah that would be great. Sending videos does only work for me if I send them first to a Threema contact then redownload them from Threema and use this compressed video. Not the easiest workaround ;-)

This was implemented


ENH: Add T2starw as a functional derivative

Needed to include a Tedana output in the func/ directory. Given that the image is derived purely from functional data (until resampled), this should be permissible.
The removed line was a duplicate.

Codecov Report

Merging #720 (159d100) into master (45ab13e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #720   +/-   ##
=======================================
  Coverage   49.21%   49.21%
=======================================
  Files          49       49
  Lines        5868     5868
  Branches      838      838
=======================================
  Hits         2888     2888
  Misses       2871     2871
  Partials      109      109

Flag
Coverage Δ

reportlettests
∅ <ø> (∅)

unittests
49.21% <ø> (ø)

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

Continue to review full report at Codecov.

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


fix: improve integration with virtualized lists

Ref #119 cc @jgoux

What: This removes checking whether there's an item at index 0.

Why: In virtualized lists there is often not an item at index 0.

How: Also, we can ensure that there will always be an items array so we can just remove that function altogether and get items by their index from this.items. Turns out this is much simpler :smile:

Checklist:

[ ] Documentation N/A
[x] Tests
[x] Ready to be merged

Codecov Report

Merging #128 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #128   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines         256    251    -5
  Branches       60     58    -2
=====================================
- Hits          256    251    -5

Impacted Files
Coverage Δ

src/downshift.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fixing terraform call by adding init call and updating disk param

Terraform deprecated disk, so I changed it to boot_disk
I wasn't able to run Terraform without running init first

Thanks @mkjelland - LGTM. Moving this over to develop.


Cleaned up per review. Added environment to Tensorflow folder.

I will work on changing the tensorflow version.


Can I do complement method is installed anew ?

Mr. randy3k,
Thanks for nice packages.
I wanna complement against new packages is installed by "install.package()".
For example, I can complement against the following word, lm
lm [Tab]
↓
# Logistic regression model
modelName <- glm(DV ~ predictor1 +* predictor2, data = df, family = "binomial")
summary(modelName)

But, I can't complement new packages, I can only complement method name...
What should I do ?

Would you rephrase your question? I could not get it.

Sorry.
I wanna complement like plot or lm against new Packages.
for exapmle,
install.packages("dplyr")
db = src_postg [Tab]
↓
db = src_postgres(dbname='',host='',port='',user='',password='')

it is supported in a slightly different way in the recent refactoring.


Add indonesian translation

Hey, looks like you need translations into different languages for your app
Can i work on Indonesian translation?
Thanks :smiley:

Feel free to make a PR please :+1:


Update amazon-linux.md

Bad link for 2016.03 :

SELECT * from details WHERE product like '%linux%';

:bulb: The good news are it's the one and only one :

The linked blog post in the tweet is already archived. I agree, better point to the release notes link than a tweet.

Yep, I gvae it a try :sweat_smile:

:point_right:  BTW, that was the only twitter link :
SELECT *
    from details
where
    link like '%twitter%';


Allow the configuration of the SUD class and params separately

The goal is to be able to do the following
<configuration>
...

<systemUnderDevelopment>com.greenpepper.systemunderdevelopment.DefaultSystemUnderDevelopment</systemUnderDevelopment>
<systemUnderDevelopmentArgs>ARG1;ARG2</systemUnderDevelopmentArgs>

This is equivalent to the previous (still possible)
<configuration>
...

<systemUnderDevelopment>com.greenpepper.systemunderdevelopment.DefaultSystemUnderDevelopment;ARG1;ARG2</systemUnderDevelopment>

Impact on the runners

Introduction of the new optional template parameter {fixtureFactoryArgs}

Fixed by #91


experimental files for n-pentane and NOx IDTs, RCM

As seen on TV: https://doi.org/10.26434/chemrxiv.13720105 (preprint)
RCM IDTs for n-pentane doped with NO or NO2, Pc=15 bar, 650-1000 K
(50% n-pentane + NOx in air) + 50% diluent

Looks great, thanks @mefuller. I think we just need to wait for a DOI for the paper and it's good to go

Ok, that's fine.
My initial thought was to reference the preprint and then update later when the published version is available.
I'll update the branch once the "real" DOI is assigned

If you put the preprint on arXiv or a similar thing with a permanent identifier, that'd be fine too.

All fixed - should be ok

one other thought - do the SMILES codes need to be enclosed in quotes for proper processing, e.g. [Ar] needs to be "[Ar]" to avoid reading it in as a list?

Yes, I believe they do. Your also can't have any blank keys, just the word followed by a colon and then the next line at the same indentation.

Ok, I'll get that all fixed up and push a corrected set of files.
Next time should be better
Is there already a wiki page re #4? If yes, apologies and, if not, I'll try to put what I've learned from the corrections into it

There is not, feel free to add one, or suggest some text in that issue

I think this is now good to go and I wrote in some functionality for me to update the DOI/article information in the future if desired

There's a systematic error affecting some of these files (just discovered) - closing for now and will submit a new PR when ready


Azure SDK Review - [IC3 ACS Calling]

New SDK Review meeting has been requested.
Service Name: IC3 ACS Calling
Review Created By: Bob Gao
Review Date: 06/11/2024 02:05 PM PT
Release Plan:
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Android, iOS, .NET,
Description:
Detailed meeting information and documents provided can be accessed here

Meeting updated by Ronnie Geraghty
Service Name: IC3 ACS Calling
Review Created By: Bob Gao
Review Date: 06/17/2024 02:05 PM PT
Hero Scenarios Link: Not Provided
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: Not Provided
APIView Links: Android, iOS, .NET,
Description:
Detailed meeting information and documents provided can be accessed here

Calling SDKs

Android

2.10.0

iOS

2.12.0

Windows

1.8.0


SIGNAL doesn't compile for ARM processor (adafruit WICED feather)

Trying to run this on an adafruit WICED feather and the compiler doesn't like the SIGNAL.
test2:34:8: error: expected constructor, destructor, or type conversion before '(' token
SIGNAL(TIMER0_COMPA_vect) {
^

Same error for Feather HUZZAH.  I'm going to poke at it for a while to see if I can figure out what's wrong.

Same for me. Sad.


Improved RGB clipping logic

As suggested in https://github.com/ioam/holoviews/pull/3305 this fixes the clipping logic in bokeh's RGBPlot. It is now identical to the logic used by matplotlib's imshow internally.

Being as consistent between plotting extensions as possible is a good thing. Merging.


Memory Termination Support

Is there any interest in integrating memory termination support directly into KSCrash, or is this something we want to leave up to those who integrate KSCrash?
Here's what I've done for Sentry, I'd love to bring this into KSCrash so a lot more teams get to process OOMs more efficiently.

Hey, thanks so much for your commitment to improving KSCrash! Integrating memory termination support directly into the library sounds like a fantastic idea. I know a lot of teams have been using the algorithm from that Facebook article to handle OOMs on their own, so having this built into KSCrash could be super beneficial for everyone.
@bamx23 what do you think?

Wow, thanks! Yes, definitely worth bringing to KSCrash. OOMs are one of the most tricky types of crashes so any work towards making them easier to detect is a great work!
@naftaly feel free to create a PR to v2.0 branch and we can discuss the implementation there. Or if you want we can start with overall concept of the implementation here in this issue.

@bamx23 There are some great comments here and here in my Crashlytics PR that can help give an idea of how it works.
I'll put a diff in the next days.

@bamx23 @GLinnik21 I've put some work into this over the lats few days. Have a look at [this branch in my fork](https://github.com/naftaly/KSCrash/tree/Memory-Termination and) when you have a chance and tell me what you think. It's not totally done but the idea is there.

Not OOM related...
One thing that could make the system even more robust is "install folders". Basically, for each install there is a place on disk where things go which is different from the previous session (using GUIDs or something).
This allows for a few things:

No need to rush at startup to store previous data because it'll be overwritten by new data.
Failures to report don't matter since you can always process later.
...

@bamx23 @GLinnik21 I've put some work into this over the lats few days. Have a look at this branch in my fork and when you have a chance and tell me what you think. It's not totally done but the idea is there.

Hey, awesome work on this! I've gone through the code on your branch and it's looking solid. I do have a couple of minor questions, but we can get into those details once you draft the PR. Could you go ahead and create the PR targeting the release-2.0 branch? We can continue the discussion there. Great job again!

PR Draft: https://github.com/kstenerud/KSCrash/pull/476


[request] remember window position and size

[request] remember window position and size

This should already be implemented and working on Windows, but it's possible a recent PDCurses upgrade messed it up.

It remembers the X and Y coordinates but does not remember the width and height.

Fixed for the next release. https://github.com/clangen/musikcube/pull/295/commits/1504e48680806f2558d62fe1429ca06d2416401c


[RFC] Multi tenancy pattern for all chat platforms

Summary
As discussed in https://github.com/howdyai/botkit/pull/800 we created this issue in order to find a common ground on the integration of multi-tenancy on Botkit.
Motivation
The main use case is to be able to talk to many facebook pages / slack teams / other chat platform using the same pattern. Right now we have different implementation and it would be nice to consolidate around the same pattern.
Detailed design
WIP
ping @jonchurch @alamusi @goodpic @m4olivei @peterswimm @acrollet @srbartlett, obviously feel free to ping more relevant people.

What is the state of this RFC? is there any workarounds I can use to implement this?

Right now we are waiting for different parties to make a statement :P

With the intro of the messaging pipeline, this is a perfect time to revisit this issue!
Id say a middleware to retrieve bot config info is in order, and can be dropped into place in the receive or ingest stage to modify the bot before it is filtered through the handlers.
@benbrown have you guys thought about this any lately? Slack is multi team by nature, and a similar implementation for facebook is appropriate.
Ill keep my eyes on this and propose a PR for people to poke at and modify!


concat_dim getting added to all variables of multifile datasets

Code Sample
Using the following example data set:
example_jan.nc
#!/usr/bin/env python3

import xarray
ds = xarray.open_mfdataset('example_jan.nc', concat_dim='Time')
print(ds)

The result from xarray 0.10.2 (and all previous various xarray versions we've worked with):
Dimensions:                                                      (Time: 1, nOceanRegions: 7, nOceanRegionsTmp: 7, nVertLevels: 100)
Dimensions without coordinates: Time, nOceanRegions, nOceanRegionsTmp, nVertLevels
Data variables:
    time_avg_avgValueWithinOceanLayerRegion_avgLayerTemperature  (Time, nOceanRegionsTmp, nVertLevels) float64 dask.array<shape=(1, 7, 100), chunksize=(1, 7, 100)>
    time_avg_avgValueWithinOceanRegion_avgSurfaceTemperature     (Time, nOceanRegions) float64 dask.array<shape=(1, 7), chunksize=(1, 7)>
    time_avg_daysSinceStartOfSim                                 (Time) timedelta64[ns] dask.array<shape=(1,), chunksize=(1,)>
    xtime_end                                                    (Time) |S64 dask.array<shape=(1,), chunksize=(1,)>
    xtime_start                                                  (Time) |S64 dask.array<shape=(1,), chunksize=(1,)>
    refBottomDepth                                               (nVertLevels) float64 dask.array<shape=(100,), chunksize=(100,)>
Attributes:
    history:  Tue Dec  6 04:49:14 2016: ncatted -O -a ,global,d,, acme_alaph7...
    NCO:      "4.6.2"

The results with xarray 0.10.3:
<xarray.Dataset>
Dimensions:                                                      (Time: 1, nOceanRegions: 7, nOceanRegionsTmp: 7, nVertLevels: 100)
Dimensions without coordinates: Time, nOceanRegions, nOceanRegionsTmp, nVertLevels
Data variables:
    time_avg_avgValueWithinOceanLayerRegion_avgLayerTemperature  (Time, nOceanRegionsTmp, nVertLevels) float64 dask.array<shape=(1, 7, 100), chunksize=(1, 7, 100)>
    time_avg_avgValueWithinOceanRegion_avgSurfaceTemperature     (Time, nOceanRegions) float64 dask.array<shape=(1, 7), chunksize=(1, 7)>
    time_avg_daysSinceStartOfSim                                 (Time) timedelta64[ns] dask.array<shape=(1,), chunksize=(1,)>
    xtime_end                                                    (Time) |S64 dask.array<shape=(1,), chunksize=(1,)>
    xtime_start                                                  (Time) |S64 dask.array<shape=(1,), chunksize=(1,)>
    refBottomDepth                                               (Time, nVertLevels) float64 dask.array<shape=(1, 100), chunksize=(1, 100)>
Attributes:
    history:  Tue Dec  6 04:49:14 2016: ncatted -O -a ,global,d,, acme_alaph7...
    NCO:      "4.6.2"

Problem description
The expected behavior for us was that refBottomDepth should not have Time as a dimension.  It does not vary with time and does not have a Time dimension in the input data set.
It seems like https://github.com/pydata/xarray/issues/1988 and https://github.com/pydata/xarray/pull/2048 were intended to address cases where the concat_dim was not yet present in the input files.  But in cases where concat_dim is already in the input files, it seems like only those fields that include this dimensions should be concatenated and other fields should remain free of concat_dim.  Part of the problem for us is that the number of dimensions of some of our variables change depending on which xarray version is being used.
Expected Output
That for 0.10.2 (see above)
Output of xr.show_versions()

/home/xylar/miniconda2/envs/mpas_analysis_py3/lib/python3.6/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
INSTALLED VERSIONS
commit: None
python: 3.6.5.final.0
python-bits: 64
OS: Linux
OS-release: 4.13.0-38-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
xarray: 0.10.3
pandas: 0.22.0
numpy: 1.14.2
scipy: 1.0.1
netCDF4: 1.3.1
h5netcdf: 0.5.1
h5py: 2.7.1
Nio: None
zarr: None
bottleneck: 1.2.1
cyordereddict: None
dask: 0.17.2
distributed: 1.21.6
matplotlib: 2.2.2
cartopy: 0.16.0
seaborn: None
setuptools: 39.0.1
pip: 9.0.3
conda: None
pytest: 3.5.0
IPython: None
sphinx: 1.7.2

cc @pwolfram

What happens if you open multiple files with open_mfdataset(), e.g., for both January and February. Does it result in a dataset with the right dimensions on each variable?

@shoyer, in that case as well, Time is added to refBottomDepth in v 0.10.3, which was not the case in previous xarray versions.

👍 This is a persistent problem for me as well.
I often find myself writing a preprocessor function like this
def process_coords(ds, concat_dim='time', drop=True):
    coord_vars = [v for v in ds.data_vars if concat_dim not in ds[v].dims]
    if drop:
        return ds.drop(coord_vars)
    else:
        return ds.set_coords(coord_vars)
ds = xr.open_mfdataset('*.nc', preprocess=process_coords)

The reason to drop the coordinates is to avoid the comparison that happens when you concatenate coords.

@shoyer, I stand corrected.  in 0.10.1, I also see the Time variable getting added to refBottomDepth when I open multiple files.  So maybe this is not in fact a new problem but an existing issue that happened to behave as I expected only when opening a single file.

@rabernat, so you're fooling xarray into not including the time dimension in your non-time variables by making them coordinates in the above example?

so you're fooling xarray into not including the time dimension in your non-time variables by making them coordinates in the above example?

Exactly. They are coordinates. Those variables are usually related to grid geometry or constants, as I presume is refBottomDepth in your example.

Yes, true.  I'm trying to think if there are any examples where the fixed-in-time variables would not be coordinates.  So far, none come to mind.  Thanks for the tip.

But this issue raises an important basic point: we might want different behavior for variables in which concat_dim is already a dimension vs. variables for which it is not.

I stand corrected. in 0.10.1, I also see the Time variable getting added to refBottomDepth when I open multiple files. So maybe this is not in fact a new problem but an existing issue that happened to behave as I expected only when opening a single file in previous versions. Sorry for not noticing that sooner.

OK, in that case I think #2048 was still the right change/bug-fix, making multi-file and single-file behavior consistent.
But you certainly have exposed a real issue here.

But this issue raises an important basic point: we might want different behavior for variables in which concat_dim is already a dimension vs. variables for which it is not.

Yes, we shouldn't implicitly add a new dimensions to variables in the case where the dimension already exists in the dataset. We only need the heuristics/comparisons when an entirely new dimension is being added.

@rabernat, your suggestion above has worked perfectly to get our unit tests working again in MPAS-Analysis so that will tide us over until this issue can be addressed directly in xarray.  Thanks!

I'm glad!
FWIW, I think this is a relatively simple fix within xarray. @xylar, if you are game, we would love to see a PR from you. Could be a good opportunity to learn more about xarray internals.

Hmm, I agree that it shouldn't be hard but I don't really have time to do this right now.  If no one has had a chance to look into it by mid May I might be able to take it on then.

I recently ran into a similar issue and found a potential solution.
The functionality, as far as I understand, is already in the open_mfdataset function in the data_vars ='minimal' argument, in this case variables without concat_dim are included without adding the dimension. The current default is data_vars ='all' which include all variables with the added dimension. If the desired functionality shouldn't implicitly add new dimensions shouldn't the default be set to 'minimal' instead?
I think this is a very non-intrusive solution since it only affects the open_mfdataset function, and if you for some reason want the old behavior it is still there. An alternative way is to rewrite the _dataset_concat function I guess.
This is my first time attempting to contribute, does this sound like a good idea? I can try to make a pull request but would very much value some input first.

I'm a new developer at the SciPy 2019 Sprints, and I'm interested in making a pull request for this issue as a learning step.
@henrikca Would this be useful?  Or would this conflict with your work?

So there are some units tests that assert the behavior for open_mfdataset() is identical to the behavior for concat().   This implies that if we change the default data_vars value from "all" to "minimal" for one function, we need to change it for both functions.
@shoyer I think you suggested that concat() default behavior should change in #2145, in the same way it will change for open_mfdataset.   So I am going to add this change to the pull request.

@bonnland I don't think you want to change the default data_vars but instead update the heuristics as in this comment

we shouldn't implicitly add a new dimensions to variables in the case where the dimension already exists in the dataset. We only need the heuristics/comparisons when an entirely new dimension is being added.

@dcherian . I believe you are correct in principle, but there is a logical problem that is expensive to evaluate.  The difficult case is when two datasets have a variable with the same name, and that variable does not include the concatenation dimension.  In order to align the datasets for concatenation, both variables would need to be identical.   The resulting dataset would just have one (unchanged) instance of that variable, say from the first dataset.   I think someone along the way decided this operation was too expensive.   This is from concat.py, lines 302-307:
    # stack up each variable to fill-out the dataset (in order)
    for k in datasets[0].variables:
        if k in concat_over:
            vars = ensure_common_dims([ds.variables[k] for ds in datasets])
            combined = concat_vars(vars, dim, positions)
            insert_result_variable(k, combined)

So I think some consensus needs to be reached, about whether it is a good idea to load these variables into memory to check for identical-ness between them.
Or another possibility is that we leave "unique" variables alone:  if a variable exists only once across all datasets being concatenated, we do not add the concatenation dimension to it.   This might solve the original poster's issue when opening a single dataset.

The logic for determining which variables to concatenate is in the _calc_concat_over helper function:
https://github.com/pydata/xarray/blob/539fb4a98d0961c281daa5474a8e492a0ae1d8a2/xarray/core/concat.py#L146
Only "different" is supposed to load variables into memory to determine which ones to concatenate.
Right now we also have "all" and "minimal" options:

"all" attempts to concatenate every variable that can be broadcast to a matching shape: https://github.com/pydata/xarray/blob/539fb4a98d0961c281daa5474a8e492a0ae1d8a2/xarray/core/concat.py#L188-L190
"minimal" only concatenates variables that already have the matching dimension.

Recall that concat handles two types of concatenation: existing dimensions (corresponding to np.concatenate) and new dimensions (corresponding to np.stack). Currently, this is all done together in one messy codebase, but logically it would be cleaner to separate these modes into two separate function:

In "existing dimensions" mode:

"all" is currently broken, because it will also concatenate variables that don't have the dimension.
"minimal" does the right thing, concatenating only variables with the dimension.

In "new dimensions" mode:

"all" will add the dimension to all variables.
"minimal" raise an error if any variables have different values. If you're datasets have any data variables with different values at all, it raises an error. This is pretty much useless.

Here's my notebook testing this out: https://gist.github.com/shoyer/f44300eddda4f7c476c61f76d1df938b
So I'm thinking that we probably want to combine "all" and "minimal" into a single mode to use as the default, and remove the other behavior, which is either useless or broken. Maybe it would make sense to come up with a new name for this mode, and to make both "all" and "minimal" deprecated aliases for it? In the long term, this leaves only two "automatic" modes for xarray.concat, which should make things simpler for users trying to figure this out.

@shoyer Your explanation makes sense, but there are unit tests that expect the default concat() behavior to be the same as default behavior for Pandas concat(), which tries to perform an "outer" join between dataframes.
Therefore, from my limited understanding, the default behavior for xarray concat() should be to preserve all variables.   If this default behavior changes, then it may break code making these expectations.
Can we get a perspective from the author of concat.py, @TomNicholas ?   Thanks.

@shoyer I'm sorry I didn't look at your examples more closely at first.  I see now that your first example of using data_vars='minimal' is already preserving one instance of the variable x, and I was suggesting earlier that this variable was not being included in the concatenation.
So I am not clear on why so many unit tests fail when I switch the default value for data_vars to 'minimal'.   The output from your examples seems compatible with Pandas concat, though I don't understand Pandas very well yet.
I wonder if the unit tests that fail are written correctly.   I have to add that I spent an entire day trying to understand the code in concat.py, by stepping through it for several unit tests.    I found the code quite difficult to understand.

Specifically, what should the default behavior of concat() be, when both datasets include a variable that does not include the concatenation dimension? Currently, the concat dimension is added, and the result is a "stacked" version of the variable. Others have argued that this variable should not be included in the concat() result by default, but this appears to break compatibility with Pandas concat().

Can you give a specific example of the behavior in question?

Can you give a specific example of the behavior in question?

Here is the most specific thing I can say:  If I switch the default value for data_vars to 'minimal' for concat() and open_mfdataset(), then I get a lot of failing unit tests (when running "pytest xarray -n 4".  I may be wrong about why they are failing.  The unit tests have comments in them, like "Check pandas compatibility"; see for example, line 370 in test_duck_array_ops.py for an example instruction that raises a ValueError exception.  Many failures appear to be caused by a ValueError exception being raised, like in the final example you have in your notebook.
I hope this is specific enough; I realize that I'm not deeply comprehending what the unit tests are actually supposed to be testing.

UPDATE: @shoyer it could be that unit tests are failing because, as your final example shows, you get an error for data_vars='minimal' if any variables have different values across datasets, when adding a new concatentation dimension. If this is the reason so many unit tests are failing, then the failures are a red herring and should probably be ignored/rewritten.

This seems very likely to me. The existing behavior of data_vars='minimal' is only useful in "existing dimensions mode".
Xarray's unit test suite is definitely a good "smoke test" for understanding the impact of changes to concat on our users. What it tells us is that we can't change the default value from "all" to "minimal" without breaking existing code. Instead, we need to change how "all" or "minimal" works, or switch to yet another mode for the new behavior.
The tests we should feel free to rewrite are cases where we set data_vars="all" or data_vars="minimal" explicitly for verifying the weird edge behaviors that I noted in my earlier comments. There shouldn't be too many of these tests.

Maybe it would make sense to come up with a new name for this mode, and to make both "all" and "minimal" deprecated aliases for it?

I'm in favour of this. What should we name this mode?
One comment on "existing dimensions" mode:

"minimal" does the right thing, concatenating only variables with the dimension.

For variables without the dimension, this will still raise a ValueError because compat can only be 'equals' or 'identical'. It seems to me like we need compat='override' and/or compat='tolerance', tolerance=... that would use numpy's approximate equality testing. This checking of non-dimensional coordinates is a common source of mfdataset issues. What do you think?

I have a draft solution in #3239. It adds a new mode called "sensible" that acts like "all" when the concat dimension doesn't exist in the dataset and acts like "minimal" when the dimension is present. We can decide whether this is the right way i.e. add a new mode but the more fundamental problem is below.
The issue is dealing with variables that should not be concatentated in "minimal" mode (e.g. time-invariant non dim coords when concatenating in time). In this case, we want to skip the equality checks in _calc_concat_over. This is a common reason for poor open_mfdataset performance.
I thought the clean way to do this would be to add the compat kwarg to concat and then add compat='override' since the current behaviour is effectively compat='equals'.
However, merge takes compat too and concat and merge support different compat arguments at present. This makes it complicated to easily thread compat down from combine or open_mfdataset without adding concat_compat and merge_compat which is silly.
So do we want to support all the other compat modes in concat? Things like broadcast_equals or no_conflicts are funny because they're basically merge operations and it means concat acts like both stack, concat and merge. OTOH if you have a set of variables with the same name from different datasets and you want to pick one of those (i.e. no concatenation), then you're basically doing merge anyway. This would require some refactoring since concat assumes the first dataset is a template for the rest.
@shoyer What do you think?

I have tried to understand why the xarray developers decided to provide their own options for concatenation.   I am not an experienced user of xarray, but I can't find any discussion of how the current options for concatenation were derived.
The pandas concat() function uses the option join = {'inner', 'outer', 'left', 'right'} in order to mimic logical database join operations.    If there is a reason that xarray cannot do the same, it is not obvious to me.   I think the pandas options have the advantage of logical simplicity and traditional usage within database systems.

Thanks for your input @bonnland.

The pandas concat() function uses the option join = {'inner', 'outer', 'left', 'right'} in order to mimic logical database join operations. If there is a reason that xarray cannot do the same, it is not obvious to me. I think the pandas options have the advantage of logical simplicity and traditional usage within database systems.

We do have a join argument that takes these arguments + 'override' which was added recently to skip expensive comparisons. This works for "indexes" or "dimension coordinates". An example: if you have 2 dataarrays, one on a coordinate x=[1, 2, 3] and the other on x=[2,3,4], join lets you control the x coordinate of the output. This is done by xr.align.
What's under discussion here is what to do about variables duplicated across datasets or indeed, how do we know that these variables are duplicated across datasets when concatenating other variables.

#3239 has been merged. Now minimal is more useful since you can specify compat="override" to skip compatibility checking.
What's left is to change defaults to implement @shoyer's comment

So I'm thinking that we probably want to combine "all" and "minimal" into a single mode to use as the default, and remove the other behavior, which is either useless or broken. Maybe it would make sense to come up with a new name for this mode, and to make both "all" and "minimal" deprecated aliases for it? In the long term, this leaves only two "automatic" modes for xarray.concat, which should make things simpler for users trying to figure this out.


Removes TPKeyboardAvoiding

It wasn't being used.

Nope. My guess is that it was used at one point, but the usage was removed and the library was left in place.


MigMigration name should be surfaced in the UI somehow

Currently when the UI creates a MigMigration object to start a stage or final
migration, it creates a UUID and uses that for its name. The name of the object
is significant, because it serves as the object's primary key identifier in
k8s. This is the value you would use if you wanted to drop to the cli and
oc get migmigration -o yaml <name> to get the object details.
We'd like to surface this value somehow in the UI. It doesn't make sense to
add as a column, since it's not semantically useful on its own.
I'd propose we add a tooltip to the migration row that when clicked, shows
the uuid with something like an easy "copy-to-clipboard" icon.

@vconzola let me know what you think about this one, I can elaborate more if you have any q's.

@eriknelson What's a UUID look like - how many characters?

They're 37 characters: 918e4c8a-4f85-43f3-be6f-f9dd73ac7897
On Mon, Aug 3, 2020 at 4:56 PM Vince Conzola<EMAIL_ADDRESS>wrote:

@eriknelson https://github.com/eriknelson What's a UUID look like - how
many characters?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/konveyor/mig-ui/issues/982#issuecomment-668237182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADEKNKJBKX6LKPQHITVECDR64P6LANCNFSM4PQ2PK7Q
.

Mockups here: https://docs.google.com/presentation/d/1GzKTrUY9p_JBS4_jwh9gqxEnjLK_YxGsREOKLvxPvVI/edit?usp=sharing

Struggling to add a copy button inside the tooltip contents as PF tooltips close when you hover over the contents. PF issue has been open for a while here: https://github.com/patternfly/patternfly-react/issues/1457


Brown clusters aren't very useful

Looking at the Brown clusters of the lower case words:
81.5% fall into cluster 0, (429279/526544)
02.4% fall into cluster 22,
<1% fall into every other cluster.
Perhaps the problem is that the clustering capped at 1000 clusters?
import operator
import spacy
parser = spacy.en.English()

clusters = {}
count_of_lower = 0
for w in parser.vocab:
    if w.is_lower:
        count_of_lower +=1
        clusters[w.cluster] = clusters.get(w.cluster, 0) + 1
sorted_clusters = sorted(clusters.items(), key=operator.itemgetter(1),reverse=True)
print('lower case tokens: ' + str(count_of_lower))
for i,c in enumerate(sorted_clusters):
    if i>10:
        break
    print('% of tokens in cluster_' + str(c[0]) +': ' +str(round(c[1]/count_of_lower,3)))

A potential solution would be to compute the hierarchical Brown cluster for each token in the form '100010100100...' where each bit is a fork in tree and and then have the token.cluster be a function that takes 'depth' as an argument, going from outside in. So a depth of 0 would be the token itself, depth of 2 would be a cluster of 4, depth of 5 would be a cluster of 32, and so on. This would allow comparison of nearby clusters of an arbitrary size.

I think you're getting tricked by Zipf's law here. Count the cluster frequencies by token, not by type --- or weight the count by exp(w.prob). You'll come to a very different perspective.
Admittedly the Brown clusters are a bit mysterious. If you inspect the clustering, they don't look at all impressive. But empirically they improve the accuracy of the POS tagger and parser quite a lot.

Neat, running the above weighted by probability each word does paint a somewhat more balanced set of classes, but I'd still argue that the current clustering implementation isn't reaching its full potential.
This paper has some interesting stuff on this topic: http://www.derczynski.com/sheffield/papers/brown_impact.pdf

The best possible c behaves oppositely with class-only and path-only information. For class-only, with small corpora, c should be high (or set to |V |); as the corpora grow, so the best c levels off. Conversely, for path-only, small corpora benefit from lower c, whereas larger corpora do better with high values of c. This is be- cause as c → |V |, more path information is added, whereas clustering information decreases, as suggested in Section 3.

With the class-only experiment, performance peaks and then declines as c → |V |, as expected (Section 3).

In the path-only experiment, performance increases with both |T | and c.

Curiously, the metric you mention (PoS tagging) appears to peak in their tests at a c=640, I imagine this is because semantic similarity matters far less, so if you build your clusters with PoS in mind it results in clusters that are semantically dissimilar but very well at the task.

For PoS tagging, we see that there is a peak performance with c = 640, after which accuracy drops unstably (Table 4). This matches our expec- tations. In fact, the performance for c = 1000 (the value used to generate the original clusters for the CMU tagger) is a local minimum in our test. The PoS task involves a lot of other factors, and so is not as close an estimate of clustering quality as the NER task is, but it does make use of both the clus- ters and the hierarchy. No clear result came from varying |T | with a fixed c (Table 5), unlike in NER, where increasing |T | had a strong impact.

Well, the feature functions for the POS tagger, NER and parser actually use 4 and 6 bit prefixes of the cluster codes, following Koo and Collins (2010).
The details of the clustering stuff are overall pretty unsatisfying, but it does all work, and re-engineering the details of the linear models aren't a priority at this point. It's much better to complete the switch to the neural network. I'll therefore close this.


Backing up wekan causes permission denied error

I am attempting to back up wekan-snap, so I followed the documentation on the wiki. This is what I get when I try to back it up.
$ sudo wekan.database-backup test
using default value: MONGODB_BIND_UNIX_SOCKET='/var/snap/wekan/64/share'
using default value: MONGODB_BIND_IP=''
using default value: MONGODB_PORT='27019'
using default value: MAIL_URL='smtp://user:pass@mailserver.examples.com:25/'
using default value<EMAIL_ADDRESS>ROOT_URL='http://<IP_ADDRESS>:80/wekan'
PORT='80'
using default value: DISABLE_MONGODB='false'
using bind ip
2017-09-26T10:20:22.327+0400	Failed: open test: permission denied

I currently have 0.43 installed.
Also, since this is a snap, is it supposed to go to the system /var directory? Or is there a snap specific /var directory that it uses?

@9a3eedi
I updated backup documentation, please try again with those changes.

@xet7 it works now. Thank you very much!

thank @xet7
@9a3eedi to explain about permissions where snap package can access.
Snap's writable service data for snap's sandbox are store in
/var/snap//common
/var/snap//current
There are also then writable data per user if you invoke snap's application from user's shell, that is then under:
/<user's home>/snap//common
/<user's home>/snap//current
In case of wekan, we do not really use per user data as it runs as a service.

Dear i can't restore a backup, it says there are conflicts :
2018-09-26T09:40:08.022+0200    error: multiple errors in bulk operation:

E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "4ZxcrsoDESWFzDAz8" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "7bkxnMKjS7md9ig5A" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "yasNJiLvHEYqEYwaK" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "LpCWiiD4oyj6tiNCw" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "RLmryYL8pLqcnkwpg" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "87kk44ji227rsNRLZ" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "2THA3N6zxZLREBr2r" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "8R3276eRKxEaYJ2nE" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "hXZWcrMssm4ydqsPe" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "hJYaWCNhYuc2arGWx" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "pa3PigQSnJwgoJx4g" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "z8ovZLZH3PRDwHyHE" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "9o378YKLh3d6vu3eS" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "G7eBg2b3XAzkRPxPB" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "oPmha2kbANPTALBHG" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "E4PMa59fJgobppzvd" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "Zf8vutK5ANeWQ2pp5" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "KZ38SJqfkaAytbRs4" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "z84YzTySLturvcjgH" }
E11000 duplicate key error collection: wekan.meteor-migrations index: id dup key: { : "KKtQer2zApiHtDknD" }

i tried to reinstall but now i can't backup i get this : Backing up users and roles is only supported for clusters with auth schema versions 1 or 3, found: 5


geo/geomfn: implement ST_LineSubstring({geometry,float8,float8})

PR: https://github.com/cockroachdb/cockroach/pull/58125
From release notes:

Implemented the geometry-based builtins ST_LineSubstring({geometry,float8,float8}). [#58125][#58125] {% comment %}doc{% endcomment %}

in the docs: https://www.cockroachlabs.com/docs/v21.1/functions-and-operators.html#spatial-functions


only starts once

I can only see the interval and what's it going for once it doesn't come up multiple times

just like that
is it normal???

Hi! Please refer to #5 : https://github.com/PrinceS25/StockAlertBot/issues/5#issuecomment-751171453


Breaking202205

https://breaking202205--helix-project-boilerplate--adobe.hlx.page/

Assuming we won't merge this right away...

pushed a minor change to the grid display in nav, see before vs. after:
before (tools overhang hero):

after (tools in nav bar):


Make a PR to cairo repo to change MemBuffer visibility to public

From: https://github.com/software-mansion/protostar/pull/2144#discussion_r1246606530
MemBuffer from cairo repo https://github.com/starkware-libs/cairo/blob/main/crates/cairo-lang-runner/src/casm_run/mod.rs#L394 should have it visibility changed to public so it can be used in our cheatcode hint processor.

https://github.com/starkware-libs/cairo/pull/3588

related https://github.com/software-mansion/protostar/issues/2164


Feature request Deep Learning models

Is your feature request related to a problem? Please describe.
Adding different deep learning models using CNN, ANN, with oversampling & anamoly detection techniques
Describe the solution you'd like
Creating different depp learning models with their visual structure too.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Approach to be followed (optional)
A clear and concise description of the approach to be followed.
Additional context
Add any other context or screenshots about the feature request here.

@UppuluriKalyani can you assign me this issue as gssoc-ext contributor.


Undefined offset: 25 in phpFITFileAnalysis.php on line 1228

1228: $this->types = $this->endianness[$architecture];

Undefined offset: 25 in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1228
Undefined offset: 139 in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1228
Undefined offset: 97 in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1228
Undefined offset: 113 in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1228
Undefined offset: 119 in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1228
Undefined offset: 109 in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1228
Undefined offset: 108 in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1228

array_unique() expects parameter 1 to be array, integer given in vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1535

Hi @OlafvdSpek - obviously something is going quite horribly wrong! The $architecture should only be 0 or 1 for big or little-endianess.
Are you able to share a FIT file where this is happening? What type of activity and what device have you used to record it?
Cheers, Adrian.

Sure, what's your email address?

You can save a file as a .TXT and upload it here if that works? Cheers

I'd like the file to stay (semi) private..

<EMAIL_ADDRESS>

Hi Adrian,
Any luck?

@adriangibbons Ping ;)

Hi @OlafvdSpek
Are you still having issues? You'd shared two files with me via email:
garmin.fit
As discussed via email, neither of us is able to reproduce the error and everything seems OK.
1.fit
This doesn't seem like an activity is recorded in the file. This is confirmed if I upload to Strava and there isn't any data:

$ composer info
adriangibbons/php-fit-file-analysis v3.2.2 A PHP class for analysing FIT files created by Garmin GPS devices

$ md5sum *.fit
acc6b419f903165f9a6cc4dd8e8a60d2  1.fit
60df7628d0105886a16f63dfcfdc846c  2.fit
c4b2b452ad461e1415b6f337d3e4523b  issue-49.fit

$ php test-fit.php 1.fit
PHP Warning:  array_unique() expects parameter 1 to be array, integer given in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1543

Warning: array_unique() expects parameter 1 to be array, integer given in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1543

$ php test-fit.php 2.fit
PHP Notice:  Undefined offset: 14 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232

Notice: Undefined offset: 14 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232
PHP Notice:  Undefined offset: 73 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232

Notice: Undefined offset: 73 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232
PHP Notice:  Undefined offset: 14 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232

Notice: Undefined offset: 14 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232
PHP Notice:  Undefined offset: 14 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232

Notice: Undefined offset: 14 in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 1232
PHP Fatal error:  Uncaught Error: Unsupported operand types in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php:2720
Stack trace:
#0 /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php(1143): adriangibbons\phpFITFileAnalysis->processHrMessages()
#1 /home/forge/fit/test-fit.php(6): adriangibbons\phpFITFileAnalysis->__construct('2.fit')
#2 {main}
  thrown in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 2720

Fatal error: Uncaught Error: Unsupported operand types in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php:2720
Stack trace:
#0 /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php(1143): adriangibbons\phpFITFileAnalysis->processHrMessages()
#1 /home/forge/fit/test-fit.php(6): adriangibbons\phpFITFileAnalysis->__construct('2.fit')
#2 {main}
  thrown in /home/forge/fit/vendor/adriangibbons/php-fit-file-analysis/src/phpFITFileAnalysis.php on line 2720

This doesn't seem like an activity is recorded in the file (which is only 494 bytes) - maybe something went wrong when recording this.

Hi Adrian,
Maybe, but shouldn't the lib be able to handle that without PHP warnings?
I've emailed you 2.fit, a file that causes other warnings / errors. Let me know if you can reproduce them.

Just made a quick commit that should fix the issue with 1.fit
Family, work, life, other projects have been preventing me from progressing the other issues.

Thanks!


Legacy JS doesn't work after build

When using this (for testing purposes):
{{ craft.twigpack.includeSafariNomoduleFix() }}
{{ craft.twigpack.includeJsModule("app.js", false) }}

JS doesn't work anymore, simply doesn't get executed and no errors.
I couldn't really find the issue, but it seems that the optimizer is misconfigured and the chunks are wrong.
styles: {
  name: settings.vars.cssName,
  test: /\.(pcss|css|vue)$/,
  chunks: 'all',
  enforce: true
}

When removing this block vue works fine but the chunks are wrong of course. Seems that putting al styles in one chunk is breaking js/vue.
When using the module version everything is fine.
Sorry for the bad description, but I'm still learning webpack.

Seems related to this:
https://github.com/webpack-contrib/mini-css-extract-plugin/issues/113
https://github.com/webpack/webpack/issues/7300
Manually including styles.js (injecting inline or prepending to bundle) fixes the issue.
When not using styles in vue components this js file is empty but if you don't include it screws up the webpack internals and the entry js just fails quietly.
For now I fixed it by not writing styles into the components and removing the splitChunks config object and just including app.css.
The legacy JS still fails in IOS 10.2 (simulator) but that might be unrelated.

Let me know if you find any more information @thaoms

Maybe I can add my two cents here. Even when we use
{{ craft.twigpack.includeJsModule("app.js", false) }}
Javascript isn't executed on IE11 and we don't get any error.
This thread and problem took me on a Google journey and it seems that this is a WebPack bug. When WebPack combines all CSS assets into a single file it also generates a styles.js file. This needs to be loaded as well for it to work in IE11.
My solution was to add
<script nomodule>{{ craft.twigpack.includeFileFromManifest("styles.js") }}</script>
to my _layout template after
{{ craft.twigpack.includeSafariNomoduleFix() }} {{ craft.twigpack.includeJsModule("app.js", true) }}
Then you also need to installa Promise polyfill and import it in your app.js.
This made it work for me, but I don't understand why Andrew or others don't seem to have this issue and why it does happen here. I started a new project from scratch and still had the issue.

I was having the same issue as you @TomDeSmet and resolved it by reading this post: https://github.com/webpack/webpack/issues/10057. Loading styles.js from the manifest is the approach I took as well.

The reason not everyone is seeing it is that it happens only when Vue components from node_modules includes in .vue single file components, and is apparently a webpack bug that we can hope is fixed in webpack 5.
But yes, the work-around is to include {{ craft.twigpack.includeFileFromManifest("styles.js") }}

I am happy to report that this has been fixed in webpack 5!


Issue 454 add matplotlib

@durack1

@doutriaux1 thanks, this fixes #454 - I will wait for the tests to pass and merge


hwclock doesn't work

hwclock --systohc --localtime --debug
hwclock from util-linux 2.20.1
hwclock: Open of /dev/rtc failed: No such file or directory
No usable clock interface found.
hwclock: Cannot access the Hardware Clock via any known method.

Thanks for reporting the issue.  We currently do not have support for /dev/rtc, see ls -al /dev for a list of supported devices.
Please provide feedback on the uservoice page so we can prioritize this scenario.

Closing since no feedback since April. Please upvote on UserVoice if you'd like us to consider in the future.


Every 60 seconds and above

I am trying to run a job 'every 190 seconds' but nothing works. I read somewhere the max for seconds is 59 seconds. Is this true?

I have also noticed this behavior.

@gustawdaniel its not working either


are you or is anyone looking at updating to support the latest ui.bootstrap lib

ui.bootstrap has been updating it's directives to uib-xxxxx such as 'collapse is now deprecated. Use uib-collapse instead'.  are you or anyone looking at updating the dashboard framework to the new directives?
thank you
Frank

Version 0.11.0 uses angular-bootstrap 0.14.x with uib-...


⚠ Bofrost.lu has degraded performance

In f852948, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2109 ms

Resolved: Bofrost.lu performance has improved in 8ee7bfb after 28 minutes.


Adding authors to a post should add them to the access list

Javascript only, because we decided it'd be awkward to do it behind the scenes and you might want to contrive the situation so that's not the case or you might really care about the access list matching up or whatever.

If this is what I'm thinking it is, it really confused me when I first starting writing on the site. Would be happy to take this on.


Stuck on "generating payload"?

Hi,
I just grabbed Chuckle and the latest tool pre-reqs and once I run chuckle.sh, I'm fine until the point of "Generating payload..." and then I'm hung there forever.  Any ideas what might be gumming things up behind the scenes?
Thanks,
Brian

Hey, sorry you're having issues. I find that veil-evasion can be little tricky to get installed and working sometimes. Make sure you are able to make a GO reverse HTTPS payload in veil, if this fails it's likely the issue.
Craig

Craig,
You're right on.  I had run veil-evasion and thought it installed correctly, but running the command by itself now seems to kick off the install...again. Hmm I'll work through this and see if it doesn't solve my chuckle problem, thanks!
B
On Mar 19, 2016, at 11:52 PM, craigsblackie <notifications<EMAIL_ADDRESS>wrote:
Hey, sorry you're having issues. I find that veil-evasion can be little tricky to get installed and working sometimes. Make sure you are able to make a GO reverse HTTPS payload in veil, if this fails it's likely the issue.
Craig

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/nccgroup/chuckle/issues/4#issuecomment-198848214

I'll close this for the meantime, let me know if you have any more issues.


implemented Event::FilesDropped for Windows

Disclaimer : this feature is not ready to be merged yet
Related forum thread: https://en.sfml-dev.org/forums/index.php?topic=19096
I have implemented this feature for personal use. I write simple map editors and other small tools with the aid of SFML because I don't want to bring out the big guns like Qt. For something like a map editor, the drag and drop functionality is very handy, especially for textures and materials.
As can be seen in the linked forum thread, I am not the only one wanting this feature. Therefore, I went ahead and created a pull request. This feature is not complete yet but it already works on Windows. I figured that once I take the first step maybe others are more inclined to chime in and collaborate.
Progress:

[x] Modify sf::Event to store the relevant information
[x] Implement interface in sf::Window (see setFileDroppingEnabled)
[x] Implement backend for Windows
[ ] Implement backend for *nix
[ ] Implement backend for OSX
[x] Add an example or another kind of test

Issues:

sf::Event::FileDropEvent is inevitably part of a union, so I couldn't use a vector but had to resort to a raw pointer. Personally, I think this is not a big deal because the usage is still very straight forward:
for (int i=0; i<event.droppedFiles.count; ++i) event.droppedFiles.files[i];
But maybe there's a better solution?
As sf::Event::FileDropEvent only stores pointers but not the filenames themselves, their lifetime is bound to the window, not to the event. Is there any case in which this window-specific buffer can cause trouble?
Am I right in assuming that WindowImplWin32::processEvent doesn't need to be thread-safe?

Unfortunately, I'm not versed well enough in X11 to implement this feature for Linux, but I have found this which might be handy for anyone wanting to implement this feature.

Question: do Windows/Linux support drag operation for other kinds of data (e.g. text, image, ...)? Mac does, that's why I'm asking. It actually works like a clipboard for that. So, like in #1221, we can start by only supporting one kind of data (i.e. paths) but it'd be nice to have a rough idea how things could be extended to support other data formats.
In that respect, maybe a different API would be good. With 3.x we can think about using std::variant-like. But if we want to have something in 2.x, maybe something like this could be more flexible:

struct Data {
  template <T> T get();
  bool isPath(); // if true then get<String> is legal
  bool isImage(); // if true then get<Image> is legal
  // ...

  // or maybe something like that:
  struct Kind { Path, Image, ... };
  Kind getKind();
};

/* implementation defined */ Cookie;

struct DropEvent {
  Cookie cookie;
};

struct Window {
  // Let's say the Data for a Cookie is valid at least until
  // the next display/popEvent/whatever makes sense.
  Data getDropData(Cookie);

  void enableDropData(Kind kind, bool enable);
};

What do you think?

It looks like Linux does? It sends a MIME type with the data, from what I can tell here and the code from the link at the end of the issue text (although I'll admit I didn't understand most from either places).
Windows does seem to, since I just dragged highlighted text from Firefox directory to Visual Studio and it put the text in the editor. I can also drag an image directly from Firefox into MS Paint. (I'm lost trying to find documentation on how this works though. It seems related to clipboard formats? From what I read here, Windows can actually give multiple formats at once.)
The API just proposed can't support Windows having multiple types at once, so if that is chosen SFML would need to decide what is best and only give that (or generate an event for each supported type, but that doesn't sound good either).

Linux supports multiple formats as well.  A lot of software wrongly just chooses the first in the list, which causes things to not work properly quite often (try dragging some selected text to Dolphin for example, the first entry is a pointer to the text rather than the text itself (a later option), and it writes 4/8 bytes (the pointer itself) to a new file.  Always give the option to specify the formats and give the list of all available on linux.

Has there been any progress? Fytch's idea seems outstanding.

A bit late here, but instead of storing a raw pointer (not very user friendly) why not wrapping it into a custom class, which will manage it properly without any risks?
(and maybe then implement a forEach function or use iterators to iterate in the list, just like a std::vector or a std::list would, (we could even use std::list maybe? since I don't see a reason not to have FIFO behavior while reading the content of any drag'n drop event)
I don't know how much could be implemented of hat I said, or even if it is relevant since I am not a maintainer of the library, just a random user sorry in advance


Site Content

https://docs.google.com/document/d/1dC05GsqblST2bUySsomD3agtU6_UjXZXIaOYprqQPSY/edit?usp=sharing

a program overview
a class schedule for all sections in the current semester
a detailed page describing each course, the topics covered, the professor, and the grading
a detailed page describing each professor, their contact information and the courses taught

Each team should include detail pages for at least 3 courses and 3 professors. Try to include more informative details than what is currently included on the public website. You might consult the detailed week by week topic outline for each course.

Made changes for the program overview! Still waiting on the class schedule


Add a Hacktoberfest badge component

Hacktoberfest is upon us, and what better way to celebrate then to make it easier for repo maintainers to show the right badge for it? There is no official badge, but Shields.io offers a badge that is recommended by the Hacktoberfest team.
This issue is meant to be a good first issue to get into this repository, and into open source contribution over all, as it will require to touch few existing files and is documented extensively. If this is your first contribution to open source, I encourage you to try this issue.
Please read our contribution guidelines before picking this up, it will also lead you through the process of setting up the repository.
Describe the solution you'd like
This is what the badge should look like in its most basic form:

It should be accessible as <HacktoberfestBadge year={2020} />, with an optional prop suggestionLabel that can be used as <HacktoberfestBadge year={2020} suggestionLabel='good first issue' /> to render to:

The badge needs to be exported from the badges index barrel file.
Please note that you will also have to provide test coverage and documentation for me to be able to accept the PR.
Additional context
A good badge to base this new badge on is the Codecov badge.
You can find out more about the badge on the shields.io page. It is in the issue-tracking category as "GitHub Hacktoberfest combined status" and "GitHub Hacktoberfest combined status (suggestion label override)".

Hello, Is this available? I would love to take this!

Yes. Please keep in mind that for this to count as part of Hacktober fest, the PR must be provided in October. I will also add more beginner-friendly issues over time, with the idea of giving multiple people the chance to get to know the repo.

Thanks, i'll keep in mind that!

Hello @dbartholomae,
Could you please assign the issue to me?
Submitted a PR, waiting for your review.


Remove discrepancy between query string token in plain http vs WebHook

Currently, WebHooks use code=... (coming from WebHook SDK), while plain http use key=... (coming from our own logic).
One way or another, let's find a way to make them be the same.

I'm planning on creating a pull request against the WebHooks SDK to make it configurable. I investigated but didn't want to do url rewrite on our side, because the SDK error messages would still mention 'code' which would confuse our users.

As long as we can push for a new version of WebHooks SDK to be published promptly, that would be a good approach. So key everywhere.

So further investigation shows that the 'code' nomenclature is spread throughout many of the SDK WebHook receiver implementations. It's a pattern, not a one off to tweak/configure. See here. Therefore, we probably should just bite the bullet and follow suit, using a 'code' query parameter for our key values as well.

+1 - this is likely the right thing to do for debug-ability


Add console warning when running 2n.fm with no TURN configured

The TURN variables are configured in the .env.local file:
VUE_APP_TURN_USERNAME
VUE_APP_TURN_CREDENTIAL
If doing local dev, or even on production, a console warning should appear saying no variables found.

Solved in bfaea90473bdfa92c59fc8dd845e14c13e54c66e


Initialize Font::m_stroker

The following code crashes on android with the latest SFML version.  When reassigning font3, the font that it contained is destructed and it crashes in the cleanup function because m_stroker was not initialized.
The fix is to simply initialize m_stroker in the constructor and also copying its value in the copy constructor.
Forum post: http://en.sfml-dev.org/forums/index.php?topic=19776.msg142320#msg142320

Looks good to me. No other attribute seems to be uninitialised. :+1:

Don't forget operator=. The member variable should be swapped, too.

Ok, the commit has been updated to also copy the variable in operator=

Good catch @Bromeon!

:+1: from the person that forgot about it :stuck_out_tongue:

This PR has been added to my merge list, meaning it will be merged soon, unless someone raises any concerns.

src/SFML/Graphics/Font.cpp:444: warning: (warning) Member variable 'Font::m_stream' is not assigned a value in 'Font::operator='.

You'll have to assign "something" for m_stream by using the Android #define thingy. See the copy/constructor.

It isn't really related to this pull request, but I can add it if you want. As a separate commit or merged with the previous one?

Since this is very specific to Android and not related to m_stroker, I'd say a different commit. But it's worth fixing. :-)

Yeah I didn't really think about it. I just saw the line in the static analysis report next to the others and thought it would make sense to report it here. Thanks for adding it!
Comparing operator= and the copy constructor I wonder if the m_stream shouldn't also be set to NULL. I guess we'll need @MarioLiebisch.

It is being set to NULL because temp.m_stream is NULL.
But now you no longer have to change the operator= if you decide to at some point change the code in the copy constructor, so I think this code is better than assigning NULL directly.

Ah I see. Sounds good.

Yes, if possible, copy-and-swap should be preferred over custom logic in assignment operators -- it's more clear as it's a common idiom. Even our inline variant that we use in SFML ;)

Merged in 23ea17eab7d31c017bd2f326090f91faa29b5c31