Localized String with placeholder uses wrong encoding on Umlaut Argument

I have the following Gerrman String in my Localized.strings file:
"rsvp_text" = "Bitte gib uns bis zum %1$s Bescheid, ob du dabei bist.";
As you can see there's a %1$s argument which is recognized by R.swift perfectly fine.
Now the string which is inserted as an argument is also localized and looks like this:
"rsvp_date" = "01. März 2022";
Please note the German Umlaut "ä".
The code I'm using to compose this string is the following:
text = R.string.localizable.rsvp_text(R.string.localizable.rsvp_date())
The problem is, the date is displayed as follows: Bitte gib uns bis zum 01. M√§rz 2022 Bescheid, ob du dabei bist.
So far I tried to use .cString versions with different encodings, different Localizeable.strings file encoding, but nothing helped.
Basically Umlauts are working fine in "plain" strings but as soon as there's argument strings with Umlauts, they're getting messed up.
Do you know a workaround for this or can even provide a fix?
Thank you in advance!

Use %@ for strings, or in your case<EMAIL_ADDRESS>See:  https://developer.apple.com/library/archive/documentation/Cocoa/Conceptual/Strings/Articles/formatSpecifiers.html#//apple_ref/doc/uid/TP40004265-SW1

Thank you so much, that worked. %s was a relict from Android Strings.


[release-1.20] Upgrade k3s-root to 0.9.1 version

Backport fix for "Upgrade k3s-root to 0.9.1 version" to branch release-1.20:

#3625

Validated sonobuoy conformance tests are passing on v1.20.10-rc1+k3s1


Chapter attention mechanisms/transformer

Job d2l-zh/PR-772/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-772/

Job d2l-zh/PR-772/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-772/

Job d2l-zh/PR-772/3 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-772/


Webpack 5, Angular 12: TypeError: Cannot read property 'add' of undefined; fileDependencies undefined on compilation property

When I want to use Webpack 5 with Angular I got the following error:
TypeError: Cannot read property 'add' of undefined
at
node_modules\@ngtools\webpack\src\ivy\plugin.js:170:86
The error occurs also when I want to use it with Angular 11.
The error occurs at this position in the plugins.js:
...
            if (webpack_version_1.isWebpackFiveOrHigher()) {
                allProgramFiles.forEach((sourceFile) => compilation.fileDependencies.add(sourceFile.fileName));
            }
...

compilation seems not to have a fileDependencies property.
My package.json:
...
		"webpack": "^5.37.0",
		"webpack-bundle-analyzer": "^4.4.0",
		"webpack-cli": "^4.5.0",
		"@types/webpack": "^5.28.0",
		"@types/node": "^15.0.3",
		"@angular-builders/custom-webpack": "^11.1.0",
		"@angular-devkit/build-angular": "^12.0.0",
		"@ngtools/webpack": "^11.2.13",
...

The build and serve statement looks like this in angular.json:
...
				"build": {
					"builder": "@angular-builders/custom-webpack:browser",
					"options": {
						"outputPath": "dist/apps/app",
						"index": "apps/app/src/index.html",
						"main": "apps/app/src/main.ts",
						"polyfills": "apps/app/src/polyfills.ts",
						"tsConfig": "apps/app/tsconfig.app.json",
						"aot": true,
						"assets": ["apps/app/src/favicon.ico", "apps/app/src/assets"],
						"styles": [
							"libs/shared/ui/tailwind/src/lib/styles/tailwind.scss"
						],
						"scripts": [],
						"customWebpackConfig": {
							"path": "./libs/shared/webpack/src/lib/config/webpack.config.js"
						}
					},
					"configurations": {
						"production": {
							"fileReplacements": [
								{
									"replace": "libs/core/config/src/lib/environments/environment.ts",
									"with": "libs/core/config/src/lib/environments/environment.prod.ts"
								}
							],
							"optimization": true,
							"outputHashing": "all",
							"sourceMap": false,
							"extractCss": true,
							"namedChunks": false,
							"extractLicenses": true,
							"vendorChunk": false,
							"buildOptimizer": true,
							"budgets": [
								{
									"type": "initial",
									"maximumWarning": "2mb",
									"maximumError": "5mb"
								},
								{
									"type": "anyComponentStyle",
									"maximumWarning": "6kb",
									"maximumError": "10kb"
								}
							],
							"customWebpackConfig": {
								"path": "./libs/shared/webpack/src/lib/webpack.prod.config.js"
							}
						}
					}
				},
				"serve": {
					"builder": "@angular-builders/custom-webpack:dev-server",
					"options": {
						"browserTarget": "app:build",
						"proxyConfig": "apps/app/proxy.conf.json"
					},
					"configurations": {
						"production": {
							"browserTarget": "app:build:production"
						}
					}
				},
...

The webpack.config.js looks like:
module.exports = {
	node: {
		fs: 'empty'
	}
};

From the above package.json contents, it appears that @ngtools/webpack version 11.x is being used but can most likely be removed from the package.json.  Only version 12.x officially supports Webpack 5 and this version is automatically provided by<EMAIL_ADDRESS> Angular CLI 12 also includes and will only use Webpack 5.  As a result, Webpack 5 can also probably be removed from a project's package.json (assuming it is not used in some other non-Angular CLI capacity).
It also appears that a third-party builder (@angular-builders/custom-webpack) is in use.  Third-party builders are not directly supported by the Angular team.  The builder may also not have been updated to support Angular v12 yet.  Please contact the maintainer of the third-party builder if the project continues to experience issues after making the above suggested changes.


R: Added stub tidyverse in R

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Why only a stub? It has no value. I suggest closing.

Hi,
Thanks for this pull request (PR).
Unfortunately, we are closing this PR, because we do not believe it adds a significant improvement.
Feel free to submit another PR though.
Thank you


scripts/generate_deployment_manifest = empty manifest?

Hello!
When I run:
./scripts/generate_deployment_manifest openstack ~/workspace/git-cloudfoundry-on-openstack-talk/private/cf/cf-manifest.yml > cf-deployment.yml

I get:
±  |master ?:1 ✗| → cat cf-deployment.yml
{}

Anyone know what the deal here is?

