Update pre commit config

Please briefly answer these questions:

forbidden to commit directly to the main or master branch
run unit tests before every commit

what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)

what changes does this pull request make?

are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

Can we check the DCO signature and the commit message format in pre commit

@themanforfree， the pre-commit hook is executed before committing. We don't have any commit messages yet at that time, so we can not check DCO and validate commit messages at the stage of pre-commit.


ios9 keyboard issues

Short description of the problem:
When i click on an input field in my app, I get two unexpected behaviour.

The page becomes blank and only the background image is shown on the page
The keyboard pops up but nothing is seen. see pic that only the Done button and the Back and Next button is shown but the keyboard doesnt show at all.

What behavior are you expecting?
Expecting it to show the keyboard when input in on focus and to close when input focus is lost.
config.xml
<preference name="KeyboardShrinksView" value="false" />
<preference name="KeyboardDisplayRequiresUserAction" value="true" />
<feature name="Keyboard">
      <param name="ios-package" value="IonicKeyboard" onload="true" />
  </feature>

in my app.js
if(window.cordova && window.cordova.plugins.Keyboard) {
      cordova.plugins.Keyboard.disableScroll(true);
}

in my view i am using ion-content like this
<ion-content  class="adj-13" style="padding: 15px; overflow-y:hidden;">
</ion-content>

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Which Ionic Version? 1.x or 2.x
1.7.14
Plunker that shows an example of your issue
For Ionic 1 issues - http://plnkr.co/edit/jnOrQRtE8rN2WCXDQYQY?p=info
For Ionic 2 issues - http://plnkr.co/edit/sSFR9jbfTsxgAmrm9hUR?p=info
Run ionic info from terminal/cmd prompt: (paste output below)
Your system information:
Cordova CLI: 6.1.1
Ionic Version: 1.1.0
Ionic CLI Version: 1.7.14
Ionic App Lib Version: 0.7.0
ios-deploy version: Not installed
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v4.4.2
Xcode version: Xcode 7.3 Build version 7D175

Are you able to provide a plunkr to reproduce this behaviour?
The keyboard won't be displayed on the emulator. Try on a real device.

tried on a real device and it works fine so far. Thanks for the suggestion @dylanvdmerwe


Add the "_" param to the request, Denon seems to use it for some sort…

… of cache busting on my X1200w.  Without it I rarely see the list of inputs returned to the device handler.

Thank you!


Update README with further guidelines

We should provide guidelines for people that need to open issues.

Hey @LostCrew here we go https://github.com/nostalgiaz/bootstrap-switch/commit/f1c53cbca9d2047767274fcacdfef9f1eaccf064 . Can you double check this commit?


Custom Trigger/Binding Serialization in Isolated worker

Not sure if this belongs here or in the Functions Host repo.
This applies to v4 Isolated Worker Functions
Languages: C#
In our organization, we have developed a Custom Trigger/Output Bindings extension for one of the major message brokers that don't currently have support for Azure Functions out of the box, that allows us to use the functions to send/receive to their queues/topics. We have being using this extension successfully for the last couple of years.
One of the major features of the extension is to deserialize/serialize the payload base on custom types provided by the function's method signature. We currently support sending/receiving string, byte[] and POCO classes. The extension provides JSON serialization by default, but it's replaceable by implementing an interface and registering the instance with DI. We have cases where serialization is AVRO, for example.
In the hosted model, everything works as expected, because the serializer that's registered with the custom trigger is in the same process, and has access to the runtime type specified by the function signature. In the Isolated model, this is not the case since the custom trigger would run on the host and the function signature would be on another process.
We would like for the custom trigger to support the Isolated model, but we've found no way to go forward with this, the trigger in the host only receives either a byte[] or a JSON serialized representation of the object.
For reference I'm including a pseudo-code snippet of how a function signature would look, the Message class is defined in the extension, CustomPoco is defined in the Isolated Functions project:
[Function("FunctionName")]
[CustomBindingOutput]
public Message<CustomPoco> MyFunctionName([CustomTrigger(Queue = "my-queue")] CustomPoco payload)
{
  ...
  return new Message<CustomPoco>("destination-topic-or-queue", payload);
}

The extension uses whichever serializer is registered in the DI container with interfaces like this:
public interface IInputSerializer
{
  T ReadObject<T>(byte[] input);
}

and
public interface IOutputSerializer
{
  byte[] WriteObject<T>(T input);
}

That allows us to provide a default serializer but also let the consumer register their own if needed. We also access other properties of the Message class for headers, etc.
I would like to understand what would be the path forward to support this type of scenario for developing custom extensions, for either triggers and output bindings.

@jponcedeleon-kr this looks like it can be accomplished by implementing an IInputConverter and adding an attribute for it on your CustomTriggerAttribute. You will have full control of binding the input payload to the parameter in your IInputConverter implementation.
[InputConverter(typeof(CustomerInputConverter))]
[ConverterFallbackBehavior(ConverterFallbackBehavior.Default)]
public class CustomTriggerAttribute : TriggerBindingAttribute
{
}

@jponcedeleon-kr are you using a binding output or is this just the return value of the function you want to have custom serialization for?
If it is the former, the general advice for dotnet isolated is to favor using an input binding which is a complex type that will perform your output binding. IE: for blob, we would have an input binding for the BlobClient and then the function can perform output by writing to that client.
If it is the later, you can customer return serialization one of two ways:

Customize JsonSerializerOptions for your expected return type.
Insert a middleware which reads the existing function invocation result and updates it. Here is an example: https://github.com/Azure/azure-functions-durable-extension/blob/dev/src/Worker.Extensions.DurableTask/DurableTaskFunctionsMiddleware.cs#L37

Let me see if I can explain better, I understand it's difficult without examples, bear with me.
We do have a custom extension for the functions hosted model that implements input/output trigger/bindings for a message broker, similar to the ServiceBus extension.
In that model, for sending messages to the broker (output) we can use the IAsyncCollector interface or the function's return value, they both call into the AddAsync implementation method of the binding. In that method is where serialization is done, and the consumer doesn't need to worry about serialization inside the function call, and can be customized depending on what's registered in DI with a specific interface.
In the hosted model this is possible because the extension is loaded into the same process as the host, and thus it has access to the runtime type provided in the function's method signature.
We are in the process of trying to support the Isolated model, there's no IAsyncCollector anymore, and the pattern is to use the return value to send the message to the broker. We would like to also have the ability to serialize there in the extension, so the consumer wouldn't need to worry about that, just provide (return) an instance of its own custom POCO class, like we do in the hosted model.
The example in my first post is trying to show that the function returns an instance of a POCO class, that would need to be serialized by the extension before sending the message to the broker's queue. It would be nice to have a mechanism similar to the InputConverter, but for output, with a matching interface to IInputConverter, that gets invoked after the function returns and before the output is sent to the host process. In the example above, the returned instance of CustomPoco would need to be serialized by the extension.
The functions middleware doesn't seem the right place to hook up and do any type of transformation, but I'm going to look into that to see if that's a plausible solution.
Thanks for all the help, appreciated.

@jponcedeleon-kr I understand the scenario, and the routes I suggested are still what I recommend, but I can expand upon it:

All function return values are already being serialized by WorkerOptions.Serializer before being sent to the host, so you can customize serialization there. By default it uses System.Text.Json, and you can customize it by IServiceCollection.Configure<JsonSerializationOptions>(o => ...)
IAsyncCollector is not present because routing complex output through the host is inefficient. Instead, you would consider where the final destination of this output is to be and provide an InputBinding which is a type of your own design that has the AddAsync methods on it. Then instead of sending to the host at all you can just send it to its final destination. This is assuming the final destination is something accessible from the worker.


Update Footer.js

updating Facebook linked file to GitHub repository
link only changed

please update it to
https://www.facebook.com/devalopers.fb/


How to reference Azure SDK dlls in project.json?

The official Azure SDK assemblies are not available via NuGet.
Under aspnet-core Web API project, which is configured to run as a worker role on Azure, I have used "Bin syntax" (https://github.com/aspnet/Home/wiki/Project.json-file#bin-syntax-wrapping-a-dll) in project.json as follow:
"frameworks": {
  "dnx451": {
    "dependencies": {
        " /* eradicated for brevity */ "
    },
    "bin": {
      "assembly": "C:\\Program Files\\Microsoft SDKs\\Azure\\.NET SDK\\v2.8\\ref\\Microsoft.WindowsAzure.ServiceRuntime.dll"
    }
  }
},

In WorkerRole.cs, I am getting Cannot resolve symbol 'ServiceRuntime' at using Microsoft.WindowsAzure.ServiceRuntime.dll.

Is this usage flawed?
In cases such as these, when NuGet package is not an option provided by vendor, can we add assembly in source tree and reference it with relative path in project.json?
Is there another (better, special) way to handle Azure SDK assemblies?

Found the solution:
I created a Class Library (package) project (AzureSDKServiceRuntimeWrapper), which wraps the Microsoft.WindowsAzure.ServiceRuntime.dll, added a project.json with following content:
{
  "frameworks": {
    "dnx451": {
      "bin": {
        "assembly": "C:\\Program Files\\Microsoft SDKs\\Azure\\.NET SDK\\v2.8\\ref\\Microsoft.WindowsAzure.ServiceRuntime.dll"
      }
    }
  }
}

Then referenced this project in Web API project's project.json:
"frameworks": {
  "dnx451": {
    "dependencies": {
      "AzureSDKServiceRuntimeWrapper": "1.*"
      ".. other dependencies.."
    }
  }
}

Finally, worked on postbuild event to copy the API project assembly in two places: from "artifacts/bin/{projectname}/{configuration}/{tfm}/{projectname}.dll" to "artifacts/bin/{projectname}/{configuration}/" and "artifacts/obj/{projectname}/{configuration}" (the latter one is required by worker role's .targets file provided by v2.8 SDK). The worker role application is now packagable and hence publishable (although facing some further hiccups once package gets restored on Azure; i.e. worker role is continuously rebooting, but I will manage to investigate it further).
Would be nice to clarify the wrapper project aspect in docs. I would happy to send a (reviewable) PR if there is a repo with docs in the source tree. :)


Login screen lacks link to Virtualmin Password Recovery

Virtualmin Password Recovery allows users to reset or recover their password if they forgot it.
This feature is optional, but if activated, it shows up a link on the login screen.
However, if using Authentic as default theme, the link does not show.
The URL is something like https://foo.bar:10000/virtualmin-password-recovery/
Would be nice if Authentic could support that :)

