Dev mode does not recompile existing classes when new classes are added

Steps to create the problem:

git clone https://github.com/OpenLiberty/guide-microprofile-rest-client.git -b dev-mode
cd guide-microprofile-rest-client/start
mvn liberty:dev
wait until [INFO] Press the Enter key to run tests on demand.
cp ../finish/src/main/java/io/openliberty/guides/inventory/client/SystemClient.java src/main/java/io/openliberty/guides/inventory/client/SystemClient.java

see following errors are expected

/Users/gkwan/tasks/CNAI/guides/testsite/guide-microprofile-rest-client/start/src/main/java/io/openliberty/guides/inventory/client/SystemClient.java:21: error: cannot find symbol
  public Properties getProperties() throws UnknownUrlException, ProcessingException;
                                           ^
  symbol:   class UnknownUrlException
  location: interface SystemClient
/Users/gkwan/tasks/CNAI/guides/testsite/guide-microprofile-rest-client/start/src/main/java/io/openliberty/guides/inventory/client/SystemClient.java:15: error: cannot find symbol
@RegisterProvider(UnknownUrlExceptionMapper.class)
                  ^
  symbol: class UnknownUrlExceptionMapper
2 errors
[INFO] Source compilation had errors.
[INFO] Source compilation was successful.

cp ../finish/src/main/java/io/openliberty/guides/inventory/client/UnknownUrlException.java src/main/java/io/openliberty/guides/inventory/client/UnknownUrlException.java
cp ../finish/src/main/java/io/openliberty/guides/inventory/client/UnknownUrlExceptionMapper.java src/main/java/io/openliberty/guides/inventory/client/UnknownUrlExceptionMapper.java
cp ../finish/src/main/webapp/META-INF/microprofile-config.properties src/main/webapp/META-INF/microprofile-config.properties
cp ../finish/src/main/java/io/openliberty/guides/inventory/InventoryManager.java src/main/java/io/openliberty/guides/inventory/InventoryManager.java
Got the following but SystemClient was created before

[INFO] [AUDIT   ] CWWKZ0003I: The application guide-microprofile-rest-client updated in 0.294 seconds.
/Users/gkwan/tasks/CNAI/guides/testsite/guide-microprofile-rest-client/start/src/main/java/io/openliberty/guides/inventory/InventoryManager.java:20: error: cannot find symbol
import io.openliberty.guides.inventory.client.SystemClient;
                                             ^
  symbol:   class SystemClient
  location: package io.openliberty.guides.inventory.client
/Users/gkwan/tasks/CNAI/guides/testsite/guide-microprofile-rest-client/start/src/main/java/io/openliberty/guides/inventory/InventoryManager.java:32: error: cannot find symbol
  private SystemClient defaultRestClient;
          ^
  symbol:   class SystemClient
  location: class InventoryManager
2 errors
[INFO] Source compilation had errors.

It was because SystemClient initially missing UnknownUrlException and UnknownUrlExceptionMapper and failed be compiled.
Should all classes be recompiled?
I have to visit SystemClient to force the dev mode to recompile the class, then the application can run.

The issue is dev mode only compiles classes that are added/changed.  We should have a way to recompile existing classes when other classes are added/changed.

@ericglau @cherylking Can raise this issue to high priority now? Some guides requires this issue be fixed for dev mode conversion?

One possible approach is when a source file is detected, iterate through all Java source files to see if there is a corresponding .class file in the output dir.  If not, recompile the Java source files for the missing .class files.  The same should apply for test sources and test classes.

Another scenario: import classes but not available, then add dependencies in the pom.xml.


Add helm chart

Hey everyone,
I've created a first draft of a helm chart for the parquet-exporter. I do not have a setup where I can use AWS as of now, therefore the actual push to a S3 bucket is not yet tested. However, I was able to install the chart and run the cronjob in my minikube.

Used helm-docs to  create a rough documentation around the chart
The chart itself is relatively straight forward I think

Let me know what you think!

Do we want to move this to https://github.com/opencost/opencost-helm-chart/ and maybe put it under charts/opencost-parquet-exporter? That would keep OpenCost Helm charts in 1 location. @lmello or @sntxrr want to weigh in?

Do we want to move this to https://github.com/opencost/opencost-helm-chart/ and maybe put it under charts/opencost-parquet-exporter? That would keep OpenCost Helm charts in 1 location. @lmello or @sntxrr want to weigh in?

Sure thing. I did put it here because it was listed as a todo here. Therefore I assumed that it was meant to be here. But I can close here and move the code to https://github.com/opencost/opencost-helm-chart/

@cklingspor if you want to close this one and re-open it over in opencost-helm-charts that would be appreciated.

Continued here: https://github.com/opencost/opencost-helm-chart/pull/195


cidata: automatically upgrade existing containerd/nerdctl ; nerdctl: update to v0.17.1

When the timestamp of /mnt/lima-cidata/nerdctl-full.tgz:bin/nerdctl is newer than
/usr/local/bin/nerdctl, containerd/nerdctl is automatically upgraded.
Drawback: increases boot time by 4-5 seconds.
Fix #303

Also bumps up the default nerdctl version to 0.17.1: https://github.com/containerd/nerdctl/releases/tag/v0.17.1

Let me merge and open an issue ticket for optimization in future


Add trusted builder key to Endorsement Release Authorization Logic Policy

From what @rbehjati tells me, the outputs of trusted builders like Github Actions and Google Cloud Build will be signed using a unique keypair for these trusted builders. This signing does not yet exist (even independently of authorization logic), but we should add the relevant keys into the authorization logic policy once the signing is set up.
I think I heard that @mariaschett is working on adding key management for this (?).
Do either of you have more relevant details for this?

We have this issue on Oak for tracking the signing of the Oak provenances: https://github.com/project-oak/oak/issues/2626, but we are still working out the details.


Cannot address views in views

Hi, how can we address views in views? Please see my example below:
$stateProvider
         .state( "probandSearch", {
            url: "/proband-search/:keyword",
            views: {
               "": {
                  templateUrl: "app/modules/partials/proband/search/search.html"
               },
               "table@probandSearch": {
                  controller: "TableController",
                  controllerAs: "vm",
                  templateUrl: "app/modules/partials/proband/search/table/table.html"
               },
               "details@probandSearch": {
                  controller: "DetailsController",
                  controllerAs: "vm",
                  templateUrl: "app/modules/partials/proband/search/details/details.html",
                  views: {
                     "contact": { // how to address this view?
                         // tried: contact@probandSearch
                        // tried: contact@details@probandSearch
                        templateUrl: "app/modules/partials/proband/search/details/contact/contact.html"
                     }
                  }
               }
            }
         } )

      ;

HTML: works just fine.

In details.html, I want to use contact:

However, the view "contact" is not displayed.

Add it as a child state. Or try $view.load. If you need additional help, try asking this on stackoverflow.


Unknown value type: "CONTEXT"

Hi tinylog-creators,
with the jdbc configuation below I get the following output:
LOGGER WARNING: Unknown value type: "CONTEXT"
LOGGER ERROR: Failed to activate configuration (java.sql.SQLException: Number of columns and values must be equal, but columns = 8 and values = 7)
JDBC config:
tinylog.writer = jdbc
tinylog.writer.url = jdbc:mysql://localhost:3306/<db_name>
tinylog.writer.table = log_entries_table
tinylog.writer.columns = context, level, message, class, line, date, method, exception
tinylog.writer.values = CONTEXT, LEVEL, MESSAGE, CLASS, LINE, DATE, METHOD, EXCEPTION
For the context column I took a text datatype in the database (MySQL).
Everything other than CONTEXT works fine. How can I make this work?
Best regards

Fixed by commit 7bbe3ab99ce917b804836cc7f0c7d8ef5a7bd877 and released as tinylog 1.1-rc2. Can you confirm that the fix works in your environment, too?

Very nice! Thank you @pmwmedia! Yes it works for me as well :)


Typings For AngularJS Extensions For RxJS

Initial Commit Of Minimum Support For The AngularJS Extensions For RxJS

@mickdelaney could this also be merged directly into the rx.angular.js project as well?

Sure, whatever you guys think, I'm working with rx & typescript on angular most days now so whatever makes that easier 😉 do you want me to send another pull request with the code moved over ?

angular-rx/rx.angular.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/angular-rx
http://bower.io/search/?q=angular-rx
others?

[X] has a test file? (angular-rx/rx.angular-tests.ts or others)
[ ] pass the Travic-CI test?

I can't found angular-rx project... where is it?

https://github.com/Reactive-Extensions/rx.angular.js

i called it angular-rx to follow the naming conventions for the rest of the angular-* projects on definitelytyped.  i guess we could stick to they're convention though, which would be rx-angular

yep, please use rx-angular/rx-angular.d.ts

i've move the angular rx typings to rx-angular

@mickdelaney sorry for delay. It merged. thanks!


Is Django supported?

I think it could be used with Django, but you might be missing whatever integrations Django currently has with Jinja2. The project would welcome a contribution to implement that!

Just worked on a Django project recently and was looking for a similar solution. Found this:
Django Render Block
https://pypi.org/project/django-render-block/
It works fairly similar. For example, if you have a view that is accepting an htmx request, it looks something like this:
def random_view(request):
    some_objects = SomeObject.objects.all()
    if request.headers.get("HX-Request"):
        html = render_block_to_string(
            "slideshow/index.html",                    # name of the template
            "block_name",                                   # name of block name to be rendered
            {"some_objects":  some_objects},     # additional context (accessible through your block
            request=request
        )
        return HttpResponse(html)

Only the "block_name" is sent (and rendered) in the case of an htmx request.
Hope that helps. (Sorry so late to responding)

django-render-block is a good alternative to use with Django as it has partial support for jinja. It seems you only need to be mindful of this limitation:

Partially supports the Jinja2 engine: it does not currently process the extends tag.

Closing as the original question has been responded.


Use gopkg.in/blackfriday.v2

This enables compatibility with dep until dep supports go mod's version suffixes. Compatibility with dep is required to vendor this repo in https://github.com/knative/docs.
See https://github.com/matcornic/hermes/issues/46

Doesn't work with go mod apparently.


Ops 3.2

updated OPS endpoint version to 3.2
updated documentation
removed RegisteredClient, since OPS v3.2 requires authenticated usage (there is, therefore, only one kind of client now: just Client [which requires credentials])

@eltermann 🙏


Add Meson related info to README

Add more information about how one can get a Meson wrap file.

Coverage remained the same at 100.0% when pulling f0c55ce0e01b2d1ecc5f270e53eed5bacdaf9b80 on koponomarenko:add-meson-info into 4b2a00641c43da1feac5169b56bc3216c9c453ee on nlohmann:develop.