Seems to be a spiff thing [https://github.com/cloudfoundry-incubator/spiff/issues/81]


Query invalidation not working on mutate success

I am using mutate and and on successful mutate I am calling query invalidate.
this gets fired for sure but doesn’t invalidate the query. I saw someone else was also having same issue in discord channel

please create a codesandbox reproduction. Most common gotchas are:

new QueryClient() created in the app component, which will create a new cache when re-rendering
Using the wrong query key


libupnp.pc: drop -pthread from Cflags

while discussing with MPD upstream it was pointed out that -pthread isn't necessary in the output of pkg-config --cflags libupnp.
Other than that it strikes me that @PTHREAD_CFLAGS@ is used in the Libs: line and I wonder if this is correct. (For static linking -pthread is needed even for single threaded consumers. And it seems it doesn't need to be at the end to get it in the right linking order while the order of -lupnp -lthreadutil -lixml matters.) Will think about this for a while ...

Found this on the autoconf ML archive:
http://lists.gnu.org/archive/html/autoconf-archive-maintainers/2013-06/msg00021.html
So it looks strange, but is by design.


New Hallow's End items ilvl update

Is your feature request related to a problem? Please describe.
Per https://www.wowhead.com/wotlk/news/hallows-end-holiday-event-guide-wotlk-classic-335605 the new ilevel on Hallow's End items is 232. Confirmed this is true for the rings by witnessing them in game. Have not seen a helm or sword.
Describe the solution you'd like
Updated stat blocks for these items.
Describe alternatives you've considered
I don't know if it's possible to manually add items or not, I suspect not. If so, that would work too.
Additional context
n/a

added in #3923


Encoded OpenAgro full submission into JSON

Encoded OpenAgro full submission into JSON

OpenAgro provided incomplete information in their submission to enable us complete the review process. The following indicators need to be addressed:

Clear Ownership. They are missing copyright, trademark, or other publicly available information to demonstrate clear ownership of everything the project produces.
Adherence to Privacy and Applicable Laws. The project indicated they are not compliant with relevant privacy laws, and all applicable international and domestic laws, which is not a valid answer.

PS: We shall review their feedback once we hear from them again.

AI Agro initially submitted with several gaps in the DPG standard indicators which need to be addressed before we can proceed with the review:

[ ] Clear Ownership
[ ] Platform Independence
[ ] Documentation
[ ] Mechanism For Extracting Data
[ ] Adherence to Standards & Best Practices
[ ] Adherence to Privacy and Applicable Laws
[ ] Do No Harm
[ ] Data Privacy & Security
[ ] Protection from Harassment
[ ] Deployment countries

I have contacted the person who made the submission and will update once they come through.


Slight tweak to checkInternet

I searched around for a better way to check internet connection and this seemed to be the most often used and suggested method for doing so without going into too much detail and code. Would you have suggested or done something different?

This looks good.
We should also have a method to see if the connection to the internal network was successful. I think probably the best way to check that is if the current IP address is not an APIPA address i.e. not beginning with 169.x.x.x


Sandbox Mode

https://sendgrid.com/docs/Classroom/Send/v3_Mail_Send/sandbox_mode.html
Here is my sandbox response :
{"errors":[{"message":"Bad Request","field":null,"help":null}]}
Totally and completely useless!!!!!

Hello @usulix,Thanks for your suggestion or question! In an attempt to clear out our backlog,                                              we are closing out issues over 90 days old.                                             If this is still an issue, please feel free to reopen it.                                             Team SendGrid DX-Docs


Add static error for instantiating an abstract class

Motivation
The runtime has a check for this but it can be defeated if the abstract
class defines initialize. Efficently fixing that problem in the
runtime involves using prepend, which is risky as it invalidates
assumptions scattered around the codebase.
This only checks calling new on a class marked directly with
abstract!. Subclasses of abstract classes that are not concrete
already get errors about missing definitions so we should be okay there.
Thanks to @jez for suggesting where to put this check!
Closes #2143
Test plan

See included automated tests.

I tested this on Stripe's codebase: (results: http://go/8QYdsV86)
It introduces 146 new errors for Attempt to instantiate abstract class.
@DarkDimius Would you mind taking a look at this since you're on run?

Unfortunately, I don't think this will fly in ruby.
class A;
  abstract!
end
class B < A
end

sig{params(a: T.class_of(A)).void}
def takesA(a)
  a.new
end

def assignsA
  a = T.let(B, T.class_of(A))
  a.new
end

^^^ are common patterns.

Thanks for the reviews. Huh, that's very interesting. It looks like this check can't be performed without the dynamic type of the receiver. It's not very doable in the static checker, then.


fix(inputs): add background color on disabled state

Context
Add greyLightest  background  color to disable state for

Dropdown
InputText
RadioField
CheckBoxField

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


startup issue

Describe the bug
So, basically i have ticked the option of enable on startup but it doesn't auto start on startup its disabled
To Reproduce
idk man i have put attachment pls see
Expected behavior
should go on startup
Screenshots

Desktop (please complete the following information):

OS: [Windows 10]
Python Version: latest

did you manually check the registry?

wdym? pls explain i m new

did you manually check the registry?

if you checked registry aka regedit.exe

yes i did

if you checked registry aka regedit.exe


implement --timestamp, iso8601-like utc timestamp string or reference file

note: this needs bug #282 to be fixed first, because it will create timestamps with microseconds==0.
Without the #282 bug fix, one can work around by always giving a non-zero fraction of a second in the timestamps, e.g. 2015-04-19T16:14:15.1Z.
fixes #263.

Hmm, that's annoying that github puts each of my line comments as a separate pull request comment. It makes it rather noisy and loses the context of the comments. (If you can't tell, this is my first github code review as a reviewer).

I really feel it is wrong to force users to use UTC. datetimes should always be stored in UTC, but user interaction should default to their local time (with an option for UTC if the user desires).
Allowing only UTC timestamp strings shows up as in incongruity when listing the archives in a repository. The output of 'attic list ' explicitly converts archive timestamps to local time (see helpers.py:format_archive() and helpers.py:to_localtime()).
I do not feel it is necessary to support arbitrary timezones (just localtime and UTC), and this can be achieved without external dependencies (such as pytz which is usually called in as soon as you need to deal with multiple timezones). Implementing an equivalent helper to to_localtime (from_localtime) could be done to support this.

@camh- thanks for the feedback, I'll see how I can improve it.

@camh- I agree with all what you said and improved the code.
BUT: tz support sucks. :| Thus, I did not add localtime or TZ support. I didn't want to require additional dependencies or complicate the code too much.

Generic TZ handling is painful (and requires something like pytz as an additional dependency), but local time support is not too hard. How about something like (untested in totality):
def from_localtime(dt):
  seconds = time.mktime(dt.timetuple())
  return datetime.utcfromtimestamp(seconds).replace(microsecond=dt.microsecond)

And the body of the time string conversion:
import itertools

dates = ['%Y-%m-%d', '%Y-%j']
times = ['T%H:%M:%S.%f', 'T%H:%M:%S', 'T%H:%M', 'T%H', '']
zones = ['', 'Z']
for format in itertools.product(dates, times, zones):
  try:
    dt = datetime.strptime(s, ''.join(format))
    return dt if format[2] == 'Z' else from_localtime(dt)
  except ValueError:
    pass

Generic TZ handling is painful (and requires something like pytz as an additional dependency), but local time support is not too hard. How about something like (untested in totality):

Agreed, to_localtime uses a similar trick.

@camh- nice trick with the product. I'll work on integrating that and localtime support. BTW, I kept the "T" separate in case someone wants rather a space than a T, but that can be done in the same way.

OK, updated.
While doing that, found a slight issue with to_localtime - it does not support fractions of a second.
Chose to do the same with from_localtime and in general not to support fractions of a second with --timestamp (likely noone needs that anyway).

closing this pull request, seems unwanted.


chore: Add javascript hindi course link

Fixes Issue
closes #2078
Changes proposed
Added code with harry javascript course link to category: languages , sub-category: javascript
Screenshots

Note to reviewers

@ankush-w17
Make sure to mention the channel name, I always see it before assigning. Code with harry already exists for web development.
We will go forward with this, but make sure to do it next time.
Didn't you check existing issues, it's better because PR getting closed is not worth it.

@Anmol-Baranwal I will make sure to mention the channel name from next time and have edited the description of this PR to update the name.
However, regarding the issue that Code With Harry already exists for web development, I think mentioning the course in the javascript section will be helpful to the viewers of the site as it will spare them from the search on the channel.

@Anmol-Baranwal or @CBID2 can you please review this PR. Sorry for disturbing you but I am in a hurry as hacktoberfest is ending, hope you would understand.

@ankush-w17
You don't need to worry, we will resolve all the issues & PRs that will be created by today or earlier.


mangaraw.is

Source name
mangaraw.is
Source link
https://mangaraw.is/
Source language
japanese
Other details
18+/NSFW = yes
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Site redirects to https://raw1001.co/

New URL: https://mangaraw.fan/


Harfbuzz version too old (1.1.3)

Extension Version: 0.4.5
VS Code Version: 1.51.1
OS Version:  Linux x64 5.4.74-1-MANJARO
Node.js Version (result of command node -v): v14.15.0 (Installed through nvm)
NPM Version (result of command npm -v): 6.14.8

Steps to Reproduce:

Install extension while having an updated version of pango library.
Try to use it.

Possible solution:
While searching for this error, I saw that a lot of people say that updating the electron used for the package fixed this. This extension is also using electron-prebuilt which is marked as deprecated and it was renamed to electron. I also saw that you were pointing people into downgrading glibc(even though you were saying "I don't know whether this is best solution or not") which is not a great solution in my opinion. This extension needs to be updated as it uses packages that have severe vulnerabilities.

Hi @ferretwithaberet, thank you for the comment.
I think that this extension has already finished its part since VS Code supports internal Color Picker.
I recommend using internal Color Picker instead.

No reply came, then this abandoned issue is closed.


UIMenuController causes text in bubbles to flicker rapidly

Attempting to paste text by holding down in the text field and bringing up the UIMenuController causes some of the text (but not all) in existing message bubbles to start flickering rapidly and randomly. It doesn't happen 100% of the time, but happens quite frequently.
See attached GIF for video:

I am experiencing the same issue. After testing, it looks to me like this is an iOS 8 specific issue. Can anyone else confirm?

Same happen to me

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


QBO: Fix connection missing

Details

While connecting to QBO, we can see the Last synced with an empty date appeared briefly. This PR fixes it.
Fixed Issues

$ https://github.com/Expensify/App/issues/46445
PROPOSAL: https://github.com/Expensify/App/issues/46445#issuecomment-2407089686
Tests

Create new workspace.

Go to the Accounting screen, and click on QBO connection to connect with QBO.
Verify that you can connect to QBO successfully and during the process, you won't see the Last synced with an empty date.

[x] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Same as Tests.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/b73a6063-62a0-447a-9fbd-ad6a69cc82cd

Android: mWeb Chrome
https://github.com/user-attachments/assets/f9c8cc7f-8f65-4722-9e7b-385965ff4deb

iOS: Native

https://github.com/user-attachments/assets/9ef51326-0fe6-4f5b-b1be-50efcd59db07

iOS: mWeb Safari

https://github.com/user-attachments/assets/50b220c8-a250-4180-bccd-98ba9c46436a

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/87e144e5-0cc5-47d5-93ad-24954a65945b

MacOS: Desktop

https://github.com/user-attachments/assets/56a4841e-1c6a-4d02-ace8-b1942efd6b02

@ishpaul777 oh shot! Looks like you can reproduce the bug in the OP. Can you reproduce it consistently. If Yes can you please inspect the Pusher event on verbose tab and let me know the result.

Hmm I don't think I can reproduce, do you have specific steps to reproduce it?
https://github.com/user-attachments/assets/30f363ad-9a5c-4d52-bebc-be072fe3fba7

Yeah, if you've navigated to new.expensify.com, it looks like you were connected to prod env. I used to reproduce https://github.com/Expensify/App/pull/50766#issuecomment-2417222994 on Staging but couldn't reproduce it after few days while working on this idea. .
Are you turning on the Use Staging Server switch?

As I explained in here: https://github.com/Expensify/App/issues/46445#issuecomment-2390712753
The RCA of the missing menu item is that Pusher event never sent any updates until next manual sync. I think there is a PR on Staging fixed it but it isn't deployed to Prod yet.

Sorry for the delay. Taking a look now.

Tests well! Thanks!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Conflict version of twig/twig in composer.json file

Q
A

DoctrineAuditBundle version
2.5.0

PHP version
7.2.14

Database
MySQL

Summary
The conflict version of twig/twig seems to be incorrect in the composer.json file.
Current behavior
Composer return an error due to a conflict with twig/twig if you try to install the bundle with a version of twig/twig higher or equal than the 1.40.
How to reproduce
Create a new folder for the test
composer require "twig/twig:^1.0"
composer require "damienharper/doctrine-audit-bundle:^2.5"
The CLI return :

Expected behavior
The bundle should be installed successfully if the version of twig is higher or equal than the 1.40.
Put the conflict version of twig/twig to "<1.40|<2.9,>=2.0" should solve the problem.
I just tried for the 2.5 version of the bundle.
The behavior is maybe the same for the 3.x.

@Phibus Thanks for the feedback, I'll have a look at it ASAP.

Also, could you please try with the latest version of the bundle?

@Phibus could you post your composer.json (at least the require section) to ease debugging please?

Hello @DamienHarper.
You can find the composer.json I used below.
{
    "require": {
        "twig/twig": "^1.0",
        "damienharper/doctrine-audit-bundle": "^2.5"
    }
}

The composer's version used is the 1.9.1.

@Phibus could you try updated-twig-deps branch?

@Phibus do you still experiment the issue with master branch since twig des have been updated?

@Phibus I close this issue as it hasn't received feedback recently. Feel free to reopen it if needed.


i can't run demo.html

I'm using local files. i was download all required files and project.
Examples folder html files working well. but demo.html page receiving errors begin with
"polymer-micro.html:74 Uncaught TypeError: prototype.registerCallback is not a function"

Hey all, I'm closing this ticket. The master branch will be updated to reflect the 'big-refactor' branch (v2 release). I encourage you to try out the functionality on that branch and if you still have a concern, please open a new ticket or email<EMAIL_ADDRESS>


Fixing the retrieval of server ids

This function is buggy and was throwing exceptions.

Maybe data structure changed, since this library quite old. Could you please posted raw result of current request?
Thanks

Sorry i don't the exact informations atm but you were expecting a complex array of results in the current function.
Online.net does now send a simple array of ip just as that : ["<IP_ADDRESS>", "<IP_ADDRESS>"].

I see thanks for your information :)


