After upgrade to 1.0, Redmine won't show changesets

I've upgraded to the new stable version 1.0, but after the upgrade, Redmine no longer shows any changesets. All commits have a refs #1234 or fixes #1234 are not shown at the issue detail view.
Hooks seem to be working, as the following is shown, if I'm running the hooks in non-async mode with debugging enabled:
remote:
remote: Notifying Redmine about changes to this repository : 'herocentral/' ...
remote: Looking for additional global post-receive hooks...
remote:
remote: Looking for additional local post-receive hooks...
remote:   - No local hooks found
remote:
remote: Calling additional post-receive hooks...
remote:   - Executing extra hook 'hooks/post-receive.d/mail_notifications'
remote:     multimailhook is disabled
remote:

The only strange thing I can see, is the repository name, as the repository URL for cloning is
<EMAIL_ADDRESS>

.gitconfig content:
[hooks "redmine_gitolite"]
        url = http://tracker.example.com/githooks/post-receive
        debug = true
        asynch = true
[redminegitolite]
        redmineurl = http://tracker.example.com/githooks/post-receive/redmine
        debugmode = true
        asyncmode = false
[multimailhook]
        mailer = sendmail
        environment = gitolite
        smtpauth = false
        smtpserver = localhost
        smtpport = 25

My Redmine installation is using https://, this is the only obvious error I'm seeing in this URL (but it gets redirected, at least in the browser).
Trying to open redmineurl, will result in a 404 Page not found error message, but I don't know, if there is a part of the URL missing.

I've fixed the hooks URL in the settings.

It's working now. Maybe there should be an error message, if the hooks URL isn't working. Now it's showing the following during a push:
remote:
remote: Notifying Redmine about changes to this repository : 'herocentral/' ...
remote:
remote:   - Fetching changesets for 'herocentral' repository ...  [success]
remote:
remote: Looking for additional global post-receive hooks...
remote:
remote: Looking for additional local post-receive hooks...
remote:   - No local hooks found
remote:
remote: Calling additional post-receive hooks...
remote:   - Executing extra hook 'hooks/post-receive.d/mail_notifications'
remote:     multimailhook is disabled
remote:

My Redmine installation is using https://, this is the only obvious error I'm seeing in this URL (but it gets redirected, at least in the browser).

This is the cause of the 'bug'. The hook expect a 200 status code and not a 301.

Fixed in commit https://github.com/jbox-web/redmine_git_hosting/commit/cbb6f0028afba0d611bbbc13ed9e2b62595177e1
Hook will throw an error if the status code is not a 200.


Support writing to a branch

Feature Request / Improvement
Right now we hardcoded that we write to the main branch all the time, would be great to make this configurable.

In order to write to a branch, the branch needs to be created first.
From https://iceberg.apache.org/docs/latest/spark-writes/#writing-to-branches:

the branch must exist before performing the write. The operation does not create the branch if it does not exist.

The API should look something like
def append(self, df: pa.Table, branch: str = MAIN_BRANCH)
...
def overwrite(self, df: pa.Table, overwrite_filter: BooleanExpression = ALWAYS_TRUE, branch: str = MAIN_BRANCH)

First pass, just refactoring
#312

We first need a create branch API.
Then update places currently gated by MAIN_BRANCH.

https://github.com/apache/iceberg-python/blob/9e0394939bab8d6b26cdde6f71173bdd42b55b2e/pyiceberg/table/metadata.py#L124
https://github.com/apache/iceberg-python/blob/9e0394939bab8d6b26cdde6f71173bdd42b55b2e/pyiceberg/table/__init__.py#L592

And finally add a test for writing to a branch

also note dev/provision.py which is used for integration tests already have statements to create tags and branchs
https://github.com/apache/iceberg-python/blob/9e0394939bab8d6b26cdde6f71173bdd42b55b2e/dev/provision.py#L146-L148

I had an offline chat with @kevinjqliu , I shall work on this to build off of the PR created by kevin.

