Add support for advertised addresses

This implementation assumes that the address that is being listened on (e.g. <IP_ADDRESS>:1234), is the same address that other nodes will use to connect. This is usually the case for testing locally, but in staging/production, it's common to use domain names, and/or different addresses.
I don't think there is much to change except removing the address comparison here
To get around this, I am manually adding Transport and using the advertised address as the address variable for the NewRaft function.

Apologies for the delay in getting to this. This seems simple enough - to be clear, you would like to be able to bootstrap a node with a configuration that contains a differing address than the address the node is initialized with?
For example, let's say that that we initialize a node using the NewRaft function with ID n and address <IP_ADDRESS>. You would like the ability to call Bootstrap on this node with a configuration containing an ID-address pair such as (n, <IP_ADDRESS>). Is that right?

Yep, basically 2 different addresses. Listen Address (which could be <IP_ADDRESS>:1234) and Advertised Address (which could be node1.domain.top:4321).
Advertised Address should always default to the Listen Address.

Great. In the future, feel free to make pull requests yourself - you don't have to, but I occasionally get caught up with work or other stuff and it may take me a while to get to it. If it's a simple change like this and you put up a PR, I'll make sure to review it in a timely manner.


Database navigator does not show view columns if their value is NULL

Description
I'm working on Databricks so I don't know if this happens with other databases as well.  If a view DDL (CREATE VIEW AS SELECT...) includes a column which is null (NULL AS columname, ) then that column  is not shown in the database navigator window
DBeaver Version
Community Edition <IP_ADDRESS>411031831
Operating System
Windows 10 professional
Database and driver
Default Databricks driver (Azure)
Steps to reproduce
Create a view in Databricks which has some columns NULL:
CREATE OR REPLACE VIEW viewname AS SELECT
NULL AS quantity1,
line_item_quantity AS quantity2
FROM etc. etc.
Run the DDL and look at the column names in the Database navigator: you can see only
viewname
quantity2
Column quantity1 is invisible in the database navigator
Additional context
No response

Thanks for the bug report


spectral-language-server

Awesome Spectral JSON/YAML linter with OpenAPI/AsyncAPI support
Spectral is a flexible object linter with out of the box support for OpenAPI v2 and v3, JSON Schema, and AsyncAPI.
https://github.com/luizcorreia/spectral-language-server
MIT

Should be trivial to package using buildNpmPackage.


Find results in the settings editor do not show up in the scroll bar

Not sure if this used to work but it's hard to find settings that you already have set because they don't show up in the scroll bar:

Duplicate of #39543


add interface for reference object of the slice

Add interface for reference object of the slice,  to check if the slice use off-heap or heap memory. And also we can use reference (cast to bytebuffer) to free off-heap memory.

The reference is really an internal detail of this library. For such a use case, I would suggest the caller should wrapper the Slice or keep track of the reference some other way.


Financial decimal parsing

Allow correct financial calculation using the Decimal data type by letting the JSON parser turn float-like values into Decimal.
This behaviour can be specified during class creation using the kwarg 'json_parse_int' and 'json_parse_float' to parse INTs and FLOATs into Decimal respectivelly.
Example:

    from decimal import Decimal
    Stock('aapl',
            json_parse_int=Decimal,
            json_parse_float=Decimal
        ).get_dividends(range='1y')

Will look into this. I'm not sure if it's standard to have this sort of feature included in the library, from what I can see with others which are similar. Often people handle on their end, or just use Pandas output formatting (which uses np.float64 for floats, which supports the desired calculations)

