uninitialized constant Sigh::DeveloperCenter (NameError) on Version 0.3.4

I've updated the gem and my builds stopped working . I had to downgrade it to run everything smoothly again.
/Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/actions/sigh.rb:12:in run': uninitialized constant Sigh::DeveloperCenter (NameError) 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/fast_file.rb:81:in block (2 levels) in method_missing'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/actions/actions_helper.rb:28:in execute_action' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/fast_file.rb:80:in block in method_missing'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/fast_file.rb:79:in chdir' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/fast_file.rb:79:in method_missing'
from (eval):11:in block (2 levels) in parse' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/runner.rb:17:in call'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/runner.rb:17:in block in execute' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/runner.rb:11:in chdir'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/runner.rb:11:in execute' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/lane_manager.rb:15:in block in cruise_lanes'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/lane_manager.rb:14:in each' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/lib/fastlane/lane_manager.rb:14:in cruise_lanes'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/bin/fastlane:31:in block (2 levels) in run' 	from /Library/Ruby/Gems/2.0.0/gems/commander-4.2.1/lib/commander/command.rb:180:in call'
from /Library/Ruby/Gems/2.0.0/gems/commander-4.2.1/lib/commander/command.rb:180:in call' 	from /Library/Ruby/Gems/2.0.0/gems/commander-4.2.1/lib/commander/command.rb:155:in run'
from /Library/Ruby/Gems/2.0.0/gems/commander-4.2.1/lib/commander/runner.rb:421:in run_active_command' 	from /Library/Ruby/Gems/2.0.0/gems/commander-4.2.1/lib/commander/runner.rb:81:in run!'
from /Library/Ruby/Gems/2.0.0/gems/commander-4.2.1/lib/commander/delegates.rb:8:in run!' 	from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/bin/fastlane:59:in run'
from /Library/Ruby/Gems/2.0.0/gems/fastlane-0.1.6/bin/fastlane:63:in <top (required)>' 	from /usr/bin/fastlane:23:in load'
from /usr/bin/fastlane:23:in `'

Could you please run
fastlane -v
and
sigh -v
You'll be able to fix this problem by running sudo gem update fastlane

Ok .. now it works .. But i had to update the sigh by hand .. updating fastlane does not trigger sigh updates . This is the default behavior ?  If so .. only call this "sudo gem update fastlane" is not enough and i'd still ned to call "sudo gem update sigh"
fastlane -v
#######################################################################
fastlane 0.1.12 is available.
It is recommended to use the latest version.
Update using '(sudo) gem update fastlane'.
To see what's new, open https://github.com/KrauseFx/fastlane/releases.
#######################################################################
INFO [2015-02-24 12:03:57.02]: Successfully loaded custom action '/Users/lioy/dev/concrete/kanui/ios/fastlane/actions/build_ipa.rb'.
INFO [2015-02-24 12:03:57.05]: Successfully loaded custom action '/Users/lioy/dev/concrete/kanui/ios/fastlane/actions/config_app.rb'.
INFO [2015-02-24 12:03:57.07]: Successfully loaded custom action '/Users/lioy/dev/concrete/kanui/ios/fastlane/actions/deploy_mobr.rb'.
fastlane 0.1.6
Thiagos-MacBook-Air-2:ios lioy$ sigh -v
#######################################################################
sigh 0.3.4 is available.
It is recommended to use the latest version.
Update using '(sudo) gem update sigh'.
To see what's new, open https://github.com/KrauseFx/sigh/releases.
#######################################################################
sigh 0.3.2
Thiagos-MacBook-Air-2:ios lioy$ sudo gem update fastlane
Password:
Updating installed gems
Updating fastlane
Fetching: cert-0.1.0.gem (100%)
Successfully installed cert-0.1.0
Fetching: fastlane-0.1.12.gem (100%)
Successfully installed fastlane-0.1.12
Fetching: aws-sdk-v1-1.63.0.gem (100%)
Successfully installed aws-sdk-v1-1.63.0
Parsing documentation for cert-0.1.0
Installing ri documentation for cert-0.1.0
Installing darkfish documentation for cert-0.1.0
Parsing documentation for fastlane-0.1.12
Installing ri documentation for fastlane-0.1.12
Installing darkfish documentation for fastlane-0.1.12
Parsing documentation for aws-sdk-v1-1.63.0
Installing ri documentation for aws-sdk-v1-1.63.0
Installing darkfish documentation for aws-sdk-v1-1.63.0
Gems updated: cert fastlane aws-sdk-v1
Thiagos-MacBook-Air-2:ios lioy$ gs
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean
Thiagos-MacBook-Air-2:ios lioy$ gem list | grep sigh
sigh (0.3.2, 0.2.1)
Thiagos-MacBook-Air-2:ios lioy$
Thiagos-MacBook-Air-2:ios lioy$ sudo gem install sigh
Password:
Successfully installed sigh-0.3.4
Parsing documentation for sigh-0.3.4
Installing ri documentation for sigh-0.3.4
1 gem installed

Thanks again for letting me know, that should be resolved with the latest version.


Add ability to run Dockerfiles product profiles without interactive questions

Currently, running wso2 product images with just default profile, asks the users if they wish to connect to the running container. It would be nice to have the ability to silent the questioning so that the test scripts can continue without having to pause for the input.
This requirement originates from Dockerfiles test framework.

Added a workaround by having the test script answer questions.


Update to support new version of jenkins-cli

Pull Request (PR) description
Update support for newer version of jenkins while also supporting the old versions
This Pull Request (PR) fixes the following issues
https://github.com/voxpupuli/puppet-jenkins/issues/944
https://guides.github.com/features/mastering-markdown/

This time if fails with something that have nothing too do with the added code. Strange, apparently it didnt support ::fqdn and i am not a fan of the solution of using Localhost instead.

Something still seems off here... when I update from a Jenkins 2.190.2 to 2.230 I get the following:
Notice: /Service[jenkins]/ensure: ensure changed 'running' to 'stopped'
Notice: /Stage[main]/Jenkins/Jenkins::Systemd[jenkins]/Transition[stop jenkins service]/enable: transition state {"ensure"=>"stopped"} applied to Service[jenkins] (corrective)
Info: Computing checksum on file /etc/init.d/jenkins
Info: /Stage[main]/Jenkins/Jenkins::Systemd[jenkins]/File[/etc/init.d/jenkins]: Filebucketed /etc/init.d/jenkins to puppet with sum dd138cf0c94c82385eeb48138112a75c
Notice: /Stage[main]/Jenkins/Jenkins::Systemd[jenkins]/File[/etc/init.d/jenkins]/ensure: removed (corrective)
Notice: /Stage[main]/Jenkins::Service/Service[jenkins]/ensure: ensure changed 'stopped' to 'running' (corrective)
Info: /Stage[main]/Jenkins::Service/Service[jenkins]: Unscheduling refresh on Service[jenkins]
Error: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]: Failed to call refresh: Could not find command 'Class[Jenkins::Service]'
Error: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]: Could not find command 'Class[Jenkins::Service]'
Notice: /Stage[main]/Jenkins::Cli/File[/usr/lib/jenkins/jenkins-cli.jar]: Dependency Exec[jenkins-cli] has failures: true
Warning: /Stage[main]/Jenkins::Cli/File[/usr/lib/jenkins/jenkins-cli.jar]: Skipping because of failed dependencies
Warning: /Stage[main]/Jenkins::Cli/Exec[safe-restart-jenkins]: Skipping because of failed dependencies
Warning: /Stage[main]/Jenkins::Cli::Reload/Exec[reload-jenkins]: Skipping because of failed dependencies
Warning: /Stage[main]/Jenkins::Cli_helper/File[/usr/lib/jenkins/puppet_helper.groovy]: Skipping because of failed dependencies
Warning: /Stage[main]/Jenkins/Anchor[jenkins::end]: Skipping because of failed dependencies
Info: Class[Jenkins]: Unscheduling all events on Class[Jenkins]
Info: Stage[main]: Unscheduling all events on Stage[main]

I think it may be a chicken and egg problem - the jar is trying to be downloaded while Jenkins is still starting up. Also, connections to the 127 address redirect to the url defined from the server.
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: --2020-04-16 12:03:55--  http://<IP_ADDRESS>/jnlpJars/jenkins-cli.jar
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: Connecting to <IP_ADDRESS>:80... connected.
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: HTTP request sent, awaiting response... 301 Moved Permanently
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: Location: https://cinext-jenkinsmaster-staging-1.delivery.puppetlabs.net/jnlpJars/jenkins-cli.jar [following]
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: --2020-04-16 12:03:55--  https://cinext-jenkinsmaster-staging-1.delivery.puppetlabs.net/jnlpJars/jenkins-cli.jar
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: Resolving cinext-jenkinsmaster-staging-1.delivery.puppetlabs.net (cinext-jenkinsmaster-staging-1.delivery.puppetlabs.net)... <IP_ADDRESS>
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: Connecting to cinext-jenkinsmaster-staging-1.delivery.puppetlabs.net (cinext-jenkinsmaster-staging-1.delivery.puppetlabs.net)|<IP_ADDRESS>|:443... connected.
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: HTTP request sent, awaiting response... 302 Moved Temporarily
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: Location: http://maintenance.puppetlabs.com/ [following]
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: --2020-04-16 12:03:55--  http://maintenance.puppetlabs.com/
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: Resolving maintenance.puppetlabs.com (maintenance.puppetlabs.com)... <IP_ADDRESS>
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: Connecting to maintenance.puppetlabs.com (maintenance.puppetlabs.com)|<IP_ADDRESS>|:80... connected.
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: HTTP request sent, awaiting response... 308 Permanent Redirect
Notice: /Stage[main]/Jenkins::Cli/Exec[jenkins-cli]/returns: 2020-04-16 12:03:56 ERROR 308: Permanent Redirect.

I only saw it now, but I guess, the line I suggested is also wrong, there is no $jenkins::port variable. Maybe the
$port = jenkins_port()
$prefix = jenkins_prefix()
need to be moved up to be available, and then used in the URL to fetch the jar.
not sure, but maybe the chicken/egg situation can be worked around with the
tries       => $cli_tries,
try_sleep   => $cli_try_sleep,

as it is done at the bottom of the cli.pp in the exec 'safe-restart-jenkins' ?
just ideas, haven't tried it.

Superseded by 189a6ecdde7c83e414155df108db17dd154b9113


URDF robot model support

This set of commits enables to use URDF robot models in DART.
Summary of changes:

install DART library with a pkg-config file to a destination directory for system-wide access to the library files
enable to provide the tracker with an external dart model instance instead of providing a file path
new method dart::readModelURDF to obtain a dart model from a URDF file
read meshes in format obj and dae (collada) and ensure that only triangle meshes are added

I used the readModelURDF function to read URDF models and thankfully, it works. I think this part written by Christian should be merged into master.


Fixes #875

What the PR does

extracts close function into variable to ensure unsubscribe function gets called when END event is emitted.
updates advanced docs for channels to show that unsubscribe function will be called

Note:

this is my first code PR for redux-saga, I'm not sure the best way to test this? I see the appropriate test code here but I'm used to using Sinon to test that a function got called and I don't see that in this repo.
thanks to @Andarist for the help! 😄

We do not have sinon dependency, and probably not worth including it now.
I would recommend to just make some let variable in the closure, so it can be set in unsubscribe. Create eventChannel, emit END there and check if that variable has changed.
Great PR and the finding!

I added 2 tests:

one to test that unsubscribe function gets called when channel is closed
another test to check that unsubscribe function gets called when END event is emitted

However, I'm having a problem with this test.
TypeError: unsubscribe is not a function

I think it's because this code is when the function is defined, and it gets called here which is before the function is defined.
Looking at the code, the subscribe function returns the unsubscribe function. And the unsubscribe function calls the close function, which tries to call the unsubscribe function. This doesn't seem possible as far as I understand about JavaScript. Perhaps we need to revamp this.

Interesting case. Probably if you do emitter(END) in next tick (i.e. setTimeout(() => emitter(END), 0) it should work as unsubscribe will be defined by then. Could u try it?
But also its quite interesting case especially in terms of something like BehaviourSubject from rxjs.
eventChannel((emitter) => {
	const unsubscribe = new Rx.BehaviorSubject('smth').subscribe(emitter);
	emitter(END);
	return unsubscribe;
});

this emitter in subscribe will get called sync for the first time as BehaviourSubject is holding its last value ('smth' in this case). Its quite a circular dependency but could probably be tackled a little bit in the lib.
Just make sure that when closing via END unsubscribe is called only if its already defined + check after returning unsubcribe if channel is closed and if it is then call immediately unsubscribe.

I tried the suggestion and changed this line to setTimeout(() => emitter(END), 0);, which meant the unsubscribe function was defined by that time. But this means the test itself needs to be wrapped in a setTimeout. Is that ok?

Ok @Andarist I've made the suggested changes. I even added a test case here for the eventChannel defined in the docs here.
The one odd part is that I'm using a setTimeout in the test since the eventChannel uses a setInterval. Is there a better way to do this in mocha?

As to more complex time dependant scenarios I've used before such test utility - https://github.com/baldwmic/redux-saga/blob/6516de097870805aaed0d6e75c64063a69d961f5/test/sagaHelpers/throttle.js#L9 to control time passing.
However here I think we could make this test independant of others and use something like
chan.take(input => {
  assert.equal(...) // should be END here
  assert.end()
})

that way we do not have to hack the global clock and still the test should run till the completion just right

@baldwmic is the PR finished on ur side? should i review it again and possibly merge in?
thanks for great work!

@Andarist i still need to finish updating the tests according to your last review. i expect to finish by the end of today. thank you for all the help in explaining things! 😄

Looks great, when I get back home I'm gonna add one more test (code mentioned here) and merge it in. Seems reasonable to have real-life like example in the tests.


Fixup broken links in authentication user guide

Fixes a couple of broken links that I came across while browsing the earthaccess documentation. Resolves https://github.com/nsidc/earthaccess/issues/778

[x] Please review our
contributing documentation
before getting started.
[x] Ensure an issue exists representing the problem being solved in this PR.
[x] Populate a descriptive title. For example, instead of "Updated README.md", use a
title such as "Add testing details to the contributor section of the README".
[x] Populate the body of the pull request with:

A clear description of the change you are proposing.
Links to any issues resolved by this PR with text in the PR description, for
example closes #1. See
GitHub docs - Linking a pull request to an issue.

[ ] Update CHANGELOG.md with details about your change in a section titled
## Unreleased. If such a section does not exist, please create one. Follow
Common Changelog for your additions.
[x] Update the documentation and/or the README.md with details of changes to the
earthaccess interface, if any. Consider new environment variables, function names,
decorators, etc.

Click the "Ready for review" button at the bottom of the "Conversation" tab in GitHub
once these requirements are fulfilled. Don't worry if you see any test failures in
GitHub at this point!

Pull Request (PR) merge checklist - click to expand
Please do your best to complete these requirements! If you need help with any of these
requirements, you can ping the @nsidc/earthaccess-support team in a comment and we
will help you out!

[x] Add unit tests for any new features.
[x] Apply formatting and linting autofixes. You can add a GitHub comment in this Pull
Request containing "pre-commit.ci autofix" to automate this.
[x] Ensure all automated PR checks (seen at the bottom of the "conversation" tab) pass.
[ ] Get at least one approving review.

📚 Documentation preview 📚: https://earthaccess--777.org.readthedocs.build/en/777/

I'm noticing now that the other "user guide" pages also have this problem.
I also just noticed another broken link on this page: https://earthaccess.readthedocs.io/en/latest/howto/search-granules/ ("Introducing NASA earthaccess" tutorial link).

I'm noticing now that the other "user guide" pages also have this problem.

I updated those pages w/ new commits to this PR.

I also just noticed another broken link on this page: https://earthaccess.readthedocs.io/en/latest/howto/search-granules/ ("Introducing NASA earthaccess" tutorial link).

I'm not sure what this is supposed to link to. The URL is https://nsidc.github.io/earthaccess/tutorials/demo/#querying-for-datasets, but I do not see any "demo" directory under tutorials.

I wonder why our build process didn't catch these broken links. It caught others! :thinking:
Anyway, congrats on opening lucky PR #777 :partying_face:


user12986714: Watch vyavsay.com

user12986714 requests the watch of the watch_keyword vyavsay\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

vyavsay\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


AOT Fail

I use couple of RXJS operators, I have added them in gulpfile.js
      // A list of IDs of modules that should remain external to the bundle
      // See https://github.com/rollup/rollup/wiki/JavaScript-API#external
      external: [
        '@angular/core',
        '@angular/common',
        '@angular/http',
        'rxjs/Observable',
        'rxjs/Subject',
        'rxjs/add/observable/timer',
        'rxjs/add/operator/do',
        'rxjs/add/operator/takeWhile',
        'rxjs/add/operator/switchMap'
      ],

gulp build output:
[02:29:00] Finished 'ngc' after 1.41 s
[02:29:00] Starting 'rollup:fesm'...
build/services/browser-xhr.provider.js (1:17) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten
build/services/browser-xhr.provider.js (1:25) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten
[02:29:00] Finished 'rollup:fesm' after 159 ms
[02:29:00] Starting 'rollup:umd'...
build/services/browser-xhr.provider.js (1:17) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten
build/services/browser-xhr.provider.js (1:25) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten
No name was provided for external module '@angular/core' in options.globals – guessing '_angular_core'
No name was provided for external module '@angular/common' in options.globals – guessing '_angular_comm
on'
No name was provided for external module 'rxjs/Subject' in options.globals – guessing 'rxjs_Subject'
No name was provided for external module 'rxjs/Observable' in options.globals – guessing 'rxjs_Observab
le'
No name was provided for external module '@angular/http' in options.globals – guessing '_angular_http'
[02:29:00] Finished 'rollup:umd' after 64 ms

I don't understand why I am getting these errors!
Now when I import the dist package in a new app, I get no errors while compiling, but it is broken at runtime, getting these errors:

@MurhafSousli — Are you importing the rxjs operators in your library code?
import 'rxjs/add/observable/timer';
import 'rxjs/add/operator/do';
import 'rxjs/add/operator/takeWhile';
import 'rxjs/add/operator/switchMap';

@jvandemo I already have these imports in the library.
Another thing I noticed is in these errors:
build/services/browser-xhr.provider.js (1:17) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten
build/services/browser-xhr.provider.js (1:25) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten
[02:29:00] Finished 'rollup:fesm' after 159 ms
[02:29:00] Starting 'rollup:umd'...
build/services/browser-xhr.provider.js (1:17) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten
build/services/browser-xhr.provider.js (1:25) The 'this' keyword is equivalent to 'undefined' at the to
p level of an ES module, and has been rewritten

I think there is an issue with rollup and super() keyword
@Injectable()
export class NgProgressBrowserXhr extends BrowserXhr {

  private currentRequest: number = 0;

  constructor(private service: NgProgressService) {
    super();
  }

  public build() {
    const xhr = super.build();
    .....
  }
}

@MurhafSousli — Is it possible to share your code? Thanks!

There are two rollup modes (umd and fesm). I resolved the problem declaring externals in both. Did you put them in both ?

The gulp file has two rollup modes/tasks (umd and fesm), did you put the same externals in both modes ?

This fixed it for me...
I was having the same problem... I also noticed some WARNINGS when building (see #149).
I had no idea what to do so I tried fixing those warnings first, I read somewhere that I should try exporting every component, model, service, etc that is being used by the module and its components in src/index.ts file (I apologise, I lost the reference to that comment).
Anyway, fixing those warnings also fixed r is not a constructor issue for me.
Hope this helps!

@sebastianteres probably do you refer to export all lambdas ? https://medium.com/@isaacplmann/making-your-angular-2-library-statically-analyzable-for-aot-e1c6f3ebedd5

@MurhafSousli Can you please apply @sebastianteres's recommendations and see if that works for you? Thanks!

@diego-d5000 I finally found the reference! It was buried in my search history from yesterday 😄
See #126
To give you an example:
My library has 3 components:

Component Foo

Component Foo!

Component Bar

Component Bar!

Component FooBar

<foo></foo>
<bar></bar>

Library module exports component Component FooBar
So my src/index.ts looks like this:
export * from './foo-bar/foo-bar.component';

I believe this will cause the Warnings mentioned in #149 and #126.
So to fix it you would have to edit src/index.ts as follows:
export * from './foo/foo.component';
export * from './bar/bar.component';
export * from './foo-bar/foo-bar.component';

To be honest I didn't know if this was necessary to begin with but I assumed I could only export what I wanted the library users to use and not to have to export everything. In this case I think is a valid scenario to only want to export fooBar and not foo or bar individually.
To be even more honest I'm not sure what I should be exporting or if I could leave anything out from src/index.tsbut what I did was add an export for every component, service or class my library uses internally.
Be that as it may, this fixed both the Warnings and the Runtime error for me.

@sebastianteres — Thank you so much for sharing, much appreciated! 👍

Hi,
I've got the same problem. When using inheritance inside my library. For example I have a BaseComponent all other components like for example EmailComponent inherits from it.
When I dont use inheritance (which leads to a lot of boilerplate code :() the warnings in the console are gone.
Any idea on how to fix this?

Just posted the same issue here:
https://github.com/jvandemo/generator-angular2-library/issues/229
I followed the steps from the docs, I didn't change a word....


remove " from echo when setup up token

Invalid token after all commands
In file .env is "TOKEN=MY_BOT_TOKEN". When i remove " all is ok.

I cannot reproduce your issue. What OS/Shell are you using?


doc: fix link in contribute.html

@bradfitz I made this another PR with appropriate change for #26611 , does this looks good to you ?


How control jetson-utils log/trace output in python

Hi, probably it's somethng trivial, but I'm stuck there...
is there a way in python to control/disable tracing output from jetson.utils functions?
Example:

whenever I call
"img = jetson.utils.loadImage('somefile')"
then on the console output I see
"[image]  loaded '.somefile'  (1280x720, 3 channels)"
If I have to load nK inages is going boring...

Tryed to catch the  python logging.lastResort setting logLevel to logging.CRITICAL but seems not to work
Any help would be apreciated!
Flavio

Hi @fmolinelli, normally this is done with one of the logging command-line arguments:
logging arguments:
  --log-file=FILE        output destination file (default is stdout)
  --log-level=LEVEL      message output threshold, one of the following:
                             * silent
                             * error
                             * warning
                             * success
                             * info
                             * verbose (default)
                             * debug
  --verbose              enable verbose logging (same as --log-level=verbose)
  --debug                enable debug logging   (same as --log-level=debug)

However this relies you on passing the command-line to a class like videoSource/videoOutput/imageNet/ect, where they get parsed and the logging level set.   My guess is that is not happening in your code.
For now, what I would do is change the default logging level here:
https://github.com/dusty-nv/jetson-utils/blob/7de695d94271ba1ae77a4358f427c0887b09084f/logging.h#L70
And then re-build/re-install with:
$ cd jetson-inference/build
$ cmake ../
$ make -j$(nproc)
$ sudo make install

Hi @dusty-nv ,
just awesome! you gave me a lot of hints!
Solved the problem, going to dig more in depth with jetson-inference: just started yesterday ;)
closing the issue...


Update NodeLatencyMonitor unit tests

Add missing crdInformerFactory.Start call for TestNodeAddUpdateDelete to avoid cache sync error logs.
Add extra logging for TestUpdateMonitorPingInterval, which has been failing infrequently in CI. I cannot figure out the reason for the failures, nor have I been able to reproduce it locally, so adding some logs may help.

/skip-all


SqsListener String index out of bounds issue

I'm encountering a problem when trying to use the @SQSListener annotation from the Spring Cloud module. My Listener method is defined as follows:
@SqsListener(value = "myproject-dev-au-error-queue")
public void listenPhoenix(String message) throws IOException {

    logger.info(message);
 }

And when I start receiving messages from the queue i get the following stack trace:
Exception in thread "simpleMessageListenerContainer-4" Exception in thread "simpleMessageListenerContainer-6" Exception in thread "simpleMessageListenerContainer-9" Exception in thread "simpleMessageListenerContainer-10" java.lang.StringIndexOutOfBoundsException: String index out of range: -1
    at java.lang.String.substring(String.java:1931)
    at org.springframework.cloud.aws.messaging.core.QueueMessageUtils.getNumberValue(QueueMessageUtils.java:93)
    at org.springframework.cloud.aws.messaging.core.QueueMessageUtils.getMessageAttributesAsMessageHeaders(QueueMessageUtils.java:80)
    at org.springframework.cloud.aws.messaging.core.QueueMessageUtils.createMessage(QueueMessageUtils.java:56)
    at org.springframework.cloud.aws.messaging.listener.SimpleMessageListenerContainer$MessageExecutor.getMessageForExecution(SimpleMessageListenerContainer.java:375)
    at org.springframework.cloud.aws.messaging.listener.SimpleMessageListenerContainer$MessageExecutor.run(SimpleMessageListenerContainer.java:336)
    at org.springframework.cloud.aws.messaging.listener.SimpleMessageListenerContainer$SignalExecutingRunnable.run(SimpleMessageListenerContainer.java:392)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    at java.lang.Thread.run(Thread.java:745)

With the problem being in QueueMessageUtils.getNumberValue(...), more specifically
private static Object getNumberValue(MessageAttributeValue value) {
    String numberType = value.getDataType().substring("Number".length() + 1);
    ....
}

This is happening on all versions of Spring Cloud (Dalston, Finchley, Edgeware, etc.)
Any ideas on what might be causing this behaviour?

Also having this issue. Seems the issue is here:
https://github.com/spring-cloud/spring-cloud-aws/blob/64ec6d15c67e98c3372ab88c9e5b31d5a89a68f8/spring-cloud-aws-messaging/src/main/java/org/springframework/cloud/aws/messaging/core/QueueMessageUtils.java#L92-L101
spring-cloud-aws expects numbers in SQS message attributes to have a typename of Number.${JAVA_TYPE_NAME}, e.g., Number.java.lang.Integer. This is definitely not part of the AWS SQS interface, and is a bug and should be fixed.

Hi I'm having this issue, so i go and download to my project Finchley.SR3 which is on december, i thought that version was going to have this fix, in which version of maven dependency i can find this fix you're applying here ?

Any update on this? We are closing in on a year since first reported

Any progress on this issue? It would be good to fix it

Any update on this bug? There's still the same issue in the latest version.

Hi @DPetrunov @youngkjun FedorRomanov . I can not reproduce the bug with spring-boot-starter-parent:2.3.1.RELEASE and spring-cloud-dependencies:Hoxton.SR5. Do you still encounter this issue?

@mkatircioglu , Please take a look at @kevinhwang's comment

@DPetrunov the issue is fixed and will be released in 2.3.

@maciejwalkowiak Any idea when 2.3 will be released?

I can't tell exact date as there are quite a few factors related to this release - like changing Maven coordinates - but I can tell for sure that it's not going to happen in next 2-3 weeks.

@maciejwalkowiak is it visible now when it will be released? Is it possible to fastrack that as 2.2.4?

I can confirm to have the same issue in spring-cloud-aws-messaging-2.2.4.RELEASE.jar which ships with Hoxton.SR8

It seems the problem have been fixed on master: https://github.com/spring-cloud/spring-cloud-aws/blob/master/spring-cloud-aws-messaging/src/main/java/org/springframework/cloud/aws/messaging/core/QueueMessageUtils.java. When will this version be released?

Spring Cloud AWS 2.3 has been released in March https://spring.io/blog/2021/03/17/spring-cloud-aws-2-3-is-now-available


Revert "voidtools.Everything version <IP_ADDRESS>5 (#14381)"

This reverts commit 229c8175772ef96968872719c4329eb60c1f49a1.

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?
[X] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@arjun-msft here we go again

/azp run


TopicBuilder Improvements

add assignReplicas()

Any thoughts about failed build on Travis?
I wonder if we have everything correctly in Gradle config in regards to JUnit 5...
Otherwise LGTM


JSON.parse pushed down to request

Added parsing response and errors via JSON.parse to _request method. Now it is easier to handle responses from bot methods.
CI pipeline may not pass because the environment variables are not set, e.g. Error: Bot token not provided. Tests successfully pass on my machine.

Closed by an accident 🤷♂


Failed to install on Alpine linux

Bicep version
latest
Describe the bug
Hello! were trying to add support for bicep in the Github Super-Linter
I have added the lines of code
curl -Lo bicep.bin https://github.com/Azure/bicep/releases/latest/download/bicep-linux-x64 \
    && chmod +x ./bicep.bin \
    && mv ./bicep.bin /usr/local/bin/bicep \

and it appears I have zlib installed, but when I try to run bicep --help I'm always met with the same error...
I have validated in /lib the existence of libz.so.1 but it seems like not pathed properly?
I have also looked and I can see that by default /lib should be pathed for libraries, so not sure the disconnect.
Any help is greatly appreciated!
To Reproduce
Steps to reproduce the behavior:

Clone the Dockerfile from the github/super-linter
Add the lines of code to install bicep
build the container

docker build -t github/super-linter:bicep -f Dockerfile .

run the container

docker run --it --entrypoint /bin/bash github/super-linter:bicep

run command

bicep --help

Additional context

Error message

bash-5.1# bicep --help
bicep: error while loading shared libraries: libz.so.1: cannot open shared object file: No such file or directory

@shenglol - I know you looked at a lot of the various linux install nuances for ensuring docker support worked as expected. Can you take a look at this one?

Actually, we have already fixed this. For a lightweight linux distro like alpine, you should use the bicep-linux-musl-x64 asset in the release instead:

On our end we should probably update the install instructions/scripts to mention this as well.

On our end we should probably update the install instructions/scripts to mention this as well.

@stephaniezyen / @mumian / @tfitzmac - can we update the docs to detail that lightweight distros should install bicep-linux-musl-x64 instead of bicep-linux-x64?

@alex-frankel, @stephaniezyen, you can assign it to me.

I will add the following note in the Install manually/Linux section:

[!NOTE]
For lightweight Linux distributions like Alpine, use bicep-linux-musl-x64 instead of bicep-linux-x64 in the preceding script.

A PR is created - https://github.com/MicrosoftDocs/azure-docs-pr/pull/161215


Make bootstrap client secure again

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

LGTM


Maybe better to have tf.tidy() around tensor operations?

The following lines involves tensor operations, including creating temporary ones.
https://github.com/maru-labo/doodle/blob/2f6b3143b02d86b965767cd9d6380fa7ce5a2e85/examples/tensorflow_js/src/cmps/doodle.vue#L74-L80
Checking google's example code like the following, tf.tidy() is used around such operations:
https://github.com/tensorflow/tfjs-examples/blob/bf51024bd04fa6197b410db10e5210d4604aef43/mnist/index.js#L114-L122
I'm wondering whether it is needed for this case, although I realize that the tensor that hold the result is explicitly disposed, so there maybe no need for tf.tidy().
https://github.com/maru-labo/doodle/blob/2f6b3143b02d86b965767cd9d6380fa7ce5a2e85/examples/tensorflow_js/src/cmps/doodle.vue#L70
Still, it might be easier for beginners to read the code as a reference if tf.tidy() is used instead of the explicit dispose().
I'm creating this issue, just in case.

Thank you! Fix to use tf.tidy(). 8c91e6c


Introduce generating configuration files from templates feature

Currently, the following files are installed into RPM:

/etc/td-agent
/usr/sbin

Generating configuration files are also based on treasure-data/omnibus-td-agent@9d86d39.


Issue Number: # 307 Added icons

Successfully added icons on the navbar in all the sections
before

after

@khushi-joshi-05  kindly review the PR and also mention the label( 1/2/3)  on the issue created

The menu is not working in this PR @khushi-joshi-05 will confirm this PR.

@sunny0625  please update the status , as i checked it and it is working in my pc


Popup not correct

When imagepopup shows the like button is out of the box.

Thanks for reporting the issue, I will update status as soon as I manage to fix this layout issue.

Hello, I'm still struggling to setup the CSS for this problem. Advanced CSS is not really my stronger side :). If you manage somehow to make it look properly, please share.
In case generated HTML structure needs to be updated to resolve this, please let me know so I can update it.
Thank in advance for help


Hello, this is the research area I am currently working on. Do you have any datasets that you can provide me for learning? Thank you very much.

Hello, this is the research area I am currently working on. Do you have any datasets that you can provide me for learning? Thank you very much.

Hello. There are several public databases that you could use such as ADNI, AIBL, and OASIS, etc.
ADNI and AIBL:
https://ida.loni.usc.edu/login.jsp
OASIS:
https://www.oasis-brains.org/


Please include release information for unified solution Installation Package(md4ws.msi).

[Enter feedback here]
The unified solution Installation Package(md4ws.msi) for Windows Server 2012 R2/ 2016  is updated on an irregular basis.
However, there is no site to check for fixed content and the latest version.
Since there is currently no way to check, could you please include a description of the new version as it is released?

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

ID: 8c5eb6e9-4010-6077-4746-ec9f93148f9e
Version Independent ID: 8c5eb6e9-4010-6077-4746-ec9f93148f9e
Content: What's new in Microsoft Defender for Endpoint
Content Source: microsoft-365/security/defender-endpoint/whats-new-in-microsoft-defender-endpoint.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

The whats new pages underwent recent changes which include version numbers etc,


update plutus

Update Plutus to current master.

Hang on a sec, might want to sneak one more thing in.


Question: How I can verify jwt token at the backend side?

Prerequisites

[X] I have searched the repository’s issues and the Kinde community to ensure my documentation issue isn’t a duplicate
[X] I have read the contributing guidelines
[X] I agree to the terms within the code of conduct

How can we improve the docs or what is missing?
About secret salt
Provide extra context, such as what you were trying to do and your requirements
Hi I found in documentation that:

But I have no way to get the secret salt, how can I verify jwt token from my backend API?
Thank you very much
Linh
If the docs page already exists, please provide a link
No response

Hey @hdlinh1808 I'm not entirely sure what you mean by secret salt, but we do have a doc on verifying JWTs
if you're looking for your JWKS file you can find this at https://<your_subdomain>.kinde.com/.well-known/jwks
In terms of how you verify the JWT this will depend on what technology are you using on your backend.
Here is a rough example in NodeJS but you can do it in whatever technology your API is
import jwksClient from "jwks-rsa";
import jwt from "jsonwebtoken";

const client = jwksClient({
  jwksUri: `https://<your_subdomain>.kinde.com/.well-known/jwks`,
});