Thanks!


'log' in the README image

I see logarithm (log) method in the README image. But it's neither in the documentation nor it is a method.

It's a mistake. I must have left it in when I copied the table from decimal.js.
I'll fix the image presently. Thanks for the notification.


Proximal map of inpainting operator

For the simple forward model (binary selection) used for image inpainting,  Its way to avoid computing the proximal map using gradient descent and instead just use the closed form solution ?

Hey @svvenkatakrishnan, in the case of Inpainting, the class inherits from the DecomposablePhysics class, which has its own closed form implementation in the prox_l2 method, so gradient descent is not used!
More generally, we've set the default computation of the prox as computed with an iterative method so that even if no implementation of the prox is provided by the user, a solution is returned. However, most Physics in the library overwrite this gradient descent implementation. As we've seen above, this is the case for all children of the DecomposablePhysics class, but not only - for example in some cases for Downsampling.

Thank you for the detailed response @matthieutrs . Since it is using a closed form I was not sure what the "step size" parameter was doing. For example:
Here is a code snippet I am trying for an inpainting problem with PnP-ADMM.
physics = dinv.physics.Inpainting(
mask=mask_tensor,
tensor_size=y.shape,
noise_model=dinv.physics.GaussianNoise(sigma=0),
device=device
)
denoiser=DRUNet(in_channels=1, out_channels=1, nc=[64, 128, 256, 512], nb=4, act_mode='R', downsample_mode='strideconv', upsample_mode='convtranspose', pretrained='download', train=False, device=device)
data_fidelity = L2()
prior = PnP(denoiser=denoiser)
params_algo = {"stepsize": _ss, "g_param": _gp, "lambda": 1.0}
model = optim_builder(iteration="ADMM",
prior=prior,data_fidelity=data_fidelity,
early_stop=early_stop,
max_iter=max_iter,
verbose=verbose,
params_algo=params_algo
)
x_pnp_cnn, metrics = model(y, physics, x_gt=x, compute_metrics=True)

I have noticed that when I change the stepsize parameter the solution changes. Does the step size correspond to another variable in these cases ?

Yes, this is indeed expected!
As you can see in ADMM the stepsize is used here. More generally, just like in any other algorithm, a stepsize needs to be set. However, unlike what happens in optimization where the solution is independent of the stepsize, the solution of PnP algorithm depends on the external parameter chosen.
BTW, this is what makes DPIR quite interesting and powerful, stepsizes have been tuned with a specific decay for good performance :)

@matthieutrs : Thank you.  However I think there is some terminology which I may be missing or is overloaded in the library.
To solve the proximal map for the forward problem for the inpainting: ||y-Ax||{2}^{2} + \gamma ||x - \tilde{x}||{2}^{2}; it looks like the step size here is referring to \gamma.
Instead, if this same prox is solved using say gradient descent, then stepsize could refer to something different in that context. This way one would need to set two parameters to solve the problem.
Does that sound right ?

Got it. It may not be urgent, but in a future release it would be good to expose the "inner iteration" parameters such as the step size, max inner iterations etc. as advanced settings that can be accessed using the params_algo.


feat: custom status line hints

closes #20
Features

Allows users to pass a hint to a hydra when config.hint.type = "statusline". The hint is parsed and highlighted using the same rules as window hints
Adds "statuslinemanual" hint type to enable passing a hint string while not automatically showing a hint in the status line (the same way setting config.hint = false does for a normal auto generated status line. Issue is when you pass a hint, the type is assumed to be window, so we need this new type if we don't want to change behavior).

Done in two steps if you would like to review the PR that way:

chore: window hint parsing
feat: custom status line hints

This PR also fixes a few README problems that I found, and adds an .editorconfig file b/c the majority of the codebase uses 3-space tabs so I'm setting that as the default.
This PR does not currently append unused heads to the end of a manual status line hint. I can add that I guess if we care to be consistent.

Example of what you can now do:
Hydra({
  name = "QuartoNavigator",
  hint = "_j_/_k_: ↑/↓ | _o_/_O_: new cell ↓/↑ | _l_: run | _s_how | _h_ide | run _a_bove | _q_uit",
  config = {
    color = "pink",
    invoke_on_body = true,
    hint = {
      type = "statuslinemanual",
    },
  },
  mode = { "n" },
  body = "<localleader>j",
  heads = {
    { "j", keys("]b"), { desc = "↓" } },
    { "k", keys("[b"), { desc = "↑" } },
    { "o", keys("/```<CR>:nohl<CR>o<CR>`<c-j>"), { desc = "new cell ↓", exit = true } },
    { "O", keys("?```.<CR>:nohl<CR><leader>kO<CR>`<c-j>"), { desc = "new cell ↑", exit = true } },
    { "l", ":QuartoSend<CR>", { desc = "run" } },
    { "s", ":noautocmd MoltenEnterOutput<CR>", { desc = "show" } },
    { "h", ":MoltenHideOutput<CR>", { desc = "hide" } },
    { "a", ":QuartoSendAbove<CR>", { desc = "run above" } },
    { "<esc>", nil, { exit = true, desc = false } },
    { "q", nil, { exit = true, desc = false } },
  },
})

-- in addition to some status line code which calls `require("hydra.statusline").get_hint()`

TODO:

[x] Function Values in the status line

I will try to get to this this evening. My initial gut reaction to this is "do we really need a way to set things in the statusline?" though lol.
A quick read through the code and it looks fine. If I am understanding what it is doing properly, this is generating a statusline compatible string for you but not inserting it into the statusline. Is that correct?
As an additional question, is there currently a way to retrieve this statusline hint without the statusline specific string wrapping? I am thinking about lualine users which would just want the literal string to embed in their lualine since lualine performs the proper statusline "wrapping" for them. If my understanding is correct, this would not be usable with something like lualine.
I will find out tonight, but you may have an answer before then :)

do we really need a way to set things in the statusline

There's already a way, but there was only a way to put auto generated hints in the status line. this just adds support for custom hint strings.

If I am understanding what it is doing properly, this is generating a statusline compatible string for you but not inserting it into the statusline. Is that correct?

It can do both. If you pass a hint with config.hint.type = "statusline" then the Hydra plugin will replace the status line automatically (and restore it when the hydra exits). If you pass a hint with config.hint.type = "statuslinemanual" then the status line is not automatically replaced and you can use require("hydra.statusline").get_hint() to get the status line string to put into your own status line "manually".

a way to retrieve this statusline hint without the statusline specific string wrapping ... lualine users which would just want the literal string

there's the get_hint() method but that does return the string wrapped with highlights and stuff. Lualine should be able to display the wrapped string properly, I can test it quickly. I use MunifTanjim/nougat.nvim and that doesn't have trouble with it.
There is no way to get it without the wrapping.

Yup, lualine will display the wrapped string properly:

Perfect! I'll give this the ole once over after work but my quick glance through, it all looked good :)

I don't use the statusline mode as is so I cannot speak to how well this works in comparison to that. I do wonder if it makes sense to replace the "non-manual" version with something that just talks to this instead (to avoid duplicate/similar logic).
Thoughts on that? I could be completely off base as well.
Outside that, this looks good. Once I hear your thoughts on my question I have no issue merging this :)

I do wonder if it makes sense to replace the "non-manual" version with something that just talks to this instead

like some code that generates a default hint string (something like: _h_: description, _j_: description) and passes it to the same class that would parse a "user entered" hint string?
That could be nice, but this would kinda flip some of the current inheritance on its head I think. I can look into it maybe later today or tomorrow, but I'd be okay with merging it in its current state as well.

Ya I am just thinking, we currently have 2 ways to generate statusline text. Acknowledging my little knowledge there I figure it makes sense that we have 1 way to generate statusline text and the wrap that with something to auto inject it into the statusbar or whatever the "auto" statusline config option does.
For now this is fine, but it probably makes sense to clean that cruft up a bit. In either case, I am fine merging this :)


sasjs compile optimisations

As projects get bigger, compilation takes longer and longer.  It would be great to find ways to optimise this process, eg:

Only compile macros once - ie, if two jobs share "mac1.sas", and "mac1.sas" uses "mac2.sas" and "mac3.sas", we should compile "mac1.sas" ONCE and re-use that compiled macro across the two jobs
Minimise disk operations (if not already minimised)

possibly linked: https://github.com/sasjs/cli/issues/1095

The current approach cyles through each Job / Service / Test, one by one, and spends a lot of time on disk operations.
This results in performance issues on windows.  We would save a lot of time by loading the macro content (and list of dependencies) into an in-memory object - thereby only 'touching' each macro once during the sasjs compile process.  The in-memory object would have four attributes:

Macro name
Macro content
List of dependencies
Directory from which it was compiled

During compilation of a job, service or test, we currently take the following steps:

extract the initial list of macros
extract a distinct list of recursive sub-dependencies from the macro headers on disk
insert content from disk into the compiled job in sasjsbuild

The proposal is to modify this as follows:

extract the list of macros in the job
extract a distinct list of recursive sub-dependencies from the object (if they exist, else headers on disk)
insert content from the object (if it exists, else disk) into the compiled job in sasjsbuild
for every macro that is read from disk, the content / location / list of dependencies are also stored in the object

Note - the above is only in relation to <h4> SAS Macros </h4>.  The Includes and BinaryFiles are not recursively compiled, and so are less problematic.


Zeromq and its headers required to build quilc

Building quilc requires rpcq which in turn requires the ZeroMQ library and development headers. I'd recommend adding this dependency into the README's installation instructions for quilc

Good spot!


OMIS notification adviser added

This sends a notification to OMIS advisers when they are added to an order.

Codecov Report

Merging #617 into develop will increase coverage by <.01%.
The diff coverage is 92.3%.

@@            Coverage Diff             @@
##           develop    #617      +/-   ##
==========================================
+ Coverage    96.49%   96.5%   +<.01%
==========================================
  Files          126     126
  Lines         3828    3837       +9
  Branches       329     330       +1
==========================================
+ Hits          3694    3703       +9
  Misses          95      95
  Partials        39      39

Impacted Files
Coverage Δ

datahub/omis/notification/signal_receivers.py
100% <100%> (ø)
:arrow_up:

datahub/omis/notification/client.py
100% <100%> (ø)
:arrow_up:

datahub/omis/notification/constants.py
100% <100%> (ø)
:arrow_up:

datahub/company/models/adviser.py
85.48% <66.66%> (+0.48%)
: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 8cc2736...766f22a. Read the comment docs.


Bug:  clicking on row in Dashboard does not load application

To recreate:

