ingress: prefer annotation over ingressClassName

[x] All code is covered by unit and/or runtime tests where feasible.

The Kubernetes API documentation says that the ingressClassName field "replaces the deprecated kubernetes.io/ingress.class annotation. For backwards compatibility, when that annotation is set, it must be given precedence over this field." Other Kubernetes software relies on this, for example cert-manager and the Hashicorp Vault Helm chart.
This patch makes the Cilium ingress controller conform to the spec, by preferring the annotation if it is set, and then falling back to ingressClassName.
Fixes: #22340
Signed-off-by: Nikhil Jha<EMAIL_ADDRESS>Cilium now prefers the `kubernetes.io/ingress.class` annotation over the `spec.ingressClassName` field when handling a new Ingress

The change looks okay to me, but we definitely need some tests here, I think.

Yep! I made the PR early but was planning on adding tests (left that box unchecked). I just added a copy of the basic Ingress test, but with the annotation rather than ingressClassName, which should be sufficient(?)

Maybe a test that the annotation value overrides the field correctly? The current test makes sense, but we should test that extra type of behavior.

This should probably be needs-backport/1.13 as well, but I can't add tags 😁

@nikhiljha Typically, if we label a PR with release-note/bug, then we'd need to consider how we might want the release note to be read by users. Currently, the PR title will be taken unless there's a release-note  in the description, see step 12 in docs. As it stands, the PR title is not readable enough to users, i.e. it doesn't describe what the impact of the prior behavior is and what this PR fixes. This is so that users can read the changelog and understand what has been fixed in newer versions.
What's not clear to me is did the behavior prior to this PR exist in an already released version of Cilium (besides the RC)? If so, then I'd agree with release-note/bug, otherwise release-note/misc would be more appropriate.
cc @youngnick

FWIW, I think that not supporting the annotation per the spec previously was kind of a bug. But either release note category is fine with me.

Do you think it's worth backporting this? If it's more of a bug then it might be nice to at least get into 1.13.

Is there anything else needed on this one? I just rebased the changes on latest master branch.

Looks like there's some failing tests, and I think this will need another rebase, but I think there's all the required approvals now. Maybe @joestringer can confirm there?

Yep, looks like all of the required review approvals are in, it's mainly just testing that is blocking this PR. I'm not familiar with the latest state of the tests, but if the ingress conformance test is failing then there's a fair likelihood that the changes are related to that failure.

Ingress tests passed. Merging


feat: extended images for custom CSS properties

This is a new feature that allows users to extend
images by setting custom CSS properties.
The properties are written with an inline YAML map
(like {width: 50%, height: 400px}) placed after the image.
Only certain properties are allowed to avoid any potential abuse.
The extended images work in both preview and publishing.

#1273
Pull Request Checklist
You can go to dendron pull requests to see full details for items in this checklist.
General
Quality Assurance

[x] add a test for the new feature
[x] make sure all the existing tests pass
[x] do a spot check by running your feature with our test workspace
[ ] after you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: if you running mac/linux, check the windows output and vice versa if you are developing on windows

Special Cases

[ ] if your tests changes an existing snaphot, make sure that snapshots are updated
[ ] if you are adding a new language feature (graphically visible in vscode/preview/publishing), make sure that it is included in test-workspace. We use this to manually inspect new changes and for auto regression testiing

Docs

[ ] Make sure that the PR title follows our commit style
[ ] Please summarize the feature or impact in 1-2 lines in the PR description
[ ] If your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Example PR Description
# feat: capitalize all foos

This changes capitalizes all occurences of `foo` to `Foo`

Docs PR: <URL_TO_DOCS_PR>

Special Cases
First Time PR

[ ] sign the CLA which will be prompted by our github bot after you submit the PR
[ ] add your discord alias in the review so that we can give you the horticulturalist badge in our community

Analytics

[ ] if you are adding analytics related changes, make sure the Telemetry docs are updated

Since this is a language syntax change, we should learn from existing implementations and adapt for our use case. Specifically, common mark has a standard for defining attributes as well as one for image-size.
Note for the below, we currently have limited support for nunjucks templating and are planning on expanding it. This will make the { { } } syntax hard to support since it interfereces with nunjucks variable interpolation.
Personally not a fan of the proposed image spec[^1] in common mark. We could extend the { } syntax to be what commonmark is thinking about for applying general CSS styles {{}}.
So long aside to only come back to what you've had. This is mostly a note for future changes - when we make markdown changes, those should be captured in an RFC and some prior work should be examined.

common mark css styling proposal  [^2]

Styling
-------

Since HTML is such a popular output format and it is almost always used with CSS for styling, there is a special syntax to include inline CSS rules.

```````````````````````````````````````````````` example
{ {property: value} }
.

- sizing proposal [^1]
```markdown

Width and Height
----------------

+ Also see: [Image Size extension](image-size)

*Not final yet!*

```example
{ =400*300 }
{ 400*300 }
{ *300 }
{ 400* }
{ 400:300 }
.

[^1]: <https://github.com/Crissov/CommonMark/blob/extensions/extensions/image-size.txt>
[^2]: <https://github.com/Crissov/CommonMark/blob/extensions/extensions/attribute.txt>


Extend environment() function to expose Azure RBAC roles

Is your feature request related to a problem? Please describe.
Feature reaquest.
Describe the solution you'd like
I would like to be able to fetch rbac role Ids using a function. Example usage:
Instead of:
@description('List of all Azure RBAC roles')
import {
  Roles
} from 'roletypes.bicep'

...

roleDefinitionIds: [
  Roles.Owner
]

I would like to:
roleDefinitionIds: [
  environment().roles.Owner
]

Closing as duplicate of #1895


Fix WEBGL-only selection and throw for unknown renderer type

This PR

Adds a new feature
Fixes a bug

Fixes #5583
Fixed: When the device does not support WebGL, creating a game with { type: Phaser.WEBGL } will fail with an error. It will not fall back to CANVAS as AUTO does.
New: Creating a game with an invalid renderer type will throw an error. Before it would be treated like AUTO.

👍


onRendered doesn't get current template instance as argument

onCreated and onDestroyed receive the current template instance as argument, so stuff like this is convenient:
onCreated(instance) {
   instance.subscribe('docs');
}

Is there a reason, why onRendered doesn't receive the template instance as argument? In Blaze it's bound as this in all three life cycle functions, so it would make sense to have a similar behavior across all of them in ViewModel as well?

I have no idea what happened then. Probably some silly human error. I just tried this in a random component and indeed it works like you said. Sorry for the trouble, closing.


Set max_open_files for large clusters

The ansible cookbook defaults to setting max_open_files to 128k. Users of the Chef cookbook can say node['ceph']['config']['global']['max open files'] = 131072, but perhaps this should be the default?
It seems like the setting defaults to whatever ulimit is set to.

I don't think our cookbook must override any default application setting.
It's too complicated to maintain.
If someone want to share some good experience, he can do a cookbook wrapper :)


STAGE 1 TRAINING

Thank you for your great work!
In the stage 1 training mentioned in the paper, is the input of llm images and text，because the description ‘After the pretraining stage, the model is capable of generating images for single text descriptions’  in the article is unimodal?
And,What does ‘The Language Model (LLM) is then tasked with only generating these placeholders for text creation’ mean? Does the stage 1 llm only need to generate placeholders without other token embedding?

The unimodal here just means we only learn texts to image alignment in stage 1. I hope this figure can help you out.


Nextjs/Babel 7.10

i had a few problems running the tests in a nextjs project.
using the @babel/polyfill module with the .babelrc config described in the README.md file, causes the next app to crash.
a solution that worked for me is describet at https://stackoverflow.com/questions/33527653/babel-6-regeneratorruntime-is-not-defined/53736090#53736090
in short, if you have the same issue do the following:
npm install --save-dev @babel/core @babel/preset-env @babel/register @babel/preset-react

npm install --save @babel/runtime
npm install --save-dev @babel/plugin-transform-runtime

Your test script in package.json:
"test": "riteway -r @babel/register 'src/**/*-test.js' | tap-nirvana ",

and then in your .babelrc
{
  "presets": [
    [
      "@babel/preset-env",
      {
        "targets": [
          "last 2 versions",
          "safari >= 7"
        ]
      }
    ],
    "@babel/preset-react"
  ],
  "plugins": [
    [
      "@babel/transform-runtime"
    ]
  ]
}

This should be fixed.


Fix purge yum cache

The "purge yum cache" task that is in purge-cluster.yml doesn't work as designed.  yum clean all will only clean caches for installed repos.  Since purge-cluster.yml removes the repo before purging the cache, it does not remove the Ceph repo cache.  I tested putting the task before the yum erase that removes the repo, but yum rebuilds the cache just before removing the repo.  The only way to purge the yum cache for the Ceph repo is to move it to the install.yml so it purges the cache just after installing the repo but before it uses it.
I verified this worked correctly by purging and reinstalling several different versions of ceph on the same hardware.  Without the change the install will fail trying to install a chached version of ceph rpm's that don't exist in the currently installed repo.

Thanks @jsaintrocc


Add proto validation for RuleType

Summary
Ref https://github.com/stacklok/minder-stories/issues/94
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 54.759% (-0.01%) from 54.77%
when pulling 0de26946cd8005b07ac438040d3cc3dc0dbe7496 on validate-ruletype
into 67bd38d9a35e26a19041c1084a4f0be91c06ea82 on main.


[dev] print info logs to stderr and real logs to stdout

so users can pipe their logs reasonably. should also think about other ways to structure dev's log output

closing as duplicate of #1283


Changes Attraction Strength serialization

Description
Changes how the AttractionStrength CV gets serialized, now using xmltodict to convert to/from OpenMM's XML serializer.

Codecov Report

Merging #29 (72491c6) into main (3345c4b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files


fix: Running as root issue

Fixes https://github.com/schultek/jaspr/issues/310

@schultek any chance we could get this merged :smile: ?

The fix was already released from another branch, but is now on master.


Add support for PATH_INFO server variable

Hi there,
Can you please provide support for the PATH_INFO server variable on IIS. I'm trying to migrate my legacy MVC website to the new one I've built ASP.Net Core. I'm trying to migrate the URL rewrite rules but they don't work as they rely on the PATH_INFO server variable.
I get the following error when trying to run my application:
Unrecognized parameter type: 'PATH_INFO', terminated at string index: '10'
Microsoft.AspNetCore.Rewrite.Internal.IISUrlRewrite.ServerVariables.FindServerVariable(string serverVariable, ParserContext context, UriMatchPart uriMatchPart)
Extract from my configure method in startup.cs which reads in an xml file:
using (StreamReader iisUrlRewriteStreamReader =
    File.OpenText("IISUrlRewrite.xml"))
            {
                var options = new RewriteOptions()
                    .AddIISUrlRewrite(iisUrlRewriteStreamReader);

                app.UseRewriter(options);
            }

