feat: Support for Gitea Webhook Prebuilds

Description
This PR adds support for webhooks for Gitea prebuilds. It implements five methods: webhook get, register, unregister, comparing commit ranges and parsing event data.

[ ] This change requires a documentation update
[ ] I have made corresponding changes to the documentation

Related Issue(s)
closes #996
/claim #996
Screenrecord
https://github.com/user-attachments/assets/da1e8441-8a52-491c-aa46-a096ad5f4999

Nice work!

Thanks!


Clarify HP Borrowing rules statement

SWG:
It was noted in meeting when we merged the HP Borrowing rules that we want to clarify these rules before submission.

Fix the formatting.


Repository badges

Below is an example for the coveralls coverage, I will send you a badge link once you enable this github actions. Feel free to modify to be applicable to the current regression testing.
name: CI Regression

on: [push, pull_request]

jobs:

  test:
    runs-on: ubuntu-latest
    strategy:
      max-parallel: 4
      matrix:
        # 3.6 will reach EoL in December 2021
        # https://devguide.python.org/#status-of-python-branches
        python-version: [ "3.8", "3.9", "3.10" ]
    steps:
      - uses: actions/checkout@v2
      - name: Setup python
        uses: actions/setup-python@v2
        with:
          python-version: ${{ matrix.python-version }}
          architecture: x64
      - name: Install test dependencies
        run: |
          sudo add-apt-repository ppa:ubuntugis/ubuntugis-unstable
          sudo apt-get update
          sudo apt-get install -y gdal-bin libgdal-dev build-essential
          python -m pip install --upgrade pip
          pip install https://github.com/rasterio/rasterio/archive/master.zip
          pip install .[test]
          pip install coveralls
      - name: Run tests
        run: coverage run -m pytest -v
      - name: Submit coveralls
        if: github.event_name != 'pull_request'
        run: coveralls --service=github
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Done


Automate rolling upgrade in Kubernetes

PR: https://github.com/cockroachdb/cockroach/pull/19995

Need to add details to https://www.cockroachlabs.com/docs/v2.0/orchestrate-cockroachdb-with-kubernetes.html.


Tutorial grooves

#29 Tutorial Groove Types

Content:

Tutorial on how to use the grooves
profiles have an attrribute dictionary with a unit entry which is None by default
plots will automatically name the x axis y direction in *unit and the y axis z direction in *unit because the welding direction is the x axis and the heigth is usually the z axis.
the *unit will be defined by either a special option in the plot function or if that is not given by the unit in the profile attributes dictionary

profiles have an attribute dictionary with a unit entry which is None by default
plots will automatically name the x axis y direction in *unit and the y axis z direction in *unit because the welding direction is the x axis and the height is usually the z axis.
the *unit will be defined by either a special option in the plot function or if that is not given by the unit in the profile attributes dictionary

thanks, I changed "unit" to "units" for consistency


Allow any extension for PCH files

Fixes #74

I would say most projects one of the standard set of header extensions for C/C++/ObjC (.h, .hpp, .hxx, .hh, .H, and I'm probably missing some) but Xcode allows any extension.

If you'd prefer the list to still be limited I can update the PR.

If Xcode allows any extension, I think we should allow that as well 👍


Refresh token: expire after a certain interval?

We're currently using refresh tokens with the following setup:
    // Allow use of sliding refresh token, which changes handle every refresh request
    AllowOfflineAccess = true, // Client requests scope 'offline_access' to be issued a refresh token
    AccessTokenLifetime = 3600, // 1 hour
    AbsoluteRefreshTokenLifetime = 0,
    SlidingRefreshTokenLifetime = 1296000, // 15 days
    RefreshTokenExpiration = TokenExpiration.Sliding,
    RefreshTokenUsage = TokenUsage.OneTimeOnly,
    UpdateAccessTokenClaimsOnRefresh = true

We want to keep using the sliding token expiration, but we want to allow for the client making several quick requests in succession with the same refresh token by having it expire after, say, 1 hour instead of immediately.  My understanding is that if we change RefreshTokenUsage to TokenUsage.ReUse, the refresh token won't expire at all, but OneTimeOnly makes it expire immediately.  Is there a way we can achieve this "expire after x interval" behaviour?

https://leastprivilege.com/2020/06/29/refresh-tokens-in-identityserver4-v4/

@brockallen Was the "mark a refresh token as consumed rather than delete it from the DB" behaviour implemented in version 4.0.1 or something?  On the version 3.x we're using, it actually is deleting it.

yes. Starting with v4

@leastprivilege Is there any guide for migrating from identityserver 3.x to 4.x?

https://github.com/IdentityServer/IdentityServer4/issues/4592


[CPAP-11786] Ensures that null values are filtered correct

I believe this is the correct fix to CDAP-11786. idx represents the column number that is being affected by the filtering and the -1 check is checking if the column name is in the table. object represents the actual value in the column for the particular entry and this is where we have to ensure whether or not null values are being properly handled. I have also included a test.
@nitinmotgi does this seem right to you?

LGTM 👍


Backport of lookup sync on startup to 0.11.0

Backporting #4758

@himanshug @leventov

:+1: let us merge after build passes

@a2l007 in the future it's better to cherry-pick merge commit: https://github.com/druid-io/druid/commit/c07678b143663dde069569ab77c9803901d78a73

@a2l007 could you please update this patch to be based off the most recent 0.11.0 branch? It has #4978 in it, which should help CI pass.


added: new empoyee and did some minor adjustments (WIP)

@shahzaib834 is this ready for review?

@shahzaib834 is this ready for review?

Sir, Need to add a couple of images. Just got the image. Updating PR and letting you know.


Ignore status packets

no need to print these packets out, I think they're just the ADS server spamming any connected clients for their status.

Sorry, this was supposed to be against my fork.


Using this as a library

Hi, I'd like to use this as a library in a web app I'm writing. Is this possible?

Yes this would definitely be possible. Are you writing your web app in Node.js?

@johnroper100 see if my other repo works better if you're still interested. Libmoji.

Ok, thanks


Published pngencoder-0.12.0.pom contains ${project.groupId}:${project.artifactId}

https://repo1.maven.org/maven2/com/pngencoder/pngencoder/0.12.0/pngencoder-0.12.0.pom contains

${project.groupId}:${project.artifactId}

… causing an awkward display in IntelliJ's dependencies view:

:scream:

I'll just hardcode it to com.pngencoder:pngencoder

Will be fixed with next release to maven.org


Upgrade Heroku stack from heroku-16 to herkou-18

We're using an out-of-date stack in all our Heroku apps. We should review the changelog, test out the latest version, and get ourselves up to date.
There's a button in the UI to do the upgrade. We could also try it on a review app first, before doing it on staging.

The official announcement contains little information, apart a link to the main docs and stating that it is based on Ubuntu 18.04.
This page contains a detailed table with the versions of all the different packages on the image.
I've got a feeling that we should just go ahead and give this a try. However, I'm not sure we can just upgrade the review apps in isolation, given that these are based on the parent shields-staging app.

Agreed. I'm not sure either, though I think it might be possible to upgrade a review app that has been created for an existing PR.

https://shields-staging-pr-4968.herokuapp.com/ will be upgraded on next deployment, i.e. commit on that PR.

Everything seems to be running happily on that particular review app, therefore I'm about to upgrade the staging app which will in turn impact all new review app deployments.
I suggest we leave the resulting state of affairs in place for a few days. If we don't notice anything odd on the various review apps and the staging environment, we can then upgrade one of the production regions first, and gain confidence that it runs smoothly in a real production environment before upgrading the second one.
For reference, a rollback of stack versions can easily be performed via the CLI, see the instructions here.

Nothing unexpected so far. Will aim to upgrade the production environment tomorrow morning European time.

Well, opening the Heroku console this morning, it's already done on the production environments... Slightly worrying that big changes like this go through without anyone being aware and with no traces in the activity log.

Great!
That would be a good thing to have in the logs, for sure. Fortunately it happens infrequently, and usually seamlessly.

Slightly worrying that big changes like this go through without anyone being aware

Is it possible someone promoted the staging instance to production after you deployed the new build pack to staging and didn't realise?

Ah, that's a good point. Flipping the buildpack switch in production doesn't in practice have any effect, since the builds are created on the staging app.


switch to new middleware format

next release should be 1.11+ only, so can switch to new middleware format

Hey @jamesturk I new to Django and I wanna learn step by step so pls help learn it by contributing to this project :)


Bring back yticks for ICA properties

Fixes #6718. In addition to bringing back the yticks and labels, I've removed the space before the colon in the title, and " (index)" from the x axis label (because "segment" is sufficient).
Here's what it looks like after the changes:

I'd also like to change the horizontal position of the y axis label for the ERP/ERF - the "AU" label is too far to the left and should be closer to the actual axis. Anyone know how to do that quickly? Otherwise, this is ready to merge.

@SophieHerbst had suggested to also report the unit of the channel but I realize that the unit of an ICA source is arbitrary. That's probably why we had put it in AU...
thoughts?
@SophieHerbst I don't see how you can have V or T/m unless you enforce some normalization of the unmixing matrix which is not done by default

@agramfort Yes, that's why we have AU, I don't think we should change that.

From a user perspective it would be nice to have a real unit, but unless we change our normalization (and make sure it is the same for every ICA algorithm) we should stay with AU for now. Maybe the docstring should add an explanation why we show AU.

@SophieHerbst says she is fine with AU, so if anyone can help me move the label closer to the y axis I could include that, otherwise we can also go ahead and merge as is.

@cbrnr did you try the labelpad arg here:
https://matplotlib.org/3.1.1/api/_as_gen/matplotlib.axes.Axes.set_ylabel.html
(For Google-fu reference, this link was the first result for "matplotlib adjust ylabel spacing")
One of the examples linked from there (thanks Sphinx-gallery!) shows another option you could consider, fig.align_ylabels:
https://matplotlib.org/3.1.1/gallery/pyplots/align_ylabels.html#sphx-glr-gallery-pyplots-align-ylabels-py
If these do not work, often for mpl there are some StackOverflow questions that you can find that will offer other solutions

Thanks @larsoner, but at least with my limited MPL understanding it's not that easy. fig.align_ylabels would be perfect, but for this to work the axes need to be part of a subplot, which isn't the case for this figure. I tried the following line just before returning the fig:
fig.align_ylabels([image_ax, erp_ax])

