Fix Tailwind CSS Class Errors for Consistent Styling

This change is important because it ensures proper integration with Tailwind CSS, a utility-first CSS framework. The bg-gray class is incomplete and needs a numeric value (e.g., bg-gray-100) to specify the color intensity. Similarly, group-hover:bg-hovergray is not a standard Tailwind class, and if it's a typo or not defined in the configuration, it will not apply any styles. Correcting these issues guarantees that the intended styles are applied, improving the functionality and visual consistency of the component.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

@futreall thanks for your submission here, but if you look at the apps/bridge/tailwind.config.js, you'll see that hovergray is defined, so there is no issue here.


Add capabilities to INFO message

mod_http2 currently issues an INFO log message at start up that lists version numbers of itself and nghttp2. But of same importance are the capabilities that the module was compiled with. Those do not depend on the linked version, but the one found during configuration.

Implemented in v1.5.3


context.preview returns undefined

I tried to implement simmillar logic to this ticket https://github.com/chapter-three/next-drupal/issues/23 .
But no matter what i do, context.preview is undefined.
I also have configured preview.ts file:
import { NextApiRequest, NextApiResponse } from 'next'

import { drupal } from 'lib/drupal'

export default async function handler(
  request: NextApiRequest,
  response: NextApiResponse
) {
  // @TODO FIX ANY ERROR
  const anyResponse = response as any

  return await drupal.preview(request, anyResponse)
}

Because of this, I cannot make preview work for unpublished content.

@Kerchik Is this on local? It's probably the cookie not being set when in an iframe. Can you try the following where you create the DrupalClient?
export const drupal = new DrupalClient(
  process.env.NEXT_PUBLIC_DRUPAL_BASE_URL,
  {
    forceIframeSameSiteCookie: process.env.NODE_ENV === "development", // <-------
  }
)

From the docs:

If you're running your site in development and the host address is different from the iframe preview, you might run into issues with SameSite cookies.

You can read more about this here: https://next-drupal.org/docs/configuration#forceiframesamesitecookie

@shadcn Yes it happens on local, but even when I add forceIframeSameSiteCookie preopert context.preview still returns undefined:
import { DrupalClient } from 'next-drupal'

export const drupal = new DrupalClient(
  process.env.NEXT_PUBLIC_DRUPAL_BASE_URL,
  {
    auth: {
      clientId: process.env.DRUPAL_CLIENT_ID,
      clientSecret: process.env.DRUPAL_CLIENT_SECRET,
    },
    previewSecret: process.env.DRUPAL_PREVIEW_SECRET,
    forceIframeSameSiteCookie: true,
  }
)

Hmm, what happens if you copy the iframe URL and open it in a new tab? Does it enable preview?

@shadcn No, context.preview is undefined

@Kerchik Did you figure this out?


si.graphics() has big impact on CPU temp

Describe the bug
Every call that I make to si.graphics() increases CPU temp in 10-20 degrees celsius. Is it a known issue or related to my system overall?
To Reproduce
Steps to reproduce the behavior:

Monitor your CPU temp with MSI Afterburner
Make a si.graphics() call in a interval (using 5 seconds in my sample)
Notice the spike in CPU temp every time the si.graphic is called.

My code:
const si = require('systeminformation');

function updateAllData() {
    const memElement = document.querySelector('#mem');
    const cpuElement = document.querySelector('#cpu');
    const gpuElement = document.querySelector('#gpu');

    si.mem().then(data => {
        memElement.innerHTML = `<b>${Math.floor(data.free / 1e9)}gb</b> de um total de <b>${Math.floor(data.total / 1e9)}gb</b>`;
    });

    si.graphics().then(data => {
        gpuElement.innerHTML = `<b>${data.controllers[0].temperatureGpu}°C</b>`;
    });

}

setInterval(updateAllData, 5000);

Environment (please complete the following information):

systeminformation package version: latest
OS: Windows 10 Pro
Hardware: RTX 2070 Super, i5 7600k, 16gb RAM
Additional context
Add any other context about the problem here.

@alansiq nothing discovered so far here on all of my machines ... But I would be happy, if you can keep an eye on it on your side. If you have further details, please feel free to reopen this issue.


[1.9.x] Fixed #25997 -- Fixed JS variables being escaped during inline edit of related model in the popup.

Yes, I'll have time to finish test in the next 24h. Let you know when it's finished.

Yes, I'll have time to finish test in the next 24h. Let you know when it's finished.

Ok done.

Ok done.

buildbot, test this please

buildbot, test this please

Could you elaborate on why this is a better/more correct fix than the patch I proposed? In particular, the add view doesn't have a problem before this patch, so it's not clear why it needs modification. It might be better if the test used a string foreign key so that we can verify all the unescaped characters.

Could you elaborate on why this is a better/more correct fix than the patch I proposed? In particular, the add view doesn't have a problem before this patch, so it's not clear why it needs modification. It might be better if the test used a string foreign key so that we can verify all the unescaped characters.

I'll use your proposed patch and update tests in a few hours.
пятница, 1 января 2016 г. пользователь Tim Graham написал:

Could you elaborate on why this is a better/more correct fix than the
patch I proposed? In particular, the add view doesn't have a problem before
this patch, so it's not clear why it needs modification. It might be better
if the test used a string foreign key so that we can verify all the
unescaped characters.
—
Reply to this email directly or view it on GitHub
https://github.com/django/django/pull/5902#issuecomment-168310349.

I'll use your proposed patch and update tests in a few hours.
пятница, 1 января 2016 г. пользователь Tim Graham написал:

Could you elaborate on why this is a better/more correct fix than the
patch I proposed? In particular, the add view doesn't have a problem before
this patch, so it's not clear why it needs modification. It might be better
if the test used a string foreign key so that we can verify all the
unescaped characters.
—
Reply to this email directly or view it on GitHub
https://github.com/django/django/pull/5902#issuecomment-168310349.

On the other hand, we'll probably need the UUIField models for the issue on master, so you can probably leave them.

On the other hand, we'll probably need the UUIField models for the issue on master, so you can probably leave them.

Ok done.

Ok done.

buildbot, test on selenium.

buildbot, test on selenium.

I guess what I meant was to keep the tests for UUID, not just the models. Also, for clarity, it would be nice if the tests for the add views were in a separate commit, just for clarity that they aren't related to the fixed issue.
I'm having trouble with the test locally on Chrome, though the test passed on the CI server. Getting something like "The requested URL /test_admin/admin/admin_views/parentwithcharpk/z>bq'w-"un=lj;</u2028/u2029t&c//change/ was not found on this server." when opening the edit popup.
Anyway, I'll look at this more tomorrow and the release is delayed for at least one more day.

I guess what I meant was to keep the tests for UUID, not just the models. Also, for clarity, it would be nice if the tests for the add views were in a separate commit, just for clarity that they aren't related to the fixed issue.
I'm having trouble with the test locally on Chrome, though the test passed on the CI server. Getting something like "The requested URL /test_admin/admin/admin_views/parentwithcharpk/z>bq'w-"un=lj;</u2028/u2029t&c//change/ was not found on this server." when opening the edit popup.
Anyway, I'll look at this more tomorrow and the release is delayed for at least one more day.

I guess what I meant was to keep the tests for UUID, not just the models. Also, for clarity, it would be nice if the tests for the add views were in a separate commit, just for clarity that they aren't related to the fixed issue.

Ok, updated PR.

I'm having trouble with the test locally on Chrome, though the test passed on the CI server. Getting something like "The requested URL /test_admin/admin/admin_views/parentwithcharpk/z>bq'w-"un=lj;</u2028/u2029t&c//change/ was not found on this server." when opening the edit popup.

I think that was caused by the slash in the PK.

buildbot, test on selenium.

