iOS 14 release Invalid Binary related to Invalid font when adding a Widget Extension

I've a problem with publishing app to App Store related to font issues:

ITMS-90853: Invalid font - There was an error validating the font named 'All3PIcons-Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/fonts/MaterialIcons-Regular.otf'. ‘name’ table structure.
ITMS-90853: Invalid font - There was an error validating the font named 'All3PIcons-Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/fonts/MaterialIcons-Regular.otf'. ‘name’ table usability.

Do you have any idea about this problem? @xster
Originally posted by @ltdangkhoa in https://github.com/flutter/flutter/issues/60013#issuecomment-693790004

@xster Who do we need to escalate this to?  Could this be happening for every App Store submission?

@ltdangkhoa what Xcode version did you build your IPA with?
cc @willlarche to triage potential .otf issues with material icons

Hi! Thank you for reporting this. Please let us know the Xcode version.
Was this an upload for enterprise, test flight, or production?

Hi,
Here is my Xcode & Flutter version:
[✓] Flutter (Channel dev, 1.22.0-12.0.pre, on macOS 11.0 20A5374g, locale en-US)
    • Flutter version 1.22.0-12.0.pre at /Users/krisluu/development/flutter
    • Framework revision a27c242b0e (11 days ago), 2020-09-11 17:46:57 -0400
    • Engine revision fee6f9e533
    • Dart version 2.10.0 (build 2.10.0-110.0.dev)

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0, Build version 12A7209
    • CocoaPods version 1.8.4

This issue happened as below flow:

Build from Flutter
Archive from Xcode
Distribute to App Store via Xcode
Receive an email warning about Push Notification Entitlement
Receive an email for completed processing
Setup for TestFlight internal testing -> OK, can download app from TestFlight app and app work well
Submit App Review -> receive an email that notify the issues:

ITMS-90853: Invalid font - There was an error validating the font named 'All3PIcons-Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/fonts/MaterialIcons-Regular.otf'. ‘name’ table structure.

ITMS-90853: Invalid font - There was an error validating the font named 'All3PIcons-Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/fonts/MaterialIcons-Regular.otf'. ‘name’ table usability.

Try to setup TestFlight for external testing -> receive the same email as "Submit App Review" phase

I've found a topic from Apple Developer Forum with same error code: https://developer.apple.com/forums/thread/657598 but no luck.

Thank you. Let me speak with the fonts team.

Looks like they don't like that I change the name of it to keep backwards compatibility. Let me work on a solution.

Font team working on solution. They are going to see if they can give it to us in the old name. If not, we use TTX to change the name table.

Beta release blocker? This will cause App Store rejections until there's a fix.
\cc @pcsosinski

I think you're right. I'll let Fonts know this is a blocker.

This is just 1.22? Or also older versions?

Internal ticket #169277125.

Tim, I'm not sure. I believe it was introduced in #61778

So, thanks to @ltdangkhoa for working on master and bringing this to our attention!

OK. I have a PR that hopefully will fix it. #66524

I tested it by validating on my Xcode uploader

Hi,
Last night I've built another app from scratch (same version of Xcode & Flutter as previous comment) and successfully passed the App Store Review.
There are different between 2 app building phase:

Issued app: all deployment target has been set to 14.0 with Widget extension
Successful app: all deployment target keep using default Flutter configuration (now is 9.0)

I'll try several build for better understanding on issued target

I can confirm that issues only happened when adding Widget Extension to project. So I'll wait for #66524 being merged and try again later

It's merged so please do try and report back.

Hi,
There are still issues but with another name

ITMS-90853: Invalid font - There was an error validating the font named 'MaterialIcons-Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/fonts/MaterialIcons-Regular.otf'. ‘name’ table usability.

ITMS-90853: Invalid font - There was an error validating the font named 'MaterialIcons-Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/fonts/MaterialIcons-Regular.otf'. ‘name’ table structure.

Just like my previous comment, only stuck while added Widget Extension. When remove the extension and rebuild, it passed thought the binary checking of App Store system

Ok, gtk. Now that you've isolated that condition, I can I test it like that before I upload another.

So, I have a concern that I've been unable to reproduce this error. Tell me how you added the widget extension. Maybe I'm not doing it right.

Here is my steps:

Create new Flutter App
Build iOS release from CLI
From Xcode:

New Target -> Widget Extension (Activate “widgetExtension” scheme)
Change iOS Deployment Target to 14.0
Update build number of widgetExtension target to match with Flutter Runner target

Rebuild iOS release from CLI
From Xcode:

Archive Product: Passed
Distribute App: Passed (1)

App Store Connect:

Response "Build Processing Completed" email (2)
TestFlight for Internet Group: Passed

Download app from TestFlight and run app: success with availability to add home screen widget
App Store Connect:

Select build for External Group or production version and "Submit for Review"
After several minutes: received an email with issues (ITMS-90853: Invalid font....)
Build version automatically changed to "Invalid Binary"

From my experiences, I often got issues from state (1) or (2) but with this issue, it only occur after submit app review. I've contact Apple Support and here is their short response

The invalid binary message appears when your build is received, but an issue with your build is detected. When our systems detect an issue with your build, a system-generated email detailing the issue is sent to the user who delivered it. Check your inbox and spam folders for this email. You can then redeliver a corrected build.

A/W I really appreciate with all your support in this issue

@ltdangkhoa Please do a pull and retest.
I was able to reproduce this by following your steps. I've created a font with twice as much metadata in the name table. I did a test on my side uploading to AppStore and made it all the way to review and rejection.
Let me know if this solves your problem.

@willlarche passed the Binary checking like a charm. Thanks for all your hard working on the issue 🍻

Yes! And the icons look right?
On Fri, Sep 25, 2020 at 23:42 Kris<EMAIL_ADDRESS>wrote:

@willlarche https://github.com/willlarche passed the Binary checking
like a charm. Thanks for all your hard working on the issue 🍻
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/65991#issuecomment-699322324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWNZOI7YWC2HM47TPZTFTSHVPK5ANCNFSM4RPZBRTA
.

I've just tested with a demo app. I'll apply to production app and notify you soon

Thanks!
On Sat, Sep 26, 2020 at 00:14 Kris<EMAIL_ADDRESS>wrote:

I've just tested with a demo app. I'll apply to production app and notify
you soon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/65991#issuecomment-699364058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWNZKYCBW6AQTFTOZM2D3SHVTBPANCNFSM4RPZBRTA
.

@willlarche all things are smooth right now 👌


Adding a side navbar for easier navigation of topics within a particular course

🛠 Fixes Issue

Closes #253

👨💻 Changes proposed
Added a side navbar for easier navigation of topics within html course

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📷 Screenshots

@Jayanth-MKV can u fix this

rest all is good

hey @Jayanth-MKV try to fix if
and add it in HTML,CSS,ML courses


New to cdk, trying to use the tutorial with a JAVA handler and fail

:question: Guidance Question

The Question

I was following the tutorial and at first write js handler and everything works fine. Later I want to try write a Java handler and test it. I added the java file in the same lambda directory with package name example, class name MyHandler, writing a simple java handler returning the same info as the js handler. I then go back to stack and change the props, change Runtime to JAVA_8, change handler to example.MyHandler (also tried example.MyHandler::handleRequest), and then deploy it.
But when I try to test it on lambda console, it gives ClassNotFound error, it can't find my example.MyHandler class. Anyone know what's wrong? or is it possible to add a handler version in Java to the tutorial?
Environment

CDK CLI Version:
Section:
Browser:
Language:

Other information

Hmmm I'm running into this issue as well @ydl1991, I've never created a lambda function in java before though so I'm not sure if I'm doing something wrong. I'll see if I can figure this out

This is due to Java being a compiled language. You can't deploy the code as simply as you can with js or python; you need to deploy a compiled program and its dependencies in a container or zip file.
Here's an example which does this with the CDK: Link to Stack, Link to Function
I hope this helps. We have it on our agenda to have each workshop deploy the lambda function in the same language as the workshop 😄

Thank you so much for explaining it! And yes, having each workshop deploy lambda in the same language is definitely a big help for newbies like me!

No problem @ydl1991, do you still need any help with this?

I just started looking into it, regarding how to deploy the function code as compiled program in container or zip, is it the step showing in this page? https://docs.aws.amazon.com/lambda/latest/dg/gettingstarted-package.html

still not quite get on how to mannually upload the deployed package correctly. I follow this post: https://docs.aws.amazon.com/lambda/latest/dg/java-package.html, and build the deploy package using Maven, which generates a jar file. then I go to lambda console and mannually upload the jar file to the source code section. And the test still fail.
Is there any documents you are aware of that shows the process of deploying a compiled program and in container or zip? @peterwoodworth

Gonna be honest with you @ydl1991, I've never actually deployed a lambda function in Java before. So unfortunately I won't be of much help here. Here's a tutorial which looks helpful: https://www.baeldung.com/java-aws-lambda

@peterwoodworth  thanks , I was just thinking that if my handler code need to be in a seperate package from my cdk app stack package? cuz we need to compile just the java handler code itself and deploy the jar file, right?

Were you able to figure this out @ydl1991? Sorry for leaving you hanging here

Hey @ydl1991,
Did you figure out the asset deployment for Java-based lambdas? If you haven't, consider checking out the Java examples in the cdk examples repo


Update NonSteamLaunchers.sh

Dropped a double equation mark. Restoring fixes authentication issue after clean install.

@newb23 good catch brother, i habnt been able to trst much recently but good job finding that!


Update nested_rbac.bicep

Newer schema version of Microsoft.Authorization/roleAssignments, now not a preview schema anymore.

@aavdberg thanks for the contribution, we are currently testing to move to latest version of CARML which uses new schemas

@aavdberg forgot to comment on this one, API version was updated to Microsoft.Authorization/roleAssignments@2022-04-01 during a CARML module update.


Update README

