fix: validate user config

Signed-off-by: Bing Han<EMAIL_ADDRESS>Move the cfg.Validate below to validate user config. Without this fix, the test will fail:
--- FAIL: TestValidateUserConfig (0.00s)
    server_test.go:33:
        	Error Trace:	/Users/hanbing/repo/go/envoyproxy-gateway/internal/cmd/server_test.go:33
        	Error:      	An error is expected but got nil.
        	Test:       	TestValidateUserConfig
FAIL
FAIL	github.com/envoyproxy/gateway/internal/cmd	0.601s
FAIL

Codecov Report

Merging #1028 (de5e904) into main (5115330) will increase coverage by 0.13%.
The diff coverage is 66.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1028      +/-   ##
==========================================
+ Coverage   62.03%   62.17%   +0.13%
==========================================
  Files          55       55
  Lines        8804     8801       -3
==========================================
+ Hits         5462     5472      +10
+ Misses       2967     2952      -15
- Partials      375      377       +2

Impacted Files
Coverage Δ

internal/cmd/certgen.go
12.82% <0.00%> (ø)

internal/cmd/server.go
17.05% <80.00%> (+10.07%)
:arrow_up:

internal/xds/translator/route.go
81.08% <0.00%> (-1.53%)
:arrow_down:

internal/gatewayapi/route.go
85.12% <0.00%> (-0.11%)
:arrow_down:

internal/gatewayapi/listener.go
100.00% <0.00%> (ø)

internal/provider/kubernetes/controller.go
50.33% <0.00%> (+0.11%)
:arrow_up:

internal/xds/translator/authentication.go
63.78% <0.00%> (+0.51%)
:arrow_up:

internal/provider/kubernetes/helpers.go
81.75% <0.00%> (+0.67%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

LGTM, thanks !


tentative roadmap

Codecov Report

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

@@           Coverage Diff           @@
##           master      #48   +/-   ##
=======================================
  Coverage   81.85%   81.85%
=======================================
  Files          16       16
  Lines        2866     2866
=======================================
  Hits         2346     2346
  Misses        520      520

Continue to review full report at Codecov.

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

Looks good! I just wonder about the version numbers, since, if we assume semantic versioning, having them all 1.x means that they keep full backward compatibility. Rust and Go took their time to reach 1.0 and start giving compatibility promises — I wonder if we shouldn't do the same.

Yep, the version numbers themselves are a big part of the "tentative" part of the roadmap. They can be a bunch of v0.x, v0.(x+1), v0.(x+2) etc., or v1.0, v1.1, v2.0, v2.1, etc. languages usually take the first route and move slowly once they go 1.0, there are even languages like Elm that are already pretty popular and used in production and still at 0.x (Elm is at 0.18). The counterexample is probably Swift, which is already at Swift 4 after just three years from Swift 1.0.

I think we can be "conservative" at version numbers and use 0.x as we build up the language.

The counterexample is probably Swift, which is already at Swift 4 after just three years from Swift 1.0.

One has to take into account that Swift was an internal project at Apple for 4 years before the announcement, so they had their long 0.x road as well. :)

I think we can be "conservative" at version numbers and use 0.x as we build up the language.

Ok, but we do not have to go with a public release as v0.1, as a lot of the work that we have already done (and have to do to get this first release ready) counts as point releases, in my view. I pushed some tentative tags on what I thought were important milestones so far, and updated the roadmap accordingly.

There is a lot of work to put into each release of these milestones listed in the roadmap. Each of these bullets is a lot more functionality than the delta added in the v0.1-v0.5 you tagged (I'm not even sure these should have been marked as versions). Having feature targets for numbered releases adds a lot of anxiety. Let's bump 0.x numbers freely as we add user-visible features.
I suggest we either:

remove the version numbers from the README altogether
remove version numbers from the bullets and write it so that the language at the end of that roadmap will be Titan 1.0.
make each bullet there a major x.0 version.

A for the current 0.x tags, let's leave them as is, but let's add a CHANGELOG.md file listing what's new/changed in each of these versions.

There is a lot of work to put into each release of these milestones listed in the roadmap. Each of these bullets is a lot more functionality than the delta added in the v0.1-v0.5 you tagged (I'm not even sure these should have been marked as versions). Having feature targets for numbered releases adds a lot of anxiety. Let's bump 0.x numbers freely as we add user-visible features.

The tags are just a first suggestion, they can be changed freely. Could just as well be just 0.1 (the state of the repository at the time of the Lua workshop) and 0.2 (strings, basically the scope of my original "v0" email), modules can be 0.3, with the first public release in the roadmap 0.5. I just did not want to tag that first release as 0.1, as we want it to be a basic but quite usable language.

remove version numbers from the bullets and write it so that the language at the end of that roadmap will be Titan 1.0

In all, this is probably the best suggestion. We can tag the first public release with an arbitrary v0.X number, and keep bumping it freely on the way to 1.0 when adding new features.

In all, this is probably the best suggestion. We can tag the first public release with an arbitrary v0.X number, and keep bumping it freely on the way to 1.0 when adding new features.

+1!

@mascarenhas Please check my review of the roadmap — I made it focused on the feature order towards 1.0, not version release milestones.

It would be create to have a change log where we can check which features were included or removed from one version to another.

@andremm yes, that would be a welcome addition too. I'd just like to check with @mascarenhas if the roadmap is good to merge as is before hitting the green button. :)


Huge memory consumption in batch jobs on 3D variables

The drift calculation code works fine on jasmin notebooks. However, it returns memory issues on lotus batch jobs. For some reason, the code starts to consume a lot of memory even though it does not require it. The same code works fine for 2D vars. There is even performance improvement with dask-mpi. This needs investigation.

Observation: Rechunking within the code  leads to memory blow-up
Specifying chunks while reading data (as below) works fine
ds1 = xr.open_dataset(file, chunks={'time':1})
If rechecking is performed within the code (e.g. as below), then the Dask-mpi fails with "out_of_memory" error
ds1 = ds1.chunk({"time": 1, "lev":10, "x":-1, "y":-1})
Cause is not clear. Needs investigation.


Our HAR files don't load in the Chrome dev tools

While working on https://github.com/tweaselORG/docs.tweasel.org/issues/7, I noticed that the HAR files we create (via mitmproxy's har_dump) don't load in Chrome.
Example HAR (renamed to .json, because GitHub doesn't allow .har uploads): be.stib.mivb.mobile.har.json
If I try to load that in the Chrome network tab, I get:

Failed to load HAR file with following error: Invalid date format

I am running: Chromium Version 123.0.6312.122 (Official Build) snap (64-bit)

This was fixed in mitmproxy 10 (we depend on v9): https://github.com/mitmproxy/mitmproxy/pull/5814
Indeed, with just an empty pages array, it works in both Firefox and Chrome.

Since mitmproxy 10.1 brings native HAR export anyway, I guess it makes most sense to defer this issue to #36.

I've changed my mind. This is quite an unfortunate problem and I would really rather not keep producing incorrect HAR files.
While I do indeed not have the time to properly test upgrading to a new mitmproxy version right now, this is easy enough to hotfix on our end.


docs: update minimum Node version

Node version 16.15.1 gives "contexts.at is not a function" error

Thanks!


fog cannot be loaded in 0.0.27

Hello,
I updated vagrant-libvirt to 0.0.27 this morning and am now greeted with:
/home/babilen/.vagrant.d/gems/gems/vagrant-libvirt-0.0.27/lib/vagrant-libvirt/action/connect_libvirt.rb:1:in `require': cannot load such file -- fog (LoadError)
	from /home/babilen/.vagrant.d/gems/gems/vagrant-libvirt-0.0.27/lib/vagrant-libvirt/action/connect_libvirt.rb:1:in `<top (required)>'
	from /home/babilen/.vagrant.d/gems/gems/vagrant-libvirt-0.0.27/lib/vagrant-libvirt/action.rb:273:in `block in action_read_state'
	from /home/babilen/.vagrant.d/gems/gems/vagrant-libvirt-0.0.27/lib/vagrant-libvirt/action.rb:271:in `tap'
	from /home/babilen/.vagrant.d/gems/gems/vagrant-libvirt-0.0.27/lib/vagrant-libvirt/action.rb:271:in `action_read_state'
	from /home/babilen/.vagrant.d/gems/gems/vagrant-libvirt-0.0.27/lib/vagrant-libvirt/provider.rb:21:in `action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/machine.rb:180:in `block in action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/environment.rb:516:in `lock'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/machine.rb:178:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/machine.rb:178:in `action'
	from /home/wwentland/.vagrant.d/gems/gems/vagrant-libvirt-0.0.27/lib/vagrant-libvirt/provider.rb:70:in `state'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/machine.rb:480:in `state'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/machine.rb:141:in `initialize'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/vagrantfile.rb:75:in `new'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/vagrantfile.rb:75:in `machine'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/environment.rb:614:in `machine'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/plugin/v2/command.rb:168:in `block in with_target_vms'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/plugin/v2/command.rb:201:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/plugin/v2/command.rb:201:in `block in with_target_vms'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/plugin/v2/command.rb:200:in `map'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/plugin/v2/command.rb:200:in `with_target_vms'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/plugins/commands/status/command.rb:20:in `execute'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/cli.rb:42:in `execute'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/environment.rb:301:in `cli'
	from /opt/vagrant/bin/../embedded/gems/gems/vagrant-1.7.2/bin/vagrant:174:in `<main>'

This might be due to 749b431bc85a6a94dbda77e95e5e005b335953a7, but I can't be sure of that as ~/.vagrant.d/gems/gems/fog-libvirt-0.0.1 can be found on the system.
I believe that https://github.com/pradels/vagrant-libvirt/blob/master/lib/vagrant-libvirt/action/connect_libvirt.rb#L1 should be require 'fog/libvirt' rather than require 'fog'.Could that be the case?

I'm facing same issue, since update. :-(

Same problem here. Manually updating connect_libvirt.rb as @babilen suggests gets it working again for me.

thanks. fix added.


Extension TreeView and Configuration issues after upgrade

VS Code Version: 1.91.1
OS Version: macOS 14.2

My extension has registered a treeview. After releasing a new version, users upgraded the extension, but the corresponding treeview did not successfully update and prompted "no view is registered with id." Additionally, in some cases, some users were unable to adjust newly added configurations after upgrading the extension.

Can you please provide steps to reproduce?


Incorrect value returned for aws_ec2_ami table

Describe the bug
Trying to find all AMI's that exist in a particular AWS account, using the "self" attribute. Per the docs:  https://hub.steampipe.io/plugins/turbot/aws/tables/aws_ec2_ami
image_owner_alias | text | The AWS account alias (for example, amazon, self) or the AWS account ID of the AMI owner.

However the actual results are returning <null> instead of self
Steampipe version (steampipe -v)
v0.10.0
Plugin version (steampipe plugin list)
v0.42.2
To reproduce

Create an ami owned by the local account
query:  sql select title, image_owner_alias from aws_ec2_ami where public = 'false'

Expected behavior
Should return a list of local amis with any owned by the local account having self in image_owner_alias
Additional context
https://steampipe.slack.com/archives/C01UECB59A7/p1639956521079700

@dboeke tanks for testing this out. We are looking into this.


Better username evaluation in get_user_obj

If Cognito was instantiated with an access token and without a username then get_user will return a user without a username property:
c = Cognito(user_pool, client_id, access_token)
u = c.get_user()
print(u.username) # prints None

This PR allows get_user_obj to check if username is not set and if so try to set it to self.username or the Username from attribute_list if it exists.
Order of preference for what username should be:

passed in username
self.username
Username from attribute_list

@ludeeus This is a bug I believe and this change should address it. Should I start opening issues to link to in PRs?


Add currentTime prop

This allows you to override the time set by visjs.
Works in the same way as specified here: http://visjs.org/docs/timeline/#Methods

I've also added support for groups, taken from #3

Awesome, thanks for this @ldabiralai. Does this handle groups optionally, i.e. if you just pass items?

Thanks for taking a look @willmcclellan 😀
There's actually an issue at the moment with my implementation, where we default to [] for groups, but instead we should default to null as visjs won't display any items if they don't map to a group when group is [], here https://github.com/almende/vis/blob/master/lib/timeline/Timeline.js#L38
I'll go ahead and update this, so the defaultProp is null (just to be verbose about it).
Feel free to push back if you don't think this is the way to do it.

Looks great, thanks @ldabiralai! 👍


Question: faster listview?

Currently, when using lists, the whole list is generated at once, aka all of its elements are added to the tree, which may cause issues if there are a lot of elements, so my question:
Is there a UI element that allows me to create a list that only shows elements that are currently on-screen? Afaik, UI toolkit default ListView already have this functionality, so would it be possible to have a similar feature here? Maybe as a separate component even

The functionality of UIToolkit's ListView only works when each element has the same height.
RosettaUI does not use this feature because it works at different heights.
ScrollView also has the same problem , so I will try to solve it eventually.

RosettaUI.UIToolkit 1.1.0 now supports list item virtualization🎉
ScrollView is not supported at this time because it would be too much work.

Cool! Are there examples of it?

UI.List() will always have this behavior.

It supports elements of different sizes!
However, UIToolkit's ListView is buggy, so I recommend using the latest version of Unity 2021 or 2022.

Great! This is excellent, then. Does it also means that RosettaUI now supports unity 2022?

Yes:thumbsup:


同步社区

NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute

同步社区，更新fork

同步社区，更新fork


Getupdate process killed

After a few hours when my bot has no traffic, its process killed.
this is my my code:
require 'daemons'
require 'telegrammer'
require_relative 'aparat'

class AparatMe
  attr_accessor :bot, :aparat
  def initialize
    p 'INIT'
    @bot = Telegrammer::Bot.new('XXX')
    @aparat = Aparat.new
  end

  def process_message_and_send
    @bot.get_updates do |message|
      puts "In chat #{message.chat.id}, @#{message.from.username} said: #{message.text}"
      rec_message = message.text
      if (!rec_message) || (rec_message.strip! == '')
        p 'ERROR'
      else
        begin
          if rec_message[0..5] == '/start'
            @bot.send_message(chat_id: message.chat.id, text: "Please send a keyword with /video for example: /video dog")
          elsif rec_message[0..6] == '/video '
            result = @aparat.video_by_search(rec_message[7..-1])
            @bot.send_message(chat_id: message.chat.id, text: "First item of search: #{result}")
          else
            # @bot.send_message(chat_id: message.chat.id, text: "Please send your keyword with /video command.")
          end
        rescue Exception => e
          p e.message
        end
      end
    end
  end
end

Process.daemon(true)
aparat_me = AparatMe.new
aparat_me.process_message_and_send

Yeah I faced this too. Have no idea as well

I implemented telegram api in my app and this problem solved. I will publish my application source code in github soon.

@abmahmoodi what was the issue and how did you fixed it?

This is a version of telegram API implemented by myself:
class  TelegramBot
  attr_accessor :offset, :token

  END_POINT = 'https://api.telegram.org'

  def initialize(token)
    @offset = 0
    @token = token
  end

  def api_response(response)
    if response.code < 500
      @body = response.response_body

      data = MultiJson.load(@body)
      @success = data["ok"]

      if @success
        @result = data["result"][0]
      else
        false
      end
    else
      # TODO: Raise an exception
    end
  end

  def get_updates(&block)
    loop do
      request = run_api('getUpdates', {offset: @offset, timeout: 60})
      begin
        res = api_response(request)
        if (res != nil) and (res != false)
          message = Message.new(res)
          @offset = message.update_id + 1
          yield message
        end
      rescue Exception => e
        p e.message
      end
    end
  end

  def send_message(chat_id:, text:)
    run_api('sendMessage', {chat_id: chat_id, text: text})
  end

  private
  def run_api(api_method, params)
    Typhoeus.post(
        "#{END_POINT}/bot#{@token}/#{api_method}",
        headers: {
                "User-Agent" => "AparatBot/0.1.0",
                "Accept" => "application/json"
              },
        body: params
      )
  end
end

I think there is an unhandled exception in Telegrammer.

@abmahmoodi @kennetham  what exception are you getting?
@kennetham I suspect that this is something related with having long polling sessions without receiving data. Maybe for these situations Typhoeus is not the best option for this situation.
I've been very busy these days, but I have in mind to migrate to something like Faraday and test with some HTTP clients that explicitly support persistent connections, like Net::HTTP::Persistent, Patron or HTTPClient.

@mayoral I can't remember the error, but the long polling crashed as @abmahmoodi faced. No worries on that.
https://github.com/shouya/telegram-bot
I think it would be great if some of these components can be used in Telegrammer. It's very easy to call commands etc, and it looks cleaner. Otherwise it's going to be a long switch case of commands.

@abmahmoodi @kennetham I've just uploaded version 0.3.0 of the gem. Now it uses HTTPClient to do the requests, so it'll use persistent connections if the endpoint supports it.
Hopefully this will solve the crashes you are having with get_updates after a long time.
Please test it and close the issue if the problem is solved for you.

Closing for now. If the problem persist we'll reopen it again.


Electron 15: UnhandledPromiseRejectionWarning in extensions/emmet/test-workspace integration tests

Steps to Reproduce:
1) git co electron-16.x.y
2) yarn && yarn watch
3) scripts/test-integration.sh

