Using Checkboxes to get Input and respond depending on the selection

I have the a message block with a checkbox set to display in the chat when a key-word is detected. I want the user to be able to select options and then my bot to respond accordingly, depending on the option selected (i.e: Sensitive Data, Malware, Vulnerability Severity etc ... ).
Reproducible in:
slack-bolt==1.6.1
slack-sdk==3.6.0

Python runtime version
Python 3.7.2
OS info
Microsoft Windows [Version 10.0.17763.1999]
Steps to reproduce:
(Share the commands to run, source code, and project settings (e.g., setup.py))
{
	"blocks": [
		{
			"type": "section",
			"text": {
				"type": "mrkdwn",
				"text": "I noticed you mentioned a failed scan ... Please select which control is failing the Scan:\n\n"
			}
		},
		{
			"type": "input",
			"element": {
				"type": "checkboxes",
				"options": [
					{
						"text": {
							"type": "plain_text",
							"text": "Vulnerability Severity",
							"emoji": true
						},
						"value": "value-0"
					},
					{
						"text": {
							"type": "plain_text",
							"text": "Sensitive Data",
							"emoji": true
						},
						"value": "value-1"
					},
					{
						"text": {
							"type": "plain_text",
							"text": "Malware",
							"emoji": true
						},
						"value": "value-2"
					}
				]
			},
			"label": {
				"type": "plain_text",
				"text": "Please select all relevant options:",
				"emoji": true
			}
		}
	]
}

Expected result:
I am looking for a way to get the bot to display specific messages, depending on the checkbox option selected in the message above.
What are my options in trying to achieve this? Any help would be much appreciated!
Actual result:
(Tell what actually happened with logs, screenshots)
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Hi @alecrevangelista,
Thanks for the code sample! It helps a lot.
I think you're on the right track. To respond to a checkbox selection, you'll need to add an action_id. For example,
{
	"type": "input",
	"element": {
		"type": "checkboxes",
		"action_id": "security_action_id",
		"options": [ ... ],
		...
}

After declaring an action_id you can listen and respond to the action using Bolt:
@app.action("security_action_id")
def security_action(ack, say):
    # Acknowledge action request
    ack()
    say("Security report received 👍")

Hope that helps and please let me know how it goes!
Michael

Hi @mwbrooks ,
Thank you for the response! So far, I have got it to a point where I have my checkbox and three options, with an action_id = action1 just as you have described above:
{
	"type": "input",
	"element": {
		"type": "checkboxes",
		"action_id": "action1",
		"options": [ ... ],
		...
}

I want the app to have a distinct response for each of the three checkbox responses -- So I have my listener is defined as follows:
@app.block_action("action1")
async def handle_options(ack, respond, body, say, logger):
    await ack()
    for i in body["actions"][0]["selected_options"]:
        if i["text"]["text"] == "Vulnerability Severity":
            await say(text="Message 1!!")
        elif i["text"]["text"] == "Malware":
            await say(text="Message 2!!")
        elif i["text"]["text"] == "Sensitive Data":
            await say(text="Message 3!!")

The problem in my code here is that every time a checkbox item is selected, the message will repeat for the options that are already selected. I am wondering if there is a better way for me to handle each option selection here?
Also, using the say() here is displaying a new message, but ideally, I want it to reply as a thread to the original message with the checkbox. or only visible to the person selecting the options --  Is that possible?

Hey @alecrevangelista,
It's good to hear that you've making some progress!

The problem in my code here is that every time a checkbox item is selected, the message will repeat for the options that are already selected. I am wondering if there is a better way for me to handle each option selection here?

Checkboxes used directly inside messages could make this more difficult. While the API allows you to do this, it wasn't necessarily designed around this use-case. Normally, checkboxes would be found in views (app home screen or modals).
It's also challenging to store metadata and state inside a Slack message. The closest way to store state is the private_metadata value available to views. Normally, we'd recommend that you try to persist data in your own database and use something like the message timestamp as a unique identifier.
A more traditional approach would be to display a message with an "Edit" button. Pressing the button would open a modal with the checkboxes selected based on your database. When the modal is submitted, the database can be updated and - if you want - the original Slack message could also be updated.

Also, using the say() here is displaying a new message, but ideally, I want it to reply as a thread to the original message with the checkbox. or only visible to the person selecting the options -- Is that possible?

One of the easiest ways it to use the chat.postMessage method available with client:
@app.action("action1")
def approve_request(ack, body, say, client):
    ack()
    client.chat_postMessage(channel=body["channel"]["id"], thread_ts=body["message"]["ts"], text="Request approved 👍")

Hi @mwbrooks,
Thanks for the info!
I have decided to take a different approach with this. So I am now using a "multi_static_select" object instead of the "checkbox ", which prompts the user to select some options as follows:
{
	"blocks": [
		{
			"type": "section",
			"block_id": "section678",
			"text": {
				"type": "mrkdwn",
				"text": "Pick items from the list"
			},
			"accessory": {
				"action_id": "action1",
				"type": "multi_static_select",
				"placeholder": {
					"type": "plain_text",
					"text": "Select items"
				},
				"options": [
					{
						"text": {
							"type": "plain_text",
							"text": "OPTION 1"
						},
						"value": "value-0"
					},
					{
						"text": {
							"type": "plain_text",
							"text": "OPTION 2"
						},
						"value": "value-1"
					},
					{
						"text": {
							"type": "plain_text",
							"text": "OPTION 3"
						},
						"value": "value-2"
					}
				]
			}
		}
	]
}

I have defined my listener as follows -- This will look at the selections made in the object above, and will output a specific message when the "Confirm" button is clicked (i.e "action_id": "action1"). I want this message to be sent as a "thread" and have used chat.postMessage as you have suggested above.
@app.block_action("action1")
async def func(ack, respond, body, say, logger, client):

    logger.info(body["actions"][0]["selected_options"])

    for i in body["actions"][0]["selected_options"]:

        await client.chat_postMessage(channel=body["channel"]["id"], thread_ts=body["message"]["ts"], text="You have selected{}\n".format(i["text"]["text"]))

However, when I do this, the app sends an entirely new message, and does not reply to the original thread where the prompt was asked in the first place. Is there something I am doing wrong here?
Thanks!

@alecrevangelista

However, when I do this, the app sends an entirely new message, and does not reply to the original thread where the prompt was asked in the first place. Is there something I am doing wrong here?

The body["message"] can have body["message"]["thread_ts"] in addition to body["message"]["ts"]. In this case, your app needs to set the thread_ts, not the ts of the message.


[PictureRecorder] Allow PictureRecorder to draw inside of an isolate

Currently, you can not use PictureRecorder inside an isolate or it ends up in:
Exception: UI actions are only available on root isolate.
	#0      PictureRecorder._constructor (dart:ui/painting.dart:4458:59)
	#1      new PictureRecorder (dart:ui/painting.dart:4457:23)
	#2      IconGenerator._drawCircle (package:client/src/common/utils/map/icon_generator.dart:87:32)
	#3      IconGenerator.buildCircle (package:client/src/common/utils/map/icon_generator.dart:51:26)
	#4      MarkerGenerator._buildDots.<anonymous closure> (package:client/src/common/utils/map/marker_generator.dart:62:37)
	#5      MappedListIterable.elementAt (dart:_internal/iterable.dart:417:31)
	#6      ListIterator.moveNext (dart:_internal/iterable.dart:343:26)
	#7      Future.wait (dart:async/future.dart:406:26)
	#8      MarkerGenerator._buildDots (package:client/src/common/utils/map/marker_generator.dart:68:19)
	#9      MarkerGenerator.build (package:client/src/common/utils/map/marker_generator.dart:30:22)
	#10     buildMarkers (package:client/src/widget

Use case
In my app we have a lot of markers and building them every time is causing some UI freezes, it would be useful to be able to create them on an isolate so the UI does not freeze.
An example of what we do to create a marker using a PictureRecorder:
// https://stackoverflow.com/a/56534916/273119
  Future<Uint8List> _drawColoredMarker(String text, Color color) async {
    final pictureRecorder = ui.PictureRecorder();
    final canvas = Canvas(pictureRecorder);

    final paint = Paint()..color = color;
    canvas.drawRRect(_rect, paint);

    final textPainter = TextPainter(
      textDirection: TextDirection.ltr,
      text: TextSpan(text: text, style: _textStyle),
    );
    textPainter.layout();
    final offset = Offset(
      (_width * 0.5) - textPainter.width * 0.5,
      (_height * 0.5) - textPainter.height * 0.5,
    );
    textPainter.paint(canvas, offset);

    final image = await pictureRecorder
        .endRecording()
        .toImage(_width.floor(), _height.floor());

    final data = await image.toByteData(format: ui.ImageByteFormat.png);
    return data!.buffer.asUint8List();
  }

Which ends up drawing markers like this:

Proposal
I'm not really sure how to do it that, but it is related to this: https://github.com/flutter/flutter/issues/30604

There are already multiple issues for this feature/bug


[v1.2][Raymond Zheng] DeleteTuitionTaskCommand

Pull Request Test Coverage Report for Build 108

0 of 24 (0.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.07%) to 86.741%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/model/tutee/TuitionSchedule.java
0
5
0.0%

src/main/java/seedu/address/logic/commands/DeleteTuitionTaskCommand.java
0
19
0.0%

Totals

Change from base Build 102:
-1.07%

Covered Lines:
1714

Relevant Lines:
1976

💛  - Coveralls


URO for POST Request

Can i use URO for decluttering post request to use fetch unique requests using ffuf

no


CMakeList: remove DSN_IN_CORE & improve BOOST linking

What problem does this PR solve?
https://github.com/XiaoMi/pegasus/issues/361#issuecomment-514975986
What is changed and how it works?
Check List
Tests

No code

Related changes

Need to cherry-pick to the release branch
Need to be included in the release note

waiting for updating rdsn


Question: merging nested objects

Question:
How do you think about...

Organizing produce functions for a reducer with nested objects
Using nested produce functions every time you update a nested object

Context
Using immer and nevernull for nested object updates.
I am moving our boilerplate reducer code to use immer.
Trying to modify code that merges objects (potentially nested):
bar: {...state.foo.bar, ...action.nextBar},

Problem
I am unsure if I need to produce drafts of new objects, or if produce will add its immutability properties to new, nested objects (my intuition tells me no). For example:
Example
I have a state tree so that
state = {
  foo: {
    bar: null
  }
}

some steps later..
state = {
  foo: {
    bar: {
       baz: 1
     }
  }
}

Intuition
If I want to update baz to 2, I want to do:
produce(state, draft => draft.foo.bar.baz() = 2)
However, I THINK what I'll need to do is make bar a subreducer so that
produce(state.bar, draft => draft.baz() = 2)

I am unsure if I need to produce drafts of new objects, or if produce will add its immutability properties to new, nested objects (my intuition tells me no).

See here: https://github.com/immerjs/immer/issues/260#issuecomment-443630375


Fix CLI usage

Hi, I really look forward to introduce a tooling like this for our component libraries. Unfortunately, the described usage in the readme didn't work for me, so I'd suggest some minor changes :)

to provide a binary entrypoint for the cli tool one needs to declare it in the manifest
the entry file for the bin needs to have a shebang declared
as the fs-extra package is used at runtime, it's a dependency not a devDependency.

Thanks for the improvements! I'll get this released soon


Issue: Cost calculated for trace when using GPT4-Vision is wrong

Issue you'd like to raise.
Hello, I'm running the following configuration:
    const chat = new ChatOpenAI({
      modelName: 'gpt-4-vision-preview',
      streaming: true,
      maxTokens: 1024,
    }).withConfig({ runName: 'VisionChain' });

    const message = new HumanMessage({
      content: [
        {
          type: 'text',
          text: "...",
        },
        {
          type: 'image_url',
          image_url: {
            url: `data:image/jpeg;base64,${base64Data}`,
            detail: 'low',
          },
        },
      ],
    });

I suspect langsmith is counting the image passed as base64 as it was a normal string instead of being tread as an image for the vision api. A call that actually costed me 0.03 is being reported as $2.2. This can be also observed by the token count that  takes in consideration the full base64 encoded string (which is not wrong per se). Display of the submitted image in the prompt is working as expected so thumbs up for that :)
Suggestion:
The cost should be calculated using the pricing model of the gpt-4-vision-preview and not the standard model.