Adds screenshot and updates the explanations.
Trello: https://trello.com/c/d3RlTJwt
To be reviewed by two people from @alphagov/team-finding-things.

:question: Can we provide an example of what a policy/subpolicy actually is and how end users see them? Is subpolicy just a policy with a parent (similar to a "subtopic") or is it a distinct thing?

@MatMoore I think that is mentioned in the nomenclature? https://github.com/alphagov/policy-publisher/blob/readme-update/README.md#nomenclature

Whoops, missed that part. In that case :+1: from me.

👍


Still no exes?

I'm afraid your engine will be regarded as private until you provide exes for Windows.

Hi Gabor,
I'm so sorry that I'm just now replying. I recently started a masters program at Georgia Tech, and I've been very busy, spending 70+ hours a week between work and school. I will try to compile for windows and push the exes as soon as possible. However, I might need to wait until winter break to make another attempt.
Apologies,
Ellie


cannot make JointGroupVelocityController work

Hi all,
i'm using ros2_controllers on rolling to control a robot with the velocity interface.
My controllers config looks like this :
forward_velocity_controller:
      type: velocity_controllers/JointGroupVelocityController

forward_velocity_controller:
  ros__parameters:
    joints:
      - shoulder_pan_joint
      - shoulder_lift_joint
      - elbow_joint
      - wrist_1_joint
      - wrist_2_joint
      - wrist_3_joint
      # interface_name: velocity

when I show the hardware interfaces with the CLI command, it shows:
command interfaces
	elbow_joint/position [available] [unclaimed]
	elbow_joint/velocity [available] [claimed]
	shoulder_lift_joint/position [available] [unclaimed]
	shoulder_lift_joint/velocity [available] [claimed]
	shoulder_pan_joint/position [available] [unclaimed]
	shoulder_pan_joint/velocity [available] [claimed]
	wrist_1_joint/position [available] [unclaimed]
	wrist_1_joint/velocity [available] [claimed]
	wrist_2_joint/position [available] [unclaimed]
	wrist_2_joint/velocity [available] [claimed]
	wrist_3_joint/position [available] [unclaimed]
	wrist_3_joint/velocity [available] [claimed]

The controller picks up the commands published on "/forward_velocity_controller/commands", I checked by printing them in the forwardcommandcontroller Update call.
After the controller receives the commands, they are not passed down to the hardware interface.
I also tested the hardware interface extensively, and I never receive commands from the controller.
Am I doing something wrong?

Running the controller manager with level debug, shows this:
Deduced interface prefix 'shoulder_pan_joint' - searching for the controller with the same name.
[ros2_control_node-1] [DEBUG] [1677662455.266703502] [ControllerManager::utils]: Required command interface 'shoulder_pan_joint/velocity' with prefix 'shoulder_pan_joint' is not reference interface.
[ros2_control_node-1] [DEBUG] [1677662455.266711384] [ControllerManager::utils]: Deduced interface prefix 'shoulder_lift_joint' - searching for the controller with the same name.
[ros2_control_node-1] [DEBUG] [1677662455.266718056] [ControllerManager::utils]: Required command interface 'shoulder_lift_joint/velocity' with prefix 'shoulder_lift_joint' is not reference interface.
[ros2_control_node-1] [DEBUG] [1677662455.266725295] [ControllerManager::utils]: Deduced interface prefix 'elbow_joint' - searching for the controller with the same name.
[ros2_control_node-1] [DEBUG] [1677662455.266731897] [ControllerManager::utils]: Required command interface 'elbow_joint/velocity' with prefix 'elbow_joint' is not reference interface.
[ros2_control_node-1] [DEBUG] [1677662455.266739593] [ControllerManager::utils]: Deduced interface prefix 'wrist_1_joint' - searching for the controller with the same name.
[ros2_control_node-1] [DEBUG] [1677662455.266746390] [ControllerManager::utils]: Required command interface 'wrist_1_joint/velocity' with prefix 'wrist_1_joint' is not reference interface.
[ros2_control_node-1] [DEBUG] [1677662455.266753378] [ControllerManager::utils]: Deduced interface prefix 'wrist_2_joint' - searching for the controller with the same name.
[ros2_control_node-1] [DEBUG] [1677662455.266759739] [ControllerManager::utils]: Required command interface 'wrist_2_joint/velocity' with prefix 'wrist_2_joint' is not reference interface.
[ros2_control_node-1] [DEBUG] [1677662455.266766476] [ControllerManager::utils]: Deduced interface prefix 'wrist_3_joint' - searching for the controller with the same name.
[ros2_control_node-1] [DEBUG] [1677662455.266772893] [ControllerManager::utils]: Required command interface 'wrist_3_joint/velocity' with prefix 'wrist_3_joint' is not reference interface.

I cannot tell if it's related.
Thanks

You interfaces are claimed. Therefore, they are passed. (I am quite certain because having a bug there would influence all the tests and users we have).
Which version, hardware interface are you using. What other relevant details you can provide?

I'm using version 2.37.1.
The commands I published with "ros2 topic pub" were often ignored.
I could see the commands with "ros2 topic echo", so I think this is JointGroupVelocityController's problem.

Hi @daihen-hijikata. Can you reproduce your issue with ros2_control_demos? If not, can you create a MWE for us to reproduce it?

a mimimum working example, something you can share with us where we can reproduce your issue. otherwise we cannot help you..

I created 41joint sample robot. But my issue was not reproduced. I cannot understand that my robot often ignore the command...

are you using the same environment (DDS vendor, network setup etc)?

I'm using Isaac Sim.
The control_manager run very high frequency when I set use_sim_time parameter.
similar to https://github.com/ros-controls/ros2_control/issues/859
At very high frequencies, the commands may be often ignored.
My sample is here.
https://github.com/hijimasa/isaac-ros2-control-sample/tree/control_manager_high_freq_issue
diffbot often ignore the command at very high frequencies.

I can reproduce my issue.
Below is the example.
issue_sample.zip
This video was taken when the command was ignored.
https://github.com/user-attachments/assets/91c35172-17fc-46b9-bb3a-4e55f182128c

I was not able to reproduce it with this publisher
test_publisher.zip
I've read the linked PR above and this is for sure related to the update rate of the cm, and subsequently, the controllers. In my environment, the joint_states are sent with ~1600Hz. I guess you are using the topic_based_ros2_control with isaac sim?

I'll close this issue here as the new issue seems to be not related at all to the initial question, and there was no update for half a year now.

Where should I comment? Should I create new issue in ros2_controllers repo?

IMHO that the behavior you encountered is a consequence of the update rate issue with use_sim_time and not related to the ros2_controllers repo.


Wrong handling of UTF-8 text in the XML module

I am experiencing issues when parsing HTML containing non-ASCII characters. For example:
require "xml"

str = "
České psaní
"
html = XML.parse_html(str)
el = html.xpath("/html/body/p")
if el.is_a?(XML::NodeSet)
	print(el[0].text)
end

The expected output is České psaní, but I get ÄeskÃ© psanÃ, which seems as if the correct UTF-8 bytes were re-interpreted as ISO-8859-1 (or similar) encoding and converted into UTF-8 again.
The issue appeared all of sudden in a binary which had previously worked correctly, i.e. without recompilation of my program. Therefore, I suspect an external library is to blame, perhaps libxml2, but I'm posting here because I am not sure, and also because I do not know how to test libxml2 directly.
Can anybody confirm the issue? Is this a Crystal issue or not?
Thanks in advance.
My specs are:
Operating system: Manjaro (Linux 5.15.120-1)
Crystal version:
Crystal 1.8.2 (2023-05-11)

LLVM: 15.0.7
Default target: x86_64-pc-linux-gnu

libxml2 version: 2.11.4-1

I can reproduce this behaviour with libxml2 2.11.4. But 2.10.4 is still correct.
libxml2's changelog notes for 2.11

Refactoring has begun on some buffering and encoding code with the goal of
simplifying this part of the code base and improving error reporting.

Maybe that broke something?
This seems to appear only with htmlReadMemory, not xmlReadMemory.

I have filed an upstream issue: https://gitlab.gnome.org/GNOME/libxml2/-/issues/570
Let's see what they have to say.

Thanks for the prompt resolution. I am confident the issue is solved.


[DO NOT MERGE YET] Revert "HACK for SWDEV-173477"

This reverts commit d941f19399e9b4f040aec5d41123e7073913cbc3.
Since relevant fixes in lower level component such as ROCR runtime and ld.lld has been introduced to ROCm master, filing this PR to remove this undesirable workaround in HIP.
Please do NOT merge the PR yet, as no HIP directed tests can cover this scenario just yet. I'd like to make a couple more local verification based on TensorFlow XLA first.

lower-level components and associated TensorFlow XLA tests have been verified by CQE team. removing [DO NOT MERGE YET] tag.

@gargrahul to my understanding the fix in ROCR would only be in ROCm 2.4, NOT ROCm 2.3. So could you make sure this PR does NOT get into release branch for 2.3?

@whchung we have already branched for 2.3 and will be cherry-picking very strictly into release branch, so I believe this won't get in 2.3 release.

2.3 hasn't been released yet so isn't it too early to revert this now?

@scchan Yes so that's why I asked @gargrahul to ensure this patch doesn't enter ROCm 2.3. It should be ok to have it landed onto master as TF CI uses stock ROCm releases whenever possible (albeit sometimes we have to deviate from it), rather than HIP mater.

To clarify, we do need PR #981 to be landed in ROCm2.3 release package.
Here's the expectations:
ROCm2.3: include HIP PR#981, not include PR#1004
ROCm2.4: include HIP PR#1004

@whchung  there are users that use HIP  master on ROCm 2.2 or 2.3 so it seems that reverting this patching should be cherrypick into a future 2.4 branch of HIP and then lands in master with the release of ROCm 2.4.

@scchan I see. Then let me add [DO NOT MERGE YET] back. @gargrahul sorry for bring your confusion.