export async function checkJWT(req) {
  try {
    const authHeader = req.headers.authorization;
    const token = authHeader && authHeader.split(' ')[1];

    //verify token
    const { header } = jwt.decode(token, { complete: true });
    const { kid } = header;
    const key = await client.getSigningKey(kid);
    const signingKey = key.getPublicKey();

    const payload = await jwt.verify(token, signingKey);
    const { data } = payload;

    // Do something with payload here
    console.log({ data });
  } catch (err) {
    if (err instanceof Error) {
      console.error(err.message);
    }
  }
}


Python debugging doesn't work in ColabCode (via ngrok and Google Colab)

I'm using amazing ColabCode by Abhishek Thakur. It uses ngrok as backbone and Google Colab as "frontend". But when I try to debug Python script, it doesn't launch any debug action. launch.json is empty. When I fill it with next:
"version": "0.2.0",
"configurations": [
{
"name": "Python: Current File",
"type": "python",
"request": "launch",
"program": "${file}",
"console": "integratedTerminal"
}
]
I've got next error:
Couldn't find a debug adapter descriptor for debug type 'python' (extension might have failed to activate). Python extension installed properly. Script runs in terminal only (not by F5 button).
How can I debug Python script on ngrok code-server?

HI @krivenkoz ! Can you please fill out the issue template?
And can you provide reproduction steps? I'm not super familiar with ColabCode.
And lastly, have you tried debugging a Python script when not using ColabCode? Wondering if there may be an underlying issue.

