Setting STRAPI_URL on runtime instead of on build

Hi, I'm trying to setup a nuxt3 - strapi v4 project to be able to run in a k8 cluster.
I'd like to be able to set the strapi url when the container starts by using the environment variables in my hosting provider.
But it seems the environment variable is fetched only on build, not on start.
I have been trying to use the publicRuntimeConfig because Nuxt3 docs say you can overwrite runtimeconfig values in production if you use the same variable name but i can't get it to overwrite the strapi url. Is this possible?
I have been trying to get this to work locally using:
nuxt.config.ts:
  ... publicRuntimeConfig: {     STRAPI_URL: process.env.STRAPI_URL || 'localhost:1337'   }, ...
.env:
STRAPI_URL = "http://oldruntime.com"
npm run build STRAPI_URL='http://clruntime.com' npm run start
In my nuxt app i then log useStrapiUrl() and useRuntimeConfig():
useStrapiUrl():
http://oldruntime.com/api
useRuntimeConfig():
STRAPI_URL: "http://clruntime.com", strapi:{   cookie: {}   prefix: "/api"   url: "http://oldruntime.com"   version: "v4" }
So the STRAPI_URL is being overwritten in runtime but the strapi.url is still the same url as on build.

I want to ask you if you have solved this problem

Hi,
We found out that to override the public runtime config variables you need to set the config key to NUXT_PUBLIC_STRAPI_URL instead of STRAPI_URL. This was badly documented in the nuxt3 docs but this has been improved.
If you set up your runtime config like this:
runtimeConfig: {     public: {       strapi: {         url: 'http://localhost:1337',       }     }   },
You can override this on runtime by using NUXT_PUBLIC_STRAPI_URL=api.live.com
This could be a nice improvement to the docs of this module itself as well but that might be outside of the scope.

You can override the url through the environment variable NUXT_PUBLIC_STRAPI_URL.
This is possible due to the implementation in the module: https://github.com/nuxt-community/strapi-module/blob/8db8b671aa4300cd5edae4a787c328dc55acc5b6/src/module.ts#L81-L97

Hi, sorry if this question sounds a bit silly.
I can't way to proxy the strapi url to the url where nuxt is running.
Lets say I'm running my strapi on a url https://cms.lalala.io
And I've my nuxt application running on https://my-nuxt.io
Using the next3 proxy i can forward all traffic from  https://my-nuxt.io/api to  https://cms.lalala.io
    vite: {
        server: {
            proxy: {
                "/api": {
                    target: 'https://cms.lalala.io'
                },
            },
        },
    },

The problem is how to point this plugin to accept it own location as the url to be used.
    strapi: {
        url: ///// HERE I can't manage to tell @strapi/nuxt to use it own location.
        prefix: '/api',
        version: 'v4',
        cookie: {},
        cookieName: 'strapi_jwt',
    },

Please any help is welcome.


secure view has changed

Has the underlying view changed? can you please assist as this is a secure view
dbtcli-snowflake % dbt run --model l10_staging --no-version-check
Running with dbt=0.21.0
[WARNING]: Configuration paths exist in your dbt_project.yml file which do not apply to any resources.
There are 2 unused configuration paths:

models.dbtcli_snowflake.l30_mart
models.dbtcli_snowflake.l20_transform

Found 2 models, 0 tests, 0 snapshots, 0 analyses, 359 macros, 0 operations, 0 seed files, 2 sources, 0 exposures
00:29:02 | Concurrency: 200 threads (target='dev')
00:29:02 |
00:29:02 | 1 of 2 START view model l10_staging.base_knoema_fx_rates............. [RUN]
00:29:02 | 2 of 2 START view model l10_staging.base_knoema_stock_history........ [RUN]
00:29:04 | 2 of 2 ERROR creating view model l10_staging.base_knoema_stock_history [ERROR in 1.98s]
00:29:04 | 1 of 2 OK created view model l10_staging.base_knoema_fx_rates........ [SUCCESS 1 in 2.10s]
00:29:04 |
00:29:04 | Finished running 2 view models in 5.12s.
Completed with 1 error and 0 warnings:
Database Error in model base_knoema_stock_history (models/l10_staging/base_knoema_stock_history.sql)
002057 (42601): SQL compilation error:
View definition for 'KNOEMA_ECONOMY_DATA_ATLAS.ECONOMY.USINDSSP2020' declared 12 column(s), but view query produces 13 column(s).
compiled SQL at target/run/dbtcli_snowflake/models/l10_staging/base_knoema_stock_history.sql
Done. PASS=1 WARN=0 ERROR=1 SKIP=0 TOTAL=2

This has been resolved.  Knoema fixed the view definition.


WIP: Performance improvements to low-barrel example

Includes new functionality for suspending layouts.

