linux : demo crashing free(): invalid pointer: 0x0000000002281d10

hello and thanks for the code
this is happening on KDE neon user (Ubuntu 16.04)
I tried to recompile caffe without cudnn but same problem happens
./demo deep_image_analogy/models/ deep_image_analogy/demo/content.png deep_image_analogy/demo/style.png deep_image_analogy/demo/output/ 0 0.1 2 1
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:537] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 574671192
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:537] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 574671192
Finding nearest neighbor field using PatchMatch Algorithm at layer:conv5_1.
Finding nearest neighbor field using PatchMatch Algorithm at layer:conv4_1.
Finding nearest neighbor field using PatchMatch Algorithm at layer:conv3_1.
Finding nearest neighbor field using PatchMatch Algorithm at layer:conv2_1.
Finding nearest neighbor field using PatchMatch Algorithm at layer:conv1_1.
Refining photo transfer.
Saving flow result.
Finished finding ann. Time : 121.885
*** Error in `./demo': free(): invalid pointer: 0x0000000002281d10 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x777e5)[0x7f00373a67e5]
/lib/x86_64-linux-gnu/libc.so.6(+0x8037a)[0x7f00373af37a]
/lib/x86_64-linux-gnu/libc.so.6(cfree+0x4c)[0x7f00373b353c]
/usr/lib/x86_64-linux-gnu/libprotobuf.so.9(_ZN6google8protobuf8internal28DestroyDefaultRepeatedFieldsEv+0x1f)[0x7f002eb5e8af]
/usr/lib/x86_64-linux-gnu/libprotobuf.so.9(_ZN6google8protobuf23ShutdownProtobufLibraryEv+0x8b)[0x7f002eb5db3b]
/usr/lib/x86_64-linux-gnu/libmirprotobuf.so.3(+0x233b9)[0x7f001d3ea3b9]
/lib64/ld-linux-x86-64.so.2(+0x10de7)[0x7f003cdc9de7]
/lib/x86_64-linux-gnu/libc.so.6(+0x39ff8)[0x7f0037368ff8]
/lib/x86_64-linux-gnu/libc.so.6(+0x3a045)[0x7f0037369045]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf7)[0x7f003734f837]
./demo[0x407729]

Hi, thanks a lot for your interest on our paper and codes. I guess although it showed such error information, the results were still saved in a folder. You can try to find them just follow the documentation or the codes. For the error, I guess it could be caused by the free function, I would check it later. Thanks for the issue.


Feature Request: Configurable audio receive tick interval

Is your feature request related to a problem? Please describe.
The current audio system processes new voice channel audio data on a hardcoded 500ms interval. For applications intending to use the resulting audio data as real-time as possible, this is a total blocker, as it results in a minimum of 500ms of audio latency.
Line in question: https://github.com/brainboxdotcc/DPP/blob/ffa2653f7504a701735ed23cfb2d503648253a96/src/dpp/discordvoiceclient.cpp#L167
Describe the solution you'd like
Having the ability to set this interval manually would be very helpful. I've tested down to 10ms intervals and dpp::on_voice_receive with no issues.
Describe alternatives you've considered
There's no way to even receive the raw opus payloads directly. Alternative for me is to go back to discordgo (what I was previously using) in order to get realtime access to the RTP payloads as they come in.
Additional context
None

Hm, I like the sound of this! I guess it wouldn't be too hard to have something where the cluster could have something like bot.audio_interval(10). I'd like to see what others think of this and how we could implement it.

My only concern with implementing this is what you stated in #1008, it caused the audio data to be "completely garbled". Of course, we'd need to figure how to fix that 😅

Hmm, that is slightly alarming. If you can easily reproduce it, it may be
worth filing a bug report for it.
On Wed, 22 Nov 2023 at 11:07, tt2468 @.***> wrote:

My only concern with implementing this is what you stated in #1008
https://github.com/brainboxdotcc/DPP/issues/1008, it caused the audio
data to be "completely garbled". Of course, we'd need to figure how to fix
that 😅
Well I'm not sure that it's the tick rate causing garbled audio... I think
it's probably something else, considering the release binaries don't work
either.
—
Reply to this email directly, view it on GitHub
https://github.com/brainboxdotcc/DPP/issues/1009#issuecomment-1822562547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALF2B6JPTKOD7OUD5ZOOLYDYFXMH7AVCNFSM6AAAAAA7RSY772VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRSGU3DENJUG4
.
You are receiving this because you commented.Message ID:
@.***>

The PR has been merged to implement this!


Title: Make the website more attractive

Is your feature request related to a problem? Please describe.
I've checked the whole website, and it's too simple for such a great project like this. I believe the website has a great potential.
Describe the solution you'd like.
I want to add some minimal colour effects to the background, hover effects to the buttons and the project boxes, and a simple cursor trail animation, which will definitely boost the overall UI of the website and make it more attractive, thus holding the user's attention.
Describe alternatives you've considered.

None -

Add any other context or screenshots about the feature request here.
No response

this is my telegram @leo1228codeforce . contact me.

I'm new to open source , can you please assign this to me under Gssoc 24

@Sambit-Mondal Thank you for providing your insight but it is not required as of now


Delete all of the placeholder

Fixes #2636.

Pull Request Test Coverage Report for Build 8698

44 of 48 (91.67%)  changed or added relevant lines in 8 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.1%) to 61.239%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/token/generator/EncryptColumnTokenGenerator.java
0
1
0.0%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/token/generator/InsertColumnsTokenGenerator.java
0
1
0.0%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/SQLRewriteEngine.java
24
26
92.31%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/SQLRewriteEngine.java
1
90.54%

Totals

Change from base Build 8697:
0.1%

Covered Lines:
8405

Relevant Lines:
13725

💛  - Coveralls


Add channel view

pulled out of #8
This adds a screen_view for channels (tab paths starting with '#'),
and adds a message_meta for messages to indicate their channel.
Performance of loading channel tabs is improved significantly by dominictarr/ssb-query#1

:100:

merged into 1.2.0, thanks!


Add and rename board column in-place

Closes #482
23-11-20
This is a work-in-progress. As I won't be available in this and the coming week, I'm taking down some notes here for a quick restart, or for the reference of any interest in the dev progress and contributing.
Known problems

Rename to or add options of a existing one would leave redundant field options.
Width of the proposing column doesn't follows the board settings.
Try to drag a card or an existing board column would stuck the dnd feature.

Design on column visablility

Now, the columns with a predefined value as the header is considered to be permanent, while others temp fields. Temp fields would not shown if there's no matching note that contains this value.
Newly added column will be added to options.
Renaming columns won't change the temp/permanent status.
May be we can provide a better visual hint, or allow direct conversion in the column setting.

Close as stale codes. Working locally for a refactored version and will open another PR soon.


Made MAX_WIND_INTENSITY configurable. Fixed links to canvas_layer.js in the examples.

Making MAX_WIND_INTENSITY configurable allows plotting vectors other than wind of different scale (say sea currents).

Apologies for the slow reply, thanks for sending this along. Had just a few comments -- will merge when I hear back.


Relationship syntax is inconsistent with ember-data

Thanks for the addon, it seems to work really well! However while playing around with it this morning I came across a stumbling point whilst trying to make undoable updates on a hasMany relationship.
Initially I wrote my code treating the relationship as async, as I would expect to with a normal model:
posts.get("comments")
    .then(comments => comments.pushObject(comment));

However, I got the error posts.get(...).then is not a function. I found that updating the code to treat the relationship as synchronous,
posts.get("comments").pushObject(comment);

made it work, with undoAll then rolling back additions/removals on the comments relationship as hoped.
It would be useful if ember-time-machine treated relationships in a way that was consistent with ember-data, so that introducing the addon to an existing project doesn't require code changes. It also concerns me as to how async relationships are being treated - if a relationship is lazy-loaded and the data has not yet been fetched from the server, will using the synchronous access I had to resort to cause problems?

Im assuming posts is a time machine.
When you do posts.get('comments') it returns a new time machine wrapped around comments which wont have a .then. What you can do instead is:
const comments = posts.get('comments');
posts.get('comments').invoke('then', (result) => comments.pushObject(result));

// OK

comments.get('content').then(/* blah blah blah */);

Let me know if that works out okay for you!

Thanks for getting back so quickly! posts (should've been post, oops) is indeed a time machine.
I'm a little confused by your first suggestion - I'm not familiar with the invoke method, but I'd expect result to be the comments relationship, rather than the comment I want to push into that relationship.
The second suggestion returns a promise, but again necessitates a difference between looking up a relationship on a plain model vs. a time machine wrapped model. To me, ideally, there would be no difference syntactically between the two, i.e. you would be able to access relationships on time machine models exactly the same way as with plain ember data models. Maybe that isn't possible/valuable though.
In my use case I ended up preloading the relationship for other reasons, which allows me to just continue using the synchronous posts.get("comments").pushObject(comment) approach.

Thank you for the explanation! What you're saying makes perfect sense, and in the end, having to occasionally .invoke or .get("content") is a small price to pay for the excellent functionality your addon offers :smiley: Closing the issue.

Unfortunately I'm back to reopen this! I've just found that if I use the post.get("comments.content").then(comments => comments.pushObject(comment)) suggestion you gave me earlier, rolling back via post.undoAll() does not remove comment from the comments relationship. If I use the synchronous-style post.get("comments").pushObject(comment), the relationship gets rolled back correctly.

@AndyPyeSC thats because you're now only interacting with the actual model and no long with the comments time machine.
Try something like this instead:
const comments = posts.get('comments'); // Reference to our comments time machine
comments.invoke('then', (result) => comments.pushObject(result)); // Call pushObject on the comments TM

I've had a play with the .invoke(...) approach and wasn't able to get anything working. However, looking at the documentation for invoke I figured I could do this:
const comments = post.get("comments");
comments.get("content").then((/* commentsContent */) => {
    comments.pushObject(comment);
});

I then realised that this is exactly what you were suggesting originally, except I made the mistake then of pushing comment into the non-time machine commentsContent array, instead of the time machine comments array. Should've thought it through better to start with... thanks for clearing that up though! :smiley:

I then realised that this is exactly what you were suggesting originally, except I made the mistake then of pushing comment into the non-time machine commentsContent array, instead of the time machine comments

Glad we are finally on the same page 😺 ! Weird that invoke didnt work out... Ill need to take a look into that and see whats going on there.


Builds action failure

Problem
Builds workflow seems to be broken at the moment: https://github.com/project-chip/connectedhomeip/actions?query=workflow%3ABuilds .
It displays the same error in forks. Comparing logs, some things have changed:

Ubuntu: 20210123.1 -> 20210131.1
Docker: 1.40 -> 1.41
CodeQL: 2.4.0 -> 2.4.2

It seems that TestInetLayerDNS is also now run. It was not the case in my previous log.
What is surprising is that unchanged code now fails.
Proposed Solution
Identify the root cause of the issue and fix it.

https://github.com/project-chip/connectedhomeip/issues/4670

(Working on it now!)


Add VS Code support for tutorials

Summary of the PR
Fixes: #273
Adding tasks and launch configurations to allow running and debugging
tutorial projects directly from Visual Studio Code.
Newly added tasks also allow publishing tutorial projects and running
them under under the wather.
Verification:

Ran each added launch configuration from VS Code
Published a couple projects and made sure artificats were written to
the expected location
Ran a couple projects under the watcher

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please see the latest discussion on the issue. We're only interested in a launchsettings.json file which is standard across most C# IDEs. I'm not sure what the level of support of this is in VSCode, however we don't want IDE-specific files in the repository.

My understanding is that launchSettings.json is used by ASP.Net Core and the tutorial projects are not ASP.Net projects. As such I don't think it is possible to add launchSettings.json to these projects that would work in any C# IDE. As a side note, launchSettings.json is also not supported by VS Code - see the comment in the vs code repo https://github.com/microsoft/vscode/issues/71007#issuecomment-475921297.

Ah ok, I didn't know that. Seeing as the .vscode directory is seemingly used by all Monarch-based editors such as VS Code and GitHub Codespaces for example, I think I'm finally just gonna give in and let this slide unless @HurricanKai has any objections.

I'm good with this. I would still like to have a launchSettings.json file in the future though.

I am working on approval to sign CLA.

Any update on the CLA front? As long as you didn't make this PR during paid hours or as part of your usual working duties, you shouldn't need approval from your employer.

Still waiting for approval. It's outside my control - last time it took about two weeks.

I signed the CLA.


Support sourceSet customization and rev to 2.0.0

Fixes #53
Before the build lifecycle was implemented such that if sourceSets were customized,
kotlinter would have already created tasks based on the original sourceSets. If we
do this instead during project.afterEvaluate it can work off of the final definitions.
One consequence is that any customization of kotlinter created tasks must be wrapped in an
afterEvaluate block since they don't exist during the first build configuration pass. This
prompts a bump to 2.0 since it will be a breaking check for some builds.

closing in favor of #55


Add guide for Vite Dark mode

Sorry, I'm not sure what this mdx file looks like because the extension MDX Preview shows an error about it.
And I don't know how the preview code:  works, so I just copy&paste my code. it works fine and didn't need any extra packages, so it's easier to fix any potential bugs.

#814 implements the same layout of installation page.

Good to know, thanks


[Miro] can you make it that I can find boards in my team, and not just my own owned ones please

Extension
https://www.raycast.com/felixhaeberle/miro
Description
There isn't a way of searching for boards I don't own that I can find. 9/10 I'm opening a board used by my team that I didn't create and for this, the extension doesn't seem to help.
Who will benefit from this feature?
This would be brilliant in an enterprise setting where people create and share boards across teams.
Anything else?
No response

Is this likely to ever be responded to?

Feel free to do a PR with the desired behavior and I will consider merging it!:)


The Install module button does nothing

Reproduction steps:

click "Install some from here" from the "No Modules Installed" notification
click "Install" button for SRanipalTrackingModule
nothing happens

Expected result:
Module gets downloaded and installed.
Version: <IP_ADDRESS>
Platform: Windows 10 22H2

Log output:
[UnifiedLibManager]` Information: Starting initialization tracking
[UnifiedLibManager] Information: Tearing down all modules...
[UnifiedLibManager] Debug: Initializing requested runtimes...
[UnifiedLibManager] Information: Initializing runtimes...
[UnifiedLibManager] Warning: No modules loaded.
[VRCFaceTracking.Core.Services.ModuleDataService] Debug: Rating for 74f18332-1bcf-4b55-832d-9c91069afa31 was not found
[UnifiedLibManager] Information: Tearing down all modules...
[VRCFaceTracking.Core.Services.ModuleInstaller] Debug: Uninstalling module 74f18332-1bcf-4b55-832d-9c91069afa31
[VRCFaceTracking.Core.Services.ModuleInstaller] Warning: Module 74f18332-1bcf-4b55-832d-9c91069afa31 could not be found where it was expected in C:\Users\Ante Vukorepa\AppData\Roaming\VRCFaceTracking\CustomLibs\74f18332-1bcf-4b55-832d-9c91069afa31
[App] Error: Unhandled exception
[App] Critical: Stacktrace:    at System.IO.FileSystem.MoveDirectory(String sourceFullPath, String destFullPath, Boolean isCaseSensitiveRename)
   at System.IO.FileSystem.MoveDirectory(String sourceFullPath, String destFullPath)
   at VRCFaceTracking.Core.Services.ModuleInstaller.InstallRemoteModule(TrackingModuleMetadata moduleMetadata)
   at VRCFaceTracking.Views.ModuleRegistryDetailControl.Install_Click(Object sender, RoutedEventArgs e)
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object state)
   at Microsoft.UI.Dispatching.DispatcherQueueSynchronizationContext.<>c__DisplayClass2_0.<Post>b__0()
