Provision full VPC on AWS using Resource Controller

This PR

Cleans up the AWS plugin to

Deprecate logrus
Honor the properties=true bool flag when Describing instances to return properties of resources
Make proper attachment of subnets to routes if specified.

+ Adds a full `mystack.yml` that will create a full VPC with 1 internet gateway and 2 subnets
+ Adds an `inventory.yml` for configuring the inventory controller which will monitor all the resources created by the resource controller (as driven by `mystack.yml`) and make all the resource properties (e.g. VpcId) be available in metadata form.
+ Updates the compute instance / spot instance playbook files to look at the metadata in suggesting default values such as availability zones and subnet ids.

Signed-off-by: David Chung<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "aws-resources" git@github.com:chungers/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354174864
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


Call action implementation

Call action implementation
High Level Description
This PR implement the call action functionality.

Call action can execute any arbitrary call to any address.
Call action forward the msg.value of the transaction equal to the amount filed.

Should be reviewed/merged after #136 .
Code

[x] Unit test 100% coverage
[x] Does your code follow the naming and code documentation guidelines?

Documentation

[x] Is your code up to date with the spec?
[x] Have you added your tests to the testing doc?

The callFunction() have the vault owner and vault id as arguments, to allow for implementation that need access to those data for any kind of verification/process.

Is there a way that we can make sure user don't accidentally send eth to us in operate?

Looks good to me! Can we in the future also add a test to make sure we can do a trade on uniswap / balancer? (not urgent now)

We can do it in the integration tests.

BTW are you adding the whitelist call actions address functionality here or in a different PR?

BTW are you adding the whitelist call actions address functionality here or in a different PR?

Will add it here.


Front page links

Solves https://github.com/OSU-Net/cyder/issues/834

@church29 can't remember from the meeting, but were you going to change the "register a new device" link to go directly to the create form?

I was going to, but you mentioned users should know where to go to do these things, so I had it redirect to the system list view. I can change if needed though.

Yeah, I'm going to go back on that.. lets just make it easy for them.  How
about you put (Create a System) after the existing text so they at least
know what the heck they're doing.
On Mon, Feb 23, 2015 at 7:36 PM, church29<EMAIL_ADDRESS>wrote:

I was going to, but you mentioned users should know where to go to do
these things, so I had it redirect to the system list view. I can change if
needed though.
—
Reply to this email directly or view it on GitHub
https://github.com/OSU-Net/cyder/pull/840#issuecomment-75693998.

Superseded by PR #879.


Confusion during transfer to OMP resurfaced old bug re file separator characters

Needs to be re-fixed before 0.1.0 release date.

Re-fixed.


Bug fix: train_act_op + .gitignore

Hello, cool project!
I started running it and noticed that I was getting:
TIMER:mil.py:init_network: building TF network (Elapsed: 1.575583s)
Done with map.
TIMER:mil.py:init_network: building TF network (Elapsed: 1.215247s)
Traceback (most recent call last):
  File "main.py", line 296, in <module>
    main()
  File "main.py", line 285, in main
    train(graph, model, saver, sess, data_generator, log_dir, restore_itr=FLAGS.restore_iter)
  File "main.py", line 125, in train
    input_tensors.append(model.train_act_op)
AttributeError: 'MIL' object has no attribute 'train_act_op'

Looks like https://github.com/tianheyu927/mil/commit/d0bd91de7fe147528e6dbe19032c35deb0d33cd0#diff-56b3e8c3f075dbee16d90f98896ad21dL45 pulled out the actual train_act_op.
This PR does 2 things.

Removes the stale train_act_op in https://github.com/tianheyu927/mil/commit/ecfe07cfed8f3b84c10f8f0258886ce71edf2ec4
Minor cleanup & Adds a project friend .gitignore to prevent extra data, python, etc files from getting checked in. https://github.com/tianheyu927/mil/commit/cd4598e279f763e8dbc618c253756cde0ad87d17

LGTM. Thanks!


Create documentation and tutorials

There is a doc branch, where the now documentation is being created. Feel free to contribute.

I added few lines yesterday. After some polish and proofreading, I think the main tutorial is finished. However, the advanced stuff are still incomplete.

Closing as doc was merged to dev.


Add hardware solutions of dexterous hands supporting tactile sensing in the Open Source section

I think we should not only focus on tactile sensors, but also focus on hardware solutions integrated with tactile sensors.
Related information is shown in the PR #15.
Note that I added a new subsection (Dexterous Hand with Tactile Sensing) in the Open Source section, but I'm not sure whether it's appropriate, so I opened this issue to discuss.

Yes, that's great. Thanks a lot for improving the collection.


[Issue] Axles always always placing upwards

active mods:
animania 1.2, BiblioCraft 2.3.2, gravestone 1.6.6, level-up-mod, Pam's harvestcraft 1.11.2h, tconstruct <IP_ADDRESS>, ToughAsNails <IP_ADDRESS>
I am no more able to place axles the desired ways.
Any ideas?

Heyo
Also having this issue. Axel's were placed against the GearBox's while standing on the Laips block. Interestingly enough, the Treated Axel will place correctly but do not work.

Versions

Better With Mods-1.2.35-1.11.2
Forge-1.11.2-<IP_ADDRESS>8-universal.jar
Complete List

this is a duplicate of #466
seems this is fixed in dev already.
treated axles can be placed on the inputs of IE kinetic generators to generate RF from the BWM mechanical system.

it is a dublicate of #466 idd but it's clearly not fixed (-> still havin' the issue, no capability to flip the axles around) nor is there any comment stating any fix.

except here, of course

true :). Didn't see this one. So lets w8 for the official submit :).


Remove refs from Browser component

Type

refactoring

Related JIRA Issue(s)
ENSWBSITES-174
Description
The task description says "Check why useRef is used in the Browser.tsx and remove it if it’s not required".
useRef was used for 2 purposes in the Browser component:

to hold reference to the html element into which genome browser renders the canvas element, and
to get reference to the last value of active genome id coming from the props

Regarding the first usage, it is still relevant (some React components, such as zmenu, may need to be able to access the wrapper around canvas directly to calculate the dimensions of the canvas), but the ref itself can be moved from Browser to BrowserImage.
Second usage seems to be redundant at the moment.
Note: BrowserImage contained code that communicated with genome browser using the old mechanism (sending custom events via a DOM element) to tell genome browser stuff about cogs. I've changed the communication method to the new one (via browser-messaging-service) and moved this code to BrowserCogList component.
Views affected
Genome browser screen.

Is there anything else that needs to be done in this PR?


Automator: update ztunnel@master in istio/istio@master

Generated by Automator - 2024-06-28T03:23:07+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
fbfdbc176989a296b3eec99b493c68f81f8523d5
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
fbfdbc176989a296b3eec99b493c68f81f8523d5
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


Add support for global aggregations on subqueries

Current coverage is 81.08% (diff: 78.57%)

Merging #4404 into master will increase coverage by 0.04%

@@             master      #4404   diff @@
==========================================
  Files          1302       1302
  Lines         44060      44083    +23
  Methods           0          0
  Messages          0          0
  Branches       7011       7015     +4
==========================================
+ Hits          35706      35744    +38
+ Misses         5668       5656    -12
+ Partials       2686       2683     -3

Powered by Codecov. Last update c5e6798...c34188b

@dobe please review


feat: allow overriding preventDefault for modifier keys

Overview
In response to this issue: https://github.com/BuilderIO/qwik/issues/3871
The only place to preventDefault or not is in the loader, unfortunately this makes it impossible to preventDefault on anchors while also letting users use browser-native modifier keys (ctrl, shift, alt) to open new tabs, etc on these same links.
This impl. adds an attribute at the same level as preventdefault:* that allows overriding it for modifier keys. The reason for this is to allow devs to still preventDefault even on these modifier keys.
Given the widespread use of preventdefault and the niche use of modifier keys, I think keeping this check as false by default and opt-in only when needed makes more sense.
What is it?

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Thanks @jordanw66 !
Somewhat related to #4496 (as it's about adding a change to the qwikloader, which should be done on very rare ocassions IMO)

@shairez @jordanw66 this is even better. we can than fine-grain these options in Link or custom components.
It's not breaking if it's a union with boolean. @jordanw66 let me know if you need anything. I'm pretty free tomorrow.

@shairez Interesting, didn't see that request. That will require a slightly larger increase to loader, do you guys know which direction you want to go in? I could take a stab at implementing that if you want. Could replace this PR with more functionality.

Thanks! yes this will resolve so many future issues as well 😅
The only concern I had is about how to convey that this is a syncornous function to the developer... but maybe if it's withouth the $ then it's fine :)
Let's wait to hear what @manucorporat @adamdbradley @mhevery have to say about this before proceeding with a PR

The optimizer can already inline functions, but it's implicit there is not way to signal this
wonder if you can do this:
preventdefault:click={$$((event) => {

})}

$$ would be a new compiler API, like $, but syncronous, and you cant have imports!
the optimizer already knows to do that, but it's not public

Not a fan of $$ and it would be very advanced API. Maybe:
_inlined_fn_

@manucorporat let us know if you know how to move forward. I think synchronous would be crazy in this async context.
I still can't get my head around, why we can't use event.preventDefault() in the Link-Component. You pass it through in dispatch, but I haven't read all of this and it ends up beeing any at some point.
I really need to read more about the qwik loader 🥲

@jordanw66 Thanks for the kick to my head,it works better now 😂
Your idea sounds good. Keeping track of them by referencing might be the best solution.
I have popcorn 🍿 now. Manu will find the right solution, he always does.

@manucorporat @jordanw66 is this still ongoing?
The browser's default behavior should actually be the default. I would propose disallowModifiers should be a prop.
For me even @jordanw66 PR would mean I have to change already over 100 links in our project.
I am thinking about writing a custom link component because of this behavior.
If you hit the meta keys to open in a new silent tab or window to copy the link faster or compare products in or ecommerce shop there is no way we will leave the link like this.

Hey guys, this issue has been stale for a long time, is it possible we could revive this and decide on a solution?
It's a poor experience to not be able to use default browser behaviour with links.

@manucorporat So we want custom preventDefault filtering. Do we ever want any other sync event processing? What if we want to have the same custom preventDefault on several event types?
You proposed but did not like preventdefault:click={$$((event) => {})}, but I think we could special-case this? So just preventDefault:whatever={(ev) => ...}. The optimizer can emit the callback as an inline function and replace it with some sort of IQRL (inlined function index)?