Looks good to me - although it is a shame we need suspend layout :-(
I think this example illustrates a couple of important points:

Using components to build more complex examples, I like the way that each of the charts is an independent component.
Eventing - you'd probably expect each chart to handle the events exposed by the others, but instead the react / flux style architecture means that events always bubble up to the top-level code which re-renders everything.

It does feel like the tooltip should be rendered via the legend component. I had a quick go at that here:
https://github.com/ColinEberhardt/d3-financial-components/commit/a3a04611ddaaaa95612720b4302c0a6d27328bea
One issue I had was that the legend needs to be rendered via data-join, whereas in yours, as it decorate the crosshair, it's 'lifecycle' is already managed.
I do wonder whether we need an SVG legend as well?

Thanks for the feedback, I've also just spotted that my last minute refactoring has broken it.... I'll convert this to WIP for now.

Forgot to mention, I think the point snapping is wrong - should snap on y-only
On Tue, Oct 20, 2015 at 9:07 a.m., ScottLogic/d3fc
<EMAIL_ADDRESS>wrote:
Thanks for the feedback, I've also just spotted that my last minute refactoring
has broken it.... I'll convert this to WIP for now.
—
Reply to this email directly or view it on GitHub
[https://github.com/ScottLogic/d3fc/pull/636#issuecomment-149470090] .[https://github.com/notifications/beacon/ABDBfvYePhAhLDC02N4HS1POQDF85gzrks5o9e2tgaJpZM4GRjTo.gif]

Agreed, will fix.
On Tue, 20 Oct 2015 at 09:08 Colin Eberhardt<EMAIL_ADDRESS>wrote:

Forgot to mention, I think the point snapping is wrong - should snap on
y-only
On Tue, Oct 20, 2015 at 9:07 a.m., ScottLogic/d3fc
<EMAIL_ADDRESS>wrote:
Thanks for the feedback, I've also just spotted that my last minute
refactoring
has broken it.... I'll convert this to WIP for now.
—
Reply to this email directly or view it on GitHub
[https://github.com/ScottLogic/d3fc/pull/636#issuecomment-149470090] .[
https://github.com/notifications/beacon/ABDBfvYePhAhLDC02N4HS1POQDF85gzrks5o9e2tgaJpZM4GRjTo.gif]
—
Reply to this email directly or view it on GitHub
https://github.com/ScottLogic/d3fc/pull/636#issuecomment-149470514.

[ ] Investigate legend rendering
[ ] Create rAF component
[ ] Create new namespace
[ ] Fix layout suspension

I should update the yahoo example to follow these patterns at some point as well
...
On Tue, Oct 20, 2015 at 9:10 a.m., ScottLogic/d3fc
<EMAIL_ADDRESS>wrote:

Investigate legend rendering
Create rAF component
Create new namespace
Fix layout suspension

—
Reply to this email directly or view it on GitHub
[https://github.com/ScottLogic/d3fc/pull/636#issuecomment-149471033] .[https://github.com/notifications/beacon/ABDBfv-P24zhxjM1aruTk08Oynw2y6sQks5o9e5tgaJpZM4GRjTo.gif]

I've done everything apart from the legend rendering. The current legend implementation which uses an HTML table doesn't easily fit into the structure, I think an SVG implementation would be a good idea but out of the scope of this PR.

WIP removed ready for review.

Great, I'll have a look on the train home ...
On Tue, Oct 20, 2015 at 12:14 p.m., ScottLogic/d3fc
<EMAIL_ADDRESS>wrote:
I've done everything apart from the legend rendering. The current legend
implementation which uses an HTML table doesn't easily fit into the structure, I
think an SVG implementation would be a good idea but out of the scope of this
PR.
—
Reply to this email directly or view it on GitHub
[https://github.com/ScottLogic/d3fc/pull/636#issuecomment-149530586] .[https://github.com/notifications/beacon/ABDBfhgeU7sRs4KGgmi7TAgUnPjXb4Yiks5o9hmmgaJpZM4GRjTo.gif]

Thanks, one final thing I've (just) done is to wrap everything in a root component example.lowBarrel.

:+1:


fix: request jupyterlab installation, not jupyter (outdated metapackage)

The fact that the error message said jupyter not jupyterlab made me install the wrong package. jupyter is a really outdated metapackage (last update 2015).

I don't want to recommend lab - I for one don't use it, and I know a lot of our community doesn't either. I only intended 'jupyter' here to mean 'whatever version of jupyter you like'!


add icon.png, logo.png

This will add an icon to HA's integrations page, HACS updates page, etc.
icon:

logo options:

After discussing icon branding with Frenck, it seems we should stick to manufacturer icons.
In that case - would it be possible to symlink directly to the "airtouch" directory, since it already contains the brand icons?

Added the icons via home-assistant/brands pull request.


API enhancement: add MapSettings::new_with_map_type()

Currently MapSettings::new() is locked to the TilemapMeshType::Square map type.
It would be nice to have a MapSettings::new_with_map_type() that would accept the different TilemapMeshType options.

No longer relevant with the new API.


New version: PortSwigger.BurpSuite.Professional version 2024.5.5

Created by 🥟 Dumplings in workflow run #6299.

Log
Updated: 2024.5.4 → 2024.5.5
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

This package installs and launches normally on a Windows 10 VM.


Can't highlight Turkish letter İ (dotted I)

To Reproduce

Go to 'https://tr.wikipedia.org/wiki/Çokludoğrusal_harita'
Select the word 'İki' in the beginning of the second paragraph
Right click and click 'Save highlights'

Expected behavior
The word 'İki' should get highlighted.
Actual behavior
The word 'İki' gets saved to the highlights, but the word itself on the page is not highlighted.
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 127.0.6533.72 (Official Build) (64-bit)

Thanks for bug report! Will be fixed in upcoming update


Cache

Hello,
Cache for only one day, it's possible?
Thanks

Hey,
if I understand you correctly you want a feature, that saves the current image of the day to a local caching directory, so that during a 24 hour period of time no further API requests for this image are necessary? Is this correct?
I like the idea and this should be easy to implement. I'll look into it!

Hi again,
Any changes? :)

Sorry for the late response. I finally found time to work on this: See the caching branch. A few small details are missing, but not much. I'm going to finish this in the next days. Feel free to test it ;).

@emresaracoglu This feature is available now.

I must have overlooked your first message. I'm sorry about that.
Thank you for your work.
cc @grubersjoe


I want to combine data using MakeCohortVcf after processing 500 samples multiple times in cohort mode

I was able to complete the processing of 500 samples in cohort mode using GATKSVPipelineBatch.wdl and would like to merge the results according to the instructions in the README.
I would like to follow the GATK-SV workflow for merging because the result of GATK-SV cohort mode is very nice and removes a lot of noise.
How do I pass the output of GATKSVPipelineBatch.wdl as input to MakeCohortVcf? An example of input.json or a script to create input.json would be very nice.

Hi @c2997108, GATKSVPipelineBatch runs MakeCohortVcf already. If you ran all 500 samples together, then output you're looking for is the "clean_vcf." We have some new downstream filtering steps that you can run to further reduce false positives and are documented in #695.

Thanks for your reply, mwalker174. Yes, as per your comment every 500 samples are merged and the output clean_vcf files are great. But I have 2000 samples and the README says to process 100-500 each, so I want to merge 500 x 4 outputs.
Or could I use GATKSVPipelineBatch.wdl to process the 2000 samples?

I see, so GATKSVPipelineBatch is designed for cohorts consisting of a single batch. If there are multiple batches, we run each module individually. One important step is MergeBatchSites, which is run prior to genotyping and ensures that all sites across the cohort get genotyped into every batch. Better documentation and a Terra workspace are coming soon for this.
If you saved all the outputs from GATKSVPipelineBatch, you should be able to pick up at that step. See the Quickstart section of the readme for directions on how to build example inputs. You'd want to start re-running from MergeBatchSites.
If re-running is not an option for you, you could attempt to simply cluster the cleaned vcfs from all 4 batches using GATK SVCluster or svtk vcf-cluster. However, this is not a recommended practice.

I am looking forward to "Better documentation and a Terra workspace".
I thought MergeBatchSites.wdl is not used in GATKSVPipelineBatch.wdl but it is needed when merging.
I just want to ask for confirmation because it doesn't seem to be clearly stated in the documentation. Do I need to input the multiple output of GATKSVPipelinePhase1.wdl of the GATKSVPipelineBatch.wdl workflow into MergeBatchSites.wdl for merging, and then continue from GenotypeBatch.wdl in the GATKSVPipelineBatch.wdl workflow?

Yes, that's correct: you can take your existing GATKSVPipelinePhase1 outputs, run MergeBatchSites on the filtered_pesr_vcf and filtered_depth_vcf files from each batch, then run GenotypeBatch and onwards, using the outputs of MergeBatchSites as the cohort_depth_vcf and cohort_pesr_vcf inputs. While the Terra workspace and documentation are still in progress, the draft of the workspace dashboard might still be useful documentation for a multi-batch cohort.


TypeError when running Vuetify locale with Jest and Typescript

🐛 Bug Report
Dependencies

Vue 2.6.11
Vuetify 2.5.5
Jest 24.9.0

To Reproduce
Using the Vuetify import vuetify/src/locale/ throws a TypeError.
 TypeError: Unable to require `.d.ts` file.
    This is usually the result of a faulty configuration or import. Make sure there is a `.js`, `.json` or another executable extension available alongside `en.ts`.

Jest config
module.exports = {
  roots: ["<rootDir>/src"],
  testMatch: [
    "**/__tests__/**/*.+(ts|tsx|js)",
    "**/?(*.)+(spec|test).+(ts|tsx|js)",
  ],
  transformIgnorePatterns: ["node_modules/(?!vuetify)"],
  transform: {
    "^.+\\.(ts|tsx)$": "ts-jest",
    "^.+\\.vue$": "vue-jest",
    ".+\\.(css|styl|less|sass|scss|png|jpg|ttf|woff|woff2)$":
      "jest-transform-stub",
    "^.+\\.(js|jsx)?$": "babel-jest",
  },
  preset: "ts-jest/presets/js-with-babel",
  globals: {
    "ts-jest": {
      diagnostics: false,
    },
  },
};

You didn't add a reproducer
Also don't import .d files, rather ignore them for tests. The message is pretty clear about the issue
Your vuetify has .d file in it which tests do not understand

You didn't add a reproducer
Also don't import .d files, rather ignore them for tests. The message is pretty clear about the issue
Your vuetify has .d file in it which tests do not understand

The problem was fixed using adding:
{
    transformIgnorePatterns: ["node_modules/(?!vue-router|@babel|vuetify)"]
}

To jest.config.js. I thought this was a bug, sorry.
I will be closing the issue.


Portuguese from Portugal translation

Portuguese from Portugal translation

Need to do something more?

Everything fine now. Thanks for contributing!

Thank you for your integration, that I use since the start. Used to make template sensors 😉. 💪👍

@FragMenthor Are you able to do an update of translations?
I have added a lot of translations lately (also for services and playbook feature) so would be nice if pt is also complete.
You can use some compare tool to see what's missing.

Of course, I will try to do it today! Thanks!

Of course, I will try to do it today! Thanks!

awesome!


(🎁) include pyprojectx's dependencies in the pw.lock file

Today my pipelines started failing because of a bad version of filelock being released (3.15.0). Because px does not lock its own dependencies, it just installed this latest version and failed when this was run: .\pw --install-context pdm
So what if px included its own dependencies in the lock file it generates:
> px --lock
...
> cat pw.lock
[self]
requirements = [..., filelock="3.14.0", ...]
hash = "369465d3c365b066bab596c033706a36"

[pdm]
requirements = [...]
hash = "469465d3c365b066bab596c033706a35"

I guess that the wrapper would need logic to extract this information from the lock file.

Pyprojectx should have its own dependencies locked. I'll look into it...

pyprojectx uses the pdm-build-locked plugin to lock its own dependencies. Unfortunately I missed a breaking change in the plugin, causing it to silently stop locking all dependencies.
The result is that pyprojextx versions between 2.0.3 and 2.1.9 fail in its main main reason of existence: a stable dev experience 🙁
Thanks for reporting the issue and apologies for breaking your builds.

The result is that pyprojextx versions between 2.0.3 and 2.1.9 fail in its main main reason of existence: a stable dev experience 🙁

@houbie i just want to let you know that pyprojectx has significantly improved our dev experience. i cant put into words how frustrating it was to deal with other tools (like poetry) that expect to be installed globally and constantly screw up my python installation before i found out about pyprojectx. im shocked that this project isnt more well known.
i really appreciate how quickly you address the issues we've raised. you single handedly made python bearable for me and i hope pyprojectx gets the recognition it deserves


Create reg_export_fullkey.py

Greetings
Credits to the team for creating and maintaining this wonderful library. It is really useful for working with registry files using python.
I worked with the reg_export.py script and realized that it only exports the content of a registry key path/registry value of a key path and displays it to the console.
I created this script to extend the reg_export.py script further by exporting out values of subkeys in registry key path(s) read from a text file. All the content generated would be exported and generated into a .reg file.
Let me know what you think!
Regards
Heng Yeow

Hmm sounds like something I have done before, and it seems I never created any PR for it NiKiZe/python-registry@c2fd43ca018cc8a38d48ca9d08186c4d4f7f39b9
Will your code generate files identical to the windows registry as well?

i tried and tried to make my .reg exports exactly the same layout as what regedit exports, but never got it to work. there are certain conditions where things wrap with an extra byte (and not based on length of the row).
While the content was exactly the same in the .reg files, when you compare them things will be different as regedit .reg files may have 55 bytes on a line and 22 on the next in some cases whereas mine might have 54 on one line and 23 on the next, etc.
in the end the data is the same once imported, but depending on how you are doing your testing and validation i found it very difficult to just diff 2 .reg files and say they are the same
@NiKiZe were you able to generate identical .reg files for a wide variety of data types (numbers, long strings, long binary data, etc)?

@EricZimmerman It was a few years ago so my memory isn't exactly fresh ;) but if I'm not mistaken all the testcases I had at the time got identical output. But indeed it was not as varied testcases as one could want.
@williballenthin do you know of any case where you expect the reg_export.py to only export one level? Or would it be okay to update that sample? (instead of having multiple samples doing almost the same thing)

@NiKiZe So far I've tested inserting the generated .reg files into Windows and they work fine. This script uses the current reg_export.py to generate the required format in the destination .reg file. It doesn't replace it so it's ok to have 2 of them sitting in the same folder :)

@tanhengyeow I think you misunderstood ...
Having 2 files that does almost the same thing is a bit confusing, and it is also harder to maintain 2 files that does almost the same thing. So I think the function should be in the same file not 2 different.
And I'm talking about if the generated files are identical to if you used windows to export the files. (not if they work to import)
The reason for this is so that you can run diff -u org.reg new.reg to test if something has changed, and it could also be used as a way of testing the code itself after changes, (think of it as a way of running unit tests)

if you go the diff route, make sure you use data other than dwords =)
long strings and long binary data (hundreds of bytes) should show the problem across a few hives.


Doesn't seem to work

Using a local home assistant install, the tablet is hooked up to the same wifi. If I go to the ip address in chrome it works but it doesn't do anything in the app.

Try writing the protocol (http:// or https://) and ensure you touched the "done" button in keyboard.

http:// fixed it thanks


Description
Include a summary of the change and relevant motivation/context. List any dependencies that are required for this change.
Fixes # [ISSUE]
Type of Change:
Delete irrelevant options.

Code
Quality Assurance
User Interface
Outreach
Documentation

Code/Quality Assurance Only

Bug fix (non-breaking change which fixes an issue)
This change requires a documentation update (software upgrade on readme file)
New feature (non-breaking change which adds functionality pre-approved by mentors)

How Has This Been Tested?
Describe the tests you ran to verify your changes. Provide instructions or GIFs so we can reproduce. List any relevant details for your test.

@rupeshmohanty Please mention the level of issue(i.e. level 2) on this PR. And kindly review it.

@anika-001 Please review this PR.


[GC Forms UI] Add a block modal - style proposal

The "Add a block" modal needs more research and may be reformatted as we add more "blocks" for users. Carolyn has a proposed design for this, but in the meantime we can use this design to address some of the style and size problems we are seeing.
Private Zenhub Image

Working to add specs to this design :)

Private Zenhub Image

Dave merged updates with https://github.com/cds-snc/platform-forms-client/pull/2666


Allow custom pagination buttons

Allow a <slot> element
Think about specs for what type of element is allowed as "pagination"
It may be difficult as the slider adds/removes pagination buttons.
Maybe split pagination and navigation into a separate component and expect as a replacement something with the same APIs

Idea: split the pagination in a separate WC, and also provide a different type of pagination (e.g. a progress bar)


Limit action runners

Tweak GitHub actions settings.

@ieivanov if you know a better way to get NDTIFF examples we can also switch to it. Low priority since this is only making the test after merging PRs slower.

Sorry for the late review - I'll leave the implementation up to you.


General documentation

General docs, bilingual. Mainly describes the structure of the project and how to contribute.
Changelog:
+ added general docs (SK & EN)

Closing to reopen from a diffferent base branch.


Determine how to add color schemas for Boilerplate UI

The Boilerplate currently ships with Light and Dark color schemes, which can be toggled in a configuration setting. Determine how to add custom color Schemes, and create a new one based on the Apinf Design Book.
User Stories
As an API Administrator (Harri)
I would like to change the colors of our API related page(s)
so they match our company branding and style

Deliverables

[x] Document describing how to create new design scheme (file locations, syntax, etc.)
[x] Example configuration of scheme containing color scheme and logo if possible

Created document in hackpad to save this information:
The hackpad document

Related to #79 Will work with Illya next week on this.


Distance calculating app.

Is your feature request related to a problem? Please describe.
An app that can show the distance between your current position(located using user's current IP) and user searched position on Map.
Additional context.
This app will have a frontend too, to show searched position on the map.

please assign this to me under swoc.

Sure Go ahead @pritamp17


Could you please show how to use the calendar in SwiftUI?

Hi there,
I am new to coding, could you please show how to use the calendar in Swiftui? Thanks a lot

Sorry, SwiftUI is not supported...


Turn FeatureSet into a generic EnumBitset

To allow it to be reused with other enums

Codecov Report

Merging #155 (286cd5d) into main (cfc3cc0) will increase coverage by 0.01%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main     #155      +/-   ##
==========================================
+ Coverage   36.59%   36.60%   +0.01%
==========================================
  Files          80       80
  Lines        8936     8932       -4
  Branches     4741     4740       -1
==========================================
  Hits         3270     3270
+ Misses       2845     2841       -4
  Partials     2821     2821

Impacted Files
Coverage Δ

oi/Features.cpp
45.45% <ø> (+12.12%)
:arrow_up:

oi/EnumBitset.h
40.00% <40.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


this.props.onLayoutChange is not a function

Sorry if I'm doing something obviously wrong, I'm new to react and this component.
I used your example in 6-dynamic-add-remove.jsx. On line 73 (this.props.onLayoutChange) I get an error: "this.props.onLayoutChange is not a function.". If I comment the line out, I can use the code (but there are some issues with resizing). I render in a similar fashion to you:
document.addEventListener('DOMContentLoaded', function (event) {
var contentDiv = document.getElementById('content');
var gridProps = window.gridProps || {};
React.render(React.createElement(AddRemoveLayout, gridProps), contentDiv);
});
Differences: I'm not using your hooks, and I noticed that you are using a specific requires for "PureRenderMixin" whereas requiring React and using React.PureRenderMixin should also work?
You wouldn't happen to know if this is a common pitfall?

I don't believe React.PureRenderMixin is actually defined - is that a 0.13 change? I don't see it locally.
Anyway, yes, just remove that line. It is there because that component, in the demo, updates the little text box that shows the internal state of each layout item. It is safe to remove. See how it is initialized.

I must have seen it somewhere, but it was likely an unofficial source, so I just went with:
var PureRenderMixin = require('react/addons').addons.PureRenderMixin;
Removed the line as per your suggestion and everything seems to be working smoothly, thanks!


When preventDeletion is set to false, the field is removed

Bug description
preventDeletion is deleted from the tenant spec when it's on false.
How to reproduce
Steps to reproduce the behavior:

Provide the Capsule Tenant YAML definitions

kind: Tenant
metadata:
  name: test-tenant
spec:
  owners:
    - clusterRoles:
        - admin
      kind: Group
      name: test-dev-admin-group
  preventDeletion: false

When preventDeletion is set to false in the tenant spec, this field is deleted in the applied spec, because the default is false. Due to this, some of our tenants keep out of sync in our GitOps tooling (ArgoCD), because Argo tries to add preventDeletion every time, and it gets deleted by Capsule.
Other than that, I think when you specifically set a value (even it's false), it should be added in the config spec.

Helm Chart version: (helm list -n capsule-system)
Helm chart 0.6.2

Kubernetes version: (kubectl version)
OpenShift 4.14.18

You can use this until this is fixed:
  ignoreDifferences:
  - group: capsule.clastix.io
    kind: Tenant
    jqPathExpressions:
      - >-
      - '.spec.preventDeletion'


refactor: move height caluclation

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Customize job name for multiple KeyCloak clusters on the same Prometheus PushGateway

Motivation
https://github.com/aerogear/keycloak-metrics-spi/issues/100
What
This Pull Request introduces a new system environnment variable PROMETHEUS_GROUPING_KEY_JOB  to override default "keycloak" job name.
Why
Being able to define our own custom PROMETHEUS_GROUPING_KEY_JOB in order to separate indicators of different clusters
How
Add new system env PROMETHEUS_GROUPING_KEY_JOB
Verification Steps
Add the steps required to check this change. Following an example.

Build the code
Copy the artifact into $KC/standalone/deployments
set system var env to PROMETHEUS_GROUPING_KEY_JOB=keycloak_cluster_A
Make sure the metrics logger is turned on
Play with Keycloak a bit
Look at the following url: "$KC/auth/realms/master/metrics", job name is equals to keycloak_cluster_A

Checklist:

[ ] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Progress

[x] Finished task
[ ] TODO

@pb82 : is-it possible to check this PR please ?

Yep, seems like a small enough change. I'll try to get it in asap.


Fix ouuid_to_uuid in readme to be deterministic

The ouuid_to_uuid function in README is deterministic as uuid_to_ouuid is and should be declared as such. It prevents the following error:
General error: 1418 This function has none of DETERMINISTIC, NO SQL, or READS SQL DATA in its declaration and binary logging is enabled (you *might* want to use the less safe log_bin_trust_function_creators variable)

Thank you for contributing! I've accepted a different PR that changes these functions and also adds the DETERMINISTIC keyword.


Update 087-3Dknots.md

Knot in a knot in a knot
<EMAIL_ADDRESS>

wow 😍. Thank you for this contribution!

❤❤❤


Sync eng/common directory with azure-sdk-tools for PR 1161

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1161  See eng/common workflow

/check-enforcer override

/check-enforcer override


PaginationView - mutating the model results in "Index out of range" crash

Hey,
I think there is a bug concerning PaginationView's currentIndex after mutating the model.
Description

It seems like the currentIndex won't be reset or adjusted (e.g. 0) after the model is replaced by an array of smaller page count and / or the model has completely different array items (--> Identifiable).

Adjusting the currentIndex via the currentPageIndex binding along with the array (model) update did not work either.

As a result, if the updated page count is lower than the currentIndex, the PaginationView crashes --> "Index out of range".

The attached Playground.swift file includes a showcase to reproduce the described behavior / crash.
Playground.swift.zip
Steps to reproduce:

Xcode 12.5
Simulator: iPhone 12 Pro / iOS 14.5
Start App including Playground.swift.zip
Tap the "+"-Button
Swipe to the right (currentIndex = 1)
Tap the "reload"-Button --> crash

Console output:

Swift/Range.swift:304: Fatal error: Index out of range

Debugger:
Stops at: UIHostingPageViewController : 61

viewController.rootView.page = content.content(content.data[viewController.rootView.index])

What I expected:

currentIndex is reset to 0, if the array (model) is replaced by different values (Identifiable)
If the model updates because of an ".append()", currentIndex should not be reset because it points to the same page (same id - Identifiable)

@geraldvoit thank you for such a well-written and detailed bug report, this has now been fixed on master!


Feature Request: Collapse Activity Bar on focus change

VSCode Version: 1.17.1
I've looked through the default settings for the feature, but didn't find it. Also asked on stackoverflow, someone suggested to set it to a hotkey/suggest it on here.

Please clarify what feature you want.
And make sure it is not a duplicate by searching through our database. Thanks

A default setting to collapse the activity bar on focus change. Its current default behavior is to stay opened, and needs to be manually collapsed back into a strip with just the activity bar icons. I looked through the database and didn't find any pertinent info, but I'm pretty new to coding so a lot of the features terminology is unfamiliar to me. Please let me know if it's still unclear, thanks.

Dup of https://github.com/Microsoft/vscode/issues/3742


Add close button to predefined infowindow templates

In https://github.com/CartoDB/cartodb.js/pull/1302 we added support for .js-close element in infowindows. We need to add the following markup to the predefined templates:

  ...

@matallo will existing layers use :point_up: markup as the infowindow template after we change it? thx
cc: @piensaenpixel

Assigning this myself, consider it done, man!


Product roadmap link

closing due to unclear conflict.
I will open a new PR from main to fix the link


Changes to modules int and boolean related to multiple ports and not filtering packets with the same value on these node types.

These modifications are consequence of discussions related to
pr #928

+1

If the tests and samples are OK after that, go for it.

@glima tests are OK for sure. I've tested a few samples using int arithmetic and they looked to be working fine.

merged


overline in markdown

Overline is popular text attribute in datasheets.
Can be done with CSS:
o {
    text-decoration: overline;
}

<o>WR</o>

missing from markdown CSS
can be added as "combining overline" unicode caracter: ̅

I noticed that your method of doing overline doesn't work well with many fonts.  W̅R̅  is what you see.
I used
W̅R̅
 to get this:
W̅R̅
Also, for some reason the things in your example ds that should be overlined are underlined, except for R̅D̅ and W̅R̅


should remappings.txt be ignored or upload to git repo?

I used to upload remappings.txt to git repo, however, when my teammate cloned the repo and forge build the project. it turned out that something wrong with remappings.txt. I'm use macOS while my teammate uses windows. so I wonder should I upload remappings.txt to git repo or should this file be ignored and generated with forge remappings > remappings.txt? thanks

bump @Azleal, per comment above, could you please provide a project sample to reproduce the issue? thank you

@Azleal going to convert this to discussion until more details to figure out if it's a bug or config related. Thank you!


Using PowerCommander in Azure Function App

We are trying to use PowerCommander module for various automation tasks.
Testing in a Function App fails during logon with "connect-keeper -Username $user -Password $pass"
[Information] OUTPUT: Read-Host: C:\home\site\wwwroot\Modules\PowerCommander\AuthCommands.ps1:424
[Information] OUTPUT: Line |
[Information] OUTPUT: 424 | $action = Read-Host -Prompt $prompt
[Information] OUTPUT: | ~~~~~~~~~~~~~~~~~~~~~~~~~
[Information] OUTPUT: | A command that prompts the user failed because the host program or the command type does not support user interaction. The host was attempting to request confirmation with the following message:
Any advice?

Login to Keeper may interrupted by the backend to request:

device approval
two factor authentication
password

In your case it is either 1 or 2
Once device approved or 2FA passed for the particular installation or device connect-keeper will go through.
Device configuration is stored into config.json file in your $HOME/.keeper (or %HOME%.keeper) folder.
You can create config.json file using desktop PowerShell make sure the login is not interrupted then transfer that configuration file to your Azure function home directory.

Login to Keeper may interrupted by the backend to request:

device approval
two factor authentication
password

In your case it is either 1 or 2 Once device approved or 2FA passed for the particular installation or device connect-keeper will go through.
Device configuration is stored into config.json file in your $HOME/.keeper (or %HOME%.keeper) folder. You can create config.json file using desktop PowerShell make sure the login is not interrupted then transfer that configuration file to your Azure function home directory.

I'm pretty sure the account we are testing with has no 2FA or device approval needs.
It has only access to run PowerCommander.
We "commented out" the read-host lines and tested locally, again that works fine.
Running the same modified module in azure function removed the initial error, but prompts for password.
No script error, but waiting for user input.
Could it be some problems creating the config.json? Is it possible to change default path of this file?

It worked by copying the config.json.
Thanks a lot for your help!


faulty-related import statements that are considered faults of omission

Hi @rjust and @Greg4cr,
While performing the analyzis described in #500 I found two bug-introducing patches that could further improved, regarding the changes each patch makes to some import statements.

The current bug-introducing patch for Cli-7 adds an import statement to the buggy version and removes another import statement:

+import java.util.HashSet;
 import java.util.Iterator;
-import java.util.LinkedHashSet;
 import java.util.Set;

as both changes occur in non-consecutive lines of code, in the buggy version, the import java.util.HashSet; statement is considered part of the buggy behavior and the removed import java.util.LinkedHashSet; statement is considered a fault of omission.

Similar, the current bug-introducing patch for Cli-36 (OptionGroup.java file) adds an import statement to the buggy version and removes another import statement:

+import java.util.HashMap;
 import java.util.Iterator;
-import java.util.LinkedHashMap;
 import java.util.Map;

as both changes occur in non-consecutive lines of code, in the buggy version, the import java.util.HashMap; statement is considered part of the buggy behavior and the removed import java.util.LinkedHashMap; statement is considered a fault of omission.
I would suggest we re-order those import statements in the bug-introducing patches as
 import java.util.Iterator;
+import java.util.HashSet;
-import java.util.LinkedHashSet;
 import java.util.Set;

and
 import java.util.Iterator;
+import java.util.HashMap;
-import java.util.LinkedHashMap;
 import java.util.Map;

so that those import statements are only considered buggy-related and not faults of omission (which they are not).
Extra: related to this, the current bug-introducing patch for Cli-1 could also be further improved.
The patch-minimization guide states that code refactoring should be removed from bug-introducing patches

Since refactoring does not affect the behavior of the code, it is not considered part of a bug fix, and the changes introduced during refactoring should be removed from the Defects4J patch.

I wonder whether introducing a custom refactoring of import statements would somehow incentive others to perform other kind of refactors in the bug-introducing patches provided by Defects4j.  Please, let me know what you think about it.

Ah, interesting. I support making this change, as it more accurately reflects the actual bug (and its fix).

I wonder whether introducing a custom refactoring of import statements would somehow incentive others to perform other kind of refactors in the bug-introducing patches provided by Defects4j. Please, let me know what you think about it.

Patch minimization requires some degree of interpretation. I am certain we have made other similar "refactorings" when performing minimization. I am not bothered by a limited amount of refactoring to isolate the bug. I think the key is to emphasize that such refactoring:

Should only be in service of isolating and minimizing the bug.
Should be limited to smallest amount of change required to separate the bug from unrelated code changes.


Cannot read property 'match' of undefined

I am using the Flutter driver, but the error is reported:
Startup info:
desired_caps = {}
desired_caps['platformName'] = 'Android'
desired_caps['platformVersion'] = '9.0'
desired_caps['deviceName'] = 'ce10171a2a6f7d23047e'
desired_caps['app'] = r'D:\sm.apk'
desired_caps['automationName'] = 'flutter'
driver = webdriver.Remote('http://<IP_ADDRESS>:4723/wd/hub', desired_caps)
finder = FlutterFinder()
Encountered internal error running command: TypeError: Cannot read property 'match' of undefined
[W3C]     at Object.exports.processLogToGetobservatory (C:\Program Files\Appium\resources\app\node_modules\appium\node_modules\appium-flutter-driver\build\driver\lib\sessions\observatory.js:118:9)
[W3C]     at Object.exports.getObservatoryWsUri (C:\Program Files\Appium\resources\app\node_modules\appium\node_modules\appium-flutter-driver\lib\sessions\android.ts:34:21)
[W3C]     at Object.exports.startAndroidSession (C:\Program Files\Appium\resources\app\node_modules\appium\node_modules\appium-flutter-driver\lib\sessions\android.ts:25:28)
[W3C]     at processTicksAndRejections (internal/process/task_queues.js:85:5)
[W3C]     at FlutterDriver.exports.createSession (C:\Program Files\Appium\resources\app\node_modules\appium\node_modules\appium-flutter-driver\lib\sessions\session.ts:18:47)
[W3C]     at AppiumDriver.createSession (C:\Program Files\Appium\resources\app\node_modules\appium\lib\appium.js:371:35)
[HTTP] <-- POST /wd/hub/session 500 72181 ms - 647

Same here. I am using the example from this repository.

Could you share the full appium server log as GIST?

I am not sure if that's what you're looking for.
appium-server-logs.txt

Please use the latest one with Appium 2 https://github.com/appium-userland/appium-flutter-driver#installation
[info] [Appium] Appium v1.22.0 creating new FlutterDriver (v0.0.32) session
[debug] [Appium] There are no active sessions for cleanup

Solved following these steps:

Run $ npm install -g appium@next, so it will enable some additional features in your Appium;
Run $ appium driver install xcuitest we need to install the default drivers manually;
Run $ appium driver install uiautomator2 we need to install the default drivers manually;
Run $ appium driver install flutter finally installing flutter driver;
Remove /wd/hub from Appium baseUrl.

hm, 2 and 3 are not necessary since 4 includes them in the package dependency


cmd/ctrl-click doesn't open link in new tab

When cmd/ctrl-click a link chrome (and other browsers I guess) opens the link in new tab. This doesn't work with puppetjs.

Confirmed. Good catch!

@miyconst could you take a look? This should be fixed in puppet-dom.js.

Okay, I disabled morphing for ctrl + click and middle mouse button.

Please also check for event.metaKey, which represents the cmd key on Mac

Sorry, completely forgot about Mac users. Done.


[SP-4308] - Backport of PDI-11089 - Cannot sort preview data (NullPoi…

…nterException ) (7.1 Suite)

Backport of PDI-11089 - Cannot sort preview data (NullPointerException ) (7.1 Suite)

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -f core/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/row/value/ValueMetaBase.java
  core/test-src/org/pentaho/di/core/row/value/ValueMetaBaseTest.java

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/core/test-src/org/pentaho/di/core/row/value/ValueMetaBaseTest.java

:heavy_exclamation_mark: '(' is followed by whitespace. at Line: 948 Column: 28

:heavy_exclamation_mark: ')' is preceded with whitespace. at Line: 948 Column: 38

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -f core/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/row/value/ValueMetaBase.java
  core/test-src/org/pentaho/di/core/row/value/ValueMetaBaseTest.java


[PDI-14185] right-click and selections; delete on job when no selections

@mdamour1976 please review

Build Commands:
Build:

rm -r ~/.kettle; rm -r ~/.pentaho; echo removed hidden folders
ant -Djunit.jvmargs=-XX:PermSize=256m -f ui/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
ui/src/org/pentaho/di/ui/spoon/AbstractGraph.java
ui/src/org/pentaho/di/ui/spoon/job/JobGraph.java
ui/src/org/pentaho/di/ui/spoon/trans/TransGraph.java
ui/test-src/org/pentaho/di/ui/spoon/GraphTest.java
Unit test coverage change:
org.pentaho.di.ui.spoon.trans.TransGraph:

linePercentCoverage :0.39%
methodPercentCoverage :0.48%
complexityPercentCoverage :0.14%
instructionPercentCoverage :0.33%

org.pentaho.di.ui.spoon.job.JobGraph:

branchPercentCoverage :0.35%
instructionPercentCoverageChange : +0.46%
linePercentCoverageChange : +0.76%
complexityPercentCoverageChange : +0.67%
methodPercentCoverageChange : +1.86%

org.pentaho.di.ui.spoon.AbstractGraph:

branchPercentCoverage :21.43%
linePercentCoverage :12.50%
methodPercentCoverage :5.26%
complexityPercentCoverage :12.12%
instructionPercentCoverage :9.24%


spelling, grammar, formatting

@rbaumert thanks for the PR. I fixed some TODO. But you revert them back. :(
Can you please check.


[New Port Request] tlx

Library name: tlx
Library description: Collection of Sophisticated C++ Data Structures, Algorithms, and Miscellaneous Helpers
Source repository URL: https://github.com/tlx/tlx
Project homepage (if different from the source repository): http://tlx.github.io/
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

Hi @iwongdotcn
Thanks for posting this request.
tlx has been added to vcpkg now. Please try to update vcpkg to the latest version and use it.


Tekken 7: Most projectiles cause stuttering on hit or block

Most projectiles cause heavy stutter when they hit or get blocked. Eliza's qcf2 Dark Wave exhibits it. Geese's qcf2 Double Reppuken exhibits it. Both DLC, damn! When I encountered this issue, I was really worried that it was only happening with DLC characters! Fortunately, Akuma exhibits it too with his hcf1 and hcf2 Shakunetsu Hadouken.
Can't really play the game on my laptop just yet like this, especially since Geese and Eliza are both very popular in my scene. :(
I took a video of the issue. It seems like it wastes a little bit of time at the start by practicing against a projectile-less character, but that's because there used to be an issue where the first hit spark and the first counterhit spark would cause stutter and I wanted to catch it on video. But that one seems to have gone away so I just focused on testing the projectiles.
Software information
Tekken 7, medium preset with high textures and medium and low effects.
System information

GPU: Geforce 1060
Driver: 390.48
Wine version: Staging 3.5
DXVK version: 00a452e

Apitrace file(s)
https://drive.google.com/open?id=1GWLUx32Hdb1ygXSvnsA8EMgk-txIR9wr
Log files

d3d11.log: http://paste.ubuntu.com/p/BGDWbpcsry/
dxgi.log: http://paste.ubuntu.com/p/WFHv2qqkkq/

Uh... I didn't realize that the video's audio was terribly desynced, and forgot I wasted all that time doing 1p moves before doing the actual problematic 2p moves. I apologize for both of those things. If you could use a better video, I'll try and fix up my recording script and do a more focused run on just one of the moves that exhibits the behavior.

Okay so I've been watching your commit log closely, and I remember making a point to test master after you made the commit for getting rid of the on-disk shader cache to reduce the stutter that another player was experiencing in their game. I have my wine prefixes on my HDD, so it made sense that it might have fixed it. I remember noticing that the stutter was reduced a lot but not eliminated; in fact, it was identical to what Windows users are currently experiencing. That is to say, the video I initially submitted, that kind of stutter, that's more severe than what Windows users are experiencing, but what they're experiencing is what I'm currently getting.
So I'm trying to figure out the timeframes here to see if you actually fixed this issue, but the patch hit just in time to make it look like it wasn't actually quite fixed, or if the issue still isn't quite fixed and for some reason Windows users are now seeing the same behavior as what we see. And if that is the case, I'm curious if they'll share what the fix is and if we can learn from that.


Macro: Export of canvas to ket file cause app to crash

Steps to Reproduce

Switch to Macromolecules view
Load this mol file Chem.zip (unzip first)
Save canvas to ket file (name - doesn't matter)
Switch to Molecules view
System throws exception: localhost-1702890390888.log
Since that moment - app behavior become unstable.
Switch to Macromolecules view
Switch to Molecules view

Actual behavior
App crashes

Expected behavior
App works as intended.
Desktop (please complete the following information):

OS: win10
Browser Chrome 120.0.6099.110 (Официальная сборка), (64 бит)

Ketcher version [e.g. v2.4.2].

There is a strange symbol "?x" instead of attachment point number.
It is not clear how to create such mol file:

@AlexeyGirin could you please clarify how to reproduce it?

Unable to reproduce


tests.unit.lbrynet_daemon.test_allowed_origin.TestAccessHeaders raises OSError

This seems like an edge case and while uncertain whether it should be ticketed as an issue, I think it makes sense to get feedback on this considering 339 of the 343 tests passed and 3 were skipped. The kernel is 5.12.8-arch1-1 and for context the /etc/hosts file includes only these three lines:
<IP_ADDRESS> localhost
::1 	  localhost
<IP_ADDRESS> X.localdomain X
................................................
ERROR: test_headers (tests.unit.lbrynet_daemon.test_allowed_origin.TestAccessHeaders)
Traceback (most recent call last):
File "/lbry/lbry-sdk/lbry/extras/daemon/daemon.py", line 470, in start
await rpc_site.start()
File "/lbry/lbry-sdk/lbry-venv/lib/python3.7/site-packages/aiohttp/web_runner.py", line 102, in start
reuse_port=self._reuse_port)
File "/usr/local/lib/python3.7/asyncio/base_events.py", line 1389, in create_server
% (sa, err.strerror.lower())) from None
OSError: [Errno 99] error while attempting to bind on address ('::1', 5299, 0, 0): cannot assign requested address
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/lbry/lbry-sdk/lbry/testcase.py", line 135, in run
self.loop.run_until_complete(self.asyncSetUp())
File "/usr/local/lib/python3.7/asyncio/base_events.py", line 574, in run_until_complete
self.run_forever()
File "/usr/local/lib/python3.7/asyncio/base_events.py", line 541, in run_forever
self._run_once()
File "/usr/local/lib/python3.7/asyncio/base_events.py", line 1778, in _run_once
handle._run()
File "/usr/local/lib/python3.7/asyncio/events.py", line 88, in _run
self._context.run(self._callback, *self._args)
File "/lbry/lbry-sdk/tests/unit/lbrynet_daemon/test_allowed_origin.py", line 79, in asyncSetUp
await self.daemon.start()
File "/lbry/lbry-sdk/lbry/extras/daemon/daemon.py", line 475, in start
raise SystemExit()
SystemExit

Ran 343 tests in 57.703s
FAILED (errors=1, skipped=3)

Thanks for the detailed report @7astro7. We'll check it out soon.
Can we show you some https://lbry.com/faq/appreciation?

I'm not sure this is a real problem. This message appears only if the lbrynet daemon is already running, but it doesn't if it's not running.
Start the daemon.
python lbry/extras/cli.py start

In another terminal, run the test.
python -m unittest tests.unit.lbrynet_daemon.test_allowed_origin.TestAccessHeaders

======================================================================
ERROR: test_headers (tests.unit.lbrynet_daemon.test_allowed_origin.TestAccessHeaders)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/git/lbry-sdk/lbry/extras/daemon/daemon.py", line 480, in start
    await streaming_site.start()
  File "/opt/git/lbry-sdk/lbry-venv/lib/python3.7/site-packages/aiohttp/web_runner.py", line 104, in start
    reuse_port=self._reuse_port)
  File "/usr/lib/python3.7/asyncio/base_events.py", line 1389, in create_server
    % (sa, err.strerror.lower())) from None
OSError: [Errno 98] error while attempting to bind on address ('<IP_ADDRESS>', 5280): address already in use

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/git/lbry-sdk/lbry/testcase.py", line 135, in run
    self.loop.run_until_complete(self.asyncSetUp())
  File "/usr/lib/python3.7/asyncio/base_events.py", line 574, in run_until_complete
    self.run_forever()
  File "/usr/lib/python3.7/asyncio/base_events.py", line 541, in run_forever
    self._run_once()
  File "/usr/lib/python3.7/asyncio/base_events.py", line 1778, in _run_once
    handle._run()
  File "/usr/lib/python3.7/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/opt/git/lbry-sdk/tests/unit/lbrynet_daemon/test_allowed_origin.py", line 79, in asyncSetUp
    await self.daemon.start()
  File "/opt/git/lbry-sdk/lbry/extras/daemon/daemon.py", line 486, in start
    raise SystemExit()
SystemExit

----------------------------------------------------------------------
Ran 1 test in 0.122s

FAILED (errors=1)

But if the daemon is not running, the test passes successfully.
python lbry/extras/cli.py stop

python -m unittest tests.unit.lbrynet_daemon.test_allowed_origin.TestAccessHeaders

.
----------------------------------------------------------------------
Ran 1 test in 0.694s

OK

Alright, I wasn't sure so will close this issue. @lyoshenka the kind offer is appreciation enough dwai


Maximum night exceed because user can select dates before start date

Hi,
As screenshot attached, user can select date before start date so it exceed maximum nights

This issue fixed at 2.0.4.
Thanks for your reporting!


Parent not receiving message

Hey, I'm not able to receive message on my parent
parent.html
<iframe id="content" src="child.html"></iframe>

parent.js
import 'seamless/build/seamless.parent'

$(document).ready(() => {
    let child = $("iframe#content").seamless();
    child.receive(function(data, event) {
        // Print out the data that was received.
        console.log(data);
    });
});

child.js
import 'seamless/build/seamless.child'

let parent = window.seamless.connect();

// Send a message
parent.send({
    myparam: 'This is anything you want it to be...'
});

The problem, as far as I can see, is that on the parent.receive the check if (data.__id && (data.__id === _self.id)) fails, because the data.__id is 0. Looks like the child doesn't get the parents ID on connect.
Any idea what I did wrong?

Seeing a similar issue here.


Improve Task 03 - Test the prompt in Content Safety instructions to reduce confusion

The role assignment is a bit ambigous in the document:
https://microsoft.github.io/TechExcel-Operationalize-LLMs-with-LLMOps-automation/docs/01_intro_to_llms/01_03.html
This text:
[!IMPORTANT] If you encounter an error message stating Your account does not have access to this resource, please contact your resource owner to get access, make sure to add the role assigment of Azure AI Developer to your user account and wait 10 minutes for the permission to propagate and try again.
could be improved to :

call out that is an azure resource level RBAC Role as opposed to an entra role assignment
It should also be noted that this requires owner rights contributor is not enough

additionally, I would include this example command to perform the assignment via cloud shell:
az role assignment create --role "Azure AI Developer" --assignee "<upn>" --resource-group <rg-name>

This has been added.


Add use_arm option to packer template

This adds another variable to the packer template, so that users can easily enable an arm packer build.
closes #716

It tested it, packer successfully created an arm64 snapshot

This option shouldn't be needed in my opinion, we should by default make both x86 and arm snapshot and autodetect if a node is CAX or not and run the corresponding logic. This would allow to easily have hybrid cluster if someone wants that functionality.
I'll try to implement it today, but PRs are always appreciated just like this one (:

Just tried this, seems to fail deploying due to it still trying to apply an x86 snapshot

I'm working on it right now, will need testers later.

Just tried this, seems to fail deploying due to it still trying to apply an x86 snapshot

Jeah, I was simply focussing on packer

Just tried this, seems to fail deploying due to it still trying to apply an x86 snapshot

Jeah, I was simply focussing on packer

After reapplying with terraform, the cluster was created successfully. Did you change the server types in your kube.tf?

This option shouldn't be needed in my opinion, we should by default make both x86 and arm snapshot and autodetect if a node is CAX or not and run the corresponding logic. This would allow to easily have hybrid cluster if someone wants that functionality.
I'll try to implement it today, but PRs are always appreciated just like this one (:

As it is cheaper and would double the RAM, maybe even make arm the new default for controlplane nodes?

As it is cheaper and would double the RAM, maybe even make arm the new default for controlplane nodes?

Interesting idea, if that works in a hybrid setup I would be all for it. @mysticaltech thoughts?

I've made another PR that needs thorough testing.

Thanks @Kakadus, this is now supplanted by #723 but thanks a lot anyways, PRs are always welcome! 🙏

As it is cheaper and would double the RAM, maybe even make arm the new default for controlplane nodes?

We could do this when arm servers become available in more locations, as they are currently only in FSN it would make Super HA (control planes in different locations) impossible.


PowerShell arrays simply being added to

PowerShell arrays are finicky at best.  When empty they could be null or plain empty.  When a single item, they only return that single item.  Part of what the space cloner does is it will add item to internal arrays (such as environment lists) when a new item is created.  There is a util function Update-OctopusList that makes adding to those lists much easier.  Several of the cloners are not using them, but they should be.

Fixed in 3.2.2


Update images.md

Changing "depencies" to "dependencies"

software depencies. Container images are executable software bundles that can run

software dependencies.


Replace the original default spec with containerd default spec

The original default spec contain seccomp configuration, but some OS do not support this feature, such as ubuntu14.04, and make test-cri always fail. The containerd default spec dosen't contain seccomp, so I think we could replace the default spec with containerd default spec.
The difference between original default spec and containerd default spec:

original default spec
containerd default spec

-
mounts: /run

seccomp
-

process.capabilities.ambient
-

env: TERM=xterm
-

Signed-off-by: Yanqiang Miao<EMAIL_ADDRESS>

@miaoyq Thanks a lot for the comparison! Nice table!
I'm fine with changing to containerd default, and I don't think the default should contain seccomp config, either.
Only one question I have is that why does containerd create /run, that doesn't seem to be the behavior of docker. Is there any particular reason?

I asked on slack channel, let's wait for the response. :)

I'm fine with this PR, and we'll be able to get rid of the /run mount after https://github.com/containerd/containerd/pull/1333 is merged.

LGTM


[Bug] [TFLite] AttributeError: 'ResizeBilinearOptions' object has no attribute 'HalfPixelCenters'

When I load a TFlite model with relay.frontend.from_tflite API, I get an AttributeError, as show below：
Actual behavior
Traceback (most recent call last):   File "tflite_main.py", line 59, in <module>     mod, params = relay.frontend.from_tflite(tflite_model)   File "/workspace/tvm/python/tvm/relay/frontend/tflite.py", line 4205, in from_tflite     op_converter.convert_op_to_relay()   File "/workspace/tvm/python/tvm/relay/frontend/tflite.py", line 276, in convert_op_to_relay     ret = self.convert_map[op_code_str](op)   File "/workspace/tvm/python/tvm/relay/frontend/tflite.py", line 748, in convert_resize_bilinear     return self._convert_resize("linear", op)   File "/workspace/tvm/python/tvm/relay/frontend/tflite.py", line 726, in _convert_resize     half_pixel_centers = resize_options.HalfPixelCenters() AttributeError: 'ResizeBilinearOptions' object has no attribute 'HalfPixelCenters
Environment

TVM:0.13.dev0
TFLite:2.1.0

Root Cause and Solution

Because I followed the documentation guide and install the TFLite with version 2.1.0，so we can modify the documentation with a higher TFlite version(such as 2.10.0)
We can modify this line https://github.com/apache/tvm/blob/f172f6c70d8e72556225877c7522ef13b7b250e2/python/tvm/relay/frontend/tflite.py#L726 with：
try:     half_pixel_centers = resize_options.HalfPixelCenters() except AttributeError:     half_pixel_centers = False
As for ResizeBilinear Op, this attribution default to be false:https://www.tensorflow.org/api_docs/cc/struct/tensorflow/ops/resize-bilinear/attrs#:~:text=TF_MUST_USE_RESULT Attrs-,Defaults to false.,-Public attributes

Guys, do you have any suggestions?
Triage

needs-triage
frontend:tflite

cc @leandron

Hi @ChaojifeixiaDazhuang
I also encounter this problem recently, do you have any solution right now?

seems to be an issue with PyTorch models: https://github.com/pytorch/pytorch/issues/48389


Response code is 302. Body: Something went wrong. Please report issue.

Trying to login with username, password.
 if (static::HTTP_OK !== $response->code) {     throw new InstagramException('Response code is ' . $response->code . '. Body: ' . static::getErrorBody($response->body) . ' Something went wrong. Please report issue.', $response->code); }

have made a pull request to fix this issue
https://github.com/postaddictme/instagram-php-scraper/pull/942


Fix ellipse developer-error messages

As per the title.  Fixes the two documentation issues noted in #3064.

Thanks @nmschulte-aviture!  I think we should word this without the negative like other exceptions, e.g.,
"semiMajorAxis must be greater than or equal to semiMinorAxis."

Updated.  @pjcozzi, can you comment on this concern from #3064?

Last, it looks like this constraint on axes is only applied during construction. If one were to change the value afterward, it seems like this might cause issues in the rendering/graphics API if it's not checking (as it doesn't look like the entity API is).

Thanks @nmschulte-aviture.
The geometry is immutable so the axes can't be changed after construction.


Add gdx-miniaudio support

I think this could be an interesting library for users who don't like the limited support of libGDX for Audio.. MiniAudio is cross platform library, but I didn't make the support for GWT/HTML5 (too much for me!), so for now LWJGL3, Android and iOS are supported.

Looks reasonable to add; is iOS support complete? I have no way of testing since I don't have a Mac. It is a little interesting how Android needs several architectures specified and iOS only has one, but I can see how that could be the case. I'll merge this now; even if there might be an issue with one platform, it's not enough to make the rest of gdx-miniaudio less useful.

@tommyettinger yes iOS and macOS are fully tested and at moment I can't find issues with them. Regarding Android it's mandatory in JNI to pass the AssetsManager from Java code to the native one, as far as I know this is the only way to do that.. Asset directory is private in file system so I guess this is a kind of security reason.. Also iOS has a sort of protected assets directory, but there I can workaround with an Obj-C function, so it's masked.... The only issue at moment is that the library is not able to read audio files directly from a jar package (any desktop), I've opened a issue for that but I don't really know how to workaround.. The best thing is to unpack audio to a temporary directory and read from it


Rebranding changes

Because of the meilisearch’s new rebranding 🎉  we need to apply some changes to our repositories

Change the logo on every README if needed
Change the naming of MeiliSearch into meilisearch. Do not apply any changes regarding the codebase, I mean, do not change MeiliSearchClient into MeilisearchClient since it will break the API of the SDK. Every breaking changes for the users should NOT be done now. We will discuss later in another issue.

TODO

[ ] meilisearch-dotnet
[ ] meilisearch-dart
[ ] meilisearch-go
[ ] meilisearch-java
[ ] meilisearch-js
[ ] meilisearch-php
[ ] meilisearch-python
[x] meilisearch-ruby
[ ] meilisearch-rust
[ ] meilisearch-swift
[x] meilisearch-rails
[ ] meilisearch-symfony
[ ] meilisearch-angular
[ ] meilisearch-react
[ ] meilisearch-vue
[ ] instant-meilisearch
[ ] docs-searchbar.js
[ ] firestore-meilisearch
[ ] gatsby-meilisearch
[ ] strapi-plugin-meilisearch
[ ] vuepress-plugin-meilisearch
[ ] meilisearch-digitalocean
[ ] meilisearch-aws
[ ] meilisearch-gcp
[ ] meilisearch-kubernetes
[ ] cloud-scripts
[ ] js-project-boilerplate
[ ] meilisearch-migration

Be careful I think the aws logo has change see

Not an issue, it's normal since this PR changed the logos ;)
https://github.com/meilisearch/integration-guides/pull/169

I forgot to link the second PRs to this central issue sorry 😢

@curquiza should we add a label to these PR's?

I think it need to be skip-changelog like this https://github.com/meilisearch/meilisearch-symfony/pull/157#event-5964196175

I also forgot to add automatically the skip-changelog label to every PR, the second run of PR was really bad, sorry 😓

This is dooooooooooone!!!!!
Congrats @brunoocasali @alallema and @bidoubiwa for your reactiviy!

I've added skip-changelog to all PR's who haven’t that yet :)


Tensorflow2.0 Training OOM when tf.function retraces excessively

When I am training my model using tf.function CPU memory is leaking and after some steps of training it is getting killed.
System information

OS Platform: - Linux Ubuntu 16.04):
TensorFlow installed from (source or binary): binary
TensorFlow version: 2.0-beta
Python version: 3.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.01
GPU model and memory: p100, 16GB

