Worst Cut In Detection

This cut in detection was one of the worst in recent memory. The car in front clearly is in a lane that ends and merges into my lane. The lead car did not merge well at all and OP did not handle the situation with the right approach.
comma 3
master-ci 0.8.9
dfb993cc43ba69dc|2021-09-02--07-53-46--21
08:14::13 local time
2020 Toyota Corolla Hatchback
Additional context
One thing I've noticed is that cut in detection is significantly worse if they don't use a turn signal. That inference of when someone might want over is just lacking. The context of this situation is more clear than if both lanes continued... so this type of cut in should be improved.

While not directly related, this cut in detection did decent given the circumstances. I'll be excited to see the wide angle be used to improve this type of merge even better!
comma 3
master-ci 0.8.9
dfb993cc43ba69dc|2021-09-03--07-04-43--20
07:24::42 local time
2020 Toyota Corolla Hatchback


Resolution of richtext elements

There are some problems with resolving the RichText
When creating lists using RichText, a <p> element is erroneously displayed before the <ul> element. This results in invalid HTML structure and causes errors in Next.js.
Current Workaround
To avoid these errors, we are currently rendering all paragraph elements as <div>. While this prevents errors, it also removes <p> elements entirely from the RichText content.
{
    "data": {},
    "content": [
        {
            "data": {
                "data-fs-style": "0",
                "data-fs-property-style": "0"
            },
            "content": [
                {
                    "data": {},
                    "content": [
                        {
                            "type": "text",
                            "content": "list",
                            "data": {}
                        }
                    ],
                    "type": "listitem"
                },
                {
                    "data": {},
                    "content": [
                        {
                            "type": "text",
                            "content": "list",
                            "data": {}
                        }
                    ],
                    "type": "listitem"
                }
            ],
            "type": "list"
        }
    ],
    "type": "paragraph"
}

The same applies to the H1-H6 formats. Here, “Block” is played as the type although you would expect it to play as type “H1” “H2” etc. instead of via the data-style

Hi CaarLo1337,
you are correct, in FirstSpirit lists (UL) are always nested in a paragraph (P). But it's important to note that neither in CaaS nor in the content API we are creating HTML. Both provide documents models which are defined in FirstSpirit and which not directly compatibly to the HTML document model. So it must be mapped with some effort to HTML. That the used names in FirstSpirit are HTML tag names does not make this discussion easier.
To sum this up: your solution is a perfectly valid way to handle the transformation to HMTL.

To rectify the structure, you could split the content array on type=list, and pull the list item up a level
e.g.
const input = [
  {
    content: [text1, text2, list, text3],
    type: "paragraph",
  },
];
// gets transformed to
const transformed = [
  { content: [text1, text2], type: "paragraph" },
  {
    content: list.content,
    type: "list",
  },
  [{content: [text3], type: "paragraph"}]
];


Tooling instruction missing

https://www.npmjs.com/package/@material/mwc-list
but an additional tooling step is required to resolve bare module specifiers, as well as transpilation and polyfills for IE11. See [here](https://github.com/material-components/material-components-web-components#quick-start) for detailed instructions.

Im guessing the instructions aren't necessary.


Pass response data from service

It would be nice if response data from KIA API was passed in response variable so that notifications of command success/failure could be sent after completion.

Are you looking for confirmation of success or failure of actions? Or raw api data?
If success failure a pr was created for this but the person didn't finish it.  Open to PRs on that function.  It has been done before just needs some time to complete.

Confirmation that command was actually completed successfully.  Right now I'm sending a climate start request, waiting 1 minute, forcing update, checking engine sensor, then sending mobile notification based on sensor state.  Thanks for replying.

Open to a PR on this.  I don't own one of these car anymore so can't implement as easily as I used to.


improved sentry tagging

:x: 43 Tests Failed:

Tests completed
Failed
Passed
Skipped

789
43
746
0

View the top 3 failed tests by shortest run time

apps/audit/tests.py::::test_change_context_returns_none_without_request_with_team

Stack Traces | 0.003s run time

apps/audit/tests.py:22: in test_change_context_returns_none_without_request_with_team
    with current_team(AuthedRequest.Team()):
.../hostedtoolcache/Python/3.11.11.../x64/lib/python3.11/contextlib.py:137: in __enter__
    return next(self.gen)
apps/teams/utils.py:46: in current_team
    set_current_team(team)
apps/teams/utils.py:34: in set_current_team
    sentry_sdk.get_current_scope().set_tag("team", team.slug)
E   AttributeError: 'Team' object has no attribute 'slug'

apps/audit/tests.py::::test_change_context_returns_value_for_authorized_team_req

Stack Traces | 0.003s run time

apps/audit/tests.py:41: in test_change_context_returns_value_for_authorized_team_req
    with current_team(AuthedRequest.Team()):
.../hostedtoolcache/Python/3.11.11.../x64/lib/python3.11/contextlib.py:137: in __enter__
    return next(self.gen)
apps/teams/utils.py:46: in current_team
    set_current_team(team)
apps/teams/utils.py:34: in set_current_team
    sentry_sdk.get_current_scope().set_tag("team", team.slug)
E   AttributeError: 'Team' object has no attribute 'slug'

apps/audit/tests.py::::test_change_context_returns_value_for_unauthorized_team_req

Stack Traces | 0.003s run time

apps/audit/tests.py:35: in test_change_context_returns_value_for_unauthorized_team_req
    with current_team(AuthedRequest.Team()):
.../hostedtoolcache/Python/3.11.11.../x64/lib/python3.11/contextlib.py:137: in __enter__
    return next(self.gen)
apps/teams/utils.py:46: in current_team
    set_current_team(team)
apps/teams/utils.py:34: in set_current_team
    sentry_sdk.get_current_scope().set_tag("team", team.slug)
E   AttributeError: 'Team' object has no attribute 'slug'

To view more test analytics, go to the Test Analytics Dashboard
:loudspeaker:  Thoughts on this report? Let us know!


Use different seed to noise each shard

Use different seed to noise each shard
Description

Category: bugfix
JIRA issue: none

Previously, we used the same seed in each shard. This lead to weird levels of noise correlation between shards, particularly when the shards were small (especially noticeable in survey datasets).
Testing
I verified that after this change, the ACS and CPS do_not_respond rates were well-calibrated in the RI data, and in the sample data (run across many overall seeds to get a good sample size).

This you will need to make sure you run the slow tests for.

This you will need to make sure you run the slow tests for.

Good call-out! It looks like there was one test I needed to change. Double-checking that it passes now.


Simplify ENV var config customisation for container use

Overview
When using Docker containers, ENV vars are often used to customise instance specific values.
by adding os.environ to the ConfigParser of the cfg file, we can use something like:
docker run --name pywps -d -p 80:80 --hostname pywps.example.com pywps ....
[default]
# baseurl can be overridden using env vars ala Docker
baseurl=http://%(HOSTNAME)s

[server]
url=%(baseurl)s/ncwps
# the output path and url need to correlate
outputurl=%(baseurl)s/ncwps/outputs

or, if you have an external https proxy like traefik:
docker run --name pywps -d --env BASEURL=https://pywps.example.com:443 pywps ...
[default]
# baseurl can be overridden using env vars ala Docker
baseurl=http://%(HOSTNAME)s

[server]
url=%(baseurl)s/ncwps
# the output path and url need to correlate
outputurl=%(baseurl)s/ncwps/outputs

Related Issue / Discussion
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage decreased (-1.0%) to 73.103% when pulling b053a1f15e7e7214e96c5c45cb201890954c948d on SvenDowideit:add-os.environ-interpolation-to-ConfigParser into 1306d4e76b4e80bd41ba8bd02293e1d9ffc9f622 on geopython:master.

@jachym :)

Cool, thanks a lot!


Rename annotations in InlineInstances

depends on: ~https://github.com/freechipsproject/firrtl/pull/967~ https://github.com/freechipsproject/firrtl/pull/1126
The current InlineInstances does not do any renaming of annotations. This PR updates InlineInstances to use chained rename maps for renaming of inlined targets.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: albertchen-sifive:x: azidarYou have signed the CLA already but the status is still pending? Let us recheck it.

@albertchen-sifive I believe this PR can be updated now?

yes I'm working on it. and I think I ran into  a bug in my PR while trying to update this PR
andThen ignores the chained field of the next map so a.andThen(b.andThen(c)) is missing the renames in c
https://github.com/freechipsproject/firrtl/blob/master/src/main/scala/firrtl/RenameMap.scala#L43

@azidar updated, now it depends on https://github.com/freechipsproject/firrtl/pull/1126 for RenameMap chaining fixes


document multi-line triggers

I attempted to improve the documentation for using multi-line cron/scm entries. My initial intuition for this was to use multiple cron()'s, but only one showed up. I'm happy to adjust this language as desired and add it other places so others can discover this; I don't think it should require specific language/parsing knowledge to know how to separate lines. Or, perhaps the solution is that multiple cron()s and scm()s should be allowed. Just throwing one documentation solution out there :)

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


Switch enum underlying type comparison to use decimal rather than long

Fixes issue #467 by using decimal rather than long. I didn't see any place to add enum comparision tests so I made a new file.

Thanks for the PR. Would you be willing to adapt the specs to use the AAA syntax and naming conventions? E.g. like https://github.com/dennisdoomen/FluentAssertions/blob/master/Tests/FluentAssertions.Shared.Specs/ComparableSpecs.cs#L57

Sure, I probably won't get to it today, but happy to fix the naming.

Hi @msackton, you've abandoned your PR?

Sorry, I was out of town and forgot about this. I've cleaned up the tests.


Clearbit Discovery API