Hmmm.. I will try adding it to the version 12. But you would have to wait for some time! I'm covered in some other work from head to toes! ;)

Could you please add a screenshot of how it looks in default theme? By the way, you said it's optional? Does this feature automatically activated when module virtualmin-password-recovery is installed or it was to be enabled somewhere?

Alright, figured this out. It will be added to version 11.10 shortly.

Sorry for not getting back to you sooner, but glad you figured it out already :)


[MRG] Add support for event_type=3 when reading CNT data

.cnt files used by NeuroScan systems support different ways of encoding events (perhaps these reflect old and new versions of the file format). MNE currently only supports types 1 and 2. This PR adds supports for the newer type 3 events.
Closes #3882

Closes #3882
:D

Problem is: I don't have an easy means of generating a small test CNT file with these type of events.

Problem is: I don't have an easy means of generating a small test CNT file with these type of events.

Me neither.
But the code is exactly what I did for my own file.

But the code is exactly what I did for my own file.

actually, could you double verify that for me? Do you have a test file yourself for which you know the proper event indices? I'm currently checking this against eeglab but I might get it wrong.

(seems to check out with eeglab on my data)

Current coverage is 85.58% (diff: 40.00%)

Merging #3911 into master will decrease coverage by 0.01%

@@             master      #3911   diff @@
==========================================
  Files           347        349     +2
  Lines         61934      61975    +41
  Methods           0          0
  Messages          0          0
  Branches       9494       9501     +7
==========================================
+ Hits          53016      53040    +24
- Misses         6255       6269    +14
- Partials       2663       2666     +3

Powered by Codecov. Last update 953764b...852f501

@jona-sassenhagen feel free to test this version with any CNT file you may have laying around. It should give exactly the same output as eeglab.

Tried it on my own files, works perfectly.

I think this is ready to merge. I don't believe the travis failures are related to this PR.

LGTM

+1 for merge when Travis is happy

cov plot is breaking things again. I'll try to look soon. In the meantime the CIs are happy otherwise.

Thanks @wmvanvliet

Thanks @wmvanvliet !

I'm looking at this thread, does anyone have testing data with event_type 2 and 3 ?
cross reference #5493


Update 142.1-rubiks-cube.md

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

Link to repo: https://github.com/AbhinavSharma24/3D-Rubiks_Cube


[Bug]: Add comments to .bal files without attaching it to a token.

Description
Currently, when adding comments to a .bal file, we have to attach it as a comment minutiae of the token immediately following the comment.  For example consider the following code,

// AUTO-GENERATED FILE. DO NOT MODIFY.

// This file is an auto-generated script by Ballerina.
// It should not be modified by hand.
import ballerina/time;

When adding the comment to the file, it has to be attached with import token
Is there any alternative way to plug the comment to a syntax tree without attaching it to the token of the subsequent line?
Steps to Reproduce
We have used the mentioned method to add comments at the following line.
https://github.com/sahanHe/persist-tools/blob/6a7579c681ebbc4480f0ac1305335c7dd5de9571/persist-cli/src/main/java/io/ballerina/persist/nodegenerator/BalSyntaxTreeGenerator.java#L1094
Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Code can be updated to pass leading minuate to the getImportDeclarationNode() function.


[ELWSS] Sub Striker - MBBSEmu crashes around 10 seconds after loading MBBSEmu

Module Information

Identifier: ELWSS
Module Name: Sub Striker
Version: 1.03b

Describe the bug
MBBSEmu crashes after 10 seconds (only module loaded)
To Reproduce
Steps to reproduce the behavior:

Add module to moduleConfig.json
Load MBBSEmu
Wait around 10 seconds
See error

Expected behavior
Module loads and able to connect to BBS
Screenshots
Unhandled exception. System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'Unable to locate FFFF:02E1')
   at MBBSEmu.Memory.MemoryCore.GetWord(UInt16 segment, UInt16 offset)
   at MBBSEmu.HostProcess.ExecutionUnits.ExecutionUnit.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.Module.MbbsModule.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassSetState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.HostProcess.MbbsHost.WorkerThread()
   at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ThreadHelper.ThreadStart()

Software Information:

OS: Windows 2019 Server Standard
Telnet Client: NetRunner
MBBSEmu: Build #100220-5

This is a bug -- and my bad. _8087 was implemented in #149 but it was implemented as a method, when it should have been implemented as a value. We need to change this to have a declared memory address with the value 3 set at the address.


DXCDT-284: Remove dead env vars

🔧 Changes
Removing references of AUTH0_CLI_REUSE_CONFIG, AUTH0_CLI_OVERWRITE and AUTH0_CLI_DOCS_PATH in the codebase. They are relics from earlier iterations of the Auth0 CLI and have no other practical purpose.
This removal serves to reduce the noise and eliminate potential confusion.
🔬 Testing
Able to run doc generation script, code compiles, otherwise no functional changes.
📝 Checklist

[x] All new/changed/fixed functionality is covered by tests (or N/A)
[x] I have added documentation for all new/changed functionality (or N/A)

Should we target the v1 branch instead?

@sergiught because there are no true breaking changes here, I feel comfortable targeting main.


How to set the parameters of reproducing WN18RR

Hello, I am very interested in the method mentioned in your paper. Can you tell me all the parameters of reproducing WN18RR in detail? Looking forward to your reply, thank you!

Hi, we've mentioned the parameters for the different datasets in the README for the project. For WN18RR you can use this command - python interacte.py --data WN18RR --batch 256 --train_strategy one_to_n --feat_drop 0.2 --hid_drop 0.3 --perm 4 --ker_sz 11 --lr 0.001


Testnet support

Think about what a deploy testnet GitHub action might look like and what it would do.

ref #105 (public, hetzner cloud) and #107 (private self hosted, which can still use hetzner) both done.
Closing


Add e2e tests for uc_and_membership program

Right now, verify (non)membership is only tested in solidity through fixtures. It'd be nice to test this e2e as well

Should add this after #66 since we can test with non-ibc paths


request: start and end arrows.

Icons for indicating movement to the end of a list.
If this will be merged I can create android-arrow-end and android-arrow-start svg files.

Can you visually show me examples of these icons from Android's MD icon pack?

something like this:
http://www.freepik.com/free-icon/navigate-to-end-right-arrow-symbol-with-vertical-bar_723615.htm

Icons like this will be in the next version of ionicons (3.0). Stay tuned!

For sure!

check the 3.0 branch. the skip-forward and skip-backward icons are what we're going with for this.

Ok, so I was trying to use this for pagination. As the skip-forward and skip-backward icons look completely different from any other arrow this does not really resolve that issue.
Not pressing at all, but I am sure I am not the only one that will encounter this

It's close enough for this pack. We are trying to reduce the file size of
Ionicons 3.0, so adding the slightly different version is off the table.
Thanks!
On Mon, Jan 11, 2016 at 3:47 PM, Jan Wirth<EMAIL_ADDRESS>wrote:

Ok, so I was trying to use this for pagination. As the skip-forward and
skip-backward icons look completely different from any other arrow this
does not really resolve that issue.
Not pressing at all, but I am sure I am not the only one that will
encounter this
—
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ionicons/issues/325#issuecomment-170700920.

--
Ben Sperry
Co-founder, Ionic.
Twitter @benjsperry • Skype ben.sperry • Phone<PHONE_NUMBER>


And check conservation of water also without PrognosticSurfaceTemperature

And check conservation of water also without PrognosticSurfaceTemperature

The issue with using t=0 is that water_total will be zero for most of the cases (where the initial condition is dry).

The issue with using t=0 is that water_total will be zero for most of the cases (where the initial condition is dry).

Okay, it makes sense.

The issue with using t=0 is that water_total will be zero for most of the cases (where the initial condition is dry).

I rolled back to checking with end (even if I think we should not do that), and singled out the change to check water conservation also without PrognosticSurfaceTemperature.

Actually I want to output water conservation check without PrognosticSurfaceTemperature as well. I'll push to this PR.


Refactor duplicated code block in bottomup partition rdo

@lu-zero , found one more block in rdo_partition_decision(), which is used by topdown partition rdo.

Coverage decreased (-0.01%) to 83.977% when pulling 560fe8aaae41339aeb2936858ec4b8678bc9c892 on ycho:refactor_bottomup_0 into de2daec7cfd48e5a38ecd0dd49d1aabce02f3da3 on xiph:master.

There is no chance of coding efficiency with this PR.
AWCY link


oh-my-zsh.sh no such file or directoy

System information
OS: macOS Ventura 13.2
arch: arm64
Package Manager: Homebrew 3.6.20
zsh: 5.9 (arm-apple--darwin-22.1.0)
Bug description
Error message prompt: /Users/username/.zshrc:source:30: no such file or directory: /Users/username/.oh-my-zsh/oh-my-zsh.sh
Steps to reproduce

./bootstrap.sh all
answer all questions with 'Y'
Restart terminal

Expected behavior
No error messages at all.
Logs, Screenshots, Captures, etc
No response

It looks like oh-my-zsh was either not installed at all or not installed properly. Can you confirm this? Were there any errors when running the bootstrap script?
I'm currently not aware of anything in that regard, and I can't reproduce it.

oh-my-zsh was not installed at all. I can reproduce this issue. Let me know if you need further information to analyze this behavior.

Can you test it again with the latest version @bjoerntorben?

It works again under Linux.

I retried the installation and the original error message is gone. Instead i got three new messages:

[ERROR]: When using instant prompt, Powerlevel10k must be loaded before the first prompt.

