Step 12 screen shot is Skype for Business Server 2015, not 2019

The final screen shot displays Skype for Business Server 2015 and should be the 2019 screen shot.
Thanks, Tim.

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

ID: e9131450-3053-fff6-0e01-36c7b3fd076f
Version Independent ID: 0ad5a095-cbfd-5bcb-6f75-6f696b8215e7
Content: Prepare Active Directory for Skype for Business Server
Content Source: Skype/SfbServer/deploy/install/prepare-active-directory.md
Product: skype-for-business-itpro
GitHub Login: @kenwith
Microsoft Alias: kenwith

@officedocsbot assign @scanum

Hello @timhaintz
Thank you to taking the time to submit feedback.
Actually, this article is for both version, this is known issue described here https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/1861 and is currently in the process of being fixed.
Thank You!

@timhaintz  We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close


Switch from git submodules to Meson wrapfiles

This pull request is based on #4. All my pull requests will be based on #4 until (if) it will be merged because working with CRLF files is difficult on my Linux system.
Meson wrapfiles should be preferred in a Meson project. They make integration with Meson simpler (no need to specify include dirs or other things, it's all in the dependency object). They are also use versioned releases of libraries and not arbitrary commits. Versioned libraries can be more stable and reliable. If a wrapped library is already installed on the system, it will be used instead of the wrap one which is an advantage over git submodules.
This adresses one of the TODO items.

The change in itself seems ok, though I encountered an issue while testing it:
The meson build file fetched from wrapdb https://github.com/mesonbuild/wrapdb/blob/master/subprojects/packagefiles/emilk-loguru/meson.build seems to have a syntax error:
subprojects/loguru-2.1.0/meson.build:74:0: ERROR: List item must be one of <class 'str'>, not <class 'list'>
This is probably due to the definiton of compile_opts and api in declare_dependency:
compile_opts = [
  '-DLOGURU_SCOPE_TEXT_SIZE=@0@'.format(get_option('loguru_scope_text_size')),
  '-DLOGURU_FILENAME_WIDTH=@0@'.format(get_option('loguru_filename_width')),
  '-DLOGURU_THREADNAME_WIDTH=@0@'.format(get_option('loguru_threadname_width')),
  '-DLOGURU_SCOPE_TIME_PRECISION=@0@'.format(get_option('loguru_scope_time_precision')),
]

api = []

emilk_loguru_dep = declare_dependency(
  include_directories: '.',
  link_with: emilk_loguru_lib,
  compile_args: [compile_opts, api]
)

Fixing these is quite straightforward (just a ' '.join() for compile_opts and defining api = ''), perhaps either of us can submit a patch upstream and then this can be merged.

I think your Meson is too old.

I see. Although now the point is that the wrapdb meson.build lists meson_version: '>=0.50', therefore I think the minimum version there should be bumped. From my tests it builds correctly by having at least 0.63.0. I suspect it has to do with argument flattening happening, but I wasn't able to find any evidence documenting this change in behaviour. I suppose I can also add a word of warning about this in the README.

You can set the minimal Meson version of this project to >=0.63.0.


Add action to check that domain name linked to given account

Instead of adding core mechanism, as planned in GolosChain/cyberway#140, GolosChain/cyberway#157,  we can simply add checking action to system account. That action should be added to transactions, where someone wants to ensure domain name(s) still linked to account(s) at a moment when action applied

PR #57


Uncaught exception when task is finished.

When I run any task in my file, I get the following error when the task completes succesfully:
ERROR: Uncaught exception in luigi
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/luigi/retcodes.py", line 74, in run_with_retcodes
    worker = luigi.interface._run(argv)['worker']
  File "/usr/local/lib/python2.7/dist-packages/luigi/interface.py", line 238, in _run
    return _schedule_and_run([cp.get_task_obj()], worker_scheduler_factory)
  File "/usr/local/lib/python2.7/dist-packages/luigi/interface.py", line 198, in _schedule_and_run
    logger.info(execution_summary.summary(worker))
  File "/usr/local/lib/python2.7/dist-packages/luigi/execution_summary.py", line 418, in summary
    return _summary_wrap(_summary_format(_summary_dict(worker), worker))
  File "/usr/local/lib/python2.7/dist-packages/luigi/execution_summary.py", line 356, in _summary_format
    str_output += '{0}\n'.format(_get_str(group_tasks[status], status in _PENDING_SUB_STATUSES))
  File "/usr/local/lib/python2.7/dist-packages/luigi/execution_summary.py", line 147, in _get_str
    params = _get_set_of_params(tasks)
  File "/usr/local/lib/python2.7/dist-packages/luigi/execution_summary.py", line 190, in _get_set_of_params
    params[param] = {getattr(task, param[0]) for task in tasks}
  File "/usr/local/lib/python2.7/dist-packages/luigi/execution_summary.py", line 190, in <setcomp>
    params[param] = {getattr(task, param[0]) for task in tasks}
TypeError: unhashable type: 'list'

Any suggestions as to what this might be?

Something is a list but it shouldn't be.

Seems like a luigi parameter in one of your tasks is of type list. Luigi is attempting to write a summary of the task and stumbling whilst trying to hash this parameter. You MIGHT be able to work around this issue and keep your list parameter by declaring the parameter as insignificant with [significant=False]. Making a parameter insignificant ignores it when testing task equality and excludes it from certain reporting output. If that doesnt work you may be forced to refactor that list as some kind of alternative structure that is hashable (maybe a comma-separated string?)


Fixing the issue with AWS_REGION credentials.

Pipelines now use EC2_REGION, can't use string variable as credentials as it is https://github.com/fusor/mig-ci/pull/57.

LGTM


Getting protein changes

Great package, it would be useful to be able to get protein changes from GnomAD exomes for downstream tasks. Is this something you support?

Hi @Michael-Bar,
Thank you for the feedback!
Unfortunately, this is not on the roadmap for now.
Please let me know if you have any further questions.
Best,


cleanup header guard names to be consistent

I agree that having consistent header guards is good, but underscore followed by an upper case character are reserved names. I suggest simply using PICO_CONFIG_H etc.


Intellisense suggest shouldn't catch all.

Suggest has a catch all:
https://github.com/microsoft/Power-Fx/blob/ace1d1fa33c34eb40a96404de59b67562779250f/src/libraries/Microsoft.PowerFx.Core/Texl/Intellisense/Intellisense.cs#L65
Intellisense should never throw - even on invalid input.  Any exception is a bug.
This can be hiding real issues.  Instead, let the exception bubbleup and a top-level can catch and log it.  LSP already does this today with #251.

Verify we're not breaking power apps - we may need to add a top level catch there, or something.
This should let us remove Exception field on result.

Fixed with https://github.com/microsoft/Power-Fx/pull/1576


[POC] Feature/issue 14 with-transaction macro

Just a POC, do not merge.
Here's a quick first pass at a with-transaction macro, based off the implementation from conman.
The macro optionally accepts a defrefable, which is handy when the developer is using a system like mount to manage the gungnir ds state.

[ ] Add multiple ds tests
[ ] Write documentation

Because the fixtures clear the database after every is, I had to stuff everything into one is which is ugly but unavoidable unless the each fixtures are disabled.
next.jdbc/with-transaction returns a HikariProxyConnection object which implements the java.sql.Connection interface, hence the updates to the specs.

Codecov Report

Merging #15 into master will increase coverage by 0.41%.
The diff coverage is 96.13%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   90.42%   90.84%   +0.41%
==========================================
  Files          10       11       +1
  Lines         752      819      +67
  Branches       25       25
==========================================
+ Hits          680      744      +64
- Misses         47       50       +3
  Partials       25       25

Impacted Files
Coverage Δ

src/clj/gungnir/database.clj
85.60% <93.39%> (+0.88%)
:arrow_up:

src/clj/gungnir/factory.clj
100.00% <100.00%> (ø)

src/clj/gungnir/query.clj
94.28% <100.00%> (+0.56%)
:arrow_up:

Continue to review full report at Codecov.

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

From Slack:
kwrooijen
6:21 PM
I think we need to do the following things.  what are your thoughts?

Think of a nice structure for aggregating the executions in data. If a more functional way is desired we can create honeysql-like wrapper functions to build the structure
We will need a transaction-connection dynamic definition which will take precedence over any other connection (for the binding in transact)
In the exception handler we need to check if transaction is set, if it is then we need to throw the data instead of returning it
Create a transaction function in factory. This function will take the datastructure and execute each one in sequence. If any fail, extract the data with ex-data and return that with the pairing identifier key. (edited)

Casey
6:36 PM
Sounds good
6:37
I find it helpful to write an example of what I think the user api would look like, I suppose we could do that with the elixir account transfer example

Closing this to continue the discussion in the linked issue #14. This PR is a dead end.


How to use simple share success callback ?

Hello, i success to share through the whatsapp, but i don't know how to use the callback,
because i want use success callback to close my modal box. thanks very much.
Share.open(shareOptions);

Hi

Share.open(shareOptions).then((info) => {
  //  ok close
}).catch((err) => {
  //  error close
});

@EstebanFuentealba does that work for ios and android?

@EstebanFuentealba Is there something similar for shareSingle()?  Nothing seems to be fired when I try:
Share.shareSingle(shareOptions).then((info) => {
  //  ok close
}).catch((err) => {
  //  error close
});

Yeah! same issue here... :(
Nothing is firing


File Not Closed in OutlookMessageLoader of langchain_community Library

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_community.document_loaders import OutlookMessageLoader
import os

file_path = "example.msg"

loader = OutlookMessageLoader(file_path)
documents = loader.load()
print(documents)
try:
    os.remove(file_path)
except Exception as e:
    print(e)

Error Message and Stack Trace (if applicable)
Traceback (most recent call last):
File "test.py", line 16, in
os.remove(file_path)
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'example.msg'
Description
Describe the bug
It seems that the OutlookMessageLoader does not close the file after extracting the text from the .msg file.
To Reproduce
Steps to reproduce the behavior:

Use the following example code:

from langchain_community.document_loaders import OutlookMessageLoader
import os

file_path = "example.msg"

loader = OutlookMessageLoader(file_path)
documents = loader.load()
print(documents)
try:
    os.remove(file_path)
except Exception as e:
    print(e)

Run the code and observe the error.

Expected behavior
The file should be closed after processing, allowing it to be deleted without errors.
Error
[WinError 32] The process cannot access the file because it is being used by another process: 'example.msg'

Additional context
I looked into the email.py file of langchain_community.document_loaders and found the following code in the lazy_load function:
import extract_msg

msg = extract_msg.Message(self.file_path)
yield Document(
    page_content=msg.body,
    metadata={
        "source": self.file_path,
        "subject": msg.subject,
        "sender": msg.sender,
        "date": msg.date,
    },
)

It seems like the file is not being closed properly. Adding msg.close() should resolve the issue.
System Info
Langchain libraries:
langchain==0.2.2
langchain-community==0.2.3
langchain-core==0.2.4
langchain-openai==0.1.8
langchain-text-splitters==0.2.1
Platform: windows
Python: 3.12.3

@MatanelAbayof please close this issue as this has been fixed.


Hi, is this project dead?

Hi, is this project dead?

Looking for maintainers and contributors, see #11

@rahul-thakoor I wish I can be a developer myself. Oh well.

The project is not dead :-)

@dcroote In my opinion in the near future there should be a "write once run everywhere" where it works on other miniboards such as the Odroid etc :)

