Update vulnerable dependency

Noticed this on my project, that has morfologik as a dependency.
com.beust:jcommander:1.48, referenced on morfologik-tools/pom.xml  should be updated to 1.75 or later.
https://snyk.io/vuln/SNYK-JAVA-COMBEUST-174815
It is not a simple update. A simple change of version in the pom will throw this:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.3:compile (default-compile) on project morfologik-tools: Compilation failure: Compilation failure:
[ERROR] /home/pc/morfologik/morfologik-stemming/morfologik-tools/src/main/java/morfologik/tools/CliTool.java:[181,7] method usage in class com.beust.jcommander.JCommander cannot be applied to given types;
[ERROR]   required: no arguments
[ERROR]   found: java.lang.StringBuilder
[ERROR]   reason: actual and formal argument lists differ in length
[ERROR] /home/pc/morfologik/morfologik-stemming/morfologik-tools/src/main/java/morfologik/tools/CliTool.java:[187,7] method usage in class com.beust.jcommander.JCommander cannot be applied to given types;
[ERROR]   required: no arguments
[ERROR]   found: java.lang.StringBuilder
[ERROR]   reason: actual and formal argument lists differ in length
[ERROR] /home/pc/morfologik/morfologik-stemming/morfologik-tools/src/main/java/morfologik/tools/CustomParameterConverters.java:[9,1] morfologik.tools.CustomParameterConverters is not abstract and does not override abstract method getConverter(java.lang.Class<?>) in com.beust.jcommander.IStringConverterFactory
[ERROR] /home/pc/morfologik/morfologik-stemming/morfologik-tools/src/main/java/morfologik/tools/CustomParameterConverters.java:[19,51] name clash: getConverter(java.lang.Class<T>) in morfologik.tools.CustomParameterConverters and getConverter(java.lang.Class<?>) in com.beust.jcommander.IStringConverterFactory have the same erasure, yet neither overrides the other
[ERROR] /home/pc/morfologik/morfologik-stemming/morfologik-tools/src/main/java/morfologik/tools/CustomParameterConverters.java:[18,3] method does not override or implement a method from a supertype
[ERROR] -> [Help 1]
[ERROR]

Why do you have a dependency on the tools? This isn't a library package -- it's a set of end-user programs.

This was an FYI, after looking at the snyk warning, and I believe that since this library is used by hundreds of projects, a vulnerability here can be significant.
If I understand the report correctly, even though there is a low change that you got an altered version of that dependency, if that was the case, it can contain anything in it.

Why do you have a dependency on the tools? This isn't a library package -- it's a set of end-user programs.

The use I give is irrelevant to this problem, but the tools are integrated to help language developers build or update the morfologik dictionaries used.

Updated to jcommander 1.78.


HyperX Cloud Stinger

Hello !
Do you have the Stinger hid commands ? (to get battery level, etc).
Tks.

I don't have documentation, but you can try by capturing the bytes on windows by using the official hyperx app.

Hi @alexwbaule, did you succeed?

No yet.


Rounding improvement and sleep to prevent 429 error

ETH should also be logged with 8 digits, same as BTC, in compound.py. And as we discussed on Discord; protection againt 429 error is not handled in Py3CW lib so added a simple sleep in the hope it will do. Tried to judge and added the sleep to scripts working with deals, as it seems to more intensive ones.

Thanks again!


Add telemetry

Dropped feature as we barely have enough time for a Proof Of Concept for the app. This will be reopened in case we decide to develop it into a MVC.


HttpClient isn't disposed if transport fails to start when using HttpConnection and HubConnection

We don't dispose the HttpClient if the transport fails to Start. This could cause resource leaks when using HubConnection in a retry loop since it can't call DisposeAsync because it doesn't even have a reference to the ConnectionContext if ConnectAsync fails to start.

Reopening to track porting the fix from dev into release/2.1 in the next milestone (pending shiproom approval)

We'll bring this back to the shiproom.


NameError: uninitialized constant ForecastIO::Forecast

I installed the gem and i can do configure ForecastIO.
If i want to use the ForecastIO::Forecast.new command somehow my application can not pick up the Forcast class.
I'm running ruby 2.4.0p0 and Rails 5.0.2.
forecast_io (2.0.1)
I was trying to run some api calls in the terminal.
Somehow the constants get not loaded.
puts ForecastIO.constants.inspect
[:VERSION, :Configuration]
=> nil

I think this gem has the same name, and is being used instead: https://github.com/darkskyapp/forecast-ruby
I had the same issue and the methods are different so I tried this gem and it worked.


Support tests for "Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface"

Hi @matthiasnoback,
If it is not already possible, this bundle should be capable to invoke the Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface::prepend() method in order to unit test it as well as the load method...
Do you agree?
Thank you!

Issue can be closed. It is already possbile in newer versions.


Summary of alerts not sent to pagerduty

When an alert is sent to pagerduty the description is part of what's sent, but not the summary. We should make both available.

Looks like this has been implemented,


Create prime.h and introduce is_prime function

This commit creates number_theory/prime.h header file and implements
is_prime function for integer types.

I find that std::sqrt is not constexpr....
If we use std::sqrt, we can't mark is_prime as constexpr.


Multiple storage blocks generated in config.json

Preflight Checklist

