Literate CoffeeScript files crashing with an "unmatched parenthesis" error

Started randomly getting this problem last night, did a clean reinstall of Atom and it still persists. These files work after they are hand-edited and converted to regular .coffee. This same problem has previously occurred with PHP.
Steps to reproduce:

open a Literate CoffeeScript file with the .litcoffee extension

Atom Version: 1.2.3
System: Mac OS X 10.10.5
Thrown From: Atom Core
Stack Trace
Uncaught Error: end pattern with unmatched parenthesis
At /Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:770

Error: end pattern with unmatched parenthesis
    at Error (native)
    at Scanner.module.exports.Scanner.createScanner (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/first-mate/lib/scanner.js:31:24)
    at Scanner.module.exports.Scanner.getScanner (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/first-mate/lib/scanner.js:37:31)
    at Scanner.module.exports.Scanner.findNextMatch (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/first-mate/lib/scanner.js:56:22)
    at Rule.module.exports.Rule.findNextMatch (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/first-mate/lib/rule.js:99:28)
    at Rule.module.exports.Rule.getNextTags (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/first-mate/lib/rule.js:141:21)
    at Grammar.module.exports.Grammar.tokenizeLine (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/first-mate/lib/grammar.js:150:44)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLineForRowWithText (/Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:546:28)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLineForRow (/Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:532:19)
    at TokenizedBuffer.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:485:34)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLinesForRows (/Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:494:10)
    at TokenizedBuffer.module.exports.TokenizedBuffer.handleBufferChange (/Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:376:34)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:69:24
    at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/event-kit/lib/emitter.js:86:11)
    at TextBuffer.module.exports.TextBuffer.applyChange (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:645:20)
    at TextBuffer.module.exports.TextBuffer.revertToCheckpoint (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:799:16)

Commands
     -0:10.7.0 core:save (atom-text-editor.editor.is-focused)
     -0:01.2.0 snippets:next-tab-stop (atom-text-editor.editor.is-focused)
     -0:01.2.0 snippets:expand (atom-text-editor.editor.is-focused)
     -0:01.2.0 editor:indent (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "solarized-dark-syntax"
    ]
  },
  "editor": {
    "invisibles": {}
  }
}

Installed Packages
# User
atom-beautify, v0.28.19
minimap, v4.16.1
open-recent, v4.0.1

# Dev
No dev packages

This has already been fixed and should be out in the next 1.2 patch release.  If you need the fix now, execute apm install language-coffee-script from the command line.


Updates stf-run-ci setup playbook to use cert-manager in stable-v1

For newer OCP versions (OCP >= 4.12), cert-manager should be consumed from the stable-v1 channel. This differs from previous versions of OCP, in which we were consuming cert-manager from the tech-preview channel.
The way of installing both operators is different (the tech-preview requires a specific namespace and operator group, whereas the stable-v1 version does not).
This change adds a conditional block checking the versions of OCP in which we are running this playbook and executing the installation steps that correspond which each version.

Ran smoke tests on an environment with cert-manager from stable-v1 installed in the way that it is being installed in this patch set to verify sanity of the deployment, it is passing.
Some validation of the creation of the subscription should be added, is https://github.com/infrawatch/service-telemetry-operator/blob/master/build/validate_deployment.sh a good place for this? cc @csibbitt @leifmadsen @elfiesmelfie

Ran smoke tests on an environment with cert-manager from stable-v1 installed in the way that it is being installed in this patch set to verify sanity of the deployment, it is passing.

:+1:

Some validation of the creation of the subscription should be added, is https://github.com/infrawatch/service-telemetry-operator/blob/master/build/validate_deployment.sh a good place for this? cc @csibbitt @leifmadsen @elfiesmelfie

I think if the cert-manager doesn't work, then validate_deployment.sh will fail while waiting for the QDR to deploy.

test

There was some side discussion that we may want to pin testing to the tech-preview channel in all cases where it's available, since that's what we'd expect people to be using if they installed following the docs. If that's the case then we might want to keep this structure and just adjust the versions?

Considering we will stick to tech-preview for STF up to 1.5.2 we should

Drop this change, since we still have cert-manager from tech-preview channel available in OCP 4.12
Revert https://github.com/infrawatch/service-telemetry-operator/pull/424

For OCP 4.13 we will add dependency resolution and customers are no longer expected to pre-install cert-manager. We will need to reflect those changes in our CI. For that case, the steps followed will look like

Create namespace
Create operatorgroup
Create subscription to STO
Verify all dependencies are installed (SGO, cert-manager, AMQ Interconnect, OBO)

Related PR https://github.com/infrawatch/service-telemetry-operator/pull/444
Will close this PR once we agreed on the approach


Stacking lib: add reward methods

The API returns burnchain rewards now: https://blockstack.github.io/stacks-blockchain-api/#tag/Burnchain.
The stacking library could provide methods to get the rewards for a set address

@marcosc90 @zone117x what are your thoughts on this? I was thinking this could be a new method inside the stacking lib, eg getRewardsForBtcAddress()

Make's sense to me 👍
@marcosc90 is this something you can take on?

I have started working on this issue. Assigning it to myself.


info/steps option

Add an option to get the list of known and possible steps, possibly also all the command line options that are possible to specify for each step

Fixed by 08821e226e8f5bdfc254850aa649323a33a8c022. Implemented as an env command-line function


Prefix in Apartmentex.drop_tenant not being escaped properly?

I'm trying to get apartmentex working for me. I'm assuming I'm not doing anything too crazy :)
I'm using "slugs" as the tenant identifier on my Postgres schemas (I'm not using integer IDs, so would end up using a UUID, or a slug).  So for example, my database schemas look like tenant_a-test-person.
Adding a tenant is working just fine. For example:
Apartmentex.new_tenant(Repo, client.slug)
# --> Apartmentex.new_tenant(Repo, "a-test-person")

However, when I go to drop the schema, I'm getting errors due to the presence of - in the schema name.  For example:
Apartmentex.drop_tenant(Repo, client.slug)
# --> Apartmentex.drop_tenant(Repo, "a-test-person")

This ends up calling DROP SCHEMA tenant_test-person CASCADE [] in the DB, and throws an error, as per below:
[debug] QUERY ERROR db=0.5ms
DROP SCHEMA tenant_test-person CASCADE []
[debug] QUERY OK db=0.2ms
rollback []
{:error, :drop_schema,
 %Postgrex.Error{connection_id: 70573, message: nil,
  postgres: %{code: :syntax_error, file: "scan.l", line: "1053",
    message: "syntax error at or near \"-\"", pg_code: "42601", position: "24",
    routine: "scanner_yyerror", severity: "ERROR"}}, %{}}

I think all that needs to happen here is surround the schema name in quotes, e.g. DROP SCHEMA "tenant_test-person" CASCADE [] - at least it worked for me testing in a PG shell.
Is there a way to accomplish this with the library as-is?  I don't think using UUIDs (instead of a slug) would make any difference here, as they also include - characters.
Thanks for your help and putting this library together!

there's a pull request which fixes this apparently, haven't tried it just pointing it out :)
https://github.com/Dania02525/apartmentex/pull/25

@brandonparsons The PR referenced by @mhussa has been merged and published to hex- please change your deps entry to -> 0.2.2.  Let me know if this issue resolves for you.  Thanks!

@Dania02525 Whoo hoo! Yep it works. I obviously didn't know you guys were already all over this :)
Again, thanks for putting this library together. I'm just getting into Elixir/Phoenix, and wondering about converting a reasonably young Rails app over. If this library didn't exist, it wouldn't even be an option for me :) Will see if I can get up to speed fast enough!

@brandonparsons as of Ecto 2 you thankfully don't actually need this library. However, the added sugar that Apartmentex supplies is very nice and saves you from learning too much advanced Ecto before one's ready.