The tentatively proposed interface for the Clearbit Discovery API (#2) is:
clearbit discover --tech marketo --sort alexa_asc

The actual query API is quite rich, supporting many fields, and and or queries, etc. It seems like a nice approach would be to support a few important fields as flags (--tech above), and then allow users to write more complex queries directly if needed.
Looking through Clearbit's supported discovery fields, the most likely to warrant flags look to be:

employees
location
raised
tech

We've also created our own DSL for searching which we'll parse on the backend.
Might be easier to allow person to submit a string query instead of having the library know about each attribute
clearbit discover --query 'employees:~100 or:(tech:marketo tech:google_apps)'  --sort alexa_asc

https://clearbit.com/docs#discovery-api-andor-queries


Add support for matplotlib styles and setting figiure size based on inches

If it would be great to support matplotlib styles and setting figure size based on inches.  It will be great for people who want create some plot for papers.

One sizing Matplotlib plots, HoloViews offers the fig_inches option for Maplotlib plots so I think this in theory is possible. This is explained in more details in the Plotting with Matplotlib guide of HoloViews.
I'm not sure to which extent HoloViews handles Matplotlib styles.
These are good feature requests, they are probably not high priority though for now.
What feels pretty important to me in this case is to document how hvPlot handles sizing plots.


Design to allow multiple versions of primitive VSA operators

Allow for multiple, different versions of the primitive VSA operators (bind, bundle, permute, and maybe others).
This is needed, for example to introduce HRR binding, and randsel bundling (#75).
This is related to #72 and #25, which appear to imply dispatching on hypervector type. However, people will want to introduce and compare different operators applied to the same hypervectors, so this issue is orthogonal to hypervector type.

How about an API such as this (not an exhaustive list but just an illustration):
import torchhd

# binding operations
torchhd.multiply(a, b)  # works for all data types (implements XOR for boolean tensors)
torchhd.convolution(a, b)
...

# bundling operations
torchhd.sum(a, b)
torchhd.mean(a, b)
torchhd.randsel(a, b)
...

With this design our current torchhd.bind and torchhd.bundle will simply be replaced with torchhd.multiply and torchhd.sum.
We should add sections in the documentation to group the different binding and bundling implementations so it's clear for people what they are meant to do.

Sorry - I am so totally not a pythonista that I can't tell what that does.
If it's any help, my motivation for the request is:

The space of VSA primitive operations is broadly indexed by hypervector type and operator algorithms (which are roughly orthogonal).
Some VSA researchers will want to compare the same abstract VSA circuit across differing definitions of VSA primitives.
Researchers doing that would generally want consistent VSA primitive definitions throught the abstract VSA circuit (i.e. not using multiple different implementation in the one VSA circuit).
Ideally, they would define the abstract VSA circuit once and then be able to specifiy the different VSA primitive definitions without rewriting the circuit code (thus avoiding a possible source of error).

This is also the motivation for #79. If the encoding functions are defined in terms of abstract bind and bundle then the impact of different instantiations of bind and bundle can be investigated.

In my comment above I tried to present a sketch of the functions that Torchhd could expose/provide to allow users to switch between multiple methods for the primitive operations. So besides the current default (sum for bundling and product for binding) we would also expose circular convolution for binding and randsel for bundling, among many other possible methods.
I think to make this design very ergonomic for the user we should still expose generic bind, bundle and permute functions but allow users to override them. For this API design I was thinking we can take inspiration from multiprocessing.set_start_method.
My idea is a design as follows:
import torchhd

A, B = torchhd.random_hv(2, 10000)

torchhd.bundle(A, B)  # addition by default
torchhd.bind(A, B)  # product by default

# here comes the new part
torchhd.set_bundle_method(torchhd.randsel)
# torchhd.set_bind_method(...)
# torchhd.set_permute_method(...)
# from now on any call to torchhd.bundle will execute randsel
torchhd.bundle(A, B)
# you could equivalently do
torchhd.randsel(A, B)

A benefit of this design is that our data structures such as hash tables and graphs internally use the bundle and bind functions which means that their behavior will automatically update when torchhd.set_bundle_method is called. Another benefit is that users could specify their own bundling function which takes two hypervector tensors and produces a third and can set torchhd to use this in all the code, like so:
import torchhd

def my_custom_bundle(a, b):
    return a + b

torchhd.set_bundle_method(my_custom_bundle)

A, B = torchhd.random_hv(2, 10000)
torchhd.bundle(A, B)  # uses my_custom_bundle

I think this could be very useful for research purposes.
By default the code as users write it with the current version stays the same but more flexibility becomes available because the primitives can be changed.
@rgayler could you perhaps help me compile or point to an overview of all the various versions of bind, bundle and permute that we should implement? I am looking for general descriptions of the methods such that we can implement them for the broadest range of hypervector types. For example, multiply binding is equivalent to XOR for binary hypervectors so I would like to keep these as one function whose execution depends on the hypervector type, instead of providing both torchhd.multiply and torchhd.xor.

One potential problem I thought of it that currently we also provide multibundle and multibind which are more efficient implementations when bundling or binding many hypervectors. Although they are mathematically equivalent to a for loop over bundle/bind the speed up of performing this in a single call can be pretty significant.
One way to combat this is to allow the user to override both the single and multi version of each primitive. When a user only provides the implementation for a single we can fallback to a for loop in the multi-methods implementation. And for the build-in methods the user could specify them as a string so that we can set the single and multi versions correctly behind the scene. Here are some examples that will hopefully make it more clear:
import torchhd, torch

# assuming we have randsel implemented in the library
torchhd.set_bundle_method("randsel")

hypervectors = torchhd.random_hv(2, 10000)
A, B = hypervectors

torchhd.bundle(A, B)  # randsel
torchhd.functional.multibundle(hypervectors)  # efficient multi-randsel

def my_custom_bundle(a, b):
    return torch.add(a, b)

# user only specifies custom single bundle method
torchhd.set_bundle_method(my_custom_bundle)
torchhd.bundle(A, B)  # uses  torch.add
torchhd.functional.multibundle(hypervectors)  # will use a for-loop with torch.add

def my_custom_multibundle(a):
    return torch.sum(a, dim=-2)

# user specifies both a custom single and multi bundle method
torchhd.set_bundle_method(my_custom_bundle, my_custom_multibundle)
torchhd.bundle(A, B)  # uses  torch.add
torchhd.functional.multibundle(hypervectors)  # uses torch.sum

One potential problem I thought of it that currently we also provide multibundle and multibind which are more efficient implementations when bundling or binding many hypervectors. Although they are mathematically equivalent to a for loop over bundle/bind the speed up of performing this in a single call can be pretty significant.
One way to combat this is to allow the user to override both the single and multi version of each primitive. When a user only provides the implementation for a single we can fallback to a for loop in the multi-methods implementation. And for the build-in methods the user could specify them as a string so that we can set the single and multi versions correctly behind the scene. Here are some examples that will hopefully make it more clear:
import torchhd, torch

# assuming we have randsel implemented in the library
torchhd.set_bundle_method("randsel")

hypervectors = torchhd.random_hv(2, 10000)
A, B = hypervectors

torchhd.bundle(A, B)  # randsel
torchhd.functional.multibundle(hypervectors)  # efficient multi-randsel

def my_custom_bundle(a, b):
    return torch.add(a, b)

# user only specifies custom single bundle method
torchhd.set_bundle_method(my_custom_bundle)
torchhd.bundle(A, B)  # uses  torch.add
torchhd.functional.multibundle(hypervectors)  # will use a for-loop with torch.add

def my_custom_multibundle(a):
    return torch.sum(a, dim=-2)

# user specifies both a custom single and multi bundle method
torchhd.set_bundle_method(my_custom_bundle, my_custom_multibundle)
torchhd.bundle(A, B)  # uses  torch.add
torchhd.functional.multibundle(hypervectors)  # uses torch.sum

I'm working in the feat-primitive-override branch and coded the basic functionality to be able to override the bind, multibind, bundle, multibundle, and permute methods. I still have to make it more robust, figure out how inverse bind fits into this design and update documentation and tests.

could you perhaps help me compile or point to an overview of all the various versions of bind, bundle and permute that we should implement? I am looking for general descriptions of the methods such that we can implement them for the broadest range of hypervector types. For example, multiply binding is equivalent to XOR for binary hypervectors so I would like to keep these as one function whose execution depends on the hypervector type

@mikeheddes there are a few survey papers around (I'll get pointers to the ones I know) that list multiple instantiations of the generic operators. From memory, these don't make a point of highlighting which instantiations are special cases of others. FWIW binding with XOR and bipolar multiplicative binding can both be seen as equivalent to complex multiplicative binding where pahse angle has been quantised to two levels.
I understand why it's attractive to minimise the number of function names and dispatch on argument type, but if it turns out that's too hard I don't think it would be a tragedy to have a bunch of related instantioations of primitive operators and just note the relationships in the documentation.

These are the obvious recent surveys:
https://doi.org/10.1145/3538531
https://doi.org/10.1007/s10462-021-10110-3
But you already know about them because you've cited them in the TorchHD paper.

@rgayler thank you for the papers, I wanted to make sure there are no important ones that we were not aware of yet. I will try to make a table where I group special cases of methods. Then we can refine that before we implement them.

I can foresee that once we have a broader range of VSA/HDC models implemented we could add a method like set_model_primitives("MAP") which would internally call set_bundle_method, set_bind_method etc.


Run script for 2011 Eng Wales and check with new data

Been digging into this throughout the day and saw that 48,000 rows are NA! Not sure why this is, it seems to be case in the raw data. Apart from that, the pipeline does run as it should.


CSP srcpt-src unsafe-inline

Describe the bug
Hello everyone,
I am currently trying to switch from react-scripts to vite. Unfortunately, I now have a problem with the CSP. My CSP looks like this:
<meta http-equiv="Content-Security-Policy"
      content="block-all-mixed-content;
        base-uri 'self';
        form-action 'self';
        default-src 'none';
        script-src 'self' 'report-sample';
        style-src 'self' 'unsafe-inline';
        font-src 'self';
        worker-src 'self';
        manifest-src 'self';
        frame-src 'self';
        connect-src 'self';
        img-src 'self';
        media-src 'self'">

The problem here is script-src 'self' 'report-sample', because vite unfortunately creates inline scripts in the index.html file:

Is there a way how to not generate those inline scripts? We do not want to allow inline scripts in our app in order to protect it better.
Reproduction

Steps to reproduce

Create a project with vite.
Add the above csp to the header
Run the vite project
A blank page will be shown.

System Info
System:
    OS: macOS 14.1.2
    CPU: (8) x64 Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz
    Memory: 4.84 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.4.0 - /usr/local/bin/node
    Yarn: 1.22.21 - /usr/local/bin/yarn
    npm: 10.2.5 - /usr/local/bin/npm
  Browsers:
    Chrome: 120.0.6099.109
    Safari: 17.1.2
  npmPackages:
    @vitejs/plugin-legacy: ^5.2.0 => 5.2.0
    @vitejs/plugin-react: ^4.2.1 => 4.2.1
    vite: ^5.0.10 => 5.0.10

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

The inline scripts were created due to the legacy plugin.

I have the same problem. Please help to find a proper way to configure Vite so it doesn't create inline scripts/styles so CSP meta tags could work.

plugin-legacy depends on the inline scripts. You need to add the corresponding hashes to your script-src list.
https://github.com/vitejs/vite/blob/main/packages/plugin-legacy/README.md#content-security-policy


Having a denyUpdate() and denyInsert() validators

Since they would be used for many reason, like for example not permitting to change a userId after the first insert that would have been done server side, i think denyUpdate() and denyInsert() could be part of the core astronomy validators. They could look like this :
Astro.createValidator({
    name: 'denyUpdate',
    validate: function(fieldValue, fieldName, options) {
        if(this._id && _.has(this.getModified(), fieldName)) {
            return false;
        }
        return true;
    }
});

What do you think ?

I think it's not the role of validator. It's more like application logic. However if you want such validator you can always write your own :-). As I said you should rather implement it outside of Astronomy internals.

Hum, i'm not sure i see why it should be outside of a validator, since it's just to validate that a value hasn't changed...Especially if you use like an update directly from the client instead of using a Meteor.methods . But i do see your point as we can implement such things into the Allow/Deny that meteor provides. I just find it easier to directly put it inside the schema.

A validator should check a value agains certain rules you specified. Checking if a value had been modified is not checking value itself but it's document's or in this situation field's state. So that's why it should be outside the validator.
Meteor.methods({
  '/update/item': function(doc) {
    if(doc._id && _.has(doc.getModified(), fieldName)) {
      throw new Meteor.Error('Can not change it');
    }
  }
});

In fact, you should do something like this (it works only in the newest version).
Meteor.methods({
  '/update/item': function(doc) {
    if (doc.validate()) {
      doc.save([
        'first',
        'second',
        'third'
      ]);
    }
  }
});

However, I'm thing about some replacement for my and your solution. How is it made in other databases and tools? Do you have some example? How do "mark" field as unchangeable?
Maybe it's a role for a behavior that would use e.preventDefault() in the beforeSet event.
events: {
  beforeSet: function(e) {
    if (this._id && e.field === 'fieldName') {
      e.preventDefault();
    }
  }
}

I have not implemented preventDefault for the set events but I will :)

Well, i do get the idea when using a methods. But if i want to use the save() function client side, then it's not really possible, has it would be possible to override the validation. And in this case, i would have to either implement the idea in my Collection.allow() which i don't really like or check it against a Validators which i'm currently doing with the one on my first post. Now, because of LazyCompensation, it maybe better to actually implement everything with methods directly...
As for others tool and DB, i don't think any DB actually implement this denyUpdate() idea, has it's not really it's role (but i may be wrong, to be checked...). Homewer, in other tools, and most notably to one you know about which is Collection(@aldeed), there's simply two properties called denyUpdate and denyInsert with a boolean set to false by default, like this:
createdAt: {
    type: Date,
    denyUpdate: true
  }

Maybe it should be an extra option in the field definition. I would rather call it updateable. And I don't see any reason for denyInsert property. For what I've read in the simple schema it's like optional. And we have the required property.
Item = Astro.Class({
  name: 'Item',
  collection: Items,
  fields: {
    createdAt: {
      type: 'date',
      updateable: false
    }
  }
});

I will add this to the features list however it won't appear in the next version because even though I have a lot of work and it would post pone the release even more :)

Well, it's definitely a trivial feature anyway, has we already have multiple ways to implement it ourselves. It's sort of a convenience feature, so not having it like on the next version wouldn't be a drama :D .
updateable is perfectly fine for me, it's even more in accordance with the way your seing the feature so...
And i've actually never used the denyInsert() idea, apart from ensuring something like making sure that the updatedAt doesn't get set on insert, which is kind of an non-issue and with your timestamp behaviors, it's just, well, useless... Now there may be some reason behind this idea, i'll try to find out if it could actually be useful in some situation.

Ok so we will come back to discuss it when I start implementing it.
And of course if you find any use case for denyInsert then let me know :)