I'm seeing the same thing just with code-server 3.9.1 and the latest ms-python extension installed.
creating test.py simply
print('hello world')

and creating launch.json as
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python: Current File",
            "type": "python",
            "request": "launch",
            "program": "${file}",
            "console": "integratedTerminal"
        }
    ]
}

Press F5 in  test.py and it fails with the above error message.  Worse, I don't see any log output anywhere explaining the issue.  Please help.

Hey @ddurham2 - thanks for letting us know! Just to clarify, I should be able to reproduce following these steps?

install latest version of code-server 3.9.1
install the latest ms-python extension
create test.py
crate launch.json (using what you have above)
hit F5 in test.py

Expected
I don't use Python much these days and also haven't tried debugging it in VS Code/code-server so I'm not sure what should happen?
Actual
Couldn't find a debug adapter descriptor for debug type 'python' (extension might have failed to activate).
Python extension installed properly. Script runs in terminal only (not by F5 button).

Possibly related to:

https://github.com/cdr/code-server/issues/2341

Yes, precisely.  And I think #2341 is what I should focus on, and possibly this issue's fundamental cause too.
I have learned since that if you look in the browser dev console that this behavior is caused by the ms-toolsai.jupyter failing to load.  That seems to be because it's missing some files in the extension ( /home/coder/.local/share/code-server/extensions/ms-toolsai.jupyter-2021.2.0/out/client/* )
I think #2341 already has discovered this.
Also, I've found that<EMAIL_ADDRESS>is the version just before they started requiring the ms-toolsai.jupyter extension.  I'm currently having to stick with code-server v3.7.1 (due to #2626 )  and specify this version of the ms-python plugin.

Awesome! Thanks for the additional info. I've added this to the next milestone so hopefully we can figure this out!

@krivenkoz The latest python extension is now working in 3.9.1.  You might check if your issue is resolved.


Length edge

Hello,
The edge length is the distance between startNode center and endNode center ? Or between startNodeDeviationXY and endNodeDeviationXY ?

Between the center of the nodes: The allowedDeviationXY is
a) optional (and the presence of an optional attribute shall not change the semantic of another attribute)
b) only an additional requirement which shall be meet, when a node is reported as passwd by the vehicle.