You can:
  - Recommended: Change the way Powerlevel10k is loaded from ~/.zshrc.
    See https://github.com/romkatv/powerlevel10k/blob/master/README.md#installation.
    * You will not see this error message again.
    * Zsh will start quickly.
  - Disable instant prompt either by running p10k configure or by manually
    defining the following parameter:
      typeset -g POWERLEVEL9K_INSTANT_PROMPT=off
    * You will not see this error message again.
    * Zsh will start slowly.
  - Do nothing.
    * You will see this error message every time you start zsh.
    * Zsh will start slowly.

    I added the following entry manually to .zshrc, to get rid of this message:
    echo "typeset -g POWERLEVEL9K_INSTANT_PROMPT=quiet" >> .zshrc

[oh-my-zsh] nvm installation cannot be found

[oh-my-zsh] set NVM_DIR to your installation

Just to clarify, the system is a freshly installed macOS 13.4.1 with the following configuration:

brew (git, to clone this repo)
xcode-select install
iTerm
Meslo Fonts

Do you need additional informations @MoonLiightz?


Fix small typos in README.md

Description
Only some small typos in the README.md
Hi there, while reading through the README.md file, I saw some little spelling mistakes.
This is really not very important, but I would like to fix them.
I really want to participate in this project and mabye this would be a good start.
Domain
Design
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of bhai-lang.

Raise a PR.

@aniketsingh0104, @lostinwebdev . 👉  here you go with a PR and few more additions. ✅
Let me know how it goes. https://github.com/DulLabs/bhai-lang/pull/231 ✌


Allow rem and em units to be used with evalute math

In addition to supporting math operations for values with fixed units (e.g., 3in / 4), also support math operations for values with relative units (e.g, 3em / 4) as well.

One thing we'll need to decide is where the unit belongs or is permitted. With a relatively small change, we can support:
3 / 4em

That's because 3/4 is evaluated first, then "em" is tacked on. With a bit more effort, we could support the units on the first number.
3em / 4

The latter is consistent with CSS's calc function, which states the the divisor must be a unitless number. However, this does require some changes to the parser to support. If possible, I'd like to at least support the form where the unit comes after the operation (e.g., 3 / 4em).


Set Line Endings improvement

I have a text file created by Geany on Linux that uses Linux line endings. I want to change it to use Windows line endings. I selected Windows line endings. But nothing happened. I realized I need to type something into the document, save it (and then delete the dummy text I have typed and save again), for it to actually change the line endings from Linux to Windows.
With Geany, when you select the new line endings, it will update the document immediately. I can understand the decision of ecode because Geany's behavior is bad for large text files. But instead of having me use the trick I described, ecode should ask me if I want to change to the new line endings and let me choose. It could be more clever, that is, only ask when the new line endings are different from the current ones.

Download
https://www.mediafire.com/file/7ybpxn8lil1av70/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Makes sense, it should give some feedback to the user, i'll think about how to solve it. I don't like saving the file, and I don't like showing too many message boxes.

Makes sense, it should give some feedback to the user, i'll think about how to solve it. I don't like saving the file, and I don't like showing too many message boxes.

You could copy Geany's behavior. Geany updates the document immediately when you choose new line endings but it doesn't save the file. It marks the document as modified and let the user decided to save it or not. ecode can't handle even a 10mb json file (#311), so you could drop the consideration for performance when handling large text files.

You could copy Geany's behavior. Geany updates the document immediately when you choose new line endings but it doesn't save the file. It marks the document as modified and let the user decided to save it or not.

I'll do that.

ecode can't handle even a 10mb json file (https://github.com/SpartanJ/ecode/issues/311), so you could drop the consideration for performance when handling large text files.

ecode can handle perfectly fine 10mb json files, what it does not handle gracefully is huge single line files, which aren't usually expected to be used for editing, so I don't care about it.
Example, handling +25mb files, including a single line one. And I can't show you other editors because they take too long to even open and won't do highlighting (not even vscode). So I'm happy enough with current ecode's capabilities. I closed the issue because he didn't share the file and cannot reproduce. If you can reproduce the issue please share the file that triggers it for you.
https://github.com/user-attachments/assets/0cbb13ea-7548-495a-bcba-29715fae9678

@SpartanJ Try to stringify these JSONs and open them with ecode to see if ecode can handle it. My JSON file is indeed a bookmarks backup generated by Firefox. Imagine you have a bunch of obsolete domains that you want to use the editor's find and replace feature to replace with new domains. I don't think for such a simple task I need to use a real programming language. I don't care if this JSON has syntax highlighting or not.

Yes, it does work for me. Without syntax highlighting it will work faster given that the most expensive task is disabled, that's why other editors do not even try to do syntax highlighting for those cases. If you have the crash log please share it.
There are a few things I can disable to improve the overall experience and I guess that's what you encountered:

Disable matching brackets (finding matching brackets for huge files can be really really expensive).
Disable creating the auto-complete dictionary, this is currently being limited by line count but not file size, i'll add that, because for single line examples is critical.
Disable syntax highlighting for huge single line files.

You can try doing that manually, it might work / help.

Done.


Run setActive a bit more reliably. Fixes #578.

Now setActive is always called after the window has been created (no longer luck based / race condition)

This may actually be a more sound solution than my opentk-level fix (though I was hoping to fix opentk itself). If possible, can you roll back the opentk nuget package one version and test your changes again (I'll do this but am looking at something else today)?

Here's the change I made in an attempt to fix initial focus not being correct. Unfortunately this seemed to also be incorrect, causing the window to potentially (at startup) be in a non-active state when it should be active:
https://github.com/ppy/opentk/commit/9fe16fb87b41834351a5773cc39893eb9536fbcd

Please give your branches unique names in the future, rather than pull requesting from master.

Closing this for the time being due to inactivity (and the fact that it most likely needs a serious fix at opentk's end, since as @smoogipoo mentioned the Actived/Deactivated methods are used elsewhere.


Duplicate game announcements

The date field for example here is changed when the announcement is edited. Ignore it, use only gid.

Should be fixed now. I've deleted all announcements and re-fetching them now.


Fully implemented test for drawInstanced()

Changes:

Fully implemented the test for drawInstanced() with instancing - This test was incomplete when the PR with draw calls was opened due to issues getting the basic pipeline working. Instancing has since been added and this test now runs as advertised.

Remaining draw tests and the implementation for Vulkan's readTextureResource to allow these tests to run on Vulkan are coming later.

Closing this PR and opening a new one as I've finished the test for drawIndexedInstanced, which involves a major refactoring of the code in this file.


Add samplers as an ETL module

Class imbalance problems seem to be frequent for our use case, and weighted loss functions don't seem to be enough. They require methods for sampling data, bootstrapping, and duplicating data based on a label (ref).
https://docs.google.com/document/u/1/d/19iCoT3kVtTxzOd72rvvMsMUiwq6sXhQVbjIwb9jkN-4/edit?usp=gmail#heading=h.fo3yjcilgbz5

/cc @eraly @AlexDBlack for comments (a 👍 reaction is good to)

Handling class imbalance better is something we definitely need. However, I'm not sure exactly what the best approach is here, even just for the sampling-based techniques.
For example,

Do we produce a new dataset (say, a new CSV), with undersampling of frequent classes, or oversampling of infrequent classes?
Do we build sampling into the RecordReader.next() method? (Easy enough for images, pretty hard to do for CSV etc)
Do we preprocess our data and build this at the DL4J/ND4J level? https://github.com/deeplearning4j/nd4j/issues/1521#issuecomment-269857986

Will be handled in pending rewrite.


Add tests

Thanks for the package, would you a accept a pull request that adds test cases to the package?

My coworker opened a pull request implementing the tests: #6


Adding cli command to clear out pending symantec certificates.

Coverage decreased (-0.1%) to 60.459% when pulling 96e65ad883b6c54e846252be012269869767f42f on kevgliss:clear-pending into c402f1ff874350cd0352764e4e95e4d5d84ba26b on Netflix:master.

Coverage decreased (-0.1%) to 60.459% when pulling 6a05e41356fabee0bb80b01919dcde86ff44cae7 on kevgliss:clear-pending into c402f1ff874350cd0352764e4e95e4d5d84ba26b on Netflix:master.


contacts with no name are not fetched by package.

I'm trying to fetch all device contacts but contacts with no name are not fetched.
Contacts.getAll((err, contacts) => {
        if(err){
          if(err.type === 'permissionDenied'){
            alert('Please go to Settings > Privacy > Contacts to allow "xyz" to access your Contacts');
          }
          else{
            alert(err);
          }
        } else {
          context.getcontact();
        }
      })

Platform : iOS,
React-Native - 0.24.
Thanks

please use triple backticks (```) around code snippets

Please test on master - I believe this is fixed.

@npomfret  same issue with updated version.

@morenoh149 i tried with triple backticks (```). but same result can you plz share the sample code.

Resolve this by commenting following code in  RCTContact.m
if(!hasName){
//nameless contact, do not include in results
//return nil;
}
Thanks

That code is not on master, it's been removed.  Please check again.  Change your package.json to point to master rather than the released version

Thanks a lot.


Feature request: Manually change the values returned to reflect system idiosyncracies

Description
In Rhino, neofetch presently returns OS: Ubuntu Noble Numbat (development branch) x86_64
x86_64 can be turned off with --os_arch off, but it doesn't appear the OS can be turned off.  Further, it can't be changed.
The on/off options present in neofetch are sufficient and preferable in most cases, however this is a use case where in the config_file or on the cli, the user could specify other data for all of the fields displayed by neofetch including changing OS to read e.g.,
OS: Rhino Linux based on Ubuntu Noble Numbat (development branch) x86_64

i guess this is already there.


Custom model limit, clean up (?) if/else logic

I don't know if this is just me, but I found the if/else logic hard to read. Does this code do what you intended and do you consider it is it an improvement? As far as I understood we want to:

reject queries with custom models > 500km
disable alternatives with custom models > 200km

Ok, but what is the 200_000 meter limit for then? I thought you want to use alternatives below 200_000, not use alternatives between 200_000 and 500_000 and do not send a request at all above 500_000.

like this?

Sorry yes sure my bad.

Probably I should have written some simple tests to make refactoring easier :)

better just straightforward code that does not require thinking :)