@whchung Is it possible to guard this change with ROCr work week (or equivalent version) ?

To clarify, we do need PR #981 to be landed in ROCm2.3 release package.
Here's the expectations:
ROCm2.3: include HIP PR#981, not include PR#1004
ROCm2.4: include HIP PR#1004

@sunway513 Regarding #981 in 2.3, @mangupta has already initiated the cherry-picking process. Let us follow up internally.

@gargrahul Looking at ROCR CMake I don't believe it has workweek feature, and its version number seem to bump relatively slow:
https://github.com/RadeonOpenCompute/ROCR-Runtime/blob/master/src/CMakeLists.txt#L76
So I think it's not possible to do the checks you proposed based on ROCR version.

@gargrahul @mangupta
We need this PR merged to roc-master ASAP.
Could you make sure this PR would be landed on roc-master before ROCm2.4 is branched?

Removing "[DO NOT MERGE]" prefix per request from @sunway513


Add cupy to requirements.txt

Cupy is somehow messy when it comes to checking if it's already installed. So if it's already installed on your system, it might try to reinstall it. For this reason, I recommend installing cupy separately if you don't already have it, rather than having it as a requirement.


Invalid escape sequence fixes

Thanks for submitting a PR!
Please make sure to check for the following items:

[ ] Add unit tests and integration tests where applicable.
If you've added an error code or changed an error code behavior,
you should probably add or change a test case file under tests/test_cases/ and add
it to the list under tests/test_definitions.py.
If you've added or changed a command line option,
you should probably add or change a test in tests/test_integration.py.
[x] Add a line to the release notes (docs/release_notes.rst) under "Current Development Version".
Make sure to include the PR number after you open and get one.

Please don't get discouraged as it may take a while to get a review.

Done, hope I got it right now. Let me know if you'd prefer a clean rebase instead of what's here now.

Looks correct, but what issue does this fix exactly? Can you add some tests?

Avoids warnings:
$ python3 -Wd -c 'foo = "[^\.].*"'
<string>:1: DeprecationWarning: invalid escape sequence \.

...which will become errors at some point, https://docs.python.org/3/reference/lexical_analysis.html#string-and-bytes-literals

Changed in version 3.6: Unrecognized escape sequences produce a DeprecationWarning. In some future version of Python they will be a SyntaxError.

Wrt testing, I think it's indirect and kind of out of scope for this PR, but running tests with warnings enabled and someone looking at the logs would do somewhat. Or maybe turn them to errors. Probably requires upgrading pytest to >= 3.1 and... something, maybe a -Wd flag to pytest (although that didn't catch these for some reason in a quick test).

In python3.8 this becomes a SyntaxWarning and in future versions this will be a SyntaxError

Re: tests, now that we have py3.8 in the CI, should we just add a -W flag to pytest so that it fails on any warnings that are emitted? This will ensure that there are no future regex additions like these.

Closing this as it has been inactive for a while. Please feel free to re-open if this is still relevant.

it looks like #445 superseded this


Multiple spelling and spacing issues on the timeline - Addressing iss…

Issue #50 plus some additional spelling and spacing issues.


How to count entries in Affected Assets?

