Fix vm endpoint delete command syntax

The last parameter is not the port but the endpoint-name (as in endpoint update). Add [options] as a parameter.

Hi @fjaguero, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Hi.... Is this in service or resource mode? Just want to make sure we are in the same mode.....

I think is service mode -> http://azure.microsoft.com/en-us/documentation/articles/virtual-machines-command-line-tools/

Great! Thanks for the fix. Seems hard to miss, yes? :-)

@deneha Merge please. :-)

I'm building an internal app that automates the creation and provisioning (with custom software) of VMs so I'm using many of the features of the CLI :D
On Fri, May 8, 2015 at 7:50 PM, Dene Hager<EMAIL_ADDRESS>wrote:

Merged #3518.
Reply to this email directly or view it on GitHub:
https://github.com/Azure/azure-content/pull/3518#event-300468405


Bare metal support

Bare metal support would be great so we can have an EKS@AWS + EKS@DC hybrid but slightly consistent K8s solution.

Bare metal support will be available in the EKS Anywhere v0.10 release in the next week or two. It'll show up here when it is ready https://github.com/aws/eks-anywhere/releases

Here's the latest release of EKS-A with bare metal support: https://github.com/aws/eks-anywhere/releases/tag/v0.10.0


client-side throttling resulting in long wait between custom resource creation and sync call

We have been noticing for a couple months that metacontroller is at times very slow to call our sync hook (up to an hour sometimes) after the custom resource was introduced into the cluster. We see a lot of client-side throttling at those times:
{"level":"info","ts":1641464159.2145114,"msg":"Waited for 1.188381736s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/scheduling.k8s.io/v1?timeout=32s\n"}
{"level":"info","ts":1641464179.2140453,"msg":"Waited for 1.194210847s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/autoscaling/v2beta1?timeout=32s\n"}
{"level":"info","ts":1641464199.2149317,"msg":"Waited for 1.192041362s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/coordination.k8s.io/v1beta1?timeout=32s\n"}
{"level":"info","ts":1641464219.214881,"msg":"Waited for 1.195390969s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/authorization.k8s.io/v1?timeout=32s\n"}
{"level":"info","ts":1641464239.2139485,"msg":"Waited for 1.190286683s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/events.k8s.io/v1?timeout=32s\n"}
{"level":"info","ts":1641464259.2137074,"msg":"Waited for 1.193234203s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/autoscaling/v2beta2?timeout=32s\n"}
{"level":"info","ts":1641464279.2138858,"msg":"Waited for 1.189809758s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/authorization.k8s.io/v1beta1?timeout=32s\n"}

We have recently seen this behavior in some of our own apps that talk to the kubernetes api during runtime and we determined that in our case this was caused by the kubectl cache directory not being writeable by the non-root user running the container.
Can someone clarify if this is expected behavior, a bug or a mistake in our deployment of metacontroller? In any case it is hurting our production environments and users because sync actions are very late.

Hi @rouke-broersma you can increase the values on:
--client-go-qps	Number of queries per second client-go is allowed to make (default 5, e.g. --client-go-qps=100)
--client-go-burst	Allowed burst queries for client-go (default 10, e.g. --client-go-burst=200)
Client-go is handling calls to the k8s api server as well as sync hooks to controllers. These configure the desired behavior for client side throttling.
Also, another argument you may benefit from is:
--workers	Number of sync workers to run (default 5, e.g. --workers=100)
This controls the number of objects that are processed in parallel from the queue.
See https://metacontroller.github.io/metacontroller/guide/configuration.html
Please let me know if this helps or you need any additional info 👍🏻

For reference I am currently configured at:
--client-go-qps=150
--client-go-burst=300
--workers=50
I rarely see any throttling messages with this config.

Hi @mjsmith1028 we deploy metacontroller with helm. How would you suggest we change these settings? There does not appear to be a configuration value for these in the helm chart.

@rouke-broersma The arguments are in the helm chart here: https://github.com/metacontroller/metacontroller/blob/master/deploy/helm/metacontroller/templates/statefulset.yaml#L38
You are right they are not exposed via values.yaml.
I have a branch with some tweaks to metacontroller and have the statefulset modified as:
        {{- with .Values.args }}
        args:
          {{- toYaml . | nindent 10 }}
        {{- end }}

Then in values.yaml
args:
  - --zap-log-level=4
  - --zap-devel=false
  - --zap-encoder=json
  - --zap-stacktrace-level=error
  - --discovery-interval=1m  # interval for checking if new resources added to cluster, similar to kubectl api-versions
  - --cache-flush-interval=10m  # informer caches k8s object, this flushes the cache and resyncs with k8s api server
  - --metrics-address=:8080  # if 0 will not enable, prometheus metrics address
  - --client-go-qps=150  # queries per second for go rest client
  - --client-go-burst=300  # max queries per second that go rest client can burst to
  - --workers=50  # number of workers that can process the workqueue executing sync hooks in parallel
  - --events-qps=0.0033  # rate at which k8s events can be sent to k8s api server 0.0033 = 5 mins
  - --events-burst=25  # max number of events that can be sent at given rate

I think https://github.com/metacontroller/metacontroller/issues/302 will resolve this. @grzesuav  @AmitKumarDas thoughts?

yes, thanks @mjsmith1028 for explaining. It would be ideally to have a bit refactor the chart and use some third party registry for storing them

Closing s duplicate of https://github.com/metacontroller/metacontroller/issues/302


'undefined is not an object' when using

Hi,
I'm new to the react-native space but I have stumbled on a problem that I cannot seem to workaround with this. When using the  tag to render a simple test (I'm literally using your <FormattedDate value={new Date()} skeleton="yMMMdhm" /> example from the guide), I run into an error that complains about "undefined is not an object (evaluating 'this.context.globalize.getDateFormatter')". This is the message thrown back from a catch block I have in my render method.
The tag is being used within a class method that creates entries for a ScrollView - everything works perfectly until I try to use the FormattedDate tag. I moved the tag to be within the renderScene method that I am using to fill out the Navigator but the problem is exactly the same. It's almost as if something has dropped out of scope but the problem is not obvious to me.
I'm using v4.24 of Node, react-native 0.18.1, react-native-cli 0.1.10, and a Genymotion Android emulator configured for v6 API Level 23.
Thanks

Are you using ES6 classes? If so, you need to be careful due to the lack of autobinding: https://facebook.github.io/react/docs/reusable-components.html#no-autobinding
If your method is called renderRow, just stick this.renderRow = this.renderRow.bind(this); in the constructor.
Let me know if that solves it.

Thanks for the response. I'm creating all my classes using the React.createClass() wrapper with a provided specification. I'm a little new to this space so perhaps my understanding is not 100% but I think this means I am not using the ES6 classes and as such do not need to perform the extra bindings, constructor for state, etc. At least, I have encountered warnings in the past, from react-native, of the form "React does this for you automatically in a high performance way" when I have had bind statements in my code as I attempted to learn some examples. I'm probably doing something simple in the wrong way. I'll strip back my app later and perhaps post a bare skeleton example of the problem if that would help to identify a possible bug or perhaps me as the cause of the problem. Thanks again for your advice so far.

You're right that you're not using ES6 classes if you're using React.createClass. I just reread your original message with that in mind and I'm pretty sure I know the problem. In order to access the message formatter functions directly through the context, you need to specify your component's contextTypes:
var MyComponent = React.createClass({
    contextTypes: {
        globalize: React.PropTypes.object.isRequired,
    },
    // ...
});

You can read more about React's context here: https://facebook.github.io/react/docs/context.html. And I'll update the docs to reflect this requirement. Hope that solves the issue for you.

Thanks again for the response and the information. Sadly, the issue is still prevailing for me. I stripped back a screen from my app, removed any entanglements with the Navigator component that I was using to go between screens and took it back to a minimal implementation. Both with and without the contextTypes code mentioned, I get the same error of 'undefined is not an object'. The error message on the emulator reports the innermost exception is thrown at line 53, in the render method, of FormattedDate.js, the line that reads:
var formatDate = this.context.globalize.getDateFormatter(this.props);
After debugging the code and stepping through the chain of calls, adding some alert statements to the FormattedDate.js file, I discovered that the context object is present but the globalize object is coming through as null. The code I am using for the cutdown example is shown below. Perhaps I am missing something, my apologies if so but I can't see why the code is failing. For a more complex example where I was using the Navigator component to render scenes, I thought perhaps you had pinpointed the problem of the context losing objects down the chain but in this code, it seems to be at the upper most level, if I understand it correctly.
'use strict';

var React = require('react-native');
var Dimensions = require('Dimensions');
var windowSize = Dimensions.get('window');
var RNGlobalize = require('react-native-globalize');

const config = require('./config')

var {
  AppRegistry,
  StyleSheet,
  Component,
  View,
  Text,
  TextInput,
  ListView,
  TouchableOpacity,
  Image,
  AsyncStorage
} = React;

var {
	FormattedDate
} = RNGlobalize;

var LoginScreen2 = React.createClass({
  contextTypes: {
    globalize: React.PropTypes.object.isRequired,
  },

  getInitialState() {
    return {
      username: '',
      password: '',
    }
  },

  render() {
    return (
      <View style={styles.container}>
        <Text><FormattedDate value={new Date()} skeleton="yMMMdhm"/></Text>
      </View>
    );
  }
});

var styles = StyleSheet.create({
  container: {
    flexDirection: 'column',
    flex: 1,
    backgroundColor: 'transparent'
  }
});

module.exports = LoginScreen2;

The globalize object needs to be added to the context using getChildContext on a parent component. If you read the first part of the documentation, it'll point you in the right direction. Either use the FormattedWrapper component at the root of your application so that all child components receive the required context object, or look at how FormattedWrapper works and provide the context using one of your own components.

Ah, ok! Thank you again so much Josh. I had a misunderstanding of how the context was passed on from component to component. I thought that by not having any custom components I did not have to pass on child context and I just couldn't see it. My apologies and gratitude for your assistance. I used the FormattedWrapper component around the root of my app and I can use the FormattedDate in a simple example now. Looks like I still have a lot of reading to do. Thank you once again for your time.


