Syntax issue in Unusual Commandline Detection

There are a couple of syntax issues with unusual commandline detection:

use label instead of output since Streaming ML has migrated the schema of Adaptive Thresholding to use label as the output column. Also, output = "True" is incorrect as it compares a boolean with a string. instead use | label
There was an extra comma in eval cmd_line_norm= line leading to parsing issues

This should be the right syntax
| eval timestamp=parse_long(ucast(map_get(input_event, "_time"), "string", null))
| eval cmd_line=ucast(map_get(input_event, "process"), "string", null),
dest_user_id=ucast(map_get(input_event, "dest_user_id"), "string", null),
dest_device_id=ucast(map_get(input_event, "dest_device_id"), "string", null),
process_name=ucast(map_get(input_event, "process_name"), "string", null)
| where cmd_line!=null and dest_user_id!=null
| eval cmd_line_norm=replace(cast(cmd_line, "string"), /\s(--?\w+)|(\/\w+)/, " ARG"),
cmd_line_norm=replace(cmd_line_norm, /\w:\\[^\s]+/, "PATH"),
cmd_line_norm=replace(cmd_line_norm, /\d+/, "N"),
input=parse_double(len(coalesce(cmd_line_norm, "")))
| adaptive_threshold algorithm="quantile" entity="process_name" window=60480000
| where label AND quantile>0.99
| first_time_event cache_partitions=1 input_columns="dest_device_id,cmd_line"
| where first_time_dest_device_id_cmd_line
| eval start_time = timestamp,
end_time = timestamp,
entities = mvappend(dest_device_id, dest_user_id),
body = "TBD";

To Generate a dataset to test this, use:
import numpy as np
import pandas as pd

processes = ["c:\\\\program files\\\\splunkforwarderforsplunkinc\\\\bin",
                  "C:\\\\Windows\\\\system32\\\\conhost.exe 0xffffffff",
                  "C:\\\\Program Files\\\\SplunkForwarderForSplunkInc\\\\bin\\\\splunk-powershell.exe",
                  "C:\\\\Program Files\\\\SplunkForwarderForSplunkInc\\\\bin\\\\splunk-admon.exe",
                  "C:\\\\Program Files\\\\SplunkForwarderForSplunkInc\\\\bin\\\\splunk-winhostinfo.exe"]

anomalous_process = "relog.exe C:\\\\Program Files\\\\Microsoft\\\\Exchange Server\\\\V15\\\\Logging\\\\Diagnostics\\\\PerformanceLogsToBeProcessed\\\\ExchangeDiagnosticsPerformanceLog_09242014.blg -f csv -o C:\\\\Program Files\\\\Microsoft\\\\Exchange Server\\\\V15\\\\Logging\\\\Diagnostics\\\\PerformanceLogsToBeProcessed\\\\ExchangeDiagnosticsPerformanceLog_09242014.csv tmp -y"

all = list(np.random.choice(processes, 20))
all.append(anomalous_process)
data = ["{" + "\"process\": \"{}\", \"dest_user_id\": \"a\", \"dest_ip_id\": \"<IP_ADDRESS>\", \"_time\": {}, \"process_name\": \"b\"".format(v, i) + "}"
 for (i, v) in zip(range(len(all)), all)]

with open("unusual_commandline.json", "w") as outfile:
    outfile.write("\n".join(data))

unusual_commandline.txt

To test this in smle:
| from read_text("s3://smle-labs-test-customer-bucket/ssa/unusual_commandline.json")
| eval input_event=from_json_object(value)
| eval timestamp=parse_long(ucast(map_get(input_event, "_time"), "string", null))
| eval cmd_line=ucast(map_get(input_event, "process"), "string", null),
dest_user_id=ucast(map_get(input_event, "dest_user_id"), "string", null),
dest_device_id=ucast(map_get(input_event, "dest_device_id"), "string", null),
process_name=ucast(map_get(input_event, "process_name"), "string", null)
| where cmd_line!=null and dest_user_id!=null
| eval cmd_line_norm=replace(cast(cmd_line, "string"), /\s(--?\w+)|(\/\w+)/, " ARG"),
cmd_line_norm=replace(cmd_line_norm, /\w:\\[^\s]+/, "PATH"),
cmd_line_norm=replace(cmd_line_norm, /\d+/, "N"),
input=parse_double(len(coalesce(cmd_line_norm, "")))
| adaptive_threshold algorithm="quantile" entity="process_name" window=60480000
| where label AND quantile>0.99
| first_time_event cache_partitions=1 input_columns="dest_device_id,cmd_line"
| where first_time_dest_device_id_cmd_line
| eval start_time = timestamp,
end_time = timestamp,
entities = mvappend(dest_device_id, dest_user_id),
body = "TBD";

Each detection for SSA now has a test file as that CI will run: https://github.com/splunk/security-content/tree/develop/tests/endpoint as well as individual notebooks used for manual testing: https://github.com/splunk/security-content/tree/develop/notebooks closing this for now.


Combobox - placeholder not visible

Describe the bug
placeholder in combobox component not visible
How to reproduce
can be seen in clarity documentation
Expected behavior
placeholder should be visible
Versions
App

Angular: [9]
Node: [10.16]
Clarity: [4.0.0]

It looks like we are not passing placeholder attr down to the native element.


Migrate off of Grafana to Dash/Plotly

How to embed a Dash app into an existing Flask app

Closed with #244.


feat: add support for spanPrefix to HttpRpcRouter

we should probably make toHttpApp dual then

we should probably make toHttpApp dual then

how so? also RpcRouter.toHandler isn't.

Adding an optional options arg is breaking, well actually it's breaking anyway


Code snippet in documentation showing line numbers

Describe the bug
The code snippet in docc documentation previews showing line numbers.

To Reproduce
Steps to reproduce the behavior:

Create a swift package.
Add code snippets in any documentation.
Hover over the documented symbol to preview documentation.
See the bug.

Expected behavior
Code snippets should not show line numbers as in Xcode.

Environment

OS: macOS 13.3.1
Swift version:

swift-driver version: 1.75.2 Apple Swift version 5.8 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>.100)
Target: arm64-apple-macosx13.0

Visual Studio Code version: 1.77.1
vscode-swift version: 1.2.0

This is coming from SourceKit-LSP. You should add an issue there https://github.com/apple/sourcekit-lsp/issues.
I'm not sure it's a bug issue. VSCode doing something different from Xcode is not a bug.


crop farm

how do you use it. it has power, water and wheat, carrot seeds but nothing is happening?

does it have a hoe?... or is the land already tilled?
water is also optional if the land is naturally wet.

It doesnt gave a hoe. Do you have a guide somewhere that tells us what your machines need to operate? Does it work with mod plants?
Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------From: faceofcat<EMAIL_ADDRESS>Date: 9/10/17  8:32 PM  (GMT-08:00) To: faceofcat/Tesla-Powered-Thingies<EMAIL_ADDRESS>Cc: LocoMinerDad<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [faceofcat/Tesla-Powered-Thingies] crop farm (#6)
does it have a hoe?... or is the land already tilled?
water is also optional if the land is naturally wet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/faceofcat/Tesla-Powered-Thingies","title":"faceofcat/Tesla-Powered-Thingies","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/faceofcat/Tesla-Powered-Thingies"}},"updates":{"snippets":[{"icon":"PERSON","message":"@faceofcat in #6: does it have a hoe?... or is the land already tilled?\r\n\r\nwater is also optional if the land is naturally wet."}],"action":{"name":"View Issue","url":"https://github.com/faceofcat/Tesla-Powered-Thingies/issues/6#issuecomment-328406743"}}}

not yet... :(
but... it's pretty straight forward:

provide it with power (Tesla, Forge Energy, RF, Mekanism, MJ)
either provide it with a hoe, or till the land yourself (you can use the 'show working area' thing to see where it will look for plants)
either provide it with water, or make sure the land is naturally moisturized (from nearby water blocks)
either provide it with seeds, or plant the initial plants yourself)
(optional) provide it with bonemeal

that should be it...

all good thx


OpenAI plugin should use self.get_key()

The self.get_key() mechanism means that Python library users get access to the same persisted API keys as CLI users. Without that anyone using the Python library has to manually set the key themselves. llm-claude-3 does this correctly, but the default OpenAI plugin does not.

Relevant code: https://github.com/simonw/llm/blob/d075336c6900768b664277a7ca7c76a79de9a0cf/llm/default_plugins/openai_models.py#L341-L347
And this method: https://github.com/simonw/llm/blob/d075336c6900768b664277a7ca7c76a79de9a0cf/llm/models.py#L215-L240

Before:
>>> import llm
>>> print(llm.get_model("gpt-4o-mini").prompt("hi"))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/simon/Dropbox/Development/llm/llm/models.py", line 109, in __str__
    return self.text()
  File "/Users/simon/Dropbox/Development/llm/llm/models.py", line 112, in text
    self._force()
  File "/Users/simon/Dropbox/Development/llm/llm/models.py", line 106, in _force
    list(self)
  File "/Users/simon/Dropbox/Development/llm/llm/models.py", line 91, in __iter__
    for chunk in self.model.execute(
  File "/Users/simon/Dropbox/Development/llm/llm/default_plugins/openai_models.py", line 306, in execute
    client = self.get_client()
  File "/Users/simon/Dropbox/Development/llm/llm/default_plugins/openai_models.py", line 352, in get_client
    return openai.OpenAI(**kwargs)
  File "/Users/simon/.local/share/virtualenvs/llm-p4p8CDpq/lib/python3.10/site-packages/openai/_client.py", line 105, in __init__
    raise OpenAIError(
openai.OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable

After:
>>> import llm
>>> print(llm.get_model("gpt-4o-mini").prompt("hi"))
Hello! How can I assist you today?


Add a worker node(s)

SDG has started taking longer. We should add another worker or two and start tracking load with grafana #250

Completed 04/22


Average speed is reported incorrectly for resumed downloads

Checklist

[x] I've searched for similar issues.
[x] I'm using the latest version of HTTPie.

Minimal reproduction code and steps

Download a file using http -dc <address> -o ./somefile
Interrupt the download at a high percentage done (e.g. 95%)
Resume the download with the same command in Step 1
When it completes, observe the average speed printed

The code to compute average speed this divides total file size by the time spent in the current (resumed) invocation of httpie. so the output is a much higher average speed than actual.
We should keep track of how much of the file was downloaded in the current invocation and use that as the numerator over here: https://github.com/httpie/httpie/blob/30a6f73ec806393d897247b4c7268832de811ff7/httpie/output/ui/rich_progress.py#L41
Current result
When I download the last 10% of a file at 5MB/s, the printed average speed is 50MB/s (1/10 the file size => 10x the speed)
Expected result
When I download the last 10% of a file at 5MB/s, the printed average speed should be 5MB/s

Debug output
Please re-run the command with --debug, then copy the entire command & output and paste both below:
$ http --debug <COMPLETE ARGUMENT LIST THAT TRIGGERS THE ERROR>
<COMPLETE OUTPUT>

Additional information, screenshots, or code examples
…

Hi, first-time contributor here, wanted to try myself at this issue :) I will propose a PR.


Revert all packages back into meta.yaml

After testing installations on different platforms ,  we have decided to only support conda installation for mac and linux.
Windows installations should be done via pip.

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

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.


Next export: making links and static folder work locally?

After running next export and opening the out/index.html on my dev machine in the browser, the links won't work. E.g a link to /courses will point to file:///courses/.
Also, assets from the /static folder won't be linked properly either. E.g if there's an <img src='/static/bla.png'>, it will try to load it from file:///static/bla.png.
Is there any way to fix this within next.js to allow testing exports locally instead of needing to deploy every time?

You don't need to deploy it. Just run a local HTTP server. Here's the simplest one:
python -m SimpleHTTPServer

Then you can access your app in http://localhost:8080
Serving the folder directory without a HTTP server is not a goal we need to go for.

You can also use serve -s (npm install -g serve) from ZEIT 👌


Add support for nodejs12.x

AWS now supports nodejs12.x as a runtime, but linting is throwing an error that it is not supported.

You need to update your version of cfn-lint since we use that package to do the actual linting.
https://github.com/aws-cloudformation/cfn-python-lint#pip-install
Included as of version v0.25.2
Included with PR 1204

Thank you! This can be closed then


start-vswitch.sh: allow vpp device descriptors to be overridden

maybe I should wait until you are done ;-)

Sorry, I did not check what you were doing.  I have made by updates, so you
should be able to add the change for vpp.
On Mon, Jul 17, 2017 at 2:05 PM, Karl Rister<EMAIL_ADDRESS>wrote:

maybe I should wait until you are done ;-)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/atheurer/dpdk-rhel-perf-tools/pull/2#issuecomment-315850991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AInm7eK5hqZLIV1RlQR7Wr4S9b9F-cFmks5sO7B0gaJpZM4OaYyl
.