@jordanw66 but is there a plan to implement it as default? Qwik is the only framework I know, overriding browser standards here.

@tzdesign this is tracked by #3871  right? looks like @mhevery self-assigned that, so guessing he's planning to make use of sync() to solve that once he's merged the sync() feature itself

@tzdesign this is tracked by #3871  right? looks like @mhevery self-assigned that, so guessing he's planning to make use of sync() to solve that once he's merged the sync() feature itself

Oh gosh, I forgot to check my own issue 😅
Good to know. With @mhevery its in the best hands 🙌


Increased mocha test timeout

The default test timeout for mocha object is 2 seconds which randomly caused false failures while waiting for an operation to complete. This may be due to slow access we have from Sydney to us-west-2. We suggest increasing timeout to 5 seconds as this fixed our problem, and we think  this would work well for everybody regardless of geolocation.

I've made the change and set the timeout to 6000ms to be in line with the serverless defaults.
Change incorporated in package 0.5.13

Thanks @mpuittinen . This awesome plugin gets better and better. Thanks for the hard work.

is it possible to set a global timeout from CLI command? For example:
sls invoke test --all --stage testing --timeout 10000


Fix support for diffusers 0.15.0

To support latest diffusers you also need to wait for this PR

Yeah, I just realised the positional arg do_classifier_free_guidance is also causing problems with the latest release.
Let's see if that can be fixed in the PR you mentioned on the diffusers side. If it doesn't get resolved in the next week or so, we can adjust the code in this repo.
For now, I will make it explicit that diffusers==0.14.0 is currently supported.

@remorses It seems that the issue does not occur with 0.16.0? Is it okay to close this down?

I would merge this PR because direct use of model parameters is discouraged, it’s better to use the config object
feel free to close the PR if you don’t want to

Right, there is that too. I agree, will merge it then.


Unit Test seem to be broken

In PR #8 you asked for unit tests. I tried to execute the current tests, bad failed badly.
Can you explain how to run them, or what would be your preferred way to add tests?
Regards,
Bianco

@zivillian Sorry for the late reply :)
We've teamed up with @LordMike and moved all the code to a new repository, https://github.com/DiscUtils/DiscUtils; and your last PRs are in progress of being moved over as part of DiscUtils/DiscUtils#18.
Good news is that the unit tests are back up and running; so if you're still up to it, unit tests are still more than welcome :).
For new PRs, it would also be great if you could send them to DiscUtils/DiscUtils.
Thanks!


Error: no implicit conversion of nil into String

I'm having an issue using the default values for some of the available options in this plugin. If I just add "require 'capistrano/symfony'" to my deploy.rb, I get an error about symfony_cache_path being null. I played around with this locally to see if I could figure out what was happening, it seems like either defaults.rb or paths.rb isn't really running or is running in the wrong order, but I'm not familiar with ruby/capistrano enough to really know how it's supposed to happen.
Note, this is an old project that I was in the process of converting over from capifony to capistrano 3 and it is stuck on symfony 2.2.1 for now. Not sure if that would cause this issue though.
Workaround
This is a totally valid workaround, so I'm not really concerned about this bug getting resolved, I just wanted to note it in case anyone else ran into this issue.
It works fine if you just set the cache_path in deploy.rb. I get other errors but I didn't look into those as deeply. I really wanted to just use the defaults, but I wound up just copying the defaults from the readme and adding those to my project's deploy.rb.

set :app_path, "app"
set :cache_path, fetch(:app_path) + "/cache"

More things in case anyone wants to try to resolve this...
Stack Trace
** Invoke symfony:create_cache_dir (first_time)
** Execute symfony:create_cache_dir
DEBUG [eab82b51] Running /usr/bin/env if test ! -d /srv/oved/prps/releases/20151023124234; then echo "Directory does not exist '/srv/oved/prps/releases/20151023124234'" 1>&2; false; fi as<EMAIL_ADDRESS>DEBUG [eab82b51] Command: if test ! -d /srv/oved/prps/releases/20151023124234; then echo "Directory does not exist '/srv/oved/prps/releases/20151023124234'" 1>&2; false; fi
DEBUG [eab82b51] Finished in 0.086 seconds with exit status 0 (successful).
cap aborted!
SSHKit::Runner::ExecuteError: Exception while executing as<EMAIL_ADDRESS>no implicit conversion of nil into String
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/gems/2.0.0/gems/sshkit-1.7.1/lib/sshkit/runners/parallel.rb:16:in `rescue in block (2 levels) in execute'
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/gems/2.0.0/gems/sshkit-1.7.1/lib/sshkit/runners/parallel.rb:12:in `block (2 levels) in execute'
TypeError: no implicit conversion of nil into String
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/2.0.0/pathname.rb:392:in `initialize'
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/2.0.0/pathname.rb:392:in `new'
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/2.0.0/pathname.rb:392:in `join'
/usr/local/rvm/gems/ruby-2.0.0-p643/gems/capistrano-symfony-0.4.0/lib/capistrano/symfony/dsl/paths.rb:20:in `symfony_cache_path'
/usr/local/rvm/gems/ruby-2.0.0-p643/gems/capistrano-symfony-0.4.0/lib/capistrano/tasks/symfony.rake:59:in `block (4 levels) in <top (required)>'
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/gems/2.0.0/gems/sshkit-1.7.1/lib/sshkit/backends/abstract.rb:77:in `within'
/usr/local/rvm/gems/ruby-2.0.0-p643/gems/capistrano-symfony-0.4.0/lib/capistrano/tasks/symfony.rake:58:in `block (3 levels) in <top (required)>'
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/gems/2.0.0/gems/sshkit-1.7.1/lib/sshkit/backends/netssh.rb:54:in `instance_exec'
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/gems/2.0.0/gems/sshkit-1.7.1/lib/sshkit/backends/netssh.rb:54:in `run'
/usr/local/rvm/rubies/ruby-2.0.0-p643/lib/ruby/gems/2.0.0/gems/sshkit-1.7.1/lib/sshkit/runners/parallel.rb:13:in `block (2 levels) in execute'
Tasks: TOP => symfony:create_cache_dir
The deploy has failed with an error: Exception while executing as<EMAIL_ADDRESS>no implicit conversion of nil into String