(node:10567) UnhandledPromiseRejectionWarning: Canceled: Canceled
    at canceled (/Users/demohan/github/vscode/out/vs/base/common/errors.js:109:23)
    at Delayer.cancel (/Users/demohan/github/vscode/out/vs/base/common/async.js:247:57)
    at Delayer.dispose (/Users/demohan/github/vscode/out/vs/base/common/async.js:259:18)
    at ThrottledDelayer.dispose (/Users/demohan/github/vscode/out/vs/base/common/async.js:287:26)
    at dispose (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:99:27)
    at DisposableStore.clear (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:160:17)
    at DisposableStore.dispose (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:153:18)
    at FileWatcher.dispose (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:195:25)
    at FileWatcher.dispose (/Users/demohan/github/vscode/out/vs/platform/files/node/watcher/nodejs/watcherService.js:103:19)
    at dispose (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:99:27)
    at DisposableStore.clear (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:160:17)
    at DisposableStore.dispose (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:153:18)
    at dispose (/Users/demohan/github/vscode/out/vs/base/common/lifecycle.js:115:17)
    at DiskFileSystemProviderChannel.unwatch (/Users/demohan/github/vscode/out/vs/platform/files/node/diskFileSystemProviderIpc.js:158:41)
    at DiskFileSystemProviderChannel.call (/Users/demohan/github/vscode/out/vs/platform/files/node/diskFileSystemProviderIpc.js:39:45)
    at ChannelServer.onPromise (/Users/demohan/github/vscode/out/vs/base/parts/ipc/common/ipc.js:259:35)
    at ChannelServer.onRawMessage (/Users/demohan/github/vscode/out/vs/base/parts/ipc/common/ipc.js:232:33)
    at /Users/demohan/github/vscode/out/vs/base/parts/ipc/common/ipc.js:177:73
    at Emitter.fire (/Users/demohan/github/vscode/out/vs/base/common/event.js:435:38)
    at /Users/demohan/github/vscode/out/vs/base/common/event.js:43:93
    at Emitter.fire (/Users/demohan/github/vscode/out/vs/base/common/event.js:435:38)
    at /Users/demohan/github/vscode/out/vs/base/common/event.js:54:106
    at Emitter.fire (/Users/demohan/github/vscode/out/vs/base/common/event.js:435:38)
    at IpcMainImpl.fn (/Users/demohan/github/vscode/out/vs/base/common/event.js:237:44)
    at IpcMainImpl.emit (node:events:394:28)
    at Object.<anonymous> (node:electron/js2c/browser_init:161:9959)
    at Object.emit (node:events:394:28)
(Use `Electron --trace-warnings ...` to show where the warning was created)
(node:10567) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)

@bpasero I am assigning it to you as a start since the trace has file watcher paths.


Unhandled Exception: RangeError (index): Invalid value

[VERBOSE-2:ui_dart_state.cc(186)] Unhandled Exception: RangeError (index): Invalid value: Valid value range is empty: 0

Seems to be working fine, except when no items within range distance.
It throws the above mentioned error

This is not a georange issue. Check length of the array result returned after the fetch request takes place


Koa middleware

Coverage increased (+0.04%) to 87.652% when pulling 0fcd30db29b4992d99ee788199f3f79b66996865 on koa-milldeware into 61c8cbc0bb9cb50f5cc85dc6ba708f1de9a89252 on master.


Y066 cases that can't be rewritten

Consider this case:
import sys

if sys.version_info < (3, 8):  # Y066 use "if sys.version_info >= (3, 8)"
    def bar(x): ...
elif sys.version_info >= (3, 10):
    def bar(x, *, bar=True): ...
else:
    def bar(x, *, bar=True, baz=False): ...

I don't think it is possible to rewrite this code with if sys.version_info >= (3, 8). Should Y066  be raised in this case?

Specifically, Y066's suggestions are only applicable if Y066 is raised on every if and elif condition in the chain.

Can't your example be rewritten to
if sys.version_info >= (3, 10):
    def bar(x, *, bar=True): ...
elif sys.version_info >= (3, 8):
    def bar(x, *, bar=True, baz=False): ...
else:
    def bar(x): ...

?

My bad! You're right, the else clause is equivalent to >= 3.8.

It does get a bit dicey in some cases, like:
if sys.version_info < (3, 8):  # Y066 use "if sys.version_info >= (3, 8)"
    def bar(x): ...
elif sys.version_info == (3, 10):
    def bar(x, *, bar=True): ...
else:
    def bar(x, *, bar=True, baz=False): ...

is not equivalent to:
if sys.version_info == (3, 10):
    def bar(x, *, bar=True): ...
elif sys.version_info >= (3, 8):
    def bar(x, *, bar=True, baz=False): ...
else:
    def bar(x): ...

For Python 3.11, it will execute the wrong block, but it's hard to argue about that one.

I think we have another rule that warns against == comparisons with sys.version_info, because sys.version_info is always a tuple longer than 2 elements, so sys.version_info == (3, 10) will always resolve to False


⚠ Bofrost.at has degraded performance

In fab9732, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1004 ms

Resolved: Bofrost.at performance has improved in aaaca05.


Metric in Signals

I.e. a numerical value that is saved as a dimension, and is graphable