Sorry, it looks like the solution I suggested doesn't work in Chrome, e.g.
======================================================================
FAIL: test_inline_uuid_pk_edit_with_popup (admin_views.tests.SeleniumAdminViewsChromeTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/media/evo/djangō/tests/admin_views/tests.py", line 4609, in test_inline_uuid_pk_edit_with_popup
    self.assertEqual(inner_html, str(parent.id))
AssertionError: u'0390d34c\\u002D02b9\\u002D4792\\u002Db503\\u002D297fc6ec3c01' != '0390d34c-02b9-4792-b503-297fc6ec3c01'

I'm not sure why the CI server isn't flagging this -- looking through the logs, it appears maybe it's not running the tests properly. I'll have to investigate more later. I think the best thing to do is to defer this issue from today's release so we have more time to investigate the proper solution and do comprehensive testing on all browsers.

I've updated tests - please re-check.

As I said, the proposed fix doesn't seem to work in Chrome. Have you tried it there?  Here is the output of the tests - http://dpaste.com/02RG4XH - but I also tested manually.

Right, last solution does not work properly in Chome, re-checking now initial solution.

The root cause is double escaping -- in the view AND the template. This seems to fix it.
diff --git a/django/contrib/admin/options.py b/django/contrib/admin/options.py
index 2557de9..a650ac0 100644
--- a/django/contrib/admin/options.py
+++ b/django/contrib/admin/options.py
@@ -1113,9 +1113,9 @@ class ModelAdmin(BaseModelAdmin):
             new_value = obj.serializable_value(attr)
             return SimpleTemplateResponse('admin/popup_response.html', {
                 'action': 'change',
-                'value': escape(value),
-                'obj': escapejs(obj),
-                'new_value': escape(new_value),
+                'value': value,
+                'obj': obj,
+                'new_value': new_value,
             })

         opts = self.model._meta

Sorry, this was my mistake!

Ok, updated PR. Please review my solution regarding u2028 and u2029 chars in tests.
Also, should I move release notes to 1.9.2?

Yes, on moving the release note. Please remove the inadvertently added file docs/doc_warnings.
Maybe we can omit the string PK test for now. It's still not passing for me and with the current solution, it seems to me that we're mostly covered by the unit tests for escapejs. What do you think?
======================================================================
FAIL: test_inline_char_fk_edit_with_popup (admin_views.tests.SeleniumAdminViewsFirefoxTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/python3.5.1/lib/python3.5/unittest/case.py", line 58, in testPartExecutor
    yield
  File "/opt/python3.5.1/lib/python3.5/unittest/case.py", line 600, in run
    testMethod()
  File "/home/tim/code/django/tests/admin_views/tests.py", line 4591, in test_inline_char_fk_edit_with_popup
    self.assertEqual(str(inner_text), unescaped_name)
  File "/opt/python3.5.1/lib/python3.5/unittest/case.py", line 820, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/opt/python3.5.1/lib/python3.5/unittest/case.py", line 1193, in assertMultiLineEqual
    self.fail(self._formatMessage(msg, standardMsg))
  File "/opt/python3.5.1/lib/python3.5/unittest/case.py", line 665, in fail
    raise self.failureException(msg)
AssertionError: 'b;\'<-=>y&k g\\"sntzih' != 'b;\'<-=>y&k g\\"sntzih '
- b;'<-=>y&k g\"sntzih
+ b;'<-=>y&k g\"sntzih
?                     +

Maybe we can omit the string PK test for now. It's still not passing for me and with the current solution, it seems to me that we're mostly covered by the unit tests for escapejs. What do you think?

I think we could remove u2028 and u2029 from random string with escaped chars.

Yes, on moving the release note. Please remove the inadvertently added file docs/doc_warnings.

Ok, done, updated PR.

@timgraham most of changes except char FK tests were merged in modified commit a839d71d8562abe0b245024e55ca1d02a45e58fd.
So in terms of this PR we want to finish char FK tests and also to leave tests for inline add though popup in separate commit? Please suggest.

Yes, let's open a new ticket and pull request for the char FK tests and fix. The UUID fix/tests are merged in a839d71d8562abe0b245024e55ca1d02a45e58fd and 5052f79df45d843d1e44dcc47152ed503220098f, thanks!


Revisit timeout operator

Operator timeout seems to have some hiccups. It needs to be revisited, and at the very least, add more tests.

This has been addressed.


Cat() not returning a literal for literal inputs

When Cat() (or ##) is called on literal nodes, the result is not a literal node anymore:
val hwInfo = Cat(numCores.U(4.W), buildNum.U(4.W))
assert(hwInfo.isLit()) //fails

This is at least surprising to a Chisel newbie like me. I known that the Cat function merely generated the corresponding FIRRTL statement, and constant folding is done later in the process, so this might be out-of-scope. On the other hand, printing out the value of literals is very handy for the automatic creating of documentation (register maps in my case).
One might consider adding a LitCat() function, which could than be even called outside of a Module context, which would be very handy for testing.

Type of issue: feature request

Impact: no functional change

Development Phase: request
Please tell us about your environment:

version: 3.3.2

What is the use case for changing the behavior?
Automatic generation of documentation, building literals in test code (in the LitCat case)

Incidentally, there is an effort to support more constant-propagation in the frontend (for different reasons--to improve the performance and memory consumption of Chisel and Firrtl on large designs).  I think it would be cleaner to constant-propagate ##/Cat as part of that effort, rather than introduce a new operator.
FWIW, I can understand why you find this behavior surprising, but normally I'd assemble these literals using Scala Ints or BigInts before constructing a UInt, e.g., (numCores | buildNum << 4).U.

Ah, I'd love to see a more general overhaul of constant propagation, Chisel really feels a bit weak here (esp. in the realm of testing). Is there already an issue for tracking that?
This can be closed then, I guess.


X graphics

Hi! I've been watching the project for a little while. I've been experimenting trying to find a solution for our ever repeated graphic issue. While looking around, it dawned on me that we could use the X windowing system. I thought I'd bring it up. There seem to are already be nodejs bindings for the x windowing system. (see https://github.com/sidorares/node-x11) I have a bit of experience writing window managers and I believe we could create a complete graphical desktop environment using x11. Using x11 would allow non-js applications to run, and by writing the desktop environment in nodejs, users would have full control of how their desktop looked and operated. They could completely modify it, like all the other systems in the Os.
I'm anxious to hear your opinions! Have a great day.

hi @lite20  - take a look at https://github.com/sidorares/ntk and https://github.com/sidorares/react-x11 , these are higher level wrappers on top of node-x11. No documentation yet and very much still WIP but I'm keen to continue developing them

@piranna does not really like the idea of using X. I only checked it out once, but it seems like X has billions upon billions of dependencies, and you'll most likely need to include different graphics drivers too. But I think it would be great, if we could compile a minimal X, and make an optional build param in the NodeOS build script to use it.

@GothAck is working on implementation of X server that works in the browser :)
https://github.com/GothAck/javascript-x-server

Hmm, looks interesting, but it needs a browser to show the stuff that you want to render, so I guess this could be a remote only option, right?

yes
"X has billions upon billions of dependencies" - that's not always the case. It could really be just a small single executable

Do you have a build script ready for it? Because it would be really great, in my opinion.
There are two ways I could imagine this could go:

A super minimal X build, and we could use node-x11 to render everything by ourselves.
Or an X build that only has the dependencies to be able to run a Chrome build, and basically Chrome would become the "window manager".

no, but various distributions should have it. Full X server was even part of Nokia N900

First of all, interesting and collaborative issue, let's keep moving on! :-)
Ok, my opinion. From my point of view, NodeOS has more power as a server-only OS, but it's true people usually ask about having a GUI (and that's one of the reasons I ported node-canvas to the framebuffer). By using a web browser for the GUI frontend we can have a remote GUI, and oviously this web browser can be executed on the same machine where NodeOS is running, having a so called "native" GUI. Why a browser and not other thing like node-X11? Nothing special besides the fact that web browsers are easier to program, they are everywhere and that Node.js developers would be more familiar with them since it's HTML+Javascript :-) Also there's a problem with "native" GUIs, that's apps need to be ported between platform. By using pure web technologies (and not hybrid ones like nw.js) we can be sure to make it work everywhere and with network support by default.
As @netlovers told, I'm against X11 by several reasons:

it's overcomplicated
add several dependencies
requires administrator priviledges and a central configuration (hope that someone wrongs me here)
and more than this, that's another abstraction layer that decrease performance compared to other alternatives like DRI or EGL

This statement is not an absolute "NO!" but mostly an advice to the ones that want to do it. I think it would be better to start trying to build and pack a Chromium content_shell using DRI (framebuffer) as a npm package that anyone can launch himself (specially now that users have secure access to the devices, including framebuffer) and later try to accelerate it with EGL, so we could have no middle layers the same way as ChromeOS does and gain the best performance. Said that, if somebody manages to made NodeOS to start a X11-based web browser, I will be more than willingly to help him :-)
Anyway, in my opinion NodeOS should never have a GUI on it's core, and if so it should be an independent project that can also run on other OSes. This doesn't means I don't want NodeOS to have a desktop, and in fact I have been the last days thinking about it... :-D

Oooh very fancy doodles! I've read some pretty enlightening things. I'm going to mess around with your framebuffer port and see if I can get an implementation of webkit or chrome to render to it. With the x11 process I did mean using x11 coupled with a browser like CEF. That would be baked right into said window manager.
This is my plan. I'll monitor the progress of https://github.com/GothAck/javascript-x-server and see if I can maybe contribute. I will also work to remove CEF's or webkit's render scripts and replace them with a framebuffer implementation. Hopefully someone with more EGL experience can join on that and help with the EGL implementation.
Ideally, I would love to end up running NodeOS on a phone I've developed as a side project. I find NodeOS's hackability a beautiful feature that suits the phone perfectly. It's definitely a step from where we are now, but projects like these have no limits.
I really like those doodles by the way. Reminds me of the design sketches up in my bedroom.

Just as a side note, maybe it may be a good idea to create a second branch for the graphical edition of NodeOS since we want to keep them separated. That way the main branch can  remain dedicated to NodeOS server, free of graphic clutter, while NodeOS's graphic branch will expand on it. Just a suggestion :smile:

Oooh very fancy doodles!

Thank you! ^^ They need some explains, but I think show the general idea I have in my mind :-)

I'm going to mess around with your framebuffer port and see if I can get an implementation of webkit or chrome to render to it. With the x11 process I did mean using x11 coupled with a browser like CEF

Before wrong here: my node-canvas port is just only to paint on the screen from Node.js, it doesn't have anything about making CEF to run on it, but it was a good starting point.
Regarding content_shell on framebuffer (CEF is the full Chromium, while we are interested only on the rendering engine), I think it should be mostly a matter of following this instructions to compile the embebed version with the DRM platform. It will be software rendered, but would allow to move forward later for accelerated GBM platform, and later figure out how to increase the resolution from 800x600 to fullscreen. I think @netlovers or @formula1 achieved some progress on this area in the past...

Ideally, I would love to end up running NodeOS on a phone I've developed as a side project. I find NodeOS's hackability a beautiful feature that suits the phone perfectly. It's definitely a step from where we are now, but projects like these have no limits.

That's a cool idea! Do you have somewhere some info about that phone you've build? :-)

Just as a side note, maybe it may be a good idea to create a second branch for the graphical edition of NodeOS since we want to keep them separated. That way the main branch can remain dedicated to NodeOS server, free of graphic clutter, while NodeOS's graphic branch will expand on it. Just a suggestion :smile:

We could be able to do it in a separate branch or a separate fork, and later select it in by using a build flag.

Hi, whilst yes I am writing an X server in JS, it is in no way feature complete, and is mostly a testing ground for ES6 / ES2016 features (with the goal of one-day making it feature complete). If you'd like to use it, you'll need a full v8 based browser (probably chromium) to get the benefit, which probably defeats the purpose :/
Any help with the x server would be appreciated though :P

you'll need a full v8 based browser (probably chromium) to get the benefit

I was intending on making a client + server styled renderer too, and then create X support. There are things about the x API that could be improved seeing that we're using nodejs. (Js makes everything better :laughing: )

Any help with the x server would be appreciated though :P

I'll try to swing by and throw what I can!

Do you have somewhere some info about that phone you've build?

At the moment, I haven't released anything on it. My little organization (hopefully to be a tech company some day) is contemplating how we want to release it. It's an incredible powerful phone, but we don't have the resources to mass manufacture and honestly I just want everyone to have a good phone once and for all.
I know someone with one of these :sob: .

I'll probably throw something about it on the website eventually. It's only been running for a week :stuck_out_tongue: (not complete, but executable).

Before wrong here: my node-canvas port is just only to paint on the screen from Node.js, it doesn't have anything about making CEF to run on it