Hello, can I hop back on this train if no one else is actively working on this (again building off of kevin's work)? @kevinjqliu @Fokko

@Gowthami03B definitely, I've assigned the issue to you!

@Gowthami03B definitely, I've assigned the issue to you!

Opening up this again to the community as I am gonna be out for the next month!

@kevinjqliu @Fokko
I can take this forward if no one is actively working on this.

@vinjai yes! please go ahead.

Hi @vinjai thank you very much for working on this issue. I'm just working through the list of open items to check if they are still actively being worked on. Are you still interested in contributing this feature to PyIceberg? 🧊

Hey @sungwy
I am working on this at the moment
Will open the PR for review by next week

PR is ready for review


⚠ File access (beta) has degraded performance

In 37d5fec, File access (beta) (https://beta.fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 7463 ms

Resolved: File access (beta) performance has improved in 5c0be0d.


Add explicit dependency on parser

We use this gem here: lib/haml_lint/ruby_parser.rb, we should also require it.
In my set up (after updating bundler and some other gems) it could not find the parser gem.
Then I found out that haml_lint uses it, but does not list it as a dependency.

Unless I'm missing something, haml-lint's dependency on the rubocop gem should account for this. I want things to break if RuboCop stops using parser, since that would mean haml-lint would be using a different Ruby-parsing implementation than RuboCop.
haml-lint is pretty tightly coupled to RuboCop, so unless there's a technical reason we need to explicitly list parser as a dependency, I'd rather not have to remember to update the parser dependency every time we update haml-lint's dependency on RuboCop.
I guess I'm looking for a reason why you're seeing this behavior and I'm not? Also, can you reproduce this with the master branch of haml-lint?

I have done multiple updates, and the only one (from my opinion) taht could be responsible is the bundle update.
Bundler went from 1.10.6 to 1.11.2.
We updated rubocop (and its dependencies), but the latest version also has the parser gem (they updated the required version, but since that also is the version which haml_lint uses after adding it to the dependencies, that should not be the problem):
rubocop from 0.35.1 to 0.36.0
ast from 2.1.0 to 2.2.0
parser from <IP_ADDRESS> to <IP_ADDRESS>
Removed: tins (1.6.0)
Removed:  astrolabe (1.3.1)

and here are the other gem updates i have done, which really should not be the problem
bson 2.3.0 from 3.2.6
colorize 0.7.5 to 0.7.7
connection_pool (2.1.1 to 2.2.0
ethon from 0.8.0 to 0.8.1
exception_notification 4.1.1 to 4.1.4
geocoder 1.2.12 to 1.2.14
hashie from 3.4.2 to 3.4.3
ipaddress from 0.8.0 to 0.8.2
lol_dba from 2.0.1 to 2.0.3
mini_portile from 0.6.2 to mini_portile2 2.0.0
mongoid from 4.0.0 to 4.0.2
moped from 2.0.2 to 2.0.7
newrelic_rpm from <IP_ADDRESS>3 to <IP_ADDRESS>1
nokogiri from <IP_ADDRESS> to <IP_ADDRESS>
rake from 10.4.2 to 10.5.0
require_all from 1.3.2 to 1.3.3
values from 1.5.0 to 1.8.0

hmm, the error haml_lint throws when not adding the parser gem to the dependencies is:
/Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/gems/haml_lint-0.15.2/lib/haml_lint/ruby_parser.rb:1:in `require': cannot load such file -- astrolabe/builder (LoadError)
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/gems/haml_lint-0.15.2/lib/haml_lint/ruby_parser.rb:1:in `<top (required)>'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/gems/haml_lint-0.15.2/lib/haml_lint.rb:12:in `require'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/gems/haml_lint-0.15.2/lib/haml_lint.rb:12:in `<top (required)>'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/gems/haml_lint-0.15.2/bin/haml-lint:3:in `require'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/gems/haml_lint-0.15.2/bin/haml-lint:3:in `<top (required)>'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/bin/haml-lint:23:in `load'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/bin/haml-lint:23:in `<main>'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/bin/ruby_executable_hooks:15:in `eval'
	from /Users/dennisvandehoef/.rvm/gems/ruby-2.1.7/bin/ruby_executable_hooks:15:in `<main>'

I just tested it with the old version of bundler, got the same problem.

Hey @dennisvandehoef, the error message you posted in https://github.com/brigade/haml-lint/pull/109#issuecomment-173493877 (specifically cannot load such file -- astrolabe/builder (LoadError)) suggests this is a different issue, for which a fix was implemented in 5bf550f923f0bfb6f73fe77bd1a1bfa0f4f0ebf5.
I've just released haml-lint 0.16.0, so give that a try and see what happens.

it helps, thanks.


Each of your Bootstrap buttons should be nested within its own div element with the class col-xs-4.

Challenge Use the Bootstrap Grid to Put Elements Side By Side has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

   <button class= "col-xs-4" "btn btn-block btn-primary">Like</button>
   <button class= "col-xs-4" "btn btn-block btn-info">Info</button>
   <button class= "col-xs-4" "btn btn-block btn-danger">Delete</button>

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

  <form action="/submit-cat-photo">
    <input type="radio" name="indoor-outdoor"> Indoor
    <input type="radio" name="indoor-outdoor"> Outdoor
    <input type="checkbox" name="personality"> Loving
    <input type="checkbox" name="personality"> Lazy
    <input type="checkbox" name="personality"> Crazy
    <input type="text" placeholder="cat photo URL" required>
    <button type="submit">Submit</button>
  </form>

Thank you for reporting your issue.
Your code is wrong.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification.
Thank you, and happy coding.

CatPhotoApp
Click here for cat photos.

Like
Info
Delete

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Crazy

Submit

Like
Info
Delete

Like
Info
Delete


Make sure pip is available for integration testing

The latest rawhide does not include pip by default.

@jscotka, what about packing requre for epel8 as well so that enabling an extra copr repo or pypi installation is not needed?


Correct documentation to reflect correct defaults for CONTENT_ENCODING

Overview

CONTENT_ENCODING environment variable currently defaults to true. The documentation states this defaults to false. This change corrects the documentation to reflect the actual default.

Solves for #21

@wesmcouch Thank you! I did not notice this. Sorry for the inconvenience.


Added initial async support, refactoring (use CCError.t instead of ex…

…ceptions).
For review. I'm not sure using error monads was a good move.
Overall I tend to prefer an error monad to an exception nowadays, but I'm a bit worried to make this interact with async.
Basically I made those change since I'm working on an async backend…

Looks good but it's a little strange to use CCError.t in conjunction with async and not core's own error type.
Of course the alternative is writing functors so I can understand.

There is a bug in this code with respect to Async handling of file descriptors. I'm gonna push a fix. And also write more tests.

About CCError, the advantage of it is that is uses polymorphic variants, which can easily coexist with… whitequark's code by example, as opposed to normal variants.

Yes, although in the future, core's error type will be in pervasives.

On 02/06/2015 18:15, Rudi Grinberg wrote:

Yes, although in the future, core's error type will be in pervasives.

Would be awesome! I'm sucks to have to work with different versions of
the same thing.
--
Vincent


[OCPCLOUD-1587] Validate discriminated unions via webhooks

This PR is made up of 2 commits. The first of which is a prefactor to make the second commit easier to work with.
The new content here is a set of validations added so that discriminated unions are validated correctly. IE, when the discriminant is set, only the correct field within the union is set, no extras. This should prevent users from setting invalid configurations

/retest-required

/override ci/prow/e2e-aws-upgrade
This PR doesn't affect the E2Es yet as this isn't in payload


[SPARK-50615][SQL] Push variant into scan.

What changes were proposed in this pull request?
It adds an optimizer rule to push variant into scan by rewriting the variant type with a struct type producing all requested fields and rewriting the variant extraction expressions by struct accesses. This will be the foundation of the variant shredding reader. The rule must be disabled at this point because the scan part is not yet able to recognize the special struct.
Why are the changes needed?
It is necessary for the performance of reading from shredded variant. With this rule (and the reader implemented), the scan only needs to fetch the necessary shredded columns required by the plan.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test.
Was this patch authored or co-authored using generative AI tooling?
No.

@cloud-fan @gene-db @cashmand Please help review, thanks!

@cloud-fan Thanks! I have made the changes you recommended.

thanks, merging to master!


Add option to avoid duplicates

Perform an additional check when True before saving a historical record. Same as in django-reversion
https://django-reversion.readthedocs.io/en/stable/api.html#registration-api. I can submit a pull request.

This has been requested as a feature several times in the past. I'm sure there is interest in having it. I believe a suggested strategy was to leverage Field Tracker. I'm sure a simple dirty flag would work just as well.

Adding on to what @macro1 said, django-reversion mentions the downside of this:

Checking for duplicate revisions adds significant overhead to the process of creating a revision. Don’t enable it unless you really need it!
Without an efficient solution, I don't see it as something that's the responsibility of this library

I could create a configuration that matches the record with diff_against on the last record before saving. The whole thing then costs a database query and some computing power extra, which would be worth it to me personally.
Should I contribute this solution?

Doesn't seem worth it to me with an extra db query. If you could use django-dirtyfields and avoid the db query, I'd be more open to it

I'd love to see any implementation, one which does not produce extra db queries would be preferable, but any implementation would be a start.
Currently we have issues with duplicate histories from calls to Model.objects.update_or_create.
Could we go with @nick-traeger's idea of a quick and dirty implementation, and then eventually transition to a more efficient solution using django-dirtyfields or similar?

What is the consensus on this, are we interested in doing a quick-and-dirty and eventually transitioning to a efficient solution, or will the efficient solution be required before a PR will be merged?
This is quite a critical issue for me, and I think I'd work on it, but I wanna know what the criteria is beforehand. I'm currently using a bulk version of the clean_duplicates method, but it takes a while since historical tables grow quite rapidly:
                       relation                       | total_size
------------------------------------------------------+------------
 public.webapp_historicalconstraintvalue              | 119 GB
 public.webapp_constraintvalue                        | 27 GB

Snippet from postgresql showing the extend of historical records. A rough review shows that 80% of the history entries are duplicates.

Why not have it under some form of setting for the people who need write speed vs people who prefer to avoid background jobs cleaning up the database?


admin area: create new list view to assign languages to tags

already integrated it in current admin interface.
Now I have to make it possible to destroy the unchecked value.


feat: send emails for create/approve/reject and more tests

removed the snapshot tests for various reasons:

requires you to re-snapshot on any single UI change
changes/warnings are not the easiest to understand
updating snapshots is necessary before any push with UI changes

This leads to blindly updating snapshots whenever a UI change is made because updating snapshots are something you have to do anyways. I don't think this achieves anything.
Maybe this will be more useful once everything is completed, when not so many changes are needed to be made at a time


Response rate API call

Adds a response rate API call
Backend checklist

[x] Formatted my code by running autoflake -r -i --remove-all-unused-imports src && isort . && black . in app/backend
[x] Added tests for any new code or added a regression test if fixing a bug
[x] All tests pass
[x] Run the backend locally and it works
[x] Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

@kalvdans: I added a lot more documentation for the returned data, hope that's clearer now!

Thank you. It would be more appropriate to assume unreplied messages follow the same distribution as the replied messages. Otherwise if a person gets bombarded with requests, its p33 and p66 percentiles suddenly goes up, even if the person afterwards replies to them in the usual time.

Thank you. It would be more appropriate to assume unreplied messages follow the same distribution as the replied messages. Otherwise if a person gets bombarded with requests, its p33 and p66 percentiles suddenly goes up, even if the person afterwards replies to them in the usual time.

I've thought about this. Firstly, hopefully response times are in the order of a couple of days, so the time during which you have a worse response rate than your usual is limited. Secondly, I do think it's a good "rate limiter"/regulator: if you get a tonne of requests, and your response rate goes down for two days, then fewer people will send you a request and that's presumably a good thing? We'd prefer no individual is bombarded with all requests anyway.

@kalvdans happy for me to merge?


fix(MessageBox): MessageBox should have tabIndex, role, and aria-label

Needs to match PF guidelines for scrolling content.

Preview: https://virtual-assistant-pr-virtual-assistant-238.surge.sh


add merely useful + turing way to list of resources?

Hi all,
raising an issue about these adding these resources for RSE to the links page (https://intersect-training.github.io/training-links/) instead of making a PR because I'm not sure what category these belong in -- maybe add a section for general guides?
There's the semester long Python-RSE class from @gvwilson and the group collaborating on Merely Useful:
https://merely-useful.github.io/py-rse/index.html
and they also have an Rstats RSE guide under development:
https://merely-useful.github.io/
and there's multiple guides from @KirstieJane @malivakasharan et al. at Turing Way:
https://the-turing-way.netlify.app/welcome.html
(see side panel)
e.g. this could go under collaboration
https://the-turing-way.netlify.app/collaboration/collaboration.html
apologies if these are resources / links you are already aware of or other people have submitted, but thought it might be good to connect diff't groups on GitHub 😁
also, for packaging, see the excellent Python guide (sorry for throwing this in to the same issue)
https://py-pkgs.org/
from @TomasBeuzen and @ttimbers
... wish people would converge on an approach like this wherever possible

@NickleDave - thanks very much for passing these along! These look excellent. And no worries - the issue works great.
We struggled with how to effectively categorize. I have a few other resources others have sent in that haven't been posted yet for the same reason. We'll take a look and see if the categories need some adjustment. Honestly, I'm sure they'll continue to evolve over time. This is just the beginning step.
We'll take a look in more detail at everything and I'll leave the issue open until these get incorporated into the list.
Thanks again!

Glad to hear these are helpful and I haven't just added another issue to your pile :) cheers
And yes totally understand that the categories will evolve


Allow scrolling theme preview when the control picker is active

Fixes https://github.com/godotengine/godot/issues/54778.
I'm not sure if there are any considerations about moving gui_input() override of ScrollContainer to public methods, but some other controls also have it like so for similar reasons (namely, ItemList).

Thanks!


Some question of pretrain_nmt.py

I have some confusion about the pretrain_nmt.py
I just saw that for the first few lines in your pretrain_nmt.py,
from transformers import AutoTokenizer, MBartTokenizer, MBart50Tokenizer, BartTokenizer, AlbertTokenizer from transformers import MBartForConditionalGeneration, BartForConditionalGeneration, MBartConfig, get_linear_schedule_with_warmup
According to my understand，you have rewrite some script such as some Class in (https://github.com/prajdabre/yanmtt/tree/main/transformers/src/transformers/models/mbart)/modeling_mbart.py
in order to reach the goal of further pre train based on mBart.
And why don't you use the function in your new modeling_mbart.py ? I mean why don't you import Class in (https://github.com/prajdabre/yanmtt/tree/main/transformers/src/transformers/models/mbart)/modeling_mbart.py ?

I have some confusion about the pretrain_nmt.py
I just saw that for the first few lines in your pretrain_nmt.py,
from transformers import AutoTokenizer, MBartTokenizer, MBart50Tokenizer, BartTokenizer, AlbertTokenizer from transformers import MBartForConditionalGeneration, BartForConditionalGeneration, MBartConfig, get_linear_schedule_with_warmup
According to my understand，you have rewrite some script such as some Class in (https://github.com/prajdabre/yanmtt/tree/main/transformers/src/transformers/models/mbart)/modeling_mbart.py in order to reach the goal of further pre train based on mBart.
And why don't you use the function in your new modeling_mbart.py ? I mean why don't you import Class in (https://github.com/prajdabre/yanmtt/tree/main/transformers/src/transformers/models/mbart)/modeling_mbart.py ?

Sorry, I missed the information that recover the installation of Transformer from your fold, I will try it


Some empty If node is not locally dce'ed in for-range

The following snippet:
  def rangeForeach1(n: Int): Unit = {
    for (i <- 0 until n)
      println(i)
  }

somehow produces the following JS code:
$c_Lhelloworld_HelloWorld$.prototype.rangeForeach1__I__V = (function(n) {
  var isEmpty$4 = (n <= 0);
  if (isEmpty$4) {
    /*<skip>*/
  };
  var lastElement$4 = (isEmpty$4 ? (-1) : (((-1) + n) | 0));
  inlinereturn$8: {
    if ((!isEmpty$4)) {
      var i = 0;
      while (true) {
        var v1 = i;
        $m_Lhelloworld_HelloWorld$().println__O__V(v1);
        if ((i === lastElement$4)) {
          break inlinereturn$8
        };
        i = ((1 + i) | 0)
      }
    }
  }
});

Although the while loop itself is nice, there is a mysterious empty if lying around:
  if (isEmpty$4) {
    /*<skip>*/
  };

Note that the above is the JSPrinter's way of displaying an If(VarRef("isEmpty$4"), Skip(), Skip()), which we could also write as:
  if (isEmpty$4) {
    /*<skip>*/
  } else {
    /*<skip>*/
  };

It's obviously dead-code in statement position, so it should be removed. We even have code in the optimizer that is supposed to properly dce an if node in statement position at https://github.com/scala-js/scala-js/blob/c30cfe5cc9ef4a8038594d71a6d98a29feed2d8a/tools/shared/src/main/scala/org/scalajs/core/tools/linker/frontend/optimizer/OptimizerCore.scala#L1197-L1201
We should figure out why this happens and fix it. Although that particular example is harmless (the JS VM can obviously dce the above code away), it probably hides a more important shortcoming of the optimizer.

Also removing this one from the talk. And since I think creating PreTransforms for the rest of our dce-able IR nodes will fix this at its root, I'm putting this on hold.


TimeStampedModel improvement ideas

Hi I think the behavior of the TimeStampedModel class is a little misleading / less useful because of the following:

modified is set even though the model was only saved once (when creating).
modified is different than created even though they were set at the same time.

I propose two possible changes to the TimeStampedModel so it can be possible to tell whether or not a model was only created or created and later modified/updated.

modified is set to null on create and only changed on subsequent saves.
modified is set the the exact same value as created on create.

It seems like 1. is a better solution to me because that way the modified field and the AutoModifiedField aren't tied directly to the AutoCreatedField also existing on the model.

The advantage of solution 2 is that it doesn't break backward-compatibility for people relying on the current behavior. For solution 1, we would need to find a backward-compatible deprecation path, we can't just suddenly change that behavior and impact existing code.
With that caveat, I'm really open to either option.

👍 for option 2. It also means you can use the modified time without explicitly checking for null first.


extract sequences after mmseqs search

Hi
Thanks for the great contributions here.
I have a fasta file containing 36 sequences, I am trying to do the homology search against uniprot, Here is what I did:
mmseqs databases UniProtKB db/uniprot tmp
mmseqs createdb input_fasta queryDB
mmseqs search queryDB db/uniprot  resultDB tmp
it looks like everything is running fine, I got the resultDB.* files.
Also I did mmseqs convertalis queryDB targetDB resultDB resultDB.m6
But my real goal here is to get the sequences for each fasta, not just the ids and scores, as my next step is to deduplicate the sequences and then do clustering.
I did
mmseqs convert2fasta resultsDB resultsDB.fasta
it seems like it is not converting:
MMseqs Version: 7ebd2e0441e5c3bdec585317c2b1c3cdbf943568
Use header DB   false
Verbosity       3
Database resultsDB needs header information
Any help or suggestion is greatly appreciated. Thank you

What exactly do you need? The MSA or the sequence detected in the uniprot? For the second you can just use the mmseqs convertalis queryDB  db/uniprot resultDB resultDB.m8 --format-output query,target,tseq here the last value should be the detected target sequence from the uniprot.


Support for hosting multiple openapi schemas with in a Axum application

Currently, the aide library does not seem to support hosting multiple OpenAPI schemas within the same Axum application.
This limitation restricts the flexibility of the application, especially when it comes to managing and documenting larger projects with multiple APIs.
The ability to host multiple OpenAPI schemas would allow developers to better organize their APIs, especially in microservice architectures or in applications with distinct sections requiring separate API documentation.
This feature request is to add support for hosting multiple OpenAPI schemas within the same Axum application.
This would involve modifying the ApiRouter struct and related functions to handle multiple schemas, and ensuring that each schema can be accessed and managed independently.
This feature would greatly enhance the flexibility and scalability of applications built with aide and Axum.

@y-haidar yes! thanks a lot!!


NativeModule call performance reduced since RN58

We at Discord just upgraded from RN57 to RN59 and immediately noticed a slow down while doing certain actions in our app. Since our chat view is actually a native component we ship it large arrays of messages and emojis. It became clear the bottleneck was coming from this. After doing some research of what changed since RN57 it was clear the JSExecutor has been refactored and seems to have changed how it handles arguments of native module methods. Large arrays or dictionaries are up to 10x slower to serialize in method calls.
We tried the most obvious solution and changed our native modules to take a JSON blob that we manually JSON.stringify and then we deserialize manually in the native module. For an average channel on Discord the call time went down from 40ms to 4ms and for our emoji picker it went from 400ms to 30ms.
We solved this problem for ourselves with a workaround, but generally wanted to ask here if this was an intended side effect of the refactor (which I assume was done to enable the coming TurboModules)?
If this is intended feel free to close this issue.
React Native version
React Native Environment Info:
System:
OS: macOS 10.14.5
CPU: (12) x64 Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
Memory: 3.35 GB / 32.00 GB
Shell: 5.3 - /bin/zsh
Binaries:
Node: 11.4.0 - /usr/local/bin/node
Yarn: 1.15.2 - /usr/local/bin/yarn
npm: 6.5.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
IDEs:
Xcode: 10.2.1/10E1001 - /usr/bin/xcodebuild
npmPackages:
react: 16.8.6 => 16.8.6
react-native: discordapp/react-native#0.59.8-discord => 0.59.8-discord
npmGlobalPackages:
react-native-cli: 2.0.1
Steps To Reproduce

Create a basic native module that receives an array as an argument.
Create an array of 1000 small dictionaries.
Invoke NativeModules.MyNativeModule.myTestMethod(array)
Try again on RN57 and notice the performance is much better.

Hey 👋
We just released the first RC for 0.60, would you mind creating a small repro using that version?

@kelset Just tried it out and looks like it is behaving the same as 0.59.8, so unfortunately not fixed.
Anecdotally even with our fix for our specific call sites the whole app feels slower than on 0.57 and not sure the best way to measure the delta. If true that means this is probably affecting other things like React's diffs being sent to the main thread.

Understood - currently how are you benchmarking/which tools are you using?

For the repro steps above can simply wrap it with a console.log for the most simple debug tool :)
const startTime = Date.now();
NativeModules.MyNativeModule.myTestMethod(array);
console.log('Call Time', Date.now() - startTime);

Assuming you follow the repro steps this should spit out a very large number.


Incorrect input type for bulk delete operation

The exported interface for delete on bulkOperation (https://github.com/contentstack/contentstack-management-javascript/blob/master/types/stack/bulkOperation/index.d.ts#L8) should include a named attribute "details" to contain the interface BulkDeleteConfig.
The delete operation (https://github.com/contentstack/contentstack-management-javascript/blob/master/lib/stack/bulkOperation/index.js#L209) expects it.

Thanks @michauds for raising the issue, we will work on this and let you know once we release fix for the issue.

HI @michauds we will be taking this issue in the upcoming sprint and let you know once the fix is released
cc: @netrajpatel @sunil-lakshman @shafeeqd959

This has already been fixed @cs-raj. Hence closing this.


chore: Cleanup OS files

See: #4098

@DaniHalfin Any thoughts on this PR?

This is a reasonable thing to do for contributors using Git / Git Extension / Git GUI applications to contribute while also browsing their local cloned repository folder on Apple MacOS (.DS_Store) or Microsoft Windows  (Thumbs.db). This should make it easier to avoid OS files being added in Pull Requests (or even added in commits via MS Docs team members using Git from their local computer OS).

@DaniHalfin @lizap @mjcaparas Any thoughs on this PR that introduces changes to Git dotfile? Thanks.

@Dansimp Following up for updates.

The default branch for this repository has changed. Commenting to remember to merge this to the new default, if this PR gets merged.

@DaniHalfin I used the Edit to change the target branch to "public"

Thanks, @nschonni !

No problem, you probably have the ability to do this to the other PRs if you want.
PS: I prefer the old name though, because git alias scripts assume that as the name. Also, other docs repos that have changed it usually use "live".

Thanks for the comments, useful to know. BTW, isn't Live branch already in use in the private branches?


Allowing editor to inherit height

Currently, the editor has a max height due to a hard-coded style property.
Removing it allows the editor to take all the available height.

Tested in local and worked.
Here's the editor in a 800px height container


[Event Request] Codeunit "User Setup Management" -

We need to be able to control the allowed posting period.
Our suggestion is to include an event in Codeunit User Setup Management (ID 5700) in the function IsPostingDateValid:

I like


[FEATURE REQUEST] add getters for SQLServerSpatialDatatype members

Is your feature request related to a problem? If so, please give a short summary of the problem and how the feature would resolve it
I am working on a project where I need to translate a Geography object into another spatial class, that implements a standard other than WKT/WKB
Describe the preferred solution
I would like to be able to access the members of SQLServerSpatialDatatype that describe the spatial region, i.e. pointList, figureList and so on.
Describe alternatives you've considered
Another option would be to change the visibility of the constructors of Geography from private to protected, so that I would be able to inherit from Geography and access the members from there, since they are protected.

Hi @vforchi, I've addressed this issue in PR 1035 (the same PR for the last related issue). I've declared the default constructor as protected so that Geometry/Geography can be extended.

I think it would be useful to also have the other constructors protected, otherwise I will just end up copying that code in one of my constructors.

STGeomFromText and STGeomFromWKB methods give you access to the constructor (it's basically the same as calling the constructors themselves, except we do it in SQL Server fashion). Do you still need to copy the contents of the private constructors?

Yes, but those methods would create a Geography, and not my class.

Ok, I made them protected as well. Unless there's anything else you need added, after internal reviews I'll merge in the PR.

Thanks, I think that's all I need.

Fixed in PR #1035. Closing issue.


When clicking mouse down on a columns edge, change the whole column edge style to a thicker blue line

The only visual clue that you are dragging a columns' edge is that of a smaller drag handle cursor style.  It would be better to enhance the column's edge visually to allow users to much more quickly spot and see that they grabbed a column's edge.  The problem is exacerbated especially when there is not much data or no data in a cell.
Proposed solution
The main purpose is to give the user a bigger indication that they have a good handle on a columns' edge.
It would be visually appealing and easier to see a columns edge whiling resizing it, if the entire column edge line could be highlighted with a somewhat different bolder style (a thicker blue line), like other apps do?  I would only want the style to change to a thicker blue line, once the mousedown event was fired (or rather when the dragging event started).  I would not prefer it if the implementation was done on the hover event while just moving the mouse along header rows, for example, it would cause undue confusion for users that are just moving their mouse, and have not yet decided to grab a column edge mousedown and move it mousemove.
Alternatives considered
No enhancement and keep the thin blue column edge style while dragging and the only indiction of successful grab of a column edge with the small black cursor drag handle, but this makes it harder to quickly see that you are dragging a column's edge, especially depending on if there is text in the cell, or just an empty cell.
Additional context

We can also make it that while the user is dragging, only the thick blue line is moving, and not the actual column boundary. The advantage of this is that we avoid "reflowing" the elements continuously, potentially making the whole experience more fluid on machines with limited resources. This is what Google Sheets does, for instance.

This is highly dependent I'd say on implementation testing for jankyness maybe with 1000 displayed rows. IF there is a lot of jankyness...then sure, you could do something like a helper, but I also wonder how performance might be with ghosting and how it might feel or look like and it's performance during implementation testing across the 3 browsers.


Backport allowed_outbound_hosts and explain SDK interaction

Outbound restrictions for databases and Redis are applied on the basis of SDK version not manifest version.  Therefore, there is no need to update v1 SDK docs to refer to outbound allow-lists.  (There is some nuance about what is a v1 SDK vs what is a v2 SDK but that's best avoided.  For now, we want people to err on the side of allow-listing their hosts.)
As v2 SDKs may be used with v1 manifests, we are adding allowed_outbound_hosts to the v1 manifest (but only in Spin 2).
Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[ ] The title, template, and date` are all set
[ ] Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
[ ] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
[ ] Has passed bart check
[ ] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[ ] Headings are using Title Case
[ ] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[ ] Have tested with npm run test and resolved all errors
[ ] Relates to an existing (potentially outdated) blog article? If so please add URL in blog to point to this content.

I've reworked this significantly based on the changes in the latest Spin PRs.  Would be good to nail it ASAP @rylev if you are still around.

@tpmccallum Unless @rylev is around, I think you might need to override the request changes (if it looks okay to you) - I'm pretty sure I've done what Ryan needs but it's late where he is but I don't want to wait a full 24-hour cycle before merging this.  We can of course make further changes, if he has them, in a follow-up PR.


copr:sub_date_and_string

What problem does this PR solve?
Issue Number: fix #9016
Problem Summary:
What is changed and how it works?
What's Changed:add push-down & test for sub_date_and_string
Check List
Tests

Unit test

/cc @skyzh

PTAL @andylokandy

This PR seems stale. Would you please sign DCO and fix merge conflicts?


Palm to google

deprecates *-palm methods
adds *-google methods instead
warnings for everything
still sends "*-palm" to weaviate as older versions do not support *-google
removes some unneeded classes that we added during the named vectors introduction

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.26%. Comparing base (624e2e2) to head (46a8dec).
Report is 90 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1329      +/-   ##
==========================================
- Coverage   94.38%   94.26%   -0.13%
==========================================
  Files         220      220
  Lines       20953    21002      +49
==========================================
+ Hits        19776    19797      +21
- Misses       1177     1205      +28

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Any plan to support Microsoft Edge?

Chrome Store 在中国大陆无法访问。

Of course, Microsoft Edge is in our support plan. Maybe in version 0.9 or version 1.0. Please stay tuned!

Hey @zombie110year . Rust Search Extension is available on Microsoft Edge, welcome to give it a try! 🎉
https://microsoftedge.microsoft.com/addons/detail/ebibclchdmagkhopidkjckjkbhghfehh


CLI Release Candidate tarball is not being updated

The CLI docs provide a URL to download the release candidate tarball over here:
https://developer.salesforce.com/docs/atlas.en-us.sfdx_setup.meta/sfdx_setup/sfdx_setup_install_cli_rc.htm
But when you pull that URL, the tarball is actually the CLI from 4 or 5 weeks ago (7.140 to be exact)
https://developer.salesforce.com/media/salesforce-cli/sfdx/channels/stable-rc/sfdx-linux-x64.tar.xz
This is similar to the bug report over on #1476 where it appears as though what once worked (pulling the NPM module via the stable-rc tag) is no longer working?
It should be noted that the instructions for pulling the stable tarball are still accurate (i.e. it still pulls the most current stable version)
https://developer.salesforce.com/docs/atlas.en-us.sfdx_setup.meta/sfdx_setup/sfdx_setup_install_cli.htm#sfdx_setup_install_cli_linux
Expected:

the tarball at that URL should resume being updated weekly with the latest release candidate
OR, less desirable, the docs should be updated AND the old URLs should return 404 errors so that people who have automation and depend on that URL don't mistakenly think that it's the "release candidate" when it's most certainly not

Thanks for reporting @daveespo - getting right on this!

This will be fixed with this PR: https://github.com/salesforcecli/sfdx-cli/pull/532
After the release tomorrow the .xz files should be updated.  The .gz files were being properly copied to that location so if you want to temporarily switch to them you could.

Actually @daveespo they are updated now.  Closing this.  Thanks for reporting!

@shetzel  thanks for the speedy fix -- confirmed, my CI is now building with sfdx-cli/7.146.0 linux-x64 node-v16.14.2


End game - ve

Changes proposed in this PR:

#231
#248
#254
#263
#251

#261
#259

I think this looks good.

Tests checkout. Merging.


pip package not working

the package required to upload our own package, twine, constantly throws an "Command not found" error.
for some reason, aide_document package cannot be imported even though the version number and license are updated to the correct version

Please see what I wrote here. Don't use twing. When something says"command no found" it means that probably the command is not installed... ie, the program twine is not installed. You need to upload the package using python setup.py sdist bdist_wininst upload as I specified in the ReadMe.

If you want to use twine, you can, just make sure to install it! https://pypi.python.org/pypi/twine

All of us were sure that twine was installed, and uninstalled/reinstalled it numerous times, but it still wouldn't run the command. I tried to manually add the command as well, but I couldn't find it on my computer.
I tried running the python setup.py sdist bdist_wininst upload command after changing the version number, but I keep getting a LookupError: unknown encoding: mbcs in the Python 3.6 distribution itself. The problem seems to be stemming from the entire setup.py file, and I can't localize it (no matter what lines I comment out, it still throws the same error.
I'm thinking that we should just give up on the pip install for now? Or would you be able to try and distribute it from your machine?

First thing to test is see if you are able to distribute anything from your machine. Go ahead and download the funniest package from this tutorial and try to upload it to your own pip account. I can look at it if you are not able to get that running.

I went through the tutorial, and it seems to work exactly as specified - you can try downloading this package and then running
$ python
>>> import pip_pack_test
>>> pip_pack_test.joke()

which should output some joke in German. However, when I try going through the upload process (as detailed in the tutorial) on the aide_document package, it runs into an authentication error. I've tried copy/pasting the password, messing with the .pypirc file, and typing out the password manually, but the error persists at the upload step.
Could you try running python setup.py register sdist upload in our repo and seeing if that causes problems too?

Upload failed (410): Gone (This API has been deprecated and removed from legacy PyPI in favor of using the APIs available in the new PyPI.org implementation of PyPI (located at https://pypi.org/). For more information about migrating your use of this API to PyPI.org, please see https://packaging.python.org/guides/migrating-to-pypi-org/#uploading. For more information about the sunsetting of this API, please see https://mail.python.org/pipermail/distutils-sig/2017-June/030766.html)
error: Upload failed (410): Gone (This API has been deprecated and removed from legacy PyPI in favor of using the APIs available in the new PyPI.org implementation of PyPI (located at https://pypi.org/). For more information about migrating your use of this API to PyPI.org, please see https://packaging.python.org/guides/migrating-to-pypi-org/#uploading. For more information about the sunsetting of this API, please see https://mail.python.org/pipermail/distutils-sig/2017-June/030766.html)

This happened after running python setup.py register sdist upload on an OS X

Okay - I finally got it to work! Turns out that there was some extraneous stuff in my setup.py file that was preventing python setup.py sdist from packaging the actual code that we needed into the .tar.gz installation file. I deleted everything that I could, made sure that I had packages=find_packages() within setup.py, and then it built everything correctly. Then, I ran twine upload dist/*, uploaded onto pypi.org, and now our import statements work.
 I'll write up a CONTRIBUTING.md to go through the upload process and start using tags in this repo to keep our versions in check.

Thanks! Yeah - it's these kinds of things that take a surprisingly long time 😛


Use SOSL instead of SOQL for lookup search box

Lookup searches in Gift Entry should perform better inside of orgs with large data volumes for the object being searched.
When hovering over a lookup option in Gift Entry, the full name of the result will be displayed in a tooltip.
When the Gift Entry lookup component loses focus, the lookup options dropdown will now close
When the Gift Entry lookup component gains focus, and a result has not yet be selected but there is a search term present, the lookup options dropdown will re-open if there are search results.

Critical Changes
Changes
Gift Entry Updates

Orgs with large data volumes will see results faster when searching in lookup fields.

Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch: attach W-041797

Tracking W-041797

I think this is OK and maybe even preferable. Thanks!
On Wed, Jun 17, 2020 at 10:52 AM Vladimir Martinov<EMAIL_ADDRESS>wrote:

@vmartinov-sforg commented on this pull request.
The code for the sosl implementation looks good.
I am searching for the string 'si' in the Existing Donor Contact field,
and the query that we submit is FIND 'si' IN NAME FIELDS Returning
Contact(Name) LIMIT 50. We get some results that we may not expect that
have the 'si' substring in the account name. This is a byproduct of using
sosl, and it may be okay, but I would like ux and product to confirm.
@ameyersalesforce https://github.com/ameyersalesforce
[image: Screen Shot 2020-06-17 at 10 22 45 AM]
https://user-images.githubusercontent.com/48025329/84931901-51a48a00-b088-11ea-8de5-657a43140806.png
[image: Screen Shot 2020-06-17 at 10 19 27 AM]
https://user-images.githubusercontent.com/48025329/84931915-56693e00-b088-11ea-83e2-4728ce35d456.png
In src/classes/GE_LookupController.cls
https://github.com/SalesforceFoundation/NPSP/pull/5550#discussion_r441698116
:

@@ -46,12 +47,16 @@ public with sharing class GE_LookupController {
*/
@AuraEnabled(cacheable=true)
public static List doSearch(String searchValue, String sObjectType) {

   String likeLike = '%' + searchValue + '%';

   String queryString = 'SELECT Id, Name FROM ' + String.escapeSingleQuotes(sObjectType) + ' WHERE Name LIKE :likeLike';

   String formattedValue = '\'' + String.escapeSingleQuotes(searchValue) + '\'';

   String searchTemplate = 'FIND {0} IN NAME FIELDS Returning {1} LIMIT 50';

Can we define a constant for the LIMIT number?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SalesforceFoundation/NPSP/pull/5550#pullrequestreview-432604375,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG3H4RGJTCA7JT3PKV4GYATRXD7E3ANCNFSM4NZOV3LQ
.

--
Alison Meyer
Sr Product Manager | Salesforce.org http://salesforce.org/
Mobile:<PHONE_NUMBER>


Create a form to add new collections

The form should contain the following input elements (bold is required):

Unique identifier for in Dexter (auto generated and read-only)
Title
Description (can also be used as a general notes field)
Main or sub collection (the collection need to be dynamic, so we still have think about this)
Creator (in a 'Dexter collection' this will always be the user itself)
Subject (keywords, can also be used to filter the collections)
Rights
Access (open, restricted, closed)
Created
Spatial (the spatial space of the collection. E.g. Tunisia, Turkey, Algeria, etc.)
Temporal (temporal range of collection. E.g. 1920–1960)
Language (all languages in collection)
(Contributor) (optional for now)

Behind the scenes, the following elements need to be added to the metadata:

Persistent identifier to the collection in Dexter
Last updated
Exact creation date and time of collection
Which user created the collection

A form where you can add new collections has been implemented. You can also edit the existing collections. The 'main or sub collection' functionality is not yet implemented.


Fix the comment of ServiceInstance

What is the purpose of the change
Fix the comment of ServiceInstance

Codecov Report

Merging #8674 (de5c51a) into 3.0 (82f9a6b) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.0    #8674      +/-   ##
============================================
- Coverage     63.76%   63.64%   -0.12%
+ Complexity      312      311       -1
============================================
  Files          1141     1141
  Lines         47846    47800      -46
  Branches       7194     7159      -35
============================================
- Hits          30508    30424      -84
- Misses        13985    14018      +33
- Partials       3353     3358       +5

Impacted Files
Coverage Δ

.../apache/dubbo/registry/client/ServiceInstance.java
50.00% <ø> (ø)

...ng/transport/dispatcher/WrappedChannelHandler.java
43.47% <0.00%> (-15.22%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-13.80%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

...moting/transport/netty4/NettyEventLoopFactory.java
22.22% <0.00%> (-7.78%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

...ache/dubbo/remoting/api/NettyEventLoopFactory.java
30.00% <0.00%> (-6.37%)
:arrow_down:

.../threadpool/support/limited/LimitedThreadPool.java
83.33% <0.00%> (-5.56%)
:arrow_down:

...nt/metadata/StandardMetadataServiceURLBuilder.java
86.04% <0.00%> (-4.66%)
:arrow_down:

... and 40 more

Continue to review full report at Codecov.

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


Mitm

Hi!
From docs i cant get fully understandig how i can use mitm.
i need to generate  my own sertificates or use  mitm sertificates from scrapoxy
Can u help with config please?
i use aws configs
    "proxy": {
        "mitm":true
    },

or
    "proxy": {
        "mitm":{
            "cert_filename":"/home/ubuntu/app/scrapoxy/certs/cert.pem",
            "key_filename":"/home/ubuntu/app/scrapoxy/certs/key.pem"
        }
    },

or by default certificates from scrapoxy
    "proxy": {
        "mitm":{
            "cert_filename":"",
            "key_filename":""
        }
    },

Hello,
The first solution (true) will works.
Fabien.


Kpt pkg init: load the upstream info if the package is in gitops

I have a package pushed up to a git repo and would like to run kpt pkg diff against it.
Kpt requires to have a Kptfile under the package, so I did kpt pkg init my-pkg. However, kpt pkg diff still failed because of Error: must specify repo.
What I ended up doing is to delete my package locally, and then use kpt pkg get<EMAIL_ADDRESS>to get the package and Kptfile.
$ kpt pkg diff my-pkg
Error: must specify repo

$ rm -rf my-pkg
$ kpt pkg get git@github.com:nan-yu/my-repo.git/my-pkg .

I was thinking kpt pkg init could automatically load the upstream info if the package is in git.
Instead of a simple initial Kptfile:
apiVersion: kpt.dev/v1alpha1
kind: Kptfile
metadata:
  name: my-pkg
packageMetadata:
  shortDescription: sample description

It should include the upstream section:
apiVersion: kpt.dev/v1alpha1
kind: Kptfile
metadata:
    name: my-pkg
upstream:
    type: git
    git:
        commit: 98c819b489a1aa93ce063264445ad97485592021
        repo: git@github.com:nan-yu/my-repo
        directory: /my-pkg
        ref: master

I think we are clubbing 2 different use-cases here.

User does kpt pkg init when they want to create a package on local and publish it.
Population upstream information is done by kpt pkg get which is a consuming use-case.

Initializing with upstream is not straight forward as there are some additional information which user should pass such as directory, tag etc. I think we should keep consumption and publishing use-cases separate and initialize Kptfile the way we do now.


undefined method model exists on rails g devise_two_factor

/devise_two_factor_generator.rb:19:in inject_field_types': undefined method model_exists?' for #ActiveRecord::Generators::DeviseTwoFactorGenerator:0x00007fd2c317e6c8 (NoMethodError)

@aki1221 can you provide a little more background for this issue?  eg. steps to reproduce, full backtrace, etc?

@williamatodd  sure,
steps to reproduce
1.create a project with rails 6 and ruby 2.6 and setup devise
2.install devise -2fa gem
3. on running rails g devise_two_factor:install (will give adepriciation warning)
4.running rails g devise_two_factor User will raise the above error

@aki1221 I'm seeing the same exception.  When you encountered this problem, did the generator produce a migration file and insert :two_factorable into the User's Devise configuration?


Add control_chars check

Description
This pull request addresses a rendering problem that was identified with unacceptable control characters present in builds.  The pull request adds a new check to the googlefonts profile that addresses the question: "Are there any unacceptable control characters present in the font file?".
Unacceptable CC are defined as all Unicode control character block (U+0000 - U+001F) glyphs except:

.null (U+0000)
CR (U+000D)

This PR adds three new test font files under SIL OFL v1.1 that include:

acceptable control characters
one unacceptable control character
multiple unacceptable control characters

and a single new unit test in the googlefonts_test.py module that tests this new check against single / multi-font file testing and single + multi unacceptable control character contents.  This is passing locally for me with the Py37 interpreter.
To Do

[x] update CHANGELOG.md
[x] wait for checks to pass (except github/pages, which is stuck)
[x] request a review

cc @davelab6

Please, also add an entry below "New Checks" on CHANGELOG.md including check-id, short-string description and a hashtag reference to this pull request (just format it the same way we've done in past releases).

Changelog updated with new check.  I think that I've addressed everything that you requested here Felipe.  Let me know if you need anything else.

It all looks good. Please only rename the check to com.google.fonts/check/family/control_chars and then we'll be good to merge.

Please only rename the check to com.google.fonts/check/family/control_chars and then we'll be good to merge.

Sorry I missed that Felipe!  Addressed that in:
https://github.com/googlefonts/fontbakery/pull/2430/commits/c158fe86c5683e7b6058eaeebe16cfa4b895d5c5

thanks, @chrissimpkins

my pleasure! interested to see what we find on a check across the entire GF set :)

well... I tried to do it now and fontbakery treats all 2500 TTFs as a single family. We will need to improve FB to build clusters of files by directory before running family-wide checks for the whole collection at once.
I made a quick and hacky change to the code locally to run this check per single font file.
This is what I got:

This gives us a rough idea of the state of the collection.
But you can see FB stopped at 93%. I got a heavy slow-down and a "Cannot allocate memory" crash :-P

Yikes... is it running all of them in parallel?

@davelab6 see Felipe's image in https://github.com/googlefonts/fontbakery/issues/2430#issuecomment-478185050 for the state of control chars in ~ the entire collection.


[Bug]: ts_project with esbuild with transpiler enabled produced stale build

What happened?
When ts_project is used with transpiler and the outputs be made available to esbuild, it results in stale build. The code changes are not reflected in the final compiled bundle.
What I expect: it should build correctly.
Version
Development (host) and target OS/architectures: Rock Linux 9.1 (Blue Onyx) x86_64
Output of bazel --version: 6.2.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
Indicated in the provided link of the repo
Language(s) and/or frameworks involved: JavaScript, TypeScript, React
How to reproduce
I have a repo to reproduce it here https://github.com/Exegetech/buggytsproject
Any other information?
No response

I suspect this was https://github.com/aspect-build/rules_esbuild/issues/58 - which is just recently fixed. Could you confirm if that fixes it for you pls @Exegetech ?


Bootstrap 5 + Jquery animate scrollTop laggy on Chrome only

Prerequisites

[x] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Bootstrap 5.1.3 and Jquery 3.6.0 scrollTop animation not start immediately after click on Chrome, but works normlly on other browser(Firefox).
Sometimes the animation start after a while (maybe 500 ms) after click, sometimes it start with slow speed then as normal.
Reduced test cases
I tried this minimal code below (Bootstrap 5.1.3) and the problem appears. I also tried several code with other versions of Bootstrap (from 5.0.0 to 5.1.2) and the problem still appears.
<!DOCTYPE html>

    <link rel="stylesheet" media="all" href="https://cdnjs.cloudflare.com/ajax/libs/bootstrap/5.1.3/css/bootstrap.min.css" />

Back to Top

I tried another code below (older version of Bootstrap 4.6.1), animation works normally.
<!DOCTYPE html>

    <link rel="stylesheet" media="all" href="https://cdnjs.cloudflare.com/ajax/libs/bootstrap/4.6.1/css/bootstrap.min.css" />

Back to Top

So this problem may only appears on Bootstrap 5 and Chrome, I have no idea what caused it and how to fix it.
What operating system(s) are you seeing the problem on?
Windows, Linux
What browser(s) are you seeing the problem on?
Chrome
What version of Bootstrap are you using?
5.1.3

jQuery scroll script has conflict with scroll-behavior: smooth in _reboot.scss line 33. Just override it with css.
html {
  scroll-behavior: auto !important;
}

You can also add inline style using jQuery css() method:
$(document).ready(function (){

  $("html").css('scroll-behavior', 'auto');

  // click handler here

}};

If anyone thinks this should be in our docs, please make a PR. Thanks!

Hi guys, FYI I encountered an issue with Behat tests when upgrading to Bootstrap 5 and it seems Panther users have the same issue, see https://github.com/symfony/panther/issues/502.
@crftwrk's fix solved my issue, thanks!

To disable smooth scroll, I'd recommend you set $enable-smooth-scroll to false in a custom Sass build.
Since there's a core option to handle this and a simple CSS workaround, I'm closing this for now.
FWIW scroll-behavior is standard CSS, so I'd say the jQuery plugin should support it.

Lol. I was migrating from 4 to 5. I changed my code, classes, scripts... Many hours of not creative work that i hate :D Finally I find out that my scrollTop animations are not working good with my custom easings... Another 3 hours of scratching my head... Finally got here :D Ty for explaining the problem cause :D


Save All issue...

When I merged all the three(Luyten GUI, RSyntaxArea and Procyon Decompiler), I found an issue when trying to save the sources in the form of zip. Please fix the issue for Save All...menu item.
Thanks

Could you describe the issue in greater detail, I'm unsure what you issue for Save All... I attempted to save-all / decompile itself and I'm successful, If you can elaborate on your issue I'd be happy to help.

Duplicates #11


feat: some measure preserving equivalences on pi-types

Also fix the statement of some lemmas of an earlier PR.
From the Sobolev project
There are more that depend on #7341

Co-authored-by: Heather Macbeth<EMAIL_ADDRESS>

bors merge


there is not dps310_i2c.h

there is not dps310_i2c.c

Hi guys,
Thanks for your interest. And sorry for delay. Here is my implementation for a SAMR34 MCU
https://github.com/sensidev/sensor-dps310/blob/master/examples/dps310_i2c.c

Thanks for the sharing


Use absolute URL's for the assets

I want to be able to use clean URLs with trailing slashes for the links in my generated docs (e.g. page.html becomes /page/), however all the references to the assets are relative, so they all break when using clean URLs with a trailing slash.
Is there any way to make the links to the assets relative (I provide a base path)?

Momentarily I am doing this bad hack in typedoc.js to get base path/url prefix
  const Handlebars = require('handlebars');

  const BASE_URL="https://some.app/";

  Handlebars.registerHelper('relativeURL', function(url) {
    return BASE_URL + url;
  });

  module.exports = {
  // typedoc configs
  }

I went for another hack. It's broken only for the index.html page. There are not many links, so this should be good enough:
sed -re 's/<(\w+)(.*)href="([^http].*)"(.*)>/<\1\2href="api\/\3"\4>/g' -i ./dist/index.html
sed -re 's/src="([^http].*)">/src="api\/\1"/g' -i ./dist/index.html

Basically I use the sed utility to create my own path relative a subroute api.
I am sure my regexp isn't 100% reliable, but for this use case it should be good enough.

Momentarily I am doing this bad hack in typedoc.js to get base path/url prefix
  const Handlebars = require('handlebars');

  const BASE_URL="https://some.app/";

  Handlebars.registerHelper('relativeURL', function(url) {
    return BASE_URL + url;
  });

  module.exports = {
  // typedoc configs
  }

sorry to bother, but could you explain more? Because I tried to config in the same way but didn't work.

Momentarily I am doing this bad hack in typedoc.js to get base path/url prefix
  const Handlebars = require('handlebars');

  const BASE_URL="https://some.app/";

  Handlebars.registerHelper('relativeURL', function(url) {
    return BASE_URL + url;
  });

  module.exports = {
  // typedoc configs
  }

@RohitRox sorry to bother, but could you explain more? Because I tried to config in the same way but didn't work.

I get it. Basically I need to overwrite the custom helper "relativeURL" initally defined in typedoc, to do that I need to register another one and make sure that both typedoc and my own project are using the same Handlebars.

+1

+1

Slightly confused how this works in the actual typedoc website where generated docs are hosted at /example and /api? Links throughout the docs are all generated relative to those. Ie. /example/assets/main.js

GitHub Pages doesn't work with relative paths. This option is necessary.
https://www.pluralsight.com/guides/fixing-broken-relative-links-on-github-pages

It doesn't work for Zeromq.js. I just broke the documentation so that you can check:
http://zeromq.github.io/zeromq.js/modules/_index_.html
The source
http://zeromq.github.io/zeromq.js/
https://github.com/zeromq/zeromq.js
Here is the reason:
https://maximorlov.com/deploying-to-github-pages-dont-forget-to-fix-your-links/

The site you linked to is built with an ancient version of TypeDoc... which isn't even TypeDoc, it's an unmaintained fork. The problem you're running into is not relative links, but that the githubPages option was added in 0.22.4. Prior to that, if deploying to GitHub pages, you had to manually create a .nojekyll file in your deployed directory to prevent GitHub pages form removing files starting with _ from your deployed site.

Slightly confused how this works in the actual typedoc website where generated docs are hosted at /example and /api? Links throughout the docs are all generated relative to those. Ie. /example/assets/main.js
edit: ah, trailing slash is important. I was using next.js as a proxy for the generated output which strips trailing slashes by default.

I was in the same situation (deploying to a subdirectory in Vercel: /public/docs). As @chrisui stated, make sure all links to your TypeDoc directory end with a trailing slash until this issue is resolved or an option is provided.
Do NOT do this:
example.com/docs

Do this:
example.com/docs/

An option --useHostedBaseUrlForAbsoluteLinks is included in TypeDoc 0.26, which is releasing 2024/06/21


Runtime error

I'm working with a Macbook Pro, Intel, python 3.9.11, using MacOS Ventura. I've followed the instructions in the readme to install the dependencies. I'm just trying to run the first example in a python script like this:
import holmes_extractor as holmes
manager = holmes.Manager("en_core_web_trf", number_of_workers=1)
manager.register_search_phrase('A big dog chases a cat')
manager.start_chatbot_mode_console()

I get the following traceback:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/spawn.py", line 116, in spawn_main
    exitcode = _main(fd, parent_sentinel)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/spawn.py", line 125, in _main
    prepare(preparation_data)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/spawn.py", line 236, in prepare
    _fixup_main_from_path(data['init_main_from_path'])
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/spawn.py", line 287, in _fixup_main_from_path
    main_content = runpy.run_path(main_path,
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/runpy.py", line 268, in run_path
    return _run_module_code(code, init_globals, run_name,
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/runpy.py", line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/Users/jose/workspace/dm_disclosure_extraction/disclosure_prototypes/holmes.py", line 3, in <module>
    manager = holmes.Manager("en_core_web_trf", number_of_workers=1)
  File "/Users/jose/.pyenv/versions/3.9.11/envs/experiments/lib/python3.9/site-packages/holmes_extractor/manager.py", line 271, in __init__
    self.multiprocessing_manager = MultiprocessingManager()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/context.py", line 57, in Manager
    m.start()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/managers.py", line 554, in start
    self._process.start()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/process.py", line 121, in start
    self._popen = self._Popen(self)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/popen_spawn_posix.py", line 32, in __init__
    super().__init__(process_obj)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/popen_fork.py", line 19, in __init__
    self._launch(process_obj)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/popen_spawn_posix.py", line 42, in _launch
    prep_data = spawn.get_preparation_data(process_obj._name)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/spawn.py", line 154, in get_preparation_data
    _check_not_importing_main()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/spawn.py", line 134, in _check_not_importing_main
    raise RuntimeError('''
RuntimeError:
        An attempt has been made to start a new process before the
        current process has finished its bootstrapping phase.

        This probably means that you are not using fork to start your
        child processes and you have forgotten to use the proper idiom
        in the main module:

            if __name__ == '__main__':
                freeze_support()
                ...

        The "freeze_support()" line can be omitted if the program
        is not going to be frozen to produce an executable.
Traceback (most recent call last):
  File "/Users/jose/workspace/dm_disclosure_extraction/disclosure_prototypes/holmes.py", line 3, in <module>
    manager = holmes.Manager("en_core_web_trf", number_of_workers=1)
  File "/Users/jose/.pyenv/versions/3.9.11/envs/experiments/lib/python3.9/site-packages/holmes_extractor/manager.py", line 271, in __init__
    self.multiprocessing_manager = MultiprocessingManager()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/context.py", line 57, in Manager
    m.start()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/managers.py", line 558, in start
    self._address = reader.recv()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/connection.py", line 255, in recv
    buf = self._recv_bytes()
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/connection.py", line 419, in _recv_bytes
    buf = self._recv(4)
  File "/Users/jose/.pyenv/versions/3.9.11/lib/python3.9/multiprocessing/connection.py", line 388, in _recv
    raise EOFError
EOFError

The advice in the stack trace refers to your script rather than to the library. I reproduced the problem on my own Mac and solved it by replacing your code with
 if __name__ == '__main__':
    import holmes_extractor as holmes
    manager = holmes.Manager("en_core_web_trf", number_of_workers=1)
    manager.register_search_phrase('A big dog chases a cat')
    manager.start_chatbot_mode_console()


activate-global-python-argcomplete: error: [Errno 13] Permission denied: '/etc/bash_completion.d/python-argcomplete'

I am getting activate-global-python-argcomplete: error: [Errno 13] Permission denied error how can I fix it?
$ activate-global-python-argcomplete
Installing bash completion script /etc/bash_completion.d/python-argcomplete
usage: activate-global-python-argcomplete [-h] [--dest DEST] [--user] [--no-defaults] [--complete-arguments ...]
activate-global-python-argcomplete: error: [Errno 13] Permission denied: '/etc/bash_completion.d/python-argcomplete'
Please try --user to install into a user directory, or --dest to specify the bash completion modules directory

Have you tried following the directions in the error message?

First I tried sudo activate-global-python-argcomplete // but activate-global-python-argcomplete --user I was looking for. Thanks


fix: use darkmode state hook to manage mode

I think we can just use the hook now to manage everything since the root component syncs the dark mode config to the app state.
Fixes #1248
Let me know if I'm missing something 🤔
Changing via the app state
https://github.com/scalar/scalar/assets/6374090/a45a3d35-bc6d-43ab-b306-4bbbd5f1c6f4
Changing via the config object
https://github.com/scalar/scalar/assets/6374090/52765838-0aff-459e-be68-b155d5fdec27

@hwkr this hook has side effects that should be applied outside the top level component. This is why the isDarkMode value is provided as a prop.

No worries, this was just trying to fix the regression #1123. I'm not quite sure what your vision was for handling the data here so I'll just close this PR and you can do what your original plan was 😊


v4: Android hardware back button doesn't work

Bug Report
Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
Ionic:

   ionic (Ionic CLI)          : 4.0.0-rc.9 (C:\Users\ape\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework            : @ionic/angular 4.0.0-alpha.7
   @angular-devkit/core       : 0.6.8
   @angular-devkit/schematics : 0.6.8
   @angular/cli               : 6.0.8
   @ionic/ng-toolkit          : not installed
   @ionic/schematics-angular  : 1.0.0-rc.9

Cordova:

   cordova (Cordova CLI) : 8.0.0
   Cordova Platforms     : android 6.3.0 ios 4.5.4

System:

   Android SDK Tools : 25.2.5
   NodeJS            : v10.5.0 (C:\Program Files\nodejs\node.exe)
   npm               : 6.1.0
   OS                : Windows 10

Describe the Bug
Android hardware button doesn't work as intended. It seems to be because of back button behavior overriding in generated cordova.js:
var backButtonChannel = cordova.addDocumentEventHandler('backbutton');
        backButtonChannel.onHasSubscribersChange = function() {
            // If we just attached the first handler or detached the last handler,
            // let native know we need to override the back button.
            exec(null, null, APP_PLUGIN_NAME, "overrideBackbutton", [this.numHandlers == 1]);
        };

which is caused by:
https://github.com/ionic-team/ionic/blob/181cd559cc68542d48a33e6ff58408e2ffc2077c/angular/src/providers/platform.ts#L42
Steps to Reproduce
Steps to reproduce the behavior:

Clone repo https://github.com/ionictheme/ionic4-start-theme
Install deps and run on device
Click on android hardware back button
Nothing happens

Expected Behavior
Android hardware back button should return the previous page or exit app.

Are there any updates about this bug?

Will it be supported by default in Ionic 4 or are the developers supposed to use approach like this https://stackoverflow.com/a/51729093/2755616

@pvskasyrskiy @artemtool the workaround mentioned by @naveedahmed1

Will it be supported by default in Ionic 4 or are the developers supposed to use approach like this https://stackoverflow.com/a/51729093/2755616

Was just confirmed working

Works for me but just for pages outside of the tabs page? Inside tabs page I have 4 additional router-outlets how to handle this navigation stack?

There are other cases as well, such as Back button for modal does't work either.
Just keep this in mind that the solution on stackoverflow is a temporary fix. we should wait for permanent fix from ionic team, which I believe should cover all cases and should be available in next updates.

Just to bring it into notice, the fix should work for the nested router outlets as well. For example for the cases where app component has:
<ion-router-outlet main></ion-router-outlet>
and a component A is rendered in this outlet which also has a router outlet for child components
<ion-router-outlet main></ion-router-outlet>

@naveedahmed1
for pages inside the TabsPage it is not working for me. I added this workaround in the TabsPage instead:
``
export class TabsPage {
@ViewChildren(IonRouterOutlet) routerOutlets: QueryList<IonRouterOutlet>;

constructor(
    private platform: Platform,
    private router: Router
) {
    // subscription to native back button
    this.platform.backButton.subscribe(() => {
        this.routerOutlets.forEach((outlet: IonRouterOutlet) => {
            if (outlet && outlet.canGoBack()) {
                outlet.pop();
            }
        });
    });
}

}
``

also ionic 4 cant close app, how to solved it?

Back button should be the priority of the Ionic's dev team

@borgogelli see how many commits they did about that subject recently, it definitely looks like it's one of their priorities
@crebuh for me it worked with modals but not yet with navigation https://github.com/ionic-team/ionic/issues/15492

Try This:
// active hardware back button
    backButtonEvent() {

        // subscription to native back button
        this.platform.backButton.subscribe(async (data) => {
            alert(data);

            try {
                const element = await this.actionSheetCtrl.getTop();
                if (element) {
                    element.dismiss();
                    return;
                }
            } catch (error) {
            }

            try {
                const element = await this.popoverCtrl.getTop();
                if (element) {
                    element.dismiss();
                    return;
                }
            } catch (error) {
            }

            try {
                const element = await this.modalCtrl.getTop();
                if (element) {
                    element.dismiss();
                    return;
                }
            } catch (error) {
            }

            this.routerOutlets.forEach((outlet: IonRouterOutlet) => {
                if (outlet && outlet.canGoBack()) {
                    outlet.pop();
                } else if (this.router.url === '/home') {
                    // Double check to exit app
                    if (new Date().getTime() - this.lastTimeBackPress < this.timePeriodToExit) {
                        this.platform.pause.subscribe(() => alert('Pause Work')); // Exit from app

                    } else {
                        this.toast.show(
                            `Press back again to exit App.`,
                            '2000',
                            'center')
                            .subscribe(toast => {
                                // console.log(JSON.stringify(toast));
                            });
                        this.lastTimeBackPress = new Date().getTime();
                    }
                }
            });
        });
}

this.platform.exitApp(); doesn't work.

this.platform.exitApp(); doesn't work for ionic v4.1.2

navigator['app'].exitApp(); how yo setup it?

@manucorporat PR https://github.com/ionic-team/ionic/pull/15615 fixes the remaining problem with the handler registered by overlays.tsx.

The temporary fix from @Riyaz0001 worked for me. However when there are controls on the page I go back to via the physical back button, the first tap on a control does not work. For example a plain ion-button:
<ion-button (click)="doit()">Test</ion-button>
It does not happen, when I got back with the <ion-back-button>. Only with the physical back button and the backButtonEvent().


File is missing (from .ts_augmentations import apply_transformation)

Hi,
I hope you are doing well, I am implementing this project. while implementing the code by running the file dataloader.py there is a line (from .ts_augmentations import apply_transformation) in it. This line of code needs to find a directory. But there is no directory with this name. So, its humble request to you to clear this issue. Thank you very much

Thanks a lot for reporting. I cleared that line, as it was not necessary here.


Organizers/Team

Main organizers and contributors to be listed on the webpage similar to listing in BHS webpage if everyone is ok with?

I'll work on this one. But I'm not able to self-assign...

@sofievdbos Please do not forget to add the Donostia team as a sublisting was done in the OHBM Brainhack page https://ohbm.github.io/hackathon2020/team/ but the style will be as in BHS webpage.  I guess you might want to ask the Donostia team who should be listed there. Since some are not in the chat group still works in the background to help us.

@smoia could you give us a list of organizers/volunteers and their Github/twitter accounts and if they do not have a GitHub could you provide some images somewhere secure please so we can work on that?

@complexbrains do you mean people that helped in the BHG or the full organising team of the BH Donostia? In the latter case, this is the list: https://brainhack-donostia.github.io/about/

The core framework to add the Team is ready. For now, I created two categories: BHG 2020 Core Team & Volunteers (where I would list the Donostia team who've helped in the BHG 2020). Were those the subcategories you had in mind? In that way, others who might reach out to volunteer can also be listed under the Volunteers section. However, feel free to suggest other options/names/subcategories!! 😀 Happy to change however you would like @smoia @complexbrains!
What we list for now:

required: an avatar image (grabbed from GitHub account), name + surname
optional: social (email, Twitter, GitHub), when hovering over the image: position (e.g. PhD candidate at ...)

Thank you, Sofie.
I guess it would be better to keep the naming of the team the same as we used it in our emails, documents and in many other places which is "Brainhack Global Organizing Committee". I believe Volunteers should be fine.
We might also need a Code of Conduct team, if we can manage helpers, buddies but those are not certain yet, so let's leave it as those two now.

"Brainhack Global Organizing Committee"

Good point! Thanks! 🙏

Code of Conduct team

Creating a new subcategory is very easy! So whenever we want to add one, we can do so! 😀

Going back to the volunteers from BHD that did something specific (contributing with ideas/meetings/items/...) for BHG, if you don't want to list the whole BHD team, they are:

@afurgoni20 (Alberto Furgoni)
@danalclop (Daniel Alcalá)
@eurunuela (Eneko Uruñuela)
@smoia (Stefano Moia TW: @SteMoia)

@eurunuela @danalclop @afurgoni20 please write your twitter handle if you have one!

Thank you for this guys!
My Twitter handle is @eurunuela just like the GH one.

Created a PR #29 --> If you would like me to change or add something, let me know! 😀 --> e.g. if you would like another (non-GitHub) avatar image, if you would like to add your email address and/or Twitter handle!

PR #29 has been merged, so closing the issue.


Add ISSUE TEMPLATES

@krishdevdb Check it once!!

@krishdevdb Check it once!!
We have also gotta change repo owner in .allcontributorsrc to resetercss

I am thinking of abandoning all-contributors


ENH : Added group_delay to scipy.signal

Hi!
I used the description here to implement this function. I hope it could be helpful.

According to the test bot, this seems to not work for old numpy.
======================================================================
ERROR: test_fir (test_filter_design.TestGrpdelay)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/scipy/scipy/build/testenv/lib/python2.6/site-packages/scipy/signal/tests/test_filter_design.py", line 2107, in test_fir
    assert_array_almost_equal(gd, np.full(512, 0.5 * N), decimal=5)
AttributeError: 'module' object has no attribute 'full'
======================================================================
ERROR: test_singular (test_filter_design.TestGrpdelay)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/scipy/scipy/build/testenv/lib/python2.6/site-packages/scipy/signal/tests/test_filter_design.py", line 2135, in test_singular
    assert_warns(UserWarning, grpdelay, b, a, w=w)
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/numpy/testing/utils.py", line 1467, in assert_warns
    func(*args, **kw)
  File "/home/travis/build/scipy/scipy/build/testenv/lib/python2.6/site-packages/scipy/signal/filter_design.py", line 322, in grpdelay
    format(", ".join("{:.3f}".format(ws) for ws in w[singular]))
  File "/home/travis/build/scipy/scipy/build/testenv/lib/python2.6/site-packages/scipy/signal/filter_design.py", line 322, in <genexpr>
    format(", ".join("{:.3f}".format(ws) for ws in w[singular]))
ValueError: zero length field name in format

This looks good at first sight. I'm not a fan of the Matlab name though, would prefer group_delay. This naming issue has come up several times recently; I'll bring it up on the mailing list.

As devil's advocate, I like the cryptic Matlab names for scipy.signal functions.  They are named so cryptically that I immediately know that they are Matlab clones.  I don't even need to look at their docstrings, I can immediately go to Matlab's (or Octave's) online docs to see what they do.

Thanks for the feedback!
I wasn't sure how to name the function, but decided to go with @argriffing arguments.  And I read people complain that some functions in scipy named like in MATLAB, but don't work quite the same way.
About the algorithm: I'm not sure than handling of "singular" case is 100% correct / optimal. Any comments on that?

A comment from the peanut gallery: How about adding both? Name the function in a human-readable way, no_more_than_three_words_with_underscores, and immediately add a mtlbstl six-letter alias.
This way, you're forward compatible with the discussion following
http://mail.scipy.org/pipermail/scipy-dev/2015-February/020397.html

On the mailing list there are three +1's so far for more Pythonic names, and a useful suggestion to put the equivalent Matlab name in the docstring.
I'm -1 on now adding a new function under two different names.

On the mailing list there are three +1's so far for more Pythonic names, and a useful suggestion to put the equivalent Matlab name in the docstring.

If we are counting votes, mine would be +/-0.  My devil's advocate's position was only half-serious, and also I have not used scipy.signal much.

Verdict from the mailing list seems to be: group_delay, and mention that this corresponds to Matlab function grpdelay in Notes section of the docstring

@rgommers I took into account all your detailed remarks. Please take a look at the updated version.

@endolith @Eric89GXL is the API of this function okay from your perspective? The Matlab equivalent also takes SOS input, which cannot be added easily to this function anymore once a release is made.

Can it take a system as the first argument instead of b and a as separate arguments?  Instead of group_delay(b, a, w, whole), use group_delay((b, a), w, whole)?  Then it could also be group_delay(sos, w, whole), etc.

@rgommers it seems like we're going to have the same problem with this function that we have with many of the others that have b, a is inputs. So I'm not sure what a good solution is, since I don't think we've totally converged on the best solution there yet. :(

Thanks both. I realized this issue a bit late. I think @endolith's suggestion is good - for other b, a functions we can't do this without breaking backwards compat, but here it is easy to do.
First argument may be called system and could accept a tuple, an lti instance or sos.

and it could just accept system=(b, a) for now, and add the other types in other PRs to keep this one simple

I did all polishing work.
Question: how to point to a specific item from Reference within the docstring?
If it is settled I will update API to group_delay(system, ...) and keep it simple to system=(b, a) as @endolith suggested.

For a reference like .. [1] xxx you can refer to it with [1]_ and it'll turn into a cross-link.
(note that formatting of your reference would look a bit better if you align the second line with xxx)

@Eric89GXL just to make sure we're not missing something: do you think system=(b, a) is OK for this function?

Yeah, that's fine by me. It makes in inconsistent with the rest of the scipy API, but it is progress toward unification I suppose.

Thanks.
@nmayorov would be good to also add this to the 0.16.0 release notes (in doc/release/0.16.0-notes.rst), then we can merge this for that release after the API change.

Check the docstring for the new interface. Is it fine?

The changes look fine to me. I'll leave this open for a day or two and merge if no more comments.

I took a look and LGTM, +1 for merge

otherwise looks good to me

I'm done with final fixes.

Merged, thanks @nmayorov!

Rebasing and squashing is fine. If you have a few commits it's not necessary, once you get more than it's desirable, 10+ commits for a PR like this is around the point where I'd do some history rewriting myself before merging in case the contributor doesn't know how.


Can't start docker container with NCS

Plugin for Intel® Movidius™ Neural Compute Stick is distributed only in a binary form, so loading models on NCS is possible only with binary built docker image.

What does this mean?
So can't I use your built-in docker image?
I tried with ssd_inception_v2 model and got this error:

I was able to inference the model using this sample code directly:
https://gist.github.com/kodamap/9a9c971f43bffdc457a628c4313947fe
Any idea?

Hi, docker image available on docker hub does not support NCS.
To have docker image that supports NCS you need to build it yourself following steps in:
https://github.com/IntelAI/OpenVINO-model-server/blob/master/docs/docker_container.md#building-the-image.
First you need to get openvino toolkit for linux (tgz file). You can register and download it from
https://software.intel.com/en-us/openvino-toolkit/choose-download.
Then place downloaded tgz file under root of OpenVINO-model-server and use make to build OVMS docker image that supports NCS:
cp (download path)/l_openvino_toolkit_p_2019.x.xxx.tgz .
make docker_build_bin http_proxy=$http_proxy https_proxy=$https_proxy

@mzegla
Thanks for your quick reply!
Is there anything I can configure to enable NCS before building OVMS docker image?
Or, just follow your instruction above?

Hmm, after you build your image make sure your NCS is available on host machine.
https://github.com/IntelAI/OpenVINO-model-server/blob/master/docs/docker_container.md#starting-docker-container-with-ncs
Also check out Updating udev rules, you might need it.
Once you've done it and got docker image built you can try running it.
Let me know how did it work 😄

I will give it a try tomorrow morning early.
Thanks!


Support new Laravel 5.5 API Pagination JSON structure

refer to ratiw/vuetable-2-tutorial#67

Pagination works with the setting : pagination-path="meta"
Hope it helps

The links inside links path may not work if other pagination components that use VuetablePaginationMixin tries to use it.


V24.5

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ pre-commit run (or $ nox -s pre-commit) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[ ] All tests pass: $ python run-tests.py --all (or $ nox -s tests)
[ ] The documentation builds: $ python run-tests.py --doctest (or $ nox -s doctests)

You can run integration tests, unit tests, and doctests together at once, using $ python run-tests.py --quick (or $ nox -s quick).
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Triggered another deployment after the last commit: https://github.com/pybamm-team/PyBaMM/actions/runs/9668643828, cancelling the tests here for now

Oops, I updated the tag but I missed pushing the tag 😅 I don't think the workflow ran at all because it was stuck in the queue, I cancelled it just now. I think I will let you do it because I am about to log off

Oops, I updated the tag but I missed pushing the tag 😅 I don't think the workflow ran at all because it was stuck in the queue, I cancelled it just now. I think I will let you do it because I am about to log off

Yeah I already triggered the correct one. Hopefully you did not cancel mine

I think I didn't, it's still running: https://github.com/pybamm-team/PyBaMM/actions/runs/9668726233

Tests and deployment passed except for lychee

Merging this


Gesture Builder Database Imported COM Exception

Hi WHen I replace my own db(HandsOverHead.gbd) with HandsAboveHead.gbd in GestureDetector.cs
Lab 12: private readonly string gestureDatabase = @"Database\HandsAboveHead.gbd";
Replace : private readonly string gestureDatabase = @"Database\HandOverHead.gbd"; // Seated.gbd
It through's CoM Exception (System.Runtime>interopService.COMException)
Error : HRESULT E_FAIL has been rturned from a call component. Error code: -21476677259
Exception on code is:
using (VisualGestureBuilderDatabase database = new VisualGestureBuilderDatabase(this.gestureDatabase))

Check the the dv fe in your project is marked as conent and copy always is checked. When the project gets deployed as a package all files requied for the project must be self contained and can only be loaded from within that container.

Thanks
*Regards : *
Muhammad Afzal
Eng.: BS (IT)
Cell.#: +92-307_2007747
On Wed, May 6, 2015 at 7:50 PM, Carmine Sirignano<EMAIL_ADDRESS>wrote:

Check the the dv fe in your project is marked as conent and copy always is
checked. When the project gets deployed as a package all files requied for
the project must be self contained and can only be loaded from within that
container.
—
Reply to this email directly or view it on GitHub
https://github.com/Kinect/tutorial/issues/6#issuecomment-99498697.

Could you please elaborate more on what you mean with the term dv fe?

worked like a champ.  thanks...

Sorry, I still have the problem. I have add the HandsAboveHead.gbd problem into a Database folder from the complete app, but still not work and get that message.


Adding reporting, observability and gantt-charts

Signed-off-by: vamsi-amazon<EMAIL_ADDRESS>Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2435 (01ccf65) into main (9b229e8) will decrease coverage by 2.78%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #2435      +/-   ##
============================================
- Coverage     94.62%   91.84%   -2.79%
  Complexity       27       27
============================================
  Files           210       59     -151
  Lines          4317      282    -4035
  Branches         29       29
============================================
- Hits           4085      259    -3826
+ Misses          226       17     -209
  Partials          6        6

Impacted Files
Coverage Δ

.../bwc_test/bwc_test_runner_opensearch_dashboards.py

...orkflow/integ_test/integ_test_runner_opensearch.py

src/run_bwc_test.py

src/test_workflow/perf_test/perf_test_cluster.py

src/manifests_workflow/component_opensearch.py

src/sign_workflow/signer.py

src/ci_workflow/ci_target.py

..._workflow/input_manifests_opensearch_dashboards.py

src/system/config_file.py

src/manifests/distribution.py

... and 141 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


Fixing sender/receiver code base to compile with MSVC

flyby: using try_catch_exception_ptr in p443_executor, adding support for the with_annotation property

This fixes compilation of the sender/receiver tests on MSVC by adding a workaround to util::optional and by adding a (slightly) modified version of MPark's variant.

retest cscs

retest lsu

The CSCS Jenkins instance has hit the quota on its scratch filesystem... I'll try to get it resolved as soon as possible.

retest

retest lsu


Refactor Dockerfile post locale errors

Basing BigchainDB docker image from python:3.6 in Dockerfile and Dockerfile-dev.
Added ENV var for websocket interface
Optimization to Dockerfile-dev to save build time during dev/test.
Pre-installing the pynacl dependeny in separate command so that devs
do not need to wait while testing. The trade-off is that devs will need
to clear (rmi) the image to start from scratch.
Travis docker image build tests on every commit.
Ref for travis integration for docker build on every commit:
https://docs.travis-ci.com/user/docker/
https://github.com/travis-ci/travis-ci/issues/5448
https://docs.travis-ci.com/user/trusty-ci-environment#tl%3Bdr---Using-Trusty

Codecov Report

Merging #1409 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1409   +/-   ##
=======================================
  Coverage   98.37%   98.37%
=======================================
  Files          56       56
  Lines        2588     2588
=======================================
  Hits         2546     2546
  Misses         42       42


tensorflow build from source c++

System information

Windows Platform
TensorFlow installed from (source or binary): source
TensorFlow version:1.15
Python version:3.6
Installed using virtualenv? pip? conda?:conda
Bazel version (if compiling from source):2.0.0

Build Toco using bazel run at the end error show up, is this necessary, or it is ok to leave it be?
ERROR: C:/tensorflow/tensorflow/core/framework/BUILD:591:1: C++ compilation of rule '//tensorflow/core/framework:bfloat16'

@HardworkingPig
Please, provide the exact sequence of commands / steps that you executed before running into the problem.Thanks!

Please post full error message, not only the first line