Ensure @rematch and nextEmbedded can be used together in Monarch grammar

Similar to the repro case I created for #90266, I created a local
playground with the following:
const DEMO_LANG_ID = "demo";
const SQL_QUERY_START = "(SELECT|INSERT|UPDATE|DELETE|CREATE|REPLACE|ALTER|WITH)";

const languageConfiguration = {
	tokenizer: {
		root: [
			[
				`(\"\"\")${SQL_QUERY_START}`,
				[
					{
						"token": "string.quote",
					},
					{
						token: "@rematch",
						next: "@endStringWithSQL",
						nextEmbedded: "sql",
					},
				]
			],
			[
				/(""")$/,
				[
					{
						token: "string.quote",
						next: "@maybeStringIsSQL",
					},
				]
			],
		],
		maybeStringIsSQL: [
			[
				/(.*)/,
				{
					cases: {
						[`${SQL_QUERY_START}\\b.*`]: {
							token: "@rematch",
							next: "@endStringWithSQL",
							nextEmbedded: "sql",
						},
						"@default": {
							token: "@rematch",
							switchTo: "@endDblDocString",
						},
					}
				}
			],
		],
		endDblDocString: [
			[
				"[^\"]+",
				"string"
			],
			[
				"\\\\\"",
				"string"
			],
			[
				"\"\"\"",
				"string",
				"@popall"
			],
			[
				"\"",
				"string"
			]
		],
		endStringWithSQL: [
			[
				/"""/,
				{
					token: "string.quote",
					next: "@popall",
					nextEmbedded: "@pop",
				},
			]
		],
	},
};
monaco.languages.register({
	id: DEMO_LANG_ID,
	extensions: ['.example'],
});
monaco.languages.setMonarchTokensProvider(DEMO_LANG_ID, languageConfiguration);

const value = `\
mysql_query("""SELECT * FROM table_name WHERE ds = '<DATEID>'""")

mysql_query("""
SELECT *
FROM table_name
WHERE ds = '<DATEID>'
""")
`;

var editor = monaco.editor.create(document.getElementById("container"), {
	value,
	language: DEMO_LANG_ID,

	lineNumbers: "off",
	roundedSelection: false,
	scrollBeyondLastLine: false,
	readOnly: false,
	theme: "vs-dark",
});

Without my change to monarchLexer.ts, I get this error in the console:

errors.ts:26 Uncaught Error: demo: cannot pop embedded mode if not inside one

But with my change, I see a Python-esque multiline string literal with
SQL syntax highlighting within it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: bolinfest sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

/cc @alexdima

Thank you!


Consider using Snafu to define Error types

As a follow-up of #84 where we remove Snafu on jf-utils's TaggedBlobError temporarily.
We aim to try to use Snafu among all jf-* crates and enabled for all the error types we defined and we internally depend on (such as TBase64Error etc)
cc @jbearer

Jeb and I briefly discussed enforcing a library-wide anyhow-only policy. Anyhow is far more ergonomic when it's used everywhere. Otherwise there's a constant need for boilerplate code to translate among error types and cruft at every ?.


Destroy DNS before destroying the VPC

/lgtm

@sjenning @csrwng can you re-tag? The lgtm got lost after I moved the VPC deletion to the end

/lgtm


Avoid SELECT DISTINCT SQL queries where possible

Every so often, running mytop shows a bunch of queries running for many seconds (sometimes over 10), in state Copying. Copying means copying to a temporary table, and is inefficient.
It looks as though using SELECT DISTINCT queries forces the creation of temporary tables [1]. We should get rid of these unless there's no way around them.
[1] https://dba.stackexchange.com/questions/34674/mysql-performance-tuning-queries-stuck-on-copying-to-tmp-tables

Let's re-open this to remind ourselves that there are still 2 spots where SELECT DISTINCT is in use.

This is what is causing names to show up on the Archive description more than once, when the reader has more than one section, correct? It would be nice to get this fixed.

This is what is causing names to show up on the Archive description more than once, when the reader has more than one section, correct? It would be nice to get this fixed.

... maybe? :) Could you link to an example? If that's indeed the case, we can revert the part of https://github.com/LibriVox/librivox-catalog/commit/29255e2fb5e94088f5aecce22b224a91eb2b0014 that's causing it.

This is what is causing names to show up on the Archive description more than once, when the reader has more than one section, correct? It would be nice to get this fixed.

... maybe? :) Could you link to an example? If that's indeed the case, we can revert the part of 29255e2 that's causing it.

Right now this one shows duplicate readers: https://archive.org/details/fairylady_2112_librivox but we MCs go in and edit the description, so you won't see it for long.


Reveal is outputting debug message to console

Reveal is writing "focus" to the console log in the open() method: console.log('focus'); I assume this is a debug message which should not have been included in a production build.

Thanks @silentmiles,
It has been already fixed here https://github.com/zurb/foundation-sites/commit/1e0a3d4da95f25427058d9908fee26be8c75b2c1


bug: docs sections are broken

Describe the bug
Anything else than type conversions is not rendered as a proper section on the docs site.
To Reproduce
N/a
Expected behavior
.
Code Snippet
.

Version of software
Docs
Additional context
.
Code of Conduct

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

Hi @andig,
Yes, the documentation site needs a lot of attention, and type conversions are the only ones I've done at this time!
I left this issue open because it is indeed an issue, but it will serve more as a "feedback" issue for the documentation. I will write proper documentation over time as a full group of functions are migrated.
Thanks for the time according to the project !

The documentation are updated with the v0.5.0 https://sprout.atom.codes ✔
Thanks for the report


Make ligature support optional

Add option to enable/disable ligature.

Published.
<EMAIL_ADDRESS>


docs: Outdated GitHub Pages documentation

Describe the bug
The documentation at https://vitejs.dev/guide/static-deploy.html#github-pages is outdated. Previously it would suggest creating a shell script that built the application and git pushed the dist folder to the remote. However, at this moment it is already possible to automate this similar to how it's documented at https://vitejs.dev/guide/static-deploy.html#github-pages for GitLab.
Reproduction
https://vitejs.dev/guide/static-deploy.html#github-pages
Steps to reproduce
Go to https://vitejs.dev/guide/static-deploy.html#github-pages and observe the outdated documentation. I have built an example GitHub Actions workflow that would substitute the bash script shown there.
# Simple workflow for deploying static content to GitHub Pages
name: Deploy static content to Pages

on:
  # Runs on pushes targeting the default branch
  push:
    branches: ["master"]

  # Allows you to run this workflow manually from the Actions tab
  workflow_dispatch:

# Sets permissions of the GITHUB_TOKEN to allow deployment to GitHub Pages
permissions:
  contents: read
  pages: write
  id-token: write

# Allow one concurrent deployment
concurrency:
  group: "pages"
  cancel-in-progress: true

jobs:
  # Single deploy job since we're just deploying
  deploy:
    environment:
      name: github-pages
      url: ${{ steps.deployment.outputs.page_url }}
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
      - name: Build
        run: yarn && yarn build
      - name: Setup Pages
        uses: actions/configure-pages@v3
      - name: Upload artifact
        uses: actions/upload-pages-artifact@v1
        with:
          # Upload dist repository
          path: './dist'
      - name: Deploy to GitHub Pages
        id: deployment
        uses: actions/deploy-pages@v1

System Info
N/A

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Also, I should note I haven't found an issue template for documentation issues or suggestions, maybe add something like that? Again, since there's no option to create something like that the best place I know of to suggest this change is here :) so sorry for being off-topic.

Duplicate of #10938


Traffic light at start of lane not supported

When a traffic light is at the start of a lane, the following exception occurs:
org.opentrafficsim.road.gtu.lane.tactical.util.TrafficLightUtil.respondToTrafficLight (98): Traffic light must be downstream.
This is reproduced by adjusting the traffic light position in the T-junction demo to 0.0m.

This issue is no longer reproduced. Perhaps that a change in the LaneStructure (#118) solved the underlying issue.


Skip port mappings when no changes are detected

The dokku_ports task always completes with changed=true, even when the same ports are already set for an application.
This PR checks if the existing ports match the ports to set, if so, it completes with changed=false.
The command for checking existing ports dokku --quiet proxy:ports {app} returns a non-zero exit code when no ports are defined, so I replaced this with a call to dokku --quiet proxy:report {app}.
dokku --quiet proxy:report {app}
       Proxy enabled:                 true
       Proxy port map:                http:80:80 https:443:80
       Proxy type:                    nginx

Thanks @erikvdv1 , also this fix looks great.
As with the other PR, adding a small test would be appreciated if you can find the time.

Done

Thanks for fixing the layout!


Disable docs publishing for azure-spring-data-cosmos

We do not document spring packages on docs.microsoft.com or github.io. Spring packages frequently break the docs.microsoft.com CI builds and this package did break the build.

@danieljurek can we please discuss why the docs broke and what can be done to fix them instead of disabling docs generation for spring-data-cosmos?
cc: @TheovanKraay - we should work on fixing the docs wherever possible.

@kushagraThapar -- From the Git history this package appears to have persisted despite our policy to not publish docs for spring packages in our ref docs. It's possible there is an exception specifically for this package in a place outside of my awareness. I'll try and track down relevant information for that.
Information on the build error which blocks the docs CI: https://apidrop.visualstudio.com/Content CI/_build/results?buildId=399067&view=logs&j=fd490c07-0b22-5182-fac9-6d67fe1e939b&t=a04569be-1b08-5c9a-7e65-a83076350a0c&l=288143
In light of this, the more prudent thing to do is to pin the package at an earlier version until the issue preventing the docs CI build from succeeding is fixed.

That sounds good @danieljurek, we can pin it at an earlier version until docs are fixed. Lets do that.

I can see the issue on the linked CI, thanks, can we run that CI for spring-data-cosmos SDK? I can try to fix it.

Also, I don't think this PR is actually correct, because it is updating only 1 ci.yml.
What about these other ci.yml files which include azure-spring-data-cosmos module ->
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/spring/ci.cosmos.yml
https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/spring/ci.yml
We need to update these as well. Because now azure-spring-data-cosmos is not released with other Cosmos DB Java SDKs, rather it is released with azure spring SDKs.


FunctionsAnalyzer - false positive for constant and function definition

Class is @internal so rename is fine.

Thanks @kubawerlos.


haas.errors.AllocationError should probably inherit from ServerError, Not APIError

This isn't really the user's fault, and should be a 500-range error.

yeah. APIError returns 400 (Bad request) but resource exhaustion is not a users fault.

Hm, then maybe we should be using something else on your vlan pr?

The name AllocationError sounded appropriate to me. I just dont think the error code is appropriate.

Right, but if we're going to change it to 5xx, we should probably use something else for #823. Maybe BlockedError?

Cool. so in #823 I'll use BlockedError (code 409 = conflict ). But for this issue we should probably change AllocationError to some 500 level ServerError

Yep.

I'm going to go ahead and change this.


Bump dataflow-java dependency.

Which provides:

partial responses
the ability to operate on a subset of call sets
choice of variant merge strategy

Also make summarization of reference matching calls configurable.

Coverage decreased (-1.2%) to 61.062% when pulling b01bc3d38328e25c5cfb27ea7c37e15f5bd357b4 on deflaux:summarizeRefMatches into 7038c14a8efbfcadfb2814fc19cac549d98c91c8 on googlegenomics:summarizeRefMatches.

LGTM


Environment variables should be listed on "Cockroach commands"

As a user, I would like to see at a glance exactly which environment variables correspond to which cockroach command (when applicable).
Currently we say to

see the documentation for each command

However, opening a new tab and searching for "environment" or in some cases "env " and hoping to find what I need is too hard (I am a very lazy user! :-) ).
This issue may actually want to grow into something more comprehensive like 'List all CRDB env vars in one place'.