Hi,
since pwndoc is storing the Affected Assets in an editor field (for ordered list e.g.: "scope" : "

http://scanme.nmap.org

https://scanme.nmap.org

"), is there a way to check how many entries are in the list?
I would like to do something like:
if amount("
") > 1 do:
"affected components are: ..."
else if amount("
") < 2 do:
"affected component is: ..."

I looked up the docxtemplater documentation, but there is only an example for length check, if the data is being stored in another format: https://docxtemplater.com/demo/#conditions

You may count lines of Affected Assets using :
{affected | lines | length}

Note : this work if you doesn't put assets in a (ordered) list :

If you want to keep hosts in an ordered list, you can opt for a loop like :
{#affected | loopObject}
1.	{@value}
{/}

You can also opt for custom field. Finally if you'd like to parse the html of a complex text input, I think you'll need to code your own custom filter (see https://github.com/pwndoc-ng/pwndoc-ng/blob/master/backend/src/lib/report-generator.js#L110).


Button widget active, hover and clicked states are not defined on Figma and need design review

Creating a separate issue for this since it needs design change

Got that. Is this for the buttons on Appsmith or is it for the button widget that the users can drag and drop?

Widget that user can grab and drop

Can I get permissions to view the document @momcilo-appsmith ?

@ramsaptami Just following up here, did this get resolved?

Yes, this seems to be resolved now @momcilo-appsmith . Changes have been implemented and present on current release. I have a quick question about the difference between focus and active states though - when do we see this @albinAppsmith

Closing this since this has been implemented


failing to remove x3daudio_7.dll

i had a problem with the dll file getting detected as a trojan before but allowed it after searching it up, now it cant seem to remove that file either, i can also install and update cache just fine

You need to add an exemption to your antivirus or otherwise fix it interfering with this file (which is used by mint).


bug

When using only the Segmentation method, Energy and DAabsCoef and IAabsCoef columns are all NaN, but there is no problem if other Method combinations are selected.

This was a simple syntax error. I believe the issue is resolved but please raise any other issues you might find. Cheers.


Add some convenience redis environments

I thought maybe it's nice to provide some default environment variables for the users?

I thought about it again and decided not to include further environment variables.


Feature Request: Add support for database/sql package

Is your feature request related to a problem? Please describe.
Automatically generate spans for calls to SQL databases
Describe the solution you'd like
Add a new instrumentor for datadbase/sql under https://github.com/open-telemetry/opentelemetry-go-instrumentation/tree/main/pkg/instrumentors/bpf
Describe alternatives you've considered
Manual Instrumentation

Is the community accepting contributions for packages other than http, grpc and mux? Is there any documentation or guidelines on how to extend this further? It would be awesome if we could automatically generate spans for popular open-source packages, such as the list under https://github.com/DataDog/dd-trace-go/tree/main/contrib. I am happy to contribute, would like to know how I can get involved in this project?

Is there any documentation or guidelines on how to extend this further?

I suggest starting by getting familiar with both https://github.com/open-telemetry/opentelemetry-go-instrumentation/blob/main/docs/how-it-works.md and https://github.com/open-telemetry/opentelemetry-go-instrumentation/tree/main/docs/getting-started.

I proposed to remove it from beta during last SIG meeting and nobody disagreed.


yii\log\EmailTarget sends Emails for extended HttpException even if except is filled.

What steps will reproduce the problem?
I have an EmailTarget  and doesn't wanted to get the email of an extended HttpException.
My extended Exception:
namespace app\components\exceptions;
class CallSubmissionException extends HttpException
{
}

My throw:
throw new \app\components\exceptions\CallSubmissionException(410,Yii::t('app', 'The requested submission call has ended.'));

My EmailTarget config:
  $config['components']['log']['targets']['email'] = [
    'class' => 'yii\log\EmailTarget',
    'except' => [
        'app\components\exceptions\CallSubmissionException',
        'app\components\exceptions\CallSubmissionException:*',
        'app\components\exceptions\CallSubmissionException:410',
    ]

What is the expected result?
That the Error Email isn't sent.
What do you get instead?
I get the error email.
Only when I add 'yii\web\HttpException:410'or 'yii\web\HttpException:*', for except the email wouldn't been sent.
When I use CallSubmissionException extends yii\base\UserException (and remove the status code from the exception throw) i could use this except   'app\components\exceptions\CallSubmissionException' and the email isn't sent which is correct.
But then the HTTP error which is returned to the browser is 500 instead of 410.
It is also 500 when I use exactly the code from the original HttpException.
class CallSubmissionException extends UserException
{
public $statusCode;
public function __construct($status, $message = null, $code = 0, \Exception $previous = null)
{
$this->statusCode = $status;
parent::__construct($message, $code, $previous);
}
   public function getName()
   {
       if (isset(Response::$httpStatuses[$this->statusCode])) {
           return Response::$httpStatuses[$this->statusCode];
       }
       return 'Error';
   }

}
The email isn't set but the http error code is not the given one.
So i guess there is some handling of the HttpException for the http error code elsewhere.
So in order to return the correct http error code I must extend from HttpException.
But then i couldn't disable for a specific HttpException at one point but only by the error code.
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.1.4

Operating system
Windows 10

Is outdated for current code.

Is outdated for current code.


main.yml: add coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2b84211). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   76.22%
=========================================
  Files             ?        6
  Lines             ?      307
  Branches          ?        0
=========================================
  Hits              ?      234
  Misses            ?       73
  Partials          ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Restarting clickhouse fails constantly

Hi, I am running a single pod clickhouse server using this operator, and when I try to restart the server by manually deleting and creating the pod, it seems trying to load tables, and always stuck at TablesLoader: 99.6% something, then the pod automatically restart, when restart times exceed 7 it will on the state of crashbackoff.
I wonder what's the block? is there any settings I can relax the constraints so it can restart properly?
Thanks

You could define a custom podTemplate with custom readinessProbe and livenessProbe
something like this
apiVersion: clickhouse.altinity.com/v1
kind: ClickHouseInstallation
metadata:
  name: custom-probes
spec:
  configuration:
    clusters:
    - name: default
      layout:
        replicasCount: 1
        shardsCount: 1
  defaults:
    templates:
      podTemplate: custom-probes-pod-template
  templates:
    podTemplates:
    - name: pod-template
      spec:
        containers:
        - image: clickhouse/clickhouse-server:22.8
          imagePullPolicy: IfNotPresent
          name: clickhouse
          livenessProbe:
            httpGet:
              path: /ping
              port: 8123
              scheme: HTTP
            initialDelaySeconds: 600
            periodSeconds: 30
            timeoutSeconds: 10
            failureThreshold: 10
          readinessProbe:
            httpGet:
              path: /ping
              port: 8123
              scheme: HTTPS
            initialDelaySeconds: 600
            periodSeconds: 30
            timeoutSeconds: 10
            failureThreshold: 10


feat(css_formatter): Bootstrap the CSS Formatter!

Summary
Styles!
This PR bootstraps all of the infrastructure for biome_css_formatter, including:

the biome_css_formatter crate itself
configuration through biome_cli
configuration through biome_service
adding CssFileSource to be able to load css files as well as text from stdin
adding some codegen logic to handle css things specifically

Also included are a few other things that stuck out while I was going through all of the files:

Moved JsonParserSettings out of file_handlers/javascript.rs and into file_handlers/json.rs.
Fixed some typos in various places that were relevant
Updated READMEs and comments to reference the appropriate crate (i.e., "run cargo test biome_json_formatter", but it referenced biome_js_formatter before).

This doesn't do any actual formatting yet! It's just the framework so that we can start writing tests and implementing the formatting itself in the future.
Going through this whole process, I definitely think we need to reduce the amount of boilerplate needed to add a new language. Things like configuration options have to get duplicated in like 7 places currently, there's a lot of duplicated code across languages that could possibly be done with generics? Maybe some other things as well. I'm not totally sure, but I'm positive that it's not a great experience if we ever want to open up a plugin system. But that's something to solve another day!
Test Plan
All the tests are passing and utilize CSS configuration options and such to make sure all of that is wired up correctly, plus the basic spec and smoke test for biome_css_formatter passes.

Thank you @faultyserver , that's exciting!!
How do you plan to release this to the users? If we decided to merge the PR as is, users would think that the formatter would be ready for format CSS files, instead it's not like this.
My suggestion would be to hide the CLI options, via the hide macro option, and keep CSS files still unhandled by the workspace.

How do you plan to release this to the users? If we decided to merge the PR as is, users would think that the formatter would be ready for format CSS files, instead it's not like this.
My suggestion would be to hide the CLI options, via the hide macro option, and keep CSS files still unhandled by the workspace.

That sounds like a good plan to me. I think hiding the CLI options is pretty straightforward, but I'm not sure about how to wire up the workspace for that. I could just remove the capabilities maybe? That way the configuration is still valid to add and use, it just won't actually do anything when it runs. It might be strange for users to be able to configure something that doesn't currently "exist" from their perspective, but I figure that's probably fine to allow, since most people won't even notice that it can be added yet.

That's actually pretty simple, when retrieving the language for the file extension, we can just return Unknown and the logic should just work like the CSS files are still unrecognised

Okay, with the hiding in place, running format on a css file continues to show that it is "unhandled", which i think is what we want:

As a cherry on top, we should update the website, language support page, and put the hourglass emoji on the format cell


Fonts not rendering on Mac

Installed via pip3
I get the following error: Unable to revert mtime: /Library/Fonts when running frc-characterization drive new
This is how it renders:

Is this a duplicate of #46?

Looks like it to me… Is @kushagharahi running pre-Catalina macOS? If they are then it’s almost certainly the same bug.
(The workaround is switching out of dark mode, btw.)

Yes this is a duplicate of #46. I tried to make sure there weren't dups, but I failed.
Thanks for the help guys! I love the tool.


sing-box: 1.10.1 -> 1.10.2

Diff: https://github.com/SagerNet/sing-box/compare/v1.10.1...v1.10.2

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Superseded by https://github.com/NixOS/nixpkgs/pull/359561


Fix data refetching

Interpreted the cause of the "roadmap users not updating on email change" -problem wrong. Actually the user's new email would not get updated in any project they are in. Should be fixed now.


Arguments that are not updated by the function

Wouldn't it be better to make variables like the title of this issue explicitly constant?
For example,  Pmodel  to input for encode:
https://github.com/diegodox/rangecoder-cpp17/blob/7c0b48da9a885e428a4bc96600c3f604912c0451/rangecoder.h#L168
->
auto encode(const PModel &pmodel, int index) -> int

@hiroki-kojima
Seems nice. Do you plan to create pull request?


Support lz4 WAL compression

Problem
Reduce WAL size by enabling compression of FPI.
Summary of changes
Support lz4 compression of FPI in WAL records in page server wal decoder
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Some points:

lz4 is available since postgres 15, as is zstd. before, there was only builtin lz which was apparently quite slow. better we don't use that one.
only a small fraction of our databases are on postgres 14, so I think it's okay to use 15+ compression methods
zstd is in general better compressed than lz4 but lz4 offers better compression and decompression speeds
the async-compression crate does not have lz4 support, it only has zstd support

it is questionable to use that crate anyways.

why? could you expand? we use it in many places.

it is questionable to use that crate anyways.

why? could you expand? we use it in many places.

Compression is CPU intensive task. Not so much reasons to make it async. It can be useful  if we have existed async stream (file, socket,...) and what to add compression to it. But for example in case of WAL compression, or more precisely - compression of FPI in WAL records) we can just use compress/decompress block functions. And not async API is needed here.

CPU intensive tasks also need to be made async because you must yield to the executor regularly,

This is true in general, but it looks like this decompression call has a bounded runtime because it's only handling one page at a time.  I would suggest just adding a  tokio::task::yield_now() after the decompression, so that if we have a WAL record with many blocks, we are not hogging the executor for the entire decode of the WAL record, but just for one block's decompression work at a time.

I would like to see at least one test in this PR before merging it.

CPU intensive tasks also need to be made async because you must yield to the executor regularly, whether it's due to waiting on I/O or due to CPU work.

Please notice that we are decompressing just one 8kb page. And decompressor is not reading this page from the disk - it is already present in memory. So there is completely no sense to make this code async.

I would like to see at least one test in this PR before merging it.

Hmm... Not clear what do you want to check? Been enabled, decompression will be used in ALL tests. So I do not see any sense to write some particular test specially for decompression. The only reason  may be to estimate  reduce of WAL size because of using compression. And it greatly depends on workload.

Hmm... Not clear what do you want to check? Been enabled, decompression will be used in ALL tests. So I do not see any sense to write some particular test specially for decompression.

I'm missing the part where it is enabled -- is that something implicit on the postgres side?  What I was expecting in a test was to run postgres with some particular config that enables compression, but I don't know the mechanism by which postgres decides to use it.

I'm missing the part where it is enabled

I suppose where the WAL is being created, i.e. on the enpoint?

At least for pgbench -I -s 500 wal compression doesn't cause some noticeable difference in size:

wal_comperssion
wal size
storage size

no
6.3G
6.7G

lz4
5.9G
6.2G

It would be great to have at least one test that runs basic pgbench with compression enabled.

I have also added compression page sent by page server to compute.
At least for local Neon installation it doesn't have some noticeable impact on performance.
Time of the following query:
 select sum(abalance) from pgbench_accounts;

is reduced from  17264 msec to  16155 msec

I have also added compression page sent by page server to compute. At least for local Neon installation it doesn't have some noticeable impact on performance. Time of the following query:
 select sum(abalance) from pgbench_accounts;

is reduced from 17264 msec to 16155 msec

did you test with network overhead? Or just local setup w/o network delays?
Did you compress all the requests? Because I thought about having threshold in which it makes sense to compress

did you test with network overhead? Or just local setup w/o network delays?

Just local setup. Certainly with real network connections effect should be more noticeable.

Did you compress all the requests? Because I thought about having threshold in which it makes sense to compress

All GetPage responses.

So overall this is very promising. Lots of different things are being discussed, I'll try to dissect this to separate features to have a clear path forward:

Supporting WAL compression, where PostgreSQL compresses full-page images being written to WAL. This is controlled by the wal_compression GUC. The pageserver does not currently support digesting such compressed records, so it will fall back to storing the record as is, and use Postgres wal-redo process to WAL redo such pages. That's inefficient especially if one WAL record modifies multiple pages, because we will duplicate the whole record, with images for all of the modified pages, for each of the pages being modified. I think that can arise on CREATE INDEX for example. It would be nice to craft a test for that worst case scenario.
Even though we don't set wal_compression in the computes today, it would be good to support it. Firstly, we could then consider setting it. Secondly, as we implement support for streaming WAL from an external system to neon, the external system might have wal_compression set, whether we like it or not.
For this, we should support both LZ4 and zstd, because we don't necessarily control what the wal_compression has been set to in the originating system. But we don't necessarily need to store the pages in a compressed format, we could just decompress the pages as they're digested.

Compress page images stored in image layers. The benefit is obvious: compressed data requires less space.
Compression is always a tradeoff between CPU and disk space. We can choose to use LZ4 or zstd. I'm leaning towards zstd, it's a good and flexible algorithm that allows tuning for different tradeoffs. And I'd love to do dictionary mode compression, because it's cool technology :-). You did some earlier experiments with that at https://github.com/neondatabase/neon/pull/1595. Seriously speaking, I'm not sure if the dictionary mode is worth the complexity. And any compression is better than nothing, really.

Support "image+hole" format of storing and passing around pages. This would save some disk space without implementing full-blown compression. Probably doesn't gain much if we do compression anyway. But still it would be nice to have such a format in the pageserver, and in the pageserver-compute protocol, to save some memory and network bandwidth.

Support storing pages in the same compressed format that Postgres uses with wal_compression, without having to recompress them. Recompressing might be OK in practice, but it sure seems silly. The main problem is that PostgreSQL compresses the image before updating the LSN on the page, but we'd like to store the image with the new LSN. One solution is to store the new LSN in addition to the compressed image. After the image is decompressed, update the LSN.

Compressing delta layers. Similar benefits to compressing image layers. Not part of this patch, but would probably be beneficial just like compressing image layers. Can be done separately later.

Compressing images sent over the wire. Nice feature. Orthogonal to the other parts though. It is included in this PR, but I'd suggest to leave it out, to reduce the scope and focus on the more important parts.

Migration path. We need to be able to roll back deployments, so this needs to have feature flags so that we can deploy the code without writing anything in the new format yet. Later, when we're confident that don't need to roll back, we can enable the feature for some subset of tenants, or all.

I suggest the following steps forward:

A PR to support digesting WAL pages compressed in PostgreSQL with wal_compression. Decompress them in the pageserver and digest. This doesn't require any storage format changes, so there's no worry of backwards-compatibility. For this, I'd like to see some tests to demonstrate the benefit in the best case, like at a CREATE INDEX that emits lots of full-page images in a single record.

A PR to implement zstd or lz4 compression of image layers. It needs to include a pageserver option so that it can be turned on and off, on a per-tenant basis. Default to off for now. Pay careful attention to the storage format changes, because we'll be stuck with the new format for a long time. It's OK to not include points 3. and 4. above, but let's pay close attention to the storage format, so that those will be straightforward to add on later. Include performance tests to demonstrate the effect on disk size, but also on CPU usage. How much CPU will we burn on the compression? I believe our pageservers currently have plenty of spare CPU, but let's measure it.

More PRs for the points 3-6 above. They're not urgent.

Right now this PR support compression of

WAL FPI
Page images in inmemory layer
Page images in delta layer
Image_layer pages
get_page response

doesn't require some API of format changes.
and 3) requires adding CompressedImage to Valuer enum because this layers are serlialized using serde.
Not sure if we need image compression for this layers:
From one hand: them are stored on disk in PS and S3  and are transferred fro PS to S3.
From the other hand - this layers are somehow intermadiate and most likely we have to extract and decompress image to be able to apply wal records to it and perform page reconstruction.

In case 4) we really need special tag for compressed images (and it is added by this PR).

It needs to include a pageserver option so that it can be turned on and off, on a per-tenant basis.

I wonder if it should be part of TenantConfig or implemented somehow else. Should we enable/disable compression for particular tenant or region? Right now there is no TenantConfig available in blob_io where compression is performed.
And there is no nice way to pass it here. I do not think much sense in enabling/disabling compression for particular tenant. So many be some global feature flag which can be used to turn compression on/off for particular region is better choice?

A PR to implement zstd or lz4 compression of image layers.

I do not think that we should support different compression algorithm, i.e. lz4 and std. From my experiments lz4 provides the best size/speed ratio: it is much faster than competitors and difference in compressed size is not so large.
Also I do not see much sense in splitting this PR into 3: separating 1) and 5)