Auto-layout constraint warnings (re-open of #322)

Hi guys, this happens in version 6.1.3, iOS 8.1.2, storyboard+size classes sometimes when the view render messages
2015-02-24 16:34:52.289 xxx[9544:274546] Unable to simultaneously satisfy constraints.
    Probably at least one of the constraints in the following list is one you don't want. Try this: (1) look at each constraint and try to figure out which you don't expect; (2) find the code that added the unwanted constraint or constraints and fix it. (Note: If you're seeing NSAutoresizingMaskLayoutConstraints that you don't understand, refer to the documentation for the UIView property translatesAutoresizingMaskIntoConstraints)
(
    "<NSLayoutConstraint:0x7f885acd7550 V:[JSQMessagesLabel:0x7f885acd7200'Today 4:29 PM'(20)]>",
    "<NSLayoutConstraint:0x7f885acdc420 V:[JSQMessagesLabel:0x7f885acdc0d0'Tony Parker has invited F...'(20)]>",
    "<NSLayoutConstraint:0x7f885acf9fc0 V:|-(0)-[JSQMessagesLabel:0x7f885acd7200'Today 4:29 PM']   (Names: '|':UIView:0x7f885acd7130 )>",
    "<NSLayoutConstraint:0x7f885acfa0b0 V:[JSQMessagesLabel:0x7f885acd7200'Today 4:29 PM']-(0)-[JSQMessagesLabel:0x7f885acdc0d0'Tony Parker has invited F...']>",
    "<NSLayoutConstraint:0x7f885acfa150 V:[JSQMessagesLabel:0x7f885acdc0d0'Tony Parker has invited F...']-(0)-[UIView:0x7f885acdcab0]>",
    "<NSLayoutConstraint:0x7f885acfa1f0 V:[UIView:0x7f885acdcab0]-(0)-[JSQMessagesLabel:0x7f885acf96c0]>",
    "<NSLayoutConstraint:0x7f885acfa290 V:[JSQMessagesLabel:0x7f885acf96c0]-(0)-|   (Names: '|':UIView:0x7f885acd7130 )>",
    "<NSAutoresizingMaskLayoutConstraint:0x7f885afe5f20 h=--& v=--& V:[UIView:0x7f885acd7130(20)]>"
)

Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x7f885acdc420 V:[JSQMessagesLabel:0x7f885acdc0d0'Tony Parker has invited F...'(20)]>

hey @federicocappelli - size classes are not fully support as this library is iOS 7+

ok thanks @jessesquires any plans for future releases?

@federicocappelli - definitely. #499

Marking this as a duplicate of #322 and continue tracking that issue here.

For record-keeping purposes, I'm not using size classes, and my subclass of JSQMessagesViewController doesn't touch a Storyboard.

@harlanhaskins - can you provide steps or show how to repro with the demo project?

@jessesquires I'll pull out our messaging code into a small demo and post it here with some dummy data.

I've tried a bit to pull my messaging code out, but it's complex and unfortunately fairly coupled to the rest of my implementation.
I see the Auto Layout issues usually when adding messages,
func messageGroup(group: MessageGroup, didAddMessages messages: [Message]) {
    // Build a list of valid index paths from the messages provided, by throwing out
    // messages that don't exist in the list.
    let paths: [NSIndexPath] = mapMaybe(messages) { message in
        if let index = find(group.messages, message) {
            return NSIndexPath(forItem: index, inSection: 0)
        }
        return nil
    }
    self.collectionView?.performBatchUpdates({
        self.collectionView?.insertItemsAtIndexPaths(paths)
        return
    }, completion: { finished in
        self.finishReceivingMessages()
   })
}

@jessesquires Hi Jesse, I may have found cause of the constraint troubles... it seems that placing this (little bit ugly and hacky) line
[self.contentView setBounds:CGRectMake(0, 0, CGRectGetWidth(self.contentView.frame), 15000)];

15000 - some big number ==> temporary cell height
right above
[self jsq_updateConstraint:self.messageBubbleContainerWidthConstraint
withConstant:customAttributes.messageBubbleContainerViewWidth];
inside method:
- (void)applyLayoutAttributes:(UICollectionViewLayoutAttributes *)layoutAttributes
fixes constraints conflicts (at least in our app). Apparently constraint conflict pops up when you update cell with new for example customAttributes.cellTopLabelHeight and the sum of current updated constraint constants is bigger than incoming dequeued cells bounds.height which makes constraint gods confused.
Setting bounds.height to some big number takes care of the problem because there is a lot of space during constraint updates. Then, bounds are properly updated during collection view cell update process.
There might be some more elegant solution but this seems to work. ;-)
(this comment gave me a hint: https://github.com/Alex311/TableCellWithAutoLayout/commit/bde387b27e33605eeac3465475d2f2ff9775f163#commitcomment-4633188)

It looks like it's related to NSAutoresizingMaskLayoutConstraints. It doesn't look like [self.contentView setTranslatesAutoresizingMasksIntoConstraints:NO]; is happening anywhere.

@harlanhaskins I am experiencing this autolayout issue when scrolling quicky, and noticed you mentionned that occured also to you when scrolling in another thread. Have you found a solution for the scrolling error? I am not using storyboard or xib.

It definitely happens when I scroll quickly. When I do a load earlier messages call and I get about 30 messages, self.collectionView.reloadData() causes all these warnings. So it's gotta be related to rapid cell insertion.

@stefanocdn @harlanhaskins: Hi, have you tried the trick with bounds height mentioned above?

@jardamach I can confirm that this helped with the constraint warnings. I was seeing them for the same reason @harlanhaskins was (rapid cell insertions/scrollToBottom).
Nice fix, thanks.

Thanks @jardamach for the fix. Worked for me.

I take that back. Reloading items now leads to weird flickering.

Did you guys managed to fix this? Getting layout constraints warnings, when I have different heights of top labels for individual cells. No warnings, if they are all the same.

@jardamach  Your solution works for me. Thank you for that.

I just triggered this problem inadvertently and it took me a good couple hours to figure out the cause. While debugging an autolayout bug, I mistakenly turned off autoresizing in the navigation controller:
self.navigationController.view.translatesAutoresizingMaskIntoConstraints = NO;
Setting translatesAutoresizingMaskIntoConstraints = NO in the JSQMessagesViewController class prevents the NSLayoutConstraints exception, but wrecks the app by helpfully re-sizing everything in the view hierarchy to a width of 0.
I can reproduce the problem in the demo project by adding the same line to TableViewController:viewDidLoad. Of all the views in my app, this only happens with JSQMessagesViewController but I don't know if it is a bug, or just a "don't do that".
Anyhow, if anyone else has this bug, I hope this info helps.

this only happens with JSQMessagesViewController but I don't know if it is a bug, or just a "don't do that".

ha, i think this is "just don't do that" 😄 we want translatesAutoresizingMaskIntoConstraints = YES in this case.

I have a similar issue when I return different heights in labels height methods.

@jardamach trick worked. Do I have to create a fork to use it or there's a "real" plan to fix this issue without hacks? I can create the PR with the hack but not sure if resetting bounds before setting constraints could break any other thing or cause performance problems.

@jardamach found an issue with this approach, when I add a new cell and call finishSending method and it triggers collectionView.reloadData and layout goes crazy

Changing code on finishSending to avoid reloadData and insert row instead fixes the issue. I've also added a check to avoid the hack when expected layout is smaller than current (when this hack is not needed)

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


Make Format Definition page prettier

Instead of just displaying the json, create a view-rendered version of the json, maybe a nice list of structured lists or something similar. Maybe use icons for some of the contents of each attribute definition object.

Made the page a little bit nicer. Not too great through. branch change-actions


Convert README.rst from markdown to reStructuredText

so that it renders on GitHub 😉

Thanks for the PR, but the content of that file was just temporary at that point and it has now been completely replaced with other content.


does anyone know where the "Start typing..." is in which file

does someone know which file it is in, because i when throw all of the them.

\src\modules\launcher\PowerLauncher\LauncherControl.xaml
This actually needs to be moved to a resource file
toward end of file
        <TextBox
            x:Name="QueryTextBox"
            x:FieldModifier="public"
            Canvas.ZIndex="0"
            Margin="10, 0, 0, 0"
            VerticalAlignment="Center"
            FontSize="24"
            Style="{StaticResource QueryTextBoxStyle}"
            Tag="Start typing..."
            />


error

Message: Undefined index: details
File: /includes/GeneralFunctions.php
Line: 81
URL: http://final-space.com/game.php
PHP-Version: 7.3.27
PHP-API: litespeed
Version: 1.2.0
Debug Backtrace:
#0 /includes/GeneralFunctions.php(81): errorHandler(8, 'Undefined index...', 'FILEPATH ...', 81, Array)
#1 /includes/common.php(194): getFactors(Array)
#2 /game.php(26): require('FILEPATH ...')
#3 {main}

Did you install the clean version or update yourself? Try to clear your cache.

I had no problem installing the package from the branch master. Are you sure you did everything right?

after cache cleaned its ok now

you can close thanks


Angular 2 integration using typescript

https://stackoverflow.com/questions/44987260/how-to-add-jstree-to-angular-2-application-using-typescript-with-types-jstree?noredirect=1#comment76955233_44987260

I am not familiar with angular, but there are a few good jstree wrapper directives as far as I know - have a look at them.

hi,
i have to need tree strcture to use the angular5


Error on after_database_authentication at users/invitation path

Backtrace
View full backtrace and more info at honeybadger.io
DPUL shouldn't have database authentication or user invitations. There may be something we need to disable.

To replicate:
Add an external email address as a user in a collection's configuration.
If that user hits the "invitation instructions" link, the site errors.
@kelea99 have you heard people have this issue? Can you give any more context?


Latest NPM release (19.0.0) still errors out with Vuetify 2.0

Wondering if the v19.0.0 includes the compatibility update for Vuetify 2.0? I am still getting errors after upgrading. I am using Vuetify 2.0.6.
[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.

Looks like you are using a-la-carte.
https://vuetifyjs.com/en/customization/a-la-carte
I guess a lot of people are using this. I will need to rework a few things. You must then import the component via import VImageInput from 'vuetify-image-input/a-la-carte' in the next version.

a-la-carte is threeshaking.
It's supported now.
import VImageInput from 'vuetify-image-input/a-la-carte';

Ok installed the latest version and the error for VImageInput is gone however now I get errors for more components.
[Vue warn]: Unknown custom element:
[Vue warn]: Unknown custom element:
[Vue warn]: Unknown custom element:
[Vue warn]: Unknown custom element:
All of these components are used in my application's single file templates.
While manual a-la-carte provides the same results as automatic tree shaking with Vuetify loader. It appears that Vuetify loader cannot resolved functional components automatically and I believe this is how Vuetify Image Input is written. Even if the package imports components manually I am suspecting that they will be included twice in the project increasing the bundle size (trying to research this now).
See the note about functional components below. (Recommended import is global import)
https://vuetifyjs.com/en/customization/a-la-carte


Create Separate Admin Script

Separate out admin functionality into its own script, rather than being a part of app.js.

Fixed in 3159a302f0209478d67c9e380bb0fde1e2819531.


docker-machine upgrade google compute instance failed

Greetings,
tried to upgrade my google instance with docker-machine upgrade dev01 from 1.9.1 to 1.10 but failed, here is the debug log
-> % dm ls
NAME    ACTIVE   DRIVER       STATE     URL                          SWARM   DOCKER    ERRORS
dev     *        virtualbox   Running   tcp://<IP_ADDRESS>:2376            v1.10.0
dev01   -        google       Running   tcp://xxx.xxx.xxx.xxx:2376           v1.9.1

-> % dm --debug upgrade dev01
Docker Machine Version:  0.6.0, build e27fb87
Found binary path at /usr/local/bin/docker-machine
Launching plugin server for driver google
Plugin server listening at address <IP_ADDRESS>:58715
() Calling .GetVersion
Using API Version  1
() Calling .SetConfigRaw
() Calling .GetMachineName
command=upgrade machine=dev01
(dev01) Calling .GetState
Waiting for SSH to be available...

Building dependency tree...
Reading state information...
Package 'lxc-docker' is not installed, so not removed
0 upgraded, 0 newly installed, 0 to remove and 33 not upgraded.

package: action=upgrade name=docker-engine
(dev01) Calling .GetSSHHostname
(dev01) Calling .GetSSHPort
(dev01) Calling .GetSSHKeyPath
(dev01) Calling .GetSSHKeyPath
(dev01) Calling .GetSSHUsername
Using SSH client type: external
{[-o BatchMode=yes -o PasswordAuthentication=no -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -o LogLevel=quiet -o ConnectionAttempts=3 -o ConnectTimeout=10 -o ControlMaster=no -o ControlPath=none docker-user@ -o IdentitiesOnly=yes -i /Users/jeanepaul/.docker/machine/machines/dev01/id_rsa -p 22] /usr/bin/ssh}
About to run SSH command:
DEBIAN_FRONTEND=noninteractive sudo -E apt-get install -y  docker-engine
SSH cmd err, output: <nil>: Reading package lists...
Building dependency tree...
Reading state information...
docker-engine is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 33 not upgraded.

Restarting docker...
(dev01) Calling .GetSSHHostname
(dev01) Calling .GetSSHPort
(dev01) Calling .GetSSHKeyPath
(dev01) Calling .GetSSHKeyPath
(dev01) Calling .GetSSHUsername
Using SSH client type: external
{[-o BatchMode=yes -o PasswordAuthentication=no -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -o LogLevel=quiet -o ConnectionAttempts=3 -o ConnectTimeout=10 -o ControlMaster=no -o ControlPath=none docker-user@ -o IdentitiesOnly=yes -i /Users/jeanepaul/.docker/machine/machines/dev01/id_rsa -p 22] /usr/bin/ssh}
About to run SSH command:
sudo systemctl daemon-reload
SSH cmd err, output: <nil>:
(dev01) Calling .GetSSHHostname
(dev01) Calling .GetSSHPort
(dev01) Calling .GetSSHKeyPath
(dev01) Calling .GetSSHKeyPath
(dev01) Calling .GetSSHUsername
Using SSH client type: external
{[-o BatchMode=yes -o PasswordAuthentication=no -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -o LogLevel=quiet -o ConnectionAttempts=3 -o ConnectTimeout=10 -o ControlMaster=no -o ControlPath=none docker-user@ -o IdentitiesOnly=yes -i /Users/jeanepaul/.docker/machine/machines/dev01/id_rsa -p 22] /usr/bin/ssh}
About to run SSH command:
sudo systemctl -f restart docker
SSH cmd err, output: <nil>:
(dev01) Calling .GetConfigRaw
Making call to close driver server
(dev01) Calling .Close
Successfully made call to close driver server
Making call to close connection to plugin binary
(dev01) DBG | Closing plugin on server side

tried ssh and reboot the machine, but to no avail..
root@dev01:~# docker --version
Docker version 1.9.1, build a34a1d5

created the google machine with a script that I created
#!/bin/bash

$NAME=${NAME:-google-machine-01}

docker-machine create --debug --driver google \
  --google-project $PROJECT_ID \
  --google-zone $MACHINE_ZONE \
  --google-machine-type $MACHINE_TYPE \
  --google-disk-size $MACHINE_DISK_SIZE \
  $NAME

any news on this? had to downgrade my client to 1.9 because I cant create/run containers...

#3009 similar issue being exhibited in Ubuntu LTS 14.04

If you give it some time and re-run the upgrade (or daemon/service reload), does it work?  1.10 had a content addressability migration that I suspect might be causing issues.

Greetings,
I don't get what you mean by give it some time.. could you be more specific? machine upgrade dev01 then?
I will update my post on how did I created the machine..

Please try https://github.com/docker/machine/issues/3308#issuecomment-210181219, and open a new issue if that does not help you upgrade.

dupe from https://github.com/docker/machine/issues/3314

Found my way here because I'm trying to upgrade 1.10.3 to 1.11.1b but the docker-machine is centos image running on GCE so script above won't work and upgrade fails


[Bug] dash behind subfolder nginx proxy

Description of the bug
As shown here: https://homarr.dev/docs/widgets/dashdot-widget?
Is the an environment config where i can set the base path of the app?
How to reproduce
No response
Relevant log output
No response
Info output of dashdot cli
No response
What browsers are you seeing the problem on?
No response
Where is your instance running?
No response
Additional context
No response

Duplicate of #450


rename challenge to evaluation

I used the challenge section of the docs to brainstorm. Since there are actually challenges, it is not important right now to write them down. Thinking about the difference and keeping in mind that it is all about getting to somewhere, I don't care about challenge documentation. I care about problem solving and for this I need to evaluate ideas.

Fixed by #47


TypeError - class or module required for rescue clause

TypeError - class or module required for rescue clause:
qiniu-rs (1.1.0) lib/qiniu/rs/auth.rb:64:in rescue in request'   qiniu-rs (1.1.0) lib/qiniu/rs/auth.rb:62:in request'
qiniu-rs (1.1.0) lib/qiniu/rs/rs.rb:27:in publish'   qiniu-rs (1.1.0) lib/qiniu/rs.rb:95:in publish'
carrierwave-qiniu (<IP_ADDRESS>) lib/carrierwave/storage/qiniu.rb:88:in setup_publish_bucket_and_domain'   carrierwave-qiniu (<IP_ADDRESS>) lib/carrierwave/storage/qiniu.rb:75:in init'
carrierwave-qiniu (<IP_ADDRESS>) lib/carrierwave/storage/qiniu.rb:27:in initialize'   carrierwave-qiniu (<IP_ADDRESS>) lib/carrierwave/storage/qiniu.rb:142:in qiniu_connection'
carrierwave-qiniu (<IP_ADDRESS>) lib/carrierwave/storage/qiniu.rb:112:in store'   carrierwave-qiniu (<IP_ADDRESS>) lib/carrierwave/storage/qiniu.rb:150:in store!'
carrierwave (0.11.2) lib/carrierwave/uploader/store.rb:59:in block in store!'   carrierwave (0.11.2) lib/carrierwave/uploader/callbacks.rb:17:in with_callbacks'
carrierwave (0.11.2) lib/carrierwave/uploader/store.rb:58:in store!'   carrierwave (0.11.2) lib/carrierwave/mount.rb:375:in store!'
carrierwave (0.11.2) lib/carrierwave/mount.rb:207:in store_file!'   activesupport (4.2.6) lib/active_support/callbacks.rb:432:in block in make_lambda'
activesupport (4.2.6) lib/active_support/callbacks.rb:228:in block in halting_and_conditional'   activesupport (4.2.6) lib/active_support/callbacks.rb:506:in block in call'
activesupport (4.2.6) lib/active_support/callbacks.rb:506:in call'   activesupport (4.2.6) lib/active_support/callbacks.rb:92:in run_callbacks'
activesupport (4.2.6) lib/active_support/callbacks.rb:778:in _run_save_callbacks'   activerecord (4.2.6) lib/active_record/callbacks.rb:302:in create_or_update'
activerecord (4.2.6) lib/active_record/persistence.rb:120:in save'   activerecord (4.2.6) lib/active_record/validations.rb:37:in save'
activerecord (4.2.6) lib/active_record/attribute_methods/dirty.rb:21:in save'   activerecord (4.2.6) lib/active_record/transactions.rb:286:in block (2 levels) in save'
activerecord (4.2.6) lib/active_record/transactions.rb:351:in block in with_transaction_returning_status'   activerecord (4.2.6) lib/active_record/connection_adapters/abstract/database_statements.rb:213:in block in transaction'
activerecord (4.2.6) lib/active_record/connection_adapters/abstract/transaction.rb:184:in within_new_transaction'   activerecord (4.2.6) lib/active_record/connection_adapters/abstract/database_statements.rb:213:in transaction'
activerecord (4.2.6) lib/active_record/transactions.rb:220:in transaction'   activerecord (4.2.6) lib/active_record/transactions.rb:348:in with_transaction_returning_status'
activerecord (4.2.6) lib/active_record/transactions.rb:286:in block in save'   activerecord (4.2.6) lib/active_record/transactions.rb:301:in rollback_active_record_state!'
activerecord (4.2.6) lib/active_record/transactions.rb:285:in save'   app/controllers/admin/videos_controller.rb:11:in block in create'
actionpack (4.2.6) lib/action_controller/metal/mime_responds.rb:212:in respond_to'   app/controllers/admin/videos_controller.rb:10:in create'
actionpack (4.2.6) lib/action_controller/metal/implicit_render.rb:4:in send_action'   actionpack (4.2.6) lib/abstract_controller/base.rb:198:in process_action'
actionpack (4.2.6) lib/action_controller/metal/rendering.rb:10:in process_action'   actionpack (4.2.6) lib/abstract_controller/callbacks.rb:20:in block in process_action'
activesupport (4.2.6) lib/active_support/callbacks.rb:117:in call'   activesupport (4.2.6) lib/active_support/callbacks.rb:555:in block (2 levels) in compile'
activesupport (4.2.6) lib/active_support/callbacks.rb:505:in call'   activesupport (4.2.6) lib/active_support/callbacks.rb:92:in run_callbacks'
activesupport (4.2.6) lib/active_support/callbacks.rb:778:in _run_process_action_callbacks'   activesupport (4.2.6) lib/active_support/callbacks.rb:81:in run_callbacks'
actionpack (4.2.6) lib/abstract_controller/callbacks.rb:19:in process_action'   actionpack (4.2.6) lib/action_controller/metal/rescue.rb:29:in process_action'
actionpack (4.2.6) lib/action_controller/metal/instrumentation.rb:32:in block in process_action'   activesupport (4.2.6) lib/active_support/notifications.rb:164:in block in instrument'
activesupport (4.2.6) lib/active_support/notifications/instrumenter.rb:20:in instrument'   activesupport (4.2.6) lib/active_support/notifications.rb:164:in instrument'
actionpack (4.2.6) lib/action_controller/metal/instrumentation.rb:30:in process_action'   actionpack (4.2.6) lib/action_controller/metal/params_wrapper.rb:250:in process_action'
activerecord (4.2.6) lib/active_record/railties/controller_runtime.rb:18:in process_action'   actionpack (4.2.6) lib/abstract_controller/base.rb:137:in process'
actionview (4.2.6) lib/action_view/rendering.rb:30:in process'   actionpack (4.2.6) lib/action_controller/metal.rb:196:in dispatch'
actionpack (4.2.6) lib/action_controller/metal/rack_delegation.rb:13:in dispatch'   actionpack (4.2.6) lib/action_controller/metal.rb:237:in block in action'
actionpack (4.2.6) lib/action_dispatch/routing/route_set.rb:74:in dispatch'   actionpack (4.2.6) lib/action_dispatch/routing/route_set.rb:43:in serve'
actionpack (4.2.6) lib/action_dispatch/journey/router.rb:43:in block in serve'   actionpack (4.2.6) lib/action_dispatch/journey/router.rb:30:in serve'
actionpack (4.2.6) lib/action_dispatch/routing/route_set.rb:817:in call'   rack-pjax (0.8.0) lib/rack/pjax.rb:12:in call'
rack (1.6.4) lib/rack/etag.rb:24:in call'   rack (1.6.4) lib/rack/conditionalget.rb:38:in call'
rack (1.6.4) lib/rack/head.rb:13:in call'   remotipart (1.2.1) lib/remotipart/middleware.rb:27:in call'
actionpack (4.2.6) lib/action_dispatch/middleware/params_parser.rb:27:in call'   actionpack (4.2.6) lib/action_dispatch/middleware/flash.rb:260:in call'
rack (1.6.4) lib/rack/session/abstract/id.rb:225:in context'   rack (1.6.4) lib/rack/session/abstract/id.rb:220:in call'
actionpack (4.2.6) lib/action_dispatch/middleware/cookies.rb:560:in call'   activerecord (4.2.6) lib/active_record/query_cache.rb:36:in call'
activerecord (4.2.6) lib/active_record/connection_adapters/abstract/connection_pool.rb:653:in call'   activerecord (4.2.6) lib/active_record/migration.rb:377:in call'
actionpack (4.2.6) lib/action_dispatch/middleware/callbacks.rb:29:in block in call'   activesupport (4.2.6) lib/active_support/callbacks.rb:88:in run_callbacks'
activesupport (4.2.6) lib/active_support/callbacks.rb:778:in _run_call_callbacks'   activesupport (4.2.6) lib/active_support/callbacks.rb:81:in run_callbacks'
actionpack (4.2.6) lib/action_dispatch/middleware/callbacks.rb:27:in call'   actionpack (4.2.6) lib/action_dispatch/middleware/reloader.rb:73:in call'
actionpack (4.2.6) lib/action_dispatch/middleware/remote_ip.rb:78:in call'   better_errors (2.1.1) lib/better_errors/middleware.rb:84:in protected_app_call'
better_errors (2.1.1) lib/better_errors/middleware.rb:79:in better_errors_call'   better_errors (2.1.1) lib/better_errors/middleware.rb:57:in call'
rollbar (2.12.0) lib/rollbar/middleware/rails/rollbar.rb:24:in block in call'   rollbar (2.12.0) lib/rollbar.rb:735:in scoped'
rollbar (2.12.0) lib/rollbar/middleware/rails/rollbar.rb:22:in call'   actionpack (4.2.6) lib/action_dispatch/middleware/debug_exceptions.rb:17:in call'
rollbar (2.12.0) lib/rollbar/middleware/rails/show_exceptions.rb:22:in call_with_rollbar'   web-console (2.3.0) lib/web_console/middleware.rb:28:in block in call'
web-console (2.3.0) lib/web_console/middleware.rb:18:in call'   actionpack (4.2.6) lib/action_dispatch/middleware/show_exceptions.rb:30:in call'
railties (4.2.6) lib/rails/rack/logger.rb:38:in call_app'   railties (4.2.6) lib/rails/rack/logger.rb:20:in block in call'
activesupport (4.2.6) lib/active_support/tagged_logging.rb:68:in block in tagged'   activesupport (4.2.6) lib/active_support/tagged_logging.rb:26:in tagged'
activesupport (4.2.6) lib/active_support/tagged_logging.rb:68:in tagged'   railties (4.2.6) lib/rails/rack/logger.rb:20:in call'
actionpack (4.2.6) lib/action_dispatch/middleware/request_id.rb:21:in call'   rack (1.6.4) lib/rack/methodoverride.rb:22:in call'
rack (1.6.4) lib/rack/runtime.rb:18:in call'   activesupport (4.2.6) lib/active_support/cache/strategy/local_cache_middleware.rb:28:in call'
rack (1.6.4) lib/rack/lock.rb:17:in call'   actionpack (4.2.6) lib/action_dispatch/middleware/static.rb:120:in call'
rack (1.6.4) lib/rack/sendfile.rb:113:in call'   rack-cors (0.4.0) lib/rack/cors.rb:80:in call'
railties (4.2.6) lib/rails/engine.rb:518:in call'   railties (4.2.6) lib/rails/application.rb:165:in call'
rack (1.6.4) lib/rack/lock.rb:17:in call'   rack (1.6.4) lib/rack/content_length.rb:15:in call'
rack (1.6.4) lib/rack/handler/webrick.rb:88:in service'   /home/quinlee/.rvm/rubies/ruby-2.2.4/lib/ruby/2.2.0/webrick/httpserver.rb:138:in service'
/home/quinlee/.rvm/rubies/ruby-2.2.4/lib/ruby/2.2.0/webrick/httpserver.rb:94:in run'   /home/quinlee/.rvm/rubies/ruby-2.2.4/lib/ruby/2.2.0/webrick/server.rb:294:in block in start_thread'

这个错误是什么原因?

初次接触不是很了解

你使用的版本太低

哪个版本才不算低?是要指定版本吗?

gem carrierwave-qiniu,  '~> <IP_ADDRESS>'
gem qiniu-rs,  '~>1.1.0'
这是我安装的版本

确实是因为版本低的问题,换成最新版本就好了,感谢!


Resized animated image has different fps

We use a self hosted version of this software and it seems that the fps change after resizing the image.
Original Image:
https://cloudflare-ipfs.com/ipfs/QmYmPLnTEDwD8CwVSFjgWUpUHWV1DUxrMKWnMeHeHXgLCQ
Resized Image:
https://images.weserv.nl/?h=370&w=370&fit=inside&we&q=75&output=webp&n=-1&url=https://cloudflare-ipfs.com/ipfs/QmYmPLnTEDwD8CwVSFjgWUpUHWV1DUxrMKWnMeHeHXgLCQ
(uploaded result: https://cloudflare-ipfs.com/ipfs/QmUGwA3zJkyTyeU29R3MY2s1SccTiZ5hgSvSh9FXkDogaY because its not visible on the public endpoint)
The resized image animation is a lot faster than on the original one. On the public endpoint it seems that the source image is too large but it works on our private one.

Thanks for the report, PR https://github.com/libvips/libvips/pull/2145 should resolve this. For now, you can workaround this by appending the undocumented &delay=100 parameter to the URL.

Seems like it was merged. Does it has to be updated here as well or will it just work?

That PR is part of the libvips v8.10.6 release (which will probably be released in a couple of days). It should be automatically integrated into the Docker image after this version lands on Remi's RPM repository.
You could try to compile libvips from the 8.10 branch if you want to test this earlier.

libvips v8.10.6 has been landed on Remi's RPM repository with commit 0cc4753, so a recompilation of the Docker image should automatically pick this up.
You could also use the prebuilt Docker image available on GitHub's Container Registry:
https://github.com/orgs/weserv/packages/container/package/images


feat: allow setting main-branch-name via environment variable

For our CI system we have the concept of multiple main branches. We enabled the "Only build pull requests" option in circleci so that we could reliably determine the base branch based on the PR. However due to a limitation in circle we aren't able to dynamically pass the main branch name after we've computed it from the pull request. As a workaround it would be great if there was a simple environment variable that we could set instead of relying on the circleci parameter to set the main branch name. I'm opening this PR for further discussion.

NX_... environment variables are defined within the Nx codebase and are used by the core functionality. This one only refers to your current run and is passed to CircleCI API, so it shouldn't have NX_ prefix.
But generally, I'm not sure how setting the environment variable is better than passing a parameter to nx-set-shas like seen here: https://github.com/nrwl/nx-orb/blob/main/src/examples/custom.yml#L20
Can you elaborate on why the above parameter doesn't work? Perhaps you can paste a snippet of your CI yaml config.

NX_... environment variables are defined within the Nx codebase and are used by the core functionality. This one only refers to your current run and is passed to CircleCI API, so it shouldn't have NX_ prefix.

That makes sense, can you suggest a better name for it?

Can you elaborate on why the above parameter doesn't work? Perhaps you can paste a snippet of your CI yaml config.

Sure thing, we're trying to dynamically set the main branch name from an environment variable like this:
      - nx/set-shas:
          main-branch-name: $NX_MAIN_BRANCH

Before this we have a step which dynamically determines the main branch to use based on if it's a pull request or not like this:
if [[ $CIRCLE_PULL_REQUEST == '' ]]; then
	  # If not building a PR then it means we're building a main branch
	  # so we look for the last successful commit on the current branch
	  echo "NX_MAIN_BRANCH: $CIRCLE_BRANCH"
	  echo "export NX_MAIN_BRANCH='$CIRCLE_BRANCH'" >>$BASH_ENV
else
      # It's a pull request, use the base branch from the PR
	  PR_NUMBER=$(echo "$CIRCLE_PULL_REQUEST" | sed "s/.*\/pull\///")
	  echo "PR_NUMBER: $PR_NUMBER"
	  echo "export GITHUB_PR_NUMBER=$PR_NUMBER" >>$BASH_ENV

	  API_GITHUB="https://api.github.com/repos/$CIRCLE_PROJECT_USERNAME/$CIRCLE_PROJECT_REPONAME"
	  PR_REQUEST_URL="$API_GITHUB/pulls/$PR_NUMBER"
	  PR_RESPONSE=$(curl -H "Authorization: token $github_access_token" "$PR_REQUEST_URL")

	  PR_BASE_BRANCH=$(echo $PR_RESPONSE | jq -e '.base.ref' | tr -d '"')
	  echo "NX_MAIN_BRANCH: $PR_BASE_BRANCH"
	  echo "export NX_MAIN_BRANCH='$PR_BASE_BRANCH'" >>$BASH_ENV
fi

According to circleci's support there is no way to dynamically pass the parameter like this so they suggested making a PR here to add the functionality:

That makes sense, can you suggest a better name for it?

Just name it MAIN_BRANCH_NAME, the same as the param.

That makes sense, can you suggest a better name for it?

Just name it MAIN_BRANCH_NAME, the same as the param.

Done!

Thank you!

@meeroslav it looks like the build did not publish as the commit message was wrong, could you take a look please? 😄

Yes, @mattlewis92, I will publish it today.

Awesome, thank you so much! 🙌


Kafka connect - getting Connect Timed Out for Google GCS sink connector

Hello,
Please help me sort out an error i keep getting when trying to connect a topic to Google Cloud Storage.
I am running the Strimzi operator in a Kubernetes cluster behind a proxy. I have followed this guide and set up the https_proxy and http_proxy env variables in the strimzi operator deployment yaml.
However, when launching the connector keep getting this error:
error.txt
I have also set up the proxy env variables in the Kafka connect cluster. Any ideas for a fix?
Thank you!

The error log you added contains failure to connect with OAuth provider. I assume there some mismatches with the authentication configuration in your setup. See this part:
Caused by: java.io.IOException: Error getting access token for service account: connect timed out
at com.google.auth.oauth2.ServiceAccountCredentials.refreshAccessToken(ServiceAccountCredentials.java:401)
at com.google.auth.oauth2.OAuth2Credentials.refresh(OAuth2Credentials.java:181)
at com.google.auth.oauth2.OAuth2Credentials.getRequestMetadata(OAuth2Credentials.java:167)
at com.google.auth.http.HttpCredentialsAdapter.initialize(HttpCredentialsAdapter.java:96)
at com.google.cloud.http.HttpTransportOptions$1.initialize(HttpTransportOptions.java:159)
at com.google.cloud.http.CensusHttpModule$CensusHttpRequestInitializer.initialize(CensusHttpModule.java:109)
at com.google.api.client.http.HttpRequestFactory.buildRequest(HttpRequestFactory.java:93)
at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.buildHttpRequest(AbstractGoogleClientRequest.java:381)
at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.buildHttpRequest(AbstractGoogleClientRequest.java:357)
at com.google.cloud.storage.spi.v1.HttpStorageRpc.open(HttpStorageRpc.java:775)
... 25 more

I hope this helps. I'm closing this issue as there does not seem to be any issue with the connector itself.


Use more standard form of release flag in MESON_ARGS

There are two different ways to set up a release flag to meson, --buildtype release, and -Dbuildtype=release. Apparently the latter is more common; at least it's the recommended way in the docs.
Since meson-python will by default add -Dbuildtype=release, the fact that the compiler-activation does the other kind leads to a duplicate specification and forces us to strip it out again as follows (e.g. in scipy, but affects all feedstocks using meson-python).
# meson-python already sets up a -Dbuildtype=release argument to meson, so
# we need to strip --buildtype out of MESON_ARGS or fail due to redundancy
MESON_ARGS_REDUCED="$(echo $MESON_ARGS | sed 's/--buildtype release //g')"

To avoid this, use the more common form of the flag.
PS. Stumbled over this old PR, where this was discussed with @rgommers
PPS. Sister PR of https://github.com/conda-forge/clang-compiler-activation-feedstock/pull/112

This might break packages that does the opposite. So, it's not clear which one is better.

This might break packages that does the opposite.

That's clear, though I think it'd be good to decide on one and try to be consistent.

So, it's not clear which one is better.

Given that meson-python uses -Dbuildtype (not the conda-forge version, but upstream), and that the meson docs use this variant too, I think the balance tilts towards that.

Given that meson-python uses -Dbuildtype (not only the conda-forge version, but built-in upstream), and that the meson docs use this variant too, I think the balance tilts towards that.

I agree.  It is right for all Python packages. In case there are non-Python packages that have an explicit --buildtype=release in their build.sh script in conda-forge, you'll get a clear error:
ERROR: Got argument buildtype as both -Dbuildtype and --buildtype. Pick one.

You'll fix that once, which is a simple 1-line diff, and does not require argument parsing.
meson-python isn't only relevant here because it's getting more popular, but because it's an intermediate layer that adds the incompatible flag. Such a layer does not exist for other languages, so there isn't the opposite problem if we make the switch here to using -Dbuildtype.

I'm adapting the changes discussed on the clang side into #121


Add min and max TLS version options

Added the plumbing for specifying the min and max TLS versions. This was implemented in the grpc core but not for the cpp wrapper. This PR exposes the ability to specify the min and max TLS version in the cpp wrapper.
This builds on PR #23165.
The TlsVersionRange struct was created on the client side because we needed to create a default value for the min and max TLS versions while maintaining API stability. Without the struct, we would not be able to set a default value for min and max TLS versions because adding a constructor to set default values would break struct initialization, as no constructor is currently defined. This was not needed for the server side because a constructor was already defined and we could define the defaults in the constructors.
@matthewstevenson88

This change is

Thanks for the PR @Ryanfsdf! I've left a few follow-up comments. Additionally,

Please remove [WIP] from the title.
Reference PR #23165 in the body of the description (this is the PR that enabled TLS 1.3 in the C-core).
In the body of the description, please add an explanation why the TlsVersionRange struct was added (and only added on the client-side). (Namely, for API stability reasons.)

Please hold on this PR until we know if we need to temporarily disable TLS 1.3 in grpc core.

Currently user cannot control TLS version. It is not a good idea if we explicitly expose an API to allow user to set TLS max/min version, but it won't work due to PR #23608.
Shall we hold on this PR until PR #23608 has been reverted? @matthewstevenson88 Any suggestion?

Just to check: you mean it is a good idea if we (eventually) expose an API to allow setting min/max versions, correct? :)
(I assume this is what you meant because both gRPC-Java and gRPC-Go expose such an API already.)
I think we should do the following:

Get this PR fully reviewed now (because @Ryanfsdf's internship ends in a few weeks). In particular, I want the gRPC team to double check the changes Ryan made to the e2e testing framework.

Wait to merge this PR until #23608 is reverted. In the meantime, Ryan can add "DO NOT MERGE" to the title/description.

WDYT?

SG. I agree the value of such API.
Let do the review first, but wait until #23608 is reverted before merging this one.

@ZhenLian will own the gRFC on new TLS creds.
I agree with @markdroth that we are deprecating SSL creds soon -- once we have credential provider/distributor ready. We should avoid changing old SSL APIs.
@matthewstevenson88 Does Zatar interop code require explicitly setting TLS version? If not, TLS creds should be changed first.


Does not work on virtual devices

Unfortunately, I get all the virtual devices no matter whether PC or server the following known message:
Unhandled Exception: System.NullReferenceException: Object reference not set to an instance of an object.
at Marksman.Sentry.RunWMI()
at Marksman.Marksman.Main(String[] args)
Does anyone have the same problem with the virtual devices and does anyone know a solution? Thank you very much for your help.

Thanks for the bug report. A few questions:

Can you give an example of a virtual device that returns this error?
Do you mean virtual machines, i.e. a VM running inside a host machine?

I found out that I do not have the problem with virtual W7 machines. It only appears on W10 and Server 2016.

@velaar Does it make sense to support virtual machines? This was originally written as a utility to keep track of hardware inventory, not VMs and containers.

I think you're right. I only use the VM computer for testing. The problem also appears on physical devices with the Windows 10 not on Windows 7. Can it be that it runs only with certain .Net versions?

@velaar Does it make sense to support virtual machines? This was originally written as a utility to keep track of hardware inventory, not VMs and containers.

VM is such a niche case. I think it can be achieved with a settings switch for testing purposes. However there are so many different VMs around and Windows WMI reacts in different ways to what they emulate. I think shouldn't implement this support as there is little practical value.
Close the case?

Agreed. Given that this is a hardware inventory tool, I'm not sure it makes sense to include VMs.


check free space when use external sort/aggerator

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Category (leave one):

Improvement

Short description (up to few sentences):
When enable dumping temporary data to the disk to restrict memroy usage during GROUP BY/SORT, it didn't check the free disk space. The fix add a new setting min_free_disk_space, when the free disk space it smaller then the threshold, the query will stop and throw ErrorCodes::NOT_ENOUGH_SPACE

I thought it should be checked while writing. We may implement LimitWriteBuffer like LimitReadBuffer and wrap the most deep WriteBuffer (that actually writes to file) in it.

Thanks for comments.
Does the team has the plan to implement it, or I can implement it now, since we want to use it in the production environment.
As my understand from your comment, we can implement LimitWriteBuffer, the origin code
WriteBufferFromFile file_buf(path);
CompressedWriteBuffer compressed_buf(file_buf);
can be written as
LimtWriteBufferFromFile file_buf(path);
CompressedWriteBuffer compressed_buf(file_buf);
So if the origin data is bigger and free space may not enough, but after compressed, the space may be enough, so the query can still works.

Does the team has the plan to implement it, or I can implement it now, since we want to use it in the production environment.

We don't have to implement it in our team.

As my understand from your comment, we can implement LimitWriteBuffer, the origin code
WriteBufferFromFile file_buf(path);
CompressedWriteBuffer compressed_buf(file_buf);
can be written as
WriteBufferFromFile file_buf(path);
LimeitWriteBuffer limit_file_buf(file_buf);
CompressedWriteBuffer compressed_buf(limit_file_buf);

Yes.

So if the origin data is bigger and free space may not enough, but after compressed, the space may be enough, so the query can still works.

Actually your variant is good enough. I can merge it.

Overestimation of free space is fine.

I think, this implementation is Ok, no need to implement LimitWriteBuffer.


[build] lit-element css not minified correctly

lit-element contains partial css, which breaks the minifier

We still need to fix this

I'm working on this in a couple of PRs to the minifier plugin.
https://github.com/cfware/babel-plugin-template-html-minifier/pull/33, https://github.com/cfware/babel-plugin-template-html-minifier/pull/34 and https://github.com/cfware/babel-plugin-template-html-minifier/pull/35

Sorry for the delay on this one I just reviewed the PR for partial CSS.


Fix nested pagination

Summary
Orders in a customer are not loaded properly. Every new page loads a cumulative  number of orders instead of orders only related to that page. This is because for every loop, we print all orders instead of printing only the orders in ordersResponse

ping @mairaw
Do you know who is responsible for astoria docs? I'd like someone that knows that framework to review this.

@stevestein Can you review these changes?

This change should be made in the docs repo now.

Closing due to lack of response.

This is interesting. I am a Microsoft Software engineer in the OData Team and had hoped to contribute to WCF. Btw this PR was closed due to lack of response from who?


fix(examples): calendar chart metric should be metrics

SUMMARY
One of the example calendar heatmap charts has incorrectly defined the metric control value, while it really should be metrics (the value should also be an array of strings, not a string).
BEFORE

AFTER

TESTING INSTRUCTIONS

Populate examples data
Open the "Calendar Heatmap" chart
See that previously it didn't work, but after the changes it works properly.

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

I've restarted the presto/hive tests a third time now. If it doesn't pass, it doesn't seem like flakiness. The error doesn't look like it has anything to do with this PR though.

You might need to rebase this to bring in https://github.com/apache/superset/pull/15163 to fix the Presto/Hive tests


Bugfix divide by 0 on template.php

I got a divide by 0 exception with a fresh installation, after creating a pair of usiers and going to staffpanel.php with the browser with my "admin"  (id = 1) user logged in.
Found that divide by 0 on this file and the $MemStats['cmd_get'] is the denominator on that operation that makes the exception, so I made changes to check it before the division.

Dont't merge this one. I forgot to delete the line that produces the divide / 0 that is duplicated inside the conditional sentence.


Is this plugin now actively maintained?

Hello @SergeyKanzhelev, I saw that you've started making commits to this repo in May. Before that, the commits stopped for about 2 years (2016 and 2017).
Is this repo being maintained now?
Thanks!

@duncanmak I dusted it off and ready to accept contributions. There are few people using it as far as I know.
There is a chance this repository (or at least functionality) will be moved to Open Census. This will be announced in this repo

Closing for now - let me know if you need more info


add ESM source module

This fixes #9
I did the following in this MR:

convert keycharm.js from UMD to ESM module.
moved the new ESM module source file to the src folder.
added a rollup build script to bundle the ESM module to UMD.
documanted the differnet imports in the README

I would recommend publishing this as a minor change, because the old keycharm.js file is still there and still supports UMD.

Looks good, thnx!

I've published this as 0.4.0. Cheers!


Issue #464: Replaces Grunt with NPM scripts

https://github.com/opentable/oc/issues/464

Replaces Grunt with NPM scripts
Removes text-limit on changelog messages
Updates contributing and readme docs with run-script commands

wow @jakeclements this is an amazing work!

Give us some time to review but I just wanted to start saying thank you for this!

Thanks for the review @matteofigus, say the word once your PRs are in and I'll merge this.

Wohoo, super excited for this.
Thanks 💯  @jakeclements for the 💅 😍 , awesome stuff!

@matteofigus thanks for the review, no problem about the timeframe.
I've updated based on the requested changes, take a look when you get a chance and let me know if you would like me to merge.

Welcome to the contributors! 👍


AttributeError: 'dict' object has no attribute 'iteritems'

It terminated after loading bed file with following error
➜  SALSA git:(master) ✗ python2 run_pipeline.py -a A_plus_collapsed.fa -l A_plus_collapsed.fa.fai -b alignment.bed -e GATC -o scaffolds -i 3
bedfile loaded
Traceback (most recent call last):
  File "/home/urbe/Tools/SALSA/correct.py", line 28, in <module>
    input_seqs = parse_fasta(open(sys.argv[1],'r'))
  File "/home/urbe/Tools/SALSA/correct.py", line 20, in parse_fasta
    for short_name, nuc_list in fa.iteritems():
AttributeError: 'dict' object has no attribute 'iteritems'

mv: cannot stat 'scaffolds//alignment_iteration_1.tmp.bed': No such file or directory
mv: cannot stat 'scaffolds//asm.cleaned.fasta': No such file or directory
Starting Iteration 1
python /home/urbe/Tools/SALSA/make_links.py -b scaffolds/alignment_iteration_1.bed -d scaffolds -i 1 -x abc
  File "/home/urbe/Tools/SALSA/make_links.py", line 158
    print len(contig_links)
            ^
SyntaxError: invalid syntax

How to fix that?

Hi,
Which version of python are you using? The python 3.x doesn't  have .iteritems() method defined for dictionary objects. I would recommend using python 2.7 to run SALSA2.

Hi @machinegun
I am using Python2.7 !

I have same issue, can you please write specific command you followd?

run_pipeline.py call many python programs, it required Python version2.7
Set the python path https://bioinformaticsonline.com/snippets/view/37505/install-python-locally-on-shared-linux-server


WIP: Remote port forwarding

Fixes #6

Could you use any help with finishing this?

Could you use any help with finishing this?

Hey @sini! Yes I got a bit stuck with refactoring the code to support this, and I have other commitments currently. If you want to start a separate draft or if you have any ideas, I'm super open to it! 👍

Could you use any help with finishing this?

Hey @sini! Yes I got a bit stuck with refactoring the code to support this, and I have other commitments currently. If you want to start a separate draft or if you have any ideas, I'm super open to it! 👍

Thanks for the prompt response! I'm new to rust but have a need for this and would be happy to hack at it versus starting a new project from scratch. I'll try and pick this PR up. :) If you have any notes/pointers for next steps you would have taken I'm sure that could be super helpful.

Closing for now; I will implement piece by piece in master.


Fix check for if shmem is valid to take into account detached shmem

Problem
We can segfault if we update connstr inside of a process that has detached from shmem (e.g. inside stats collector)
Summary of changes
Add a check to make sure we're not detached
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

happened one more time: https://neondb.slack.com/archives/C039YKBRZB4/p1701776414823319


Multi-arch support (buildx)

Hi, thanks so much for this. I was wondering if there is any plan to have multi-arch support (not super familiar with this, I've just discovered here) for arm (Apple M1) compatibility (without rosetta).
Thanks

Also not familiar with this, looks like github does not yet provide M1 build hosts
https://github.com/actions/virtual-environments/issues/2187
Any info on what we need to do to implement this or a pr would be welcome.

And there is a buildx action available
https://github.com/marketplace/actions/docker-setup-buildx

You can build arm64v8 images on Intel 64-bit hosts
I also needed this support so copied most of the owners dockerfile to publish multi-arch (amd64/arm64) images for scala-sbt - this can be found here https://hub.docker.com/r/westwater/scala-sbt/tags?page=1&ordering=last_updated
This is only temporary while I wait for the owner to publish his own

Great. Any instructions on how to build these would be welcome. That should help me get started tweaking the GitHub actions.

I used docker's buildx utility to publish multi-arch images (this doc explains it really well)
docker buildx create --use // to create the buildkit daemon and use the builder
docker buildx build --platform linux/amd64,linux/arm64 -t ${IMAGE_TAG} --push .

finally got this working, see https://hub.docker.com/r/hseeberger/scala-sbt/tags?page=1&ordering=last_updated
@mfirry @westwater please let me know if the image works as expected


[Error] Installed ruby version but not detected

When I try to run bundle install on my terminal, I have that error :
Warning: the running version of Bundler (1.17.2) is older than the version that created the lockfile (1.17.3). We suggest you upgrade to the latest version of Bundler by running `gem install bundler`.
Your Ruby version is 2.6.6, but your Gemfile specified 2.6.5

So, I tried rbenv versions and I had that :
  system
* 2.6.5 (set by /Users/username/code/username/repository/.ruby-version)
  2.6.6

With writing ruby -v, I got ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-darwin20]
and if I try bundle install again, I still have the same error :
...
Your Ruby version is 2.6.6, but your Gemfile specified 2.6.5

Can you help me to find why I do have Ruby version 2.6.5 but it does not detect it ?
Small precision that might help and change something : I’m using the new MacBookPro M1 with Terminal Rosetta.

Problem solved :
a line in my ~/.zshrc file was redirecting lots of paths at the wrong place and made also troubles for Ruby directories

Problem solved :
a line in my ~/.zshrc file was redirecting lots of paths at the wrong place and made also troubles for Ruby directories


Unused Argument found in generate regional grid loss function

This is just an FYI that the function generate_regional_grid_loss in eia_trans_dist_grid_loss.py has three arguments, the first being final_database described in the doc string below; however, this data frame parameter is never used; rather, a new database is created using the build_generation_data function found in eia923_generation.py module.
You may want to consider removing this requirement.
def generate_regional_grid_loss(final_database, year, subregion="all"):
    """This function generates transmission and distribution losses for the
    provided generation data and given year, aggregated by subregion.

    Arguments:
        final_database: dataframe
            The database containing plant-level emissions.
        year: int
            Analysis year for the transmission and distribution loss data.
            Ideally this should match the year of your final_database.
    Returns:
        td_by_region: dataframe
            A dataframe of transmission and distribution loss rates as a
            fraction. This dataframe can be used to generate unit processes
            for transmission and distribution to match the regionally-
            aggregated emissions unit processes.
    """

Found in electricityLCI version: 1.0.1

Note that you are also missing the third argument from this doc string ;)


magic number of tombstones in readme

Hi, please correct the magic number of tombstones in tsdb/docs/format/tombstones.md
instead of being a magic(0x130BA30) to be a magic(0x0130BA30)

Great !!! Thanks.


feat(paging): factor out common controller code

Create paging factory for creating common controller for the
pagination and pager components

This finishes off #4803.

What I am not happy about is the tests. You refactorized it in different files but there is no paging tests.

Do we require tests with this change? All this is doing is technically exposing the lack of tests on the old UibPaginationController, since the tests in this library is behavioral in general.

I hear you Wes. But I don't like to add new "components" without any kind of tests. I know that all we do is behavioral.
So here we are pretending that the .create method just do what you think it does.

Added tests

It is good for me. I still need to sweep all of this, so I will do a second pass by then.


Room.find() 发送消息提示 say is not a function

如图，有一个 test 的群。
      const r = Room.find({topic: 'test'})
      r.say('hello')

提示 r.say is not a function

const r = await Room.find({topic: 'test'})
You need to add await before the function which will return a Promise.


The OS is not supported

Environment
Device and OS: OpenSuSe
App version: Tumbleweed
Kubernetes distro being used: N/A
Other:
wanted to report this as a bug besides an issue with testing. After running the command:
zarf package deploy oci://ghcr.io/defenseunicorns/packages/dubbd-k3d:0.11.0-amd64 --set APPROVED_REGISTRIES="ghcr.io/runyontr/* | ghcr.io/defenseunicorns/* | nvcr.io/nvidia/k8s/* | semitechnologies/*"
I get the following error:
📦 SET-ZARF-VARS COMPONENT
✔  Completed "Set pvc reclaim policy Zarf variable"
✔  Completed "Set admin domain Zarf variable"
📦 PREFLIGHT COMPONENT
✔  Completed "Checking for kubecontext..."
✔  Completed "Checking if cluster is healthy..."
v1.28.1+k3s1
✔  Completed "Checking cluster version..."
✔  Completed "Checking if zarf has been initialized..."
✔  Completed "Checking if Gitea is runnning..."
✔  Completed "Checking if this is an upgrade..."
✔  Copying 1 files
Server version is supported
✔  Completed "./preflight.sh"
✔  Completed "rm preflight.sh"
📦 LOAD-CERTS COMPONENT
✔  Copying 2 files
✔  Completed "Read cert file into values"
✔  Completed "Read key file into values"
✔  Completed "Read admin cert file into values"
✔  Completed "Read admin key file into values"
✔  Completed "rm bigbang.dev.cert"
✔  Completed "rm bigbang.dev.key"
📦 GENERATE-CREDENTIALS COMPONENT
✔  Completed "if [ ${ZARF_VAR_IS_UPGRADE} = true ]; then;   if ! ./zarf..."
✔  Completed "if [ ${ZARF_VAR_IS_UPGRADE} = true ]; then;   if ! ./zarf..."
✔  Completed "if [ ${ZARF_VAR_IS_UPGRADE} = true ]; then;   if ! ./zarf..."
✔  Loading the Zarf State from the Kubernetes cluster
✔  Starting helm chart generation admin-interface-creds
✔  Processing helm chart raw-dubbd-k3d-generate-credentials-admin-interface-creds:0.1.1698640525 from Zarf-generated helm chart
📦 DOWNLOAD-FLUX COMPONENT
✔  Copying 7 files
✔  Completed "Clean up previous flux install"
The OS is not supported
ERROR:  Failed to deploy package: unable to deploy all components in this Zarf Package: unable to deploy
component download-flux: unable to run component after action: command "Extract flux binary" timed
out after 0 seconds
Steps to reproduce
N/A
Expected result
N/A
Actual Result
N/A
Visual Proof (screenshots, videos, text, etc)
N/A
Severity/Priority
High
Additional Context
Add any other context or screenshots about the technical debt here.

That error is actually not from Zarf and is an artifact of that package.  Going to shift this over to that repository so that that team can look more specifically at this issue but here are the offending lines: https://github.com/defenseunicorns/uds-package-dubbd/blob/main/defense-unicorns-distro/scripts/extract_flux.sh#L4
Likely Tumbleweed isn't reporing $OSTYPE as linux-gnu and that script falls through.


Tap to copy is not tappable

Account Number “Tap to Copy” text doesn’t actually tap to copy when tapped. You would expect both the address and the text to allow this functionality, esp since it’s underlined. If you don’t want “Tap to copy” to have this functionality, then it should be in parentheses.

Fixed in #861


Refactored test token generation

Related to https://github.com/almighty/almighty-core/pull/807

Codecov Report

Merging #812 into master will decrease coverage by -0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #812      +/-   ##
==========================================
- Coverage   70.05%   70.03%   -0.02%
==========================================
  Files          76       76
  Lines        5276     5267       -9
==========================================
- Hits         3696     3689       -7
+ Misses       1227     1226       -1
+ Partials      353      352       -1

Impacted Files
Coverage Δ

token/token.go
95.23% <ø> (+3.08%)
:white_check_mark:

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 06ef417...298544b. Read the comment docs.

@alexeykazakov I made a PR to your branch https://github.com/alexeykazakov/almighty-core/pull/2

@hectorj2f yeah, new package did the trick. Thanks!


Support for batch inference

Is there any plan to support batch inference, instead of single input ?

This would definitely be a game changer


RuntimeError: Input type (torch.cuda.FloatTensor) and weight type (torch.FloatTensor) should be the same

Traceback (most recent call last):
File "examples.py", line 566, in
ppo_pixel_atari(game)
File "examples.py", line 391, in ppo_pixel_atari
run_steps(PPOAgent(config))
File "/home/simon/Desktop/DeepRL.old/deep_rl/agent/PPO_agent.py", line 25, in init
torchsummary.summary(self.network,(4, 84, 84))
File "/home/simon/anaconda3/lib/python3.6/site-packages/torchsummary/torchsummary.py", line 72, in summary
model(*x)
File "/home/simon/anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/simon/Desktop/DeepRL.old/deep_rl/network/network_heads.py", line 173, in forward
phi = self.network.phi_body(obs)
File "/home/simon/anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/simon/Desktop/DeepRL.old/deep_rl/network/network_bodies.py", line 19, in forward
y = F.relu(self.conv1(x))
File "/home/simon/anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "/home/simon/anaconda3/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 320, in forward
self.padding, self.dilation, self.groups)
RuntimeError: Input type (torch.cuda.FloatTensor) and weight type (torch.FloatTensor) should be the same