bug: Remove Input Focus when open Menu via swipe gesture

Bug Report
Ionic version:

[x] 5.x
Current behavior:

When i'm currently focused in an Input Element and Keyboard is open, then open the Menu via Swipe Gesture, i'm still in the Input Field and the Keyboard is still open. When i open the Menu via the Menu Toggle Button it works and i'm not anymore focused in the Input and the Keyboard closes.
Expected behavior:

I expect that in the Moment where i start the swipe Gesture of the Menu, the Keyboard closes and the Input loses the Focus.
Steps to reproduce:

Select a Input
Open the Menu via Swipe Gesture

Other information:

Ionic info:

Ionic:

   Ionic CLI                     : 6.1.0 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.0.1
   @angular-devkit/build-angular : 0.803.25
   @angular-devkit/schematics    : 8.3.25
   @angular/cli                  : 8.3.25
   @ionic/angular-toolkit        : 2.1.2

Cordova:

   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : android 8.1.0, ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 53 other plugins)

Utility:

   cordova-res (update available: 0.9.0) : 0.8.1
   native-run                            : 0.3.0

System:

   Android SDK Tools : 25.2.3 (/Users/hanskrywalsky/Library/Android/sdk)
   ios-deploy        : 1.10.0
   ios-sim           : 8.0.2
   NodeJS            : v10.16.3 (/usr/local/bin/node)
   npm               : 6.13.7
   OS                : macOS Catalina
   Xcode             : Xcode 11.3.1 Build version 11C504

Thanks for the issue. Can you try the following dev build and let me know if it resolves the issue?
npm i<EMAIL_ADDRESS>

Hey Liam. Just tested the DEV Build and it works 🎉💪🏼

Thanks for the issue. This has been resolved via https://github.com/ionic-team/ionic/pull/20638 and a fix will be available in an upcoming release of Ionic Framework.


allow .devfile.yaml

Signed-off-by: anandrkskd<EMAIL_ADDRESS>What type of PR is this?
/kind feature
What does this PR do / why we need it:
This PR adds a feature with which we can use .devfile.yaml without adding --devfile flag.
With this PR in, odo will prefer devfile.yaml and if not present will use .devfile.yaml if   .devfile.yaml exists.
Which issue(s) this PR fixes:
Fixes #3126
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:
Simplest way to check if .devfile.yaml is used without using --devfile flag
git clone https://github.com/openshift/nodejs-ex && cd nodejs-ex

odo create nodejs

mv devfile.yaml .devfile.yaml

odo push

@anandrkskd It would be nice to have a more complete test with a .devfile.yaml file, covering the most important odo commands


Fix minor typo

Fix minor typo in README.
Checklist

[X] 👋  I have run the unit tests, and all unit tests have passed (not functional code).
[X] ⚠  This pull request might incur a breaking change.

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

Thank you for finding that typo!

@guikcd Thanks for your finding, and the pull request is merged.


GH-37722: [Java][FlightRPC] Deprecate stateful login methods

Rationale for this change
The existence of these interfaces confuses users and leads them to antipatterns.
What changes are included in this PR?
Deprecate (but not for removal) the old interfaces.
Are these changes tested?
N/A
Are there any user-facing changes?
Yes.

Closes: #37722

@lidavidm Should I follow suit with the same on the Go side?