Note that in general, we use environment variables for things we intend to keep undocumented (because they're experimental or only for internal use). That's one of the reasons for using env vars instead of flags (which would show up in --help). There are only a few env vars that we want to document for public use (mainly the client-side ones), and COCKROACH_SKIP_ENABLING_DIAGNOSTIC_REPORTING.

Closing, based on Ben's feedback.


[Feature Request] Add option to backup game saves before launch

Automatically backup saves when launching game using xenia manager or manager created shortcut.
Preferably store backups in xenia manager subfolder or custom directory rather then desktop.
Possibly enable/disable feature per game. Main purpose is to prevent save corruption considering  current xenia state.

I was going to add this in 1.15.0, but I ended up delaying it because there is some work being done that might affect this so I ended up delaying until it's released.

Small update on this feature request
How I'm most likely going to make this feature is by having the user select which profile slot (1-4) he wants to have automatic backups enabled (by default it will backup Profile Slot 1).

Added in v2.0.0


Helm v2 CI failing on caching issue

When make bootstrap is run in environments that already have a Glide cache there's now the error:
[ERROR]	Update failed for google.golang.org/genproto: The Remote does not match the VCS endpoint
[ERROR]	Failed to install: The Remote does not match the VCS endpoint

This is affecting caching in the CI system as well (example here).
This is because the source for google.golang.org/genproto moved from https://github.com/google/go-genproto to https://github.com/googleapis/go-genproto. It it now hosted on a different GitHub org.

This is because the source for google.golang.org/genproto moved from https://github.com/google/go-genproto to https://github.com/googleapis/go-genproto. It it now hosted on a different GitHub org.

@mattfarina Where did you confirm this information from

Click on the first link. It will redirect to the latter.

@bacongobbler Thanks. 😅

@mattfarina  how you fix this issue? change the glide.lock file?


Proposal for Knative Eventing Spec Conformance Test Plan

Changes
Adding proposed structure to define the Knative Eventing Conformance Test Plan, including an example for the Control Plane Broker Lifecycle Tests. These tests define a set of resources that needs to be created, the assertions that need to be done, and the expected output that needs to be generated to be able to validate conformance.
The intention of this proposal is to enable people to start automating these tests, also to reach a consensus about what means to test for conformance and how it needs to be done.

:gift: Proposal for Conformance Test Plan structure and manual steps

/kind enhancement

Fixes #

Release Note

Docs

fixes #40

Fixes #59

@lionelvillard @ahmetb who else do we need to look into this to move it forward? so we can add more sections?

@devguyio (not ahmetb)

@devguyio @lionelvillard should we add someone else to this issue for review?
I want to make sure that this gets reviewed before Wednesday.

Shall we define where the manifests used for manual test plans are to be stored within the repo, and what hierarchy they should follow?

@antoineco sure, I wanted to keep it simple. This PR is using the following approach:
eventing/test-plan/control-plane/<SECTION>/<TEST>
We should organize the resources into directories based on scenarios, we can move those files around when we have more tests. I wouldn't optimize too early here.

Sounds good

/lgtm
/approve
/hold (feel free to /unhold)

/unhold

/lgtm
For the record: I'm fine with this approach even though I feel it adds a lot of overhead. I understand this process is about decoupling the what from the how (to some extend) but I wish I could just write this test plans directly in Go. Note that I don't want to disrupt anything here, thus the /lgtm

@lionelvillard I have the same feeling but I also understand the need for explaining to other people what needs to be tested. From this test plan, we can all agree on what needs to be tested, and from the description, we can automate it. If at some point we feel that we can drop the over head, we should talk about that.

+1

/assign @duglin @spencerdillard

As soon as the CLA bot is happy I can merge this
CC: @salaboy @gabo1208

@googlebot I consent

@gabo1208 do you mind recommending those changes so @salaboy can easily commit them?

closing this PR and moving to test-plan/ in a separate PR


Backport 14941, 15149 and 15210

Intended for merge via merge commit

Happy now!?! :D

Looks good now :) pls squash

Squashed


Update label demo code with explicit code snippits to improve dev ux

Pull Request (for consideration)
📖 Description
While using the library to quickly develop, I found it odd to have demo code show non-applicable implementations. The foreach loop is fairly intuitive but in most cases these types of UI libraries provide a dev UX that allows for tasty copy-pasta. This PR simply unwinds the foreach loop for a better dev UX that quickly allows copying out code snippits in the Label page on the demo site.

🎫 Issues
None
👩💻 Reviewer Notes
This is a developer user experience improvement within the demo site that might be slightly subjective.
📑 Test Plan
No tests affected
✅ Checklist
General

[ ] I have added tests for my changes.
[X ] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[X ] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[ ] I have added Unit Tests for my new component
[ ] I have modified an existing component
[ ] I have validate Unit Tests for an existing component

⏭ Next Steps

@bitobrian please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree


Improve the baseline image with a bakeoff!

Improved the specification of the bookworm image
Made Ansible provisioner work
Changed timeout to 30m from 15m (15m times out in some environments).

Hi @MostAwesomeDude . Would you consider completing https://github.com/artificialwisdomai/origin/wiki/1_Runtime_Environment_Build_and_Automation#61-virtualization-kernel-parameters-profile.

recheck

@MostAwesomeDude I have merged this work into https://github.com/artificialwisdomai/origin/pull/149. In effect, this is a duplicate, so I am closing with a comment. Thanks for the review on this PR. If you have feedback for https://github.com/artificialwisdomai/origin/pull/149, would love to read your review.
Thank you,
-steve


Feature/hosts

Codecov Report

Merging #45 (56ef5c2) into master (21d1e85) will decrease coverage by 1.42%.
The diff coverage is 78.78%.

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
- Coverage   63.03%   61.60%   -1.43%
==========================================
  Files          30       30
  Lines        2916     2870      -46
==========================================
- Hits         1838     1768      -70
- Misses       1078     1102      +24

Impacted Files
Coverage Δ

turbo_tunnel/chain.py
63.15% <50.00%> (-1.45%)
:arrow_down:

turbo_tunnel/conf.py
76.97% <80.00%> (-0.28%)
:arrow_down:

turbo_tunnel/route.py
85.18% <81.81%> (-7.13%)
:arrow_down:

turbo_tunnel/__init__.py
85.71% <100.00%> (ø)

turbo_tunnel/utils.py
67.79% <100.00%> (-4.56%)
:arrow_down:

tests/__init__.py
80.00% <0.00%> (-20.00%)
:arrow_down:

turbo_tunnel/ssh.py
51.10% <0.00%> (-6.58%)
:arrow_down:

turbo_tunnel/socks.py
43.36% <0.00%> (-2.40%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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


Failed to resolve: com.youth.banner:banner:1.4.10

Could not determine the dependencies of task ':app:compileDebugJavaWithJavac'.

Could not resolve all task dependencies for configuration ':app:debugCompileClasspath'.
Could not find com.youth.banner:banner:1.4.10.
Required by:
project :app

Possible solution:

Declare repository providing the artifact, see the documentation at https://docs.gradle.org/current/userguide/declaring_repositories.html

download
https://bit.ly/4evXrQ6
I put the necessary dlls in the archive

ok


Highlight currently selected token within drop down when editing or duplicating a token

Is your feature request related to a problem? Please describe.
When editing or duplicating a token, I am frustrated by having to carefully scroll though a very long list of tokens in order to find the token that is currently being referenced, so I can change it to the next in the list - i.e. changing from color.background.primary to color.background.secondary
Describe the solution you'd like
As a first iteration I would like to see the currently selected token highlighted so that it stands out when scrolling fast. Secondly, an in an ideal world, I would like the drop to open where the selected token is within the list.
Describe alternatives you've considered
Typing in the token name, but this often results in a not found error.
Additional context
We have hundreds of tokens and repeatedly scrolling through them reduces efficiency and builds frustration in the team.

Migrated to Featurebase

Added to Featurebase Request-	🪙 Token organization enhancements


Guild Functionality

The HypixelGuild currently has a few skeleton methods to retrieve information about itself. It needs for the rest of the methods to be added to it and for them to be filled out with a JSONObject. Also, a discussion should be had if it needs a RequestFactory being passed into it's constructor or just a JSONObject.

The HypixelGuild currently has a few skeleton methods to retrieve information about itself. It needs for the rest of the methods to be added to it and for them to be filled out with a JSONObject. Also, a discussion should be had if it needs a RequestFactory being passed into it's constructor or just a JSONObject.

I can get started on guilds this weekend, I think JSONObject should be fine unless we require anything else but I can't think of it off the top of my head. 💯


Additional palettes suggested by Patrick

Reported by karl smith on 8 Nov 2016 18:21 UTC
From Patrick Brockmann - issue added to PyFerret github.
New palettes prepared from matplotlib (especially 'viridis', 'inferno', 'plasma', 'magma') could be added to pyferret distribution.

Read the thread from http://www.pmel.noaa.gov/maillists/tmap/ferret_users/fu_2015/msg00474.html
The tar ball is available from: http://www.pmel.noaa.gov/maillists/tmap/ferret_users/fu_2015/msg00475.html

It seems like these should be added to both Ferret and PyFerret.  Maybe Patrick is asking about changing the default palette in PyFerret, which I don't think is going to happen.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2482

Comment by ansley.b.manke on 8 Nov 2016 18:41 UTC
We had added these back in Nov 2015, using these palette files that Patrick sent and which are in the latest Ferret distribution:
> Fpalette mpl
* * * * * * * * in /home/users/tmap/ferret/rhel6-x86_64-linux/ppl
mpl_PSU_inferno.spk:
mpl_PSU_magma.spk:
mpl_PSU_plasma.spk:
mpl_PSU_viridis.spk:

I have added copies of these under the simpler names viridis, inferno, plasma, magma.
And of course close the loop with Patrick to make sure he knows these are in the distribution. If he's brought this up recently, please go ahead and reply.


feat: support HTTP/2.0

feat: support HTTP/2.0

Codecov Report

Merging #1223 (46c4e17) into master (8aabf5a) will decrease coverage by 9.66%.
The diff coverage is 61.53%.

@@            Coverage Diff             @@
##           master    #1223      +/-   ##
==========================================
- Coverage   67.88%   58.21%   -9.67%
==========================================
  Files          62       35      -27
  Lines        7352     4648    -2704
==========================================
- Hits         4991     2706    -2285
+ Misses       2147     1722     -425
- Partials      214      220       +6

Flag
Coverage Δ

unittests
58.21% <61.53%> (-9.67%)
:arrow_down:

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

Impacted Files
Coverage Δ

hrp/step_rendezvous.go
30.18% <ø> (-2.95%)
:arrow_down:

hrp/testcase.go
63.98% <58.33%> (-1.54%)
:arrow_down:

hrp/step_request.go
52.07% <59.01%> (+<0.01%)
:arrow_up:

hrp/response.go
65.43% <100.00%> (+0.21%)
:arrow_up:

hrp/runner.go
60.15% <100.00%> (+1.88%)
:arrow_up:

httprunner/compat.py

httprunner/__main__.py

httprunner/builtin/functions.py

httprunner/make_test.py

httprunner/response.py

... and 22 more

Continue to review full report at Codecov.

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

close #1188


Volume slider with Flash + fullscreen?

Big thanks for creating mediaelement.js, but just wondering:
When using the fallback Flash-player in fullscreen, the volume slider is not visible and I tested that with several browsers on Win/Mac. The slider is visible when the video is playing on regular state (not fullscreen).
So is there some technical issues that prevent showing the volume slider when fullscreen with Flash, or might this be a bug? I could bet I've seen JS controlling Flash-movie volume somewhere.

The trouble here is that there are really two different sets of controls. In the regular state, the controls are HTML/CSS on top of the Flash movie. But in fullscreen, the controls are rendered by Flash itself and they differ slightly


Grab a ref on fake resolver generator before scheduling a closure

https://source.cloud.google.com/results/invocations/8a7f7da4-cc91-4499-9cb1-66c8916676a7/targets/%2F%2Ftest%2Fcpp%2Fclient:client_channel_stress_test@poller%3Dpoll/log
The fake resolver response generator is sometimes getting destroyed by unreffing before SetResponseLocked completes, and that function is triggered by a closure from SetResponse. Since the unref/destruction might come in between SetResponse and the closure that it schedules, there should be a ref/unref pair on the creation/completion of that closure.
I've run client_channel_stress_test on this attempted fix through ASAN with 1000 runs and no flakes.

Post-merge followup: ASAN with 10000 runs, no flakes

It looks like this fixes #18511.  Thanks for taking care of this, Vijay!

Glad to have helped!


Compilation error: request for member 'Reset' in 'ft', which is of non-class type 'Fishduino()'

Hi
When compiling the example: Fishduino_test
the error message appears:
Compilation error: request for member 'Reset' in 'ft', which is of non-class type 'Fishduino()'
Does anyone have an idea how to fix the error.
R
Stefan

You probably need to add FishDuino.h and FishDuino.cpp to your project.


Tag std specs that need network access

Some test environments do not grant privileges to use the network (even to bind/listen on sockets).
In order to still run most of the test suite in those environments, this tags tests that need those features so they can be disabled if needed with --tag ~network

socket_spec should also be tagged. And addrinfo_spec as well.

Thanks @straight-shoota I added those as well


Is Dexie.Syncable using OT?

From the looks of it, I'm guessing it's doing operational transforms (OT). I didn't see this mentioned anywhere in the docs or code so just wanted to confirm I understand correctly.

Hmm this sounds correct, at least from what I understand by reading a bit about operational transforms. But this only holds true for dexie syncable itself. What the server does with the data is another issue. There are 2 server implementations that I know of that are open source. Both consider the last change when changing the same object from two different clients so data might get lost there. At least this is what I remember about the implementations. I would have to check the code to be sure.
Maybe if you give me more info about OT, I could give you some more information.

I'm no expert in distributed systems, I trust your understanding of OT is likely better than mine.
I ask because you mention the possibility of making use of CRDTs for conflict-free sync, so this had me wondering what the current implementation was doing. I dug around the code but didn't see any specific algorithm mentioned.

I haven't worked on the code for quite some time but this is what I remember:
Dexie-Syncable records every ADD/UPDATE/DELETE operation. When it is time to send the operations to the server the operations are firstly merged. I guess merge could be seen as OT. For each operation we check the ID and merge all operations which are defined for that ID. For example for the ID 1 we might have an ADD, UPDATE, UPDATE then those 3 are merged into 1 ADD operation which contains the two UPDATES. UPDATES are defined on a property level. Changing a property in an object is an UPDATE but we don't record what exactly changed in the property just the the property was changed and the new value.
Now the server comes in play. There is an official server in the Dexie examples and one that I wrote. Both basically work the same. They record all operations those got from the various dexie clients. Everytime we get a new set of operations, the server tries to merge the operations. Now if you have for example two UPDATE operations for one ID and both change the same property, then the last change would win. I'm not sure if this type of conflict resolution is allowed in OT. After the server is done all the new operations are sent back to the client.
So this is how dexie syncable + the server implementations which I know of work. Now I well let you decide if this could be considered OT or not.


Release Google.Cloud.VpcAccess.V1 version 2.2.0

Changes in this release:
New features

Enable REST transport in C# (commit f9ccce7)

Created release for Google.Cloud.VpcAccess.V1-2.2.0


Update sha2, sha3 and curve25519-dalek dependencies

Also merge in some updates from upstream w3f/master.
References https://github.com/mobilecoinfoundation/curve25519-dalek/pull/2
Supports https://github.com/mobilecoinfoundation/mobilecoin/issues/1326

PTAL, I've updated the patched curve dep to use the MCF fork.

PTAL, I had to force-push after signing the commits.


Chrome Web Store -- New policy limiting how user data can be used

Literally every time we publish a new version, there are new rules from Google, and more forms to fill :^)

Ref. https://blog.chromium.org/2020/11/transparent-privacy-practices.html
I'll try to fill forms, hopefully there won't be any real blockers :crossed_fingers:

Submitted below for both Stable (v2.16) and  Beta (v<IP_ADDRESS>7) channels:

Will close this issue when both are approved.

We got approved :+1: all good!


update feedstock to upstream refcount 1.1.1

Checklist

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

v1.1.0 (2022-08-19)
Main changes

Expand some features to cater for macos
Tidy up and reengineer some of the legacy functions in the platform utilities putils to facilitate library loading. Minor breaking changes, but probably for no-one but the author in effect.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Error: could not find tiller

My cluster is RBAC enabled. I created a static IP and installing nginx using Helm as instructed above using this command
helm install stable/nginx-ingress --namespace kube-system --set controller.service.loadBalancerIP="<Static IP Address>"
I am getting an error: "Error: could not find tiller"
Regards
Badal

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

ID: ada58160-dc50-b977-d3bc-65f92406e16d
Version Independent ID: 5c596e04-fddb-effe-3b72-575f4df42d4e
Content: Configure ingress with Azure Kubernetes Service (AKS) cluster
Content Source: articles/aks/ingress.md
Service: container-service
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

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

@badalk this is directly related to #14698
I will close this one and we can work off a single issue.


add auto comment

Add auto comment based on pull_request_target. This PR will be merged first to check that whether the workflow can be triggered in coming PR.

@jiaqima I am merging this PR for testing purpose. auto-comment do not run any code in new PR, so there is hopefully no vulnerabilities


Check port 8090 is free before starting Gatsby