@tf.function
def train_step(self, x, step, grad_clip=True, clip_value=1.0):
    with tf.name_scope("input_data"):
        inputs = x[:, :-1]
        targets = x[:, 1:]

     with tf.GradientTape() as tape:
          predictions, _ = model(inputs, training=True)
          loss = self.get_loss(targets, predictions)

      with tf.name_scope("gradients"):
          gradients = tape.gradient(loss, self.trainable_variables)
          if grad_clip:
              gradients = [(tf.clip_by_value(grad, -clip_value, clip_value))
                             for grad in gradients]
          self.optimizer.apply_gradients(zip(gradients, self.trainable_variables))

        accuracy = self.get_padded_accuracy(targets, predictions)
        assert self.train_writer is not None
        with tf.name_scope("summary_writer"):
            with self.train_writer.as_default():
                tf.summary.scalar("loss", loss, step=step)
                tf.summary.scalar("accuracy", accuracy, step=step)
       return loss, accuracy

**Stats of CPU memory
After 100 steps - 5.08 GB
After 500 steps -  14.8 GB
After 1000 steps - 27 GB**

When is graph mode of training using tf.function CPU memory gets leaked and after some steps, it gets killed but when I train my model using eager mode it works fine, I am using above code for graph mode training as recommended by TensorFlow 2.0 community.