Hi @Raj2032 - I'm not familiar with those boards specifically, but for compiling a single source against an arbitrary target I'd highly recommend cross. I used it recently to cross-compile this crate for the Raspberry Pi Zero W.

@dcroote I see thanks mate

I think @Raj2032 is looking for a generic hardware abstraction layer for other mcus?
perhaps checkout out https://github.com/rust-embedded/awesome-embedded-rust


⚠ Bofrost.lu has degraded performance

In b8c86b0, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1310 ms

Resolved: Bofrost.lu performance has improved in c829fe5 after 16 minutes.


Use setup file

It would be nice if the linter could use a setup.cfg or tox.ini etc. if found in the root project directory.
Normally flake8 will read these automatically if there's one in the same directory as it is invoked from, but there is also a --config option to specify a path.

I can put together a PR for this; I'm just not sure of how to resolve the package settings. (Personally, I don't think there should be any package-level linter config and instead it should just rely on the normal flake8 configuration behavior.)

@yellottyellott @AtomLinter is actually making a copy of the file in a tmp directory, which can mess up a bunch of stuff like this. I mentioned it in AtomLinter/Linter#213 but it seems unlikely to be changed. Unfortunately, that means individual linter packages have to reimplement this kind of logic :poop:


How to use placeholder

I want to use tensorflow model in android.But I don't know how to use placeholder.Can you help me?

Please refer this link

@amitshekhariitbhu Thanks,but,I know how to use placeholder in python.But in java,I don't know how to use it.

@JcmeLs ok I will check few reference and tell u

@amitshekhariitbhu Thank you very much!Looking forward to your reply！Thanks again!

Hi~I did not find argmax in official documents.Do you have any suggestions?

I used C++ API to run my model.So close.


config: read CompanyIdentification from the file config

Fixes: https://github.com/moov-io/paygate/issues/525

Codecov Report

Merging #528 into master will decrease coverage by 0.05%.
The diff coverage is 48.71%.

@@            Coverage Diff             @@
##           master     #528      +/-   ##
==========================================
- Coverage   53.71%   53.65%   -0.06%
==========================================
  Files         100      100
  Lines        3742     3757      +15
==========================================
+ Hits         2010     2016       +6
- Misses       1378     1383       +5
- Partials      354      358       +4

Impacted Files
Coverage Δ

pkg/transfers/fundflow/first_party.go
60.60% <0.00%> (-1.90%)
:arrow_down:

pkg/validation/microdeposits/micro_deposits.go
71.23% <25.00%> (ø)

pkg/config/odfi.go
38.66% <33.33%> (+0.20%)
:arrow_up:

pkg/achx/files.go
78.57% <50.00%> (-6.05%)
:arrow_down:

pkg/achx/batch.go
77.77% <100.00%> (ø)

pkg/achx/ppd.go
74.68% <100.00%> (ø)

pkg/transfers/router.go
59.42% <100.00%> (+0.19%)
:arrow_up:

pkg/validation/microdeposits/router.go
71.59% <100.00%> (+0.32%)
:arrow_up:

Continue to review full report at Codecov.

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


chore: add network switch test

closes #417

@Tbaut thank you for the review and suggestion! Please take a look on updated version

Great!
Let's do more a couple more checks. The issue mentions (maybe not clearly): Let's make sure there's a Multisig in the MultiProxy dropdown for rococo (checking and it's there). We can also check that the url contains a adress=[the expected multisig]. And then make sure on Kusama there's no multiproxy selector, and no address= in the url bar. Check the landing page message (with buttons to create/watch)
Maybe for another issue, but I'm thinking, we should check that

add a watched account that has a multisig on Rococo, and add an account that has a multisig on Kusama
a certain multiproxy is selected automatically on Rococo (this we kind of see it already, in the current test)
go on the home page
Switch network
verify that the other multiproxy is selected by default.

Great! Let's do more a couple more checks. The issue mentions (maybe not clearly): Let's make sure there's a Multisig in the MultiProxy dropdown for rococo (checking and it's there). We can also check that the url contains a adress=[the expected multisig]. And then make sure on Kusama there's no multiproxy selector, and no address= in the url bar. Check the landing page message (with buttons to create/watch)
Maybe for another issue, but I'm thinking, we should check that

add a watched account that has a multisig on Rococo, and add an account that has a multisig on Kusama
a certain multiproxy is selected automatically on Rococo (this we kind of see it already, in the current test)
go on the home page
Switch network
verify that the other multiproxy is selected by default.

I updated the code:

check for multisig in dropdown
Address verification on rococo
Kusama contains no address in the URL
Kusama has no multisig selector
Kusama has a connect button