Ah yes - we have a fix in the pipeline- thank you for flagging!

I'm seeing a similar issue with Gemini Pro Vision, no cost data is displayed, but the number of tokens for a request with an image are in the millions. Hoping the fix mentioned above covers this as well?
model = ChatVertexAI(model_name="gemini-pro-vision", max_output_tokens=2048, temperature=0.01)

    msg = model.invoke(
        [
            HumanMessage(
                content=[
                    {"type": "text", "text": prompt},
                    {
                        "type": "image_url",
                        "image_url": {"url": f"data:image/jpeg;base64,{img_base64}"},
                    },
                ]
            )
        ]
    )

When we configured our model with stream_usage=True, it looks like the correct token usage started being reported to LangSmith instead of the incorrect one based on the base64 data.
model = ChatOpenAI(model="gpt-4o", streaming=True, stream_usage=True)


ISPN-14544 RESP endpoint cache shouldn't require no expiration config…

…ured
https://issues.redhat.com/browse/ISPN-14544

Merged, thanks


bump gql to prerelease 3

Status
Ready
Description
github broke their schema in https://github.com/octokit/graphql-schema/commit/18c40d2bb68d8f27acfc62e768d34aadfca9e8b2#diff-9fcde326d127f74194f70e563bdf2c118c51b719c308f015b8eb0204a9a552fbR21764
gql v2 uses graphql-core-legacy which does not handle that properly
in v3 they moved to use graphql-core
currently the integrations fails on Syntax Error GraphQL (1:1) Unexpected Name "null"

Docker Image Ready - Production
Docker automatic build at CircleCI has deployed your docker image: demisto/graphql:<IP_ADDRESS>80
It is available now on docker hub at: https://hub.docker.com/r/demisto/graphql/tags
Get started by pulling the image:
docker pull demisto/graphql:<IP_ADDRESS>80

Docker Metadata

Image Size: 22.69 MB

Image ID: sha256:7c3c99f7d9138f8621dcc9602ae2e1cd800912e2febc0b0c421232734d26dd13

Created: 2021-03-11T07:51:35.353553525Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=E3FF2839C048B25C084DEBE9B26995E310250568
PYTHON_VERSION=3.8.6
PYTHON_PIP_VERSION=20.2.4
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/fa7dc83944936bf09a0e4cb5d5ec852c0d256599/get-pip.py
PYTHON_GET_PIP_SHA256=6e0bb0a2c2533361d7f297ed547237caf1b7507f197835974c0dd7eba998c53c
DOCKER_IMAGE=demisto/graphql:<IP_ADDRESS>80

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:936c9cfb567c3b2a77a0474f317023d672cb7c40
org.opencontainers.image.version:<IP_ADDRESS>80


Identifying  new streams

When we get a call to on_new_stream(), is there a way to find out which stream is being created? For example, suppose a connection calls lsquic_conn_make_stream() several times, how can we tell which call caused the stream now being processed?
If it were not for the fact that stream creation can be delayed, a solution would be to have lsquic_conn_make_stream() return  the StreamID. Another solution would be the ability to pass something into lsquic_conn_make_stream() that will be relayed to the new stream or to on_new_stream().
Is there an existing solution that I have not seen?

Why you need to identify a new stream?
All new streams from the same connection are pretty much identical, other than the stream ID.

OK, Got it. So my solution will be to give connections a queue of pending sessions and have new streams just pop a task off of that queue. Thanks!


Invite rest client

Invite rest client
There is currently no Invite rest client.
This should be added in V2.0.0
Resources
https://discord.com/developers/docs/resources/invite

Added in the 2.0 Branch


update dependencies

@loosebazooka another security issue was found in commons compress

I think you need to also update the .travis.yaml for java8 builds to be something like: https://github.com/GoogleContainerTools/jib/blob/master/.travis.yml#L7

Travis passed. For some reason Github didn't pick that up.

Weird, they're optional though, and usually so external contributors can see failures. You can probably go ahead and merge this. Or you can trigger a rebuild directly on travis.

I restarted one because of a cloud sdk installation failure. Lets see what happens.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #802   +/-   ##
=========================================
  Coverage     76.84%   76.84%
  Complexity      611      611
=========================================
  Files            98       98
  Lines          2418     2418
  Branches        314      314
=========================================
  Hits           1858     1858
  Misses          413      413
  Partials        147      147

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 8316f06...076121a. Read the comment docs.


Make ImStr#as_ptr public

This allows using ImStr in combination with functions (widgets) exported by imgui-sys.

Looks good! Thanks!
As a side note: is there a widget that is missing from the main crate that you would like to use?