macOS Mojave camera access permission issues

I'm having trouble getting camera input on macOS Mojave due to a requirement for requesting permission.
I'm not sure what combination of changes I need to make so that the OS knows to prompt the user for permission. It may require code-signing. If anyone has experience with this, please let me know. If not, I'll share information here when I figure it out.
If changes to webcam-capture are required we should make them. If not, we should at least document what needs to be done to make it work.

So, it looks like all you have to do to get it working is add the following to your Info.plist:
<key>NSCameraUsageDescription</key>
<string>Uses camera to YOUR USE CASE HERE</string>

@msgilligan
I have put those lines into /Applications/IntelliJ IDEA CE.app/Contents/Info.plist, but asking for camera permission dialog does not pop up.


What is the CmpId for Publishers not using an IAB-Compliant CMP?

should they apply for a custom id? or should they use id=0?

Just register for 350 EUR as a CMP on the website and you are safe. If you sent a consent-string as a not registered CMP, you are violating the policies. That's more about legal, not about tech.
Technically you could use whatever CMP-ID you want, but it's still legally not valid.

thnx. registration has been sorted out


[Feature Request] Giving name for the root compound

I have the following struct which represents a schematic
#[derive(Serialize, Deserialize, Debug)]
#[serde(rename_all = "PascalCase")]
pub struct Schematic {
    blocks: ByteArray,
    data: ByteArray,
    tile_entities: Vec<Value>,
    entities: Vec<Value>,
    width: i16,
    height: i16,
    length: i16,
    materials: String,
}

But when I write it into a file I get the following error by WorldEdit "Tag 'Schematic' does not exist or is not first". Opening the file with an NBT Editor it shows me that the root compound's name is empty. Renaming the root compound to "Schematic" in the editor fixes the error.
Is there a way to do this with fastnbt?

adding
let mut bytes = fastnbt::to_bytes(&structure).unwrap();
bytes.splice(2..=2, b"\x09Schematic".to_owned());
bytes.extend([9,0]);

solved the issue temporarily

Hey, I don't think there's a built in way to do this at the moment. It doesn't neatly fit the serde data model as far I can tell, so it's not super clear how to add this.
Up until now it hasn't been a raised issue. It's good to have a real use for it. I will think on how to add this functionality.

fastnbt 2.5.0 now allows setting this with to_bytes_with_opts and to_writer_with_opts. There's some info in the ser module docs (once docs.rs catches up with 2.5 at least!).

I tested it now and it's working! Thank you very much 🌺

You appear to be writing some bytes to the end of your output in your workaround. As far as I can tell that's not necessary, am I missing something? It's been a while since I took a hard look at NBT's format.

I haven't ether so I don't know if it's necessary. I compared the correct schematic's binary with the generated one and saw it


Add project summary to console output and web report

This PR gathers the project summary and coverage gain information and then displays it on the console output. It also outputs the information as a JSON file in the result directory for including this information in the web report.

 Sample output in the web report. Ignore the number as it is dummy value. Will run larger experiment to test it.

Nice, I like this. I think eventually it may be nice to have a dedicated project page with more insights about improvements for a specific project, but this can wait for a bit.

Experiments running in https://github.com/google/oss-fuzz-gen/pull/492

This is really interesting and useful, thanks!
Just some minor changes above : )

@arthurscchan I'm running an experiment through in https://github.com/google/oss-fuzz-gen/pull/492 but am not seeing results, which indicates an error -- did some bug get introduced during the refactoring?

@arthurscchan could you redo this PR as a branch off OSS-Fuzz-gen so we can run gcbrun?


Delete older snapshot releases

For #2462.
Since the workflow is not active on pull requests, CI will not actually test the PR before it is merged.

It looks like the GitHub action will only delete 20 releases at a time, so it will take a few commits before we only have 10 snapshot releases left.

This needs to be restricted to only run on the OpenRefine organization, not forks. It's currently failing on my fork. It should be conditionalized the same way the various "Upload release" steps are.

Create Release0s
Upload Release Asset Windows0s
Upload Release Asset Linux0s
Upload Release Asset Mac0s
Delete older releases1s
🔴 no GITHUB_TOKEN found. pass GITHUB_TOKEN as env
Run<EMAIL_ADDRESS>🔴 no GITHUB_TOKEN found. pass GITHUB_TOKEN as env


Fallback to DefaultAuthenticateScheme

Fixes https://github.com/aspnet/Security/issues/1264
@Tratcher

Replaced by https://github.com/aspnet/HttpAbstractions/pull/873


[EPIC] WorkflowsAndLlamaAgents

Description
Incorporate Workflow into llama-agents.

[x] #178
[x] #179
[x] #180
[x] #181
[x] #182
[x] #183
[x] #184

This is done


AWS_S3_002 Check is not accurate

It seems like S3 location check is not working as it should. Here is a quick explanation of the observed behaviour
I anonymized this example by changing account names and bucket names for privacy.
Test conditions & setup
Situation :

single AWS Account, spanning over 2 regions (us-east-1 and eu-west-1)
2 config entries in .yayas.yml, one per region :aws:
  - name: "Account EU WEST 1"
    profile: "profile"
    sso: false
    region: "eu-west-1"
  - name: "Account US EAST 1"
    profile: "profile"
    sso: false
    region: "us-east-1"

Setup :

create bucket "eu-storage" in "eu-west-1"
create bucket "us-storage" in "us-east-1"

Observed behaviour on check AWS_S3_002 S3 buckets are not global but in one zone :
Name: Account EU WEST 1 (XX/63)
[...]
❌ AWS_S3_002 S3 buckets are not global but in one zone - (XX/YY)
	S3 bucket eu-storage is in eu-west-1
	S3 bucket us-storage is global but should be in eu-west-1
[...]
Name: Account US EAST 1 (XX/63)
[...]
❌ AWS_S3_002 S3 buckets are not global but in one zone - (XX/YY)
	S3 bucket eu-storage is global but shoud be in us-east-1
	S3 bucket us-storage is in us-east-1
[...]

Expected behaviour on check AWS_S3_002 S3 buckets are not global but in one zone :
Name: Account EU WEST 1 (XX/63)
[...]
❌ AWS_S3_002 S3 buckets are not global but in one zone - (XX/YY)
	S3 bucket eu-storage is in eu-west-1
[...]
Name: Account US EAST 1 (XX/63)
[...]
❌ AWS_S3_002 S3 buckets are not global but in one zone - (XX/YY)
	S3 bucket us-storage is in us-east-1
[...]

Identified cause of bug
In plugins/aws/s3/getter.go, L19 :
	resp, err := svc.ListBuckets(context.TODO(), params)

The ListBuckets function here uses the equivalent ListBuckets AWS API endpoint. This is therefore the same as running aws s3 ls. It turns out that the returned list of buckets is a global list, even if AWS_REGION=eu-west-1 is set : the region parameter is ignored.
Maybe this check should be modified, in order to check in bucket options of global replication is enabled rather than comparing the LocationConstraint on buckets ?

With new plugin architecture, the issue moved to https://github.com/StanGirard/yatas-aws/issues/5.
Closing


RangeFormattingAdapter passes internal IRange objects to extensions with proposed formatMultipleRanges

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

VS Code Version:
OS Version:

Steps to Reproduce:

Use the new formatMultipleRanges proposed API in an extension to format multiple ranges at once, such that the provider canFormatMultipleRanges.
Trigger range formatting e.g. via the FormatModifiedLines command.

The IRange passed in as range is converted to the type known to extensions on the public API as ran before being sent to provideDocumentRangeFormattingEdits. Meanwhile, options is cast as <any> and passed in as well, with an array full of internal IRanges.
Adding a line like if (options.ranges) options.ranges = options.ranges.map(typeConvert.Range.to) as any;will fix this issue, but I'm not sure if a more full-featured typeConvert for options would be preferred.

Adding a line like if (options.ranges) options.ranges = options.ranges.map(typeConvert.Range.to) as any;will fix this issue, but I'm not sure if a more full-featured typeConvert for options would be preferred.