Hi @akanyaani, is the shape of x varies between each batch?

Yes, the shape of x varies between each batch.

Hi @akanyaani, is the shape of x varies between each batch?

Yes, The shape of x varies between each batch.

That's the reason why! Because tf.function will re-trace the graph due to the variable sequence lengths or variable batch sizes, the number of cached graphs will increase over time if it do not see this shape before., and this OOM occurred. To avoid re-tracing, you might want to specify the input_signature. For detailed explanation, please refer to
https://www.tensorflow.org/beta/tutorials/eager/tf_function
https://www.tensorflow.org/beta/tutorials/text/transformer (search tf.function)

Thanks for your prompt response.
But for that I was calling this method for prevent tensorflow to create new graph tf.compat.v1.get_default_graph().finalize()
But let me try with the input signature

It tried with @tf.function(input_signature=[tf.TensorSpec(shape=(None, None), dtype=tf.int32)]) with variable sequence length but still, it is happening, but when I run with a fix sequence length it works fine but it will cost extra computation.

@akanyaani ,
Can you please provide a code snippet to reproduce the issue reported here.Thanks!

focus on

Hi,
My model has transformer-based encoder-decoder as explained here.
https://www.tensorflow.org/beta/tutorials/text/transformer

Hi, @kkimdev please look at Gpt model written in this repo.
https://github.com/akanyaani/gpt-2-tensorflow2.0