Haha yes I'm aware!

(CEF is the full Chromium, while we are interested only on the rendering engine), I think it should be mostly a matter of following this instructions to compile the embebed version with the DRM platform.

That sounds like a good plan. I'm curious to see how the canvas running on an HTML renderer would perform compared to a direct canvas port. I might write a "directC" (direct canvas :stuck_out_tongue:) mode for game developers who may potentially want a direct canvas. (if the game is full-screen, why go through all the work of rendering the browser when you can directly draw your rectangles?)
Feedback would be awesome!

I'm curious to see how the canvas running on an HTML renderer would perform compared to a direct canvas port

Definitely worse, at least until we get hardware acceleration.

I might write a "directC" (direct canvas :stuck_out_tongue:) mode for game developers who may potentially want a direct canvas. (if the game is full-screen, why go through all the work of rendering the browser when you can directly draw your rectangles?)

That's one of the reasons I ported node-canvas to NodeOS, just for games ;-)

I'm interested in using Node-canvas to do this. I'm exploring to find hardware acceleration methods but for the mean time, I believe I will begin developing the frame-buffer implementation using node-canvas and we will build off that. I'm having a bit of trouble getting node-canvas going. If you could hop on the IRC at some point, I'll be getting on in 30 minutes and it'd be great if I could throw my questions directly to you. Thanks!

That's one of the reasons I ported node-canvas to NodeOS, just for games ;-)

Yes, that makes sense :stuck_out_tongue_closed_eyes:

@lite20 https://github.com/luismreis/node-openvg-canvas is accelerated canvas library ( but not sure how specific to PI is it )

@lite20 https://github.com/luismreis/node-openvg-canvas is accelerated canvas library ( but not sure how specific to PI is it )

Thanks! I'm going to see what I can make of it. Thanks!

I'm interested in using Node-canvas to do this.

Do you really want to develop the GUI using node-canvas?!?! Hostly I don't believe it's a good idea...

@lite20 https://github.com/luismreis/node-openvg-canvas is accelerated canvas library ( but not sure how specific to PI is it )

Fairly Pi-specific. node-openvg-canvas makes use of OpenVG libraries, that are specific for each graphic card, and although there are implementation for other ones not all of them have it. Also, having OpenVG working would be as difficult as having EGL working since they are somewhat related, and having EGL we could have accelerated Chromium :-) Baby steps better here ;-)

Do you really want to develop the GUI using node-canvas?!?! Hostly I don't believe it's a good idea...
My plan is to create a canvas implementation of sorts. Whether I use Node-canvas, or my own implementation (called BoxedBox).
BoxedBox is like Node-canvas except without Cairo. The part that Cairo handled, generating the buffers to  write to the frame buffer, I do from scratch. I directly bind the canvas commands to frame buffer implantations. For example when when you called to draw a square to the canvas, it'd draw it directly to the frame buffer instead (unless you specify that you want to call a refresh point).
If I used Node-canvas I would have to go through the stress of finding and implementing all these pixel rendering algorithms. However Node-canvas is fighting me right now and I'm having issues installing it so BoxedBox looks like the way to go. (BoxedBox is on my Bitbucket account, if you were looking for it)

If I used Node-canvas I would have to go through the stress of finding and implementing all these pixel rendering algorithms

node-canvas offer the 2D context W3C canvas API, the same used on browsers. I don't find it difficult to use...

If we implement BoxedBox now, we can easily swap it out for an updated faster EGL renderer in the future without users needing to update their code because it'd use the same interface.

The same happens with node-canvas, since Cairo has support for EGL.

node-canvas offer the 2D context W3C canvas API, the same used on browsers. I don't find it difficult to use...

Sorry no I meant if I used BoxedBox I'd have to do that XD

Then what are the downsides of using node-canvas?

Lower level API in comparation to a HTML renderer. node-canvas offer you only lines, dots & rectangles to paint with. For games or demos is good, for high level UIs it's better HTML and more compatible.

For games or demos is good, for high level UIs it's better HTML and more compatible.

Ah okay, so you would like me to attempt to write a CEF (cef is easier to embed, than just blink, the renderer) implementation to render directly to frame buffer just as the canvas renderer. Sounds . I'll give it a shot.

I just noticed the NodeJS project a day or two ago, and I find this idea really interesting. My take on the GUI is that it should be using X11 and have support for Xvfb, even if it's a bit more work.
The reason being is that all modern linux systems (and some macs) use it, and so the way the browser graphics get rendered works as expected.  The canonical way of doing this is to install Xvfb and run your tests 'headlessly' for Firefox or Chrome. To further debug your environment, you can get an X-server up and use the GUI for more in-depth debugging to see how/where your tests error out.
If something else gets used for a framebuffer or GUI client-server model, then it runs the risk of not rendering graphics exactly the way it would look on a modern Linux (or otherwise) system running modern browsers, which would make selenium tests harder to debug.

Hi Risto, and welcome. NodeOS is a server-oriented OS at least on its architecture and will not have a GUI on its core, but as you could see on this or other issues, the "official" (note the quotes) GUI architecture will be based on HTML5. NodeOS is Linux based so there's no a big hassle to run X11 on it, but we are not going to support it officially, in the mid-term the GUI will be used remotely from regular web browsers and in the long term it's planned to use a static build of Webkit or Gecko using directly EGL to remove intermediate layers like X11 and improve performance. Anyway, if you are interested on make NodeOS to run a X11 server we can help you and give you advice, I'm interested to know how far could it go :-)

the "official" (note the quotes) GUI architecture will be based on HTML5.

"The Official, Unofficial Architecture :stuck_out_tongue: "

If something else gets used for a framebuffer or GUI client-server model, then it runs the risk of not rendering graphics exactly the way it would look on a modern Linux (or otherwise) system running modern browsers, which would make selenium tests harder to debug.

All the rendering methods being experimented with in NodeOS at the moment are based around common rendering engines, like Webkit, CEF, and Blink. So the results of programs would be identical to rendering your app in for example firefox or Google Chrome ^_^ making design and implementation rather easy for developers as you don't even need a NodeOS system to develop NodeOS apps.

Closing due to a rejections of X, and progress on noGUI


Export TensorBuffer on index.ts

Fixes #787

This change is

I signed it!

