mpi-win64-smp various issues

Original issue: https://charm.cs.illinois.edu/redmine/issues/489

After copying the conv-mach-smp.* files from net-win64 I get the following errors:
../bin/charmc   -o machine.o  -DFOR_CPLUS=1 machine.c
machine.c
machine.c(347) : warning C4047: 'initializing' : 'CmiNodeLock' differs in levels of indirection from 'void *'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(357) : error C2223: left of '->recv' must point to struct/union
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(357) : warning C4033: 'CmiMyRecvQueue' must return a value
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(365) : error C2054: expected '(' to follow '_Cmi_mype'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(372) : error C2223: left of '->pe' must point to struct/union
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(372) : error C2198: 'pe_lToGTranslate' : too few arguments for call
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(374) : error C2059: syntax error : 'constant'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(377) : error C2059: syntax error : 'type'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(380) : error C2059: syntax error : 'constant'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(383) : error C2059: syntax error : 'type'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(386) : error C2059: syntax error : 'constant'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(405) : warning C4047: 'initializing' : 'CmiState' differs in levels of indirection from 'int'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(445) : warning C4047: 'initializing' : 'CmiState' differs in levels of indirection from 'int'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(1177) : warning C4047: '=' : 'CmiState' differs in levels of indirection from 'int'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(1386) : warning C4047: 'initializing' : 'CmiState' differs in levels of indirection from 'int'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(1422) : warning C4047: 'initializing' : 'CmiState' differs in levels of indirection from 'int'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(1442) : warning C4047: '=' : 'CmiState' differs in levels of indirection from 'int'
c:\cygwin64\home\jim\charm\mpi-win64-smp\tmp\machine-common-core.c(1463) : error C2065: '_Cmi_sleepOnIdle' : undeclared identifier
machine.c(497) : warning C4047: 'initializing' : 'CmiState' differs in levels of indirection from 'int'
machine.c(892) : error C2223: left of '->recv' must point to struct/union
machine.c(892) : error C2198: 'PCQueueEmpty' : too few arguments for call
machine.c(1000) : warning C4047: 'initializing' : 'CmiState' differs in levels of indirection from 'int'
unix2nt_cc Fatal Error in directory /home/jim/charm/mpi-win64-smp/tmp
   Error executing CL.EXE /nologo /W1 /EHsc /D_WINDOWS /DNOMINMAX /IC:\PROGRA~2\WI3CF2~1\8.0\Include /IC:\PROGRA~2\MICROS~1.0\VC\Include /MT /Fomachine.o /D_CRT_SECURE_NO_DEPRECATE /IC:\PROGRA~1\MI564C~1\Inc /IC:\cygwin64\home\jim\charm\mpi-win64-smp\include /D__CHARMC__=1 /DFOR_CPLUS=1 /c /Tcmachine.c
unix2nt_cc exiting...
Fatal Error by charmc in directory /home/jim/charm/mpi-win64-smp/tmp
   Command unix2nt_cc -D_CRT_SECURE_NO_DEPRECATE -I /cygdrive/c/PROGRA~1/MI564C~1/Inc -I../bin/../include -D__CHARMC__=1 -DFOR_CPLUS=1 -c machine.c -o machine.o returned error code 1