[Plushies] Can be placed in admin claims

Describe the bug
After my previous ticket, I tested and you can also place plushies.  That's all the blocks I have found so far but I'm sure there are more. Additionally, in the end they can be placed anywhere in the admin claim.
To Reproduce

Go to spawn or the end
Place a pushie on any of the listed blocks above
Watch it place

Expected behavior
They shouldn't be able to be placed
Username: _Peyton_7
Server name: FunPolice (all applicable):

forwarded to internal tracker


cjxl fails on "MotionPhoto" jpegs

Describe the bug
MotionPhotos are a cursed file where a video appended to a jpeg file. cjxl will currently fail to transcode the image with lossless bitstream reconstruction data with the error "JPEG bitstream reconstruction data could not be created. Possibly there is too much tail data."
To Reproduce
cjxl filein.jpeg fileout.jxl
File provided
Expected behavior
A) CJXL could discard the data and throw a message to the user. While discarding this data isn't technically lossless as far as the entire image file is concerned. This extra data doesn't pertain to the actual jpeg itself needed for proper reconstruction of the "image". CJXL should likely by default exit with an "Error" to bail out of any scripts unless an extra flag is passed.
B) CJXL could append the same video file to the end of the JXL file in the same manor as a motionphoto. This would retain "true" losslessness of the entire file and not just the jpeg data.
Screenshots
Not Aplicable
Environment

OS: Linux
cjxl/djxl version string: cjxl v0.10.2 e1489592 [AVX2,SSE4,SSE2]

Additional context
MotionPhotos are cursed, but there are a good number of them out in the wild and failing to transcode these with lossless reconstruction is a pain point for at least some users
Exiftool can identify multiple types of motionphotos "Motion Photo" is the most common, However some "Micro Videos" exist too. exiftool can find the appropriate offset for those using exiftool "$FILENAME" | grep "Micro Video Offset" | sed -E 's/^[^:]+: ([0-9]+)$/\1/'
Almost all motion photos seem to be mp4 based and have a ftypmp42 signature to them. Identification based on this seems reliable.
to extract the video from a MotionPhoto exiftool -b -EmbeddedVideoFile motionphoto.jpg > out.mp4
to extract the JPEG data from a motionphoto (replaces the current file, creates a new "original file") exiftool -b -trailer:all= motionphoto.jpeg
Example Image

More generally it would be good if we have a better way to treat "semantically meaningful" tail data when transcoding JPEG files. Besides these "MotionPhoto" cases, there is also UltraHDR (which has a gain map in the tail data) and MPF in general (Multi Picture Format) like some examples I've seen where the main image has fake bokeh applied to the background but the original image with a sharp background is still there in the tail data; also just a concatenation of JPEG files (which is something e.g. ffmpeg can ingest) could be seen as an example of this.
Currently tail data is stored in the jbrd and just treated like any other "unknown" data, like unknown APP markers or padding bits: it simply gets Brotli-compressed and in that way it becomes "hidden" in the JXL file, in the sense that the only way to retrieve such info is to apply the JPEG bitstream reconstruction procedure first.
It would be cleaner if we would define some kind of recursive structure in the file format where there is an optional box, name TBD but for the purpose of discussion we could call it tail, which contains the "tail data" of the main image. Then if the tail data is a movie, it could simply be copied into such a tail box, but if the tail data is another JPEG bitstream, then JXL's JPEG bitstream recompression could be applied recursively to that tail JPEG, and the tail box of the main image would then contain another JXL file. This could be nested arbitrarily deep, e.g. if you would concatenate three JPEGs then the third one would be in the tail box of the JXL file corresponding to the second one, which would be in the tail box of the first one.
Maybe instead of tail it could be called next or something. Such a structure would not allow seeking (it's a linked list, not a random access data structure), but for sequential access it would be fine. It would also make it straightforward to strip the tail data while still allowing reconstruction of the main image.
This method could also be used to extend the JXL file format to allow representing sequences of images that are coded completely independently (i.e. not as frames within a single JXL codestream), which can be useful since multi-frame single-codestream JXL has the limitation that all frames need to have the same dimensions and colorspace.
I am turning this issue into a discussion since this is not just a libjxl issue but changing anything here would require extending the spec (ISO/IEC 18181-2 in this case).


Fixing Skins issue

Fixes #1, Different skins seem to have default orientation offsets about their x-axis. Keeping this in mind, changes have been made to accommodate all skins with a new parameter in the plugin called default_rotation. The default value for which in 1.57 to accommodate gazebo's default actor. A new skin has been added whose default_rotation is 0 for reference.

I've tested the new changes in the repository on my system and everything seems to be working well. Additionally, I've clarified the definition for the 'default_rotation' parameter. I believe we can proceed with merging the pull request.


GIT_ASKPASS is set unconditionally

VSCode Version: 1.51.1 (OSS)
OS Version: Linux x64 5.9.11-arch2-1

I disable the git authentication with
    "github.gitAuthentication": false,
    "git.terminalAuthentication": false

But git push errors because vscode still set GIT_ASKPASS:
$ git push
/usr/lib/code/extensions/git/dist/askpass.sh: line 3: /usr/lib/electron7/electron: No such file or directory
/usr/lib/code/extensions/git/dist/askpass.sh: line 3: /usr/lib/electron7/electron: No such file or directory
remote: No anonymous write access.

Furthermore, this also occurs with --disable-extensions, which makes no sense to me because it tries to execute extensions/git/dist/askpass.sh, even though I have the git extension disabled?

Does this issue occur when all extensions are disabled?: Yes

It seems like it sets GIT_ASKPASS in the term environment unconditionally, regardless whether git authentication is enabled or not.
If I unset GIT_ASKPASS, this error goes away.

Verified but created a clean up task https://github.com/microsoft/vscode/issues/146082


OME-Zarr parsing for sparsely recorded fields

Can .zarr containers contain only the data present in the fields that actually contain data within a well? This would require the parsing of the Yokogawa experiment to be able to extract (z,y,x) stage positions, assigning corresponding values to corresponding fields.
As a first step, we need to parse the x,y,width and height from the metadata file from yokogawa experiments, and one example is found in the yokogawa_image_collection_task from the Liberali Workflows repository. All metadata is saved in the so-called mlf_frame file
On the simplest case, the sparse fields would still fall onto a grid pattern, so parsing should then be able to extrapolate the size of the entire grid, and be able to place the recorded fields in the correct grid positions.
Currently Drogon creates an overview of 0's where fields were not recorded, filling up the gaps. The saved overviews also comprise of a mask overview which is used for retaining only the imaged fields for processing and avoid overfilling memory.
One interesting question here is whether OME-Zarr can actually deal with sparse field grids, meaning that the empty fields would remain empty, or would be assigned to same-sized stacks filled with 0's.

To facilitate tackling this issue:

We need to fix https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/13 to use the metadata instead of row & col parameters (let's not throw away the row & col logic: It's useful to have, but should not be the default and should not be used when there is metadata available).
Let's add to this to make sure it can also handle a search-first (grid-based): See Search-first 1) here https://github.com/fractal-analytics-platform/fractal/issues/213
Let's test it on a larger search first dataset: See Search-first 2)

Let's discuss 1) in https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/13
For 2)
Careful, this dataset has different channels. Until we have fixed https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/5, let's make sure we specify them manually
When the processing works correctly, it should look like this (see also above from @gusqgm ):

Still like a grid, but not all parts have content. We will need to ensure that we build an empty array of the correct size initially to be able to fill in all the positions. And we can't make assumptions based on the count of FOVs or just look at first and last (otherwise, we're not tackling part 3)). => We will need to look at the metadata table, find the top left and bottom right corner based on all the x & y position and the image sizes.

We will need to look at the metadata table, find the top left and bottom right corner based on all the x & y position and the image sizes.

At the moment, this happens based on the FOV-ROI indices, rather than on the x/y physical positions. In the parsing task there is a block like
        adata = read_zarr(f"{zarrurl}/tables/FOV_ROI_table")
        fov_rois = convert_ROI_table_to_indices(adata, full_res_pxl_sizes_zyx=pxl_size)

        max_x = max(roi[5] for roi in fov_rois)
        max_y = max(roi[3] for roi in fov_rois)
        max_z = max(roi[1] for roi in fov_rois)
        [...]
        canvas = da.zeros((max_z, max_y, max_x), dtype=sample.dtype, chunks=(1, chunk_size_y, chunk_size_x))

where 5,3,1 are the indices corresponding to end_z,end_y,end_x, as in roi=[start_z, end_z, start_y, end_y, start_x, end_x]. Note that the max_z part is a bit redundant, as all FOV should have the same number of Z planes.
@jluethi, do you notice anything unexpected in the way we are doing it?

I'd be a bit cautious with using an index-based selection of the columns here. It assumes that the ordering of the AnnData ROI tables will always be the same. While our system should be consistent here, we may want to support other OME-Zarr files that contain tables with the same information, but the order of columns shouldn't be a part of that spec
=> Can we switch to selecting columns based on their names in var? Also, can we implement that such that we have default column names, but those can be overwritten as an optional input? (because the spec for how those columns are named may change in the future).
It's good to have max_z, as it may vary between wells, see e.g. the test case /data/active/fractal/3D/PelkmansLab/CardiacMultiplexing/Cycle1_5x5_10wells (https://github.com/fractal-analytics-platform/fractal/issues/213). Mostly, I expect FOVs within a well to be consistent, but I'm not even sure that will always be the case and is not required by the specification.
Given that it handles the 2x2 case, it seems to be able to handle negative coordinates. The 2x2 case has X positions like X="-1448.3", so that's good :)
The general canvas definition and max position finding looks good to me

As of https://github.com/fractal-analytics-platform/fractal-tasks-core/commit/5c1410edb6ca628084991e1250adf43b0f805660, the new function get_ROIs_bounding_box by @mfranzon and me is meant to take care of this last comment.
Missing features:

The column names can be overwritten, but at the moment this is not exposed to the user.
By now we are not shifting ROI positions (to make them start from zero), because our AnnData tables are already shifted (that takes place in prepare_FOV_ROI_table). We probably need to be more general, if we want to read zarr files produced outside fractal, but I'd say we can postpone the implementation of this feature.