The example content of the IISUrlRewrite,xml file is:
<rewrite>
  <rules>
    <rule name="Redirect /foo to foo2" stopProcessing="true">
      <match url=".*" />
      <conditions>
        <add input="{PATH_INFO}" pattern="^/foo$" />
      </conditions>
      <action type="Redirect" url="http://www.foo2.co.uk" appendQueryString="false" />
    </rule>
  </rules>
</rewrite>

If I change {PATH_INFO} to {QUERY_STRING}, the application loads but this isn't the redirect I need. Can you please provide support for this ASAP please?
In the meantime, is there a workaround I can use that will support {PATH_INFO} patterns on IIS?
Thanks

I've reviewed the serverVariables file and can see that you support HTTP_URL. I should be able to change the rules to use this instead.

From what I can tell, HTTP_URL is supposed to include the query string, however after I reread the implementation, it seems that we didn't include the query string. Try swapping PATH_INFO with HTTP_URL?

I've been able to use HTTP_URL but will appreciate if you can update this issue when PATH_INFO is supported.


ISSUE [w/ Fix] : Bug with Masking causes Major Toggle Issue

There is a bug with the toggle that when certain elements become rendered on the screen, the masking becomes messed up.

This style is required to resolve the issue:
   -webkit-mask-image:  url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAIAAACQd1PeAAAAGXRFWHRTb2Z0d2FyZQBBZG9iZSBJbWFnZVJlYWR5ccllPAAAAA5JREFUeNpiYGBgAAgwAAAEAAGbA+oJAAAAAElFTkSuQmCC);

When this is added via inspector, it resolves the problem:

interesting @bradennapier - do you have simple code example so I can reproduce it?

I also got this for certain toggles, though I haven't been able to isolate why. It seems to happen specifically in Chrome; Firefox and Edge show fine. The fix also works when applied via JS:
const toggleFixStyle = {
  WebkitMaskImage: 'url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAIAAACQd1PeAAAAGXRFWHRTb2Z0d2FyZQBBZG9iZSBJbWFnZVJlYWR5ccllPAAAAA5JREFUeNpiYGBgAAgwAAAEAAGbA+oJAAAAAElFTkSuQmCC)'
};
...

  <Toggle /> {/* masking messed up */}
  <Toggle style={toggleFixStyle} /> {/* masking fixed */}

All right, it has to do with adjacent elements that have -webkit-backface-visibility: hidden. Here's a simple test case:

  <Toggle />

The toggle should have bugged masking. Removing -webkit-backface-visibility: hidden via the inspector fixes it.

Probably just a Chrome bug we'll have to deal with.

@rchanou are you still using it? does the bug still exist?

I don't know, I removed the offending -webkit-backface-visibility: hidden a while ago. Unless @bradennapier wants to chime in, I would say go ahead and close this because it appears to be a Chrome issue (which may be fixed by now), it's a rare case, and there's a documented workaround.


Create UnitTest project within solution

In order to test portions of the GLTFWrapper, create a unit test project within the solution.

Created PR for this #13
I plan to use this for testing mesh transformations and other GLTFWrapper components.