ok, should be ready now


Code fails to compile

Hello VinDuv, I'm trying to use this library with an Arduino UNO R3 to make a physical controller connected to an Arduino that can communicate with the Switch. Before trying to modify the swsh.c file to take input instead of perform automated tasks I decided I wanted to try and compile the code into the .hex files like you stated in the readme but I get this error. Do you have any insight?
avr-gcc -mmcu=atmega328p -flto -fuse-linker-plugin -Wl,--gc-sections -o src/swsh.elf src/swsh/swsh.o src/lib/automation.o src/lib/automation-utils.o src/lib/user-io.o
/usr/local/opt/avr-binutils/bin/avr-ld: /var/folders/zb/bwstvvb94t92w5mkj5wxg3hr0000gn/T//swsh.elf.yK8ICC.ltrans0.ltrans.o: in function move_in_circles': <artificial>:(.text+0xbb2): undefined reference to pause_automation'
/usr/local/opt/avr-binutils/bin/avr-ld: /var/folders/zb/bwstvvb94t92w5mkj5wxg3hr0000gn/T//swsh.elf.yK8ICC.ltrans0.ltrans.o: in function go_to_nursery_helper': <artificial>:(.text+0xc2c): undefined reference to pause_automation'
/usr/local/opt/avr-binutils/bin/avr-ld: /var/folders/zb/bwstvvb94t92w5mkj5wxg3hr0000gn/T//swsh.elf.yK8ICC.ltrans0.ltrans.o: in function main': <artificial>:(.text.startup+0x128): undefined reference to pause_automation'
/usr/local/opt/avr-binutils/bin/avr-ld: :(.text.startup+0x260): undefined reference to pause_automation' /usr/local/opt/avr-binutils/bin/avr-ld: <artificial>:(.text.startup+0x3ae): undefined reference to pause_automation'
collect2: error: ld returned 1 exit status
make: *** [src/swsh.elf] Error 1
Thank you!

Hi!
Hmm, pause_automation is defined inline in src/lib/automation.h, but apparently in you case it wasn’t inlined properly, resulting in a link error.
Which AVR toolchain are you using? I tried on macOS with CrossPack-AVR-20131216 and on Debian Stable with gcc-avr 1:5.4.0+Atmel3.6.2-1+b1/avr-libc 1:2.0.0+Atmel3.6.2-1.1 and it builds fine.

I fixed the compilation issue, It's a "hacky" fix but I moved the pause_automation method in full to wherever it was defined. I'm facing another issue through, I'm trying to use your code to make a physical Nintendo Switch controller with an Arduino and when passing through analog readings from an analog stick the Arduino's TX and RX Led's flash 2 or 3 times. What could those error codes mean? I have the inputs from the analog stick scaled down from 0 to 255.

I have a line in swsh.c like this:
send_update(BT_NONE, DP_NEUTRAL, S_COORD((int)l_potXScaled, (int)l_potYScaled), S_NEUTRAL);
l_potXScaled and l_potYScaled are values between 0 and 255.
Unless I follow up the update IMMEDIATLEY with
send_update(BT_NONE, DP_NEUTRAL, S_NEUTRAL, S_NEUTRAL);
Both the TX and RX leds flash twice. Even if I just put a delay statement in between them, it panics.
I need to be able to remove the "send neutral" statement because with that in use, the controller acts like it it rapidly being centered and then pushed in a certain direction instead of being held. Any idea why it's panicking?

Two blinks means that src/usb-iface/usb-iface.c called panic(2), which means either invalid data received or no message received from the main µC (I should have put separate codes…). I suspect it’s the second one in your case.
The important thing is that you don’t need delay calls in your code. You should call send_update as fast as you can. If the USB µC has not processed the previous message, send_update will automatically wait.
In your case, your code should probably do something like this:
init_automation();
for (;;) {
  <determine the current inputs>
  send_update(<current inputs>);
}

When plugged on the Switch, the USB µC needs new data at a rate of 125 Hz (Switch USB poll rate) / 5 = 25 Hz, or once very 40 ms. send_update also takes some time to send the data, probably something like 10 ms. So as long as your code manages to take less than 30 ms between calls to send_update, you should be fine.
If you suspect your code needs more time between send_update calls, you can change the cycle count of the USB µC : https://github.com/VinDuv/switch-arduino-controller/blob/master/src/usb-iface/usb-iface.c#L162
If you change the 5 to a 10, the USB µC will repeat each message 10 times instead of 5, so it will request new data half as frequently.

Thank you so so much! That was the issue, my code is working perfectly now.


VSCode feature implementation in IntelliJ

Does Karate have plans to implement the feature run in IntelliJ at any point?

@rshelby5 already works: https://github.com/intuit/karate/wiki/IDE-Support


support for Python3

XCode11 LLDB's Python scripting is now based on Python3;
FBDebugCommands.py has a syntax error.

There's also #266 working on this.

Ok, I close this PR.

Thanks for adding it!


building reference strip outside the viewer

Hi guys,
i know that right now there isn't an option to have the reference strip outside of the viewer,
but is there any way of doing this?
like a viewer under the viewer?
i would like to have it static, and also not covering the image view area..
would be nice to get information on how to approach this issue, i don't really know where to start.
a hint on how to begin or any help would be much appreciated!
thanks,
Michael

Hello,
I don't know much about the reference strip. My guess would be that you have to build it yourself.
Having another viewer with all the images drawn onto it looks like the best approach to me.
You would use the full multi-image mode, prevent zooming and vertical panning and rebind the click event to open the correct image on the main viewer.
It sounds like it could be a good plugin that we could add to the list here: http://openseadragon.github.io/#plugins
So please share if you get something working :)

thanks for the hint will start working on it.
will update once i got it to work

ok got a bit stuck..
so i supposed to declare a new Viewer object?
wouldn't that interfere with the current OSD Viewer?
also should i name it completely different or should i change the reference class itself to match my needs?

Yes you can have multiple viewers in a web page.
Just specify a different id for each:
var mainViewer = OpenSeadragon({
        id:              "mainViewer",
        prefixUrl:       "/openseadragon/images/",
        tileSources:     ...
    });

var refStrip =  OpenSeadragon({
        id:              "refStrip",
        prefixUrl:       "/openseadragon/images/",
        tileSources:     ...
    });

And in your html you should have one div with id mainViewer and another one with id refStrip.

ok thanks!
and what would your suggestion be-
write a whole new class for the refstrip or manipulate the existing?

I would start from scratch because the reference strip is not leveraging the multi image.

ok. thanks

@avandecreme i got stuck,
i'm trying to implement the multi-image mode with not much success, either it ads all images above each other or just takes the final image of the array (not sure which one it is) .
first i create an OSD viewer passing the id of the div i'm going to use -
var thumbbarViever = new OpenSeadragon({
       id : this.thumbviewerID,
       panVertical : false,
       panHorizontal: true,
     });

then i open the imageSource array , arrange it but it wouldnt show all of the images,
also by world.getItemCount() i get 0 all the time.
thumbbarViever.open(this.imageSource);
     thumbbarViever.world.arrange({rows:1,
       immediately : true,
       layout : "horizontal",
     });
 thumbbarViever.innerTracker.scrollHandler=false;
 var count = thumbbarViever.world.getItemCount();

not sure how to prevent zoom as you suggested.
there isn't really a lot of documantation about the multi-image mode..

ok it seems that i mixed up a lot of stuff.. so here is my current code -

    var thumbbarViever = new OpenSeadragon({
        id : this.thumbviewerID,
        visibilityRatio: 1,
        maxZoomLevel: 1,
        showNavigator:          false,
        showNavigationControl:  false,
        constrainDuringPan: true,
    });
    imgSrcLength = this.imageSource.length;
    for(i=0; i < imgSrcLength; i++){
        thumbbarViever.addTiledImage({
            tileSource: this.imageSource[i],
            index: i,
            opacity : 1,
            x: -(i *1),
            height: 1,
        });
    }
    thumbbarViever.innerTracker.scrollHandler=false;
    var count = thumbbarViever.world.getItemCount();
    console.log("Count = " + count);

