feat(dao) move ssl certificates/snis to new DAO

This PR continues #3315:
Migrations

ssl_certificates table renamed to certificates
ssl_servers_names table renamed to server_names. Added a new field (id, UUID) and made that the PK. But also added indexes for the name field.
In postgresql these changes involved altering the tables. In Cassandra the changes involved copying all the records from the original tables to new tables (this was done with a custom migration helper which has its own PR).

Admin API

/snis is now /server_names and is automatically generated by the new DAO admin api generator-from-schema.
/certificates is also generated, but it has some overrides, to deal with three "special" things:

the server_names accepts a comma-separated string or an array of strings. It
allows "setting the server names in one go" when creating (POST) or updating
(PATCH) a certificate.
the server_names attribute is automatically added to the certs when getting
each one individually or when getting a list of them.
a certificate can be referenced by its id or also by the name attribute of
any of its associated server_names.

PUT is not allowed any more.
Deactivates some /server_names endpoints that are not needed any more:

GET /server_names is not needed any more now that GET /certificates also returns the list of server names associated to every cert
POST /server_namesis not needed sincePOST /certificates/id/server_names` is easier to use and does the same
GET /server_names/[name]/certificate was not needed since you can do GET /certificates/[name]` to get the same info
The only active server names endpoints are DELETE /server_names/x (to remove a server name quickly), PATCH /server_names/x (to move server names quickly between certificates), GET /server_names/x (for symmetry with DELETE and PATCH).

DAO

ssl_certificates and ssl_servers_names' schemas translated to new DAO's schemas (renamed to certificates and server_names).
I had to include a read-before-delete in all the generated dao:delete* methods. This read was needed in order to propagate the structure through the worker events so it could be deleted properly (worker events can use any attribute
from the entitity, for example for the server names you need their name attribute. So if you delete a server name using its id you still need to read it in order to propagate an event with the name field).
certificates has a custom DAO to deal with the server_names pseudo-attribute and for "looking for a certificate using one of its server names' name"
server_names also has a custom DAO for "using the server names as if they were lists of strings", which is easier to do than "handling them as instances of the server_names schema".

Tests

the DAO:delete* methods now returns the entity which was erased instead of true, so some tests had to be changed from is_true to is_truthy.
the tests for PUT where just removed
I also removed the admin api tests from spec-old-api because they were a carbon copy of the new ones (there are no APIs, Services or Routes involved in SSL). I double-checked that we don't do anything "extra" on it before removing.
Added tests for the new admin API endpoints.

Related work

The migrations are tested in https://github.com/Kong/kong-upgrade-tests/pull/5
Since we're doing the read-before-delete on the DAO we probably don't need the delete_by_field methods on the strategies (we can use the PK, we have it because of the "read"). We could remove those methods.

Thanks for the review @bungle I have added changes for most things. Answering to your general comments: My initial PR was much smaller - it moved only certificates to the new DAO, leaving ssl_server_names in the old DAO and not touching the admin API. But I was told to include all the other changes as well.
Both you and @hishamhm have mentioned that you prefer other name instead of server_names. I can rename server_names to snis everywhere (table name, DAO, admin api). Would you consider this better?

Both you and @hishamhm have mentioned that you prefer other name instead of server_names. I can rename server_names to snis everywhere (table name, DAO, admin api). Would you consider this better?

This would make the endpoints more compatible with the current ones, right? If so, I think it would be a better idea. We can think of better names when/if we merge the entities in the future.

Renamed server_names to snis pretty much everywhere, tests passing.

I made a change that makes kong/admin/routes/certificates.lua smaller: instead of "adding new methods" to the Certificates DAO (like Certificates:page_with_name_list or Certificates:insert_with_name_list) I just overrode the default (Certificates:page and Certificates:insert). These are the methods the generated admin api uses, so I didn't have to override those any more.
I did not dare changing Certificates:select because it is used in other places besides the Admin API, generating more queries for getting all the SNIS every time a cert was selected. So I still had to override one of the generated methods.

The PUT was moved to later work, and it is not blocking this merge.

```POST /snisis not needed sincePOST /certificates/id/server_names` is easier to use and does the same

where I can find the related doc?


PACKAGES: mod-main tests

Adding tests in mod main for:
1 . Campaigns
2. Users
3. Support Roles
4. Answer
Cleaning and fixing bugs.
Adding steps in makefile to easy run.

issue #400

Great. Have s good weekend


Frontend/about us page

This is the about us page.
It contains an explanation about the project and the course

Codecov Report

Merging #275 (e2fb231) into develop (8cd3a6e) will decrease coverage by 0.03%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##           develop     #275      +/-   ##
===========================================
- Coverage    99.04%   99.00%   -0.04%
===========================================
  Files           52       53       +1
  Lines         2409     2415       +6
===========================================
+ Hits          2386     2391       +5
- Misses          23       24       +1

Impacted Files
Coverage Δ

app/main.py
96.66% <ø> (ø)

app/routers/about_us.py
83.33% <83.33%> (ø)

Continue to review full report at Codecov.

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


Update alexa-remote2 dependency to 3.8.1

Hi @cakebake! Following the discussion here https://github.com/cakebake/node-red-contrib-alexa-remote-cakebaked/issues/22 @Apollon77 has released a new version 3.8.1 of alexa-remote2, which fixes this wonderful node-red node. This PR only sets the dependency to 3.8.1, I believe everything else needed to release an update is probably automated by scripts.
Thank you!

Project has been forked -> https://github.com/bbindreiter/node-red-contrib-alexa-remote2-applestrudel

🤔


Implement support of binding

Because a lot of implementations will be "binded" to some annotation, it have to be implement some binding logic where the Annotation can be binded to some class (implementation of some interface). For inspiration look at binding implementations to their interfaces in Google Guice (IOC).

Nearly finished, last step is implementation of abstract binding factory.

Posponed. The binding factory will be designed after #33

Interface for binding factory is done, now can be created abstract binding factory which uses Binder, Bindings...

Moved implementation for binding factories into another issue. Resolved.


Allow to set the position of the spinner

Thanks for the rewrite.
It is currently not possible to change the position from bottom right to top right. It would be great if you could support changing the spinner position again. I've looked through the source code and couldn't find a wired config option for it.

Yeah it’s not supported in the rewrite yet but I do plan on adding it back next. It should only require modifications to window.lua, with extra care taken about how the feature interacts with y_offset.


fix: wrap folder name with quotes

Fixes

Wrap folder name in quotes when running kustomize

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

I'm guessing we need to do this in more places? and also when running Helm?


Use the library with Arduino Uno