Log into the prod analyst portal
Click on any row in Dashboard
Error - click on row does not take the user to the application page.
After logging out and logging back in a couple of times this bug seems to resolve it self.

Expected behaviour:
When clicking on the row, it should take the user to the application page for that row.

@MarsRomer I'm not sure what to do with this one as it is working fine for me. It may have just been a one time networking issue as there has been no changes to the dashboard for some time and we haven't seen it before. Can you please let us know if it happens again.

@marcellmueller is their anything in our monitoring logs that would shed some light on this.  Since it happened to someone else as well around the same time I am hoping there is some information that would tell us if there was a networking issue or something else going on at the time


Update speech-sdk.md

@yulin-li : Thanks for your contribution! The author(s) have been notified to review your proposed change.

TTS has already support JS and Go.


Make necessary traits pub and add basic error handling

This PR makes the traits that are intended for external usage public, which was forgotten before.
Also, some functions are refactored to be able to return errors as part of a Result<..,..> type.

I think trying to return the trait Error from these functions in the result was what got me started down the Box<..> route, so using anyhow::error removed the need for Boxes all over the place.
I agree that it would be nicer to define a proper error type and have those error be part of the api though.


fix: Add restitution to Ball

Description
This adds some restitution to the ball (this means that it will bounce off things) which makes the physics simulation look a bit more realistic. Currently when it hits the pads for example the ball just stops, which looks a bit unnatural.
Preferably I would like to have changed the friction of the ball not to be 0 too, but since the rendering currently depends on the direction of the ball and the 0 friction is used to ensure that the ball doesn't rotate that is a much bigger change.
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[x] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Thanks @spydon for the PR!
Physics changes require to be play tested to see if the game is still as fun and to check the ball can't get stuck or looped on a game spot.

Changes LGTM, but as @alestiago we need to do a run of play test to check how this plays.
But also something to consider is that this will change how the game plays, which means that, in theory, to be fair with the new scores, that we should clear the scoreboard, as the game will plays different from when those scores were generated.

Thanks for your contribution. We've actually decided to archive this repo since we don't have anyone working on it full time these days, since it was mostly a demo for an event, but please don't hesitate to fork the repo (it's open source!) if you want to keep it alive!


is there any way to paste picture inside vsc from clipboard then insert it by Anki ?

This interests me also

Could you go into more detail on what you want to happen here? Which card does the picture go to? Is this pasting into a markdown file or straight into a new VSCode file? I’ve never pasted an image directly into code so don’t understand this workflow

Search for packages in vscode related to pasting images. https://github.com/imlinhanchao/vsc-markdown-image works well. It will embed the image into your markdown file and store the image in a specified directory.
Assuming vscodeanki handles the link properly, this should be all you need. 👍

@Stedag would you be up for working on this?

This feature is available through the marketplace.
@abedahmeduhp and @ThrashAbaddon  I tested images pasted with Markdown Image are incorporated properly. It works great so far:

-becomes->

@jasonwilliams you can probably close the issue unless you're hoping to incorporate these features into VScodeAnki.
I am very interested in working on #18 because I use LaTeX. Unfortunately I am new to regex, javascript, and extension development so I'm going to try a workaround in python first. If someone is willing to provide some direction as to where I would place a search & replace function within the extension that would possibly help me along... I'll continue this discussion at that issue.

This sounds good to me, I’m closing
@Stedag please ask me questions in that issue I can help, I’ll also write a contributing guide and ping you that too


Product Searching

Implement functionality so that users can search by name, category, or price range
SP:2
Acceptance Criteria: Users should be able to search using various criteria and receive accurate results

done


extend api with mask option

I added an option to the api to set the bit masks directly, this change wouldn't / shouldn't move the api boundary as it's always possible to convert the masks into array on the library side if the collision group system gets changed.

Thanks!


Make baseFeePerGas optional on pre-london blocks

Per this discussion on discord: https://discord.com/channels/595666850260713488/804019759934078987/852562227780255754

Apparently I can't assign myself to this issue since i'm not a member of the repo, but I'll handle this.

Should it be optional on pre-London blocks or should we make it clear it is simply not present, @lightclient ? The latter will allow users to know if the chain supports 1559 at a certain block, vs. there can be cases where a chain may have a base fee of 0.

Ah sorry - poor wording there. I agree with you, it shouldn't exist pre-london. Thanks for the clarification.


Support K2 mode

As JetBrains is preparing to enable the new K2 Kotlin compiler mode in Intellij Idea by the end of 2024 (currently in Beta):
https://blog.jetbrains.com/idea/2024/03/k2-kotlin-mode-alpha-in-intellij-idea/
https://blog.jetbrains.com/idea/2024/05/intellij-idea-2024-2-eap-2/
It would be great to see this plugin support K2 mode as well, since currently the plugin is not loaded if K2 mode is enabled in intellij idea. Once the new mode becomes the default, this plugin will be impossible to use, unfortunately.

Hi @dkuznetsov-aligntech! Thanks for opening an issue about it. As the blog post that you shared says:

Third-party IntelliJ IDEA plugins depending on the Kotlin plugin will be disabled. We will provide migration guidance for third-party plugin authors soon!

The migration guide is not out yet, but once it is I will update the plugin!

The K2 is now supported in Coroutine Stacks 1.0.3!


obs-studio: add pipewire to buildInputs

obs-studio (git master) has support for capturing wayland windows via
pipewire.
To build further releases of obs-studio this adds pipewire to buildInputs
https://github.com/obsproject/obs-studio/pull/4287

Motivation for this change
Capture wayland screens. Use obs-studio git-master for latest features
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118354 at 8b4dff7a run on x86_64-linux 1

1 package failed to build:

obs-ndi

5 packages built successfully:

obs-gstreamer
obs-move-transition
obs-studio
obs-v4l2sink
obs-wlrobs

3 suggestions:

warning: build-tools-in-build-inputs
swig is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/applications/video/obs-studio/default.nix:57:3:
   |