right now i got a row which is great, still cant prevent zoom by click (and touble ta on touch),
also how could i add scroll functions and bind click event when clicking on image (how would i get position of image in array if count shows 0?)
i did the x: -(i *1) in order to get the images in RTL mode, i'm not quite sure thats the right way to approach this issue.
the problems now are -

the image row can be dragged a bit (up and down) i want it to be static with the viewer.
the row can be still zoomed in
i have no idea how to handle the click on the image
on window size change the images resize as well i.e. when the width of screen is sized to smaller
the images get smaller and vice versa and i want them to fit the viewer height also during resize of
the window.
*need to change click and touch event to get position of image in array and bind pageTo() to mainviewer.

It looks like you are making good progress :)

i did the x: -(i *1) in order to get the images in RTL mode, i'm not quite sure thats the right way to approach this issue.

Sounds reasonable to me. I would just write -i though :)

the row can be still zoomed in and out.

The second example on this page should answer your question (except that you want to switch vertical/horizontal).

on window size change the images resize as well i.e. when the width of screen is sized to smaller the images get smaller and vice versa and i want them to fit the viewer height also during resize of the window.

Hopefully, the above solution will fix that as well.

i have no idea how to handle the click on the image

Not sure what you mean

need to change click and touch event to get position of image in array and bind pageTo() to mainviewer.

This example should get you going: https://codepen.io/avandecreme/pen/YGYdgQ
Also don't hesitate to look around here for other examples: https://codepen.io/collection/APoyjJ/2/#

bind scroll buttons to viewer.

I am not sure what you mean? Isn't it the case already?

@avandecreme thanks for the answer.

Sounds reasonable to me. I would just write -i though :)

you're right, i thought that the multiplication would give me more margin between the images.

The second example on this page should answer your question (except that you want to switch vertical/horizontal).

and-

Hopefully, the above solution will fix that as well.

I could not find an example of that, the first link is to get the position of the image on mouseover, but the second is a collection of examples..

i have no idea how to handle the click on the image

Not sure what you mean

I need that the click on the thumbnail will open that image in the mainviewer. Currently it starts to zoom the thumbnail! Which function do I need to bind to allow this to happen.

bind scroll buttons to viewer.

I am not sure what you mean? Isn't it the case already?

I want to have control buttons on the side of the viewer like in the Demo mainviewer (arrows on both sides)-
http://viewetest.kehilot.netai.net/
that will scroll the images to the left or right depending on the button used.
which function do I need to use in order to scroll the images once for every click?

UPDATE
thank you @avandecreme for your help!
Now I'm able to select image and show it, although on desktop some clicks are still zooming in, but the main function is working.
Now I need to get the scrolling with the buttons going, add some options like -

toggle hide the bar.
set position of the bar - if on sides switch to vertical
direction set - RTL or LTR

and then I think it's going to be the first version of the plugin :)

Glad you got it working.
For the clicks still zooming in, I am not sure why that is the case but you could disable click to zoom all together by setting zoomPerClick to 1 (or by turning off clickToZoom in the corresponding gesture setting). See http://openseadragon.github.io/docs/OpenSeadragon.html#.Options for more information.
Also, the calculation of the image clicked seems a bit off, for example clicking on the right most image open the second to right image.

Ok will try with the zoomPerClick.
I checked on mobile it worked good, the imaged i touched was shown,
but now i checked on desktop and you are right.
any idea why does this happen?
i copied from the example you gave me.

It looks like you sorted it out right?
It is starting to looks good!
It is a bit hard to pan the thumb bar, I suspect it is because of this issue: https://github.com/openseadragon/openseadragon/issues/753

Thanks.
yes i think it had to do with the dblClickToZoom which i then gave the value of false.
you are right with the lag of the panning i turned the constrainDuringPan to false and it indeed helped, thanks for that.
I do have a problem with the scrolling in the thummbar that if started to scroll by touching (clicking) on a thumbnail, when i release the mouse or finger it jumps to the image i tapped to scroll, in other words in order to avoid that issue one must only scroll by touching the space between the thumbs which is a bit irritating.
Do you know a way to overcome this issue?
also i need help how to scroll to an image position inside the thummbar if i want to add scroll buttons on the side of the thumbbar, and to bind the pageTo (i.e. when a user clicks the Next or Previous button)  from the mainviewer to scroll to the selected image in the thumbbar.
thanks again for all your help so far!

I do have a problem with the scrolling in the thummbar that if started to scroll by touching (clicking) on a thumbnail, when i release the mouse or finger it jumps to the image i tapped to scroll, in other words in order to avoid that issue one must only scroll by touching the space between the thumbs which is a bit irritating.
update - this issue occurs also when i release the mouse over a thumb after finishing the scroll.
Do you know a way to overcome this issue?

I am not sure why this happen actually. The mouse tracker has a clickDistThreshold option with a default to 5 so I believe the click event shouldn't be raised. Maybe @msalsbery has some idea.

also i need help how to scroll to an image position inside the thummbar if i want to add scroll buttons on the side of the thumbbar, and to bind the pageTo (i.e. when a user clicks the Next or Previous button) from the mainviewer to scroll to the selected image in the thumbbar.

If you know the (viewport) coordinate of the center of the thumbnail, you can call thumbbarViewer.viewport.panTo(thumbnailCenter, immediately)

maybe i messed up with the code?
here is how it looks like -
( i cut out the not relevant code)
thumbbarViever.addHandler('canvas-click', function(event) {
    self.setImageMain(event);
    if (!event.quick) {
        return;
    }
});

$.Thumbbar.prototype = {
        setImageMain : function(event){
        console.log("inside setImageMain");
        var index =   self.getImagePosition(thumbbarViever.viewport.pointFromPixel(event.position));
        console.log("inside setImageMain index: " + index);
        if (index !== -1) {
            thumbbarViever.viewport.fitBounds(thumbbarViever.world.getItemAt(index).getBounds());
            this.mainViewer.goToPage(index);
        }
      },
      getImagePosition : function(position){
        console.log("inside getImagePosition " + position);
        var box;
        var count = thumbbarViever.world.getItemCount();
        for (var i = 0; i < count; i++) {
            box = thumbbarViever.world.getItemAt(i).getBounds();
            if (position.x > box.x &&
                    position.y > box.y &&
                    position.x < box.x + box.width &&
                    position.y < box.y + box.height) {
                        console.log("inside getImagePosition i =" + i);
                return i;
            }
        }

        return -1;
      }
}

about the coordinates - how could i find out?
is there a way to save the coordinates while adding the thumbs to the viewer?

I don't see anything wrong with your click handling code.

about the coordinates - how could i find out?
is there a way to save the coordinates while adding the thumbs to the viewer?

No need to save, you can just call thumbbarViever.world.getItemAt(index).getBounds().getCenter() when needed.

Ha actually it is the quick property which is important. Your handler should look like this:
thumbbarViever.addHandler('canvas-click', function(event) {
    if (event.quick) {
        self.setImageMain(event);
        return;
    }
});

Ha actually it is the quick property which is important. Your handler should look like this:

Thanks a lot!
that actually did it!
and thanks for the typo hint as well :+1:
now i'm going to work on the click handling.

@avandecreme i need your help again.
i was trying to implement the scroll buttons, by moving one index at a time,
but somehow i can't get the currentPage() of the thumbbarViewer it always returns the first index i.e. 0 and pans to the beginning.
isn't there a currentPage in full-multi-image mode?

It took me a loot of tweaking but I finally got it.
I added an addHandler for the pan event then saved the center as a global variable,
and added a new Point with the coordinates of x -1 or +1 depending on which scroll button was pressed.
hope this is the right way

@michaelw3 This is great! The old reference strip has a lot of limitations, and we've been wanting to spin it out as a plugin (but an improved version)... looks like you're well on your way to that!

@iangilman Thank you.
I'm not an expert in JavaScript, and new to building plugins, so I hope that I'm doing it right.
Open to suggestions on features it should have.
For now -

Default position is under the Main Viewer and setting the width the same as it has.
Scrolling via Scroll-controls, Touch, Mouse.
Option to hide and show Scroll-controls (separately and together).
Panning Thumbbar to image index when page change occur in Main Viewer .
Changes Page in Main Viewer on click of thumb in Thummbar.
RTL or LTR layout support.

Planning on adding -

Option to add page number over each thumb(still not sure how to do that).
Option to place the Thumbbar also on Left and right side of Viewer.
Option to give a customized placement and size of Viewer.
Option to toggle slide the Thumbbar.

@iangilman Thank you.
I'm not an expert in JavaScript, and new to building plugins, so I hope that I'm doing it right.
Open to suggestions on features it should have.
For now -

Default position is under the Main Viewer and setting the width the same as it has.
Scrolling via Scroll-controls, Touch, Mouse.
Option to hide and show Scroll-controls (separately and together).
Panning Thumbbar to image index when page change occur in Main Viewer .
Changes Page in Main Viewer on click of thumb in Thummbar.
RTL or LTR layout support.

Planning on adding -

Option to add page number over each thumb(still not sure how to do that).
Option to place the Thumbbar also on Left and right side of Viewer.
Option to give a customized placement and size of Viewer.
Option to toggle slide the Thumbbar.

All sounds lovely!
For the page number, you could use an overlay I suppose.
Do you have a work in progress somewhere I could play with?

I'm not so familiar with Overlay's, is there an option to have on every tiled image a different one?
can i add text?
For sure, on my GitHub page -
https://github.com/michaelw3/OSD-ViewerTest/tree/production

Here's some information on overlays:
http://openseadragon.github.io/examples/ui-overlays/
They attach to the viewer, not any specific image, so you'll have to position them just the same as you position images. But yeah, they're just HTML elements, so you can include text.
I downloaded the project from your GitHub page, but it doesn't include any of the images so nothing works. Maybe you could publish it somewhere so interested parties could just hit it in their web browser? No rush, though...

So my guess is that I'll need to calculate and save the position of the thumb to an array during the initialization of the images in the Thummbar?
I have a problem to publish those pictures (its private from the company i work for and they allow me to use it for testing internally ), but i guess i could create and upload other DZI images.
It would take some time though...

Yeah, something like that. You can also get the thumb locations later if you want... something like viewer.world.getItemAt(0).getBounds().
No rush on getting a demo up with new images, but by the time you publish it it would be good to have a demo people can play with before downloading your code.

