Get the editor to fill parent height

Hello! I have a div of say... 1000px tall. How do I set the editor height to 100% to get it to fill full space? I know about min-height and max-height but the problem is, my div is dynamic. It changes its size in different screens.
Is it possible to get the editor to fill its parent? If so, how?
Thanks for all the work. Much appreciated.

I managed to achieve this by writing these styles:
.EasyMDEContainer {
    height: 100% !important;
    display: flex !important;
    flex-direction: column;
}
.CodeMirror {
    flex: 1;
}

Not sure if it's the correct way or a bad hack. Keeping this issue open still, in hope to hear from the author.

This seems to work.
.EasyMDEContainer, .CodeMirror {
    height: 100% !important;
}

.EasyMDEContainer .CodeMirror:not(.CodeMirror-fullscreen) {
    height: 100%;
}


How can I add debugging for json-loader?

How can I add debugging or logging for json-loader in webpack 4?
Feature request

What is the expected behavior?
What is motivation or use case for adding/changing the behavior?
How should this be implemented in your opinion?
Are you willing to work on this yourself?
yes

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Artist not found and Album not found when browsing "Show artist" or "Show album"

For some reason when I try to see the Artist page from the context menu on the musicbox's default webclient (from wouter) I get a strange behaviour.
Not only does it say artist not found, but it seems that the angular or single page app is broken - instead of just
http://<IP_ADDRESS>/#browse
I now get
http://<IP_ADDRESS>/#artists?local:artist:md5:259c6b3c747416f3b5273566559b756c
A little looking and I see that if I go the route of  browse->artist the code is:

 !!!

However in the queue page with the show artist popup, the code is
Show Artist !!!

Without looking into the details, this looks more like an issue in woutervanwijk/Mopidy-MusicBox-Webclient to me..,

AFAICS there's already an issue for this: woutervanwijk/Mopidy-MusicBox-Webclient/issues/62


throw new Error("Connection failure event: " + eventData.error.toString());