57 |   buildInputs = [
   |   ^

warning: build-tools-in-build-inputs
makeWrapper is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/applications/video/obs-studio/default.nix:57:3:
   |
57 |   buildInputs = [
   |   ^

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/applications/video/obs-studio/default.nix:126:5:
    |
126 |     license = licenses.gpl2;
    |     ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

Support should be optional like it is with pulseaudio and alsa.

@ofborg eval


Route framback pop through the navigator

Need to update the navigator's current request back to outer page so, for
example, RequestContext.getCurrentPath() works correctly.
This is a more natural flow through the back-nav.

lgtm :+1:


fix: deprecated response method

Response now being sent using res.status(status).send(body) instead of  deprecated res.send(status, body)

This should have been fixed in version v0.5.2, can you confirm you're using the latest version?

Yes, indeed. My bad.


FCC-70 Remember Me Toggle

Linked and working. Mike's endpoint would throw an error if the field wasn't received
Token expiry is also changing:
Remember:

Forget:

Rating: 5/5


Make taking screenshots easier and more useful

Current PrntScr takes a screenshot with only a date in the name and also works as a regular PrntScr taking a picture of the entire screen, which is quite garbage when you're playing windowed mode and only want to post a screenshot of the game to Discord or something.
Change the naming to something more useful like chart name, rate, score, screen or date and maybe add a hotkey to copy the screenshot to clipboard, not unlike what ShareX allows you to do.

It seems, at least on windows, copying images to the clipboard is much more complicated than text. Maybe linking https://github.com/dacap/clip/ and using it directly in the base HOOKS (architecture hooks) singleton class for clipboard text/image functions might be worth it (That way we'd also get clipboard copying and clipboard copy/pasting would be cross platform instead of windows only)
This is the function that names screenshot files: https://github.com/etternagame/etterna/blob/13267c3a4bfcf1c0713b77496ff1a4361e1ce4ed/src/StepMania.cpp#L1300 (More specifically, the line that does so)
It seems the evaluation screen calls that function: https://github.com/etternagame/etterna/blob/13267c3a4bfcf1c0713b77496ff1a4361e1ce4ed/src/ScreenEvaluation.cpp#L781
In here the current song/steps(aka chart) should be accessible. We could add a last optional parameter to the SaveScreenshot function for a forced filename, generate a name and pass it. What format for files would be appropiate? Should we support multiple formattings for screenshots based on a preference?

We could add a simple input screen in the options menu somewhere to allow changing the format, given a particular format string input. Like replacing {date} in the string input with the actual date or whatever.

this seems redundant and burdensome


pwa plugin did not inject meta tag with manifest

IMPORTANT: Please use the following link to create a new issue:
https://new-issue.vuejs.org/?repo=vuejs/vue-cli
If your issue was not created using the app above, it will be closed immediately.

I used the tool - and it generated content, then immediately disappeared before I pressed "submit".


nocited sourcemap does not apply in 2.17 dev

Consider
\documentclass{article}
\usepackage{biblatex}

\DeclareSourcemap{
  \maps[datatype=bibtex, overwrite]{
    \map{
      \step[nocited=true, final]
      \step[fieldset=note, fieldvalue={I was NOCITED}]
    }
  }
}

\addbibresource{biblatex-examples.bib}

\begin{document}
Lorem \nocite{sigfridsson,worman}

\printbibliography
\end{document}

With 2.16 I get

but with
2.17 I get

It appears that the nocited marker no longer applies properly.

Was a brace placement issue causing writing of nocites twice (one as normal cites) in the .bcf


callback in pump firing before _flush complete

This is probably a request for clarification rather than a bug report. I have a transform stream with an async _flush method. However, pump's callback fires before I call the callback in flush. As a result, my app terminates early. If instead of using pump, I just resolve a promise on the 'end' event, my app functions normally. Is there something about using async await inside _flush in a transform stream that breaks pump, or is bad practice?

Pump calls the callback when the final stream in the pipeline emits finish which is an event indicating that the writable side of that stream is fully flushed.
The flush method is called just before the readable side of the stream ends.
Use _final instead if you want to do something before the writable side is done.

Thanks for clarifying!


charts: cleanup labels

Move sunburst/treemap to radio boxes and add currency select for the
treemap.
Close #274
Ref #304

Current coverage is 56.16% (diff: 100%)

Merging #355 into master will not change coverage

@@             master       #355   diff @@
==========================================
  Files            12         12
  Lines          1013       1013
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            569        569
  Misses          444        444
  Partials          0          0

Powered by Codecov. Last update 1b89590...fe4bb9b


Unable to use Opcode. GameCrash

Hi,
I installed XicAlexander today to challange Ping issues and got into a bad predicament. Here are my steps:
Put XivAlexander on my D-Drive (Note: FFXIV.exe is on C:)
Installed on running game (load once). After startup I saw the warning "Cannot save config file".
I then tried to look into the opcode and it was empty even after trying to fetch a new one.
Then I tried to manually save the opcode file with this in it:
https://github.com/Soreepeong/XivAlexander/commit/d13dd090ded05fcb0a64891b9bbc3a1aec2285b3
After hitting save I got an execption [XivAlexander_20220413_121140_624] and Game+Xivalexander crashed. A restart gave me another exception [XivAlexander_20220413_121210_498].
Edit: I already tried to give Xivalexander.exe all permissions to read/write manually.
XivAlexander_20220413_121140_624.log
XivAlexander_20220413_121210_498.log

Are you using Protected Folders (Windows feature), or any sort of antivirus?

Hi. I am using no protected folders and gave Alexander full rights to all folders on C:.
I use Bitdefender Antivirus. The history there did not show anything.


Gatekeeper with NGINX auth_request

Title
Gatekeeper with NGINX auth_request
Summary
Hello
I've tried to use Gatekeeper to authenticate/authorize users using NGINX with auth_request module, but unable to get it working.
The flow I am trying to achieve is this (ignoring cookies):
user -> nginx -> backend -> 401 -> nginx auth_request -> gatekeeper -> keycloak -> user signs in -> gatekeeper -> nginx -> backend -> reply to user
When i tried that setup, gatekeeper returns 303 See other to nginx auth_request which is treated as an error, as of specification of auth_request
"If the subrequest returns a 2xx response code, the access is allowed. If it returns 401 or 403, the access is denied with the corresponding error code. Any other response code returned by the subrequest is considered an error."
Gatekeeper config
  client-id: ...
  client-secret: ...
  discovery-url: http://identity-keycloak:8080/realms/master
  secure-cookie: false
  skip-openid-provider-tls-verify: true
  no-proxy: true
  enable-default-deny: true
  encryption-key: ...
  listen: :3000
  listen-admin: :4000
  enable-metrics: true
  enable-logging: true
  enable-request-id: true
  enable-login-handler: true
  enable-refresh-tokens: true

Nginx config
location / {
        root /data/www;
        proxy_pass http://resource-server:7072;
        proxy_set_header Host $host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Scheme $scheme;
        proxy_set_header X-Forwarded-Host $host:80;
        proxy_set_header X-Forwarded-Port 80;
        proxy_set_header X-Forwarded-Server $host;
        proxy_set_header X-Forwarded-Proto $scheme;
        proxy_connect_timeout 1;
        proxy_read_timeout 30;
        proxy_send_timeout 30;
        proxy_http_version 1.1;
        auth_request /oauth/authorize;
        auth_request_set $user   $upstream_http_x_auth_request_user;
        auth_request_set $email  $upstream_http_x_auth_request_email;
        proxy_set_header X-User  $user;
        proxy_set_header X-Email $email;
        auth_request_set $token  $upstream_http_x_auth_request_access_token;
        proxy_set_header X-Access-Token $token;
        auth_request_set $auth_cookie $upstream_http_set_cookie;
        add_header Set-Cookie $auth_cookie;
        auth_request_set $auth_cookie_name_upstream_1 $upstream_cookie_auth_cookie_name_1;
        if ($auth_cookie ~* "(; .*)") {
            set $auth_cookie_name_0 $auth_cookie;
            set $auth_cookie_name_1 "auth_cookie_name_1=$auth_cookie_name_upstream_1$1";
        }
        if ($auth_cookie_name_upstream_1) {
            add_header Set-Cookie $auth_cookie_name_0;
            add_header Set-Cookie $auth_cookie_name_1;
        }
}

location = /oauth/authorize {
        internal;
        proxy_pass http://gatekeeper:3000/;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Scheme $scheme;
        proxy_set_header X-Forwarded-Port 80;
        proxy_set_header X-Forwarded-Server $host;
        proxy_set_header X-Forwarded-Proto $scheme;
        proxy_set_header X-Forwarded-Host $host;
        proxy_set_header X-Scheme $scheme;
        proxy_set_header Content-Length "";
        proxy_intercept_errors on;
        error_page 303 = @handle_redirect;
}

location @handle_redirect {
        set $redirect_location 'http://gatekeeper:3000/$upstream_http_location';
        proxy_pass $redirect_location;
        proxy_intercept_errors on;
        error_page 303 = @handle_redirect_auth;
}

location @handle_redirect_auth {
        set $auth_redirect_location '$upstream_http_location';
        proxy_pass $auth_redirect_location;
 }

Gatekeeper logs

info    accces token for user has expired, attemping to refresh the token       {"client_ip": "<IP_ADDRESS>", "remote_addr": "<IP_ADDRESS>:53070", "email": "", "sub": "5db927ed-53f6-4fe1-8b07-e0b608bbe006"}
warn    refresh token has expired, cannot retrieve access token {"client_ip": "<IP_ADDRESS>", "remote_addr": "<IP_ADDRESS>:53070", "email": "", "sub": "5db927ed-53f6-4fe1-8b07-e0b608bbe006"}
info    client request  {"latency": 0.00566194, "status": 303, "bytes": 86, "client_ip": "<IP_ADDRESS>", "remote_addr": "<IP_ADDRESS>:53070", "method": "GET", "path": "/"}
info    client request  {"latency": 0.000054795, "status": 303, "bytes": 297, "client_ip": "<IP_ADDRESS>", "remote_addr": "<IP_ADDRESS>:53074", "method": "GET", "path": "/oauth/authorize"}

Nginx logs

<IP_ADDRESS> - - [06/Oct/2023:21:22:54 +0000] "GET / HTTP/1.1" 500 579 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36" "-"
[error] 30#30: *3 auth request unexpected status: 303 while sending to client, client: <IP_ADDRESS>, server: resource-server-gatekeeper, request: "GET / HTTP/1.1", host: "localhost"

problem here is that auth_request module doesn't count that gatekeeper will do full code flow, forward-auth should be used only for yes/no decisions. It is only feature of gatekeeper that it does full code flow, not just simple yes/no...

Thank you
For anyone looking to use it with nginx-ingress:
 nginx.ingress.kubernetes.io/configuration-snippet: |       auth_request /auth;   nginx.ingress.kubernetes.io/server-snippet: |       location ^~ /auth {         internal;         proxy_pass <gatekeeper-url>;         proxy_pass_request_body off;         proxy_set_header Content-Length "";         proxy_set_header X-Forwarded-Proto $scheme;         proxy_set_header X-Forwarded-Host $host;         proxy_set_header X-Forwarded-Method $request_method;         proxy_set_header X-Forwarded-URI $request_uri;       }
Gatekeeper config:
no-redirects: true
no-proxy: true
...


hbs support

Would love to see https://handlebarsjs.com/ support 🚀

This plugin depends on nvim-treesitter plugin, so if there's no support there, there's nothing to do here.

@pedro757 Seems like tree sitter supports hbs

fixed


I might remove it and restore the old scrollback patch until scroll is more bugfree.

i hate it too. still using your old 0.8.2 cause of this.
at least with the scrollback patch, text staid fine if you did a horizontal split, while with scroll gets drunk anyhow that you split, and then poorly assembles the text.
it also changes the colors a tiny bit, though the colorscheme was not touched in the config.h
while i dont really like going back (bc i like vertical split (super+t)), i just wished scroll worked how its supposed to.

Thirded. I'm also getting the issue @BachoSeven mentioned in #274, "some terminals with neovim open in them "freeze"".
Only happens after resizing/opening additional terminal windows. Only fix is to super+q them, then deal with the swap file. Updated just yesterday since I couldn't be arsed to deal with the color changes and it's already doing my nut. Rolling back again for now.

Old scrollback patch is restored. If anyone keeps track of the scroll program, let me know if it gets good.


stable/datadog Add configMap for datadog.yaml config file

What this PR does / why we need it:
Added a configMap to provide the datadog.yaml file. This allows the configmap to configure the agent beyond what is available via environment variables.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #
N/A

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ x] Chart Version bumped
[ x] Variables are documented in the README.md
[ x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @CharlyF

Thanks for the contribution @grumpymatt - We are just about to get this one https://github.com/helm/charts/pull/13088 merged, I think it could be interesting to offer a configmap to configure each of the agents.
I'll discuss with the team tomorrow.
I see that that the linter is failing though, master is already at 1.30.9, could you rebase and bump it ?

@CharlyF Ok, I rebased and bumped the version. I am using these changes in one of our k8s clusters. We needed to be able to configure the agent to use haproxy as described here:
https://docs.datadoghq.com/agent/proxy/?tab=agentv6#using-haproxy-as-a-proxy
I could not find all the environment variables need to to make it work. So I think this configmap is needed, or more environment variables need to be added to allow equivalent configuration that way.

@grumpymatt Thanks for your patience, we got the change I was referring to merged so we are going to move forward with your contribution now.
Just so it is not a breaking change however, the default CM we ship should match https://github.com/DataDog/datadog-agent/blob/master/Dockerfiles/agent/datadog-kubernetes.yaml but more importantly, it should be behind a config flag.
Any way you can make these changes ?
Best,
.C

What do you want the flag to be?
Put it under the daemonset values?
Maybe like .Values.daemonset.useConfigMap
Then add to values:
  ## @param useConfigMap - boolean - optional
  #  Configures a configmap to provide the agent configuration
  #
  # useConfigMap: false

Yes, that's good! Could you just rebase and sign the DCO ? I'll approve thereafter (The lint check should pass when you bump the chart)

@CharlyF Ok, I made a couple mistakes while rebasing, hence the multiple pushes, but I think it looks good now?

Tested on my end, it looks good.

[x] Without the option (standard case), upgrading the chart is a no-op so it's not breaking.
[x] With the option, I can see the CM being created and used by the agent.

Mounting:
    Mounts:
      /etc/datadog-agent/datadog.yaml from datadog-monitoring-datadog-yaml (rw)
[...]
  datadog-monitoring-datadog-yaml:
    Type:      ConfigMap (a volume populated by a ConfigMap)
    Name:      datadog-monitoring-datadog-yaml
    Optional:  false

Usage in the lifecycle of the agent:
$ kubectl exec -ti datadog-monitoring-j7h4k bash
root@datadog-monitoring-j7h4k:/# agent status
Getting the status from the agent.

[...]

  Paths
  =====
    Config File: /etc/datadog-agent/datadog.yaml
    conf.d: /etc/datadog-agent/conf.d
    checks.d: /etc/datadog-agent/checks.d

And
root@datadog-monitoring-j7h4k:/# cat /etc/datadog-agent/datadog.yaml
## Provides autodetected defaults, for kubernetes environments,
## please see datadog.yaml.example for all supported options

# Autodiscovery for Kubernetes
listeners:
  - name: kubelet
config_providers:
  - name: kubelet
    polling: true

# Enable APM by setting the DD_APM_ENABLED envvar to true, or override this configuration
apm_config:
  enabled: false
  apm_non_local_traffic: true

# Use java cgroup memory awareness
jmx_use_cgroup_memory_limit: true

/ok-to-test

/approve

/lgtm


Unable to run prisma examples

Hello. Any tips to run the prisma examples?
I performed pnpm install. After that I tried to use pnpm generate but I get the following error:
$ pnpm generate

><EMAIL_ADDRESS>generate /home/fcanela/downloads/repo/pothos/examples/prisma
> prisma generate

Prisma schema loaded from prisma/schema.prisma
Error: Generator at prisma-pothos-types could not start:

node:internal/modules/cjs/loader:942
  throw err;
  ^
Error: Cannot find module '../lib/generator.js'
Require stack:
- /home/fcanela/downloads/repo/pothos/packages/plugin-prisma/bin/generator.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:939:15)
    at Module._load (node:internal/modules/cjs/loader:780:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/home/fcanela/downloads/repo/pothos/packages/plugin-prisma/bin/generator.js:2:1)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Module._load (node:internal/modules/cjs/loader:827:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/home/fcanela/downloads/repo/pothos/packages/plugin-prisma/bin/generator.js'
  ]
}
Node.js v18.2.0