Maybe a full-featured typeConvert for the sake of re-usability (api commands) but no more complex the outlines in the first part of the sentense

This will be fixed by the API refinement, see https://github.com/microsoft/vscode/pull/179166


Action.checkAccelerator is broken

If you define an accelerator for a Ctrl+ hotkey, the accelerator will not work.
This is because the KeyFlag enum tracks whether the left or right control keys have been pressed and the accelerator is checked for exactly equal flags.
Since in addition to the control flag, the lcontrol  or rcontrol flags are being set, the flags are no longer equal.
So we need to special-case the key flags that have multiple meanings because otherwise the accelerators will not be matching.

I believe it should be fixed in v0.8.24
Distinct left and right ctrl/shift/alt/win key in accelerators should work too.
KeyFlag.LControl should work for left ctrl only, while KeyFlag.Control should work for both left and right control keys.
Please confirm if issue is fixed

Works fine for me.

Great! Thank you!


Use devComponent in lazy for devtools support

Because lazy doesn't call the component function using JSX or createComponent it doesn't show up in the devtools properly.
This replaces the function call with devComponent if in dev mode.
Although it may be a bit odd, as devComponent is an internal-only api, and lazy should probably be fully implementable in userland, right?
Maybe there is a way to just use createComponent there, as the logic is very similar.
return createMemo(() => (Comp = comp()) ? createComponent(Comp, props) : "")

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.3%) to 86.957%

Files with Coverage Reduction
New Missed Lines
%

packages/solid/src/reactive/scheduler.ts
1
80.95%

Totals

Change from base Build<PHONE_NUMBER>1:
1.3%

Covered Lines:
2332

Relevant Lines:
2612

💛  - Coveralls


example produces no sensible output

There was a concern about bids-apps and git-annex'es symlinks. I have tried to run the BIDS-app/example on two sample datasets (ds000001 and 5 from openneuro) to arrive to the same results regardless of the files being symlinks or not.  Since ds005 was originally "demonstrated" in README.md here is full protocol for its run:
smaug:/mnt/datasets/datalad/tmp
$> datalad install -g -s ///openneuro/ds000005
[INFO   ] Cloning http://datasets.datalad.org/openneuro/ds000005 [1 other candidates] into '/mnt/datasets/datalad/tmp/ds000005'
[INFO   ] access to dataset sibling "s3-PRIVATE" not auto-enabled, enable with:
| 		datalad siblings -d "/mnt/datasets/datalad/tmp/ds000005" enable -s s3-PRIVATE
install(ok): /mnt/datasets/datalad/tmp/ds000005 (dataset)
action summary:
  get (ok: 81)
  install (ok: 1)
datalad install -g -s ///openneuro/ds000005  85.82s user 22.10s system 365% cpu 29.496

$> docker run -i --rm -v $PWD/ds000005:/bids_dataset:ro -v $PWD/:/outputs bids/example /bids_dataset /outputs group
	1: This file is not part of the BIDS specification, make sure it isn't included in the dataset by accident. Data derivatives (processed data) should be placed in /derivatives folder. (code: 1 - NOT_INCLUDED)
		/.datalad/.gitattributes
			Evidence: .gitattributes
		/.datalad/config
			Evidence: config
		/.gitattributes
			Evidence: .gitattributes

	2: You should define 'SliceTiming' for this file. If you don't provide this information slice time correction will not be possible. (code: 13 - SLICE_TIMING_NOT_DEFINED)
		/sub-01/func/sub-01_task-mixedgamblestask_run-01_bold.nii.gz
		/sub-01/func/sub-01_task-mixedgamblestask_run-02_bold.nii.gz
		/sub-01/func/sub-01_task-mixedgamblestask_run-03_bold.nii.gz
		/sub-02/func/sub-02_task-mixedgamblestask_run-01_bold.nii.gz
		/sub-02/func/sub-02_task-mixedgamblestask_run-02_bold.nii.gz
		/sub-02/func/sub-02_task-mixedgamblestask_run-03_bold.nii.gz
		/sub-03/func/sub-03_task-mixedgamblestask_run-01_bold.nii.gz
		/sub-03/func/sub-03_task-mixedgamblestask_run-02_bold.nii.gz
		/sub-03/func/sub-03_task-mixedgamblestask_run-03_bold.nii.gz
		/sub-04/func/sub-04_task-mixedgamblestask_run-01_bold.nii.gz
		... and 38 more files having this issue (Use --verbose to see them all).

        Summary:                 Available Tasks:          Available Modalities:
        498 Files, 3.54GB        mixed-gambles task        T1w
        16 - Subjects                                      inplaneT2
        1 - Session                                        bold

/run.py:86: RuntimeWarning: Mean of empty slice.
  fp.write("Average brain size is %g voxels"%numpy.array(brain_sizes).mean())
/usr/local/lib/python3.4/dist-packages/numpy/core/_methods.py:80: RuntimeWarning: invalid value encountered in double_scalars
  ret = ret.dtype.type(ret / rcount)

$> cat avg_brain_size.txt
Average brain size is nan voxels%

so would be nice to have example to provide sensible measure in the output

tested locally and see also circle-CI: https://app.circleci.com/pipelines/github/bids-apps/example/77/workflows/38b90c2e-adb3-4150-8579-7838a9cb2937/jobs/301?invite=true#step-107-2297_49


Paper about the Galaxy Training Material

Hi all,
We are writing a paper about the Galaxy Training material initiative and its infrastructure. You can find the draft here: https://docs.google.com/document/d/1m3yp7yq-Nr7nIBOql5EWriMkcYzlcQABhHyrWoR0W9U/edit?usp=sharing
If you feel you should be an author of this paper, you can add yourself between Saskia and Galaxy Training Network (while following the alphabetical order). Otherwise, we assume you are part of Galaxy Training Network.
We will close this issue in one week and submit the paper.
Saskia and Bérénice
ping @abretaud @aisanjiangw @annasyme @bagnacan @bernt-matthias @dannon @delphine-l @dpryan79 @dyusuf @erasche @fidelram @fmareuil @friedue @guerler @hmenager @hrhotz @jennydg @jfallmann @jgoecks @jj-umn @jmchilton @joachimwolff @lecorguille @loraine-gueguen @malloryfreeberg @manabuishii @martenson @moheydarian @mwolfien  @odoppelt @kilpert @pajanne @polkhe @pvanheus @pratikdjagtap @slegras @slugger70 @stortebecker @tealiie @thomasmanke @torhou @valentinmarcon @vdda @vivekbhr @willdurand @wm75 @yhoogstrate @yvanlebras

ping @valentinmarcon @vdda @vivekbhr @willdurand @wm75 @yhoogstrate @yvanlebras

Any guidelines on what sort of contribution you consider is required to participate on the paper?

Hello,
I have been contributing to tutorials by revising or reporting mistakes to
revise them when I found any mistakes in a tutorials.
Thank You,
Aisanjiang Wubuli
_______________________
MSc, Molecular Diagnostics and Biotechnology
University of Camerino, Italy
B.Sc, Biological Technology
Shandong University, P. R. China
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>Skype: uyturk2010
On Wed, Nov 15, 2017 at 9:23 AM, Fidel Ramirez<EMAIL_ADDRESS>wrote:

Any guidelines on what sort of contribution you consider is required to
participate on the paper?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/galaxyproject/training-material/issues/596#issuecomment-344519234,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Abv7KK5ObdTvuQOGsD9pqnv5lev6JlhVks5s2p9xgaJpZM4QcP2c
.

Any kind of contributions: work on the infrastructure, development or improvement of content, etc.
The question is more if you identify yourself with this project and this paper :smile:

Although I did contribute to the training material, It's difficult for me to assess whether the contribution warrants authorship. I'd rather leave this decision to the lead authors.

It is been a week: I am closing this issue. We will submit the article to bioarxiv today

Hi all,
To give you some updates, the paper has been accepted by Cell Systems :tada:
Hopefully it will be online soon. In the mean time, it is still on BioRXiv: https://www.biorxiv.org/content/early/2018/04/05/225680

Congratulations everyone - a nice reward for great efforts!

Hi all,
The paper is online since this morning: https://www.cell.com/cell-systems/fulltext/S2405-4712(18)30230-8
Thanks to all of you for this effort!

Hooray! Thanks @bebatut and all!
No open access though?

No open access though?

It should, that is a mistake, the editor is contacted.

the DOI does not seem to work yet: https://doi.org/10.1016/j.cels.2018.05.012

@martenson try this one: https://doi.org/10.1016/j.cell.2018.05.012

This is a shareable link with open-access: https://authors.elsevier.com/a/1XI288YyDfU-ra

@bgruening yours works, but the broken one I shared is listed on the cell page

Actually the one @bgruening shared redirects to another paper.

@martenson The original DOI works now.


Fix guard-else typo

This PR contains just a typo fix in GenericsManifesto.md file.

Thanks!
@swift-ci Please smoke test and merge

*sigh* We'll have to wait until #20036 lands.

@jrose-apple should we call the ci again?


tag the example images

i noticed that the guestbook service image, and the frontend image are both using the "latest" tags.
kubernetes will automatically mark the deployment to always pull images when using the latest tag. this could slowdown future deployment.
but most importantly, the "always" policy will be applied when using kubectl edit deployment ... unless changed otherwise.

Besides https://github.com/redhat-developer-demos/kubernetes-lab/blob/master/kubernetes/mysql-rc.yaml, all of them are now tagged.


Mongodb support

Could you have a look to this issue on crinsane.
I use this fixe for several years but it has never been merged.
https://github.com/Crinsane/LaravelShoppingcart/pull/478/files