as i continue with this project and test it i discovered that the way i load the images isn't quite efficient.
this is the way i do it -
  for(i=0; i < this.imgSrcLength; i++){
            this.thumbbarViewer.addTiledImage({
                tileSource: this.imageSource[i],
                index: i,
                opacity : 1,
                 x: -i,
                 height: 1,
            });
          }

but it gets problematic when i have a book with a lot of pages, it gets stuck..(memory wise)
is there a way of loading theonly the images needed to be visible in the viewer and one before and after(outside the viewer) , sort of an cursor adapter in Android on a listview (if anyone knows what i mean), or do i have to build one by my own?

As long as the images are off the screen, it shouldn't be loading any tile data for them. It will still load the info files (if, for instance, this.imageSource is an array of DZI urls) all at once, though. This shouldn't be a big problem for memory, but it may jam up your network pipe. A couple options  for that:

Convert your DZI info into a single giant file that you can load once instead of having so many network calls. You'd then parse the file yourself and feed the info to OSD.
Pay attention to the scrolling of the thumb bar and attach new images (using addTiledImage) as new areas come into view. You'd do this by watching the pan event on the thumb bar's viewer and then doing the math to figure out what's visible.

That second option (which is basically what you have proposed, and yes, you'd have to build it yourself) would help with memory as well, I suppose, though I'd recommend getting to the bottom of why you're running out of memory first... is it really loading tiles for images that are off-screen? If so, sounds like a bug in OSD. Or is the memory being filled up some other way?
If the memory is being filled up with tiles, you can enforce a smaller tile cache with the maxImageCacheCount option when creating your viewer.

Now when I read your comment I'm not quite sure i understand how to load the pictures in the first place.
the code above i posted is executing as soon as i have an imagesource in the mainViewer,
I'm using the multi-image mode, so I asume when I iterate over the array and addTiledImage for eacch DZI file the image would be drawn on the canvas, isn't that the case?
right now the browser crashes.
I wouldn't mind building my own cursor adapter, but then again i have only an option to add a tiled image and not to remove one, so my guess is that when i get to lets say image 25 the browser will crash if i couldn't unload the images that are current outside the viewer.

If you're using a DZI url (e.g. http://example.com/foo.dzi), OSD first needs to load that file (which is very small, since it's just a little bit of XML information) before it can load any tiles (which are larger files in JPEG or other image format). OSD loads the DZI for every image you add immediately, regardless of where the viewer is zoomed/panned to. However, it then only loads tiles for images that are actually visible in the viewport (i.e. not panned off the screen).
So, if you have 200 DZIs and you add them all immediately, but only 5 of them are visible in your viewport, what will happen is: OSD will fire off 200 AJAX requests to get the DZI data, and then start loading the tiles for just the 5 that are visible.
Furthermore, it only loads the tiles that it needs based on the zoom level, so if you're dealing with a thumb strip that's maybe 200 pixels tall then it'll only load the low resolution (i.e. ~200 pixel tall) tiles. Because of this, I doubt you're dealing with a memory issue (unless you've actually found a bug in OSD's tile loading).
Does that help you understand how the pictures are loaded?
As for being able to remove images, you call viewer.world.removeItem(image), so you have complete control over adding and removing whenever you want.
Would it be possible to post an example of the issue you're experiencing so I can take a look at it in the browser?

Thanks for your detailed answer, sorry for the late reply.
Yes your answer helped me to understand how the file loading works, yet i couldn't see anything in the link you supplied ( http://example.com/foo.dzi).
in the example in the link below  I use addSimpleImage, but the same thing happens in addTiledImage
and you can see in the Network tab(in developer mode) that it is actually loading the images and than crashes.
Example-
http://viewetest.kehilot.netai.net/

Sorry, example.com was just meant to be an example of how you might form the URL; there's nothing there.
What browser are you on? The example link doesn't crash for me on Mac Firefox or Chrome.
As for how things load, addSimpleImage is actually quite different from the DZI path I described above... it uses the ImageTileSource which loads the entire full resolution image (since it's meant for images that haven't been tiled like DZI). If you load 84 full resolution images I wouldn't be surprised if you ran out of memory (depending on the size of the images). This is where DZI makes a big difference, since you can load just the low resolution tiles.
When you were using addTiledImage was it with DZI or with simple images?


Fix $0 global replace with \U, \u, \L, and \l

This PR fixes #148894
So this currently makes search and replace behave the same as the search widget find and replace.  But, I guess my question is, why was this implemented to originally parse the 0 to an ampersand?  Was this implemented for some older replace logic that no longer exists?  It could make sense that it was once intended behavior and was just missed in an update.
This does make the two features work identically to each other, though, so I assume this is what we'd want.
https://user-images.githubusercontent.com/13624065/169431667-e570edf7-89dd-4c37-bd2c-44ecaaad1daa.mov

I'm not really sure why that code is there. $0 should be the same as $& right? So now I wonder if it's mapped to $& why wouldn't \U$& work?

Intermediately deleted failing unit tests (not the best approach but it gets the build to pass while researching why \U$& wouldn't work).

I'm not really sure why that code is there. 0shouldbethesameas& right? So now I wonder if it's mapped to & why wouldn't \U& why wouldn't \U& work?

\U$& just turns something like say "what" into "\Uwhat"
Seems like the replace code was just taking that $& and saying "Okay, I guess we'll just put the match string where $& is and move on."

This regex returns null when executed here

Sorry for the slow response. I see that \U$0 works now, thanks. \U$& isn't working, although that wasn't part of the original issue so it's alright for now unless you feel like fixing it.
But I'm not sure why those tests would have failed - seems like they should still pass. I would need those passing to merge this.

@MachineMitch21 will you be looking into this again?

@MachineMitch21 will you be looking into this again?

Hi.  Yeah, I can look into it.  Sorry, I must have missed the reply.  I think last we saw the unit tests around the $ character were failing because of how the logic around it was changed.  (It was changed because it had to work similarly to the other search and replace component)  I can look more into it, though.

Oh, wait, nvm.  I forgot that I removed the unit tests that were broken because of the change.  (I assume removing those is okay)  Looks like the "basic checks" failed.  I will rebase the feature branch on vscode main after I update my fork, then I'll push up again.  (Maybe that will fix it cause I don't think I made updates that would effect what "basic checks" is complaining about)

Do you plan to look at this again @MachineMitch21?

@roblourens @andreamah Hey, I am looking at this right now.