The /data/active/fractal/Liberali/1_well_15_fields_20_planes_SF_w_errors/D10_R1/220304_172545_220304_175557 search first dataset is processed successfully with the current Fractal version:

The other 2 search first test cases appear to have some issues in the metadata parsing (https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/109 & https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/110), but that's most likely unrelated to the search first part of it.
I'll need to have a brief check whether the segmentation and measurements have worked correctly on this dataset. After that, we can close this issue


qrenderdoc crashes when capturing OpenGL application

I mentioned this as a comment on #778. The crash occurs rarely when the "Trigger Capture" or "Queue Capture" buttons are pressed, but PrtScrn and F12 usually segfault. The "Collect Callstacks" option does not change anything. Notably, the crash does not happen on all applications. For example gl-320-texture-cube seems to always work but gl-320-program often causes the crash when full-screen. I feel like larger application cause it more often, but I can't really back that up.
I am running Arch Linux and using the Aur package (RenderDoc v0.91) and my laptop's onboard graphics:
00:01.0 VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Richland [Radeon HD 8450G] (prog-if 00 [VGA controller])
	Subsystem: Lenovo Richland [Radeon HD 8450G]
	Flags: bus master, fast devsel, latency 0, IRQ 35
	Memory at e0000000 (32-bit, prefetchable) [size=256M]
	I/O ports at 3000 [size=256]
	Memory at f0300000 (32-bit, non-prefetchable) [size=256K]
	[virtual] Expansion ROM at 000c0000 [disabled] [size=128K]
	Capabilities: <access denied>
	Kernel driver in use: radeon
	Kernel modules: radeon

The capture still works (saves the file), the application remains running, and I can capture with PrtScn/F12 again. All that happens is qrenderdoc crashes. I can restart qrenderdoc and then open the saved capture(s) with no issue.
Attached is an example of gl-320-program exploding. It did take a few tries, the last capture is the one that blew up.
qrenderdoc_crash.tar.gz

A large PBR rendering demo I wrote never fails to crash. Maybe because it has lots of huge framebuffers and things that gl-320-program doesn't?

I've tried to repro this on my system but I'm not having much luck.
Can you try a couple of things:

Are you able to get the crash without fullscreening the program? For me gl-320-program-uniform is a fixed size small window, and I noticed some of your captures are coming back with large framebuffers. That might affect the issue.
Can you share your rendering demo that always crashes? That might be a more reliable repro.
Lastly, can you build renderdoc from source on the v0.x branch and get a more useful callstack to try and see what's going wrong?

My current working theory is there's some kind of timing issue or race condition and I'm not getting the crash just by luck.

Without more information I can't investigate this bug any further, so I'm closing this issue. If you have any more to add then please comment here. As an addendum to the above, if you do build from source to investigate you should build on the new v1.x branch.


channels with dots in their name cause a crash in indexing

Solution: skip non-files
fixes #540

Thanks!


Object initializer instead of properties initialization

I mean use the following code:
var itemDto = new DataTransferObject.UserDto
{
    UserId = item.UserId,
    Username = item.Login,
    Created = item.Created,
    Email = item.Email
};

instead of:
var itemDto = new DataTransferObject.UserDto ();
    itemDto.UserId = item.UserId;
    itemDto.Username = item.Login;
    itemDto.Created = item.Created;
    itemDto.Email = item.Email;

It can be additional setting in MapHelper.tt

Initially it was, but when we needed map to ObservableRangeCollenction it had rewritten.
See MappingExtension.g.cs:L67
But this generated code, what difference would it look like?
Object initializers are just syntactic sugar, in your compiled assembly IL they translate into separate statements, check it on ILSpy.

Is it possible to map ObservableRangeCollenction by usual way and other objects by object initializer? But you are right: it is not important feature.

ObservableRangeCollenction is readonly in each ViewModels in our projects.


As Portal User, I want to see a message displayed in the portal when I entered a location so that I would know if it is supported or not supported.

From @czdiana2321 on May 21, 2018 19:39
USER->UI
Acceptance Criteria:

The system is able to display content that is relevant globally.
The system is able to display a message if the location entered is supported or not.
The Portal User is able to enter location to display content that is relevant globally.
The Portal User is able to view a message if the location entered is supported or not supported.

Copied from original issue: Microsoft/Access2Justice#82

Issue resolved.


reading output

Hi, thanks for sharing the code.
When I run the code, it generates an edgepart file.
Can you tell me how to read the output of the file?
Thanks.

Could you share one small sample of the file?
It has been a while and I cannot quite remember the detail.

Could you share one small sample of the file?
It has been a while and I cannot quite remember the detail.

I am partitioning the Orkut file, the program produces a partitioning file. Here is the output of the partitioning file.

I am partitioning the Orkut file, the program produces a partitioning file. Here is the output of the partitioning file.

hi @snash4 , the parition results are written to a <basename>.edgepart.<p> binary file.
the output code is in edgepart.hpp. i would suggest you to rewrite edgepart_writer to print the results in plain text.

close due to inactive.


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2023-12-17T02:25:07+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d6ced7462c9215ab8684461718dd2eed48285340
link
true
/test integ-security-multicluster

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. I understand the commands that are listed here.

/test integ-security-multicluster


Jbuilder port

Putting this up for @avsm

Pushed a few more fixes to Rudi's branch and am still working on this.  Need to test libffi detection next...

Closing in favour of #588 where i've rebased and documented the progress.


Support for ElasticSearch Appliance

For applications that require full document indexing and ranking at search time, ElasticSearch would be a helpful appliance to be bundled by default.

Elasrticsearch is on our list of appliances to add next.

Are there any plans already setup for the implementation of this?
I may have some time to work on this.

@lalloni We have not spent much time on it yet. There are some architectural questions, like for example should we try to make a discoverd plugin for discovery/leader election, similar to the now outdated Zookeeper plugin?

@titanous What would be the need satisfied by such discoverd plugin? Can you provide a link to zookeeper discoverd plugin so I can take a look anyway? (I can't find it using github/google).

https://github.com/grmblfrz/elasticsearch-zookeeper
The idea is that it would mitigate some of the stability issues in ES: https://www.elastic.co/guide/en/elasticsearch/resiliency/current/index.html

+1 and happy to help

+1 also happy to help

What is the current status of this issue?

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


Record expert multi env lstm

This adds multi envs and LSTM support to record_expert. The Changes a manly to only use the first env if multiple envs a present. for LSTM support the done variable from the first env becomes mask to reset the LSTMs.
The original implementation has no test script in tests, so a am also ignoring it.
Closes #260

Hello,
Thanks for your pr. Before i do the review: please update the changelog and add a test (located in test_gail.py)

Note: if the build errored, because you are using multiple env, you should call env.close() to exit the subprocesses, otherwise, it will not release memory and travis will hang.

I hope you mend at the end of the recording.you mean

I mean, it only makes sense at the end of the recording.

@XMaster96 we are almost there =)
Please do not forget to update the changelog and add your name at the bottom ;)

ok done :)

Thanks for your PR, merging ;)


Jupyter Notebook output cell display object contents

When printing an object in a Jupyter Notebook code cell there should be an option to switch to more detailed output of objects.
Example:
from sklearn.feature_extraction.text import CountVectorizer
count_vector = CountVectorizer()
print(count_vector)

Outputting the parameter values of the object will be helpful to determine what is set at that point or set by default.
VSCode implementation outputs the class name CountVectorizer() without the parameters, whereas online Jupyter instances often include the parameters in the output.

whereas online Jupyter instances often include the parameters in the output.

Please could you provide a screenshot of the expected output (i.e. the output you see in Jupyter).

Hi Don,
Thanks for looking into this. Here's an example of a printed class instance
where I immediately see the parameters...
I hope it helps. Otherwise feel free to get back to me.
[image: image.png]
On Mon, 24 Jan 2022 at 23:50, Don Jayamanne @.***>
wrote:

whereas online Jupyter instances often include the parameters in the
output.
Please could you provide a screenshot of the expected output (i.e. the
output you see in Jupyter).
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-jupyter/issues/8790#issuecomment-1020628853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC2LLHTSJBL37YBNQPE4UXTUXXJVBANCNFSM5MWUVX5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

@brew-guy Please could you re-upload the image, looks like it didn't get uploaded

Hi,
Try this...
image.png
https://drive.google.com/file/d/1VfsTVyMIlRTFDByYoiSm7UnU5o2dU7J8/view?usp=drivesdk
On Wed, 26 Jan 2022, 20:04 Don Jayamanne, @.***> wrote:

@brew-guy https://github.com/brew-guy Please could you re-upload the
image, looks like it didn't get uploaded
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-jupyter/issues/8790#issuecomment-1022502690,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC2LLHTQZUEEMTZ64GDW7HTUYBAUDANCNFSM5MWUVX5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @DonJayamanne, no worries. I hope it works this time.

thanks.
please could you provide a screenshot of what it looks like in vscode,

Oh ofcourse! Here it is. VSCode does not output anything but the name of the instance:

Thanks, unfortunately this isn't something we control.
The output we get for print(count_vector) is returned by the Python kernel, and should be the same, it its not then its likely you're using a different kernel or there's something else different in your setup.
Here's a screenshot of the exact same code for me in Jupyter (which is inline with VS Code, i.e. its the same)

Closing this as the output is not controlled by the VS Code extension, but controlled by the Python kernel itself.


app deployed successfully but not shown in repo

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d3p7gaoq9nj5qh
AWS Region
us-east-1
Amplify Hosting feature
Build settings, Deployments
Question
i'm building this app from this repo :  https://github.com/malawadd/chainlink-truflation/tree/main
now my question is :  why doesnt it show the building status on my github you know the ✅ or ❌ ?

Hi @malawadd 👋 , thanks for reaching out to us!
I observed that your Amplify app d3p7gaoq9nj5qh has deployed successfully and the website is also resolving just fine.
Can you please elaborate on the behavior that you are observing with the GitHub repository? It would be helpful if you could also provide screenshots. Thanks!


Redbird client as a stored class property