Someone wanted these in collection2 and I added it. Personally I think it's better to handle all security in a single place, and not in a schema. But there is some gray area, for example preventing createdAt from ever being updated is more of a schema/data integrity thing than it is security. It might make more sense with a name that doesn't imply security, like immutable: true.

You're probably right in my case scenario of forbidding to update an userId field. But i feel it's also convenient to use it this way, and i do feel that in those rather simple but important cases, handling it directly at the source (schema) gets you a clearer view of your data.

Yes, the immutable name describes its usage very well.
I think that need for the denyInsert was caused by somebody's wrong application logic. I've never seen such thing before but it doesn't mean that I'm right.

The immutable property implemented. Will appear in the 1.0 release.


04 Awari – Ruby

Ruby port of Awari.
Attempts to mimic the input handling of Vintage Basic.

Thank you! 💎


fix(legacy): add useCaseSensitiveFileNames wherever needed

Summary

Fixes #3665
Test plan

No idea how to test this, but expect CI to be green
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 93.971%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
1119

Relevant Lines:
1156

💛  - Coveralls


[Sleek] Cannot change theme color

Describe the bug
Cannot change theme color
To Reproduce
Steps to reproduce the behavior:

Install Sleek from the Marketplace
Use the powershell line to change the theme color,
Apply
See error

Expected behavior
Theme changes color preset.
Logs
Add logs from console. To do that

Run spicetify enable-devtools in terminal
Spotify will be restarted
Hit Ctrl + Shift + I to open DevTools window
Navigate to tab Console
Copy console window content.

Specifics (please complete the following information):

OS: Windows 11
Spotify for Windows <IP_ADDRESS>4.g540a4cc6
Spicetify version v2.13.1
Sleek

I suggest you use the Marketplace to install the Sleek theme as it's available there.

I used the marketplace, but I cannot change the color.

And that's the reason you cannot change your color scheme: to change a marketplace installed theme you must do it from the top right dropdown menu in the marketplace section.


Errors in the KV view/editor

Hello there !
There's some errors in the console when I go to the KV page of the admin panel. They seem to be related to cloudflare.
I don't know if it's related to my personal case neither if it's some serious errors, but I wanted to tell you !

Could you hard reload the app?

You mean reloading with Maj + Cmd + R (firefox on macOS) ? It doesn't change anything
Actually the errors appear as soon as I go the the admin panel, even before I select a project. Noticed that it doesn't affect how well the app runs !

Oh, nevermind. I thought I disabled all the blockers with firefox and ublock origin to be sure it wasn't on my side, but I forgot the duckduckgo extension which was not in the toolbar and still on...
I'm sorry for the false alert !!


_routerExtensions.back() causes crash after closing modal

Environment
have only tried on android
Describe the bug
After closing a modal, back navigation via routerExtension doesn't work resulting in a runtime error.
To Reproduce
Open this nativescript playground

using router, navigate from component1 to component2
open a modal inside component2
close the modal
click back to attempt to navigate back from component2 to component1
runtime error says: ERROR TypeError: Cannot read property 'isPageNavigationBack' of undefined

Expected behavior
The back navigation should take you back to component 1
Additional context
This may be related to #1350

Hi @jeffswitzer,
I was able to recreate the issue while using your demo app and will mark this problem as a bug. It seems that there is an issue with the router-outlet. As a workaround, I would suggest using page-router-outlet. Check out the example here.


Fixed Pdu.as_reply_to/2 type spec.

The function should also take RawPdu.t() to form a response to an unparsed request.

@savonarola The checks didn't pass. But my fix shouldn't have affected this. Could you restart the checks please?

Thanks!


Pivotal Cloud Foundry: Service Broker

Background: https://airtable.com/tblD3oZPLJgGhCmch/viwdV1sM5KEK2L14K/reccIOeC6KPngbPWC
PRs:

No docs needed on our site for now.


Fix blame on install deps

k.  Magically, that still places me as contributor number 5 :-D  Anyway, I'm sure it all averages out over time.


helmenv to v1.0.2

Codecov Report

Merging #147 (9ed52f8) into main (5a21a5e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #147   +/-   ##
=======================================
  Coverage   11.34%   11.34%
=======================================
  Files          11       11
  Lines        1402     1402
=======================================
  Hits          159      159
  Misses       1227     1227
  Partials       16       16

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 5a21a5e...9ed52f8. Read the comment docs.


[BUG] Subsume instance creation failure causes underlying target resource to be deleted