About the other suggestions, I believe that the test should test a specific functionality like here we are testing the network switch (it's more like an integration test) or complete user flow. Switching the pages back and forth doesn't bring an intuitive understanding of the purpose of the tested functionality. So let's be precise about the topic and scope of the test.
The scenario you shared can be integrated as a separate test as you want to make sure that the multiproxy selector is working as expected during the network switch. And in my current test case I focus on the network switch behaviour


ENYO-3097: Spinner can live inside Button

Enyo-DCO-1.1-Signed-off-by: Blake Stephens<EMAIL_ADDRESS>

Closing out as request for this change was deferred.


Fix https json authFor.  Reader fromConfig throws IllegalArgumentAcce…

docker login creates a json auth config file without a scheme on the host.  There was code to handle this but it assumed the method would return null on failure.  The method fromConfig throws and IllegalArgumentException if it is unable to find the configuration.  I've update the method to handle the fallback case in the catch block.

Codecov Report

Merging #836 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #836      +/-   ##
============================================
- Coverage      66.5%   66.44%   -0.07%
  Complexity      742      742
============================================
  Files           166      166
  Lines          3132     3135       +3
  Branches        357      357
============================================
  Hits           2083     2083
- Misses          892      895       +3
  Partials        157      157


Fix not able to input underscore.

Proposed Changes
Change the underscore unicode in keyMap.
Testing

tapOn: input
inputText: foo_bar

Issues Fixed
https://github.com/mobile-dev-inc/maestro/issues/365

Could you please exclude version number changes from this PR?

Thanks for pointing out the solution! Since you did not update PR I just pushed the fix myself

Note that what got pushed in this PR is this change:
'_' to keyPressShiftedToEvents(95),
but from the comments, I  thought people agreed that the change should be:
'_' to keyPressShiftedToEvents(45),


Olm to-device event format missing sender field

Link to problem area: https://spec.matrix.org/unstable/client-server-api/#molmv1curve25519-aes-sha2
Issue
The example to-device payload is missing the sender field which is present in actual implementations.
Expected behaviour
Presumably the example should be modified to mention the sender field.

The sender field is set by the server, and will be present in every to-device message that is received.  The example there in the spec shows the things that are set by the sender (event type and content), though not in the exact format that the sender sets them in.  And it's not ideal that someone needs to know more stuff about to-device messages in general to know that there will be a sender field available.  So, the documentation could be improved there.  But I'm not entirely sure that just adding sender to the example is the right thing.  Will think about it more.

I don't think the example as it stands makes sense either way, since it is neither the thing the client sends (the client only sends the content field value), nor the thing received from the server.

Agreed.  I think maybe we should just show the content in the code block, say that it's sent with type m.room.encrypted, and mention the fields that the server will add.


Implement Docker best practices - Round 2

Problem
Hadolint is a tool which analyzes Dockerfile and provides applicable best practices.  Version pinning reduce failures due to unanticipated changes in required packages. The CHIP integration Docker images don't have version pinning implemented.
Summary of Changes
These changes implement version pinning best practice as well as reduce some instructions like using WORKDIR Docker  and --no-install-recommends apt instructions

Hi @electrocucaracha I re-opened this as the docker builds don't work. I have a couple suggestions coming (just running a test build now), but would you be able to run build-all.sh in the integrations/docker/images path beforehand? We don't have great CI to verify them yet unfortunately :(

Hi @electrocucaracha I re-opened this as the docker builds don't work. I have a couple suggestions coming (just running a test build now), but would you be able to run build-all.sh in the integrations/docker/images path beforehand? We don't have great CI to verify them yet unfortunately :(

@woody-apple I ran build.sh script on every folder, I'm going to try to run build-all.sh so see if there is a difference.

Going to verify this... not sure it builds yet...

nope, doesn't build...
 => ERROR [7/8] RUN set -x     && git clone --depth=1 https://github.com/google/bloaty.git     && mkdir -p bloaty/build     && cd bloaty/build     && cmake ../     && make -j8     && make install      3.0s


ISPN-10611 Jenkins ignores server/runtime test failures

https://issues.jboss.org/browse/ISPN-10611
Some new test failures should appear, the thread leaks should be fixed by #7107

Merged, thanks


Call count for mocks

Are there any plans to add the ability of verifying how many times an expectation was called? Sometimes a simple yes/no is not enough to track down some unexpected behaviour.
Keep up the great work. 🙌

Is this not sufficient?
verify {
    repeat(5) { myAmazingApi(isAny()) }
}

At the moment, we'll consider that using repeat is sufficient.
Feel free to reopen this issue with a use case if that is not enough.


Enable vsock as a transport for gRPC

This patch is based on https://android-review.googlesource.com/c/platform/external/grpc-grpc/+/1205919 where we enabled vsock support in the AOSP gRPC tree
We would like to bring this support back to the mainline tree
Any and all help in getting this reviewed and merged would be greatly appreciated.

I've added the necessary labels and triggered the tests to run. It looks like there are a lot of failures, so please take a look at them.

I looked at some of the build failures, looks like I had a few compile errors that my local build didn't catch. I think I addressed them, but happy to get new results & keep iterating. Unfortunately, I don't have access to anything but a Linux host.

Please also update the PR description into a form that will be appropriate for the gRPC release notes.

What's a good PR to look at for this?

Reviewed 28 of 28 files at r1.
Reviewable status: all files reviewed, 8 unresolved discussions (waiting on @egranata)

BUILD, line 753 at r1 (raw file):

        "src/core/lib/iomgr/lockfree_event.cc",
        "src/core/lib/iomgr/logical_thread.cc",
        "src/core/lib/iomgr/novsock.cc",

To match our existing naming convention, please call these two files resolve_vsock_linux.cc and resolve_vsock_fallback.cc, respectively.

Done

BUILD, line 1060 at r1 (raw file):

        "src/core/ext/filters/client_channel/local_subchannel_pool.cc",
        "src/core/ext/filters/client_channel/parse_address.cc",
        "src/core/ext/filters/client_channel/parse_address_novsock.cc",

Now that we're needing more conditional compilation here, we should probably bite the bullet and move the parse_address library from the client_channel tree into the iomgr tree, since we're not really supposed to have any ifdefs outside of that tree.

I can give it a shot, but I don't know if I am familiar enough with gRPC to do a good job at it.

src/core/lib/iomgr/resolve_address_posix.cc, line 60 at r1 (raw file):

  }

  if (name[0] == 'v' && name[1] == 's' && name[2] == 'o' && name[3] == 'c' &&

Please remove this. The code we have above this for "unix:" URIs is a really ugly hack that I've been meaning to get rid of. And even though I haven't yet gotten around to doing that, I don't want to repeat the existing mistake here.
The iomgr name resolution code is used both on the client and on the server, but the ability for it to understand "unix:" or "vsock:" names is only needed on the server side, because the client side only uses this code when it already knows it's a DNS name -- if you specify a "unix:" or "vsock:" URI on the client side, it selects a different resolver that does not call the iomgr name resolution code in the first place.
So instead of adding this hack here, we should instead move this check into the server code, which is the only place where it's actually needed.
For bonus points, feel free to move the existing "unix:" code at the same time.

Same as above

src/core/lib/iomgr/sockaddr_utils.cc, line 253 at r1 (raw file):

    case GRPC_AF_UNIX:
      return "unix";
#ifdef GRPC_AF_VSOCK

Why do we need an ifdef for vsock here but we don't need it for unix? It would be better to be consistent.

Because AF_UNIX as a symbol is pretty much defined anywhere (including on Windows, much to my surprise!) but AF_VSOCK is not. So we have to guard its usage. One thing I could do of course is
#ifdef HAVE_VSOCK
#define GRPC_AF_VSOCK AF_VSOCK
#else
#define GRPC_AF_VSOCK (-1)
#endif

just to have the symbol resolve and avoid the ifdef. Would that work better?

src/core/lib/iomgr/sockaddr_utils.cc, line 275 at r1 (raw file):

    case GRPC_AF_INET6:
      return grpc_ntohs(((grpc_sockaddr_in6*)addr)->sin6_port);
#ifdef GRPC_AF_VSOCK

As above, this ifdef should not be needed.

See above

src/core/lib/iomgr/sockaddr_utils.cc, line 277 at r1 (raw file):

#ifdef GRPC_AF_VSOCK
    case GRPC_AF_VSOCK:
#ifdef GRPC_HAVE_LINUX_VSOCK

This ifdef probably is needed, but it would be better not to bury it inside of this file. Instead, I suggest defining a function called something like grpc_vsock_addr_get_port() with a real definition in vsock.cc and a no-op definition in novsock.cc, and then unconditionally call that function here.

You'd still need the #ifdef GRPC_AF_VSOCK (but see above, or happy to take suggestions). Otherwise, I can make that change yes.

src/core/lib/iomgr/sockaddr_utils.cc, line 310 at r1 (raw file):

          grpc_htons(static_cast<uint16_t>(port));
      return 1;
#ifdef GRPC_AF_VSOCK

This ifdef should not be needed.
src/core/lib/iomgr/sockaddr_utils.cc, line 312 at r1 (raw file):

#ifdef GRPC_AF_VSOCK
    case GRPC_AF_VSOCK:
#ifdef GRPC_HAVE_LINUX_VSOCK

Same comment as on line 277 above.

See above. It seems like a lot of the feedback is about how to avoid the #ifdef GRPC_AF_VSOCK sprinkled around. I am happy to figure out a good path for that and go with it. What do you think?

Friendly ping

Ah, @markdroth thanks for getting back to me; I will attempt to update the pull request soon & get back to you w/ an update
Apologies for the delay as well; this did fall off my radar w/ WFH & all


Updated and renamed upgrading.rst to upgrading.md (v2)

Apologies for accidentally creating a new PR for the updated upgrading.md file with review comments.
Replaces https://github.com/wagtail/wagtail/pull/8761

I am happy with how you have added my name to the Contributors list. Thanks for the opportunity. One thing, I see 'text' and 'console' appear in the output. I hope they are correct.

Thanks for the heads up. Looks like something went wrong there. I'll fix it up.

In an attempt to next the code snippets in the list - looks like it treated the while thing as a code block.

PR up to fix the above issue https://github.com/wagtail/wagtail/pull/8793


CLI: Ambiguous parameter names such as -no are misinterpreted as positional parameter values

Steps to reproduce
# -no is ambiguous: could be -NoProfile, -NoLogo, ...
powershell -no -command get-date

Expected behavior
An error complaining about the ambiguity of the parameter name and refusal to continue.
Actual behavior
PS Core and Windows PS behave differently, due to the recent Core change to default to -File instead of -Command:

PowerShell Core:

The argument '-no' to the -File parameter does not exist. Provide the path to an existing '.ps1' file as an argument to the -File parameter.
PowerShell v6.0.0-beta.4
Copyright (C) Microsoft Corporation. All rights reserved.

Windows PowerShell:

-no : The term '-no' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was
included, verify that the path is correct and try again.
At line:1 char:1
+ -no -command get-date
+ ~~~
    + CategoryInfo          : ObjectNotFound: (-no:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Environment data
PowerShell Core v6.0.0-beta.4 on macOS 10.12.5
PowerShell Core v6.0.0-beta.4 on Ubuntu 16.04.2 LTS
PowerShell Core v6.0.0-beta.4 on Microsoft Windows 10 Pro (64-bit; v10.0.15063)
Windows PowerShell v5.1.15063.413 on Microsoft Windows 10 Pro (64-bit; v10.0.15063)

-no might be a valid script file, so I don't think we want check against ambiguous intended parameters (-Command didn't do this either).  However, we can improve the error message.  How about:
-no : The term `-no` is not recognized as a valid script file.  Check the spelling of the name, or if a path was
included, verify that the path is correct and try again.

@SteveL-MSFT:
For consistency and predictability, you should never assume that --prefixed tokens are parameter values, unless they follow a recognized, unambiguous parameter name (such as -NoProfile):

If a positional token looks like a parameter name - i.e., if it starts with - followed by a letter -  it should be interpreted as such.
If that parameter name is not recognized / cannot be resolved unambiguously, an error should occur, just as would happen inside PowerShell with something like get-item -f *.txt (Get-Item : Parameter cannot be processed because the parameter name 'f' is ambiguous. Possible matches include: -Filter -Force.).

If it a positional parameter value happens to start with - (e.g., a script file named -no), the explicit parameter name must be used to disambiguate: -File -no.

@SteveL-MSFT: I just realized that even PowerShell's internal parameter parsing has inconsistencies - see #4576

I have a change that I think addresses your valid concern:
PS> .\powershell.exe -no -file foo
Invalid argument '-no', did you mean:

        -nologo
        -noexit
        -noprofile
        -noninteractive

Need to add tests


refactor checkCountAndReduceRecordsLimit and bugfix to_many_through_sql + generic associations

Refactor the resolver function checkCountAndReduceRecordsLimit to use a helper function.
This includes:

move recordLimit reduction and conditional error generation into helper
leave current resolver function, but make it invoke the above helper
rename arg query to resolverName
in generic associations and the hasManyThroughShortAndRememberableNameSql.. (to_many_through_sql_cross_table) do the count in the respective resolver and then invoke the above helper.


enable Dependabot v2

https://github.blog/2020-06-01-keep-all-your-packages-up-to-date-with-dependabot/

I think we're fine with manual dependency management and updates for now. Thanks!


"Your account is not active" using OpenAI API, although there is no issue (screenshot included)

Having a stranger issue (followed the YouTube tutorial to make sure everything was correct), where when trying to use an OpenAI model, no matter if I set the default key in .env.local or in the chat session, I get an error "Your account is not active, please check your billing details on our website."
Tested my OpenRouter models and they work fine, and also tested multiple OpenAI API keys, as well as confirming they work in MacGPT.
Any suggestions?

You should check your OpenAI account settings to see if there are any unpaid bills. Make sure that a payment method for settling bills is set up.

Check OpenAI Account via their site. That's an issue with their billing.


Replace axios with fetch

I don't see much of a need to use axios over the built-in fetch as our HTTP client, so we should remove another dependency by replacing it

💯 I really don't understand the JS ecosystem sometimes


Handle newline \n\n in the string

Here is the code that shows the problem.
from redlines import Redlines
import markdown
from IPython.display import display, Markdown,  HTML
test_string_1 = '''
Happy Saturday,

Thank you for reaching out, have a good weekend

Sophia
'''
test_string_2 = '''Happy Saturday,

Thank you for reaching out. Have a good weekend.

Sophia.'''

diff = Redlines(test_string_1 ,test_string_2, markdown_style="red")
html=markdown.markdown(diff.output_markdown)
display(HTML(html))

When displaying Markdown in a Jupyter Notebook or on a webpage, we need to convert it to HTML. For most Markdown packages, they will convert two or more consecutive '\n' to a new
. The example above is very common when comparing an original email to a proofread and corrected text. The current code cannot handle it properly.
Because diff.output_markdown may include
weekend\n\nSophia \nweekend.\n\n\n\nSophia.'

The markdown package will add a

 to replace the \n\n and it turned into
.....

......

The  is broken.
The result looks like this.
Happy Saturday,Thank you for reaching out, have out. Have a good weekendSophia weekend.Sophia.
Solution:

For delete and replace, and for _seq1, remove all the \n in the string because we will keep \n in _seq2, and the markdown will add the right number of
 in the HTML.
For insert and replace, and for _seq2, move the substring with two or more \n out of the tag  or  .

The new result looks like this.
Happy Saturday,
Thank you for reaching out, have out. Have a good weekendSophia weekend.
Sophia.

There may be a better solution. Will PR later.


C++23's Lambda Attribute Syntax (and why it's needed)

https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2021/p2173r1.pdf

https://compiler-explorer.com/z/hhMYYo35q

recorded

Coming in episode 386


Install gpu Ubuntu16.04 misses crucial steps: "unable to locate package nvidia-driver-450"

URL(s) with the issue:
https://www.tensorflow.org/install/gpu?hl=ur#ubuntu_1604_cuda_110
Description of issue (what needs changing):
19th line:
sudo apt-get install --no-install-recommends nvidia-driver-450
leads to error:
E: Unable to locate package nvidia-driver-450

Was able to reproduce the issue. Please take a look at the below screenshot for reference

Thanks!

@GrigoriiTarasov,
With reference to issue #44301, ignoring the sudo apt-get install --no-install-recommends nvidia-driver-450 command seems to work in this case.
Could you please skip that particular step and check if you are able to install TensorFlow. Thanks!

@amahendrakar
Thanks a lot!
I confirm that skipping 19th line works.


directly open "Camera permission" settings page on Android

Usecase:

Android
User explicitly denied access to camera before
App gets camera_access_denied error from native platform
User needs to go to: /Settings/Apps/APPNAME/Permissions/Camera

I could not yet find which method would navigate there in current build 0.3.1

You might be able to achieve the desired result by calling the customMessage method on the OpenSettingsPlusAndroid instance. Here's an example of how to do this:
ListTile(
    onTap: () => settings.sendCustomMessage(
        'android.settings.APPLICATION_DETAILS_SETTINGS',
    ),
    title: const Text("Open app details"),
    trailing: const Icon(Icons.chevron_right),
),

If this works for you, we can include this functionality in the next release. If you're not being redirected to the exact page you want, try changing the message to match your use case and reply here so we can further investigate the issue.


change appearance of unavailable catalogs in CatalogChoiceField

for #629

coverage: 90.77% (-0.02%) from 90.792% when pulling 9d37ffdef38f4841a8e6c92b6ee4c4b4c484fb94 on feature/629-appearance-unavailable-catalogs into 6824ca3178f7423eac02ff16b2f63964f659c1fa on dev-1.11.0.


Mezgoodle functions

GetUser();
Book();
CheckBookings();
Review();

So, to resolve conflict I deleted GetUser() function, if it disappear, we will just commit again into master

Oh no, now also GetRoom, for what?


Loading-images-using-streams

Created a sample app which shows how to load images using streams in flutter

@AjayThakur12 The repo is only for the dart pad samples.

@divyanshub024  thank you for pointing out, working on how to run this project on DartPad will edit README.


Autoload plugin and Vimscript config support

Configure using vim.g.virt_column allows for configuration in
Vimscript
plugin/ autoloads this plugin, no need for the user to call setup()

This PR is fully backwards compatible. Users can choose to use vim.g.virt_column = ... or require("virt-column").setup(...). I've currently only changed the documentation for this in the README, since the help page only describes the Lua API, so I didn't know if a description in the help doc was needed for this feature.
Thanks in advance for considering this PR!

Ah okay. I based my approach on the following resources:

https://mrcjkb.dev/posts/2023-08-22-setup.html
https://github.com/nvim-neorocks/nvim-best-practices?tab=readme-ov-file#x-dont-4
https://zignar.net/2022/11/06/structuring-neovim-lua-plugins/#lazy-loading-lua-code

They suggest that setup() (or to be fair, more precisely: initializing a module) is an antipattern, and config / initializing modules should be separated.

Marc has a lot of good suggestions, but a lot of people, me included, disagree with him about setup and global variables.
Most plugins do not do it his way.


ppp: cherry-pick of PR 6224 to release-14.12

Please test if you use PPP on release-14.12.
See https://github.com/NixOS/nixpkgs/pull/6224.

No idea what the Travis problem is. Travis gives a lot of false positives so I will ignore it.


[Web] TextField receiving newline on TextInputAction.next

In a form with two consecutive TextFields with TextInputAction.next, pressing Return on the first field, inserts a newLine on the second after advancing focus.
The only way to avoid this problem is setting maxLines=1 on every field, preventing multiline inputs from being used.
Steps to Reproduce

Execute flutter run on the code sample (web Chrome).
Set focus on the first field (with "A")
Press enter/return on the physical keyboard to move focus to next field ("B")
Type "123"

Expected results:
The second field with the content "B123".
Actual results:
The second field with the content "B{newline}123".

Code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);
  @override
  Widget build(BuildContext context) => const MaterialApp(title: 'Input Error Demo', home: Home());
}

