Fix type errors when using fully-typed inputs like QueryFilters<Data, Error>, and test all QueryClient methods to detect similar issues

After adding type parameters to QueryFilters, some methods are proving to be incompatible with fully-typed usage
This PR adds a complete suite of fully-typed calls to all QueryClient methods, and method overloads which accept these inputs.
The core issue is that while TFoo & { bar: string } can be assigned to TFoo, if there are any callbacks (ie. predicate) that pass TFoo back, typescript will attempts to assign TFoo to TFoo & { bar: string }, and the default of unknown has the same problem, this is a limitation of subtyping in typescript. To get around this we make the QueryFilter type a generic parameter which can adopt the type of the method argument

This is impacting the new tRPC client which the QueryFilter typings were originally added for, in two ways:

Firstly the error type needs declaring because tRPC doesn't expect an Error to serialise across the wire, second once declaring this the data type shows up as incompatible because the QueryFilters type is ignored. This PR adds generic type parameters for QueryFilters to improve both situations

Hey @TkDodo I don't want to rush this one in and this changeset is wide-reaching. I experienced quite a few potential breaking changes I had to work around while developing this, so want to be really sure I'm not going to cause problems.
The described issues on 5.62.0 should not impact current usage of RQ, but if anybody wants to use the QueryFilters<...> form then they'll quickly find they can't call many of the QueryClient methods with it. This PR addresses that

Okay simplified the changeset quite a lot now, instead of adding overloads it's looking a lot more like it looks on other methods which took generics and the new tests & existing builds seem happy


the statement in line 78 should be repalced

"hbias[i] += lr * (ph_mean[i] - nh_means[i]) / N" is my modification.

Why you put dataes such as "int N;
int n_in;
int n_out;
double **W;
double *b;"
in public,why not in protected or private?


Bug - When you click the import private key, getting blank page

When you click the import private key(in create mnemonics page), getting blank page

It's fixed in latest

As of now, i'm not able to see the link for the import private key in the mneumonics page , i believe it is removed


feat(docloaders): undetected-playwright

Uses a virtually non-intrusive method to hide browser control features. Can bypass most human-authentication algorithms.

Really cool! Can you give me a webpage example that can bypass? Let's have a chat on Discord ;)

I wrote some simple examples and put them here https://github.com/QIN2DIM/undetected-playwright/tree/main/examples
In addition, there are special ways to make the browser run in a "headful" way inside the container.

In addition, I maintain a number of projects that directly address multimodal human-computer challenges, but it looks like Scrapegraph-ai is still in its "infancy".
So I don't want to do too much intrusive PR, but if the time is right we can talk about it later haha.

Yess I checked your repo and tested your PR, actually the time is perfect, let's discuss it later ;) DM me on Discord

yeah lets talk! write me on<EMAIL_ADDRESS>


Use consisting file names in the download tab

The resulting filenames when downloading semantic layers in the download tab in most cases follow the pattern: {id}.{playname}.-{layer slug}.{file format ending} but there are some inconsistencies:
e.g. Network data: {${play.id}-${play.name}.graphml} (no dash no slug of semantic layer) https://github.com/dracor-org/dracor-frontend/blob/main/src/components/DownloadLinks.js#L37
TEI has a dot instead of a dash ${play.id}-${play.name}.tei.xml, see https://github.com/dracor-org/dracor-frontend/blob/main/src/components/DownloadLinks.js#L146
I haven't checked all, but maybe we could use a consistent naming convention always including the semantic layer slug ?

@ingoboerner How about the following naming scheme:

[play.id]-[play.name].network.{csv,gexf,graphml}
[play.id]-[play.name].relations.{csv,gexf,graphml}
[play.id]-[play.name].spoken.{json,txt}
[play.id]-[play.name].stage.txt
[play.id]-[play.name].stage-with-speakers.txt
[play.id]-[play.name].cast.{json,csv}
[play.id]-[play.name].tei.xml
[play.id]-[play.name].rdf


GithHub icon in footer and Copyright date

I was at a small local event, chatted about Zeek, got someone interested. He browsed to zeek.org to find its way to GitHub. Instinctively scrolled all the way down and immediately and drily said: "Copyright 2020, no GitHub icon".
Might also be nice to put an icon to the forum there.

Somebody (likely @0xxon or our contractor) recently updated the year — it does now say 2024.
I figured out that the social icons are implied by the theme we're using, and I don't immediately see any way to add to them. There's a checkbox that toggles their visibility in total, though, and since we're already listing all the social sites in the footer I simply unchecked that for now.


Add dependency from classesDirs to DefaultSourceSetOutput

We removed this dependency in an earlier change, though it caused a regression where 'classes' no longer depends upon 'processResources'. While this behavior is not strictly necessary, we add it back for backwards compatibility.
Fixes: #22484
Reverts change from: https://github.com/gradle/gradle/commit/4d1aadc0ee64586d397491c3bab0fe6587f37e48

@bot-gradle test RfN

OK, I've already triggered the following builds for you:

ReadyForNightly build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build


gorm not supoort  interpolateParams

set interpolateParams=true in DSN will allow sql use client side param bind other than service side.
go-sql-driver/mysql/connection.go have a Query witch will check this opt and do that. but in gorm, it seems it not support this?

I think you can still enable this feature by having interpolateParams=true in DSN?


Break time

It would be cool to have breaks supported.
Ending an attendance block and creating a new one it not really sufficient if you only want to track a break, e.g. for lunch.
Maybe as a switch itself. But those switches would need to be synced somehow, e.g. that turning the "main attendance" switch off also deactivates the break automatically in case it's still turned on.

@alexregier can you elaborate further why just ending an attendance block is not sufficient?

@alexregier can you elaborate further why just ending an attendance block is not sufficient?

Turning it on and off will provide a correct entry.

Oh great!
I thought one would need to explicitly add a break when using the API.
Good to know that this also works :)
Thank you!


React-Router v7 compatibility?

Now that React-Router v7 is released and is the de-facto version 3 of Remix, will you adopt this template to work with RR v7?

I think a new version of this package should be created @vercel/react-router?
@TooTallNate any plans for this already?

I see that there is a template in the react-router git: https://github.com/remix-run/react-router-templates/tree/main/vercel
Does this mean we no longer need this package? or does this package still achieve additional requirements for vercel deployments?

Would be nice if we could hear something from Vercel

@TooTallNate Any update you can share here? Even a "we're discussing internally" is fine. I think folks here would just like some word from Vercel. Thanks.

Hey y'all – we're tracking support requests here for this https://vercel.community/t/support-for-react-router-7/2769. Let's move the conversation there 😄
Definitely want to add support for RRv7 in the future!


fix readthedocs footer width on sidenav

Codecov Report

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

@@           Coverage Diff           @@
##           master     #306   +/-   ##
=======================================
  Coverage   98.41%   98.41%
=======================================
  Files          18       18
  Lines        1707     1707
=======================================
  Hits         1680     1680
  Misses         27       27

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 5a3ed31...85c40d4. Read the comment docs.


[demoing-storybook] allow for remark plugins inside mdx transforms.

It would be nice to be able to allow some AST based transformations of mdx files running through es-dev-server.
It would allow us to tweak documentation/demo contents by writing our own plugins, but also to use existing plugins (for instance to support emojis).
I was thinking about something like below:
In .storybook/main.js:
module.exports = {
  esDevServer: {
    ...
    remarkPlugins: [
      images,
      [emoji, { padSpaceAfter: true }],
    ],
   ...
}

In @open-wc/demoing-storybook/src/sharedcreateMdxToJsTransformer.js, where mdx method gets called, we can add the plugins like:
const jsx = mdx.sync(mdxText, {
  remarkPlugins: pluginsProvidedByUser,
})

Also see: https://mdxjs.com/advanced/plugins
@LarsDenBakker @daKmoR Would it be ok if I do a merge request for this?

This is now possible with storybook-addong-markdown-docs


Use Ruff as linter and formatter

🚀 Feature Request
Currently, this project uses mypy for type checks and black plus isort for code formatting. It would be great to add ruff as a linter and use its formatter to replace black + isort.
🔈 Motivation
Ruff is very fast and we can use one tool instead for linting and formatting.
🛰 Alternatives
Just add flake8. But ruff also implements those rules.
📎 Additional context
n/a

Hello @RafaelWO, I would like to contribute on this issue.

@amitsgh Sure, I'm happy for any contributions :) Please read the contribution guidelines before you start. Then fork the repo and you're good to go!

@amitsgh as I did not see that you have started with this issue, I took over. I hope this is fine for you 🙂


Not working with laravel 5.8. Status true but image not created in server.

Not working with laravel 5.8. Status true but image not created in server. no issue when running but image not created from destination folder.

same issue here with laravel 5.7.28

are you developing on a windows system?

NO google cloud ubuntu
On Fri, Nov 29, 2019 at 2:32 PM Ajith Lal<EMAIL_ADDRESS>wrote:

are you developing on a windows system?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lakshmaji/Thumbnail/issues/26?email_source=notifications&email_token=AM3II7V6W64XFDGVIR5S4GTQWDLBDA5CNFSM4II7LT5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFOI3IA#issuecomment-559713696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM3II7TUIREMWBP4AI7IJA3QWDLBDANCNFSM4II7LT5A
.

Its working issue is in code thanks for reply
On Sat, Nov 30, 2019 at 4:08 PM Nikul Vnnovate<EMAIL_ADDRESS>wrote:

NO google cloud ubuntu
On Fri, Nov 29, 2019 at 2:32 PM Ajith Lal<EMAIL_ADDRESS>wrote:

are you developing on a windows system?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lakshmaji/Thumbnail/issues/26?email_source=notifications&email_token=AM3II7V6W64XFDGVIR5S4GTQWDLBDA5CNFSM4II7LT5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFOI3IA#issuecomment-559713696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM3II7TUIREMWBP4AI7IJA3QWDLBDANCNFSM4II7LT5A
.

Closing as duplicate #25


Landing page freezes Ubuntu using Firefox

Firefox (Version 36.0.4 | Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:36.0) Gecko/20100101 Firefox/36.0) almost freezes on Ubuntu 14.04 if I visit the front page (not the dashboard, but www.codewars.com without being logged in).
(Originally filed on CW by bkaes)

@bkeas is this reproducable or was it a one time thing?

It happened at least three times, Wednesday, Thursday and Friday. I usually open Firefox in private mode, so it needs to redownload all files and I need to sign in.
At this point, the browser freezes completely, till Firefox recognizes the problem and asks whether the script should continue or whether the execution should be canceled. Unfortunately, I don't have access to the critical machine till monday. I'll try tomorrow with FF 36 on Win 8.1 with a similar configuration (Core2Duo, ~4GB RAM).
The current PC doesn't have this problem (Win 8.1, FF 28; Mozilla/5.0 (Windows NT 6.1; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0), so it might be machine specific.