This gives the following error:
Traceback (most recent call last):
  File "/Users/clemens/Repositories/mne-python/.venv/lib/python3.7/site-packages/IPython/core/interactiveshell.py", line 3326, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-2-5a67bd76d7a2>", line 1, in <module>
    fig.align_ylabels([image_ax, erp_ax])
  File "/Users/clemens/Repositories/mne-python/.venv/lib/python3.7/site-packages/matplotlib/figure.py", line 2597, in align_ylabels
    ss = ax.get_subplotspec()
AttributeError: 'Axes' object has no attribute 'get_subplotspec'

The labelpad argument for set_ylabel would work, but there's no way I can perfectly align both y labels because this is in points so it depends on the screen resolution.
Ideally, I'd want to use fig.align_ylabels - any idea how I can make this work?

I think it's good enough as is.

I think I can do it by modifying _create_properties_layout as follows:
import matplotlib.gridspec as gridspec
grid = gridspec.GridSpec(2, 2, figure=fig)
grid_fine = gridspec.GridSpecFromSubplotSpec(4, 1, subplot_spec=grid[0, 1])
axes = []
axes.append(fig.add_subplot(grid[0, 0], label='topomap'))
axes.append(fig.add_subplot(grid_fine[:-1], label='image'))
axes.append(fig.add_subplot(grid_fine[-1], label='erp'))
axes.append(fig.add_subplot(grid[1, 0], label='spectrum'))
axes.append(fig.add_subplot(grid[1, 1], label='variance'))

Then I can align the ylabels with fig.align_ylabels([image_ax, erp_ax]), but I also have to remove the \n character in the label of the ERP. This gives nicely aligned labels for the image and ERP axes, but I need to adjust the spacing between the axes:

I suggest that this should be tackled in another PR.

Image without changing the subplots but after removing the line break:

Definitely good enough (so now ready to merge), but the alternative I suggested is also still much better!

I like the good enough option. :) The gridspec version would look better IMO if the column and row space were not equally divided (that is: upper row would have more space, and first column would be narrower)

@mmagnuski a gridspec layout like you suggest can be easily implemented. But let's leave this for another PR and do it after this gets merged.

@cbrnr Yeah, I know, I just like the way it is now. :)

Thanks @cbrnr


Option to set the cache size

I didn't want to use the cache option as it is taking more time to debug the two renders (one from cache and other normal). It would be good to have an option similar to Turbolinks.pagesCached(size) to restrict the size.

I am currently using below hackish way to achieve the same, if anyone wants to set it before the option is available
Turbolinks.Controller.prototype.clearCache = function() {
  return this.cache = new Turbolinks.Cache(requiredSize); // requiredSize == 0 in my case
}
Turbolinks.clearCache();

Do you really want to adjust the cache size? Or do you just want to turn it off entirely?

I wanted to turn it off entirely to check if an issue I was facing was because of pages being cached.

OK. Closing this for now.
In the next release, you’ll be able to opt out of caching on a per-page basis using a <meta> tag:
<meta name="turbolinks-cache-control" content="no-cache">

See #8 for details.

+1


called Option::unwrap() on a None value

Hi Prisma Team! My Prisma Client just crashed. This is the report:
Versions

Name
Version

Node
v18.16.1

OS
debian-openssl-3.0.x

Prisma Client
5.4.2

Query Engine
ac9d7041ed77bcc8a8dbd2ab6616b39013829574

Database
postgresql

Logs
prisma:tryLoadEnv Environment variables loaded from /home/rafael/work/fwd/.env
prisma:client checkPlatformCaching:postinstall
prisma:client checkPlatformCaching:ciName
prisma:tryLoadEnv Environment variables loaded from /home/rafael/work/fwd/.env
prisma:client dirname /home/rafael/work/fwd/node_modules/.prisma/client
prisma:client relativePath ../../../packages/api-core-lib/prisma/reverse-etl-db
prisma:client cwd /home/rafael/work/fwd/packages/api-core-lib/prisma/reverse-etl-db
prisma:client clientVersion 5.4.2
prisma:client:libraryEngine internalSetup
prisma:client checkPlatformCaching:postinstall
prisma:client checkPlatformCaching:ciName
prisma:tryLoadEnv Environment variables loaded from /home/rafael/work/fwd/.env
prisma:client dirname /home/rafael/work/fwd/node_modules/.prisma/client
prisma:client relativePath ../../../packages/api-core-lib/prisma/reverse-etl-db
prisma:client cwd /home/rafael/work/fwd/packages/api-core-lib/prisma/reverse-etl-db
prisma:client clientVersion 5.4.2
prisma:client:libraryEngine internalSetup
prisma:get-platform Found distro info:
{
  "targetDistro": "debian",
  "familyDistro": "debian",
  "originalDistro": "tuxedo"
}
prisma:get-platform Trying platform-specific paths for "debian" (and "ubuntu")
prisma:get-platform Found distro info:
{
  "targetDistro": "debian",
  "familyDistro": "debian",
  "originalDistro": "tuxedo"
}
prisma:get-platform Trying platform-specific paths for "debian" (and "ubuntu")
prisma:get-platform Found libssl.so file using platform-specific paths: libssl.so.3
prisma:get-platform The parsed libssl version is: 3.0.x
prisma:client:engines:resolveEnginePath enginePath /home/rafael/work/fwd/node_modules/.prisma/client/libquery_engine-debian-openssl-3.0.x.so.node
prisma:get-platform Found libssl.so file using platform-specific paths: libssl.so.3
prisma:get-platform The parsed libssl version is: 3.0.x
prisma:client:engines:resolveEnginePath enginePath /home/rafael/work/fwd/node_modules/.prisma/client/libquery_engine-debian-openssl-3.0.x.so.node
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine sending request, this.libraryStarted: false
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started

Client Snippet
// PLEASE FILL YOUR CODE SNIPPET HERE

Schema
// PLEASE ADD YOUR SCHEMA HERE IF POSSIBLE

Prisma Engine Query
{"X":true}}}}}

Hi @rafakwolf
Could you please share your schema and the code snippet? Without that information we won't be able to reproduce and fix this issue.
Thank you!

Hey @rafakwolf  ,
Did you get a chance to have a look at my previous comment? It'd be helpful for us to move your issue forward.
Without further information, we'll have to close this issue. Thank you 🙏

Hi @SevInf  go ahead and close it, I am not able to reproduce this anymore


springcloud中eureka，指定eureka.instance.hostname

springcloud中eureka，指定eureka.instance.hostname=本机域名，但是在注册中心注册的依然是ip，如何解决该问题？

Can you provide the English translation?

@ryanjbaxter I use Spring Eureka and set eureka.instance.hostname with the server hostname，but it‘s still ip in the center；
For example：eureka.instance.hostname=bigdataXXX.server.org，the server hostname is bigdataXXX.server.org；
How should I solve this problem?

@ryanjbaxter The center shows ip like <IP_ADDRESS>:nemdp-web:8080

Do you have spring.application.name set?  <IP_ADDRESS>:nemdp-web:8080 looks like the instance id to me.  What problem is this causing?

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.

@ryanjbaxter I want instance id (<IP_ADDRESS>:nemdp-web:8080) to be shown as (bigdataXXX.server.org:nemdp-web:8080)

The instance id just has to be unique. It's not what is registered as the host in eureka


Fixes double-header on 404 pages

Makes 404.hbs compatible with the contentOnly templates
Change app.js to pass the contentOnly parameter to 404.hbs

Fixes #416

LGTM

LGTM, I fixed the trailing space in the merge.


Cannot remove shoot, if the credentials are wrong

What happened:
I created the shoot cluster with the wrong credentials and I want to delete it. Unfortunately I can't.
What you expected to happen:
I can delete the shoot cluster with the wrong credentials.
How to reproduce it (as minimally and precisely as possible):
Create shoot cluster with the wrong credentials and try to delete it without fixing the credentials, you'll see in the logs:
time="2019-07-11T16:12:31Z" level=error msg="Infrastructure did not get ready yet, lastError is: Error reconciling infrastructure: Terraform execution job 'soil-dev.infra.tf-job' could not be completed. The following issues have been found in the logs:\n\n-> Pod 'soil-dev.infra.tf-pod-94aae' reported:\n* provider.openstack: Authentication failed\n\x1b[0m\n\x1b[0m\x1b[0m\x1b[0m" operation=reconcile shoot=garden/soil-dev

Anything else we need to know?:
https://github.com/gardener/garden-setup/pull/58

Does this still happen? If the cluster was initially created with incorrect credentials then the respective extension controller cannot create any infra resources, hence, the TF state configmap should be empty. According to my understanding this should allow the extension controller to finalize the Infrastructure object. If the cluster was initially created with correct credentials and resources got created then it's actually desired behaviour to let the cluster deletion fail if not all resources have been cleaned up yet.
Please reopen if you think the problem is not yet solved.


Load script into body

What is the current behavior?
i want to load a script form into body and show it instead load it into head tag
my code is:
<Helmet>                         <div className="modal-content">                             <script                                 src="https://17edw.freshsales.io/web_forms/f3f3297be32b2d88c2aed5564e828b79d83374e4844b4198cf5922c4c6f50c1d/form.js"                                 crossOrigin="anonymous"                                 id="fs_f3f3297be32b2d88c2aed5564e828b79d83374e4844b4198cf5922c4c6f50c1d"                             >                             </script>                         </div>                     </Helmet>
but it does not show the form for me.

x2

hello @17Edwin  , did you find solution?

and @jhoanborges , did you find a solution?


Bluetooth hci address family not supported

Home Assistant release with the issue:
 0.75.2
Last working Home Assistant release (if known):
None
Operating environment (Hass.io/Docker/Windows/etc.):
 Docker - https://hub.docker.com/r/homeassistant/home-assistant/
Component/platform:
 bluetooth_tracker
Description of problem:
Attempting to use a bluetooth usb dongle, It fails to setup and spits the following error
OSError: [Errno 97] Address family not supported by protocol
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

device_tracker:
  - platform: bluetooth_tracker