@akanyaani, it looks like the general suspicion is that subsequent calls to train_step cause the function to be re-traced (a slow process that should only happen once or twice), likely due to the parameter configuration. For example, it could be because the shape of x varies, but it could also depend on the value of step.
I'm not sure I can completely follow your instructions for reproducing it, but here is a way to diagnose this issue: add a print call to train_step, like so:
def train_step(self, x, step, grad_clip=True, clip_value=1.0):
    print('tracing', x, step, grad_clip, clip_value)
    with tf.name_scope("input_data"):
      ...

Since the pure Python functions like print are only called during tracing, it's an effective way to tell whether the function is re-traced too much. If it does, it also helps explain why.
We've just updated the API docs as well with a bit more relevant information, it's worth checking out here: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/eager/def_function.py#L989

Hi @mdanatg, Sorry for my late response, Actually I tried with one parameter also but the same thing was happening. So I found that the sequence length of x was the culprit. It was working fine with fix sequence length. And I have noticed that if you restart training with different batch size same problem happens.
def train_step(self, x):
    with tf.name_scope("input_data"):

Thanks! Just to double-check, you are still seeing the leak if you try:
@tf.function(input_signature=[tf.TensorSpec(shape=(None, None), dtype=tf.int32)])
def train_step(self, x):
    print('tracing')
    with tf.name_scope("input_data"):