Hi buddy (4-20mA),
I just found that your github ModbusMaster library perfectly match my needs. Namely, I have an ElNet LT Energy&Powermeter (http://www.getelnet.com/wordpress/2009/10/15/elnet-lt/) that supports Modbus RTU communication an has built-in Functions 03, 04, 06, and 16.
I'm also planning to use max485 ESA transceiver and Arduino Uno board.
Now, I would like to know is it possible to use your library with Arduino Uno board? I saw that you made testings by using Duemilanove board that is similar to Uno bot not the same.
Also, I'm confused how to connect max485 pins:

For RO and DI pins, I know that they are connected to Arduino serial pins (0 and 1, respectively)

For A and B pins, I know that they are connected with the ElNet LT Powermeter device

For Vcc pin I know it is connected to 5V DC

What I don't know is where to connect on Arduino board RE and DE pins of max485. Can you help me with this?

Thank you very much for your time and effort to help me.
Sincerely,
Bojan.

Thr library worked fine with duemilanove. I hvnt check it with arduino uno. And short pin RE and DE and connect it with any digital pin of arduino. Bt configure that digital pin to set directionality of max 485 as max 485 is a half duplex

Did you get your question answered?

Closed due to inactivity.


Correct formatting of bullet list

Just a missing blank line; apparently that's what's missing to render that page correctly.
This is what's online right now:

This is the expected rendering:

Codecov Report

Merging #7794 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7794      +/-   ##
==========================================
+ Coverage   41.31%   41.31%   +<.01%
==========================================
  Files         474      474
  Lines       63837    63837
==========================================
+ Hits        26375    26377       +2
+ Misses      34024    34021       -3
- Partials     3438     3439       +1

Impacted Files
Coverage Δ

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

models/repo_list.go
73.09% <0%> (+1.01%)
:arrow_up:

models/unit.go
67.56% <0%> (+5.4%)
:arrow_up:

Continue to review full report at Codecov.

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


WIP dual-stack apiserver documentation

Enhancement: https://github.com/kubernetes/enhancements/issues/2438

/assign @nate-double-u

/milestone clear
/unassign

@danwinship The v1.24 milestone no longer tracks the KEP that this PR supports. Thanks for your effort on this documentation; I recommend saving this work for a future release.
/close
If you feel that this PR has been closed in error, please feel free to reopen it against another branch.
Thank you!


[OperationalInsights] Prepare release 1.0.0

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

One comments here https://apiview.dev/Assemblies/Review/0ab7afb3131d4eacb1bfc1b0230fece8#0f7e5b49e7834449948add6835edc823

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.OperationalInsights


VS Live Share Global Search not woking

Issue Type: Bug
After sharing code via "VS Live Share" Search is not working. Please check this.
VS Code version: Code 1.24.1 (24f62626b222e9a8313213fb64b10d741a326288, 2018-06-13T17:41:50.128Z)
OS version: Darwin x64 17.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5575R CPU @ 2.80GHz (4 x 2800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
8.00GB (0.14GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

jshint
dba
0.10.19

vscode-eslint
dba
1.4.12

php-debug
fel
1.12.3

node-debug2
ms-
1.21.1

vsliveshare
ms-
0.3.335

debugger-for-chrome
msj
4.7.0

vscode-icons
rob
7.24.0

wordpress-snippet
tun
1.1.2

This isn't supported by Live Share yet. Please file other live share issues on their repo at https://github.com/MicrosoftDocs/live-share

This isn't supported by Live Share yet. Please file other live share issues on their repo at https://github.com/MicrosoftDocs/live-share

This isn't supported by Live Share yet. Please file other live share issues on their repo at https://github.com/MicrosoftDocs/live-share


Failing to Reproduce Qualitative Results

Hey, thank you for the release!
I downloaded diffcalib-best.zip, diffcalib-pcd.zip, and res101 from BaiduNetDisk but the UNet for diffcalib-pcd only has 8 channels which causes the following error in diffcalib_pipeline_rgb_12inchannels_depth.py, which passes rgb + incident + depth latents into the UNet (12 channels):
    incident_pred_raw, normal_pred_raw = self.single_infer(
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/home/alex/DiffCalib/diffcalib/diffcalib_pipeline_rgb_12inchannels_depth.py", line 373, in single_infer
    noise_pred = self.unet(
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs
)
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/diffusers/models/unets/unet_2d_condition.py", line 1162, in forward
    sample = self.conv_in(sample)
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 458, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/opt/conda/envs/diffcalib/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 454, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Given groups=1, weight of size [320, 8, 3, 3], expected input[10, 12, 96, 96] to have 8 channels, but got 12 channels instead

Using the diffcalib-best 12-channel UNet outputs outputs way worse than the Readme shows:

Can you help me understand how diffcalib-pcd should be used?

Hello, it's my fault for uploading the incorrect checkpoint of U-Net. Now you can download the correct diffcalib-pcd.zip from this link. Thanks for pointing out the problem, I've corrected the link in README.

@shuiyued thank you!


(consoleapp) Switch from ReadLine to getline.cs

Fixes part of #182. :tada: While not being perfect in any way, the achievement made here in a single evening is a noticeable improvement from the previous behavior. I'm willing to merge this as a first-step.
(Couldn't have been done without the previous effort of @migueldeicaza in https://github.com/mono/mono/blob/master/mcs/tools/csharp/getline.cs, thanks a lot. :pray:)

Missing before merge: add changelog entry.


BUG: wrong file name for contact.html

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hello
I have found that when we click on contact us page it shows 404 page not found error

Actually this is due to wrong file name.

As we can see it seems Contact(1).html ..but it should be contact(1).html.
@Anushkabh please assign this issue to me under GSSOC 2024 and add necessary labels to it
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@SalehaTabassum Assigned

Fixed in #445


fix: added if statement to filter out dir blob files

Fixes #62  🦕
Current version of sample doesnt check if blob is directory or .json file. Then, it  downloads as bytes and tries to parse json from the dir blob file which will cause error.

Can you add a test that covers this json / not json input?

add a test that covers this json / not json input?
I will add that test in a separate PR.
I just created new github issue for tracking. #67


Adding files to the Image resizing

I am a GSSOC 21 participant. Kindly, assign me this issue to work on the image resizing.

Hey @mansi75, it has already been assigned.
Please check #13.


ReferenceError: navigator is not defined

Hey Jose,
Started using this wrapper for a build just today - thanks for putting time into building and maintaining it.
It works great for me, but when my colleague pulled down my changes he was hit with the following error:
/Users/{user}/localhost/{project}/node_modules/react-plyr/lib/defaultProps.js:95
   language: (navigator.language || navigator.userLanguage).split('-')[0]
              ^
ReferenceError: navigator is not defined

For context, the code lives here in react-plyr/lib/defaultProps.js:
// Captions settings
  captions: {
    active: false,
    language: (navigator.language || navigator.userLanguage).split('-')[0]
  }

https://developer.mozilla.org/en-US/docs/Web/API/Navigator
Any ideas why this might be occurring for him?

Could be an issue with Server Side Rendering. Possible fallback solution:
export function getInterfaceLanguage() {
  const defaultLang = 'en-US';
  if (typeof navigator === "undefined") {
    return defaultLang;
  }
  if (!!navigator && !!navigator.language) {
    return navigator.language;
  } else if (!!navigator && !!navigator.languages && !!navigator.languages[0]) {
    return navigator.languages[0];
  } else if (!!navigator && !!navigator.userLanguage) {
    return navigator.userLanguage;
  } else if (!!navigator && !!navigator.browserLanguage) {
    return navigator.browserLanguage;
  }
  return defaultLang;
}

Credit to @stefalda https://github.com/stefalda/react-localization/blame/65c882b6c4bfaceb57575f2c9c0ae639b16c4521/src/utils.js#L2-L16

I went to make a pull request and I see this has been already fixed in https://github.com/xDae/react-plyr/pull/30 !
I'll apply the fix above!

Hi @AllanPooley, as you mentioned, the fix is already merged to master, Im going to release a new patch version hopefully today

Hey guys,
Any update on this issue ? Trying to build with Gatsby and without touching this library, I had to defer the loading of Plyr to avoid server side rendering which is a bit of shame.
Thanks


Text Color is not working

When you set Text Color to any color value, it will remain white.
I tried to set options.txtColor in JavaScript Initialization Code, but it always white.
Thanks.

It's not possible to set the text color through JavaScript Initialization Code, it can be done only in the plug-in attributes ( Text Color ).

Ok, I see.
But it will be nice if plug-in attributes ( Text Color, Background Color ) can support Substitutions: Application, Page Items and System Variables.
Thanks.


[WIP] Add MMSB Tutorial

OK you won't be able to sample from a model with discrete random variables using pure HMC. Try to use PG or the Gibbs interface.

To kind of echo what @xukai92 said, the issue is that
z_nm ~ Categorical(pi[n])
z_mn ~ Categorical(pi[m])

are discrete variables, so they need to be sampled using one of the discrete samplers. I'd try a sample call like the one from the HMM tutorial:
g = Gibbs(1000, HMC(2, 0.001, 7, :m, :T), PG(20, 1, :s))
c = sample(BayesHmm(y, 3), g);

In your case, you'd pass :z_nm and :z_mn to PG and everything else to NUTS.

The chain did draw samples successfully but my reconstructed graph seems quite inaccurate. Does this require more iterations to run? This already ran for about an hour or so for 1000 iterations.

I think 1,000 iterations is probably not enough for 167 parameters. A visual inspection of the trace plots would suggest that few to none of these have converged. Plus, I think that the Dirichlet prior doesn't work super well with HMC (link).
I wonder what happens if you sample with just PG? Probably won't get very far in terms of convergence, but it might be interesting to see.

I sampled used the PG sampler. It was fast but it gives hard assignments for the clusters (which makes sense). Also tried the HMC with 4000 iterations but it seems that we still have not reached convergence. Should I try using a smaller graph or try some other sampling algorithm like SGLD? Maybe I could sample the distribution differently as described in the paper that you linked though I am not sure how effective it would be.

Why not both? Try SGLD first and then reduce the graph size and go back to the original sampler specs with HMC.

SGLD would not be an appropriate sampler for this case since it is a constrained problem. I tried changing the model definition a bit according to this MMSB implementation. Even after reducing the graph size, I am still not able to obtain convergence. Can changing the parameters of the samplers or increasing the number of iterations make a difference?

First, I think you should first try a 2x2 to make sure the model works.
Second, you're getting a ton of numerical errors with HMC. Try reducing the step size. You might also try running this on the master branch since there is a ton of stuff that's happened recently.
Third, you are getting this domain error which you should probably correct:
 DomainError with -2.220446049250313e-16:
log will only return a complex result if called with a complex argument. Try log(Complex(x)).

I would try making the Bernoulli call in your model be lower-bounded at zero:
val = max((pi[n])'*eta*pi[m], 0)
graph[n, m] ~ Bernoulli(val)

I think the model is working correctly since the graph with N=2, K=2, and N=3, K=2  is converging. I ran the latter 3 times and it perfectly predicted the generated data. Maybe, for larger graphs, we would need to use VI as MCMC is too slow at convergence. Keeping the number of iterations same, the prediction was decent for N=4, K=2. Do you think I should increase the size and run this for more iterations? I thought it would get too long for a tutorial.

It's interesting that it mostly converges after ~15k iterations. There's probably room to expand either N or K, if it gets interesting results.

Closing this and sending a PR to TuringExamples.


Groups creation for property

In my API, I have a list of companies and I want to be able to retrieve all of them or retrieve a specific one. So I have two endpoints :

GET /companies -> [Company]
GET /companies/:id -> Company

But the Company model contains a lot of data, so I want to create groups for the endpoints.
For example :

GET /companies -> [Company]

Returns
[
    { id: 0, name: "Company1", description: "BlaBla" },
    { id: 1, name: "Company2", description: "BlaBla" },
    { id: 2, name: "Company3", description: "BlaBla" }
]

GET /companies/:id -> Company

Returns
{
   id: 1,
   name: "Company2",
   description: "BlaBla",
   creationDate: "2021-02-02",
   data: []
},

Is this possible with property decorators or do I need to use a custom solution like custom serializer ?

Of course, this is probably the most basic case scenario. Just use jsonConvert.deserializeArray to deserialize the whole array and jsonConvert.deserializeObject if you have only one object. All you need to do is set your classes up as in the ReadMe.

This is what I’ve done in my project but how can I tell, in this scenario I want an array of Company with propertyA and propertyB and in this scenario I just want one Company but with propertyA, propertyB and propertyC ?
I use JsonProperty decorator to serialize/deserialize a property but this one is always serialize/deserialize, can I choose when I want it in my returned object?
Thank you for your response

Yes, you can use optional properties as well (see docs for ConvertingMode).

I found a way to achieve what I wanted by using PropertyConvertingMode :
@JsonProperty("value", Number, PropertyConvertingMode.PASS_NULLABLE)
Thank you Andreas

Yes, this is the way I would do it. The TypeScript type should then be value: number|null = null.


[ObjC] error EM0009: The feature Returning type TextReader from native code is not currently supported by the tool

error EM0009: The feature Returning type TextReader from native code is not currently supported by the tool

This appears to be the general "we don't support things that don't match a pattern we've listed" error:
https://github.com/mono/Embeddinator-4000/blob/master/objcgen/objcgenerator.cs#L1225

After doing some poking, I've verified this is the general "we only support limited return/param types" with classes with export. The HasClass \ HasProtocol check above is what we're failing.
This hits us if we try to use a Stream, or a other BCL types.
One could imagine even if this works, how would you use that TextReader, unless we also exposed every single method on it to ObjC as well.
I would suggest changing your API to return an object that contains the TextReader internally and exposes whatever items you need in simpler terms.

I'm going to convert this issue into a "we should drastically improve the error message".


Update hamcrest/hamcrest-php dependency

Resolves https://github.com/graze/hamcrest-test-listener/issues/6.
/cc @garethellis36

That was fast! Thanks.


TextInput doesn't allow to change height

I'd like to change the height of TextInput to 46px - 56px.
But looks like this textInput is not able to adjust height size. Kindly please allow to change the height by css.
https://github.com/SwingbyProtocol/explorer/blob/config/201102/initial-setup/src/components/Explorer/ExplorerMain.styles.ts#L30
https://github.com/SwingbyProtocol/explorer/blob/config/201102/initial-setup/src/components/Explorer/ExplorerMain.styles.ts#L30

The <TextInput /> component supports 3 heights: 56, 48 and 40px as shown below.
<TextInput size="country" />

<TextInput size="state" />

<TextInput size="city" />

Noted, it's working. Thank you.


Integer.MAX_VALUE is not portable across clients

Bug description
The following places use numbers that are too large for the LSP spec's uinteger type, which can have max value 2^32 - 1.
https://github.com/prisma/language-tools/blob/c464b8f5a785d67b06da6bf3f273c7eafacdccb2/packages/language-server/src/rename/renameUtil.ts#L186
https://github.com/prisma/language-tools/blob/c464b8f5a785d67b06da6bf3f273c7eafacdccb2/packages/language-server/src/rename/renameUtil.ts#L190
https://github.com/sublimelsp/LSP/issues/1951#issuecomment-1071391571 (note that you cannot use Number.MAX_SAFE_INTEGER)
How to reproduce
Run the "rename" request in Sublime Text.
Observe stack trace:
Traceback (most recent call last):
  File "C:\Sublime Text\Lib\python33\sublime_plugin.py", line 1488, in run_
    return self.run(edit, **args)
  File "C:\Users\DELL\AppData\Roaming\Sublime Text\Installed Packages\LSP.sublime-package\plugin/edit.py", line 38, in run
  File "C:\Sublime Text\Lib\python33\sublime.py", line 1493, in text_point_utf16
    return sublime_api.view_text_point_utf16(self.view_id, row, col_utf16, clamp_column)
OverflowError: int too big to convert

Expected behavior
Rename works.

Minor correction, but the uinteger max value is defined to be 2^31 - 1, not 2^32 - 1.

Hmm, I'm not the most familiar with Sublime here but I installed it so I could take a look.
But it seems to be working for me?

These are the packages I installed after getting a fresh Sublime through Homebrew

Could you please verify if this is still happening for you @rwols?

We guard for this here: https://github.com/sublimelsp/LSP/pull/1952
Prisma should still fix this, as it's a spec violation and may result in bad behavior in other text editors.

Hi everybody! I'm from the IntelliJ WebStorm team and just several days ago we released a new plugin that supports Prisma ORM.
This issue backfired quite hard, because we use the java library lsp4j for the LSP integration, and the Range type fields are defined as 32-bit integers, which conforms to the specification. So, we have an exception during the deserialization of the response and can't really do anything about it on our side 😢.
There are only several usages of Number.MAX_SAFE_INTEGER or Number.MAX_VALUE, so it shouldn't be hard to fix. Otherwise, it looks like such issues also will occur in other java based IDEs. Thank you in advance!
https://github.com/prisma/language-tools/blob/main/packages/language-server/src/diagnosticsHandler.ts#L27
https://github.com/prisma/language-tools/blob/main/packages/language-server/src/util.ts#L26
https://github.com/prisma/language-tools/blob/main/packages/language-server/src/util.ts#L33
https://github.com/prisma/language-tools/blob/main/packages/language-server/src/MessageHandler.ts#L222


Package request: Decent Sampler

Decent Sampler
I am asking for the standalone app to be packaged.
This way you can just plugin a MIDI keyboard and play music.
I am not asking for automatic VST, VST3, *.so, linking.
So you can just run the DecentSampler binary directly and that is it.
I have tried just running it directly in NixOS but I get this error:
bash: ./DecentSampler: cannot execute: required file not found
Project description
The Decent Samples plugin is a FREE sample player plugin that allows you to play sample libraries in the DecentSampler format (files with extensions: dspreset and dslibrary). It is available in the following formats:
- Windows (32-bit/64-bit): VST, VST3, AAX, Standalone
- Mac (Intel & M1): VST, VST3, AU, AAX, Standalone
- Linux (Intel 64-bit): VST, VST3, Standalone
- [iOS: AUv3, Standalone](https://apps.apple.com/us/app/decent-samples/id1530441977)

This app automatically downloads samples from the net, so it needs a mutable directory.
I currently do not know the default dir, but will update when I figure that out.
Metadata

homepage URL: https://www.decentsamples.com/product/decent-sampler-plugin/
sources URL: https://www.dropbox.com/sh/dwyry6xpy5uut07/AABBJ84bjTTSQWzXGG5TOQpfa?dl=0
Current Latest Dynamic-linking source URL: https://www.dropbox.com/sh/dwyry6xpy5uut07/AABBJ84bjTTSQWzXGG5TOQpfa?dl=1&preview=Decent_Sampler-1.8.12-Linux-Dynamic-x86_64.tar.gz
license: not clear at the moment but it clearly states it is a FREE sample player
platforms: unix, linux, darwin, windows, iOS

Important Note about Linux builds
There are two different builds for linux.
One tries to staticly link needed dependencies and the other dynamically.
Dropbox API downloading
You need to have ?dl=1 to get the direct download link.

This is the Arch AUR version
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=decent-sampler-bin

This is the closest I could get, but when I try to run DecentSampler binary it segfaults.
{ lib, stdenv, fetchzip, patchelf, ripgrep, alsa-lib, freetype, nghttp2 }:

stdenv.mkDerivation rec {
  pname = "decent-sampler";
  version = "1.8.12";
  static = "-Static";
  dynamic = "-Dynamic";
  type = static;

  src = fetchzip {
    url = "https://www.dropbox.com/sh/dwyry6xpy5uut07/AADhKQCQZt9HYZhXL1d-6pICa/Decent_Sampler-${version}${type}-Linux-x86_64.tar.gz";
    sha256 = "sha256-9Xjo8Jk4c6dCjz6Yd0h5FKKhpRz3VZCCsbVoO/MFL1k=";
  };

  nativeBuildInputs = [ patchelf ripgrep ];
  buildInputs = [ alsa-lib freetype nghttp2 ];

  prePatch = ''
    echo Starting Patch
    patchelf --print-interpreter DecentSampler
    echo REQUIRED LIBARIES... -----------------------------
    patchelf --print-needed DecentSampler
    echo SEARCH ...............
    ldd DecentSampler | rg 'not found'
    echo REQUIRED LIBARIES... -----------------------------
  '';

  installPhase = ''
    ls
    mkdir -p "$out/bin"
    cp ./DecentSampler "$out/bin/"
    cp ./DecentSampler.so "$out/bin"
    cp -r ./DecentSampler.vst3 "$out/bin"
  '';

  preFixup = let
    libPath = lib.makeLibraryPath [
        nghttp2  # libnghttp2.so
        alsa-lib # libasound.so
        freetype # libfreetype.so
    ];
  in ''
    patchelf \
      --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)" \
      --set-rpath "${libPath}" \
      $out/bin/DecentSampler
  '';

  meta = with lib; {
    description = "An audio sample player";
    longDescription = ''
        Decent Sampler is an audio sample player.
        Allowing you to play sample libraries in the DecentSampler format
        (files with extensions: dspreset and dslibrary).
        It claims to be free but we currently cannot find any license
        that it is released under.
    '';
    homepage = "https://www.decentsamples.com/product/decent-sampler-plugin/";
    license = licenses.unfree;
    platforms = platforms.linux;
  };
}

Tried using autoPatchelfHook
{ lib, stdenv, fetchzip, patchelf, ripgrep, alsa-lib, freetype, nghttp2, autoPatchelfHook }:

stdenv.mkDerivation rec {
  pname = "decent-sampler";
  version = "1.8.12";
  static = "-Static";
  dynamic = "-Dynamic";
  type = static;

  src = fetchzip {
    url = "https://www.dropbox.com/sh/dwyry6xpy5uut07/AADhKQCQZt9HYZhXL1d-6pICa/Decent_Sampler-${version}${type}-Linux-x86_64.tar.gz";
    sha256 = "sha256-9Xjo8Jk4c6dCjz6Yd0h5FKKhpRz3VZCCsbVoO/MFL1k=";
  };

  nativeBuildInputs = [ autoPatchelfHook patchelf ripgrep ];
  buildInputs = [ alsa-lib freetype nghttp2 ];

  installPhase = ''
    install -m755 -D DecentSampler $out/bin/DecentSampler
  '';

  meta = with lib; {
    description = "An audio sample player";
    longDescription = ''
        Decent Sampler is an audio sample player.
        Allowing you to play sample libraries in the DecentSampler format
        (files with extensions: dspreset and dslibrary).
        It claims to be free but we currently cannot find any license
        that it is released under.
    '';
    homepage = "https://www.decentsamples.com/product/decent-sampler-plugin/";
    license = licenses.unfree;
    platforms = platforms.linux;
  };
}

I have raised an issue with the dev here:
https://www.decentsamples.com/bugtracker/view.php?id=37
Anyone who wishes to comment on the bugtracker to help get it resolved will be most welcome.
Thanks

I have some good news. I managed to get Decent Sampler working on NixOS via pkgs.buildFHSEnv.
Since this is the first time I am using this build method I am not sure how this is working, but I am posting the details here now to keep a good log of what is happening. I also am not sure how to package this for nixpkgs.
Here I am able to run DecentSampler binary without it being patched (manually or via autoPatchelfHook).
So here you just download DecentSampler as a normal Linux download then point this shell.nix file's runScript attribute at the Decent Sampler binary (./nixpkgs/source/DecentSampler) and it just works™.
I simply copied the example from https://ryantm.github.io/nixpkgs/builders/special/fhs-environments/ and added freetype and nghttp2.
I have tried removing  ++ (with pkgs.xorg; [ libX11 libXcursor libXrandr ]) but it goes back to segfaulting.
So this might be the reason?
# file: shell.nix
# use nix-shell to run
{ pkgs ? import <nixpkgs> {} }:

(pkgs.buildFHSEnv {
  name = "simple-x11-env";
  targetPkgs = pkgs: (with pkgs; [
    udev
    alsa-lib
    freetype
    nghttp2
  ]) ++ (with pkgs.xorg; [
    libX11
    libXcursor
    libXrandr
  ]);
  multiPkgs = pkgs: (with pkgs; [
    udev
    alsa-lib
  ]);
  runScript = "./nixpkgs/source/DecentSampler";
}).env

Can anyone explain why this works when the other methods end with a segfault?

Did some more digging and it apears that only pkgs.xorg.libX11 is needed.
If I remove that then we go back to segfaulting.

{ pkgs ? import <nixpkgs> {} }:

(pkgs.buildFHSEnv {
  name = "decent-sampler-env";
  targetPkgs = pkgs: (with pkgs; [
    xorg.libX11

    alsa-lib
    freetype
    nghttp2

  ]);
  runScript = "./nixpkgs/source/DecentSampler";
}).env

Is the simplest FHSEnv needed....

{ pkgs, lib, stdenv, fetchzip, autoPatchelfHook, alsa-lib, freetype, nghttp2}:

stdenv.mkDerivation rec {
  pname = "decent-sampler";
  version = "1.9.8";
  static = "Static";
  dynamic = "Dynamic";
  type = static; # this will get patched by autoPatchelfHook

  src = fetchzip {
    url = "https://www.dropbox.com/sh/dwyry6xpy5uut07/AACSicfMpcDXSbYg2UqIIymLa/Decent_Sampler-${version}-Linux-${type}-x86_64.tar.gz?dl=0";
    sha256 = "sha256-O/0R70tZOmSGgth6nzt4zPiJr1P8890uzk8PiQGnC6M=";
  };

  nativeBuildInputs = [ autoPatchelfHook ];
  buildInputs = [ pkgs.xorg.libX11 alsa-lib freetype nghttp2 ];

  installPhase = ''
    mkdir -p $out/{bin,lib}
    install -m755 -D DecentSampler $out/bin/
    install -m755 -D DecentSampler.so $out/lib/
  '';

  meta = with lib; {
    description = "An audio sample player";
    longDescription = ''
        Decent Sampler is an audio sample player.
        Allowing you to play sample libraries in the DecentSampler format
        (files with extensions: dspreset and dslibrary).
        It claims to be free but we currently cannot find any license
        that it is released under.
    '';
    homepage = "https://www.decentsamples.com/product/decent-sampler-plugin/";
    license = licenses.unlicense;
    platforms = platforms.linux;
  };
}

Updated this to have pkgs.xorg.libX11 still doesn't work. It seems that pkgs.buildFHSEnv is working for another reason other than simply pkgs.xorg.libX11.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/trying-to-contribute-to-nixpkgs-but-only-buildfhsenv-works-for-binary/36579/1

I have finally figured it out by copying this package's methodology: https://github.com/NixOS/nixpkgs/blob/12a0a9da151da0750988c8dd489c5794d7cc7c83/pkgs/development/tools/hover/default.nix
Here is the final and tested (on my machine) default.nix file:
# default.nix
{ lib
, stdenv
, fetchzip
, buildFHSEnv
, alsa-lib
, freetype
, nghttp2
, xorg
, }:

let
  pname = "decent-sampler";
  version = "1.9.4";

  decent-sampler = stdenv.mkDerivation {
    inherit pname version;

    src = fetchzip {
      # dropbox link: https://www.dropbox.com/sh/dwyry6xpy5uut07/AABBJ84bjTTSQWzXGG5TOQpfa\

      # Internet Archive should be more reliable
      url = "https://archive.org/download/decent-sampler-linux-static-download-mirror/Decent_Sampler-${version}-Linux-Static-x86_64.tar.gz";
      sha256 = "sha256-O/0R70tZOmSGgth6nzt4zPiJr1P8890uzk8PiQGnC6M=";
    };

    installPhase = ''
      mkdir -p $out/bin
      install -m755 -D DecentSampler $out/bin/
    '';

  };
in

buildFHSEnv {
  name = pname;

  targetPkgs = pkgs: [
    decent-sampler
    alsa-lib
    freetype
    nghttp2
    xorg.libX11
  ];

  runScript = "DecentSampler";

  meta = with lib; {
    description = "An audio sample player";
    longDescription = ''
        Decent Sampler is an audio sample player.
        Allowing you to play sample libraries in the DecentSampler format
        (files with extensions: dspreset and dslibrary).
        It claims to be free but we currently cannot find any license
        that it is released under.
    '';
    homepage = "https://www.decentsamples.com/product/decent-sampler-plugin/";
    license = licenses.unlicense;
    platforms = platforms.linux;
    maintainers = with maintainers; [ adam248 ];
  };
}

@PowerUser64 just letting you know that I have now made a pull request, please add a thumbs up to it to help get it merged.... thanks


WIP: Added some missing field info to the EngineApi interface

This probably doesn't have any effect on how the engine works, but it is nice for documentation and engine API export purposes.

Added WIP tag, until we can figure out how to handle Vector stuff.

Structs that can't have access permission issues:

Vectors (IntVector, FloatVector, BoolVector)
UUID
MatrixF

Closed in favor of #2366


Find the system limits

Recommendations in terms of memory & minimum Amazon instance required

Check https://github.com/EyeSeeTea/ESTools/wiki/Dhis2-Benchmarking#dhis2androiddashboard
Some notes:

Performance is similar across dhis2 versions (2.25, 2.26, 2.27).
Let's take for example 2.27 (current). We can see that the "mean elapsed time by user"/"users" define a linear increase, ~ 20 secs/user.
As we saw with the ab tests, the DHIS2 server hangs when more than 40 users hit the server on the "UsageController" (the main culprit: https://play.dhis2.org/android-previous1/api/reportTables/xIWpSo5jjT1/data.html)
The success rate is pretty good until the server hangs.
The performance is not limited by memory.  When heap memory is doubled (-Xmx4096m), we have the same behaviour (and the same hang for 40 simultaneous users in reportTables).
An increase on CPU should reduce roughly proportionally the mean times, but more tests on real scenarios should be made.
This dhis2 server crash should be further investigated.
How many users can the instance manage? If the server hang cannot be fixed, let's say we can accept 75% of the load before hanging: ~ 30 users. On the other hand, if the server hang can be fixed, new tests should be made to see when the success rate of requests goes down.

I've added a wiki page as a preliminary analysis of the hangs:
https://github.com/EyeSeeTea/ESTools/wiki/DHIS2-server-hangs-on-heavy-concurrent-requests


[Feature] your website is need makeover

Is your feature request related to a problem? Please describe.
Hello @swapnilsparsh,
I’ve taken a look at your website and I believe it has great potential. I would like to offer my services to revamp it and enhance its user-friendliness. I’m eager to undertake this project as part of GSSoC 2024. Please consider assigning this task to me.
Describe the solution you'd like.
User Experience Audit: Conduct a thorough review of the current website to identify areas that can be improved for better user engagement and ease of use.
Design Overhaul: Implement a modern, clean design that aligns with your brand and improves the overall aesthetic appeal.
Navigation and Structure: Simplify the navigation to ensure that users can find information quickly and intuitively.
Describe alternatives you've considered.
Template-Based Redesign: Instead of a custom design, you could opt for a high-quality, pre-designed template. This can be a cost-effective and time-saving option, though it might limit customization.

please assign this task to me under Gssoc 2024

@swapnilsparsh please assign  this task to me under Gssoc 2024

@pandeyji711 Can you share some designs before I can assign you the issue.

@swapnilsparsh, I am actively working on this task and am confident that my contributions will significantly enhance it. Could you kindly assign this task to me under GSSoC 2024, Level 3?
https://github.com/swapnilsparsh/30DaysOfJavaScript/assets/154527778/3515f8e9-7ea6-467a-a4ac-f3d4c1ccbe71

@pandeyji711 We need to show the photos.

@swapnilsparsh I will add this please assign this  in Gssoc24 and give some level

@pandeyji711 I want UI like the below photo and Good Luck 👍🏻

@swapnilsparsh what function you want in live button
and is you have collection of image or i have to search for it

@pandeyji711 When clicking on live button then that particular project should open and clicking on github that particular github project folder should open
You can find all the images here

okk

@swapnilsparsh please review and merge with Gssoc and level 3 label


Workspace trust is based on path, not authority + path

Connect to two different remotes that happen to have folders with the same path
Trust one of them
The other gets automatically trusted

Would be the same with a local folder + remote folder with the same path. Seems like the authority (remote name) also needs to be considered
https://user-images.githubusercontent.com/323878/110219622-417e4600-7e75-11eb-8f08-2da0f8f11f27.mp4

ssh remote trusted paths should probably be stored on the remote instead of using any paths on the local machine. that would resolve this issue and makes sense so if you connect to a remote box from multiple devices you don't have to retrust.
what do you think?

I would assume that trust data would be subject to settings sync anyway. Do we store any other user data besides remote settings on the remote? I'm not sure. And should I be able to untrust a workspace from a local window?

I would assume that trust data would be subject to settings sync anyway.

No, we trust on the machine level (we talk about folder paths, not repositories)

And should I be able to untrust a remote workspace from a local window?

Perhaps, though I'm not sure how that should work. The SSH scenario has 2 machines that need to trust the source I suppose but it is a user's choice. e.g. I would not expect that I connect to a remote and open a folder that is shared between users on the remote and it is already trusted due to another user's choice.

Perhaps, though I'm not sure how that should work

I bring it up because it would be a point in favor of only keeping trust data locally. If that data is kept on the remote, then consider this scenario. If I have trusted folders on some remote, and I want to untrust them then open them, here is what I do:

Have some local window already open
Open the workspace trust manager
I don't see my remote folders
I have to know that I can open an empty window on the remote, then open the workspace trust manager where I see my folders on that remote, and now I can un-trust them

Data on the remote is stored per remote user. If people are sharing accounts, that's on them, they will already be sharing settings and extensions.

Good to know thanks for clarifying.
If the only benefit to having the setting remote is so that the user doesn't have to re-trust on a new machine, I suppose local management wins out. Let's see if there are other arguments for each and what @lszomoru thinks.

The UX for remotes gets pretty confusing though. Right now we try to make it simple and being able to think about the local folder path keeps it that way. A user having to figure out which machine a path applies to and how to input that information gets more dicey.

I wouldn't worry about displaying it, we already have to do that kind of thing. But if we want to support them editing and entering paths on remotes from scratch, that might be complicated.

We'd need something different than this for remotes and how to deal with multiple remotes


Migrate check workflow to Node.js 18

This PR updates Node.js to 18 (lts/hydrogen) in the workflows/checks.yml file.
This is in alignment with the cwa-website.
Verification
npm test
and ensure that there are no errors reported.
Examine the Actions log and make sure that there are no errors.

It's uncertain why https://github.com/corona-warn-app/cwa-documentation/actions/runs/3370675638 has failed. I suspect it is just a glitch and it will work again the next time, just like it worked the previous time.

@MikeMcC399 Thanks, LGMT!


Add optimizer rule to add a limit beneath outer/cross join

Summary of the changes / Why this improves CrateDB
This adds the optimizer rule optimizer_move_limit_beneath_join which reduces the number of intermediate rows created when a LIMIT clause is used in combination with a left/right/cross JOIN operator, by adding the limit beneath the join.
Checklist

[x] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

retest this please


letting arRPC run in the background on linux

i'm currently running endeavouros and i wish there was a way to just let arRPC run in the background instead of having a terminal window open and unusable all the time because it's running node src all the time, is there a way to actually make it just run and let me use my terminal or like, make it be a background daemon or something? i dont know jackshit about nodejs so im sorry about this if it is a stupid question, but i also saw nothing about it in the README so i figured i should ask it here

Use pm2

will give a look into it, thanks for the suggestion.


Add sidebar option to show hosts

Allow a sidebar widget that shows the hosts' photos - in the row layout, it will be in the sidebar; in the grid layout it is the hero element.

Fixed in #54


"Ulmeni" is listed in the drop-down menu

Description:
"Ulmeni" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Ulmeni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Ulmeni, BUZĂU (Ulmeni)".


Cleanup

Updated the human test data with better reads taken from the tool fusioncatcher.

@apeltzer can you confirm the final changes?

Yes, thank you @matq007 !


Turn off Rainmaker (CON-1091)

How do I turn off Rainmaker? I don't see an obvious config option to turn it off.

@jonsmirl Rainmaker is not enabled in this repository, we have all examples in esp-rainmaker repository https://github.com/espressif/esp-rainmaker/tree/master/examples/matter/.
In this repo, what we have is just the declaration and definitions of the clusters required by rainmaker.
It would be helpful if you could elaborate more on the problem.

I am using some library code which kept failing on:
    esp_sntp_setoperatingmode(SNTP_OPMODE_POLL);
    esp_sntp_setservername(0, server);
    esp_sntp_init();

I thought it might be conflicting with Rainmaker which also uses NTP.
  if (!esp_sntp_enabled()) {
    esp_sntp_setoperatingmode(SNTP_OPMODE_POLL);
    esp_sntp_setservername(0, server);
    esp_sntp_init();
  }

If I protect it with esp_sntp_enabled() it doesn't fail. My stuff is working now and I did not track down which other code was turning on NTP.


Surface IP addresses of connected clients

It would be helpful if the IP addresses of the clients could be part of the logging. Working on an educational application for shared terminal window and it would help with debugging.

+1 on this idea, also if you could see the HTTP status code i.e. 404 it would mean I could use fail2ban to block bad behaviour.

Sounds a nice idea. I'll working on it soon.

Implemented.  Thanks for such a quick turn around!

Closing as the issue is solved :)


TestResult path is too long for Jenkins

Stashing/Archiving artifacts is mandatory in order for the reporting and comparison pipelines to work.
Stashing/Archiving artifacts can break when the full path of the children leaves for TestResult folder goes beyond 255 characters, especially for long test case names and perf test cases.
Example test case result with path:
C:\Jenkins\workspace\mypipelinename_PR-500-ZP7CWWYIL7BDYIQTW5GZABHKP5IZWDQKBYJDUFXDIED7VK2J5RZA\TestResults\2018-04-12-09-33-58-6691\PERF-NETWORK-TCP-SINGLE-CONNECTION-THROUGHPUT-Synthetic\iperf3Data\iperf-client-tcp-IPv4-buffer-65536-conn-1-instance-1.txt
We have to minimize this issue somehow. Some options on how to do it:

Fix Windows nodes by executing: Set-ItemProperty -Path HKLM:\SYSTEM\CurrentControlSet\Control\FileSystem -Name LongPathsEnabled  -Value 1
Reduce clutter from test case names
Reduce jenkins workspace root path length
Reduce the LISAv2 "2018-04-12-09-33-58-6691" like string

this has been resolved by adding LongPathsEnabled registry entry.


Is it possible to wrap elements of the same level?

Hi,
I know this is more of an question than an issue, but is it possible to have a div wrap all consecutively appearing elements of the same level?

Hi, not sure if I get your question. Since the items are at your disposal and item has level, you can write your own logic in the map function. But it seems to me more like a data or UI design challenge (instead of trying to write complex rendering logic, maybe format the data in a way that you can easily  present with this library. Hope it helps.

Hi, not sure if I get your question. Since the items are at your disposal and item has level, you can write your own logic in the map function. But it seems to me more like a data or UI design challenge (instead of trying to write complex rendering logic, maybe format the data in a way that you can easily  present with this library. Hope it helps.


start-server.sh vs StartServer.sh in modpack

Hello,
I am trying to get Enigmatica 4 running, but I was running into an issue where on boot it would error with Please make sure you are using the server version of the FTB modpack!
I believe the issue is that this particular modpack has a start-server.sh file instead of StartServer.sh. By renaming the file and creating a new zip, the modpack successfully boots
I don't have the brain power to dive through the code, but might it be possible to look for either script name to determine if the server modpack is valid?
sample docker-compose file in case I was doing something dumb.
---
version: "3"
services:
        minecraft-mod:
                ports:
                        - "25566:25565"
                environment:
                        EULA: "TRUE"
                        TYPE: CURSEFORGE
                        CF_SERVER_MOD: Enigmatica4Server-0.5.4-hack.zip
                        VERSON: '1.14.4'
                image: itzg/minecraft-server
                restart: unless-stopped
                volumes:
                        - /mnt/raid/docker-data/minecraft-mod:/data

Ah yes, I’m pretty sure I can add that to the start script names it looks for.

Latest image now fixes this. Thanks for reporting the issue.

Open source at work! Thanks for making an excellent docker image
On Sat., May 23, 2020, 11:00 Geoff Bourne<EMAIL_ADDRESS>wrote:

Latest image now fixes this. Thanks for reporting the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/itzg/docker-minecraft-server/issues/540#issuecomment-633070975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACYSEM4FNQNCVQM6XPM5ODRS7QJLANCNFSM4NIHAZBQ
.


CP-12797: WeaklyConnectedComponentsAlgorithm

From unknown CodePlex user on Thursday, 12 March 2009 01:51:47
Why this algorithm not stored in QuickGraph.Algorithms.ConnectedComponents ?

Form unknown CodePlex user on Sunday, 29 March 2009 08:38:46
Resolved with changeset 32640.


Extension period incorrect

Based on the doc here : https://learn.microsoft.com/en-us/microsoftteams/limits-specifications-teams
"Microsoft 365 live event limit increases
To continue supporting our customers' needs, we will extend temporary limit increases for live events through June 30, 2023, including: "
This doc still says Dec 2022

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

ID: 905e2c09-ef3e-07cc-0809-6ff171157c47
Version Independent ID: c0d33162-183d-fff8-32e9-a999f391ea51
Content: Limits and specifications for Microsoft Teams - Microsoft Teams
Content Source: Teams/limits-specifications-teams.md
Service: msteams
GitHub Login: @MicrosoftHeidi
Microsoft Alias: heidip

Hi @dipinn26592, thank you for your feedback.
Sorry but I think both are the same article and it includes the following note:

[!IMPORTANT]
Microsoft 365 live event limit increases
To continue supporting our customers' needs, we will extend temporary limit increases for live events through June 30, 2023, including:

Event support for up to 20,000 attendees
50 events can be hosted simultaneously across a tenant
Event duration of 16 hours per broadcast

Additionally, Live Events with up to 100,000 attendees can be planned through the Microsoft 365 assistance program. The team will assess each request and work with you to determine options that may be available. Learn more.

Sorry i think this is a duplicate but for a different article


Build with latest source to prevent deprecation warnings

I understand this is a no longer supported package, however, this might be helpful to others still using this package.  In #23 the source was updated to prevent deprecation warnings (thank you!), however, the dist packages were not updated
This PR builds the files in /dist with that latest source. In order to do so using newer tools, I had to update to a newer jake. So, I then went ahead updated all dependencies to latest. As part of that, I had to make a minor tweak to the UglifyJS config.
Changes:

Newer dependencies
Modified package version specifier to require leaflet-1.0.0 or newer. (Probably not required in hindsight).
Modified uglifyjs config to remove a no longer supported option.
Updates the version in package.json to 0.1.1 in case this ever gets pushed to NPM registry.

Note that running jake test results in an error, likely due to the significant jake version changes. I did not debug.

Thanks for sharing this ; we are facing a "double-bundling" of leaflet (https://github.com/etalab/transport-site/issues/3376) and it looks like this PR could solve it!
Poke @teastman if you are still around and willing to merge / release package!


Maven Package Charts

Hello, is there a maven package for the charts?

If you need help write, I had some experience with maven central

@TalosDx Sorry, I'm just a little tired. I'll do this in this weekend.

@rbbl-dev @TalosDx I have published compose-charts to maven central.
Click README to know about how to use compose-charts dependency.

@rbbl-dev @TalosDx I have published compose-charts to maven central.
Click README to know about how to use compose-charts dependency.


npm install?

Hey there!
This looks like what I am looking for, I have two questions

is there a way to install this with NPM or a CDN?

We are on AngularJS 1.6, do you think there is a better module than this one that does something similar?

Also I have this:

              <jv-json-viewer data="gitLogJSON" editor="false"></jv-json-viewer>

if gitLogJSON is an array or a stringified array, either way nothing is showing up, kinda weird

Hi @ORESoftware!

I can look into registering this as an NPM package if you would like to use this module for your project. Since I am not actively maintaining this currently, it might be best to fork the project and register an updated version. If you do get it working with your code, lmk and I will be happy to get this version into the npm registry.
I'm not sure what the compatibility is with Angular 1.6. I haven't touched this project in a long time ever since moving to other frameworks 😞 I haven't researched it, but I would assume there are alternative JSON viewer components available. Again, this one is rather old.

Make sure your div with the fixed height also is position: relative. The viewer is absolutely positioned and will need a relative parent as a reference. 👍

Yeah thanks, let me see if it works with position: relative if not, might have to find something else. Surprisingly hard to google for "JSON AngularJS view/display", not that much comes up.


Line Space Problem

Problem
Still meet the space problems mentioned in:

https://github.com/readthedocs/sphinx_rtd_theme/issues/1029
https://github.com/readthedocs/sphinx_rtd_theme/pull/1202

I have no idea how to fix it~
Environment Info

Python Version:  3.7.10
Sphinx Version:  4.4.0
Sphinx-rtd-theme Version: 1.0.0
Docutils Version: 0.17.1

I'd like to know if this is a design decision or not. As said in this issue report it has been reported twice before in #1029 and #1202, but the line space in the TOC-tree seems to sneak back into the design every so often. How can this happen or is this on purpose?


pyopenssl api support

all openssl operations does via pyopenssl api without any syscalls
support certificate generation with subjectAltName extension for prevent ERR_CERT_COMMON_NAME_INVALID error (at the base of @j31d0 fix).

Hello @zhskyy , I would like to extract the URL that is being visited and then based on some computations, decide whether the user should be allowed to continue browsing or not. Kindly help me out which part I should edit.
I will also be building upon python 3 so are there any functions or API that I should convert or change?


Add Game pid funs and List.seek

Related to TaelinArena#165
Added:

TaelinArena.Game.get_thing_by_pid
TaelinArena.Game.get_position_by_pid
TaelinArena.Game.map_stage
TaelinArena.Game.with_thing
List.seek: <A: Type> -> (A -> Bool) -> List(A) -> Maybe(A):  seeks the first ocurrence that matches a condition (same as Formality's List.find)

Please send a new PR renaming List.seek to List.find, and the current List.find to List.ifind.


Mapzen places: autocomplete UI

First part implemented by #235. Outstanding items below.

[ ] Finish PlaceAutocompleteActivity once an item has been selected and populate search box text.
[ ] Provide back arrow in PlaceAutocompleteActivity toolbar to exit without making a selection.
[ ] Clicking soft keyboard search button should dismiss keyboard (currently crashes).
[ ] Add MODE_OVERLAY for PlaceAutocompleteActivity giving the option to be styled as a dialog.
[ ] Add PlaceSelectionListener to deliver selection results.
[ ] Add ability to launch PlaceAutocompleteActivity using PlaceAutocomplete.IntentBuilder.
[ ] Add ability to set bounds bias to focus results on a given bounding box.
[ ] Add ability to filter by type (address, POI, etc.)
[ ] Add autocomplete overlay to PlacePicker component.


Add protos for TSS Roster

As part of TSS Roster effort, a protobuf is required for a RosterEntry object that will be stored in the state. The state proto is also updated to include candidate and active roster hashes.

duplicate


setTranslation with shouldMerge throws out old translations when combined with lazy loading

I'm submitting a ...  (check one with "x")
[ x ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
My setup: I use lazy loading to load separate modules, every module has its own version of a custom loader loading some translations specific to the module when they are loaded, this works fine and the different instances share translations as intended (they are not isolated)
But I also have additional translations to be loaded deeper into a module, for this I load the translation, then use "setTranslation" with shouldMerge=true to add the translations to the current ones.
When I do this, the whole thing behaves very oddly, it merges the new translations with the one from the lazy loaded version of the custom loader, but the originally loaded translations are dropped.
Expected/desired behavior
setTranslation with shouldMerge = true shouldn't drop already loaded resources
Reproduction of the problem
http://plnkr.co/edit/aNfjJyfP1aLjSAItXeps
Simple setup with a lazy loaded module and another component loading additional translations. As you can see in the AdditionalComponent, the translation loaded by the lazy loaded module ("TEST") and the additionally loaded translation ("ADD") are still available after I call setTranslation, but the original one ("TITLE") is gone.

ngx-translate version: 6.0.0

Angular version: 2.4.8

Browser: Chrome 56.0.2924.87 ]

I am seeing the same behavior.
I first load a "common" translation file in my forRoot.
Then I go to a lazyloaded module that loads its translation file in a forChild.
But the "common" translation file never get loaded...

Hello, I'm closing this issue because it's too old.
If you have a similar problem with recent version of the library, please open a new issue.


use React.FC in tsrafc

tsrafc generates
import React from 'react'

interface Props {

}

export const Component = (props: Props) => {
    return (

    )
}

a more "React" way of doing things would be generating something like
import React from 'react'

interface Props {

}

export const Component:React.FC<Props> = (props) => {
    return (

    )
}

I prefer the current template because it's more aligned with the functional programming style React is evolving towards. Inheritance is an OOP concept. If anything, the interface should be turned into a simple type.

I like the suggested change specifically because it makes it so you no longer have to declare children as a prop... I would also support just having children being added into the interface "Props" as optional by default...
For react 17; however, _tsrafc should probably be a named import of FC... And for earlier versions it wouldn't hurt having it be a named import either I guess...
`
import { FC } from 'react'
interface Props {
}
export const Component:FC = (props) => {
return (

)

}
`

Why not:
import React from 'react';

interface Props {

}

export const Component = (props: Props): JSX.Element => {
    return (

    )
}

Anyway a return type is important. ESLint/TSLint/etc often fall into hysteria without it :)

I would suggest naming Props as Component and making those two (the interface and the component name) being in the same highlight/tab group. They should receive the same name for all purposes.

"I would suggest naming Props as Component and making those two (the interface and the component name) being in the same highlight/tab group. They should receive the same name for all purposes."


godot-version-manager: Add version 1.16.1

Download, install and manage any version of Godot Engine from a simple app.
This is an application similar to godot-manager.

[x] I have read the Contributing Guide.

/verify


set the version of assertj to the version 2 release due to java 7 com…

…patibility
seen in https://github.com/raskasa/metrics-okhttp/issues/15

Not sure if it's to smart to drop java 6 support yet since selenium still works with java 6. So maybe I'll downgrade this to 1.x later and add java 6 support again. Btw how come you removed the java 7 support in your other PR? Is there a valid reason for that?

java 7 support removed by using assertj 3 which uses Optionals and this breaks a compilation

Sorry I meant to ask why you removed java 6 support in your other PR. Though now I see that the latest version of selenium is actually compiled to java 7 so using java 7 as target is actually the right way to do it.

I did not see any jdk version pinning so I set it to the last supported version. I searched for any jdk minimum version and did not found one.
So it was by accident.


⚠ Bofrost.lu has degraded performance

In e32de51, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1522 ms

Resolved: Bofrost.lu performance has improved in 9f409f4 after 15 minutes.


DefaultScheme does not require MS-outer interface

Resolves #536.
Debated which should be the default behavior. I think this makes more sense. It also makes it easier to later switch to defaulting to state-swap based things.

Passed test. Ready for review.

Wow. That was quick... Merging. Thanks

:-) wasn't too hard. Once you pointed out the problem, I knew exactly how to fix it!


30 finish data models with children and parents

change Color to String and add child and parent to Entry and Category

👍


consider a character other than □ to label 3rd dimension

Every time I see □ I think "oh a unicode character that my font is missing a glyph for", even though I know at an intellectual level it's referring to the "sheets" of the array.  I'm not sure there's a better option though.  @ericphanson suggested that ↗ might work since it's "going off in another direction", or alteratively that we "choose a random arrow from https://www.sascha-frank.com/Arrow/latex-arrows.html every time"

Yes I can see that, sorry! Something grid-like such as ⊞ or ▦ or ▨ might convey this with less surprise? Unicode doesn't seem to have a 3D cube... but does have many weird arrows like ↬, ↯, ⇲.
Should the 4th differ from the 3rd?

Petition unicode consortium to add TimeCube
...
Profit?

I think any of the grid-like would be good, as well as ❒.  I think if you wanted to have a different symbol for 3rd and 4th axis, I'd pick ▨ for 3rd (just because crosshatching is fun) and ❒ for 4th (since it's got a "brick" flavor).


Better coloring of matching braces.

Request.
Currently when we have the cursor near a ( ] } etc, the matching brace color is extremely faint, and difficult to spot in the middle of the code.
A stronger and contrasting background color ( as it happens on the Brackets editor) would help tremendously.

try downloading 1 of the 2 extensions available on the marketplace,they work preety good.
https://marketplace.visualstudio.com/items?itemName=2gua.rainbow-brackets
https://marketplace.visualstudio.com/items?itemName=CoenraadS.bracket-pair-colorizer

Thanks, Mkayxx,
I'm using rainbow brackets..It helps a lot for javascript.
However in HTML the matching tag has a light gray as background and that, for me is difficult to spot in a crowded document and dark theme.
Also if you try the high contrast theme in html, you will realize that matching tags can't be seen.
instead of a gray background, a stronger colour would be much better, in my opinion.

Try yo code, its really good for customizing your theme and brackets.Do some research and you will find what you want

will do. Many thanks.

https://github.com/Microsoft/vscode/issues/3788


Feature request: Apply replace also to README's included with [[include]]

Currently plugin seems to be replacing text only in project's README.md, but not ones which are included with include.

Hi. I haven't used this feature of TypeDoc before. Could you give an example on how you are using that include tag?

We have module, that publishes multiple modules as namespaces. Now I'm able to include README for each namespace as those would be modules. So a bit like monorepo, but without real modules.
Index.ts is like
/**
 * [[include:./module-1/README.md]]
 */
export { * as module_1 } from './module-1';

/**
 * [[include:./module-2/README.md]]
 */
export { * as module_2 } from './module-2';

Those READMEs have now both images and other md files linked, that point to wrong place in generate document.
For images things can be solved with media, and for other markdowns with @knodes/typedoc-plugin-pages, but links can't be suitable for those in the original READMEs as then those wouldn't work in repository view.
Hopefully this clarifies.

Example repo: https://github.com/Hi-Fi/typedoc-plugins-example/tree/external_pages.
Expected replacement in https://github.com/Hi-Fi/typedoc-plugins-example/blob/external_pages/src/module-2/README.md which is referred to be included in https://github.com/Hi-Fi/typedoc-plugins-example/blob/external_pages/src/index.ts#L13.

I looked into this and it turns out to be more difficult than I expected.
The markdown files that are included with [include] are parsed and included into the documentation by the renderer of TypeDoc at the end of creating the documentation. There is no way to replace things in those files the same way the plugin does in code comments or the README file.
A possible solution is to subscribe to the MarkdownEvent.PARSE event of the renderer and to do the replacing while the renderer is including the files.
The downer of this solution is that with that event the plugin can only replace things in the already pared text. So you have to specify your replacement patterns according to how the parsed MD file (which is HTML) will look like and not according to the original MD file that is included.
Would that work for you?

Hmm... I think it could work, as the paths are quite simple at least in case I tried with demo and the project where I encountered this issue.

I was thinking about this a little bit more and I really don't like the idea of replacing in the already parsed markdown content. That is not in line with how the plugin is replacing texts in comments and the README.
I think we are going to have to extend TypeDoc itself to have a nice solution, so I created a corresponding issue here: https://github.com/TypeStrong/typedoc/issues/2083

Added in 2.1.0


Use prefetcher for level merge

since level merge will read blob files sequentially, we should use prefetcher to accelerate merging.
experiment result:
Load 300GB (average value size 1KB) on Huawei NVME device, time to read blob file reduced about 50%.

Codecov Report

Merging #116 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #116      +/-   ##
=========================================
- Coverage   86.96%   86.9%   -0.07%
=========================================
  Files          44      44
  Lines        3253    3261       +8
=========================================
+ Hits         2829    2834       +5
- Misses        424     427       +3

LGTM.
nit: Maybe we can leverage Prefetch for fallback too.

LGTM.
nit: Maybe we can leverage Prefetch for fallback too.

PTAL

Please fix the sign-off

Please fix the sign-off

I signed, don't know why it's not work

Please fix the sign-off

@Connor1996 I have signed, don't know why it doesn't work

You should sign all previous commits

Let me just force merge..


Unorganized Navbar

Is your feature request related to a problem? Please describe.
The buttons on the Navbar are unorganised and are randomly oriented.
On hovering over the login button, the text of the button gets camouflagued with the background.

Describe the solution you'd like
A clean and organised navbar buttons.
Changing color of the text of the login button after hovering.

👍👍thanks for this issue, we will fix it later


Add dev branch

Issue #, if available:
Description of changes:
Adds an option for developers to test their working branch.
agbench run <config_file> --dev-branch <git_uri#branch_name>

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Is there a reason we need to provide the ability to run a remote dev branch, rather than just letting them execute the version of the code they have locally? We could support this though copying the working directory into the image and then doing a pip install -e . or by building a wheel and copying it into the image for build.
In the current solution, we require GitHub in the development loop when I'm not sure its necessary if all we need is to run working changes in the Dockerfile. Additionally, it might open up opportunity for weirdness if other people push code to the dev brach.

Is there a reason we need to provide the ability to run a remote dev branch, rather than just letting them execute the version of the code they have locally? We could support this though copying the working directory into the image and then doing a pip install -e . or by building a wheel and copying it into the image for build.
In the current solution, we require GitHub in the development loop when I'm not sure its necessary if all we need is to run working changes in the Dockerfile. Additionally, it might open up opportunity for weirdness if other people push code to the dev brach.

The reason to support remove dev branch is because we are not able to do pip install -e .. The reason is that we are trying to install the package "within" the package itself.  pip install -e . is not possible without being under the root level of the package. That's why currently we can only do pip install autogluon.bench==RELEASED_VERSION.

Is there a reason we need to provide the ability to run a remote dev branch, rather than just letting them execute the version of the code they have locally? We could support this though copying the working directory into the image and then doing a pip install -e . or by building a wheel and copying it into the image for build.
In the current solution, we require GitHub in the development loop when I'm not sure its necessary if all we need is to run working changes in the Dockerfile. Additionally, it might open up opportunity for weirdness if other people push code to the dev brach.

The reason to support remove dev branch is because we are not able to do pip install -e .. The reason is that we are trying to install the package "within" the package itself. pip install -e . is not possible without being under the root level of the package. That's why currently we can only do pip install autogluon.bench==RELEASED_VERSION.

Is there a reason why we can't manually build a wheel file and install it in the container?

Is there a reason we need to provide the ability to run a remote dev branch, rather than just letting them execute the version of the code they have locally? We could support this though copying the working directory into the image and then doing a pip install -e . or by building a wheel and copying it into the image for build.
In the current solution, we require GitHub in the development loop when I'm not sure its necessary if all we need is to run working changes in the Dockerfile. Additionally, it might open up opportunity for weirdness if other people push code to the dev brach.

The reason to support remove dev branch is because we are not able to do pip install -e .. The reason is that we are trying to install the package "within" the package itself. pip install -e . is not possible without being under the root level of the package. That's why currently we can only do pip install autogluon.bench==RELEASED_VERSION.

Is there a reason why we can't manually build a wheel file and install it in the container?

I believe building a wheel require a setup.py? Then we need to have another setup.py inside the package if I'm not wrong. It might not be best practice to keep two setup.py for the project.


Entity Hider Addition request

Is your feature request related to a problem? Please describe.
I frequently run essence to players at ZMI and I have to toggle the Entity Hider addon while running and waiting for the crafter. In detail, I enable entity hider while at the altar to see the clan member's animation and I disable it while running so I can follow the player in front of me.
Describe the solution you'd like
I would like for users to be able to select an area in which the entity hider is enabled and then disabled elsewhere. Maybe have the original options changeable for every set of coordinates given.
Describe alternatives you've considered
The only alternative would be to just have the current settings with a region selector and it is enabled while in that region and disable while not.

Just use the attacker check in entity hider I guess, otherwise no this will not be implemented


fix: Preprocess modules_dir to ensure successful module loading on Windows

Probably for consistency with vim.fn.stdpath, core.helper.path_join uses '' as the path separator on Windows. However, even on Windows, the path returned by vim.fs.find still uses /. This caused pack:load_modules_packages to error when extracting the module name from the module path. To avoid compatibility issues with other places where core.helper.path_join is used, only preprocess modules_path with vim.fs.normalize in pack:load_modules_packages.

另外，有两个地方看符号名似乎用反了
https://github.com/nvimdev/dope/blob/d7ed25abf2a5395b34e1b2a01049c704fc5cff46/lua/core/helper.lua#L10
https://github.com/nvimdev/dope/blob/d7ed25abf2a5395b34e1b2a01049c704fc5cff46/lua/core/helper.lua#L18
我没用dope脚本，不太确定


Add ExplosiveProjectile

This PR creates a new ExplosiveProjectile interface, which represents projectiles such as WitherSkulls and Fireballs which create an explosion in addition to functioning as Projectiles.
Following the discussion in issue https://github.com/SpongePowered/SpongeAPI/issues/339, I've decided not to create a separation between block damaging and non block damaging projectiles.
This resolves https://github.com/SpongePowered/SpongeAPI/issues/339

Why is the SmallFireball  in explosives package?

@ST-DDT: It still creates an explosion, although it's not block damaging. See my explanation in the description, as well as the linked issue.

63da58927e4762e678c


Mobileclient.get_all_stations()

It's not quite clear what this method returns. Is it all radio stations played by the user?

Seems that's what it is. Might be worth adding to the docs.

Huh, yeah, I haven't looked into this since it was added, which was when stations were a first-class thing that the user created and maintained like playlists.

In that case, it might also be worth extending the radio station access in general. I threw together a basic radio track scraper last night, and currently the only way to grab the station of interest is to play it and then take it out of the 'last played' dictionary.
I could do it, but it'll take me a while to figure out, unless you have any thoughts on how to go about it.

Seems that's what it is. Might be worth adding to the docs.

You don't actually have to play them. When subscribed, stations you listen to are added automatically, but there's also an option to add a station to your library without listening even without a subscription. It would probably be worth looking to see if there's an endpoint for that so it could be added to gmusicapi.

In that case, it might also be worth extending the radio station access in general.

gmusicapi is stuck with what functionality Google provides through endpoints. There's a fair amount that the mobile app provides that is done client-side which, in the case of gmusicapi users, means it would be done in user code. So, if it's something that exists in the Google Music API, it could be added. Otherwise, we're out of luck.

I threw together a basic radio track scraper last night, and currently the only way to grab the station of interest is to play it and then take it out of the 'last played' dictionary.

Have you tried using Mobileclient.search to find the station?


Transpile to ES5 not working for imported modules

I am importing js classes as modules to other js files and import them to my angular 5 ts project like this:
if (typeof module !== 'undefined') {
  var CPE = {};
  CPE.a1 = require('./a1');
}

CPE.a2 = class a2 extends a1 {}

if (typeof module !== 'undefined') {
  var CPE = {};
  CPE.a3 = require('../a/a3.js');
  CPE.a4 = require('../a/a4.js');
}

CPE.a5 = class Curve{
  constructor (params = {}) {
    params.className = 'a5';

    super(params);

    this.modifiers = {
      a3: new CPE.a3(this),
      a4: new CPE.a4(this)
    };
  }
 ....

In ts file:
import * as CPE from '../../../../../app/aa/a/a1';

When ng build --prod and transpile to ES5, some of js class are still 'class', not transpiled at all. Also weird thing is each time the files not transpiled properly are not same.
Here is the file main.XXX.bundle.js after build:
if (typeof module !== 'undefined') {
  var CPE = {};
  CPE.a3 = require('../a/a3.js');
  CPE.a4 = require('../a/a4.js');
}

CPE.a5 = class Curve{
  constructor (params = {}) {
    params.className = 'a5';

    super(params);

    this.modifiers = {
      a3: new CPE.a3(this),
      a4: new CPE.a4(this)
    };
  }
 ....

it is literally the original js class, not transpiled at all.
Versions
tsconfig,json:
{
  "compileOnSave": false,
  "compilerOptions": {
    "allowJs": true,
    "outDir": "./dist/out-tsc",
    "sourceMap": true,
    "baseUrl": "./src/",
    "declaration": false,
    "moduleResolution": "node",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "target": "es5",
    "types": [
      "underscore"
    ],
    "typeRoots": [
      "node_modules/@types"
    ],
    "lib": [
      "es2017",
      "dom"
    ],
    "plugins": [
      { "name": "tslint-language-service"}
    ]
  }
}

package.json
  "devDependencies": {
    "@angular/cli": "1.7.4",
    "@angular/compiler-cli": "^5.0.0",
    "@angular/language-service": "^5.0.0",
...

The log given by the failure
Not build errors. But the 'class' keyword fails on IE11
Is there anyone has clue what is happening and how to fix it? Thanks.

The CLI does not transpile Javascript code.  Third-party code is expected to be in the appropriate format.
For project files that are javascript and imported in TypeScript files, the allowJs option may allow the javascript files to be processed by the TypeScript compiler.  Please note that this is untested and not officially supported.


Add ability to build on ios

Thanks a lot for the PR @vinbrule!
I squashed and merged your changes to the master branch in commit 88f6555ef8b87a0bf71869990098508253d118d5.


Dockerfile: install newer gh

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

apt installs an ancient gh that is missing some important flags. In
particular, we probably do want the --match-head-commit flag for the
extra security it gives us.
See Homebrew/homebrew-core#128951 and Homebrew/homebrew-core#128959.
If we don't want Homebrew gh installed all the time in our image, then
we could maybe do brew install gh at the start of every publish job.

I'm opposed to installing anything via brew in our Docker image. AFAIK this image should be a clean Homebrew installation. Maybe we could just download the latest gh binary from GitHub Releases and place it in /usr/local/bin for our usage?

AFAIK this image should be a clean Homebrew installation.

Not convinced that installing from brew is any different from installing from another source when it comes to how clean the installation is, but we can use the gh Debian repository to install a newer gh.

I'm opposed to installing anything via brew in our Docker image.

Agreed.

Maybe we could just download the latest gh binary from GitHub Releases and place it in /usr/local/bin for our usage (same could be done with skopeo)?

This works for me.

Not convinced that installing from brew is any different from installing from another source when it comes to how clean the installation is, but we can use the gh Debian repository to install a newer gh.

brew test-bot at the very least will attempt to remove gh every time if it's installed through Homebrew. Having Homebrew dependencies at CI time ends up being pretty painful (which is why we stopped doing this in the past).

Ok, fair. This already installs a newer gh without using brew.

For future reference, in case the gh installation needs fixing/updating, the steps are lifted practically verbatim from https://github.com/cli/cli/blob/trunk/docs/install_linux.md


Next-translate does not work with turbo pack

What version of this package are you using?
2.6.2
What operating system, Node.js, and npm version?
System:Windows, node:20.15.0, npm:10.7.0 (pnpm:9.4.0), next.js:14.2.4
What happened?
It does not display the content of the translations using the useTranslation hook and the Trans component only displays the i18nkey that was provided.
No error in any of the consoles both next.js and browser.
What did you expect to happen?
To work with turbo pack
Are you willing to submit a pull request to fix this bug?
No

same

same


Whether to support jjwt format

Hello,
Can you provide a bit more detail here?
The only thing that I could find is this Java library:https://github.com/jwtk/jjwt
As such, if the library produces spec compliant JSON web tokens with symmetric keys, this tool should work already.


Fix backup listing

mscs list-backups did not show last backup. Changed parser to use 'date'

This change was merged with commit e0ee185be9d2d1c8f143ea734e326b64d463ff53.  Removing duplicate pull request.


input camera information to render

This is a great job that has saved me a lot of time, thank you! Is it possible to support inputting a given camera parameter file for rendering, as sometimes it needs to be made into a video?

Not yet, I will add support.

But, by the way, in the meantime, you can easily load a trained checkpoint and use the unified python API to render anything.

Hi, it is now implemented (in main) and will be included in the next release.


KeyError in Thellier GUI when reading in (some) MagIC files

I am getting a KeyError when opening MagIC files in Thellier GUI. All data files have been auto-converted to data model 3. Some open without this error. I have attached two example data files (one that is working for me and one that is not).
Traceback (most recent call last):
  File "/Users/lukefairchild765/GitHub_files/PmagPy/programs/pmag_gui.py", line 493, in on_run_thellier_gui
    evt_quit=ThellierGuiExitEvent)
  File "/Users/lukefairchild765/GitHub_files/PmagPy/programs/thellier_gui.py", line 356, in __init__
    self.read_criteria_file(os.path.join(self.WD, self.crit_file))
  File "/Users/lukefairchild765/GitHub_files/PmagPy/programs/thellier_gui.py", line 2739, in read_criteria_file
    crit['criterion_value'])
KeyError: 'measurement_step_max'

Example data file that gets this error:
Tauxe_2006.zip
Data file that opens successfully in Thellier GUI:
Zhu-et-al_2008.zip

Thanks Luke, I will check into this.

On Oct 3, 2017, at 1:30 PM, Luke Fairchild<EMAIL_ADDRESS>wrote:
I am getting a KeyError when opening MagIC files in Thellier GUI. All data files have been auto-converted to data model 3. Some open without this error. I have attached two example data files (one that is working for me and one that is not).
Traceback (most recent call last):
File "/Users/lukefairchild765/GitHub_files/PmagPy/programs/pmag_gui.py", line 493, in on_run_thellier_gui
evt_quit=ThellierGuiExitEvent)
File "/Users/lukefairchild765/GitHub_files/PmagPy/programs/thellier_gui.py", line 356, in init
self.read_criteria_file(os.path.join(self.WD, self.crit_file))
File "/Users/lukefairchild765/GitHub_files/PmagPy/programs/thellier_gui.py", line 2739, in read_criteria_file
crit['criterion_value'])
KeyError: 'measurement_step_max'
Example data file that gets this error:
Tauxe_2006.zip https://github.com/PmagPy/PmagPy/files/1353772/Tauxe_2006.zip
Data file that opens successfully in Thellier GUI:
Zhu-et-al_2008.zip https://github.com/PmagPy/PmagPy/files/1353769/Zhu-et-al_2008.zip
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/249, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGItTP8J2cN4XM_76Ibdf1mPD58jyYks5soplagaJpZM4PsudY.

These were downloaded directly from MagIC (Tauxe, 2006 and Zhu et al., 2008). Thanks for looking into this Lori.

Tauxe 2006 uses 'meas_step_max' in the criteria file.  ('measurement_step_max' in 2.5).  Thellier GUI generates acceptance_criteria by calling pmag.initialize_acceptance_criteria.  'measurement_step_max' is missing from those default criteria, hence the KeyError.
Measurement_step_max isn't in the display for Thellier GUI's acceptance criteria, but should it be?
Options:

Add a default criteria value for measurement_step_max into pmag.initialize_acceptance_criteria.

Add a default criteria value for measurement_step_max into thellier GUI's method add_thellier_gui_criteria.

Change thellier GUI's method read_criteria_file to add an entry into self.acceptance_criteria for any unexpected criteria found.

If we do any of the above three options, we would need to change Thellier GUI's display to show measurement_step_max.  Or we could just ignore any unexpected criterion -- like measurement_step_max -- entirely.
@ltauxe, @ronshaar?

yes measurement_step_max should be in Thellier GUI’s acceptance criteria.  so let’s do options 1 and 2?  if 3 is straightforward, then we can do it.
until that is done, we should just ignore unexpected criteria, perhaps with a warning.
lisa

On Oct 9, 2017, at 3:32 PM, Lori J<EMAIL_ADDRESS>wrote:
Tauxe 2006 uses 'meas_step_max' in the criteria file. ('measurement_step_max' in 2.5). Thellier GUI generates acceptance_criteria by calling pmag.initialize_acceptance_criteria. 'measurement_step_max' is missing from those default criteria, hence the KeyError.
Measurement_step_max isn't in the display for Thellier GUI's acceptance criteria, but should it be?
Options:
Add a default criteria value for measurement_step_max into pmag.initialize_acceptance_criteria.
Add a default criteria value for measurement_step_max into thellier GUI's method add_thellier_gui_criteria.
Change thellier GUI's method read_criteria_file to add an entry into self.acceptance_criteria for any unexpected criteria found.
If we do any of the above three options, we would need to change Thellier GUI's display to show measurement_step_max. Or we could just ignore any unexpected criterion -- like measurement_step_max -- entirely.
@ltauxe https://github.com/ltauxe, @ronshaar https://github.com/ronshaar?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/249#issuecomment-335308026, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDattuf9OFK8D5J24Znm6gom-HDGZks5sqp7ugaJpZM4PsudY.

Ok, I need some help with the defaults:
category: IE-SPEC
criterion_name: measurement_step_max
value:  ?
threshold type:  'high'
decimal: ?

It should be added to the specimen acceptance criteria, right?

@lfairchild, you should be able to at least run Thellier GUI on contributions using unrecognized criteria.   I will need to do a bit more digging to figure out how to deal more elegantly with these kind of criteria.

Great, thanks. The file now opens successfully with the warning -W- Your criteria file contains measurement_step_max, which is not currently supported in Thellier GUI.

Adding measurement_step_max into Thellier GUI is a bit awkward.  The rest of the IE-SPEC acceptance criteria are SPD statistics which are re-calculated every time new bounds are selected (except for measurements.experiment, which is just used to select only measurements from a thellier type experiment).  I'm not sure how measurement_step_max should be handled here.  Should it be added to the SPD module?  Or used behind the scenes in Thellier GUI to select available measurements?  It obviously isn't recalculated when you change the bounds....  Whatever the solution, I don't think it can be generalized as a way to handle just any unrecognized criterion.
Also, the description of meas_step_max could be clearer ("Step included in calculation, Higher bound").  @rminnet, should this description specify that we are talking about a max temperature?  Or can it also be microwave power?  Either way, a little additional clarity would be helpful.

Yes, I believe specimens.meas_step_min/max are intended to describe lower and upper limits on the parameter that was swept for the experiment and they don't have to be describing just temperature or microwave power. Perhaps @njarboe could help clarify the descriptions next week?

Here is what I have changed the descriptions to:
meas_step_min: The lower bound for the parameter varied during the experiment
meas_step_max: The upper bound for the parameter varied during the experiment
If there are two experiments on a specimen that both use step_min and step_max, then we will just have separate lines for each? How would one know which experiment it was if the specimens.experiments value includes all experiments?

After conversation with @ltauxe, we will ignore this!


Possible Memory Leak

Issue Type: Bug
Left vscode running overnight. Is now consuming up to 25 GB of memory.

VS Code version: Code - Insiders 1.30.0-insider (df8ea54c5397e67b3e47df15a355bd04345c6934, 2018-11-15T06:21:02.187Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2133 CPU @ 3.60GHz (12 x 3600)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.66GB (0.83GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

vscode-nmake-tools
Mic
1.1.180913009

cpptools
ms-
0.21.0-insiders

@lilustga could you please follow https://github.com/Microsoft/vscode/wiki/Performance-Issues to help troubleshoot? You can also run code-insiders --status to take a look which process is using so much memory.

I'm not experiencing the issue at the moment. Thanks for the troubleshooting resource, I'll get more information if it happens again. Feel free to close the issue.


Fix issue with Reveal#destroy().

Noticed that Reveal's destroy() runs into an error because the $element is appended to the $overlay and the $overlay is then removed before unregisterPlugin() gets called. Therefore plugin.$element.data('zfPlugin') failed because $element as already removed with $overlay.
Moved $element outside of $overlay to prevent error unregisterPlugin().

Looks good


Vulkan: Adding a material to a mesh renders it black (clustered) or invisible (mobile)

Godot version
4.0 alpha 1
System information
MacBook Pro (13-inch, M1, 2020), MacOS: 11.6.2
Issue description
Adding a material to a mesh renders the mesh black or invisible.

Black in the clustered renderer
Invisible in the mobile renderer

Changing material parameters, I tried albedo, metallic and roughness do not have any effect.
It also did not help to add a WorldEnvironment with a procedural sky or custom clear color.
In the clustered renderer, the color matches the clear color if no sky is used.
Clustered renderer:

Mobile renderer (effect is the same w/o shadows):

Steps to reproduce
Create a MeshInstance and add a material override.
Minimal reproduction project
MatTest4.zip

seems to be a duplicate of #51898

Duplicate of https://github.com/godotengine/godot/issues/51898 (same cause).