Hi rilut - I'm curious, why do you need to export TensorBuffer? Ideally you're just operating on a set of Tensors and call data() or dataSync()` when you need data stored on the Tensor.

 Thanks!

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Add --runtime-log option

@ilyazub with my current setup, it makes it easier to cache between builds, the Rails tmp directory is wiped between builds, so storing it in a different directory makes life much easier (and cheaper).

@ilyazub is this something you'd consider merging or should I close the PR and find an alternate method?

TY @ilyazub


Update f0e51ff6-34f4-43be-ffcb-d935fa0513c7.md

improve the relevancy of the error() code

Hi @Kagre, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

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

TTYL, MSBOT;

Hi @Kagre, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

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

TTYL, MSBOT;

Thank you for your contribution to improving the quality of our VBA documentation.
Office Scale Team


[zh-cn]Update cluster-upgrade.md

Preview:https://github.com/kubernetes/website/pull/35071/files?short_path=7b82cdf#diff-7b82cdf19706f810f71fa7b368b797e17bd50f8d93bda5a17cbf85b3754daa00

@tengqm cc

/lgtm
/approve


Arithmetic coercion warning

There are a few things in here:

Long literals are now of type long
Constructors must now match the name of the class
@deprecated is implemented for auto-coercion only
Errors and/or warnings are less likely dropped to the floor

An example of new warnings emitted:
/Users/bozzo/XLII/datawire/reactivesocket/tck/rxlib.q:57:9: warning: constructor name 'RequestResponse' does not match class name 'RequestServer'. Missing return type?
/Users/bozzo/XLII/datawire/reactivesocket/tck/rxlib.q:88:9: warning: constructor name 'RequestClient' does not match class name 'SimpleRequestClient'. Missing return type?
/Users/bozzo/XLII/datawire/reactivesocket/tck/rxlib.q:204:29: warning: Use of deprecated methodmacro '__to_int'. "Long used in integer context will silently truncate the result. Please use truncateToInt() or reverse the operands."
 done.


create Branch duplicate 'plus' icon

VSCode Version: 1.41.0
OS Version: Mac Os Catalina

Steps to Reproduce:

Click on your current branch name (bottom left action)
Dropdown open with all your git branches

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

@Lmzd I'm having trouble reproducing this - did this only start happening with the update today?

Hi @RMacfarlane, this morning I could not reproduce this as well. I close this issue. Thanks


Use newer node version

It seems that my PR #58 travis test failed due to old version of node (v0.10.42) used by travis. It works fine with my node v5.6.0.
Some limitations of node v0.10.42: no const in strict mode, no arrow functions.
To make unit testing possible, we should switch to newer node version. This decision includes upgrading node on developers' systems, and setting version used by travis 1.

Agree, let's do it.

Closed @ #58.


Typo in Documentation on GCP/Deploying Kubeflow/Deploy using CLI

Hi, could you please review the changes for the issue https://github.com/kubeflow/website/issues/785 ? please advise me if any additional changes required in the doc.

This change is

/assign @richardsliu

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

I signed it

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

I signed it!

/assign
/ok-to-test

Fixes https://github.com/kubeflow/website/issues/785

Preview: https://deploy-preview-813--competent-brattain-de2d6d.netlify.com/docs/gke/deploy/deploy-cli/

/retest

Thanks!
/lgtm
/approve


Update node-providers.md

Please also add Getblock to the list of providers on the website https://docs.optimism.io/builders/tools/rpc-providers

Please open this on the docs repo: https://github.com/ethereum-optimism/docs


If content is deleted off A, delete it off B

First of all, love the application. It’s been very useful for my 4K library.
I originally thought that this application would do a full sync meaning anything from A that’s not on B it would add to B and anything removed from A would be removed from B. But the latter doesn’t seem to work. If I delete a movie from A, it isn’t deleted from B. Maybe that’s on the roadmap but I would love this feature. I’ve found that there is some content that I don’t want anymore and would like to free up space. I know I can manually go into each instance and remove them but this feature would save the effort especially if I’m removing a bunch of content at the same time.

If content is deleted off A how can I know that it was deleted later on to be able to delete it from B? I'd maybe have to keep some record somewhere of all content and check but that's it's own complexity right there.

I am looking for this also. For example, I just want to control my 1080p Radarr instance and have it duplicate to the 4K instance.

I would need this as well :)

Revisiting this. This would be very helping in keeping both versions of Radarr in Sync

Revisiting this. This would be very helping in keeping both versions of Radarr in Sync

Lists in Radarr can do this natively now.


fix the response time of gRPC is not right

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

Why you have so many commits?

We require you to explain what is the issue, and how do you fix.


SLB - Instead of the city the message "Aricestii-Rahtivani is not a city in Romania" is written

ENVIRONMENT: production
URL: https://scrapers.peviitor.ro/src/slb/index.html
Browser: Chrome
Device: Laptop
OS: Windows 11
STEPS TO REPRODUCE:
1.Open the URL in the browser
2.Click on the "Run Scraper" button
3.Check the location/city for jobs where "Aricestii-Rahtivani is not a city in Romania" appears
4.Open https://careers.slb.com/job-listing and filter by "Romania"
5.Check the job location for jobs in the city of "Aricestii-Rahtivani" from the SLB website
EXPECTED RESULTS:
The city of the job is written in accordance with the city on the company website
Electronics Technician, Aricestii-Rahtivani, județul Prahova
Equipment Operator, Aricestii-Rahtivani, județul Prahova
Equipment Operator, Aricestii-Rahtivani, județul Prahova
WIS Field Specialist, Aricestii-Rahtivani, județul Prahova
WIS Field Specialist, Aricestii-Rahtivani, județul Prahova
Electronic Technician Trainee ,Aricestii-Rahtivani, județul Prahova
Electronic Technician Trainee, Aricestii-Rahtivani, județul Prahova
Mechanical Technician Trainee, Aricestii-Rahtivani, județul Prahova
Mechanical Technician Trainee, Aricestii-Rahtivani, județul Prahova
WCF Cementing Specialist, Aricestii-Rahtivani, județul Prahova
ACTUAL RESULTS:
Instead of the city, the message "Aricestii-Rahtivani is not a city in Romania" is written

Fixed!

Verificat! Este Ok acum.


Ctrl underscore sends RET in Emacs and terminal

Windows Terminal version (or Windows build number)
Windows Terminal Version: 1.9.1942.0
Other Software
Bug is seen in Emacs 27.2 too in WSL
 wsl --list -v
 NAME            STATE           VERSION
 * Ubuntu-20.04    Running         2

Steps to reproduce
ctrl underscore got by pressing ctrl and _ (underscore)key sends RET in terminal as well as emacs in WSL. ubuntu 20.04
 $uname -a
 Linux CompName <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Fri Apr 2 22:23:49 UTC 2021 x86_64 x86_64 x86_64     GNU/Linux

$ showkey -a

Press any keys - Ctrl-D will terminate this program

_        95 0137 0x5f
^M       13 0015 0x0d
^D        4 0004 0x04

Expected Behavior
In Emacs Ctrl + _ is usually mapped to undo. It seems to be mapped to RET in emacs as well as terminal as reported above
Actual Behavior
getting RET passed to terminal and in emacs instead of _ or undo in emacs

What keyboard layout are you using? This is what I am seeing with US-104:

does this help?  And I don't remember changing my keyboard settings for quite some time; this seems to be recent.  Unless, the windows update cycle has changed it, the keyboard layout that is.

@sivaramn it depends how the keys are pressed. For Emacs undo to work first press crtl then shift+underscore. This will execute undo function in emacs inside WSL instance.
Here's a sample screenshot, notice Undo! in the echo area:

And this is the output of showkey:
_        95 0137 0x5f
^_       31 0037 0x1f

First line is ctrl followed by shift+underscore. The 2nd line is crtl+shift+underscore.

@sivaramn it depends how the keys are pressed. For Emacs undo to work first press crtl then shift+underscore. This will execute undo function in emacs inside WSL instance.
First line is ctrl followed by shift+underscore. The 2nd line is crtl+shift+underscore.

Yes, I'm aware of this.  I've been using emacs for quite some time and know to C-S _ .   C-h k calls RET from simple.el for the key chord which was weird.  This seems to be in WSL terminal only as other terminals dont have this issue.

problem seems to go away if I switch to

user1@CompName:~$ showkey -a

Press any keys - Ctrl-D will terminate this program

^_       31 0037 0x1f
^D        4 0004 0x04

Still, wouldn't it be a bug that it works differently when the keyboard is switched?

@sivaramn Emacs has "C+x u" for undo actions. Why not use this key combination instead of "C+S+_" which I find more difficult to type than "C+x u".

@sivaramn Emacs has "C+x u" for undo actions. Why not use this key combination instead of "C+S+_" which I find more difficult to type than "C+x u".

True.  But I work with both Win and Linux OSes, ctrl-X does cut in Win apps;  Ctrl _ works universally without any side effects and unintended results.

Still, wouldn't it be a bug that it works differently when the keyboard is switched?

Yep, that's a real bug alright. There's a little bit of weird handling for a couple ctrl+key combos, ^_ is one of them. I'll toss this on the backlog, thanks for following up!

I get this for Ctrl+Shift+Minus (i.e. CTRL+_)
$ showkey -a

Press any keys - Ctrl-D will terminate this program

^M       13 0015 0x0d

I would also like to see a very quick fix to this issue as I use Ctrl+_ as my Undo in emacs.
I installed my windows from scratch today and don't remember seeing this before, so something changed when I installed everything again.

For those of you that are getting a ^M when your press Ctrl+Shift+Minus, this is because your keyboard driver doesn't have a mapping for that key combination, so the terminal is forced to generate an appropriate mapping itself. The way it does that is by taking the base key (in this case -, which is ASCII 0x2D), and masking off the lower 5 bits. The result of that mask is 0x0D, which is ^M.
The reason it works correctly on a US keyboard (and several others) is because the keyboard itself has a built in mapping for that key combination, so it doesn't rely on the terminal to do the translation.
I've recently been experimenting with a rewrite of the VT keyboard handler which fixes this bug (amongst other things). However, I don't want to promise that it's going to be fixed soon - it's still very much a work in progress.

@j4james do you mean that the US keyboard has a mapping for Shift+Minus or specifically for Ctrl+Shift+Minus.
Indeed I'm on a Finnish keyboard, which is quite different from the US one. On my keyboard Shift+Minus is mapped to produce the underscore, i.e. _, which is 0x5F.

@hvesalai I mean it has a mapping specifically for Ctrl+Shift+Minus. You can see what the mappings are on the Keyboard Layout Info website. Compare the US keyboard layout and the Finnish layout.
On both of them, the minus key is VK_OEM_MINUS (although the position of the key is different). And the US layout has the SHIFT+CONTROL state for that key mapped to ␟ (i.e. 0x1F), while the Finnish keyboard has the CONTROL state alone mapped to ␟. Many keyboard layouts won't have a mapping for that character at all.
What that means is that on your keyboard, Ctrl+Minus should produce the 0x1F character, but don't get too use to that behavior. With my new keyboard implementation that'll stop working, and you'll need to use Ctrl+Shift+Minus, the same as you would on a US keyboard.

@j4james OK, that makes a lot of sense. Indeed when using the Terminal, CTRL+Minus works as an Undo in emacs (after I remove the mapping from Terminal's zoom/scaling).


[BUG]Under certain conditions, settings cannot be pressed.

After a few quick clicks on a setting, the setting cannot be opened afterwards.
Conditions: Windows 11 (latest version as of 06/23/2023)
ChatALL 1.18.35

I made a mistake. The version of ChatALL is 1.28.35.


Different/configurable distance metrics

Background
We've been doing some work with LIME in relatively high-dimension problems (1000+ features), and have found that the current distance metric doesn't tend to perform well in that context [related paper].  For very high dimensions, the euclidian distance is inflated to a large value, which given the kernel you are using:
sqrt(exp(-d^2/k^2))
Can be tuned by using very large kernel widths.  A simple example is the classification problem defined by using a default sklearn RandomForestClassifier on a dataset created with:
sklearn.datasets.make_classification(n_classes=2, n_features=1000, n_samples=1000)
In this case, a kernel width of 3 leads to the weight for the first record (the sample being interpreted itself) having a weight of 1, while all other samples are weighted at near 0, which of course leads to a bad interpretation.  Using very large kernel_widths increases the average score (~0.3 with a width of 10e9), but in general, I think this just isn't the best metric for higher dimension problems.  I've tried some other metrics, and without tuning the kernel width, they are performing much better than euclidian norm can do with a well tuned kernel width).
Just for reference, using the default kernel width of 3, and changing only distance metric on the above problem led to a range of scores between 0 and 1, with cityblock and L1 distance performing best of those I tried, at the cost of a small slowdown.  The scores per sample within each run had low variance, but the scores per metric differed quite a bit, which indicates to me that this is an important tuning parameter.  Chebyshev distance performed better than euclidean but not the best, and was slightly faster than the current euclidean code.
Changes
This PR exposes an option to the user to specify a metric from this pretty comprehensive list: sklearn pairwise distances, defaulting to euclidian for tabular and cosine for text which is the the same as the current measures.  So for existing users, this changes nothing, but optionally, for cases like I described above, you'll be able to use different metrics.
I've also included a benchmarks folder which has a script for each of the explainers that loops through a sample dataset and records scores and times, it may be useful for other similar optimizations.

This is a problem that I had already identified and was on my 'todo' stack. The problem with euclidean distance for tabular is not only that it is bad for high dimensions, but that it is really hard to come up with a sensible default value (as you noted yourself).
The benchmark folder is also a good idea.
Thanks you very much, I will merge this soon.

Cool, happy to help.
Perhaps it would be useful to have a contributing.md file with some of this kind of 'todo' item for existing or other potential contributors?

Yes, that's a great idea. When I have a bit of time, I'll think try to write a file with known problems and ideas.
If you have any intuition or suggestions for a reasonable distance / kernel pair for tabular data, let me know!
Thanks again for contributing.

I'll put together a PR with some experimental results but I think just scaling the distance by the number of columns before squaring it makes a single default kernel width work well with a wider range of datasets.  I've only tested it with the Euclidean distance so far but I would expect pretty similar improvements or no change for the other metrics.


Evaluate benchmarks

Add a prototype infrastructure for leadership board evaluation. Merge this to ensure that the beta version of the Dockerfile works.

Can you move this PR to the flow-project branch? Or is it meant to be private.

PR has been migrated to flow-project and is hence closed.


issue with invoker

when trying to deploy, I run into this issue with the invoker.
Failed to start container with id 8d9125bf2d3711312a98a8b98de15306e495883cc470a03beb6689b34895791f with error: rpc error: code = 2 desc = failed to start container "8d9125bf2d3711312a98a8b98de15306e495883cc470a03beb6689b34895791f": Error response from daemon: {"message":"mkdir /usr/lib/x86_64-linux-gnu: read-only file system"}
Error syncing pod, skipping: failed to "StartContainer" for "invoker" with rpc error: code = 2 desc = failed to start container "8d9125bf2d3711312a98a8b98de15306e495883cc470a03beb6689b34895791f": Error response from daemon: {"message":"mkdir /usr/lib/x86_64-linux-gnu: read-only file system"}: "Start Container Failed"

on the logs of the pod in k8s I see the following
failed to open log file "/var/log/pods/7fce67a4-4d69-11e7-a736-0242ac110003/invoker_41.log": open /var/log/pods/7fce67a4-4d69-11e7-a736-0242ac110003/invoker_41.log: no such file or directory

I checked if the host drive was filling up, but everything is all good there.
I run k8s 1.6.2 on CoreOs 1353.7.
I edited the configure_whisk.yml to use the 1.6.2 version.
      - name: configure-openwhisk
        image: danlavine/whisk_config:v1.6.2
        imagePullPolicy: Always

I tried 3 times the deployment with the same result.

On Slack, it was suggested to try removing the /usr/lib/x86_64-linux-gnu/libapparmor.so.1 mount from the invoker.yml as that is an Ubuntu-specific path. Docker is trying to create this non-existent path on the host and failing because it doesn't have sufficient permissions to do so. In your case, you can probably remove the apparmor volume and volumemount entries completely.
While in there, you should review all the mounts and ensure they match the paths on CoreOs. You can test these changes quickly by editing  https://github.com/apache/incubator-openwhisk-deploy-kube/blob/910f7b808a39e91aedff064f444f453f1da1c8c4/ansible-kube/environments/kube/files/invoker.yml to update the paths and then deleting (kubectl delete statefulset/invoker) and re-creating (kubectl create -f /path/to/edited/invoker.yml) the Invoker.

hey @mimizone I updated the PR to include a troubleshooting section for issues like this. Would that be able to close this PR or do you think that there is something else missing?

@DanLavine I commented all the lines related to apparmor in the invoker as you suggested.
https://github.com/mimizone/incubator-openwhisk-deploy-kube/commit/fc53714adb608376952e62e568b3de7971a108ea
It seems to do the trick!

I think this PR can be closed.  Can someone with the ability to do that please take care of it?


Android: Story traversal progress bar is inaccurate in folders

That circular progress bar embedded in the Next Story button looks like it is inaccurate when reading by folder. Tapping on it also shows the wrong count.

This happens in "all stories" for me

@idontusenumbers does the issue go away if you re-sync, and then click back into the folder (might need to try a few times)?
The cause might be the local reading session carrying over unreads from bad places, and a sync would clear that. Still, the queries will need to be looked at if that's the case

How to skip a JavaBean property
Sometimes there is a need to avoid dumping some JavaBean properties.
Use the @Transient annotation on the getter:
import java.beans.Transient;

class MyBean {
	private int value;

	public MyBean(int value) {
		this.value = value;
	}

	@Transient
	public int getValue() {
		return value;
	}

	public void setValue(int value) {
		this.value = value;
	}
}

The requirement may be dynamic, based on the value of the property. For instance empty collections or 'null' values should not be dumped (serialized). This is an example how it can be implemented:
class PointRepresenter extends Representer {

    @Override
    protected NodeTuple representJavaBeanProperty(Object javaBean, Property property,
            Object propertyValue, Tag customTag) {
        if (javaBean instanceof Point && "location".equals(property.getName())) {
            return null;
        } else {
            return super.representJavaBeanProperty(javaBean, property, propertyValue, customTag);
        }
    }
}

where * javaBean - is the actual instance of the JavaBean * property - internal representation of the JavaBean property * propertyValue - the actual value which holds the property (in this example it is not used)

Sorry about that previous content, wrong repo.
I uninstalled, reinstalled, cleared data and cache, opened, logged in, clicked "all stories", circle was still inaccurate

Ha. oops. Thought so.
Thanks for testing that

I presume this update was released but I am still experiencing this issue.

@idontusenumbers Can you submit a screenshot here?

Viewing "all stories" I can't swipe to see more. This is what it shows:

After that I hit back and see this:

When I tap the next button, it doesn't advance but it does pop up a tool tip saying there are 359 unread stories

Couldn't replicate the story traversal issue. Pushed a small change that might help with unread stories sync under some circumstances.
@idontusenumbers Are you still having the progress bar issue?

Seems to be working at the moment. Feel free to close the bug as obsolete or resolved =)


Disabling Firewall/Mcafee Solves Connection Problem..Almost..Need directory list

On disabling the firewall/Mcafee Virus completely , the program Sparkly R , works without problem!!! But at the enterprise level this is not sustainable. He can only disable the firewall to particular directories..not to the Temp directory as a whole !, only files or subdirectories in the Temp..(company rules )
We tried to disable the firewall to only the following directories
"C:\Users\grbortz\AppData\Local\rstudio\spark\Cache\spark-2.0.2-bin-hadoop2.7"
"C:\Users\grbortz\AppData\Local\rstudio\spark\Cache\spark-2.0.2-bin-hadoop2.7\tmp\hadoop"
'C:\Users\grbortz\AppData\Local\Temp\
The following Errors are occuring now

sc <- spark_connect(master = "local")
Error in file(con, "r") : cannot open the connection
In addition: Warning message:
In file(con, "r") :
cannot open file 'C:\Users\grbortz\AppData\Local\Temp\RtmpyQvjkq\file9f06da6313_spark.log':
Permission denied

On connecting again after the above

library(sparklyr)
library(dplyr)
sc <- spark_connect(master = "local")
Error in force(code) :
Failed while connecting to sparklyr to port (8880) for sessionid (5627): Gateway in port (8880) did not respond.
Path: C:\Users\grbortz\AppData\Local\rstudio\spark\Cache\spark-2.0.2-bin-hadoop2.7\bin\spark-submit2.cmd
Parameters: --class, sparklyr.Backend, "C:\Users\grbortz\Documents\R\win-library\3.2\sparklyr\java\sparklyr-2.0-2.11.jar", 8880, 5627

Please inform me of the full directory list and files that I need to disable Mcafee antivirus/ Firewall on to get this to work. WE CANNOT DISABLE MCAFEE/FIREWALL OVER THE WHOLE PC...THE COMPANY WONT ALLOW THIS!  It works when we do this ...but its of no use.
Thanks
Graham

You will need to disable the firewall on all directories constructed within 'C:\Users\grbortz\AppData\Local\Temp\, if that is not happening already. The R session directory is constructed using a unique directory name within that path, and R + RStudio will need to be able to read and write files in that directory.

Hi Kevin
He has disabled this ...\Temp directory ( not sustainable ...though) but
even this didnt work.  He states that even if it did work the company would
not allow this as this directory is prone to virus attack. He would
therefore have to Prename the temp sub directories that need to be unlocked
and the files that spark creates. Are these subdirectories and files  the
same or are they created on each new spark session so they wont be seen by
Mcafee on startup? Is this not going to be a problem for everybody that is
subject to corporate rules..and normally at enterprise level this would be
the case?  By scaling R using Spark, is it not the intention to use R at
the Enterprise level. Do you not see this as a problem for the deployment
and expansion in the use of Sparkly R?
We also notice that e ports are different each session and the Mcafee
controller has to also free up ports, but if they are dynamic and unique
for each session this is impossible. The first port mentioned 8880 is the
same, but the second is changing for each execution!
The strange thing is that Mcafee Picked up all the directories mentioned
and that these directories are blocked ie the Hadoop / Spark directories
and the Temp directory. However it is still giving a connection problem anf
Mcafee does not pick up the blockage.
We will be getting support from Mcafee to see why the firewall is still
blocking.  The local Mcafee expert asked if perhaps there is an exe file
that should be run by rstudio  when the connection is set up but is being
blocked from execution and where this executble may lie. We only see the
.cmd files.
We know its the Mcafee Firewall as I am told that Mcafee disables the
Window Firewall and uses its own. As soon as we disable it entirely the
code on the Tutorial home page runs perfectly.
This is most unfortunate as it may end my attempt to get to use Sparkly R
:(😞😞
Is the no way Rstudio can produce a workaround. How are other people
,subject to corporate rules (controlled by  a central network adminstrator
, where the user does not have FULL rights , and where Mcafee is used),
coping with this ?
This does not occur with Defender apparently as that still uses the windows
firewall, which can be controlled.
Regards
Thanks
Graham
On Wed, Jan 11, 2017 at 7:53 PM, Kevin Ushey<EMAIL_ADDRESS>wrote:

You will need to disable the firewall on all directories constructed
within 'C:\Users\grbortz\AppData\Local\Temp, if that is not happening
already. The R session directory is constructed using a unique directory
name within that path, and R + RStudio will need to be able to read and
write files in that directory.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rstudio/sparklyr/issues/420#issuecomment-271941143,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGglEtnaVS2qbpaSj2M2rBgedV9mP9i2ks5rRRcegaJpZM4LgUNM
.

In conclusion I must however thank you guys at Rstudio and the community
for all the support given to me over thislast few weeks,. Although I havent
been able to make a  success of this yet, your attempts are still
appreciated, especially Kevin and Edgar.
Regards
Graham
On Wed, Jan 11, 2017 at 9:36 PM, Graham Bortz<EMAIL_ADDRESS>wrote:

Hi Kevin
He has disabled this ...\Temp directory ( not sustainable ...though) but
even this didnt work.  He states that even if it did work the company would
not allow this as this directory is prone to virus attack. He would
therefore have to Prename the temp sub directories that need to be unlocked
and the files that spark creates. Are these subdirectories and files  the
same or are they created on each new spark session so they wont be seen by
Mcafee on startup? Is this not going to be a problem for everybody that is
subject to corporate rules..and normally at enterprise level this would be
the case?  By scaling R using Spark, is it not the intention to use R at
the Enterprise level. Do you not see this as a problem for the deployment
and expansion in the use of Sparkly R?
We also notice that e ports are different each session and the Mcafee
controller has to also free up ports, but if they are dynamic and unique
for each session this is impossible. The first port mentioned 8880 is the
same, but the second is changing for each execution!
The strange thing is that Mcafee Picked up all the directories mentioned
and that these directories are blocked ie the Hadoop / Spark directories
and the Temp directory. However it is still giving a connection problem anf
Mcafee does not pick up the blockage.
We will be getting support from Mcafee to see why the firewall is still
blocking.  The local Mcafee expert asked if perhaps there is an exe file
that should be run by rstudio  when the connection is set up but is being
blocked from execution and where this executble may lie. We only see the
.cmd files.
We know its the Mcafee Firewall as I am told that Mcafee disables the
Window Firewall and uses its own. As soon as we disable it entirely the
code on the Tutorial home page runs perfectly.
This is most unfortunate as it may end my attempt to get to use Sparkly R
:(😞😞
Is the no way Rstudio can produce a workaround. How are other people
,subject to corporate rules (controlled by  a central network adminstrator
, where the user does not have FULL rights , and where Mcafee is used),
coping with this ?
This does not occur with Defender apparently as that still uses the
windows firewall, which can be controlled.
Regards
Thanks
Graham
On Wed, Jan 11, 2017 at 7:53 PM, Kevin Ushey<EMAIL_ADDRESS>wrote:

You will need to disable the firewall on all directories constructed
within 'C:\Users\grbortz\AppData\Local\Temp, if that is not happening
already. The R session directory is constructed using a unique directory
name within that path, and R + RStudio will need to be able to read and
write files in that directory.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rstudio/sparklyr/issues/420#issuecomment-271941143,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGglEtnaVS2qbpaSj2M2rBgedV9mP9i2ks5rRRcegaJpZM4LgUNM
.

You can configure the temporary directory that R uses for resources by setting the TMP environment variable. The easiest way to do this in an R-specific way is to create a file a file in your home directory called .Renviron, and add the lines:
TMP = C:\path\to\tmp
```