Some Other Info
I'm on a Vagrant instance running a precise32 box, here are some versions of things:
& ruby -v
ruby 2.0.0p643 (2015-02-25 revision 49749) [i686-linux]
& rvm -v
rvm 1.26.11 (latest) by Wayne E. Seguin<EMAIL_ADDRESS>Michal Papis<EMAIL_ADDRESS>[https://rvm.io/]
& gem list
*** LOCAL GEMS ***
bigdecimal (1.2.0)
bundler-unload (1.0.2)
capifony (2.8.6)
capistrano (3.4.0, 2.15.6)
capistrano-composer (0.0.6)
capistrano-file-permissions (0.1.1)
capistrano-maintenance (0.0.3)
capistrano-symfony (0.4.0)
chunky_png (1.3.4)
colored (1.2)
colorize (0.7.7)
compass (1.0.3)
compass-core (1.0.3)
compass-import-once (1.0.5)
executable-hooks (1.3.2)
ffi (1.9.10)
gem-wrappers (1.2.7)
highline (1.7.8)
hipchat (1.5.2)
httparty (0.13.7)
i18n (0.7.0)
inifile (2.0.2)
io-console (0.4.2)
json (1.8.3, 1.7.7)
mimemagic (0.3.0)
minitest (4.3.2)
multi_json (1.11.2)
multi_xml (0.5.5)
net-scp (1.2.1)
net-sftp (2.1.2)
net-ssh (3.0.1)
net-ssh-gateway (1.2.0)
psych (2.0.0)
rainbow (2.0.0)
rake (10.4.2, 0.9.6)
rb-fsevent (0.9.6)
rb-inotify (0.9.5)
rdoc (4.0.0)
ruby-progressbar (1.4.1)
rubygems-bundler (1.4.4)
rvm (<IP_ADDRESS>)
sass (3.4.19)
scss_lint (0.42.2)
scss_lint_reporter_checkstyle (0.2.0)
sshkit (1.7.1)
terminal-notifier (1.6.3)
test-unit (<IP_ADDRESS>)

+1

@jgettings Thanks for the detailed report, if you have a chance to play around with the new release and this is still a problem can you reopen?
https://github.com/capistrano/symfony/releases/tag/1.0.0.rc1

i have still the problem with the 1.0.0.rc1 and rc2 ... the work around doesn't work. I have open an issue for this.

Same problem


PostgreSQL - Authentication success with pg, but fail with knex?

Environment
Knex version: v0.15.2
Database + version: PostgreSQL v10
OS: Mac OSX
Bug

Explain what kind of behaviour you are getting and how you think it should do
As my title said authentication fail in knex, but works in pg with the same connection config. It seems like knex do accept the config but does not pass it to driver.

Error message
error: password authentication failed for user "qwang"

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to mysql or postgresql, then single file example which initializes
needed data and demonstrates the problem.

// db.js
const dbConfig = require('config').dbConfig
//  dbConfig: {
//    client: 'pg',
//    connection: {
//      host: process.env.PG_HOST,  -> '<IP_ADDRESS>'
//      port: process.env.PG_PORT, -> '5432'
//      user: process.env.PG_USER, -> 'owner'
//      password: process.env.PG_PASSWORD, -> 'MyStore12345'
//      database: process.env.PG_DATABASE, -> 'mystore'
//    },
//    pool: {
//      min: 3,
//      max: 7,
//    },
//    acquireConnectionTimeout: 5000
//  }

const knex = require('knex')(dbConfig)
const client = new knex.client.driver.Client()

console.error(knex.client)

client.connect(err => {
  if (err) {
    console.error('Error with connecting database: \n', err)
    process.exit(-1)
  }
  console.log('Postgres is ready to use')
  client.end()
})

module.exports = knex

Part of output:
...
Client_PG {
  config:
   { client: 'pg',
     connection:
      { host: '<IP_ADDRESS>',
        port: '5432',
        user: 'owner',
        password: 'MyStore12345',
        database: 'mystore' },
     pool: { min: 3, max: 7 },
     acquireConnectionTimeout: 5000 },
  logger:
   Logger {
     _debug: undefined,
     _warn: undefined,
     _error: undefined,
     _deprecate: undefined },
  connectionSettings:
   { host: '<IP_ADDRESS>',
     port: '5432',
     user: 'owner',
     password: 'MyStore12345',
     database: 'mystore' },
  driver:
   PG {
     defaults:
      { host: 'localhost',
        user: 'qwang',
        database: 'qwang',
        password: null,
        connectionString: undefined,
        port: 5432,
...

In output, the client accept correct config as its' config and connectionSettings. However, in the driver, the defaults values ('qwang' which is my username in OS) are wrong and they were used in real connections. It causes authentication error.

Try to use directly pg driver to connect it will fail the same way that knex does when you are trying to make query. I don't see anything here which would suggest there is bug in knex causing this.
If you still think this is bug in knex please create single file test with DB initialization etc. to make it possible for others to verify that it doesn't work.


Use reflection for invoking the Session.getQuery() methods

Closes #1917

Target is master branch, so 4.x, but backport seems feasible.

I am ok with dropping support for older HIbernate versions.

I think doing it like @olivergierke suggested is better, however there haven't been any plans yet for Querydsl 5.
But I suppose we'll have to now. 👍

Querydsl is now compatible with hibernate 5.


replace code to make ssh arguments.

Q
A

Bug fix?
No

New feature?
Yes

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

Do not forget to add notes about your changes to CHANGELOG.md
ssh command on rsync.php execute only identityFile option.
so, modify to make ssh command line options from Ssh module on deployer base system.

@antonmedv Could we tag this feature?

Done.


Widget TaggitSelect2 looks like Multiple choice select, not Tag input with autocomplete

Here is screenshot (dropbox). Code is absolutely identical to docs.

Yep, just fixed a but about it in master. Updated select2 and taggit tests pass again.


Log a direct link to the graylog message

A link to the stream is useful, but it's even more useful to be able to get
directly to the message. This is especially true if you are looking at an older
message that's outside of the time window given.
This commit misses one thing: referencing the correct index. Hopefully that can
be fixed before a merge. In the meantime I'm using graylog_deflector, which
will the index active at the time of using the URL.
I would love some help on how to get the current index. As mentioned in the
code comment, using msg.getFieldAs(String.class, "_index") gave me null.

 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.

Undid a change that wasn't meant to make it upstream. If we figure out the index name, I can squash it all into one beautiful little commit 😄

@joschi could you have a look and help out with getting the index the message is in?
Being able to find the exact message is way more useful than the coarse /messages?q=*&rangetype=relative&relative=3600 link.

This looks like what I had wanted with #35. Started digging into this today with @theunforgiven and we too wondered how to even get the index out of an entry

Anything I can do to move this pull request forward?

Is it possible to get this merged?

What's holding this?

I'm not sure what's holding it. I did have a question about getting the current index instead of using graylog_deflector, however I've been using this patch since I wrote it without problem. Perhaps it should be merged as is and then we can work on getting the correct index as a separate issue.

@johshoff I'm taking over the repo maintenance now. We can def get this moving forward. Can you post a sample message of what your patch adds please ? Also, templated messages have been merged in, which removed a lot of old options that are no longer needed. I'd like to add this feature as a first class template variable. Thoughts ?

Great news!
It simply changes "New message" in the graylog message from plain text to a link directly to the log message. It looks like this before and after:

I think adding it as a template is a good idea. I won't have time to do that work immediately, so feel free to do it. Otherwise I can try to get around to it at some point.

@johshoff , i can merge this as soon as the conflicts are resolved.

@Aenima4six2 Thanks. Finally got around to merging it. Github didn't like the way I merged, so is displaying all intermediate commits. I created a new pull request with exactly the same code that displays nicer: #60 . Feel free to close one of them and merge the other.


[perf] Use color triplets

Ran a benchmark before making changes. It's just testing the example from the readme.
basic                   time:   [99.085 us 99.918 us 100.82 us]

Then replaced the color parsing with hard coded integers, and ran the benchmarks again.
basic                   time:   [13.337 us 13.388 us 13.443 us]
                        change: [-86.708% -86.590% -86.469%] (p = 0.00 < 0.05)
                        Performance has improved.

:+1:


Story List: 'Update view model story order' is failing after move

StoryListCtrl.updateViewModelStoryOrder is telling us that something is wrong with the view model, as from time to time the "Something unknown happened with the move. Need to refresh page" error is being reached.
I don't know how to reproduce the issue, but it is certainly happening from time to time. A refresh of the page brings us back to a valid story order and no data is lost.

I think this is fixed with the migration to Vue.


Copy gist url does not respect github enterprise domain

Right now its not possible to copy the url of the gist from the context menu if the extension is configured to github enterpise. The URL always build as a public gist URL.
It should be like this: <host>/gist/<username>/<gist-id>
(Not sure if this is always the case)

Thanks for reporting this! I suspect there are a handful of other locations that are hard coding the public GitHub URL. I’ll fix all of these today 👍

In the case of public GitHub, the URL is https://gist.github.com/{userName}/{gistId}. So it appears like in GitHub Enterprise, it's not just the domain that's different, but also that the path is pre-pended with /gist? Apologies for the stupid question, but I don't have a GHE account to actually test any of this :)
If you can confirm that for me (try it with and without the /gist/ part), then I can make this change. It sounds like I likely need to replace the Gistpad: Api Url setting with a GistPad: Host setting, and then apply that to both the API URL as well as the gist URL.
BTW, one other question, have you already configured your GHE host with another extension? I just ask since I'd like to automatically pick this up if possible, to prevent users from needing to set their GHE host name twice.

Other commands that need to be updated:

View Profile on GitHub
Pasting images into the editor
Git SSO
Relative paths in the playground webview

In the case of public GitHub, the URL is https://gist.github.com/{userName}/{gistId}. So it appears like in GitHub Enterprise, it's not just the domain that's different, but also that the path is pre-pended with /gist?
If you can confirm that for me (try it with and without the /gist/ part), then I can make this change. It sounds like I likely need to replace the Gistpad: Api Url setting with a GistPad: Host setting, and then apply that to both the API URL as well as the gist URL.

The /gist part is indeed required. Without it I'll get a 404 as result.

BTW, one other question, have you already configured your GHE host with another extension? I just ask since I'd like to automatically pick this up if possible, to prevent users from needing to set their GHE host name twice.

Kind of yes. There are a few extensions which require configuration of GHE. All a bit different.
I've maintaining vscode-github, the MS extension github.vscode-pull-request-github and yours at least are the ones that I know of.

Closing this as a duplicate of #96, since GistPad no longer supports GitHub Enterprise after integrating with VS Code's native auth. We'll need to wait for VS Code to add that support, before GistPad can re-introduce it.


Youtube video titles blank in version 2019.07.27

Checklist

[ x] I'm reporting a broken site support
[ x] I've verified that I'm running youtube-dl version 2019.07.27
[x ] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://youtu.be/cHEIYcs7LRs', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.07.27
[debug] Python version 3.7.4 (CPython) - Darwin-18.6.0-x86_64-i386-64bit
[debug] exe versions: avconv 12.3, avprobe 12.3, ffmpeg 4.1.4, ffprobe 4.1.4, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] cHEIYcs7LRs: Downloading webpage
[youtube] cHEIYcs7LRs: Downloading video info webpage
WARNING: Unable to extract video title
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://r3---sn-8xgp1vo-cvne.googlevideo.com/videoplayback?expire=1564470442&ei=Spg_XZeUCpiU8wSf5rP4Aw&ip=<IP_ADDRESS>&id=o-ABKsg5QBbJSY7JlwtyPrtWCx7gqcNl7lxA1Wpl6GJ7Mm&itag=313&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C271%2C278%2C313&source=youtube&requiressl=yes&mm=31%2C29&mn=sn-8xgp1vo-cvne%2Csn-ab5l6nsr&ms=au%2Crdu&mv=m&mvi=2&pl=16&pcm2=yes&initcwndbps=2046250&mime=video%2Fwebm&gir=yes&clen=812591363&dur=677.310&lmt=1563958863024256&mt=1564448712&fvip=6&keepalive=yes&c=WEB&txp=2316222&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cpcm2%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=ALgxI2wwRAIgD1PlcC-gcBkM6sFwQe7C8aTAUgqdvxMW_IOT2Wj-we4CIHF2VpkaXMQmnG6IW-LDHXzYMgWlh91ZTIK3wnjv5gpz&lsparams=mm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AHylml4wRQIhAOdecZfCy0aJO5A7j0Qs8WiGTyA-meVv7qOdJz-gW4cyAiA5aSFXrFfBcwwMq9iEkD2diUeL7Tev7B9_UWvJEVoVRg%3D%3D&ratebypass=yes'
[download] Resuming download at byte 22986591
[download] Destination: _-cHEIYcs7LRs.f313.webm
Description

For video's that I try after upgrading to 2019.07.27, the video's title is saved as _
Ex:
youtube-dl https://youtu.be/cHEIYcs7LRs
[youtube] cHEIYcs7LRs: Downloading webpage
[youtube] cHEIYcs7LRs: Downloading video info webpage
WARNING: Unable to extract video title
Video's name will be:
_-cHEIYcs7LRs.webm
Video's name should be something like:
Dell XPS 15 7590 REVIEW - The Perfect Laptop-cHEIYcs7LRs.webm

confirmed, same issue using
youtube-dl.exe --console-title --no-progress --no-check-certificate --ignore-errors --playlist-start 1 -xf bestaudio "RDPRy59ec5csQ"


jsonpath for JSON Extra-files in yaml config

TL;DR
Enable the usage of JSONPath to annotate properties in YAML configuration when specifying extra-files of type JSON for release-please, similar to the existing functionality in release-please-config.json.
Detailed design
Proposed Solution;
  - name: Create Release
    uses: google-github-actions/release-please-action@v3
    with:
      release-type: node
      command: release-pr
      extra-files:
        - path: public/manifest.json
          type: json
          jsonpath: $.version

Additional information
Annotations such as the one provided in the docs are not possible when dealing with JSON files. The request is made as I would like release-please to be able to update my Manifest file version.

For complex configurations (including extra-files), please configure a release-please manifest configuration.
The configuration is getting very complex and it's not maintainable to map every complex configuration option into GitHub action yaml input counterparts. We don't plan on proactively trying to map this inputs moving forward.


BUG ps missing from ide

I assume it is being purged by some of the apt autoremove commands or something

https://github.com/AnubisLMS/IDE/commit/d78390cace36fcf9162f7999cb4dc298ea932642


Error while running ecs-anywhere-install.sh

I was getting timeout issue as gpg file was not present into my Ubuntu 20.4 version while running ecs-anywhere-install.sh Below commands resolved the issue.
$ sudo apt-get update
$ sudo apt-get install awscli -y

Interesting. Thanks for bringing this up. Question: was the Ubuntu machine built with Vagrant (per the option in this tutorial) or was this one of your machines you built with your own processes? I am asking because when I tested this with Vagrant it worked and it did not need any additional command.


No backspace mode

We could consider adding a "no backspace" mode like in a real typewriter, but this may be out of scope of this plugin.

Hey thats a great idea. The plugin would live upto it's name, maybe add this something like an experimental feature.

Definitely upvoting this! Something I've been looking for is an app or plugin that would get me out of the habit of over-editing and that would be fantastic to have.

I just found out, that there is a plugin that implements just this: jobedom/obsidian-hemingway-mode. I am thus closing this issue for now.


semantic-ui: frontpage styling issues

Small browser window issues:

Numbers are not centered
Text too small to read in boxes below numbers

Closed by #164


gasd

TL;DR
This pull request creates a new daemon called gasd which improves the longitudinal controller on certain makes by directly learning the correct gas and brake signals to provide a specified acceleration. This learning is done in a similar way to torqued, and in fact much of the code is borrowed from there. The new controller scheme is entirely optional, and is implemented in a way that requires no code changes for makes and models that wish not to use it.
Why this is important
As part of its operation on certain makes, openpilot must convert the CC.actuators.accel into gas and brake CAN signals sent to the car. I will hereafter refer to this as the "controls map". Currently, the controls map uses lookup tables and usually a univariate linear interpolation. These lookup tables are generally reused between several models, do not capture individual vehicle dynamics such as cargo weight or brake wear, and do not take into account externalities such as pitch. These discrepancies have historically been dealt with by the long PID controller and any issues are swept under the rug and often blamed on the PID tune, not on the controls map. It is also largely unclear where these lookup tables come from and how they can be improved.
The problem becomes immediately apparent when considering the relationship between the controls map zero point and long deadzones. If the accel command is zero, but the controls map sends a gas command that produces nonzero acceleration, the car will proceed to accelerate or (usually) decelerate instead of maintaining exact velocity. (One might call this a controls mismatch.) In the case of openpilot ACC long, this may not matter so much, as the growing speed difference will eventually cause the controller to desire sufficient acceleration to overcome the deadzone. However, with the end-to-end longitudinal controller, this is not guaranteed, and indeed the steady deceleration may well condition the model not to accelerate as it should. For reference, on master my car appears to require a CC.actuators.accel input of about 0.3 m/s/s in order to maintain speed at 40mph.
My subjective experience driving with the (offline) fit is that it produces noticeable improvements to experimental mode, especially in the braking department. Compared to these changes, master tends to brake late or insufficiently. My hope with this pull request is to bring such benefits to all vehicles.
Past work
The only recent attempt I am aware of at improving the controls map has been by @twilsonco in https://github.com/commaai/openpilot/pull/25453. His work only updates the lookup tables to be velocity-dependent, but does not provide any sort of live parameter learning, and all results are derived from a single vehicle, so individual vehicle dynamics are ignored. It is also implemented in a way that makes it difficult to add additional input variables.
Implementation details
gasd functions similarly to torqued. It collects three input variables (CarState.aEgo, CarState.vEgo, LiveLocationKalman.calibratedOrientationNED.value[1]) and two output variables (CarControl.actuatorsOutput.gas, CarControl.actuatorsOutput.brake). It buckets these and performs a corectilinear least squares fit (i.e. it ignores gas points where brake is nonzero and vice versa, and clips/rectifies each output appropriately). The outputs are coefficients that are used to compute the gas and brake signals.
In my analysis of my own car, I found that the gas and brake signals have a nearly affine relationship to the three input variables. Hence, using LLS here is a good fit. I assume the affine relationship holds for other models and makes as well.
In controlsd, after computing CC.actuators.accel, this is then fed into the long controller with v_ego and pitch, and the gas and brake signals are computed by taking dot products with the learned coefficients. These are then stored in the currently unused (deprecated?) actuators.gas and actuators.brake fields.
In the GM carcontroller, I check if both of these are not null, and if so, use them instead of the current lookup table controls map.
Assumptions made

The gas and brake signals have a roughly affine relationship with the three input variables, and the these input variables are sufficient to capture the dynamics of the car.
The braking signal is always positive. This appears to be the case on GM, Honda and Subaru, which are the three makes that currently use controls maps.
The OLS problem is sufficiently well-conditioned to perform a numerically stable fit with np.linalg.lstsq.

Possible changes
Here are some possible changes that could be made to what is already here:

Better bucketing would be boundlessly beneficial. The current setup does not always produce a good fit. Too few points and the fit is not ideal, too many buckets and it takes a long time to collect a sufficient amount of points (especially if the buckets are to include pitch information, it may be impossible for those who live in flat areas). If @nuwandavek could clarify how the bucket size was chosen for torqued, that would be helpful here.
The gas offset should be removed from CarParams or CarControllerParams so that it is not duplicated in two places
Much of the code is duplicated from torqued. This could be refactored into a common library. (I did this a little bit, but wanted to touch torqued as little as possible.)
We may consider using velocityCalibrated and accelerationCalibrated instead of CarState.vEgo and CarState.aEgo. It is unclear to me which would be preferred here.

TODO:

[ ] Identify better sizes and bounds of buckets as detailed above.
[ ] Validate the linear relationship between the inputs and outputs for other makes and models
[ ] Generate offline initial fits for other makes and models
[ ] Refactor other carcontrollers on relevant makes:

[ ] to use new system
[ ] to set new_actuators.gas and .brake

[ ] Possibly attempt some basic feature engineering. Analysis on my car shows the possibility that log(vEgo) may have a more linear relationship with gas and brake than vEgo itself, for instance

Hey @nworb-cire, thanks for the PR and the thoughtful notes. This is a good idea, but is not something we will be able to upstream currently. Here are a few reasons:

Things that affect multiple platforms (even if we start with only a few initially) across the board require a massive validation effort (especially the assumption wrt. linear relationships), which we cannot accommodate in our roadmap at the moment.
Using new inputs like pitch also fall under the above category (we spent multiple months making sure roll is as perfect as it is right now).
A lot of the controls issues exclusive to experimental mode are because the current models are pretty dumb. Something like gasd would be the wrong layer of abstraction to try and fix them. We're working on some very exciting ground-up model solutions to these problems, which will have marked improvements. We expect a lot of the planning/noise to be handled by the model better.
We started the lateral controls effort by having good metrics around openpilot's inadequacies. We're working on things to ship on this front in the near future. I expect longitudinal controls to follow a similar arc.

That said, I think forks are the best place for something like this. I can definitely answer any questions you may have about torqued (the bucket bounds were drawn from the distribution of data, and the size was determined from experimentation and expected convergence time). We may revisit this in the future, but I'm going to close this now.


Add support for organization configuration

The same way we can configure repos we should be able to configure organizations

done


Wildcards for caching folders

Hello,
I understand that we do not have support for the input path to have wild cards.
However it would immensely solve our problem if we added this enhancement, because in our use case we are having this mono-repo in which we need to cache a lot many of underlying folders. If we had support for wild cards to identify these underlying folders, it would be great.
Thanks

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

ID: 8dd39237-2778-c4f1-3ce4-bde343ecb59d
Version Independent ID: e3daf460-128d-cb57-e899-ca0c841dff07
Content: Pipeline caching - Azure Pipelines
Content Source: docs/pipelines/release/caching.md
Product: devops
Technology: devops-cicd-release
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@pushkargupta9807 Thank you for your feedback! Please feel free to request a feature on Azure DevOps Developer Community


Cards and switches lack contrast in Neutral theme

Hi @dragonwocky :)
I feel like cards and switches lack contrast in Neutral theme. Take a look at these examples: the card doesn't catch the eye because its color is too similar to the background. Furthermore, it looks like the color is the exact same one as the weekend calendar background, resulting in the card not being visible.
Same goes for the switches color: I only know that they're on because the dot is on the right, but it's difficult to see at a glance. On the default theme, for example, they're blue and gray, so you can see instantly which are which.

Would it be possible to bump the contrast of this color a little?

I've made some slight changes to improve contrast in the calendar, but the colours are as intended.

Really looks better with the new changes, thank you.


Meetings and Trainning Documentation

Some meetings and training were done with the members who will work in this project. It is important to keep tracking of what was already done

In a meeting, the Product Engineering team has decided to create one issue for each training that already has done.
Pieces of training:

Agile
Git
Docker


Typo

Under the "View a cluster configuration as a JSON file" section, the text "JSON view is ready-only" appears

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

ID: 8ab17b71-1d88-c847-59e5-002a11915f67
Version Independent ID: 80916cf1-234f-37f0-150a-2f8fbaa16bf2
Content: Manage clusters - Azure Databricks
Content Source: databricks/clusters/clusters-manage.md
Service: azure-databricks
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

Thanks for bringing this to our attention. I've updated the doc and the changes will go live next time we publish.
#please-close


WIP: provisioning flow cleanup

[x] add centerChildren flag to Box2
[x] test signup flows
[x] test new error flows
[x] gen paperkey page
[x] use changeincrement
[x] waiting handlers on promise rpcs
[x] sagaize all of signup
[x] can back out of signup flow
[x] per platform default device names
[x] actual signup call
[x] test mobile
[x] split prs

replaced by 4 prs


FIX: Reorder set_and_wait to avoid race condition

set_and_wait currently accesses signal.tolerance on the first line. This is a problem because the tolerance property is wrapped with a raise_if_disconnected decorator, which prevents the whole method from being run if the EpicsSignal object has not finished connecting.
This happened to me with the AreaDetectorComponent objects. Since these are the only Component objects that are set as lazy=True, there was no guarantee that the connection finished in time for my set_and_wait call.
I expect that these epics_pvs utils should work even if the connection hasn't finished yet, so here I've reordered the method so that the put and get calls happen first, which call wait_for_connection on the _write_pv and _read_pv attributes respectively. This guarantees that the connection has happened in time for the tolerance check.

Ah!  This is probably the source of the failures we saw when we flipped the default, everything blew up, and then we flipped the default back with out chasing it to ground.
The test failures are failure to import pytest in the conda envs, restarted to see if it is a transient thing or of we need to dive down a packaging rabbit hole.

I think accidentally let this languish because some Travis builds were randomly failing. I re-ran the builds, and one more passed; the other failed on something unrelated.
Thanks @ZLLentz and sorry for the hold-up. Feel free to kick the Travis builds if this happens again. Do you have the permissions to do so?


@unocss/core types are not exported

I want create a custom preset by reuse some core types, but i can't import those types from @unocss/core/src/types.ts.

Maybe I can solve it by creating the preset in a fork project

@chenzhq just use:
import type { UnoGenerator } from '@unocss/core'
import { createGenerator } from '@unocss/core'


Merge branch 'master' into jstorm-runner at commit 727253e

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @kennknowles

retest this please

SUCCESS
--none--

FAILURE
--none--

FAILURE
--none--

@peihe @kennknowles what is the status of this PR?


[TEST] Testing issue autoclosure

This is a test of https://github.com/Expensify/Web-Expensify/pull/33496

Closing, this was a test, sorry for the ping Kevin


Have Runge-Kutta methods default to scan_stages=True with bounded while loops.

In particular, the bounded while loop would be part of _vector_tree_dot.
Prerequisite: for JAX to allow custom_vjp to be forward-mode autodifferentiated.

Done in v0.4.0!


Bump GHC and Cabal versions in Ubuntu install instructions

Not sure if there's a reason the install instructions were on 7.8, but I noticed the book is referencing 7.10 so thought perhaps it should be updated?

Thank you :)


Changed UUID of AdmittanceModels.jl.

related: https://github.com/rigetti/AdmittanceModels.jl/pull/13
related: https://github.com/rigetti/DiscreteExteriorCalculus.jl/pull/4

Hey @ajkeller34 the Travis CI for this repo currently shows a failed build for master after merging this PR. Is that because you squashed these two commits into one when merging to master or something? If so, could you please trigger a new build on Travis CI so the README shows passing tests? Thank you!


Failed to start web app

I am using mac, and I followed all the steps to set up the app.
pbiviz start used this command to start the app. And accessed the app in browser with the url https://localhost:8080
But app is not loading. I checked the console and saw below error.

plugin sample268168F1AD2D48B3953BF3D61A3C3B15_DEBUG
visual.ts:36 Visual constructor undefined
visual.ts:38 Uncaught TypeError: Cannot set property 'element' of undefined
    at new Visual (visual.ts:38)
    at Object.create (visualPlugin.ts:8)
    at (index):20

Verified in app.powerbi.com
It works well.
Closing as this is not relevant anymore.


Weekday adjustment to use Clarabel instead of ECOS

Description
doctor_visits signal smoothed_adj_cli has a bug where sometimes some percentage values are calculated to be abnormally big (90%+). We found out it's because of this ECOS solver used by default by cvxpy when calculating weekday adjustments.
Per their recent update:

CVXPY has used ECOS as the default solver for many years; however, it has known issues with performance and numerical stability in edge cases. Recently, a new solver, Clarabel, that improves the algorithm and implementation of ECOS has been under development.

Changelog

Force weekday adjustment to use Clarabel in all versions of cvxpy, not just 1.5+.

Fixes

Annoying doctor_visits bug

Since the python client bug is fixed, i reran the test actions.  Now test_weekday.py is failing because the expected output values have changed slightly.

You can cherrypick this commit if you want to fix just that CI issue 3559664a25bfab4c33b066a4c0d4c91441c47f60. It's included in the #1952 and (the same commit with a different hash is in) #1905.

weird!  @dshemetov do you know what caused that?

@melange396 I'm gonna guess it's related to clarabel solver being made default recently in cvxpy instead of ecos.

Yea, it's coming from differences in the solvers. Clarabel is a Rust implementation and possibly with algorithmic differences in the optimization procedures as well. Some details in this changelog.

ah yes, the default changed with the new version, and it bit them in those other PRs not long after the release.  This is an argument for using version pinning...  I will create a GH issue for that.

That would make sense; it's a thing to do after this has gone live and been run on the old data though, right? Added as a subtask on the Jira patch ticket
Judging by the effect on the test case, it should cause changes on the order of ~.1%, which is probably not worth worrying about too much. If anything the values should be closer to the true optimum.


Adjust yaml spec (obsdatain) according to the removal of the "_0000" on the file name.

Description
This PR adjusts the yaml obsdatain spec for the test_soca_letkf_split_solver test by removing the "_0000" suffix on the input file name. This is in accordance with jcsda-internal/ioda/pull/901.
Issue(s) addressed
See jcsda-internal/ioda/pull/901 for details.
Testing
No new tests, the test_soca_letkf_split_solver test covers the ioda change.
Dependencies

merge with jcsda-internal/ioda/pull/901
merge with jcsda-internal/ioda-data/pull/132

note to myself, and @kbhargava , this might require a small change in soca-science, not sure


fix: underscore crate name

Resolves #37
Changes

Adds the replace call as mentioned in the issue's comments
Includes corresponding unit and acceptance test

Does normal crates name liked normal-crate still work?
I am not sure if this is tested?

Unit test looks fine. But I am missing an acceptance test. I think we already have a test crate in the tests folder which could be extended.

Should I create another create-segment-path for instance ?  or just a sub-folder ?
EDIT: the issue is on the crate name, so the only acceptance test that could be done is by creating a new test crate
EDIT 2: the tests folder doesn't seem to be compiled anywhere

Yes just create another test crate.
I noticed this too so I already created a fix: https://github.com/ProbablyClem/utoipauto/pull/39

Unit test looks fine. But I am missing an acceptance test. I think we already have a test crate in the tests folder which could be extended.

Should I create another create-segment-path for instance ?  or just a sub-folder ?
EDIT: the issue is on the crate name, so the only acceptance test that could be done is by creating a new test crate
EDIT 2: the tests folder doesn't seem to be compiled anywhere

Yes just create another test crate.
I noticed this too so I already created a fix: #39

Ok so I will rebase once your fix is merged

@EvolveArt
PRs have been merged you can now merge main into your branch and create the acceptance test

@EvolveArt
PRs have been merged you can now merge main into your branch and create the acceptance test

Done, you can check. Not sure if that's the best way but it does provide a correct acceptance test.
I think acceptance tests could be refactored in another PR.

Please rename the folder and the name to folder_in_src The name is more descriptive.
Otherwise your pr looks pretty good. Thank you for your work :)