And you see just one "tracing" message at the console?

I met the same problem. Did you solve the same problem?

Hi @zxk19981227,
Can you give me some idea about your model architecture, then I will be able to give you the answer.
Thanks

@mdanatg I have use “print” in train_step(), it shows that each epoch has a "print" work. and my code killed because of OOM.
are there some solutions for it?
thank you in advance.

I'm sorry  to hear that. But I solved my problems by turning off auto
trace. Maybe your problem is not the 'print' but the auto trace. You can
try this.
I hope it helps
anna<EMAIL_ADDRESS>于2020年3月8日周日 下午6:30写道：

@mdanatg https://github.com/mdanatg I have use “print” in train_step(),
it shows that each epoch has a "print" work. and my code killed because of
OOM.
are there some solutions for it?
thank you in advance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/31842?email_source=notifications&email_token=AJTMZW4BHJKNCR5OCKDNWXDRGNXVLA5CNFSM4IOHPLUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOESGFY#issuecomment-596189975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTMZW3KTZJJQKQFWOLYSDTRGNXVLANCNFSM4IOHPLUA
.

@akanyaani ,
Could you please update on the above comment, let us know if the issue still exist.

@akanyaani ,
Could you please update on the above comment

@akanyaani ,
Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

That's the reason why! Because tf.function will re-trace the graph due to the variable sequence lengths or variable batch sizes, the number of cached graphs will increase over time if it do not see this shape before. Thus OOM occurred. To avoid re-tracing, you might want to specify the input_signature. For detailed explanation, please refer to
https://www.tensorflow.org/beta/tutorials/eager/tf_function
https://www.tensorflow.org/beta/tutorials/text/transformer (search tf.function)

