Custom Error Log In CodeIgnitor

I have observed an issue with codeignitor db_query and loader file in the core files.
If we ask to load any view file which does not present in the web folder then it will not provide any log in the log file location as there is no log_message in that function and it is being handled by the function so I the user puts try catch or error logging it will not generate any log as the script ended gracefully.
On the other side in query builder class if there is any query error again it will just log that message and display the error but let if the user wants the custom logs like with error, he want to log few session variables.The user has to change the core function for the query_builder.
I think to avoid this we should also throw the appropriate Throwable object so that the user can customise the use if he wants otherwise the core logging will work

I have observed an issue with codeignitor db_query and loader file in the core files.

It's CodeIgniter.

If we ask to load any view file which does not present in the web folder then it will not provide any log in the log file location as there is no log_message in that function and it is being handled by the function so I the user puts try catch or error logging it will not generate any log as the script ended gracefully.

Man ... I had to read this 3 times and I still barely understand what you're trying to say. Yes, there's no log message but there's an entire error screen triggered when you try to load a non-existing view.

On the other side in query builder class if there is any query error again it will just log that message and display the error but let if the user wants the custom logs like with error, he want to log few session variables.The user has to change the core function for the query_builder.

Again, I barely understand what you're trying to say, and again I don't see a problem.

I think to avoid this we should also throw the appropriate Throwable object so that the user can customise the use if he wants otherwise the core logging will work

To avoid what?

Let me give a clear scenario
suppose we provide the table name and view page name dynamically by the session and if something among the both is wrong(either a wrong table or a view page) so just to get the error log, we have to add a log message in the view and change the log message(custom log message) in the query builder. which will change the log message for every query.
To Avoid this we should throw a Throwable(Exception) object on such errors(wrong query) so that developer can catch it in try and catch block and provide his own custom log.
With this we can add the as many different custom log even for every function(depending on the need).
Please let me know if the scenario if clear?

Quite ridiculous, but clearer, yes. I have to say, custom logs are probably the weakest argument for exceptions.
But regardless, transforming all error conditions to exceptions is a plan that we're already committed to for version 3.2.0. See #4189 for example.


Validation set division on HICO-DET

Hi@fredzzhang,
We recently wanted to carve out validation sets in HICO-DET. We use the following code:

But we run into a lot of obstacles:
（1）Run directly
Namespace(alpha=0.5, aux_loss=True, backbone='resnet50', batch_size=16, bbox_loss_coef=5, box_score_thresh=0.2, cache=False, clip_max_norm=0.1, data_root='./hicodet', dataset='hicodet', dec_layers=6, device='cuda', dilation=False, dim_feedforward=2048, dropout=0.1, enc_layers=6, eos_coef=0.1, epochs=20, eval=True, fg_iou_thresh=0.5, gamma=0.2, giou_loss_coef=2, hidden_dim=256, lr_drop=10, lr_head=0.0001, max_instances=15, min_instances=3, nheads=8, num_queries=100, num_workers=2, output_dir='checkpoints', partitions=['train2015', 'test2015'], port='1234', position_embedding='sine', pre_norm=False, pretrained='', print_interval=500, repr_dim=512, resume='/home/quan107552101247/upt/checkpoints/jokex2/ckpt_47040_20.pt', sanity=False, seed=66, set_cost_bbox=5, set_cost_class=1, set_cost_giou=2, weight_decay=0.0001, world_size=1)
=> Rank 0: continue from saved checkpoint /home/quan107552101247/upt/checkpoints/jokex2/ckpt_47040_20.pt
  0%|                                                  | 0/7527 [00:00<?, ?it/s]Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/multiprocessing/spawn.py", line 116, in spawn_main
    exitcode = _main(fd, parent_sentinel)
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/multiprocessing/spawn.py", line 126, in _main
    self = reduction.pickle.load(from_parent)
  File "/home/quan107552101247/upt/pocket/data/base.py", line 167, in __getattr__
    if hasattr(self.dataset, key):
  File "/home/quan107552101247/upt/pocket/data/base.py", line 167, in __getattr__
    if hasattr(self.dataset, key):
  File "/home/quan107552101247/upt/pocket/data/base.py", line 167, in __getattr__
    if hasattr(self.dataset, key):
  [Previous line repeated 993 more times]
RecursionError: maximum recursion depth exceeded
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/multiprocessing/spawn.py", line 116, in spawn_main
    exitcode = _main(fd, parent_sentinel)
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/multiprocessing/spawn.py", line 126, in _main
    self = reduction.pickle.load(from_parent)
  File "/home/upt/pocket/data/base.py", line 167, in __getattr__
    if hasattr(self.dataset, key):
  File "/home/upt/pocket/data/base.py", line 167, in __getattr__
    if hasattr(self.dataset, key):
  File "/home/upt/pocket/data/base.py", line 167, in __getattr__
    if hasattr(self.dataset, key):
  [Previous line repeated 993 more times]
RecursionError: maximum recursion depth exceeded
  0%|                                                  | 0/7527 [00:05<?, ?it/s]
Traceback (most recent call last):
 ......
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 199, in spawn
    return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 157, in start_processes
    while not context.join():
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 118, in join
    raise Exception(msg)
Exception:

-- Process 0 terminated with the following error:
Traceback (most recent call last):
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 872, in _try_get_data
    data = self._data_queue.get(timeout=timeout)
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/queue.py", line 179, in get
    self.not_empty.wait(remaining)
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/threading.py", line 306, in wait
    gotit = waiter.acquire(True, timeout)
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/utils/data/_utils/signal_handling.py", line 66, in handler
    _error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 3874908) exited unexpectedly with exit code 1. Details are lost due to multiprocessing. Rerunning with num_workers=0 may give better error trace.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
......
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/tqdm/std.py", line 1195, in __iter__
    for obj in iterable:
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 435, in __next__
    data = self._next_data()
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1068, in _next_data
    idx, data = self._get_data()
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1024, in _get_data
    success, data = self._try_get_data()
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 885, in _try_get_data
    raise RuntimeError('DataLoader worker (pid(s) {}) exited unexpectedly'.format(pids_str)) from e
RuntimeError: DataLoader worker (pid(s) 3874908, 3875008) exited unexpectedly

(2)num-workers==0
Traceback (most recent call last):
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 19, in _wrap
    fn(i, *args)
  File "/home/upt/main.py", line 107, in main
    ap = engine.test_hico(test_loader)
  File "/usr/local/anaconda3/envs/pocket/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 26, in decorate_context
    return func(*args, **kwargs)
  File "/home/upt/utils.py", line 170, in test_hico
    inputs = pocket.ops.relocate_to_cuda(batch[0])
  File "/home/upt/pocket/ops/relocate.py", line 63, in relocate_to_cuda
    return [relocate_to_cuda(item, ignore, device, **kwargs) for item in x]
  File "/home/upt/pocket/ops/relocate.py", line 63, in <listcomp>
    return [relocate_to_cuda(item, ignore, device, **kwargs) for item in x]
  File "/home/upt/pocket/ops/relocate.py", line 71, in relocate_to_cuda
    raise TypeError('Unsupported type of data {}'.format(type(x)))
TypeError: Unsupported type of data <class 'PIL.Image.Image'>
···

How do we solve it?

Thank you so much!
yaoyaosanqi.

Hi @yaoyaosanqi,
The error was caused by the incorrect data type for the input image. In the DataFactory class, you will find a transform applied to the network inputs here. This converts a PIL image to a tensor and does image normalisation. After doing the dataset split, I don't think you applied the same transform to the new datasets.
Fred.

It ran successfully!
But there is a small issue that num-workers must be 0 to run, can I modify what settings make num-workers greater than 0 and work properly?
In the work we've done, the resnet50-based model is very close to the resnet101-based model in the non-rare category (about 0.4 mAP), but the rare class resnet101 outperforms resnet50 by about 1mAP. What could be due to? (The same goes for UPT and QPIC)
We noticed that some work replacing Resnet 50 with resnet101 was a huge boost for non-rare classes rather than rare classes.(The same goes for CDN and SDT)
Thanks！
yaoyaosanqi.

Hi @yaoyaosanqi,

But there is a small issue that num-workers must be 0 to run, can I modify what settings make num-workers greater than 0 and work properly?

I'm not sure what's causing problems with more than one workers. Does setting the worker number to zero affect the speed?

In the work we've done, the resnet50-based model is very close to the resnet101-based model in the non-rare category (about 0.4 mAP), but the rare class resnet101 outperforms resnet50 by about 1mAP. What could be due to?

I don't have a definitive answer for this. But intuitively, having a backbone with higher depth could help extract richer features for the rare classes. The non-rare classes, on the other hand, benefit from the large amount of data, thanks to which features can already be learned effectively. This then leaves less space for improvement.
Fred.

We have decided to temporarily set aside this issue and continue researching it in the future if needed.
Thank you!


Clarify documentation for datadog dev proceedure

What
If you don't have dev keys (or have expired ones) in your environment when enabling datadog you will get strange errors due to a terraform bug and a datadog bug. This clarifies the documentation to ensure you won't forget to publish keys.
How to review
Check that the procedure described makes sense.
Who can review
You! Certainly not @jonty.

lgtm, will merge. Was there anything we could do from make or in the pipeline to verify the keys in use before we actually try to use them?

@benhyland We could curl an endpoint that returns a 200|403 at the beginning of the pipeline, or in the Makefile when turning Datadog on. I'm not sure if this is something that impacts anyone regularly enough to be worth implementing - perhaps worth keeping an eye out for and revisiting if it hurts someone again?


Asynchronous actions2 functions are being implemented as type: 'classical' preventing exits.