After thinking a little bit more and observing problems with building pg14 (which doesn't;t support lz4 compression)
I decoded to create separate PR for compression only image layers:
https://github.com/neondatabase/neon/pull/7091


Unit Testing = Can't create tenant outside the public schema. Current schema is test.

Hello,
I am attempting to write a unit test which involves creating a tenant. However, it errors with the following:
Creating Tenant
Can't create tenant outside the public schema. Current schema is test.

My tests.py contents look like this:
import json
from django.core.urlresolvers import resolve, reverse
from django.db import transaction
from django.test import TestCase
from django.utils import translation
from django.contrib.auth.models import User
from django.contrib.auth import authenticate, login, logout
from rest_framework import status
from rest_framework.test import APIClient
from rest_framework.test import APITestCase
from rest_framework.authtoken.models import Token
from api.views import authentication

from django.core.urlresolvers import resolve, reverse
from django_tenants.test.cases import TenantTestCase
from django_tenants.test.client import TenantClient

TEST_USER_EMAIL =<EMAIL_ADDRESS>TEST_USER_USERNAME = "yyy"
TEST_USER_PASSWORD = "xxx"

class APIClientTestCase(APITestCase, TenantTestCase):
    fixtures = [
        # 'banned_domains.json',
        # 'banned_ips.json',
        # 'banned_words.json',
        # 'groups',
        # 'permissions',
    ]

    @classmethod
    def setUpTestData(cls):
        user = User.objects.create_user(  # Create our User.
            email=TEST_USER_EMAIL,
            username=TEST_USER_USERNAME,
            password=TEST_USER_PASSWORD
        )
        user.is_active = True
        user.save()
        Token.objects.create(user_id=user.id)  # Create our User's Token.

    @transaction.atomic
    def setUp(self):
        translation.activate('en')  # Set English.
        super(APIClientTestCase, self).setUp()

        self.user = User.objects.get()
        token = Token.objects.get(user__username=TEST_USER_USERNAME)
        self.unauthorized_client = TenantClient(self.tenant)
        self.authorized_client = TenantClient(self.tenant, HTTP_AUTHORIZATION='Token ' + token.key, format='json',)

    @transaction.atomic
    def tearDown(self):
        users = User.objects.all()
        for user in users.all():
            user.delete()
        tokens = Token.objects.all()
        for token in tokens.all():
            token.delete()

    @transaction.atomic
    def test_create_tenant(self):
        from base.models import Client

        # First tenant.
        tenant = Client(
            schema_name='demo',
            name='Den Demo',
            paid_until='2016-12-05',
            on_trial=True
        )
        try:
            print("Creating Tenant")
            tenant.save() # migrate_schemas automatically called, your tenant is ready to be used!
        except Exception as e:
            print(e)

Do you guys have any ideas how I can create a unit test where I can create a tenant?
Thanks!

Addendum. Also I attempted to run the command from console and I get the same error:
    @transaction.atomic
    def test_api_create_tenant_with_success(self):
        from django.core.management import call_command
        call_command('create_tenant', '--domain_url=newtenant.net --schema_name=new_tenant --name=new_tenant', verbosity=0, interactive=False)

I get this error:
Traceback (most recent call last):
  File "/usr/local/Cellar/python3/3.4.2_1/Frameworks/Python.framework/Versions/3.4/lib/python3.4/contextlib.py", line 30, in inner
    return func(*args, **kwds)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/api/tests/test_client.py", line 68, in test_api_create_tenant_with_success
    call_command('create_tenant', '--domain_url=newtenant.net --schema_name=new_tenant --name=new_tenant', verbosity=0, interactive=False)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/env/lib/python3.4/site-packages/django/core/management/__init__.py", line 119, in call_command
    return command.execute(*args, **defaults)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/env/lib/python3.4/site-packages/django/core/management/base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/env/lib/python3.4/site-packages/django_tenants/management/commands/create_tenant.py", line 58, in handle
    tenant = self.store_tenant(**tenant_data)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/env/lib/python3.4/site-packages/django_tenants/management/commands/create_tenant.py", line 85, in store_tenant
    tenant = get_tenant_model().objects.create(**fields)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/env/lib/python3.4/site-packages/django/db/models/manager.py", line 122, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/env/lib/python3.4/site-packages/django/db/models/query.py", line 401, in create
    obj.save(force_insert=True, using=self.db)
  File "/Users/anotherguy/Developer/someguy/py-the-new-project/env/lib/python3.4/site-packages/django_tenants/models.py", line 76, in save
    "Current schema is %s." % connection.schema_name)
Exception: Can't create tenant outside the public schema. Current schema is test.

You have you answer at Exception message, you are on schema test, you need to switch tenant to public schema. Change it using db connection, or  by Tenant(schema_name='public').activate() or by Tenant.objects.get(schema_name='public').activate() or whatever, there is plenty of ways to switch tenant to public schema ;)
I like using descriptors and with: smth like:
with Tenant(schema_name='public'):
    # Create tenant in this block
# Here would be restored old tenant

then you don't need to restore previous tenant, as it would be restored automatically.
In first post, you used TenantTestCase which sets schema to test schema, and in second post looking at traceback it's the same problem. So this issue should be closed, as there is no real issue. I'm creating and deleting many tenants each day like this ;)
Please read Exceptions next time.

Thank you very much @kosz85. This completely fixed my issue! I do have to admit, there is an opportunity to improve the documentations so future developers won't make my mistake. I think something like this appended to the Tests section would do the trick:
(In: https://django-tenants.readthedocs.io/en/latest/test.html )
Tests
Running the tests
...
Updating your app’s tests to work with django_tenants
...
Additional information
...
Unit Testing with Different Schemas (NEW SECTION)
Unit testing with different schemas is possible and works as follows. Let's assume your TenantMixen model looks as follows:
models.py
from django.db import models
from django.contrib.auth.models import User
from django.utils.translation import ugettext_lazy as _
from django_tenants.models import TenantMixin, DomainMixin

class Organization(TenantMixin):
    # General Information.
    created = models.DateTimeField(auto_now_add=True)
    last_modified = models.DateTimeField(auto_now=True)
    owner = models.ForeignKey(
        User,
        help_text=_('The user whom owns this thing.'),
        blank=True,
        null=True
    )
    name = models.CharField(
        _("Name"),
        max_length=255,
        help_text=_('The name of the item.'),
        blank=True,
        null=True,
    )

    # Payment Information.
    on_trial = models.BooleanField(default=False)
    paid_until =  models.DateField(
        auto_now_add=True,
        blank=True,
        null=True,
    )

    # Django-Tenant Information.
    auto_create_schema = True
    auto_drop_schema = True

class Domain(DomainMixin):
    pass

Now to unit test using different schema names, follow this example:
tests.py
from django.db import transaction
from django.utils import translation
from django_tenants.test.cases import TenantTestCase
from django_tenants.test.client import TenantClient
from models import Organization   # Our test model which inherits TenantMixin

class MultiSchemaOrganizationTestCases(TenantTestCase):
    fixtures = []  # Place any fixtures you would like.

    @classmethod
    def setUpTestData(cls):
       pass  # Place any setup test data if you would like.

    def setup_tenant(self, tenant):
        """
        Add any additional setting to the tenant before it get saved. This is required if you have
        required fields.
        """
        tenant.name = "Test Company"

    @transaction.atomic
    def setUp(self):
        translation.activate('en')  # Set English
        super(MultiSchemaOrganizationTestCases, self).setUp()
        self.c = TenantClient(self.tenant)

    @transaction.atomic
    def tearDown(self):
        pass

    @transaction.atomic
    def test_as_public_schema_view(self):
        with PublicOrganization(schema_name='public'):
           # Create tenant in this block
            org = PublicOrganization.objects.create(schema_name='public')
            self.tenant = org

            # Write your tests here.
            print(self.tenant.schema_name) # Output will be 'public'

        # Here would be restored old tenant

    @transaction.atomic
    def test_as_tenant_schema_view(self):
        # Write your tests here.
        print(self.tenant.schema_name) # Output will be 'test'

What do you guys think?

You did smth really wierd :)
To test not as tenant just don't use TenantTestCase as base for your test class.
The test_as_public_schema_view is awful, you can do it just as:
    @transaction.atomic
    def test_as_public_schema_view(self):
        with Organization(schema_name='public') as org:
           # Create tenant in this block
            # Not needed
            #org = PublicOrganization.objects.create(schema_name='public')
            #self.tenant = public

            # Write your tests here.
            print(org.schema_name) # Output will be 'public'

        # Here would be restored old tenant