It seems that when the creation of a subsume instance fails, it causes the broker to trigger a deprovision which actually deletes the underlying DB that was being subsumed. I assume it does this is a way to rollback partially created resources when something fails which makes sense for new resources, but not when subsuming.
I saw this happen when the subsume instance failed with the following error:
Service [rates-sql-uat] creation/update failed: Error: Error waiting on create/update future for SQL Failover Group "rates-sql-uat-failover" (Resource Group "MFC-CAC-PCF-PSB", Server "azugessqlprod"): Future#WaitForCompletion: context has been cancelled: StatusCode=200 -- Original Error: context deadline exceeded on main.tf line 56, in resource "azurerm_sql_failover_group" "failover_group": 56: resource "azurerm_sql_failover_group" "failover_group" { exit status 1

I think a solution would be for subsume instance creation to not trigger a deprovision when it fails.

Ugh. This is automatic behavior by terraform when a step in the process fails and I don't immediately see a way to prevent it at the moment. Investigating.

Little more investigation here - this behavior is limited to fail over group subsume (since it imports multiple resources.) Terraform believed it had successfully created the primary and secondary db and then couldn't create the failover group and rolled back. Still looking for way to tell terraform not to do rollback.

Root cause here is that applying any change to a secondary db causes the secondary DB to be re-created. Issue opened and active in terraform azurerm provider - https://github.com/terraform-providers/terraform-provider-azurerm/issues/11024

Just to clarify the issue here, the fact that Terraform recreates the secondary DB causes it to delete the other resources as well when rolling back due to a failure?

Tested the terraform provider fix and unfortunately it's still failing. The provider is expecting an extra parameter to be passed in on updates to the secondary, otherwise it fails. When passing in the parameter, it goes back to recreating the database.
Raised a new issue with them here.

This is fixed now in the latest release of the Terraform Azure provider, 2.59.0

Thanks @aristosvo, this worked fine.
Many thanks for all your help!

Work to consume the new provider version and other changes needed on our side has been completed. This fix will be included in our next release.

Hi @erniebilling,
Shall we reopen this case as we found the latest release doesn't resolve issue 53 please?
Thanks a lots.


Mapping /var/lib/ldap to a local directory starts container with empty data

docker run --rm \
           -v ~/OpenLDAP/ldap:/var/lib/ldap \
           -p 10389:10389 \
           -p 10636:10636 \
           -d \
           rroemhild/test-openldap

This is because the LDAP database is initialised when the image is created, rather than when it is booted for the first time. The idea behind this is that this Docker image is used to create a working LDAP for unit testing etc, not to store the data persistently.


Typing error on release page (v20220924)

There is a typing error on contribution
my id is @YukihoAA  not @ yuki
😂

updated :P
Somehow the Author name show up as Yuki in git log

Thanks you ><


Remove complex iDOM wrapping code

We wrap A LOT around Incremental DOM. The notable parts are:

Better skipping
Stateless functional components (function helpers)

Skipping isn't really necessary, especially if Incremental DOM now ignores trees that never had any nodes in the first place.
Stateless functional components are more complex. I think we should simplify this as much as possible, but still be able to pass functions as tag names, but we should simplify the logic as much as possible, even if it changes the API slightly.

Sounds great. It is really hard to extend the iDOM functionality.

Replaced Preact with iDOM. See referenced PR for details.


Got Task Failed error

Tried out your 3.1.1 version of Sql4Dclient with the sample data, using the simple INSERTs. The .csv file was successfully created, and mysql duid_tasks table registered and showed the tasks.  But it returned 'Task Failed' message.  Wasn't able to pinpoint any type of issues.  I only ran zookeeper and the overlord server, should I run the broker, coordinator and historical servers too?  Any pointers will be appreciated.

Go through this blog http://druidwithsql.tumblr.com/post/108054375927/simple-insert-deleting-data-drop-table-in-druid . Let me know if you still face issues. BTW you can use trace=true command to enable debugging in the Slq4DClient. If you want to really see what happened with your task use Overlord console.

Thank you.  This is the log that I got with 3.1.0 when trace=true and the error
Http org.apache.http.client.HttpResponseException: Not Found
nutch@ubuntu:~/Sql4D-old$ java -jar lib/Sql4DClient-3.1.0.jar -bh localhost -bp 8080 -ch localhost -cp 8082 -oh localhost -op 8083 -i 50

trace=true;
INSERT INTO adf (timestamp, provider, title, uuid, DOUBLE_SUM(click) AS click) VALUES ('2014-11-01 00:00:00', 'siri', 'FE', 'adb-dfgdf-32de1', '5') WHERE interval BETWEEN '2014-11-01' AND '2014-11-02' BREAK BY 'day';
Written to /tmp/022735a0-a9c8-4533-8446-ea07be4a5611.csv
{
"granularitySpec": {
"gran": "day",
"intervals": ["2014-11-01/2014-11-02"],
"type": "uniform"
},
"dataSource": "adf",
"firehose": {
"baseDir": "/tmp",
"type": "local",
"parser": {
"timestampSpec": {
"column": "timestamp",
"format": "yyyy-MM-dd HH:mm:ss"
},
"data": {
"columns": [
"timestamp",
"provider",
"title",
"uuid",
"click"
],
"dimensions": [
"provider",
"title",
"uuid"
],
"format": "csv"
}
},
"filter": "022735a0-a9c8-4533-8446-ea07be4a5611.csv"
},
"type": "index",
"aggregators": [{
"name": "click",
"fieldName": "click",
"type": "doubleSum"
}]
}

Written to /tmp/b308773c-7904-4ea7-91ec-756dc0ee7e70.csv
Http org.apache.http.client.HttpResponseException: Not Found
(0.641000 sec)

The 3.1.1 version of client, on the other hand, gave a different type of errors, likely with json parsing issues.

Sometimes got different errors:
Http java.net.SocketTimeoutException: Read timed out.
Could be related to access/availability of those servers' listening ports...

If you followed the blog i sent you then the option should be
-op 8087
and not -op 8083.

Thank you for the quick response.
I changed to druid 0.6.154 and kafka 0.7.2 and it is working:
druid.extensions.coordinates=["io.druid.extensions:druid-kafka-seven:0.6.154"] instead of
druid.extensions.coordinates=["io.druid.extensions:druid-kafka-eight:0.6.160"]
The sql4dclient is 3.1.1.
Cool.
Thanks for the good work!


Support latest GoogleSignIn ios SDK

Issue
The GoogleSignIn Podfile was upgraded to 5.0.0 recently, which appears to have renamed (or deleted?) a few things. Attempting to compile<EMAIL_ADDRESS>against this version results in the following error:
.../node_modules/react-native-google-signin/ios/RNGoogleSignin/RNGoogleSignin.h:11:74: Cannot find protocol declaration for 'GIDSignInUIDelegate'; did you mean 'GIDSignInDelegate'?
Environment
iOS

ARE U FIX THAT?

same problem!

Possible to find a documentation, to what has changed?

this answer fix my issue

here's the changelog https://developers.google.com/identity/sign-in/ios/release
if someone wants to migrate the apis to v5, feel free to contribute! Thanks.

For people who landed here and don't want to update react-native-google-signin you can head to your podfile and force GoogleSignIn to use the older version for now
pod 'GoogleSignIn', '4.4.0'

done in v3

if someone encounters the error, removing the ios/build folder or cleaning the project should help


Add "Ctrl + W" shortcut for closing the app

Problem Statement
The Calculator app doesn't have the keyboard shortcut to close the app. Adding the keyboard shortcut will make the app useful.

Evidence or User Insights
At this time, users have to use the close button at the top of the window to close the Calculator app. Many apps running on Windows have Ctrl + W shortcut to close the app.

Proposal
Please add Ctrl + W shortcut for closing the app.

Goals
Users can use Ctrl + W shortcut to close the Calculator app.

Non-Goals

Low-Fidelity Concept

Requested Assignment

I'm just suggesting this idea.  I don't want to implement it.

[Alt]+[F4] is still available. It is the universal shortcut that we can use to close any app on Windows, even Windows.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

'Alt + F4' is the standard shortcut to close applications on Windows. 'Ctrl + W' is used in some applications to close tabs or documents.


[doc] Add visual section header anchors in HTML documentation

Adding a visual element to represent the anchor in (sub)section headers makes it easier to link to specific sections of HTML documents. For instance, GitHub does this by displaying a link icon when you hover over a section:

IMO, we should do the same or something similar for the OpenTitan HTML documentation.

Closed by PR #13292.


Advanced queries not working

I have been trying to work with advanced queries on the SearchTimeline and that doesn't seems to work, thing is, a query like this one #twitter near:"alabama" within:15mi won't return anything, but if you go to the advance search page of twitter it does returns some tweets.
Is there anything I'm missing? I have tried that with and without UTF-8 URL encoding and is just not working. Wondering if is on the API side...

You cannot really compare the results from API with Twitter Web. They are different systems and can return different results. I think you will get a better response by posting on twittercommunity.com

Thanks for the help, I found that the "near" operator is not available on the REST api hence the lack of results with the library.
I'm closing this one, agains thanks for the response.


Use right operation for creating exception string

Otherwise your exception will like this one
RuntimeError: ('recvmsg failed: rv=%d', -1)

Hi,
Before accepting your contribution, I need to make sure you have signed the Apache CLA. Have you (as an individual) or your company signed the CLA? Please send me an email with your name and affiliation at<EMAIL_ADDRESS>

Thanks
On Thu, Jun 23, 2016 at 5:27 PM, Antonin Bas<EMAIL_ADDRESS>wrote:

Merged #19 https://github.com/p4lang/ptf/pull/19.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/p4lang/ptf/pull/19#event-702649795, or mute the
thread
https://github.com/notifications/unsubscribe/ABr779bVQapiJtSeR8Ox2THOMYWaULv7ks5qOyR5gaJpZM4I2o03
.

