fetch doesn't work with Brotli on Android

Description
When making an HTTP request with br on Accept-Encoding, the request throws an error on Android, but works perfectly well on iOS
React Native Version
0.72.5
Output of npx react-native info
System:
  OS: Linux 5.15 Linux Mint 21.2 (Victoria)
  CPU: (4) x64 AMD A8-9600 RADEON R7, 10 COMPUTE CORES 4C+6G
  Memory: 11.27 GB / 15.07 GB
  Shell:
    version: 5.8.1
    path: /usr/bin/zsh
Binaries:
  Node:
    version: 20.8.0
    path: ~/.nvm/versions/node/v20.8.0/bin/node
  Yarn:
    version: 3.6.3
    path: /usr/bin/yarn
  npm:
    version: 10.1.0
    path: ~/.nvm/versions/node/v20.8.0/bin/npm
  Watchman: Not Found
SDKs:
  Android SDK: Not Found
IDEs:
  Android Studio: AI-223.8836.35.2231.10671973
Languages:
  Java:
    version: <IP_ADDRESS>
    path: /usr/bin/javac
  Ruby: Not Found
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.72.5
    wanted: ^0.72.5
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: Not found
  newArchEnabled: Not found
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

Steps to reproduce
Try to run the following code, both on iOS and Android:
import { useEffect } from 'react';
import { Alert, Text, View, StyleSheet } from 'react-native';

export default function App() {
  useEffect(() => {
    async function doApiRequest() {
      try {
        const result = await fetch('http://worldtimeapi.org/api/timezone/America/Sao_Paulo', {
          method: 'GET',
          headers: {
            'Accept-Encoding': 'br'
          }
        })

        const json = await result.json()
        Alert.alert(JSON.stringify(json))
      } catch (error) {
        Alert.alert('error', JSON.stringify(error))
      }
    }

    doApiRequest()
  }, [])

  return (
    <View style={styles.container}>
      <Text style={styles.paragraph}>
        Hey!
      </Text>
    </View>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    backgroundColor: '#ecf0f1',
    padding: 8,
  },
  paragraph: {
    margin: 24,
    fontSize: 18,
    fontWeight: 'bold',
    textAlign: 'center',
  },
});

Snack, screenshot, or link to a repository
https://snack.expo.dev/@andresribeiro/effe49

the request throws an error on Android, but works perfectly well on iOS

What's the error?

@cortinico on the example i provided, it catchs "{}" on the try catch

I think we would need to add this: https://github.com/square/okhttp/blob/master/okhttp-brotli/README.md

We would need to get internal version of OKHttp updated further to do this I think (we recently moved that from 3.6.0 to 3.14.9, but that is still behind what OSS RN is on).

I think we would need to add this<EMAIL_ADDRESS>
I think the user should be able to override the default OkHTTP client, and provide its own implementation with the Brotli interceptor. We don't need this to live inside core afterall.
Also the OSS version of OkHTTP we use is 4.9.x so the interceptor should just work

I think we would need to add this<EMAIL_ADDRESS>
I think the user should be able to override the default OkHTTP client, and provide its own implementation with the Brotli interceptor. We don't need this to live inside core afterall. Also the OSS version of OkHTTP we use is 4.9.x so the interceptor should just work

I fully understand that React Native tries to keep it's core as minimal as possible, but I really think that this should be implemented inside core
1 - Brotli it's an official content encoding
2 - It's supported and used by all major browsers
3 - It's used by a lot of companies, including Cloudflare
4 - Even Expo started using Brotli for the expo-updated

I fully understand that React Native tries to keep it's core as minimal as possible, but I really think that this should be implemented inside core

This is an implementation detail, and it's up to us to decide where the integration point should live. As a last resort, we could just provide a thin layer that adds Brotli and other commonly used interceptors in a separate NPM package that gets added by default.
Still you can use this today by feeding your own OkHTTP instance, so there is no blocker in using it.

I tend to agree it would be best for RN to provide this out of the box. 99% of users won't know to add this, and it means network may be more expensive for RN users on Android in the meantime.
As a matter of practicality, I don't think RN will add this in the near-term, just because of the internal OKHttp dependency. But as @cortinico has mentioned, RN does allow customizing the default OKHttp client, so you could add this for your app.


Allow to control "ControllerUserInteractionEnabled" on tvOS

Apple's HIG requires that the menu button of the remote exits the game to the home screen if you are on the first screen of an app.
This is achieved by

Setting GCEventViewController.ControllerUserInteractionEnabled = true
Calling base.PressesBegan() when handling input
Afterwards, the property can be set to false again.

MonoGame handles this already correctly. IMHO it would not be the worst idea to derive from iOSGameViewController (which is unfortunately not public in MonoGame). This would prevent reinventing the wheel. Otherwise, CocosSharp should provide its own GCEventViewContrller subclass which handles this.
At the moment I handle it like this (changed CCGameView.tvOS.cs):
public override void PressesBegan (NSSet<UIPress> presses, UIPressesEvent evt)
        {
            FillPressesCollection (presses);

            if(this.UseDefaultBackButtonHandling && presses.AnyObject != null && presses.AnyObject.Type == UIPressType.Menu)
            {
                var eventController = UIApplication.SharedApplication.Delegate.GetWindow().RootViewController as GCEventViewController;
                if(eventController != null)
                {
                    eventController.ControllerUserInteractionEnabled = true;
                }
                base.PressesBegan (presses, evt);
                if(eventController != null)
                {
                    eventController.ControllerUserInteractionEnabled = false;
                }
            }
        }

@Krumelur I think you are on the right track. In MG we only call base.PressesXXXX if we get a Menu press AND want to exit.. but that only really works if you View is derived from GCEventViewController.
Also it seemed that ControllerUserInteractionEnabled needed to be true for all the PressesBegin/Ended events, so we reset it to false when the app comes back into focus again.

@Krumelur I might need to see the code, but I'm not following your approach of moving this logic to the game view. As of CocosSharp <IP_ADDRESS>-pre1, the setup is to essentially bring your own view controller and associate it with a CCGameView. So with respect to tvOS, why can't you simply subclass  GCEventViewController and slap a CCGameView onto it? As in, make sure the current ViewController is of type GCEventViewController rather than simply UIViewController. Then you can put the above logic within your subclassed view controller.
Yes, I suppose this is slightly more cumbersome than the current MG approach, but there's a few things to keep in mind. Firstly, the project templates can simply be updated to ensure the view controller is correctly setup when targeting tvOS. I think this is something that will have to be addressed not just for CocosSharp, but for all tvOS apps.
Secondly, what happens when a user wants a multiple tabbed tvOS application? With the MG setup of taking over the entire application, this simply isn't possible. This was precisely the motivation for us to bring our interface up to the View level, to give developers control of the layout of their native application and the freedom to attach their game view where ever they see fit.

@rtabbara Yes, I can use my own GCEventViewController and in fact, I do. But the question is: how much responsibility should I have as the user of CS and how much can CS do for me? If a tvOS app must exit if the menu button is clicked, I'd like to be able to handle this inside CS, without any native code. The problem I see is:

