Include extension points?

We have designed our application to include the notion of soft-deleting Entities.  e.g. a IsDeleted flag on the Entity indicates that it is soft-deleted whereas a hard-delete does a physical delete on the data in the database.
Are there any extension points when using Include's that would allow us to filter out soft-deleted Entities from result sets?
Example:
var employees = employeeService.FindAll().Include(e => e.Address)

In the above example, employeeService's FindAll method filters soft-deleted data at the top level but it would be nice to have an unobtrusive way to filter Addresses to return only non soft-deleted rows.

You can probably close th

@rowanmiller @Bartmax
Do you know what happened to issue #1833?
I get 404, but I know there was such an issue (I got email notifications for that issue, and this one was closed as a duplicate of it) and as AFAIK GitHub doesn't provide an API for deleting issues.
Anyway, both me and @Bartmax asked how can we do stuff like
.Include(x=> x.Images.Where(image => !image.IsDeleted)

Is there a different issue tracking it?
I searched quite a lot but couldn't find anything.

Hmm, #1833 is working again.
Not sure what was the issue.


distribute Docker images rather than Dockerfiles

I recommend that instead of distributing Dockerfiles, you distribute Docker images:  You can add a GitHub action which, upon creation of a release branch, builds a Dockerfile and pushes the resultant image into the GitHub Docker registry, tagged according to the release branch. Users would then use docker pull to get the prebuilt image.  Doing this will help ensure reproducibility of computational results as well as speeding things up (since users will not have to build images).  Here's an example of a GitHub workflow that builds a Dockerfile upon creation of a release branch:
https://github.com/brucehoff/scheduled-job-example/blob/main/.github/workflows/docker_release.yml
The embedded comments link back to GitHub's doc's where further guidance can be found.

This is exactly what I envision for the repo!


The dataset about Cifar10 or SVHN

Dear Zifeng Wang
My name is Kong shuming and I'm a graduate student in ShangHai JiaoTong universitry. Recently I'm interested in your paper "Less Is Better: Unweighted Data Subsampling via Influence Function". I totally agree with you that sometimes using the smaller data via influence function will get better performance. I get the same result in MNIST dataset and cancer dataset. I follow your github and try to process the dataset such as cifar10 or SVHN. However I am confused about how to create the raw text of the dataset. So I am really appreciated if you could show me one example or give me the dataset.npy. So one line in the raw text is label plus feature vector? Can you give me one example such as cifar10 raw text? Thank you all the time. My email is<EMAIL_ADDRESS>                                                                                Best wishes                                                                                Kong shuming                                                                                2020/3/29

All the raw data used in experiments of this paper can be found in LibSVM-DataSet.


Czech localization problem / font problem

Hi,
in the version 2.6.0 of TQVaultAE, the special characters, czech letters (á é í ó ú ů ý ž š č ř ď ť ň ě and Á É Í Ó Ú Ů Ý Ž Š Č Ř Ď Ť Ň Ě) ARE NOT FULLY SUPPORTED.
In the version 2.5.6 of TQVaultAE THOSE LETTERS WERE FULLY SUPPORTED.
Could you, please, in any future release of the program, use the font, which was used in the version 2.5.6? I attach the screen of the version 2.6.0, so you can see, what am I talking about.
Thank you in advance,
Sir-Mac

There were some font fixes lately, we'll investgate what broke it.

I am using the version 3.0.0 alpha right now and in this version, the font is alright again. If it occurs (or anything else) in any of future versions, I will make a proper thread about it.
Thank you once again, Etienne Lamoureux

Glad to hear it! Although the credits go to our awesome contributors.


pip cannot install jupyter ascending

$ pip install jupyter_ascending
Collecting jupyter_ascending
  Using cached jupyter_ascending-0.1.27-py3-none-any.whl.metadata (743 bytes)
Collecting aiohttp<4.0.0,>=3.8.4 (from jupyter_ascending)
  Using cached aiohttp-3.9.5-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (7.5 kB)
Collecting editdistance<0.7.0,>=0.6.2 (from jupyter_ascending)
  Using cached editdistance-0.6.2.tar.gz (31 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [25 lines of output]
      Traceback (most recent call last):
        File "/home/oli/miniconda3/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/home/oli/miniconda3/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/home/oli/miniconda3/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-ocdndz0u/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 325, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-ocdndz0u/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 295, in _get_build_requires
          self.run_setup()
        File "/tmp/pip-build-env-ocdndz0u/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 25, in <module>
        File "/tmp/pip-build-env-ocdndz0u/overlay/lib/python3.12/site-packages/Cython/Build/Dependencies.py", line 1010, in cythonize
          module_list, module_metadata = create_extension_list(
                                         ^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-ocdndz0u/overlay/lib/python3.12/site-packages/Cython/Build/Dependencies.py", line 845, in create_extension_list
          for file in nonempty(sorted(extended_iglob(filepattern)), "'%s' doesn't match any files" % filepattern):
        File "/tmp/pip-build-env-ocdndz0u/overlay/lib/python3.12/site-packages/Cython/Build/Dependencies.py", line 117, in nonempty
          raise ValueError(error_msg)
      ValueError: 'editdistance/bycython.pyx' doesn't match any files
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

I'm getting the same error. Are there any solutions?

I had the same problem trying to install it on remote CentOS 7.
What helped was following the instructions regarding the 'editdistance' dependency mentioned in the thread below:
https://stackoverflow.com/questions/77484622/valueerror-editdistance-bycython-pyx-doesnt-match-any-files-when-downloading
I needed to download the specific version of the dependency from github, as opposed to the latest from PyPI
pip install<EMAIL_ADDRESS>

PR accepted? I'd update the version of editdistance, it breaks pip currently. Have to override with the git version 0.6.2, newest editdistance is 0.8.x


doesn't work, no examples for modern js

Tried webpack, and rollup and browserify and had constant errors.
why don't you have examples?  This code is like looking at 1990s spaghetti code.

Didn't the official React example work out for you?
It works for me.
https://github.com/ffmpegwasm/react-app/blob/master/src/

Can you elaborate your issues? It works fine for me.

Very constructive issue. Love those types of developer.
Also they have a few examples in their website. Didn't see those?
Try to elaborate more your issue next time.


Fix JS tests- restore code lost during merge commit

Code lost in: d08b649142834c3c3e3df873c86bfae761cb2f50
Fix JSHint/ Karma failure in CI- function is called, but code was lost in merge commit.

Looks good. When tests pass, or if someone more familiar with the code in question ahem @caneruguz ahem code reviews it, I'll merge it in.


layers.pop no longer removes the last layer

Hi,
I am stuck on this issue where layers.pop() no longer removes the last layer. My workaround was to use _layers.pop but this has now been deprecated. I am aware that the new usage should be to get the output from Model.layers[-2] (or whichever layer you would like as the output) and define a new model with this as the output.  The issue with this is it is now impossible to load weights saved by a model which had used the original .pop() method to remove the last n layers. Is there any workaround to get the old _layers.pop() behaviour back?
Thank you in advance

Hi PolarBean,
I was actually using your DeepSlice repository when I came across this error. Im struggling to find a workaround for     base_model._layers.pop() as well. Did you find a way of brute-forcing your way into using the Xception weights?
Thanks in advance,
Riley

@PolarBean I'm not sure if this is still useful to you or anybody else, but I was running into this issue recently, using Python 3.11.5 and Tensorflow 2.13.1. I think that I've found the solution to circumvent the bug that should be compatible with Tensorflow 2.xx.
The base_model._layers.pop() call in the original code only cosmetically removes the last two layers in the Xception model, but during the actual execution of the graph, these last two layers (the AveragePooling and Softmax layers) are still being performed. Thus, the output shape of the Xception layer in the overall DSModel.model is still (1000,) and correctly interfaces with the rest of the model. However, if you try to access base_model.layers or even base_model.summary(), these last two layers are hidden and nowhere to be seen.
My workaround is, instead of using model.load_weights(), we must manually set each of the layers using model.layers[idx].set_weights(list_of_numpy_weights) after loading the weights in with h5py. I wrote this function to be included in the neural_network.py module. It should be called whenever model.load_weights() was being called:
def load_xception_weights(model, weights):
    with h5py.File(weights, "r") as new:
        # set weight of each layer manually
        model.layers[1].set_weights([new["dense"]["dense"]["kernel:0"], new["dense"]["dense"]["bias:0"]])
        model.layers[2].set_weights([new["dense_1"]["dense_1"]["kernel:0"], new["dense_1"]["dense_1"]["bias:0"]])
        model.layers[3].set_weights([new["dense_2"]["dense_2"]["kernel:0"], new["dense_2"]["dense_2"]["bias:0"]])

        # Set the weights of the xception model
        weight_names = new["xception"].attrs["weight_names"].tolist()
        weight_names_layers = [name.decode("utf-8").split("/")[0] for name in weight_names]

        for i in range(len(model.layers[0].layers)):
            name_of_layer = model.layers[0].layers[i].name
            # if layer name is in the weight names, then we will set weights
            if name_of_layer in weight_names_layers:
                # Get name of weights in the layer
                layer_weight_names = []
                for weight in model.layers[0].layers[i].weights:
                    layer_weight_names.append(weight.name.split("/")[1])
                h5_group = new["xception"][name_of_layer]
                weights_list = [np.array(h5_group[kk]) for kk in layer_weight_names]
                model.layers[0].layers[i].set_weights(weights_list)
    return model

Oh wow thats amazing. Are you using this fix with DeepSlice? If so perhaps you could open an issue on DeepSlice and we can discuss opening a pull request? Thank you!

@PolarBean Yup, I tried it and got the same results between tensorflow 1.15 and 2.13! I'll open an issue on DeepSlice


fix(chat): fix collapsing folders on import (Issue #1958)

Description:

fix collapsing all folders on conversation import
fix collapsing folders on prompt import

Issues:

Issue #1958

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


cclient - split core_api and extended_api into individual compilation units

These large files make incremental compilations slow and increase chances of merge conflicts once multiple developers work on them.  We should split these up similar to cclient/serialization/json

fixed in #914


Change EventGrid component implementation to use the new Azure SDK with data polling

At the time of the EventGrid component creation there was no Go SDK for it. We had to use an ARM Management SDK, and only delivery via a public endpoint (pushing of events) was supported as a result. This was complicated to configure for users.
Now there is a GO SDK that supports polling. We should rewrite the component to use the official Go SDK. This would be vastly more convenient and secure for users.
https://github.com/Azure/azure-sdk-for-go/tree/main/sdk/messaging/azeventgrid

Agree. That would have helped us in a couple engagements.
Relevant would on how batching could be used and how would dapr handle accepting/ rejecting individual events?
Also great if there would be a metric that could be used to monitor the number of batches.

@denniszielke support for a native Batch/Bulk subscription would not be part of this switch in the PubSub component and that would need to come in a subsequent implementation.
Maybe we would release this updated component as a v2 so that we can retain the existing behavior as a v1 component (with no further updates however). The v2 version with polling would lack support for native bulk / batch initially. This would be a subsequent feature someone can add. You can read up on Dapr Bulk Pubsub on the Dapr docs. This is how we would need to implement bulk / batch handling.
As for the switching the Binding to the new SDK and support Batch/Bulk there. This could be implemented more freely - and suggestions or contributions are welcome here. There is no precedent that must be followed.
I do want to set realistic expectations that all of this is more of a wish list / dream I would like to see as maintainer. But neither I or anyone I'm aware of have time and resources to implement this. Contributions are welcome of course.


W-12651574-use-maven-in-studio-CP

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

@mdaloia, let me tell you that our doc team has some guidelines for publishing an image on MuleSoft's Doc page (https://confluence.internal.salesforce.com/x/f_9oF).
These guidelines explain that images should have good quality, follow some padding to make them similar in size when published, and that the .png and .graffle files should be included in the repo.
If you think that these guidelines may represent a problem in the future, please, let us know and we can go over this topic with the appropriate team. Thanks!

@mdaloia, let me tell you that our doc team has some guidelines for publishing an image on MuleSoft's Doc page (https://confluence.internal.salesforce.com/x/f_9oF). These guidelines explain that images should have good quality, follow some padding to make them similar in size when published, and that the .png and .graffle files should be included in the repo. If you think that these guidelines may represent a problem in the future, please, let us know and we can go over this topic with the appropriate team. Thanks!

Ok, my comment just was to let you know that it is not an appropriate usage of git an you (your team) will maybe face the consequences in the future. If it is how you handle it today as a team then ok but I encourage your team to review git-lfs.


created 1st floor of house in house-base inside Assets

Used built-in tool Probuilder to create the framework of the 1st floor of the house
2nd floor still in progress
disregard rough-base scene inside LevelDesign, Unity glitched out and deleted the entire model :(.

🏠


Fix typo s/harnessed/harnesscd

This PR fixes some typos on HDH; many of them are incorrect autocorrection of harnesscd-example-apps to harnessed-example-apps. Someone probably did a bulk autocorrection at some point or folks copied the incorrect repo name over and over.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[x] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Directly measuring compression information loss on my sequences

I'd like to run some of the pretrained models on my sequences and assess the information loss using different shortening factors and channel numbers. In the paper, you achieve this through measuring the performance of predictors trained using the compressed embeddings. What I am looking for is a more direct measure - I want to compare the original ESM embeddings to the ones extracted from the last layer of the hourglass model.
I was able to get the later using pipeline.decode(emb, mask), which returned matrices of size 327x16 in the case of the usage example notebook.
How can I get the original ESM embeddings (with dimensions Lx1024)?

OK, I might have figured it out:
res = pipeline.esmfold_embed_only_module.infer_embedding(sequences)
res['s']

returns the original ESM embeddings. Is that correct?

yup! that's correct :)


v0.4.8: forecast/all and chart x axis labels

Pull Request
Description

Forecast/all split historic/future data
Charts: dates on x axis
Unit tests for some utils! 🎉

Fixes #418
How Has This Been Tested?

[x] Unit tests added for split functions;
[x] Vercel preview branch
[ ] UAT (below)

UAT
Probabilistic Charts

[x] Shading appears around the lines.
[x] Plevel values in the tooltip are 0.0 or above.
[x] Probabilistic range shading and tooltip values appear on the DeltaView charts.

Quartz-specific points

[x] Does the logo look like the new Quartz logo?
[x] Does the logo link to the new Quartz website?
[x] Is the OCF logo present in the logo area?
[x] Does the OCF logo link to the OCF website?
[x] Does the Documentation link go to the new Quartz docs?

PV Forecast
National

[x] Does PV live estimate show on the plot? (black dashed)
[x] Does PV live updated show on the plot? (black solid)
[x] Does OCF forecast show up on plot? (yellow)
[x] For OCF forecast, is the future a dashed line?
[x] Can I hover on the plot to show the values on the plot?
[x] Does the data look like solar profiles?
[x] Clicking on a GSP makes the GSP plot show up?
[x] Clicking on a GSP when already selected, makes the GSP plot go away?
[x] Can I click on a time in the national plot which then updates the map?
[x] Can I click on a past time in the GSP plot which then updates the map?
[x] Can I click on a future time in the GSP plot which then updates the map? – [expected; PVnet only runs 8h ahead for GSPs]
[x] Are values higher on the map in the middle of the day, compared to early morning?

Time

[x] Is time now in Europe London time? It should be to the nearest 30 minute, rounded up.
[x] Is the hover time in Europe London time?
[x] Are the X axis in Europe London time?
[x] Does the data show yesterday as well?
[x] Does the data show tomorrow as well?

Header

[x] Is this the title clear it is National data?
[x] Does the headline current estimated PV figure display?
[x] Does it show the current Forecasted value?
[x] Does the next forecast figure display?
[x] Are the units GW?

GSP

[x] Does PV live estimate show on the plot? (dashed black)
[x] Does PV live updated show on the plot? (solid black)
[x] Does OCF forecast show up on plot? (yellow)
[x] For OCF forecast, is the future a dashed line?
[x] Can I hover on the plot to show the values on the plot?
[x] Does the data look like solar profiles?

Time

[x] Is time now in Europe London time?
[x] Is the hover time in Europe London time?
[x] Are the X axis in Europe London time?

Title

[x] Does the GSP name show up?
[x] Is the current estimated PV visible?
[x] Is the next forecast figure visible?
[x] Are the above figures in MW?

Map

[x] Does a map of the UK show up?
[x] Are the GSP boundaries displayed?
[x] Can I click on '%', 'MW' and 'Capacity'
[x] on 'Capacity' view, is there very little power in Scotland, and Melksham a shining beacon of renewables?

Four Hour View
National

[x] Does the "4hr forecast" toggle show in the Profile Dropdown menu?
Toggle on

[x] Does the 4hr forecast show up on the plot? (orange)
[x] Does the 4hr future forecast show up on the plot? (dashed orange)
[x] Does the 4hr forecast not show on the chart or tooltip when switched off?

Toggle off

[x] Does the 4hr forecast not show on the chart or tooltip when switched off?
[x] Does the 4hr future forecast not show on the chart or tooltip when switched off?

[x] Does the 4hr forecast toggle persist across refreshes/logouts using cookie?
[x] Do the 4hr forecast values show in the hover tooltip?

Delta View
National

[x] Does PV live estimate show on the plot? (black dashed)
[x] Does PV live updated show on the plot? (black solid)
[x] Does OCF forecast show up on plot? (yellow)
[x] For OCF forecast, is the future a dashed line?
[x] Does the 4-hour forecast not show on the chart or tooltip when switched off?
[x] Does the 4-hour future forecast not show on the chart or tooltip when switched off?
[x] Does the 4-hour forecast show on the chart and tooltip when switched on?
[x] Can I hover on the plot to show the values on the plot?
[x] Does the data look like solar profiles?
[x] Clicking on a GSP makes the GSP plot show up?
[x] Clicking on a GSP when already selected, makes the GSP plot go away?
[x] Can I click on a time in the national plot which then updates the map?
[x] Can I click on a time in the GSP plot which then updates the map?
[x] Are values higher on the map in the middle of the day, compare to early morning?

Time

[x] Is time now in Europe London time? It should be 30minute, but rounded up.
[x] Is the hover time in Europe London time?
[x] Are the X axis in Europe London time?
[x] Does the data show yesterday as well?

Header

[x] Is this the title clear it is National data?
[x] Does the headline current estimated PV figure display?
[x] Does it show the current Forecasted value?
[x] Does the next forecast figure display?
[x] Are the units GW?

GSP

[x] Does PV live estimate show on the plot? (dashed black)
[x] Does PV live updated show on the plot? (solid black)
[x] Does OCF forecast show up on plot? (yellow)
[x] For OCF forecast, is the future a dashed line?
[x] Can I hover on the plot to show the values on the plot?
[x] Does the data look like solar profiles?

Time

[x] Is time now in Europe London time?
[x] Is the hover time in Europe London time?
[x] Are the X axis in Europe London time?

Title

[x] Does the GSP name show up?
[x] Is the latest forecast figure visible?
[x] Is the next forecast figure visible? [not currently shown for space reasons]
[x] Is the current GSP delta figure visible?
[x] Are the above figures in MW?
[x] Is the Delta block background colour correct, i.e. pos/neg delta colour scheme?

Map

[x] Does a map of the UK show up?
[x] Are the GSP boundaries displayed?
[x] Where Deltas are available, do the correct colours for the GSP delta buckets display and match the table?

Combined views

[x] Does selectedGSP persist when switching between PV Forecast and Delta Views
[x] Does selectedTime persist when switching between PV Forecast and Delta Views
[x] Does map location/zoom persist when switching between PV Forecast and Delta Views

Dashboard Mode

[x] Does the Dashboard Mode toggle show in the Profile Dropdown menu?
[x] Does the Dashboard Mode toggle persist across refreshes/logouts using cookie?

National

[x] Do the National title and forecast values show in large font?
[x] Are the lines on the chart thicker?

GSP

[x] Do the GSP title and forecast values show in large font?
[x] Are the lines on the chart thicker?

Legend

[x] Is the legend larger?
[x] Do the legend items space nicely?

Map

[x] Is the map legend larger?
[x] Are the MW/%/Capacity buttons larger?
[x] Are the date/time larger?

General

[x] is the version visible?
[ ] has the version been bumped?
[x] Does the feedback button work?
[x] Is the legend visible
[ ] Is the database stable, check on AWS

Auth

[x] Can I log on with Auth?
[x] Can i log out?

Refresh

[x] After 10 mins, does the forecast update?
[ ] After 10 mins, does the forecast creation time update? - No

Documentation

[ ] Update documentation - https://openclimatefix.notion.site/Quartz-Solar-Documentation-0d718915650e4f098470d695aa3494bf
[ ] Do we need to email the users - No

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

Looks great @braddf, my only real question is should we do the UAT? Or is that all covered in testing now (I dont think so)

Yep @peterdudfield this was just swept through by the gracious @rachel-labri-tipton while I was on lunch, looking good and have just bumped the version for release if it deploys fine 👌 and testing not quite there yet to displace the UAT, but hopefully soon a lot of it will be defunct!


Budgie Extras: init at 1.8.0

Description of changes

As recommended by upstream, every plugin is packaged separately. This does not include the plugins that are actually separate apps.
This is still a WIP, Budgie Extras is a complex monster (see the FIXMEs).
Some plugins need the budgie-extras-daemon to run in the background.
This PR should close: #250384 and #282055
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Some of the plugins require the Budgie Extras Daemon for shortcuts, any idea on how to tackle that?

:shrug: I don't really think budgie-extras-daemon should be a "budgie plugin" as it does not actually seem to be shipping any plugins. I did saw it shipping autostart files though. (And I also saw it shipping layouts files, don't know if we are handling it right now)


Feature Request: Change calendar of Task / Move Task between calendar

Hi,
I really love this android app, but I really miss the possibility to move a task from one calendar to another. So it would be really cool, if I could change the calendar of a task after it's been created.
Thanks!

duplicate of #18, #46, #181 and #198.

This feature has been implemented and released in 1.1.8.
You can change the list of a task in the editor by hitting the list field on top.

I also could not find this without checking here first. I think people expect that this is a function you would get by longpressing on a list of tasks and finding the option: move to another task list (calendar)

We've considered to implement this by drag'n'drop on the list view and we certainly will implement it like that eventually (after we've refactored the list views).
We're trying to move away from long presses and context menu as much as possible.


expo-router Support

Description
https://docs.expo.dev/router/installation/#quick-start
https://github.com/expo/expo/tree/main/packages/expo-router
right now we support @react-navigation/native and react-native-navigation but not yet expo-router which is getting more popular day by day
https://results.stateofreactnative.com/navigation/

https://expo.github.io/router/docs/migration/react-navigation/screen-tracking/ should give a head start.

FYI I just pulled posthog into my expo-router react native app and it captured navigation/screen events without me needing to do anything special. I think because it is built on top of react-navigation 🤷 ?

Most likely but there are a few limitations as pointed out here.

Oh no, I just ran into this. Yes it appears everything related to captureTouches is missing. I see some captureScreens events but not sure how complete it is.
Is this being worked on?

@AdamDorwart not at the moment, would you like to take a look and open a draft PR? happy to help out/review.

It was always a SyntheticBaseEvent. These are challenging to share in a readable way so here's screenshot

I'm not familiar with e._targetInst but when I debug React components like this I look for e.target.__reactFiber$* which is there and has memoizedProps, and elementType.

_targetInst  isn't a public property, https://github.com/facebook/react-native/blob/dceda565bdfe24275be0ad5caae7f3d8252aef92/packages/react-native/Libraries/Renderer/implementations/ReactFabric-prod.js#L104

With expo-router many of my screens are named index.ts. Which causes a problem with autocapture since it uses the filename to define the Screen Name. So I get a lot of dupes in the PH logs. Would be great if i could change how PH autocapture generates the Screen Name instead of default using the filename.

@kwong70 you can try https://expo.github.io/router/docs/migration/react-navigation/screen-tracking/
just call posthog.screen(...) manually, does it get any better?

Hey, so for our app using expo-router for native & web, we're actually forgetting about $pageview and instead using screen.
Thought I'd share.

export const usePostHogNavigationTracker = () => {
  const posthog = usePostHog() || posthogClient
  const pathname = usePathname()
  const params = useGlobalSearchParams()

  let pathAndParams = pathname
  try {
    const strippedParams = { ...params }
    // remove props we don't care about. (screen is redundant, params is not parsable and useless)
    delete strippedParams['screen']
    delete strippedParams['params']

    pathAndParams = pathname + objectToQueryParams(strippedParams)
  } catch (e) {
    l.warn('Could not parse parameters', params, e)
  }

  useEffect(() => {
    posthog?.screen(pathAndParams)
  }, [posthog, pathAndParams])
}

This was at the beginning of the project.
The above implementation has downsides:

The urls have dynamic content => useful, but not as easy to aggregate and to identify 1 page
We often have 2 urls for what could be called a same page (1 for mobile, and one for web/wide screen)

We're going to revise our implementation as we're actually interested in all 3 data:

static route name (ie. /(main)/todos/[todoUuid]/edit)
dynamic route name (/todos/my-todo-uuid-1/edit)
page name: Todo Edit Name (ie. matches /(main)/todos/[todoUuid]/edit and /(main)/m/todos/[todoUuid]/edit)

I installed posthog react-native on my expo-router app. The Activity seems to work, it even fires off screen events.

The only thing is that it doesn't actually track the user in the analytics section of PostHog. Would firing a custom .screen() fix this?

I installed posthog react-native on my expo-router app. The Activity seems to work, it even fires off screen events.
The only thing is that it doesn't actually track the user in the analytics section of PostHog. Would firing a custom .screen() fix this?

The screen events seems to be working based on your screenshot, if you mean user clicks etc, feel free to capture custom events using the capture method.

Hello, i want to add here, i can capture screen events like this in my expo app:
const pathname = usePathname();
const params = useGlobalSearchParams();

useEffect(() => {
  posthog.screen(pathname, {
    ...params,
  });
}, [pathname, params]);

Now, i want to capture screen touches like where users press the most but i think i cannot do it. I only put custom captures on buttons but it would be a nice feature to have.


chore: add initial parse event tests + update snapshot interface

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


reserved identifier violation

I would like to point out that identifiers like “_HOBPP_FIRST” and “_HSTORE_STATE_UNBLOCKED” do not fit to the expected naming convention of the C++ language standard.
Would you like to adjust your selection for unique names?

What would you suggest?

I suggest to delete misplaced underscores in affected identifiers.

Fair enough, pretty simple change.  I will link back with a PR to resolve this and keep the issue open until that's done.  Thanks.

Renaming applied in this PR: https://github.com/Morgan-Stanley/hobbes/pull/38

Were a few update candidates left over?


Choose the ability to return to the headset or lock the FPFC Camera

@Mawntee

Implemented in https://github.com/Auros/SiraUtil/commit/c589c75c72e84b066921f386e2ad731f9a72b6d6


Buttongroups set first button checked

Add class active

And how can i remove the default value?
Whe i create a button groups the first button is always checked

@RandyOrton91 Thank you, this is a bug. I fix it today or tomorrow.

@RandyOrton91 Fixed in 4.0.7. Check it, please.

It seem's not to be fixed, I downloaded the ultimate version from github but nothing changed

Clear your browser cache

I already did it... I really don't know why they're still remaining checked

Re-read docs, also changed.

I noticed that also in the buttons documentation the last example (that with data-targets="..." attribute) still have the first button checked

the data-target attribute is needed to enumerate all the tags that are used to create the buttons

Sorry but probably I'm not being clear, I need to create a buttongroups with all the buttons unchecked

In your codepen if i delete Active attribute from the first button it will remain checked

It possible with mode multi

Yeah I know, but It'isnt possible to do it with single selection? It was possible in metro 3, I did it

Ok, I add this feature tomorrow

Awesome, thank you!

Added


Access to on premises resources via a data gateway DOES NOT come with Office 365 Seeded

Great content and it provides a solid amount of clarity that I find myself having to explain frequently. That being said - Under Seeded Power Apps, it states: "Access to on premises resources via a data gateway" which is true for Dynamics seeded, but (to my understanding) NOT Office 365 Seeded. Additionally, the bullet is clarified as only being included with D365 under the "Seeded Power Automate" section.
Am I missing a rather crucial point or can we make a correction to that bullet?
Thanks!

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

ID: c843089f-7829-b8b4-3fcd-ef52bdc4f764
Version Independent ID: a44c5d15-81cc-9d34-dd2e-8361f0852034
Content: Understanding Power Platform Licensing
Content Source: Community/powerplatformlicensingforcitizendeveloper.md
GitHub Login: @bigpix2000

We should make the change. Likely the older version of the license doc since corrected had it that way but confirming that the current doc is as you say. Easy to fix. Adding to the list to fix ASAP thanks for spotting it!
Get Outlook for iOShttps://aka.ms/o0ukef

From: TheChaunz<EMAIL_ADDRESS>Sent: Wednesday, July 8, 2020 6:42:12 PM
To: MicrosoftDocs/microsoft-365-community<EMAIL_ADDRESS>Cc: Ralph Rivas<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [MicrosoftDocs/microsoft-365-community] Access to on premises resources via a data gateway DOES NOT come with Office 365 Seeded (#220)
Great content and it provides a solid amount of clarity that I find myself having to explain frequently. That being said - Under Seeded Power Apps, it states: "Access to on premises resources via a data gateway" which is true for Dynamics seeded, but (to my understanding) NOT Office 365 Seeded. Additionally, the bullet is clarified as only being included with D365 under the "Seeded Power Automate" section.
Am I missing a rather crucial point or can we make a correction to that bullet?
Thanks!

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

ID: c843089f-7829-b8b4-3fcd-ef52bdc4f764
Version Independent ID: a44c5d15-81cc-9d34-dd2e-8361f0852034
Content: Understanding Power Platform Licensinghttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fmicrosoft-365%2Fcommunity%2Fpowerplatformlicensingforcitizendeveloper%23power-apps&data=02|01||01fe444df1784d6bebc808d8239889b0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637298485327357343&sdata=tSHT3CQ7CknQwvHuwHWgo6a2e2Y3l8zXzrf8yRAg2zw%3D&reserved=0
Content Source: Community/powerplatformlicensingforcitizendeveloper.mdhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-community%2Fblob%2Flive%2FCommunity%2Fpowerplatformlicensingforcitizendeveloper.md&data=02|01||01fe444df1784d6bebc808d8239889b0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637298485327357343&sdata=XnsfMosNkkPL0SOCo%2B7Kl%2FSVR6K%2BfgOdxp7N1Bh1sh4%3D&reserved=0
GitHub Login: @bigpix2000https://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbigpix2000&data=02|01||01fe444df1784d6bebc808d8239889b0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637298485327367337&sdata=h0A20n%2FqQxLGXokqBSYawkez2d3esVFwtqBhhDOd5eM%3D&reserved=0

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-community%2Fissues%2F220&data=02|01||01fe444df1784d6bebc808d8239889b0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637298485327367337&sdata=ROkdKqT1LXxgnJzL%2FyhnlM%2FdYj6MS56hsIWiTjkiTXk%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB2OOTUBLCFWZFEARXKNMMDR2T75JANCNFSM4OVBSO5A&data=02|01||01fe444df1784d6bebc808d8239889b0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637298485327377330&sdata=esdt4zQ7dLTgxISl5KYJIa%2FGcYg%2FK3hCPfl0X350thg%3D&reserved=0.


Proposal: Test coverage report for the project

Currently the project has no test coverage report, this can be a nice to have to maintain and improve the CSV gem.
Pros:

We can actively monitor what needs to be done on the current tests and be prepared also for the new contributions.
Analyze new areas of improvements (on test and the code)

Cons:

None. Not technical at least.

Plan (if accepted):

[ ] Add SimpleCov gem as development dependency and configure the unit test according.
[ ] Optionally, setup a service to have the info visible(as CodeClimate)

OK.

@hsbt Can you enable this repository on Code Climate or Codecov? (I don't have any opinion for code coverage report service.)

The next step should be just a simple PR with all the changes, we can use Github's suggestions to change the CodeClimate Test ID and the URL for the Badge.


VC Leader Dashboard - find a way to incorporate children count correctly.

[x] View most common requests by age range of child(ren)


Generated by :no_entry_sign: Danger


Unusable with new version of rust

The current version doesn't work with the current rust version

https://github.com/zslayton/cron/pull/27
Its fixed, just clone from github until its been updated in crates.io

I need to publish a new version to crates.io. I'll try to do this ASAP.

Version 0.4.0 is available on crates.io.


PlanToTSR fails with attribute error: self._TsrSampler

Currently the PlanToTSR function fails because there is no self._TsrSampler defined. I think there may have been a partial merge somewhere. Or do we need to define this method when we create the planner?

It was defined in constraints, so it should have been constraints._TsrSampler. There were a couple of other things that I just redid not realizing I was redoing the work in the branch bugfix/tsr. @psigen was there a reason that branch hasn't been merged in yet?

Yes, it's because after implementing the projection the optimization failed all the time in local collision minima, which is why the PlanToTSR function isn't a @PlanningMethod in master.  @siddhss5 was working on a better way of projecting using nlopt.
@rachelholladay: you're welcome to try to continue the branch bugfix/tsr, I'm planning to come back to it at some point in the next month but I don't know when.  It's possible there was a bug in my projection implementation.

Coolio. I already opened a separate issue, #5, for @siddhss5 to fix it, since he said he would :). So between that and the fixes in bugfix/tsr hopefully PlanToTSR will be up soon.


Feature: typescript 2 support

Not sure how ready is TS 2, but I see that angular-cli is already using TS 2. It would be nice to get this repo use TS 2 as well.

TS2 will be technically released within a week (there was one release but was bogus).
I will update then.


PWM write() accepts values bigger than "1" as parameter

Hello,
While doing a test on mraa v0.5.4-118-g818e1a8  I noticed in Python that for PWM class, the write() method accepts values for duty cycle bigger than 1, which physically is not correct. It returns no error though and considered the value to be a duty cycle of 1 (like a logic 1)
It would be nice to have a small message that when such a value is introduced something like "Warning: duty cycle value should be between 0 and 1; considering 1 instead of %f" to be displayed.
tested on Galileo v1

import mraa as m
pwm=Pwm(3)
pwm.period_us(1000) # 1kHz frequency
pwm.enable(True)
pwm.write(5)
0

Regards,
Costin

This was done on purpose, a warn message to syslog could be added.


Azure Storage ADLS gen2

Hi,
I'm experiencing a behaviour that wasn't present before. Ever since we've change the configuration of the Storage to use ADLS Gen2. Using the AppendBlobClient to append to existing blob storage. The UI displays a size 0KB next to the file. Perhaps when we do click in it to check the file is not empty right after, the size is not updated to the appropriate value.
Any clue ? if we're doing something wrong ?
Here's a snippet of the code
private void writeToBlob(Object item) throws Exception {
    String line = writer.writeValueAsString(item);
    try (ByteArrayInputStream dataStream = new ByteArrayInputStream(line.getBytes())) {
        client.appendBlock(dataStream, dataStream.available());
        log.debug("Appended to block the data -> {}", line);
    } catch (IOException e) {
        throw new RuntimeException(e);
    }
}

Hi, @jselamy. Thank you for asking this question.
I see you're making a call to append block. Since you're trying to use ADLS Gen 2, I wonder if you might be confusing the AppendBlobClient with the DataLakeFileClient. The latter also has an append method that is actually for use with ADLS Gen 2. Then any appends must be followed by a flush. Is there a reason why you're using the AppendBlobClient specifically?

please let us know if you need further assistance.


[Android 9.0][honor Mate20] Crash on android9.0 device

2019-12-26 10:02:00.718 23085-23114/com.xxxx E/linker: library "/system/lib/libc++_shared.so" ("/system/lib/libc++_shared.so") needed or dlopened by "/system/lib/libnativeloader.so" is not accessible for the namespace: [name="classloader-namespace", ld_library_paths="", default_library_paths="/data/app/com.xxxxx-f4jNTNt3ti2sNnL5wOwljw==/lib/arm:/data/appxxxx-f4jNTNt3ti2sNnL5wOwljw==/base.apk!/lib/armeabi-v7a", permitted_paths="/data:/mnt/expand:/data/data/com.xxxx"]
2019-12-26 10:02:00.722 23085-23114/com.xxxx E/weex: commitCriticalExceptionRT :WX_KEY_EXCEPTION_SDK_INITexceptionload c++_shared failed Detail Error is: dlopen failed: library "/system/lib/libc++_shared.so" needed or dlopened by "/system/lib/libnativeloader.so" is not accessible for the namespace "classloader-namespace"
2019-12-26 10:02:00.724 23085-23114/com.xxxxx E/SafeRunnable: SafeRunnable run throw expection:dlopen failed: library "/system/lib/libc++_shared.so" needed or dlopened by "/system/lib/libnativeloader.so" is not accessible for the namespace "classloader-namespace"
2019-12-26 10:02:00.729 23085-23114/com.xxxxx E/AndroidRuntime: FATAL EXCEPTION: WeexJSBridgeThread
    Process: com.ifaex_android, PID: 23085
    java.lang.UnsatisfiedLinkError: dlopen failed: library "/system/lib/libc++_shared.so" needed or dlopened by "/system/lib/libnativeloader.so" is not accessible for the namespace "classloader-namespace"
        at java.lang.Runtime.loadLibrary0(Runtime.java:1016)
        at java.lang.System.loadLibrary(System.java:1672)
        at org.apache.weex.utils.WXSoInstallMgrSdk.initSo(WXSoInstallMgrSdk.java:130)
        at org.apache.weex.WXSDKEngine$1.run(WXSDKEngine.java:224)
        at org.apache.weex.common.WXThread$SafeRunnable.run(WXThread.java:70)
        at android.os.Handler.handleCallback(Handler.java:907)
        at android.os.Handler.dispatchMessage(Handler.java:105)
        at android.os.Looper.loop(Looper.java:216)
        at android.os.HandlerThread.run(HandlerThread.java:65)

Weex has not support AndroidX.
Does it support Android 9.0/10.0?
Thanks

I have tested it for Android 9 (API 28), everything works fine.
If you met this problem on Android 9 with Weex 0.28, I think this is caused by the Per-app SELinux domains in Android 28.
You could remove libc++_shared.so provided by Weex.aar and add your own. Refer this exmaple.

I have tested it for Android 9 (API 28), everything works fine.
If you met this problem on Android 9 with Weex 0.28, I think this is caused by the Per-app SELinux domains in Android 28.
You could remove libc++_shared.so provided by weex.aar and add your own. Refer this exmaple.

Solved this issue by modify abiFilters in Application:
// before  abiFilters "armeabi-v7a"
//after
 abiFilters "armeabi-v7a", 'arm64-v8a'

Some different between V0.18 and V0.28.
I should declare a list of abiFilters in  V0.28，rather then single “armeabi” in V0.18

Yes. arm64-v8a is provided accroding to Goolge Play's rule.


add param to paginate

In case when we need to paginate we will do count query, when i need to select all column the count will use all column to, in my experience its not effective because in count we can just use 1 column
So i add column_count param in paginate, the param use to select which column that will be used in count query

Can you add tests ?

Can you add tests ?

i already move my new param to last param in next commit
for test, i will do it later


Add our purpose section on the landing page

Description
Add our purpose section.
Mocks
It should be below parallax image and use appropriate colors.
Definition of Done

[x] All of the required items are completed.
[x] Approval by 1 mentor.

@codetheorem assign me this


✨ ERC4337 Simple Account

Description
ERC4337 Simple Account.
Design:

Recycle Ownable to double as a initialization guard.
Not super-inlined, intentional for brevity and readability. This standard is already quite complex.
Includes a delegateExecute(address delegate, bytes calldata data) function for on-the-fly plugins.

Currently omitted functions:

getNonce()
getNonce(uint192 key)
incrementNonce(uint192 key)
supportsInterface(bytes4) (do people actually use this on-chain?)

Checklist
Ensure you completed all of the steps below before submitting your pull request:

[x] Ran forge fmt?
[x] Ran forge snapshot?
[x] Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

supportsInterface(bytes4) (do people actually use this on-chain?)

Re: ERC165 introspection support in 4337 account itself, it's not required by safe token transfer receivers (ERC721, ERC1155), ERC4337 or ERC1271, so let's not bother with adding. Similarly, I haven't ever seen a contract check for this anyways.

Sidenote: Supporting ERC1271 isValidSignature() method makes sense here. Accounts should be able to spend the signature of owner. On that front, would it make sense to put this method into own file, like Receiver.sol, or is it so opinionated to ownership that it should be framed into account itself. If Solady perspective is that Ownable == Single Owner Auth, then I think separate file would be most useful for library purposes.


Allow to install stable versions by version number

If one wants to use rustup to install old stable releases for some reason, they need to find the right date to use. For instance rustup toolchain install stable-2017-06-08 for 1.18. This is tedious. It would be easier if it were possible to say rustup toolchain install stable-1.18.0

rustup toolchain install 1.18.0

facepalm
Thanks.


[Fix] move error messages to meta property

This moves all error messages for every rule to the built-in meta.messages property to support easier rule writing in the future.
Resolves #2915

I’ve fixed the test failures on master and rebase this; the current failures are due to this PR.

@ljharb Took a few extra commits to fix the test cases that weren't caught with local unit tests. Should be good to go after the rebase issue.

@ljharb Could you provide some insight as to why the automatic rebase action is failing here? The logs say there's some conflicts but I'm not actually seeing anything.

I don't see a conflict, and I'm not sure, but don't worry about it - i can merge through that if needed.

Somehow it's finding 487 merge commits, and there's zero, so that's clearly a bug in the action :-)

I've rebased it manually, which should resolve the issue.

Turns out this further broke eslint < v4.15, since v4.15 added messageId support, but we weren't testing on eslint 3 so we didn't catch it. #2562 will restore support for it.


Route annotation support for vertx-web

In response to the message on the mailing list, "What would you like to see in Vert.x":
The most potentially useful thing missing from vertx-web right now is convenience annotation support for mapping handler classes to URL routes, etc.
Vert.x Nubes has good support for this, and the code looks decent. (It is already linked on the vertx-awesome page, so I'm sure the core Vert.x developers are very aware of it.)
I only became aware of Vert.x Nubes after implementing my own, similar framework, "Gribbit" (the repo does not yet have documentation, but the code is heavily commented; the code is roughly in a pre-beta state, and I'm providing the link more as a reference than as a suggestion to try to clone and use the code for now!)
Proposal: I propose either bringing Nubes into vertx-web, if the author of Nubes approves (and maybe removing the "Nubes" name from packages and classes!), or simply taking ideas from one or both these frameworks (Nubes and Gribbit), and creating a new, simple annotation framework from scratch in vertx-web.
Gribbit has features that overlap with Vert.x Nubes (such as being able to mark methods with annotations to handle GET and POST methods). Nubes has a number of features Gribbit doesn't have, such as supporting injection of Service classes. Gribbit has a number of features Nubes doesn't have, such as the ability to perform reverse routing (to enable a route URL to be inserted into a template based on a handler class reference), and a typesafe template engine (which is out of scope for this current annotation detection proposal).
N.B. To accomplish runtime annotation detection in vertx-web, you may find a library I created useful, FastClasspathScanner, which allows you to quickly and robustly find class and method annotations, as well as classes implementing a given interface or subclassing a given superclass. I originally implemented FastClasspathScanner because I wanted to add annotation support for mapping routes in Gribbit. FastClasspathScanner became a separate project, and has taken on a life of its own -- it is actively maintained, and at this point it is stable and highly optimized. (Nubes currently uses Reflections for its annotation support, which is generally slower and buggier than FastClasspathScanner, and supports fewer classpath specification mechanisms and fewer classloader types.)

Came across this https://github.com/zandero/rest.vertx. It Provides JAX-RS annotation processor for vert.x. Its currently in beta and has good documentation.

@udaykale that looks pretty good, thanks for the link.


releaseing 0.5.1

Coverage remained the same at 100.0% when pulling 52c76e0aecc63b566908ba6d6f6872f7edadfd7b on yuce:releasing-0.5.1 into fab3b95abb68069b7626e7124b002d7983f0899f on pilosa:master.


QuizListComponent

Change:
 import { HttpClient } from '@angular/common/http';
to
 import { HttpClientModule } from '@angular/common/http';

Thanks for the help, the solution wasn't what you tell me. Anyway, thanks a lot.
Already solved. The error was in the book.


delimiter option doesn't work

Hi,
Delimiter option doesn't work. The delimiters are not displayed in the TextField.
Could you fix it please.
Thanks

Thank you for the report. I'll take a look at it. Should be an easy fix.

Released in v1.1


Update euclid.

This change is

r? @jdm or anyone else?

The travis failure looks like a missing dependency in the Linux builders.

@bors-servo r+

:pushpin: Commit e6e7505 has been approved by jdm

:hourglass: Testing commit e6e7505626ea56b5b069e335081ebc1e67aebb9f with merge dae2f19dc8a177eb46e13cb166e65eb229c4062c...

:sunny: Test successful - checks-travis
Approved by: jdm
Pushing dae2f19dc8a177eb46e13cb166e65eb229c4062c to master...


test: chan/select3.go build failure on linux-386-387

https://build.golang.org/log/5eddfcb8a4f68a2d3e1b2cbaddcacdc83d37ce5f
One and only failure; occurred first after CL 41820 (enabling concurrent compilation). Investigating.

##### ../test
# go run run.go -- chan/select3.go
exit status 1
panic: function did block

goroutine 1 [running]:
main.testBlock(0x80a9dd4, 0xc, 0x80ae3c8)
	/tmp/workdir/go/test/chan/select3.go:50 +0x140
main.main()
	/tmp/workdir/go/test/chan/select3.go:153 +0x34b
exit status 2

FAIL	chan/select3.go	1.843s

I've been running select3.go in a loop on a gomote for the last two hours. No failures. Reading the code for this particular test, I notice that it depends on timing:
func testBlock(signal string, f func()) {
	c := make(chan string)
	go func() {
		f()
		c <- never // f didn't block
	}()
	go func() {
		time.Sleep(1e8) // 0.1s seems plenty long
		c <- always     // f blocked always
	}()
	if <-c != signal {
		panic(signal + " block")
	}
}

Maybe due to other noise on the machine, 0.1s wasn't plenty long enough. Maybe concurrent compilation, which generally uses CPU more intensely, increased the noise and contributed.
Out of ideas, and can't reproduce. Going to wait to see whether it occurs again.

It is possible that #20250 isn't just a pessimization but is the source of a real problem. It does affect package cmd/link/internal/ld.

I've been running this in a loop again and still can't get a failure. I'm hesitant to bump up the blocking time, since the test takes 10*sleeptime to run, and we've only seen a single failure, ever.
I'm going to close this as not worth more effort; if it happens again, we can re-open and bump up the timeout and/or investigate.

It happened again: https://storage.googleapis.com/go-build-log/0a0e8fbd/linux-386_56a2d1b0.log .

I think the most likely culprit here is CPU availability / scheduling. We could refactor the test so that all "should block" cases happen concurrently and bump up the timeout. Objections to that?

No objections. You have time to do that, @josharian?

I'll try to make time, but I'm at a family reunion / on vacation for the next two weeks. If it is important or time critical, someone else should step in.


feat(core): do not include DNS query time in HTTP response time

Fix #49

Summary

See #49

Fixing response time for other request types needs to be added in a newer issue
I don't know how to add a test for this, it would need some kind of mock for getting the time to be able to override it in the test.

Checklist

[ ] Tested and/or added tests to validate that the changes work as intended, if applicable.
[ ] Updated documentation in README.md, if applicable.

I know the issue's been open for a while, but I'm perplexed about whether this is the right call.
In a real-life production scenario, a DNS call does occur, so is excluding the DNS call from Gatus really the right move? 🤔

I can probably make this an optional feature (disabled by default) for each HTTP endpoint. But then maybe it makes more sense to add the feature for each kind of request.

@TwiN What do you think about creating new placeholders specifically for the values for DNS query time and protocol response time, and leaving the [RESPONSE_TIME] placeholder untouched?
My patch has to be improved in any case because it is not clear if the DNSDone event can be triggered multiple times (for example if the first matching address is down?).

@TwiN I'm thinking about working on this PR again. But I'm not quite sure about what should be implemented.
Let alone the naming, it could be possible to add new placeholders such as DNS_RESPONSE_TIME (and others possible with httptracing - to be implemented also on other protocols), but I think that computing RESPONSE_TIME - DNS_RESPONSE_TIME in a new placeholder would not make sense.
What do you think about adding an element of language to be able to add operations? For example a function subtract:
subtract(RESPONSE_TIME, DNS_RESPONSE_TIME) < 2s

I'm not sure if it should support nested function calls, e.g. subtract(subtract(RESPONSE_TIME, DNS_RESPONSE_TIME), FIRST_BYTE_TIME), as it would increase the complexity of the condition parser. Maybe for a future version.
Adding a function add would make sense along with subtract.

I really like your idea of adding the add and subtract functions, if you're up for the challenge. exp, however, may be a bit too difficult to implement. That said, conditions is something I'd like to improve as well, since it is currently somewhat lacking. For instance, supporting [STATUS] == 200 || [STATUS] == 429 would be nice, as opposed to how I currently implemented ([STATUS] == any(200, 429)), but it would require a significant amount of work, which is why perhaps going the add and substract route may be easier.


Add a Signal.Destroy method

It would be nice to have an alternative mapping to Signal.disconnect - namely, Signal.Destroy.
This would make it play nicer with the popular Maid class by Quenty.

disconnect lines up with the RBXScriptConnection API, roughly -- camelCase instead of PascalCase.
Maids can already handle disconnecting events, so it should be a matter of also supporting this casing. Alternatively, I believe it's possible to give Quenty's Maid class a function to call, which makes wrapping any existing libraries very easy.


Hard to collect double valued percentages...

I am using kamon-jmx-collector to collect statistics from ehCache. However some statistics like CacheHitPercentage are stored in JMX as Double (i.e from 0.0 to 1.0). As far as I can understand, this library converts the Double to Long and I lose the precision of the percentage...
How do you think this can be improved? A user defined function to scale values?

Hi there,
thanks for the feedback. This sounds like a very important feature request.
I think the code needs to be updated in 2 places:

https://github.com/Segence/kamon-jmx-collector/blob/master/src/main/scala/kamon/jmx/collector/MetricCollector.scala#L138 to include Double type
https://github.com/Segence/kamon-jmx-collector/blob/master/src/main/scala/kamon/jmx/collector/SupportedKamonMetricTypes.scala the recordValue methods need to be overloaded so not only Long values are accepted.

I could look at these but probably in a bit of time. Well let me know if you want to give it a go otherwise I'll let you know when I made some progress with it.

Hi!
I think that those changes are not enough because kamon metrics only support Long:
https://github.com/kamon-io/Kamon/blob/794fbf02664ac8c31072d8b955d897901f1f22e0/kamon-core/src/main/scala/kamon/metric/Gauge.scala#L20-L28
https://github.com/kamon-io/Kamon/blob/794fbf02664ac8c31072d8b955d897901f1f22e0/kamon-core/src/main/scala/kamon/metric/Histogram.scala#L25-L31
so the only thing we can do is specify the MeasurementUnit. We can't escape the Double to Long conversion.
As soon as we find a path forward I can try to implement it...

Hi there,
a user defined function sounds like a good idea.
Have you had a chance to think about the implementation?
I might be able to have a look at this soon.

Hi there,
Never got the chance to give this a try... Please go ahead...


Get Exchange time

I want to get KLines as described here in Wiki.

It requires start time and end time, how do you define them? If I know current exchange time, I can put for example:
current_exchange_time = get_exchange_current_time()

start_time = current_exchange_time - 86400
end_time = current_exchange_time
interval = 86400 / 24  # one hour

And then I can get KLines with this python wrapper:
from ViaBTCAPI.ViaBTCAPI import ViaBTCAPI
api = ViaBTCAPI("https://localhost:8080")  # the URL to accesshttp ViaBTC service

available_markets = api.market_list()["result"]

market = available_markets[0]
kline = api.market_kline(market, start_time, end_time, interval)
print(market, kline["result"])

So can you please give me an advice how to call market.kline? Thanks.

@LinkChain Can you please share the arguments that you have used?


Items not playing.

Hey.
I installed this one last night and tried it out.
Nothing is playing though.
There's the "loading" animation and then nothing happens.
Are there any known issues and fixes for this?
Am I missing any deps?
The following is from the log file:

12:19:24.743 T:6012   ERROR: Previous line repeats 6 times.
12:19:24.743 T:6012  NOTICE: VideoPlayer: Opening: https://level3-usp-dash.cmore.se/catchup/ISMROOT/CFILM/2017-09-07/STEVEJOBS(3925954_ISM).ism/STEVEJOBS(3925954_ISM).mpd
12:19:24.744 T:6012 WARNING: CDVDMessageQueue(player)::Put MSGQ_NOT_INITIALIZED
12:19:24.769 T:6012   ERROR: DXVA::CProcessorHD::IsFormatSupported: Unsupported format 105 for 1.
12:19:24.769 T:13108  NOTICE: Creating InputStream
12:19:24.901 T:13108   ERROR: AddOnLog: InputStream Adaptive: Could not open / parse mpdURL (https://level3-usp-dash.cmore.se/catchup/ISMROOT/CFILM/2017-09-07/STEVEJOBS(3925954_ISM).ism/STEVEJOBS(3925954_ISM).mpd)
12:19:24.901 T:13108   ERROR: CVideoPlayer::OpenInputStream - error opening [https://level3-usp-dash.cmore.se/catchup/ISMROOT/CFILM/2017-09-07/STEVEJOBS(3925954_ISM).ism/STEVEJOBS(3925954_ISM).mpd]
12:19:24.901 T:13108  NOTICE: CVideoPlayer::OnExit()
12:19:24.902 T:6012   ERROR: Playlist Player: skipping unplayable item: 0, path [plugin://plugin.video.cmore/?action=play&video_id=3925954]
12:19:24.903 T:6012  NOTICE: CVideoPlayer::CloseFile()

Using 17.4
Edit:
I can play some things now, but I can't play the stuff I am paying Cmore for (Movies, live sports etc), through KODI that is. I have entered my creds correctly.
Edit2: I got it to work with the latest nightly build (v18 2017-Sep-18 04:02), but it's way too unstable.
Got the winevinecdm.dll but still not working on 17.4.
What am I missing? And where do I get it?

Hi! Please post a full, non cut-off debug log.

The log can be found here:
http://paste.ubuntu.com/25564316/

@Berringer It doesn't look like you have enabled debug logging in Kodi. http://kodi.wiki/view/Log_file/Advanced#GUI_settings

My guess is that you're using a 64-bit version of the Widevine CDM. Download https://dl.google.com/widevine-cdm/<IP_ADDRESS>8-win-ia32.zip and unzip widevinecdm.dll to %APPDATA%\Kodi\cdm.
Make sure to read LICENSE.txt as well so you know what you're getting yourself into.

@emilsvennesson Yes, thank you, that sorted it, now it works!
Great job with this, keep it up!
And I did read the LICENSE.txt, thanks for pointing that out aswell!


[Docs] Add an example of a custom webpack configuration

Hi! Thank you for maintaining this package.
Since CRA is a little bit deprecated (see https://github.com/reactjs/react.dev/pull/5487#issuecomment-1409720741, https://github.com/reactjs/react.dev/pull/5487), it would be great to add an example of a custom webpack plugin.
I think we can take the key points from this issue, I faced all of them when was implementing the config - https://github.com/keycloakify/keycloakify/issues/47

btw, I've built it without CRA, so I think I can contribute.

Agreed,
Do you suggest we eject the Keycloakify starter? 🤔
Or maybe we should just focus our efforts on supporting vite. I think I can get it done by september.
Is there that many pepoles still using webpack anyway?
Beside all the project that didn't migrate away from CRA yet I mean.

Ok, good points.
Don't worry I won't deprecate it, it's true that there's a ton of very usefull plugins that are Webpack specific.
Your PR will be much appreciated thanks!

Ok, good points. Don't worry I won't deprecate it, it's true that there's a ton of very usefull plugins that are Webpack specific.
Your PR will be much appreciated thanks!

Could you please check my comment in this issue - https://github.com/keycloakify/keycloakify/issues/47? I'm trying to use the custom config, but unfortunately, there are still some issues.

I can probably fix this

I can probably fix this

Thanks! Any idea why it's not working? Also, do you know of any temporary fixes?

I just released a patch, should be fixed in 7.15.1 let me know if it isn't the case

I just released a patch, should be fixed in 7.15.1 let me know if it isn't the case

It solves the issue with missing process.env, but URLs are still invalid (see the related issue) - GET http://localhost:8080/src_login_KcApp_tsx.bundle.js net::ERR_ABORTED 404 (Not Found). I'll create a repro tomorrow morning.
process.env is not the main issue, since it can be hacked via DefinePlugin. But the problem is that URLs to resources are not updated, they are from '/'.
If you have some time, could you please explain why we have process.env in a browser, and how it works in CRA. Maybe it will help me to fix the issue.
Thanks for your hard work, I appreciate it

If you have some time, could you please explain why we have process.env in a browser, and how it works in CRA. Maybe it will help me to fix the issue.

In CRA all environements variables prefixed by REACT_APP_ available when the app is build (npx react-scripts build) are made available through process.env.
process.env is where you read the ENV in node so they made this decision to make the two runtime as isomorphic as possible.
Regarding the PUBLIC_URL env. This one is special.
See, in the package.json of a CRA project there is usually a "homepage" property. Well, when this property isn't defined or when its something like "homepage": "https://xxx.yy" the process.env["PUBLIC_URL"] will be `` but when it's something like homepage": "https://xxx.yy/a/b/c" it will be /a/b/c.
This is to enable the app to be hosted elsewhere than at the root of the domain.
I need to read this value only for the mocks, to know where to find public static assets that are otherwise provided by Keycloak.

In v5  I had a "Supported framwork" section in the doc: https://docs.keycloakify.dev/v/v5/#supported-react-frameworks
Maybe it can be of some use...

Have you though about starting from the starter project and running npx react-scripts eject.
https://create-react-app.dev/docs/available-scripts/#npm-run-eject

Have you though about starting from the starter project and running npx react-scripts eject.
https://create-react-app.dev/docs/available-scripts/#npm-run-eject

I tried, it's very complex and it's really hard to figure out what's going on there

If you have some time, could you please explain why we have process.env in a browser, and how it works in CRA. Maybe it will help me to fix the issue.

In CRA all environements variables prefixed by REACT_APP_ available when the app is build (npx react-scripts build) are made available through process.env. process.env is where you read the ENV in node so they made this decision to make the two runtime as isomorphic as possible.
Regarding the PUBLIC_URL env. This one is special. See, in the package.json of a CRA project there is usually a "homepage" property. Well, when this property isn't defined or when its something like "homepage": "https://xxx.yy" the process.env["PUBLIC_URL"] will be `` but when it's something like homepage": "https://xxx.yy/a/b/c" it will be /a/b/c. This is to enable the app to be hosted elsewhere than at the root of the domain. I need to read this value only for the mocks, to know where to find public static assets that are otherwise provided by Keycloak.

Ok, thank you. But it seems that the problem is not in the PUBLIC_URL since when you run locally, it's on '/'. Do you have any ideas why resources to URLs are not updated using the custom webpack configuration?

I made a reproduction - https://github.com/ddubrava/keycloakify-custom-webpack-repro

I've taken a look at your repository and have pinpointed the source of the issue.
As I previously communicated in this guide, it is indeed true that the build directory should contain an index.html. However, it's also crucial to remember that all static assets, which include *.bundle.js files, must be within a build/static directory, regardless of their specific locations beyond that.
In the current setup of your repository, all files are situated at the root of the build directory. It's essential to tweak your webpack configuration to align with the requirements I detailed in this document.
For a clearer picture, I suggest comparing your build directory with that of the starter repository.
Beside, If your objective is to develop a standalone theme, you might find it beneficial to run this script.
I hope this information assists you in resolving the problem. If you come across any further issues, please do not hesitate to contact me again. Alternatively, you are welcome to join our Discord channel for ongoing support and updates.
Best regards,

I've taken a look at your repository and have pinpointed the source of the issue.
As I previously communicated in this guide, it is indeed true that the build directory should contain an index.html. However, it's also crucial to remember that all static assets, which include *.bundle.js files, must be within a build/static directory, regardless of their specific locations beyond that.
In the current setup of your repository, all files are situated at the root of the build directory. It's essential to tweak your webpack configuration to align with the requirements I detailed in this document.
For a clearer picture, I suggest comparing your build directory with that of the starter repository.
Beside, If your objective is to develop a standalone theme, you might find it beneficial to run this script.
I hope this information assists you in resolving the problem. If you come across any further issues, please do not hesitate to contact me again. Alternatively, you are welcome to join our Discord channel for ongoing support and updates.
Best regards,

Many thanks, I completely forgot to check the build directory

Well, I managed to resolve the 404 errors. You were right; we needed to maintain the same structure. The only downside is that now all the complexity from CRA has shifted to the custom configuration. Once everything is completed, I'll submit a PR along with the necessary documentation changes

Congratulation for pulling it out and sorry it was such a complex process to acheive.
I'm sorry, I'll have offered a patch for not requireing the assets to be within a sub directory of the build dir if it was an easy fix but unfortunately it isn't.
Looking forward for your PR.

Hi, unfortunately, I do not have enough time to submit a PR now. So if someone reads this issue, there are some tips.
Keycloakify requires a specific directory structure (from CRA, when you have static, js, css, etc.), but it relies not only on the structure but also on some approaches from CRA. For example, PUBLIC_URL in index.html. So I would strongly recommend ejecting the CRA config and adding what you need.
If you would like to clean up the CRA config, since it ejects into thousands of lines of code, I also do not recommend it. Everything is closely related, and there is a huge chance to break something.

https://docs.keycloakify.dev/keycloakify-in-my-codebase/in-your-react-project/in-your-webpack-project


Not an editor command: plug#begin('~/.vim/plugged')

I'm trying to install vim-plug. I've copied plug.vim in the ~/.vim/autoload directory.
This is my .vimrc content right now:
" Vim plug
plug#begin('~/.vim/plugged')
Plug 'junegunn/fzf'
plug#end()

When I start vim, I get these error messages printed to the terminal:
Error detected while processing /home/kevin/.vimrc:
line    2:
E492: Not an editor command: plug#begin('~/.vim/plugged')
line    4:
E492: Not an editor command: Plug 'junegunn/fzf'
line    6:
E492: Not an editor command: plug#end()

And any plugin fails to load, of course.
I'm using Ubuntu 15.05.

@turnabout Hello. Please see the example vimrc. The problem is you forgot the keyword call. plug#begin is an autoload function not an editor command. Once the script is sourced, Plug & the associated commands are available.
You may wish to read a more complete explanation at Learn Vimscript The Hard Way

Ah, that was silly of me. Thank you.

@turnabout No problem, have fun with vim. :)


NestJS - Type instantiation is excessively deep and possibly infinite

Describe the bug
The object's type returned from betterAuth() is too slow and takes about 10 seconds to iterate the types in NestJS. if I use any properties of the auth.api property, it gives me the error: Type instantiation is excessively deep and possibly infinite
To Reproduce
Steps to reproduce the behavior:

build a nest project using npx  @nestjs/cli new my-project
make a new .ts file somewhere in the /src directory
import the betterAuth and run simply call it.

Expected behavior
The type-intellisense must be much much much faster than this.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Node Version: 20.11.0
Package Manager: Yarn
Typescript Version (as from the official starter template, v5.1.3)

Additional context
tsconfig.json
{
  "compilerOptions": {
    "module": "commonjs",
    "declaration": true,
    "removeComments": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "allowSyntheticDefaultImports": true,
    "target": "ES2021",
    "sourceMap": true,
    "outDir": "./dist",
    "baseUrl": "./",
    "incremental": true,
    "skipLibCheck": true,
    "strictNullChecks": false,
    "noImplicitAny": false,
    "strictBindCallApply": false,
    "forceConsistentCasingInFileNames": false,
    "noFallthroughCasesInSwitch": false
  }
}

set declaration:false and strict:true
https://www.better-auth.com/docs/concepts/typescript

@Bekacru Thanks. it worked


Add LLVM-C.dll on Windows

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Fixes #212

@isuruf, I'd very much like your opinion here. The first point is whether we should set LLVM_BUILD_LLVM_C_DYLIB=ON on osx as well (where, contrary to windows, it's off by default).
I see you added the =NO on windows in https://github.com/conda-forge/llvmdev-feedstock/commit/8059f3a076bfc1770ab993c22e882fc7279e66a3 (though I don't see relevant comments in the corresponding PR). That commit is also relevant because it introduced another point about this recipe I'm not 100% sure about - currently on windows, libllvm16 & llvm are completely empty packages - I presume because this would be for the shared libraries which we're not building currently.
I see from the build log that Library/bin/LLVM-C.dll is shared though, so I wonder in which output this should go[^1]. The unix version of the library will end up in libllvm16 AFAICT. Another, larger question is then whether we should try again to build the shared libraries on windows and populate libllvm16 also on windows.
[^1]: that said, llvmdev currently contains two DLLs on windows already, LTO.dll & Remarks.dll

PS. Thanks for the PR @metab0t! Sorry if this will still need some hashing out.

Ping @isuruf. Might be good to figure this one out for the 17 cycle?

This should be a separate output. libllvm-c16

@isuruf, this is ready for review, PTAL. I refactored the installation scripts away from:

install then delete unwanted bits

towards

install into temporary prefix and copy specific files

which I think is clearer in intent and less error-prone, see https://github.com/conda-forge/llvmdev-feedstock/pull/213/commits/22fb387d61a1de9a5097d4f570b1bbb405ecaede.
This would be an independent improvement from this PR, but it makes sense in this context, because that way, it is a lot easier to extend for adding a libllvm-c output.

Ping @isuruf if you want to review. Otherwise this follows your request of putting things in a separate output and I'll merge it in a few days.


Panic: index out of bounds when using flatten for going from a 3D -> 1D array

Describe the bug
When I try to flatten a 3D tensor I get a panic message:
panicked at 'index out of bounds: the len is 1 but the index is 1', /Users/mrvollger/Desktop/repos/burn/burn-tensor/src/tensor/api/base.rs:147:9

This might not be a bug, but I think should be a Error message instead of a panic if it is not a bug.
To Reproduce
Here is a test case that reproduces it:
#[cfg(test)]
mod test {
    use burn::tensor::{Shape, Tensor};
    use burn_ndarray::NdArrayBackend;
    type Backend = NdArrayBackend<f32>;

    #[test]
    fn error_example() {
        let tensor = Tensor::<Backend, 3>::ones(Shape::new([1, 5, 15]));
        let flattened_tensor: Tensor<Backend, 1> = tensor.flatten(1, 2);
        let expected_shape = Shape::new([75]);
        assert_eq!(flattened_tensor.shape(), expected_shape);
    }
}

Expected behavior
I think the expected behavior is for the shape of the output to be Shape::new([75]);. Maybe that logic is not correct, but even so avoiding a panic here and having an error would be really helpful!
I ran into this trying to load an onnx model created using pytorch, so I think all the logic is correct in the model since I have used it extensively in pytorch.
Screenshot
This is a screen shot of the onnx model I am trying to load and a link to the onnx model incase it is of interest. But you do not need it to recreate the error, the minimal test case enough is above.

https://github.com/fiberseq/fibertools-rs/raw/onnx_burn/src/model/revio.onnx
Desktop (please complete the following information):

OS: MacOS
Version: main (latest)

This might be a very silly mistake on my part... but I'd appreciate any help!
Thanks,
Mitchell

Maybe this is special case when the non flattened and leading dimension(s) are "1" and are maybe pytorch and onnx expects Shape::new([1, 75]); to be allowed to be "cast" into Shape::new([75]);?

This function should not compile. However, since constant expression evaluation isn't fully stable yet, we cannot automatically determine the rank of the tensor. In this specific case, we flatten dimensions 1 and 2, which results in a tensor of rank 2. Despite this, we essentially 'force' it to behave as if it were rank 1. We should probably add a check to print better error messages.

I will look into this since I have worked on flatten method.

Awesome. Thanks to both of you!
Let me know if there is anything I can share that would be of use!

Thanks @antimora! I really appreciate the fix. I will test the PR against my expectations for the model :)


dbeaver/dbeaver-ee#1993 add grants to inherited objects

Closes https://github.com/dbeaver/dbeaver-ee/issues/1993

Verified


[Cherrypick] Update onebranch Dockerfile for windows agent #2159 to release/0.11

Cherrypick #2159 to release/0.11
Why make this change?

We were getting the error message "no matching manifest for windows/amd64 10.0.20348 in the manifest list entries".
the problem is that the Docker image we're trying to pull (mcr.microsoft.com/dotnet/aspnet:6.0-cbl-mariner2.0) does not support the architecture of our Docker host (Windows/amd64).
The 6.0-cbl-mariner2.0 tag of the dotnet/aspnet image is designed for the CBL-Mariner (Common Base Linux) operating system, which is a Linux distribution developed by Microsoft. This image will not work on a Windows host of our onebranch pipeline.

What is this change?

changed the base images from Linux-based images to Windows-based images.
mcr.microsoft.com/dotnet/aspnet:6.0

How was this tested?

[X] pipeline passing

/azp run


Cu 86b17nztq implement commands

vehicles now obey the traffic light
things I'm not 100% confident about:

  # only accelerate when the vehicle is on a StraightEdge
  elif vehicle.direction_angle % 90 == 0:

this is the logic for making sure vehicles don't accelerate when they are on a CircularEdge

I also changed the value of max_angle_diff to 50

bug as a result of max_angle_diff = 30: when the leading vehicle is turning, by the time the trailing vehicle is about to turn, the angle diff between these two cars is way above 30 degrees. As a result, the trailing vehicle speeds up during the turn thinking its leading vehicle is gone, resulting in a crash.

!!!STILL NEED TO WORK ON LEFT TURNS!!!

Task linked: CU-86b17nztq Implement commands

please look into the 0.01 number! almost good to merge

good name for interval 👍


Downloading and Listening made impossible

Hi, it is I again...
I've got an issue again
Whatever file downloaded is 157 bytes in size, and clicking the listen button does nothing... However, the correct size of the files appears upon clicking on the "down" arrow prompting to choose other formats....
Cheers

Fixed in https://github.com/alashow/datmusic-api/commit/df677fbd7e8e231a8571450d1770b7ffb6839068

Well heck that was fast...


Inner loops

It would be nice if one could replace:
https://pls.watch/#&v=s2u3hHc0Icc&t=8s;29.8s+29.8;30+29.8;30+29.8;30+31s
with:
https://pls.watch/#&v=s2u3hHc0Icc&t=8s;29.8s+3x29.8;30+31s
Well, you could replace it with just: https://pls.watch/#&v=s2u3hHc0Icc but that will not be the same :)
It would also help if for some reason you want to send someone one fragment of a video, but without looping it.

&t=8s;29.8s+3x29.8;30+31s

IMO it is not worth increased complexity, but if someone submits a PR I'll be happy to review.

send someone one fragment of a video, but without looping it.

I think this is a valid feature request. Mind filling separate issue for this?


Error: Unknown module @plasmicapp/react-web/skinny

Hi 👋
I just setup my new Plasmic App using npx create-plasmic-app (Next.js + TypeScript + PlasmicApp scheme), everything is well installed. I copy the Simple Light Landing Page, project is published but I have this error on yarn dev.

And even on yarn build :
info  - Using webpack 5. Reason: Enabled by default https://nextjs.org/docs/messages/webpack5
info  - Checking validity of types
info  - Creating an optimized production build
info  - Compiled successfully
PlasmicLoader: doing a fresh fetch...
info  - Collecting page data

Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error
Error: Unknown module @plasmicapp/react-web/skinny

I didn't find any issue related to this yet.
How to fix this?

Thank you for reporting the bug! This was in fact a bug on our end. We just deployed a fix, could you try again and reopen if there is still an issue?

Hi @plasmicops! Thanks.
I tried again by running :

yarn dev : same error
yarn build : same error
yarn add @plasmicapp/loader-nextjs and yarn dev : same error

Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error
Error: Unknown module @plasmicapp/react-web/skinny


ensures address string & normalizes letter case on addresses read from cache

Fixes an issue detected and described in #149.

Would be really helpful to merge this soon 🙏 Facing the same issue

cc @petejkim @hieronymus777

One of our projects has relied on the wallet link and it would really be nice if this gets approved sooner .. Facing the same issue as well.

Would be really glad if this PR could be merged soon !
Thanks ! 🙏

✌✍🙏
Sent from Yahoo Mail for iPhone
On Thursday, September 9, 2564, 2:12 AM, Alban Bleicher @.***> wrote:
Would be really glad if this PR could be merged soon !
Thanks ! 🙏
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@petejkim @hieronymus777 pinging you again for this, could you please review & merge this pull request?

@petejkim @hieronymus777 pinging you again for this, could you please review & merge this pull request?

just adding a +1 for this.

Sent from Yahoo Mail for iPhone
On Wednesday, September 22, 2021, 7:05 AM, Andrew Willis @.***> wrote:
@petejkim @hieronymus777 pinging you again for this, could you please review & merge this pull request?
just adding a +1 for this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

I ran into this same issue, tested this change in my code - worked perfectly.  Let's get it merged please.

thank you. this has been resolved with https://github.com/coinbase/coinbase-wallet-sdk/pull/315/files


security/acme-client: Make "log level" GUI control always visible

Log level is probably one of the most useful options for anyone using a plugin such as LE. I don't think any other function or plugin that offers a log level option, hides it from the user behind a "show/hide advanced" button.
This PR simply moves the control in the GUI to be always visible, without needing to be exposed by "show advanced".

@stilez Merged, thanks!


Native bindings

One scenario that comes up a lot are services that have their bindings as a k8s Secret, but don't have a duck typed CR nor the extension annotations.
This issue proposes that we allow ServiceBinding.spec.service to point to a regular k8s Secret and behave as if it was defined via a duck type.  This would enable ServiceBinding authors to connect to various existing services without the need to wrap it with other CRs (such as the ProvisionedService concept).
Taking this further, and inspired by Service Binding Operator's detection support, we could even allow for Ingress / Route resources to be natively bindable - for example, if a microservice just needs to know the URL to connect to.

This is a concept I've been torn by in the past. The implementation is not hard, but to what end?
A bare secret is unlikely to contain the metadata necessary to actually consume a service unless it was created with that intent in mind. The ProvisionedService resource is a crutch, but creating it is meaningful because it is blessing a Secret as containing that information, or at least that intent.
In order to provide the best user experience, our goal is for service providers to implement the provisioned service duck type and not say that users referencing a Secret directly is good enough.

Taking this further, and inspired by Service Binding Operator's detection support, we could even allow for Ingress / Route resources to be natively bindable

Not sure what Route you're talking about (Knative?), but the Ingress resource supports virtual hosts with multiple hostnames and  backend services, so I'm not sure how a ServiceBinding could make sense of it without already knowing what it's looking for.
I'm in favor of allowing implementations to understand service references beyond the provisioned service duck type, and likewise for applications, but I'm unsettled on adding that to the spec.

Ya - I guess it's a balance in terms of which part of the spec we favour adoption:  allowing Secrets to be referenced directly doesn't provide motivation for service providers to adopt the spec (they can say "hey, I am already exposing the information via a Secret, leave me alone" 😄 ), but, it does allow spec adoption for ServiceBinding authors that want to consume that service.
I think there are vastly more ServiceBinding authors than there are service providers.

We can see some of the side effects of thinking of arbitrary Secrets as binding material here. https://github.com/k8s-service-bindings/spec/issues/96#issuecomment-689611046
I fully agree that we need to be realistic about interop with the existing world, but we should avoid encouraging behavior that will lead to confusion or downstream violations of the spec (like a projected volume not including a type, or worse including a misleading type)

I think the concept of the duck type can apply here:  if a Secret conforms to the spec's view of a bindable Secret, it can be used a spec Secret.
In other words, if it has things like type, etc, then why not allow that to be bindable?

Arthur to create a PR and continue discussions there

@arthurdm After thinking about the "native binding" I wrote this. Let me if it captures your proposal. Let me know if you want me to send this as a PR.
## Direct Secret Binding

There are scenarios where Provisioned Service is not available.  But there is a
secret meeting the requirements of Provisioned Service available for binding.  A
custom projection with an additional annotation **MAY** be used for directly
binding this secret into the application.

The author **MUST** set the `projection.service.binding/method` annotation to
`Direct`.

The `Secret` **MUST** contains a `type` entry with a value that identifies the
abstract classification of the binding.  It is **RECOMMENDED** that the `Secret`
also contain a `provider` entry with a value that identifies the provider of the
binding.  The `Secret` **MAY** contains any other entry.

The `.spec.service.kind` attribute **MUST** be `Secret` and the
`.spec.service.apiVersion` **MUST** be `v1`.  The `spec.service.name` atrribue
**MUST** be a `LocalObjectReference`-able to a `Secret`.

Since the direct secret biding is using custom projection, the behavior
described in the [Custom Projection](#custom-projection) section is applicable
here.

### Direct Secret Binding Example Resource

```yaml
apiVersion: service.binding/v1alpha2
kind: ServiceBinding
metadata:
  name: account-service
  annotations:
    projection.service.binding/type: "Custom"
    projection.service.binding/method: "Direct"
spec:
  application:
    apiVersion: apps/v1
    kind:       Deployment
    name:       online-banking

  service:
    apiVersion: v1
    kind:       Secret
    name:       prod-account-service-secret

status:
  binding:
    name: prod-account-service-projection
  conditions:
  - type:   Ready
    status: 'True'
  - type:   ProjectionReady
    status: 'True'

Probably the annotation could be:
projection.service.binding/source: "Secret"

is the projection.service.binding/source: "Secret" annotation redundant with the service reference:
  service:
    apiVersion: v1
    kind:       Secret

If they are redundant, we should drop the annotation, if they are not we should explain how they are different.

Edit 2: Emphasize this is a Custom Projection

Why limit this to Custom Projections? The source of the service secret is orthogonal to how it is projected into an application.

is the projection.service.binding/source: "Secret" annotation redundant with the service reference:
  service:
    apiVersion: v1
    kind:       Secret

If they are redundant, we should drop the annotation, if they are not we should explain how they are different.

Yes, the annotation is redundant. I am fine with dropping it.

Edit 2: Emphasize this is a Custom Projection

Why limit this to Custom Projections? The source of the service secret is orthogonal to how it is projected into an application.

Yes, there is no need to restrict this into Custom Projection. But this feature should be still an extension for the sake of portability, right?

I have changed the extention proposal without using Custom Projection.
## Direct Secret Projection

There are scenarios where Provisioned Service is not available.  But there is a
`Secret` resource available for binding.  This extension, named Direct Secret
Projection, allows Service Binding resource with the service directly pointing
to a secret resource.

The `Secret` **SHOULD** contain a `type` entry with a value that identifies the
abstract classification of the binding.  It is **RECOMMENDED** that the `Secret`
also contain a `provider` entry with a value that identifies the provider of the
binding.  The `Secret` **MAY** contain any other entry.

The `.spec.service.kind` attribute **MUST** be `Secret` and the
`.spec.service.apiVersion` **MUST** be `v1`.  The `spec.service.name` attribute
**MUST** be a `LocalObjectReference`-able to a `Secret`.

### Direct Secret Projection Example Resource

```yaml
apiVersion: service.binding/v1alpha2
kind: ServiceBinding
metadata:
  name: account-service

spec:
  application:
    apiVersion: apps/v1
    kind:       Deployment
    name:       online-banking

  service:
    apiVersion: v1
    kind:       Secret
    name:       prod-account-service-secret

status:
  binding:
    name: prod-account-service-projection
  conditions:
  - type:   Ready
    status: 'True'

hey @baijum - sorry for the delay in responding to this.  I like the latest proposal a lot better 👍   I definitely agree this is orthogonal to projection - this is about the "other" side, the provisioned service just having a single Secret.  The projected app should see no differences.
An extension sounds like the right way to go, since we already have annotations / x-descriptors as extensions.
Basically the options for a service provider would be:
(a) status.binding duck type (core spec, recommended)
(b) Secret as-is (extension)
(c) annotations / x-descriptors (extension)
If you wanted to make a PR with your latest post, that would be awesome @baijum!  I appreciate that.
My suggestion is to call it something like Direct Secret Reference (dropping the word Projection to avoid confusion).

Thanks for the feedback @arthurdm I have sent a PR: #104

Closing as completed


Jupyter notebook can't be launched. It opens as text

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:1.56.2
OS Version: macos 11.4

I cannot launch the jupyter notebooks as usual. Now when I open any jupyter notebook that exists on my mac, it becomes text editor(as attached). However, when I create a new notebook in vscode, it can work.

/extJupyter


Hotkeys - Passive Listener

Since Electron doesn't have a passive listener I've written one you can fork here, this PR has the code to integrate it in place of the Electron stuff.

Thank you looks good! I'll test it later.


fix(costmodels): get current cost model from page and not redux

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1573   +/-   ##
=======================================
  Coverage   77.50%   77.50%
=======================================
  Files         233      233
  Lines        3832     3832
  Branches      704      704
=======================================
  Hits         2970     2970
  Misses        772      772
  Partials       90       90

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 9e0f3b6...b566c6a. Read the comment docs.

Thank you @dlabrecq for reporting the bug, reviewing and merging the fix and @ddonahue007 for your review!


escape characters are shown on Elixir cheat sheet

On the gleam for elixir users page under Operators, the escape characters are shown, so || displays as \|\|.  On the Erlang cheat sheet the issue isn't present, however the | character looks huge, unlike the rest of them.
Elixir:

Erlang:

Thank you