I added another commit on top of main merge commit to brain this branch up-to-date with main.  The new commit on top of the merge commit removes the replaceWithCharacter variable from that logic as it was no longer used.  The Compile & Hygiene pipeline was not a fan of its definition, and I saw no value in leaving it defined with const (which was the Compile & Hygiene pipeline's suggestion).
As mentioned before, the unit tests were no longer valid due to the necessity of the logic change because of the info mentioned here.

@MachineMitch21 would you be able to reply to the follow-up here? In case you missed it. https://github.com/microsoft/vscode/pull/149990#discussion_r1149573354


Issue: Langchain ChatOpenAI chat_model.complete error message: 'chatopenai' object has no attribute 'complete'

I am new to Langchain and OpenAI models. I am creating a custom PDF reading in Python using Langchain ChatOpenAI model to interact with the chat completion endpoint. I keep getting this error message and I don't know what to do. Please help. All suggestions are welcome!
Here is a part of my code where the error message was generated.
User Input
    current_prompt = st.session_state.get('user_input', '')

    prompt_placeholder = st.empty()

    # Check if a submission has been made
    if 'submitted' in st.session_state and st.session_state.submitted:
        prompt = prompt_placeholder.text_area("Ask questions about your PDF:", value='', placeholder="Send a message", key="new_user_input")
    else:
        prompt = prompt_placeholder.text_area("Ask questions about your PDF:", value=current_prompt, placeholder="Send a message", key="user_input")

    submit_button = st.button("Submit")

    if submit_button and prompt:
        # Indicate that a submission has been made
        st.session_state.submitted = True

        # Update the last input in session state
        st.session_state.last_input = prompt

        # Process user message
        user_message = HumanMessage(content=prompt)
        st.session_state.chat_history.append(user_message)

        try:
            # Similarity check
            docs = VectorStore.similarity_search(query=prompt, k=3)

            # Initialize chat model
            chat_model = ChatOpenAI(model_name="gpt-3.5-turbo")

            # Add a system message to the chat history to define the role
            system_message = SystemMessage(content="You're a helpful assistant")
            st.session_state.chat_history.append(system_message)

            # Get a response from the chat model
            completion_response = chat_model.complete(
                messages=st.session_state.chat_history,
                temperature=0.9  # Adjust temperature as needed
            )
            response_content = completion_response.choices[0].message['content']

            # Process AI message using AIMessage
            assistant_message = AIMessage(content=response_content)
            st.session_state.chat_history.append(assistant_message)

            # Load the question-answering chain
            llm = OpenAI(model_name='gpt-3.5-turbo')
            chain = load_qa_chain(llm=llm, chain_type="stuff")

            # Run the question-answering chain with the documents and the user's prompt
            with get_openai_callback() as cb:
                response = chain.run(input_documents=docs, question=prompt)
                # This print statement is for debugging purposes
                print(cb)
            # st.write(response)
            # st.write(docs)

            # Append the QA chain response as an AI message
            qa_response_message = AIMessage(content=response)
            st.session_state.chat_history.append(qa_response_message)

        except Exception as e:
            st.error(f"An error occurred: {e}")

        # Clear the input after processing
        prompt_placeholder.text_area("Ask questions about your PDF:", value='', placeholder="Send a message", key="pdf_prompt")

    # Save chat history
    with open(chat_history_file, "wb") as f:
        pickle.dump(st.session_state.chat_history, f)

    # Display the entire chat
    chat_content = ""
    for message in st.session_state.chat_history:
        if isinstance(message, HumanMessage):
            role = "User"  # Set the role manually for HumanMessage
            content = message.content  # Access the content attribute directly
        elif isinstance(message, AIMessage):
            role = "AI"  # Set the role manually for AIMessage
            content = message.content  # Access the content attribute directly
        else:
            # Handle other types of messages or raise an error
            role = "Unknown"
            content = "Unsupported message type"

        chat_content += f"
**{role}:** {content}
"
    st.markdown(chat_content, unsafe_allow_html=True)

if name == 'main':
main()

I have a simlar problem with running chat_model = ChatOpenAI(...)  # Initialize with your parameters. Is it because the latest openai v1.1.1 change has not been adapted into langchain.chat_models.ChatOpenAI? Does anyone can help?


ZkCoordinator needs an injection test

As noted in https://github.com/druid-io/druid/pull/1913

Did #1913 fix this?

Closing as fixed


Update to node16 required

Github reports a warning when using this:
The following actions uses node12 which is deprecated and will be forced to run on node16<EMAIL_ADDRESS>Could you please update the workflow action?

Hi @christoph-nimagna - Thanks for the report! New version will be available shortly!

The new version should be available now. To be safe, I decided to move to v2 with these changes (the first commits in the repo in over a year).
Could you test to see if everything works as expected? In your workflow file, change:
-- uses: fbactions/setup-winsdk@v1
+- uses: fbactions/setup-winsdk@v2

@friederbluemle - works like a charm! Thanks for the swift fix and for maintaining this build action! 💪 🚀


Automated Package Upgrade

:tada: This PR is included in version 12.0.32 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add inbound parse webhook example

Fixes
#787
Checklist

[x]  I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added in line documentation to the code I modified

Short description of what this PR does:

This is a docker container that can receive Inbound Parse webhooks from SendGrid

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

@reedsa I don't have a cloud environment at the moment to document everything in a cloud deployment.  I added Kubernetes manifests and documented how to deploy using Kubernetes running locally.

Hello @aroach,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

@aroach Awesome work on the docs! Thanks for the extra effort on this one, I know there was a lot to take care of. 💯 👍
Not certain if another issue should be created to cover the deployment specific stuff at a later time?

@reedsa let me see if I can get setup for something to document it
tomorrow.
On Tue, Oct 9, 2018 at 8:53 PM Elmer Thomas<EMAIL_ADDRESS>wrote:

Merged #792 https://github.com/sendgrid/sendgrid-nodejs/pull/792 into
master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-nodejs/pull/792#event-1894401166,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACL-snWtWVYFovuDC_VB4E4WLqgImqQtks5ujWFAgaJpZM4XNytF
.


[SVCS-895] Upgrade Tornado From 4.3 to 5.1.1

PR Description TBD

Coverage increased (+0.01%) to 91.863% when pulling 4e5656eed288d7088f50135af19e7e53714e710e on cslzchen:feature/upgrade-tornado into 2cc62a1ef2c426238271d5d58a4473acecf0878e on CenterForOpenScience:develop.


Add a timestamp endpoint to SCT QueryService

Is your feature request related to a problem? Please describe.
The SCT service stores timestamps for each block height, but this information is not accessible to clients. Users would like to know things like transaction times, which could be identified by block timestamp. Graph axes such as candlestick graphs could be labelled with timestamps.
Describe the solution you'd like
Add a timestamp endpoint to the SCT QueryService that allows querying the timestamp of a specific block height
Describe alternatives you've considered
Include block height timestamps in other queries that mention block heights, such as the CandlestickData query, or a CompactBlock query

i think a lower resolution such as epoch timestamp would also be fine, for tx logs or graph purposes

Is this blocking on the web side @turbocrime ? If it is a nice to have we won't have bandwidth proximally

closed by #4523


Feature Request: ⬇ button to appear when scrolled many pages up

Description of the new feature/enhancement
Sometimes people using the Terminal need to scroll very far up, and read a lot of previous output. They then want to return immediately all the way to the bottom of the terminal, to see the latest output.
Currently: users scroll a lot to get to the bottom of the terminal.
Proposed: when scrolled more than X lines up, a ⬇ button should appear that scrolls to UI all the way down to the latest output. I'll leave the UI discussion up to you.

I'm going to tag this as "extensibility", because this seems like a really good extension idea. Maybe not something we'd build ourselves, but a great optional feature someone could add. Plus, it has a lot of functionality I'd want to see our extensions support.


loss todo ico

I'm unable to reproduce this issue. Could you please review your CSS snippets for any potential style conflicts or cascading interferences?

It conflicts with headingIndex

Thank you for your feedback. Already solved. You can download the latest theme.css file and replace your local version for immediate resolution.


Allow authoring and fix abstract slot inference

Allow authoring of static abstracts in interfaces

Apply abstract slot inference

Tested manually, it's OK to merge this into the PR I think
TODOs to add

[ ] Check that we can't author static abstracts in classes or structs
[x] Add tests for authoring static abstracts in interfaces
[x] Add tests for abstract slot inference

Tested manually on
type IAdditionOperator<'T> =
    static abstract op_Addition: 'T * 'T -> 'T

type C() =
    interface IAdditionOperator<C> with
        static member op_Addition(x, y) = C()

and then by inspecting generated IL.

I'm getting
        "Feature 'static abstract interface members' is not supported by target runtime." }]

for all the tests - how do we run them on .NET 7?


about class StateSpace

At class StateSpace in Controller.py, many functions e.g. embedding_encode(self, id, value) use state = self[id], I am confused that if it is legal in python ? how could we index self?
another question: why the variable state here is a Dictionary? shouldn't it be a StateSpace object?

I've declared the getitem method inside of StateSpace. After this, the entire class can behave as an array, and therefore be indexed just as above. This is just a convenience function, you can of course simply uses the expanded version as below.
def __getitem__(self, id):
        return self.states[id % self.size]

The benefit is that when the id is being received by the caller, it can be from multiple cells in a loop, so I don't need to worry about which cyclic index this is when I use self[id]. It's a somewhat roundabout way to do something safely without having the client worry about checks.
As to the second question, the StateSpace is a wrapper over all of the states it holds. It internally uses a dictionary. Therefore the client only needs one StateSpace, register all the states to it and then use it to get the states ready for use by the RNNs or for viewing.
Can you list the line where you feel it should be replaced?

I forgot another question.
At StateSpace.embedding_encode(self, id, value) function, line 85 in controller.py,
:one_hot[np.arange(1), value_idx] = value_idx + 1,
I think the expression should be : one_hot[np.arange(1), value_idx] =  1 because one hot vector should be
binary. what do you think so

Ah ordinarily you would be correct. Originally, the code did one hot the data, and then passed it to the RNN without an embedding step, as in the data was passed as a one hot over time.
The paper suggests using embedding matrices to simplify learning and make it correct.
Now that "one hot" function actually returns an integer to index the embedding matrix instead.
I should rename that function. It's not a one hot anymore after all.


Recurring Integration Http vs Https

I am unable to get the data from UAT/TEST environment using dequeue job but there is no issue with getting data from dev environment. Download location that is returned from dev environment contains "Https://" but from test environment contains "Http://". I have seen in this article that, URI scheme has to be set properly in middleware while accessing the test environment. How to set the URI scheme in POSTMAN tool?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2535f3a8-a225-3925-fea4-b22769ef90d0
Version Independent ID: 99dbbaab-c4e6-925b-0fb9-e444af567aa6
Content: Recurring integrations - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/data-entities/recurring-integrations.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

@Sunil-Garg : Can you take a look? Thanks.

Created bug 353172 to track this issue.


relation field: add check_value

closes https://github.com/inveniosoftware/invenio-rdm-records/issues/760
needs https://github.com/inveniosoftware/invenio-records/pull/264

Im still unsure about the whole changes but code wise LGTM. I understand basically we want to only link publication resource (sub)types. I am wondering will it allow to link the publication type? since it is not a subtype?

We declare the value we want to search for when defining the relation:
 resource_type=PIDRelation(
            ...,
            value_check=dict(props=dict(relation_type='depositable')),
)

This will  check if the vocabulary value we are trying to submit is marked as  depositable, like this:
- id: publication
  props:
    relation_type: depositable

And it will fail if it's not matching. (If there is no relation_type declared in the props we assume it's standard and it won't fail).
If we want to accept only publication types, we could by doing:
 resource_type=PIDRelation(
            ...,
            value_check=dict(props=dict(type='publication')),
)

But it's not our case ATM.

Maybe adding some tests at REST API level to see the failure if the record contains a relation to a non linkable resource type might be helpful :)

Tests for this are added in invenio-records (here).


SCM Sync: Consider showing short diff, per commit and file

Testing #194016
We currently show 2 files changed, 8 insertions(+), 11 deletions(-) only in the All Changes group, and nothing for commits and files:

I would suggest simplifying and unifying this information, showing it on all 3 levels. A short label like +8 -11 (2). The user should be able to hover this label and see the full description of it.

+8 -11 (2) makes sense to me!

That makes sense to me as well. What I think we are struggling with is that:

Commit - on one line we have to fit commit message, author, and short diff
File - on one line we have to fit file name, file path, short diff and file decoration

This quite some information that we need to fit in one line so I would like for us to explore some other potential layouts (ex: maybe use two lines for commits) that enables us to include more information.

Closing this issue as now we should a statistics pill for each commit.


upgrade to Node 12

(These changes break Node 6. See below)
Fix for Node 12:
General information about the V8 API changes between Node 10 and 12

Summary https://docs.google.com/document/d/1g8JFi8T_oAE_7uAri7Njtig7fKaPDfotU6huOa1alds/edit
"v8/node deprecated APIs and how to handle them" https://github.com/bcoin-org/bcrypto/issues/7

These fixes frequently involve the following:

the isolate object - get via Isolate* isolate = v8::Isolate::GetCurrent()
the context object - get via Local<Context> context = Nan::GetCurrentContext()

https://github.com/nodejs/nan/blob/master/doc/v8_misc.md#nangetcurrentcontext

converting MaybeLocal types returned by many Nan functions to Local using maybeLocalObj.ToLocalChecked()

Specific error fixes and my specific sources of information

from s->WriteUtf8(buf) to s->WriteUtf8(isolate, buf)

https://github.com/nodejs/node/pull/22531/files#diff-f77b9fed8a3d7f7899ae2316e2bb3230R24

from X->ToString() to Nan::To<String>(X).ToLocalChecked()

https://github.com/astro/node-expat/pull/196/files#diff-2b6f822b34553a6cadd4627d4dc9e84fR29

from X->ToObject() to Nan::To<Object>(X).ToLocalChecked()

https://github.com/astro/node-expat/pull/196/files#diff-2b6f822b34553a6cadd4627d4dc9e84fR29

from X->ToInteger() to Nan::To<Integer>(X).ToLocalChecked()

following the pattern of the above
see also https://github.com/nodejs/nan/blob/master/test/cpp/converters.cpp#L37

from X->GetOwnPropertyNames() to Nan::GetOwnPropertyNames(cache).ToLocalChecked()