There are two ways to fix it:

convert your network to cuda
call torchsummary.summary with device='cpu'

https://github.com/sksq96/pytorch-summary/issues/36


feature: 3D labels / legend

Add labels/legend for NiiVue renders.

Labels / Legend should support the following:
Different sized text
Text justification
Lines to a specific point in a volume / mesh


option to disable facepile just for mentions

i like the current logic for mentions that shows them when they're short enough. i'd like to keep that even with facepiles, by exempting either all mentions or short mentions below the threshold. example: https://snarfed.org/podcasts#comment-744582

this is not that easy because we are filtering by comment type... except we remove the „webmention“ comment type for these mentions...


Control: Enter and Exit is not called when mouse is pressed

Godot version:
3.1.1
OS/device including version:
Windows 7
Issue description:
On Controls:
Enter and Exit is not called when mouse is pressed (on holding mouse key) when you go over or out of the control with the mouse.

public override void _Notification(int what)
{
if (what == Control.NotificationMouseEnter)
{
Console.WriteLine("ENTER");
}
else if (what == Control.NotificationMouseExit)
{
Console.WriteLine("EXIT");
}
}

This is expected behavior.
When you press a button, you don't want it to "unpress" while the mouse is pressed just because you got out of the boundaries of the button. It stays pressed until you release the mouse button.

