Update resource-name-rules.md

The resource name rules in the document for Image Gallery and Image Definition are conflicting. The Gallery and Image cannot have dashes (hyphens) in the name.
Ref: https://docs.microsoft.com/en-us/azure/virtual-machines/troubleshooting-shared-images

@benhurjoel : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@benhurjoel - thanks for this update. I see that underscores should be added to gallery images, but I was able to create one that included a hyphen. Also, the regex for images seems to indicate that hyphens are allowed. Can you add hyphens back to the allowed characters for both resource types?

@tfitzmac Thank you for your response. I tried creating it on portal and I'm getting the error

Also, the regex mentioned in this doc page seems to be incorrect as well (I've opened another PR)
https://docs.microsoft.com/en-us/azure/virtual-machines/troubleshooting-shared-images

@olayemio - can you take a look at this feedback? Galleries/images definitely accepts hyphens. Either the portal is enforcing a rule that doesn't actually apply, or perhaps the error message is related to creating the gallery which doesn't accept these characters.

@tfitzmac you are referring to gallery images (aka image definitions). @benhurjoel you are referring to galleries, which are the parent of images.
Galleries do not allow hyphens but gallery images do. I have a PR for the update for gallery images.

I have submitted an update that I believe reconciles the requirements.

#please-close

Thank you @tfitzmac and @olayemio

Thank you @tfitzmac and @olayemio


Update metadata.proto

Add comment to kythe.proto.MappingRule.end about it being exclusive rather than inclusive.

Closing extremely stale PR.


Koa app 500 error when trying to access js files

Thanks for your great project!
I have read the documentation, but still cannot figure out how to serve my Koa app.
Here is my app.js:
require('dotenv').config();
const Koa = require('koa');
const app = new Koa();
const bodyParser = require('koa-bodyparser');
const config = require('./config');
require('./prototypes');

app.use(bodyParser());

const router = require('./routes');
app.use(router.routes());

const server = app.listen(config.port, () => {
  console.log(`HITMers-server is running on port ${config.port}`);
});

module.exports = server;

And this is my now.json:
{
  "version": 2,
  "name": "hitmers",
  "builds": [
    { "src": "src/app.js", "use": "@now/node-server" }
  ],
  "env": {
    "DATABASE_URL": "@database_url",
    "SECRET": "@secret",
    "STREAM_USERNAME": "@stream_username",
    "STREAM_PASSWORD": "@stream_password",
    "TZ": "Asia/Shanghai",
    "REGISTER_CODE": "@register_code"
  }
}

The problem is that after I deployed my project, I always get HTTP ERROR 500. I think it is because all other js files except app.js are not served, but is there any to serve the whole project?

Solved by using version 1 instead of version 2.

@upupming Koa is possible with v2 as well, and recommended. Example:
https://github.com/zeit/now-examples/pull/157

Tried it, but got following error in the logs:
11/25 06:59 PM (29s)
Error while initializing entrypoint: { Error: EACCES: permission denied, scandir '/builddir'
    at Object.fs.readdirSync (fs.js:904:18)
    at mapDir (/var/task/user/src/app.js:38575:30)
    at dirs.forEach.dir (/var/task/user/src/app.js:38579:17)
    at Array.forEach (<anonymous>)
    at mapDir (/var/task/user/src/app.js:38578:8)
    at Object.<anonymous> (/var/task/user/src/app.js:38593:18)
    at Object.module.exports.VERSION (/var/task/user/src/app.js:38595:30)
    at __webpack_require__ (/var/task/user/src/app.js:21:30)
    at Object.<anonymous> (/var/task/user/src/app.js:34569:21)
    at __webpack_require__ (/var/task/user/src/app.js:21:30)
  errno: -13,
  code: 'EACCES',
  syscall: 'scandir',
  path: '/builddir' }
11/25 06:59 PM (29s)
REPORT RequestId: 2d9ee993-f0a1-11e8-a206-8da80bfd4385	Duration: 15.53 ms	Billed Duration: 100 ms 	Memory Size: 1024 MB	Max Memory Used: 52 MB

Get this error too. fs.readdirSync not working as expected, but it pretty useful pattern for dynamic loading.
fs.readdirSync(path.join(__dirname, 'routes')).forEach(routeFileName => {
  if (/\.js$/.test(routeFileName)) {
    require(`./routes/${routeFileName}`)(app)
  }
})


method_missing': undefined method this' for #<Gem::Specification:0x3fc56a02c2bc activesupport-<IP_ADDRESS>> (NoMethodError)

I have installed both mailcatcher versions 0.6.3 and 0.6.1.
When I run the command mailcatcher, I get the following error:
/Users/jquigl44/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/specification.rb:2159:in `method_missing': undefined method `this' for #<Gem::Specification:0x3fc0e842c3fc activesupport-<IP_ADDRESS>> (NoMethodError)
	from /Users/jquigl44/.rvm/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/rubygems/specification.rb:1057:in `find_active_stub_by_path'
	from /Users/jquigl44/.rvm/rubies/ruby-
. . .
	from /Users/jquigl44/.rvm/gems/ruby-2.2.2@avant_basic/bin/ruby_executable_hooks:15:in `<main>'

Please help.

gotta lock down the requirements in gemspec @sj26
installing this version of active support fixes the issue
activesupport  4.0.13

It seems like activesupport 4.0.13 works great with mailcatcher. I've just tested it. Avant basic requires activesupport 3.2.22 at the moment.

Experiencing the same issue as @jayqui. Activesupport 4.0.13 works great with mailcatcher. Just tested and it works. =)

I'm having the same issue. Any tips are appreciated.

This looks like a rubygems bug, folks. Not what, I can't replicate it. What version of rubygems are you using? (Post the output of gem environment).

Can you also please post the full stack trace? The meaningful bit is probably in the middle somewhere.

Right. So I've made mailcatcher explicitly activate its gem dependencies. Please install 0.6.4 and see if it solves your issues.

Hi @sj26 , thanks for working on this. I've got v0.6.4 installed and getting this still:
vagrant@vagrant-ubuntu-trusty-64:/vagrant$ mailcatcher

/usr/local/rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/specification.rb:2158:in `method_missing': undefined method `this' for #<Gem::Specification:0xcaf724 mailcatcher-0.6.4> (NoMethodError)
	from /usr/local/rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/specification.rb:1057:in `find_active_stub_by_path'
	from /usr/local/rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:64:in `require'
	from /usr/local/rvm/gems/ruby-2.3.0@unique/gems/mailcatcher-0.6.4/bin/mailcatcher:3:in `<top (required)>'
	from /usr/local/rvm/gems/ruby-2.3.0@unique/bin/mailcatcher:23:in `load'
	from /usr/local/rvm/gems/ruby-2.3.0@unique/bin/mailcatcher:23:in `<main>'
	from /usr/local/rvm/gems/ruby-2.3.0@unique/bin/ruby_executable_hooks:15:in `eval'
	from /usr/local/rvm/gems/ruby-2.3.0@unique/bin/ruby_executable_hooks:15:in `<main>'

Same Issue here after bundle update ... So workaround for me is using a different (i.e. the old) gemset - Create a separate directory with .ruby-gemset and .ruby-version and install/launch it there.

@kendrikat can you help me understand? What is different in your old gemset allowing it work?

$ gem environment
RubyGems Environment:

RUBYGEMS VERSION: 2.5.1

$ gem install mailcatcher
Successfully installed mailcatcher-0.6.4

$ mailcatcher
/Users/bengolden/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/specification.rb:2158:in method_missing': undefined method this' for #<Gem::Specification:0x3fec7981888c mailcatcher-0.6.4> (NoMethodError)
from /Users/bengolden/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/specification.rb:1057:in find_active_stub_by_path' 	from /Users/bengolden/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:64:in require'
from /Users/bengolden/.rvm/gems/ruby-2.3.0/gems/mailcatcher-0.6.4/bin/mailcatcher:3:in <top (required)>' 	from /Users/bengolden/.rvm/gems/ruby-2.3.0/bin/mailcatcher:23:in load'
from /Users/bengolden/.rvm/gems/ruby-2.3.0/bin/mailcatcher:23:in <main>' 	from /Users/bengolden/.rvm/gems/ruby-2.3.0/bin/ruby_executable_hooks:15:in eval'
from /Users/bengolden/.rvm/gems/ruby-2.3.0/bin/ruby_executable_hooks:15:in `'

RubyGems Environment:

RUBYGEMS VERSION: 2.5.1
RUBY VERSION: 2.2.3 (2015-08-18 patchlevel 173) [x86_64-darwin14]

RubyGems Environment:

RUBYGEMS VERSION: 2.5.2
RUBY VERSION: 2.2.2 (2015-04-13 patchlevel 95) [x86_64-darwin14]

it looks like both versions are not working for me

RubyGems Environment:

RUBYGEMS VERSION: 2.5.2
RUBY VERSION: 2.2.2 (2015-04-13 patchlevel 95) [x86_64-darwin14]
RubyGems Environment:
RUBYGEMS VERSION: 2.5.1
RUBY VERSION: 2.2.3 (2015-08-18 patchlevel 173) [x86_64-darwin14]