Done :) looking forward to help more, this crate has been really useful to simplify our utoipa doc generation

Note:
I am not sure if we should introduce a "main" testing crate where we could merge all things that need to be tested across multiple crates (like this pr).
But this has nothing todo with this pr.
Just a small note from me.
Note 2:
I do have a plan for creating a larger test crate that tests multiple things maybe we can combine the ideas.

@EvolveArt Could you rebase your pr one more time?
I fixed the ci a bit

@EvolveArt Could you rebase your pr one more time? I fixed the ci a bit

done

Thanks guy for your work


Fix glitch regarding /Q parameter

From format.com help:
/Q              Performs a quick format. Note that this switch overrides /P.
This is also consistent with the /P parameter description.

@HelderMagalhaes : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Make Google Maps show up

Need to create a new API key that matches the new bundle identifier

Fixed in f5dbc54


Send Keyboard Shortcut

RainLoop version, browser, OS:
RainLoop 1.11.3, Chrome 61.0.3163.100 (64 bit), Win 10 (1703)
Expected behavior and actual behavior:
It would be nice to have a shortcut for sending a mail. Like Ctrl + Enter default for gmail.
Steps to reproduce the problem:
Logs or screenshots:

https://github.com/RainLoop/rainloop-webmail/issues?utf8=✓&q=send shortcut
Close issue if fixed.