If I try to start it directly, it also fails (make sense as I've skipped the previous step):
$ pnpm start

><EMAIL_ADDRESS>start /home/fcanela/downloads/repo/pothos/examples/prisma
> node -r @swc-node/register src/server.ts

node:internal/modules/cjs/loader:489
      throw e;
      ^

Error: Cannot find module<EMAIL_ADDRESS>    at createEsmNotFoundErr (node:internal/modules/cjs/loader:960:15)
    at finalizeEsmResolution (node:internal/modules/cjs/loader:953:15)
    at resolveExports (node:internal/modules/cjs/loader:483:14)
    at Function.Module._findPath (node:internal/modules/cjs/loader:523:31)
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:925:27)
    at Function.Module._load (node:internal/modules/cjs/loader:780:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/home/fcanela/downloads/repo/pothos/examples/prisma/src/builder.ts:1:41)
    at Module._compile (node:internal/modules/cjs/loader:1105:14) {
  code: 'MODULE_NOT_FOUND',
  path<EMAIL_ADDRESS>}

Node.js v18.2.0
 ELIFECYCLE  Command failed with exit code 1.

Any tips on how are you running it? Do I need to use an specific Node version or something?

Sorry, this got lost in my notifications.  I think you just need to build the plugin first.   Easiest option is probably just to run pnpm ci, which will get everything built and run tests to make sure it's all working

Let me know if you have any more issues, going to close this for now


Updating pcre dependency to pcre2

No, that seems to be the action trying to build shards. And that fails because the shards Makefile doesn't work on Windows.
I have no idea why it wants to build shards though. That seems to be completely unrelated to this change.


Closing a scope from multiple locations

How would one close a scope when it is being simultaneously accessed from multiple locations? For example in Android if you wan't to share a scope between two activities:
     Shared Scope
      /        \
     /          \
Activity 1    Activity 2

And you wan't to close it when both the activities are destroyed i.e after both of their onDestroy are called. Where exactly will you call Toothpick.closeScope(SharedScope.class) here?

You have to call close when your scope is actually closing.. :)
In the case of a scope spanning multiple activities, you have to close the scope when the user leaves this flow:

activity 1 on back key pressed or navigation up
activity 2 starting an activity outside the flow.
(if activity 1 can also start an activity outside the flow, i.e. an activity that is NOT activity 2, then you have to close the scope as well).

There is no way for TP to know which activities are grouped together in the flow and no way it can handle that for you.

Thanks a lot for the explanation!
But what if the scope is spanning an activity and a service and the order of them closing is undetermined?
One possible solution could be TP only closes a scope when closeScope has been called exact number of times openScope has been called?
If openScope was called 3 times closeScope should be called 3 times for it to close.


Renamed .css.scss files to .scss

Renamed the vendor/assets/stylesheets/ .css.scss to only .scss in order to silence Rails 5.0 deprecation warnings.

Awesomeness. Thanks, @hbroselid!

New release (v0.1.5) pushed: https://rubygems.org/gems/jquery-textcomplete-rails.


Add lazy loading to some components

Summary
Adding some lazy loading to certain components that are not shown always, like the invitation modal, purchase modal, or marketplace, for example.
The previous root bundle (loaded always): 6.5MB, 3.66MB gzipped
The new root bundle: 4.74MB, 2.52MB gzipped.
Ticket Link
Not yet
Release Note
NONE

@hanzei yes, we can do that, but probably we should do that as part of the Suspense component already there. I don't think is strictly  needed, but but can be added later.

@hanzei yes, we can do that, but probably we should do that as part of the Suspense component already there. I don't think is strictly needed, but but can be added later.

Sounds good to me. Should there be a ticket for that?

/update-branch

/update-branch

Error trying to update the PR.
Please do it manually.

@jespino can you please update branch. I'd like to run E2Es after update. Thanks.

@jgilliam17 done!

Thanks @jgilliam17 :ok_hand:

Thanks @jespino
E2E report looks good, one issue
3 tests are failing on "Add user to channel" it appears that multi-select list is displayed within the modal, see image

@jgilliam17 interesting, it is failing in my branch but not in master, so... the problem is related to how we do the lazy loading there. I'll take a look, thanks!

@jgilliam17 Fixed! can you take another look?

Thanks @jespino Here's the new E2E report
Re-tested manually and the issue with add member drop down not overflowing out of the modal is resolved
There is an issue with Getting Started "Complete your profile" section, on first load it is not expended. If you skip that step and expand "Setup notifications" and come back to Profile it will expand as expected.

@jgilliam17 really interesting bug, I'll take a look. Thanks!

@jgilliam17 I removed the lazy loading from that part because it is built in a way that is hard to fix the problem, so I reverted that part of the changes.

Thanks @jespino
Tested, looks good to merge.

Verified listed components are loading as expected on first try. E2E report looks good, no PR related failures.
One question before merge:
Is the next_steps component for Tips and Next Steps? I know that we still have Getting Started, and Tips and Next Steps were removed from the app recently. Just want to make sure I understand which one is changed here. Let me know. Thanks