[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.
[X] I agree to follow the Code of Conduct.

Vault Helm Chart Version
1.20.3
Bank-Vaults Version
No response
Kubernetes Version
1.30
Kubernetes Distribution/Provisioner
EKS
Expected Behavior
Deploying the chart with custom values for vault.config.storage should not generate a config.json with two storage blocks:
Values file
vault:
  vault:
    config:
      storage:
        dynamodb:
          ha_enabled: true
          region: us-east-1
          table: "vault-backend"
...

Expected config json:
{
  "api_addr": "http://<IP_ADDRESS>:8200",
  "storage": {
    "dynamodb": {
      "ha_enabled": true,
      "region": "us-east-1",
      "table": "vault-backend"
    }
  },
  "telemetry": {
    "statsd_address": "localhost:9125"
  },
  "ui": true
}

Actual Behavior
Two storage blocks are added to the config.json
Generated config json:
{
  "api_addr": "http://<IP_ADDRESS>:8200",
  "storage": {
    "dynamodb": {
      "ha_enabled": true,
      "region": "us-east-1",
      "table": "vault-backend"
    },
    "file": {
      "path": "/vault/file"
    }
  },
  "telemetry": {
    "statsd_address": "localhost:9125"
  },
  "ui": true
}

This causes an error in the vault container since there is more than one storage block
error loading configuration from /vault/config/config.json: error parsing 'storage': only one "storage" block is permitted
Steps To Reproduce
No response
Logs
No response
Additional Information
The storage block for file seems to be from values.yaml#L194, previously this was set to {}. This was reverted in PR#103

also having this issue. is there any workaround?

We also have this issue after switching to this chart from the banzaicloud chart (because their domain has been down for ages). We use mysql storage.
Generated config:
{
  "api_addr": "http://<IP_ADDRESS>:8200",
  "disable_mlock": false,
  "listener": {
    "tcp": {
      "address": "[::]:8200",
      "tls_cert_file": "/vault/tls/server.crt",
      "tls_disable": true,
      "tls_key_file": "/vault/tls/server.key"
    }
  },
  "storage": {
    "file": {
      "path": "/vault/file"
    },
    "mysql": {
      "address": "pxc.database.svc.cluster.local:3306",
      "password": "<redacted>",
      "username": "vault"
    }
  },
  "telemetry": {
    "statsd_address": "localhost:9125"
  },
  "ui": true
}```


Error could not find CMAKE_PROJECT_NAME in Cache

Hello,
I ran
sudo python3 setup.py install
as instructed and got an error
Error: could not find CMAKE_PROJECT_NAME in Cache
Here is the full output:
running install
Checking .pth file support in /usr/local/lib/python3.6/dist-packages/
/usr/bin/python3 -E -c pass
TEST PASSED: /usr/local/lib/python3.6/dist-packages/ appears to support .pth files
running bdist_egg
running egg_info
writing habitat_sim.egg-info/PKG-INFO
writing dependency_links to habitat_sim.egg-info/dependency_links.txt
writing requirements to habitat_sim.egg-info/requires.txt
writing top-level names to habitat_sim.egg-info/top_level.txt
reading manifest file 'habitat_sim.egg-info/SOURCES.txt'
writing manifest file 'habitat_sim.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
running build_ext
Error: could not find CMAKE_PROJECT_NAME in Cache
Traceback (most recent call last):
File "setup.py", line 325, in
zip_safe=False,
File "/usr/lib/python3.6/distutils/core.py", line 148, in setup
dist.run_commands()
File "/usr/lib/python3.6/distutils/dist.py", line 955, in run_commands
self.run_command(cmd)
File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
cmd_obj.run()
File "/usr/lib/python3/dist-packages/setuptools/command/install.py", line 67, in run
self.do_egg_install()
File "/usr/lib/python3/dist-packages/setuptools/command/install.py", line 109, in do_egg_install
self.run_command('bdist_egg')
File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
self.distribution.run_command(command)
File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
cmd_obj.run()
File "/usr/lib/python3/dist-packages/setuptools/command/bdist_egg.py", line 161, in run
cmd = self.call_command('install_lib', warn_dir=0)
File "/usr/lib/python3/dist-packages/setuptools/command/bdist_egg.py", line 147, in call_command
self.run_command(cmdname)
File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
self.distribution.run_command(command)
File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
cmd_obj.run()
File "/usr/lib/python3/dist-packages/setuptools/command/install_lib.py", line 23, in run
self.build()
File "/usr/lib/python3.6/distutils/command/install_lib.py", line 109, in build
self.run_command('build_ext')
File "/usr/lib/python3.6/distutils/cmd.py", line 313, in run_command
self.distribution.run_command(command)
File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
cmd_obj.run()
File "setup.py", line 166, in run
self.build_extension(ext)
File "setup.py", line 220, in build_extension
shlex.split("cmake --build {}".format(self.build_temp)) + build_args
File "/usr/lib/python3.6/subprocess.py", line 311, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['cmake', '--build', 'build', '--config', 'RelWithDebInfo', '--', '-j']' returned non-zero exit status 1.

I'm not really sure what's going on here.  It could be that sudo is messing things up as root can have slightly different paths.  The recommended way to install habitat-sim is in a conda environment, https://docs.conda.io/en/latest/miniconda.html, can you try that?

Fixed by adding project(habitat) to CMakeLists.txt


Enable telemetry daemon tracing on Windows and macOS

Re-enable the telemetry daemon trace listener for Windows and enable it for macOS (using the byte-based transmission mode).
Resolves #739
Resolves #529

Do we have any idea about the maximum message size we might see?  (asking for a friend)

Do we have any idea about the maximum message size we might see? (asking for a friend)

The sort of minimum payload size would be 100-150 bytes (version info, mount & enlistment IDs).
A simple git status results in a message of the order of 500-800 bytes.
Something like a single background maintenance step is about 220 bytes.
During mount we send at least one message that includes a bunch of information about the enlistment that is about 2000 bytes.

WRT max message size.  That's fine.  I'm receiving into a 64k buffer and the socket layer is likely chunking traffic to me into 8k chunks.  So we should be fine.

Closing and reopening to trigger a rebuild.

@jrbriggs was there anything left on this PR that you wanted changing? (I believe your review vote is the only thing blocking this now :-) ).


Base64 not displayed on iOS devices 10.3.2

Ionic version:  (check one with "x")
2.3.0
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
The base 64 images are not displayed on any views with iOS 10.3.2
This is currently working with android devices and web.
I saw that people have issues with ionic view but here I am testing on real device (iPhone SE)
I tried with
<img src="data:image/{{profilePicture.type}};base64,{{profilePicture.elementBase64}}">
<ion-img="data:image/{{profilePicture.type}};base64,{{profilePicture.elementBase64}}">
Expected behavior:
The base64 image must be displayed
Steps to reproduce:
Create a sample project with ionic 2.3.0
Create a variable profilePicture which contains type png and elementBase64 with the base64 content
ionic build iOS and deploy on iOS device 10.3.2

Hello,
So after an update of my app by removing the Content-Security-Policy the base64 image appears. However it is just shown when the page is reloaded (example I navigate to a page and go back to the one where the image is not displayed)

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!

@jgw96 this is not support. This is a bug. The base64 is not loaded the first time and the issue only occurs on iOS devices. The base64 is there as I tried to display the code as text and I can see it.
You can make a simple try and you will see.

Just for information I even put a timeout and the pictures is grayed.
Once I switch the tab and go back the picture appears
This only occurs on ios.
This is reproducible with serve --lab


Integrate Apache PLC4X for additional connectors

Connectors extensions
Anyone ever thought about integrating or using Apache PLC4X for additional connectors?

Thanks for the input, yes, we are actively appraising this. We consider base runtime requirements, transitive dependencies, functionality, licensing and project activity when we decide on which libraries to incorperate and indeed we currently have an abstraction in place for PLC4X that is going through this process. Hope this help to answer your question. Please do let us know of any thoughts you have around this or any other considerations.

The abstraction is complete and resides on a branch. No implementations are yet ready to be shipped. We could merge the module to master with build flags or removal of manifest definitions.

I cannot merge this branch until the status work is complete so this is pending the merge.

This is in master


Two ports per host

Hello. I have a need to pass both ports (80 and 443) to my server, but I can't find a solution exactly how to do this. Similar questions have already been found on the Internet, but there are no answers there. There is no such function in the NPM interface, I think this can be done in the NPM configuration files, or in the "advanced" tab, but I don't know how. Can you help with this issue? Any option will suit me, even if I need to manually configure nginx, as long as this option works.

What is your current "docker run" command or compose file or equivalent method to start NPM?

NPM is launched using the docker-compose file.yml with these parameters:
version: '3.8'
services:
app:
image: 'jc21/nginx-proxy-manager:latest'
restart: unless-stopped
ports:
- '80:80'
- '81:81'
- '443:443'
volumes:
- ./data:/data
- ./letsencrypt:/etc/letsencrypt

Apologies, I think I misread your original question - you would like a single proxy host entry in NPM to handle both http and https for a given server?

Apologies, I think I misread your original question - you would like a single proxy host entry in NPM to handle both http and https for a given server?

Yes, that's right.

I did some testing and got it working using the advanced config shown below:
location / {
    # backend targets
    set $backend_http "http://test:80";
    set $backend_https "https://test:443";

    # proxy to https backend
    if ($scheme = "https") {
        proxy_pass $backend_https;
    }

    # proxy to http backend
    if ($scheme = "http") {
        proxy_pass $backend_http;
    }
}

where "test" is the name of the nginx container I created to sit behind NPM.
tests using curl to observe the desired behavior, since the web browser was adamant on upgrading http to https no matter what I tried:
(I redacted the values for my environment with mydomain and myip)
C:\>curl -f http://mydomain -v
* Host mydomain:80 was resolved.
* IPv6: (none)
* IPv4: myip
*   Trying myip:80...
* Connected to mydomain (myip) port 80
> GET / HTTP/1.1
> Host: mydomain
> User-Agent: curl/8.7.1
> Accept: */*
>
< HTTP/1.1 200 OK
< Server: openresty
< Date: Thu, 19 Sep 2024 01:45:16 GMT
< Content-Type: text/plain
< Content-Length: 16
< Connection: keep-alive
<
Hello from HTTP!* Request completely sent off
* Connection #0 to host mydomain left intact

C:\>curl -f https://mydomain -v
* Host mydomain:443 was resolved.
* IPv6: (none)
* IPv4: myip
*   Trying myip:443...
* Connected to mydomain (myip) port 443
* schannel: disabled automatic use of client certificate
* ALPN: curl offers http/1.1
* ALPN: server accepted http/1.1
* using HTTP/1.x
> GET / HTTP/1.1
> Host: mydomain
> User-Agent: curl/8.7.1
> Accept: */*
>
* Request completely sent off
< HTTP/1.1 200 OK
< Server: openresty
< Date: Thu, 19 Sep 2024 01:48:03 GMT
< Content-Type: text/plain
< Content-Length: 17
< Connection: keep-alive
<
Hello from HTTPS!* Connection #0 to host mydomain left intact

nginx.conf file for this test:
events {}

http {
    server {
        listen 80;
        server_name mydomain;

        location / {
            return 200 "Hello from HTTP!";
        }
    }

    server {
        listen 443 ssl;
        server_name mydomain;

        ssl_certificate /etc/nginx/ssl/test.pem;
        ssl_certificate_key /etc/nginx/ssl/test.key;

        location / {
            return 200 "Hello from HTTPS!";
        }
    }
}

Although it looks weird to have a proxy host which completely ignores what you configure in the "scheme" and "forward port" fields.

I would rather configure two proxy hosts, one for each port, but NPM does not allow it with the same FQDN 😏 (which must be the reason for your post here).
Cheers

I did some testing and got it working using the advanced config shown below:
location / {
    # backend targets
    set $backend_http "http://test:80";
    set $backend_https "https://test:443";

    # proxy to https backend
    if ($scheme = "https") {
        proxy_pass $backend_https;
    }

    # proxy to http backend
    if ($scheme = "http") {
        proxy_pass $backend_http;
    }
}

I understand that this code should be written in the "advanced" tab? If so, will that be enough? Or will it be necessary to additionally write code for the nginx.conf file?

I did some testing and got it working using the advanced config shown below:
location / {
    # backend targets
    set $backend_http "http://test:80";
    set $backend_https "https://test:443";

    # proxy to https backend
    if ($scheme = "https") {
        proxy_pass $backend_https;
    }

    # proxy to http backend
    if ($scheme = "http") {
        proxy_pass $backend_http;
    }
}

I understand that this code should be written in the "advanced" tab? If so, will that be enough? Or will it be necessary to additionally write code for the nginx.conf file?

Yes, this section above goes into the Advanced tab for the NPM proxy host.
The code for nginx.conf is not necessary for NPM, I just included it here for completeness (how I tested what I provided) and for reference, in case you need to troubleshoot and compare the test backend I used with your actual setup.

Hi,
It looks like it is exactly that I need too.
But my reverse proxy behaves diferently and I am getting some certificate error:
curl -f https://my.doma.in -v
*   Trying XX.XXX.XXX.26:443...
* Connected to my.doma.in (XX.XXX.XXX.26) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
*  CAfile: /etc/ssl/certs/ca-certificates.crt
*  CApath: /etc/ssl/certs
* TLSv1.0 (OUT), TLS header, Certificate Status (22):
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.2 (IN), TLS header, Unknown (21):
* TLSv1.3 (IN), TLS alert, unrecognized name (624):
* error:0A000458:SSL routines::tlsv1 unrecognized name
* Closing connection 0
curl: (35) error:0A000458:SSL routines::tlsv1 unrecognized name

While I still not sure are this problem came from my local machine.
It is VM with mailinabox script, that I ran and I am not sure that it managed to make right certificate.
While I can test it working on lan https via browser.
Browser tells that domain is right, but there is the same date for certificate start and end.
brgds


cubegen for unit cells

Trying to create .cub files for periodic systems has a few issues. Among them:

There is no way to currently create a .cub file using pbc_eval_gto since it defaults to eval_gto (and no PBC tag is added)
Box margins for periodic systems should be determined wholly by the unit cell
The symmetric mesh causes inconsistencies across periodic images.

I changed code in several places to get a functional version of what I am looking for, which may or may not be the best or cleanest way to go about it, but it works (I removed docstrings in code blocks for brevity):
Change #1:
https://github.com/sunqm/pyscf/blob/2efe5942b60868857f58cd62c0538b0a896589f3/pyscf/tools/cubegen.py#L103-L144

Added a margin argument to cubegen.orbital(...) so that a user can actually specify a margin manually.
This change is separate from the rest, and isn't necessary (or even used) for periodic systems

def orbital(mol, outfile, coeff, nx=80, ny=80, nz=80, resolution=RESOLUTION, margin=BOX_MARGIN):
    cc = Cube(mol, nx, ny, nz, resolution, margin)

    # Compute density on the .cube grid
    coords = cc.get_coords()
    ngrids = cc.get_ngrids()
    blksize = min(8000, ngrids)
    orb_on_grid = numpy.empty(ngrids)
    for ip0, ip1 in lib.prange(0, ngrids, blksize):
        ao = numint.eval_ao(mol, coords[ip0:ip1])
        orb_on_grid[ip0:ip1] = numpy.dot(ao, coeff)
    orb_on_grid = orb_on_grid.reshape(cc.nx,cc.ny,cc.nz)

    # Write out orbital to the .cube file
    cc.write(orb_on_grid, outfile, comment='Orbital value in real space (1/Bohr^3)')
    return orb_on_grid

Change #2:
https://github.com/sunqm/pyscf/blob/2efe5942b60868857f58cd62c0538b0a896589f3/pyscf/tools/cubegen.py#L204-L227

Changed the __init__ function for Cube objects to define the box, origin, and mesh symmetry differently for Cells

    def __init__(self, mol, nx=80, ny=80, nz=80, resolution=RESOLUTION,
                 margin=BOX_MARGIN, origin=ORIGIN, extent=EXTENT):
        self.mol = mol
        coord = mol.atom_coords()

        from pyscf.pbc.gto import Cell
        # If the molecule is periodic, use lattice vectors as the box
        # and ignore magin, origin, and extent arguments
        if isinstance(mol, Cell):
            self.box = mol.lattice_vectors()
            box = numpy.diag(self.box)
            self.boxorig = (numpy.max(coord, axis=0) + numpy.min(coord, axis=0))/2 - box/2
        else:
            if extent is None:
                box = numpy.max(coord,axis=0) - numpy.min(coord,axis=0) + margin*2
                self.box = numpy.diag(box)
            else:
                self.box = numpy.diag(extent)

            if origin is None:
                self.boxorig = numpy.min(coord,axis=0) - margin
            else:
                self.boxorig = numpy.array(origin)

        if resolution is not None:
            nx, ny, nz = numpy.ceil(numpy.diag(self.box) / resolution).astype(int)
        self.nx = nx
        self.ny = ny
        self.nz = nz

        # Use an asymmetric mesh for tiling unit cells
        if isinstance(mol, Cell):
            self.xs = numpy.arange(nx) * box[0] / nx
            self.ys = numpy.arange(ny) * box[1] / ny
            self.zs = numpy.arange(nz) * box[2] / nz
        else:
            # .../(nx-1) to get symmetric mesh
            # see also the discussion https://github.com/sunqm/pyscf/issues/154
            self.xs = numpy.arange(nx) * box[0] / (nx - 1)
            self.ys = numpy.arange(ny) * box[1] / (ny - 1)
            self.zs = numpy.arange(nz) * box[2] / (nz - 1)

Change #3:
https://github.com/sunqm/pyscf/blob/8d29c05eca89c83d21edb6f935ea2aad79d6dbf2/pyscf/dft/numint.py#L47-L106

Changed the feval of eval_ao to start with "PBC" if it is a unit cell. This, in particular, may have a better solution than checking for a Cell object directly. It may even be worth adding a periodic extension to numint, but I'll leave that up to you.

def eval_ao(mol, coords, deriv=0, shls_slice=None,
            non0tab=None, out=None, verbose=None):
    comp = (deriv+1)*(deriv+2)*(deriv+3)//6
    if mol.cart:
        feval = 'GTOval_cart_deriv%d' % deriv
    else:
        feval = 'GTOval_sph_deriv%d' % deriv

    from pyscf.pbc.gto import Cell
    if isinstance(mol, Cell):
        feval = "PBC" + feval
    return mol.eval_gto(feval, coords, comp, shls_slice, non0tab, out=out)

Thanks for the code patch and suggestions. It is added in PR #534


jQuery Change event not firing on input element when bootstrap switch is clicked

In my form there are other scripts watching the checkbox for a change event, but when clicking the bootstrap switch it's not being fired.
I'm now using the following code to patch this up:
jQuery("[type=\'checkbox\']").on('switchChange.bootstrapSwitch', function(event, state) {
  jQuery(this').trigger('change');
});

Is this a bug or a problem with my implementation? I'm using jQuery 1.8, is there a minimum required version?

Did you copy-paste the code? I think that $("[type=\'checkbox\']") should be $('[name="my-checkbox"]')

I have the same issue. Check this jsFiddle to reproduce it: https://jsfiddle.net/fguillen/nvt833c5/3/


beam.BeamMutex -> beam.Mutex

also, make it public!  It's currently private, so we can be confident that no one is using it =P


Ghidra Server and CIS

Is there any guidance or info on how to get Ghidra Server to properly run on a CIS-hardened server?
By design there are some restrictions, particularly over the /tmp directory. I'm unable to get the parameters -Dwrapper.tmp.path or -Djna_tmpdir to stop using it. I've tried adding these to server.conf but this doesn't seem to change. The server won't start because of issues with the directory:
Exception in thread "main" java.lang.UnsatisfiedLinkError: /tmp/jna17948382034945794661.tmp: /tmp/jna17948382034945794661.tmp: failed to map segment from shared object
Pulling my hair out. Ideally, I shouldn't have to ditch CIS but not out of the question at this point.

Could you describe how you put the params in server.conf?
I haven't tried anything yet, but at first glance it should be:
wrapper.java.additional.10=-Djna.tmpdir=/other/path

That's what I am using, although I used the next logical number in the ordering. Does it need to be .10?
wrapper.java.additional.7=-Dwrapper.tmp.path
wrapper.java.additional.8=-Djna.tmpdir (Note I've also tried Djna_tmpdir)
Still get this when starting Server;
From wrapper.log; Exception in thread "main" java.lang.UnsatisfiedLinkError: /tmp/jna11201110639933190708.tmp: /tmp/jna11201110639933190708.tmp: failed to map segment from shared object
Screen showing tmp paths haven't changed

It will be neccessary to get the initial wrapper.jar launch to have a similar settings (i.e., the process that reads server.conf).  This may require significant modification of the ghidraSvr scripts.  Since we do not use the standard YAJSW scripts it may be time for an overhall of our ghidraSvr script.  You can find the standard scripts within a clean unpack of the YAJSW distribution jar found in Ghidra/Features/GhidraServer/data to get an idea how they recommend launching.
I will open an internal ticket to address this.

Out of my league here, but what about a naive  export _JAVA_OPTIONS=-Djava.io.tmpdir=/tmpdir_with_exec or java -Djava.io.tmpdir=/tmpdir_with_exec <ghidra invocation>  ? I've been using that for a few applications that don't like when /tmp is noexec

Out of my league here, but what about a naive  export _JAVA_OPTIONS=-Djava.io.tmpdir=/tmpdir_with_exec or java -Djava.io.tmpdir=/tmpdir_with_exec <ghidra invocation> ? I've been using that for a few applications that don't like when /tmp is noexec

I do have this set in server.conf and it works but only after the server is started from what I understand. It sounds like the wrapper launcher itself uses /tmp before we read the .conf file and that's an issue.
Is there a way to tell Java not to use /tmp at all to avoid this? I am not a Java guy.

Options have to be forced onto java wrapper execution (see ghidraSvr script).  We have established an internal ticket to allow the tmp directory to be properly configured.

I'm not sure what changes are required so the only solution that currently works is to export _JAVA_OPTIONS, but this doesn't persist after a reboot. I can live with this assuming the fix is soon.

Add the JVM command line arg to these calls to the wrapper in server/ghidraSvr:
"${JAVA_CMD}" -jar "${WRAPPER_HOME}/wrapper.jar"

After some trial and error, I am able to add these paths but they seem to get overridden:

I'm implementing the fix for this now.  I opened a bug with yajsw because their documention references jna.tmpdir when it should be jna_tmpdir.

I'm implementing the fix for this now. I opened a bug with yajsw because their documention references jna.tmpdir when it should be jna_tmpdir.

Thanks for the update. Is this in the 10.2.3 release? I didn't see anything in change notes.

No, it didn't make it into 10.2.3.

@t0ta11ed if possible, can you try out the master branch with the new WRAPPER_TMPDIR variable in ghidraSvr?  It works as far as I can tell but I didn't test on a system with a restricted /tmp dir, so i can't be 100% sure a rogue file isn't getting created there behind the scenes.

@t0ta11ed if possible, can you try out the master branch with the new WRAPPER_TMPDIR variable in ghidraSvr? It works as far as I can tell but I didn't test on a system with a restricted /tmp dir, so i can't be 100% sure a rogue file isn't getting created there behind the scenes.

Happy to give that a whirl but not 100% sure how I would do that. I normally grab your release file. Would I be compiling it from the branch myself? Afaik, all you have to do is set /tmp to NOEXEC to recreate, which is what CIS controls are doing.


Operator precedence

The following ABAP snippet:
a = a * b + c * d.

is transpiled to the following JavaScript code:
a.set(a.multiply(b.add(c.multiply(d))));

If you set a, b, c, d to 2, 3, 4, 5, the result of the transpiled expression will be 2 * (3 + (4 * 5) = 46, due to the way the expression is constructed. The correct result would be 26.
This happens because the syntax tree abaplint creates for this expression looks like this:

Other parsers like the ones generated by ANTLR resolve this problem by moving preceding operations "further down" in the syntax tree (for example: https://github.com/antlr/grammars-v4/blob/master/arithmetic/arithmetic.g4).
I attempted to replicate this behavior by directly modifying the output of getStructure() before passing and it seemed to work: https://github.com/BurnerPat/transpiler/commit/5222318662b2307b71e7356f44fcd1e939a0a241
This changes the tree shown in the issue description to this:

@larshp Can you share your thoughts on this approach?

Its a workaround, the transpiler is a workaround as is, so its okay for now
As a long term solution, I think its possible to change the grammar so it gets the correct tree
https://syntax.abaplint.org/?filter=source#/expression/ArithOperator
but it might be a bit more work and run into other issues

Yeah, I agree; it's definitely not a long-term solution.
I'll look into getting an MR ready for the changes and after that maybe see if I can figure something out for the grammar aswell.

the following two examples give different outputs, it should not
output = 100 * noun + verb.
output = ( 100 * noun ) + verb.

I guess there is the same problem with
IF 1 = 2 OR 1 = 2 AND 1 = 2.
ENDIF.

I would like to implement a abaplint rule which requires the precedence to be explicit, for both arithmetic and boolean, something like the second part of https://docs.abapopenchecks.org/checks/59/, the rule would have quick fixes defaulting to the default precedence
the first part of AOC59 is done in https://rules.abaplint.org/many_parenthesis/

https://github.com/abapGit/abapGit/blob/c554ff2fc754227dcba60d7e476f5845c94bb334/src/zcl_abapgit_version.clas.abap#L242

  DATA int TYPE i.
  int = 4 - 0 + 1.
  ASSERT int = 5.

@BurnerPat I tried the above fix, did not work for all my unit tests, I might do a few tweaks
  it("Arithmetics, precedence 1", async () => {
    const code = `ASSERT 4 - 0 + 1 = 5.`;
    const js = await run(code);
    const f = new Function("abap", js);
    f(abap);
  });

  it("Arithmetics, precedence 2", async () => {
    const code = `
  DATA int TYPE i.
  int = 2 * 2 + 3 * 3.
  WRITE int.`;
    const js = await run(code);
    const f = new Function("abap", js);
    f(abap);
    expect(abap.console.get()).to.equal("13");
  });

  it("Arithmetics, precedence 3", async () => {
    const code = `ASSERT 100 * 10 + 2 = ( 100 * 10 ) + 2.`;
    const js = await run(code);
    const f = new Function("abap", js);
    f(abap);
  });

  it("Arithmetics, precedence 4", async () => {
    const code = `ASSERT 4 + 1 - 2 = 3.`;
    const js = await run(code);
    const f = new Function("abap", js);
    f(abap);
  });

  it("Arithmetics, precedence 5", async () => {
    const code = `ASSERT 4 + ( 1 - 2 ) = 3.`;
    const js = await run(code);
    const f = new Function("abap", js);
    f(abap);
  });

  it("Arithmetics, precedence 6", async () => {
    const code = `
  DATA int TYPE i.
  int = 4 - 1 - 2.
  WRITE int.`;
    const js = await run(code);
    const f = new Function("abap", js);
    f(abap);
    expect(abap.console.get()).to.equal("1");
  });

done, closing


Data might be lost when power failure happens because SQLite WAL mode is used

ChiselStrike uses SQLite WAL mode which is more performant than the standard SQLite recovery mode. The WAL mode is enabled here.
The problem is that writes to WAL are stored in memory only[1]. This memory gets flushed to disk only during checkpoint operation which happens automatically when the size of the WAL file grows to 1000 pages or more by default[2].
If a power failure happens when some writes reside only in the memory, those writes will be lost.
The WAL file size that triggers auto-checkpoint can be configured. Checkpoint can be also triggered by user. It is also possible to force checkpoint operation after every commit by using PRAGMA synchronous=FULL but there's probably a performance price for that.
[1] D. Richard Hipp - SQLite [The Databaseology Lectures - CMU Fall 2015]
[2] SQLite documentation about WAL

To be precise, the writes are not stored in memory only (it might have been true in 2015 during that talk perhaps?). There exists a .wal file where writes are appended, and it's up to the operating system's page cache to persist these changes to disk, so writes might survive a power failure even if there was no checkpoint.
That doesn't change the general conclusion from this issue that we should consider one of the below:

full synchronous mode
explicitly checkpointing more often
assuming that losing some of the new writes (but without corrupting the database file) is an acceptable property of our system
assuming that people have UPS set up in production and the problem is negligible for us
assuming that people use ephemeral disks from "the cloud" which disappear forever on failure anyway, and thus there's no corruption


Configure warnings

This PR does two things:

It makes version mismatch warnings silent by default
It enables warnings for unrecognized tags to be silenced

@nden, this seems like the right way to handle the large number of version mismatch warnings in jwst in the long term. However, I think that unrecognized tag warnings should be enabled by default, and there are a few other more serious warnings which are still not possible to silence at all.
Also, while this resolves #313 for the most part, it does not add a module-level configuration mechanism. There are too many questions about precedence that make it a little too complicated to implement without sufficient motivation. Please let me know if you disagree.

Coverage increased (+0.004%) to 94.595% when pulling 5aec3929374b50afd71f0d3ce846e56496df2cd4 on drdavella:configure-warnings into a23105aef15e604d825c7d99537fbbba818ba1e6 on spacetelescope:master.

@nden, despite the appearance of the diff, this is a pretty small change. I'd like to merge it if you have no objections. It should not cause any more warnings to be produced by the JWST pipeline.

@drdavella This looks good to me. I have one suggestion.
Would it be better to update *Type classes to include the supported_versions attribute you added in #284. I've added supported_versions to transform types and I don't see warnings any more. I think it's good to explicitly state which versions are supported and then warnings will be issued for those that are not.
If so, the default of ignore_version_mismatch can be set to False and let application choose how to deal with warnings. The reasoning is that sometimes the warning may point to a real problem.
If you agree, just set the default of  ignore_version_mismatch to False in this PR. Since I've added supported_versions to experiment I can make a PR for that.

@nden I've been trying to clarify my thoughts on this issue. I think I need to add some documentation that describes these changes and the philosophy behind them. In the meantime, here are my current thoughts (which maybe can eventually be converted into documentation in some form):
There are really two different stages at which warnings can occur:

schema validation, and
tag conversion

Also, tag classes now have two different attributes related to versioning:

version, and
supported_versions

There are also now two levels of warning control:

ignore_version_mismatch which is set to True by default, and
ignore_unrecognized_tag, which is set to False by default

The version attribute of a tag class describes the current (i.e. latest) version of that tag. It should correspond to the version of the latest available schema for that tag. For example, if we have a schema that defines foo-3.2.0, then the version attribute of the FooType class will be "3.2.0".
During schema validation, we detect mismatched tag versions. For example, if I read an ASDF file that references foo-3.0.0 or foo-3.3.0 but the latest schema is foo-3.2.0, then a version mismatch warning will occur. This has nothing to do with the tag conversion stage: this is strictly a statement about schema validation. This kind of warning is now controlled by ignore_version_mismatch, and it is currently ignored by default since it does not indicate a serious problem.
The supported_versions attribute is used to describe the versions of a tag that can successfully be translated by a particular tag class. By default, it is assumed that all versions of a particular tag can be converted by the corresponding tag class. However, this is not always true. For example, if foo-3.0.0 introduced backwards-incompatible changes, our tag class may only be able to convert version 3.0.0 and later. We can declare this by setting supported_versions=AsdfSpec('>=3.0.0').
During tag conversion, we determine whether a particular schema can successfully converted to a custom Python data structure by a particular tag implementation. Continuing with the example above, if we encounter foo-3.0.0, our supported_versions attribute tells us we can successfully convert it, so there is no warning, even though the latest version we know of is foo-3.2.0. However, if we encounter foo-2.7.1, we can't convert it, so there will be a warning. This warning can't be ignored at all.
The ignore_unrecognized_tag option controls warnings that occur when no corresponding tag class can be found for a particular tag. For example, if I read an ASDF file that makes reference to baz-1.2.0, but I don't have a BazType class, then this warning will occur. These warnings are not suppressed by default.
TL;DR: It is useful and a good idea to explicitly define supported_versions for tag classes (including transform types), but it is not strictly necessary in most cases. However, since supported_versions has no bearing on schema mismatch warnings, and since these warnings indicate a relatively minor issue, it seems reasonable to ignore them by default.

@nden reported that the use of supported_versions affected the behavior of YAML validation warnings, which is not what I intended. This has now been fixed and I believe that it is ready to be merged as soon as the tests pass.

Coverage increased (+0.3%) to 94.904% when pulling ff9ec0c4845cf442be8b69027ee75e681beb501a on drdavella:configure-warnings into a23105aef15e604d825c7d99537fbbba818ba1e6 on spacetelescope:master.

OK, so really the version_mismatch warning is of no interest to end users. I think this can merged. I'll open a separate issue about supported_versions as I see some potential issues.

I'm not sure it will never be of use to end users, but I think the warnings are of a sufficiently minor nature that they should be have to be explicitly enabled (maybe think of it like -Wall for a C compiler).


Can't refer to 'this in generic bounds?

use std::io::Read;
struct Foo<R: Read>(R);
struct Bar<'a, R: Read + 'a>(&'a Foo<R>);

#[ouroboros::self_referencing]
struct Test<R: Read + 'this> {
    foo: Foo<R>,
    #[borrows(foo)]
    #[covariant]
    bar: Bar<'this, R>,
}

I'm trying to define a selfreferencing structure where the referencing type needs a lifetime on its generic parameter, but adding 'this doesn't work - it reports it as an undeclared lifetime.
Am I going about this wrong? Is this possible to support?

Unfortunately, ouroboros does not support this use case. The primary
difficulty is that the generic parameter is used in places that aren't
self-referencing, such as the Heads struct returned by .into_heads().
Theoretically, this might be supportable if ouroboros can detect which
fields generic parameters are used in, but that is a level of complexity I
am not willing to add at the moment.
On Sat, Nov 27, 2021 at 5:30 PM Torne Wuff @.***> wrote:

use std::io::Read;struct Foo<R: Read>(R);struct Bar<'a, R: Read + 'a>(&'a Foo);
#[ouroboros::self_referencing]struct Test<R: Read + 'this> {
foo: Foo,
#[borrows(foo)]
#[covariant]
bar: Bar<'this, R>,
}
I'm trying to define a selfreferencing structure where the referencing
type needs a lifetime on its generic parameter, but adding 'this doesn't
work - it reports it as an undeclared lifetime.
Am I going about this wrong? Is this possible to support?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/joshua-maros/ouroboros/issues/50, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOL2YRLTOLDWHKIKOZQ3ZJ3UOGA2FANCNFSM5I4WXJXA
.

I needed that too to store an iterator. I kind of making a workaround with specific type
pub(super) type EnumeratedDepsLinsIter<'a> = Filter<
    Map<Enumerate<Lines<'a>>, fn((usize, &str)) -> (usize, &str)>,
    fn(&(usize, &str)) -> bool
>;

and something like this
        // The closures are coerced to function pointers so that their types could be named.
        //
        // Quting the reference, "A closure expression produces a closure value with a unique,
        // anonymous type that cannot be written out.".
        //
        // Source: https://doc.rust-lang.org/reference/types/closure.html
        //
        // The type of the whole iterator is needed so that ouroboros crate could handle the
        // field of the type of the iterator.
        let trim_lines: fn((usize, &str)) -> (usize, &str) = |(i, line)| (i, line.trim());
        let is_trimmed_dep_line: fn(&(usize, &str)) -> bool = |(_i, line)| line.starts_with(":dep");

but it's ugly.

damn I was looking for this exactly.  I needed to store a context and a codegen struct I am making for llvm.  since if the context gets dropped it would create an issue for generated modules being invalidated.  guess i'll have to look else where.

Hi, is there any updates? Thanks!

The primary
difficulty is that the generic parameter is used in places that aren't
self-referencing, such as the Heads struct returned by .into_heads().

Then is it possible that, when using Bar<'this>, the Heads features are disabled? In my scenario (hoping to use this in https://github.com/fzyzcjy/flutter_rust_bridge), I only need to borrow the bar, and no need for functions like into_heads.


HDDS-2972. Any container replication error can terminate SCM service.

What changes were proposed in this pull request?
Catch the exception thrown in processing container under class ReplicationManage.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2972
How was this patch tested?
Will be added.

Thanks @elek for the review and merge!


Initial setup: Consolidate building setup to CMake

This adds a new utils/create-build-files script that can be used, via cmake, to produce build setups meant to replace build.py and the Xcode workspace and projects to build swift-corelibs-foundation without building the rest of the Swift codebase.
This isn’t fully done, but allows the Foundation and CoreFoundation targets to build in a generated Xcode (utils/create-build-files --xcode) or Ninja setup (utils/create-build-files --ninja).

@swift-ci please test

cc @gottesmm @compnerd — this is a first step in getting rid of the legacy build system.

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


FS-2390: Fix compatible html elements

Change description
For lists in html, we should ensure if we have alternative elements they're encapsulated in a list item.  This change does that on two instances that were raised by axe dev tools.  Some translations weren't working because they included % signs, I've fixed them as well while I was in the area.
How to test
Checkout the branch, re-start your docker container to ensure the new compiled translations are picked up.  Then visit the all questions page for cof r2w3, you can observe the fixed translations.  You can also run the axe analysis using the extension I linked above.
Screenshots of axe analysis

Note: This also included pre-commit-ci changes, which fixed issues throughout the entire codebase, hopefully we'll only have these once per repo.

You probably already tried it but rather than split translations would it make more sense to use the escaped version of %? So Can provide 50% of the total. Not sure if it would work though...

I thought about this but didn't try it! I'll give it a shot.

@srh-sloan Genius! Thanks for that recommendation, have added in https://github.com/communitiesuk/funding-service-design-frontend/commit/6c87693a47d940f9dd2403df71342d1b22da9833


Fibaro FGRGBWM441 RGB controller no color wheel

I dunno if this is the right place but apparently this controller is only partially supported. Instead of getting a color whel I get a light switch for each color. What do you need to debug or where do  I have to raise this issue?

additional info: I can control the color with the color switch which is not exposed to home assistant

Please raise issues that are not specific to this library in the Home Assistant core repository. You can help us by creating a dump of your network and attaching it as a text file in the issue.

@thundergreen i did it for you, this is the right place for fibaro devices when using the new zwave js integration in hass https://github.com/home-assistant/core/issues/46138


Bad conditional formatting

if (impl(ghc >= 9.2)&& impl(ghc < 9.3))

There should be a space before &&.

Discovered here: https://github.com/haskell/haskell-language-server/pull/4229#discussion_r1599311876

Rather than my janky Chunk type, I think it makes sense to use the proper Condition ConfVar for this.
https://hackage.haskell.org/package/Cabal-syntax-<IP_ADDRESS>/docs/Distribution-Fields-ConfVar.html#v:parseConditionConfVar
It will require a bit of a rework to parse the arguments for if (and elif when cabal-version: >= 2.2), but I think it's worth it.

Unfortunately the Condition type doesn't have a CParen constructor. That means it can't losslessly round-trip a conditional like this:
if !((x || y) && z)

The AST will be correct, but simply rendering it will give you this:
if !x || y && z

That could potentially be fixed by using a showsPrec-style renderer. However sometimes users insert parentheses that aren't strictly necessary. I think Gild should retain those. So I'll probably need to write my own Condition type, along with a parser and pretty printer.
https://github.com/haskell/cabal/blob/4072eb8d658a250af35ae45c65da77f59eca3a5e/doc/cabal-package-description-file.rst#conditional-blocks

The various ConfVar types also have problems for me. Both OS and Arch have the concept of "strictness". The parser picks a strictness for me, so I can't make it permissive (unless it happens to already be that way). This is a problem because they have aliases. For example, AArch64 is the actual constructor, but it can allow an arm64 alias. I would prefer to keep whatever the user gave without translation. If that's not possible, I'd prefer to be as lax as possible, meaning allowing all aliases and converting them into their canonical form.
(Also they're case-insensitive, but that's not controlled by the strictness.)
https://hackage.haskell.org/package/Cabal-syntax-<IP_ADDRESS>/docs/Distribution-System.html#t:ClassificationStrictness


[ZEPPELIN-4104] Zeppelin Interpreter On Docker

What is this PR for?
Zeppelin service runs on local server. zeppelin is able to run the interpreter in the docker container, Isolating the operating environment of the interpreter through the docker container. Zeppelin can be easily used without having to install python, spark, etc. on the local node.
Design Doc : https://docs.google.com/document/d/1ESQEHmGlw4kaXkK5Hn2AOJYBLWDZRU9shB3WHuGV-CM/edit
Key benefits are

Interpreter environment isolating
Not need to install python lib, spark binary package, etc. environment on the local node
Docker does not need to pre-install zeppelin binary package, Automatically upload local zeppelin interpreter lib files to the container
Automatically upload local configuration files (such as spark-conf, hadoop-conf-dir, keytab file, ...) to the container, so that the running environment in the container is exactly the same as the local.
Zeppelin server runs locally, making it easier to manage and maintain
Support zeppelin all interpreter
Support spark interpreter, local[*], cluster-client, cluster-yarn mode.

What type of PR is it?
[Feature]
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4104

How should this be tested?

CI Pass

Screenshots
Sample1 : Spark(yarn-cluster mode) interpreter run docker

Sample2 : Spark(yarn-cluster mode) interpreter run docker

Sample3 : Shell interpreter run docker

Questions:

Does the licenses files need update? NO
Is there breaking changes for older versions? No
Does this needs documentation? YES

@zjffdu , Please help me review code, Thanks!

@zjffdu , Have any more comments?

Will merge if no more comments


:bug: Fix: bad package.json2

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

#1254  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


SQLCipher support

What version of Flyway are you using?
4.1.2
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Command-line
What database are you using (type & version)?
SQLCipher-3.15.2 (SQLite-3.15.2)
What operating system are you using?
WIndows 10 X64
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
flyway -url=jdbc:sqlite:data-template.db -locations=filesystem:.db/migration -password=testkey migrate
What did you expect to see?
Migrate successfully.
What did you see instead?

Flyway 4.1.2 by Boxfuse
Database user:
Database: jdbc:sqlite:.data-template.db (SQLite 3.0)
ERROR:
Unable to check whether table "main"."schema_version" exists
SQL State  : null
Error Code : 26
Message    : [SQLITE_NOTADB]  File opened that is not a database file (file is encrypted or is not a database)

Closing due to lack of interest.

Are there any plans for the SqlCipher support in the near future?


CoreNLP 3.8.0

Hi, CoreNLP is now at version 3.8.0 (https://stanfordnlp.github.io/CoreNLP/other-languages.html), can I just download this new version and will the interface work with it?

Yes, it should be compatible. Not tested extensively.


Action recognition model & Kinetics400 dataset

This PR contains the dataset, TCN model, training and evaluation codes for action recognition using the Kinetics400 dataset.
I also made little fix to train.py, because it would unnecessarily load the training set in --evaluate mode. This was a rather important efficiency issue with 40GB of training videos.

Moved folding outside dataset codes (as in https://github.com/MaximIntegratedAI/ai8x-training/pull/206)
Provided a "silent fail" case which would happen if ffmpeg was not installed in the system. Now, it raises a RuntimeError with instructions on how to install FFmpeg.

since FFmpeg is not a python package, I don't think we can integrate

Seems good to me.


Wrong folder used for output files with fpocket [debian:11-slim]

Describe the bug
fpocket try store files in / where ever is the input file
To Reproduce

In any folder, for example /tmp/Z
put a pdb file in the folder, for example z.pdb
from /tmp/Z, launch fpocket: fpocket -f z.pdb
Error:
***** POCKET HUNTING BEGINS *****
! The file /z_VMD.sh could not be opened!
! The file /z_PYMOL.sh could not be opened!
! The file /z_out.pdb could not be opened!
Segmentation fault (core dumped)

Expected behavior
No crash and output file stored where the pdb file is stored
Desktop (please complete the following information):

OS: Debian
Version 11-slim

Source of problem:
in src/fpout.c, the function write_out_fpocket use 2 times sprintf with destination and source using same memory buffer (out_path) and as the manual explain:
C99 and POSIX.1-2001 specify that the results are undefined if a call to sprintf(), snprintf(), vsprintf(), or vsnprintf() would cause copying to take place between objects that overlap (e.g., if the target string array and one of the supplied input arguments refer to the same buffer).
Using a temporary buffer, seems fix the issue at least in fpocket called only with -f

oh interesting one, that might be because you are working in /tmp, will check that out.

Looking into this now. Cannot reproduce the error at all. However I guess I identified the line (88) you refer to @odemengeon :
      sprintf(out_path, "%s/%s", out_path, pdb_code);

adressed & merged in different PR -> already on current master


Table Filter doesn't work with calculated columns

Table filter doesn't work if column value is calculated base on other columns
  <CDataTable
     items={publishers}
     fields={fields}
     scopedSlots={{
            name: (item) => (
              <td>{`${item.lastName} ${item.firstName}`}</td>
            ),
  }}/>

Do you have any other options how to make filter work with calculated columns?

The table is filtered by content passed in items prop. You can map publishers variable, it would look something like:
const mappedPublishers = publishers.map(publisher => {...publisher, name: publisher.lastName + ' ' publisher.firstName})


Add Tailwind 2’s peer dependencies to preset

Follow up to #40
Tailwind 2 has two peer deps:

PostCSS 8
Autoprefixer 10

In reality, only the missing PostCSS dep will cause building to fail, but I thought I should just add both to be safe.

I think that maybe the proper solution to the increase in Node requirement is to also modify the engines property in package.json. This would require a new method on the Preset class.


Handling a graph with global property

❓ Questions & Help
Hi!
I'm wondering whether I can transform graph data with global property in Data object.
For instance, the meta layer implemented in 'torch_geometric.nn' employs graph level property to process graph data. However, I can't find a way for packing the global properties of graphs in Data object.
thanks :+1:

Simply add your global data properties to your data object, e.g.:
data.u = torch.randn(1, 64)

That's awesome!

Simply add your global data properties to your data object, e.g.:
data.u = torch.randn(1, 64)

Is these approach fully compatible with the custom dataset?

Simply add your global data properties to your data object, e.g.:
data.u = torch.randn(1, 64)

Is these approach fully compatible with the custom dataset?

self-reply: It seems so

Yes :)


Crop coordinates

Can I get the crop coordinates (x and y position plus area size) ? I want to use the coordinates so that next time user wants to crop the same image again to position his cropping area right on the old crop.

I share the same question here!
I'd like to store the crop coordinates, but still upload the entire image + coordinates.

I have this doubt too....I've the same problem...I need store only coordinates...

Hi, I'm also interested in this functionality.

Folks, I've solved the issue right after asking the question. It's as straightforward as using (in HTML)
<img-crop image="f  | ngfDataUrl" area-coords="myCoords"  result-image="myCroppedImage"></img-crop>

and you will have the coordinates available in $scope.myCoords

Great!! its ready to download?


Connect directly to terminal without import kubeconfig

Hello, guys!
Hope you have a good day !
I have the wonder like how to access the terminal without manually importing the kubeconfig file.
The big idea is i already have some kubeconfig file in the local OS and the webkubectl will import these file and auto create some session and my job is simply click on connect and access into the cluster ! ( For example i have a lot of kubeconfig file and what if i import manually. It will take a lot of time)

For example i have a lot of kubeconfig file and what if i import manually. It will take a lot of time

Ok...
I think batch upload will solve your problem.

Yeah i got your point but it doesn't solved my problem because what if i have maybe 300 or 400 kubeconfig file (example). Yeah actually i can do it step by step and one by one but i just want to upgrade our job like everything will be automation. I have the directory and in this directory will have 400 kubeconfig file (example) i will create multi container and each container will manage some session (maybe 50 or 100 session). And each container will mount some kubeconfig file in some directory i defined so that i can access into the session without import manually

@liqiang-fit2cloud @wan92hen did anyone have any recommendation please

Okay !! Thank you very much


Portainer don't start on docker For Windows 18.03 stable

BUG REPORT INFORMATION
Description
Just updated Docker for Windows to last 18.03 stable (stable channel and no experimental feature). All was working great on the previous version.
Starting the container bring the following error:
ERROR: for portainer Cannot create container for service portainer : b'Mount denied:\nThe source path "\var\run\docker.sock:/var/run/docker.sock"\nis not a valid Windows path' ERROR: Encountered errors while bringing up the project.
I should add it seems the new docker release change some way on security or path structure, because same problem arise for nginx-proxy too which use the docker socket. It doesn't start on our stack too, with the same error than portainer.
For reference, some user begin to have this problem: https://stackoverflow.com/questions/49507912/docker-jwilder-nginx-proxy-container-create-issue
I don't know if portainer need to be updated or this is docker wich need to be fixed.
Steps to reproduce the issue:
A simple compose file like:
version: "3.5"

services:
    portainer:
        image: portainer/portainer:latest
        volumes:
            - "portainer:/data"
            - "/var/run/docker.sock:/var/run/docker.sock"

Technical details:

Portainer version: latest
Target Docker version (the host/cluster you manage): Docker for Windows to last 18.03 stable (stable channel and no experimental feature)
Platform (windows/linux): Windows 10 Profesionnal up-to-date
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer): see compose file above please
Target Swarm version (if applicable): not a swarm node
Browser: in the cli (try several cli, same problem)

Thanks

This error seems to be related to Docker and not to Portainer.

ERROR: for portainer Cannot create container for service portainer : b'Mount denied:\nThe source path "\var\run\docker.sock:/var/run/docker.sock"\nis not a valid Windows path' ERROR: Encountered errors while bringing up the project.

I'll close this issue, you'll need to wait for an update in https://github.com/docker/for-win/issues/1889 for the solution to this issue.


Kanban board for recipe-app

In this issue the following were completed.

Create kanban board for the recipe app
assign tasks to each group member

Team Composition
there are three group members in this project

@BiancaMe
@chimwemwe007
@vikas924

link to Kanban board:
https://github.com/users/BiancaMe/projects/4

Hi @chimwemwe007  @BiancaMe  @vikas924 👋🏼👋🏼👋🏼,

Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights✅✅✅✅

[x] Link to project has been provided👍🏼.
[x] Currently, points are fairly distributed👍🏼.
[x] Tasks are properly assigned👍🏼.
[x] The required columns are in the kanban board👍🏼.

Required Changes ♻

[ ] Kindly ensure you add a description for each card as this is a requirement in this milestone👍🏼. Currently, your unit test cards don't have any descriptions. please try to ensure all cards have a description👍🏼.

[ ] Some of the requirements are still not taken into account in the Kanban board😉. Kindly ensure all the requirements of the project that concern your group are captured in the Kanban board👍🏼. Currently, you have captured the n+1 requirement and the recipe details requirement for a group with 3 members👍🏼. Lastly, remember to ensure the points are fairly distributed when you take these changes into account.

Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.

Hi @chimwemwe007 , @BiancaMe and @vikas924   ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉 🎉 🎉
Highlights
✔  Implemented the changes requested to add missing tasks and ensure all tasks have a description
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


feat: Support weight streaming

Description

Weight streaming feature is exposed as compiler option to set percent or weight streaming bytes
Create a network with kSTRONGLY_TYPED and set kWEIGHT_STREAMING to builder config
Same dtypes are required for layers in strongly typed network

Fixes # (issue)
Type of change
Please delete options that are not relevant and/or add your own.

New feature (non-breaking change which adds functionality)
This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project (You can use the linters)
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[ ] New and existing unit tests pass locally with my changes
[ ] I have added the relevant labels to my PR in so that relevant reviewers are notified

We probably need to think about what the user flow is here:
So @ compile-time:

Users tell us to make an engine that is weight streamable

@ runtime

How do we set up the engine as a default? should there be a default weight budget?
User now wants to explicitly set the engine weight budgets

How can they do this from the module level? What happens if there are multiple engines in the graph?
We need to recreate the execution context, imo this needs to be done atomically with this call to keep it out of the forward function. i.e. as part of set_weight_budget we recreate the execution context.

Hi @narendasan

When only compiler option is provided,

automatic weight streaming budget is applied. test case

Weight stream size is set manually

torchtrt.runtime.weight_streaming_context() runtime api is added to get streamable size and set budget. test case

Context recreation

added decorator in set_weight_streaming_budget.

multiple subgraph

I'm thinking of applying normalized size if multiple module is in runtime module. impl. Too small value will have bad impact. I will test various size.

Model device budget:

Include pytorch subgraph weights as well in the device budget (count parameters and estimate size)
Conclusion is estimate new budget by subtracting pytorch submodule sizes. If the pytorch weights size is greater than the user provided budget, error out and inform user.

User API:

Enable weight streaming flag 2) device budget

User provides: total_budget for model
trt_engine_budget = total_budget - sum([p.numel * p.element_size() for p in model.parameters()])
Then we distribute trt_engine_budget proportionally to all engines
Error if trt_engine_budget is <=0 or if any one engine's allocation is less than its min_device_budget

No pending items. I think this PR can be merged.


Enable app container to bring a Fuseki dataset (managed by fuseki container) online and offline

I want to enable the app container to bring a Fuseki dataset (in fuseki container) online and offline. When a dataset is online, the nmdc-tdb2 make target (which people run from within the app container) fails with an error saying that it could not obtain a lock on a file.
I have an idea for this, which I'll document in this issue.
CC: @turbomam

The app container can access the fuseki container via the hostname, fuseki
Fuseki has an HTTP API that includes an endpoint clients can use to bring a dataset online/offline

It is documented here: https://jena.apache.org/documentation/fuseki2/fuseki-server-protocol.html#active-and-offline
Access to that endpoint is restricted

Here's the command I expected would put the nmdc-tdb2 dataset into an "offline" state; but it doesn't seem to me to do so:
curl --user 'admin:password' 'http://fuseki:3030/$/datasets/nmdc-tdb2?state=offline'

The following curl command seems to work, for putting the nmdc-tdb2 dataset into an "offline" state!
curl -H "Authorization: Basic $(echo -n admin:password | base64)" -H 'Content-Type: application/x-www-form-urlencoded; charset=UTF-8' --data 'state=offline' 'http://fuseki:3030/$/datasets/nmdc-tdb2'

It's based on a code snippet I stumbled upon in this stain/jena-docker PR from 2019.
Here's the corresponding one for putting the dataset back into an "active" state:
curl -H "Authorization: Basic $(echo -n admin:password | base64)" -H 'Content-Type: application/x-www-form-urlencoded; charset=UTF-8' --data 'state=active' 'http://fuseki:3030/$/datasets/nmdc-tdb2'

Based on what I described in the previous message, here are the two commands we can issue from the app container, to toggle the state of the nmdc-tdb2 dataset between "active" and "offline":
curl --user 'admin:password' --data 'state=active' 'http://fuseki:3030/$/datasets/nmdc-tdb2'
curl --user 'admin:password' --data 'state=offline' 'http://fuseki:3030/$/datasets/nmdc-tdb2'

Based on what I wrote in the previous message, here's a diff for project.Makefile:
diff --git a/project.Makefile b/project.Makefile
index 6e65da0cc..7ac9fb0ad 100644
--- a/project.Makefile
+++ b/project.Makefile
@@ -637,10 +637,21 @@ local/nmdc-sty-11-aygzgv51-tdb: local/nmdc-schema-v8.0.0.owl.ttl local/nmdc-sty-
 # Note: We expect people to run this make target from within the `app` container.
 #
 nmdc-tdb2:
+       echo "Putting dataset into 'offline' state."
+       curl \
+               --user 'admin:password' \
+               --data 'state=offline' \
+               'http://fuseki:3030/$/datasets/nmdc-tdb2'
+       echo "Populating dataset."
        tdb2.tdbloader \
                --loc=/nmdc-schema/local/fuseki-data/databases/$@ \
                --graph=https://w3id.org/nmdc/nmdc \
                /nmdc-schema/project/owl/nmdc.owl.ttl
+       echo "Putting dataset into 'active' state."
+       curl \
+               --user 'admin:password' \
+               --data 'state=active' \
+               'http://fuseki:3030/$/datasets/nmdc-tdb2'

 .PHONY: filtered-status
 filtered-status:

FYI, @turbomam, in case you want to try this out before I work on this next.

@eecavanna this is excellent. Thanks.


WSL cannot start Windows Store exe files

WSL is unable to start windows executables that have been created by windows store apps e.g.
C:\Users\Klaus\AppData\Local\Microsoft\WindowsApps\EmEditor.exe
strace:
ubuntu@PC-001:~$ strace EmEditor.exe
execve("/mnt/c/Users/Klaus/AppData/Local/Microsoft/WindowsApps/EmEditor.exe", ["EmEditor.exe"], 0x7fffc9aa51d0 /* 18 vars */) = -1 ENOEXEC (Exec format error)
fstat(2, {st_mode=S_IFCHR|0660, st_rdev=makedev(4, 1), ...}) = 0
ioctl(2, TCGETS, {B38400 opost isig icanon echo ...}) = 0
write(2, "strace: exec: Exec format error\n", 32strace: exec: Exec format error
) = 32
getpid()                                = 67
exit_group(1)                           = ?
+++ exited with 1 +++

This may occur because these files are empty and are a special kind of symbolic link (at least I guess that's how it works)

This is a duplicate of #4969.

/dupe #4969


Start ReScript with -w if build.watch is set

Sometimes you'd like to just start the rollup watcher without the dev server with vite build --watch. This change makes it so that the ReScript compiler also starts in watch mode if build.watch is set to true (or a truthy value).

Awesome! I wasn't aware of vite bild --watch.
Thanks for contribution!

This PR has been released in v1.1.3.

Thanks for the quick response and release!


PARTITION BY vs ORDER BY

I was studying docs for a while but can't really figure out, how PARTITION BY and ORDER BY impact  performance exactly of MergeTree.
I managed to figure out that ORDER BY specifies primary key of the table and therefore the index so when filtering using those columns in PREWHERE and WHERE it can leverage the index.
PARTITION BY says how to merge and allocate partitions. But how to choose the proper partitioning? What is the rule there? Also how to determine whether to specify only some DateTime partitioning key or include other keys as well? The only information I managed to find is not to make partitions to much granular ("This means that you shouldn't make overly granular partitions (more than about a thousand partitions), or SELECT will perform poorly").
And what is the role of filtering by datetime in all this?
I made following assumptions:

In ORDER BY I should specify columns that I plan to usually filter by. This also means more columns more disk space occupied. But the search is faster then.
PARTITION BY says how things are merged together so I should probably set it so it merges data that usually go together. (?) Let's say I usually want to find data for certain month in certain country so partitioning by month and country would make sense. Or is it better to partition by month only? Why would I want PARTITION BY toMonday()?
Filtering by datetime ... (?) Does it play any role in PARTITION/ORDER BY settings?

Can anyone make it more clear please? Are those assumption correct? Maybe we can extend the documentation after that.
Thanks!

PARTITION BY - declare how to split data in table to a parts which can be manipulated separately. Partitions should be nor too small nor too big. Usually it's months / week / days - that allow you to drop old partitions, or do other manipulation with partition level. Clickhouse allows you to drop / optimize / freeze / detach / attach parttions, clear certain columns in that. I.e. partition is kind of element in clickhouse, you can do some manipulation with.
When you use a condition which allows to read less partitions - that will be used. I.e. if you have 12 monthly partitions and you use condition where event_date =  '2018-01-01' only one partition, containing that date will be read. If you will not provide extra conditions on primary key - whole partition will be read.
Inside each partition data is physically sorted according to your primary key, and there is an index on that. If you will provide conditions containing fields used in primary key - clickhouse will read only some range of rows inside that partition which contain primary key you provided.
In all other case fullscans are used.


How to run a gitlab-ci runner in rancher?

Rancher Version:
1.1.0.dev3
Docker Version:
1.11.1
OS and where are the hosts located? (cloud, bare metal, etc):
One bare-metal Ubuntu 16.04
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
see above
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
I have a standard gitlab-ce container and a gitlab-ci-runner container running in rancher.
A load balancer routes requests to http://gitlab to the gitlab container.
I want to setup gitlab-ci runners but when I register the runner and want to build something, the docker container launched by the runner cannot resolve the gitlab host.
How can I make the containers launched by the gitlab-ci-runner see my gitlab container in rancher?
configuration:
helloworld:
  labels:
    io.rancher.container.pull_image: always
  tty: true
  hostname: flask
  image: mnoethe/flask-helloworld
  stdin_open: true
LoadBalancerDockerCloud:
  ports:
  - 80:80
  labels:
    io.rancher.loadbalancer.target.helloworld: flask-app:80=5000
    io.rancher.loadbalancer.target.gitlab: gitlab:80=80
  tty: true
  image: rancher/load-balancer-service
  links:
  - gitlab:gitlab
  - helloworld:helloworld
  stdin_open: true
gitlab:
  domainname: gitlab
  labels:
    io.rancher.container.pull_image: always
  tty: true
  hostname: gitlab
  image: gitlab/gitlab-ce:latest
  volumes:
  - /home/max/gitlab-test/config:/etc/gitlab
  - /home/max/gitlab-test/data:/var/opt/gitlab
  - /home/max/gitlab-test/logs:/var/log/gitlab
  stdin_open: true
docker-ci-runner:
  labels:
    io.rancher.container.pull_image: always
  tty: true
  image: gitlab/gitlab-runner:latest
  links:
  - gitlab:gitlab
  volumes:
  - /home/max/gitlab-test/gitlab-runner:/etc/gitlab-runner
  - /var/run/docker.sock:/var/run/docker.sock
  stdin_open: true

And the rancher-compose:

helloworld:
  scale: 1
LoadBalancerDockerCloud:
  scale: 1
  load_balancer_config:
    haproxy_config: {}
  health_check:
    port: 42
    interval: 2000
    unhealthy_threshold: 3
    healthy_threshold: 2
    response_timeout: 2000
gitlab:
  scale: 1
docker-ci-runner:
  scale: 1

You need to configure the runner to register itself. So it appears it's less about the name resolution and more that your container needs to register? I've built a runner that has a few extra bells and whistles, but you can see how it takes some environment variables to register when it starts. That would help with scaling up, if you ever want to do that.
https://github.com/BradJonesLLC/gitlab-docker-ci

The runner works fine, the containers spawned by the runner to do the actual ci jobs cannot resolv the gitlab domain.

@MaxNoe Ah, if you are trying to use the internal GitLab DNS services, you will need to spawn them with the appropriate label, so they are configured for the Rancher network. See http://docs.rancher.com/rancher/latest/en/concepts/#networking

Another question: Is it possible for rancher to manage the gitlab-ci jobs (distribute them equally on the cluster?) Right now, all jobs are starting on the host where the gitlab-ci service is running.

The short answer is, not really. I'm sure you could script something, and the provider => consumer model that's part of #2179 could probably help, but not out of the box.

@MaxNoe Please re-open if you still have further questions. It looks like @bradjones1 has done a great job helping you out. :)

@MaxNoe have you solved this? I'm trying to get the runner working with the regular hosted gitlab.com edition, but when I register it nothing happens

Also how do you configure your runners?

I did nothing special to solve, just registered the runner and it worked since or server is now reachable from the web. The problem arised only in my test environment.  I'm using the docker backend for the runners.

Is this something that could qualify to be part of the Rancher catalog next to the already existing Gitlab setup?

@MaxNoe well in my case after I register it nothing happens, gitlab shows it doesn't connect to gitlab, but after you give it access to the docker socket, you won't see docker instances in rancher right? Since they're managed on docker side

@alex88 Rancher shows all running docker images, also those not created or managed by it. So you should see the containers spawned by the gitlab-ci-runner.
What is missing is the possibility to load balance those containers to all machines in the rancher cluster, not only to the machine where the gitlab-ci-runner service is running.

@MaxNoe what I think gitlab should implements is the ability to just run a container with a daemon that runs the builds in it,  instead of relying on the docker socket on something that you register/unregister

The solution is to set the host inside of extra_hosts. e.g. extra_hosts = ['gitlab:<IP_ADDRESS>']
https://docs.gitlab.com/runner/configuration/advanced-configuration.html#the-runners-docker-section
https://docs.gitlab.com/runner/configuration/advanced-configuration.html#how-clone_url-works should also works.


Azure Sphere Linux OS Github

I was wondering if the Linux OS/Kernel is up on Github?
I saw a presentation where Microsoft stated it is open source and they count on contribution.
I can however not find it?

Hi - please take a look at Third party disclosures and source code, and filter on 'Azure Sphere'.

There is no Azure Sphere repo there:(

Hi @mifurm - the source isn't currently available directly here on GitHub, but it is available for download at the link above. Please let us know if you have any trouble accessing it. Thanks.

@viktor-morin We hope your issue has been addressed.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


COI: Add name of tabs

AC

[ ] Add the name of the tab in the form for Employee Acknowledgements and Employee Disclosure Statement tabs

Example

Ready for test

Both tabs have been created

@ayushdamani Can you add a space here below the heading

space added

Space has been added - moving to PO

looks good


text field style fix

RTL support introduced a regression on TextFields with custom children, which we use extensively.

@oliviertassinari this commit was removed from #1845

@chrismcv Thanks, I will have a close look at this on in priority (regression) tomorrow.

@chrismcv Thanks for this. Indeed the RTL introduce a breaking change by moving a style property.

Unfortunately, this fix breaks styles for DropDownMenu in SelectField component (and propably other usages as well). Style property SelectField sets on its internal DropDownMenu is ignored and overriden by TextField own input styles which results in SelectField being several pixels taller than TextField and breaking layout where both are used side-by-side.
More, styles of TextField and its child are not actually merged as the implementation suggests - child style should be referenced as this.props.children.props.style instead of current this.props.children.style.
This problem is also root cause of #2163.


Deploy 1.1.0-snapshot.3 x509: certificate signed by unknown authority

Describe the bug
deploy bookinfo example failed in version 1.1.0-snapshot.3
I deploy by kubectl apply -f install/kubernetes/istio-demo.yaml
But when create bookinfo by kubectl apply -f samples/bookinfo/platform/kube/bookinfo.yaml, there are no pods up.
By checking kube-apiserver log, I find this
ATCH /api/v1/namespaces/default/events/details-v1-6d7ccf8ffc.156d541453321efd: (2.651089ms) 200 [hyperkube/v1.12.0 (linux/amd64) kubernetes/59ab39c/system:serviceaccount:kube-system:replicaset-controller [::1]:47352]
W1205 12:05:36.811036   81858 dispatcher.go:72] Failed calling webhook, failing closed sidecar-injector.istio.io: failed calling webhook "sidecar-injector.istio.io": Post https://istio-sidecar-injector.istio-system.svc:443/inject?timeout=30s: x509: certificate signed by unknown authority
I1205 12:05:36.811499   81858 wrap.go:47] POST /api/v1/namespaces/default/pods: (168.099701ms) 500

Resolved， I updated my host dns config.


Add setSilent() method to URI class

Description
Using $uri->setSilent() method will disable throwing exceptions. May be handy for using with methods like getSegment().
Ref: #2952, #2949
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

Oh great... now it not mergeable... I wonder what went wrong.

I don't know how to resolve this, so I'm closing this PR in favor of a new one: #3005


Add support for database files with CRLF line endings

Fixes chjj/node-tiny#16

4 years down the line I don't assume this PR will get merged...


Implement penalize_validator helper methods for block processing.

PR Description
Implement penalize_validator helper method for block processing.
Fixed Issue(s)
Not sure there's one specific issue for this PR, but it contributes to getting block processing updated to spec v0.1.

Thanks, issues you've commented on have been fixed.


allow codeblocks to optionally collapse+expand or be scrollable

Adds a mode attribute. mode="collapse" collapses the code block with a fade you expand. mode="scroll" sets the max-height and lets the user scroll.
Fixes #6

Very cool


Cannot setup CFT development environment - make docker_test_prepare

Hi, while following the instructions in "5.1 Set environment variables and create test project" - https://codelabs.developers.google.com/codelabs/cft-onboarding/#3 I am encountering the following two issues after running make docker_test_prepare:
First: a permission error: Error: Error reading Organization Not Found : <MY_ORG_NUMBER_ID>: googleapi: Error 403: The caller does not have permission, forbidden

To try to fix this, I made my GCP account and the Service account both Org Admin and still I saw the same proble.

Second: a warning on google_project_services deprecation.

I could not determine if the error and the warning are related or they are two separate issues.

Thank you
 $ make docker_test_prepare
docker run --rm -it \
        -e SERVICE_ACCOUNT_JSON \
        -e TF_VAR_org_id \
        -e TF_VAR_folder_id \
        -e TF_VAR_billing_account \
        -v /home/cleonardo/tech_labs_crostini/src/terraform-google-cloud-storage:/workspace \
        gcr.io/cloud-foundation-cicd/cft/developer-tools:0.10.0 \
        /usr/local/bin/execute_with_credentials.sh prepare_environment
Activated service account credentials for: [<MY_SERVICE_ACCOUNT>]
Activated service account credentials for: [<MY_SERVICE_ACCOUNT>]
Initializing modules...

Initializing the backend...

Initializing provider plugins...

The following providers do not have any version constraints in configuration,
so the latest version was installed.

To prevent automatic upgrades to new major versions that may contain breaking
changes, it is recommended to add version = "..." constraints to the
corresponding provider blocks in configuration, with the constraint strings
suggested below.

* provider.google-beta: version = "~> 3.21"
* provider.null: version = "~> 2.1"
* provider.random: version = "~> 2.2"

Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.
module.project.module.gsuite_group.data.google_organization.org[0]: Refreshing state...

Warning: google_project_services is deprecated - many users reported issues with dependent services that were not resolvable.  Please use google_project_service or the https://github.com/terraform-google-modules/terraform-google-project-factory/tree/master/modules/project_services module.  It's recommended that you use a provider version of 2.13.0 or higher when you migrate so that requests are batched to the API, reducing the request rate. This resource will be removed in version 3.0.0.

  on .terraform/modules/project/terraform-google-project-factory-3.3.1/modules/core_project_factory/main.tf line 165, in resource "google_project_services" "project_services_authority":
 165: resource "google_project_services" "project_services_authority" {

Error: Error reading Organization Not Found : <MY_ORG_NUMBER_ID>: googleapi: Error 403: The caller does not have permission, forbidden

  on .terraform/modules/project/terraform-google-project-factory-3.3.1/modules/gsuite_group/main.tf line 26, in data "google_organization" "org":
  26: data "google_organization" "org" {

Makefile:34: recipe for target 'docker_test_prepare' failed
make: *** [docker_test_prepare] Error 1

I've just rewritten that section to make the steps clearer. In particular, make sure your service account also has Organization Viewer access.
Can you give it a try and see if it works?
The deprecation warning is unrelated and can be ignored.

Works perfectly now.
Thank you Morgante!


Switching to a feeless exchange

Yeah... Saving money on a feeless exchange? What do you think?

@kylerchin
https://github.com/DunnCreativeSS/pytrader
Switched all the core functions to Liquid.com (0 fees). The Get Deposits function doesn't exist on their API and I've had mixed luck with the balances... but I haven't had any luck with the learning algorithms spitting anything out after 1000 prices.


Add --validate-db to validate all on-disk database files

This optional flag enables validation of all ImmutableDB epoch files, instead of only the most recent one, which is the default.
Note that we can't yet detect all kinds of corruption: we can detect truncation, but not random bitflips. We're working on that, see https://github.com/input-output-hk/ouroboros-network/issues/1253.

bors r+


Bookmarks List doesn't get updated when a session is unbookmarked from BookmarksFragment

Initially

Actual Behaviour
Session is unmarked but it still remains in the list.

Expected Behaviour
Session should be removed from the list.

Steps to reproduce it
Bookmark a session from Tracks Fragment.
Go to Bookmarks Fragment and unmark it.
The session still remains in the list (unmarked).
Can I work on this ?

I have submitted PR #1098
@mariobehling @the-dagger Please review

@the-dagger can i work on this issue.

Ok
On Saturday, February 25, 2017, Amit Tiwary<EMAIL_ADDRESS>wrote:

@the-dagger https://github.com/the-dagger can i work on this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/1081#issuecomment-282478054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQtlmo9WrIR9RuCS2uiX6W_9__Jf4uFks5rgBBWgaJpZM4L3esy
.

--
Best,
Harshit Dwivedi

@the-dagger @amitiwary999, @TejasBhitle has already submitted the PR and there is some merge conflict which as soon as he resolves can be merged! In my opinion, will it be wiser to do the same work again?

i think so the pr for this issue have been submittted. @harshitagupta30  there is no point to do the same work again.

@arp95 I suppose I am also saying the same thing! :+1:

@the-dagger Merge conflict has been resolved in the PR #1098


On 3.0, TlsStore CRD seems to not work

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I was using this block to make Traefik serve a certificate stored in the cloudflare-tls secret by default for all tls routers, without having to specify the secret in all ingresses or copy the origin certificate to a dozen namespaces.
apiVersion: traefik.containo.us/v1alpha1
kind: TLSStore
metadata:
  name: default
  namespace: kube-system
spec:
  defaultCertificate:
    secretName: cloudflare-tls

I'm seeing lots of changes to 3.0 in the docs, but nothing that would suggest this would break. The relevant section is unchanged, but that may be just because it hasn't being changed yet
What did you see instead?
Upon upgrade to 3.0, updating my CRDs (including this one) to traefik.io ... it won't work. I also tried using the old CRDs to no avail. EDIT: see below post
What version of Traefik are you using?
docker.io/traefik:3.0-beta2
What is your environment & configuration?
values.yaml
    image:
      tag: "3.0"

    experimental:
      v3:
        enabled: true

    deployment:
      enabled: true
      kind: DaemonSet

    updateStrategy:
      type: OnDelete

    hostNetwork: true
    dnsPolicy: ClusterFirstWithHostNet
    affinity:
      podAntiAffinity:
        requiredDuringSchedulingIgnoredDuringExecution:
          - labelSelector:
              matchExpressions:
                - key: app.kubernetes.io/name
                  operator: In
                  values:
                    - traefik
            topologyKey: kubernetes.io/hostname

    ports:
      metrics:
        # When using hostNetwork, use another port to avoid conflict with node exporter:
        # https://github.com/prometheus/prometheus/wiki/Default-port-allocations
        port: 9101
      web:
        port: 80
        redirectTo: websecure
      websecure:
        port: 443

    # Not very useful under HostNetwork
    service:
      enabled: false

    logs:
      general:
        level: DEBUG

    tlsStore:
      default:
        defaultCertificate:
          secretName: cloudflare-tls

    securityContext:
      capabilities:
        drop: [ALL]
        add: [NET_BIND_SERVICE]
      readOnlyRootFilesystem: true
      runAsGroup: 0
      runAsNonRoot: false
      runAsUser: 0

If applicable, please paste the log output in DEBUG level
2023-05-23T23:36:35Z DBG github.com/traefik/traefik/v2/pkg/server/configurationwatcher.go:217 > Configuration received config={"http":{},"tcp":{},"udp":{}} providerName=kubernetes
2023-05-23T23:36:35Z DBG github.com/traefik/traefik/v2/pkg/tls/tlsmanager.go:294 > No default certificate, fallback to the internal generated certificate tlsStoreName=default

Note that it works if the IngressRoute specifies the secret name, despite technically being across namespaces and in theory not being able to access it (I'm... fine with that)

@mloiseleur
This is what I mean
If the obsolete CRDs for v3 is left as valid CRDs, there will be plenty of these kinds of bug reports.

so which one I should really use in order to get certs to working? I am trying v3.0 tag and using traefik.io crds. However, it will not work. So should I now manually remove traefik.io crds and install traefik.containo.us crds?

blah finally, it was needed to delete crd once and create it again.

@davhdavh my problem back then was that Traefik seemed to monitor the old CRDs and not the new ones for this particular resource.
I just had to manage this server again for a different breakage and it randomly stopped working now too and no amount of applying the tlsoption to any CRD is having it working again. I know there is a very specific set of steps that cause it to work, but there's something definitely weird happening.

Yes, after downgrading the helmchart to our previous version (23.0.1) I can run Traefik v3 with the TLSOptions in the traefik.containo.us/v1alpha1 group.
I would say this is likely a bug in Traefik v3, they forgot to change TLSOptions to the new API?

@p4block FWIW I am able to consistently reproduce this issue using 26.0.0 and 26.1.0:
No default certificate, fallback to the internal generated certificate tlsStoreName=default
The presence of the deprecated CRDs seems to make no difference. I get the same result with both "flavors" of TLSStore present in the namespace, and with just a TLSStore from the traefik.containo.us API.
I did most of my testing using v3.0-rc3, but I encountered the same thing with v2.11.
Will try downgrading to an older version of the chart and see what shakes out.

@nmengin @p4block Did a lot of experimenting last night and discovered a potentially interesting behavior when deploying in a namespace
2024-03-29T05:33:58Z DBG github.com/traefik/traefik/v3/pkg/server/configurationwatcher.go:227 > Configuration received config={"http":{},"tcp":{},"udp":{}} providerName=kubernetes
**2024-03-29T05:33:58Z ERR github.com/traefik/traefik/v3/pkg/provider/kubernetes/crd/kubernetes.go:1057 > Secret traefik/scuzz-fm-tls does not exist TLSStore=default namespace=traefik providerName=kubernetescrd**
2024-03-29T05:33:58Z DBG github.com/traefik/traefik/v3/pkg/provider/kubernetes/ingress/kubernetes.go:177 > Skipping Kubernetes event kind *v1.Endpoints providerName=kubernetes
2024-03-29T05:33:58Z DBG github.com/traefik/traefik/v3/pkg/provider/kubernetes/crd/kubernetes.go:174 > Skipping Kubernetes event kind *v1.Endpoints providerName=kubernetescrd
2024-03-29T05:33:58Z DBG github.com/traefik/traefik/v3/pkg/tls/tlsmanager.go:321 > No default certificate, fallback to the internal generated certificate tlsStoreName=default

What this suggests to me is that tlsmanager (or something else in Traefik itself) successfully locates the secret, but for some reason cannot load it properly. Note that the k8sclient in crd/kubernetes.go does not return any errors when retrieving the certificate.
Would've tried debugging the Helm chart further, but I gave up and decided to configure the cert secret in the IngressRoute directly, which works without issue. I suspect this is a bug in Traefik, but don't know if I have the patience to dive into the source code more than I already have. Cheers.


Add "Disabled/Remove Notifications" under Neutron or Debloat?

Settings -> Notifications -> Off
I know this is posted in the wrong section but I must be a collaborator or something to post in "Pull Requests" section.
My co-workers and I both would love this addition added, just another little simple add-on to make setups clean and simple.
Hope you don't mind me posting here, just a suggestion! Love the work, always checking back every week or so for updates.
Oh, and DropBox program added to Neutron? unless you're still keeping it GS tools approved.
Love the auto update feature in the settings, too.

Hey @Bloopy442
Done
https://github.com/SkylerWallace/ATOM/commit/a2e0cc973413b27e3c427de35cc77ad7a3408541