Handle bundles in aggregator

What is this PR doing?
This updates the client and aggregator code to use the new bundle/operation format in VerficationGateway.processBundle to work with the latest bls-wallet contracts.
How can these changes be manually tested?

Start the aggregator ./programs/aggregator.
Run tests/premerge script ./programs/premerge.

Does this PR resolve or contribute to any issues?
Resolves https://github.com/jzaki/bls-wallet/issues/35
Checklist

[x] I have manually tested these changes
[x] Post a link to the PR in the group chat

Guidelines

If your PR is not ready, mark it as a draft
The resolve conversation button is for reviewers, not authors

(But add a 'done' comment or similar)

Ready for final review, merge at will.


model binding IFromFile

I have sent details of this issue in #12223, but my problem is not resolved yet.
thanks.

Thanks for contacting us, @bahmansh.
Please provide details about what you are trying to do and what is failing for you. Please also provide a minimalistic repro (ideally as a GitHub repo).

@bahmansh also, did you look into @pranavkm 's response in the referenced issue? Didn't that resolve your concerns?

Hi. We're closing this issue as we have heard no response from you for some time now. If you have more details and are encountering this issue please add a new reply and re-open the issue.


Update LibreWolf.LibreWolf to v118.0.2-1

(This pull-request was auto-generated, ping @maltejur)
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Domain


Add code to compute the base reward rate for epoch e in the last block of epoch e-1 (this ensures that the reward rate for the current epoch is always available)

Closed in #331


[Voice SDK] playEffect not work playing specified audio files on Android Emulator in v2.8.1 and higher versions.

The IAudioEffectManager#playEffect method not work playing specified audio files on Android Emulator (Android 9) in Voice SDK v2.8.1, v2.8.2 and v2.9.0.
A default sound play instead.
But this method work playing specified audio files on Android Emulator (Android 9) in Voice SDK v2.8.0.
Please help it.

Can you give an example?  And are they of certain audio format, or files by link?

@brianxcli  I've create example project.
https://github.com/kkoudev/AgoraTest
Please set the App ID of Agora project in MainActivity.kt If you want to launch this project.
And The Agora version is set in app/build.gradle.
Please confirm it.

Android simulator only supports mp3 format, AFAIK

@yuhaonong
As with m4a format, mp3 files could not be played using the playEffect method on Android Emulator in v2.8.1 and higher versions.
But it have be able to play using the playEffect method on Android Emulator in v2.8.0.
I've added a mp3 file to this example project.
https://github.com/kkoudev/AgoraTest
Please confirm it.

Sorry, forgot to mention that the mp3 file has to be in sdcard for emulator. and make sure you have grant the storage permission.

Ummm....
I want to know why audio files in assets directory can no longer be played on Android Emulator since v2.8.1.
Because, it can be played on Android Emulator in v2.8.0 and lower versions.
This issue is bug or specification change of agora since v2.8.1?
And do you know how to solve this problem?

There was some issue in detecting emulator before v2.8.1 since some api was deprecated in Android-O onward. The requirement for audio files in SD card is probably because of some historical compatibility issue.
As a workaround for emulator, you can simply copy the asset file to sdcard.

@yuhaonong
Thank you for your reply.
I've tried to play a mp3 file in the sdcard with playEffect method and it works fine.
Thanks!


Can't override title search boost value for elasticsearch

