Fix the alignment of Add New Post button header and close button on Media page in portal

Describe the bug
On the portal's media page header of Add New Post button is not aligned with the Close button.
To Reproduce
Steps to reproduce the behavior:

Go to 'Media'
Click on 'Add New Post'
See error

Screenshot

Expected behavior
The alignment of add new post header and the close button should be on the same level
Environment in which the issue is occurring

All environment

@ImRishabhRawat I am not able to understand the expected behaviour. You can also add a reference modal screenshot  from other pages according to which we can make changes to our current modal

As you are working on this issue so you also need to provide the breakdown of what all action items you will be doing to perform this task and how many hours you are expecting to complete it

Expected Time

[ ] Align the close button  to the same level as Add New Post heading.  30 min
[ ]  Remove the top blank card  and the outer card.  1-2 Hour
[ ] For testing and code review  2-3 hours

Closing the issue as it is completed.


Fix NPE

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

@beinan can you take a look?

We decided to close this PR in our sync meeting


Question: How do i build and debug

How do i set this project up for testing and hopefully contributing? When i tried, the UI folder is empty, if i copied resources from Swagger UI that didnt work; but then how do i test and debug it?

Can you take some screenshots of you Visual Studio solution and post it here

Here is how mine looks like, showing TestExplorer and SolutionExplorer

I was more wondering about how you debug it; where does it show the information i need to determine if it works or not:

I tried making changes, building and then pointing my actual API project to the nuget package but i just got a UI with missing assets and all a bit messed up.

Your test explorer is empty I will start focusing there...

@coderangerdan - I have this same question. I'd also like to know how to build Swashbuckle from the ground up.
I know when I'm looking at it, sitting in my Windows Explorer after checkout, there's a swagger-ui folder that is empty.. and there's a corresponding gitmodules file that targets that directory.
If I run this git command, it will properly hydrate that sub-directory...
git submodule update --init
I'm curious to know how these swagger-ui files get incorporated as assets in the final Swashbuckle.Core.DLL assembly - I don't see how it's mapped into it at all


Add stricter parameter types for Object.values and Object.entries

Fixes #20550
Added  | string | number | boolean | symbol to avoid falling into the second overload unnecessarily; Object.values on a primitive returns [] of these so it's effectively a never[] in that case (though we infer {}[]).
I'm also not too happy about returning any from builtin functions. --noImplicitAny won't help you there. The only case I found where the first overload won't match any more is Object.values(() => {}), in which case a return of {}[] is probably better since a function's unlikely to have the properties you intend to get.

@andy-ms We could just define values as values<T>(o: T): T[keyof T][]; (and similarly for entries). These definitions are pre-index types, and we can just do better now, I think.

@weswigham the problem there is the unsoundness because of structural non-sealed types
const a = { a: 'a string', b: 123 }
const b: Pick<typeof a, 'a'> = a
const values = Object.values(b) // string[]; but actually has a number inside too

@weswigham What would you think of that signature with T extends object? The function will technically work at runtime if you pass in a string, it's not really useful unless you're passing in an object.

@Kovensky any time you alias a type you lose information w.r.t. the index operator. But the thing is, I'm pretty sure that definition (at least for T extends object, like @andy-ms suggests), is more practically useful, since you'll get the types for all the fields we do know about (instead of the unsafe any). Honestly, your complaint is just a function of how the keyof type operator works; but the more tightly your types match your behavior, the more accurate it is. You use it on any type you've aliased (thus discarded information) and you'll get results for the alias, not the original type.


Only warn for same console if console is current

If I enable the ubuntu console using sudo ros config set and then switch to the console using sudo ros console switch ubuntu, then I get the warning that I'm already in the console even though it's not actually running yet.
$ sudo ros config set rancher.console ubuntu
$ sudo ros config get rancher.console
ubuntu
$ sudo ros console list
disabled centos
disabled debian
current  default
disabled fedora
enabled  ubuntu
$ sudo ros console switch ubuntu
WARN[0000] Console is already set to ubuntu

The warning should only occur if the console is the "current" one.

Verified with v0.5.0-rc2
$ sudo ros console list
disabled centos
disabled debian
current  default
disabled fedora
enabled  ubuntu
$ sudo ros console switch ubuntu
Switching consoles will destroy the current console container and restart Docker.
Note: You will also be logged out.
Continue [y/N]: N


Test bigtable change

@ishanrc: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test-bigtable
9f583f00c78c6064d26be7d7c80cbbacdd9ced53
link
/test test-bigtable

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


Partition Id

Maybe it's not best practice and should be avoided, but i noticed something in tests (1 dev node), if you publish broker service and then publish subsriber service, subscriber will be registered with partition id at that moment, it will start to receive messages, broker service will publish messages to all subscribers at that moment for that message type, now if you republish same service not upgrade, service will get new partition id, this will be regiatered in broker as 2 subscribers, same subsriber same endpoint but with different partition id, and when that hapens broker will publish same message to both subsribers, but in this case same subsriber will receive 2 messages
Subsriber service is singleton, maybe checking when creating service proxy for subsriber on that partition id if it is alive?

Nice catch again. I recommend to unsubscribe your service when it shuts down.
I've changed the ServiceReferenceWrapper code to not use the PartitionGuid anymore. Instead, it uses the app name, service name and if present partitionkey / partitionname.
Let me know if this works out for you.

I have implemented unsubscribe onclose etc what ever is available to overwride, but it looks like sf when you publish the app is deleting old one without normal shutdown, unsubscribe is never received by broker, will test and let you know

Just quick question, is there a reason why this library isnt switched to net standard? I have done this without any issues, have to do it because im using net core services

Haven't gotten around to it yet.
Happy to take pr's!
Sent from Ninehttp://www.9folders.com/

Van: danijel-peric<EMAIL_ADDRESS>verzonden: dinsdag 26 juni 2018 21:08
Aan: loekd/ServiceFabric.PubSubActors
Cc: loekd; Comment
Onderwerp: Re: [loekd/ServiceFabric.PubSubActors] Partition Id (#44)
Just quick question, is there a reason why this library isnt switched to net standard? I have done this without any issues, have to do it because im using net core services
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/loekd/ServiceFabric.PubSubActors/issues/44#issuecomment-400428622, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALkGjPK0Ttk9REi-vxeEgkdDcWGf4p6kks5uAoabgaJpZM4Uxipu.

Nice example project!
v7.4.0 has netstandard2.0 support.
Please let me know if it works for you.

Working


Add note about #disable-next-line to each usage and FAQ

Overview/Summary
Add note about #disable-next-line to each usage and FAQ
This PR fixes/adds/changes/removes

Fixes #153

Breaking Changes
None
Testing Evidence
Linting & Bicep Build will suffice which will run as part of this PR submission
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.

Not entirely clear what the testDnsZones.bicep file is for?

Good catch thought I removed it but clearly hadn't. Have now 👍


Cookie not set in Request Headers, even with 'same-origin' credentials.

Making requests to a Django API requires setting a csrftoken cookie. I am having trouble setting this and sending the Cookie header in a fetch request. I have looked at:

https://github.com/github/fetch/issues/163#issuecomment-112180606
https://github.com/github/fetch/issues/142
https://github.com/github/fetch/pull/56#issuecomment-69113848
http://stackoverflow.com/a/29664267/651952

I've made sure that I'm setting the credentials to 'same-origin', as noted many times in the above resources. However, the Cookie header is still missing from the request. Headers seem to be properly changed for every other attribute but Cookie. I feel like I'm missing something obvious, but cannot figure out what it is. Below is the example js I'm using.
fetch('/api/v2/user/me', {
  method: "GET",
  headers: {
    'Accept': 'application/json', // This is set on request
    'Content-Type': 'application/json', // This is set on request
    'X-CSRF-Token': 'abcdefghijklmnop', // This is set on request
    'Cache': 'no-cache', // This is set on request
    credentials: 'same-origin', // This is set on request
    'Cookie': 'csrftoken=abcdefghijklmnop' // This is missing from request
  }
})
.then(response => {
  if (response.status >= 200 && response.status < 300) {
    return response.json();
  } else {
    throw error;
  }
})
.catch(error => { console.log('request failed', error); });

You will note the header missing from the request:

I tried changing the cookie key to something other than csrftoken; that did not work either. Thoughts on this?

You can't manipulate cookies manually in either XMLHttpRequest nor fetch(). The browser handles cookies automatically. If you want a cookie to be sent, you have to first set it by writing to document.cookie prior to making a requests.
Can't the backend read the CSRF token from the x-csrf-token header, though?

@mislav Even with document.cookie set prior, the cookies are missing from the request. I assumed (incorrectly) that because they weren't being passed along, I maybe needed to add the cookie manually. So, for example:
document.cookie = 'test=abcd';
fetch('/api/v2/user/me', {
  method: "GET",
  headers: {
    'Accept': 'application/json',
    'Content-Type': 'application/json',
    'Cache': 'no-cache',
    credentials: 'same-origin'
  }
})

Still results in:

Note that any other requests on the page do pass along cookies:

Can't the backend read the CSRF token from the x-csrf-token header, though?

I could do this, but there's still a reasonable question as to why the cookies are missing, I think. Why don't any of the cookies on document.cookie get sent along with the request whilst using fetch? Is there a config value I'm supposed to be setting, and not?

@mislav I've figured it out. The credentials property is not supposed to be defined in the headers object. This works:
fetch('/api/v2/user/me', {
  method: "GET",
  headers: {
    'Accept': 'application/json',
    'Content-Type': 'application/json',
    'Cache': 'no-cache'
  },
  credentials: 'include'
})

...duh. For others.

Yikes. Sorry I didn't spot that. Thanks for the update

FWIW:
If you set credentials: 'same-origin' it will only send the cookies from the same domain--which is handy if 3rd party client-side libraries are adding lots of cookies and you don't want to muddle up your server-side request with lots of useless cookie junk.
Example call:
fetch('/api/v2/user/me', {
    method: "GET",
    headers: {
        'Accept':  'application/json',
       'Content-Type': 'application/json',
       'Cache': 'no-cache'
    },
    credentials: 'same-origin'
})

hello @chalisegrogan ,
i did the same as you did here is my code
    return fetch(`${jwtConfig.fetchUrl}${url}`, {
      method,
      headers: {
    'Accept': 'application/json',
    'Content-Type': 'application/json',
    'Cache': 'no-cache',
     Authorization: localStorage.getItem("id_token") || undefined,
  },
  credentials: 'include',
      body: JSON.stringify(data),

    })
    .then(response => response.json())
    .then(res => res)
    .catch(error => ({ error: "Server Error" }));
};

but i dont get any cookies
here is my screenshot