Relying on float numbers for financial calculations is on the long run not a good idea.
This resource (http://buildingskills.itmaybeahack.com/book/programming-2.6/html/p13_modules/p13_c03_decimal.html) has a good explanation of it.
My PR  proposes a convenient and discrete method to use the Decimal class to work with numbers. Users which do not need this class an accuracy can remain with the defaults.

I am aware. Again, it's not common practice for a wrapper such as this to include the Decimal type explicitly (this package is designed with many of the Pandas Datareader, Yahoo Finance, and Google Finance paradigms in mind). I'm not very experienced in these kinds of integrations though, so I am looking into it. Doesn't seem like it would be too much of a problem given that normal user behavior is unchanged.

Also would like to avoid the additional dependency (six). This was my mistake, as we weren't supposed to be enforcing flake8 in that unicode instance. It there's a similar workaround without the dependency that would be great. Thanks again for your help.

It there's a similar workaround without the dependency that would be great.

Well yes, there are workaround, however you need to catch exceptions, which is also not pretty.
A couple of examples are listed here: https://stackoverflow.com/questions/11301138/how-to-check-if-variable-is-string-with-python-2-and-3-compatibility
try:
  basestring
except NameError:
  basestring = str

#[...]

isinstance(s, basestring)

@reixd thanks. I merged this. Realized there isn't a docs entry but I'll take care of it. We can leave the six dependency for now, since its on the testing side.


[IZPACK-508] Only append quotes when required

This should resolve the following problem on linux (ubuntu) when doing a re-install.
[ Starting processing ] Starting process Remove Previously Installed Service (1/1) I/O error: java.io.IOException: Cannot run program ""/usr/local/app/uninstall-service.sh"": error=2, No such file or directory
Note the double quotes in the exception, caused by a recent PR:
https://github.com/izpack/izpack/pull/749

Good point, can you please change the commit message as below? [IZPACK-508] Execution of executable that contains "(" character fails - only append quotes when required
Also, please add PR link to the ticket https://izpack.atlassian.net/browse/IZPACK-508.

This should be ok now?

Good point, can you please change the commit message as below? IZPACK-508 Execution of executable that contains "(" character fails - only append quotes when required
Also, please add PR link to the ticket https://izpack.atlassian.net/browse/IZPACK-508.

This should be ok now?

Looks good, thanks.

Hello,
I have some problems with this patch. It doesn't seem to solve the initial problem.
I have written a little example that reproduce the problem (https://gist.github.com/ThibLind/e74efea815ea2c9f08fe634724135606). In fact, on my Debian (tested with jdk8 and jdk11), I obtain "java.io.IOException: Cannot run program ""/usr/bin/xclock"": error=2, Aucun fichier ou dossier de ce type"
Sincerely,
Thibault L.

@ThibLind, can you please try your test program by changing the if condition as below and let us know the result?
if (System.getProperty ("os.name").contains("Windows") && !(filename.startsWith("\"") && filename.endsWith("\"")))

@hiteshbosamiya, its work on Debian and on Windows too ! Thanks. Do I need to make a PR ?

@ThibLind, yes, please raise a PR. You need to add the PR link to the ticket and the commit message should be "[IZPACK-508] Execution of executable that contains "(" character fails - taking care of Linux issue".

Hello, here is the link of my PR : https://github.com/izpack/izpack/pull/763


Implementation of sigmoid for chainerX

Ref issue:  #6423

The travis error seems to be because of code formatting issues in chainerx_cc/chainerx/routines/math_test.cc.(Extra whitespace at the end of the line)

@aksub99 fixed it, thanks :)

@dido1998 I'd suggest using cpplint for doing static C++ checks locally in case you aren't already using it. Installation instructions can be found here:
https://github.com/cpplint/cpplint

Could you take a look at the comment above? It LGTM otherwise 👍

@hvy I have made the changes, please do take a look.

The tests for chainer are failing even though I didn't alter any code for chainer?

Great, thanks. It looks good overall but I'm wondering if you could also write tests for the Python bindings too. Those are more easily parameterized than the C++ tests. Here is an example.

The CI failure is not related to your changes so you can ignore it. It's a stochastic test (e.g. randomizing input values) and unfortunately fails quite often.

@hvy Yes sure, working on the python tests now.

@dido1998 I think it's better to follow the Camel Case convention for test class names as in #6505 .

Noticed that Python tests should have a wider coverage. Could you take a look at the comment above?

Yes, I will put similar tests as tanh.

jenkins, test this please.

Jenkins CI test (for commit 0f8b34b74b913d8c4c359a73b0e5db3a9aba6e3c, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

the failure was due to the MaxPoolingND routine. I think this is because of the same reason you mentioned earlier regarding stochasticity? @hvy

jenkins, test this please.

Jenkins CI test (for commit 0f8b34b74b913d8c4c359a73b0e5db3a9aba6e3c, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

Jenkins CI test (for commit 0f8b34b74b913d8c4c359a73b0e5db3a9aba6e3c, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Are you able to (sync with latest master and) run the C++ tests locally? The Sigmoid backward tests seem to be failing due to
C++ exception with description "Invalid eps dtype" thrown in the test body.

@hvy I will try it locally again. I wasn't trying with the latest master.

The math tests have been cleaned up a bit. Sorry but could you rebase/merge with master?

Can one of the admins verify this patch?

Could you remove the temporary Python implementation of sigmoid? https://github.com/chainer/chainer/blob/0f8b34b74b913d8c4c359a73b0e5db3a9aba6e3c/chainerx/activation.py#L21-L36 It might be the case that this overwriting your binding.

Yepp sure I will do it by tomorrow. I haven't been able to yet merge with the current master as I have been preoccupied by exams which are ending tomorrow. I will finish the both the changes tomorrow.

@hvy since I rebased to the current master, most of the tests are being skipped when I run pytest? Do you know how to fix this ?

Jenkins CI test (for commit 6631520ca6fa23d541b2fe7626cff011689593eb, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Regarding the skips, the test utilities have been refactored and as an unfortunate side effect, the forward, backward and double backward tests for each function is considered a single test case. Thus, if either of them is skipped, the test case is rendered S (i.e. if backward is skipped but forward is actually tested.)

Jenkins, test this please.

Seems to conflict with master again.

It wasn't showing before. I'll try to fix it.

Jenkins CI test (for commit b39d1f21514cc0fa5cdaec8710f362b6d18f5290, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Are you able to take a look at the Travis failures? There are some minor style errors which should be quite easy to fix.

Jenkins, test this please.

Jenkins CI test (for commit a3e4e8a68019b3dac1eedb6b6ffd3825716f5316, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Thanks for the long running PR, there are some CI failures but they look unrelated to these changes. LGTM!


PathObserver should take the path as a list of symbols

 Issue by jmesserly
Originally opened as dart-lang/sdk#13473

instead of:
    new PathObserver(model, 'foo.bar.baz');
It should be:
    new PathObserver(model, [#foo, #bar, #baz]);
I'm not sure if Node.bind should be changed too, but I think so.

 Comment by sigmundch

Added this to the M8 milestone.

 Comment by jmesserly

Removed this from the M8 milestone.
Added this to the Later milestone.

 Comment by sigmundch

Removed this from the 1.1 milestone.
Added this to the 1.2 milestone.

 Comment by anders-sandholm

Removed Library-Observe label.
Added Pkg-Observe label.

 Comment by clayberg

Removed this from the M9 milestone.
Added this to the 1.1 milestone.

 Comment by jmesserly

no current plans to change this

Added NotPlanned label.

 Comment by sigmundch

Removed this from the Later milestone.
Added this to the M9 milestone.


try to add some e2e-test

Aiming to make a production ready vue vscode-extension, It's necessary add some test, like vetur

I want to add unit tests for vscode-vue-languageservice as replacement, because:

vscode-vue-languageservice will publish to npm as a module, unit-test is necessary.
in future we will have multiple IDE support, e2e-test for each IDE is too much work.
e2e-test very slow. :P

Implement in: https://github.com/johnsoncodehk/volar/pull/55


Add more HA docs

What would you like to be added?

add docs for HA from scratch both for helmfile and manual install
Update helmfile for HA to include GK installation so users can use one helmfile to install Ratify HA from scratch

Anything else you would like to add?
No response
Are you willing to submit PRs to contribute to this feature?

[X] Yes, I am willing to implement it.

added gatekeeper to helmfiles in #1209


Increasing Accuracy

Hello,
I have trained the model with the given configs. But i am getting only 80% accuracy on 2 object class data.
Is there any possibility to increase it further??  What parameters should i change to increase the accuracy??
I am using mobilenet 0.25 version as base network..

Overfit？
Or try to use ROI Align replace ROI Pooling？
Ram124<EMAIL_ADDRESS>于2018年10月22日周一 下午5:59写道：

Hello,
I have trained the model with the given configs. But i am getting only 80%
accuracy on 2 object class data.
Is there any possibility to increase it further?? What parameters should i
change to increase the accuracy??
I am using mobilenet 0.25 version as base network..
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WalterMa/gluon-faster-rcnn/issues/8, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AHNbxBd2tNIOVVTht9xBUVi0VFuQSbRYks5unZccgaJpZM4XzEVv
.

@WalterMa
Should i include that in the mobilenet.py in rcnn/base_model by replacing the below line
x = F.ROIPooling(data=feature_map, rois=rois, pooled_size=(7, 7), spatial_scale=1.0 / self.feature_stride)
as
x = F.ROIalign(data=feature_map, rois=rois, pooled_size=(7, 7), spatial_scale=1.0 / self.feature_stride)
Is this correct??

Yes
refer to
https://mxnet.incubator.apache.org/api/python/symbol/contrib.html?highlight=roi#mxnet.symbol.contrib.ROIAlign
Ram124<EMAIL_ADDRESS>于2018年10月25日周四 下午5:56写道：

@WalterMa https://github.com/WalterMa
Should i include that in the mobilenet.py in rcnn/base_model by replacing
the below line
x = F.ROIPooling(data=feature_map, rois=rois, pooled_size=(7, 7),
spatial_scale=1.0 / self.feature_stride)
as
x = F.ROIalign(data=feature_map, rois=rois, pooled_size=(7, 7),
spatial_scale=1.0 / self.feature_stride)
Is this correct??
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/WalterMa/gluon-faster-rcnn/issues/8#issuecomment-432990091,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHNbxIEdHNlySDJbocfBXyjCoSloMpzJks5uoYrcgaJpZM4XzEVv
.

Can I use the already trained .params files to use the weights ??
I have changed the cfg.resume to True.
But it is throwing an error due rcnnhead layers.
I am using 3 layers in rcnnhead. how to initialize them??
only base network layers are initialized from .params file.
the error is : conv13 is not initialized.  (conv13 is in rcnnhead)

Yes.
I have already tested on vgg16, that I could resume training from saved params.
So I'm not sure about reasons of your problem.


oops i idd it again: API change, pass down full components or use provided Question component

/!\ This is still missing documentation /!\
i wanted to have a check with you of what you think of this direction:
instead of passing just the label and constructing all the component in a
loop, we create a Question component that you can use instead of the
if you only want the previous API, or you can implement your own that just
needs to implement an onChange callback whenever the data is ready to be
added to values.
this allows to implement maps, typeahead, or any other kind of custom
components. we should probably ship some of those so that we have pretty and
useable stuff right away.
cheers,

I think this is a good change, not sure if you saw my comments here: https://github.com/GopherLabsLtd/react-eureka-form/pull/2
I'll merge soon

I think it should not be merged without doc, and as it's yet another API breakage, maybe we want to merge it with a few more question types too, and a bit more brain cycles.

(I had not seen your comments, answered there)

just pushed to the README.md, note that it's pseudo code, i'll write what i need for my current project tomorrow.

for future reference:
Full API
In order to implement your own Question Helper you just need to call
onChange when your value changes, if you are going to use an <input/>
tag, please pass down the type property to make sure HTML5 validation
still works. You should also make sure to gracefully handle the children
prop, as it'll be usual to pass down the question component there.
class InputQuestion extends React.PureComponent {
    render () {
        const { onChange, type, children } = this.props
        return (

                {children}
                <input onChange={onChange} type={type}/>

        )
    }
}

class ListQuestion extends React.PureComponent {
    render () {
        const { options, onChange, children } = this.props
        return (

{children}

                { options.map(opt, i => (

 onChange(opt)}>{opt}

                )}

        )
    }
}

const MyForm = ({values = {}}) => (
    <EurekaForm onSubmit={doStuff}>
        <InputQuestion type='name'>
          What's your name ?
        </Question>
        <ListQuestion type='country' options={['Argentina', 'Brazil', 'Canada']}>
          Hello {values.name}, and your country ?
        </Question>
    </EurekaForm>
)

Cool, I like that idea actually!

this actually proved to be quite broken...
i'm in the process of re-writing it to work, need to split out all the code that selects the input, had to re-do the animation logic and am in the process of passing validation to the child too....

ok, animation logic could actually use some re-doing 😄
Thanks!

lots of patches =/ couldn't do it with less, and this still needs some love.
this patchset reworks a lot of the internal logic and the animation code.
there are still a few open issues that i'd like to discuss:

we will animate any  we find in the <Question/> component maybe we need to className it ?
we introduce a validate() and focus() method that require the <Question/> implementer to provide a this.inputRef to work. the 'correct' way to make that work would be to look for a this ref with ReactDOM, but that would introduce a new dependency, and probably make even harder to port our code to React Native.
we have both a focus() method and a focus prop, that we use to focus on one side and disable the input on the other, it's a bit shady to export that to downstream users

it is NOT ready to merge, see previous comment for points that needs to be discussed.
in the changes i did, i had to force <Question> to be a React.Component (instead of using functional components). the reason for that is the validate() and focus() issues stated before, they require a ref that require a full component.
that said in the last commit pushed here i've added an example that i'll keep up to date.
it currently has issues with CSS, as the Question input styling has to be independent and i couldn't find a nice way to do that (help wanted)
cheers.

I've been banging my head all day around this approach, and while i think it has a lot of good points, i'm not sure i can come up with an implementation, i'm now spliting out <input> in an Input component to style it, and need to pass down a lot of methods from Question that break generalizability, maybe it needs to go back to the drawing board...


feat: reuse STAKING_ID_TO_NAME constants in base opportunities names

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3281   👈

PR #3278

PR #3280

This comment was auto-generated by Graphite.


Add zip to the default exclusion list

Zips are deliberately not on the exclusion list because I've seen various games use them as uncompressed file containers.  The compressibility check will still quickly skip over compressed archives.


Mass Effect Legendary Edition: a line across the screen is visible on some materials when Ambient Occlusion is enabled

In Mass Effect 1 Legendary Edition, a diagonal line going from the top left corner of the screen to the bottom right corner is visible on some materials when Ambient Occlusion is enabled. The line looks like a typical ambient occlusion shadow so it's visible enough to be annoying in movement but not enough to be very noticeable in screenshots.
Here's a magnified screenshot where  the problem can be seen:

I wasn't able to solve this by reinstalling the game, with different versions of DXVK (including current master), or by changing any of the settings in the DXVK configuration file.
The only unusual thing I can see in the logs is this:
warn:  D3D11Query: Unknown interface query
warn:  9eb576dd-9f77-4d86-81aa-8bab5fe490e2

Video of the issue: ME1-video.mkv
Apitrace file: ME1-trace.tar.zst
DXVK log files:
MassEffect1_d3d11.log
MassEffect1_dxgi.log
Software information
Mass Effect 1 Legendary Edition
System information

GPU: AMD Radeon RX 6900XT
Driver: Mesa 21.3.7 (tested both RADV and AMDVLK)
Wine version: Wine-GE-Proton7-8
DXVK version: tested 1.8 through 1.10.1 incl. current master
OS: Manjaro 21.2.5, KDE, Xorg, Kernel 5.16

Are you sure this is actually a DXVK problem rather than just a game bug?

I don't see this line on Windows and I can't find anyone complaining about it online so I guess it's either DXVK or Mesa.

Looks like the HBAO shader source file is present in the game files in the Engine/Shaders folder, maybe I can do something to it...

Still present in Mesa 22.0.1 :(

Can't reproduce here. It also doesn't make much sense for this to happen since the AO texture itself is computed via a compute shader, and then blended onto a render target with regular linear filtering.
Do you have any options enabled to enforce anisotropic filtering or something?

The problem still occurs with a clean install of the OS (tried both Xorg and Wayland just in case) but it works fine in Windows. WineD3D seems to be affected as well.
@doitsujin can you tell me how you're running the game? OS, screen resolution, etc.? The problem is not visible in every area of the game, I can give you a save file if you want.

Quick update on this, I just tested this on two more PCs (i5 6400+R9 290x, i7 6700k+RX 5500XT) with the same results, the line is present on linux but not on windows.
Are you 100% sure that you don't have this problem on your 6900XT? What else can I try to analyze this problem?

If you can share the save file i can try to see if i can reproduce it

Sure! BioWare.zip
Just load the last save in the citadel presidium and move around, you should see it.

Ah yes i see it. Easier to see in that spot when you turn stuff like film grain off.
R9 380 mesa 22.0.1 and also on amdvlk 2021Q2.5 (latest i can use). The game or launcher crashes before it's starts when i try with WINED3D.
Didn't find that any dxvk conf options affected it.

@Blisto91 you can bypass the launcher and launch the game directly with this command:
MassEffect1.exe -NoHomeDir -SeekFreePCConsole

The line is visible with WineD3D as well on my PC.
Can you confirm that it goes away if you set it to windowed mode instead of fullscreen?

Ye can confirm windowed gets rid of the issue.
Still having some issues with WINED3D, but since you've confirmed it's also present there thats not so important.
I can try to launch the game on my nvidia setup when i have time, and if it can handle it. Only 2GB vram hehe.

I can repro it on my 6900xt with both radv and amdvlk but not on my 1060, radv_disable_aniso_single_level=true doesn't help.

Thanks for testing on Nvidia. My 960 was screaming in pain just attempting to boot it 😁
I also tried various radv variables without a change.

Figured out what's wrong, D3D and Vulkan use different rules for computing texture coordinates when sampling with NEAREST filtering, and since the game uses a fullscreen quad to apply the AO effect, one triangle has tiny interpolation errors that lead to sampling offsets.
Unfortunately this is pretty much impossible to fix on our end.

Good work. Followed the Convo on discord. 😁
So mesa issue and cross fingers? 👀

@doitsujin Thanks for investigating the issue.
Do you happen to know why I can't edit any of the .usf files? I want to try and modify the shaders to work around this issue but the game won't start if I change even a comment in those files.

Do you happen to know why I can't edit any of the .usf files? I want to try and modify the shaders to work around this issue but the game won't start if I change even a comment in those files.

Legendary Edition does not come with a shader compiler. Thats why shaders arent editable.

Just checked again, the issue is still present as of june 2023 on the latest mesa master, was this reported to mesa? I can't find it in their issues

I don't believe so

Ok, opened an issue on mesa gitlab: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9253

Still here :(
I'm willing to put a bounty on this issue if the project allows it, I'm tired of having to reboot into windows every time I want to play this game, and this issue likely affects other late UE3 games that use the same ambient occlusion shader.

I saw this issue mentioned here, have you tried the workaround posted there?

@mbriar that seems to have fixed it. The code was moved to radv_sampler.c but once I found it and patched it, the line was gone and AO looks correct.
I'm not sure what the implications of this change are though, maybe one of you veterans can explain: if this was only enabled when using the nearest filter, I assume there's a reason for that, what could this break? how do coordinates get truncated?

Fixed in mesa https://gitlab.freedesktop.org/mesa/mesa/-/commit/9e9eb87dd3a2ffe782ac85998a69623f59306caf


Return first geometryBuffer if no best batch found

Description
If no exact match by attribute is found in _findBestGeometryBuffers return the first one.  I believe this was the original intention with this function returning 0 for the index.
This allows I3S Scenes to render with geometries missing either ["position", "uv0"] (See issue / sandcastle).

Issue number and link
Fixes #12114
Testing plan

Create sandcastle with the following added as an I3S dataProvider:

https://basemaps3d.arcgis.com/arcgis/rest/services/OpenStreetMap3D_Buildings_v1/SceneServer?layers=ca0470dbbddb4db28bad74ed39949e25

Verify the scene renders and errors are no longer thrown
Confirm other I3S scenes are rendering fine

Sandcastle Link
Author checklist

[x] I have submitted a Contributor License Agreement
[x] I have added my name to CONTRIBUTORS.md
[x] I have updated CHANGES.md with a short summary of my change
[ ] I have added or updated unit tests to ensure consistent code coverage
[x] I have updated the inline documentation, and included code examples where relevant
[x] I have performed a self-review of my code

Thanks for the PR @ljowen! The fix looks good to me.
Is it possible to add a unit test for this case? We can provide guidance as needed.

Thanks @ggetz I did begin working on a unit test, will add it in a separate PR

Thanks @ljowen, that would be very much appreciated! Let us know if you'd like any input.


Missing source-map-support dependency, fails with Yarn Berry PnP

When upgrading from ts-node 10.2.0 to 10.4.0, I'm seeing a failure running a script that executes require('ts-node').register({ transpileOnly: true }).
This is the error I see:

@cspotcode/source-map-support tried to access source-map-support, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.
Required package: source-map-support
Required by: @cspotcode/source-map-support@npm:0.7.0 (via /Users/noah/.yarn/berry/cache/@cspotcode-source-map-support-npm-0.7.0-456c3ea2ce-8.zip/node_modules/@cspotcode/source-map-support/)

Using Yarn PnP 3.1.0. In yarn pnp, any package required needs to be declared in its package.json. I've tried to understand where source-map-support is required and whether it should be listed, but I had some trouble with the source code. If we want to check for the existence of the package, throwing it in a try...catch would be ideal.

Getting a similar error when using Jest with this and Yarn 3.1.1:
Cannot find module 'source-map-support' from '/Users/francis/d/IdeaProjects/product/.yarn/cache/@cspotcode-source-map-support-npm-0.7.0-456c3ea2ce-9faddda775.zip/node_modules/@cspotcode/source-map-support'

  at resolveSync (../../.yarn/cache/resolve-patch-bad885c6ea-c79ecaea36.zip/node_modules/resolve/lib/sync.js:111:15)

This Yarn 2+ patch makes the problem go away for me, but is likely not the right thing to do.
cspotcode-source-map-support.0.7.0.txt

@cspotcode can you offer some guidance on this issue?

@noahnu is there more detail you can offer about the error message you're getting?  For example, if there is a detailed stack trace, are you able to share that?  I see the error messag in the issue description but I'm not sure if you've truncated a stack trace, or if the error omits the stack trace.

@noahnu is there more detail you can offer about the error message you're getting? For example, if there is a detailed stack trace, are you able to share that? I see the error messag in the issue description but I'm not sure if you've truncated a stack trace, or if the error omits the stack trace.

The errors shown by my and @noahnu are complete. That's all you get when you use jest. If you can look at my patch, you can see how it's fixed by removing the resolve call. I'm happy to try out another proposed change in my Yarn2 environment.

I'm trying to put together a repro repository to share.

Great, thanks for confirming.  I'm going to attempt a reproduction on my end, too.  If I can't reproduce, I'll share the reproduction with you so that you can fork and modify it to produce the error.

I've started a branch with yarn 3 PnP and ts-node.
https://github.com/TypeStrong/ts-node-repros/tree/node-source-map-support-issue-35
https://github.com/TypeStrong/ts-node-repros/blob/e84281dde030b5fd977474bc8466421c4e9f9c5f/run.sh#L1-L14
If you are already making progress on your own reproduction, feel free to share that.  Or, if it is easier, you can fork the branch I've created and commit modifications to your fork, then share a link here.

Hmm, might be more of a jest issue as francisu mentioned. Confirming now

Okay, here's the repro: https://github.com/noahnu/repro-source-map-error
At least from how I was using it, it happens when you try to run jest using ts-node. I'm using a custom script which loads jest which is why we need the ts-node rather than purely rely on babel or ts-jest. This repro just tries to call jest directly though.

At least from how I was using it, it happens when you try to run jest using ts-node. I'm using a custom script which loads jest which is why we need the ts-node rather than purely rely on babel or ts-jest. This repro just tries to call jest directly though.

My experience as well; thanks for doing the repro.

What output do you get when running the reproduction?  I am not seeing the error message described in your initial post.  I do not see "ambiguous and unsound."

What output do you get when running the reproduction? I am not seeing the error message described in your initial post. I do not see "ambiguous and unsound."

Did you get anything bad? I did not get the ambiguous and unsound message either. Just 'cannot find module'.

The repro I shared was somewhat incomplete. With the current setup, this is my output:
λ yarn ts-node --transpile-only $(yarn bin jest)
 FAIL  ./index.test.ts
  ● Test suite failed to run

    Cannot find module 'source-map-support' from '/home/noah/repos/repro-source-map-error/.yarn/cache/@cspotcode-source-map-support-npm-0.7.0-456c3ea2ce-9faddda775.zip/node_modules/@cspotcode/source-map-support'

      at resolveSync (.yarn/cache/resolve-patch-bad885c6ea-c79ecaea36.zip/node_modules/resolve/lib/sync.js:111:15)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        0.058 s
Ran all test suites.

If I first add 'source-map-support' via yarn add -D source-map-support, then it passes. I believe the "ambigous and unsound" error is because in the use case which prompted this issue, I had an extra level of indirection so yarn recognized there was a source-map-support package it just didn't have access to.

At a higher level, what's the purpose of requiring source-map-support? Should it be listed in the package.json as a peer dependency?

I don't think we are, though.  I think the opposite: jest is trying to require source-map-support, and we're redirecting to @cspotcode/source-map-support as per https://github.com/cspotcode/node-source-map-support/pull/23
The motivation for this is explained here: https://github.com/TypeStrong/ts-node/issues/1441
It's also worth keeping in mind that jest does a lot of magic.  The more magic jest has, the more likely it is to break.  In this case,  I see jest using this function.  I don't know what it's supposed to do, but jest is using it to require source-map-support:
https://github.com/facebook/jest/blob/fcd626d202069073e6212acaf730d672323c7859/packages/jest-runtime/src/index.ts#L849-L870

Some more relevant comments here https://github.com/TypeStrong/ts-node/pull/1496

If anyone in the future is reading this, please keep in mind that we are only forced to do this because source-map-support has not merged the corresponding PR to their repository.

Here is a link to that PR: https://github.com/evanw/node-source-map-support/pull/209

Ah I see. I haven't reproduced my original error (ambiguous and unsound) outside of my work repo but I imagine the way the redirect is happening is causing the require to come through this repo's version of source-map-support which causes the ancestor chain to break -- hence the "ambiguous and unsound" error.
I wonder if listing source-map-support as an optional peer might still make sense. At some point this week when I'm at my work computer, I can try narrow the ambiguous and unsound aspect down further.

Here's where jest is doing the weird internal require thing:
https://github.com/facebook/jest/blob/3a85065fe5604655e1337ffc1631f9999722c821/packages/jest-runner/src/runTest.ts#L213-L219
Looks like it's asking node for the path to source-map-support, then it's trying to require('source-map-support') relative to the path it gets.  This second step is failing.

@noahnu try clearing the jest cache (it's persistent): yarn jest --clearCache

You might consider asking the jest folks what they're trying to do here: https://github.com/facebook/jest/blob/3a85065fe5604655e1337ffc1631f9999722c821/packages/jest-runner/src/runTest.ts#L213-L219
It seems like they're trying to require source-map-support through some custom, reimplementation of node's module resolution algorithm.  I'm not sure why that's appropriate in this case, because I don't think they're trying to mock or otherwise replace that module.  This seems like it would work better:
// For tests
  const resolved = require.resolve('source-map-support');
  runtime
    .requireInternalModule<typeof import('source-map-support')>(
      resolved,
      resolved,
    )
    .install(sourcemapOptions);

Have you had any luck asking jest about the above ^ ?
I will make an issue for them later today.

Thanks!  If you share it here, I'll be sure to subscribe and follow along.

I tried your suggestion above in repro provided by @noahnu:
 % yarn ts-node --transpile-only $(yarn bin jest)
 FAIL  ./index.test.ts
  ● Test suite failed to run

    TypeError: (resolved , resolved).install is not a function

      at runTestInternal (.yarn/unplugged/jest-runner-npm-27.5.0-3919650619/node_modules/jest-runner/build/runTest.js:342:90)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        0.17 s
Ran all test suites.
(node:53783) UnhandledPromiseRejectionWarning: Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'source-map-support' imported from /Users/francis/d/IdeaProjects/repro-source-map-error/.yarn/unplugged/jest-runner-npm-27.5.0-3919650619/node_modules/jest-runner/build/runTest.js
Did you mean to import<EMAIL_ADDRESS>    at packageResolve (internal/modules/esm/resolve.js:650:9)
    at moduleResolve (internal/modules/esm/resolve.js:691:18)
    at Loader.defaultResolve [as _resolve] (internal/modules/esm/resolve.js:805:11)
    at Loader.resolve (internal/modules/esm/loader.js:88:40)
    at Loader.getModuleJob (internal/modules/esm/loader.js:241:28)
    at Loader.import (internal/modules/esm/loader.js:176:28)
    at importModuleDynamically (internal/modules/cjs/loader.js:1028:27)
    at exports.importModuleDynamicallyCallback (internal/process/esm_loader.js:30:14)
    at runTestInternal (/Users/francis/d/IdeaProjects/repro-source-map-error/.yarn/unplugged/jest-runner-npm-27.5.0-3919650619/node_modules/jest-runner/build/runTest.js:342:11)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:53783) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:53783) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

% yarn info @cspotcode/source-map-support -AR
└─ @cspotcode/source-map-support@npm:0.7.0
   ├─ Version: 0.7.0
   │
   └─ Dependencies
      └─ @cspotcode/source-map-consumer@npm:0.8.0 → npm:0.8.0

The require.resolve() call is returning this:
/Users/francis/d/IdeaProjects/repro-source-map-error/.yarn/cache/<EMAIL_ADDRESS>
Here is the stack trace of the original exception:
onParseError(), url.js:279
URL(), url.js:355
defaultResolve(), resolve.js:767
resolve(), loader.js:88
getModuleJob(), loader.js:241
import(), loader.js:176
importModuleDynamically(), loader.js:1028
exports.importModuleDynamicallyCallback(), esm_loader.js:30
runTestInternal(), runTest.js:343

I'm not sure what's going wrong here, nor what I should ask the Jest people. If you could propose a patch that would fix the problem in the repro, I will do the leg work of making an issue with them and show how your patch fixes it.

This looks suspicious to me.
TypeError: (resolved , resolved).install is not a function
Almost as if there's typescript generics syntax in a JS file.

This looks suspicious to me.
TypeError: (resolved , resolved).install is not a function
Almost as if there's typescript generics syntax in a JS file.

What's happening there is the runtime.requireInternal() call is returning false.
In any case, I'm not sure how 'source-map-support' is supposed to resolve to '@cspotcode/source-map-support' as they are two different modules. But I'm missing the context of this whole thing (I'm just trying to get my build to work).

Actually, removing the '<typeof import('source-map-support')>' made it work, and I see now why. It looks like a typescript generic, but since it's in a JS file, it's a greater than and less than, and thus returning false, causing the error.
So I have a working patch and repro that I can make an issue with for the Jest team. I will do that shortly. Thanks for following up with me.

OK, let's see what they do.

Ha ha, howdy @noahnu. Thank you all for sorting so much of this out already! I must confess that I have been putting off investigating it for months. The good news is that this issue was fixed a week ago by Jest's prolific maintainer. The bad news is that while the fix is released in v28.0.0+, it didn't make v27, and ts-jest v27 rather sensibly pins Jest to v27. But wait, there's more good news! ts-jest 28.0.0-next.1 was released 2 days ago, and bumps Jest to v28. It works for me at Yarn 3.2.0, so I think this issue can be closed if it works for others as well. Fair warning that Jest v28 has a lot of breaking changes, but I found their migration guide very helpful in my case.

Closing since this was a jest issue, so no action required here.


The following actions failed │ λ build conf-python-2-7 1.2

Hello， when I run the following command，it occur some error， could you give me some suggests?
$ opam pin add fstar --dev-repo
[NOTE] Package fstar is already pinned to git://github.com/FStarLang/FStar (version 2021.12.31~dev).
[fstar.2021.12.31~dev] no changes from git://github.com/FStarLang/FStar
fstar is now pinned to git://github.com/FStarLang/FStar (version 2021.12.31~dev)
The following actions will be performed:
∗ install conf-python-2-7 1.2             [required by z3]
∗ install z3              4.8.5           [required by fstar]
∗ install fstar           2021.12.31~dev*
===== ∗ 3 =====
Do you want to continue? [Y/n] y
[z<IP_ADDRESS>] downloaded from https://github.com/Z3Prover/z3/archive/Z3-4.8.5.tar.gz
[ERROR] The compilation of conf-python-2-7 failed at "dash.exe -ec  python2.7 test.py".
#=== ERROR while compiling conf-python-2-7.1.2 ================================#
context     2.0.8 | win32/x86_64 | ocaml-variants.4.12.0+mingw64c | git://github.com/fdopen/opam-repository-ming
w.git#opam2
path        ~/.opam/ocaml-variants.4.12.0+mingw64c/.opam-switch/build/conf-python-2-7.1.2
command     dash.exe -ec  python2.7 test.py
exit-code   127
env-file    ~/.opam/log/conf-python-2-7-20960-1872ec.env
output-file ~/.opam/log/conf-python-2-7-20960-1872ec.out
output
dash: 1: python2.7: not found
<><>    Error report    <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
┌─ The following actions failed
│ λ build conf-python-2-7 1.2
└─
╶─ No changes have been performed
[NOTE] Pinning command successful, but your installed packages may be out of sync.

It seems that a system package required by python2.7 is absent. To know which one, I'd suggest using depext following https://fdopen.github.io/opam-repository-mingw/depext-cygwin/

In fact,  you need to install Python 2.7 via Cygwin (you need to use Cygwin Python, not Windows Python), it seems that depext does not detect that missing package.

Could you help me to give some concrete steps?

Having the same problem while installing via OPAM. However, I found that z3>=4.8.13 (latest is 4.11.2) would no longer need the deprecated Python 2.7 and uses Python 3 instead. I see that the version is intentionally pinned; any chance we could bump it?

Related: https://github.com/FStarLang/FStar/issues/2431

Nowadays macOS does not come with python2.

This should be fixed by #3069


New version: vim.vim.nightly version 9.1.0906

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run


Minotar Alternatives

I found one, http://minecraftskins.herokuapp.com/
Source Code: https://github.com/a9io/cors-minecraftskins
Change lines 197 and 1008 in index.html if you want to use their hosted site.It worked fine for me.
197: img.src = 'http://minecraftskins.herokuapp.com/skin/earthiverse.png';
1008: img.src = 'http://minecraftskins.herokuapp.com/skin/' + document.getElementById('username').value + '.png';

Crafatar is a nice alternative, we're going with that for now.


Travis-CI integration

Just automatically do the tests on travis-ci

@alanjds Could you do me a favor and squash these commits down to one?  Also, I'm curious what you're using byterun for. :)

Also, a thing I learned about tox:  the -e parameter is read from a TOXENV environment variable if it exists, so if you set TOXENV instead of TOX_ENV, then you can simply use "tox" instead of "tox -e $TOX_ENV".

Hum... my GIT knowledge seems subpar, as I have no clue on how to squash things.
For tox, I just grabbed some recipe floating around and it worked. I will update with your proposal.
I am investigating several ways to run Python in an Android browser. The Brython implementation is fast, the PyPy.js one is compatible. Brython have no bytecode, PyPy.js does. One line of work is making byterun runs right on Brython, and PyPy.js complile stuff to bytecode before sending the bytecode to the browser.
Anyway, this seems reasonable?

Here are good instructions on how to squash the commits: http://davidwalsh.name/squash-commits-git
You have an interesting idea, I did not think that byterun would be used to actually run Python code.  I wonder what the performance would be like...

Nice. Thanks.

(...) I did not think that byterun would be used to actually run Python code. I wonder what the performance would be like...

??? Well, I guess this is what it has made for, dont ;) ?
The idea is to feed python into CPython/PyPy locally, store the .pyc, spin byterun on Brython in the browser and feed the stored .pyc to byterun.
If performance sucks, reimplement some parts in javascript, as Brython allows it. Worst than non-JIT PyPy.js it will not be...

Uhh...I know PyPy.js takes forever to load, but I'm also sure that the JIT would kick in with something like byterun. Even without the JIT, Brython still is not the fastest kid on the block...

I noticed this too, but with JIT PyPy.js crashes on Android.

@alanjds Are you sure? I have a web app that uses PyPy.js, and I use it from Chrome on Android all the time. Maybe it's actually the wrapper JS? Besides, you should probably file that as a bug...

...or Android Browser crashes, to be correct.


Karpenter v1.0.7 - Nodeclaim stuck at Unknown state forerver

Description
Observed Behavior:
I'm having the issue that node being created but not joining the cluster and forever stuck in Unknown state
Expected Behavior:
Nodeclaim should change Ready to True
Reproduction Steps (Please include YAML):
EC2NodeClass
apiVersion: karpenter.k8s.aws/v1
kind: EC2NodeClass
metadata:
  annotations:
    karpenter.k8s.aws/ec2nodeclass-hash-version: v3
    karpenter.sh/stored-version-migrated: "true"
  finalizers:
  - karpenter.k8s.aws/termination
spec:
  amiSelectorTerms:
  - alias: al2023@latest
  blockDeviceMappings:
  - deviceName: /dev/xvda
    ebs:
      encrypted: true
      volumeSize: 40Gi
      volumeType: gp3
  metadataOptions:
    httpEndpoint: enabled
    httpProtocolIPv6: disabled
    httpPutResponseHopLimit: 2
    httpTokens: required
  role: KarpenterNodeRole
  securityGroupSelectorTerms:
  - tags:
      karpenter.sh/discovery: ...
  subnetSelectorTerms:
  - tags:
      karpenter.sh/discovery: ...

Nodepool
apiVersion: karpenter.sh/v1
kind: NodePool
metadata:
  annotations:
    compatibility.karpenter.sh/v1beta1-nodeclass-reference: '{"kind":"EC2NodeClass","name":"default","apiVersion":"karpenter.k8s.aws/v1beta1"}'
    karpenter.sh/nodepool-hash: "4458080172936680623"
    karpenter.sh/nodepool-hash-version: v3
    karpenter.sh/stored-version-migrated: "true"
    kubernetes.io/description: NodePool for provisioning spot capacity
  name: spot
spec:
  disruption:
    budgets:
    - nodes: 10%
    consolidateAfter: 0s
    consolidationPolicy: WhenEmptyOrUnderutilized
  limits:
    cpu: "8"
  template:
    metadata:
      labels:
        nodepool: spot
    spec:
      expireAfter: 720h
      nodeClassRef:
        group: karpenter.k8s.aws
        kind: EC2NodeClass
        name: default
      requirements:
      - key: karpenter.sh/capacity-type
        operator: In
        values:
        - spot
      - key: kubernetes.io/arch
        operator: In
        values:
        - amd64
      - key: kubernetes.io/os
        operator: In
        values:
        - linux
      - key: node.kubernetes.io/instance-type
        operator: In
        values:
        - t3.medium
        - t3a.medium
        - t2.medium
        - c5a.xlarge
        - c6i.xlarge

Logs
{"level":"INFO","time":"2024-11-04T15:13:30.311Z","logger":"controller","message":"Starting workers","commit":"901a5dc","controller":"nodepool.readiness","controllerGroup":"karpenter.sh","controllerKind":"NodePool","worker count":10}
{"level":"INFO","time":"2024-11-04T15:13:30.311Z","logger":"controller","message":"Starting workers","commit":"901a5dc","controller":"nodeclaim.disruption","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","worker count":10}
{"level":"INFO","time":"2024-11-04T15:13:30.311Z","logger":"controller","message":"Starting workers","commit":"901a5dc","controller":"status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","worker count":10}
{"level":"INFO","time":"2024-11-04T15:13:30.311Z","logger":"controller","message":"Starting workers","commit":"901a5dc","controller":"nodeclass.hash","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","worker count":10}
{"level":"INFO","time":"2024-11-04T15:13:30.610Z","logger":"controller","message":"discovered ssm parameter","commit":"901a5dc","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","EC2NodeClass":{"name":"default"},"namespace":"","name":"default","reconcileID":"95358f1e-4cad-4b64-bf6f-c89472c89cb7","parameter":"/aws/service/eks/optimized-ami/1.31/amazon-linux-2023/arm64/standard/recommended/image_id","value":"ami-0ae1e07e02f98b306"}
{"level":"INFO","time":"2024-11-04T15:13:30.636Z","logger":"controller","message":"discovered ssm parameter","commit":"901a5dc","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","EC2NodeClass":{"name":"default"},"namespace":"","name":"default","reconcileID":"95358f1e-4cad-4b64-bf6f-c89472c89cb7","parameter":"/aws/service/eks/optimized-ami/1.31/amazon-linux-2023/x86_64/standard/recommended/image_id","value":"ami-0be82d98bb3e7f36c"}
{"level":"INFO","time":"2024-11-04T15:13:30.670Z","logger":"controller","message":"discovered ssm parameter","commit":"901a5dc","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","EC2NodeClass":{"name":"default"},"namespace":"","name":"default","reconcileID":"95358f1e-4cad-4b64-bf6f-c89472c89cb7","parameter":"/aws/service/eks/optimized-ami/1.31/amazon-linux-2023/x86_64/nvidia/recommended/image_id","value":"ami-0ed2f679097182d7a"}
{"level":"INFO","time":"2024-11-04T15:13:30.694Z","logger":"controller","message":"discovered ssm parameter","commit":"901a5dc","controller":"nodeclass.status","controllerGroup":"karpenter.k8s.aws","controllerKind":"EC2NodeClass","EC2NodeClass":{"name":"default"},"namespace":"","name":"default","reconcileID":"95358f1e-4cad-4b64-bf6f-c89472c89cb7","parameter":"/aws/service/eks/optimized-ami/1.31/amazon-linux-2023/x86_64/neuron/recommended/image_id","value":"ami-0c42cc0277c8e37ac"}
{"level":"INFO","time":"2024-11-04T15:16:39.902Z","logger":"controller","message":"found provisionable pod(s)","commit":"901a5dc","controller":"provisioner","namespace":"","name":"","reconcileID":"b8f66e83-5fe0-403e-8a92-8ec5c7d2117d","Pods":"worker-port-scan-stage2/worker-port-scan-stage2-679564f457-5pqch","duration":"46.212393ms"}
{"level":"INFO","time":"2024-11-04T15:16:39.902Z","logger":"controller","message":"computed new nodeclaim(s) to fit pod(s)","commit":"901a5dc","controller":"provisioner","namespace":"","name":"","reconcileID":"b8f66e83-5fe0-403e-8a92-8ec5c7d2117d","nodeclaims":1,"pods":1}
{"level":"INFO","time":"2024-11-04T15:16:39.917Z","logger":"controller","message":"created nodeclaim","commit":"901a5dc","controller":"provisioner","namespace":"","name":"","reconcileID":"b8f66e83-5fe0-403e-8a92-8ec5c7d2117d","NodePool":{"name":"spot"},"NodeClaim":{"name":"spot-5q86d"},"requests":{"cpu":"680m","memory":"785Mi","pods":"8"},"instance-types":"c5.xlarge, c5a.2xlarge, c6a.2xlarge, c6i.xlarge, c7i-flex.xlarge and 7 other(s)"}
{"level":"INFO","time":"2024-11-04T15:16:43.397Z","logger":"controller","message":"launched nodeclaim","commit":"901a5dc","controller":"nodeclaim.lifecycle","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","NodeClaim":{"name":"spot-5q86d"},"namespace":"","name":"spot-5q86d","reconcileID":"70b8ee7c-e93d-4d6f-989f-844ee57c3c73","provider-id":"aws:///ap-southeast-1c/i-0665d2caaa8e38f76","instance-type":"c5.xlarge","zone":"ap-southeast-1c","capacity-type":"spot","allocatable":{"cpu":"3920m","ephemeral-storage":"35Gi","memory":"6584Mi","pods":"58","vpc.amazonaws.com/pod-eni":"18"}}
{"level":"INFO","time":"2024-11-04T15:17:03.759Z","logger":"controller","message":"deleted nodeclaim","commit":"901a5dc","controller":"nodeclaim.termination","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","NodeClaim":{"name":"spot-kngcp"},"namespace":"","name":"spot-kngcp","reconcileID":"08090e8d-9157-4e71-a7a9-77a950fe68e1","Node":{"name":""},"provider-id":"aws:///ap-southeast-1b/i-0865258af1c615c6f"}
{"level":"INFO","time":"2024-11-04T15:17:44.539Z","logger":"controller","message":"deleted nodeclaim","commit":"901a5dc","controller":"nodeclaim.termination","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","NodeClaim":{"name":"spot-fkcd6"},"namespace":"","name":"spot-fkcd6","reconcileID":"8f517525-5166-4d6b-8fe4-cf9858a0dc6b","Node":{"name":""},"provider-id":"aws:///ap-southeast-1c/i-031a02558cc13b16a"}
{"level":"INFO","time":"2024-11-04T15:31:49.255Z","logger":"controller","message":"found provisionable pod(s)","commit":"901a5dc","controller":"provisioner","namespace":"","name":"","reconcileID":"08e5d957-23e7-4de8-b2cf-2108b51ac98b","Pods":"microservice-public-web-manage/microservice-public-web-manage-5549b5b56f-xrk9w, microservice-admin-dashboard/microservice-admin-dashboard-cbf6cb68d-h6zhm, worker-zzz-scan-stage-complete/worker-zzz-scan-stage-complete-7746f99f85-smk8v, worker-auto-unlock-bug-report/worker-auto-unlock-bug-report-c4477f77d-bbc4c, keda/keda-metrics-apiserver-c5b6b66c-lsgsx and 2 other(s)","duration":"26.295328ms"}
{"level":"INFO","time":"2024-11-04T15:31:49.255Z","logger":"controller","message":"computed new nodeclaim(s) to fit pod(s)","commit":"901a5dc","controller":"provisioner","namespace":"","name":"","reconcileID":"08e5d957-23e7-4de8-b2cf-2108b51ac98b","nodeclaims":1,"pods":7}
{"level":"INFO","time":"2024-11-04T15:31:49.268Z","logger":"controller","message":"created nodeclaim","commit":"901a5dc","controller":"provisioner","namespace":"","name":"","reconcileID":"08e5d957-23e7-4de8-b2cf-2108b51ac98b","NodePool":{"name":"spot"},"NodeClaim":{"name":"spot-2nm9f"},"requests":{"cpu":"1780m","memory":"1981Mi","pods":"14"},"instance-types":"c5.xlarge, c5a.2xlarge, c6a.2xlarge, c6i.xlarge, c7i-flex.xlarge and 6 other(s)"}
{"level":"INFO","time":"2024-11-04T15:31:52.384Z","logger":"controller","message":"launched nodeclaim","commit":"901a5dc","controller":"nodeclaim.lifecycle","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","NodeClaim":{"name":"spot-2nm9f"},"namespace":"","name":"spot-2nm9f","reconcileID":"797858ad-e717-4641-9673-ae97537038fa","provider-id":"aws:///ap-southeast-1c/i-0bc5178ae30d55bca","instance-type":"c5.xlarge","zone":"ap-southeast-1c","capacity-type":"spot","allocatable":{"cpu":"3920m","ephemeral-storage":"35Gi","memory":"6584Mi","pods":"58","vpc.amazonaws.com/pod-eni":"18"}}
{"level":"INFO","time":"2024-11-04T15:33:02.965Z","logger":"controller","message":"deleted nodeclaim","commit":"901a5dc","controller":"nodeclaim.termination","controllerGroup":"karpenter.sh","controllerKind":"NodeClaim","NodeClaim":{"name":"spot-5q86d"},"namespace":"","name":"spot-5q86d","reconcileID":"5ef88cce-50cd-4d4c-8014-ed005543fbba","Node":{"name":""},"provider-id":"aws:///ap-southeast-1c/i-0665d2caaa8e38f76"}

Nodeclaim
kg nodeclaim
NAME         TYPE        CAPACITY   ZONE              NODE   READY     AGE
spot-2nm9f   c5.xlarge   spot       ap-southeast-1c          Unknown   3m14s

k describe nodeclaim spot-2nm9f
Name:         spot-2nm9f
Namespace:
Labels:       karpenter.k8s.aws/instance-category=c
              karpenter.k8s.aws/instance-cpu=4
              karpenter.k8s.aws/instance-cpu-manufacturer=intel
              karpenter.k8s.aws/instance-ebs-bandwidth=4750
              karpenter.k8s.aws/instance-encryption-in-transit-supported=false
              karpenter.k8s.aws/instance-family=c5
              karpenter.k8s.aws/instance-generation=5
              karpenter.k8s.aws/instance-hypervisor=nitro
              karpenter.k8s.aws/instance-memory=8192
              karpenter.k8s.aws/instance-network-bandwidth=1250
              karpenter.k8s.aws/instance-size=xlarge
              karpenter.sh/capacity-type=spot
              karpenter.sh/nodepool=spot
              kubernetes.io/arch=amd64
              kubernetes.io/os=linux
              node.kubernetes.io/instance-type=c5.xlarge
              topology.k8s.aws/zone-id=apse1-az3
              topology.kubernetes.io/region=ap-southeast-1
              topology.kubernetes.io/zone=ap-southeast-1c
Annotations:  compatibility.karpenter.k8s.aws/kubelet-drift-hash:<PHONE_NUMBER>1425564585
              karpenter.k8s.aws/ec2nodeclass-hash:<PHONE_NUMBER>3262261599
              karpenter.k8s.aws/ec2nodeclass-hash-version: v3
              karpenter.sh/nodepool-hash:<PHONE_NUMBER>594823858
              karpenter.sh/nodepool-hash-version: v3
              karpenter.sh/stored-version-migrated: true
API Version:  karpenter.sh/v1
Kind:         NodeClaim
Metadata:
  Creation Timestamp:  2024-11-04T15:31:49Z
  Finalizers:
    karpenter.sh/termination
  Generate Name:  spot-
  Generation:     1
  Owner References:
    API Version:           karpenter.sh/v1
    Block Owner Deletion:  true
    Kind:                  NodePool
    Name:                  spot
    UID:                   f7479647-2be2-4c33-88eb-0261218ad48f
  Resource Version:        60672567
  UID:                     84b5c89d-9192-475a-96b3-3a1931748888
Spec:
  Expire After:  720h
  Node Class Ref:
    Group:  karpenter.k8s.aws
    Kind:   EC2NodeClass
    Name:   default
  Requirements:
    Key:       kubernetes.io/os
    Operator:  In
    Values:
      linux
    Key:       node.kubernetes.io/instance-type
    Operator:  In
    Values:
      c5.xlarge
      c5a.2xlarge
      c6a.2xlarge
      c6i.xlarge
      c7i-flex.xlarge
      m5.xlarge
      m6a.xlarge
      r5.xlarge
      r6i.xlarge
      t2.2xlarge
      t3.2xlarge
    Key:       karpenter.sh/nodepool
    Operator:  In
    Values:
      spot
    Key:       karpenter.sh/capacity-type
    Operator:  In
    Values:
      spot
    Key:       kubernetes.io/arch
    Operator:  In
    Values:
      amd64
  Resources:
    Requests:
      Cpu:     1780m
      Memory:  1981Mi
      Pods:    14
Status:
  Allocatable:
    Cpu:                        3920m
    Ephemeral - Storage:        35Gi
    Memory:                     6584Mi
    Pods:                       58
    vpc.amazonaws.com/pod-eni:  18
  Capacity:
    Cpu:                        4
    Ephemeral - Storage:        40Gi
    Memory:                     7577Mi
    Pods:                       58
    vpc.amazonaws.com/pod-eni:  18
  Conditions:
    Last Transition Time:  2024-11-04T15:31:52Z
    Message:               Node not registered with cluster
    Reason:                NodeNotFound
    Status:                Unknown
    Type:                  Initialized
    Last Transition Time:  2024-11-04T15:31:52Z
    Message:
    Reason:                Launched
    Status:                True
    Type:                  Launched
    Last Transition Time:  2024-11-04T15:31:52Z
    Message:               Initialized=Unknown, Registered=Unknown
    Reason:                UnhealthyDependents
    Status:                Unknown
    Type:                  Ready
    Last Transition Time:  2024-11-04T15:31:52Z
    Message:               Node not registered with cluster
    Reason:                NodeNotFound
    Status:                Unknown
    Type:                  Registered
  Image ID:                ami-0be82d98bb3e7f36c
  Provider ID:             aws:///ap-southeast-1c/i-0bc5178ae30d55bca
Events:
  Type    Reason             Age                  From       Message
  ----    ------             ----                 ----       -------
  Normal  Launched           3m28s                karpenter  Status condition transitioned, Type: Launched, Status: Unknown -> True, Reason: Launched
  Normal  DisruptionBlocked  86s (x2 over 3m26s)  karpenter  Cannot disrupt NodeClaim: state node doesn't contain both a node and a nodeclaim

Versions:

Chart Version: 1.0.7
Kubernetes Version (kubectl version): v1.31.0-eks-a737599

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Have you followed our troubleshooting guide around nodes not joining the cluster? https://karpenter.sh/docs/troubleshooting/#node-launchreadiness

In my case, the problem was I placed the node instance in the public subnet, but the subnet did not Enable auto-assign public IPv4 address. This issue can be closed


Something question about datasets processed (with CLIP-ViT/B-32)

Could you help me? You provide several commonly used datasets that you have already processed (with CLIP-ViT/B-32) in datasets.json. So how I should do in my datasets and how to keep processed image order as origin unprocessed image?
Another question is why running train.py needs the parameter with val_dataset?

You can have a look at dataset_tool.py and revise some code there to maintain order, or their original names.
That is optional, you do not have to use that if you are running on your own dataset. It is used following previous works because they evaluate metrics on COCO by generating images with text from validation set, instead of text from training set.


creating hex arrays for Rom module

While the below mentioned problem has been fixed, there are still a few open points. Namely

The current code base implies that the Rom Module is the only use case for the mentioned function. Is this correct?
What should happen when an empty list is passed to the function?
Currently the function is used to construct an array before that array is handed to the Rom object
https://github.com/es-ude/elastic-ai.creator/blob/28ca7d82e8cd0fc7040f3bc30eada4367f214ae6/elasticai/creator/vhdl/generator/rom.py#L17
and
https://github.com/es-ude/elastic-ai.creator/blob/28ca7d82e8cd0fc7040f3bc30eada4367f214ae6/elasticai/creator/examples/generate_vhdl_files/generate_rom_vhd.py
Do you see a specific reason why we should not simply use that function inside the Rom class?

The new function is clear and short, but with a bug, when the hex_list only contains one element, the length of the output array should still be 2. which means you need to append a zero to it. below is an example of quick fix

import math
hex_list = ["1111"]

hex_number_length = len(hex_list[0])
next_exponent_for_power_of_two = math.ceil(math.log2(len(hex_list)))
if next_exponent_for_power_of_two==0:
  next_exponent_for_power_of_two = 1
number_of_missing_zeros = 2**next_exponent_for_power_of_two - len(hex_list)
res_val=hex_list + ["".join(["0"] * hex_number_length)] * number_of_missing_zeros
print(res_val)

Originally posted by @SuperChange001 in https://github.com/es-ude/elastic-ai.creator/pull/83#discussion_r866670501

the ROM module is also used here
In practice, we would never need an empty room. Unless a ROM contains all values with ZEROS is the use case we need. Therefore, if the array(list) is empty, the generator should warn the user.
Put the hex/binary formatting process into the ROM class would be perfect.
In addition, since we are kind of open to the task of rewriting the rom_generation, so I attach a pre-designed template here.
template_rom.vhd.txt

This issue is resolved by https://github.com/es-ude/elastic-ai.creator/pull/94


bvh

How should my own bvh file be structured in a uniform way with Mixamo's bvh file?

Closing this issue as we are already discussing it in issue #12. Feel free to re-open.


Detect IRCAM audio file

Reference

http://www-mmsp.ece.mcgill.ca/Documents/AudioFormats/IRCAM/IRCAM.html

I've found sample files and identification, but I cannot find the MIME type.
@odan, I think it should be closed because it's very outdated.


Uncaught TypeError: _ctx.$t is not a function

hi there, I want to publish my vue3+vite package to npm but after publishing, I encountered "Uncaught TypeError: _ctx.$t is not a function" in a test project and my package is not working, any suggestions... ?
PS: I'm using options api
here is my vite.configs.js
import { defineConfig } from 'vite'
import vue from '@vitejs/plugin-vue'
import vueI18n from '@intlify/vite-plugin-vue-i18n'

// https://vitejs.dev/config/
const path = require("path")
export default defineConfig({
  build: {
    lib: {
      entry: path.resolve(__dirname, 'src/install.ts'),
      name: 'vcp',
      fileName: (format) => `vcp.${format}.ts`
    },
    rollupOptions: {
      external: ['vue'],
      output: {
        exports: 'named',
        globals: {
          vue: 'Vue',
          vcp: 'Vcp'
        }
      }
    },
  },
  plugins: [
    vue(),
    vueI18n({
      include: path.resolve(__dirname, 'src/assets/translations.ts'),
      compositionOnly: false,
    })
  ],
  server: {
    port: 8080
  },
  resolve: {
    dedupe: ['vue'],
    alias: {
      "~": path.resolve(__dirname, "./src"),
      "@": path.resolve(__dirname, "./src"),
    },
  },
})

Originally posted by @alireza0sfr in https://github.com/intlify/bundle-tools/discussions/122

I realized that include only accepts JSON and YAML files therefore I added a JSON file for my translations but the issue still remains.

close, I've answered in https://github.com/intlify/bundle-tools/discussions/122#discussioncomment-2844181


Tool Export Variable Evaluation Order Incorrect

Godot version
4.0 beta 10
System information
Kubuntu 18.04 NVIDIA GeForce GTX 1660
Issue description
When creating a @tool-enabled file with get and set directives, the set directive gets called before the scene is added to the scene tree. When using this mechanism to initialize file-level object state, this causes the game to halt with an error accessing scene objects.
If I start with this:

It results in:

So far, so good.
But if I uncomment line 15, the script stops with an access error:

I would expect the object's resources to be loaded (and _ready() called) prior to the set and get directives being called.
Steps to reproduce

Create an @export var that specifies get and set directives.
Implement set and/or get functions that access file-level resources.
Run the project (or relevant file), and watch the program die at the file-level resource access.

Minimal reproduction project
Github won't let me upload my minimum reproduceable project, for some reason.

@export variables are initialized very early, right after _init() (or even before, I always forget xd). You'd need @onready @export variable in your case, but it's not supported yet, see this proposal: https://github.com/godotengine/godot-proposals/issues/325
As a workaround, you can await ready in the setter if the node can't be found yet.


Change rake to a runtime dependency from a development dependency

Hi @bolandrm , we have a gem that depends on SassC, one of our end-users was unable to install it. The fix was gem install rake. This crops up in the default ruby package in the Ubuntu repos, which supplies a rake executable in /usr/bin/ but does not include the gem.
This change makes sense to me since libsass is compiled in a Rake task.

Relates to #60 and #42


Generate documentation (in literate programming style) for anonymize_ccx_pipeline_log.py

Generate documentation (in literate programming style) for anonymize_ccx_pipeline_log.py script

Implemented in https://github.com/cloudhubs/logscraper/pull/159


Module structure and decompose

Landed in 96c0c972649782e99b6870681d19812cc63d26df

@tshemsedinov if you don't add PR-URL metadata, you can just rebase, there's no need to cherry-pick.

@tshemsedinov though there must be a way to go to a PR with discussion from a commit.


Add Compatibility Version

This is a basic functionality of a framework and it is required for it to function in a dependency graph. If the compatibility doesn't exist or is set to 0 you'll get:
dyld: Library not loaded: @rpath/SnapKit.framework/SnapKit   Referenced from: /private/var/containers/Bundle/Application/2EAB6DCA-EF3C-461B-97FD-1666A542B2E7/XXX.app/Frameworks/XXX.framework/XXX   Reason: Incompatible library version: XXX requires version 1.0.0 or later, but SnapKit provides version 0.0.0
As you can see, SnapKit tells dylid that it is compatible with version 0.0.0 which is not true.

@hlineholm @LinusU this needs to be rebased against develop not master otherwise it's hard to decipher the changes at the moment

#559 has already been merged already and no longer needs to be in here :)

Believe this is obsolete now


I cant reset

Im not able to reset my save, ive typed in my name, yes, ok, YES, OK and other things.

Screen recording 2024-09-11 09.11.54.webm

@SCOTTLANDFOREVAAHHHHHHHH Ah, I see, you are supposed to type either Y for yes or N for no (Because of the [Y/N] after the question).
I see how that could be a bit confusing, so I should probably change it to a confirmation button instead 😅

Welp im stupid. Thanks for the help.


use pinned deps to specify sysimage contents

see #4
This was easier than anticipated; I just put the extraction of pinned deps in yet another docker stage.
README is updated to reflect new convention.

@kolia great use of a multi-stage build for this :italian-chef-kiss:


Delete worker nodes using dynamic client and add an option to remove Kubernetes packages when resetting the cluster

What this PR does / why we need it:
This PR refactors the reset command to use dynamic client instead of kubectl over SSH tunnel for deleting worker nodes.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #447
Fixes #397
Release note:
Fix reset failures if there are no Machine objects or Cluster-API CRDs are not deployed
Add the --remove-binaries flag which removes Kubernetes binaries after resetting the cluster

/assign @kron4eg

This is a yamled flake.
/retest

/hold
I'll take care of #397 as well

/hold cancel

/retest

/retest

/retest

/retest

/retest

/hold
This is a second flake related to reset not being able to delete the MachineSet. It should be fixed before merging the PR.

/lgtm

/lgtm

/retest

/hold cancel


[desktop_webview_window v0.2.3] Webview crashes if window is manipulated before fully initializing

When the webview window is created, if you press minimize too early the entire application crashes.
Reproduce Steps
Steps to reproduce the behavior:

Open Flutter app
Call WebviewWindow.create
As soon as window opens, press minimize
App crashes

Expected behavior
The window should minimize without crashing

Version (please complete the following information):

Flutter 3.22.3, Dart 3.4.4
Windows 11 Pro | 23H2 | 22631.3880
desktop_webview_window v0.2.3

EXCEPTION_ACCESS_VIOLATION_READ / 0x0

webview_window::WebView::UpdateBounds (web_view.cc:202)

note that adding null check resolves issue and everything behaves as expected, haven't delved deep enough to see if has any repercussions

Here's the parameters the webview is initialized with

May I ask under which file to add the empty check? I also encountered the same problem

If the window is closed before complete initialization, Webview will also crash

如果在完成初始化之前关闭窗口，Webview 也会崩溃


Blank App (C++/WinRT) project begins with “winrt_”

Just passing along feedback from a customer (for context, and customer contact info for repros/questions/whatnot, please see https://github.com/MicrosoftDocs/windows-uwp/issues/2254).
It seems that if the name of a new Blank App (C++/WinRT) project begins with “winrt_” (lower-case), then the build fails. I’ve pasted in the details from the original issue.
It is related to the name of the solution. I've tried a few solution names and here is the result:
HelloWinUICppWinRT >> success
HelloWinUI_CppWinRT >> success
HelloWinUI >> success
WinRTBlank >> success
winrt_blank >> fail
winrt_demo >> fail
win_rt_demo >> success
This is the build output in the fail cases:
1>Processing input metadata file Debug\Unmerged\App.winmd.
1>Processing input metadata file Debug\Unmerged\MainPage.winmd.
1>Processing input metadata file Debug\Unmerged\XamlMetaDataProvider.winmd.
1>Saved output metadata file winrt_demo.winmd.
1>Validating metadata file Debug\Merged\winrt_demo.winmd.
1>C:\Program Files (x86)\Windows Kits\10\bin\10.0.18362.0\XamlCompiler\Microsoft.Windows.UI.Xaml.Common.targets(308,5): Xaml Internal Error error WMC9999: Name should contain ::winrt prefix
1>Done building project "winrt_demo.vcxproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

Is this issue still relevant? Should I keep it open (in which case should it be assigned to someone?), or should I close it?
Many thanks!
Steve

@stevewhims - I'll assign it to myself, thanks!

Closing this due to low customer impact and available workaround


When discarding backup email there is no way to close it.

When pressing backup key and it opens the email view with your encrypted key and stuff in there, if you press cancel and discard it. the email stays on screen and you can no longer interact with the view. Need to close app and start again.

done


Cache is not preserved between routes

When I switch from one route to another route, the same data is fetched again from server. is it normal?

Caching is not built in to Relay Modern. You have to set it up yourself.


language: node.js is incorrectly recognized as Node.js repository

This YAML file https://github.com/wblankenship/varstring/blob/163167248d9c77b353e2c6fb54b20d0ce3e54bd6/.travis.yml is recognized as a Node.js repository by the linter. However, it is an incorrect value, and we run it as a Ruby repo.

try language: node_js


Replace & improve terminal emulator

Addressed in https://github.com/learn-co/learn-ide/pull/409/commits/937767afdde7daf1e0d3529066dcef345019a488:

#408
#394
#393
#389
#382
#200
#173
#412
resizable terminal that correctly fits its rows and avoids overflow

Todo
Parity:

[ ] terminal pop out/in (related: #253)
[x] handle openPath & openLab (addressed in https://github.com/learn-co/learn-ide/pull/409/commits/1c4b3bdee6eccfbbf1a03874859c598a54fe0267)
[x] copy/paste (addressed in https://github.com/learn-co/learn-ide/pull/409/commits/122b52443b996e73aeb20374ddaec33aa44d034f)

[x] keymaps
[x] Edit menu (Edit > Copy/Paste)
[x] terminal context menu

[x] keymap to toggle terminal focus (addressed in https://github.com/learn-co/learn-ide/pull/409/commits/f3060d2a3048272c88a5c225b9e5d9a10f279d35)
[ ] while terminal is focused, keymaps to increase/decrease/reset font size
[ ] persistent terminal font size (if changed, changes persist after restarting app)
[ ] configurable terminal background & font colors

New stuff:

[ ] #253
[ ] #285
[x] #106 (addressed in https://github.com/learn-co/learn-ide/pull/409/commits/9c764142dc6da5f47fd9ddb0b694304e55b376f9)
[ ] All terminal colors configurable (for reference https://github.com/sourcelair/xterm.js/issues/59)

@drewprice how do you pop out the terminal in the IDE?

@mobesa check out the Learn IDE's help center

Pending a QA pass, I think we will go ahead and merge as is.


Missing feature multitenancy

Migration to postgres
I read the tutorial Multitenant Business Application with PostgreSQL, to get multitenant support in my SaaS-Application.  The package used in the tutorial, is the deprecated cds-pg package.
After that I found out that there is no support in cds-js/postgres yet.
Is the support available soon? are there other ways to achieve multitenancy with postgres?
Thanks in advance for your help.

@BastiInnovation I had a quick look over the linked blog post and there seems to have been quite a bit that changed in the area of multitenancy as well as the Postgres support.
Therefor I want to give some more insight into the topic.
Probably the most important change is that the fundamental implementation of multitenancy support in CAP has been put into the @sap/mtxs package (npm). Which currently has native support for SQLite and HANA as documentation here. In principle it should be quite straight forward to add Postgres support as well.
There is a big difference between Postgres and the already supported databases. Where currently a Postgres service instance is provided by the cloud platform directly and relies on the application owners to manage the database fully. Where HANA comes with additional services that provide multitenancy specific functionalities to ensure secure isolation between the tenants.
Here is a simple illustration of how it work in the case of HANA. Where the credentials are handled in a central place in the platform. This broker service comes with a lot of security features. Which either cannot be supported for Postgres or would come with a lot of added complexity for CAP to support.
sequenceDiagram
    CAP->>HANA Broker: create tenant
    HANA Broker-->>CAP: tenant credentials
    CAP->>HANA tenant: connect
    Note right of HANA tenant: credential rotation
    CAP->>HANA tenant: connect fails
    CAP->>HANA Broker: read tenant
    HANA Broker-->>CAP: tenant credentials
    CAP->>HANA tenant: connect

So how it would look like for Postgres is as follows:
sequenceDiagram
    CAP->>Postgres: connect with credentials
    CAP->>Postgres: create tenant schema
    Postgres-->>CAP:
    CAP->>Postgres: deploy tenant contents
    Postgres-->>CAP:
    CAP->>Postgres: switch tenant schema
    Postgres-->>CAP:
    CAP->>Postgres: switch tenant schema
    Postgres-->>CAP: ...

So the following security relevant topics should be kept in mind:

A single user owns all data
All tenant isolation is done on schema level
All data is stored in a single physical database instance
No tenant level encryption possible

Is there any update on the support for multitenancy and extensibility in PostgreSQL? Thanks

is there any plan to support this?, thanks.

@BastiInnovation I had a quick look over the linked blog post and there seems to have been quite a bit that changed in the area of multitenancy as well as the Postgres support.
Therefor I want to give some more insight into the topic. Probably the most important change is that the fundamental implementation of multitenancy support in CAP has been put into the @sap/mtxs package (npm). Which currently has native support for SQLite and HANA as documentation here. In principle it should be quite straight forward to add Postgres support as well.
There is a big difference between Postgres and the already supported databases. Where currently a Postgres service instance is provided by the cloud platform directly and relies on the application owners to manage the database fully. Where HANA comes with additional services that provide multitenancy specific functionalities to ensure secure isolation between the tenants.
Here is a simple illustration of how it work in the case of HANA. Where the credentials are handled in a central place in the platform. This broker service comes with a lot of security features. Which either cannot be supported for Postgres or would come with a lot of added complexity for CAP to support.
sequenceDiagram
    CAP->>HANA Broker: create tenant
    HANA Broker-->>CAP: tenant credentials
    CAP->>HANA tenant: connect
    Note right of HANA tenant: credential rotation
    CAP->>HANA tenant: connect fails
    CAP->>HANA Broker: read tenant
    HANA Broker-->>CAP: tenant credentials
    CAP->>HANA tenant: connect

So how it would look like for Postgres is as follows:
sequenceDiagram
    CAP->>Postgres: connect with credentials
    CAP->>Postgres: create tenant schema
    Postgres-->>CAP:
    CAP->>Postgres: deploy tenant contents
    Postgres-->>CAP:
    CAP->>Postgres: switch tenant schema
    Postgres-->>CAP:
    CAP->>Postgres: switch tenant schema
    Postgres-->>CAP: ...

So the following security relevant topics should be kept in mind:

A single user owns all data
All tenant isolation is done on schema level
All data is stored in a single physical database instance
No tenant level encryption possible

Hi all,
Just for curiosity I’ve spend a few hours last night creating the Postgres lib for the mtx package, and it works like a charm ;) ( t0 created, and also new subscriptions ), there are few comments:

one db is created for mtx
Every tenant has their own user and schema
The db encryption is for the entire db
You can have the multi AZ at db level

The night was not long enough to see how this package  behaves with this approach, but I hope to not find any blocker.

Any update on this?
Seems like an obvious next step to get more feature-parity with the commitment to the Open Plugin Architecture...


Missing newline

I have an installer config with some gpu_mem stuff and a custom /boot/config.txt containg:
# Enable optional hardware interfaces
dtparam=i2c_arm=on
dtparam=i2s=on
dtparam=spi=on

After installation completed /boot/config.txt looked like this:
# Enable optional hardware interfaces
dtparam=i2c_arm=on
dtparam=i2s=on
dtparam=spi=onstart_x=1
disable_camera_led=1

gpu_mem=512

Looks like a newline was missing from my config.txt and when the final config.txt was produced it didn't add a newline, causing the dtparam option to get garbled: dtparam=spi=onstart_x=1

Included in v1.4.0 it last minute.

Still seeing the issue, refs #36

Confirmed it's fixed.


ℹ About Page

Challanges

About App / App Design / App Architechture
App Idea and Application
App Source Code Link
About Page for developers
Must include about developers link to github profile opening in browser

i want to work on "about page" which one is the "about page " in figma ?
it will be better if you put comments in figma designs above every page corresponding to the issue assigned in the git hub.

You can start working on it ... and about design you can get inspiration from existing design in the app .
Use any plugin to open browser from app when user wants to see source code .

for more reach me out i can tell you more details on a meet

hey lakhan is this project still active?


Admin Panel + BuildPack

Hey, I'm working on adding an admin panel and heroku buildpack to 0x-launch-kit in a fork. Wondering if I should keep it as a separate fork, or merge into 0x-launch-kit.

Closing this out. Due to heroku's ephemeral file system, its going to be a big change to get this running on heroku


Make Sequence::sb_size_log2 const fn

Work towards #1464.
The same cannot be done with corresponding FrameInvariants functions, because trait bounds other than Sized are not supported with const fn currently (FrameInvariants has a Pixel bound).

Coverage decreased (-81.2%) to 0.0% when pulling 60362bd43e05b72ed3069a49bca98594982a8e80 on rzumer:const_fn_2 into cced3ac3626e281b221155c0542b65f6e4e41ed3 on xiph:master.

Keep the #[inline(always)], it might still be needed (we should doublecheck).

Done and force-pushed (also to remerge properly/fix Clippy warning)


Just straight dicts, not Context’s for django Templates

This is new in Django 1.8: https://docs.djangoproject.com/en/1.8/releases/1.8/#cleanup-of-the-django-template-namespace

Easy to digest and concise PR  :green_apple: I made the same change to another project after a port to Django 1.8 and had no sideeffects in the last weeks. So :+1: on this


Use default version when tag is null

This fixes issue #878 by falling back to default version when tag is not specified.  If both are undefined, it will use 'latest'.  If 'latest' also does not exist, then the current error will be shown.

Changes Unknown when pulling 46a68dff0fcaeaf2c96fa73ea3c780e45e71a7e7 on feature/issue-878 into ** on develop**.


Clarify cloud provider optionality in installation docs

#1057  👈
main

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

Join @mkandler and the rest of your teammates on  Graphite


Missing Terms of Service acknowledgement in registration

[ ] Add to an appropriate Epic.

Tester (if different from issue creator):
Site: hcommons-dev.org
URL: https://registry.hcommons-dev.org/registry/co_petitions/petitionerAttributes/385649/token:8c5ffde049a23361a3ff41d1c44b72fe5a91ee89
Test Account: hctester50
Area (CORE, sites, profiles, groups, general): registration
If issue discovered on dev, does it also exist on prod? no
Describe the issue

When I enrolled with the last user I didn't notice, and it prompted me to acknowledge it before I created the username/password. This is replicable.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Optional details, if they seem relevant
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

SCG is working on a patch to put the T&C acknowledgment back near the beginning of the flow for us.

This is fixed on dev


How to call another function in one function?

I think I saw it before somewhere but I can't find it anymore...
I have a master-workers model, master function will decide different strategies for workers, and all the workers will deal with the same data with decided strategies, then master will collect and compare their results. All workers should do their work simultaneously. Master and worker may be built with different languages(go and python).
However, I don't find a way to directly do that in fdk-go/fdk-python. I don't need Flow either because it calls the next function only after the previous one is finished, while I want something like a map-reduce model.
Is there any way to do so in fdk? Or should I just manually launch a http request to trigger them? If it is the latter, how can I pass custom data to it instead of using test.json, ~and how can the master collect workers' results?~ (I guess I can encode it inside http request)
Thanks.
Describe the results you expected:
I can call other functions within a function
Output of fn version (CLI command):
Client version is latest version: 0.5.8
Server version:  0.3.588

Additional environment details (OSX, Linux, flags, etc.):

Ubuntu 16.04.5
Started using fn start without any options.

Hi and welcome!  First of all, with a new fn-2-trigger model you have exactly 2 ways to call a function:

invoke API (HTTP POST only)
HTTP trigger API (supporting all HTTP methods)

With an invoke API all you need is a function ID that you'd want to call.
All you need is to use the following pattern:
<fn-api-url>/invoke/{fn_id}
(for instance, http://localhost:8080/invoke/01CQR004J4NG8G00GZJ0000002)
With an HTTP trigger API, you may need an app name and trigger path, URL pattern:
<fn-api-url>/t/{app}/{trigger_path}
(for instance http://localhost:8080/t/testapp/test-trigger).
The trigger path is what you specify in fn's trigger section in your func.yaml:
triggers:
- name: test-trigger
  type: http
  source: /test-trigger

So, in order to bypass the next function ID or trigger path you may configure them through a function's config.
Please note, with an invoke API all your custom HTTP headers will appear as is, but with an HTTP triggers all your custom headers will be prefixed with Fn-Http-H-. At this moment, we work hard on bringing a coverage for a new http-stream format, it will speed up your functions due to certain tech improvements we did in Fn recently. Currently only FDK-go|java support that, FDK-python and FDK-node are the next ones.
With an FDK changes, we'd change the context contract and an FDK will provide 2 different contexts:

invoke context (is the default one your function receives)
http context (can be instantiated from an invoke context, An HTTP context will decap/encap your HTTP headers)

I hope this would help.

You've mentioned test.json if you're referring to fn test - it will go away in a day or two. The only way to test your functions is to write your own unit tests that would not use Fn at all.

Thanks a lot for your timely reply. But how can I get the function id from fdk-python? fn list functions pythonapp only shows the function name and image, not the fn_id. Also, could you show where to put custom data when doing invoke, should I put it into http header or into body with json?
About the http-stream, if I get it right, do you mean fdk-python is slower but workable, or it is not compatible right now?
Lastly as for the test.json, I did not run fn test, instead, I directly run fn deploy --app myapp --local and it seems that the function responds with the data in test.json.

Another thing(perhaps irrelavant) is that I've noticed fdk-python provides decorations
@decorators.fn(fn_type="sync", dependencies={         "requests_get": requests.get     })
Can I achieve my goal with fdk.application.decorators.fn and fn_type="async"? So that I can create many functions in just one python file rather than creating lots of .py files.

But how can I get the function id from fdk-python? fn list functions pythonapp only shows the function name and image

You can get fn ID with fn inspect fn <app>  <fn>

Also, could you show where to put custom data when doing invoke, should I put it into http header or into body with json?

echo -e '{"key": "value"} | 'fn invoke <app> <fn> or use curl for the rest of things.

About the http-stream, if I get it right, do you mean fdk-python is slower but workable, or it is not compatible right now?

http-stream PR merged, a new version 0.0.34 of FDK-python released less than an hour ago.

But how can I get the function id from fdk-python?

With ctx.FnID() you may get your current Fn ID, but only works with FDK 0.0.34

Another thing(perhaps irrelavant) is that I've noticed fdk-python provides decorations
@decorators.fn(fn_type="sync", dependencies={ "requests_get": requests.get })
Can I achieve my goal with fdk.application.decorators.fn and fn_type="async"? So that I can create many functions in just one python file rather than creating lots of .py files.

That code is outdated and needs to be fixed later.

@WhoisZihan Fn apps decorators will work again shortly (https://github.com/fnproject/fdk-python/pull/54). There was a need to switch to Fn API V2 to work with triggers/fns. So, now it works again, all I need is an approval from other team members to land the PR and release 0.0.35 version.

Closing this for now. If you'd like to ask more questions - feel free to hit me on Slack.

Hi @denismakogon I know this has been closed for a few days, but I tried HTTP trigger API today and it keeps giving me

{"message":"error receiving function response"}

Here is the directory of my example application (Both root function and sub function are written in Go, deployed with fn deploy --all --local)
.
├── app.yaml
├── func.go
├── func.yaml
├── Gopkg.toml
├── LICENSE
└── LRFunction
    ├── func.go
    ├── func.yaml
    └── Gopkg.toml

The root function can decide different strategies for each LRFunction, and LRFunction will just do simple calculation.
I am sure LRFunction is working because I can get correct result directly from it if I bypass root function (command is echo -e '{"name": "example", "dataset": "1,2,0.62,1,3,0.85,2,3,0.98", "params": "0.13,0.23"}' | fn invoke cvalid cvalid-serverless).
I did almost the same as the source code of fn cli, except that I use HTTP trigger API rather than invoke API. Here is the code snippet in root function, which calls LRFunction
var modelTrigger = map[string]string{
	"LRFunction": "http://localhost:8080/t/k-fold/lrfunction-trigger",
}

func invokeModel(model string, data []byte, wg *sync.WaitGroup) {
	defer wg.Done()

	// This line gets the trigger url from an array
	triggerURL := modelTrigger[model]
	req, err := http.NewRequest("POST", triggerURL, bytes.NewReader(data))

	httpClient := http.Client{Transport: http.DefaultTransport}
	resp, err := httpClient.Do(req)
	defer resp.Body.Close()
	if err != nil {
		fmt.Println("Error in model " + model)
		return
	}
	// other stuff
}

Then I get the 502 error, am I missing anything?
I thought about invoke API that fn cli source uses, but that would make the function more complicated, so I'd like to see if I can do it in pure HTTP API.
Thanks

Maybe because your root function can't talk to localhost. If you want to call one function from another you need to make your Fn run over the public IP, but not on the localhost. For fast prototyping try https://ngrok.com to get a public IP address.

Also, you need to check your function logs (for that you need to set up the syslog or papertrail) to see what's going on.
{"message":"error receiving function response"} also means that your function is corrupted somehow, you need to debug that.

Thanks, I will try ngrok first.
About the log, does Fn only support sending log to server through network? The same question for retrieving result of async function, it seems the only way is sending it to X-callback-url?

In Fn API v2 there’s no async functions, for now. Http triggered function works in sync mode only.
And yes, our logs API as it is right now will go away, so, the only way is to use syslog.

I see, but why removing async functions in v2, it doesn't seem useless anyway.

A new type of almost-async function’s will be delivered as a new trigger with a 1-2 month.

I get it. Thanks,


@zenstackhq/trpc : $Schema import compatibility update

🆕   Changes related to #1177

Update trpc generated router $Schema import to use import * as instead of default import
Generate test project routers

✅   Tests ran successfully

Summary by CodeRabbit

Refactor

Updated import method for $Schema in the router schema generation process to enhance compatibility and maintainability.

Thank you @jbruxelle . The change looks great! I'll merge and include it in the upcoming release around the end of this week.


Issue 58 - Add Superseded Info to Job Detail API

Implemented according to the original issue, except root_superseded_job_id and superseded_job_id fields are replaced with nested objects per the standard REST conventions.

LGTM


Add create ticket action

Description
Added a create ticket action to allow for zendesk tickets to be created
How can (was) this tested?
Published and tested with an agent in Sema4.ai Studio version 1.0.60.
Was able to successfully create a ticket.
Screenshots (if needed)
Checklist:

[x ] I have bumped the version number for the Action Package / Agent
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation - README.md file
[ ] I have updated the CHANGELOG.md file in correspondence with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x ] OAuth2: I have made sure that action has necessary scopes (works in whitelisted mode)

This PR is missing CHANGELOG.md update.


Print all package versions at start of trainer for diagnostics

To check in particular regressions when packages get updated.
AB#4223

All package versions are now specified in environment.yml - does this make this FR redundant?