Traceback (if applicable):
Error setting up platform bluetooth_tracker
Traceback (most recent call last):
  File "/usr/src/app/homeassistant/components/device_tracker/__init__.py", line 183, in async_setup_platform
    disc_info)
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/homeassistant/components/device_tracker/bluetooth_tracker.py", line 72, in setup_scanner
    for dev in discover_devices():
  File "/usr/src/app/homeassistant/components/device_tracker/bluetooth_tracker.py", line 50, in discover_devices
    lookup_class=False)
  File "/config/deps/lib/python3.6/site-packages/bluetooth/bluez.py", line 26, in discover_devices
    device_id = _bt.hci_get_route()
OSError: [Errno 97] Address family not supported by protocol

Additional information:
Unraid 6.4
Bluetooth adapter works fine with any VMs I pass it through to

Have this issue also when running HA in Docker without --net=host.


aggregate_statistics should only optimize MIN/MAX when relation is not empty

Which issue does this PR close?

Closes #8911.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thank you @alamb


hard-wired paths in scripts/generate-bosh-lite-manifest

scripts/generate-bosh-lite-manifest assumes default paths are in $HOME/workspace/...
There's no reason to impose parent pathnames on other developers.  One could use $(git rev-parse --show-toplevel) to find the location of this directory, or better still, provide an option to specify the cloudfoundry dir (I for one keep my cloudfoundry/ and cloudfoundry-incubator/ repo's in separate directories).

Not an issue - use of this directory is documented in github/c-i/diego-release/docs


[Bug]: I7-12700 vpl encoder change resolution problem

Which component impacted?
Encode
Is it regression? Good in old configuration?
None
What happened?
https://github.com/oneapi-src/oneVPL-intel-gpu/releases/tag/intel-onevpl-23.1.5
vbr
1:init()
2: Encode 100 frames of 1920x1080 video, then modify the resolution to 960x540 or 640x480 Reset function will report error: MFX_ERR_INVALID_VIDEO_PARAM(-15)
3: We try to Close & Init the encoder when the Reset function report error -15, but it's init failed with this error: [ERROR], sts=MFX_ERR_UNSUPPORTED(-3), AllocFrames, Query (for encoder) failed.
This method of changing the resolution is what we have been using in mediasdk and vpl just started.
What's the usage scenario when you are seeing the problem?
Video Conference, Others
What impacted?
No response
Debug Information
No response
Do you want to contribute a patch to fix the issue?
Yes, I'm glad to submit a patch to fix it

Hello, @DaveHu-TVU which format do you want to encode to? e.g. AVC, AV1, VP9 or others?

H.265 1080p
I tested H.264 1080p is works well.

H.265 1080p I tested H.264 1080p is works well.

@thomasli21801 could you please take a look? Thanks

Any update for this issue? Thanks.

Hi  @thomasli21801
Any update for this issue? Thanks.


Create mackenzista.txt

@vitorruiz Could you please follow the rules emphasized at the bottom of this page to add your domain to the list as quick as possible?
In particular, it says:

We merge pull requests manually, and check the information which you have provided us with prior to merge. Thus, if you wish to make the verification easier for us and therefore much quicker, please mention the following in your comment to the request:
-the university official website URL, if different from the domain you are submitting
-an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the university
-an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

@philipto
The university official website URL: http://portal.mackenzie.br/
URL of an IT-related course: http://up.mackenzie.br/graduacao/sao-paulo/sistemas-de-informacao/
Screenshot showing that the university recognizes the domain as an official email domain for the students. This screenshot represents my account in the university moodle.

@vitorruiz Thank you for the update! In fact, the screenshot shows the email address in a domain @mackenzista.com.br, and the file lib/domains/br/mackenzista.txt represents a domain @mackenzista.br. If the domain which you wish to add is @mackenzista.com.br, please rename the file to lib/domains/br/com/mackenzista.txt. Thank you!

@vitorruiz I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen or create a new one if you have an update regarding the issue described above. Thank you for your understanding!


Nested schemaless changesets

I'm wondering if this is supported right now?
I am trying to build a helper macro for doing request parameter validation but am stuck at implementing a solution for nested parameter
Gist below:
https://gist.github.com/imranismail/eb60c709b230c1cbf344553888b9387d

No, this is not supported right now.

It would be really great if we could use embeds with schemaless changesets. Ecto is awesome for casting JSON data into structs, but it gets difficult when the JSON contains several levels of nesting.

Is this still not supported? If not, is this something the project owners would be okay with as a new feature?  I would like to possibly implement this if there is a desire to have it.

Is this still not supported? If not, is this something the project owners would be okay with as a new feature?  I would like to possibly implement this if there is a desire to have it.


ELRS spi rx support

I am out of the loop on ELRS but hope to dive in soon.  I'm being told that there are fc being built with ELRS receiver connected to mcu via SPI?  If anyone has a circuit diagram for an ELRS spi based receiver - please link it here.  I would love to build one for the breadboard to make debugging easier and get started on support.

Happymodel have a CBF4 with it, bet you could guess the pin assignment as it's likely the same circuit as the other CBF4 series, newer ones at least. It's going to be the same spi connections as frsky etc surely. Someone from Elrs would have the rest of the info.


Support Android

Hello everyone,
I'm working on an application using libp2p. I'm receiving the following message:
failed to sufficiently increase receive buffer size (was: 160 kiB, wanted: 7168 kiB, got: 2929 kiB). See https://github.com/quic-go/quic-go/wiki/UDP-Buffer-Sizes for details.

Is there anything that can be done on Android?

It's all described on the linked page.
That said, running a p2p stack on a mobile device doesn't seem like a good idea...


Purview Share C# SDK V3 - Add new API List Share Resources

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

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Analytics.Purview.Sharing


Path Solutions Review (Jupyter)

Provide a way for path creators to page through the last few solutions submitted for Jupyter activities. Provide a way to show a read-only code editor (so that there is highlighting)  with the code from the code block the learner was allowed to edit. Only the submitted code needs to be shown.

Unable to view the individual solutions. Also prefer to show userid instead of the User name.

@Thangamanir can you please share the error or the video of the problem you are having in viewing the individual solution. Below is a video link, which shows the functionality working correctly in development environment.
https://drive.google.com/file/d/1vsjWBKoICLyLqZx3wRTSGwO661eYQXE3/view


'Available terms' message printed once for recursive call

Simple fix to move the print statement out of the recursive call to avoid having it print multiple times

Looks good to me


[WFCORE-5373] Fix missing resource handling

https://issues.redhat.com/browse/WFCORE-5370
If the resource to be removed doesn't exist, the operation should fail

@jasondlee thanks


Footer adjustment

the footer needs some upgrades, the fonts are very large , the font is not attractive , the button and form have some collision , In general, it is not attractive . Please assign me this task, and I will make it beautiful.

Can you elaborate and share the design?

As this issue is not resolved for more than a week, and the website is now updated with new changes


Add support for HEIC images for training via libheif library

Still images at high quality take up a lot of space.
The High Efficiency Image File Format (HEIF) solves that by using better compression resulting in lower file sizes at the same quality level Wikipedia: High_Efficiency_Image_File_Format when compared to similar image compression format Comparison
Describe the solution you'd like
I propose a support of HEIF files (HEIC,AVIF) via libheif. It is already being used in a lot of open source projects like GIMP, ImageMagick or OpenImageIO

Hi is there any update on this. Perhaps any hints on how to implement this?


Add units concept for modulable functions of a repository

In this PR, I create a new abstract thing unit. An unit is a function for a repository. Every unit will have a tab UI on repository home page. For example, Code is an unit, Issues is another.  Currently, we have 9 unit types. If #733 is merged, then we will have a new unit Reviews. Unit concept will be a basis for many further features in future.

LGTM

Would it be better to have the keys of RepoUnit.Config be an enum type instead of string literals? I see two advantages to using an enum:

We don't need to have "magic" string literals in our code. It is easy to accidentally mistype something like "ExternalTrackerFormat", and the compiler won't tell you if you do.
When the RepoUnit.Config field is converted to JSON to store in the database, the generated JSON object will have shorter fields, and thus will use up less disk space.

build error

done.

LGTM

@ethantkoenig please confirm.


fix: add support for @* treesitter highlight groups