Any chances this will be implemented?
Numerical properties for the events is a fundamental thing for any app. I like everything else about Telemetry but this thing stops me :(


Checks if the environment.yml file is empty and if it contains a dict…

…ionary.
If the environment.yml is empty, an empty dictionary is instantiated.
This is going to raise a TypeError if the environment.yml file contains a list
or other data structure different than a dictionary.
closes #304

this LGTM and travis is happy, so I'm gonna merge! Thanks @GladysNalvarte !!!


Can't install successfully published package

Hello.
I've ran into issue while trying to install my successfully published package using elm package install
Error
actual error log is following:
Some new packages are needed. Here is the upgrade plan.

  Install:
    turbomack/chaetree 1.0.1

Do you approve of this plan? [Y/n] y
Starting downloads...

  ✗ turbomack/chaetree 1.0.1
Error: Problem when downloading the turbomack/chaetree 1.0.1 code.

Package
This happens for both https://github.com/turboMaCk/chae-tree/tree/1.0.1 and
https://github.com/turboMaCk/chae-tree/tree/2.0.0
More debugging info
When I look to ~/.elm/0.17.1/package/turbomack/chaetree/[version] I see only elm-package.json. File is identical to one from github under specific version.
cat ~/.elm/0.17.1/package/turbomack/chaetree/2.0.0/elm-package.json
{
    "version": "2.0.0",
    "summary": "Create multi-level navigation in elm easily.",
    "repository": "https://github.com/turbomack/chaetree.git",
    "license": "BSD-3-Clause",
    "source-directories": [
        "src"
    ],
    "exposed-modules": [
        "Chae.Id",
        "Chae.Node",
        "Chae.Tree"
    ],
    "dependencies": {
        "elm-lang/core": "4.0.5 <= v < 5.0.0"
    },
    "elm-version": "0.17.1 <= v < 0.18.0"
}

Additional info
At first I thought that it might be same issue as described here: https://github.com/elm-lang/elm-package/pull/211 (there is also opened PR https://github.com/elm-lang/elm-package/pull/241) because:

My github user name is turboMaCk but lowercase version is used in repository (turbomack)

Upper case letters are not allowed in repository name so I make whole repository field lowercase

Original name of github repository was ChaeTree.

So I tried to:

Rename repository to lower case version chaetree
Change my user name to lowercase turbomack

This did not help.
Question
I guess I made some mistake in settings but I'm still not sure if cases are the only problem. I'm also thinking about renaming package and republishing it with fixed user name (turboMaCk) and new name chae-tree. Anyway I'm not sure if this solves the issue and I don't want to spam http://package.elm-lang.org with more invalid packages.
I've raised the question about this on elmlang.slack.com but none seems to be familiar with this.
Feel free to remove invalid package from repository if you want to.

Okay, I took it down for you. Republish it as turboMaCk/chae-tree and make sure that matches the name on github including capitalization.
The lower case restriction is on the package names. We can't control people's user names, and I don't think that'd make any sense to try to do.

Thank you @evancz!
This policy seems reasonable to me.
Have a great day and thanks for fast feedback.

No problem, thanks for bringing this up. Maybe it'll be possible to have additional checks some point later so this kind of thing cannot happen. Working on package stuff like this is high on the todo list.


Decide on version to use for xSDK release

It seems that both master and 3.1-rc1 are currently failing for xSDK (https://github.com/xsdk-project/xsdk-issues/issues/108#issuecomment-694401442). We should figure out what we want to have in the xSDK release and fix the issues.

Aren't they failing because of the missing submodule?

I haven't checked yet what needs to be done.

From what i see in spack, submodules are only enabled when cloning master branch. Versions are pulling releases. As releases done using github, they likely don't contains tribits submodule.

I think we just need to add the additional dependencies introduced for the SplineInterpolationOperator to the spack package or use a commit before that. I am not quite sure if we should have a tagged commit for that or not.

Agreed about dependencies, and we probably should tag and make a release, making those dependencies version dependent.

I'm building it right now.

Anyone opposed to tagging the commit before SplineInterpolationOperator for a release to be used by xSDK? That would likely be easiest.


key parameter is incorrectly highlighted as a keyword

 Issue by prabushi
Tuesday Feb 23, 2021 at 13:57 GMT
Originally opened as https://github.com/ballerina-platform/plugin-vscode/issues/231

Description:

Suggested Labels:

Suggested Assignees:

Affected Product Version:
SL Alpha2
OS, DB, other environment details and versions:
Steps to reproduce:
// Function to add an entry to a `map`.
function addEntryToMap(map<string|int> m, string key, string|int value) {
    m[key] = value;
}

Related Issues:

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/31841


What is happening to react-toolbox.com ?

Seems your website was either hijacked, or you forgot to renew your domain?
http://react-toolbox.com/ returns a grey page with keywords such as "cash advance debt consolidation insurance react-toolbox.com"...

A dns flush should fix it

Cool thanks.
I did a dns flush on my machine, but it doesn't work, probably because I'm on a corporate network that has its own cache.
I tried accessing the website at home, and it's working ok, so the issue is indeed caused by my crappy corporate network.
We might want to leave this issue open for a few days though, as I'm surely not the only one facing this issue within a corporate environment.
Thanks!


Symptoms-Checker: Global page should have a red banner with local emergency number

Banner should display users local emergency number

This number is dynamic and we could use the IP for determining the location and then use this table: https://airtable.com/shrZOKYyIC2vZF0Px


fixing doc url

fixes #117

@qshao-pivotal review

LGTM! Thanks for the quick fix!

[clabot:check]


Experience running tools\make netstandard

Hi,
first of all, thanks a lot for the great library 👍
I want to share my experience of running tools\make netstandard.
My environment is:

Win 10 ver 1709 build 16299.309
VS 2017 (using x64 Native Tools Command prompt)

So, the things what were not smooth:

Lack of information about now to build netstandard library :) Really, I spent a lot of time before I found that netstandard is a valid option for tools\make digging inside the Makefile.csharp.mk. It would be great to see it in the tools\make output.
The prerequisites list on https://developers.google.com/optimization/introduction/installing/source.html#requirements_windows does not contain a lot of stuff to be installed: Windows SDK 8.1, Windows CRT and others. Probably it's because I did not have complete version of VS installed.
There is also no info about what to do with the output (i.e. that Core dll should be referenced, and Native dll should be copied). It would be great to have it described as well.
Building OrTools.NetCore.csproj failed with 2 errors:
a)

C:\Program Files\dotnet\sdk\2.1.103\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.Sdk.targets(114,5): error MSB4018: The "GenerateDepsFile" task failed unexpectedly. [c:\Work\or-tools-master\objs\netstandard\OrTools.NetCore.csproj]
C:\Program Files\dotnet\sdk\2.1.103\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.Sdk.targets(114,5): error MSB4018: System.UnauthorizedAccessException: Access to the path 'c:\Google.OrTools.Core.deps.json' is denied. [c:\Work\or-tools-master\objs\netstandard\OrTools.NetCore.csproj]

I had to create this file manually on the root of C: disk. Probably it would not be a problem if I run tools\make under Admin account, but anyway, why it uses root folder?
b) The next time it failed with
C:\Program Files\dotnet\sdk\2.1.103\Microsoft.Common.CurrentVersion.targets(4118,5): error MSB3021: Unable to copy file "obj\x64\Debug\netstandard2.0\Google.OrTools.Core.dll" to "\Google.OrTools.Core.dll". Access to the path 'c:\Google.OrTools.Core.dll' is denied. [c:\Work\or-tools-master\objs\netstandard\OrTools.NetCore.csproj]

However, the dll was already in the obj folder, so I just took it and it worked.