Thanks mate! Its working now.
But there must be setting in admin panel or could be enabled by default.


Custom auth documentation on wiki insufficient

Custom auth documentation on wiki insufficient

to guide one on how to implement custom auth in cloud code and on client with self-hosted parse-server
to guide one on how to migrate existing custom auth cloud code working on Parse.com to self-hosted parse-server

You can't really migrate custom auth from parse.com to hosted parse, but i agree the documentation could me clearer.
What do you need in particular? Using the provided auth services or developing new ones?

We have custom cloud code working on Parse.com that exchanges an auth token from our auth provider for a Parse session token. This code was implemented prior to revocable sessions and the app on Parse.com has not been migrated to revocable sessions. We are now working on migrating to self-hosted parse-server which only supports revocable sessions. The documentation for custom auth for Parse.com was never very good and it's even worse for parse-server. Trying to do both things at once, migrate to revocable sessions and self-hosted parse-server, with custom cloud code is proving to be a mess.

For parse-server you don't need custom cloud code for custom authentication if you wish to leverage the current infrastructure. Otherwise your code should work as is, (given that you upgrade to revocable sessions). The new oauth mechanism have to be triggered from the clients, not from Cloud Code.

Is there any documentation on the new oauth mechanism for mobile, non-web clients? Is there any documentation on how to support revocable sessions in custom cloud code?

See: http://parseplatform.github.io/docs/parse-server/guide/#oauth-and-3rd-party-authentication
And: https://parseplatform.github.io/Parse-SDK-iOS-OSX/api/Classes/PFUser.html#/c:objc(cs)PFUser(cm)registerAuthenticationDelegate:forAuthType:

Yeah, I've looked at the first link and that documentation on third-party auth is so thin. It describes the what but not the how.

There are plenty of examples here: https://github.com/ParsePlatform/parse-server/blob/master/spec/OAuth.spec.js

Hello,
I have been struggling now for a few days to get an 3rd party auth working.
Case is as followed:

I have an existing parse application with user accounts and want to move in an 3rd authentication & identity provider (keycloak).
I have three clients : one IOS App, one Android app and one web app in Angular 1 -> the web client uses Parse Javascript SDK. Keycloak provides their own javascript SDK
For now I am working on the web app authentication. Intention is when the web page is opens , the user get's redirected to the identity provider's logon page, gives id and pass, gets authenticated, provider redirects with oauth tokens..

After a lot of searching in the code and examples I have managed to make the following work.

I created a auth adapter and plugged it in the parse server. The adapter gets via authData the token to verify -> calls keycloak REST endpoint for verification and returns if ok . This works fine.
I have integrated the keycloak javascript API in the Angular app, the api gets initialized, user gets redirected, tokens are recieved and stored in session. This also works fine