Why don't you use the signals mouse_entered and mouse_exited?
I assume you use C#, so that should do the trick:
public override void _Ready() {
	Connect("mouse_entered", this, nameof(MouseEntered));
	Connect("mouse_exited", this, nameof(MouseExited));
}

public void MouseEntered() {

}
public void MouseExited() {

}

Why don't you use the signals mouse_entered and mouse_exited?
I assume you use C#, so that should do the trick:
public override void _Ready() {
	Connect("mouse_entered", this, nameof(MouseEntered));
	Connect("mouse_exited", this, nameof(MouseExited));
}

public void MouseEntered() {

}
public void MouseExited() {

}

because it has the same behaviour!

This is expected behavior.
When you press a button, you don't want it to "unpress" while the mouse is pressed just because you got out of the boundaries of the button. It stays pressed until you release the mouse button.

Its a problem!
Because i cant write logic with this behaviour.

Anyway that's a duplicate of #20881

Anyway that's a duplicate of #20881

And here:
https://github.com/godotengine/godot/issues/23296
Well known problem. ... but old one.

I'm having issues with that one as well.

Yeah same, I ended up just writing a custom mouse enter and exit signal using (Control.get_global_rect().has_point(get_global_mouse_position()))


Improve switch toggle behavior

I don't quite understand the current toggle behavior of the switch. It stays "on" indefinitely.
I'd expect the switch to automatically toggle to "off" once the sound has finished playing (or the configured delay has elapsed). The Sonos device should then go back to whatever has been playing before. If nothing has been playing before, it should play nothing again (and not start the current playlist).
Thanks @Julius-Bendt for your work so far. Feel free to ignore my requests. ;)