https://github.com/nodejs/nan/blob/master/doc/maybe_types.md#api_nan_get_own_property_names
or, to X->GetOwnPropertyNames(context).ToLocalChecked()

https://stackoverflow.com/questions/40533193/v8-c-how-to-get-object-key-values-provided-as-arguments
https://v8docs.nodesource.com/node-8.11/db/d85/classv8_1_1_object.html#a79a6e4d66049b9aa648ed4dfdb23e6eb

from X->GetFunction() to X->GetFunction(context).ToLocalChecked()

https://www.nearform.com/page/7/?cat=-1
https://nodejs.org/api/addons.html
see alternative https://github.com/astro/node-expat/pull/196/files#diff-2b6f822b34553a6cadd4627d4dc9e84fR29

from String::Utf8Value X(Y) to Nan::Utf8Value X(Y)

https://github.com/nodejs/nan/blob/master/doc/v8_misc.md#api_nan_utf8_string
https://github.com/microsoft/node-pty/pull/288/files#diff-8fc148e2d2217506b0e485a3033310feR174
https://v8docs.nodesource.com/node-8.11/d4/d1b/classv8_1_1_string_1_1_utf8_value.html

from X->Int32Value() to X->Int32Value(context).FromJust()

https://github.com/microsoft/node-pty/pull/288/files#diff-8fc148e2d2217506b0e485a3033310feR174

from X->IntegerValue() to X->IntegerValue(context).FromJust()

https://github.com/microsoft/node-pty/pull/288/files#diff-32a8618c58cb849e9b5535ba19ffacc5R186

from kApiPointerSize to kApiSystemPointerSize

https://github.com/v8/v8/commit/81a654e7a0d8dc86896dd94773ad839b6ffc2383#diff-4637d1e4b9c1cbab90cbfd580aceff15R29

from isolate->GetCpuProfiler() to ... (larger fix, because whereas the V8 engine previously had just 1 CpuProfiler, it now has multiple)

https://github.com/nodejs/node/pull/18959/files
https://v8docs.nodesource.com/node-10.6/d2/d34/classv8_1_1_cpu_profiler.html

from const Handle<String> X to const Local<String> X

https://electronjs.org/blog/nodejs-native-addons-and-electron-5
https://github.com/microsoft/node-pty/pull/288/files#diff-8fc148e2d2217506b0e485a3033310feR174

update node-gyp from v3 to v4

https://github.com/nodejs/node-gyp/issues/792
tbh I don't know in detail how this solves the problem

Specific deprecation warning fixes and my specific sources of information

from X->Get(Y) to Nan::Get(X, Y).ToLocalChecked()

https://github.com/nodejs/nan/blob/master/doc/maybe_types.md#nanget

Notes

'module->Set' will soon be deprecated - @richardlau has provided an example below of how to  update our code

a different example https://github.com/astro/node-expat/pull/196/files#diff-2b6f822b34553a6cadd4627d4dc9e84fR29

These changes break Node 6:
Problem:
Error output from `npm i` on Node 6
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:21:10: fatal error: 'utility' file not found
#include <utility>
         ^~~~~~~~~
1 warning and 1 error generated.
make: *** [Release/obj.target/appmetrics/geni/appmetrics.o] Error 1

This can be resolved simply following https://github.com/nodejs/node-gyp/issues/1564 and https://github.com/nodejs/node-gyp/issues/1574. However, that reveals that my upgrade to Node 12 is incompatible with Node 6:
Error output from `CXXFLAGS="-mmacosx-version-min=10.9" LDFLAGS="-mmacosx-version-min=10.9" npm i` on Node 6
In file included from ../src/plugins/node/prof/watchdog.h:21:
../src/plugins/node/prof/compat-inl.h:308:38: error: no member named 'New' in 'v8::CpuProfiler'
    cpu_profiler_ = v8::CpuProfiler::New(isolate);
                    ~~~~~~~~~~~~~~~~~^
Release/obj.target/appmetrics/geni/appmetrics.cpp:467:29: warning: 'Call' is deprecated [-Wdeprecated-declarations]
        listener->callback->Call(argc, argv);
                            ^
../node_modules/nan/nan.h:1673:3: note: 'Call' has been explicitly marked deprecated here
  NAN_DEPRECATED inline v8::Local<v8::Value>
  ^
../node_modules/nan/nan.h:103:40: note: expanded from macro 'NAN_DEPRECATED'
# define NAN_DEPRECATED __attribute__((deprecated))
                                       ^
Release/obj.target/appmetrics/geni/appmetrics.cpp:530:27: error: no matching constructor for initialization of
      'String::Utf8Value'
        String::Utf8Value str(isolate, Nan::To<String>(info[0]).ToLocalChecked());
                          ^   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2428:14: note: candidate constructor not viable:
      requires single argument 'obj', but 2 arguments were provided
    explicit Utf8Value(Local<v8::Value> obj);
             ^
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2438:5: note: candidate constructor not viable:
      requires 1 argument, but 2 were provided
    Utf8Value(const Utf8Value&);
    ^
Release/obj.target/appmetrics/geni/appmetrics.cpp:542:27: error: no matching constructor for initialization of
      'String::Utf8Value'
        String::Utf8Value str(isolate, Nan::To<String>(info[1]).ToLocalChecked());
                          ^   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2428:14: note: candidate constructor not viable:
      requires single argument 'obj', but 2 arguments were provided
    explicit Utf8Value(Local<v8::Value> obj);
             ^
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2438:5: note: candidate constructor not viable:
      requires 1 argument, but 2 were provided
    Utf8Value(const Utf8Value&);
    ^
Release/obj.target/appmetrics/geni/appmetrics.cpp:570:27: error: no matching constructor for initialization of
      'String::Utf8Value'
        String::Utf8Value topicArg(isolate, Nan::To<String>(info[0]).ToLocalChecked());
                          ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2428:14: note: candidate constructor not viable:
      requires single argument 'obj', but 2 arguments were provided
    explicit Utf8Value(Local<v8::Value> obj);
             ^
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2438:5: note: candidate constructor not viable:
      requires 1 argument, but 2 were provided
    Utf8Value(const Utf8Value&);
    ^
Release/obj.target/appmetrics/geni/appmetrics.cpp:572:27: error: no matching constructor for initialization of
      'String::Utf8Value'
        String::Utf8Value commandArg(isolate, Nan::To<String>(info[1]).ToLocalChecked());
                          ^          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2428:14: note: candidate constructor not viable:
      requires single argument 'obj', but 2 arguments were provided
    explicit Utf8Value(Local<v8::Value> obj);
             ^
/Users/richard.waller@ibm.com/.node-gyp/6.17.1/include/node/v8.h:2438:5: note: candidate constructor not viable:
      requires 1 argument, but 2 were provided
    Utf8Value(const Utf8Value&);
    ^
1 warning and 5 errors generated.
make: *** [Release/obj.target/appmetrics/geni/appmetrics.o] Error 1

Solution 1
Drop Node 6. This PR achieves this.
Solution 2
Support Node 6 alongside 8, 10, and 12. We would need to spend considerable time converting probably ~60% of the fixes in this PR to use NAN and if blocks.
Ideal long-term solution
Rewrite much of appmetrics using N-API, but that is beyond the scope of this issue.

I think option 1 is ok given that Node 6 is no longer in LTS.  Agree that a longer term plan to support N-Api would be the right one to avoid issues in the future.  We should document that Node 6 is no longer supported and the last known working appmetrics level

Dropping 6.x support sounds like a good idea to me, but it requires bumping the appmetrics major (because dropping 6 support is a breaking change for people using node 6). That's NBD, but don't forget to do the book keeping: it eans finding all the other runtimes packages (prometheus, etc) that depend on appmetrics, and publishing an updated major of them, that depends on the new appmetrics major.

The issues raised in the comments will be dealt with as separate PR's

Travis problem
Build intermittently failing on Windows (various versions of Node) with seemingly the same error in a test in http-outbound-probe-test.js.
Error output:
[Thu May 30 09:47:02 2019] com.ibm.diagnostics.healthcenter.mqtt INFO: Connecting to broker localhost:1883
Error: connect ECONNREFUSED <IP_ADDRESS>:49860
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1191:14)
{ name: 'TAP',
  autoend: true,
  errno: 'ECONNREFUSED',
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 49860,
  tapCaught: 'uncaughtException',
  test: 'TAP' }
Error: socket hang up
    at createHangUpError (_http_client.js:342:15)
    at Socket.socketCloseListener (_http_client.js:377:23)
    at emitOne (events.js:121:20)
    at Socket.emit (events.js:211:7)
    at TCP._handle.close [as _onclose] (net.js:561:12)
{ name: 'TAP',
  autoend: true,
  code: 'ECONNRESET',
  tapCaught: 'uncaughtException',
  test: 'TAP' }
    # time=92.67ms
not ok 3 - tests/probes/http-outbound-probe-test.js # time=5743.445ms
  ---
  timeout: 120000
  file: tests/probes/http-outbound-probe-test.js
  childId: 2
  command: 'C:\ProgramData\nvs\node\8.16.0\x64\node.exe'
  args:
    - '-r'
    - 'C:\Users\travis\build\RuntimeTools\appmetrics\node_modules\esm\esm.js'
    - tests/probes/http-outbound-probe-test.js
  stdio:
    - 0
    - pipe
    - 2
  cwd: 'C:\Users\travis\build\RuntimeTools\appmetrics'
  exitCode: 1
  ...

Solution

On Travis, Appmetrics passes tests ~60% time on Windows with Node 8, 10, and 12.
We have manually verified that Appmetrics passes tests on Windows with Node 8, 10, and 12.
Travis says "Windows builds are in early access stage"

Therefore, we will keep running the Windows tests on Travis but flag them as allowed to fail.

That way we will at least be able to monitor whether the Windows tests pass more consistently when Windows builds are no longer just early access.
If the tests still fail intermittently at that point, then we should investigate these failures further

I've updated the docs, so this PR also resolves https://github.com/RuntimeTools/appmetrics/issues/571


Feature Request: Add example of adding handlers/commands to demo app

Could an example of how to add additional handlers/commands be added to the demo app so consumers of the new architecture (as a package) know how they can add their own items that aren't core to the Response package?
More of a question, as an side to this, would consumers be able to override certainly handlers/actions? For example, at the moment we have a custom channel name convention for incidents -- instead of an incrementing number as the default channel name, we switched to date time to work around the issue in a development environment where the channels already existed.

Yes, I really like this idea - better than documentation, as we can write tests for it and check that new releases haven't regressed anything.