Currently, if this port is already taken (usually by a previous ghost gatsby process that wasn't killed properly) then it will spend 2 minutes building the docs, and only then show an error when it tries to start the server.
Would be nice to quickly check this upfront, and show an error recommending that the user kill their ghost server.

Gatsby should check if the port is open and then offer to do it on a different port if not.
Maybe this isn't happening because we're forcing port 8090? We can remove that part and leave it up to Gatsby.

I would rather have it locked to 8090 because then you can refresh the same browser tab after the server crashes and restarts. It's annoying to have to read a different port number every time it starts

So I think just checking for the port open status before would be better

Going to close this one as a "won't fix." I'd rather file an issue with Gatsby if we want to improve this experience.


pod OEM 1.0 (Planet of Death) Unknown resource format: D3DDDIFMT_UNKNOWN

Hi there,
It seems theres a Texture Format that DDrawCompat can't handle.
I want to play pod OEM 1.0 (not pod 2.0). Because the OEM version have unique features and has been removed in pod 2.0
Can you make DDrawCompat work with pod oem ?
DXWnd can run pod OEM, but this program can't scale the internal resolution like do DDrawCompat .
I've attached the log.
Cheers.
DDrawCompat-PODMMX.log

I don't have this issue. Please set LogLevel=debug in DDrawCompat.ini and upload the resulting debug logs.

I've tested with and w/o PodHacks patcher from nicode, but the problem still persist.

https://svn.nicode.net/podhacks/bin/
DDrawCompat-PODMMX.zip

Ah, you haven't mentioned that you're using an older version of DDrawCompat (v0.5.1 based on your screenshot). Can you check with the latest version (v0.5.2) too? Although I can't reproduce the issue with v0.5.1 either.

the same issue with 0.5.2
maybe you don't have test the same pod.
i mean, pod oem 1.0, not pod retail 2.0
here's the iso:
https://archive.org/details/pod_oem
my podmmx.exe is 1.194 KB

Yes, I'm using the 1.0 version, more specifically <IP_ADDRESS>, even though you linked to the wrong version above (2.1).
Based on your debug logs, I suspect the issue is that the game tries to establish exclusive fullscreen mode from a second thread, without properly releasing the exclusive mode already established by the first thread. The issue is somewhat similar to https://github.com/narzoul/DDrawCompat/issues/286#issuecomment-2029973283 except that the second thread tries to release the first thread's DirectDraw object, which then (I assume) will fail to release the exclusive mode mutex, which can only be released by the first thread itself. This might be confirmed by debug logs with DDrawCompat v0.5.2.
In the other issue it was mentioned that the "EmulateDirectDrawSync" shim could "fix" this issue, so you could try that, if you know how to use Microsoft Compatibility Administrator.
The reason I'm not getting the same error is that in my case, all attempts to establish exclusive fullscreen mode happen on the first thread. Additionally, this whole multiple attempts to establish exclusive fullscreen mode only happens when PodHacks is enabled. If you properly remove it, then the game should only do it once, at least based on my own logs, so you shouldn't be getting this error. I don't know why my game sets the fullscreen mode always from the same thread, while yours is switching threads. I'm not familiar with PodHacks, and despite it being open source, I did not find any direct calls to such functions in it, so it must be doing something else to trigger this behavior.
Please make sure you remove PodHacks properly, test with DDrawCompat v0.5.2, and upload debug logs again, if it still doesn't work.

i've tested in a new profile on windows, but nothing.
ill test in a vm to see what happens.
ps: DDrawCompat can work on vm? (vmware)

I've found the problem, and this is about you comment, to use Microsoft Compatibility Administrator
Compatibility mode (Win95 / Win98) 🤦♂
😂😂😂😂

This work with and without PodHacks! ✨🎉
Another thing that I've forgot to mention is that I've two monitors, can be this the problem?
I think not, because I've tryed turning off one of them but the problem persist.
Here's the LOGs of the two modes (with and without podhacks).
without podhacks DDrawCompat-PODMMX.zip
with podhacks DDrawCompat-PODMMX.zip
Last question/request  (if exist a trick or somewere).
You mention the following:
but it won't get you what you want. PODMMX doesn't use Direct3D, it has its own software renderer
knowing this is a propietary software renderer, is impossible in anyway, tool, trick, black voodo magic to upscale this in today?
i know, maybe the last hope is reverse engineering, like Wipeout Phantom edition:
https://github.com/wipeout-phantom-edition/wipeout-phantom-edition
The following you can skip (I hop not).
Historical memories of my brain: I remember a lost pod forum (cant remember its name), many guys debating about releasing the source code of pod 2.0 retail and pod 1.0 oem. But one guy didn't want to release the source code for "ideological reasons", the debate was something like:
Ubisoft managed the development of POD through 2 parallel projects: One team was in charge of the software render, and the other team handled the 3dfx and Direct3D version 3 and 5 renders. The whole main development (software and 3dfx/D3D) was done in the Watcom 5 language (a derivative of the C language of that time, 1997/1998). Both engines operated on a single pipeline for everything, making it extremely difficult to develop a source port for these engines.
That about pipeline I can't figure out what it means, but I think they mean that all graphics, physics, and sound pass through one pipeline, and this complicates the entire process.
Well, thanks very much for your help! 😅👍
I hope you can read this. I'd like to know the opinion of developers (although I'm not one of them, sadly).

ps: DDrawCompat can work on vm? (vmware)

Maybe, if the VM has good enough WDDM GPU drivers, but it's not supported by me.

Compatibility mode (Win95 / Win98)

Yes, those two include the EmulateDirectDrawSync compatibility shim, among others.

Another thing that I've forgot to mention is that I've two monitors, can be this the problem?

No, I don't think it's related.

knowing this is a propietary software renderer, is impossible in anyway, tool, trick, black voodo magic to upscale this in today?
i know, maybe the last hope is reverse engineering, like Wipeout Phantom edition

Yes, without access to the source code, it can only be done by reverse engineering and modding the renderer.

That about pipeline I can't figure out what it means

I don't know either in this context.


math_tests__math__iree_vulkan__functions__atanh failing on vulkan

Preemptively filing because disabling due to a failure in an upcoming change.
Diagnostics:
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/framework/op_def_library.py:748:0: error: failed to legalize operation 'math.log1p'
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/ops/gen_math_ops.py:1332:0: note: called from
/usr/local/google/home/laurenzo/src/iree/integrations/tensorflow/e2e/math/math_test.py:672:0: note: called from
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/autograph/core/function_wrappers.py:117:0: note: called from
/usr/local/google/home/laurenzo/src/iree/integrations/tensorflow/e2e/math/math_test.py:672:0: note: called from
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/framework/func_graph.py:983:0: note: called from
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/eager/def_function.py:668:0: note: called from
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/framework/func_graph.py:1007:0: note: called from
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/eager/function.py:3291:0: note: called from
/usr/local/google/home/laurenzo/src/iree/iree.venv/lib/python3.9/site-packages/tensorflow/python/eager/function.py:3456:0: note: called from
<unknown>:0: error: failed to run translation of source executable to target executable for backend vulkan*
<unknown>:0: error: Failures have been detected while processing an MLIR pass pipeline
<unknown>:0: note: Pipeline failed while executing [`mlir::iree_compiler::IREE::HAL::TranslateExecutablesPass` on 'hal.executable.target' operation: @vulkan_spirv, `mlir::iree_compiler::(anonymous namespace)::ConvertToSPIRVPass` on 'module' operation]: reproducer generated at `/usr/local/google/home/laurenzo/src/iree-build/integrations/tensorflow/e2e/math/math_tests__math__iree_vulkan__functions__atanh_test_tmpdir/tf/math/atanh/static_dims_non_complex/reproducer__iree_vulkan.mlir`
<unknown>:0: error: conversion from source -> vm failed

@stellaraccident if you still have the reproducer in the tmp dir listed above that would help :)

Lol, I do not.

I can fetch it for you in the morning if it helps.

I am building this locally. I should be able to get the repro.


Stuck on Login Page

When I login with github, I get stuck on login page and it just leaves the message "Loading Site Data".

Now https://github.com/sveltia/sveltia-cms-auth/pull/2 is merged. Can you redeploy the auth script and try again? Thanks!

Please feel free to reopen (possibly with a browser console log) if the problem persists!

Hello, I have this exact same issue. I'm also using the https://github.com/sveltia/sveltia-cms-auth sveltia auth system.
My localstorage shows content but it's stuck on the loading screen.

The reported issue has been fixed so the cause might be different. Do you see any errors in the console? Is this your repository in question? https://github.com/0x0m0t0/astro-sveltia

Thanks for getting back to me, should I open a new issue?
There are no errors at all in the console. Although after I login with auth, the popup git window just dissapears and then I get stuck on the loading screen.
This is indeed my repo, although my working branch is more up to date
https://github.com/0x0m0t0/astro-sveltia/tree/sveltia-vercel

When I open your repository locally or just visit https://astro-sveltia.vercel.app/admin/ I see this error on the CMS UI:

The configuration file could not be retrieved. Please solve the issue and try again.

The config path seems wrong. This line can be removed:
<link href="admin/config.yml" type="text/yaml" rel="cms-config-url" />

Okay, I could reproduce the issue after solving the config path issue, probably because there is no content yet. Since it’s a different problem from the original one, I’ll create a new issue.


unable to use getUserConfirmation of BrowserRouter

This is my intent (choose one)

[x] I want to request a feature or change

The problem
Given I have a Form
When the user enters data in the form
And navigates away without saving
Then I want to display a warning "Do you really want to discard..."
BUT I can not do this since the implementation of the BrowserRouter does not allow me to set a custom getUserConfirmation
Proposed solution
hops-react should provide a way for me as a consumer to provide navigation guards.
Since I'm still not familiar with the intend/concept of the context api I find it hard to propose how to do that exactly.

@Xiphe you could just import the ReactContext and extend it by overwriting the enhanceElement method:
import { BrowserRouter } from 'react-router-dom';
import hopsConfig from 'hops-config';
import { render, combineContexts, ReactContext } from 'hops-react';

class MyReactContext extends ReactContext {
  enhanceElement(reactElement) {
    return (
      <BrowserRouter basename={hopsConfig.basePath} getUserConfirmation={getConfirmation}>
        {reactElement}
      <BrowserRouter>
    );
  }
}

render(<MyApp />, combineContexts(MyReactContext));

@Xiphe does this shed some light on the current context implementation?

@ZauberNerd, your snippet breaks server side rendering since enhanceElement is applied on both: server and browser. The BrowserRouter can not be rendered on the server.
Next step for me: adding a platform switch in my app - wondering why i'm using hops-react at all.

@Xiphe please check out #298 - I think that feature ought to enable you to achieve what you need without having to differentiate between client and server. From what I gather from StaticRouter's docs, it should ignore the getUserConfirmation prop.


Mention the setup for dependencies WRT iOS examples

Hey all, this caught me off guard ( I was preparing to test offline, and couldn't get around to doing it till later ) so I figured it was worth documenting in the README.
I think it shouldn't be needed for android, but didn't test that

Opened PR, added the npm scripts https://github.com/orta/native-navigation/pull/1


chore(ci): fix the formatting issues on AST Changes.

I'm not sure what causes this issue so this PR is going to start as an experimentation

#4153  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite

@DonIsaac Pardon me mate, But I think you've dropped a review here by mistake.


Move deletion of webhook in GitHubService API to SPI

Depends on actually creating a GitHubServiceSpi, reflected by #61
This cannot stay in the API:
/**
     * Deletes all webhooks in a specific GitHub repository
     *
     * @param repository - the value object that represents the GitHub repository
     * @throws IOException
     * @throws IllegalArgumentException If the parameter is unspecified
     */
    void deleteWebhooks(final GitHubRepository repository) throws IllegalArgumentException;

Needs also to reflect not deletion of all webhooks, but of a single webhook; in tests we only want to delete the webhook we created, not all.

85551d461c9ff8fd7c4694298be88897f4cea213
@rimolive Be sure to check your inputs too: 85738c18341c52480bd799c95fced84595ed6071  :)
Well done.