Yes, anything based on Handshake should not be preferred by new application code (it's probably just a footgun at best)


Vertex lighting objects disappear on Android, GLES 2.0 renderer

Godot version:
3.1 Stable (downloaded from godotengine.org)
OS/device including version:
Samsung Galaxy S6, Android 7.0
Issue description:
I don't know if this has been asked before.
Using the GLES 2.0 renderer, when the 'Vertex Lighting' checkbox is enabled, objects seem to disappear once they enter the range of an omni light. This doesn't happen on my desktop PC, but it occurs once I export the game to Android. I've tested with GLES 3.0 and this issue doesn't occur.
Steps to reproduce:

Use GLES 2.0 renderer
Add an omni light
Add a 'MeshInstance', assign it a material making sure 'Vertex Lighting' is enabled
Place this 'MeshInstance' below the omni, or inside it's range
Export to Android and run

Minimal reproduction project:
Please find the attached files
vertex_light_issue.zip

Happens to me as well.

Are you on a Samsung phone as well? Happens to me on Samsung J6, Android 8.1. And it's only for shaded materials with vertex lighting on (as expected). @Janders1800

I'm on an Alcatel 1 with Android 8.1, also happens with directional light.

I encountered this issue as well, GPU: Andreno 305, OS: Android 5.1, here's the error:
--From Vertex Shader: Error: Symbol m_roughness defined with different precision in vertex and fragment shaders. --From Fragment Shader: Error: Symbol m_roughness defined with different precision in vertex and fragment shaders.
  <C Source>     drivers/gles2/shader_gles2.cpp:129 @ _display_error_with_code()
A workaround I made for this was to convert the SpatialMaterial to a ShaderMaterial, then add a high precision modifier to the roughness uniform.
Original: uniform float roughness : hint_range(0,1);
Fixed: uniform highp float roughness : hint_range(0,1);
Hopefully someone can find a real fix for it.

@yetha What's your Godot version?

3.1, the error still gets printed by the way (only once), but the mesh is visible and with vertex lighting.

Can you test with this recent build of the master branch? https://downloads.tuxfamily.org/godotengine/3.2/alpha0-unofficial/

Tested with that build and the issue no longer exists!

It was possibly fixed by #30331, which has been cherry-picked to the 3.1 branch for Godot 3.1.2 (yet unreleased). If you're able to compile the 3.1 branch from source (both editor and Android template) to confirm that the issue is fixed there too, that would be great. Otherwise I might have a 3.1.2-rc1 build available within a week.

Looks like it was fixed. Closing.


Error Handling

Hello, I am having trouble with error handling.
The stack trace and web searches show that the following class location:
"pymongo.errors.DuplicateKeyError" or "pymongo.errors.WriteError"
But, when I am using this, the application is giving me the following error:
"TypeError: catching classes that do not inherit from BaseException is not allowed"
I have tried using
try:
// code
except mongo.errors.DuplicateKeyError
except mongo.db.errors.DuplicateKeyError
except mongo.cx.errors.DuplicateKeyError
except mongo.errors.DuplicateKeyError
(where mongo is instantiated correctly in the app. I am able to write to the database.)
I would really appreciate any help regarding this issue. Thank you.

You'll need to import and use the Exception from pymongo itself, not from the mongo object that's created by Flask-PyMongo. Try:
from pymongo.errors import DuplicateKeyError

# later
try:
    # ...
except DuplicateKeyError as dke:
    # handle the exception

@dcrosta thank you for the response.


Remove third pass when validating generic function parameter lists

When checking a generic function we make up to three passes over the parameter list because of how member types (T.X) work:

(If the function has a generic parameter list) First, we collect requirements and form unresolved dependent member types, so that we can build a generic signature.
(If the function is in generic context) Then, we build fully resolved dependent member types using the new generic signature from step 1 (or the generic signature from context if we skipped step 1), and use these types to form the interface type
Finally, we make one final pass, using the generic parameter to archetype resolver

The third pass is actually unnecessary, and instead we perform the second pass unconditionally even when there is no generic signature.
The only useful work done by the third pass that wasn't done in the second pass is validating generic requirements involving type parameters. For example, if I had a parameter containing Set<U>, and U was not hashable, the third pass produced a diagnostic because the archetype for U is known not to conform to Hashable. The second pass was previously unable to diagnose this because a generic parameter never conforms to anything.
A small refactoring to checkGenericArguments() allows us to map type parameters into context before checking requirements. With this checking now done in the second pass, the third pass is no longer necessary.
This is NFC, but hopefully in the future will make it easier to move getInterfaceType() and getGenericSignature() to the request evaluator.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please test


new gate dnn model

adding embedding gate and hidden gate to dnn to make new models.

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


Inactive_account message not used in TokenCreateSerializer

"inactive_account" is set to settings.CONSTANTS.messages.INACTIVE_ACCOUNT_ERROR but never used in TokenCreateSerializer, so if the user can't log in because his account has not been activated he'll get the invalid_credentials error.
That's the validate method from TokenCreateSerializer
    def validate(self, attrs):
        password = attrs.get("password")
        params = {settings.LOGIN_FIELD: attrs.get(settings.LOGIN_FIELD)}
        self.user = authenticate(**params, password=password)
        if not self.user:
            self.user = User.objects.filter(**params).first()
            if self.user and not self.user.check_password(password):
                self.fail("invalid_credentials")
        if self.user and self.user.is_active:
            return attrs
        self.fail("invalid_credentials")

@Frohus Check this

@Frohus Check this

I have already amended it in my project, but I believe it needs looking at anyway.


IPv6: Handle it correctly, no NAT

Error description:
Container unable to see IPv6 IP address and as result ip restrictions not working.
Steps to reproduce
Have IPv6 enabled on host.
Expected result
App container sees IPv6 address and policies would work correctly.
Comments
There should be NO NAT for IPv6, period and it should use an actual prefix, which could be asked in .env files.
https://github.com/evertramos/docker-compose-letsencrypt-nginx-proxy-companion/blob/7cab8d8b8b0884fe548dd189233a00088e5331d1/start.sh#L17-L18 BIG NO on the NAT.

Please try the new version (v0.5), in case of error open a new issue. Thanks!
use the option '--activate-ipv6'


Add default configuration

I want to apply some options to all the views I create unless I say the opposite, like extending a certain layout or having certain elements. This could be archieved by adding a config file.

I feel like this would add unnecessary bloat / complications to a simple php artisan make:* package. How would the options work if this was implemented? Add new ones like --no-sections? Override them with empty values (--section=)?

It would take the values from the config if defined, then check if there are options and overwrite if they are.

Right, so lets say I have a config file set up like this (mockup, obviously):
return [
    'extend' => 'layouts.app',

    'sections' => [
        'title',
        'content',
    ],
];

This would then always generate a view like this:
@extends('layouts.app')

@section('title')

@endsection

@section('content')

@endsection

What if I wanted to create a view that doesn't include the @extends() and @section()s? At the moment I can think of a few ways this could work, some maybe better than others 😉:
$ php artisan make:view my-view --section=<X> --extend=<X> # where <X> is a predefined keyword.
$ php artisan make:view my-view --no-config
$ php artisan make:view my-view # and only read the config file if a --with-defaults flag is set, maybe?

I'd say, if you have a config, you have to overwrite it with options. If you find yourself overwriting it very often, then you migth want to remove that config option...

Hmm. I'm very much in two minds about this. I don't see myself (or many other people) reach for this. I mean, how often do you create a view that you would need defaults?
On the other hand, it could be pretty cool to have for the power-user, and you could even take it a step further and introduce the concept of presets:
return [
    'presets' => [
        'foo' => [
            'extends' => 'layouts.foo,
            'sections' => ['title', 'content'],
        ],

        //
    ],
];

$ php artisan make:view --preset=foo

I normally extend layouts.app always except on the landing page, and I'm sure most applications with a dashboard do this. Also, I love the preset idea!

This package has been abandoned because of the introduction of the new make:view command included in Laravel since v10.23.0. For that reason, this issue will not be worked on.


Error parsing a copybook in 1.0.0 pre-release

The parser throws a syntaxt error for the following copybook:
       01  R.
                03 N     PIC X(1).
                03 V     PIC X(10).

The error is this:
Syntax error in the copybook at line 3: Invalid input 'V' at position 3:19

It works fine with this one:
       01  R.
                03 N     PIC X(1).
                03 K     PIC X(10).

This happens on pre-1.0.0 release branch.
@tr11 , at first glance the grammar should be able to parse the above copybook. I'm wondering why it does not allow for 'V' as a field name?


fix(cargo): publish default to crates.io

Codecov Report
All modified lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.


Issues accessing density of wood-based flows

I'm not sure if this is the right place to ask this question. If there's a better spot, please redirect me to the appropriate resource.

I am loading the USLCI and US Electric JSON-LD from this repository to have it loaded into our platform at @PlanetFWD so that we can keep our database up to date with USLCI. I've been able to work through most of the details of getting this to work properly, but I'm having trouble with a units mismatch on some flows. For example, Wood fuel, MDF, generated on-site, combusted in industrial boiler (in units of mass) uses Medium density fiberboard (MDF), at MDF mill (with a reference flow in units of volume). I can see in the description of the reference material that the density is 741 kg/m3, but I'm looking for a reliable way to parse this information programmatically. Any guidance here?
For reference, I have noticed that this issue impacts the following reference materials:

Solid strip and plank flooring, hardwood, E
Sawn lumber, hardwood, planed, kiln dried, at planer mill, SE
Engineered flooring, hardwood, unfinished, E
Roundwood, hardwood, green, at logyard, SE
Medium density fiberboard (MDF), at MDF mill
Particleboard, average, softwood, particleboard mill
Roundwood, hardwood, green, at mill, E
Roundwood, hardwood, green, at mill, NE-NC
Roundwood, hardwood, green, at mill, SE
Roundwood, hardwood, green, at logyard, NE-NC
Bucked and debarked log, hardwood, green, at veneer mill, E

Separately, I've noticed that there is a precedent with natural gas of storing the density of the material on the flow object within the flowProperties key, but this pattern does not appear to be followed for wood-based flows. For example, Natural gas, combusted in industrial boiler, at hydrocracker, for butadiene uses Natural gas, processed, for energy use, at plant and the density is available from the associated flow Natural gas, production mixture, to energy use. Is it common to store the density in flowProperties like this? If so, is there an appropriate place to create a pull request to add this in for these wood-based flows?
My temporary workaround has been to just create a local density map for wood-based flows, but this is a really unsatisfying and prone to inconsistencies with USLCI that I would prefer to avoid.
Thanks in advance for your help. 🙏

It looks like the Wood fuel, MDF, generated on-site, combusted in industrial boiler process uses one of the co-products from the MDF Mill. Both the Wood Fuel, and the Wood Fuel, sanderdust are in units of Mass, though the reference flow of the MDF is in volume. So in this case, the units shouldn't need conversion (they can remain as mass). Note that in this process, it is currently constructed to use a mass allocation - see the note in the "intended application" field and on the allocation factors tab.
Yes, the Flow Properties field could be a place to find that kind of conversion information, and I expect this would support your efforts to incorporate that data programmatically. We will review if this can be added for the next release.

If so, is there an appropriate place to create a pull request to add this in for these wood-based flows?

Unfortunately not, though thank you for the offer -as the Federal LCA Commons data do not use git directly. But raising this issue here is the next best step.

I'm not sure if this is the right place to ask this question. If there's a better spot, please redirect me to the appropriate resource.

Yes - this is a fine place to raise these issues related to USLCI. I'll also point you to the companion documentation site which has a lot of resources for USLCI and all repositories on the Federal LCA Commons

@bl-young Thanks for getting back to me. Much appreciated :pray:
I'm a little confused though so I just want to confirm or complicate my understanding. From your comment above, it sounds like the allocation factor for Medium density fiberboard (MDF), at MDF mill (labelling this as A, for convenience) already accounts for the conversion to mass for the processes that use it like Wood fuel, MDF, generated on-site, combusted in industrial boiler (labelling this as B, for convenience)? How should we interpret the allocation of upstream emissions for B from A in this situation then?
To be more precise, A has a reference flow of 1 m3 and particulate matter emissions of 0.363 kg. So if B uses 0.44 kg of A in the flow of Wood fuel, at MDF mill, the allocation factor for A to B is 0.0615. To get the upstream particulate matter emissions for B, I would have expected to calculate:
upstream_particulate_matter_B = (0.363 kg particulate matter) / (1 m3 ref flow) * (0.44 kg ref flow) / (density ref flow) * (0.0615)

Is this correct? From your statement above, it sounds like we do not need to divide by the density and that the allocation factor is somehow in the appropriate units so that we shouldn't have to worry about the units of the reference flow. If so, how should we be calculating the upstream particulate matter emissions for B in this situation?
I'm being pedantic about this example to make sure I'm understanding how this is supposed to work. My next question will be clarifying how the JSON-LD format encodes this information, but that will hopefully be straightforward once I clarify the mechanics.

@bl-young Really appreciate your diligence here. Thanks for identifying the outflows that were in units of mass; I did not notice that and it's an important part of the puzzle. I'm digging into this further and compared both of our interpretations to results from OpenLCA, which presumably would be the definitive source:

In order to reproduce the 0.00018 kg particulate matter result for just the "Wood fuel, at MDF mill" flow (AB for convenience below), here's what I'm seeing we have to do:
upstream_particulate_matter_B = (0.44 kg AB) * (0.363 kg particulate_matter / A ref flow) / (54 kg AB / A ref flow) * (0.0615 allocation factor)
                              = 0.44 * 0.363 / 54 * 0.0615 kg particulate_matter
                              = 0.0001819033 kg particulate_matter

There are two key differences that I didn't fully understand:

I had no idea the mass outflows were logged like this in the JSON-LD format. Thanks for the pointer there. It seems like there is an additional case that I need to deal with when there are allocation factors involved and I'll need to investigate that more carefully.
The 0.44 kg IS NOT in units of kg A but instead in units of kg AB. That's not how I was interpretting things before but it does help me understand what the different numbers are intended to represent.

If I'm understanding this correctly, this calculation is saying that process B requires 0.44 kg of material AB from process A. Process A produces 54 kg of AB, 6.15% of which is used in process B. Is that the correct interpretation?

I had no idea the mass outflows were logged like this in the JSON-LD format..

Sorry I'm not sure what you mean by this? But yes, you should be able to find the allocation data within the json-ld for the relevant processes.

If I'm understanding this correctly, this calculation is saying that process B requires 0.44 kg of material AB from process A. Process A produces 54 kg of AB, 6.15% of which is used in process B. Is that the correct interpretation?

Your math is correct there in that it matches openLCA, but I don't think that interpretation is correct. Can you confirm, is there a default allocation method selected for that process? And/or what did you select when you ran results for allocation?
@pweilerERG  or @bnjmnmorelli - can you review this tracking of results related to multi-product output flows to make sure we have this explained correctly?

@bl-young @pweilerERG @bnjmnmorelli I didn't make any special selections in terms of getting the openLCA result. I'll be curious to better understand how I should be interpretting these results. Thank you all for your help.

@deanmalmgren It seems like you have two main goals. Understand the allocation going on between your two processes and try to deal with the conversion between volume and mass programmatically. I am not sure I will answer the second of these two questions without more information but I will try.
I am going to start with the 'Medium density fiberboard (MDF), at MDF mill' process. I may duplicate a little of what has been covered above, but hopefully this helps. As you see below it is allocated using 'Physical' allocation. You can see that the highlighted MDF flow is in m3. If you do the math the physical allocation factors are correct and they manually apply the stated density, 741 kg/m3, to the reference flow, 1 m3. If you open the process and click on the 'Calculate factors' button you will get an error that the process is unable to calculate these factors automatically.

In order to facilitate automatic calculation of the allocation factors you need to add the density conversion as a flow property as you mention. I have done so in the picture below and confirmed that it recreates the allocation factors in the original process.

I think it is probably a good idea to add in these conversions such that we can calculate allocation factors automatically and so that the conversion is more transparently documented. However I consider this more of a housekeeping item than an absolute necessity. The manually calculated allocation factors should work just fine. In openLCA, it would not be a problem even if your downstream process (in this case - Wood fuel, MDF, generated on-site, combusted in industrial boiler) relied on medium density fiberboard as in input in volume units (which it doesn't). I have confirmed this in test processes. If this is somehow causing an issue in your application it might be helpful for us to understand this better so that we can consider your use case in how we are managing the USLCI.
As Ben Young stated, your calculation of how the allocation functions is correct, but I would phrase your interpretation a little differently.

If I'm understanding this correctly, this calculation is saying that process B requires 0.44 kg of material AB from process A. Process A produces 54 kg of AB, 6.15% of which is used in process B. Is that the correct interpretation?

How I would explain the allocation is as follows. I stick to your words as closely as possible:
This calculation is saying that process B requires 0.44 kg of material AB from process A. Process A produces 54 kg of AB, which is 6.15% of the total outputs in process B (on a mass basis). When you divide 0.44 by 54 you are determining the share of that 6.15% which is actually passed on to process B.
Hopefully this helps! If not, maybe a quick call will be an easier way to resolve your issue. You can reach me at<EMAIL_ADDRESS>

@bnjmnmorelli @bl-young thank you both for your help. Much appreciated. I believe I'm now clear on both the math and the interpretation of how these calculations are run. I had definitely missed the flow of AB for processes that had coproducts and this helps me resolve the units mystery on wood products. Gosh I'm glad I said something because I would have misinterpreted the data by quite a bit in some situations.
I'll close this out.


🎉 4.0.7

ℹ About this release

Version: 4.0.7
Type: patch
Last Release: 4.0.6 (2/5/2023, 7:14:39 PM) [?]
Commits to merge: 30 [?]

🐛 Bug Fixes

Fix semantic-release import (0b9dc54)

📦 Dependencies

Update cosmiconfig from ^8.0.0 to ^8.1.3
Update semantic-release from ^20.1.0 to ^20.1.3
Update simple-git from ^3.16.0 to ^3.17.0

📦 Development Dependencies

Update eslint from ^8.33.0 to ^8.36.0
Update eslint-plugin-jsonc from ^2.6.0 to ^2.7.0

:tada: This PR is included in version 4.0.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Profile card popups not working with "Media (Images / Videos)" option

When selecting "Media (Images / Videos)" to hide media, hovering over a username doesn't popup the normal profile summary.

Fixed. Found a different class name to attach to.


Don't auto play videos if a video is already playing

When entering the details view of a video it starts playing automatically. This isn't desirable when a video is playing in the mini player. Navigation should not have side effects like aborting what you've already decided to watch/listen to.

It could be argued that it's not desirable at all, since you can't read the description without entering the detail view.
If you're e.g. listening to music while browsing and deciding whether to watch something then you currently have to choose between either a) interrupted playback of what you've already decided to listen to (the music), or b) not reading any video description.

Similar to the Android app we're working on, we want to make this a 3 way setting: 1) always, 2) only when not playing, 3) never

Sounds reasonable, if #\2 applies to any app (I won't know how to escape # to not link internally on GitHub...)

@anohren What would the intended UX be if a user clicks on another video, but doesn't start it playing, then wants to go back to the first video? Currently the video view slides in/out, and it gets replaced when switching videos.
Should adding a new video add to the stack?
Maybe once the next video is started the first video can be removed from the stack such that when dismissing there's only one video to dismiss?

What would the intended UX be if a user clicks on another video, but doesn't start it playing, then wants to go back to the first video?

I imagine that the mini player is already active at that point and that you tap on it to bring up the active (first) video. To me the video player/page looks modal, currently.

Should adding a new video add to the stack?

Here I'm not sure exactly what "adding a new video" refers to. Do you mean navigating to a new video?
I'm also a bit unsure about whether you mean "stack" as in the usual navigation stack or an invisibly maintained stack of some sort. If the former, then I have trouble visualizing the stack since to me neither the mini player nor the regular video page seem to have regular navigation stack behavior (?)
I think I'd find it a bit disorienting if the decision of whether a video page is opened in the navigation stack or in the (current) "modal" video view were context sensitive. But yes, I'd expect the second video to be pushed onto the navigation stack. I'd expect the first one to as well, I think, and only become modal if I were to access it through the mini player after having navigated away.

I imagine that the mini player is already active at that point and that you tap on it to bring up the active (first) video. To me the video player/page looks modal, currently.

Do you mean that the miniplayer should be kept even on the video viewer page?

Here I'm not sure exactly what "adding a new video" refers to. Do you mean navigating to a new video?

Yes

I'm also a bit unsure about whether you mean "stack" as in the usual navigation stack or an invisibly maintained stack of some sort. If the former, then I have trouble visualizing the stack since to me neither the mini player nor the regular video page seem to have regular navigation stack behavior (?)

It slides in from the bottom and is closed by swiping down, so I'm wondering if it should be some kind of stack where you can go back to a previous video.

I think I'd find it a bit disorienting if the decision of whether a video page is opened in the navigation stack or in the (current) "modal" video view were context sensitive. But yes, I'd expect the second video to be pushed onto the navigation stack. I'd expect the first one to as well, I think, and only become modal if I were to access it through the mini player after having navigated away.

I don't quite get what you mean.

Do you mean that the miniplayer should be kept even on the video viewer page?

I think that is the most straightforward behavior compared to the current behavior if you want to open up for allowing users to play one video while also being able to navigate to other video pages. Otherwise, if there weren't even the mini player, you'd be left with only the audio and no visual indication that another video is playing in the background.
Ok, I think I understand what you mean. A separate, always accessible stack of video pages with a max depth of 2. That doesn't correspond well to my mental model from most other apps, but then again I don't use the YouTube app which has probably established the strongest precedent
and expectations of how an app like this should behave. Maybe it's a good idea to fulfill YouTube users' UI expectations if you want to provide a frictionless experience? I should probably take a look myself...

I don't quite get what you mean

You can disregard that. I guessed wrongly that you wanted to push video pages onto the navigation stack, but only if another video was already playing.
I think I personally would like video pages to simply be put on top of the navigation stack, just like content views in other apps, and to receive no special treatment unless I've started playback and navigated back (at which point it is preserved by transitioning to e.g. a mini player). In that case it'd be quite natural for the mini player to be visible while navigating to other video pages. Right now they feel like a modal singleton to me.
I don't know if I'm answering your question at all now, but that's comes to me off the top of my head.

The YouTube app doesn't even have any confirmation, and only the currently playing video is available in its miniplayer.
I guess if the mini player is visible on video pages, there's no need for the multiple "stack" of video views, since the currently playing video will always be accessible.
Then you can navigate around looking for your next video, and click the miniplayer to go back to the currently playing video.
However, once the next video has been started, would you want there to be any way to get back to the previous video? This will have to change the navigation flow quite a lot, so if it isn't strictly necessary I'd just do the "miniplayer always visible" change.

However, once the next video has been started, would you want there to be any way to get back to the previous video?

I can't think of any particularly compelling reason for that. When I decide to watch something else then I signal that I'm done with the current video.
That's not to say that playback history lists a la iOS Music app don't have value, should you choose to add one in the future.

We should consider adding a stack history in the future so that the back button would work, but miniplayer access is fine for now. Thank you again @anohren !

If YouTube is the "gold standard" of video apps, I can happily say that with #378, we will have well surpassed that. (YouTube has a miniplayer but as soon as you open something else it goes away 🙄 )
@anohren The behaviour I've implemented for #378 is:

If nothing is playing, immediately start playing.
If another app is playing, immediately start playing. This is consistent with Apple Developer app and YouTube app.
Miniplayer stays visible if opening another file.
If miniplayer is visible, don't immediately start playing.
Clicking play button removes miniplayer.
Clicking miniplayer goes to that file.

Is the three-way setting of 1) always, 2) only when not playing, 3) never still necessary with this behaviour?