Question is now , how do I link this with the existing account in Parse ?
I can create an authData object with access_code and call Parse.User.loginWith('keycloak', authdata). This works off course, but I always get a new user object.
I cannot figure out where the link is made with the existing user? I expected to get at some point the actual Parse user object back to continue to work in the web app.
I am obviously missing something here, but cannot find a lot of more info.
Can you help me ?
Thank you
Bart

Hey Bart,
You can use the user._linkWith('keycloak', authdata) if you have a user already authenticated, I'm not sure why the methods is _linkWith as it should be public now :)

Hi, thank you for the quick reply.
I understand. But where do I get the user object ? At this stage I am not yet logged in in parse.
I already tried searching for a user based on username:
`
var uquery = new Parse.Query("_User");
uquery.equalTo("username", $rootScope.currentUser.id);
    uquery.first().then(
      function(user) {
        user._linkWith('keycloak', authData);
      }
    );

`
I get somewhere in my logs a 400 error on a POST
POST https://dev2.api.learn2act.net/classes/_User/HnYaR1wINR 400 (Bad Request)
In my production app, the public read is false by the way. Not sure if this can work. Master key is also not an option in the client.
Any suggestions ?

Hi,
The 400 error shows up as followed in my server log:
3|dev-iam- | error: Error generating response. ParseError { code: 206, message: 'Cannot modify user hHuQ3k41Aj.' } code=206, message=Cannot modify user hHuQ3k41Aj. 3|dev-iam- | error: Cannot modify user hHuQ3k41Aj. code=206, message=Cannot modify user hHuQ3k41Aj.

Hi,
I found following section in the REST documentation:

If I create the user manually in the Dashboard and add the authData field manually ... calling the
/users endpoint with the same authData seems to work.
Now need to figure out how to incorporate this in a register-login-logout flow.

Hi
I have added more to the js guide
http://docs.parseplatform.org/js/guide/#linking-users

Awesome! I believe we can close that one now.

All those links are 404... Do you know where they have been moved to?

Im almost 100% certain that this part does not work (with google auth at aleast): https://docs.parseplatform.org/js/guide/#linking-1
You end up getting the following error:
 error: Cannot modify user JzDZZ4XjW5. ParseError: 206 Cannot modify user JzDZZ4XjW5.

I will create a separate issue for this.

@bartroelands Can you post the code you used to connect the server to Keycloak? This would help me and the community a lot about the implementation. Maybe I could improve the documentation with this info.


Capture URL as http.url

captures the request URL to the canonical attribute for this.

^ This has a high chance of leaking PII/sensitive data into the logs since http.url includes query parameters.

It was omitted on purpose in the first implementation @conradludgate

Yeah, we're going to publish this in a major version. @tl-rodrigo-gryzinski realised the same. We should consider having an easy opt-out, not sure the best approach for that just yet.
In an ideal world, REST APIs shouldn't have any PII in the path/query 😞

Should we make this opt-in instead?

I think it should be opt-in rather than opt-out - defaults should be as secure as possible and then the user can lower the barriers if they think it makes sense.

We've reverted this behaviour and introduced a SpanBackendWithUrl to compensate


Error thrown when process response from tsserver

Information
VIM version
NVIM v0.3.1-9-g90bb6fee7
Build type: Release
LuaJIT 2.0.5
Compilation: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -I/Users/chemzqm/lib/neovim/build/config -I/Users/chemzqm/lib/neovim/src -I/Users/chemzqm/lib/neovim/.deps/usr/include -I/Library/Frameworks/Mono.framework/Headers -I/usr/include -I/Users/chemzqm/lib/neovim/build/src/nvim/auto -I/Users/chemzqm/lib/neovim/build/include
Compiled by<EMAIL_ADDRESS>Features: +acl +iconv +jemalloc +tui
See ":help feature-compile"
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/usr/local/share/nvim"
PASTE JUST THE FIRST TWO LINES OF :version HERE.

MacOS
:ALEInfo
 Current Filetype:
Available Linters: []
  Enabled Linters: []
 Linter Variables:

 Global Variables:

let g:ale_cache_executable_check_failures = v:null
let g:ale_change_sign_column_color = v:null
let g:ale_command_wrapper = v:null
let g:ale_completion_delay = v:null
let g:ale_completion_enabled = 0
let g:ale_completion_max_suggestions = v:null
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'E'
let g:ale_echo_msg_format = '[%linter%] %s [%severity%]'
let g:ale_echo_msg_info_str = 'Info'
let g:ale_echo_msg_warning_str = 'W'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_history_enabled = 1
let g:ale_history_log_output = 1
let g:ale_keep_list_window_open = v:null
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_filetype_changed = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_lint_on_insert_leave = 0
let g:ale_linter_aliases = {}
let g:ale_linters = {'vim': ['vint'], 'swift': ['swiftlint'], 'typescript': ['tsserver'], 'markdown': [], 'scss': [], 'wxss': ['stylelint'], 'json': ['json5'], 'html': [], 'ruby': [], 'javascript': ['eslint'], 'css': ['stylelint'], 'python': ['pylint']}
let g:ale_linters_explicit = 0
let g:ale_list_window_size = v:null
let g:ale_list_vertical = v:null
let g:ale_loclist_msg_format = v:null
let g:ale_max_buffer_history_size = 20
let g:ale_max_signs = v:null
let g:ale_maximum_file_size = v:null
let g:ale_open_list = v:null
let g:ale_pattern_options = {}
let g:ale_pattern_options_enabled = 0
let g:ale_set_balloons = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 1
let g:ale_sign_error = '>>'
let g:ale_sign_info = '💡'
let g:ale_sign_offset = v:null
let g:ale_sign_style_error = v:null
let g:ale_sign_style_warning = v:null
let g:ale_sign_warning = '⚠'
let g:ale_statusline_format = v:null
let g:ale_type_map = v:null
let g:ale_use_global_executables = v:null
let g:ale_warn_about_trailing_blank_lines = 1
let g:ale_warn_about_trailing_whitespace = 0

What went wrong

Reproducing the bug
Happens sometime, I have no idea of how to reproduce.

This bug will be difficult to fix without steps to reproduce. If you see the error again, please copy and paste the text from the errors, as I can't copy and paste characters from a PNG file into a test case. It looks like a problem related to the clumsy way NeoVim gives you output from programs, combined with receiving just the right kind of JSON RPC data.

I tried to think my way through the error for a while, but I can't figure out what input what cause that error. If you come up with some steps for repeating the bug, let me know.


python311Packages.rapidfuzz: 3.9.3 -> 3.9.4

Description of changes
Diff: https://github.com/maxbachmann/RapidFuzz/compare/refs/tags/v3.9.3...v3.9.4
Changelog: https://github.com/maxbachmann/RapidFuzz/blob/refs/tags/v3.9.4/CHANGELOG.rst

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 324326 run on x86_64-linux 1

2 packages marked as broken and skipped:

python312Packages.pynitrokey
python312Packages.pynitrokey.dist

6 packages failed to build:

python311Packages.pdf2docx
python311Packages.pdf2docx.dist
python312Packages.autofaiss
python312Packages.autofaiss.dist
python312Packages.pdf2docx
python312Packages.pdf2docx.dist

114 packages built:

anime-downloader
anime-downloader.dist
atomic-operator
atomic-operator.dist
creds
creds.dist
emacsPackages.lsp-bridge
eyewitness
ibus-engines.uniemoji
instawow
instawow.dist
knowsmore
knowsmore.dist
ledger-autosync
ledger-autosync.dist
mealie
mealie.dist
mokuro
mokuro.dist
nitrokey-app2
nitrokey-app2.dist
osc-cli
osc-cli.dist
paperless-ngx
poetry
poetry.dist
poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
poetryPlugins.poetry-plugin-up
poetryPlugins.poetry-plugin-up.dist
puddletag
puddletag.dist
pynitrokey (python311Packages.pynitrokey)
pynitrokey.dist (python311Packages.pynitrokey.dist)
python311Packages.attacut
python311Packages.attacut.dist
python311Packages.autofaiss
python311Packages.autofaiss.dist
python311Packages.avwx-engine
python311Packages.avwx-engine.dist
python311Packages.cleo
python311Packages.cleo.dist
python311Packages.epitran
python311Packages.epitran.dist
python311Packages.fire
python311Packages.fire.dist
python311Packages.fuzzywuzzy
python311Packages.fuzzywuzzy.dist
python311Packages.jarowinkler
python311Packages.jarowinkler.dist
python311Packages.jiwer
python311Packages.jiwer.dist
python311Packages.levenshtein
python311Packages.levenshtein.dist
python311Packages.manga-ocr
python311Packages.manga-ocr.dist
python311Packages.nkdfu
python311Packages.nkdfu.dist
python311Packages.panphon
python311Packages.panphon.dist
python311Packages.rapidfuzz
python311Packages.rapidfuzz.dist
python311Packages.ssg
python311Packages.ssg.dist
python311Packages.syncedlyrics
python311Packages.syncedlyrics.dist
python311Packages.thefuzz
python311Packages.thefuzz.dist
trytond (python311Packages.trytond)
trytond.dist (python311Packages.trytond.dist)
python311Packages.videocr
python311Packages.videocr.dist
python312Packages.attacut
python312Packages.attacut.dist
python312Packages.avwx-engine
python312Packages.avwx-engine.dist
python312Packages.cleo
python312Packages.cleo.dist
python312Packages.epitran
python312Packages.epitran.dist
python312Packages.fire
python312Packages.fire.dist
python312Packages.fuzzywuzzy
python312Packages.fuzzywuzzy.dist
python312Packages.jarowinkler
python312Packages.jarowinkler.dist
python312Packages.jiwer
python312Packages.jiwer.dist
python312Packages.levenshtein
python312Packages.levenshtein.dist
python312Packages.nkdfu
python312Packages.nkdfu.dist
python312Packages.panphon
python312Packages.panphon.dist
python312Packages.rapidfuzz
python312Packages.rapidfuzz.dist
python312Packages.ssg
python312Packages.ssg.dist
python312Packages.syncedlyrics
python312Packages.syncedlyrics.dist
python312Packages.thefuzz
python312Packages.thefuzz.dist
python312Packages.trytond
python312Packages.trytond.dist
python312Packages.videocr
python312Packages.videocr.dist
spotdl
spotdl.dist
streamrip
streamrip.dist
sublime-music
sublime-music.dist
ulauncher
ulauncher.dist