Merged.


  Connect to *address* (a 2-tuple ``(host,
2022-10-26T09:52:02.4921635Z         port)``) and return the socket object.  Passing the optional
2022-10-26T09:52:02.4922347Z         *timeout* parameter will set the timeout on the socket instance
2022-10-26T09:52:02.4923074Z         before attempting to connect.  If no *timeout* is supplied, the
2022-10-26T09:52:02.4923815Z         global default timeout setting returned by :func:`socket.getdefaulttimeout`
2022-10-26T09:52:02.4924598Z         is used.    Connect to *address* (a 2-tuple ``(host,
2022-10-26T09:52:02.4955386Z         port)``) and return the socket object.  Passing the optional
2022-10-26T09:52:02.4956097Z         *timeout* parameter will set the timeout on the socket instance
2022-10-26T09:52:02.4956820Z         before attempting to connect.  If no *timeout* is supplied, the
2022-10-26T09:52:02.4957605Z         global default timeout setting returned by :func:`socket.getdefaulttimeout`
2022-10-26T09:52:02.4958376Z         is used.  If *source_address* is set it must be a tuple of (host, port)
2022-10-26T09:52:02.4959180Z         for the socket to bind as a source address before making the connection.
2022-10-26T09:52:02.4959966Z         An host of '' or port 0 tells the OS to use the default.
2022-10-26T09:52:02.4960408Z         """
2022-10-26T09:52:02.4960743Z
2022-10-26T09:52:02.4961171Z         host, port = address
2022-10-26T09:52:02.4961608Z         if host.startswith("["):
2022-10-26T09:52:02.4962134Z             host = host.strip("[]")
2022-10-26T09:52:02.4962557Z         err = None
2022-10-26T09:52:02.4962899Z
2022-10-26T09:52:02.4963578Z         # Using the value from allowed_gai_family() in the context of getaddrinfo lets
2022-10-26T09:52:02.4964310Z         # us select whether to work with IPv4 DNS records, IPv6 records, or both.
2022-10-26T09:52:02.4965077Z         # The original create_connection function always returns all records.
2022-10-26T09:52:02.4965647Z         family = allowed_gai_family()
2022-10-26T09:52:02.4966022Z
2022-10-26T09:52:02.4966377Z         try:
2022-10-26T09:52:02.4966809Z             host.encode("idna")
2022-10-26T09:52:02.4967277Z         except UnicodeError:
2022-10-26T09:52:02.4967843Z             return six.raise_from(
2022-10-26T09:52:02.4968700Z                 LocationParseError(u"'%s', label empty or too long" % host), None
2022-10-26T09:52:02.4969125Z             )
2022-10-26T09:52:02.4969373Z
2022-10-26T09:52:02.4969957Z         for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
2022-10-26T09:52:02.4970580Z             af, socktype, proto, canonname, sa = res
2022-10-26T09:52:02.4971010Z             sock = None
2022-10-26T09:52:02.4971360Z             try:
2022-10-26T09:52:02.4971921Z                 sock = socket.socket(af, socktype, proto)
2022-10-26T09:52:02.4972272Z
2022-10-26T09:52:02.4972953Z                 # If provided, set socket level options before connecting.
2022-10-26T09:52:02.4973621Z                 _set_socket_options(sock, socket_options)
2022-10-26T09:52:02.4973965Z
2022-10-26T09:52:02.4974545Z                 if timeout is not socket._GLOBAL_DEFAULT_TIMEOUT:
2022-10-26T09:52:02.4975150Z                     sock.settimeout(timeout)
2022-10-26T09:52:02.4975635Z                 if source_address:
2022-10-26T09:52:02.4976202Z                     sock.bind(source_address)
2022-10-26T09:52:02.4976570Z >               sock.connect(sa)
2022-10-26T09:52:02.4977107Z E               ConnectionRefusedError: [WinError 10061] No connection could be made because the target machine actively refused it
2022-10-26T09:52:02.4977463Z
2022-10-26T09:52:02.4977880Z .tox\py3.9-win\lib\site-packages\urllib3\util\connection.py:85: ConnectionRefusedError

I encountered it as well, CI run: https://github.com/valory-xyz/open-autonomy/actions/runs/3330798733/jobs/5509904702
E               AssertionError: HTTPConnectionPool(host='localhost', port=26657): Max retries exceeded with url: /status (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x00000230B3E43A00>: Failed to establish a new connection: [WinError 10061] No connection could be made because the target machine actively refused it')

This has not been observed lately. - closing for now


Remove Is A Trap From Iodine

Added the function isValid which returns true/false depending whether there was an error or not.

Thanks for this! I'm making some changes to the readme and will then release.


Having trouble with TF tree

Hello there
Just a quick thanks for this great piece of software, it lets me do so much that I wouldnt otherwise!
I am trying to get kinect2_bridge working with a real world robot, alongside rtabmap. At the moment my tf tree looks like the attached view_frames output. I have publish_tf=true enabled in my launch file. I would like the published frames  to link with the shelf1_kinect2_rgb_optical_frame as this is where I have the kinect2 specified in my urdf. At the moment I cannot get the kinect2 output in RViz and rtabmap complains of lack of input, presumably because the tf's do not line up.
Do I need to add another line to my launch file to somehow link the two sets of frames?
frames.pdf
rtabmaptxt.txt
Thank you!
Oh I should add, when building my robot urdf I used your iai_robots/iai_kinect2_description/urdf/kinect2.urdf.xacro where shelf1 is my prefix and parent, which is what you see at left bottom of my tf tree.

The solution appears to be to add a static transform to the launch file. I got this from the rtabmap forums:
   <arg name="optical_rotate" value="0 0 0 -$(arg pi/2) 0 -$(arg pi/2)" />
   <node pkg="tf" type="static_transform_publisher" name="kinect2_base_link"
   args="$(arg optical_rotate) shelf1_kinect2_rgb_optical_frame kinect2_link 100" />


fixed write-good issues to improve readability of readme

I basically too https://github.com/btford/write-good and fixed most issues to try and improve the readability of the readme file, especially for non native speakers

Thanks!


Sdk automation script

For reference please checkout the generateOutput.json here

Hi @weshaggard, when I was creating this PR I thought I'll move this PR to azure-sdk-for-net repo once I'll do the cleanup but now I'm not able to change the base repo as Azure/azure-sdk-for-net in this PR, that's why I've to create a new PR in azure-sdk-for-net repo here; so that others can also take a look at it. I resolved all the comments in this PR. Hence closing this PR.
Please review and give further feedback here. Sorry for all the inconvenience.


fix: impresonation regression for service accounts

Fixes service account impresonation regression discussed in https://github.com/projectcapsule/capsule-proxy/issues/388#issuecomment-1970791462.

Yay! The checks are passing! :)

Welcome to Capsule and thank you a lot @rgruchalski-klarrio!

Thank you @rgruchalski-klarrio! LGTM

@rgruchalski-klarrio thanks for your contribution. Please fix the commit syntax. We have good documentation here: https://github.com/projectcapsule/capsule/blob/main/CONTRIBUTING.md#semantics

Also run make golint to get semantic passing for your code.

@maxgio92 what does this even mean?
/Users/radek/dev/golang/src/github.com/projectcapsule/capsule-proxy/bin/golangci-lint run -c .golangci.yml
internal/request/http_test.go:17: File is not `gci`-ed with --skip-generated -s standard -s default -s prefix(github.com/projectcapsule/capsule-proxy) (gci)
	"sigs.k8s.io/controller-runtime/pkg/client"

internal/request/http_test.go:20: File is not `gci`-ed with --skip-generated -s standard -s default -s prefix(github.com/projectcapsule/capsule-proxy) (gci)
	"k8s.io/apiserver/pkg/authentication/user"

Furthermore, how do you want me to tackle commit names? I'm very sorry but the bar to entry is a bit annoying.

@rgruchalski-klarrio That means your imports are not correctly ordered

Core imports
3-P imports
Project  local Imports

Okay, this seems to be a puzzle I really don't want to think about, what's the correct order?
import (
	"fmt"
	h "net/http"
	"regexp"
	"strings"

	authenticationv1 "k8s.io/api/authentication/v1"
	authorizationv1 "k8s.io/api/authorization/v1"
	"k8s.io/apiserver/pkg/authentication/serviceaccount"
	"k8s.io/apiserver/pkg/authentication/user"
	"sigs.k8s.io/controller-runtime/pkg/client"
)

They should look like this, not really a puzzle:
import (
	"context"
	"crypto/tls"
	"crypto/x509"
	"crypto/x509/pkix"
	"fmt"
	"net/http"
	"reflect"
	"regexp"
	"sort"
	"testing"

	authenticationv1 "k8s.io/api/authentication/v1"
	authorizationv1 "k8s.io/api/authorization/v1"
	"k8s.io/apiserver/pkg/authentication/serviceaccount"
	"k8s.io/apiserver/pkg/authentication/user"
	"sigs.k8s.io/controller-runtime/pkg/client"

	"github.com/projectcapsule/capsule-proxy/internal/request"
)

Ah, thank you, alphabetical order within a group. All resolved and the CI is passing.

@rgruchalski-klarrio We only accept GPG signed commits, please sign your commit

omg... yes, let me have a look at this once again for you.

There we go. Should be much better now.

It's not for me, no need to get upset ☺

nobody got upset, just extremely busy and the ping ping kinda annoyed me. Glad we're done with it!


[BUG] The docker container's ports are not being binded on the NodeJS docker template

Describe the bug
I did configure it to use the server port of 3000 and project it at 6000 but it wont work. Also any port I assign on my project works so basically it allows all ports for some reason.
To Reproduce
Steps to reproduce the behavior:
Just make a docker discord.js server and put on a port binding
Expected behavior
All ports should be blocked and the source port should be forwarded to the target port (mentioned in config)
Screenshots
N/A
Desktop (please complete the following information):

OS: Windows10
Browser Chrome
Version idk

Additional context
Couldnt find help on discord

as the release notes in discord show there is no support for windows


[SPARK-48328][BUILD] Upgrade Arrow to 16.1.0

What changes were proposed in this pull request?
The pr aims to upgrade Arrow from 16.0.0 to 16.1.0.
Why are the changes needed?
The full release notes: https://arrow.apache.org/release/16.1.0.html
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @dongjoon-hyun @LuciferYang


add -l option on get command which immediately look after get

ghq get -l motemen/ghq

which clones the repository and instantly look into the directory.
Also, this command line is reusable.
You never wonder if the repository has been cloned or not.

This close #84

Thanks!


Database: Refactor database actions for Instances

Depends on #10541.
Sorry for the size, there was a lot of collateral damage in this PR, which made for a lot of failed tests!
This is part of #10183, and refactors db.Instance into cluster.Instance and strips out fields that reference other tables: Profiles, Config, Devices. Using the v2 generator, these fields are accessible via cluster.GetInstanceConfig, and similar functions, which are generated in the cluster package.
To access a "complete" Instance object, there is InstanceToArgs and ToAPI, which convert a cluster.Instance into a type that automatically fills Profiles, Config, and Devices from the database.
Other changes

InstanceToArgs, and instance.Instance now hold the entire structs for the instance's list of profiles. To get the names we were querying the whole profiles table anyway, so this prevents us from having to do additional queries again every time we want to check the profile's config or devices.
InstanceArgs is now used as the instance type for InstanceList, as it has access to all the fields pertaining to an instance. As we are pre-fetching all instance profiles now, InstanceList can be simplified to just do the InstanceToArgs conversion, and fetch the projects.

This is slightly less efficient than before, since we are calling ToAPI on the same profile multiple times, but I chose to go with this simpler version to make the PR more easily digestible. I think once this is through, we can add a helper that creates a list of InstanceArgs in a more efficient manner.

api.Instance now has an api.URL method for getting its UsedBy url, rather than having this live in the db package.
Some now-unused functions / types are removed. A few more will be removed once Snapshots is refactored. That's ready to go once this is merged.

@tomponline I've addressed your comments: GetProfileIfEnabled is now GetProfilesIfEnabled, which absorbs the for loops that GetProfileIfEnabled was mostly called in and only checks the project for the profiles feature once. I didn't want to remove the helper entirely because I think it's helpful to have as a reference that ProjectHasProfiles should be checked when fetching profiles for a specific project.
Also added back the && !agent argument to the build constraints that I just removed :)

@tomponline I've addressed your comments: GetProfileIfEnabled is now GetProfilesIfEnabled, which absorbs the for loops that GetProfileIfEnabled was mostly called in and only checks the project for the profiles feature once. I didn't want to remove the helper entirely because I think it's helpful to have as a reference that ProjectHasProfiles should be checked when fetching profiles for a specific project.
Also added back the && !agent argument to the build constraints that I just removed :)

Thanks sounds good.

jenkins: test this please

jenkins: test this please

I think this is as good as we're going to get it given the issues with Jenkins runners in Canonical's Boston DC, there's a known network issue ATM.


next step button should (instantly?) finish the current animations before starting the next command

Done, there is one last bug: first animation cannot be skipped.


LPE: Don't use home as local origin

@jgoppert Any consequences for flight indoors to be expected?

@jgoppert we're still reseting the origin on arming I see. This isn't correct.

@mhkabir, thanks for pointing that out, will delete it. @ecmnet main change indoors is this reset part, 0,0 will be where you power it on, not where you arm it.


DHCP - Dynamic DNS Registration - DDNS hostname is not set in configuration with static config

When setting static host-names in dhcp4 configuration, the original client identifier is passed to the dns server, not respecting either the client identifier or the static configured host-name in the dhcp configuration
To Reproduce
Steps to reproduce the behavior:

DHCPv4 configuration
Setup DDNS to a remote DNS Server
Add a static entry for a mac address to add a forward and reverse map of the host name in DNS (Overriding the client-host-name)
The log shows the "original client identifier sent to the DNS server for forward and reverse map

Expected behavior
The static-host name or configured client-identifier should be used to set a ddns-hostname entry in the dhcp configuration files, maybe a feature to provide a checkmark to override client host-name. However, what would be the point of setting a static entry for hostname if that is not updated when ddns is configured set a forward/reverse map in DNS, ie
ddns-hostname "my-static-hostname";
Log
2020-11-11T02:11:15 | dhcpd[75584] | Added reverse map from <IP_ADDRESS>.in-addr.arpa. to HS200.skynet.net
-- | -- | --
2020-11-11T02:11:15 | dhcpd[75584] | Added new forward map from HS200.skynet.net to <IP_ADDRESS>

Configuration
subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
  pool {
    option domain-name-servers <IP_ADDRESS>;
    ddns-update-style interim;
    range <IP_ADDRESS> <IP_ADDRESS>;
  }

  option routers <IP_ADDRESS>;
  option domain-name "skynet.net";
  option domain-search "skynet.net","wifi.net,"dyn.net";
  ddns-domainname "skynet.net";
  option domain-name-servers <IP_ADDRESS>;
  default-lease-time 7200;
  max-lease-time 86400;
  option ntp-servers <IP_ADDRESS>;

}
host s_opt3_22 {
  hardware ethernet 50:c7:bf:ec:bc:87;
  option dhcp-client-identifier "hallway-light";
  fixed-address <IP_ADDRESS>;
  option host-name "hallway-light";
    ddns-update-style interim;
}

Environment
OPNsense 20.7.4-amd64

While I am not an expert in coding, I think if we add to the following file:
src/etc/inc/plugins.inc.d/dhcpd.inc  line 1094:
$dhcpdconf .= "  option host-name \"{$dhhostname}\";\n";
$dhcpdconf .= "  set hostname-override = config-option host-name;\n";
if (isset($sm['ddnsupdate'])) {
    $dhcpdconf .= "  option ddns-hostname \"{$dhhostname}\";\n";
}

and after line 1564:
$dhcpdv6conf .= "  option host-name {$dhhostname};\n";
$dhcpdv6conf .= "  set hostname-override = config-option host-name;\n";
if ($need_ddns_updates) {
    $dhcpdv6conf .= "  option ddns-hostname {$dhhostname};\n";
}

@fichtner I resolved the conflict on the merge for the patch to add the ddns-hostname commands.
Sort of related to this, I'm trying to update my fork of opnsense, however, it fails with 400 conflicts from master. Can I just delete my fork without impacting the ddns patch?

Slightly adjusted your submission as https://github.com/opnsense/core/commit/37eae60b221699959ec14f64487c2d065875255c, thanks!


Add more tests

Added tests for the hash_scripts file
Bumped min Coverage to 38

Codecov Report

Merging #85 (e4792db) into main (9c1a199) will increase coverage by 9.99%.
The diff coverage is 100.00%.

:exclamation: Current head e4792db differs from pull request most recent head 9a51105. Consider uploading reports for the commit 9a51105 to get more accurate results

@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
+ Coverage   28.43%   38.42%   +9.99%
==========================================
  Files          11       11
  Lines         204      203       -1
==========================================
+ Hits           58       78      +20
+ Misses        146      125      -21

Impacted Files
Coverage Δ

lib/src/util/hash_scripts.dart
96.07% <100.00%> (+41.36%)
:arrow_up:

lib/src/docker/run_image.dart
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

nice!


Interface to define custom directives

It would be nice to able to define additional directives that can be attached to fields to avoid defining repetitive custom resolution.  They would be useful for things like formatting values of fields in a different way then they are stored in the database, and for doing field specific permission checks.
Directives could be defined as functions that receive (value, obj, args, context) and return a replacement value or throw an error to fail a permission check.
for example we could define a directive to restrict access to certain fields:
export default types => ({
  name: 'restricted',
  arguments: {
    to: Types.string,
  },
  action: (value, object, fieldArgs, context, directiveArgs, type, field) => {
    if (
      object[directiveArgs.to] == null ||
      object[directiveArgs.to].id != context.session.currentUserId
    ) {
      throw `Unauthorized access to ${field.name} on ${type.name}, restricted to ${directiveArgs.to}.`;
    }
    return value;
  },
});

And another to translate text to a language stored in the session:
export default types => ({
  name: 'translate',
  action: (value, object, fieldArgs, context) => {
    return translateText(value, context.session.language);
  },
});

And then use them
type Post extends Node {
  id: ID!
  text: String! @translate
  viewCount: Int! @restricted(to: 'author')
  author: User @relationship(path: "<-AUTHORED-")
}
type Comment extends Node {
  id: ID!
  text: String! @translate
  votes: Int! @restricted(to: 'author')
  author: User @relationship(path: "<-AUTHORED-")
}

I want to make sure to think this through before actually adding it.  Some questions:

is this the best possible approach?
should directives also be able to modify the schema during the preprocessing step? (ie substitute types, etc..)
would it be better to use one directive that calls user defined functions by name to avoid possible namespace collisions?  IE: @filter(action "translate") instead of @translate

Having thought about this, I think it makes sense to handle these in javascript rather than the IDL.


All rst files must have a slide when autoslides=False

When the autoslides setting=False it's a problem that all rest files must then contain at least one slide. My use case is for a bunch of html notes to supports some lecture slides. It'd like to have only the lectures as slides, and the notes as html. This seems impossible currently however. The error is:
Exception occurred:
File "/Users/ben/Envs/lectures/lib/python2.7/site-packages/docutils/nodes.py", line 954, in replace_self
'Losing "%s" attribute: %s' % (att, self[att])
AssertionError: Losing "ids" attribute: ['mydocumenttitle']
Where mydocumenttitle is the top level heading of the page without a slide.

That sort of makes sense given where Hieroglyph lives in the Sphinx
processing pipeline. I'd have to dig a little deeper, but I think this is
going to be a little gnarly to fix: the "right" thing to do is along the
lines of detecting that we're in this state (autoslides off), and then only
processing files that contain a slide. That'd only work, then, when set in
the project config (as opposed to in the per-file header).
On Thu, Feb 19, 2015 at 1:42 PM, puterleat<EMAIL_ADDRESS>wrote:

When the autoslides setting=False it's a problem that all rest files must
then contain at least one slide. My use case is for a bunch of html notes
to supports some lecture slides. It'd like to have only the lectures as
slides, and the notes as html. This seems impossible currently however. The
error is:
Exception occurred:
File
"/Users/ben/Envs/lectures/lib/python2.7/site-packages/docutils/nodes.py",
line 954, in replace_self
'Losing "%s" attribute: %s' % (att, self[att])
AssertionError: Losing "ids" attribute: ['mydocumenttitle']
Where mydocumenttitle is the top level heading of the page without a slide.
—
Reply to this email directly or view it on GitHub
https://github.com/nyergler/hieroglyph/issues/86.

I think it's more reasonable - or at least explicable - that it fails when you set autoslides to off within a file but you don't explicitly provide a slide to process.  Fixing the case where autoslides is set in conf.py  would be a great first step at least.
I'm not an expert on docutils, but isn't it possible to read a flag set in the file from the hieroglyph processor?

The issue is that by the time Sphinx has processed an RST file into a docutils tree, I think it's too late to bail out and remove it from the set. And it's definitely possible to hook into Sphinx and modify the list of files to process, but that happens well before the files are parsed. So there's something of a chicken/egg problem here: you want look at some data from the parse, but you want to look at it before Sphinx has parsed the files.
It's probably worth digging into Sphinx to see if there's a call we can make to drop a file from the set after parse, or at least after it's read.

Fixing this is/seems simple. There are two ways (used) to replace self
node.replace_self(something)                  # This may raise an error; as described
# OR
node.parent.replace(node, something)          # Will NOT raise an error

The latter typically calls the former.
Both constructs are used in hieroglyph; but I don't know the reason for that. I was wonderring about replacing the second by the first, as part of a code cleanup (in my branch, to fix other items). But walked into the same error; so I didn't.
Given the request; it possible to work around the raised error. However, i'm a but reluctant as I have study the consequences yet: what does it functionally do/change?
Do you have an opinion on that @nyergler?

My recollection is that there were cases where the node didn't necessarily have a parent. I seem to recall that docutils did something slightly different in the two cases when there wasn't a parent. That said, in this particular case there'd always be a parent node, so probably doesn't matter.
Hieroglyph has better test coverage than it used to, so it might be interesting to see if replacing it works.

I'm having trouble with the autoslides option. If it is set False in config.py, the hieroglyph build fails, apparently in any document with a toctree:
Extension error (hieroglyph.directives):
Handler <function process_slideconf_nodes at 0x7f6a2b4621f0> for event 'doctree-resolved' threw an exception (exception: Losing "ids" attribute: ['tier-i-to-tier-ii-data-extraction'])
make: *** [Makefile:35: slides] Error 2
A work-around is to turn on autoslides for those totcree  documents, but that can lead to problems with toctrees longer than the slide (which can again be manually overcome with a next_slide directly).
Or, use 2 different configs as mentioned by @NatalieZelenka.
A native fix for autoslides/toctree would be ideal.


Prevent setting net.netfilter.nf_conntrack_max in case kube-proxy is deployed.

How to categorize this PR?

/area networking
/area robustness
/kind enhancement
What this PR does / why we need it:
Prevent setting net.netfilter.nf_conntrack_max in case kube-proxy is deployed.
Previously, the kernel setting net.netfilter.nf_conntrack_max was always set during node initialisation. In case kube-proxy was deployed to the nodes it also set the same kernel setting. However, the static initialisation by Gardener used a static value, but kube-proxy calculated a value based on the number cpu cores.
To reduce the permissions of kube-proxy, an init container was introduced, which set the kernel settings. However, the ordinary container also tries to do the same thing in case the value is not as expected. This can lead to an endless crash loop. In case the node initialisation is performed multiple times as can happen with gardener node agent.
A simple solution to the problem is to only set kernel setting net.netfilter.nf_conntrack_max in the node initialisation if kube-proxy is not enabled.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Kernel setting net.netfilter.nf_conntrack_max is only set on nodes by sysctl.d if kube-proxy is not running.

/cherry-pick release-v1.101

/cherry-pick release-v1.100

/cherry-pick release-v1.102


Type check in scripts ignores #r placement

Not very common case, I know, but IDE should show some error here:

It's a feature ;)

Yes, this is by-design.  The language service analyzes scripts with a whole-file view, as if they are executed all-at-once.  It's just how it is, we don't attempt to scope the referenced assemblies, it would just be really hard to do that.


Automated currency rate updater

This diff introduces a script to automatically update currency rates file from HRMC goverment data.
It still needs to be run manually but it could be integrated either in the CI automation or directly live in the software with the fallback on the offline file.

Implemented by https://github.com/KapJI/capital-gains-calculator/pull/157.


Octree; 'NoneType' object has no attribute 'device'

When I try to build an OCTree, it returns the following error.
oc_tree = Octree(depth=8, full_depth=2)
oc_tree.xyzb(8)

'NoneType' object has no attribute 'device'

The oc_tree has the following type
ocnn.octree.octree.Octree

As far as I understand, we have to build oc_tree after calling the class and the only method that works fine for this matter is  build_octree(point) Cause in any case we have to feed the octree with the points.
But there is a conflict between the size when applying the voxel with built octree
oc_tree = ocnn.octree.Octree(depth=8)
oc_tree.build_octree(point)
ocnn.nn.octree2voxel(tensor_cloud, oc_tree, 8)

RuntimeError: shape mismatch: value tensor of shape [841348, 3] cannot be broadcast to indexing result of shape [109320, 3]

please refer to the example files in this folder https://github.com/octree-nn/ocnn-pytorch/blob/master/projects/datasets to figure out the usage of build_octree

Thanks, But I could not find related examples to build_octree, they used the Point class only.
I think we can relate this issue to #5

Please first run the provided examples following the tutorials like Classification, Segmentation


feat(paeckchen-cli): add debug cli flag

belongs to #67

Changes Unknown when pulling 33d98dc54ea26fce7674e19dd03ce3ed2b1cec67 on cli/debug into ** on master**.

Coverage remained the same at 100.0% when pulling 33d98dc54ea26fce7674e19dd03ce3ed2b1cec67 on cli/debug into 19c7d65dda6a929791f4a599bc92fb5857d4bbbd on master.


fix(client): add a workaround for "MalformedPOSTRequest"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 12 (16.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 83.259%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tensorbay/client/segment.py
2
12
16.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
5565

Relevant Lines:
6684

💛  - Coveralls


Unable to open file (php-debug XDebug)

After starting the debugging i get the following error message:
Unable to open 'C:\XAMPP\htdocs\testpath\C:\XAMPP\htdocs\testpath\test.php'
EINVAL, open 'C:\XAMPP\htdocs\testpath\C:\XAMPP\htdocs\testpath\test.php'
I assume the program wants to open the file via Browser? So the correct path would be localhost/testpath/test.php.
I'm sorry but i'm an absolute beginner in configuring atom and the php debug package.
-webdev9

I'm facing same issue, can't see any mention of this in the docs as far as how to configure it. Atom itself already has the files open.


Feature/429 incomplete status code

Respond with a 404 when a metadata record cannot be exported because of insufficient metadata (fixes #429).
Rename "No ISBN provided" errors to "Missing ISBN", as "provided" could be understood as a client error

Looks good. "Missing ISBN(s)" feels a bit ambiguous to me still and maybe "No ISBN(s) found" would be better, but I'm on the fence.

How about a longer version like "This work does not have any ISBN(s)"?

Looks good. "Missing ISBN(s)" feels a bit ambiguous to me still and maybe "No ISBN(s) found" would be better, but I'm on the fence.

How about a longer version like "This work does not have any ISBN(s)"?

That's good, although the Google Books logic only looks for certain ISBNs, so the error could come up even if the work did have some ISBNs. Perhaps "This work does not have any ISBNs" for Crossref and "This work does not have a PDF, EPUB or paperback ISBN" for Google Books?


Optimize the deleteAll() method in EntityManager

The deleteAll method in EntityManager is not optimized as it loops over every entity and sends a delete request to the Datastore. It would probably run faster if we do the deletes in batches (e.g. delete a batch of hundred).

Updated the method to do the deletions in chunks of 100. Also modified the method signature to return a long (delete count) instead of void. This should still be backward compatible unless there are custom implementations of the interface, which is highly unlikely.


Add json functions to .sq dsl

This adds support for json1 extension functions to the sqldelight-compiler

Whoops. Meant to open this on our fork


Move check for tags once resource completed

Checking for tags after the check for the existence of the resource arn may cause flaky tests where the tags aren't created/applied yet. Tests have failed with tags returning as empty even though after checking that resource later on console tags do exist.
This pr moves this check for after the resource has reached created/completed status which should provide sufficient time for tags to be created/applied.

/test sagemaker-kind-e2e

/test sagemaker-kind-e2e

/approve
/lgtm

/lgtm cancel

/test sagemaker-kind-e2e

/approve

/lgtm

/test sagemaker-kind-e2e


weight or src at simple_net?

I think I found an error at line:89 of examples/simple_net.cpp.
Maybe it should be conv_prim_desc.weights_primitive_desc(), not conv_prim_desc.src_primitive_desc().
What's weird is that both can pass the test, why?

Hi @tensor-tang,
Thanks for pointing to this issue. You are right, this is a typo indeed.
The test passes for two reasons:

reorder doesn't check consistency of input and output memory
there is no actual check of the obtained result, so even the result is garbage the example is care...

Will fix this soon. Thanks!


GEODE-3131: Replaced Thread.sleep

* Replaced sleeps with an Awaitility clause which waits for the region sizes to be correct before starting validation.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Potential Reviewers:
@ladyVader @boglesby @jhuynh1 @upthewaterspout @gesterzhou @DivineEnder

Thank you @ladyVader .
Fixed the typo


Looping epsilon in PrivateReader gives UnknownTypeException error

Setting epsilon using values from a list in the PrivateReader throws an UnknownTypeException error. The code works when manually trying to add the values. Epsilon values were generated using np.linspace(0.1, 1, 11)

Can you try this instead:
private_reader_censor = PrivateReader(reader_censored, meta, privacy=Privacy(epsilon=float(eps_list_item, delta=float(delta))

The problem here is that the eps_list_item is a numpy type, not a native python float type. I'll push a patch that fixes this in the OpenDP library.

works now. thanks!

Fixed in opendp https://github.com/opendp/opendp/pull/394


Find In Files broken

Find In Files behavior is not predictable. If I have multiple files open which I know contains the search term, not all matching files will be shown.

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:07:25.676Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

unique-lines
bib
1.0.0

xml
Dot
1.9.2

Ionide-fsharp
Ion
2.33.2

vscode-insertdatestring
jsy
2.1.0

csharp
ms-
1.12.1

PowerShell
ms-
1.4.1

vscode-sql-beautify
sen
0.0.4

Steps to Reproduce:

Open multiple individual files which you know contains a search term.
Perform a Find In Files for the search term.
In my case, one matching file was displayed in the results.
Select the tab of another file - doesn't matter if this file matches or not.
Run the search again (hit Enter in the search field or use the Refresh icon)
This time multiple matching files are displayed.

Reproduces without extensions: Yes, except this time zero matches were found, then one, then two, depending on which file tab was selected.

Can you give an example of a file and query that sometimes matches, and sometimes doesn't? And is it consistent, like always matching or never matching in the same scenario?
It might make a difference whether the file is open or not open, since search is implemented differently for the two cases.

I can't give actual code but I was able to reproduce the issue on 4 identical files with dummy text (attached). All 4 files are open when I perform a search for "nisi" - only 2 results were returned (screenshot attached). When I selected a different tab and refreshed the search, all 4 results were returned (2nd screenshot attached.) Repeating this process gives different results, sometimes 3 results are shown for example.
lorem1.zip
Thanks

Do you have files.encoding set?

Never mind, I see the issue - I regressed search in a window that doesn't have a folder open.

Correct, I don’t have a folder open.

On 2017-08-28, at 17:57, Rob Lourens<EMAIL_ADDRESS>wrote:
Never mind, I see the issue - I regressed search in a window that doesn't have a folder open.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Microsoft/vscode/issues/33163#issuecomment-325495014, or mute the thread https://github.com/notifications/unsubscribe-auth/ABGoNlW0eOhg-W82FsoK-Qi47oMfxLMqks5sczezgaJpZM4PC5He.

I’ll check tomorrow, if you still need this.

On 2017-08-28, at 17:50, Rob Lourens<EMAIL_ADDRESS>wrote:
Do you have files.encoding set?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Microsoft/vscode/issues/33163#issuecomment-325493226, or mute the thread https://github.com/notifications/unsubscribe-auth/ABGoNnwz-hxRiZ5ei2DF4oAuEjaDpfPhks5sczYTgaJpZM4PC5He.

I don't need an answer to the files.encoding question anymore. Fix will be in tomorrow's Insiders build, and the next stable release.

Great, thanks

On 2017-08-28, at 18:54, Rob Lourens<EMAIL_ADDRESS>wrote:
I don't need an answer to the files.encoding question anymore. Fix will be in tomorrow's Insiders build, and the next stable release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Microsoft/vscode/issues/33163#issuecomment-325505691, or mute the thread https://github.com/notifications/unsubscribe-auth/ABGoNlthd5OjS_1aM-Tdi69dGQjAbuU7ks5sc0UkgaJpZM4PC5He.


Some icon is not showing on the Courses page

Description
I have tried the following:
Confirmed that the icon library is included in the HTML.
Verified that the icon class names are correct.
Used the "Inspect" tool and found the icons present in the DOM.
Checked the CSS but found no rules hiding the icons.
Steps to Reproduce
Open the Courses page.
Check the  for the icon library inclusion.
Verify the icon class names in the HTML.
Inspect the DOM to confirm the icon elements are present.
Review CSS rules for any visibility issues.
Refresh the page to see if the icons appear.
Expected Behavior
I expected the icons to display correctly on the Courses page, visually representing their intended functions and enhancing the overall user interface.
Actual Behavior
The icons did not display on the Courses page, resulting in missing visual elements that were intended to enhance the user interface and convey functionality.
Screenshots

Additional Information
I believe the issue is due to a missing or non-functional icon library.
What browser are you seeing the problem on?

[ ] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@Sadab-Hussain assigning this issue to you,
Make sure to complete this issue within 2 days.

yeah sure thank you for assign to me


Run script at startup - task bar jump list support

Please allow terminal to run scripts at startup from a task bar jump list.
What would be nice is having the Windows Terminal jump list provide a list of startup scripts, same way VS allows me to open projects form the jump list. I could pull up the jump list from the task bar and select whichever project I was working on. For example I could select a script from the jump list to take me to a particular working directory and setup command line compilation for the terminal instance.

This is /dupe #576. Please search before filing new issues.

Issue #576 is all about profiles and which tabs to open. Does that encompass what I've proposed here?

well you can add profiles with different startup commands to do just that, i guess.
once the jump list is in place,  a whole bunch options open up for things to be featured there.


Optimize when call getSessionCookieName & getSessionUriParamName

Description
Optimization when call getSessionCookieName & getSessionUriParamName
Explanation
Existing code calls getConfiguredSessionCookieName even if the context is empty.
In getConfiguredSessionCookieName, only act to get the session cookie if the context is non-null.
I think it's possible to avoid calling getConfiguredSessionCookieName and use Default defined by spec if context is null.
It's a small optimisation, but I think it saves us one less function call.
Additionally, modified getConfiguredSessionCookieName to not require a null check on its result.
Comment
One fewer comparison operation if context is non-null. and I think it makes the code a little easier to understand for the remaining priority comments.

Saving a single function call is unlikely to make any difference. I'd expect the compiler to optimise that.
Changes that reduce duplication and/or improve the readability of the code are the ones that are most likely to be applied. If there is a trade-off between duplication and readability, the priority will nearly always be readability.
In this instance I think you'd be better keeping the null check for context in the static method. That further reduces duplication and has the benefit that all of the logic is then in a single place and the code would more clearly follow the priority order documented in the code.

@markt-asf
Thanks for the nice comment.
I hadn't considered that partial compiler optimisation.
I agree, readable code should be a priority and I moved the null-check logic to a static method...

Tx for the PR.

@markt-asf thx for good comment XD


Play bytes?

Play mp3 bytes from a int8 list in Dart?

Hi, @jinfagang, thanks for your feedback! That would be a very welcome addition, as it has many use cases! If you are interested, you can contribute with a PR!
I believe you could add a new method parallel to setUrl, like setSourceData, that takes the int8 list and sets the DataSource in Android/iOS using that.
In the docs it looks like the Uint8List type on Dart can be automatically converted to a byte[] on Java or a FlutterStandardTypedData typedDataWithBytes on iOS, so it wouldn't be a problem to cross the bridge with the data.

Hi,
Expecting this feature

I need that feature too!

I need that feature too! +1

I need that feature!

me too!

Closing in favour of https://github.com/luanpotter/audioplayers/issues/182


Connect With Azuer

Operating system: ✍ Windows
Last upstream commit (run git log --author="Robert Lord" | head -n 1): ✍ TODO
language version:(nodejs version, ruby version ....) ✍ CSharp
Cortex version:  ✍ Last

Do you have an example to connect with Azure ??
Or some documentation ??
Thanks

Sorry @dfalagian I don't understand your question.
What are you trying to do exactly?
Are you talking about Microsoft Azure? Or another service/software?

Hi.
We want to connect with Microsoft Azure, to store the data.
We have searched for information but there is nothing. We need an example or documentation.
Thanks

hi @dfalagian I believe if you want to store the data returned by Cortex to Azure you will need to consult the Microsoft Azure documentation instead of Cortex one.
https://azure.microsoft.com/en-us/services/storage/blobs/

Hi.
We are interested in storing the data in the Microsoft Azure cloud.
We know that you have to do it through WebSocket. Our intention was to see if you had some documentation or some example of the connection with Azure.

Cortex doesn't interact with Microsoft Azure in anyway, so we don't have any documentation or example for that.


Travis setup

Attempt to go back to line by line TRAVIS to catch all errors.

currently setup with failing adjoint test to verify it does what it should

Yes the comments are removed. TRAVIS refuses to recognize the yml file as valid with the comments in.

Looks like we're back on track. Log looks as before my mess up.

@fablup so this is finally working as expected.
It is failing as designed too to verify TRAVIS works correctly. If you're happy with it I'll put back the correct test and should be goood to go.

Fairly confident I can tidy that up in Jenkins, yes. Having spent the last few days neatening a Jenkinsfile for OK I have a bunch of ideas to simplify/neaten the devito Jenkinsfile.

Fairly confident I can tidy that up in Jenkins, yes. Having spent the last few days neatening a Jenkinsfile for OK I have a bunch of ideas to simplify/neaten the devito Jenkinsfile.

Awesome!

The proliferation of if-then-else really starts scaring me.

Should technically only need one but we want to run the examples each individually

SEISMIC

true, will rename, it RUN_EXAMPLES fine?

true, will rename, it RUN_EXAMPLES fine?

OK. However, if RUN_EXAMPLES == True if and only if INSTALL_TYPE == conda, then I wonder why we don't just use INSTALL_TYPE ?

if and only if INSTALL_TYPE == conda

No also False for YASK backend

Codecov Report

Merging #555 into master will decrease coverage by 10.51%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #555       +/-   ##
===========================================
- Coverage   86.34%   75.82%   -10.52%
===========================================
  Files          74       74
  Lines        7021     7021
  Branches     1427     1427
===========================================
- Hits         6062     5324      -738
- Misses        764     1524      +760
+ Partials      195      173       -22

Impacted Files
Coverage Δ

devito/yask/types.py
0% <0%> (-100%)
:arrow_down:

devito/yask/function.py
0% <0%> (-97.15%)
:arrow_down:

devito/yask/__init__.py
0% <0%> (-93.88%)
:arrow_down:

devito/yask/data.py
0% <0%> (-87.69%)
:arrow_down:

devito/yask/utils.py
0% <0%> (-86.67%)
:arrow_down:

devito/yask/operator.py
0% <0%> (-86.55%)
:arrow_down:

devito/yask/wrappers.py
0% <0%> (-82.88%)
:arrow_down:

devito/yask/dle.py
0% <0%> (-37.5%)
:arrow_down:

devito/symbolics/extended_sympy.py
52.54% <0%> (-25.43%)
:arrow_down:

devito/__init__.py
89.18% <0%> (-10.82%)
:arrow_down:

... and 15 more

Continue to review full report at Codecov.

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

I have got no idea why Codecov is complaining that much

I have honestly no idea. Can't figure out why it has such a huge difference.

after quite in-depth inspection, I still can't spot anything odd with codecov. Merging

Merged.


Query on compressed image camera ros2 topics

Hi @MartyG-RealSense , I would need to generate the compressed image ros2 topics. Would suggest any ros2 plugin for the same or create a node to subscribe to the image topics & compress independently using the cv2bridge?

Thank you. Can you kindly let know the steps to be followed after installing the plugin for a specific ROS2 build?

After installing the plugin, simply perform the rs_launch.py launch process and then check the list of published topics with the command ros2 topic list to confirm that the compressed topics are being published.
https://github.com/IntelRealSense/realsense-ros?tab=readme-ov-file#published-topics

Dear @MartyG-RealSense , I wanted to update you that for ROS2 foxy, we need to use the sudo apt install ros-humble-compressed-image-transport to get the compressed streams. I understand that foxy is EOL. But for some application we were using the same and this was required.
Between, I had another question, about how these compressed topics are generated from the plugin? The camera is connected via USB3.0 cable/port to the NUC. Here we have the realsense ros package which creates the raw image/ camera info topics using the data from the camera. Will the image-transport plugin subscribe to these raw images and generate the compressed topics? I mean can we have only the ros2 compressed topics in the ros2 topic list and not the raw images?

The compressed topics plugin for a specific ROS version should ideally be installed.  For Foxy that would be:
sudo apt install ros-foxy-compressed-image-transport
However, Foxy is only supported by the RealSense ROS2 wrapper up until version 4.51.1, which should be used with librealsense 2.51.1.
An explanation of how image_transport works can be found at the link below, which states that "image_transport itself provides only "raw" transport so as not to impose unnecessary dependencies on client packages".
https://wiki.ros.org/image_transport
It is possible to unsubscribe from specific topics during runtime after launch has completed so that they are not included in the topic list.  Usually though the raw topics are active when publishing compressed topics.
https://answers.ros.org/question/409553/how-can-i-remove-active-topic/

Hi @girishsmurthyIntel  Do you require further assistance with this case, please?  Thanks!

Hi @MartyG-RealSense I was able to get the compressed topic working. Appreciate your kind support.

You are very welcome, @girishsmurthyIntel - I'm pleased that I could help.  As you were able to get the compressed topic, I will close this issue.  Thanks very much for the update!

Hi @MartyG-RealSense I was able to get the compressed topic working. Appreciate your kind support.

Hello! @girishsmurthyIntel
Could you please let me know how you got the compressed topic working, and the details of the machine you are running your system on? Could you also please specify the librealsense version and realsense ros wrapper version that you used?
Thanks!

We had to install the sudo apt install ros-foxy-compressed-image-transport (for ROS2 foxy) and when you launch the rs_launch.py, we already start to get the compressed topics. You can check this by running ros2 topic list on another terminal.  The libRealSense supported version: v2.51.1  We had built the realsense ros2 workspace from the commit 86ab0c8358214ae9a526395396c1619f9488a195 (HEAD -> ros2-development, origin/ros2-development, origin/HEAD)

Thank you for your prompt reply! @girishsmurthyIntel


fix(tsconfig): fix extends in default.json

:tada: This PR is included in version 1.1.0-next.12 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix Requirements, Add Colab

Hi,
I just ran your code, and fixed some stuff(attrdict and espeak-ng were missing in the setup). I also made a colab notebook for the LJSpeech model inference.
Thanks

Thanks for fixing the requirements. However, your Colab notebook doesn't really work because you didn't actually download the pre-trained models. Instead, you copied them from your Google Drive. You can use gdown --id GOOGLE_DRIVE_ID to download these files directly in the Colab notebook from your Google Drive once you have shared them. Once you have fixed it I can merge it into the main branch. Thank you for your help.

Ah right, I didn't know that. Will fix it. Thanks.

Changed to gdown, added colab file to demo folder

@yl4579 pls review.

Hi, @nivibilla are you able to run Emotional Speech Synthesis from https://styletts.github.io/ . Like prosody cloning.

@satani99 yes it uses the sample audio as reference.

You can add your own audio as reference with different emotions and do the prosody cloning.

Where do I get code for inference @nivibilla ?

https://colab.research.google.com/drive/18xNehSLiClZC3ub3eXiPCOnR4bz3r5Ur?usp=sharing


How to add filter input field?

I added filter = true in component also added the following tag
<app-custom-listbox [source]="source" [key]="key" [display]="display" [(destination)]="target" height="200px" [filter]="filter" > but filter is not showing in html page.
Please suggest me what mistake I am doing?

Does your custom template still have a filter to show for each list? Each list will need something like this:
<form *ngIf="filter" class="filter">
  <input class="form-control" name="filterSource" [(ngModel)]="available.picker"
    (ngModelChange)="onFilter(available)">
</form>

I'm sorry but I don't understand where I have to put this kind of code:

Hello I am trying to add  the filter feature , the input field appears but it's not working I have done all the steps I added in the custome tlist Template :

also added in my html :  [filter]="filter"
and in my ts : filter = true
but its not working


Installation succeeds, but cli then fails

I installed gitlab thus:
[root@XXXXX ~]# gem install gitlab
Fetching: terminal-table-1.4.5.gem (100%)
Successfully installed terminal-table-1.4.5
Fetching: multi_xml-0.5.5.gem (100%)
Successfully installed multi_xml-0.5.5
Fetching: httparty-0.13.5.gem (100%)
When you HTTParty, you must party hard!
Successfully installed httparty-0.13.5
Fetching: gitlab-3.4.0.gem (100%)
Successfully installed gitlab-3.4.0
Parsing documentation for terminal-table-1.4.5
Installing ri documentation for terminal-table-1.4.5
Parsing documentation for multi_xml-0.5.5
Installing ri documentation for multi_xml-0.5.5
Parsing documentation for httparty-0.13.5
Installing ri documentation for httparty-0.13.5
Parsing documentation for gitlab-3.4.0
Installing ri documentation for gitlab-3.4.0
Done installing documentation for terminal-table, multi_xml, httparty, gitlab after 3 seconds
4 gems installed

But when I attempted to execute the cli I got a failure:
[toby.ferguson@XXXX ~]$ export GITLAB_API_ENDPOINT=http://XXXX:8888
[toby.ferguson@XXXX ~]$ export GITLAB_API_PRIVATE_TOKEN='XXXX'
[toby.ferguson@XXXX ~]$ gitlab help
/usr/local/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require': cannot load such file -- readline (LoadError)
	from /usr/local/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/lib/ruby/gems/2.2.0/gems/gitlab-3.4.0/lib/gitlab/shell.rb:5:in `<top (required)>'
	from /usr/local/lib/ruby/gems/2.2.0/gems/gitlab-3.4.0/lib/gitlab/cli.rb:4:in `require_relative'
	from /usr/local/lib/ruby/gems/2.2.0/gems/gitlab-3.4.0/lib/gitlab/cli.rb:4:in `<top (required)>'
	from /usr/local/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/lib/ruby/gems/2.2.0/gems/gitlab-3.4.0/bin/gitlab:5:in `<top (required)>'
	from /usr/local/bin/gitlab:23:in `load'
	from /usr/local/bin/gitlab:23:in `<main>'

My environment:
[toby.ferguson@XXXX ~]$ ruby -v; bundle -v; gem -v
ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
-bash: bundle: command not found
2.4.5

@TobyHFerguson That's strange.  Readline is part of the stdlib that comes with Ruby: http://ruby-doc.org/stdlib-2.2.0/libdoc/readline/rdoc/Readline.html

@TobyHFerguson what OS are you using?

Oracle Linux 6
On Sat, Jun 20, 2015 at 12:44 PM, Nihad Abbasov<EMAIL_ADDRESS>wrote:

@TobyHFerguson https://github.com/TobyHFerguson what OS are you using?
—
Reply to this email directly or view it on GitHub
https://github.com/NARKOZ/gitlab/issues/135#issuecomment-113812968.

--
toby

@TobyHFerguson Did you compile ruby yourself or did you install a package from Oracle?  If you compiled it yourself could you share your ./configure?

I downloaded ruby-2.2.2.tar.gz from somewhere and compiled it myself.
Attached are the config.log and configure files
On Fri, Jun 26, 2015 at 8:33 AM, Sean Edge<EMAIL_ADDRESS>wrote:

@TobyHFerguson https://github.com/TobyHFerguson Did you compile ruby
yourself or did you install a package from Oracle? If you compiled it
yourself could you share your ./configure?
—
Reply to this email directly or view it on GitHub
https://github.com/NARKOZ/gitlab/issues/135#issuecomment-115733221.

--
toby

@TobyHFerguson What do you get when you do ldd $(which ruby) and find /usr/local/lib -name readline.so?

[root@ca-ovsx32 ~]# ldd $(which ruby)
    linux-vdso.so.1 =>  (0x00007fffb18c1000)
    libpthread.so.0 => /lib64/libpthread.so.0 (0x00007ffbfcd4e000)
    librt.so.1 => /lib64/librt.so.1 (0x00007ffbfcb45000)
    libdl.so.2 => /lib64/libdl.so.2 (0x00007ffbfc941000)
    libcrypt.so.1 => /lib64/libcrypt.so.1 (0x00007ffbfc70a000)
    libm.so.6 => /lib64/libm.so.6 (0x00007ffbfc485000)
    libc.so.6 => /lib64/libc.so.6 (0x00007ffbfc0f1000)
    /lib64/ld-linux-x86-64.so.2 (0x00000038a9800000)
    libfreebl3.so => /lib64/libfreebl3.so (0x00007ffbfbeee000)
[root@ca-ovsx32 ~]#

[root@ca-ovsx32 ~]# find / -mount -name readline.so
/opt/gitlab/embedded/lib/ruby/2.1.0/x86_64-linux/readline.so
/usr/lib64/python2.6/lib-dynload/readline.so
/usr/lib64/ruby/1.8/x86_64-linux/readline.so
[root@ca-ovsx32 ~]#

On Wed, Jul 8, 2015 at 8:46 AM, Sean Edge<EMAIL_ADDRESS>wrote:

@TobyHFerguson https://github.com/TobyHFerguson What do you get when
you do ldd $(which ruby) and find /usr/local/lib -name readline.so?
—
Reply to this email directly or view it on GitHub
https://github.com/NARKOZ/gitlab/issues/135#issuecomment-119630993.

--
toby

@TobyHFerguson I only see readline.so for system Ruby (1.8) and Gitlab's embedded Ruby (2.1.0).  Not the 2.2.x Ruby you are trying to run.  I think the problem might be you were missing the readline-devel package when you built Ruby.  Try installing those header files and recompiling.
Disclaimer: I don't think it's a good idea to do any of this stuff on your Gitlab server - which appears to be what you're doing based on the find output.  A friend of mine put RVM on his Gitlab server and it caused so many problems that he ended up having to reinstall.

Thanks for the information, sorry about the tardy response.
Given my current constraints I'll have to forgo the pleasure of using this module then - the gitlab server is also the place where I'm attempting to run my programs, and with the warning above that's a problem!
I'll close this issue for now. Thanks again for the support. Much appreciated.


Pulling Googlecentric Files

I Might be doing something wrong but is there away to pull gdocs, and gsheets? When I pull the down they come as two separate files, "Foo" & "Foo.desktop", but not "Foo.gdoc".
Any help would be greatly appreciated!
Regards,
Austin

Hello again @viruscmd! Thanks for reporting this.
So Google Docs + Sheets cannot be downloaded raw but only exported. The files that get downloaded are an empty file for consistency and a .desktop file if you are on Linux -- .desktop file is clickable.
In order to export such a file, you'll need to pass in flag "--export " eg "drive pull --export pdf,docs,txt 2013Docs"
The README actually talks about this https://github.com/odeke-em/drive#exporting-docs.
Please let me know if this works for you.

@odeke-em  Thanks for the info. I just want to clarify that if I use --export xlsx,docx it will export the all gsheets as xlsx, and gdocs as docx? i.e.
folder foo has two files, bar.gsheet and baz.gdocs. Using ''--export  xlsx,docx" the pulled files will be "/_exports/bar.xlsx" & "/_exports/baz.docx".
Thanks,
Austin

Yes in deed. If it finds xlsx exportable, it will export to xlsx if it finds docx exportable it will export to that.

@odeke-em You are the best! Thanks.

Thanks for the kind words @viruscmd :) and for reaching out!


Having issues with three of the files- "expects" "higher order functions" and "inheritance" are not showing up when I refresh the browser. Those subjects have disappeared!

In summary. there were 8 files and subjects. when I refresh the browser, it only shows 5 files and subjects.  "expects" "higher order functions" and "inheritance" are not showing up when I refresh the browser. Those subjects have disappeared!
Here's what it looks like:

Ditto, same problem, though different subjects. Arrays and Objects also disappeared for me before finishing them (somewhere around the 3rd or 4th test).

I was experiencing the same problem throughout: I'd get something incorrect, one of the subjects disappeared from the "Subjects covered" count at the top of KoansRunner.html, the "Koans Learned" count decreased, and KoansRunner.html jumped to the next subject prematurely.
I found that this occurred whenever I had a typo, for example if I closed an array with two right brackets. You could try pasting the problematic section of your code into http://jshint.com to see what's wrong; just ignore the undefined variable errors.
[Same comment posted on issue #45 Disappearing Koan... same fix for same issue.]

@j0narch3r do you mean you started the koans from the beginning?

@v-node Yes. The only way I could see to guarantee I was getting through all of them and not missing subjects to the bug.

I have the same issue happening with most of the koans. I always refreshed through Cmd + R and never clicked on the "Meditate again" links. I've checked over my code many times to make sure there were no typos (such as an extra closing bracket). I've even redownloaded the files again and started over, but still had the same issue. Does anyone know what's causing this? Could it be the browser? I'm using Firefox version 37.0.2. Do I need to upgrade to a newer version? Does this issue occur for people using Chrome? Thanks.

Would be great if someone fixed this
Thanks,
Fareez
On Sun, May 17, 2015 at 4:50 AM, Joyce Liu<EMAIL_ADDRESS>wrote:

I have the same issue happening with most of the koans. I always refreshed through Cmd + R and never clicked on the "Meditate again" links. I've checked over my code many times to make sure there were no typos (such as an extra closing bracket). I've even redownloaded the files again and started over, but still had the same issue. Does anyone know what's causing this? Could it be the browser? I'm using Firefox version 37.0.2. Do I need to upgrade to a newer version? Does this issue occur for people using Chrome? Thanks.
Reply to this email directly or view it on GitHub:
https://github.com/mrdavidlaing/javascript-koans/issues/38#issuecomment-102769197

I had this issue, until I fixed a couple typos with an extra closing bracket in my answers. Using Chrome.

I had the same issue.  I actually got it when I added a toString function to a number.  Sure enough editing that out fixed it.


Permissions for Ambassadors / Admins

Need to empower ambassadors and admins to block out multiple timeslots without any restrictions for booking out rooms for office hours or events.
Acceptance Criteria

[ ]  An ambassador / admin should be able to reserve any or multiple rooms for any duration without any restrictions.
[ ] An ambassador / admin should be allowed to reserve rooms anytime in the future.

Make sure to think about how you can distinguish an admin reservation for a specific purpose from a regular reservation! I think it would be useful to have some sort of way to indicate what the reservation is for !

That's a good point! We were thinking of just making an entirely different view for admin reservations. We can borrow a lot of the frontend code that already exists since I believe that the table is actually a custom widget.


Getting error Some schema elements were defined by the model (which is no longer supported), but they do not exist in the database.

Hello,
I'm stuck with my application because of this error:
Some schema elements were defined by the model (which is no longer supported), but they do not exist in the database. Run the following to create them if you haven't already:

rake neo4j:generate_schema_migration[constraint,StudyVersion,uuid]

And then run rake neo4j:migrate

I tried to do so as asked in the terminal but after I have same error again and I cannot run the app.
Please provide me a solution for it.
Thank you
Additional information which could be helpful if relevant to your issue:
Code example (inline, gist, or repo)
Repo where the problem occurs: https://gitlab.com/kubus41/AppDevDK.git
Branch name: neo4j-dev
Runtime information:
Neo4j database version: 3.0.10
neo4j gem version: 9.0.5
neo4j-core gem version: 8.0.1

Hey @JakubA3informatics !
Did you run rake neo4j:migrate and restart the server after running the migration?
I would also check the :schema command to make sure the constraint was created
I can't access the GitLab repo.  Here's my account if you want to give me access: https://gitlab.com/cheerfulstoic

Hey Brian,
I did those as you suggested but still some issues.
I get you the access to the Repo so you can take a closer look.
Thank you ☺
From: Brian Underwood<EMAIL_ADDRESS>Reply-To: neo4jrb/neo4j<EMAIL_ADDRESS>Date: Tuesday, 12 December 2017 at 14.42
To: neo4jrb/neo4j<EMAIL_ADDRESS>Cc: Jakub Lemiszewski<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [neo4jrb/neo4j] Getting error Some schema elements were defined by the model (which is no longer supported), but they do not exist in the database. (#1452)
Hey @JakubA3informaticshttps://github.com/jakuba3informatics !
Did you run rake neo4j:migrate and restart the server after running the migration?
I would also check the :schema command to make sure the constraint was created
I can't access the GitLab repo. Here's my account if you want to give me access: https://gitlab.com/cheerfulstoic
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/neo4jrb/neo4j/issues/1452#issuecomment-351054440, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AffYoyCDvbCyQa4AHOrvYpj7arBbk0Kuks5s_oKqgaJpZM4Q-o2D.

Hey @JakubA3informatics
I set up the app against a Neo4j server and tried running StudyVersion.first in a rails console session.  That gave me the error:
Neo4j::DeprecatedSchemaDefinitionError:           Some schema elements were defined by the model (which is no longer supported), but they do not exist in the database.  Run the following to create them if you haven't already:

rake neo4j:generate_schema_migration[constraint,StudyVersion,uuid]
rake neo4j:generate_schema_migration[index,StudyVersion,state]

And then run `rake neo4j:migrate`

I shut down my console, ran rake neo4j:migrate, and then when I run StudyVersion.first in the console it works fine (returning nil because I don't have anything in the DB).
Did you run :schema to see if the constraints were created in the DB?  Are you getting the error in the server or the console?  Did you restart the console as well?  I'm at a bit of a loss, unfortunately...

Hey Brian,
I resolved my issue doing this steps:
rake neo4j:generate_schema_migration[constraint,Amendment,uuid]
then run
neo4j:migrate RAILS_ENV=test or for development
After this seems all working correctly and I restarted the console.
I also purged the Test DB with MATCH (n) DETACH DELETE to avoid issues.
Thank you for your help ☺
Jakub
From: Brian Underwood<EMAIL_ADDRESS>Reply-To: neo4jrb/neo4j<EMAIL_ADDRESS>Date: Thursday, 14 December 2017 at 03.44
To: neo4jrb/neo4j<EMAIL_ADDRESS>Cc: Jakub Lemiszewski<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [neo4jrb/neo4j] Getting error Some schema elements were defined by the model (which is no longer supported), but they do not exist in the database. (#1452)
Hey @JakubA3informaticshttps://github.com/jakuba3informatics
I set up the app against a Neo4j server and tried running StudyVersion.first in a rails console session. That gave me the error:
Neo4j::DeprecatedSchemaDefinitionError:           Some schema elements were defined by the model (which is no longer supported), but they do not exist in the database.  Run the following to create them if you haven't already:
rake neo4j:generate_schema_migration[constraint,StudyVersion,uuid]
rake neo4j:generate_schema_migration[index,StudyVersion,state]
And then run rake neo4j:migrate
I shut down my console, ran rake neo4j:migrate, and then when I run StudyVersion.first in the console it works fine (returning nil because I don't have anything in the DB).
Did you run :schema to see if the constraints were created in the DB? Are you getting the error in the server or the console? Did you restart the console as well? I'm at a bit of a loss, unfortunately...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/neo4jrb/neo4j/issues/1452#issuecomment-351592044, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AffYo2mMQbhcOBv_Kc2GCGmzDRAq7X6dks5tAIuhgaJpZM4Q-o2D.

:tada: Thanks for the update!


[FP]: shared-resourcemapping for multiple cve

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:opentelemetry:opentelemetry:0.32.0:::::::*
CVE
CVE-2023-43810
ODC Integration
{"label"=>"Gradle Plugin"}
ODC Version
11.1.1
Description
Multiple CVE reported are
CVE-2023-45142
CVE-2023-47108
CVE-2023-43810
Why I think it is False Positive?
The CVEs are reported in python and go of opentelemetry

approved


html-pdf - header content not displaying properly

If i give large content in Header section . It is displaying behind of the body content .
Body content should start after header content finished
and adding extra spaces before starting header content.
Can you please help  how to resolve this ?
Here with i attached sample PDF which i generated using html-pdf
businesscard.pdf

Same here.


estimatesmartfee always returns -1

I tried to call the estimatesmartfee so that I can get the feerate, but it always returns the -1. So I want to know the reason. And is there a default feerate that I can use when it returns -1.
request:
curl --location --request POST 'localhost:44555' \
--header 'Authorization: Basic dGVzdDp0ZXN0MTIzNA==' \
--header 'Content-Type: application/json' \
--data-raw '{
    "jsonrpc":"1.0",
    "id":"curltest",
    "method":"estimatesmartfee",
    "params":[
        6
    ]
}'

response:
{"result":{"feerate":-1,"blocks":25},"error":null,"id":"curltest"}

Note that currently, transaction size is rounded in Dogecoin fee calculation. The recommended fee consists of:
1 DOGE per transaction
1 DOGE per 1000 bytes rounded down
1 DOGE per dust output
So fee per kilobyte is 1 DOGE, but if you don't add per-transaction and per-dust components, your calculation will result in insufficient fee.

Note that currently, transaction size is rounded in Dogecoin fee calculation. The recommended fee consists of:
It is bad if the price of the coin increases, but the min fee is always fixed at 1 doge

@uranik777 You are welcome to discuss it on reddit: https://redd.it/lg4hp7

Are you on regtest?

Actually, I take that back ^. This happens if you start requesting nblocks large enough (like 100, etc)

Do you know how to reproduce with the dogecoin-cli?

@p-j01 dogecoin-cli estimatesmartfee  100

@exp0nge How many blocks do you need to have synced? I get
{
  "errors": [
    "Insufficient data or no feerate found"
  ],
  "blocks": 0
}

Confirming Dogecoin doesn't use smart fees, and has no plans to adopt them. Instead we use fixed fee rates as per early Bitcoin. @shibe2 's comment is nearly perfect, except that we round up transaction size to the nearest 1kB, not down. Hope that helps.


review service buttons remains disabled if the service is again sent for approval

Step 1 - CSP approves a service.
Step 2 - ISV updates the service again.
Step 3 - CSP has still not logged out. Just refreshed the table.
Step 4 - The approve and reject button remains disabled.
https://github.com/eclipse-xpanse/xpanse/assets/54129659/bd05b048-82ce-4b8d-8ae7-fee379d732ed

let me do it

@WangLiNaruto we have new issues after this fix. The approve and reject buttons are always enabled for all services.


[feature request] Consistent API Schema

Is your feature request related to a problem? Please describe.
When using the API there does not appear to be a standardized schema. This is not necessarily a problem but does create issues for specific use-cases where someone might want to collate all of the data from various products into one view. Example is using a JSON import function into a google sheet, the JSON structure is different for the GKE and EKS products so column names are not lined up.
from: https://endoflife.date/api/amazon-eks.json the schema appears to be:
{
"cycle":"value",
"releaseDate":"datetime",
"eol":"datetime",
"latestReleaseDate":"datetime",
"latest":"value",
"lts": "BOOLEAN_VALUE"
}

from: https://endoflife.date/api/google-kubernetes-engine.json
{
"cycle":"value",
"eol":"datetime",
"support":"datetime",
"releaseDate":"datetime",
"latestReleaseDate":"datetime",
"latest":"value",
"lts":"BOOLEAN_VALUE"
}

The addition of "support" causes the schema to change here. Also note that the order of keys is not consistent among the two (lesser issue and can be fixed on client side).
Describe the solution you'd like
A structured and consistent API response schema across all products that is created and enforced by the community. I am not sure the best approach to take here but whatever consistent keys are agreed upon by the community could always be present but present an empty value or a bool (FALSE) if no data exists.
Example could be (building off the previous example):
{
"cycle":"value",
"releaseDate":"datetime",
"support":"value (in this instance we would return an empty string or bool value since it is not present in this specific product)
"eol":"datetime",
"latestReleaseDate":"datetime",
"latest":"value",
"lts":"BOOLEAN_VALUE"
}

from: https://endoflife.date/api/google-kubernetes-engine.json
{
"cycle":"value",
"eol":"datetime",
"support":"datetime",
"releaseDate":"datetime",
"latestReleaseDate":"datetime",
"latest":"value",
"lts":"BOOLEAN_VALUE"
}

Describe alternatives you've considered
I have not considered alternatives yet, for my specific use case I am attempting to ingest this data into a spreadsheet (yuck I know :) )
Additional context
Add any other context or screenshots about the feature request here.

We have a v1 API being stabilized at #2080. This is the relevant EKS page (this doesn't get updated regularly, so please don't rely on this yet). https://deploy-preview-2080--endoflife-date.netlify.app/api/v1/products/amazon-eks/
Hopefully, that should solve most of these concerns.

Excellent! This is an amazing project, and thanks for all the effort that goes into this.

Closing this for now, but feedback on our new API is welcome at #2080.


Loop detection in linked list

I can implement loop-detection using different algorithm and also how to remove them. Please assign this to me.

Hi I am Archi. I would like to contribute to this as a part of GSSOC'21. Please assign me this as I have prior knowledge of LinkedList

Please assign this issue to me for Java.

present at #189


Track service/actor life cycle

It would be very handy to have automatic events tracked for the various life cycle events actors and services go through: registration, startup, shutdown, communication listener open/close, abort, etc.
For more information: https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-reliable-services-lifecycle.

@ohadschn I just implemented some logging like this yesterday with the use of Autofac.DynamicProxies to decorate all service instances with logging.
A caveat is getting the async decoration right which we managed to get right with the help of this answer https://stackoverflow.com/a/39784559
The only problem I face right now is that the operation logging does not have correlation to the tracked remote calls that are logged by this lib here. (Why i dig through the issues here and found yours ;) )
I also did not manage to decorate the hosting ActorService instance but only the actor classes them self.
Besides from that I am pretty happy with the logs generated and i can now see if any of the mentioned methods have performance problems etc.
Here is an example from the deployment yesterday afternoon:

I am afraid I can not share the actual code as it is from a customer project.


Add meta-search to third-party plugins

Hi there,
I've been using meta for a couple years and have written a couple utilities for internal use that I've started open-sourcing.

NPM package: https://www.npmjs.com/package/meta-search
GitHub: https://github.com/fluxsauce/meta-dash

The initial version includes a package search, which is useful for finding out-of-date packages.
Would it be possible to list it under the third-party plugins?
Thanks, let me know if you have any questions or concerns!

Codecov Report

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

@@           Coverage Diff           @@
##           master     #230   +/-   ##
=======================================
  Coverage   48.27%   48.27%
=======================================
  Files           1        1
  Lines          29       29
  Branches        5        5
=======================================
  Hits           14       14
  Misses         11       11
  Partials        4        4

Continue to review full report at Codecov.

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

/lgtm

Awesome, thanks!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Flexible bson encoder decoder

Closing this in favor of https://jira.mongodb.org/browse/PYTHON-1750


Could you add Resuming/Resumed State for Room

Describe the bug
Where is no way to know Room.State (reconnecting, resuming) when using ReconnectType.FORCE_SOFT_RECONNECT
To Reproduce
Steps to reproduce the behavior:

val room = LiveKit.create(appContext = context)
room.setReconnectionType(ReconnectType.FORCE_SOFT_RECONNECT)
room::state.flow.collect { state -> if (state == Room.state.RECONNECTING) { state.show() }}

Expected behavior
Please add existing reconnecting state or add new state resuming for Room.state

room.setReconnectType is not meant for production use, and is only there for testing purposes. It's marked @VisibleForTesting for this very reason.


Core electron + core backend ESM

Notables:

Add dual cjs/esm builds for core backend and electron
Electron needs min range bumped from 23 to 28 BREAKING
(WIP) Exploring converting DTA's backend code to be ESM, and consume esm version of backend and electron.

Note: tsc does not add .js extensions to extensionless imports (unlike Vite :( ), so had to add .js at the end of our relative import paths in DTA's backend.

Pausing work on this PR for now, but documenting the work so far:

Moved display-test-app to all ESM, but having runtime issues.
When moving core-electron to use export maps, there's one important thing to note: Any ESM-based application that uses a dependency's export maps requires that dependency to be ESM (which is to have "type": "module" in it's package.json). This is relevant to @itwin/electron-authorization, because we have export maps in there. I experimented with making that package to be ESM, pushing my work to a remote branch. I confirmed that electron-authorization's tokenStore was able to run, successfully accessing my OS-based credentials and utilizing 'electron-store' under the hood, so the branch I linked above demonstrates the feasibility of publishing a ESM-based NPM package for backends.
I haven't dug into the __dirname is not defined error, but it pops up on the frontend side of the electron app, suggesting that our vite.config.js needs to be updated to use electron's ESM version, rather than the CJS version. I suspect that if we are able to solve this particular issue, we'll see more than just the spinner on startup
I haven't dug into why the preload script is not being able to be loaded - everything does point to the backend being able to find ElectronPreload.mjs though (there was a prior runtime error saying it couldn't find the file). Maybe @GytisCepk could have a better idea on what is going on...
To reproduce what I see in DTA, go to core/electron/ and run just pnpm build:esm - I'm using the import.meta.url in ElectronHost.ts, so building CJS will throw build errors. Then go to test-apps/display-test-app and run pnpm build followed by pnpm start.
Have not seen evidence otherwise that we can build dual CJS/ESM for core-electron, solely because of import.meta.url. CJS allows require but ESM doesn't. ESM allows import.meta.url but CJS doesn't. Why is this important? core-electron's ElectronHost requires a path to ElectronPreload.mjs passed into one of it's config, and the 2 ways above are the 2 unique and correct ways to resolve file paths within a module.

Image of the runtime issue:

I haven't dug into why the preload script is not being able to be loaded - everything does point to the backend being able to find ElectronPreload.mjs though (there was a prior runtime error saying it couldn't find the file).

ESM is not supported for preload script if sandbox is enabled (Electron ESM support matrix). If we plan to drop CJS from both backend and frontend code for Electron, we'll still need to transpile preload script to CJS.

I haven't dug into the __dirname is not defined error, but it pops up on the frontend side of the electron app, suggesting that our vite.config.js needs to be updated to use electron's ESM version, rather than the CJS version.

Isn't __dirname only available in CJS ? __dirname Node.js docs