--
With best regards, Pavel S. Shirshov


Review ACCESS_MASK for accuracy

See @hmemcpy's feedback on gitter

Work in progress in the fix194 branch.

Interestingly, in adjusting other enums that define ACCESS_MASK-like values, I found Kernel32+FileAccess. But NtDll already references Kernel32 so it can't also go the other way around.
Circular dependency.

I'm going to move ACCESS_MASK into Kernel32. Problem solved.


Support Matrix for Dynamics CRM apps

Supported Dynamics CRM versions that can be migrated using ASR ? I can see Dynamics CRM 2011 is not supported. Is there a way to migrate 2011 versions ? Can you create a support matrix and reasons for not supporting.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f2eca019-3dc3-a07a-3ff5-a3b67eb437b0
Version Independent ID: 68aa4031-bcb2-500e-764a-71b274d4baad
Content: What workloads can you protect with Azure Site Recovery?
Content Source: articles/site-recovery/site-recovery-workload.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@msazurelearn Thanks for your comment. The feedback has been shared with the content owner for further review. Thanks for your patience.

The versions supported by Site Recovery are up-to-date in the article. In this article we state what's supported rather than reasons for what's not supported.
Thanks!
#please-close


Update example in the README.md

Let me know if this should be the default. It seems like a reasonable choice since running a redis sentinel cluster locally for testing is not that straightforward.

I don't understand where "a redis sentinel cluster" comes from, but I want to keep this example as simple as possible. Thanks for trying to improve readme anyway.


Karsten/slack app

Basic slack app with some instructions.
NOTE:

Not production ready, just development. Will need to add commands and code for prod
Look at slack events and web APIs for ways to subscribe and respond to events

PR to wrong repo.


.ini file is not loading when launching the application via appium

Our desktop application has .ini file that contains the basic settings that are loaded when launching the application.
When I manually launch the application, settings in .ini file is getting loaded where as when I try to launch the application via appium desired caps, the settings are not loaded.
Appium version: 1.22.3
desired_caps = {
"deviceName": "",
"platformName": "Windows",
"app": ""
}

2 things to try

Try including another capability of "appWorkingDir" and then specify the parent folder that your application.exe is found in. Maybe that will pull in those settings? Havent had to use .ini files for our application but I do know that I needed to include this capability as well as the "app" capability.
Are these essentially just optional command line arguments? You can add these with the "appArguments" capability

Great!! Thanks @Tree55Topz  @Shakevg . Adding the appWorkingDir works for me.
{
"deviceName": "",
"platformName": "Windows",
"app": "",
"appWorkingDir": ""
}


re-establish connection to hub upon disconnect/reconnect

Correctly re-establish connection to hub upon disconnect/reconnect
Small logging tidy up

Thank you Luca!


Provider response download fails when userdata is passed. file_name is None

Describe the bug
When an onchain asset (function call) is requested, the download fails. The error happens because although the provider returns the contract data, the file_name is None.
WARNING:ocean_lib.data_provider.base:It was not possible to get the file name. Expected type '<class 'str'>' for attribute 'header' but received type '<class 'NoneType'>')
raise TypeError(TypeError: Expected type '<class 'str'>' for attribute 'file_name' but received type '<class 'NoneType'>')

To Reproduce

Download an onchain asset that pass parameters to the function call

@calina-c Can you make sure the fix is included in the brownie branch as well?

@jfdelgad it seems to me like this is a duplicate of #1031. Can you confirm? Also, did you do any fix related to the typing error? I can't seem to find it although it was discused in the sprint planning meeting.

@calina-c This has been already included in the main. There were two things, on change in the ocean provider repo, made by Alex and a change in ocean made by me. The type returned is still NoneType for contract functions with parameters, as Alex indicated that there are not file_name for contract functions, so ocean.py manages this particular situation.

here is the PR made and merged to main: https://github.com/oceanprotocol/ocean.py/pull/1050

Merged to main-brownie: https://github.com/oceanprotocol/ocean.py/pull/1055

Closing since the fix is done in both main and main-brownie.


feat: Add util files to Keptn delivery

A few Keptn use-cases require some additional files in the Helm chart and it would be great if these were here by default when cloning the repository. Another helpful file to add would be a rollouts.yaml Argo file.
If you think that these files should not be added, just let me know and I will search for another solution.

Currently not needed anymore therefore, I will close the PR.


feat(modules): infer modules types in config

🔗 Linked issue

fix #2685
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Hello 👋,
This feature uses typegen to add modules types within the NitroConfig interface to allow autocompletion in the nitro.config.ts file.
With the following module:
interface ModuleOptions {
  bar: string;
  foo: {
    baz: string;
  };
}

export default defineNitroModule<ModuleOptions>({
  name: "playground",
  configKey: "playground",
  setup: (nitro, moduleOptions) => {},
});

TypeScript will help you with autocompletion when you are configuring the module in the nitro.config.ts file:
import { defineNitroConfig } from "nitropack/config";

export default defineNitroConfig({
  playground: {
    bar: "bar",
    foo: "baz",
  },

  compatibilityDate: "2024-06-12",
});

Of course, the foo property should be an object, and TypeScript will raise an error.

📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

Hi, thanks for the PR i appreciate it but at the current state, i prefer to not depend more on auto generated types. it can both make conflict with higher level framework types and user experience.

Have you any plans to improve the DX of modules? Is it possible to help?

Not until we officially release Nitro Kit and I think i am more likely to recommend direct module imports (like vite plugins that accept options inline) and not nuxt style. but have to think more about it.


[FEATURE] Mode for moblie usage / Low Data Mode

Is your feature request related to a problem? Please describe.
Too much data usage when ure using Amethyst.
Describe the solution you'd like
Add an mode which reduced the data consumption .
( no vids ,pics etc. )
Regards

Just go to settings and choose to never show videos and pictures :)

Oh damn ,ive overlooked this.
May a  "flightplane" button which u can switch the mode on/off on ur own is an idea.
Regards


HBASE-23116: Fix a couple of load balancer logging nits.

Fixes a typo and adds table name to the logging context when
StochasticLoadBalancer is configured "per table". Added some
test coverage with per-table balancer enabled and manually
verified the logs to make sure the table name is formatted
correctly.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
0m 37s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  0s
No case conflicting files found.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  0s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

:green_heart:
mvninstall
7m 12s
master passed

:green_heart:
compile
1m  0s
master passed

:green_heart:
checkstyle
1m 29s
master passed

:green_heart:
shadedjars
4m 49s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
0m 41s
master passed

:blue_heart:
spotbugs
4m 39s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
4m 35s
master passed

_ Patch Compile Tests _

:green_heart:
mvninstall
6m 31s
the patch passed

:green_heart:
compile
1m 10s
the patch passed

:green_heart:
javac
1m 10s
the patch passed

:broken_heart:
checkstyle
1m 34s
hbase-server: The patch generated 2 new + 170 unchanged - 0 fixed = 172 total (was 170)

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:green_heart:
shadedjars
5m 39s
patch has no errors when building our shaded downstream artifacts.

:green_heart:
hadoopcheck
19m 25s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

:green_heart:
javadoc
0m 39s
the patch passed

:green_heart:
findbugs
4m 58s
the patch passed

_ Other Tests _

:broken_heart:
unit
168m 40s
hbase-server in the patch failed.

:green_heart:
asflicense
0m 34s
The patch does not generate ASF License warnings.

237m 19s

Reason
Tests

Failed junit tests
hadoop.hbase.security.access.TestSnapshotScannerHDFSAclController

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/687

JIRA Issue
HBASE-23116

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 977c3daa96bb 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-687/out/precommit/personality/provided.sh

git revision
master / 5aa8d3a20b

Default Java
1.8.0_181

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/1/artifact/out/diff-checkstyle-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/1/testReport/

Max. process+thread count
4480 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

LGTM, +1. Also ran this previously failed TestSnapshotScannerHDFSAclController and it's passing locally, so it should be unrelated. Will merge this PR.

There are fix versions for 1.x on the JIRA but no commits to branch-1s. Is there a backport in progress?

Answering my own question - I am doing the backport now. Trying to get 1.5.0 RC out.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
1m 10s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  0s
No case conflicting files found.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  0s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

:green_heart:
mvninstall
5m 21s
master passed

:green_heart:
compile
0m 56s
master passed

:green_heart:
checkstyle
1m 21s
master passed

:green_heart:
shadedjars
4m 36s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
0m 37s
master passed

:blue_heart:
spotbugs
4m  3s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
4m  1s
master passed

_ Patch Compile Tests _

:green_heart:
mvninstall
4m 58s
the patch passed

:green_heart:
compile
0m 53s
the patch passed

:green_heart:
javac
0m 53s
the patch passed

:green_heart:
checkstyle
1m 19s
the patch passed

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:green_heart:
shadedjars
4m 34s
patch has no errors when building our shaded downstream artifacts.

:green_heart:
hadoopcheck
15m 33s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

:green_heart:
javadoc
0m 37s
the patch passed

:green_heart:
findbugs
4m 25s
the patch passed

_ Other Tests _

:broken_heart:
unit
307m 57s
hbase-server in the patch failed.

:green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

365m 17s

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestSnapshotCloneIndependence

hadoop.hbase.regionserver.TestRegionReplicasWithModifyTable

hadoop.hbase.master.TestSplitWALManager

hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas

hadoop.hbase.client.TestAsyncTableAdminApi

hadoop.hbase.client.TestFromClientSide

hadoop.hbase.client.TestFromClientSideWithCoprocessor

Subsystem
Report/Notes

Docker
Client=19.03.2 Server=19.03.2 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/687