Thanks! Yes, I am using igCombo2 (https://github.com/mneumann/hypernsga/blob/master/examples/graph_opt_gui.rs#L553) and igPlotLines2 (https://github.com/mneumann/hypernsga/blob/master/examples/graph_opt_gui.rs#L582). That's why I needed the public as_ptr() method.


Documentation fixes

Just some small observations I made. Cool project, will see if I can find time to contribute.

Don't think this is a problem for a small project like this but I was doing
some Linux kernel builds yesterday where I usually use nproc and I just
used -j and my machine deadlocked and needed to be pulled from the wall :/
On Sun, Apr 5, 2020 at 6:02 AM juj<EMAIL_ADDRESS>wrote:

@juj commented on this pull request.
In README.md
https://github.com/juj/fbcp-ili9341/pull/133#discussion_r403645654:

@@ -394,7 +394,7 @@ Enable the option #define DISPLAY_ROTATE_180_DEGREES in config.h. This shoul

Edit the file config.h in a text editor (a command line one such as pico, vim, nano, or SSH map the drive to your host), and find the appropriate line in the file. Add comment lines // in front of that text to disable the option, or remove the // characters to enable it.
-After having edited and saved the file, reissue make -j in the build directory and restart fbcp-ili9341.
+After having edited and saved the file, reissue make -j `nproc`  in the build directory and restart fbcp-ili9341.
Ok, let's go with just make -j then, no need to specify the nproc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/juj/fbcp-ili9341/pull/133#discussion_r403645654, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD7JRHDN4445RPF6PG5IZRTRK77FDANCNFSM4LTPNSPA
.


Implicit firstprivate generated when compiling with OpenACC

Since 23.3 (?) nvfortran writes Generating implicit firstprivate(...). Though this is not a problem, to be consistent with our "implicit is better than implicit" philosophy. It would be good to explicitly declare these with firstprivate attribute (even though the standard mentions that this is OpenACC's default for scalar values).

Let's let nvfortran generate these prompts for now, as we are doing things as expected from the OpenACC standard.


Resolve module location

Add spring.bus.location property

This will assign appropirate location for the module when its configuration properties are lookedup
for the module options etc.,

Exclude dependencies in boot repackage

Closing this one to re-issue a better fix.


CREATE  + toms connectors is bad lag maker

what ahpepsn is stuff 'like this'.   chute or smart shut above a flat inventory connector ( unfiltered).  and inventory is coming into the chute and into the connector to the storage network . but as soon as the spot is empty   toms sees the chute or funnel. or belt slot touching it.  as an empty inventory space and  other contraptions that are inputting to the storage network   filmlthose spots ( white ) which then dumps it to the toms. which then dumps it to the chute.. createing a loop of bad lag generating computes.. and with  create speeds. tis even worse ..
solution :  the big square connect all connected storage block of toms needs to have a filter slot  for adding either. a type of storage  item. (like chests).  or flat connector. and be able to set the flat connectors as input surfaces or output surfaces  ( can see into it . or just accepts  inputs).  or a separate input surface connector..
those could have filters too for multi block storage. that. can say how far it can see   or what types of storage it will accept to look into
spark report on the loop lag. causes a lot of ticking entities that don't need to tick .
connecting a crafting  or storage terminal. should not need an inventory connector behind it. or. the 'connect everything touching block)
also a netherite upgrade request for. the toms hoppers.   so you can move a stack at a time. please)

this also happens if you put a hopper pushing into the inventory connector flat
as a test point a hopper at an flat inventory connector and put a stack in.  you wills it go 64. 63. 64. 63 64 63. etc
these loops are bad lag creators..
if a hopper white funnel or other item mover like a belt is connected to a flat plate. it should become an input only system
if its pointed the other way.  ( sucking chute or sucking hopper top.  or a belt pulling out of a funnel.  then it should be an output plate only ..

toms hoppers could fix this if they connected to al late same storages  on its surfaces. but it does not and it needs to move stacks not one at a time.. ;
ie toms hoppers do not connect to chutes or hoppers. but do connect to belts and  not vaults

feature request..  paintable controller modules backgrounds with.  plank blocks or stripped logs..

if you puta toms block on a vault..
(block shaped storage connector connect all block)
and a funnel outputting to a depot.  . it creates a loop
or a funnel outputting and inputting into a belt.  it makes a loop  they all get treated as a multiblockstorage and  then lag happens.  spiking tps to the pits

I was trying to locate lag on our server so I installed Observable and found these inventory hoppers are making lots of ticks (shown in red)

I was trying to locate lag on our server so I installed Observable and found these inventory hoppers are making lots of ticks (shown in red)

I have the exact same problem Claudius. My server can handle Create Mod and a bunch of other wild stuff like Immersive Portals, but specifically, a single hopper from Tom's causes more lag than anything else.

It appears to be a loop because the way create looks at things like belts.
It looks at as a single storage and Tom will try to pull or push into that
and create a loop.
Thank you ,
Paul Cole
“Defeat is a state of mind. No one is ever defeated until defeat has been
accepted as reality. "
- Bruce Lee
On Thu, May 2, 2024 at 6:09 AM CleverBalloon @.***>
wrote:

I have the exact same problem Claudius. My server can handle Create Mod
and a bunch of other wild stuff like Immersive Portals, but specifically, a
single hopper from Tom's causes more lag than anything else.
—
Reply to this email directly, view it on GitHub
https://github.com/tom5454/Toms-Storage/issues/288#issuecomment-2090461439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQBDDKKICVO54SGHG7UDZLZAI3HHAVCNFSM6AAAAAA5R3G4EKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJQGQ3DCNBTHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update model setup file

I updated it to the current bridging model set and the current base model (2018.40).  The sensitivities will need to be updated after those are done.

and added in the last years model 00_45_2017base, all that is remaining is sensitivities after base model settled

this should be basically updated, pending additions/subtractions related to sensitivity models and SRG request models.  closing as no need to have as a reminder any longer

Awesome - thanks for running all the sensitivities.


updated build status badge

Yeah. currently the preview won't load on github. I think its because its not authenticated. Better the link being right though


overwrite layout component style no work

Environment
Nuxt Version: 3.0.0
Reproduction
https://stackblitz.com/edit/github-hcvfnh?file=components/MainHeader.vue
Describe the bug
i want to overwrite   style, but it is no work
<template>
  <NuxtLayout name="header" />
</template>

<style lang="scss" scoped>
.header {
  color: black;
}
</style>

Additional context
No response
Logs
No response

This does not work because scoped styles only work for the root component of the direct child. In this case, there are multiple internal components in between:


Change log level for missing user properties

Fixes #204
What has been done

Missing apiKey/user error message log level change from info to error (see https://github.com/bintray/gradle-bintray-plugin/commit/de1c3ced0349e77dd222371bd5a27b724203c889)
Ignored autogenerated project files
General code clean up (removed semicolons)

How to test

Set user or key to null in bintray closure:

bintray {
    user = null
    key = null
}

Execute ./gradlew clean build install bintrayUpload -DdryRun=true command and observe logs

Before the change
:dependencies-overview:build
:dependencies-overview:install
:dependencies-overview:bintrayUpload

BUILD SUCCESSFUL

After the change
:dependencies-overview:build
:dependencies-overview:install
:dependencies-overview:bintrayUpload
Skipping task 'dependencies-overview:bintrayUpload' because user or apiKey is null.

BUILD SUCCESSFUL

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Members who are in a private debate are unmuted when it's concluded.

When a private debate is concluded, everyone is un-muted even though they shouldn't be.

This was fixed in  5424a04.


Change redispipe package to go-redis for state management

Describe the proposal
The go-redis package is a common and highly used package for interacting with Redis for Go. It's used by Redis for Pub/Sub, while the Redis State Store uses redispipe, which favors implicit pipelining but does not support things like TLS.
I propose to standardize on go-redis as the package for Redis interactions in components-contrib.

Closing as its tracked by https://github.com/dapr/components-contrib/issues/213


find_linked join with table alias

Resolve #163

Can we have a more comprehensive test case with more levels of joins?

Added some more test cases 1a61f09

Can you move the Link definitions outside of the Entity files? perhaps to a separate file called entity_links?
I feel like it is cluttering the Entity file a bit

Yes we can!


Update README.md

Removed Demos and Possibilities under Tutorials.

Great!!!

Hi
Can you accept the pull request please?


Add compatibility macros for dbt-audit-helper

Motivation
Fixes #19. Will push a new tag once it's approved!
Tips for reviewer
Verified that the integrations tests run successfully for dbt-audit-helper.

Can you update CI to run these tests too?

Ah, thanks! I missed that.

In theory, we could instead create a new MaterializeInc/materialize-dbt-audit repository for these shims. Does dbt have guidance on which option to choose?

I based this on the tsql-utils example, which sounds a little saner than having a separate repo for each package we might want to shim. I'll double check with Anders, though!

Thanks, @benesch! The macros are mostly copied from the Postgres implementation, with the exception of a slight twist in compare_column_values.sql and compare_queries.sql to work around an unsupported (window) function that I replaced with a harmless hack (if such a thing exists 👹). Added a note to both so we know it's there, though!


Update and rename channels/ae.m3u to  channels/ae.m3u

I don't get it, you just replaced the EPG url with a colon. What for?


Filter by NULL

How to filter by NULL on a GET request?
If I make a query without the equal sign the parser even finds out the right parameters hash, but the filter code ignores it!
Started GET "/api/v2/sales?filter%5BclosedAt%5D" for ::1 at 2015-07-18 17:42:22 -0300
Processing by Api::V2::SalesController#index as API_JSON
  Parameters: {"filter"=>{"closedAt"=>nil}}
  Sale Load (0.6ms)  SELECT "sales".* FROM "sales" WHERE 1=0  ORDER BY nfe desc, updated_at desc, "sales"."id" ASC
Completed 200 OK in 4ms (Views: 0.2ms | ActiveRecord: 0.6ms)

If you specify an empty filter the find method does not by default assume you are looking for null values. There are several options you could take. To make an empty search for relationships find null values you could override the verify_relationship_filter
def self.verify_relationship_filter(filter, raw, _context = nil)
  if raw.is_a?(Array) && raw.length == 0
    return [filter, nil]
  else
    super
  end
end

You could also override the apply_filter method to implement a keyword that indicates a search for null, such as 'none':
def self.apply_filter(records, filter, value, options)
  if value == 'none' || (value.is_a?(Array) && value[0] == 'none')
    records.where(filter => nil)
  else
    return super
  end
end

Of course you may only want that filter to work on select fields. You could then check the filter as well.

Great! For searches of strings should I override apply_filter or find? Who’s that?

@henrymazza I think apply_filter is easier implement since it's setup to let you modify each filter. But the same thing could be done by overriding find, however it's probably more complicated and would be overkill in most situations.
For your string searches you could do this (not tested code, so there are possible errors):
def apply_filter(records, filter, value, _options = {})
  t = _model_class.arel_table
  records.where(t[:filter].matches("%#{value}%"))
end

That should make all filters do a case insensitive like search. You should probably conditionally add the text search on just some fields by looking at filter. The same could be done in find, but it's going to be more work, and won't work any better.

Works great (if you delete the colon :smile_cat:)!
apply_filter is really growing here. It’s checking for NULL, if query is asking for a different scope and now searching. It’s probably the main method to look after when porting an old rails app. I was asking for more switches, but now I’m felling home.

Great, glad it mostly worked! Also, I cleaned up the colon in the example.

When @top_level_meta_include_record_count=false should it still make the second query to database asking for the counting? It’s breaking me when I include some fancy stuff on select clausule because Postgres wants it there (order by similarity).
If it’s really a bug I can open another issue.

Sounds like a bug to me. It shouldn't be running the count query if you don't ask for the count.

Ok. Here it is: https://github.com/cerebris/jsonapi-resources/issues/335

ActiveRecord don't like to filter by empty array. it use WHEN 1=0 condition to show it.
we could sort by NULL like this:
def apply_filter(records, filter, value, _options = {})
  value = value.is_a?(Array) && value.empty? ? nil : value
  super(records, filter, value, _options)
end

@lgebhardt Regarding the original topic of this issue (filtering with NULL as the value), now that overriding apply_filter is deprecated, is there a new preferred way to do this?

@grantovich I think something like this would work:
filter :open, apply: ->(records, value, _options) {
  if value == 'none'
    records.where('date_resolved' => nil)
  else
    return records.where.not('date_resolved' => nil)
  end
}

And you can still override apply_filter, it's just cleaner to use a callable on the filter definition.

Thanks @lgebhardt, in fact I already implemented a solution similar to that – what I'd like to know is whether there's a good non-deprecated way to permit the use of 'none' on all filters without having to configure it on each new filter. This is what the old apply_filter override enabled, I think.
From the README:

Note: It is also possible to override the self.apply_filter method, though this approach is now deprecated

It's deprecated only in the sense that for most use cases it's cleaner to handle it with a callable so you don't have to also switch on the filter. In your case for wanting none for all filters I would override the apply_filter method. I can't promise we won't change it, but I don't see it as likely that we will either.

Okay 👍 Usually when I see "deprecated" I read it as "this will definitely be removed in a future release, so don't write any new code that depends on it". It sounds like overriding apply_filter is merely "not recommended" at this point, rather than officially "deprecated". 😃

@grantovich In this case deprecated was meant to convey that there is a better way to do something, though I can certainly see how you might be worried about removal as that is a common reason to deprecate a feature. I'll update the docs soon to clarify.
https://en.wikipedia.org/wiki/Deprecation


Clarification on behavior for duplicate parent refs for routes.

What would you like to be added:
An additional conformance test specifying the behavior when a gateway has only a single listener and an http route specifies the gateway twice: once in specifying the gateway with the listener and once without specifying the specific listener. According to the docs this should result in an invalid route configuration.
This feels like a potentially uncovered edge case in the docs as it is neither two identical parents (as could be detected by the webhook) nor two distinct sections, and might be a valid configuration if the Gateway had a second listener which could match the parentRef not specifying a listener name.
Why this is needed:
To clarify the above behavior for implementers  of the gateway spec.

Sounds good, lined this up for our next release (though it may flood over into v0.8.0 ultimately) and if there's a contributor that would like to jump in and help with this one it would be very appreciated!
/help

I wouldn't mind writing the tests themselves, more looking for clarification on what the expected behavior is here.

Awesome, appreciate the interest! For extra clarity, can you provide an example YAML configuration of the currently ambiguous situation?

hey @shaneutt sorry this took a little bit to get back to you on, below is a yaml configuration that shows the ambiguous situation:
apiVersion: gateway.networking.k8s.io/v1beta1
kind: Gateway
metadata:
  name: apigw
spec:
  gatewayClassName: api-gateway
  listeners:
  - name: http-listener
    protocol: HTTP
    port: 30002
    hostname: '*.example.com'
    allowedRoutes:
      namespaces:
        from: All
---
apiVersion: gateway.networking.k8s.io/v1beta1
kind: HTTPRoute
metadata:
  name: http-route
spec:
  hostnames:
    - ref.example.com
  parentRefs:
    - group: gateway.networking.k8s.io
      kind: Gateway
      name: apigw
    - group: gateway.networking.k8s.io
      kind: Gateway
      name: apigw
      sectionName: http-listener
  rules:
    - backendRefs:
      - kind: Service
        name: some-service
        port: 80
        weight: 1
      matches:
      - path:
          type: PathPrefix
          value: /

so this HTTPRoute references the same gateway twice, but specifies the listener in one parent ref and does not specify it in the other ref. This could be a potentially valid situation if the gateway had more than one listener.

Thank you for the illustration. Yes this seems like something we would want to specify so let's do this:
/triage accepted
We'll need whoever takes this on to take some time to build consensus among the community about how to resolve it. It would probably be good to sort it out prior to GA:
/kind conformance
/milestone v1.0.0
/help

@robscott is going to create a follow-up issue for this to create a conformance test to resolve this.

I haven not forgotten about this one, waiting for clarification on https://github.com/kubernetes-sigs/gateway-api/issues/2326 before writing up what we'd expect a conformance test to look like for this.

I talked with @sunjayBhatia about this one and he can take this one on.
/assign @sunjayBhatia
We expect resolving #1626 completes most of this.
Thanks @sunjayBhatia !

Looks like https://github.com/kubernetes-sigs/gateway-api/pull/2350 covers this issue? I refrained from making any changes regarding this in https://github.com/kubernetes-sigs/gateway-api/pull/2396 since that PR already existed

Ah sorry, yes it does, I should have taken this issue.

No worries! I can assign it over to you since you've got that in progress already (if I have powers to do so)
Generally looks like this has a lot of overlap with https://github.com/kubernetes-sigs/gateway-api/issues/2326 so probably best resolved together?
/assign @youngnick


What should happen every day?

How do we make this minimal commitment strategic?
What if it was a challenge for everyone?

This was an idea that came up at Axe-Con, I didn't want to forget. But I forgot LOL


Update Windows GUID to SDL uid conversion.

Should fix #38869. However, since I don't have controllers to test this with, I need help with testing.
cc @hoontee @RonanZe
Note: This is specific to Windows. I've checked the Linux version and this does not appear to need to be updated. The other OSes don't appear to use the SDL mapping database.

For the reference if any Windows user wants to help testing, a Windows build for this PR is available on AppVeyor: https://ci.appveyor.com/project/akien-mga/godot/builds/33120494/artifacts

For the reference if any Windows user wants to help testing, a Windows build for this PR is available on AppVeyor: https://ci.appveyor.com/project/akien-mga/godot/builds/33120494/artifacts

I would but is it Godot 4.0 master? And is the code compatible with 3.2?

That's the master branch yeah, so it's not guaranteed to work well for everyone yet.
You can try it safely nevertheless, but do so on a new branch and not on one of your 3.2.x projects.

All good for me.
I've test the Windows build with a DualShock4 and a WiiU Pro.
Every Inputs matches, the ID and Name are corresponding to the gamecontrollerdb.txt
note: I have "XInput" for the ID and Name of the DualShock4 but I think it's how it works with the driver I use

Godot:
print(Input.get_joy_name(0)) --> "XInput Gamepad"
print(Input.get_joy_guid(0)) --> "__XINPUT_DEVICE__"

Buttons seem to still be mapped correctly.
SDL2 Gamepad Tool:
"XInput Controller", 030000005e040000ff02000000007801 (mapping available)
Environment variable "SDL_GAMECONTROLLERCONFIG" is not defined

@hoontee Thanks for your help. I'm confused. When you say "Buttons seem to still be mapped correctly" what are you using to determine the correct mapping?
The reason I'm confused is that the mapping that Godot is currently using for XInput devices is very different to the generic XInput mapping that SDL provides:
Godot:

__XINPUT_DEVICE__,XInput Gamepad,
a:b12,b:b13,x:b14,y:b15,start:b4,back:b5,leftstick:b6,rightstick:b7,leftshoulder:b8,rightshoulder:b9,dpup:b0,dpdown:b1,dpleft:b2,dpright:b3,leftx:a0,lefty:a1,rightx:a2,righty:a3,lefttrigger:a4,righttrigger:a5,platform:Windows,

SDL:

xinput,XInput Controller,
a:b0,b:b1,back:b6,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,dpup:h0.1,guide:b10,leftshoulder:b4,leftstick:b8,lefttrigger:a2,leftx:a0,lefty:a1,rightshoulder:b5,rightstick:b9,righttrigger:a5,rightx:a3,righty:a4,start:b7,x:b2,y:b3,platform:Windows,

Whereas the SDL generic XInput mapping is almost the same as the mapping for your GUID matched controller:

030000005e040000ff02000000000000,Controller (Xbox One For Windows) - Wired,
a:b0,b:b1,back:b6,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,dpup:h0.1,leftshoulder:b4,leftstick:b8,lefttrigger:+a2,leftx:a0,lefty:a1,rightshoulder:b5,rightstick:b9,righttrigger:-a2,rightx:a3,righty:a4,start:b7,x:b2,y:b3,platform:Windows,

Using Input.is_action_just_pressed I was able to determine that all joypad buttons work - except for L2 and R2, which are missing from the list and aren't sending InputEventJoypadButton events.

@hoontee Thanks again for your help.

except for L2 and R2, which are missing from the list and aren't sending InputEventJoypadButton events. They are however sending InputEventJoypadMotion events.

This is correct. L2 and R2 are (half) axis controls and not buttons. They send signals between 0 and 1. This is a change in 4.0 provided by #38151, which provided the required support for half and inverted axes.

Thanks!

Cherry-picked for 3.2.2.

Adding that this or other recent joypad commits fixed a strange issue I was having.  With a game that started in full screen and switched to window mode, if a joypad was connected, it would constantly report that it's right analog stick was pointing almost directly and fully to the right.  When minimizing the window it would stop doing this and return to normal.


python310Packages.angr: 9.2.36 -> 9.2.37

Description of changes
Bump angr stack to 9.2.37

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, as applicable:

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

[ ] 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/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

16 packages built:

python310Packages.ailment
python310Packages.angr
python310Packages.angrcli
python310Packages.angrop
python310Packages.archinfo
python310Packages.claripy
python310Packages.cle
python310Packages.pyvex
python311Packages.ailment
python311Packages.angr
python311Packages.angrcli
python311Packages.angrop
python311Packages.archinfo
python311Packages.claripy
python311Packages.cle
python311Packages.pyvex

Result of nixpkgs-review pr 215280 run on aarch64-darwin 1

2 packages failed to build:

python310Packages.angrcli
python311Packages.angrcli

14 packages built:

python310Packages.ailment
python310Packages.angr
python310Packages.angrop
python310Packages.archinfo
python310Packages.claripy
python310Packages.cle
python310Packages.pyvex
python311Packages.ailment
python311Packages.angr
python311Packages.angrop
python311Packages.archinfo
python311Packages.claripy
python311Packages.cle
python311Packages.pyvex

@ofborg build python311Packages.pyvex


Add max LogEnum level of logging per channel

Each channel should log messages at or below their max LogEnum. Additionally, this should be set for modulations as well.
E.g. channel has max level set to INFO,
-> Error, Warning, and Info messages will be passed to the modulation objects, but no Debugs.
Within this channel, I might have 2 modulations: TerminalModulation and FileModulation.
File modulation might have max level --> Info. E.G. any message of level info or below will be logged.
TerminalModulation  might have max level --> Error. E.g. Only error messages will be logged.

Closed in e456521


Array buffer allocation failed

RangeError: Array buffer allocation failed
at new ArrayBuffer (<anonymous>)
at new Uint8Array (<anonymous>)
at v.serialize in src/vs/editor/common/model/editStack.ts:101:13
at S.close in src/vs/editor/common/model/editStack.ts:200:28
at g.pushStackElement in src/vs/editor/common/model/editStack.ts:396:16
at Mn.pushStackElement in src/vs/editor/common/model/textModel.ts:1191:24
at g.updateModel in src/vs/editor/common/services/modelService.ts:439:9
at Ps.updateTextEditorModel in src/vs/workbench/common/editor/textEditorModel.ts:209:22
at Ps.Db in src/vs/workbench/services/textfile/common/textFileEditorModel.ts:559:9
at Ps.Bb in src/vs/workbench/services/textfile/common/textFileEditorModel.ts:521:9
at Ps.Ab in src/vs/workbench/services/textfile/common/textFileEditorModel.ts:452:16
at async l.P in src/vs/workbench/services/textfile/common/textFileEditorModelManager.ts:416:4
at async l.O in src/vs/workbench/services/textfile/common/textFileEditorModelManager.ts:322:3
at async Object.factory in src/vs/workbench/services/textfile/common/textFileEditorModelManager.ts:166:6

Go to Errors Site

Issue hit over 100 times on 1.91. Corresponding issue will be marked as triaged.


Jeff Schaller: Watch cognitivebehaviourtherapytoronto.com

Jeff Schaller requests the watch of the watch_keyword cognitivebehaviourtherapytoronto\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

cognitivebehaviourtherapytoronto\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by double-beep in Charcoal HQ


Unable to Install

I get this error, any idea what's wrong?
 sudo npm install -g bottr-cli
npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "-g" "bottr-cli"
npm ERR! node v4.2.6
npm ERR! npm  v3.10.8

npm ERR! uid must be an unsigned int
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>
npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "-g" "bottr-cli"
npm ERR! node v4.2.6
npm ERR! npm  v3.10.8

npm ERR! uid must be an unsigned int
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>
npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "-g" "bottr-cli"
npm ERR! node v4.2.6
npm ERR! npm  v3.10.8

npm ERR! uid must be an unsigned int
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/name/npm-debug.log

My node and npm versions.
 node -v
v4.2.6
npm -v
3.10.8

@ahmadawais  I found this Issue on the NPM Github Page https://github.com/npm/npm/issues/13946 - Let me know if this helps

@ahmadawais Have you fixed this issue as per @jcampbell05 suggestion?

@ummahusla Nops!
I have to many dependencies running on Node, current projects, that I cannot afford to spend a day or two on debugging things if I reinstall Node. Which is why I am not going to try it out right now.

@ahmadawais Okay, please raise the issue again, if after time it still appears.


URL from Microsoft AI Lab Project: Synthetic Data Generator not correct

Hi guys, just for your information the page for Microsoft AI Lab Project: Synthetic Data Generator is using an URL (for the button "try the demo") which is not correct.
It's pointing to https://synthetic-data-showcase.azurewebsites.net/ and I guess it should be https://microsoft.github.io/synthetic-data-showcase/ as given in #readme

Thanks for letting us know about the broken link @rabbyn. I will look into updating the link on the AI Lab page.


Added new test for debugging

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+9.6%) to 82.517%

Files with Coverage Reduction
New Missed Lines
%

src/DevOpsMetrics.Core/Models/Common/Build.cs
3
90.77%

Totals

Change from base Build<PHONE_NUMBER>:
9.6%

Covered Lines:
2596

Relevant Lines:
3091

💛  - Coveralls


Add support for GHCJS

This provides a (AFAICT) fully working GHCJS package set.
@peti Can I get your feedback on how to best merge this in? I'm sure there are some things to consider, like if/how we should specify that a given package supports GHCJS (or more generically, any particular compiler).

I'm running test builds at http://hydra.cryp.to/jobset/nixpkgs/haskell-updates ...

I'm running test builds at http://hydra.cryp.to/jobset/nixpkgs/haskell-updates ...

@cstrahan, could you please rebase this branch relative to master and force push the update?

@cstrahan, could you please rebase this branch relative to master and force push the update?

Merged in 616bf50. @cstrahan, if you have the chance, you should post an announcement to the nix-dev mailing list to let people know about this new feature!

Merged in 616bf50. @cstrahan, if you have the chance, you should post an announcement to the nix-dev mailing list to let people know about this new feature!


Internatiolization

Apply i18n.

i can help with french

Hey @Abdur-rahmaanJ.

i can help with french

That would be great!
I think we should start by initialising a translation tool, marking all string messages and creating the POT file. This would be all for this issue. Then a new issue to add French support could be created.
PS.: By translation tool I mean making use of gettext module that I'm familiar with, but if you or anyone else knows a better Python library to deal with i18n I'm all ears.

@julianolf see polib, fairly straightforward

Guys, I've sent a suggestion for the i18n implementation using the polib library. Any feedback is welcome. 🙂

Well, as the PR #148 added i18n support and the PR #151 added portuguese translation as an example of translation I believe everything required by this issue has been done, so I'm going to close it.


Add author pages

This addresses issue #21

Sorry, should have put these in separate branches. Latest commits should close #24


Simplify installer

This matches the structure that Solid Queue now uses (after https://github.com/rails/solid_queue/pull/321) and Solid Cache now uses (after https://github.com/rails/solid_cache/pull/202)

@npezza93: Thanks. Updated


[5367] bulk update csv downloads show UTC not BST

Context
When you download a CSV at the start of the bulk recommend journey, it adds a timestamp to the filename, but it’s in UTC..
Make it show the correct (London timezone) time.
This is probably incorrect for all report exports.
Also fix it for the bulk recommend error CSV export (when you re-download the CSV to review errors).
Changes proposed in this pull request
Label for report time stamp now uses .in_time_zone("London")

 [S] Bulk update - CSV export filenames are UTC not BST

There's also the timestamp in app/controllers/bulk_update/recommendations_errors_controller.rb for the CSV with errors.

done


ArgumentError in tutorial on Hidden Markov Models

I tried running the tutorial on Hidden Markov Models but encountered ERROR: ArgumentError: index m not found when running the following lines:
# Extract our m and s parameters from the chain.
m_set = c[:m].value.data
s_set = c[:s].value.data

I am not sure how this can be fixed, i.e. how does one access a set of parameters that belong to the same array in a chain object?

You can use namesingroup or group, see https://github.com/TuringLang/MCMCChains.jl#groups-of-parameters. I guess it is also more natural to use Array to obtain an array ouput, .value is an implementation detail (see https://github.com/TuringLang/MCMCChains.jl#exporting-chains).

Thanks, the following works:
m_set = Array(group(c, :m))
s_set = Array(group(c, :s))

There is still an issue with the plotting, but I think this will now generally be addressed in TuringLang/TuringTutorials#86.


merge: dev to master

only refactoring, no release

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


1365 webapp changes for netcdf res

This PR is to address #1365:

a new url pattern term "HS_NETCDF_FILE_NAME" is added, which represents the full filename of the .nc file in a NetCDF res.

A new metadata term is added into WebApp called "SupportedSharingStatus", which is a new criteria to associate a webapp with certain res types in certain sharing status. Example: if a webapp supports NetCDF res type and its "SupportedSharingStatus" value is set to "Public", this webapp will only show up on Public NetCDF res landing page.

some minor UI changes and bug fixes (see #1371 )

A live instance is deployed on https://riverdev.hydroshare.org/

+1  @zhiyuli Merge after Jenkins check is successful.


googleapi: Error 403: The client is not authorized to make this request., notAuthorized

Hi,
I try to connect to my cloudsql instance using the Proxy Docker Image, but it fails on authorisation issue, and could use some help in debugging the issue.
All seems to be configured right when pod is deployed, when i look in the logs it states:
2017/08/17 15:24:33 using credential file for authentication<EMAIL_ADDRESS>2017/08/17 15:24:33 Listening on <IP_ADDRESS>:3306 for [project-id:region:instance-name]
2017/08/17 15:24:33 Ready for new connections

but it keeps failing when I open a connection with the message
couldn't connect to "[project-id:region:instance-name]": ensure that the account has access to "[project-id:region:instance-name]" (and make sure there's no typo in that name).
Error during createEphemeral for [project-id:region:instance-name]: googleapi: Error 403: The client is not authorized to make this request., notAuthorized

However if I use the cloud_sql_proxy from my local machine all works like a charm:
./cloud_sql_proxy -instances=project-id:region:instance-name=tcp:3307 -credential_file=keys/service-account-key.json

I have als tried resetting the roles on the service-account as suggested in issue #37 but that did not help, removing the roles made the ./cloud_sql_proxy also fail on notAuthorized, and adding them enabled it but the docker image proxy still is failing.
So, same service account, same project-id, same cloudsql instance, just the location differs. Any idea where to start debugging this?

Hi,
Would you please send the name of your cloud sql instance to
<EMAIL_ADDRESS>(with a link to this page) so I may be able to help out
looking at some internal data.
Also, please paste the full command line you run both with and without Proxy Docker Image, and the logs when connection.

Did you follow the URL in the error log to enable the API?

@Carrotman42
actually I havn't.. .
I enabled it, recreated credentials - just incase..
now I get the following error:
couldn't connect to "work-capital:us-central1:bank": ensure that the account has access to "xxxx:us-central1:myinstance" (and make sure there's no typo in that name). Error during createEphemeral for xxxxx:us-central1:myinstance: googleapi: Error 403: The client is not authorized to make this request., notAuthorized

I'm seeing the same issue with my instances. What should be the next steps?

Actually I found in some of the cases there's an additional square bracket around the instance name in the pod command ran.  Please remove the bracket and try again.  Especially for the ones work with cloud_sql_proxy but not from the container.
If yours is not the case, then:
Please send an email with following information to<EMAIL_ADDRESS>so I may be able to help out looking at some internal data.

with a link to this page, particular if you have left a message here, link to the message.
describe the issue you have if the link doesn't cover all the details.
the full name of your cloud sql instance.
the command your run
the full logs

Yup, me too - same error but getting it from local machine

It was indeed the extra square brackets in my case, thanks alot for pointing that one out!

I have exactly the same issue.


Typing snippets are inserted in incorrect Position

Since latest vscode update(and previously vscode-insiders) I started experiencing typing snippets are inserted in incorrect Position. (this even made me use default vscode as I just can't write code  without this extension now)
Example:
https://user-images.githubusercontent.com/74474615/216970854-62b7c236-35ed-4eb4-a99f-1dc26ff9596e.mp4
Didn't dive into it but looks like smth has changed with the editor.insertSnippet() behaviour. Gonna propose quick solution in a minute

this even made me use default vscode

What do you mean ?

this even made me use default vscode

What do you mean ?

I mean this didn't reproduce in vscode-stable before 1.75, so I used it instead of code-insiders as correct work of your extension is essential for me. I mentioned it because I saw you are primarily using code-insiders on your machine, so you maybe already did some workarounds.


Add FileIO support

Codecov Report

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

@@             Coverage Diff             @@
##            master       #7      +/-   ##
===========================================
- Coverage   100.00%   97.22%   -2.78%
===========================================
  Files            1        1
  Lines           35       36       +1
===========================================
  Hits            35       35
- Misses           0        1       +1

Impacted Files
Coverage Δ

src/KMLTracks.jl
97.22% <0.00%> (-2.78%)
: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 52d4819...ff6f151. Read the comment docs.

See also: https://github.com/JuliaIO/FileIO.jl/pull/262

Closes #6


数据导入及个别样式优化

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Custom OAuth Reponse

There seems to be a need for allowing users to hook into the package to generate custom response formats.
The default response format is hard-coded into the package in 'issueAccessToken' method of 'AuthorizationServerProxy' class.
I have forked the repo & am trying to implement a custom interface for handling responses whose implementation class can be indicated by user in oauth2.php config file.
Will be back with results & probably a pull request.
Edit: Also, OAuthFilter must also respect the new response interface provided by user.

Create an middleware and call it in $middleware array like this
public function handle($request, Closure $next)
    {
        try {
            $response = $next($request);
            // Was an exception thrown? If so and available catch in our middleware
            if (isset($response->exception) && $response->exception) {
                throw $response->exception;
            }
            return $response;
        } catch (OAuthException $e) {
            $data = [
//                'error' => $e->errorType,
//                'error_description' => $e->getMessage(),
                'error' => 'Custom Error',
                'error_description' => 'Custom Description',
            ];
            return \Response::json($data, $e->httpStatusCode, $e->getHttpHeaders());
        }
    }


Add .editorConfig file

This introduce .editorConfig file for common options & specifically our naming style.
.editorConfig is supported by R# & VS both. It overrides user-defined settings.
part of #9911

You should run a code cleanup before and after these changes.

@AndriySvyryd @anpete @bricelam @maumar Can each of you look at this change and the associated change to the output of Code Cleanup (https://github.com/aspnet/EntityFrameworkCore/pull/10328) and comment if you have issues. If there are no issues, then please sign-off on this PR and we'll move to these settings.

Just a note: #10328 is code clean up before this change.
As of now editorConfig is VS does not have a way to do a "cleanup" but it suggests in editor that you are violating a rule.
If anyone wants to test this file with R# then feel free.

Updated.

Can I get sign-off on this?


Wait for layer file processing to complete before proceeding

fixes #734
In the Has Modified Files container check, we start up a goroutine to receive paths for files contained in a layer via a channel, and once that channel closes, we append that layer's files to a slice ([][]string) representing each layer's files.
In certain cases (seems like larger layers?), the untar function would close out the channel, causing the goroutine to start the process of appending that layer's files to the mapping. In some cases, that append does not take place fast enough, and so the rest of the logic in the check evaluates the list of files for layer 0 and comes up empty, causing us to panic.
This PR adds a waitgroup and forces us to wait for the goroutine to complete before we continue processing the check.
CC @imperialguy thanks for reporting!
Signed-off-by: Jose R. Gonzalez<EMAIL_ADDRESS>

Coverage decreased (-0.1%) to 83.855% when pulling 7c29b20aec537b7cc51620e69dd609af480d60b1 on komish:fix-race-in-mod-files into aef70ad78816bd5b17b606b28949c1b619dc0995 on redhat-openshift-ecosystem:main.

coverage: 83.541% (-0.5%) from 83.992%
when pulling 7c29b20aec537b7cc51620e69dd609af480d60b1 on komish:fix-race-in-mod-files
into aef70ad78816bd5b17b606b28949c1b619dc0995 on redhat-openshift-ecosystem:main.


set-output= command is getting deprecated

https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Actions now warn about this:

Looks like it's used in two places:

https://github.com/DamianReeves/write-file-action/blob/e19fd875ed54f16fc583a3486e62547ce4a5dde8/src/index.ts#L29
https://github.com/DamianReeves/write-file-action/blob/e19fd875ed54f16fc583a3486e62547ce4a5dde8/src/index.ts#L44

I don't know the purpose of these lines to know if removing them will change the behaviour or what the appropriate replacement would be.

I think @actions/core is at 1.10.0 now which should help with this error.

Fixed in v1.2


Clear the container at the end of the DefaultExecutorRepository#destroyAll method

Clear the container at the end of the DefaultExecutorRepository#destroyAll method
Reuse destroyRegistered variable

Codecov Report

Merging #7820 (ede388d) into master (e845e16) will decrease coverage by 0.18%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #7820      +/-   ##
============================================
- Coverage     59.28%   59.10%   -0.19%
- Complexity      529      555      +26
============================================
  Files          1077     1086       +9
  Lines         43565    43754     +189
  Branches       6368     6358      -10
============================================
+ Hits          25826    25859      +33
- Misses        14879    15027     +148
- Partials       2860     2868       +8

Impacted Files
Coverage Δ
Complexity Δ

.../threadpool/manager/DefaultExecutorRepository.java
80.95% <100.00%> (-4.54%)
0.00 <0.00> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
76.66% <100.00%> (-1.49%)
0.00 <0.00> (ø)

...a/org/apache/dubbo/rpc/filter/AccessLogFilter.java
28.35% <0.00%> (-50.75%)
0.00% <0.00%> (ø%)

...va/org/apache/dubbo/rpc/support/AccessLogData.java
53.16% <0.00%> (-37.98%)
0.00% <0.00%> (ø%)

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
0.00% <0.00%> (ø%)

...ng/transport/dispatcher/WrappedChannelHandler.java
43.47% <0.00%> (-19.57%)
0.00% <0.00%> (ø%)

...bo/rpc/cluster/support/FailbackClusterInvoker.java
52.45% <0.00%> (-18.04%)
0.00% <0.00%> (ø%)

...rg/apache/dubbo/remoting/utils/PayloadDropper.java
76.92% <0.00%> (-15.39%)
0.00% <0.00%> (ø%)

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
0.00% <0.00%> (ø%)

...in/java/org/apache/dubbo/common/utils/JVMUtil.java
81.13% <0.00%> (-11.33%)
0.00% <0.00%> (ø%)

... and 97 more

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 e845e16...ede388d. Read the comment docs.


gtm_auth - where to get this?

Do I need to replace the gtm_auth: 'AB7cDEf3GHIjkl-MnOP8qr'?
If so, where do I get this?

When you create additional environments in GTM, these 3 query parameters are added to the snippet.
If you're just using the default environment (Live, I assume), then you don't need to set these values. If you still want to specify them for whatever reason, then you should be able to find the values in the Environments section (in the provided snippets).

@glen-84 Good to know.
If you want, please open a PR an add your comment as documentation to e.g. the README.md
Please only use the main branch for this. I will keep track of the extra vue2-gtm branch by myself.


Set long description type to markdown for pypi.

Fixes #12

Fixed in pre-release version: https://pypi.org/project/duet-async/0.1.1.dev202102110857/.


Drop the Following feed settings onboarding step

I think this is enough to remove it.

This can probably be closed now as it was removed in #4224.


Update resolveHostname to match Vite 3

Describe the problem
The resolveHostname function has slight mismatch over-time.
Describe the proposed solution
Update it when Vite 3 is near stable. May want to watch out for https://github.com/vitejs/vite/issues/7075
Alternatives considered

Importance
nice to have

do we need to detect vite version and have it work for both or upgrade peer dep to 3 and people have to stick with the old version for 2?

https://github.com/vitejs/vite/pull/8986 would make it a lot easier to handle.


Organization of Maxwell Fundamentals

From @thast in #284.
Also, Do we want to put some thoughts about the organisation of Maxwell Fundamental? Few examples:

It is indeed not clear where to look to find the dipoles for example. They are right now in Maxwell Equation/Maxwell’s Equations in Homogeneous Media (so there are not referred as dipole). But there is also a section Analytic in Solving Maxwell Equation.

The section Quick Guide to Maxwell’s Equations would do a great introduction page to the Maxwell's equation section in my opinion. It seems light compared to other same-level sections. Does it deserve its very own category in the index?

@lheagy @yangdikun @dougoldenburg : some thoughts about it?

Moving the Quick Guide to Maxwell's Equations earlier deserves consideration - it could be a good "grounding point": you see all of the equations, then start diving in and unpacking them.
The organization of dipoles and plane waves are being discussed in #314, #315 : so we can focus that discission there.

This is being discussed in further detail in #346


Add DPG: MERON

Add DPG: MERON (Methods for Extremely Rapid Observation of Nutritional Status)

Bisa Health initially submitted with several gaps in the DPG standard indicators which need to be addressed before we can proceed with the review:

[ ] Clear Ownership
[ ] Documentation
[ ] Adherence to Standards & Best Practices
[ ] Adherence to Privacy and Applicable Laws
[ ] Do No Harm
[ ] Protection from Harassment

I have contacted the person who made the submission and will update once they come through.


better bpf logging

What type of PR is this?

What this PR does / why we need it:
This is the first step to provide more readable and meaningful log.
The following work should to design some Format String to represent svc id, etc. And to do efficient replace in userspace.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

nit:
why
 BPF_LOG(
            ERR, SERVICE, "locality loadbalance match nothing in STRICT mode, service_id %d ret:%d\n", service_id, ret);

not
 BPF_LOG(ERR, SERVICE, "locality loadbalance match nothing in STRICT mode, service_id %d ret:%d\n", service_id, ret);

It is formatted by clang

/lgtm
/approve


Migrate to NotChunkVector (Draft)

Roadmap: https://github.com/skyzh/tikv/issues/2

Highlights of these changes:

remove some functions for better testing, add rpn_fn support for Option<&T> as in https://github.com/skyzh/tikv/pull/3 .
implement Into<Option<&T>> for ScalarValueRef. If we need to get Option<&T> from ScalarValueRef, we need to call svref.clone().into().
refactor related code

Highlights of above changes:

code now compile successfully
rpn_fn now handles vargs and normal-with-context correctly
some tests still don't pass

Failed attempts to migrate AsRef: If we use 3 traits for as_ref, there will be ambiguous in compile error.


Umount failed by Alluxio FUSE

Alluxio Version:
2.0
Describe the bug
Umount Alluxio FUSE mount point failed, while there is no other process occupy this mount point.
\# umount /mnt/alluxio-fuse/
umount: /mnt/alluxio-fuse: target is busy.
        (In some cases useful info about processes that use
         the device is found by lsof(8) or fuser(1))

To Reproduce
loop execution:
alluxio-fuse mount -o allow_other /mnt/alluxio-fuse/ /
touch /mnt/alluxio-fuse/mount-test/mount-test
umount /mnt/alluxio-fuse/

any finding from kernel level logs on why the mount point was busy, @liuhongtong ?

Hi @liuhongtong, to unmount Alluxio fuse, the expected way is
alluxio-fuse umount /mnt/alluxio-fuse/ which will kill the alluxio fuse client process.
is there any reason you have to use system umount rather than alluxio-fuse umount?

@apc999 umount syscall need add reference of the mount point. umount syscall may success only when the reference is 2, while the default reference is 1. If another umount executed, the reference will be 3, then all the two umount will fail.

@apc999 alluxio-fuse umount is the right way. But FUSE is one file system of Linux, so Alluxio FUSE should support umount command as well.


Add e2e test for pushing empty list of rules

This should match the behavior from rules-objstore, which is being tested in https://github.com/observatorium/rules-objstore/pull/15.

@saswatamcode done, sir.


Sync price plans and rate limits

Depends on:

https://github.com/CartoDB/cartodb-platform/pull/4150
https://github.com/CartoDB/cartodb-central/issues/2134

:rocket:


Analyze current "Case Log" query

The existing "Case Log" query on the main IAIP navigation screen was showing too many open cases during the demo overview.
This was discussed in discussion https://github.com/gaepdit/sbeap/discussions/9

Reviewed this again with the program. The unexpected additional open cases were assigned to former staff members and never closed. They should be administratively closed during the migration.  See https://github.com/gaepdit/sbeap/issues/27#issuecomment-1560074781 for follow-up.


CSS Support

Could you please add support for CSS based on Idiomatic CSS ?

+1,
Or/and based on KSS : http://warpspire.com/kss/
Thx a lot :)

For comments based on Idiomatic CSS I've solved with easy custom atom snippets.

I found another workaround for idiomatic comment with : atom-idiomatic-comments-css-snippets.
It's cool, but it's not enough when i work on large project with complex styleguide, lot of css module.
And i think KSS are very close to jsDoc but for CSS :)

What does support for CSS mean, just the basic functionality or autocomplete with code analysis too?

@MoritzKn just basic functionality but actually I solved the need easily with custom snippets and I believe it's also a better solution the adding it in docblockr


bots: Add view to access state.

The first three commits are more general improvements and should be mergeable without exposing access to the state.

I merged the first commit.  Can you split the bots: Validate state size limit directly in set_bot_state(). commit to just move the code around in one commit and move the quota to be a field on Realm in a second commit?  (I'm actually not entirely sure we want to move the quota to Realm as opposed to a server-level setting, but regardless, it seems like a totally independent change).  And of course moving the location of code and changing its content makes things really hard to review :).

Yup, will do! I actually thought adding the field in the realm table would mean adding it to a server-level setting, guess I'm not completely confident about the difference. Which table would we use for server-level settings?

zulip.readthedocs.io/en/latest/settings.html explains the server-level settings system.

@timabbott split bots: Validate state size limit directly in set_bot_state(). into several commits. One thing that made me skeptical about splitting this up is that it introduces temporary imports/variables like
+ from zerver.lib.bot_lib import StateHandler
+ state_size_limit = StateHandler.state_size_limit

These are sort of hacky and made obsolete with the next commit, and shouldn't really go in the server code. But I think it's definitely easier to review now.
Also updated the tests so that the new views have 100% coverage.

Updated with some more commits to add a "remove entry" view and document the quite long state view test case.

Those temporary imports are fine; and this is a lot more readable.  Merging about half the commits now; don't have time to read the rest before the weekend.

Resolved the merge conflicts.

@derAnfaenger can you rebase this one?

Rebased. For some reason, the complete get_keys_in_bot_state() function was gone, and I wasn't able to find it in the history. (but maybe I ran a wrong command here for search).

Rewrote it.

OK, this is good enough, merged, thanks @derAnfaenger!  I think we might want to consider doing a quick rename from user_state to user_storage for the endpoints before we do an API release that has these names, since that's more consistent with what we're calling it elsewhere (and URLs are one of the most unpleasant things to change).
While you're updating this, I'd also consider adding some newlines in the the tests for this to make them more readable; they feel very dense :).


BIDS Segmentation labels?

https://bids-specification.readthedocs.io/en/stable/derivatives/imaging.html#common-image-derived-labels
The BIDS common derived labels are different to ANTs:
Integer value	Description	Abbreviation (label)
2	White Matter	WM
3	Cerebrospinal Fluid	CSF
8	Cortical Gray Matter	CGM
9	Subcortical Gray Matter	SGM
10	Brainstem	BS
11	Cerebellum	CBM
We can either roll with these, or use a tsv file to correspond to standard ANTs labels.
Whatever solution we choose, it will affect both segmentations generated by the pipeline, and segmentations input to use as priors.

This is now implemented.
One issue is that this makes for bad visualization with ITK-SNAP default labels.

Possible workarounds for label coloring:

ITK SNAP devs could solve by setting default colors in order of existing labels, rather than 1,2,3,4,5,6.

Make and share snap label definitions. Unfortunately these have to be loaded every time.

ITK-SNAP could support BIDS label definitions (possible but maybe not likely given that it is not a BIDS app)

Label defs + command line aliases would work for me, but IDK about others


Added django base server project

Changes

Added sckeleton for server project based on django
Implemented behaviour of users type of (login, register)
Login and resgister uses rest api
To not forget user using tokens

Hightlights

For DB will implement MongoDB
All json responses for now can be found in /server/users/views,py
Roots can be found looking in urls.py both on server and users

#28 Added server


[bug]: file option in uploadFiles.onUploadProgress is inaccurate

Provide environment information
n/a

Describe the bug
the onUploadProgress callback in uploadthing/client's uploadFiles function is inaccurate and has a file property:
https://github.com/pingdotgg/uploadthing/blob/62810d6f623a1c755e1e099bac960115a158cb2f/packages/uploadthing/src/types.ts#L35
This is inaccurate since the progress is based on the total progress of all files, so the event should not be bound to a specific file. See here where the percent is based on the total progress, so the fileName will just be the "random" file that triggered the progress event, but the progress is not related to that file at all:
https://github.com/pingdotgg/uploadthing/blob/62810d6f623a1c755e1e099bac960115a158cb2f/packages/uploadthing/src/client.ts#L206-L207
We should change this signature to just be (progress: number) => void like we have in @uploadthing/react and @uploadthing/solid:
https://github.com/pingdotgg/uploadthing/blob/62810d6f623a1c755e1e099bac960115a158cb2f/packages/react/src/types.ts#L43
https://github.com/pingdotgg/uploadthing/blob/62810d6f623a1c755e1e099bac960115a158cb2f/packages/solid/src/types.ts#L43
Link to reproduction
n/a
To reproduce
n/a
Additional information
No response
👨👧👦 Contributing

[X] 🙋♂ Yes, I'd be down to file a PR fixing this bug!

Code of Conduct

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

Reason for filing an issue instead of just PRing directly is that this is a breaking API change and thus I just wanted to start tracking it.

Actually this seems to be working as inteded


Cannot find name 'AddEventListenerOptions'.

I'm submitting a bug report

Library Version:
dialog#1.0.0-beta.3.0.1

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
8.15.0

NPM Version:
6.4.1

JSPM OR Webpack AND Version
JSPM 0.16.41

Browser:
all

Language:
TypeScript 3.3

Current behavior:
Build failing due to typing issue originating out of this packages typings:
typings\modules\aurelia-dialog\index.d.ts(1576,44): error TS2304: Cannot find name 'AddEventListenerOptions'.
Expected/desired behavior:
to not get typings errors

@falconmick Can you at least update to latest version of v1? v1.0.0-beta.x is a bit hard to support/debug now

If you still get the same error with v1, please help comment here

I'll get back to you. It's an older project and nobody left at the company knows it so It's kinda not a fun day when this stops working! Thanks for getting back to me

There is something strange here. First there is no such type - AddEventListenerOptions, used in the 1.0.0-beta.3.0.1 types. Second, this is a DOM API, I have no recollection of having anything explicitly typed with it. Third, what is this line number - 1576? The .d.ts file for this version is only 231 lines.

typings\modules\aurelia-dialog\index.d.ts(1576,44)

looks like aurelia-templating.d.ts is imported from the latest dist, see this commit:
https://github.com/aurelia/templating/commit/d2da4143389291077a7efd05d960f5ed45dcc0cf#diff-d4ec8ae7ec00cac9b4cc47b504a8882c

So this issue can be closed, since it's not in the dialog?

Issue must have been due to build server and dev machines being on a certain typescript version and then somthing made both of em jump to another as the issues were caused by wrong versions of typescript


New WebUI - Generation queue/delayed batch processing

Is your feature request related to a problem? Please describe.
I find I am often using webUI to explore variation branches and vague prompts, so I am usually creating 3+ images a generation in a  MidJourney-esque workflow. This is fine, but I often see different branches I want to revisit. digging through past results can be difficult to find the specific image out of all the variations.
Describe the solution you'd like
An optional Generate mode that will save all of the settings for the image generation, but added to a queue that could be executed at a later time (say the end of the day or when I have to work on other things) as one large batch process. This queue could use the average generation time to also give an approximate compute time for the queued images so generation can be managed.
This may also need a separate 'queue manager' view in the UI so an overflowing queue can be effectively managed in selected batches
Additional context
In my head, I see this as a dropdown option on the 'Generate'

Queued jobs are the most important feature request for me too.

Additionally, maybe having the option to "add to queue" once a batch is running would be great as well.


[TA] rename abstract -> abstractive

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-textanalytics


[DOCFIX] Fix doc for audit logging.

@aaudiber
Could you take a look? Thanks!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17010/
Test PASSed.


Rewrite shebang lines in scripts so conda can understand them

Rewrite the shebang lines in the autoconf scripts without spaces or ending flags so that conda can properly modify them when creating an environment.
Required to work around conda/conda#3999

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Trying to build this with conda-build 2.0.x is going to be even more challenging.  The shebang line for tools created in the build and used later in the build are longer than those allowed by most systems (127 characters).  Luckily the package does not contain binary prefixes so there is no need to rebuild this with conda-build 2.0.x.

OS X is not happy with that change:
+ sed -i '1s~.*~#!/Users/travis/miniconda3/envs/_build_placehold_placehold_placehold_placehold_pla/bin/perl~' /Users/travis/miniconda3/envs/_build_placehold_placehold_placehold_placehold_pla/bin/autoheader
sed: 1: "/Users/travis/miniconda ...": undefined label 'ravis/miniconda3/envs/_build_placehold_placehold_placehold_placehold_pla/bin/autoheader'
Command failed: /bin/bash -x -e /Users/travis/miniconda3/conda-bld/work/autoconf-2.69/conda_build.sh

Apparently GNU sed and BSD sed have different syntax for doing in-place substitution.  I'm not sure if this is the best solution anyway.  Let me think on this a bit and see if there is a better way.  If conda/conda#3999 gets fixed this might not be needed anyways.
Additionally I've found a better solution to the original reason why I went down this path, so it is no longer a priority for me.

Someone added GNU sed awhile back. Would recommend just adding sed as a build dependency instead of messing around with the syntax.

@jjhelmus what is the status of this? Should we close it?

I don't believe this is needed anymore.  closing.

Trying to build this with conda-build 2.0.x is going to be even more challenging. The shebang line for tools created in the build and used later in the build are longer than those allowed by most systems (127 characters). Luckily the package does not contain binary prefixes so there is no need to rebuild this with conda-build 2.0.x.

We have a requirement to build packages (including autoconf) in an air-gapped environment using conda-build 3.18.11 and are getting the bad interpreter errors due to length of shebang during the build (as you would know the autoconf build uses sed to substitute in $PREFIX before running scripts as part of the build).
As of yet I have not worked out a solution - any ideas?


Fixed Linux Y-axis scrolling bug

https://github.com/octalmage/robotjs/issues/301

I too am having this issue, but I tested these changes and it works!  How about we get this merged?

I'll take a look this weekend, thanks for the PR!

@octalmage Is this resolved?

Sorry everyone, releasing and building is currently difficult since I have to spin up a bunch of VMs. I'm going to work on getting TravisCI publishing so it's easier to test and deploy. Thanks for the PR!


Fix reserialization of broadcasted tables

Fixes #3266.
Re-serializing a broadcast caused issues because it purposely deserializes into a different state on the executor than it does on the driver, so serializing on the executor due to memory pressure after being deserialized could cause an NPE.
This updates the broadcast to always deserialize only to memory but then throw away the host buffers once the GPU batch is requested at which point it creates it.

build

build

build

build

Test failures were related to row-count-only tables.  I updated to match the old behavior where we don't try to serialize or deserialize the types if there's only rows.  I could simplify this code by serializing the datatypes in all cases to avoid the special-case scenarios at the cost of a bit bigger serialization data for the row-count-only cases.  Let me know if that's desirable.

build


left behind trailing whitespace prints location as file:line:line:column

Thanks for reaching out. This is a duplicate of #5463, and #5506 is already open to resolve the issue


Document algorithm lifecycles

Following on from #14298, the OTC decisions revolving around this need to be enforced.
The enforcement will happen at a later date, for now it suffices to document what the life-cycles are for the newly introduced APIs.

This has been done.


Possible global pollution

Hi Matt,
I just noticed this and wondered if you knew about it?

Thanks
James

That's just how webpacks UMD works, it exports the module name so if you're using commonjs you can do angular.module('myApp', [require('angular-bootstrap-calendar')]);
As you're using globals it exports the module name on the window (I think). AFAIK this is OK, I based this method on what I saw several other popular angular modules doing to support UMD loading.

OK, cool. :) Just thought I'd mention...


Chore/deploy GitHub ci

[#8] [Backend] Set up CI for github actions
What happened 👀

Update test.yml for my own docker image since the template is using same username as github for finding images in dockerhub.
Update Dockerfile files for Ruby 3.1.3
Update bin/docker-assets-precompile for some Ruby 3.1 issues as you can find here
Update environments/test.rb due to removing Figaro gem. We need to load env. If this is bad practice, we can move the variables to Github repo.
Add spec/support/disable-animation.rb file and spec/system folder as config/environments/test.rb and .github/workflows/test.yml requirements.

Insight 📝
We are using nimble/rails-template to create this project. The project Github actions are already included in the template and failing due to config errors for docker image and new ruby version.
Proof Of Work 📹

This is Test production build passing

This is test.yml passing

Danger has errored
[!] Invalid Dangerfile file: no implicit conversion of nil into String. Updating the Danger gem might fix the issue. Your Danger version: 9.2.0, latest Danger version: 9.3.1
 #  from Dangerfile:10
 #  -------------------------------------------
 #  # Runs Brakeman on modified and added files
 >  brakeman.run
 #
 #  -------------------------------------------

  Generated by :no_entry_sign: Danger


PrimaryTextCard-Covid-Indicator doesn't work on mobile iOS/Android

Looks like the tables, images in the markdown are not supported on mobile in card view.

https://github.com/pnp/sp-dev-fx-aces/blob/64c0313f8492532f846f7a3556dc9848c4fdff0e/samples/PrimaryTextCard-Covid-Indicator/src/cards/cardView/CardView.ts#L18
The heading markdown tag doesn't render properly on iOS Quick View.
Expected

ios you can see ### before date and strange orange color

@joaojmendes Looping you in. I know there are some issues with mobile device support. Is there anything you can do here to assist?

Hi , I tested on my IOS in SharePoint App and Browser Safari and everything is working

![IMG_6696](https://user-images.githubusercontent.com/12551125/156813

687-afe46b8c-d1c3-4ddd-bd85-e0682677e4f3.PNG)
.

Hi I'm not talking about SharePoint App or Browser Safari. It doesn't work for Teams App for ios inside Viva Connections App.


Bounding box is sometimes mirrored or collapses

Depending on the tilt of the barcode, the bounding box sometimes appears mirrored at an edge of the barcode (as in the attached image), or it collapses to a single line.

@undecaf is it fixed with your PR? (just published 1.0.2)

good to know! closing the issue


elastic: make sure the system doesn't get stuck due to overloaded ES DB

We add in the case of topology DB updates currently for ES only
channel that will make bulk updates to storage and clear the update buffer
and clear the buffer in case and we reach some watermark. Reaching it will mean
that ES doesn't stand the frequency of updates.

Close the issue since we fix it with buffer overflow management for flows in the meantime.


Generating a blog app with OAuth 2.0 fails to compile

Describe the bug
I tried generating an app with the blog-oauth2.jdl. It fails to compile.
To Reproduce
jhipster jdl blog-oauth2.jdl --blueprints quarkus

Unfortunately, this results in a compilation error.
[INFO] --- maven-compiler-plugin:3.8.1:compile (default-compile) @ blog ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 30 source files to /Users/mraible/quarkus/target/classes
[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /Users/mraible/quarkus/src/main/java/com/jhipster/demo/blog/domain/Blog.java:[42,12] cannot find symbol
  symbol:   class User
  location: class com.jhipster.demo.blog.domain.Blog
[INFO] 1 error
[INFO] -------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  59.470 s
[INFO] Finished at: 2020-12-10T09:09:28-07:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project blog: Compilation failure
[ERROR] /Users/mraible/quarkus/src/main/java/com/jhipster/demo/blog/domain/Blog.java:[42,12] cannot find symbol
[ERROR]   symbol:   class User
[ERROR]   location: class com.jhipster.demo.blog.domain.Blog
[ERROR]
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

@mraible as for now the Oauth2 does not produce a User entity as upstream JHipster. I guess in your JDL your Blog entity has a relation to the User entity.
That's why it fails by default.
Add a custom User entity in your JDL and I'm pretty sure it will work :)

I'm not comfortable with generating User domain object when there is no user management and authentication actually managed by the application. (I'm not with the upstream generator too BTW).
In the Blog sample, we need a user mainly to fullfill the Blog ownership relation. I tend to separate the authentication/authorization aspect of the User from it's data segregation function which is domain dependant.
AFAIK, .Net Core or nodejs blueprints aren't implementing it neither.

Since you're the maintainer of this blueprint, the choice is up to you. I thought it was a cool feature when I added it to the main generator because it allows you to create relationships with the User object, just like you can with JWT authentication. It also works in a microservices architecture.
If you try to create a JWT-based microservice with a relationship to User, the generation succeeds, but the compilation fails (similar to how it does here). I think it'd be better if the generation fails and warns the user before they try to compile.
If you have entities that need a relationship to the current user, how would you suggest a developer implements it?
The "save the user right after they log in" implementation might not be ideal. A better solution would be to use something like Okta Event Hooks which can call an endpoint to sync a user when they're created. However, I don't think Keycloak has anything like that, and it's not a standard part of OAuth.
FWIW, this does work in the Micronaut blueprint. I haven't tested this JDL with the .NET Core or Node.js blueprint, but I'd be happy to sponsor their implementation.

If I try this now, it results in the following error:
ERROR! An error occured while running jhipster-quarkus:bootstrap-application#configuringTemplateTask
ERROR! ERROR! Cache provider ehcache is not supported
Error: Cache provider ehcache is not supported