Hi @Glurz,
I think i fixed the bug. I can't replicate it anymore, at least.
Sorry for the long wait!

Hi @Julius-Bendt
Thanks for checking this. At first glance, it seems to work! Thanks.

Hi @Julius-Bendt
I don't know if it's a regression introduced with the latest fix but I noticed the following behavior:
The sound I'm playing as a notification is around 20 seconds. I set the timeout value to 5 seconds. After 5 seconds, the switch toggle goes to off again (which is expected), but the sound continues to play for the entire 20 seconds. Shouldn't it stop after the configured timeout of 5 seconds?
Thanks!

Logically then yes, it should. I had to remove some code in the last release to fix another bug. I'll take a look at it, and see if I can figure something out

@Glurz this should be fixed in the next release.
Please notice, that the next release will have some configuration changes. I expect to get the release out sunday


[ci:component:github.com/gardener/egress-filter-refresher:0.11.0->0.12.1]

Release Notes:
Replace shell scripts by go implementation and add logging.

Will be updated in #64


Asynchronous transformTags call

Hi, I met problem in case when I want to make some API call inside transformTags function it never be awaited. As I understand sanitize-html does not support async calls in itself. Maybe somebody knows some workaround how to make it work?
Thanks

sanitize-html is meant to be a very fast library for sanitizing of user input. If you need to make API calls I suggest checking out something like cheerio which lets you load the document, manipulate it with a full jQuery-like API (but not just in the browser), and generate the HTML again when you're ready after doing whatever you wish.