nvim-treesitter recently dropped support for the TS* highlight groups (See: https://github.com/nvim-treesitter/nvim-treesitter/pull/3656) which broke Treesitter support for many themes, including this one.
I have added the corresponding @* highlight groups which should return the syntax highlights back to normal. The old TS* highlight groups can likely be removed in the future, but I have kept them untouched for now in case they are needed for backwards compatibility with older Treesitter versions.
Edit: It seems like these changes aren't 100% in line with the previous highlight groups, so some changes need to be made before this can be merged. TODO tags, for example, aren't using the iris color anymore.

Thanks so much! Unable to test at the moment but will merge so people don't have a broken theme :)


ISSUE on Windows for any official .NET MAUI examples in the repo that uses TableView

Running any official .NET MAUI example on Windows that uses the TableView to list the examples to try, I face this problem:
https://github.com/dotnet/maui-samples/tree/main/8.0/UserInterface/Views/ListViewDemos
On the MainPage I click one item in the list to open the clicked example in a new page.
When I came back from that new page, I can click on any items in the list of examples, but they won't open any new example page anymore :-(
It works only for the first example you click.
It works on Android and iOS.

Solved in .NET MAUI 9.0.12 SR1.2


Attempt to serialize closure './dataset/miniImagenet.lua:31' with 1 upvalues

/home/i/torch/install/bin/luajit: /home/i/torch/install/share/lua/5.1/dataset/Reader.lua:52: Attempt to serialize closure './dataset/miniImagenet.lua:31' with 1 upvalues; first is: '_'. Consider using ipc.workqueue.writeup() instead.
What wrong?

I reach the same error

Yes, maybe this is a problem of torch-ipc. But anyone can provides any help?

Has anyone able to figure out the problem? I've got the same error.
/home/yang/torch/install/bin/lua: /home/yang/torch/install/share/lua/5.2/dataset/Reader.lua:52: Attempt to serialize closure './dataset/miniImagenet.lua:31' with 2 upvalues; first is: '_ENV'.
Consider using ipc.workqueue.writeup() instead.

I think the algorithm has been abandoned, since it's novel but not promising.
and nobody care about the bugs in the source project.

Hi, unfortunately I'm not actively working in torch anymore so I'm not sure of the solution off the top of my head. I would suggest making sure you have the right version of torch-ipc as stated in the README.

@sachinravi14 I'm using LUA 5.2, I just want to make sure which version of LUA did you use? Does it has any effect?

So, I have installed torch again with LUA 5.1. Then I download the right version of torch-ipc as stated in the README, and run luarocks make in the torch-ipc folder. I also run luarocks make for torch-dataset. Now, the "code is running without any error".

@abdullahjamal could you please tell me how to download the right version of torch-ipc ?

It is stated in description. You can go to this commit 'c1b2984c4c2dae085005d385996f4c0660173b27'


Restructure Java code.

Part of https://github.com/census-instrumentation/opencensus-experiments/issues/17.
Restructure Java code according to @bogdandrutu's suggestion:

integration/java/grpc/... contains the java implementation for grpc
integration/java/http/... contains the java implementation for http

EDIT:
Closes https://github.com/census-instrumentation/opencensus-experiments/issues/17.

@songy23 might need a rebase from master but otherwise LGTM.

Rebased - thanks!

It is probably good to move gradle stuff to java directory. That way the root directory get simplified.

Sounds good, I created https://github.com/census-instrumentation/opencensus-experiments/pull/24 to move the gradle files.


Create usecases.md

added some notes about how people will use this thing.

@scottagt @crherlihy, can you think of other usecases for this package?


Checking distro...
Ubuntu detected
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100     9  100     9    0     0     41      0 --:--:-- --:--:-- --:--:--    42
Reading package lists... Error!
E: Invalid archive signature
E: Internal error, could not locate member control.tar{.zst,.lz4,.gz,.xz,.bz2,.lzma,}
E: Could not read meta data from /tmp/tmp.GiKx6rznCl.deb
E: The package lists or status file could not be parsed or opened.

Resolved! Thank you for reporting 🙏
The debian package build required additional poking, after the workspace adjustment in v14.0.1 broke things


Development

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage remained the same at 60.454% when pulling a98b41c525008d50962d999f86666175c54086c7 on development into f7ecbfb463d2db54f545fa199c9b684122357c16 on master.


Fail structs

Lot's of changes to for better reporting of failures, both through the CLI and programmatically.

Coverage increased (+0.5%) to 97.062% when pulling e8efef97b4e965dfb2e60760802c2f6fb98c8735 on fail_structs into cded7fd57c3de1816711822502300e3a549babbb on master.


Typo on the page

The word "Frabric" should be "fabric"

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

ID: 80a333ac-8208-4146-2d99-6b6258b64a3f
Version Independent ID: 553642a3-6125-062f-350e-2538e3401a5d
Content: Deploy a secured Service Fabric cluster in Azure Stack
Content Source: articles/azure-stack/user/azure-stack-solution-template-service-fabric-cluster.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mattbriggs

@cenkersoy  Thanks for noting this. I've updated the topic. #please-close

@cenkersoy We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


run unittests in test dbs; add cleanup

Update the broker baseTest to accommodate DB-308:

Update the db names in the application config, setting them to temporary database names that will be used during the unit tests
Drop the temporary databases during the class tear-down method

No problems here, just waiting on core PR so we don't break dev by using things that don't exist yet.


Promotions: Assigning a newly created product rule in the discount tab does not work

Description: If you create a new prodct rule directly fro the "Discount/Rabatt" tab inside the promotions, this new rule does not automatically get assigned to the Discount as a rule but needs to be manually assigned
Steps to reproduce:

Create promotion & Save
Switch to Discount tab, add discount
Select "Only for selected products" toggle
Create a new Rule from within the "product rules" dropdown

Expected result:

New rule should be selected

Actual result:

Rule Field stays empty.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-36806.
[created from NEXT-36806, comment 486655]


Improve conntrack kernel-support detection for in-kernel compilations

Fixes #1826

Quick q about availability, but other than that LGTM.


[20.03] leftwm: 0.1.10 -> 0.2.2

(cherry picked from commit feb9b1b406833f9dafee3aa8072b533f90e22973)
Backport of https://github.com/NixOS/nixpkgs/pull/82188
ZHF: #80379

The commit messages don’t need the [20.03] on them. I think it’s implied, since they’re on the branch.


sparse unique indexes

During the backend meeting it came up that we might want to test "sparse unique indexes". Backend team (probably @sorenbs) has more context.

I need more context indeed for this. I want to know whether this refers to multi column indexes or many uniques index on fields of the same model.

It is about exactly what it says - about "sparse unique indexes" which are its own concept: https://docs.mongodb.com/manual/core/index-sparse/ So it is a unique index on a field, where many instances of NULL are allowed for example.

Is sparse unique indexes still on the agenda? I would really like to see that feature in Prisma.


fix: add expr error log

Signed-off-by: Ryota Sakamoto<EMAIL_ADDRESS>

Codecov Report

Merging #32 (205d501) into master (6e7a433) will decrease coverage by 0.12%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   46.71%   46.58%   -0.13%
==========================================
  Files          26       26
  Lines        1460     1464       +4
==========================================
  Hits          682      682
- Misses        630      632       +2
- Partials      148      150       +2

Impacted Files
Coverage Δ

pkg/triggers/service.go
64.86% <16.66%> (-7.87%)
:arrow_down:

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 6e7a433...205d501. Read the comment docs.

@alexmt
That's right. They cannot realize if error has happened now.


Encryption is pretty slow

First of all, thank you for your great work!
In a recent project (Flutter iOS and Android), we have quite a few places where we do encryption and decryption. This is done using different packages.
Today we were writing a flutter_secure_file_storage package that encrypts and saves as files. This is done using pointycastle. Before we used another package to verify/decrypt JSON web signature and JSON Web Encryption. If I remember correctly we used jose
The saving took around 8 seconds on a high-end Android device and decryption took ~4 seconds. This being a high-end device and us having millions of users, that's obviously not good enough. Therefore we went to bridging in both cases. Below are the logs I used to compare.
You can clearly see a huge difference. This could be due to the packages, but I don't think this is the case seeing that this is for all the encryption we encountered and the difference is so big. What do you think is the cause of this, and if it's the packages, how can we develop faster packages for this?
I/flutter (25662): ========decrypt List=========
I/flutter (25662): native verifyAndDecrypt done 475
I/flutter (25662): dart decrypt done 3933

I/flutter (16076): ========decrypt List=========
I/flutter (16076): native verify done 367
I/flutter (16076): dart verify done 2047
I/flutter (16076): native decrypt done 221
I/flutter (16076): dart decrypt done 1125

Using dart
I/flutter ( 4526): FILE 1: done with file storage in 12 ms
I/flutter ( 4526): FILE 1: done with storage in 1 ms
I/flutter ( 4526): FILE 2: done with file storage in 152 ms
I/flutter ( 4526): FILE 2: done with storage in 3 ms
I/flutter ( 4526): FILE 3: done with file storage in 50 ms
I/flutter ( 4526): FILE 3: done with storage in 25 ms
I/flutter ( 4526): FILE 4: done with file storage in 7903 ms
I/flutter ( 4526): FILE 4: done with storage in 89 ms
I/flutter ( 4526): FILE 5: done with file storage in 8014 ms
I/flutter ( 4526): FILE 5: done with storage in 116 ms

Using bridge to native Android
I/flutter (17902): FILE 1 - File storage write took 25 ms
I/flutter (17902): FILE 1 - Storage write took 3 ms
I/flutter (17902): FILE 2 - File storage write took 25 ms
I/flutter (17902): FILE 2 - Storage write took 2 ms
I/flutter (17902): FILE 3 - File storage write took 18 ms
I/flutter (17902): FILE 3 - Storage write took 2 ms
I/flutter (17902): FILE 4 - File storage write took 17 ms
I/flutter (17902): FILE 4 - Storage write took 17 ms
I/flutter (17902): FILE 5 - File storage write took 20 ms
I/flutter (17902): FILE 5 - Storage write took 10 ms

Thanks for filing the issue!  I've transferred this to the Dart sdk repo to start, but it's not clear whether this is an issue with the Dart compilers or the package implementation.  @mraleph may have some advice on narrowing this down further.

With such a big difference, I would check first if the code is being run with assertions enabled, as pointycastle makes a lot of use of assertions.

The pointycastle comparison was done in debug, so with assertions enabled. But the jose comparison was in production (some phones took 20 to even 60 seconds for decrypting). When decrypting native, we saw a reduction of 8 times that.
I was thinking how to check if this is due to the packages and I'll test some things out tomorrow to see if I can get a simple codebase together that illustrates the issue.

I've created a new branch on our package to time the durations see here.
Tested on an Android 8 pro in release with a random file of 10MB using AES/GCM/NoPadding
I/flutter (14940): decryption dart (on main thread) took 11380 ms
I/flutter (14940): decryption native took 145 ms
I/flutter (14940): decryption compute took 11993 ms
I/flutter (14940): decryption computer.compute took 11968 ms

I am not sure I can find the exact source code for the program which have outputted the output in your latest post. Maybe you can give me a direct link?
But I found the following:
https://github.com/icapps/flutter_secure_file_storage/blob/6f9ca9bbec98a911d627a4916a40932898dcfb69/example/lib/main.dart#L29
Are you handling your data as 10MB big String objects?

I've added logging in this file https://github.com/icapps/flutter_secure_file_storage/blob/temp/timing-decryption-duration/lib/src/flutter_secure_file_storage.dart and ran the example project
Is this an issue that we handle such big Strings? In our app we download a file as String (it's encrypted) and save it locally to use it offline.

Is this an issue that we handle such big Strings? In our app we download a file as String (it's encrypted) and save it locally to use it offline.

Normally, you what to make a solution where you can stream the data in chunks so you can decrypt and process the content as a stream of data to be e.g. saved to disk. By doing so, we can save memory since we don't need to have the full content of the file loaded at the same time.
When you have a String at 10MB in size, it is an indication of some badly designed architecture in the solution. E.g. an example where a developer tries to fit everything JSON so it can be saved as documents in a cloud solution instead of using object or file storage.

First a high-level remark. In general I would strongly recommend going with native crypto rather than taking any sort of pure Dart crypto libraries or writing any sort of pure Dart crypto libraries from scratch. Native crypto has more eyes on it and consequently is more secure.  It's also going to be faster in general. I personally recommend using https://pub.dev/packages/webcrypto - which is a portable package providing access to high quality native cryptographic algorithms (through BoringSSL natively and through WebCrypto on the Web).
That being said we should definitely take a look at the performance of pointycastle to see if there are any low hanging fruits, we could pick and make it faster.

When you have a String at 10MB in size, it is an indication of some badly designed architecture in the solution. E.g. an example where a developer tries to fit everything JSON so it can be saved as documents in a cloud solution instead of using object or file storage.

We have a pretty big list (~100k) of id's that are hashed and then the whole file is encrypted. This causes it to be this big. We can look into this again since it was not initially this big.
@mraleph You do have a great point there. I've tested it out, and works as expected so I'll implement that one instead.
I'll be closing this ticket. Thank you for your time!

I would like to keep this open so that we take a look to check that nothing is awfully wrong on our side either.


Simplify UTF-8 comparison on null character

Null character (byte zero) is not allowed in string values ingested into Pinot (all the string values ingested will be sanitized and all the characters after the first null character will be truncated), so we don't really need to guarantee the ordering when the input string value from the query has null character in the end because the original value is anyway modified. This can simplify the comparison logic and reduce overhead when string dictionary has paddings.
E.g. if the value in the dictionary is "abc\0\0\0" after padding, the original input value could be "abc", "abc\0", "abc\0\0", "abc\0\0\0" or even "abc\0\0\0\0", "abc\0abc".
After the change, the comparison result is:

= "abc"
= "abc\0"
= "abc\0\0"
= "abc\0\0\0"
< "abc\0\0\0\0"
< "abc\0a"

Codecov Report

Merging #10151 (a2f44ed) into master (d4356b4) will decrease coverage by 54.97%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10151       +/-   ##
=============================================
- Coverage     68.60%   13.64%   -54.97%
+ Complexity     5770      176     -5594
=============================================
  Files          2006     1951       -55
  Lines        108689   106199     -2490
  Branches      16493    16196      -297
=============================================
- Hits          74563    14487    -60076
- Misses        28838    90570    +61732
+ Partials       5288     1142     -4146

Flag
Coverage Δ

integration1
?

unittests1
?

unittests2
13.64% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

 Learn more

Does LoaderTest.testPadding pass with this change? I'm all for simplification (assuming the whatever the motivation for adding LoaderTest.testPadding was no longer matters) but this isn't going to move the needle much performance wise.

Does LoaderTest.testPadding pass with this change? I'm all for simplification (assuming whatever the motivation for adding LoaderTest.testPadding was no longer matters) but this isn't going to move the needle much performance wise.

This test is removed in #10087 where we removed the non-zero padding support. That test is added mainly for testing the behavior of legacy '%' padding, which is much more common than null.

With the current code, length == bytes.length && length != 0 && bytes[length - 1] == 0 case is not handled (the new added ValueReaderComparisonTest is flaky right now). To keep the behavior exactly the same as before, we need to do the following check:
      if (padded) {
        if (length < bytes.length) {
          return -1;
        }
        if (length == bytes.length) {
          if (length == 0) {
            return 0;
          } else {
            return dataBuffer.getByte(startOffset + length - 1) == 0 ? -1 : 0;
          }
        }
        // length > bytes.length
        // need to figure out whether the unpadded string is longer than the parameter or not
        if (bytes.length == 0) {
          // just need nonzero first byte
          return dataBuffer.getByte(startOffset) == 0 ? 0 : 1;
        } else if (bytes[bytes.length - 1] == 0) {
          return -1;
        } else {
          // check if the stored string continues beyond the length of the parameter
          return dataBuffer.getByte(startOffset + bytes.length) == 0 ? 0 : 1;
        }

IMO it is too much and quite easy to make mistakes without adding much value


6.8.3 Changing checkbox input's checked attribute to null does not uncheck it

I am building a React project with webpack and noticed that after updating from 6.8.2 to 6.8.3 that setting a checkbox input's checked attribute from whatever is was before to null does not result in the checkbox unchecked. It seems that if it started checked, it will stay checked when set to null, but if it started null, it will appear unchecked until it is clicked.
I expected that setting an attribute to null would be the equivalent of removing that attribute. Is that not the case?
Also, it may be possible that this change was caused by something other than babylon updating, but not as far as I can tell.
Example of the React snippet I am referring to:
<input type="checkbox" checked={condition ? "checked" : null} />

Well, first of all, I don't see how this can be possibly related to a code parser.
Second, checked in React is a boolean and takes true / false and not HTML-like "checked" string and certainly not null.

Maybe it was because of the recent react update? But I also can't see a relation to babylon.

Yep, just confirmed this issue wasn't caused by babylon updating. Guess I jumped the gun opening this ticket.
Thanks for the replies.


Link contributing.md to top-level readme.

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


Allow screen orientation variants

Sometimes different UI panels are required for portrait and landscape. Have inbuilt support for this.

Shouldn't be a part of UGX. Handle orientation changes yourself.


Rm social index from compare page

Re-opening, because it's been removed completely, so can't be used in pairwise comparisons. Need to reinstate generally, and then just remove for single layer view.


TransactionId in MeterValues is optional, but exception with "missing 1 required positional argument"? (Ocpp1.6)

2022-02-09 10:19:27 NEXT-TBD00151: receive message [2,"6913c269-87c7-4347-bec6-30a5d2f0e3be","MeterValues",{"connectorId":1,"meterValue":[{"sampledValue":[{"value":"0","measurand":"Energy.Active.Import.Register"}],"timestamp":"2022-02-09T09:18:51Z"}]}]
2022-02-09 10:19:27 Error while handling request '<Call - unique_id=6913c269-87c7-4347-bec6-30a5d2f0e3be, action=MeterValues, payload={'connectorId': 1, 'meterValue': [{'sampledValue': [{'value': '0', 'measurand': 'Energy.Active.Import.Register'}], 'timestamp': '2022-02-09T09:18:51Z'}]}>'
Traceback (most recent call last):
  File "/home/cestonat/.local/lib/python3.7/site-packages/ocpp/charge_point.py", line 194, in _handle_call
    response = handler(**snake_case_payload)
  File "/home/cestonat/.local/lib/python3.7/site-packages/ocpp/routing.py", line 47, in inner
    return func(*args, **kwargs)
TypeError: on_meter_values() missing 1 required positional argument: 'transaction_id'
2022-02-09 10:19:27 NEXT-TBD00151: send [4,"6913c269-87c7-4347-bec6-30a5d2f0e3be","InternalError","An unexpected error occurred.",{}]

How does the request handler for MeterValues look? Does it's interface defines an argument transaction_id?

Ok ... sorry for that. It's my fault :(


support writing output to a different dir/file

Could be helpful, for report and config file location: https://github.com/adrg/xdg
Additionally, target report file path should be providable via cmdline parameter, yup.


JMS destinations are not available

Even though we are deploying the src/test/resources/wildfly-tck-min.yml when starting wildfly, the destinations don't seem available.  The only queue/topic deployment messages in the log are:
2024-08-23 01:49:11,861 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221043: Protocol module found: [artemis-server]. Adding protocol support for: CORE
2024-08-23 01:49:11,862 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221043: Protocol module found: [artemis-amqp-protocol]. Adding protocol support for: AMQP
2024-08-23 01:49:11,862 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221043: Protocol module found: [artemis-hornetq-protocol]. Adding protocol support for: HORNETQ
2024-08-23 01:49:11,862 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221043: Protocol module found: [artemis-stomp-protocol]. Adding protocol support for: STOMP
2024-08-23 01:49:11,886 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221034: Waiting indefinitely to obtain primary lock
2024-08-23 01:49:11,886 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221035: Primary Server Obtained primary lock
2024-08-23 01:49:12,324 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221003: Deploying ANYCAST queue jms.queue.ExpiryQueue on address jms.queue.ExpiryQueue
2024-08-23 01:49:12,327 INFO  [org.apache.activemq.artemis.core.server] (ServerService Thread Pool -- 82) AMQ221003: Deploying ANYCAST queue jms.queue.DLQ on address jms.queue.DLQ

and any deployment with a queue/topic reference fails with:
2024-08-23 22:40:49,984 INFO  [org.jboss.as] (Controller Boot Thread) WFLYSRV0025: WildFly 33.0.0.Beta1-SNAPSHOT (WildFly Core 25.0.0.Beta4) started in 1956ms - Started 394 of 637 services (344 services are lazy, passive or on-demand) - Server configuration file in use: standalone-full.xml - Minimum feature stability level: community
2024-08-23 22:40:51,316 INFO  [org.jboss.as.repository] (management-handler-thread - 1) WFLYDR0001: Content added at location /home/starksm/Dev/Jakarta/wildflytck-new/wildfly/dist/target/wildfly-33.0.0.Beta1-SNAPSHOT/standalone/data/content/17/2ce03bc6909d80d6869aaf146d5bb8b2bbac97/content
2024-08-23 22:40:51,325 INFO  [org.jboss.as.server.deployment] (MSC service thread 1-7) WFLYSRV0027: Starting deployment of "closedQueueConnection_servlet_vehicle.ear" (runtime-name: "closedQueueConnection_servlet_vehicle.ear")
2024-08-23 22:40:51,694 INFO  [org.jboss.as.server.deployment] (MSC service thread 1-1) WFLYSRV0207: Starting subdeployment (runtime-name: "closedQueueConnection_servlet_vehicle_web.war")
2024-08-23 22:40:51,947 WARN  [org.jboss.weld.deployer] (MSC service thread 1-3) WFLYWELD0013: Deployment closedQueueConnection_servlet_vehicle.ear contains Jakarta Contexts and Dependency Injection annotations but no bean archive was found (no beans.xml or class with bean defining annotations was present).
2024-08-23 22:40:51,992 ERROR [org.jboss.as.controller.management-operation] (management-handler-thread - 1) WFLYCTL0013: Operation ("add") failed - address: ([("deployment" => "closedQueueConnection_servlet_vehicle.ear")]) - failure description: {
    "WFLYCTL0412: Required services that are not installed:" => [
        "jboss.naming.context.java.jms.QueueConnectionFactory",
        "jboss.naming.context.java.MY_QUEUE"
    ],
    "WFLYCTL0180: Services with missing/unavailable dependencies" => [
        "service jboss.naming.context.java.module.closedQueueConnection_servlet_vehicle.closedQueueConnection_servlet_vehicle_web.env.jms.MyQueueConnectionFactory is missing [jboss.naming.context.java.jms.QueueConnectionFactory]",
        "service jboss.naming.context.java.module.closedQueueConnection_servlet_vehicle.closedQueueConnection_servlet_vehicle_web.env.jms.MY_QUEUE is missing [jboss.naming.context.java.MY_QUEUE]"
    ]
}
2024-08-23 22:40:51,993 ERROR [org.jboss.as.controller.management-operation] (management-handler-thread - 1) WFLYCTL0013: Operation ("add") failed - address: ([("deployment" => "closedQueueConnection_servlet_vehicle.ear")]) - failure description: {
    "WFLYCTL0412: Required services that are not installed:" => [
        "jboss.naming.context.java.jms.QueueConnectionFactory",
        "jboss.naming.context.java.MY_QUEUE"
    ],
    "WFLYCTL0180: Services with missing/unavailable dependencies" => [
        "service jboss.naming.context.java.module.closedQueueConnection_servlet_vehicle.closedQueueConnection_servlet_vehicle_web.env.jms.MyQueueConnectionFactory is missing [jboss.naming.context.java.jms.QueueConnectionFactory]",
        "service jboss.naming.context.java.module.closedQueueConnection_servlet_vehicle.closedQueueConnection_servlet_vehicle_web.env.jms.MY_QUEUE is missing [jboss.naming.context.java.MY_QUEUE]"
    ]
}
2024-08-23 22:40:51,993 ERROR [org.jboss.as.server] (management-handler-thread - 1) WFLYSRV0021: Deploy of deployment "closedQueueConnection_servlet_vehicle.ear" was rolled back with the following failure message:
{
    "WFLYCTL0412: Required services that are not installed:" => [
        "jboss.naming.context.java.jms.QueueConnectionFactory",
        "jboss.naming.context.java.MY_QUEUE"
    ],
    "WFLYCTL0180: Services with missing/unavailable dependencies" => [
        "service jboss.naming.context.java.module.closedQueueConnection_servlet_vehicle.closedQueueConnection_servlet_vehicle_web.env.jms.MyQueueConnectionFactory is missing [jboss.naming.context.java.jms.QueueConnectionFactory]",
        "service jboss.naming.context.java.module.closedQueueConnection_servlet_vehicle.closedQueueConnection_servlet_vehicle_web.env.jms.MY_QUEUE is missing [jboss.naming.context.java.MY_QUEUE]"
    ]
}

It just seems like the src/test/resources/wildfly-tck-min.yml is not being read or read correctly. What I have done is to add the messaging configuration to the src/test/resources/standalone-full.xml and that is what I'm running against. With this a majority of the tests are passing, but there are still a number of failures to work through.


Modules

<filename>.skrypt file in directory should allow accessing its variables using <filename>.<varname> notation

Ready, two-way binder is next


data: project addition by Niladri2003

Add a new project to the list
I want to add a new project to the list
Yeah I have cool responsive landing page for a educational platform
Here is the website link
Eduhub
Code of Conduct

[ ] I agree to follow this project's Code of Conduct

Completed. Thanks for your contributions 🎉 ❤


Fixed the unit step function

This PR corresponds to issue #99. Fixed the implementation of unit step function.

Before pull requesting, did you tested the program that whether your output is desired now?

Kindly add test for the function, AS FROM NOW , adding test will be compulsory , else Pull Request will not get merged

Before pull requesting, did you tested the program that whether your output is desired now?

yes

Kindly add test for the function, AS FROM NOW , adding test will be compulsory , else Pull Request will not get merged

Alright. One more thing, my points in the leaderboard are not getting updated and no one responding to my messages in gitter. Please acknowledge if u are one of the maintainers.

@parva-jain don't worry about your leaderboard it will be updated. Rohan was a busy ig he will update withing a day or 2.

@parva-jain don't worry about your leaderboard it will be updated. Rohan was a busy ig he will update withing a day or 2.

Is there something wrong with this PR?


Fix flaky docker integration test

What does this PR do?
Fix a race condition in the TestDockerListenerSuite/TestListenBeforeStart and TestDockerListenerSuite/TestListenAfterStart integration tests
Motivation
The tests started failing around 8/28 with the following errors:
    --- FAIL: TestDockerListenerSuite/TestListenAfterStart (11.95s)
        docker_listener_test.go:235:
                Error Trace:    /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:235
                                                        /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:186
                Error:          Expected nil, but got: &errors.errorString{s:"timeout listening for services, only got 0, expecting 2"}
                Test:           TestDockerListenerSuite/TestListenAfterStart
        docker_listener_test.go:236:
                Error Trace:    /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:236
                                                        /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:186
                Error:          "map[]" should have 2 item(s), but has 0
                Test:           TestDockerListenerSuite/TestListenAfterStart
        docker_listener_test.go:279:
                Error Trace:    /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:279
                                                        /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:186
                Error:          "map[]" should have 2 item(s), but has 0
                Test:           TestDockerListenerSuite/TestListenAfterStart
    mock.go:33: 2024-08-28 19:45:52 UTC | INFO | (comp/core/autodiscovery/listeners/workloadmeta.go:142 in Listen) | ad-containerlistener initialized successfully
    --- FAIL: TestDockerListenerSuite/TestListenBeforeStart (11.94s)
        docker_listener_test.go:235:
                Error Trace:    /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:235
                                                        /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:202
                Error:          Expected nil, but got: &errors.errorString{s:"timeout listening for services, only got 0, expecting 2"}
                Test:           TestDockerListenerSuite/TestListenBeforeStart
        docker_listener_test.go:236:
                Error Trace:    /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:236
                                                        /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:202
                Error:          "map[]" should have 2 item(s), but has 0
                Test:           TestDockerListenerSuite/TestListenBeforeStart
        docker_listener_test.go:279:
                Error Trace:    /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:279
                                                        /go/src/github.com/DataDog/datadog-agent/test/integration/listeners/docker/docker_listener_test.go:202
                Error:          "map[]" should have 2 item(s), but has 0
                Test:           TestDockerListenerSuite/TestListenBeforeStart
FAIL

FAIL    github.com/DataDog/datadog-agent/test/integration/listeners/docker      27.564s
FAIL

Looking at the debug logs, I see:
    mock.go:33: 2024-08-28 19:45:40 UTC | INFO | (comp/core/workloadmeta/impl/store.go:557 in startCandidates) | workloadmeta collector "docker" could not start. error: component workloadmeta-docker is disabled: Agent is not running on Docker

Which leads us to think this is a race condition because the WLM component start is possibly not a blocking operation
Job failures on main
Describe how to test/QA your changes
Integration test suite should remain green
Possible Drawbacks / Trade-offs
Additional Notes
Possibly related to https://github.com/DataDog/datadog-agent/pull/28703

/merge


Allow to reassign local variables in shaders

Currently, the shader fails if the following code is defined (Redefinition Error) :
	float x = 0.0;
	{
		int x = int(x);
	}

This PR fixes this incorrect behavior.

Should there be a warning if people do this?

The user may intend to use it + there is no warning system in shaders right now


[BUG] Wrong description in parameters file

Describe the bug
The description of parameter "parCustomerPolicySets" does not make sense.
Log Analytics Workspace
To Reproduce
Steps to reproduce the behavior:

Go to Log Analytics Workspace
Read description

Expected behavior
The description should read for example: Optional array of customer specified policy assignments to the top-level management group.

Issue has been resolved, closing the item.


NEW with wrong parameters gives a silent error

When you use NEW in the console, it simply doesn't makes any error, and the previous cart in memory is untouched. Maybe there should be an error for NEW wrong parameters?

now it shows unknown parameter error
fixed in .41


Change dot notation in add column documentation to tuple

A tuple must be used to make columns in structs as described in add_column:
"Because "." may be interpreted as a column path separator or may be used in field names, it is not allowed to add nested column by passing in a string. To add to nested structures or to add fields with names that contain "." use a tuple instead to indicate the path."
This PR corrects the documentation to use tuples instead of dot notation.
From issue 1407

Heres where the errors happens
https://github.com/apache/iceberg-python/blob/b0ea716c91f19281d3d9cd7b6965d5d01f6cc3d5/pyiceberg/table/update/schema.py#L184-L192
And some debugging statements:
(Pdb) path
('details', 'confirmed_by')
(Pdb) name
'confirmed_by'
(Pdb) parent
('details',)
(Pdb) parent_full_path
'details'
(Pdb) parent_id
-1
(Pdb) len(parent) > 0
True

parent_field = self._schema.find_field(parent_full_path, self._case_sensitive)

is where it errors.
Seems like we're missing the case where no "parent" is present

Looks like there might be a bug with this change. I tried to follow the docs
from pyiceberg.catalog.sql import SqlCatalog
from pyiceberg.schema import Schema
from pyiceberg.types import NestedField, StringType, DoubleType, LongType

warehouse_path = "/tmp/warehouse"
catalog = SqlCatalog(
    "default",
    **{
        "uri": f"sqlite:///{warehouse_path}/pyiceberg_catalog.db",
        "warehouse": f"file://{warehouse_path}",
    },
)

schema = Schema(
    NestedField(1, "city", StringType(), required=False),
    NestedField(2, "lat", DoubleType(), required=False),
    NestedField(3, "long", DoubleType(), required=False),
)
catalog.create_namespace_if_not_exists("default")
try:
    catalog.drop_table("default.locations")
except:
    pass

table = catalog.create_table("default.locations", schema)

# with table.update_schema() as update:
#     # In a struct
#     update.add_column("details.confirmed_by", StringType(), "Name of the exchange")

with table.update_schema() as update:
    update.add_column(("details", "confirmed_by"), StringType(), "Name of the exchange")

errors

Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
  File "/Users/kevinliu/repos/iceberg-python/pyiceberg/table/update/schema.py", line 192, in add_column
    parent_field = self._schema.find_field(parent_full_path, self._case_sensitive)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/kevinliu/repos/iceberg-python/pyiceberg/schema.py", line 215, in find_field
    raise ValueError(f"Could not find field with name {name_or_id}, case_sensitive={case_sensitive}")
ValueError: Could not find field with name details, case_sensitive=True

Yes, the struct has to exist before you can insert anything into it. This can be adjusted in the code to automatically create the parent. For now, it is detailed in the documentation changes. Should I write more explicitly?

Yes, the struct has to exist before you can insert anything into it.

ah i see, that makes sense. in that case, can we edit the example so that it works out of the box?
Also i think its valuable to move the comment to the top level docs of "Add Column".

i found another dot notion in Move column, do we need to change this too?
https://py.iceberg.apache.org/api/#move-column


Is this repo still active? Build failed

fresh ubuntu16.04 with docker correctly installed
$ docker build .
pull access denied for google/golang, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

have tried docker login and other solutions on the web without any success, then noticed that this project's last commit was 3 years ago with all travis checks failed. Seems like an inactive but not deprecated repo so just wanted to know if dockersh still works or are there any alternatives that provide the same function?

login shell on machines with multiple interactive users.
...
places user sessions into their own individual docker containers in a secure and locked down manner

Understood, quite a pity though...thought I'd found a simple solution to my "multi-user-one-box" system while not messing with multiseat or kvm etc... thanks anyway!


test_module.test_factorization_machine_module has fixed seed that can mask flakiness

The unit test in title have been using fixed seed to mask flakiness. Suggested action:

Evaluate whether the test is flaky without fixed seed. If not, remove seed. Else move to 2
If test is flaky, determine whether it's an actual uncaught edge case. If so, fix the operator. Else move to 3
If numerical instability is inevitable, adjust tolerance level appropriately.

Thanks for filing this issue. We will investigate this Flaky test

I could reproduce the issue using random seed<PHONE_NUMBER>.  The issue is accuracy mismatch

Assigned JIRA ticket: https://issues.apache.org/jira/browse/MXNET-770

Fixed unit test and added training test. PR: https://github.com/apache/incubator-mxnet/pull/12023

@szha PR is merged. Please verify and close this issue. Thanks!

@sandeep-krishnamurthy Related PR is merged, should be good to close.


Add component options (args) to YAML, add as JSON to package

We want to be able to present the component options (arguments) in GOV.UK Design System. To do this, they need to be generated for inclusion in package.
This PR:

Adds all component options in the component YAML as params.
Introduces isComponent option (so that the options for that component can be linked to in the Design System).
Copies the component options as JSON files to package when gulp build:package --destination 'package' is run.
Amends the tests for checking contents of package to allow filenames with wildcards in additionalFilesNotInSrc
Adds a test to check all package/components have a macro-options.json that contains valid JSON with the expected attributes name, type, required, description

Created this as one PR so that the generated JSON files can be tested against https://github.com/alphagov/govuk-design-system/pull/518
More on component options:

We now refer to these as “options” in client facing docs as calling them “arguments” is technically incorrect and “parameters” was felt to be less clear than “options”.
Regarding the "or" relationship between text and html attributes, we are continuing to explain this in the description of the attributes, ie. "If html is set, this is not required." There didn't seem to be a sensible way of trying to show this relationship in a simple table so using just text appeared to be  the most accessible method.
This PR retains the convention of referring to them as “params” in component macro and template files - we might want to roll out renaming “params” to “options”  in all component macros and templates as a separate PR that can be quickly merged in without causing too many conflicts with other PRs.

To test, run npm run build:package
https://trello.com/c/QN0imhXt/722-define-arguments-for-components-in-the-component-yaml-file
https://trello.com/c/00iDbQOU/1323-allow-argument-yamls-to-be-transformed-into-json-and-copied-to-package

Do we still have a frontend test app? Maybe we can read the macro options json and write to a page in that, so we can see it works?

@joelanman Thanks a lot for the review 🙌
We test if the file contains JSON and if the JSON contains attributes (name, type, required, description) that we would expect it to have. We could use the frontend test app for testing the JSON but it seems like a slightly odd way of testing it (maybe put it in a table?) since the intention isn't to surface the JSON in GOV.UK Frontend review app in any way. Do you there are specific advantages to testing it in this way?

Arguments that have been checked:

[x] hint @igloosi
[ ] input @igloosi
[ ] label  @igloosi

@hannalaakso

Do you there are specific advantages to testing it in the test app or anything in particular we should look out for?

I think it could be useful to easily browse the macro option documentation in this project, instead of having to import it into the Design System to see it properly. But this can be another PR, if we want to do it


Hardhat Account for Local Development

Temp Account for testing purpose

For Now Account '0x5FbDB2315678afecb367f032d93F642f64180aa3' use this as Market Address


Ability to Delete a Step

[x] Ability to delete temporary step (not connected to an integration)
[x] Ability to delete middle step of an integration

This has now been implemented. Following will be a debounce handling of user input to improve UX of the Editor, and to prevent multiple unnecessary requests to the API.


[Event Request] Codeunit 13636 OIOUBL-Export Sales Invoice

We would like to have the possibility to embed a pdf of the posted docunemt into the OIOUBL file.
In order to do that we would need a new publisher in Codeunit 13636:
[IntegrationEvent(false, false)]
local procedure OnBeforeInsertAccountingSupplierParty(var InvoiceElement: XmlElement; SalesInvoiceHeader: Record "Sales Invoice Header")
begin
end;
Called from procedure CreateXML:
...
// Invoice->OrderReference
if SalesInvoiceHeader."Order No." <> '' then
OIOUBLXMLGenerator.InsertOrderReference(XMLCurrNode,
SalesInvoiceHeader."External Document No.",
SalesInvoiceHeader."Order No.",
SalesInvoiceHeader."Order Date")
else
OIOUBLXMLGenerator.InsertOrderReference(XMLCurrNode,
SalesInvoiceHeader."External Document No.",
SalesInvoiceHeader."Pre-Assigned No.",
SalesInvoiceHeader."Order Date");
    **OnBeforeInsertAccountingSupplierParty(XMLCurrNode, SalesInvoiceHeader);**

    // Invoice->AccountingSupplierParty
    OIOUBLXMLGenerator.InsertAccountingSupplierParty(XMLCurrNode, SalesInvoiceHeader."Salesperson Code");

Regards
Alexander

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Use a SerializedBlock type to help serializing blocks

Motivation

As part of #3707, the GetBlock response type must contain a data field with a hexadecimal string with the block's serialized bytes. In order to use the type system to make sure that the data field contains a valid block, it needs to use a type that:

Represents the bytes of a serialized block
Can be serialized as a hexadecimal string

Solution

A new SerializedBlock type is added to the zebra_chain::block::serialize module. It meets the requirements above by:

It can only be constructed from a borrowed Block type, so it is guaranteed to only contain the bytes of a serialized block.
It implements AsRef<[u8]>, which allows it to be used as a direct parameter for hex::encode and hex::serde::serialize.

Review

@oxarbitrage is working on the original issue and @teor2345 is reviewing it.
Reviewer Checklist

[x] Code implements Specs and Designs
[x] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

I think CI failed because of the GCP rate limit. Is it okay to merge as is, or should we wait a while and re-run the full sync test?

I requested a quota increase in GCP, and also re-ran the checks

This can be merged even though Test / Test full validation sync from cached state failed, as this is being fixed in another PR.


Request logging through syslog broken in DW1.0?

We have been using the following snippet to log requests through syslog:
# Listeners
server:
  applicationConnectors:
    - type: http
      port: 6720
  adminConnectors:
    - type: http
      port: 6721
  requestLog:
    appenders:
      - type: syslog
        threshold: ALL
        logFormat: "app-ms %msg"

This was still working fine in DW0.9.2. After upgrading to DW1.0.2 the requests are no longer getting logged through syslog; the regular application logging through syslog is still working fine.
Do we have to make some configuration changes?

Hi, @wilbertpol,
Thanks for reporting this issue. We migrated request logging in 1.0 from a native Jetty solution to logback-access. Unfortunately, it doesn't support syslog appenders, see LOGBACK-317.
I think it can be considered a regression for Dropwizard, and we need to figure out a way to workaround this limitaion.

The logs could be configured to the console appender, then you can update your process command to redirect console to syslog.

Hi @wilbertpol,
Could you please try to update to Dropwizard 1.0.4? We brought back old-style request logging with logback-classic and it should work with the syslog appender. You need explicitly set the type of request logging factory, though:
  requestLog:
    type: classic
    appenders:
      - type: syslog
        threshold: ALL
        logFormat: "app-ms %msg"

I did a quick check and request logging is working again. Thanks!


Bug report: Answer field in json format

Describe the bug
The answer field shows a json format instead of a table with answers / entropies
To Reproduce
Steps to reproduce the behaviour or a link to the recipe / input used to cause the bug:

add magic
add rot13
works with any text

Expected behaviour
Should rotate the characters of the magic output
Screenshots
If applicable, add screenshots to help explain your problem.

Version: 9.37.0
Compile time: 29/03/2022 17:07:59 UTC
User-Agent:
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:98.0) Gecko/20100101 Firefox/98.0
Link to reproduce

Magic outputs a JSON object under the hood which is then rendered to the HTML table you'll see if it's the final stage of a recipe. What are you trying to achieve?

Okay, thank you. My magic had a single output, thought I could simply pass it to Rot. Just looked kind of weird when I first encountered that answer


Access (auth) refactor & database transactions

Changes:

Move accesses functions from routes/accesses.py to crud/accesses.py
Use database transaction for atomicity for user/device creation

1st point lead to split crud module into submodules base (legacy) & accesses (new). I would need some feedback on this part. I needed to change patching during unit tests, fetch_all does not seem to work.

@jeanpasquier75 any update? I'm changing the release target as this isn't critical for the incoming demo

So @jeanpasquier75, what should we do with this PR?
Considering the (very favorable) pace of PRs/modif of the repo, it's better to process PR rather quickly :sweat_smile:

So @jeanpasquier75, what should we do with this PR?
Considering the (very favorable) pace of PRs/modif of the repo, it's better to process PR rather quickly 😅

Sorry for the delay 😅 I will dry to come up with something by the end of the week. As you mentioned in #76 this PR was supposed to fix access deletion with the SQL cascade mechanism as well

@jeanpasquier75 #76 was merged, let us know if you think we should keep this PR for th upcoming release or staging it for the next one!

Closing this outdated PR resolved in #90


Python Dependencies

Hello @hereariim
It seems you added some packages in the dependencies that are no external packages but actual python modules that alredy come with python
    collections
    shutil
    subprocess
    time
    threading
    queue

Could you correct these dependencies?

Thanks you @goanpeca
I just removed these packages from the setup.cfg file.


Gradient on button

Use case

Proposal

@trapeye
Looking at the issue title, you are looking to have gradient effect on material buttons. If so, there are lot of solutions available in the community that helps to achieve it. See below links for example:
https://stackoverflow.com/questions/52243364/how-to-add-a-gradient-to-a-button-in-flutter
https://medium.com/codechai/create-gradient-on-button-in-flutter-cad65b8b3da4
You can also check for similar solutions there to achieve your requirement.
I am going ahead and closing this. If you disagree or think your requirement is different, write in comments and I'll reopen it.
Thanks.


Fix typo

Thanks for this, but this project is deprecated in favor of https://gitlab.com/gitlab-org/gitlab-ci-multi-runner


add a local docker registry

in order to to run this project locally a local docker registry is needed

[x] create a subfolder for the registry
[x] create yml to deploy into kubernetes
[ ] test docker build against the registry
[ ] test kubernetes deployment against the registry
[ ] update subfolder readme
[ ] update project readme

https://rominirani.com/tutorial-getting-started-with-kubernetes-on-your-windows-laptop-with-minikube-3269b54a226

kubectl.exe run hello-nginx --image=nginx --port=80
deployment "hello-nginx" created

kubectl.exe expose deployment hello-nginx --type=NodePort
service "hello-nginx" exposed

minikube.exe service --url=true hello-nginx

minikube.exe service hello-nginx

kubectl -n default port-forward $POD_NAME 9090:9090

https://gist.github.com/coco98/b750b3debc6d517308596c248daf3bb1

eval $(minikube docker-env)

@FOR /f "tokens=*" %i IN ('minikube docker-env') DO @%i

kubectl get pods --selector=k8s-app=kube-registry-proxy -o=name


Remember tags when autocompleting a task

Currently only the project is automatically added when using autocompletion for the task. It would be nice to also re-add all given tags.

+1

works in master on osx

Thanks :)


Miss the account name in the URL of one public Emulator blob container

Storage Explorer Version: 1.16.0-dev
Build: 20200911.5
Branch: main
Platform/OS: Windows 10
Architecture: ia32
Storage Emulator Version: 5.10
Regression From: Not a regression
Steps to reproduce:

Install and run Storage Emulator -> Open Storage Explorer.
Expand the node 'Local & Attached' -> 'Storage Accounts' -> (Emulator - Default Ports) (Key) ->  Blob Containers.
Create a blob container -> Set its public access level to 'container'.
Connect the blob container to public blob container -> Check the URL of the public blob container.

Expect Experience:
The URL displays correct.

Actual Experience:
Miss the account name in the URL.

Verified this issue on build 20210109.1. Fixed.

Verified this issue on build 20210109.1. Fixed.


May conflicts with Mobz

Hi there, I tested this mod and found it may conflict with mod Mobz, here's the game log:
latest.log
Game Version: 1.16.1
Mod Version: netherite-plus-mod-trinkets-beta-1.1.2.jar

ah thank you, it shouldnt be hard for me to fix this, i think

Hi there, I tested this mod and it still conflicts with Mobz mod.
Game Version: 1.16.1
Netherite Plus Mod Version: beta-1.7.0 for 1.16.1
Mobz Mod Version: 2.4.4 for 1.16.1
Here's the game log:
https://paste.ee/p/1FXdl

This is a conflict of api's, and i dont know if there will be an easy fix. I do know that there is an api in the works for shields, and that may fix it, but until then, id say these mods are incompatible


[Server] Enhance Design Import Notification

Current Behavior and Desired Behavior

Implementation

Use Meshery Server's Event Notification framework to enhance these types of messages.

Acceptance Tests

Notification message has the filename of the imported file in double quotation marks.
Notification message has a hyperlink to the design file.

Mockups

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hello @leecalcote! I'd like to take on this issue.
PS: I want to mention that I might require a bit more time as I'm new to Go. However, I am confident that I can successfully complete the task.

@ASK-03, good deal. @theBeginner86 is accountable for seeing this completed. If he delivers before you do, you can review his PR, which will be an excellent way for you to ramp up.

[UPDATE]
Hello, I've located the code and successfully completed the first acceptance test. I'm confident that I'll be able to tackle the second one soon as well. Although it's taking some time to navigate through things in GO, I'm gradually gaining a better understanding of it.

Hello, @leecalcote! In the second acceptance test, is the link in the notification for the design file that is imported or the design file generated for the user?

Hey @leecalcote @theBeginner86! I made the required changes and would appreciate your reviews. I included the user-provided design file link in the notification. I had to tweak the arguments for the formatPatternOutput function to make the feature work. So far, it seems fine, but I'm currently testing it to be sure. Let me know your thoughts!

Hey @leecalcote @theBeginner86! I made the required changes and would appreciate your reviews. I included the user-provided design file link in the notification. I had to tweak the arguments for the formatPatternOutput function to make the feature work. So far, it seems fine, but I'm currently testing it to be sure. Let me know your thoughts!

@aabidsofi19 changes proposed to the formatPatternOutput function...
@ASK-03, instead of "saved", use "imported". Instead of "URL", use "from". Instead of two sentences, use a single sentence:
Design "<design-name>" imported from "<source-URI-either-filesystem-or-URL-path>".

If the source location of the design file is a URL, then they formatPatternOutput or some other function should make this URL clickable in the event notification.

@aabidsofi19 changes proposed to the formatPatternOutput function...
@ASK-03, instead of "saved", use "imported". Instead of "URL", use "from". Instead of two sentences, use a single sentence:
Design "<design-name>" imported from "<source-URI-either-filesystem-or-URL-path>".

If the source location of the design file is a URL, then they formatPatternOutput or some other function should make this URL clickable in the event notification.

Thanks! The notification looks much better now 😄. I've implemented the changes and am planning to test all the related functionalities to ensure I don't break anything 😅. Regarding making the URL clickable, is there an existing solution in the backend for this? Also, are there any other notifications that include URLs, so that I can reference how they implemented it?

Hey @leecalcote! I have completed this task. I have made the link clickable in the notification. I am removing the extra code that I added for development. Will create a PR in sometime for review. Till then, please have a look at the video attached below of the functionality and share you reviews.
Enhanced Notification Demo Video

@leecalcote, I've submitted the pull request for this issue. Kindly review it when you get a chance.
PS: Two workflows are failing (Golangci-lint and Meshkit error codes), I am working on it to remove the issue.


Updated to support the new ibm-watson library

I have updated server.js to support the new ibm-watson library found here:
ibm-watson
I haven't added the instructions for the new authentication method into the README.md just yet, but the authentication method can be found on the npm page.
Basically how I ran the project is by placing the ibm-credentials.env file in the root directory of the project and passing in the environment variable like so:
IBM_CREDENTIALS_FILE=./ npm start

Thank you very much! I haven't had time to work on this example for a while, but I appreciate this very much.

You're most welcome. This example ended my struggle in trying to find an open source project that does speech to text transcribing, so I'm glad to be able to help!


Update 006_ZigZag_conversion.java

An empty file was found, so update it.

To avoid merged commits in a new pull request, it's recommended to clean up the local master branch (as well as origin master branch), and create a new branch for each pull request.


item grid wont save

trying to resize the item grid but it seems it wont save when i try to make it smaller. i click save, open it, and it's default size again. it does save when i make it bigger.

Hi, i have the same problem,

first one is default size, and the second is the rescaled size


noobaa - remove from community-operators

This is needed downstream to avoid conflict with OCS (storage) over noobaa CRDs ownership.
Upstream NooBaa - not affected.
Downstream OCS:

OCS owns noobaa CRDs (so we removed to avoid owner conflict)
OCS deploys the downstream noobaa operator
OCS will be available only on redhat operators catalog
OCS will not be added to upstream-community-operators

Signed-off-by: Guy Margalit<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[ ] Has you operator nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Is your submission signed?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

FYI @nimrod-becker @tamireran @jarrpa  @obnoxxx


[DARGA] Add missing policy buttons to Network Manager Relationship pages

(cherry picked from commit 35bfb21)
fixing https://bugzilla.redhat.com/show_bug.cgi?id=1347223
Refactor ems_common.rb controller in app/controllers and toolbar_chooser.rb
in app/helpers/application_helper and add missing policy buttons to some
of the Network Manager Relationship pages to enable tagging more items
at once when managing a cloud provider in Compute -> Clouds -> Provider.
Fix policy buttons on some of the Network Managers pages to enable
tagging when reaching Networks -> Providers/Networks/Subnets/etc.
Before:

After:

@miq-bot add_label ui, bug, darga/yes


Make BufDelete only fire in response to bdelete command (or bwipeout)

Buffer deletion is an ambiguous concept in Vim. Most people think of deletion as "what happens when you bdelete a buffer," which is unloading, unlisting, and clearing some buffer-specific settings. The BufDelete event, by contrast, fires specifically when a buffer is unlisted - the buffer can still be present in a window when this happens. The documentation is clear about the current semantics - it does say that BufDelete means "Before deleting a buffer from the buffer list," and it's distinct from BufUnload. But I think what people probably want to be saying when they register events on BufDelete is "run this when bdelete happens," not "run this when the buffer is unlisted." I think the event should specifically be tied to this command, as well as run on bwipeout prior to BufWipeout.

And to support plugins that depend on the existing semantics, maybe create BufUnlisted that they can use instead.

I am not a big fan of changing the semantics for an autocommand that has been working like this for 25 years and which also breaks backward-compatibility. So I am slightly negative to it.

I think in practice it won't be noticed. Specifically -- the only circumstance in which this would be noticed and people would be sad about the change (as opposed to them noticing broken behavior now working properly) is when the following are true:

A plugin is specifically interested in knowing when a buffer has been removed from the buffer list and does something interesting if the buffer is still valid when that happens.
A user (or that plugin) unlists a buffer that they do not otherwise delete, triggering that behavior. The user doing this themselves is I think quite unusual -- and if they do explicitly unlist a buffer it's not likely that they're doing that with the expectation of some plugin having special behavior around this.

So mayyybe some sort of buffer list management plugin that exists somewhere cares about this?
This stands in contrast to the likely hundreds of vim plugins that are currently actually have subtle bugs today because their authors didn't think deeply about what BufDelete means and took it at face value. This would be a fix to bring behavior in line with those expectations.

I am not going to change the existing behaviour.


GIT_URL is dead

https://github.com/IBM-Bluemix/ibm-cloud-developer-tools/blob/master/linux-installer/idt-installer#L13 has GIT_URL="https://github.com/ibm-cloud-tools/ibm-cloud-developer"
This URL is dead.

Thanks @ianpartridge -- I updated url.