[Bug] Unit Tests can't build in new Shell App with CommunityToolkit

Description
When adding XCTK to a fresh shell project (so no UWP support), an existing unit test project will fail to build. This seems like it might be the same underlying issue as #985 but I'm getting a build failure instead of a warning.
Steps to Reproduce
See sample repo for steps: https://github.com/Siphonophora/XCTC_TargetWindowsBug/commits/main
Expected Behavior
No error.
Actual Behavior
Fails to build.
Error	NETSDK1136	The target platform must be set to Windows (usually by including '-windows' in the TargetFramework property) when using Windows Forms or WPF, or referencing projects or packages that do so.	TestProject1	C:\Program Files\dotnet\sdk\5.0.201\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.DefaultItems.targets	362		NETSdkError
Basic Information

Version with issue: 1.1.0
Last known good version: Unknown
IDE: VS Community 16.9.2
Platform Target Frameworks: Sort of N/A since it blocks build of the unit test project only. But this project would has Android and iOS projects.
Nuget Packages:
Affected Devices: N/A

Workaround
Unknown
Reproduction Link
https://github.com/Siphonophora/XCTC_TargetWindowsBug/

@Siphonophora if your unit test target the netCore version, does this error happens?

Apparently netCore 3.1 builds fine. Added that to the repo link

Same problem here

@Hottemax what's the dotnet version that you're using?

@Hottemax what's the dotnet version that you're using?

Also net5.0

Can you try netCore and see if it works for you?

Can you try netCore and see if it works for you?

Well, I am using C#9 language features (record etc), so I can't easily use .NET Core 3.1 ...

This is a known bug with targeting Windows from a .NET 5.0 project.
The solution is to use .NET Core 3.1 (or lower) for the Unit Test project and set GenerateErrorForMissingTargetingPacks to false in the Unit Test CSPROJ file.
https://stackoverflow.com/a/65161916/5953643
<PropertyGroup>
  <TargetFramework>netcoreapp3.1</TargetFramework>
  <GenerateErrorForMissingTargetingPacks>false</GenerateErrorForMissingTargetingPacks>
</PropertyGroup>


Add left/right/center align support to wrapped text

Hello. This covers part of #160.
I probably did plenty of stuff wrong.
The challenging bit was getting the underlines and strikethroughs to move with the text, so I added another parameter to drawLine that moves everything over on the x.

I haven't tried it, but at first, it looks neat.
just:

remove the "printf("setting font align=%i\n", align);"
align the "if else" condition

please test with valgrind, strike / underline and Blended / Shaded / Mono

remove the "printf("setting font align=%i\n", align);"

Ah, I hoped I could force push fast enough that no one would notice. 😅
I get what you mean with the if/else align.
I might have trouble with valgrind on this system, but this should be good with memory because I didn't do any mallocs.

@Starbuck5 just re-align the "if else if" but that's not a big deal
I am going to try the patch

Removing the extra comment from the commit message would be good.

Removing the extra comment from the commit message would be good.

Oh sorry, that was an artifact of my squash. Not intentional.

It seems to me it crashes when rendering a corner case.
Actually this is an "Invalid write of size 4"
This happen in Draw_Line (
SDL_memset4 (SDL_stdinc.h:453)
==24065==    by 0x4A96084: Draw_Line (SDL_ttf.c:927) )
align: right
wrappedLenght: 661 (eg big)
string:   "="
Blended
font: /system/fonts_AndroidN_Preview/DancingScript-Regular.ttf
I cannot provide a real testcase, I use a test app. with fuzzing. SDL_ttf_testapp.c.txt
(you need to fill with font path insides),
F1= help
press 'r' to do random rendering
( comment out the line 615, this is to 'replay' a crash configuration)
I may have a closer look, but not right now!

@Starbuck5
I looked again, and we need to change 'wrapLength' to 'width'
because if the surface won't be wider that wrapLength for sure, it may be smaller!
        if (font->horizontal_align == TTF_ALIGN_RIGHT) {
            xoffset = width - line_width;
        } else if (font->horizontal_align == TTF_ALIGN_CENTER) {
            xoffset = width / 2 - line_width / 2;
        } else {
            xoffset = 0;
        }

I'll run the fuzzing in background to detect any other issue ...
Otherwise, maybe we should rename the function so that it's clear the alignment is for the Wrapped methods ?
Any suggestion @slouken ?

surface won't be wider that wrapLength for sure, it may be smaller!

Ah, I was assuming it was guaranteed at the wrapLength width.
For API, maybe it should be TTF_SetFontWrappedAlign.

because if the surface won't be wider that wrapLength for sure, it may be smaller!

Oh, this might be a problem for right and center aligned text. For example, if it's right aligned at wrapLength=500, it should end at the 500th pixel, not the last pixel of whatever the longest width was.

because if the surface won't be wider that wrapLength for sure, it may be smaller!

Oh, this might be a problem for right and center aligned text. For example, if it's right aligned at wrapLength=500, it should end at the 500th pixel, not the last pixel of whatever the longest width was.

yes, then add:
line ~3510
             /* Don't go above wrapLength if you have only 1 line which hasn't been cut */
             if (font->horizontal_align == TTF_ALIGN_RIGHT) {
                 width = SDL_min((int)wrapLength, width);
             }
+            if (font->horizontal_align != TTF_ALIGN_RIGHT) {
+                width = wrapLength;
+            }

so that it creates a wrapLength surface

for me it passes the valgrind/fuzzing testing with 100 000 wrapped string with no issue detected.

for me it passes the valgrind/fuzzing testing with 100 000 wrapped strings with no issue detected.

Nice!
I pushed some more changes. One of which was directly suggested by you. The other one I changed up a bit.
Since the surface can only be less than wrapLength if it's a single line, and only if it's ALIGN_LEFT, it seemed simpler to have it always be wrapLength wide. Maybe that would also be convenient to people calling the function.
New problem I found is that it no longer handles wrapLength = 0 as it used to. I don't think it's from this latest commit.

Indeed I fixed the issue with wraplength = 0;
https://github.com/libsdl-org/SDL_ttf/commit/3e702ed9bf400b0a72534f144b8bec46ee0416cb
Please keep the legacy behavior so that the surface may be < wrapLength if align is left

Please keep the legacy behavior so that the surface may be < wrapLength if align is left

Alrighty.
@1bsyl I believe this is now logically equivalent to how it was before. It can only be less than wrapLength if it's 1 line and if it's left aligned.

Yes, this seems ok to me !
just the function name, and if we want to add a function to store a meta style struct as @slouken suggested

Alright, I'll change the name to TTF_SetFontWrappedAlign, if that's alright. It's more descriptive than what I've got in now.
The funny thing about the struct idea is that I'm implementing this so I can use it downstream in Pygame, and I was planning to expose that as an extra "layout" dictionary argument.
But on this level, with just a couple of them, I think it would be simpler (for us and for the users) to do it with functions as I've done here.

@Starbuck5 let's wait for @slouken last word so you don't change back and forth multiple times :)

Don't worry, we can "squash and merge" in the end

Definitely!
If we're going to go down the TTF_SetFontWrappedAlign path, maybe the constants should be named like TTF_WRAPPED_ALIGN_LEFT.

@slouken Waiting on you for an executive decision on the API 🙂

If we're going to go down the TTF_SetFontWrappedAlign path, maybe the constants should be named like TTF_WRAPPED_ALIGN_LEFT.

Yep, this renaming sounds good.


Update sumitishere.json

Updated the social links, please change my urls

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Hi @sumitishere! :) I just gave some suggestions on how to improve your profile. I highly recommend referring to the quickstart when updating it in the future. Other than that, you're off to a great start! 😄 👍🏾

@CBID2 please refer to my comments on your inline suggestions - these actually rendered the JSON invalid. Also, if you are suggesting documentation to contributors so they can update their profile, it is best to recommend the Editing Guide (rather than the QuickStart guide - given this is designed to help contributors when they are starting their profile, not updating it in future)

Noted


Fix CVE-2018-8256

PR Summary
Fix CVE-2018-8256

Fix issue dealing with Relative paths in Update-Help on Linux and macOS
Update to latest version of Archive module for a fix dealing with relative paths

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

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

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[x] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

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

[ ] Add [feature] if the change is significant or affects feature tests

Can we close #7985?


Fix Timestamp and str comparison

Convert str with date into type date when needed

opensignals v.0.0.4 contains the fix.


[YUNIKORN-1587] document resource quota annotations

What is this PR for?
Update description of these.

yunikorn.apache.org/allow-preemption
yunikorn.apache.org/parentqueue
yunikorn.apache.org/namespace.quota

Note that some annotations are deprecated.

yunikorn.apache.org/namespace.max.cpu
yunikorn.apache.org/namespace.max.memory

What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring
[x] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-1587
How should this be tested?
./local_build.sh run
Screenshots (if appropriate)

Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Understood.
I am going to update this PR.

I update context according to feedbacks.


frank deem & tom azinger

Pull Request Test Coverage Report for Build 120