[release validation] No-op change

Details

Motivation

Context

Pull request checklist

[ ] Addresses an existing issue: Fixes #0000
[ ] Added relevant unit test for your changes. (yarn test)
[ ] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[ ] Ran precheckin (yarn precheckin)
[ ] Described how this PR impacts both the ADO extension and the GitHub action

Codecov Report

Merging #1390 (c50910e) into main (7d8488c) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head c50910e differs from pull request most recent head 5ec5229. Consider uploading reports for the commit 5ec5229 to get more accurate results

@@           Coverage Diff           @@
##             main    #1390   +/-   ##
=======================================
  Coverage   91.92%   91.92%
=======================================
  Files          62       62
  Lines        1609     1609
  Branches      206      206
=======================================
  Hits         1479     1479
  Misses        130      130

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


fix: replace pkg_resources for python 3.12

Description
Follow up to #2312, Python 3.12 also removed bundled setuptools (aka pkg_resources)
QC

[ ] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[ ] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Rebased to pick up 97f428ba2921cebc2a311a4933c1c55536d34335

Looks like a network flake, can we rerun ci?
_______________________ ERROR collecting tests/tests.py ________________________
tests/tests.py:17: in <module>
    from .common import *
tests/common.py:72: in <module>
    not is_connected() or not has_gcloud_service_key(),