In fact you can just create decorator to switch schema, with: is just code block decorator. You can create TenantAndPublicTestCase  with setup and teardown for tenant test and create decorator use_schema(name) then just use as use_schema('public') or use_schema('test').
I don't really understand why you are trying this PublicOrganization.objects.create(schema_name='public') when you are running test, your DB is already migrated, so public schema is already there, more correct would be use get(schema_name='public')  as I suggested.
You always have just one public schema. If you need more then switch schema (so you need your data from DB) do just smth like:
with Organization.objects.get(schema_name='public') as public:
    # your block of code

Thanks again for your explanation. I see what you mean.
May I kindly request we get documentation on this issue? I would greatly appreciate it!

Please tell what you exactly miss here, maybe I would have couple of minutes to do it.

Lets pretend you have a simple app with two URLs. One URL you are ONLY allowed to be accessed from a public schema and the other URL can ONLY be accessed by tenants schema. I would like to see help documents how to set up unit testing for those two schemas (public and tenant) and see how it affects this simple app.

It would be really helpful it there were an example of running unit tests with ‘public’ and ‘tenant’ schemas

I figured it! No need to write anything up @kosz85. I'll add to this ticket later an explanation of how to do it. (Or perhaps open up a new Ticket for updating the documentation)

@bartmika Did you ever open a ticket or update the documentation?


#4 more accurate description

I think that the wording of #4 could maybe better reflect the intended description of the issue.

flex shorthand declarations with unitless flex-basis values are ignored

Can also be taken as " If you write out ( flex: 1 ) you must declare the full line with ( flex: 1 1 auto ) to achieve default for the remaining flex parameters. "
Or that is how I took it at first haha.
So Initially I thought that the descriptions of #4 and #6 where conflicting with one another.
But this is probably due to me not being super savy on the terms, being that you said "Unitless" I should have known that referred specifically to the unit type needing to be set if you wanted to set it to 0.
Although, it could be slightly more clear with:

"flex shorthand declarations with a flex-basis value that does not have a unit declared are ignored."

Probably not necessary for most, but it made me stumble for a second.

"flex-basis value that does not have a unit declared"
--vs--
"unitless flex-basis values"
Means the exact same thing to me, in fewer words. Being concise is a good thing.


Use GITHUB_OUTPUT environment file instead of deprecated ::set-output command in GitHub Actions

Pull Request Details

Use GITHUB_OUTPUT environment file instead of deprecated ::set-output command in GitHub Actions
Description

Removed deprecated command in Build and Release GitHub Actions.
See the following resources for more details:

GitHub blog
Same modification in another JetBrains repository

Related Issue

#184

Motivation and Context

See description and related issue
How Has This Been Tested

I ran GitHub Actions
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the README.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

LGTM, thank you!
Have you synced it with the https://github.com/JetBrains/gradle-intellij-plugin project? :)

LGTM, thank you!
Have you synced it with the https://github.com/JetBrains/gradle-intellij-plugin project? :)

Yes, i copied the commit, that i have quoted in the PR description


Getting the Beacons up and running

My hope is to be able to go back to Greenlink this week and get the beacon app up and running this week. @dacohenii @Linktheoriginal Would it be possible to get the web app tied in to the beacon so we can test this bad boy on a live site?
-Adam

It's currently driven off the new .NET API.  Anything that's being pushed in there will show on the app.  The only thing not driven off the API is the route information + showing the active route path.
http://linktheorigi.webfactional.com/trolley/

@ajhodges @bikeoid Is this something y'all can tie in to the API?

The API should be ready to go - I'm not familiar with the platform(s) in the example web app; I tend to do this type of interface in raw Javascript interacting with Leaflet.
I noticed that there is an offset to the route schedule - I think the current server is in another time zone, so it might return routes that are off by several hours.    The 'Active routes' query works though.  Even if the routes are off, the trolley beacons should work.
The new server is being set up by WallingIS.com  - when it's ready, the URL and login credentials will change.    I will refrain from making changes to that server so that the beacon will operate as soon as Verizon wakes from their long winter nap.

I talked to a guy the other day who says he helped install the wifi access
points on all greenlink buses, but greenlink experienced overages $1000+
more per month than expected. However, he says that not only did Greenlink
shut off the units, but they also removed their antennas. I wonder if this
is part of the reason they're taking so long to get it working.
As far as the web app is concerned, it's leaflet+ jQuery with a Django
backend that calls the API. We could potentially serve the files statically
on the same domain and call the API directly.
On Mon, Aug 10, 2015 at 7:13 PM bikeoid<EMAIL_ADDRESS>wrote:

The API should be ready to go - I'm not familiar with the platform(s) in
the example web app; I tend to do this type of interface in raw Javascript
interacting with Leaflet.
I noticed that there is an offset to the route schedule - I think the
current server is in another time zone, so it might return routes that are
off by several hours. The 'Active routes' query works though. Even if the
routes are off, the trolley beacons should work.
The new server is being set up by WallingIS.com - when it's ready, the URL
and login credentials will change. I will refrain from making changes to
that server so that the beacon will operate as soon as Verizon wakes from
their long winter nap.
—
Reply to this email directly or view it on GitHub
https://github.com/codeforgreenville/trolley-tracker/issues/45#issuecomment-129643743
.

The new server is up, and I've pointed the simulated beacon for Trolley 999 away from the old server to the new server  at http://tracker.wallinginfosystems.com/   (DB Visualizer doesn't work on the new server at the moment).

@agautsc and I just set up both trolleys with the beacon app and their internet seems to be working. The lunch trolley service starts in a little over an hour so I guess we'll have our moment of truth!!! Trolleys are numbered 5 and 6 BTW (which conveniently corresponds to the last digit of their respective license plates).


Fix Cypress MTB test project build issue.

Description
Fix Cypress MTB test project build issue.

Some test related files and path are missing by the Cypress ModuleToolBox makefile project file. Those path used to be in the /aws_tests/afr.mk.
the /aws_tests/afr.mk and /aws_demos/afr.mk are no longer used. projects/cypress/make_support/afr.mk is used instead.

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


Hello!

Hey, not so much an issue as it is a few questions and intros.
My name is Bri, and I'm a game design intern at the Wilson Center in D.C.  I just wanted to double check that this is okay with you.
If that is the case, would you happen to have documentation as to how to make individual levels? Your game would pop in between moments of dialog and we'd like it so that one level would appear at a time (I can explain further if need be).
Thanks so much!
Bri Williams

Hello, using this game in another project would be cool. Any part of the code or assets are free to use.
It's important to keep in mind that Nodulus runs in the Unity game engine, so embedding it in another game would very likely require the other game to run in Unity. What technologies are you using for your game? Also, do you have performance/resource constraints? For instance, running on in mobile device app is different from running in a desktop browser window.
I don't have documentation (yet) on how to create custom levels, but I made sure it's pretty simple to do. All you need is to edit a text file. I can update the wiki as necessary. Running different levels at different points in time should be simple, but I need more information on the tech of the base game (e.g., programming language, operating system, frameworks).
Thank you for your comment, and good luck on your game.

We're also using Unity to make our game so that's not a problem. Our game will be running inside a web browser, and server space isn't quite an issue.

So I assume you're using WebGL? I've tested the game with WebGL on the desktop and it runs fine. I will have a WebGL build running once the game releases very soon.
Are you able to open the game in Unity (requires 2017.2.0+)? The main scene is located under Assets/Scenes/NodulusGame.unity. If so, the first thing to test would be to run the scene alongside your main scene with Multi Scene Editing.

Yes, we are using WebGL and we are using is the latest version of Unity. The main concern I have right now is that when I import your game as a package into mine, most of the yaml syntax get's lost in transfer. Could you clarify what collections you use when concerning yaml (the using parameters in the script).
And I just wanted to say, thanks again for all your help! I really appreciate it.

Ohhhh okay, it was the YamlDotNet asset that was causing some issues on my end. I think not having it installed made my scenes go haywire. I'll install it and work from there.

Do you still wish to pursue integrating Nodulus into your project? I'm happy to help if you still do.


feat(chart-controls): Show detailed data type tooltip when hovering type icon

SUMMARY

Fix for issue - https://github.com/apache/superset/issues/13248
Minor improvement: It show the type as in VARCHAR(50), text, double precision when hovering the icon in the metadata panel Please refer the after screenshots below for more understanding.
AFTER SCREENSHOTS

TESTING INSTRUCTIONS

Please verify changes from my fork