JIRA Issue
HBASE-23116

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 3210a8a8ed2d 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-687/out/precommit/personality/provided.sh

git revision
master / 800c35a30e

Default Java
1.8.0_181

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/2/testReport/

Max. process+thread count
4782 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-687/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.


⚠ Cioos Atlantic CKAN has degraded performance

In 4ea1756, Cioos Atlantic CKAN (https://catalogue.cioosatlantic.ca/) experienced degraded performance:

HTTP code: 503
Response time: 18163 ms

Resolved: Cioos Atlantic CKAN performance has improved in 2d6e687.


JAVA_DEBIAN_VERSION is too old

The JAVA_DEBIAN_VERSION in the file for "java/openjdk-8-jdk/Dockerfile" is currently "8u66-b01-1~bpo8+1". This does not work for me any more, as it results in an error from apt complaining of no package being found.
Updating this to '8u66-b17-1~bpo8+1' worked for me.

Not sure what you mean; the value you listed as working is what's there currently: https://github.com/docker-library/java/blob/master/openjdk-8-jdk/Dockerfile#L16
(as of 200ecf22e5a23cb48cbb3ce47aa08aa3b49a0d2d, ~16 days ago)

I somehow must have been on an old commit, as I still saw the old version. Looks like this my my mistake!

Cool, glad we got it figured out! :+1:


ReactiveProperty dispose behavior

ReactiveProperties are Disposables. As soon as its reference is out of scope it becomes available for deletion. Therefore, it calls its dispose-method, sending on-completed-notifications to each listening observer in the process. The same goes with the readonly counterpart.
The problem is that Godot4-style propeties (get-set syntax) returning a ReactiveProperty view (get: return _prop.to_readonly()) are problematic as the getter effectively creates a new ReadOnlyReactiveProperty instance on the ReactiveProperty, which, in theory, is totally fine. However, this means that subscriptions on the returned value are immediately closed if the ReadOnlyReactiveProperty is not stored somewhere.
I do not know if this is expected behavior when dealing with ReactiveProperties but it is very unintuitive in Godot and causes the need for an additional property variable to be created. The view should likely not be disposed as long as its underlying Observable lives.

Okay, I managed to find an answer to this: Since a ReadOnlyReactiveProperty is still as the name implies a 'property' it should not exist without the underlying instance which created it. Thus, returning a new RORP in a GD-Script-style set-get-property is probably a no-no.
The good news: I have refactored ReactiveProperties and ReactiveCollections to have proper interfaces. Additionally, GDRx features from now on ReactiveSignals similar to GodotSx (Why did I not come up with this earlier?). Reactive Signals are super useful. I have used their use case so many times in my project and now they are finally featured.
These changes are still on a separate branch. I will test the refactor in my own project a bit. Should everything be find, there will be the merge into 4.1.


Compatible with Grid Layout?

Hello,
First, thank you to sharing with us this great project, this is very useful!
Now I would like to apply the same kind of process but for elements inside of a grid layout that is inside a VerticalScrollRect.
Basically, it the almost same thing instead of I have more than one element by row, and it doesn't seem to work well from now.
I will try to update the code to make this possible, but if you have any advice of how you would do it, it will be great! 👍

Hello @Noxalus,
Thanks for the feedback.

If you can share your project/code, it would be great.
Thanks again!

Sorry for the delay, here is what I would like to do:

Here is my fork with the corresponding code.

No problem @Noxalus,
I understood your need. I will try to do it tonight.
Once I am done, I will let you know here.

😮
Thank you very much for your quick answer! 👍

Hi there, awesome library indeed but I was about to do the same post as @Noxalus! Has this being progressed at all?

hey @psarras
I am sorry, but I still didnt had time to work on this.
But you can achieve the same "effect" by making each element of the list be a list on its own.
So lets say you have a vertical list, so then you can make each element of it a horizontal list. By doing so, you have a grid.
I will try to have a look at this in the near future, but I can promise.
Also, feel free to send pull requests if you find a good solution.
Thanks!


Renderer doesn't work with core node/electron modules

I'm new to bankai/choo so apologies if I missed something obivous.
When I try to do something like require("fs") then it fails. I can work around it with window.require("fs"), is this the recommended way of doing it or is there a better way?

perhaps we should include https://github.com/blahah/electron-renderify


Idea: Saving pfx to keyvault

This is exactly what i'm looking for.
An enhancement I'd love to see is saving the pfx to keyvault along with the password.
Awesome work!

Good idea, I could add a setting for it so the user can decide if saving to keyvault or not.
Thanks :)

may I ask if there are any updates on this topic as I would love to see this feature too

@hendriksteinhorst I still have plans to add that to the service, but I've been super busy and haven't had enough time to do this properly. As you can imagine it's a feature that needs to be done with the right amount of attention to be sure everything works properly.
I'm gonna do it as soon as I can.

I have done it manually recently to use the certificate in Azure Front Door. Here is what I learned:

Azure Front Door accepts only certificates with RSA private key (see line 145 in CertificatesHelper.cs)
the certificate password must be empty, otherwise Azure Front Door can't import it

I don't know the security implications of these changes as I'm no expert in certificates.


Type 'DecoderCallback' not found while build app on flutter version 3.16.0

Type 'DecoderCallback' not found while build app on flutter version 3.16.0

To Reproduce
Steps to reproduce the behavior:

Update flutter version to 3.16.0
Run app on Android

https://github.com/ScerIO/packages.flutter/issues/448

#448
Got it. Thanks for your reply


WIP: Convert remove clone flag to ample

Converted the remove clone flag to use ample for metadata actions.
Most of the clone operations in MetadataTableUtil are not using ample and will need to be converted.
Due to the amount of code changes, I'm keeping these PRs small to hopefully aid in reviewing.

Is there more work being done here? Is this still WIP?

Is this still valid?  At this point it may be better to target 3.1 for these changes.

Closing this stale issue due to inactivity and lack of response. It can be re-opened if activity resumes.


Bug (NetworkManger): The key already existed in the dictionary