CocosSharp allows me to detect the press of the menu button conveniently via an abstraction (some listener)
This allows me to find out that "menu" has been pressed
However, now it is already too late to handle the press natively: PressesBegan() has already been run when I get the callback
If I want to exit the game if "menu" is pressed, PressesBegan() would have to call base.PressesBegan() and set eventController.ControllerUserInteractionEnabled = true (if I'm using GCEventViewController) to allow tvOS to handle the press and run the exit procedure (there is no way on tvOS to exit to the main menu directly - tvOS handles this internally if the press bubbles all the way up)
Problem 1: CS does not call base
Problem 2: CS does not set ControllerUserInteractionEnabled, after all, it is not aware of the GVEventViewController

If we change CS to always call base, my only option is to override PressesBegan() in my GCEventViewController, do the detection of "menu" in there natively and handle it. But like I already said: why provide an abstraction and then we require users to use native code to handle the same thing again.
I'd like to see something where I can set a button click to a handled or not handled state. CS would tell me "menu was pressed!" and my code decides to return a value that indicates that I don't want to handle it but let the default behavior run instead. On tvOS this would let the event bubble up and exit my game.
Bottom line is: I'd really like Apple to provide a proper method to exit to the home screen instead of having the current behavior. If you google for this issue, you'll find that many Apps have been rejected because they don't exit properly. If so many people fail in using an API, the API is no good.

If we change CS to always call base, my only option is to override PressesBegan() in my GCEventViewController, do the detection of "menu" in there natively and handle it. But like I already said: why provide an abstraction and then we require users to use native code to handle the same thing again.

Because CCGameView is not an abstraction. It's just a UIView that happens to render game content. To me, it would be akin to requiring a  UITableView to handle the exit code. It just seems very strange.
But I understand the frustration. To me the crux of the issue is this requirement of having to toggle ControllerUserInteractionEnabled. As you've said, this is a very bizarre setup.
We'll have to think a bit more carefully about how to do this right. Something along your suggestion of having a listener/event handler for when the menu-button is pressed seems to be a good compromise.


PySimpleGui Themes in Vdiclient.ini.example

On line 4 of the vdiclient.ini.example file, it reads as:
This is the PySimpleGui Theme that is used. Run pvevdi.py with flag --list_themes for a list of themes
this should be:
This is the PySimpleGui Theme that is used. Run vdiclient.py with flag --list_themes for a list of themes
i've attached a screenshot if needed

Great Project saves me having to log in to proxmox UI every time I need access.

Fixed in commit c2221da


Add basic CI using Github Actions

Trello: https://trello.com/c/PogZfjR4/517-work-out-how-to-run-the-performance-tests
Check that it compiles.

I think we may be able to do this without a third-party action, could we try e.g. like this example? https://github.com/actions/starter-workflows/blob/main/ci/scala.yml

Good idea - that works


Changing Delay times for Overworld

seems like changing the delay times for the overworld just breaks it after the first song plays

the vanilla datapack contained some biome music that wasn't affected by the mod, 0.1.5 should fix this


Is there a way to fetch a particular version of a document? (not the latest)

I'm interested in fetching the whole JSON for a document as it existed at a particular version number

Did you try mongoose-patch-history yet?


Driver is receives proposed match from rider after the rider has cancelled the ride request

This is a bit of an issue for the demo because it's impossible to test any rider-driver set... even if you cancel the ride request, the driver's next proposed match email will contain 2 of the same address 😱

@sasjkia probably a bit tricky to patch this in time for the demo 😞

what time is the demo?
I have a fix
On Wed, Jun 7, 2017 at 7:38 AM, Jon Kelly<EMAIL_ADDRESS>wrote:

@sasjkia https://github.com/sasjkia probably a bit tricky to patch this
in time for the demo 😞
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/voteamerica/backend/issues/166#issuecomment-306644259,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALXSaCJUIvCZmzEVAKnFUxciYT1afGGnks5sBeMMgaJpZM4Nxh-L
.

fix is on test environment, I can load it on live if you can confirm your
demo scenario works on test
On Wed, Jun 7, 2017 at 7:48 AM, David Milet<EMAIL_ADDRESS>wrote:

what time is the demo?
I have a fix
On Wed, Jun 7, 2017 at 7:38 AM, Jon Kelly<EMAIL_ADDRESS>wrote:

@sasjkia https://github.com/sasjkia probably a bit tricky to patch
this in time for the demo 😞
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/voteamerica/backend/issues/166#issuecomment-306644259,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALXSaCJUIvCZmzEVAKnFUxciYT1afGGnks5sBeMMgaJpZM4Nxh-L
.

@sasjkia @dmilet fix looks good

This still open?

@gmisura hi there, thanks for the interest @sasjkia @dmilet is this still an issue?

@gmisura I think this has been resolved.
You are welcome to take a look at https://github.com/voteamerica/backend/issues/163
Bear in mind that issues like these involve the entire system. Sometimes the issue isn't totally clear whether it's web page or email/sms issue. The docker setup doesn't send emails or sms messages, so these issues involve looking at the db tables for "outgoing_email" and "outgoing_sms"
Let me know if I can assist in any way


refactor homogeneous transforms module

This PR tries to fulfill the feature request in #74
Implements:

relative_transformation
inverse_transformation
compose_transformations.

Moves transform_points from module conversions to transformations.

@versatran01 this is a first proposal for #74 .
Take a look at it and share your thoughts.

@edgarriba Thanks for your effort. I have a few suggestions.

eps should default to 0, I don't know why pytorch people suggest it but if you add eps to a rotation matrix, then it is not a rotation matrix any more (det R != 1) so I'm strongly against that.

The check in each function for tensor shape should allow both (Bx4x4) and (4x4) tensor.
As a result, in inverse_transformation

rmat_21: torch.Tensor = torch.transpose(rmat_12, 1, 2)

should become
transpose(mat_12, -1, -2)

In compose transformation, the parameters should be named T_0_1 and T_1_2 to emphasize that they are composable. And the return value should be named T_0_2.

In relative_transformation, the frames of each transformation should be clearly stated as opposed to just trans1 and trans2. For example, one could force these two transformations to be expressed wrt to a common frame, thus the function signature should be

def relative_transformation(T_0_1, T_0_2) -> T_1_2:
    return (T_0_1)^-1 * T_0_2 = T_1_0 * T_0_2  = T_1_2

@versatran01 @tiancheng-zhi could you do a quick review before I merge ? those are the basics, we can fix/improve later.

@edgarriba The current implementation contains bug. I tested DepthWarper and found it cannot warp the image correctly. I'm not sure where the bug is currently (maybe in DepthWarper, or other functions). I'm investigating this and will come back later.

@edgarriba @versatran01 I found the bug: From my point of view,  trans_02=compose_transformation(trans_01, trans_12) should equals trans_02=torch.matmul(trans_12, trans01). Right? But the current code does not perform like this.

@tiancheng-zhi I don't think that's the bug. From my guess (I haven't looked at the code), before the code could be using extrinsics which is T_c_w, with the refactor of transformation, all transformation are poses which means they are T_w_c, so some care has to be taken when using these new functions.

I assume the extrinsic matrix in DepthWarper is "world-to-camera" rather than "camera-to-world". Looking at how DepthWarper calls relative_transformation, I guess the main issue is the definition of "trans_AB". Does it mean the matrix transforms a point from the view of A to the view of B? If so, then what @versatran01 mentioned:
def relative_transformation(T_0_1, T_0_2) -> T_1_2:
return (T_0_1)^-1 * T_0_2 = T_1_0 * T_0_2  = T_1_2
is incorrect. The correct one should be:
def relative_transformation(T_0_1, T_0_2) -> T_1_2:
return T_0_2 * (T_0_1)^-1

@edgarriba @versatran01
I guess there are several things need to clarify:

Is camera extrinsics "world-to-camera"?
What does it mean by transformation trans_AB? Is trans_AB a matrix that transforms a point in frame A to frame B? i.e., X_B = trans_AB * X_A ？
What is the definition of compose_transformation?
since X_B = trans_AB * X_A
similarly,    X_C = trans_AC * X_A
X_C = trans_BC * X_B
Then should trans_AC = trans_BC * trans_AB ?
What is the definition of relative_transformation? Should the relative transformation between trans_AB and trans_AC be " trans_BC = trans_AC *  (trans_AB)^-1 " ?

It looks like all the problems come from the unclear definitions.

@tiancheng-zhi
As for the definition of T_0_1:
p_0 = T_0_1 * p_1
This is a convention in robotics/cv.

Extrinsics is world to camera which is T_c_w.
T_A_B is B to A. so X_A = T_A_B * X_B.
you got the convention wrong, if you use the correct one you will see that they are implemented correctly.
relative transformation is very vague, a common definition is
def relative_transformation(T_0_1, T_0_2) -> T_1_2:

@versatran01 Thanks very much for your clarification.
@edgarriba  According to the convention, should Line 84 in depth_warper be "relative_transformation(inverse_transformation(extrinsics_dst), inverse_transformation(extrinsics_src))" rather than "relative_transformation(extrinsics_src, extrinsics_dst)"?
(Note that there're two changes: 1. added inverse, 2. swapped src and dst)

hey, thanks for the review. Just few things:
@versatran01

Regarding to the transform_points - it's a generalized function that convert points from different types of transformation such as homographies, homogeneous transforms. So, we cannot assume all the time that there's an Rt. In the case of the homography, we need the whole transformation. I'd keep as it is. Notice that I fixed the epsilon thing in the homogeneous conversion.
Regarding the signatures name, is the first time I see it in this convention (boxplus and boxminus) but if you think it's worth changing and make the whole thing more readable just let's do it. I suggest to do it in a new PR and before the end of this week since I want to do the release of v0.1.2 as soon as posible.

@tiancheng-zhi the convention I was using is the one that @versatran01 points out. Just to make it clear again: transformation from B to A == T_A_B == trans_ab. We read it from right to left and to make the code more readable I prefer to name the transformation variables trans_  rather that T_.
Regarding, the issue you mention in DepthWarper, do the tests fail in your machine ? Because I'm trying it, and seems to produce the expected output. What's more, by adding your fix seems to produce the same output.
@versatran01 @tiancheng-zhi Besides, I would like to hear your opinion about the DepthWarper API  since I'm still not convinced the way it works. I adapted it from old codes that we had which accepts in the constructor a bunch of pinhole cameras, however, in practice I'm trying this pattern in a project and it doesn't scale at all. I'm thinking to go back and just accept a single PinholeCamera object instead of a list of them.

@edgarriba  Plase see https://github.com/tiancheng-zhi/reproduce_bug to reproduce the bug. The images, depth map, and poses are taken from ScanNet dataset.
About the depthWarper API, I feel that there's no need to input a list of cameras. If the user wanna do so, he/she can make it a batch or use a for-loop by himself/herself.

@edgarriba

I agree with you, so just leave it that way.
As for the notation boxplus/boxminus, see this: https://arxiv.org/abs/1107.1119

I assume this code is taken and adapted from sfmlearner-pytorch?
In my own implementation, I use inverse depth instead of depth, which handles point at infinity and also more numerically stable (no divide by 0 any more hence no eps needed). My function signature looks like this
warp(images: [B3HW], idepths: [B1HW], cameras: [B4], poses [B44]) -> warped [B3HW]

@versatran01 right, I took the code from there. Actually, @ClementPinard helped with that porting and he's willing to contribute with some stuff as well. So, do you think is it worth to reimplement using idepth instead ? Do you have the code anywhere to take a look at it ? I actually like the pattern of having the warpers as modules since that's how we have been using it for a while at arraiy, but it can be discussed.

My code is currently private, but I'm willing to share it once I clean it up a bit.
I think it makes sense to warp using inverse depth because that's what the network directly predicts.
It can be both a module and a function.

@versatran01 sounds good. I guess that the inverse depth feature can be a future improvement. Let me learn about the signature name but I prefer to improve it later as well. If you don't mind could you open tickets to keep tracked about this so that we can organize ourselves and assign tasks ?

Hi thanks for this contribution.
Just wrote down on paper the inverse warp (note that it's not a warp operation) operation with inverse depth, it checks out and it's indeed more elegant regarding the stability argument : instead of multiplying the point cloud to depth so that (u,v,1) is (x,y,z) you multiply the translation with inverse depth and everything is back to normal.
It's worth noting though that this implies at some point the existence of two [B,H,W,3] arrays : R*(u,v,1) and 1/z * (Tx, Tz, Tz) while before we could broadcast the point cloud with the [B,3] translation tensor and have only one. Don't know how it would result memory wise for GPUs.

Also regarding the eps parameter, I don't advocate for its application like this, but you still have to treat the case where the resulting depth goes to negative values, for example in the case of big rotations. otherwise, the grid_sample might output garbage while it just should output 0. (Note that this does not imply adding an eps values, but rather clamping the depth values to a minimum of eps >0)
With the current implementation how can we know from homogeneous coordinates that the points did have a valid depth (or inverse depth) ?

@ClementPinard thanks for the feedback. I will open a ticket to state this. I guess this feature might be included after the v0.1.2 release.


Add a Help menu item

In GitLab by @nwalczak on Jul 3, 2019, 10:18
It would be good to have some documentation to help users out.  We could also use it to print version information for the script, to help ensure users are using the proper version.

In GitLab by @nwalczak on Jul 3, 2019, 10:18
Here's the current text I have in mind:

Intent Parser version ${VER_STRING}
</p

The purpose of the intent parser add-on is to create a suite of tools that aids the user in creating well documented experimental plans.

Right now, there are three different actions that can be performed to help the user.  The first action is analyzing the document, which searches for terms present in the SD2 spreadsheet dictionary and offers to insert links to the SynbioHub definition of those terms.  Additionally, the document can be scanned for terms that don't exist in the spelling dictionary and suggest them as terms to possibly add into the SD2 spreadsheet dictionary or to have a link manually added.  Users can also highlight arbitrary terms and add a definition to SynbioHub for that term.  This add dialog will also query SynbioHub for matches to the selected term and links to those terms can be added.

In GitLab by @nwalczak on Jul 3, 2019, 11:44
created branch 70-add-a-help-menu-item

In GitLab by @nwalczak on Jul 3, 2019, 11:44
mentioned in merge request !23

In GitLab by @nwalczak on Jul 3, 2019, 13:48
closed via merge request !23

In GitLab by @nwalczak on Jul 3, 2019, 13:48
closed via commit 372c06fcea7b588a15f44c59ff5f5a5258e03e01

In GitLab by @nwalczak on Jul 3, 2019, 14:26
mentioned in commit 372c06fcea7b588a15f44c59ff5f5a5258e03e01


Memory leak when closing over large constants in compiled jax.lax.cond statement

Description
I was running into an issue that I could condense to an elementary example. It seems to me as if closing over large constants in lax.cond creates a memory leak.
I've reported a similar issue concerning lax.fori_loop in #11448 which was solved in PR #11461. However, my followup post in the original issue got no attention so I decided to close the original issue since the originally reported issue was fixed and to open this one. I hope that is ok.
I'm aware that closing over large constant is not recommended and instead they should be passed as arguments. Nonetheless, this leaking behaviour is unexpected. Maybe it can be -- similar to #11448 --  easily fixed taking weak references to true_fun and false_fun.
MRE
The side-effect-free function leak creates in ints body a large constant and later compiles a function which closes over the large constant within a lax.cond statement. The compiled function called. When leaving the function leak, all references to the large constant are destroyed, including the closure and the compiled closure and the memory should be freed. However, it is not.
import numpy as np
import jax
import psutil

def print_mem_usage() -> None:
    print(
        f"Resident memory: {psutil.Process().memory_info().rss // (1024 * 1024)} MB"
    )

def print_live_buffers() -> None:
    lbs = jax.lib.xla_bridge.get_backend().live_buffers()
    print(f"Number live buffers: {len(lbs)}")
    for idx, lb in enumerate(lbs):
        print(f"\tlbs[{idx}]: shape {lb.shape} at 0x{lb.unsafe_buffer_pointer():x}")

def print_num_live_executables() -> None:
    les = jax.lib.xla_bridge.get_backend().live_executables()
    print(f"Number live executables: {len(les)}")

def leak():
    # ~ 1 GB of float32s
    size = 1024 * 1024 * 1024 // 4
    data = jax.device_put(np.empty((size, ), dtype=np.float32) + 1)

    def g():
        return jax.lax.cond(
            True,
            lambda: data[0],
            lambda: data[1],
        )

    jg = jax.jit(g)

    _ = jg().block_until_ready()

    del g, jg, data, _

def run() -> None:
    for iter in range(5):
        print(f"iter: {iter} ================================================================")
        print_mem_usage()
        _ = leak()
        print_mem_usage()
        print_live_buffers()
        print_num_live_executables()

if __name__ == "__main__":
    run()

Output
$ python leak3.py
iter: 0 ================================================================
Resident memory: 116 MB
Resident memory: 1168 MB
Number live buffers: 1
	lbs[0]: shape (268435456,) at 0x293698000
Number live executables: 0
iter: 1 ================================================================
Resident memory: 1168 MB
Resident memory: 2192 MB
Number live buffers: 2
	lbs[0]: shape (268435456,) at 0x2d3698000
	lbs[1]: shape (268435456,) at 0x293698000
Number live executables: 0
iter: 2 ================================================================
Resident memory: 2192 MB
Resident memory: 3216 MB
Number live buffers: 3
	lbs[0]: shape (268435456,) at 0x313698000
	lbs[1]: shape (268435456,) at 0x2d3698000
	lbs[2]: shape (268435456,) at 0x293698000
Number live executables: 0
iter: 3 ================================================================
Resident memory: 3216 MB
Resident memory: 4242 MB
Number live buffers: 4
	lbs[0]: shape (268435456,) at 0x353698000
	lbs[1]: shape (268435456,) at 0x313698000
	lbs[2]: shape (268435456,) at 0x2d3698000
	lbs[3]: shape (268435456,) at 0x293698000
Number live executables: 0
iter: 4 ================================================================
Resident memory: 4242 MB
Resident memory: 5266 MB
Number live buffers: 5
	lbs[0]: shape (268435456,) at 0x393698000
	lbs[1]: shape (268435456,) at 0x353698000
	lbs[2]: shape (268435456,) at 0x313698000
	lbs[3]: shape (268435456,) at 0x2d3698000
	lbs[4]: shape (268435456,) at 0x293698000
Number live executables: 0

What jax/jaxlib version are you using?
jax v0.3.21, jaxlib v0.3.20
Which accelerator(s) are you using?
CPU
Additional system info
Python 3.10.5 on a Mac with M1Pro/Ultimate (MacOS 12.6)

October 10 2022! Oh man...
If we comment out this @cache, the leak goes away.
And looking at the code, that cache is clearly holding strong references to the callables! We need it to be more like weakref_lru_cache. But the first arg is a list here...


Bump CAPI+providers

Find which providers have new releases
Build prime images and fork releases for them
Update https://github.com/rancher/turtles/blob/main/internal/controllers/clusterctl/config.yaml

The following CAPI providers have releases newer than the ones we have in our forks:

[ ] CAPZ: v0.16.1 -> v0.17.0
[ ] CAPV: v1.11.0 -> v1.11.2

A new PR is up to add automated bump of versions in Turtles when new releases are published. I'd like to wait for it to be merged to release CAPI+providers to use it as a real life test.


Update ReadMe

I split off the ReadMe.md changes from PR #313. I added a few fixes that I noticed.
Changes look good. Having example code in nice.
The last section on map tiles seems to just show function prototypes rather than example code.

More long term, it would be nice to have compiler checked code examples. Maybe we can pull examples from the unit tests. If there is insufficient good example code, perhaps we can add to the unit tests.

Thanks for separating this out. I hadn't finished the map section.
-Brett

I think I'm complete with the readme update. There are basic code examples for everything but the stream and bitmap libraries. I feel like the bitmap library really isn't mature enough to document. The stream library documentation might be nice, but I didn't feel up to writing it.
I'd like to merge this and canvas the forums to see if anyone would read it and provide feedback (perhaps techcor) before continuing to work on it.
I pasted all the code examples into Visual Studio and fixed any errors that cropped up when compiling. Hopefully this will keep the quality of the examples up. I wouldn't mind in the future trying to use the unit tests for documentation or something else, but I don't think we are there yet on unit test maturity.
-Brett


any example and document(english)?

thanks

Thank you for your issue.
Sorry that I can't live up to your expectations.
I cannot write them for a while, because I don't have enough time.
If I feel like myself later, I will write. so, this issue will not close.
However, code below might be helpful.

example/probability_inference/main.cpp
bayesian/test

thanks,

@godai0519 Can you please add an example.csv file to the example?
This example is hard to understand when you don't know how the data (CSV file) looks like.
It would be awesome to just have a simple CSV file with 10 lines in it, doesn't have to be complicated. That would ease up understanding the usage much much more.

Sorry, that example is old.
Please use the following instead.
bn::sampler sampler;
sampler.set_filename("samples.csv");
sampler.load_sample(graph.vertex_list());
sampler.make_cpt(graph);

A example of CSV which has 10 nodes is below:
14816 2 1 1 0 2 2 1 1 1 1
41 2 0 0 0 1 2 1 0 2 1
39139 2 2 2 1 2 2 1 1 1 1
1737 2 0 1 0 1 2 1 1 1 2
14601 2 1 2 0 2 2 2 1 1 1
6260 2 0 1 0 1 2 2 1 1 1
60475 0 0 1 0 2 2 1 1 1 1

The first line of the example indicates that there are 14816 samples with X1=2, X2=1, ..., X10=1, where X1,...,X10 are random variables (node) in graph.
Please feel free to reply if there is anything that you don't understand.


Add Sept.15 Steering Group shareable

PR shares latest Steering Group meeting: executive summary, action items, minutes

thanks!


the "pageSizeOptions" inAPI should be changed to "pageSizeSelectorValues"

Bug Report or Feature Request

The parmas "pageSizeOptions" seems is not exist , when I try to use it to change the pageSize Options , the browser gave me a error , but "pageSizeSelectorValues" works !!!
What is expected?

use "pageSizeOptions" to change the pageSize Options
What is actually happening?

the browser hints mistake

ng-alain version: 0.7.1
Angular version: 5.0.0
Browser:
- Chrome (desktop) version 58.0.3029.110

ng-alain.com currently at version 1.0.0-beta, i will publish 0.8.x version document.


Made some settings' comments clearer

Describe your PR, what does it fix/add?
Fixes some possible clarity issues in the comments in the example config file
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Is it ready for merging, or does it need work?
Ready for merging

maybe we should default sens to 1?

Seems like a good idea. I exclusively change the DPI on my mouse instead of sensitivity.


Problem with Sqlite-net 1.7.335

Code:
var connection = new SQLiteConnection(@"C:\Path with space\Folder\Database.db");

Throw:
System.TypeInitializationException: 'The type initializer for 'SQLite.SQLiteConnection' threw an exception.'
ArgumentException: The path is not of a legal form.

此例外狀況原先在此呼叫堆疊擲回:
    System.IO.Path.NormalizePath(string, bool, int, bool)
    System.IO.Path.InternalGetDirectoryName(string)
    SQLitePCL.NativeLibrary.MakePossibilitiesFor(string, System.Reflection.Assembly, int, SQLitePCL.NativeLibrary.LibSuffix)
    SQLitePCL.NativeLibrary.MyLoad(string, System.Reflection.Assembly, int, System.Action<string>)
    SQLitePCL.NativeLibrary.Load(string, System.Reflection.Assembly, int)
    SQLitePCL.Batteries_V2.MakeDynamic(string, int)
    SQLitePCL.Batteries_V2.DoDynamic_cdecl(string, int)
    SQLite.SQLiteConnection.SQLiteConnection()

Sqlite-pcl v1.7.335
SQLitePCLRaw.bundle_green v2.0.3

Try with this:
var connection = new SQLiteConnection(@"C:\Path with space\Folder\Database.db");

By bad,
actually, my code is
var connection = new SQLiteConnection(dbPath);

and dbPath = @"C:\Path with space\Folder\Database.db"
i did't copy it from Visual Studio but typing with keyboard

Ok, have you try with this?
var connection = new SQLiteConnection($"DataSource={dbPath}");

not work

Are you suggesting that the space in the path is the problem?
Does your code work if you change the path to something that does not include a space?


Display language icons in script create dialog

As languages match respective editor icon names, those are set by traversing each language registered.

Does it work for C# scripts?

I don't use C# to test this, but if it fails it would display an empty space for C# I guess. Can't seem to find any mono/C# icons either.

@akien-mga I think we need to create modules/mono/icons/icon_c#_script.svg. Looking at the other language modules suggests everything else is automatic.
If I knew what font the NativeScript and VisualScript icons use I could try to make one for C#. In the mean time I don't know what would happen if the icons is not found.

Made it more fail-proof by checking if language name matches with the icon. If not, it won't add an empty icon space if the icon is not found per language.
So for C#, you'd have to make sure that the name reported by ScriptServer::get_language(i)->get_name() matches the editor icon name in that case.

@neikeq I tried to reverse match the font from the NativeScript and VisualScript icons, but I couldn't find a match. I think @djrm drew them specifically for those icons (and other icons with text like flt, vec2, etc.).
The "c" could be copied from some of the existing icons (e.g. icon_color or icon_key_position). For the "#" we'd have to draw it, unless we use "cs", in which case "s" is already available in the VisualScript icon.

"cs" is fine. I will try to make one. This PR as is is fine now with the new checks.

Thanks!


Clean up Visual Studio Project Files

Just cosmetic changes.

Codecov Report

Merging #1418 into master will increase coverage by 0.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1418      +/-   ##
==========================================
+ Coverage   69.54%   70.06%   +0.52%
==========================================
  Files         185      184       -1
  Lines       26996    26126     -870
  Branches     3375     3318      -57
==========================================
- Hits        18774    18306     -468
+ Misses       6792     6434     -358
+ Partials     1430     1386      -44

Impacted Files
Coverage Δ

examples/platforms/posix/misc.c
0% <0%> (-66.67%)
:x:

src/core/coap/coap_client.cpp
50% <0%> (-16.3%)
:x:

src/core/coap/coap_client.hpp
57.89% <0%> (-15.79%)
:x:

src/ncp/ncp_base.cpp
24.8% <0%> (-13.4%)
:x:

examples/platforms/posix/uart-posix.c
59.55% <0%> (-3.38%)
:x:

src/core/api/thread_api.cpp
45.45% <0%> (-1.74%)
:x:

src/core/common/message.cpp
93.99% <0%> (-1.66%)
:x:

src/core/net/ip6_mpl.cpp
92.36% <0%> (-1.53%)
:x:

src/core/net/ip6.cpp
68.78% <0%> (-0.98%)
:x:

src/core/thread/network_data_leader_ftd.cpp
67.99% <0%> (-0.71%)
:x:

... and 12 more

Continue to review full report at Codecov.

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


small multiples are not longer accessible on mobile

Actual Behavior:

When I take a look at v1.8
https://cmu-delphi-covidcast.netlify.app/?sensor=doctor-visits-smoothed_adj_cli&level=county&date=20200926&signalType=value&encoding=color&mode=overview&region=42003
I was able to toggle between map and line charts in the overview:

This is not longer possible in v1.9

was this intentional?

This was not intentional.  Thanks for catching this.


push would change hash while browserHistory is being used

This behavior only occurs with push action from react-router-redux. If I call react-router's push directly, the URL changes as expected.
VERSIONS:
react-router-redux: 4.0.5
react-router: 2.4.1
history: 3.0.0

You must be passing the wrong history instance into react-router-redux. Maybe sure you're applying the right instance with the middleware.


Add EPLSE

GPLSE stands for Generalized PLSE?

To do this, you may need to think about #13 first...!

I think this will be done in an another repo

Renamed as Extended PLSE (EPLSE)


langchain_google_genai : Expose safety_settings for Gemini integrations on google-generativeai

Replace this entire comment with:

**Description:Expose safety_settings for Gemini integrations on google-generativeai
**Issue:NA,
**Dependencies:NA
**Twitter handle:@aditya_rane

@lkuligin for review

@efriis please, take a look when you have time.


New version: TupleTools v1.2.0

Registering package: TupleTools
Repository: https://github.com/Jutho/TupleTools.jl
Created by: @Jutho
Version: v1.2.0
Commit: ad34513fb3644f6e58741358252e457461dba113
Reviewed by: @Jutho
Reference: https://github.com/Jutho/TupleTools.jl/commit/ad34513fb3644f6e58741358252e457461dba113#commitcomment-35615651

Your new version pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request during the next cron job.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

[noblock]


RFC- Non-consensus nodes

Thanks to @vanishreerao and @nathan for helping with the finality, @cmr with the libp2p stuff and @imeckler  for the staged-ledger-validity solution.
This doesn't cover the entire design needed for running it on a  browser like the networking stuff and things needed for js_of_ocaml to compile it to javascript

[ ] Tests were added for the new behavior
[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

Updated design and list of requirements here https://github.com/CodaProtocol/coda/issues/3762#issuecomment-558303802

Closing this for now. Needs the libp2p and scan state related details flushed out.


Fix typos in client endpoint, projectId & key opts

What does this PR do?
As per subject, fixes spelling error (of Appwrite) in the --endpoint, --projectId and --key options of the client in generic.js.
Test Plan
N/A
Related PRs and Issues
N/A
Have you read the Contributing Guidelines on issues?
Yes

@jocubeit, thanks for your PR! 🙏 Unfortunately, this repo is read-only since we generate the code using our SDK Generator. Please feel free to close this PR and create one in that repo.

Recreated at https://github.com/appwrite/sdk-generator/pull/513.


created the manual setup guide doc

Pull Request: Added Manual Setup Guide for Next.js Project with Trigger.dev
Description
This pull request adds comprehensive documentation for setting up a Next.js project with Trigger.dev. The guide provides step-by-step instructions, code snippets, and explanations for each stage of the setup process. It covers configuring the Trigger Client, defining jobs, creating API routes, setting environment variables, running the Next.js site, and using the CLI dev command. The documentation is provided in a clear and organized manner to assist developers in seamlessly integrating Trigger.dev into their projects.
Changes

Created a new markdown file manual-setup-guide.md in the docs directory.
Added detailed instructions for each step of the setup process, along with relevant code snippets.
Included explanations and optional advanced setup steps for running Next.js server with CLI.
Added title and description metadata to the document for clarity and organization.

Closes #277

/claim #277

@Chigala i can see two missing pieces

mint.json file update with the new added mdx file
adding configuration to package.json file

@Chigala went with @yashug with this one but gave you a tip for this PR. Thanks!

Thanks a lot


[Graphite.SQL] Add stored procedure to send metric for series of measures

Thanks for the pull request.
I added some small fixes and merged it into the master branch.

hello, is it possible to generate and publish the nugget for Graphite.SQL ?


Very long blank visualization card for csv files

This is for general csv visualization. If you load a big csv file which doesn't show a chart with the default parameters, it shows a black very long visualization card (it has a scroll bar on the page which you can scroll for long). Once you set the correct parameters, it's set to a proper card. I have attached the file I tried with .

frequent_trigrams.csv

Is there a csv of similar size that works? It might be helpful in testing. I had some but most of my csvs are too large.

I'm noticing that I get very inconsistent behavior here.
I ran the populate fake data script. So this csv file works when I see it in the fake data dataset.
final.csv

but when I download that file and upload it back to one of my datasets, now I don't see a visualization. Same bug as reported for the larger file.


Missing keycode 172

The keycode 172 is the pipe | on Norwegian keyboard layouts, but I am unsure what it is on American keyboards. It is the key which is left of the 1 key (it is sometimes used to toggle the console in games).

On my keyboard it's "Home page" shortcut, it opens the browser or goes to home page on current tab.
Key: Fn+F9
Browser: Chrome 49
OS: Win 8.1
Keyboard: X2 Mirage (US layout)
Input layout: Polish (programmers)


cpython pathlib tests

This adds the current cpython pathlib tests to universal_pathlib.
Tests are taken from this point in the commit history: https://github.com/python/cpython/blob/10bf2cd404320252ef162d5699cb7ce52a970d44/Lib/test/test_pathlib.py
The pathlib tests are by default deselected via the pathlib marker (see pyproject.toml).
The hdfs skip logic has been removed too in favour of simply deselecting the hdfs marker.
The --skiphdfs commandline options has been removed too. Use -m 'not hdfs' instead.
To enable pathlib tests run pytest -m pathlib upath. Default is -m 'not (hdfs or pathlib)'.

Waiting for #105


DCOS-22453: prevent unnecessary updates in ServiceBreadcrumb

fix: prevent service breadcrumb to update without changes
This adds a shouldComponentUpdate function to the ServiceBreadcrumb component. This ensures the
component is only updated if any of the props has changed.
To test you can use the following service config:
{
  "id": "/volumes",
  "containers": [
    {
      "name": "container-1",
      "resources": {
        "cpus": 0.1,
        "mem": 128,
        "disk": 0
      },
      "image": {
        "kind": "DOCKER",
        "id": "alpine"
      },
      "exec": {
        "command": {
          "shell": "sleep 1000;"
        }
      },
      "volumeMounts": [
        {
          "name": "home",
          "mountPath": "/var/home"
        },
        {
          "name": "nothome",
          "mountPath": "/var/nothome"
        }
      ]
    }
  ],
  "volumes": [
    {
      "name": "home",
      "persistent": {
        "type": "root",
        "size": 1,
        "constraints": []
      }
    },
    {
      "name": "nothome",
      "persistent": {
        "type": "root",
        "size": 1,
        "constraints": []
      }
    }
  ],
  "networks": [
    {
      "mode": "host"
    }
  ],
  "scaling": {
    "instances": 1,
    "kind": "fixed"
  },
  "scheduling": {
    "placement": {
      "constraints": []
    }
  },
  "executorResources": {
    "cpus": 0.1,
    "mem": 32,
    "disk": 10
  },
  "fetch": []
}

This is going to create a multi container application with multiple volumes. Then you should
navigate to the service detail page.
Check if you can navigate to the task detail page and back.
Check if you can visit the volume detail page.
Check if you can switch between the two volume detail pages.

Copy the URL of the first volume Detail page,
Visit the second one, go to the address bar paste the URL and hit enter.
After this you may use the browser history to go back and forth.

Close DCOS-22453

Commit:
6922ed07c4a4a5044263227ace2bf3673b2cb25c
Testwise from https://github.com/dcos/dcos-ui/pull/2962/files#diff-90b25207bf60e5708a7ba1ffdec09510R86 to the bottom is interesting this is new code the rest is moved. I created a separate describe block since the old one had a before each So I thought it made sense to create a separate block.
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[x] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

@user please open a PR to back-port the adjustment to DC/OS UI 1.10 and 1.11

Yes @orlandohohmeier I have this on my plate I just wanted to get the change once merged.


ClangD shows clang-tidy messages from inside of a macro provided by a system header at the macro use site

When using the clang-tidy check modernize-use-trailing-return-type and GoogleTest, I get errors that I would expect to be suppressed due to originating in macros from system headers:
// The next line (with `TEST` underscored) triggers the clang-tidy check.
TEST(SomeTest, SomeMethod) {
  // ...
}

The fact that the macro expands to a non-trailing-return-type declaration shouldn't be reported here as it is in a macro expansion originating in a system header.
The most confusing thing is that running clang-tidy directly doesn't show this error. So it is something specific about clangd that leads to this.
Logs
clangd-log-snippet.txt
System information
Output of clangd --version:
Homebrew clangd version 15.0.6
Features: linux
Platform: x86_64-unknown-linux-gnu

Editor/LSP plugin:
VSCode + ClangD
Operating system:
Mac, but with SSH-plugin of VSCode to a Linux server where ClangD is running.

Could you run the "expand macro" code action on the affected invocation, share the expansion, and (assuming you get the diagnostic in the expansion too) indicate which line of the expansion it occurs on?
I ask because when I expand this macro in a simple test file, the only function declarations/definitions in the expansion have void return type, and modernize-use-trailing-return-type does not warn on those (i.e. I haven't been able to reproduce the diagnostic you're seeing yet).

For example:
TEST(IndirectValueTest, ConstAccess) {
  const IndirectValue<int> v = 42;
  EXPECT_EQ(*v, 42);
  EXPECT_EQ(v.GetPointer(), &*v);
}

In the code here:
https://github.com/carbon-language/carbon-lang/blob/trunk/common/indirect_value_test.cpp#L14,L18
Exhibits this on line 14.
And when I expand the macro:
static_assert(sizeof("IndirectValueTest") > 1,
              "test_suite_name must not be empty");
static_assert(sizeof("ConstAccess") > 1, "test_name must not be empty");
class IndirectValueTest_ConstAccess_Test : public ::testing ::Test {
 public:
  IndirectValueTest_ConstAccess_Test() = default;
  ~IndirectValueTest_ConstAccess_Test() override = default;
  IndirectValueTest_ConstAccess_Test(
      const IndirectValueTest_ConstAccess_Test&) = delete;
  IndirectValueTest_ConstAccess_Test& operator=(
      const IndirectValueTest_ConstAccess_Test&) = delete;
  IndirectValueTest_ConstAccess_Test(
      IndirectValueTest_ConstAccess_Test&&) noexcept = delete;
  IndirectValueTest_ConstAccess_Test& operator=(
      IndirectValueTest_ConstAccess_Test&&) noexcept = delete;

 private:
  void TestBody() override;
  static ::testing ::TestInfo* const test_info_ __attribute__((unused));
};
::testing ::TestInfo* const IndirectValueTest_ConstAccess_Test ::test_info_ =
    ::testing ::internal ::MakeAndRegisterTestInfo(
        "IndirectValueTest", "ConstAccess", nullptr, nullptr,
        ::testing ::internal ::CodeLocation(
            "/usr/local/google/home/chandlerc/src/carbon/carbon-lang/common/"
            "indirect_value_test.cpp",
            14),
        (::testing ::internal ::GetTestTypeId()),
        ::testing ::internal ::SuiteApiResolver<::testing ::Test>::
            GetSetUpCaseOrSuite("/usr/local/google/home/chandlerc/src/carbon/"
                                "carbon-lang/common/indirect_value_test.cpp",
                                14),
        ::testing ::internal ::SuiteApiResolver<::testing ::Test>::
            GetTearDownCaseOrSuite(
                "/usr/local/google/home/chandlerc/src/carbon/carbon-lang/"
                "common/indirect_value_test.cpp",
                14),
        new ::testing ::internal ::TestFactoryImpl<
            IndirectValueTest_ConstAccess_Test>);
void IndirectValueTest_ConstAccess_Test ::TestBody() {
  const IndirectValue<int> v = 42;
  EXPECT_EQ(*v, 42);
  EXPECT_EQ(v.GetPointer(), &*v);
}

The clang-tidy check fires on these two methods:
  IndirectValueTest_ConstAccess_Test& operator=(
      const IndirectValueTest_ConstAccess_Test&) = delete;

  IndirectValueTest_ConstAccess_Test& operator=(
      IndirectValueTest_ConstAccess_Test&&) noexcept = delete;

Weirdly, there are other enabled clang-tidy checks that fire on this code, but only the one shows up in vscode on the original TEST macro.

For example:
TEST(IndirectValueTest, ConstAccess) {
  const IndirectValue<int> v = 42;
  EXPECT_EQ(*v, 42);
  EXPECT_EQ(v.GetPointer(), &*v);
}

In the code here:
https://github.com/carbon-language/carbon-lang/blob/trunk/common/indirect_value_test.cpp#L14,L18
Exhibits this on line 14.
And when I expand the macro:
static_assert(sizeof("IndirectValueTest") > 1,
              "test_suite_name must not be empty");
static_assert(sizeof("ConstAccess") > 1, "test_name must not be empty");
class IndirectValueTest_ConstAccess_Test : public ::testing ::Test {
 public:
  IndirectValueTest_ConstAccess_Test() = default;
  ~IndirectValueTest_ConstAccess_Test() override = default;
  IndirectValueTest_ConstAccess_Test(
      const IndirectValueTest_ConstAccess_Test&) = delete;
  IndirectValueTest_ConstAccess_Test& operator=(
      const IndirectValueTest_ConstAccess_Test&) = delete;
  IndirectValueTest_ConstAccess_Test(
      IndirectValueTest_ConstAccess_Test&&) noexcept = delete;
  IndirectValueTest_ConstAccess_Test& operator=(
      IndirectValueTest_ConstAccess_Test&&) noexcept = delete;

 private:
  void TestBody() override;
  static ::testing ::TestInfo* const test_info_ __attribute__((unused));
};
::testing ::TestInfo* const IndirectValueTest_ConstAccess_Test ::test_info_ =
    ::testing ::internal ::MakeAndRegisterTestInfo(
        "IndirectValueTest", "ConstAccess", nullptr, nullptr,
        ::testing ::internal ::CodeLocation(
            "/usr/local/google/home/chandlerc/src/carbon/carbon-lang/common/"
            "indirect_value_test.cpp",
            14),
        (::testing ::internal ::GetTestTypeId()),
        ::testing ::internal ::SuiteApiResolver<::testing ::Test>::
            GetSetUpCaseOrSuite("/usr/local/google/home/chandlerc/src/carbon/"
                                "carbon-lang/common/indirect_value_test.cpp",
                                14),
        ::testing ::internal ::SuiteApiResolver<::testing ::Test>::
            GetTearDownCaseOrSuite(
                "/usr/local/google/home/chandlerc/src/carbon/carbon-lang/"
                "common/indirect_value_test.cpp",
                14),
        new ::testing ::internal ::TestFactoryImpl<
            IndirectValueTest_ConstAccess_Test>);
void IndirectValueTest_ConstAccess_Test ::TestBody() {
  const IndirectValue<int> v = 42;
  EXPECT_EQ(*v, 42);
  EXPECT_EQ(v.GetPointer(), &*v);
}

The clang-tidy check fires on these two methods:
  IndirectValueTest_ConstAccess_Test& operator=(
      const IndirectValueTest_ConstAccess_Test&) = delete;

  IndirectValueTest_ConstAccess_Test& operator=(
      IndirectValueTest_ConstAccess_Test&&) noexcept = delete;

Weirdly, there are other enabled clang-tidy checks that fire on this code, but only the one shows up in vscode on the original TEST macro.

Ok, it seems the reason I couldn't reproduce the error is that I was using an older version of gtest where the return type of the generated class's assignment operator was void.
Upgrading to trunk gtest, I do see the same diagnostic as you.
However, I see the same error with a command-line clang-tidy invocation as well!
Looking at the relevant part of the gtest source, they have /* NOLINT */ comments after the assignment operator declarations, but they're not effective because the declaration is spread out over multiple lines, and the /* NOLINT */ comment is on the last line but the diagnostic is on the first line.
Moving the /* NOLINT */ comment to the first line of the declaration in the gtest source successfully suppresses the diagnostic for me with both clang-tidy and clangd.
As for why you don't get the diagnostic with a command-line clang-tidy invocation, I don't have an explanation for that. (But note, I haven't set up the carbon codebase locally, just a minimal workspace that includes the gtest headers.)

Looking at the relevant part of the gtest source, they have /* NOLINT */ comments after the assignment operator declarations, but they're not effective because the declaration is spread out over multiple lines, and the /* NOLINT */ comment is on the last line but the diagnostic is on the first line.
Moving the /* NOLINT */ comment to the first line of the declaration in the gtest source successfully suppresses the diagnostic for me with both clang-tidy and clangd.

Filed https://github.com/google/googletest/issues/4110 for this.

Filed google/googletest#4110 for this.

I wrote a patch to fix it, too, but that seems to require a Google CLA, which in turn requires a Google account. Sigh.

Filed google/googletest#4110 for this.

In any case, this was closed as wontfix with the comment "This is not a style we're willing to adopt, nor do we believe we it should require NOLINT annotation." :shrug:
@chandlerc I don't think there's anything further to do here, unless you'd like to delve further into why clang-tidy didn't produce the diagnostic for you (since it does for me), which would require some additional details (e.g. your clang-tidy version, relevant config files, etc.)

I'm sympathetic to the desire to not have to mark things in this way.
I think the thing to do would be to disable this clang-tidy check from within macros, or at least within macros defined within a system header. Should I move that to a separate issue?

Disabling within macros: there's no way to make everyone happy here, diverging from clang-tidy's position here seems like a bad idea.
Disabling within system macros: Yes, I think we should do this. clang-tidy has a flag for this that defaults to off, but it's implemented in the diagnostic consumer which we don't use in clangd.
I think this bug is fine, the title reflects the issue well in the end.

reviews.llvm.org/D141495

From that review, it may also explain why my clang-tidy runs weren't showing these -- I suspect it is successfully mapping the headers to system headers, and may not be for the reduced test you used @HighCommander4 -- anyways, super delighted to see a nice solution that aligns well across the tools here! Thanks so much!


UI gets squished for Output views to external addresses

It'd be great to fix this in the next FE deployment.


Update Replace placeholder icon #36

I update placeholder for website
if any problem can u inform me i will immediate update

can you verfify its my first contribution in the git hub can you help me

I cannot, but you can see it on your own GitHub profile 😃

ok how i will resolve

The image 512x512 uses the incorrect image. It should be the one with the round white background :)

And you are missing the 384x384 image 😄

And you are missing the 384x384 image 😄

for this image what would to size

And you are missing the 384x384 image 😄

for this image what would to size

it should be 384x384. As mentioned here:
https://github.com/flowcore-io/open-source-conference-app/issues/36#issuecomment-1964442565

can u check new pull request


My question

I’m trying to figure out how to run NIM packages ( https://github.com/thisago/subscribestar ) and PowelShell packages ( https://github.com/aelrou/Filenamer ) and Ruby packages ( https://github.com/hernanrz/subscribestar_ruby ) and PHP packages ( https://github.com/SocialiteProviders/Subscribestar ) and IR ( https://github.com/Alessandro-Gobbetti/IR ) on TryItOnline. Can you please upload Video Tutorials on how to do so?

You cannot, it is sandboxed so you have no network access to download them. It's a security concern


Feature/update organic fract option

[X] closes #836
[ ] tests passed
[ ] new tests added
[ ] science test figures
[X] ran uncrustify prior to final commit
[x] ReleaseNotes entry

This PR enables the ORGANIC_FRACT option to be turned on in VIC 5 (it was tested as part of this PR) and also enables the BULK_DENSITY_COMB option to be used in conjunction with it.

I just made an issue for doing some follow-up analysis that will look at the impact on soil temperatures of turning on the ORGANIC_FRACT option. See #844 for reference

@dgergel : If you resolve the conflict then I can merge

@bartnijssen resolved the conflict earlier, this should be ready to merge


There is an incorrect plural form for French.

Hi!
I found out that the library returns 3 plural forms for the French language - https://php-gettext.github.io/Languages/ (http://i.imgur.com/8sjFFM6.png). This is a screenshot from my debug code - http://i.imgur.com/L4GEBXs.png
But other sources say that there are only two plural forms:

http://docs.translatehouse.org/projects/localization-guide/en/latest/l10n/pluralforms.html?id=l10n/pluralforms (http://i.imgur.com/Kq8OvXp.png)
https://www.gnu.org/software/gettext/manual/html_node/Plural-forms.html (http://i.imgur.com/lV6zUnT.png)
Poedit - http://i.imgur.com/3Lp0SoB.png , http://i.imgur.com/rsMoaX9.png

Could you please help me with this?
Thanks!

That's correct: the third plural form for French has been introduced in the Unicode CLDR data (it's the source of the data of php-gettext/Languages): see https://cldr.unicode.org/index/downloads/cldr-38#h.7ahoxbsngtxp
This third plural category is used to translate the multiple of millions.
For example

English
French

1 view
1 vue

2 views
1 vues

1.000.000 views
1.000.000 de vues (read as "1 million de vues")

That's not my choice: Unicode people switched to that approach...

@mlocati
Thank you for the explanation!


QuantumCircuit qasm2 dumps generates wrong OpenQASM code

URL to the relevant documentation
No response
Select all that apply

[ ] typo
[X] code bug
[ ] out-of-date content
[ ] broken link
[ ] other

Describe the fix.
When exporting QuantumCircuit to OpenQASM U gate exported as 'u' instead of 'U' this leads to error when importing generated OpenQASM code using qasm2.loads()
Step to reproduce:
Run program:
from qiskit import qasm2
from qiskit import QuantumRegister, QuantumCircuit
q = QuantumRegister(1, "q")
qc = QuantumCircuit(q)
qc.u(0.3, 0.2, 0.1, q[0])
qq =  qasm2.dumps(qc);
print(qq);
circuit = qasm2.loads( qq );

Output:
OPENQASM 2.0;
include "qelib1.inc";
qreg q[1];
u(0.3,0.2,0.1) q[0];
Traceback (most recent call last):
File "C:\work\1\python\tele2.py", line 12, in
circuit = qasm2.loads( qq );
File "C:\work\quantum\qiskit\lib\site-packages\qiskit\qasm2_init_.py", line 604, in loads
return _parse.from_bytecode(
File "C:\work\quantum\qiskit\lib\site-packages\qiskit\qasm2\parse.py", line 214, in from_bytecode
for op in bc:
qiskit.qasm2.exceptions.QASM2ParseError: ":4,0: 'u' is not defined in this scope"
But if replace in generated code last line to "U(0.3,0.2,0.1) q[0];" then code imported correctly

moved to https://github.com/Qiskit/qiskit/issues/11508


ConfirmAsync blocks if dismissed by clicking outside it.

ConfirmAsync blocks indefinitely on android if dialog dismissed by clicking outside it.
This is because the task result is only set if the positive or negative buttons are clicked, if the user dismisses the dialog by clicking outside it then the async call will never complete.

Based on the docs, I am guessing setCancelable(false) or hook up a setOnCancelListener to return the same as the negative button.


[TimeSeries-BarChart] Upon adding multiple metrics, the legend bar disappears.

Superset version
"version": "0.20.3"
Expected results
Upon adding multiple metrics, the legend bar should not get disappeared.
Actual results
Upon adding multiple metrics, the legend bar disappears.
Steps to reproduce
create a slice i.e. Time Series - Bar Chart
add multiple metrics to plot
enable the legends
query to view the data
the legends bar won't be visible

screen shot is attached.

@NaukhaizAhmad you have to scroll to the right end to view the legends .


Naive Bayes Classifier Algorithm in Machine Learning

Closes: #1519
Description
This issue proposes adding the Naive Bayes Classifier algorithm to the repository. The Naive Bayes Classifier is a simple yet powerful probabilistic classifier based on applying Bayes' theorem with strong (naive) independence assumptions between the features. It is handy for classification tasks where the dimensionality of the input is high.
Issue_Reference: #1519

@pankaj-bind the task has been completed. Kindly review the PR


Yield not staying in div or honouring its initial position.

I'm having issues with the yield statement not staying in it's assigned div.
FlowRouter.route('/', {
	action() {
		// Render a template using Blaze
		this.render('Page_body', 'Page_home', {menu: 'menu'});
	}
});

<body class="g-sidenav-show bg-gray-100">
	{{> yield}}
	<!--   Core JS Files   -->
	<script src="/assets/js/plugins/threejs.js"></script>
</body>

<template name="Page_body">
	{{> menu}}
	<main class="main-content border-radius-lg ">
		{{> main}}
		{{> yield }}
	</main>
</template>

Basically, I need to have the  statement at the root because I must have the class statements with it.
Page_body then needs to be added to the html body by the yield.
Page_home then needs to be added to the Page_body yield.
The problem is that on first render, the yield in Page_body stays within the  tag which is what is needed, but as soon as I run a different route, it pops out of the  and gets dumped at the bottom of the  tag. This seems to be the case with any use case of yield not honouring its initial position.

@CyberCyclone this is Blaze issue not honoring template location on re-render; This is known issue, template won't honor its location within sibling elements; Recommended approach to always have wrapping parent element
Hope that helps

My answer looks straightforward. Perhaps it is fixed in the latest Blaze release. I'm working on compatibility release. Feel free to reopen it in case if the issue is still persists on your end.


Add Priority Queue to GraphPriorityQueueSearch and TreePriorityQueueSearch

Goals :-

Priority queue improves readability for graph and tree algorithms as mentioned in the description of the algorithm.
Prevents many calls to function getNodeComparator().compare(child, n) in removedNodeFromFrontierWithSameStateAndLowerPriority().

@ctjoreilly  I have updated #250 .

I am going to close the pull request without merging it into the main project. However, I would like to state it highlighted a usability issue in the existing AbstractQueueSearchForActions hierarchy in terms of expected default behavior. I have corrected this by adding AbstractPriorityQueueSearchForActions and re-organizing the hierarchy appropriately (see latests changes pushed up to the repository). I also, added comments to your code changes to indicate where I saw issues with your proposal. While not accepted, this was still a good submission as it highlighted a flaw in the existing implementation. Let me know what you think of the re-organized hierarchy and if it addresses the concerns you had that motivated this submission.
Thanks
Ciaran

@ctjoreilly
I completely understand that by my implementation, code dependent on class was made, but I wanted to preserve the hierarchy. This led to the class dependence. By re-organizing the hierarchy, my goal of having underlying priority queue for searches TreePriorityQueueSearch, GraphRLPriorityQueueSearch and GraphRLPriorityQueueSearch  is met.
Much appreciated.
Thank you. :smile:


[6.0🍒][Concurrency] Infer nonisolated for a mutable storage of a value type accessed from within the module.

Explanation: As per SE-0434, Stored properties of Sendable type in a global-actor-isolated value type can be declared as nonisolated without using (unsafe), and are treated as nonisolated when used within the module that defines the property.
Scope: An improvement to the concurrency model, guarded behind a feature flag.
Issue: N/A
Original PR: #72473
Risk: Low (the changes are gated behind a feature flag.
Testing: Added a test to the testing suite, and validated the correctness of the test cases where the diagnostics stopped from being emitted.
Reviewer: @hborla

@swift-ci please test

@swift-ci please test


How to break and return a value to client in before hook

I want to return a value in before hook, like:
service.before({
    create (hook) {
        // const serviceName = '...'
        // const query = { ... }
        return new Promise((resolve, reject) => {
            hook.app.service(serviceName).Model.findOne(query, (err, res) => {
                if (err) {
                    return reject(err)
                }

                // if data has existed, return data
                if (res !== null) {
                    hook.result = res
                }

                return resolve(hook)
            })
        })
    }
})

How can I do it?
Thanks

You can read about that here https://docs.feathersjs.com/guides/step-by-step/basic-feathers/writing-hooks.html#returning-a-result

You do it by setting hook.result exactly like in the code example that you posted. What exactly is the question/issue?

@eddyystop Thanks

@daffl Sorry, because of the dependency service's cause, my example code did not working properly. But I did not find it when debugging, now it is normal.
Thank you for your reply.


image_types_ostree: Fix OSTree ref-bindings

This fixes an unforeseen issue caused by https://github.com/advancedtelematic/meta-updater/pull/489.
When ostree commit is used it creates a ref with the passed in OSTREE_BRANCHNAME. The resulting ref file in ostree_repo contains the commit hash pertaining to this ref. This hash can then be used to locate the respective hash.commit file in ostree_repo which contains the metadata for that commit. One such piece of metadata is the ref-binding which is a list of ref names that correspond to this commit.
The ostree refs --create added in the above pull request introduced an issue however. Using ostree refs --create creates a ref file in ostree_repo with the same hash as a preexisting commit. But this doesn't seem to modify the commit metadata at all.
This leads to problems when manipulating this ostree_repo with certain ostree commands. Namely ostree pull will fail when pulling the ref created by ostree refs --create. This is because ostree pull verifies the ref-bindings metadata to see if the ref that is being pulled is listed in the ref-bindings for the commit metadata being referenced by that ref.
In short any refs created by ostree refs --create will fail when pulled by ostree pull since they lack the necessary metadata to pass the basic security verification.
What this commit does is add the --bind-refs option to ostree commit which adds the passed in string to be added in that commits ref-bindings list. Note that this only adds a name to the ref-bindings list, and does not actually create the ref, but here the ref will be created after the commit anyways.

Thanks! I ran this in our CI and it passed (as expected). It looks like this might fix https://github.com/advancedtelematic/meta-updater/issues/583 -- @psaavedra can you confirm?

Thanks! I ran this in our CI and it passed (as expected). It looks like this might fix #583 -- @psaavedra can you confirm?

Confirmed. Thanks @jsrc27 for providing a proper fix for this.


Some fixes regarding "unclear"

Hi Michael,
I just submitted a PR to the critical.xslt file (v.1.0.0). I was having a problem when transforming this file:
https://github.com/scta-texts/wodehamordinatio/blob/master/b1-d3-qun/b1-d3-qun.xml#L15894
I noticed that if an unclear element was present in a lemma (see line 15894 of the xml), something happened that moved a ledsidenote into a Bfootnote, which is not allowed by reledmac. I'm not sure what the problem was, but this "solution" made the trick.
I also changed the priority of the unclear template in order to avoid an ambiguity.
Cheers,
Nick

Thanks for the PR!
Seems good to me. Good work.


Update Glossary.md

Added Column Formatting and View Formatting entries.

Thanks! Keep it coming; the community appreciates it.


Latest babel preset doesn't work with rollup

rollup.config.js
// Rollup plugins
import babel from 'rollup-plugin-babel';
import resolve from 'rollup-plugin-node-resolve';
import commonjs from 'rollup-plugin-commonjs';
import replace from 'rollup-plugin-replace';
import uglify from 'rollup-plugin-uglify';

export default {
    entry: 'src/main.js',
    dest: 'build/js/main.min.js',
    format: 'iife',
    sourceMap: 'inline',
    plugins: [
        resolve({
            jsnext: true,
            main: true,
            browser: true,
        }),
        commonjs(),
        babel({
            plugins: ['external-helpers'],
            externalHelpers: true,
            exclude: 'node_modules/**',

        }),
        replace({
            exclude: 'node_modules/**',
            ENV: JSON.stringify(process.env.NODE_ENV || 'development'),
        }),
        (process.env.NODE_ENV === 'production' && uglify()),
    ],
};

.babelrc
{
  "presets": ["latest"],
  "plugins": [
    "external-helpers"
  ]
}

Then this error pops up
Error transforming src\main.js with 'babel' plugin: It looks like your Babel configuration specifies a module transformer. Please disable it. See https://github.com/rollup/rollup
-plugin-babel#configuring-babel for more information

I'm getting the same thing

Did you read the documentation that the error message pointed you to?

I'm having a related problem. I have followed the documentation. My .babelrc:
{
  "presets": [
    "es2015",
    {
      "modules": false
    }
  ]
}

With babel 6.5.2 and babel-preset-es2015 v6.18.0, I get the following error:
Error: [BABEL] /.../src/x.js: Using removed Babel 5 option: foreign.modules - Use the corresponding module transform plugin in the `plugins` option. Check out http://babeljs.io/docs/plugins/#modules
    at error (/.../node_modules/rollup/dist/rollup.js:170:12)
    at /.../node_modules/rollup/dist/rollup.js:8926:6
    at process._tickDomainCallback (internal/process/next_tick.js:129:7)

In looking at the referenced babel docs, it seems to suggest using a module transformer, but I assume rollup/rollup-plugin-babel should be handling any transformations, which is why the plugin documentation requires modules to be configured to false.
Any ideas what's happening?

@Rich-Harris, I also ran into this issue in a repo of mine which previously worked. I made a minimal repository reproducing the issue: https://github.com/SpaceK33z/rollup-babel-issue
I have not yet found a workaround for this.

I’m experiencing this as well with babel-preset-env.

I had the same problem with the latest preset and solve it with the following .babelrc:
{
  "presets": [
    [
      "latest",
      {
        "es2015": {
          "modules": false
        }
      }
    ]
  ],
  "plugins": [
    "external-helpers"
  ]
}

Note that the modules option is passed down to the es2015 preset.

The above tip with "latest" doesn't work for me - I won't what the difference it?
Another clue: it might have to do with that code is actually passed through the plugin. I'm working with this to Angular4-AOT many small example problems. Some succeed, others fail with the error in this issue.

@kylecordes I installed babel-preset-latest and it worked for me.

I wonder what the difference is, why babel-preset-latest works for you (and for about 75% of my cases) but not for the other 25% of my cases.
(I've actually set it aside for the moment, I got the thing I'm working on up and running with Buble instead. It has a setting to tolerate a subset of for-of loops, and it appears that that subset matches the subset TypeScript outputs.)

I'm having this exact issue with babel-preset-latest.

I realize the following is very unhelpful because most people reading here specifically need Babel, they are using one of its myriad configurable and pluggable features. But:
I just needed something that would compile ES2015 to ES5, for my specific use case. I tried Babel first because it is very popular, feature rich, and high quality. But for the basic job, there are other tools to do fine. In the case I mentioned above I swapped to Buble, you can see it in action in this repo:
https://github.com/OasisDigital/angular-aot-es2015-rollup
In another variant (branch link below) I used Closure Compiler (which grew extensive ES2015->ES5 capability in the last couple years):
https://github.com/OasisDigital/angular-aot-es2015-rollup/tree/clojure-compiler
On the other hand, if someone wants to actually debug and fix this, the code in question is right here:
https://github.com/rollup/rollup-plugin-babel/blob/master/src/preflightCheck.js#L23
I suspect that sticking a console.log right in there to see what exactly the preflight check transformed code looks like at the point of failure, might reveal the nature of the bug.

@kylecordes I'd like to use buble, but I'm having trouble finding documentation on how to integrate buble options with rollup... any advice?  rollup-plugin-buble just says to use buble(), which isn't helpful for my situation.

The lack of options is a "feature" of Buble. It is really suitable only for the most straightforward vanilla ES2015->ES5 cases. It has few options, I needed to set one of them shown below:
https://github.com/OasisDigital/angular-aot-es2015-rollup/blob/master/rollup.config.js#L39
If you are currently setting a bunch of options for Babel, it is unlikely Buble will do what you need. (I say that as a fan of both tools.)

@kylecordes all I know is that buble didn't like my classes until I passed it some options

I was able to solve the above error by referencing a previous issue, #35.
It turns out I needed to add the babelrc: false option to the gulp task. Otherwise the config posted above was conflicting with my gulpfile.babel.js, which was using babel-register.
In this case, in order to use the es2015 preset, you must instead specify es2015-rollup.
Here is my whole config:
import gulp from 'gulp';
import sourcemaps from 'gulp-sourcemaps';
import rollup from 'gulp-better-rollup';
import babel from 'rollup-plugin-babel';

gulp.task('js-rollup', function() {
  return gulp.src('./js/main.js')
    .pipe(sourcemaps.init())
    .pipe(rollup(
      {
        plugins: [babel({
          exclude: 'node_modules/**',
          babelrc: false,
          presets: ['es2015-rollup']
        })]
      },
      {
        format: 'cjs',
      }
    ))
    .pipe(sourcemaps.write())
    .pipe(gulp.dest(built + '/js'));
});

Also @josdejong has posted another example, if you are using an external rollup config: see #35.

@miketrexler Still, what if I do want to use .babelrc?
It feels unnatural to me to relocate a big configuration chunk of my well defined environment agnostic babelrc to some specific place (rollup config) because of some internals I shouldn't have to care about IMHO.
If I understand the problem well, it conflicts with rollup using babel to parse its configuration file. If so, we should file them an issue about this and ask them to provide a facility to allow us customize/bypass rollup transpilation (something like ava does for example).
I'm under the impression that a nice to have feature of rollup is becoming a real pain for common use cases.
What do you think?

@robhicks I think your problem is that you have to wrap the preset and it's option into an extra array:
{
  "presets": [
    ["es2015", { "modules": false }]
  ]
}

I recommend using babel-preset-env, it's really easy to setup and maintain. The preset also offers an option to use modules, so babel configuration should look something like this:
{
  "presets": [
    ["env", {
      "targets": {
        //"node": "current"
        //"browsers": ["last 2 versions"]
      },
      "modules": false
    }]
  ],
}

More info: babel-preset-env

babel-preset-env is not work with rollup, right?
But I need babel-preset-env and rollup......

@musicode: what's not working for you? It works fine for me.

rollup.config.js
  plugins: [
    babel({
      presets: [ 'es2015-rollup' ],
      babelrc: true,
      comments: true,
      runtimeHelpers: true
    }),
    resolve({
      jsnext: true,
      main: true,
      browser: true,
    }),
    commonjs()
  ],

.babelrc
{
  "presets": [
    ["env", {
      "targets": {
        "ie": 8
      }
    }]
  ]
}

result
(babel plugin) It looks like your Babel configuration specifies a module transformer. Please disable it. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information

@musicode you shouldn't transform ES6 modules syntax.
 "presets": [
    ["env", {
      "targets": {
          "ie": 8
      },
      "modules": false
    }]
  ],

Also I don't think you should use this presets: [ 'es2015-rollup' ], since you are already using babel-preset-env.

@prichodko
Thank you for the suggestion.
I remove babel plugin in rollup.config.js, and use your env preset config, the result is ES6 code in one file.
But I need ES5 code, how can I do that?

No you shouldn't remove babel plugin, just the babel-preset-es2015-rollup.

@prichodko
rollup.config.js
import babel from 'rollup-plugin-babel'
import resolve from 'rollup-plugin-node-resolve'
import commonjs from 'rollup-plugin-commonjs'

// plugins
  plugins: [
    babel({
      comments: true,
      runtimeHelpers: true
    }),
    resolve({
      jsnext: true,
      main: true,
      browser: true,
    }),
    commonjs()
  ]

.babelrc
{
  "presets": [
    [
      "env",
      {
        "targets": {
          "ie": 8
        },
        "modules": false
      }
    ]
  ]
}

result
(babel plugin) It looks like your Babel configuration specifies a module transformer. Please disable it. See https://github.com/rollup/rollup-plugin-babel#configuring-babel for more information

why?

@musicode I've noticed one more thing, you usually want to put the babel-plugin last. See https://github.com/rollup/rollup/issues/1148. Let me know if it helps.

@prichodko
change the order like this:
  plugins: [
    resolve({
      jsnext: true,
      main: true,
      browser: true,
    }),
    commonjs(),
    babel({
      comments: true,
      runtimeHelpers: true
    })
  ]

result is the same.

It's a so terrible experience.
Why a simple feature is so difficult to configure.

@musicode Is your repository public? I would have a look.

@prichodko link

To make it complete, for anyone working with babel-preset-env, config file I sent above works correctly. The problem @musicode had, didn't have anything to do with babel-preset-env.

What exactly does this piece of code do now?
{
  "presets": [
    ["es2015", { "modules": false }]
  ]
}

I am kind of confused.

@stephan-v transpiles everything from the es2015 spec, except ES6 modules syntax, because it's handled by a rollup plugin.

I got this to work:
import globals from 'rollup-plugin-node-globals';
import builtins from 'rollup-plugin-node-builtins';
import commonjs from 'rollup-plugin-commonjs';
import nodeResolve from 'rollup-plugin-node-resolve';
import babel from 'rollup-plugin-babel';
import replace from 'rollup-plugin-replace';

export default {
    input: 'build/restful.fetch.js',
    output: {
        file: 'dist/restful.js',
        format: 'umd',
        name: 'restful.js'
    },
    moduleContext: {
        [require.resolve('whatwg-fetch')]: 'window'
    },
    plugins: [
        globals(),
        builtins(),
        nodeResolve({
            // use "module" field for ES6 module if possible
            module: false, // Default: true

            // use "jsnext:main" if possible
            // – see https://github.com/rollup/rollup/wiki/jsnext:main
            jsnext: false,  // Default: false

            // use "main" field or index.js, even if it's not an ES6 module
            // (needs to be converted from CommonJS to ES6
            // – see https://github.com/rollup/rollup-plugin-commonjs
            main: true,  // Default: true

            // some package.json files have a `browser` field which
            // specifies alternative files to load for people bundling
            // for the browser. If that's you, use this option, otherwise
            // pkg.browser will be ignored
            browser: true,  // Default: false

            // not all files you want to resolve are .js files
            // extensions: [ '.js', '.json' ],  // Default: ['.js']

            // whether to prefer built-in modules (e.g. `fs`, `path`) or
            // local ones with the same names
            preferBuiltins: true,  // Default: true

            // If true, inspect resolved files to check that they are
            // ES2015 modules
            modulesOnly: false  // Default: false
        }),
        commonjs({
            namedExports: {
                'node_modules/immutable/dist/immutable.js': [ 'fromJS', 'List', 'Map', 'Iterable' ],
            }
        }),
        babel({
            exclude: 'node_modules/**',
            babelrc: false,
            presets: [
                ["latest", { "modules": false }]
            ],
            plugins: [
                'external-helpers'
            ],
        }),
        replace({
            exclude: 'node_modules/**',
            ENV: 'production',
        })
    ]
};

I'm using the following config fairly similar to @DevEclipse :
getting err [!] Error: Cannot find module 'babel-core'
import babel from 'rollup-plugin-babel';
import resolve from 'rollup-plugin-node-resolve';
import uglify from 'rollup-plugin-uglify';

const pkg = require('./package.json');
const external = Object.keys(pkg.dependencies);

export default {
    input: 'src/index.js',
    output: [
        {
            file: pkg.main,
            format: 'cjs'
        },
        {
            file: pkg.module,
            format: 'es'
        }
    ],
    external,
    sourceMap: 'inline',
    plugins: [
        resolve(), // resolve external modules from npm
        babel({
            exclude: '/node_modules/**' // only transpile our source code
        }),
        process.env.NODE_ENV === 'production' && uglify()
    ]
};

"devDependencies": {
    "@babel/core": "7.0.0-beta.42",
    "@babel/plugin-external-helpers": "7.0.0-beta.42",
    "@babel/polyfill": "7.0.0-beta.42",
    "@babel/preset-env": "7.0.0-beta.42",
    "@babel/preset-react": "7.0.0-beta.42",
    "@babel/preset-stage-2": "7.0.0-beta.42",
    "babel-eslint": "8.2.2",
    "babel-preset-es2015-rollup": "3.0.0",
    "eslint-config-airbnb": "16.1.0",
    "eslint-plugin-import": "2.9.0",
    "eslint-plugin-jsx-a11y": "6.0.3",
    "eslint-plugin-react": "7.7.0",
    "rimraf": "2.6.2",
    "rollup": "0.57.1",
    "rollup-plugin-babel": "3.0.3",
    "rollup-plugin-commonjs": "9.1.0",
    "rollup-plugin-node-resolve": "3.3.0",
    "rollup-plugin-uglify": "3.0.0",
    "the-answer": "1.0.0"
},
"dependencies": {
    "material-ui": "1.0.0-beta.39",
    "react": "16.3.0",
    "react-dom": "16.3.0",
    "react-proptypes": "1.0.0",
    "react-tap-event-plugin": "3.0.2"
},
"peerDependencies": {
    "react": "16.2.0",
    "react-dom": "16.2.0",
    "react-proptypes": "1.0.0",
    "react-tap-event-plugin": "3.0.2"
}

any thoughts?

Figured it out. You need "rollup-plugin-babel": "4.0.0-beta.3", with "@babel/core": "7.0.0-beta.42",. Posting it as it could be useful for others.

The issue seems to be resolved, it's all about disabling module transform and as other have posted the easiest nowadays is just to use babel-preset-env and have it configured like this:
{
  presets: [['env', { modules: false }]],
}

Please remember that when passing a preset/plugin to babel with options you have to wrap it in a tuple of [name, options]
Broken config might look like this:
{
  presets: ['env', { modules: false }],
}

presets: [               ["@babel/preset-env", {                 loose: true,                 modules: false,   // If you are intended to set modules as false, namely we want to support es6 module, then you must set loose as true               }] ]

presets: [
  ["@babel/preset-env", {
      loose: true,
      modules: false, // If you are intended to set modules as false, namely we want to support es6 module, then you must set loose as true
    }
  ]
]

I'm still getting "ReferenceError: require is not defined" in my browser console. Any help would be appreciated. I've attached my package.json, rollup.config.js and .babelrc files (renamed with a .log file extension so your site will allow them)
.eslintrc.js.log
package.json.log
rollup.config.js.log

If it helps, the call to require in my source file is:
const watsonSpeechRecognizer
= require('watson-speech/speech-to-text/recognize-microphone');
which appears in the output file, but with 'var' instead of 'const'.

presets: [
  ["@babel/preset-env", {
      loose: true,
      modules: false, // If you are intended to set modules as false, namely we want to support es6 module, then you must set loose as true
    }
  ]
]

@ ZhongyiChen was spot on. Gotta set loose!


Please add USBTX & USBRX to the list of drop-down values in SerialRedirectTo block

Dear Developers,
I'm using your programming environment to program micro:bit in Blockly mode.
From the page https://lancaster-university.github.io/microbit-docs/ubit/serial/
it seems that the underlying Serial library supports setting the serial transmission speed on the built-in USB port in the same way it is done for a custom Serial built around non-USB pins.  If it's true, could you please also add the "USBTX" to the "TX" drop-down and "USBRX" to "RX" drop-down list?
I believe it is a similar issue to this one:  https://github.com/Microsoft/pxt/issues/1902
Many thanks in advance!

You can bundled a C++ file in your project that calls straight into mbed until we get this one in.

@pelikhan  I don't know how to mix languages in micro:bit yet.  Can you refer me to any documentation?
( I was thinking of trying my MIDI-over-USB experiment completely in mbed/C++ environment, just to make sure that the hardware side of things works. )
How often do you usually have releases?

See https://pxt.microbit.org/packages
We try to stay on a 2 week schedule but it might take a bit more this time as some Microbit FOundation members are OOF.

@pelikhan we've just had another request for this on support - would be good to to work through any issues around bringing this stuff to light.

In ^v.0.14.1

There is a "redirect serial to USB block now":

I see. Thanks very much!


add code

To test it
Prerequisites

Endpoint : https://p9y9bbcruj.execute-api.us-west-2.amazonaws.com/default/simplePostLambda

Using postmen or smiler

Copy the endpoint and post it on the postmen url
Change method from get to post
select body, raw ,and change text to JSON
sending json as {"feedback":"write something here "}
click send
getting response with 201 when it is succeed.
getting response 400 for empty string

Using terminal

% curl -d '{"feedback":"Good job!"}' -H "Content-Type: application/json" -X POST https://p9y9bbcruj.execute-api.us-west-2.amazonaws.com/default/simplePostLambda
getting response with 201 when it is succeed
getting  response 400 for empty string

*It only accepts {"feedBack":"write something here "} syn , other way , it will sends errors.
It is not mange any get error therefore it will get you empty feed back , I am working to solve that .
Gallery
When the syntax is correct

When the syntax is dose not have a feedback.

When the syntax is incorrect and not follow syntax.

Close #81


Unbound or Dnsmasq to Override DNS Hostnames on the Interface level

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
No, just a request for improving existing functionality
Describe the solution you like
I need a way to override DNS hostnames on a per interface basis rather than globally.
Describe alternatives you considered
I looked at both Unbound and Dnsmasq.  Although I can enable them per interface, I can not configure them per interface.
Additional context
Unbound service appears to be the most capable to support this feature (at least according to the UI options).  I'm sure Dnsmasq can do the same. Doesn't matter to me which one it goes to.
This may require launching multiple Unbound or Dnsmasq daemons that each use separate config files and setup to listen on their respective interfaces.  Uses more RAM, makes things more complicated for sure.

Both are not suitable for the job, at least not in any form that is going to be implemented in OPNsense. The best bet is to use one service per interface or rewrite a custom unbound plugin that uses a custom configuration that achieves your goals.
To my knowledge, however, Unbound is a big server software for a single purpose so attempting to coerce it to play nice for multiple interfaces might be futile.
Cheers,
Franco
Cheers,
Franco

Pretty sure this is doable with Unbound. See [Unbound DNS

Advanced Configurations
](https://docs.opnsense.org/manual/unbound.html#advanced-configurations)

server:
    	access-control-view: <IP_ADDRESS>/24 somevlan
view:
  name: "somevlan"
  view-first: yes
  local-data: "host.example.com 60 IN A <IP_ADDRESS>"

Both might have issues in conjunction with python-script (https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#python-module-options), which is the feature we use to hook large lists. In our experience local-data has issues coping with larger sets of data, which is why we choose the script hook for blocklists for example, which unfortunately has some other disadvantages (such as serving from a single cache independent of the client subnet).
For blocklists we did add subnets in our business edition (https://docs.opnsense.org/vendor/deciso/extended_dnsbl.html), but it has some disadvantages, which also made it less attractive to rewrite the overrides to use the same kind of logic (a single facility to modify answers would be more practical in terms of maintenance).

Probably could play with module-config ordering. It was just a quick hint on what to look at, personally do not have a pressing use case for this. And yes, large lists are definitely better handled with python scripts.


Support data for URL is wrong

The doc suggests that Firefox does not support it while it has for some time and does now https://developer.mozilla.org/en-US/docs/Web/API/URL#Browser_compatibility
Same for Chrome which supports it since Chrome 32 apparently.

In fact our docs show polyfillable in Chrome, missing in FF:

Chrome makes sense. It has only fully supported URL since 49, which we are not yet showing on our compat table.  Firefox claims full support since 29, but is failing one of our tests on its native implementation:

So we could skip this test, on the basis that the bug is not important enough to stop us showing the native support exists.

@triblondon To be clear, the docs show Feature missing instead of Feature supported natively in FF because of that failing test?

Yes

Just for info, in the Firefox 47 (dev edition) console:
var url = new URL('http://example.com');

url.searchParams.append('this\x00&that\x7f\xff', '1+2=3');

console.log(url.search === '?this%00%26that%7F%C3%BF=1%2B2%3D3');
console.log(url.href === 'http://example.com/?this%00%26that%7F%C3%BF=1%2B2%3D3');

both yield true, so I guess the tests started passing sometimes between FF39 and FF47.
Anyway, thanks for addressing this!

@DavidBruant we have data for FF41 so it's somewhere between 42 and 47.
As a footnote to this, an alternative solution is to add this test to the detect.js file, which makes passing this test part of the definition of whether the feature is present or not. That would mean that we would serve the polyfill to FF < ~47, on the basis that the native impl is not detectable. However, it's a judgement call, and I take the view that it unnecessary to polyfill a feature that has a native implementation which for most use cases is not broken.


[PT Run] Replace tblimp-Microsoft.Search.Interop package with source implementation

Summary of the Pull Request

What is this about:
This PR removes the tlbimp-Microsoft.Search.Interop NuGet package and replaces with COM Interop interfaces/classes to provide an implementation that works with what is currently used .
What is included in the PR:

tblimp-Microsoft.Search.Interop NuGet package removed
SearchAPI implementation of SearchManager, SearchQueryHelper, and SearchCatalogManager
Microsoft.Search.Interop.dll removed from Setup

How does someone test / validate:

Open Power Toys Run and enter a search term that should return results from files on your computer.
Place a breakpoint at the end of the ExecuteQuery method in WindowsSearchAPI.cs. Verify the results list contains results from your local search index.

Quality Checklist

[X] Linked issue: #8557
[X] Communication: I've discussed this with core contributors in the issue.
[X] Tests: Added/updated and all pass
[X] Installer: Added/updated and all pass
[X] Localization: All end user facing strings can be localized
[X] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[X] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

🔥🔥🔥

Thanks a lot for opening this PR @snickler
This will really help in supporting arm64
Where do the class definitions come from, though? Any Microsoft docs or were they reverse engineered? Could you please link the source here or explain how you generated those?
Thanks, in advance.

Thanks a lot for the explanation @snickler !


Instantiate client with combined API key in readme

So that users don’t have to worry about:

base URL
service ID

Depends on:

[x] alphagov/notifications-admin#968

@leohemsted 	b590e3e


DBZ-4669 Use the newer version of plugins to consolidate remaining duplicates plugin versions

Additional consolidation duplicate plugin versions properties per request of @jpechane in #4027 (https://github.com/debezium/debezium/pull/4027#issuecomment-1307223030).

@jpechane I was able to duplicate the formatting check errors on my other branch DBZ-2439, so I went ahead and applied them in PR #4045 since that is addressing the other formatting/style changes. I'm not clear if that is what you wanted, or if I should revert that change in DBZ-2439 and add it here.
I also went ahead and rebased this branch. The PR is still failing for Checkstyle and Formatting, and would need rebasing again once #4045 is merged.

I have combined this PR with #4045


Omnisci DBE: Set root_abs_path relative to libDBEngine

An issue has been found during the conda-forge packages:
To locate RuntimeFunctions.bc, omniscidb binary uses /proc/self/exe  to define the root path. This works fine because QueryEngine/ and bin/ directories live under the same base (opt/omnisci-cpu) and omniscidb location is in environment PATH.
The problem is with libDBEngine.so that uses the same method but apparently, the standard approach of creating its symbolic link to $PREFIX/lib does not work because the root path from /proc/self/exe  is going to be different from the QueryEngine/ base path and libDBEngine.so will not be able to locate RuntimeFunctions.bc ...
Some changes will be required to omniscidb, the WIP branch is https://github.com/pearu/omniscidb/tree/pearu/conda.

PR created: https://github.com/omnisci/omniscidb/pull/646
Discussion: https://github.com/pearu/omniscidb/commit/e3203c0016588dfbfe6078b34c3ff2b3cdaf8153

PR closed since the issue was fixed by https://github.com/omnisci/omniscidb/pull/647


Create Image from Text

Hi,
I need to create a png having text. Also i need to apply opacity to that. How do i do that?

try it https://github.com/andresousa/react-native-text-to-image#readme