Hi,
Thanks a lot for your feedback !
Currently .Netstandard is still Experimental (and I'm still learning .Net stuff and .Net languages relationship)

I'm currently thinking of moving it to its own makefiles/Makefile.netstandard.mk and add its targets to make help. (I guess netstandard don't need csharp output files contrary to F# which rely on C# library)
note: people would want to have one nuget package for net40 and netstandard2.0 cf #588 (currently we only provide C# net40)

I would say, the long term goal is to use a ortools nuget package to integrate ortools to your .Net project and only build using Makefile/CMake if you want to build a custom local version.
I won't say adding/hacking an example then call make netstandard; make test_netstandard to use ortools netstandard is a good workflow to integrate it...

I need to figure out what is the purpose of this .csproj... Is it really needed if you have a nupkg ? (I didn't check if nuget package need it)

So still not sure how to deal with all of this (any experience/proposal are welcome)

I need to figure out what is the purpose of this .csproj... Is it really needed if you have a nupkg ? (I didn't check if nuget package need it)

I think that OrTools.NetCore.csproj is the one which produces Google.OrTools.Core.dll and then a nupkg with it. For some reason it tries to write that dll in to C:\ not into the relative bin\ directory.

All configuration files are located in <rootdir>/tools/netstandard. .csproj seems to be use by dotnet build/pack to create/copy artifacts...
It seems OutputPath is not defined which Cmaybe could explain why <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory> copy to C:\
Need to do some test...
src: https://msdn.microsoft.com/en-us/library/bb629394.aspx

With the merge on master now make dotnet should be more user friendly
doc here: https://github.com/google/or-tools/blob/master/ortools/dotnet/README.md


Default the database on AppEngine server not require a password?

Looking at the following code, 'PASSWORD': '<your-database-password>' is only assigned to local settings. Should the PASSWORD field not be added for the AppEngine settings as well?

if os.getenv('SERVER_SOFTWARE', '').startswith('Google App Engine'):
# Running on production App Engine, so use a Google Cloud SQL database.
DATABASES = {
'default': {
'ENGINE': 'django.db.backends.mysql',
'HOST': '/cloudsql/:',
'NAME': '',
'USER': 'root',
}
}
else:
DATABASES = {
'default': {
'ENGINE': 'django.db.backends.mysql',
'NAME': '',
'USER': '',
'PASSWORD': '',
'HOST': '',
'PORT': '3306',
}
}

You bring up a good point, but it is not needed.
When the  if os.getenv('SERVER_SOFTWARE', '').startswith('Google App Engine'): is true, the Application is run from Googles servers. When setting up the database, you can choose which application is allowed to access that Cloud SQL instance based on the application ID. So when the application is authorized from the database instance, that serves as enough authentication to allow the connection. (This is done under the Access Control tab within the Cloud SQL instance.)

What @nitrax92 said. App Engine talks to CloudSQL through a Unix root socket that only it can use, so no auth is needed. Maybe if you were doing something fancy where App Engine can only talk to certain tables/databases on the instance, but I didn't want to overcomplicate things.
If you have any issues actually connecting, file a new issue, but this is works-as-intended.

Thanks for the explanation! I actually had to set the password for the AppEngine env in order to access the SQL instance. Perhaps I did not configure the Access Control correctly.
But one thing sounds magic to me: Both TCP and Unix socket are ways to connect to the SQL instance... However when I reach the instance using unix socket and GAE firewall allows my app to access the instance... I thought SQL would still ask for the password! At least traditional MySQL instances have this behaviour as of my understanding!
:+1:


[Bug]: Attempted import error: 'File' is not exported from '@web-std/file' (imported as 'File$1').

Prerequisites

[x] I have checked that @thirdweb-dev/sdk is on the latest stable version and the bug still exists.
[x] I have checked existing issues and I was unable to find a duplicate of this.

Describe the bug
After I upgraded from v2.1.3-0 to v2.3.38, I'm stuck while building the app with react-app-rewired , error happened.
Failed to compile.

<EMAIL_ADDRESS>Attempted import error: 'File' is not exported from '@web-std/file' (imported as 'File$1').

Thanks for reporting this. We're currently working on a new storage package with better separation between node and browser environments. That should fix this issue. Will keep you updated. cc @adam-maj

@joaquim-verges
I think this issue was fixed by following release, right?
https://github.com/thirdweb-dev/typescript-sdk/releases/tag/v2.3.39
But another issue occurred.
Failed to compile.

<EMAIL_ADDRESS>Attempted import error: 'IpfsStorage' is not exported from '@thirdweb-dev/sdk/dist/browser'.

@quytxv that error should be fixed in the latest react version

Thanks for your reply.
In the latest react version(18.2.0), this error occurred.
<EMAIL_ADDRESS>Module not found: Can't resolve './aes' in '_[my path]_/node_modules/@thirdweb-dev/sdk/dist/browser'


noirup fails due to Barretenberg build failure

Description
Aim
Successful installation of the nargo tool in order to perform Noir development.
Expected behavior
For invocation of noirup to successfully install nargo. Something along the lines of:
$ noirup
# ...
$ echo $?
0
$ nargo
# ...
$ echo $?
0

Bug
$ sudo apt-get install libomp-dev cmake
$ curl -L https://raw.githubusercontent.com/noir-lang/noir/master/noirup/install | bash
$ noirup
HEAD is now at e239f415 Handle predicate operator during inlining (#544)
  Installing nargo v0.1.0 (/home/jmcph4/.nargo/noir-lang/noir/crates/nargo)
    Updating crates.io index
   Compiling psm v0.1.21
   Compiling barretenberg_wrapper v0.1.0 (https://github.com/AztecProtocol/barretenberg?rev=804c7dcf21111acd1302a768a8fa2f453dcec50f#804c7dcf)
error: failed to run custom build command for `barretenberg_wrapper v0.1.0 (https://github.com/AztecProtocol/barretenberg?rev=804c7dcf21111acd1302a768a8fa2f453dcec50f#804c7dcf)`

Caused by:
  process didn't exit successfully: `/home/jmcph4/.nargo/noir-lang/noir/target/release/build/barretenberg_wrapper-6e0ec251307286f5/build-script-build` (exit status: 101)
  --- stdout
  CMAKE_TOOLCHAIN_FILE_x86_64-unknown-linux-gnu = None
  CMAKE_TOOLCHAIN_FILE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_TOOLCHAIN_FILE = None
  CMAKE_TOOLCHAIN_FILE = None
  CMAKE_GENERATOR_x86_64-unknown-linux-gnu = None
  CMAKE_GENERATOR_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_GENERATOR = None
  CMAKE_GENERATOR = None
  CMAKE_PREFIX_PATH_x86_64-unknown-linux-gnu = None
  CMAKE_PREFIX_PATH_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_PREFIX_PATH = None
  CMAKE_PREFIX_PATH = None
  CMAKE_x86_64-unknown-linux-gnu = None
  CMAKE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE = None
  CMAKE = None
  running: "cmake" "/home/jmcph4/.cargo/git/checkouts/barretenberg-6ce83dfea69613eb/804c7dc/barretenberg_wrapper/../barretenberg" "-DCMAKE_INSTALL_PREFIX=/home/jmcph4/.nargo/noir-lang/noir/target/release/build/barretenberg_wrapper-24257eaf3fa79116/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=/usr/bin/c++" "-DCMAKE_ASM_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_ASM_COMPILER=/usr/bin/cc" "-DCMAKE_BUILD_TYPE=Release"
  -- The CXX compiler identification is GNU 12.2.0
  -- The C compiler identification is GNU 12.2.0
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: /usr/bin/c++ - skipped
  -- Detecting CXX compile features
  -- Detecting CXX compile features - done
  -- Detecting C compiler ABI info
  -- Detecting C compiler ABI info - done
  -- Check for working C compiler: /usr/bin/cc - skipped
  -- Detecting C compile features
  -- Detecting C compile features - done
  -- Found OpenMP_C: -fopenmp (found version "4.5")
  -- Found OpenMP_CXX: -fopenmp (found version "4.5")
  -- Found OpenMP: TRUE (found version "4.5")
  -- Multithreading is enabled.
  -- Found Python: /home/jmcph4/.pyenv/shims/python3 (found version "3.9.0") found components: Interpreter
  -- Performing Test CMAKE_HAVE_LIBC_PTHREAD
  -- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
  -- Found Threads: TRUE
  -- Failed to find LLVM FileCheck
  -- Found Git: /usr/bin/git (found version "2.37.2")
  -- git version: v1.6.1 normalized to 1.6.1
  -- Version: 1.6.1
  -- Looking for shm_open in rt
  -- Looking for shm_open in rt - found
  -- Performing Test HAVE_CXX_FLAG_STD_CXX11
  -- Performing Test HAVE_CXX_FLAG_STD_CXX11 - Success
  -- Performing Test HAVE_CXX_FLAG_WALL
  -- Performing Test HAVE_CXX_FLAG_WALL - Success
  -- Performing Test HAVE_CXX_FLAG_WEXTRA
  -- Performing Test HAVE_CXX_FLAG_WEXTRA - Success
  -- Performing Test HAVE_CXX_FLAG_WSHADOW
  -- Performing Test HAVE_CXX_FLAG_WSHADOW - Success
  -- Performing Test HAVE_CXX_FLAG_WERROR
  -- Performing Test HAVE_CXX_FLAG_WERROR - Success
  -- Performing Test HAVE_CXX_FLAG_WSUGGEST_OVERRIDE
  -- Performing Test HAVE_CXX_FLAG_WSUGGEST_OVERRIDE - Success
  -- Performing Test HAVE_CXX_FLAG_PEDANTIC
  -- Performing Test HAVE_CXX_FLAG_PEDANTIC - Success
  -- Performing Test HAVE_CXX_FLAG_PEDANTIC_ERRORS
  -- Performing Test HAVE_CXX_FLAG_PEDANTIC_ERRORS - Success
  -- Performing Test HAVE_CXX_FLAG_WSHORTEN_64_TO_32
  -- Performing Test HAVE_CXX_FLAG_WSHORTEN_64_TO_32 - Failed
  -- Performing Test HAVE_CXX_FLAG_FSTRICT_ALIASING
  -- Performing Test HAVE_CXX_FLAG_FSTRICT_ALIASING - Success
  -- Performing Test HAVE_CXX_FLAG_WNO_DEPRECATED_DECLARATIONS
  -- Performing Test HAVE_CXX_FLAG_WNO_DEPRECATED_DECLARATIONS - Success
  -- Performing Test HAVE_CXX_FLAG_WNO_DEPRECATED
  -- Performing Test HAVE_CXX_FLAG_WNO_DEPRECATED - Success
  -- Performing Test HAVE_CXX_FLAG_WSTRICT_ALIASING
  -- Performing Test HAVE_CXX_FLAG_WSTRICT_ALIASING - Success
  -- Performing Test HAVE_CXX_FLAG_WD654
  -- Performing Test HAVE_CXX_FLAG_WD654 - Failed
  -- Performing Test HAVE_CXX_FLAG_WTHREAD_SAFETY
  -- Performing Test HAVE_CXX_FLAG_WTHREAD_SAFETY - Failed
  -- Performing Test HAVE_CXX_FLAG_COVERAGE
  -- Performing Test HAVE_CXX_FLAG_COVERAGE - Success
  -- Performing Test HAVE_STD_REGEX
  -- Performing Test HAVE_STD_REGEX
  -- Performing Test HAVE_STD_REGEX -- success
  -- Performing Test HAVE_GNU_POSIX_REGEX
  -- Performing Test HAVE_GNU_POSIX_REGEX
  -- Performing Test HAVE_GNU_POSIX_REGEX -- failed to compile
  -- Performing Test HAVE_POSIX_REGEX
  -- Performing Test HAVE_POSIX_REGEX
  -- Performing Test HAVE_POSIX_REGEX -- success
  -- Performing Test HAVE_STEADY_CLOCK
  -- Performing Test HAVE_STEADY_CLOCK
  -- Performing Test HAVE_STEADY_CLOCK -- success
  -- Using optimized assembly for field arithmetic.
  -- Looking for unistd.h
  -- Looking for unistd.h - found
  -- Looking for crc32c_value in crc32c
  -- Looking for crc32c_value in crc32c - not found
  -- Looking for snappy_compress in snappy
  -- Looking for snappy_compress in snappy - not found
  -- Looking for malloc in tcmalloc
  -- Looking for malloc in tcmalloc - not found
  -- Looking for fdatasync
  -- Looking for fdatasync - found
  -- Looking for F_FULLFSYNC
  -- Looking for F_FULLFSYNC - not found
  -- Performing Test HAVE_CLANG_THREAD_SAFETY
  -- Performing Test HAVE_CLANG_THREAD_SAFETY - Failed
  -- Performing Test HAVE_CXX17_HAS_INCLUDE
  -- Performing Test HAVE_CXX17_HAS_INCLUDE - Success
  -- Looking for sqlite3_open in sqlite3
  -- Looking for sqlite3_open in sqlite3 - found
  -- Performing Test HAVE_KYOTOCABINET
  -- Performing Test HAVE_KYOTOCABINET - Failed
  -- Configuring done
  -- Generating done
  -- Build files have been written to: /home/jmcph4/.nargo/noir-lang/noir/target/release/build/barretenberg_wrapper-24257eaf3fa79116/out/build
  running: "cmake" "--build" "." "--target" "install" "--config" "Release"
  [  0%] Building CXX object src/aztec/env/CMakeFiles/env_objects.dir/logstr.cpp.o
  [  0%] Built target env_objects
  [  1%] Building CXX object src/aztec/numeric/CMakeFiles/numeric_objects.dir/random/engine.cpp.o
  [  1%] Built target numeric_objects
  [  2%] Building CXX object _deps/googletest-build/googletest/CMakeFiles/gtest.dir/src/gtest-all.cc.o

  --- stderr
  CMake Warning:
    Manually-specified variables were not used by the project:

      CMAKE_ASM_COMPILER
      CMAKE_ASM_FLAGS

  In file included from /usr/include/c++/12/ios:40,
                   from /usr/include/c++/12/ostream:38,
                   from /usr/include/c++/12/bits/unique_ptr.h:41,
                   from /usr/include/c++/12/memory:76,
                   from /home/jmcph4/.nargo/noir-lang/noir/target/release/build/barretenberg_wrapper-24257eaf3fa79116/out/build/_deps/googletest-src/googletest/include/gtest/gtest.h:57,
                   from /home/jmcph4/.nargo/noir-lang/noir/target/release/build/barretenberg_wrapper-24257eaf3fa79116/out/build/_deps/googletest-src/googletest/src/gtest-all.cc:38:
  In static member function ‘static constexpr std::char_traits<char>::char_type* std::char_traits<char>::copy(char_type*, const char_type*, std::size_t)’,
      inlined from ‘static constexpr void std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::_S_copy(_CharT*, const _CharT*, size_type) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]’ at /usr/include/c++/12/bits/basic_string.h:423:21,
      inlined from ‘constexpr std::__cxx11::basic_string<_CharT, _Traits, _Allocator>& std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::_M_replace(size_type, size_type, const _CharT*, size_type) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]’ at /usr/include/c++/12/bits/basic_string.tcc:532:22,
      inlined from ‘constexpr std::__cxx11::basic_string<_CharT, _Traits, _Alloc>& std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::replace(size_type, size_type, const _CharT*, size_type) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]’ at /usr/include/c++/12/bits/basic_string.h:2171:19,
      inlined from ‘constexpr std::__cxx11::basic_string<_CharT, _Traits, _Alloc>& std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::insert(size_type, const _CharT*) [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]’ at /usr/include/c++/12/bits/basic_string.h:1928:22,
      inlined from ‘constexpr std::__cxx11::basic_string<_CharT, _Traits, _Allocator> std::operator+(const _CharT*, __cxx11::basic_string<_CharT, _Traits, _Allocator>&&) [with _CharT = char; _Traits = char_traits<char>; _Alloc = allocator<char>]’ at /usr/include/c++/12/bits/basic_string.h:3541:36,
      inlined from ‘static std::string testing::internal::StreamingListener::UrlEncode(const char*)’ at /home/jmcph4/.nargo/noir-lang/noir/target/release/build/barretenberg_wrapper-24257eaf3fa79116/out/build/_deps/googletest-src/googletest/src/gtest.cc:4882:27:
  /usr/include/c++/12/bits/char_traits.h:431:56: error: ‘void* __builtin_memcpy(void*, const void*, long unsigned int)’ accessing<PHONE_NUMBER>854775810 or more bytes at offsets [2,<PHONE_NUMBER>854775807] and 1 may overlap up to<PHONE_NUMBER>854775813 bytes at offset -3 [-Werror=restrict]
    431 |         return static_cast<char_type*>(__builtin_memcpy(__s1, __s2, __n));
        |                                        ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
  cc1plus: all warnings being treated as errors
  gmake[2]: *** [_deps/googletest-build/googletest/CMakeFiles/gtest.dir/build.make:76: _deps/googletest-build/googletest/CMakeFiles/gtest.dir/src/gtest-all.cc.o] Error 1
  gmake[1]: *** [CMakeFiles/Makefile2:1185: _deps/googletest-build/googletest/CMakeFiles/gtest.dir/all] Error 2
  gmake: *** [Makefile:146: all] Error 2
  thread 'main' panicked at '
  command did not execute successfully, got: exit status: 2

  build script failed, must exit now', /home/jmcph4/.cargo/registry/src/github.com-1ecc6299db9ec823/cmake-0.1.49/src/lib.rs:1104:5
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: failed to compile `nargo v0.1.0 (/home/jmcph4/.nargo/noir-lang/noir/crates/nargo)`, intermediate artifacts can be found at `/home/jmcph4/.nargo/noir-lang/noir/target`
noirup: command failed: cargo install --path ./crates/nargo --bins --locked --force --root /home/jmcph4/.nargo

To reproduce

Install noirup (as per https://github.com/noir-lang/noir/tree/1d1b592039fa73ce561d6b6886c36b02b36ada53/noirup#readme)
Invoke noirup

Environment
$ uname --all
Linux foobar 5.19.0-2-amd64 #1 SMP PREEMPT_DYNAMIC Debian 5.19.11-1 (2022-09-24) x86_64 GNU/Linux
$ cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux bookworm/sid"
NAME="Debian GNU/Linux"
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

Additional context
I initially attempted to follow the official documentation, but this failed for other reasons. I then found noirup and decided it was the more graceful approach.
The build appears to be failing during the Barretenberg build section. I appear to satisfy the Barretenberg dependences.

Since moving to nix, this issue has been solved. Feel free to reopen if you encounter it again!


Dynamic import not transformed when used in unreachable code path.

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Build script fails
If the current behavior is a bug, please provide the steps to reproduce.
if (false) {
    import('./module'); // <-- Fails to transform this or remove this
}

What is the expected behavior?
Probably transforming this expression as well and letting uglify to get rid of it later.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
node -v
> v8.1.3

I came across this issue because we use feature flags at build time that include specific code if feature is enabled.

Related: https://github.com/webpack/webpack/issues/4857

duplicate of #4857


Pacman

🎮 Game Request
Pac-Man is one of the most iconic video games of all time, and most people (even non-gamers) have at least a passing familiarity with it. The purpose of the game is very simple — the player is placed in a maze filled with food (depicted as pellets or dots) and needs to eat all of it to advance to the next level. This task is made difficult by four ghosts that pursue Pac-Man through the maze. If Pac-Man makes contact with any of the ghosts, the player loses a life and the positions of Pac-Man and the ghosts are reset back to their starting locations.
Point down the features

There would be two ghost
Pacman has to eat all the dots one by one
Be safe from the ghost
It is like pacman vs ghost.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @utkarshgupta04092003! 🙂
It seems like you are not following proper guidelines !! 👀
Read documentation properly !!🙏
If you have any queries reach out to us on Discord🕹 .

Hey @utkarshgupta04092003 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @kunjgit ,
Can you please explain why you closed this issue, I dont understand.


Document .gxignore

It looks like there's support for a .gxignore file. How does that work? It should be documented.

I put a note in the readme about the gxignore, it works just the same as a .gitignore file.


RS-374: Update docs according to image RHACS image changes released in 3.68

For reviewers - please check my self-comments and share your opinion!

Unless anyone has any pending comments or objections, I have one approval and I would like to merge this soon in order not to stand in a way of release.

Sorry, I didn't manage to give it a look, but you have my full trust to merge @msugakov :)

I'm merging now. If anyone still has any follow-ups, I'd be happy to address.


RECORD_LAYOUT fields are not sorted correctly

RECORD_LAYOUT fields should be sorted by the position (index) in an ascending order. Some tools give warning about this.
Current behavior:
/begin RECORD_LAYOUT
      MAP_XU16_YU16_ZU16
      AXIS_PTS_X 3 UWORD INDEX_INCR DIRECT
      AXIS_PTS_Y 4 UWORD INDEX_INCR DIRECT
      FNC_VALUES 5 UWORD ROW_DIR DIRECT
      NO_AXIS_PTS_X 1 UBYTE
      NO_AXIS_PTS_Y 2 UBYTE
/end RECORD_LAYOUT

Expected behavior:
/begin RECORD_LAYOUT
      MAP_XU16_YU16_ZU16
      NO_AXIS_PTS_X 1 UBYTE
      NO_AXIS_PTS_Y 2 UBYTE
      AXIS_PTS_X 3 UWORD INDEX_INCR DIRECT
      AXIS_PTS_Y 4 UWORD INDEX_INCR DIRECT
      FNC_VALUES 5 UWORD ROW_DIR DIRECT
/end RECORD_LAYOUT

Thanks for the report.
The issue is this: there is no special case for the ordering of newly created RECORD_LAYOUTs in the code, so its optional components are written alphabetically. RECORD_LAYOUTS that were read from a file have attached layout info, so they don't have this issue.

I just published a2lfile 1.5.0 with this fix and several others


[Event Request] Codeunit 445 Purch.-Post Prepmt. (Yes/No)  – PostPrepmtInvoiceYN

[Event Request] Codeunit 445 Purch.-Post Prepmt. (Yes/No)  – PostPrepmtInvoiceYN
Could we please have a publisher OnCheckBeforePostPrepmtInvoice in function PostPrepmtInvoiceYN Codeunit 445 Purch.-Post Prepmt. (Yes/No)?
[External] PostPrepmtInvoiceYN(VAR PurchHeader2 : Record "Purchase Header";Print : Boolean)

PurchHeader.COPY(PurchHeader2);
WITH PurchHeader DO BEGIN
  IF NOT ConfirmManagement.ConfirmProcess(STRSUBSTNO(Text000,"Document Type","No."),TRUE) THEN
    EXIT;

  // >>>>>>>>
  OnCheckBeforePostPrepmtInvoice(var PurchHeader);
  // <<<<<<<<

  PostPrepmtDocument(PurchHeader,"Document Type"::Invoice);

  IF Print THEN

Yes, but it is more logical to add event OnBeforePostPrepmtDocument in procedure PostPrepmtDocument?


@broskoski => Add existy check on main_layout html

Something @joeyAghion pointed out in #web the other day: https://artsy.slack.com/archives/web/p1489725867036265
We just need to make sure lab_features exists on CURRENT_USER.

Sorry, missed this. Looks good, nice tests.


All progress lost since checking yesterday

Yesterday, when I was logging issues and playing with the site, I made sure to save all my progress. Upon checking today all my progress is lost and I have to re-tick off everything I've mastered.
I'm guessing something in your update process is nuking the database? It's going to really frustrate users if you keep this up.

Weird, that shouldn't have happened.  The only thing I could think of that would wipe your progress is an issue with the auto save system. Let us know if you manage to get your data reset again.


Caching Proxy Full Requests

In the proxy, we need to capture full requests and responses from reproduction requests with a special header so that we can fuzz the frontend without modifying the state of the application and do it in parallel.

Added in 6983efe14d6cecd30ff10a8661e6a4617f6fe0b2


Fix: If Node moved to another parent then no any action to delete is …

…need.

If Node moved to another parent then no any action to delete is needed

nodes don't move by themselves around, obtrusive libraries might move nodes around though but that violates the purpose of the differ which owns the elements.
this change breaks every library based on domdiff so I am afraid this won't happen.
what you maybe want is to use .remove() instead of parentNode.removeChild() but diffing across two elements makes no sense, also because you can just move all elements at once via fragment.append(...latestNodesInBody).

@WebReflection  this is fix for use case - https://codepen.io/anthony-redfox-the-lessful/pen/BaVqLzq?editors=0010
when node was re used in another place. I expected that will be render all nodes (with fix it is works)
foo
bar
baz
The same example with Uhtml - https://codepen.io/anthony-redfox-the-lessful/pen/ExRGVoG?editors=0010
Can you say what I doing wrong with hyperHtml or Uhtml

The same example with Uhtml

drop the .for(...) and no error happens

Can you say what I doing wrong with hyperHtml or Uhtml

you want a single node to be simultaneously part of two different parts of the tree ... that can't happen with the DOM, so your option is to not wire the node and just use html tag as is (in uhtml).
Alternatively you can fork this project which is basically dead and use node.remove() instead of parentNode.removeChild(node) within the drop function but you have a fundamental issue with your logic, imho, and that's not worth fixing to me.


error during pull dynamics-nav:2018-de

If the scripts try to pull the dynamics-nav:2018-de image, it ends with

DockerDo : ExitCode: 1
Commandline: docker pull  microsoft/dynamics-nav:2018-de
In ....microsoft-navcontainerhelper\Container
ps1:253 Zeichen:13

        DockerDo -command pull -imageName $imageName | Out-Null

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,DockerDo

after retrying to download some parts.

If you update the navcontainerhelper - you should get a better error message, which might indicate what is wrong.
Might be that you need to run docker logout (have seen that a few times)

Clean up my virtual machine resolves this issue.
It was not enough disk space for docker to pull the images.

Ok, thanks for the info


Update Roslyn to 4.3.0-3.22329.4

Fixes https://github.com/OmniSharp/omnisharp-vscode/issues/5248

I think I need to scrap this and start over. I believe the problem is with the O# implementation of ILegacyGlobalOptionsWorkspaceService that I added to Roslyn. I let it take all defaults because I didn't expect a lot of things to still be using the Legacy options, however, I see where it could be inheriting Environment.NewLine as its NewLine character instead of taking the appropriate formatting option.

Superseded by https://github.com/OmniSharp/omnisharp-roslyn/pull/2420


mix format everything

Coverage remained the same at 93.75% when pulling 88546241c4973b988f7c8aea3d0cd239ac2eb4b7 on mix_format into c378ed44722699f020dbccd6b5e8f035e9a98b6c on master.


Configure file for user defined alias like .gitconfig

Thanks  for this great tool!
I want to configure user defined alias, e.g.
$pipenv i # equals pipenv install

I think different people have different preference on this abbreviation, so it would be great if there was a configure feature like .gitconfig.

Pipenv team are against involving more config files other than Pipfile itself.
Anyway you can achieve this by aliases.


python libs in python binding cannot be found when option build_opencv_world is choosed

python libs in python binding cannot be found when option build_opencv_world is choosed.
this is strange.

Please provide more information about your system configuration, build options, etc.


fix: make navigation item styles consistent

Currently we must click on the label on navigation items that are buttons, as opposed to links who behave correctly. This PR fixes just that, by making behavior consistent for both links and buttons.

Hmm, this seems to give me weird styling on Safari

Hmm, this seems to give me weird styling on Safari

Ah, good looking out. I don't have Safari to test unfortunately... Firefox, Chrome and Edge look correct. Can you show me a screenshot of the navigation after a paste is saved? So I can figure out what needs to be changed.

Ah, good looking out. I don't have Safari to test unfortunately... Firefox, Chrome and Edge look correct. Can you show me a screenshot of the navigation after a paste is saved? So I can figure out what needs to be changed.

Hey @innocenzi, I've merged the other PR that was sent in which seems to fix things for me: https://github.com/laravelio/paste.laravel.io/pull/80
Still thank you for helping out here!

Hey @innocenzi, I've merged the other PR that was sent in which seems to fix things for me: https://github.com/laravelio/paste.laravel.io/pull/80
Still thank you for helping out here!

No problem. It's still lacking the focus styles though (there's a black outline only around the actual text right now), but we can click the whole button, so it's enough for me.

No problem. It's still lacking the focus styles though (there's a black outline only around the actual text right now), but we can click the whole button, so it's enough for me.


Getting AttributeError: 'NoneType' object has no attribute 'total_seconds' requesting calendar for XNYS

following code (jupyterhub notebook):
import pandas as pd
import numpy as np
from datetime import datetime, date, timedelta
import exchange_calendars as xcals
xnys = xcals.get_calendar('XNYS')
Get following error: AttributeError: 'NoneType' object has no attribute 'total_seconds'
Full trace in file attached.
CalendarErrors.txt

Hi @chadale, I'm unable to replicate this error. If you're not already using the latest version of exchange_calendars(?) could you try upgrading and see if that fixes it. If that doesn't work then I'd suggest upgrading the dependencies to the latest versions. Let us know if none of that does the trick.


mac error:Current directory does not contain any .nswag files.

$ ./node_modules/.bin/nswag run
NSwag NPM CLI
NSwag command line tool for .NET Core NetCore21, toolchain v<IP_ADDRESS> (NJsonSchema v<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>))
Visit http://NSwag.org for more information.
NSwag bin directory: /Users/janeydeng/projects/DotNet/April/alain/node_modules/nswag/bin/binaries/NetCore21
Current directory does not contain any .nswag files.
Duration: 00:00:00.0304700