Click on charts from headers section to see all charts
Click and open any chart
On the left side there will be columns of table and column options
If we hover on the icon (abc, # etc.) of column type, it should open a tooltip with detailed data type of that particular column

ADDITIONAL INFORMATION

[x] Has associated issue: Fixes https://github.com/apache/superset/issues/13248
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

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

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

@mistercrunch @junlincc Can you please review and approve. Thanks!

Thanks @rusackas ! I will add some tests soon. Also, I have applied prettier linting issue fixes.

@rusackas I have added some tests now for my changes.

@rusackas I have run  npm run lint and npm run test and there were no issues found. Can you please re-run the checks.

@rusackas All checks have passed now. I have verified all UI behaviors in dev from my end. Please let me know in case anything else is required. Thank you for prompt response!

@rusackas I have run npm run lint and npm run test and there were no issues found. Can you please re-run the checks. cc: @michael-s-molina , @kgabryje , @kasiazjc

Thank you @ved-kashyap-samsung for working on this! I have one suggestion (if possible) can you centre the caret/arrow from the tooltip so that it points to the centre to the icon? With current implementation the caret seems not to be anchored to anything specific and it can be confusing, I think

@kasiazjc , @rusackas Now, I have made changes to align the tooltip arrow to the center of the icon in this PR. Please check some sample pics from the modified version. Thanks!

Really appreciate this fantastic PR, particularly the quick turnaround on suggestions (both code and design), and clear communication. You even added tests, and managed to make CI happy! This is a very well-done first PR, and I hope we see more like it in the future :) I added a couple more (non-blocking) suggestions, but I'm happy to approve it as-is, and look forward to merging this!

Really appreciate this fantastic PR, particularly the quick turnaround on suggestions (both code and design), and clear communication. You even added tests, and managed to make CI happy! This is a very well-done first PR, and I hope we see more like it in the future :) I added a couple more (non-blocking) suggestions, but I'm happy to approve it as-is, and look forward to merging this!

Thanks @rusackas ! Sure, I am looking forward to contribute more and more to superset community in future. Also, thank you for suggesting the changes, I have included your review comments in my latest commit.


[Model Update]:  SingleLevelUsageAsPlanned include versions and validity

Update Reason
We need to update this Aspect to include all Part versions and validity (asPlanned) to cover the versioning of components and vehicles. As in #73
MS1 Criteria

[x] The model that should be updated exists
[x] The referenced use case exists
[ ] The potential updates are discussed with all stakeholders
[ ] The potential update will be either

[ ] backward compatible or
[ ] a (chain of) migration strategy(-ies) from all non-deprecated previous versions to the new model will be developed

Modeling_Team

original model needs to be merged from fork

Version will be explicitly mention in description. for Validity period new optional properties need be introduced.
Will be backwards compatible.


Ricardo Pena - Check in

my fork of module repository

Learning Objectives

GitHub:

[x] 🥚 Creating Issues: You can create a new issue in a GitHub repository using an issue template. You can correctly use labels, milestones, and assignment to make your issue findable.
[x] 🥚 Searching Issues: You can find issues in a repository by label, author, and milestone.
[x] 🥚 Project Boards: You can use issues on a GitHub project board to track your class assignments and help your classmates when they need it.

HYF Workflows:

[ ] 🥚 Asking for Help: You can use the question and help-wanted discussion templates in your class repository to ask for help, and to help your classmates when they are blocked.
[x] 🥚 Check-Ins: You can use the check-in template to track your module learning objectives, and your weekly progress, and to post a retrospective at the end of the module.
[x] 🥚 Deliverables: You can create an issue for this module's deliverable and track your progress on the class project board.

Your Computer:

[x] 🥚 File System Management: You can create and organize a new folder in your personal computer for the materials you will study at HYF.

Visual Studio Code:

[x] 🥚 Install and manage extensions
[x] 🥚 Open a folder for editing
[x] 🥚 Create new files/folders inside of the folder
[x] 🥚 Edit files and save changes

Week X
I Need Help With:
1 - Setting the sub linux system
What went well?
1 - Installing all the necessary apps and extensions
What went less well?
1 - SSH connection can be tricky.
Lessons Learned
1 - Proper use of Github
2 - Basic use of terminal and VScode
3 - Use of repositories and issues to get help.
4 - Discord
Sunday Prep Work
1 -

hi @TheRick88 glad to see that you have already learned a lot. SSH can be tricky indeed, we will take a look at it in class.
What operating system are you using now? Andrii has installed VirtualBox and Linux inside it, that might be a good idea and he might be able to help.

hi @TheRick88 glad to see that you have already learned a lot. SSH can be tricky indeed, we will take a look at it in class. What operating system are you using now? Andrii has installed VirtualBox and Linux inside it, that might be a good idea and he might be able to help.

Thank you Daniel! I will try the virtualBox option and see how it goes, see you on Sunday!


Hey, why would you delete Android?

Hi there,
There was a commit in https://github.com/alanjhughes/expo-shazamkit/commit/dd3e5ed02c6f37bd655d9f78a4f4cc5414505d0d
I wonder why did you delete this commit? Does it work?

👀

Adding an aar to a module required altering the main app module and I wanted to avoid that. I'll revisit this soon.

Any update on this? Would love to use Android


Polygon IO limited aggregate bars per request. Fix.

Description
Due to recent Api changes
https://polygon.io/blog/aggs-api-updates/
user will encounter the limitations when requesting data over a long time interval. A large request now needs to be split into several smaller ones.
Related Issue
NA
Motivation and Context
Restore correct work of historical provider.
Requires Documentation Change
NA
How Has This Been Tested?
Adds a new test case
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ X] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[ ] My code follows the code style of this project.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Hey @IlshatGaripov! Just checking in to see what's the status on this one?

Note: I was trying to load minute bars for SNP stocks for the last 10 years and could not get any data on dozens of securities. For example, FB. They were not listed 10 years ago, and as a result at the first iteration in GetTradeBars() we got a null result for the first time interval that led to the end of the loop because of a break; cond., this bug was fixed in the last commit.

Hey @IlshatGaripov! Would be nice to get this one ready for shipping, what's pending?

Hey @Martin-Molinero, to admit it honestly, it's been hard lately to find time for the fixes that do not intersect directly with algorithm development and work for the company. But we have few tick resolution algos in developments I should be fixing the remaining for crypto and forex - this should cover the requested changes. Thanks!


Bug [Admin]: 'Save' button is absent on 'Place' tab

Steps to reproduce:

Go to http://greencityclient.herokuapp.com/.
Login as Admin.
Click on dropdown list of 'Admin' button on the top right corner of the window.
Select 'Admin'.
Select 'Place' tab.
Pay attention to the right down corner.

Actual result:
'Save' button is not present there.
Expected result;
'Save' button is present there.

https://ssu-jira.softserveinc.com/browse/GCS-44

This bug has already been created. Please check previous bugs. thank you


Changes to LiveStreamMonitor Service

Channel List now a ReadOnlyCollection so it cant be changed from outside the Service anymore
Check if there was atleast 1 channel added to the monitoring list ... if not it now throws an UnintializedChannelListException if you call StartService
various styling fixes
fixed a "suspisious equals operation" (VS yelled at me)

Regards
Syzuna and LuckyNoS7evin ;)

Good work guys, thanks :D


New Release

Hey!
saw that you fixed the output command, can we get a release of the action so we can pin the new version nicely?
Thanks for the action!

please, thank you


Awkward passing of serial device on linux

Hi!
Thanks for this initiative. I'm trying to use this library to communicate with a SCPI-enabled device over a serial link, running in linux.
The serial device that it's connected to is /dev/ttyACM0 and the underlying port that should be passed on to pyvisa should be ASRL/dev/ttyACM0::INSTR.
To make this work, I need to initialize the instrument using
inst = scpi.Instrument("COM/dev/ttyACM0"), which is a bit awkward under linux as the "COM" terminology isn't used there.

Unfortunately I don't have a Linux machine to test this with.
Can you please pull the new code and test it out before I push it to PyPI?

Hey @bavovanachte ,
I wanted to see if you were able to test the new package. IF so please let me know the results.
If I don't hear from you in a few days I will close this issue.

Apologies for the delay. I've switched to just using PyVisa for now, but plan to try your library again at some point. I'll reopen or open another issue if it doesn't work

This should be resolved in version 0.1.1.


Ambiguous information on k8s.io/docs/setup/independent/install-kubeadm/

This is an Ambiguous information
Problem:
The table of the open ports for the master nodes specifies 6443*. What does the * means here ? I expected a foot note which is not there, and other information elsewhere suggests the port should be 443. Can you clarify ?
Proposed Solution:
Explain what it means.
Page to Update:
http://k8s.io/docs/setup/independent/install-kubeadm/

There is an explanation below the tables.

Any port numbers marked with * are overridable, so you will need to ensure any custom ports you provide are also open.

Ok, right, maybe that'd be cool to add it right before the ** footnote. I'll propose it in a PR. Thank you for your help !


Accessibility Audit - Carbon Website: Patterns - Disabled states

This epic is tied to a larger "release" that is meant to track the progress of the Accessibility Audit for the Carbon Website. We will audit all of the pages of the carbon website based on AVT 1, AVT 2, & AVT3 - which can be found here.

AVT 1: will audit the site against the IBM Accessibility Checker - browser extension

AVT 2: will audit the site against accessible keyboard interactions

AVT 3: will audit the site against screen readers

Legend:

🌶 fully audited - on all three AVT levels

:one: 🔍 AVT 1 audited
:two: 🔍 AVT 2 audited
:three: 🔍 AVT 3 audited

:boom: issues and documentation - on all three AVT levels are complete

:one: 🚨 AVT 1 issues submitted
:two: 🚨 AVT 2 issues submitted
:three: 🚨 AVT 3 issues submitted

:sparkles: tickets are triaged

Current status of this page:

Patterns - Disabled states - 2 issues - no new issues found:

1⃣ 🔍 AVT 1 audited & 🚨 AVT 1 issues submitted

Closing due to inactivity. Will reopen if/when ready to be worked on again.


[Warhammer 4e Character Sheet] 1.51 missing translations for XP tab

Changes / Comments

XP tab translations added

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Crash during journal file rename