Status of this library?

Not exactly sure who or where to reach out, but is this library still actively maintained? I know postmark still has a link to this repo but my biggest concern with using this library are the outdated dependencies.  A lot of them haven't been updated in years.
Related, are there plans to using something a little more modern like webpack instead of grunt?  I'm looking into expanding some of the functionality and would prefer to use es modules instead of the old commonJS way of doing things.
Thanks

@pzimmermaninmo Hi! I don't think this project is actively maintained, but it worked well for us. I built a complete mail builder less than 3 months ago and it works well.
If you are using Postmark, it's the best solution I found.

@pzimmermaninmo Hi! I don't think this project is actively maintained, but it worked well for us. I built a complete mail builder less than 3 months ago and it works well.
If you are using Postmark, it's the best solution I found.

Yeah we actually started using it and it seems to work its just unfortunate its no longer being maintained.  If it breaks one day we might be SOL.  (Not to mention all the security vulnerabilities with these outdated libraries.  Hell, one of the dependencies spits out this gem:
warning<EMAIL_ADDRESS>Deprecated because Grunt is practically unmaintained. Move on to something better. This package will continue to work with Grunt v1, but it will not receive any updates.

Yeah we actually started using it and it seems to work its just unfortunate its no longer being maintained. If it breaks one day we might be SOL.

We are in the same boat. I think it's more of the grunt "helper" dependencies that aren't being maintained. At this point we are debating rolling our own email build process so that we can mitigate this risk and also reduce the vulnerabilities reported.
Currently: 37 vulnerabilities (4 low, 6 moderate, 22 high, 5 critical)

I think using React Email and the Postmark Node SDK we could achieve the same thing.
Anybody tried it? I was just playing with it locally and seems promising.


add bomb flower and block way back from LMF bk room to prevent softlocks

Tested on dolphin, should be good to merge after a test on console

build: https://nightly.link/lepelog/ssrando/workflows/build.yaml/lmf-lock/dist windows-latest


Minification, Concat, Image Optimization..ect

This needs to be done in order to reduce the amount of request and the size of the server files for faster page load.

Indeed this is important... +1

CSS Minify and Concat is done with 751f9c11c95a4f52703594f1e97d033a030552b8

[x] CSS Minify and Concat
[ ] JS Minify and Concat
[ ] Image Optimization


Feedback for “Deploy a contract”

where project is?  i don't konw src/Counter.sol this file where is from which project ?

Counter.sol is just a mock smart contract to deploy


[BUG] Failed to npm update

Describe the bug
Failed to npm update
To Reproduce
Steps to reproduce the behavior:
Encountering version conflicts when using npm update.
% npm update
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   packages/ui
npm ERR<EMAIL_ADDRESS>npm ERR!     node_modules/flowise-ui
npm ERR!       workspace packages/ui from the root project
npm ERR!       1 more (flowise)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer react@">= 0.14.0 < 18.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/react-device-detect
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   packages/ui
npm ERR<EMAIL_ADDRESS>npm ERR!     node_modules/flowise-ui
npm ERR!       workspace packages/ui from the root project
npm ERR!       1 more (flowise)
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
Setup

Flowise Version : main branch or 1.2.12
OS: macOS

@ZhuPeng1314  let me check. i will get back to you

I use npm install -g flowise everytime to automatically reinstall the latest version


Use the $conf_template variable to override the config template

The conf_template override wasn't being used, this should fix that.

Let's wait for @Dharmender-Singh review, I'm not particulary skilled on puppet.

Thanks @Dharmender-Singh :) that was quick.
Thanks @bgmncwj for the contribution.


libbpfgo: Add map batch operations

This patch adds a few batch operations for maps including:
Lookup, LookupAndDelete, Update and Delete.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @grantseltzer thanks for the feedback!
I've addressed most of your review except for that which I've commented on. Also I've moved the test over to it's own directory in selftest/map-batch and removed the corresponding tests in libbpfgo_test.go.

Made a small edit I didn't want to bother you with. This looks good to me! Thanks so much @derekparker !


Unsupported Filters error produces broken errors UX

Unsupported Filters error allows users to still interact with the application but further errors are being ignored (and not displayed to the user). The unsupported filters error should be changed to be treated as "dismissible".
This came up with an improperly defined facet url and an invalid sort modifier also being applied.

This behavior has been fixed in the React implementation. We're now showing the Unsupported filter first and after dismissing it, the sort error is displayed to the users.


feat: Update documentation with auth topics; Move documentation site to main repo

Update documentation with auth topics; Move documentation site to main repo
Description
Main driver was to move the docs site in the main repo, so we can allow developers to add docs changes as part of the DoD or as standalone changes. Docs are now public and easy to improve! 🥳 . Details:

Added new section to the docs site - "Build Your Solution"
Added new page Authentication with general tech info and diagram supporting light and dark modes (slick!)
Added DataSettings page for Data config settings
Added JWT Settings page for Data config settings
Added new page "Authentication"
Move Docs website from https://github.com/endatix/docs-website -> https://github.com/endatix/endatix/. Now it's in docs\entaix-docs folder in the root
Add automated deployment to https://docs.endatix.com/ on main branch update
Add manual trigger for deployment to https://docs.endatix.com/
Fix deployment filters to for main release to ignore changes in the docs\** glob pattern

Related Issues
closes #29, closes #115
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] Documentation update

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Screenshots

Can we configure Docusaur to launch on a port different than 3000, because it is conflicting with Endatix-hub?
If you launch Docusaur after Endatix-hub there is no problem - Docusaur prompts you to automatically move to 3001. Howver if you launch Docusaur before Endatix-hub, then the Endatix-hub app doesn't show any errors and all appears to be running fine, but the docs are still responding on 3000. If you stop the docs then the hub appears as if nothing happened.
This should be fixed in the Endatix-hub app, because other things might be running on that port out of our control. However a quick fix here would be to change the default port on the docs.

@orilux  - I am merging. Let's leave this out of scope for now and solve this later. We might also think about a good port strategy, e.g select port range for Endatix apps, so it's logical and intuitive. Not a bridge we should cross now 🌉


remove the sharebuttons

Hi,
is there a sulution to remove the sharebuttons?
best regards
andy

I'm sure there's a way to do this, but I can't provide any help for customization.
Editing this function could be a good starting point. Maybe that helps a bit.

I hope that this suggestion is acceptable to post here.  First time commenting on anything in Github.
I was able to achieve this by editing the main.js file for my purposes.  I wanted the share button removed when not logged in for both albums and pictures.  I altered the following section:
$("#button_share, #button_share_album").removeClass("button--eye").addClass("button--share").find("use").attr("xlink:href","#share")
Changing it to this:
$("#button_share, #button_share_album").remove()

@anmue
See Pull Request #632, hope it helps solve your problem.


Jacinto Rammirez

What does the program do ?
Imprime una variable con el texto "Hola Mudo, Hacktoberfest 2018"
In what programming language it is written?
PHP

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Salesforce clean-up

Lots of accounts in our Salesforce database (especially international) have missing information. This means that they are excluded from certain filter views (city/state/country, student FTE). In addition to title and website, at minimum, every account should include address (at least city, state/province, and country), and campuses must include a number under "FTE Enrollment (All Degrees)". We should also try to incorporate latitude-longitude coordinates wherever possible for future mapping features. Would be good to get an export of this data and assign someone to take care of this.
Note: This may also be addressed during the migration process of the new AMS.

All campuses also need lat/long to be included on the STARS map.
On Tue, Apr 26, 2016 at 6:51 PM, monika-aashe<EMAIL_ADDRESS>wrote:

Lots of accounts in our Salesforce database (especially international)
have missing information. This means that they are excluded from certain
filter views (city/state/country, student FTE). In addition to title and
website, at minimum, every account should include address (at least city,
state/province, and country), and campuses must include a number under "FTE
Enrollment (All Degrees)". Would be good to get an export of this data and
assign someone to take care of this.
Note: This may also be addressed during the migration process of the new
AMS.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/AASHE/hub/issues/303

--
The AASHE Sustainability Awards are now open! Submit your application
http://www.aashe.org/about/aashe-awards by Friday, June 3 at 5 p.m. ET.
Connect with us:   Website http://www.aashe.org/ | Facebook
https://www.facebook.com/aasheorg | Twitter
https://twitter.com/AASHENews | LinkedIn
http://www.linkedin.com/company/aashe

Email Confidentiality Notice
http://www.aashe.org/email_confidentiality_notice

Are we looking at incorporating maps into the Hub at some point down the road?

It's probably a way off, but I would love to have maps for many of the content types. I can see it as being especially relevant for project content types like solar installations and campus farms. It would be cool for academic programs too actually.

That's what I figured. We may need to consider recording coordinates for non-campus orgs as well, in that case.

Yes, we will likely need to do a "data scrub" as we migrate to the new AMS. It would be a good idea to think about all of the different attributes we want to capture in the AMS for each institution so that we can build them into the new AMS.

Check w/ Ben whether changes made now in SF or later in new AMS automatically update in the Hub (I seem to recall a recent ticket about this that Bob implemented)

January 2017 is when transition to new AMS will happen. Programs staff are welcome to make edits.

Monika to request report of all organizations with resources in the Hub and will cross-reference against SF report - priority to ensure that all of these orgs have all info included.
Probably after the conference.

In progress with ongoing AMS maintenance - closing since not really related to Hub IT.


Api 598 vessel api update shipdataprocess library

https://globalfishingwatch.atlassian.net/browse/API-598

@aperdizs I've added one more fix regarding the suffix "N" which shouldn't be deleted to be consistent with the BQ UDF functions, and a couple test cases.


Doc change: Maven build source command

This PR, changes the build command in the source install docs from mvn clean install to
$ mvn clean install -T 1.0C -Pnone && cd assemblies && mvn install -T 1.0C -Pdev && cd ..

Because take advantage of the multithread capabilities of almost all the modern computers.
I've got results with my Machine (Intel i5 Macbook Pro 2020) up to 400% faster compared with the single mvn clean install single thread command.
PS: Thanks @mtneug for the tip!
Your pull request should…

[x] have a concise title
[ ] close an accompanying issue if one exists
[x] be against the correct branch
[x] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits

Due a new information, I need to stop this PR. because the parallel build can affect the Opencast Logging, making much more difficult to track bugs and issues.