@jgilliam17 the changes in this PR are just making things load lazily, which means, only gets loaded on first use. That makes a lot of sense for things like modals, or things that are only shown once during onboarding, for example.
The last changes has reverted part of that (just the lazy loading part), because some animations were depending the existence of that component with the real final size (that wasn't happening with this lazy load) so it was generating some issues, I just reverted the change to do it later.
I'm not sure if I'm answering your question here.

My question is related to this PR that removed Next Steps, just trying to understand if we need to make any changes to component we no longer use

@M-ZubairAhmed do you still need to take a look?

@jgilliam17 I think that is just removing one specific step on that form, so it shouldn't affect this PR, specially now that we are removing the lazy loading there.


Does azure-cosmos-spark_3-3_2-12 4.20.0 support Spark 3.3.2 in Databricks runtime 12.2?

Query/Question
Does azure-cosmos-spark_3-3_2-12 4.20.0 support Spark 3.3.2 in Databricks runtime 12.2?
In the README.md file, it is stated that only Spark 3.3.0 in Databricks runtime 11.x is supported by the latest azure-cosmos-spark_3-3_2-12 4.20.0.
But in the Quickstart documentation, it says:

Azure Cosmos DB Spark Connector supports Spark 3.1.x and 3.2.x and 3.3.x.
Throughout this quick tutorial, we rely on Azure Databricks Runtime 12.2 with Spark 3.3.2

Which one is correct?
I have been monitoring the README file for a while, hoping to see that Databricks 12.x gets supported so we can update. Is it the correct place to look for the latest compatibility information?
Why is this not a Bug or a feature Request?
I found an inconsistency in the documentation and would like to get clarifying information.
Setup (please complete the following information if applicable):

OS: Databricks runtime 12.2
IDE: N/A
Library/Libraries: azure-cosmos-spark_3-3_2-12 4.20.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Query Added
[X] Setup information Added

Spark 3.3.x is correct. At the time of release only 3.3.0 was available but we will support the patch versions. We'll update the readme soon to be consistent.

Yes, the Databricks runtime information is outdated for the Spark 3.4 readme, this will be corrected in next release.

Great! Again thanks for the info!

@TheovanKraay is this done? Can we close this issue?


sync2: add sqlstore

Motivation
The database-aware sync implementation is rather complex and splitting
it in parts might help the review process.
Description
The sqlstore package provides simple sequence-based interface to the
tables being synchronized. It is used by the FPTree data structure as
the database layer, and doesn't do range fingerprinting by itself.
SyncedTable and SyncedTableSnapshot provide methods that wrap the
necessary SQL operations. sql/expr package was added to facilitate
SQL generation.
The sql/expr package is also used by Bloom filters (#6332), which
are to be re-benchmarked and updated later. The idea is to use this
simple AST-based SQL generator in other places in the code where we're
generating SQL. The rqlite dependency which is introduced is also to
be used to convert schema SQL to a "canonical" form during schema
drift detection, and to filter out comments properly in the migration
scripts.
#6358 needs to be merged before this one.

bors merge


Differences between MBAR and BAR.

When performing FEP, I found there are significant differences between MBAR and BAR, as the figure shows.
The system is benzol and oxylene in water (solvated).

When only putting data of two adjacent states into MBAR,  MBAR results are the same with BAR.
But putting data of all states, the gap appears.

This is my code
from pymbar import MBAR as MBAR_
import numpy as np

N = [1000 for i in range(16)]
u_kn  = np.loadtxt("u.txt")

mbar_ori = MBAR_(u_kn, N)
out = mbar_ori.getFreeEnergyDifferences(return_dict=True)
print(out["Delta_f"][0, -1])

This is original file. The result of MBAR is -3.88 kbT, i.e,  -9.61852 kj /mol. But results of BAR and TI are nearly -17 kJ/mol.
u.txt
In my view, MBAR and BAR are both unbiased estimators, and thus the estimation (free energy difference ) should be  the same.  Could you please give me some instructions ? Thanks !

Thanks for the detailed report!
Could you provide the code and data needed to reproduce the BAR estimates too?
One thing we find often leads to this kind of error is incorrect units: BAR and MBAR will necessarily give almost identical output for two states no matter whether the energy units are correct (units of kT) or not, but once there are three states are more, the results will diverge if the energetics aren't self-consistent in the units you provide (which should be units of kT). One quick way to check this is to simply use exponential averaging (exp[-\Delta u(x)]) to reweight the p(u) from one sampled state to another sampled state and make sure they overlap. If the units are wrong, then it will be obvious the reweighted distribution does not overlap with the other state.

Thanks very much for your reply!
This is the original dhdl.xvg files obatined from gromacs.
data.zip
Actually I first used alchemistry-analysis to obtain the original TI , BAR , MBAR results.
My code is
python alchemical_analysis.py -d . -q xvg -p dhdl -u kJ

After I found they are inconsistent,  I tried to use pymbar directly , and gromacs gmx bar to obtain BAR results. This is gmx bar results.

Thanks for the detailed report!
Could you provide the code and data needed to reproduce the BAR estimates too?
One thing we find often leads to this kind of error is incorrect units: BAR and MBAR will necessarily give almost identical output for two states no matter whether the energy units are correct (units of kT) or not, but once there are three states are more, the results will diverge if the energetics aren't self-consistent in the units you provide (which should be units of kT). One quick way to check this is to simply use exponential averaging (exp[-\Delta u(x)]) to reweight the p(u) from one sampled state to another sampled state and make sure they overlap. If the units are wrong, then it will be obvious the reweighted distribution does not overlap with the other state.

So, it's rather hard to debug something occurring in alchemical-analysis, since it's in python2, which is not supported anymore.
Could you try the same calculating with alchemlyb (https://alchemlyb.readthedocs.io/en/latest/)? And see what you get there?

So, it's rather hard to debug something occurring in alchemical-analysis, since it's in python2, which is not supported anymore.
Could you try the same calculating with alchemlyb (https://alchemlyb.readthedocs.io/en/latest/)? And see what you get there?

Thanks for your reply!
Actually I've tried alchemlyb before, and my results are the same.

`
#!/usr/bin/env python
import pandas as pd
from alchemlyb.parsing.gmx import extract_dHdl, extract_u_nk
from alchemlyb.preprocessing import statistical_inefficiency
from alchemlyb.estimators import BAR, MBAR, TI
if name == 'main':
import glob
import os
import argparse

parser = argparse.ArgumentParser(description="Perform a simple alchemical analysis")

parser.add_argument('datadir', type=str,
                    help="directory of XVG files")
parser.add_argument('-T', type=float,
                    help="Temperature (K) simulations performed at")

args = parser.parse_args()

# get a list of files from the given directory
files = glob.glob(os.path.join(args.datadir, '*.xvg*'))

# extract and subsample dHdl using statistical inefficiency
print("Gathering dHdl from {} files".format(len(files)))
#dHdl = [extract_dHdl(i, T=args.T) for i in files]
dHdl = pd.concat([extract_dHdl(i, T=args.T) for i in files])
print(files)
print("Subsampling dHdl on statistical inefficiency")
#dHdl = pd.concat([statistical_inefficiency(i, i.iloc[:, 0]) for i in dHdl])

# extract and subsample u_nk using statistical inefficiency
print("Gathering u_nk from {} files".format(len(files)))
#u_nk = [extract_u_nk(i, T=args.T) for i in files]
u_nk = pd.concat([extract_u_nk(i, T=args.T) for i in files])
print("Subsampling u_nk on statistical inefficiency")
#u_nk = pd.concat([statistical_inefficiency(i, i.iloc[:, 0]) for i in u_nk])

# fit dHdl estimators
print("Fitting TI on dHdl")
ti = TI().fit(dHdl)

# fit u_nk estimators
print("Fitting BAR on u_nk")
bar = BAR().fit(u_nk)
print("Fitting MBAR on u_nk")
mbar = MBAR().fit(u_nk)

print("====== Results ======")

# print free energy estimates between lowest lambda and highest lambda
# these will all be in units of kT!
# NOTE: for BAR we would need to perform bootstrap sampling to get
# uncertainties of the endpoints
print("TI: {} +/- {} kT".format(ti.delta_f_.iloc[0, -1], ti.d_delta_f_.iloc[0, -1]))
print("BAR: {} +/- {} kT".format(bar.delta_f_.iloc[0, -1], "unknown"))
print("MBAR: {} +/- {} kT".format(mbar.delta_f_.iloc[0, -1], mbar.d_delta_f_.iloc[0, -1]))

`

Thanks!  That will make it a lot easier to debug. Could you perhaps upload that as a file (or a .zip, if can't be uploaded to make sure I get the indentation right?  It didn't copy over correctly.  Also, if you post the command you used.

OK, I think I have it working now!

Thanks a lot ! These are the script and data.
My command is python3  ~/research/19fall/fep/alchemical-analysis/alchemical-cal.py  -T 298
alchemical-cal.py.zip
data.zip

One question about the underlying data - I notice that the mass-dhdl is nonzero.  Are the masses changing from state A to state B? If so, is the potential or total energy being printed out to the dhdl?

One question about the underlying data - I notice that the mass-dhdl is nonzero. Are the masses changing from state A to state B? If so, is the potential or total energy being printed out to the dhdl?

The masses don't change from state A to state.  In my mdp files, the settings are
fep-lambdas              = 0.0   0.067  0.133  0.2  0.267  0.333  0.4   0.467   0.533  0.6   0.667   0.733   0.8    0.867   0.933   1.0 vdw-lambdas              = 0.0   0.067  0.133  0.2  0.267  0.333  0.4   0.467   0.533  0.6   0.667   0.733   0.8    0.867   0.933   1.0 mass-lambdas             = 0.0   0.0    0.0    0.0  0.0    0.0    0.0   0.0     0.0    0.0   0.0     0.0     0.0    0.0     0.0     0.0

In the topology, are the A and B states different masses? And it is the total or potential energy that is printed out?  And if so, do any of the atoms that have masses changing have constraints?
The fact that the mass-lamdas are zero eliminates some of the concerns, but I'm just trying to eliminate the possibility there is something weird going on with the underlying GROMACS data.  lambda-dependent masses haven't been tested nearly as much in GROMACS.

In the topology, are the A and B states different masses? And it is the total or potential energy that is printed out? And if so, do any of the atoms that have masses changing have constraints?
The fact that the mass-lamdas are zero eliminates some of the concerns, but I'm just trying to eliminate the possibility there is something weird going on with the underlying GROMACS data. lambda-dependent masses haven't been tested nearly as much in GROMACS.

In this case, A is benzol  and B is oxylene.  So there are dummy atoms in A which transform into H atoms in B. There are slight differences between their masses.

I didn't set constraints in this case.
This is the topology file.
processed.top.zip
In dhdl.xvg file, I didn't find potential energy and total energy. But I can find them in log file. Are there some incorrect settings ?

Ah, OK, the masses are definitely changing. You can check the mdout.mdp - depending on the version, there should be a dhdl-print-energy energy setting that is either total or potential.  What does it say?
If it would be POSSIBLE to run this calculation again in 1) NVT and 2) with constant masses, that would be helpful for debugging.   You can just leave the A masses constant; the free energies should be independent of the masses.
It may be something else, but this will eliminate some possibilities.

Did this ever get resolved?


improve performance by making Roblox API requests async and running them in parallel when possible

Since #16, Tokio has been added for making asynchronous requests to AWS. We can use this to improve the performance of deploys by parallelizing Roblox API requests. We will have to run the requests in batches and only run the next N requests which are not dependent on each other.

Consider fixing #89 when working on this

Switch from ureq to reqwest to implement this. Also look into creating a reusable HTTP client and saving cookies in them using the built-in session store rather than handling them manually.

Closing this as it is really not a priority (performance is great anyways) and would require significantly more complex and less readable core resolver logic.


microsoft/bcsandbox missing data

I have noticed that Docker image for Business Central has no data in it.
Is this intended?
Docker image version: 12.2.22932.23097.
Database: FinancialsW1.
Company: My Company.
Command for creating the container:
docker run -it --name mytest -h mytest -m 4g -e accept_eula=Y microsoft/bcsandbox

Edit: I am talking about W1 image.

Company "CRONUS USA, Inc." should have Cronus data

Yes, for W1 docker image it is intended.
W1 Docker images come exactly as you described. Only with My Company and without demo data pre populated.
Demo data is, however, loaded to image (look for Configuration Files, you'll find both Evaluation and Standard RS packages). You just need to import and apply the package.

thanks @ikoletic for correcting me. The company I mentioned only exists in the bcsandbox:us image

Thank you. I have found configuration packages.
But it is still unclear how to import them using Powershell (I already tried ordinary way).
I tried to use Powershell command Import-NavConfigurationPackageFile, but that did not do anything.

Did you get some errors when running the command? Have you specified all required parameters and executed it from the Docker powershell command prompt?
https://docs.microsoft.com/en-us/powershell/module/microsoft.dynamics.nav.management/import-navconfigurationpackagefile?view=dynamicsnav-ps-2018

No. Command runs just fine. The Import-NavConfigurationPackageFile command only imports package so it would be used when creating new company (from user interface).
New-NavCompany command can only use EvaluationCompany data - for example, you cannot use Extended data package, or custom data package.


Traversable definition uses undefined terms

I'm bothered by the definition of Traversable.  The remaining specifications use plain English, depend on one another, and build on "value" and "equivalent" from the Terminology section.
But Traversable uses three terms not covered in any of these places:

natural transformation
Id
Compose

My least favorite course in grad school was Category Theory, and I remember little of it, but it means that when I run across this stuff 25 years later, it's at least familiar, and I have some intuitive feel for what makes something a natural transformation.  But intuitive feel is not a good basis for a specification.
And it's pretty easy to guess at what Id and Compose should be and how they should work.  But there should be no guessing in a spec.
Unfortunately, I don't feel a strong enough grasp of Traversable to offer a reasonable alternative, and I apologize.  But I still think it's worth pointing out the issue.

It would be great to have natural transformation definition in terms of laws a JS function must obey in order to be natural transformation.
I'm trying to grasp it by reading https://en.m.wikipedia.org/wiki/Natural_transformation but not there yet.

As for Id and Compose maybe we can refer to implementations js files in this repo? We don't have one for Compose though, but I think I could make a PR.

Hm, isn't natural transformation a pair of functions and not one function?

For example, given our functors are Array and Maybe, we need a function that can convert [] -> Maybe and another one that converts Maybe -> []. And those two functions must obey the 'naturality square' law.
Is that correct?
What confuses me is that in Traversable law only one function is used. How a one function alone can be natural transformation?

I tried to express law for natural transformation in js, is it correct?
t(f.of(x)).map(a) ≡ t(f.of(x).map(a))

As for Id and Compose maybe we can refer to implementations js files in this repo.

I believe that a specification should not refer to implementations.  Moreover, I think it would be better to reconstrue this in terms of plain functions rather than these otherwise unmotivated types.

Hm, but laws written using snippets of JS code anyway. Isn't id.js and compose.js just bigger snippets?
Although Id.js has irrelevant to the Traversable law code in it, which may be a bad thing if we refer to it from spec. Because in this case it's not clear whether that additional code part of law or not.
Maybe we could inline minimal implementations of Id and Compose needed for that law in .md file?

The problem with not referring to specific data types is that traverse and sequence require an applicative of some other type. The laws only make sense for certain applicatives, because the laws encode the effects of those applicatives. So when the identity law is stated, it's saying, that if you "traverse" a data type with no additional effects, it's the same as embedding the data type into another with no additional effects.
If you try to formulate this with functions you end up complicating the actual law based on the effects that the Applicative instance of (->) r has.
You end up with something like:
u.map(x => y => x).sequence(x => y => x)(z) is equivalent to (x => y => x)(u)(z) (identity)
Or pulling out the duplication:
u.map(always).sequence(always)(z) is equivalent to always(u)(z) (identity)
Is that clearer than this formulation?
u.map(Id).sequence(Id.of) is equivalent to Id(u) (identity)
I would say not. Specifically because you have to explain the Applicative effects either way, and it's much easier to explain the effects of identity than (->) r. I am also not sure how to formulate the composition law.

u.map(x => y => x).sequence(x => y => x)(z) is equivalent to (x => y => x)(u)(z) (identity)

Yes, this is less clear. I don't understand it. So our of method here is x => y => x, which returns a function, but of should return a value that implements Applicative, which a function not implements.

You end up with something like:
u.map(x => y => x).sequence(x => y => x)(z) is equivalent to (x => y => x)(u)(z) (identity)
Or pulling out the duplication:
u.map(always).sequence(always)(z) is equivalent to always(u)(z) (identity)
Is that clearer than this formulation?
u.map(Id).sequence(Id.of) is equivalent to Id(u) (identity)

I would argue that both are clearer. (The latter only if you actually define always somewhere in the spec as x => y => x.)
If the spec read
u.map(Floop).sequence(Floop.grizzle) is congligial to Floop(u) (identity)
would you object?  What if we changed from "congligial" to "equivalent"?  What if we included the Floop type alongside the spec but made no reference to it anywhere within?  What if we switched from grizzle to of so at least it looked like an Applicative?
If you would object to
u.map(Floop).sequence(Floop.of) is equivalent to Floop(u) (identity)
with the local (unreferenced) version of Id renamed to Floop, but are happy with
u.map(Id).sequence(Id.of) is equivalent to Id(u) (identity)
then I believe there is something very wrong with how we are doing this specification.  Yes, things should be named to appeal to our intuitions, but if the very name of an undefined term is central to our understanding, we are probably standing on shaky ground.

I am probably not the right person to answer these questions :). Most of the time when I read unfamiliar specs or docs, I am aware that I won't know what all the terminology in it means and I'm fine with that. I just assume it'll become clear later.
If Id gets renamed to Floop it makes no difference, so long as the Applicative effects don't change. It's when the Applicative effects change that it makes things more complex.
Maybe I don't understand what I meant when I asked if one was clearer than the other. But it sounds subjective.
While the version:
u.map(x => y => x).sequence(x => y => x)(z) is equivalent to (x => y => x)(u)(z) (identity)
might be easier to read, it is necessarily more complex than:
u.map(Id).sequence(Id.of) is equivalent to Id(u) (identity)
precisely because a data type with no effects (Id) is simpler than a data type with ANY effects (Function).
I'll say this, Traversable isn't like the other algebras we've defined. It's probably the most useful one and definitely the most complex one as well. So whatever we can do to simplify and make more clear, I'm behind. But changing from Id to Function for the Applicative effects only makes it more complex.

Yes, this is less clear. I don't understand it. So our of method here is x => y => x, which returns a function, but of should return a value that implements Applicative, which a function not implements.

Whoops forgot about that. Then we'd have to monkey patch Function in the spec for the spec to be accurate.

@joneshf:
I think we might be talking past each other here:

If Id gets renamed to Floop it makes no difference, so long as the Applicative effects don't change. It's when the Applicative effects change that it makes things more complex.

But how do you know what those effects are?  Floop is not in the spec.  Neither is Id.  That's what is bothering me.  The spec is using 'Id' and 'Compose' as well as 'natural transformation' as though the user will understand them, without defining them anywhere.  So how do you know anything about the Applicative effects of Id.  Am I missing something?  Is it something you can actually derive from
u.map(Id).sequence(Id.of) is equivalent to Id(u) (identity)
?  Because I thought this was the other way around, trying to explain sequence in terms of (supposedly-) understood properties of Id.  Moreover is there something magic about Id in this definition, or is this simply a handy Applicative to use?
I was probably being a bit hyperbolic before when I said that this is a clearer formulation:
u.map(x => y => x).sequence(x => y => x)(z) is equivalent to (x => y => x)(u)(z) (identity)
It's not that this is actually clear to me.  I haven't really tried to understand any of them yet.  But I feel that I might have a chance of understanding this one without having to make leaps of faith that I understand what these undefined terms are about.  This depends on 'map', defined as part of Functor, 'equivalent', given as an initial definition, and standard Javascript notation to explain a law affecting 'sequence'.
The other formulation requires me to know about something called 'Id', which has a method called 'of', that from what I can tell might or might not be the Applicative of.  Is it?  How do you know that?  These are my concerns.

Moreover is there something magic about Id in this definition, or is this simply a handy Applicative to use?

Hm, this is interesting. I was convinced that this law about the Applicative with no effects. But now I'm not so sure. Naively it seems like it should work with any Applicative.

It's not that this is actually clear to me. I haven't really tried to understand any of them yet. But I feel that I might have a chance of understanding this one without having to make leaps of faith that I understand what these undefined terms are about.

I agree, If we find a way to express the law with some functions, that would be great. But this particular example doesn't seem right to me. And as it seems to me these laws can't be expressed with functions, we need some types like Id and Compose here.
IMO, the solution lies somewhere near inlining minimal Id and Compose implementations in the .md file, or we could write down the laws Id and Compose must obey in the spec.

I'm still very curious about formalizing natural transformation laws in terms of JS code. Could someone please answer my question from https://github.com/fantasyland/fantasy-land/issues/128#issuecomment-197854700 ?

Sorry for the late reply.

Am I missing something? Is it something you can actually derive from
u.map(Id).sequence(Id.of) is equivalent to Id(u) (identity)
? Because I thought this was the other way around, trying to explain sequence in terms of (supposedly-) understood properties of Id. Moreover is there something magic about Id in this definition, or is this simply a handy Applicative to use?

You're not missing anything. I don't think you can derive that. However, your question raises another one. Is it possible to generalize that law? Can we say:
For any `Applicative` `F`
`u.map(F.of).sequence(F.of)` is equivalent to `F.of` (identity)

It seems like we can. This would clean up that part of the spec by not requiring any specific Applicative.
Assuming we can, I wonder if something similar can happen for composition.

The other formulation requires me to know about something called 'Id', which has a method called 'of', that from what I can tell might or might not be the Applicative of. Is it? How do you know that?

It's implicit in the types of the values. This is unclear. Let's fix this.

Hm, this is interesting. I was convinced that this law is about the Applicative with no effects. But now I'm not so sure. Naively it seems like it should work with any Applicative.

Agreed. See above :).

IMO, the solution lies somewhere near inlining minimal Id and Compose implementations in the .md file

That is probably the quickest solution.

tried to express law for natural transformation in js, is it correct and complete?
t(f.of(x)).map(a) ≡ t(f.of(x).map(a))

If you remove that f.of part, then yes. That's correct. It should be:
t(x).map(a) ≡ t(x.map(a))

Opened a PR just to have something more concrete to discuss.

This probably can be closed since #131 merged, @CrossEye?

Closing this, reopen if you've got any concerns.


common: improve the rendering pipeline

enhanced the rendering composition target to better support features such as alpha blending, blending, masking, post effects, and more. This allows rasterizers to prepare the composition context with more precise control.
Additionally, resolved a crash in the software engine's post-effect process caused by a buffer size mismatch during XY buffer flipping.
issue: https://github.com/thorvg/thorvg/issues/3009
issue: https://github.com/thorvg/thorvg/issues/2984

@mgrudzinska could you please confirm that crash is fixed with this? https://github.com/thorvg/thorvg/issues/2984

@mgrudzinska could you please confirm that crash is fixed with this? #2984

yes, I confirm that this resolved all asan error for effects


Added interactive version of notebook 01

This interactive version copies Kaggle Learn courses by adding questions and hints and solutions.
Students who like Kaggle Learn, DataCamp, or Codecademy will likely enjoy using this notebook.

thank you for this!


Github login infinite loading

The login will keep displaying the login progress emblem without ever logging in

Hello @Belgianix ,
can you please provide me your Godot version?
Could you also take a screenshot/paste the messages printed in the output console from the plugin?
Thank you.

Sorry for the late reply but I'm currently running V3.2.1. Below is the screen and terminal which it gets stuck at

@Belgianix Don't worry about the delay in replying.
The plugin is currently being developed and supporting Godot Engine 3.2.2, but anyway I can't  reproduce the issue you are having.
Have you tried downloading the latest plugin version?

@Belgianix I had that problem and all I did is to open a new project and install it there, it worked great, hope that helps you aswell

#46 I'll mark this as resolved.


[Creator] Enable option to generate SAS token using ARM template functions

The listAccountSas function along with date functions can be used to generate SAS token for linked templates stored in a storage account. This is especially helpful when one has a complex hierarchy of linked templates and to be able to use parametersLink to reference a parameters file without having to worry about passing the SAS token along.
The master template could have a variables section with the following:
"variables": {
    "newDateTime": "[dateTimeAdd(parameters('currentTime'), 'PT20M')]",
    "sasProperties": {
        "signedServices": "b",
        "signedPermission": "rl",
        "signedExpiry": "[variables('newDateTime')]",
        "signedResourceTypes": "co"
    },
    "storageAccountResourceName": "[concat('Microsoft.Storage/storageAccounts/', parameters('StorageAccountName'))]"
  }

The linked templates would then be referenced like so:
"templateLink": {
          "uri": "[concat(parameters('LinkedTemplatesBaseUrl'), '/myLinkedTemplate.template.json?', listAccountSas(variables('storageAccountResourceName'), '2018-11-01', variables('sasProperties')).accountSasToken)]",
          "contentVersion": "<IP_ADDRESS>"
        },

Thank you for your input on the creator tool.  We are currently looking at the future of the creator tool and would like your feedback.  Please could you fill in the survey at https://forms.office.com/r/w7rQp88NwH - your input is appreciated.

This project is now in "maintenance" mode. This means that no new features will be accepted and fixes will be limited to security fixes. We expect to retire the DevOps Resource Kit by the end of 2023. At that time, the DevOps Resource Kit repository will be archived and there will be no further updates.
While you are free to fork the DevOps Resource Kit yourself, we recommend that you migrate to the following products:

If you are using creator, switch over to using Bicep templates. Bicep is a descriptive language for describing infrastructure, fully supports all features of Azure API Management, and is covered in any support agreement that you have with Azure. You can find bicep modules and examples for handling API Management APIs at https://github.com/azure-samples/api-management-sample-apis.
If you are using extractor, switch your pipelines to use APIOps. APIOps applies the concepts of GitOps to API deployments. It is faster and more flexible than the DevOps Resource Kit.


Issue with transparency of bitmaps with jpeg compression setting in swf library

And one more issue (develop branch, openfl 4.5.2, lime 3.5.2)
At using a swf library, images in library, which have transparent areas (and in the properties panel have jpeg compression), shown with black areas in places where there should be a transparent pixels.
However, when using openfl 4.5.0 and lime 3.5.0, it works correctly. Despite the jpeg compression in swf, we have a normal transparent pixels.2. At using a swf library, images in library, which have transparent areas (and in the properties panel have jpeg compression), shown with black areas in places where there should be a transparent pixels.
However, when using openfl 4.5.0 and lime 3.5.0, it works correctly. Despite the jpeg compression in swf, as a result we have a normal transparent pixels.

Please try the latest round of commits, I've made a lot of changes that should improve the situation here. Thanks :smile:

Thanks, now it works correctly!


Follow-up work to #612

Look at the PR description in https://github.com/get10101/10101/pull/612 for context.
This is primarily a reminder for myself that I don't want to forget about the corners we cut when working on #612.

I am removing MVP as the (my) follow up remarks are definitely not critical for MVP :)

Tibo's recent updates to the forks we are using of rust-dlc and rust-lightning should be very relevant here.


rake -T failed, cannot load such file bundler/setup

Hi,
after updating atom to 1.13.0 the rake plugin fails to load.
Command failed: rake -T
rake aborted!
cannot load such file -- bundler/setup
/Users/Shared/xxx/server/config/boot.rb:3:in `<top (required)>'
/Users/Shared/xxx/server/config/application.rb:1:in `<top (required)>'
/Users/Shared/xxx/server/Rakefile:4:in `<top (required)>'
(See full trace by running task with --trace)

Stack Trace

Error: Command failed: rake -T
rake aborted!
cannot load such file -- bundler/setup
/Users/Shared/xxx/server/config/boot.rb:3:in `<top (required)>'
/Users/Shared/xxx/server/config/application.rb:1:in `<top (required)>'
/Users/Shared/xxx/server/Rakefile:4:in `<top (required)>'
(See full trace by running task with --trace)
    at ChildProcess.exithandler (child_process.js:218:12)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at maybeClose (internal/child_process.js:877:16)
    at Socket.<anonymous> (internal/child_process.js:334:11)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at Pipe._handle.close [as _onclose] (net.js:493:12)

I tried bundle install and updating bundler and bundle exec rake rails:update:bin but no fix so far. Any guesses?
This only happens right after starting atom. When hitting cmd+alt+t it loads the targets (after a while).

Happens on another machine yet with atom 1.12.6, too.

Guess I figured it out. The error is popping up when atom is starting into the settings pane (or any other non-project tab). This mostly happens after updating packages (for I'm hitting restart from the settings pane).

I'm still seeing the same thing on Atom 1.18.0:

Project
arclight-demo
.ebextensions
.git
app
bin
config
data
db
lib
log
public
solr
test
tmp
vendor
.gitignore
.solr_wrapper.yml
.travis.yml
config.ru
Gemfile
Gemfile.lock
LICENSE.txt
Rakefile
README.md
downloads.yml
schedule.rb
untitled
untitled
repositories.yml
ARS.0046_ead.xml
robots.txt
 1
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 •
50
51
52
53

  city: 'Stanford'
  state: 'CA'
  zip: '94305-6003'
  phone: '650-723-1754'
  contact_info: ''
  thumbnail_url: 'https://www.hoover.org/sites/default/files/styles/one_by_one/public/library/slides/poster_rusu_02237.jpg'
  visit_note: 'Archival holdings are brought from the stacks to the reading room at 9:00 a.m., 10:30 a.m., 11:30 a.m., 1:30 p.m., and 3:00 p.m. Some of the archives’ collections are stored off-site.'

nlm:
  name: 'National Library of Medicine. History of Medicine Division'
  description: 'NLM’s History of Medicine Division collects, preserves, makes available, and interprets for diverse audiences one of the world’s richest collections of historical material related to human health and disease.'
  building: 'Building 38, Room 1E-21'
  address1: '8600 Rockville Pike'
  address2: ''
  city: 'Bethesda'
  state: 'MD'
  zip: '20894'
  country: 'USA'
  phone: ''
  contact_info<EMAIL_ADDRESS>  thumbnail_url: "https://collections.nlm.nih.gov/pageturnerserver/ajaxp?theurl=http://localhost:8080/fedora/get/nlm:nlmuid-101421040-img/THUMB"
  google_request_url: 'https://docs.google.com/a/stanford.edu/forms/d/e/1FAIpQLSeOamhY_IcFw4sPnz0ddwWWkrPaHbM5wp7JVbOLOL_mIusEyw/viewform'
  google_request_mappings: "document_url=entry.1980510262&collection_name=entry.619150170&collection_creator=entry.14428541&eadid=entry.996397105&containers=entry.1125277048&title=entry.862815208"

sul-ars:
  name: 'Stanford University Libraries. Archive of Recorded Sound'
  visit_note: 'The Archive’s materials are for on-site use only. It is advised that visitors contact the Archive in advance so that required materials can be prepared.'
  description: 'The Archive of Recorded Sound’s collection contains over 400,000 items that span sound recording history--from its beginnings to the present day. Nearly all formats ever developed to record sound are represented: piano rolls; wax cylinders; shellac, acetate, aluminum, and vinyl discs; magnetic wire and tape recordings; compact discs. Collections are particularly strong in the areas of jazz and opera.'
  building: 'Braun Music Center'
  address1: '541 Lasuen Mall'
  address2: ''
  city: 'Stanford'
  state: 'CA'
  zip: '94305-3076'
  country: 'USA'
  phone: ''
  contact_info<EMAIL_ADDRESS>  thumbnail_url: 'https://library.stanford.edu/sites/default/files/styles/150x150/public/branch/image/Edison_002LowRes.jpg'

sul-spec:
  name: 'Stanford University Libraries. Special Collections and University Archives'
  visit_note: 'Special Collections and University Archives materials are stored offsite and must be paged 36 hours in advance.'
  description: 'The Special Collections and University Archives Department of Stanford University Libraries is committed to acquiring, preserving, and providing access to primary source materials that support the research needs of the Stanford community and beyond. There are three distinct divisions in our department: Manuscripts, Rare Books and University Archives. We welcome all students, scholars and members of the general public to the Field Reading Room. Learn more about using and
  accessing our collections.'
  building: 'Green Library'
  address1: '557 Escondido Mall'
  address2: ''
  city: 'Stanford'
config/repositories.yml1:1
LFUTF-8YAMLgit+
master0 files
Ooops. Something went wrong in the Rake build provider.
Command failed: rake -T
rake aborted!
cannot load such file -- bundler/setup
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/config/boot.rb:3:in `<top (required)>'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/config/application.rb:1:in `require_relative'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/config/application.rb:1:in `<top (required)>'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/Rakefile:4:in `require_relative'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/Rakefile:4:in `<top (required)>'
(See full trace by running task with --trace)
Hide Stack Trace
Error: Command failed: rake -T
rake aborted!
cannot load such file -- bundler/setup
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/config/boot.rb:3:in `<top (required)>'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/config/application.rb:1:in `require_relative'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/config/application.rb:1:in `<top (required)>'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/Rakefile:4:in `require_relative'
/Users/matienzo/Documents/Code/sul-dlss/arclight-demo/Rakefile:4:in `<top (required)>'
(See full trace by running task with --trace)
    at ChildProcess.exithandler (child_process.js:218:12)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at maybeClose (internal/child_process.js:877:16)
    at Socket.<anonymous> (internal/child_process.js:334:11)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at Pipe._handle.close [as _onclose] (net.js:493:12)

For me it's working on atom 1.47.0, linux


Access Denied when using PKG on MacOS and Linux

Hi there,
I installed the latest pkg (4.3.4):
npm install pkg -g
Then I execute pkg:
pkg ./bin/backend.js --target node6-win-x86 -o ./dist/backend.exe
Error Message:
`><EMAIL_ADDRESS>
Error! Error: EACCES: permission denied, scandir '/.DocumentRevisions-V100'`

My node version: 7.5.0
I tried the following so far:

reinstalling pkg
setting PKG_CACHE_PATH to the current working directory like:
PKG_CACHE_PATH=/Users/someuser/mypkgcache

It also occurs on my build server (Linux), only the folder is different: /.cache
Any ideas here?

It is indeed the same error. I found a module which contains filepaths to "/.js" etc. I fixed the problem locally by checking for leading slashes and removing them. Also added a pull request for the fix.

Can confirm. Had the same error from a third party dependency to a array-from polyfill. Watch the debug output and check the last package.json for things like https://github.com/studio-b12/array-from/blob/24de20f9f765a3ba18400fda43b2e4e1b70a0541/package.json#L28

@fisch3r Thanks. I can confirm that this bug is no longer apparent when using your version of pkg (npm install --save-dev fisch3r/pkg#bug/524).


Use Variational Autoencoders for smrt

sknn was not working: its Autoencoder is deprecated, and the author is unresponsive. We had to implement our own... these commits add:
- smrt.autoencode.AutoEncoder
- smrt.autoencode.VariationalAutoEncoder

... and related classes. Furthermore, it adds an implementation of smote against which to compare results. The .TeX file is still in progress, but the code is now functional.

@jasonw247 before merging into master, I want your opinion on the code. Could you look at everything from docstrings to PEP 8, etc.? If it's going into master, I want your thoughts first.

(Btw, we're in a deadlock with Coveralls until this merge—long story)

Forcing merge since we're deadlocked with coveralls until the badge is on the master branch README (after we changed the name of the repo, things got hairy).