4 of 13 (30.77%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 66.98%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

scripts/retire.py
2
3
66.67%

scripts/utils.py
2
3
66.67%

scripts/new_person.py
0
7
0.0%

Totals

Change from base Build 118:
-0.2%

Covered Lines:
854

Relevant Lines:
1275

💛  - Coveralls


Path filtering based on changes in "root" directory of project

Describe Request: Path filtering based on changes in "root" directory of project
Examples: With current version of path-filtering orb, I cannot specify root directory mapping and trigger selective job in work flow. I tried using various combinations of specifying mapping e.g. "./.*"
Intent here is to triggered selective jobs based on changes in root directory like it does for subdirectories in the project.
Supporting Documentation Links:

`version: 2.1
setup: true
orbs:
path-filtering<EMAIL_ADDRESS>Define the jobs we want to run for this project
workflows:
setup:
jobs:
- path-filtering/filter:
name: Check-updated-files
base-revision: codiac
config-path: .circleci/continue_config.yml
#Syntax:
mapping: |
packages/abc/.* build-abc true
packages/xyz/.* build-xyz true
packages/shared/.* build-all true
packages/bin/.* build-all true
/home/circleci/project/.* build-all true
For subdirectoriespackages/[abc,xyz]/.*` patterns jobs are triggering. However if I change any file under repo's ROOT directory, jobs do not trigger.
For specifying root mapping pattern I have used following
.* ./.* ~/project/.* /home/circleci/project/.*
None of above expressions works to compare and trigger jobs for changed files in ROOT path of repo.

bump. This is a result of some forced regex.
for those who run into this and need a temp fix as of @1.0.0 change ./.foo foo-var true -> `.*.foo


Testing farm???

Noticed that packit failed in our repo on rawhide and it seems like it due to somekind of testing farm feature being magically enabled for everyone.
It's failing/error-ing out on rawhide seemingly due to some ansible python path deprecation warning.
http://artifacts.dev.testing-farm.io/4bd8b02a-63f1-444e-aeb2-8645419d874d/
Arguably you should not be blindly enable "features" in packit for everyone.
Upstream/downstream maintainers should arguably be those that enable those if and then when they intend on using those features.

Well the main problem here of course is that we are getting a failed test for something we did not, in a sense ask for.
What this does is this leads to additional cost of paid/contributors time. The time it takes looking into this +  the cost of distraction ( CoD ) as in the PR submitter looks into why his or her PR fails, the reviewers looks into why the PR's for the submitter is failing ( and now you are looking into my issue due to this being failing, as well as @thrix looking into this and fix what is broken + the time of the reviewers of his fix and then rinse and repeat for more issues of similar nature like this ).
On average a "ping" of some sort ( email, a phone call, coworker asking for something ) costs on average an half an hour, to an hour, factoring in the cost of distraction ( the point you got pinged, to the point you are back focusing on what you were doing prior to the ping ) and ofcourse meetings completely destroy any kind of productivity, which is why people should only schedule/attend meetings at the beginning ( 08:00 - 10:00 ) or end of the day ( 14:00 - 16:00 ) and never on Monday's and Friday's.
Anyway it's best if people/projects can configure/enabled/disable ( within the dashboard or FAS ) what actually is run since in our case we currently only do our testing upstream and basically care for little to no other tests in Fedora other than "does it build" since if it does, it should just work.

What this does is this leads to additional cost of paid/contributors time.

Sure, but packit can't work alone. We mainly integrate other builders/test-runners/Fedora systems. Our aim is to hide this from you (=user) as much as possible but we can't avoid using them. But sure, we can provide better UX when something breaks on another side and we are improving this on the way.

Anyway it's best if people/projects can configure/enabled/disable ( within the dashboard or FAS ) what actually is run since in our case we currently only do our testing upstream and basically care for little to no other tests in Fedora other than "does it build" since if it does, it should just work.

This is enabled by default but can be disabled by specifying jobs in your config:

You can find more in our documentation.
Doing this in the dashboard would be nice but probably not doable in the recent future.
We agreed on turning this on by default after some discussion. It can bring some value without any work required. (Package builds does not mean package can be installed.)
If it leads to a misleading page instead of providing pass/fail on checking installability, it's a bug on our side and we need to fix that. (Either by fixing this or by communicating with the testing-farm team. I am not sure if we can do better with this.)

...  in our case we currently only do our testing upstream and basically care for little to no other tests in Fedora other than "does it build" since if it does, it should just work.

Just to comment on this as well, this is not true for all. In Fedora (and on other distributions as well) you can get one version of your dependencies and can't do much about it. So for some projects (our Fedora packages including), it is really nice not only to try installing the package but also have some tests in that environment. It has saved us multiple times from pushing a non-compatible change to Fedora.

Well the question is, brings value to whom?
You do realize that you have to "sell" the idea of packit to upstream for it to be carried in the tree right so if the defaults are not tailored to upstream ( not downstream ) it can be a pretty hard thing to sell right. ( many upstreams dont even play distro's et all )
Now arguably the only value upstream sees in packit is if upstream maintainer(s) are also downstream maintainers and packit helps reduce the time he or she spends on downstream maintainership so that time can be better spent focusing on the project itself, or more projects if she or he is involved in more than one project ( which also is often the case) or to free time do some personal stuff.
If packit increases existing contributors time ( as opposed to reduce it ) then it will simply end up being removed from tree's ( not implemented/used et al ).
Anyway I suggest that you update the job section with what the "default" jobs that are run are ( also clearly document all defaults ) and how to disable/override them.

You do realize that you have to "sell" the idea of packit to upstream for it to be carried in the tree right so if the defaults are not tailored to upstream ( not downstream ) it can be a pretty hard thing to sell right. ( many upstreams dont even play distro's et all )

Yes, selling distribution to upstream developers is hard in general. I hope I don't need to talk about the benefits of distributions, but if upstream cares about downstream, the ideal state is that their tooll/package/library works in an "expected way" in the distribution, what means:

Package can be created/build. (=> copr build done by Packit)
Package can be installed. (=> default test-case)
Package works as expected in the distribution. (=> Upstream test-suite passes in the distribution environment. / If you run your tests in the testing-farm.)

I agree that UX can be improved, but don't think those checks are not relevant to "upstream-only" contributors.

Anyway I suggest that you update the job section with what the "default" jobs that are run are ( also clearly document all defaults ) and how to disable/override them.

Sounds like a good suggestion for improvement.

To conclude what can help in this situation:

Users don't get links to different systems => everything goes through packit's page. (#506)
Hide problems of the other systems we use internally. (This is ongoing work based on retries, checking of the pending states periodically,...)
[ ] Have clear documentation of the defaults. Make this information easily accessible to newcomers. (Task of this issue.)
Think about defaults (and all features) from the perspective of upstream-only contributors and what value we are giving and how do they understand it. This is very broad but important. (Not a task of this issue. It's a general rule.)

Looking at the current state of docs so the info is there but what can be done to improve the current state:

Default jobs can be easily found by newcomers. (Not sure how we can improve this.)
Provide info about installability test (Maybe also in the dashboard on the result page to give some context right away.)


HTTP Server is slow

The HTTP Server is responding very slowly in V3.  How can I collect additional information on this?
I'm not on my normal network - therefore it's possible it's something else, but it does seem odd.

One of the things it does on its first response is get the external IP address which requires pulling from an outside source (ddns) but it should cache that information after its first used for 72 hours
Is it slow on subsequent requests?
You can monitor the serial output of the device when performing HTTP commands for more information as to where the hang up is

What's the benefit to pulling the external IP address?  Can that be disabled if we don't need it?

I removed the code for pulling the external IP and live is much better.

Its used for the interface when capturing codes to generate the URL for you to use with IFTTT or other similar services
It shouldn't be too bad but I guess internet speeds and regional variability will impact this. The external IP is cached so once it has retrieved the external IP it should be instant for the next 72 hours
That being said I know some people may use this only locally and may not even have an external connection. I went ahead and added 2 new const bool options getTime and getExternalIP at the start of the blueprint. Changing these to false will disable retrieving the time/external IP
I also added some serial output that indicates how long the externalIP request took (averages about 100-200ms for me in testing, which isn't bad for a 72 hour request).
Finally I made retrieving the externalIP part of the boot process (if enabled) so your first code transmitted or page request should be instant after the device is booted and may make the delay from retrieving less noticeable
Hopefully these changes help


Passing options as Object

Hi, In previous version of angular ui-select at https://github.com/angular-ui/ui-select2, you could pass the options as an object like below :
$scope.select2Options = {
'multiple': true,
'simple_tags': true,
'tags': ['tag1', 'tag2', 'tag3', 'tag4']  // Can be empty list.
};
How would I do this in the new version of ui-select.
Thanks
Andy

This can't be done, please refer to wiki on how to use attributes to configure ui-select


dmr-client import exception handling

[ ] Show descriptive error message in output
[ ] Exception details in Trace logs

Let's take a fresh look at the updated dmr-client and re-create/re-open if needed.


[Relay][Memory][VM]

Explicitly manifest memory and tensor allocations in Relay. This is to enable future optimizations and transformations.
v0.0.3
fn (%x: Tensor[(10,), float32], %y: Tensor[(10,), float32]) -> Tensor[(10,), float32] {
  %1 = fn (%p0: Tensor[(10,), float32], %p1: Tensor[(10,), float32], Primitive=1) -> Tensor[(10,), float32] {
    %0 = add(%p0, %p0) /* ty=Tensor[(10,), float32] */;
    subtract(%0, %p1) /* ty=Tensor[(10,), float32] */
  };
  let %x1: Tensor[(10,), float32] = %1(%x, %y) /* ty=Tensor[(10,), float32] */;
  %x1
}

v0.0.3
fn (%x: Tensor[(10,), float32], %y: Tensor[(10,), float32]) -> Tensor[(10,), float32] {
  %0 = annotation.alloc_storage(320);
  let %x1: Tensor[(10,), float32] = let %output: Tensor[(10,), float32] = annotation.alloc_tensor(%0, meta[relay.Constant][0], meta[relay.attrs.AllocTensorAttrs][0]);
  %2 = fn (%p0: Tensor[(10,), float32], %p1: Tensor[(10,), float32], Primitive=1) -> Tensor[(10,), float32] {
    %1 = add(%p0, %p0) /* ty=Tensor[(10,), float32] */;
    subtract(%1, %p1) /* ty=Tensor[(10,), float32] */
  };
  %3 = (%x, %y);
  %4 = (%output,);
  let %v = annotation.invoke_tvm_op(%2, %3, %4);
  %output;
  %x1
}

WIP

@jroesch what kind of optimizations will this enable?

@jroesch This PR models memory allocation/deallocation explicitly in relay expression, it will open the opportunity to write passes to analyze memory operation and try allocation coalescing etc. It will also simplify vm bytecode generation.

Any updates? @jroesch

@jroesch I checked out the PR and debugged it. The compilation error is caused by converting tensors from ndarray to raw shape. On i386, I believe it is int32 but x86 it int64.


Attempting to define property on object that is not extensible

I'm trying to draw data in a carousel, but when I try to open a screen with this carousel, I get this error.
Here is the code and interface of the data I am trying to render:

Also, when you try to open a screen with a carousel, the application crashes without indicating any errors.

Versions (please complete the following information):

react: 17.0.2
react-native: 0.66.1
react-native-reanimated: 2.10.0
react-native-reanimated-carousel: ^3.1.3
react-native-gesture-handler: ^2.7.1

Smartphone (please complete the following information):

Device: [Iphone 14 Pro, Android 13]
OS: [IOS 16.0, API 31]

Also please note, I'm getting data from useSelector but there shouldn't be any problem since the array of data comes in. If I use a simple array from a variable, for example const someVar =  ['1', '2', '3'], then the carousel works. The useSelector state stores no more than 30 entities

3.1.5


Add syscall table for x86 and x86_64

I'm wondering if we should use SYS_write or __NR_write. Both are available to C programs.

Thanks! Looks like CI may be failing?
I don't mind which is added, we can always add both eventually!

I think we should stick with one set, two sets will unnecessarily enlarge the binary. Let's stick with SYS_*, since it's what I've seen in most tutorials.
Well, it didn't like a pub const after extern fn. Fixed that (I think :))

Looks like CI is still failing?

@bors: r+

:pushpin: Commit c0aaf42 has been approved by alexcrichton

:hourglass: Testing commit c0aaf42b3d154fd591ef866ce94f817011d5e80a with merge 18b15ad91f3a106667dbe6aadbef9c7f857844fc...

:broken_heart: Test failed - status-travis

@bors: retry
On Tue, Jul 18, 2017 at 10:22 AM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - status-travis
https://travis-ci.org/rust-lang/libc/builds/254884548?utm_source=github_status&utm_medium=notification
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/libc/pull/672#issuecomment-316099457, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95ArUWtHRt1fvl7wibsx6Ntawqk6Dks5sPM3TgaJpZM4OZ01p
.

:hourglass: Testing commit c0aaf42b3d154fd591ef866ce94f817011d5e80a with merge ec1e5ab1ef8baca57f8776bbebd9343572a87082...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing ec1e5ab1ef8baca57f8776bbebd9343572a87082 to master...

@marmistrz @alexcrichton In linux, syscall expect a c_long
We should rewrite all these constants to be of type c_long.
nix crate cannot be built for these targets anymore

Oops sorry I missed that breakage! I'll send a PR.

I wasn't fast enough to fix it myself :)

Thanks for the quick fix !


Some init improvements

fix init command bug allowing to run the command in a non-empty directory
now init generates an example about page

Fixed #83
Fixed #66

Coverage decreased (-0.65%) to 69.68% when pulling 38471c2cf9690dbdbcce4e69e95a0f111f6f36e6 on init-empty-dir-fix into b26df719ad55524b801b3bac266164417ff4b51e on master.


What file icon theme are you using?

Love the color scheme, was just wondering what file icon theme you're using in the screenshots?

@automaticlightyears May I ask what the theme name is? I still can not find it. :(

Thanks, I'm using the "Material Icon Theme".  I really advise you to start
using the real material theme that's being maintained by the original
creator. I don't have the link at hand but give it a search. Cheers.
On Mon, Mar 13, 2017 at 9:11 PM, automaticlightyears <
<EMAIL_ADDRESS>wrote:

Love the color scheme, was just wondering what file icon theme you're
using in the screenshots?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JarvisPrestidge/vscode-material-theme/issues/5, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGXEGO567TKBLlN2o77ZSd1__TN1pE0wks5rlbEfgaJpZM4Mb0q6
.

OK. Thanks a lot.

@JarvisPrestidge May I ask the font name in the screenshots? Thanks a lot! :)


Compile time Fixes for Spotify so it works

Seems like such a simple fix.  Took a bit of testing to figure it all out.  :grinning:

Thanks @NathanaelA ! I'll try this on the device today 👍


left menu should be scrollable for smaller screen factors.

On a 8 inch tablet, the last  menu items are not visible.
the scrolling on the main region works like a charm.
cheers

Temporary fix is that you select the 'scrolling' layout in Dashboard -> Theme.
I'm also not happy with how this works so I will have to dive into enhancing it. Issue is related to #40 .

As I was not completely happy with the AdminLTE theme that I implemented, I've implemented rdash-angular in a temporary branch.
This template has much less bloated than AdminLTE and it supports the resizing on smaller devices very well.
Please let me know what you think of it!

Of course @movibe @macbroadcast @bmcintyre @rhalff @rborn (and everybody else) are more then welcome to discuss this too! :)

I have a demo online on Heroku: http://loopback-rdash.herokuapp.com/

Very nice !
For those not knowing username password<EMAIL_ADDRESS>/ admin.
The left menu scroll bar is doing the job.
Maybe the windows mobile alike tiles in the AdminLTE were a bit more sexy, but that's probably just a matter of taste.

It was much better.
I would like you to take a look also in a version with the Gulp instead of Grunt, where I added the Less version of the original theme Admin LTE and some improvements.

@paulvanbladel I agree that the tiles of AdminLTE looked better, i'll merge them along with some other stuff.

@movibe i've looked at your Gulp version, which does look very nice! I am thinking about moving to gulp instead of Grunt anyway.
The thing that I don't like about AdminLTE is that it is too bloated, there are simply too many options that won't be implemented in this base project. Next to that the interface is buggy (especially on small screens) and very picky about how it's implemented, I had to change the base AdminLTE to get it to work in this project.
What I like about RDash is that is is clean and simple, and provided as a bower module.
Could you tell what you like more about AdminLTE? If it is just the way it looks we could implement some sort of theming system where we mimic the looks of AdminLTE.

I'm glad you liked my version, just could not solve the dependency error and since you would like to ask you if you could help me solve?
As for AminLTE, imagine that you have used it to be free, but what I wanted to add was more the advantage of compiling Less version and give the freedom to easily change the theme to use in other projects without the need to have large changes in CSS, and added a task to Gulp, but if I like the layout of it, I confess that I find very simple and swollen like you said.
But as you are the leader of the project, you have the greatest word decision.
I confess that the version you did was much prettier and better for tablets, but it will run away from the base of AdminLTE and become a unique template, this is not bad, it may be the best way, but depends on its future plans for the project and I will collaborate as planned.
And forgive my English, I speak Brazilian Portuguese and use google translator rs

@movibe Your English is still better than my Portuguese :)
I guess I'll take this route for now and implement some of the other stuff that AdminLTE offers.
If you can than rebase the gulp version to the new layout I'll check and implement it too!

I definitely second the switch to rdash-angular, a much lighter framework to expand on!

Closing in favor of work on project based on Angular 2+ / TypeScript. Follow here https://github.com/beeman/loopback-angular-admin/issues/202


How to control the char number of output mesh?

The image above is my unwrapped uv's world position which I used to bake lightmap.The mesh is the gun from the famous PBR assets. I hope to decrease output chart number, but I don't know which paramaeter can do this.
Thanks in advance.

It looks as if your mesh was disconnected. The atlas tool is not able to merge disconnected mesh pieces. It expects you to provide a mesh where each connected component has shared vertices. You can join the vertices trivially if you know the mesh is 2-manifold, otherwise it's much more tricky and I recommend exporting the topology information from the authoring tool. Hope this helps!

Also, can you point to the asset you are using? I can take a look at it and check if this is indeed the problem.

Yes, my testing mesh was disconnected, I have given up all optimization when imported it.The indices have the same count as the vertex, they are all 100629.My testing mesh was downloaded from http://artisaverb.info/PBT.html, you can download it at the bottom of the web page : ).
And then since I have got the reason, I resume vertex merge optimization when imported.Too sad atlas_generate() will access invalid memory address so to throw a exception for this testing mesh.I use assimp to load mesh.I will debug into it to get why it can access invalid memory address, I think it's the last step before I inject thekla_atlas into our baking engine.
Thank you very much for your help.

And, I initialized the input mesh as the following code snippet, although what's the meaning of first_colocal.

The first colocal index is used to determine the mesh connectivity. In a
mesh with vertices that have shared positions, but different normals or
texture coordinates, you need to know if those vertices are connected, or
separated. The first colocal index allows you to do that. If two vertices
have the same index, it means they are connected. In some cases you can
assume that all vertices that share the same position are connected, in
that case, you can set the colocal index to the first index of those
vertices. However, in some cases, that may result in a non-manifold mesh.
Ideally, you should use connectivity information from the DCC app or
authoring tool to initialize the first colocal index in a way that produces
valid manifold meshes. For example, traversing the one-ring, and setting
the index of all vertices in the one-ring. Hope that makes sense!
On Thu, May 24, 2018 at 1:26 AM, 陈默<EMAIL_ADDRESS>wrote:

And, I initialized the input mesh as the following code snippet, although
what's the meaning of first_colocal.
[image: 3333]
https://user-images.githubusercontent.com/8615289/40473578-232f6418-5f6f-11e8-834e-f2efcdef75e3.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Thekla/thekla_atlas/issues/18#issuecomment-391632351,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhcM2dWCfJYNWOoSltQG_rKabPd-9Ckks5t1m7RgaJpZM4UJ59A
.

--
Ignacio Castaño
<EMAIL_ADDRESS>

I have understood the meaning of the first colocal index by your replay.
By using it marking out disconnected edge so I can split mesh into different charts. I will take some time to record vertex merge info when import it to generate the first colocal index info to generate lightmap uv  further.
And then I will go to resolve the "non-manifold mesh" problem.Because I have met it before. : ).

You may not have to worry about non-manifold meshes. In many cases meshes are already split into groups and each group is manifold, so as long as you only connect/merge vertices within the same group you should be fine. Anyway, good luck with your project!

Hi, I'm back. Thanks for your prompt about non-manifold meshes first, although I have not met it now. : )
I have been working for filling index info into the first colocal index for these days, and now it works for one of my testing mesh.It looks like this, wonderful!

But there is a mesh making my program crashed when atlas_generate() was called.Would you help me to have look at whether it crashed on your environment? : ) Here is the compressed mesh file.
Cerberus_LP.zip
I have written some code to process .fbx mesh file instead of using assimp. I didn't merge any vertex but only record which vertices share the same control point.Then I fill index info into first colocal index as the following:


is xcause.pil updated on exceptions as well?

I assume if an exception occurs during an interrupt that xcause.pil is updated.  I only see mention in the spec under the section "Changes to xcause CSRs" that xcause.pil is updated due to an interrupt.   Should this be clarified in the spec?

Clarified in #236.

The priv spec uses trap to mean both interrupts and exceptions so the intent of the wording change was to make clear that exceptions will also update xpil.
I would expect xcause.pil= pre-exception "former" mintstatus.xil.
e.g., for a horizontal (m-mode to m-mode) exception where the hart was running at interrupt level ("former il"), I would expect mcause.pil= former il=current il because horizontal sync exception traps are serviced with the same interrupt level as the instruction that raised the exception.
The spec says xinhv is set by hw at start of hw vectoring and cleared by hw and end of successful hw vectoring so xinhv I suppose saying that xcause.xinhv is "updated" on traps isn't precise since that field is only set/cleared by hw vectoring logic or csr write to xcause.

Closed with https://github.com/riscv/riscv-fast-interrupt/pull/287


Mfma fp16

add fp16 fwd pass

@fronteer  https://github.com/ROCmSoftwarePlatform/iGEMMgen/pull/63


"ERROR: unable to open for writing: [Errno 22] Invalid argument: " for specific Twitter video

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

G:\aaaa>youtube-dl -v https://twitter.com/malagurski/status/1353659288779173889
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://twitter.com/malagurski/status/1353659288779173889']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.8.5 (CPython) - Windows-10-10.0.16299-SP0
[debug] exe versions: ffmpeg git-2020-08-31-4a11a6f, ffprobe git-2020-08-31-4a11a6f
[debug] Proxy map: {}
[twitter]<PHONE_NUMBER>779173889: Downloading guest token
[twitter]<PHONE_NUMBER>779173889: Downloading JSON metadata
[twitter]<PHONE_NUMBER>779173889: Downloading m3u8 information
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://video.twimg.com/ext_tw_video/1353656173204037632/pu/vid/540x274/5Y5i1xod6T6JaclM.mp4?tag=10'
ERROR: unable to open for writing: [Errno 22] Invalid argument: 'Boris Malagurski - Elderly Serbian woman claims #Serbia’s President @avucic invented COVID-19 vaccine. This is the result of state-sponsored propaganda in Serbia. When asked what she’d do to those who offend the President, she responds - “I’d slaughter them.”-1353659288779173889.mp4.part'
Traceback (most recent call last):
  File "c:\python\lib\site-packages\youtube_dl\downloader\http.py", line 259, in download
    ctx.stream, ctx.tmpfilename = sanitize_open(
  File "c:\python\lib\site-packages\youtube_dl\utils.py", line 2054, in sanitize_open
    stream = open(encodeFilename(filename), open_mode)
OSError: [Errno 22] Invalid argument: 'Boris Malagurski - Elderly Serbian woman claims #Serbia’s President @avucic invented COVID-19 vaccine. This is the result of state-sponsored propaganda in Serbia. When asked what she’d do to those who offend the President, she responds - “I’d slaughter them.”-1353659288779173889.mp4.part'

Description

Getting this error when trying to download that particular Twitter video. Tried with other Twitter videos and they download with no issue. What I think might be an issue based on provided debug logs is that the video title contains quotes (note that quotes are in title, not URL, so I can't escape them)

for this link - https://twitter.com/malagurski/status/1353659288779173889
just press F12 to open the debug menu, go to network - type m3u8 in the search box
copy the first link that starts with 3Kv1 and feed it into youtube-dl to download the video.

for this link - https://twitter.com/malagurski/status/1353659288779173889
just press F12 to open the debug menu, go to network - type m3u8 in the search box
copy the first link that starts with 3Kv1 and feed it into youtube-dl to download the video.


remove internal-only link

What do you think of removing this link that 404s for people who don't have access to the private repo substra-specs?

@AurelienGasser  Any update on this one ?


Not compatible with Kubernetes 1.22

When trying to install the cass operator on Kubernetes 1.22 like this
kubectl apply -n cass-operator -f https://raw.githubusercontent.com/k8ssandra/cass-operator/v1.7.1/docs/user/cass-operator-manifests.yaml
it fails with the following error message:
unable to recognize "https://raw.githubusercontent.com/k8ssandra/cass-operator/v1.8.0-rc.1/docs/user/cass-operator-manifests.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
unable to recognize "https://raw.githubusercontent.com/k8ssandra/cass-operator/v1.8.0-rc.1/docs/user/cass-operator-manifests.yaml": no matches for kind "ValidatingWebhookConfiguration" in version "admissionregistration.k8s.io/v1beta1"

The reason for this is that apiextensions.k8s.io/v1beta1 and admissionregistration.k8s.io/v1beta1 are not supported anymore and have been replaced with the */v1 version.
┆Issue is synchronized with this Jira Task by Unito
┆Issue Number: K8SSAND-884
┆Priority: Medium

1.7.1 is known to not work on Kubernetes 1.22. These versions were updated for the 1.8 release, but there's only a rc-1 available at this point.
If you wish to try 1.8-rc-1, then installation using Kustomize is available.
kubectl -k github.com/k8ssandra/cass-operator/config/default
The config/default creates namespace "cass-operator" and installs there (use kustomize to modify that). It requires cert-manager if you have webhooks enabled.

I already tested 1.8.0-rc1 and unfortunately it appears to have the same issue.

I think it makes sense to get this fixed for 1.8.0.

All CustomResourceDefinitions are v1 in 1.8.0. Same for ValidatingWebhookConfiguration. The deployment model is different, one can't deploy from 1.8.0 branch the old 1.7.1 docs (they will not deploy 1.8.0).

In short, this error message:
unable to recognize "https://raw.githubusercontent.com/k8ssandra/cass-operator/v1.8.0-rc.1/docs/user/cass-operator-manifests.yaml":

Indicates the 1.7.1 was installed from 1.8.0 branch. It's still only going to deploy 1.7.1 (the docs of 1.8.0 branch have not been updated).

If that is the case, how can I install the 1.8 branch, if the docs are still in version 1.7.1?
Or does it only mean that the docs need to be updated?

Try the: kubectl apply -k github.com/k8ssandra/cass-operator/config/default?ref=v1.8.0-rc.1 command.

Unfortunately, that command produces different errors:
unable to recognize "github.com/k8ssandra/cass-operator/config/default?ref=v1.8.0-rc.1": no matches for kind "Certificate" in version "cert-manager.io/v1"
unable to recognize "github.com/k8ssandra/cass-operator/config/default?ref=v1.8.0-rc.1": no matches for kind "Issuer" in version "cert-manager.io/v1"

Yes, you need to install cert-manager if you went for default installation and don't have it.
kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.3.1/cert-manager.yaml

Thanks a lot for the quick and good help! That did the trick...


Node frendly functions

before example from your test.js

  before(function (done) {
    db.dropTable('test', function () {
      db.createTable('test', function () {
        db.dropTable('person', function () {
          db.createTable('person', ['id', 'name', 'gender'], function () {
            async.eachSeries(person, function (item, done) {db.insert('person', item, function () {done()})}, done)
          })
        })
      })
    })
  })

but users want to use async here:
before(function (done) {
  async.series([
    db.dropTable.bind(db, 'test'),
    db.createTable.bind(db, 'test'),
    db.dropTable.bind(db, 'person'),
    db.createTable.bind(db, 'person'),
    async.series.bind(async, person, db.insert.bind(db, 'person'));
  ], done);
});

this is impossible with your api, because you pass result as first arg, but node convention waits err as first argument.
After you start to support node style you feel free to add some other features to your functions call. It may be promises or carreing
For example after carreing, previous code will look like:
before(function (done) {
  async.series([
    db.dropTable('test'),
    db.createTable('test'),
    db.dropTable('person'),
    db.createTable('person'),
    async.eachSeries.bind(async, person, db.insert('person'));
  ], done);
});

Or shortly:
function series() {
  return async.series.bind.apply(async.series, [async].concat(arguments));
}
function map(obj, func) {
  return Object.keys(obj).map(function(key) {
    return func(obj[key]);
  })
}
before(series([
  db.dropTable('test'),
  db.createTable('test'),
  db.dropTable('person'),
  db.createTable('person'),
  series(map(person, db.insert('person')));
]));

Look at ramda if you want to expose functional programming api in your lib.
So using promises your before statement can look like:
before(function(done) {
  db.dropTable('test')
    .createTable('test')
    .dropTable('person')
    .createTable('person')
    .then(function(resolve, reject) {
      async.eachSeries(person, db.insert.bind(db, 'person'), function(err, result) {
        if (err) return reject(err);

        resolve(result)
      });
    })
    .then(done);
});

So keep in mind, that I don't read all your code at current time. This is only fast review and maybe I missed the main reason why you doesn't use node function style.

Thanks for the detailed review!
I'm thinking about using promises or something else for calls chaining.
However, removing 'err' argument from the first place is done by design, since most of the time it's not used and just creating additional code bloating.

Reverted to err, data node format in upcoming 2.0.0 version


Default pvid can not be configured by nad controller

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
Create nad with default pvid 1
Result:
NAD controller change bridge vlan, resulting in network unreachable. The arp reply accepted by ens160 (configured nic) will not be sent to harvester-br0.

The correct default pvid configuration should be like that:
port	vlan ids
ens160	 1 PVID Egress Untagged


Javascript promise errors in console

I'm getting a lot of Javascript Promise errors in the console like the one in the screenshot below. The code snippets are showing correctly, but still the console log errors are annoying.

@esimov can you post a Snippet or Carbon URL that I can try testing on to debug the issue?

Sure, here it is:
https://carbon.now.sh/embed?bg=rgba(187%2C187%2C187%2C0)&t=seti&wt=sharp&l=text%2Fx-go&ds=true&dsyoff=3px&dsblur=13px&wc=true&wa=true&pv=56px&ph=56px&ln=false&fl=1&fm=Hack&fs=14px&lh=133%25&si=false&es=2x&wm=false&code=%252F%252F%2520Render%2520calls%2520the%2520%2560requestAnimationFrame%2560%2520Javascript%2520function%2520in%2520asynchronous%2520mode.%250Afunc%2520(c%2520*Canvas)%2520Render()%2520%257B%250A%2509var%2520data%2520%253D%2520make(%255B%255Dbyte%252C%2520c.windowSize.width*c.windowSize.height*4)%250A%2509c.done%2520%253D%2520make(chan%2520struct%257B%257D)%250A%250A%2509if%2520err%2520%253A%253D%2520det.UnpackCascades()%253B%2520err%2520%253D%253D%2520nil%2520%257B%250A%2509%2509c.renderer%2520%253D%2520js.FuncOf(func(this%2520js.Value%252C%2520args%2520%255B%255Djs.Value)%2520interface%257B%257D%2520%257B%250A%2509%2509%2509go%2520func()%2520%257B%250A%2509%2509%2509%2509c.window.Get(%2522stats%2522).Call(%2522begin%2522)%250A%250A%2509%2509%2509%2509width%252C%2520height%2520%253A%253D%2520c.windowSize.width%252C%2520c.windowSize.height%250A%2509%2509%2509%2509c.reqID%2520%253D%2520c.window.Call(%2522requestAnimationFrame%2522%252C%2520c.renderer)%250A%2509%2509%2509%2509%252F%252F%2520Draw%2520the%2520webcam%2520frame%2520to%2520the%2520canvas%2520element%250A%2509%2509%2509%2509c.ctx.Call(%2522drawImage%2522%252C%2520c.video%252C%25200%252C%25200)%250A%2509%2509%2509%2509rgba%2520%253A%253D%2520c.ctx.Call(%2522getImageData%2522%252C%25200%252C%25200%252C%2520width%252C%2520height).Get(%2522data%2522)%250A%250A%2509%2509%2509%2509uint8Arr%2520%253A%253D%2520js.Global().Get(%2522Uint8Array%2522).New(rgba)%250A%2509%2509%2509%2509js.CopyBytesToGo(data%252C%2520uint8Arr)%250A%2509%2509%2509%2509pixels%2520%253A%253D%2520c.rgbaToGrayscale(data)%250A%2509%2509%2509%2509res%2520%253A%253D%2520det.DetectFaces(pixels%252C%2520height%252C%2520width)%250A%2509%2509%2509%2509c.drawDetection(res)%250A%250A%2509%2509%2509%2509c.window.Get(%2522stats%2522).Call(%2522end%2522)%250A%2509%2509%2509%257D()%250A%2509%2509%2509return%2520nil%250A%2509%2509%257D)%250A%2509%2509c.window.Call(%2522requestAnimationFrame%2522%252C%2520c.renderer)%250A%2509%2509c.detectKeyPress()%250A%2509%2509%253C-c.done%250A%2509%257D%250A%257D
This is the url where i'm getting the errors: https://esimov.com/2020/01/pigo-wasm

I am fairly sure this was caused by the same issue as: https://github.com/carbon-app/carbon/issues/929
I think the link you posted is a broken link. Can you try posting a new one?

I'm posting a new one:
https://carbon.now.sh/embed?bg=rgba(187%2C187%2C187%2C0)&t=seti&wt=sharp&l=text%2Fx-go&ds=true&dsyoff=3px&dsblur=13px&wc=true&wa=true&pv=56px&ph=56px&ln=false&fl=1&fm=Hack&fs=14px&lh=133%25&si=false&es=2x&wm=false&code=%252F%252F%2520FetchCascade%2520retrive%2520the%2520cascade%2520file%2520trough%2520a%2520JS%2520http%2520connection.%250A%252F%252F%2520It%2520should%2520return%2520the%2520binary%2520data%2520as%2520uint8%2520integers%2520or%2520err%2520in%2520case%2520of%2520an%2520error.%250Afunc%2520(d%2520*Detector)%2520FetchCascade(url%2520string)%2520(%255B%255Dbyte%252C%2520error)%2520%257B%250A%2509d.respChan%2520%253D%2520make(chan%2520%255B%255Duint8)%250A%2509d.errChan%2520%253D%2520make(chan%2520error)%250A%250A%2509promise%2520%253A%253D%2520js.Global().Call(%2522fetch%2522%252C%2520url)%250A%2509success%2520%253A%253D%2520js.FuncOf(func(this%2520js.Value%252C%2520args%2520%255B%255Djs.Value)%2520interface%257B%257D%2520%257B%250A%2509%2509response%2520%253A%253D%2520args%255B0%255D%250A%2509%2509response.Call(%2522arrayBuffer%2522).Call(%2522then%2522%252C%2520js.FuncOf(func(this%2520js.Value%252C%2520args%2520%255B%255Djs.Value)%2520interface%257B%257D%2520%257B%250A%2509%2509%2509go%2520func()%2520%257B%250A%2509%2509%2509%2509buffer%2520%253A%253D%2520args%255B0%255D%250A%2509%2509%2509%2509uint8Array%2520%253A%253D%2520js.Global().Get(%2522Uint8Array%2522).New(buffer)%250A%250A%2509%2509%2509%2509jsbuf%2520%253A%253D%2520make(%255B%255Dbyte%252C%2520uint8Array.Get(%2522length%2522).Int())%250A%2509%2509%2509%2509js.CopyBytesToGo(jsbuf%252C%2520uint8Array)%250A%2509%2509%2509%2509d.respChan%2520%253C-%2520jsbuf%250A%2509%2509%2509%257D()%250A%2509%2509%2509return%2520nil%250A%2509%2509%257D))%250A%2509%2509return%2520nil%250A%2509%257D)%250A%250A%2509failure%2520%253A%253D%2520js.FuncOf(func(this%2520js.Value%252C%2520args%2520%255B%255Djs.Value)%2520interface%257B%257D%2520%257B%250A%2509%2509go%2520func()%2520%257B%250A%2509%2509%2509err%2520%253A%253D%2520fmt.Errorf(%2522unable%2520to%2520fetch%2520the%2520cascade%2520file%253A%2520%2525s%2522%252C%2520args%255B0%255D.String())%250A%2509%2509%2509d.errChan%2520%253C-%2520err%250A%2509%2509%257D()%250A%2509%2509return%2520nil%250A%2509%257D)%250A%250A%2509promise.Call(%2522then%2522%252C%2520success%252C%2520failure)%250A%250A%2509select%2520%257B%250A%2509case%2520resp%2520%253A%253D%2520%253C-d.respChan%253A%250A%2509%2509return%2520resp%252C%2520nil%250A%2509case%2520err%2520%253A%253D%2520%253C-d.errChan%253A%250A%2509%2509return%2520nil%252C%2520err%250A%2509%257D%250A%257D

BTW something has broken in the meantime because some of the snippets from the link provided above are not working anymore, and i'm pretty sure that they have worked at the time when it has been posted. For example this snippet is not working anymore:
But it worked last week. You can see that on this page there are many empty (missing) spaces where the code should be.

@mfix22 do you have any update regarding this issue? For example I simply could not export this code snippet as an iframe, because it gives me the following error:
function fetchAndInstantiate(url, importObject) {
    return fetch(url).then(response =>
        response.arrayBuffer()
    ).then(bytes =>
        WebAssembly.instantiate(bytes, importObject)
    ).then(results =>
        results.instance
    );
}
var go = new Go();
var mod = fetchAndInstantiate("lib.wasm", go.importObject);
window.onload = function () {
    mod.then(function (instance) {
        go.run(instance);
    });
};

and the result is an empty page:


Add CullMode enum

Still getting familiar with the codebase, so I decided to do something simple.
This change adds a CullMode enum that can be passed to Gx::set_cull_mode(). It helps abstract away the numeric value that corresponds to each mode.
This also includes some changes that seem to be from libogc. Hopefully that's not a problem.

Remember to update the examples!!

Thank you both for the reminder :slightly_smiling_face:
I've updated the example and cargo checked it, seems to work.


MINOR: fix incorrect exception message

While playing with client got the next exception:
java.lang.IllegalArgumentException: Invalid partition given with record: 1 is not in the range [0...1].

It's obviously incorrect, so I've fixed it.

I see no errors in test results, but build is failed. Any issue on my side?

LGTM. Merged to trunk. Thanks @stepio !


findspam.py: what_is_this_pharma_title()

Idiosyncratic title formatting, starting with lowercase "what is this".
Metasmoke results are approximate because it doesn't support case-significant searching.
I get 101/114 TP rate with this, but the actual result is closer to 90/90 (however with some low-hanging fruit being missed by the rule): https://metasmoke.erwaysoftware.com/search?body=&body_is_regex=1&commit=Search&feedback=&reason=&site=&title=^what+is+this+([A-Z]|http%3A%2F%2F)&title_is_regex=1&user_rep_direction=>%3D&user_reputation=0&username=&utf8=✓&why=

Codecov Report

Merging #693 into master will increase coverage by 0.04%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master     #693      +/-   ##
==========================================
+ Coverage    61.6%   61.65%   +0.04%
==========================================
  Files           5        5
  Lines        1086     1090       +4
==========================================
+ Hits          669      672       +3
- Misses        417      418       +1

Impacted Files
Coverage Δ

findspam.py
81.71% <75%> (-0.08%)
:arrow_down:

Continue to review full report at Codecov.

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

I think a better reason name for this would be 'Pattern-matching title'. More general reason names should be used so that we can match different patterns down the line under the same reason name.

Like Metasmoke, Google and Stack Exchange do not allow me to search for this specific capitalization pattern.  I have googled for inurl:/what-is-this-* and found only matches with proper case capitalization (but I guess any non-spam post would quickly get reformatted by an astute editor at least on the high-traffic sites).

I'm not a fan of this premature refactoring, and there are other rules which should be refactored as well if we want to be consistent; but here goes.
Discussion at http://chat.stackexchange.com/transcript/message/37131127#37131127


Support std::function<Status(Transaction)> in Commit()

The spanner::Client::Commit() function consumes a std::function<StatusOr<Mutations>(Transaction)> mutator parameter:
https://github.com/googleapis/google-cloud-cpp/blob/e84a574ad5c761a4cc771a9b1f1102077fb6ec66/google/cloud/spanner/client.h#L749-L750
Applications using just SQL statements may wonder why they need to return an empty set of spanner::Mutations from the Commit() mutator.  If they had no intention of using Mutations they could just return Status{} on success.

Nice to have, not urgent.  Let's plan for 2022-Q4

If they had no intention of using Mutations they could just return Status{} on success.

You can't assign an OK Status to a StatusOr<T>.  So, the logical way forward here would be to introduce a new Client::Commit() overload ...
  StatusOr<CommitResult> Commit(
      std::function<Status(Transaction)> const& mutator, Options opts = {});

which could then be called with something like ...
  auto commit =
      client.Commit([&](google::cloud::spanner::Transaction const& txn) {
        auto result = client.Op(txn, ...);
        if (!result) return std::move(result).status();
        // ... etc
        return google::cloud::Status{};
      });

Unfortunately such a call is ambiguous with the existing
  StatusOr<CommitResult> Commit(
      std::function<StatusOr<Mutations>(Transaction)> const& mutator,
      Options opts = {});

overload, so something more complicated would be required, which kind of defeats the purpose.

Applications using just SQL statements may wonder why they need to return an empty set of spanner::Mutations from the Commit() mutator.

I'm not sure why they need wonder too much, or be confused by it.  They're just saying that there are no additional mutations.

Applications using just SQL statements may wonder why they need to return an empty set of spanner::Mutations from the Commit() mutator.

I'm not sure why they need wonder too much, or be confused by it. They're just saying that there are no additional mutations.

People should only learn as many new concepts as needed to use the library.  It seems unfortunate that to use the SQL interface I would need to learn about the spanner::Mutations too, even if I do not plan to use them.

After some thought, we don't think this is needed.


Create units tests for Accessor and Mutator traits

AccessorTest introduced in 0.8.11 https://github.com/macder/wp-form-validation/pull/72
MutatorTest introduced in 0.8.12 https://github.com/macder/wp-form-validation/pull/73


Fixed some words in Indonesian language

Fixed some words to make more humane in indonesian language
Consistent invalid words to tidak valid

Thanks!


Filter by tag in reporting

There is currently no way to list the scenarios based on tags from my tests, in my report.
I would like to see how my tests are split based on the different tags that exist, across spec files.
Things that can help in reporting:

Count of scenarios across tags
Scenario heading under each tag
Ability to drill into the scenario to see the steps

Thanks for bringing this up @harinee. Looks like this depends on or is related to #113 and #97. We'll look into this, but seems like it is something we'll want to take up after a while.

If this is just about reporting, perhaps it would be better to move this issue to the html-report or xml-report projects?

This issue was moved to getgauge/html-report#22

@Thunderforge @mahendrakariya I think the changes won't be just on the reporting side. AFAIK gauge core is not sending any information about the filtered steps. I may be wrong.

Update: We will treat this issue as a request to provide the ability to filter and drill down by a given tag on the HTML-report plugin UI. There is another issue(#113) which talks about the need for an api that will fetch all scenarios/specs given a tag expression.


Fonts broken in Storybook

Most of the storybook examples have broken font (defaults to Times), is that intentional?

@mittalchirag do something

Yeah, this is intentional.


[release-2.5] use oc from builder image

Signed-off-by: Subbarao Meduri<EMAIL_ADDRESS>

/lgtm

/retest-required


Feat: add logs of archive details in archiving process

This PR is linked to issue/ ticket #1214
API contract link: https://github.com/Real-Dev-Squad/website-api-contracts/pull/145
API data modal link: https://github.com/Real-Dev-Squad/website-data-models/pull/3
To add the archive details of the users in the archiving process.
Logs structure
const archivedUserDetailsModal = [
  {
    type: "archived-details",
    meta: { },
    body: {
      reason: "test reason",
      archived_user: { user_id: "R5kljdsleH4Gr2t7tvr0Z", username: "testUser1" },
      archived_by: {
        user_id: "ReMyuklislajwooncVL",
        roles: {
          in_discord: true,
          super_user: false,
          member: true,
          archived: false,
        },
      },
    },
    timestamp: {
      _seconds:<PHONE_NUMBER>,
      _nanoseconds: 912000000,
    },
  },
]

API endpoints

GET /logs/archivedUsers - This endpoint will fetch all the logs of archive details
GET /logs/archivedUsers?username=test - This endpoint will fetch the logs of specific archive details based on the query username

Test Cases
GET /logs/archivedUsers

[x] Should return an empty array if no logs found
[x] Should fetch all archived logs

GET /logs/archivedUsers?username=test

[x] Should fetch all archived logs for a given username
[x] Should throw response status 404, if username is incorrect in the query

Test Case Result

Test Coverage

These updates enhance the archiving process, allowing us to store and retrieve valuable archive details for users efficiently.

Please add Test Coverage as well

Please add Test Coverage as well

Added https://github.com/Real-Dev-Squad/website-backend/pull/1329#issue-1822371683

If changes are made, would it be possible to update the data model as well?

If changes are made, would it be possible to update the data model as well?

the current structure follows the same data structure as https://github.com/Real-Dev-Squad/website-data-models/pull/3,
the changes are instead of passing details in meta, we are passing details in the body, and the meta will remain empty.
So there is no need to update the data modal.
example structure is in the description.


Updated react to 14.9; added prop-types package and necessary changes

React.PropTypes is being removed from the react package and into its own package. This will become a breaking change in version 16 of React. While this package itself is not using version 16 of React, those projects which include react-redux-custom-store throw deprecation warnings about the impending change.
This PR makes some simple updates to use the new prop-types package.
####Summary of Changes

Updated react to 14.9 for compatibility
Added prop-types package
Changed React.PropTypes to use the prop-types package.

There are a lot of eslint errors to fix that go beyond straight syntax rules. I'll need to revisit this.

There are a lot of eslint errors to fix that go beyond straight syntax rules. I'll need to revisit this.


Add & update backend integration tests

This will address just a couple of the items from #212.

Update all backend-specific integration tests to fail quickly if not set up, rarther than silently ignoring or hanging
Add one test case per backend storage class

Coverage increased (+0.02%) to 95.479% when pulling d0f9ca767ad39fc3bed6c142cd66d36b68126193 on JWCook:backend-integration-tests into abb5adb8c06b11355ef7aa1be70c31193538116f on reclosedev:master.


Create Event: Creating an event is successful but shows "Error Code: 404" Page before redirecting to Home Page. Inform user when Create Event is successful

ok ra na lagi ni?

Size: 6

commit id : ccf225b59d83f75cd5f0183d7e9deaa3a66fedac

VERIFIED


processing of too-long model declaration lines

Something is going off during compilation of model declaration lines that are "too long".  A warning is issued at compilation time, which looks indicative of a larger problem.
This may also have something to do with processing of dynamic indexing... not sure.
Reproducible example:
library(nimble)

code <- nimbleCode({
    log.mu <- indA[1]*beta[1]*covs[1,scale[1],1] +
        indA[1]*indA[2]*beta[2]*(covs[1,scale[1],1]^2) +
        indA[3]*beta[3]*covs[1,scale[2],2] +
        indA[3]*indA[4]*beta[4]*(covs[1,scale[2],2]^2) +
        indA[5]*beta[5]*covs[1,scale[3],3] +
        indA[5]*indA[6]*beta[6]*(covs[1,scale[3],3]^2) +
        indA[7]*beta[7]*covs[1,scale[4],4] +
        indA[7]*indA[8]*beta[8]*(covs[1,scale[4],4]^2) +
        indA[9]*beta[9]*covs[1,scale[5],5] +
        indA[9]*indA[10]*beta[10]*pow(covs[1,scale[5],5],2) +
        indA[11]*beta[11]*covs[1,scale[6],6] +
        indA[11]*indA[12]*beta[12]*(covs[1,scale[6],6]^2) +
        indA[13]*beta[13]*covs[1,scale[7],7] +
        indA[13]*indA[14]*beta[14]*(covs[1,scale[7],7]^2)
})

data <- list(covs = array(0, c(10,3,7)))
inits <- list(scale = rep(1,7))
Rmodel <- nimbleModel(code = code, data = data, inits = inits)

Cmodel <- compileNimble(Rmodel)
## compilation finished.
## Warning message:
## In if (is.na(code)) stop("Character NA is not supported for compilation.") :
##   the condition has length > 1 and only the first element will be used

Upon inspection, the code element that's being processed here in the function nimGenerateCpp has value:
> code
[1] "Warning: dynamic index out of bounds: indA[1] * beta[1] * covs[1, scale[1], 1] + indA[1] * indA[2] * beta[2] * (covs[1, scale[1], 1]^2) + indA[3] * beta[3] * covs[1, scale[2], 2] + indA[3] * indA[4] * beta[4] * (covs[1, scale[2], 2]^2) + indA[5] * beta[5] * covs[1, scale[3], 3] + indA[5] * indA[6] * beta[6] * (covs[1, scale[3], 3]^2) + indA[7] * beta[7] * covs[1, scale[4], 4] + indA[7] * indA[8] * beta[8] * (covs[1, scale[4], 4]^2) + indA[9] * beta[9] * covs[1, scale[5], 5] + indA[9] * indA[10] * beta[10] * pow(covs[1, scale[5], 5], "
[2] "Warning: dynamic index out of bounds:     2) + indA[11] * beta[11] * covs[1, scale[6], 6] + indA[11] * indA[12] * beta[12] * (covs[1, scale[6], 6]^2) + indA[13] * beta[13] * covs[1, scale[7], 7] + indA[13] * indA[14] * beta[14] * (covs[1, scale[7], 7]^2)"

I can take a look.
We dealt with long variable names in issue #1039, so there may be a connection.

The issue is that we call deparse on a long expression and when the output is >60 chars long, it get's split into multiple strings. This causes downstream problems. I'm going to look for other cases where this might occur, beyond warning about dynamic index out of bounds and see if I should write a "safe" version of deparse.

I modified deparse in all_utils.R to optionally only return the first line for cases where the result is so long that R splits into multiple lines. For warning and error messages throughout our codebase I then make use of this to limit to one line of output from deparsing code. I did not modify anything where we are using deparse to actually generate code as text.

Side note: We might want to add a default argument to nimble::deparse that throws an interpretable error when the result of deparse is so long that multiple lines are produced. This functionality would be used when we are using deparse for code generation and processing, for which I suspect that in all cases that multiple lines would produce hard-to-trace errors.

In PR #1167, I've now revised things so that we have a safeDeparse that calls out to our existing nimble::deparse. See details in the PR.


xrdp is not aware of configuration file location change for xrdp-sesman

In a recent release, the ability to specify the path to the configuration file was added to both xrdp and xrdp-sesman (-c option) - thank you!  However, if changing the configuration path for xrdp-sesman, there's no way to make xrdp aware of that change, and hence it will still try to read the port from sesman.ini.  This is likely an unintended bug.

The use case you're alluding to here is one I suggested in #1941, which is to run two different instances of xrdp on two different ports (or IP addresses). In this case, the sesman used by each would be different, and xrdp needs some way to find the port for sesman.
Solution is for xrdp to be passed the sesman port or config file on the command line. I'll mark this as an enhancement request, as I believe we'd met your original use-case for for adding -c' in #1588


VideoSourceError

Describe the bug
We are downloading the videos with the same library, then we uploading it to the Instagram with the same script. Its throwing same video source error
To Reproduce
import  json
import os
import pytz
import shutil
import time
# from datetime import datetime,timedelta
import datetime
from time import strftime, strptime
from  instagrapi import  Client
# ACCOUNT_USERNAME = 'name'
# ACCOUNT_PASSWORD = 'Qwerty@123'
# ACCOUNT_USERNAME = 'laxmijoshi275'
# ACCOUNT_PASSWORD = '2345678'
# IG_CREDENTIAL = ACCOUNT_USERNAME
cl = None
def write_file(data, filename):
    fh = open(filename, "w")
    try:
        fh.write(json.dumps(data))
    finally:
        fh.close()
def read_file(filename):
    fh = open(filename, "r")
    return json.load(fh)
accounts={
    'pranika_dhakshu_fan':'Qwerty@123',
    'laxmijoshi275':"2345678"#in this  username  and pass

}
time =datetime.datetime.now()
print(time)
houss=datetime.timedelta(hours=12)
main_time=time-houss
fan_usernames=['pranika_dhakshu_fan','laxmijoshi275']#add username  here
celevraty_account=[]
map_ing={
    'pranika_dhakshu_fan':'pranikadhakshu',
    'laxmijoshi275':'dheeraj_jooo'#add dict of  target and fan account
}

caption={
    'pranika_dhakshu_fan':'this  is caption  of  @pranika_dhakshu_fan  #pranika_dhakshu_fan',
    'laxmijoshi275':'this  is caption  of  @plaxmijoshi275  #laxmijoshi275'#add dict of  caption
}

for name in fan_usernames:
    IG_CREDENTIAL = name
    print(name) #to show  which account  is continue  running
    cl=Client()
    try:
        cl.login(username=name,password=accounts[name])
        print(f'ligin  {name}')
    except Exception as e :
        print(e)
        print(" Plz... check the above exception and solve it  then  try  again ")

    user_id=cl.user_id_from_username(map_ing[name])
    media_start=cl.user_medias(user_id,5)
    medias=[]
    for  i in media_start:
        medias.append(i.json())
    name=name +"-"
    for js in medias:
        ja=json.loads(js)
        ff=datetime.datetime.strptime(ja['taken_at'],'%Y-%m-%dT%H:%M:%S+00:00')
        if ff>main_time:
            os.mkdir(f"{name}")
            files=os.listdir(name)#this  err  is  because  i put the  same  name  of  them  folder  as  the session  name
            all_paths=[]
            for file in files:
                if file.endswith(('.jpg', '.png', 'jpeg')):
                    all_paths.append(file)
            if ja["media_type"] == 1:
                try:
                    print("in photo")
                    cl.photo_download(ja['pk'],name)
                    files=os.listdir(name)
                    all_paths=[]
                    for file in files:

                        if file.endswith(('.jpg', '.png', 'jpeg')):
                            all_paths.append(file)
                    cl.photo_upload(f'{name}/{all_paths[0]}',caption[name.replace("-","")])#caption yes  i am  getting that  from the  post  itself ohk  will  create  a map also
                    shutil.rmtree(name)
                except Exception as e:
                    print(e)
                    shutil.rmtree(name)

            elif ja["media_type"] == 2 and ja["product_type"] == 'feed':
                try:
                    print("in video")
                    cl.video_download(ja['pk'],name)
                    files=os.listdir(name)
                    all_paths=[]
                    for file in files:
                        if file.endswith(('.jpg', '.png', 'jpeg','.mp4')):
                            all_paths.append(file)
                    print(all_paths)
                    print(all_paths[0])
                    cl.video_upload(f'{name}/{all_paths[0]}',caption[name.replace("-","")])
                    shutil.rmtree(name)
                except Exception as e:
                    print(e)
                    shutil.rmtree(name)

            elif ja["media_type"] == 8:
                shutil.rmtree(name)

            elif ja["media_type"] == 2 and ja["product_type"] == 'clips':
                try:
                    print("in reels")
                    cl.clip_download(ja['pk'],name)
                    files=os.listdir(name)
                    all_paths=[]
                    for file in files:
                        if file.endswith(('.jpg', '.png', 'jpeg','.mp4')):
                            all_paths.append(file)
                    print(all_paths)
                    print(all_paths[0])
                    cl.clip_upload(f'{name}/{all_paths[0]}',caption[name.replace("-","")])
                    shutil.rmtree(name)
                except Exception as e:
                    print(e)
                    shutil.rmtree(name)
            cl.logout()
        else:
            print("media  is old ")

Traceback
Expected behavior
Traceback (most recent call last):
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\instagrapi\mixins\private.py", line 298, in _send_private_request
    response.raise_for_status()
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\requests\models.py", line 960, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://i.instagram.com/api/v1/media/configure/?video=1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "c:\Users\dell\Desktop\fan_page\times.py", line 103, in <module>
    cl.video_upload(f'{name}/{all_paths[0]}',caption[name.replace("-","")])
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\instagrapi\mixins\video.py", line 290, in video_upload
    raise e
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\instagrapi\mixins\video.py", line 271, in video_upload
    configured = self.video_configure(
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\instagrapi\mixins\video.py", line 365, in video_configure
    return self.private_request(
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\instagrapi\mixins\private.py", line 445, in private_request
    raise e
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\instagrapi\mixins\private.py", line 432, in private_request
    self._send_private_request(endpoint, **kwargs)
  File "C:\Users\dell\AppData\Local\Programs\Python\Python39\lib\site-packages\instagrapi\mixins\private.py", line 354, in _send_private_request
    raise UnknownError(**last_json)
instagrapi.exceptions.UnknownError: server processing error: VideoSourceError

Screenshots

Desktop (please complete the following information):

OS: windows
Python version 3.9.5
instagrapi version 1.16.14
moveipy version if used
imagemagick version if used

Additional context
Add any other context about the problem here.

Cannot reproduce, try this code:
from instagrapi import Client

cl = Client()
cl.login(username, password)

path = cl.clip_download(cl.media_pk_from_url('https://www.instagram.com/p/CaMYnNZIJCA/'), folder='/tmp')
media = cl.clip_upload(path, 'Test caption')


iRule problem

Hi all,
I have to test my iRule using this tool.
When i run this command "jtcl test_jtcl_irule.tcl" the output is correct ("The jtcl-irule extension has successfully been installed") so i have installed correctly the tcl on my mac.
The problem is reflected in the next step...
I have the following problems:

I can't find the file specified ("test_simple_irule.tcl") on the directory downloaded here.
if I try to run other irules I get compilation errors as if it didn't recognize the language (iRule)
ex:
iRule test.tcl:
when HTTP_REQUEST {  if { [HTTP::host] starts_with "fir3net.com" } {    HTTP::redirect http://www.fir3net.com[HTTP::uri]  }  }

i run jtcl test.tcl and i got this error:

invalid command name "when"

Someone could help me?
Thanks

Nice to see that you are using testcl, I know this wasnt your question but id like to give some pointers on your testcase.
If you write a testcase where an attacker points her own domain fir3net.com.attacker.horse and point it to your F5 ip they can use it to redirect to an arbitrary uri on clear text http.
Redirecting to http is also dangerous because its clear text, your domaim has a working TLS cert so redirection should rather go to https://
starts_with is an unsafe comparison for hostnames because its the ending of the hostname that makes the domain name uniquely qualified. Anyone can own a domain name and create a subdomain that starts with fir3net.com


Update tensor.py with no_grad

Just like train var, allows for idiomatic with statements that effectively save and restore the state of no_grad.

can you add tests? there's some todos in the test/ where this can be used.


MIME types?

Crazy thought - maybe I'm missing something obvious here, but wouldn't it be useful to put mime types into the http response?
Files.swift - line 18 - the content type is always going to be text/html
Files.swift - line 26 - do a lookup for the content type based on the file extension?
One existing source is gitbub - MimeType.swift
Thanks!

You are right 👍

Let's see if I can figure out how to suggest my change be merged - I've been a user, not a contributor in the past.    Seeing as it's in your 'demo server' function is should be safe.    Basically I simply integrated a lookup into a reasonable mime type list to get the content type.


Fix validation in APIv1 when checking for errors in ical export.

No need for this anymore. The traceback has not reappeared and the v1 of the API is going to be retired soon.


Google sheets - sheet name ends with blank space

Hi
When a google spreadsheet sheet name ends with a blank space - " " - it causes:
Error: Unexpected Google Spreadsheets error: Failed to load sheet value from API: HTTP status client error (400 Bad Request) for url (https://sheets.googleapis.com/v4/...

I think we likely need to encode the sheet name so the space is url encoded.


Framerate/FPS jumps to 110, wasting CPU, when game is minimized

v3.0.2.stable.official
Windows 10
Monitor shows the fps going straight for double the normal when minimized.
Happens even when OS.set_low_processor_usage_mode(true)

It seems that when minimized VSync is ignored. I've seen this on OS X too. A possible solution for this would be to limit the FPS to the display frequency when the app is minimized.

Doesn't even have to be minimized. Just hidden.
On my 2017 MBP, my project is set vsync=on and I turned on "print FPS", and then ran a scene.

With the game window visible (not fully occluded), the FPS remains stable at 60 and Godot CPU usage is at ~5%
With the game window fully occluded (e.g. web browser), CPU usage spikes to 92-95% and the FPS jumps:

Game FPS: 60
Game FPS: 855
Game FPS: 2698
Game FPS: 2416
Game FPS: 2129
Game FPS: 1875
Game FPS: 1814
Game FPS: 1828
Game FPS: 1775
Game FPS: 1734
Game FPS: 511
Game FPS: 60

If I set a FPS limit in the debug settings, then the problem does not occur; however, this is a workaround and not a solution.

This could be solved by adding a FPS limit that applies only when the project is minimized. #29297 does this for the editor, but also when it's unfocused, which may be undesired for some projects.
As for the "fully occluded" issue, we'd need a way to detect when the window is fully occluded by another window. This is technically possible (it's been done in Chromium), but it's not particularly easy.

Still valid in d8825fb

See also https://github.com/godotengine/godot-proposals/issues/2001.


wc: fails to append a dash to output line when a dash is given on the command-line

Environment: Ubuntu 18.04, GNU wc version 8.28
wc is failing to append a dash to the output line when a dash is given on the command-line to represent reading from stdin.
What happens now: no difference when calling wc with and without a dash
$ cat tests/fixtures/wc/alice_in_wonderland.txt | ./target/debug/coreutils wc -clmw
   5  57 302 302
$ cat tests/fixtures/wc/alice_in_wonderland.txt | ./target/debug/coreutils wc -clmw -
   5  57 302 302

What I expected to happen: GNU prints the dash as the "name" of the file
$ cat tests/fixtures/wc/alice_in_wonderland.txt | wc -clmw
      5      57     302     302
$ cat tests/fixtures/wc/alice_in_wonderland.txt | wc -clmw -
      5      57     302     302 -

(There are also two issues with the spacing, those issues are represented in issues #2173 and #2186.)

I'm working on a fix.


Append/Insert files into an existing Zip file?

Hey All,
Does this library support appending/inserting files into an existing zip? I've got a use-case where the disk space is limited and so short having to download X number of files and then zip them up I'd like to download them into a buffer in memory and then add that data as entries within a zip as I'm iterating over a list. Is that possible here? I saw you can append with tar just didn't see anything for zips.
Thanks in Advance,
Chris

I don't think it's possible with the zip format, even the zip command line tool temporarily extracts the archive and re-creates it, apparently: https://wpguru.co.uk/2018/12/how-to-add-files-to-an-existing-zip-archive-on-macos-and-linux/
Tar is appendable because the format is basically just like delimiter-separated files, so you can easily add to it.
Closing, but feel free to continue discussion if needed.

@mholt I found this library which does exactly what I need. I found some github issues online where this was requested by the below developer to be included in OOTB golang but core devs pushed back saying there wasn't enough need for it but if the community really wanted it they could look at putting it in.
https://github.com/STARRY-S/zip

Ah wow, I didn't know that was possible since even the official zip tooling doesn't support it.
Looks complicated. I can see why it's not obvious...
Let me see if I can get that library working with this one.

Hmmm... I cannot get it to create a zip file that isn't corrupted (according to unzip -vl test.zip). Were you able to get it to work?

D'oh -- just kidding. I forgot to close the Updater.
I have a commit that works with a single test I did (:sweat_smile:) so you can use it if you want!

@mholt nice! Let me know how it goes and if you're able to optimize things one way or another or "make the impl better" whatever that means in this context :) Once you've got something I can give it a go here. We've basically got a highly parallel process which keeps open the "zip updater", and as files are finished, we're appending them to the zip and then shipping that off. I don't think that other library is maintained or active anymore so if you can whip something up I'd be more than happy to take yours ;)

I mean, that lib works as far as I can tell, I don't know of another pure Go solution. You can try the latest commit I just pushed and see if it works for you. But if you're doing custom concurrency stuff maybe just best to use that lib directly...

@mholt I'll give it a go and see how things fare. We haven't rolled out to production yet and so are still coding things up.

@mholt with this new addition, and the way you implemented it, is it possible here to use a different compressor to get a smaller zip size? We're not necessarily concerned about how fast it takes to build the zip but more if we can get the zip size as small as possible. The files inside are all textual.

@cdancy Set the Compression field of your Zip struct: https://pkg.go.dev/github.com/mholt/archiver/v4#Zip.Compression

@mholt I'm having no luck :( No matter what I use I can't get the zip of the file to not be significantly larger than what I get with the starry-zip library. Same 4 files I'm using there and here results in a 57K versus 764K respectively. Maybe I'm doing something wrong?
        // have to use zip.NewWriter otherwise library was complaining that zip was not valid

	zipWriter := zip.NewWriter(zipFile)
	zipWriter.SetComment("Hello, World!")
	zipWriter.Close()

	zipper := archiver.Zip{
		Compression: flate.BestCompression,
	}

	err = zipper.Insert(context.Background(), zipFile, files)
	require.NoError(t, err)

@cdancy I think you might need to use zip.Deflate instead of flate.BestCompression, which is probably a uint8 that isn't recognized, so maybe it treats it as "store" instead of "compress".

@mholt yeah I tried that as well but still nothing. When I open the zip written by the starry golang lib I see Defl:N compression used by default but no matter what I do here I only ever see Stored. I'm on mac-os so not sure if that plays into things at all.
L105342MUS:kadiv14103107114 dancc$ unzip -vl example.zip
Archive:  example.zip
Hello, World!

 Length   Method    Size  Cmpr    Date    Time   CRC-32   Name
--------  ------  ------- ---- ---------- ----- --------  ----
   77921  Defl:N     5752  93% 00-00-1980 00:00 be9df062  file1.txt
  390349  Defl:N    28739  93% 00-00-1980 00:00 c6890ae4  file2.txt
  354646  Defl:N    18367  95% 00-00-1980 00:00 9150eeac  file3.txt
  687922  Defl:N    40838  94% 00-00-1980 00:00 2f6acde1  file4.txt
--------          -------  ---                            -------
 1510838            93696  94%                            4 files

I don't know ...


optional headers type safety

One day I will document how this crazy type system works.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

11 of 11 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 86.637%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
2818

Relevant Lines:
3302

💛  - Coveralls


Power BI

Hello,
Thank you for your fantastic work!
If you could by any chance share a PowerBI pbix file as an example it would great.
I tried with no luck to use your file in Shape Map and Drilldown Choropleth with no luck.
Thanks you!

Hi Bernard,
I will send you a pbix with the map when I’m home
The github contains a bunch off maps because those are for clients of mine.
Kind Regards,
Arne
Outlook voor iOShttps://aka.ms/o0ukef downloaden

Van: bernardcornet<EMAIL_ADDRESS>Verzonden: Wednesday, February 5, 2020 6:20:29 PM
Aan: arneh61/Belgium-Map<EMAIL_ADDRESS>CC: Subscribed<EMAIL_ADDRESS>Onderwerp: [arneh61/Belgium-Map] Power BI (#2)
Hello,
Thank you for your fantastic work!
If you could by any chance share a PowerBI pbix file as an example it would great.
I tried with no luck to use your file in Shape Map and Drilldown Choropleth with no luck.
Thanks you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/arneh61/Belgium-Map/issues/2?email_source=notifications&email_token=AKUKJH4BXK4D7ZFPPMA3DZLRBLYN3A5CNFSM4KQO65X2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ILIZ4OA, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKUKJH4FHUKZROASC7OYZCLRBLYN3ANCNFSM4KQO65XQ.

Wow. That's super kind! Thank you!
By the way I'm doing this as a pro bono project to help NGO's identify
where yougsters without work not following a traning and not in education
are located.
Bernard
On Wed, 5 Feb 2020, 18:41 arneh61<EMAIL_ADDRESS>wrote:

Hi Bernard,
I will send you a pbix with the map when I’m home
The github contains a bunch off maps because those are for clients of mine.
Kind Regards,
Arne
Outlook voor iOShttps://aka.ms/o0ukef downloaden

Van: bernardcornet<EMAIL_ADDRESS>Verzonden: Wednesday, February 5, 2020 6:20:29 PM
Aan: arneh61/Belgium-Map<EMAIL_ADDRESS>CC: Subscribed<EMAIL_ADDRESS>Onderwerp: [arneh61/Belgium-Map] Power BI (#2)
Hello,
Thank you for your fantastic work!
If you could by any chance share a PowerBI pbix file as an example it
would great.
I tried with no luck to use your file in Shape Map and Drilldown
Choropleth with no luck.
Thanks you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/arneh61/Belgium-Map/issues/2?email_source=notifications&email_token=AKUKJH4BXK4D7ZFPPMA3DZLRBLYN3A5CNFSM4KQO65X2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ILIZ4OA>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/AKUKJH4FHUKZROASC7OYZCLRBLYN3ANCNFSM4KQO65XQ

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arneh61/Belgium-Map/issues/2?email_source=notifications&email_token=ABEHGGN7URA3IZ42CTDCIQ3RBL26JA5CNFSM4KQO65X2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK4KJJI#issuecomment-582526117,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEHGGK3PTF4GH4AH4PZ6HTRBL26JANCNFSM4KQO65XQ
.

Here is the file,
I've used the drilldown choropleth for the map. This custom visual allows me to use maximum 3 levels for my geo hierachy.
I didn't had the time to add proper id's, that why the names are somewhat weird. This is to avoid issues with the visual (matching names for provinces & districts)
MAP.zip

Fantastic! Thank you!
On Wed, 5 Feb 2020, 22:49 Arne Hoefkens<EMAIL_ADDRESS>wrote:

Here is the file,
I've used the drilldown choropleth for the map. This custom visual allows
me to use maximum 3 levels for my geo hierachy.
I didn't had the time to add proper id's, that why the names are somewhat
weird. This is to avoid issues with the visual (matching names for
provinces & districts)
MAP.zip https://github.com/arneh61/Belgium-Map/files/4162170/MAP.zip
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arneh61/Belgium-Map/issues/2?email_source=notifications&email_token=ABEHGGL75RP7C6VTMQAJBL3RBMX6NA5CNFSM4KQO65X2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5DQEY#issuecomment-582629395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEHGGOLJMH5WNYHX7U34R3RBMX6NANCNFSM4KQO65XQ
.


nixos: postpone optionsDocBook deprecation one release cycle

Description of changes

home-manager still depends on it, see
https://github.com/nix-community/home-manager/issues/4273.
The deprecation warning should (ideally) be targeting developers, but
currently all home-manager users get it. (It's not nice to show
warnings to users that aren't in a position to do something about it.)
With this patch, the warning keeps beeing emitted on nixpkgs-unstable,
but silenced for NixOS 24.11.

Things done

Built on platform(s)

[ ] 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

@bjornfor can this be closed then?

@bjornfor can this be closed then?

Yes.


uart_set_line_inverse() mask parameter not specified (IDFGH-2469)

I need to invert TXD line of the UART.
This used to be achieved by
uart_set_line_inverse(n, UART_INVERSE_TXD);
But UART_INVERSE_TXD is not defined anymore in the new IDF. The doucmentation is also unclear:
Choose the wires that need to be inverted. Using the ORred mask of uart_signal_inv_t

uart_signal_inv_t is also undefined.
What value is required when TXD is to be inverted?
Why has UART_INVERSE_TXD been removed?

Hi, @JB-DX
uart_signal_inv_t is define in component/soc/include/hal/uart_types.h, and if you want to inverse TXD line, UART_SIGNAL_TXD_INV should  be passed to uart_set_line_inverse
thanks !!

Thanks !
It now inverts the signal, but

uart_set_line_inverse(n, 0)

doesn't put it back to normal (non-inverted)
I use that to generate a break signal.
uart_set_line_inverse(n, UART_SIGNAL_TXD_INV );
vTaskDelay(200 / portTICK_PERIOD_MS);
uart_set_line_inverse(n, 0);

earlier idf versions work fine.

Hi,
The driver refactoring brings a breaking change to this API. we will fix this soon.
thanks !!


Fix typo in color CLI option for JSCS maker

The latest JSCS uses --no-colors and not --no-color

How will this behave with old versions?

It'll fail. So your choice is: do you want it to fail with previous versions, or do you want it to fail for every new version going forward?

:+1:


Kafka Connection loss with remote node

Hi!
I'm trying to run the Flink Test with a remote Task Manager. While everything works fine with local Task Managers, Zookeeper (I think) can't connect to the remote one. Other Flink Jobs work with it however...
Here's what the log prints out right after the job execution is started:
104818 [NIOServerCxn.Factory:<IP_ADDRESS>/<IP_ADDRESS>:2181] INFO  o.a.s.s.o.a.z.s.NIOServerCnxnFactory - Accepted socket connection from /<IP_ADDRESS>:44174
104821 [NIOServerCxn.Factory:<IP_ADDRESS>/<IP_ADDRESS>:2181] INFO  o.a.s.s.o.a.z.s.ZooKeeperServer - Client attempting to establish new session at /<IP_ADDRESS>:44174
104822 [SyncThread:0] INFO  o.a.s.s.o.a.z.s.ZooKeeperServer - Established session 0x16ade864f620003 with negotiated timeout 40000 for client /<IP_ADDRESS>:44174
[2019-05-22 09:53:29,102] INFO Closing socket connection to /<IP_ADDRESS>. (kafka.network.Processor)
[2019-05-22 09:53:44,617] INFO Closing socket connection to /<IP_ADDRESS>. (kafka.network.Processor)
120372 [ProcessThread(sid:0 cport:-1):] INFO  o.a.s.s.o.a.z.s.PrepRequestProcessor - Processed session termination for sessionid: 0x16ade864f620003
[2019-05-22 09:53:44,630] INFO Closing socket connection to /<IP_ADDRESS>. (kafka.network.Processor)
120372 [NIOServerCxn.Factory:<IP_ADDRESS>/<IP_ADDRESS>:2181] WARN  o.a.s.s.o.a.z.s.NIOServerCnxn - caught end of stream exception
org.apache.storm.shade.org.apache.zookeeper.server.ServerCnxn$EndOfStreamException: Unable to read additional data from client sessionid 0x16ade864f620003, likely client has closed socket
	at org.apache.storm.shade.org.apache.zookeeper.server.NIOServerCnxn.doIO(NIOServerCnxn.java:228) [storm-core-1.2.2.jar:1.2.2]
	at org.apache.storm.shade.org.apache.zookeeper.server.NIOServerCnxnFactory.run(NIOServerCnxnFactory.java:208) [storm-core-1.2.2.jar:1.2.2]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_212]
120375 [NIOServerCxn.Factory:<IP_ADDRESS>/<IP_ADDRESS>:2181] INFO  o.a.s.s.o.a.z.s.NIOServerCnxn - Closed socket connection for client /<IP_ADDRESS>:44174 which had sessionid 0x16ade864f620003
[2019-05-22 09:53:46,453] INFO Closing socket connection to /<IP_ADDRESS>. (kafka.network.Processor)

The local host is at <IP_ADDRESS> and the remote task manager at <IP_ADDRESS>.

I was able to make out that for sone reason the load generattor doesn't reach the Kafka topic when the Kafka broker is running on a hostname other than "localhost". Any ideas?


No file generated with DBFlow 1.8.0

I tried to build my project with the following dependencies:
dependencies {
    compile fileTree(dir: 'libs', include: ['*.jar'])
    compile 'com.android.support:appcompat-v7:22.0.0'
    apt 'com.raizlabs.android:DBFlow-Compiler:1.8.0'
    mod "com.raizlabs.android:{DBFlow-Core, DBFlow}:1.8.0"
}

but the folder build/generated/source/apt/debug or release are always empty.
If i use version 1.4.0 it generates the files as expected. Any idea why?
I use
        classpath 'com.android.tools.build:gradle:1.1.0'
        classpath 'com.raizlabs:Griddle:1.0.3'
        classpath 'com.neenbedankt.gradle.plugins:android-apt:1.4'

Gradle version is 2.2.1 and AndroidStudio 1.1.0. Same result with command line.

I did some test and it appear that the change occurs between 1.6.2 and 1.7.0. Do you have any idea why?

Problem resolved, corrupted aar was the reason....


would be good to describe the satellite image case a bit

even though Johan will probably explain it. Would be nice to include at least two pictures side by side, original on left and the processed image with identified water on the right

On it.

Added some text in 7a8367d


Adding interim license file, preps for improved docs

Adding an interim MIT License file till agreed upon.
Made .github directory in preparation for website documentation.
Updated checklist to include Documentation tasks.

Make Lab41 the copyright holder

Yep, sorry I hadn't caught that before. Correctly changed to Lab41