i use react for front and laravel for backend

@constantinosergiou ++ having the same problem, also using React. Help, anyone? :)

fetch became too much of a pain for me, so I went for axios library.. after I read this article.
https://medium.com/@shahata/why-i-wont-be-using-fetch-api-in-my-apps-6900e6c6fe78?source=linkShare-a084d3e16929-1531565170
On Jul 14, 2018, 1:26 PM +0300, Faramarz Razmi<EMAIL_ADDRESS>wrote:

hi,
I also have this problem. When I remove credentials: 'include', then add option like Set-Cookie: 'value=value1', it works. But, I want to set just Cookie to have option Cookie in request headers not Set-Cookie: 'value=value1'(because the server works in Cookie: 'value=value1' syntax!)
appreciate any body's help.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Precompiled view not picking up edits to cshtml

Write a simple taghelper that outputs some content.
    [HtmlTargetElement("product")]
    public class ProductTagHelper : TagHelper
    {
        [HtmlAttributeName("for")]
        public ModelExpression Model { get; set; }

        public override void Process(TagHelperContext context, TagHelperOutput output)
        {
            output.TagName = "div";
            output.Content
                .AppendEncoded("")
                .Append(Model.ModelExplorer.GetExplorerForProperty("Id").Model.ToString())
                .AppendEncoded("");
            output.Content
                .AppendEncoded("")
                .Append(Model.ModelExplorer.GetExplorerForProperty("Name").Model.ToString())
                .AppendEncoded("");
            output.Content
                .AppendEncoded("")
                .Append(Model.ModelExplorer.GetExplorerForProperty("Price").Model.ToString())
                .AppendEncoded("");
        }
    }

Add a usage in a view and add precompilation to the site.
@model IEnumerable<Product>

@foreach (var product in Model)
{

<product for="@product"></product>

}

Hello!

Run the site, everything is fine.
Now edit the razor code to make the taghelper self-closing. This should hide its output, but in this case the changes aren't picked up when you rerun the site.
@model IEnumerable<Product>

@foreach (var product in Model)
{

<product for="@product" />

}

Hello!

In order to see the changes picked up, do something more drastic like edit the _ViewImports.cshml

The issue is with DTH - it doesn't cause the application to recompile if no CS files have changed. Placing this in backlog until DNX \ DTH has a work around for this (which won't be RC1).

cc @davidfowl

I think this regression is related to https://github.com/aspnet/dnx/pull/2972 aka https://github.com/aspnet/dnx/issues/2683


Using in Unity3d