guide: Data Versioning

Main page to review: Data Versioning page (data-versioning.md)

[ ] Replace some links from /doc/use-cases/versioning-data-and-models to here (doc/user-guide/data-versioning) ?

Pages to review:

Data Management guide (index.md) - https://dvc-org-data-mgmt-cli-s-hk2oqu.herokuapp.com/doc/user-guide/data-management
+ Data Versioning page (<cards> section) - https://dvc-org-data-mgmt-cli-s-hk2oqu.herokuapp.com/doc/user-guide/data-management/data-versioning

My take on this one: in its current form, it seems too similar to what's in #4042, so I would probably close this one and focus on that one.

Closing this to focus on #4042. @jorgeorpinel if there are some useful things here, please feel to move them to a different PR.


Revisit the assertions libraries

As discussed here, I think we're all having some mixed feelings about Spectral as the library we want to use for assertion testing. It can be a little verbose in lieu of simpler assertions (like the core assert!/assert_eq! macros). Now might be a good time to decide.
/cc @jlegrone and @glyn

There's a lot to be said for avoiding unnecessary dependencies, so I'd favour using core macros unless the code becomes a lot more verbose, in which case it might be worth adopting another library.


Enable jumping between menus using left-right arrow keys

If there is no sub-menu to a menu-entry, I would like to switch to the next menu when I press the left/right arrow keys. Also when I'm on the last nesting level in a sub-menu, I'd like to switch to the next main menu entry. If the said feature already exists it does not work for me on Ubuntu 14.04 using Atom version 0.187.0.
To clarify what I'm saying:

In the above scenario, I would like to be able to switch to the "Selection" menu when I press the right arrow key, other than when I press it on the "developer" menu-entry, which will open the sub-menu like it already does. However, inside the developer sub-menu also pressing the right arrow key does not take me to "Selection".
Another such issue is that once I've opened a menu with "alt+ ", say "View" with alt+V, I'm not able to open selection with "alt+S" unless I close the view menu first.
Thanks.
EDIT:- It's UbuntuMATE 14.04, so I'm in a MATE-desktop environment.

What shell are you using with Ubuntu? To your first issue, I can't reproduce this in Unity – pressing arrow keys switches between menus as expected.
The second issue is unrelated, but is consistent across all applications I tested on Unity, so that is not an Atom bug.

I'm using UbuntuMATE 14.04.
For the second issue, you're right, it does seem to be the case accross all third party applications I have installed. I'll edit the issue to reflect that. However, I tried the arrow keys in sublime text and Firefox, and they do switch menus as expected. Not in Atom though. So the arrow key issue is specific to Atom for me.

Just wanted to mention that this works for me on OSX.

I am getting the same symptoms as the OP.
Left / right arrow keys do not change from one main (IE Edit <===> view) menu to the next.  They will open/close, move in and out of submenus (IE: Edit <==> Edit/Lines)
Moving between main menus using arrow keys works for the Terminal program, so this does seem to be atom specific behaviour.
Environment info:
Fedora 21
Linux 3.18.9-200.fc21.i686 #1 SMP Mon Mar 9 15:48:51 UTC 2015 i686 i686 i386 GNU/Linux
atom 0.187.0-abf368e
Installed Packages
# User
tabs-to-spaces, v0.8.1

# Dev
No dev packages

@mMerlin Are you running GNOME 3?

@mnquintana Yes,
$ gnome-shell --version
GNOME Shell 3.14.3

It works on OS X and Unity, because there the menu isn't handled by Atom/Chromium but the Desktop Environment.

related bug:
https://github.com/atom/atom/issues/4057

I decided to give Atom a try again, but I'm afraid I'm having the same problem with Atom 1.0.5 on Windows 8.1. Have not tried it on Linux this time though. Thought I should let you guys know.
@kolAflash alt + {letter} does work for me. Would you possibly be having a workaround for the arrow keys as well?
/cc @mnquintana @izuzak

@rootAvish No, sorry. I don't have a workaround for that.

+1

:+1:

Same thing happens when running gnome-shell, seems like a pretty basic ommision for keyboard navigation.

@mnquintana Just want to say that this is still not fixed in the latest version for me.

@rootAvish Thanks for reporting back :bowing_man: I have reopened the issue and added the Windows label because I can reproduce this on Windows as well.
I'm sorry that the bot closed the issue without also adding a warning comment. There is an open issue for this on the bots repository /cc: https://github.com/probot/stale/issues/26

Every few months I will try atom, or something based on it like VS code, and then see this behaviour is still broken - next time I hit it, I will double check this ticket is still open.

Does anybody know where the code for menus + keyboard handling is ?    Surely this isn't hard to fix.

This is indeed an issue with Electron as it implements its own menu system instead of using the native OS menu behavior.  You can read more about it here: https://github.com/electron/electron/issues/2504.
I can also reproduce this issue on Windows with both Atom and VS Code so it's definitely not specific to Atom or Linux.

I get the same issue in Fedora 28+ and in Windows 9 (sorry 10 forgot that they skipped 9 due to there genius)

hey, same issue w/ atom 1.45.0 on manjaro cinnamon. should i open another bug on electron? this one seems to be closed w/o having been fixed https://github.com/electron/electron/issues/2504

People in the other bug mention it being an accessibility issue.
Maybe a way to raise visibility would be to add a PR for their accessibility docs, for blind people mentioning that Alt to show menu shortcuts doesn't work and left / right navigation is not implemented either.


Create file saved_parameters.pkl when it is non existing yet in the reuploading classifier example

Dear authors,
When playing around with the example reuploading_classifier for another project, I realized that if the file saved_parameters.pkl does not exist yet the code complains and crashes.
I believe it means just simply changing the line checking the file. That would be nice to see this changed, so that if the file does not exist yet the classifier creates it on the fly.
Best
Julien

@jubaglio thanks for reporting this issue. @AdrianPerezSalinas could you please take care?

I have a very un-optimized solution, though, in the file main.py. After the line
ql = single_qubit_classifier(dataset, layers)  # Define classifier
I've written
try:
    with open('saved_parameters.pkl', 'rb') as f:
        # Load previous results. Have we ever run these problem?
        data = pickle.load(f)
    try:
        parameters = data[dataset][layers]
        print('Problem solved before, obtaining parameters from file...')
        print('-'*60)
    except:
        print('Problem never solved, finding optimal parameters...')
        result, parameters = ql.minimize(
            method='l-bfgs-b', options={'disp': True})
        data[dataset][layers] = parameters
        with open('saved_parameters.pkl', 'wb') as f:
            pickle.dump(data, f, pickle.HIGHEST_PROTOCOL)
except IOError:
    print('Problem never solved and file not existing yet, finding optimal parameters...')
    data = {}
    data['3_circles'] = {}
    data['circle'] = {}
    data['4_squares'] = {}
    data['square'] = {}
    data['crown'] = {}
    data['tricrown'] = {}
    data['wavy_lines'] = {}
    open('saved_parameters.pkl', 'a').close()
    result, parameters = ql.minimize(
        method='l-bfgs-b', options={'disp': True})
    data[dataset][layers] = parameters
    with open('saved_parameters.pkl', 'wb') as f:
        pickle.dump(data, f, pickle.HIGHEST_PROTOCOL)

Hi @jubaglio , thank you very much or pointing this out. I have not seen the issue yet, hopefully I can do it tomorrow, but I will take you advice into account.
Let me understand what happens. If the file does not exist, it just crashes, isn't it? On the other hand, if it exists but the problem has never been solved, it works properly. So what we should do is just to avoid this crash?

Hi @AdrianPerezSalinas,
If there is no file, I get this:
Traceback (most recent call last):
File "./main.py", line 47, in
main(**args)
File "./main.py", line 23, in main
with open('saved_parameters.pkl', 'rb') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'saved_parameters.pkl'
Now if a file is there, but empty (as if I do a touch to create it):
Traceback (most recent call last):
File "./main.py", line 47, in
main(**args)
File "./main.py", line 25, in main
data = pickle.load(f)
EOFError: Ran out of input
That's why my solution is simply to create the file the first time the program is called in case the file does not exist yet, and dump the data structure into it.

Hi @jubaglio , I have opened a PR #401 with the changes you suggest. I did it in a much simpler way, with just a pair of lines. Let me know if it works for you!

Hi @AdrianPerezSalinas ,
It still fails:
Traceback (most recent call last):
File "./main.py", line 29, in main
parameters = data[dataset][layers]
KeyError: 'tricrown'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./main.py", line 49, in
main(**args)
File "./main.py", line 36, in main
data[dataset][layers] = parameters
KeyError: 'tricrown'
I believe we should also initialize the dataset blocks in data, as:
    data['3_circles'] = {}
    data['circle'] = {}
    data['4_squares'] = {}
    data['square'] = {}
    data['crown'] = {}
    data['tricrown'] = {}
    data['wavy_lines'] = {}

With this added to your fix, it works and is indeed much simpler than my solution!
Best
Julien

Hi @AdrianPerezSalinas
It works, thnaks!


Ark RCON does not work

# Bug Report
An integral part of the Ark module rcon, does not work. Shutdowns and scheduled commands are not possible. I can use a tool such as Arkon to connect to the instance by grabbing the password from the ini and issue saveworld and doexit without issue. It seems like the built in rcon client does not function properly or does not connect from the start.
## System Information

Ubuntu 16.04 headless
AMP Titan Mainline Local install
v<IP_ADDRESS>, built 18/06/2019 09:23

## Symptoms
*** What are you trying to do?**
Stopping the server, issuing saveworld / any console command.
*** What are you expecting to happen?**
Commands to be sent to the server
*** What is actually happening? (Nothing is not an acceptable answer!)**
Seems like it fails to connect or losses its connection and never reconnects. Have not seen it function even from a cold start.
Reproduction
*** Starting from a clean install, how do you reproduce the issue that causes the symptoms described above?**
Clean install, created new instance, setup Ark server, ran the server and rcon never functions.
EDIT: Added build/version number

Can you give us what build exactly your on? Titan Mainline isn't really helpful, because you might be running day old code or month old code.
The build date and values should be at the bottom left of your instance.

Can you give us what build exactly your on? Titan Mainline isn't really helpful, because you might be running day old code or month old code.
The build date and values should be at the bottom left of your instance.