More of a question, as an side to this, would consumers be able to override certainly handlers/actions? For example, at the moment we have a custom channel name convention for incidents -- instead of an incrementing number as the default channel name, we switched to date time to work around the issue in a development environment where the channels already existed.

Definitely. We have this in the Monzo private fork, so we'll definitely need to port this over to the response package - I'm thinking of some kind of callback/hook that lets users plug in their own logic.
An even cooler feature I had an idea for was automatically generating a channel name from the incident name, prefixed by date - e.g. "External API latency high" could become #inc-jul15-ext-api-lat or something - we've found that there is a minute or two wasted at the beginning of incidents by having to rename the channel


minor: Add deploy with balena button to README

Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

@balena-ci rebase

I fixed up the formatting of the balena.yml so we should be good to go. I would approve but you added the draft label so I leave it to you!

Yeah, I wanted to add some sane defaults or prompts for the user. Added persistent logging as well.

@balena-ci I self-certify!


feat(nuxt): warn about ignored char while parsing route segment

🔗 Linked issue
fix #30392

📚 Description
This could go into a patch version IMO.
This PR add a warnign (not sure why this was removed) when a a character is ignored when building route path

I see, maybe should we add a documentation link to avoid having a long warning ?
I may have another idea in addition to this.
How about storing all build warnings somewhere and let the devtools display them, show the stack trace and why this happened ?
IMO, the devtools is way more UX friendly than the old CLI


Can't edit settings.json

Description of the problem or steps to reproduce
Every time I'm editing & saving ~/.config/micro/settings.json it is being reset by something. And the editor prompts with The file on the disk has changed.. Permissions are correct. Ownership is correct. Doing this as the owner. Nothing is using the file, according to sudo fuser ~/.config/micro/settings.json.
Specifications

OS: Bazzite Holographic 40 (Silverblue) x86_64
Terminal: Ptyxis 46.6
Commit hash: 225927b

Micro does some slightly annoying auto-formatting on the file whenever settings are changed (with the set command or config.SetGlobalOption in Lua) even if the new value for the setting is the same as the existing one. You might have have some plugin that is programmatically changing a setting.
You can disable "The file on the disk has changed" prompt by setting set reload auto.

Also 225927b is 2.0.11 i.e. a pretty old version of micro. Since 2.0.14 we have the fix #3009 which avoids unexpected rewriting of settings.json in most cases.

a pretty old version of micro

I am currently using micro from Fedora repo. Will contact maintainers to ask them to update it.

I am currently using micro from Fedora repo. Will contact maintainers to ask them to update it.

Hm, right...pretty old, what they package right now. Even debian offers a newer package from testing onwards.
Anyway, there is already the "bug" reported/identified by the upstream release monitor (2238002) of Fedora, while for Debian it is 1080422.
Nevertheless you can use our generic and static artifact for v2.0.14 (micro-2.0.14-linux64-static.tar.gz + micro-2.0.14-linux64-static.tar.gz.sha) as long as it isn't available as package in your favorite distribution.
We should be ready to close this issue, am I right?


fix ToC hash links

This should make all table of content links clickable 👌
GitHub's parser removes all non-word characters instead of repacing them with a dash (see here).