A hard -1 to this pull request in general.  This method of building Opencast is very fragile and can result in seemingly successful, but incorrect builds. If you don't exactly know what you are doing, you should not use this. I think that recommending this to newcomers is a really bad idea.
I'm fine with parallel builds, but before you recommend that in general, Opencast's nolule build dependencies need to be specified correctly.

Thanks, Lars for your comments, that's Why I changed to draft, because that compilation way it is not ready for all users,  I can change it to add it to the dev docs.
But close it?, that was a little harsh 😢


Harden the Pulsar transport

MOAR error handling in PulsarListener

I think this could be handled by more generic handling in the various receivers


API improvement for when there is no state arguments.

We should be able to do:
const { classNames } = useClassNames();

instead of
const { classNames } = useClassNames({});

When we have no state argument, e.g:
const { useClassNames } = createUseClassNames()(
    theme=>({
    //...
    })
);

done


[BUG] Getting Val_AUC metric as 0 along with other losses like regularization_loss etc

Bug description

I am following along the notebook 2b that was used for the GTC dry run on my own dataset. When I run model.fit, it trains but gives me "val_auc: 0.0000e+00" on every model that I have tried. I would like to understand why this is happening and I am missing anything here. Also, is this auc metric AUC curve or accuracy of a model? I would like comprehend other metrics displayed in the progress bar of these models as well like in the below snapshot, if auc is accuracy, then the training accuracy being 50%, how is the loss 68% as these numbers don't add up to 100%. Similarly, val_auc is 0 but val_loss is .8095.
I am attaching sample training data and test data parquet files as well as the code snapshots of this training.
Steps/Code to reproduce bug

import os
os.environ["TF_GPU_ALLOCATOR"]="cuda_malloc_async"
import cudf
import glob
import gc
import pandas as pd
import nvtabular as nvt
from nvtabular.ops import *
from fit_transform import workflow_fit_transform
from merlin.schema.tags import Tags
from merlin.schema import Schema
import merlin.models.tf as mm
import merlin.models.tf.dataset as tf_dataloader
from merlin.io.dataset import Dataset
from merlin.schema.io.tensorflow_metadata import TensorflowMetadata
from merlin.models.tf.blocks.aggregation import CosineSimilarity
from merlin.models.tf.blocks.core.aggregation import CosineSimilarity
import tensorflow as tf
train_path = '/workspace/data/train/.parquet'
test_path = '/workspace/data/test/.parquet'
output_path = '/workspace/data/processed'

%%time
Define NVT the pipeline
We add tags for user_id, item_id and target. NVTabular will provide an output file
We categorify the user_id and item_id to be continuous integers 0, ..., |C|
user_id = ["userid"] >> AddMetadata(tags=[Tags.USER_ID]) >> Categorify()
item_id = ["contentid"] >> AddMetadata(tags=[Tags.ITEM_ID]) >> Categorify()
targets = ["relevant"] >> AddMetadata(
tags=[str(Tags.BINARY_CLASSIFICATION), "target"]
)
outputs = user_id+item_id+targets
etl_description = 'etl1'
%%time
workflow_fit_transform(outputs, train_path, test_path, output_path)
schema = TensorflowMetadata.from_proto_text_file(output_path + '/train/').to_merlin_schema()
target_column = schema.select_by_tag(Tags.TARGET).column_names[0]
schema.column_names
model = mm.MatrixFactorizationModel(
schema,
dim=64,
prediction_tasks=mm.BinaryClassificationTask(target_column, metrics=[tf.keras.metrics.AUC()]),
aggregation=CosineSimilarity()
)
model_description = 'MF'
model.compile('SGD', run_eagerly=False)
batch_size = 16*1024
tf_dataloader.
train_dl = tf_dataloader.BatchedDataset(
Dataset(output_path + '/train/*.parquet', part_size="500MB"),
batch_size = batch_size,
label_names = [target_column],
shuffle= True,
schema = schema,
)
test_dl = tf_dataloader.BatchedDataset(
Dataset(output_path + '/test/*.parquet', part_size="500MB"),
batch_size = batch_size,
label_names = [target_column],
shuffle = False,
schema = schema,
)

model.fit(train_dl, validation_data=test_dl)

Expected behavior

Environment details

Merlin version: 22.03
Platform: Ubuntu 18.04
Python version: 3.8
PyTorch version (GPU?):
Tensorflow version (GPU?): 2.8

Additional context

Dataset:
userid: unique ids of user
contentid: unique ids of items
relevant: target column (whether a user interacted with the item or not, interactions here are viewing a page, updating a page or creating the page)
I am attaching a code snippet of how I am generating this binary column.
I have used other user features as well with merlin models ie MLP, MF and DLRM but I still get val_auc as 0 with all of them.

test_data.csv
train_data.csv

@mamtaed8811 remove metrics=[tf.keras.metrics.AUC()]  from  the mm.BinaryClassificationTask(target_column, metrics=[tf.keras.metrics.AUC()])  then you will get other metrics printed out.

Thanks @rnyak. I am able to print other metrics now and comprehend it a bit better.

@rnyak Can this be closed ?

@viswa-nvidia and @rnyak . I am seeing that the performance metrics for the trained models like accuracy and loss don't add up any time in my training. It's a bit confusing as to why it doesn't so I wanted help understanding why it isn't.

Thanks @rnyak. I am able to print other metrics now and comprehend it a bit better.

what was the issue here, I am having the same issue and unable to understand what is happening

@netinupur can you please provide more details?what's your env?   which model you are running?  and what error you get and what's your model build and training script? thanks.


add belongs_to primary_id (aka. primary_key in rails) support

as described here in rails
https://apidock.com/rails/ActiveRecord/Associations/ClassMethods/belongs_to

@u007 I'll take a look soon, but why not using primary_key? Is there a reason to use primary_id instead? :)

hi @stanislas-m cox we are already using fk_id


cannot use safe JSON deserialization for Dialog State object

Version
4.5.1
Describe the bug
To save/load the dialog state, the bot need to serialize/deserialize the dialog states with JSON.
Currently, the dialog states contains a list of dictionaries of  objects, which is black box to the bot author programmer. To deserialize the string to the dialog state , the type information must be put into the JSON when searlizing the object ot string. For example, with Newtonsoft.Json, we need to specify TypeNameHandling = TypeNameHandling.All for serializer, just like the botbuilder sample code
But it is not safe to put the type information into the serialized JSON string, because an attacker could modify the serialized data to include unexpected types to inject objects with malicious side effects. There are some published document about it:

Do not use insecure deserializer ObjectStateFormatter
(MS Internal Security Requirement) Safe deserializers must be used in functions that process untrusted data

Per the above documents, when  we serialize/deserialize the dialog state, we should specify specify TypeNameHandling = TypeNameHandling.All for JSON serializer.  But currently it will cause exception in Bot Builder.
Bot Framework should provide a way to allow the bot to serialize/deserialize the dialog state safely.
To Reproduce
Steps to reproduce the behavior:

download the sample code of BotBuilder: 42.scaleout
change the code in the line from

private static readonly JsonSerializer StateJsonSerializer = new JsonSerializer() { TypeNameHandling = TypeNameHandling.All };

to
private static readonly JsonSerializer StateJsonSerializer = new JsonSerializer() { TypeNameHandling = TypeNameHandling.None };

Compile the project and run it
Use the bot emulator to talk to the bot