will take a look today, thanks


Fixed - 100% usage by CommandsQueue in rare cases. #6153

There is also some problem after the original fix of issue #6153.
I think the actual reason of the deadlock is that the CommandsQueue#write are invoked twice in one stack. Obviously, it cannot obtain the lock at the second time.
I'm not sure what exactly causes the second time invocation of the CommandsQueue#write. But if there is no other problem, just turn the lock to a ReentrantLock to solve it.

deadklock.html.pdf
Here is the flame graph that the deadlock triggered again. Thread#yeild doesn't work.

Can you reproduce it?

Thanks for comment. Fixed in https://github.com/redisson/redisson/commit/01e1259eb9a6ed974c3393acdc7004767d588ffc

Can you reproduce it?

It's not easy to re produce normally. And I'm afraid that I can't provide productive environment stack or heap info.
But you can reproduce it by debug (change some variable to mock the slow or block I/O).
Configure redisson to use just one netty thread, and one connection.
Execute a command with redisson normally. As screenshot below, you will be at flush0 of netty AbstractChannel.
Make doWrite at line 931 error (the easiest way I think is to create a NullPointerException), and enter the 933 line handleWriteError (it's better to change the t variable to be a IoException).

Then you should at RedisExecuter#checkWriteFuture as picture below shows, remember to change the attempt equals to attempts, to mock all the attempts failed.

Then you should at RedisExecuter#releaseConnection, remember to change it from 1 to 10, and restore it after the check of 606 line.

Then, you should at AsyncSemaphore#tryRun as below.

Here, try to invoke redisson to send a command to add a future into the listeners, then run into line 79 in the above picture. (Which cause the second time trying to obtain of the lock).
Continue, you'll get a deadlock.

Thanks for comment. Fixed in 01e1259

It works, but I think yield is not necessary. Feel free to close the pr.


CT 2552 pin protobuf to >=4.0.0

resolves #7565
Description
Pin protobuf
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Why?


hostnames vs IP addresses in AWS

The default value is set to https://#{node['ipaddress']}:7443/arcgis if “node['fqdn'].nil? || node['cloud'] || ENV['arcgis_cloud_platform'] == 'aws'” and to https://#{node['fqdn']}:7443/arcgis otherwise.
In our case Chef probably uses https://:7443/arcgis/protaladmin URL because node['cloud']  is true.
We would like to use the hostname of the machine and not the IP address. Because this is an automated deployment, we don't know the machine name before we launch the machine so we can't specify the node['arcgis']['portal']['url'] attribute

Hey @chriswoodside,
Are we utilizing an Esri AMI for our chef deployment via AWS?

@chriswoodside
In the latest release, v3.5.0 We have added the following parameters that will allow the use of hostnames in AWS:
node['arcgis']['configure_cloud_settings'] = If set to true, makes the cookbook use cloud provider specific ArcGIS Enterprise configuration. The default value is true if node['cloud'] is defined or arcgis_cloud_platform environment varible is set to aws.


Unable to preview when cloning resources

Describe the bug

Steps to reproduce
https://github.com/seal-io/walrus/assets/26084756/4e996385-ba71-4653-9b7f-f9609fd608a2
Result

Expected behavior

Environment

Walrus version: main 7bb79389ab0436969f4c4214e38918ee6c6e9070
UI version: dev-1be55ba

Additional context
Add any other context about the problem here.

Environment

Walrus version: main aba5070fb2d7f86be3c14bd419ee7be9a8b3d302
UI version: dev-8f22705

Test Result:

Clone resources: pass

Clone environment: failed

Environment

Walrus version: main aba5070fb2d7f86be3c14bd419ee7be9a8b3d302
UI version: dev-bb39486

Test Result: pass


add option to prevents deep links from opening directly in one app when multiple duplicate apps are installed on the device

This pull request adds new functionality to the @capgo/in-app-browser plugin that prevents deep links from opening directly in one app when multiple duplicate apps are installed on the device. For instance, if both WhatsApp and WhatsApp Business are installed, the user will now be prompted to choose which app to open the link in.
This new feature helps to provide a better user experience by allowing users to easily choose which app to use for specific links, without being redirected to a default app.
I have tested this implementation and it works as expected. I believe this feature will be a valuable addition to the @capgo/in-app-browser plugin and I look forward to your feedback

Thanks a lot for the PR !
To be able to go forward can you check your merge conflict ?
Also i see you did this change on android only, can we do it in ios ?
You need to update the Typescript declaration as well with the new option, this will generate the documentation update.

Thank you for your response!
I have resolved the merge conflicts and pushed the changes to a new branch.
Regarding your comment about implementing the change for iOS, I was facing this issue only in Android. It was working as expected in ios. That is why I made changes only in Android.

Can you run npm run fmt it seems your linter made lot of change, so please run the lint to have only the necessary change pushed

Absolutely, I've just finished running npm run fmt and have pushed the updated commit. Please let me know if there's anything else I can do to ensure the success of this pull request.


Detected the region of interest.

After experimenting with different color spaces and edge detection techniques, added to function to detect the region of interest.

It's exactly the same as your previous PR, so no point in merging it.


Preview Mode

Need special review on the Wordpress changes here because I'm not sure I put the pieces together correctly (for example, we didn't have an admin-env.php here to begin with)

