Find a way to remove bps.sleep(0.1) from the plan

https://github.com/bluesky/bluesky/pull/1596 is one path that may help.
In the first case eventually it should be the shutter and motor's job to report when they are done so in the plan you should do:
    yield from bps.mv(detector.shutter, "open", detector.z_motor, target)

Part of the assumption of blueky is that when it is running it controls the beamline so having a hybrid GDA/bluesky system is likely to have some rough edges.

Possibly the ophyd.v2.core.wait_for_value async method could be converted to return a Status, then we could pass this to bps.wait_for

Closed by https://github.com/bluesky/bluesky/pull/1598

I do not see where bps.sleep() was removed in https://github.com/bluesky/bluesky/pull/1598.

the bps.sleep is in the test plan in this repo, rather than in bluesky. It enables the following:
https://github.com/bluesky/ophyd-epics-devices/blob/c5efe8b5ee9adca2ba6dc0be9bb2844070c4cceb/tests/test_area_detector.py#L166-L174
To be written as per https://github.com/bluesky/bluesky/issues/1597:
for det in dets:
    yield from bps.complete(det, wait=False, group="complete")
done = False
while not done:
    try:
        yield from bps.wait(group="complete", timeout=FLUSH_PERIOD)
    except TimeoutError:
        pass
    else:
        done = True
        for det in dets:
            yield from bps.collect(det, stream=True, return_payload=False)

Thanks for describing that!


Fix Edited Indicator Behavior

Description
The edited indicator behavior now properly updates and aligns with Xcode's implementation.
The autosave() method provided by NSDocument automatically calls updateChangeCount accordingly. The else block in the function executed regardless of whether autosaving occurred or was enabled, triggering an unintended cleared state. As such, removing the direct call to updateChangeCount(.changeCleared) resolved the issue as the change count state is now handled properly by the autosave() method.

Related Issues

#1898

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] The issues this PR addresses are related to each other
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] My changes are all related to the related issue above
[x] I documented my code

Screenshots

With Manual Saving
https://github.com/user-attachments/assets/35f3f435-69d7-46a8-ba00-b79269ad03a7
With Auto Save
https://github.com/user-attachments/assets/5ec5bde3-82f6-4f5c-b4d0-17439e4af85b

Just added a comment 👍


Add DialogStudio

add dialogstudio: summary: licenses, URLs, titles

@shayne-longpre
fix summary file according to comments
add download and prepare function


Basic project structure for performance testing app

Part of #261
Adds the basic project structure for a performance testing app. I'm looking for feedback on this before I start writing the actual performance testing code.
I decided to extend cesium.omniverse.app.kit rather than create a completely new app. In the future I might explore omni.kit.quicklayout and creating a custom UI, but for now I think it's best to keep it simple.
The Python code creates a "Cesium Performance Testing" window similar to cesium.powertools.
One thing I'm not sure about is whether the cesium.performance extension should live in the cesium-omniverse/app folder like it is now, or if it should live in the cesium-omniverse/exts folder with the other extensions. Any thoughts on that @weegeekps?

One thing I'm not sure about is whether the cesium.performance extension should live in the cesium-omniverse/app folder like it is now, or if it should live in the cesium-omniverse/exts folder with the other extensions.

@weegeekps thoughts on this question? I can't decide which approach I prefer.

@lilleyse I like it living in the apps folder. Prevents any confusion about what it's for.
Thinking about it though, you may want to rename the module cesium.omniverse.app.performance just to further clarify that it's an app extension.

Good idea. I ended up renaming the app to cesium.performance.kit and the extension to cesium.performance.app. This seems to be consistent with the Code and Create naming convention (see below).
I tried renaming the app to cesium.omniverse.performance.kit and the extension to cesium.omniverse.performance.app but it couldn't load due to module conflicts with the cesium.omniverse extension. I'm not sure if prefixing Python-based apps with cesium.omniverse will work.
I also think we should remove the word app from the other .kit files, e.g. rename cesium.omniverse.app.kit to cesium.omniverse.dev.kit or something, but I didn't want to do it in this PR.

App
(\w+)\.(\w+)\.kit
E.g.

cesium.performance.kit

App extension
(\w+)\.(\w+)\.app(\.(\w+))?
E.g.

cesium.performance.app

... or if there are sub extensions

cesium.performance.app.setup
cesium.performance.app.resources

@weegeekps Follow up PR here: https://github.com/CesiumGS/cesium-omniverse/pull/357


Smart Contract Not Compatible with IBM Blockchain Platform 2.5 using Fabric 2.x Images

The smart contract uses an older shim level that is not supported with Hyperledger Fabric 2.x images.
Fabric runtimes that use Fabric 2.x images have Node version 12 instead of Node version 8 which is used in Fabric 1.4.x runtimes.
The smart contract fabric-contract-api & fabric-shim node modules must be at v1.4.5 or greater to work with Fabric 2.x images.
This is from the smart contract package.json:
"dependencies": {
"fabric-contract-api": "1.4.1",
"fabric-shim": "^1.4.1",
"mem": "^5.1.1",
"os-locale": "^4.0.0"
},
This should be updated to:
"dependencies": {
"fabric-contract-api": "1.4.5",
"fabric-shim": "^1.4.5",
"mem": "^5.1.1",
"os-locale": "^4.0.0"
},

@celder628 if you already have this working on your computer, could you make this PR?

From what I can see, there are quite a few of these samples/demos.  I cannot be responsible for the updates and retests needed for them all.

No - wasn't asking for all. Just this one - if you had a working sample. No problem. I will try and update it when I get a chance.


Different versions of mmdet yields different results

I have a mmdet V2.3.0 in my server and another mmdet V2.11.0. However, all with bs=32 per GPU (I use two Nvidia V100s), size 640*640, their results are different, and gap nearly 1.5AP. Is there any problem with different versions? Below are my results of retinanet.
2021-04-19 07:48:19,950 - mmdet - INFO - Exp name: retinanet_r50_fpn_1x_coco.py 2021-04-19 07:48:19,951 - mmdet - INFO - Epoch(val) [12][1833]  bbox_mAP: 0.3290, bbox_mAP_50: 0.5110, bbox_mAP_75: 0.3450, bbox_mAP_s: 0.1400, bbox_mAP_m: 0.3720, bbox_mAP_l: 0.4760, bbox_mAP_copypaste: 0.329 0.511 0.345 0.140 0.372 0.476 (mmdet)
2021-04-21 10:33:21,727 - mmdet - INFO - Epoch(val) [12][1833]  bbox_mAP: 0.3140, bbox_mAP_50: 0.4920, bbox_mAP_75: 0.3310, bbox_mAP_s: 0.1320, bbox_mAP_m: 0.3590, bbox_mAP_l: 0.4490, bbox_mAP_copypaste: 0.314 0.492 0.331 0.132 0.359 0.449

@yarkable Do you have any questions now?


DagRun scheduled after end_date

Hello
Environment
Please fill out any appropriate fields:

Airflow version: v1.7.0
Airflow components: webserver and scheduler with a postgres database and LocalExecutor
Python Version: 2.7.9

Description of Issue
When I write a new DAG, it is automatically picked by the scheduler as intended. However the end_date parameter is not accounted when creating a DagRun, so that DagRuns scheduled after end_date are being displayed  in the web interface  and are staying in the running state forever as the scheduler doesn't launch them.
From my reading of the sources, the end_date paremeter is never used.
Is there any reason for this behaviour ?
Regards
dud

@dud225 Please use Apache Jira for this as requested. In addition the the line just above what you highlighted considers end_date so your analysis does not seem entirely correct.
Closing.


fix: add redirect when policy connection is enabled

Details

https://github.com/Expensify/App/issues/42789
Fixed Issues

$ https://github.com/Expensify/App/issues/42789
PROPOSAL: N/A
Tests

Open the app and log in
Create a new workspace
Navigate to the workspace settings
Open More features
Enable Accounting
On smaller screens, verify that user is returned to workspace settings, the Accounting feature is highlighted

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

Offline tests

N/A
QA Steps

Open the app and log in
Create a new workspace
Navigate to the workspace settings
Open More features
Enable Accounting
On smaller screens, verify that user is returned to workspace settings, the Accounting feature is highlighted

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

PR Author Checklist

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/30054992/3cef9ff6-ac2d-48c4-8aad-c98817d56ba1

Android: mWeb Chrome

https://github.com/Expensify/App/assets/30054992/f0034aef-8e3a-48f6-9f74-22bebd6f3ddf

iOS: Native

Getting iOS build issues

iOS: mWeb Safari

https://github.com/Expensify/App/assets/30054992/7321d27c-9787-40e2-945d-1b6e92d3375c

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/30054992/d3770901-5ce8-480f-bf4b-8b92fbaec3e4

MacOS: Desktop

https://github.com/Expensify/App/assets/30054992/691b3368-ad9d-4252-87c7-4eb3ddc6b803

@sobitneupane Will be reviewing this @allgandalf.

I don't think so. According to my experience, any C+ approval triggers internal engineer assignment.

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

🚀 Deployed to production by https://github.com/luacmartins in version: 1.4.81-11 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Combine getTouchingWord and getTouchingPropertyName