Issue Summary
I have created a page model with search_fields with SearchField and AutoCompleteField with different boost value. When index is created and looking at the index settings in Elasticsearch, title still uses a boost value of 2.
Steps to Reproduce

Create Page model with custom search_fields with different boost value for title.
update index
Look at Elasticsearch index and see that boost value 2 is still used

Technical details

Python version: 3.9.19.
Django version: 4.2.11
Wagtail version: 5.2.4

Working on this
This happens because index settings are applied for every model. The order of the models applied is based on installed apps: https://github.com/wagtail/wagtail/blob/main/wagtail/search/index.py#L130. Now the last model will win and override the settings. And installed apps are mostly in order of own apps first and then the rest. Wagtail core Page model will always override the index settings.
I think to emulate how Django templates works, we can give get_indexed_models in reversed so that top apps are the last one to apply settings.

Thanks for the report @krukas!

Look at Elasticsearch index and see that boost value 2 is still used

Can you give more details about the exact thing you're looking at here, please? If I remember correctly, the Elasticsearch backend creates separate indexes for the base wagtailcore.Page model and the specific page model, and boosts are applied at querying time, so it may well be that BlogPage.objects.search("some term") respects the custom boost value on title but Page.objects.search("some term") does not.
Also, please can you test this against the latest Wagtail version and confirm whether the issue still exists? There were some significant changes to the boosting logic in Wagtail 6.0, and these may have already fixed this.

@gasman I'm looking in the index settings in Elasticsearch at the copy_to for title, this remains _all_text_boost_2_0. Also I have only on index wagtailcore_page that has all the fields from all page models in the format <app>_<model>__<field>
All pages extends from a abstract BasePage that extends from wagtail.models.Page

I have tested with 6.0.3 and have the same results.

Thanks for clarifying! I see what's going on here now - the indexing code is generating a mapping configuration for every subclass of Page in turn, and pushing that to the wagtail__wagtailcore_page index, and apparently Elasticsearch handles that by merging it into the previous configuration.
That works fine if the definitions for the core fields such as title are identical across all subclasses - the mappings for those fields will be unchanged by the merge, and it will just add the fields from the specific model, appropriately namespaced (e.g. blog_blogpage__body). However, if the core fields are modified (e.g. by adding a custom boost value on title), the model which is pushed last will "win". That's fine if the modified definition is part of a common BasePage shared by all classes, but it has unwanted consequences in other cases - for example, if the last model in the list is BlogPage with SearchField("title", boost=10), then all titles will be boosted by 10, not just BlogPage instances.
I can't see a good way to solve this - we would need to configure the index in such a way that the title field is copied to _all_text_boost_10_0 when pushing a BlogPage, and copied to _all_text_boost_2_0 for all other page types. We could do that by avoiding Elasticsearch's copy_to feature entirely, and doing the copying on the Python side instead (so that Elasticsearch is passed a JSON document with the appropriate _all_text_boost_... field filled in) - however, that wouldn't handle searches with explicit fields, e.g. Page.objects.search("foo", fields=["title"]), which don't use the _all_text fields.

My solution would indeed not work if you have multiple boost values for the same field name. I Think the move from copy_to to python side is probably the best solution. I don't know the whole working of Wagtail search, if we fill all the _all_text fields in python instead of the copy_to. Nothing has changed on the available index fields in Elasticsearch and everything should still work the same?
For us all the page title should have the same boost value, for now I use this hack/fix to change it:
# In Django apps ready function
from wagtail.models import Page
Page.search_fields[0].boost = 4  # SearchField
Page.search_fields[1].boost = 4  # AutocompleteField


New Feature: weather lookup

depends on location set by user
there is another bot that has this feature and its pretty nice! its only thing this bot is missing that i could think of!
☀ Weather for Sundsvall, SE
2°C / 36°F, 'clear sky'
0% Clouds, Windspeed 5.7m/s
Barometric pressure: 1003hpa @80% humidity
@NAME

Nadeko already has weather. "~we city country/countrycode"


Fix dynamic alloc

Refactors the allocation of Q to ensure the correct size for iterating over the histogram size.
Also fixes an accumulation bug where Q was not previously accounting for the time delayed contributions.
Removes hard coded limits and issues warnings when users supply values that may violate original model assumptions.
A few code cleanup/refactors for simplifying and optmization loops.
A few other small bug fixes and comments added.
Note this PR is dependent on #37.  It is branched from this commit on the PR branch: cfbdec5258dfa7174e50b1aa9efefb9f482b06b0

Given that #37 is a big functional change that's still in draft mode, would it make sense to

Target this PR against the branch for #37 rather than master, and/or
separate out commits of bug fixes that are applicable directly to master, and put them up for review and integration separately?

Incidentally, the test/main_unit_test_bmi.c here is a decent illustration of the things that I want to capture in a BMI module usage/conventions checking utility. So, I'm glad I came to look.

As far as merge/integration is concerned, I'm happy to rebase this again and reorganize the commits as appropriate/needed. I'm not sure a PR against the feature branch of #37 would help significantly, based on discussions with @Ben-Choat I think that work is nearly ready to merge, and that might complicate things. As long as this doesn't go stale, I can rebase on master once the feature is complete.


Disable user-submitted python code by default

Because user-submitted python code is a security hole big enough to fly a 747 through, we should default to not allowing it and include a server-side flag to enable it.
Don't forget to document.

I'm thinking that this should not prevent people from running existing transforms that include python code, but should prevent them from creating new transforms or editing (the code of?) existing transforms that include python code.


Grid example was using incorrect objects

fixes #106

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

demo/src/main/java/com/vaadin/polymer/demo/client/sampler/vaadin/VaadinGridSample.java, line 84 at r1 (raw file):

                // Reuse elements in the cell to improve performance
                HTMLElement e = (HTMLElement)c.getElement();
                IronIconElement elem = (IronIconElement)e.getFirstChild();

The names e and elem are confusingly similar, despite they are different elements and types.
Maybe rename elem to icon or iconEl to reduce confusion here?

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

demo/src/main/java/com/vaadin/polymer/demo/client/sampler/vaadin/VaadinGridSample.java, line 84 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
> The names `e` and `elem` are confusingly similar, despite they are different elements and types.
>
> Maybe rename `elem` to `icon` or `iconEl` to reduce confusion here?

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


sol::table:size()

Hi
I am very new to github so please forgive my noobness: I started using Sol and managed to integrate it (it's very easy, no problem here) in my test program. I tried the following: load a lua file which has a generateDungeon function which returns a 4096 elements table. Then in C++ I do the following:
sol::function dungeonFunc = luaVM.get<sol::function>("generateDungeon");
sol::table dungeon = dungeonFunc.call<sol::table>();

Everything works flawlessly but when I want to read all the values from table:
for(std::size_t i = 1; i <= dungeon.size(); ++ i)
{
	const auto value = dungeon.get<int>(i);

	// etc...
}

loop finishes but I have seg fault errors in random places. I tested commenting here and there and my conclusion points to sol::table::size(). I took dungeon.size() call out from for loop:
auto dungeonSize = dungeon.size();
for(std::size_t i = 1; i <= dungeonSize; ++ i)

With this change everything works. I checked sol::table:size() and we have the following:
size_t size() const {
    push();
    return lua_rawlen(state(), -1);
}

With my total lack of knowledge of how sol and lua work, I would say that push() makes some stack get bigger in each call to size() and finally makes my app crash. Is it possible this to be a bug?
Thanks for your library by the way ;)
Regards

It's a bug that's been fixed here: https://github.com/Rapptz/sol/pull/61#discussion_r30457613 . If you want the fix, pull from the develop branch of sol.
Note that in the develop branch, specifically to get around this problem, we created table.for_each function for your use. Please note that the iteration of lua is not guaranteed to get in numeric order, however.
Also, be careful: the raw length is the number of consecutive integer/number keys. If a table happens to have string keys, that size will only count up to that first string key, and then you'll stop before hitting the rest of the table's keys and values. Finally, if you know for CERTAIN that you'll only have integers for keys, then your table should be just fine; this is just a heads up! Happy coding.

One final note: the develop branch deprecates userdata as a name and has usertype instead, so if you're using userdata<T>, you may have to go through and change them to usertype<T>.

Thanks for this quick and extensive response ;) My fault not looking at develop branch, only in the master one.


Image grayscale

When I run the script:
cargo run mono D:\University\skoltex\image\right.tar D:\University\skoltex\img -f png

Images were saved in grayscale. I want to save images in RGB.  What flag should I use? or how сan I fix this problem? (OS windows 10 )

Add -d flag, which will apply bi-linear demosaicing. You can see documentation for other options using -h flag.
BTW you can download pre-compiled version from the releases page.

When I added the flag -d, I got an error :

I use the last version

I meet the same problem. Have you solved it yet?

When I added the flag -d, I got an error :  I use the last version


[core-util] delay() with rejection

https://github.com/Azure/azure-sdk-for-js/pull/25279#discussion_r1140660605
Often we have scenarios where we wait for a period of time then throw/reject with an error.  It would be useful to have something like
  delay(retryTimeoutInMs, { abortSignal, rejectWith: { name: "OperationTimeoutError", message: "..." }})

or a more specific wrapper? like
  timedOut(retryTimeoutInMs, errorToThrow, { abortSignal })

Working on another Service Bus PR I found now that I want to resolve when abort signal is fired, that is, abortable promise that does not reject.  So maybe extending createAbortablePromise to allow overriding onAbort/onTimeout etc.? @deyaaeldeen

I think there are a few asks in this thread that I would like to comment on:

have delay reject with a given value when it completes: I think chaining a promise that rejects is simple enough, e.g. delay(timeoutInMs).then(() => throw new Error(...)). I am not against adding an option per se but I would like to learn more about the use case to make a more informed opinion.
replace the AbortError with a custom error in delay: As @mpodwysocki mentioned, chaining a promise that converts the error is a simple solution, e.g. delay(timeoutInMs).catch(() => throw new Error(...)). Notice that I didn't check what is the type of error thrown by delay because it is a thin wrapper around setTimeout and AFAIK, it doesn't throw errors so the only potential thrown error is the abort one. But again, I am not against adding an option here in case there is a good use case for it.
resolve instead of throwing AbortError in createAbortablePromise: This one is interesting, again one solution is to eat up the abort error but there is a question about the unnecessary overhead of throwing only to catch the error and resolve. @jeremymeng suggested adding an option to override the behavior on the abort event. I wonder what does other folks think about this one? /cc @xirzec @mpodwysocki

I think there are a few asks in this thread that I would like to comment on:

have delay reject with a given value when it completes: I think chaining a promise that rejects is simple enough, e.g. delay(timeoutInMs).then(() => throw new Error(...)). I am not against adding an option per se but I would like to learn more about the use case to make a more informed opinion.
replace the AbortError with a custom error in delay: As @mpodwysocki mentioned, chaining a promise that converts the error is a simple solution, e.g. delay(timeoutInMs).catch(() => throw new Error(...)). Notice that I didn't check what is the type of error thrown by delay because it is a thin wrapper around setTimeout and AFAIK, it doesn't throw errors so the only potential thrown error is the abort one. But again, I am not against adding an option here in case there is a good use case for it.
resolve instead of throwing AbortError in createAbortablePromise: This one is interesting, again one solution is to eat up the abort error but there is a question about the unnecessary overhead of throwing only to catch the error and resolve. @jeremymeng suggested adding an option to override the behavior on the abort event. I wonder what does other folks think about this one? /cc @xirzec @mpodwysocki

I think the first option should be good enough with chaining here


Fix signed ints in osmosis-std package

I'm in Seoul right now doing a hackathon and happened to notice several signed integers in the osmosis-std package.
They're for seconds, nanoseconds, etc, and if we have them as signed ints, then we're limiting how high the number can go, since those values can never be negative.
I changed them to be unsigned since we know they'll be positive integers.
Since I'm trying to be heads-down on this hackathon, I kept my focus specific to this package and will have to move on, but want to note that there's one place that might need to be investigated and that's the repr macro like here:
https://github.com/osmosis-labs/osmosis-rust/compare/main...mikedotexe:osmosis-rust:make-several-ints-unsigned?expand=1#diff-03f2479269d2a37584890ff32d065ccc88fd864a9d161417eaaea30c32b9a6b2R72
Cheers

Thanks for the PR, but this is what defined in chain's protobuf, so I would keep it consistent to avoid weird errors due to inconsistencies.


SA_post_hoc_analysis.plot_output_scatter(..), but with laying out the grid of subplots.

has been tested with 4 output variables but with 7 output variables, it fails with this error:
In [59]: sap.plot_output_scatter(results, targets, r2lim=.9)
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
Cell In[59], line 1
----> 1 sap.plot_output_scatter(results, targets, r2lim=.9)

File /work/mads_calibration/SA_post_hoc_analysis.py:381, in plot_output_scatter(results, targets, r2lim, rmselim, mapelim)
    377   break
    379 for col in fig_indices:
    380   # Plot target line across number of samples
--> 381   ax[row, col].plot(results.index, np.ones(len(results.index)) * targets[targets.columns[count]].values, 'k--')
    382   # Scatter plots for results from all samples
    383   ax[row, col].scatter(results.index,results[results.columns[count]])

File ~/.pyenv/versions/3.8.6/lib/python3.8/site-packages/pandas/core/indexes/base.py:5039, in Index.__getitem__(self, key)
   5036 if is_integer(key) or is_float(key):
   5037     # GH#44051 exclude bool, which would return a 2d ndarray
   5038     key = com.cast_scalar_indexer(key, warn_float=True)
-> 5039     return getitem(key)
   5041 if isinstance(key, slice):
   5042     # This case is separated from the conditional above to avoid
   5043     # pessimization com.is_bool_indexer and ndim checks.
   5044     result = getitem(key)

IndexError: index 7 is out of bounds for axis 0 with size 7

Got this one. Breaks for loops in the wrong order. Changed this to if/else and using set_axis_off() for subplots without plots in square matrix.
SA_post_hoc_analysis.plot_output_scatter(results, targets, r2lim=0.9, rmselim=50, mapelim=0.25)


Adding a warning class to console output

For artisan commands, add 'warn' to the available output types. It will print the message in yellow text using Symfony's OutputFormatterStyle class.

Please fix cs.

cs?

Right, sorry. Changed them to tabs.

Hopefully this has been fixed now.

Personally I would've put an empty line after 345 but I think that's getting very nitpicky at this point. Needs a squash to one commit I think
:+1:

Squashed it.

@captbrogers Not really there are still 5 commits... Did you forget to force push?

Sorry, late night coding after a long day isn't always a good idea.

Going to hold off for now.


Add header and footer in cart.html

Kindly review my PR
Fixes issue : #33

Fix the Position of Dark & light Button Logo in Mobile Version

Hello kindly check it i have done with your requirements


The value "fill_parent" is deprecated

Please use "match_parent" in the example

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

ID: 5fbbd04c-fbe3-f6d9-fc7b-1df151f788d1
Version Independent ID: e1829e6f-d78c-5542-718d-1e23858259bf
Content: Using the RelativeLayout in Xamarin.Android - Xamarin
Content Source: docs/android/user-interface/layouts/relative-layout.md
Product: xamarin
GitHub Login: @mgmclemore
Microsoft Alias: mamcle

Hello @wierikr, thank you for the feedback! This topic has been updated and the changes should be published soon.


#3613 fixing off-by-one error on compiler errors

As both the python filename array and the Input_X indexing is 0 based, the -1 operation is not needed.
fromFileArray in closure-compiler uses 0 based indexing. see:
https://github.com/google/closure-compiler/blob/09cca3b536923dfb86d4e2ea34c8ee97d3ab471c/src/com/google/javascript/jscomp/gwt/client/JsRunnerMain.java#L816

The basics

[x] I branched from develop
[x] My pull request is against develop
[x] My code follows the style guide

The details
Resolves
https://github.com/google/blockly/issues/3613

My bad, should have made against upstream repo.


Arbitrary message from doltremoteapi/Dolt in Dolt client

The super schema and unique tags changes necessitate rewriting history of repos, and thus breaking local copy of various repositories. We need users to do some kind of "force pull" operation, but we don't have a way to tell them they need to do this.
To be future proof, we should allow Dolt client to render arbitrary message from remote so that we can furnish users with information about backwards non-compatible changes in the future.

https://github.com/liquidata-inc/dolt/pull/569


Makefile: move SSL options into a block and refine rules

Makefile: move SSL options into a block and refine rules

Move SSL options into a block to make it easy to read, and change
global rules to make it easy to maintain. For the further step, it
gets extensible to add another type.

New version of the library building rule:
        static: $(STLIBNAME) $(SSL_STLIB)
        dynamic: $(DYLIBNAME) $(SSL_DYLIB)

Compare with the orignal version:
        dynamic: $(DYLIBNAME)
        static: $(STLIBNAME)
        ifeq ($(USE_SSL),1)
        dynamic: $(SSL_DYLIBNAME)
        static: $(SSL_STLIBNAME)
        endif

If we want to add a new type(Ex, RDMA), for the new version, we can
do like this:
        ########### RDMA options start ###############
        .....
        ########### RDMA options end   ###############

        static: $(STLIBNAME) $(SSL_STLIB) $(RDMA_STLIB)
        dynamic: $(DYLIBNAME) $(SSL_DYLIB) $(RDMA_DYLIB)

Signed-off-by: zhenwei pi<EMAIL_ADDRESS>

Hi, thanks for the PR.
If you rebase your branch against master (and move the deps targets) I will test and get this merged.

Hi, thanks for the PR.
If you rebase your branch against master (and move the deps targets) I will test and get this merged.

Force updated, thanks for your suggestion.

Looks like this breaks the tests in Windows, because it's unconditionally adding -lssl -lcrypto -lpthread regardless of the USE_SSL value.
This small modification on my fork seems to solve the problem.
If that looks OK to you I can include the patch when I merge.

Looks like this breaks the tests in Windows, because it's unconditionally adding -lssl -lcrypto -lpthread regardless of the USE_SSL value.
This small modification on my fork seems to solve the problem.
If that looks OK to you I can include the patch when I merge.

Thanks!

Merged, thanks!


Update docs/quickstart.rst

Updated quickstart.rst to use commands.Bot instead of discord.Client.
About this pull request
Updated docs/quickstart.rst to use commands.Bot instead of discord.Client.
Changes
Some changes were made to the example code blocks in the quickstart page of the docs to use commands.Bot instead of discord.Client to be consistent with the migration page.
Checklist

[ ] I've checked this pull request runs on Python 3.6.X.
[ ] This fixes something in Issues.

Issue:

[ ] This adds something new.
[ ] There is/are breaking change(s).
[ ] (If required) Relevant documentation has been updated/added.
[x] This is not a code change. (README, docs, etc.)

Is this really needed? The thing about using commands.Bot() is that it has a kwarg for command_prefix but because we are relying on interactions within the Discord API, this prefix is never inferred upon because of Slash Contextualization. The reason it was given discord.Client() instead was solely because of this.

I think the example in the readme should be the "most basic setup" needed for slash command to work. So commands.Bot don't look like something necessary to me.

Okay then, we can keep discord.Client.

Do I close the PR?

That's just my opinion tho, the lead dev can think otherwise 🤷

If you're sure on keeping it, then you're able to close it yourself.

I have an idea; I will make another example showing how to use discord-py-slash-command with discord.py, would that be better?

Made another commit

I'm also thinking about making changes to how the events are listened for, which would remove the issues/annoyances when using discord.Client.
I agree that there is not much point with using commands.Bot if there are no text-based commands or cogs.

I am closing this because frankly I see no point or need in having this as a change for the Quick Start.


Fix: Return context if loaded context failed

@vchan will appreciate your review here

Was there something that prompted this change? Won't this cause problems for endpoints that are expecting a loaded context but instead got an unloaded one? An unloaded context won't have DAG/models and can't plan, produce lineage, etc.

If we have broken model file and user do sqlmesh ui it wont start the server and throw Error: Failed to parse a model definition


Fixing ecdb Address Sanitizer test fail

This fix is related to the following issue:-
https://github.com/iTwin/itwinjs-backlog/issues/1189

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


"add stream" is broken on "edit channel" screen

screen hangs loading

button removed anyway, the whole workflow needs an update anyway


[SPARK-42706][SQL][DOCS] Document the Spark SQL error classes in user-facing documentation.

What changes were proposed in this pull request?
This PR proposes to document Spark SQL error classes to Spark SQL Guide.

Error Conditions

SQLSTATE Codes

Error Classes that includes sub-error classes (INVALID_FORMAT as an example)

Why are the changes needed?
To improve the usability for error messages for Spark SQL.
Does this PR introduce any user-facing change?
No API change, but yes, it's user-facing documentation.
How was this patch tested?
Manually built docs and check the contents one-by-one compare to error-classes.json.