Great, it helps. Could you please add a warning in this function to alert users?


[Bug]: Weird behavior for spread model with additional properties

Describe the bug
For this spec which is simplified from a service definition here, there is a usage of spread model with additional properties (though i think it should be banned). Also, the http body is composed from both the spread thing and azure core traits thing, then the weird thing happened, the http body parameter type will have a query parameter from the trait, which will not happen is the spread model has no additional properties.

I'm wondering if it is a bug for body resolving. But I think a good solution is not allow to spread model with additional properties.
Reproduction
Compile with the playground or the spec provided.
Checklist

[x] Follow our Code of Conduct
[x] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[x] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[x] The provided reproduction is a minimal reproducible example of the bug.

But I think a good solution is not allow to spread model with additional properties.
This is allowed by design, something we explicitly  designed a few month ago, i am not sure I see the issue in teh simplified spec, everthing looks correct no?

But I think a good solution is not allow to spread model with additional properties.

This is allowed by design, something we explicitly designed a few month ago, i am not sure I see the issue in teh simplified spec, everthing looks correct no?

if i spread a model A with additional properties to model B, then the model B will have additional properties? what if the spread happen in operation parameter?
the problem is showed in the screenshot, the body type of http operation will have a property of api version query parameter. if remove the ...Record<unknown>, the query will not in the body type.

oh I see,  yeah seems like we do have a special case that bypass the filterModelProperties part if it has an indexer in the body resolution.
So 2 things:

we probably have a fix to do in http library
We are not filtering properties for any nested types today. This is something the http typekits would I think improve on. This means if you are having an issue with this pattern you would also not exclude nested query params most likely

op getEmbeddings(@bodyIgnore nested: {@query foo: string}, name: string): void;

playground


feat: add TERRAGRUNT_INCLUDE_DIR var

Description
Added a TERRAGRUNT_INCLUDE_DIR variable, to control --terragrunt-include-dir via env variables (usefule in CI/CD pipelines)

TODOs
Read the Gruntwork contribution guidelines.

[x] Update the docs.
[ ] Run the relevant tests successfully, including pre-commit checks.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added parsing of TERRAGRUNT_INCLUDE_DIR env to control --terragrunt-include-dir parameter.
Migration Guide

Hey @hojerst,  All missing env vars were added in another PR, you should wait a bit
https://github.com/gruntwork-io/terragrunt/blob/62dba950bda3d366a9acbdb851ad2a12fcef096d/cli/commands/flags.go#L77

This ENV var have been added in v0.67.0 release, docs.
I am closing this issue.


[nri-statsd] Add DaemonSet option

Is this a new chart
No
What this PR does / why we need it:
This PR is adding an option to run StatsD as a DaemonSet
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #
N/A

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

[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [mychartname])

Hi @thilinapiy , released at https://github.com/newrelic/helm-charts/releases/tag/nri-statsd-1.0.2


WFLY-5391 access log does not recognise common pattern

Linux with security manager Build 3149 outcome was FAILURE using a merge of 6c28a947cd167931f52770a93d7d775da919ba74
Summary: Tests passed: 561, ignored: 183; exit code 1 (new) Build time: 00:03:10

Linux Build 8316 outcome was FAILURE using a merge of 6c28a947cd167931f52770a93d7d775da919ba74
Summary: Tests passed: 561, ignored: 183; exit code 1 (new) Build time: 00:03:35

Windows Build 3352 outcome was FAILURE using a merge of 6c28a947cd167931f52770a93d7d775da919ba74
Summary: Tests passed: 561, ignored: 183; exit code 1 (new) Build time: 00:04:37


[#1194] Product thumbnail refresh after filter change

fixes issue described in #1194
@Igloczek check it please

Its great you fixed it! My fix for that was to use instant flag to disallow lazy loading for first X pictures (it’s in the develop); but would be better to have lazy loading for all objects

I wrote lots of words in this box, but then found a solution and had to remove everything :D
Now we are using mix of v-lazy and v-lazy-container, which seems to be redundant and kidna confusing solution.
But we can do it better :)
v-lazy can take object as a value, so if we will create computed that returns object with all necessary URLs, we can reduce code to this:
imgObj () {
  return {
    src: thumbnail,
    loading: placeholder,
    error: placeholder
  }
}

<div v-if="!instant">
  <img
    :src="placeholder"
    :alt="product.name"
    v-lazy="imgObj"
    height="300"
    width="310"
  >
</div>

Seems quite good. Does it work with the filter changes as it's expected?

I didn't try it.

Could You, please? :D @Igloczek

RODO vs Free time
500:0

:D OK, I'll test it later then or @mercs600 please do If you could

@Igloczek @pkarw it's done

@mercs600 please just fix the lint error and I'm merging it in:
https://travis-ci.org/DivanteLtd/vue-storefront/jobs/383323726

@pkarw I missed it. It's done. Right now I noticed some other bugs on catalog theme, so I will fix it.


Strange error with GDScriptCompiler at _is_class_member_property. Maybe cyclic dependency?

Godot version
v4.0.beta15.official.4fa6edc88
System information
Vulkan API 1.2.231 - Apple M1 Max
Issue description
I'm getting this error: Parse Error: Identifier not found: some_info when running my project.
E 0:00:00:0201   _is_class_member_property: Condition "!nc" is true. Returning: false
  <C++ Source>   modules/gdscript/gdscript_compiler.cpp:63 @ _is_class_member_property()

Error happens on Enemy script on line: print(some_info).
The error is gone if:

I replace enemy: Enemy by enemy on _on_enemy_collision function from Entity script.
or add var x = preload("res://entity.gd") on main scene script.

Looks like a circular dependency problem.
Steps to reproduce
Download the project and run the main scene sandbox.tscn.
Minimal reproduction project
preload 3.zip

I managed to create a MRP. And cleanup the problem. Looks like a cyclic dependency.

@akien-mga this is the issue I mentioned on mastodon.

CC @adamscott

Can you try again with the latest RC? I cannot reproduce with the attached project, I don't get any kind of error.

@vnen I'm downloading the RC3 (It is super slow because tuxfamily servers) to test again. But I noticed I uploaded the project with an uncommented line, and the uncommented line is a workaround to solve the issue.
This way the error doesn't happen:
sandbox.gd:

This way the error happens:
sandbox.gd:

@stephannv thanks for checking. Indeed, if I comment the line I do get the error.

This is definitely because of circular dependency. The base scripts are accepted but since they might not have fully compiled yet it fails to find the member variable.
One solution to this is to rely on data from previous compilation steps instead of searching again for the identifier, which is actually something I wanted to do for a while but never got the chance. Unfortunately that would be a big change so it won't be done before 4.0 stable.

@vnen Do you think it you can fix it for 4.1? Don't hesitate to poke me if you want to work together on this.

4.1. is right around the corner (beta 2 today), so I'm doubtful...

When 4.1 comes out I will upload a MRP using the newer version.

I tried to reproduce this on Windows 11, Godot 4.4.dev7.
I commented line 4 as said in the steps to reproduce.
I don't see the error.
Has this been fixed?

I tried to reproduce this on Windows 11, Godot 4.4.dev7. I commented line 4 as said in the steps to reproduce. I don't see the error. Has this been fixed?

I will try to reproduce it with latest Godot 4.4 in the next days.

@vnen @GreenCrowDev @adamscott this problem doesn't happen anymore on Godot 4.4.dev7. Thanks. I'm closing this issue.


[REEF-160] Fast-fail when protoc version differs from protobuf

Add protoc version check to root pom.
JIRA: [REEF-160] https://issues.apache.org/jira/browse/REEF-160

I will test and merge this

I confirmed this works on Windows.

This PR caused build failed.
At root level, build with command line:
mvn clean install -PDotNet -PBridge -DskipTests
It causes \lang\cpp\reef-bridge-clr\src\main\Cpp\CppBridge\JavaClrBridge\JavaClrBridge.sln build failed

@jwang98052 Please file this as a JIRA such that we can track it.


Support for a different sql backend (e.g. mysql)

Hi - I'd consider working on this as an enhancement : but would appreciate some insight/ advice on it. It is my belief there were some PR for that on the original yhat repo but I can not find it now.
lmk.  stephen boesch

To realize that feature, replace database connection object conn to another sql backend connection objects that supported by pandas https://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.to_sql.html
If you implement that and PR, I'll leave some advices.

Create another SQLDFXXX class for each sql backend.
Create factory method to get matched SQLDF client.


Update to version 1.6.10 (latest) version and DNS Resolution feature

Hi,
we need the DNS resolution feature of v1.6. I updated to the latest version from the webside and added the config option. If you like to merge just download the package.
Cheers
Johannes

Thanks! Can you resolve the merge conflicts and I'll merge + cut a new release?

Conflict should be resolved now.

Released in v8.0.8


[terra-search-field] Select To Component List Slingshot

Bug Report
Description
The select to Component in Healthelife, when opening with multiple recipients, has an issue where if the list of recipients exceeds the dropdown menu, and the user has to scroll on the page, continually slingshots the menu back to the top of the list over and over again. This is especially obvious in smaller mobile devices, and can be reproduced in both iOS and Android devices.
Steps to Reproduce

Reach out to @mstracy13 for login details
Ensure that reply all is turned on for that organization
type in "as" into the select to component, see the dropdown menu appear, and try scrolling on that page. It will continually shift the focus on the page to the top

Expected Behavior
User should be able to scroll with Select To without it shifting
Environment

Component Version:
Browser Name and Version: Any browser
Node/npm Version [e.g. Node 8/npm 5]:
Webpack Version:
Operating System and version (desktop or mobile):

Additional Context / Screenshots
Can't show it in screenshots, but I have video evidence of it if you want me to send it to you directly.

@mstracy13 Can you confirm if the values passed to select option in your form-select component are all unique? or if you could point us to your code we can take a look and see what's going on. Thank you.

I'm not the one who coded this. I'm simply logging a defect I was seeing.
Reach out to Reena Sharma from our team, or maybe Cody Price, since I think he was the one from the Terra team that had designed the Select To Component.

I'm not the one who coded this. I'm simply logging a defect I was seeing.
Reach out to Reena Sharma from our team, or maybe Cody Price, since I think he was the one from the Terra team that had designed the Select To Component.

Have reached out to Reena Sharma, awaiting response.

Concluding that consumer implementation is incorrect and hence closing this issue.


Failed to get ntoskrnl.exe