tests/common.py:49: in is_connected
    urllib.request.urlopen("http://www.google.com", timeout=1)
...
../../../micromamba/envs/snakemake/lib/python3.11/socket.py:706: in readinto
    return self._sock.recv_into(b)
E   TimeoutError: timed out
=========================== short test summary info ============================
ERROR tests/tests.py - TimeoutError: timed out


jsonb field not showing up in SHOW/EDIT

Related Issue: https://github.com/SoftwareBrothers/adminjs/issues/796.
We can show the json data if we have predefined structure.
In my case json data is huge. and I don't have predefined structure. How can i show this on UI?

Currently it will only be possible through a custom component. You can read your JSON field's structure and create virtual properties from it's keys.


IntelliJ error Could not find artifact com.intuit.karate:karate.archetype:pom:0.9.5 in central (https://repo.maven.apache.org/maven2)

IMPORTANT: If you have a general question please use Stack Overflow instead where Karate has a dedicated "tag": https://stackoverflow.com/questions/tagged/karate
If you are sure you have found a bug, please make sure you follow the instructions here: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue

Hi
Trying to creating a new Maven project  in IntelliJ via archetype com.intuit.karate:karate.archetype
Get the following error
IntelliJ error Could not find artifact com.intuit.karate:karate.archetype:pom:0.9.5 in central (https://repo.maven.apache.org/maven2)

works for me. try stack overflow

Hi Peter,
“Works for me” is not a very helpful answer to be honest. IntelliJ support say that it is not an issue with their software so I am a bit stuck
I will explain a bit more
I was watching your Getting Started video  https://www.youtube.com/watch?v=W-af7Cd8cMc&feature=youtu.be via the Karate page and it was all going very well except the IntelliJ version I am using is 2020.1 whilst the YouTube video is using IntelliJ Version 2019.1.3
The BUILD SUCCESS message showed and I went into src/test/examples/users.feature but the new version of IntelliJ (2020.1) is slightly different and prompts you to install Cucumber and Gherkin and the Substeps IntelliJ Plugin earlier so I installed all 3 before getting to the part of the video (4 minutes and 15 seconds) which tells you not to install Substeps IntelliJ Plugin
Realising my mistake I immediately  tried to uninstall Substeps IntelliJ Plugin via Settings-Plugins (but I could not see and option). The Cucumber plug in showed a Restart IDE button so I clicked that and the IntelliJ IDE tried to restart but was not able to (Hung for at least 15 minutes).
So I uninstalled IntelliJ and reinstalled again and tried to create a new Maven/Karate  Project but this time the Build failed with error
IntelliJ error Could not find artifact com.intuit.karate:karate.archetype:pom:0.9.5 in central (https://repo.maven.apache.org/maven2)
Any tips advice or a workaround would be greatly appreciated.
Regards
Donovan Lines
Snr Project/Process Analyst
Quality CoE
FedEx Express Europe
Tel +44(0) 2476 702722
Mob: +44 7969 588 246
From: Peter Thomas<EMAIL_ADDRESS>Sent: Wednesday, April 22, 2020 12:16 PM
To: intuit/karate<EMAIL_ADDRESS>Cc: Donovan Lines<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] [Marketing Mail] Re: [intuit/karate] IntelliJ error Could not find artifact com.intuit.karate:karate.archetype:pom:0.9.5 in central (https://repo.maven.apache.org/maven2) (#1112)
works for me. try stack overflow
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_intuit_karate_issues_1112-23issuecomment-2D617713742&d=DwMCaQ&c=fpIeS8RwmHkY2r_dFRL5tQ&r=D9kELe8XbTWAqYtiSvWRZA&m=-Ipp8AMQxZWNSOVoZ3jtMJbuf3imsA1CvSr2DsI1OSw&s=yDcVpI5XxBq3DPm8xYB-PO0Mk3p2EA49JA7fD8LJu-k&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_APJKY442J7EZ6E2XI2YCEV3RN3GVJANCNFSM4MOCDQGA&d=DwMCaQ&c=fpIeS8RwmHkY2r_dFRL5tQ&r=D9kELe8XbTWAqYtiSvWRZA&m=-Ipp8AMQxZWNSOVoZ3jtMJbuf3imsA1CvSr2DsI1OSw&s=SkkRUHMjfKX3HiFDfa9_R3x_vmeBcc7uT5osIbIUocg&e=.

I will explain a bit more

that's better. here's proof that karate exists: https://mvnrepository.com/artifact/com.intuit.karate/karate-apache/0.9.5
it can happen that your local maven repo is corrupted: https://stackoverflow.com/q/7408545/143475
if nothing works, maybe you should just use the ZIP release: https://github.com/intuit/karate/wiki/ZIP-Release
I asked you to try Stack Overflow since there is a wider community that will answer questions, but your call.

Hi Peter,
Thanks for the links.
I really like the karate framework but I am not impressed with IntelliJ so far. I think both the IntelliJ uninstall and install are a bit flaky. I used to use Eclipse but my new team are all using IntelliJ so I had to switch so not to be seen as a rebel.
I fixed the issue by:
·        Uninstalled IntelliJ v2020.1
·        Deleted all the files manually in c:\Users<BlaBla>.m2\repository
·        Installed IntelliJ v2019.3.2
** One question is it best to install Maven locally on my laptop or to let IntelliJ use its bundled version?
Regards
Donovan Lines
Snr Project/Process Analyst
Quality CoE
FedEx Express Europe
Tel +44(0) 2476 702722
Mob: +44 7969 588 246
From: Peter Thomas<EMAIL_ADDRESS>Sent: Wednesday, April 22, 2020 2:05 PM
To: intuit/karate<EMAIL_ADDRESS>Cc: Donovan Lines<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] [Marketing Mail] Re: [intuit/karate] IntelliJ error Could not find artifact com.intuit.karate:karate.archetype:pom:0.9.5 in central (https://repo.maven.apache.org/maven2) (#1112)
I will explain a bit more
that's better. here's proof that karate exists: https://mvnrepository.com/artifact/com.intuit.karate/karate-apache/0.9.5https://urldefense.proofpoint.com/v2/url?u=https-3A__mvnrepository.com_artifact_com.intuit.karate_karate-2Dapache_0.9.5&d=DwMCaQ&c=fpIeS8RwmHkY2r_dFRL5tQ&r=D9kELe8XbTWAqYtiSvWRZA&m=-gqGFMZ8OcZoLrQgmRIHhcnpy5xscKKugNOyhNDviSs&s=iS5Bg0RdoRT2m_i8OWBvby01sCANuMHrozxfF-FqgfU&e=
it can happen that your local maven repo is corrupted: https://stackoverflow.com/q/7408545/143475https://urldefense.proofpoint.com/v2/url?u=https-3A__stackoverflow.com_q_7408545_143475&d=DwMCaQ&c=fpIeS8RwmHkY2r_dFRL5tQ&r=D9kELe8XbTWAqYtiSvWRZA&m=-gqGFMZ8OcZoLrQgmRIHhcnpy5xscKKugNOyhNDviSs&s=u-xNZJ0z-ptz2eEWQrkvcwChib5rgkebALTq1EqOhG0&e=
if nothing works, maybe you should just use the ZIP release: https://github.com/intuit/karate/wiki/ZIP-Releasehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_intuit_karate_wiki_ZIP-2DRelease&d=DwMCaQ&c=fpIeS8RwmHkY2r_dFRL5tQ&r=D9kELe8XbTWAqYtiSvWRZA&m=-gqGFMZ8OcZoLrQgmRIHhcnpy5xscKKugNOyhNDviSs&s=rAPhDY7YWYVhxXFEvsSz0iBp5TyUq0C9bety_3FjsZ4&e=
I asked you to try Stack Overflow since there is a wider community that will answer questions, but your call.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_intuit_karate_issues_1112-23issuecomment-2D617766401&d=DwMCaQ&c=fpIeS8RwmHkY2r_dFRL5tQ&r=D9kELe8XbTWAqYtiSvWRZA&m=-gqGFMZ8OcZoLrQgmRIHhcnpy5xscKKugNOyhNDviSs&s=3PVhr9x41ArHds66uJ-Uvn_cIhT68LawOAWNuR6wDNw&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_APJKY44QITHN7TOH7FL7ZT3RN3TOLANCNFSM4MOCDQGA&d=DwMCaQ&c=fpIeS8RwmHkY2r_dFRL5tQ&r=D9kELe8XbTWAqYtiSvWRZA&m=-gqGFMZ8OcZoLrQgmRIHhcnpy5xscKKugNOyhNDviSs&s=O9qflrSAN1V-Tqz03t3tdXEGSEw3Gz8Npbp85mi4kVU&e=.

One question is it best to install Maven locally on my laptop or to let IntelliJ use its bundled version?

both are fine, so you can pick whichever you are comfortable with


⚠ Bofrost.be has degraded performance

In 940c213, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1601 ms

Resolved: Bofrost.be performance has improved in 88d0005.


How to show "other" if nothing is on the autocomplete

I wish to show the item labelled "other" if nothing is matching the autocomplete

Hi,
sorry for my late reply but I was in vacations. Well you just need to implement this logic in your items-method such that if the incoming query is not matching anything then you need to return the other item object such that the user is able to select this.
I hope this helps. Please notify me if that worked for you such that I can close the issue.
Thanks and regards,
Guy

Hi,
as there was no activity for some weeks I will close the issue. If there is still something you can add a comment and reopen it.
I hope it works now.
Thanks and regards,
Guy


fix(frontend): update cookie module

NOTE: This is a breaking change for the session cookies. All previously created session cookies will be invalid regardless of their expiration time and users will have to login again. There should be no other issues with this change.
Updates session cookie configuration to utilize better security practices.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Still has some checkstyle issues


Fix NPE in FacebookProvider

My bad, I forgot to test how other providers reacted the Facebook change. Not well apparently since users would get a NPE every time they tried to log in with a provider besides Facebook. Thankfully, this wasn't released. 😌

:+1:


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.3 to before v4.14.265",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=47dceb8ecdc1c3ad1818dfea3d659a05b74c3fc2",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=899e0319b3f58d85ac9a2f1d2895a71a275e2f4e"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "47dceb8ecdc1c3ad1818dfea3d659a05b74c3fc2",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "899e0319b3f58d85ac9a2f1d2895a71a275e2f4e",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "af_packet: fix data-race in packet_setsockopt / packet_setsockopt\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.265 by commit 899e0319b3f58d85ac9a2f1d2895a71a275e2f4e, it was introduced in version v4.3 by commit 47dceb8ecdc1c3ad1818dfea3d659a05b74c3fc2. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000108


Update documentation/comment for checkPortConflicts

This fix update the documentation/comment for checkPortConflicts so
that it is clear conflicts check will skip against the service that
is being udpated.
This fix is related to the dicsussion in 1349:
https://github.com/docker/swarmkit/pull/1349#discussion_r74348186
cc @stevvooe
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

LGTM

Current coverage is 55.28% (diff: 100%)

Merging #1361 into master will increase coverage by 0.02%

@@             master      #1361   diff @@
==========================================
  Files            80         80
  Lines         12620      12620
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6974       6977     +3
+ Misses         4685       4682     -3
  Partials        961        961

Powered by Codecov. Last update f0e46aa...ebc5708


Fixes issue #6071 - TIFF with 1 bit-depth

This commit is a simple solution to this issue.
The PDF on this issue has a TIFF image with bit-depth equal to 1. This issue was created because on stream.js, the case on  readBlockTiff() function which treats the TIFFs with bits == 1 was reading bytes wrongly: not only does it invert the colors, but I believe it also affects resolution.
The thing is, I tried to read TIFF specification and adapt the code that was already written, but I couldn't find this specific situation on the specification and I couldn't figure out what was wrong with the code.
I read in the issue page that the else fallback condition was working, and that the bits == 1 condition was there because of performance enhancement.
So I tried to write the else condition that I knew was working and optimize it for 1 bit depth.
I concluded that when a received bit is 1, the buffer bit (the bit that will belong to the predictor) has to be the opposite of the previous bit. When a received bit is 0, the bit given to the predictor stays the same.
For example, in TIFF 1-bit, a byte can be 128 or 0, so it can be either 1000 0000 or 0000 0000. The complement variable starts at 0 and only changes when there is a 1 in the received bits. I start reading the inbyte from the highest bit. I take the resulting complement value and I put it in the lowest order bit of the outbyte.
Now, according to TIFF specification, an image with 1-bit depth was not supposed to have any other values besides 128 and 0, but this one had, so I am forced to check for all the bits in a byte, besides the highest order one.
I have tested the performance between the else case and my 1-bit case, and although I based myself on the first, the second can be faster and it fixes the bug.
This is what I got, by running only the PDF on the issue page 20 times:

browser
stat
Count
Baseline(ms)
Current (ms)
+/-
%

Firefox
Overall
20
95
68
-27
-28.17

Firefox
Page Request
20
3
6
3
130.19

Firefox
Rendering
20
92
62
-30
-32.77

I'm unsure of what caused the Page Request to take longer.
I've tested it using gulp test with ReferencePages created before the merge and all worked fine.
Also, working on this, I noticed that the PDF does not have support for the PhotometricInterpretation tag that some 1-bit depth TIFFs can have. This tag defines whether WhiteIsZero or BlackIsZero, this is the color which a bit with value 0 represents.
I couldn't work out how to get that tag out of the PDF file, but once the value is retrieved, implementing it is very easy. The initial value of the "complement" variable could change according to the tag, inverting the color scheme. For example, if on the issue PDF you start complement with 1, the background turns black and the letters white, but with better resolution than the current code.

Hello timvandermeij and Snuffleupagus. Thanks for the feedback. I rearranged some things and forgot about the code style. I've corrected both the identation and the commit message as you asked.
Are there any more things you want to point out or know about the code?

Are there any more things you want to point out or know about the code?

I'd suggest that you remove the Corrected sintaxe according to style guide. part of the commit message, since that's unnecessary (and also, please note that the correct spelling is syntax :-)
Unfortunately I don't know the TIFF code at all, so I'm not sure that I'd be able to do a meaningful code review here. Perhaps @yurydelendik can help with that, if he has time!?

Sorry, it was my first squash commit and I thought it only kept the first message. Correction done ;)

Hi again, sorry for the delay in answering.
@yurydelendik I took your approach and created a slightly more efficient version. When you told me to eliminate the inner cycle I didn't really know if you meant what I wrote or creating a pre-mapping array.
In this next table, baseline is your code and current is my commit:

browser
stat
Count
Baseline(ms)
Current (ms)
+/-
%

Firefox
Overall
20
113
103
-10
-9.09

Firefox
Page Request
20
12
9
-3
-22.50

Firefox
Rendering
20
101
93
-7
-7.46

I don't know if my testing conditions were different than the initial commit, because these values are higher than the ones I first posted... But testing with my first commit, this code was 14% faster.
Also, about the PhotometricInterpreation thing, I'll try to test it, but I'm having problems:
I can create a TIFF image with 1-bit depth and 1 color, but I can't convert it to .pdf format and "activate" this block we're testing. Any chances you know how to solve this and convert/incorporate the TIFF image correctly?

Just some proof:

Also, about the PhotometricInterpreation thing, I'll try to test it,

I would like to keep PhotometricInterpreation out-of-scope of this PR.

@yurydelendik Sorry for not explaining the optimization. I couldn't figure it out alone either, but got help from Stack Overflow because I believed there was a way to do it. Turns out it was quite simple. Not a huge jump in optimization, but still better.
Let me know if there's anything else to change. I'll open a PR for PhotometricInterpretation once I can test it.

/botio test
can you add a reference test?

@yurydelendik How could I add a test? And should I use the pdf of the issue? Do we have the copyrights to use it?

You will need to modify test_manifest.json file (see it's history https://github.com/mozilla/pdf.js/commits/master/test/test_manifest.json for examples). Depending on if pdf's origin is unknown (or copyrighted) and we does not know if it's allowed to distribute it, we just including the reference (via webarchive?) -- link test. But we prefer if pdf author agree to include the example and we can add it to the repo.

Example: https://github.com/mozilla/pdf.js/commit/c5b06cb40d032e782f5afa3a0ed73d4ee80f160a (only the changes for files in the test/ folder.

Should I make another commit or squash the addition of the test? Btw, should I upload the PDF itself or a webarchive link to it? If I point the link, can it still be added to test_manifest?

You should squash the test into the existing commit. Since we do not have explicit permission to add the PDF to the repository, a Web Archive link would be good. Example of a test manifest entry with a Web Archive link: https://github.com/mozilla/pdf.js/commit/54ee83eb125ff6bbf95a4e7a0e7ef2493e4aae4d

I tried doing that, but since the PDF is hosted on DropBox, I cannot save a link to it on WebArchive, unless I reupload it somewhere else.

Sry for bumping this one up again, but what should I do regarding the ref test? The DropBox link is protected and I cannot upload it to WebArchive.

I'm sorry for forgetting to answer this. You can use this Web Archive link: https://web.archive.org/web/20170107214304/https://www.pdf-archive.com/2017/01/07/issue6071/issue6071.pdf

I believe I added the test correctly. Please review the new commit and let me know if something is wrong or should be added.

/botio test

Note that the Linux bot failed to download the new file (but it worked on the Windows bot):
Downloading https://web.archive.org/web/20170107214304/https://www.pdf-archive.com/2017/01/07/issue6071/issue6071.pdf to pdfs/issue6071.pdf...
Error during downloading of https://web.archive.org/web/20170107214304/https://www.pdf-archive.com/2017/01/07/issue6071/issue6071.pdf: HTTP 503
WARNING: File was not downloaded. See "pdfs/issue6071.pdf.error" file.

Says it was the checksum that failed, but why would it fail on Linux and now on Windows?

I don't think it's an issue with your pdf. The error was 503, let's just re-try linux bot.

/botio-linux test

Linux bot says the checksum is different, but I checked on Linux just now, with the md5sum command, and it gives me the same checksum as Windows. But I'm checking other .pdf.link files and they all have the first line with the link and a line break, making it 2 lines. Mine only has 1. Should I fix that and retry?

/botio-linux test

Okay, I placed it in the linux's pdf-cache

/botio makeref

Thank you for the patch

Thanks for guiding me in here. I'm glad I could help even though it was a small contribution. ;)

Hi - I seem to be getting the same inversion issue when using the svg renderer -- the test file (and all of my other images) come out with white rendered as black and black rendered as white. Before I post additional details, should this fix described above also work for SVG or is the TIFF handling code completely separate ? So far in all of my digging this thread has come closest to describing the problem. Note that when I use the canvas approach it comes out perfectly.
Happy to post my code and examples, please let me know if this is an appropriate place for this !
Thanks

It looks like this is unrelated to this PR. Please open a new issue with the requested information.