Both are not working for me =(

same / similar bug here:
$ mailcatcher
/Users/joe/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/specification.rb:2158:in `method_missing': undefined method `this' for #<Gem::Specification:0x3fec1208eac4 mailcatcher-0.6.4> (NoMethodError)
	from /Users/joe/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/specification.rb:1057:in `find_active_stub_by_path'
	from /Users/joe/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:64:in `require'
	from /Users/joe/.rvm/gems/ruby-2.3.0@giftticket/gems/mailcatcher-0.6.4/bin/mailcatcher:3:in `<top (required)>'
	from /Users/joe/.rvm/gems/ruby-2.3.0@giftticket/bin/mailcatcher:23:in `load'
	from /Users/joe/.rvm/gems/ruby-2.3.0@giftticket/bin/mailcatcher:23:in `<main>'
	from /Users/joe/.rvm/gems/ruby-2.3.0@giftticket/bin/ruby_executable_hooks:15:in `eval'
	from /Users/joe/.rvm/gems/ruby-2.3.0@giftticket/bin/ruby_executable_hooks:15:in `<main>'

$ gem --version
2.5.1

I get these same errors when I try to run mailcatcher from my Rails root. Runs ok if I call the direct path to mailcatcher AND I run it from my home directory:
ju@Dullhead:~$ which mailcatcher
/Users/ju/.rvm/gems/ruby-2.3.0@tasks/bin/mailcatcher
ju@Dullhead:~$ pwd
/Users/ju
ju@Dullhead:~$ /Users/ju/.rvm/gems/ruby-2.3.0@tasks/bin/mailcatcher
Starting MailCatcher
==> smtp://<IP_ADDRESS>:1025
==> http://<IP_ADDRESS>:1080
*** MailCatcher runs as a daemon by default. Go to the web interface to quit.

I ran in to this error when trying to run use the travis gem. Running gem pristine --all fixed it for me.

@cameronstanley , that worked for me even when rebuilding the Vagrant VM didn't work. Thanks!

@jnamkung , that worked for me.  My specs match @bengolden

RubyGems Environment:

RUBYGEMS VERSION: 2.5.2
RUBY VERSION: 2.3.0 (2015-12-25 patchlevel 0) [x86_64-darwin15]

:(

I'm getting this error too, cameronstanley's fix didn't work for me.

I still can't reproduce this. It might be based on the combination of gems installed. Anybody affected could you please provide as much information as possible: what is your ruby version, how is it installed, the full output of gem environment and gem list.

Let me know if anything else could help you.
✗ ruby -v
ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-darwin15]
✗ rvm -v
rvm 1.26.11 (latest) by Wayne E. Seguin<EMAIL_ADDRESS>Michal Papis<EMAIL_ADDRESS>[https://rvm.io/]
✗ gem list
*** LOCAL GEMS ***
aasm (4.7.0)
actionmailer (4.2.5)
actionpack (4.2.5)
actionview (4.2.5)
activejob (4.2.5)
activemodel (4.2.5)
activerecord (4.2.5)
activesupport (4.2.5)
acts_as_list (0.7.2)
addressable (2.4.0)
arel (6.0.3)
autoprefixer-rails (6.3.1)
bcrypt (3.1.10)
bigdecimal (1.2.8)
binding_of_caller (0.7.2)
bootstrap-multiselect-rails (0.9.9)
bootstrap-sass (<IP_ADDRESS>)
builder (3.2.2)
bundle (0.0.1)
bundler (1.11.2)
bundler-unload (1.0.2)
byebug (8.2.1)
cancancan (1.13.1)
capistrano (2.15.7)
capybara (2.6.0)
climate_control (0.0.3)
cocaine (0.5.8)
coffee-rails (4.1.1)
coffee-script (2.4.1)
coffee-script-source (1.10.0)
concurrent-ruby (1.0.0)
daemons (1.2.3)
database_cleaner (1.5.1)
debug_inspector (0.0.2)
delayed_job (4.1.1)
delayed_job_active_record (4.1.0)
devise (3.4.1)
did_you_mean (1.0.0)
diff-lcs (1.2.5)
email_spec (2.0.0)
erubis (2.7.0)
eventmachine (<IP_ADDRESS>)
exception_notification (4.1.4)
execjs (2.6.0)
executable-hooks (1.3.2)
factory_girl (4.5.0)
factory_girl_rails (4.5.0)
gem-wrappers (1.2.7)
globalid (0.3.6)
highline (1.7.8)
htmlentities (4.3.4)
httparty (0.13.7)
i18n (0.7.0)
io-console (0.4.5)
jbuilder (2.4.0)
jquery-rails (4.1.0)
json (1.8.3)
launchy (2.4.3)
loofah (2.0.3)
mail (2.6.3)
mailcatcher (0.6.4)
mime-types (2.99)
mini_portile2 (2.0.0)
minitest (5.8.4, 5.8.3)
multi_json (1.11.2)
multi_xml (0.5.5)
net-scp (1.2.1)
net-sftp (2.1.2)
net-ssh (3.0.2)
net-ssh-gateway (1.2.0)
net-telnet (0.1.1)
nokogiri (<IP_ADDRESS>)
orm_adapter (0.5.0)
paperclip (4.2.4)
pg (0.18.4)
power_assert (0.2.6)
psych (2.0.17)
rack (1.6.4)
rack-protection (1.5.3)
rack-test (0.6.3)
rails (4.2.5)
rails-deprecated_sanitizer (1.0.3)
rails-dom-testing (1.0.7)
rails-html-sanitizer (1.0.2)
railties (4.2.5)
rake (10.5.0, 10.4.2)
rdoc (4.2.1)
responders (2.1.1)
roo (2.0.1)
rspec-core (3.4.1)
rspec-expectations (3.4.0)
rspec-mocks (3.4.1)
rspec-rails (3.4.0)
rspec-support (3.4.1)
rubygems-bundler (1.4.4)
rubyzip (1.1.7)
rvm (<IP_ADDRESS>)
rvm-capistrano (1.5.6)
sass (3.4.21)
sass-rails (5.0.4)
sdoc (0.4.1)
sinatra (1.4.7)
skinny (0.2.4)
slim (3.0.6)
spring (1.6.2)
sprockets (3.5.2)
sprockets-rails (3.0.0)
sqlite3 (1.3.11)
temping (3.3.0)
temple (0.7.6)
test-unit (3.1.5)
thin (1.5.1)
thor (0.19.1)
thread_safe (0.3.5)
tilt (2.0.2)
turbolinks (2.5.3)
tzinfo (1.2.2)
uglifier (2.7.2)
warden (1.2.4)
web-console (2.2.1)
will_paginate (3.0.7)
will_paginate-bootstrap (1.0.1)
write_xlsx (0.83.0)
xpath (2.0.0)
zip-zip (0.3)
✗ gem environment
RubyGems Environment:

RUBYGEMS VERSION: 2.5.1
RUBY VERSION: 2.3.0 (2015-12-25 patchlevel 0) [x86_64-darwin15]
INSTALLATION DIRECTORY: /Users/guillaumelegendre/.rvm/gems/ruby-2.3.0@investissement_pierre
USER INSTALLATION DIRECTORY: /Users/guillaumelegendre/.gem/ruby/2.3.0
RUBY EXECUTABLE: /Users/guillaumelegendre/.rvm/rubies/ruby-2.3.0/bin/ruby
EXECUTABLE DIRECTORY: /Users/guillaumelegendre/.rvm/gems/ruby-2.3.0@investissement_pierre/bin
SPEC CACHE DIRECTORY: /Users/guillaumelegendre/.gem/specs
SYSTEM CONFIGURATION DIRECTORY: /Users/guillaumelegendre/.rvm/rubies/ruby-2.3.0/etc
RUBYGEMS PLATFORMS:

ruby
x86_64-darwin-15

GEM PATHS:

/Users/guillaumelegendre/.rvm/gems/ruby-2.3.0@investissement_pierre
/Users/guillaumelegendre/.rvm/gems/ruby-2.3.0@global

GEM CONFIGURATION:

:update_sources => true
:verbose => true
:backtrace => false
:bulk_threshold => 1000
"gem" => "--no-document"

REMOTE SOURCES:

https://rubygems.org/

SHELL PATH:

/Users/guillaumelegendre/.rvm/gems/ruby-2.3.0@investissement_pierre/bin
/Users/guillaumelegendre/.rvm/gems/ruby-2.3.0@global/bin
/Users/guillaumelegendre/.rvm/rubies/ruby-2.3.0/bin
/Users/guillaumelegendre/.rvm/bin
/usr/local/bin
/usr/bin
/bin
/usr/sbin
/sbin

I've just noticed: everybody seems to be using rvm, and gemsets. Have you tried installing without gemsets?

No I didn't and i'm not on my personnal computer so I don't want to mess around with the config. But it works in other gemsets. I used them currently to use mailcatcher. So the version of ruby changes and the gem list change but rvm is the same.

@GuillaumeLegendre I've installed the same version of rvm, ruby, created a gemset, and installed all the same gems and versions, but I don't see the error:
https://gist.github.com/sj26/35dc5e343d45f0177a07
Are your gems distributed between your gemsets?

@cameronstanley, that worked for me. Thanks!

+1 not working for me neither

This worked for me.
$ gem uninstall mailcatcher (from your local gemset)
As per the instructions on mailcatcher.me, run the following two lines:
$ rvm default@mailcatcher --create do gem install mailcatcher
$ rvm wrapper default@mailcatcher --no-prefix mailcatcher catchmail
$ mailcatcher
Starting MailCatcher
==> smtp://<IP_ADDRESS>:1025
==> http://<IP_ADDRESS>:1080
*** MailCatcher runs as a daemon by default. Go to the web interface to quit.

For reference, this issue appears related: https://github.com/rubygems/rubygems/issues/1420
There are a few other possible solutions noted

@jaydorsey thanks! That sounds exactly like the problem. It only seems to occur in rvm with gemsets. This sounds likely:
https://github.com/rubygems/rubygems/issues/1420#issuecomment-167939914
There is a patch proposed:
https://gist.github.com/mpapis/bac6163f8855e12f7833
I haven't tried reproducing using bundler but I'll give it a go.

@charazn - thanks, worked great for me -- most appreciated.

I had the same issue and fixed it by re-installing mailcatcher in the way @charazn describes above.
For reference my issue started after upgrading bundler.

This is unfortunately a bug in the combiantion of Rubygems/Bundler/RVM. @charazn's instructions per the README are the safest way to install MailCatcher with RVM.

why does this seem to only happen with mailcatcher, and no other gem?

It doesn't.
It seems to happen in a particular edge case with rubygems, rvm and bundler. MailCatcher being an independent gem often used within projects with similar dependencies using bundler seems to hit the edge case more often.

It's a bug in rubygems, may be below listed help you.

Open file  rubygems/specification.rb
/Users/amitagarwal/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/specification.rb
add method
`` def this; self; end
in the end of file
rvm reload
restart your application

I was able to fix this error by upgrading rubygems to the latest version
gem update --system

@rnmccandless  isn't this will effect the other apps in the system ?

@rnmccandless  isn't this will effect the other apps in the system

Please try upgrading rubygems and mailcatcher and open a new issue with a fresh backtrace if you're still having issues.


Ability to download multiple classes at once

Hey there!
I have been using the skillshare downloader and it works great, but I have a problem. I can't manually download the videos by editing it all the time since I have a large collection of classes to download.
I was wondering if there is a way to download multiple videos at once, it could be VERY handy. The program overall does the job fine and keep the great work up man!

Hey, glad you like it!
No, there's is not some built-in "automagic" way to download more classes than you specifically tell it but it is just python-code after all so should be easy to put something together yourself if you're up for the challenge. :)
A simple change like this would at least let you queue up a bunch of classes at once and then leave the computer to do the job.
# download by class URL: for url in [     'https://www.skillshare.com/classes/Art-Fundamentals-in-One-Hour/189505397',     'some other class url here',     'and another one here, etc...', ]:     dl.download_course_by_url(url)


wl-mirror: init at 0.5.0

Motivation for this change
Implement wl-mirror, which lets you mirror wayland displays to a surface.
Depends on #147296 147296
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Brilliant, works perfectly. Didn't expect external software to be able to fill this feature, but here we are!

Should be good to go :)
[xxx@xxx]:~/projects/nixpkgs> nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
these 2 paths will be fetched (0.09 MiB download, 0.42 MiB unpacked):
  /nix/store/mj03ngg8sq0m7x8za9mlb9mmp3f0l0pg-nixpkgs-review-2.6.3
  /nix/store/sc4zj3a736kyrfiv2ig7h1ycmmr77cdv-bash-interactive-5.1-p8-dev
copying path '/nix/store/sc4zj3a736kyrfiv2ig7h1ycmmr77cdv-bash-interactive-5.1-p8-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/mj03ngg8sq0m7x8za9mlb9mmp3f0l0pg-nixpkgs-review-2.6.3' from 'https://cache.nixos.org'...
$ git -c fetch.prune=false fetch --no-tags --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0
From https://github.com/NixOS/nixpkgs
   20a166d2603..9dc12546ab3  master     -> refs/nixpkgs-review/0
$ git worktree add /home/xxx/.cache/nixpkgs-review/rev-b71ed45cfc402ee498b973dc919e8bcf433deeea/nixpkgs 9dc12546ab3d3b619a79ebecc1d6759b4f1f2024
Preparing worktree (detached HEAD 9dc12546ab3)
Updating files: 100% (29020/29020), done.
HEAD is now at 9dc12546ab3 Merge pull request #147030 from jtojnar/linkchecker
$ nix-env --option system x86_64-linux -f /home/xxx/.cache/nixpkgs-review/rev-b71ed45cfc402ee498b973dc919e8bcf433deeea/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit --no-ff b71ed45cfc402ee498b973dc919e8bcf433deeea
Automatic merge went well; stopped before committing as requested
$ nix-env --option system x86_64-linux -f /home/xxx/.cache/nixpkgs-review/rev-b71ed45cfc402ee498b973dc919e8bcf433deeea/nixpkgs -qaP --xml --out-path --show-trace --meta
1 package added:
wl-mirror (init at 0.5.0)

$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /home/xxx/.cache/nixpkgs-review/rev-b71ed45cfc402ee498b973dc919e8bcf433deeea/build.nix
1 package built:
wl-mirror

$ nix-shell /home/xxx/.cache/nixpkgs-review/rev-b71ed45cfc402ee498b973dc919e8bcf433deeea/shell.nix

Friendly ping :) Anything else I should do to get this in?

Eh no lgtm

(Wanna git this in stable because it's a pretty major QOL inprovement for Sway users)


(Linters) Adding missing --update which should ...

(Linters) Adding missing --update which should enhance the chance that
everyone sees the same/latest linter errors

This change is

/lgtm
/approve


aws[minor]: Add ChatModel that uses Bedrock.converse API

Decisions to discuss:

supports bedrock converse and anthropic format inputs
outputs anthropic format messages

The reason being converse will not work in a chain unless we have prompt tenmplate, retrievers...

Not sure i understand, why wouldn't it work with other components? its still implementing the BaseChatModel interface

The reason being say I define a chain like
Prompt | llm | outputparse
And if I try and call it by doing chain.converse(..) that will not work so I have to invoke it using chain.invoke(..) however in LLM (ChatBedrock ) class we can fork and call converse api
Secondly from design we encapsulate the converse api since that is native and will proprietary to Bedrock only. SageMaker for example will not support converse api
From: Bagatur @.>
Reply-To: langchain-ai/langchain-aws @.>
Date: Thursday, June 13, 2024 at 3:24 PM
To: langchain-ai/langchain-aws @.>
Cc: "Grewal, Rupinder" @.>, Comment @.***>
Subject: Re: [langchain-ai/langchain-aws] aws[minor]: Add ChatModel that uses Bedrock.converse API (PR #74)
The reason being converse will not work in a chain unless we have prompt tenmplate, retrievers...
Not sure i understand, why wouldn't it work with other components? its still implementing the BaseChatModel interface
—
Reply to this email directly, view it on GitHubhttps://github.com/langchain-ai/langchain-aws/pull/74#issuecomment-2166890776, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYMBZRRLRUDCUJXNGE5JMSDZHILWPAVCNFSM6AAAAABJJGOL7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWHA4TANZXGY.
You are receiving this because you commented.Message ID: @.***>

The reason being say I define a chain like Prompt | llm | outputparse And if I try and call it by doing chain.converse(..) that will not work so I have to invoke it using chain.invoke(..) however in LLM (ChatBedrock ) class we can fork and call converse api Secondly from design we encapsulate the converse api since that is native and will proprietary to Bedrock only. SageMaker for example will not support converse api From: Bagatur @.> Reply-To: langchain-ai/langchain-aws @.> Date: Thursday, June 13, 2024 at 3:24 PM To: langchain-ai/langchain-aws @.> Cc: "Grewal, Rupinder" @.>, Comment @.> Subject: Re: [langchain-ai/langchain-aws] aws[minor]: Add ChatModel that uses Bedrock.converse API (PR #74) The reason being converse will not work in a chain unless we have prompt tenmplate, retrievers... Not sure i understand, why wouldn't it work with other components? its still implementing the BaseChatModel interface — Reply to this email directly, view it on GitHub<#74 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYMBZRRLRUDCUJXNGE5JMSDZHILWPAVCNFSM6AAAAABJJGOL7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWHA4TANZXGY. You are receiving this because you commented.Message ID: @.>

to clarify, this is using the bedrock client's converse API but the way you use the result langchain chat model is still via invoke. so your chain would continue to work

Right but it would not be using the converse api. Our goal is to have the ChatBedrock use and switch to the converse api internally since that will be the dominant way of running the workloads. Hence I feel a new class by itself will not be so useful if I cannot plug it into any chain
From: Bagatur @.>
Reply-To: langchain-ai/langchain-aws @.>
Date: Thursday, June 13, 2024 at 5:52 PM
To: langchain-ai/langchain-aws @.>
Cc: "Grewal, Rupinder" @.>, Comment @.***>
Subject: Re: [langchain-ai/langchain-aws] aws[minor]: Add ChatModel that uses Bedrock.converse API (PR #74)
The reason being say I define a chain like Prompt | llm | outputparse And if I try and call it by doing chain.converse(..) that will not work so I have to invoke it using chain.invoke(..) however in LLM (ChatBedrock ) class we can fork and call converse api Secondly from design we encapsulate the converse api since that is native and will proprietary to Bedrock only. SageMaker for example will not support converse api From: Bagatur @.> Reply-To: langchain-ai/langchain-aws @.> Date: Thursday, June 13, 2024 at 3:24 PM To: langchain-ai/langchain-aws @.> Cc: "Grewal, Rupinder" @.>, Comment @.> Subject: Re: [langchain-ai/langchain-aws] aws[minor]: Add ChatModel that uses Bedrock.converse API (PR #74https://github.com/langchain-ai/langchain-aws/pull/74) The reason being converse will not work in a chain unless we have prompt tenmplate, retrievers... Not sure i understand, why wouldn't it work with other components? its still implementing the BaseChatModel interface — Reply to this email directly, view it on GitHub<#74 (comment)https://github.com/langchain-ai/langchain-aws/pull/74#issuecomment-2166890776>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYMBZRRLRUDCUJXNGE5JMSDZHILWPAVCNFSM6AAAAABJJGOL7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWHA4TANZXGY. You are receiving this because you commented.Message ID: @.>
to clarify, this is using the bedrock client's converse API but the way you use the result langchain chat model is still via invoke. so your chain would continue to work
—
Reply to this email directly, view it on GitHubhttps://github.com/langchain-ai/langchain-aws/pull/74#issuecomment-2167020667, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYMBZRUBGOQ2J4NOABAFVGTZHI5B7AVCNFSM6AAAAABJJGOL7CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRXGAZDANRWG4.
You are receiving this because you commented.Message ID: @.***>

@baskaryan is there a way for me to test this (other than copying the code locally)? I would really like to give it a try.

@baskaryan is there a way for me to test this (other than copying the code locally)? I would really like to give it a try.

Nevermind, I got it, leaving it here for reference:
poetry add git+https://github.com/langchain-ai/langchain-aws@bagatur/bedrock_converse#subdirectory=libs/aws

Here are my findings after playing with the ChatBedrockConverse model:

We should keep model_id instead of model (remove the alias) so that is in line with BedrockBase
Found two fields that are optional but that had no default value (added them to #76)
Found some fields that might have been typos (e.g., toolUseID vs toolUseId)

From a user perspective, will it be confusing to switch class name yet again, to ChatBedrockConverse? If the converse API will be the primary API going forward, can ChatBedrock not just switch to using that?

From a user perspective, will it be confusing to switch class name yet again, to ChatBedrockConverse? If the converse API will be the primary API going forward, can ChatBedrock not just switch to using that?
BedrockChat -> ChatBedrock -> ChatBedrockConverse is getting a little crazy IMO 😅

idea is to eventually replace ChatBedrock with ChatBedrockConverse. issue is converse doesn't currently support guardrails or custom models afaik, so it'd be breaking to replace now. but for all regular functionality converse provides a more reliable/universal api (eg we can implement tool calling for all models in one go) so it seems worth exposing now.
very open to different names (eg ChatBedrockV2). also this is being marked as beta and we can make clear in the docs that this is a preview of what's to come, and folks should only switch to it if they are having issues with ChatBedrock or want to get ahead of the curve

We cannot change the name as that is 1/ breaking change and will cause confusion for existing customers 2/  We need to follow the naming convention of LangChain which is ChatXYZ.... like ChatAnthropic etc. So ChatBedrock will need to stay 3/ Converse API is a internal to Bedrock and not a feature of LangChain and hence it has to be encapsulated in the ChatBedrock class

I echo @austinmw's and @rsgrewal-aws position on this. Would it make more sense to have an optional parameter for ChatBedrock defaulting to false, called enable_converse_api or something along the lines?
@3coins thoughts?

also there is another PR -- https://github.com/langchain-ai/langchain-aws/pull/76 may be we can review that

@austinmw @anjanvb
Having this new class provides option to users who want to migrate to the converse API (tool support) early while also allowing users who want to use guardrails/custom models to keep using the existing bedrock classes. Existing Bedrock classes are already hard to maintain and test with all the conditional logic, and any additional code to support converse in the same class will make it worse.

Understood. Would have been ideal IMO to turn on converse functionality in ChatBedrock directly with an optional parameter, but if that makes the class logic too complicated to maintain then I guess a new beta class will have to do.

i would suggest to then add the builder pattern to compose this class at run time and inject the objects which provide or do not provide the functionality. But changing the Class name and pivoting to a new class has impacts and consequences wider and we will tend to loose customer trust . This will be the 3rd class we will be introducing for Bedrock

think we could maybe get the best of both worlds, added support for
ChatBedrock(beta_use_converse_api=True)

https://github.com/langchain-ai/langchain-aws/pull/74/commits/55710e41bcd1b2069f8028d187d54fe1e59751ce
which delegates logic to ChatBedrockConverse. what do folks thinks @3coins @austinmw @anjanvb?

@baskaryan
Looks good to me!
@rsgrewal-aws @austinmw
Are you folks ok with this change? Would you like to test the changes and provide your feedback here?

@baskaryan Looks good to me!
@rsgrewal-aws @austinmw Are you folks ok with this change? Would you like to test the changes and provide your feedback here?

I'm good with this approach @3coins @baskaryan

Awesome -- good with this  for now - this will help a lot. I think we can may be start to refactor this class and do the builder / composite pattern to start to reduce the complexity and reduce all the if-then statements

Updated the other PR to submit just the new changes for the stuff I found during testing with the anthropic claude model using the new bedrock converse endpoint.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1193 ms

Resolved: Bofrost.lu performance has improved in 203f45c after 6 minutes.


Support for loading YAML-files

Typst supports loading data from a variety of standard data formats, but YAML is not one of them. I guess the main reason is that typst Dicts expect a mapping Str -> Value while YAML Mappings can be of any value to any value.
However, I still think that being able to load YAML data would be useful even if it treats dictionary keys as strings unconditionally.
What are your thoughts on this? I can implement it myself if you want.

Being worked on in #447.

Merged

I added this comment in Discord, but my employer does not let me access Discord at work, so I'll state it here.
I like the use of YAML, but I would prefer to see the yaml object refer to a group of related methods for serialization and deserialization, rather than a standalone method that loads a file (as documented https://typst.app/docs/reference/data-loading/yaml/)
For example:

yaml.load_file(some_path.yaml) returns the loaded yaml file.
yaml.load(some_string) converts the string into an object
yaml.dump(some_object) serializes the object into a string
yaml.dump_file(some_object) serializes the object into a file (if writing to a file is allowed in typst, otherwise don't include this)

As far as use cases for yaml.dump, here are a few I can think of, and which I have used frequently in various reports and other cases:

You have some hierarchical metadata structure which you would like to show the user. (I use build time, build machine, version control info, etc. in HTML reports that I've generated in Python, and the easiest way to make them readable is to dump them in YAML format.)

You have a YAML file for which you would like to show excerpts, so something like:

all = #yaml.load_file(...)

Here's part of it under "foo":

#raw(yaml.dump(all['foo']))

Here's part of it under "bar":

#raw(yaml.dump(all['bar']))


Various small updates to the main readme, include references to Slack and Github Discussions

Feel free to make suggestions to make any changes.

Going to merge given thumbs up from George.


gRPC errors on push from using bst 1.93.3 caching

See original issue on GitLab
In GitLab by [Gitlab user @christopherphang] on May 26, 2020, 09:03
Summary
The following gRPC errors are observed when using bst 1.93.3 and buildbox 0.07
[00:00:34][07b3a35c][    push:bootstrap/pcre-build-deps.bst ] FAILURE Failed to push artifact blobs: Unexpected error in RPC handling
        Printing the last 80 lines from log file:
        /builds/freedesktop-sdk/freedesktop-sdk/cache/buildstream/logs/freedesktop-sdk/bootstrap-pcre-build-deps/07b3a35c-push.1102.log
        ======================================================================
        [--:--:--] START   bootstrap/pcre-build-deps.bst: Push
        [--:--:--] LOG     bootstrap/pcre-build-deps.bst: Build environment for element bootstrap/pcre-build-deps.bst
            PATH: /usr/bin:/bin:/usr/sbin:/sbin
            SHELL: /bin/sh
            TERM: dumb
            USER: tomjon
            USERNAME: tomjon
            LOGNAME: tomjon
            LC_ALL: en_US.UTF-8
            HOME: /tmp
            TZ: UTC
            SOURCE_DATE_EPOCH: '1320937200'
            CFLAGS: '-O2 -g -pipe -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions
              -fstack-protector-strong -grecord-gcc-switches -fasynchronous-unwind-tables -fstack-clash-protection '
            CXXFLAGS: '-O2 -g -pipe -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions
              -fstack-protector-strong -grecord-gcc-switches -fasynchronous-unwind-tables -fstack-clash-protection '
            LDFLAGS: '-Wl,-z,relro,-z,now -Wl,--as-needed '
            PYTHON: /usr/bin/python3
            PYTHONHASHSEED: '0'
        [--:--:--] STATUS  bootstrap/pcre-build-deps.bst: Pushing data from artifact 07b3a35c -> https://push.public.aws.celduin.co.uk
        [00:00:34] FAILURE bootstrap/pcre-build-deps.bst: Failed to push artifact blobs: Unexpected error in RPC handling
        ======================================================================

Steps to reproduce
A representive CI job can be seen at https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/542055129
Other relevant information
This is not seen with 1.93.1, see https://gitlab.com/libreml/libreml/-/merge_requests/270
There is extensive discussion at https://gitlab.com/celduin/infrastructure/celduin-infra/-/issues/144

In GitLab by [Gitlab user @christopherphang] on May 26, 2020, 09:05
marked this issue as related to celduin/infrastructure/celduin-infra#144

In GitLab by [Gitlab user @willsalmon] on May 28, 2020, 14:05
Steps to reproduce locally and quickly:
Checkout commit 6c9aff5af109a197f6d5b757c303e2b0736bd89a (HEAD -> willsalmon/bst2-buildorder, origin/willsalmon/bst2-buildorder) in freedesktop-sdk
Add a push-able cache eg ~/.config/buildstream.conf
artifacts:
  - url: https://push.cache.test.co.uk
    client-cert: path/to/cert
    client-key: path/to/key
    push: true

Use this commit of bst, https://gitlab.com/BuildStream/buildstream/-/commit/0a2fc1816b51441c86eedd2091a3e6f4d85647fe

if you rebase to master it will take longer to reproduce

Then clear or set a new cache with XDG_CACHE_HOME=/path/to/new/cache
run:
bst build tests/check-platform.bst

bst will then pull artifcts from the cache in project.conf and then push them back to the new cache you added in your ~/.config/buildstream.conf
The patch I added will then  push them back 1000 times not the single time that normally would happen
I find that I get a failure before 150 pushes

In GitLab by [Gitlab user @coldtom] on May 28, 2020, 14:12
I think this is related to #1322, after [Gitlab user @juergbi] reproduced (irc logs editorialised for brevity):
13:56 <WSalmon> <juergbi> std::runtime_error exception thrown at [buildboxcommon_client.cpp:1020], errMsg = "FindMissingBlobs() request failed"
13:57 <juergbi> that exception is the reason for the unexpected gRPC error
13:57 <juergbi> issue in findmissingblobs
13:58 <juergbi> (buildboxcommon_client.cpp:1045 in master)

This leads me to think that the source of the regression was !1837, which presumably increased the load on casd, especially for FindMissingBlobs calls. This patch was added after 1.93.1, which is our last known good version.

In GitLab by [Gitlab user @willsalmon] on May 28, 2020, 14:26
Im not sure about source of the regression
Maybe !1837 cause the frequency of calls to casd to increase to the point were hitting the underlying issue in casd became a common occurrence, for the size of project that [Gitlab user @coldtom] and myself typically use?

In GitLab by [Gitlab user @coldtom] on May 28, 2020, 14:28
That was poor phrasing on my part, I mean to say "where we started to see the bug"

In GitLab by [Gitlab user @juergbi] on May 28, 2020, 16:00
First step towards a fix: https://gitlab.com/BuildGrid/buildbox/buildbox-common/-/merge_requests/278
buildbox-casd also needs changes to actually solve this issue.

In GitLab by [Gitlab user @juergbi] on May 28, 2020, 16:36
Improving the error message: https://gitlab.com/BuildGrid/buildbox/buildbox-casd/-/merge_requests/188. Now I get
Failed to push artifact blobs with status UNAVAILABLE: UploadMissingBlobs: Retry limit exceeded. Last gRPC error was 14: Socket closed

In GitLab by [Gitlab user @juergbi] on May 28, 2020, 17:36
I can no longer reproduce the issue with this buildbox-common branch: https://gitlab.com/BuildGrid/buildbox/buildbox-common/-/commits/juerg/grpc-retry

In GitLab by [Gitlab user @juergbi] on May 28, 2020, 19:18
While !1837 slightly increases the load of local buildbox-casd, it shouldn't trigger any extra requests to remote CAS servers. This issue appears to be about failures when (the connection to) the Buildbarn CAS server is under load, not local buildbox-casd load.

In GitLab by [Gitlab user @willsalmon] on May 28, 2020, 19:26
I will try this out in FD CI, Thanks [Gitlab user @juergbi]

In GitLab by [Gitlab user @willsalmon] on Jun 1, 2020, 09:41
https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/574850923 has been running for over 10h's so far so it looks like that branch fixes things.

In GitLab by [Gitlab user @juergbi] on Jun 2, 2020, 11:54
mentioned in merge request BuildGrid/buildbox/buildbox-common!283

In GitLab by [Gitlab user @juergbi] on Jun 3, 2020, 11:43
This has been merged to buildbox-common master: https://gitlab.com/BuildGrid/buildbox/buildbox-common/-/merge_requests/283

In GitLab by [Gitlab user @willsalmon] on Jun 3, 2020, 14:08
This https://gitlab.com/BuildGrid/buildbox/buildbox-common/-/merge_requests/283 seems to have fixed the issue for my testing eg. https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/578770889
I think this issue should be closed once the buildstream website links to a version of buildbox-* that has the above fix.

In GitLab by [Gitlab user @juergbi] on Jun 10, 2020, 15:22
mentioned in merge request buildstream-docker-images!170

In GitLab by [Gitlab user @juergbi] on Jun 10, 2020, 15:34
mentioned in merge request website!141

In GitLab by [Gitlab user @juergbi] on Jun 10, 2020, 16:25
mentioned in merge request !1962

In GitLab by [Gitlab user @juergbi] on Jun 10, 2020, 17:20
All relevant changes have been merged.

In GitLab by [Gitlab user @juergbi] on Jun 10, 2020, 17:20
closed


does it make sense using IORING_POLL_ADD_MULTI together with IOSQE_IO_LINK ?

hi,
I currently use POLLIN with IOSQE_IO_LINK that is folllowed up by IORING_OP_READ of size 8 to drain my eventfd descriptor.
I wanted to add IORING_POLL_ADD_MULTI option but then understood that it would probably be ill-defined with  IOSQE_IO_LINK.
Am I right? Does kernel allow such combination?

It won't really work. Links are started when a request is done, and a multi-shot poll would never really be done (unless you hit an error, for example).


clean: switch to ajson wrapper (Outreach deep connector)

Switched to the ajson wrapper for consistency(and cleanness) in the package.

@Jkarage please fix the build errors, then you can merge.


udpate opendp notebook

The opendp library has updated its API. We should update our notebook using opendp.

fixed in #103


fix error cookie override when path=/

Thanks for this patch!
Have you tested this change with the site you saw the original issue on (https://github.com/duckduckgo/duckduckgo-privacy-extension/issues/1161)? If so, could you also share the stack value that caused the original error? Think might help us create a test-case for this.

So I included this change into https://github.com/duckduckgo/content-scope-scripts/pull/78/ thank you for bringing this to our attention!


Make cmake install shared libraries with full install names by default

Allows executables to find cmake-installed shared libraries at runtime on Darwin. Addresses #6821.

Judging by the rebuild needed for testing this change on Darwin, I think this PR should probably be made against the staging branch.

Is that something I can change on my end?

Unfortunately, it seems you can't change the target branch of an existing PR (http://stackoverflow.com/questions/24159036/how-to-modify-a-pull-request-on-github-to-change-target-branch-to-merge-into).
The cleanest solution would be to cherry-pick your change onto a new cmake-full-install-name (or whatever) branch created from staging instead of master, close this PR, and open a new PR with the staging branch as the target.

Moving to PR #7095 in order to enable merge to staging.


Added solution.user.txt to show user-oriented solutions.

This adds compatibility for a new file in levels, solution.user.txt, where git gud solution --confirm will output the contents of this file instead of the solution.txt.
This is useful for when we have programmatic solutions for the level that we don't want the user to actually use as a solution.
makelevel.py has not been updated because most levels' solutions in solution.txt are what we want the user to use, but there are still levels that need solution.user.txt.

Closed and cherry-picked over to #227


PopupMenuButton retains wrong position on layout change

Steps to reproduce

Run the attached minimal sample on any platform
Tap an overflow icon button to reveal the overflow menu
Rotate the screen from portrait to landscape or vice versa

Expected results
The overflow menu hides after rotation or recalculates its right position based on its anchor
Actual results
The overflow menu is still visible, but in the wrong position
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'PopupMenuButton position',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const HomePageView(),
    );
  }
}

class HomePageView extends StatefulWidget {
  const HomePageView({super.key});

  @override
  State<HomePageView> createState() => _HomePageViewState();
}

class _HomePageViewState extends State<HomePageView> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('PopupMenuButton position')),
      body: OrientationBuilder(builder: (context, orientation) {
        int tileCount = orientation == Orientation.portrait ? 2 : 5;
        return Padding(
          padding: const EdgeInsets.all(8),
          child: GridView.count(
            crossAxisCount: tileCount,
            crossAxisSpacing: 8,
            mainAxisSpacing: 8,
            childAspectRatio: 1,
            children: List.generate(17, (index) {
              return GridTile(
                footer: GridTileBar(
                  title: Text(
                    'Item $index',
                    style: TextStyle(
                      color: Theme.of(context).colorScheme.onPrimaryContainer,
                    ),
                  ),
                  trailing: PopupMenuButton(
                    itemBuilder: (context) {
                      return [
                        const PopupMenuItem(
                          child: Text('Archive'),
                        ),
                        const PopupMenuItem(
                          child: Text('Edit'),
                        ),
                        const PopupMenuItem(
                          child: Text('Delete'),
                        )
                      ];
                    },
                  ),
                ),
                child: Ink(
                  decoration: BoxDecoration(
                    color: Theme.of(context).colorScheme.primaryContainer,
                    borderRadius: const BorderRadius.all(
                      Radius.circular(8),
                    ),
                  ),
                  width: 175,
                  height: 250,
                ),
              );
            }),
          ),
        );
      }),
    );
  }
}

Screenshots or Video
No response
Logs
Logs
Running Gradle task 'assembleDebug'...
√ Built build\app\outputs\flutter-apk\app-debug.apk
Installing build\app\outputs\flutter-apk\app-debug.apk...
D/FlutterFileDialogPlugin(24349): onAttachedToEngine - IN
D/FlutterFileDialogPlugin(24349): doOnAttachedToEngine - IN
D/FlutterFileDialogPlugin(24349): doOnAttachedToEngine - OUT
D/FlutterFileDialogPlugin(24349): onAttachedToEngine - OUT
D/FlutterFileDialogPlugin(24349): onAttachedToActivity
D/FlutterFileDialogPlugin(24349): doOnAttachedToActivity - IN
D/FlutterFileDialogPlugin(24349): doOnAttachedToActivity - OUT
Debug service listening on ws://<IP_ADDRESS>:56853/L-wxvMvU6kE=/ws
Syncing files to device Pixel Fold...
D/ProfileInstaller(24349): Installing profile for it.returntrue.novelist
D/VRI[MainActivity](24349): visibilityChanged oldVisibility=true newVisibility=false
I/ImeTracker(24349): it.returntrue.novelist:2c275c54: onRequestShow at ORIGIN_CLIENT reason SHOW_SOFT_INPUT fromUser false
D/InputMethodManager(24349): showSoftInput() view=io.flutter.embedding.android.FlutterView{113df2b VFE...... .F....ID 0,0-2208,1756 #1 aid=1073741824} flags=0 reason=SHOW_SOFT_INPUT
D/InputConnectionAdaptor(24349): The input method toggled cursor monitoring on
D/InsetsController(24349): show(ime(), fromIme=true)
W/InteractionJankMonitor(24349): Initializing without READ_DEVICE_CONFIG permission. enabled=false, interval=1, missedFrameThreshold=3, frameTimeThreshold=64, package=it.returntrue.novelist
I/ImeTracker(24349): it.returntrue.novelist:2c275c54: onShown
D/InputConnectionAdaptor(24349): The input method toggled cursor monitoring off
D/InsetsController(24349): hide(ime(), fromIme=true)
W/WindowOnBackDispatcher(24349): sendCancelIfRunning: isInProgress=falsecallback=ImeCallback=ImeOnBackInvokedCallback@70107678 Callback=android.window.IOnBackInvokedCallback$Stub$Proxy@935dc34
I/ImeTracker(24349): it.returntrue.novelist:42f354f: onRequestHide at ORIGIN_CLIENT reason HIDE_SOFT_INPUT_ON_ANIMATION_STATE_CHANGED fromUser false
I/ImeTracker(24349): com.google.android.inputmethod.latin:8f8da5d4: onHidden
I/ImeTracker(24349): it.returntrue.novelist:d8a3be57: onRequestHide at ORIGIN_CLIENT reason HIDE_SOFT_INPUT fromUser false
D/VRI[MainActivity](24349): visibilityChanged oldVisibility=true newVisibility=false

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.22.3, on Microsoft Windows [Versione 10.0.19045.4651], locale it-IT)
    • Flutter version 3.22.3 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (12 days ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\Alessandro\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0--11852314)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[!] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.8.34408.163
    X Visual Studio is missing necessary components. Please re-run the Visual Studio installer for the "Desktop development with C++" workload, and include these components:
        MSVC v142 - VS 2019 C++ x64/x86 build tools
         - If there are multiple build tool versions available, install the latest
        C++ CMake tools for Windows
        Windows 10 SDK

[√] Android Studio (version 2024.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = C:\Program Files\Android\Android Studio
    • Java version OpenJDK Runtime Environment (build 17.0.11+0--11852314)

[√] VS Code (version 1.71.2)
    • VS Code at C:\Users\Alessandro\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.48.0

[√] Connected device (4 available)
    • Pixel Fold (mobile) • 35181FDHS00206 • android-arm64  • Android 14 (API 34)
    • Windows (desktop)   • windows        • windows-x64    • Microsoft Windows [Versione 10.0.19045.4651]
    • Chrome (web)        • chrome         • web-javascript • Google Chrome 126.0.6478.185
    • Edge (web)          • edge           • web-javascript • Microsoft Edge 127.0.2651.74

[√] Network resources
    • All expected network resources are available.

Reproducible using the code sample provided above.
https://github.com/user-attachments/assets/a369c3a2-bc01-46f4-95b8-448d99916378

flutter doctor -v
[!] Flutter (Channel stable, 3.22.3, on macOS 14.5 23F79 darwin-arm64, locale en-GB)
    • Flutter version 3.22.3 on channel stable at /Users/nexus/dev/sdks/flutter
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutters/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutter. Consider adding /Users/nexus/dev/sdks/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutters/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutter. Consider adding /Users/nexus/dev/sdks/flutter/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (12 days ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Flutter (Channel master, 3.24.0-1.0.pre.324, on macOS 14.5 23F79 darwin-arm64, locale en-GB)
    • Flutter version 3.24.0-1.0.pre.324 on channel master at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9d5ede0f5a (6 hours ago), 2024-07-29 01:28:40 -0400
    • Engine revision 2cf3986cae
    • Dart version 3.6.0 (build 3.6.0-88.0.dev)
    • DevTools version 2.37.1


new version nuxt not surpport async component?

I have load editor component async like bellow:
...
async created () {
      const editor = await import('~/components/editor.vue')
      Vue.component('editor', editor)
}
...

but when I upgrade the nuxt version to ^1.0.0-rc8 ， the editor  component can't load, has some where changed?
the source code is here https://github.com/awesomes-cn/new-awesomes/blob/master/components/comment.vue#L184

Why u're not using 'asyncData` ?

@lukasborawski  I want to  async  load component, not async data

Can you try with rc11 @bimohxh ?

With rc8 & rc9, it works fine in async mounted () { ... }  method

You're using a weird syntax for async components, try doing it this way Vue documents it: https://vuejs.org/v2/guide/components.html#Async-Components.
Vue.component('async-webpack-example', function (resolve) {
  // This special require syntax will instruct Webpack to
  // automatically split your built code into bundles which
  // are loaded over Ajax requests.
  require(['./my-async-component'], resolve)
})```

I haven't tried it in RC10, but it's worked perfectly as advertised for me.

Actually, I'm also having some issues, and async components appear to be playing a role...

@Atinux  I have upgrade nuxt to 1.0.0-rc11 ( vue version is 2.4.2) but still got error
vue.runtime.esm.js:475 [Vue warn]: Failed to mount component: template or render function not defined.

found in

---> <Editor>
       <Comment> at components/comment.vue
         <Pages/repo/Owner/Name/index.vue> at pages/repo/_owner/_name/index.vue
           <Nuxt> at .nuxt/components/nuxt.vue
             <Default> at layouts/default.vue
               <Root>

@NicoPennec I try to place the code into async mounted but still not work

I fonud the error is in components/comment.vue

Don't place the code in async mounted, put it in the components hash, but with a deferred result, like this:
components: {
  editor: () => import('~/components/editor.vue')
}

@dts  Thanks, It seems work


train_test_split  with fixed random seed gives random result

The method [train_test_split| https://github.com/RasaHQ/rasa/blob/3f3c8830a99278578ea687bdfc3645d7c368e86b/rasa/shared/nlu/training_data/training_data.py#L517] has an optional random_seed parameter, which is used to seed the random shuffle applied to the training examples (in the _split_cases method within the [split_nlu_examples| https://github.com/RasaHQ/rasa/blob/3f3c8830a99278578ea687bdfc3645d7c368e86b/rasa/shared/nlu/training_data/training_data.py#L567] method) -- so if we pass in the same random seed multiple times, we expect to get the same splitting of the data.
However, instead we're observing different splits, apparently because split_nlu_examples converts self.training_examples into a [Set| https://docs.python.org/3/tutorial/datastructures.html#sets] object:  training_examples = set(self.training_examples)
and this randomly reorders the training_examples (Set objects do not preserve order). The Set object is useful for automatically removing duplicates, and for performing fast Set subtraction:  training_examples = training_examples - set(examples)
I have tried adjusting the code to avoid the conversion to a Set object, and replaced the Set subtraction with list comprehension:  training_examples = [t for t in training_examples if t not in examples]  This has the desired result (we get the same data split when running multiple times with the same random seed), but the list comprehension can take significantly longer. Is there a better solution?

Thanks for raising this issue, @ArjaanBuijk will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

Closing as stale


Problem with

Steps to Reproduce

New workfow
Try to Add a Nuget from VSTS

Expected Behavior
It works when I hit nuget.org. So I can bring in nugets for use with my workbook.
Actual Behavior
It doesn't work work when attempting to load a nuget from my VSTS Packages. I can bring packages into my Visual Studio
[Error][00:00:52.5704437] PackageManagerWindow (OnSearchTextChanged): exception: NuGet.Protocol.Core.Types.FatalProtocolException: Unable to load the service index for source https://tfsmadison.pkgs.visualstudio.com/_packaging/Dev_Architecture/nuget/v3/index.json. ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 401 (Unauthorized).
at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
at NuGet.Protocol.HttpSource.<>c__DisplayClass12_01.<<GetAsync>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at NuGet.Common.ConcurrencyUtilities.<ExecuteWithFileLockedAsync>d__21.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at NuGet.Protocol.HttpSource.d__121.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at NuGet.Protocol.ServiceIndexResourceV3Provider.<GetServiceIndexResourceV3>d__9.MoveNext()    --- End of inner exception stack trace ---    at NuGet.Protocol.ServiceIndexResourceV3Provider.<GetServiceIndexResourceV3>d__9.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at NuGet.Protocol.ServiceIndexResourceV3Provider.<TryCreate>d__8.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at NuGet.Protocol.Core.Types.SourceRepository.<GetResourceAsync>d__161.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at NuGet.Protocol.PackageSearchResourceV3Provider.d__1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at NuGet.Protocol.Core.Types.SourceRepository.d__16`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Xamarin.Interactive.NuGet.PackageManagerViewModel.d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Xamarin.Interactive.Client.Windows.Views.PackageManagerWindow.d__38.MoveNext() @ C:\vsts_work\5\s\Clients\Xamarin.Interactive.Client.Windows\Views\PackageManagerWindow.xaml.cs:128
Environment
Windows 10.
Xamarin Workbooks

Detail
Value

Version
1.4.3

Git Branch
1.4-release

Git Hash
db27525

VSTS Definition
Workbooks & Inspector (Win)

System Info

Component
Value

Windows
Microsoft Windows NT 10.0.15063.0 (64-bit)

CPU Cores
8

Physical Memory
16 GB

System-Installed Software

Component
Version

Xamarin Simulator
1010209018

VisualStudio.15.Release Components

Component
Version

Visual Studio Professional 2017
15.8.4+28010.2026

Xamarin iOS
11.14.0-13

Xamarin Android
9.0.0-19

Log File
Xamarin Workbooks 2018-09-18.log

I believe Visual Studio is using some sort of VSTS authentication plugin that lets this work more automatically from the IDE.
For Workbooks, you need to make sure your NuGet.Config file (located at %appdata%\NuGet\NuGet.Config) lists the feed and includes authentication information. These instructions for using VSTS feeds with dotnet are also applicable to Workbooks: https://docs.microsoft.com/en-us/azure/devops/artifacts/nuget/dotnet-exe?view=vsts .
I'm guessing your NuGet.Config file already lists the repos, but lacks the VSTS PAT credentials.


Inconsistencies in arbitrum forked anvil

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (9444c62 2024-07-26T00:22:19.243928000Z)
What command(s) is the bug in?
No response
Operating System
None
Describe the bug

Block's constructed on forked arbitrum constructs on L1BlockNumber.
Any block specific queries(cast --block) made are compared with the l2 block number to check if the block predates the fork block(as it should IMO).
If you run a cast run <tx> on a tx mined on anvil, it will use tx's block number's parent block number to query the state, but that block number will be L1BlockNumber, so it'll predate the fork, even though it is not. It is not querying the correct state.

# launch a forked anvil
$ anvil --fork-url https://arb1.arbitrum.io/rpc

# check the latest block number
$ cast block | grep number
<this is the l2 block number>

# mine a new block
$ cast rpc evm_mine

# check the latest block number now
$ cast block | grep number
<this is forked block's `L1BlockNumber` + 1>

I think the correct fix is all new blocks that get mined on anvil should mine off the fork l2 block number, any historical state queries should also compare l2 block number to determine if the queried block is before or after fork.
For new blocks, it needs to be determined what the L1BlockNumber should be? Should it also increment by 1 everytime? Should it stay the same? Maybe make it a function of time since forkBlockTimestamp and then make it a function of forkBlockL1BlockNumber + (elapsedTime / 12).

I think the workaround for now is you explicitly specify the chain id to a non-arbitrum chain id so the l1BlockNumber specific logic doesn't trigger

it looks like something that was overlooked in https://github.com/foundry-rs/foundry/pull/7360 though there's a test to check correct number on mine_api ...
https://github.com/foundry-rs/foundry/blob/df5f45c8c1a7e138203f1f479940f1500752b3dc/crates/anvil/tests/it/fork.rs#L1163-L1166


add builtin profiles

implements the idea suggested in https://github.com/dalibo/pg_activity/pull/387#issuecomment-1623178466

The content of built-in profiles is to be discussed. More profiles may be added, also. Suggestions welcome.


Add support for allowing all parameter types, complex or primitive, to be defined by Type Mappers.

This is a byproduct of issue #621.
Example:
public Task<HttpResponseMessage> PostRevision(
            string documentId,
            [MaybeMultipart]DocumentRevisionDto revisionDto,
            [DocumentCreationSource][NonBody]DocumentCreationSource source = null,
            string sourceId = null,
            string transactionId = null)

Here,

[MaybeMultipart] creates a custom HttpParameterBinder that reads from the body. Thus should be considered a SwaggerParameterKind.Body.
[DocumentCreationSource] creates a custom HttpParameterBinder that reads from a primitive query string parameter, source that has constraints source type enumerations (e.g. source=[word|powerpoint|excel|googledrive|onedrive])

DocumentCreationSource is a complex object that can be created from a string; hence the custom binder. According to the OpenAPI spec, you can only use a schema to describe body parameters. However, you can still describe all other parameters using many JsonSchema fields.
SwaggerGenerator currently calls GenerateAndAppendSchemaFromTypeAsync when creating body parameters. This appends the associated schema. I'm looking into a solution for parameters that may be complex types described as primitive types. An interesting find in NJsonSchema is that PrimitiveTypeMapper does not add the schema to the SchemaResolver, whereas ObjectTypeMapper does.
My hunch is that if you were to define a body type parameter using a PrimitiveTypeMapper, then you'd encounter an issue because SchemaResolver wouldn't contain a mapping for it and:
GenerateAndAppendSchemaFromTypeAsync()...
...
return new JsonSchema4 { SchemaReference = SchemaResolver.GetSchema(type, false) };

would return null or throw an exception. This last bit is most likely a non-issue since you probably shouldn't be describing your body parameter as a primitive type, albeit, I'm not sure that it's not allowed.

Continuing the last thought expressed:
OpenAPI actually does support non-referenced schema body types. They provide an example:
{
  "name": "user",
  "in": "body",
  "description": "user to add to the system",
  "required": true,
  "schema": {
    "type": "array",
    "items": {
      "type": "string"
    }
  }
}

return new JsonSchema4 { SchemaReference = SchemaResolver.GetSchema(type, false) }; will not handle this case correctly.

The fix for this is as follows: OWIN/WebAPI only.
For custom ParameterBinderAttribute implementations, you must decorate either the parameter or the ParameterBinderAttribute with a new attribute that is defined:
[AttributeUsage(AttributeTargets.Class | AttributeTargets.Parameter)]
public class WillReadBodyAttribute : Attribute
{
    public WillReadBodyAttribute(bool willReadBody = false)
    {
        WillReadBody = willReadBody;
    }

    public bool WillReadBody { get; }
}

Example: 1
[NSwag.Annotations.WillReadBody(false)]
[AttributeUsage(AttributeTargets.Class | AttributeTargets.Parameter)]
public class CustomParameterBinderAttribute : ParameterBindingAttribute {...}

OR
Example: 2
public class MyController : ApiController
{
    public string PostExample([MyParameterBinder][WillReadBody(false)]MyParameter data = null)
}

I don't get what's the practical use of the WillReadBodyAttribute the tests in the solution yield the same swagger for all body actions.


Handler

for #1
This is first step of serve swagger-ui. This PR does not fix #1.

test is failed for https://github.com/podhmo/reflect-openapi/issues/46


[feature request] Controlling docker via discord bot

Hi,
I don't know if this is possible but just a suggestion. Since it already integrate docker socket, it would be awesome if we can use discord bot to control the instance.
Following is the project that thijsvanloef contributed to control docker via discord bot. Could this project help to achieve this goal?
https://github.com/allenrkeen/server-bot?tab=readme-ov-file

Hi @chowhi123! Thank you for your suggestion! Could you give me an example of functionality that would be useful to have in my package, instead of running the package you linked separately?

First, I can't thank you enough for such an awesome project. This project provides such a huge benefit in controlling palworld docker server instance. It uses RCON to grab user presence and monitors the 8211 port to wake up the instance. If following bot function is added instead of running separately, it will provide alternative way to interact with the palworld server when RCON is broken like v0.15 patch. It will give user control back when watcher function is unable to correctly start stop the server!
Thank you again!

@chowhi123 Thank you for the kind words! I'm happy to hear something I initially made for myself also serves others well :)
I think you have a great point there! I see the value in that. I'm gonna start working on that feature, and I'll make sure to update you here when that's implemented!

Hi @chowhi123, I've just added a Discord integration! It's available in v0.1.3. For steps on how to set it up, please refer to the readme.
If you come across any issues, don't hesitate to open a new ticket!

Hi @chowhi123, I've just added a Discord integration! It's available in v0.1.3. For steps on how to set it up, please refer to the readme.
If you come across any issues, don't hesitate to open a new ticket!

So many thanks! I will check out right away!

Hi @chowhi123, I've just added a Discord integration! It's available in v0.1.3. For steps on how to set it up, please refer to the readme.
If you come across any issues, don't hesitate to open a new ticket!

So many thanks! I will check out right away!
Edit-This is working amazing!

So glad to hear! 😄


Error: invalid_client

I did all of the steps from the README.md file and replaced the variables for client ID and client secret with real values like this:
app.server.config["GOOGLE_OAUTH_CLIENT_ID"] = os.environ.get("client-id-here")
        app.server.config["GOOGLE_OAUTH_CLIENT_SECRET"] = os.environ.get("client-secret-here")

However, when I ran python app.py and opened "localhost:5000" in my browser I got the following error:
401. That’s an error.

Error: invalid_client

The OAuth client was not found.

Request Details
response_type=code
client_id=None
redirect_uri=http://localhost:5000/login/google/authorized
scope=profile email
state=1kPQ9gCwR3EkGbXV5LdnPLu1TgQEpn
access_type=offline
approval_prompt=force
That’s all we know.

I put the following into the “Authorized redirect URIs” field: http://localhost:5000/login/google/authorized
When I tried out the example from the Flask Dance Guide link you have provided, I got redirected to Google login. After I logged in I tried running app.py again and this time I could access the page because I was already authenticated.
Is there a way to fix this problem? I would like to your google example in my application, thank you.

@delaleva you can see from the error message that the client_id isn't being set properly. os.environ.get() attempts to locate an environment variable in your bash profile, so the value here should be a key rather than the client id itself.
If you want to set the config values directly (fine for testing but never do this in production), your code would need to look like this:
app.server.config["GOOGLE_OAUTH_CLIENT_ID"] = "client-id-here"
app.server.config["GOOGLE_OAUTH_CLIENT_SECRET"] = "client-secret-here"

What a silly mistake, thanks a lot.


Macro-expanded test code doesn't parse

Steps to reproduce:
cargo new foobar
cd foobar
rustup override set nightly
cargo rustc -q --tests --color=always -- -Z unstable-options --pretty=expanded | rustfmt

Here, cargo rustc generates this code: https://gist.github.com/willcrichton/2599f6f19238f95904baee8b8de285fc
And it fails with this error:
error: expected `{`, found `self`
  --> stdin:85:70
   |
85 |                                                                      self::test::assert_test_result(::__test_reexports::tests::it_works())),}];
   |                                                                      ^^^^

