project.scripts not syncing on Windows

Steps to Reproduce
# create a random project on Windows
rye init asydhj
cd .\asydhj\
rye sync

# the default script should work:
# [project.scripts]
# hello = "asydhj:hello"
rye run hello
# Hello from asydhj!

# now change the script to:
# asd = "asydhj:hello"

rye sync

# the new script does not work
rye run asd
# error: invalid or unknown script 'asd'

Expected Result
The new script asd should run as expected.
Actual Result
PS C:\Files\asydhj> rye run asd
error: invalid or unknown script 'asd'

Version Info
> rye --version
rye 0.25.0
commit: 0.25.0 (d8e00cea1 2024-02-19)
platform: windows (x86_64)
self-python<EMAIL_ADDRESS>symlink support: true
uv enabled: true

Stacktrace
No response

Disabling uv works fine, so this should be an uv specific issue. It appears that changes in pyproject.toml were not recognized.

Seems related https://github.com/astral-sh/uv/issues/1651

Same here with macOS. Uv fails to sync the changes in 'project.scripts' while disabling uv would work.

Hi, I have the issue that the package is not installed to venv on linux. Do you have a workaround for that?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_26yywk32hu/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_26yywk32hu/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_26yywk32hu/clone/recipe/meta.yaml:31: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_26yywk32hu/clone/recipe/meta.yaml:39: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_26yywk32hu/clone/recipe/meta.yaml:39: missing_description: The recipe is missing a description
ERROR: /tmp/abs_26yywk32hu/clone/recipe/meta.yaml:39: missing_dev_url: The recipe is missing a dev_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_58nfzgu03l/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_58nfzgu03l/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_58nfzgu03l/clone/recipe/meta.yaml:31: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_58nfzgu03l/clone/recipe/meta.yaml:39: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_58nfzgu03l/clone/recipe/meta.yaml:39: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_58nfzgu03l/clone/recipe/meta.yaml:39: missing_description: The recipe is missing a description
Errors were found


Add Tika parsed Metadata to json object

Forgive me, my java skills are really rusty.
This is basically #26 I think? Though I'm not sure if this is the right json object structure to have with S3 metadata separate from Tika extracted metadata.

Hi @albertsun,
You're right : your contribution is the main part of #26. One point is that we should probably also the update the mapping definition through S3RiverUtil.buildS3FileMapping() by adding a metadata property within the file object.
Thank you !

For now, I'm busy trying to migrate to ES 1.4 but will be back for merge ASAP ;-)

While we're looking at this part of the code. What's the reasoning for having title, file._name and file.title all being the same?

Hi @albertsun,
I finally had time to check this out and integrated your contribution to close #23. I dit not directly merge the pull request because of some other refactoring making copy/paste a lor faster.
Thank you !


[Event Request] Codeunit 1032 "Job Planning Line-Reserve" - OnVerifyChangeOnBeforeHasError

Can you please provide a publisher in function VerifyChange (Similiar to VerifyChange in 99000835 "Item Jnl. Line-Reserve")
procedure VerifyChange(var NewJobPlanningLine: Record "Job Planning Line"; var OldJobPlanningLine: Record "Job Planning Line")
...
    if NewJobPlanningLine.Type <> OldJobPlanningLine.Type then begin
        if ShowError then
            NewJobPlanningLine.FieldError(Type, Text004);
        HasError := true;
    end;

    VerifyBinInJobPlanningLine(NewJobPlanningLine, OldJobPlanningLine, HasError);
    // -NewEvent
      OnVerifyChangeOnBeforeHasError(NewJobPlanningLine, OldJobPlanningLine, HasError, ShowError);
    // +NewEvent

    if HasError then

...
//Signature
// -NewEvent
[IntegrationEvent(false, false)]
local procedure OnVerifyChangeOnBeforeHasError(NewJobPlanningLine: Record "Job Planning Line"; OldJobPlanningLine: Record "Job Planning Line"; var HasError: Boolean; var ShowError: Boolean)
begin
end;
// +NewEvent

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


remove travis_wait

The build has been failing since #172 was merged due to the addition of travis_wait. This is because the build takes longer than 20 minutes, and travis_wait only extends the build by up to 20 minutes by default.
While a parameter can be passed to travis_wait to make the period longer, this shouldn't be required after adding --verbosity=normal to the dotnet test command, since each test result will be printed as it happens, and no individual test takes anywhere near 10 minutes.

Also, it's better not to use travis_wait if possible, to preserve the real time log output.


Not able to install package from PyPI

Getting error like this with Python3.5
ERROR: Could not find a version that satisfies the requirement deepface (from versions: none)
ERROR: No matching distribution found for deepface
Also getting error like and it stopped executing
/tmp/tmpgzzshuxx.py changed

Deepface requires python 3.6 min. You have to upgrade your python environment.


Support lower compileSdk version

Hi,
I came across this library, and I'm really impressed with its design. However, I'm facing an issue with importing it into my project due to its dependency on 'androidx.emoji2:emoji2:1.4.0,' which requires an update to compile version 34. This dependency seems unrelated to the primary purpose of this library, which feels a bit unusual.
Additionally, upgrading my current project to compileSdk version 34 is quite time-consuming.
Therefore, I would like to inquire if it's possible for the library to provide documentation support for lower versions, like compileSdk version 33. I'm wondering if this request is feasible.
Thank you.

@Ten-Wang
Thank you for your interest in this library.
You can use Zoomable v1.4.3 if you want to use compileSdk version 33.
Zoomable depends on compose.foundation. And compose.foundation depends on emoji2. Therefore Compose v1.5 require compileSdk 34.
If you want to use compileSdk version 33, you should use Compose v1.4.x. It's same for Zoomable.


Workbranch

@FraGard - a reminder to take a look at the comments above.

Hi @willu47, back on it now. Based on the points also discussed by @AgnesBelt , wonder whether one could separate each post in two: 1) the general part (that will include info from our review of best practices); 2) an example from CCG: i.e. an example of workflow to be adopted for the particulat knowledge output in CCG, for it to comply with the best practices?


[load_stac]: error if /search endpoint unavailable?

I'm trying to implement load_stac on the python side.
Should the linked catalog/collection/item have a /search endpoint, if we also provide spatial extent etc as parameters?
Maybe we could add an additional error if we provide a STAC item/collection that doesn’t have /search endpoint but we would like to filter with the given parameters?

Can be a search endpoint, but bbox and temporal extent could also be filtered for specific collections. GET /collections/{id}/items can support bbox and datetime parameters.
For static catalogs it is more difficult. Here the server must download all metadata files (e.g. via PySTAC) and filter itself. This probably only works good for smaller catalogs.
load_stac can be implemented iteratively. Start with the simplest (search?) and then evolve from there and throw an error regarding what is not supported.
PRs are welcome.


[WFCORE-3448] Handle the situation where the ssl enabled-protocol and…

… enabled-cipher-suites are set as system properties that contain multiple values

I put the Hold label on this because while this solution might be fine for a downstream backport branch, I'm not sure it's how I want this handled in master. This is a basically a one-off workaround for 1 attribute.

#2973 is a generally applicable fix.


sync_data: combining ignore_checksum = False and verify_checksum = True always generates checksum mismatch error.

When executing sync_data, ignore_checksum = False prohibits checksum of iRODS objects being retrieved, causing the checksum verification (when verify_checksum = True) to compare the checksum of the downloaded file to None.
Two possible solutions:

Retrieve checksum on iRODS-objects when verify_checksum = True, regardless of the value of ignore_checksum (also requires changing the __eq__ function of FileObject).
Do not accept the combination of verify_checksum = True and ignore_checksum = False.

Another solution might be to simplify the API to have fewer arguments. There are currently three arguments involving checksums, and that might be a bit much to be honest. Perhaps this is a case where we should just always take the safe route, just so that unexpected things won't happen.

Yes, good idea (will also rid us from Pylints too-many-arguments). We can just keep verify_checksum, which if set to True  implies the current ignore_checksum to be False, and by default halts on an error (rather than warn and proceed). What do you think, @chStaiger ?

I agree. The verify_checksum should be enough for the user to trigger the correct behaviour!


Jenkins 2.0 support in Hygieia Jenkins plugin

This is a working version. Wanted to open up for review and suggestions. There are scopes of refactoring (removing code duplications) but will get to that later.

@amitmawkin @satishc1 - I have tested this with 2.35 version of Jenkins 2.0 and multiple versions of Sonar (for Sonar publish). Can you test with your version of Jenkins?


build.prod task failing after adding external dependency

I Added dependencies to firebase and angularfire2 using https://github.com/mgechev/angular2-seed/wiki/Add-external-dependency
Added following to project.config.ts
(this.SYSTEM_CONFIG_DEV.paths)['firebase'] =
${this.APP_BASE}node_modules/firebase/lib/firebase-web;
(this.SYSTEM_CONFIG_DEV.paths)['angularfire2'] =
${this.APP_BASE}node_modules/angularfire2/angularfire2;
Added firebase to typings.json
And added code to work with angularfire2. dev build is running fine but when i try to build for prod  ( npm run build.prod) i get following error
[21:17:47] Starting 'build.bundles.app'...
Unhandled rejection Error on fetch for firebase.js at file:///Users/User1/Proj1/node_modules/firebase.js
Loading angularfire2/angularfire2.js
Loading dist/tmp/main.js
Error: ENOENT: no such file or directory, open '/Users/User1/Proj1/node_modules/firebase.js'
at Error (native)

How are you importing firebase in your component?
I added ng2-file-upload to a project I was working and followed the instructions in the wiki. At first during development it worked, but during production it didn't. From following the tutorial, I imported ng2-file-upload as:
import {FILE_UPLOAD_DIRECTIVES, FileUploader} from 'ng2-file-upload';

and it worked during development. With the following change, it works in development and production:
import {FILE_UPLOAD_DIRECTIVES, FileUploader} from 'ng2-file-upload/ng2-file-upload';

I think this line tells it to look for the module under (...)/node_modules/ng2-file-upload/ng2-file-upload.js, check where firebase.js is located and change accordingly.

@dominik-beste thanks for the assistance!
@pinalbhatt follow back if you have any issues.

@dominik-beste  @mgechev  this does not seem to be working.
Tried with both
import {FIREBASE_PROVIDERS, defaultFirebase, AngularFire} from "angularfire2/angularfire2";
and
import {FIREBASE_PROVIDERS, defaultFirebase, AngularFire} from "angularfire2";
Problem is with firebase.js . My code does not have any direct access to actual firebase js . AngularFire2 may be referring  it. Not sure its angularfire2 issue or angular2-seed.
Posting this issue over here as dev build is working fine. issue is only with prod build.

asking  @jeffbcross if he has any idea as he is involved with both angularfire2 and this project.

@pinalbhatt I'll investigate the issue today and follow back in a few hours.

@pinalbhatt here's how you can use angularfire2: https://github.com/mgechev/angular2-seed/wiki/Integration-with-AngularFire2.

Thanks a lot @mgechev