thank you!! honestly we need a better auto TOC generator.. :(  I have an issue here asking for suggestions https://github.com/sw-yx/react-typescript-cheatsheet/issues/2


C: Check whether memory allocation was successful

What problem is this PR intended to solve?
The C extension didn't check whether memory allocations were successful (= malloc/calloc return a value other than NULL). In the rare case, that memory allocation fails, this leads to undefined behavior, which usually results in a segmentation fault.
This problem was discovered using the static analyzer (-fanalyzer) of GCC.

Have you included adequate test coverage?

There is no easy way to test this behavior, hence I omitted test cases.
Does this change affect the behavior of either the C or the Java implementations?
If no more memory is available rb_eNoMemError is raised. As far as I can tell, it is not necessary to change the Java implementation, since Java would automatically raise an OutOfMemoryError exception.

Wouldn't it make sense to use ruby_xmalloc instead?

Good idea. I don't have much experience with the Ruby API and wasn't aware of this function. I'll change it.

+1 we should have been using ruby_xmalloc all along (an in fact this is what we configure libxml2 to use internally).

@Garfield96 Thanks so much for submitting this PR. Are you comfortable making the change that Lars suggested? If not, that's OK and I'm happy to do it.
Please also note there exists a ruby_xcalloc that would replace calloc.

I started to implement it with ruby allocators and I'll push the new version tomorrow.

Ruby allocators are now used everywhere, except for vasprintf. I tried to adapt the vasprintf implementation in nokogiri.c but it results in a segmentation fault. To not block this PR, I'll open another PR for vasprintf.

Thanks! I'm on vacation for a few days but will take a look as soon as I
can.
On Sat, Mar 19, 2022, 8:32 AM Christian Menges @.***>
wrote:

Ruby allocators are now used everywhere, except for vasprintf. I tried to
adapt the vasprintf implementation in nokogiri.c but it results in a
segmentation fault. To not block this PR, I'll open another PR for
vasprintf.
—
Reply to this email directly, view it on GitHub
https://github.com/sparklemotion/nokogiri/pull/2480#issuecomment-1073001923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACADYBB5CH6SFDYPJNBU3VAXCMRANCNFSM5RBTP5SA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

I pushed a second commit that uses ruby_xfree to free the nokogiriXsltStylesheetTuple struct in xslt_stylesheet.c, which I think you missed. Otherwise this looks good to me.
I've pinged @larskanis for his review as well, because he's very familiar with memory handling in C extensions.

Appveyor is green, not sure why that's not reflected here in the PR checks. I'll give Lars a chance to comment, but I'm pretty confident this is good to merge so I'm planning to do that this weekend. Thank you!

Merged! Thank you for your contribution, @Garfield96. I'll credit you in the CHANGELOG.


Add Support for Major Chromium Based Browsers

A good feature that would be useful and lower amount of useless bug reports (main error being: 'cannot find Chrome binary') would be support for other chromium based browsers. Looking into the documentation for the Selenium WebDriver, any browser that is based off of Chromium (two major ones being Opera and Microsoft Edge), can use the chromedriver packaged with the tool.
As for implementing this, A simple config option for which browser to use will work. Then just adding a check for what browser is picked, and lastly setting the binary_location to where the specified browser is located.

This is a very good suggestion but program is seleniumless now so not very helpful now lol


Facing error after upgrading to 1.1.2

Receiving this error after upgrading to version 1.1.2
6:63: error: 'self' captured by a closure before all members were initialized
self.flutterViewController = flutterViewController ?? self.flutterViewController

Thanks for filing the issue. Unfortunately, I am unable to reproduce it. Can you run flutter clean and see if that resolves the issue?
If it doesn’t, what version of macOS are you running your application on? What is your minimum deployment target?
Also, does your app use any other macOS plugins? Have you made any changes to any Swift files in your project? I assume you have modified your MainFlutterWindow.swift file. Can you post it here?

Hello thank you for quick response.
I'm try to run the example app from the repo. So haven't made any modifications to the code
Running on:
Flutter (Channel stable, 3.7.10, on Mac OS X 10.15.7 19H2026 darwin-x64,
locale en-GB)
Xcode - develop for iOS and macOS (Xcode 12.3)
CocoaPods version 1.11.3

I can see that you’re running a much older version of macOS than I am, so at first I suspected you were running into some sort of conditional compilation problem, but after checking the code I don’t think that’s the case.
Alright, could you perhaps clone the macos_window_utils repo on your machine and see if the same issue arises when running its example project? flutter_acrylic depends on this package for its macOS functionality, so technically speaking, this is the package that is causing the bug.
If you observe the same issue, could you open the macos/Classes/MacOSWindowUtilsViewController.swift file, and change line 16 from
self.flutterViewController = flutterViewController ?? self.flutterViewController

to
self.flutterViewController = flutterViewController ?? FlutterViewController()

and see if that resolves the issue?

I made the mentioned changes
Launching lib/main.dart on macOS in debug mode...
Building macOS application...
Error waiting for a debug connection: The log reader stopped unexpectedly, or never started.
Error launching application on macOS.

Interesting. Does running flutter clean help?

No luck, same issue

Does running the macos_window_utils_example.app.zip (from Release v1.1.2) cause the same issue, or does it only occur when the app has been compiled on your end?

Yes this worked, so yes maybe the compiling issue

With
self.flutterViewController = flutterViewController ?? FlutterViewController()
I tried running it with xcode (was not able to run without this line), I was able to compile but it only shows as blank screen.

Same issue on debug and release mode

For time being, I've downgraded to version 1.1.1 which works fine on my device. Thank you for the help!

Be sure to version lock macos_window_utils to 1.1.1 by adding macos_window_utils: 1.1.1 to your pubspec.yaml (not macos_window_utils: ^1.1.1, since you don’t want pub to be allowed to upgrade to 1.1.2).


VSCode v1.51.1 User for Window. Intellisense only show 1 suggestion.

VSCode Version: 1.51.1
OS Version: Window 7 64bit

Steps to Reproduce:
1.Open Angular 8 or 10 typescript file
2. In a method type this.  Only one suggestion shows up.
This works fine on version 1.50.1.
I have uninstalled v1.51.1 and reinstalled v1.50.1 intellisense working as expected

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

Is there only one suggestion or is there only one visible suggestion? Have you tried resizing the suggestions?

This happened to me. Only one intellisense suggestion. It turned out the window had been resized to only show one item, as mjvbz mentioned.

Is there only one suggestion or is there only one visible suggestion? Have you tried resizing the suggestions?

Thanks that was what it was. Suggest setting a reasonable default size to start!
Thanks Again.. (I Love VSCode)
-Greg

Solution was resize suggestion. Closing

Is there only one suggestion or is there only one visible suggestion? Have you tried resizing the suggestions?

Life savior!


[buck-yeh-bux] fix clang-cl builds

Please submit a PR to @buck-yeh https://github.com/buck-yeh/bux ; they have been very responsive.
Thanks!


The system shall establish a secure connection to VA sites for exchange of patient Identity information  by following VA standards for privacy.

Per conversation with Paul DeSantis (1/14/16): This feature is out of scope. There is a known vulnerability for TCP/IP in HL7 messaging. Whatever security is in place for the VistA HL7 subscriber links will be maintained for this prototype.


[SPARK-43040][SQL] Improve TimestampNTZ type support in JDBC data source

What changes were proposed in this pull request?

https://github.com/apache/spark/pull/36726 supports TimestampNTZ type in JDBC data source and https://github.com/apache/spark/pull/37013 applies a fix to pass more test cases with H2.
The problem is that Java Timestamp is a poorly defined class and different JDBC drivers implement "getTimestamp" and "setTimestamp" with different expected behaviors in mind. The general conversion implementation would work with some JDBC dialects and their drivers but not others. This issue is discovered when testing with PostgreSQL database.
This PR adds a dialect parameter to makeGetter for applying dialect specific conversions when reading a Java Timestamp into TimestampNTZType. makeSetter already has a dialect field and we will use that for converting back to Java Timestamp.
Why are the changes needed?

Fix TimestampNTZ support for PostgreSQL. Allows other JDBC dialects to provide dialect specific implementation for
converting between Java Timestamp and Spark TimestampNTZType.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Existing unit test.
I added new test cases for PostgresIntegrationSuite to cover TimestampNTZ read and writes.

cc @cloud-fan @dongjoon-hyun @beliefer

also cc @yaooqinn

thanks, merging to master!


Upload slides from speakers?

Can we upload the slides from the speakers into each session?  Would we prefer to upload the PPT or convert to PDF?

Should we edit each session node to include links to the presentation and recording or should we take the schedule view and link from there... or both?

I think editing the session node for the presentation and recording would
be great. If we communicate where everything is, that should give people
what they need.

Working to find a cure for blood cancers. Donate and save a life! -
http://pages.teamintraining.org/nca/yourway15/dsheffler
On Mon, Jul 27, 2015 at 9:16 AM, Jessica Dearie<EMAIL_ADDRESS>wrote:

Should we edit each session node to include links to the presentation and
recording or should we take the schedule view and link from there... or
both?
—
Reply to this email directly or view it on GitHub
https://github.com/Drupal4Gov/Drupal-GovCon-2015/issues/143#issuecomment-125201109
.

@dsheffler and @jdearie this is now OBE since the youtube channel is up, right? I added the keynote links to the keynotes, too. But feel like this can be closed.

Don't we want the slides available as well as the recordings?
-James

James King, MLS, FSLA
http://about.me/cmndr_king
-------- Original message --------
From: Kirsten<EMAIL_ADDRESS>Date:08/06/2015 9:48 AM (GMT-05:00)
To: Drupal4Gov/Drupal-GovCon-2015<EMAIL_ADDRESS>Cc: "King, James (NIH/OD/ORS) [E]"<EMAIL_ADDRESS>Subject: Re: [Drupal-GovCon-2015] Upload slides from speakers? (#143)
@dshefflerhttps://github.com/dsheffler and @jdeariehttps://github.com/jdearie this is now OBE since the youtube channel is up, right? I added the keynote links to the keynotes, too. But feel like this can be closed.
—
Reply to this email directly or view it on GitHubhttps://github.com/Drupal4Gov/Drupal-GovCon-2015/issues/143#issuecomment-128370025.

We've been putting the recordings in our YouTube channel and embedding them on the session pages. We've also been uploading the slides into the session nodes as well.
We don't have all of them yet... I sent another email to the speakers this morning - only to those we have not received anything from yet.
Maybe we give it through the weekend and then close this issue ... we can still update sessions if stuff trickles in.

no slides for the panel that I know of.
and not sure we would need them as the video does show them.
If we have them, I can link to the slideshare. I believe I have them for
Darrell.
On Thu, Aug 6, 2015 at 10:19 AM, cmndr-king<EMAIL_ADDRESS>wrote:

Don't we want the slides available as well as the recordings?
-James

James King, MLS, FSLA
http://about.me/cmndr_king
-------- Original message --------
From: Kirsten<EMAIL_ADDRESS>Date:08/06/2015 9:48 AM (GMT-05:00)
To: Drupal4Gov/Drupal-GovCon-2015<EMAIL_ADDRESS>Cc: "King, James (NIH/OD/ORS) [E]"<EMAIL_ADDRESS>Subject: Re: [Drupal-GovCon-2015] Upload slides from speakers? (#143)
@dshefflerhttps://github.com/dsheffler and @jdearie<
https://github.com/jdearie> this is now OBE since the youtube channel is
up, right? I added the keynote links to the keynotes, too. But feel like
this can be closed.
—
Reply to this email directly or view it on GitHub<
https://github.com/Drupal4Gov/Drupal-GovCon-2015/issues/143#issuecomment-128370025

.

—
Reply to this email directly or view it on GitHub
https://github.com/Drupal4Gov/Drupal-GovCon-2015/issues/143#issuecomment-128387289
.

--
Kirsten
http://uniongal.blogspot.com
Join Uniongal on Reddit http://www.reddit.com/r/Uniongal/


Update supported Python versions

Python 2.6, 3.3 and 3.4 are EOL and no longer receiving security updates (or any updates) from the core Python team.

Version
Release date
Supported until

2.6
2008-10-01
2013-10-29

2.7
2010-07-03
2020-01-01

3.0
2008-12-03
2009-06-27

3.1
2009-06-27
2012-04-09

3.2
2011-02-20
2016-02-27

3.3
2012-09-29
2017-09-29

3.4
2014-03-16
2019-03-16

Source: https://en.wikipedia.org/wiki/CPython#Version_history
Also add new versions 3.6 and 3.7.

@nikhilkumarsingh Hello, please could you review and merge this? Thank you! 🎉

Closing due to lack of interest.


Remove the comments.

@jethrohitosis2314 I think you have certain experience in programming and this issue seems rather trivial for you, hence closing this PR. Kindly find other issues in my projects that suits you better 😄


adding contributing.md

RFR @giantswarm/dev
Are we ok with using the guidelines mentioned here?

Do we need certificates of origin? Or is that more of a later thing?

Not sure, but does it hurt in any way?

If we don't need it, I wouldn't bother. It's definitely a barrier to entry.

I have a feeling that the discussion around CONTRIBUTING.md exceeds the task at hand. We should define a basic contribution policy. If there are controversial items, we should rather eliminate (=delete) them instead of adding more.

What is the state of this PR? Others kind of rely on it. Can we merge this? What needs to be done?

There's kind of an open question if we do want it to be such a "fully featured" CONTRIBUTING.md. I'd like to have it like I made it here, cause then we don't need to change it in the future. But I also don't want to ignore the comments made above, before giving them a chance to reply. So what do you guys think @marians @JosephSalisbury and maybe @teemow can chime in.

Had a little think, I'm not super certain having a file in every repo is the best way to handle this - would a page on the website work better? It means we don't have to update every repo when we change the contributing file, it can look a lot nicer, and then link to the contributing page in every readme
with regards to content, i think the current is fine

OS repos need this cause people might not visit the website, they might just find us on github or get a link to the repo and then go from there, having these files in the repo is pretty much common practice from what I've seen

We also wanted to have those political documents tracked with the source code for transparency reasons. Users always can comprehend changes of our policies.

ok, so if I get another LGTM on the content I'll be merging this

Okay, that's convinced me! Just need to make sure we update them all :D
LGTM

My https://github.com/giantswarm/example-opensource-repo/pull/2#discussion_r49810213 was ignored I guess. So no LGTM from me.

@xh3b4sd I thought that comment was about adopting a new way of working for all our dev team. I didn't realize it was meant as an addition to this file.
Thinking about it now, I'm not sure why you would want commits to be squashed together, though.  I think having commits of logical units makes the commit history already quite clean.
All in all, I think the 1 commit per 1 PR thing is quite a huge change and would need discussing more deeply with all of @giantswarm/dev. I'm ok with keeping this PR open until that is solved.

I just wanted to have a response, even I am not happy with the response. I don't mind. Go ahead.

@xh3b4sd I'm not the right person to give you a response on this, as I won't be one of the guys committing a lot ;) That's why I pinged the team. I think you do deserve a response that is more than my rough assessment above and told by someone who's actually gonna commit more stuff.

@puja108 @xh3b4sd
I totally agree on your comment about squashing and having a single comment (well described). I had to do it when I committed my PRs to open source projects.
So I'd say a big YES, because otherwise it would be a mess for us to handle their contributions. On the other hand, I'd suggest to include unit tests when possible to add them to our code. That's a common thing and I also recently did it  for CoreOS rkt.

Btw, most of this PR is from CoreOS etcd project. I'm still not sure why it is hard to review a PR that is several commits, if each commit is a logical unit of progress. I'd like to understand that and maybe also get some more input on this from @giantswarm/dev

This is not about the review process. It is about having a mess in master commit history. So optimally you would have one commit in master for each PR that was merged. In master this is then the logical unit you commit. One commit per PR. Reality also shows than humans are rather bad at doing proper commits that really make sense. IMO nobody is able to achieve this all the time. What everybody can achieve is to just squash all commits of the PR right before merging. Just to have a clean commit history within the master branch. When you look at https://github.com/coreos/etcd/commits/master, exactly that happens. At the end, the PR's all have one commit. Here I am not sure if this is by accident or on purpose, because when you force push the history within the PR you don't see if there were once multiple commits. Anyway, the history within master is more clean.

Ok, thanks for explaining the reasoning behind it. I can agree with that. What would be a good wording for that? is "please squash all commits of a PR into one before merging the PR" good?

IMO something along these lines should make it.

I added a bullet for unit tests like we talked about with @hectorj2f and commit squashing like talked about with @xh3b4sd and @hectorj2f
Any vetos? @giantswarm/dev

@puja108 Yes, I like the sentence "please squash all commits of a PR into one before merging the PR"

LGTM, but the checks for this PR fail. Don't know what that means.

that is strange, cause it's only adding a .md @marians you know why Travis is acting up?

Travis config was missing, because the branch was rather old.


Ship bundled JS licenses

Prerequisites

[X] I am using the latest version of Locust
[X] I am suggesting a new feature, not asking a question

Description
Without knowing too much about the current JS build chain: it would be lovely if the full text of the licenses of the shipped vendored libraries (not the build chain itself) were made available in, at least, the sdist, in a well-known location.
This could be a folder of the licenses with a sensible naming convention, or a more report-based, machine-readable single file.
Noted while looking at changes in https://github.com/conda-forge/locust-feedstock/pull/82

I dont know what needs to be done, but we definitely want conda support, so if you have any suggestion then it is very welcome.

Likely there is a plugin for this for the build chain: akin to https://www.npmjs.com/package/webpack-license-plugin

PRs welcome, but marking this stale.