charmc exiting...
Makefile:471: recipe for target `../lib/libconv-cplus-n.a' failed

Original date: 2014-05-26 05:26:04

One of the NAMD people - please take a first look. Upping priority.

Original date: 2014-11-09 16:39:35

Ping. It would be very helpful if the necessary fixes got fully implemented before the core meeting this week. It should hopefully all be fairly simple changes. If you need help, ask.

Original date: 2015-09-17 20:19:45

Ping. Was this merged?

Original date: 2015-10-13 20:02:33

There are some changes on ronak/mpi-win64-smp-fix, but it doesn't fully fix the problem.  It builds fine, but then there is a runtime failure, which I'm still trying to diagnose and fix.

Original date: 2015-10-26 22:21:39

Gerrit Link: ~~https://charm.cs.illinois.edu/gerrit/#/c/885/~~ https://github.com/UIUC-PPL/charm/commit/8c55a0801d056610d7d59b6dc1b668ef8e0df05c

Original date: 2015-10-28 03:26:10

On testing with --no-build-shared --enable-tracing --enable-tracing-commthread -optimize I get the following warning on every file:
c:\cygwin64\usr\tmp\charm-6.7.0-proj-build-2015-oct-27-179786-mpi-win64-smp\charm-6.7.0-pre\mpi-win64-smp\tmp\conv-config.h(137) : warning C4005: 'CMK_NODE_QUEUE_AVAILABLE' : macro redefinition^M
c:\cygwin64\usr\tmp\charm-6.7.0-proj-build-2015-oct-27-179786-mpi-win64-smp\charm-6.7.0-pre\mpi-win64-smp\tmp\conv-mach-smp.h(4) : see previous definition of 'CMK_NODE_QUEUE_AVAILABLE'^M
../bin/charmc  -optimize  -I.   -c -o Refiner.o Refiner.C
and two unresolved symbols:
libconv-cplus-y.a(machine.o) : error LNK2019: unresolved external symbol TraceTimerCommon referenced in function handleOneRecvedMsg^M
libconv-cplus-y.a(machine.o) : error LNK2019: unresolved external symbol START_TRACE_RECVCOMM referenced in function PumpMsgs^M
Building --no-build-shared --with-production the warnings are still there but tests link and run successfully.

Original date: 2015-10-28 21:04:34

We're aware of the warning on every file.
Do you mean that you're getting errors when building charm or when building an application?  I'm able to build with ./build charm++ mpi-win64 smp --no-build-shared --enable-tracing --enable-tracing-commthread -optimize with no errors.  However, I am getting errors building when actually building applications (unresolved START_TRACE_RECVCOMM).  We'll take a look.

Original date: 2015-10-28 22:10:19

Turns out this is broken across all MPI builds, as that symbol is only defined when comm thread tracing is turned off.  Commit 8bbf1f4855f1e87b51fae1704f4a1b4ab8b081b9 took out the definition of START_TRACE_CREATION, as it should now be traced in machine-common-core.  I'll talk to Bilge to see what the recommended way to trace the comm thread is with the current machine layer code stack.

Original date: 2015-10-29 01:20:06

Thanks. I only care about mpi-smp on Windows and never actually do any tracing so this isn't an issue. I only noticed because I've added compiling megatest to my builds.


BRANCH smir-syntax: Local / LocalToken not parsing

Description
Local is not recognised in function signature when attempting to parse any function signature.
INFO 2023-09-05 14:55:43,510 pyk.ktool.kprint - Running: kast /tmp/tmpus8i2vey --definition /home/daniel/.kbuild/mir-semantics/5a0ac5c/target/v6.0.82/llvm --input program --output pretty --sort Mir
[Error] Inner Parser: Parse error: unexpected token '_1' following token '('.
        Source(/tmp/tmpus8i2vey)
        Location(3,13,3,15)
        3 |     fn sum_to_n(_1: usize) -> usize {

At first inspection this appears to be because of the definition of LocalToken as "_" Int [token], but changing for previous definition of r"_[0-9]+" [token] produces the same error. The error can be trivially avoided by hard coding "_1" instead of Local, that is to say that this parses as a valid function signature:
"fn" DefPath "(" "_1" ":" Type ")" "->" Type

Reproducer
cd kmir
make build
source set-env.sh
poetry shell
cd src/tests/integration/handwritten-rust/
kmir parse --verbose --debug --output pretty test-sum-to-n.mir

k versions 6.0.0 and 6.0.82 were tested and had same results.
Solution
Unsure at this stage, more investigation required.

It is also worth mentioning that if the first _1 is hardcoded in a string, the next fails. So it isn't JUST in the function signatures that the problem arises in case that was unclear.

Tried around but no solution found. Will need expert's help.

Hey guys, could you please try to use [prefer, token, prec(2)] as defined in Integer Syntax?
I tried it here, and it parses _1 as expected, although I got a new error:
[Error] Inner Parser: Parse error: unexpected token 'scope' following token
';'.
	Source(/tmp/tmpiohyjr9p)
	Location(11,5,11,10)
	11 |	    scope 1 {

But I wonder if it's related to the same issue as this or if it's something completely different!

I tried it and it passed for me:
radu@radu:~/work/mir-semantics/kmir$ . /home/radu/.cache/pypoetry/virtualenvs/kmir-meUFJs27-py3.10/bin/activate
(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir$ cd src/tests/integration/
proofs/    test-data/
(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir$ cd src/tests/integration/
proofs/    test-data/
(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir$ cd src/tests/integration/test-data/
compiletest-rs/   handwritten-mir/  handwritten-rust/
(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir$ cd src/tests/integration/test-data/
compiletest-rs/   handwritten-mir/  handwritten-rust/
(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir$ cd src/tests/integration/test-data/handwritten-rust/
(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir/src/tests/integration/test-data/handwritten-rust$ kmir parse --verbose --debug --output pretty test-sum-to-n.mir
INFO 2023-09-05 17:56:08,871 pyk.ktool.kprint - Running: kast /home/radu/.kbuild/mir-semantics/8bbd6b6/target/v6.0.81-0-g4ca5636989/llvm/parser_Mir_MIR-PARSER-SYNTAX --definition /home/radu/.kbuild/mir-semantics/8bbd6b6/target/v6.0.81-0-g4ca5636989/llvm --module MIR-PARSER-SYNTAX --sort Mir --gen-glr-parser
INFO 2023-09-05 17:56:10,911 pyk.ktool.kprint - Completed in 2.040s with status 0: kast /home/radu/.kbuild/mir-semantics/8bbd6b6/target/v6.0.81-0-g4ca5636989/llvm/parser_Mir_MIR-PARSER-SYNTAX --definition /home/radu/.kbuild/mir-semantics/8bbd6b6/target/v6.0.81-0-g4ca5636989/llvm --module MIR-PARSER-SYNTAX --sort Mir --gen-glr-parser
INFO 2023-09-05 17:56:10,912 pyk.ktool.kprint - Running: kast /tmp/tmpb_po2pip --definition /home/radu/.kbuild/mir-semantics/8bbd6b6/target/v6.0.81-0-g4ca5636989/llvm --input program --output pretty --sort Mir
INFO 2023-09-05 17:56:12,210 pyk.ktool.kprint - Completed in 1.297s with status 0: kast /tmp/tmpb_po2pip --definition /home/radu/.kbuild/mir-semantics/8bbd6b6/target/v6.0.81-0-g4ca5636989/llvm --input program --output pretty --sort Mir
fn sum_to_n :: .FunctionPath ( _1 : usize  , .ParameterList ) -> usize  { debug n => _1 ;  .DebugList let mut _0 : usize  ;  let mut _2 : usize  ;  let mut _4 : bool  ;  let mut _5 : usize  ;  let mut _6 : usize  ;  let mut _7 : usize  ;  .BindingList scope 1 { debug sum => _2 ;  .DebugList let mut _3 : usize  ;  .BindingList scope 2 { debug counter => _3 ;  .DebugList .BindingList .ScopeList }  .ScopeList }  .ScopeList bb0  : { _2 = const 0_usize ;  _3 = _1 ;  .Statements goto -> bb1  ; }  bb1  : { _5 = _3 ;  _4 = Gt ( move _5 , const 0_usize ) ;  .Statements switchInt ( move _4 ) -> [ 0 : bb3  , .SwitchTargets , otherwise : bb2  ] ; }  bb2  : { _6 = _3 ;  _2 = Add ( _2 , move _6 ) ;  _7 = _3 ;  _3 = Sub ( move _7 , const 1_usize ) ;  .Statements goto -> bb1  ; }  bb3  : { _0 = _2 ;  .Statements return ; }  .BasicBlockList }  fn test_sum_to_n :: .FunctionPath ( .ParameterList ) -> ( ) { .DebugList let mut _0 : ( ) ;  let  _1 : usize  ;  let mut _4 : usize  ;  let mut _5 : bool  ;  let mut _6 : ! ;  .BindingList scope 1 { debug n => _1 ;  .DebugList let  _2 : usize  ;  .BindingList scope 2 { debug golden => _2 ;  .DebugList let  _3 : bool  ;  .BindingList scope 3 { debug sucess => _3 ;  .DebugList .BindingList .ScopeList }  .ScopeList }  .ScopeList }  .ScopeList bb0  : { _1 = const 10_usize ;  _2 = const 55_usize ;  .Statements _4 = sum_to_n :: .ExpressionPathList ( _1 , .ArgumentList ) -> bb1  ; }  bb1  : { _3 = Eq ( move _4 , _2 ) ;  _5 = Not ( _3 ) ;  .Statements switchInt ( move _5 ) -> [ 0 : bb3  , .SwitchTargets , otherwise : bb2  ] ; }  bb2  : { .Statements _6 = core :: panicking :: panic :: .ExpressionPathList ( const "assertion failed: sucess" , .ArgumentList ) ; }  bb3  : { .Statements return ; }  .BasicBlockList }  fn main :: .FunctionPath ( .ParameterList ) -> ( ) { .DebugList let mut _0 : ( ) ;  let  _1 : ( ) ;  .BindingList .ScopeList bb0  : { .Statements _1 = test_sum_to_n :: .ExpressionPathList ( .ArgumentList ) -> bb1  ; }  bb1  : { .Statements return ; }  .BasicBlockList }  .Mir

(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir/src/tests/integration/test-data/handwritten-rust$ git status
On branch smir-syntax
nothing to commit, working tree clean
(kmir-py3.10) radu@radu:~/work/mir-semantics/kmir/src/tests/integration/test-data/handwritten-rust$

@radumereuta Yes, the same implementation passes on the master branch, but fails on the defector/smir-syntax branch (there is more structures defined).
Couldn't figure out why.

Hey guys, could you please try to use r"_[0-9]+" [prefer, token, prec(2)] as defined in Integer Syntax?
I tried it here, and it parses _1 as expected, although I got a new error:
[Error] Inner Parser: Parse error: unexpected token 'scope' following token
';'.
	Source(/tmp/tmpiohyjr9p)
	Location(11,5,11,10)
	11 |	    scope 1 {

But I wonder if it's related to the same issue as this or if it's something completely different!

Thank you, Robert, I couldn't think of a good reason why it is fail.
Master branch has the same implementation but no problem there.
On the refactor branch, more identifiers patterns are defined, I am suspecting there is a conflict or ambiguity.
It would be helpful to know how tokens are recognised and matched to patterns.
Will there be more detailed documentation? If not, pointers to the implementation in K would also be helpful. @radumereuta please look here too.

Ping @radumereuta

I'm working on this https://github.com/runtimeverification/k/issues/3647
to make it easier to debug such problems.
I met with Daniel last week for a short debug session, but the definition is very large, and hard to figure out where the problem is coming from.
One thing that I know for sure is that there are some productions that are too general and match too much.

FWIW, I tried to make this work, and I may have succeeded, see the syntax-demo branch. My guess is that someone already fixed at least some of the the issues mentioned above.
Also, FWIW, I think that having "()" as a token is questionable (e.g. in the syntax FnSig ::= ... | "fn" DefPath "()" "->" Type declaration), but without it there is a weird ambiguity related to empty list parsing (I get an error about the empty list being parsable as either .Stuff or as _,_Stuff(.Stuff, .Stuff)), so I guess that's the reason for using a token there. I'm guessing that's a K bug, most likely empty list parsing should just work, and the declaration should be syntax FnSig ::= ... | "fn" DefPath "(" Args ")" "->" Type

Empty lists parsing should just work in programs.
In the semantics you need to specify some non whitespace characters. That's why you have .List instead of nothing.
Otherwise the parser is going to fail unpredictably.
So, @virgil-serbanuta what production are you talking about?

@radumereuta I'm sorry, I was wrong. The error was:
[Error] Inner Parser: Parsing ambiguity.
1: syntax ArgList ::= Operand "," ArgList [userList(*)]
    `_,__MIR-SYNTAX_ArgList_Operand_ArgList`(`.List{"_::__MIR-SYNTAX_DefPath_Identifier_DefPath"}_DefPath`(.KList),`.List{"_,__MIR-SYNTAX_ArgList_Operand_ArgList"}_ArgList`(.KList))
2: syntax ArgList ::= ".ArgList" [klabel(.List{"_,__MIR-SYNTAX"}), userList(*)]
    `.List{"_,__MIR-SYNTAX_ArgList_Operand_ArgList"}_ArgList`(.KList)
        Source(/tmp/tmpawfk2g_r)
        Location(3,28,3,29)
        3 |             _1 = test_sum_to_n() -> bb1;

so the first option was actually a one-element list made of an empty DefPath list.
If I force DefPaths to be non-empty (see the syntax-demo-1 branch, also needs a few similar changes), the semantics parses the file above.

Which file is that?
The source in your error message points to a temp file. Probably generated or copied by your build system.

By "The file above" I meant the file in the initial comment (test-sum-to-n.mir).

The production of syntax ... ::= "()" is a problem. Jost pointed out that it violates longest match principle

The production of syntax ... ::= "()" is a problem. Jost pointed out that it violates longest match principle

Yes, that's what I meant above by "having "()" as a token is questionable". It was preventing a few other errors from showing up, though. Anyway, in the syntax-demo-1 branch I removed that token.


Upgrade version of package in npm

Please upgrade version of package in npm.
The changes from https://github.com/skortchmark9/reselect-tools/pull/18 is not in last version.

Ah yes, I will put this on the 0.0.8 milestone. Thanks for the reminder.


Rewritten S3 Sync, plus Rackspace Sync

Probably still room for some cleaning up, but here's something that I'm pretty happy with. S3 Sync now is managed internally with Fog, and you can introduce Parallel to speed things up if you like. The code isn't super complicated.
And given things were almost identical for Rackspace, I added that into the mix as well.
I've just started using this in production, and it seems to be fine, but you should definitely run it through some tests of your own before merging it in.
Let me know what you think :)

Thanks Pat! I will have a look!

Looking good Pat!
Just to confirm:
From what I can see is that if you add 2 directories like so:
s3.directories do |directory|
  directory.add "/path/to/directory/to/sync"
  directory.add "/path/to/other/directory/to/sync"
end

This would invoke Parallel.each(){} twice right? So basically it will build an array of all_files for each of the above directories and iterate through them. If so I'm cool with that. That still means that even if the first directory contains, say, 300  directories and 5000 files, all of them will be processed in parallel am I right? Then once all that's finished, it will move on to the second directory in @directories and do the same.
From what I can see/read that's what it does, but just to confirm.

Next:
def local_hashes
  `find #{directory} -print0 | xargs -0 openssl md5 2> /dev/null`
end

I work on OSX in general, do you know whether this yields the same results on a Linux machine?

I assume since you mentioned in your blog post that you've used this with Rackspace?