Hi,
i want to use you library in Unity3d and i will get an error which i don't know to handle. I've switched your Project and Unity3D to .net 4.6 and both are working correctly (separate).
The ArduinoUploader.dll and  RJCP.SerialPortStream.dll are compiled also with 4.6
When i will start the Project in Visual Studio the Music sounds out of the Arduino correctly
I've copied the following files to unity:
ArduinoDriver.dll
ArduinoDriver.dll.config
ArduinoUploader.dll
IntelHexFormatReader.dll
NLog.dll
RJCP.SerialPortStream.dll
RJCP.SerialPortStream.xml
When i will start my Project and send the Music to the Arduino the yellow LED is blinking and  i will get the following error:
IOException: Device Error
RJCP.IO.Ports.SerialPortStream.WriteCheckDeviceError () (at :0)
RJCP.IO.Ports.SerialPortStream.WriteCheck (System.Byte[] buffer, System.Int32 offset, System.Int32 count) (at :0)
RJCP.IO.Ports.SerialPortStream.Write (System.Byte[] buffer, System.Int32 offset, System.Int32 count) (at :0)
ArduinoUploader.BootloaderProgrammers.ArduinoBootloaderProgrammer.Send (ArduinoUploader.BootloaderProgrammers.Protocols.IRequest request) (at :0)
ArduinoUploader.BootloaderProgrammers.Protocols.STK500v1.Stk500V1BootloaderProgrammer.EstablishSync () (at :0)
ArduinoUploader.ArduinoSketchUploader.UploadSketch (System.Collections.Generic.IEnumerable`1[T] hexFileContents) (at :0)
I know ist not directly your issue but i'm hoping that you can identify the Problem when you see the error message. May be some files missing?
THX Forward!!!

Hey Gagagu,
Interestingly I was also looking into getting this library working on Unity3d also.
I have downloaded the source, and changed the Unity API compatibility level to 4.6.
Then when using all the source files you can see the issue is with the SerialPortStream library where the DeviceError is thrown. There is a lot going on behind the scenes where it actually uses a native windows (or unix) library. I think this is the issue where from Unity the SerialPortStream isn't communicating to the port library natively.
However, I did manage to get half of this running in Unity (the communication, not uploading the software hex).
If you're happing uploading the ArduinoListener to your Arduino using the standard Arduino software (not using the autobootstrap the package this package does), and simply using this library to communicate with the Arduino Listener.
To do so, download the source files from this repo and drop the in unity.
Download and extract just the 'hardware' source files from ArduinoSketchUploader
Swap the namespace from 'using RJCP.IO.Ports;' to 'using System.IO.Ports;'
Swap the class from 'SerialPortStream' to 'SerialPort'
Remove the last method in 'InitializeWithAutoBootstrap' in 'ArduinoDriver.cs'
Include the other library used for Arduino Driver (NLog.dll)
That should be enough to get the sample mario tunes program running.
However, the port library originally supplied is supposedly better, check the readme.md in SerialPortStream


vim error while loading libXt.so.6 library

missing library: libXt.so.6
~ $ vim
vim: error while loading shared libraries: libXt.so.6: cannot open shared object file: No such file or directory

update vim binary on PR #2 to use a statically linked Vim binary


[Publish a Ride] 'Publish a Ride' UI doesn't match with mock-up.

Environment:  Android 10, Samsung A7.
Reproducible: always.
Build found: v1.0.425
Preconditions
The user is logged in to the 'Car'
Steps to reproduce

Go to 'Add A Ride'
Select 'From' and 'To' edit boxes (e.g. Lviv-Kyiv)
Tap 'Confirm'
Scroll down to the bottom of page

Actual result
'Publish a Ride' UI doesn't match with mock-up.
Expected result
'Publish a Ride' UI matches with mock-up.
User story https://github.com/ita-social-projects/Car-Front-End/issues/58
Labels to be added
"Bug" ("bug"), Priority ("pri: low ")

mock- up

Verified


Failed to debug native node addon if attach node process

VSCode Version: Code - Insiders 1.17.0-insider (2c385b98066eebb8e39dfd98b1fcb8cc254196b6, 2017-09-11T05:24:31.374Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

rainbow-brackets
2gu
0.0.6

MochaSnippets
Ala
0.0.1

Handlebars
and
0.2.0

tslint
eg2
1.0.3

vscode-auto-open-markdown-preview
hnw
0.0.4

vscode-favorites
how
1.1.5

language-julia
jul
0.7.0

Align
ste
0.2.0

(2 theme extensions excluded)

Steps to Reproduce:

node --debug-brk index.js
attach and run
Debug Console shows as below

While VSCode 1.15.1 works on the same code and launch.json

I tried with procmon to see what happen.

node process which was spawned from vscode tried to load the native addon. And it failed because of lack of dlls.
But in VSCode 1.15.1, my process loaded native addon and everything went ok.

Fixed via https://github.com/Microsoft/vscode-node-debug/commit/7c55e9e41862e5de2a27adb3f0cb581b3283aa99
It should be fixed in tomorrow's insiders. If that is not the case please let me know - thanks!

@isidorn Thanks. It works.

@wip0 thanks for checking. Adding verified label


revert: "refactor(gms): move gms restli resources (#1619)"

This reverts commit 6a309332f321fc6847a721629c276e62867bf52c.
This commit is causing GMS to return 404 for all top-level resources. Reverting it for now until we figure out the exact cause.
Should fix https://github.com/linkedin/datahub/issues/1620
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

I believe @jywadhwani is about to submit a fix for this, we might need to drop this.

Dropping this in favor of https://github.com/linkedin/datahub/pull/1625


A bug in libsvm format ?

My dataset consists of over 10 million samples by 57 feature columns in libsvm format. And the max_feature_idx is 57. When I use the csv format, max_feature_idx becomes 56.

Did you use the 1-based (the index of first column starts from 1) libsvm format ?

@guolinke yes. So Is that the reason?

yes.
It is not a bug. we use the 0-based libsvm format.

Oh, thanks for answering! : )


Lowercase package path due to repository rename

Closing, as this would be a breaking change. See https://github.com/nytimes/gziphandler/issues/78#issuecomment-470574019.


Series of changes to docstrings for SEIRD

Issue

[x] Which issue does your pull request address?

#23
Testing, documentation, and checks

[x] If your pull request introduces new code, you should write automated tests (explained here).
[x] Check that all tests pass using devtools::check().
[x] Build documentation using devtools::document().

@ben18785 I fixed some lint issues, but the rest looks good to me. One problem I see: you changed the description of the generic for run() to say that we return one data frame with S,E,I,R,D and time and one with incidences, deaths and time. You didn't change this in the description for the actual method (it still says one df with S,E,I and R and one with incidences and deaths) and looking at the code, I also don't think this is right (the description with the method is correct, I think). I might be wrong though, so have a look! It should at least be consistent between the generic and the method.

Actually, now that I think about it, if the generic is going to apply to all classes whereas the method is class specific, shouldn't the description of the generic be much more general so as to cover all cases?? Or will that mess things up?

This has been superceded by #50


⚠ Bofrost.lu has degraded performance

In 0bdcf89, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1096 ms

Resolved: Bofrost.lu performance has improved in 380b494.


[ios] Not subscribed

Hello,
I recently update my app to the latest react-native versions, and after some tests my device still Not subscribed when I check the BO. (I receive the playerId correctly and I'm able to send tags)
So I went to the Troubleshooting page and I checked all issues but all seems good.
In Xcode, I have the steps checked correctly, so what could be wrong ?
Also, I guess, I have no conflicting package :
"base-64": "0.1.0",
"keymirror": "0.1.1",
"lodash": "4.17.4",
"moment": "2.17.1",
"react": "15.4.2",
"react-native": "0.42.0",
"react-native-background-geolocation-android"<EMAIL_ADDRESS>"react-native-image-picker": "0.26.1",
"react-native-localization": "0.1.28",
"react-native-modal-picker-fix": "0.0.20",
"react-native-onesignal": "3.0.3",
"react-native-scrollable-tab-view": "0.7.2",
"react-native-signature-capture": "0.4.5",
"react-native-vector-icons": "4.0.0",
"react-redux": "5.0.3",
"redux": "3.6.0",
"redux-thunk": "2.2.0"

Thanks for your help
EDIT:  I tried on the project example and I got the same result, does it mean it's a certificat problem ?

@DCKT Thanks for the report! Just fixed the issue in commit 5973c56ffd85000d00c14599148e514aa54deef7. We will be releasing SDK 2.4.1 with this fix included along with a unit test for it.
I'll reply here when the release is available for you to try out.

Hi @jkasten2
Yesterday, I've removed the iPhone from the test user and create a new one and it worked perfectly after that. Quite strange
Thanks for your answser

@DCKT Good to hear you're up and running now.
@DCKT @avishayil Looks like we can close this issue.


⚠ Bofrost.be has degraded performance

In 3de4b38, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 919 ms

Resolved: Bofrost.be performance has improved in 3effa2a after 12 minutes.


[2.0.2] order by @rid desc query generates error

This issues seems to be related to #2966. I'm currently running a document database with the version 2.0.2
Here is the error I get from the console :
2015-02-18 10:17:32:478 SEVERE Internal server error: com.orientechnologies.common.io.OIOException: You cannot access outside the file  size (0 bytes). You have requested portion -65536-0 bytes. File: File: formula. cpm os-size=1024, stored=0, filled=0, max=0 [ONetworkProtocolHttpDb]

A quick overview of the db structure:
Entity (abstract)
    -> Formula
        -> DiscountFormula                  (count : 2)
        -> GeneralFeeFormula             (count : 12)
        -> InstallationFeeFormula         (count : 3)
        -> ProfitFeeFormula                  (count : 1)

The error occurs on query
select from formula order by @rid desc

There is no error if the desc instruction is not present, when the query is run from the studio
Here is the error I got when the query is created with the Java API, the error is display in loop.
févr. 18, 2015 11:05:47 AM com.orientechnologies.common.log.OLogManager log WARNING: Connection re-acquired transparently after 7ms and 1 retries to server 'orientdb:2424/CBWeb': no errors will be thrown at application level

févr. 18, 2015 11:05:47 AM com.orientechnologies.common.log.OLogManager log WARNING: Caught I/O errors from orientdb/<IP_ADDRESS>:2424 (local socket=/<IP_ADDRESS>:51695), trying to reconnect (error: com.orientechnologies.common.io.OIOException: You cannot access outside the file size (0 bytes). You have requested portion -65536-0 bytes. File: File: formula.cpm os-size=1024, stored=0, filled=0, max=0)

févr. 18, 2015 11:05:47 AM com.orientechnologies.common.log.OLogManager log SEVERE: Removing disconnected network channel 'orientdb:2424/CBWeb'...

Here is the Java code :
List<ODocument> resultset = getDatabase()
				.query(new OSQLSynchQuery<ODocument>(
						String.format(
								"select from %s order by @rid desc",
								className), 1).setFetchPlan(fetchPlan));

The problem seems related to the polymorphism + order by @rid desc -> inverted iterator

Any updates on that issue?

Hi @dlaberge
I was not able to reproduce the problem on v 2.1 (btw, #2966 was fixed some time ago)
Could you please confirm that the problem is fixed?

Forgot about this issue. Closing ...


Add native GitHub "Sponsor" button linking to Patreon account

(I just found out that you're on Patreon, the link in the README is pretty easy to miss!)

Simply merge this pull request and activate the "Sponsorships" option in Mezzano's project settings to get a cute little "Sponsor" button linking to your Patreon account!
Here's a preview of the end result.
Here's the official documentation for this nice feature.
(Note that you do not need to be sponsored by GitHub to use this feature.)

Also consider urgently applying to GitHub Sponsors before 1 january 2020 to benefit from ✕2 matching!

Thanks, this looks good. I didn't want the Patreon link to be overly obtrusive.

Great! Now you just need to activate the "Sponsorships" option in Mezzano's
project settings to make the button appear.
On Mon, Dec 23, 2019, 2:08 PM froggey<EMAIL_ADDRESS>wrote:

Merged #148 https://github.com/froggey/Mezzano/pull/148 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/froggey/Mezzano/pull/148?email_source=notifications&email_token=AAADFSEOTG2HLD3FQACT6B3Q2EEBVA5CNFSM4J6WXWN2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVUW33BQ#event-2905456006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAADFSECBWYULT4FJBX5EVTQ2EEBVANCNFSM4J6WXWNQ
.

Great! Now you just need to activate the "Sponsorships" option in Mezzano's
project settings to make the button appear.
On Mon, Dec 23, 2019, 2:08 PM froggey<EMAIL_ADDRESS>wrote:

Thanks, this looks good. I didn't want the Patreon link to be overly
obtrusive.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/froggey/Mezzano/pull/148?email_source=notifications&email_token=AAADFSAV2ARELWQ2D5O6H33Q2EEBFA5CNFSM4J6WXWN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHRXU5A#issuecomment-568556148,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAADFSAWDLLBZBXQBWILIVLQ2EEBFANCNFSM4J6WXWNQ
.

Done, thanks again.


Add example how OAM could corporate with Kustomize/Helm to manage complicated parameters

Now OAM use parameters/variables/properties which could be very long and complicated, we should use tools such like kustomize or helm template to solve this.
Maybe we could add some examples.
/cc @hongchaodeng @resouer

Yep. I think this example can also further clarify the relationship between kustomize/helm and OAM. We REALLY need to show they intend to solve very different set of problems based on the feedback from our initial release.

We will include examples of using parametering tool (helm, kustomize etc) instead of fromParam as well (will need an admission hook in this case https://github.com/oam-dev/rudr/issues/349).
The goal is to clarify plain YAML + external tools is totally supported with OAM without any syntax sugar like fromParam. Also, it will help to prove the sugar is actually sweet :-)


Modularize code

Separate the notebook into split modules

assign this to me

@melvinjjoseph has been assigned this issue


Remove the unused bundler override

We no longer install bundler via definition since it's part of ruby
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

https://buildkite.com/chef/chef-chef-main-omnibus-adhoc/builds/764

i was wondering about that.


Javadoc needed - use checkstyle to enforce

All public methods should have javadoc! This is not true for this repository.
http://checkstyle.sourceforge.net/ is what InterMine uses to enforce. Strongly recommended

https://maven.apache.org/plugins/maven-checkstyle-plugin/


ArgumentNullException in OperationNameTelemetryInitializer

Customer reported:
Problem Desc:
 on 2017-02-21T17:16:16.109Z we encountered a runtime exception in the Application Insights integration module.
System.ArgumentNullException: Value cannot be null.
Parameter name: provider
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetService[T](IServiceProvider provider)
   at Microsoft.ApplicationInsights.AspNetCore.TelemetryInitializers.OperationNameTelemetryInitializer.OnBeforeAction(ActionDescriptor actionDescriptor, HttpContext httpContext, RouteData routeData)
   at Proxy_Method_From_<>f__AnonymousType0`3_To_Void OnBeforeAction(Microsoft.AspNetCore.Mvc.Abstractions.ActionDescriptor, Microsoft.AspNetCore.Http.HttpContext, Microsoft.AspNetCore.Routing.RouteData)(Object , Object , IProxyFactory )
   at Microsoft.Extensions.DiagnosticAdapter.DiagnosticSourceAdapter.Write(String diagnosticName, Object parameters)
   at Microsoft.Extensions.DiagnosticAdapter.DiagnosticSourceAdapter.System.IObserver<System.Collections.Generic.KeyValuePair<System.String,System.Object>>.OnNext(KeyValuePair`2 value)
   at System.Diagnostics.DiagnosticListener.Write(String name, Object value)
   at Microsoft.AspNetCore.Mvc.Internal.MvcCoreDiagnosticSourceExtensions.BeforeAction(DiagnosticSource diagnosticSource, ActionDescriptor actionDescriptor, HttpContext httpContext, RouteData routeData)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeAsync>d__20.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ApplicationInsights.AspNetCore.ExceptionTrackingMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()

Application is an aspnet core application built in .Net461 full framework. Application insights has been applied through Microsoft.ApplicationInsights.AspNetCore 1.0.2 – Now upgraded to 2.0.0
We have enabled it from the app service “Application Insights” setting as well. The application is published though “Continous Delivery”.
It happened only once. The application communicates to Redis “as a service”, sql “as a service”, Azure tables.
We have configured application logging to send data to azure storage.

Looking at code I think the most probable root cause is that IHttpContextAccessor  is null. I binged and saw couple links with the same problem.
@pakrym what may be the cause? If this is expected behavior - we'd need to remove the check and fail more gracefully.

Hm, it seems I jumped into conclusion too fast. It probably has http accessor since it fails in Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetService[T](IServiceProvider provider), not in telemetry initializer itself.

Failed here
So httpContext.RequestServices was null in this code:
var telemetry = httpContext.RequestServices.GetService<RequestTelemetry>();

In version 2.X the call was replaced to:
var telemetry = httpContext.Features.Get<RequestTelemetry>();

Now the question is what might be a condition when httpContext.RequestServices became null and whether the same condition may apply to httpContext.Features

The only reason I know httpContext.RequestServices to be set to null is when request is completed and HttpContext disposed. I think it might happen to httpContext.Features too, but request code should be done executing at this time.

@pakrym can dispose happen for some reason while in Microsoft.AspNetCore.Mvc.Internal.MvcCoreDiagnosticSourceExtensions.BeforeAction? Looking at stack I do not have any explanation how it can happen, but clearly customer experienced this. Any ideas?


Custom color swatch

Q
A

Bug fix?
[ ]

New feature?
[ X]

New sample?
[ ]

Related issues?
N/A

What's in this Pull Request?
Added the ability to add a third Color Swatch Group called custom.  This will allow you to add custom colors to the font color selector.
sample:
<RichText
                value={this.props.richTextValue}
                onChange={(text)=>this.props.onRichTextChange(text)}
                customColors={[
                  {
                    color: 'White',
                    id: '#ffffff',
                    label: 'White'
                  },
                  {
                    color: 'Grey',
                    id: '#efefef',
                    label: 'Grey'
                  }
                ]}
              />

Codecov Report

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

@@           Coverage Diff           @@
##           master     #378   +/-   ##
=======================================
  Coverage   72.88%   72.88%
=======================================
  Files          17       17
  Lines         767      767
  Branches      149      159   +10
=======================================
  Hits          559      559
  Misses        161      161
  Partials       47       47

Impacted Files
Coverage Δ

src/loc/en-us.ts
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 329cbc8...ff25aa7. Read the comment docs.

Hey @eweintraub!
That is a great addition to the control!
Could you please reference dev branch instead of master in your PR (close this one and open a new one referencing dev)?
Thank you!


Make session submission staticplaceholders available for editors

Describe the bug
On http://devday-test.mms-at-work.de:8080/speaker/register/ there is no placeholder for instructions.
Also, there are no placeholders for the other pages that editors could change
Expected behavior
Add staticplaceholders to the templates so that editors can edit all relevant placeholders for all pages relevant to speaker registration and talk submission.

Make sure the static placeholders can be edited also when the session submission is closed (i. e. on the "session submission is closed" page. And/or implement a template and page that includes all static placeholders.


libde265: 1.0.12 -> 1.0.14

Changes:

https://github.com/strukturag/libde265/releases/tag/v1.0.13
https://github.com/strukturag/libde265/releases/tag/v1.0.14

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

@ofBorg eval

@GrahamcOfBorg eval


How to implement onVideoCompletion Callback ?? Help Required

After watch video I have to mark it watched so I need a onCompletion Callback. I tried with listener as `class _ChewieDemoState extends State {
TargetPlatform _platform;
VideoPlayerController _controller;
VoidCallback listener;
bool isCompeleted = false;
@override
void initState() {
super.initState();
_controller = new VideoPlayerController.network(
'http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerEscapes.mp4',
);
listener = () {
//      if (!mounted) {
//        return;
//      }
//debugPrint("listening .....");
  if(_controller.value.duration!=null){
    debugPrint("duration"+_controller.value.duration.inMilliseconds.toString()+" _ position _"+"duration"+_controller.value.position.inMilliseconds.toString());
    //debugPrint(_controller.value.position.inMilliseconds.toString());
    if(_controller.value.duration.inMilliseconds==_controller.value.position.inMilliseconds){
      debugPrint("competed .....");
      _controller.removeListener(listener);
    }

  }

//      if (isCompeleted == _controller.value.completed) {
//        initialized = _controller.value.initialized;
//        setState(() {});
//      }
};
_controller.addListener(listener);
}`
But it don't work and _controller.value.duration.inMilliseconds and _controller.value.position.inMilliseconds never become same ,quite surprising why ? Thanks in advance

I would like to know this too. It is important to set the videos as "watched" or not. Also would be nice to have a "max progress" just like how netflix shows a red bar under our previously watched videos


Green icon not updated when switching bank accounts

Using Chrome on Linux (version 45), when I click on my bank accounts to switch the view of the operations list (which works fine), the green icon in the left menu of the left side of the bank account label, is not updated.

The first time I click, everything is OK though
If I go to the settings and then back to the reports (which hides and displays the bank accounts), the icon is updated

I cannot reproduce with Firefox. There is nothing in the client console nor in the logs.

The "active" class is correctly set on the item I just clicked, but it seems that Chrome does not repaint the icon…

I can still reproduce the bug with the latest Chromium, so this might actually need fixing.

I really think this comes from Chrome : highlighting the text (in this block) solves the issue.
I can't reduce the issue to a simple testcase though.


⚠ ClamAV has degraded performance

In 831cf2d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 15cd120 after 9 minutes.


NetQueue concurrency

Why wouldn't you have used ConcurrentQueue<T> instead of rolling your own (NetQueue<T>)? Asking as I am studying the code base for a deeper understanding of the client/server topography in games.

As far as I remember it was for performance; ConcurrentQueue was expensive, esp. when uncontested. It was a very long time ago tho; no idea if this is the case today.


Look for the config files in www

This adds an extra location to look for the firebase config files. Because the ionic package command only uploads www, package.json, config.xml and resources paths to the cloud, we need to place the config files in www.
The copy config from app-scripts can copy the config files from root to www so we don't need to keep anything in www in VC.

@dwieeb this PR adds an extra location where the hook should search for config files. I explained in my prev PR why it wouldn't work otherwise.

Hmm, I wish there was a better way, but this makes sense for today.

Thanks!

@dwieeb Yes, me too. I tried to brainstorm some other ways this could be achieved. In the previous PR I added some comments.


Update Gemfile.lock and remove libsodium pinning

Description
Update Gemfile.lock and remove libsodium pinning
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]
Check List

[ ] New functionality includes tests
[x] All tests pass
[ ] RELEASE_NOTES.md has been updated if required (not required for bugfixes, required for API changes)
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

woot
@jaymalasinha says this was green in https://buildkite.com/chef/chef-opscode-pushy-client-master-omnibus-adhoc/builds/58.


[Question] Is it possible to generate embedding (feature extraction) similar to BLIP-2?

Question
BLIP-2 allows extracting Unimodal features like:
features_image = model.extract_features(sample, mode="image")
features_text = model.extract_features(sample, mode="text")
print(features_image.image_embeds.shape)
# torch.Size([1, 32, 768])
print(features_text.text_embeds.shape)
# torch.Size([1, 12, 768])

Is it possible to do the same with LLaVa?

same question

Any answers ?

Same question. Have you got a solution? Thanks

+1. Did anyone find a solution that is reasonably straightforward?


Change solution project build order.

Include xaml in the project.

@scccboom What was your issue? Or what is the benefit for this change?

@punker76 I may be missing something, but the demo project doesn´t have the ".xaml" files referenced in it. Only the code behind files ".xaml.cs" files appear in the solution explorer, making it harder to open the designer in VS. Is it intentional? If it´s not, I think this PR will help.

@MurariAlex It's all there 😕

@punker76 Strangely it´s not working for me. I just downloaded the latest src to test it, through the "clone or download" button.
I didn´t change anything, just downloaded and opened.

@MurariAlex Just tested this on VS 2017 15.4.0 and VS 2015 14.0.25431.01 and I can see no difference... it's all there. So what is your VS version?

@punker76  Visual Studio 2015 14.0.25431.01 Update 3. Trying to figure out if an extension could be the resposible for this, but for now I have no clue.

@punker76 Adding the xaml files through "Add -> Existing Item" solves the problem, but obviously the "MahApps.Metro.Demo.Shared.projitems" file is modified:
Adding the ButtonsExample.xaml and ColorsExample.xaml files result in an new nod an new items:
  <ItemGroup>
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\ButtonsExample.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\ColorExample.xaml" />
  </ItemGroup>

This is basically what this PR does, so in not the only one facing this issue... now I really wanna now why your VS correctly displays the items.  😕

I experience the same issue, no .xaml files in the sample applications.

@MurariAlex @thoemmi Normally if the Compile tag in a csproj file contains the DependentUpon the .xaml.cs and .xaml (or any other file) should be shown as grouped.
<Compile Include="$(MSBuildThisFileDirectory)ExampleViews\ButtonsExample.xaml.cs">
  <DependentUpon>ButtonsExample.xaml</DependentUpon>
</Compile>

<Page Include="$(MSBuildThisFileDirectory)ExampleViews\ButtonsExample.xaml">
  <SubType>Designer</SubType>
  <Generator>MSBuild:Compile</Generator>
</Page>

I forced this on this branch, so please test it on your machines
https://github.com/MahApps/MahApps.Metro/tree/feature/SharedProject_Xaml_DependentUpon
thy

@MurariAlex @thoemmi damn <Compile Update="**\*.xaml.cs" SubType="Code" DependentUpon="%(Filename)" /> doesn't work in VS 2015.

@MurariAlex @thoemmi I've never had any trouble with it. Maybe it helps you delete the . vs, . suo files and the bin and obj folders?

by the way, if I load this PR in VS the demo project doesn't load anymore...

@punker76 Yeah, Compile tag is invalid, the project doesn´t even load. This PR introduces this tag... it breaks VS2015 compatibility.
But adding this to the MahApps.Metro.Demo.Shared.projitems solves for me and doesn´t break anything (I think).
  <ItemGroup>
    <None Include="$(MSBuildThisFileDirectory)App.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\ButtonsExample.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\ColorExample.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\CustomDialogExample.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\DataGridExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\DateExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\HamburgerMenuSample.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\OtherExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\SelectionExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\SliderProgressExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\SplitViewExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\TabControlExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\TextExamples.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleViews\TilesExample.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleWindows\CleanWindowDemo.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleWindows\DynamicFlyout.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleWindows\FlyoutDemo.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleWindows\InteropDemo.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleWindows\ShowcaseFlyout.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleWindows\SizeToContentDemo.xaml" />
    <None Include="$(MSBuildThisFileDirectory)ExampleWindows\VSDemo.xaml" />
    <None Include="$(MSBuildThisFileDirectory)MainWindow.xaml" />
    <None Include="$(MSBuildThisFileDirectory)Navigation\HomePage.xaml" />
    <None Include="$(MSBuildThisFileDirectory)Navigation\InterestingPage.xaml" />
  </ItemGroup>

@MurariAlex it's not working for me...

@punker76 VS 2017?

@MurariAlex VS 2015

@punker76 Same as mine, VS 2015 Pro. That´s weird... gonna dig deeper, find out what´s going on.

@MurariAlex maybe I found the problem and when I read it, I couldn't believe it. It seems it' necessary to install the Windows 8.1 SDK :-(
https://social.msdn.microsoft.com/Forums/sqlserver/en-US/26291339-e8fd-4596-980a-7befa63a1f7c/missing-folders-files-add-item-options-in-shared-project?forum=visualstudiogeneral
https://www.cool-it.at/blog/Oktober-2016/Visual-Studio-2015-Fehlende-Code-Behind-Verknupfun

@MurariAlex ...

@punker76 That´s definitely weird... hate messing with VS installation... but i'm gonna give it a try. Thy.

@MurariAlex Have you tried it out?

@punker76 Not yet, busy week, couldn't modify the VS installation. Gonna try it tomorrow, today I have a Mike Portnoy concert to go. ,,/

@MurariAlex np, have fun!

@punker76 Tried it, it worked. So that's was it... downside is that it adds 7GB of data to the VS installation.

@MurariAlex I know that's confusing. I didn't know that this must be installed to get full support for shared projects...

I can confirm that after installing the Windows 8.1 SDK, XAML files finally appeared in Visual Studio 2017.

Nevertheless it's not straight-forward. Thanks for the hint, @punker76 :+1:


Multiple start and end dates

Added a separate start and end date to differentiate from when the dates are selected or changed from outside the directive. Also, if the start and/or end date are changed from outside the directive to a month that is not being displayed on the calendar, the visible dates will be changed updated to show the most resent date change.

@chadfawcett Looks good, just needs a bit of code comment as to why debounce is needed

@jonotron Added it above the debounce function declaration so that it is not written twice in the watches.


[Error] Failed to compare two elements in the array.

06-02 10:02:37.632|ERROR|MainMenu:Main menu extension action failed.
System.InvalidOperationException: Failed to compare two elements in the array. ---> System.NullReferenceException: Object reference not set to an instance of an object.
   at HowLongToBeat.Views.HowLongToBeatUserView.<>c.<.ctor>b__5_0(TitleList x, TitleList y) in H:\GitHub\Lacro59\playnite-howlongtobeat-plugin\Views\HowLongToBeatUserView.xaml.cs:line 47
   at System.Array.FunctorComparer`1.Compare(T x, T y)
   at System.Collections.Generic.ArraySortHelper`1.PickPivotAndPartition(T[] keys, Int32 lo, Int32 hi, IComparer`1 comparer)
   at System.Collections.Generic.ArraySortHelper`1.IntroSort(T[] keys, Int32 lo, Int32 hi, Int32 depthLimit, IComparer`1 comparer)
   at System.Collections.Generic.ArraySortHelper`1.IntrospectiveSort(T[] keys, Int32 left, Int32 length, IComparer`1 comparer)
   at System.Collections.Generic.ArraySortHelper`1.Sort(T[] keys, Int32 index, Int32 length, IComparer`1 comparer)
   --- End of inner exception stack trace ---
   at System.Collections.Generic.ArraySortHelper`1.Sort(T[] keys, Int32 index, Int32 length, IComparer`1 comparer)
   at System.Array.Sort[T](T[] array, Int32 index, Int32 length, IComparer`1 comparer)
   at System.Collections.Generic.List`1.Sort(Comparison`1 comparison)
   at HowLongToBeat.Views.HowLongToBeatUserView..ctor() in H:\GitHub\Lacro59\playnite-howlongtobeat-plugin\Views\HowLongToBeatUserView.xaml.cs:line 47
   at HowLongToBeat.HowLongToBeat.<GetMainMenuItems>b__18_0(MainMenuItemActionArgs mainMenuItem) in H:\GitHub\Lacro59\playnite-howlongtobeat-plugin\HowLongToBeat.cs:line 178
   at Playnite.DesktopApp.Controls.MainMenu.<>c__DisplayClass14_2.<AddExtensionItems>b__1(Object _, RoutedEventArgs __) in E:\Devel\Playnite\source\Playnite.DesktopApp\Controls\MainMenu.cs:line 358

same deal with #66 after authentication and trying to view user data on a game it throws the above error message, along with the above stuff in the playnite.log, have tried the same stuff I have mentioned in my previous issue thread.
Edit: The error appears in v1.3 unlike how #66 stopped throwing errors in v1.3 but it seems to be a slightly different error (not sure if it is worth posting or not but I will as a good measure):
06-02 10:14:41.972|ERROR|MainMenu:Main menu extension action failed.
System.NullReferenceException: Object reference not set to an instance of an object.
   at HowLongToBeat.Views.HowLongToBeatUserView..ctor() in H:\GitHub\Lacro59\playnite-howlongtobeat-plugin\Views\HowLongToBeatUserView.xaml.cs:line 47
   at HowLongToBeat.HowLongToBeat.<GetMainMenuItems>b__18_0(MainMenuItemActionArgs mainMenuItem) in H:\GitHub\Lacro59\playnite-howlongtobeat-plugin\HowLongToBeat.cs:line 178
   at Playnite.DesktopApp.Controls.MainMenu.<>c__DisplayClass14_2.<AddExtensionItems>b__1(Object _, RoutedEventArgs __) in E:\Devel\Playnite\source\Playnite.DesktopApp\Controls\MainMenu.cs:line 358

Still not fully sure what actually caused this problem, but I completely reset my HLTB profile by removing pretty much every single game from my playthroughs lists and then it started working on 1.4.1 without throwing an error, I'm not able to test on 1.4.2 due to the #68
Doing a complete reset of my HLTB profile seems to have fixed #66 as well..

Same deal with #66 . seems to work correctly still in 1.4.3. Left a cvs of my games before the purge in #66 if you want to take a look, could help narrow down what had caused it.

If you try the Playnite 9 beta, can you check with the v2.0?

tbh I haven't messed with playnite in a long time, but I'll check this out and #66 when I get a chance.


Use semilight font for Multiline textfield

related to #510

Hi @lixonic, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

In Fabric 3.0 we're introducing a new way of assigning font weights. Rather than having entirely different font-family names, we're using a common family variable of $ms-font-family-base. The font-weight property is set separately using one of these variables:

$ms-font-weight-light
$ms-font-weight-regular
$ms-font-weight-semilight
$ms-font-weight-semibold

Would you mind updating your PR to match this new pattern?

@mikewheaton done 👍

Approved! Thanks for the quick update here. No trouble getting the latest master to build?

@mikewheaton  yep, https://monosnap.com/file/olY0WQC4cEMmPQKbjLWCiIIiAFWgtA
npm install solved the issue

@lixonic Can you add a newline to the bottom of the file you modified? The build linter is throwing a warning and preventing the initial build from finishing.

@wdo3650 It doesn't look like he removed the newline from the end of TextField.scss, and it seems to build fine in Travis.

@mikewheaton sounds good. Let's merge this in and make sure to add the newline.

Approved.

Thanks for merging this. I just pulled down the latest master (including this merge) and it builds without errors. Must have been a glitch with the linter you saw before.


feat: ✨ vendor syn-input

Description
As a Synergy developer I have to be able make components conform to brand. The syn-input per default contains a pill and a filled attribute which isn't conform. We have to find a way to either remove them, document them properly or similar.
Possibilities are:

Using a transform in vendorism.js to remove the lines
Just remove the CSS so it doesn't change anything
Edit custom-elements-manifest.json to document that it shouldn't it be used in SICK
Think about other ways

We have to have in mind that we're doing a vendor-first approach and to be updatable as much as possible without violating brand.
DoD

[ ] make filled brand compliant
[ ] make pill brand compliant
[ ] Tested: Playwright + Chromatic

Relates to https://github.com/orgs/SickDesignSystem/projects/2/views/1?filterQuery=remove&pane=issue&itemId=41077052

blocked by #57


Is there a method can check Table exist in the SqliteDB?

Is there a method can check Table exist in the SqliteDB?

You can try this.
var table = _connection.Table<Person>();
if (table == null)
{
	// table not exists
}
else
{
	// table exists
}


Set latin-1 as fallback

We still have some problems fallbacking to utf-8 when the encoding is not right, let's use a wider encoding.

[sc-84370]


MultiChat prefixes invisible

I am using today's multichat+luckperms+vault 1.6.1
and all my prefixes are invisible, sometimes randomly after maybe some restarts they might show up but if I restart again or add/remove a prefix from a group they break again...

You need to ask MultiChat for support.
I don't use it personally, and can't really help you.


Updated README example with v5+ information

Example in the README is not correct after the migration to v5.
I have updated it using the proper values.
Hope it helps.
Best regards.

LGTM! A few nits, but great job updating this!

Hi @exterkamp all requested changes are done. Hope everything is ok now.
Best regards

Build is failing with FAILED_DOCUMENT_REQUEST & (Details: net::ERR_ACCESS_DENIED) which is a flake afaict.


[warn] load balancing method redefined ... in error.log since migrating to 2.1.0

Summary
Since migrating from kong 2.0.4 to 2.1.0 i can see a warning
[warn] 187505#0: load balancing method redefined in <KONG_PREFIX>/nginx-kong.conf:57
in <KONG_PREFIX>/logs/error.log
Message also appears there even after upgrade to 2.1.1 and 2.1.2
Steps To Reproduce

Migrate from 2.0.4 to 2.1.0
start/restart kong

Additional Details & Logs
There was one change in nginx_kong.lua which seems the cause for the warnig:
nginx_kong.lua since 2.1.0:

if (role == "traditional" or role == "data_plane") and #proxy_listeners > 0 then
upstream kong_upstream {
server <IP_ADDRESS>;

# injected nginx_upstream_* directives

for _, el in ipairs(nginx_upstream_directives) do
$(el.name) $(el.value);
end

balancer_by_lua_block {
    Kong.balancer()
}

}
nginx_kong.lua 2.0.4:

if (role == "traditional" or role == "data_plane") and #proxy_listeners > 0 then
upstream kong_upstream {
server <IP_ADDRESS>;
balancer_by_lua_block {
Kong.balancer()
}

# injected nginx_upstream_* directives

for _, el in ipairs(nginx_upstream_directives) do
$(el.name) $(el.value);
end

}
The order of the balancer and injected nginx_upstream_* directives changed. If i change the order back to the old version the warning no longer occurs.

This is expected. We will investigate how to silent that warning. Nothing to worry about it.

ok,  thx, why did you change the order?

@michaelhsk is it happening because of order? I didn't try, but I started to see this as well.

Yes, it seems to be order related, fixing it.

@michaelhsk see https://github.com/Kong/kong/pull/6259 for a fix.


Unable to Convert String to Integer

Hi,
I can't seem to figure out how to convert a string into an integer.
I've poured through the documentation and tried every trick I could think of,
but integers always seem to be converted to a string, rather than the other way round.
In my specific case I am trying to pass the values given from regex.matches to color.rgba. Let's say
the values '23', '48', and '128' as an example. In theory this would work great, but seemingly
because regex returns strings, color.rgba cannot process them, and in turn makes the background black.
Any second opinions on this issue would be a great help.
I apologize in advance if this is the incorrect place to ask this kind of question.
Thanks.
theme
{
	// theme.name = auto, classic, white, black, or modern
	name = "modern"

	// view = auto, compact, small, medium, large, wide
	view = view.small
	dark = !reg.get('HKCU\Software\Microsoft\Windows\CurrentVersion\Themes\Personalize', 'SystemUsesLightTheme')

	$bgColor = reg.get('HKCU\Control Panel\Colors', 'Window')
	$bgColorRGB = regex.matches(bgColor, '[0-9]{1,3}')

	background
	{
		color = color.rgba(bgColorRGB[0], bgColorRGB[1], bgColorRGB[2], 255)
		opacity = 100
		effect = 0
	}
}

toint
theme.background.color = color.rgba(toint(bgColorRGB[0]), toint(bgColorRGB[1]), toint(bgColorRGB[2]), 50)

also the fort parameter is alpha: 0-100

This works perfectly. Thanks for your help.


Swtich to llamafile for dev command.

To streamline the development workflow and improve cross-platform compatibility, we propose implementing the dev command using llamafile. Llamafile offers the advantage of providing a single binary compatible with all major platforms, simplifying integration and maintenance for the project.
Benefits

Simplifies builds
Adds support for linux
Adds support for windows
Adds some nice features

Challenges

Kit CLI becomes 160MB instead of 10MB

Llamafile is working on a new server implementation that should speed it up by 3-4x.

@gorkem is this complete now?


make a donation from vuejs to vue bangalore

$20
from opencollective.com/vuejs to https://opencollective.com/vue-bangalore

done.


The Wireframe designs, and explanations in powerpoint

Wireframes and Explanation.zip

I'm stepping out for a while but ill have my phone on me so any comments ill be able to answer, just won't be able to make edits


Upgrade jackson version to <IP_ADDRESS> in /zeppelin-server

What is this PR for?
Upgrade jackson version to <IP_ADDRESS> due to CVE-2022-42003, CVE-2022-42004
What type of PR is it?
Improvement
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-5943

@jongyoul would it possible to merge this jackson patch upgrade due to security fixes?


NULL_IF cannot be empty list in stages

Provider Version
0.15.0
Terraform Version
Describe the bug
When I configure a stage with these file format parameters:
resource "snowflake_stage" "mystage" {
  name = "MY_STAGE"
  url = ...
  schema   = ...
  database =  ...
  comment = ...
  file_format = "TYPE = parquet NULL_IF = [] COMPRESSION = none"
}

when the stage is created the providers tries to run the query and gets and error
CREATE STAGE "DB"."SCHEMA"."MY_STAGE" URL = '....' STORAGE_INTEGRATION = XYZ FILE_FORMAT = (TYPE = parquet NULL_IF = [] COMPRESSION = none) COMMENT = ....

SQL compilation error: syntax error line 1 at position 172 unexpected '['.

So then if I remove the "NULL_IF = []" from the file format, the stage is created. But in the next apply round the providers reads that NULL_IF is empty list and so it tries to change the resource back to the original value:
"TYPE = parquet NULL_IF = [] COMPRESSION = none" -> "TYPE = parquet COMPRESSION = none"

So to prevent this I edited the state.tf resource definiton and added back the "NULL_IF = []" part. And now it is ok. But in the case of recreate it would fail again.
Expected behavior
I think the generated query should be run as NULL_IF = "[]" - so the brackets are quoted.

When I define the NULL_IF as empty char,
 file_format = "TYPE = parquet NULL_IF = '' COMPRESSION = none"

the create works, but again in the next apply it reads from Snowflake that the value is empty list:
     ~ file_format         = "TYPE = parquet NULL_IF = [] COMPRESSION = none" -> "TYPE = parquet NULL_IF = '' COMPRESSION = none"

Based on the timeline above, this issue appears to already be resolved, via https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/264, and can be closed

We are closing this issue as part of a cleanup described in announcement. If you believe that the issue is still valid in v0.89.0, please open a new ticket.


[Announcement] SignInAsync will throw by default when given an unauthenticated identity

Historically, SignInAsync would accept any principals/identities, this included identites where IsAuthenticated = false.  This was problematic because by default these principals would get rejected by [Authorize]/RequireAuthenticatedUser.
There is a new flag on AuthenticationOptions called RequireAuthenticatedSignIn which is true by default as of 3.0-preview6, you can turn this off if you require the old behavior.
For more context see: https://github.com/aspnet/AspNetCore/issues/9255

Wrong repo: real announcement: https://github.com/aspnet/Announcements/issues/361


Release 0.1.4

For users that subclassed BaseRetrievalStrategy this is a breaking change. Going to release 0.2.0.


Add new configuration MinimumEmptyLines for Style/EmptyLineBetweenDefs

Add new configuration MinimumEmptyLines for Style/EmptyLineBetweenDefs
Allows config for multiple consecutive empty lines, e.g.
# bad
def a
end

def b
end

# good
def a
end

def b
end

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[ ] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

Also, @bbatsov, are the extra trailing spaces in manual/cops_style.md (e.g. line 18, 217, 465, etc.) on purpose? It doesn't seem to make a difference: https://gist.github.com/Dorian/68c20fd65f1520035e92aa8a778df203
(e.g. my vim config removes extra trailing spaces by default)

I'm surprised this is something you want. Could you please explain it more?
The style guide might contradict this a bit: https://github.com/bbatsov/ruby-style-guide#two-or-more-empty-lines

@mikegee Yes, I don't personally agree with it, but that's just the way it is. Configurations options are exactly there for people who deviate from the normal style guide in my opinion, so this seems legit.

I'm on the fence about this, as I don't like adding options that'd be useful to just a handful of people. The increased complexity of the code is simply not worth it. At any rate MinimumEmptyLines is a bad name for this - it should be RequiredEmptyLines or something like this. Allowing a flexible number of empty lines between method definitions seems like a really idea to me for the code consistency.

@bbatsov The previous behavior was to require a minimum of 1 empty line between, e.g. this test pass:
  it 'allows more than the required number of newlines' do
    source = ['    def n',
              '    end',
              '    ',
              '    ',
              '    def o',
              '    end']
    inspect_source(cop, source)
    expect(cop.offenses).to be_empty
  end

Maybe having a minimum and a maximum would be better, e.g. people might want 1 or 2 lines but no more, or 0 or 1 line, or 0 only, or 1 only or 2 only, etc. I'm gonna implement it this way because it also improves the 1 line case

@bbatsov @mikegee What do you think now? It even autocorrects down to the number of newlines required :)
(e.g. worst case scenario for this PR, I make the Style/EmptyLineBetweenDefs better)

@bbatsov The previous behavior was to require a minimum of 1 empty line between, e.g. this test pass:

Hmm, I guess I didn't notice it. In light of this your current suggestion makes sense. Even though I don't like using varying number of blank lines, probably due to this oversight some people were using them, so your changes look like a step in the right direction.

You should extend the class documentation a bit.

This PR is implemented well, and fixing the existing bug is nice. I still think it is weird that nobody has explicitly requested this behavior.

This PR is implemented well, and fixing the existing bug is nice. I still think it is weird that nobody has explicitly requested this behavior.

In the absence of the bug someone might have asked for it. :-)

@bbatsov @mikegee Ok, updated docs, docs generation doesn't let me make double empty lines, not sure what to do

@bbatsov @mikegee Ok, updated docs, docs generation doesn't let me make double empty lines, not sure what to do

I'm not sure what do you mean by this.

In the class documentation I put two newlines as an example but in the
generated docs it only has one empty line, will fix in my other markdown pr
On Thu, 2 Mar 2017 at 21:50, Bozhidar Batsov<EMAIL_ADDRESS>wrote:

@bbatsov https://github.com/bbatsov @mikegee
https://github.com/mikegee Ok, updated docs, docs generation doesn't
let me make double empty lines, not sure what to do
I'm not sure what do you mean by this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bbatsov/rubocop/pull/4083#issuecomment-283871808, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAH4faSZiUqGt6tCXTRAphPIKIgRpN-fks5rh6oqgaJpZM4MQEJr
.

@bbatsov Anyway, looking good now :)

(got to handle the case of non def lines in between (E.g. private / comments / any code?)

So, still need to fix auto-correct when there are comments in the middle, gonna leave the "private/public + comments" case to EmptyLinesAroundAccessModifier (that I need to work on too after that).
def a
end
private
# hello
def b
end

Ok, it just ignores the the groups edge cases, e.g.
def a
end

# comment

def b
end

Why do we want Min and Max config? Wouldn't just a single (exact) number of lines setting suffice? In other words, is enforcing a bounded but inconsistent number of empty lines something anyone would want?

I think some people have a tendency to separate groups of related methods with 2 lines, but I'm not sure what @Dorian's particular coding style is.

Yeah, maybe...
My concern (other that the need for the feature which I'm not sure either) is that adding this makes the feature harder to understand (because of naming). It's not obvious that min will default to max if max is not present and one might think that things like
Style/EmptyLineBetweenDefs
  MinimumEmptyLines: 2

require further tweaking to get a max configuration.
I think I'd by -1 for the min/max feature, but if we're adding it, what about a single config which can be a single number or an array?
Style/EmptyLineBetweenDefs
  NumberOfEmptyLines: 1

Style/EmptyLineBetweenDefs
  NumberOfEmptyLines: [1, 2]

I think we've got plenty of cops with confusing configuration. :D But I see your point. I was also reminded the name of the cop should probably be updated as well - originally it was simply enforcing a single empty line between defs (at least it was supposed to, turned out it was buggy).

@Dorian you changed the code, but not the commit message.

I think some people have a tendency to separate groups of related methods with 2 lines, but I'm not sure what @Dorian's particular coding style is.

@deivid-rodriguez @bbatsov It's the project's coding style to separate methods with two newlines, I also see people separating sections of related methods with two new lines, e.g.:
def handle_apples
end

def handle_pears
end

def process_apples
end

def process_pears
end

@deivid-rodriguez I actually like your NumberOfEmptyLines better, so I changed it to that

the name of the cop should probably be updated as well

@bbatsov The default being one line, I'm thinking of keeping it Style/EmptyLineBetweenDefs.

@bbatsov  About the extra empty spaces in source examples, I removed them when there were not specified by the spec description.


Double include of Batch Rendered 2D

Including batchrenderer2d.h 2 times in main.cpp on line 11 and 12
#include "src/graphics/batchrenderer2d.h"
#include "src/graphics/batchrenderer2D.h"

Thanks; fixed. :)


No longer able to play online

Since the latest game update, attempting to enter online while modded will kick you back to the main menu after the "walls are shifting" screen. It makes sense to prevent online with unmodded randoms but it sucks not being able to play mods with friends anymore.

Did you re-extract from the clean update to make sure your running the mods on the latest version? You can confirm the version when you start the game and make sure it's 1.20.1c

Did you re-extract from the clean update to make sure your running the mods on the latest version? You can confirm the version when you start the game and make sure it's 1.20.1c

No, we had only re-packed the old version. It still works when updating correctly. Thank you for the fast response and sorry for the dumb mistake.

No, we had only re-packed the old version. It still works when updating correctly. Thank you for the fast response and sorry for the dumb mistake.

No worries. thanks for reporting back that it's working!

No worries. thanks for reporting back that it's working!


"showMessage" was actually released in GHC 7.10 as "displayException"

This patch is needed in order to build properly on GHC 7.10.

Thank you @ekmett I obviously didn't test things well enough there.


Open Remote - WSL not working on my windows system with debian distro

Hello, I am encountering this issue everytime I am trying to connect to WSL using your vscodium extension.

 This is the output it shows after it fails

[Info  - 14:28:18.780] Resolving wsl remote authority 'wsl+Debian' (attemp #1)
[Trace  - 14:28:18.783] Running WSL command: wsl.exe --distribution Debian -- bash -c '
# Server installation script

TMP_DIR="${XDG_RUNTIME_DIR:-"/tmp"}"

DISTRO_VERSION="1.89.1"
DISTRO_COMMIT="a045dcfceda27f896936bfd8cbf524f6a93a2f46"
DISTRO_QUALITY="stable"
DISTRO_VSCODIUM_RELEASE="24130"

SERVER_APP_NAME="codium-server"
SERVER_INITIAL_EXTENSIONS=""
SERVER_LISTEN_FLAG="--port=0"
SERVER_DATA_DIR="$HOME/.vscodium-server"
SERVER_DIR="$SERVER_DATA_DIR/bin/$DISTRO_COMMIT"
SERVER_SCRIPT="$SERVER_DIR/bin/$SERVER_APP_NAME"
SERVER_LOGFILE="$SERVER_DATA_DIR/.$DISTRO_COMMIT.log"
SERVER_PIDFILE="$SERVER_DATA_DIR/.$DISTRO_COMMIT.pid"
SERVER_TOKENFILE="$SERVER_DATA_DIR/.$DISTRO_COMMIT.token"
SERVER_OS=
SERVER_ARCH=
SERVER_CONNECTION_TOKEN=
SERVER_DOWNLOAD_URL=

LISTENING_ON=
OS_RELEASE_ID=
ARCH=
PLATFORM=

# Mimic output from logs of remote-ssh extension
print_install_results_and_exit() {
    echo "cf9415c8e5f51ae208730c95: start"
    echo "exitCode==$1=="
    echo "listeningOn==$LISTENING_ON=="
    echo "connectionToken==$SERVER_CONNECTION_TOKEN=="
    echo "logFile==$SERVER_LOGFILE=="
    echo "osReleaseId==$OS_RELEASE_ID=="
    echo "arch==$ARCH=="
    echo "platform==$PLATFORM=="
    echo "tmpDir==$TMP_DIR=="

    echo "cf9415c8e5f51ae208730c95: end"
    exit 0
}

# Check if platform is supported
PLATFORM="$(uname -s)"
case $PLATFORM in
    Linux)
        SERVER_OS="linux"
        ;;
    *)
        echo "Error platform not supported: $PLATFORM"
        print_install_results_and_exit 1
        ;;
esac

# Check machine architecture
ARCH="$(uname -m)"
case $ARCH in
    x86_64 | amd64)
        SERVER_ARCH="x64"
        ;;
    armv7l | armv8l)
        SERVER_ARCH="armhf"
        ;;
    arm64 | aarch64)
        SERVER_ARCH="arm64"
        ;;
    *)
        echo "Error architecture not supported: $ARCH"
        print_install_results_and_exit 1
        ;;
esac

# https://www.freedesktop.org/software/systemd/man/os-release.html
OS_RELEASE_ID="$(grep -i '\''^ID='\'' /etc/os-release 2>/dev/null | sed '\''s/^ID=//gi'\'' | sed '\''s/"//g'\'')"
if [[ -z $OS_RELEASE_ID ]]; then
    OS_RELEASE_ID="$(grep -i '\''^ID='\'' /usr/lib/os-release 2>/dev/null | sed '\''s/^ID=//gi'\'' | sed '\''s/"//g'\'')"
    if [[ -z $OS_RELEASE_ID ]]; then
        OS_RELEASE_ID="unknown"
    fi
fi

# Create installation folder
if [[ ! -d $SERVER_DIR ]]; then
    mkdir -p $SERVER_DIR
    if (( $? > 0 )); then
        echo "Error creating server install directory"
        print_install_results_and_exit 1
    fi
fi

SERVER_DOWNLOAD_URL="$(echo "https://github.com/VSCodium/vscodium/releases/download/\${version}.\${release}/vscodium-reh-\${os}-\${arch}-\${version}.\${release}.tar.gz" | sed "s/\${quality}/$DISTRO_QUALITY/g" | sed "s/\${version}/$DISTRO_VERSION/g" | sed "s/\${commit}/$DISTRO_COMMIT/g" | sed "s/\${os}/$SERVER_OS/g" | sed "s/\${arch}/$SERVER_ARCH/g" | sed "s/\${release}/$DISTRO_VSCODIUM_RELEASE/g")"

# Check if server script is already installed
if [[ ! -f $SERVER_SCRIPT ]]; then
    if [[ "$SERVER_OS" = "dragonfly" ]] || [[ "$SERVER_OS" = "freebsd" ]]; then
        echo "Error "$SERVER_OS" needs manual installation of remote extension host"
        print_install_results_and_exit 1
    fi

    pushd $SERVER_DIR > /dev/null

    if [[ ! -z $(which wget) ]]; then
        wget --tries=3 --timeout=10 --continue --no-verbose -O vscode-server.tar.gz $SERVER_DOWNLOAD_URL
    elif [[ ! -z $(which curl) ]]; then
        curl --retry 3 --connect-timeout 10 --location --show-error --silent --output vscode-server.tar.gz $SERVER_DOWNLOAD_URL
    else
        echo "Error no tool to download server binary"
        print_install_results_and_exit 1
    fi

    if (( $? > 0 )); then
        echo "Error downloading server from $SERVER_DOWNLOAD_URL"
        print_install_results_and_exit 1
    fi

    tar -xf vscode-server.tar.gz --strip-components 1
    if (( $? > 0 )); then
        echo "Error while extracting server contents"
        print_install_results_and_exit 1
    fi

    if [[ ! -f $SERVER_SCRIPT ]]; then
        echo "Error server contents are corrupted"
        print_install_results_and_exit 1
    fi

    rm -f vscode-server.tar.gz

    popd > /dev/null
else
    echo "Server script already installed in $SERVER_SCRIPT"
fi

# Try to find if server is already running
if [[ -f $SERVER_PIDFILE ]]; then
    SERVER_PID="$(cat $SERVER_PIDFILE)"
    SERVER_RUNNING_PROCESS="$(ps -o pid,args -p $SERVER_PID | grep $SERVER_SCRIPT)"
else
    SERVER_RUNNING_PROCESS="$(ps -o pid,args -A | grep $SERVER_SCRIPT | grep -v grep)"
fi

if [[ -z $SERVER_RUNNING_PROCESS ]]; then
    if [[ -f $SERVER_LOGFILE ]]; then
        rm $SERVER_LOGFILE
    fi
    if [[ -f $SERVER_TOKENFILE ]]; then
        rm $SERVER_TOKENFILE
    fi

    touch $SERVER_TOKENFILE
    chmod 600 $SERVER_TOKENFILE
    SERVER_CONNECTION_TOKEN="7141ea9d-eeb8-452f-8a1a-c40ad4f33fb5"
    echo $SERVER_CONNECTION_TOKEN > $SERVER_TOKENFILE

    $SERVER_SCRIPT --start-server --host=<IP_ADDRESS> $SERVER_LISTEN_FLAG $SERVER_INITIAL_EXTENSIONS --connection-token-file $SERVER_TOKENFILE --telemetry-level off --use-host-proxy --disable-websocket-compression --without-browser-env-var --enable-remote-auto-shutdown --accept-server-license-terms &> $SERVER_LOGFILE &
    echo $! > $SERVER_PIDFILE
else
    echo "Server script is already running $SERVER_SCRIPT"
fi

if [[ -f $SERVER_TOKENFILE ]]; then
    SERVER_CONNECTION_TOKEN="$(cat $SERVER_TOKENFILE)"
else
    echo "Error server token file not found $SERVER_TOKENFILE"
    print_install_results_and_exit 1
fi

if [[ -f $SERVER_LOGFILE ]]; then
    for i in {1..5}; do
        LISTENING_ON="$(cat $SERVER_LOGFILE | grep -E '\''Extension host agent listening on .+'\'' | sed '\''s/Extension host agent listening on //'\'')"
        if [[ -n $LISTENING_ON ]]; then
            break
        fi
        sleep 0.5
    done

    if [[ -z $LISTENING_ON ]]; then
        echo "Error server did not start sucessfully"
        print_install_results_and_exit 1
    fi
else
    echo "Error server log file not found $SERVER_LOGFILE"
    print_install_results_and_exit 1
fi

# Finish server setup and keep script running
if [[ -z $SERVER_RUNNING_PROCESS ]]; then
    echo "cf9415c8e5f51ae208730c95: start"
    echo "exitCode==0=="
    echo "listeningOn==$LISTENING_ON=="
    echo "connectionToken==$SERVER_CONNECTION_TOKEN=="
    echo "logFile==$SERVER_LOGFILE=="
    echo "osReleaseId==$OS_RELEASE_ID=="
    echo "arch==$ARCH=="
    echo "platform==$PLATFORM=="
    echo "tmpDir==$TMP_DIR=="

    echo "cf9415c8e5f51ae208730c95: end"

    echo "cf9415c8e5f51ae208730c95: Server installation script done"

    SERVER_PID="$(cat $SERVER_PIDFILE)"
    SERVER_RUNNING_PROCESS="$(ps -o pid,args -p $SERVER_PID | grep $SERVER_SCRIPT)"
    while [[ -n $SERVER_RUNNING_PROCESS ]]; do
        sleep 300;
        SERVER_RUNNING_PROCESS="$(ps -o pid,args -p $SERVER_PID | grep $SERVER_SCRIPT)"
    done
else
    print_install_results_and_exit 0
fi
'
[Trace  - 14:28:32.600] Server install command stdout:
Error downloading server from https://github.com/VSCodium/vscodium/releases/download/<IP_ADDRESS>30/vscodium-reh-linux-x64-<IP_ADDRESS>30.tar.gz
cf9415c8e5f51ae208730c95: start
exitCode==1==
listeningOn====
connectionToken====
logFile==/home/snoep/.vscodium-server/.a045dcfceda27f896936bfd8cbf524f6a93a2f46.log==
osReleaseId==debian==
arch==x86_64==
platform==Linux==
tmpDir==/mnt/wslg/runtime-dir==
cf9415c8e5f51ae208730c95: end

[Error  - 14:28:32.600] Error resolving authority
Error: Couldn't install vscode server on remote server, install script returned non-zero exit status
	at t.installCodeServer (c:\Users\Ezza\.vscode-oss\extensions\jeanp413.open-remote-wsl-0.0.4-universal\out\extension.js:1:26854)
	at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
	at async c:\Users\Ezza\.vscode-oss\extensions\jeanp413.open-remote-wsl-0.0.4-universal\out\extension.js:1:1801

 I really need vscodium to work as it is required for a course that I am following.
I transitioned into vscodium as I learned its benefits over vscode. But currently I am unsure if there is a fix for this?
If there isn't could someone clarify this for me so that I can transition back to vscode (at least for in the short term)

Thanks for your help!

Please VSCodium, then try again report back. The error is noted as being a download error (sadly we still don't output any details). and I guess you wget/curl from WSL using URLs.

Please VSCodium, then try again report back. The error is noted as being a download error (sadly we still don't output any details). and I guess you can wget/curl from WSL using URLs.

Thanks for your reply, yeah I figured a long while ago to just go back to standard vscode for now.

If that works then VSCodium and this extension should work as well...

I am a bit unclear about the solution you are suggesting. I have already applied the open-remote-wsl extension by jeanp413 to my VSCodium setup to use WSL, but it didn't work, which is why I opened this issue.
To clarify, I have already downloaded both wget and curl in my linux distrubtions as instructed per the README.md requirements paragraph.
If there is something I am missing in your suggestion, please provide more details.

So with a recent VSCodium you still get the same "Error downloading server" (just with 1.91 in the URL)?

So with a recent VSCodium you still get the same "Error downloading server" (just with 1.91 in the URL)?

I have re-downloaded VSCodium and applied the "Open Remote WSL" extension by jeanp413 to my VSCodium setup to use WSL, and it worked!
I am unsure what caused the issue in the first place, as I do recall always having wget and curl on my Debian instance.
Anyway, I’m really happy that it works now. Thank you for following up with me, @GitMensch!


Allows MQTT Shared Subscriptions for MQTT-In core node

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes

This change allows to subscribe to shared subscriptions in the MQTT-In core node.
Before the topic check would fail because subscribe topic and the topic in the message are not the same for shared subscriptions.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
--> I created a topic on the forum
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.05%) to 76.231% when pulling df161ce67204474a12434e1c1969007735d03747 on SPIRIT-21:master into d373105b32d7b04cca6115b891f042c2b21339c2 on node-red:master.

Awesome thnx! I was waiting for this quite some time. And it works nicely :) 👍

Out of curiosity, what broker currently supports that feature?

We use it with VerneMQ: https://vernemq.com/
HiveMQ supports it as well.

We use the emqtt.io broker.
I tested and it looks like its working fine with $share.
Not yet tested with $queue. Maybe this is something that needs to be added as well?
https://emqttd-docs.readthedocs.io/en/latest/advanced.html#shared-subscription


[BUG] Error page is learned when the site accessed by playwright is not HTTP Status 200

Describe the bug
https://github.com/aws-samples/bedrock-claude-chat/blob/02473ad46fc6fc0804d72dc44e75cc0a21cbb39c/backend/embedding/loaders/playwright.py#L156
if response.status != 200:
    raise ValueError(f"page.goto() returned Not HTTP Status 200 for url {url}")

Filling out this field will help us investigate the issue efficiently. Providing detailed information allows us to set the appropriate priority. We appreciate your cooperation.
A clear and concise description of what the bug is.
To Reproduce
Filling out this field will help us investigate the issue efficiently. Providing detailed information allows us to set the appropriate priority. We appreciate your cooperation.
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@k70suK3-k06a7ash1 Thank you for reporting the issue. Even if 400 or 500 are responded, the task should be succeeded. But error log should be output so that user can debug on it. If you have any good suggestion, please let us know.


New App Icon and small fixes.

Hi,
I've made new App Icon, very special just for Buildasaur, because he deserves one, I hope it'll look cool... It's just fast concept, I may slightly change background color or size in the future.
I've also changed rendering mode for menu item, now it's set as template by default, so no extra work in code is needed.
And at the last, I've added "Quit Buildasar" action to menu.
Preview of App Icon
Menu bar icon:

Small App icon:

App icon in Dock:

Result of integration 1
Integration took 1 minute and 9 seconds.
Perfect build! All 22 tests passed. :+1:

I've also noticed, there is an unused method in MainViewController class: setupMenuBarIcon(). It seems everything related to MenuItem was moved to new class MenuItemManager.
So I assume it can be deleted.

Very cool icon :clap: I like it.
Just a thought - on iOS, you can simply add a single .pdf-file and Xcode will render all the required resolutions of the resource when compiling. I assumed it was like this for OS X as well, but I may be wrong?

@accatyyc Hmm I've actually never tried it, but thanks for tip, I'll look at it... :)

Result of integration 2
Integration took 24 seconds.
Perfect build! All 22 tests passed. :+1:

I've updated Builda to be slightly more adorable by rounding his eyes and making smile on his face, but I cannot do another pull request before you process this one.

Just push more commits into this branch, it will update the pull request.

Ok, in that case, it should be okay. :)

Result of integration 1
Integration took 1 minute and 16 seconds.
Perfect build! All 22 tests passed. :+1:


Replace semaphores with task notifications

This completely removes all semaphores and uses task notifications and critical sections to replace their functionality.
The effect is a significant reduction in memory use (approx. 2kB/connection to devices with 6 attributes).  This also reduces flash size by a few kB and improves performance of the operations changed by nearly 50%.
Also changed in this:

NimBLEClient::connect() will now stop scanning automatically when called.
getValue() method added to NimBLERemoteDescriptor.
minor bugfixes/cleanup

Very nice!
I am not experienced in tasks, so just curious: do task notifications have the same functionality as semaphores? No drawbacks? Will they work if the library was used in multitasking?
One question: any reason not to bring getValue() in line with remoteDescriptor::getValue() using  template<typename T>? Or has a descriptor always have the same type? If so, why not add e.g. uint16_t getValue() instead of std::string getValue()?

I am not experienced in tasks, so just curious: do task notifications have the same functionality as semaphores? No drawbacks? Will they work if the library was used in multitasking?

They are similar but actually quite different. Semaphores will block any other task trying to take the semaphore, task notifications will not, they simply block when take (actually xTaskNotifyWait internally) is called and only block that task. No drawback if used correctly, as I believe I have done here, the stack calls should lock internally so concurrency risk. The semaphores were mainly used previously to block the tasks while waiting for the callback and to prevent concurrent data access, task notifications and critical sections proved to be a better choice for this purpose.

One question: any reason not to bring getValue() in line with remoteDescriptor::getValue() using template? Or has a descriptor always have the same type? If so, why not add e.g. uint16_t getValue() instead of std::string getValue()?

Simple answer; I forgot 😄. Descriptors can be used in the same way as characteristics if they are custom, they just don't have notification functionality. I'll add the templates later today, thanks for pointing that out.

As long as the deprecated functions are there, you could replace them with a template call, eg readInt32() can be as easy as return readValue<int32_t>(). Same is true for the other read functions.

NimBLERemoteService could use the template readValue() too.
I was questioning myself, why adding getValue() and a timestamp to NimBLERemoteDescriptor? For the NimBLERemoteCharacteristic it is introduced to be able to access a value set by a notification, and to know at what time it was set. But descriptors can only be read, so timekeeping is not necessary, and getValue() could be skipped, although keeping it offers a small benefit of not having to copy the data yourself if you need it several times.

you could replace them with a template call

Yes, that should be done.

why adding getValue() and a timestamp to NimBLERemoteDescriptor?

Only added for consistency with the same methods from NimBLERemoteCharacteristic. I contemplated removing the timestamp for descriptor and might still, but it doesn’t hurt anything to leave it.

why adding getValue() and a timestamp to NimBLERemoteDescriptor?

Only added for consistency with the same methods from NimBLERemoteCharacteristic. I contemplated removing the timestamp for descriptor and might still, but it doesn’t hurt anything to leave it.

Changed my mind and removed it all, no need to store the value of a descriptor, the application can do that if desired. Should be left for a separate commit if requested in the future anyway.

no need to store the value of a descriptor, the application can do that if desired

Indeed! 😄


skip SwinV2 attention mask buffers

Allows more flexibility in the resolutions accepted by SwinV2.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@MarcoForte thanks, one more change needed to make this apply to current checkpoints, will do that in another commit .. but does allow using different img_size + window size with existing models...


[CLARIFICATION] PG locked with only selects executed

Opening a PG connection (NO autocommit, read committed mode) and just issuing a select locks a DDL in a psql session.
Maybe not  bug, but for sure a very strong behavior.
Also, and I highlighted this before, setting a connection to "Development" database, which implies "AUTO-COMMIT" (it is set so, no custom change), still connects with manual commit/read committed (in the combo box).
What am I doing wrong?

Correction.
Isolation level is taken by "How you set it last time", which is even more confusing since If I work on DEV DB, than close DBeaver, than open it again and work on prod, if I don't remember to switch i am in autocommit mode. Disaster...

@wiwwo I'm not sure I understood the situation with switching from dev to prod connection. Am I doing something differently in my example? Isolation level is supposed to depend on connection type by default.

@Matvey16 I might have done something wrong there.
Still, opening a connection with DBeaver blocks DDL (drop table etc), which is the bigger problem 9and the subject of this ticket, sorry for confusion... I might need vacation :D )

@wiwwo Yeah, it looks like an expected behavior, you have a pending transaction, so it locks the table, it is expected when using manual commit mode.


Integration Test - Release - 2019/05/06 22:43:02

Auto create for integration test. IssueCommandFlow

@PRMergerTest3 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Should toString have a buffer effect?

Should the toString function eliminate the BUFFER effect?
Since it converts it to a string there's no more potential for side effects. So should it be something like forall e. Eff ( | e ) String or just String?

I think buffers are mutable, in which case the effect makes sense.

Yeah.. after thinking about it a little more I think you're right. Any side effect in the buffer will affect code further into the program, even if it is converted to a string:
exclaim :: String -> String
exclaim x = x <> "!"

msg :: forall e. Eff ( buffer :: BUFFER | e ) String
msg = fromString "foo" UTF8

-- scenario 1: no additional mutations happen on the buffer
-- scenario 2: underlying buffer gets mutated to string "bar" as UTF8

-- assuming toString removes the effect, at which point you would assume side effects
-- wouldn't affect further code
msg' :: String
msg' = runPure $ toString UTF8 msg

main = log $ exclaim msg'
-- scenario 1: logs "foo!"
-- scenario 2: logs "bar!"


abs() behavior is inconsistent

Steps to reproduce the behavior (Required)

select abs(cast(128 as tinyint));

select abs(cast(-128 as tinyint));

select cast(abs(cast(-128 as tinyint)) as tinyint);

Expected behavior (Required)
128
Real behavior (Required)
mysql> select abs(cast(128 as tinyint));
+---------------------------+
| abs(CAST(128 AS TINYINT)) |
+---------------------------+
|                      NULL |
+---------------------------+

mysql> select cast(abs(cast(-128 as tinyint)) as tinyint);
+---------------------------------------------+
| CAST(abs(CAST(-128 AS TINYINT)) AS TINYINT) |
+---------------------------------------------+
|                                        NULL |
+---------------------------------------------+

StarRocks version (Required)
+------------------------+
| current_version()      |
+------------------------+
| QA_TEST_MASTER f6f6228 |
+------------------------+

This should be right.  tinyint range should from -128 to 127. so cast 128 to tinyint will overlow, will get the null result.