It works in windows,but not in mac.When I change the runtime netcore 31 version,the NSwag bin directory still points to NetCore21.
The path is not match with the config.
cd nswag
../node_modules/.bin/nswag run

or
./node_modules/.bin/nswag run ./nswag/service.config.nswag
It works in mac.
the NSwag bin directory still points to NetCore21.

nswag run nswag.json --runtime NetCore31

For mac it should work: node_modules/.bin/nswag run nswag/service.config.nswag
Make sure your backend is running.


Create missingness chunk in preprocessing

Create a chunk that displays the missingness of each sample on each measure.
Chunk currently exists called: missingness

[ ] Include missingness  per individual
[ ] Include missingness per questionnaire item/total questionnaire

We will also need to decide what to do for the sum scores for participants that have missingness in the measures (PHQ, GAD, PCL)

Have created these chunks.


Error encounterred when upgrading to Ansible 2.8.0

I hit this error in my project after upgrading to Ansible 2.8.0
    TASK [install openshift client] ************************************************
ERROR! 'become' is not a valid attribute for a TaskInclude
The error appears to be in '/tmp/molecule/setup/default/roles/andrewrothstein.alpine-glibc-shim/tasks/main.yml': line 2, column 3, but may
be elsewhere in the file depending on the exact syntax problem.
The offending line appears to be:
---
- name: fix alpine
  ^ here

    PLAY RECAP *********************************************************************
    instance                   : ok=1    changed=0    unreachable=0    failed=0    skipped=0    rescued=0    ignored=0