Did you try to add Ninefold (since we're using Fog anyway)? If not then I might go ahead and add it for whoever wants it since the subclass is quite simple to add and shouldn't take long at all.

I might go ahead and rename a few things here and there such as bucket_object - since a bucket is specific to S3. I believe Rackspace refers to such things as "containers'. But using that would make it "Rackspace" specific. So instead I think I might rename it to "directory_object" since that's what Fog calls it and it's a generic term. What do you think?
The Syncers were pretty much the only major thing we needed to fix. Since we did a major overhaul of the internals (10.000~20.000 lines of code I believe) in the past weeks in every section of the gem, we haven't yet had time to think about the Syncers, but your pull request pretty much takes care of that task for us and turned it from a quick nasty (half-broken, unsupported) patch to something that was thought out and implemented with the right tools without the need of external dependencies. So again, thanks a lot for this! :)
Cheers!

In order: yes, your understanding of calling Parallel.each per directory is correct. The shell call works the same for me on my Mac and on the Ubuntu machines I've deployed it to. Am using it with both S3 and Rackspace.
I did look at Ninefold, but there's no simple way to get the MD5 hash of their files, so it's not nearly so easy (if possible at all). And sure, rename bucket_object - maybe repository? directory_object is a bit ambigious (we already have a list of directories).
:)

Gotcha. repository might be less confusing yeah.
So I tried it out and it seems that only the last directory in directory.add gets synced. Well, all of them sync, but the one overwrites the other.
s3.directories do |directory|
  directory.add "/Users/Michael/Desktop/backup.wiki"
  directory.add "/Users/Michael/Desktop/backup.old"
end

Both are being synced, but backup.old is the only one that remains, because it overwrites backup.wiki. Thus, when you re-sync, backup.wiki overwrites backup.old and then backup.old overwrites backup.wiki again.

This is of course only when mirror is true. When I disable it, both backup.wiki and backup.old reside in the same path on S3.

Yeah, I'm seeing the same problem - trying to get to the bottom of it now.

I'm looking at it as well. I believe something goes wrong in:
def remote_files
   ...
end

Try adding this to the end of the sync! method:
puts "Directory: #{directory}"
  remote_files.each do |k,v|
  puts k
end

It seems that it actually "reverses" the actual files it should be finding. I also noticed that sometimes (depending on whether the directories you add end with trailing slashes) one or more of your regular expressions will receive: some/path/directory//etc (get double-slashed) - might want to use File.join() when doing any form of path concatenation to be sure that never happens.
I'll keep investigating this as well and report back here

From what I can tell, fog's files object wasn't working with Enumerable as expected - and so the select call to filter down remote files to just the relevant ones for a given directory was not filtering. A to_a call (and a path fix) makes things happy. I should be using File.join more though, you're right.

Ah! That's quite the "gotcha". I was wondering what I was doing wrong trying to change the filtering to something else and always yielded the same result. I'll pull this down either later tonight or tomorrow and re-test it with the changes in place.
I'll let you know! Thanks!

@freelancing-god finished reorganizing and testing it. All works well now. I did make some name/dsl changes to make the DSL/API more consistent with the storages. (for example Rackspace was renamed to CloudFiles, just like the CloudFiles storage)
Here's the full list of changes since your commits: 5ed083910552607796daf7ba30ad4538659c23f5
Thanks again!

Great to know it's all merged in - also, @benschwarz just sent me a patch for some logging, perhaps it's worth you adding this in too?
https://github.com/freelancing-god/backup/pull/1/files

Is the parallel stuff ready to roll as well? I couldn't get it working…

@freelancing-god will take a look.
@benschwarz haven't released a gem yet. and i'm probably renaming something in the latest commit. If you pull from HEAD and build a gem from that, or simply symlink the ./bin/backup utility from the cloned github repository to your /usr/local/bin /usr/bin or w/e bin folder you have. Or wait until I release an official gem which shouldn't be long from now. There is one issue I would like to address before releasing the next gem, and that is this ticket.

@freelancing-god ended up renaming concurrency_using to concurrency_type 2fd76a650a076b539a928156450f9b52cbf52a11

Renamed the concurrency_using method to concurency_type.

@freelancing-god / @benschwarz improved logging for Syncers e9f9980ee944773f9291a07ee0728cf1cbc16512

Improved STDOUT/Logging for the Syncers. [transferring] / [removing] / [skipping] will be prefixed for each file in the list of files that are to be synced/mirrored. Wrapped a Mutex around the Logger operations to ensure STDOUT and Logs are written synchronously, and are always properly formatted/multi-lined.

Nearly ready for a new gem release which will contain all of this. Should happen later today.

The new gem has been released. gem install backup (version 3.0.22)

Thanks Michael - I'll give it a spin soon.

I gave it a run last night. Looking really good.
Well done guys!

Hi, are there any gotchas with this rewrite? I just updated Backup, but didn't change my config. My 'synced' folder is now completely uploaded every time i run the backup task. I even renamed my old folder on s3 to give the new mechanism a clean slate, but it still always uploads all the files, even though they are already there.
Any help would be appreciated :-)

I can confirm that I've been witnessing the same result.
Haven't had time to look into it at all yet. I was actually using backup for one-shot transfers.
On 02/03/2012, at 12:29 AM, Fabian<EMAIL_ADDRESS>wrote:

Hi, are there any gotchas with this rewrite? I just updated Backup, but didn't change my config. My 'synced' folder is now completely uploaded every time i run the backup task. I even renamed my old folder on s3 to give the new mechanism a clean slate, but it still always uploads all the files, even though they are already there.
Any help would be appreciated :-)

Reply to this email directly or view it on GitHub:
https://github.com/meskyanichi/backup/pull/267#issuecomment-4257936

I'm syncing files daily, and - at least when I run it manually occasionally, as opposed to by cron - I see some files being skipped. Fabian, Ben, are you seeing this behaviour when running it from the command line yourself, or through logs after the fact?

I've been running the task by hand from the command line, in fact to check if the syncing works alright because the feature has been rewritten. I had a little time yesterday and investigated, I opened issue #290 which contains some more explanation and a (temporary) fix. It seems to be an issue with fog.

@burns I tried your branch out, but couldn't get it to work. However, I don't think it is related to your changes. All kinds of commands start failing:
[2012/03/06 09:06:46][message] Using Compressor::Bzip2 for compression.
[2012/03/06 09:06:46][warning]   Pipeline STDERR Messages:
[2012/03/06 09:06:46][warning]   (Note: may be interleaved if multiple commands returned error messages)
[2012/03/06 09:06:46][warning]
[2012/03/06 09:06:46][warning]   sh: Syntax error: Bad fd number

The same happens for OpenSSL. The exact same backup config works fine with the latest stable version.

@fschwahn What OS are you running, and how exactly were you initiating the job?

I'm running Ubuntu 11.04, and I initiated the command through a bundler generated binstub. Note that I'm using rbenv, so the shebang looks like this:
#!/usr/bin/env ruby-local-exec

@fschwahn Ok, this should be resolved now.
The issue was that Ubuntu's default shell dash didn't like the syntax I was using to close the STDOUT file descriptor for the last command in the new Pipeline. This could have been solved using 3>&1 1<&- instead of 3>&1-, but I decided instead to redirect this to STDERR instead of closing it. It's not really an issue now, since the final command in the pipeline will either be split or a cat > file, but it does make more sense to do this.
So, give it another shot and let me know :)

Cool! But I don't think you have updated the cloud-syncers branch, I see the changes only in the develop branch.

@fschwahn You may have noticed this already, but it's there. This branch was just rebased to 'develop'. I wouldn't force push a rebase to the 'develop' branch, but I don't worry about with these temporary branches :)


VM: Build separate platform dills for different VM configurations

Due to https://github.com/dart-lang/sdk/issues/36306 we want to avoid late evaluation of environment constants in the VM for the platform dill.
Environment constants are used in the platform dill to distinguish between product and release mode in the VM.
One way to achieve this is to build separate, fully evaluated, versions of the platform dill for the different VM configurations, plus an unevaluated one for the AOT compiler.