@ccorda this is deployed and working on bubs-next WP Engine staging server (https://bubsnexts.wpengine.com/wp-admin) now. Please give the code, especially the PHP portions some review. I realize this is a big chunk of code to be pulling in.
Once this is merged, we can enable preview mode for clients by:

making sure the URLs in admin-env are up to date
adding wordpress secret token and jwt signing secrets to wp-config of client sites (potentially can be streamlined but with no jwt signing token variable set, graphql will totally break on a site)
enabling jwt plugin
creating graphql user as editor
using CURL to get the auth token
saving auth token to Vercel

I estimate this adds about an hour or two to each site's setup. Please note that because the vercel staging urls change for every git branch, and the staging url needs to be hardcoded in php, actual preview mode for staging wordpress probably won't work as clients expect. Not sure if clients use staging enough for this to be an issue.

Merge conflicts resolved


[BEAM-7850] Makes environment ID a top level attribute of PTransform.

Removes SDKFunctionSpec and replaces all usages of it with FunctionSpec.
Maintains the current behavior as much as possible. For example, added a list of known transform for which we'll be setting the environment (will leave the default value, empty string, for others). If needed, this behavior can be changed in a future change.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

cc: @robertwb and @lukecwik
I'm adding the rest of the refactoring needed for this but can you take a quick look to see if the proto changes look good ?
I did not preserve tags since I think we do not worry about backwards compatibility at this point but lemme know if I should. Thanks.

The proto changes makes sense to me. The one bit I'm not sure of is how this will look for cross-language UDFs, but I think that will still look very different than the current SdfFunctionSpecs, and possibly will be modeled as "side" PTransforms which this would be in line with.

Thanks. Yeah, design for cross-language UDFs should be done separately and SdfFunctionSpec is inadequate for this.

R: @robertwb
I'm trying to get the last set of failing tests to work but I think this is ready to be reviewed now.

Heads up, in #10393 I updated the Go proto compiler version and rebuilt the generated code.You'll probably need to update the generated Go code before merging this (and make sure you update the protoc-gen-go version to the most recent one this time). If it says the generated code is using ProtoVersion3 instead of 2, then your change is good to go.


Create KB4088787_Fix.ps1

Description

Checklist

[ ] This pull request was tested on both of:

[ ] PowerShell 5.1 (Windows)
[ ] PowerShell 6.x
[ ] PowerShell 7.x (Latest PowerShell)

[ ] Scripts do not contain static passwords or other secret tokens.
[ ] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform and PowerShell version: [pscustomobject]$PSVersionTable | Select-Object OS, BuildVersion, PSVersion
Az version: (Get-InstalledModule -Name Az).Version

#sign-off

@mikefrobbins I am a Content PM in CSS Content, and working with an escalation engineer to publish a PowerShell script. It has been tested and vetted by Support and our writer JC Mackin has created this PR for it. Can you let us know if you can merge this PR or if there is anything else you need from us? Thanks!

@JarrettRenshaw Is this script Azure PowerShell related? It does not appear to use any Azure PowerShell related cmdlets. This repo is specifically for Azure PowerShell sample scripts. Looping in @dcaro

@mikefrobbins This script is only indirectly related to Azure. It fixes a problem in AD FS, which as you know is often used for SSO authentication in connecting Azure and on-prem deployments.  However, it doesn't use any Azure PowerShell cmdlets. If this isn't the correct repo, would you be able to recommend another that is more suitable?

The purpose of this repo is to give scenario oriented samples how to manage Azure resources with PowerShell.
Since this script aims at fixing an issue with ADFS, I think that this script should be on a different repo or directly in the KB article.
@jc-mackin unfortunately I do not have any pointers.

There's a process for onboarding examples. The only other option I'm aware of that hasn't already been mentioned would be to publish it to the PowerShell Gallery where users could install it using the Install-Script cmdlet.


Make Local Builds emulate Production Builds

Closes #1805
Add flags for warnings as errors.
Add flag for nit picky

@blowdart @danroth27

Hi @spboyer, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:shipit:


Missing --index in parse_requirements

Hi guys, I think here:
https://github.com/kennethreitz/pipenv/blob/master/pipenv/cli.py#L236
the --index should be tested as well, not only -i. I'm not sure though. I can make a PR if I'm correct.
Regards!

it's normally used as -i, but --index would be nice too. send a pr!


Delete startup args txt files

Deletes ns_startup_args.txt and ns_startup_args_dedi.txt.
As per https://github.com/R2Northstar/NorthstarLauncher/pull/99 we no longer need them. NorthstarLauncher should just auto-create them when missing.

Relevant discussion on Discord:
https://discord.com/channels/920776187884732556/965323514884263986/969358300565090366
(channel not publicly accessible)

@pg9182 would it affect docker image if we remove ns_startup_args.txt and ns_startup_args_dedi.txt?


recevied transaction id from geth node server but not able to see that transaction id at etherscan.io

Did anyone face the same problem?

Hi, @ashishonmobile .
The fact that you received a transaction id does not mean that it has passed. First, check the status of the local node (eth.pendingTransactions). Most often the problem is caused by an incorrect nonce or low gas price. Also, make sure that your local node is synchronized.

@BaldyAsh Thanks for the update. I got the issue and the problem was gas price was very very low and calculation of gasPrice was not correct.


Clean up dependencies

This contains two commits:

Removes unused dependencies.
Sorts dependencies in Cargo.tom files.

This is meant to be committed without squashing, see the commit messages for details.
If the 2nd commit is too controversial I'm happy to remove it.

@b5 great, added that


Add support for generic interface methods

We need to allow defining generic methods in an interface signature.
//TEST(compute):COMPARE_COMPUTE:-xslang -use-ir
//TEST_INPUT:ubuffer(data=[0 0 0 0], stride=4):dxbinding(0),glbinding(0),out

RWStructuredBuffer<float> outputBuffer;

interface IVertexInterpolant
{
    float2 getUV();
}

interface IBRDFPattern
{
    __generic<TVertexInterpolant:IVertexInterpolant>
    float evalPattern(TVertexInterpolant interpolants);
}

struct StandardVertexInterpolant : IVertexInterpolant
{
    float2 getUV() { return float2(0.5); }
};

struct MaterialPattern1 : IBRDFPattern
{
    float base;
    __generic<TVertexInterpolant:IVertexInterpolant>
    float evalPattern(TVertexInterpolant interpolants)
    {
        float rs = base + interpolants.getUV().x;
        return rs;
    }
};

__generic<TPattern : IBRDFPattern, TInterpolant: IBRDFPattern>
float test(TPattern pattern, TInterpolant vertInterps)
{
    float rs = pattern.evalPattern<TInterpolant>(vertInterps);
    return rs;
}

[numthreads(4, 1, 1)]
void computeMain(uint3 dispatchThreadID : SV_DispatchThreadID)
{
    StandardVertexInterpolant vertInterp;
    MaterialPattern1 mp1;
    mp1.base = 0.5;
    float outVal = test<MaterialPattern1, StandardVertexInterpolant>(mp1, vertInterp);
    outputBuffer[dispatchThreadID.x] = outVal;
}

Currently, the compiler complains that MaterialPattern1 does not implement evalPattern.

Just to go through the motions and make sure we actually need this feature:

You don't necessarily have to support generic methods in interfaces for this example to work. You could instead have the IBRDFPattern interface either be generic in the vertex-attribute type, or use an associated type to export it

Thinking about the actual scenario you are modeling, it seems weird to have the IVertexInterpolant interface that only exposes a single texture coordinate, since pattern generation in a real engine would probably want to have access to whatever attributes are defined on the mesh that the material is being applied to, right? So if we tried to solve that more real problem, it seems like we almost want a material pattern to have an associated type that is an conjunction/intersection of interfaces, and then constrain the actual vertex type to supporting that composite interface...

If you decide to enable this, I suspect the front-end work won't be too bad (mostly just adding another case for interface requirement satisfaction checks). Note that there is already a bit of machinery in check.cpp related to checking if two functions have the same signature (for detecting redeclarations and target-specific overloads), and that logic includes support for generic methods. Ideally we should find a way for the code that checks for interface requirement satisfaction to use that same logic.
The one big gotcha I can see is that we probably need to make the requirement-satisfaction checks do a pass that checks for associated types first, and then checks other kinds of requirements next, so that those checks can be done using substitutions that will be able to map the associated types in the requirement to their concrete realizations.

If we support a generic interface then we don't need generic methods. I think it would be easier to add generic method than to add generic interface.
Yes I have already modified conformance satisifaction check for generic decl members. Now I am running into an issue where ResolveOverload() fails to find an appropriate candidate for the evalPattern() call.

I agree that generic interface methods should be easier than generic interfaces, at least for the back-end. My expectation is that we can construct a witness table that just references the generic IR function, and then the code sequences will involve a lookup_interface_method that gets fed into a specialize, and the specialization of the lookup step will enable specialization of the method.
Generic interfaces might not actually be that bad either; hopefully it just means having a witness table where the "keys" are specialized decl-refs to the interface requirements.
The hard part as I understand it will be generic types that conform to interfaces, because at that point we need generic witness tables, and we'll have to support specialization of witness tables...

Support is added in PR #251


Student view - seeing all their stuff

Right now a student in Classy sees all their repos. A nice-to-have is the ability to link to the course README (which would need to be specified in the deliverable config).
An even more nice to have, when thinking about #51, is students seeing all repos for all courses. If we have the notion of a sub-course or something like that, we'd just need to format the page nicely so that all repos pertaining to a "sub-course" are grouped together.

We'd also want the grades grouped by sub-course.

You could imagine showing them everyone on one giant page (as we do now) or having a dropdown where they select a sub-course (i.e. MDS course) and see the repos/grades for that course only.


Sample VBA code for optional parameters with default values does not seem to work on Word 2016

The sample code for optional parameters with default value does not appear to work in Word 2016. The sub does not show in list of macros when the default value is given in the sub declaration. Removing the default value makes sub appear on macros list.
Sub notify(ByVal company As String, Optional ByVal office As String = "QJZ")
    If office = "QJZ" Then
        Debug.WriteLine("office not supplied -- using Headquarters")
        office = "Headquarters"
    End If
    ' Insert code to notify headquarters or specified office.
End Sub

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

ID: 03d7e221-57bc-3524-1f74-490d5bbfaf26
Version Independent ID: bac92a76-a343-ac1f-2e90-01e19ae7963a
Content: Optional Parameters - Visual Basic
Content Source: docs/visual-basic/programming-guide/language-features/procedures/optional-parameters.md
Product: dotnet-visualbasic
GitHub Login: @KathleenDollard
Microsoft Alias: kdollard

/cc @KathleenDollard

@kaanchan VBA isn't the same as VB.NET. You should read the article related to VBA that I found: https://docs.microsoft.com/en-us/office/vba/Language/Concepts/Getting-Started/understanding-named-arguments-and-optional-arguments
However, the end of the article says to use this link for support: https://docs.microsoft.com/en-us/office/vba/articles/feedback-support


Save some bytes by declaring variable as extra function argument

Also modified a test to check for a case that another proposed byte reduction change didn't handle.
-3 bytes

@JorgeBucaran I updated the test for h covering numbers so that all the use cases for the logic I tried to simplify are now covered. I wasn't able to cut the size so I reverted that change, but I left the other for a smaller 3 byte diet.

Thanks! I feel we are doing that trick of passing fake arguments to functions too much. I'm going to merge this, but I may revert it later, along with the others to increase readability if it only means some bytes.

@okwolf Can you change the commit message (and PR title) to something less mischievous? 🙏

@JorgeBucaran oh, I was actually mostly joking with this PR to begin with. I changed the title, but I'd rather just make a new one for different commit messages/branch name.
I'm not super attached to the only actual byte shrinking change left, but I do want to get the gap in test coverage for h patched.

@infinnie I tried both approaches and neither yielded fewer bytes actually haha.

@JorgeBucaran I tried both approaches and neither yielded fewer bytes actually haha.

Gzip can be funny sometimes about when it likes duplication and when it doesn't.

Merging. Thanks @okwolf! 🎉

you guys are doing it all wrong. check out the gzthermal tool used here [1].
::runs away and hides::
[1] https://blog.usejournal.com/of-svg-minification-and-gzip-21cd26a5d007

@leeoniya If you are going to come tell people they are wrong but don't have the time to tell them exactly where or how then please don't. Unless you are Kaepora Gaebora.

i guess sarcasm doesnt come across well over the interwebs. i will improve my attempts at humor in the future.
anyways, i figured you guys might be interested in gzthermal since you're so OCD about byte size. but it can also lead to some very time-consuming code-golfing.

Or an IIFE…

@leeoniya I like your humor, but you always seem to be in a rush when posting around here, aren't you? I think you are extremely smart, but when sharing wisdom, there's always so much more than what you reveal, which is inconvenient for those less enlighted (me). 😉

i will also point out, that if you guys compress with closure compiler, you don;t need to do this all these optimizations by hand. here's the same pretty-printed hyperapp function compiled via Closure Compiler [1], it already does this:
  function y(a, d, c) {
    if ("string" === typeof a) {
      c = document.createTextNode(a);
    } else {
      c = (d = d || "svg" === a.name) ? document.createElementNS("http://www.w3.org/2000/svg", a.name) : document.createElement(a.name);
      a.props.oncreate && x.push(function() {
        a.props.oncreate(c);
      });
      for (var b = 0; b < a.children.length; b++) {
        c.appendChild(y(a.children[b], d));
      }
      for (var h in a.props) {
        E(c, h, a.props[h]);
      }
    }
    return c;
  }

[1] https://closure-compiler.appspot.com/code/jsca1d643ec6d5e87dac7ed5b01a23f25c/default.js

@infinnie That would be cool, but actually when you bundle Hyperapp with your app, it is fewer bytes because you are not using the UMD bundle. 😆

@leeoniya I've heard about the wonders of google closure compiler before, but last time I checked, it seemed too hard to setup. Also last time I looked at domvm I tried to see what you were doing, and you had a build script, which I want to avoid.
Having said that, it would be much nicer if we didn't have to write obfuscated code, so I'll have another look. 🤔

Maybe you could try a gzip directly after comment and extra space removal?

domvm's build script does a lot more stuff. and it also used to use the Java version of Closure Compiler.
rollup has a ready-made plugin for the the js version of Closure: https://www.npmjs.com/package/rollup-plugin-closure-compiler-js

@leeoniya I heard that the JS version of the compiler is not as good as the Java one. Do we get the same benefits you said above using the JS version?

in my testing, the js version only differed by encoding html entities within strings [1]. it was easy to work around in my build script via a simple regex replace (but i also didn't need to produce source maps for the minified builds since i only generate source maps for the unminified bundles).
[1] https://github.com/google/closure-compiler-js/issues/79

@leeoniya How did you produce the code you posted here? I tried https://closure-compiler.appspot.com, but it gave all sort of warnings.

you can do hyperapp's npm run bundle and paste the non-minified output into there. (leave the commented header)


Typo

Publish local repository to GitHub
You can also initialize "and" local repository and publish it directly to GitHub.
That should be "a" instead of "and"

Fixed in https://github.com/microsoft/vscode-docs/pull/7687

@ntrogh please close this issue now that you have merged the PR. Thanks!


[composer] "create new thing" dialog with fuzzy autocomplete instead of action menu

See current Table create workflow.


Option to remove ineffective graceful shutdown code

What this PR does / why we need it: This PR removes the preStop lifecycle hook intended for graceful shutdown, because is actually killing both the agent and the job immediately. The default behaviour of the terminationGracePeriodSeconds variable is already taking care of gracefully shutting down the pods.
The code for tini only forwards the signals to the process which it specifically starts, not all children processes.
Note that this is a successor to #87 with the feedback from @rimusz. @DavidVtwilio is on PTO so thought I'd quickly hack this together. I'm hoping this is about what you were asking for? The behavior, by default, remains the same. I've also through in relevant hooks for just straight up overriding the preStop and postStart lifecycle.
Please also note that I do not have full confidence in my Helm abilities -- especially my Helm stylistic ability.

-q in pidof is not a valid option. I changed my preStop command to this and it's working successfully to get the lowest PID:
kill $(pidof buildkite-agent | tr -s ' ' '\n' | sort -n | sed '1q'); while pidof buildkite-agent > /dev/null; do sleep 1; done

If using dind, that container will be stopped before the agent container is stopped. To fix this I use the BUILDKITE_AGENT_HEALTH_CHECK_ADDR variable and had a preStop on the dind container to stop after the agent container stopped:
while wget --quiet --output-document /dev/null http://localhost:19303; do sleep 1; done

@calvinbui would you mind to do a PR with that fix?

@rimusz Forgive me, I'm very new to helm, but did I break the contract for "should remain the same" by not specifying a | default? I'm trying to test this as a subchart dependency right now and I think I must be doing something wrong on my end, but figured I'd reach out in case I did break it. Specifically, I'm expecting a no-op change of just updating to have no changes, but I am in fact seeing changes -- the lifecycle is disappearing 😬

Oh no, I believe I have forgotten to bump the version of this chart in this pull request. Because of that, I believe we've overridden the version 0.5.2 of this chart with the version I believe should be 0.5.3 I think 0.5.2 may need to be recreated and this version needs to be bumped


test multivector with SymEngine::Expression value type

Change-Id: Iac7a864f864e2ae4cb170fcdc4ca0c7933ca6aff

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 62.38%. Comparing base (b9ecd6b) to head (b7ad75c).

Additional details and impacted files
@@                            Coverage Diff                             @@
##           I82ed3651ff9417ae7dcbb368db96a5619b2ceb35      #40   +/-   ##
==========================================================================
  Coverage                                      62.38%   62.38%
==========================================================================
  Files                                             18       18
  Lines                                            210      210
  Branches                                           4        4
==========================================================================
  Hits                                             131      131
  Misses                                            79       79

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Footer Sizing Issue with Footer Items

Is your feature request related to a problem? Please describe.
I've found the library and it is awesome, but when I added footerItems, the footerSeparator was in a bad position, in the middle of the sidebar, instead of the bottom, as expected.
Describe the solution you'd like
I've created a fork and added a enum "FooterFitType" that can be "fit" or "expand". And send it as a parameter to the SidebarX class, setting the default value as "expanded" so the new update won't break existing projects.
Describe alternatives you've considered
I've considered setting as default to always fit the items, instead of expanding, but maybe someone needs the expanded approach. Later, I've considered using a boolean to tell if it fits or not, but found more readable being a enum.
Additional context
Current approach:
Code:

Sidebar:

My solution:
Code:

Sidebar:

hello @mrhaubrich have you code changes merged?

hello @mrhaubrich have your code changes merged?

yes, it is in MR #62


publish<EMAIL_ADDRESS>

Requested by: @Lms24
Merge target: lms-fixup-7.10.0-release-registry
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave the #retract comment under this issue to retract the release (original issuer only).
Targets

[ ] aws-lambda-layer
[ ] gcs
[ ] github
[ ] npm
[ ] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

For context/future Readers: This run should only update release registry after we had some problems with our original 7.10.0 run that released packages to NPM but failed while updating release registry.

#retract


TypeError in Q12 Integer to Roman

TypeError: 'float' object cannot be interpreted as an integer
https://github.com/qiyuangong/leetcode/blob/056459464f15df885538aa90f6db882a2f4dbd20/python/012_Integer_to_Roman.py#L38

To correct this error, try for j in range(int(k)):.

Thank you @lowspace !
That's my fault. My previous solution is for Python2, in Python3 it will raise TypeError.
Have change to Python3 :)
Issue closed


Adds graphql yoga sample app

Are there any models or things to actually query in this sample?

@ewucc there's a hello query that returns "Hello from Klotho!". No complex models. It's basically a "Hello, world!" app.


Mouse doesn't work when in the middle of the screen

Steps to reproduce:
1- Put the mouse in a region in the center of the screen.
2- Try to do anything with the mouse.
Expected outcome:
The mouse does the action I have requested.
Noted outcome:
The mouse does nothing. Clicking, dragging, scrolling, and even auto focus when switching workplaces don't work.
Another thing to note is that on workspaces with the kitty terminal, the mouse is doubled in size outside of the central area, and only has a normal size within it.
Here's a demonstration:
https://user-images.githubusercontent.com/91404038/180819399-89c9cb8f-9e27-4ecd-a180-447001b4aa2e.mp4

Nvm, it was eww on my end

Although the doubling of cursor size part still remains, it's just that now it's on the whole screen

my man's forgot to export XCURSOR_SIZE=24 omegalul

my man's forgot to export XCURSOR_SIZE=24 omegalul

It works, but that's never been needed in any other WM or DE I've ever used.

Thanks for the tip tho

cuz they usually export it themselves


arduino-cli: fix fhsenv version

part of #336576
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/2134

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 359959

x86_64-linux

:white_check_mark: 2 packages built:

arduino-ci
arduino-cli


Disable XCM filters

As of Polkadot 1.x.x (will need to check exactly) we no longer need to have XCM filters present, let's disable to allow custom integrations.
https://github.com/galacticcouncil/hydration-node/blob/aa61ed23a1b7f26b238ea7fd6c6303e369efb875/runtime/adapters/src/xcm_execute_filter.rs#L68-L83

Fixed by #885 #909


Broken cloudflare workers sourcemaps

What version of Remix are you using?
1.3.2
Steps to Reproduce

Build a project targeting Cloudflare workers with remix build --sourcemap
Run npx source-map-explorer build/index.js

Expected Behavior
Sourcemap is correctly parsed
Actual Behavior
Output:
build/index.js
  Your source map refers to generated column 1 on line 20, but the source only contains 0 column(s) on that line.
  Check that you are using the correct source map.

doesn't appear that remix build --sourcemap generates server source maps, but just client ones

doesn't appear that remix build --sourcemap generates server source maps, but just client ones

It generates inline sourcemaps: https://github.com/remix-run/remix/blob/main/packages/remix-dev/compiler.ts#L452
If you look at the last line in your generated worker javascript file you'll see it starts with //# sourceMappingURL=data:application/json;

My PR #2065 fixes this to use "linked" sourcemaps. It also rewrites route module references so breakpoint debugging works without any hacks.

@tiagoad can you try again and tell us if it's fixed for you now?
There's also #3768 that have been reported after #2065 got merged, in case of problems, can you tell us if it affects you as well?

Closing this as no response was provided from the original issue author. If you still feel like something is wrong on our side, please open a new issue using the latest remix version.

I am seeing the same error message with Architect (AWS Lambda) projects.


Respawn Animation

Have it be a reverse of the death/explosion animation.
I don't really know how this would be done exactly, you could try moving the particles to their fade area and changing their emission direction and just doing that, or maybe we could just create a completely unique one that looks "similar"-enough to convey the idea that they are exploding into stars and then respawning in from the same stars...

#27


Proper vector-format-based text outlining

Godot version:
3.0 stable
OS/device including version:
Any
Issue description:
Godot uses its own imperfect way of outlining texts after the glyphs are already rendered by FreeType from the vector format. This results in visible artifacts and outlining looking usually bad. FreeType does support outlining based on the vector format which provides perfect result (see Stroker API). Godot should use that API instead of relying on its own algorithm for fonts in vector format.

I guess it's because it originally came from bitmap fonts.
it should be handled separated way between bitmap & dynamic font then.
I'm not against to implement it.

I just assigned myself to this issue, but can't say when I can do about it.
I'd appreciate anyone leaves comment if want to do it.

Would be great to have such a feature. Outlining is important, sometimes text wouldn't be readable otherwise.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
de9d7f6 -> 0685378

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.38.2 -> v1.38.3

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.38.3
Compare Source
Bug Fixes

deps: update gomod (#864) (802df12)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

2 additional dependencies were updated
The go directive was updated for compatibility reasons

Details:

Package
Change

go
1.22 -> 1.22.0

github.com/IBM-Cloud/power-go-client
v1.7.1 -> v1.8.1

github.com/IBM/platform-services-go-sdk
v0.69.0 -> v0.69.1

/run pipeline

/run pipeline

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

GitHub release
v1.7.21

Your semantic-release bot :package::rocket:


Add 'Bookmark Saved' Toast

Reviewer:
Asana:
CC:
Description:
Steps to test this PR:

Save or edit any bookmark.

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR DRI Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications (Database connections, Grafana stats, CPU)

Fix for issue #34

Perfect! Thanks for contributing


#71 added basic support for oci driver in OracleURLParser

Pull Request Test Coverage Report for Build 248

5 of 6 (83.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 47.951%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/io/opentracing/contrib/jdbc/parser/OracleURLParser.java
5
6
83.33%

Totals

Change from base Build 247:
0.06%

Covered Lines:
468

Relevant Lines:
976

💛  - Coveralls


Return an error if the client is forbidden from starting a watch

This makes watchLatest wait until RetryWatcher starts the first watch attempt. It'll return a forbidden error if the client does not have watch permissions on the resource.
If the RetryWatcher stops due to an unrecoverable error and the new watch is forbidden, the watch is cleaned up and a reconcile for each watcher is called.

FYI, this is needed for expanding hub templates access outside of the namespace since ACM doesn't control the permissions of the service account.

/hold


invoices are serving a 'unexpected error'

It's failing in TransactionDetails.js because transaction.host is null.

It’s also failing for other users like Facebook. @xdamman can this be related to the fix you did yesterday?

Yes it's related to this: https://github.com/opencollective/opencollective/issues/1154
I'm pushing a fix to be able to see the list of transactions. The invoice PDF is still being generated by the old website repo. So will need a separate fix for that.


Passing in binding for index disables animations.

Example:
    ...Other code

    @State var currIndex: Int = 0

    var body: some View {
        ACarousel(items, id: \.id, index: $currIndex, ...) { ... }
    }

In this case, the animations will not occur properly. Demo:
https://drive.google.com/file/d/1asAYFM6B3ZREpgqhD4Kr3behYqe2bsy9/view?usp=sharing

any solutions on this?

Yes, the problem still persists and this turns the package unusable for many situations, which is unfortunate given its quality... I hope that this can get fixed!

@josephmannis @miguel-arrf
It's related to the id supplied into ACarousel. Try using just strings your items and make id: \.self and it should work
e.g
ACarousel(["1","2","3","4"], id: \.self, index: $currIndex,...){ ... }

e.g with struct
struct Item: Identifiable, Hashable {
    let id = UUID()
    let name: String
}

ACarousel([Item(name:"t"), Item(name:"e"), Item(name:"s"), Item(name:"t")], id: \.name, index: $currIndex,...){ ... }

For some reason, I haven't figured out yet using id: \.self doesn't work even though the struct conforms to Identifiable

@josephmannis @miguel-arrf It's related to the id supplied into ACarousel. Try using just strings your items and make id: \.self and it should work
e.g
ACarousel(["1","2","3","4"], id: \.self, index: $currIndex,...){ ... }

e.g with struct
struct Item: Identifiable, Hashable {
    let id = UUID()
    let name: String
}

ACarousel([Item(name:"t"), Item(name:"e"), Item(name:"s"), Item(name:"t")], id: \.name, index: $currIndex,...){ ... }

For some reason, I haven't figured out yet using id: \.self doesn't work even though the struct conforms to Identifiable

Using your code I get
ForEach<Array<Item>, Int, ModifiedContent<ModifiedContent<ModifiedContent<ZStack<TupleView<(ModifiedContent<_ShapeView<RoundedRectangle, Color>, _AppearanceActionModifier>, ModifiedContent<Text, _PaddingLayout>)>>, _FrameLayout>, _FrameLayout>, _ScaleEffect>>: the ID 7 occurs multiple times within the collection, this will give undefined results!
in the console, is this ok? I guess that's how wrap works in this library?


GEODE-9742:fix resource leak due to Files.list and Files.walk

see https://issues.apache.org/jira/browse/GEODE-9742

You'll need to remove unused imports to get the build jobs to pass the spotless check.
You can use spotlessApply to automatically reformat code based on our spotless config:
./gradlew spotlessApply
You can use spotlessCheck to run the same format validation that failed in the build jobs for you PR:
./gradlew spotlessCheck
Note that spotlessApply is unable to remove unused imports, so you'll have to do that step manually.


[DOC] Clarify Interaction of JVM Memory Settings via OPENSEARCH_JAVA_OPTS with Default Values

Description
Add a note to clarify how JVM memory allocation settings specified via the OPENSEARCH_JAVA_OPTS environment variable interact with JVM default values in OpenSearch.
Page Updated: 🔗 Install and Configure » Install OpenSearch » Index » Important Settings
Issues Resolved
Closes https://github.com/opensearch-project/documentation-website/issues/7563
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.

Thanks, @c-neto, for this contribution! I am looking for someone to do a tech review sign-off.


Fix doc code examples

So if you don't have the space, Sphinx decided to simply not show the sample, without displaying a warning ...


Sync feature toggle using Hazelcast lists.

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I added multiple integration tests (Spring) related to the features (with a high test coverage)
[x] Server: I documented the Java code using JavaDoc style.

Motivation and Context
Artemis has feature toggles that allow administrators to disable certain features of Artemis without having to restart the application server (currently, only programming exercises can be disabled).
The feature toggle state (whether a certain feature is enabled or disabled) was currently only stored in an Enum on the instance where the administrator was connected to when performing the change but was not distributed between all instances.
This leads to inconsistent states where a feature can be enabled on one instance but disabled on another.
This PR synchronizes the feature state between all instances.
Description
Instead of the Enum that holds the state (which cannot easily be shared), two lists are shared between all instances:

One list containing the enabled features
One list containing the disabled features

When an administrator disables a feature, the feature is removed from the enabled list and added to the disabled list (and vice versa).
Note: In theory, one list containing disabled (or enabled) features would be sufficient as a feature is either in this list or not.
However, this causes an issue as the list will always be empty on the startup of the system: We can not detect whether the list is empty because the system just started or if it is empty because no features are disabled.
We thus use two lists as we can detect if the system just started (both lists are empty) or was already running (at least one list is non-empty).
Steps for Testing
This PR can only really be tested if you are running at least two instances of Artemis (thus you should use TS2 for testing).
You can however also test on TS1/3 and just make sure nothing broke.

Log in to Artemis
Navigate to Server Administration >  Feature Toggles
Toggle the state of a feature: The change should immediately be effective to all users, even if they are connected to a different instance.

This can be tested on TS2 with multiple testers connected to different instances, just DM me (or anyone else with access to TS2) on Slack, so that we can verify that the testers are indeed connected to different instances.
Test Coverage
Server:

FeatureToggleService: 100%

I would suggest using a Map<Feature, Boolean> instead of managing two lists and keeping them consistent. Would that also be an option?
I think this could simplify the code and remove possible synchronization/concurrency problems or inconsitencies.

@MaisiKoleni Good idea, I changed it.


Embedding scripts/styles into html

Very useful feature for single-page apps is an option, which force insert the content of all external scripts and styles directly into html.

Duplicate of #104


Fix LicenseObject type

Not going into all details of #77, but small steps make big journeys ;)
IMHO having LicenseObject correctly validated worth a fix.
Thanks!

👍

Available as 5.8.0 on pypi.

Available as 5.8.0 on pypi.

Yay, thanks :)


SSO for all authentication requests

Hi @flytreeleft. Thank you for your plugin.
I have been enjoying your plugin for several months and working well.
But I found one issue - when I am admin and I try to change password of any user, it requires authentication again but not SSO, local authentication dialog popping up.

Is this a known issue?

@cuttingedge1109 It's normal, the Nexus3 will re-authenticate the admin user by confirming its password, so you should make sure the admin user is a local user instead of a Keycloak user.

@cuttingedge1109 It's normal, the Nexus3 will re-authenticate the admin user by confirming its password, so you should make sure the admin user is a local user instead of a Keycloak user.

Well, but other users are same.
Even though I logged in using keycloak user, it requires such a dialog to change the password.

@cuttingedge1109 When does it happen for the normal user? After login or try to change other's password?

The exact case is;
I logged in using keycloak user. This keycloak user is the admin.
Besides, there are several local users who have only read permissions on specific repositories.
I am a keycloak user but admin so I need to change the password of any local users.
When I try to change the local user's password, it requires to input new password and confirm password.
After that, it requires password for this admin user, keycloak user.

@cuttingedge1109 Just login with a Nexus local admin user, then change the local users' password. Do not try to change the Nexus user's password via the Keycloak admin user.

I think it is fairly reasonable that there is no need to use local users once we use SSO. For our case, we remove the all permissions from the default admin local user and only use keycloak users.
Well, is there any milestone for this functionality? I am willing to contribute.

@cuttingedge1109 :+1: Actually, I have no idea how to prevent to popup the confirm dialog, if you find the solution, just create a PR for that, it will be very useful, and thanks for your contribution too. :-)

Close the inactive issue, if there still are more news, just please comment here. Thanks :)