Hosting the website on SNTC server

Currently the website's URL is kamandprompt.github.io, which is OK for the start. But I would like this to become as kamandprompt.iitmandi.ac.in OR pc.iitmandi.ac.in (whatever you guys like). In order to do that, we will need to host this website on our own server, and SNTC server is the best for it. @aashish-ak and Prabhakar know much about the hosting. I would like everyone to work on this starting now, since the site has taken a basic structure.

Thanks for bringing this up, we've already started working on this.

It will be also beneficial for you and the juniors that you also start involving them into this initiative.

@abhigyank I asked you this. Had any progress?

Hosted on pc.iitmandi.co.in .


Swap the write_modes in hudi "merge" strategy

recreating the PR from a fix branch - old PR ref - https://github.com/aws-samples/dbt-glue/pull/145
@armaseg pls take a look

@nandubatchu can you help me understand why would you use only "append" on Hudi since "Overwrite" is faster with bulk inserts for 1st time creating a table ? As such, I would go with what @jduggin-Parchment suggested

@AmineIzanami @jduggin-Parchment @menuetb - incorporated the suggestion!

Thanks, @nandubatchu, for the PR, but seems duplicate of #149, I'll close the PR for now.


Negative character class failing to find question mark

This code will not match the question mark character:
{ // in settings.json
  ...
  "findInCurrentFile": {
    "questionable": {
      "find": "[^\\n]",
      "replace: "=",
      "isRegex": true,
    },
  ...
}

Input:
A simple test. Is this right? I think, indeed I know, not!
I'd pay $1 (or more) to lurn [sic] why. |\@#%^&*{}<>/~_-+=

Output:
============================?=============================
==========================================================

But it works find if placed in runInSearchPanel.

I'm afraid the explanation is complex and has to do with escaping characters in settings - which is a pain.  This works (in both the Find and Search):
      "title": "myTitle Find",
      "find": "[^\n]",            //  note that there is no need for double-escaping here
      "replace": "=",
      "isRegex": true,
    }

In general, you do not have to double-escape the \n in a find value.  You will notice that you do NOT get any error squigglies if you do not double-escape so that usually indicates you are good without double-escaping.
Note that in the Find widget, [^\\n] would not do what you want at all.
But there was a larger problem revealed by your issue which I believe I have fixed.
In order to be platform dependent, vscode replaces \n with \r?\n.  So this extension must do the same.  However, it shouldn't be done in a regex with [\n in here somewhere], like your [^\\n], as [^\r?\n] would be read by a regex engine to exclude the ? character not as an optional indicator - just the way character classes like [] work.
SO I had to come up with a fancy regex which does replace \n with \r?\n everywhere but not in [].  And \\n should work in a character class just like \n.
I uploaded a new version v5.2.0. Please update to that and let me know if the problem persists.
Thank you.

Huh. I was repeated writing patterns without double-escaping, and then needing to double-escape to make them work. I'll reread your notes about double-escaping and review where I thought I needed the double-escapes.
I'll download the updated extension tomorrow and check. Thanks for the quick update.
In the meanwhile, I had found a workaround using .*\n rather than [^\n]*\n that worked in my circumstance. I was glad not to be forced into ([\n]|\?).

I wonder if there is a "right" answers to making [\n] and [^\n] platform independent?

Maybe that's okay. Didn't it used to be the case that \n was Un*x, \r\n was DOS, and \r was Mac?

Okay. Here's a question that I expect to be asking tomorrow. If I don't need to double-escape [\n] then surely I can't double-escape it, as [\\n] would then be interpreted as the character class consisting of \ and \n. And if it isn't, why not? And how would I express the character class equivalent to \\|n?

I removed transforming \n to \r?\n altogether inside those character classes [....].  It isn't necessary there on any OS but is elsewhere.  So the extension will not create [\r\n] or [^\r\n] of its accord.
And I added a check for the EOL so it only makes the transform on CRLF systems (Windows) - when not in a character class.

And how would I express the character class equivalent to \|n?