Sorry about that, current build is v<IP_ADDRESS>, built 18/06/2019 09:23

Something is wrong with ARKs RCON in general. It's supposed to be the standard Source RCON protocol - but it absolutely refuses to respond.

Fixed in a recent release.


Content testing (topics+ test) French

Content sheet Topics French

Row 58 B
Two subtitles with the same name
we have two PREVENEZ LA COVID-19 in the topic tab
Content sheet tests French
Row 2 G
There should be three links in the app. the third link is missing.

The first issue fixed by Melika. Please test on the latest update.

Second issue to be communicated to Dr. Cleo.

58 B Issue: Fixed
2 G Issue: Need to be discussed with Dr. Cleo

Thanks Juveriya.

2G - Emailed client. Waiting for feedback.

Included in code and tested


work the datacenter and services tags on the server view? (just underneath the bar graph stats)

...and subsequently reflect these within the Isotope filters?
...and perhaps even pre-populate this automatically based on common running services? e.g. Nginx, MySQL, Hadoop

here underneath the current load on the server detail

on it

done


Introduce "watch" option to webpack config

To reduce compile time, the watch option is useful.
If we introduce this process to samples, we need to support web-pack bundle for sample files at first.

Related: https://github.com/actnwit/RhodoniteTS/issues/704

@KaYaF Do you need this issue now?

@emadurandal I think yes.
I think it would be more efficient to watch the ./src and ./samples at the same time.


[docker] Docker build failures

For whatever reason, when I attempt to build, I get the following error.
I build with the --no-cache option.
This error seems specific to my system, and will work as long as I take USER dev out (which however leads to other minor issues that can be worked around).
I can't figure out what's different about my setup that's causing this, so let me know if anyone has an idea.
Step 28/36 : RUN groupadd dev     && useradd --create-home -g dev dev     && usermod -p '*' dev     && passwd -u dev
 ---> Running in fb719128e4e7
passwd: password expiry information changed.
Removing intermediate container fb719128e4e7
 ---> 65f00193577e
Step 29/36 : USER dev
 ---> Running in 64e1bfb1be11
Removing intermediate container 64e1bfb1be11
 ---> 329a005b4c9b
Step 30/36 : COPY sw/vendor/rustup/rustup-init.sh /tmp/rustup-init.sh
 ---> f988406b4660
Step 31/36 : RUN /tmp/rustup-init.sh -y --default-toolchain ${RUST_VERSION}
 ---> Running in da594716d393
/bin/sh: /tmp/rustup-init.sh: Permission denied
The command '/bin/sh -c /tmp/rustup-init.sh -y --default-toolchain ${RUST_VERSION}' returned a non-zero code: 126

After cleaning out my cache and building from scratch I'm having the same probelm, now too. I looked into this further and got the image to build by moving the copying of therustup-init.sh and python-requirements.txt into the Docker image (and setting the file permissions on those files) before the USER dev stage where the execution privileges change (after some debugging I realized that when the rustup-init.sh and python-requirements.txt the are owned by root not dev (see below).

However, while the build succeeds, when I go to run the container with docker run -ti -v $(pwd):/home/dev/src  -e DEV_UID=$(id -u) -e DEV_GID=$(id -g) opentitan:latest bash (to set the correct user and group ID to that of my host) it hangs at the exec gosu dev:dev /bin/bash -c "$@" line in the util/container/start.sh script.
However if I run the following command to launch the container docker run -ti -v $(pwd):/home/dev/src  opentitan:latest bash (NOT setting the user and group ID of the container to match my host) it enters the container no problem.
Some very strange behavior ... still debugging ...
Here is my local branch with the fix I made, in case anyone wants to play around with it more: https://github.com/timothytrippel/opentitan/tree/fix-docker-image

#8597 should fix the permission/user part of the problem.
I'm unable to reproduce the long delay when running the container that you experienced, @timothytrippel. Can you retry with #8597 applied? And can you check the subshell commands to ensure it's not them taking a long time?
time echo docker run -ti -v $(pwd):/home/dev/src -e DEV_UID=$(id -u) -e DEV_GID=$(id -g) opentitan:latest bash

Confirmed, above is just a local issue, closing this.


Add challenge questions connector config

This PR includes:

Bump identity-apps version
Add challenge questions connector config to the toml

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7658275456

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7658275456
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7658609102

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7663053758

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7663053758
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7663475596

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7663475596
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7663948102

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7663948102
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7664012437

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7664012437
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7665338887

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7665338887
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7671415759

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7671415759
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7692305680

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7692305680
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7707303511

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7707303511
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7724252887

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7724252887
Status: success


build(changelogs): Use automated changelogs from Craft

See https://github.com/getsentry/craft#changelog-policies for details
Moved 'versioning policy' to README so Craft does not push it down.

@untitaker @mitsuhiko


Publish awesome-hugo.dev to a "real" website instead of a GitHub redirect

Rework the list into a real Hugo website and let people end up on a nice (interactive) site. You could host on Netlify. Add a search. Add a form to add items. Don't make it GitHub-centric.
I saw you have a domain awesome-hugo.dev - let's use it ;)

I like the idea, but I haven't had time lately.

I'll invest, please keep this open ;)


QR scan doesn't scan QR codes.

Hi, I implemented this. And when I try to scan it does nothing. What could be the problem?
And when I open the scanner, the camera is wider than the overlay, is it possible to fix that?

Could you please provide more information (logs and screenshot), so that we can identify the issue

Hi, I forgot to mention that this only happens when I host (firebase hosting) the app, and use mobile browser. When I install the app on my Android it works perfectly fine.

Hi, I forgot to mention that this only happens when I host (firebase hosting) the app, and use mobile browser. When I install the app on my Android it works perfectly fine.

it did happened to me the same , also it shows a black square in front of the camera preview

Hi, I forgot to mention that this only happens when I host (firebase hosting) the app, and use mobile browser. When I install the app on my Android it works perfectly fine.

@AlexMunth Dude, have you solved this problem? I also scanned in the browser without any response


On idea info page autor is not fetching

after merging #16
authors name is not fetching in idea info page checkout the backend

id of the admin is changed
rechange the id to admin
@Trisha-tech


Fix linting issues

Running npm run lint currently reports a number of issues.
Some of this is in our newly added code, and some is in existing code.
Can we address this so that running the lint command gives no issues (it could be through lint config).
It shows that we have paid attention to code quality and puts us in a better position for handing over the code.

Current issues:
lint_output.txt

I have fixed 3191 issues. 39 issues left.


Deprecation Warning

Hi, Thanks for this,  I get this in console log.  I've pulled the latest:
Warning: React.render is deprecated. Please use ReactDOM.render from require('react-dom') instead.

@msimonc thanks for filing the issue. I have updated the repo to use react-dom here https://github.com/alicoding/react-webpack-babel/commit/f8797b847598f1b9f7dbbc2828179f564b40a3ea


doc: GPIO_INT config option dependencies aren't clear

As reported on the dev mailing list, there's a dependency between GPIO_INT_EDGE and GPIO_INT that's not clear. Can this dependency be added to the Kconfig file?
I don’t understand what the documentation says for the flags:
GPIO_INT
   GPIO pin to trigger interrupt.

GPIO_INT_EDGE
   Do Edge trigger.

So, what I’m concluding is that you need to specify GPIO_INT to enable interrupts on the pin and then another flag like GPIO_INT_EDGE to say when in the signal processing you want the interrupt to occur? If so... makes sense (now). I thought saying GPIO_INT_EDGE was enough to enable interrupts and to say they should occur on edge. My bad.

I agree. But I think it more intuitive to remove the flag to enable GPIO_INT in the documentation and leave it implicit:
/** @cond INTERNAL_HIDDEN */
/** GPIO pin to trigger interrupt. */
#define GPIO_INT                (1 << 1)
** @endcond */

/** GPIO pin trigger on level low or falling edge. */
#define GPIO_INT_ACTIVE_LOW     ((0 << 2) | GPIO_INT)

/** GPIO pin trigger on level high or rising edge. */
#define GPIO_INT_ACTIVE_HIGH    ((1 << 2) | GPIO_INT)
.
.
.

And using Kconfig to set this flag can be a bit annoying. I had to do something like this to set 16 GPIO ports for lack of API option.

Should be fixed by #16648 (not yet on master). Only the configuration flags which are supposed to be used by the application code have doxygen documentation. All of these compound flags explicitly include GPIO_INT_ENABLE.

This has been addressed in the GPIO rework


Add script to poll New Relic for alert violations and post them to SQS

This script will poll New Relic for alert violations that have occurred within a threshold of time and send them as messages to SQS.
The threshold is important because New Relic considers violations either open or closed, and reports both a time when the violation began and its duration. The intent is that if this polling script is run every 60 seconds, the threshold for new violations should be 60 seconds, that way only violations that have occurred within the last 60 seconds get posted to SQS and enter our alert pipeline.
Run it like:
cfgov/alerts/poll_new_relic.py --queue_url QUEUE_URL --aws_access_key_id ACCESS --aws_secret_access_key SECRET  --newrelic_token TOKEN --threshold 60

The message body for New Relic alert violations will look like:
[Alert name] - [New Relic product name] [Alert type] [Condition that was violated]

For example,
cf.gov test monitor - Synthetics Monitor Check Failure

Additions

Script to poll New Relic for alerts

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Assignee tool :arrow_right:

I've added a --filter argument that will take a regular expression to filter New Relic policy names for violations.
I would anticipate using this for cf.gov-related alerts by added --filter cf.gov and ensuring that all our cf.gov-related alerting policies have "cf.gov" in their name. That way we only get alerts that are relevant to cf.gov.


cloud limit column size in MetaService and support handle rowset schema in backend side

in local mode we limit column size in commit_txn phase, in cloud mode we limit it in write_schema_dict which in commit_rowset phase
add config to decide whether fill schema in BE side or MS side

run buildall

run buildall

run buildall

run buildall

LGTM


[ffmpeg] avformat's dependency is not copied to the build directory

avformat-57.dll depends on libbz2.dll which is not being deployed to the build directory by VS/MSBuild.
The issue was spotted on x64 debug build (in x64 release build the library is deployed correctly).

I noticed that too. Debug version of avformat-57.dll linked with release version of libbz2

waiting for pr #4250 to resolve.

Thanks for posting this issue. PR #6570 has been merged. This issue has been fixed. So I am closing this issue for now.