Hello,
More a question than an issue: i'm playing with your redis client and I'm wrapping it in a class to abstract db specific stuff in my app. I have a problem initializing this class though since init of Redis throws.
Here's what I'm trying to do:
class RedisManager {
  let client: Redbird
  init(address: String, port: UInt16, password: String? = nil) {
    do {
      let config = RedbirdConfig(address: address, port: port, password: password)
      client = try Redbird(config: config)
    } catch {
      Log.error("Could not connect to Redis instance")
    }
  }
...
# Api like setArray, getString ...
}

This fails with error:

Return from initializer without initializing all stored properties

I want to keep an instance of Redis as a stored property because it seems wasteful to recreate it inside a do catch every time I want to access db.
Any idea how to instantiate Redis in init of my class and assign it to self.client?

Yeah I definitely recommend keeping just one connection open instead of creating it every time.
The problem is that when Redbird throws an error, you catch it, log, but don't assing into the client variable.
I'd recommend making the RedisManager throwing initializer, so change it to
class RedisManager {
  let client: Redbird
  init(address: String, port: UInt16, password: String? = nil) throws {
      let config = RedbirdConfig(address: address, port: port, password: password)
      client = try Redbird(config: config)
  }

This way, if the creation of RedisManager throws, you know you can't access it. On the other hand, if it runs the initializer successfully, you know you're connected and everything should work fine.
With the changes above the compiler error should go away.
Please let me know if this helped 😊

ok, thanks for the reply. I'm still getting used to the new way of handling errors in Swift :)

Happy to help :blush:


There are no constructor takes 2 arguments

EdgeOptions edgeOptions = new EdgeOptions(false, "webview2");

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

ID: 758ec47a-0630-ed93-b312-66cc2c6f48d5
Version Independent ID: 6b666839-119d-433e-8756-0089378b5269
Content: Automating and Testing WebView2 with Microsoft Edge Driver - Microsoft Edge Development
Content Source: microsoft-edge/webview2/how-to/webdriver.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Hi @superbull - in the sample code we suggest to instantiate EdgeOptions like this:

I think we might have made this doc update after you created the PR, but let me know if this is still a current problem with the doc.


Security page proposal

@sleeping-h here is a first draft for a security page
@jef-n @pblottiere @lbartoletti @elpaso  I'd be pleased to here from you about this proposal. The idea is to clarify things for users, and reduce traffic for false positives.
I think we miss a section to adress all those requests to fill forms for various regulations. I will also open a discussion about activating the dedicated services of github so that we rely less on our seecurity mail, but that's for the future
cc @mbernasocchi @timlinux

Summary by CodeRabbit

New Features

Introduced a "Security Vulnerability Reporting" section to guide users on how to report security issues.
Launched a comprehensive "Security" page detailing how security concerns are managed, including handling vulnerabilities, securing deployments, and the security workflow.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: haubourg:x: jef-nYou have signed the CLA already but the status is still pending? Let us recheck it.


DNS problem when hit F5 in VS2017

My AKS is all right and functional.  Only AZDS unable to run from VS2017.  It looks like a DNS problem.  So weird.
Using dev space 'default' with target 'MyAKS'
Release "default-webapplication3-a5c18fa2" does not exist. Installing it now.
Error: release default-webapplication3-a5c18fa2 failed: Ingress.extensions "webapplication3" is invalid: spec.rules[0].host: Invalid value: "webapplication3.cf869ce5a6ed4c76b57f.WestUS2.aksapp.io": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
Exception when starting service on Kubernetes, Helm install failed with exit code '1': Release "default-webapplication3-a5c18fa2" does not exist. Installing it now.
Error: release default-webapplication3-a5c18fa2 failed: Ingress.extensions "webapplication3" is invalid: spec.rules[0].host: Invalid value: "webapplication3.cf869ce5a6ed4c76b57f.WestUS2.aksapp.io": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

Start service failed.

Release "default-webapplication3-a5c18fa2" does not exist. Installing it now.
Error: release default-webapplication3-a5c18fa2 failed: Ingress.extensions "webapplication3" is invalid: spec.rules[0].host: Invalid value: "webapplication3.cf869ce5a6ed4c76b57f.WestUS2.aksapp.io": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
Exception when starting service on Kubernetes, Helm install failed with exit code '1': Release "default-webapplication3-a5c18fa2" does not exist. Installing it now.
Error: release default-webapplication3-a5c18fa2 failed: Ingress.extensions "webapplication3" is invalid: spec.rules[0].host: Invalid value: "webapplication3.cf869ce5a6ed4c76b57f.WestUS2.aksapp.io": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

Service cannot be started.

Any clue?

I don't think it likes the upper-case letters in the domain name:
webapplication3.cf869ce5a6ed4c76b57f.WestUS2.aksapp.io
Was the Helm chart generated by Dev Spaces (using Visual Studio, the prep command palette option in VS Code, or the azds prep CLI command) or was it one that you created manually?

The Helm chart generated by Dev Spaces using Visual Studio 2017.   I do nothing manually.
I was creating AKS by the following commands:
az group create --name MyAKS --location WestUS2
az aks create -g MyAKS -n myaks --location WestUS2 --kubernetes-version 1.11.2 --enable-addons http_application_routing

That is possibly cause this bug.  I think AZ or AZDS should take care of this.

Fully agree that AZDS should handle this and create an appropriate chart. Thanks for reporting -- we'll look into it.

Hi @doggy8088 can you share the host application routing domain setting for your AKS cluster. You can find it in the azure portal. This setting will be inside overview blade of the aks cluster.
In Devspace we use this setting as is, I want to make sure if the aks provided us with the correct setting.

Closing as it has been over a month without customer response.  We do handle upper cases letter correctly in the latest versions of the tools.


fea: add aws-lambda-alb support

🔗 Linked issue
#2072
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
WHAT?
Add support for ALB lambda function targets.
WHY?
ALBEvent is similar to APIGatewayProxyEvent. However, it has key differences such as:

Query string parameters are not URL-decoded automatically.
The events either have headers/queryStringParameters or multiValueHeaders/multiValueQueryStringParameters, but
not both.
Headers without values are removed from the event's input.

Without this change, using nitro with Lambda + ALB yields unexpected behaviors like double URL encoding.
RESOLVES?
#2072
📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

Thanks for PR.
Considering major differences, i think it would be nicer if we develop a standalone subpreset + entry such as aws-lambda-alb and avoid changes to current aws-lambda (that as linked issue states is purpose built for gateway proxy)

Thanks for the reply @pi0!

Considering several differences, i think it would be nicer if we develop a standalone subpreset + entry such as aws-lambda-alb

Just to confirm, do you mean something like the following?
Update:
https://github.com/nitrojs/nitro/blob/f982f99fbab57d3bfc51d155f5fff73c7ab2a4ec/src/presets/aws-lambda/types.ts#L1-L3
to:
export type AwsLambdaOptions =
  | {
    // backwards compatibility, the presence of `type` overrides this
    streaming?: boolean;
  }
  | {
    /**
     * The Lambda function type:
     *
     * - `alb`: An ALB Lambda target group.
     * - `api-gateway`: An API Gateway REST API Lambda proxy integration, HTTP API Lambda integration, or Lambda function URL.
     * - `function-url`: A synonym of `api-gateway`.
     * - `function-url-stream`: A Lambda function URL with response streaming.
     *
     * @default "api-gateway"
     */
    type?: "api-gateway" | "alb" | "function-url" | "function-url-stream";
  };

And update:
https://github.com/nitrojs/nitro/blob/f982f99fbab57d3bfc51d155f5fff73c7ab2a4ec/src/presets/aws-lambda/preset.ts#L10-L16
to:
hooks: {
  "rollup:before": (nitro, rollupConfig) => {
    if ("type" in nitro.options.awsLambda) {
      if (nitro.options.awsLambda.type === "alb") {
        (rollupConfig.input as string) += "-alb";
      } else if (nitro.options.awsLambda.type === "function-url-stream") {
        (rollupConfig.input as string) += "-streaming";
      }
    } else if (nitro.options.awsLambda.streaming) {
      (rollupConfig.input as string) += "-streaming";
    }
  },
},

I mean making new variant / export in presets (similar to cloudflare/)


LGTM.com - false positive

Description of the false positive
LGTM tells me that I don't have any CSRF protection and points to an express-route that doesn't access that session-cookie (req.session). I'll move the middleware a bit down so that route can be contacted and skip the middleware. I think that will stop LGTM from detecting but it still is a false-positive.
URL to the alert on the project page on LGTM.com
https://lgtm.com/projects/g/Mc-Auth-com/Mc-Auth-Web/snapshot/8bd0dbe24320c793f24992d001986a6ddc11b7c5/files/server.js?sort=name&dir=ASC&mode=heatmap#x548881a38f2438d4:1

Thank you for your report! Unfortunately this query is a little prone to false positives, we'll look into sprucing it up a bit to avoid falsely flagging this code.

Closing as the reported false positive has been fixed.
@Sprax2013 it seems to me the remaining alerts are true positives, but please let us know if you think that's not the case.


Lowering: Move refindex from Variable to Statement

Stack:

#1659 ⬅
#1656

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

crates/cairo-lang-lowering/src/objects.rs line 153 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
Are the changes enougth? didn'y you talk about keeping track of all the reference in every statement?

please document that this is immediatly after the statement and add an example.


Deterministic build process

We should have a documented way to build runtimes based on the SDK deterministically, for both SGX and default targets (with priority on SGX targets). We should run daily tests on CI to make sure that at least the example runtimes in this repository can be built deterministically using the documented process.

I tried this morning to use Nix via crate2nix and got pretty far until the bootstrap failed because of macos issues. So, if you need a VM to build with Nix, why not just build in a docker container? cargo vendor might help, but I think a lockfile will suffice.


Jetson torch1.9.0 torchvison0.10.0 python  setup.py install           error

🐛 Describe the bug
virtual AVCodec* findCodec(AVCodecParameters* params)
Versions

toch1.8,1.7 has the same error

What version of ffmpeg / av / ... are you compiling against?

What version of ffmpeg / av / ... are you compiling against?

Thanks. I solved this problem because of ninja and ffmpeg


Fixed #29295 -- Fixed management command crash when using subparsers.

I applied  @timgraham  diff, and added tests for it.

This appears to make it difficult to use CommandParser in Django 2.1 and older versions.
django-configurations uses it here: https://github.com/jazzband/django-configurations/blob/217cf7ed070dfb431671c905fb987c86f4980d13/configurations/importer.py#L85-L87.
What about accepting cmd=None still?  (likely not possible?!)
But then any cmd could be popped from kwargs, so that django-configurations could use cmd=None.
OTOH it could also handle the TypeError, of course - so just a heads up.


Implement notify several actions call

https://github.com/wonnova-development/woost-gamification-php-sdk/pull/13


Opening files with different encoding than UTF-8 is really slow

Hello,
I have a file that has characters different than UTF-8. It takes really long to change the encoding of the file to the correct one. When I chage the default encoding in Atom, opening such a file takes very long too.
Steps to reproduce

Download the following file test.txt
Set the default encoding of Atom to Western(Windows 1252)
Open the file you just downloaded in Atom

Opening will take pretty long(on my machine it takes around 10 seconds)
When changing the default encoding back to UTF-8 in Atom and opening the file, it loads very fast.
Version of Atom I tried that were affected

1.3.0 beta6
1.2.4

Operating Systems tested on

Windows 7
Windows 10

@as-cii Does atom/text-buffer#119 close this?

@nathansobo: yes, I just bumped text-buffer to the latest version on master (b3b8b8d) and this should be fast now. :racehorse:
We will release it in Atom v1.4.0 and v1.4.0-beta0.. Thanks for the report, @krial057!


Add Ability to Zoom on a Range

A lot of the other charting frameworks contain the ability to zoom, but usually on a range. An example being highcharts:
http://www.highcharts.com/demo/line-time-series
This is maybe more of a questions that an issue, but was there any thought to add the ability to select a range similar to the link shown above?
I did most of the work to currently add in this functionality, but it doesn't seem to fit in nicely with how this plugin in currently setup. The min and max are calculated are based upon the use of a wheel mouse as opposed to selecting a range. I didn't know if it would be better to create a separate plugin or try to move things around in this plugin and add in an option to add in this functionality.

@pml984 I'm happy to look at PR if you can create one that adds this functionality.

Done in #28


Fixed Footer Implementation with latest slickgrid

Hello Folks,
Does any one implemented fixed footer for aggregating specific columns?
Like fixed header in 2.0 alpha, when we reorder the columns the footer also needs to be reordered. When we filter the rows, appropriate data should be updated in footer section.
Please suggest me.

Any update on this ?


DOC: optimize.root: outdated naming of fprime in method=’hybr’

Issue with current documentation:
The documentation for optimize.root(method=’hybr’) https://scipy.github.io/devdocs/reference/optimize.root-hybr.html contains references to a parameter fprime which does not exist for that function. Specifically, these references appear in the description of the parameters band and eps.  The parameter fprime seems to be renamed to jac in optimize.root in scipy 0.11.
Idea or request for content:
Replace the two occurrences of fprime with jac in the documentation for optimize.root(method=’hybr’).
Additional context (e.g. screenshots, GIFs)
No response

thanks @lukjp, your analysis sounds correct to me. Would you like to submit a fix? Our contributor docs are at https://scipy.github.io/devdocs/dev/index.html.

@oceanthunder go ahead! (we don't typically assign issues in SciPy). Our contributor docs are at http://scipy.github.io/devdocs/dev/index.html :)


Example: Replace wrong PostgreSQL mention in comment

In the mySQL example the comments mention PostgreSQL.

@googlebot I signed it!


manifestServiceListener.Get to pass down options parameter

options parameter gets lost in the manifestServiceListener.Get. That causes issues for proxyManifestStore,Get which needs options to pass down to client.
Signed-off-by: Serge Dubrouski<EMAIL_ADDRESS>

Current coverage is 51.18% (diff: 100%)

Merging #1943 into master will decrease coverage by 9.78%

@@             master      #1943   diff @@
==========================================
  Files           124        122      -2
  Lines         11082      10683    -399
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6757       5468   -1289
- Misses         3441       4467   +1026
+ Partials        884        748    -136

Powered by Codecov. Last update 49c1a62...a1a2757

LGTM

LGTM


Appstudio update hacbs-test-project-dhzk

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-dhzk-on-pull-request-
creating pipelinerun hacbs-test-project-dhzk-on-pull-request- in namespace rhtap-demo-ezua-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


C# Code block is not loaded

The C# code no is loading after text “Port 0
When the port number 0 is specified, Kestrel dynamically binds to an available port. The following example shows how to determine which port Kestrel actually bound at runtime:”

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

ID: f22dcddb-0c25-be1a-9a02-9e71dce86bdb
Version Independent ID: 3e0e12c2-e544-5655-e001-d2ec180b11a8
Content: Kestrel web server implementation in ASP.NET Core
Content Source: aspnetcore/fundamentals/servers/kestrel.md
Product: asp.net-core
GitHub Login: @Rick-Anderson
Microsoft Alias: tdykstra

@DevOpsChallengeBr  thanks for bringing this to our attention.

Thanks @DevOpsChallengeBr ... I'll get this fixed in a few hours. Thanks again for calling it out.


Conflict with CompressionWebpackPlugin

Config:
import OptimizeJsPlugin from 'optimize-js-plugin';
import CompressionWebpackPlugin from 'compression-webpack-plugin';

plugins: [
  new webpack.optimize.UglifyJsPlugin(options),
  new OptimizeJsPlugin({
    sourceMap: false
  }),
  new CompressionWebpackPlugin(options)
]

Current behavior:
Chunks are getting optimized, but compressed files contain original uglifyjs output without optimizations.
Expected/desired behavior:
Compressed files should contain optimized chunks instead of originals.
UglifyJsPlugin -> OptimizeJsPlugin -> CompressionWebpackPlugin

@ntkoso Thanks for finding it, seems to be a bug. Will have a look.

@ntkoso fixed it in 0.0.4


Comparison view

Leaftel side-by-side
Now uses leaflet-side-by-side -plugin to view satellite images next to each other with a slider control. The functionality of this component will be accessible through the bottom/sidebar menu, which is currently in progress (see issue #52).

Now you should be able to control the opacity of either left or right image.


mvn test failing

Ripme version: latest code
Java version: 1.8.0_66
Operating system: os x (10.10.5)

'mvn test' is failing. I have attached the output here.
test_log.txt

In the home page, it was written to use 'mvn test'. But when I run 'mvn test', most of the tests are failing. I think we should run only few tests in mvn test, if possible. That way it would be easy for beginners.

Which commit as shown at https://github.com/RipMeApp/ripme ? ATM when I checked it's https://github.com/RipMeApp/ripme/commit/4e2d5f3c6e551b95a7106e8eeb1c52eedbac610c while the whole string reads Latest commit 4e2d5f3 4 days ago.
Besides, you're on an ancient Java.

@rautamiekka Hi, Did 'mvn test' run fine for you? If so, can you share your log file.
For me, few test cases are failing. Some are because of SSL handshake failure(need to configure my eclipse). Remaining I have to check.

^ I haven't tried. Last time I tried to setup Eclipse or IDEA was for CraftBukkit, and it never succeed.
It's probable your ancient Java is the problem cuz there has been development in the certs and otherwise web hosting (algos getting dropped, etc.), and one of the latest 8 Updates banned the use of MD5+RSA for signing the JARs, among other things changing.
Now when I checked: Update 141 is the newest, when 131 was the newest not many days ago. http://www.oracle.com/technetwork/java/javase/downloads/jdk8-downloads-2133151.html
Release notes overview page: http://www.oracle.com/technetwork/java/javase/8u-relnotes-2225394.html

mvn test is broken, yes. FWIW your log looks incomplete. I remember something like 10 tests failing and some number of tests with an error of some sort. The nature of this project is that websites change and through no fault of the developers, rippers (and therefore the tests which test them) become broken. There are not enough volunteer hours on this project to keep all rippers and tests working at all times, so as a policy, we do our best to fix the things that are the biggest problems for our users (starting with the biggest websites), and anyone who is not a primary maintainer fixes whatever they feel like fixing (sites they personally care about).
One option is to disable broken tests until they are fixed. But that's also a progressive time investment that would make more sense if we had more dev-hours per week.
To your other point: mvn test should have more tests, not less. For beginners, running mvn test is the absolute simplest burden I can imagine. It's quite easy compared to many projects. I'm not sure I understand how fewer tests would make things easier for beginners.
Also, more (working) tests allows beginners to know whether something they have changed has broken the tests. I understand the tests need to be working in the first place for this to be helpful.
If you're willing to indulge the broken state of the tests, you should run the tests on master, then make your changes, and run the tests again. If the number of failures has not increased ("no regressions"), then your change is probably fine.

@metaprime In the README.md, it was written to run 'mvn test'. May be we should mention that not all tests are passing. Otherwise developers may think there is something wrong with their setup

@raviteja1992y Good point. Also, I should just go through and disable failing tests. No sense in running known broken tests. Doesn't let us see when things get worse / regress / a PR is bad. Unfortunately I haven't set up a travis instance for RipMeApp/ripme yet either. There's lots to do :P

@raviteja1992y Added note to the README

Update on this, at this point only 7 of test cases are still failing

I'm going to disable the remaining failing tests and open bugs for them so that we can get the CI green.

When #188 is merged, this issue will be closed. The test failures will be addressed in the follow-up bugs I opened.


Add gem webpacker

Should we add gem webpacker to the Gemfile.txt?
Because It has some issues when trying to create a new project with React environment using webpack.
I use this command --webpack=react.

@Lahphim yes we could. But maybe we can add it conditionally: if the option webpack is passed in the rails new command then it could be added, otherwise not. How about you having a dab at it?


Native Input Element Disabled

What happened?
Hi there 👋,
As of today the propagation of the native input event is currently commented as a result of a bug.
This way we are not able to react to any changes when the user selects a value from the suggestion-list. Is there already a solution how this will be handled in the future?
Affected version
Latest
Reproduction URL
https://github.com/digitalfortress-tech/typeahead-standalone/blob/70668092359607fee5cbebdde1e59bf94a2b8bc6/src/typeahead-standalone.ts#LL345C1-L346C1
How to reproduce?

Attach an input event handler to the input field
Type a prefix of a value and select a value from the suggestion list

Browsers
Chrome
Relevant log output
No response
Additional context
No response

Hi @niketpathak ,
Thanks for the fast reply - this would work out as a workaround. I still think its not wise to break the native capabilities when setting the value without dispatching the corresponding native input event. Is there a specific reason why you opted out of this?
With this solution the consumer of the library is now responsible to ensure parity to the native behavior. At the moment this breaks many of the form libraries.
Have a great day! 👍

Hello @HansMusterWhatElse , after pondering about your last comment. I agree that to be at parity with respect with 3rd party libraries, an input event must be emitted: Thank you for your feedback. This has been implemented in v4.21.0.
Updated documentation can be found here.

Hello @HansMusterWhatElse , after pondering about your last comment. I agree that to be at parity with respect with 3rd party libraries, an input event must be emitted: Thank you for your feedback. This has been implemented in v4.21.0.
Updated documentation can be found here.

That's great news! Thank you so much! 🙂👍


Parser: Set ordered list_item_open token info to input marker.

Rendering plugins may be interested in the actual values of the markers in an ordered list. For instance, in a list
1. Item 1
20. Item 2

a plugin might want to render the input markers, 1 and 20, rather than the auto-incremented marker values (1 and 2).
This commit uses the Token.info slot to store the input markers for list_item_open tokens. In the example above, the first list_item_open token will have token.info === '1', and the second will have token.info === '20'. The default renderer is unaffected, but plugins can access token.info if they are interested.
Benchmark performance seems similar.

You've made skipOrderedListMarker return 2 values:

pos (normal return value)
firstPos.pos (via modifying object)

What is the purpose in that?
As far as I understand it, firstPos.pos === pos - 1 always for successful parses, can't it be calculated from whereever the function is called?

Not sure if info is appropriate (could do either { info: '20', markup: '.' } or { info: '', markup: '20.' })... Nevermind, info is probably better for this.

For an ordered list with more than one-digit markers, like “2000.”, firstPos !== pos - 1.

I'd be happy to put this in markup but wasn't clear whether existing plugins depended on markup === '.'.

But yes you're right that using start, the way the code was written before, is equivalent, and I'll do that instead!

please do; changes look good to me otherwise


Fixes for Houdini 17

Based off Version 17.0.336

recheck

Merging this PR as discussed in the TSC meeting yesterday.


Use custom types in data structures and constants

Original issue: https://github.com/ethereum/eth2.0-specs/issues/667
The goal is to improve Spec readability by using alias custom types in data structures and as constant units instead of generic types.
What was done

Add Hash and Bitfield custom types. The Hash type is treated as a result of any 256-bit cryptographic hash function or its derivative preserving hash properties (like xoring two hashes)
Replace generic SSZ types in data structures with custom types where appropriate
Add/update constant units according to custom types

What else can be done

Add more custom types (like Time, ForkVersion, Bitfield64) which are now referred at 1-2 places in the Spec
As per @hwwhww suggestion in https://github.com/ethereum/eth2.0-specs/issues/667 cover all elements with custom types
In some way describe array index types (like  'validator_balances': [ValidatorIndex => Gwei])

I’m inclined to have custom types cover all elements. Compromise: define a general Uint64?

IMHO, defining aliases for types which are used just in a couple of places in the spec wouldn't make the spec clearer.
As well I see no big reason to make UInt64 => uint64 alias.
We probably need someone else opinion here. @djrtwo ?

Hmm, I think this PR would break CI tests...
Basically, it requires (i) defining a custom type in Python and also (ii) converting Python types to SSZ types when defining SSZ classes.
@Nashatyrev could you rebase it and then let's see how to fix it in function_puller.py?

@hwwhww rebased. Let me know if I can help you with CI tests

Is this still up to date? (I don't think so)
Do we still plan to merge this?
FYI: this breaks the typing syntax in many of the SSZ utilities in the pyspec. I.e. the default zero values, random instantiation of ssz, and minimal_ssz.py itself. These basic types are declared with strings, not types: they instruct what the typing looks like, they are not the type itself. I recommend to add an alias functionality, where we just lookup the "raw" type for any given type used in the SSZ container type declarations, and only then serialize/hash it etc.

Do we still plan to merge this?

I'm very pro adding aliases for readability. Do you want to go ahead @protolambda and fix whatever needs fixing? We can then merge.

I think we addressed most of the typing ideas with #1077. We can go further with it and use NewType here and there to describe some things in addition to typing them (e.g. Slot), but I think it's clean enough right now with the current data-types. So I'm closing this PR. Anyone who is interested is welcome to open a new PR for typing improvements, if anything can be improved on the current dev branch (this PR is quite out of date now).
:clap: for bringing this up so early, and apologies from us all that typing didn't make it into the spec so quick, I hope the new more complete typing support makes up for it.


Compiling Debug builds fails with undefined reference to 'rust_begin_unwind'.

I have been unable to get any of examples from Chapter 06, Hello World, on to compile in debug without modifying Cargo.toml.
$ xargo build --target thumbv7em-none-eabihf
Compiling usart v0.1.0 (file:///home/cldershem/Dev/LearningStuffs/LearnRust/DiscoveryBook/discovery/src/06-hello-world)
error: linking with `arm-none-eabi-gcc` failed: exit code: 1
  |
  = note: "arm-none-eabi-gcc" "-L" "/home/cldershem/.xargo/lib/rustlib/thumbv7em-none-eabihf/lib"
  //
  // lots of stuff omitted
  //
  = note: /home/cldershem/.xargo/lib/rustlib/thumbv7em-none-eabihf/lib/libcore-1e711822e5d5d922.rlib(core-1e711822e5d5d922.0.o): In function `co
    re::panicking::panic_fmt::hb7c1829b06bc1b3c':
          core.cgu-0.rs:(.text.cold._ZN4core9panicking9panic_fmt17hb7c1829b06bc1b3cE+0x24): undefined reference to `rust_begin_unwind'
          collect2: error: ld returned 1 exit status
error: aborting due to previous error
error: Could not compile `hello-world`.
To learn more, run the command again with --verbose.

$ uname -a && xargo --version && rustc --version
Linux cldershem-xps 4.10.0-35-generic #39-Ubuntu SMP Wed Sep 13 07:46:59 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
xargo 0.3.9
cargo 0.23.0-nightly (e447ac7e9 2017-09-27)
rustc 1.22.0-nightly (417c73891 2017-10-05)

Compiling with --release works fine. I can get it to compile in debug if I change the default opt-level in the Cargo.toml.
[profile.dev]
opt-level = 1

I cannot be sure if this is something specific to my setup or if it's an issue with a newer nightly. Any guidance on how to troubleshoot this issue will be appreciated.

This is a nightly regression (cf. rust-lang/rust#44853). You can workaround the issue by adding these two lines to Cargo.toml:
[profile.dev]
codegen-units = 1

This workaround was applied to all Cargo projects in this repo some days ago; you probably have an old checkout.

I love that this is a fast passed enough world that "some days ago" is "an old checkout".
I did a pull and tested one of the chapters. Everything compiled without errors. Thanks for pointing that out, I should have done a pull first.


Switch cell/cluster index for cluster/channel ID#

For various functions within the Analysis GUI, the user needs to specify the cluster index (when performing individual cell analysis). This can be confusing as the user needs to know beforehand what the cell index is before performing the analysis (particularly if the general filter has been applied). To remedy this, a fix will be put in place where the user will select the cell via a dropdown menu (instead of entering the cluster/cell index within an editbox).

This will need to be added to the following functions:
CLUSTER MATCHING

Fixed/Free Cluster Matching (Fixed/Free Multiple)
Matched Signal Comparison (Fixed/Free Single)
Matched Cluster ISI Metric (New Method) (Fixed/Free Multiple)
Inter-Spike Interval Distributions (Fixed/Free Multiple)
Matched Cluster Metric (Old Method) (Fixed/Free Multiple)

CLUSTER CLASSIFICATION

Cluster Cross-Correlogram (Fixed Multiple)*

ANGULAR HEAD VELOCITY ANALYSIS

Individual Cell Correlation (Fixed) (Fixed/Free Single)

ROTATION ANALYSIS

Rotation Trial Spiking Rates (Fixed Single)
Phase Spiking Rate Comparison (Individual Cell) (Fixed Single)
Spiking Rate Heatmap (Fixed Single)
Motion/Direction Selectivity (Fixed Single)
Kinematic Spiking Frequency (Fixed Single)

UNIFORMDRIFT ANALYSIS

UniformDrift Trial Spiking Rates (Fixed Single)
UniformDrift Phase Spiking Rate Comparison (Individual Cell) (Fixed Single)
UniformDrift Spiking Rate Heatmap (Fixed Single)

ROC ANALYSIS

Direction ROC Curves (Single Cell) (Fixed Single)
Velocity ROC Curves (Single Cell) (Fixed Single)

DIRECTION LDA

Shuffled LDA (Fixed Single)

SINGLE EXPERIMENT ANALYSIS

Mean Spike Signals (Fixed Single)*
Auto-Correlogram (Fixed Single)*
Cross-Correlogram (Fixed Single)*

=> * - For these functions, the free experiments need to be removed from the experiment list


rubyLibs are now broken

I appriciate work on new ruby support, but rubyLibs, are now broken. I think next time before we do this kind of changes, we should take back compatibility in consideration.
@cstrahan

Where does rubyLibs live? I can't find the string anywhere in nixpkgs.

Where does rubyLibs live? I can't find the string anywhere in nixpkgs.

@joelteon and I phased out rubyLibs in favor of having Ruby apps use the specific versions they require per their Gemfile.
I can bring it back though, if other people have infrastructure using rubyLibs. Unless someone makes a compelling argument, I would treat that as a deprecated feature, though.

Here is the old ruby/generated.nix file that contains a list of packages that were removed along with rubyLibs.
I have no idea how many of those have been moved to the new bundlerEnv system, but if the answer isn't "most" then I don't see how rubyLibs has been "phased out". It might be good to have rubyLibs alongside the new system for as long as packages are being phased in.
I was (read: still am) using at least two of those packages daily: tmuxinator and Jekyll. So far I've been surviving with a branch of the commit before they were removed (git branch rubyLibs ea5594679c438a4666f29a209f68b6cce81054f5), but it would be nice to have them back in my configuration.nix. I'm in the process of making a bundlerEnv derivation for tmuxinator based on the instructions on the mailing list, but I haven't quite figured it out yet.
Sorry, I don't want to make too much of a stink about this (I still have my rubyLibs branch), but something probably needs to happen before this hits NixOS stable and trips up even more people.

@cstrahan I tried to package jekyll following the instructions in the mailing list, but ran into trouble: http://comments.gmane.org/gmane.linux.distributions.nixos/15702
I use jekyll regularly at work, so it would be really great to have it back in nixpkgs.

I'm using compass for building sass in production. As long there's no
suitable alternative for ruby gems, rubyLibs from @marcw should ne kept.
After that all the packages should be migrated.
On Feb 21, 2015 10:15 AM, "cillianderoiste"<EMAIL_ADDRESS>wrote:

@cstrahan https://github.com/cstrahan I tried to package jekyll
following the instructions in the mailing list, but ran into trouble:
http://comments.gmane.org/gmane.linux.distributions.nixos/15702
I use jekyll regularly at work, so it would be really great to have it
back in nixpkgs.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/6340#issuecomment-75363318.

We already have new ruby support backported to nixos 14.12, that includes
rubyLibs, so i think master should too.
On Feb 21, 2015 11:44 AM, wrote:

I'm using compass for building sass in production. As long there's no
suitable alternative for ruby gems, rubyLibs from @marcw should ne kept.
After that all the packages should be migrated.
On Feb 21, 2015 10:15 AM, "cillianderoiste"<EMAIL_ADDRESS>wrote:

@cstrahan https://github.com/cstrahan I tried to package jekyll
following the instructions in the mailing list, but ran into trouble:
http://comments.gmane.org/gmane.linux.distributions.nixos/15702
I use jekyll regularly at work, so it would be really great to have it
back in nixpkgs.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/6340#issuecomment-75363318.

@offlinehacker can this be closed?

I have an open PR to resolve this: https://github.com/NixOS/nixpkgs/pull/6580
@offlinehacker Does that look good to merge?

@offlinehacker ping?


Adjust table to be able to have variable internal items types

Make use of the pandas variable type parameter. Boolean, string or number. Adjust the table column to have its input type be based upon the pandas type, number, boolean, or string.
That way can make a "checkbox list" with the table.

Will implement this with the following method instead:
https://github.com/SimonBiggs/scriptedforms/issues/106


Font replacement is not successful in the HyperOS CN environment.

When I install the module, it only changes in some places.
For example, a page in Chrome or a clock in the status bar will work, but everything else will be MiSans.
Environment:
Redmi K70E
Xiaomi HyperOS <IP_ADDRESS>.UNLCNXM
No other magisk modules installed.

Can you share this file:
/system/etc/fonts.xml
I don't have HyperOS device now so I can't check, but maybe there's some changes in that file.

I uploaded the file to GDrive.
https://drive.google.com/file/d/1--GL5uULMZoYGWuH3rDTc-Ge5jr_5QDV/view?usp=drivesdk

OK, please test this one:
magisk-module-koruri-font.zip

I can see MiSans being replaced by Koruri Font in Font Settings, but most other apps, Control Center, etc. use MiSans.

hmm, I'll check after my device receives HyperOS update...

It looks like HyperOS forces MiSansVF if theme font isn't applied, even if the module replaces MiSansVF.ttf.
I added workaround to apply font as theme font in v26.

Font substitution was successful in my environment as well. Thank you for your response.


Fragment queries with plugin-scope-auth and plugin-errors

I have a project with the Prisma Plugin, Auth Plugin and the Errors Plugin set up. I'm trying to achieve the recommended Errors Plugin usage with using fragments in the queries.
The builder has the following settings:
export const builder = new SchemaBuilder({
  ...,
  plugins: [
    // no matter the order, the behavior is the same
    ErrorsPlugin,
    ScopeAuthPlugin,
    ...
    PrismaPlugin,
    PrismaUtils,
  ],
  scopeAuthOptions: {
    authorizeOnSubscribe: true,
    // treatErrorsAsUnauthorized and unauthorizedError aren't used as they don't result in the desired behavior
  },
  errorOptions: {
    defaultTypes: [UnauthorizedError],
  },
})

The fields are generated with prisma-generator-pothos-codegen with the following structure:
builder.prismaObjectField('User', 'username', (t) => t.field({
  ...
  authScopes: {
     admin: true
  },
  unauthorizedResolver: (q, r, i, c, e) =>
    throw new UnauthorizedError(e.message)
  },
  // adding "errors" field results in the field being split into fragments, which makes the queries too complex to write
})

Query fields have their own authScopes set up, as well as a list of available errors.
Running the following query:
query findUser {
  findFirstUser(where: {
    id: {
      equals: "21599895-8f9c-4537-abc2-0cac7ecec6c7"
    }
  }) {
    ... on QueryFindFirstUserSuccess {
      data {
        username
      }
    }

    ... on UnauthorizedError {
      message
    }
  }
}

Returns the following response:
{
  "errors": [
    {
      "message": "Not authorized to resolve User.username",
      "path": [
        "findFirstUser",
        "data",
        "username"
      ],
      "extensions": {
        "code": "UNAUTHORIZED",
        "serviceName": "server"
      }
    }
  ],
  "data": {
    "findFirstUser": null
  }
}

This behavior is undesired since now findFirst returns null on an object which wasn't retrieved because of authorization issues, which are being caught.
Setting nullable: false on the query field only results in the response changing to:
{
  "errors" [...],
  "data": null
}

The resolve-helper aggregates the issues into a single object:
https://github.com/hayes/pothos/blob/da2bc980dfb348830f8a46c0b05e30ec6b38be30/packages/plugin-scope-auth/src/resolve-helper.ts#L44
The failures can later be accessed as shown in the documentation. However, I wasn't able to throw the errors in a way which would let Errors Plugin handle them.
Is there a way in the current API to let the Errors plugin handle the Auth Plugin field issues?

I've created an example here, but I'm not sure if I am understanding your issue correctly:
https://stackblitz.com/edit/typescript-opc1zd?file=index.ts
Can't use the prisma plugin in browser, but hopefully this is good enough to either show how to handle the errors, or to better explain the desired behavior so I can update the example

Thank you for your swift response!
I have extended your example with the issue described more closely and comparison of three approaches to the issue:
https://stackblitz.com/edit/typescript-vxyqg3?file=index.ts
Thank you for the opportunity to work on the example! The code helped me understand the approaches better.
The second approach "scopes on object with errors" is matching the requirements of validating every field individually, but as you can see, it results in larger queries. This can be easily overcome on the front-end with utilities generating the Error fragments on the fields. It would look prettier to if the "data" field for Objects was moved into the root of the success fragment, but that can't be done with primitive types, so it would introduce inconsistencies. I suppose some query building utilities would help with this too.
Do you find this approach to be the optimal approach for setting up auth scopes for individual fields, or could there be some other approach I'm missing?

I think I understand your question better now, thanks for sharing that updated version!
To answer the simplest piece first, you can set directResult: true to avoid having the extra nested data field as long as it's returning and object, and not a list, scalar, or enum.
Unfortunately, its not possible to create your ideal version (currently).  Pothos is fundamentally just a schema builder that creates the schema types and resolvers, and is bound by how graphql js executes queries.
When you move the auth checks down to the User type, the checks won't be run until after the query field is fully resolved, and it's already been determined from graphqls perspective that you are returning a User and not an Error.
When the auth error is thrown, there isn't a way to go back and change what's being returned by the query resolver.  Unfortunately this means that the error plugin can currently only handle errors from its own resolver (or from things like auth, that Pothos adds into the resolver) but anything nested in a child resolver will happen too late for the error plugin to do anything.

Thank you for the wonderful answer! Now I better understand the workings of the system. I think that the way it currently works is great, and the somewhat longer syntax is something that can be easily dealt with but gives a lot of possibilities.
The directResult option is exactly what was missing!
Looking further into auth-scopes, it seems that mutations can only be dealt with on the field itself. Plus, if the field scope checks run only after the query is resolved, it would throw only after the mutation for that field is complete, which is undesirable.
So one option for per-field authorization is checking the scopes for each field inside the resolver before running any functions inside of the resolver. The functions would run on allowed fields only and on success return the values of the modified fields, and otherwise return ForbiddenError:
return {
  forbiddenField: new ForbiddenError(),
  allowedField: "new value"
}

However, for this approach to work, especially with authscope functions to retrieve parent data, it would be necessary to access the authScopes for an ObjectType inside of the resolver.
Is there a way to achieve that, or should a different approach be taken with the current API?

Mutation fields are treated somewhat specially in GraphQL, and it is generally advised not to perform any mutations outside of the root mutation fields.  The only real difference form an implementation standpoint is that fields on the Mutation type are executed serially while fields on all other types are executed in parallel.
What I would recommend is that all mutations happen directly inside resolvers on the Mutation type, and anything nested is purely for querying the response of the mutation.   When you follow that pattern, you can put all your authorization checks for the mutation on the field itself, and then the auth checks on nested fields just for making sure the user is authorized to read that part of the response.   This makes it a lot easier to reuse types between queries and mutations, which can be important for efficient cache updates if you are using a client that has some sort of normalized cache


Update status of appointment

@tmccormi @Leo24 for Appointment status, can you return and update status through list_options table? We have a new requirement for a new mapping. So when you get an appointment status through FHIR, look at the status, compare with list_options table mapping column, and then store the appropriate option_id in the libreehr_calendar table. Conversely, when you send appointment data out through FHIR API, can you look at appointment status (which will be a symbol like @ or ~) look up appointment in list_options table, and return the value from the mapping column instead of option_id column?
To get appointment status SQL to provide to FHIR:
SELECT mapping FROM list_options WHERE list_id = 'apptstat' AND option_id ='@';
// Then return the mapping instead of the value stored in LibreEHR appointment table
To set appointment status SQL to receive from FHIR:
SELECT option_id FROM list_options WHERE list_id = 'apptstat' AND mapping ='available';
// Then set the option_id instead of the mapping inside LibreEHR appointment table


Small README.md update

Simple README.md update regarding simultaneous connections and users.
I've spent some time figuring it out so it can help someone.

Don't need this, sorry.


Update to Berkshelf 3

Wercker doesn't have a service box for Postgres 9.3, only 9.2, which doesn't have JSON support. Forked the wercker box repo to updated it, but it relies on Berkshelf v1 and wasn't able to get all the cookbooks needed to build. Got tired of waiting for the berkshelf overrided in the postgres box to rebuild when testing that box, so forked the berkshelf box here to update to v3.
Tested and validated, will merge TBR.

Wrong repo. sorry.


core: Reduce logging in failed SSH attempts

Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci I self-certify!


endOfNight.sh: aid in debugging

Add completion messages to log to make it easy to see when the commands finished.

Since you're doing this.. why not move the 'complete' statement to just below the generation, and add another inside the upload conditional saying $thing upload complete  :)

@linuxkidd Michael, the reasons I was going to give for how I do it don't make sense after I looked at the code again, so I will change to how you suggested.  Since almost all the issues users have are with the generation of those files and not uploading them, putting the 2nd message right after the generation will make it easier to find issues in the log file.

@linuxkidd Ready for review.


Clean up ApplicationHelper::ToolbarChooser spec

Fix describes and contexts to improve readability
Remove unnecessary includes and extends
Remove method_missing hack
Upgrade to expect syntax

@miq-bot add_label ui

@miq-bot add_label test

@miq-bot add_label refactoring

@Fryguy @jrafanie Here's the daily deletion.  Happy Turkey day! :chicken:

Nice @bdunne!  Looks good.
Good use of emoji in your commit message titles!


Ignore the 'ErrKeyNotFound' error for the kv provider

What does this PR do?
Fixes #8092
Motivation
When the root key does not exist, listing the kv store will result in an 'ErrKeyNotFound' error. In this case, the kv provider will not send a dynamic.Message to the configurationChan for updating the configuration. So when we delete the root key in the kv store to clear all traefik configurations, traefik will always retain the last configuration.
More

[x] Fix the kv provider, improve the buildConfiguration function.
[x] Updated 'ConsulSuite' integration tests.
[x] Improve the 'displayTraefikLog' function in the integration test to make the log more human-readable

Additional Notes
While fixing this bug, I encountered another potential bug caused by the kvtools/redis library when using Redis as the kv store. The kv provider utilizes the WatchTree interface to monitor events of key changes in the kv store. However, when we delete the root key in Redis, the WatchTree fails to notify the kv provider about the modification of certain keys. Therefore, even though I have fixed the buildConfiguration function of the kv provider, this bug may still persist when Redis is used as the kv store. About the bug of kvtools/redis , I have created an issue: https://github.com/kvtools/redis/issues/4

Can someone help me rerun the unit test ? I have successfully passed it in my local machine.

Hello,
your PR doesn't target the right branch, can you rebase your PR on the branch v2.10

Hello,
your PR doesn't target the right branch, can you rebase your PR on the branch v2.10?
can you also remove all the unrelated changes?

https://github.com/traefik/traefik/pull/10082/files#diff-edb80201b1a2847d7a1dfbc356e9b5d7dfbd9706cda7d622ff489d05d640e87bR28-R47
https://github.com/traefik/traefik/pull/10082/files#diff-9f5fd20913c5fe80c85263153fa9a0b28dbd1407e53da4ab5d09e13d2774c5dbR269-R280

Okay, I will remove the second change. However, for the first one, I moved the setupStore function to SetUpSuite and the cleanup action to TearDownSuite, which is executed by the go-checker. This means we don't need to care about setting and cleaning some common things in every testcase. Are you sure I should remove it?

Are you sure I should remove it?

yes


[GH-529] No attachment text when error happens while uploading attachment

Summary
Attachment text not shown if error occurs while adding attachment.

Ticket Link
Fixes https://github.com/mattermost/mattermost-plugin-jira/issues/529

Thanks @hectorgabucio for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

I'm a contributor, hector2 but I changed the Github username :roll_eyes:

/check-cla

Codecov Report

Merging #535 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #535      +/-   ##
==========================================
- Coverage   33.92%   33.91%   -0.01%
==========================================
  Files          37       37
  Lines        4604     4605       +1
==========================================
  Hits         1562     1562
- Misses       2893     2894       +1
  Partials      149      149

Impacted Files
Coverage Δ

server/issue.go
16.35% <0.00%> (-0.03%)
: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 d776dd3...48fec4a. Read the comment docs.


Instructions

Please add instructions for installing and running in the readme

did anyone figure this out yet? ive been trying to get this to work for hours now...


Update th-th.xaml

okay I understand At first I thought your program would support Windows 10 and 11.

okay I understand At first I thought your program would support Windows 10 and 11.

The function is provided by Microsoft, not me.

okey i'm motivation for your team I may be just a small part of your program development. But I'm happy to help ^_^


Add navbar logo text

Close #540. See https://github.com/scientific-python/scientific-python-hugo-theme/issues/398.

Before

After


Fix clang-362823 make step

During execution via check_smoke_fails only 1/3 compile steps was attempted, this was due to an ordering issue.
Note, this test still (expectedly) fails:
lld: error: duplicate symbol

Unsure how this behavior was caused by my previous PR https://github.com/ROCm/aomp/pull/978
Could be replicated by issuing make: only one object file would be compiled.
make check and make run did show the expected behavior 2 successful compiles + 1 link fail.


Some start-up questions

Hey!
Trying to get the current CLI tool to work but running into some issues, and want to make sure there's not something fundamentally wrong with my setup.
$ ./bin/run

Evently.cloud CLI tools to develop applications and manage your ledgers.

VERSION
  evently-cli/0.0.1 linux-x64 node-v18.15.0

USAGE
  $ evently [COMMAND]

COMMANDS
  commands  list all the commands
  help      Display help for evently.
  update    update the evently CLI

Note that there's no mention here of a 'ledger' command. Running this:
$ /bin/run ledger
Ledger commands

USAGE
  $ evently ledger:COMMAND

Note that there's no commands registered, and I can't run either of these:
./bin/run ledger:download

 ›   Warning: ledger:download is not a evently command.
Did you mean help? [y/n]:
 ›   Error: Run evently help ledger for a list of available commands.

The docs in the code suggest ./bin/run ledger download but that also doesn't work:
  static examples = [
    `$ evently ledger download
Validated 13,438 ledger events.
`]

Just want to make sure I'm not missing something obvious before I pull this apart ;)

I needed to run npm run build


[DX] Lead lib to be PHP5.6 compliant ?

I'm not a fan of it. PHP5.6 is in security fixes only since January . We should encourage the use of PHP 7.X.


enyo-launcher: 2.0.2 -> 2.0.5, renamed from enyo-doom

Description of changes
This is a new package, potentially a duplicate of enyo-doom, which has been changed to "enyo-launcher" and has a different upstream repo, and three minor version releases since (from 2.02 to 2.05).
New Gitlab repo: https://gitlab.com/sdcofer70/enyo-launcher
I did not test for breaking dependent packages, as it has no dependent packages. I also only have access to x86_64-linux  on NixOS right now, so it's all I could test the build on.

Things done

Built on platform(s)

[✅] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[✅ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[✅ ] Tested basic functionality of all binary files (usually in ./result/bin/)

Result of nixpkgs-review pr 188557 run on x86_64-linux 1

1 package built:

enyo-launcher

Ofborg ran out of space, appears to be an infra problem.

Can we just rename the existing enyo-doom package and change its src URL? This doesn't warrant a new package imo.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/613

@ofborg build enyo-launcher

Please rebase to fix the merge conflict and adjust the commit message to what I renamed the PR to.

Shoot, I probably need to squash those commits, but don't seem to be able to find them all with git rebase -i HEAD~2 which should have my merge that I used to rebase the commit.


Removing cookies are failing when the domain or path were set in the cookie

When trying to delete a cookie created by setting options.domain or options.path it is not working.
const cookieStore = new WebStorage('cookieStorage');

cookieStore.setItem('testing', [1,4,7], {
  domain: '.wordpress.com',
  expires: { minutes: 10 },
});

cookieStore.getItem('testing'); // ok, it exists
cookieStore.removeItem('testing'); // failing!
cookieStore.getItem('testing'); // failing, still exists!

Take into account only the commit 42ea67e.
All other commits were generated because of the command:
git commit -a --amend --no-edit


[RESTEASY-1025] async ClientHttpEngine documentation.

Thanks Rebecca, merging. I'm actually moving the doc added here to the RESTEasy Client API chapter, as it's not really something strictly related to JAX-RS 2.1


Update Maps/Edit menu for consistency

Minor enhancement for consistency.

Add the 'Change Privacy' option to the Edit menu for a selected map
Add the 'Duplicate Map' option to the dashboard shortcuts for a selected map

Note: The Change Privacy option is available from the dashboard shortcuts when a map is selected from the Maps dashboard. This is not consistent with how the options appear compared to the Datasets dashboard.  Is this by design?  If not, fix for consistency (not that doc will need to be reflected if changed).
See example screen capture for Maps dashboard shortcuts and Edit menu:

Note the difference for the Datasets dashboard shortcuts and Edit menu:

Sure thing, good catch. There is already a ticket for the duplicate thingy. But adding this to a milestone to prioritize. Thanks!


[FR]: support webpack 4

What is the current behavior?
Worker crashes with webpack 4, not testing with webpack 4.
Describe the feature
Webpack 4 is still widely used and probably worth supporting.

Do you have some client code that updates the worker, at least we could stash in a PR here so we have a chance of using it to resolve this issue.