I managed to get a similar error on the feature request page with a borrowed netbook (Win 7, FF 28; Mozilla/5.0 (Windows NT 6.1; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0; Asus Eee PC; AMD E-350 1.6 GHz, 4 GB RAM). The original dialog message:

Ein Skript auf dieser Seite ist eventuell beschäftigt oder es antwortet nicht mehr. Sie können das Skript jetzt stoppen oder fortsetzen, um zu sehen, ob das Skript fertig wird.

Skript: http://d3l8wp33uu8nxs.cloudfront.net/assets/application-db8e9b5e11a53a89c5eb849aec5970dc.js:14

Roughly translated:

A script on this page is broken or doesn't respond. You can stop or continue the execution to check whether the script finishes:

script: http://d3l8wp33uu8nxs.cloudfront.net/assets/application-db8e9b5e11a53a89c5eb849aec5970dc.js:14

I think this kind of freeze/lock usually only happens on pages with many DOM elements, such as the feature request page.

OK. Yeah the Feature Reqeusts page makes more sense. Its kind of a mess right now. The landing page is a bigger concern, that shouldnt be a heavy page.

Firefox definitely freezes a little bit on my current machine (Notebook from 2009; Core2Duo T6500 @ 2x2.1GHz, 4GB RAM, Win 8.1 [64bit]). The network analysis showed that http://cdn.mxpnl.com/libs/mixpanel-2.1.min.js took the most time, but that could be due to the hickup of ~3-5 seconds.
I just noticed that the console shows the following message right after the browser gets responsible again:

too much recursion www.codewars.com:3

Unfortunately, I can't check the profile.json generated by the profiler (Firefox takes forever to show it), but a simple codewars loading page rendering in private mode generates a ~5MB profile, so there might be something up. I'll definitely post the profile generated by the first machine on Monday.

I'm now back at the first machine (the one where Firefox freezed), this time using Windows 7. Firefox hangs for two seconds, during a DOM event (event logging) that happens after DOMContentLoaded.
Again, the errors message "too much recursion" gets logged the same moment Firefox gets responsible again. The generated profile is again ~5MB (gist) and cannot be inspected in Firefox :/.

Did you change something? The page doesn't freeze anymore on with the first configuration (FF 36, Ubuntu 14.04), the profile.json shrank to ~4MB (-1MB), and the previous ~2second DOM event went down to ~42ms. I'll try again later, but so far the issue seems to be gone.

I updated the intercom library yesterday. That may have been the issue. I'm fairly certain its caused by a 3rd party library but haven't gotten a chance to look into it yet.

I updated the intercom library yesterday. That may have been the issue.

It still happens from time to time, but I can't really provide a reliable way to reproduce this behaviour, except

clean cache (or start private browsing session) in FF,
visit www.codewars,
repeat step one if nothing happened on step two.

I haven't tried any other browser, so it might be FF specific.

It also happens in IE (IE11, 11.0.9600.17690 on Win8.1x64) occasionally:

[Rough translation] A script with long execution time prevents that "codewars.com" reacts.

Additionally, the tab crashed when I tried to cancel the execution. This can be reproduced by simply visiting "www.codewars.com".

Still happens in FF and IE on the front page. Additional, when one tries to hook up the browser's debugger, the debugger freezes completely (tested in FF).

The (small) freeze also happens on kata's with many comments, as http://www.codewars.com/kata/550f22f4d758534c1100025a/discuss#554068a05b88f1d903000051 has just passed the freeze threshold.

I'm having a similar problem in iceweasel 37 on debian
Error: Script terminated by timeout at:
jQuery.wait@http://d3l8wp33uu8nxs.cloudfront.net/assets/application-db8e9b5e11a53a89c5eb849aec5970dc.js:31:7770
t@http://d3l8wp33uu8nxs.cloudfront.net/assets/application-db8e9b5e11a53a89c5eb849aec5970dc.js:32:7376
t@http://d3l8wp33uu8nxs.cloudfront.net/assets/application-db8e9b5e11a53a89c5eb849aec5970dc.js:32:8392
t@http://d3l8wp33uu8nxs.cloudfront.net/assets/application-db8e9b5e11a53a89c5eb849aec5970dc.js:32:8392
t@http://d3l8wp33uu8nxs.cloudfront.net/assets/application-db8e9b5e11a53a89c5eb849aec5970dc.js:32:8392

Solved at my end by disabling adblock, can't explain why though since it only seems to block analytics and facebook. Might not be the same bug.

Still happens occasionally. No idea why. AdBlock is deactivated on CodeWars, but I'm also using disconnect.me

*Bump*: Just happened on Windows 7 Home Premium, using Firefox 38.0.5.


[Bug]The user is not allowed to add the special character in there page name and widget name

Description
[What happened]The user is not allowed to add the special character in there page name and widget name
Steps to reproduce the behaviour:

Go to Application
Click on Page name /Widget name
Try Entering Special Character /UTF 8 Character
Verify the display

Actual  Result: The user is not allowed to enter the Special Character
expected result: The user should be allowed to add Special Character as Page and Widget name

Version: [Cloud ]
OS: Windows
Browser Chrome
Environment [production]

@RakshaKShetty can you please mention which utf-8 character you used in the page name? There was a solve for this issue: https://github.com/appsmithorg/appsmith/issues/2210
Thanks so much.

Hi @kaushik94 I am not able to use @,$,& all these character in my page names

Understood. Working on it.

Special characters should not be allowed as discussed in #765

For more detail, since page names & widget names are used as variables while writing JS code, we cannot allow special characters in their name. This will violate JS coding conventions.


Not making attack rolls

Wow this module is amazing!!! Its the targeting and movement is working great for me but it is not rolling attacks for some reason.

Edit: seems to be an incompatibility with MIDI QOL

I thought I had midi-qol working at some point. I'm probably not up to date. Let me look into it...

I remember now. I disabled it when 0.7.5 hit because a few of its dependencies hadn't updated yet. I assume that, since you're using it, that isn't a problem any more. I'll update and correct the compatability issue.

From midi-qol documentation:
"Better Rolls If you are using BetterRolls (which is a great module), midi-qol takes over once the hit/damage card is placed by better rolls. This means that resource consumption, template placement and critical/fumble determination is all handled by BetterRolls before midi-qol kicks in. Midi-qol checks hits, saves, applies damage and calls active effects."
I believe this means I need to invoke better rolls directly, if it is enabled, so I will remove the check for midi/minor-qol, and I have tested that damage will be rolled correctly this way with each module combination.

Thank you for looking into that! I do not have Better Rolls installed just MidiQOL! Look forward to using this great mod!

The fix should be live now. I'm verifying it now. I'm sorry it took so long; I'm new to git and finding it absolutely awful compared to other version controls I've used.

I don't think foundry has the updated version yet. I just uninstalled and reinstalled and its still 0.0.1. Appreciate the swiftness of your response!

Works great now! Can I put in a feature request to apply "multiattacks" not sure if that's even possible.


feature(repository): add updateStatus method

Githup API docs: https://developer.github.com/v3/repos/statuses/

Other existing tests are failing, the test that I added is passing:
  ✓ should update commit status


Add instructions for how to install vmstat Ruby package in Sensu's Ruby

Your changelog mentions that the Ruby vmstat package has been removed as a dependency, but there are no instructions on how to install it in Sensu.
With the new 1.0 release, I'm getting this error:
/opt/sensu/embedded/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require': cannot load such file -- vmstat (LoadError) from
/opt/sensu/embedded/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require' from
/opt/sensu/embedded/lib/ruby/gems/2.3.0/gems/sensu-plugins-memory-checks-1.0.2/bin/check-ram.rb:36:in `<top (required)>' from /opt/sensu/embedded/bin/check-ram.rb:22:in `load' from
/opt/sensu/embedded/bin/check-ram.rb:22:in `<main>' Sensu::Plugin::CLI: ["Not implemented! You should override Sensu::Plugin::CLI#run."]

There's a note in the README about the vmstat dependency. https://github.com/sensu-plugins/sensu-plugins-memory-checks#installation. I will clarify the instructions.
You install the vmstat gem with /opt/sensu/embedded/bin/gem install vmstat --no-ri --no-rdoc or the equivalent in your config management.

@eheydrick When I try to install, I get this error:
$ sudo -H --user sensu /opt/sensu/embedded/bin/gem install --no-ri --no-rdoc vmstat
Fetching: vmstat-2.1.0.gem (100%)
ERROR:  While executing gem ... (Errno::ENOENT)
    No such file or directory @ rb_sysopen - /opt/sensu/.gem/ruby/2.3.0/cache/vmstat-2.1.0.gem

You need to be root to write into the sensu dir. Try it sudo'ing to root and not the sensu user.
$ sudo /opt/sensu/embedded/bin/gem install vmstat --no-rdoc --no-ri
Fetching: vmstat-2.1.0.gem (100%)
Building native extensions.  This could take a while...
Successfully installed vmstat-2.1.0
1 gem installed

I updated the README with instructions for installing the vmstat gem.


Graph is always showing dollar price

When switching the currency to e.g. Euro, the price shows correctly in the table, but the graph still shows the Dollar price.
I've tried to refresh the graph, changed lots of settings, deleted my user settings, etc. but it keeps behaving that way.

When I checked GetCoinGraphData. In here "usd" is explicitly called instead of selected currency.

hey @eingedanke could you check if this pull request solves your issue ? It seems to work for me though


error: linker cc not found

hello, when I build this crate or cargo run example  I will get that error
➜  systemstat git:(master) cargo build --release
   Compiling autocfg v1.0.1
   Compiling bitflags v1.2.1
   Compiling radium v0.5.3
   Compiling ryu v1.0.5
   Compiling libc v0.2.82
   Compiling version_check v0.9.2
   Compiling lexical-core v0.7.4
   Compiling memchr v2.3.4
error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: could not compile `lexical-core`

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: build failed
➜  systemstat git:(master) uname -a
Linux ubuntu 5.8.0-40-generic #45~20.04.1-Ubuntu SMP Fri Jan 15 11:35:04 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

➜  systemstat git:(master) cargo run --example info
   Compiling libc v0.2.82
   Compiling ryu v1.0.5
   Compiling radium v0.5.3
   Compiling bitflags v1.2.1
   Compiling memchr v2.3.4
   Compiling lexical-core v0.7.4
   Compiling lazy_static v1.4.0
   Compiling bytesize v1.0.1
   Compiling num-traits v0.2.14
error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: could not compile `lexical-core`

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: build failed

➜  systemstat git:(master) cargo run --example info
   Compiling libc v0.2.82
   Compiling ryu v1.0.5
   Compiling radium v0.5.3
   Compiling bitflags v1.2.1
   Compiling memchr v2.3.4
   Compiling lexical-core v0.7.4
   Compiling lazy_static v1.4.0
   Compiling bytesize v1.0.1
   Compiling num-traits v0.2.14
error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: could not compile `lexical-core`

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: linker `cc` not found
  |
  = note: No such file or directory (os error 2)

error: aborting due to previous error

error: build failed

os version
➜  systemstat git:(master) uname -a
Linux ubuntu 5.8.0-40-generic #45~20.04.1-Ubuntu SMP Fri Jan 15 11:35:04 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

os version
➜  systemstat git:(master) uname -a
Linux ubuntu 5.8.0-40-generic #45~20.04.1-Ubuntu SMP Fri Jan 15 11:35:04 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

solved it
sudo apt install gcc

solved it
sudo apt install gcc


no icon on toolbar button

I just did a fresh install of Nylas v0.4.10 and added the plugin. When I've selected an email to unsubscribe from, the toolbar button doesn't show the icon. I've hovered over it to make sure its the correct plugin. Not entirely sure what needs to be changed as I've got the latest version of n1-unsubscribe(v1.0.2).
Cheers!

Hi @akshaysth! Can you verify that the folder name that you downloaded was n1-unsubscribe not n1-unsubscribe-master? This change in directory name prevents N1 from finding the plugin's assets. A quick way to check is to look at the plugins page in the N1 client (Preferences > Plugins). You should see our plugin like this:

If the name is different then this can be resolved by uninstalling, changing the directory name to n1-unsubscribe and then re-installing.
Thanks for checking this plugin out!

I've downloaded both the master branch and the release. The release folder's directory name was "n1-unsubscribe" and it wasn't working however I re-downloaded the master branch and use your suggestion and that seems to fix it. Quite odd.

Oh weird that is definitely a bug. We will check it out and I'm glad the master branch release worked!


Fix cmake build with custom OpenSSL library

Describe the bug
To compile dynamic Azure SDK libraries I link them against certain OpenSSL distributed with our product:
$ /snap/cmake/912/bin/cmake .. -DOPENSSL_ROOT_DIR=$TREASURE_ROOT/OpenSSL/1.1.1  -DOPENSSL_INCLUDE_DIR=$TREASURE_ROOT/OpenSSL/1.1.1/include -DOPENSSL_CRYPTO_LIBRARY=$TREASURE_ROOT/OpenSSL/1.1.1/a64_lnx/lib/libcrypto.so.1.1 -DOPENSSL_SSL_LIBRARY=$TREASURE_ROOT/OpenSSL/1.1.1/a64_lnx/lib/libssl.so.1.1 -DCMAKE_INSTALL_PREFIX=$TREASURE_ROOT/Azure/main/ -DBUILD_SHARED_LIBS=ON -DCMAKE_BUILD_TYPE=Release -DAZ_ALL_LIBRARIES=OFF

$ make clean
$ make -j5

In file included from /home/hadoop/Desktop/azure-sdk-for-cpp/sdk/core/azure-core/inc/azure/core/http/policies/policy.hpp:17:0,
                 from /home/hadoop/Desktop/azure-sdk-for-cpp/sdk/core/azure-core/inc/azure/core/internal/client_options.hpp:13,
                 from /home/hadoop/Desktop/azure-sdk-for-cpp/sdk/keyvault/azure-security-keyvault-keys/inc/azure/keyvault/keys/cryptography/cryptography_client_options.hpp:13,
                 from /home/hadoop/Desktop/azure-sdk-for-cpp/sdk/keyvault/azure-security-keyvault-keys/src/cryptography/cryptography_client_options.cpp:4:
/home/hadoop/Desktop/azure-sdk-for-cpp/sdk/core/azure-core/inc/azure/core/uuid.hpp:20:10: fatal error: openssl/rand.h: No such file or directory
 #include <openssl/rand.h> //for RAND_bytes

Here is a git format-patch output for the one-liner-fix proposed:
0001-Fix-cmake-build-with-custom-OpenSSL-library.patch.txt

Is this the same issue as https://github.com/Azure/azure-sdk-for-cpp/issues/2560?

The #2560 diff s/PRIVATE/PUBLIC also helps me.


Add calc_isinstance.py example

The example exercises usage of text.scoping.tools.textx_isinstance() to inspect model objects types during traversal.
See #123

Thanks.


fix broken link to ALM

(It could be worth linking to the original(?) publication https://www.cs.auckland.ac.nz/~lutteroth/publications/LutterothStrandhWeber2008-AucklandLayoutModel.pdf too)

Thanks! I added the paper, too, as you suggested.

Cool :)


cdetheme: init at 1.3

Motivation for this change
Gtk2 / Gtk3 theme mimicking CDE / Motif
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'm getting a 410 gone from the URL:
trying https://dllb2.pling.com/api/files/download/j/eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpZCI6IjE1MjU3Mzc1MTAiLCJ1IjpudWxsLCJsdCI6ImRvd25sb2FkIiwicyI6ImI2NTBiZjk0NDEzYjRjYmRmNzgxMWZhNmIyZjAzMzYwNmUzNDIxNzYxYmY0MDMzYTgxYjk0ZDUwZTdkMDlkOWIxYTQ5MzYzODk4NTA0YThmODQxYWMxYTZlMzMwZDY1MmRhMzI3ZDc1ZDRkYTIzMzAwZWI5ZWU3MGJiOTBjNGI2IiwidCI6MTU5MDc1MjU2MSwic3RmcCI6ImJjYTRlNTQ4NDlhMmJkM2I2MGY5MGY0NmU4OTc1MjEzIiwic3RpcCI6IjE3Ni4xOTMuMTQuMTA2In0.1-A_r3bCm-5pJQKviPcWKfrsdm8Xp6AyHiU0d7LJj3w/cdetheme1.3.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 410 Gone
error: cannot download cdetheme1.3.tar.gz from any mirror

Seems like this is a temporary URL?

@sauyon Fixed, thanks

The details section of the theme description seem to suggest that it's licensed under GPLv3, but that's not found in the repo you added as a source. Are you the author of the theme? It looks like you own the opencode repo, at least.
Otherwise, this LGTM.
I'll test this on Arch in a bit.

@sayuon No, I'm not the author, just uploaded it to opencode. I've contacted the author with kind ask to upload it by himself and add license.

@sauyon Theme author uploaded it to github and added license. I've updated the package accordingly.

@romildo Fixed, thank you!

Tested in lxappearance on nixos.
Thanks for your contribution!
Result of nixpkgs-review pr 89163 1

1 package built:
- cde-gtk-theme

@romildo @NickHu @sauyon Thanks!


Add support for doubly-repeated sections (without voltas)

i.e. of the form:
     |::.........|.......::|

which is now supported in the voltas branch of abc-parser.

Fixed by 7580d84ea1327eab006b892ef15906c7f73ddb01 in the morevariants branch

Fixed by 7580d84ea1327eab006b892ef15906c7f73ddb01 in the morevariants branch


Create textodeejemplo.json and _deta.textodeejemplo.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description
The idea is to use it as a website for the "company" (we are 3 friends) that we have to upload apps to the play store as they require it.
Right now we only have 1 upload, and we are preparing two more (they appear on the web).
Link to Website
Link: https://textodeejemplo-1-b4120513.deta.app/
Extra Info
I’m using deta.space to host the website.
To add the domain, they require both an A record (textodeejemplo.json) and a TXT record in _deta (_deta.textodeejemplo.json).
More information about use custom domains in Deta.
Thank you!

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA


Docker 1.6.0 ubuntu package can not be authenticated

Same as issue #8605 (which was for Docker 1.3).
$ sudo apt-get install lxc-docker=1.6.0
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following extra packages will be installed:
  aufs-tools cgroup-lite lxc-docker-1.6.0
The following NEW packages will be installed:
  aufs-tools cgroup-lite lxc-docker lxc-docker-1.6.0
0 upgraded, 4 newly installed, 0 to remove and 168 not upgraded.
Need to get 4,752 kB/4,849 kB of archives.
After this operation, 15.9 MB of additional disk space will be used.
Do you want to continue? [Y/n] Y
WARNING: The following packages cannot be authenticated!
  lxc-docker-1.6.0 lxc-docker
Install these packages without verification? [y/N]

curl -sSL https://get.docker.com | sh
Cannot reproduce
On Friday, April 17, 2015, akafred<EMAIL_ADDRESS>wrote:

Same as issue #8605 https://github.com/docker/docker/issues/8605 (which
was for Docker 1.3).
$ sudo apt-get install lxc-docker=1.6.0
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following extra packages will be installed:
aufs-tools cgroup-lite lxc-docker-1.6.0
The following NEW packages will be installed:
aufs-tools cgroup-lite lxc-docker lxc-docker-1.6.0
0 upgraded, 4 newly installed, 0 to remove and 168 not upgraded.
Need to get 4,752 kB/4,849 kB of archives.
After this operation, 15.9 MB of additional disk space will be used.
Do you want to continue? [Y/n] Y
WARNING: The following packages cannot be authenticated!
lxc-docker-1.6.0 lxc-docker
Install these packages without verification? [y/N]
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/12478.

Some details on my system:
$ cat /etc/apt/sources.list.d/docker.list
deb http://get.docker.io/ubuntu docker main

$ uname -a
Linux vm-ship 3.13.0-32-generic #57-Ubuntu SMP Tue Jul 15 03:51:08 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 14.04.1 LTS
Release:	14.04
Codename:	trusty

This is a duplicate of #12461. Please look at that issue to see how to fix this problem.
If you run into this issue, please look at #12461 and don't comment here.


Lazy loading

I don't think lazy loading is worth it. For now I won't add additional lazy loading configs.


Update README.md, fix typos

I really enjoyed using this project, nice work! Caught a couple of typos in the README though 😛

Thanks!


Kafka Connect for Aeron cluster

I am designing  Kafka Sink and Source connectors to stream messages in to and from Aeron cluster. Can Archiving Media Driver used a starting point for stream in message in to a Kafka topic from an Aeron clusters?

Using the Archive is a common option allowing you to rate adapt between the cluster and a downstream system. If you're interested in exploring options for your particular use case, please drop an email to<EMAIL_ADDRESS>


remove m -> rn mapping

Sorry to be second-guessing this AGAIN but the m -> rn thing has been bothering me; it seems like the most likely to interfere with legitimate use (although it's still extremely unlikely).
"Purely alphabetic lowercase ASCII identifiers do not collide with each other" seems right to me.

All good, we'll work it out as we go regardless, always appreciate the work my friend <3


Refactor project structure

Move style sheets to a isolate file on the respectives screens components

Solved with commit #ba90da4


All Derpibooru searches show no results after an API change.

Bug description
All Derpibooru searches show no results after Derpibooru switched from the old Booru-on-rails backend to their new Philomena backend.
Steps to reproduce

Make sure "derpibooru.org" is checked in the sources window. (I don't remember if this comes with Grabber by default, but if you don't have it in Sources, add it and set the booru type to Booru-on-rails.)
Search for a tag in the search bar that is known to have more than 0 results.
Below the search bar it will say "Derpibooru - No result"

Expected behavior
The results for that tag appearing below the search bar.
Context
main.log
Screenshots
Derpibooru box checked in the sources window:

The lack of search results:

System information

OS: Windows 7 SP1 (x64)
Grabber version: 7.1.1 (x64) and 7.2.1 (x64)

Additional context
I first noticed this bug today on 7.1.1 (x64), and updated to 7.2.1 (x64) today in an attempt to fix it, but it still wasn't fixed. https://derpibooru.org/pages/api was last updated 4 days ago, so I'm assuming that's when they dropped their compatability with the old API.
Calls to Derpibooru's search.json now return 400 Bad Request, but my main.log (from 7.2.1, attached above) shows it as "Bad Request (302)" for some reason.

Can confirm. They actually announced the old API was deprecated and would get dropped within a couple months after Philamena's public release to give people enough time for migration to new API.

For reference:

API documentation: https://derpibooru.org/pages/api
Post about migration: https://derpibooru.org/forums/meta/topics/philomena-open-beta-breaking-api-changes


test74 failing on master

When running basic_tests.py I get the following error. All other tests pass successfully:
test74 failed with should have gotten a ValueError, <class 'AssertionError'>

Process finished with exit code 0

I'll have a look.

Hmm...can't reproduce so far. What version of Python are you using?

Python 3.9 on Windows.
PS C:\Projects\hikaru\tests> C:\Python39\python.exe .\basic_tests.py
self=dos and other is overlord
self=ns1 and other is ns2
self=first and other is second
test74 failed with should have gotten a ValueError, <class 'AssertionError'>
self=hello-kiamol-3 and other is adsgad
PS C:\Projects\hikaru\tests>

Interesting...it appears that some other test before test74 generates side effects that change the internal catalogue of the Pod object being tested. If run under pytest, all tests pass, but in Python itself only that one fails. If you only run test74 in Python and no others then it also passes. If you make a new Pod object then test74 passes. It seems there's a bug in the catalogue re-population code; if you eliminate that test then test74 passes. Apparently pytest orders the tests differently hence this was never a problem. I'll see that it gets fixed with tests next release.


Max word length is 64 chars?

I think from the following warning message that your max word size is 64 characters.  Do you truncate to 64 characters if a word is longer than that?  Is this for all Text Analytics services or a subset of them (I saw it in the context of key phrase extraction)?  You might want to document all this on this "data limits" page.
"Warning: The document contains very long words (longer than 64 characters). These words may result in truncated and/or unreliable model predictions."

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

ID: 526ed5de-a06f-a7d3-228a-c8ef4cfd7f3e
Version Independent ID: 92870eb2-ded9-e2a5-9187-4dd5b71cd1a0
Content: Data limits for the Text Analytics API - Azure Cognitive Services
Content Source: articles/cognitive-services/text-analytics/concepts/data-limits.md
Service: cognitive-services
Sub-service: text-analytics
GitHub Login: @aahill
Microsoft Alias: aahi

@jennifermarsman
Thanks for reaching out to us. Since this repo is for Machine Learning Notebook only, please open a new forum thread in MSDN forum as it could be better place to get help on your scenario. These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

@aahill Hi, could you please take a look on this issue and update the document if necessary? Thank you very much.

Hi @jennifermarsman
I confirmed the following with @ashly1yeo :
Model versions of Key Phrase Extraction prior to 2020-07-01 have a 64 character limit. This limit is not present in later model versions or other endpoints.
I've added this info as a note on the language support article: https://docs.microsoft.com/azure/cognitive-services/text-analytics/language-support?tabs=key-phrase-extraction
Thanks!
#please-close

Also as a quick note, I think this note will be moving to the data limits article shortly.


upgrade base image to fix valnerability

Signed-off-by: Zhiwei Yin<EMAIL_ADDRESS>issue: https://github.com/stolostron/backlog/issues/21224

/lgtm


Get approvals refactor

Coverage increased (+0.04%) to 93.476% when pulling 6341caa3e7307431ed4f5720aae82bc4e8c14a68 on thongsav-usgs:getApprovals_refactor into 92184f139543e1f9bf22f6e6cee0a280f38284ee on USGS-R:master.

Coverage increased (+0.1%) to 93.534% when pulling 419f6c396d00b8bd4d06dd4f097d867d279f5d2b on thongsav-usgs:getApprovals_refactor into 92184f139543e1f9bf22f6e6cee0a280f38284ee on USGS-R:master.


JobFunnel 3.0 with localization, ABC and improved scraping

Description
This is version 3.0 of JobFunnel with numerous improvements including:

support for localization
abstract base class implementation of JobFunnel and Scrapers
significantly improved preemption of scraping and filtering of results
implementation of OO features such as Job JobField and JobFilter
signifiantly easier path towards making new scrapers via a get() and set() style of API with configurable priority and delay
addition of Remote and Wage scraping
implementation of Cerberus for Schema and validation of YAML configuration files

This will affect anyone currently developing off of the old branch, as the rebase will be un-tenable. I may need to squash this down a lot more.
If you are reading this, please give this branch a go, I find the easiest non-distruptive way is just to clone this repo as ABCJobFunnel and simply run
cd ABCJobFunnel
python3 -m jobfunnel

A good place to start is
Issues affected:

#85 this resolves our existing improper ABC usage
#83 added preemption for job get/set when an attribute fails a check.
#79 added warnings and handling for empty search results with TFIDF filter.
#60, #45, #37 - added support for localised scrapers with custom URLs and customisation of scraping without needing to copy/re-write lots of code (via inheritance)

Context of change

[x] Software (software that runs on the PC)
[x] Library (library that runs on the PC)
[x] Tool (tool that assists coding development)  -- added a call-graph GraphViz generation script.

Type of change
I have updated all documentation.
Existing master CSV files can be ported by adding missing columns, but it is recommended just to start fresh.
Existing cache files and block lists are not compatible, block lists could however be made compatible, this one might be worth pursuing.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
General monkey testing, but I need to up test coverage to be truly confident that the code quality is there. Would appreciate anyone reading this to just try running it and to try breaking it. Respond here with any bugs you find.
Checklist:

[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Additional TBD:

[ ] Up test coverage, unfortunately existing tests are not well-adapted to the new and improved codebase.
[ ] Add versioning to Cache
[ ] maybe fix glassdoor as a part of this PR as well (#87)
[ ] add missing Cerberus dependency to Travis PyEnv

FYI I've put this up before I've re-upped the coverage / fixed the pyenv to make it accessible. Fixing the coverage will take some time, but I don't anticipate making any further large changes to the structure of the codebase.

I reviewed as much as I could but ended up stopping part way as this update seems to break debugging using PyCharm. I was able to run JF normally, but debugging would cause it to stall out indefinitely. The issue seems to stem from the use of properties within this new version; more details about the issue can be found within this thread on Jetbrain's support forum.

Thanks for taking a look guys, I'll be fixing the CLI issues tomorrow, I might need to add some functional testing as well to make sure I've smoke tested this a bit better (in lieu of complete unit testing)

It would also seem that USA_ENGLISH locale is broken for the default settings.yaml, need to look into this.

Codecov Report

Merging #90 into master will decrease coverage by 24.33%.
The diff coverage is 34.11%.

@@             Coverage Diff             @@
##           master      #90       +/-   ##
===========================================
- Coverage   58.34%   34.01%   -24.34%
===========================================
  Files          13       25       +12
  Lines        1150     1373      +223
===========================================
- Hits          671      467      -204
- Misses        479      906      +427

Impacted Files
Coverage Δ

jobfunnel/__main__.py
0.00% <0.00%> (-35.90%)
:arrow_down:

jobfunnel/backend/jobfunnel.py
0.00% <0.00%> (ø)

jobfunnel/config/cli.py
11.36% <11.36%> (ø)

jobfunnel/backend/tools/filters.py
22.91% <22.91%> (ø)

jobfunnel/backend/tools/delay.py
24.07% <24.07%> (ø)

jobfunnel/backend/job.py
28.57% <28.57%> (ø)

jobfunnel/backend/scrapers/monster.py
30.21% <30.21%> (ø)

jobfunnel/backend/tools/tools.py
30.66% <30.66%> (ø)

jobfunnel/backend/scrapers/glassdoor.py
31.42% <31.42%> (ø)

jobfunnel/config/proxy.py
31.57% <31.57%> (ø)

... and 39 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 aac4008...9b95f26. Read the comment docs.

Verified that USA_ENGLISH functions for Indeed and Monster, added a locale scrape for USA_ENGLISH to round things out a bit more on Travis.

having a bit of a time with the CLI vs YAML vs defaults still, Made some progress just now just need to add the defaults injection.
I was having a hard time writing tests for the methods I made, so I've broken it down a bit further. Going to take another crack at it later.
I guess I can see now why most programs allow just YAML or just CLI

The fact that this is so hard to test indicates to me that perhaps we shouldnt let user mix YAML and cli arguments and default values. It gets bogged down in invalid cases and combinational cases...
Perhaps we can make the yaml and cli mutually exclusive?

OK, I'm just working on getting a few final things in, but seperating the CLI out made things alot easier. Finally moving past that mess and added some simple tests to verify It actually works.

OK, I've tested this enough for now.
Master is pretty broken compared to this so I'm going to merge and fix bugs as they come in from now on.
Still TODO:
[  ] Inter-scrape duplicates by TFIDF
[  ] GlassDoor scraper (webdriven)
[  ] more testing


Error - False port in use

Hi,
I'm trying to run this app in docker and get the port in use error with any port I use. Please help
Here is my Dockerfile:
FROM node:latest

ENV DEBIAN_FRONTEND noninteractive
ENV HUBOT_NAME myhubot
ENV HUBOT_OWNER none
ENV HUBOT_DESCRIPTION Hubot
ENV EXTERNAL_SCRIPTS "hubot-help,hubot-pugme"

RUN useradd hubot -m

RUN npm install -g hubot coffeescript yo generator-hubot && \
    npm install --save hubot-google-hangouts-chat

USER hubot

WORKDIR /home/hubot

RUN yo hubot --owner="${HUBOT_OWNER}" --name="${HUBOT_NAME}" --description="${HUBOT_DESCRIPTION}" --defaults && sed -i /heroku/d ./external-scripts.json && sed -i /redis-brain/d ./external-scripts.json && npm install hubot-scripts --save

EXPOSE 8080

VOLUME ["/home/hubot/scripts"]

CMD node -e "console.log(JSON.stringify('$EXTERNAL_SCRIPTS'.split(',')))" > external-scripts.json && \
    npm install $(node -e "console.log('$EXTERNAL_SCRIPTS'.split(',').join(' '))") && \
    bin/hubot -n $HUBOT_NAME --adapter google-hangouts-chat

And here is the error I get while trying to the run the image created from above:

<EMAIL_ADDRESS><EMAIL_ADDRESS>updated 2 packages and audited 178 packages in 1.283s
found 0 vulnerabilities

audited 178 packages in 1.277s
found 0 vulnerabilities
body-parser deprecated undefined extended: provide extended option node_modules/hubot/src/robot.js:445:21
[Mon Nov 19 2018 16:26:45 GMT+0000 (Coordinated Universal Time)] INFO Hangouts Chat adapter initialized successfully
[Mon Nov 19 2018 16:26:45 GMT+0000 (Coordinated Universal Time)] WARNING Loading scripts from hubot-scripts.json is deprecated and will be removed in 3.0 (https://github.com/github/hubot-scripts/issues/1113) in favor of packages for each script.
Your hubot-scripts.json is empty, so you just need to remove it.
[Mon Nov 19 2018 16:26:45 GMT+0000 (Coordinated Universal Time)] INFO Server is running in port - 8080
[Mon Nov 19 2018 16:26:45 GMT+0000 (Coordinated Universal Time)] ERROR Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:8080
at Server.setupListenHandle [as _listen2] (net.js:1290:14)
at listenInCluster (net.js:1338:12)
at doListen (net.js:1469:7)
at process.internalTickCallback (internal/process/next_tick.js:72:19)

I have quite the same issue... Following the instructions here, running bin/hubot -a google-hangouts-chat in my project I can then see in the logs:
[Thu Nov 29 2018 15:16:26 GMT-0500 (EST)] INFO Server is running in port - 8080
[Thu Nov 29 2018 15:16:26 GMT-0500 (EST)] ERROR Error: listen EADDRINUSE <IP_ADDRESS>:8080
  at Server.setupListenHandle [as _listen2] (net.js:1360:14)
  at listenInCluster (net.js:1401:12)
  at doListen (net.js:1510:7)
  at _combinedTickCallback (internal/process/next_tick.js:142:11)
  at process._tickCallback (internal/process/next_tick.js:181:9)

The only thing changed is the remove of the engines section in package.json because it was refering to something like node 0.10.
Seems that 2 processes are trying to start. Digging.

@jichu4n FYI

OK just to confirm following that:
mkdir myhubot
cd myhubot
yo hubot
# answer google-hangouts-chat for bot adapter
bin/hubot -a google-hangouts-chat

and my output:
[Thu Nov 29 2018 16:18:16 GMT-0500 (EST)] ERROR Error: listen EADDRINUSE <IP_ADDRESS>:8080
  at Server.setupListenHandle [as _listen2] (net.js:1360:14)
  at listenInCluster (net.js:1401:12)
  at doListen (net.js:1510:7)
  at _combinedTickCallback (internal/process/next_tick.js:142:11)
  at process._tickCallback (internal/process/next_tick.js:181:9)

My config
npm 6.4.1
node 8.12.0

the package.json:
{
  "name": "myhubot",
  "version": "0.0.0",
  "private": true,
  "author": "xxxxxxxxxxxxxxxxxxxxxxxx",
  "description": "A simple helpful robot for your Company",
  "dependencies": {
    "hubot": "^3.1.1",
    "hubot-diagnostics": "^1.0.0",
    "hubot-google-hangouts-chat": "^1.0.3",
    "hubot-google-images": "^0.2.7",
    "hubot-google-translate": "^0.2.1",
    "hubot-help": "^1.0.1",
    "hubot-heroku-keepalive": "^1.0.3",
    "hubot-maps": "0.0.3",
    "hubot-pugme": "^0.1.1",
    "hubot-redis-brain": "^1.0.0",
    "hubot-rules": "^1.0.0",
    "hubot-scripts": "^2.17.2",
    "hubot-shipit": "^0.2.1"
  },
  "engines": {
    "node": "0.10.x"
  }
}

Testing with the HTTP server start commented...
https://github.com/bobman38/hubot-google-hangouts-chat/commit/39cfb79f51f35137e7170367051dbcd17573332a
Why the adapater is starting a second HTTP server ?

@a4sh3u Can you try setting EXPRESS_PORT and PORT environment variables explicitly as shown in https://developers.google.com/hangouts/chat/how-tos/integrate-hubot#deploy_in_appengine
As @bobman38 mentioned, there are two express servers being initialized today - an express server that comes built-in with Hubot and one that is spinned up by this adapter. If the EXPRESS_PORT and PORT environment variables are not explicitly set, both try to listen to the same port 8080 and hence clash.
This is not ideal. We will look into fixing this.

@bobman38 Let me make sure using the hubot built-in express server is OK and get back to you on the PR.

@kumarse thanks. that works

To disable the default HTTP daemon, just pass -d to bin/hubot e.g.
bin/hubot -d -a google-hangouts-chat
The developer doc has been updated:
https://developers.google.com/hangouts/chat/how-tos/integrate-hubot


Versions of .NET supported?

Can you comment on what versions of .NET are required? Would it be supported on .NET Framework 2.0?

It supports .NET Framework 4.5 and up. And .NET Standard 1.3 and up.
It would be difficult to make it support anything below 4.0 as it is entirely based on the TPL (introduced in .NET 4.0) and task-based async and await (introduced in .NET 4.5 / C# 5.0)

Will close this for now.
I might be able to provide 4.0 support, but 2.0 would essentially make this a different library.


Update pulumi-terraform.

This brings in support for default config and property values provided
by environment variables.

Do we expect to have changes here to pick up default project, zone, etc. from environment variables?

Do we expect to have changes here to pick up default project, zone, etc. from environment variables?

Yes. I need to update resources.go as well.


[WIP] Make footer sticky

Deployed at http://biochemist-albert-44178.bitballoon.com/

[ ] Fix for blog layout

Tested across Browserstack:
IE8

IE9

IE10

IE11

Safari

Firefox

Chrome

iPad 4 (Safari 7)

Samsung Galaxy S6

Based on the amount of content each page now has, I don't think there is a need for this anymore.


[URP] Underwater Curtain/Meniscus broken in URP 7.4.1

Describe the bug
Underwater effects does not render. Both the curtain and meniscus.
Screenshots / video

Log
Nothing logged.
Versions
Crest 4.2
URP 7.4.1+
Unity 2019.4+
To Reproduce
Go underwater in main scene.
Platform

Editor
Mac

Haven't tested other platforms
Additional Notes
I checked the frame debugger, the curtain draw call is there but nothing renders. Also, 7.4.1 is not verified for 2019.4.

Is this maybe that issue where you need to toggle a feature on the underwater curtain material to get the shader to work?
(@EliGould actually i seem to remember you reported the underwater curtain not working in the past - we seem to get this sometimes and simply toggling an option on the material seems to wake it up :/, but i could not figure out why. mentioning you in case its helpful).

I gave it a try and it didn't fix it.

I looked into it further. The curtain material preview is black when normally it would be an ocean colour. Also, the meniscus doesn't work either. Looks like it is something else.

Daleeidd - thanks for your previous reply re my post in regard to #533.  I agree that the symptoms I am experiencing would appear to be this bug (I am also using the same product versions).

@Snigros No worries.
For anyone using 2019.4 and want this working, please consider using URP 7.3 (verified for 2019.4). I couldn't see anything in the change log that would be the cause. I will most likely submit a bug to Unity tomorrow.

Daleeidd thanks for your reply.  I had swapped to URP 7.4.1 to see if it fixed a minor terrain billboard rendering issue I was experiencing (and I hadn't noticed that URP 7.4.1 wasn't verified for 2019.4).
I can confirm that swapping back to URP 7.3.1 has fixed this issue (for me at least!).   Thanks again for your assistance.  Cheers, S

Might this be the problem?

And
NullReferenceException: Object reference not set to an instance of an object
Crest.UnderwaterEffect.Validate (Crest.OceanRenderer ocean, Crest.ValidatedHelper+ShowMessage showMessage) (at Assets/Plugins/Crest/Crest/Scripts/Helpers/UnderwaterEffect.cs:293)
Crest.UnderwaterEffect.Start () (at Assets/Plugins/Crest/Crest/Scripts/Helpers/UnderwaterEffect.cs:66)

@NiekoEugenijus Thanks. I have already handled that exception which will be fixed for 4.4. That cbuffer notice won't be the issue either.
I'll check again tomorrow. I think it might have something to do with the vertex part of the shader.

I couldn't work this out so I submitted a bug to Unity today. I will post here with updates.

I had another look and it looks like the unity_CameraToWorld._13_23_33 is inverted. If I invert it, it works correctly. I am under the assumption that this matrix is safe to use without being concerned about platform specific issues like z-flipping. So I am hesitant to fix it as it could be working fine for most people despite the number of reports we get. I have updated the case with Unity.
We could pass the camera forward in ourselves with scripting. I might resort to that if I hear nothing from Unity before the next release.

That's pretty bizarre, have no idea why that would or could be flipped for some people. It almost seems like the value is not being set properly in the pass setup then (maybe it's uninitialised and taking whatever value is there from previous draw calls, but I doubt that is the case).
We could pass it in yes.

Will be fixed in 4.5

FYI...I had been sticking with 2019.4.7f,  URP 7.3.1 and Crest 4.4 as a workaround to this issue.  Migrating to 2020.1.1f1,  URP 8.2.0 and Crest 4.4 provided a noticeable (overall) performance boost - so I would like to proceed with this stream going forward.   However the Underwater Curtain/Meniscus is also broken when using this combination....

Fix will still be in for 4.5 and beyond. But here is the Unity issue for this bug:
https://issuetracker.unity3d.com/product/unity/issues/guid/1257518

4.5 is released which solves this issue. Eventually we can revert this fix once Unity has fixed it on their end.

Thanks for the 4.5 update.  I just tried using it with Unity 2020.1.2f1 and URP 8.2.0 - but underwater issue still there in my project unfortunately.  Probably has no bearing but I am also using Cinemachine cameras (2.6.0 and just tried 2.6.1) and Direct3D12.  To do this test I:
(a) made a new copy of my 2019.4.8f1 project (which works properly with URP 7.3.1 and Crest URP 4.4); (b) deleted the Crest files outside of Unity (c) upgraded the whole project to 2020.1.2f1, and then, (d) imported the new Crest 4.5 package e.g. the usual sort of update process.  Everything worked as expected - except the underwater effects.
FYI...
Cheers, S

@Snigros does re-import the underwater curtain material before entering play mode fix it?

Yes it does!
Thanks so much for pointing this out - and for the regular updates - much appreciated.  Cheers, S


Add TorchArrow Nightly CI Test

🚀 The feature
TorchArrow nightly build is now available for Linux (other versions will be next).
We should add TorchArrow nightly CI tests for these TorchArrow dataframe related unit tests.
Motivation, pitch
This will ensure that our usages remain compatible with TA's APIs.
Additional context
This is a good first issue for people who want to understand how our CI works. Other domain CI tests (for Vision, Text) can serve as examples on how to set this up.

@NivekT I'd like to take up this issue


added yarn stylelint and removed stylelint related errors

Signed-off-by: shoeb<EMAIL_ADDRESS>Resolves #186
I have added yarn packages stylelint and stylelint-config-standard. I have also added a hidden .stylelintrc.json file to configure some setup to avoid usage of !important
How to test
run the following command to test the changes
yarn stylelint "/*.{scss,vue}"**
Screenshots
Screenshots of the changes working

Please ignore my previous pull request. I had some issues with merging so I created a fresh new fork and updated the repository

Thanks for making these changes. At a quick glance, they look good, but I'll let @ermish take a look.
It might be useful to also call the stylelint command inside the yarn lint command, would you be able to add that?

LGTM.
I added a comment, but we fix in a separate PR.
Thanks for wrapping this up, @shoebsd31!!! 👾 💯

Thanks @ermish

side note, cool portfolio @shoebsd31 😎

checked preview, site looks consistent 👍 merging


Developing #153 and Close #218 - support JSON schema

Succeded to implement dynamic properties' archiving in the MetadataObject.key.
Also, succeeded to support those dynamic properties at JSON schema first.
Therefore, validators and stringifier would support them, soon.

Not Close #217, but Close #218


key "url" not found in dictionary

Repro:
Use this dependency file:
dependencies:
  com.google.guava:
    guava:
      version: 26.0-jre
      lang: java

bazel run  //:parse -- generate -r /home/me/Projects/foo -s 3rdparty/workspace.bzl -d dependencies.yml

Expected:
Correct dependency for guava in 3rdparty/jbm/com/google/guava
Actual:

INFO: Build completed successfully, 1 total action
wrote 6 targets in 6 BUILD files

workspace.bzl has these dependencies:
def list_dependencies():
    return [
    {"artifact": "com.google.code.findbugs:jsr305:3.0.2", "lang": "java", "sha1": "25ea2e8b0c338a877313bd4672d3fe056ea78f0d", "repository": "http://central.maven.org/maven2/", "name": "com_google_code_findbugs_jsr305", "actual": "@com_google_code_findbugs_jsr305//jar", "bind": "jar/com/google/code/findbugs/jsr305"},
    {"artifact": "com.google.errorprone:error_prone_annotations:2.1.3", "lang": "java", "sha1": "39b109f2cd352b2d71b52a3b5a1a9850e1dc304b", "repository": "http://central.maven.org/maven2/", "name": "com_google_errorprone_error_prone_annotations", "actual": "@com_google_errorprone_error_prone_annotations//jar", "bind": "jar/com/google/errorprone/error_prone_annotations"},
    {"artifact": "com.google.guava:guava:26.0-jre", "lang": "java", "sha1": "6a806eff209f36f635f943e16d97491f00f6bfab", "repository": "http://central.maven.org/maven2/", "name": "com_google_guava_guava", "actual": "@com_google_guava_guava//jar", "bind": "jar/com/google/guava/guava"},
    {"artifact": "com.google.j2objc:j2objc-annotations:1.1", "lang": "java", "sha1": "ed28ded51a8b1c6b112568def5f4b455e6809019", "repository": "http://central.maven.org/maven2/", "name": "com_google_j2objc_j2objc_annotations", "actual": "@com_google_j2objc_j2objc_annotations//jar", "bind": "jar/com/google/j2objc/j2objc_annotations"},
    {"artifact": "org.checkerframework:checker-qual:2.5.2", "lang": "java", "sha1": "cea74543d5904a30861a61b4643a5f2bb372efc4", "repository": "http://central.maven.org/maven2/", "name": "org_checkerframework_checker_qual", "actual": "@org_checkerframework_checker_qual//jar", "bind": "jar/org/checkerframework/checker_qual"},
    {"artifact": "org.codehaus.mojo:animal-sniffer-annotations:1.14", "lang": "java", "sha1": "775b7e22fb10026eed3f86e8dc556dfafe35f2d5", "repository": "http://central.maven.org/maven2/", "name": "org_codehaus_mojo_animal_sniffer_annotations", "actual": "@org_codehaus_mojo_animal_sniffer_annotations//jar", "bind": "jar/org/codehaus/mojo/animal_sniffer_annotations"},
    ]

And the workspace basically fails to load
bazel build 3rdparty/jvm/com/google/guava
ERROR: "[REDACTED Project]/WORKSPACE:55:1: Traceback (most recent call last):
	File "[REDACTED Project]/WORKSPACE", line 55
		maven_dependencies()
	File "[REDACTED Project]/3rdparty/workspace.bzl", line 81, in maven_dependencies
		callback(hash)
	File "[REDACTED Project]/3rdparty/workspace.bzl", line 58, in callback
		jar_artifact(artifact = hash["artifact"], name ..."], <4 more arguments>)
	File "[REDACTED Project]/3rdparty/workspace.bzl", line 61, in jar_artifact
		hash["url"]
key "url" not found in dictionary
ERROR: Error evaluating WORKSPACE file
ERROR: Skipping '3rdparty/jvm/com/google/guava': error loading package 'external': Package 'external' contains errors

% bazel info release
release 0.18.0

Digging into existing bugs, it could be a duplicate of #184

Yes, workaround is to add these options in dependencies.yml:
options:
  languages: ["java"]
  resolverType: "coursier"

Yeah it seems aether is currently broken.
I’m very inclined to remove aether because it has other bugs we’ve found as well and requires new features to be written twice.


Parameter in Example 1 incorrect?

I believe Example 1 on this support page is using an incorrect parameter.
Example 1:  Get-MessageTrace -MessageTraceId 2bbad36aa4674c7ba82f4b307fff549f -SenderAddress<EMAIL_ADDRESS>-StartDate 06/13/2018 -EndDate 06/15/2018 | Get-MessageTraceDetail
I believe it should be:  Get-MessageTrace -MessageId 2bbad36aa4674c7ba82f4b307fff549f -SenderAddress<EMAIL_ADDRESS>-StartDate 06/13/2018 -EndDate 06/15/2018 | Get-MessageTraceDetail

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

ID: 6f784c39-5041-578f-7d9f-5b38310cc89c
Version Independent ID: 4ab7a6db-db36-c736-6ce5-2f0e74b0cf6a
Content: Get-MessageTraceDetail (exchange)
Content Source: exchange/exchange-ps/exchange/Get-MessageTraceDetail.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

Hi @hrn111, thank you for your feedback and help us to improve docs.microsoft.com.
If you are using also -SenderAddress I think the -MessageTraceId parameter is fine. Parameter description:
"The MessageTraceId parameter can be used with the recipient address to uniquely identify a message trace and obtain more details. A message trace ID is generated for every message that's processed by the system."

@dariomws  Thank you very much for the contribution and sharing this explanation. @hrn111 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Handle Omitted Queues

In the current state of work #36 we ran into Browser Specific Issues handling the Feedback Queue. After several different attempts to solve this issue, it appears to be the easier solution, if we only consider the 1-client-case for now, and only delete Query-ID's from the Feedback Queue, after receiving the corresponding evaluation from the client. This would allows us to not rely on receiving empty feedback booleans from the client, which does not (yet) work for all browsers, while still enabeling us, to re-render unevaluated Queries.

Resolved in cff1c4049343f1fbe66f3d4e49f2637bed59069f

Resolved in cff1c4049343f1fbe66f3d4e49f2637bed59069f.


[patch] Add adapter info to Issues template

What do you think about asking for adapter info and version?
Several times I see that being asked on issues, or that confusion happens because of things working for one adapter but not in an other one.
As always, my 2 cents.

@luislobo I like this, but it requires more than just a change to the template, since we'd want this info to be optional.  Right now @sailsbot doesn't do "optional"...

:+1: alright!

@luislobo I like this -- let's modify slightly to:
**DB adapter name**: N/A
**DB adapter version**: N/A

@sailsbot doesn't do "optional", so we need the "N/A" placeholder for issues where the adapter is not important.

@sgress454 done!

@sgress454 Just a reminder so that you don't forget about this one


Pin karma, karma-mocha dev dependencies

Codecov Report

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

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


Revoker improvements. Stop removing all permissions in one time

every 24 hours or by schedule provided by the user revoke all user-level assignments that are not known and not tracked by SSO elevator
keep group-level assignments
use dynamo or event bridge as a state
alert on inconsistencies - post to slack

Commit: 542b85e


Fix deps for *.deb on Ubuntu 19.10

lsb_release -a:
Distributor ID:   Ubuntu
Description:      Ubuntu 19.10
Release:          19.10
Codename:         eoan

Python3 Version: 3.7.5

When installing Dragonfire on Ubuntu 19.10, installation fails due to dependency issues. The following error is reported:
The following packages have unmet dependencies:
 dragonfire : Depends: python3 (< 3.7) but 3.7.5-1 is to be installed
              Depends: portaudio19-dev but it is not going to be installed

portaudio19-dev is in the repos, so why it says it is not going to be installed, I don't know:
$ apt search portaudio19-dev
Sorting... Done
Full Text Search... Done
portaudio19-dev/eoan 19.6.0-1 amd64
  Portable audio I/O - development files

MANUALLY installing portaudio19-dev does resolve the problem with that package, but the python problem persists.
The command I used for installation was:
sudo apt install ./dragonfire_1.0.4_amd64.deb

This works the same as the dpkg -i command listed on the README.md, but also handles dependencies, just for disclosure.

@Batcastle we only support the latest Ubuntu LTS which is 18.04 with the .deb package. The dragonfire_1.0.4_amd64.deb package requires Python version to be less than 3.7. Because of that, it fails.
The message says Depends: portaudio19-dev but it is not going to be installed. That means you have the portaudio19-dev package on your system and it will skip the installation of this package.
Please try to clone and install Dragonfire with sudo make install as it's stated in README.md.
I remember that installing a Debian package with apt install might cause issues in some edge cases so we chose to suggest dpkg -i instead of apt install. Though I don't remember the exact reason...
Last but not least, feel free to send pull requests :blush:

Thank you for the clarification. Maybe put that you only support the latest Ubuntu LTS in the README? I checked just now and did not see it.
On a side note,

The message says Depends: portaudio19-dev but it is not going to be installed. That means you have the portaudio19-dev package on your system and it will skip the installation of this package.

I don't have portaudio19-dev installed on my system. If I did, it wouldn't have said anything. That's why I said manually installing it fixes the issue. Why it's doing this eludes me as I checked the control file and it was in there correctly. But, if it's working correctly on Ubuntu 18.04 then it's probably not a big deal for right now since you guys only support the latest LTS, as you said.
But, as I said, just a side note, so not a big deal. ¯_(ツ)_/¯

@Batcastle would you like to add that Ubuntu LTS notice to README.md by sending a PR?
I have misunderstood your statement about portaudio19-dev package sorry. I suspect that since we always use a system with Python 3.6 to build the Debian package, it somehow automatically creates python3 (< 3.7) dependency. Could you please;

follow Build the Debian package guide and generate a new .deb package on your machine
try to install that .deb package with either apt install or dpkg -i
then share the results under this discussion

You might need to install the development dependencies with sudo make dev-install

I downloaded the latest dragonfire_1.1.1_amd64.deb and tried installing via dpkg -i and encountered the above error. Installing via make file also shows the same error. I am currently using Ubuntu 19.10.


@LoadBalanced RestClient

Fixes gh-1293.

Hi @Buzzardo, could you please review documentation changes?

Thanks, @Buzzardo 🙂


RFC: Move from relational to document store database?

Discussion
Reasoning

In discussing a way to allow for community-driven dataset updates, I had a thought to create another repository with a collection of JSON files (probably organized by country then state/providence) which could then be compiled and update a DB via Github Actions or another CI/CD process. With this process, it would likely be easier to update a DB like Mongo or DynamoDB.

To Research 📚

[x] Is ~key-value~ Document Store / NoSQL database able to do location-based queries like PostGIS?

https://github.com/chrisjm/openbrewerydb-rails-api/issues/34#issuecomment-575385527
MongoDB Geospatial Queries

Pros and Cons
Relational DB (PostgreSQL, MySQL, MariaDB, etc)
Pros
👍  Able to perform more complicated queries such as the full-text search and PostGIS / geospatial searching (not fully researched; key-value might be able to do this as well)
👍  Relational databases are more robust for data management
Cons
👎  Somewhat complicated to set up
👎  Expensive (~$180/year with AWS RDS reserved)
Document Store DB (MongoDB, DynamoDB, etc)
Pros
👍  Easier to set up from a development standpoint
👍  Cheaper (<$12/year, with current stats; maybe even free)
👍  Can be very performant if data is organized properly
Cons

@chrisjm I'm going to work on protyping this out with mongodb.  I'll keep you updated with progress.

Awesome @jpthompson212! Let me know if there's anything I can do to help!

We've decided to stick with a relational database for this particular incarnation of the API. v2 may allow for different data stores.


Detaching secondary screen closes Workspacer

Setup
HW
Lenovo T14 + Dell  U2719D via HDMI attached to Lenovo Thinkpad USB-C Dock Gen2
SW
Windows 10.0.19043.1645 + Workspacer 0.9.10 with default config
Steps to reproduce

Attach secondary screen, e.g. external monitor through a docking station
Start Workspacer with default config
Detach secondary screen

Expected Result
Workspacer should handle the detachment of the secondary screen gracefully. Opened workspaces should remain accessible through the primary screen.
Actual Result
Workspacer immediatly dies in result of the secondary screen being detached.

Any workarounds for this? Using routes restart happens nearly seamless, but still not as good as graceful detach/attach.

Someone will have to debug the issue, or possibly install a post-mortem debugger and get the stack-traces from that.
Then someone comfy working with the might be able to add code to handle things more gracefully.

Windbg and cdb make Windows unresponsive after attaching to workspacer pid, probably due to mouse events.
Actually unstable build is not crashing, due to try/catch block in Bar. Window movement even works, however, SystemTray menu, Bar, Keybinds and I assume rest modules gone as well as working area remains the same as it was.
Actually crashes wouldn't be such an issue if icons weren't disappearing from taskbar (on crash and sometimes even without). Explorer restart, alttab, taskview won't help. I tried to swap SW_HIDE for SW_MINIMIZED, but sadly wasn't a simple fix. I also tried powershell scripts, but SW_SHOW requires cached .MainWindowHandle, so, unless it's will be saved to file on workspacer side, impossible.


commit spanproto

commit SpanProto algorithm.
paper: SpanProto: A Two-stage Span-based Prototypical Network for Few-shot Named Entity Recognition (EMNLP2022)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.王嘉宁 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Error building evdi on Fedora 39 / Linux 6.7.9

Are you using the latest driver?

Installed Packages
Name         : displaylink
Version      : 1.14.1
Release      : 2

Are you using the latest EVDI version?

johan@fedora ~/evdi (main)> git remote -v
origin  https://github.com/DisplayLink/evdi (fetch)
origin  https://github.com/DisplayLink/evdi (push)
johan@fedora ~/evdi (main)> git status
On branch main
Your branch is up to date with 'origin/main'.

nothing to commit, working tree clean
johan@fedora ~/evdi (main)> git log -1
commit 9905b9bfed8b7e49befa0530bfb6e464380d938f (HEAD -> main, tag: v1.14.2, origin/main, origin/HEAD)
Author: Łukasz Spintzyk<EMAIL_ADDRESS>Date:   Wed Jan 31 11:30:22 2024 +0100

    Set version to 1.14.2
johan@fedora ~/evdi (main)>

If you are using a DisplayLink device, have you checked 'troubleshooting'
on DisplayLink's website?

The same device works fine with either Windows and with KDE-Neon using the Ubuntu packages from the Synaptic web site.

Is this issue related to evdi/kernel?

Building evdi

Linux distribution and its version

johan@fedora ~/evdi (main)> uname -a
Linux fedora 6.7.9-200.fc39.x86_64 #1 SMP PREEMPT_DYNAMIC Wed Mar  6 19:35:04 UTC 2024 x86_64 GNU/Linux

Linux kernel version
6.7.9-200.fc39.x86_64

Xorg version (if used)
Plasma 5.27.11, kwin, Not really relevant.

Desktop environment in use
KDE/Plasma, not really relevant.

I am hoping that building a newer version of evdi will solve the issue, as I dont know what else to look at.  (The problem I am trying to solve is best described by watching this video: https://youtu.be/uZeN9UesBUY)
The error below occurs when trying to build.
johan@fedora ~/evdi (main)> make
CFLAGS="-isystem./include -isystem./include/uapi -Werror -Wextra -Wall -Wno-error=missing-field-initializers -Werror=sign-compare -Wmissing-prototypes -Wstrict-prototypes -Werror=discarded-qualifiers " make -C module
make[1]: Entering directory '/home/johan/evdi/module'
make -C /lib/modules/6.7.9-200.fc39.x86_64/build M=$PWD
make[2]: Entering directory '/usr/src/kernels/6.7.9-200.fc39.x86_64'
warning: the compiler differs from the one used to build the kernel
  The kernel was built by: gcc (GCC) 13.2.1 20231205 (Red Hat 13.2.1-6)
  You are using:           gcc (GCC) 13.2.1 20240316 (Red Hat 13.2.1-7)
  CC [M]  /home/johan/evdi/module/evdi_platform_drv.o
  CC [M]  /home/johan/evdi/module/evdi_platform_dev.o
  CC [M]  /home/johan/evdi/module/evdi_sysfs.o
  CC [M]  /home/johan/evdi/module/evdi_modeset.o
  CC [M]  /home/johan/evdi/module/evdi_connector.o
  CC [M]  /home/johan/evdi/module/evdi_encoder.o
  CC [M]  /home/johan/evdi/module/evdi_drm_drv.o
  CC [M]  /home/johan/evdi/module/evdi_fb.o
  CC [M]  /home/johan/evdi/module/evdi_gem.o
/home/johan/evdi/module/evdi_gem.c: In function ‘evdi_gem_fault’:
/home/johan/evdi/module/evdi_gem.c:214:40: error: comparison of integer expressions of different signedness: ‘long unsigned int’ and ‘loff_t’ {aka ‘long long int’} [-Werror=sign-compare]
  214 |         if (!obj->pages || page_offset >= num_pages)
      |                                        ^~
cc1: all warnings being treated as errors
make[4]: *** [scripts/Makefile.build:243: /home/johan/evdi/module/evdi_gem.o] Error 1
make[3]: *** [/usr/src/kernels/6.7.9-200.fc39.x86_64/Makefile:1929: /home/johan/evdi/module] Error 2
make[2]: *** [Makefile:246: __sub-make] Error 2
make[2]: Leaving directory '/usr/src/kernels/6.7.9-200.fc39.x86_64'
make[1]: *** [Makefile:86: module] Error 2
make[1]: Leaving directory '/home/johan/evdi/module'
make: *** [Makefile:13: module] Error 2

The below commit seem to be to blame
johan@fedora ~/e/module (main) [128]> git log -p evdi_gem.c
commit 294af3d4ec5b22012174a3b26b773f939dad7dbd
Author: Łukasz Spintzyk<EMAIL_ADDRESS>Date:   Wed Sep 6 07:49:36 2023 +0200

    Fix possible OOB in evdi_gem_fault

diff --git a/module/evdi_gem.c b/module/evdi_gem.c
index 499d065..efae6e1 100644
--- a/module/evdi_gem.c
+++ b/module/evdi_gem.c
@@ -205,12 +205,13 @@ int evdi_gem_fault(struct vm_fault *vmf)
 #endif
        struct evdi_gem_object *obj = to_evdi_bo(vma->vm_private_data);
        struct page *page;
-       unsigned int page_offset;
+       pgoff_t page_offset;
+       loff_t num_pages = obj->base.size >> PAGE_SHIFT;
        int ret = 0;

        page_offset = (vmf->address - vma->vm_start) >> PAGE_SHIFT;

-       if (!obj->pages)
+       if (!obj->pages || page_offset >= num_pages)
                return VM_FAULT_SIGBUS;

        page = obj->pages[page_offset];

I also ran into this. I'm not sure if this is the right way to handle this, but I worked around it with the following. I cast, what I think is the smaller type of pgoff_t (long unsigned int)  to the larger signed loff_t (long long int)
diff --git a/module/evdi_gem.c b/module/evdi_gem.c
index efae6e1..46dc5fa 100644
--- a/module/evdi_gem.c
+++ b/module/evdi_gem.c
@@ -211,7 +211,7 @@ int evdi_gem_fault(struct vm_fault *vmf)

        page_offset = (vmf->address - vma->vm_start) >> PAGE_SHIFT;

-       if (!obj->pages || page_offset >= num_pages)
+       if (!obj->pages || (loff_t)page_offset >= num_pages)
                return VM_FAULT_SIGBUS;

        page = obj->pages[page_offset];

@tanj  the same Issue still occurs on Pop_OS! (kernel 6.8.0)
/my/directory/path/evdi/module/evdi_gem.c: In function ‘evdi_gem_fault’:
/my/directory/path/evdi/module/evdi_gem.c:214:40: error: comparison of integer expressions of different signedness: ‘long unsigned int’ and ‘loff_t’ {aka ‘long long int’} [-Werror=sign-compare]
214 |         if (!obj->pages || page_offset >= num_pages)
|                                        ^~
cc1: all warnings being treated as errors
make[4]: *** [scripts/Makefile.build:243: /my/directory/path/evdi/module/evdi_gem.o] Error 1
make[3]: *** [/usr/src/linux-headers-6.8.0-76060800daily20240311-generic/Makefile:1927: /my/directory/path/evdi/module] Error 2
make[2]: *** [Makefile:240: __sub-make] Error 2
make[2]: Leaving directory '/usr/src/linux-headers-6.8.0-76060800daily20240311-generic'
make[1]: *** [Makefile:86: module] Error 2
make[1]: Leaving directory '/my/directory/path/evdi/module'
make: *** [Makefile:13: module] Error 2

Fixed in 1.14.3


fix: limits key prevent using symfony standard rate limiter

1. Why is this change necessary?
To use a standard symfony rate limiter without errors on a standard rate limiter definition by shopware.
2. What does this change do, exactly?
It adds the limits key to the array_filter list to remove this configuration key, if a standard symfony rate limiter should be used.
3. Describe each step to reproduce the issue or behaviour.
Add the following to the config/packages/shopware.yaml file in your project:
shopware
    api:
        rate_limiter:
            oauth:
                enabled: true
                policy: 'sliding_window'
                limit: 2
                interval: '60 minutes'

This should activate the sliding_window rate limiter by symfony for a oauth route. If you want to login in administration you will get an error on the oauth/token route.
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-22045
5. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Stephan Niewerth seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-22055
Please use this issue to track the state of your pull request.

In my opinion there is a second fault to let run the default oauth/token route with a symfony standard rate limiter. This route (used by administration) call the create method for the RateLimiterFactory two times. At the second time the enabled key is missing. You can't login on a dev environment. I think in production the rate limiter is not working (NoLimiter).
Should I add a fix to this here or do you prefere a second PR?


Dependencies management

Hi,
I think that it's a bad idea to put your dependencies like that in the source code under "plugins". jQuery and Bootstrap are not plugins of AdminLTE, they are dependencies. You should install your dependencies with bower/npm:
bower install --save bootstrap jquery
or
npm install --save bootstrap jquery
and use a specific version (if you really want a specific version) by saying it on your bower.json/package.json file.
Once installed, those dependencies are under bower_components/node_modules directory and this directory should NOT be part of your git repository.
This is a bad practice to do what you're doing right now because you possibly have a buggy version of jQuery until you update your version of jQuery on your repository (which is normally automatic when installing via npm, it install the last fixed version).
Also, you have specific version of jQueryUI right now, and it's unclear which version it is, if it was installed via npm/bower, the version would be specified in the package.json/bower.json.
An other problem (mentioned here #1430) is that if someone install admin-lte trough npm or bower, he could have redundant repositories in node_modules and the amount of files would increase drastically for big projects. Nobody wants to play with three different jQuery versions at the same time...
Hope this can be fixed :)

Hello @martinRenou,
We are implementing support for Bower and NPM. The feature will be available in v2.4.
Thanks for the detailed report! We will keep this open until the dependencies manager is implemented.

Some plugins don't provide a fully compiled version when downloaded via a package manager but we managed to use NPM and Bower for the majority of packages.


Enable MP1 and MP2 support in minimp3 and editor

This adds only 3.5k to the template size (Win/64bits).
fixes #72688

I don't know if it makes sense to add even 3.5k to the export template to support one game. Couldn't you transcode these audio files or maintain a separate export template? 3.5k doesn't seem like much but for web games where it's common to strip down the export template via a custom build, every kb counts.

I don't know if it makes sense to add even 3.5k to the export template to support one game. Couldn't you transcode these audio files or maintain a separate export template? 3.5k doesn't seem like much but for web games where it's common to strip down the export template via a custom build, every kb counts.

The difference in HTML5 export templates is likely smaller (since these use optimize=size). It may be worth creating release HTML5 builds with and without this change to check if the difference is significant.

We're talking about a feature that likely only one person will ever use, though. And it seems like @Ithamar is more than capable of making their own export templates. I know 3.5k is practically nothing, but I still don't think this is a valuable feature for the mainline engine to have, and I think guidance is still to refuse-by-default to keep the engine small, right? https://godotengine.org/article/will-your-contribution-be-merged-heres-how-tell/

@ellenhp would a build-time option be acceptable? Or are you suggesting I maintain a fork?

A build time option would be fine by me! I'm not opposed to this feature existing especially considering how little code maintenance burden it is for us (it's all in minimp3) but I just think that when someone goes to create a stripped-down web export for their game by removing every node that they don't use, that web export shouldn't have to know how to decode MP2. :)

Sorry for the "redo everything" review, at least I suggested a lot of changes, so you should be able to just apply them if you agree.

No problem, its not like it is a lot of code, so even if I do it manually it is not a lot of work ;)

BTW Don't forget to squash everything once you're done!

Ah, Github does not do that automagically? I'm more used to Gitlab which has an PR option to do this at merge time.

I'm more used to Gitlab which has an PR option to do this at merge time.

I doubt that most of us use this platform out of favour :P

Actually, turns out Github has the option too, it might be an idea to simply enforce this for PRs if it is the expected behaviour anyway......

We don't use this option on the main repo, because it doesn't create merge commits.

@ellenhp thinking about it, are there some benchmarks about template size in relation to available features? I think that there might be lots of similar cases of "mostly unused thing that could be hidden behind a switch" that we could get rid of.

I made https://github.com/Calinou/godot-size-benchmarks a few years ago, but I also had another script that invidiaully compiled Godot with each module disabled and found that optimize=size had a greater impact on binary size compared to disabling all modules.

So I updated this PR to current master again, is there anything else I can do to help get this merged?

This looks fine overall as an option, but there's a problem with the implementation, as it's not self-contained. Modules are meant to be self-contained so the module-specific config can't modify SConstruct.
Thankfully there's a way to register new options from the module itself, I just tested it locally so I'll push an update to your PR directly with this change.

This looks fine overall as an option, but there's a problem with the implementation, as it's not self-contained. Modules are meant to be self-contained so the module-specific config can't modify SConstruct.

Ah, I was not aware of that.

Thankfully there's a way to register new options from the module itself, I just tested it locally so I'll push an update to your PR directly with this change.

Awesome, I'll take a look at your changes to see how it is done in case I need something similar in the future.
Thanks very much!

Seems like I don't have permission to push changes to the PR branch, so I pushed changes in my fork: https://github.com/akien-mga/godot/commit/e75da97e3b70ec83aebd0ccaaa6e363700901635
This is making use of get_opts in config.py to add a module-specific option.
If you can push this commit to your fork's branch to update this PR, it should be good to go.

Thanks! And congrats for your first merged Godot contribution :tada:


Restoring the deleted original english file.

This is due to the fact that the contributor edited directly the file instead of making a new one.

Because too many contributors were corrupting the history of important files (#319), I reverted the master branch to Feb 7. So this revert isn't necessary.


AccountNavigation.spec.ts breaks randomly on cy.click()

Description
Test case "should follow links from account with few tokens"
in AccountNavigation.spec.ts test breaks randomly on the same test case with the same message:
CypressError: Timed out retrying after 8050ms: cy.click() failed because this element is detached from the DOM.
This breakage occurs when test is run on GitHub.
Full message:
...
Cypress requires elements be attached in the DOM to interact with them.
The previous command that ran was:
> cy.eq()
This DOM element likely became detached somewhere between the previous and current command.
Common situations why this happens:
  - Your JS framework re-rendered asynchronously
  - Your app code reacted to an event firing and removed the element
You typically need to re-query for the element or add 'guards' which delay Cypress from running new commands.
https://on.cypress.io/element-has-detached-from-dom
      at $Cy.ensureAttached (http://localhost:38920/__cypress/runner/cypress_runner.js:167985:76)
      at runAllChecks (http://localhost:38920/__cypress/runner/cypress_runner.js:154413:12)
      at retryActionability (http://localhost:38920/__cypress/runner/cypress_runner.js:154493:16)
     at tryCatcher (http://localhost:38920/__cypress/runner/cypress_runner.js:13022:23)
      at Promise.attempt.Promise.try (http://localhost:38920/__cypress/runner/cypress_runner.js:10296:29)
      at whenStable (http://localhost:38920/__cypress/runner/cypress_runner.js:173538:65)
      at http://localhost:38920/__cypress/runner/cypress_runner.js:172993:14
      at tryCatcher (http://localhost:38920/__cypress/runner/cypress_runner.js:13022:23)
      at Promise._settlePromiseFromHandler (http://localhost:38920/__cypress/runner/cypress_runner.js:10957:31)
      at Promise._settlePromise (http://localhost:38920/__cypress/runner/cypress_runner.js:11014:18)
      at Promise._settlePromise0 (http://localhost:38920/__cypress/runner/cypress_runner.js:11059:10)
      at Promise._settlePromises (http://localhost:38920/__cypress/runner/cypress_runner.js:11139:18)
      at Promise._fulfill (http://localhost:38920/__cypress/runner/cypress_runner.js:11083:18)
      at http://localhost:38920/__cypress/runner/cypress_runner.js:12697:46
  From Your Spec Code:
      at Context.eval (http://localhost:38920/__cypress/tests?p=tests/e2e/specs/AccountNavigation.spec.ts:735:14)
Steps to reproduce
Runs Hedera Mirror Node Explorer End-to-end Tests action from GitHub env.
Additional context
No response
Hedera network
other
Version
Built Mon, 29 Aug 2022 16:06:12 GMT
Operating system
No response

This is fixed by https://github.com/hashgraph/hedera-mirror-node-explorer/pull/504


Prepare project to be closed and released to maven central repo

No one objected; therefore, I am assuming everybody is ok with the changes that are introduced here.


Advancement Criteria Questions - Location & Inventory Change

https://github.com/skylinerw/guides/blob/master/java/advancements/triggers.md#location
For the Location criteria, it doesn't seem to allow negative coordinates - is there a way around this?
https://github.com/skylinerw/guides/blob/master/java/advancements/triggers.md#inventory_changed
Could you please do a guide on the nbt condition?

Sorry for the late reply.
Negative values work for me in the latest snapshot. Make sure you're setting min to be the smaller of the two, otherwise you can provide the advancement here and I can check for any issues.
NBT fields are just the same as you'd use in commands. For example:
"nbt": "{Invulnerable:1b}"

However, keep in mind that nested quotes will need to be escaped with a backslash, otherwise those un-escaped quotes will close the field early, causing a syntax error. For example, the following is incorrect is will throw an error:
"nbt": "{SomeTag:"value"}"

While the following is correct:
"nbt": "{SomeTag:\"value\"}"

Hi, thanks for the reply.
I was using 1.12 when I opened the thread so I'll give 1.13 a try.
For nbt, in the case of multiple tags would this be correct?
{display:{Name:"CustomName",Lore:["Line1","Line2"]}},{Enchantments:[{id:#,lvl:#},{id:#,lvl:#}]}

You closed the root compound (indicated by curly brackets) and opened a new one. This will throw a syntax error (i.e. you have {data},{data} when it should be {data,data}). Fixed:
{display:{Name:\"CustomName\",Lore:[\"Line1\",\"Line2\"]},Enchantments:[{id:#,lvl:#},{id:#,lvl:#}]}

{     "__comment": "Made with TheDestruc7i0n's advancements generator: https://advancements.thedestruc7i0n.ca",     "display": {         "title": "city",         "description": "Visit city",         "icon": {             "item": "minecraft:furnace"         },         "frame": "task",         "show_toast": true,         "announce_to_chat": false,         "hidden": false     },     "criteria": {         "visit_city": {             "trigger": "minecraft:location",             "conditions": {                 "dimension": "overworld",                 "position": {                     "x": {                         "min": -7012,                         "max": -7016                     },                     "y": {                         "min": 102,                         "max": 102                     },                     "z": {                         "min": -7955,                         "max": -7961                     }                 }             }         }     },     "parent": "placeholder:tourist" }
I've just had another attempt in 1.13. The above doesn't work for me.
Here's a link to a pastebin version thats easier to look at https://pastebin.com/uxgzmi71


Simple nav UI

This PR implements a simple UI nav for the project. Fontawesome is used for the icons with each module specifying a navIcon property in its loadModule() call. The icons are automatically built by the main script and a showing event is fired on the primary UI element given to the module in the render method.
@rposborne for your review...

🚢