Nice work. I hope it will be received as an improvement!

If another app is playing, immediately start playing. This is consistent with Apple Developer app and YouTube app.

This one's interesting. While YouTube actually auto plays when navigating to a video page, the Developer app doesn't (for me). That means that YouTube has a legitimate reason to stop previous playback, while the fact that the Developer app does the same is best viewed as a bug; there's no good reason for it from a user perspective.
Whether or not videos actually should auto play upon navigation — the way they do in the YouTube app — is perhaps not so much a design decision as a business decision. It reduces friction of consumption, while it's a bit harder to argue that it's expected behavior.
Regardless, looking forward to try it out.


Can't view the dataset

Hi, @simongraham and @vqdang .I have met a understanding issue when I run the train by used command "python train.py --view train", it only gave two warns and then the procedure was killed . I can't find any image to see. Just like this:

I search the MatplotlibDeprecationWarning online , and find someone say it not a problem. so what cause this issue , the last warn or others? ( by the way , train.py is run well if use the command : python train.py --gpu='0' )

It appears you are not quite using the correct command. Instead use python train.py --view='train'

@simongraham ，Hi , Thanks for your reply . It seems that there is still a problem, the image is not displayed, is it because I am connecting to the Linux server through XShell ?

It has been in this state for eight minutes

If you are remoting like ssh, you dont have the GUI. The code will just stuck there.

thanks for your help .


MuTransfer for Mamba

Hello,
I have been training models with mamba (v1) and I'm enjoying it. I would like to use MuTransfer for Mamba. Should I just scale the width params (matrices dim and conv dim) or are there other constants that need to be scaled like in transformers attention_scores?

We don't have much experience with MuTransfer


Hide Path in Debug Console

As with #115625 we would like the option to completely hide the path. On a HD monitor with multiple panes it renders the debug messaging unreadable and is still waaaay too long. A simple elipsis is enough for the developer to click on, why show the entire path or even a portion of it??

https://github.com/microsoft/vscode/issues/168033


How to maintain scalaz-stream after this becomes master?

My initial plan was to take this branch, and then port back to a scalaz-stream maintenance branch for my work and for the rest of the community that isn't ready to get off scalaz-stream.  Then any future pull requests to the fs2-cats, where we expected most of the action, could be merged back.
But the diff is getting enormous.  I don't want to do this entire port in reverse.  We'll need a reasonable source management strategy so I don't lose my mind.
This ties into the discussion on #875 a bit.

We are going to do bugfixes only on the scalaz-stream branch after 0.16 is released.  The strategy will be to cherry pick what's relevant, but we can drop efforts to keep the two branches in sync.


Revert #2496

#2496 introduced a bug which was spotted by my daily CI in
https://github.com/sequencer/playground/runs/6243944547
An MVP should be:
RegEnable(next = true.B, init = true.B, enable = true.B)

won't compile, and Scala complains:
macro applications do not support named and/or default arguments