See e.g. `?tempdir` and `?Startup` for some more information. In general, these deployment issues are the responsibility of the organization attempting to deploy Spark + sparklyr to handle, but we do try to provide as much direction as possible.

@javierluraschi, is it possible to configure the port used for gateway connections? (The port returned by the gateway for connection to the Spark backend)

@gbortz27 - I'm sorry, but I can't recall if I already asked you to copy the entire spark-2.0.2-bin-hadoop2.7 folder to your workspace (which you should have full control on).  Then try to connect but setting the spark_home argument to point to that new copy.
@kevinushey / @javierluraschi  - Is it possible to default the Spark installation to the user's workspace instead of their AppData folder?  I think the restriction is that the anti virus is blocking the winutils.exe file because it's trying to start inside the AppData folder.

@edgararuiz: I think the installation directory can be configured by setting the spark.install.dir R option; e.g.
options(spark.install.dir = "C:/path/to/Spark")

This would need to be specified in a startup file (e.g. .Rprofile) to make sure that new R and RStudio sessions make use of it.

Hi All
I tried to follow your suggestion by bringing the Spark directory over to
the home directory (which if correct is my Username directory ie
Users\username.... and ten change the environmental variables to point to
the Spark Home and the Hadoop home ( as well as setting and cheacking this
using Sys,getenv() and Sys.setenv() ..and it still gave same error ..in the
error message it is still pointing to the original Spark directory where
Rstudio put Spark....
As far as the Temp directory is concerned ..we tried (although it wouldnt
be permanent ) again by  freeing up  the Temp directory to test the
software and it still came up with the same error messages that it couldnt
access a file or directory.
But we will try Kevins suggestion of reallocating the Temp directory to a
different (say Home directory ) and revert.
Still keen to get an answer on moving the Spark Home out of Apps path...is
there more instructions that I need to give rstudio to force it out.
Maybe Mcaffee will be able to assist ..
I cant remember but does R studio , on install , give the user the chance
to change to directory where spark installs and not accept default.
Many Thanks
Graham
On Wed, Jan 11, 2017 at 11:37 PM, Edgar Ruiz<EMAIL_ADDRESS>wrote:

@gbortz27 https://github.com/gbortz27 - I'm sorry, but I can't recall
if I already asked you to copy the entire spark-2.0.2-bin-hadoop2.7 folder
to your workspace (which you should have full control on). Then try to
connect but setting the spark_home argument to point to that new copy.
@kevinushey https://github.com/kevinushey / @javierluraschi
https://github.com/javierluraschi - Is it possible to default the Spark
installation to the user's workspace instead of their AppData folder? I
think the restriction is that the anti virus is blocking the winutils.exe
file because it's trying to start inside the AppData folder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rstudio/sparklyr/issues/420#issuecomment-272002611,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGglEkRhsnkw0yvc4Iv1GGdAR9-3zY4_ks5rRUukgaJpZM4LgUNM
.

@edgararuiz https://github.com/edgararuiz: I think the installation
directory can be configured by setting the spark.install.dir R option; e.g.
options(spark.install.dir = "C:/path/to/Spark")
This would need to be specified in a startup file (e.g. .Rprofile) to make
sure that new R and RStudio sessions make use of it.
Where exactly would I put this startup file ..does the file exist and do I
put it in the R studio or R path and exactly where
On Wed, Jan 11, 2017 at 9:59 PM, Kevin Ushey<EMAIL_ADDRESS>wrote:

You can configure the temporary directory that R uses for resources by
setting the TMP environment variable. The easiest way to do this in an
R-specific way is to create a file a file in your home directory called
.Renviron, and add the lines:
TMP = C:\path\to\tmp

See e.g. `?tempdir` and `?Startup` for some more information. In general, these deployment issues are the responsibility of the organization attempting to deploy Spark + sparklyr to handle, but we do try to provide as much direction as possible.

@javierluraschi, is it possible to configure the port used for gateway connections? (The port returned by the gateway for connection to the Spark backend)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/rstudio/sparklyr/issues/420#issuecomment-271977457>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AGglElouj0AcpGdG35gBm_uEiI3aAr8rks5rRTS-gaJpZM4LgUNM>
.

Due to Mcafee I need to cahnge the temp directory. It seems taht I have succeeded in forcing R to initiate Spark from a new directory out of Appdata directory path. I think this is correct as the error I get says it invoked Spark.cmdr2 from the new Spark directory.
I need to change the default temp directory setting for R , I tried your suggestion in the Renviron file and it made no difference ..same error. Can you set the default temp directory for R in the .rprofile file and what is the command. Can I just add it at the bottom of the file.
Will this change the scratchpad for Spark to that directory ie say C:\Users\grbortz\tmp

@gbortz27
The way I changed the default temp directory for R is to a new location by changing this from Rprofile.site
In the following way you can edit the file Rprofile.site. Location of this file is R_installed_directory/etc
.First <- function() cat("\n   Welcome to R!\n\n")
setwd("E:/RWORKINGDIR")
Sys.setenv(R_USER = "E:/Users/user1")
Sys.setenv(TMPDIR = "D:/R_packages/Rspark/Temp")
Sys.setenv(TEMP = "D:/R_packages/Rspark/Temp/4")
Sys.setenv(TMP = "D:/R_packages/Rspark/Temp/4")
Sys.setenv(SPARK_HOME = "D:/R_packages/Rspark/spark-2.0.0-bin-hadoop2.6")
.Last <- function()  cat("\n   Goodbye!\n\n")
But unfortunately I still the hive/metastore issue. This is a major issue in windows of sparklyr for spark version > 1.6.2. Till 1.6.2 my sparklyr works fine.
@kevinushey @javierluraschi this is most critical issue.  Please have a look into the same.

@gbortz27 sorry for the late reply, the suggestions from dsblr should help here.
@dsblr have you tried with sparklyr 0.6? Sems like the hive/metastore issue is unrelated to antivirus software, correct? If that's the case, could you open a new issue if you are still experiencing this?
For reference, I ran with the default McAffee trial in Windows 10 under a VM and things worked just fine:


Work In Progress badge

Hi @hughsk!
I always use these labels, but one badge I oftentimes find needed but absent here is work in progress badge. Factually that is a status of project, indicating that it is too early to use it.
What are thoughts about that?
Thanks.

Hello @dfcreative you could create own badge at [![h](https://img.shields.io/badge/Press this link-shields.io-red.svg?link=http//shields.io

don't press that because will not work here

Yeah, now I do like that. I hoped it may become official

This might merely be a difference in definitions, but why not use "Experimental" or "Unstable" for "work-in-progress"?
To me they would both indicate "too early to use"...

@Potherca the difference is that WIP is oftentimes not ready to be used and is going to be finished, like under construction, whereas experimental is proof of concept, i. e. finished to some extend but not developed enough.
For example font-atlas is finished but experimental, whereas audio is work in progress - not ready to be used at all, but planning to be somewhat stable.

@dfcreative Thanks for elaborating! I can see the difference between "WIP" and "Experimental".
I still have trouble grokking the difference between "WIP" and  "Unstable".
To quote Robert Burns:

The best laid schemes o’ Mice an’ Men Gang aft agley

In other words, regardless whether a project might at some point become stable, while work is underway it is not stable.
To my mind, WIP = Unstable (Although not all Unstable project are necessarily WIP). It might just boil down to my perspective that "Unstable" expresses the intent to (someday) become "Stable".
(I hope you don't mind my attempt to better understand the difference. Project life-cycle/stages is a topic that genuinely fascinate me. Enough to have made my own attempt at a definition).


[SPARK-19715][Structured Streaming] Option to Strip Paths in FileSource

What changes were proposed in this pull request?
Today, we compare the whole path when deciding if a file is new in the FileSource for structured streaming. However, this would cause false negatives in the case where the path has changed in a cosmetic way (i.e. changing s3n to s3a).
This patch adds an option fileNameOnly that causes the new file check to be based only on the filename (but still store the whole path in the log).
Usage
spark
  .readStream
  .option("fileNameOnly", true)
  .text("s3n://bucket/dir1/dir2")
  .writeStream
  ...

How was this patch tested?
Added a test case

Test build #73691 has started for PR 17120 at commit aeb10d1.

-1, non binding
I understand the rationale for this, to aid migration from s3/s3n to s3a, but given the need is schema independence, you should be using the full path name from Path.getUri().getPath() instead of getName(), which means only the filename is checked.
match only on name and the two files
s3a://bucket/incoming/dataset.avro
s3a://bucket/2015/12/dataset.avro

will be mistaken for the same file, even when they aren't. If this scenario arises then someone will end up fielding support calls about missing data, or worse, incorrect query results.
If you use the full path, that problem goes away and the filtering is only on schema and filesystem/bucket name.

Test build #73691 has finished for PR 17120 at commit aeb10d1.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 class SeenFilesMap(maxAgeMs: Long, fileNameOnly: Boolean)

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73691/
Test PASSed.

Merged build finished. Test PASSed.

@steveloughran thanks for the comments.
@marmbrus @zsxwing it'd be great if you could share some thoughts!

The use case here is when you have truly unique filenames (i.e. they contain a guid).  This is actually pretty common in my experience.  We definitely shouldn't turn this on by default, but as implemented I think the semantics are pretty clear and this option is useful.

I know that's the current use case, but I'm thinking about future confusion, especially as the use case you espoused, "move from s3n to s3a within the same window" isn't likely to be that common in a running app, is it?
At the very least, the documentation needs to be explicit about what works and what doesn't here.

Note streams can be very long running, so this isn't about some short window.  It could even be that I'm moving to a different bucket (but don't want to loose my exactly once guarantees of a very long running stream).
I agree the documentation should be explicit about the expectations of the filename for this parameter.

Test build #73884 has started for PR 17120 at commit c59f35f.

Thank you @marmbrus @steveloughran for the feedback. I've added some explicit docs. Here's a screenshot of the affected section from the programming guide:

Please take a look again.

Thank you @marmbrus @steveloughran for the feedback. Added some explicit docs. Here's a screenshot of the affected section from the programming guide:

Please take a look again.

Test build #73885 has started for PR 17120 at commit 2354ae6.

Test build #73884 has finished for PR 17120 at commit c59f35f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73884/
Test PASSed.

Test build #73885 has finished for PR 17120 at commit 2354ae6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73885/
Test PASSed.

Merged build finished. Test PASSed.

Test build #73900 has started for PR 17120 at commit f9e525e.

Test build #73900 has finished for PR 17120 at commit f9e525e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73900/
Test PASSed.

Merged build finished. Test PASSed.

Jenkins retest this please

Test build #74062 has started for PR 17120 at commit f9e525e.

Test build #74062 has finished for PR 17120 at commit f9e525e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74062/
Test PASSed.

Test build #74236 has started for PR 17120 at commit 7da2a9c.

Test build #74236 has finished for PR 17120 at commit 7da2a9c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74236/
Test PASSed.

Merged build finished. Test PASSed.

Test build #74245 has started for PR 17120 at commit aab7554.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74245/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins retest this please

Test build #74260 has started for PR 17120 at commit aab7554.

Test build #74260 has finished for PR 17120 at commit aab7554.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74260/
Test PASSed.

LGTM. Merging to master.


Call to undefined function Jose\Util\gmp_init()

Got Call to undefined function Jose\Util\gmp_init() when trying to encrypt (JWE) a JWT.
gmp is installed correctly.
Any idea?

Hi,
Can you tell me where the error occurred (file/line)?

Sure, Error [1]: Call to undefined function Jose\Util\gmp_init() in /web/cloud/vendor/spomky-labs/jose/src/Util/BigInteger.php on line 39

Thanks.
Does this error come from the CLI or from your web application?
You mentioned that the extension is correctly installed and I am pretty sure it is.
If it comes from your web app, my understanding is that the extension is correctly installed on you CLI configuration, but not on your web one.
On Debian/Ubuntu servers, you will find that configuration in the folder /etc/php/{PHP_VERSION}/{apache2 or fpm}/php.ini

You have to find the location of the php.ini file.
Create a web page:
<?php

phpinfo();

Check if the gmp module is available and where the php.ini is.
If the module is not available, then just add extension=php_gmp.so or extension=gmp.so

Closing as not related to the library but to the environment.

@Spomky I am running into this same issue. I have GMP installed and included it in my php.ini file. I am trying to create an RSAKey pair like so:
$private_key = JWKFactory::createRSAKey(     4096, // Size in bits of the key. We recommend at least 2048 bits.     [         'alg' => 'RSA-OAEP-256', // This key must only be used with the RSA-OAEP-256 algorithm         'use' => 'enc',    // This key is used for encryption/decryption operations only     ]);
Then building my JWE like this:
$jwe = $jweBuilder->create()->withPayload($payload)->withSharedProtectedHeader(['alg' => 'RSA-OAEP-256', 'enc' => 'A256CBC-HS512',])->addRecipient($public_key)->build();
If i run your example provided in the documentation for building a JWE i do not get the error but when building a different Key then i run into the error. Any ideas of what may be the issue?
Error:
Fatal error: Uncaught Error: Call to undefined function gmp_init() in /Volumes/Source/authserver/vendor/web-token/jwt-core/Util/BigInteger.php:47 Stack trace: #0 /Volumes/Source/authserver/vendor/web-token/jwt-core/Util/RSAKey.php(213): Jose\Component\Core\Util\BigInteger::createFromBinaryString('0xd4b03a1d0e2b4...') #1 /Volumes/Source/authserver/vendor/web-token/jwt-core/Util/RSAKey.php(188): Jose\Component\Core\Util\RSAKey->convertBase64StringToBigInteger('1LA6HQ4rQ3q1b20...') #2 /Volumes/Source/authserver/vendor/web-token/jwt-core/Util/RSAKey.php(83): Jose\Component\Core\Util\RSAKey->populateBigIntegers() #3 /Volumes/Source/authserver/vendor/web-token/jwt-core/Util/RSAKey.php(92): Jose\Component\Core\Util\RSAKey->__construct(Object(Jose\Component\Core\JWK)) #4 /Volumes/Source/authserver/vendor/web-token/jwt-encryption-algorithm-rsa/RSA.php(30): Jose\Component\Core\Util\RSAKey::createFromJWK(Object(Jose\Component\Core\JWK)) #5 /Volumes/Source/authserver/vendor/web-token/jwt-encryption/JWEBuilder.php(366): Jose\Component in /Volumes/Source/authserver/vendor/web-token/jwt-core/Util/BigInteger.php on line 47

@juanm18 What is the output of php --ri gmp? Also, I think @Spomky is focusing on https://github.com/web-token/jwt-framework as the replacement for this lib. Not sure he is doing much more than security fixes on this now.

Sorry i totally overlooked the repo name but i am using the jwt-framework library. As for your question, this is what i get
`gmp
gmp support => enabled
GMP version => 6.1.2`  @moderndeveloperllc


[Docs] Mesheryctl mesh deploy command is missing in the documentation

Current State
In the https://docs.meshery.io/reference/mesheryctl#service-mesh-lifecycle-and-configuration-management the mesheryctl mesh deploy command is missing.
Desired State
In the https://docs.meshery.io/reference/mesheryctl#service-mesh-lifecycle-and-configuration-management the mesheryctl mesh deploy command should be added.
Additional Resource
The more about this command can be found in this spreadsheet

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

deploy

deploy a service mesh into the cluster
mesheryctl mesh deploy

/assign

Here you go @afzal442


Added hierarchy of exceptions

Continuation of #142. /claim #74
Hi @domartynov @jdegoes , what do you think? I have created a separate ZTimeoutException to handle SQLTimeoutException.
Also, the hierarchy is now flatten and captures Fatal and Recoverable errors. I have fixed the build so it should pass all the tests.

Now it is fixed the CI in Scala 3

@pablf Can you resolve conflicts when you get a chance? 🙏


pxd files for posix and libc should use system includes

Currently, all *.pxd files in Cython/Includes use the cdef extern from "some_header.h" syntax instead of cdef extern from "<some_header.h>". I believe at least for libc and posix, Cython should generate system includes instead.
Migrated from http://trac.cython.org/ticket/878

Fixed in https://github.com/cython/cython/commit/7bc65befc98b3d52ab3a38d0ee098724a72c16d7 and predecessors.


Add to docker

The necessary files are generated so that developers can have a universal development environment.
We just have to build the app { docker-compose build }
Generate migrations { docker-compose run --rm app rake db:create db:migrate }
Start the app { docker-compose up }

Thank you!

@mudstart i will update the readme with this information for better enviroment setup


Merge nav page updates to live

Docs Build status updates of commit 4da9c98:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/how-to-open-a-spreadsheet-document-for-read-only-access.md
:white_check_mark:Succeeded

docs/packages-and-general.md
:white_check_mark:Succeeded

docs/presentations.md
:white_check_mark:Succeeded

docs/spreadsheets.md
:white_check_mark:Succeeded

docs/word-processing.md
:white_check_mark:Succeeded

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


Popup Notifications - Save Settings

Hello,
First of all thank you for vmPing!  Excellent ping software.  Is there a way of saving popup notifications settings so that when you start the application, it's default on "Never" or anything you want it to be?
Thanks for a quick answer :-)

+1

Hey there.  I didn't realize I wasn't saving that option.  Should be fixed in the next version.

Thanks so much!
From: Ryan Smith<EMAIL_ADDRESS>Sent: 24 August 2020 15:40
To: R-Smith/vmPing<EMAIL_ADDRESS>Cc: simonabraham<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [R-Smith/vmPing] Popup Notifications - Save Settings (#41)
Hey there. I didn't realize I wasn't saving that option. Should be fixed in the next version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/R-Smith/vmPing/issues/41#issuecomment-679203751 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AP76PIVN56ZSMRUTQEZW4QTSCKCVPANCNFSM4OBPHTFQ .  https://github.com/notifications/beacon/AP76PIW4PKOQBYU6YWX75P3SCKCVPA5CNFSM4OBPHTF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFB55HJY.gif
--
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus

hi i love your program but it's still not fixed

Add here: 6a19572db6b90ea37404718bea4613100acea016, here: 3b3eca7b7242adc14ceadc82a28f09a1af0bb28e, and here: a3564d6f633ec0672fe6b12adaccb63807be6303
This will be included in the next release.

The changes are now available in release 1.3.8.


i was do a server part + litle client

This pull request includes several changes primarily related to project configuration files for IntelliJ IDEA, Git, and Maven. The changes aim to streamline project setup and management by adding and modifying configuration files and .gitignore entries.
IntelliJ IDEA Configuration:

Added vcs.xml to configure version control settings for the project. (.idea/vcs.xml, .idea/vcs.xmlR1-R6)
Updated workspace.xml to remove old change list entries and add new configuration settings. (.idea/workspace.xml, [1] [2] [3] [4]

Git Configuration:

Added .gitignore entries to exclude build directories, IDE-specific files, and OS-specific files. (LibraryClient/.gitignore, LibraryClient/.gitignoreR1-R38)
Added a .gitignore file specific to IntelliJ IDEA to exclude certain directories and files. (LibraryClient/.idea/.gitignore, LibraryClient/.idea/.gitignoreR1-R8)

Maven Configuration:

Added maven-wrapper.properties to configure Maven wrapper settings. (LibraryClient/.mvn/wrapper/maven-wrapper.properties, LibraryClient/.mvn/wrapper/maven-wrapper.propertiesR1-R2)
Added mvnw script to allow running Maven commands using the Maven wrapper. (LibraryClient/mvnw, LibraryClient/mvnwR1-R316)

Additional IntelliJ IDEA Project Files:

Added encodings.xml to set file encodings for the project. (LibraryClient/.idea/encodings.xml, LibraryClient/.idea/encodings.xmlR1-R7)
Added misc.xml to configure various project settings, including the Java SDK version. (LibraryClient/.idea/misc.xml, LibraryClient/.idea/misc.xmlR1-R14)
Added vcs.xml to set up version control mappings for the project. (LibraryClient/.idea/vcs.xml, LibraryClient/.idea/vcs.xmlR1-R6)

@wd29a


classgraph 4.8.115 (was 4.8.90)

Updates io.github.classgraph:classgraph from 4.8.90 to 4.8.115.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.classgraph", artifactId = "classgraph" } ]

labels: library-update, semver-patch

Superseded by #195.


[Question]: Can you publish an article referencing your project?

Describe your problem
Can you publish an article referencing your project? If so, how should I cite the reference?
If you need further assistance or specific guidelines on how to structure your article or citation, feel free to ask!

Hi @Darryl-lilin, Apologies, but I don't quite understand your point. Could you elaborate a bit?

Blog articles about RAGFlow can be found on either medium.com or our official site, which is ragflow.io.

Hi @Darryl-lilin, Apologies, but I don't quite understand your point. Could you elaborate a bit?嗨，抱歉，但我不太明白你的意思。您能详细解释一下吗？

I want to make modifications based on the code and then write a paper to publish it.

Thanks for the clarification. You can add a reference to our github repo, https://github.com/infiniflow/ragflow, in your paper.

Thank you for your response.


adds eslint

Adds a linting npm task and fixes lint errors that were in the project.

:money_with_wings:
LGTM

LGTM :+1:


bluemap: 3.21 -> 5.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/BlueMap-Minecraft/BlueMap/releases.
meta.description for bluemap is: 3D minecraft map renderer
meta.homepage for bluemap is: https://bluemap.bluecolored.de/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.4 with grep in /nix/store/5d775bfg3djy0p2if1cgmln0fa2cyzwq-bluemap-5.4
found 5.4 in filename of file in /nix/store/5d775bfg3djy0p2if1cgmln0fa2cyzwq-bluemap-5.4

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

bluemap

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5d775bfg3djy0p2if1cgmln0fa2cyzwq-bluemap-5.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A bluemap https://github.com/r-ryantm/nixpkgs/archive/8b3ea38036b96d4a493784bf9f811bc81f301a8a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8b3ea38036b96d4a493784bf9f811bc81f301a8a#bluemap

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5d775bfg3djy0p2if1cgmln0fa2cyzwq-bluemap-5.4
ls -la /nix/store/5d775bfg3djy0p2if1cgmln0fa2cyzwq-bluemap-5.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

bluemap

Maintainer pings
cc @dali99 @h7x4 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Will fix in #332245


Add RBAC support to Logentries (aka: InsightOps)

This PR provides an RBAC policy for LogEntries (aka: InsightOps). It fixes this error that the fluentd pod produces when there is no RBAC policy:
2019-05-07 08:35:57 +0000 [error]: config error file="/fluentd/etc/fluent.conf" error_class=Fluent::ConfigError error="Exception encountered fetching metadata from Kubernetes API endpoint: pods is forbidden: User \"system:serviceaccount:kube-system:default\" cannot list resource \"pods\" in API group \"\" at the cluster scope ({\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"pods is forbidden: User \\\"system:serviceaccount:kube-system:default\\\" cannot list resource \\\"pods\\\" in API group \\\"\\\" at the cluster scope\",\"reason\":\"Forbidden\",\"details\":{\"kind\":\"pods\"},\"code\":403}\n)"

I've moved on and am no longer using fluentd with Logentries, so I won't have anything to test with. I'll go ahead and close this PR. Anyone is welcome to take the changes in this PR and move it to a separate file.


fix locally failing tests on linux

There are some missing required features for the examples needed for tests to pass locally

The problem with pango seems to be druid/cargo.toml:90 requiring GTK. I dont want to remove it in this PR yet since I dont know the reason behind those lines.


Trying to decrease VM size and running into some issues

Hello, all
I am trying to create a new deployment file for minimizing VM size, and I started the work based on https://github.com/cloudfoundry-community/cf-boshworkspace
It looks it's not maintained right now, my question got no response for several days.
Anyway, I create an deployment file based on it, and when deploying I always got error about consul component.
The deployment file starts from https://github.com/cloudfoundry/cf-release/tree/master/example_manifests and adds diego components initially.
  Started updating job backbone_z1 > backbone_z1/0 (canary)
  Started updating job data > data/0 (canary)
  Started updating job services_z1 > services_z1/0 (canary)
  Started updating job health_z1 > health_z1/0 (canary)
  Started updating job cell_z1 > cell_z1/0 (canary)
  Started updating job ha_proxy_z1 > ha_proxy_z1/0 (canary)
  Started updating job api_z1 > api_z1/0 (canary)
  Started updating job brain_z1 > brain_z1/0 (canary)
     Done updating job backbone_z1 > backbone_z1/0 (canary) (00:02:24)
     Done updating job brain_z1 > brain_z1/0 (canary) (00:03:25)
   Failed updating job ha_proxy_z1 > ha_proxy_z1/0 (canary): `ha_proxy_z1/0' is not running after update (00:10:28)
   Failed updating job data > data/0 (canary): `data/0' is not running after update (00:10:39)
   Failed updating job cell_z1 > cell_z1/0 (canary): `cell_z1/0' is not running after update (00:10:55)
   Failed updating job health_z1 > health_z1/0 (canary): `health_z1/0' is not running after update (00:11:25)
   Failed updating job services_z1 > services_z1/0 (canary): `services_z1/0' is not running after update (00:11:27)
   Failed updating job api_z1 > api_z1/0 (canary): `api_z1/0' is not running after update (00:14:08)

Error 400007: `ha_proxy_z1/0' is not running after update

And after checking the log of ha_proxy_z1 job, I can see the main problem is the consul.
Finally Itt complains
==> Error starting agent: Failed to start Consul client: Failed to start lan serf: Failed to start TCP listener. Err: listen tcp <IP_ADDRESS>:8301: bind: address already in use

I checked on on working cf instance, and I can see the same consul processes are running, and I didn't see any difference between them.
minimal-vms.txt
ha_proxy_z1.0.2016-03-17-19-05-08.zip
Attached are the deployment file and logs of ha_proxy_z1.
Could someone show some guide on it, thanks in advance.

In my case, I see error "Err: listen tcp <IP_ADDRESS>:8301: bind: address already in use", which means the consul_agent is not started correctly, and I also see it's not correct via "sudo monit summary"
But I do see they are running
root@81a1c783-da5d-4d5d-a419-c2c44345e285:/var/vcap/sys/log/consul_agent# ps axu | grep consul
root      4171  0.0  0.0  18124  1696 ?        S<   10:50   0:00 /bin/bash -exu /var/vcap/jobs/consul_agent/bin/agent_ctl start
root      4172  0.0  0.0  18124  1696 ?        S<   10:50   0:00 /bin/bash -exu /var/vcap/jobs/consul_agent/bin/agent_ctl start
root      4173  0.0  0.0   4352  1404 ?        S<   10:50   0:00 tee -a /var/vcap/sys/log/consul_agent/consul_agent.stdout.log
root      4174  0.0  0.0   4344  1488 ?        S<   10:50   0:00 logger -p user.info -t vcap.consul-agent
root      4175  0.0  0.0   4352  1400 ?        S<   10:50   0:00 tee -a /var/vcap/sys/log/consul_agent/consul_agent.stderr.log
root      4176  0.0  0.0   4344  1524 ?        S<   10:50   0:00 logger -p user.error -t vcap.consul-agent
vcap      4184  0.2  0.4<PHONE_NUMBER>0 ?        S<l  10:50   0:47 /var/vcap/packages/consul/bin/consul agent -config-dir=/var/vcap/jobs/consul_agent/config -recursor=<IP_ADDRESS>
root     24312  0.0  0.0  10472  2176 pts/0    S+   17:15   0:00 grep --color=auto consul

Hi, @StanleyShen,
I'm closing this issue since it's related to cf-release, not diego-release. Please open an issue on the cf-release repository if you're seeing issues colocating or deploying certain jobs from cf-release. Also please include what version of CF you're using and how you're generating the manifest when you file the report.
One thing you could check on the failing VMs is what processes have already bound to port 8301. Use bosh ssh to log into the VM, run sudo -i to get root, and then run netstat -anp and look for port 8301 in the output. From the ps output, it may be that monit has lost track of the consul job.
Best,
Eric