Running on WIN11 23H2 25992.1000

You are using Windows Insider build version, latest version for public usage is still, 22631, since Windows 11 insider versions have constants changes until become a release i will not check further the issue, make sure you are not using any other thing like an antivirus that can intercept or block the NtQuerySystemInformation and try to fix your self, you are open to write down here what is going on with the fix and i will be glad to add to the main repository

2 solutions to grab ntoskrnl virtual address from user mode

find a driver that reads msr
msr(0xc0000082) = virtual address of KiSystemCall64
pattern scan ntoskrnl.exe from disk to grab rva of KiSystemCall64
(virtual address of KiSystemCall64) - (rva of KiSystemCall64) = ntoskrnl virtual address
find a driver that reads physical memory/maps physical memory to user mode (MmMapIoSpace wont work because it cant map pages that contains page tables)
enumerate possible kernel cr3 values -> enumerate possible ntoskrnl virtual address using cr3 as page table base -> verify cr3+ntrosknl pair
pseudocode

for(uint64 cr3 = 0; cr3 < highestPhysicalAddress; cr3+=0x1000)
    for(uint64 pNtoskrnl: allReadExecutePagesOf(cr3))//this is actually a 4 level for loop to walk page tables that starts from cr3
        if(verify(cr3, pNtoskrnl))
            return {cr3, pNtoskrnl};
return not_found;

This issue will be fixed by granting DebugPrivilege

hard code that shit


Migrate orderbook into coordinator

Follow-up on #155

I found that, but it also did not work for me. https://github.com/diesel-rs/diesel/blob/master/guide_drafts/backend_installation.md

bors r+

bors r+


Change form results endpoint to return data for all form versions

Status
READY
Description

changes /form/:form-id/results to /forms/:form-key/results so that results for all versions of a form are returned

/forms/:form-key/results
/forms/:form-key/results/:form-version


Prepare to upgrade squeeze to OpSet 13

The version 11 and 13 for squeeze/unsqueeze are not compatible. This PR creates two versions for them and changes the existing code for v11 to use ONNXSqueezeV11Op. Now ONNXSqueezeOp is for v13, the corresponding code, such as shape inference, lowering, and rewrite, for v13 is not there yet. @NathanielMcVicar may put his local fix for upset 13 implementation in on top of this PR.
Details in changes:

utils/gen_onnx_mlir.py: use opName, which is a combination of schema.name and version, to provide fine control in ONNXOp.td.inc generation.
All the usage of ONNXSqueezeOP are changed to ONNXSqueezeV11Op. In fact, Squeeze/Unqueeze are used a lot in rewriting and lowering.

Jenkins Linux s390x Build #1676 [push] Prepare to upgrade squee... passed after 50 min

Jenkins Linux amd64 Build #1678 [push] Prepare to upgrade squee... passed after 52 min


Using await for Async API instead of .Result in example

Example in section Executing a point query using the Storage Client Library should use await instead of .Result.

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

ID: cbd87ab1-9a5b-4d23-05ce-d9d5f5bb74a0
Version Independent ID: 0bf2b20a-1830-02e5-67bd-a05074a7236b
Content: Azure storage table design patterns
Content Source: articles/storage/tables/table-storage-design-patterns.md
Service: storage
Sub-service: tables
GitHub Login: @tamram
Microsoft Alias: tamram

@nguyenquyhy Thanks for the feedback! We are currently investigating and will update you shortly.

#reassign: @pauljewellmsft

Thanks for bringing this issue to our attention! The code snippet has been updated and the change should go live later today.
#please-close


Extended handling of the headers in a BarChart

Extended handling of the headers in a BarChart
Added enabling of legend in a BarChart
Added enabling of Grid lines in a BarChart
Added test class for extended BarChart

I merged to branch dev71. https://github.com/QtExcel/QXlsx/tree/dev71
After testing, if there is no problem, I will move to the master branch.

merged to master. https://github.com/QtExcel/QXlsx/commit/d0b1d583a65240082876004a65f0b088b7790fd5


All localhost links in Safari try to launch SAFE Browser instead

Expected Behavior
Click on a localhost link in Safari -> redirect Safari to that link/path
Current Behavior
Click on a localhost link in Safari -> alert window pops up asking to launch SAFE Browser -> confusedly stare at the pop-up, wondering whether I recently downloaded some malware accidentally -> search disk for SAFE Browser -> find it in the Downloads folder and remember that I had downloaded it a year ago but never got around to using it
Possible Solution
Uninstall SAFE Browser
Steps to Reproduce (for bugs)
Demonstrated in this video: https://www.youtube.com/watch?v=-zX5g-WRTm8
Context

It ruined my day.... What am I trying to accomplish? To work in peace on my local projects!
Your Environment

Version or Branch used : 0.10.2
Operating System and version : macOS 10.14.1 (Mojave)
Link to your project : localhost:8888

Nevermind, deleting the localhost:8888 from the link, keeping only the directory fixed it.


Fix links to enterprisecontract.dev

Codecov Report

Merging #714 (756e9a9) into main (477dab8) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #714   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           81        81
  Lines         2730      2730
=========================================
  Hits          2730      2730


Fix seeding from random device w/o getrandom syscall

Use select to wait for /dev/random in readable state,
but do not actually read anything from /dev/random,
use /dev/urandom first.
Use linux define __NR_getrandom instead of the
glibc define SYS_getrandom, in case the kernel headers
are more current than the glibc headers.
Fixes #8215

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This needs a kernel patch to be fully functional:
https://marc.info/?l=linux-kernel&m=155023911204953&w=2
But without the kernel patch it is also an improvement, because
the select will wait until at least 64 bits of entropy are available
instead of the usual 128 bits of entropy.
Also by not reading out data from the /dev/random it prevents
other application which use syscall getrandom from starving.

Actually with the kernel patch the syscall getrandom will be faster, because the
blocking pool will be completely empty and need to wait for at least the number
of entropy bits, in /proc/sys/kernel/random/read_wakeup_threshold, 64 by default.

Any update on this?

I have meanwhile posted a V5 of the linux patch here:
https://marc.info/?l=linux-kernel&m=155059615110666&w=2
I got a first reply from Theodore Y. Ts'o, but I am still waiting for his final decision.
It is possible that the patch needs to be split up in smaller pieces addressing
single issues, or some more changes will be required.
The main issues this patch addresses, are:

the first few bytes from /dev/random after system boot can have
less entropy in the input_pool than expected.  Depends on the configuration.
The default configuration implies 64 bits entropy.
This is since linux 2.6.12 earlier versions are not in the git archive.
/dev/random is readable before the /dev/urandom is fully seeded.
This started with linux v4.8 and prevails till today.
Before linux v4.8 it was true, that reading one byte from /dev/random
did in fact ensure that /dev/urandom is properly seeded, but the
first few bytes from /dev/random had less entropy than expected.

This fixes also some issues, I did not notice initially:

Actively reading from /dev/random can prevent the CRNG from
ever be fully seeded.
This started with linux v4.8 and prevails till today.
On system with little entropy input there it can happen that select on /dev/random
indicates readability and then the CRNG is reseeded and takes so much entropy
out of the input pool that the select changes state, without any read from /dev/random
This started with linux v4.8 and prevails till today.
Before v4.8 the reseeding did reserver the last 2*random_read_wakeup_bits,
and I want to restore that behavior.
If a system runs long enough that the jiffies overrun the CRNG re-seeding
does not happen every 5 minutes, but immediately, since the condition for a
reseed time_after(crng_global_init_time, crng->init_time)  will start to trigger,
after 248 days system uptime, there are systems that run such long time.
This would make /dev/random block unexpectedly.
This started with linux v4.8 and prevails till today.

That's all related to the kernel, and it's good to know. Does it
have any effect on this PR? Can it be merged?

Well yes, I think so.  @levitte do you agree?
It is theoretically possible to see select blocking but only when you use a mis-configured
cross compiler, like I did, or install a new kernel without updating the kernel headers.
But over time, the kernel will be fixed.

Merged to master as 38023b8, and 1.1.1 as c352bd0. Thanks!

Let me know if you want more info - or want me to try something out on this platform.

@paulidale @jon-oracle

OK, something more to patch-out downstream. :(

Should we change the select on /dev/random a Linux only thing?

That would make sense.


Feature: Add domain rotation to mitigate Perlin squareness.

Contains
An improvement to the existing Perlin noise algorithm.
Perlin noise, since deprecated in this project in favor of Simplex, produces visually significant square bias as a flaw of the algorithm's underlying design. However, the noise is still used in the project in some places. Its visual results can be greatly improved for plane-based voxel world generation, using a simple coordinate-space rotation on the 3D noise that rotates Y to point up the grid's main diagonal (<1, 1, 1>).
Without domain rotation:

With domain rotation:

How to test
You can generate the above images by temporarily dropping the following RenderPerlin.java into the same directory as PerlinNoise.java and running its main method through IntelliJ. The first image was generated without this PR's change, and the second was with it.
package org.terasology.engine.utilities.procedural;

import java.awt.image.BufferedImage;
import javax.swing.*;
import java.awt.Color;

public class RenderPerlin {
    private static final int WIDTH = 512;
    private static final int HEIGHT = 512;
    private static final float FREQ = 1.0f / 32.0f;
    private static final boolean RIDGED = true;

    public static void main(String[] args) {

        PerlinNoise noise = new PerlinNoise(0L);

        // Image
        BufferedImage image = new BufferedImage(WIDTH, HEIGHT, BufferedImage.TYPE_INT_RGB);
        for (int y = 0; y < WIDTH; y++) {
            for (int x = 0; x < HEIGHT; x++) {
                double n = noise.noise(x * FREQ, 0, y * FREQ);
                n = RIDGED ? 1 - Math.abs(n) : n * 0.5 + 0.5;
                if (n < 0) n = 0;
                else if (n > 1) n = 1;
                int v = (int)(n * 255);
                Color color = new Color(v, v, v);
                image.setRGB(x, y, color.getRGB());
            }
        }

        // Display it
        JFrame frame = new JFrame();
        JLabel imageLabel = new JLabel();
        imageLabel.setIcon(new ImageIcon(image));
        frame.add(imageLabel);
        frame.pack();
        frame.setResizable(false);
        frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        frame.setVisible(true);
    }
}

It is worth noting that I experience a crash (issue #4801) while trying to configure and/or add a custom world type, in this example the Perlin option. The code I provided verifies the modified noise generation itself is sound, but you may wish to perform further testing before merging this PR.

@KdotJPG hmm, wonder why its deprecated. nothing inherently wrong with it though :?

@KdotJPG have you joined the discord. the expertise is really well appreciated. :D

@KdotJPG I think we're nearly ready to merge this, but I'd like to get the comments that we have in Github now into docstrings - could try to finish this up, please?
From the org meeting on 2021-07-03:

we discussed the topic and will go ahead with merging the changes and un-deprecating Perlin noise

Oh - working on it! I had misunderstood hearing about this thinking it had already been merged! Will get these changes in tonight.