This was should blame to Scala Compiler in https://github.com/scala/scala/blob/21a56430f9902b5b8b81535fd76371caf23c0d10/src/compiler/scala/tools/nsc/typechecker/ContextErrors.scala#L630
which seems to be a limitation to Scala compiler(both 2.12 and 2.13) that the function which used macro cannot use default or named arguments.
Thus #2496 introduced a breaking change to related function calls.
There exists three options to this:

Reverting #2496 to make downstream happy(keep compatibility but the detail source info is lost)
Regard this as a breaking change, user should bear with it.(we should add this to release note)
Refactoring SourceInfo and CompileOption macro to Scala Plugin, put it earlier than macro.(refactor a lot but user will be happier)

Contributor Checklist

[ ] Did you add Scaladoc to every public function/method?
[ ] Did you add at least one test demonstrating the PR?
[ ] Did you delete any extraneous printlns/debugging code?
[ ] Did you specify the type of improvement?
[ ] Did you add appropriate documentation in docs/src?
[ ] Did you state the API impact?
[ ] Did you specify the code generation impact?
[ ] Did you request a desired merge strategy?
[ ] Did you add text to be included in the Release Notes for this change?

Type of Improvement

bug fix

API Impact

Backend Code Generation Impact

Desired Merge Strategy

Release Notes

Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (Bug fix: 3.4.x, [small] API extension: 3.5.x, API modification or big change: 3.6.0)?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you mark as Please Merge?

There might be a 4th option, we could maybe use @deprecatedName to sort of deprecate the use of by-name parameters on these methods.
We really want to add source locators if at all possible but if we can avoid a hard breakage (without a deprecation step) that would definitely be ideal.

I am curious to hear your thoughts on this @jackkoenig : How hard / how useful would it be to implement source locator propagation through a compiler plugin instead of using macros?

@deprecatedName sounds to be a great idea!

wait are we saying that the change to adding source locators means we can't pass named arguments to RegNext and friends? That seems sad in general...

Yes, it seems that Scala compiler doesn't like this, I have seen two downstreams beaten by this:(
https://github.com/chipsalliance/rocket-chip/pull/2986
https://github.com/sifive/sifive-blocks/pull/177

I am curious to hear your thoughts on this @jackkoenig : How hard / how useful would it be to implement source locator propagation through a compiler plugin instead of using macros?

All compiler plugin stuff is difficult, we'd essentially be reimplementing a subset of what def macros do. It's probably doable, but I'm not actually convinced that named arguments would work if we did it in a plugin. Macros paradise was implemented as a plugin prior to Scala 2.13 and it has never supported named arguments so I suspect there's some fundamental limitation here.

wait are we saying that the change to adding source locators means we can't pass named arguments to RegNext and friends? That seems sad in general...

@mwachs5 - Yes, it is a really annoying limitation but it is why Chisel3 moved to things like UInt(8.W) instead of UInt(width = 8) and RegInit(0.U) instead of Reg(init = 0.U). It's a super annoying limitation.


This is a security test as part of a bug bouty, plese ignore.  tgwkvg

This is a security test as part of a bug bouty, plese ignore.  tgwkvg

go away


Fix issue in bar_plot with misspecified confidence intervals

If the confidence intervals are misspecified, e.g., greater lower bound than upper bound, the method threw an error before. This, however, can sometimes happen due to precision errors in some algorithms and lead to random build fails. This change fixes the issue and ignores invalid intervals accordingly.

Build should be fixed after: https://github.com/py-why/dowhy/pull/1080


add custom 404

& coincidentally needed force pages rebuild

This isn't rendering with any styles or layout. Intentional?

No, just didn't think. I mostly added this to force a rebuild without resorting to an empty commit or finding some other meaningless change. #33 should fix.


Due support

I am wondering about whether you could add support for the Arduino Due?  Presently it fails to compile for Due processors.

also request this, how define variables for this unknown CPU?

Same here... i followed the seeedstudio tzutorial but get an #unknown cpu error.
Can anybody help please?

@pushmeaway2 the tutorial's instructing to use the soft i2c, why could you encounter the unknown CPU error? To have the I2cMaster library support Arduino Due, it's not just as simple as adding some varibles in I2cMaster.h, the interrupt mechanism is different from that of AVR chips, it seems that I2cMaster library has no plan to support Arduino Due since the Due's user volume. So please consider using the soft i2c.

@pushmeaway2 has fixed Seeeduino 4.2  compilation error "error unknown CPU" by adding ARDUINO_ARCH_AVR  to the board library.

Will close this issue, please open a new issue if you have further questions.


v3: Defining an array densities does not generate images

Great job on the new version, really like the dx.
Version:

"next-optimized-images": "^3.0.0-canary.10"

Setup:

everything per recommended in the docs
next.config.js:

javascript
    [
      optimizedImages,
      {
        images: {
          handleImages: ['jpeg', 'png', 'webp'],
          webp: {
            quality: 100,
            nearLossless: true,
          },
        },
      },
    ],

images.config.js:

module.exports = {
	default: {
		webp: true,
		densities: [1, 2]
	},
};

Usage:
...

...

The problem

original image is png; the only thing generated is an optimized png and webp image, but no images with defined densities

various densities reference the "same" image element

also does not work when defining densities inline, on the Img elements themselves

HTML looks like the following

<picture>
  <source type="image/webp" srcset="/_next/static/chunks/images/landing-hero-d7a2b56784e764765f7be1e94ebcc13a.webp, /_next/static/chunks/images/landing-hero-d7a2b56784e764765f7be1e94ebcc13a.webp 2x, /_next/static/chunks/images/landing-hero-d7a2b56784e764765f7be1e94ebcc13a.webp 3x">
  <source type="image/png" srcset="/_next/static/chunks/images/landing-hero-0de1f1910a9d4f45cf113b0277fe4f8a.png, /_next/static/chunks/images/landing-hero-0de1f1910a9d4f45cf113b0277fe4f8a.png 2x, /_next/static/chunks/images/landing-hero-0de1f1910a9d4f45cf113b0277fe4f8a.png 3x">

</picture>

Expected behavior

images with defined densities get generated and referenced in the HTML as expected

Any news on this?


Speak button broken

In Firefox (40.0.3 - IBM CCK), if I visit http://text-to-speech-demo.mybluemix.net/ and click the red Speak button, it logs the following:
InvalidStateError: An attempt was made to use an object that is not, or is no longer, usable

The error appears to be coming from the audio.currentTime=0 line.
Chrome appears to work fine.

Thank you for fixing this @nfriedly, this was an annoying bug.
Dani


Some Ideas for ozw672 usefull also for ozw772

I am a novice in the Home Assistant community and therefore I apologize if I have not yet learned the best rules of conduct.
I had the idea of trying the integration written for OZW672 also for OZW772 and as I expected it seems to work perhaps well enough for what I need.
It would probably be best for me to attempt the required changes myself. But I am really still a beginner and I think that what I have found is also very useful for users looking for a custom OZW772.
I therefore only have the following observations, requests or ideas to report.
Observations:
I have detected all the problems already recognized and therefore await developments.
Requests and/or Ideas.
1_ custom_name vs ???
In config_flow.py :
'' ### Support a Customized name for the Device being monitored.
Using the custom_name gives me the names of the items in Italian while I prefer the names in English;
is it possible to enter an optional choice for not custom_name instead ?
2_ exclusion of datapoints with the same names
In config_flow.py
there seems to be a point where you try to avoid inserting already polled datapoints.
I report the text found here :
existing_dp_items = self._datapoints
''### If we are already polling a variable - don't list it.
Since an existence test is performed only on the names of the final nodes of menutree instead of on the entire path, unfortunately it does not allow you to insert nodes with datapoints that have the same name but are on different menutree paths.
Is it possible to add an optional choice here too, to avoid excluding what should be inserted instead ?
Many Thanks in advance.

This is a good idea.  I will investigate implementation.

Thanks for your reply.
In the meantime I managed to use your Custom Integration and I have some observations that can be useful to you.
I adapted to using Italian for the language of the Menu and Datapoints texts, because it was better than nothing.
To create the same Data point under two different paths for two different heating circuits I commented out the source lines of your integration where it does the test to avoid duplication.
It was just enough to create two Devices for the two Circuits, but it wasn't easy because the behavior of the program is a bit unpredictable.
When I created the second device with the second datapoint, it often doubled the first one and so I was forced to delete them both until at least one of the two reached a Not Loaded state. From this state, the Integration was forced to create the second correct one as per my instructions, and then I went to do a definitive delete of one of the two from before (after a restart of HA obviously).
When I had to add a new item, the program decided under which Device to put it even if I declared the first or the second at the beginning, and at the first restart I found it repeated on both.
Somehow I managed to use your integration and continue my project, so I thank you very much.
In the meantime I asked Siemens for the API documentation to see if I could give you a hand with the Integration, but I encountered insufficient Siemens support and after having made me sign a Non Disclosure Agreement they still haven't managed to give me access to the API. The only useful information is the confirmation that ozw672 and ozw772 have the same APIs as polymorphism requires.
If you have the API Documentation and you are not bound by an NDA, can you send them to me?
If you need further information and/or clarifications I am at your complete disposal.
Best Regards Maurizio.

I'm preparing a release with two fixes.

Introduction of a Boolean you can turn off to use the default "Address DeviceType" name instead of the custom name.
Check for existing DataPoints by ID instead of Name