cc @srielau @MaxGekk @cloud-fan This is user-facing documentation for SQL error classes.

@itholic Did you write those files manually, or generated by a script?

@itholic Did you write those files manually, or generated by a script?

I did both. I used the script, and for the parts that didn't print correctly, I fixed them manually by checking the page one by one. Since the script is not very polished for now, let me submit it as a follow-up task if we aim for automation in the future.

+1, LGTM. Merging to master.
Thank you, @itholic and @srielau for review.

@MaxGekk  should we merge it to 3.4?

@gatorsmile I stoped merging PRs related to error classes after RC0. Need to re-check carefully which error classes are in branch-3.4 already. @itholic Please, backport the changes to branch-3.4.

It's probably ok to mention non-existing errors in the doc in 3.4

3.4: https://github.com/apache/spark/pull/40433


Sometime the price axis is not big enough

Lightweight Charts Version: 3.5.0
Steps/code to reproduce:

Actual behavior:

Expected behavior:
The axis will show enough space for the label
Screenshots:

CodeSandbox/JSFiddle/etc link:
https://codesandbox.io/s/axis-number-width-qve8d?file=/index.html
You will need Open in New window to check it

Propose solution:
Currently, the chart width is determined by the length of Min and Max label. So I think it better to check width by all mark label visible + a medium point (min + max)/2

Hi there, it seems that we already fixed this on tradingview.com, I'll migrate a fix to lightweight-charts soon.

Sound great


Update Read function to also check contents of the file

I've updated the resourceLocalFileRead function to also check the contents of the file, so we can replace it when content is changed manually.
Note: this is my first ever attempt to write Go 😉

I'm going to clean up the spacing to match what was already in the project and merge this. Thanks!


Cursor offset from actual position

Others have also reported the same issue, however I am unable to reproduce the same on my setup.
Most probably it seems like an issue with the fonts width or css.
Attempting a fix

Update: Located the issue, it is because of the editor not calculating (or miscalculating) the 'DM Mono' text width when the IDE loads for the first time. Which updates once you add a new cell, the cells rerender and recalculate the width.
As a temporary fix I've reverted to using default monospaced font in 643e391


Allow onScrollToChange() in ArrowKeyStepper to override scrollTo parameters