I have the same error in production with lodash. It works fine in dev mode. If I add
'lodash': 'node_modules/lodash/lodash.min.js',
in SYSTEM_BUILDER_CONFIG, then it seems works.

@Geoide The SYSTEM_BUILDER_CONFIG doesn't look like that any more.  I am having trouble with lodash in production, do you have an updated config you could share?
I tried: -
'lodash': {
                main: 'lodash.js',
                defaultExtension: 'js'
            }

I think you don't need any special config for the builder. Do you have 'node_modules/*': 'node_modules/*', in SYSTEM_BUILDER_CONFIG.paths?

I had /node_modules/**/package.json.  I have it working, but I'm don't fully understand why it works.  What config do you have to add lodash?  Does it depend on how I import.  I am currently using: - import * as _ from 'lodash';
Everything was working fine in Dev mode, but prod mode needed a lot more config.  I have my extra dependencies in SYSTEM_BUILDER_CONFIG,packages, it builds, but when running the server I get a problem with ng2-bootstrap (where it says System.registerDynamic is not a function).
I'm trying to understand how it all hangs together, it is working fine in Dev mode, I didn't think there would be much more effort needed for prod.  I'm trying to figure out where I've gone wrong

We only added lodash to tsconfig types array. It works without any import. We just use it like _.get....
In our project there is no special builder configuration for prod mode. The above mentioned line seems to be needed so that system js builder can fetch and use the package.json files in node_modules. Then it can automatically look up the main file of each package and resolve the module content. I think you have to add packages which don't have a main entry in package.json.
My seed config is identical to current version in master.


Implement TextGrad (bounty)

Feature request
We would like to implement https://github.com/zou-group/textgrad.
Motivation

It works out-of-the-box for a variety of tasks,
where the users only provide the objective function without tuning components or prompts of
the framework.

Bounty
A paid bounty is available. Please suggest a price range 🙏

@abrichr wanted to work on this in $750.looking forward to discuss more about it .

Sounds good @kernel-loophole ! Please DM me on Discord 🙏

@abrichr can you share your discord username

@abrichr send you request on discord

@abrichr  I will implement TextGrad into OpenAdapt, leveraging its framework for textual gradients to optimize and automate GUI workflows with Large Multimodal Models (LMMs). By integrating TextGrad's natural-language loss function and optimization through textual feedback, I aim to enhance process automation in OpenAdapt, allowing AI models to iteratively refine solutions based on GUI inputs, similar to backpropagation in neural networks. This will improve task accuracy, reduce repetitive manual input, and optimize performance across various desktop and web-based automation tasks.

Hi @kernel-loophole , thank you for your interest. Can you please be more precise? e.g. where will you implement textgrad? How will you evaluate whether the implementation has been successful?

@abrichr currently i do know the repo code architecture but i think where you pass the LLM the input ,the textgrad will be implement in that code where model can learn from backpropagation .and for evaluation we can use the same llm with textgrad and without textgrad ,then compares the performance

hello @abrichr , you can create a bounty on Bountyhub, it's integrated into github, (a bot will comment to let other people know that there is a bounty) and if no one solves your bounty you can retract it and get your money back, it's a simple process where you import issues by pasting the issue URL and the payment is upfront so that developers attempting to solve the issue know that the amount is already available.

hi there @abrichr have not hear from you on this, any suggestions or feedback where i can start .


++!@HERE’S!!* WAY TO WATCH Los Angeles Chargers vs. Pittsburgh Steelers LIVE STREAMS

Watch the Los Angeles Chargers vs Pittsburgh Steelers game on Hulu + Live TV. You can watch the NFL, including Fox, with Hulu + Live TV. The bundle ...
Watch the Los Angeles Chargers vs Pittsburgh Steelers game free with Fubo. Live TV streaming service Fubo offers the same top-tier programming you .
🔴 GO LIVE==►► CLICK HERE TO WATCH LIVE

🔴 STREAMING==►► CLICK HERE TO WATCH LIVE
How to Watch Buccaneers vs. Broncos: Time, TV Channel and Live Stream – 2024 Week 3
Check out the full NFL football schedule on FOX Sports! Or visit FOX Sports' NFL hub for everything you need to make the most of this season's action.
How to Watch Buccaneers vs. Broncos
When: Sunday, September 22, 2024 at 1:00 p.m. ET
Where: Raymond James Stadium in Tampa, Florida
TV: Watch on FOX
Learn more about the Tampa Bay Buccaneers vs. the Denver Broncos on FOX Sports!
🔴 GO LIVE==►► CLICK HERE TO WATCH LIVE

🔴 STREAMING==►► CLICK HERE TO WATCH LIVE
Buccaneers vs. Broncos: Head-to-Head
Star QB Baker Mayfield and the Tampa Bay Buccaneers face the Denver Broncos on Sunday at 1:00 p.m. ET at Raymond James Stadium.
Despite having a bottom-five defense that ranks fifth-worst in the NFL (381 yards allowed per game), the Buccaneers have put up better results on the other side of the ball, ranking 20th in the NFL by totaling 304 yards per game. The Broncos rank fifth-worst in total yards per game (263), but they've been more effective on the defensive side of the ball, ranking ninth in the NFL with 277.5 total yards ceded per contest.
See more details below, including how to watch this matchup on TV.
Former Duck’s quarterback Bo Nix leads the rebuilding Denver Broncos against the revamped Tampa Bay Buccaneers in Week 4 regular season NFL action. Tampa Bay is led by Baker Mayfield, who has been crowned “the guy” after an offseason contract extension and hot start to 2024. This game kicks off at 10 a.m. PT/1 p.m. ET on Sunday, September 22 with a live broadcast on FOX, and streaming live on demand.
• WATCH: Broncos vs. Bucs live for free with Fubo (free trial), or with DirecTV Stream (cheapest streaming plans, $25 off first month) or see more streaming options below.
STREAMING GUIDES FOR SPORTS, TV & MOVIES
What TV channel is Peacock on? Is there a free trial? Promo code, student discount and how to login
ESPN Plus college football 2024 streaming guide, subscription cost, game schedule; is there a free trial?
What is BritBox? Is there a free trial? Shows list, subscription cost, is it worth getting in 2024?
What TV channel is Investigation Discovery on? How to watch true crime series on ID Network streaming free online
🔴 GO LIVE==►► CLICK HERE TO WATCH LIVE

🔴 STREAMING==►► CLICK HERE TO WATCH LIVE
What is Philo TV? Cost, free trial promo, channels lineup; why it’s the cheapest live streaming, on demand service
Who: Los Angeles Chargers vs Pittsburgh Steelers
When: Sunday, Sept. 22 at 1 p.m. EST
Where: Raymond James Stadium in Tampa, Fla.
Stream: fuboTV (free trial); or Sling; DirecTV Stream (free trial) or NFL+
What is FuboTV?
FuboTV is an internet television service that offers more than 200 channels across sports and entertainment including Paramount+ with SHOWTIME. From the UEFA Champions League to the WNBA to international tournaments ranging across sports, there’s plenty of options available on FuboTV, which offers a free trial, and $30 off the first month for new costumers.


Wrong WireMock.defaultInstance when switching back to previous application context

I have a test suite with tests that use two different @SpringBootTest configurations, say A and B. Due to its caching capabilities, the Spring test support will only create two application contexts, regardless of how many tests there are. Depending on the order in which test classes are executed, it may be that tests first use the contexts for A, then B, and then A again. In this situation, WireMock will not work for the tests that use A again because WireMock.defaultInstance always points to the WireMock instance belonging to the application context that was last created, not to the one belonging to the currently active application context.
The reason is that WireMock.defaultInstance is only updated on context creation, where org.springframework.cloud.contract.wiremock.WireMockConfiguration.start() is called, which in turn calls updateCurrentServer() and this calls com.github.tomakehurst.wiremock.client.WireMock.configureFor(WireMock).

I was able to work around this problem with a custom TestExecutionListener:
public class MyWireMockTestExecutionListener implements TestExecutionListener {
	private static final Logger LOGGER = LoggerFactory.getLogger(WireMockTestExecutionListener.class);

	@Override
	public void beforeTestClass(TestContext testContext) throws Exception {
		WireMockInstanceStore instanceStore = testContext.getApplicationContext().getBean(WireMockInstanceStore.class);

		if (instanceStore.get() != null) {
			// WireMock was started for this application context before; make this the (thread local) current instance in case a different application context was active in between
			WireMock.configureFor(instanceStore.get());
		} else {
			LOGGER.info("Detected a new application context; storing WireMock instance for this context");
			instanceStore.set(getConfiguredWireMockInstance());
		}
	}

	private static WireMock getConfiguredWireMockInstance() throws Exception {
		Field wireMockInstanceField = WireMock.class.getDeclaredField("defaultInstance");
		wireMockInstanceField.setAccessible(true);
		ThreadLocal<?> fieldValue = (ThreadLocal<?>) wireMockInstanceField.get(null);

		WireMock wireMockInstance = (WireMock) fieldValue.get();
		if (wireMockInstance == null) {
			throw new IllegalStateException("WireMock is not started");
		}
		return wireMockInstance;
	}
}

... and a simple helper to hold the WireMock instance:
@Component
public class WireMockInstanceStore {
	private WireMock wireMockInstance;

	public void set(WireMock wireMockInstance) {
		this.wireMockInstance = wireMockInstance;
	}

	public WireMock get() {
		return wireMockInstance;
	}
}

This listener (or something similar) should probably be part of the library.

BTW, I'm not running into the documented port limitation – I just get a separate random random port per application context. This works because org.springframework.cloud.contract.wiremock.WireMockApplicationListener.registerPort(ConfigurableEnvironment) runs once per application context.

Hi @oberlies ; would using @DirtiesContext help here? I know it's not the best thing to do, but it would be helpful to know if that solves the issue or not.

No, @DirtiesContext doesn't help. The WireMock.defaultInstance still always points to the instance of the last created ApplicationContext.


Can't use multi instance

How can I use a multi-instance of this component on the same page?

Release v3.0.1


Shared Links: Permission denied when trying to access draft when logged in

Describe the bug
The draft is not accessible right now if a user creates a shareable link for it by generating it with access Can preview drafts, and then trying to open that link, logged in as a different user.
Steps to Reproduce

Create and save a draft
Click on Share and generate a link with access set as Can preview drafts
Cope the share link
Log in as a different user and go to the shared link

Expected behavior
It should open up the draft
Screenshots (if applicable)

Resolved


947 fix security vulnerability warning in survey runner

What is the context of this PR?
A high severity security vulnerability is being reported in one of the project dependencies. See https://nvd.nist.gov/vuln/detail/CVE-2017-18342 for more details.
This change updates the version of PyYAML to a patched version to mitigate the issue.
The dependency can be changed back to '*' in the Pipfile once a released version has been made available on PyPI.
How to review
All tests and checks should pass.
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

I previously avoided doing this because the only version with a fix for it is a prerelease version.
If you actually look at it, the github notification is not great, it's been an issue in pyyaml for a long time (years). They're just saying that you shouldn't use load and should always use safe_load. This is noted clearly in the docs.