Get this crash from one of the users
Device: Samsung Galaxy Note 4
Retrofit: v1.9
okhttp: v2.6.0
Fatal Exception: java.lang.RuntimeException: java.io.IOException: failed to rename /data/data/com.bandlab.bandlab/cache/journal.tmp to /data/data/com.bandlab.bandlab/cache/journal
       at com.squareup.okhttp.internal.DiskLruCache$1.run(DiskLruCache.java:181)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
       at java.lang.Thread.run(Thread.java:818)
Caused by java.io.IOException: failed to rename /data/data/com.bandlab.bandlab/cache/journal.tmp to /data/data/com.bandlab.bandlab/cache/journal
       at com.squareup.okhttp.internal.io.FileSystem$1.rename(FileSystem.java:85)
       at com.squareup.okhttp.internal.DiskLruCache.rebuildJournal(DiskLruCache.java:409)
       at com.squareup.okhttp.internal.DiskLruCache.access$400(DiskLruCache.java:88)
       at com.squareup.okhttp.internal.DiskLruCache$1.run(DiskLruCache.java:177)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
       at java.lang.Thread.run(Thread.java:818)

I also get this crash from Crashlytics.
Device:
VIVO: X5PRO D, X5S L,  X5 M
SAMSUNG:   SMA7000,GT 19500, SM G9250
OPPO: R7T, R8207, R8107
okhttp : v2.3.0
Fatal Exception: java.lang.RuntimeException: java.io.IOException: failed to rename /data/data/com.liwushuo.gifttalk/cache/net_response/journal to /data/data/com.liwushuo.gifttalk/cache/net_response/journal.bkp
at com.squareup.okhttp.internal.DiskLruCache$1.run(SourceFile:181)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
at java.lang.Thread.run(Thread.java:848)
Caused by java.io.IOException: failed to rename /data/data/com.liwushuo.gifttalk/cache/net_response/journal to /data/data/com.liwushuo.gifttalk/cache/net_response/journal.bkp
at com.squareup.okhttp.internal.io.FileSystem$1.rename(SourceFile:85)
at com.squareup.okhttp.internal.DiskLruCache.rebuildJournal(SourceFile:408)
at com.squareup.okhttp.internal.DiskLruCache.access$400(SourceFile:88)
at com.squareup.okhttp.internal.DiskLruCache$1.run(SourceFile:177)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
at java.lang.Thread.run(Thread.java:848)

Getting the same in crashlytics, don't know how to reproduce and fix the same.

same error

We recently also see lots of these crashes in Crashlytics.
OkHttp: 3.2.0

when fixed this

Same here, Crittercism/Hockey app is complaining in our released app. Using OkHTTP 2.6.0

same here, using okhttp 3.2.0

It has been fixed in Pull request 2388 if I'm correct. Waiting on a new release now..

Fixed in OkHttp 3.3, coming soon.

just got an instance of this in 3.6.0

And in 3.11.0

still get this error from okhttp:3.13.1.
Caused by java.io.IOException
failed to rename /data/user/0/com.goodboystudios.petparade/cache/picasso-cache/journal.tmp to /data/user/0/com.goodboystudios.petparade/cache/picasso-cache/journal
com.squareup.okhttp.internal.io.FileSystem$1.rename (FileSystem.java:85)

We have just updated to okhttp: 4.2.2 and still getting this error
Caused by java.io.IOException
failed to rename /data/user/0/com.goodboystudios.petparade/cache/picasso-cache/journal.tmp to /data/user/0/com.goodboystudios.petparade/cache/picasso-cache/journal
com.squareup.okhttp.internal.io.FileSystem$1.rename

@dickyabd how common/predictable is it? Is it happening for production clients in the wild?
At this point in DiskLruCache and FileSystem, we have successfully deleted the target file "journal" and are failing with what would hopefully be an atomic rename (same file system) from "journal.tmp" to "journal".

hi @yschimke thanks for the response.
This is very common and this is on production.
here is the exception log :

everytime we released an update, the exceptions log warning always highlighted this issue as trending stability issue.
We're not sure why in our app this issue still happening while i saw some comments said they are fixed, as you can see my partner @kevinROctavian  has left comment in Februari https://github.com/square/okhttp/issues/2119#issuecomment-465940509

I'm seeing this issue on okhttp 4.9.0. Though I catch it so it doesn't crash the app. Not sure what is causing it - maybe because I'm using a lru cache.


Custom fields

Based on on #69
Add custom fields as labels. This is an early and simple approach, may be we need some more filters for more complex custom fields.  For now only primitive types (bool, str, int) are exported. For more complex types like multiple choice or relations we need to investigate how to export them to labels.

Related issues:  #34 and #65


✏ Create actual README for public facing project

The current README is just developer documentation. Since this has been moved to the docs directory and the wiki, we should create a public facing README.

Closed by #567


Updated flashlight to use HTTPS

Thank you for the contribution. It was merged directly as 680918d6e9a6f5739825803184ac34bdc0fe1d2a to keep commit history cleaner. Your contribution is still credited to you.


Added tooltips for find next/previous

Description of Changes
Added tooltip texts for find next and find previous buttons in findreplace. (Sort of a small PR, but coudn't really figure out anything else related to add...)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

(Turns out that it LICEcap does not trigger tooltips...)

Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Oscar Gustafsson/oscargus

Hello @oscargus! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/widgets/findreplace.py:

Line 93:46: E128 continuation line under-indented for visual indent
Line 94:46: E128 continuation line under-indented for visual indent


Error when deleting a VM deployment in the playground

Version 2.0.0-rc12 deployed on https://playground.test.grid.tf
Description

In the console log:
Error while deleting deployment "TypeError: undefined is not an object (evaluating 'e.filter')"

Steps to reproduce
State: My account did not have any deployments
Deploy a Micro Virtual Machine
Delete the deployment in the Micro Virtual Machine page

same as

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/400

The original issue was verified in https://github.com/threefoldtech/tfgrid-sdk-ts/issues/400#issuecomment-1600496471
and i tried a micro vm deployment on the latest playground version on qanet 2.0.0-rc14 and i was able to delete the deployment without any issue.
Screencast from 21-06-23 12:21:00.webm


Resizing Primary Side Bar opens terminal Panel

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

VS Code Version: 1.88.1
OS Version: Windows 11 Pro 23H2
Commit: e170252f762678dec6ca2cc69aba1570769a5d39
Date: 2024-04-10T17:41:02.734Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Windows_NT x64 10.0.22635

Steps to Reproduce:

Open Primary Side Bar with Panel closed
Resize the Primary Side Bar.

It always opens the Panel where the terminal lives, it didn't happened before.
Did something change? Is there a config that I didn't find to disable this behaviour?
https://github.com/microsoft/vscode/assets/74925460/338b5586-baa8-48b8-8c2c-020c357b8367

The same thing is happening for me. It only happens when the terminal is positioned on the right. If the terminal is on the bottom or on the left, then it does not appear when I resize the right panel.

The same thing is happening for me. It only happens when the terminal is positioned on the right. If the terminal is on the bottom or on the left, then it does not appear when I resize the right panel.

hmmm,  just tested here and it indeed just happens with the Panel position in the same side as of the ActivityBar.
Since I bought my ultrawide last year I had to bring the panel to the one of the sides so I didn't lose much of vertical space


c-blox logo

This is the opposite of an issue

Can I have a sticker of the proper logo please? :heart_eyes:

Half size


Declare village id format

Where it comes from, as Brian explained on an email. Is a concat of nested level ids.
Intended to complement in e.g. http://api.nightlights.io/ documentation.

Oof, sorry for the delay here @brunosan .  Thanks for the patch.


Custom CA should be installed on the RKE nodes deployed on a cloud provider.

On a 100% air gap installtion it is not possible to build an full working RKE on a cloud provider which is connected via a VPC network. Rancher is using our internal certificates from our comapny self-signed CA.
docker run -d --restart=unless-stopped -p 80:80 -p 443:443 \  -e<EMAIL_ADDRESS>\  -e<EMAIL_ADDRESS>\ -e NO_PROXY="localhost,<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>/16,<IP_ADDRESS>/8,example.com" \  -v /etc/pki/tls/certs/server.crt:/etc/rancher/ssl/cert.pem \ -v /etc/pki/tls/private/server.key:/etc/rancher/ssl/key.pem \ -v /etc/pki/tls/certs/ca.crt:/etc/rancher/ssl/cacerts.pem \ -v /data/rancher:/var/lib/rancher \ -v /data/certs:/container/certs -e SSL_CERT_DIR="/container/certs" \ rancher/rancher:latest
Regarding to the air gap installation we have our own registry build with certificates and configured it in the Rancher settings.
On an RKE-Azure szenario, Rancher is building the nodes and stuck on the point where it is triggering docker to download the Rancher images:
2018/09/04 13:11:04 [ERROR] cluster [c-jsthz] provisioning: Can't pull Docker image [rancher.example.com:8443/rancher/rke-tools:v0.1.13] for host [172.xx.xx.xx]: Error response from daemon: Get https://rancher.example.com:8443/v1/_ping: x509: certificate signed by unknown authority
Rancher should upload the CA into the docker directory for this registry to: /etc/docker/certs.d/example.com:8443/ca.crt
on the node, because Rancher does have all informations about this.
As the node is part of the immutable infrastructure, this shouldn't done manually!

Useful
Info

Versions
Rancher v2.0.8 UI: v2.0.67

Access
`` admin

Route
authenticated.cluster.nodes.index

I'm using unsecure registry for the moment ...


Daijishō problem history... (frequent failures)

I have a lot of frequent errors since I installed the latest version, with the previous version there were no such errors.

Which version are you using?

Can you open the logger and check if there is any errors. (The one with red prohibit icon)

This?

Yes but the one with red prohibit icon and error tag.

There is no red one, they are all ⚠

Thanks for the report. Please leave debug mode on to record the succeeding crashes, or have to wait for the future update to possibly fix the crash. Thanks.

Closed due to no further reports.  Maybe the crashes have been mitigated.