fixed this in v2.0.9 of the role. currently in the process of upgrading the dependent roles.

published v1.0.10 of the andrewrothstein.openshift-origin-client-tools role incorporating this updateif ur trying to use that.


Feat(UI) #9204: Added preview panel for all the data assets.

Describe your changes :

I worked on #9204

[x] Adding data assets summary details panel for all the data assets.
[x] Styling improvements
[x] bug fixes

Type of change :

[x] Bug fix
[x] Improvement

Frontend Preview (Screenshots) :

Tables:

Dashboards:

Pipelines:

MlModels:

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Frontend: @open-metadata/ui

@aniketkatkar97 Did we also make the first result selected by default?

@aniketkatkar97 Did we also make the first result selected by default?
Yes we did, also the selected card will remain highlighted


Inconsistency between adp.affected and cna.affected in CVE-2024-32761

While reviewing the JSON files, I discovered an inconsistency between the adp.affected and cna.affected fields in the CVE-2024-32761 record. I have corrected this mistake to ensure data accuracy and consistency across the CVE record. Please review the changes.
https://my.f5.com/manage/s/article/K000139217

@kotakanbe thanks for the catch! I've passed this up to our analysts to review and fix. I'm going to merge this request, but be aware that the changes will get overwritten once the data is fixed upstream.


Customise default fields components

Description
When Im trying to overwrite any default MediaField component (just want to add some new buttons), and i put it in resources/assests/js/components (I already have some overwrited components there) it's not working for me. Seems like twill is not working with my overwrited MediaField and using default one
Steps to reproduce

Choose one of fields vue component in frontend/js/components
2)Copy and Add something new in this component inside your project
Build application though php artisan commands

Expected result
Changes are shown in component in Admin panel
Actual result
Twill still using default component
Versions
Twill version:  2.8.7

I need help for overriding ItemList.vue component too. Couldn't seem to call it when placed in resources/assests/js/components

I'm sorry we didn't get to you earlier but I just found out this is just a silly casing issue. Using Mediafield.vue and Itemlist.vue works fine to override them. As always, this folder is there for convenience but you can always take more control over the Twill frontend build and fully override components of you need to.


Problem setting gutter to zero

Gutter value 0 doesn't seem to work:
cell(2, 3, g: 0)
However, using 0.0001 works. Is there something I miss or is this a bug?

Hi, are you sure that you don't have any other styles which could overwrite gutter styles? Maybe some paddings or margins on the same element?