Actuall behavior
Exception happens:
Unable to cast object of type 'Newtonsoft.Json.Linq.JObject' to type 'Microsoft.Bot.Builder.Dialogs.DialogState'.

   at Microsoft.Bot.Builder.Dialogs.ComponentDialog.ContinueDialogAsync(DialogContext outerDc, CancellationToken cancellationToken) in d:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\ComponentDialog.cs:line 75
   at Microsoft.Bot.Builder.Dialogs.DialogContext.ContinueDialogAsync(CancellationToken cancellationToken) in d:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\DialogContext.cs:line 160
   at Microsoft.Bot.Builder.Dialogs.DialogExtensions.RunAsync(Dialog dialog, ITurnContext turnContext, IStatePropertyAccessor`1 accessor, CancellationToken cancellationToken) in d:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\DialogExtensions.cs:line 18
   at Microsoft.BotBuilderSamples.DialogHost.RunTurnAsync(Dialog dialog, ITurnContext turnContext, JObject state, CancellationToken cancellationToken) in C:\src\others\BotBuilder-Samples\samples\csharp_dotnetcore\42.scaleout\DialogHost.cs:line 67

Expected behavior
There is no error.
Screenshots

[bug]

any updates on this ?

Related: https://github.com/microsoft/botbuilder-dotnet/issues/2450

Hi this issue has been opened for 3 months, could someone work on them?

Hi @gmajian
A Draft PR addressing this is here: https://github.com/microsoft/botbuilder-dotnet/pull/2944  This is awaiting discussion and review.

https://github.com/microsoft/botbuilder-dotnet/pull/3023 has been merged to master
AzureBlobStorage, MemoryStorage, and CosmosDbStorage now all provide a constructor allowing custom json serializer settings.  A few changes to BotState are also included: which enable TypeNameHandling.None  These changes are scheduled to be released with v4.7 early Dec

Hello guys,
Any update on this?
I am experiencing the same issue when setting the JsonSerializer to TypeNameHandling.None.
Many thanks.

any updates on this? it is still the issue for ComponentDialog

Per discussion, @johnataylor expects to close this out this week.

We can provide a sample in samples/experimental that shows users how they can avoid type serialization.
Issue in samples: https://github.com/microsoft/BotBuilder-Samples/issues/2870
Regarding the security documentation linked to from this issue. The stated concern here is loading types from untrusted stores. The bot state store in a normal deployment is a trusted data source.


Memory leak

Vue version
3.5.4
Link to minimal reproduction
http://cant.insert.code/commercial_project
Steps to reproduce
The problem started to appear in Vue 3.5.* (including version 3.5.4). There are no problems in version 3.4.38.
In my case, the approximate source of the problem is an active websocket connection.
What is expected?
No memory leak
What is actually happening?
After establishing a socket connection from which data is coming, the memory grows infinitely. Moreover, the more socket connections, the more the memory grows without stopping. With 10 connections in 5-10 minutes, the heap size consumption is more than 3 GB.
An example of how a leak starts after the application starts:

At the same time, taking memory snapshots is very difficult because the snapshot formation often gets stuck at the stage (Building dominator tree).

If you still manage to take a memory snapshot, it turns out that the preliminary cause is deps (I may be wrong).

I want to clarify that in case of a leak:
DOM Nodes - do not change.
JS Event Listeners - do not change.
System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (12) x64 12th Gen Intel(R) Core(TM) i5-12400
    Memory: 13.66 GB / 31.77 GB
  Binaries:
    Node: 18.12.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.4 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Electron": 32.0.1
  Browsers:
    Chrome: 128.0.6613.120
  npmPackages:
    vue: ^3.5.4 => 3.5.4

Any additional comments?
No response

Solution: After I removed {deep: true} in all "watch" functions the problem disappeared.
But because of the many such "watch " now I will have to reconsider the logic in the application.

Could you please check the reproduction link? It seems to be inaccessible.

I can't provide full repro code because I have a complex commercial project from which it is difficult to extract repro code. The fact is that "watch" with {deep: true} starts to leak.

I also noticed that it's not just watch that's leaking. I have a couple of computed functions that are also leaking (the events array is updated several times per second). I compared several snapshots and noticed that the deps class is not being cleared, and that's a problem on your end.

Example of comparing snapshots 3 minutes apart:

We can't analyze the bug without runnable minimal reproduction.
Please follow the issue requirement.

I don't know if this mini demo is what you wanted to see, but even from it you can see that when comparing memory snapshots, the Dep class is not cleared by GC.
But the computed leak could not be confirmed.
Here is the link:
https://play.vuejs.org/#[encoded data, 7000 characters]

Still unable to reproduce

in Chrome incognito mode, ensure there are no browser extensions
manually collect garbage
JS heap size, DOM Nodes, JS event listeners. They can all be recycled.

I'm not sure if it is related but when debugging a reactivity issue in one of my vitest tests I noticed an issue with the double-linked list for tracking dependencies:

when traversing the list starting from depsTail using prevDep, there are 6 dependencies (not expected)
when traversing the list starting from deps using nextDep, there are 2 dependencies (expected)

I wonder if a memory leak could be caused by the redundant Dep objects referenced by depsTail.prevDep.prevDep.prevDep.*? Maybe in some situations they could be accumulating much more than in my test code?
Unfortunately I'm still struggling to isolate the issue. :-/
PS. The issue I'm debugging is about a computed not registering as a dependency of a shallowRef, even though I can see that the track function is called by Vue as expected.

Still unable to reproduce

in Chrome incognito mode, ensure there are no browser extensions
manually collect garbage
JS heap size, DOM Nodes, JS event listeners. Their numbers will not continue to grow.

I also deliberately launched the demo in incognito mode. As you can see from the screenshot, the memory is gradually growing from snapshot to snapshot. Deps is also growing and is not cleared. Even deliberately calling the garbage collector from devtools does not improve the situation.

Playground with PROD env
There are no memory leaks in PROD env.
Steps to confirm

open the link, take heap snapshot
click start button and wait 10 seconds
click stop button
collect garbage
take heap snapshot
compare two snapshots
repeat 2 - 6

Maybe related to b14cd9a68bab082332b0169be075be357be076ca

Thank you very much for this patch, it explains why all my applications made with the Quasar framework lagged, and the browser froze.

Hello,
@yyx990803 @ppyl-datBoi  :  Version 3.5.5 had solved the problem, the new version 3.5.6 brought back the problem, but now my application made with Vue lag after 15 min of use, and crashes totally after 1h to 2h, is this a new memory leak?

@rxdiscovery
Does the reproduction of this issue reproduce your problem? if not, please create a new issue with a minimal reproduction.

Might be related to c74bb8c2d
@rxdiscovery can you try the commit release here (set vue dependency in package.json to https://pkg.pr.new/vue@cbc39d5) to see if it works?

Hello,
I'm currently investigating the cause, there are 3 potential causes  :

Firefox + Live debug mode with Vite / Webpack
Vue
Quasar framework

@edison1105 @yyx990803
I've also reported the problem here with a video demonstration and the steps to reproduce it.
https://github.com/quasarframework/quasar/issues/17520

@rxdiscovery
please also try the preview packages from https://github.com/vuejs/core/pull/11971 to confirm if it can resolve your problem.

Hello,
to avoid any pollution, I created a new virtual machine with debian and the latest version of nodeJS and the latest version of Firefox.
The same problem occurred
then I tested your suggestions:
@yyx990803

modification of the dependense

"dependecies" : {
...
"vue" : "https://pkg.pr.new/vue@cbc39d5"
...
}

yarn cache clean
rm yarn.lock
yarn install
quasar clean
quasar dev
clean browser cache
check ===> same problem

@edison1105

modification of the dependense

"dependecies" : {
...
"vue" : "https://pkg.pr.new/vue@11971"
...
}

yarn cache clean
rm yarn.lock
yarn install
quasar clean
quasar dev
clean browser cache
check ===> same problem

even if I force my Vue version, does quasar still use 3.5.5 internally?

@rxdiscovery
Please try to provide a minimal reproduction without quasar. I will take a deep look then.

There is a leak in Vue, it is located at the level of :
<EMAIL_ADDRESS>Chrome memory profiling :

Firefox memory profiling :

the only thing we need is a minimal reproduction not screenshots

@edison1105  I'm currently analyzing the source code of Quasar's “QSelect”, to reproduce something similar in Vue, then I'll be able to give you a code without any dependencies.

@rxdiscovery try v3.5.7 by the way. We fixed some memory leaks yesterday.

@rxdiscovery try v3.5.7 by the way. We fixed some memory leaks yesterday.

I've already done the test, but unfortunately the problem persists.


Request: Game Consoles

PS1, PS2, PS3, PS4, Xbox, Xbox360, Xbox one, Snes, Nes, Mega drive, Wii, WiiU, Gameboy, etc.

There are icons now for PlayStation, Xbox, Nintendo and Sega. I’m not going to go down to the specific console logos (where they exist). Hope that’s enough for you!


Replace openshift/no-webhook manifests with legacy

What this PR does / why we need it:
This releases a new manifest type "legacy" to support Kubernetes <1.15 and OpenShift 3.
This version uses the webhook but disables the conversions as they are not supported.
For this reason only the v1alpha2 API is added in these manifests.
All newer APIs are filtered by the filter-crd tool.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Fixes #2639
Release note:

Replace openshift/no-webhook manifest variants with a "legacy" variant

Okay your refactoring makes a lot more sense now 😄  thanks!

/lgtm
/approve

/retest


fix(core) on_error accepts strings and numbers

Not defaulting to Lapis error page when parameter is not a table.
Full changelog

on_error accept non table parameters

I think I will merge this as a feat (from our chat, there is no "broken" feature because of this behavior iirc, it is rather a safeguard?)


how to edit the type of an existing attribute in a domain-model in Studio 9?

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: domain-model
Document link: _index.md
My Issue/Suggestion
how do I modify an existing attribute's type? I am using Studio 9.

Hi @goodfriends1 ,
Thank you for your feedback.
If you click an attribute in the domain model and then open the Properties tab on the right, you should be able to change the attribute type there.
Best regards,
Maria

@goodfriends1,
We also have a document on attribute type migration that contains more detailed information. I hope this helps.
Best regards,
Maria


Expose Regex memory usage info (fixes #943)

@BurntSushi sorry for the ping but I wanted to bring your attention to this PR, it would be great if this could get merged

@spektom can you also expose the function on RegexSet?

I have concerns about this that I explained in #943. Those need to be resolved first.

I have almost the same usecase as @Ambyjkl - I basically want a "LRU cache" of regex rules as I'm in a semi-constrained memory environment. In my opinion, calling it memory_usage_estimate is more than good enough providing we document it properly.


Explain unnamed arguments

The first use of an unnamed argument is here: https://plfa.github.io/Relations/#transitivity
However, the syntax isn't explained... and in fact, _ is explicitly ruled out (erroneously, but perhaps for the best) as a character that may occur in an Agda name.
We should probably address this.

What do you mean by "unnamed argument"? If you mean _, it is explained in the text. If you mean implicit arguments, they are explained in a section earlier.

Oh, so it is! I must've read over that.


Block domain 6969plum.litemint.store

Domain: 6969plum.litemint.store
Requested by @jaggernaut85

Fraud. Harrassment

@jaggernaut85 Can you supply information about what his NFT creator has done that you believe is fradulent. NFT's are a different kind of crypto that is not the kind of asset that is normally flagged.
This issue can be reopened if you have new information.


[BUG] Fix GitHub Actions to Match Pattern Used in Template Repository

Description
The current GitHub Actions in this repository do not fully align with the structure and patterns used in the template repository. To ensure consistency and maintainability across projects, it is necessary to update this repository's GitHub Actions to match the template.
Tasks

Review the GitHub Actions used in the template repository.
Identify differences between the current and template workflows.
Update the workflows in this repository to follow the same structure, naming conventions, and practices as the template.
Test the updated workflows to ensure they work as expected.

Expected Outcome
The GitHub Actions in this repository should have the same structure, patterns, and functionality as those in the template repository, ensuring consistent CI/CD practices.
Additional Information

Link to template repository for reference: [Template Repository Link]
Ensure compatibility with existing project dependencies and requirements.

@gstraccini create labels


treat headers as case-insensitive, as they should be

I don't really like the name "parsed", maybe better if I redo the naming to be headers and original_headers? Assuming we need to keep the headers that haven't been casefold'ed?

Coverage increased (+0.1%) to 75.676% when pulling eea39dc309fa753fca24564c23c2cb4616fe7439 on tsloughter:case-insensitive-headers into 581ca7ad81b198ab5dcc30a81ac075ba30baced5 on elli-lib:main.

Fails on 19 and 18.
Do we need to support those or is 23, 22, 21, and 20 enough?

I'd support 19.3. What's failing?

redo the naming to be headers and original_headers

Won't this break interface for developers relying on name headers, or is this opaque? I'd prefer not to break interface, but it's ultimately not my call.

@paulo-ferraz-oliveira just internal and not different from the API in this PR.
But note that this will need to be elli 4.0 anyway, since get_header now returns different results than before.

@tsloughter, sure. I'm OK with breaking stuff when it's justifiable :) I just thought headers wouldn't be, but since it's internal it's no prob.

How about support for 19, though? We'll surely move to 20 or 21 when rebar3 does, but that might still take a while.

:+1: to headers and original_headers. I should be able to review properly, and maybe fix the OTP 19 issue tomorrow.

fix the OTP 19 issue tomorrow

This would be much appreciated. 👍

Supporting 19 is pretty bad.. forgot string:lowercase won't take a binary in 19. I've updated the PR now to convert the binary to a list and then back to a binary, but that is obviously inefficient. We can keep 19 if people find that really important but I'd argue supporting 4 major versions is adequate and 19 is worth dropping.

Supporting 19 is pretty bad.. forgot string:lowercase won't take a binary in 19. I've updated the PR now to convert the binary to a list and then back to a binary, but that is obviously inefficient. We can keep 19 if people find that really important but I'd argue supporting 4 major versions is adequate and 19 is worth dropping.

Hm... I don't actually mind you dropping it. It just means we'll get stuck with the latest (as-is = 3.3.0) for a while longer, but we've not experienced issues with it, either, so that might be fine.

future backports to the 3.3.0

That sounds good to me. I wonder how many users are still on 19. Does hex.pm get any useful metadata from clients?

That would be useful, but I don't think Hex does.

Removed support for 19.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@fbec0ce). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #92   +/-   ##
=======================================
  Coverage        ?   75.56%
=======================================
  Files           ?       12
  Lines           ?      749
  Branches        ?        0
=======================================
  Hits            ?      566
  Misses          ?      183
  Partials        ?        0

Continue to review full report at Codecov.

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


[Icon] - cloud upload error

Steps to Reproduce

Do something
"element-ui": "^2.3.4",

Expected
"cloud upload icon"
Result
"map outline icon"
Testcase
(fork this to get started)
http://jsfiddle.net/rduvhn8u/1/

Can't seem to replicate http://jsfiddle.net/8ej7ew50/


Configuring gazebo and questions about gazebo_ros2_control

Hello, I wanted to create a custom controller but it seemed that it's impossible to make a custom controller and link it to gazebo or moveit within this repository. Could someone give some advice on how to make a custom configuration for gazebo or how to custom the gazebo_ros2_control.
Is there actually a way to make a custom controller and simulate it ?
thank you.

Also I wanted to ask if someone would be able to explain how does the simulation here actually works. I saw that there is a fake controller that mocks the robot but I don't really understand how does it link with the simulation.


mgmt, local litegen paloaltonetworks 1.1.0

Description

Updated codegen to 4.1.24
Use local specs with x-ms-pageable removed

Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-paloaltonetworks-ngfw

We may actually check in the directive.
At least notify the chat group, so next people knows about this.

We may actually check in the directive.
At least notify the chat group, so next people knows about this.

Sure. Will notify in the chat group.
They said they will fix this in 2023-10-10 version next month, so I prefer not checking in the directive. The fixed swagger(as well as the non-fixed one) will cause breaking changes, hence the reminder.


JSHint should not return break if all execution branches are terminated

Given:
function foo(response)
{
    var settings = response.settings;
    var xhr = response.xhr;
    switch (xhr.status)
    {
        case 201:
        {
            try
            {
                return new URI(xhr.getResponseHeader("location"));
            }
            catch (e)
            {
                throw Utilities.onAjaxError(e, settings, xhr);
            }
        }
        default:
            throw Utilities.onAjaxError("Unexpected response code", settings, xhr);
    }
};

JSHint warns: Expected a 'break' statement before 'default'.
This is incorrect because all branches of case 201 return or throw an exception.
This was originally brought up in https://github.com/jshint/jshint/issues/145 but the issue was resolved by providing a workaround to disable the warning instead of fixing the actual detection logic.
I think adding /* falls through */ at the end of the case statement would be misleading (because execution does not fall through) and would confuse code maintainers.

Isn't adding a break statement that never gets hit a good way of telling people reading the code that it doesn't fall through and stopping the error? I think that's the best option for now.
This probably involves the same kind of fixes I discussed on #2446 in order to calculate what is and is not reachable.

Luke,
I'm against your proposal. Human beings are smart enough to understand that the above statement does not fall through. Every time we have to clutter the code for the sake of jshint we end up reducing readability. jshint should just do the right thing.
If humanly possible, implementation details shouldn't leak to the end-user.

I did mark this as a bug (p3) but its a big job and you have a workaround hence the lower priority.  My "proposal"  was only a temporary fix.

Sorry for the misunderstanding. Thank you for providing a workaround.

@cowwoc When you have a lint rule for unreachable code there is no way to use switch statement as you proposed. You either will get the lint error for not having a break, or if you add a break you'll get a lint error saying you have an unreachable code (the break statement).
Unless I didn't thought of something, there is no workaround other than dropping one of the rules or dropping the switch altogether.

+1, I was looking at my pretty simple if-return-if-else-return-else-throw for a quite a few minutes trying to understand what I did wrong.


fix: add support for Elixir v1.17.0

Elixir v1.17.0 just released. The install script is failing because the OTP version if/else block isn't up-to-date.
Renovate is effectively broken for Elixir right now: https://github.com/sheerlox/elixir_renovate_demo/pull/6
Renovate log from the PR above (Mend Renovate job id 7e88e954-625d-4a07-804b-45eefc1a151c):
[17:27:45.046] INFO (193): Installing tool<EMAIL_ADDRESS>installing v2 tool elixir v1.17.0
Download failed: https://github.com/elixir-lang/elixir/releases/download/v1.17.0/elixir-otp-24.zip
[17:27:45.616] INFO (221): Downloading file ...
    url: "https://github.com/elixir-lang/elixir/releases/download/v1.17.0/elixir-otp-24.zip"
    output: "/tmp/renovate/cache/containerbase/f7649696d9e36012fd17fe14a3344673897f8c089328bc8a15abaf579e0f5ab1/elixir-otp-24.zip"
[17:27:45.764] FATAL (221): Response code 404 (Not Found)
    err: {
      "type": "HTTPError",
      "message": "Response code 404 (Not Found)",
      // ...
    }
[17:27:45.890] INFO (221): Download completed with errors  in 281ms.
[17:27:45.901] FATAL (193): Command failed with exit code 1: /usr/local/containerbase/bin/install-tool.sh elixir 1.17.0
    err: {
      "type": "Error",
      "message": "Command failed with exit code 1: /usr/local/containerbase/bin/install-tool.sh elixir 1.17.0",
      "stack":
          Error: Command failed with exit code 1: /usr/local/containerbase/bin/install-tool.sh elixir 1.17.0
              at makeError (/snapshot/dist/containerbase-cli.js:39455:13)
              at handlePromise (/snapshot/dist/containerbase-cli.js:40170:29)
              at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
              at async InstallLegacyToolService.execute (/snapshot/dist/containerbase-cli.js:56573:5)
              at async InstallToolService.execute (/snapshot/dist/containerbase-cli.js:56765:9)
              at async InstallToolShortCommand.execute (/snapshot/dist/containerbase-cli.js:57006:14)
              at async InstallToolShortCommand.validateAndExecute (/snapshot/dist/containerbase-cli.js:2428:26)
              at async _Cli.run (/snapshot/dist/containerbase-cli.js:3541:22)
              at async _Cli.runExit (/snapshot/dist/containerbase-cli.js:3549:28)
              at async main (/snapshot/dist/containerbase-cli.js:57200:3)
      "shortMessage": "Command failed with exit code 1: /usr/local/containerbase/bin/install-tool.sh elixir 1.17.0",
      "command": "/usr/local/containerbase/bin/install-tool.sh elixir 1.17.0",
      "escapedCommand": "\\"/usr/local/containerbase/bin/install-tool.sh\\" elixir 1.17.0",
      "exitCode": 1,
      "cwd": "/tmp/renovate/repos/github/sheerlox/elixir_renovate_demo",
      "failed": true,
      "timedOut": false,
      "isCanceled": false,
      "killed": false
    }
[17:27:46.150] INFO (193): Installed tool elixir with errors in 1.1s.

Thanks for the PR. I was wondering: should we pick the minimum supported, the maximum, or try to find a common It looks like 1.14-1.17 all support 25, for example

Are there any benefits for this container image to use a common like 25?
The reason I've picked the latest for 1.17 is so we don't need to upgrade this script back for longer :sweat_smile:
For 1.14 I just kept what was already in there.

Didn't realize the script supported versions < 1.14, so initially got a bit lazy with my bash operators lol. That should fix it.

Are there any benefits for this container image to use a common like 25?

I guess the benefit is simplicity, I've updated so 25 is used from 1.14 to 1.16 (keeping 27 for 1.17 for the reason exposed above)

because someone can rely on older erlang

because someone can rely on older erlang

If Erlang can be installed alongside Elixir, then this script should probably be aware of which Erlang version is installed right?

don't remove compability code for older elixir versions

Although not expressed in the same way, my latest commit should now preserve the previous behavior

@sheerlox thanks a lot for the fix!

Looks like the release workflow failed because ppa.launchpad.net was unavailable (503  Service Unavailable).
Any chance we can retry the failed job? :pray:

released around two hours ago 🤗

Can confirm Elixir dependencies upgrade is working again in Renovate 37.407.4 (on a private GitHub Actions workflow).
Mend Renovate is still behind though so FOSS repositories are still impacted.

@nabeelsaabna can you update the app on Sunday?


Fixed typos

Q
A

Bug fix?
no

New feature?
no

BC breaks?
yes maybe

Deprecations?
no

Related tickets
n/a

Documentation
n/a

License
MIT

What's in this PR?
Fixed some typos

Thank you for this PR.
Can you make StyleCI happy as well?

@Nyholm done.

Thank you!


[low prio] Improve invitation mail

[ ] Add URL to application
[ ] Maybe add some details about game, like name.

@SeineEloquenz to add the url to the application, we will need a way to get the address to the tomcat server, so i'd say we get that from the properties file

You mean adding the application url to the config?

yes

5a5cc8f added the config setting @bendixsonnenberg. Docker repo still has to be updated

Added the value to the docker repo, all users of the docker-compose solution will have to update

Game name as well as application url are now added to the invitation mails as of 144aff7694d4d295b18b2ffc0320a4924a2ba4b6


added full stop at the end of the first paragraph

Related Issue
Updated readme section
Description
Added a full stop at the end of the first paragraph to make it more grammatically accurate.
Screenshots

invalid