Node version: 12.16.2
Sails version (sails):  1.5.0
ORM hook version (sails-hook-orm): ^3.0.2
Sockets hook version (sails-hook-sockets): ^2.0.1
Organics hook version (sails-hook-organics): ^2.2.0
Grunt hook version (sails-hook-grunt): ^2.2.0
Uploads hook version (sails-hook-uploads): N/A
DB adapter & version (e.g. sails-mysql@5.55.5<EMAIL_ADDRESS>Skipper adapter & version (e.g. skipper-s3@5.55.5): ^0.9.1

My asynchronous actions/helpers/etc implementationType are being set to classical implementation when I use an async function. This prevents them from using exits and work arounds have to be used to make them asynchronous.
module.exports = {
  friendlyName: 'Testing async func',
  description: '',
  inputs: {
  },
  exits: {
  },

  fn: async function (inputs, exits, env) {
    console.log(Object.keys(this), this.arguments, inputs, exits, env && Object.keys(env))
    return exits.success('ok yep.')
  }
};

If I use implementationType: 'analog', within the actions object, it works just as expected. IE
module.exports = {
  friendlyName: 'Testing async func',
  description: '',
  implementationType: 'analog',
  inputs: {
  },
  exits: {
  },

  fn: async function (inputs, exits, env) {
    console.log(Object.keys(this), this.arguments, inputs, exits, env && Object.keys(env))
    return exits.success('ok yep.')
  }
};

Here is synchronous file with async functions...
async function asyncTest(inputs, exits) {
    console.log(Object.keys(this), this.arguments, inputs, exits, env && Object.keys(env))
    // ... await async stuff in here ...
    return exits.success('ok yep.')
}

module.exports = {
  friendlyName: 'Testing async func',
  description: '',
  inputs: {
  },
  exits: {
  },

  fn: function (inputs, exits, env) {
    return asyncTest(inputs, exits)
  }
};

Where does this get determined? Should I just include the implementationType parameter in all action2 files? There is no documentation on this parameter or why this is happening.

Hi @grantbi, here is @mikermcneil's response to your questions:
You have two options:

//(1) analog
fn: function(inputs, exits) {

	let harry = await User.findOne(1);
	exits.success(harry.fullName);
}

//(2) classical
fn: async function(inputs) {

	let harry = await User.findOne(1);
	return harry.fullName;
}

// want to use `await` in an analog implementation?
fn: function(inputs, exits) {

	(async ()=>{
		let harry = await User.findOne(1);
		return harry.fullName;
	})().then((resultMaybe)=>{
		exits.success(resultMaybe);
	})
	.catch((err)=>{
		exits.error(err);
	});
}

// ... but why?  Just use classical and then you can use await.

The best way is to use classical.  This is what @mikermcneil recommends and how sails actions are generated.
// What if you have to use a library that doesn't support `await`?
// (we'll pretend for a sec that User.findOne doesn't support await even though it does)
// ==========================================================================================

// this approach is ok.... (there are better ways though)
fn: function(inputs, exits) {

  // it's not great that we have other code for the action (e.g. responding through the exits)
  // that is entangled inside of this callback.  it would be better if we could move it down into
  // safe, normal, await-enabled-land.  (outside of the dark scary _∏_)
	User.findOne(1).exec((err, harry)=>{
		if (err) { return exits.error(err); }
		exits.success(harry);
	});//_∏_

}

// this is way better
// (why?  Because it enapsulates the naughty non-await-using logic in a single part of your code, and then you can move on like normal)
fn: async function(inputs) {

  // this will throw if there's a problem, otherwise
  // it sets "harry" as expected
	let harry = await new Promise((resolve, reject)=>{
    User.findOne(1).exec((err, harry)=>{
      if (err) { return reject(err); }
      resolve(harry);
    });//_∏_
  });

  // Now it's just a normal action (all the crazy stuff is encapsulated above in the function we pass in to the Promise contstructor)
  return harry;

}// What if you have to use a library that doesn't support `await`?
// (we'll pretend for a sec that User.findOne doesn't support await even though it does)
// ==========================================================================================

// this approach is ok.... (there are better ways though)
fn: function(inputs, exits) {

  // it's not great that we have other code for the action (e.g. responding through the exits)
  // that is entangled inside of this callback.  it would be better if we could move it down into
  // safe, normal, await-enabled-land.  (outside of the dark scary _∏_)
	User.findOne(1).exec((err, harry)=>{
		if (err) { return exits.error(err); }
		exits.success(harry);
	});//_∏_

}

// this is way better
// (why?  Because it enapsulates the naughty non-await-using logic in a single part of your code, and then you can move on like normal)
fn: async function(inputs) {

  // this will throw if there's a problem, otherwise
  // it sets "harry" as expected
	let harry = await new Promise((resolve, reject)=>{
    User.findOne(1).exec((err, harry)=>{
      if (err) { return reject(err); }
      resolve(harry);
    });//_∏_
  });

  // Now it's just a normal action (all the crazy stuff is encapsulated above in the function we pass in to the Promise contstructor)
  return harry;

}

"What do you mean by analog in this context?"

Versus  “composite” which was the Treeline word for “circuit”.  The JSON representation of a simplified abstract syntax tree that you manipulated in the browser
Composite (which I called “analog”) is the alternative to fn: function (inputs, exits) {...}
Then later, when async/await was added to JavaScript and made it to node 7.9 in 2016, we introduced a new way of defining machines (aka machinepack functions aka actions2 aka helpers aka scripts in sails)
The new way was called “classical”.  It’s still pretty much like analog, except that you don’t have to deal with callbacks.  Instead, you write your code just like a normal function that can return or throw: fn: async function(inputs) {
If implementationType is not set (the default), Sails automatically reads your function signature and determines what to do based on what it sees.
If it sees exits, it assumes “analog”.  Otherwise, if it sees a 1-ary async function, it assumes “classical”
The difference is that an analog function will hang forever if exits.success() (or some other exit) is never called.  A classical function will only hang until it returns or throws.
Classical is way better. You don't have to worry about programmers accidentally calling the exits.*() callbacks more than once, or there being a rare edge case where no callback is triggered that turns into a memory leak in production as more and more hanging functions stack up.  Most importantly, classical functions are easier to work with for people who have never done asynchronous programming and threads and all that time-wasting, complicated business that keeps you from building features that humans want.
To learn more, scroll down deep into the README for GitHub.com/mikermcneil/parley and you’ll find some of the clearest explanations Ive ever worked on for flow control
cc: @DominusKelvin

@eashaw sorry I just get to see this. I should look out for GitHub notifications more often! @grantbi I go with Mike on this as I find the classical way more easy to reason about as it feels like returning in a normal function as opposed to calling exits. Wrote a bit about it here https://blog.sailscasts.com/return-or-throw-your-response/


Trialization

Depends on:

interface PR
element-miniscope PR suggestion

Added

Trial data files
ingestion functions first pass - needs testing bc element-interface version did not include allow_direct_insert arg
analysis.py script for downstream analysis - adjusted for attribute differences
fully functional notebooks except 04

Needs

integration tests - Nice to have
adjustment of scan time in analysis.py
verification of fake equipment data in notebooks/03

Note: I have the following in my .bash_profile to expedite jupytext process, which motivated the scripts/ -> py_scripts/ rename. We could adopt another convention.
alias jupysync='jupytext --sync notebooks/0*ipynb; mv notebooks/*py notebooks/py_scripts'

Drafting notebooks 05 and 07 now

@kabilar - ran most of both notebook 05 and 07, but ran into errors in each case where miniscope doesn't match structure from calcium imaging. If you use the u24_ prefix on dss-db, you can pick up where I left off.
05 error:
average_image = (miniscope.MotionCorrection.Summary & curation_key
                ).fetch1('average_image')
plt.imshow(average_image);
# ---
TypeError: Invalid shape (1, 600, 600) for image data

07 error
analysis.ActivityAlignment.populate(display_progress=True)
# ---
UnknownAttributeError: Unknown column 'mask' in 'order clause'

Hi @CBroz1, just a heads up, I made the mistake of directly committing Thinh's suggestions to your fork/branch.

Thank you for these updates, @CBroz1!

I removed trial 50 since the start time occurred before trials 47-49.


Missing keys for sbt-dependency-graph?

Running dependencyGraph with the coursier SBT plugin enabled seems to trigger the default SBT resolution. Some extra update-related keys may need to be overloaded.

sbt-dependency-graph seems quite entangled with the Ivy resolution of SBT... Closing this one.


Can not auth using Redis

Hello everybody,
I'm trying to auth using Redis, but it keeps failing. Could you please verify if I'm doing anything wrong?
Here is my docker-compose env vars:
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__AUTH_REDIS__ENABLED=on
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__HOST=redis
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__PORT=6379
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__PASSWORD=hidden
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__DATABASE=0

and here is my Redis key:
get "[\"\", \"pedro2\", \"lwm2m_adapter\"]"
"{\"passhash\": \"$2a$12$Y7hZuD921ScEQCO/dgwDEOSo9WV.bFCub/CU6J5rUK8RM9vGdVBQe\", \"subscribe_acl\": [{\"pattern\": \"devices/deviceA/#\"}, {\"pattern\": \"devices/deviceB/#\"}, {\"pattern\": \"devices/deviceC/#\"}]}"

So why is it failing when trying to connect with the same username and client id (pedro2 and lwm2m_adapter respectively)?
vernemq_1         | 2017-07-17 11:14:35.531 [error] <0.352.0>@vmq_diversity_script_state:handle_info:175 can't call function auth_on_register with args [{addr,<<"<IP_ADDRESS>">>},{port,33366},{mountpoint,<<>>},{client_id,<<"lwm2m_adapter">>},{username,<<"pedro2">>},{password,<<"olhaaquiotoken">>},{clean_session,true}] in "/usr/share/vernemq/lua/auth/redis.lua" due to {exit,{noproc,{gen_server,call,[auth_redis,{checkout,#Ref<<IP_ADDRESS>2>,true},5000]}}}
vernemq_1         | 2017-07-17 11:14:35.532 [warning] <0.484.0>@vmq_mqtt_fsm:check_user:546 can't authenticate client {[],<<"lwm2m_adapter">>} due to error
vernemq_1         | 2017-07-17 11:14:35.540 [error] <0.375.0>@vmq_diversity_script_state:handle_info:175 can't call function auth_on_register with args [{addr,<<"<IP_ADDRESS>">>},{port,33368},{mountpoint,<<>>},{client_id,<<"lwm2m_adapter">>},{username,<<"pedro2">>},{password,<<"olhaaquiotoken">>},{clean_session,true}] in "/usr/share/vernemq/lua/auth/redis.lua" due to {exit,{noproc,{gen_server,call,[auth_redis,{checkout,#Ref<<IP_ADDRESS>4>,true},5000]}}}
vernemq_1         | 2017-07-17 11:14:35.541 [warning] <0.485.0>@vmq_mqtt_fsm:check_user:546 can't authenticate client {[],<<"lwm2m_adapter">>} due to error

Am I doing something wrong ?!

Hi, on the above docker env vars I don't see that you have enabled vmq_diversity itself (a line corresponding to plugins.vmq_diversity = on in the vernemq.conf file)

You're right, I did not paste the whole thing, but the redis part only. It is there, though:
      - DOCKER_VERNEMQ_PLUGINS__VMQ_DIVERSITY=on
      - DOCKER_VERNEMQ_PLUGINS__VMQ_PASSWD=off
      - DOCKER_VERNEMQ_PLUGINS__VMQ_ACL=off

ok - to me it looks like the redis client inside vernemq failed to connect to the redis instance when VerneMQ was started and therefore the client didn't start at all (hence the noproc - no process - in the last part of the errors: {exit,{noproc,{gen_server,call,[auth_redis,{checkout,#Ref<<IP_ADDRESS>2>,true},5000]}}}).
Can you double check for any errors/warnings earlier in the console log indicating this problem?

Thank you. Those logs are all I've gathered from vernemq. Redis is definitely running and vernemq seems to be able to connect to it, though.

Ok, so I've set the logging level to debug, and now I can see it is not being able to connect to redis due to a password error (don't know why, as I'm 100% sure it is ok):
vernemq_1         | 2017-07-17 16:57:36.175 [debug] <0.343.0> Supervisor vmq_diversity_sup started vmq_diversity_plugin:start_link() at pid <0.344.0>
vernemq_1         | 2017-07-17 16:57:36.180 [debug] <0.343.0> Supervisor vmq_diversity_sup started vmq_diversity_script_sup:start_link() at pid <0.345.0>
vernemq_1         | 2017-07-17 16:57:36.190 [debug] <0.343.0> Supervisor vmq_diversity_sup started vmq_diversity_cache:start_link() at pid <0.346.0>
vernemq_1         | 2017-07-17 16:57:36.210 [info] <0.31.0> Application vmq_diversity started on node<EMAIL_ADDRESS>vernemq_1         | 2017-07-17 16:57:36.211 [debug] <0.228.0>@vmq_plugin_mgr:load_app_modules:622 loading modules: [vmq_diversity,vmq_diversity_app,vmq_diversity_bcrypt,vmq_diversity_cache,vmq_diversity_cli,vmq_diversity_ets,vmq_diversity_http,vmq_diversity_json,vmq_diversity_lager,vmq_diversity_memcached,vmq_diversity_mongo,vmq_diversity_mysql,vmq_diversity_plugin,vmq_diversity_postgres,vmq_diversity_redis,vmq_diversity_script,vmq_diversity_script_state,vmq_diversity_script_sup,vmq_diversity_script_sup_sup,vmq_diversity_sup,vmq_diversity_utils]
vernemq_1         | 2017-07-17 16:57:36.723 [info] <0.347.0>@vmq_diversity_app:start:71 enable auth script for redis "/usr/share/vernemq/lua/auth/redis.lua"
vernemq_1         | 2017-07-17 16:57:36.722 [debug] <0.348.0> Supervisor {<0.348.0>,vmq_diversity_script_sup_sup} started vmq_diversity_script_state:start_link(1, "/usr/share/vernemq/lua/init.lua") at pid <0.349.0>
vernemq_1         | 2017-07-17 16:57:36.724 [debug] <0.348.0> Supervisor {<0.348.0>,vmq_diversity_script_sup_sup} started vmq_diversity_script:start_link([<0.349.0>]) at pid <0.354.0>
vernemq_1         | 2017-07-17 16:57:36.724 [debug] <0.345.0> Supervisor vmq_diversity_script_sup started vmq_diversity_script_sup_sup:start_link("/usr/share/vernemq/lua/init.lua") at pid <0.348.0>
vernemq_1         | 2017-07-17 16:57:36.792 [debug] <0.361.0> Supervisor inet_gethost_native_sup started undefined at pid <0.362.0>
vernemq_1         | 2017-07-17 16:57:36.792 [debug] <0.60.0> Supervisor kernel_safe_sup started inet_gethost_native:start_link() at pid <0.361.0>
vernemq_1         | 2017-07-17 16:57:36.848 [error] <0.360.0> CRASH REPORT Process <0.360.0> with 1 neighbours exited with reason: {connection_error,{authentication_error,{unexpected_data,{ok,<<"-ERR invalid password\r\n">>}}}} in gen_server:init_it/6 line 344
vernemq_1         | 2017-07-17 16:57:36.849 [error] <0.357.0> CRASH REPORT Process <0.357.0> with 0 neighbours exited with reason: no match of right hand value {error,{connection_error,{authentication_error,{unexpected_data,{ok,<<"-ERR invalid password\r\n">>}}}}} in poolboy:new_worker/1 line 275 in gen_server:init_it/6 line 352
redis_1           | 1:M 17 Jul 16:57:36.864 - Accepted <IP_ADDRESS>:38381
vernemq_1         | 2017-07-17 16:57:36.851 [error] <0.358.0> gen_server <0.358.0> terminated with reason: no match of right hand value {error,{connection_error,{authentication_error,{unexpected_data,{ok,<<"-ERR invalid password\r\n">>}}}}} in poolboy:new_worker/1 line 275
vernemq_1         | 2017-07-17 16:57:36.852 [error] <0.358.0> CRASH REPORT Process <0.358.0> with 0 neighbours exited with reason: no match of right hand value {error,{connection_error,{authentication_error,{unexpected_data,{ok,<<"-ERR invalid password\r\n">>}}}}} in poolboy:new_worker/1 line 275 in gen_server:terminate/7 line 812
redis_1           | 1:M 17 Jul 16:57:36.880 - Client closed connection

Any help?

New update: when I remove auth @ Redis, the client connects succesfuly (no CRASH REPORT is printed), but I still can not connect an user. This is what vernemq reports:
vernemq_1         | 2017-07-17 17:16:30.494 [warning] <0.496.0>@vmq_mqtt_fsm:check_user:546 can't authenticate client {[],<<"lwm2m_adapter">>} due to error
vernemq_1         | 2017-07-17 17:16:30.495 [debug] <0.496.0>@vmq_ranch:teardown:123 session normally stopped

Hi @PedroMD, sorry for our response-delay.
Have you been able to solve the issue? How did you do it?

Hey @dergraf
I've decided to use rabbitmq's mqtt plugin with http-based backend authentication. Sadly, I don't think I've managed to get this working...

Hi @PedroMD,
wow that's quite a turn ;) But if you're happy with Rabbit, that's great!
On another note, VerneMQ supports HTTP based authentication and authorization too, in case you would like to check it out at some time.

Hi @dergraf - Grüess uf Basel!
We have had exactly the same setup as @PedroMD. The problem is that VerneMQ couldn't resolve the redis server name and therefor couldn't connect. If we passed in the docker ip address it worked.
The fix was to link or hostname the containers (using the docker internal dns).
More to read here.
Here is a complete setup in case someone else stumbles over the same issue:
# REM Redis
docker run -d --rm `
    --name vernemqaclredis `
    -h vernemqaclredis ` <--- HERE
    -p 6379:6379 `
    redis

# REM VerneMQ
docker run -d `
    -e "DOCKER_VERNEMQ_LOG__CONSOLE=both" `
    -e "DOCKER_VERNEMQ_ALLOW_ANONYMOUS=off" `
    -e "DOCKER_VERNEMQ_PLUGINS__VMQ_DIVERSITY=on" `
    -e "DOCKER_VERNEMQ_PLUGINS__VMQ_PASSWD=off" `
    -e "DOCKER_VERNEMQ_PLUGINS__VMQ_ACL=off" `
    -e "DOCKER_VERNEMQ_VMQ_DIVERSITY__AUTH_REDIS__ENABLED=on" `
    -e "DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__HOST=vernemqaclredis" ` <--- HERE
    -e "DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__PORT=6379" `
    -p 1883:1883 `
    -p 8883:8883 `
    -p 8888:8888 `
    --name vernemq `
    erlio/docker-vernemq

thanks for sharing your insights!
Do you see anything we could improve In VerneMQ related to this issue?
Grüsse in den Thurgau! :)

Actually this is a general docker configuration thing and I would not try to solve this within VerneMQ domain (other than maybe pointing it out in the docs).
 If you use docker-compose (or kubernetes etc.) dns service will be provided by the config file.
For clarification, if you do something like this, it works without setting any hosts manually (docker-compose.yml):
version: '3.4'
services:
  acl.redis:
    image: redis
    ports:
      - "6379:6379"
  service.mqtt:
    image: erlio/docker-vernemq
    environment:
      - DOCKER_VERNEMQ_LOG__CONSOLE=both
      - DOCKER_VERNEMQ_ALLOW_ANONYMOUS=off
      - DOCKER_VERNEMQ_PLUGINS__VMQ_DIVERSITY=on
      - DOCKER_VERNEMQ_PLUGINS__VMQ_PASSWD=off
      - DOCKER_VERNEMQ_PLUGINS__VMQ_ACL=off
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__AUTH_REDIS__ENABLED=on
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__HOST=acl.redis
      - DOCKER_VERNEMQ_VMQ_DIVERSITY__REDIS__PORT=6379
    ports:
      - "1880:1883"
      - "8883:8883"
      - "8888:8888"
    depends_on:
      - acl.redis


Idea: New plugin "status"

Hi @lelegard ,
I describe this here only as an idea:

When running a tsp process in an interactive environment, the debug output can be (or not) extremly verbose. However, some simple console messages can be useful. So I suggest some plugin that only pass all messages, and prints some console output. For example, print a dot . every N seconds or X bytes. Or get the input size (if it's a file) and print the percentage of the processed data, etc.

I'm sure you can evaluate the usefulness of such a plug-in and determine the best options for it.
What you think?
Regards.

Hi @lelegard ,
I discovered the --monitor option for the tsp. Perhaps you'll want to include some of the requested stuff inside this class. Futhermore, I need to comment that the monitor functionality is useless in Windows. Try this: tsp.exe -m -I file test.ts -O drop and it only prints:
* [MON] 2019/09/13 09:21, resource monitoring started
* [MON] 2019/09/13 09:21, resource monitoring terminated

I'm not sure, but I feel this isn't the expected behaviour.
Regards.

The monitor option is used to display the CPU and virtual memory usage of the tsp process. It prints messages at regular intervals. It's purpose is to make sure that no slow memory leak occurs on long-running instances of tsp. It works well on all O/S but your command starts and stops at the same time (just the time to read and drop a file), so there is nothing to display.
The time profile for the system monitor thread can be found in class SystemMonitor:
    const TimeProfile monitor_time_profile[] = {
        { 2 * MN,  MN / 6},  // up to start + 2 mn, log every 10 seconds
        {10 * MN,  1 * MN},  // up to start + 10 mn, log every minute
        {20 * MN,  2 * MN},  // up to start + 20 mn, log every 2 minutes
        {60 * MN,  5 * MN},  // up to start + 1 hour, log every 5 minutes
        {      0, 30 * MN},  // after start + 1 hour, log every 30 minutes
    };

The idea is to get frequent messages when the application starts and then less and less frequent. When the application starts, the virtual memory may vary rapidly until some form of equilibrium is reached.

OK. Thank you for the explanation.
And regarding the idea of "print out" at regular intervals information about the processing data?


Fix arguments in README example

Also, may be worth adding an example like this:
    json.setDiagnosticsOptions({
      validate: true,
      schemas: [{
        uri: '*',
        fileMatch: ['*'],
        schema: myFullyDereferencedSchema,
      }],
    });

For people who just want their schema without references to start working immediately. I had to dig through a lot of docs/issues to figure this one out.

Is this still relevant?


Add files instead of replacing

[Enhancement] It would be helpful if instead of overwriting an already solved question, another file would be added with some extension like _1 etc.. because sometimes we solve a question then we think of another method which is better. SO, instead this(the latest one) gets pushed ond overwritten. When we could check and compare the current code with the previous one.

need this as a feature


Jeet 7 or 6.2?

Hi all,
Really want to push all these nice changes out asap.
Just wondering what you think about the tag number being? I feel like #397 and #398 are pretty big changes for a lot of people, and could break layouts. So tagging as v7.0.0 might give people some warning against blindly updating.
It's not technically breaking the API. They could drop in the new code and it would compile fine, but it will break people's layouts in some cases.
Let me know what you think.

7 for sure

Cool. Glad we agree.


Change fetch_working_order to use orderSize instead of size.

Fixes #117

There's more to be fixed here. Will set the version to 1 to be inline with other code for the time being.


Identify discord tokens

Here is la regex :
[N]([a-zA-Z0-9]{23})\.([a-zA-Z0-9]{6})\.([a-zA-Z0-9]{27})

Thanks! Got an example? :)

This regex only works on some, pastebin has a bunch of examples:
https://pastebin.com/T41ej5hq

This regex only works on some, pastebin has a bunch of examples:
https://pastebin.com/T41ej5hq

This regex works with most tokens unless Discord switches they structure of the tokens. There are some exceptions yes


App language request

There is a Scribe (duvemeister) user who requested for new App Languages

Arabic
Persian (Farsi)
Turkish
Urdu (India)
Urdu (Pakistan)

Update: The files were shared with (duvemeister) for proofreading in the following languages. Folder links are given on the file name. (Common Urdu used in translation)

Arabic - ar.js and ar.json
Persian (Farsi) - fa.js and fa.json
Turkish - tr.js and tr.json
Urdu - ur.js and ur.json


feat: Add conditionals to delivery policy

Description
Adds conditionals to the ALB/NLB log delivery policy to help prevent the confused deputy problem.
Motivation and Context
https://github.com/terraform-aws-modules/terraform-aws-s3-bucket/issues/279
Breaking Changes
None
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

@antonbabenko are we a yay or nay on this?


wasm-opt fails due to duplicated symbol

Describe the Bug
wasm-opt fails with:
<==
getInt8: 146 (at 72306)
getInt8: 1 (at 72307)
getU32LEB: 146 ==>
getInlineString: _dyn_core_13c557ea88c6c04a___ops__function__FnMut_____Output______as_wasm_bindgen_10a7af47601b6690___closure__WasmClosure___describe__invoke______ ==>
<==
getInt8: 0 (at 72454)
getU32LEB: 0 ==>
<==
getInt8: 189 (at 72455)
getInt8: 247 (at 72456)
getInt8: 2 (at 72457)
getU32LEB: 48061 ==>
read one
<==
<==
getInt8: 146 (at 72458)
getInt8: 1 (at 72459)
getU32LEB: 146 ==>
getInlineString: _dyn_core_13c557ea88c6c04a___ops__function__FnMut_____Output______as_wasm_bindgen_10a7af47601b6690___closure__WasmClosure___describe__invoke______ ==>
[parse exception: duplicate export name (at 0:72606)]
Fatal: error parsing wasm. here is what we read up to the error:
(module
 (type $0 (func (param externref) (result externref)))
 (type $1 (func (param externref) (result i32)))
 (type $2 (func (param externref externref)))
 (type $3 (func (result externref)))

I can't tell why. I tried a lot of things: mostly changing version of everything I could: relevant dependencies, involved tools, rust toolchain; tweaking build options.
I have no idea why this is happening.
Steps to Reproduce
It's in our internal project and I can't share it RN.

Do you by any chance run wasm-opt before running wasm-bindgen-cli post-processing? If so, you have to run the post-processing by wasm-bindgen first!

I'm not aware where a detail like that is documented. But AFAIK wasm-pack does the right thing. Compiles with Rust -> post-processes with wasm-bindgen-cli -> optionally calls wasm-opt. So it is probably unrelated.
Let me know when you have something reproducible to share I can look at!

@daxpedda Welll... I do have something reproducible, but it is far from anything like "minimized". Our app is source-available, and last public release is also affected. We use Nix flakes to make dev process smooth, so if you have / don't mind setting up Nix you could repro it with just:
> g clone git@github.com:fedixyz/fedi.git
> cd fedi
> nix develop
> env CARGO_PROFILE=release ./scripts/build-wasm.sh

...
    Finished `release` profile [optimized + debuginfo] target(s) in 5m 04s
[INFO]: found wasm-opt at "/nix/store/9g4bdnnr95j7a01dq9in670zxi8qd8al-binaryen-116/bin/wasm-opt"
[INFO]: Optimizing wasm binaries with `wasm-opt`...
[parse exception: duplicate export name (at 0:53002)]
Fatal: error parsing wasm (try --debug for more info)
Error: failed to execute `wasm-opt`: exited with exit status: 1
  full command: "/nix/store/9g4bdnnr95j7a01dq9in670zxi8qd8al-binaryen-116/bin/wasm-opt" "/home/dpc/tmp/fedi/target-nix/pkgs/wasm-pack/wasm-pack-out/fedi_wasm_bg.wasm" "-o" "/home/dpc/tmp/fedi/target-nix/pkgs/wasm-pack/wasm-pack-out/fedi_wasm_bg.wasm-opt.wasm" "-O"
To disable `wasm-opt`, add `wasm-opt = false` to your package metadata in your `Cargo.toml`.
Caused by: failed to execute `wasm-opt`: exited with exit status: 1
  full command: "/nix/store/9g4bdnnr95j7a01dq9in670zxi8qd8al-binaryen-116/bin/wasm-opt" "/home/dpc/tmp/fedi/target-nix/pkgs/wasm-pack/wasm-pack-out/fedi_wasm_bg.wasm" "-o" "/home/dpc/tmp/fedi/target-nix/pkgs/wasm-pack/wasm-pack-out/fedi_wasm_bg.wasm-opt.wasm" "-O"
To disable `wasm-opt`, add `wasm-opt = false` to your package metadata in your `Cargo.toml`.

but get ready for a lot of building.

How about you compile this app with cargo build --target wasm32-unknown-unknown on your side and send me the .wasm file instead. Make sure that you are compiling with wasm-bindgen v0.2.99.

@daxpedda https://dpc.pw/fedi_wasm_bg.wasm

I specifically meant only running cargo build --target wasm32-unknown-unknown, not post-processing it with wasm-bindgen. Rustc compiled a *.wasm file which is then post-processed by wasm-bindgen-cli. To debug this I only need the first step. Let me know if anything is unclear.
But this is clearly a bug because wasm-bindgen is producing an invalid Wasm module.

Oh, sorry, I forgot about it over the weekend. :facepalm:
So this is what I'm doing now:
> cargo build --release --target wasm32-unknown-unknown -p fedi-wasm
...
    Finished `release` profile [optimized + debuginfo] target(s) in 8m 15s
> ls -alh target-nix/pkg/fedi-wasm/wasm32-unknown-unknown/release/fedi_wasm.wasm
-rwxr-xr-x 2 dpc dpc 194M Dec 24 15:00 target-nix/pkg/fedi-wasm/wasm32-unknown-unknown/release/fedi_wasm.wasm
> cp target-nix/pkg/fedi-wasm/wasm32-unknown-unknown/release/fedi_wasm.wasm ~/pub/blog/static/fedi_wasm-raw.wasm

Should be here now: https://dpc.pw/fedi_wasm-raw.wasm


Fixed all Warnings

Fixes #58

@DerekGooding your pull request changes logic, not only fixes warnings. There also new updates available, so please update your pull request, if you want it to be merged. I also don't see why we need lower case id instead of Id alias (as it agains C# coding style tradition).

This is a year old. Don't revive things. Just move on.
This is a public repo with a Help Wanted issue label. People search for those to help contribute to open source projects.
 Feb to Nov? Then you worked on it for a month without seeing this PR? Super unprofessional.

Sorry, but I'm only human after all :)
I cannot to be everywhere at the same time.
I wrote you as soon as I noticed your pull request.
Thank you for your efforts anyway. It is nice to know that people are willing to help.

By the way, we can keep structure to be named Id or even IdMarker and let the alias be id as a compromise solution. Though this way of constructing nested links structures was an experimentation.


Add CI tests

Add CI tests that ensure:

[x] unit tests all pass
[ ] code is correctly formatted
[ ] the code can be executed both as AS and TS

Added a GitHub action that runs npm test https://github.com/ColinEberhardt/assemblyscript-regex/commit/773b20ba9a8205de315c5e2d96f4f8a1d5fa25ab

Added a GitHub action that runs npm test https://github.com/ColinEberhardt/assemblyscript-regex/commit/773b20ba9a8205de315c5e2d96f4f8a1d5fa25ab


IDocumentOperations.QueueSqlCommand to insert into jsonb-column

E.g.
// Setup
Table myTable = new("my_table");
myTable.AddColumn("data", "jsonb");

options.Storage.ExtendedSchemaObjects.Add(myTable);

Creates a table with a jsonb-column.
internal class MyProjection : IProjection
{
    public void Apply(IDocumentOperations operations, IReadOnlyList<StreamAction> streams)
    {
        foreach (IEvent @event in streams.SelectMany(static s => s.Events))
        {
            const string Sql = "insert into my_table (data) values (?);";

            // Use the object as parameter directly
            operations.QueueSqlCommand(Sql, @event.Data);
        }
    }
}

fails with
System.InvalidCastException: Can't write CLR type XYZ with handler type TextHandler
         at Npgsql.Internal.TypeHandlers.TextHandler.ValidateObjectAndGetLength(Object value, NpgsqlLengthCache& lengthCache, NpgsqlParameter parameter)
         at Npgsql.NpgsqlParameter.ValidateAndGetLength()
         at Npgsql.NpgsqlParameterCollection.ProcessParameters(TypeMapper typeMapper, Boolean validateValues, CommandType commandType)
         at Npgsql.NpgsqlCommand.ExecuteReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken)
         at Npgsql.NpgsqlCommand.ExecuteReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken)
         at Marten.Internal.Sessions.QuerySession.ExecuteReaderAsync(NpgsqlCommand command, CancellationToken token)

Trying to insert the JSON as string like
internal class MyProjection : IProjection
{
    public void Apply(IDocumentOperations operations, IReadOnlyList<StreamAction> streams)
    {
        foreach (IEvent @event in streams.SelectMany(static s => s.Events))
        {
            const string Sql = "insert into my_table (data) values (?);";

            string json = JsonSerializer.Serialize(@event.Data);
            operations.QueueSqlCommand(Sql, json);
        }
    }
}

fails with
 Npgsql.PostgresException (0x80004005): 42804: column "data" is of type jsonb but expression is of type text

Is there any way to insert JSON into a jsonb-column via IDocumentOperations?
If not, then I believe the problem results from https://github.com/JasperFx/marten/blob/56fb16e3342088df5498049bd7d3670af31883e1/src/Marten/Internal/Operations/ExecuteSqlStorageOperation.cs#L40 (at for the "insert json as string"-case), as the type is string, so it tries to insert via TextHandler.
If this gets changed to
                var dbType = PostgresqlProvider.Instance.TryGetDbType(_parameterValues[i].GetType());

                parameters[i].NpgsqlDbType = dbType != null
                    ? dbType.Value
                    : NpgsqlTypes.NpgsqlDbType.Jsonb;

then the object could be used a parameter directly, and NpgSql would care about serialization and the insert to the jsonb-column.
Could this have any downsides? Is there possibly a better way by getting the actual underlying db-column-type and set the NpgsqlDbType based on that?
A different approach could be to pass the DB-types as arguments to QueueSqlCommand (additional overload).

Use const string Sql = "insert into my_table (data) values (?::jsonb);"; to convert the json string value to jsonb using ::jsonb

Fantastic 👍🏻 Thank you!
In Execute custom SQL in session a hint could be added that Postgres casting operator can be appended to parameters and that these casts aren't changed / removed by Marten. Then it would be ideal.
Something like

"?" placeholders can be used to denote parameter values. Postgres type casts :: can applied to the parameter if needed.

If that sounds good, I can do a PR for this.

@gfoidl Quick note, the use case which you outlined is not a typical one which users would use especially jsonb.

Yep, I know. Normally the document store features are sufficient. I came accross this issue while playing around with the outbox pattern (and wanted to have the jsonb column, not a plain string / text column).

Got you!


Behat Plugin

Hi
I have anebaled the behat plugin via my build settings, it runs my Behat tests, but I cannot see them in the UI on the actual build
I have folloiwng in my build steps:
test:
  php_loc:
  behat:

Looking at the UI

Is there any other configuration  I need to do?
I'm running the master branch

Same issue here, seems that the behat metadata are not stored anymore in the database.

I've even tried the 1.6.0 and the 1.5.2 tags, but still get the same.
Surely someone must have got this working before...


[WIP]fix(cloudevent): update aliyun lookupevent api

这个 PR 实现什么功能/修复什么问题:

[x] 功能、bugfix描述
[ ] 冒烟测试
[ ] 单元测试编写

是否需要 backport 到之前的 release 分支:

release/3.5

/area coudevent
/cc @zexi

Codecov Report

Merging #9966 (1fdf903) into master (54823a6) will increase coverage by 9.94%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9966       +/-   ##
===========================================
+ Coverage    5.97%   15.91%    +9.94%
===========================================
  Files         849      124      -725
  Lines      132219    12840   -119379
===========================================
- Hits         7896     2044     -5852
+ Misses     123621    10605   -113016
+ Partials      702      191      -511

Flag
Coverage Δ

aFlag
15.91% <ø> (+9.94%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudprovider/retry.go

pkg/multicloud/aws/loadbalancerlistenerrule.go

pkg/compute/regiondrivers/huawei.go

pkg/multicloud/aws/iam_user.go

pkg/compute/models/vcenters.go

pkg/monitor/alerting/scheduler.go

pkg/util/iproute2/errors.go

pkg/appsrv/radix.go

pkg/mcclient/options/k8s/fed_resource.go

pkg/compute/models/cloudproviders.go

... and 715 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #9966 (1fdf903) into master (54823a6) will increase coverage by 9.94%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9966       +/-   ##
===========================================
+ Coverage    5.97%   15.91%    +9.94%
===========================================
  Files         849      124      -725
  Lines      132219    12840   -119379
===========================================
- Hits         7896     2044     -5852
+ Misses     123621    10605   -113016
+ Partials      702      191      -511

Flag
Coverage Δ

aFlag
15.91% <ø> (+9.94%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudprovider/retry.go

pkg/multicloud/aws/loadbalancerlistenerrule.go

pkg/compute/regiondrivers/huawei.go

pkg/multicloud/aws/iam_user.go

pkg/compute/models/vcenters.go

pkg/monitor/alerting/scheduler.go

pkg/util/iproute2/errors.go

pkg/appsrv/radix.go

pkg/mcclient/options/k8s/fed_resource.go

pkg/compute/models/cloudproviders.go

... and 715 more

Continue to review full report at Codecov.

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


TypeError: EventFeed() missing 1 required positional argument: 'ICalFeed'

Following instructions here: https://django-ics.readthedocs.io/en/latest/usage.html
django 2.2
  File "/home/otcomm/otcomm/otcommproj/dallas/urls.py", line 15, in <module>
    url(r'^latest/feed.ics$', views.EventFeed(), name='event_feed'),
TypeError: EventFeed() missing 1 required positional argument: 'ICalFeed'

Hey @ebdavison! Could you post your view code? Here's a working example of the iCal feed usage from an active project that is deployed to here and offers the feed here.

Sure, this was right from the documentation/sample:
def EventFeed(ICalFeed):
    """
    A simple event calendar
    """
    product_id = '-//otcommittee.com//Calendar//EN'
    timezone = 'UTC'
    file_name = 'event.ics'

    def items(self):
        return Meeting.objects.all().order_by('-meeting_time')

    def item_title(self, item):
        return item.meeting_time

    def item_description(self, item):
        return item.meeting_time

    def item_start_datetime(self, item):
        return item.meeting_time

@ebdavison you have defined a function called EventFeed with def EventFeed(...). Use class EventFeed(...) instead to declare a class :)

oh ... well that certainly makes a difference.  But now a different error.  I do not see either item_link or get_absolute_url in your example mentioned above.  I am going to see what I can find on this ...
  File "/home/otcomm/otcomm/otcommenv/lib/python3.6/site-packages/django/contrib/syndication/views.py", line 62, in item_link
    'item_link() method in your Feed class.' % item.__class__.__name__
django.core.exceptions.ImproperlyConfigured: Give your Meeting class a get_absolute_url() method, or define an item_link() method in your Feed class.

The error code is pretty self evident. You can find the documentation for setting the feed properties up in the Django syndication framework and feeds documentation. Good luck!


{Compute} Detach Azure Stack from Profiles

Related command

Description

add command loader for azure stack
build azure stack context

Two files are still in common:

_help.py
linter_exclusions.yml

Testing Guide

az cloud set --name AzureCloud --profile 2018-03-01-hybrid
azdev test vm --no-exitfirst

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

/azp where

Compute

For some complex custom methods (such as create_vm and create_vmss), I think we can customize the CliCommandType for Azure Stack related commands to avoid duplication of code, so as to avoid introducing more complexity and maintenance costs


interested in working together?

hi,
i'm making something similar. a blender add-on installer. Plugget

handle dependencies
install from a public repo with 1 click, no technical steps
a add-on to browse and search add-ons

maybe we can work together?

Let me check out the code, thank you for contacting!

@hannesdelbeke Please let me see if I have free time - I have transferred to the new job and I need some time to adjust my schedule

no rush, take your time :)
and congrats on the new role

package versions
my main focus atm is better UX for versions.
recenttly support for installing different version was added, but not yet exposed to the UI in blender.
ideally there would be a dropdown letting the user select a specific version, or latest, or a beta/prerelease version.
e.g. rolling out a beta version to an artist, should be as simple as telling them to refresh, and select my-tool-beta from the dropdown.
instead of sending them a private addon, they have to manually install, then potentially also ask them to pip install dependencies.
I'm planning to write a generic UI in qt, so it can be reused outside of Blender.
However it d be nice if the UI in the Blender addon also had these dropdowns.
other

you have checksum verifying, it d be nice to bring this to plugget core library.
plugget actions need some cleaning up. lots of shared logic between e.g. blender pip , krita pip, and unreal pip.
need UI to add custom manifest repos, e.g. custom manifests repo for the half sword projecct
ran into this while testing plugget on an actual project from a friend. the artist struggled to run the code.

but overall just reading through the code and letting me know what you think would also be helpfull.
even if you think it's bad, it'd be great to get another pair of eyes on it.

overall vision

artists/end-users can install tools with 1 click
devs package any tool, including 3rd party tools from another repo. without having to fork it.
the packaging info lives on a manifest repo. which devs can contribute too, or they can mae their own manifest repo.
This results in code that is very open and not limited by 1 dev. e.g. the creator of a tool doesnt has to also package the tool. another dev could do this.
artists/end-users can install a collection of tools with 1 click. e.g. install all the tools for project X .
Right now main focus is on Blender, later I will add whole pipeline support. e.g. install houdini, maya, blender plugins wiith 1 click. and add support for more advanced setup steps. e.g. if a dependency is a piece of software instead of a pypi package


feat(upgrade): set io-engine DaemonSet to use OnDelete updateStrategy

Signed-off-by: Niladri Halder<EMAIL_ADDRESS>This PR sets the spec.updateStrategy on the io-engine DaemonSet to OnDelete. This is required for releases beyond 2.0 when we want to have granular control over the delete sequence of the Mayastor data plane components. This, together with Nexus HA, opens up the possibility for upgrade with live volumes.

bors try

@niladrih there seem to be other changes as well in this PR. Can we skip those?

bors try

@niladrih there seem to be other changes as well in this PR. Can we skip those?

I've separated those changes into separate commits. This PR should be revert-friendly.

bors merge


Build templates everywhere

This series is another step in the direction of Makefile.rules simplification. The final result here is the builtins.h generation rules removal, with that we stick to the templates schema and let it happen in a uniform way.

While I like the idea, I'm not really sold on the implementation. Since things inside {{ .. }} blocks are Python code, and include "foo" isn't really Python, things can get quite confusing. Maybe a function to include a template could be exported instead?

Yep, the idea of exporting a include() function seems good. Thanks for reviewing.

v2 comming....


feat: forgot password analytics

Checklist

[ ] JIRA - <LW-10055>
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Logging revamped forgot password analytics in PostHog.
Testing
Check PostHog documentation and events a you go through the forgot password flow for 1) enter wallet 2) next click after recovery phrase and 3) paste from clipboard
Screenshots
Attach screenshots here if implementation involves some UI changes

Changes unknown
when pulling 05b700c7fbbd50f256cb335fcd03ba0b3b448241 on feat/lw-10055-forgot-password-analytics
into ** on feat/lw-9651-revamp-onboarding-develop-branch**.

Lock wallet will be re-disabled after QA.


[Bug] chrome web store ，input box

In the chrome web store, after typing in the input box, click the first suggestion, and the selected one is indeed the last one. This is the case in all cases.

It is possibly because you have the bottom address bar, see #89

It's fixed in last build

It's fixed in last build

Unfortunately, this problem has not been fixed in Generation<PHONE_NUMBER>. Please watch the video.
https://user-images.githubusercontent.com/55195642/127329989-ca6fb010-2e30-4577-80d5-5f6a4f2a4eda.mp4

I can reproduce the issue on stable Chrome so it's a JS or CSS issue on their website. Which means we need a custom patch injected in their layout
@d3ward maybe if you have idea

This is probably the website issue. Tried in chrome, Samsung & Edge. Each have same issue.

Ridiculous that Google has still not made their Chrome Webstore mobile friendly. Yikes.

It's funny because they have tablet support with touch events ( seems to be the one that make the selection of search result bugged )

Ridiculous that Google has still not made their Chrome Webstore mobile friendly. Yikes.


Add styles for input[type="file"]

Hi, I don't see input[type="file"] in the css, I think it should be added.

Browsers do not let you style file inputs. File Style plugin fixes this problem. It enables you to use image as browse button. You can also style filename field as normal textfield using css. It is written using JavaScript and jQuery.
~ Stack Overflow: http://stackoverflow.com/a/4532736/227176


Publish as a NuGet package

Once the final API is fixed, this can be released on NuGet. Consider releasing this as a source-only package, too.

Done: see 1.0.0-alpha on NuGet.


ENH: Add test on random inpu to volume computations

As a follow-up from pull request #5014 , I propose to add a test to the spatial module in order to check that the volume computed is correct.
I generated a set of random points, and computed the volume from qconvex to compare it with scipy's output.
Hervé Audren

@haudren the TravisCI tests actually never ran on this PR because it had a merge conflict when you submitted it. Can you please rebase on latest master?

You may also want to squash the commits into a single one and start the commit message with TST: . (see http://docs.scipy.org/doc/numpy-dev/dev/gitwash/development_workflow.html#writing-the-commit-message)

@rgommers : Just rebased and squashed, and used a more proper commit message. I guess we now just have to wait for Travis to complete :)

Thanks @haudren, merged.

Thank you @rgommers , always a pleasure to contribute :)


Telnet on integrated terminal doesn't report correct window size when rendering mode is canvas/webgl

VSCode Version: 1.50.0 (system setup)
OS Version: Windows 10 x64 KB4579311

Steps to Reproduce:

Connect to a telnet session that adapts to the client's window size (e.g. a Windows telnet server) from the integrated terminal with rendering mode set to canvas or webgl.
The client reports a window width of 16 no matter what the real width is (see screenshot).
Switch rendering mode to dom: the telnet client immediately sends a window size negociation request to the server, this time reporting the correct size (see screenshot).

Does this issue occur when all extensions are disabled?: Yes

Strange, are you sure it wasn't some unrelated bug and switching the renderer just refreshed the size? I don't believe the window dimensions report sequence has anything to do with the renderer code.
Does the same happen in webgl if you resize the terminal, that normally reevaluates its size.

The problem always happens on canvas and WebGL renderers and never on DOM. I've tried resizing, restarting the terminal, restarting VS Code, even restarting my computer, but the only way I've gotten it to work is by switching to the DOM renderer


creat window prevent serial port receiving.

I am using 2 page in my project.
Page1: main.

Page2:

I can Click button into second page calling lv_win_create()
I also use one serial port to receive data from remote device to update status in pages.
The realtime data can be shown correctly in page1.
But I notice one problem:
When I go to Page2, data cannot update in time.

I also dump data received by serial port as below:

In dump data picture, the only one thing different is that I go to page2 by calling lv_win_create().
It sounds like serial port receiving goes strange after I create new window.
Does "LV_WIN_CREATE" prevent USART INTERRUPT handler?
I don't know why. How can I fix it?

Sorry, I solved it.
I use lv_task_create to create new task to update page2, then USART can receive data correctly.
I will close this issue.

Glad you figured it out!
We use GitHub issues for development related discussions.
In the future, please use the forum to ask questions.


Onlyshown correct language for EN but others like RU and UA isnt! correct

Hi this is awesome software  but why when i choose Ru or Ua languages  indicator is shown not correct like this  only whork good for EN
https://i.imgur.com/Bb0ghaZ.jpeg

Edit/create layin.ini file to
[Languages]
EN = 1033;0bb0bb
UA = 1058;5555ff

Hi this is awesome software but why when i choose Ru or Ua languages indicator is shown not correct like this only whork good for EN
https://i.imgur.com/Bb0ghaZ.jpeg

Edit/create layin.ini file to
[Languages]
EN = 1033;0bb0bb
UA = 1058;5555ff


USBDevice: fix missing defined check for TARGET_F411RE

Fix USBDevice usage on NUCLEO F411RE boards by adding missing defined(...)
USBSerial was tested and works fine.

Hello, can you sign please http://developer.mbed.org/contributor_agreement/ ?

I can't locate clementlege on mbed, can you share your nick there?

Sorry, here is my profile link: https://developer.mbed.org/users/clemvangelis/
I already agreed for the contributor agreement.

Thanks


chore(CI): support macOS

Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

Codecov Report

Merging #2 (624e9cc) into main (e8576d2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   84.46%   84.46%
=======================================
  Files           1        1
  Lines         103      103
=======================================
  Hits           87       87
  Misses         10       10
  Partials        6        6

Continue to review full report at Codecov.

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


NIH UIUC RIFTcov resubmission

Description
Updating the latin hypercube sampling to include uncertainty in the change in NPI compliance between calibration and projection. Includes new model output and updated metadata, abstract.
Checklist

[ X] Specify a proper PR title with your team name.
[ ] All validation checks ran successfully on your branch. Instructions to run the tests locally is present here.

@LucieContamin Just resubmitted to incorporate further sources of uncertainty during the projection period. Let me know if any questions!

Hi @BarnabyLlama , thank you for the update. I all looks good to me, just for information I cannot merge the PR while it's a draft. Once you are ready, please feel free to change it to "ready for review" or I am also happy to do it if you prefer (just let me know when) and merge it.
Please let me know if any issues or questions,
Best,
Lucie

Hi @LucieContamin, sorry! Thought I had marked it for review already. Now done.
Best,
Sophie


Benchmarking etc

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3cc208b). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #29   +/-   ##
=========================================
  Coverage          ?   96.37%
=========================================
  Files             ?       19
  Lines             ?      690
  Branches          ?        0
=========================================
  Hits              ?      665
  Misses            ?       25
  Partials          ?        0

Impacted Files
Coverage Δ

src/MLJLinearModels.jl
100% <ø> (ø)

src/glr/constructors.jl
100% <ø> (ø)

src/glr/d_l2loss.jl
100% <100%> (ø)

src/fit/default.jl
66.66% <100%> (ø)

src/loss-penalty/robust.jl
100% <100%> (ø)

src/glr/utils.jl
87.5% <100%> (ø)

src/glr/d_logistic.jl
100% <100%> (ø)

src/loss-penalty/generic.jl
86.07% <100%> (ø)

src/scratchspace.jl
100% <100%> (ø)

src/glr/d_robust.jl
100% <100%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


Plans for .NET Core Support

Do you have plans to support the new .Net Core Framework?

Sorry for the delayed reply. As is, the current library should be compatible with the new core. We don't have a rewrite planned but if need be we can always push an updated version. Let me know if you have any issues using the current version in the new .Net core.


Fix sig-cluster-lifecycle statement and small other things in the draft

@idvoretskyi @sarahnovotny @spiffxp @kubernetes/sig-cluster-lifecycle

I'm happy to merge if subject matter experts have taken a look.  LGTM, would like to see someone else weigh in if possible.

LGTM.


feat: use zeepki.st instead of zeepkist.wopian.me

:tada: This PR is included in version 1.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Provide way to specify the server version with 'code tunnel'

Currently there is no way to downgrade to a previous version of Visual Studio Code Server.  It always uses the latest version, regardless of the version that is locally installed (full or cli).  Please provide a way for us to specify the server version and/or commit.
I ask this because there is a recent Visual Studio Code version (1.81.1) which has a major bug that prevents me from using VSC Server and I have had to resort to direct ssh connections.  I would like to downgrade to 1.80 until the bug is resolved but there does not seem to be any way to do this.

This is not in the scope of code tunnel. The tunnel must always use the VS Code server version that the client connects with, otherwise communication between the remote and local instances will probably not work. To use a different version of VS Code on the remote, connect with a different version of VS Code on the client.

@connor4312, my client does not have VS Code installed.  The client is using the web tunnel.  Is there any way to have the client web use a specific version?

Not presently. If whatever bug you're hitting is fixed on Insiders, though, you can access insiders.vscode.dev


cmd/go: build failure on amd64 Linux with an error in TestTwoGopathShlibsGccgo from commit  4e8be99590 onward

What version of Go are you using (go version)?
The latest git tip I've tested this on is go version devel +926373ea79 Thu Oct 12 00:11:33 2017 +0000 linux/amd64.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOOS="linux"

Specifically I'm on Fedora 26. Since this test appears to use gccgo, it may be relevant that my reported (system) gccgo version is gccgo (GCC) 7.2.1 20170915 (Red Hat 7.2.1-2).
What did you do?
Trying to do ./all.bash build now fails in TestTwoGopathShlibsGccgo:
[....]
##### ../misc/cgo/testcshared
PASS

##### ../misc/cgo/testshared
--- FAIL: TestTwoGopathShlibsGccgo (0.68s)
        shared_test.go:371: /tmpfs/testshared953842952/pkg/gccgo_linux_amd64_5577006791947779410_fPIC/shlibs/libdep2.so is not linked to anything matching libdepBase\.so
FAIL
exit status 1
FAIL    _/tmp/go/misc/cgo/testshared    13.913s
2017/10/11 20:36:31 Failed: exit status 1

##### ../misc/cgo/testplugin
PASS
something
[...]

Git bisection says that this started happening at commit 4e8be99590, 'cmd/go: clean up compile vs link vs shared library actions'. I hacked in some instrumentation and it appears that libdep2.so here is only linked to libgo.so.11, libgcc_s.so.1, libc.so.6, and ld-linux-x86-64.so.2.

CC @rsc @mwhudson

Sorry about this. The test is being skipped on my Linux box because "gccgo too old". I will update my Linux distribution.

I have a new Linux and new gccgo and can reproduce the problem. Will fix, thanks.

This is failing on ppc64le as well, as long as your machine has gccgo and it is new enough.
I can see in the verbose output that the link for libdep2.so is different than before. I don't think this test is expecting libdepBase.a to be linked in.
/usr/bin/gccgo -o $WORK/b021/libdep2.so -Wl,-( -Wl,--whole-archive /tmp/testshared785817683/pkg/gccgo_linux_ppc64le_5577006791947779410_fPIC/libdep2.a /tmp/testshared785817683/pkg/gccgo_linux_ppc64le_5577006791947779410_fPIC/libdepBase.a -Wl,--no-whole-archive -Wl,-) -zdefs -shared -nostdlib -lgo -lgcc_s -lgcc -lc -fPIC
On Ubuntu 16.04 with gccgo 6, there is an additional failure.
--- FAIL: TestGoPathShlibGccgo (2.83s)
shared_test.go:371: ./bin/exe is not linked to anything matching libdepBase.so
--- FAIL: TestTwoGopathShlibsGccgo (1.10s)
shared_test.go:371: /tmp/testshared302107344/pkg/gccgo_linux_ppc64le_5577006791947779410_fPIC/shlibs/libdep2.so is not linked to anything matching libdepBase.so
shared_test.go:371: ./bin/exe2 is not linked to anything matching libdep2
shared_test.go:371: ./bin/exe2 is not linked to anything matching libdepBase.so
FAIL

Sorry, my fix for this was preempted by the point release work. If you want to add a skip to the test pointing at this issue, that's fine.


libremote-processor must link against libpfw_utility

It uses Utility::fullRead and Utility::fullWrite and must therefore link
against libpfw_utility.

Coverage remained the same at 70.15% when pulling c20d0daf7b9ff127961c0fd7f685914d2ab7b232 on dawagner:fix-remote-processor-link into a460839bd77647606c9db21fc13b92c8a254f2d8 on 01org:master.

@krocard please review.

Coverage remained the same at 70.15% when pulling ba3b750aee4b5c7190bd9e9e32d6e96e7403fa8b on dawagner:fix-remote-processor-link into a460839bd77647606c9db21fc13b92c8a254f2d8 on 01org:master.

:+1:


Fix detection of CPU vendor on ARM

cpuinfo_vendor is the vendor of the CPU core, not CPU package
Remove cpuinfo_vendor_ampere as cpuinfo currently doesn't detect any cores designed by Ampere Computing

@malfet PTAL

@Maratyszcza I've added only "Ampere", the "Qualcomm" was already there... Nevertheless, the new way will make things easier, thanks for it. Did you tested it on Ampere Altra, Microsoft Surface X and Microsoft Dev Kit 2023 already or should we test it?


Fix switching to monoscopic

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

@rahul27 Please let me know how this works for you. Thanks

:+1: that works


Installed torch with LUA51 - failed in Assertion `lua_gettop(L) == top'

Hi,
I have installed torch with TORCH_LUA_VERSION=LUA51.
After installation when I try to run the main_train.lua, it throws me an error
lua: parser.c:2059: parse_root: Assertion `lua_gettop(L) == top' failed.
Aborted (core dumped)
what am I missing, any help is appreciated.
Thanks,
Ashwini

Did you able to fix this ?

No..
On 17 Dec 2017 6:59 p.m., "Deepak C R"<EMAIL_ADDRESS>wrote:

Did you able to fix this ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bgshih/crnn/issues/72#issuecomment-352255732, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AcwHixuYz6PWZrGcGr6Hq6atBElq0P2oks5tBRc9gaJpZM4ObSno
.


Support for "ExLibris Primo"

Primo is supported since 3ac7591. We do not have account support yet.


Create codeql-analysis.yml

Hi, who is responsible for this PR? You merged these to all our repos without any discussion and the check is failing on this one. I think there should have been discussion before merging this at all, but certainly where it doesn't actually work on this repo
Thanks,
Matt


model start failed, dbgpt is deployed by docker

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information

=3.11

DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[ ] Chat Knowledge
[X] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[ ] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
cpu
Models information

What happened
can't start model
What you expected to happen
can't start model
How to reproduce
can't start model
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

The llama model I downloaded was placed in that directory

could you show more model started failed log?

could you show more model started failed log?

only so many，，，，


App "minimizes" immediately

Nexus 5x (2016), Android 7.1.1
When I open the app, it shows for a second, black screen with the "My uploads" title bar, then minimizes itself. It keeps running in the apps list, but when I switch to it, same thing happens.

Hi @magnusmanske ,
Thanks for reporting this. Has this happened the entire time you have been using the app, or is it only a recent occurrence that followed some sort of change (e.g. updating to Android 7.x)?

I believe it was working prior to 7.x. I had the app removed for a while, so I'm not sure exactly when the breaking started.

Hmm. Can anyone with a 7.x device please help test and see if it breaks for them as well? 7.x hasn't been pushed to my (unrooted) Android device yet, and I am stuck with a slow laptop for the time being so running an emulator is difficult (my desktop is currently en route to Australia via sea, heh).

Finally managed to test on a 7.1.1 device, not able to reproduce this bug. Is anyone still experiencing this?

Magnus do you use two-factor authentication? I experienced the same bug a few weeks before and had to unistall, disable 2FA and install again (maybe i did not have to uninstall and reinstall, just logging in/our would be enough)

I just experienced this too while writing https://github.com/commons-app/apps-android-commons/commit/db19c20bc10bb923186403a5d086a5236e556d26 which adds logging into to 2fa.
I have a feeling that it relates to the attempted login of accounts where details are already stored by the app. If you have 2fa enabled the app needs to ask for your 2fa code again, but instead it just thinks the login has failed and bails.

Oh, this now happens even with non-2fa logins lol.

Fixed in #1048


ctrl+click for "go to declaration"

from @Nicolab

Suggestion: it would be more efficient to make a ctrl+click like in Eclipse IDE for "go to declaration"

as

https://github.com/atom/symbols-view/issues/18

I think this should be reopened as alt+click event is not triggered in the latest Atom, it instead highlights the options in the topbar. Pressing ctrl+alt+click triggers the event but I don't think it's a good way of doing it.

I think this should be reopened as alt+click event is not triggered in the latest Atom, it instead highlights the options in the topbar. Pressing ctrl+alt+click triggers the event but I don't think it's a good way of doing it.

Also alt-click is not working in ubuntu, because alt-click triggers window move (affects not only ubuntu I assume). Maybe ctrl+shift+click will work?

+1, this does not work in Ubuntu and should have a different default bind.

I would like to suggest revisiting this binding.  It would be great to allow configuring the keystrokes for go-to-declaration since alt-click also collides w/ sublime style column select.

@n3bulous +1. Please allow the mouse binding to be configurable. I too use the sublime-style column select and this busts it (or vice versa, if you please)

how about alt+cmd+ click?

:grinning: OK OK stop this!!!.
All other editors is using Ctrl + Click. Do not invent the wheel again.

Hey, just to put it out there. I've created declarations package, it's the base package for declarations and is pretty neat and consistent across providers

Subscribing for Ctrl + Click possibility (configurable?)

Subscribing for Ctrl + Click possibility (configurable?)

+1, ctrl + click is a standard in different editors

+1000

+1 ctrl + click

+1 for ctrl + click

Is there a way to disable this? I'm constantly double clicking a word and I'm used to already press ctrl while clicking to do ctrl+c to copy, which then will go to definition instead of copying.

+1 for ctrl + click.
Even if ctrl+click is used for the multiple cursor feature, I think we can have have a goto-definition on this same shortcut like in Wbestorm (or any Jetbrains IDE).

+1 for ctrl + click

+1 cmd+click or ctrl+click


Conditional selection without generic base interface

The documentation on Conditional selections with .grab results in this interface:
type QueryResult = (
  | { _id: string; name: "Bulbasaur"; hp: number }
  | { _id: string; name: "Charmander"; attack: number }
  | { _id: string }
)[];

However, I want to end up with this interface, without a generic { _type: string } base interface:
type QueryResult = (
  | { _type: "bulbasaur"; name: string; hp: number }
  | { _type: "charmander"; string; name: string; attack: number }
)[];

Using a query something like this (example code, this doesn't work):
q("*")
  .filter("_type == 'bulbasaur' || _type == 'charmander'")
  .grab(
    {}, // Empty object because I need something here
    {
      "_type == 'bulbasaur'": {
        _type: q.literal("bulbasaur"),
        name: q.string(),
        hp: ["base.HP", q.number()],
      },
      "_type == 'charmander'": {
        _type: q.literal("charmander"),
        name: q.string(,
        attack: ["base.Attack", q.number()],
      },
    }
  );

Is there a way to perform a query in GROQD and end up with a union interface containing only the types I requested? Is there something more appropriate than conditional selections I should be using instead?

Great question! This is fundamentally a short-coming of .grab, but @littlemilkstudio recently added/documented a new .select method that should resolve this! In cases like this, I believe you can swap out the .grab with two arguments, for a .select with just a single argument, and won't have to worry about the pesky empty interface.
If that doesn't resolve this problem for you, let's reopen this issue and discuss further!

It feels a little cleaner to use select here, but I still have a similar issue. If I drop the default I end up with a type similar to this:
type QueryResult = (
  | { _type: "bulbasaur"; name: string; hp: number }
  | { _type: "charmander"; string; name: string; attack: number }
  | Record<string, never>
)[];

If I do include default, e.g. default: { _type: q.literal("unsupported") }, I end up with a type similar to this:
type QueryResult = (
  | { _type: "bulbasaur"; name: string; hp: number }
  | { _type: "charmander"; string; name: string; attack: number }
  | { _type: "unsupported" }
)[];

It is a little clearer at least. Not sure how to improve this. Thanks for looking into this.

Very clever. Using default like that makes a lot of sense. This solves my issue, thanks!
Thanks for the tip on the GROQ tool also.


Problems installing siconos on Ubuntu 22.04 LTS (jammy)

Hi,
I tried to install siconos on Ubuntu 22.04 LTS (jammy) for the user 'stuser' running
cmake -DCMAKE_INSTALL_PREFIX=/home/stuser/siconos_installation -DUSER_OPTION_FILE=/home/stuser/siconos/ci_gitlab/siconos_confs/siconos_default.cmake ~/siconos
with a changed 'siconos_default.cmake' :
set(siconos_python_install "user" /home/stuser/siconos_installation  "Install mode for siconos python package").
'cmake', 'make' and 'make install' all run without any errors - except for some warnings during cmake (c.f. owncloud.gwdg.de/cmakeWarnings/).
But executing python3 ~/siconos-tutorials/examples/mechanics/GeometricPrimitives/sphere.py
throws
"ModuleNotFoundError: No module named 'siconos.mechanics.collision.tools' ".
-- Which makes sense, given the fact, that I just installed siconos in a non-standard path.
Thus I executed and added
export PYTHONPATH=$PYTHONPATH:/home/stuser/siconos_installation/lib/python3.10/site-packages/
export PYTHONPATH=$PYTHONPATH:/home/stuser/siconos_installation/local/lib/python3.10/dist-packages/
export PATH=/home/stuser/siconos_installation/bin:$PATH

to '~/.bashrc'.
Executing 'sphere.py' now gives (full errors: c.f. owncloud.gwdg.de):
"ImportError: cannot import name '_pybase' from partially initialized module 'siconos.mechanics.collision' (most likely due to a circular import) (/home/stuser/siconos_installation/local/lib/python3.10/dist-packages/siconos/mechanics/collision/_init_.py)".

As much as I understand the whole process, this import error seems to origin from siconos having installed python files in two seperate paths:
/home/stuser/siconos_installation/lib/python3.10/site-packages/
/home/stuser/siconos_installation/local/lib/python3.10/dist-packages/

As now in neither are all needed files at once.
For example are in ~/siconos_installation/lib/python3.10/site-packages/siconos/mechanics/collision : _pybase.so, _pybodies.so, _pynative.so ;
and in ~/siconos_installation/local/lib/python3.10/dist-packages/siconos/mechanics/collision: base.py, bodies.py, convexhull.py, __init__.py, native.py, tools.py .
But base.py seems to try to import _pybase.

Is there a way to make siconos install all python modules in one path?
Or is there maybe even some other problem with the installation setup on Ubuntu 22.04?
I'm looking forward to any ideas!

Hi Franck,
yes! After going for user install (Case 1) the example files in siconos-tutorials/examples/mechanics/GeometricPrimitives/ work flawlessly!
Just one little remark, if some other people try to install siconos this way:
the siconos_default.cmake is located in ci_gitlab/siconos_confs/siconos_default.cmake .
So the full command for the user installation would be like:
cmake -DUSER_OPTIONS_FILE=/ci_gitlab/siconos_confs/siconos_default.cmake -DCMAKE_INSTALL_PREFIX=somewhere  /path/to/siconos/source
Thanks a lot for the quick response!


SDK for .NET Core 1.1.1?

The latest .NET Core App package (https://www.nuget.org/packages/Microsoft.NETCore.App/) delivers version 1.1.1. However:
The specified framework 'Microsoft.NETCore.App', version '1.1.1' was not found.

Check application dependencies and target a framework version installed at:
C:\Program Files\dotnet\shared\Microsoft.NETCore.App
The following versions are installed:
1.0.0
1.0.1
1.1.0
Alternatively, install the framework version '1.1.1'.

I can't seem to find a download for SDK 1.1.1; am I missing something or was 1.1.1 released prematurely?
Thanks in advance
-Simon

I have the same issue!

@leecow ?

The SDK install links at https://www.microsoft.com/net/download/core will be up at 7:30 pacific time. In the meantime you can use https://github.com/dotnet/core/blob/master/release-notes/download-archives/1.1.1-download.md.

https://www.microsoft.com/net/download/core is live.