[12/28/2023, 9:48:26[PM] [JciHitachi Platform] Restarting Process...
[12/28/2023, 9:48:27[PM] [JciHitachi Platform] Launched child bridge with PID 27067
[12/28/2023, 9:48:28[PM] Registering platform 'homebridge-jci-hitachi-platform.JciHitachi Platform'
[12/28/2023, 9:48:28[PM] [JciHitachi Platform] Loaded homebridge-jci-hitachi-platform v1.0.10 child bridge successfully
[12/28/2023, 9:48:28[PM] [JciHitachi Platform] Attempting to log into JciHitachiAWSAPI.
[12/28/2023, 9:48:28[PM] Homebridge v1.6.1 (HAP v0.11.1) (JciHitachi Platform) is running on port 57837.
[12/28/2023, 9:48:29[PM] [JciHitachi Platform] You have 7 devices.
[12/28/2023, 9:48:31[PM] [JciHitachi Platform] Host User : Roget,Chou
/homebridge/node_modules/homebridge-jci-hitachi-platform/src/jci-hitachi-aws-api.ts:817
throw new Error("Connection failure event: " + eventData.error.toString());
^
Error: Connection failure event: Error: libaws-c-mqtt: AWS_ERROR_MQTT5_CONNACK_CONNECTION_REFUSED, Remote endpoint rejected the CONNECT attempt by returning an unsuccessful CONNACK
at Mqtt5Client. (/homebridge/node_modules/homebridge-jci-hitachi-platform/src/jci-hitachi-aws-api.ts:817:19)
at Mqtt5Client.emit (node:events:513:28)
at Mqtt5Client.emit (/homebridge/node_modules/homebridge-jci-hitachi-platform/node_modules/aws-crt/lib/common/event.ts:95:22)
at /homebridge/node_modules/homebridge-jci-hitachi-platform/node_modules/aws-crt/lib/native/mqtt5.ts:623:20
at processTicksAndRejections (node:internal/process/task_queues:77:11)
[12/28/2023, 9:48:31[PM] [JciHitachi Platform] Child bridge process ended
[12/28/2023, 9:48:31[PM] [JciHitachi Platform] Process Ended. Code: 1, Signal: null

Due to changes in the AWS MQTT authentication protocol, the original process is no longer effective and requires further investigation.

Fixed. Please see commit

Fixed.

I've tested, the new version works! Thank you for your hard working!!!!

I've tested, the new version works! Thank you for your hard working!!!!


Ignoring node['rvm']['default_ruby']

I have the following attributes:
override['rvm']['default_ruby']      = "2.1"
override['rvm']['user_default_ruby'] = "2.1"
override['rvm']['rubies'] =  ['2.1']

And recipe:
...
include_recipe "rvm::system"
...
rvm_shell "gem install chef" do
  code %Q{gem install chef --no-rdoc --no-ri -v 11.14.6}
  timeout 36000
end

This machine bootstrapping via Vagrant.
During installing chef I'm getting the following:
==> knife-solo: ================================================================================
==> knife-solo: Error executing action `run` on resource 'script[gem install chef]'
==> knife-solo: ================================================================================
==> knife-solo:
==> knife-solo:
==> knife-solo: Mixlib::ShellOut::ShellCommandFailed
==> knife-solo: ------------------------------------
==> knife-solo: Expected process to exit with [0], but received '1'
==> knife-solo: ---- Begin output of "bash"  "/tmp/chef-script20141118-5776-1elfd1m" ----
==> knife-solo: STDOUT: Now using system ruby.
==> knife-solo: STDERR: ERROR:  Error installing chef:
==> knife-solo:
==> knife-solo:     mime-types requires Ruby version >= 1.9.2.
==> knife-solo: ---- End output of "bash"  "/tmp/chef-script20141118-5776-1elfd1m" ----
==> knife-solo: Ran "bash"  "/tmp/chef-script20141118-5776-1elfd1m" returned 1
==> knife-solo:
==> knife-solo: Cookbook Trace:
==> knife-solo: ---------------
==> knife-solo: /tmp/vagrant-chef-2/chef-solo-1/cookbooks/rvm/providers/shell.rb:99:in `script_wrapper'
==> knife-solo:
==> knife-solo: /tmp/vagrant-chef-2/chef-solo-1/cookbooks/rvm/providers/shell.rb:46:in `block in class_from_file'
==> knife-solo:
==> knife-solo: Resource Declaration:
==> knife-solo: ---------------------
==> knife-solo: # In /tmp/vagrant-chef-2/chef-solo-1/cookbooks/rvm/providers/shell.rb
==> knife-solo:
==> knife-solo:  76:   s = script new_resource.name do
==> knife-solo:  77:     interpreter   "bash"
==> knife-solo:  78:
==> knife-solo:  79:     if new_resource.user
==> knife-solo:  80:       user        new_resource.user
==> knife-solo:
==> knife-solo:  81:       if user_rvm && new_resource.environment
==> knife-solo:  82:         environment({ 'USER' => new_resource.user, 'HOME' => user_home }.merge(
==> knife-solo:
==> knife-solo:  83:           new_resource.environment))
==> knife-solo:
==> knife-solo:  84:       elsif user_rvm
==> knife-solo:
==> knife-solo:  85:         environment({ 'USER' => new_resource.user, 'HOME' => user_home })
==> knife-solo:  86:       end
==> knife-solo:  87:     end
==> knife-solo:
==> knife-solo:
==> knife-solo: Compiled Resource:
==> knife-solo: ------------------
==> knife-solo: # Declared in /tmp/vagrant-chef-2/chef-solo-1/cookbooks/rvm/providers/shell.rb:76:in `script_wrapper'
==> knife-solo:
==> knife-solo:
==> knife-solo:
==> knife-solo: script("gem install chef") do
==> knife-solo:   action [:nothing]
==> knife-solo:   retries 0
==> knife-solo:   retry_delay 2
==> knife-solo:
==> knife-solo:   guard_interpreter :default
==> knife-solo:   command "\"bash\"  \"/tmp/chef-script20141118-5776-1elfd1m\""
==> knife-solo:   backup 5
==> knife-solo:   returns [0]
==> knife-solo:   timeout 36000
==> knife-solo:   code "    if [ -s \"${HOME}/.rvm/scripts/rvm\" ]; then\n      source \"${HOME}/.rvm/scripts/rvm\"\n    elif [ -s \"/etc/profile.d/rvm.sh\" ]; then\n      source \"/etc/profile.d/rvm.sh\"\n    fi\n\n    rvm use system\n\n    gem install chef --no-rdoc --no-ri -v 11.14.6\n"
==> knife-solo:
==> knife-solo:   interpreter "bash"
==> knife-solo:   cookbook_name :"knife-solo"
==> knife-solo:
==> knife-solo: end
==> knife-solo:
==> knife-solo:
==> knife-solo:
==> knife-solo:
==> knife-solo: ================================================================================
==> knife-solo: Error executing action `run` on resource 'rvm_shell[gem install chef]'
==> knife-solo: ================================================================================
==> knife-solo:
==> knife-solo:
==> knife-solo: Mixlib::ShellOut::ShellCommandFailed
==> knife-solo: ------------------------------------
==> knife-solo: script[gem install chef] (/tmp/vagrant-chef-2/chef-solo-1/cookbooks/rvm/providers/shell.rb line 76) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
==> knife-solo: ---- Begin output of "bash"  "/tmp/chef-script20141118-5776-1elfd1m" ----
==> knife-solo: STDOUT: Now using system ruby.
==> knife-solo: STDERR: ERROR:  Error installing chef:
==> knife-solo:     mime-types requires Ruby version >= 1.9.2.
==> knife-solo:
==> knife-solo: ---- End output of "bash"  "/tmp/chef-script20141118-5776-1elfd1m" ----
==> knife-solo: Ran "bash"  "/tmp/chef-script20141118-5776-1elfd1m" returned 1
==> knife-solo:
==> knife-solo:
==> knife-solo: Cookbook Trace:
==> knife-solo: ---------------
==> knife-solo: /tmp/vagrant-chef-2/chef-solo-1/cookbooks/rvm/providers/shell.rb:99:in `script_wrapper'
==> knife-solo:
==> knife-solo: /tmp/vagrant-chef-2/chef-solo-1/cookbooks/rvm/providers/shell.rb:46:in `block in class_from_file'
==> knife-solo:
==> knife-solo:
==> knife-solo: Resource Declaration:
==> knife-solo: ---------------------
==> knife-solo: # In /tmp/vagrant-chef-2/chef-solo-1/cookbooks/knife-solo/recipes/default.rb
==> knife-solo:
==> knife-solo:
==> knife-solo:
==> knife-solo:  40:     rvm_shell "gem install #{name}" do
==> knife-solo:
==> knife-solo:  41:       code %Q{gem install #{name} --no-rdoc --no-ri -v #{version}}
==> knife-solo:
==> knife-solo:  42:       timeout 36000
==> knife-solo:
==> knife-solo:  43:     end
==> knife-solo:
==> knife-solo:  44:   else
==> knife-solo:
==> knife-solo:
==> knife-solo:
==> knife-solo: Compiled Resource:
==> knife-solo: ------------------
==> knife-solo: # Declared in /tmp/vagrant-chef-2/chef-solo-1/cookbooks/knife-solo/recipes/default.rb:40:in `block in from_file'
==> knife-solo:
==> knife-solo:
==> knife-solo: rvm_shell("gem install chef") do
==> knife-solo:   action :run
==> knife-solo:   retries 0
==> knife-solo:
==> knife-solo:   retry_delay 2
==> knife-solo:   guard_interpreter :default
==> knife-solo:   cookbook_name :"knife-solo"
==> knife-solo:   recipe_name "default"
==> knife-solo:   code "gem install chef --no-rdoc --no-ri -v 11.14.6"
==> knife-solo:
==> knife-solo:   timeout 36000
==> knife-solo:
==> knife-solo:   ruby_string "default"
==> knife-solo:
==> knife-solo:   returns [0]
==> knife-solo:
==> knife-solo: end
==> knife-solo:
==> knife-solo:
==> knife-solo:

When I ssh to this Vagrant box, I see that no ruby were installed with RVM. I've noticed on this line ==> knife-solo: STDOUT: Now using system ruby., so I've tried to add somewhere before include_recipe "rvm::system" removing OS wide installed ruby package. After that everything working as expected.
A month ago or so it was successful with the same Vagrant box.
Host OS Ubuntu 14.10, RVM 1.25.28, Ruby 2.1.2, Chef 11.14.6, Berkshelf 3.1.5
Vagrant is CentOS 6.5

Closing due to inactivity.
If this is still an issue please reopen or open another issue. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help!
Thanks,
Sous-Chefs


Possibility to search across all repositories of the organization

We need to add the ability to search across all the organization's repositories. This shouldn't trigger a reindexing of all repos, but it should enable searching through all the organization's data.
We'll need to modify the structure to make this happen and use the same interface we currently use for individual repositories. However, we'll add extra filters that will allow us to also filter by repositories within the organization.

Implemented in this pull request: https://github.com/manticoresoftware/manticore-github-issue-search/pull/9

We're closing this issue because the pull request has been merged. The deployment will coincide with the vector search release.


Youtube videos slider

I have creted a slider of youtube iframes.
I have set enableTouch to true, and also enableDrag to true.
But there is something in iframe preventing the swipe to work.
I can't find what it is.
Does anyone have an idea of how to fix it?
Thanks in advance

@mcdesignpro  could you replicate the issue on jsfiddle, http://jsfiddle.net


ERROR 1: Image file is too small

What cause this:
ERROR 1: Image file is too small

Please provide more information as requested on versions of isce, gdal etc. There is no information in this ticket to respond meaningfully.

Please provide more information as requested on versions of isce, gdal etc. There is no information in this ticket to respond meaningfully.

stackSentinel.py -s ../org -d ../dem/demLat_N32_N34_Lon_E107_E110.dem -o /Volumes/MSWAP/S1_orbit -a ../aux -c 2 -n 2 -b '33.49 33.55 108.60 108.70' -z 1 -r 3 -e 0.7 -W slc
when i use the  -W slc, there is no error now!


Could not find method compile() error on Android

Encountered an error on latest RN version 0.68.1
A problem occurred evaluating project ':react-native-radial-gradient'.

Could not find method compile() for arguments [com.facebook.react:react-native:+] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

Environment:
"react": "17.0.2",
"react-native": "0.68.1",
"Platform": "Android only"

Same error for me on Android

"react-native": "0.68.0"
"react-native-radial-gradient": "^1.0.9"

compile is not available since Gradle 7.0. Not sure how to use implementation with the project function

This worked for me, as a temporary fix.
Edit node_modules\react-native-radial-gradient\android\build.gradle to replace compile with implementation.
In the android/app/build.gradle file use
implementation(project(':react-native-radial-gradient'))

inside the dependencies block.


Fixture url updated

URL no longer works.  New link = https://docs.pytest.org/en/latest/reference/reference.html#pytest.fixture

Thanks, but this whole page is getting rewritten so I'm going to close this since it will just be replaced.


[Style] Allow passing a custom theme object to change the styling

Is your feature request related to a problem? Please describe.
There is no easy way to custom kepler.gl UI styling
Describe the solution you'd like
kepler.gl uses styled-component, its looking & feel mostly controlled by a single json [theme object][https://github.com/uber/kepler.gl/blob/master/src/styles/base.js].
We will use dependency injection, add themeFactory as another dependency of KeplerGl component, and make the theme object as the return of themeFactory.  User can import default theme object, merge it with their customTheme, and inject a customThemeFactory.

is this related to this one? https://github.com/uber/kepler.gl/issues/346

Implemented in this one: https://github.com/uber/kepler.gl/pull/424


Provide all metrics from status page in metrics endpoint

Describe your issue or feature request
For our VerneMQ setup we established monitoring via the Prometheus endpoint (http://localhost:8888/metrics) as documented here https://docs.vernemq.com/monitoring/prometheus.
Due to the way we have structured our integration tests, we had an intermittent issue with a too high amount of offline sessions. During the cleanup of this issue we noticed that the amount of offline sessions, as well as other interesting metrics are only available via the status page
{
	"0": {
		"VerneMQ@<node>": {
			"num_online": 1,
			"num_offline": 2,
			"msg_in": 3,
			"msg_out": 4,
			"queue_in": 5,
			"queue_out": 6,
			"queue_drop": 0,
			"queue_unhandled": 0,
			"num_subscriptions": 7,
			"num_retained": 8,
                        [...]
		}
	}
}

although vernemq_retain_messages{node="VerneMQ@<node>"}is available in the metrics endpoint, e.g.  num_offline is not.
Describe the solution you'd like
It would be very helpful if all metrics from the status page

num_online
num_offline
msg_in
msg_out
queue_in
queue_out
queue_drop
queue_unhandled
num_subscriptions
num_retained

would also be available in the metric endpoints
Describe alternatives you've considered
We implemented an automatic scraping via cronjob, which now is a monitoring solution that runs in parallel to the metric endpoints based ones.
Environment

VerneMQ Version:
OS:
Erlang/OTP version (if building from source):
Cluster size/standalone:

Add any other context, code samples or screenshots about the issue report here.
No response
Code of Conduct

[X] I agree to follow the VerneMQ's Code of Conduct

@pellepelster Thanks. In short any exported metrics are based on the same list of metrics. You can display this with vmq-admin metrics show. You can also use grep with the command: vmq-admin metrics show | grep queue
Most of the metrics you mention should be there, they might have slightly different names, though. Like gauge.retain_messages.
num_subscriptions on the Status page will be gauge.router_subscriptions.
You are right about the number of offline and online clients. This is calculated in the Javascript of the Status page. We have gauges for the queue processes and gauges for online connections, so the difference must be the offline clients.
You could do this kind of queries in Prometheus as well.
Let me know if you need more info.

👉 Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
👉 Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

Thanks for your quick response, that answers all questions

@pellepelster Good to hear, thank you for the feedback :)

👉 Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
👉 Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.


FCL - error

Dear all,
Pulling pinocchio from the repo master:master, I have an error with FCL when compiling:
include/pinocchio/multibody/geometry.hpp:144:36: error: no match for ‘operator==’ (operand types are ‘const fcl::CollisionResult’ and ‘const fcl::CollisionResult’)        return (fcl_collision_result == other.fcl_collision_result
README.md does not specify which fcl version to take and the dependency.
Best,
Olivier.

You're fully right.
The mandatory version of FCL is hpp-fcl (https://github.com/humanoid-path-planner/hpp-fcl/tree/devel) on the devel branch.
We have planed to put this version on the master branch of the repo and to push it on robotpkg.

I just pushed the devel on the master of hpp-fcl.

Thanks


DivIcon marker doesn't $compile the HTML inside itself

If I am trying to create an icon from an HTML template in angular-leaflet, I use the syntax:
icon: {
    type: 'div',
    className: 'marker-default',
    iconSize: null,
    html: '

'
}

where I declared $scope.clicked somewhere else. It seems expected behavior should be for angular-leaflet to compile the HTML inside the icon before displaying it, but it doesn't, for instance if I set $scope.clicked = true. Is this a bug or is this how angular-leaflet is supposed to work? If the latter perhaps it should be changed because as it is I need to $compile the HTML manually, which violates MVC.

I think this would be a really great feature because we would be able to display complex content inside markers!
However, we need to find a way to provide the scope that the HTML has to be compiled with...

Did you get this working with compile? If so, would you mind putting it up here? Struggling with the same thing right now.

Nope because I couldn't manage to pass the scope so far...

Can anyone shed some light on this?

Managed to got a result by creating a small Leaflet plugin that works like DivIcon but take directly a dom element to be append to the marker. Then inside Angular Leaflet I add a special case to handle icon of domIcon type where I compile the template string with the scope provided through a getMarkerScope function.
Here's the code:

Warning: this is just a prototype, there seems to be some problems when updating a marker:

The Leaflet plugin:
L.DomIcon = L.DivIcon.extend({
    options: {
        element: false
    },

    createIcon: function() {
        var div = this.options.element;

        this._setIconStyles(div, 'icon');

        return div;
    }

});

L.domIcon = function(options) {
    return new L.DomIcon(options);
};

Then inside this block, I add the following code:
if (isDefined(iconData) && isDefined(iconData.type) && isDefined(iconData.getMarkerScope) && iconData.type === 'dom') {
    var markerScope = iconData.getMarkerScope(),
        template = $compile(iconData.html)(markerScope);
    iconData.element = template[0];
    return new L.domIcon(iconData);
}

Then to use it inside your controller:
$scope.your_variable = 'THIS WORKS';

//inside your map config object
markers: {
    osloMarker: {
        lat: 59.91,
        lng: 10.75,
        icon: {
            type: 'dom',
            iconSize: [20, 20],
            className: 'your-class',
            getMarkerScope: function() {
                return $scope
            },
            html: '
{{ your_variable }}
'
        }
    }
}

Any remarks are welcome. I'm still working on this to improve it and to be able to provide a templateUrl to be loaded inside the icon...

I will submit a dedicated PR tomorrow if people are interested by this feature.
I think that this enables us to do many cool stuffs. I already tried crazy stuffs like input fields on the map or a slider for instance. I'm sure that people will find cool features to implement thanks to it.

Yeah I actually switched over from google maps specifically for this feature thinking it would work. I think it would be quite useful to have interactive markers.

This issue was moved to angular-ui/ui-leaflet#83

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.

Actually, I submitted the PR I mentioned above and it was merged in master. It added a DomMarker plugin to handle this use case: https://github.com/tombatossals/angular-leaflet-directive/blob/master/src/services/leafletMarkersHelpers.js#L70 . So this issue was already fixed I think.


[🐞] navigate() to a child route no longer works

Which component is affected?
Qwik City (routing)
Describe the bug
Calling navigate to a child route does not navigate and the promise never resolves.
But, navigating to a route 2 levels deep did work.
This this happens in a useVisibleTask or a QRL called from a useVisibleTask
useVisibleTask$(async () => {
    console.log("This is called");
    await navigate("/demo/");
    console.log("This never runs");
  });

Reproduction
https://github.com/DustinJSilk/qwik-issue-navigate
Steps to reproduce
Install and run
expect the home page to redirect to /demo/
System Info
N/A

Additional Information
No response

There are several issues open about awaits in tasks, notice it's holding the render. This is a cause of a dead-lock, not sure how we can solve this without you dont returning the promise.

As a workaround:
useVisibleTask$(() => {
    console.log("This is called");
    navigate("/demo/").then(() => {
      console.log("This never runs");
    });
  });

Perfect thanks for taking a look @manucorporat !

As a workaround:
useVisibleTask$(() => {
    console.log("This is called");
    navigate("/demo/").then(() => {
      console.log("This never runs");
    });
  });

ie, dont return the promise

Perhaps this should be mentioned on the docs for tasks until (and if) it is fixed, as currently if one hits such an issue, they'd have to find this (or one of the other) issues which is less than ideal.


Cannot read properties of undefined (reading 'rest_id')

hey, looks like sometimes twitter sends invalid response on user.replies.
on 4.1.0 version got Cannot read properties of undefined (reading 'rest_id')
on latest one got One or more validation error(s) occured, check details field.
[
  ValidationErrorDetails {
    field: 'results',
    constraints: [ 'results must be empty' ]
  }
]

found that sometimes tweet.quoted_status_result is {} on this line

hot fix base on 4.1.0 version

Sorry for the late reply.
Thanks for the hotfix. I did discover the cause of the issue, unfortunately, I've been a lot busy lately so couldn't find the time to work on it. I'll try to merge the hotfix from your fork, which will will buy me time for working on a solution.
Thanks for the contribution and feedback. Means a lot.

Hey! I'm playing around with this package (thank you btw!) and I'm facing the same problem with the search method.
Debugging I found that in the constructor of the Tweet class is checking for tweet.quoted_status_result to assign the value for quoted:
this.quoted = tweet.quoted_status_result ? new Tweet(tweet.quoted_status_result.result) : undefined;
But I got a case where tweet.quoted_status_result was an empty object {} so it was trying to do new Tweet({}) ending in the previous error because the property result wasn't found in quoted_status_result:

Logs:
[Rettiwt-API] [2024-10-16T19:53:51.363Z] [AUTHORIZATION] {"authenticated":true}
[Rettiwt-API] [2024-10-16T19:53:51.363Z] [VALIDATE] {"target":"FETCH_ARGS"}
[Rettiwt-API] [2024-10-16T19:53:51.363Z] [GET] {"target":"HTTPS_AGENT"}
[Rettiwt-API] [2024-10-16T19:53:51.363Z] [GET] {"target":"USER_CREDENTIAL"}
[Rettiwt-API] [2024-10-16T19:53:52.127Z] [DESERIALIZE] {"id":"1844376999810433054"}
[Rettiwt-API] [2024-10-16T19:53:52.127Z] [DESERIALIZE] {"id":"1844369358476439922"}
[Rettiwt-API] [2024-10-16T19:53:52.127Z] [DESERIALIZE] {"id":"1844364557973848458"}
...\fetch-tweet\node_modules\rettiwt-api\dist\models\data\Tweet.js:21
        this.id = tweet.rest_id;
                        ^

TypeError: Cannot read properties of undefined (reading 'rest_id')```

Issue has been fixed. Please upgrade rettiwt-api to v4.1.2.

There was a "Maximum call stack exceeded" error while deserializing tweet(s)  in v4.1.2.
Please upgrade to v4.1.3 instead.


export configuration ast_zoness and bump torch for generic array factory

couldnt figure out what was going on at D_ZO_602AE94 and couldnt figure out how to change the name of D_601996C

couldnt figure out what was going on at D_ZO_602AE94 and couldnt figure out how to change the name of D_601996C

I fixed the name thing, and took a look at D_ZO_602AE94, there's definitely another type of hitbox here we don't know about or something.
@petrie911 can you take a look?

Looking at the data, it really looks like a Vec3f array. Could be more than one. Unclear what they represent. Perhaps orphaned meshes from cut colpolys?


A lot more incompatibilities from the Dotty Community projects

As soon as this is merged I will upgrade to 0.26.0-RC1 and remove the javaVarargs that was solved.

I also added the documentation of the Dotty auto-application rewrite


Danger changelog rule

Checklist

[x] New Extension
[ ] New Test
[ ] Changed more than one extension, but all changes are related
[x] Trivial change (doesn't require changelog)

          2 Messages

  :book:
  Executed 148 tests, with 0 failures (0 unexpected) in 1.476 (1.734) seconds

  :book:
  Executed 136 tests, with 0 failures (0 unexpected) in 1.009 (1.278) seconds

  Generated by :no_entry_sign: danger

Current coverage is 36.79% (diff: 100%)

Merging #323 into danger will not change coverage

@@             danger       #323   diff @@
==========================================
  Files            46         46
  Lines          2082       2082
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            766        766
  Misses         1316       1316
  Partials          0          0

Powered by Codecov. Last update b45a1e0...e511f9c

should be merged as soon as #266 is merged


Feature: AI features should force a login

Type of feature
🍕 Feature
Current behavior
We are offering free access to our OpenAI generation. A user with the extension installed should see the extension button, but they should be sent to the login flow when they click the button while not logged in.
Suggested solution

While logged out of the extension
Click the pr description generation
Start login flow

Additional context
I noticed this was not the flow when opening this PR https://github.com/open-sauced/hot/pull/470
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

We are doing that in the button's onclick handler.
https://github.com/open-sauced/ai/blob/43d4161eeb02c4ccb250abc41cfadc39673c3672/src/content-scripts/components/GenerateAIDescription/DescriptionGeneratorButton.ts#L25-L29.
I'll recheck this.

@bdougie, the redirect works for me. Can you try to reproduce the bug you encountered?

@bdougie, can we close this as we haven't been able to reproduce the issue?


refs counter for each overloads [feature]

Sup, can we have refs counter and filter for each overloads ?  ((per signature))
thanks

current:

expected:

This feature is powered by the TypeScript project. Please file a feature request upstream
But note, this also applies to find all references and you likely can't do this safely because functions can be passed as objects


docs/guide-ja minor updates

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
--

Fixed issues
--

ja doc updates

Thank you


[Enhancement] Move code to GitLab or Codeberg

[Delete before submission] Please use the latest build from Github Actions (Artifacts)
Is your feature request related to a problem? Please describe.
GitHub is not an open source platform, and being owned by Microsoft makes it more of a danger to open source projects, rather than a help.
Describe the solution you'd like
It would be nice if the project's code were moved to GitLab or Codeberg.

GitHub is not an open source platform

@julianfairfax I know this issue is old and has been closed, but I'd like to say that this statement isn't entirely accurate. You can't control what code Codeberg or GitLab runs, nor see that it's actually what is running there. Open source is irrelevant when it comes to other people's servers.

being owned by Microsoft makes it more of a danger to open source projects

Being owned by someone or something is irrelevant. We're here talking now, and if something does happen, the project can be moved elsewhere. Who says Codeberg or GitLab won't change policy or go offline? Every platform poses a risk; being owned by Microsoft or Google etc doesn't make it worse.


add nethttp extension

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.60%. Comparing base (6a4f0ea) to head (4985f6d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #169   +/-   ##
=======================================
  Coverage   82.60%   82.60%
=======================================
  Files          18       18
  Lines         713      713
=======================================
  Hits          589      589
  Misses        102      102
  Partials       22       22

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Bubblegum support

Machine config and kernel only for the time being.

merged as-is. we can fix the description in another PR.


bug: llama2 needs model assignment

🐛 Describe the bug
Just like OpenAI or GPT4All, Llama2 apparently needs a model to work. It could also be an issue with Replicate as the provider.

find out if this is a replicate or llama2 issue
find a suitable model name
set it as default in the Llama2 llm

File "pydantic/main.py", line 341, in pydantic.main.BaseModel.init pydantic.error_wrappers.ValidationError: 1 validation error for Replicate model   none is not an allowed value (type=type_error.none.not_allowed) [15/Sep/2023 20:16:18] "POST /bot-response/ HTTP/1.1" 500 86328

Open for community contributions.

I'll check for it and see if I can do it.

It's working for Llama2App. Where is it failing then?

Hi @cachho @deshraj Any update on this?


Virtualbox Guest Additions fix

This seems to fix issue #16641 for some people.

This patch fixes the problem for me. Can this also be cherry-picked into release-16.03 please?
Thanks @matthewbauer!

I would prefer to just rollback the VirtualBox update to 5.0.12 on release-16.03. We really don't need the newer version in 16.03 AFAICT.

@matthewbauer don't we actually want the virtualbox 5.0.* releases on 16.03 since they are maintenance releases fixing bugs? (of course in this case it caused a bug...)

Either way is fine with me. I'm just not sure if there are more issues introduced with the 5.0.16 update.,

5.0.20 is running without problems for me.

LGTM
please cherry-pick into release-16.03 for users of newer VirtualBox versions on OS X and Windows.


Update OneBranchPipeline.yml file

Signed-off-by: xiaowuhu<EMAIL_ADDRESS>add variable ob_outputDirectory so the pipeline can run with the predefined template.

I scheduled a run for your change: https://aiinfra.visualstudio.com/Lotus/_build/results?buildId=217604&view=results


fix: Simplifying PR title check with icons

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Tensorflow documentation on Image Captioning has non existing methods.

URL(s) with the issue:
I was running Image captioning example given in tensorflow website as it is in Colab. I found out that the documentation made a mention about "image_features_extract_model" which was not defined anywhere earlier. Neither could I find any method related to that on Google.
The lines that caused the error is given below:
# Get unique images
encode_train = sorted(set(img_name_vector))

# Feel free to change batch_size according to your system configuration
image_dataset = tf.data.Dataset.from_tensor_slices(encode_train)
image_dataset = image_dataset.map(
  load_image, num_parallel_calls=tf.data.experimental.AUTOTUNE).batch(16)

for img, path in image_dataset:
  batch_features = image_features_extract_model(img)
  batch_features = tf.reshape(batch_features,
                              (batch_features.shape[0], -1, batch_features.shape[3]))

  for bf, p in zip(batch_features, path):
    path_of_feature = p.numpy().decode("utf-8")
    np.save(path_of_feature, bf.numpy())

The original documetation can be found at the following url:
https://www.tensorflow.org/tutorials/text/image_captioning
I was confused whether i should report this as a bug or documentation error. I went with the latter.
Is the link to the source code correct?
Yes
The original documetation can be found at the following url:
https://www.tensorflow.org/tutorials/text/image_captioning

Hi, I think image_features_extract_model is defined in the notebook. See this line in the "Initialize InceptionV3 and load the pretrained Imagenet weights" section.
image_model = tf.keras.applications.InceptionV3(include_top=False,
                                                weights='imagenet')
new_input = image_model.input
hidden_layer = image_model.layers[-1].output

image_features_extract_model = tf.keras.Model(new_input, hidden_layer)

I'm running the notebook in Colab. If I can reproduce your error, I will report back here.

@Mahe-git2hub, As @jaketae  mentioned image_features_extract_model is already defined in notebook. I have successfully compiled above mentioned tutorial and there is no error. For your reference link of gist is here.
Hope it helps.

Hi, I am new here so bear with me and let me know if I should make a new thread! I get a bit similar error from the same part of the same tutorial code:
`NameError                                 Traceback (most recent call last)
 in ()
----> 1 encode_train = sorted(set(img_name_vector))
2
3 # Feel free to change batch_size according to your system configuration
4 image_dataset = tf.data.Dataset.from_tensor_slices(encode_train)
5 image_dataset = image_dataset.map(
NameError: name 'img_name_vector' is not defined`
Have you others not had this or have you fixed it? img_name_vector seems to be defined in the tutorial so I really wonder what the problem is.


ReferenceError: check is not defined with Meteor <IP_ADDRESS>

I can't start meteor because of the following error:
ReferenceError: check is not defined
    at new Files (packages/ostrio:files/files.coffee:147:5)
    at Files.allow.insert (packages/app-filestore/lib/files.js:1:1)
    at /home/jokke/code/projects/tagfs/.meteor/local/build/programs/server/packages/app-filestore.js:28:4
    at /home/jokke/code/projects/tagfs/.meteor/local/build/programs/server/packages/app-filestore.js:92:3
    at /home/jokke/code/projects/tagfs/.meteor/local/build/programs/server/boot.js:222:10
    at Array.forEach (native)
    at Function._.each._.forEach (/home/jokke/.meteor/packages/meteor-tool/.<IP_ADDRESS>sddkj++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
    at /home/jokke/code/projects/tagfs/.meteor/local/build/programs/server/boot.js:117:5

Thank you for contribution, please update to v1.3.7


Cannot use on 1.12.2 Paper

Describe the bug
Just cannot use this lib.
NPCLib usage option
I tried in all options.
NPCLib JAR name
npclib-plugin-2.11.1-SNAPSHOT.jar
Server version
Paper version git-Paper-1618 (MC: 1.12.2) (Implementing API version 1.12.2-R0.1-SNAPSHOT)
To Reproduce
Steps to reproduce the behavior:

Used code

See error

Expected behavior
I want to spawn a NPC...

Can you provide the full class?

Can you provide the full class?

@Mooselk

https://github.com/Xierip/TestNpcLib

@Mooselk

https://github.com/Xierip/TestNpcLib

It happens on everytime :/

It happens on everytime :/

Looks like an issue with tinyprotocol, it’s probably incompatible with paper. I don’t know enough about it to fix it.

Looks like an issue with tinyprotocol, it’s probably incompatible with paper. I don’t know enough about it to fix it.

I’ll see what I can do about it at the end of the week. I’m a bit busy at the moment.

I’ll see what I can do about it at the end of the week. I’m a bit busy at the moment.

Can you link the URL to your specified PaperSpigot version? I can't seem to reproduce this issue using the latest version of PaperSpigot...
From what I see in the code is that there's a pending and confirmed array for the server connection class, at least in the 1.16.5 version. It doesn't seem plausible that this is what is causing this strange null pointer, so I will investigate that a little bit more.

Can you link the URL to your specified PaperSpigot version? I can't seem to reproduce this issue using the latest version of PaperSpigot...
From what I see in the code is that there's a pending and confirmed array for the server connection class, at least in the 1.16.5 version. It doesn't seem plausible that this is what is causing this strange null pointer, so I will investigate that a little bit more.

@JitseB This is weird, I runned now this same server, and now its working O_o
I added it to minigame plugin and its working fine too.

@JitseB This is weird, I runned now this same server, and now its working O_o
I added it to minigame plugin and its working fine too.

But on lobby plugin for minigame not working, server setup is equal

But on lobby plugin for minigame not working, server setup is equal

Oh ok, its issue with old protocollib plugin i think.
Thats really weird

Oh ok, its issue with old protocollib plugin i think.
Thats really weird

Extraordinary. ProtocolLib should not interfere with TinyProtocollib. Good to hear that you got it working now!
Also makes sense now that I can't reproduce it. I only have NPCLib and a test plugin on my server.
Closing issue, if you happen to find that this issue is caused by something else, please do tell!

Extraordinary. ProtocolLib should not interfere with TinyProtocollib. Good to hear that you got it working now!
Also makes sense now that I can't reproduce it. I only have NPCLib and a test plugin on my server.
Closing issue, if you happen to find that this issue is caused by something else, please do tell!


No route for adding to basket from tutorial

The tutorial doesn't mention a route to post for products when adding them to the basket. What is that route?
Here is my current routes.rb based on the tutorial. What is the post path? I've tried using the shoppe-example ones
  mount Shoppe::Engine => "/shoppe"
  get 'product/:permalink' => 'products#show', :as => 'product'
  #root :to => 'products#index'
  get 'store', :to => 'products#index'
  get 'basket' => 'orders#show'
  delete 'basket' => 'orders#destroy'

Also, the API Docs are not there
Here is the error I get when Clicking "add to basket" for a seed product
Started POST "/product/snom-870" for <IP_ADDRESS> at 2015-02-26 14:41:58 -0500

ActionController::RoutingError (No route matches [POST] "/product/snom-870"):
  web-console (2.0.0) lib/action_dispatch/debug_exceptions.rb:22:in `middleware_call'
  web-console (2.0.0) lib/action_dispatch/debug_exceptions.rb:13:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  actionpack (4.2.0) lib/action_dispatch/middleware/show_exceptions.rb:30:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  railties (4.2.0) lib/rails/rack/logger.rb:38:in `call_app'
  railties (4.2.0) lib/rails/rack/logger.rb:20:in `block in call'
  activesupport (4.2.0) lib/active_support/tagged_logging.rb:68:in `block in tagged'
  activesupport (4.2.0) lib/active_support/tagged_logging.rb:26:in `tagged'
  activesupport (4.2.0) lib/active_support/tagged_logging.rb:68:in `tagged'
  railties (4.2.0) lib/rails/rack/logger.rb:20:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  actionpack (4.2.0) lib/action_dispatch/middleware/request_id.rb:21:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  rack (1.6.0) lib/rack/methodoverride.rb:22:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  rack (1.6.0) lib/rack/runtime.rb:18:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  activesupport (4.2.0) lib/active_support/cache/strategy/local_cache_middleware.rb:28:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  rack (1.6.0) lib/rack/lock.rb:17:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  actionpack (4.2.0) lib/action_dispatch/middleware/static.rb:113:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  rack (1.6.0) lib/rack/sendfile.rb:113:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  airbrake (4.1.0) lib/airbrake/user_informer.rb:16:in `_call'
  airbrake (4.1.0) lib/airbrake/user_informer.rb:12:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  railties (4.2.0) lib/rails/engine.rb:518:in `call'
  railties (4.2.0) lib/rails/application.rb:164:in `call'
  newrelic_rpm (<IP_ADDRESS>9) lib/new_relic/agent/instrumentation/middleware_tracing.rb:67:in `call'
  rack (1.6.0) lib/rack/content_length.rb:15:in `call'
  thin (1.6.3) lib/thin/connection.rb:86:in `block in pre_process'
  thin (1.6.3) lib/thin/connection.rb:84:in `catch'
  thin (1.6.3) lib/thin/connection.rb:84:in `pre_process'
  thin (1.6.3) lib/thin/connection.rb:53:in `process'
  thin (1.6.3) lib/thin/connection.rb:39:in `receive_data'
  eventmachine (1.0.7) lib/eventmachine.rb:187:in `run_machine'
  eventmachine (1.0.7) lib/eventmachine.rb:187:in `run'
  thin (1.6.3) lib/thin/backends/base.rb:73:in `start'
  thin (1.6.3) lib/thin/server.rb:162:in `start'
  rack (1.6.0) lib/rack/handler/thin.rb:19:in `run'
  rack (1.6.0) lib/rack/server.rb:286:in `start'
  railties (4.2.0) lib/rails/commands/server.rb:80:in `start'
  railties (4.2.0) lib/rails/commands/commands_tasks.rb:80:in `block in server'
  railties (4.2.0) lib/rails/commands/commands_tasks.rb:75:in `tap'
  railties (4.2.0) lib/rails/commands/commands_tasks.rb:75:in `server'
  railties (4.2.0) lib/rails/commands/commands_tasks.rb:39:in `run_command!'
  railties (4.2.0) lib/rails/commands.rb:17:in `<top (required)>'
  bin/rails:4:in `require'
  bin/rails:4:in `<top (required)>'
  -e:1:in `load'
  -e:1:in `<main>'

Hey, I'm going to be working on updating the documentation so expect to hear something either over the weekend or next week :)

You should be able to use the example though as it is currently running at http://demo.tryshoppe.com/

thanks, but thats the example I mentioned in my comment. It is out of date by at least a year.

Hi @cdesch ,
I faced the same issue, so this is what worked.
route:
post 'buy/:permalink' => 'products#buy', :as => 'buy_product'

link in products/show.html.haml
= link_to "Add to basket", buy_product_path(@product.permalink), :method => :post

Hope this helps!
Regards

Thank you @SkyKOG  You just saved me ;_;

@5sp4lr dont mention it :) :+1:

The documentation on the site has now been updated :+1:  http://tryshoppe.com/docs/tutorials/setting-up


[Question]: Why do you require pandas 1.3.5?

Question form pre-submit checklist.

[X] I have searched the existing issues to ensure there isn't already an issue about this question.
[X] My question has to do with the Python SDK and isn't a general question about the API. (If it is please open your issue here)
[X] My question isn't about how to do a specific algorithm or asking for trade advice (answers to these are outside the scope of this repo).

Question
Is there a reason (a good reason) why this SDK requires pandas version 1.3.5?
I installed it yesterday to slowly ease into the new SDK. During installation, pandas was downgraded from 1.4.X to 1.3.5
This change broke my code. Turns out there are major improvements made to pandas since 1.3.5

Hey @PBbanana thanks for bringing this up. We've updated the pandas requirement.
Versions v0.5.5 and above reflect this change.


don't set the DB engine factory to None by default

by setting to None, we override the default. Using **kwargs so that we don't override the default.

Good catch. This probably should've been caught by a unit test


Publishing the code search guide on the learn site?

Do you all think the learn site is a good place for the Code search guide to live? That linked gdoc has an overview of what it will contain. IMO the learn site would work well for it (and thank you for making it! :tada:).
@jennybergen Were you thinking it'd live here, or did you have somewhere else in mind?
Our model is an article like https://stripe.com/atlas/guides/saas-pricing, which is also on a "learn site" (i.e., not a technical doc site).
Other options:

We could publish the code search guide in our doc site like https://docs.sourcegraph.com/adopt/code_search_in_monorepos (which is another general article that's not completely Sourcegraph-specific). Cons: this would make it seem less neutral and more Sourcegraph-specific, and we want the code search guide to describe all code search tools.
We could publish the code search guide on its own site, but that would require a lot more effort/maintenance.
We could publish the code search guide on about.sourcegraph.com, but that also seems less neutral and more Sourcegraph-promotional.

We are definitely open to this, would love to hear more from the Content team!

@sqs after speaking with @ltagliaferri and Erica, we agree that the Learn site would be a good place for the Code Search Guide.
I do want to surface one temporary concern. Since the current state of the Learn site features only Sourcegraph-specific content, I worry that it may suffer from the same issue you pointed out above under posting on the docs site:

Cons: this would make it seem less neutral and more Sourcegraph-specific, and we want the code search guide to describe all code search tools.

I know the plan with the Learn site is to create product-agnostic content, so I don't think it's a problem in the long term but still want to mention it to make sure this is not an issue when it comes to asking others outside of Sourcegraph to contribute.
The other awesome thing about posting on the Learn site is that we have engineering support to make the guide look nice right now! I spoke with Katy about building this on the Learn site today (with Lisa's blessing!) and showed her the Stripe example for inspiration. Their layout is similar to what's already on the Learn site, so Katy said getting a V1 of the guide built and published by Thursday of next week (July 29), should be doable.
And lastly, Erica and I discussed having the URL be https://learn.sourcegraph.com/guides/code-search. We want to include "/guides" in the URL to have a home for future guides on the Learn site. Okay by you both?

Thank you for the update @jennybergen — this all sounds really great to me! Really glad to provide a home to this important work.
I will see if we can also privilege some quick changes to the look of the site to make it seem less Sourcegraph-centric

Closing this as this will be living elsewhere, see https://github.com/sourcegraph/codesearchguide.org


Input for recipe item groups

Value Proposition
As a user, I want to add entire item groups to my shopping list according to specific recipes, so that I can ease item input.
Description

Acceptance

[ ] a second input field for recipe names
[ ] typing into the recipe input displays a drop down with auto-completed recipes
[ ] clicking on a drop down option, opens an overlay frame with listed recipe items
[ ] those items can be dropped by clicking a delete button
[ ] dropped items are removed from the rendered list in the overlay frame
[ ] clicking a "Check-In" button at the bottom, puts all non-dropped items on the shopping list
[ ] a Cancel button on the top right, which closes the overlay frame for the recipe items

Tasks

[ ] add second input field in the form
[ ] implement drop-down menu for auto-completions, used in this user story: Auto-completion for item input
[ ] create hidden overlay component for displaying recipe items

[ ] displays name & type of recipe + listed recipe items
[ ] every item has a delete button

[ ] create onClick function for drop down options

[ ] displays hidden overlay component with respective recipe

[ ] create delete function for items on the recipe list, which removes it from the list
[ ] create "Check-in" button for recipe list, to add recipe items to shopping list state
[ ] create a "Cancel" button
[ ] create onClick function on Cancel button, which closes the recipe overlay frame

Wow! What a great feature and what a great story! 😃
My only concern is that the story is quite large, so your PR may contain too much code? But I don't have a good idea how to cut it smaller either.

Wow! What a great feature and what a great story! 😃 My only concern is that the story is quite large, so your PR may contain too much code? But I don't have a good idea how to cut it smaller either.

 Habe mir deinen Comment zu Herzen genommen und diese riesige user story in 4 kleine verpackt :)

Hey nice user-story 👍
Your using input fields as component right? You could write something like

[ ] add Input component (recipes) - (editor page)

i think there is a Typo:
maps across the filtered array - map


Bug fix for Issue #177

I encountered the same bug as Issue#177, so I created this PR.

Thanks for this helpful fix and sorry for my absence lately due to 996...


create api-error error subclass. checks status for error handling whe…

…n reading compliance info

Codecov Report

Merging #992 into master will decrease coverage by 0.38%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##           master     #992      +/-   ##
==========================================
- Coverage    27.8%   27.42%   -0.39%
==========================================
  Files         171      171
  Lines        4643     4649       +6
  Branches     1221     1222       +1
==========================================
- Hits         1291     1275      -16
- Misses       3352     3374      +22

Impacted Files
Coverage Δ

wherehows-web/app/constants/errors/errors.ts
100% <ø> (ø)
:arrow_up:

wherehows-web/app/utils/api/shared.ts
100% <ø> (ø)
:arrow_up:

wherehows-web/app/utils/api/datasets/compliance.ts
63.63% <80%> (-7.34%)
:arrow_down:

wherehows-web/app/utils/api/errors/errors.ts
69.23% <87.5%> (-19.66%)
:arrow_down:

wherehows-web/app/components/dataset-compliance.ts
22.18% <0%> (-5%)
:arrow_down:

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 2fa1f66...543a72d. Read the comment docs.

Coverage remained the same at 27.651% when pulling 543a72d3647bb2f82c8c20ecce628b5c18400780 on theseyi:api-errors into 2fa1f66014fea152dc042e198c7e8d65b7710838 on linkedin:master.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build
for the Release configuration

deleted the branch


Query: GroupJoin queries should use new Include compiler.

Continuing the Include refactoring to apply to GroupJoin queries with ref Includes.

Doing some more work on this.


Right joy-con analog input lag

Hey, I'm not sure if this is simply how the program interacts with games on steam but I've noticed when trying the program that when using the right joy-con analog, there is either an input lag of around half a second to a full second when adjusting camera angles and in other game I've tried, the camera seems to jerk around instead of smoothly following the analog inputs. Is there a way to fix this or something I can do to help in the config or something to that affect?
Games I've tried where the camera proved to be a bit of an issue: Dark Souls III, Risk of Rain 2

Same problem for me.


EF generic context

By putting the SqlConnection in the DbContext, isn't this coupling the EF Context to Sql database ? Is there a way to avoid this ?

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

ID: 31b0d33c-f88e-d50c-2926-7933940f3b92
Version Independent ID: 4aaef906-f501-6947-5d23-1b704470f09d
Content: Secure SQL Database connection with managed identity - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-connect-msi.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

Hi @swasifali, Thanks for the feedback! We are currently investigating and will update you shortly.

@swasifali After reviewing the doc and reproducing it, I do see how this might be a cause for concern. As a result, I have assigned the feedback to the doc author to have them determine if this could cause issues and whether or not a change is necessary.

@swasifali Thanks for your feedback. However, the EF context isn't the focus of the tutorial, but showing how to add the managed identity token is. The EF context approach is the same as the approach taken by the default MVC template so this article isn't the best place to discuss whether that's the right approach or not. #please-close


Paraquire - paranoidal require

Greetings from Russia, comrads!
I've prepared pull request using paraquire to limit permissions of some dependencies of browserify. It will protect against some cases of dependency hijacking.
Unfortunately, in some cases it is unclear what right does the library need.
Please feel free to discuss and ask questions.

Update: I see the problems with Node <= 0.12 and I'm working on compatibility or fallback


Disable windows build

So... with the windows tests actually passing now, why do we need to do that?

thread-local stack cl breaks windows build and I it seems at this point not useful to support windows. So I think we can disable it for now.


Change code for support Android 5.1.1

Thank you for amazing library JFoenix. One year ago I found it for my projects, during this year you are fixed my issues for support Android 5.1.1 platform. If you have a time, please fixed some "forEach" codes in sources of JFoenix:
https://github.com/jfoenixadmin/JFoenix/search?q=forEach&unscoped_q=forEach

Other foreach statements are either part of the demo app or deprecated API and will be removed later.
Regards,


Move to an organization

This will make it easier for us to collaborate and manage the repo / travis CI settings.
I've created a new organization chip8-rust and invited contributor @robo9k. Once the invitation is accepted I'll transfer the repo there (waiting for acceptance so that assigned issues transfer too).
cc: #8 #5

I've just now accepted the invitation to @chip8-rust

Starting this now. After you'll want to update your remote urls:
git remote set-url origin $new_url

Thanks for the heads-up!
I'm not entirely certain, but I might need Owner rights to create a token to use Travis CI to push to GitHub Pages. Investigating..

Not sure if this helps:
https://help.github.com/articles/requesting-organization-approval-for-your-authorized-applications/

I think pushing commits via Travis could work with encrypted deploy keys. I'll look into that once everything else it set up, but I'm sure there is a way :book:

Sounds good. Let's continue that discussion on the respective issues.


[CSV-303] Add revapi to fail build on breaking API changes

This pull request solves ticket CSV-303 by adding org.revapi:revapi-maven-plugin to the build section of the library's maven POM.
Please note that config file revapi-config.json is added to exclude CSVFormat.serialVersionUID from checks. The field should probably be updated to something != 1 along with custom de-serialization for backward compatibility (see ticket CSV-302).

-1 We already do that using JApiCmp: See https://github.com/apache/commons-csv/blob/0fd5302689d7e366ad0ac398030f774f364756fb/pom.xml#L190

-1 We already do that using JApiCmp: See [..]

which failed to recognize the serialization bug you introduced in CSV-302
revapi is the best available tool to enforce api compatibility.
IMO you're being too hasty closing CSV-303.

Hello @sman-81
One way to maintain sanity when developing and maintaining the over 20 components that makeup Apache Commons is to centralize standard plugins in commons-parent. I really do not want to special case one component to use one API checking plugin when all other components use another. If the community wants to change from JApiCmp to RevAPI, that's fine, but let's attempt to maintain consistency for (my) sanity's sake (at least). I'll let others opine...

I searched for posts/tweets about revapi & jacimp. Found cases where one performed better than the other. But revapi appears to be under more active work.
@garydgregory do you think this could be something like jacoco & cobertura, where both are allowed in components, but with the configuration in the parent? Or would it be best to have a single library for all commons for consistency?
I don't have a preference, but if revapi is able to detect issues that escape jacimp, Id' be in favour of switching to revapi in the parent (after also checking with others in the mailing list).

-1 We already do that using JApiCmp: See
https://github.com/apache/commons-csv/blob/0fd5302689d7e366ad0ac398030f774f364756fb/pom.xml#L190

I don't think many developers use simply "mvn" to run Maven builds (I don't). I use mvn clean verify most of the time and expect essential build steps to be part of the configured build lifecycle.
japicmp does not bind by default to any lifecycle phase and as there is no execution configured, it won't be executed in builds.
The configuration of defaultGoal is nice, but I am afraid most of us won't spot it unless pointed to it (e.g. in a readme). Shall we add a line to CONTRIBUTING.md?
I also suggest to add an execution for japicmp-maven-plugin to the POM to bind goal cmp to phase verify. Wdyt?
          <execution>
            <phase>verify</phase>
            <goals>
              <goal>cmp</goal>
            </goals>
          </execution>

I am not much familiar with japicmp but I know revapi (log4j2 uses it btw). If japicmp does all that revapi does, then there is no need for revapi. If coverage is different, then the two can co-exist as @kinow suggested.
I think @garydgregory is right, such configuration -if any- should go to commons-parent (impressive POM btw!).
Fun fact: commons-parent has japicmp and clirr :)

I don't think many developers use simply "mvn" to run Maven builds (I don't). I use mvn clean verify most of the time and expect essential build steps to be part of the configured build lifecycle.

@sman-81 that's more of a convention that you might find in other Apache Commons components.

The configuration of defaultGoal is nice, but I am afraid most of us won't spot it unless pointed to it (e.g. in a readme). Shall we add a line to CONTRIBUTING.md?

Good idea, but I'm not sure if that's common to all components. The CONTRIBUTING.md file is auto-generated in some components during the release process - https://github.com/apache/commons-build-plugin/blob/master/src/main/resources/commons-xdoc-templates/contributing-md-template.md
So modifying that information would have to be in the commons-build-plugin, used by the parent component. But if that's not common to all components, then we'd have to think in some way to handle that…

I think @garydgregory is right, such configuration -if any- should go to commons-parent (impressive POM btw!).

:tada:

Fun fact: commons-parent has japicmp and clirr :)

Yup, there's some room for choosing different plug-ins and libraries in the Commons components, when these are inherited from the parent, e.g. https://github.com/apache/commons-parent/blob/8ca044d50e8eb64cf841ad4c758c02d86b92f44f/pom.xml#L1483-L1508
@sman-81 let's wait for @garydgregory 's reply, and see if we will have to move the question about revapi to a place where other committers are subscribed—not everybody gets github notifications, and within ASF the mailing lists are normally the meeting-place :)
Thanks!

A more typical approach for projects derived from Commons Parent would be to create:
src/main/resources/profile.japicmp

This will activate the JApiCmp profile in commons parent which binds the cmp goal to the verify phase. It also includes a reporting profile for the site build. So by doing it this way you get to collect all the configuration added to Commons Parent and adopt any changes as that evolves over time.
Note that CSV has a profile.jacoco but not one for japicmp. So you just copy the file and rename as appropriate.
CSV's default goal uses the clirr:check. This was OK when CSV was on an older version of Java but is not appropriate for new Java 8 features (the current build target). So switching to japicmp and dropping clirr seems to be the simplest solution rather than adding revapi will the goal of moving configuration to commons parent.
FYI: Commons RNG uses both japicmp and revapi because neither tool is perfect. Using either one on their own would not work for that component. However each can be configured to ignore non-issues that the other tool does not identify; and used to spot real issues that the other tool does not identify.

Hi All:
The developer's mailing is a better place for this conversation to have better exposure.
There are a lot of plugins to configure for an Apache Commons component build. The goal of avoiding binary breakage is one of them. CLIRR still exists in the parent POM because not all 20+ components have migrated to JApiCmp, though, at this point, they are also likely behind in commons-parent versions and Java versions. Most if not all active components are now on Java 8 which CLIRR does not know about.
Step one would be to drop CLIRR from the POM. Step two would be to discuss if JApiCmp should stay or be replaced.
You usually and ideally run the following in a default build which most of our default goals do: Apache RAT, Checkstyle, Spotbugs, Javadoc, PMD, JApiCmp. That's how our GitHub builds validate PRs, using the default goal. We can tell devs "Run mvn on the command line to check your PR.", but nothing more complicated.
Using the default Maven goal is a no-brainer IMO: One line in the POM gives you the ability to say "mvn" on the command line, and in CI builds like our GitHub Actions build. Arguing that adding 6 lines x 6 plugins instead of using the default Maven goal is just arguing for more noise and copypasta in the POM, and a POM is verbose enough as it is.  When something goes wrong or the build takes too long, I have to know more about Maven to turn off these plugins that insert themselves when I am trying to do something specific on the command line.
These are goals for me:

A simplest command that runs everything: mvn
The ability to run other Maven plugins without other plugins inserting themselves all over the place when I did not ask.

FYI: Commons RNG uses both japicmp and revapi because neither tool is perfect [..]

I agree with @aherbert's statement and think japicmp and revapi should be allowed to co-exist, and ideally do so in commons-parent. commons-rng (which has a custom revapi config much like the one suggested in my pr) and possibly other commons libraries would benefit from having a suitable configuration in one place: commons-parent
clirr should be dropped, it is hopelessly outdated, fully agree with @garydgregory.
revapi should be added and configured in a way that resembles the japicmp config as much as possible (using a profile.revapi file etc.)
All projects that derive from commons-parent should upgrade to the latest released version sooner rather than later.
If this summarizes the points and you all are in agreement I will write a message on<EMAIL_ADDRESS>


Revert "Docs: Remove gulp-sourcemaps because it is built-in (#2592)"

This pull request reverts a previous change to the Browserify documentation relating to sourcemaps.
Unfortunately the previous change results in the following error:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received an instance of Object
    at validateString (internal/validators.js:124:11)
    at Object.resolve (path.js:1039:7)
    at module.exports (node_modules/vinyl-source-stream/index.js:13:34)

You are half correct, vinyl-soure-stream won't take the option, but you can still write them with gulp. Want to fix that?

Wouldn't it make more sense to do that in a different pull request? gulp-sourcemaps is still used in 7 other recipes.

As noted on the recipes (I forget exactly where), they are bad and probably should just be removed until people have a chance to write them better. They are only there as a reference for when people want to pick things up.
I think we either make them better or leave them alone.

Reverting the breaking change and leaving it alone doesn't sound like a bad idea.

We don't do reverts, we only do fixes. I'm done arguing with you. If you want to fix the issue, feel free to send another PR.


Improve code-behind feature file compilation speed

🤔 What's changed?
When generating a Reqnroll project with many feature files, compilation can be improved (times for BigReqnrollProject with 500 feature files) from 5.4s to 1.1s:

original/main 5.4s
making FixVB RegEx static 5.2s
don't call VBFix in C# project 2.1s
cache gherkin dialect (provider) and avoid assembly resource loads 1.2s
avoid enum boxing 1.1s

This PR reduces compilation time by 80% (for large projects).
⚡ What's your motivation?
Trying to improve the compilation time of Reqnroll projects.
🏷 What kind of change is this?

:bank: Refactoring/debt/DX (improvement to code design, tooling, etc. without changing behaviour)

♻ Anything particular you want feedback on?
I used the
📋 Checklist:

[x] Users should know about my change

[x] I have added an entry to the "[vNext]" section of the CHANGELOG, linking to this pull request & included my GitHub handle to the release contributors list.

I added the PR number 🙂


Fixes incorrect HP display on cards in slack

I forgot to pass nature values to the card model's engine ZZZZ
closes https://github.com/austinpray/kaori/issues/113
adds priority to https://github.com/austinpray/kaori/issues/101

@ridhoq yeah it was a silly bug, I think I meant to go back and finish that function but didn't


gzip?

Hi, this is a random person who got this repo in github recommendations.
Comment1: cool and nice, but ungzipped matrix market is a very expensive option. Comparison with gzip is more valuable IMO
Comment2: And as a baseline, it's valuable to see something terribly simple based on columnar format, e.g.:
import polars as pl
from scipy import sparse
import numpy as np

x = sparse.coo_array(np.random.randn(1000, 1000)) # not sparse, I know

# save to parquet
pl.DataFrame(dict(col=x.col, row=x.row, data=x.data)).write_parquet('/tmp/sparse_coo.pqt')

df = pl.read_parquet('/tmp/sparse_coo.pqt')
x2 = sparse.coo_array((df['data'].to_numpy(), (df['row'].to_numpy(), df['col'].to_numpy())))

np.allclose(x.todense(), x2.todense())

and see how these compares in terms of speed (because from perspective of floating precision binary formats certainly win)

Thanks for the feedback!

Comment1: cool and nice, but ungzipped matrix market is a very expensive option. Comparison with gzip is more valuable IMO
You're right that Matrix Market files are both large and compress well.

Curious, what's your workflow that uses gzipped files? I'm only aware of one popular sparse matrix library that can load gzipped files directly, SciPy, though FMM can be easily configured to do so (and the Python bindings do that already). All the others accept a plain .mtx file and will choke on anything not uncompressed.
It's probably worthwhile to add some compression benchmarks as well. I'll include GZip, but only to show how slow and a bottleneck it is. Something like lz4 or zstd would fit this application better.

Comment2

Good point, I'll add something like that we I get a bit of time. I'm curious how it would go myself.

single-cell-whatever, like single-cell rnaseq, single-cell atac, and other omics assays are (at some level) are basically huge matrices of cell x gene.
They use their own formats to keep different data associated, usually on top of hdf5 or similar format.
And they usually just keep row/col/data as I've done in the python excerpt above.
https://anndata.readthedocs.io/en/latest/ and h5seurat are (I think) the most widely adopted, but 1) it clearly targets the application 2) additionally keeps different analysis artifacts, which makes the format more complicated.
OTOH anndata package is quite user-friendly.

compressed mtx seems still supported: https://support.10xgenomics.com/single-cell-gene-expression/software/pipelines/latest/output/matrices

compressed mtx seems still supported: https://support.10xgenomics.com/single-cell-gene-expression/software/pipelines/latest/output/matrices

Ok cool. So the Python side uses SciPy's loader:

mat = scipy.io.mmread(os.path.join(matrix_dir, "matrix.mtx.gz"))

FYI the SciPy loader is as slow as anything on this list (for now). I wrote FMM partly to speed it up, and it does. You can expect 20x speedup on your laptop: https://github.com/alugowski/fast_matrix_market/tree/main/python
It'll handle gzipped files too, but I haven't benchmarked the effect on speed. I expect the decompressor to be the bottleneck (for FMM, not for Python scipy.io). Probably still worth it for the disk usage savings, though maybe a native C++ decompressor might be worth looking at.
I do say "for now", because we're close to getting FMM merged into scipy, so in the future that example code should magically get faster for free for you.
Based on that example it looks like R's readMM can handle gzipped files too. That's good to know.

so in the future that example code should magically get faster for free for you.

cool.
But just in case: I don't use mtx/mtx.gz and I think very few do these days.
I appreciate a lot your endeavors because scipy's mmread is indeed fantastically slow, but I think we'd better promote some binary format as an alternative.  A bit faster reads, a bit slower writes. File size is smaller than MM, but larger than compressed MM.
Bad performance when the matrix is a large fraction of memory, but that is common, especially in Python land.
Honestly I'm disappointed in all the binary formats I've tried. They should be maxing out the IO, not going head-to-head with a human-readable archival format.
Again this is on a 6-core laptop. FMM can use more cores.

What does your hdf5 workflow look like? What sorts of performance do you see?

FMM can use more cores.

Not sure I'd love using many cores for IO, but tastes differ :)

op:read/impl:Polars/format:Parquet/0/iterations:1/real_time        27.7 s         0.053 s

0.053 sec? strange, likely just IO bound. You have good ssd?
I've ran my benckmark with dense-pretends sparse with 1GB file:
reading:
CPU times: user 2.35 s, sys: 778 ms, total: 3.13 s
Wall time: 2.43 s
construction of scipy matrix
CPU times: user 167 ms, sys: 153 ms, total: 319 ms
Wall time: 347 ms
That's mac m1, but most systems with reasonable SSD should be in the same range.
Parquet has settings for better compression, just as gzip.

For hdf5, pipeline is the same - you use h5py  to store row, cols, indices.
Or use https://anndata.readthedocs.io/en/latest/api.html with read_h5ad and write_h5ad, which does this and saves additional metadata.
HDF5 <> parquet: I think these days parquet is more widespread than hdf5, and tooling like fastparquet is usually better. Also parquet got traction from big tech + it is more like a standard, whereas hdf5 is more like 'standard implementation'.

FMM can use more cores.

Not sure I'd love using many cores for IO, but tastes differ :)

Of course inefficient I/O isn't good, and matrix market certainly isn't optimal for speed.
But if those cores are idle and the operation is compute bound, which ALL of these appear to be, Parquet included, then why not use the idle cores to finish faster?

op:read/impl:Polars/format:Parquet/0/iterations:1/real_time        27.7 s         0.053 s

0.053 sec? strange, likely just IO bound. You have good ssd?

27.7 wallclock time.
Notice it's a 10GB file on a machine with 16GB RAM. The point of that test is to see how well the I/O library handles matrices that take up more than half of RAM. Often they like to duplicate the matrix, either explicitly somehow or implicitly by a cache.
This is one example of bad performance. The 10G file is 10x larger than 1G but takes 100x as long.

I've ran my benckmark with dense-pretends sparse with 1GB file:
reading:
CPU times: user 2.35 s, sys: 778 ms, total: 3.13 s Wall time: 2.43 s

So 10x slower than what my benchmark showed? I saw 0.272s for a sparse 1GB file with Parquet.
Just export your file to .mtx and run python bench_polars.py to run my code against it. It'll also compute effective bytes/second which while imperfect in its own ways at least gives some context whether the number is good or bad.

construction of scipy matrix
CPU times: user 167 ms, sys: 153 ms, total: 319 ms Wall time: 347 ms
That's mac m1, but most systems with reasonable SSD should be in the same range.
Parquet has settings for better compression, just as gzip.

I also have an M1 mac.

I've pointed to unusually low CPU usage, now I see the reason.

This is one example of bad performance.

benchmark makes no practical sense.
If you don't have twice memory, you can't operate it in python (as practically anything creates a copy). You could appreciate that a trivial and efficient format was assembled in a couple of lines, and I put zero efforts in optimizing it.
And yes, there is conversion from polars to numpy that creates a copy.
If you put some efforts (namely, reading separately col-row-data) you could alleviate the "problem" without performance downsides. That's three more lines, but why solving non-issue?

Just export your file to .mtx

I don't have that much space on drive - mtx is 6 times larger for smaller sizes.
Compressing would make this comparison fair and meaningful, but will reduce any margins. That was my original point.
For the sake of science, side to side, but on smaller size:
saving/loading/converting to scipy
Wall time: 992 ms  CPU times: user 686 ms, sys: 401 ms, total: 1.09 s
Wall time: 601 ms  CPU times: user 925 ms, sys: 278 ms, total: 1.2 s
Wall time: 60 ms  CPU times: user 43.4 ms, sys: 16.7 ms, total: 60.1 ms
saving/loading/converting to scipy with snappy compression, level=1
Wall time: 564 ms  CPU times: user 322 ms, sys: 326 ms, total: 647 ms
Wall time: 247 ms  CPU times: user 530 ms, sys: 113 ms, total: 643 ms
Wall time: 295 ms  CPU times: user 161 ms, sys: 43.1 ms, total: 204 ms
fmm  (in python, reading to scipy, includes conversion)
CPU times: user 7.62 s, sys: 767 ms, total: 8.39 s
Wall time: 1.34 s
filesizes:
2.3G Jul 16 18:56 /tmp/sparse_coo.mtx
476M Jul 16 18:55 /tmp/sparse_coo.pqt # snappy
390M Jul 16 18:59 /tmp/sparse_coo.pqt # default
I don't doubt you have the fastest mtx reader in the west.
I doubt mtx is a way forward. Community finally (after so many years) makes conversion csv > parquet, similar should happen with other formats, they should be replaced with scalable ones. You know, bills for AWS storage converts people better than arguments 😄
There are a ton of issues around using sparse in python if you're curious for some practical problems:

mtx does not have binary representation and thus bit-level correspondence during parsing can't be guaranteed. Specially io in your lib is updated, and you spend hours trying to figure out why numbers changed. At the time there is no standard binary format.
there are dramatic problems with sparse matrices/ndarray in scipy itself - matrices are legacy with bad API, and ndarrays did not flush that legacy.
Xarray has a number of request for support of sparse, but does not add ones because of scipy's limitations: https://github.com/pydata/xarray/issues/3213#issuecomment-615499609
sparse ndarrays https://sparse.pydata.org/en/stable/generated/sparse.save_npz.html#sparse.save_npz with ndarrays and a better API, but use python-specific npz for serialization. It could become a standard for sparse matrices, but presence of sparse in scipy prevents this.
no standard sparse with labeled axes (as opposed e.g. to R with sparse dataframes). AnnData is close to filling that gap, but it is used only among biologists

I have to disagree that large matrix benchmarks make no sense. Maybe your use case doesn't care, but there are many that do.
Naturally 10GB is an extreme example, but on purpose. The slowdowns begin around 4GB.  That's a big drop.
Also the writes are at 200MB/s at all file sizes.
A binary format should be able to do much better than that.
Maybe you're ok with writing custom code for every file, but that should not be required. It's certainly a deal breaker in a lot of applications.
I'm not trying to sell Matrix Market as the future for everyone. It does have its applications (maybe you're not in the target market for it, that's ok) and benefits that I won't list here because I know you don't care about them (and that's ok). I use it here because it's the only universal format so it serves as a useful comparison target.

There are many ways to do compression in C++, if you have favorites that would work well with this repo LMK.


pull request for e2e test at 01/31/2017 21:56:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


SelectionArea doesn't allow user to select the first TextSpan

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Run the sample code in dartpad.dev
Drag over Lorem to select it

Expected results

The word is selected

Actual results

It is not selected.

Code sample
https://dartpad.dev/?id=76db6a5a5629b306b0d00fcf8c773f81
Screenshots or Video

Screenshot

Logs
No logs.
Flutter Doctor output
No doctor output.

Thanks for the report @daohoangson
I verified this using latest master and stable and observed that on web and desktop, the reported behavior occurs but only on stable. If you run the scenario on latest master on both platforms, you'll notice that it works as expected.

Since it works properly on latest master, I'll consider this as fixed already and will close.


⚠ Site ANTS has degraded performance

In c4ae26d, Site ANTS (https://ants.gouv.fr) experienced degraded performance:

HTTP code: 200
Response time: 12576 ms

Resolved: Site ANTS performance has improved in 3631c08.


mitogen 0.3.x: async with docker_container hangs forever

Hi, I have a minimally reproducible bug report of async not working (hanging until cancelled) with community.docker.docker_container using mitogen 0.3.x (or master) and ansible 2.10.9.
requirements.txt
ansible>=2.10,<2.11
mitogen==0.3.0rc1

playbook.yml
---

- name: mitogen async test
  hosts: all
  tasks:
  - name: Test mitogen async
    become: yes
    async: 320
    poll: 3

    community.docker.docker_container:
      name: "mitogen-async-test"
      image: "debian:bullseye-slim"
      command: "echo mitogen async test"

bootstrap conda env
conda create -n mitogen-async python=3.8
conda activate mitogen-async
pip install -r requirements.yml
git clone https://github.com/mitogen-hq/mitogen.git

verify it works without mitogen
ANSIBLE_PYTHON_INTERPRETER=/usr/bin/python3 ansible-playbook -u remoteuser -kKi 'someserver,' playbook.yml

doesn't work with pip installed mitogen 0.3.0rc1
ANSIBLE_STRATEGY_PLUGINS=./mitogen/ansible_mitogen/plugins/strategy ANSIBLE_STRATEGY=mitogen_linear ANSIBLE_PYTHON_INTERPRETER=/usr/bin/python3 ansible-playbook -vvv -u remoteuser -kKi 'someserver,' playbook.yml

doesn't work with master mitogen
ANSIBLE_STRATEGY_PLUGINS=$(find "$CONDA_PREFIX/lib/" -type d -name "python*.*")/site-packages/ansible_mitogen/plugins/strategy ANSIBLE_STRATEGY=mitogen_linear ANSIBLE_PYTHON_INTERPRETER=/usr/bin/python3 ansible-playbook -vvv -u remoteuser -kKi 'someserver,' playbook.yml

Which version of Ansible are you running?

2.10.9

Is your version of Ansible patched in any way?

no

Are you running with any custom modules, or module_utils loaded?

no

Have you tried the latest master version from Git?

yes

Do you have some idea of what the underlying problem may be?

no

Mention your host and target OS and versions

Host: Debian bullseye/sid
Target: Debian buster

Mention your host and target Python versions

Host: 3.8.5
Target: 3.7.3

If reporting a crash or hang in Ansible, please rerun with -vvv and include
200 lines of output around the point of the error, along with a full copy of
any traceback or error text in the log. Beware "-vvv" may include secret
data! Edit as necessary before posting.

mitogen-async.log

If reporting any kind of problem with Ansible, please include the Ansible
version along with output of "ansible-config dump --only-changed".

ansible-config dump --only-changed has no output (no config changed)

Same issue if using async for https://docs.ansible.com/ansible/latest/collections/community/aws/ec2_instance_module.html
It does nothing and you can wait forever.
Any workaround to dynamically disable mitogen for a task/role?

You can set strategy on a per play basis, strategy: linear will disable mitogen until the next play

This also makes it awkward to run mitogen with molecule, where their async docker-creation gets stuck waiting for it to complete https://github.com/ansible-community/molecule-plugins/blob/11fb13f5fc332fb69f85ee3359b847fac85e2da5/src/molecule_plugins/docker/playbooks/create.yml#L206-L214
It looks like the status files never get updated after their first creation, but I was struggling to trace in the mitogen code where exactly that ought to be happening.


Improve error messages when LCW doesn't allow unverified credentials to be sent

If a user tries to send un-verified or not-verifiable credentials, LCW shows an error message.
Two credentials, both not verified:

Initial modal is fine, it's properly styled to match others in LCW:

Details could be improved by styling them instead of reporting console for just the first credential that cannot be shared (we took a shortcut to report console entries and not fully style)

@bmuramatsu; How's this for an error message (please provide your edit):
Unable to send credentials
One or more of them are not valid.

@bmuramatsu - wondering if we should not display the invalid credentials on the send credentials page. What do you think?

Question came up that maybe it's ok to share unverified credential in some cases - like if it expired or if the issuer isn't in our registry.

Original suggestion was to style the responses instead of reporting as a console entry for consistency of UI. I was thinking something similar to the details that are shown when a wallet backup is restored.
I think I'd advocate for specific replies if we have an error message. Credential has expired. Credential has been revoked. Credential does not have a valid digital signature. (These mirror our verification statements.)

Question came up that maybe it's ok to share unverified credential in some cases - like if it expired or if the issuer isn't in our registry.

If we allow users to send credentials that we are not able to verify, it does leave them in control of what they do with their credentials.
I don't think we should add logic to only allow some unverified credentials to be sent and not others, I think we should do all or nothing for the long term support version.

Moving to backlog for team discussion soon. Thanks!


FLEX-1546 Adds properties for elster

FLEX-1546

 You should sign our Contributor License Agreement in order to get your pull request merged.


Tidy up HeadingPitchRoll API

This PR uses the new HeadingPitchRoll objects introduced in #4047 more widely in the API. It also removes the aircraftHeadingPitchRoll* functions introduced in the same PR.
@pjcozzi, could you take a quick look at this to check that it's on the right track? If so, similar work needs to be done for headingPitchRollToQuaternion and aircraftHeadingPitchRollQuaternion.

Can you fix the jsHint warnings?
Specs/Core/TransformsSpec.js
  line 364  col 13  'hpr' is defined but never used.
  line 895  col 13  'hpr' is already defined.

  ⚠  2 warnings

Looks good, thanks again @twpayne.

Thanks! I created #4506 to track the removal in 1.30.

Hello, I don't understand why did you remove aircraftHeadingPitchRoll* ? It answered to this case:
I think some people like me adress cesium for aeronautical uses and so the state of an aircraft is represented into a reference frame different of the classical East/North/Up. aircraftHeadingPitchRoll* is a sugar function that help this use case because you just need to address the heading, pitch, roll expressed in the classical aeronautical reference frame (North/West/Up) in order to work without more code lines!
What it annoys me the most is that my PR take a long time to be accepted and during this time  I explained why I need these functions and without a discussion, there are removed in the next PR. It's raging :-(

As the author of this PR, but not a representative of Cesium, the reasons for the clean-up were:

The two aircraftHeadingPitchRoll* functions were almost copy-and-pastes of the existing headingPitchRoll* functions, so it struck me that the API could be simplified by combining them.
Indeed, aircraftHeadingPitchRollQuaternion was a duplicate of headingPitchRollQuaternion.
aircraftPitchHeadingRollToFixedFrame was a near-duplicate of headingPitchRollToFixedFrame with the exception that the reference frame in the implementation was changed from east-north up to north-west up. The comment, however, was inconsistent with the implementation, again suggesting copy-and-paste.
Inconsistent use of units and reference frames leads to subtle and occasionally catastrophic bugs, such as what happened to the Mars Climate Orbiter.
For this reason, it made sense to me that Cesium should use a consistent reference frame in its API.
Cesium has a wide range of users across many domains. Although north-west up frames might be common in aviation, the API would quickly become large and confusing if each domain added functions to support its own particular conventions to the API.
The underlying change - a change of reference frame for two functions - is easily implemented in the client code that calls Cesium (e.g. by duplicating the functions and making the necessary changes).

Hello, thanks for your reply.

For your three first points, I agree with you, it's almost a copy and paste of proven code. I just adjust the code to use the newly northWestUpToFixedFrame function.
For your fourth point, I am 100% with you. Now it's difficult to confound headingPitchRoll* with aircraftHeadingPitchRoll (suggested by Cesium team during the review of the code).
For the next two points, I think that Cesium API has taken a little shortcut that twists the way to work with: a matrix, a quaternion or heading/pitch/roll is just a mean to express a rotation (between two states, two frames...). In Cesium, it seems that the East/North/Up reference is a norm see here. I understand the choice which is like academic reference frame (x to the right, y to the up and z coming towards us). Now, I saw some helping requests in the forum to understand the way to work with the frames and how to make their awaited rotations. Obviously the responses are:

generate a local ENU,
whether make some rotations to attend your goal or convert your rotations in order to work with local ENU.
I think that Cesium should help the client/user/developer with a common API: most people use heading compared with north and not with east. Moreover developers aren't necessarily expert in math and don't understand how to change referential frames (see forum questions...). Maybe the function should be named commonHeadingPitchroll* :-).
Seriously, I personally think that headingPitchRollToFixedFrame should take as parameter the fixed frame to work with. By default, it should be ENU fixed frame.
for example :

    Transforms.headingPitchRollToFixedFrame = function(origin, headingPitchRoll, pitch, roll, ellipsoid, result,fixedFrameConverter) {
        var heading;
        if (typeof headingPitchRoll === 'object') {
            // Shift arguments using assignments to encourage JIT optimization.
            ellipsoid = pitch;
            result = roll;
            heading = headingPitchRoll.heading;
            pitch = headingPitchRoll.pitch;
            roll = headingPitchRoll.roll;
        } else {
            deprecationWarning('headingPitchRollToFixedFrame', 'headingPitchRollToFixedFrame with separate heading, pitch, and roll arguments was deprecated in 1.27.  It will be removed in 1.30.  Use a HeadingPitchRoll object.');
            heading = headingPitchRoll;
        }
        // checks for required parameters happen in the called functions
        fixedFrameConverter=defaultValue(fixedFrameConverter,Transforms.eastNorthUpToFixedFrame)
        var hprQuaternion = Quaternion.fromHeadingPitchRoll(heading, pitch, roll, scratchHPRQuaternion);
        var hprMatrix = Matrix4.fromTranslationQuaternionRotationScale(Cartesian3.ZERO, hprQuaternion, scratchScale, scratchHPRMatrix4);
        result = fixedFrameConverter(origin, ellipsoid, result);
        return Matrix4.multiply(result, hprMatrix, result);
    };

Thus, there is no big changes. What do you think?

@kaktus40 your suggestion has potential, but we would have to make a new function and put the fixedFrameConverter (probably renamed to fixedFrameTransform) before the result...or see if we could achieve this with more argument shifting for backwards compatibility until the deprecated parameters are removed.
1.27 is on Tuesday so it would be tight to try to make this change by then, but please consider opening a PR when you have time.


Support Windows

It's nice if support windows.

That's really cool! Thanks! Btw, names of these constants of method names will be changed in future probably (e.g. to UREQ_GET, etc.).

:metal:


Under the arm architecture，function perf_event_mmap error happened

The following error occurred during startup
Error:unable to mmap the perf-buffer for cpu '0','65536','22':Invalid
After investigation, it was found that the mapping error occurred in the function perf_event_mmap（userspace/libscap/engine/bpf/scap_bpf.c）.
Why does a file descriptor map two segments of memory?
Hope to hear from you soon

Falco version:
0.36

Kernel:
-4.19 rhel

### Tasks

ei @zhenpeng2018 thank you for reporting!   You can find more info about the buffer dimension to choose from here: https://github.com/falcosecurity/falco/blob/0.36.2/falco.yaml#L807
How much RAM do you have on your node/local machine?

Why does a file descriptor map two segments of memory?

The buffer memory is mapped twice to handle events that wrap around the ring buffer easily. With just one mapping it would be difficult to handle events that have one part at the end of the buffer and the remaining part at the beginning (we are talking about circular buffers)

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Running image without SYS_ADMIN capability

Hi there, great work on the API, I'm liking it so far!
I would like to deploy it on the PaaS Caprover, which is built on Docker Swarm. Your documentation however asks me to use the --cap-add=SYS_ADMIN flag, but that is is completely unsupported on swarm. Is there a way to modify your dockerfile not to rely on that being available? If you could just tell me which steps it is required for maybe I could patch it myself :)

Hi @JosXa! I used tips from puppeteer docs.

Seeing other weird errors when launching Chrome? Try running your container with docker run --cap-add=SYS_ADMIN when developing locally

I'm would glad to see pr or advice with fix it :)

I tried and failed, please keep the Help Wanted label ;)

I added --no-sandbox arg for chrome launching. We visit one verified site this should not affect security.


Transport configuration checking code

Ported from refactor-transports branch; includes unit-tests and
split out "native" (asyncio/twisted) checking code.

There are some thing in here I would merge, and others not.


Conflicting values for NOTIFICATION constants and docs out of sync

Hi,
Godot version:
Master branch at this point
https://github.com/godotengine/godot/tree/42032669238bdc807bc7b8f03fff87e6cb5847f1
Issue description:
There are conflicting values for NOTIFICATION constants between
https://github.com/godotengine/godot/blob/master/scene/gui/control.h#L265
https://github.com/godotengine/godot/blob/master/scene/2d/canvas_item.h#L240
and
https://github.com/godotengine/godot/blob/master/scene/3d/spatial.h#L120
This commit
https://github.com/godotengine/godot/commit/a18989602b00f2940befa0b780af14efc7603dd8#diff-97a2869611748b85a70fc262a8876ed4
introduced documentation errors into
https://github.com/godotengine/godot/blob/master/doc/classes/MainLoop.xml#L116
-- Perbone

There's not really any conflict, I'm wondering why you may be misunderstanding the notification system. Just use the constants in proper class hierarchy, they are not global numbers.

The notifications you see on a class are the ones this class can receive. There are not cases where a class receives a notification from a different one (unless it inherits from it).

Well, I thought about the possibility of not being global, yet I found it strange that only these classes had repeated values. Anyway, I think the documents should reflect the code, right?

Translations synced in 09e94aa08.