Thanks for the effort you are putting into the changes to the ozw672 integration.
If you are interested. I discovered 2 other critical issues that I was forced to solve with patches.
Here is their description:
1_ The need to manage a "binarysensor" type HAType that is not included in your possibilities and I managed to solve my need by changing your source in a daring way.
2_ This second one is definitely much more complex because I discovered that on Siemens there is a Type that does not exist in HATypes. It is the "RadioButton" type. The RadioButton Type in Siemens also has the Buttons attribute instead of Enums and its structure is different from that of a Select with Enums. I tried to force the HAType Select with Enums instead of RadioButton with Buttons and the Integration refused to do Create of the Entry.
I guess since there is no HAType corresponding to the Radio Button there are two solutions:
A_ Create a custom HAType in HA with a structure suitable for the Siemens Type RadioButton with Buttons instead of Enums. If you need the correct structure that I got from the Debugger I can give it to you; it is very similar to that of the Select, but different enough to not be usable.
"B_ This type of DataPoint could be managed only writable and not readable, because in Siemens (RMH760 device) it has the sole purpose of giving a confirmation of Faults Acknowledge of which there is detailed information from another DataPoint. I leave it to your experience of how to translate this concept into practice.
My idea of a solution is therefore based on the possibility of launching a direct writing API on the RMH760 Datapoint for the Faults Acknowledge request without having to poll it in a Datapoint in HA.
At the moment I had to abandon this automation that is necessary for me to make Faults Acknowledge of non-severe Faults that I must therefore cancel manually. The severity depends on the persistence of the Fault that disappears as a message from another datapoint, but remains in the queue as a Fault that occurred in an annoying way, and that must be cancelled only with a confirmation of "yes" on the RadioButton datapoint.
If you need more useful information I am at your complete disposal.

Hello!
I decided to write to you even though I don't know if you have the time and desire to read and answer me since you haven't answered me since the first time.
I'll try again, because I want to believe in the community.
I'm sorry to tell you that your changes don't work!
The one to enable names in English can't change the language by changing the name from short to long, there must be some other parameter, and also the long is more annoying than the short.
The one to choose to check the uniqueness of the Entities, going from the name to the ID doesn't solve the problem because I fear that you are relying on the OpLine instead of the real ID. The 3-digit OpLine doesn't change by changing the Path from Heating Circuit 1 to 2. The real ID is another 4-digit numeric code that changes when the Path changes for the same entity type.
With my tricks, which I explained to you the other times, and above all because even if you say that your integration is not powerful, in reality I have used it for 18 entities and it works very well without problems.
I really miss only that Entity, which I explained to you the previous time, and which has a Type RadioButton with Enum Buttons that has no correspondence on HATypes.
I REPEAT THAT IF YOU SHARE YOUR DOCUMENTATION ON SIEMENS APIS WITH ME I WILL TRY TO GIVE YOU A HAND, but I see that you are not responding to this invitation; patience; I will get over it if you no longer respond, even if I am sorry.
Hoping to have stirred your curiosity,
Kind regards Maurizio.


fix parent link of vanillate O-demethylase (anaerobic) activity

This was introduced with #12211:
id: GO:0018487
name: vanillate O-demethylase (anaerobic) activity
def: "Catalysis of the reaction: vanillate + Co+ = Co3+-CH3 + 3,4-dihydroxybenzoate." [UM-BBD_reactionID:r0758]
xref: UM-BBD_reactionID:r0758
---> is_a: GO:0016623 ! oxidoreductase activity, acting on the aldehyde or oxo group of donors, oxygen as acceptor
is_a: GO:0032451 ! demethylase activity

The UM-BBD page http://eawag-bbd.ethz.ch/servlets/pageservlet?ptype=r&reacID=r0758 gives only the demethylase EC, and the only ref PMID 11344134 does not have a hint of any other activity involving aldehydes or ketones.

That was a bit cryptic by me. Note the wrong oxidoreductase parent.

So I could just remove the GO:0016623 parent?
The EC 2.1.1.- used at the EAWAG maps to methytransferases so I can change to that if we have it (I'm pretty sure we do). https://www.qmul.ac.uk/sbcs/iubmb/enzyme/EC2/1/1/

So I could just remove the GO:0016623 parent?

Exactly, in my opinion.

done


Error handling

Hey there! looks like a great package :) Would you be open to pull requests?
I've added some error handling (reporting NOAA errors when the API responds with errors), e.g.:
In [3]: coops.get_data(
   ...:     begin_date="20150727",
   ...:     end_date="20150910",
   ...:     stationid="8723214",
   ...:     product="water_level",
   ...:     bin_num=1,
   ...:     units="metric",
   ...:     time_zone="gmt",
   ...:     datum='navd88')
   ...:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
...
ValueError:  The supported Datum values are: MHHW, MHW, MTL, MSL, MLW, MLLW, NAVD, LWI, HWI

I've also changed sys.exit to raise ValueError (see guidance here: https://docs.python.org/3/library/exceptions.html#ValueError)

Hey @delgadom, I am certainly open to pull requests, feel free to suggest them as issues or ideas arise.
Agree with both your changes, this is a cleaner solution to handle errors and their output!


Keychain.app

It would be awesome if Alpha would integrate a little Keychain.app like the Mac app, with all the keys being in the Sandbox of the app, with ability to delete or modify or add.

Sounds like an valuable plugin. I'm gonna implement it when some heap plugin fixes will be done.
@Legoless Is idea of plugin acceptable for Alpha framework?

Yeah, this definitely works as an plugin.

@Legoless Isn't 0.4.0 too far for this simple plugin? We have some volume issues on 0.3.0.

Yeah, that is why I moved this into 0.4.0, but it can be implemented earlier as well.

@stuffmc Can you provide short list of requirement features for keychain plugin? It will significantly boost plugin development.

• Display the list of Keychain Items (keys) in a TableView
• Tap to show the password — probably by entering TouchID
• Swipe to Delete
That's a good start. Later you can add "Add a new Keychain Item", but start with Display & Delete first, which is why I need and can test. I don't actually need "Show the password" at first, if you want to prioritise.

Should we allow secure data changing?

I don't care about this for now :)  but yeah, it's obviously on the road map later. For now, the fastest you give a list of keys (I don't need to see the password) and swipe to delete, the better :)

I wouldn't make Touch ID for showing the password (touch to show is okay) - maybe later to access entire Alpha interface, to lock it for other testers if so required. But that's still far in the future.

There are no reason to request TouchID since application can directly provide you keys.
There are a lot of security mechanism in iOS and about 14 general keys to provide filesystem security (plus a great bunch of keys for each file, etc.). If we can bypass all this guards we shouldn't request the same security layer we passed before.
Another question is should we provide access to this plugin remotely?

Yes, we should provide it remotely for now, or add an option for developer to switch remote access off. Which should be available for all plugins anyway.

It's a security vulnerability, isn't it?
I think we should disable remote access by default since most part of users wouldn't use remote feature of Alpha.

Alpha itself is a security vulnerability if left inside the app. Only meant for testing. But I do agree that remote should be off by default.

offtop
@Legoless can we have some chat for Alpha project? It would be great to synchronise features/fixes and transfer some knowledge about architecture.

The message will be deleted after the answer.

Yeah, contact me at thelegoless on Skype

Touch base... Guys have something?

No ups from me. I'm looking for fixing existing staff as the first priority vs new functionality.
Feel free to contribute your PL.

@stuffmc, I've pushed 0.2.7, which has a basic Keychain plugin, read-only for now. Going to add delete, edit and add functionality later. Check it out, if this is something you were looking for.

@Legoless Great! Will do as soon as I need it again. Could be anytime in the project I'm busy for now.


fixes for crash on HdrHeap::attach_block

@SolidWallOfCode here is the proposal for our problems with HdrHeap::attach_block

I'm going to -1 this patch because I think it treats the symptoms, not the problem. The latter seems to be TS-4256.

That's fine, thanks Alan for all investigation. I think we later identified a possible fix with the plug-in: https://github.com/apache/trafficserver/pull/511


Add license to project

Hi,
We are interested in using this project as part of a project for a university course, but we noticed it has no license. Would it be possible to add an open source license?
Thank you!
PS. More info about projects without license here.

Hi. Are you interested in the Bomberman engine or the upcoming 2018 challenge engine? We will add the licences after we've determined the relevant licence for this type of project.

We have updated the license on all repositories for the Entelect Challenge. You're free to use the engines for your course. Additionally, we'd love to see how it's being applied in your course content!

@poinu Thanks for your interest. We hope everything is in order. Again, we'd like to see how you apply the code in your work.


I have a problem with running my Flutter apllication related to jcenter issues 🤯

FAILURE: Build failed with an exception.
* What went wrong:
Could not determine the dependencies of task ‘:app:processDevDebugResources’.
Could not resolve all task dependencies for configuration ‘:app:devDebugRuntimeClasspath’.
Could not resolve io.grpc:grpc-core:[1.44.1].
     Required by:
         project :app > project :cloud_firestore > com.google.firebase:firebase-firestore:24.2.2 > io.grpc:grpc-android:1.44.1
         project :app > project :cloud_firestore > com.google.firebase:firebase-firestore:24.2.2 > io.grpc:grpc-okhttp:1.44.1
Failed to list versions for io.grpc:grpc-core.
Unable to load Maven meta-data from https://jcenter.bintray.com/io/grpc/grpc-core/maven-metadata.xml.
Could not HEAD ‘https://jcenter.bintray.com/io/grpc/grpc-core/maven-metadata.xml’.
Read timed out
* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
* Get more help at https://help.gradle.org/
BUILD FAILED in 1m 35s
Exception: Gradle task assembleDevDebug failed with exit code 1

buildscript {
    ext.kotlin_version = '1.5.31'
    repositories {
        google()
        mavenCentral()
        // jcenter()
    }

    dependencies {
        classpath 'com.android.tools.build:gradle:3.5.0'
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
        classpath 'com.google.gms:google-services:4.3.5'
        classpath 'com.google.firebase:firebase-crashlytics-gradle:2.7.1'
    }
}

allprojects {
    repositories {
        google()
        mavenCentral()
        // jcenter()
    }
}

Changing jcenter to mavenCentral is not working on my project. 🤦♀
Anyone who having the same issue?

I'm having the same issue, I built a release on Friday without issue and my build.gradle has not changed. I have always had maven specified.

I am also having the same issue and cannot build the app for 24 hours.  Multiple other projects on github have lots of the same issue.

I fixed my problem by migrating from jCenter to mavenCentral.
I did change the build.gradle (jcenter -> maveCentral) and related files. It is better to create new project and check the environment of the new flutter project and try applying it on your project.
There was an issue being down on jCenter jcenter status.

Hello @HyunJay52. Glad to see your issue solved. Since this issue is not directly related to Flutter but was rather a problem with jcenter itself, I am closing this issue. If this issue is still reproducible, please share the new error logs and I can reopen it. Thank you.