[App] Critical: Inner exception: (null)
[App] Critical: Message: Source and destination path must have identical roots. Move will not work across volumes.
[UnifiedLibManager] Information: Tearing down all modules...

This appears to be because the app assumes it's on C:
If it's not (and it can not-be, because Microsoft allows you to change the default App drive via Settings), a lot of things will break.

Nope. Uninstalled, reconfigured app storage back to C:, reinstalled, same issue persists.
Also, no VRCFaceTracking folder in AppData.
No idea what's going on.

Okay, figured it out.
App expects the Windows Temp folder to be on the same driver as itself.
Since it uses move instead of copy (and delete), which only works within the same volume, copying the module from temp where it was downloaded, to where it needs to be, fails if Temp is elsewhere.

How can I fix it?

Nope. Uninstalled, reconfigured app storage back to C:, reinstalled, same issue persists.
Also, no VRCFaceTracking folder in AppData.
No idea what's going on.

This is the issue I been getting except I can wipe the folders within app data and the built-in modules load.
A second issue is that no matter what I do, the 3rd party modules (like eyetrackvr or pimax) will never load, not even if I install them manually as a zip, modules are added to the app data folder, but these fail to load completely.
To the devs, I did a more specific rundown on my thread on discord: https://discord.com/channels/849300336128032789/1189778058610233404
I should mention I am on windows 11 21h3, so please check if you are able to recreate this issue on the most recent w11 version.
So far, un installing, reinstalling, wiping data etc has not worked. Not even adding the modules directly on CustomLibs, it's become clear that only the modules located on StockLibs are the only ones that load correctly on the app