error: expected one of `,`, `.`, `?`, `}`, or an operator, found `self`
  --> stdin:85:70
   |
84 |                                         self::test::StaticTestFn(|| -> _
   |                                                                         - expected one of `,`, `.`, `?`, `}`, or an operator here
85 |                                                                      self::test::assert_test_result(::__test_reexports::tests::it_works())),}];
   |                                                                      ^^^^ unexpected token

I'm not sure if this is a problem with --pretty=expanded or if it's a problem with rustfmt. But it makes me sad since I can't use cargo-expand with rustfmt on code in tests :(

In general --pretty=expanded does not guarantee that its output will compile or if it does that it will have the same semantics as the input. It is also pretty buggy since it is old and poorly tested code.
Does the output parse if you try and compile it with rustc? I think it will not and thus this is a rustc problem (almost certainly with pretty expanded), rather than a rustfmt problem.

Yeah it appears that it's a rustc issue. Alas, my poor macros. I'll close this for now.


Please help to fix

see : https://github.com/twitter/compose-rules/issues/125
@myleshorton  Could you do me a favor in case to have a look at this issue by using your software? thx

Internal you see that while running lantern right?

@myleshorton so, is Lantern cause this matter? If could, please help me to fix this matter. thx


wmt15 is broken

While testing the hotfix, I tried a random other wmt release and found wmt15 to be broken:
python -c 'from datasets import load_dataset; load_dataset("wmt15", "de-en")'
Downloading: 2.91kB [00:00, 818kB/s]
Downloading: 3.02kB [00:00, 897kB/s]
Downloading: 41.1kB [00:00, 19.1MB/s]
Downloading and preparing dataset wmt15/de-en (download: Unknown size, generated: Unknown size, post-processed: Unknown size, total: Unknown size) to /home/stas/.cache/huggingface/datasets/wmt15/de-en/1.0.0/39ad5f9262a0910a8ad7028ad432731ad23fdf91f2cebbbf2ba4776b9859e87f...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/load.py", line 740, in load_dataset
    builder_instance.download_and_prepare(
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/builder.py", line 578, in download_and_prepare
    self._download_and_prepare(
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/builder.py", line 634, in _download_and_prepare
    split_generators = self._split_generators(dl_manager, **split_generators_kwargs)
  File "/home/stas/.cache/huggingface/modules/datasets_modules/datasets/wmt15/39ad5f9262a0910a8ad7028ad432731ad23fdf91f2cebbbf2ba4776b9859e87f/wmt_utils.py", line 757, in _split_generators
    downloaded_files = dl_manager.download_and_extract(urls_to_download)
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/download_manager.py", line 283, in download_and_extract
    return self.extract(self.download(url_or_urls))
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/download_manager.py", line 191, in download
    downloaded_path_or_paths = map_nested(
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/py_utils.py", line 203, in map_nested
    mapped = [
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/py_utils.py", line 204, in <listcomp>
    _single_map_nested((function, obj, types, None, True)) for obj in tqdm(iterable, disable=disable_tqdm)
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/py_utils.py", line 160, in _single_map_nested
    mapped = [_single_map_nested((function, v, types, None, True)) for v in pbar]
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/py_utils.py", line 160, in <listcomp>
    mapped = [_single_map_nested((function, v, types, None, True)) for v in pbar]
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/py_utils.py", line 142, in _single_map_nested
    return function(data_struct)
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/download_manager.py", line 214, in _download
    return cached_path(url_or_filename, download_config=download_config)
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/file_utils.py", line 274, in cached_path
    output_path = get_from_cache(
  File "/home/stas/anaconda3/envs/main-38/lib/python3.8/site-packages/datasets/utils/file_utils.py", line 614, in get_from_cache
    raise FileNotFoundError("Couldn't find file at {}".format(url))
FileNotFoundError: Couldn't find file at https://huggingface.co/datasets/wmt/wmt15/resolve/main/training-parallel-nc-v10.tgz

It's reachable for the viewer and me, so I suppose it was down at that moment?


fix(workflow): semantic-release

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

v0.1.2
GitHub release

Your semantic-release bot :package::rocket:


[DXVK] Visual artefacts in Assassins Creed: Odyssey

Visual artefacts in Assassins Creed: Odyssey using DXVK 1.7.1 and AMDVLK 2020.Q3.5:
GPU: R9 Fury
There is no issue when using the mesa radv driver, only when using amdvlk.

Not sure if this is any help, but I might as well mention it. I just upgraded to AMDVLK 2020.Q4.1 and the artefacts look different now. Those shown in the screenshot above are gone, and instead there are bright flashing colours.

thanks for the update

Tested new release (AMDVLK 2021.Q1.1) and it now renders correctly, at least on my machine.
This issue can probably be closed now, but I'll leave it to you AMD-folks in case you want to do your own testing first.

Tested new release (AMDVLK 2021.Q1.1) and it now renders correctly, at least on my machine.
This issue can probably be closed now, but I'll leave it to you AMD-folks in case you want to do your own testing first.


winsetview: also create shim for WinSetView.ps1

This script is needed to apply your preset from poweshell.

@soredake Thank you for info. The shim for WinSetView.ps1 will be created now.


Restrict index calculation by (legal) mode access

By default, an index value was always created for segments that are accessible by at least one of the modes (car, bike, walk). This lead to misinterpretation of visual outputs when no filter was applied in post-processing. Therefore, the settings file should allow for restricting index computation to segments that are (legally) accessible per mode. However, users should still be able to compute a theoretical index for segments with restricted access, making it more versatile for specific applications in planning.

Solution: When referencing a mode profile in the settings file, the following parameters can be specified:

filter_access_car: True
filter_access_bike: True
filter_access_walk: True
A full example could look like this:

profiles:
  -
    profile_name: bike
    filename: profile_bike.yml
    filter_access_bike: True

Combining filters is also supported:
profiles:
  -
    profile_name: bike_incwalk
    filename: profile_bike.yml
    filter_access_bike: True
    filter_access_walk: True

In this case, a "bike" index is computed for all segments that are accessible by bike as well as segments accessible for pedestrians (with restricted bike access).
If no filter parameters are specified, an index is computed for all segments accessible to at least one of the main modes (car, bike, walk) - thereby providing consistency with the legacy versions of NetAScore.

implemented in https://github.com/plus-mobilitylab/netascore/commit/b4a72538314e303f8869f0ae1297238e2807b859


Modsecurity V3.0.4 + Nginx: Phase 3/4 Not Blocking

Hi,
We are having an issue in Phase 3/4 rules which are detected but not blocked.
I found this issue similar with https://github.com/SpiderLabs/ModSecurity/issues/1568, however in my case I'm able to block in phase 1 and phase 2. I used a workaround to redirect to a static page hosted internally.
Tested both with action as redirect, deny, drop, but the result was the same.
SecDefaultAction "phase:4,log,auditlog,redirect:/403.html"
SecDefaultAction "phase:4,log,auditlog,deny,status:403"
As can be seen in the attached logs, multiple disruptive rules with action deny/drop/block are detected in the same transaction.
Example: 953120, 980140
logs.txt
Have anyone faced this issue and any idea how to fix it?
Thanks in advance,

Hi @s-ribeiro,
What is the version of ModSecurity?
Notice that if you are using nginx, redirection on phase 3/4 is not feasible. At this point the header already hit the user browser, the only option is to terminate the request abruptly.

Hi @zimmerle,
Thanks for your feedback. I am running 3.0.4.
I do understand your point regarding phase4, the response headers were already sent, so redirect will not work. My bad... :)
But I have the same problem in phase3 rules as well.
As per your suggestion I've changed SecDefaultAction in phase 3 and 4 to a disruptive one (example: drop) and the result was the same.
SecDefaultAction "phase:1,log,auditlog,redirect:/403.html" SecDefaultAction "phase:2,log,auditlog,redirect:/403.html" SecDefaultAction "phase:3,drop" SecDefaultAction "phase:4,drop"
 [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (non-disruptive) action: tag [159489610275.514024] [/1.html] [4] Running (disruptive)     action: block. [159489610275.514024] [/1.html] [4] Starting phase LOGGING. (SecRules 5) [159489610275.514024] [/1.html] [4] (Rule: 912110) Executing operator "Eq" with param "0" against TX:dos_burst_time_slice. [159489610275.514024] [/1.html] [4] Rule returned 0.
Then I've changed in the rule itself the action to drop instead of instead block (inheritance from SecDefaultAction).
[159489653434.461145] [/1.html] [4] Running (non-disruptive) action: tag [159489653434.461145] [/1.html] [4] Running (non-disruptive) action: tag [159489653434.461145] [/1.html] [4] Running (non-disruptive) action: tag [159489653434.461145] [/1.html] [4] Running (non-disruptive) action: tag [159489653434.461145] [/1.html] [4] Running (non-disruptive) action: tag [159489653434.461145] [/1.html] [4] Running (non-disruptive) action: tag [159489653434.461145] [/1.html] [4] Running (disruptive)     action: drop. [159489653434.461145] [/1.html] [4] Starting phase LOGGING. (SecRules 5)
But the connection is either dropped.

Hello.
I have the exact same problem. Has there been any development on this ?
I'm also using version 3.0.4 with nginx.
Is there some workaround ?
Thanks,
Ricardo Ferreira

Hi,
That is a limitation on nginx. At this phase is not possible to redirect the user any longer. The reason for that is the fact that nginx already delivered the response headers to the user. At this stage only drop is possible.


Explicit latitude/longitude settings for weather forecast lookup

If the geocoder API is over limit (see #54) you get an error rather than a forecast.
Handle the case where you know exactly where you are already, and pass in the latitude and longitude directly to the API, e.g. support queries like so:
weather --latitude 42.27756 --longitude -83.74088

which will give you Ann Arbor, MI, or the same output more or less as
weather -l "Ann Arbor, MI"

I have a work in progress to do this - the only small hitch so far is that the output routine counts on there being a nice name for the location.

Closing as wontfix, not a bad idea but not necessary.


Fix IMC Load Error

IMC would not load because machine did not have the attribute
is_shutting_down.  This checks for the attirbute and its value of true
in one if statement now.

Thanks! We really need a unit test for this. This broke quite a few times over the last years.

That was my next question.  I will write one up.  How do I do one for IMC?  Is that in the MC code?  Any specific things I need to plug into to get IMC to run, versus just the full MC in tests?

Yeah that would be in mc. Mocking all the qt parts might be a bit tricky.


Metadata issue?

So i reg:  🐵in.eth.   Somehow the 🐵 emoji looks different in my wallet than it does on OS. Even different than the other similar ones on OS. Now, i understand it works, but i need it to look like all the other 🐵 out there. PLEASE HELP!!

Here is the actual NFT overlay image from the metadata service: https://metadata.ens.domains/mainnet/0x57f1887a8BF19b14fC0dF6Fd9B2acc9Af147eA85/9909794367337237191412196562337992769890037602576462996264948860142860167855/image
For example if you deselect that font in the SVG then it displays using the browser default

So, I'm not sure anything can really be done here, I believe that's just how the font is, that's how it displays that character.
Or, do you have an example of a different ENS name with that character that displays differently in OpenSea? It might be using the old overlay metadata image...

I few examples of what i exoected to see.

I did see one oher like mine.

What it looked like when i bought it.

I did this on my phone with the regular samsung keyboard. Was i the prob? Was it my equ? Was it the mint code? Def dont want this to happen again.

Here's how that name looks in Brave:

So it's the same.

And then, I see this one which looks different:

However, I just refreshed the metadata on OpenSea, and now that name looks like this:

So I think something changed on the metadata service side?

Yooooooooo. That one aint me????

Oh, you mean fir all them?

Thx so much for your help

Here are the two SVG files, old and new, for you to compare:

Now, if I take the new SVG, and just change that one line to remove DejaVu Sans, here's what I get:

Wow, ok,  can you do that for mine?

Yes but it won't matter, it's in the metadata service, it'll have to be fixed there, you won't be able to take this SVG and use it on OpenSea because OS pulls from the ENS metadata service

So its already imbedded

Gotcha.

Hopfully maybe they can change that in the future cause the ones like mine are kinda ugly. I know its not a big deal, just a little dissapointing.

Hey hey! The reason is; in metadata service we force emojis to have exactly the same look for all the OS's, unfortunately using Apple Color Emoji (which MacOS and iOS uses) or Noto Color Emoji (which Google Chrome and Android uses) does not help alone, Apple does not allow using Noto Color Emoji on their OS and Apple Color Emoji is not available for commercial use. And they both have different set of emoji renders, that's why when people were checking their emoji names they were seeing different renders, not only emoji renders but also font size differences.
You can see the differences of emojis here if you scoll down: https://emojipedia.org/monkey-face/
The only way to have them same is forcing both to a font which is working on both systems. And it's not easy to find such font cuz then on Linux side things getting more strange. The best shot was DejaVu Sans for all available OS's.
Why this matters? The biggest issue with previous version was unpredictable character length of emojis, it was causing many problems when we try to calculate the font-size to fit the name properly. I am aware that DeJaVu renders some emojis not as emoji but as a b/w symbols. We need to experiment with some modern alternatives to overcome this issue in the future. Then I hope that everything will be settled down for all the emojis.

Awesome explanation, thx so much.
-------- Original message --------From: Muhammed Tanrıkulu @.> Date: 3/24/22  1:23 PM  (GMT-05:00) To: ensdomains/ens-metadata-service @.> Cc: petieboy128 @.>, Author @.> Subject: Re: [ensdomains/ens-metadata-service] Metadata issue? (Issue #80)
Hey hey! The reason is; in metadata service we force emojis to have exactly the same look for all the OS's, unfortunately using Apple Color Emoji (which MacOS and iOS uses) or Noto Color Emoji (which Google Chrome and Android uses) does not help alone, Apple does not allow using Noto Color Emoji on their OS and Apple Color Emoji is not available for commercial use. And they both have different set of emoji renders, that's why when people were checking their emoji names they were seeing different renders, not only emoji renders but also font size differences.
You can see the differences of emojis here if you scoll down: https://emojipedia.org/monkey-face/
The only way to have them same is forcing both to a font which is working on both systems. And it's not easy to find such font cuz then on Linux side things getting more strange. The best shot was DejaVu Sans for all available OS's.
Why this matters? The biggest issue with previous version was unpredictable character length of emojis, it was causing many problems when we try to calculate the font-size to fit the name properly. I am aware that DeJaVu renders some emojis not as emoji but as a b/w symbols. We need to experiment with some modern alternatives to overcome this issue in the future. Then I hope that everything will be settled down for all the emojis.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Dimmed theme

Summary
After releasing our dark mode beta at Universe 2020, the feedback from our community to create a theme with less contrast was loud and clear. The dimmed theme will join dark mode as a new theme designed to reduce the contrast of the background and foreground elements in GitHub.com.
Intended Outcome
Whether as a personal preference or as a way to reduce contrast and strain on the eye for those who experience light sensitivity, GitHub's dimmed theme gives developers more choice and flexibility in how and when they use GitHub.
How will it work?
From the appearance settings page, a user is able to choose the dimmed theme as their preferred light, dark or default to system color mode.

Shipped! 🚢   https://github.blog/changelog/2021-04-14-dark-and-dimmed-themes-are-now-generally-available/


Added aw-studio/docdress

A package to deploy markdown documentations from GitHub repositories in your Laravel project.

Build with Laravel and Tailwind CSS

Looks nice!

Thanks, I am going to add Algolia search support next week.

Cool!


Cache unique key constraint when running model with same prompt on multiple inputs

When running same request in parallel for the first time we get a cache miss and attempt to create as many cache entries. Instead of surfacing the error we should ignore it since the cache is already there as per cache entry unique key violation.

Solved by https://github.com/spolu/dust/pull/15


devices/lepton: refactor to remove memory allocation

Based on discussions with @mjs the driver should stop allocating memory, which requires change in the API https://periph.io/x/periph/devices/lepton#Dev.ReadImg

Done


Error Cannot read property 'selectedControl' of undefined

Hi!
We have the error Cannot read property 'selectedControl' of undefined with the 2.1.1 version, when trying to open the control modal

I am also getting this error, however it seems to work fine in the stepway demo provided with the npm module, so I am trying to debug now

Could try with minified version?
Demo uses min version.

using minified
TypeError: Cannot read property 'selectedControl' of undefined
    at e.value

I don't understand...
It should bug here but it doesn't (even no minified version).
I can't reproduce this bug.

I guess it is a matter of dependencies?

I'll experiment a bit with my dependencies and see if i can fix it or get a different error.

Thanks for help.
Searching this bug (still not found) I fixed another bug with basic select while editing in modal.
This bug prevented from manipulating options (in the table: change order, or delete...).

@MacKentoch  I added the steps to reproducing this error

@MacKentoch I can confirm that I got the same issue in the demo following these steps.

v2.1.2 fixes #64 but this #63 I don't reproduce it.
What angular js version do you use for example?

@MacKentoch I used this repo with the version of the package.json

@MacKentoch have you tried to reproduce with the new steps provided? Perhaps when you made your fixes for v2.1.2, it was fixed?


Tweet Entities having duplicates removed, not counting different indices as different entities

Tweetinvi removes duplicate entities in a Tweet without considering their entities indices.
So if a tweet mentions the same user twice, that will only show once in Tweetinvi.
Can be seen in action with:
var tweetJson = TwitterAccessor.ExecuteJsonGETQuery("https://api.twitter.com/1.1/statuses/show.json?id=656821186014281729");
ITweet tweet = Tweet.GetTweet(656821186014281729);

At a glance, it looks like TweetEntities.InitialiseEntities() calls Distinct() against each collection of entities, but the Equals() method for each entity are ignoring indices, e.g. UserMentionEntity.Equals:
public bool Equals(IUserMentionEntity other)
{
  if (!this.Id.HasValue)
    return false;
  long? id1 = this.Id;
  long? id2 = other.Id;
  if (id1.GetValueOrDefault() != id2.GetValueOrDefault())
    return false;
  return id1.HasValue == id2.HasValue;
}

This is only really an issue for Hashtags & Mentions, where the otherwise same entity could exist at multiple positions in the tweet.

Again thanks for reporting it and for investigating the issue. It will be part of Tweetinvi <IP_ADDRESS>.
Cheers,
Linvi

No worries, thanks for being so quick & fixing these little niggles!
Cheers,
Josh


Unexpected identifier TF

TF version v.12.9
azure rm
running on windows 10.
I cant see how to witch debug mode on tried --debug, but if you can let me know I'll try again to get more info.
<IP_ADDRESS> - - [13/Oct/2019 16:06:46] "GET / HTTP/1.1" 200 -
<IP_ADDRESS> - - [13/Oct/2019 16:06:49] "GET /graph.svg HTTP/1.1" 200 -
[2019-10-13 16:06:50,729] ERROR in app: Exception on /graph.json [GET]
Traceback (most recent call last):
File "C:\Python37\lib\site-packages\flask\app.py", line 2446, in wsgi_app
response = self.full_dispatch_request()
File "C:\Python37\lib\site-packages\flask\app.py", line 1951, in full_dispatch_request
rv = self.handle_user_exception(e)
File "C:\Python37\lib\site-packages\flask\app.py", line 1820, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "C:\Python37\lib\site-packages\flask_compat.py", line 39, in reraise
raise value
File "C:\Python37\lib\site-packages\flask\app.py", line 1949, in full_dispatch_request
rv = self.dispatch_request()
File "C:\Python37\lib\site-packages\flask\app.py", line 1935, in dispatch_request
return self.view_functionsrule.endpoint
File "C:\Python37\lib\site-packages\blastradius\server\server.py", line 61, in graph_json
tf = Terraform(os.getcwd())
File "C:\Python37\lib\site-packages\blastradius\handlers\terraform.py", line 26, in init
self.config = hcl.load(config_io)
File "C:\Python37\lib\site-packages\hcl\api.py", line 54, in load
return loads(fp.read())
File "C:\Python37\lib\site-packages\hcl\api.py", line 66, in loads
return HclParser().parse(s)
File "C:\Python37\lib\site-packages\hcl\parser.py", line 326, in parse
return self.yacc.parse(s, lexer=Lexer())
File "C:\Python37\lib\site-packages\ply\yacc.py", line 333, in parse
return self.parseopt_notrack(input, lexer, debug, tracking, tokenfunc)
File "C:\Python37\lib\site-packages\ply\yacc.py", line 1201, in parseopt_notrack
tok = call_errorfunc(self.errorfunc, errtoken, self)
File "C:\Python37\lib\site-packages\ply\yacc.py", line 192, in call_errorfunc
r = errorfunc(token)
File "C:\Python37\lib\site-packages\hcl\parser.py", line 318, in p_error
raise ValueError(msg)
ValueError: Line 55, column 1607: unexpected IDENTIFIER
<IP_ADDRESS> - - [13/Oct/2019 16:06:50] "GET /graph.json HTTP/1.1" 500 -

I also get this error trying to generate the json for embedded use with TF v0.12.13.