@bitdivision Yeah I assumed this was the case, but a card made its way onto our board to investigate. Should I close the PR in that case? Are we happy to accept the risk until a release is published?

I think admins have the ability to dismiss those vulnerability alerts?
Up to you really, I felt a bit weird about having prerelease versions of a library pinned, and eventually gave up because I had to change the pipenv command in travis to allow prerelease I think.
There's quite a bit of discussion around this here: https://github.com/yaml/pyyaml/issues/243
I'll approve as I think it's probably fine to merge this.

@samiwel @bitdivision Given what Rich has said I don't think we should merge this pr. I've checked the code and we only load yaml in two places (secrets and keys) and in both instances we use safe_load.

Closing this PR as per discussion above.


added the upstream option for the single snapshot mode

LGTM

please?

@masterkain Can you please rebase the change


breadwallet-android\app\src\main\jni\breadwallet-core\BRAddress.c

C:\Users\niewenlong\Documents\GitHub\breadwallet-android\app\src\main\jni\breadwallet-core\BRAddress.c
Error:(281) undefined reference to BRBech32Encode' Error:(378) undefined reference toBRBech32Decode'
Error:(404) undefined reference to `BRBech32Decode'
I don't know what the problem is. I want to ask you to help you. Thank you.

Fixed


Reduce code complexity

Describe the PR
code-refactoring
$ ~/go/bin/gocyclo -over 15 .
48 swag (*Parser).parseField parser.go:1010:1
46 swag (*Parser).ParseGeneralAPIInfo parser.go:177:1
33 swag (*Parser).parseStructField parser.go:764:1
16 swag (*Operation).parseAndExtractionParamAttribute operation.go:233:1

Relation issue
None
Additional context
Original results
Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

swag/parser.go
Line 177: warning: cyclomatic complexity 83 of function (*Parser).ParseGeneralAPIInfo() is high (> 15) (gocyclo)
Line 1081: warning: cyclomatic complexity 48 of function (*Parser).parseField() is high (> 15) (gocyclo)
Line 834: warning: cyclomatic complexity 23 of function (*Parser).parseStruct() is high (> 15) (gocyclo)
Line 694: warning: cyclomatic complexity 22 of function (*Parser).parseTypeExpr() is high (> 15) (gocyclo)
Line 1017: warning: cyclomatic complexity 17 of function (*Parser).parseAnonymousField() is high (> 15) (gocyclo)
swag/operation.go
Line 241: warning: cyclomatic complexity 29 of function (*Operation).parseAndExtractionParamAttribute() is high (> 15) (gocyclo)
Line 60: warning: cyclomatic complexity 27 of function (*Operation).ParseComment() is high (> 15) (gocyclo)

Codecov Report

Merging #487 into master will increase coverage by 1.12%.
The diff coverage is 84.74%.

@@            Coverage Diff            @@
##           master    #487      +/-   ##
=========================================
+ Coverage   86.58%   87.7%   +1.12%
=========================================
  Files           7       7
  Lines        1558    1489      -69
=========================================
- Hits         1349    1306      -43
+ Misses        120     104      -16
+ Partials       89      79      -10

Impacted Files
Coverage Δ

operation.go
95.45% <100%> (+0.36%)
:arrow_up:

parser.go
82.59% <81.17%> (+1.52%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature registrion flow

The default registration flow does not work for PGConf. It does not gather enough information about  an attendee while unnecessarily adding too many steps like requiring both a username and email.
Registration

Make Affiliation required
Change the change the info text for Affilation
Add title field
Add phone number
Change nickname label to Twitter Handle on the profile page
Split Name to first and last names
Remove Travel Schedule from Registration Info page
Remove the registration counts for events on the Registration Info Page
Remove the pre-registration block from the initial registration form
Remove username from the signup form but save it as email address
Remove the pre-register section on the registration form
Hide username from the end user and just use email
Already has a account screen should use email to login instead of username
Signup form needs the full list of required fields

CFP

Remove the registration checkbox on a new submission
Remove the whole Enabale pre-registraiton section on the edit proposal page (hide behind admin)
Difficulty Level field available on the new submission form
Make the commercial functionality available for admins only

This was merge in from the jim-pr5-merge branch to resolve some conflicts


How to route with  any  Class ?

I want to route as blew:
$app->get('/{contr}/{act}', '\app\controllers\{contr}Controller:{act}');

if it's ok,we can route any class used only one router.

Can't be done.

is is can be done?
$app->get('/default/{act}', '\app\controllers\DefaultController:{act}');

Use this
$app->any('[/{params:.*}]', function ($request, $response, $args) use ($container){
    $params = explode('/', $request->getAttribute('params'));

    return $response->write(var_export($params, true));
});

Not that I recommend this. I do not.
Something like this should work.
$app->get('/{contr}/{act}[/{args:*}]', function (Request $request, Response $response, $args = []) {
     return $this->get($args['contr'])->{$act}($request, $response, $args);
});

If you have any other questions, please reopen this issue.


Update documentation based on GET /fills deprecation notice

There's a giant deprecation notice for the GET /fills endpoint https://docs.gdax.com/#fills
The README on this project needs to be updated to have an example of how to pass in the product_id or order_id.

Coinbase::Exchange::BadRequestError: {"message":"query must contained either product_id or order_id"}

I am getting this error.

GET /products

Will provide product ids. But I am now confused on how to do pagination any idea?


[Backport 2.x] Implement segment replication event cancellation.

Manual backport of https://github.com/opensearch-project/OpenSearch/pull/4225 to 2.x.

Failing due to heap space issue. Tracked in https://github.com/opensearch-project/OpenSearch/issues/3973.
Refiring!
1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':plugins:discovery-azure-classic:compileInternalClusterTestJava'.
> java.lang.OutOfMemoryError: Java heap space

Execution failed for task ':distribution:bwc:bugfix:buildBwcLinuxTar'.
> Building 2.2.1 didn't generate expected file /var/jenkins/workspace/gradle-check/search/distribution/bwc/bugfix/build/bwc/checkout-2.2/distribution/archives/linux-tar/build/distributions/opensearch-min-2.2.1-SNAPSHOT-linux-x64.tar.gz

  2> REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testRestoreSnapshotAllocationDoesNotExceedWatermark" -Dtests.seed=2C6090E329EBBFC4 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=es-HN -Dtests.timezone=Asia/Karachi -Druntime.java=17
  2> java.lang.AssertionError:
    Expected: an empty collection
         but: <[[qfddaotgwv][1], node[Ak81Pn6GRbeGBUORxWl2bg], [P], s[STARTED], a[id=zu2e5c5bSwyE-ZDsdLv4_g], [qfddaotgwv][5], node[Ak81Pn6GRbeGBUORxWl2bg], [P], s[STARTED], a[id=bO_j9IkxT6Ot4qY8ucWCPg]]>
        at __randomizedtesting.SeedInfo.seed([2C6090E329EBBFC4:3ED836824252A07C]:0)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
        at org.junit.Assert.assertThat(Assert.java:964)
        at org.junit.Assert.assertThat(Assert.java:930)
        at org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.lambda$testRestoreSnapshotAllocationDoesNotExceedWatermark$2(DiskThresholdDeciderIT.java:260)
        at org.opensearch.test.OpenSearchTestCase.assertBusy(OpenSearchTestCase.java:1049)
        at org.opensearch.test.OpenSearchTestCase.assertBusy(OpenSearchTestCase.java:1022)
        at org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testRestoreSnapshotAllocationDoesNotExceedWatermark(DiskThresholdDeciderIT.java:260)

Codecov Report

Merging #4387 (6818bda) into 2.x (ea5b946) will increase coverage by 0.04%.
The diff coverage is 85.84%.

@@             Coverage Diff              @@
##                2.x    #4387      +/-   ##
============================================
+ Coverage     70.56%   70.60%   +0.04%
- Complexity    56923    56962      +39
============================================
  Files          4572     4573       +1
  Lines        273817   273888      +71
  Branches      40152    40159       +7
============================================
+ Hits         193207   193390     +183
+ Misses        64450    64323     -127
- Partials      16160    16175      +15

Impacted Files
Coverage Δ

...s/replication/SegmentReplicationSourceService.java
78.46% <61.53%> (+15.00%)
:arrow_up:

...s/replication/SegmentReplicationSourceHandler.java
90.78% <71.42%> (+0.93%)
:arrow_up:

...h/indices/replication/SegmentReplicationState.java
78.94% <80.00%> (-0.16%)
:arrow_down:

...ndices/replication/OngoingSegmentReplications.java
93.05% <83.33%> (+1.62%)
:arrow_up:

...s/replication/SegmentReplicationTargetService.java
58.16% <85.71%> (-8.51%)
:arrow_down:

...org/opensearch/index/shard/IndexShardTestCase.java
94.01% <90.00%> (-0.19%)
:arrow_down:

...ch/indices/cluster/IndicesClusterStateService.java
75.28% <100.00%> (+0.64%)
:arrow_up:

...ces/replication/PrimaryShardReplicationSource.java
100.00% <100.00%> (ø)

.../indices/replication/SegmentReplicationSource.java
100.00% <100.00%> (ø)

.../indices/replication/SegmentReplicationTarget.java
90.99% <100.00%> (+1.40%)
:arrow_up:

... and 516 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mch2 : Looks like a conflict, can you rebase and merge ?

REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.cluster.allocation.AwarenessAllocationIT.testThreeZoneOneReplicaWithForceZoneValueAndLoadAwareness" -Dtests.seed=196AC1473FA624F2 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=fi-FI -Dtests.timezone=Pacific/Apia -Druntime.java=17


Pagination options / endless scroll

Thank you very much for refined github. 👍
There's something that bothers me since ages:
github search for issues and similar objects currently shows 10 items per page.
It would be wonderful if this could be changed to ie 100 items per page (with prefetching) and/or endless scroll.

Big 👍. The 10 item limit is super annoying.

For anyone that wants this, please also send this feature request to GitHub<EMAIL_ADDRESS>

The limit has since been increased to 25, I think it's reasonable now. Plus in 1.5 years no one picked it up

note to self: this was added for a couple pages in #1516, but those pages just have simple "next" buttons to load, not a list of pages. I don't think that code could be easily modified as is to load the next button from that list. There'd have to be extra logic for tracking which page you've last loaded, probably similar to how RES does it.
/someone for whom 25 still isn't enough 😪


Webpack: Cannot read properties of undefined (reading 'stat')

Steps to reproduce
webpack === 5.72.1
typescript === 5.3.3
@dsnp/parquetjs == 1.6.2
node === 20
craco.config.js
webpack: {
    configure: (webpackConfig) => {
      webpackConfig.resolve = {
        ...webpackConfig.resolve,
        alias: {
          '@dsnp/parquetjs': path.resolve(__dirname, 'src', 'parquetjs', 'dist', 'browser', 'parquet.cjs.js'),
        },
        fallback: {
          crypto: require.resolve('crypto-browserify'),
          stream: require.resolve('stream-browserify'),
        },
      };

      return webpackConfig;
    },
  },

app.js
import * as pq from '@dsnp/parquetjs';

const reader = await pq.ParquetReader.openFile(file.name);

Expected behaviour
it reads the file
Actual behaviour
throws error
Any logs, error output, etc?
TypeError: Cannot read properties of undefined (reading 'stat') at http://localhost:4000/static/js/vendors-node_modules_dsnp_parquetjs_dist_browser_parquet_cjs_js-node_modules_antd_es_divider_-cbb199.chunk.js:36235:16 at new Promise (<anonymous>) at Y1 (http://localhost:4000/static/js/vendors-node_modules_dsnp_parquetjs_dist_browser_parquet_cjs_js-node_modules_antd_es_divider_-cbb199.chunk.js:36234:10) at e.openFile (http://localhost:4000/static/js/vendors-node_modules_dsnp_parquetjs_dist_browser_parquet_cjs_js-node_modules_antd_es_divider_-cbb199.chunk.js:39916:19) at e.openFile (http://localhost:4000/static/js/vendors-node_modules_dsnp_parquetjs_dist_browser_parquet_cjs_js-node_modules_antd_es_divider_-cbb199.chunk.js:39793:22) at beforeUpload

Any other comments?
...

@stefanonardo Are you running this in a browser or in Node?

If a browser, what is your file.name value?
If in Node, why are you using the browser version

hey @wilwade, thank you for answering. I moved to another library so I cannot test it anymore, you can close the issue. However, I was running it in a browser andfile.name was a string containing the name of an uploaded file (through Antd Upload component).

Thanks for the update!


Login Fail (Dont understand why)

I have this error:
C:\Users\Administrator\Desktop\Bot>node tradebot.js
socket timed out
socket closed
connecting to <IP_ADDRESS>:27017
connected
encrypt request
handshake complete
logon fail: 65, sessionID: 491969
events.js:141
throw er; // Unhandled 'error' event
^
Error: Logon fail: 65
at SteamClient.handlers.(anonymous function) (C:\Users\Administrator\Desktop
\Bot\node_modules\steam\lib\handlers\user.js:178:11)
at SteamClient._netMsgReceived (C:\Users\Administrator\Desktop\Bot\node_modu
les\steam\lib\steam_client.js:106:26)
at SteamClient.handlers.(anonymous function) (C:\Users\Administrator\Desktop
\Bot\node_modules\steam\lib\steam_client.js:192:10)
at SteamClient._netMsgReceived (C:\Users\Administrator\Desktop\Bot\node_modu
les\steam\lib\steam_client.js:106:26)
at emitOne (events.js:77:13)
at Connection.emit (events.js:169:7)
at Connection.readPacket (C:\Users\Administrator\Desktop\Bot\node_modules\s
team\lib\connection.js:50:8)
at emitNone (events.js:67:13)
at Connection.emit (events.js:166:7)
at emitReadable (_stream_readable.js:411:10)
C:\Users\Administrator\Desktop\Bot>pause
Press any key to continue . . .

I was looking to fix it but I already put the Authcode that Steam send to my email, whan I am doing wrong?
Code of bot.js
var admin = 'Censored';
var logOnOptions = {
accountName: 'Censored',
password: 'Censored'
};
var GameTime = 120;
////
var authCode = 'VHKNW';
var globalSessionID;
if (require('fs').existsSync('sentry_'+logOnOptions['accountName']+'.hash')) {
logOnOptions['shaSentryfile'] = require('fs').readFileSync('sentry_'+logOnOptions['accountName']+'.hash');
} else if (authCode != '') {
logOnOptions['authCode'] = authCode;
}

You dont know to solved it? Could you pls send me a link where I can post it to get an answer?

var authCode = 'VHKNW';
This SteamGuard code changes, it's not static. You can't use the same over and over, hence the error. Reading the docs would certainly help you. :+1:

Along with what @scholtzm said, understanding the code would also help. Just writing your credentials in the stolen CSGOFast bot won't do much when you have to fix it.


Safari, iFrames, and CORS

Hi and thanks for creating itty-cors!
There is a new issue in Safari and Safari based browsers (e.g. everything on iPhones) where Safari's preflight check attempts to match the Origin to the URL making the request, EVEN if that request is coming from an iFrame.  So when a page a AAA.net embeds an iFrame at BBB.org, Safari as of September 2023 now insists that the CORS headers coming back to request coming from BBB.org have an Access-Control-Allow-Origin header allowing not BBB.org but AAA.net.  For us, in practice this has meant that we just have to allow * or be broken in Safari.
In itty-cors, when we set Origin: ['*'] in the config, that isn't what we get in the Header.  Line 33 does this:
allowOrigin = (origins.includes(origin) || origins.includes('*')) &&       { 'Access-Control-Allow-Origin': origin }
meaning that I will get the Origin domain written into the header, not the literal "*"
When this happens, in the case of an iFrame request, it fails preflight in Safari and so breaks.
Can you just return the intact header "Access-Control-Allow-Origin": "*" for cases where the setting is "Origin: ['*']" or in the default case where nothing is set?

So, the terribly [late] answer is that:
The built-in cors in itty-router v5.x does fully support a '*' origin, and will only reflect the origin (as you're seeing) under specific circumstances, that line up with the behavior of industry standards (express.js).
I highly recommend following the migration guide here, (and likely switching over to AutoRouter to save a bunch of code as well).
https://itty.dev/itty-router/migrations/v4-v5


Problem with health bar

Good! I come to report a visual error in the health bar that is modified by the mod (ScalingHealth-1.12.2-1.3.38 + 143) with better hud (betterhud-1.12.2-1.4) and the thirst bar added by the mod (ToughAsNails-1.12.2-<IP_ADDRESS>-universal). The health and thirst bars are overlaid with those in the inventory and better hud doesn't let me move them. Only solved if I deactivate better hud
It is not possible to edit the location of both bars (Health and thirst)
How can I prevent better hud from modifying the vanilla bars?
Problem image:

Image without problem (Better hud disabled):

Configs
Desktop.zip

Hey !
Same problem here, seems like the mod don't take over the vanilla health bar so it's right on the horbar.
For strictly HUD related mod myself, I think I 'only' have two: "Overloaded Armor Bar" & "Multi Hot Bar".
Pretty gud mod ! Well made, accessability, in two word: quality content !


Citizen.Await returned (nil) before the areturn call.

Error resuming coroutine: @vrp/lib/utils.lua:69: async wait(): Citizen.Await returned (nil) before the areturn call.
stack traceback:
[C]: in function 'error'
@vrp/lib/utils.lua:69: in function <@vrp/lib/utils.lua:63>
(...tail calls...)
[string "vrp/vRP.lua"]:131: in method 'registerDBDriver'
[string "vrp_ghmattimysql/init_vrp.lua"]:66: in function <[string "vrp_ghmattimysql/init_vrp.lua"]:65>

I'm getting this error everytime I start the server.
When I join, I get this new error:

Connecting: Fitz Simmons
[GHMattiMySQL ERROR] Table 'vrp.vrp_character_homes' doesn't exist
[vRP] Fitz Simmons (<IP_ADDRESS>) joined (user_id = 1)

And once I spawn:

error: proxy call vRP:getUserId not found
error: proxy call vRP:getUserIdentity not found
Syncing weather for: Fitz Simmons
[{"locked":false},{"locked":false},{"locked":false},{"locked":false}]
Error resuming coroutine: citizen:/scripting/lua/scheduler.lua:41: Failed to execute thread: @vrp/modules/group.lua:372: attempt to index a nil value (local 'ment')
stack traceback:
@vrp/modules/group.lua:372: in upvalue 'func'
[string "vrp/vRPShared.lua"]:113: in function <[string "vrp/vRPShared.lua"]:112>
stack traceback:
[C]: in function 'error'
citizen:/scripting/lua/scheduler.lua:41: in field 'CreateThreadNow'
@vrp/lib/utils.lua:85: in function 'async'
[string "vrp/vRPShared.lua"]:112: in method 'triggerEvent'
[string "vrp/vRP.lua"]:533: in function <[string "vrp/vRP.lua"]:532>

And I'm just floating stuck in the spawn area (my model is also invisible)
Already reinstalled the database three times, and remove everything from the server cache, still no luck.
Is there any fix to this?

Nvm, didn't read about broken scheduler.lua , it's now fixed 🥇


TILA-1355 | Fix CSV faulty write

Quote all data and change writing to excel style. Some data is difficult for default style and columns are not correct. This commit fixes the issues.

Also, black reservation_units/utils/ does not re-format code :thinking:

Ah, it's isort. Never mind, fixed.


How to return an error when creating a filter with faulty values ?

I've encounterd something very strange..
When i'm trying to execute a GET request to retrieve a list of items, i can also specify a filter but when those values are wrong ( ex. guid == leaves) the result still gives me a complete list with all the entities back from the database. ( ex. GetUsers returns all the users)
It looks like there is no validation on the filters.
How do i return an error when the result of the filter returns nothing?

Just gave this a shot and it looks like it works fine for non-guids (returns an empty list when no records match the filter), but does indeed not work for guid fields. This is a bug in Sieve. I submitted a ticket to them. Will try and submit a PR to them myself at some point, but if you want to as well, go for it!
Closing this as it is not necessarily an issue with craftsman as much as Sieve.


new widget: widmsggrid: Messages Grid Widget

Displays multiple notification icons in a grid.
I only now noticed that settings.showRead ("Show widget even for read messages") isn't present in the default messages app. I commented it out in the README for now, and I'll make a PR to add it Soon. (Unless you already think it's a bad idea)

Looks great! So you have to disable the built-in messages widget?

Yes, I couldn't think of a good way to temporarily disable it, and didn't want
[install widmsggrid]->[uninstall widmsggrid]->[default widget is somehow missing]
Maybe the Readme could be a bit clearer...

Maybe the Readme could be a bit clearer...

Actually yes, it'd be great if it could mention in there how you disable the widget


[DO NOT MERGE] extend Tag for TagSuggestion, TagOption...

Just pushing this up as an example for #13  — IT IS NOT DONE. the generic type T is becoming never and unfortunately I haven't figured out why.

I am not a TS expert so I wonder if I'm doing something fundamentally wrong here, but applying <T extends Tag> everywhere kind of sucks :( and makes this never issue hard to triangulate


Add language colour for Roff

This PR adds a colour for Roff (incorrectly named "Groff" on GitHub):

I chose a colour to match the look of aged paper, which symbolises both the language's longevity and its importance in authoring documentation/manuals. No canonical implementation of Roff exists which has any branding with distinguishable colours... so I fell back on natural sources as my inspiration.

This PR also attempted to add a colour for the PostScript language, which was the same PANTONE red used in Adobe's official branding. Example:

However, the colour-proximity test was blocking every attempt at picking a close enough colour... and even shades that were noticeably different were being rejected. Here's a comparison of the conflicting colours, with the attempted PostScript shades on the right:

This might be attributed to my colour-blindness, but I fail to see how the bottom-right colour could possibly be confused with any of the other shades depicted here.

@Alhadis Neat PR as always! :+1:

This might be attributed to my colour-blindness, but I fail to see how the bottom-right colour could possibly be confused with any of the other shades depicted here.

I'm not saying it's the case here, but sometimes it's harder to distinguish colors when they are on the narrow language bar.
In any case, the formula to compute color distances is not perfect and probably never will since that depends on people's views too.

Looks good 👍

Thanks @arfon. =)

In any case, the formula to compute colour distances is not perfect and probably never will since that depends on people's views too

That's why I personally feel the colour proximity should raise a warning rather than a build error. If two colours differ visually more than they do mathematically, it shouldn't be a deal-breaker.


Steam: gameoverlayui crash after one screenshot is taken

Describe the bug
When i start a game, the hint is displayed that i can start the Steam Overlay. It works to start it and go back to game.
But when i press the key to take a screenshot in Steam (tried 0, F11 and -), i don't get the visual or audio notification that a screenshot is taken and after that, i can't open the Overlay again and also don't take more screenshots.
It worked some time ago. I think before the last update.
Steps To Reproduce
Steps to reproduce the behavior:

Start a game in Steam
Open Steam Overlay
Take a screenshot using Steam Hotkey
Try to open Overlay again (it should fail)

Expected behavior
Being able to take screenshots and open the Overlay (not that important for me).
Additional context
Tested games:

Blush Blush (via Proton)
GemCraft - Chasing Shadows (via Proton)
Half-Life 2 (via Proton)
Kingdom: New Lands (native)

so i suppose all games are affected
Terminal output:
GameAction [AppID 777130, ActionID 11] : LaunchApp changed task to ProcessingInstallScript with ""
GameAction [AppID 777130, ActionID 11] : LaunchApp changed task to SynchronizingCloud with ""
GameAction [AppID 777130, ActionID 11] : LaunchApp changed task to SynchronizingControllerConfig with ""
GameAction [AppID 777130, ActionID 11] : LaunchApp changed task to SiteLicenseSeatCheckout with ""
GameAction [AppID 777130, ActionID 11] : LaunchApp changed task to CreatingProcess with ""
GameAction [AppID 777130, ActionID 11] : LaunchApp waiting for user response to CreatingProcess ""
GameAction [AppID 777130, ActionID 11] : LaunchApp continues with user response "CreatingProcess"
/bin/sh\0-c\0/home/davidak/.local/share/Steam/ubuntu12_32/reaper SteamLaunch AppId=777130 -- /home/davidak/.local/share/Steam/ubuntu12_32/steam-launch-wrapper -- '/run/media/davidak/476c15a0-7277-4427-b7e5-74ac022724bc/SteamLibrary/steamapps/common/BlushBlush/BlushBlush.x86_64'\0
Game process added : AppID 777130 "/home/davidak/.local/share/Steam/ubuntu12_32/reaper SteamLaunch AppId=777130 -- /home/davidak/.local/share/Steam/ubuntu12_32/steam-launch-wrapper -- '/run/media/davidak/476c15a0-7277-4427-b7e5-74ac022724bc/SteamLibrary/steamapps/common/BlushBlush/BlushBlush.x86_64'", ProcID 827709, IP <IP_ADDRESS>:0
chdir /run/media/davidak/476c15a0-7277-4427-b7e5-74ac022724bc/SteamLibrary/steamapps/common/BlushBlush
ERROR: ld.so: object '/home/davidak/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.
GameAction [AppID 777130, ActionID 11] : LaunchApp changed task to WaitingGameWindow with ""
GameAction [AppID 777130, ActionID 11] : LaunchApp changed task to Completed with ""
ERROR: ld.so: object '/home/davidak/.local/share/Steam/ubuntu12_64/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS64): ignored.
ERROR: ld.so: object '/home/davidak/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.
ERROR: ld.so: object '/home/davidak/.local/share/Steam/ubuntu12_32/gameoverlayrenderer.so' from LD_PRELOAD cannot be preloaded (wrong ELF class: ELFCLASS32): ignored.

Before what update? Steam or NixOS?

@Atemu before i had this quite old Steam version: https://github.com/NixOS/nixpkgs/issues/181904#issuecomment-1364703895

I have the same version and am seeing the same issue

The issue was fixed in the latest Steam update.


Notification emails do not notify all incidents and status update.

Please include a note in the following text.
========
To sign up for email notifications of new incidents that affect your tenant and status changes for an active incident, select Preferences > Email , click Send me service heath notifications in email , and then specify:
Up to two email addresses.
Whether you want notifications for incidents or advisories
The services for which you want notification
If the service is restored immediately, or depending on the scope of the incident and advisory, email notifications may be skipped at the discretion of our data center.
As a precaution, is it possible to add a statement that notifications may be skipped?

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

ID: c2d8b9c3-7ff2-d3b7-a0ce-0310171c71d5
Version Independent ID: 7b45c0a3-94ca-b749-50e4-7d4cbf204764
Content: How to check Microsoft 365 service health - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/view-service-health.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@yutoy1439 Thank you for your feedback. I'm checking with the author and will update you shortly.
Thanks
Sri

We’ve passed on this suggestion to the documentation team.
Closing this issue.
cc: @yogkumgit @msbemba


perf: don't create duplicate function/object templates for ops

We were creating them twice - once when attaching them to Deno.core.ops, then creating all new templates when setting up the virtual ops module.
That meant that the snapshot included two copies of each template, bloating the snapshot size and hurting snapshot deserialization time.
To fix this, just reuse the templates we've already attached to Deno.core.ops.
This greatly improves the impact of object wrap (and ops in general) on snapshot size and deserialization time.
statrtup perf:
with 1000 object wrap DOMPoints vs 1000 pure JS DOMPoints (dcore-objwrap-1000-opt is this PR)
❯ hyperfine -w 500 -N "./dcore-js-1000 empty.js" "./dcore-objwrap-1000 empty.js" "./dcore-objwrap-1000-opt empty.js"

Benchmark 1: ./dcore-js-1000 empty.js
  Time (mean ± σ):      10.5 ms ±   0.5 ms    [User: 7.4 ms, System: 2.6 ms]
  Range (min … max):     9.5 ms …  11.7 ms    297 runs

Benchmark 2: ./dcore-objwrap-1000 empty.js
  Time (mean ± σ):      14.7 ms ±   0.7 ms    [User: 10.1 ms, System: 3.8 ms]
  Range (min … max):    13.4 ms …  17.5 ms    203 runs

Benchmark 3: ./dcore-objwrap-1000-opt empty.js
  Time (mean ± σ):      10.4 ms ±   0.6 ms    [User: 6.7 ms, System: 3.0 ms]
  Range (min … max):     9.3 ms …  13.7 ms    275 runs

Summary
  ./dcore-objwrap-1000-opt empty.js ran
    1.01 ± 0.08 times faster than ./dcore-js-1000 empty.js
    1.41 ± 0.11 times faster than ./dcore-objwrap-1000 empty.js

snapshot size (in particular the size of the isolate and context 1 snapshots):
❯ ./dcore-objwrap-1000 --v8-flags=--profile_deserialization empty.js
🛑 deno_core binary is meant for development and testing purposes.
Run empty.js
[Deserializing read-only space (420972 bytes) took 0.292 ms]
[Deserializing isolate (3510040 bytes) took 8.875 ms]
[Deserializing context #1 (1863760 bytes) took 5.333 ms]

❯ ./dcore-objwrap-1000-opt --v8-flags=--profile_deserialization empty.js
🛑 deno_core binary is meant for development and testing purposes.
Run empty.js
[Deserializing read-only space (420972 bytes) took 0.250 ms]
[Deserializing isolate (1865184 bytes) took 4.083 ms]
[Deserializing context #1 (1007968 bytes) took 2.666 ms]

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

Project coverage is 81.29%. Comparing base (0c7f83e) to head (517dddc).
Report is 203 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1010      +/-   ##
==========================================
- Coverage   81.43%   81.29%   -0.15%
==========================================
  Files          97       99       +2
  Lines       23877    26091    +2214
==========================================
+ Hits        19445    21211    +1766
- Misses       4432     4880     +448

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


[bug]: When using Popover with PopoverAnchor instead of PopoverTrigger, onOpenChange is never called with true, only false.

Describe the bug
See: https://github.com/shadcn-ui/ui/discussions/1684
And: https://github.com/shadcn-ui/ui/issues/2798
When using <PopoverAnchor> instead of <PopoverTrigger> in order to manually/programmatically open the Popover (via passing a boolean state to the open prop) from unrelated components, onOpenChange is only ever called with false, and at no point will be called with true.
As a workaround, I am using a 200ms timeout when calling setting the open prop state.
Affected component/components
Popover
How to reproduce
const [open, setOpen] = useState(false);

[...]

<Popover
  open={open}
  onOpenChange={(isOpen) => {
    console.log('isOpen', isOpen); // is ALWAYS false
  }}
>
  <PopoverAnchor>
    [...]
  </PopoverAnchor>
  <PopoverContent>[...]</PopoverContent>
</Popover>

[...]

 setOpen(true)}>some div

Codesandbox/StackBlitz link
Your v0 doesn't allow dependencies to be changed, and I couldn't figure out how to get shadcn up and running on codesandbox.
Logs
No response
System Info
Chrome on 2023 MacBook Pro w/ macOS 14.1, 36GB RAM, and M3 Max

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

@josheverett As mentioned in #2798 , can you try passing a setter to onOpenChange? Work fine with that.


Time period for flow count?

"the Azure networking stack supports 250K total network flows with good performance" - this leaves me puzzling, which time frame my server has to generate those 250K flows, and how/when the count is reset.
Is a flow counted for one hour, 24 hours, 1 month, ... or is the count set to zero each midnight, on the first of a month, etc.?

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

ID: c7c054b7-422c-c3b0-fc8e-91314231ee0c
Version Independent ID: 42825540-2137-92e3-08b0-0dbd84089eda
Content: Azure virtual machine network throughput
Content Source: articles/virtual-network/virtual-machine-network-throughput.md
Service: virtual-network
GitHub Login: @steveesp
Microsoft Alias: steveesp

@stracker-phil It refers to 250K flows that all exist at the same time.
Please let me know if you need any additional information.

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

@TravisCragg-MSFT  The doc says The Azure networking stack maintains state for each direction of a TCP/UDP connection in data structures called ‘flows’.   how long is this state maintained for UDP traffic?


broked becouse of 2.201

ye it is not working becouse new update

That's why I am still on 2.2.

Like there is no reason for me to update


openui5 XSD files with errors

Hello,
I'm using openui5-sdk-1.52.9
The XSD files can't be used to generate intellisense/autocompletion for Visual Studio 2017. It gives the following error:
_Type 'sap.ui.core:TooltipBaseType' is not declared, or is not a simple type.
I thought that it could be a Visual Studio issue but I downloaded "UI development toolkit for HTML5" for Eclipse and I see that there you have XML templates instead of using the XSD files, because they don't work either. Also other XSD tools I've tried complained about the XSD files in download/schemas
I have tried several views and it always happens. For example:
<mvc:View xmlns:core="sap.ui.core" xmlns:mvc="sap.ui.core.mvc" xmlns="sap.m"
          controllerName="myController" displayBlock="true">
    <App>
        <Page title="Tittle">
            <Panel id="myOuterPanel">
                <Button text="Button"></Button>
            </Panel>
        </Page>
    </App>
</mvc:View>

Could you please fix the XSDs? Maybe you can also add a build step to validate the generated XSDs files.

The fact that the Eclipse tooling doesn't use the schemas has a different reason. At the point in time when we tried last (years ago, I must admit - Eclipse tooling is no longer on our roadmap) , Eclipse XML vaildation was just not able to handle our XSDs. When I remember it correctly, then the use of substitution groups caused the issue, esp. when they had been defined and used across multiple XSDs.
But what you experience, looks differently. I tried to validate the sap.ui.core.xsd with the W3C validator and it seems to me that the recent rework of source code documentation causes the problem (or at least one problem). Our JSDoc now contains more HTML tags and sometimes those tags are not wellformed. Documentation needs to be properly XML-escaped during generation of the XSDs, then the schema should be fine again.
I removed all /<\/?(b|i|code|li|ul|br)>/ from the sap.ui.core schema, then  the W3C validator was fine with it.
If you don't mind, could you try the same and let me know whether this would fix the VS 2017 issue that you encounter?
I'll take care to adapt the generators then.

One more thing: as explained in #1176

As schema generation is not a major delivery, it does not break the build.

Thank you for the quick response
I have investigated the problem some more. The problem appears only referencing "sap.ui.core.mvc". This post seems to be in a similar situation:
https://blogs.sap.com/2016/03/07/further-tweaking-of-jetbrains-webstorm-for-ui5-development/

The case is logged internally for a reference with #1870231492

In the meantime I created valid XSDs for some of the OpenUI5 libraries here:
https://github.com/emumanu/OpenUI5Sharp
They work with Visual Studio 2017

We discussed this again internally and won't invest any further in the XSD schema generation:

The mismatch between the current schemas and the development best practices for XMLViews (library namespace vs. package namespace) are too big to fix the issues in the existing generation
we have no capacity for implementing a totally new generation, but the necessary information should already be available externally (in the form of the api.json files)
experimenting with a different approach of tailoring the XSD namespaces along the runtime packages failed due to cyclic dependencies between packages
– WebIDE has anyhow chosen to use a non-XSD approach
@emumanu (thankfully) created a workaround for OpenUI5 at least

Sorry for not having a better update.


How to run beforeInitialPageLoad conditionally?

I can run the login logic in beforeInitialPageLoad but the session is maintained in the subsequent runs. Is there any way to either:

Run beforeInitiaPageLoad only when the session is not found plus how to check if the user already has a session.

OR

Clear the session so that beforeInitialPageLoad can work without interruption.

Thanks for this wonderful project. Loving it so far.

Hi @saxenarajat, please check out the answers below:

Run beforeInitiaPageLoad only when the session is not found

Could you explain a little bit about the use case for this? One thing to try is to invoke await page.goto(url) inside beforeInitiaPageLoad and see if it can establish a session (although the session would be different from the session created from url).

how to check if the user already has a session

One way is to check the user session in the action callback via the page argument.

Clear the session so that beforeInitialPageLoad can work without interruption.

beforeInitialPageLoad is the optional callback before establishing a session, not sure what interruption means in this context. Could you elaborate more on your use cases and what problem you are trying to solve?

Thanks @JacksonGL for replying,
So I am trying to test a protected route in my app. If a user is not signed in, she would be redirected to the login screen before she could access that route. I added the logic to log in (searching of email/password fields and filling in the details) in the beforeInitialPageLoad method and it works fine.
However, when I re-run the scenario subsequently, it breaks as the user already have a session and is not redirected to the login screen.

Thanks for sharing the context @saxenarajat
I would recommend using the cookies callback and void logging in if possible.
As an alternative for re-running the scenario, the beforeInitialPageLoad could use Puppeteer API s to check the url of the redirected page, if the user is not redirected to the login page, just skip the login logic.


Fix Build Process

[x] Write documentation about build process (Android NDK, RN fork, iOS, ...)

@sebgoa If you want to have a look, I added a bit of documentation on how to build the app, as well as some issues so that it's easier to track/discuss. It could be good to move that to another file/wiki.

/lgtm


Update README.md - Add a link to the Documentation

So its obvious that this project has an documentation.

Thank you


New Provider Bunny (bunny.net)

Hi,
I hope this is the correct way.
Here is the corresponding issue: https://github.com/libdns/libdns/issues/114

This looks like a mistake, it's just some newlines in the readme. The PR should contain the code so it can be reviewed (even though it won't be merged here).
That said, I found your code from a link in the other issue and gave it a quick once-over. It looks good AFAICT. A suggestion might be to properly encode a query string rather than doing string concatenation, but that's mainly it for now.
Thanks for contributing! I'll make a repo in libdns and caddy-dns for you. (caddy-dns will be for the Caddy plugin, if you are a Caddy user).


Pass deployment namespace as workload namespace

This commit add a new parameter namespace to workload model type, which
define the namespace of the deployment and should be used to distinguish
the workload on device-worker side.
Signed-off-by: Ondra Machacek<EMAIL_ADDRESS>

and an open question, what happens if a workload is deployed in a different namespace of the device?

Currently it's not deployed at all if I am not mistaken.


Active window switched after saving when there are more one Poedit instances

Environment: Windows 7 64-bits, Poedit version 1.8.8 (build 4339)
Steps to reproduce:

run first instance, open pofile-A
run second instance, open pofile-B
edit pofile-B until 100% translated, save
-> active window switched to pofile-A
Expected result:
-> active window stays at pofile-B

I have two files, both not 100%, which also trigger window switch on save.

both window must not in minimized state
saving gnome-software.gnome-3-20.id.po with switch to polari.gnome-3-20.id.po
saving polari.gnome-3-20.id.po will not switch window
Files attached.

save-switch-window.zip

This is completely ridiculous… and actually happening, I just confirmed with your files :( Thanks a lot for your report!


prepare for v0.3.0

can we tag 0.3.0 now? @luoxiu

Didn't update podspec before tag 0.3.0? @luoxiu


More flexible computed states

Objective

Currently, if a plugin author wants to define states that are meant to be computed from a user's own state, it will be impossible, because of the orphan rule.

Solution

This PR changes the computed state mechanism to use free functions that we register on the app, instead of a trait.

#[derive(Debug, Clone, Copy, Eq, PartialEq, Hash, States)]
#[computed]
struct InGame;

fn compute_in_game(sources: AppState) -> Option<InGame> {
    match sources {
        AppState::InGame { .. } => Some(InGame),
        _ => None,
    }
}

App::new()
    .add_state_computation(compute_in_game)

Testing

I tested these changes using the computed states test suite that I adapted to the changes.

TODO

[ ] I couldn't find SET_DEPENDENCY_DEPTH used anywhere 🤔. Its docs say it helps order transitions & prevent cyclical dependencies. For testing I tried manually creating a cyclical dependency and the only error I got was the ECS system scheduler Error when initializing schedule StateTransition: System dependencies contain cycle(s).
[ ] I didn't figure out how to make the derive attribute #[states(computed)] instead of just #[computed]

Changelog
Changed

Changed the computed states feature to use a free function for computation instead of a method on the ComputedStates trait.

Migration Guide

This section is optional. If there are no breaking changes, you can delete this section.

If this PR is a breaking change (relative to the last release of Bevy), describe how a user might need to migrate their code to support these changes
Simply adding new functionality is not a breaking change.
Fixing behavior that was definitely a bug, rather than a questionable design choice is not a breaking change.

FYI @lee-orr @MiniaczQ @benfrankel :)

You can check out how the derive macro in pyri_state works for reference: https://github.com/benfrankel/pyri_state/blob/main/derive/src/lib.rs.

Some preliminary questions:

Is the ComputedStates trait still needed after this change?
Could this approach be applied for SubStates as well?

Some preliminary questions:
1. Is the `ComputedStates` trait still needed after this change?

2. Could this approach be applied for `SubStates` as well?

I think the value is in preventing the user from calling AppExtStates::add_state_computation with a non-computed state (that implements FreelyMutableState). There's nothing actually preventing someone from implementing FreelyMutableState and ComputedState at the same time, but at least the derive macro will steer users toward not doing that.

I think that could work too. But I'm not sure how to preserve the nice macro attribute syntax? Would it create a regular method on the state, which you then pass into AppExtStates::add_sub_state? I think that might work, but I'm not sure if there's downsides.

#[derive(SubStates, Clone, PartialEq, Eq, Hash, Debug, Default)]
#[source(AppState = AppState::InGame)]
enum GamePhase {
    #[default]
    Setup,
    Battle,
    Conclusion
}

App::new()
    .add_sub_state(GamePhase::should_exist)

I didn't figure out how to make the derive attribute #[states(computed)] instead of just #[computed]

Actually I'm not sure what's best between #[computed] and #[states(computed)]. On one hand the latter feels more "scoped" but on the other hand, the substates attribute's syntax is #[source(AppState = AppState::InGame)] which doesn't have that prefix
So I'll probably leave it like that unless someone wants me to change it

I updated the documentation on the trait definition.
For the PR description, I'm not sure what should go in the Changelog & Migration sections. Should I describe the change as if updating from 0.13 to 0.14, or from 0.14.0-rc.3 to 0.14.0?

As if migrating from 0.14 to 0.15 please :) This won't be in 0.14, and migration guides should be written to cover the span between released Bevy versions.

I'm in favor of this feature (in fact I support the same thing in my own states crate). However, my preferred implementation would require some bigger changes to bevy_state.
There's a Discord thread documenting high-level proposals for bevy_state 0.15, so this feature may get in as part of that work but in a different form.


Add ListTransactions to Api with stub handler (returning [])

Issue Number
#465
Overview

[x] I have ListTransactions to the Api, and a stub handler which returns [].
[x] I have added a "Range" Header represented by a new Iso8601Range "inserted-at" type
[x] I fixed the HasPath test machinery to work with Headers

Comments

Iso8601Range and its doc comments might be a bit crude, but I think this is approximately how we want it.

@Anviking Thanks for adding this.
Would it be much extra work to also add the Range parameter? Even if this parameter is currently ignored, we'd still be able to immediately start connecting the CLI up to this endpoint.

string <inserted_at={range-start}-{range-end}>
A range of date in ISO-8601 UTC format without symbols to retrieve.
Note that the start joker symbol * can be used instead of a date to refer to a lower or upper boundary. > Example: inserted-at 20190227T160329Z-*

Might have to update my local jormungandr:integration setup, but think CI should pass now.


Unpublished contact flows interrupt the rest of the backup

Hi, thanks for making this tool.
I've encountered an issue when there is an unpublished contact flow. In the logs you get
2021/10/01 18:57:38 Failed to describe flow
when I ran to describe the contact flow manually, I found the reason was
An error occurred (ContactFlowNotPublishedException) when calling the DescribeContactFlow operation: Contact flow not published
Now when it hits this roadblock it doesn't backup the rest of the contact flows and continues to backing up users. Is it possible to skip all the unpublished contactflows and backup the other contact flows?

Should be a quick fix.  I'll output any errors describing the flow and rather than exit, just continue with the loop

Fixed in v0.0.24

Thanks! Do you mind fixing the lambda too? I should've specified that was what I was using.

@hatchcanon it's the exact same code.  Did you make the lambda and redeploy it?

I apologize, I just tested again and turns out I was using the old zip. Thank you again


Implement maintenance mode

Prevent uploader from marking all replays as upload error when website is under maintenance. Add maintenance=true to website response and implement waiting on client.

Is this implemented on the serverside yet? The http way would be to respond with 503 -- service unavailable, and a retry-after header set.  Then the client could wait until, well, after.

I don't think so. Laravel has artisan down command, we probably need to check what response it returns on all requests and use this as a signal for maintenance.

https://laracasts.com/discuss/channels/laravel/artisan-down-change-view tells me
See https://laravel.com/docs/5.6/errors#custom-http-error-pages

artisan down returns a 503 error so you should create the following file: resources/views/errors/503.blade.php

That it's already a 503 is good, we can detect that. retry-after headers are nice, we can deal with that when we get there.


Added some favicon links from the KA webpage.

Thanks for the PR. I don't think we want to do this for now.


test: add data acceptance tests

Description
Adds data acceptance tests
Related Issue(s)

closes #863

Documentation
delta-incubator/dat

Thanks for the pull request! DAT should not be added directly to the tree, since they should be changing relatively frequently as the protocol evolves. Here's a build.rs from another repository which I think you could incorporate into your pull request to dynamically fetch the latest dat release

Thanks @rtyler, I will fix it up prior to requesting review! I just left it there because I hadn't done the setup part yet and wanted to make it easy for others to take a look at.
Should the build.rs drop the files on exit? or is the intention to leave them in the tree, but just add the data in an automated way?

Should the build.rs drop the files on exit? or is the intention to leave them in the tree, but just add the data in an automated way?

I don't believe there is a need to drop the dat tests in an automated way, I think it's best to avoid re-downloading the test release unless necessary.

@rtyler
I have updated the tests with your suggestion. I am encountering an issue on two of the unit tests
running 7 tests
test test_with_schema_change ... ok
test test_with_checkpoint ... ok
test test_stats_as_struct ... ok
test test_no_replay ... ok
test test_basic_partitioned ... FAILED
test test_basic_append ... ok
test test_all_primitive_types ... FAILED

failures:

---- test_basic_partitioned stdout ----
Error: External(External(External(ArrowError(InvalidArgumentError("Column 'CAST(actual.letter AS Utf8)' is declared as non-nullable but contains null values")))))

This one is unexpected.
---- test_all_primitive_types stdout ----
Error: External(Execution("Fail to build join indices in HashJoinExec, error:Arrow error: Cast error: Cannot compare two arrays of different types (Timestamp(Nanosecond, None) and Timestamp(Microsecond, None))"))

This one is expected. What is the simplest way to coerce types?

Let me look through this one


add attachment based on other columns

First, i want to search the data in AttachAction outside the recordTitleAttribute, for example, the recordTitleAttribute is 'fullname' and i want to search by 'email' and 'identity_number'.

Thanks


chore: Select component refactoring - FilterControl - Iteration 5

SUMMARY
It replaces the react-select Select component with the Antdesign one in the FilterControl components.
BEFORE/AFTER
https://user-images.githubusercontent.com/60598000/126142095-30fa08a9-beb2-4098-95f0-753f2fcbd64c.mp4
https://user-images.githubusercontent.com/60598000/126144838-20b68e04-2bb4-4acb-9f44-27e2dc8712cf.mp4
BEFORE/AFTER
https://user-images.githubusercontent.com/60598000/126142234-93cb7da6-da2e-498b-9118-9e705402bd1f.mp4
https://user-images.githubusercontent.com/60598000/126144743-79a2adcb-859b-4a97-913b-3b8b0e98017b.mp4
TESTING INSTRUCTIONS

Open a chart in Explore
Add Filters using the SIMPLE tab options and observe the behavior
Add Filters using the CUSTOM SQL options and observe the behavior

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@rosemarie-chiu @jinghua-qa

/testenv up

/testenv up

/testenv up

I think we need to clear the filter value when changing the column.
https://user-images.githubusercontent.com/70410625/133511351-cb519c45-b126-493e-9c56-d24ee0296b3f.mov

The code lgtm. But in regards to clearing the fields like @michael-s-molina suggested would be nice but to be fair it doesn't currently do that on master so I think that could be done as a follow up as that might require a little more refactoring work?

I think we need to clear the filter value when changing the column.
Screen.Recording.2021-09-15.at.6.19.33.PM.mov

Just tested on master. As @pkdotson reported this is currently not implemented. I think it makes sense to tackle that in a separate PR as it would be slightly unrelated to the objective of this PR. I'll work on that as soon as we merge this PR.

I think we need to clear the filter value when changing the column.

@michael-s-molina Just tested on master. As @pkdotson reported this is currently not implemented. I think it makes sense to tackle that in a separate PR as it would be slightly unrelated to the objective of this PR. I'll work on that as soon as we merge this PR.

Ok. Approving with the follow-up in mind.


Enable support for debugging binaries compiled from .c files.

Fixes #1532
The immediate reported problem relates to setting breakpoints. Please see #1532 for details on suspected root cause.
The strategy to fix this will be as follows:

Identify the DW_AT_language attribute of the source file. We do this at source file level, and not at the binary level, so that binaries that have a mix of C and Rust should be covered also.

NOTE: At this point, it is not clear if we should check for:

DW_AT_language == DW_LANG_C11(and later), OR
DW_AT_producer == starts_with("GNU C") or some variation thereof.

For Rust files, continue to employ the current strategy of identifying the location of valid halt locations, using DW_LNS_set_prologue_end attribute.
For C files, because this attribute is not emitted by gcc, we will employ the same 'heuristic' as gdb.

See this discussion for details on the gcc heuristic logic.

At this point, even if we get breakpoints to work, it will require additional work to unwind variable values, as we currently do not implement all the possible DWARF types and attributes, but rather just the ones that are used by Rust.
cc: @TechnoMancer, @mubes

@mubes. This branch now sets breakpoints, and allows stepping through your C code.   You can install it for testing, with cargo install --force --git https://github.com/probe-rs/probe-rs --branch noppej/issue1532 probe-rs-debugger
It correctly unwinds the stack frames and registers when it halts, but as I suspected, the unwind of C variable values is not well supported. If you confirm the breakpoints and stack frame unwind, then I will suggest we log the variable value issues for separate PR's.

I have tested this against the repository available at https://github.com/mubes/orbmule which is for a STM32F427 class CPU and includes build artefacts to avoid the need to install a toolchain.  For me it looks very promising and is a big step forward in addressing #1532 .


feat(website): Introduction to textlint

This PR is not yet perfect!
Please review the rules I have added and if there are no problems, I will fix the errors and push it again!
Summary

Related comments: https://github.com/biomejs/biome/issues/880#issuecomment-1831826077

Other OSS translation projects and the textlint rules used in them

Next.js

textlint-rule-preset-ja-technical-writing
textlint-rule-max-comma
textlint-rule-preset-ja-spacing

sveltekit & svelte-site-jp

textlint-rule-detect-bad-chars
textlint-rule-no-mix-dearu-desumasu
textlint-rule-preset-jtf-style

React

textlint-rule-preset-jtf-style

Vue

textlint-rule-detect-bad-chars
textlint-rule-no-mix-dearu-desumasu
textlint-rule-preset-jtf-style

Vite

textlint-rule-preset-jtf-style

I have installed textlint-rule-preset-jtf-style which is the most used of them all!
Test Plan

CI is successful.
No problem with the rules you set.

@yossydev After resolving this
https://github.com/biomejs/biome/pull/986#discussion_r1413934301 it would be good to me.

@nissy-dev please merge this after checking your requested changes🙂


zos: completely functional

The rest of the zOS implementation. All functional test cases pass with this.

Left a few comments but I don't have more time now. Will get back to this as soon as I can.  Please take a look at avoiding epoll emulation.

Also, thanks! We're closer to being Enterprise Ready.

Thanks! I will have a look at avoiding epoll emulation.
Also will wait for what the rest of collaborators have to say about mkdtemp.

Another thing I want to bring up is that on zOS, we have asyncio available only for INET sockets (tcp,udp)
in the form of the posix aio api.
Event notifications will occur via posix message queues
For UNIX sockets, we will stick with regular polling.
Any design recommendations would be most welcome.

Left another round of comments.

Another thing I want to bring up is that on zOS, we have asyncio available only for INET sockets (tcp,udp)
in the form of the posix aio api. With this, event notifications will occur via posix message queues
For UNIX sockets, we will stick with regular polling.
The poll call on zOS can poll an array of sockets(UNIX) as well as the message queue for notifications. (INET).

I see.  Since my experience with those is NULL, I defer to you.  Now, I do think that the code would be simpler if you don't try to emulate epoll.  Is there any spcecific reason you went that route?  Regarding asyncio: id not used ATM either, AFAIS, so is is reasonable to just user poll() now (avoiding the epoll emulation) and then add the asynctio support? If you need to store extra structures, the loop has some reserved fields for emergencies ;-)

I expected zos runtime developers might implement epoll in the future so thats why I emulated it for now. Later on integration with epoll in the runtime would be easier. But I am ok with removing epoll to make the code simpler for now. Also, I will leave the asyncio for the next release. For this PR, I will simply use poll for everything.

I expected zos runtime developers might implement epoll in the future so thats why I emulated it for now. Later on, integration with epoll in the runtime would be easier. But I am ok with removing epoll to make the code simpler for now. Also, I will leave the asyncio for the next release. For this PR, I will simply use poll for everything.

I wouldn't want you to do double the work, so if z/OS getting epoll is a reasonable expectation I guess we can live with the emulation.
Another possibility is to dump it in v1.x and switch the real z/OS epoll for v2, should it be ready.
PS: You probably need to rebase at this point.

@jBarz how is this coming along?

@saghul Took a break :-) Looking to finish this over the weekend. Aiming for Monday.

@jBarz I hope you enjoyed your break!  Good to hear, thanks!

@jBarz Any chance you can get back at it this week? After that I'll be off for a while and I'd rather have the release before that.  If you can't that's ok, we can land it later on.

@saghul Sorry about the delay. I have made all changes in response to comments.

@jBarz I still see the epoll bits, are we going for that then?

Yea, for now if that is OK.
When epoll does arrive on z/OS soon, it will be an easier transition in v2.x.

The global epoll lock is a bit tricky but will finish that early tomorrow

The global epoll lock is a bit tricky but will finish that early tomorrow

You can use a uv_once to initialize it and call it every time a loop is initialized, since it will only be called once.

Sorry, got pulled away by other issues for a while. Will make the requested changes by today.

@saghul Thanks for your suggestions. I have addressed the issues with the epoll implementation. I also fixed the code style issues.

rebased

Squashed all commits and fixed nits.
@saghul: Thanks for your help in bringing libuv to the enterprise :-)

@jBarz the commit msg body all made sense to me, other than * ancillary data, what does that mean? It lacks a verb, I think.
 Maybe I misunderstand, but the functional tests passing is just the evidence for what really happened: zos completing.

Landed in ✨  011e02e ✨ , thanks for working with us on getting in on shape! 🍰

@sam-github Doh, I was merging the commit and I missed your message :-S  I did read your mind and adapted the commit message though!

@sam-github: I should have squashed these 2 lines. The second line is not required. :-(

read ancillary data that remains on queue
ancillary data


Add the "Identity" hash function

sometimes the values to hash are smaller than a cryptographic hash digest, and merits just actually storing the data itself as the hash value. (Example use case: a multihash of an ed25519 256bit pub key [0]).
Implementation:

add a code for the "identity" hash function. (Could use 0x10 and don't add sha0, or 0x50)

Pros:

can store ECC pub keys easier and use them in projects that use multihashes as IDs (IPFS)
can store values smaller than the hash ("a type")

Cons:

this worsens the "keep things uniform" expectation people have from hashes

[0]:

"On the other hand, elliptic-curve points are typically sent as 256 bits, and then a 256-bit hash doesn't save any space. Sending R instead of H(R,M) also has the advantage of allowing batch signature verification, which is typically twice as fast as verifying one signature at a time. That's why Ed25519 doesn't use this compression." -- from http://blog.cr.yp.to/20140323-ecdsa.html

@robcat had the same idea and reminded me of this today-- we should add this.

particularly useful:

 for example, in the semantic web (addressing a concept by keyword, no lookup required)

Some example usecases here would be nice to see.

@whyrusleeping A typical example would be the representation of JSON-LD objects. Most of the values will probably be shorter than 256 B.

Let's say that 0x00 is the Identity Hash Function's code in the standard table.
Let's say that the Identity Hash Function is: (x) -> x

// example in Go
func IdentityHash(x []byte) []byte { return x }

// multihash(0x00, "foo") => 0x0003666f6f => ("identity", 3, "foo")

Added in #40
I will give a week before declaring this part of the spec.

@jbenet
I would like the spec to address the following two points:

Maximum length of the data: are implementations supposed to handle unbounded variable-size data?
Encoding of the data: in your example the value 0x0003666f6f gets automatically decoded as a string, does that imply that UTF-8 is the default encoding of the data?

@robcat

a maximum would be useful. not sure what to choose though that wont be obsoleted. This could be an implementation parameter? not sure, it's bad from a security perspective not to leave this well-defined.
sorry, my example didn't mean to decode/encode utf8. i meant it as just byte buffers.

Maximum length of the data
maybe data is length prefixed and then clients set sane max ~1 Mbyte

[varint length-of-data][data]

I came up with a few use cases while writing the various Ethereum ipld format resolvers.
The Ethereum StateTrie leaf nodes each contain a binary blob for an Ethereum Account.
In order to provide all paths off of a StateTrie node, currently the ipld format resolver needs to include the logic for Ethereum Accounts to provide the paths off of the trieNode and off the Account.
The resolver implementations would be much simpler if they were able to say "here is path to an account, and here is its data, use the account resolver to resolve that account" which it could do by providing a cid that indicated the type (eth-account) and the data inline via the identity-hash.

This is basically the data: uri of our time
https://tools.ietf.org/html/rfc2397

It was implemented and uses literal 0 as code ID.

ok great here is a PR to add it to the multicodec table https://github.com/multiformats/multicodec/pull/30
@Kubuxu is there a codec name you are using in the go impl?

Sorry, I messed up, we added Identity multibase encoding, I don't know how about the multihash and so on.

@Kubuxu ok removed the PR. What is the "Identity multibase encoding" ?

It is just binary pass through for cases when rest of the stream/data is binary and encoding isn't needed.
As multihash was merged with multicodec I don't think there is need for Identity encoding anymore, it would be saying "here is some data, but now you have no idea what it is". We should reevaluate the usecases.

(long since fixed)


vcpkg does not ignore $-prefixed keys in features object

Describe the bug
I tried to introduce comment inside of the "features" field of package's vcpkg.json. This causes error:
Errors occurred while parsing C:\ubuntu\vcpkg\../vcpkg-registry/overlay-ports\repro\vcpkg.json
    $.features (a set of features): unexpected field '$possible-features': must be lowercase alphanumeric+hyphens and not reserved
Error: Failed to load port repro from C:\ubuntu\vcpkg\../vcpkg-registry/overlay-ports

Environment

vcpkg: Vcpkg package management program version 2021-12-09-724cea8b33cbf06c645f5095fa29773697da9761

To Reproduce
Create package repro with this vcpkg.json:
{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg/master/scripts/vcpkg.schema.json",
  "name": "repro",
  "version": "0.0.0",
  "features": {
    "$possible-features": "Some comment or something",
    "fake": {
      "description": "fake feature"
    }
  }
}

vcpkg install repro --overlay-ports={path to port repro}

Expected behavior
$-prefixed field is ignored as it would be elsewhere.

cc @strega-nil-ms

Needs a release.


go/printer: produces a invalid source, for a successfully parsed AST

Consider:
type _[a (b + 3),] struct{}

This is parsed successfully by go/parser and it gets formatted into:
type _[a b + 3] struct{}

This is not valid go syntax. Not sure how this should be handled, should type _[a (b + 3),] struct{} cause a parser error?
For example, the source below causes a parser error (expected ')', found '+')
type _[a ~int, b (a+3)]struct{}

CC @griesemer

Milestone 1.24 only so it stays on the radar - this is not really urgent in any way as this code won't pass the typecheckers and thus won't appear in real code.

Looking at this some more, type _[a (b + 3),] struct{} should probably be a parser error, but there's two ways to approach it: since (b + 3) cannot be a type, the entire parameter list is considered an array length, and the , is unexpected. Alternatively, the comma implies a type parameter list, but (b + 3) cannot be a type so there's an error there.
I believe the spec's current wording permits both approaches.

Looks like the comma causes it to be parsed as a type parameter list, even though the parenexpr holds a term not a type:

Yes, the comma forces parsing as a type parameter list which is expected in the current implementation. The question is what is the most consistent error to report: is it an invalid type constraint or an invalid array length.

The parsers accept a larger language than what is syntactically permitted, both to simplify the implementation and make it more efficient, and also to be more error-tolerant. We guarantee that code is correct after type-checking. This specific issue may not be the only one where printing produces syntactically invalid code after parsing.
Certainly would be nice to fix but also not urgent or important. Marking as backlog.


Since update phpsaml 1.3.0 : loop with Plugin Enforced option

Hello,
I'm new to posting but I've always managed to find what I'm looking for in discussion forums. But this time I can't find what I'm looking for, so I've come up with this problem.
Loop during microsoftonline login.
With phpsaml version 1.2.1 : it is functional with version 10.0.10 of GLPI
On the other hand, if I wish to pass in 10.0.11 it is necessary that I pass in version 1.3.0 to correct the blocking of the white page.
But when I upgrade to version 1.3.0, whether it's GLPI version 10.0.10 or even the new version 10.0.11, I get a Token loop every second :
URL visible from the web browser :
https://login.microsoftonline.com/**ID of the resource tenant**/saml2?SAMLRequest=Token1
https://login.microsoftonline.com/**ID of the resource tenant*/saml2?SAMLRequest=Token2
https://login.microsoftonline.com/**ID of the resource tenant*/saml2?SAMLRequest=*Token3
...
In the user connection logs on our Tenant 365, we find our access attempts with the status "Operation successful" for the published application but every second we have a new one, then a new one, then ....
When we indicate No in the Plugin Enforced section and on the home page we click on the phpSaml button, it works.
So it's when we set the auto login to mandatory that it doesn't work any more and we end up with a page that runs in a loop with a series of tokens that change every second in the URL link.
Thanks in advance for your help and feedback on this subject.
Have a nice day!
Rémy

Hi @REMY31470
Thanks for your feedback. This is an issue also reported in another thread. I have been debugging this and found the issue. I will push the update this evening.
Rgrds,

I found an issue in the excludes logic and corrected it.

Hi @DonutsNL ,
Thank you very much, the auto login is working correctly.
And thank you for your quick action.
Rgrds,

Your most welcome. Thanks for reporting the issue and helping us improving the plugin.


Mention default output node name for AnimationNodeBlendTree

Fixes #59378
Also turned node types mentioned in the description into links.

Thanks!

Cherry-picked for 3.5.


Parameters argument in HTTP get requests

Originally suggested by @RX14 in a separate PR.
Currently, parameters need to be added to an HTTP::Client#get call with the HTTP::Params#encode method. A much more user-friendly option would be something like this:
HTTP::Client.get("http://www.example.com", query_params: {"foo" => "bar"})

Please, lets not create an issue for every idea anyone has in comments.
Can we start using Google Groups to discuss these things? There's no point in cluttering the repo with issues like these, specially when they are not a problem (one can achieve the same thing with different code).

I don't use Google Groups, so that's not an option. Where should I bring it up to discuss it with the core team?

Create a Google Groups account? One could also say "I don't use GitHub, where can I submit issues?"

I have no desire to use Google Groups, or any other Google product.

There's the Gitter chat, is that an ok place to talk about suggestions for the library?

@asterite the counter-argument is that it's sensible to keep all suggestions in the same place, where they're easy to manage. Google groups isn't an issue tracker, and it's fairly easy to just keep these kind of issues around on github.


Challenge 87: Create a Custom Heading

This is the challenge where you move the cat in a green circle up to the same line as the "CatPhotoApp" heading.
The first time I did the challenge, I wrote this:

CatPhotoApp

 …

The heading and cat pic are still in the topmost div, which squishes them together like this:

This was accepted and said the challenge was completed correctly. The objects were only oriented correctly when the heading and picture were placed outside of the main div like this:

CatPhotoApp

 …

(I am using Safari v8.0.7)

Update:
This squishing is happening on all subsequent challenges too. For example:

There appears to be several issues going on:

The link at the top of the editor does not use https so the console yells at you if you do take a peak.

Changing the above will change the font and the top will look normal/expected.

The second image link does not use https as well so the console will yell at you for that too.

Looks like the title stuff has already been fixed but the fixes haven't been pushed to the live site yet. But the smushing thing might be part of a larger problem. Closing this issue for the heading issue. Happy coding!

just a matter of a closing tag