Hi Brian, thanks for your ongoing support (and the community's :wink:)) on this project.
I'm using arrowkeystepper with multigrid to create a TV guide.  Layout is similar to a gantt chart.
I want to use ArrowKeyStepper's onScrollToChange() function to override the scrollTo parameters to facilitate moving between programs (which span multiple grid cells), but onScrollToChange() is not able to modify the scrollTo parameters.
What I'm suggesting is changing this:
  _updateScrollState({scrollToColumn, scrollToRow}: ScrollIndices) {
    const {isControlled, onScrollToChange} = this.props;

    if (typeof onScrollToChange === 'function') {
      onScrollToChange({scrollToColumn, scrollToRow});
    }

    if (!isControlled) {
      this.setState({scrollToColumn, scrollToRow});
    }
  }

to this
  _updateScrollState({scrollToColumn, scrollToRow}: ScrollIndices) {
    const {isControlled, onScrollToChange} = this.props;

    var scrollIndices = {
        scrollToColumn: scrollToColumn,
        scrollToRow: scrollToRow
    };

    if (typeof onScrollToChange === 'function') {
      onScrollToChange(scrollIndices);
    }

    if (!isControlled) {
      this.setState(scrollIndices);
    }
  }

Happy to raise a PR if this change appears reasonable.
I considered using isControlled and setScrollIndexes() but then I'll have extra work to replicate componentWillReceiveProps() which no longer fires with isControlled set.  This change feels logical and adds flexibility to the method.
As a background, I'm creating the guide as a grid of 1-minute intervals.  A 30 minute program is rendered with its first cell having the syle.width increased to 30 x cell width for the first minute of the program, all other overlapping cells have their width set to 0.  Programs are rendered using a button.  I maintain a 2D array for each cell that tracks the coordinates of the first cell for the overlapping program, and the first cell of the next program.  In this way I can jump from program to program by overriding scrollToColumn in onScrollToChange()  (with the proposed change of course)
I can do this in cellRenderer but multiple keypresses can be missed as rendering is async.
Thanks for considering.

I'm not a fan of that proposal (passing a mutable object to a callback and then using it after the fact). I don't think it fits with the way that any of the other named parameter methods RV uses work.
I should also add that I think the current isControlled prop is awkwardly named and could be improved in the next version of RV. That being said...

I want to use ArrowKeyStepper's onScrollToChange() function to override the scrollTo parameters to facilitate moving between programs

If you want to do this, just store the parameters passed to onScrollToChange in your parent component's state and pass them back to ArrowKeyStepper as props. Override the state/prop values when/if you want.

I considered using isControlled and setScrollIndexes() but then I'll have extra work to replicate componentWillReceiveProps() which no longer fires with isControlled set.

I don't understand what you mean by this. 😄
ArrowKeyStepper will tell you the suggested values (via onScrollToChange). All you have to do is set them, or not. Should be as simple as this, I think:
class ArrowKeyWrapper extends React.Component {
  state = {
    scrollToColumn: 0,
    scrollToRow: 0,
  };

  render() {
    return (
      <ArrowKeyStepper
        {...this.props}
        {...this.state}
        onScrollToChange={this.onScrollToChange}
      />
    );
  }

  onScrollToChange = ({ scrollToColumn, scrollToRow }) => {
    // Maybe you override the scroll-to indices here based on some custom logic

    if (
      scrollToColumn !== this.state.scrollToColumn ||
      scrollToRow !== this.state.scrollToRow
    ) {
      this.setState({ scrollToColumn, scrollToRow });
    }
  };
}

Let me know if I've misunderstood anything. 😄

Hi Brian, I understand your concern on modifying the object in that way.
What you proposed was where I was initially headed, but I had some issues with updating the props.  At least this confirmed I was on the right track.
I've gone back to that and have worked around my issue for now.
The complication (besides my very limited react experience) is that this component is being added to an existing javascript project (no jsx) and there is no parent react component/state to store the parameters.  The ArrowKeyStepperWrapper is what I was missing to manage the prop updating of the children.  A bit more work, but the right solution.
Thanks again for your time and feedback.

No worries 😄
I'm sure you're aware of this, but you can replace the JSX bit in my example above with regular JavaScript:
React.createElement(
  ArrowKeyStepper,
  Object.assign(
    {},
    this.props,
    this.state,
    {onScrollToChange: this.onScrollToChange}
  )
);

Perfect, thanks


Instead of inputing a photo can you input an array?

I was wondering if there is a way of inputing an array into the neural net instead a file path?

Hi MATTYGILO,
Sorry for the delayed response. Unfortuantely it's not supported to input an array at the moment.
The array data is generated from image file at the following line in iOS and Android respectively.
https://github.com/shaqian/flutter_tflite/blob/master/ios/Classes/TflitePlugin.mm#L166
https://github.com/shaqian/flutter_tflite/blob/master/android/src/main/java/sq/flutter/tflite/TflitePlugin.java#L156
You can try to create a new method call that accepts array directly. I will add this in future version.
Thanks,
Qian

Thank you allot. I will go about trying what you suggested. Cant wait for the future update it will allow for more differentiation for machine learning on flutter apps.

Just wondering when that update will be ready for. I had a go but i wasn't able to achieve it.

Sorry I'm behind schedule. I will find time this weekend.

Archiving the issue for now. Feel free to reopen if you have other questions.


UMAP exit with no error

Hi, when I use umap with a  bigger dataset, umap exit without no error. The dataset size is (956966, 234), but if I use 10k samples, it works well. The memory is enough.
UMAP Version:
umap-learn=0.5.3, pynndescent==0.5.8
The code: umap_res = umap.UMAP(n_components=6, n_neighbors=100, min_dist=0.1, verbose=True, low_memory=False).fit(mtx_sample)
The log:
UMAP(low_memory=False, n_components=6, n_neighbors=100, verbose=True)
Fri Jul  7 23:40:40 2023 Construct fuzzy simplicial set
Fri Jul  7 23:40:40 2023 Finding Nearest Neighbors
Fri Jul  7 23:40:40 2023 Building RP forest with 54 trees
Fri Jul  7 23:41:18 2023 NN descent for 20 iterations
1  /  20
2  /  20
3  /  20
Stopping threshold met -- exiting after 3 iterations
Sat Jul  8 00:02:39 2023 Finished Nearest Neighbor Search
Sat Jul  8 00:03:47 2023 Construct embedding
/home/ec2-user/anaconda3/lib/python3.8/site-packages/sklearn/manifold/_spectral_embedding.py:245: UserWarning: Graph is not fully connected, spectral embedding may not work as expected.
  warnings.warn("Graph is not fully connected, spectral embedding"

Could you help to check this issue? Thanks.

It seems to be caused by init initialization, I set init='random', it runs 200 epoch normally.


CoreMethods PopPageModel() throws exception when page contains CarouselView

When you call CoreMethods..PopPageModel() when the Page contains a CarouselView  it throws an exception.
 Invalid target position
Strangely, when you call it a second time it works just fine, and pops the page like it should.
The normal page based "Navigation.PopModalAsync" doesn't have the same problem.
I've only tested this on Android so far, as I haven't had the chance to test it on iOS yet.

#307 Seems to be related


Refactor ParquetExec::try_from_files in preparation for making it parallel

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
The logic in ParquetExec::try_from_files is not written in a way that makes it easy to parallize.
Describe the solution you'd like
Refactor logic to make it easier to apply threading in a separate PR.
Describe alternatives you've considered
N/A
Additional context
N/A

Replaced by https://github.com/apache/arrow-datafusion/issues/944


Can I provide only ids of related models  to the main object?

Hello,
Is were any way to provide only ids of related models to the main object (not model objects)?

you can give models with ids only

Thanks for response. I have one more question:
How can I include not related models in output json?

$encoder->encode([
    $model1,
    $model2,
    $model3
]);

I already try this, but get the error when the models are differ.

diff models are not allowed by the spec


Gestures Not working inside scrollview

While using MotionView inside a scrollview the scrollview gestures not allows to deduct the gestures from Motionview. Due to multiple gestures used in motionview, we couldn't use "dispatchTouchEvent()" in a custom scrollview.
Any optimal solution?
P.S.  Yes, you're right, it does not work with ScrollView (was not intended to work that way).
To make it work - you need to manually pass all the gestures to the ScrollView and handle them


feat: add freeform property to FluentCombobox

🙋 Feature Request
I'd like the freeform property to be added to the FluentCombobox in blazor. The React version has the property freeform which allows the programmer to decide whether or not custom values should be allowed.
🤔 Expected Behavior
The user should not be allowed to enter custom values.
😯 Current Behavior
Currently it is not possible to disallow the user to type custom values.

At the moment, we don't support this feature, but you could use the FluentAutocomplete component with the ValueText property.


feat: implement eth_coinbase/anvil_setCoinbase API endpoints

Two methods to get/set coinbase (aka author/beneficiary/miner):
#[rpc(name = "anvil_setCoinbase")]
fn set_coinbase(&self, address: Address) -> RpcResult<()>;

#[rpc(name = "eth_coinbase")]
fn coinbase(&self) -> RpcResult<Address>;

I feel like this is of very limited usefulness due to the following reasons:

The handler for eth_coinbase is usually disabled for base ETH. I tried 15 different providers and only one of them returned 0x00..00, all others responded with either "unimplemented" or "etherbase must be explicitly specified" (Geth default when --etherbase param is missing).
There is a good reason for the above: coinbase is a consensus layer thing now and it does not make sense for it to be present on the execution layer.
Neither reth nor anvil support eth_coinbase (weirdly enough latter supports anvil_setCoinbase, maybe to manually replicate EVM executions that depend on COINBASE opcode?)

At the same time cost of "faking" support for it is negligible.

@dutterbutter @popzxc What do you guys think would make the most amount of sense here? I am leaning toward adding a very simple implementation (Arc<RwLock<Address>> that is only used in these two methods) and printing something in the logs about coinbase having no impact on anything in ZKsync.
Also fine with marking them as intentionally unsupported with a similar message.

Let's mark them unsupported and wait until someone needs them. Otherwise we're risking adding unneeded functionality.


fix react createClass and PropTypes warning

https://github.com/ant-design/ant-design/issues/5678

ping~

发个新版？


!priority command

We should have a !priority command to set puzzle priority, now that bighunt will no longer allow click-to-edit in the nexus.

We already have this apparently, in !update.


[Bug]: Task dropdown selection database page crashes

ODC version
odc431
OB version

What happened?

What did you expect to happen?
The page does not crash
How can we reproduce it (as minimally and precisely as possible)?
Create SQL plan and select database
Anything else we need to know?
No response

The problem disappears


lldb-moduleimport-test: handle COFF properly

lldb-moduleimport-test would only check for MachO and ELF object file
formats.  However, Windows uses COFF object files.  Add that to the list
of formats that we check.  This allows us to inspect content built for
Windows.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @alexshap

@swift-ci please test and merge


@OAuth2RestOperationsAutowired causes just a BeanCreationException

Maybe I'm using it wrong, but there isn't much documentation.
From @jbellmann's comment I did build this:
@Configuration
public class ProducerConfiguration {

    private static final String PRODUCER_TOKEN_NAME = "producer";

    @Value("${PRODUCER_EVENTS_URI}")
    private String producerEventsURI;

    @OAuth2RestOperationsAutowired(PRODUCER_TOKEN_NAME)
    private RestOperations template;
}

In my application-dev.yml I have the following setup:
tokens:
    token-configuration-list:
        - tokenId: producer
          scopes:
              - uid
              - producer.event_stream.read
        - tokenId: nakadi
          scopes: nakadi.event_stream.write

spring:
  oauth2:
    resource:
      tokenInfoUri: https://info.services.auth.zalando.com/oauth2/tokeninfo

Starting the app (with a dummy producer URL in an environment variable) gives me just a BeanCreationException:

Caused by: org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type [org.springframework.web.client.RestOperations] found for dependency: expected at least 1 bean which qualifies as autowire candidate for this dependency. Dependency annotations: {@org.zalando.stups.tokens.annotation.OAuth2RestOperationsAutowired(value=producer)}

This doesn't change whether or not I have also stups-spring-oauth2-client in my Maven dependencies.

Thanks @ePaul,
I will try to reproduce this.

Your observation is right. It only worked in the test because beans injected in the test-unit itself where injected last. So I have to reimplement that feature.
I will inform you, when I think it will work better.

@ePaul ,
please try again with 0.9.11.M2.

And please have look into this example.

@jbellmann Thanks, my app now starts up without problems. (Not sure if this actually works, I'm not yet this far in the program.)

I can also actually exchange requests, thanks.

So, before we put this new thing into an official release, do you have any improvements (renaming maybe, annotation to long?) of the current state?

I think it is usable in the current form. (It looks like we can also autowire the RestTemplate class directly, which is needed for Riptide.)
A wish (maybe as a separate feature), though I'm not sure if it fits here:
I would like to have some way to access the list of token-configurations, preferably also with other information added inside it. For the example above, I would like to have several producer configurations and several Nakadi ones, and be able to iterate over them, or find them by name. I guess I can put this in a separate configuration section (and just use the name to refer to the token one).

You are right, RestTemplate is also possible :
    @OAuth2RestOperationsAutowired(PRODUCER_TOKEN_NAME)
    private RestTemplate template2;

To access the token-configurations just:
	@Autowired
	private AccessTokensBeanProperties accessTokensBeanProperties;

... then

accessTokensBeanProperties.getTokenConfigurationList()

Provide an PR for more.
Thanks for contributing.

Hi,
I guess I got another issue. We configured our own HttpComponentsClientHttpRequestFactory as a spring bean (for adding some interceptors for logging and tracing).
But the StupsOAuth2RestTemplate which is being built from the annotation here is using its own request factory (using the one-argument constructor), with the effect that our logging doesn't work.
Could this be made to use the constructor with a request factory, if such a spring-bean is available?


feat: Add require ebs encryption scp

what

Adds SCP to require EBS volume encryption

why

Encrypting EBS volumes is a good default security policy
Seems like a common enough policy to include in the provided catalog

references

Shamelessly stolen from Latacora's recommend policies

/terratest

/terratest

@goruha

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest


Implement on plain-Jane, non-framework, basic HTML/JS/CSS pages

Please provide instructions and code for implementing BigPicture (and BiggerPicture) on plain-Jane, non-framework, no-libraries, simple, basic HTML/JavaScript/CSS page.
I was able to get BigPicture working by copying the minimized BigPicture.js source code (via DevTools) from your demo page and experimenting, but it would be nice to have the unminimized, non-NPM source, and also the implementation code example.
BigPicture is doing the job for me for this simple site, but I would like to try out BiggerPicture as well - however, I am hesitant to go through same again (not that it took all that much time to troubleshoot, but it would be nice to make it a plug-and-play experience)

This is a plain-jane javascript library. The instructions for using it are on the home page of this repo. Download the latest build and follow the instructions in the documentation.

To expand on what JP said, you can find both the minified and unminifed production builds in the dist  folder of this repo, as well as on CDNs like jsDelivr. Same with bigger-picture. They are both vanilla javascript libraries.
The bigger-picture demo site has lots of vanilla js code examples and codesandboxes to play with, so setup can be mostly copy + paste. The readme goes into more detail.
If you run into trouble and have a specific question, let me know.


Difference Between Model and Keyword Files

What is the URL of the doc?
https://picovoice.ai/docs/api/picovoice-c/
What's the nature of the issue? (e.g. steps do not work, typos/grammar/spelling, etc., out of date)
What is the difference between model files and keyword/context files for Porcupine?  The link to the model files takes me to the keyword files directory in the Porcupine repo.
Thanks,
JP

The model files (or .pv files) are the common parameters required by the engine. As mentioned in the doc the standard models are available on GitHub and it's the same for all keyword files as long as the language does not change. However, keyword files (aka .ppn files) are specific for each keyword and you may need to generate a custom one using Picovoice Console.
The link for the porcupine model files in the doc wrongly was pointing to the keyword directory instead of the common param directory, and it will be fixed shortly


feat(pallet): rework billing loop insertion

Related to:

#709

Conceptually good but what happens with contracts that have ID's 0 to 600 ? Since the billing frequency is 600 these contracts will get index 0 assigned. Probably we can add a check for these cases :-)


stop loop until the code resolves

I'm tracking urls and sometimes the selector I'm looking for does not appear until the page is refreshed 3/4 times. I need the loop to stop until I find my selector
for (var j = 0; j < arrUrls.length; j++){

  await page.goto(arrUrls[j], {timeout: 0});

      check();
         async function check(){

              if (await page.$(selector) != null) {
                console.log('continue loop');
              }
                else{

               await page.reload();
               console.log('try');
               setTimeout(check, 1000);
             }

        }
}

Maybe this way?
for (var j = 0; j < arrUrls.length; j++){
  await page.goto(arrUrls[j], { timeout: 0 });

  while (await page.$(selector) === null) {
    await page.reload();
  }
}

Thank you !! Work for me


Add OpenAPI documentation for RestoreTimestampRequest fields

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

Comparison is base (5d2bc83) 26.90% compared to head (a8e4c11) 26.90%.

:exclamation: Current head a8e4c11 differs from pull request most recent head d7721b2. Consider uploading reports for the commit d7721b2 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #130   +/-   ##
=======================================
  Coverage   26.90%   26.90%
=======================================
  Files          35       35
  Lines        1613     1613
=======================================
  Hits          434      434
  Misses       1121     1121
  Partials       58       58

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


Argo Workflows 3.6 release changes

When 3.6 releases there will be new fields for some classes, and various new examples. This issue is to update the codebase once the official 3.6 release is made (currently rc1).

[x] Update Makefile Argo Workflows Version
[x] Multiple semaphores/mutexes (only a change to the synchronization field in the spec as we don't have a Hera wrapper for this)
[x] CronWorkflows when - extra field in CronWorkflow
[x] CronWorkflows schedule -> schedules  - extra field in CronWorkflow, make schedule: Optional
[x] Pod GC change - #1292

Hey @elliotgunton ,
There's another change in 3.6 that I am looking forward to, but I don't see listed here:
CronWorkflows StopStrategy
I hope this functionality will be included in Hera as well. Unfortunately I don't have time to contribute it myself.
Just wanted to highlight it in case it was missed.


Podfile issue

There is an error when trying to install nb_utils because the string in s.summary contains an
apostrophe without escaping.
Pod::Spec.new do |s|
s.name             = 'nb_utils'
s.version          = '0.0.1'
s.summary          = 'This package helps you daily usable function with ease. Just add nb_utils with its latest version and that's it you are ready to use.'
s.description      = <<-DESC

Got it. Will fix it very soon.
Thanks