Yes, it's a new project with almost no styles. Plus the devtools shows 'invalid property value':
flex-basis: calc(100% * .25 - 0)
In case of 0.00001 it looks like this (it's a 1/4 col):
flex-basis: calc(100% * .25 - 6.25e-7rem)

Ok, that's weird ;) I'll debug this asap.

I've tested it using something like:
@import 's-grid-settings'
@import 's-grid-functions'

.grid
  grid()
  .cell
    cell(1, 3, g: 0)

And it produces me something like:
* {
  box-sizing: border-box;
  margin: 0;
  padding: 0;
}
.grid {
  display: flex;
  flex-wrap: wrap;
  align-items: flex-start;
  flex-direction: row;
}
.grid .cell {
  flex-basis: calc(100% * 0.333333333333333 - 0rem);
  min-width: 0;
}

So it seems to work well. The output in the browser is also good in both cases with and without gutter.

Thanks, I see now where it fails. It's the compiler setting  --compress which removes the "rem" and then the CSS calc fails.
I think sGrid work around this because I guess I'm not the only one who uses --compress.

sure, good point, I'll try to fix this

Ok, you can try with version 1.2.1 It is also published on npm

Seems to work fine after the upgrade, thanks!


ImportError: To be able to use evaluate-metric/rouge, you need to install the following dependencies['nltk'] using 'pip install # Here to have a nice missing dependency error message early on' for instance'

I am now doing summarization tutorial task.
however, I encounter when I run
import evaluate
rouge = evaluate.load("rouge", use_aggregator=False)

The error message is below:
Traceback (most recent call last):
  File "/hdd/hdd2/baek26/onboarding/Summary/billsum.py", line 36, in <module>
    rouge = evaluate.load("rouge", use_aggregator=False)
  File "/home/guest-bje/.local/share/virtualenvs/onboarding-kyCypCas/lib/python3.9/site-packages/evaluate/loading.py", line 748, in load
    evaluation_module = evaluation_module_factory(
  File "/home/guest-bje/.local/share/virtualenvs/onboarding-kyCypCas/lib/python3.9/site-packages/evaluate/loading.py", line 680, in evaluation_module_factory
    raise e1 from None
  File "/home/guest-bje/.local/share/virtualenvs/onboarding-kyCypCas/lib/python3.9/site-packages/evaluate/loading.py", line 633, in evaluation_module_factory
    return HubEvaluationModuleFactory(
  File "/home/guest-bje/.local/share/virtualenvs/onboarding-kyCypCas/lib/python3.9/site-packages/evaluate/loading.py", line 489, in get_module
    local_imports = _download_additional_modules(
  File "/home/guest-bje/.local/share/virtualenvs/onboarding-kyCypCas/lib/python3.9/site-packages/evaluate/loading.py", line 265, in _download_additional_modules
    raise ImportError(
ImportError: To be able to use evaluate-metric/rouge, you need to install the following dependencies['nltk'] using 'pip install # Here to have a nice missing dependency error message early on' for instance'

I visited this issue, but couldn't solve it.
My Environment is as pipfile below:
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]
gymnasium = "*"
matplotlib = "*"
tqdm = "*"
transformers = "*"
datasets = "*"
evaluate = "*"
rouge-score = "*"
nltk = "*"
torch = "==1.12.1"
torchvision = "==0.13.1"
torchaudio = "==0.12.1"
accelerate = "*"

[dev-packages]

[requires]
python_version = "3.9"
python_full_version = "3.9.2"
````

I solved the problem by
sudo apt-get update
sudo apt-get install -y make build-essential libssl-dev zlib1g-dev libbz2-dev \
libreadline-dev libsqlite3-dev wget curl llvm libncurses5-dev libncursesw5-dev \
xz-utils tk-dev

it was pyenv problem that it didn't install python well.


Split hadoop-service

hadoop-service  ->  zookeeper + hadoop-name-node + hadoop-data-node + hadoop-resource-manager + hadoop-node-manager + hadoop-jobhistory + hadoop-batch-job

@ydye : Could you briefly explain the purpose of this change (what problem does it solve?) and what the design is (how did you split and why did you do in this way?). Please help us understand your work better. Thanks.

For VC-UPDATE

Pros and cons?


fix(impl_tokio): prevent panic on errorneous BlockingDirections

Fixes a panic in the tokio implementation of the session stream which occurs under high load.
Patch originally authored by @andras-pinter

@netthier I think it should only remove unreachable!. we should change expected_block_directions if the assert! panic


Global shortcut Ctrl+Alt+R

This global shortcut conflicts with TotalCommander.
In TotalCommander, you can use AltGr+any letter to filter the current pane.
AltGr is generally seen as the same as Ctrl+Alt.
It's handy I use it a lot but now my workflow is interrupted whenever I hit R because RepoZ appears and steals focus.
Some ideas:

Allow users to remap the global shortcut. I suppose I could use Win+G (for Git) as it seems to do nothing on my machine.
Can you distinguish AltGr and Ctrl+Alt ? For this specific conflict, if RepoZ would not trigger on AltGr I would be fine.

Win+G opens the Xbox Game Bar, unless yuu've disabled the shortcut 😅. Win+Z might be safer because it's not bound to anything by default (and is what I use to trigger AutoHotKey scripts 😇).
Regardless of the default, it would ideally be configurable.


Vendor dependencies using submodules

After trying things like Godep, glide settled on using submodules(manul helped with easing the task significantly)

Coverage remained the same at 39.772% when pulling c135bf4fd9e40de38e3afac16f21fb4884c5095e on vendor into f06da6335eb51cb5f4b1db366a50cc982b00e08e on master.

LGTM!


Insecure HTTP URLs in definition files

A large number of libraries AboutLibraries includes definition files contain insecure HTTP links instead of HTTPS even to targets like github.com or github.io which are known to support HTTPS.
Please update those files as HTTP links are insecure and can be abused by an attacker e.g. to perform a Phishing attack or may be even to compromise the Android device.
Furthermore I would recommend to create a policy that pull requests for new or updated definition files with non-HTTPS links are no  longer accepted as this may affect the security of the app and/or the Android system the app is executed on in a negative way.
Also I would recommend to update the Definition-File-Builder to provide secure default values (https://github.com as sample link for example).

I have updated the 2 def files with non secure github links and updated the def file builder.
I can't update all other urls though as it would require to check every single page if the https link would resolve.
I will keep an eye on new definition files for the future to ensure the links are https

Thank you very much for the issue

I am sorry but just replacing github links is not what I had in mind. You even missed most of the github links as you have not changed github.io links.
Therefore I am a bit dissatisfied with your response. I extracted the http urls from the XML files and tested them with a little script for https availability for each url.
The following URLs are available via HTTPS and therefore should be updated:
http://aidanfollestad.com/
http://amlcurran.github.io/ShowcaseView/
http://apache.org
http://blog.zhanghai.me/
http://chris.banes.me/
http://commons.apache.org/proper/commons-csv/
http://commons.apache.org/proper/commons-io/
http://creativecommons.org/licenses/by/3.0/legalcode
http://daimajia.com
http://daimajia.com/
http://developer.lge.com/
http://developer.lge.com/MOBILE_HELP/index.jsp?topic=%2Flge.mobilesdk.LGQSlide.api%2Fhtml%2FDeveloping%2FDeveloping%2FDeveloping+with+LG+QSlide+SDK.htm
http://facebook.github.io/rebound/
http://fsf.org/
http://gmariotti.blogspot.it/
http://google.com
http://h6ah4i.com/
http://inthecheesefactory.com/blog/correct-imageview-adjustviewbounds-with-adjustable-imageview/en
http://jakewharton.com/
http://jakewharton.github.io/butterknife/
http://jhy.io/
http://joanzapata.com/android-iconify/
http://jsoup.org/
http://koush.com/
http://ksoichiro.blogspot.pt/
http://ksoichiro.github.io/Android-ObservableScrollView/
http://libphonenumber.googlecode.com/svn/trunk/javascript/i18n/phonenumbers/demo-compiled.html
http://luke.klinker.xyz/
http://malinskiy.com/
http://mikepenz.com/
http://mixpanel.com/
http://mixpanel.github.io/mixpanel-android/index.html
http://nostra13android.blogspot.co.at/
http://opensource.org/licenses/BSD-2-Clause
http://opensource.org/licenses/BSD-3-Clause
http://opensource.org/licenses/MIT
http://paymill.com
http://roomorama.com
http://satyan.github.io/sugar/
http://scripts.sil.org/OFL
http://site.mockito.org/
http://square.github.io/
http://square.github.io/dagger/
http://square.github.io/okhttp/
http://square.github.io/picasso/
http://square.github.io/retrofit/
http://stfalcon.com/
http://su.chainfire.eu/
http://svn.apache.org/viewvc/commons/proper/io/trunk
http://tozny.com/
http://trello.com/
http://wasabeef.jp/
http://www.akexorcist.com/
http://www.apache.org/licenses/LICENSE-2.0
http://www.apache.org/licenses/LICENSE-2.0.html
http://www.avast.com
http://www.daniel-stone.uk/
http://www.devbrackets.com/
http://www.eduardereza.com/
http://www.facebook.com/
http://www.freepik.com
http://www.freepik.com/free-photos-vectors/people
http://www.gnu.org/licenses/lgpl-3.0.en.html
http://www.gotev.net
http://www.jetbrains.org
http://www.joanzapata.com/
http://www.piwai.info/
http://www.twitter.com/
http://www.wix.com/amulyakhare/portfolio

I'm always happy to merge any PR to further improve this.
I did a search and replace for GitHub links if it missed some, I'm sorry.
Seems the ide fooled me in that case.
On Sat, Oct 27, 2018, 1:02 PM Jan S<EMAIL_ADDRESS>wrote:

I am sorry but just replacing github links is not what I had in mind. You
even missed most of the github links as you have not changed github.io
links.
Therefore I am a bit dissatisfied with your response. I extracted the http
urls from the XML files and tested them with a little script for https
availability for each url.
The following URLs are available via HTTPS and therefore should be updated:
http://aidanfollestad.com/http://amlcurran.github.io/ShowcaseView/http://apache.orghttp://blog.zhanghai.me/http://chris.banes.me/http://commons.apache.org/proper/commons-csv/http://commons.apache.org/proper/commons-io/http://creativecommons.org/licenses/by/3.0/legalcodehttp://daimajia.comhttp://daimajia.com/http://developer.lge.com/http://developer.lge.com/MOBILE_HELP/index.jsp?topic=%2Flge.mobilesdk.LGQSlide.api%2Fhtml%2FDeveloping%2FDeveloping%2FDeveloping+with+LG+QSlide+SDK.htmhttp://facebook.github.io/rebound/http://fsf.org/http://gmariotti.blogspot.it/http://google.comhttp://h6ah4i.com/http://inthecheesefactory.com/blog/correct-imageview-adjustviewbounds-with-adjustable-imageview/enhttp://jakewharton.com/http://jakewharton.github.io/butterknife/http://jhy.io/http://joanzapata.com/android-iconify/http://jsoup.org/http://koush.com/http://ksoichiro.blogspot.pt/http://ksoichiro.github.io/Android-ObservableScrollView/http://libphonenumber.googlecode.com/svn/trunk/javascript/i18n/phonenumbers/demo-compiled.htmlhttp://luke.klinker.xyz/http://malinskiy.com/http://mikepenz.com/http://mixpanel.com/http://mixpanel.github.io/mixpanel-android/index.htmlhttp://nostra13android.blogspot.co.at/http://opensource.org/licenses/BSD-2-Clausehttp://opensource.org/licenses/BSD-3-Clausehttp://opensource.org/licenses/MIThttp://paymill.comhttp://roomorama.comhttp://satyan.github.io/sugar/http://scripts.sil.org/OFLhttp://site.mockito.org/http://square.github.io/http://square.github.io/dagger/http://square.github.io/okhttp/http://square.github.io/picasso/http://square.github.io/retrofit/http://stfalcon.com/http://su.chainfire.eu/http://svn.apache.org/viewvc/commons/proper/io/trunkhttp://tozny.com/http://trello.com/http://wasabeef.jp/http://www.akexorcist.com/http://www.apache.org/licenses/LICENSE-2.0http://www.apache.org/licenses/LICENSE-2.0.htmlhttp://www.avast.comhttp://www.daniel-stone.uk/http://www.devbrackets.com/http://www.eduardereza.com/http://www.facebook.com/http://www.freepik.comhttp://www.freepik.com/free-photos-vectors/peoplehttp://www.gnu.org/licenses/lgpl-3.0.en.htmlhttp://www.gotev.nethttp://www.jetbrains.orghttp://www.joanzapata.com/http://www.piwai.info/http://www.twitter.com/http://www.wix.com/amulyakhare/portfolio
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mikepenz/AboutLibraries/issues/416#issuecomment-433611032,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABaGiHZAxgfMjUDOfGEJm9i50s-VlmOSks5upD1KgaJpZM4XeH-9
.

--
Mike Penz

Mike Penz
Android | Backend | Open Source
mikepenz.com

You wrote:

I can't update all other urls though as it would require to check every single page if the https link would resolve.

I solved this problem for you. IMHO it is now up to you - I already prepared everything for you for fixing this vulnerability.

Don't take me wrong. I appreciate that you used your script to provide https links.
After all, this is an open source project. Feel free to open a PR to improve things. I am more than happy to merge, great additions. If you do not like the project itself you can also always fork and have your own fork with your own changes.
I do this all in my freetime. It takes a lot of time and effort to support multiple open source projects and keep them updated. Answering issues, helping people with questions, and hopefully improving.
All this beside a full time job which also requires a lot of time.


Allow JWT issuer to be configured

Hard-coding the JWT is not ideal, library-clients should be able to set this to whatever they like. It would be most-useful to set it from a ConferenceConfig which is loaded from JSON.
A multi-year conference, like MozFest, would like to set the JWT ISS to mozfest-2022 so that the client can discard tokens that are for the previous year's conference. This is because the conference is hosted on the same domain but JWTs that users have already stored are incompatible.

This was fixed and later tweaked in dd47c2451d8b36127fc1556836a576b9b318468e to allow a custom issuer to be set via the new jwtConfig


I cannot successfully install MatchZoo by running "python setup.py install"

It just went wrong when installing tensorflow and related packages, saying that "no local packages or working download links found for tensorflow"
Are you sure the command is bug-freed?

We have tested the installing by running "python setup.py install --user". Can you show the detail of the errors? By the way, you can install the dependencies by yourself, then, MatchZoo can work as normal.

Processing dependencies for MatchZoo==0.2.0
Searching for tensorflow-tensorboard<0.2.0,>=0.1.0
Reading https://pypi.python.org/simple/tensorflow-tensorboard/
No local packages or working download links found for tensorflow-tensorboard<0.2.0,>=0.1.0
error: Could not find suitable distribution for Requirement.parse('tensorflow-tensorboard<0.2.0,>=0.1.0')

This is the error


fixes

Audio stopped playing after verse 9 because the format changes for
requesting audio source.
i.e. this works - 01001003, but currently we would request this for
verse Genesis 1:10 010010010.  Instead of removing two numbers, we only
removed one as before.  Added bounds checks for 9 and 99.  Psalms 119
works like a charm :)

sorry duplicate


GET VANS

Dear Dev team,
Please can you add ..virtual_accounts/subaccounts/ method to your new release.
Thanks.

In PR #19

Fixed in v1.23.2


[BUG] Adding OCI Repositories to the Marketplace only works when specifying the full path to the chart

Rancher Server Setup

Rancher version: 2.9.1
Installation option: Helm Chart - RKE2 - 1.28

Information about the Cluster

Kubernetes version: 1.28
Cluster Type (Local/Downstream): Local

User Information

What is the role of the user logged in?: Admin

Describe the bug
When trying to add OCI Repository for Apps/Marketplace, the banner that gets displayed implies that you can use a parent directory as well as the path to where a specific chart (and all of it's versions) live.

For example oci://test.rancher.io/charts or oci://test.rancher.io/charts/mychart.

Only OCI URLs that point directly to a chart can be added successfully.
To Reproduce
Set up a Harbor instance that has a top level Project, helmcharts, that contains two different charts, rancher and harbor.

Attempt to add an OCI URL that points to 'helmcharts' will result in a Error 400: Bad Request

Adjusting the OCI URL to point to 'helmcharts/rancher' or 'helmcharts/harbor' successfully allows that specific application (and all of it's versions) be to available.

Result
Expected Result
We should be allowed to add a parent directory as an OCI URL or the documentation should be updated to reflect that it is not possible.

Same as issue #47073

Thanks for taking a look at this. I posted in #47073, but other tools have solved this problem by allowing the user to specify a string of repository paths where you can find a chart. This enables you to add the registry as the source, and then list the individuals charts you want queried as part of a single Repository entry. This makes it a little cleaner by not having to create a separate Repository for each chart.
Until there is a way to query a registry at a higher level for OCI charts, would it make sense to try and implement a similar feature?

I can certainly open a feature request. Looking at #47143, I agree with @bigkevmcd that the message presented to users will also have to be updated. Currently the functionality that is mentioned in the message isn't possible. What I am proposing is a way to bridge the gap between what can technically be accomplished and the functionality listed in the UI.

Do you have an example of a working repository from dockerhub? I tried oci://registry-1.docker.io/bitnamicharts but that failed with a 401, even after giving credentials. Specifying the full path to a specific chart, oci://registry-1.docker.io/bitnamicharts/common does work as expected.


request help: Can request get cookies？

Issue description
Can request get cookies？
Environment

your apisix-java-plugin-runner version

Yes, this is also part of the nginx variables, using cookie_*

I didn’t make it clear, I wanted to ask PluginFilter HttpRequest get cookie😄。
@Override
  public Mono<Void> filter(HttpRequest request, HttpResponse response, PluginFilterChain chain) {
     // request.getCookies();
}

I have solved it. According to the http protocol, the cookie is in the request header.
String cookie = request.getHeader("cookie");


feat: adds configure amtpassword local command

https://github.com/open-amt-cloud-toolkit/rpc-go/pull/441
./rpc configure amtpassword
./rpc configure amtpassword -password [current password]
./rpc configure amtpassword -newamtpassword [new password]
./rpc configure amtpassword -password [current password] -newamtpassword [new password]
Info that is not provided in a flag should be prompted for

Hi Craig, please address the following items:

Need to add the new command and examples of use to the configure help in the printConfigurationUsage function
I'd like to have the filename amtPass.go and amtPass_test.go spelled out completely amtPassword.go and amtPassword_test.go.
When I enter the wrong password for the existing AMT password, I get a dump of wsman (when not using -v or -l parameter).  Instead, we should have an error message that states that the AMT Password provided was incorrect.

Fixed the printConfigurationUsage function
Renamed the files
This functionality is consistent with main and across all configure commands. I can look into how to change it if you think this is an issue, but that would be a separate PR


LightGBM fit() issue with Scikit API

Environment info
Operating System: Linux
CPU/GPU model: aws ec2 r.4.16x large
C++/Python/R version: Python 3.6.8
LightGBM version or commit hash: 2.3.0
Error message
Sorry I am a bit of a noob but my code worked (and works) on the release 2.2.3. Passing in a pandas data frame with a combination of int,float, and categorical type columns as input and the below error is thrown. Have also attempted to manually convert the categoricals to int type and the error is still thrown.
ValueError                                Traceback (most recent call last)
<ipython-input-4-a5ada0ec0e79> in <module>
----> 1 a.run()

<ipython-input-2-ccd7a7fedf07> in run(self)
    250         self.prep_data()
    251         self.generate_validation_set()
--> 252         self.train()
    253         self.validate()
    254         self.save()

<ipython-input-2-ccd7a7fedf07> in train(self)
    126                     eval_set=[(x_test, y_test)],
    127                     eval_metric=['multiclass_ova','multi_error'],
--> 128                     early_stopping_rounds=4)
    129
    130     def validate(self):

~/.local/lib/python3.6/site-packages/lightgbm/sklearn.py in fit(self, X, y, sample_weight, init_score, group, eval_set, eval_names, eval_sample_weight, eval_class_weight, eval_init_score, eval_group, eval_metric, early_stopping_rounds, verbose, feature_name, categorical_feature, callbacks)
    593                               verbose_eval=verbose, feature_name=feature_name,
    594                               categorical_feature=categorical_feature,
--> 595                               callbacks=callbacks)
    596
    597         if evals_result:

~/.local/lib/python3.6/site-packages/lightgbm/engine.py in train(params, train_set, num_boost_round, valid_sets, valid_names, fobj, feval, init_model, feature_name, categorical_feature, early_stopping_rounds, evals_result, verbose_eval, learning_rates, keep_training_booster, callbacks)
    226     # construct booster
    227     try:
--> 228         booster = Booster(params=params, train_set=train_set)
    229         if is_valid_contain_train:
    230             booster.set_train_data_name(train_data_name)

~/.local/lib/python3.6/site-packages/lightgbm/basic.py in __init__(self, params, train_set, model_file, model_str, silent)
   1662             self.handle = ctypes.c_void_p()
   1663             _safe_call(_LIB.LGBM_BoosterCreate(
-> 1664                 train_set.construct().handle,
   1665                 c_str(params_str),
   1666                 ctypes.byref(self.handle)))

~/.local/lib/python3.6/site-packages/lightgbm/basic.py in construct(self)
   1035                                 init_score=self.init_score, predictor=self._predictor,
   1036                                 silent=self.silent, feature_name=self.feature_name,
-> 1037                                 categorical_feature=self.categorical_feature, params=self.params)
   1038             if self.free_raw_data:
   1039                 self.data = None

~/.local/lib/python3.6/site-packages/lightgbm/basic.py in _lazy_init(self, data, label, reference, weight, group, init_score, predictor, silent, feature_name, categorical_feature, params)
    849                 raise TypeError('Cannot initialize Dataset from {}'.format(type(data).__name__))
    850         if label is not None:
--> 851             self.set_label(label)
    852         if self.get_label() is None:
    853             raise ValueError("Label should not be None")

~/.local/lib/python3.6/site-packages/lightgbm/basic.py in set_label(self, label)
   1334         self.label = label
   1335         if self.handle is not None:
-> 1336             label = list_to_1d_numpy(_label_from_pandas(label), name='label')
   1337             self.set_field('label', label)
   1338             self.label = self.get_field('label')  # original values can be modified at cpp side

~/.local/lib/python3.6/site-packages/lightgbm/basic.py in list_to_1d_numpy(data, dtype, name)
     80     elif isinstance(data, Series):
     81         if _get_bad_pandas_dtypes([data.dtypes]):
---> 82             raise ValueError('Series.dtypes must be int, float or bool')
     83         return np.array(data, dtype=dtype, copy=False)  # SparseArray should be supported as well
     84     else:

ValueError: Series.dtypes must be int, float or bool

Reproducible examples
my input data structure (of type pandas.core.frame.DataFrame)
self.data.dtypes
feat1 int64
feat2 category
feat3 category
feat4 category
feat5 float64
feat6 int64

code:
    def train(self):
        '''train the gradient boosting model'''
        # split the input data into train and test sets
        # https://scikit-learn.org/stable/modules/generated/sklearn.model_selection.train_test_split.html
        print('creating a test and train set')
        self.seed = randint(0, 10**5)
        print('train test split seed = {}'.format(self.seed))
        x_train, x_test, y_train, y_test = train_test_split(self.data,
                                                            self.y,
                                                            train_size=.7,
                                                            test_size =.3,
                                                            random_state=self.seed,
                                                            stratify=self.data.feat3)

        # instantiate  lightgbm model
        # https://lightgbm.readthedocs.io/en/latest/index.html
        self.model = LGBMModel(objective='multiclass',
                            subsample=.7,
                            silent=False,
                            num_classes=12,
                            boosting_type='gbdt',
                            n_estimators=500,
                            reg_lambda=1,
                            max_depth=25,
                            num_leaves=700,
                            learning_rate=.1,
                            min_child_samples=150)

        # train the model on the data, stop early using hold out set
        print('training the model on the following features {}'.format(', '.join(self.data.columns)))
        self.model.fit(X=x_train,
                    y=y_train,
                    eval_set=[(x_test, y_test)],
                    eval_metric=['multiclass_ova','multi_error'],
                    early_stopping_rounds=4)

Steps to reproduce

my self.data is of type pandas.core.frame.DataFrame which schema as described above.
instantiate model object, run fit method.
this logic works in 2.2.3 but no longer works in 2.3.0

@yoshuae Error comes from the y, not X. Label is allowed to be only int, float or bool dtype. 2.3.0 started to check  label dtypes.

@StrikerRUS
It seems the label is transformed in https://github.com/microsoft/LightGBM/blob/master/python-package/lightgbm/sklearn.py#L759
Should it be int type after the transformation?

@guolinke Yes, it should, I think. But only in LGBMClassifier, not in LGBMModel (which @yoshuae uses).

I'll give this a check, thank you for following up.

@yoshuae Ah, make sense! We support int64 dtype, but not Int64 one.

Note IntegerArray is currently experimental. Its API or implementation may change without warning.

Or the string alias "Int64" (note the capital "I", to differentiate from NumPy’s 'int64' dtype:
https://pandas.pydata.org/pandas-docs/stable/user_guide/integer_na.html


Remove python compiled files

Update gitignore to cover these files, and removed it from the repository.

Thanks!!


Add support for Serene Seasons

Allows corn crops to be grown seasonally with the Serene Seasons mod

Thank you!

Any chance this could be integrated into the 1.16.5 branch? I'm not seeing an updated file on Curse for 1.16 :(
Or would you prefer to leave it and a datapack can just be linked?

It should be fairly trivial to perform a merge commit from main to the 1.16.5 branch, so I'll leave that to you.
In the meantime, here's a datapack that should work in that version: seasonal_corn.zip
Enjoy!

Well, 1.16.5 Updated to CF.

Well, 1.16.5 Updated to CF.

Im confused - CF?

CF is CurseForge, which means the update should be live. Thank you for the hasty progress!


Set TasksMax in addition to LimitNPROC in systemd service files

systemd sets an additional limit on processes and threads that defaults to 512 when run under Linux >= 4.3 (see here). The problem has been described in #9868 and #19124.
This patch sets the TasksMax in the example systemd service files provided with docker.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "systemd_tasks_max" git@github.com:cdauth/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

ping @jfrazelle because you're our systemd master now :smile:

This is controlling the pids cgroup, it seems. So not related to #9868 it would seem.
This should be documented as to what it's doing, because it will limit the number of containers someone can spin up and limit the number of processes a container can spin up.

I'm going to set the 1.10 milestone as the current behavior will be horrible for anyone on the newer kernel.

LGTM

LGTM

@dandric-activevideo, you are absolutely right, Do you have any alternate solution to set TasksMax for CentOS 7?

@dandric-activevideo: I’m not sure that’s relevant. This commit only affects the systemd example service files. It is up to the package maintainers what service files they actually distribute.
@mcmohd: In systemd before 228 there is no need to set it, as the task limit is not implemented yet. It is basically always set to Infinite. If you want to limit the number of tasks, use the LimitNPROC (although I think it will be per user, not per service).