class Home extends StatefulWidget {
  const Home({Key? key}) : super(key: key);
  @override
  State<Home> createState() => _HomeState();
}

class _HomeState extends State<Home> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Input Error Demo')),
      body: Column(
        children: [
          TextFormField(
            initialValue: "A",
            textInputAction: TextInputAction.next,
          ),
          TextFormField(
            initialValue: "B",
            minLines: 2,
            maxLines: 2,
            textInputAction: TextInputAction.next,
          ),
        ],
      ),
    );
  }
}

Logs

flutter doctor -v
[✓] Flutter (Channel stable, 3.3.4, on macOS 12.6 21G115 darwin-x64, locale pt-BR)
    • Flutter version 3.3.4 on channel stable at /Users/paulo/Documents/Desenvolvimento/flutter_sdks/flutter_3.3.4
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (13 days ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/paulo/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.72.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.6 21G115 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 106.0.5249.119

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

After investigation, this issue seems to be only reproducible on Web.
I removed the desktop and framework labels because I can't reproduce it on desktop (Linux and MacOS). If someone can, thank you to comment here.

Fix TextField performing both new line and input action flutter/engine#36893

@bleroux, thank you very much!
I was able to reproduce this problem on MacOS and Windows by repeatedly typing "123{return}" to cycle between fields a couple of times.

@PauloZung Interesting.
From my investigation, there is possibly to issues there:

first one, only on web: the 'return' key event from field A insert a new line on field B.
second one, on desktop: because 'B' text field is multiline, when we type 'return' on it both a new line is added and the textInputAction is performed. It might explain your observation :

by repeatedly typing "123{return}" to cycle between fields a couple of times"

Can you try help me to confirm this? To do so, thank you to try to find reproducible steps as you did in your main report. The main point is to find if the new lines inserted in field 'B' happens after using return key on field B (my theory) or also on field A (what we have on Web and that will be fixed by my PR).

@bleroux , just added a debugPrint(value) on the onChanged of both fields and realized that the second hypothesis is correct: Pressing return on the field B, adds a new line on B before the textInputAction is performed.

@PauloZung Great!
It would be very nice if you can file a new issue with the new code sample (maybe with a single field?) and repro steps.
It is better to keep those issue separated because:

this one has great repro steps for Web and will be closed by https://github.com/flutter/engine/pull/36893.
the new issue will not have the same repro steps and same labels (it is probably a framework issue) and some discussions will be needed to understand if it is intended or not (I think it is not but other people might disagree :smile:  ).


Not possible to logout sometimes

Gitea version (or commit ref): 1.11.3
Git version: <IP_ADDRESS>
Operating system: CentOS 7.7.1908
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist:

Description
Since the logout action was turned into a POST request (i.e. since 1.11.2) I cannot reliably log out of Gitea. Sometimes it works, often it doesn't.
When it doesn't work, the browser simply refreshes the page I am currently viewing, and no POST request to /user/logout gets logged by the reverse proxy.
I have a feeling that this might be some kind of race condition, because I was able to set a breakpoint in the Javascript handler code for the link action and to single-step through it (with Firefox's built-in Inspector), which then lead to the logout request being sent successfully.
Screenshots
n/a

Shoot, I implemented it to avoid having a blank # anchor, but perhaps if this issue persists it would be a better solution.

I also experienced this behaviour. It stemmed from me having a custom template that did not reference the new code applied to logging out. For me, I had to copy the new templates, and re-add in my custom items and the logging out issue went away.

I also experienced this behaviour. It stemmed from me having a custom template that did not reference the new code applied to logging out. For me, I had to copy the new templates, and re-add in my custom items and the logging out issue went away.

I don't think that is the same issue. In your case you wouldn't have had the link action on the Logout menu item due to old templates. In my case the action is there, but sometimes it doesn't get executed correctly client-side.

Is it possible that this bug also holds for the Time Tracker? I'm having the described issue with 'Sign Out' not working, but I also can't seem to start the time tracker. Stopping works at times. I reverted to 1.11.1 because of this.

That sounds like a new issue, as the stopwatch is using a correct form as far as I can tell. I think it may use some JS, though, so it could be a JS problem.

@jolheiser Just upgraded to Gitea 1.11.4 and the Time Tracker works again. I'm pretty sure it might've been affected by the same bug as I couldn't get it to work on 1.11.3. Thanks for fixing this, even thought it might've been unintended!


Remove statements about the future

Remove statements about the future
Fixes #21750

signed

@Marusyk Nice! ✨ Welcome to Kubernetes!
/lgtm
/approve

signed


Glide Version Update

This is more of a request than a bug report.
I'm using the nativescript-image-cache-it plugin alongside the mediafilepicker. It took me a while to realize that the incompatibility between the Glide version between both plugins was the reason why the picker was not loading.

By removing the image-cache-it plugin altogether, the filepicker works
If I install a much older version of the image-cache-it plugin (which has Glide Ver 4.8), the filepicker works

Current status :
Image-Cache-It :: Glide Version 4.10.0
MediaFilePicker:: Glide Version 4.8.0
Simply put, I don't know who to bring both plugins to play nicely.
Seeking your advice.
Thanks
Vijay

@themounthead Please try latest version & give feedback.


Simple doubts concerning pull requests

Hi all, being new to working on open-source, I have a few doubts which I hope you can help me clear:

I have looked at the issue list for certain projects, and for some issues, I see they have been assigned to certain people. Does that mean I cannot work on these issues as someone is already working on them?
When I find an issue that I want to work on, do I simply fork the repository and fix it, or do I comment that I'm taking/fixing this issue? Like, what's the common courtesy here?
There are issues where the person simply states the problem they are having and asks for a reply, just like on stackoverflow (this issue I am opening right now, for example). Does answering these questions also count as pull requests, or do you have to fork,write code,commit and push for it to be considered a pull request?

I wish to state that there is pretty much no one in my immediate surroundings who has worked on open-source projects before so there's no one to ask, and I can't seem to find answers to these specific questions online either. Also, if this is not the right place to post these questions, I apologise. In this case, please indicate where to actually ask such questions.
Thank you for your help.

Hi @Albert-Hanstein, I apologize that took so much time to respond to your queries.

I have looked at the issue list for certain projects, and for some issues, I see they have been assigned to certain people. Does that mean I cannot work on these issues as someone is already working on them?

I'd be better to avoid multiple people working on the same issue independently. The best way to proceed is to comment on that issue to check if the person who the issue is assigned to is still working on it and if you can help out in any way.

When I find an issue that I want to work on, do I simply fork the repository and fix it, or do I comment that I'm taking/fixing this issue? Like, what's the common courtesy here?

It's always suggest to comment on the issue before starting to work. Some issues might be not updated ( The issue might be fixed but the issue might not have been solved ). Maintainers might not be solving all issues. They'll have a vision on how to take the project ahead. So asking the maintainers is always the way to go.
Having said that, I'd suggest you to fork and clone the repo and start setting up. This work will probably not go as waste as you can work on other issues too.

Does answering these questions also count as pull requests, or do you have to fork,write code,commit and push for it to be considered a pull request?

No, it's not a pull request. Still, it a very important contribution to the community. Open source contribution is not just about pull requests and code changes. Some of the best open source contribution is to help others, it could be marketing the project.
Read through this to know more https://opensource.guide/how-to-contribute/#you-dont-have-to-contribute-code

Also, if this is not the right place to post these questions, I apologise. In this case, please indicate where to actually ask such questions.

This is the right place. We also have a slack team to help/mentor people trying to contribute to open source projects. You can join the team by going to https://goo.gl/PM1Jdh. If you ask there, others in the community can get involved too. Only a few of us read through issues here.

@Albert-Hanstein Since your questions have been answered.  I am closing this issue. If you have more doubts feel free to reopen it, and tag me(@sara-02) or @Roshanjossey

@sara-02 Thanks for closing it for me, I forgot I had to do that. Won't do this mistake again :)


levelorder_traversal.cpp

seems like this algo has already been added. Try adding algos unique to this repository


Application not working anymore?

When opening the URL http://wordleit.herokuapp.com/, I get

Yeah, the project isn't live anymore. I guess I'm going to archive the repo.

@mxro Yeah, the project isn't live anymore. I guess I'm going to archive it soon if I didn't have time to find an alternative for Heroku.


CodeblocksOverFileUploadsHandler.ts Rework

What Functionality Are You Suggesting?
Please describe the functionality below:
Reworking of the CodeblocksOverFileUploadsHandler to automatically upload plain text files to a hosting service.
Why Will This Benefit The Community?
For standard text file uploads, the current bot interaction is to delete the file and redirect them to codeblocks or a Github gist. After discussion with the other moderators, this could be improved by determining when a plain text file is uploaded and automatically uploading the file to a pastebin-like service.
This is not a dev ready feature. Ongoing discussion needs to occur about which service to use, as well as rough implementation plan.

Sorry for jumping on this 1 year later --

Why Will This Benefit The Community?
For standard text file uploads, the current bot interaction is to delete the file and redirect them to codeblocks or a Github gist. After discussion with the other moderators, this could be improved by determining when a plain text file is uploaded and automatically uploading the file to a pastebin-like service.

This is not a dev ready feature. Ongoing discussion needs to occur about which service to use, as well as rough implementation plan.

I understand the current and future implementation -- still not sure what the upside is for the community. What is the value-add for hosting .txt files as opposed to using codeblocks?

 I'm not really sure this is needed though, as we don't seem to run into this issue a lot.
By the way, thanks for taking an interest in our open source projects. Please ping me over on our Discord server so I know who you are - it'd be great to chat!

Pinged! 👍
And that makes sense!
Is there perhaps a middle-ground solution, where a user can upload a file and have it be converted to a codeblock upon upload?  This helps newer folks who aren't familiar with codeblocks be able to post formatted code by simply dragging and dropping a file to the server and prevents any hosting of files.
But the extent to which it helps with longer-form posts, well it'll still be capped at the maximum character length for a post I would imagine. Unless split-posting is a thing in which multiple posts would be generated to accomodate.

Given that a discord bot is the medium, it would depend on the functions of the discord bot API. Could it do a code block? Yeah if the file has text below the character count. But this just be over engineering, maybe it's time for the mod team to reconsider just disabling the listener for discords supported file types.
We didn't like the layout before, but it might be worth a conversation.

The more I think about this the less I like any implementation, honestly.
I ask myself questions like:

What if someone adds malware to a file and we process/upload it?
What happens if someone wants their upload removed (GDPR)?
What is the effort vs reward?

Is one of the reasons we don't do file uploads as is.
We'd have to use the discord bot To store user IDs
See my previous comment, if we want this we might just want to go back to square one


About mistranslation

Hi,
Crowdin seemed to translate To the child chain, the child chain bridge operator account is used. as 자식 체인으로 자치를 전송할때는 자식 체인 브리지 오퍼레이터 계정이 사용됩니다. in Value-Transfer doc. The word 자치 should be 가치 as appeared preceding statement, which is correct. I'm guessing this mistranslation seemed to occur by conservative context. Translation testing by Crowdin seems to be out of my authorization. If there's anyone who encountered this kind of problem, please let me know what's correct. Thank you

Hi, @hyunsooda. I apologize for the late answer 😢  If you want to participate in the translation, please follow the link: https://crowdin.com/project/klaytn-docs/invite?d=85n6j4j675t683c3h463e3t6c3q4r48383d3. Thank you so much :)


Implement wincrypto feature to use Windows Cryptography APIs instead …

Implement new Str0m feature 'wincrypto'.
When enabled, this cause Str0m to use Windows Cryptographic APIs for crypto. This feature cannot be used in conjunction with 'sha1' not 'openssl' features, as it provides the same functionality.
Windows APIs are accessed via the windows-rs crate. These calls are unsafe, so in order to isolate them from safe str0m code, all unsafe code is in a separate str0m_wincrypto crate. The code in the core str0m/crypto/wincrypto is the glue code between Str0m and this str0m_wincrypto crate.
The str0m_wincrypto crate is not intended to be a generic crypto crate, and is very much tailored to match str0m's crypto APIs.
TESTING....
The str0m_wincrypto crate has some new tests, and existing str0m tests pass when enabling the wincypto feature. This was also manually tested against a Str0m instance running OpenSSL. Thus, confirming that certificate exchange, key negotiation and ciphers all work correctly. Furthermore manually confirmed that AES Counter Mode worked, as GCM is typically used, but tested via manually disabling and verifying that OpenSSL backed Str0m communicated with the wincrypto backed one.

I add a build/test using wincrypto. The only failure is because we haven't sync'd to mainline in our fork

I had a little sneak peek before bed, I will do more in depth comments tomorrow. It looks great!
I wonder if it's possible to make wincrypto and openssl live side by side and that you select provider via config?
With Rust diamond dependency situation I think the library should never assume defaults from just feature flags.
See ureq3-rc2 on the latest how I reason. There native-tls is never picked up automatically even if the library default, rustls is turned off.
I think the same could work in str0m. Feature flags turn on abilities. Config selects ability. if you want to compile it with both backends you can and the default config option is not changing because of flags.
Thoughts?

Ok, it's slowly dawning on me I think. as I write this comment over a third time... the issue is that if two things depend on str0m in a build.. and both enable different providers, or disable a provider, then we end up in a scenario where we build with both features on (since features are additive).
I don't like telling the dev using us to care, and have to know what is or isn't built, just a bit sad. Why not implement a default preference... like if you don't specify which provider, we pick one... if only one is enabled we pick the one that is enabled (obviously). If multiple, are enabled we log the fact we picked something.  I find it odd that I can be a well behaved developer, and I have to pollute "build" knowledge (which provider(s) I enabled) into my code.
Like I get the desire to avoid unexpected behaviors, but in this case, the dev shouldn't care about wincrypto vs openssl.. they expose the same features.
....
All that said, it's not the end of the world.. just unfortunate that I need to not only have the toml file set feature flags per platform, but also update our configuration for str0m in the code per platform.

Yes. And you need third party tooling like cargo-deny to make sure nothing in your build accidentally switches on a dep you don't want. It's not great.

just unfortunate that I need to not only have the toml file set feature flags per platform, but also update our configuration for str0m in the code per platform.

I think that is the most sane way. The upshot I found is that the code typically becomes cleaner because you end up with fewer #cfg directives with awkward nots to make things mutually exclusive.
This in itself is important for maintainability. I find that code with large  amounts of logic built into conditional compilation is hard to maintain and breaks easily.

Yes. And you need third party tooling like cargo-deny to make sure nothing in your build accidentally switches on a dep you don't want. It's not great.

just unfortunate that I need to not only have the toml file set feature flags per platform, but also update our configuration for str0m in the code per platform.

I think that is the most sane way. The upshot I found is that the code typically becomes cleaner because you end up with fewer #cfg directives with awkward nots to make things mutually exclusive.
This in itself is important for maintainability. I find that code with large amounts of logic built into conditional compilation is hard to maintain and breaks easily.

since these features really don't provide anything different... should the rtcconfig setting essentially be optional? if it's not set, str0m picks a default, but that can vary based on features that are enabled. If the app, does care they can specifically set the config option? Then that provides the clean user API, while also supporting multiple providers.
If someone really wants to make sure they use wincrypto or openssl. they should set the config value. Otherwise, they get whatever is available. If multiple are available.. they get something, and we can log a warning/info if we want indicating that str0m chose between multiple impls.
Like, I'm not sure why in ureq3 you don't ever use nativetls as the default, if rustls is not enabled, assuming there are no API differences between the two.

Like, I'm not sure why in ureq3 you don't ever use nativetls as the default, if rustls is not enabled, assuming there are no API differences between the two.

It's because someone somewhere might have a requirement to NOT use rustls. If ureq defaulted to whatever is available, a user could through a depdency they don't control, suddenly find themselves using rustls against their will.
I think having a fixed default that must be overridden by config to change is maybe cumbersome, but more correct. There is however another mechanic I've been considering to introduce in ureq. That is how rustls can set a crypto provider ONCE for the process. Like a global setting, set once, nothing can override it.

Like, I'm not sure why in ureq3 you don't ever use nativetls as the default, if rustls is not enabled, assuming there are no API differences between the two.

It's because someone somewhere might have a requirement to NOT use rustls. If ureq defaulted to whatever is available, a user could through a depdency they don't control, suddenly find themselves using rustls against their will.

I get that. But if that's the case, they should explicitly say they want to use nativetls.
So, for example, of our requirement is not to ship openssl. We need to have a check that not only is it not knowingly used by our code, but by any code, so therefore we need to check that it's not compiled in. I'd imagine it'd be the same for this ureq3 case. Cause even if they explicitly ask for nativetls, they don't have any way of knowing something isn't broken and using rustls. So, if it's a legal or corporate requirement, they need to ensure it's not compiled in completely. That check needs to be done elsewhere.

I think having a fixed default that must be overridden by config to change is maybe cumbersome, but more correct. There is however another mechanic I've been considering to introduce in ureq. That is how rustls can set a crypto provider ONCE for the process. Like a global setting, set once, nothing can override it.

This is a little cleaner, since you can simply panic! if it's set multiple times, or at least to different values. And it just become a process startup, vs a session startup value. And if I had multiple things to configure to not use openssl... I could have that located in one hunk of code, rather than scattered.

Additionally I also want to be able to do cargo test --all-features, and it works without conflicting.

Additionally I also want to be able to do cargo test --all-features, and it works without conflicting.

I think it's fine that we should be able to turn on all features (although, it mean platform specific features will need guards, like wincrypto [I expect] won't compile on Linux ATM, we'll have to add checks to only enable on the feature and platform). Which of course goes back to having a single thing that represents the provider, so we don't have these checks littered everywhere.
FYI... I accidently merged the PR as is into our fork. I'll create a separate fork to keep iterating on for inclusion into mainline and add you to the reviewers.

FYI... I accidently merged the PR as is into our fork. I'll create a separate fork to keep iterating on for inclusion into mainline and add you to the reviewers.

Sounds good


The page has expired due to inactivity.  Please refresh and try again.

BookStack v0.21.0
Users are receiving an error from time-to-time when using the markdown editor.  There does not appear to be much of a pattern that can be identified. (multiple browsers and users)
"The page has expired due to inactivity.  Please refresh and try again."

Hi @dtw-chriss, Thanks for opening this issue. Are you able to provide a screenshot of this error at all. Just want to see where, when and which system the error is generated at.

Hi @ssddanbrown I had the same problem also on v0.21.0 version and after updating to v0.23.2 this problem gone away.

After more deep investigating I found that problem always return after adding more than one replica of app.
I guess for two and more replicas we need use :
SESSION_DRIVER=memcached
and additional memcached service as dependency.

Since the last comment on this issue is relatively old I'm going to close this. If the issue remains and is something you still require to be fixed please open a new issue, referencing this one.


Set chunk_size later to fix error (#2943)

Set chunk_size later to fix error when read from ES.

run starrocks_be_unittest


Explain the rationale for msgpack::object as the API type

Initially, we used boost::any as the API type. We have switched to msgpack::object for good reasons. Since this is a critical API design aspect, it would make sense to explain "the good reasons".

https://github.com/crossbario/autobahn-cpp/issues/50
https://github.com/msgpack/msgpack-c/issues/414

As far as I'm concerned, these were the main points:

boost::any required RTTI. I think Boost now offers a way to provide static typeid functions and make it work without RTTI, but even then this kind of cast will be slow because it involves (typeid) string comparisons, and providing the static typeid functions in addition to msgpack/JSON/CBOR conversion functions adds unnecessary extra complexity.
boost::any seems convenient but was actually pretty sucky, because you only supported a small number of types for packing. This means if a user has their own custom type, they would have to convert that manually to a boost::any type that is supported by autobahn-cpp. Worse, they only get the check at runtime when autobahn-cpp throws an error. With msgpack conversion functions, you know you'll be able to send it if it compiles.
Any type will require conversion functions from and to the serialized format. By using msgpack's own conversion system (which serializes any type with a packing template defined, and deserializes into msgpack::object), we allow people to reuse msgpack-c conversions for autobahn-cpp, and the other way round. I believe that's better than defining your own set of conversion functions that users will always have to implement (in addition to msgpack's or any other ones).

Note also that msgpack::object is only used when receiving messages. On sending them, we go directly from  to serialized msgpack buffers, with no intermediate variant type. For packing non-msgpack serialization formats like JSON, we would have to either go through an intermediate variant (e.g. msgpack::object) or autobahn-cpp / the JSON library offers similar SAX-style conversion functions for direct-to-buffer packing.

msgpack::object is the result of msgpack-c's unpack() function. If you convert to any other type, you (currently) have to deserialize using buffer => msgpack::object => (user type). If you choose any other variant type, you have to do it like, buffer => msgpack::object => (autobahn-cpp variant) => (user type). Whether you use boost::any, or msgpack::type::variant, or autobahn::my_custom_variant, there's always an extra step of allocations compared to the msgpack::object baseline.

This point is fixable with additional msgpack-c APIs, and that's why I asked for a SAX-style unpacking API in the msgpack-c issue.

Even if you have those additional APIs, the only way to gain efficiencies would be to get rid of variant types altogether, and have the user call e.g. session->call<MyType>(...), with the template argument specifying the return type that autobahn-cpp will use to unpack the msgpack buffer.

msgpack::object minimizes the number of heap allocations by relying on its msgpack::zone. That makes the API sort of clunky, but is pretty okay for performance compared to an intermediate variant type that uses std::string, std::map, std::vector etc., assuming that variant format is not the final type that the autobahn-cpp user uses in their code.

@jpetso Thanks a lot for this thorough summary! I think this is a list of very good reasons;) Given that, I feel it was the right decision, and eventually, above should go into the library documentation so users can follow the arguments and decisions we made regarding API (or disagree .. but then only after considering all aspects we had in mind).


Fix broken link on website

This is a broken link preventing the website from building

Heh, looks like you have overtook me with the fix 😄

#4092

There are two broken links on the page, and also fresh cut 0.74 page is affected.


Linter integration

Closes #42
WIP: code doesn't pass JS/SOL lint checks yet wow that was easier than I thought

Closing in favour of https://github.com/keep-network/keep-ecdsa/pull/328


Adding a js alias to jsconfig.json does not work.

Issue Type: Bug
I have added an alias for our common library. I got it working with Webpack and Jest, but not in vscode,
{
  "description":
    "This is to config VS Code to recongnise our aliases, ala https://medium.com/@justintulk/solve-module-import-aliasing-for-webpack-jest-and-vscode-74007ce4adc9",
  "compilerOptions": {
    "target": "es2017",
    "allowSyntheticDefaultImports": false,
    "baseUrl": "./",
    "paths": {
      "@lib/*": ["app/javascript/lib/*"]
    }
  },
  "exclude": ["node_modules", "dist"]
}

Note the item in paths is the alias for @lib
I used to be able to righ click, go to defintion on this line
import MustacheValidator from '../../../../../../lib/forms/MustacheValidator';
and it worked, but now when I do it with
import MustacheValidator from '@lib/forms/MustacheValidator';
it says No Defintion Found
VS Code version: Code 1.29.1 (bc24f98b5f70467bc689abf41cc5550ca637088e, 2018-11-15T19:06:21.742Z)
OS version: Darwin x64 18.2.0
Extensions (34)

Extension
Author (truncated)
Version

lorem-ipsum
Tyr
1.2.0

html-css-class-completion
Zig
1.17.1

Bookmarks
ale
10.0.0

react-pure-to-class-vscode
ang
1.1.7

vscode-color
ans
0.4.5

color-info
bie
0.5.0

path-intellisense
chr
1.4.2

vscode-eslint
dba
1.7.0

gitlens
eam
9.0.3

vsc-material-theme
Equ
2.6.2

prettier-vscode
esb
1.7.2

nasc-touchbar
fel
1.11.0

auto-close-tag
for
0.5.6

auto-rename-tag
for
0.0.15

code-runner
for
0.9.5

vscode-pull-request-github
Git
0.2.3

vscode-github
Kni
0.30.1

dotenv
mik
1.0.1

ruby-rubocop
mis
0.7.1

python
ms-
2018.11.0

cpptools
ms-
0.20.1

csharp
ms-
1.17.1

Go
ms-
0.7.0

mono-debug
ms-
0.15.8

node-debug2
ms-
1.30.0

debugger-for-chrome
msj
4.11.1

vscode-icons
rob
7.28.0

code-settings-sync
Sha
3.2.2

slim
sia
0.1.2

code-spell-checker
str
1.6.10

tabnine-vscode
Tab
1.0.3

vscode-jss-snippets
vis
0.2.2

ReactSnippets
xab
2.2.1

vscode-open-in-github
ziy
1.3.6

(1 theme extensions excluded)

In VS Code 1.30, can you please confirm that both files are part of your jsconfig.json project. To do this:

Open the file in VS Code
Run the JavaScript: go to project configuration command
This should open the jsconfig with your paths defined.

I see paths working as expected in a simple workspace

I had a similar problem after updating. There seemed to have been an issue with having another node project inside my workspace.
At least I got a ts error when opening the jsconfig file. It worked again after removing that project.

@zdrummond Can you please explain your project structure more or provide some example code:

Where is the jsconfig?
Where is the file that is being imported (MustacheValidator )?
Where is the file that you are adding the import in?

Sure
jsconfig
Is in the root directory
MustacheValidator
app/javascript/lib/forms/MustacheValidator.js
File importing MustacheValidator
app/javascript/bundles/Templates/LegalHoldTemplates/Form/components/lib/ValidatingRichEditor.js

I've got the same problem.
.vscode/jsconfig.json:
{
    "compilerOptions": {
        "target": "es6",
        "baseUrl": "./",
        "paths": {
            "audiensRoot/*": ["${workspaceRoot}/app/Resources/assets/js/*"],
            "audiensComponents/*": ["${workspaceRoot}/app/Resources/assets/js/react-components/*"],
        },
        "outFile": "${workspaceRoot}/build/js/**/*.js",
        "mapRoot": "${workspaceRoot}/build/js",
        "sourceMap": true
    },
    "include": [
        "app/Resources/assets/js/**/*"
    ],
    "exclude": [
        "node_modules",
        "test-js/**/*"
    ]
}

(note: I've tried with and without ${workspaceRoot}, replacing it with a dot)
When importing whatever from audiensRoot, the eslint plugin works as expected and doesn't throw me an exception, but IntelliSense is not able to go to the definition.

I have a situation where the alias/paths seem to work, but only for .js files and not .jsx files.
Directory structure:
./
  jsconfig.js
  webpack.config.js
  src/
    js/
      lib/
        cookies.js
        ErrorBoundary.jsx

Calling file, somewhere else in the tree:
import React from 'react'
import { createCookie, readCookie } from 'lib/cookies'
import ErrorBoundary from 'lib/ErrorBoundary'

Webpack config that matters:
  resolve: {
        extensions: ['.js', '.jsx'],
        alias: {
            lib: path.join(__dirname, 'src/js/lib'),
        }
    }

jsconfig.json
{
    "compilerOptions": {
        "target": "esnext",
        "allowSyntheticDefaultImports": true,
        "checkJs": false,
        "baseUrl": "./",
        "paths": {
            "lib/*": [
                "src/js/lib/*"
            ]
        }
    },
    "include": [
        "./src/**/*"
    ]
}

Removing the checkJs property and/or changing allowSyntheticDefaultImports to false doesn't seem to have effect.

Have to test with yet another extension.. But this seems to indicate that only .js files are indexed..

Actually, writing down all this above, helped me rubber duck this issue 🦆 .  I Googled more specifically and found: https://javascriptplayground.com/vscode-go-to-definition-jsx/
For me the issue is resolved by adding "jsx": "react" to jsconfig.json. I found that I can remove some other properties, so it becomes:
{
    "compilerOptions": {
        "target": "esnext",
        "baseUrl": "./",
        "jsx": "react",
        "checkJs": false,
        "paths": {
            "lib/*": [
                "src/js/lib/*"
            ],
            "theme/*": [
                "src/theme/*"
            ]
        }
    },
    "include": [
        "./src/js/**/*"
    ],
}

On a sidenote: searching does take forever. It doesn't matter if I include ony the folder that I'm interested in, or exclude all others. It does become faster though after you "Go to definition" a few times. VScode probably caches it lazily.

Thanks @publicJorn , adding "jsx": "react" to the jsconf.json file fixed the  jsx file resolution issue :pray:

@zdrummond Can you please share a small example project that demonstrates this issue?

As an alternative tsconfig seems to work in multi root envs


Upgrades occupy same slot as output of machine when being added to a machine

When I put an upgrade into a machine, it occupies the same spot as the output slot.

This has been fixed, update to v8.0.2


Allow numpy 1.24.

This removes the upper bound on numpy<1.24 to allow newer NumPy versions.

Thanks @bdice, it looks good. Opened #564 to resolve the unrelated test failures

Great. Once #564 is merged, feel free to update this PR with branch-23.06 and merge.

/merge

Thanks all! 🙏


Website in chinese even when pointing to "en"

Describe the bug
When going to https://mmdetection3d.readthedocs.io/en/latest/, which points toward "en", most of the text is still in Chinese
Reproduction

What command or script did you run?
Just go to https://mmdetection3d.readthedocs.io/en/latest/, reproduced by my and another person from a different computer. Reproduced from Firefox and Chromium.

We have PR #1121 ready to merge to fix this problem. Sorry for the inconvenience.


The gcloud tool doesn't like our set up

Our build was broken by a new update to the gcloud SDK.
View: https://travis-ci.org/google/WebFundamentals/builds/86422127
The problem seems to be with gcloud staging our files on cloud storage which it gets upset about with some files (why I have no idea - the error messages are helpful).
The build is working now because there is a config you can set to tell the gcloud SDK to use the old app engine way of deploying:
 $SDK_DIR/bin/gcloud config set app/use_appengine_api false

It will likely break again if gcloud updates and removes support for the old api.

@PaulKinlan has fixed this now.


Consider for a recipe for Orb of Origin?

There's no recipe for Orb of Origin now, that players couldn't regret their choice at the beginning of the game. However, a recipe for such a thing should be expensive.
Here's my recipe for 1.16.1:
□I□
IAI
□I□
□: None;I: Netherite Ingot;A: Nether Star
If you think it's too expensive, you can change the Nether Star to Heart of the Sea, or change Netherite Ingot to Netherite Scrap.

The orb of origin not having a recipe is intentional. The origin being unchangeable is true to the idea of the mod, but at the same time I see why people would want the orb to be obtainable. That's why I encourage people to add their own recipe via a data pack, as I don't want to prescribe how expensive or cheap the orb should be. I've heard of people making the orb quite expensive with a Nether Star as the ingredient like you proposed, but I also heard of others who added a recipe that was way cheaper.
I hope this sufficiently explains why there's no recipe for the orb by default.


Kibana 7.2.0 Plugin

64/5000
Hello! How to use the plugin for Kibana 7.2.0? Thank you very much!

hi @rgoliveirati . 7.2 is not supported yet.
You can tweek latest version like @lucaspaixos did.  https://github.com/datasweet/kibana-datasweet-formula/issues/52
Maybe he can help you.
Regards


Project Selector Refactor Part 1 - static project selection component

Refactored project selector changes drop-down with a tree view with a static component that calls into a dialog to select a project.
Here is a deployment panel with project information set and user signed in:

When no selection is made yet, the following shows:

Clicking hyperlinks or browse button calls the dialog to change the account/project selection.
This PR adds ProjectSelector (static component with hyperlinks) into new 'com.google.cloud.tools.intellij.project' package along with its helper classes and unit test.
Current ProjectSelector implementation is still used in forms and dialogs.
CloudProject and ProjectSelectionListener will be used to track changes in project selection instead of low-level text and tree selection UI events.

cool that makes sense
On Mon, Nov 27, 2017 at 5:45 PM Ivan Portyankin<EMAIL_ADDRESS>wrote:

@ivanporty commented on this pull request.
In
google-cloud-tools-plugin/src/com/google/cloud/tools/intellij/project/ProjectSelector.java
https://github.com/GoogleCloudPlatform/google-cloud-intellij/pull/1760#discussion_r153347084
:

@param projectSelectionListener the listener to add

*/
public void addProjectSelectionListener(ProjectSelectionListener projectSelectionListener) {
projectSelectionListeners.add(projectSelectionListener);
}

/**

Removes a {@link ProjectSelectionListener} from this class's internal list of listeners.

@param projectSelectionListener the listener to remove

*/
public void removeProjectSelectionListener(ProjectSelectionListener projectSelectionListener) {
projectSelectionListeners.remove(projectSelectionListener);
}

private void createUIComponents() {

It's a very simple flow layout line with two hyperlinks, so it's easy to
see how it's implemented in code, plus I was having issues with properly
wrapping the components into IDEA standard ComponentWithBrowseButton<>
widget using the form editor.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/pull/1760#discussion_r153347084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABp8QLIwpR22C49J_pBumEU66VoY1ukKks5s6zt_gaJpZM4QsDc5
.


DeepMased - stalled out on finding features

Greetings,
I am trying to see if my merge-overlap.pl script creates chimeras when I combine output from different assemblers.
As you can see, nothing has happened in days:
Using TensorFlow backend.
2020-11-10 21:33:21,376 - Indexing file: MERGED_CONTIGS_NOSPLIT_VDD.fa
2020-11-10 21:33:25,745 - Cannot find MERGED_CONTIGS_NOSPLIT_VDD_sorted.bam.bai; creating...
2020-11-10 21:35:08,541 - Processing: MERGED_CONTIGS_NOSPLIT_VDD_sorted.bam
2020-11-10 21:35:10,378 - Number of contigs in the bam file: 1682485
Checking "top" - it is still running, though I wonder it is only using 1 core (100%).
PID      USER      PR   NI    VIRT       RES         SHR      S  %CPU   %MEM     TIME+    COMMAND
40639    txxf        20   0   11.542g   9.143g   145724   R   100.0       1.2      2262:04       python3
There's nothing in the output either.
-rw-r--r--  1 txxf gmb           0   Nov 10 21:35   MERGED_CONTIGS_NOSPLIT_VDD_sorted_feats.tsv
Any ideas why it is not working?
How long should this normally take?
Best,
Teal

Hi Teal. Can you post the command that you're running?

Hi!
I ran this command:   python3 -m DeepMAsED features bam_fasta_table.tab

Maybe it's an issue with tensorflow freezing/stalling? I did have some issues with that for certain versions. Which version are you running? Does running another DL model with that version of TF work with your setup? Check out this issue for some potential solutions: https://github.com/tensorflow/tensorflow/issues/32017

When I run [ python3 -c 'import tensorflow as tf; print(tf.version)' ]
I get:  2.3.1
Our system admin had to do some voodoo to get deepmased to work:
comics
python3 -m venv venv --system-site-packages --prompt deepmased
. venv/bin/activate
pip install --upgrade pip
pip install tensorflow-cpu
That last command outputs:
(deepmased) (omics) tealfurn@alpena:~/MUSCATO/URDB_PAPER/SAMPLE_53600$ pip
install tensorflow-cpu
Requirement already satisfied: tensorflow-cpu in
./venv/lib/python3.7/site-packages (2.3.1)
Requirement already satisfied: google-pasta>=0.1.8 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (0.2.0)
Requirement already satisfied: opt-einsum>=2.3.2 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (3.3.0)
Requirement already satisfied: absl-py>=0.7.0 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (0.11.0)
Requirement already satisfied: wrapt>=1.11.1 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (1.12.1)
Requirement already satisfied: tensorboard<3,>=2.3.0 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (2.3.0)
Requirement already satisfied: grpcio>=1.8.6 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (1.33.2)
Requirement already satisfied: wheel>=0.26 in
/usr/lib/python3/dist-packages (from tensorflow-cpu) (0.32.3)
Requirement already satisfied: astunparse==1.6.3 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (1.6.3)
Requirement already satisfied: h5py<2.11.0,>=2.10.0 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (2.10.0)
Requirement already satisfied: gast==0.3.3 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (0.3.3)
Requirement already satisfied: termcolor>=1.1.0 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (1.1.0)
Requirement already satisfied: tensorflow-estimator<2.4.0,>=2.3.0 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (2.3.0)
Requirement already satisfied: numpy<1.19.0,>=1.16.0 in
/usr/lib/python3/dist-packages (from tensorflow-cpu) (1.16.2)
Requirement already satisfied: keras-preprocessing<1.2,>=1.1.1 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (1.1.2)
Requirement already satisfied: protobuf>=3.9.2 in
./venv/lib/python3.7/site-packages (from tensorflow-cpu) (3.13.0)
Requirement already satisfied: six>=1.12.0 in
/usr/lib/python3/dist-packages (from tensorflow-cpu) (1.12.0)
Requirement already satisfied: google-auth-oauthlib<0.5,>=0.4.1 in
./venv/lib/python3.7/site-packages (from
tensorboard<3,>=2.3.0->tensorflow-cpu) (0.4.2)
Requirement already satisfied: google-auth<2,>=1.6.3 in
./venv/lib/python3.7/site-packages (from
tensorboard<3,>=2.3.0->tensorflow-cpu) (1.23.0)
Requirement already satisfied: markdown>=2.6.8 in
./venv/lib/python3.7/site-packages (from
tensorboard<3,>=2.3.0->tensorflow-cpu) (3.3.3)
Requirement already satisfied: requests<3,>=2.21.0 in
/usr/lib/python3/dist-packages (from tensorboard<3,>=2.3.0->tensorflow-cpu)
(2.21.0)
Requirement already satisfied: setuptools>=41.0.0 in
./venv/lib/python3.7/site-packages (from
tensorboard<3,>=2.3.0->tensorflow-cpu) (50.3.2)
Requirement already satisfied: werkzeug>=0.11.15 in
/usr/lib/python3/dist-packages (from tensorboard<3,>=2.3.0->tensorflow-cpu)
(0.14.1)
Requirement already satisfied: tensorboard-plugin-wit>=1.6.0 in
./venv/lib/python3.7/site-packages (from
tensorboard<3,>=2.3.0->tensorflow-cpu) (1.7.0)
Requirement already satisfied: requests-oauthlib>=0.7.0 in
./venv/lib/python3.7/site-packages (from
google-auth-oauthlib<0.5,>=0.4.1->tensorboard<3,>=2.3.0->tensorflow-cpu)
(1.3.0)
Requirement already satisfied: cachetools<5.0,>=2.0.0 in
./venv/lib/python3.7/site-packages (from
google-auth<2,>=1.6.3->tensorboard<3,>=2.3.0->tensorflow-cpu) (4.1.1)
Requirement already satisfied: pyasn1-modules>=0.2.1 in
./venv/lib/python3.7/site-packages (from
google-auth<2,>=1.6.3->tensorboard<3,>=2.3.0->tensorflow-cpu) (0.2.8)
Requirement already satisfied: rsa<5,>=3.1.4; python_version >= "3.5" in
./venv/lib/python3.7/site-packages (from
google-auth<2,>=1.6.3->tensorboard<3,>=2.3.0->tensorflow-cpu) (4.6)
Requirement already satisfied: importlib-metadata; python_version < "3.8"
in ./venv/lib/python3.7/site-packages (from
markdown>=2.6.8->tensorboard<3,>=2.3.0->tensorflow-cpu) (2.0.0)
Requirement already satisfied: oauthlib>=3.0.0 in
./venv/lib/python3.7/site-packages (from
requests-oauthlib>=0.7.0->google-auth-oauthlib<0.5,>=0.4.1->tensorboard<3,>=2.3.0->tensorflow-cpu)
(3.1.0)
Collecting pyasn1<0.5.0,>=0.4.6
Using cached pyasn1-0.4.8-py2.py3-none-any.whl (77 kB)
Requirement already satisfied: zipp>=0.5 in
./venv/lib/python3.7/site-packages (from importlib-metadata; python_version
< "3.8"->markdown>=2.6.8->tensorboard<3,>=2.3.0->tensorflow-cpu) (3.4.0)
Installing collected packages: pyasn1
Attempting uninstall: pyasn1
Found existing installation: pyasn1 0.4.2
Not uninstalling pyasn1 at /usr/lib/python3/dist-packages, outside
environment /geomicro/data2/tealfurn/MUSCATO/URDB_PAPER/SAMPLE_53600/venv
Can't uninstall 'pyasn1'. No files were found to uninstall.
Successfully installed pyasn1-0.4.8
On Fri, Nov 13, 2020 at 8:14 AM Nick Youngblut<EMAIL_ADDRESS>wrote:

Maybe it's an issue with tensorflow freezing/stalling? I did have some
issues with that for certain versions. Which version are you running? Does
running another DL model with that version of TF work with your setup?
Check out this issue for some potential solutions:
tensorflow/tensorflow#32017
https://github.com/tensorflow/tensorflow/issues/32017
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/leylabmpi/DeepMAsED/issues/8#issuecomment-726757999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEGCMSVYENXYMFLFIURMR6TSPUWM7ANCNFSM4TTP2BGA
.

Our system admin had to do some voodoo to get deepmased to work:

You were not able to install DeepMAsED via bioconda?

Hi Robert,
I'm talking here with Nick Youngblut, who created DeepMased, about our
installation.
I could not get it to work on the first step - find features and figured it
was related to all those warning flags we get that you said to ignore.
I know you said you had some trouble installing, hence the virtual
environment, and maybe you could explain better why bioconda didn't work?
Best
Teal
On Mon, Nov 16, 2020 at 7:57 AM Nick Youngblut<EMAIL_ADDRESS>wrote:

Our system admin had to do some voodoo to get deepmased to work:
You were not able to install DeepMAsED via bioconda
https://bioconda.github.io/recipes/deepmased/README.html#package-deepmased
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/leylabmpi/DeepMAsED/issues/8#issuecomment-727964068,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEGCMSUQ3CNH2TUUGRQFUXDSQEOU5ANCNFSM4TTP2BGA
.

Greetings!
Robert set it up for me to run through anaconda3
module load Anaconda/3
conda create -c bioconda -n deepmased deepmased
source activate deepmased
It still stalls out in the find features.
(deepmased) tfurn@ymannx:~/PIPELINE/SAMPLE_42896/ROBERT$ python3 -m
DeepMAsED features bam_fasta_table.tab
2020-11-29 20:09:46,647 - Indexing file: MERGED_CONTIGS_NOSPLIT_XDD.fa
2020-11-29 20:09:53,984 - Processing: MERGED_CONTIGS_NOSPLIT_XDD_sorted.bam
2020-11-29 20:09:56,166 - Number of contigs in the bam file: 2238133
I also notice it is running only one core - not sure exactly what the
script is running during find features, gene prediction? MetaQuast? but
seems inefficient considering how much compute power we have on hand.
top - 12:04:16 up 40 days,  1:36,  9 users,  load average: 2.68, 2.76, 2.73
Tasks: 820 total,   3 running, 488 sleeping,   0 stopped,   0 zombie
%Cpu(s):  2.0 us,  0.6 sy,  0.0 ni, 97.4 id,  0.0 wa,  0.0 hi,  0.0 si,
0.0 st
KiB Mem : 79124640+total, 22944363+free,  5286204 used, 55651654+buff/cache
KiB Swap: 46858236 total, 46772732 free,    85504 used. 78042592+avail Mem
PID   USER  PR  NI    VIRT        RES        SHR      S   %CPU  %MEM
TIME+        COMMAND
25374  tfurn    20   0    5226716   3.995g   174120   R    100.0     0.5
953:24.88      python3 -m DeepMAsED features bam_fasta_table.tab
Let me know what I should do.
Best,
Teal
On Mon, Nov 16, 2020 at 7:57 AM Nick Youngblut<EMAIL_ADDRESS>wrote:

Our system admin had to do some voodoo to get deepmased to work:
You were not able to install DeepMAsED via bioconda
https://bioconda.github.io/recipes/deepmased/README.html#package-deepmased
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/leylabmpi/DeepMAsED/issues/8#issuecomment-727964068,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEGCMSUQ3CNH2TUUGRQFUXDSQEOU5ANCNFSM4TTP2BGA
.

not sure exactly what the script is running during find features, gene prediction? MetaQuast?

DeepMAsED features generates feature tables from the bam files via pysam. bam files are processing parallel if --procs is >1.

I also notice it is running only one core

Your command doesn't include --procs, so DeepMAsED features defaults to 1 core.

It still stalls out in the find features.

It may be an issue with pysam. DeepMAsED features is really just pysam to extract features from the bam files. It's therefore almost definitely something to do with pysam. You could try installing a different version of pysam or send me some example files so that I can try and reproduce the issue.

Hi Nick,
The option --procs is not on the github and not in the deepmased -h command
line output - you may want to update that.
I will try increasing the procs and see if I can get it to finish.
I'll check the version of pysam.
BTW: while I'm trying to get things to work. When we map our reads to
generate the bam file, how should we treat multimapping/ambiguous reads
(usually between 30-80% of total reads are multimapping for meta-NGS data
due to strains/related species) -> allow full multimap? keep 1 random?
toss?
Best,
Teal
On Mon, Nov 30, 2020 at 12:55 PM Nick Youngblut<EMAIL_ADDRESS>wrote:

not sure exactly what the script is running during find features, gene
prediction? MetaQuast?
DeepMAsED features generates feature tables from the bam files via pysam.
bam files are processing parallel if --procs is >1.
I also notice it is running only one core
Your command doesn't include --procs, so DeepMAsED features defaults to 1
core.
It still stalls out in the find features.
It may be an issue with pysam. DeepMAsED features is really just pysam to
extract features from the bam files. It's therefore almost definitely
something to do with pysam. You could try installing a different version of
pysam or send me some example files so that I can try and reproduce the
issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/leylabmpi/DeepMAsED/issues/8#issuecomment-735943216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEGCMSSSVXI2VAPNSYKWA7LSSPL7TANCNFSM4TTP2BGA
.

The option --procs is not on the github and not in the deepmased -h command line output - you may want to update that.

The UI is a standard command-subcommand format (like git). Use DeepMAsED features -h

allow full multimap? keep 1 random?

We used default bowtie2 params for mapping. You can assess different mapping approaches on accuracy if you'd like by running the DeepMAsED-SM pipeline.

Hi Nick,
*I created a much smaller 50K contig and 50M reads test data set: *
72 Dec 1 15:57 bam_fasta_table.tab
80 Dec 1 17:41 feature_file_table.tsv
6708944839 Dec 1 15:32 test_DM_50K_contigs_AMBRAND_sorted.bam
3523736 Dec 1 15:34 test_DM_50K_contigs_AMBRAND_sorted.bam.bai
12548326952 Dec 1 17:41 test_DM_50K_contigs_AMBRAND_sorted_feats.tsv
154772928 Dec 1 14:52 test_DM_50K_contigs.fa
And ran the following commands:
python3 -m DeepMAsED features bam_fasta_table.tab --procs 20      #took
2.5hrs to complete
DeepMAsED predict feature_file_table.tsv
*The features command did work after a couple hours, so I guess that means
pysam does work BUT... *
On the predict command - it throws the following error:
2020-12-01 22:42:32,578 - Loading model...
2020-12-01 22:42:32,579 -   Loading mstd...
2020-12-01 22:42:32,579 -   Loading h5...
Traceback (most recent call last):
File "/geomicro/data2/tealfurn/.conda/envs/deepmased/bin/DeepMAsED", line
10, in
sys.exit(main())
File
"/geomicro/data2/tealfurn/.conda/envs/deepmased/lib/python3.6/site-packages/DeepMAsED/main.py",
line 57, in main
args.func(args)
File
"/geomicro/data2/tealfurn/.conda/envs/deepmased/lib/python3.6/site-packages/DeepMAsED/Commands/Predict.py",
line 76, in main
Predict.main(args)
File
"/geomicro/data2/tealfurn/.conda/envs/deepmased/lib/python3.6/site-packages/DeepMAsED/Predict.py",
line 50, in main
model = load_model(F, custom_objects=custom_obj)
File
"/geomicro/data2/tealfurn/.conda/envs/deepmased/lib/python3.6/site-packages/tensorflow/python/keras/saving/save.py",
line 184, in load_model
return hdf5_format.load_model_from_hdf5(filepath, custom_objects,
compile)
File
"/geomicro/data2/tealfurn/.conda/envs/deepmased/lib/python3.6/site-packages/tensorflow/python/keras/saving/hdf5_format.py",
line 166, in load_model_from_hdf5
f = h5py.File(filepath, mode='r')
File
"/geomicro/data2/tealfurn/.conda/envs/deepmased/lib/python3.6/site-packages/h5py/_hl/files.py",
line 408, in init
swmr=swmr)
File
"/geomicro/data2/tealfurn/.conda/envs/deepmased/lib/python3.6/site-packages/h5py/_hl/files.py",
line 173, in make_fid
fid = h5f.open(name, flags, fapl=fapl)
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "h5py/h5f.pyx", line 88, in h5py.h5f.open
OSError: Unable to open file (unable to lock file, errno = 37, error
message = 'No locks available')
I can try to send you the data if you have some way to send such large
files, Globus maybe?
Best,Teal

I'm glad that you got DeepMAsED features to run. For many large bam files, I highly recommend using many threads to speed up the job.

OSError: Unable to open file (unable to lock file, errno = 37, error_message = 'No locks available')

This appears to be an open issue with h5py: https://github.com/h5py/h5py/issues/1101


Missing or non standard keyboard shortcuts

Describe the bug
Monokle doesn't have some actions mapped to keyboard, or they are assigned to non-standard.
Missing:
- New Window

Find
Replace

Assigned to non standard shortcuts
- Zoom in / out
To Reproduce

Expected behavior
Use standard keyboard short cuts for zoom in.
Desktop (please complete the following information):

OS: MacOS Monterey
Version 1.6.0

Fixed: zoom in/zoom out,
add new shortcut for opening new Monokle window.
Still no functionality for:

Find
Replace
Needs more discussion as it is a new feature.

Find design for the modal popup showing shortcuts list here: https://www.figma.com/file/fDfEVnJE9tnEFGEREjW4mg/Keyboard-shortcuts-modal?node-id=0%3A1

I think that we have to remove the Save shortcut since we have auto-save everywhere. We just need to check if the shortcut is still used anywhere or not.
@julioramosest if we remove that shortcut, I think you would have to remake the illustration from the top