Try [n\\] (I haven't tried it though.)

And \n should work in a character class just like \n.

That I can't fully explain.  There are some complications when escaping things in settings and keybindings as one escape is typically removed simply on reading that setting, etc. before actually using it in some string which if read by another function may need to be escaped again.  But \n apparently gets special treatment by vscode.  For example, if you write \n\s in this extension's keybindings or settings you will get error squigglies for the \s only so you need to use \n\\s.  You DON'T need to use \\n\\s.
I really don't know why that is other than to guess that there is specific code that handles any settings or keybindings looking for \n or \\n.  Like this code from src/vs/editor/common/core/textChange.ts
function escapeNewLine(str: string): string {
  return (
    str
     .replace(/\n/g, '\\n')
     .replace(/\r/g, '\\r')
  );
}

Vscode is changing \n to \\n there.  And see https://github.com/microsoft/vscode/blob/85e9b0786cc505d85f326991227de364a5bd7293/src/vs/workbench/services/search/node/ripgrepTextSearchEngine.ts#L577 and lots more.

Thanks! I went to reproduce the problem this morning before updating the extension, but the extension had already auto-updated and the problem was gone.
I also noted that [^\n] was matching \r.
I speculate that '\n, \r', and \t don't need double-escaping because they resolve into characters that are allowed to be where the codes were placed and don't resolve further.

Adding here as it appears this might be a knock-on effect of the bug fix.
It looks like each highlighted replacement is one character earlier than it should be, cumulatively.
First highlight is one character too early; second highlight is two characters too early; etc.

I made some more updates, closing as the original issue is resolved I believe.  Latest version is 5.2.3.

Yes. Original issue is resolved. I'll post again if there are any other
issues. Thanks!
On Thu, Jun 20, 2024 at 11:44 PM ArturoDent @.***>
wrote:

Closed #30 https://github.com/ArturoDent/find-and-transform/issues/30
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/ArturoDent/find-and-transform/issues/30#event-13237066131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADI7GKWZLA7JCXTUWTA5SWDZIOOTFAVCNFSM6AAAAABJK7ZPIOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGIZTOMBWGYYTGMI
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


slapadd: could not parse entry in config-core.ldif

Hi,
Thanks a lot for making this. 👍🏾
I'm quite inexperienced with LDAP and I hope you can help me here.
I know that your example ldap server is specifically for Debian, and I'm trying to setup this on a Mac with OpenLDAP installed. I do have slapadd in my $PATH. How do I deal with this error:
make clean && make db
rm -rf db config
mkdir db db/main db/run config
slapadd -n0 -q -F config -l config-core.ldif
60f09fa5 str2entry: invalid value for attributeType objectClass #0 (syntax <IP_ADDRESS>.4.1.14<IP_ADDRESS>.38)
slapadd: could not parse entry (line=19)
_###############       75.00% eta   none elapsed            none spd 203.1 k/s
Closing DB...
make: *** [do-db] Error 1

Hi, sorry for the delay, my access to GH is intermittent. I suppose that OpenLDAP is installed from Homebrew or MacPorts; can you post the output of slapd -VVV?

That is true. I've installed OpenLDAP from Homebrew. I don't have slapd in $PATH and instead have it at /usr/libexec/slapd.
/usr/libexec/slapd -VVV
@(#) $OpenLDAP: slapd 2.4.28 (Apr 12 2021 02:54:21) $
        root@osx307.apple.com:/AppleInternal/BuildRoot/Library/Caches/com.apple.xbs/Binaries/OpenLDAP/install/TempContent/Objects/servers/slapd

Included static overlays:
    accesslog
    dynid
    odlocales
    odusers
    auditlog
    constraint
    dds
    deref
    dyngroup
    nestedgroup
    dynlist
    memberof
    ppolicy
    pcache
    refint
    retcode
    seqmod
    syncprov
    translucent
    unique
    valsort
Included static backends:
    config
    ldif
    monitor
    bdb
    hdb
    relay

That looks like the system-supplied slapd: it's a very old version (2.4.28 was released in 2011) and has an apple.com address in the version string. I'd expect that Homebrew keeps its slapd somewhere below /usr/local. I need the -VVV output from that one.

Gotcha. Here you go:
@(#) $OpenLDAP: slapd 2.5.5 (Jun  4 2021 01:28:36) $
        openldap

Included static overlays:
    accesslog
    auditlog
    constraint
    dds
    deref
    dyngroup
    dynlist
    memberof
    ppolicy
    pcache
    refint
    retcode
    seqmod
    syncprov
    translucent
    unique
    valsort
Included static backends:
    config
    ldif
    monitor
    mdb
    relay

Homebrew's slapd is built without module support and chokes on the attempt to initialize the module list. You could try to remove the cn=modules, cn=config entry from config-core.ldif (from dn: cn=module,... to the next blank line) and run make clean && make db. Make sure that slapd is in your $PATH before starting the server.

Seems like that worked.
rm -rf db config
mkdir db db/main db/run config
"/usr/local/Cellar/openldap/2.5.5/sbin/slapadd" -n0 -q -F config -l config-core.ldif
Closing DB...
"/usr/local/Cellar/openldap/2.5.5/sbin/slapadd" -n0 -q -F config -l core.ldif
Closing DB...
"/usr/local/Cellar/openldap/2.5.5/sbin/slapadd" -n0 -q -F config -l cosine.ldif
Closing DB...
"/usr/local/Cellar/openldap/2.5.5/sbin/slapadd" -n0 -q -F config -l inetorgperson.ldif
Closing DB...
"/usr/local/Cellar/openldap/2.5.5/sbin/slapadd" -n0 -q -F config -l nis.ldif
Closing DB...
"/usr/local/Cellar/openldap/2.5.5/libexec/slapd" -h ldapi://ldapi -F config
sleep 1
"/usr/local/Cellar/openldap/2.5.5/bin/ldapadd" -x -D "cn=config" -w secret -H ldapi://ldapi -f add-db.ldif
adding new entry "olcDatabase={1}mdb,cn=config"

adding new entry "olcDatabase={2}monitor,cn=config"

kill `cat db/run/db.pid`
"/usr/local/Cellar/openldap/2.5.5/sbin/slapadd" -n1 -qw -F config -l example.ldif
Closing DB...

Thank you!


ACI Liveness Probe Page: Fix typo in 'Next steps'

@derekbekoe : Thanks for your contribution! The author, @jluk, has been notified to review your proposed change.

#sign-off thanks @derekbekoe


DrawIcon is not available in blocks mode

Start up makecode and flip into JavaScript - add this code

The block definitions are not set up correctly in the extension

This looks like it's because you have not started your block definition comments with a
// %

for a couple of the apis - draw arrow is the same problem
https://github.com/pimoroni/pxt-inkybit/blob/master/inkybit.ts


Not migrated to AndroidX

This plugin is still using the older android support dependencies and not migrated to AndroidX. Most flutter plugins are updating which now causes an issue where this plugin is used since all other packages have to be pinned to their last pre-androidx version

Hey @lubritto any chance you can merge the pull request relating to this?


Let for-loop iterate Dict with an iterator

...without creating a temporary array of all key-value pairs.

Great, thanks!


allow reading the pull secret from the podTemplate

Changes
This change is to allow chains to get the imgPullSecret from the pod template. This will allow users to configure repo auth on a per pipelineRun/taskRun basis and not modify the service account.

/ok-to-test

@joejstuart, seems like a good enhancement. Would you be willing to add unit tests?

@joejstuart, seems like a good enhancement. Would you be willing to add unit tests?

absolutely.

/lgtm

LGTM!
Main nit is we should document this here: https://github.com/tektoncd/chains/blob/main/docs/authentication.md#authenticating-to-an-oci-registry

Docs added. Thanks!


Delete a datasource definition in an EAP 7 standalone server

Adds Remove operation to Datasources.
@miq-bot add_label providers/hawkular
@miq-bot add_label wip

@miq-bot add_label enhancement
@rubenvp8510 / @pilhuhn / @ammendonca is the more appropriate term remove or delete? (considering previous hawkular UI and the deployments UI in miq)

I think that this file needs an update too (although you can see the feature in ui even without updating it which I remember wasn't happening in the past)
And there's also a relevant spec file (iirc with a similar name to the above yml file) where the features count needs to be updated since you added a new feature (so +1)

just checking if this comment was seen? https://github.com/ManageIQ/manageiq/pull/9994#issuecomment-234529208

I would prefer 'remove' - as this is more on par with what the dmr-api of WF offers, so users are more used to it/more easily find a related documentation.

I think there is no need to update the miq_product_features_spec.rb  file (with +1)  because of this PR https://github.com/ManageIQ/manageiq/pull/9924. I also saw there is a feature named middleware_datasource_delete in this file, I only renamed it to middleware_datasource_remove in order to maintain consistency in names.

@miq-bot rm_label wip

@rubenvp8510 The title of this PR is prefixed with OPR-036a.  Please remove this cryptic string from the title.  If it means something specific, please add it to the description with a link and/or a phrase/sentence summarizing what this means.

@rubenvp8510 I left a few comments inside, except for them, this is looking good,

I guess we can leave it as is for now. And when MiQ has the notification center revisit to make the message more clear.
Also I wonder if we should feed that info success/error into the the MiQ event system to show up in the timeline view. This is a bit of a grey area, as for many operations the immediate reponse is something like
:data=>{"message"=>"The request has been forwarded to feed [829c6e5f-402f-4c34-b202-676631735c12] (MessageId=ID:65a957dc-54d6-11e6-8599-21d16e823d31)"}}
which does not make too much sense to the user.
Add a line note

we could certainly feed the event, nice idea. doesn't have to be part of this PR. the message as you list below is definitely cryptic.

One smallish comment, but otherwise LGTM.

@miq-bot assign @chessbyte

I renamed the drm_operation to operation_name again, I put some comments  to try to avoid confusion on this.

@tumido this was implemented here, fyi


Feature/agent data

capture agent details to ddb, and sse stream via express

:tada: This PR is included in version 4.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error of withRouter

Bug report
Describe the bug
Error when using _app.js with the using-with-router example under ie9 (router is undefined), but ie11 is ok
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:
https://github.com/xyyjk/next.js/tree/canary/examples/using-with-router

npm run build

npm run start

open http://localhost:3000

Expected behavior
ie9 without error
Screenshots
ie9&ie10

ie11

System information

OS: Windows
Browser IE9、IE11
Version of Next.js: latest

Additional context
If you delete the _app.js file then it is OK

@xyyjk
I don't think Next.js supports <IE11 - see https://github.com/zeit/next.js#browser-support.
Maybe it'll work in IE9 with some extra polyfills, but I'm not sure about that.

@cherniavskii
I have added polyfill to index.js.
I am currently using a temporary solution, Add the following code to _app.js to work properly in IE9:
static childContextTypes = {
  router: PropTypes.object,
};

getChildContext() {
  const { router } = this.props;
  return { router };
}

@cherniavskii
We indeed use withRouter, see
https://github.com/xyyjk/next.js/blob/474e451d0440d994ef3b0cc7079a55d0abd32165/examples/using-with-router/components/ActiveLink.js#L25
But, there is router is undefined error blew this line, in IE9
https://github.com/xyyjk/next.js/blob/474e451d0440d994ef3b0cc7079a55d0abd32165/examples/using-with-router/components/ActiveLink.js#L10
can we fix this like above reply of xyyjk? and Is there any potential problems with this solution in your experience.
thanks

@xyyjk you shouldn’t override the default contextTypes, it’s guaranteed to break

@cherniavskii  Yes, the Map and Set are already included in the babel-polyfill

@timneutkens We're seeing this issue on IE11 with IE10 compatibility mode activated, I've included all the polyfills required, but now I just keep getting the router passed as undefined using withRouter from next/router. This seems like an upstream issue that can be fixed? Adding the fix from @hkongm worked but I don't really understand the fix itself which is risky.
Can we get some closure on the recommended approach for this?

Next.js 7
  static childContextTypes = {
    headManager: PropTypes.object,
    router: PropTypes.object
  }


Simplified Approach

Hi Almaz Ibragimov 👋🏻
After a quick look at your project I couldn't help myself but ask: Can't this be achieved with a lot less complexity and effort? 🤔
It's a quick rather than a thorough thought. I'm sharing this in case it might be helpful.
What about just doing this?
protocol AutoChangeable {}

extension AutoChangeable {
   func changing(_ changes: (inout Self) -> Void) -> Self {
      var copy = self
      changes(&copy)
      return copy
   }
}

// Usage
struct User: AutoChangeable {
    let id: Int
    let name: String
    let age: Int
}

// Copy the instance by changing the `name` and `age` properties
let steveJobs = steve.changing { newUser in
    newUser.name = "Steve Jobs"
    newUser.age = 41
}

Would that be sufficient or am I missing important functionality that your solution offers?
Have a great weekend 😊

Never mind, brain fart!
Fields of the struct are still let's and therefore can't be changed lol


show gene expression in a subset of cells

Dear,
how to show gene expression levels in only a subset of cells (eg. cluster 1,2 while other clusters are hidden) ?

You could subset the anndata object via:
adata_sub = adata[adata.obs.cluster.isin(['1','2']),:]
Then you can just plot the expression values of adata_sub.

It'd be great if we could use https://scanpy.discourse.group/ for this kind of questions and answers.

yes

HI @LuckyMD, quick question. After subsetting with the command: adata_sub = adata[adata.obs.cluster.isin(['1','2']),:]
I get a dataset with the subset of genes identify as variable in the 'adata', so this are not necessary the genes I would like to focus for this particular subset. How could I create a subset with all the genes prior to the filtering step?

So the step adata[adata.obs.cluster.isin(['1','2']),:] does not filter out any genes. You would have to use the adata object from before your filtering step and transfer the louvain cluster id's via something like: adata_unfiltered.obs['cluster'] = adata.obs['cluster']. And then do the above on adata_unfiltered. Note that this only works if both objects have the same number of cells and the same adata.obs_names.


Patches

Before merging in patches, we merged move_to_mth5 into patches.  This was slightly complicated due to ipynbs changing.  A git merge got a little messy, so we reverted to the last good commits of patches and move_to_mth5 and did this:

Patches well behaved pre-merge is: 8988ff5c24fa005d4b5a4533f3cdccee37c61cc0\

move_to_mth5 well beahved before merge d94bc18

forked d94bc18 to a branch named merging
on merging, checked out the 4 conflict ipynbs from patches, and committed.
merged merging into patches.

Then manually fixed notebooks and committed on patches.
When the smoke cleared, there were a few files that didn't seem to have merged as desired -- it is not clear why, at least a few of these were worked on quite recently.  The files were:
test_process_parkfield_run.py
test_process_parkfield_run_rr.py
test_fourier_coefficients.py
test_multi_run.py
test_processing.py
Minor changes were applied to these, taking the latest, correct code from both branches.


Update hdinsight.md

Docs Build status updates of commit 44e82c3:
:clock10: Full build: the average full build time is 69 min(s) 54 sec(s), this is based on the last 199 build(s) for this repository.
:warning: This is a FULL build due to the incremental build being disabled in configuration file (.openpublishing.publish.config.json in 'live' branch). For more detailed info, please click here.

Docs Build status updates of commit 44e82c3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/hdinsight.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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

Docs Build status updates of commit 1be6760:
:clock10: Full build: the average full build time is 70 min(s) 1 sec(s), this is based on the last 203 build(s) for this repository.
:warning: This is a FULL build due to the incremental build being disabled in configuration file (.openpublishing.publish.config.json in 'live' branch). For more detailed info, please click here.

Docs Build status updates of commit 1be6760:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/hdinsight.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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


MANAGEMENT_TUTORIAL_TEXT undefined

MANAGEMENT_TUTORIAL_TEXT constant (line 69 website_code/php/management/site.php) is not defined.
This constant should probably be defined in:
languages/en-GB/website_code/php/management/site.inc e.g
define("MANAGEMENT_TUTORIAL_TEXT","The tutorial text is (Add information and links to tutorials here.)");

Fixed


Application has been already initialized issue

I am using rails 5 and ruby 2.4, when I try to run rspec for my gem works locally, I am getting runtime error in jenkins as follows below I have mentioned error and rails_helper code.
LoadError:
  cannot load such file -- rspec/rails
# /usr/local/bundle/gems/activesupport-5.1.5/lib/active_support/demo.rb:292:in `require'
# /usr/local/bundle/gems/activesupport-5.1.5/lib/active_support/demo.rb:292:in `block in require'
# /usr/local/bundle/gems/activesupport-5.1.5/lib/active_support/demo.rb:258:in `load_dependency'
# /usr/local/bundle/gems/activesupport-5.1.5/lib/active_support/demo.rb:292:in `require'
# ./spec/rails_helper.rb:14:in `<top (required)>'
# ./spec/controllers/sample/demo_controller_spec.rb:4:in `require'
# ./spec/controllers/sample/demo_controller_spec.rb:4:in `<top (required)>'

An error occurred while loading ./spec/models/demo_spec.rb.
Failure/Error: Combustion.initialize! :action_controller

rails_helper.rb file:

require 'spec_helper'

ENV['RAILS_ENV'] ||= 'test'
Dir[File.dirname(__FILE__) + '../internal/config/environment/*.rb'].each {|file| require file }

require 'combustion'
require 'support/shared_examples'

Combustion.initialize! :action_controller
Object.send(:remove_const, :ActiveRecord)

require 'rspec/rails'

RSpec.configure do |config|
  config.infer_base_class_for_anonymous_controllers = false
  config.include Rails.application.routes.url_helpers
  config.infer_spec_type_from_file_location!
end

Can anyone please help me to solve this issue.

Can you share what's in your spec/rails_helper.rb and spec/spec_helper.rb? I think they might provide some helpful clues.

rails_helper.rb
require 'spec_helper'
ENV['RAILS_ENV'] ||= 'test'
require File.expand_path("../../config/environment", FILE)
Dir[File.dirname(FILE) + '../internal/config/environment/*.rb'].each {|file| require file }
require 'combustion'
require 'support/shared_examples'
Combustion.initialize! :action_controller
Object.send(:remove_const, :ActiveRecord)
require 'rspec/rails'
RSpec.configure do |config|
config.infer_base_class_for_anonymous_controllers = false
config.include Rails.application.routes.url_helpers
config.infer_spec_type_from_file_location!
end
spec_herlper.rb
require 'simplecov'
SimpleCov.start 'rails' do
Added this filter to exclude thrift files while calculating coverage report
add_filter '/lib/'
end
RSpec.configure do |config|
config.expect_with :rspec do |c|
# disable the should syntax...
c.syntax = :expect
end
end

Oh, I missed the fact that this only happens in Jenkins… that makes it more odd!
Has it ever worked for you in Jenkins? Are there any differences in how your tests are invoked on Jenkins compared to your own machine?

It has never build on jenkins. The order of execution in jenkins is first bundle will happen then specs will be executed.

I don't suppose the Jenkins server and/or your gem code are publicly online anywhere? I'm really not sure what could be causing this problem.

Thanks for the help, it got worked the issue was add_development_dependency does not work with test environment, so added all dependency gems to group development and test, it fixed the issue.

Great to know you found the solution :)

I ran into this odd error too. It has happened to me in a really odd way.
I have some tests that pass fine. My project uses both combustion and appraisal.
In my app, I have gem "chromedriver-helper" in my Gemfile.  I saw that chromedriver-helper was EOL. Experimentally, I wanted to see what would happen if I simply removed it from my Gemfile.
Removing it from my gemfile, run bundle install just to be safe, and then trying to run bundle exec rspec -- I get this weird error.
I can't figure out how it would be related, or what to do about it. But have confirmed that if I add it back into Gemfile, everything is good; if I remove it, get this error. No other changes.
RuntimeError:
  Application has been already initialized.
# /Users/jrochkind/.gem/ruby/2.5.3/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:360:in `initialize!'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:190:in `public_send'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:190:in `method_missing'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/combustion-0.9.1/lib/combustion.rb:32:in `initialize!'
# ./spec/spec_helper.rb:24:in `<top (required)>'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:291:in `require'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:291:in `block in require'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:257:in `load_dependency'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:291:in `require'
# ./spec/type/polymorphic_type_spec.rb:1:in `<top (required)>'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:285:in `load'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:285:in `block in load'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:257:in `load_dependency'
# /Users/jrochkind/.gem/ruby/2.5.3/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:285:in `load'

My spec_helper.rb:24 is just: Combustion.initialize! :all
What? I'm at a loss here.

Hrm. is it possible to alter your local copy of combustion to check whether Combustion.initialize! is being called just the once? Also, is it possible to share the full spec_helper.rb?
And of course, if the source code for this is available publicly, I'm happy to debug directly too :)

It is open source! https://github.com/jrochkind/attr_json/blob/master/spec/spec_helper.rb
While you can see that the Combustion.initialize has a block there, I removed the block, it didn't change the reproduction.
I don't expect or deserve it, but if you wanted to take a look at my source directly that would be very kind of you!
If you checkout master, and run ./bin/rspec, it should pass. If you remove the line gem "chromedriver-helper"--it fails with this error.
While appraisal is being used here -- if you just run ./bin/rspec I don't think appraisal is actually involved at all in that run, so I don't think appraisal is involved in the error.
My best guess is that removing chromedriver-helper from the gemfile somehow would be causing a problem even if combustion wasn't involved at all, but for some reason combustion being involved makes the problem exhibit as the weird "application has already been initialized" problem. Which doesn't make a lot of sense, but it's the only hypothesis I got! Especially with this already filed issue -- perhaps somehow combustion can hide a true Rails boot error and turn it into this error instead.
It's also possible the problem won't reproduce for you at all -- it actually passed on travis even when I removed the line from the Gemfile. (Apparently it didn't need chromedriver-helper for anything, at least on travis!).  So it may be somehow related to my local... state... somehow. I am not sure.
For me, if I or you don't figure it out relatively easily, I'll probably just not remove chromedriver-helper from my Gemfile for now, since I don't really need to.

Writing this out made me think, gee, maybe try deleting spec/internal/tmp/cache -- but that didn't seem to help.
Then I noticed, aha, I have a require 'chromedriver/helper' in my spec_helper.rb, that probably shouldn't be there if I don't have chromedriver-helper dependency anymore. But removing that... also didn't resolve issue, still reproduces.
This is all very odd indeed.
OK, I did put a debugger on Combustion.initialize! to see if it was being called  twice -- indeed it was being called twice!
So okay, I'm thinking there is some problem in my app that prevents Rails boot... and that somehow leads to the spec_helper being executed twice?  I can't quite figure out what would make that happen either.
Aha, and now paying more attention to my console output, I do see the original error!   cannot load such file -- selenium/webdriver. OK, so indeed my testing setup is not ok, in the absence of the chrome-webdriver gem. The only odd thing is that it results in the double-load of spec_helper and the "Application has been already initialized" error. I'm thinking in the presence of combustion, you end up seeing this error when really something else is raising during spec_helper.rb/rails_helper.rb load time, or rails app boot in general.

OK, and that's because I had accidentally removed gem "selenium-webdriver" from my Gemfile too.
When I put that back in (thanks git for making sure I can always see exactly what I've changed), I no longer get "application has already been initialized". I get failures, but they are not combustion related, they are clearly related to my removal of chromedriver-helper -- "Selenium::WebDriver::Error::WebDriverError: Unable to find chromedriver.". Great, that's what I actually expected!
So the issue may actually be -- if you have a "require" in your spec_helper.rb that fails... somehow this results in the spec_helper.rb being called multiple times (?), and a "Application has been already initialized" error from combustion. The original error is probably still there in your console, but you may miss it cause your console is full of multiple subsequent "Application has been already initialized issue".  Weird.

Glad you've found a way through this!
I wonder, do you have --require spec_helper in your .rspec files (either at a project or user level)? I'm wondering if the first require is from that, but it fails non-obviously, and then the spec file itself loads the spec_helper too, and you get the second duplicate app initialisation? 🤔


In comparison to using Realm. What are some tradeoffs and benefits of using this paging system?

In comparison to using Realm. What are some tradeoffs and benefits of using this paging system?
https://realm.io/

Realm is a database for persisting data whilst PagedArray is only for in memory allocation of data, built purposefully for table views and collection views where you want a smooth scrolling experience even though your backend API is paging.


Update faq.md

change the heading 2 to sentence capitalization.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Fair enough, thanks @scjane :)


rename ann4s to spark-annoy

spark-annoy is more suitable for the purpose of this repo.

[ ] rename repo to mskimm/sparkannoy
[ ] rename packages / classes

added hyphen (sparkannoy -> spark-annoy)


[ja] add docs for owners-dependents

Add desrtiption of owners-dependents.
https://kubernetes.io/docs/concepts/overview/working-with-objects/owners-dependents/
close #38791

Japanese finalizer page will be available after merging PR https://github.com/kubernetes/website/pull/38768.
So, how about wait for the merge, and then reflect my suggestions?

I reflected your suggestions, so I wait for it.

/hold
This PR contains a link to /ja/docs/concepts/overview/working-with-objects/finalizers/.
That is a new page and reviewing at https://github.com/kubernetes/website/pull/38768.
Feel free to unhold anyone when that PR is merged.

PR https://github.com/kubernetes/website/pull/38768 was merged.
/unhold


Document node to node messages and messaging patterns

Document every message nodes send to each other, under what circumstances, and the expected responses (where relevant)

Should be completed before the end of next sprint (26th July) earlier the better.

Initial version is here.


fix: send origin in query tagging

Pull request template

When you click on a brand recommendation, the origin param is not included in the query tagging. That's because the BaseEventButton used on the QueryPreviewButton component emits UserAcceptedAQueryPreview event without any feature param in the metadata so that the origin couldn't be created.
To fix it, we are going to pass the origin through a prop and it will be added to the event metadata.
Motivation and context

[ ] Dependencies. If any, specify:
[X] Open issue. If applicable, link: RST-1853

Type of change

[X] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that causes existing functionality to not work as expected)
[ ] Change requires a documentation update

What is the destination branch of this PR?

[X] Main
[ ] Other. Specify:

How has this been tested?

Click on a brand recommendation and check if the origin customer:no_query is sent in the query tagging.
Checklist:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review on my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.

Germán 👀


Hifiasm_error correction_question

Hi,author:
I would like to ask you how the reads after Hifiasm error correction are obtained from the bin file?
What software is used?
The original sentence in the article:
We first error-correct reads with hifiasm [16], thus reducing amount of mismatches, insertions, and
deletions in the reads.
I am looking forward to your reply. Thanks.

Hi yao-chenxin,
We extract the error-corrected sequences from the FASTA file. Apart from the bin file, you should also see a file whose name is ending in .ec.fa. This is the file where the error-corrected sequences are stored.
For example, if you run Hifiasm with the following command:
./hifiasm -o output -t 32 --write-paf --write-ec  reads.fastq

You should see a file called output.ec.fa inside your working directory.
Let me know if you have any firther questions!

Hi,author:
Thank you very much for your reply.
With best wishes!
At 2022-08-03 11:32:36, "Lovro Vrček" @.***> wrote:
Hi yao-chenxin,
We extract the error-corrected sequences from the FASTA file. Apart from the bin file, you should also see a file whose name is ending in .ec.fa. This is the file where the error-corrected sequences are stored.
For example, if you run Hifiasm with the following command:
./hifiasm -o output -t 32 --write-paf --write-ec  reads.fastq
You should see a file called output.ec.fa inside your working directory.
Let me know if you have any firther questions!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Dev

This change is

Current coverage is 88.23%

Branch #14 has no coverage reports uploaded yet.

No diff could be generated. No reports for greenkeeper-smokesignals-1.0.3 found.

Powered by Codecov. Updated on successful CI builds.


webkitgtk: 2.12.4 -> 2.12.5

Motivation for this change
https://lwn.net/Vulnerabilities/700654/
https://github.com/NixOS/nixpkgs/issues/18856
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Is the change of the hash in 2.4.nix intentionally? Looks like an accident :)

You bet it was an accident! Thanks for the :eyes:! Fixed.


[Feature] Simple progress lines

What problem does this feature solve?
I would like to create a single progress line using eCharts.
Here is in an image example: https://imgur.com/a/PhTQKxL
The colored line should change depending on it's value, for example:

When the value is 10 there should only be a small colored line.
When the value is 50 the colored should fill half the line, and grey the other half.
When the value is 100 it should be all colored.

The colored line should also animate on load.
And it should also have rounded end caps.
Is this simple type of progress line possible in eCharts? How would it be done?
If not I please suggest it is added as a feature.
What does the proposed API look like?
No current ideas for the API.

I think such a progress bar definitely can be done via HTML + CSS and it should be easy. You don't have to use ECharts to do this, though ECharts can make it.


Consider logging a message when full CLR is loaded in in-process mode app pool

One unfortunate side effect of having 2 CLR in one process is some debugging features (like VS memory snapshot feature) stop working correctly. I think sos extension in windbg also has problems because of that.


Test Issues - Does the new GtiHub Org secret work...

User Story
As a…
I need/want/expect to…
So that…
Value / Purpose
No response
Useful Contacts
No response
Additional Information
No response
Definition of Done
Example

[ ] Documentation has been written / updated
[ ] README has been updated
[ ] User docs have been updated
[ ] Another team member has reviewed
[ ] Tests are green

issues successfully added to the CloupOps project board.


Using Dataset

Hi, I'm trying to find a source for the Sort-of-CLEVR Dataset. The provided code in this repository seems to be what I'm looking for, but I need help understanding how to set it up for training, validation, and testing. Could you provide a brief example of how the included code could be used to generate a training, validation, and testing set, and then from this, iterate through these datasets in batches of chosen size?

For example, I tried rel_train, rel_test, norel_train, norel_test = load_data(), but I'm confused as to what these datasets contain. Each item seems to constitute a 3 x 3 x 75 x 75 dimension array, or I could be wrong. Can you clarify how this data maps back to the images, questions, and answers described in the dataset overview? Thank you for any help.

I think you can check it in sort_of_clevr_generator.
The idea is very simple inside. It just generates questions in one hot vector and pair the questions and answer to the object

Could you possibly show me a code example for how to retrieve the image, question, and answer (or a batch of them)?
What I got so far is: rel_train, rel_test, norel_train, norel_test = load_data().


Allow lang server restart

A bunch of refactoring as well as adding commands for swift clean, swift run, and restarting the language server.

Great changes so far!

@vknabel A lot of progress here.

The language server can be restarted (I've only confirmed sourcekitd because that's what I use)
Added commands to run, stop, clean
Cleaned up (IMO) the code for better understanding and maintenance

Could be merged as-is, but I think the configuration still needs to be cleaned up a bit. (It should all use the same prefix sde.SECTION.setting and I'd like to export an object from the configuration helper that mirrors the options–but allows us to hide the specific configuration names so we can migrate from what is to the new strings.)

Actually @vknabel I think it would make sense to merge this without the settings cleanup so we can better pinpoint where any bugs come from.
This is already a pretty major rewrite and changing how we fetch config any more seems like asking for more headaches

Hey @clayreimann. Sorry for the late response and a huge thank you for all your amazing work!
I have already started reviewing this PR. I will try to test everything as good as possible (on macOS and Linux), which is quite time consuming, but it will be worth it.
Once I find bugs, I will probably fix them directly, if that's okay for you.
Currently I am not sure how we handle the release and renamed configs or commands. Technically it would be a breaking change and would require a major version bump.
But this should be fine.
Might be a great opportunity to delete unused features.

Hey @clayreimann. Sorry for the late response and a huge thank you for all your amazing work!
I have already started reviewing this PR. I will try to test everything as good as possible (on macOS and Linux), which is quite time consuming, but it will be worth it.
Once I find bugs, I will probably fix them directly, if that's okay for you.
Currently I am not sure how we handle the release and renamed configs or commands. Technically it would be a breaking change and would require a major version bump.
But this should be fine.
Might be a great opportunity to delete unused features.

Thanks a lot @clayreimann!
I know it took quite a while. This PR really helps in the long term!
In https://github.com/vknabel/vscode-swift-development-environment/commit/b247437aece010a44341b91d51fa933c93d044ed I made a few fixes, added some changelogs and mentioned you in the README.

I created a beta release tag: 2.11.0-beta.0.
I try to test it bit, but I believe it should work fine. Will release it soon.


Support read-compacted on Reader

Motivation
Support reading through a compacted topic with the "Reader" API
Fixes #1610

retest this please

retest this please

retest this please


Is this report still available ?

I do not see anywhere that report on the https://ui.ads.microsoft.com/Reporting/v2, also using the API asking for this report I receive a response like
<?xml version="1.0"?>
<s:Envelope xmlns:s="http://schemas.xmlsoap.org/soap/envelope/">
<s:Header>
<h:TrackingId xmlns:h="https://bingads.microsoft.com/Reporting/v13">c0883eda-b5d3-4174-b728-bec4c6c32f1d</h:TrackingId>
</s:Header>
<s:Body>
<PollGenerateReportResponse xmlns="https://bingads.microsoft.com/Reporting/v13">
<ReportRequestStatus xmlns:i="http://www.w3.org/2001/XMLSchema-instance">
<ReportDownloadUrl i:nil="true"/>
<Status>Success</Status>
</ReportRequestStatus>
</PollGenerateReportResponse>
</s:Body>
</s:Envelope>
So report is ready, but there is no url to download...

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

ID: a18e439d-0d3c-4b83-2e25-8ed801151221
Version Independent ID: 4d8b6efe-00bc-e6e9-4607-658ec71a66da
Content: ProfessionalDemographicsAudienceReportRequest Data Object - Reporting - Microsoft Advertising
Content Source: Advertising/bingads-13/reporting-service/professionaldemographicsaudiencereportrequest.md
Service: bing-ads-reporting-service
GitHub Login: @eric-urban
Microsoft Alias: eur

@RoyBellingan per the documentation:
Use the download URL only if the Status element is set to Success. Even when the Status is set to Success, this element can be nil if no data is available for the submitted report parameters.
Please also note, the Professional demographics report is under 'Targeting' in the UI Reports tab. (There are no longer separate reports for the Audience network.) Please reach out if you have any follow up questions.

Thank you eric.


Extend Atomic* instead of using Atomic*FieldUpdater

Scala people (cc @viktorklang) had the same concern as you regarding memory footprint of Atomic* fields and initially went with an Atomic*FieldUpdater for the scala.concurrent.Future design.
In Scala 2.12, they came up with a nice idea: extending Atomic* (see code here and story here): no indirection level, no extra allocation, sun.misc.Unsafe level perf.
It looks to me that most of the Netty classes that use Atomic*FieldUpdater could benefit from the same design. The only exceptions are the ones that use more than one FieldUpdater (only one can be stored in the AtomicReference) and ones that extend an abstract class, such as AbstractReferenceCountedByteBuf.
WDYT?

@slandelle this will only work for things that are not public tho as otherwise it will be exposed to the user.

@normanmaurer You're right, sadly.

HashedWheelTimer maybe?

And then wait for JDK 8u76: http://shipilev.net/blog/2015/faster-atomic-fu/#_conclusion_and_parting_thoughts

@slandelle even for HashedWheelTimer there is not much we can do ....

Nervermind


Show more languages?

[ ] Add buttons to show more languages
[ ] Properly handle maximum/minimum
[ ] Reset languages when changing chart types?

A few ideas:

A simple button to toggle between 10 and 15 languages

Reduce the width if 15 languages are shown?

Or a button to show the next 5 languages

Remove the previous 5 so only 10 are shown at a time
Then we'll probably need an equivalent button to go back (at the top?)

Work is being done in show-more-languages branch


Add Vertical Zoom by scaling drawn elements

We can make a vertical zoom pressing ctrl + wheel. For this, we are
scaling the heights of each element in the TimeGraph. Also, we are preserving the
distance from the mouse to make a proper zoom similar to the horizontal
one.
This is actually in draft, to have comments.

What does this do with background elements, like the track backgrounds, and in particular the rounded corners? I recall this was one of the issues why @pierricgimmig turned this off at some point.

What does this do with background elements, like the track backgrounds, and in particular the rounded corners? I recall this was one of the issues why @pierricgimmig turned this off at some point.

These are images when you zoom-in and zoom-out. I have a TODO related with the text-size.

Well, I think that it's ready for review.
I'm not sure what to do with the text size. I think that one option could be scale similarly having a maximum size. But still have to think how to change it without change the size of the text in the bottom part (the pretty times). Maybe for another PR.


[Peek] A check for color of body element in html and change it based on theme is added.

Summary of the Pull Request

PR Checklist

[ ] Closes: #27747
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

This looks like an hack. Is this what we are doing in File Explorer add-on? Because it shows correctly on the PowerToys File Explorer add-on for code files, if I understood correctly. Both are supposed to be using the Monaco codebase underneath, so it's best if the code for both is similar.

I checked File explorer. Those are different in file explorer in shows html code. But in Peek it shows html page. In Peek there is DOM event to show page. I made some research and I found text might be also another color. I added a check to if theme is dark mode also if text is black to show it I need to change color of text in there. Thats why I applied that solution.
But if there is another way I can change it

I see, but will this override the color if the html we are previewing is trying to set it already? For example to black, will we override it to white? Is this what the browsers are doing if we open an html file?

I see, but will this override the color if the html we are previewing is trying to set it already? For example to black, will we override it to white? Is this what the browsers are doing if we open an html file?

I tested that. Its not overriding file. Now its working in Peek when dark mode. Also its white background with black text in browser in same time.

I'm sorry but this is not having the same behavior as other browsers. From the sample file, here's what Chrome is showing:

Here's what Peek is showing:

Let's say I used a different HTML code:

<title>Test HTML
this text can't be read when opened with peek
</title>

Peek Test

you don't see

testdata

This is what a browser shows:

This is what Peek shows:

I think the correct way to handle this I think would be instead of changing the html with JavaScript, change the color of the control we have behind or even the WebView2 control's color when we're peeking into a html file. There might even be other code changing the background that will conflict with your JavaScript code. I don't think injecting JavaScript is safe here.

LGTM! Can we please add to the comment that having a white background is the default for browsers showing html pages? Comments should usually tell the "Why?" when we do weird things.

Done.


Attempt to split the submissions page into multiple tabs to separate those that need actioned vs the rest

All looks good!


UX Bug: Add folder action is hidden behind "import" dropdown button

Describe the bug
"New folder" behind "Import" doesn't make sense. You can't import something that doesn't exist yet, but you can create it. "Add" instead of "Import" makes the most sense for all of the cases.
To Reproduce
Steps to reproduce the behavior:

Go to Files screen
Click on 'Import'
Notice that 'New Folder' is listed as an option under "Import"

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):
IPFS Desktop on macOS Monterey 12.4 (21F79)
UI v2.15.0
Revision fadd900
See the code
Report a bug
Additional context
N/A

@juliaxbow, we discussed this in our GUI triage meeting, and @lidel mentioned that "Import" was added in the past because it made the most sense for all of the options other than "New Folder". What do you think makes the most sense for this screen?

@SgtPooki I'd recommend switching over to "Add," as suggested, or "New," which you'll see on some other file sharing platforms. I'm leaning towards "New" but defer to you on current user flow.  Let me know if you want me to conduct some quick tests.
I would also recommend:

Modifying icons slightly to show upload vs new folder creation
Rename actions for clarity (e.g. "File Upload" or "File Import" vs "File")
Adding a line to differentiate between actions (upload vs create)

Screenshot attached for reference.

@lidel what do you think about the proposals from Julia?

I think both are :+1: thank you @juliaxbow!

The only nit around "Add" one is  that "IPFS Upload" should be changed to "IPFS Import"  (the data is already on IPFS, nothing to upload). If we fix that, it will look better than "Import" three times.
Icon sources are at https://github.com/ipfs-shipyard/ipfs-css#icons - adding new variants is nice to have.

Ok so next steps:

change button to "Add"
Change items to "File Import", "Folder Import", "IPFS Import", and "New Folder"
Ensure any icons we use are in https://github.com/ipfs-shipyard/ipfs-css#icons

I'll add in those icons shortly


Migrate backtest logic from NT

Description

Migrate experiment backtest logic from NT.
Motivation and Context

How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[ ] Fix bugs
[ ] Add new feature
[ ] Update documentation

@lihuoran  Will there be a related PR in neutrader?


Timeline: do not add local history entry when saving without changes

Type: Bug

Open a file with VSCode.
Exit VSCode without closing any tabs.
Launch VSCode again so that it continues from where you left (do not edit the file).
Press Ctrl + S to save.

VSCode will log a save for that file in the timeline even if zero changes were made. The "Date modified" field of the file also updates.
 It looks like you may be using an old version of VS Code, the latest stable release is 1.90.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

It adds only a single new entry to the local history when saving without changes. After that, "file>save" does nothing unless you actually make new changes.


Fix great wounds

Can I get a bit more explanation of what this change is intended to do and what issue its addressing?

Sorry, I should have included an explanation. Here it is.
Today, great wounds doesn't work. It throws an exception when sanitizeTokenName() is called with an actor as argument. The fix is to call sanitizeActorName() instead.

Ah! thank you very much! I'll get this tested and integrated as soon as I can.
I sincerely appreciate the contribution. ❤


SvgCahedImage missing in FFImageLoading.Svg.Forms

Hi,
I just installed Xamarin.FFImageLoading.Svg.Forms v2.2.9 from nuget (in VS 2017).
I get this error when building:
"ffimageloadingsvg:SvgCachedImage not found in xmlns clr-namespace:FFImageLoading.Svg.Forms;assembly=FFImageLoading.Svg.Forms"
Browsing "FFImageLoading.Svg.Forms" from Object Browser I saw "SvgCachedImage" is missing (see screenshot).

What I am doing wrong?

me too!

I have this issue too. Is there a workaround?

For use ffimageloadingsvg:SvgCachedImage update to 2.2.10-pre-391 in nuget (prerelease)

@RaFaGG Sorry, but I cannot find it.

Sorry, I'm getting this now:
Unhandled Exception:
FreshTinyIoC.TinyIoCResolutionException: Resolve failed: SettingsPage - Reason: Could not resolve type with token 0100004f (from typeref, class/assembly FFImageLoading.Svg.Forms.SvgCachedImage, FFImageLoading.Svg.Forms, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null)

@quakeulf  You need to update nuget packages to the same version in all projects (it's important).

Also see this: https://github.com/luberda-molinet/FFImageLoading/issues/604#issuecomment-306330945
It should be fixed in the new prerelease that I'm gonna push.

Ok, it works now, thanks for fixing!
However now the SVGs aren't showing up, but I suspect something else is the problem, lol.


build failure, Ubuntu 16.04

I followed the Linux build instructions in the README. From the build log:
==== Building 00_cube (debug64) ====
Creating ../../linux64_gcc/obj/x64/Debug/02_camera
Creating ../../linux64_gcc/obj/x64/Debug/00_tutorial
Creating ../../linux64_gcc/obj/x64/Debug/02_camera/example/02_camera
Creating ../../linux64_gcc/obj/x64/Debug/00_tutorial/example/00_tutorial
Creating ../../linux64_gcc/obj/x64/Debug/00_cube
Creating ../../linux64_gcc/obj/x64/Debug/01_shapes
Creating ../../linux64_gcc/obj/x64/Debug/02_camera/example/03_materials
Creating ../../linux64_gcc/obj/x64/Debug/00_tutorial/example/meta/00_tutorial
Creating ../../linux64_gcc/obj/x64/Debug/02_camera/src/mud
Creating ../../linux64_gcc/obj/x64/Debug/01_shapes/example/01_shapes
Creating ../../linux64_gcc/obj/x64/Debug/00_cube/example/00_cube
Creating ../../linux64_gcc/obj/x64/Debug/00_tutorial/src/mud
02_camera.cpp
Creating ../../linux64_gcc/obj/x64/Debug/01_shapes/src/mud
Creating ../../linux64_gcc/obj/x64/Debug/00_cube/src/mud
00_tutorial.cpp
01_shapes.cpp
00_cube.cpp
In file included from ../../../src/noise/Structs.h:9:0,
                 from ../../../src/noise/Api.h:2,
                 from ../../../src/mud/mud.h:13,
                 from ../../../example/02_camera/02_camera.cpp:1:
../../../src/noise/Noise.h:11:23: fatal error: FastNoise.h: No such file or directory
compilation terminated.
02_camera.make:226: recipe for target '../../linux64_gcc/obj/x64/Debug/02_camera/example/02_camera/02_camera.o' failed
make[1]: *** [../../linux64_gcc/obj/x64/Debug/02_camera/example/02_camera/02_camera.o] Error 1
Makefile:108: recipe for target '02_camera' failed
make: *** [02_camera] Error 2
make: *** Waiting for unfinished jobs....
In file included from ../../../src/noise/Structs.h:9:0,
                 from ../../../src/noise/Api.h:2,
                 from ../../../src/mud/mud.h:13,
                 from ../../../example/00_tutorial/00_tutorial.cpp:1:
../../../src/noise/Noise.h:11:23: fatal error: FastNoise.h: No such file or directory
compilation terminated.

The include path for 02_camera needs to contain 3rdparty/FastNoise, and that's missing from the generated makefile in build/projects/gmake-linux/02_camera.make.
Looking at the Lua build scripts, 'uses_mud_noise' in 'mud.lua' defines this include directory. That is part of 'mud.noise', which is part of 'mud.mud'. So, 02_camera needs to depend on 'mud.mud'.
So, in 'scripts/mud_example.lua', I added 'mud.mud' to the dependencies for 02_camera:
    mud_example("02_camera",            { mud.mud, mud.gfx, mud.gfx.ui, mud.gfx.pbr },   { _G["03_materials"] })

I guess that's appropriate since 02_camera.cpp includes "mud.mud"? But it fails, GENie blows up with an error if I do this.

i get the same error.

i get the same error.

the build has been generally broken for a few days due to a refactor, I'll fix it and get back to you

This should be fixed now.
To answer your question, you were right that mud.mud contains all of mud modules including mud.noise, but it's a table so you should have substituted { mud.gfx, mud.gfx.ui, mud.gfx.pbr } with mud.mud for genie to work.
Anyway, the dependencies are fixed and this example doesn't depend on that module anymore, so it's all good.
There are still some functional bugs in the examples due to the refactor, but I'll iron them out in the following days, and at least it builds now.

yea compile works now, about half the examples fail but you already know that.


Make neurosift a library that can be used within other web apps

Not going to work on this now, but this is an issue for the future, and I am tagging @garrettmflynn who offered to help.

In case you didn't already know, we've gone with neurosift as our primary content visualization for files in the NWB GUIDE (which can even be configured to recognize and open .nwb suffixes when you install it); this just uses a simple iframe to embed a window to neurosift however
So having a library/node package for this is still something we'd love long term

@CodyCBakerPhD I have recently taken some steps toward this.
But I think a shorter-term goal would be to identify individual components that would be useful to render within nwb-guide, and it would be pretty straightforward for me to separate these out into libraries. If you or @garrettmflynn can identify something simple we can use that as a starting point.


@Cached generates incorrect code if the method has extension parameters

Eg. for the method
@Cached
def Object someMethod(extension SomeExtension ex) { ... }

the generated code is missing the extension keyword or annotation:
private Object someMethod_init(final SomeExtension ex) { ... }

Workaround:
@Cached
def Object someMethod(extension SomeExtension ex1) {
    extension val ex = ex1
    ...
}

Parameter annotations are currently not copied.


Changed SmoothMixedL21Norm parent class

Changed SmoothMixedL21Norm parent class from Function to MixedL21Norm. Is this right?
I was getting "NotImplementedErrors" originally

The proximal operator for the SmoothMixedL21Norm does not have a closed form expression and I believe also the same holds for the convex_conjugate hence the proximal_conjugate. It can be a child of the MixedL21Norm but we need these methods to return raise NotImplemented otherwise the parent method will be used.


Separate MEI Core into modules

Opening for discussion.
mei-source.xml is currently over 20,000 lines long. A few years ago @raffazizzi and I prototyped a method of separating MEI Source into different files, each containing a separate module. We were able to do this and still maintain all of the fun and fancy code completion tools in oXygen.
I think this might prove to be a bit less intimidating for moderately-experienced people who know enough to contribute to the core, but can't quite navigate all of MEI in one file.

It could also help development tools like github handling diffs. We've had problems when the diff was too bit for github to display.

I see the advantages of breaking up mei-source into more manageable units. But, I'm not sure that splitting it up by module is the best way to go because it presumes that declarations never move.  It's fairly rare, but moving a declaration from one module to another is sometimes necessary.
It would be going too far to put every declaration in a separate file, so I think a reasonable compromise might be to put each category of declaration in a separate file; that is, one file for macroSpec, another file for classSpec, and another file for elementSpec declarations.
The moduleSpec and constraintSpec declarations could be separated out as well, but I don't think the schema needs to that atomistic yet.  Management of the 'inclusions-within-inclusions' nature of that could be difficult.

I suggested a by-module separation because I thought it might help with the development of customizations. Adding a new module (e.g., Turkish Makam or Page-based layouts) or editing an existing module (e.g., neumes) would be a lot easier if we could develop this independently and keep all of the elements, attributes, and data types for that particular module together.
From what I understand, in your suggested method I would still have to spread my work over all of MEI, keeping all of the elementspec / macrospec / etc. files open and putting the appropriate definitions in there (somewhere).
I don't foresee a big problem with moving declarations. Since oXygen and the other tools see the entire spec regardless of which file its contained in, then there shouldn't be a problem with usage and type-ahead completion, and a quick project find can still bring you to the definition.

@raffazizzi Do you still have the stuff you wrote for this when we looked at it back in Charlottesville?

Currently, develop/source/driver.xml includes the schemaSpec by using an external entity reference --
<!ENTITY schemaSpec SYSTEM "specs/mei-source.xml">

and then later --
&schemaSpec;

We can continue this pattern by creating a separate directory under /source/specs for each module and modifying mei-source.xml to have the following structure --
<div>
  <schemaSpec>
    &module_MEI;
    &module_MEI.analytical;
    etc.
  </schemaSpec>
</div>

Each included module file would provide a moduleSpec element, followed by macroSpec, classSpec, and elementSpec elements related to the module.
We can use ENTITY declarations/references first and replace them with XInclude later, if necessary.
It should be relatively easy to write an XSLT that replace the current mei-source.xml file with one that "calls" the other module files.

Bumping up the priority on this one since our discussions at MEC 2017.


(DOCUMENT-318) Add note about unless taking arrays

CLA signed by all contributors.

@peterhuene @melissa: I just noticed this PR. It hasn't been covered in PR triage because it has the 'PL' label, but @petems isn't on a team that merges to puppet despite working at PL :)
Should we drop the PL label from this (and #3800) and/or treat these as part of PR triage?

@kylog oops, that was my mistake. Thanks for catching it! I say we do what you suggest.

@petems Would you mind terribly amending the commit message to be prefixed with (docs) for this one?  Sorry this fell off our radar.  I'd like to get this merged in.

@peterhuene Done! :+1:

for future reference, just made sense it was "all" commands need to return non-zero exit codes:
0 ~/work/puppet (stable)  $ be puppet apply -e 'exec { "echo oh no": path => $path, unless => ["/usr/bin/false","/usr/bin/true"] }'
Notice: Compiled catalog for arcturus.corp.puppetlabs.net in environment production in 0.43 seconds
Notice: Applied catalog in 0.04 seconds

vs
0 ~/work/puppet (stable)  $ be puppet apply -e 'exec { "echo oh no": path => $path, unless => ["/usr/bin/false","/usr/bin/false"] }'
Notice: Compiled catalog for arcturus.corp.puppetlabs.net in environment production in 0.43 seconds
Notice: /Stage[main]/Main/Exec[echo oh no]/returns: executed successfully
Notice: Applied catalog in 0.05 seconds


fixing reception of packets with DLC=0

Packets with DLC=0 are reported in the same way as if no packet has been received. As the CAN standard allows packets with DLC=0, I have changed the behaviour of parsePacket to return -1 if no packet has been received and the packet length otherwise.

Hi @mwensing,
Please see the comment in https://github.com/sandeepmistry/arduino-CAN/issues/7#issuecomment-418209109
Maybe the examples should be updated accordingly?

Of course fixing the examples is one way, but in my opinion the return value of the parsePacket function should differ between a packet with length 0 and no packet received at all.

How can we achieve this also with the ReceiverCallback?


Research custom allocators (jemalloc/mimalloc) if they could bring better performance

It looks like jemalloc brought improvements to Rust Analyzer:

https://github.com/rust-lang/rust-analyzer/issues/1441
https://github.com/rust-lang/rust-analyzer/pull/17007

It would be awesome to verify that one of these two would also improve CairoLS. For this, we need a reproducible benchmark code, though.

Could you plz give me some more crates like jemallocator, snmalloc, mimalloc and so on?
I need to pick global allocator crates as many as possible and select which allocator is the best for our project performance.

Hey! These three are the only popular choices I know of.

I built snmalloc as a global allocator in my project, and only I got errors with snmalloc.
The others did work as well. Do you know about the reason?

No idea. What were the errors?

FROM lukemathwalker/cargo-chef:latest-rust-1 AS chef
WORKDIR /app

# Builds a cargo-chef plan
FROM chef AS planner
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

FROM chef AS builder
COPY --from=planner /app/recipe.json recipe.json
...

It is a part of my Dockerfile.
I got errors with this part in snmalloc-sys.

That's it.

looks like you haven't installed cmake in your container

I added command to install cmake in docker container, and built again.
As a result, the error is diappeard.
😄


Organizing all the skin joints into sub deformer sets as control sets.

https://github.com/mgear-dev/mgear4/issues/369
Organized the module joints and controllers into subsets, categorizing them by component name and their respective sides by default.

@joji2468ng Right now each component has the option to add it to a custom controller subset. This is used to organize the controls in categories, not components. For example all facial controls or cloth controls in the same group

I think similar approach can be useful for joint, but to be honest I am don't the advantage to have each component deformer in a separate unique set. For example in facial rigs I use many control_01 type components with only 1 joint. That will be ver impractical if  each joint is under one subset.
I am sorry to push back another PR from you. I hope you understand


Hyperlinking Related Models

I am currently customizing how related models are rendered using form_args + get_label however it would be great if there were some way of making what is rendered a hyperlink / button that navigates to the appropriate ModelView.
Eg I have an edit view with:

and it would be great if I could click on any of those boxes.

from flask_admin.contrib.sqla import ModelView
from flask import url_for
from yourapp import db

class Parent(db.Model):
    __tablename__ = 'parent'
    id = db.Column(db.Integer, primary_key=True)
    name = db.Column(db.String(50), nullable=False)
    children = db.relationship('Child', backref='parent')

class Child(db.Model):
    __tablename__ = 'child'
    id = db.Column(db.Integer, primary_key=True)
    name = db.Column(db.String(50), nullable=False)
    parent_id = db.Column(db.Integer, db.ForeignKey('parent.id'))


Temporal warnings should contain more information

For example, if a data-binding has not been satisfied, we get the current message in the problems panel in Fuse studio:
{foo} not found in data context

It would be nice to know who was looking for foo so that i could more easily jump to that spot in my code.

@kristianhasselknippe: The Diagnostic we pass to fuse-studio should contain this information already, as the SourceObject-member...

Ah, i guess this should be a tools ticket then.


[No QA] Only load full policies one-by-one as needed

Details
So we want to make sure that whenever a workspace or any of its sub-pages attempts to render, then we'll load the full policy for that workspace. So whenever we navigate to a workspace page (and did not just come from one), then we'll load that full policy.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/181676
Tests

Sign into NewDot using an account with a workspace.
Go to /settings. Verify using logs or the network console that no full policy is loaded.
Click on an existing workspace. Verify using logs or the network console that the full policy is loaded.
Refresh the page. Verify using logs or the network console that the full policy is loaded.
Click on the back arrow. Verify you are taken back to the settings page.
Click on an existing workspace. Verify using logs or the network console that the full policy is loaded.
Click on a workspace sub-page. Verify using logs or the network console that no full policy is loaded.
Refresh the page on one of the workspace sub-pages. Verify using logs or the network console that the full policy is loaded.
Click on the x to close the modal. Verify that the full settings modal closes.

QA Steps
Only regression tests.
Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Something I missed in this implementation – the WorkspaceInvitePage is in it's own navigator, so if you link directly to the workspace invite page then you're not guaranteed to have a policy loaded. I think this is just an oversight and it should really be in the same navigator as the rest of the workspace pages.

I went ahead and addressed this and moved the WorkspaceInvitePage into the same navigator as the rest of the workspace pages.

Updated and ready for review!

PR has been updated per @tgolen's comment above ^

You got conflicts

Looking good overall, but there's a conflict!

Conflicts resolved!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @Gonals in version: 1.1.14-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @AndrewGable in version: 1.1.15-15 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Allow flexible overpass settings

Until now, overpass QL settings (the first statement in an overpass query) were mostly hardcoded, with only maxsize and timeout being open to change from the high level functions of OSMnx.
In issue https://github.com/gboeing/osmnx/issues/384, the ability to modify the date for the overpass query was requested as a feature. The date in an overpass query is modified in its settings. When I looked into it, I noticed it would be probably wiser to make settings customizable in a similar fashion to the current custom_filters, instead of making an hyperspecialized design just for the date.
The proposed change allows the user to feed a customized settings string. If no string is given, the same default values that are now hardcoded will be used, thus not breaking backwards compatibility.
@gboeing , no tests for this yet. I've tried to understand the testing philosophy of OSMnx from the current tests but I haven't seen any testing related to querying Overpass. Could you kindly let me know your thoughts on this?

Coverage increased (+0.2%) to 93.06% when pulling ac62c798a03759ca0b1eac0b2bec07e98cf81ec6 on pmartincalvo:feature-flexible-overpass-settings into 15eddc0672f0ec951ada1b89eb417df44d35636e on gboeing:master.

Makes sense. I'll try to chase down all occurrences and apply the same philosophy.
On a side note, right now the default value for the settings is getting hardcoded in the function. If you approve, I would like to move it to the settings file and import it where necessary so there's a single point for modifying if necessary.
Will wait for your confirmation on that.

On a side note, right now the default value for the settings is getting hardcoded in the function. If you approve, I would like to move it to the settings file and import it where necessary so there's a single point for modifying if necessary.

I think that would make sense for usability and consistency with the rest of the package.

Footprints and POIs now also allow for custom settings. I have also set the default setting string in settings.py as discussed.
Unless you think the same needs to be applied somewhere else, I would say only testing is pending.

Tests done. Do let me know if any other changes are required.

@pmartincalvo thanks!

closes #384

Note that this custom_settings parameter has been deprecated. It is instead now easily accessible via the ox.config() function and settings module as overpass_settings. See #492.


Generate Java grpc stubs

Codecov Report

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

@@             Coverage Diff              @@
##             master    #1188      +/-   ##
============================================
+ Coverage     86.07%   86.07%   +<.01%
  Complexity     3688     3688
============================================
  Files           372      372
  Lines         14582    14584       +2
  Branches       1628     1628
============================================
+ Hits          12551    12553       +2
  Misses         1594     1594
  Partials        437      437

Impacted Files
Coverage Δ
Complexity Δ

...gle/api/codegen/viewmodel/ApiCallSettingsView.java
85.71% <ø> (ø)
4 <0> (ø)
:arrow_down:

...pi/codegen/transformer/ApiCallableTransformer.java
98.27% <100%> (+0.02%)
23 <0> (ø)
:arrow_down:

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 2304654...9daa6ac. Read the comment docs.

LGTM


policies missing under remediation folders

There are many new policies missing under " Azure-Security-Center/Remediation scripts/ " folders, but I'm currently looking to remediate the " Web apps should request an SSL certificate for all incoming requests " I do not see the deployifnotexists policy available for the same. Kindly help

Hi @farookhus
please take a look at this DINE policy, as it might cover the same intent.

Hi @farookhus
Has the existing DINE policy solved your issue?

Yes Tom, it really helped. Thanks..


Use scrapy together with warcio

I am interested in downloading a list of files through scrapy to save them in a warc file, is that possible? The documentation only shows how to do it with the requests module and a simple request so if there is any way to do this maybe it could also be added as an example of use in the documentation.

Doesn't scrapy already have warc support? Via plugins. Searching for [scrapy warc] shows several candidates.

Doesn't scrapy already have warc support?

To be honest I had not thought of looking for something like that because as the WARC format is something so "niche" I imagined that warcio was the only Python implementation available.
Following your suggestion I found this script that does what I need. I have not had the opportunity to try it but if you want you can give this issue as resolved (if I get to find something better then I will comment on it here).


MavenRepository environment variables are no longer resolved

In @file:MavenRepository the username and password are no longer resolved as environment variable. According to documentation username and password should be resolved as environmental variable string surrounded by double {} brackets. But it does not work:
@file:MavenRepository("internal", "https://artifactory/artifactory/libs-release", user="{{ARTIFACTORY_USERNAME}}", password="{{ARTIFACTORY_PASSWORD}}")

Commit 15ba17db46bc836c670177b4a12b3f1743a38a4a removed the functionality .

Indeed, it did. Sorry for the inconvenience.

Indeed, it did. Sorry for the inconvenience.


bridging the game between intallation and adding camera

Feniks is installed, I think. What I see now is "Node Info"

Return to Login loops on this page with no way to get out

Registration doesn't work and stuck here:

wheely thing shows up when pressing login but nothing else with zero error message
What's the next step to get the add camera page?

I have the sane issue as well.  Can anybody enlighten me on how to progress please?


Suggest: prometheus-config use json format instead of yaml

Proposal
when I want to modify the config, I got it. I don't know how to convert this string to yaml, yaml has the advantage of readability, but this string doesn't. If the data is json format, I can easily modify it
kubectl get configMap  prometheus-config -o yaml
apiVersion: v1
data:
  prometheus.yml: "global:\n  scrape_interval: 30s\n  evaluation_interval: 30s\nscrape_configs:\n     \ - job_name: kubelet\n    scheme: https\n    tls_config:\n      ca_file: /var/run/secrets/kubernetes.io/serviceaccount/ca.crt\n     \   bearer_token_file: /var/run/secrets/kubernetes.io/serviceaccount/token\n    kubernetes_sd_configs:\n     \     - role: node\n    relabel_configs:\n      - action: labelmap\n        regex:     __meta_kubernetes_node_label_(.+)\n      - target_label: __address__\n        replacement:     'kubernetes.default.svc:443'\n      - source_labels:\n          - __meta_kubernetes_node_name\n     \       regex: (.+)\n        target_label: __metrics_path__\n        replacement:     '/api/v1/nodes/${1}/proxy/metrics'\n  - job_name: apiserver\n    metrics_path:     /metrics\n    kubernetes_sd_configs:\n      - role: endpoints\n    scheme: https\n     \   tls_config:\n      ca_file: /var/run/secrets/kubernetes.io/serviceaccount/ca.cr ....

Yaml is a json subset by design. That's why you see offer documentations mixing [] (from json notation) or - (from yaml notation) to create a list of elements.
Saying that, that means you can paste a whole json inside a yaml file, and it will be valid (but a bit confusing for users) 😃
Only limit, json says you can't have multiple "root" node in a file.
With yaml and the triple dash separator (---) you can put multiple documents in a single file (case of kubernetes manifests as an example)

Yaml is a json subset by design. That's why you see offen documentations mixing [] (from json notation) or - (from yaml notation) to create a list of elements. Saying that, that means you can paste a whole json inside a yaml file, and it will be valid (but a bit confusing for users) 😃
Only limit, json says you can't have multiple "root" node in a file. With yaml and the triple dash separator (---) you can put multiple documents in a single file (case of kubernetes manifests as an example)

but when I want to update my config, the yaml strings leave me at a loss. It contains too many spaces and \n 😭


Blinks in Firefox when rendered on server and then re-rendered on the client.

An example:
<TextareaAutosize rows={5} defaultValue={[...5 lines]}/>

is rendered on the server (using react server-side rendering) and then re-rendered on the client using exactly the same argument.
When it is re-rendered on the client, component recomputes the height differently than when it is done by the Firefox 76.0.1 (looks fine on Chrome), so page jumps on load.
I have made a small online demo here: https://learn-idris.net/textarea-autosize-blink
What might be the solution:

fix some styles to make initial rendering identical to computed rendering (probably hard)
add an option to explicitly disable computation of height on initialization; recompute height only on a change
maybe turn on this option when both rows and minRows specified

Could you also share the source code of this demo and share more information about your environment? So far I can't see any blinking in any browser, including FireFox.

OK, full example: https://github.com/stepancheg/textarea-autosize-blink
Launch with:
yarn install
yarn dev

Then navigate to http://localhost:3000/textarea-autosize-blink
This is default render of text area:

This is how it is rendered after textarea-autosize re-renders it:

Note space at the bottom of textarea before the rerender.

This Stack Overflow question (8 years ago) describes the space in textarea.

I'm afraid that this project doesn't start for me correctly:
[S] /textarea-autosize-blink/src/server/textarea_autosize_blink.tsx:10
[S]             server_1.renderToString(<html>
[S]                                     ^
[S]
[S] SyntaxError: Unexpected token <

I've tried to reproduce the issue that you have linked to in isolation: https://jsfiddle.net/kqh108yf/ and I still couldn't verify what you are reporting on my machine. After some additional thinking, I've also tested this on Windows (as I'm using OSX on a daily basis) and voila - there it was.
I'm afraid though that there is not much I can do about it here - this issue is very specific. I couldn't even patch passed in rows conditionally after detecting the correct browser+OS combination as that would result in client-rendered output mismatching with the one provided by the server.
SSRing this component correctly is, unfortunately, tricky (if you want to have it handled automatically). I would very much like to improve the story around this but I don't know a better way to do this than what we have now. The recommendation is to use inline height style when SSRing - but, of course, you would have to know what a correct value for that is.

I'm afraid though that there is not much I can do about it here

Well, then my original feature request: make an option (off by default) to do nothing (do not recompute height) until the user presses a key.
It will still jump a little first time the user modifies text, but it won't be as annoying as jump at page open.

Well, I don't quite agree about this being better - the problem has just shifted to another point in time. I refuse to go into that direction out of principle. Sorry if that sounds harsh - I really don't intend to be a jerk, but I just don't believe this being a good solution for this library. If you think otherwise you can always fork or use patch-package to introduce this on your end. I truly believe that it's just impossible to cover for all use cases & opinions and as a library author/maintainer I need to take other factors into account (for example I try to keep this library as small as possible).


Loading an edgetpu .tflite model with tensorflow.lite.Interpreter crashes

Description
I am trying to load an *edgetpu.tflite model (for example the supplied mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite or any other model I tried) with the regular tensorflow.lite.Interpreter and the edgetpu delegate. This silently crashes (I suspect a segmentation fault in the background).
import tensorflow as tf

# Program silently crashes while executing code below
edgetpu_interpreter = tf.lite.Interpreter('./mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite',
                                         experimental_delegates=[tf.lite.experimental.load_delegate('edgetpu.dll')])

I tried with tensorflow 2.10.0 and 2.12.0. I can load the same model on the same machine with tflite-runtime 2.5.0 without any problems, however I need regular Tensorflow for dataset operations in my code and both (tflite-runtime and regular tensorflow) cannot be imported at the same time. Since the tflite Interpreter is included in the regular tensorflow package (and I have seen recommendations of using it instead of the dedicated tflite-runtime package), I would assume that this should work, shouldn't it?
Click to expand!
Issue Type
Bug
Operating System
Windows 10
Coral Device
USB Accelerator
Other Devices
No response
Programming Language
Python 3.9
Relevant Log Output
No response

@enricoschroeder our recommendation is to use  tflite-runtime 2.5.0.post1. If you would like use with tf 2.10, 2.12 or any other version, please build libedgetpu, pycoral and tflite_runtime pacakges for your platform, using the instructions at: https://coral.ai/docs/notes/build-coral/. Thanks!!

@hjonnala Thanks for the answer! Before I go down the rabbit hole to build everthing with a current tensorflow version: Am I right to expect that using the regular tensorflow.lite.Interpreter with a matching libedgetpu should work?
Also, is there any particular reason why you do not provide more up-to-date versions of everything?

@hjonnala Thanks for the answer! Before I go down the rabbit hole to build everthing with a current tensorflow version: Am I right to expect that using the regular tensorflow.lite.Interpreter with a matching libedgetpu should work?

Yes..


Task priority not cloned when duplicating a task to another project

Expected behaviour
Task priority is not copies when duplicating a task to another project.
Actual behaviour
Task is cloned in the target project but the priority is always set to 0.
Steps to reproduce
Create a task with some priority greater than zero and the duplicate it in another project.
Configuration
Application version: 1.0.28
PHP version: 5.5.9-1ubuntu4.16
PHP SAPI: fpm-fcgi
OS version: Linux 3.13.0-79-generic
Database driver: sqlite
Database version: 3.8.2
Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.102 Safari/537.36

Can you confirm that this is still happening? I just tried this on a fresh dev environment and it works for me.

I'm sorry but we stopped using kanboard a few weeks ago. Thanks!

This feature is now supported in the development branch.


[Bug]: Cannot build docker image for 1.7.9 and 1.7.10

Is there an existing issue for this?

[X] I have searched the existing issues

Description
I'm trying to redeploy to Heroku updated version of AppSmith. It worked for 1.7.8, however got broken with 1.7.9 and 1.7.10.
It fails here:
Step 17/37 : COPY ${JAR_FILE} backend/server.jar
COPY failed: no source files were specified

Full logs attached.
build_logs.txt
Steps To Reproduce
Execute steps from the Re-Deploy your App using Heroku CLI
Public Sample App
No response
Version
1.7.10

My bad. The issues was that I did not do the cd ./appsmith/deploy/heroku step which was specified in the documentation.


Extract parameters from ts types in function files

Closes https://github.com/mui/mui-toolpad/issues/2093
https://github.com/mui/mui-toolpad/assets/2109932/ce8e64f8-7cc4-46c5-884a-c6b42b2a3b47
To do:

[ ] what with parameters that change types, but already have a value assigned of a different type? (see video ☝ )
[x] docs
[ ] test on mui-private and mui-public before merging

This is really cool!
Maybe boolean types could have a "checkbox" control? Right now it's the same control as for objects.

I tried it today and it is very impressive! The code is concise now.
I was able to run the following:
export async function getRepositoryDetails(slug: string) {
  const res = await fetch(`https://api.ossinsight.io/gh/repo/${slug}`, {
    method: 'GET',
  });
  if (res.status !== 200) {
    throw new Error(`HTTP ${res.status}: ${(await res.text()).slice(0, 500)}`);
  }
  return res.json();
}

Under this PR have we also removed the need to use createfunction API when creating a custom function?

Under this PR have we also removed the need to use createfunction API when creating a custom function?

In https://github.com/mui/mui-public/pull/111 I was able to remove all createServer calls. Still have to try out the same for mui-private.


📃: Image Search Engine

:red_circle: Title : Image Search Engine
:red_circle: Aim : To get images with Description
:red_circle: Tech Stack Used : HTML,CSS.JS and Api
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name :
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.)
GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Assign me this issue @Avdhesh-Varshney to make contribution to the project
Thank you

@GAVINESHWAR is it different from #323 ?

It will work as a search engine type , displays the multiple images as google  with description as well.The #323 issue shows that just  generate image using text. @Avdhesh-Varshney please look over if any feature required I can make to it sir
Thank you

@GAVINESHWAR ok go ahead.

@GAVINESHWAR why you have commented on every pr without permission? This behaviour will debarred you from the program. It's a last warning.

@GAVINESHWAR why you have commented on every pr without permission? This behaviour will debarred you from the program. It's a last warning.

I am extremely sorry for the issue sir. These things will not repeat again Sir
Thank you


Add vulnfeed tests to cloudbuild.yaml

Adds a run_tests.sh that will run go test on the vulnfeeds directory. Does not run go vet as that is already done by the lint script.

Looks like golang needs to be installed in the Cloud Build environment

/gcbrun

Added the install go script to manually install go 1.18


Changed to SharedPtr

Changed to SharedPtr

You're English is amazing. But I want you to use Korean


Prepared statement throwing socket exception

Running a prepared statement on a normal connection throws an exception:
final stmt = await _connection!.prepare('SELECT * FROM users WHERE foo = ?');
final result = await stmt.execute(['bar']);
await stmt.deallocate();

This throws the exception:
Unhandled exception:
Exception: Skipping socket data, because of connection bad state
State: connectionEstablished
Data: [38, 0, 0, 4, 3, 100, 101, 102, 4, 116, 101, 115, 116, 4, 117, 115, 101, 114, 4, 117, 115, 101, 114, 2, 105, 100, 2, 105, 100, 12, 63, 0, 11, 0, 0, 0, 3, 35, 66, 0, 0, 0]

Also, if I try:
final stmt = await _connection!.prepare("SELECT * FROM users WHERE foo = '?'");

I get the error that the number of params does not match the provided list.

Hi Esherp. Is it happening every time, or sometimes?

I can reproduce this issue. I will check

Ok fixed here https://github.com/zim32/mysql.dart/commit/bfb1dd6248b08ff822da5f5097ebbdbf41aff278
Release 0.0.14
Waiting for feedback to close this issue.

Cheers all working


add comment

this closes #34

thanks @johnbister - the + in the Add comment seems to behave differently than the one in Bookmark Add (which also renders cleaner and feels 'right'). Could be a browser render thing, see the video grab below:
https://user-images.githubusercontent.com/98415805/197010019-a040b07d-370c-4fc4-ac1a-37dbde40b416.mp4

@silvio-hajdin this testing is so helpful! Here's what I see on my end using chrome.  My guess is that some browsers don't support vector-effect: non-scaling stroke. Let me know what browser you're using. Can you also check the zoom in and zoom out icons? I used the same vector-effect on those. We might have to try a different approach. Thank you so much!
https://user-images.githubusercontent.com/70543333/197036909-c6e688e1-adf6-40bb-b58c-1dcc582052b8.mov

totally John, same comment re:in-browser rendering as in #83 😄

Looks like there's good browser support for vector-effect: non-scaling stroke
https://caniuse.com/?search=vector-effect%3A non-scaling stroke
but I am definitely seeing differences between browsers.  I'll also investigate this further.

@silvio-hajdin @kristastarr Let me know if the scaling stroke has been resolved on this one!

@kristastarr good catch! I used transform-origin, but I did it by pixels. Maybe it needs to be by percentage? It gets out of whack when it's greater or less than 100 percent viewing, but looks fine at 100 percent.  Will work on it today!

@kristastarr I think I fixed 'add comment' for safari. Do you mind checking and adding a review before we merge?


uplaod

Do you think its possible to automate the upload of the ssh public key to GitHub too  ?

Yes it already does when there is no working ssh key 😊 Let me know if it works for you.

ah I read the readme but it was not clear.


Prepare release v1.29.0

Signed-off-by: Juraci Paixão Kröhling<EMAIL_ADDRESS>

@albertteoh, as the previous release manager, would you mind reviewing this one?


Error Handling

Currently, errors in Python/Teal cause everything to halt and fail - you have to restart the program.
Teal needs a condition signalling/exception handling mechanism.
This has two parts

the machine instructions (register condition handler, signal condition)
the teal special forms to declare handlers/restarts, and signal conditions.

Preliminary thoughts

"boxed error" return types (like Rust, Haskell, others). Requires syntax and support for "proper" types.
condition handlers (exceptions, or generic "signals" as in Common Lisp / R). Fairly easy to implement here.


fixup: decimal * null error

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Problem Summary(Required) ：

select id_decimal  * NULL from t report error "DECIMAL's precision should range from 1 to 38",  before resolving multiplication function, use DECIMAL32(0,0) as type of NullLiteral,   then invoking ScalarType.createDecimalV3Type(PrimitiveType type, int precision, int scale) on argument list (DECIMAL64, 0, 0), but the precision can not be zero, so in this function, when precision is zero, use 1 instead.

[FE PR Coverage check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
2
2
100.00%

If precision cannot be 0, why are 0 and null constructed as DECIMAL32(0,0) instead of (1,0)

starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/ScalarType.java
2
2
100.00%


return all fields for dimensions full text search

the pubmed fulltext search has two parts - first fetch pubmed's PIDs for publications that match the full text search fulltext_id_search and then get the metadata for those ids fulltext_search. I had the test run fulltext_id_search since its a shorter query and returns an easy metric to test (# of publications that match the search).


Add Nginx support

Implement nginx support.

Updated and tested ability to run on nginx and xampp.

Issue has been resolved.


cable thickness and colour

thickness:
in the line drawing shader code, I haven't found where to set the line thickness. need to figure this out.
colour:
if it is a feedback cable, colour it red? otherwise green for now?

update: each new cable is either red, green, or blue.