getTouchingPropertyName is just getTouchingWord, except it allows us to be exactly at the end of a StringLiteral, while getTouchingWord only allows us to be exactly at the end of an Identifier, and requires us to be at least one character before the end of a string literal. (They were both added in #752.) I don't think there are any cases where this difference actually matters, so we can just use getTouchingPropertyName consistently, which allows some code to be abstracted in findAllReferences which previously used one function in some places and the other function in others.

@mhegazy @sheetalkamat @armanio123 Could someone review?


[BEAM-7390] Add code snippet for GroupIntoBatches

Adding the code snippets for GroupIntoBatches.
R: @chamikaramj

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

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

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

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

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

@aaltay, I have some questions in this one:

What is the difference between GroupIntoBatches and BatchElements?
Which is the recommended approach?
I found that BatchElements was both more intuitive and flexible since it doesn't require a key, should we get rid of the GroupIntoBatches example in favor of BatchElements or is there a scenario where GroupIntoBatches make more sense?

Hi @chamikaramj and @raheelkhan, this code snippet will be displayed in the docs. I have some questions:

What is the difference between GroupIntoBatches and BatchElements?
Which is the recommended approach?
I found that BatchElements was both more intuitive and flexible since it doesn't require a key, should we get rid of the GroupIntoBatches example in favor of BatchElements or is there a scenario where GroupIntoBatches make more sense?

See the discussion on https://github.com/apache/beam/pull/8914 for differences between the two transforms.

@aaltay, from your comment and the rest of the discussion I'm gathering the following:
BatchElements

Does not use state and timer APIs (more portable across runners that don't support them)
Does not support streaming (is this correct?)
Can use keyed or un-keyed elements
Supports batch auto-sizing
Does not match any Java transform

GroupIntoBatches

Uses state and timer APIs (can't be used in some runners)
Supports streaming
Must use keyed elements
Fixed-sized batches only
Matches the Java transform

Are these observations correct? Am I missing anything?
If so, it might be worthwhile to have samples for both transforms and have a note on these differences in the docs, as well as a link to BEAM-6696 for more details.
I noticed that GroupIntoBatches requires keyed inputs, but I couldn't figure out what the keys are used for. I used None keys in the sample, but it seems cumbersome to have to add keys to use this transform.

@aaltay friendly ping :)

I think we would like to converge on one of the transforms. Probably to GroupIntoBatches once state is supported in all runners. Not sure if it makes sense to have snippets for both for now. It is a bit confusing and we might not want to advertise it.

I can remove the sample for BatchElements then, but do you at least know what the key in GroupIntoBatches is used for? I couldn't find any use, but it's still required so I used None keys.

I can remove the sample for BatchElements then, but do you at least know what the key in GroupIntoBatches is used for? I couldn't find any use, but it's still required so I used None keys.

State and timer are keyed. This pardo will keep batches per-key, and release bundles per key. Each batch seemingly will have values from the same key.

I will get rid of BatchElements and keep GroupIntoBatches. I will also update the sample to use keys as well.

@aaltay sorry for the delay on this, this should be ready for review.

retest this please

Run Python PreCommit

Run Python PreCommit

Run Python PreCommit

Last 3 test runs failed, could you check the logs?

It looks like the test was not deterministic under certain environments, especifically Python 2 and Cloud tests. I changed it so it only checks the size of the batches, but not the actual elements.

retest this please

retest this please

retest this please

retest this please

retest this please

Sorry, forgot to run the linter. I changed the nested function to a lambda and yapfed over the files so that shouldn't be an issue anymore.

retest this please

retest this please

retest this please

Do you know why tests are failing?

retest this please

I have no idea, there are a bunch of unrelated errors like ERROR:apache_beam.runners.worker.data_plane:Failed to read inputs in the data plane., as well as some errors with the flink runner.
I'll try refreshing to the latest version.

Tests passed, merging this.


WIP: Profiles

I'm unable to try it because rustup depends on too old openssl crates but does this PR allow removing already installed components (rust-docs)?

does this PR allow removing already installed components

It doesn't work yet, but it will


Build issue with my project.

I am trying to build my project currently but the following exception is being thrown:
> Could not copy file 'C:\Users\nicky\Desktop\fabric-example-mod-master\Grisha\src\main\resources\fabric.mod.json' to 'C:\Users\nicky\Desktop\fabric-example-mod-master\Grisha\build\resources\main\fabric.mod.json'.
   > Failed to parse template script (your template may contain an error or be trying to use expressions not currently supported): startup failed:
     SimpleTemplateScript6.groovy: 1: Unexpected input: '(' @ line 1, column 10.
        out.print("""{
                 ^

     1 error

Help would be greatly appreciated as I am currently under heavy time crunch and struggling >.<

Thank you for your interest in Gradle!
Your issue appears to be a question about how to use Gradle. Your question might be better answered on our forums or StackOverflow. We have several resources on help.gradle.org that you should check out first.
If you think you have come across a Gradle bug, please file a new issue describing the problem and how to reproduce it.
If you think Gradle is missing a feature, please file a new issue describing your use case for the feature.

@mlopatkin not really helpful. He should have provided an example to reproduce the error. But aside from that it's actually a gradle issue not a question how to use gradle. Please reopen this ticket or I will create a new one.
Got the same error:
Failed to parse template script (your template may contain an error or be trying to use expressions not currently supported): startup failed:
     SimpleTemplateScript34.groovy: 1: Unexpected character: '"' @ line 1, column 13.
        out.print("""

I created a minimal example to reproduce the issue.
In my case it seems gradle kotlin dsl doesn't handle comments inside property files.
Either commenting out this in build.gradle.kts:
tasks.withType<ProcessResources> {
	filesMatching("*.properties") {
		expand("version" to version, "buildTimestamp" to LocalDateTime.now().format(DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm")))
	}
}

Or removing this in application.properties:
#  \ \\\\\\\\n \\\\) test %m%n

Fixes the build.
See minimal reproducable example added:
demo.zip
logs.txt

@judos thanks, putting this back into the triage queue.

I've run into similar issue. Couldn't really find any obvious reason. Adding escapeBackslash = true to expand helped.
Example:
filesMatching('**/config*.*') {
    expand('version': project.version) {
        escapeBackslash = true
    }
}

This is affecting me too. I can confirm that it works correctly in v6.9.4, but fails in v7.0.0 (and the latest versions 7.6.1 and v8.0.1)

This seems to happen when there are other unescaped placeholder variables in the yml file.
tasks.processResources {
    filesMatching("application.yml") {
        expand("profile" to profile)
    }
}

# works fine
spring:
    profiles.active: ${profile}
    batch.job.name: \${job.name:NONE}

# error occurred
spring:
    profiles.active: ${profile}
    batch.job.name: ${job.name:NONE}
---
> Failed to parse template script (your template may contain an error or be trying to use expressions not currently supported): startup failed:
     SimpleTemplateScript23.groovy: 1: Unexpected input: '(' @ line 1, column 10.
        out.print("""spring:

tested on gradle 7.6

Can confirm! Gradle 8.4, I'm expanding namespace, package, and version.

The expanded file
```json
{
	"schema_version": 1,
	"quilt_loader": {
		"group": "${package}",
		"id": "${namespace}",
		"version": "${version}",
		"metadata": {
			"name": "Magnesium",
			"description": "Actual data manipulation for data packs.",
			"contributors": {
				"Laxystem": "Developer"
			},
			"contact": {
				"homepage": "https://laxla.quest/",
				"issues": "https://github.com/LaylaMeower/${namespace}/issues",
				"sources": "https://github.com/LaylaMeower/${namespace}/issues"
			},
			"icon": "assets/${namespace}/icon.png"
		},
		"intermediate_mappings": "net.fabricmc:intermediary",
		"entrypoints": {
			"init": {
				"adapter": "kotlin",
				"value": "${package}.${namespace}.Magnesium"
			}
		},
		"depends": [
			{
				"id": "quilt_loader",
				"versions": ">=0.19.2"
			},
			{
				"id": "quilted_fabric_api",
				"versions": ">=7.4.0"
			},
			{
				"id": "qkl",
				"versions": ">=2.1.0"
			},
			{
				"id": "minecraft",
				"versions": ">=1.20.1"
			}
		]
	},
	"mixin": "${namespace}.mixins.json"
}
```

Getting this same error when I add the following. Workaround is to provide this property through other means
logging:   pattern:     correlation: "[${spring.application.name:},%X{traceId:-},%X{spanId:-}] "

Getting same error with below property in yaml, workaround on it setting the property through an *.properties override file
logging:
  pattern:
    correlation: "[${spring.application.name:},%X{traceId:-},%X{spanId:-}] "

Same issue, workaround not possible for me as the resource is generated by a plugin, please resolve


Move to WinAppSDK-1.1.0

Summary of the Pull Request
note: Removed PowerRename method was breaking the build because of the unresolved reference. Method is not needed, therefore deleted.
Someone please test ARM64 installer.
What is this about:
What is included in the PR:
How does someone test / validate:
Quality Checklist

[x] Linked issue: Solves some of the issues in #18015
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Does this fix some of the settings bugs?

Is this able to hop on .59 or has that train already left the station?

Does this fix some of the settings bugs?

It will fix the "unable to run elevated" issue so many people are having.

Is this able to hop on .59 or has that train already left the station?

Yes, this is what we were waiting for to include in .59 .

Don't want to bother but both Settings and PowerRename are not starting after a clean build.
The error is 0xc0000005 access violation.
Could be my machine but never had any issue with SDK 1.0.3.

Don't want to bother but both Settings and PowerRename are not starting after a clean build. The error is 0xc0000005 access violation. Could be my machine but never had any issue with SDK 1.0.3.

Wow. That's pretty weird. Any other info there? Like a call stack from VS?

Looks like the error is on Microsoft.WindowsAppRuntime.Bootstrap.Net.dll side.
Can I help somehow?
System.AccessViolationException
  HResult=0x80004003
  Message=Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
  Source=<Cannot evaluate the exception source>
  StackTrace:
<Cannot evaluate the exception stack trace>

@davidegiacometti  You've installed winappsdk 1.1, right? Any errors in the command line when installing it?
https://aka.ms/windowsappsdk/1.1/1.1.0/windowsappruntimeinstall-1.1.0-x64.exe
Did you restart after?
This is pretty weird.
Do you have event viewer events? I'm thinking creating an issue like this might help, which is a similar error but the exception code is different: https://github.com/microsoft/WindowsAppSDK/issues/2573
Does the WinUI 3 Gallery from the store also crash in a similar fashion?

@davidegiacometti , this issue https://github.com/microsoft/WindowsAppSDK/issues/2492 seems to indicate somethings might need to be cleaned up. Can you try to really clean the repo with git clean -ffxd and trying to build again?

I agree that's really strange.
I have installed 1.1 runtime and both VS extensions (C# and C++).
Now I have:

Rebooted again
git clean -ffxd
Rebuild
Same error

I am also experiencing the error with WinUI 3 Gallery.

I have been able to fix it:

Removed all App SDK installations: 0.8, 1.0.3, several 1.1 previews.
Rebooted
Reinstalled 1.0.3 and 1.1
Everything works

I think the build was failing becouse of 1.1 previews installed.

Thanks for the heads-up @davidegiacometti . This is something I definitely want to repro locally to open an issue upstream.


Allow use in class

Say you want to make a method equal to createAction. Currently this doesn't work very well because this isn't accessible from the payloadCreator or metaCreator.
A simple change will make this work.
First change actionHandler from a lamba to a normal function so it takes on the this of the Class it is assigned to.
Second, bind this to payloadCreator and metaCreator before they are called.

Is this what you are trying to accomplish?
class Foo {
  constructor() {
    this.handleAction = createAction(
      'HANDLE_ACTION',
      this.payloadCreator.bind(this),
      this.metaCreator.bind(this)
    );
  }
  payloadCreator() {
  }
  metaCreator() {
  }
}

That's the intended result, yes. But of course, doing that defeats the purpose of createAction as it's purpose is to reduce code, so I want this to be able to do the same thing:
class Foo {
  this.handleAction = createAction(
    'HANDLE_ACTION',
    function () {},
    function () {}
  )
}

Not saying this isn't a valid use case, but it seems like a bad smell to make action creators tied to every instance of a class.
My approach to this kind of thing in the past is to do anything that requires context before calling the action creator with my data.
Is there a reason this would not work or is undesirable? I can see the case of reducing the amount of code to write, but that seems a lot of burden to put on a third party library.

I don't like polluting global namespaces - I prefer python over php. I don't see what's so intrinsically bad with namespacing that you have to ban others from using it.
The reason it currently does not work is because it is using lambdas which bind this at declaration time (in this case to undefined), rather than following the rules of methods. I don't see how moving from lambda to function declaration is a 'burden'. It literally does nothing for existing use cases.

I'll create the PR, I have just been burned too many times to do a PR before I know it will be accepted.

What's the purpose of creating actions in classes?

What's the purpose of creating actions in classes?

I've never seen a Redux example where a class had action creation as a shared behavior. Is your class a React Component, local business object, or a helper module?
I'd encourage simplicity and using a functional programming style with Redux where possible.

@timche To use classes. Well, that was obvious, so perhaps your real question is why is OOP even useful ever? This is probably a result of OOP functionality in javascript being traditionally extremely limited, and still lacking in many ways. However, even with JS's limitations that are cases where it is still very useful.
More on OOP here: https://www.quora.com/What-does-object-oriented-programming-do-better-than-functional-programming-and-why-is-it-the-most-popular-paradigm-when-everybody-seems-to-say-functional-programming-is-superior
I recommend reading the ENTIRETY of the above. Hope this clears things up :)

Er...

And to be specific to my use case, I'm adding methods and factories to my immutable data structures. (I personally use Immutable.js - the thing Facebook made.)

Arrow functions are not syntactic sugar - they change the binding behavior with regard to 'this'.
So when you write
const actionCreator = (...args) => {
...
}
return actionCreator;

You're actually writing
function actionCreator(...args) {
...
}.bind(this);
return actionCreator;

But nowhere in the code are you trying to attach this to anything specific - so you're adding extra lines of javascript to be executed just to destroy normal javascript behavior.
Changing this is quite trivial.

@ntucker Do you mind sharing your specific use case?

Sure, I'll show you my immutable Class.
export default function ModelRecord(defaultValues, name) {
  class Model extends Record.call(this, defaultValues, name) {
    static issueFind = createAction(
      FIND,
      undefined,     // use identity
      function() {
        return {
          Model: this,
        }
      }
    );
    ...
  }
}

Then something like for defining the model declaratively:

```javascript
export default class UserRecord extends ModelRecord({
  id: null,
  username: '',
  email: '',
  first_name: '',
  last_name: '',
}, 'UserRecord') {
  static urlRoot = `/api/user/`

  get fullName() {
    return [this.first_name, this.last_name].join(' ')
  }
}


Add usage for per-profile node affinity

/sig scheduling
Pending implementation PR: kubernetes/kubernetes#96202
Ref kubernetes/kubernetes#95738

/milestone 1.20
/hold pending merge https://github.com/kubernetes/kubernetes/pull/96202

/assign

/assign @Huang-Wei

/hold cancel
k/k merged

@annajung - is this PR you are referring merged k/k merged? https://github.com/kubernetes/kubernetes/pull/96202.
Kindly mention in your comment , thanks.

/lgtm

Please leave lgtm for sig-scheduling approver. I'll handle the comments
tomorrow.
On Mon., Nov. 9, 2020, 7:53 p.m. Kubernetes Prow Robot, <
<EMAIL_ADDRESS>wrote:

@raghvenders https://github.com/raghvenders: changing LGTM is
restricted to collaborators
In response to this
https://github.com/kubernetes/website/pull/24914#issuecomment-724377217:
/lgtm
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/pull/24914#issuecomment-724377307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ5E6ESZDTFIB6HSEQNFJLSPCFH7ANCNFSM4TL4O7BQ
.

Thanks @alculquicondor.On a note , reviewers or approves can do anyways not me.

/lgtm
from scheduling side.


Change TO jobs endpoint be backwards-compatible

Changes TO jobs endpoint be backwards-compatible,
For both the server and client,
by making the library serialize and deserialize compatible JSON.
This isn't strictly required by the current ATC versioning scheme, but it makes upgrade and rollback work for people who have been using master.
Includes 100% test coverage for added code, excepting SQL.
Because it's entirely contained in library marshal functions, the
code can easily be removed when support is dropped after
the next major version.
Has tests.
No docs, no interface change.
No changelog, no interface change.

Which Traffic Control components are affected by this PR?

Documentation
Traffic Control Cache Config (T3C, formerly ORT)
Traffic Control Health Client (tc-health-client)
Traffic Control Client
Traffic Monitor
Traffic Ops
Traffic Portal
Traffic Router
Traffic Stats
Grove
CDN in a Box
Automation
unknown

What is the best way to verify this PR?

If this is a bugfix, which Traffic Control versions contained the bug?

PR submission checklist

[x] This PR has tests
[x] This PR has documentation
[x] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

To be clear, this was a proposal and one possible implementation I was hoping to discuss. This was not intended to be merged immediately or without the people with opinions considering and discussing it.
I changed it to a Draft to make it clear this was never intended to be merged without discussion.


Dismiss NavigationView

Hi! Im presenting coordinator like this
@Route(.modal) var card = makeCard

func makeCard() -> NavigationViewCoordinator<CardCoordinator> {
    NavigationViewCoordinator(CardCoordinator())
}

Card coordinator contains 3 screens in navigation stack.
Then im trying to dismiss it like this in last view in navigation stack:
router.dismissCoordinator {
      print("dismiss")
}

After that im facing crash in dismissChild method of NavigationCoordinatable
How can i dismiss it correctly?

Hi! It looks like we have a bug in the latest version of Stinsen. Try downgrading for the moment and see if it solves your issue :)

Should be fixed in 2.0.5. Please try it and see if it solves your issue.

It works now, thanks!


Section <IP_ADDRESS> para 389, FCS_DTLSC_EXT.1.6

FCS_DTLSC_EXT.1.6 Please clarify what is meant by: “This test should be considered to be performed together with FCS_DTLSC_EXT.1.1 Test 1.” Is this normative or informative statement?
Also found in Para 427, 530, 557.

I think the statement is an informative one, so we should replace the word "should be considered". i suggest the following :"
"If the TSF provides the ability of configuring the list of supported cipher suites, the Evaluator shall establish a TLS connection using one of the possible configurations of the list of supported cipher suites. The Evaluator shall then change the configuration and repeat the test. The Evaluator shall verify that the behavior of the TOE has changed according to the modification of the list of cipher suites. This test shall be repeated for all supported TLS versions. This test could be performed together with FCS_TLSS_EXT.1.1, Test 1. If the TSF does not have the ability to configure the list of supported cipher suites, this test shall be omitted."

basically the original version says that test1 in FCS_DTLS_EXT.1.1, test1 and FCS_DTLSC_EXT.1.6 should be done together (strong statement) which i think was not the intent of the test case.
same for Para 427, 530, 557.

If the text is considered informative, then let’s remove it. Let the evaluator determine the best way to perform the test and convey the results. (An evaluator is always free to document that a single test case shows multiple results regardless of whether such a hint is provided in the test wording.)

On today's call, the TLSWG agreed to Greg's suggestion to remove the sentence.


changeSessionId() bug

changeSessionId gets called repeatedly because Session::needToChange_ is never set to false (sorry didn't notice earlier).
Adding this to the end of SessionManager::changeSessionId() fixes it:
sessionPtr->needToChange_ = false;

Oh, my fault. I'll fix it. thanks.

@pmattione I have fixed it, please check, thank you.

:thumbsup:


nodewatcher: JS out of memory

Sometime on Dec 23rd:
2019-12-23 07:02:25    NODE-WATCHER: blockIndex: 112582
2019-12-23 07:02:25    NODE-WATCHER: lastKnownBestFinalized: 332240

<--- Last few GCs --->

[26:0x560e79aab0a0] 80703180 ms: Mark-sweep 1849.1 (1897.1) -> 1849.0 (1868.1) MB, 1699.2 / 0.0 ms  (average mu = 0.555, current mu = 0.000) last resort GC in old space requested
[26:0x560e79aab0a0] 80704916 ms: Mark-sweep 1849.0 (1868.1) -> 1848.6 (1865.6) MB, 1735.4 / 0.0 ms  (average mu = 0.384, current mu = 0.000) last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 0x560e7502bb79]
Security context: 0x0a98eabc0921 <JSObject>
    1: new constructor(aka SafeSubscriber) [0x3c0ca2126501] [/node_watcher/node_modules/rxjs/internal/Subscriber.js:~113] [pc=0x35355f386b51](this=0x1fed97005869 <SafeSubscriber map = 0x230eb6dc8b91>,0x1fed97005809 <ConnectableSubscriber map = 0x230eb6dc7dd1>,0x1fed97004c59 <Object map = 0x2991fe2d5d41>,0x1aecec5c04b9 <undefined>,0x1aecec5c04b9 <undefined>)
...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
error Command failed with signal "SIGABRT".

restarting the deployments, we'll see if this was a one off or reproducible in 2 days.

An idea:
https://github.com/paritytech/Nomidot/blob/7cf5c81fd073d1cc25e448df2488c5f28038064a/back/node_watcher/package.json#L21
Maybe, in prod, we should run yarn build first to build into the JS file, then run node build/index.js, so that it runs the JS file directly, instead of transpiling on the fly.

^ ah! will try that out now

this is working locally: https://github.com/paritytech/Nomidot/pull/81/files


CLOUDSTACK-8725 RVR functionality is broken in case of isolated netwo…

CLOUDSTACK-8725 RVR functionality is broken in case of isolated networks, conntrackd fails to start.

@bvbharatk Had a quick look and it seems to solve the problem. Can you please rebase against latest master so we can have a clean merge? Thanks!
@DaanHoogland Any more questions on this one?

@remibergsma no, answer is clear and though my question expresses a reservation on design level it should not stop this PR; => LGTM

@DaanHoogland OK, thanks!

verified that conntrackd starts on redundant vpc routers.
LGTM in that respect.
Design wise i feel we need to refactor this, naming of copy_if_needed is ambiguous.
Also we're running a lot of boilerplate code which can be replaced by more widely used and maintained python packages.

@bvbharatk The rebase is needed due to the 'PEP8' style fix that has been done on CsRedundant.py Let me know if you need help :-)


Automatically change Title: header when using Dendron: Rename Note to mirror new filename

Is your feature request related to a problem? Please describe.
I was expecting that the Title: header would change automatically when using the Dendron : Rename File function
Describe the solution you'd like
Title: header mirrors by default the dendron note Filename (the lowest hierarchy).
Eventually let the possibility to have a Title different from the lowest hierarchy filename, but this should be achieved by changing the Title: field in the header manually, not the other way round.
Describe alternatives you've considered
Renaming by hand

Thanks for the feature request. The way we'll implement this is probably with a dropdown when you select the rename/refactor command. This is because some people use custom titles for names that differ from the filename so this would overwrite that.

Just adding my +1 to this feature request :)

This has just been released in the 0.17 update :)


pipewire: socketActivation = false does not fully disable socket activation

Describe the bug

socketActivation = false only prevents the first pipewire activation within a user's session, but after pipewire has been activated and then deactivated (some other way), it does not prevent future socket activations.

Steps To Reproduce
Steps to reproduce the behavior:

deploy a system with services.pipewire = { enable = true; socketActivation = false; };
start and stop the pipewire service: systemctl --user start pipewire.service ; systemctl --user stop pipewire.service
see that the socket unit is activate: systemctl status pipewire.socket
3.a. alternatively: launch a pipewire-capable media player, or redo the above with pipewire-pulse & launch a pulseaudio media player: observe that pipewire.service is now running again.

Expected behavior
socketActivation = false should disable pipewire.service's dependency on pipewire.socket. an intuitive use for socketActivation = false is if the user is manually managing the service lifetime. that socketActivation = false isn't enough to prevent pipewire.service from being implicitly (re-)activated is surprising.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context

socket activation for pipewire seems to be an assumption made within systemd. systemctl --user cat pipewire.service:

$ systemctl cat --user pipewire
# /etc/systemd/user/pipewire.service
[Unit]
Description=PipeWire Multimedia Service

# We require pipewire.socket to be active before starting the daemon, because
# while it is possible to use the service without the socket, it is not clear
# why it would be desirable.
#
# A user installing pipewire and doing `systemctl --user start pipewire`
# will not get the socket started, which might be confusing and problematic if
# the server is to be restarted later on, as the client autospawn feature
# might kick in. Also, a start of the socket unit will fail, adding to the
# confusion.
#
# After=pipewire.socket is not needed, as it is already implicit in the
# socket-service relationship, see systemd.socket(5).
Requires=pipewire.socket

Notify maintainers
@NixOS/freedesktop @K900
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.18, NixOS, 24.05 (Uakari), 24.05.19700101.dirty`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/store/infnhcrznz7r5ns1lj9l857p1pmg0iin-s9wj3vq60p57kgqghjb4z1nwh31wm5yd-nixpkgs-patched-uninsane`

(i'm on nixpkgs master from yesterday: b66514c14e85cd7d853d6dbbf1a421ba232eff10)

Add a :+1: reaction to issues you find important.

Honestly maybe we should just rip out that option. I'll talk to upstream.


add Agoric to README

Adds the following events or companies:

Agoric
https://www.eventbrite.com/e/postponed-rethinking-distributed-programming-tickets-97404019043

Checklist

[X] I have updated the event or company counts
[X] I have put the most recent relevant date in the "Last Update" column
[X] I have linked to the article about the change, not the company's website

Hi, this looks good. Is there a public post on the company policy by chance?

Not yet. We changed modes at the beginning of the week, but haven't posted a public document. We will update when we do. (Dean Tribble - CEO).

Thanks!

Unfortunately, GitHub won't let me resolve the merge conflict (company counts). Once that's fixed, I'll merge.

@drewbrew this is updated. Thank you.

Hi there! We recently re-worked the site to use yaml data files. We'd be happy to accept your PR once it's been reworked to use the right format.
Thanks!

@phildini  all done thank you :)

A small follow-up; our discussion of company policy: https://medium.com/agoric/11-things-covid-19-taught-us-about-the-decentralized-workplace-c03e1efebdc5


Update de.json

Added RollResult in German language.

In the future can you please consolidate ALL of your language PRs into one single PR.


Clarify section describing demultiplexing

Closes #109
Here I edited the methods section on processing the HTO data. I attempted to clarify that we apply three methods to identify which cells may belong to which sample and then emphasize that we do not actually separate the samples into their own objects. Instead, the multiplexed library contains counts data from all samples that were multiplexed together and the results from demultiplexing.
Does this help clarify what we did? In general, we do not separate the cells from each sample, but we provide the results so that users can separate if they so desire.

This makes sense to me - thank you! 🚀


Godot crashes during lightmap generation

Godot version:

Godot 3.1.1
OS/device including version:

Windows 10
Issue description:

When I changed light baking to Gen Lightmaps and pressed Reimport, the lightmap generation process was started, but after some time progress freezed and then Godot crashed.
Steps to reproduce:

Add floor_2.escn and/or female.escn and/or chest_2.dae to the project, select it in the file manager
Switch to the Import tab
Select Light Baking == Gen Lightmaps
Hit reimport

Models used:
floor_2.zip
female.zip
chest_2.zip
P.S.: There was an error in console just before crash:

Can't reproduce on Ubuntu 19.04 and 3.2.dev.custom_build. adae2b073
Seems that this issue exists with older version of xatlas.
Maybe a cherrypick of some of commits may fix that without any negative effects

@qarmin , I would be glad if this will work in version 3.2 then, I'll be waiting for it :)
But right now it is a common problem, unfortunately :( I've added another model (chest_2.zip), which crashes in the same way to the issue description.

No answer, so closing.

Actually #40035 says it closes this issue, so the issue is likely still valid.

Fixed in #44628

Fixed in #44628


`require': cannot load such file -- bundler/setup (LoadError)

Installation:
↪  gem install mysqltopostgres                                                                                Wed Feb  3 16:54:32 PST 2016
Fetching: i18n-0.7.0.gem (100%)
Successfully installed i18n-0.7.0
Fetching: thread_safe-0.3.5.gem (100%)
Successfully installed thread_safe-0.3.5
Fetching: tzinfo-1.2.2.gem (100%)
Successfully installed tzinfo-1.2.2
Fetching: activesupport-<IP_ADDRESS>.gem (100%)
Successfully installed activesupport-<IP_ADDRESS>
Fetching: builder-3.2.2.gem (100%)
Successfully installed builder-3.2.2
Fetching: activemodel-<IP_ADDRESS>.gem (100%)
Successfully installed activemodel-<IP_ADDRESS>
Fetching: arel-6.0.3.gem (100%)
Successfully installed arel-6.0.3
Fetching: activerecord-<IP_ADDRESS>.gem (100%)
Successfully installed activerecord-<IP_ADDRESS>
Fetching: activerecord-jdbc-adapter-1.3.19.gem (100%)
Successfully installed activerecord-jdbc-adapter-1.3.19
Fetching: jdbc-postgres-9.4.1204.gem (100%)
Successfully installed jdbc-postgres-9.4.1204
Fetching: activerecord-jdbcpostgresql-adapter-1.3.19.gem (100%)
Successfully installed activerecord-jdbcpostgresql-adapter-1.3.19
Fetching: jdbc-sqlite3-<IP_ADDRESS>.gem (100%)
Successfully installed jdbc-sqlite3-<IP_ADDRESS>
Fetching: activerecord-jdbcsqlite3-adapter-1.3.19.gem (100%)
Successfully installed activerecord-jdbcsqlite3-adapter-1.3.19
Fetching: mysql-pr-2.9.11.gem (100%)
Successfully installed mysql-pr-2.9.11
Fetching: postgres-pr-0.6.3.gem (100%)
Successfully installed postgres-pr-0.6.3
Fetching: mysqltopostgres-0.2.15.gem (100%)
Successfully installed mysqltopostgres-0.2.15
Parsing documentation for i18n-0.7.0
Installing ri documentation for i18n-0.7.0
Parsing documentation for thread_safe-0.3.5
Installing ri documentation for thread_safe-0.3.5
Parsing documentation for tzinfo-1.2.2
Installing ri documentation for tzinfo-1.2.2
Parsing documentation for activesupport-<IP_ADDRESS>
Installing ri documentation for activesupport-<IP_ADDRESS>
Parsing documentation for builder-3.2.2
Installing ri documentation for builder-3.2.2
Parsing documentation for activemodel-<IP_ADDRESS>
Installing ri documentation for activemodel-<IP_ADDRESS>
Parsing documentation for arel-6.0.3
Installing ri documentation for arel-6.0.3
Parsing documentation for activerecord-<IP_ADDRESS>
Installing ri documentation for activerecord-<IP_ADDRESS>
invalid options: -SHN
(invalid options are ignored)
Parsing documentation for activerecord-jdbc-adapter-1.3.19
Installing ri documentation for activerecord-jdbc-adapter-1.3.19
Parsing documentation for jdbc-postgres-9.4.1204
Installing ri documentation for jdbc-postgres-9.4.1204
Parsing documentation for activerecord-jdbcpostgresql-adapter-1.3.19
Installing ri documentation for activerecord-jdbcpostgresql-adapter-1.3.19
Parsing documentation for jdbc-sqlite3-<IP_ADDRESS>
Installing ri documentation for jdbc-sqlite3-<IP_ADDRESS>
Parsing documentation for activerecord-jdbcsqlite3-adapter-1.3.19
Installing ri documentation for activerecord-jdbcsqlite3-adapter-1.3.19
Parsing documentation for mysql-pr-2.9.11
Installing ri documentation for mysql-pr-2.9.11
Parsing documentation for postgres-pr-0.6.3
Installing ri documentation for postgres-pr-0.6.3
Parsing documentation for mysqltopostgres-0.2.15
Installing ri documentation for mysqltopostgres-0.2.15
Done installing documentation for i18n, thread_safe, tzinfo, activesupport, builder, activemodel, arel, activerecord, activerecord-jdbc-adapter, jdbc-postgres, activerecord-jdbcpostgresql-adapter, jdbc-sqlite3, activerecord-jdbcsqlite3-adapter, mysql-pr, postgres-pr, mysqltopostgres after 36 seconds
16 gems installed

My config:
default: &default
  adapter: jdbcpostgresql
  encoding: unicode
  pool: 4
  username: dev
  password: dev
  host: <IP_ADDRESS>

development: &development
  <<: *default
  database: main

test: &test
  <<: *default
  database: main

production: &production
  <<: *default
  database: main

mysql_data_source: &pii
  hostname: localhost
  port: 3306
  socket: /tmp/mysqld.sock
  username: dev
  password: dev
  database: main

mysql2psql:
  mysql:
    <<: *pii

  destination:
    production:
      <<: *production
    test:
      <<: *test
    development:
      <<: *development

  tables:

  # If suppress_data is true, only the schema definition will be exported/migrated, and not the data
  suppress_data: false

  # If suppress_ddl is true, only the data will be exported/imported, and not the schema
  suppress_ddl: false

  # If force_truncate is true, forces a table truncate before table loading
  force_truncate: false

  preserve_order: true

  remove_dump_file: true

  dump_file_directory: /tmp

  report_status: json    # false, json, xml

  # If clear_schema is true, the public schema will be recreated before conversion
  # The import will fail if both clear_schema and suppress_ddl are true.
  clear_schema: false

Error:
mysqltopostgres ~/Projects/gandalf/scripts/postgres/test.yaml                                                                                Wed Feb  3 17:03:17 PST 2016
/usr/local/Cellar/ruby/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require': cannot load such file -- bundler/setup (LoadError)
	from /usr/local/Cellar/ruby/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/lib/ruby/gems/2.2.0/gems/mysqltopostgres-0.2.15/bin/mysqltopostgres:5:in `<top (required)>'
	from /usr/local/bin/mysqltopostgres:23:in `load'
	from /usr/local/bin/mysqltopostgres:23:in `<main>'

Same error on OSX 10.11.13 with Ruby 2.0

You need to install bundler:
gem install bundler


Do not kill an existing daemon, but check if one exists

This removes the functionality of killing an existing daemon and instead checks for an existing one and connect to it.
This will also close #298.

@IGassmann does electron builder need to be updated again after the lastest commit in: https://github.com/electron-userland/electron-builder/issues/2516

@tzarebczan Yes, I'm waiting for 20.5.0 to be released for upgrading. It'll also solve this: https://github.com/electron-userland/electron-builder/issues/2670.
Thanks for notifying me.


Can we add Australian APRS (145.175MHz) to the list?

Thank you for this awesome tool, but would be great to add the Australian APRS frequency to the list. 145.175MHz.

Thank you for the feedback, I have added 145.175 MHz to the list.


How to create segment Slider With  NMRangeSlider?

How to create segment Slider With  NMRangeSlider?

@nhuomtv did you manage to create segmented slider with nmrangeSlider?


Added toWhereClauseSQL() & toWhereClauseString()

Fixes #2526.
I'll add docs once it's decided if this should be merged or not.

I was thinking while back about naming of this and maybe it should be even more generic like SQL expression related. Maybe this API could be used to build any kind of expressions to pass different parts of queries and not to be limited just to where clause?


UniqueValidator for multiple columns

Is there any plan for this feature?

Hello ,
I want unique validation only in insert not in update.
Please help

use scenarios. also please use the forum for questions, github is for bug tracking.


Add test for New-Item -Force

PR Summary
I created new test for checking if New-Item -Force creates a file if the folder in path is missing. It also checks that when -Force is not used, the command fails.
PR Context
This is my first PR so feel free to give your feedback what I could do better :)
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

@PoshChan Please remind me in 24 hours

@daxian-dbw, this is the reminder you requested 24 hours ago


Create or update project environment variable when the variable's name is already used in a global environment variable

Describe the bug
When we use the function ''env_vars.create_or_update_for_project", if the name of the variable is already used in a global environment variable, this function will try to update the variable and then it will raise an error:
"TransportQueryError: {'message': 'Environment variable is invalid', 'path': ['saveEnvironmentVariable'], 'extensions': {'scope': 'A Global variable must
not be linked to an entity id', 'classification': 'ValidationError'}}"

Expected behavior
When the name of the variable is already used in a global environment variable, and we use the function  ''env_vars.create_or_update_for_project", it should create a project environment variable instead to update it.
Exemple to reproduce the error
from saagieapi import SaagieApi
url_saagie = "https://saagie-workspace.prod.saagie.io"
id_platform = "4"
user = "XXX"
realm = "saagie"
pwd = "YYYY"
saagie_api = SaagieApi(
    url_saagie=url_saagie, id_platform=id_platform, user=user, password=pwd, realm=realm
)

saagie_api.env_vars.create_global(name="TEST_VAR1", value="1234")
project_id = "project_id_to_create_or_update_env_var"
saagie_api.env_vars.create_or_update_for_project(name="TEST_VAR1", value="2345", project_id=project_id)

Trello card: [#142] Create or update project environment variable when the variable's name is already used in a global environment variable


Add support for Webpack style module ids in LESS/CSS

As documented here https://github.com/webpack/less-loader#imports

@ddumont, please review

otherwise lgtm

Requested change made.


[CRIMAPP-1244] Adjust schema to accept Change in Financial Circumstances applications

Description of change
Removes restrictions on certain fields to allow CIFC applications to be submitted with null values for court hearing and offences/codefendants
Link to relevant ticket
https://dsdmoj.atlassian.net/browse/CRIMAPP-1244
Additional notes

DOES NOT UPDATE MAAT SCHEMA! Consider what the best course of action is
Cannot use draft-07 JSONSchema because the json-schema gem does not support that,  meaning if/then/else conditional schema rules are not available to use

Just remembered the version needs updating too @mseedat-moj


Add proxy-bypass option to prevent using proxy depending on rule

see API in tests

@testcafe-build-bot \retest

@testcafe-build-bot \retest

FPR

Great to see this feature beeing added! Thank you!
Is this expected to work for ressources like CSS stylesheets or JavaScript files that are linked from the page under test?
I'm sitting behind a corporate proxy and want to test a page that is served by a web server on localhost. The page loads a JavaScript library from a remote host. So the request to load the page should bypass the proxy, the request to get the library should go through the proxy.
I call TestCafé CLI with --proxy <IP_ADDRESS>:8080 --proxy-bypass <IP_ADDRESS>:8000. The page and all local ressources load correctly but the remote script does not. It keeps loading forever until a timeout is hit eventually.
To test in isolation I created this small repo: n0v1/testcafe-proxy-bypass-test

Hi
Now this feature is implemented only to bypass proxy for specific tests according to the test page url. But your case seems to be important, so I think we'll support it as well.

Leave a comment

Hello @amirse80
TestCafe CLI supports two options for using proxy. Please refer to the following articles:
--proxy and --proxy-bypass.
According to our documentation, the proxyoption specifies the proxy server used in your local network to access the Internet. If you do not have access to the second URL, I suppose that you need to use the proxy option.
The proxy-bypass option can be used together with the proxy option. It's required when you use proxy, but don't need to use proxy only for some specific resources.

Hello,
Thank you for the input. Unfortunatelly it didn´t solve my problem.
The issue is like this.

I navigate to an company internal page.    (.page `http://dap-internal.company.page´)
The chrome is started and initially navigates to the correct server.
During the page opening the URL changes, redirects, to http://pen-internal.company.page. Now this new URL works fine in the same browser that testcafe has started IF i copy the real URL manually into the new tab BUT testcafe ADDS http://<IP_ADDRESS>:61019/DSc6im7OL/ to the URL which makes it not working. So instead of the real redirected URL http://pen-internal.company.page the redirected URL is http://<IP_ADDRESS>:61019/DSc6im7OL/http://pen.internal.company.page!


Arrive at app wall + filter by cf can sometimes throw exception

Exception thrown was undefined on entity. This could be an org missing a space?
Seen when I think console session started on fresh load of services pages
Need to ensure that the filters block correctly if we're still fetching the org list (and its required org relations such as space)

Think this might be the same issue as solved by the si->space->si circular dependency issue (think Irfan has this solved in a branch). Also #2385. We should test this again next week once all are merged.

Could not reproduce with latest. Most probably fixed by items in previous comment


keypad.registerKeyPressHandle

am having a problem with this part of the programme(keypad.registerKeyPressHandle) in that when i run it, it first produce a none value before it gives the pressed values on the keypad. now when it come to compare the value with a predetermined password, the outcome is wrong because it present the none (the first value it obtained before the pressed ones)

Thanks for the report, but unfortunately this is the wrong tracker for it. This tracker can only help with issues related to the software that runs https://pypi.org itself. It cannot help with packages distributed from or installed from there. For that, you'll need to contact the maintainers of the specific package you're having trouble with.


Branch developer guide

Codecov Report

Merging #426 (d58432b) into master (1dfe5ca) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #426   +/-   ##
=========================================
  Coverage     79.89%   79.89%
  Complexity     1069     1069
=========================================
  Files           136      136
  Lines          3208     3208
  Branches        430      430
=========================================
  Hits           2563     2563
  Misses          573      573
  Partials         72       72

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 1dfe5ca...d58432b. Read the comment docs.


Remove custom caching

This removes some parts of the custom caching and determines the information directly from SDKMAN. The advantage is that the information is live and the cache does not get stale. (The list of available versions needs to be cached, still.)

Hello Manuel, thanks for all your contributions to the project !
I am pretty busy currently but I'll try my best to have a look at your modifications every now and then.

Hello Manuel, thanks for all your contributions to the project !

I am pretty busy currently but I'll try my best to have a look at your modifications every now and then.

No problem! I think #22 is "working" right now, I am using it daily to find out if it has problems. I hope I did not break anything for anyone. I tried to use ZSH where possible to avoid problems with differences in command-line tools on GNU vs. BSD.


Deutsche Telekom, SIP registration  failed

Hi Arne,
I've tried to use your addon with my Deutsche Telekom Account.
Unfortunately I cannot get it to register properly:
11:27:49.452          sip_resolve.c  ...Failed to resolve 'tel.t-online.de': gethostbyname() has returned error (PJ_ERESOLVE)
11:27:49.452           tsx0x16bbb88  ...Failed to send Request msg REGISTER/cseq=51565 (tdta0x16b9bc8)! err=70018 (gethostbyname() has returned error (PJ_ERESOLVE))
11:27:49.452            pjsua_acc.c  .....SIP registration failed, status=502 (gethostbyname() has returned error (PJ_ERESOLVE))
11:27:49.453            pjsua_acc.c  .....Scheduling re-registration retry for acc 0 in 9 seconds..
| OnRegState: 502 gethostbyname() has returned error (PJ_ERESOLVE)
11:27:49.453              sip_reg.c  ..Error sending request: gethostbyname() has returned error (PJ_ERESOLVE)
Telekom has disable the DNS-A Record and SIP Clients need to resolve the host with SRV Record.
Any chance you could add this functionality?

Hi,
from the docs it looks like I need to create a way to specify a nameserver to make pjsip use SRV records.
I'll add that in the next release.

This is in 2.4 now. Please test if this is working as expected!

Thanks for the fast response.
There's a new error now:

19:26:13.015            pjsua_acc.c  Contact for acc 0 updated: sip:<IP_ADDRESS>:5060;ob;+sip.ice
19:26:13.015            pjsua_acc.c  Acc 0: setting registration..
19:26:13.015             resolver.c  ..Transmitting 43 bytes to NS 0 (<IP_ADDRESS>:53): DNS SRV query for _sip._udp.tel.t-online.de: Success
19:26:13.015             resolver.c  ..Transmitting 43 bytes to NS 1 (<IP_ADDRESS>:53): DNS SRV query for _sip._udp.tel.t-online.de: Success
19:26:13.015            pjsua_acc.c  .Acc 0: Registration sent
19:26:13.033 _sip._udp.tel.t-online  DNS SRV resolution failed for _sip._udp.tel.t-online.de: DNS "Refused" (PJLIB_UTIL_EDNS_REFUSED)
19:26:13.033 _sip._udp.tel.t-online  DNS SRV resolution failed for _sip._udp.tel.t-online.de, trying resolving A/AAAA record for tel.t-online.de
19:26:13.033             resolver.c  Transmitting 33 bytes to NS 0 (<IP_ADDRESS>:53): DNS A query for tel.t-online.de: Success
19:26:13.052          sip_resolve.c  DNS A/AAAA record resolution failed: DNS "Refused" (PJLIB_UTIL_EDNS_REFUSED)
19:26:13.052           tsx0x14a5528  Failed to send Request msg REGISTER/cseq=46674 (tdta0x149dd88)! err=320055 (DNS "Refused" (PJLIB_UTIL_EDNS_REFUSED))
19:26:13.052            pjsua_acc.c  ..SIP registration failed, status=503 (DNS "Refused" (PJLIB_UTIL_EDNS_REFUSED))
19:26:13.052            pjsua_acc.c  ..Scheduling re-registration retry for acc 0 in 299 seconds..
| 19:26:13.052150 [1] OnRegState: 503 DNS "Refused" (PJLIB_UTIL_EDNS_REFUSED)

Looks like the name server refused to answer that query.  Can you try with <IP_ADDRESS>?

I dug a bit deeper into that topic, and pjsip should actually use a NAPTR query on tel.t-online.de, which it does not support: https://github.com/pjsip/pjproject/issues/3305
I don't understand why the cloudflare DNS server does respond to the SRV query for _sip._udp.tel.t-online.de and the official Telekom server does not.
Maybe this work-around does it for you until pjsip is able to use NAPTR records.

Hey Arne,
I'm one step further but still can't manage to register my SIP Account:

23:03:30.419           pjsua_core.c  .RX 445 bytes Response msg 403/REGISTER/cseq=26967 (rdata0x2ba4988) from UDP <IP_ADDRESS>:5060:
SIP/2.0 403 Forbidden

Via: SIP/2.0/UDP <IP_ADDRESS>:15060;received=<IP_ADDRESS>;rport=15060;branch=z9hG4bKPjtbLDXG8Wz68WGdwbN8ozFnJYe5D1iBOe
From: sip:me**********@t-online.de;tag=Z6iT5.az8XmfF5XBQQvnGMCZmXOtvJQ6
To: sip:me**********@t-online.de;tag=0204547C6A3C-14c0-c1451700-11081e-63eea832-630e1
CSeq: 26967 REGISTER
Call-ID: BvRm9TWmHPefgJl78PjtO3FYrtk5HJF3
Reason: SIP;cause=403;text="CC_IMS_UNKNOWN_USER"
Content-Length: 0
--end msg--
23:03:30.419            pjsua_acc.c  ....SIP registration failed, status=403 (Forbidden)
| 23:03:30.420054 [1] OnRegState: 403 Forbidden
i've tried the following config:

enabled: true
registrar_uri: sip:tel.t-online.de
id_uri<EMAIL_ADDRESS>realm: tel.t-online.de
user_name: "0682******"
password: SUPERSECRET
answer_mode: listen
settle_time: 1
incoming_call_file: ""

Here's a working config from MicroSIP:

Did you put the username in quotes? Because otherwise the leading zero will be removed because YAML.

Yes, i put it in quotes.
I've tried all different kind of combinations:
Phone Number as Username<EMAIL_ADDRESS>as id_uri.  Email as Id_uri etc.
Can't get it to register properly.

Here are some informations I've found in the Telekom-Forum. Maybe I'm missing something?
https://telekomhilft.telekom.de/t5/Telefonie-Internet/SIP-Phone-einrichten/m-p/937250/highlight/true#M116923

Can you try this one:
registrar_uri: "sip:tel.t-online.de"
id_uri<EMAIL_ADDRESS>realm: "*"
user_name: "phonenumber"
password: "web password"

That did the trick! I never tried @tel.t-online.de.  👍


Can't install the package

I'm getting this error when I try to install the package with pip
(.venv) PS D:\Programming\Python\lmbdFnct> pip install aws-lambda-typing
ERROR: Could not find a version that satisfies the requirement aws-lambda-typing (from versions: none)
ERROR: No matching distribution found for aws-lambda-typing

Minimal version
Is possible downgrade to 3.7? I use 3.7 to test the lambda locally.

FYI @NiumXp python >=3.6 is now supported


Starting a thread in GDScript that calls function with arguments will print error message in recent Godot build

Godot version
Godot 4.1-dev (ed63b9ea23)
System information
Manjaro Gnome
Issue description
thread.start(test.bind(0)) prints the following error:
ERROR: Caller thread can't call this function in this node. Use call_deferred() or call_thread_group() instead.
   at: get_script (scene/main/node.cpp:3536)

Steps to reproduce
func _ready() -> void:
	var thread := Thread.new()
	thread.start(test.bind(0))

func test(i: int) -> void:
	pass

Minimal reproduction project
N/A

Can confirm. Looks like the bind(0) call creates the issue, this works without any errors:
var thread := Thread.new()

func _ready() -> void:
	thread.start(test)

func test() -> void:
	print("no error")

I guess the Callable created when using bind() calls get_script() at some point during the threaded call, which will fail with the new thread safety checks. Note that normally a thread should always be waited until finished, but it's not really relevant for this example.


update labels on all open PRs when a PR is merged (Mu2e/Offline#294)

It occurred to me the other day that Mu2e/Offline#294 could be resolved in a much simpler way, by adding a few lines to process_pr rather than the more complicated steps I suggested originally.
I've added comments to the code to explain the changes.

Hi @kutschke,
While we're at it and since #24 seems to have not broken anything, this change updates the status labels on all other PRs once a PR is merged. If it looks OK to you, I can merge this one too.

Please also merge this one.  Thanks for you help with all of this.
Will this clear the "future commit" label on #350?
Is there a way to adjust the window for deciding something is a future commit - presumably time differences up to a minute or two are fairly normal unless everyone is diligent about configuring ntpd.

OK, will do
The "future commit" label may still appear, but the builds will work as normal.
FNALbuild will complain about any commit timestamped in the future, so maybe I can add a 5-10 minute allowance.

Please add 2 minutes and let’s get experience with it.
From: ryuwd<EMAIL_ADDRESS>Reply-To: Mu2e/CI<EMAIL_ADDRESS>Date: Thursday, February 25, 2021 at 9:17 AM
To: Mu2e/CI<EMAIL_ADDRESS>Cc: Rob Kutschke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Mu2e/CI] update labels on all open PRs when a PR is merged (Mu2e/Offline#294) (#20)
OK, will do
The "future commit" label may still appear, but the builds will work as normal.
FNALbuild will complain about any commit timestamped in the future, so maybe I can add a 5-10 minute allowance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Mu2e_CI_pull_20-23issuecomment-2D785978104&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=Zi5IH-k2_JiK_Wipiv3pwwSXO9UNaHiLP8sisfPz__k&m=3LVCCmXoFEGyXPk7lFp74zyfWIItHUrmXYImwntzCYY&s=yga7OsUkqu4vq_53-nPSaZn87ScnTwrTBgcq2IWyXYg&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABHY5ZVSWHW4LJ2CJ4MAANTTAZSYPANCNFSM4WRWLWBA&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=Zi5IH-k2_JiK_Wipiv3pwwSXO9UNaHiLP8sisfPz__k&m=3LVCCmXoFEGyXPk7lFp74zyfWIItHUrmXYImwntzCYY&s=DHk5wXPb-6lERDH-WTgDxGlgxbH8_rzffS2ArGBFrZg&e=.


Update README.md

[ ] Read the contribution guidelines
[ ] Added unit tests (or unit tests have already been added)
[ ] Updated the Contents

Is this a joke? Closing.


Maximum update depth exceeded when selecting a text in Rich Text Control

Category
[ ] Enhancement
[X] Bug
[ ] Question
Version
Please specify what version of the library you are using: [3.11.0]
Expected / Desired Behavior / Question
No errors when selecting text
Observed Behavior
If I select text in the Richt Text Control the app crashes with the react error "Maximum update depth exceeded"
Steps to Reproduce
Select any text in a Rich Text Control

Any news on this?

The code looks like this:
<RichText   value={values?.comment ?? "-"}   onChange={(text) => {     values.comment = text     setValues(values, true)     return text   }} />
If I select the dash in the richt text control the error is thrown and the app crashes. The SPfx version is 1.15.2.
Dependencies look like this:
     "@azure/msal-browser": "^2.32.2",     "@fluentui/example-data": "^8.3.0",     "@fluentui/react": "^8.60.1",     "@fluentui/react-file-type-icons": "^8.6.2",     "@microsoft/sp-core-library": "1.15.2",     "@microsoft/sp-lodash-subset": "1.15.2",     "@microsoft/sp-office-ui-fabric-core": "1.15.2",     "@microsoft/sp-page-context": "1.15.2",     "@microsoft/sp-property-pane": "1.15.2",     "@microsoft/sp-webpart-base": "1.15.2",     "@pnp/graph": "^3.7.0",     "@pnp/logging": "^3.7.0",     "@pnp/sp": "^3.7.0",     "@pnp/spfx-controls-react": "^3.11.0",     "csstype": "^3.0.10",     "cypress-msal-login": "^1.0.7",     "formik": "^2.2.9",     "js-cookie": "^3.0.1",     "luxon": "^2.3.1",     "office-ui-fabric-react": "7.185.7",     "react": "16.13.1",     "react-dom": "16.13.1",     "react-hot-toast": "^2.2.0",     "react-router-dom": "^6.4.3",     "rxjs": "^7.5.5",     "typescript": "^4.5.5",     "uuid": "^9.0.0",     "yup": "^0.32.11"

Hi @muecke36 sorry for the late response, I tried recreating a project with the dependencies of your last post but still works like a charm, at this point I think that probably the issue is in your code, maybe there is a loop while setting the state.


No documentation available for 6.1.0

There is no longer documentation available for versions pre 6.5.

Indeed, docs for versions older than 6.5 has been deprecated, and not implemented into our new documentation system.
I just republish the old ones. you still can get them here:
https://kodein.org/Kodein-DI/?6.1


Set log level for production env

This is required to get rid of a deprecation warning:

DEPRECATION WARNING: You did not specify a log_level in
production.rb. Currently, the default value for log_level is
:info for the production environment and :debug in all other
environments. In Rails 5 the default value will be unified to :debug
across all environments. To preserve the current setting, add the
following line to your production.rb:

lgtm


Apple CI Update

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

https://github.com/awslabs/aws-crt-swift/pull/238 <= The PR was first created and approved here. This PR is used to bring the CI changes into main.


test find scope by slug

Current coverage is 100% (diff: 100%)

No coverage report found for master at c667f66.

Powered by Codecov. Last update c667f66...6ac3171


'RNNotifications.h' file not found after updating this package to latest

Unable to build the app after upgrading the package to the latest version.
I have used this.
-    "react-native-notifications": "^2.0.6",
+    "react-native-notifications": "^3.1.2",

How can i fix this?

Fixed above by adding
node_modules/react-native-notifications/lib/ios
Getting

This is still an issue.

This is still an issue with Xcode 14.3.1


Revert "Add EmitC version of static library sample"

Reverts google/iree#7606
Conflict with https://github.com/google/iree/commit/16747ff62d46b99f092915ad45136e6e4cd497d3 and has compiler errors on MSVC that are tricky to resolve (see https://github.com/google/iree/issues/7605 and discussion on Discord)

https://github.com/google/iree/pull/7738 seems to work instead


Docs enhancements

Hi guys, first of all - great project! I started using Radium recently and I am quite happy with the results. While getting to know how to use it I stumbled upon a few minor things I thought could be improved to the docs. Here are some suggestions I have that may make it even easier to get started with Radium :)

Great, thanks for your changes! Just a couple little updates and it is ready to merge.

Hi @ianobermiller , just pushed the changes ;) Thanks!


Fixed IO exception for command sync_website_to_backend

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
Closes #1407
What's this PR do?
Fixes an IO TextWrapper error.
How should this be manually tested?

Run the sync_website_to_backend without any arguments
Verify an error is logged on the standard output

Codecov Report

Merging #1408 (bc6535c) into master (7fd8414) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1408   +/-   ##
=======================================
  Coverage   89.06%   89.06%
=======================================
  Files         234      234
  Lines        9799     9799
  Branches     1811     1842   +31
=======================================
  Hits         8727     8727
+ Misses        945      941    -4
- Partials      127      131    +4

Impacted Files
Coverage Δ

static/js/lib/api/util.ts
77.77% <0.00%> (ø)

static/js/components/StudioList.tsx
87.09% <0.00%> (ø)

static/js/pages/MarkdownEditorTestPage.tsx
23.80% <0.00%> (ø)

...ic/js/lib/ckeditor/plugins/DisallowNestedTables.ts
12.50% <0.00%> (ø)

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 7fd8414...bc6535c. Read the comment docs.


Search Feature is not implemented

There is still an issue with the search that does not actually search the code base but it will search a pre-defined set of terms it reads from a provided file for completion. The actual search action is not yet implemented.
And do we need such a search feature, anyway? Provided that the platform itself already provides for such a feature.

Perhaps we could make this a frontend for the official drupal site search? Based on the project's assigned drupal version, this would then open a browser window in the IDE, navigating to the official site?
Or should this be querying the existing code base instead? Would this not be a duplication of the already built-in IDE search features?

We should be replacing the existing icon with one that states "Search" instead of just "Drupal" or even more simpler, encodes that information into a customized version of the drupal logo, e.g. one with or behind a magnifying glass...

Just learned that it already acts a frontend  to the official drupal site :grin:. However, autocompletion is still built-in and static,  so we should make this use the official site, too.


Increase test coverage: change some tests to use KV storage instead of using mock

In an effort to increase code coverage, I want to move some tests to use the in-memory KV client instead of relying on mocks.
I might not end up doing this in all tests because it will be a major refactor and make it impossible to test some error scenarios, but I will see what can be done. I am also making some changes to the codecov configuration to exclude more files.

Codecov Report

Merging #111 (a49dbb5) into main (51975c2) will decrease coverage by 23.42%.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #111       +/-   ##
===========================================
- Coverage   50.31%   26.89%   -23.42%
===========================================
  Files          67       62        -5
  Lines        3836     3751       -85
===========================================
- Hits         1930     1009      -921
- Misses       1746     2542      +796
- Partials      160      200       +40

Components
Coverage Δ

garden-app
26.89% <ø> (-23.42%)
:arrow_down:

Closing this because I will create a new PR with the same changes + removing the non-hord storage client


Cleanup GeoTrellis server deps

We have lots of non used dependencies in this project. Sometimes (i.e. apache commons-io even two times with different names). The task is to remove unused deps and to update them in necessary.

Do y'all know about

@jisantuc this task is more trivial tbh; just to remove all unused deps and all duplicate deps from the file. It looks like we were adding it only and never removing unused once. It is also interesting to know that geotrellis-server examples for some reason still depend on Spark :shrug:

Done here https://github.com/geotrellis/geotrellis-server/pull/334


[WIP] Add black and flake8 pre-commit hooks

This is a WIP because I'm having some issues with loading flake8 plugins in my local environment. @waxlamp I'd ask that you test this yourself locally, to see if it works. Also, this change re-formats the codebase using black/flake8, so that's why there's so many changed files.
Steps to manually run the pre-commit hooks are:

Run pipenv install -d to install dev dependencies
Run pre-commit run --all-files

When I try to run the pre-commit hook, I get this error:
An unexpected error has occurred: CalledProcessError: Command: ('/home/roni/.local/share/virtualenvs/multinet-girder-0ofj-yIj/bin/python', '-mvirtualenv', '/home/roni/.cache/pre-commit/repojaxqpu0j/py_env-python3.6', '-p', 'python3.6')
Return code: 3
Expected return code: 0
Output:
    The path python3.6 (from --python=python3.6) does not exist

Errors: (none)

Check the log at /home/roni/.cache/pre-commit/pre-commit.log

Any idea what I'm missing?

@waxlamp If this looks good to you then we can go ahead and merge this


Fix resize to not switch between schemas and add permissive_schema_merge option

Why are these changes needed?
Fixes a problem in arising when there are lots of schema mismatches encountered.
Also adds permissive_schema_merge option and cli parameter
Related issue number (if any).
Fixes #429

We have agreed that the real solution is to stop all processing of files in this case, which requires an enhancement to the runtime framework.


Object Type UDMO Data Model #1905

Implementation of the object type UDMO Data Model #1905
Various changes as highlighted by Travis

Closed pending revision


spike - deploying doubtfire in codespaces

Description
This is a draft file.

Hey, @maddernd @Kachi-Okorie, please review this document and let me know if any changes need to be made - thanks.


Bucket Web - Sort chunks by thanos.downsample.resolution for better grouping

Changes
Sort series on bucket viewer page to better group compaction levels
Before

After

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Ok, should be rebased 👍

Rebased once more to make sure the DOC should still be there.


IGAPP-994: Implement rtl for pois route

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
http://localhost:9000/testumgebung/ar/locations

Test MapView Controls for RTL
Desktop: Check Panel: Icons & content in POI Detail View, ListItems in PoiList
Mobile: Check BottomActionSheet PoiDetails and PoiList

Can not test currently, as there are now pois for no language

Can not test currently, as there are now pois for no language

@SteffiStuffel you can use the production cms by entering window.localStorage.setItem('api-url', 'https://cms.integreat-app.de') in the console to test the pois.

Currently not testable as the cityModel doesn't have a bounding box. Maybe we should set a default bounding box around Germany if there is none.


fix age and xboxTitleId attributes for groupby

The original ageGroup and xboxTitleIdHex did not work with the api. The correct ones should be age and xboxTitleId

#sign-off


dsDemos: error in Matlab 2014a

SIMULATING MODEL:
Solving system using /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/solve/solve_ode_demo_cluster_3.m
Changing directory to /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/solve
Saving model parameters: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/solve/params.mat

Running simulation 1/3 (solver='rk4', dt=0.01, tspan=[0 100]) ...
Elapsed time: 0.100735 seconds.
	Data saved to /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/data/study_sim1_data.mat
	Executing post-processing function: dsPlot
		Elapsed time: 0.097045 sec
Error: Invalid or deleted object.
	 in dsAnalyze (line 237)
	 in dsSimulate/tryFn (line 1057)
	 in dsSimulate (line 751)
	 in dsDemos (line 295)
created temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757544_1
loading study file: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/studyinfo.mat
deleting temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757544_1
setting status='failed' where sim_id=1
created temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757627_1
updating simulation metadata in study file: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/studyinfo.mat
deleting temporary lock file for this process: /home/jsardid/Documents/DynaSimData/demos/demo_cluster_3/.lock_20171002112757627_1
Changing directory to /home/jsardid/Documents/DynaSimData/demos
Error using handle.handle/set
Invalid or deleted object.

Error in dsAnalyze (line 237)
        set(thisResult, 'PaperPositionMode','auto');

Error in dsSimulate/tryFn (line 1057)
            tmpresult=dsAnalyze(tmpdata,siminfo.result_functions{f},'result_file',siminfo.result_files{f},'save_data_flag',1,'save_results_flag',1,siminfo.result_options{f}{:});

Error in dsSimulate (line 751)
    tryFn(nargout)

Error in dsDemos (line 295)
dsSimulate(eqns, 'save_data_flag',1, 'study_dir','demo_cluster_3',...

Sorry for the noise, this came from me deleting a figure (trying to clean up my desktop from multiple panels, quite a lot of them opened in fullscreen) that was called afterwards.
Btw, couldn't we avoid matlab figures to be opened fullscreen? they are very annoying, at least to me...

did the sizing get fixed?

I don't think so


Interrupt the Python consumer when receiving SIGINT

Motivation
When a Python consumer is blocked on a consumer.receive() call, it will ignore the SIGINT from a Ctrl-C interruption.
The consumer appears to be frozen, and the Python process is stuck, waiting for more messages.
We need to make sure to respect the SIGINT, at least for the consumer.receive() which block undefinitely.
Fixes #667

Is this fixed and is working properly in the current (2.5.0) version?
I cannot interrupt (SIGINT, SIGTERM) consumer.receive() call.


Linter support

How is ptr not doing what you expect?
ptr doesn't support running linters.
What is your suggestion to make this better?
I want to add support for flake8 and pylint.  I'm trying to decide if it's worth supporting linters as a general, configurable thing to run, or if I should just hardcode support for flake8 and pylint and call it a day.

Happy with hard coding today - But eventually I'd like to make ptr more like flake8 in that it's more friendly for adding plugins and configuring the order things are run in. For Example:

flake8, black, mypy + unit tests / coverage
unit tests / coverage, flake8, black, mypy

I'm sure people will have opinions here ... But it seems a lot of work and not sure it's worth it.

Initial support merged. Lint away.


Self-Hosted version of vscode.dev

This has been requested multiple times and closed as "completed".
To my knowledge, it is still not possible to host a lightweight build of Visual Studio Code that runs entirely in the browser, like vscode.dev
(aka. "VS Code for the Web", "GitHub Codespaces", "Visual Studio Codespaces", "github.dev")
Related issues, supposedly completed:
#135733
#135856
#136168

Judging by https://github.com/microsoft/vscode/commit/0df549f94fb03207b215ae80fc9116e10ffc5803
The source lives at https://github.com/microsoft/vscode-dev/
  🥺
👉👈
can we haz access pls

does this work for you?
https://github.com/microsoft/vscode-test-web

@RedCMD That's very interesting! Thank you, I'm looking into it

I assume so.
you can also set it to use the https://insiders.vscode.dev/ build

Well I suppose that does settle it.


[$500] The payment popup appears after refreshing the page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.7-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Create an account, and add a free workspace
Add a reimbursement account to that workspace in NewDot
Add another reimbursement bank account via OldDot
In OldDot, create a report - it should show up as Processing, on the free workspace
Add a reimbursable expense
Reimburse (via ACH), selecting the bank account that you did NOT set up in NewDot

Expected Result:
The payment popup appears without refreshing the page
Actual Result:
The payment popup appears after refreshing the page
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/43996225/d0bf7683-c0f5-455b-b42e-0463293496f4

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fd38f0c978e98f27
Upwork Job ID:<PHONE_NUMBER>125218816
Last Price Increase: 2023-12-05

Hm, interesting!

Reimburse (via ACH), selecting the bank account that you did NOT set up in NewDot

@kavimuru - looking at the video repro, isn't this a moot point? You just select ACH and the pop up appears, there wasn't a choice either way.
@luacmartins I have a feeling this might be pusher/ related to the freePlan bank account NVP. A newly added bank account via NewDot requires a refresh of the report screen on OldDot to be "recognised" to use for reimbursement it seems.
With the changes to unravel that NVP; 1) do you think this issue will be resolved? 2) if not, is it worth moving this along now or after?

This seems pusher related. OldDot doesn't handle these updates so we need a refresh to pull the latest data. I don't think the NVP is an issue since we don't really use it in OldDot.

Yea, I think we can close it.