Fix Github Actions Simulator CI Failures

After moving Github Actions, we found two runners consistently failing with, failures seen in tests:
test_acl_sharednetwork (for example https://github.com/apache/cloudstack/actions/runs/3764630739/jobs/6399260045)
test_tags (https://github.com/apache/cloudstack/actions/runs/3764630739/jobs/6399260616)

test_acl_sharednetwork should be fixed in #7033

All are good now!


Pythonise "Working with Python arrays" (array.rst)

Continuing to Pythonize Сython documentation!
This PR Pythonises this documentation page.
Lots of process details and standardization are found here: #4187

@scoder, could you please review this?

@0dminnimda are you still willing to finish this PR? I think the only pending point is to finish the Reference part. If yes I will close my PR in favor of yours.

are you still willing to finish this PR?

Sure

I think the only pending point is to finish the Reference part.

ok, we're done with that
I wouldn't say I like this change, but for the sake of improving the docs, I won't argue about it.

Now the pr is ready for review, waiting for @scoder

Very nice. Thanks!


MaxListenersExceededWarning

MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 connect listeners added. Use emitter.setMaxListeners() to increase limit
const results = await push.send(devices, data);
console.log('Push results', results);
Why i get memory leak detected?
What i need to change? I think it is a bug.

This is a problem in the upstream node-http2 library.
This is a duplicate of #518.
See also: #557 #578 #362


⚠ Bofrost.gr has degraded performance

In 94faa1f, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 1372 ms

Resolved: Bofrost.gr performance has improved in 78fa7d2.


Present only affordances which are actually available

I'm an unauthenticated user and I see the following:

As a general rule, an unauthenticated user should not be presented with controls that they can't eventually use; stick to the real affordances. In this particular case:

the icons to take a photo, upload an image, and delete should be removed.
the form fields should be replaced with plain text (if any currently filled)
additional controls like "Add one or more.." should be removed - basically the profile should only display as though the viewer is a reader (as opposed to writer/contributor).

The tooltip on the red trashcan is "Drag here to delete" - it is unclear what it is referring to but apparently it is about the image - figured this out after quick guessing and noting what a profile with a image looks like. This can be clarified with "Drag profile images here to delete".
When I try to drag the trashcan icon back into the trashcan:

While that may seem like a silly scenario to be in, the point is that the user should not be in that situation. Default to not exposing controls inapplicable to current user. Avoid introducing potential errors altogether.

:+1:

It is improved in the sense of not prompting with tools to modify the data when don't have the authority to modify it.
It still presents blank fields which are not useful like

The test case above is https://csarven.inrupt.net/

It looks as though a flag in the form system "Suppress field if blank and uneditable".

See https://github.com/SolidOS/solid-ui/issues/560


visual studio code reports error in temporary file when compiling xaml

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

VS Code Version:
OS Version:

Steps to Reproduce:
I am building an MVVM WPF project in C# on Linux.
Visual Studio Code reports an error in a temporary file used by the XAML compiler
An example
/home/howard/Desktop/SudokuUnoTest/SudokuUnoTest/SudokuUnoTest/SudokuUnoTest/Uno.UI.SourceGenerators/Uno.UI.SourceGenerators.XamlGenerator.XamlCodeGenerator/MainPage_c6813ceb9e79be38b2f92b31a9dd5cd2.cs(1744,107): error CS0030: Cannot convert type 'string' to 'double' [/home/howard/Desktop/SudokuUnoTest/SudokuUnoTest/SudokuUnoTest/SudokuUnoTest/SudokuUnoTest.csproj::TargetFramework=net7.0]
Note that file being compiled is being created during the Xaml compilation,  The file is deleted at the end of the build.  Therefore, there is not way to determine what is causing the error
The offending XAML code is as follows

where Rectangle Collection is defined in the View Model as
public ObservableCollection RectangleCollection { get; set; }
and RectItem is a class
public class RectItem
{
public double X { get; set; }
public double Y { get; set; }
public double Width { get; set; }
public double Height { get; set; }
public double StrokeThickness {get; set;}
}
There seems to be no way to prevent dotnet from deleting the Uno.UI.SourceGenerators/ folder
Howard Weiss

There seems to be no way to prevent dotnet from deleting the Uno.UI.SourceGenerators/ folder

pls create an issue in the dotnet repo


SOUS-792 read-only rectification endpoints

This large PR adds new rectification queue endpoints, as per SOUS-792.

          1 Warning

:warning:
Big PR: 1579 lines of code changed

  Generated by :no_entry_sign: Danger

Codecov Report

Merging #582 into master will increase coverage by 0.4%.
The diff coverage is 92.09%.

@@            Coverage Diff            @@
##           master     #582     +/-   ##
=========================================
+ Coverage   69.66%   70.07%   +0.4%
=========================================
  Files         219      223      +4
  Lines       12307    12491    +184
=========================================
+ Hits         8574     8753    +179
- Misses       3127     3129      +2
- Partials      606      609      +3

Impacted Files
Coverage Δ

lib/deployment.go
80.73% <ø> (-1.77%)
:arrow_down:

lib/deploymentid.go
100% <100%> (ø)

lib/deployable_chans.go
73.33% <100%> (+0.91%)
:arrow_up:

server/server.go
90% <100%> (+9.14%)
:arrow_up:

lib/r11n_queue_set.go
93.54% <100%> (+3.54%)
:arrow_up:

util/restful/routemap.go
79.22% <75%> (-3.24%)
:arrow_down:

server/handle_deployments.go
83.33% <83.33%> (ø)

server/handle_r11n.go
89.18% <89.18%> (ø)

lib/r11n_queue.go
95.19% <91.3%> (-0.16%)
:arrow_down:

server/handle_deployment.go
92.59% <92.59%> (ø)

... and 7 more

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 e6f0c02...0b5b852. Read the comment docs.


using picam-streamer for audio only streaming and recording

Hi Iizukanao, is there a way to start picam or picam-streamer for audio only streaming and recording
Thanks and Regards
Rajesh kumar

Hi Rajesh,
Unfortunately, picam does not support audio-only streaming as it's primarily a tool for video streaming. I would recommend using ffmpeg instead.

Thanks for a prompt response🙏🙏🙏🙏


Few seconds of freeze when going back to terminal after a while

This is a little difficult to explain, but here it is:
I am having a problem that I didn't have with oh-my-zsh (hence I think it's a prezto issue not zsh). When I'm on command line working with przto everything is fine. Then I switch to e.g., my browser or IDE and work for while while my terminal is sitting on the background doing nothing. When I switch back to terminal and start to type, everything is frozen for few seconds and then goes back to normal. Once it starts to respond again, it shows all of the keys that I have pressed when the terminal was frozen.
I have this issue with all of my terminal emulators. Any ideas?

No idea.

Do you use a rotating HDD or an SSD? How much RAM do you have?
It might be tat you visited some websites that used up a lot of RAM, so your system starts to swap. Then moving back to your terminal will have to load resources from swap to ram which might take some time. You usually would hear/see your HDD working in that case.
But I have to say I THINK I saw this or a similar issue myself (~3 times), where I did have some disc activity after switching back to my terminal (it felt too short for being actually loaded from swap, my HDD is.... slow...), but it did not respond immediately after disc went quiet and system activity went to idle, there were another 5-10 seconds where the terminal was unusable. I thought this probably was an issue with tmux - @Pouya-moh do you use tmux, too? But I also used OMZ+tmux before and never saw this issue with that setup (after swapping back I could use it immediately)

@ff2000, I have a HDD. In terms of Ram I have 8 on a linux machine. It is not really a mater of memory or page swap issue since I didn't have this problem with oh-my-zsh. I also constantly check my ram usage (important for my work). I had it the issue with xfce-terminal and now that I'm testing konsole for couple of days, I see it less frequently.

I have this same issue on a newly re-imaged retina macbook pro (SSD with 16GB ram) so I don't think its hardware.I too recently switched from OMZ.
On another note when I have my terminal open now it freezes at random times just as described here and its almost as if finder is crashing as the date, user, finder and notification center disappear from the top status bar and when they pop back in thats when the freezing stops. The load on my box also jumps to like 7.
Trying to trace and look through the logs for an explanation but so far nothing.

@dude051 is the problem reproducible? What I mean is, are there a set of steps that you can take to always reproduce the problem?

So I may have narrowed it down to a notifyd issue with OS X. Sorry to have pointed at prezto as the issue, but it looks like tmux is the culprit and my issues may be unrelated to the OP.
https://github.com/tmux/tmux/issues/108


qrenderdoc: cannot proceed because VCRUNTIME140_1.dll was not found.

Description
Installing and running latest renderdoc does not launch when vcruntime140_1.dll is not on the system.
Steps to reproduce

Install RenderDoc 1.15 (28 July 2021) for Win x64 -- either installer or ZIP.
Run qrenderdoc.exe
notice vcruntime140_1.dll message box 3 times -- cannot get in.

Environment

RenderDoc version: 1.15 (28 July 2021) for Win x64
Windows 10 home
OpenGL

My Windows/System32 folder has just these DLLs:

Consider statically linking VC libraries into your exe -- or shipping all the DLLs with it.
EDIT: I was previously running an earlier version of renderdoc just fine -- hit this issue only when trying to get on the latest version. Thanks!

The zip builds does require you to have the appropriate VC runtimes set up, but the installer should be shipping the proper redistributable and installing it as needed. Can you please check to see which VC runtimes you have installed?
If you completely uninstall RenderDoc and reinstall v1.15 from fresh, does that work correctly?

Hi Baldur, Unfortunately not - hitting the same problem.

I had to go all the way back to 1.11 to get a build that works. 1.12 is the earliest build that does not.

That makes sense as in v1.12 I updated the version of Qt, and it's only Qt that needs the VS2019 runtime.
I also asked you to please check which VC runtimes you have installed, can you do that and let me know which ones you have installed?

Hi sorry, I forgot about that part. These are the only ones I have installed...

OK thanks, so you are missing the 2019 redist that the newer Qt builds require, for some reason that's not getting detected by the installer. The 2017 redist you have covers the RenderDoc build itself which is presumably why older versions of RenderDoc work.
Can you export the registry key at HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\VisualStudio and share that? either just pasting the contents or into a pastebin somewhere.

Thanks Baldur. Sorry this took so long:
vs.reg.zip

OK that's annoying, it looks like the registry key I was using to determine if the redist is available is present, but the version number is too low. Looking on google there's not an easy key to look for and instead you're supposed to compare against the major/minor version. Which of course is incredibly obtuse to do in an installer.
Can you test this installer? it's built from latest code as of the above commit. If that works, please uninstall it and then re-install v1.15. Because of how the redistributables work it should stay installed even when you uninstall the temporary RenderDoc build, so v1.15 will work. And the above fix will apply to the real installers from v1.16 onwards.

Hi Baldur, unfortunately that installer didn't work either.

Can you uninstall RenderDoc completely and run that installer with msiexec /i "installer_test_2359.msi" /L*V "installlog.txt" and upload that log so I can see what's going on?

Absolutely, log attached:
installlog.zip
:

That log shows that the installer is requesting to install the vc redist. If it's still not working I don't think this is a RenderDoc bug but I think it's a bug with the vc redist, or maybe the msm merge modules.
One curious note - the log shows that you have the 2019 runtime installed for 32-bit (you can double check in the registry by looking at HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\VisualStudio\14.0\VC\Runtimes\X86 but the installer is reading a version of 14.20, which is the oldest 2019 runtime available AFAIK). Maybe the bug is that the vc redist installer is falsely detecting the 32-bit version as being installed and so refusing to install the 64-bit version when RenderDoc needs.
Either way I don't think there's anything more I can do from my side on this, and I think trying to file a bug on this with Microsoft is pointless as it will most likely get ignored.
I'd suggest that instead you manually install the latest x64 and x86 VS2019 runtimes from MS and then RenderDoc will run fine.


xtensa/esp32: SPI Flash driver uses global sem for all MTD

Summary
SPI Flash driver uses global sem for all MTD。
Because all MTDs operate the main SPI Flash, so not only MTD internal function should be mutex, but also MTDs should be mutex.
Impact
Fix CPUs call different MTDs' functions operate SPI Flash without mutex protection.
Testing
SPI Flash file system testing.

@donghengqaz what was the issue? From the diff it looks that the only change is removing the mutex from the struct.

Thanks @acassis .

@donghengqaz what was the issue? From the diff it looks that the only change is moving the mutex out of the struct.

In my another MR: https://github.com/apache/incubator-nuttx/pull/2138, I will create some MTDs, if every MTD uses an independent semaphore, in some case like:
task_A reads MTD_1 and task_B writes MTD_2, and they run on different CPUs, the semaphore can't work as we expected.

I see, thanks for the explanation @donghengqaz. @acassis LGTM, please merge if you don't have any other concerns.

Ok, everything was fixed, thank you @donghengqaz


Pass struct length to serialize_map in case of #[serde(flatten)]

I am developing a serde-module for a binary format where I always need to know the number of fields of a struct.
This works well as long as I don't use #[serde(flatten)]. With flatten, the code generated by derive(Serialize) passes None to serialize_map.
I guess this sort of makes sense. The struct that is being flattened is completely opaque to serde apart from it needing to implement Serialize. Still, it would be nice if we didn't lose the information about how many fields we are about to serialize just because they are being flattened.
Would it make sense to extend the the interface of Serialize with some sort of "size-hint"?

I can see how building a size hint into Serialize would be helpful for some formats, but I don't feel that this is a widespread enough use case to justify expanding the API for this. Thanks anyway for the suggestion!


Respect Minimum time on Time mode

Bug
Cannot set minimum time to select when in time mode
Environment info

React native info output:
System:
    OS: macOS 10.15.2
    CPU: (4) x64 Intel(R) Core(TM) i7-6567U CPU @ 3.30GHz
    Memory: 125.68 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 10.11.0 - /usr/local/bin/node
    Yarn: 1.10.1 - /usr/local/bin/yarn
    npm: 6.9.0 - /usr/local/bin/npm
    Watchman: 4.7.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>77832
    Xcode: 11.3.1/11C504 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.8.6 => 16.8.6
    react-native: 0.60.4 => 0.60.4
  npmGlobalPackages:
    react-native-cli: 2.0.1

Library version: 2.2.3
Steps To Reproduce

Create an time field
2.add the minimumdate as new Date(today2.getTime() + 7200000)
...

Describe what you expected to happen:

The time picker should respect the limit
In my case I have two separate Datetimepickers, one for date and one for time

Reproducible sample code
minimumTime: new Date(today2.getTime() + 7200000)
<DateTimePicker
          titleIOS={dateTimePickerMode === 'date' ? 'Pick a date' : 'Pick a time'}
          mode={dateTimePickerMode}
          isVisible={isDateTimePickerVisible}
          isDarkModeEnabled={colorScheme === 'dark'}
          onConfirm={this.onDateTimePickerConfirm}
          onCancel={this.hideDateTimePicker}
          minimumDate={dateTimePickerMode === 'date' ? this.state.minimumDate : this.state.minimumTime}
          maximumDate={dateTimePickerMode === 'date' ? this.state.maximumDate : null}
          date={initialSelectedDate}
          minuteInterval={dateTimePickerMode === 'time' ? 15 : 1}

any solution here?

up

+1

Any solution on android so far?

Hello, this bug / feature request is being closed because Android Time picker does not support this functionality. This is now clarified in the readme: https://github.com/react-native-community/datetimepicker/pull/183/files


roslaunch rgbd_ros_to_lcm lcm_republisher.launch Error

Hi, thank you for your ROS-LCM converter. I want to use it with realsense D415.
Firstly i installed the realsense2_camera as you adviced.and

roslaunch realsense2_camera rs_rgbd.launch

But after normal install rgbd_ros_to_lcm , when i run

roslaunch rgbd_ros_to_lcm lcm_republisher.launch

it produce error as follow:

it seems when the program run to line 162 of lcm_republisher.cpp, it can't step over it, and the process die
sensor_msgs::Image::ConstPtr rgb_msg = ros::topic::waitForMessage<sensor_msgs::Image>(rgb_topic, nh_);
My system: ubuntu 16.04, ROS kinetic.
Any suggestion would be appreciated!

@5yler .my colleague run it successfully, so it may be the reason of my computer's configuration. Thanks again for your ROS-LCM converter.


[Request] Do not save k3s-releases.yaml in same dir

Maybe it's my OCD, but I don't like the k3s-releases.yaml being created every time I run hetzner-k3s create in the cwd. Would it be possible to store this file $TMPDIR instead? Or it it supposed to be e.g. checked in?

The file gets created only once  to cache the list of releases, because Github's API has a rate limit which is easily hit. I can change the file to a temporary path though.

Done in v0.7.0. I guess we can close?

Thank you for the suggestion!


Use batch operations for Validators

Feed the NNPolicy with batches of data can speed things up. A few things to note:

each game will have its own "ended" status which can be really tricky. Current library does not organically track ended status as an array.
don't move tensors too frequently between GPU and CPU.
I expect there might also be some structural changes of the current code base to accomodate for this.

No longer needed at the current moment.


setState_dTx does not work correctly

Reported by mdempsey on 4 Sep 2013 09:57 UTC
The function Modelica.Media.Interfaces.PartialLinearFluid.setState_dTX does not produce correct results (at least in Dymola) because the algorithm that is used to calculate state.p references state.T rather than the function input argument of T.
In the compiled model state.T is assigned after state.p is calculated which means the calculation of state.p sees the wrong temperature and therefore gives wrong results.
The current algorithm is:
state := ThermodynamicState(p=((d - reference_d) +
  (state.T - reference_T)*beta_const*reference_d)
  /(reference_d*kappa_const) + reference_p, T=T);

This should be rewritten as:
state := ThermodynamicState(p=((d - reference_d) +
  (T - reference_T)*beta_const*reference_d)
  /(reference_d*kappa_const) + reference_p, T=T);

Migrated-From: https://trac.modelica.org/Modelica/ticket/1267

Comment by beutlich on 29 Oct 2015 20:45 UTC
The proposed fix is applied by b06f0c94aa901c50a3676d59d980010ebb43e821. Please cross-check and revert should the need arise.
I guess the regression results are affected by this commit. Actually I do not understand why such tickets (I mean rather critical and with a proposed patch for a single line change) are kept unconsidered and unaccepted for years.

Changelog modified by beutlich on 29 Oct 2015 20:45 UTC
Fixed the computation of the thermodynamic state in Modelica.Media.Interfaces.PartialLinearFluid.setState_dTX.state

Modified by beutlich on 29 Oct 2015 20:47 UTC

Changelog modified by beutlich on 29 Oct 2015 20:47 UTC
Fixed the computation of the thermodynamic state Modelica.Media.Interfaces.PartialLinearFluid.setState_dTX.state

Comment by hubertus on 29 Oct 2015 22:10 UTC
I have in the past regularly searched for tickets with me as an owner and the upcoming Milestone. I have never seen this ticket though. Not sure what went wrong: I usually fix such minor things as soon as I note them.


Feature request: take pictures

Sometimes I would like to take a picture of bills but no way to do it now.
Could be interesting to implement it.

Imagine if they could be OCRd too... whistles

To motivate development of this, I have set up a bounty for this feature. Open to anyone who can accomplish the task. Feel free to sweeten the offer if you care to.

It would indeed be nice to take a picture of bills. Of course you would still have to manually enter the transaction details in GnuCash, and when you sync with desktop, the images will be ignored. But apart from that, it could work.
@bugmenot2 OCR is almost completely out of the question.

If you do keep a long-term feature wishlist for the project somewhere, I suggest adding the concept of GnuCashAndroid OCR on image capturing to it, in case someone decides to make that happen, or if tools become available in the future to do that more easily - it may sound like a long shot now, but it will not hurt to have such long term goals being noted officially as goals by the project. With enough development effort, it should be technically possible to somehow convert some graphic information to text for transaction entry. I have a business card scanner with some proprietary Chinese software that does that with reasonable success.
Regarding the first (hopefully attainable task) of doing image capture and transfer - I suggest consideration of accomplishing the transfer for sync of file attachments from GnuCashAndroid to a Gnucash thus:

In GnuCash Android, spawn phone camera to take pictures of things on command, store captured images in a folder "GnuCashAndroidCaptures" alongside the GnuCash Android App, or in a user-designated location on the SD card or whatever.
Create a relative path file link "./GnucashAndroidCaptures/file_blah_blah.foo" included for each transaction that has a file association created in the app. (does GnuCash know how to handle relative paths references yet? - I'll assume it does, or will at some point)
On data export, create a gnucash Data file, and create also a folder in the same directory called "GnuCashAndroidCaptures" that contains all the files that are linked by the transactions
To make the attachments work in GnuCash, the exported data file and it's companion folder would be copied to where the user keeps their Gnucash Data

Regarding item No. 2 above and some continuity with bugzilla. A Gnucash relative path enhancement request exists.
https://bugzilla.gnome.org/show_bug.cgi?id=767773

Following up this discussion, as I am planning on getting it going, No.2 has been implemented, but looks like it'll only be available by spring 2017 with release 2.7.

@gabrielsimoes good to hear that you're looking into getting this off the ground.
I would suggest writing out your plan for the feature and how it will impact other parts of the system so that it can be discussed and you get input before starting work.
Off the top of my head, questions like will this be an Android only feature which is not syncable to desktop. (my guess is yes, because even with No.2 implemented, GnuCash desktop just doesn't support receiving files from elsewhere. Unless we hack it - which is another level of effort).

Workflow for image capture and how it will integrate with transactions
What to do with export to QIF and OFX

And much more...

Sure.

I'm planning on allowing the user to input an associated URL (what Gnucash calls "location") or a file from the storage or the camera, which would be copied/saved to a specific folder.
A Uri field would keep the relative path to the file inside each transaction. Image capture and deletion would be done inside the transactions form, or something like that.
Synchronization would keep the associated string from phone to desktop, but files would only be accessible if the user copied the associated files folder to the desktop and configured the relative path (feature no. 2).
This way, it might be good to provide a way to export the files to a .zip or something.
I haven't thought much about what to do with QIF and OFX.

It is rather incomplete, but you can check what I'm trying to do here: https://github.com/gabrielsimoes/gnucash-android/tree/develop

Great to see this feature has been started! Well done and thank you! :-)
If my programming expertise wasn't limited to bash and .ods sheet formulae I'd give you a hand. I'd be willing to contribute some $$ as an incentive though.

@gabrielsimoes Last commit was 9 months ago. Have you abandoned this?

I'm not working on it now, and probably won't do so until January next year. Thus you can say I abandoned it, sorry.

imho:
The image could be stored in the "slot" elements.
The attachment could be an image or document, and encoded as Base64.
The image should be converted to grayscale or black+white to save space.


Modify not to refresh when language setting is changed.

Fix #264.

Awesome...!

Sorry this pr is not enough...


Ensure that the stopped channel gets closed

Regardless of whether we are using a new or an existing cluster.
Only cleanup the cluster if this is a new cluster.
Fixes tests hanging when re-using an existing Kind cluster.

Intermittent E2E test failure https://app.circleci.com/jobs/github/improbable-eng/etcd-cluster-operator/419 will be fixed by #88


Proposal: user-defined time index interface

Some time ago there was a detailed discussion about TimeSeries.jl functionality. In particular there was an idea to define a separate timestamps vector that support different time types: https://github.com/JuliaStats/TimeSeries.jl/issues/482#issuecomment-778704466
The main reason for it is to support different time representations:

time points and continuous intervals (maybe like in https://github.com/invenia/Intervals.jl),
physical periods and astronomical clock,
scales from milliseconds to days and years,
combined indexing by integer and lookup by physical value (maybe like in https://github.com/mcabbott/AxisKeys.jl).

Is there any plans or interest to support such functionality in this package?

Some time ago there was a detailed discussion about TimeSeries.jl functionality. In particular there was an idea to define a separate timestamps vector that support different time types: JuliaStats/TimeSeries.jl#482 (comment)

TSFrames already separates out index from data. In its simplest form:
struct TSFrame
    coredata :: DataFrame
end

The first column of the DataFrame is the index. This is done to provide the flexibility to support other types of index.
Having said that, there is plan to implement custom typed index for TSFrame. Currently, the index is <: Union{Int, TimeType} but it can, essentially, be any type as long as it can be sorted (I can't remember anything else but I am sure there are a few other constraints).
There is some related discussion in #165.

The main reason for it is to support different time representations:

time points and continuous intervals (maybe like in https://github.com/invenia/Intervals.jl),
physical periods and astronomical clock,

As I mentioned above, ideally, these should be possible.

scales from milliseconds to days and years,

This is already possible with DateTime index. There are functions to support this scaling like to_period() and endpoints(). For applying a function over a period there is apply().

combined indexing by integer and lookup by physical value (maybe like in https://github.com/mcabbott/AxisKeys.jl).

If I understand from what you are asking, this indexing is already possible. See: https://xkdr.github.io/TSFrames.jl/stable/user_guide/#Indexing-and-subsetting
For some reason, the API reference doc isn't being generated for all the getindex() overloads but you can look at the source here: https://github.com/xKDR/TSFrames.jl/blob/main/src/getindex.jl.

Is there any plans or interest to support such functionality in this package?

Already answered above.


merge with istanbul-lib-hook

istanbul-lib-hook, and append-transform have similar functionality; I think we should figure out how to merge the two.
istanbul-lib-hook has the added advantage that it handles hooking into the vm.createScript method:
https://github.com/istanbuljs/istanbul-lib-hook/blob/master/lib/hook.js#L109
I'd love to get this functionality added to nyc; @jamestalmage, @gotwarlost what do you think the best option is for this? Perhaps we could keep both libraries, with append-transform as a dependency in istanbul-lib-hook?

@jamestalmage says it best here: https://github.com/istanbuljs/append-transform/issues/8#issuecomment-228942804


Add a CHANGELOG

Format is loosely based on keepachangelog.com, the format that a lot of
other STM32 HAL crates use as well.
The main issues with single-file changelogs is that they cause a lot of
conflict potential. One solution is writing changelog entries to
separate files, and merging those before a release. However, that
requires additional tooling and is a bit cumbersome.
Instead, I added CHANGELOG.md merge=union to .gitattributes. I'm not
sure if it always works ideally, but at least it should reduce the
conflict potential a lot. And in the case something breaks, we can
always fix it manually.

PS: I also tagged all releases since v0.5.0!

Thank you, @dbrgn, great work!
I'm a bit skeptical of requiring contributors to add changelog entries. That puts a higher burden on them, which might be unreasonable to expect in the case of small drive-by contributions. It's also another thing to remember on the maintainer side, and I suspect it will lead to some pull request piling up, if we're strict about it.
Still, this PR is definitely an improvement, and we can work out the kinks as we go. Maybe we could just add a tag for PRs that are merged without changelog entries, and have someone go through them before the next release (preferably whoever is asking for a release to be made :smile:).


[FEATURE] - Add the ability to back up and restore the Input Bindings file

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.
Hey there,  I'm back with another ridiculous request :).
When there are game updates or you have multiple controllers, sometimes the input bindings are reset.   Having the ability to quickly recall (in a controlled way) a previous saved file would be nice.   I travel with a laptop,  so I have different controllers (Farm stick. joysticks and steering wheels) at different locations,  being able to change the config file would be a really nice feature.
Thanks for your consideration.  Keep up the great work.  This is by far the best Mod Manager ever!
Scooter
Describe the solution you'd like
No response
Anything else?
No response

yeah.  this has come up a few times.  I would be open to a mostly-manual option.  I've played with trying to automate it, but it's really, really difficult to know when that file changes - and when it does, what the "best" thing is to do with the new version.  But I could see a way to allow you to explicitly tag "known good versions" of the file, and restore as wanted.

oh,  I wasn't even thinking of automating it.   Just a way to "name" a version and to have multiple copies that could be selected and "make active".   It's easy to do manually if you know what you're doing,  but having it all part of a "managing application" is really nice.   Plus, it may provide a way to assign buttons outside of the game someday.  :)   I know that file structure is not easy to read when you open the xml,  but you've already figured out many things that I didn't think would be possible.  :)    Anyway,  it's just an idea,  if you want to pursue, that would be cool, if not, no worries.   Thanks for taking the time to look.

haha.  yeah, I looked at it once with a view to messing with it outside the game - I dunno that I'll ever get there, but a save/copy/manage tool is totally possible.  Maybe with a view function too - things get funky with controllers, but I think I can pull enough information to make a decent guess at showing what does what

As a side note, wanna send me a couple input binding files if you have kept (it'll make a viewer easier when I get that far)? Either here or in any of the discords we share works - we're both in Vertex, Mods & Links, UMC, and the official one looks like.

closing as finished


Update tree when editing/merge files

It will be great that difference tree updates when edit/merge file, so we can see live it changes.

Currently, there is the refresh button

Comparing for each save sounds fair to me.
Of course, there will be an option to turn it off so for large projects it will not cause performance issues.
Thanks!

You might update only the file that has changed in the tree without any performance cost

Indeed, this is a better approach, I was thinking about "time to develop" compare "value". Compare all over again, in most of the cases will be good enough and require less time to develop.
WDYT?

I have big projects and it could be very long to compare again and again.
I don't think it is long to develop cause extension API provides easy way to lookup file changes:
https://code.visualstudio.com/api/references/vscode-api
createFileSystemWatcher(globPattern: GlobPattern, ignoreCreateEvents?: boolean, ignoreChangeEvents?: boolean, ignoreDeleteEvents?: boolean): FileSystemWatcher

Anyway the extension will need listen to changes.
With the "compare all" approach, this is the only thing that need to be added.
With the "compare only changed file" there are more things to do.
I agree that the "compare only changed file" is better.
I'm curios, how long it takes to compare your project in the first time or when you hit "refresh"?

Keep

Keep


Make call to xylem-api when update_marathon is called, save returned values from xylem-api and show postgres fields as read-only in edit view

👍 🍰 💫 🌟


Update README.md

Grammar fix.

@tranthibaokhanh How do I merge to master? I suppose I probably need permission?

Hi Gen, what is this about?
Axel
On Mon, Nov 19, 2018 at 11:55 AM Genevieve Tran <notifications<EMAIL_ADDRESS>wrote:
Merged #1https://github.com/tranthibaokhanh/thoughtloungev2/pull/1 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/tranthibaokhanh/thoughtloungev2/pull/1#event-1975609088, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOY8BjDyzszFdg3ZanJVTyZSkn7KR3c7ks5uwwzAgaJpZM4GWHQO.
--
Axel Cramer
MBA Candidate | Class of 2020
UCLA Anderson School of Management
Mobile:<PHONE_NUMBER>
Email: axel.cramer.2020@anderson.ucla.edumailto:nicole.samanez.2019@anderson.ucla.edumailto:nicole.samanez.2019<EMAIL_ADDRESS>https://www.linkedin.com/in/axelcramerhttps://www.linkedin.com/in/axelcramer/