Add copyright notice to each file and what its code is for

@kalicki1, once we have a name, could you add the following copyright notice to each source file?
"""
[name] is licensed under the MIT License
Copyright (c) 2020 Kuba Alicki, David Baraka, Rafael Goldstein, Sebastian Zimmeck
privacy-tech-lab, https://privacy-tech-lab.github.io/
"""

"""
[filename]
================================================================================
[one sentence description]
"""

In fact, it would be even better if the description is in some format that allows to generate documentation from it. Maybe, sphinx-js (at least, for JS files, not sure whether there is any cross-language solution)? Generally, it would be nice if all comments, not just at the top of the files, were written in a format that can be parsed into documentation. E.g., per sphinx-js:
/**
 * Return the ratio of the inline text length of the links in an element to
 * the inline text length of the entire element.
 *
 * @param {Node} node - Types or not: either works.
 * @throws {PartyError|Hearty} Multiple types work fine.
 * @returns {Number} Types and descriptions are both supported.
 */
function linkDensity(node) {
    const length = node.flavors.get('paragraphish').inlineLength;
    const lengthWithoutLinks = inlineTextLength(node.element,
                                                element => element.tagName !== 'A');
    return (length - lengthWithoutLinks) / length;
}

So, if you have time, it would be good if you could reformat all comments accordingly.
(cc'ing @davebaraka and @rgoldstein01)

I pushed changes to all the comments in the JavaScript files we created according to the JSDoc format. Hopefully these new comments are formatted well enough for sphinx-js to parse. I have not tried rendering any documentation yet, so there might need to be some changes made to the comments here and there if any problems arise.
As for our non-JS files, I'm wondering if we need to do the same kind of comment updates for our CSS and HTML documents. I'm thinking probably not for our HTML, but I'm not sure about the CSS docs we have. @SebastianZimmeck, let me know what you think we should do regarding these non-JS files, as well as what you think about the general formatting so far.

Great! My suggestion would be to use CSS comments as documented here and HTML comments as documented here. I am not sure if there exists any tool to auto-generate documentation from CSS or HTML files. But that is OK.
Once you finalized those comments, feel free to open a PR.

Alright, I pushed the last of the comments for now. I will open a pull request and when we merge, I think we should be able to close this issue!

Sounds good!


InAppPuchaseData method discrepancies

We are trying to obtain subscription data from InAppPuchaseData, concretely SubValid, but we get the following error:
AndroidJavaException: java.lang.NoSuchMethodError: no non-static method with name=‘getSubValid’ signature=‘()Z’ in class Ljava.lang.Object;

As per Android doc, the method signature is isSubValid. Is there a possibility of this discrepancy being the cause?
Also ExpirationDate is calling getNumOfDiscount: https://github.com/m0skit0/hms-sdk-unity/blob/155547072df8678ff8d3e5e5da96dc61ff127713/hms-sdk-unity/HuaweiMobileServices/IAP/InAppPurchaseData.cs#L57
However, we can properly access to OrderID and ProductId

Fixed in commit master:30d56d160c37bea3dad8b20d04253d4da51a5a02 and 2018:d13940c5ddebc332f15f68fc3ae8eaf451dae1ed


Added volumes support chef/chef-provisioning-docker#29

Implements #29

I had some minor comments, but nothing that would keep this from being merged.  If you want to make the updates please open another PR!


Base64 encoded .js & .yaml files don't work in AWS

When I run my app with Swagger UI locally through everything works well, but when I run it in AWS through API Gateway and Cloudfront the .yaml and .js files are still base64 encoded when they reach the browser and the page renders nothing

We've switched to https://github.com/sylwit/aws-serverless-swagger-ui


Farmer Website

Need to download 'xampp' software to use its backend features in login and sign up page.

This pull request is not according to the contribution guidelines.
Thanks for your valuable time. Keep Hacking!


Angular 9

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
I'm seeing above warnings when i do "npm update"
Angular version 9.02
node : 12.3
npm : 6.0
Will. there be any issues with these warnings did any one facing issues ?

I haven't faced any issues yet using angular 9. All seems to be working for me (using idle and keepalive)

Hi,
I am also facing the same issue.
Angular : v9.1.4
NodeJS : 12.16.3
NPM : 6.14.4
When we will expect the Angular 9 support?
Thanks,
Sudhakar Chandrasekaran.

Hi guys,
I did update the modules core and keep-alive to angular 9.1.6. Who should I ask to be able to propose a pull request?
Marco.

I did update the modules core and keep-alive to angular 9.1.6. Who should I ask to be able to propose a pull request?
Thanks for doing this! Check out the Submitting a Pull Request section here

Addressed in #141


[UX issue] Graph model currently limited.

From my opinion there is an issue in the current implementation of graph model.
Actually if you want to add a layer with multiple input you need to merge and use the standard sequential layers. But graph model are much more powerful from my point of view. Moreover having two kinds of input arguments makes all very clutter.
At the moment we are making the Merge function really messy only to avoid this problem.
Here is an example that worth more then a 1000 words.
Let's assume that we want to create a model accepting two sentence or two images, creates a representation of the two and computes a similarity function between the two representation

model = Graph()
model.add_input(... , name='input1')
model.add_input(... , name='input2')

#i will skip some of the nodes to create the representation
model.add_node(some hidden nodes, name='inp1repr', input='input1')
model.add_node(some hidden nodes, name='inp2repr', input='input2')

#then it is possible to build a similarity function between the two representation in this way.
#Assume x = inp1repr and z= inp2repr so the similarity function can be computed as follows y =  xWz where W is similarity matrix and y is the similarity score.

model.add_node(Dense(inp1repr_size), name='intermediate_node', input='inp2repr')
model.add_node(Activation('linear'), name='similarity', inputs=['inp1repr', 'intermediate_node'], merge_mode='dot')

This implementation is confusing and redundant.
What i'm proposing is to make the Graph model more transparent giving the possibility to add Layers that accepts multiple inputs.
So it is possible to implement a Similarity layer computing y = xWz with a trainable matrix W. By doing so the implementation of the previous code would be.

model = Graph()
model.add_input(... , name='input1')
model.add_input(... , name='input2')

#i will skip some of the nodes to create the representation
model.add_node(some hidden nodes, name='inp1repr', input='input1')
model.add_node(some hidden nodes, name='inp2repr', input='input2')

model.add_node(Similarity(), name='similarity',  input=['inp1repr', 'inp2repr'])

That will remove most of the problems in the Merge Layer by simply implementing all the layers accepting multiple inputs as autonomous layers.
This change can break previous models but it can be implemented in a way that if we have a standard layer with multiple inputs and a merge mode, we can simply add two layers: a merge layer, and the chosen layer on top; in this way it should work exactly as before.
What do you think?
Since it is a big change i ask for the @fchollet opinion in particular before starting on anything.
Thanks to all of you.

But that already works... just use
model.add_node(Similarity(), name='similarity',  input=['inp1repr', 'inp2repr'], merge_mode='join')

and inside your Similarity
def get_output(self, train)
    X = self.get_input() # returns OrderedDict {'inp1repr': tensor1, 'inp2repr': tensor2}
    X = list(X.items())
    return T.dot(T.dot(X[0], self.W), X[1])

Also, you may not even need to write a whole new class. Just use a LambaLayer.

@EderSantana  I dont think you can have a trainable matrix inside a lambda function. Lambda functions are stateless functions.. just an F(x).

@farizrahman4u oh, my bad! So yeah, I think he just needs to write his new class then.

Thanks you guys i just not got how the join mode worked, but now i have another question. In order to build the layer (for the Weight matrix) i need the inputs shapes. the merge mode join do not provide any output shape, so my layer should get the two input_shapes directly, like that?
model.add_node(Similarity(1, activation='sigmoid', input_shapes=[model.nodes['input1'].output_shape, model.nodes['input2'].output_shape]), name='sim', inputs=['input1','input2'], merge_mode='join')

There is even this error that the join mode does't find the name of the inputs https://github.com/fchollet/keras/issues/975#issuecomment-156851799

Lot of people have been having trouble with join mode recently.We really need @fchollet to look into it.


Ruby claims gem extensions aren't built

First and foremost, thank you for all of your hard work on this project. Being able to write Ruby extensions in Rust with minimal fanfare is enticing; I'm eager to use this library!
I've been trying to build a Ruby gem that leverages magnus, and despite being able to build the gem itself without issue, when trying to consume it from another Ruby project, it complains that the native extensions aren't built:
Ignoring rust_blank-0.1.0 because its extensions are not built. Try: gem pristine rust_blank --version 0.1.0
To simplify things as much as possible, I've taken the rust_blank example from this repo and extended it with the bare minimum that highlights what I'm trying to do. From a clean slate within that repo, I've done the following:

Built the gem using rake in the root directory (provided you have the rake-compiler gem installed, this builds and tests the gem successfully)
Navigated into the app/ subdirectory and run bundle, to install the path-referenced gem (runs without issue)
Run bundle exec irb to get an interactive shell to try the gem (loads, but displays the error above)

Can you help shed light on what I'm doing wrong here? Unlikely to be an issue, but for completeness' sake, I'm running Ruby v3.1.3p185 and Rust v1.67.1.

Hey, thanks for taking a look at this library and the kind words.
So I think this does work, you just need to ignore the error (see the transcript below of me trying it).
It's hitting some edge case with the integration of RubyGems, Bundler, and Bundler's path dependencies feature that generates that extensions are not built error even when everything is fine.
I don't think this is a Magnus/Rust issue, I think it's a problem any extension gem will run into. I tried the same thing with the original, in C, fast_blank and got exactly the same error.

Here's how I tried it out:
Lets clone the project and run bundle:
~/gemtest> git clone git@github.com:jmacdonald/rust_blank.git
Cloning into 'rust_blank'...
remote: Enumerating objects: 25, done.
remote: Counting objects: 100% (25/25), done.
remote: Compressing objects: 100% (21/21), done.
remote: Total 25 (delta 1), reused 25 (delta 1), pack-reused 0
Receiving objects: 100% (25/25), 1.37 MiB | 4.13 MiB/s, done.
Resolving deltas: 100% (1/1), done.
~/gemtest> cd rust_blank/app/
~/g/r/app> bundle
Resolving dependencies...
Fetching gem metadata from https://rubygems.org/.
Using bundler 2.4.6
Fetching rb_sys 0.9.65
Installing rb_sys 0.9.65
Using rust_blank 0.1.0 from source at `..`
Bundle complete! 1 Gemfile dependency, 3 gems now installed.
Bundled gems are installed into `./.bundle`

Ok, so bundler says it installed the gem, lets see if it works:
~/g/r/app> bundle exec irb
Ignoring rust_blank-0.1.0 because its extensions are not built. Try: gem pristine rust_blank --version 0.1.0

Hummm, that doesn't look promising, lets try anyway:
irb(main):001:0> require "rust_blank"
~/src/bundle_test/rust_blank/lib/rust_blank.rb:3:in `require_relative': cannot load such file -- ~/src/bundle_test/rust_blank/lib/rust_blank/rust_blank (LoadError)
        from ~/src/bundle_test/rust_blank/lib/rust_blank.rb:3:in `<top (required)>'
        from <internal:~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
        from <internal:~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
        from (irb):1:in `<main>'
        from ~/.rbenv/versions/3.2.0/lib/ruby/gems/3.2.0/gems/irb-1.6.2/exe/irb:11:in `<top (required)>'
        from ~/.rbenv/versions/3.2.0/bin/irb:25:in `load'
        from ~/.rbenv/versions/3.2.0/bin/irb:25:in `<top (required)>'
        from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/cli/exec.rb:58:in `load'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/cli/exec.rb:58:in `kernel_load'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/cli/exec.rb:23:in `run'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/cli.rb:491:in `exec'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/cli.rb:34:in `dispatch'
	from ~/.rbenv/versions/3.2.0/lib/ruby/site_ruby/3.2.0/bundler/vendor/thor/lib/thor/base.rb:485:in `start'
	... 6 levels...
irb(main):002:0>

Nope, looks like it the extension really wasn't built. What happens if we build it ourselves?
~/g/r/app> cd ..
~/g/rust_blank> rake compile
mkdir -p tmp/arm64-darwin22/rust_blank/3.2.0
cd tmp/arm64-darwin22/rust_blank/3.2.0
~/.rbenv/versions/3.2.0/bin/ruby -I. -r.rake-compiler-siteconf.rb ../../../../ext/rust_blank/extconf.rb
cd -
cd tmp/arm64-darwin22/rust_blank/3.2.0
/usr/bin/make
generating target/release/librust_blank.dylib (release)
cargo rustc --target-dir target --manifest-path ../../../../ext/rust_blank/Cargo.toml --lib --release -- -C linker=clang -L native=~/.rbenv/versions/3.2.0/lib -L native=~/.rbenv/versions/3.2.0/lib -C link-arg=-Wl,-multiply_defined,suppress -l pthread -C link-arg=-Wl,-undefined,dynamic_lookup
   Compiling memchr v2.5.0
   Compiling proc-macro2 v1.0.47
   Compiling unicode-ident v1.0.5
   Compiling glob v0.3.0
   Compiling quote v1.0.21
   Compiling libc v0.2.137
   Compiling minimal-lexical v0.2.1
   Compiling cfg-if v1.0.0
   Compiling libloading v0.7.4
   Compiling regex-syntax v0.6.28
   Compiling bindgen v0.60.1
   Compiling clang-sys v1.4.0
   Compiling peeking_take_while v0.1.2
   Compiling bitflags v1.3.2
   Compiling shlex v1.1.0
   Compiling aho-corasick v0.7.19
   Compiling nom v7.1.1
   Compiling lazy_static v1.4.0
   Compiling lazycell v1.3.0
   Compiling rustc-hash v1.1.0
   Compiling shell-words v1.1.0
   Compiling syn v1.0.103
   Compiling rb-sys-env v0.1.1
   Compiling magnus v0.5.1
   Compiling regex v1.7.0
   Compiling cexpr v0.6.0
   Compiling magnus-macros v0.4.0
   Compiling rb-sys-build v0.9.64
   Compiling rb-sys v0.9.64
   Compiling rust_blank v0.2.0 (~/src/bundle_test/rust_blank/ext/rust_blank)
    Finished release [optimized] target(s) in 7.32s
cd -
mkdir -p tmp/arm64-darwin22/stage/lib/rust_blank
/usr/bin/make install target_prefix=
generating target/release/librust_blank.dylib (release)
cargo rustc --target-dir target --manifest-path ../../../../ext/rust_blank/Cargo.toml --lib --release -- -C linker=clang -L native=~/.rbenv/versions/3.2.0/lib -L native=~/.rbenv/versions/3.2.0/lib -C link-arg=-Wl,-multiply_defined,suppress -l pthread -C link-arg=-Wl,-undefined,dynamic_lookup
    Finished release [optimized] target(s) in 0.02s
installing rust_blank.bundle to ~/src/bundle_test/rust_blank/lib/rust_blank
/usr/bin/install -c -m 0755 rust_blank.bundle ~/src/bundle_test/rust_blank/lib/rust_blank
cp tmp/arm64-darwin22/rust_blank/3.2.0/rust_blank.bundle tmp/arm64-darwin22/stage/lib/rust_blank/rust_blank.bundle

All done, lets try again:
~/g/rust_blank> cd app/
~/g/r/app> bundle exec irb
Ignoring rust_blank-0.1.0 because its extensions are not built. Try: gem pristine rust_blank --version 0.1.0

Uh oh, this again, well, lets just try anyway:
irb(main):001:0> require "rust_blank"
=> true
irb(main):002:0> "".blank?
=> true
irb(main):003:0> "foo".blank?
=> false

Turns out it works, despite the error message.

Well, that's good news, despite the fact that I should have tried that myself. 😅 Very much appreciate you taking the time to give this a spin and help me out! 🍻


Test failure: CheckStartupEventLogMessage

This test fails occasionally with the following error:
'Application '.+' started the coreclr in-process successfully.' didn't match any event log messaged.
Expected: True
Actual:   False
   at Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests.EventLogHelpers.VerifyEventLogEvent(IISDeploymentResult deploymentResult, ITestSink testSink, String expectedRegexMatchString) in /_/test/Common.FunctionalTests/Utilities/EventLogHelpers.cs:line 75
   at Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests.EventLogTests.CheckStartupEventLogMessage() in /_/test/Common.FunctionalTests/Inprocess/EventLogTests.cs:line 31
--- End of stack trace from previous location where exception was thrown ---
------- Stdout: -------
| [0.001s] TestLifetime Information: Starting test CheckStartupEventLogMessage at 2018-08-27T22:59:49
| [0.004s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Deploying [Variation] :: ServerType=IIS, Runtime=CoreClr, Arch=x64, BaseUrlHint=, Publish=True
| [0.007s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\runtimes\unix\lib\netcoreapp2.0
| [0.008s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying System.Security.Principal.Windows.dll
| [0.009s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\runtimes\unix\lib
| [0.010s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\runtimes\unix
| [0.011s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\runtimes\win\lib\netcoreapp2.0
| [0.011s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying System.Security.Principal.Windows.dll
| [0.013s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\runtimes\win\lib
| [0.013s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\runtimes\win
| [0.013s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\runtimes
| [0.014s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b\wwwroot
| [0.014s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying static.txt
| [0.015s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug: Processing C:\BuildAgent\temp\buildTmp\6f2880b212414ba196db4b901ff07a9b
| [0.015s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying aspnetcorev2_inprocess.dll
| [0.017s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying aspnetcorev2_inprocess.pdb
| [0.023s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying InProcessWebSite.deps.json
| [0.025s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying InProcessWebSite.dll
| [0.027s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying InProcessWebSite.pdb
| [0.028s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying InProcessWebSite.runtimeconfig.json
| [0.030s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Authentication.Abstractions.dll
| [0.031s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Authentication.Core.dll
| [0.033s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Hosting.Abstractions.dll
| [0.034s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Hosting.dll
| [0.036s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Hosting.Server.Abstractions.dll
| [0.038s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Http.Abstractions.dll
| [0.039s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Http.dll
| [0.042s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Http.Extensions.dll
| [0.044s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Http.Features.dll
| [0.046s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Server.IIS.dll
| [0.047s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Server.IIS.pdb
| [0.049s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.Server.IIS.xml
| [0.051s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.AspNetCore.WebUtilities.dll
| [0.052s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.Extensions.Configuration.Abstractions.dll
| [0.054s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.Extensions.Configuration.Binder.dll
| [0.055s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.Extensions.Configuration.dll
| [0.057s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.Extensions.Configuration.EnvironmentVariables.dll
| [0.059s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Debug:   Copying Microsoft.Extensions.Configuration.FileExtensions.
...

Other tests within that build may have failed with a similar message, but they are not listed here. Check the link above for more info.
This test failed on master.
CC @muratg
This issue was made automatically. If there is a problem contact ryanbrandenburg.

Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

Is this actually a flaky test?

No, this is a regular test failure, fix the test/product (TODO: Link to commit/PR)
Yes, proceed below...

Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

[ ] File a bug against the product (TODO: Link to other bug)
Is it possible to change the test to avoid the flakiness?

Yes? Go to the "Change the test!" section.
No?

[ ] Disable the test (TODO: Link to PR/commit)
[ ] Wait for other bug to be resolved
[ ] Wait for us to get build that has the fix
[ ] Re-enable our test (TODO: Link to PR/commit)
[ ] Close this bug

Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

Is there is a way to change our test to avoid this flakiness?

Yes? Change the test!

[ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
[ ] Run the test 100 times locally as a sanity check.
[ ] Close this bug

No?

Is there any logging or extra information that we could add to make this more diagnosable when it happens again?

Yes?

[ ] Add the logging (TODO: Link to PR/commit)

No?

[ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

CheckStartupEventLogMessage failed with about the same error on master.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

Startup failed with about the same error on 2.2.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

@jkotalik: Addressed by #1338?

CheckStartupEventLogMessage will be addressed, but http://aspnetci/viewLog.html?buildId=541687&buildTypeId=Releases_22xPublic_Win2012 will not. That should be a separate issue.

#1350


Support specifying git short hashes for use & install

Let's users install & bind to a specific framework revision.
The test in test/commands_test.dart has not yet been updated.

@ened thanks for the PR, I can think of a few cases for this, but do you mind telling me a bit about the motivation for this change? Also I am wondering if this flow should be put behind a flag like fvm install <commit_id> --commit since it will be used in more advanced use cases

@leoafarias We are currently forced to make production builds on a specific commit in the flutter master branch, due to no beta or dev release being available. Thus, we would like to configure the specific git hash in the .fvm/ config file for the CI to pick up and apply.
I would prefer not to specify any --commit flag. We can use the long version of the git hash to make things more explicit, but in general no channel or release name will ever look like a short hash.


Missing endpoints for model monitor

Description
There is no Crud functionality in order to update Model monitors in DB.
Solution

[ ] Create the appropriate schemas and crud functionality in order to be able to update the model monitors in DB
[ ] Create the appropriate end-point (put) in for this update
[ ] Create the corresponding SDK function
[ ] Use the above SDK function in Streamlit app in order to update the status of the selected monitors
[ ] Delete monitor from UI

I believe it needs more time since there are many changes plus tests.

Updated the estimation


Improve visualisation of multiple error traces

When using the -continue option, TLC may produce multiple error traces. At the moment, the extension displays all the error traces at once, one below another, which affects usability and might lead to performance issues when there're many of them.
It would be more convenient to display only one error trace and switch between them using links next to error messages.

We might want to improve TLC's output markers with multiple traces because TLC currently makes it difficult to parse them.

Actually, it looks pretty parseable to me. The current parser already can handle such cases properly, I only need to slightly change the data structure I use for storing errors.

Available in the v1.4 beta.


rework logic around collection deletion

Description
The existing code was deleting collections that didn't match all three criteria.  I made it a little wordier with some trace logging and track each criteria separately.
Example logging while debugging this:
| 🎆 Independence Day Movies is smaller than None, DELETE                                             |
| 🎆 Independence Day Movies - managed: False - collection_is_managed: True, NO DELETE                |
| 🎆 Independence Day Movies - configured: False - collection_is_configured: False, DELETE            |
| 🎆 Independence Day Movies would NOT have been deleted by new code                                  |
| 🎆 Independence Day Movies would be deleted by old code                                             |

Trace logging produced by PR'ed code:
| 🎆 Independence Day Movies - less: None vs collection size: 4, DELETE: False                        |
| 🎆 Independence Day Movies - managed: False vs collection managed: True, DELETE: False              |
| 🎆 Independence Day Movies - configured: False vs collection configured: False, DELETE: True        |

Type of Change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)

I don't think this is correct:
 return any((delete_less, delete_managed, delete_configured))

Don't we want all three to match rather than any one?

yea you're right it should be all not any


Migrate open issues from shahpr/RiboViz to RiboViz/RiboViz

Migrate open issues from shahpr/RiboViz/issues to RiboViz/RiboViz/issues

GitHub describes Transferring an issue to another repository but this is for moving issues between repositories within a single organisation.


inconsistency with tab header names and cycled maps descriptions

differences in maps:
http://gyazo.com/16831d004a9511bc733e2a72cd454ffc
maps with correct color get player "null", and the maps that for some reason load with old color, have the name right (both maps use UUID's on their xml's)
the difference is that fractal (name = null) uses:
<author uuid="dbe909c6-2b15-4528-b12a-c77ed4031496" contribution="Design and building"/>
and Golden Drought III uses:
<author uuid="459fd919-0d58-401a-9601-6cab1eb721ab"/>
also this http://gyazo.com/21973f36ca69a00d391fb711720acaeb should have the new names
ps:
also adding to this, if the map creator is in the game, they will always be displayed as the name they had on the last match (joined red or blue or obs from the last match), because that only gets set on map load, not on player joining teams event

Some more on this :
from what i've seen:
it only uses the new names if there are collaborators
ocasionally it does work and displays new names correclty

I wrote the original system for this, and if I recall correctly, what I ended up doing is fetching player names by UUIDs using Moajng's API, and then storing them permanently. I stored them because Mojang's API has a 600 requests cap every 10 minutes, and so only 600 usernames can be grabbed. null shows up when it could get the name, and it will use old names because once it gets a name, it never again checks for an updated name. OCN gets around the 600 request limit by storing UUIDs and player names in their own databases, presumably. When a player logs onto Overcast, it updates their databases, and in return the displayed names for maps. Because many map makers won't be joining every scrim server frequently, I can't follow this same approach in Cardinal, unless I kept a public repository somewhere, although it would likely get vandalized.

@twizmwazin This is update is invoked when the server starts right?

How about storing the UUID's + player names along with a time mark (for example 24h), if the last time checked was more than 24 hours ago, then check again, unless you have passed the 600 request cap, then you have to use the old one

@Pablete1234 Feel free to make a PR

@twizmwazin i'm not that good with java, i can understeand it and writte basic code (you saw how i prevented /tp outside), but definetly i'm not an experienced programmer

@twizmwazin now that i think of it, to get the first names you should use mojang's api, but to update the names you may want to use oc.tc/oldname, and that redirects to oc.tc/newname, maybe?

@Pablete1234 Moajng's API gives a fill username history on a single request.

@twizmwazin I know how mojang's api work, but you said that there is a 600 request limit, maybe that's a way to go arround

@Pablete1234 The best way would be to regularly check for updates, on a schedule.

but as it is right now, why do I always get "null" map authors, what is filling that 600 cap?

600 different authors/contributors have been requested

Just as I start the server the first map that loads already has authors "null"

[00:18:52] [Server thread/WARN]: Couldn't look up profile properties for com.mojang.authlib.GameProfile@2a206a9c[id=91295196-9cd2-4963-b7ec-afd1a110edbe,name=Pbat,properties={},legacy=false]
com.mojang.authlib.exceptions.AuthenticationException: The client has sent too many requests within a certain amount of time
	at com.mojang.authlib.yggdrasil.YggdrasilAuthenticationService.makeRequest(YggdrasilAuthenticationService.java:65) ~[custom_server.jar:git-Bukkit-0a11d75]
	at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillGameProfile(YggdrasilMinecraftSessionService.java:175) [custom_server.jar:git-Bukkit-0a11d75]
	at com.mojang.authlib.yggdrasil.YggdrasilMinecraftSessionService.fillProfileProperties(YggdrasilMinecraftSessionService.java:168) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.TileEntitySkull.b(TileEntitySkull.java:92) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.TileEntitySkull.e(TileEntitySkull.java:71) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.TileEntitySkull.a(TileEntitySkull.java:41) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.TileEntity.c(TileEntity.java:79) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.ChunkRegionLoader.loadEntities(ChunkRegionLoader.java:414) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.craftbukkit.v1_8_R3.chunkio.ChunkIOProvider.callStage2(ChunkIOProvider.java:46) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.craftbukkit.v1_8_R3.chunkio.ChunkIOProvider.callStage2(ChunkIOProvider.java:1) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.craftbukkit.v1_8_R3.util.AsynchronousExecutor.skipQueue(AsynchronousExecutor.java:337) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.craftbukkit.v1_8_R3.util.AsynchronousExecutor.getSkipQueue(AsynchronousExecutor.java:295) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.craftbukkit.v1_8_R3.chunkio.ChunkIOExecutor.syncChunkLoad(ChunkIOExecutor.java:16) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.ChunkProviderServer.getChunkAt(ChunkProviderServer.java:115) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.ChunkProviderServer.getChunkAt(ChunkProviderServer.java:97) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.craftbukkit.v1_8_R3.CraftServer.createWorld(CraftServer.java:970) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.Bukkit.createWorld(Bukkit.java:518) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.WorldCreator.createWorld(WorldCreator.java:271) [custom_server.jar:git-Bukkit-0a11d75]
	at in.twizmwaz.cardinal.cycle.Cycle.run(Cycle.java:42) [CardinalPGM-1.0-SNAPSHOT%20(86).jar:?]
	at in.twizmwaz.cardinal.GameHandler.cycleAndMakeMatch(GameHandler.java:51) [CardinalPGM-1.0-SNAPSHOT%20(86).jar:?]
	at in.twizmwaz.cardinal.GameHandler$1.run(GameHandler.java:36) [CardinalPGM-1.0-SNAPSHOT%20(86).jar:?]
	at org.bukkit.craftbukkit.v1_8_R3.scheduler.CraftTask.run(CraftTask.java:53) [custom_server.jar:git-Bukkit-0a11d75]
	at org.bukkit.craftbukkit.v1_8_R3.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:349) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.MinecraftServer.B(MinecraftServer.java:696) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.DedicatedServer.B(DedicatedServer.java:335) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.MinecraftServer.A(MinecraftServer.java:645) [custom_server.jar:git-Bukkit-0a11d75]
	at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:555) [custom_server.jar:git-Bukkit-0a11d75]
	at java.lang.Thread.run(Unknown Source) [?:1.8.0_51]


ENH: Use lzw + predictor 3 with SNAP10 to compress SAR bands that can be despeckled

SNAP 9  and before wouldn't handle LZW compression with predictor == 3.
:x: Caused by: javax.imageio.IIOException: Illegal value for Predictor in TIFF file
Test it with SNAP10 and update sar_product.SAR_PREDICTOR to 2 if it's working.

ROLLBACK: don't use "default (and optimized) predictor in rasters.write if SNAP is version 10 or higher" as it depends not on SNAP but on javax.imageio's version.
The problem is I don't know how to check if javax.imageio can use this predictor 🤔


L1 VTS - Improve L1 based on new header File Updates (1.0.3)

This task is to consider the latest header file updates and consider if there are any changes in L1 concerning the newest header files.

Issue was resolved with pull request https://github.com/rdkcentral/rdk-halif-test-power_manager/pull/13


SetupUI sends wrong data when Bitcoin-only

I selected Bitcoin only in the setup process but when looking at the final API request the WebUi sends it says lnd:

Fixed that 3 Hours ago:
https://github.com/cstenglein/raspiblitz-web/pull/321
Mentioned you as well :O


can't see image that last opponent warband in battlegrounds.

i'm not good at English.
HDT can't create jpg image file that last opponent warband to play battlegrounds on Ubuntu.
i have a temporary solution for this.   just for battlegrounds.
last update : 20.2
download attached file and then extract below.
~/.wine.heartstone/drive_c/users/$USER/Application Data/HearthstoneDeckTracker/Images/CardPortraits/
CardPortraits.tar.gz

for some reason firefox thinks this file contains virus...

for some reason firefox thinks this file contains virus...

i don't know why but upload image file again ...
ConvertImage.tar.gz

i don't know why but upload image file again ...
ConvertImage.tar.gz

update image ...    + argent braggart
ConvertImage.tar.gz

update image ...    + argent braggart
ConvertImage.tar.gz

update image for  patch 19.4  + ( soul devourer, ring matron )
ConvertImage.tar.gz

update image for  patch 19.4  + ( soul devourer, ring matron )
ConvertImage.tar.gz

update 20.04
download and move to "~/.wine.heartstone/drive_c/users/$USER/Application Data/HearthstoneDeckTracker/Images/CardPortraits/" directory.

it is not necessary now.   it fix by HDT.


Set a minimum time for hashrate to achieve TTD

set the minimum date in hasrate to achieve ttd as 12 now(there isn't any hashrate that could travel to past and activate the merge)

Thanks!


dart can not recognize aqueduct package

As you can see, the dart can not recognize aqueduct package.
Everything works fine but it's unable to use autocomplete due to the problem.
I am using vscode 1.37.0 and dart in a stable channel.
Does anyone have same issue?

I also tested on the webstorm but it is still same.

solved.

@wurikiji Could you give a small hint how you solved it, just in case someone finds this issue.

@MyracleDesign  It was not an issue of aqueduct. I incidentally created the aqueduct project inside the other dart project. So the package dependencies were analyzed by the parent project.


fixes #1289 - Trim \n at the end of a secret when using Mesos auth

I think when we implemented this in #710 we intentionally read all of the bytes in the file.  This is because the authentication mechanism in Mesos could be an arbitrary module.  This change would make it impossible to define a secret that ends in the byte 0x0A, for example.  Finally, the Mesos protobuf definition of the secret to be of type bytes hints that we shouldn't assume UTF-8.
Maybe we could benefit from a second opinion from people who have worked on authentication in Mesos.  cc: @adam-mesos

Interesting points. Mesos' default CRAM-MD5 authentication must assume the secret does not include a newline character,  since the master's --credentials file is newline-delimited between credentials. Kerberos authentication does not use the secret, since keytabs/tickets are managed out of band. Other alternative authenticatee modules may want to include newlines in their secrets, so for their sake we may want to read the entire file, but until they complain we can probably trim any newlines when reading the secret file. Perhaps in the future, it could be up to the CRAM-MD5 authenticatee to trim out newlines.

@drexin soo sorry for bothering you on this.  I seemed to miss deleting trailing space on secret file....
echo -n 'mesos rocks' > mesos-credentials and echo -n 'rocks' > marathon-secret works fine on original authentication code.

Thanks @everpeace

@drexin
I am seeing the 'Master refused authentication' while authenticating Marathon against Mesos Master. Here is the info. Thoughts?
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.870514 32617 sched.cpp:326] New master detected at master@xx.xx.xxx.xx:5050
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.870781 32617 sched.cpp:382] Authenticating with master master@xx.xx.xxx.xx:5050
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.870803 32617 sched.cpp:389] Using default CRAM-MD5 authenticatee
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.871073 32617 authenticatee.cpp:97] Initializing client SASL
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.872035 32617 authenticatee.cpp:121] Creating new client SASL connection
May  8 01:33:35 jmesosmas-03p mesos-master[27629]: I0508 01:33:35.872970 27657 master.cpp:5521] Authenticating scheduler-1475a8a3-0937-4fe0-a17c-e6e1b85648d0@xx.xx.xxx.xx:9090
May  8 01:33:35 jmesosmas-03p mesos-master[27629]: I0508 01:33:35.873271 27657 authenticator.cpp:98] Creating new server SASL connection
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.874003 32615 authenticatee.cpp:212] Received SASL authentication mechanisms: CRAM-MD5
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.874047 32615 authenticatee.cpp:238] Attempting to authenticate with mechanism 'CRAM-MD5'
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.875255 32615 authenticatee.cpp:258] Received SASL authentication step
May  8 01:33:35 jmesosmas-03p marathon[32372]: E0508 01:33:35.876106 32615 sched.cpp:465] Master master@xx.xx.xxx.xx:5050 refused authentication
May  8 01:33:35 jmesosmas-03p marathon[32372]: I0508 01:33:35.876124 32615 sched.cpp:1131] Got error 'Master refused authentication'

Here is the Mesos Master setting
[root@jmesosmas-03p auth]# more acls
{ "run_tasks": [ { "principals": { "type": "ANY" }, "users": { "type": "ANY" } } ], "register_frameworks": [ { "principals": { "type": "ANY" }, "roles": { "type": "ANY" } } ] }

[root@jmesosmas-03p auth]# more credentials
marathon marathonsecret

[root@jmesosmas-03p mesos-master]# more acls
file:///etc/mesos/auth/acls

[root@jmesosmas-03p mesos-master]# more credentials
file:///etc/mesos/auth/credentials

[root@jmesosmas-03p mesos-master]# more authenticate
true

Here is the marathon configuration
[root@jmesosmas-03p conf]# more mesos_authentication_principal
marathon

[root@jmesosmas-03p conf]# more mesos_authentication_secret_file
/etc/mesos/auth/marathon.secret

[root@jmesosmas-03p conf]# more /etc/mesos/auth/marathon.secret
marathonsecret

Here is the list of packages
# rpm -qa | grep cyrus
cyrus-sasl-md5-2.1.26-20.el7_2.x86_64
cyrus-sasl-lib-2.1.26-20.el7_2.x86_64
cyrus-sasl-plain-2.1.26-20.el7_2.x86_64
cyrus-sasl-2.1.26-20.el7_2.x86_64

Version:
Mesos Version:0.27.1
Marathon: Version 0.15.2


Added support for Java configuration class for provided yml config.

@regunathb , @ToOnlyGaurav Please review.

Steps to run it on local machine:

Checkout my branch
Go directly inside "grpc-express" where this project has been checked out.
Run "./gradlew clean build install" to genrate various jars required to run SampleApplication.java class present inside "grpc-express-template" directory.
Checkout project inside "grpc-jexpress-template" directory as separate MVN run project.
Run SampleApplication.java class with following arguments:
"server "

@regunathb , @ToOnlyGaurav Please review.

@monishgandhi FYI.


Make BigWig files and use RSeQC geneBody_coverage2.py

Dear Phil,
The gene body coverage module requires more memory for samples with larger size bam (>100G) nearly impossible to get it done if the server memory is <128G.
Possible for you to generate bigwig and run geneBody_coverage2.py (http://rseqc.sourceforge.net/) instead bam as input which is less memory hungry? Or can you suggest any other way to overcome this issue within the pipeline?
Unless the "genebody_coverage" complete the pipeline not generating the MultiQC.
Thanks
Justin

Hi Justin,
Sorry for this problem.  We’ve already split this task out by itself for this reason and subsample the BAM file down to one million reads first to limit the memory usage. I suspect here that it’s actually this subsampling method which is eating all of the memory. We tried to be clever and randomise the subsampling but it would perhaps be better to just take the first one million reads instead. I’ll take a look into this.
Phil

Hi @justinjeyakani,
I just pushed an update in #37 which should hopefully help the memory problem. It would be great if you could pull the latest version of the code and give it a try.
I've tested with a minimal dataset and it seems to work ok. I'd like to see how many reads the subsampled BAM files are ending up with using the current default subsampling target of ~10GB - we may be able to tweak this either up (to get better / more representative gene body coverage profiles) or down (for faster execution if the profiles aren't affected).
I'm trying to test on some larger data now myself, but our HPC is having problems again 😣
Phil

Dear Phil,
Sure, will pull the latest and test with large bam files.
Any concern to include generate bigwig and run geneBody_coverage2.py on it. This serves to make use of the entire bam with less memory and an additional feature of the pipeline for visualization of the alignment.

Hi Justin,
Yes, perhaps you're right. I skipped over this suggestion initially because we'd already half-implemented the subsampling solution. But as you say, perhaps doing a bigwig conversion is the better approach (and would be simpler pipeline code too).
Phil

https://deeptools.readthedocs.io/en/develop/content/tools/bamCoverage.html
Should be sufficient to get this going?

This is now in the dev branch of the pipeline :-)


Simple Packaging and Installer Scripts

This will likely become more complicated in the future, but is good enough to start out with.
https://github.com/mono/Embeddinator-4000/issues/222

Build and tests passed.

👍  on your suggestions @spouliot. Will PR soon.

Build and tests passed.


BYOE bring your own embeddings

Add command group embeddings and add index command to put embeddings into a collection

TODO:

these embeddings are not getting loaded correctly. I don't think I'm calling _insert correctly

merging for now, we can iterate later


Reenable DefaultDiagnosticConsoleFormatter

Because DefaultDiagnosticConsoleFormatter is crashing in some projects (see https://github.com/apple/swift-docc/issues/678)  we are disabling it by default (and using the IDEDiagnosticConsoleFormatter instead).
Once we've fixed the crash and retested DefaultDiagnosticConsoleFormatter we should reenable it as the default diagnostic formatter again.
The main alternative to disabling the diagnostic formatter would be to revert the commit that made it the default but that change would be unnecessarily large.

We didn't disable this formatter so there's nothing to reenable.


[chore] update to Go 1.21

Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

It's not clear to me why the lint checks are failing, @nebula-aac


Fix bundling

attempting to provide proper meta-data (and code, via dist/only.index.js) to make this library friendly to bundlers
/CC @tonydangblog

References:

https://github.com/defunctzombie/package-browser-field-spec
https://stackoverflow.com/questions/74022786/is-browser-field-of-package-json-useless/74240581#74240581
https://webpack.js.org/guides/package-exports/

Had to go a whole different route and provide plugins for both vite and webpack, since it seems neither supports the proper config capabilities to handle copying-in and loading the external non-ESM dependency bundle.


Fix Payout calculation

Task: Update Exercise payout calculation
High Level Description

Fix payout calculation
Refactor Controller and Calculator to inherit the same OMath contract.
Refactor controller test cases to avoid using small numbers

Code

[x] Unit test 100% coverage
[x] Does your code follow the naming and code documentation guidelines?

Documentation

[x] Is your code up to date with the spec?
[x] Have you added your tests to the testing doc?

The getExcessCollateral function still seems to behave differently before and after expiry. Before expiry its returning the amount collateral needed for call options in ETH, after expiry it seems to be giving the amount of excess collateral denominated in USD / strike asset.

I don't think that's the case:
In _getMarginRequired, it's basically our formula for put / calls before and after expiry. For puts, it returns value in strike, (USDC), and for calls, it returns value in underlying (ETH), not collateral.
So In 91~94
        // get required margin, denominated in strike or underlying asset
        FPI.FixedPointInt memory marginRequirement = _getMarginRequired(_vault);
        // get exchange rate to convert marginRequirement to amount of collateral
        FPI.FixedPointInt memory exchangeRate = _getToCollateralRate(_vault.shortOtokens[0]);

after I get the marginRequirement which can be denominated in strike or underlying, I used another function to get the assets's exchange rate to convert it into amount of collateral. For puts, this function gets underlying => collateral rate, for calls, it gets strike to collateral rate.

The getExcessCollateral function still seems to behave differently before and after expiry. Before expiry its returning the amount collateral needed for call options in ETH, after expiry it seems to be giving the amount of excess collateral denominated in USD / strike asset.

I don't think that's the case:
In _getMarginRequired, it's basically our formula for put / calls before and after expiry. For puts, it returns value in strike, (USDC), and for calls, it returns value in underlying (ETH), not collateral.
So In 91~94
        // get required margin, denominated in strike or underlying asset
        FPI.FixedPointInt memory marginRequirement = _getMarginRequired(_vault);
        // get exchange rate to convert marginRequirement to amount of collateral
        FPI.FixedPointInt memory exchangeRate = _getToCollateralRate(_vault.shortOtokens[0]);

after I get the marginRequirement which can be denominated in strike or underlying, I used another function to get the assets's exchange rate to convert it into amount of collateral. For puts, this function gets underlying => collateral rate, for calls, it gets strike to collateral rate.

That makes sense. I got confused there. My bad. We should definitely try to simplify the margin calculator more 😅


binary-mode QR scanning

Is your feature request related to a problem? Please describe.
When scanning a QR code that uses byte mode encryption, the data which is returned is incorrect. On Android, I get a string encoding of the byte data, eg. "��\t�@\u00", from which I cannot recover the original bytes; on iPhone, the native barcode scanner returns a read error.
Describe the solution you'd like
I would like to have an additional scan option which allows me to specify the read-mode of the QR code, ie. binary. Then, the results.content will be either a unit8array containing the bytes in a raw form, or a base64url encoding thereof.
Describe alternatives you've considered
We have considered encoding our binary data as bae64url, and then encoding the result text into the QR code. This is inefficient because we then need 8bits to encode each base64url character, instead of 6.

Having a closer look. It appears that com.google.zxing does expose a method called getRawBytes(), which should do what I need on Android. I'll get a PR going in the next few days.

For iOS, this blog post seems to hold promising information: https://ristkof.be/posts/qr-codes-binary-on-ios.html

@wombatslayer This seems not really possible with the new ml-kit implementation as far as I was able to research, I am closing this for now.


Update index.html

removed "4 Update README.md"

👍

@BennyCarlsson why changes aren't showing on live site :cry:


White Screen

Hi, I am new to CG. My Device Specs are Amd5800h rtx3060 Laptop. The console log shows "AMD Radeon" as selected device instead of 3060. I am experiencing total white screen.  Can Anyone help me fix it?

Currently the code is selecting the AMD GPU ahead of your other card. We have already improved the GPU detection logic in multi GPU systems which will attempt to choose the best card available. This will be included in the next upcoming release


IsADirectoryError: [Errno 21] Is a directory: 'input_images/.ipynb_checkpoints'

IsADirectoryError                         Traceback (most recent call last)
 in ()
58 pbar = tqdm(total=len(images_list))
59 for image_name in images_list:
---> 60     img = Image.open(os.path.join(image_dir, image_name)).convert('RGB')
61     img_size = img.size
62     img = img.resize((768, 768), Image.BICUBIC)
/usr/local/lib/python3.7/dist-packages/PIL/Image.py in open(fp, mode)
2841
2842     if filename:
-> 2843         fp = builtins.open(filename, "rb")
2844         exclusive_fp = True
2845
IsADirectoryError: [Errno 21] Is a directory: 'input_images/.ipynb_checkpoints'

Updated colab notebook for same


Chartist CSS to Stylus

What are your thoughts on this @thefoxis ?
My understanding is the correct way to style the charts is to do what we did, drop the default and use our own CSS. Given this is done should we now move it to Stylus for the benefits that gives?
If yes, go ahead and label as a change and do.

we'd want to use most of their CSS as it handles basic chart styling, so for customisation we'd want to alter the colours mostly (or maybe type if it's present). that's already done. the problem with porting to Stylus is that the Sass syntax uses lists (the nth function) and there seems to be no direct way of mapping that behaviour back to Stylus (or maybe it would be possible with iteration). Stylus is more limited so it might not be, would have to investigate more, so far I used Sass to Stylus and it failed on that statement too.

Hmmm, so two questions come from this,

Given we are going to lean on chartist heavily how much of their base CSS do we need? Just styles?

Are we going to hit a point where we are blocked by stylus due to needing Sass to work with charts?

If we realistically need Sass to get the best out of the charts I'd be happy set the item of work as move from Stylus to Sass. Thoughts?

their entire styles can be found here. I can't see us changing the main file—chartist.scss as it includes displaying graphs, SVG styling and some math. what we want to modify are only settings.scss. so answering your question—I'd say we can port everything from Stylus to Sass to have variables in settings instead of pure CSS for easier colour theming but otherwise I don't see a reason to, unless you want to really alter the layout and display of the charts.

@thefoxis Nope, if that's all that's needed I'm fine with that!


Get-DbaErrorLogin listed on commands page but apparently does not exist

Environmental information
PSVersion                      5.1.14409.1018
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1018
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
dbatools Module version:
Name    : dbatools
Path    : C:\Program Files\WindowsPowerShell\Modules\dbatools\1.0.11\dbatools.psd1
Version : 1.0.11

1033             en-US            English (United States)
SQL Server:

Microsoft SQL Server 2016 (SP2-CU2-GDR) (KB4458621) - 13.0.5201.2 (X64)   Aug 18 2018 07:38:15   Copyright (c) Microsoft Corporation  Enterprise Edition (64-bit) on Windows Server 2012 R2 Standard 6.3  (Build 9600: ) (Hypervisor)

us_english
Report

Host used

[ ] powershell.exe
[x] ISE
[ ] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[x] Still buggy in powershell.exe

Errors Received

Steps to Reproduce
Browse https://dbatools.io/commands/ find Get-DbaErrorLogin and click on it, or run "get-command Get-DbaErrorLogin" in PowerShell.

Expected Behavior
Help page or summary for valid command.

Actual Behavior
404 Function not found (web) and "get-command : The term 'Get-DbaErrorLogin' is not recognized as the name of a cmdlet..." in PowerShell.  Suspect this is a result of a find/replace operation gone wrong (command name very similar to "Get-DbaErrorLog".

indeed that is true, thank you. i have updated the page 💯

@potatoqualitee FYI, same issue for these 8 commands:
Test-DbaDbDataGeneratorConfiguration
Get-DbaInstanceRole
New-DbaInstanceRole
Remove-DbaInstanceRole
Select-DbaObject
Get-DbaInstanceRoleMember
Find-DbaUnusedIndex
Set-DbatoolsConfig

Minor correction - these two are apparently found via get-command, but the online help pages don't exist...
Select-DbaObject
Set-DbatoolsConfig

Thanks so very much, @SQLGB I am unsure how I'm going to get the cmdlet help into there. @niphlod can you offer any suggestions? It exists in Get-Help, can we smash it into the json? @ClaudioESSilva would you be able to take a look at the webpage?

ping @niphlod

Assign this to me, I'll take a look ASAP
Il mar 9 lug 2019, 11:27 Chrissy LeMaire<EMAIL_ADDRESS>ha
scritto:

ping @niphlod https://github.com/niphlod
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/5844?email_source=notifications&email_token=AAA52BYK3SK4HLSWWUCKDMLP6RKY3A5CNFSM4H4UHA5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZPV44Q#issuecomment-509566578,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA52B3ZQBZHAJYCGNXDTRLP6RKY3ANCNFSM4H4UHA5A
.

thanks niph!

Keep this up bot
Il dom 8 set 2019, 13:00 stale[bot]<EMAIL_ADDRESS>ha scritto:

This issue has been automatically marked as stale because it has not had
recent activity. If you would like this issue to remain open:

Verify that you can still reproduce the issue in the latest version
of dbatools
Comment that the issue is still reproducible and include:

What version of dbatools you reproduced the issue on
What OS and version you reproduced the issue on
What steps you followed to reproduce the issue

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/5844?email_source=notifications&email_token=AAA52B2KREAWWKQ22KGGF4TQITLLLA5CNFSM4H4UHA5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6FNG2A#issuecomment-529191784,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA52BZBJ4J3MF2BDGFAM2DQITLLLANCNFSM4H4UHA5A
.


Adds flex styles to address unintended element wrap and overlap

Adds a flex style to all the headers to prevent unintended wrapping.

Closes #565 to fix the "Scale" / "Saving" button from overlapping the scale.

:+1:


[bug] Fixup nyxx version constraint to allow using with dev nyxx

Fixup nyxx version constraint to allow using with dev nyxx.

Fixed in 3.0.0-dev.2


Add TypeScript type definition

https://github.com/wheresrhys/fetch-mock-jest/issues/7
Add TypeScript type definition.

Thanks for the contribution


Keep Battle.net open but detect if game runs

Hey,
many thanks for this great work!
I'm wondering if it's possible to keep the Battle.net Launcher open but let OSOL detect if the is closed.
I don't want to close Battle.net to keep my games updated automatically by it.
Thanks
Tom

OSOL wasn't really designed with this sort of thing in mind, but you can try setting CloseLauncher=False and ReLaunch=False inside the OriginSteamOverlayLauncher.ini found in the directory you ran OSOL from.

I tried that before but it doesn't work. Too bad. 😕


[FEAT]: Please support multiple configured LLM providers so each workspace could use LLM from different providers.

What would you like to see?
It would be great if AnythingLLM allowed the admin to configure URLs and API keys for multiple LLM providers.  For example, I would love it if one workspace used OpenAI and another workspace used a Local AI endpoint for something like Mixtral 8x 7B.   It seems that for any given AnythingLLM instance, there is only one preferred LLM provider and that setting applies across all workspaces.
Thank you!

Was also looking into this.  For my business use case, I am using ollama as the backend and I have an ollama instance running a mistral instance and another using codellama.  I would love to have a workspace dedicated to our RAG chat bot and another for code generation.

@icsy7867  Are the Ollama models running on the same server? If so, you can do that already via the workspace's settings. If the model is available, you can override the system model.
This issue is more towards connecting a totally different provider. Unless you mean you have Ollama running somewhere and the Mistral API service is another thing you want to connect, not just Mistral running on Ollama.

@icsy7867 Are the Ollama models running on the same server? If so, you can do that already via the workspace's settings. If the model is available, you can override the system model.
This issue is more towards connecting a totally different provider. Unless you mean you have Ollama running somewhere and the Mistral API service is another thing you want to connect, not just Mistral running on Ollama.

I wasn't under the impression that an ollama instance could run multiple model simultaneosly.  Currently I am running two separate docker containers.  Will give it a whirl!

You can! You would just use the ollama serve command and it will run an openAI compatible API at http://<IP_ADDRESS>:11434. Plug that into AnythingLLM and you can list what models you want to use. I believe Ollama hot loads them - obviously more models = more RAM

Yeah, in that case ollama is a single provider of multiple models, which is great.   But just to be clear, I'm looking for is the ability to configure per-workspace LLM providers so I can associate  entirely different providers (OpenAI, Anthropic, ollama, etc.) with specific workspaces within a single AnythingLLM instance.   I don't believe that AnythingLLM supports that yet?  Currently I'd have to spin up a different instance of AnythingLLM for each provider.

@sheneman Absolutely, im clear on the scope and detail of this issue being distinct providers and not just using a different model from the singular system provider which is currently live.
We are on the same page for this issue - I was just clarifying!

@timothycarambat - Awesome.  Anything LLM is absolutely wonderful by the way!

A the very least I would like to be able to assign different Token context window values for each Chat Model selected. Thank you.

Related https://github.com/Mintplex-Labs/anything-llm/issues/969


Update links

@vpeschenkov can we rename the app to be: Watch Presentation Clicker
Sent with GitHawk

@dkhamsing Yes, sure! I'll rename it a bit later. Thanks!


[FR] Generalise the use of dot syntax to include assets

First off, thanks for the nice evaluation feature. It works in principle and I can already use it productively. However, I encountered a problem where the code folder is created twice. I have the following tree:
├── src
│   ├── index.md
│   └── pages
│       ├── physics
│       │   ├── tests
│       │   │   ├── hubbard_model.md
│       │   │   └── interacting_dot.md
...

Now if I add the following code to interacting_dot.md, a bug occurs.
```julia:./code/ex1
a = 10
@show a
```

There are two seperate code folders created in /assets/ i.e.:
├── assets
│   ├── pages
│   │   ├── code
│   │   │   ├── ex1.jl
│   │   │   └── output
│   │   │       └── ex1.out
│   │   ├── physics
│   │   │   ├── tests
│   │   │   │   └── code
│   │   │   │       ├── ex1.jl
│   │   │   │       └── output
│   │   │   │           └── ex1.out
...

Off topic: It would be great to have the dot syntax i.e. ./ also when including images or linking with markdown e.g. [Important Paper](./papers/importantpaper.pdf) in interacting_dot.md resolving to /assets/pages/physics/tests/papers/importantpaper.pdf. Right now I use your suggested LaTeX commands as workaround.

Thanks a lot for the report, I can reproduce and should be able to fix this very soon.
off topic point
yes I thought about this if what you're suggesting is that this:
![](./ex1)

corresponds to

I guess I was a bit reluctant of doing this because there's an ambiguity when users write this. In another generator, the syntax above would try to look "next to" the file so something like

which is also valid.
Of course we could tell users to not use that kind of stuff and strongly encourage them to put all their assets in the /assets/, but I'd rather leave things flexible so that users can do more or less what they want (opinion is welcome here).
alternative
What I'd suggest instead is to support more "native" latex-like commands + encourage users to define their own (like what you're doing). So for instance the current \output command works pretty much in the way you expect:
\output{./code/ex1}

(this includes the stdout output of the code, so for instance for the short snippet you showed above it would give
10
)
We could likewise have a  default\fig command that does a similar thing:
\fig{./code/ex1}

which would convert to something like

Possibly \fig{}{./code/ex1} to allow specification of another name than just ex1 for the figure so that you could also use \plot{plot1}{./code/ex1} which would look for a plot1.[ext] in the relevant assets subfolder.
Thoughts?

Initial issue is fixed, it was due to a stupid mistake on my side not specifying the path to the page currently being processed at the right spot... anyway with your exact example, I now get for the assets folder:
.
├── infra
│   ├── favicon.png
│   ├── hamburger.svg
│   └── rndimg.jpg
├── pages
│   ├── code
│   │   ├── ex2.jl
│   │   ├── ex3.jl
│   │   └── output
│   │       ├── ex2.out
│   │       └── ex3.out
│   ├── exdot.jl
│   ├── output
│   │   └── exdot.out
│   └── physics
│       └── tests
│           └── code
│               ├── ex1.jl
│               └── output
│                   └── ex1.out

Please do give it a shot and let me know.
Thanks a lot for reporting.
Ps: I'm editing the title as this issue is fixed but there's this secondary issue of paths for which your opinion is welcome.

As you said, the inital issue seems to be fixed and everything is working as expected now!
The \fig{} command looks like a great idea however, I do not believe hat an additional command \plot{}{} is necessary. As far as I see it, the only benefit is that it somehow logically links the plot name to the output of a script called ex.jl in the markdown document. The \fig{} command should be more than enough to handle this by specifying the complete image name e.g. \fig{./code/output/plot1.png} or \fig{./code/output/plot1}. An additional command would be needed for linking arbitrary files into the document using the dot syntax. Something like \link{.reviews/importantreview.pdf}`, where one always has to specify the file extension.

Sounds good, thanks for the feedback! I'll close this now, see issue #172 for the \fig and \link.


Feature request - Stretch MessageBox horizontally over the whole Window

Really nice work!
Would it not be nice if the MessageBox could stretch horizontally over the whole Window?
Something like this:

By default - no. But you may set horizontal size for your custom window

Yes, you are right. But, then it looks like this, not centered.


Float::integer_decode cannot be implemented for quadruple precision floating types.

This is the definition of integer_decode:
fn integer_decode(self) -> (u64, i16, i8)

The 64 bits allocated for the mantissa is too small, 112 bits would be required to store the mantissa of a IEEE 754 quadruple-precision float. The exponent requires 15 bits, so i16 is enough, and naturally i8 is enough for the sign.
I ran into this issue while working on bindings to the GCC __float128 type (aka libquadmath). Would changing the u64 meant for the mantissa to a u128 be too much of a breaking change?  If not, I will implement it in the next couple of days.

Would changing the u64 meant for the mantissa to a u128 be too much of a breaking change?

It would also raise the minimal required Rust version from 1.8 to 1.26.

Just an idea if the interface for integer_decode is to be redesigned. It seems cleaner to me to return a (signed_mantissa, exp) rather than (mantissa, exp, sign). I personally have many cases where I need to multiply the sign back to mantissa in order for some comparison.

@cmpute I strongly disagree with that idea. There are three components to IEEE floating point numbers - the mantissa, exponent, and sign; this function gives you them.
If you need to do a multiplication, you should do it yourself rather than asking the function to do a potentially unnecessary operation. This would also make code particularly ugly in cases where the sign was key, e.g. lots of  if mantissa > 0 { 1 } else { -1 } which is much uglier than mantissa * sign.

@jkarns275 In order to get a signed mantissa, what you actually need is mantissa as i64 * sign as i64, and I personally don't find any situation where a separate sign variable is needed, because you need two or three branches for the sign anyway. But I agree with that it requires additional multiplication and it's better to comply with the standard.

This might be a bit more relevant since f16 and f128 are now unstably available. Instead of always returning u128, it may be nicer to somehow put this into an associated type. This would also allow for the transmutation functions available in std, which softfloat libraries need:
trait FloatRepr {
    type Int;
    fn to_bits(self) -> Self::Int;
    fn from_bits(Self::Int) -> Self;
    fn decode(self) -> (Self::Int, i16, i8);
}

The exponent portion would also need to be an associated type if supporting numbers larger than f128 might be desired down the line (e.g. binary256 has a 19 bit exponent). Likely not much need for this though.

An associated type makes it more difficult to deal with the decoded result generically. Instead of just T: FloatRepr, you also need T::Int: ... for all the operations you want to do with that. We could tighten that with something like type Int: PrimInt, but then it rules out something like a hypothetical BigFloat.


navidrome: 0.44.1 -> 0.45.1 and added aarch64 support

I updated the package to the 0.45.1 version and added support for aarch64.
I'm not sure this is the best way to do that.
However I tested the package in both platforms.

@ryantm I supposed it could be so, how can I write this updateScript? I looked for a tutorial or a simple example but I haven't found anything.

@ryantm I supposed it could be so, how can I write this updateScript? I looked for a tutorial or a simple example but I haven't found anything.

@aciceri I don't know of a good tutorial. You could search nixpkgs for passthru.updateScript and you will find lots of examples.


Allow overrides per language

What problem does this feature solve?
Imagine news-articles in different languages and with a translated identifier(slug).
If you switch the language it should use the identifier supplied in your component rather than using the parameters from the url.
Current locale switcher would do something like this:
/:locale/news/:slug
/en/news/article
/de/news/article
it should be:
/:locale/news/:slug
/en/news/article
/de/news/artikel

This feature request is available on Nuxt community (#c270)

Seems to be done by 'Dynamic route parameters'


Connector/refactor backend

What type of PR is this?

refactor

What this PR does / why we need it:
finished backend part of refactoring
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Does this PR introduce a user-facing change?
None

@renzholy @cosformula please check the modification of the connector router
basically the profile / profileName part has been removed (being replaced by workspaceId, since one connector will only correspond to one workspace.
listAvailableConfigs has been replaced by getProfileConfigs, to be distinguished from getIntegrationConfigs, at the same time, listProfiles is replaced by getProfile, and removed deleteProfile (since it can no longer be deleted)

@cosformula  If we support multi-profile in the future, it will be achieved by multi-connector, in current scenario


Sample distribution

I wanted to ask whether there is a way to access the distribution of samples (i.e. the assumed model distribution). Reading through https://linkedin.github.io/greykite/docs/0.1.0/html/pages/model_components/0900_uncertainty.html I gathered that Silverkite does some sampling to estimate the uncertainty intervals. Is there a way to access the samples directly, or even better a density over the possible values?
In Prophet, one can use predictive_samples to obtain the generated samples.

Hi @jonhue by default Silverkite uses normal distribution based confidence intervals and does not have sampling. If you update to the latest Greykite (0.2.0) you can see the estimated standard errors in the prediction df.

@KaixuYang sounds good, thanks for your speedy response!


removed the google analytics function from resource.html and submission.html

Fixes

Fixes #285 by @Murdock9803

Description

The pull request removes the google analytics function code from the resource.html and the submission.html files. The functionality can be re-implemented later with proper discussion.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@possumbilities Just like the work in PR #284, This PR aims at completing the work and now all the html files are free from google analytics script code.


Add View Controller Transition Concept Maps

Add View Controller Transition Concept Maps in WWDC 2013 Custom Transition Using View Controllers.

Not sure these really help, they illustrate code that isn't mentioned in the documentation.


Feat: Consumer Parameters

Consumer Parameters for Ocean Market
This PR focuses on bringing consumer parameters as defined in the docs to Ocean Market.
We are targeting the complete flow, considering publish and edit functionalities as well as handling consumption with provided user data. This PR introduces quite a few changes that I will try to summarize below.
Related to https://github.com/oceanprotocol/ocean.js/pull/1730

Proposed Changes in this PR:
1. Publish Form
Add ConsumerParameters Form sections to publish form via a new ConsumerParameters input element

Adds a checkbox to algorithm type assets on Step 1 Metadata, that when checked displays a new input component to add consumer parameters
Adds a checkbox to Step 2 Access, that when checked displays a new input component to add consumer parameters

The InputElement/ConsumerParameter component utilizes a controlled tab component to handle display of multiple parameters

New parameters can be added (and deleted), creating new tabs to switch between for respective parameter specification
Tabs currently use the parameter name as their title

For select type parameters a KeyValueInput component is used to specify the needed parameter.options (similar to Headers in url files)
Added reusable form validation for consumer parameters (publish and edit)
Added FormConsumerParameter interface that specifically handles display of consumer parameters in the publish and edit form
Input for paramter.required is a select field with Required and Optional options, trying to improve UX compared to simple checkbox
DefaultInput (input for parameter.default) changes based on the selected parameter type

text type parameters, will use a regular text input field
number type parameters, will use a number input field
boolean type parameters, will use a select input field with true and false as options
select type parameters, will use a select input field with the options generated from specified parameter.options

Add transformation of consumer parameters in transformPublishFormToDdo utility function

2. Edit Form
Add (up to) two ConsumerParameters sections to edit form

Extend metadata tab with consumerParameters for algorithms and service.consumerParameters for all assets
Uses the same component that is used in publish form, prefilled with data parsed from DDO
Add parseConsumerParameters utility function to @utils/ddo.ts to be able to parse DDO consumer parameter data for display in edit form

3. Asset Consumption
Add UserCustomparameters to consumption forms

New Input section that parses consumerParameters from DDO to display the needed input fields to get consumer input

The following parameter configs are supported (based on docs):

text type parameters, will use a regular text input field
number type parameters, will use a number input field
boolean type parameters, will use a select input field with true and false as options
select type parameters, will use a select input field with the options parsed from DDO
for select inputs that are not required an empty option is added

The tab component is used to display up to three possible sections (in compute, one section for download):

Dataset Service consumer parameters
Algorithm Service consumer parameters
Algorithm consumer parameters (algoCustomData.json)

Refactor Download.tsx to use <Formik> to be able to collect userdata
Dynamically create validation schemes based on consumerParameter definitions from DDO
Add collected user data to the provider calls on consumption

4. Misc updates / refactorings

Add ConsumerParameter, ServiceExtended and MetadataExtended types until they are released in ocean.js (https://github.com/oceanprotocol/ocean.js/pull/1730)
Refactor InputeElement/Headers to InputeElement/KeyValueInput to be reusable for similar purposes, e.g., the select options for consumer parameters in this PR
Refactor error handling in FormInput/index.tsx with new utility helper function to get object's properties by given string path (helper function also used for consumer parameter form sections)
Add option to make atoms/Tabs component controllable by exposing selectedIndex and onSelect from ReactTabs

Hi there 👋🏽,
Hope you're doing well. I'll be reviewing your PR in these days, sorry if I'm not commenting everything at once.

We should limit the fields values to avoid this:

We should display the description of the fields when consuming:

There's a weird flickering when pressing the "Get" / "Retry" button:

https://github.com/oceanprotocol/market/assets/13741335/141338ca-a192-4cfa-83df-82e7d1abad41
I'll keep adding stuff this week.
but Awesome job!

Hi @moritzkirstein, thanks a lot for your work on this. Can you share examples of datasets and algorithms that you recommend testing it with?

Hello @jamiehewitt15, thanks for the feedback.
You can have a look at did:op:e0990acb412bc8c3d05250154b246d74f2ac970889e6c1d62f8cb1a18ee1a75a for an asset of access type and did:op:ede97064ce1651b257aad3350c3e6495acaa61d42322cc559e0fe0ccf03e17dc for a ctd asset selecting did:op:24cdd3f99c61e9e799477a0683037ccadb85649bab33af60a46be71d86ce4280 as algo.
I'll fix the other reported issues over the next few days.

Hi there 👋🏽,
Hope you're doing well. I'll be reviewing your PR in these days, sorry if I'm not commenting everything at once.

We should limit the fields values to avoid this:

* We should display the description of the fields when consuming:

* There's a weird flickering when pressing the "Get" / "Retry" button:
custom.parameter.deployment.-.Ocean.Market.-.26.June.2023.mp4
I'll keep adding stuff this week.
but Awesome job!

@EnzoVezzaro I added some chars limit for name and label (min 4, max 50) and description (min 10, max 500), let me know if the numbers are fine or if you want to change them.
As for showing the description for the different fields, is it ok to put them as help text to keep the form compact?

As for showing the description for the different fields, is it ok to put them as help text to keep the form compact?

Hi @LucaMilanese90 absolutely, it's just fine 👍🏽
As for the limit in name I'd add ellipses on the tab headers once it reaches a about 15 characters, to avoid having this crazy tab header over here.

the rest seems fine. If @jamiehewitt15 doesn't have anything else to add, after these little changes we should be ready to go.

@LucaMilanese90 can you do a rebase pls, so we can have the latest fixes from main available in the PR ?

Thanks a lot for your work on this @LucaMilanese90 @moritzkirstein. It would be great if you can also test it in production now that it has been deployed, thank you


Deleting amount from local currency box does not clear my currency boxes

To reproduce, type 15 in the local currency box. Then delete the 5, and see that the my currency boxes change to display the amounts appropriate for a local currency value of 1, as they should since 1 is all that remains in the local currency box. Then delete the from the local currency box 1, and notice that the my currency boxes still display the amounts for the value of 1. They should be cleared instead.

Confirmed; Thanks for the report!

Looks like this has now been fixed.


Add well-known contract -- Su Squares

Hello, can you please add a well-known contract to MyEtherWallet.
Su Squares
Mainnet / 0x6731560e455537c9f088EA02A47a0ECFa28a9231
ABI: https://etherscan.io/address/0x6731560e455537c9f088ea02a47a0ecfa28a9231
This is an ERC-721 contract. You can confirm this is the correct address by seeing:

https://github.com/ethereum/EIPs/blob/master/EIPS/eip-721.md
Which points to Su Squares. https://tenthousandsu.com
Which has a white paper showing the above address and also susquares.eth

I am the lead author of the ERC-721 standard. This standard is in the ACCEPTED stage of the EIP process.

The Explore 721 conference is happening this weekend. It will be very nice to have this standards-compliant ERC-721 contract in MEW by then if possible.

Moved to: #554
Please check back when a release is made to see the contract on the website.

Excellent, thank you, much appreciated.


Some general Advice please - and happy to pay for4 it, too!

Hey team, this looks exactly like something we would want to use on our website.
However, both our webdevs (Wordpress) and us have a bunch of questions that seem beyond the scope of documentation.
Apologies if some of these questions are a case of RTFM, but none of us are experts at this.

If we spin up the cloud services in our Azure tenant as described, do our webdevs still need to code C#? If I understand correctly, their job is "only" to embed the JS client component - is that correct? Can you be more specific as to what the webdevs need to do on the (WordPress) site? What instructions exactly do I need to give them? Bear in mind that most webevs do not know C#.

How are new chats isolated from each other? As in: do all chats create a new Teams Chat? If yes, how are they identified? If they all go in to the same chat, how can we identify each individual exchange?

Do you have any experience as to how much the Azure services cost? Per chat, per message or whatever?

Would any one on the team be willing to get on a call with us and/or demo what this looks like?

Thanks!

Friendly BUMP


Work Package wp-cx.csv

I would like to work on this please!

Results of Phase 1
wp-cx-results.csv

I completed this work package.

Results of Phase 1
wp-cx.csv
:

I completed this work package.

I would like to work on this please!

Phase 1 Results
wp-cx- results.csv
:

I completed this work package.

I would like to work on this please!

I would like to work on this please!

I would like to work on this please!

Results of Phase 1:
wp-cx-results.csv

I would like to work on this please!

I would like to work on this please!

Results of Phase 1: wp-cx- results.csv


Updating README

I was migrating my project to Swift 4 and since $ is not an identifier anymore I added the backticks as described on the documentation and still getting the same error.
I searched through the issues/PRs and I found out it has been renamed to Dollar but seems that this part of the documentation wasn't updated.

Thanks


change wiki to test

For testing purpose

@jiaqima The final outcome of auto commenting can be observed at the last two commits.

Close this testing PR


@storybook/addon-storysource is missing on npm

yarn add -D @storybook/addon-storysource

It's a new addon and will be released soon with v3.4

Oh! Can you release it with rest of 3.4 alpha addons ? Maybe? 😎 No? Ok 😁

@shilman, when will be the next release?

Released as v3.4.0-alpha.9


Hardcoded Domains

This page has many references to domains like appserviceenvironment.net without a note that this will only work in the public cloud. Should some sort of note be added that the domain suffix will vary if a sovereign cloud is being used?
Thanks!

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

ID: 62d67fc0-2fe8-7af2-2989-8be5d6b598b6
Version Independent ID: 131aa416-5156-c0b6-783e-32144faf9936
Content: App Service Environment networking - Azure App Service Environment
Content Source: articles/app-service/environment/networking.md
Service: app-service
GitHub Login: @madsd
Microsoft Alias: madsd

@matferrari-msft thanks for your feedback. We will review and update this doc as needed.

@madsd can you please confirm this?

I am adding a note today. Thanks.
#please-close


Replace the https://git.io link

https://git.io/ stopped working, so the link needs to be replaced with https://github.com/felladrin/PickBetterPack

Resolved in https://github.com/felladrin/PickBetterPack/commit/9253801bc55e56193f305336c5dcefd308554801.


[Dart] Add missing output @JsonKey output for @freezed option

Dart language enhancements:

Resolves missing @JsonKey() for --use-freezed option https://github.com/quicktype/quicktype/issues/2162 https://github.com/quicktype/quicktype/issues/2339
Consructors generated with --use-json-annotation or --use-freezed will respect the required keyword even if the type is considered nullable (we forgot to check if the prop was not optional!).

!prop.isOptional checks if the var has been explicitly defined as non-nullable

Fixes --use-freezed option always assuming everything is nullable and appending ? to the end of the type -- this should now be covered by the required check, plus the ? was already factored in by the Dart prop type to begin with.
Suppress comment tutorial on how to use the generated functions, when --coders-in-class is being used, since these top-level functions do not apply anymore
Remove json_annotation import if --use-freezed is being used, since the freeze annotations import already provides json_annotation annotations.

Excerpts:
./script/quicktype --src-lang schema generated_freezed.json -l dart --use-json-annotation --use-freezed --coders-in-class
generated_freezed.json
{
    "$id": "datamodel",
    "$schema": "https://json-schema.org/draft-07/schema#",
    "$ref": "#/definitions/DocumentType",
    "definitions": {
        "DocumentType": {
            "title": "DocumentType",
            "type": "object",
            "required": ["time_created"],
            "properties": {
                "time_created": {
                    "type": ["string", "null"],
                    "format": "date-time"
                }
            }
        }
    }
}

import 'package:freezed_annotation/freezed_annotation.dart';
import 'dart:convert';

part 'document_type.freezed.dart';
part 'document_type.g.dart';

@freezed
class DocumentType with _$DocumentType {
    const factory DocumentType({
        @JsonKey(name: "time_created")
        required DateTime? timeCreated,
    }) = _DocumentType;

    factory DocumentType.fromJson(Map<String, dynamic> json) => _$DocumentTypeFromJson(json);
}

Notice that time_created is nullable, but also specified as required in the JSON Schema, and this is properly reflected in the generated code.

Hi @dvdsgl !
May I kindly also request for this to be reviewed and for a new build to be released so that we may use these fixes in production without having to run our local fork?
Thank you!

It should have been released immediately.

Thanks! Let me know if there's any changes needed for this PR to be merged.

Hi @dvdsgl , apologies for the ping again but would you have some time to review this PR for this to get merged?
Thank you!

Thank you!


Add actionhero.js

Added actionhero.js according to alphabetical order in Back-end Development.

Please read the instructions for creating a new list, contributing guidelines and the awesome manifest.
Especially this part:

Wait at least a couple of weeks after creating a list before submitting to give it a chance to mature.

Ping me here in a few weeks and I'll reopen ;)

The list is very thin on content. That could either mean there aren't enough quality content for the framework or you haven't worked on it enough. Either way, that needs to be improved before being accepted.


props are undefined

I'm trying cosmos and I'm having the following problem:
Uncaught TypeError: Cannot read property '...' of undefined

Where "..." is always one of the props of the components I'm trying to render (I've tried a couple).
Some info:

I'm using components with the es6 syntax.
cosmos-js: ^0.6.6

✗ node -v
v0.10.36

✗ npm ls react
<EMAIL_ADDRESS>/Users/jm/p/ui-components
└──<EMAIL_ADDRESS>

Apparently this is caused by using es6's get. Yo can try with:
import React from 'react';

export default class C extends React.Component {
  get a() {
     return "get is the problem";
  }

  render() {
    return (
{ this.a }
);
  }
}

When I compile this component to es5, I get the problem above. If I change it to the following it works fine.
import React from 'react';

export default class C extends React.Component {
  a() {
     return "get is the problem";
  }

  render() {
    return (
{ this.a() }
);
  }
}

Therefore, I think it's a bug and should be fixed, since I don't wanna remove the [cool] gets in my code. WDYT?

I tried adding a getter to the http://github.com/skidding/cosmos-example repo to see if your error reproduces. Indeed, it crashes:
Uncaught TypeError: Cannot set property a of #<SimpleButton> which has only a getter

But if I'm not mistaken, it's triggered by https://github.com/gaearon/react-hot-loader. See https://github.com/gaearon/react-hot-loader/issues/131

One quick fix is to create a corresponding setter, but it might still have side effects.
Until the issue gets fixed, you can use a react-hot-loader fork. You can follow instructions here and then install cosmos-js again in your project. It worked for me.

We're going to support getters real soon, I promise.
Give us a few more days.

:+1:

Cosmos no longer uses react-hot-loader. It now extends react-transform-boilerplate. Check out the new docs: https://github.com/skidding/cosmos/tree/master/webpack-boilerplate/react-0.13


[New FAQ entry] Compatibility to other MODs?

Hey there,
I recently found your MOD via the beamdog forum entry.
I regularly re-play Baldur's Gate 2 EE, every some months or so, say, every
6-9 months or so - a decent delay to wait in the event that people clean up
issues or new content is released. So trying your MOD would make sense.
Unfortunately I also ran into some odd bugs when mods make changes
that are incompatible or semi-compatible. Would it be possible for the
FAQ to include a question like this:
"Is the NPC Dusk MOD known to be compatible to every other MOD?"
Now, you don't have to answer that question since it is a big one, BUT
I think it would be useful to:
a) state that there is no compatibility issue, when there is none found
so far
and
b) IF an incompatibility is found, to mention it on the main README
I think this would be useful for people who install many mods. Usually
the smaller NPCs mods rarely have problems, it's more the big mods
that make tons of changes that create issues. At any rate thank you for
reading this issue request, and please do feel free to close it at any moment
in time for any reason.

Hi!
Yes, it's good advice to leave it as a fixed item in the FAQ section. There were compatibility issues, which should have been resolved in v 1.0.0. Although, to tell the truth, we didn't specify which ones had been fixed (for now, only one). In the next release, we will include it in the main readme and leave it in the changelog. Also, we will leave this issue open in case someone finds any incompatibility.
Thank you!


test PR with Github API

Please pull this in!

Docs Build status updates of commit ce3876c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

sdp-test/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 85e16e8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

sdp-test/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 2245e73:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

sdp-test/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 33aacd3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

sdp-test/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.