Recently we have been seeing the following under load:
ex_msg: The key already existed in the dictionary. | stacktrace: System.ArgumentException: The key already existed in the dictionary.
   at System.Collections.Concurrent.ConcurrentDictionary`2.System.Collections.Generic.IDictionary<TKey,TValue>.Add(TKey key, TValue value)
   at PuppeteerSharp.NetworkManager.<OnRequestWillBeSentAsync>d__44.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at PuppeteerSharp.NetworkManager.<Client_MessageReceived>d__36.MoveNext()

This PR looks at some of the causes for this. We are still testing the changes, but wanted to get this here for your visibility and thoughts also.
I've managed to:

Remove two of the dictionaries that were previously required
Optimise the way some of the dictionaries were used (ie: using TryRemove instead of Get/Remove)
Fixed up some event invocation handling
Fixed a few minor race conditions (ie: MultiMap.Add - note that there is still a problem when the list needs to grow the array, but from what I can see the default capacity should be enough to warrant not locking)

... all whilst hopefully kept the same semantics.
It'd be really nice if chrome fixed the mentioned bug and started to send back the request id, then the entire class could be handled with a single ConcurrentDictionary. I know this is not you though ;)

I'll go throw this tomorrow. But I get your point. Sometimes it sucks branching from Puppeteer's code but this makes sense.
Let's see what AppVeyor thinks about this :p

So it seems appveyor is having the same trouble I am running the test suite. Is this a known issue or more likely something that i've messed up?

@Im5tu can you pull from master?

@kblok - Done :)

I think we still have something on this test ShouldNotCrashWhileRedirectingIfOriginalRequest

It appears that we get the redirect response prior to getting the request will send portion. I haven't managed to take a look further yet - should be able to get back to resolving this tomorrow :)


change tags to be OLI compliant

Hi OSO Team,
I made a simple script to convert all tags in the projects directory to be OLI compliant. How hard would it be to make this change on your backend? OLI is gaining traction with big data teams adopting it, would be a benefit to the whole space if we can make this change here.
More infos on OLI standard: https://github.com/openlabelsinitiative/OLI
Would love to get your thoughts?
Regards
Lorenz

Cool! We definitely want to be more aligned with OLI, but this probably warrants a longer meeting.
If we change the schema in oss-directory, we'll break all of the data pipelines in OSO downstream from it. These metrics power retro funding, and other websites that have integrated with our API for metrics.
This might take some refactoring, let's figure out a good course of action that can be more incremental? let's follow up in a group chat


no such file or directory for wav file

Hello,
after I tried to solve the reCaptcha, then I got this message
[Error: ENOENT: no such file or directory, open '/var/folders/5k/103mnjwx2rncp6btbvlt5z980000gn/T/reSOLVER-9454634224111578/out.wav'] {   errno: -2,   code: 'ENOENT',   syscall: 'open',   path: '/var/folders/5k/103mnjwx2rncp6btbvlt5z980000gn/T/reSOLVER-9454634224111578/out.wav',   name: 'ENOENTError' }
and I found there's a sound.mp3 file at that path, but not the Wav file.
thanks for this awesome library.

It seems that the first part is working, but the conversion between mp3 and wav failed.
Please try the following two things:

Check if ffmpeg is installed ffmpeg -version, if ffmpeg is not installed, you can use brew install ffmpeg to install it.
Pass VERBOSE env var to enable debug mode, which will pipe the ffmpeg output to stdout. For example: VERBOSE=1 node path/to/js


Total_loss increasing

Hey. Thank you author for top notch project, been having fun playing around with it. Amazing work again.
No issues other than increasing loss value while training custom video. I read the other post about similar issue. My video is cut to 512x512, it is 25fps, ffmpeg is installed correctly with h264 codec, inference size is 1.x for 10s clip and no errors other than (ret_CHECK failure (mediapipe/gpu/gl_context_egl.cc:303) successeglMakeCurrent() returned error 0x3008; (entering GL context))gpu during data processing.

This is 6th video I train, others have been trained to 250k epoch xxx_head and ending with around 140 total_loss value from lambda_ambient.
Anything you suggest to try?
Thank you.

Thanks for your interest on our work! It's okay to see increasing validation losses, considering the low-resource we used to train the dynamic talking head. As for the total_loss, it doesn't have much meaning, since the lambda_ambient is a learnable loss weight that controll the ambient_loss.  I'd suggest you taking lpip_loss and lip_lpips_loss as the major metric. Becasue based on my experience, mse_loss cannot always reflect the training process (We typically could observe growing image quality in the validation_results as the iteration grows, while the val/mse_loss is also increasing. )

Do you think if base model been trained with bigger amount of data, would side faces be doable?

That sounds good, maybe learning a meta-checkpoint as the initialization as DFRF does would help.


chore: update<EMAIL_ADDRESS>

Only an internal build update, hence no version bump.
Bumps microsoft-build from 14437482 -> 14558299
Please Note: Empty PR for same reasoning as https://github.com/microsoft/vscode/pull/152410#issuecomment-1158390385
Fixes https://github.com/microsoft/vscode/issues/153182

Failing smoke tests are unrelated and already fail on main branch.


Scopes configured via Env

To avoid clashing definitions of Scoping properties within same JVM but different Injector containers the specific configuration of a Scoping should come and exist within the Env used to bootstrap the Injector.

Solved by #101 and #102


Windows 10 setup results in "Something went wrong" on welcome page

I've tried setting this up locally on windows 10, but unfortunately after getting docker set up and navigating to http://localhost:8000, I just see a "Something went wrong" page.
Looking at the web console, I'm getting a 500 response from the server with no other context for what's actually going wrong

It seems we have a workaround for Windows setup by disabling UI build in dev config, and run it in a separate window. Let me know if you still have an issue.


Use of process.nextTick() at db/users.js

@jaredhanson Is the use of process.nextTick() at db/users.js necessary?
It seems you have clarified the addition of nextTick on a stack overflow question.
https://stackoverflow.com/questions/20743348/passport-js-and-process-nexttick-in-strategy
Maybe we should add a comment stating the reason of usage.
A lot has changed regarding the execution of nextTick since Node v12. Should we shift to the use of setImmediate()(since nextTick gets executed immediately in the current phase itself of the event loop)?
PS: I am a beginner. I am pondering around to improve my knowledge on the subject. Cheers.

There is a comment. Perhaps it is not clear to you :) but to me it’s that it’s “faking an async action to verify it works in all cases”.
Hope that helps explain the use logic.

@Xalior I was thinking maybe we could add some comment lines in code where it is used to make it clear to everyone including beginners like me.

shrug is it the purpose of this repo to teach node generics?
How verbose a comment should be is open to endless debate, and fundamentally the choice of the comment writer :-)

I am sorry. I thought that the purpose of an example repo was to show
everyone recommended usage of Passport. Cheers.
On Thu, Mar 19, 2020, 9:08 PM D. Rimron-Soutter<EMAIL_ADDRESS>wrote:

shrug is it the purpose of this repo to teach node generics?
How verbose a comment should be is open to endless debate, and
fundamentally the choice of the comment writer :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/passport/express-4.x-local-example/issues/23#issuecomment-601248917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFBI3AS55RVSRKRELHFGHWTRII36JANCNFSM4LPJE42Q
.

@Xalior I understand that it's pretty much the basics of Node.js. The issue is that most developers(including me) are not aware of constructs like process.nextTick(). It would be helpful if we can add a one line comment to help clarify the subject.

I can't agree more that some clarification would of saved me hours of dissecting that code months ago.
One comment to suggest what was happening there would of been a great improvement to what is supposed to be an example repository.

Really? Because googling “what does node next tick do” gives a pretty reasonable answer on the first page...
If you’re not able to google and look up one line of code you don’t understand maybe you should stop moaning at free tutorials and pay someone to teach you, because you certainly do not seem to be able to do it alone!
Sent from my iPhone

On 14 Apr 2020, at 03:43, Vincent Shury<EMAIL_ADDRESS>wrote:

I can't agree more that some clarification would of saved me hours of dissecting that code months ago.
One comment to suggest what was happening there would of been a great improvement to what is supposed to be an example repository.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@Xalior It seems the behavior of process.nextTick() has changed since node v11. I was thinking of getting more knowledge on the usage of the subject from production level developers, that too of a library that is well renown world wide. It is true what you said that more information can be found via a simple google search. But the issue remains that most developers(including the authors of blogs on the web) are unaware or has poor understanding of the intricate working of node.js event loop. I just believed that by adding a good comment section in this part of the code will help enlighten everyone including me. I am not saying we should. It's just a suggestion. Cheers.
https://github.com/nodejs/node/issues/22257
https://github.com/nodejs/node/pull/22842
https://github.com/nodejs/nodejs.org/pull/1804

While all of that is technically true, none of it applies here. These operations are not timing critical.
And, to repeat myself, it is not the job of a feature, function, or framework specific tutorial to teach you different generic node code!

@Xalior I agree with you that its not the job of a feature, function or framework specific tutorial to teach anyone generic node code. It's just a harmless suggestion to document the code well so that even beginners may grasp the concepts easily and hence lead to better usage of the library itself rather than messing around in the dark. Moreover, there is prevalent unawareness about the event loop within the node.js community. Cheers.

I personally think the code already does that, I am sorry you did not
understand the comment a which is why I responded to you me ticket in the
first place, to explain the comment further.
On Tue, 14 Apr 2020 at 09:57, Jilvin Jacob<EMAIL_ADDRESS>wrote:

@Xalior https://github.com/Xalior I agree with you that its not the job
of a feature, function or framework specific tutorial to teach anyone
generic node code. It's just a harmless suggestion to document the code
well so that even beginners may grasp the concepts easily and hence lead to
better usage of the library itself rather than messing around in the dark.
Moreover, there is prevalent unawareness about the event loop within the
node.js community. Cheers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/passport/express-4.x-local-example/issues/23#issuecomment-613315304,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA4CH7NMI3K7AKXQWJ3H3LRMQQQJANCNFSM4LPJE42Q
.

Really? Because googling “what does node next tick do” gives a pretty reasonable answer on the first page... If you’re not able to google and look up one line of code you don’t understand maybe you should stop moaning at free tutorials and pay someone to teach you, because you certainly do not seem to be able to do it alone!
…
Sent from my iPhone
On 14 Apr 2020, at 03:43, Vincent Shury @.***> wrote:  I can't agree more that some clarification would of saved me hours of dissecting that code months ago. One comment to suggest what was happening there would of been a great improvement to what is supposed to be an example repository. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

Get off open source if you are going to tell new coders they need to pay to learn more.
you POS.

@Vincent440 Hey. @Xalior shared their opinion on the matter. Don't take it personally. I am sure it was not intentional. And please retract your comment. Let's keep the discussion clean. It's up to the maintainers to decide what to do.

Wow... personal insults... that’s a super mature attitude. No wonder people do not want to support you.

I apologize maybe there was a miscommunication sometime when you attacked me for adding my input, however I do not think your comment was at all appropriate as a response to my input on this issue.
I appreciate the work everyone has done to make Passport and these examples available, I am sorry I tried to add some input honestly.

IMO, process.nextTick is not a widely known feature, so many may wonder why it's there at first glance, a more effective solution could just be putting the code inside a setTimeout, this is much more widely known and clearly let's the user know that the code is async.


Missing files in DbgProvider project causing build errors

2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4406,5): error MSB3030: Could not copy the file "E:\Source\Repos\DbgShell\DbgProvider\Debugger.Converters.clr.ps1" because it was not found.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4406,5): error MSB3030: Could not copy the file "E:\Source\Repos\DbgShell\DbgProvider\Debugger.Converters.Cui.ps1" because it was not found.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4406,5): error MSB3030: Could not copy the file "E:\Source\Repos\DbgShell\DbgProvider\Debugger.Converters.dfsr.ps1" because it was not found.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4406,5): error MSB3030: Could not copy the file "E:\Source\Repos\DbgShell\DbgProvider\Debugger.Converters.tlx.ps1" because it was not found.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4406,5): error MSB3030: Could not copy the file "E:\Source\Repos\DbgShell\DbgProvider\Debugger.Converters.utl.ps1" because it was not found.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4406,5): error MSB3030: Could not copy the file "E:\Source\Repos\DbgShell\DbgProvider\Debugger.DebuggeeTypes.cui.psfmt" because it was not found.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4406,5): error MSB3030: Could not copy the file "E:\Source\Repos\DbgShell\DbgProvider\Debugger.DebuggeeTypes.dfsr.psfmt" because it was not found.

Should the files be added, or the project updated?

Same problems with the same list of files.

My apologies for this. Thanks for letting me know! Reason number 1 that I need to get CI builds going.
There may still be a build issue regarding VC redist binaries... a problem that I thought I had already fixed, but works on one machine, but not on another... I'll hopefully get that fixed shortly.

I've just built the latest version successfully.
It works, cool! :)
Now I need to deal with symbols and DAC file.


[BUG REPORT]

An error occurred
WORKER:ERROR [2023-08-29T21:37:38.118Z]     request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
  - METHOD_NAME: verifyUser
  - STATUS_CODE: 500
  - STATUS_MESSAGE: Internal Server Error
  - STACK_TRACE FetchError: request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
    at ClientRequest.<anonymous> (/tmp/.mount_BitfinRyhkjg/resources/app/bfx-reports-framework/node_modules/node-fetch/lib/index.js:1501:11)
    at ClientRequest.emit (node:events:527:28)
    at TLSSocket.socketErrorListener (node:_http_client:454:9)
    at TLSSocket.emit (node:events:527:28)

Debug info

Version
4.10.0

Commit Hash
49d2c919d82710be9a6ef5271af7a4b62c0702b7

Commit Date
2023-08-23T09:51:15Z

Electron
21.3.3

Chrome
106.0.5249.199

Node.js
16.16.0

V8
<IP_ADDRESS>-electron.0

OS version
Ubuntu SMP PREEMPT_DYNAMIC Mon Jul 10 23:39:54 UTC 2023

OS release
Linux x64 6.2.0-26-generic

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz x 8

RAM
30.26GB (7.48GB free)

App RAM limit
4095.75Mb (15.71MB used)

Is BFX API Staging used
No

Main log
    at async DataInserter._insertApiDataArrObjTypeToDb (/tmp/.mount
    at async DataInserter._insertApiData (/tmp/.mount_BitfinKpN0Kn/resources/app/bfx-reports-framework/workers/loc.api/sync/data.inserter/index.js:461:7)
[2023-08-07 13:15:22.363] [error] WORKER:ERROR [2023-08-07T11:15:22.359Z]     request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
  - METHOD_NAME: verifyUser
  - STATUS_CODE: 500
  - STATUS_MESSAGE: Internal Server Error
  - STACK_TRACE FetchError: request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
    at ClientRequest.<anonymous> (/tmp/.mount_BitfinKpN0Kn/resources/app/bfx-reports-framework/node_modules/node-fetch/lib/index.js:1505:11)
    at ClientRequest.emit (node:events:527:28)
    at TLSSocket.socketErrorListener (node:_http_client:454:9)
    at TLSSocket.emit (node:events:527:28)
[2023-08-29 23:37:38.111] [error] WORKER:ERROR [2023-08-29T21:37:38.107Z]     request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
  - METHOD_NAME: verifyUser
  - STATUS_CODE: 500
  - STATUS_MESSAGE: Internal Server Error
  - STACK_TRACE FetchError: request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
    at ClientRequest.<anonymous> (/tmp/.mount_BitfinRyhkjg/resources/app/bfx-reports-framework/node_modules/node-fetch/lib/index.js:1501:11)
    at ClientRequest.emit (node:events:527:28)
    at TLSSocket.socketErrorListener (node:_http_client:454:9)
    at TLSSocket.emit (node:events:527:28)
[2023-08-29 23:37:38.138] [error] WORKER:ERROR [2023-08-29T21:37:38.118Z]     request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
  - METHOD_NAME: verifyUser
  - STATUS_CODE: 500
  - STATUS_MESSAGE: Internal Server Error
  - STACK_TRACE FetchError: request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
    at ClientRequest.<anonymous> (/tmp/.mount_BitfinRyhkjg/resources/app/bfx-reports-framework/node_modules/node-fetch/lib/index.js:1501:11)
    at ClientRequest.emit (node:events:527:28)
    at TLSSocket.socketErrorListener (node:_http_client:454:9)
    at TLSSocket.emit (node:events:527:28)

Worker errors
  - STACK_TRACE FetchError: request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitf
    at ClientRequest.<anonymous> (/tmp/.mount_BitfinKpN0Kn/resources/app/bfx-reports-framework/node_modules/node-fetch/lib/index.js:1505:11)
    at ClientRequest.emit (node:events:527:28)
    at TLSSocket.socketErrorListener (node:_http_client:454:9)
    at TLSSocket.emit (node:events:527:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)
WORKER:ERROR [2023-08-29T21:37:38.107Z]     request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
  - METHOD_NAME: verifyUser
  - STATUS_CODE: 500
  - STATUS_MESSAGE: Internal Server Error
  - STACK_TRACE FetchError: request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
    at ClientRequest.<anonymous> (/tmp/.mount_BitfinRyhkjg/resources/app/bfx-reports-framework/node_modules/node-fetch/lib/index.js:1501:11)
    at ClientRequest.emit (node:events:527:28)
    at TLSSocket.socketErrorListener (node:_http_client:454:9)
    at TLSSocket.emit (node:events:527:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)
WORKER:ERROR [2023-08-29T21:37:38.118Z]     request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
  - METHOD_NAME: verifyUser
  - STATUS_CODE: 500
  - STATUS_MESSAGE: Internal Server Error
  - STACK_TRACE FetchError: request to https://api-pub.bitfinex.com/v2/auth/r/info/user failed, reason: getaddrinfo ENOTFOUND api-pub.bitfinex.com
    at ClientRequest.<anonymous> (/tmp/.mount_BitfinRyhkjg/resources/app/bfx-reports-framework/node_modules/node-fetch/lib/index.js:1501:11)
    at ClientRequest.emit (node:events:527:28)
    at TLSSocket.socketErrorListener (node:_http_client:454:9)
    at TLSSocket.emit (node:events:527:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)

Worker exceptions
Empty

The issue was opened from the electron app

Hi @PetrDlouhy has this been solved


DefaultConfigurationProvider.getConfiguration stalls in tests

After upgrading from langium 3.0.0-next.e78aeba to langium 3.0.0, many of the tests in Safe-DS time out. In particular, it's all tests that validate documents. This happens because some of our validation rules can be turned off using VS Code settings, so they contain code like
if (!(await settingsProvider.shouldValidateNameConvention())) {
    return;
}

The settingsProvider is just a thin wrapper around the DefaultConfigurationProvider and calls its getConfiguration method.
➡ #1388 is likely related to this.
Langium version: 3.0.0
Package name: langium
Steps To Reproduce
Substitute SafeDs with some other language in the following code and run the tests with vitest:
import { describe, it } from 'vitest';
import { createSafeDsServices, SafeDsLanguageMetaData } from '../../../src/language/index.js';
import { NodeFileSystem } from 'langium/node';

const services = createSafeDsServices(NodeFileSystem).SafeDs;

describe('time out', () => {
    it('should not time out', async () => {
        await services.shared.workspace.ConfigurationProvider.getConfiguration(
            SafeDsLanguageMetaData.languageId,
            'validation',
        );
    });
});

Output:
Error: Test timed out in 5000ms.
If this is a long-running test, pass a timeout value as the last argument or configure it globally with "testTimeout".
    at Timeout.<anonymous> (file:///E:/Repositories/safe-ds/DSL/node_modules/@vitest/runner/dist/index.js:63:16)
    at listOnTimeout (node:internal/timers:573:17)
    at processTimers (node:internal/timers:514:7)

Link to code example:

Relevant PR
Relevant workflow run

The current behavior
Tests time out because the service never readies up.
The expected behavior
Tests should pass.

This is working as expected, see here for the intended solution:
https://github.com/eclipse-langium/langium/blob/18a38cedb427d36bb955dd854548e9173955b2f8/packages/langium/test/workspace/configuration.test.ts#L17
We cannot really identify in the ConfigurationProvider whether we are running inside of a language server (which will eventually make the provider ready) or not. You need to call initialized explicitly to make the provider ready.

Makes sense, thanks. I'm going to add the initialization of the configuration to our createSafeDsServices function and control this via an optional parameter.
Would it make sense to alter the Yeoman template to include such a parameter? Something like
export function create<%= LanguageName %>Services(context: DefaultSharedModuleContext, options?: ModuleOptions): {
    shared: LangiumSharedServices,
    <%= LanguageName %>: <%= LanguageName %>Services
} {
    const shared = inject(
        createDefaultSharedModule(context),
        <%= LanguageName %>GeneratedSharedModule
    );
    const <%= LanguageName %> = inject(
        createDefaultModule({ shared }),
        <%= LanguageName %>GeneratedModule,
        <%= LanguageName %>Module
    );
    shared.ServiceRegistry.register(<%= LanguageName %>);
    registerValidationChecks(<%= LanguageName %>);
    if (options?.initializeWithEmptyConfiguration) {
        shared.workspace.ConfigurationProvider.initialized({});
    }
    return { shared, <%= LanguageName %> };
}

interface ModuleOptions {
    inializeWithEmptyConfiguration: boolean
}

Or is this something that should be handled purely by adopters?

We should be able to do something similar to this, yes:
if (!context.connection) {
  // We don't run inside of a language server
  // Therefore, initialize the configuration provider instantly
  shared.workspace.ConfigurationProvider.initialized({});
}

@lars-reimann Do you want to contribute this?

Sure, I'll get to it right away.


[Enhancement] [OSF Meeting] Open new tab after clicking conference or author

Purpose
Currently, for part of links  on  all submission and conference page, it will not open a new tab but some of them do.
When it comes to whether a new tab should be opened, there are a lot discussions. However, for this case, we would prefer to let all of them open new tab.

Be consistent
Based on the content,  we have many links on one page, opening a new tab will be easy and quick to let them back. Otherwise, It will take a lot time if  bring back the page with all the links(Treebeard table).

Change
See target  attribute as "_blank" in author and conference name link.
Side Effect
None

Merged into feature/maintenance.


Question: is plugins repo acceptable kotlin/swift ?

Hi, I love flutter.
I found flutter plugins repo is only obj-c, Java code for native implementation.
Can I use swift/kotlin for PR?
Thanks.

@mravn-google

AFAIK, Swift currently comes with a substantial overhead to the consuming app. Kotlin less so, and adding some proguard rules may get rid of most of it.
Anyway, app bundles (apk/ipa) built from a flutter -i [objc|swift] -a [java|kotlin] create xyz projects have the following sizes:
Java apk:     7.3MB
Kotlin apk:   7.7MB
ObjC ipa:    49.5MB
Swift ipa:  187.2MB

These sizes may not mean much, but it does seem to me that the decision of whether to use Kotlin and Swift should be taken by the app author, not the plugin author.

Thank you for answering!
I will support your opinion.
Best regards.