A bit more detail (under the assumption that we are not doing https://github.com/dart-lang/sdk/issues/36306):
As far as I know, when we generate kernel for the platform, we don't supply an environment to the front end (via the environmentDefines field in CompilerOptions), which means that when we enable the constant evaluator in the front end (by flipping the flag), all environment constants will be left unevaluated. The VM does not understand these unevaluated constants.
Thus, for dill files targeted for direct consumption by the VM, an environment needs to be supplied when building the kernel, so that environment constants are fully evaluated. The environment doesn't have to encompass all names referenced. The ones not present in the provided environment will get default values.
The AOT compiler needs these environment constants unevaluated, so the platform dill supplied to the AOT compiler must be compiled without supplying an environment.
Apparently, the uses of environment constants in the VM platform are these:
sdk/lib/developer/profiler.dart:137:  @pragma("vm:entry-point", !const bool.fromEnvironment("dart.vm.product"))
sdk/lib/developer/timeline.dart:8:    const bool.fromEnvironment("dart.developer.timeline", defaultValue: true);
sdk/lib/vmservice/vmservice.dart:45:final bool useAuthToken = const bool.fromEnvironment('DART_SERVICE_USE_AUTH');
sdk/lib/vmservice/vmservice.dart:711:@pragma("vm:entry-point", !const bool.fromEnvironment("dart.vm.product"))

Possibly, some of these are only relevant for AOT. Those can just be given whatever values for the VM platform. If any of these should be given different values for different VM configurations, then the corresponding environment given to the compiler when generating the platform dills must be different accordingly.

@aam @a-siva is this required for 2.3? If it is, can you please mark as P1?

It seems that (per https://github.com/dart-lang/sdk/blob/master/pkg/vm/bin/kernel_service.dart#L166), all environment variable values specified on the command line vm will be passed to the compiler that it can use for constant evaluation. So the only work here is to make sure that we do set correct environment variables("dart.vm.product") when building "release"(or "product" for flutter) platform dill.

@askeksa-google can you please confirm that !const bool.fromEnvironment("dart.vm.product") is being evaluated by frontend (when running with --enable-experiment=constant-update-2018). I'm still seeing @core::pragma::_("vm:entry-point", !const core::bool::fromEnvironment("dart.vm.product"))    static method _printMetrics() → core::String { in the generated kernel file.

Yes, when the front end is given the constant update flag, it evaluates all constants. It has two modes of operations for this:

If the environmentDefines field in the compiler options contains a map, it uses this map to look up environment constants. Environment variables not found in the map will get default values.
If the environmentDefines field in the compiler options is null, it leaves environment constants (and constants that depend on them) unevaluated, but still moves them to the constant pool.

In both cases, you should not see the const bool.fromEnvironment("dart.vm.product") expression inside the annotation.
It looks like the flag was not properly enabled for the front-end run. How did you enable it?
The EnvironmentMap class in kernel_service.dart that you point to is used in JIT mode to provide the VM runtime environment as the compile-time environment for the front end. I would expect that this code is not involved when building the platform.

It looks like the flag was not properly enabled for the front-end run. How did you enable it?

I added --enable-experiment=constant-update-2018 to dart command line that invoked comple_platform.dart that builds platform.dill:
$FH/engine/src/out/host_debug_unopt/dart-sdk/bin/dart -Ddart.vm.product=true --enable-experiment=constant-update-2018 --packages=$FH/engine/src/third_party/dart/.packages --dfe=$FH/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot $FH/engine/src/third_party/dart/pkg/front_end/tool/_fasta/compile_platform.dart dart:core --single-root-scheme=org-dartlang-sdk --single-root-base=$FH/engine/src/third_party/dart/ org-dartlang-sdk:///sdk/lib/libraries.json vm_outline_strong.dill vm_platform_strong.dill vm_outline_strong.dill


Unable to import module 'wsgi': No module named 'werkzeug'

Suddenly getting this error again on v1.6.1
Rolling back to 1.6.0 fixes the issue

I found this in stackoverflow and it works I downgrade Flask to 1.1.4

Flask, Werkzeug and other pallets projects just had a major update, dropping python2 support and deleting _compat module. And AWS has't resolve the capability issue yet.
The simplest fix will be downgrading Flask, Werkzeug, etc. to the previous major version.

Credits to Allan Chain

my solution:
https://github.com/serverless/serverless-python-requirements/issues/469#issuecomment-1181742036


RadzenDataGridColumn.SetFilterValue causes InvalidOperationException when called from RadzenDataGridColumn.ClearFilters.

Describe the bug
Version 4.30.3 crashes with an InvalidOperationException "Unable to set property 'Data' on object of type 'Radzen.Blazor.RadzenDataGrid". This is caused when the FilterType is an (not nullable) enum as ClearFilters calls SetFilterValue(null) and SetFilterValue tries to use cast null to int at line 1034.
To Reproduce
Create a Razor component that uses RadzenDataGrid with a non-nullable column of an enumerated type.
Rendering the component throws an InvalidOperationException caused by a null reference in SetFilterValue.
Expected behavior
The component should render successfully (which works in version 4.30.2).
Desktop (please complete the following information):

OS: Windows
Browser: Chrome, but is not browser dependent
Version: 4.30.3

Additional context
Version 4.30.3 contains a fix to initialize the filter value for an enumerated type with the following three lines of code:
if (PropertyAccess.IsEnum(FilterPropertyType) || (PropertyAccess.IsNullableEnum(FilterPropertyType) && value != null))
{
    value = (int)value;
}

However SetFilterValue is called by ClearFilters passing a value of null. The first test yields true because FilterPropertyType is an enum which results in an attempt to cast value (null) to int.

Duplicate of #1493.


Error in rendering HTML

Description
failed: tabularray
failed: tabularray
failed: apacite
How could I fix this Issue; need Help ASAP.
(Optional:) Please add any files, screenshots, or other information here.

(Required) What is this issue most closely related to? Select one.
References
Internal issue ID
08e751fb-85f3-4582-bf7a-0db8b5efc6f0
Paper URL
https://services.arxiv.org/html/submission/5600974/view
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

Still getting this error:
failed: tabularray
is this package not supported by HTML yet?

I am getting this version in HTML.


to close #55 fix androidX compatibility

flutter 1.2.1 stable version with androidX compatibility. update annotation and exifinterface api.

Thanks contributing!
LGTM


Add colours for Awk and regular expression source

This PR adds a language colour for Awk, taken from the covers of "Effective Awk Programming" and "sed & awk":

Regular expression colour taken from the default colour palette of many vintage code editors (which incidentally happens to be the one I'm most used  to seeing). It's as suitable a colour as any. 😜
(Template removed because it doesn't apply).

@lildude Got room for one more?

Yup.


DefinitionUri doesnt support Intrinsic Functions

I have an external Swagger file (not in S3), and trying to do something like
DefinitionUri: !Ref SwaggerFile
or !Join or !Sub results in
Transform AWS::Serverless-2016-10-31 failed with: Invalid Serverless Application Specification document. Number of errors found: 1. Resource with id [MyAPI] is invalid. 'DefinitionUri' requires Bucket and Key properties to be specified

Whereas DefinitionUri: swagger_development.yml works fine.
https://github.com/awslabs/serverless-application-model/blob/master/HOWTO.md#using-intrinsic-functions seems to claim this should work. This makes working with a SAM template with different configs for development and testing difficult.

That error is suggesting that the DefinitionUri is not matching an expected regex pattern. It need a bucket and key, which are parts of an S3 Uri.
I think this has to be an S3 bucket Uri, which is why I am surprised that swagger_development.yml works.
If you want to parametrise DefinitionUri, try building it up from parameters using the !Sub function...
DefinitionUri !Sub s3://${ArtifactsBucket}/swagger_development.yml

I am getting the same error message, but without trying intrinsic functions, just calling the api file. I have done a lot of testing and it seems this part of the documentation is wrong. The example shown here does not work: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#example-awsserverlessapi
A similar example is shown in the Lambda developer guide: http://docs.aws.amazon.com/lambda/latest/dg/deploying-lambda-apps.html
AWSTemplateFormatVersion: '2010-09-09' Transform: AWS::Serverless-2016-10-31 Resources:   Api:      Type: AWS::Serverless::Api     Properties:       StageName: prod       DefinitionUri: swagger.yml
Has anyone been able to get this to work? Or can it only call an S3 bucket? Interestingly, in CodeStar it references a local file as well, and it works fine.

Found the answer here: https://github.com/awslabs/serverless-application-model/pull/86

If you use the AWS CLI command aws cloudformation package it will do the uploading job for you. But if you are passing a SAM template directly to CloudFormation, DefinitionUri has to be a S3 URI. Technically, the spec accepts only S3 URI for this property.


Update available_notebooks.md

A couple of spelling mistakes were fixed.

2 small spelling mistakes were fixed.


Unable to decode audio data with .ogg

This is the error i get in the console while trying to load an ogg/opus audio file.
undefined:1 Uncaught (in promise) DOMException: Unable to decode audio data
The default Chrome Audio element play the file fine.

It's a bug in Chrome: https://bugs.chromium.org/p/chromium/issues/detail?id=409402

Is there a workaround for this issue?


gradle build with 0.0.3 but I get many dependencies error, how can i resolve it?

When I execute "gradle build" on source root directory, I get below error. How can i resolve it? many thanks.
What went wrong:
Execution failed for task ':spring-security-oauth2-authorization-server:compileJava'.

Could not resolve all files for configuration ':spring-security-oauth2-authorization-server:compileClasspath'.
Did not resolve 'org.springframework:spring-jcl:5.3.0' which has been forced / substituted to a different vers
ion: '5.3.3'
Did not resolve 'org.springframework:spring-aop:5.3.0' which has been forced / substituted to a different vers
ion: '5.3.3'
Did not resolve 'org.springframework.security:spring-security-oauth2-core:5.4.1' which has been forced / subst
ituted to a different version: '5.4.2'
Did not resolve 'org.springframework.security:spring-security-config:5.4.1' which has been forced / substitute
d to a different version: '5.4.2'
Did not resolve 'com.fasterxml.jackson.core:jackson-databind:2.12.0-rc1' which has been forced / substituted t
o a different version: '2.12.1'
Did not resolve 'org.springframework.security:spring-security-web:5.4.1' which has been forced / substituted t
o a different version: '5.4.2'
Did not resolve 'org.springframework:spring-expression:5.3.0' which has been forced / substituted to a differe
nt version: '5.3.3'
Did not resolve 'com.fasterxml.jackson:jackson-bom:2.12.0-rc1' which is part of the dependency lock state
Did not resolve 'com.fasterxml.jackson.core:jackson-core:2.12.0-rc1' which has been forced / substituted to a
different version: '2.12.1'
Did not resolve 'org.springframework:spring-core:5.3.0' which has been forced / substituted to a different ver
sion: '5.3.3'
Did not resolve 'org.springframework.security:spring-security-oauth2-jose:5.4.1' which has been forced / subst
ituted to a different version: '5.4.2'
Did not resolve 'org.springframework:spring-web:5.3.0' which has been forced / substituted to a different vers
ion: '5.3.3'
Did not resolve 'com.fasterxml.jackson.core:jackson-annotations:2.12.0-rc1' which has been forced / substitute
d to a different version: '2.12.1'
Did not resolve 'com.nimbusds:nimbus-jose-jwt:9.1.2' which has been forced / substituted to a different versio
n: '9.4.1'
Did not resolve 'org.springframework:spring-context:5.3.0' which has been forced / substituted to a different
version: '5.3.3'
Did not resolve 'org.springframework.security:spring-security-core:5.4.1' which has been forced / substituted
to a different version: '5.4.2'
Did not resolve 'org.springframework:spring-beans:5.3.0' which has been forced / substituted to a different ve
rsion: '5.3.3'
Could not resolve com.fasterxml.jackson:jackson-bom:2.12.0-rc1.
Required by:
project :spring-security-oauth2-authorization-server
Could not resolve com.fasterxml.jackson:jackson-bom:2.12.0-rc1.
Could not get resource 'https://repo.spring.io/libs-release/com/fasterxml/jackson/jackson-bom/2.12.0-rc1
/jackson-bom-2.12.0-rc1.pom'.
Could not HEAD 'https://repo.spring.io/libs-release/com/fasterxml/jackson/jackson-bom/2.12.0-rc1/jack
son-bom-2.12.0-rc1.pom'. Received status code 401 from server: Unauthorized
Could not resolve com.fasterxml.jackson:jackson-bom:2.12.0-rc1.
Required by:
project :spring-security-oauth2-authorization-server > com.fasterxml.jackson.core:jackson-databind:2.12.1
project :spring-security-oauth2-authorization-server > com.fasterxml.jackson.core:jackson-annotations:2.12
.1
project :spring-security-oauth2-authorization-server > com.fasterxml.jackson.core:jackson-core:2.12.1
Could not resolve com.fasterxml.jackson:jackson-bom:2.12.0-rc1.
Could not get resource 'https://repo.spring.io/libs-release/com/fasterxml/jackson/jackson-bom/2.12.0-rc1
/jackson-bom-2.12.0-rc1.pom'.
Could not HEAD 'https://repo.spring.io/libs-release/com/fasterxml/jackson/jackson-bom/2.12.0-rc1/jack
son-bom-2.12.0-rc1.pom'. Received status code 401 from server: Unauthorized

@BestLalaVm We are looking into this now. It's related to the Permissions Changes to repo.spring.io

@BestLalaVm We are looking into this now. It's related to the Permissions Changes to repo.spring.io

Related #191

Related #191

@BestLalaVm As a result of the changes made to the Spring Artifactory instance (repo.spring.io), the builds will not work for release tags 0.0.3 or previous.
Either way, please use the latest release 0.1.0 or master as there have been many changes to the codebase.


Command bus + query bus

Codecov Report

Merging #2 into master will increase coverage by 6.73%.
The diff coverage is 81.73%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage    80.3%   87.03%   +6.73%
==========================================
  Files           8       12       +4
  Lines          66      162      +96
  Branches        0        7       +7
==========================================
+ Hits           53      141      +88
- Misses         13       21       +8

Impacted Files
Coverage Δ

application/command_handlers.py
100% <ø> (+100%)
:arrow_up:

domain/entities.py
100% <100%> (+100%)
:arrow_up:

application/query_handlers.py
100% <100%> (ø)

application/response.py
100% <100%> (ø)

domain/value_objects.py
100% <100%> (ø)
:arrow_up:

application/command_bus.py
100% <100%> (+100%)
:arrow_up:

...structure/repositories/auction_items_repository.py
100% <100%> (ø)

application/query_bus.py
100% <100%> (ø)

application/commands.py
100% <100%> (ø)
:arrow_up:

infrastructure/framework/falcon/controllers.py
38.7% <38.7%> (ø)

... and 12 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 d4c5e8f...491efd4. Read the comment docs.


Disable parse-time name lookup in swift-ide-test

I have some leftover patches from my work on ASTScope. A couple of places that create an ASTContext without a CompilerInvocation still had parser lookup enabled; swift-ide-test is one of them.

@swift-ci Please smoke test

@swift-ci Please smoke test

Thanks for the review, @rintaro!

@swift-ci Please smoke test


Modal composite

Quicker way to create a modal

Codecov Report

Merging #468 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #468      +/-   ##
==========================================
+ Coverage   99.75%   99.76%   +<.01%
==========================================
  Files         211      213       +2
  Lines        4999     5053      +54
  Branches     1163     1178      +15
==========================================
+ Hits         4987     5041      +54
  Misses         12       12

Impacted Files
Coverage Δ

src/components/modal/Modal.tsx
100% <ø> (ø)
:arrow_up:

src/components/modal/ModalComposite.tsx
100% <100%> (ø)

src/components/modal/ModalCompositeConnected.tsx
100% <100%> (ø)

src/Index.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Demo: https://coveo.github.io/react-vapor/modal-composite/


updating changlog, adding tbd for 1.1.0, updating 1.0.0 release date …

…moving #93
Description
updating changlog, adding tbd for 1.1.0, updating 1.0.0 release date moving #93
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt-bigquery next" section.

created this PR to resolve @leahwicz suggestion here https://github.com/dbt-labs/dbt-bigquery/pull/93#discussion_r773954785


Separate test environment and Continuous Integration with TravisCI

Travis CI used to incorporate continuous integration environment. And stopping the container gracefully is added in stop.sh. Docker socket is used for sending signals via docker-host-docker.
Note: Enable Continuous integration in your repository and change the readme build-path link before marging

@droidlabour have a look


style: format code with black and isort

Formatted the code with black and isort.

Why are the spaces between “=“ removed?

where?


pip install is not working

🐛 Bug
Describe the bug
icedata cannot be pip installed.
Solution
Add setup.py and settings.ini

This is expected since we're not using setuptools.
To install the package in editable mode use poetry install, or install from pypi with pip install icedata


Aliasing columns in a query results in the query's result missing data

[x] Check existing issues for a duplicate of this bug

Summary
This query returns the expected data,
SELECT carat as a, color as b FROM default.diamonds LIMIT 2;

-- Result
┌─────────┬────────┬─────┐
│ (index) │   a    │  b  │
├─────────┼────────┼─────┤
│    0    │ '0.23' │ 'E' │
│    1    │ '0.21' │ 'E' │
└─────────┴────────┴─────┘

Whereas this query returns results missing data,
SELECT carat as a, color as a FROM default.diamonds LIMIT 2;

-- Result
┌─────────┬─────┐
│ (index) │  a  │
├─────────┼─────┤
│    0    │ 'E' │
│    1    │ 'E' │
└─────────┴─────┘

Is it possible to handle this scenario properly so we get the right data for such queries?
Reproduction
You'll find a minimal and complete reproduction example here that you can run yourself https://github.com/varun-dc/databricks-nodejs-duplicate-column-select-bug-reproduction

Hi @varun-dc! Yes, we know that such behavior exists, as well as in other connectors (which also don't have any special handling for duplicated columns, so their behavior slightly differs). We're trying to find a good solution for this issue which will work across all connectors, but meanwhile can only suggest you to avoid diplucated column names in your queries. Sorry for the inconvenience
P.S. I'll definitely keep this issue open so we can continue discussion here and post updates. Also, if you have any ideas - feel free to share. Thank you!

I've seen this problem before in drivers for other databases and the common solution is the implementation of a rowMode, for example like in node's pg - https://node-postgres.com/features/queries#row-mode.

Internal ticket: [PECO-970] Probably will start working on this very soon


Validate query from frontend against json schema

Describe your problem.
Currently, there is no validation of the structured query sent by the frontend to the backend, leading to possibly malformed (or in the worst case fraudulent) queries being transferred.
Describe the solution you'd like
Check queries sent on the "run-query" endpoint against the sq v2 query schema.
If validation fails, return 422 or 400.
Also add a switch to turn validation on or off.

Please reconsider not adding the validation as being optional. The validation itself is a key part in order to ensure correctness. Any accusations of this being a bottleneck are unfounded until proven otherwise. Even if it is, I would still value correctness over performance.
/cc @juliangruendner @thkoehler11

I was explicitly told to add it by @juliangruendner - so I would pass the final decision on this over to him. However, I do agree on the prioritization of correctness fwiw.

After consulting Julian, the decision is to have the option to turn it off, but by default it is activated.

I'd still be interested in the reasoning behind this decision. There seems to be no benefit at all.


Simulating fewer reads than desired

Hi there,
I have run into an issue where ReSeq simulates fewer reads than the desired number, input with --numReads.
With following command:
reseq illuminaPE --statsIn NA12878.sorted.bam.reseq --ipfIterations 0 -j 1 --refBias file --refBiasFile refbias_test.txt -r num_reads_test.fa --numReads 100000 -1 reads_R1.fastq.gz -2 reads_R2.fastq.gz > stdout 2> stderr
Returns about 26000 reads, and is reported in the stderr like so:
"
...
21-12-20 14:11:26: info:  Read in 26 biases from 26 biases in file refbias_test.txt
21-12-20 14:11:26: info:  Aiming for 100000 read pairs, which corresponds to an approximated read depth of 911.9x
21-12-20 14:11:26: info:  Preparing for simulation
21-12-20 14:11:27: info:  Starting read generation
21-12-20 14:11:30: info:  Generated 25900 read pairs (26%).
21-12-20 14:11:30: info:  Simulation finished succesfully
"
I am wondering if you know why I am seeing this behaviour, and if it is possible to always produce the desired number of reads?
Files used in the above command you can access here: https://www.dropbox.com/s/arhgisfl0nx25j2/example.zip?dl=0
Thank you very much,
Cillian

Thank you for the awesome reporting. I could immediately reproduce the issue and it is most certainly a bug. Small fluctuations are normal and cannot be prevented, because reads are drawn independently. However, this is definitely not the issue here. I am working on finding the root cause, but most likely will not finish before the break. I will continue with it the first thing next year.
Thanks,
Stephan

This issue highlighted multiple bugs. I pushed the fixes to the debug branch. Thanks again for this report.
As I said before the number of read pairs is still not exact due to the random nature of the drawing process.
Running it 5 times I get:
Generated 99943 read pairs (100%).
Generated 100096 read pairs (100%).
Generated 99641 read pairs (100%).
Generated 100213 read pairs (100%).
Generated 100263 read pairs (100%).

This can not be prevented, but the more reads you simulate the lower is the fluctuation relative to the amount. If you need an exact amount I suggest simulating slightly more and randomly discarding the excess reads.

This issue highlighted multiple bugs. I pushed the fixes to the debug branch. Thanks again for this report.
As I said before the number of read pairs is still not exact due to the random nature of the drawing process.
Running it 5 times I get:
Generated 99943 read pairs (100%).
Generated 100096 read pairs (100%).
Generated 99641 read pairs (100%).
Generated 100213 read pairs (100%).
Generated 100263 read pairs (100%).

This can not be prevented, but the more reads you simulate the lower is the fluctuation relative to the amount. If you need an exact amount I suggest simulating slightly more and randomly discarding the excess reads.

Thank you very much for the quick fix and all your help!

Thank you very much for the quick fix and all your help!


Fix shadowing in client.IsEnabled

Fixed shadowing that were preventing client from correctly reporting
what API version is enabled/disabled.
Added unit test to verify disabled state.

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Opening balloon just by clicking in editor

Hi!
Is there a possibility to trigger the opening toolbar in balloon mode just by clicking on the editor? Some kind of fusion between balloon and inline mode. My use case is that I want to give a possibility for a user to insert a table without the need to write anything, but still keeping the balloon mode (toolbar near the caret).
Cheers!

Hi
It does mean that the toolbar can be opened by just clicking on the text in "balloon" mode?

Hi,
Yes, something like that. So, exactly the same behavior as in inline mode, but positioning of toolbar like it is in balloon mode.

Maybe it could show up after clicking the right mouse button?

@natanielcz It's possible but it doesn't seem intuitive.

The 2.26.0 version has been updated.
If there is an issue, please register as a new issue.
Thank you.


"Index was outside the bounds of the array."

on line 71 in public class VArray I get the "Index was outside the bounds of the array." whilst comparing two files:
TMP.ZIP
thanks in advance for fixing!

Code to reproduce:
var a = File.ReadAllBytes(@"D54ACE32535F00233F43B243D0EF1EDE.mus");
var b = File.ReadAllBytes(@"Smooth_Operator.mus");
var d = new MyersDiff<int>(a, b);

@TwanVanDongen Thank you for reporting this, it was a serious issue.


Failed tests not failing build

when tests fail appveyor build doesn't fail

build script changes here https://github.com/alhardy/AppMetrics/commit/00958f9dce06474fdd893fb1c125a63f9262cf06


Disable by default on all channels. Fixes #3 .

Why is explained in issue #3 .

was merged to fork


Problem importing

when I hit paste then import it doesn't give me a success message and doesn't put the macro in the list for my toon.  Also for macros I DO have when I click on create icon THAT doesn't work either.

Hi Fredric
What is different between your install and mine?  I really need a lot more information before I can trouble shoot this.  To start with what version of GSE are you using, which OS, and are you getting any error messages?  If you disable all other mods does it then work?  How many character macros does your character have in the WoW /macros interface?
TimothyLuke

On 12 Aug 2017, at 2:35 am, Fredric<EMAIL_ADDRESS>wrote:
when I hit paste then import it doesn't give me a success message and doesn't put the macro in the list for my toon. Also for macros I DO have when I click on create icon THAT doesn't work either.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Mr.   Okay so I went into WoW and copied and pasted a macro straight from a blog as opposed to a .LUA file whose text I just copied and pasted. It worked.   For the icons I discovered that if I create a NEW macro in WoW with nothing in the body and just a question mark for an icon and when I click save, Bam, I get a ? in GS. Then I can use it. (Oh, I also deleted the GS-MyMacro folder, not sure how that wasn't deleted before)
I am going to chalk this up to User Error(or Selficus Stupidious)..
Just out of curiosity, what language is this written in?
Anyway, just for info purposes, here is my setup:
MacBook Pro 13" (yes, yes, I know, WoW on a MAC??)
OS: MacOS Sierra 10.12.6
MEM: 16 gig
VER: 2.1.04 GSE: Advanced Macros
Oh and no error messages..

Hi Fredric
I hate problems as well.  I use WoW on both a mac and windows and found weird problems where things like pasting from IE in Windows was different than chrome or safari on a mac.  I ask the dumb questions as when something like this occurs it usually is something quirky that I haven't thought to test on.
There's for example a few people running on Linux with Wine and that can really do your head in.
Could you do me a favour - in your WTF folder there is a GSE.lua file.  If you could upload it I can do a check to see if there is anything in there that might cause a problem.
TimothyLuke

On 12 Aug 2017, at 12:03 pm, Fredric<EMAIL_ADDRESS>wrote:
Mr. Minahan, I hate problems, lol.. Okay so I went into WoW and copied and pasted a macro straight from a blog as opposed to a .LUA file whose text I just copied and pasted. It worked. For the icons I discovered that if I create a NEW macro in WoW with nothing in the body and just a question mark for an icon and when I click save, Bam, I get a ? in GS. Then I can use it. (Oh, I also deleted the GS-MyMacro folder, not sure how that wasn't deleted before)
I am going to chalk this up to User Error(or Selficus Stupidious)..
Just out of curiosity, what language is this written in?
Anyway, just for info purposes, here is my setup:
MacBook Pro 13" (yes, yes, I know, WoW on a MAC??)
OS: MacOS Sierra 10.12.6
MEM: 16 gig
VER: 2.1.04 GSE: Advanced Macros
Oh and no error messages..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

there is no GSE.lua in the WTF folder, but there is one in WTF/Account/"name on account"/Saved variables folder..  Should that be in the WTF root?
F

No thats where it should be.  - Thats the file needed.
On Sat, Aug 12, 2017 at 1:27 PM, Fredric<EMAIL_ADDRESS>wrote:

there is no GSE.lua in the WTF folder, but there is one in
WTF/Account/"name on account"/Saved variables folder.. Should that be in
the WTF root?
F
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/337#issuecomment-321955776,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjBFTpANMfFRrhGfAbi-4OWDlc_qZfFks5sXSK_gaJpZM4O056I
.

--
Timothy Minahan**M 0406 413 680  |  *E<EMAIL_ADDRESS>

Okay, cool.  Just tried importing a macro from forums in WLM and it said import successful, but there is no macro in the list.

has it since appeared?

Sorry for delay, yes, there were lines in the macro before the actual macro that I believe was preventing it from actually being imported even tho add-on said it was.

Wait a sec -
local Sequences = GSMasterSequences -- Dont remove this
Thats from GSE1 - If you leave that it yes it will report success as the parsing is successful but it wont return the macro.
The macros you are trying to import were for WOW 7.0.3.  7.1.5 introduced a lot of changes and those macros needed to be updated as they will not perform for you with how the classes have changed between 7.1 and now.  They may work but you will be missing out on about 200-400k dps.


Camel Avro RPC component native support

This one is for camel-avro-rpc component. We already have the avro dataformat in native https://github.com/apache/camel-quarkus/issues/782

Please, assign to me.


Shrine(2.7.0) is destroying my source file, when using it with rspec

I've got a very strange issue :
During my test, Shrine is destroying the source file of my uploads ! The issue appear in 2.7.0 version, but I don't have problem with 2.5.0...
I'm using rails 5.0.6.
Here is my Factory :
FactoryGirl.define do
  factory :hunt do
    title               Faker::Lorem.word
    user                { FactoryGirl.create(:user) }
    description         Faker::Lorem.sentence
    follower            true
    number_of_follower  { rand(1..5) }
    comment             Faker::Lorem.sentence
    started_at          Time.now + 10.days
    finished_at         Time.now + 15.days
    acceptance_deadline Time.now + 9.days
    price               { rand(1..100) }
    exchange            true
    address             'test address'
    picture             { File.open("spec/fixtures/files/picture.jpg") }
    type                { FactoryGirl.create(:hunt_type) }
  end
end

My shrine initializer :
require "shrine"
require "shrine/storage/fog"
require "fog"

if Rails.env == 'test'
  require "shrine/storage/memory"

  Shrine.storages = {
    cache: Shrine::Storage::Memory.new,
    store: Shrine::Storage::Memory.new,
  }
else
  ovh_service = Fog::Storage.new(
    provider:             ENV['PROVIDER'],
    openstack_username:   ENV['OPEN_STAK_USERNAME'],
    openstack_api_key:    ENV['OPEN_STACK_API_KEY'],
    openstack_auth_url:   ENV['OPEN_STACK_AUTH_URL'],
    openstack_region:     ENV['OPEN_STACK_REGION'],
    openstack_temp_url_key: SecureRandom.urlsafe_base64(nil, false),
  )

  Shrine.storages[:store] = Shrine::Storage::Fog.new(
    connection: ovh_service,
    directory: ENV['OPEN_STACK_FOLDER_NAME'],
    expires: 24,
    type: 'public'
  )

  Shrine.storages[:cache] = Shrine::Storage::Fog.new(
    connection: ovh_service,
    directory: ENV['OPEN_STACK_FOLDER_NAME'],
    type: 'public'
  )
end

My uploader :
require "image_processing/mini_magick"

class ImageUploader < Shrine
  # plugins and uploading logic
  include ImageProcessing::MiniMagick
  plugin :activerecord
  plugin :cached_attachment_data # for forms
  plugin :upload_endpoint
  plugin :processing
  plugin :versions   # enable Shrine to handle a hash of files
  plugin :delete_raw # delete processed files after uploading
  plugin :validation_helpers
  plugin :determine_mime_type

 Attacher.validate do
   validate_extension_inclusion %w[jpg jpeg png gif]
   validate_mime_type_inclusion %w[image/jpeg image/png image/gif]
 end

  process(:store) do |io, context|
    original = io.download

    size_800 = resize_to_limit!(original, 800, 800) { |cmd| cmd.auto_orient } # orient rotated images
    size_500 = resize_to_limit(size_800, 500, 500)
    size_300 = resize_to_limit(size_500, 300, 300)
    size_150 = resize_to_limit(size_300, 150, 150)
    size_60  = resize_to_limit(size_150, 60, 60)

    { original: io, large: size_800, medium: size_500, card: size_300, little: size_150, small: size_60 }
  end
end

And my route :
mount ImageUploader.upload_endpoint(:cache) => "/images/upload"
I really don't know from where it can be from... Any idea ?
Thanks in advance !

The delete_raw plugin is deleting your test file. The reason why this happens on version 2.7.0 and not before is that the delete_raw plugin was recently fixed to also work on File objects in addition to Tempfile objects.
There are several solutions. One solution is to configure delete_raw plugin to only delete raw files that are uploaded to permanent storage:
plugin :delete_raw, storages: [:store]

Another solution is to provide a StringIO object in your factory definition:
require "stringio"

FactoryGirl.define do
  factory :hunt do
    # ...
    picture             { StringIO.new(File.binread("spec/fixtures/files/picture.jpg")) }
    # ...
  end
end

I don't like how magical the delete_raw and similar plugins are, it would be better specifying deletion was opt-in instead of opt-out, such as passing a :delete_raw option on uploading. I will rethink this in the future, but for now these two options are the idiomatic solutions.

Hello @janko!
I use shrine 3.2.1 and have this problem, but i not use delete_raw plugin.
I have derivatives plugin. What argument pass to plugin :derivatives, that disable delete source file?

Hi @Yegorov , please ask this question on the Discourse forum for questions! https://discourse.shrinerb.com/
I am not sure exactly what you are doing, but you may be running into what's documented here. But please don't discuss this anymore on this Github issue, please use the discourse forum -- or if you are confident it's a bug, please file a separate ticket with a complete description of the problem and a reproducible test case. Thanks for using shrine and providing feedback!


Are there any key bindings for the open file (Ctrl+o)  dialog once it's open?

For the life of me I can't figure out how to go up a directory or change root via the keyboard....

Ctl+k O

Sorry, to be clear, I mean once I'm in it.  I can filter but not do much else...

Look at the default key bindings of vscode

I did try...I looked for anything to do with remoteFileDialog.  Nothing leaps out at me....
It's not like a local vscode running file open dialog so this is something coder specific I wonder?

Oh. I have not found a way to open a specific file except to symlink the file to the current working directory

Ctrl K+O works now so I'll go mark this as closed.


add GlyphSvg

This is a minimal addition of GlyphSvg to ab_glyph. The data is transferred as-is from ttfp without rendering or scaling or interpreting, as a minimally useful product.
I would rather render the SVG into a raster image somehow ti make it easier for callers but I don't want to add a dependency. (And I am not quite sure the work I did for the font I was using would apply to all fonts.)
I am rendering the SVG as needed here: https://github.com/asibahi/noor/blob/01ce89e01d8814763c1ebc1047ccad6f1c1c30fa/src/main.rs#L169 .

Thanks! This looks quite reasonable to me. Nice one for making it non-breaking. I'll find some time soon to properly review soon.


Simplify Script Variables Population

This also fixes weird behavior when the script had a parser/compilation error.
Before:

Thanks!

Removing cherry-pick label as this is master-specific code.


Use number of cpus instead of GOMAXPROCS

GOMAXPROCS could be lesser, than number of cpu.
Then when RLocker() will be called, there will be bound checking panic.

Ah, yes, good catch!


maven-git-versioning-extension extension is not supported

maven-git-versioning-extension
<extension>
    <groupId>me.qoomon</groupId>
    <artifactId>maven-git-versioning-extension</artifactId>
    <version>9.7.0</version>
</extension>

I want to disable maven-git-version-extension with the -Dversioning.disable=true configuration, but when using mvnd,
the -Dversioning.disable=true configuration does not work. But with the mvn command, this configuration works.
Does mvnd not yet support Maven-git-version-extension???

This seems to be a consequence of #1008.


Custom image background

It would be nice and encourage enthusiasm for team members awaiting an even/countdown if we could have the countdown clock imposed on top of a custom background image, also would love to see gradient color options for the background.

😩


SOLR-15484: Re-enable test, with workaround for localhost.localdomwin

https://issues.apache.org/jira/browse/SOLR-15484
Another try to enable this test, which should now work in Jenkins env, by forcing localhost instead of localhost.localdomain.

Why can't we use <IP_ADDRESS> instead of a hostname? The certificate could also be issued for that one.

Why can't we use <IP_ADDRESS> instead of a hostname? The certificate could also be issued for that one.

Sorry ignore this. See my comment in issue. I'd add all possible hostnames to the test certificate.


Extend InputDecoration with prefix/suffix padding

Allows to make some space between the suffix/prefix text and the text entered into the text field.
(fixes #19460 @HansMuller)

I signed it!

This looks reasonable however It might be simpler to just expose prefix/suffix widgets as an alternative to prefixText and suffixText. The input decorator internals (see _RenderDecorator etc) are already set up that way. Supporting widget values is more consistent with Flutter's overall approach and it scales better than adding more properties like this.

I'm afraid this issue has come up frequently; just hasn't been addressed yet. See also:

https://github.com/flutter/flutter/issues/19193#issuecomment-403850883
https://github.com/flutter/flutter/issues/18068#issuecomment-393558524
https://github.com/flutter/flutter/issues/12747#issuecomment-397105268

What about replacing prefixIcon, prefixText, ... with only prefix? Having so many options for specifying the prefix is confusing.

Yep sounds good! Will work on it tomorrow

Ok replaced the padding parameters with prefix and suffix.

We should assert if both attributes, say prefix and prefixText, are specified.

Yep, when both prefix and prefixText is declared an assertion error is thrown. Same applies for suffix and suffixText.

Thanks! I really appreciate your advice. I am struggling a bit with my english but i will try my best in future commits 😅

I am not that familiar with unit testing flutter widgets but I will try it.

Is it ok if i rebase and squash?

Is it ok if i rebase and squash?
Sure

Here we go

Can you fix up the analyzer errors? Const is no longer needed in subexpressions of a const expression.

Yep all checks passes :D


Fix conflicting eslint formatting rules and prettier

Changes for eslint and prettier are very good, thanks for submitting them.
However, changes for tsconfig aren't needed. tsconfig-base.json exists because there are two tsconfig.json files: one in root directory, one in tests subdirectory for tests. That way they share same config file, but use different paths and few slightly different options.

To make it easy on me. I will close this one and start a new pr. I'm also going to disable the rules and start another pr with turning them on and running eslint fix.


Add poet theme

Poet is a nice theme focused on writing prose: https://github.com/kunalb/poet
It would be great to have it available in doom!

Any news on integration of this one ?

Is there any update on the integration of this theme? Is it already compatible with the latest versions of emacs-doom?
Congratulations for the work with emacs-doom.


[BUG] custom icon doesn't work

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
i made sure that i got the icon to the right sizes and when i saved it and relaunched, it was still the regular bloxstrap icon
https://github.com/pizzaboxer/bloxstrap/assets/142828959/1ed18266-35be-4212-901a-32eb54987e71

What did you use to create the icon?

What did you use to create the icon?

i used a convert to ICO website

My bad, I didn't actually fully watch the video. It's actually working largely as intended, as the icon is only supposed to take effect only while launching, and it didn't really make sense to fit custom icons into the Hyperion bootstrapper style as it was only ever originally designed to use the one logo. Hope that clears it up.


New admin method for fetching group offsets for multiple topics

Fixes #989
This PR introduces a new method called fetchConsumerGroupOffsets that mimics the behavior fetchOffsets (which I now deprecated), but adds support for querying multiple topics at once, or omitting the topics parameter altogether and getting the group offsets in all consumed topics.
I chose to re-implement this method instead of changing fetchOffsets because this introduces an API breaking change: the result is now a list of topics and offsets (instead of just the offsets as it was in the original function).

Since fetchOffsets is deprecated, I would suggest removing the documentation for it and leaving only a "See fetchConsumerGroupOffsets" link, linking to the documentation for the new method.

Since fetchOffsets is deprecated, I would suggest removing the documentation for it and leaving only a "See fetchConsumerGroupOffsets" link, linking to the documentation for the new method.

I see two options:

Create a new method, as is done in this PR.
Change the signature of the old method so that it has two optional parameters, topic: string and topics: string[]. Internally, we could normalize the arguments so that topic: "foo" becomes topics: ["foo"], to avoid having to have a bunch of duplicated code. We could eventually deprecate the topic argument in favor of topics. If topic and topics are both defined, we could throw an error.

I think I'm in favor of option 2 more than option 1, because it limits the number of methods on the client and migrating from the old signature to the new is very simple. Do you see any big problem with going with that approach, @nirga?
@tulios / @JaapRood: Any input from you two?

I see two options:

Create a new method, as is done in this PR.
Change the signature of the old method so that it has two optional parameters, topic: string and topics: string[]. Internally, we could normalize the arguments so that topic: "foo" becomes topics: ["foo"], to avoid having to have a bunch of duplicated code. We could eventually deprecate the topic argument in favor of topics. If topic and topics are both defined, we could throw an error.

I think I'm in favor of option 2 more than option 1, because it limits the number of methods on the client and migrating from the old signature to the new is very simple. Do you see any big problem with going with that approach, @nirga?
@tulios / @JaapRood: Any input from you two?

@Nevon I agree with you. I moved the changes back to fetchOffset and added logic go support passing either topic or topics (tweaking the returned value accordingly). I've also updated the docs.

@Nevon I agree with you. I moved the changes back to fetchOffset and added logic go support passing either topic or topics (tweaking the returned value accordingly). I've also updated the docs.

@Nevon Fixed everything. Thanks so much for your review and responsiveness!

@Nevon Fixed everything. Thanks so much for your review and responsiveness!

Great, thanks for your contribution! Sorry you had to wait so long for a review.

Great, thanks for your contribution! Sorry you had to wait so long for a review.

I see two options:

Create a new method, as is done in this PR.
Change the signature of the old method so that it has two optional parameters, topic: string and topics: string[]. Internally, we could normalize the arguments so that topic: "foo" becomes topics: ["foo"], to avoid having to have a bunch of duplicated code. We could eventually deprecate the topic argument in favor of topics. If topic and topics are both defined, we could throw an error.

I think I'm in favor of option 2 more than option 1, because it limits the number of methods on the client and migrating from the old signature to the new is very simple. Do you see any big problem with going with that approach, @nirga?
@tulios / @JaapRood: Any input from you two?

I guess you were too fast @Nevon 😅
Glad that the PR was merged.

I see two options:

Create a new method, as is done in this PR.
Change the signature of the old method so that it has two optional parameters, topic: string and topics: string[]. Internally, we could normalize the arguments so that topic: "foo" becomes topics: ["foo"], to avoid having to have a bunch of duplicated code. We could eventually deprecate the topic argument in favor of topics. If topic and topics are both defined, we could throw an error.

I think I'm in favor of option 2 more than option 1, because it limits the number of methods on the client and migrating from the old signature to the new is very simple. Do you see any big problem with going with that approach, @nirga?
@tulios / @JaapRood: Any input from you two?

I guess you were too fast @Nevon 😅
Glad that the PR was merged.


Query.next_page() should produce a null cursor if not set on response

Currently it returns an empty string.

Noticed this during #1055

OK, checking response.batch.HasField('end_cursor') is always True, even when the cursor is ''.
@tseaver WDYT we should do here? I think returning '' and setting it as the _start_cursor value on an Iterator is not the right move.

@dhermes While investigating, it just seemed easier to make the change (to ensure I didn't break anything).

No worries.

@tseaver Turns out this issue still open.
Remaining concern: Should Connection.run_query check if the end_cursor is set on the response.batch?

@tseaver Bump

I assume you are worried about this line.  If the response protobuf doesn't have a value set, we get a None there, right?  That seems like the right sentinel to me.

See what I said above:

checking response.batch.HasField('end_cursor') is always True, even when the cursor is ''.


added elementary os freya support

Elementary OS Freya runs on Ubuntu 14.04. So ee should also work on Freya.
I have added the necessary condition to accept Freya as a valid distribution and installed ee in a newly installed Freya with the committed changes.

There are some error that i missed. Will update with fix.


header ui update on tablet

Do you want to work on this one like on the other ones before?

Do you want to work on this one like on the other ones before?

Of course

Hi @Arhell sorry for delay, I can test this tomorrow, the one thing I need is, steps to test in the issue.
I can see from your screenshot the resolution, and the screenshots, but can you please detail what is the problem, and how would we know it is fixed?  thanks in advance

I have updated the issue template, but I still need this information for this bug, @Aaron-Junker you said the PR #198 looked good to you, do you think you can fill this in?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Describe the bug
Content does not fit in one line and does not look good (ui)
To Reproduce
Open the developer panel and reduce the screen width to these values which is in the screenshot
Desktop
in this resolution everything is ok
Smartphone
in this resolution everything is ok
Additional context
A bug at a width of 768, that is, a tablet

No stop trying trying to get people info
On Sat, Jul 30, 2022 at 1:27 PM Emma @.***> wrote:

I have updated the issue template, but I still need this information for
this bug, @Aaron-Junker https://github.com/Aaron-Junker you said the PR
#198 https://github.com/microsoft/opensource.microsoft.com/pull/198
looked good to you, do you think you can fill this in?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/197#issuecomment-1200272096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYXI3ISYAUAQ4JL27PJIZQLVWVXYZANCNFSM5S7GNNLA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

No stop trying trying to get people info
On Sat, Jul 30, 2022 at 1:27 PM Emma @.***> wrote:

I have updated the issue template, but I still need this information for
this bug, @Aaron-Junker https://github.com/Aaron-Junker you said the PR
#198 https://github.com/microsoft/opensource.microsoft.com/pull/198
looked good to you, do you think you can fill this in?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/197#issuecomment-1200272096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYXI3ISYAUAQ4JL27PJIZQLVWVXYZANCNFSM5S7GNNLA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @LOGOLFGODORD you always write this, but what exactly do you mean? Can you eloborate?

 So now I’m coming for
redemption !!!
On Sun, Jul 31, 2022 at 11:53 AM Aaron Junker @.***>
wrote:

No stop trying trying to get people info
On Sat, Jul 30, 2022 at 1:27 PM Emma @.***> wrote:
I have updated the issue template, but I still need this information for
this bug, @Aaron-Junker https://github.com/Aaron-Junker
https://github.com/Aaron-Junker you said the PR
#198 https://github.com/microsoft/opensource.microsoft.com/pull/198 #198
https://github.com/microsoft/opensource.microsoft.com/pull/198
looked good to you, do you think you can fill this in?
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
—
Reply to this email directly, view it on GitHub
#197 (comment)
https://github.com/microsoft/opensource.microsoft.com/issues/197#issuecomment-1200272096
,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYXI3ISYAUAQ4JL27PJIZQLVWVXYZANCNFSM5S7GNNLA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>
Hi @LOGOLFGODORD https://github.com/LOGOLFGODORD you always write this,
but what exactly do you mean? Can you eloborate?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/opensource.microsoft.com/issues/197#issuecomment-1200460988,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYXI3IQQ2ALKDLU7CZD3EMDVW2VPVANCNFSM5S7GNNLA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[Help]: facing issue installing.Pygmtsar

Describe the problem you met
Write a brief description of the problem you met, e.g. my intf_tops.csh could not run through
OS and software version
If applicabl

i am facing installing PYGMTSAR have motioned a snapshot, I have done multiple process  pull and run, what is the next step.., what operating system and software version you are using?
OS:
GMTSAR version:
Reproduce the problem
If applicable, how to reproduce the problem from scratch?
Log file
Attach a log file output from your terminal.
Screenshot
If applicable, attach a Screenshot to help us understand your case
[Please note that if no new responses are made over six months from users, the issue will get closed]
[Also if this is a general question that's not particlarly related to GMTSAR, such as "how to correct atmospheric noise" or "how to intepret my interferogram", consider using the discussions page]

Open "Containers" tab in Docker Desktop left vertical menu to found your running containers. Do you see this one or more here?


gpsbabel: fix build for i686

I did a fix earlier but it was insufficient :-( This time I've properly verified the build in a 32-bit VirtualBox.

BTW, you don't need virtualization for this. You can just use the attribute pkgsi686Linux.gpsbabel, for example. (It builds for me now.)

@vcunat, thanks for the hint!


Question: How Azure Resource Manager deployment web server with key vault without keyvault version?

[Enter feedback here]
I  want to deploy a web server app with key vault, but from this doc, the ARM example deployment have the key vault version like below:
(variables('storageConnectionStringResourceId')).secretUriWithVersion
If I want have the version value for the key vault, below is by modify is OK, if OK, where can find the doc about it, I can not find the dos about ).secretUri
(variables('storageConnectionStringResourceId')).secretUri

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

ID: 7feaa3d0-7233-a3cb-cb9b-a9a55b1f0d7a
Version Independent ID: f41f9d51-52f8-1f45-1043-1344a3b74458
Content: Use Key Vault references - Azure App Service
Content Source: articles/app-service/app-service-key-vault-references.md
Service: app-service
GitHub Login: @mattchenderson
Microsoft Alias: mahender

@mxxmusic, If I have understood you correctly- You may simply remove the version from the reference to begin leveraging this capability. No further configuration is required.
Additionally - Key Vault references allow the app to use a system-assigned managed identity to resolve secrets from Azure Key Vault and expose them as environment variables. So, this allows teams to easily move secrets into management without code changes.

@AjayKumar-MSFT , Yes I just remove the key valut version by modify the ARM deployment((variables('storageConnectionStringResourceId')).secretUriWithVersion to (variables('storageConnectionStringResourceId')).secretUri ) by my guess, and it work. But I can not find any docs about the secretUriWithVersion and secretUri. I want to find the official docs about it.

Apologies for the delay! The doc  section highlights about -the difference in usage of the reference string.
"The SecretUri should be the full data-plane URI of a secret in Key Vault, **optionally including a version**,", and "The SecretVersion is optional but if present indicates the version of the secret to use.' - The ARM template example shared in the doc, highlights includes 'secretUriWithVersion' -but yes the similar example without the 'version' string should work.
I shall also share your feedback with the content owner for further review to include ARM template example without the 'WithVersion' in future doc releases. Thanks again for your feedback, we will close this for now.


Remote Rendering Build Errors

Describe the bug
While building with all of the latest MRTK packages I am returned the following errors:
Library\PackageCache\com.microsoft.azure.remote-rendering@5551f252e88c-1615764029433\Runtime\ARRServiceUnity.cs(368,54): error CS0234: The type or namespace name 'Storage' does not exist in the namespace 'Microsoft.Windows' (are you missing an assembly reference?)
Library\PackageCache\com.microsoft.azure.remote-rendering@5551f252e88c-1615764029433\Runtime\ARRServiceUnity.cs(369,43): error CS0234: The type or namespace name 'Launcher' does not exist in the namespace 'Microsoft.Windows.System' (are you missing an assembly reference?)
To reproduce
Install all of the latest MRTK packages using the MR feature tool for unity, and try to build a scene.
Expected behavior
No errors.
Screenshots
If applicable, add screenshots to help explain your problem.
Your setup (please complete the following information)

Unity Version 2019.4.24f1 LTS
MRTK Version 2.6.1, azure remote rendering versin 1.0.2

Target platform (please complete the following information)
Hololens 2
Additional context
Add any other context about the problem here.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!