A question when played back the ***.bag

Hello, thank you for sharing this awesome dataset about the paper 《The Multivehicle Stereo Event Camera Dataset: An Event Camera Dataset for 3D Perception》
I am a graduate student from China, and I am currently learning about event camera. I want to look your dataset. I know they are composed of rosbag and need the rpg_dvs_ros package. I have successfully installed them, but when I downloaded a rosbag package and wanted to play it back, an error occurred: nothing can be played back.But I can play back the rosbag file which I recorded by myself.
Can you tell me what should I do, or am I missing what I need to view the rosbag?
I would be very grateful if you could reply to me.

What is the error you're getting?

Thanks for your reply.  What I mean is how to visualize the rosbag file you published. Is it visualized with this link：https://github.com/uzh-rpg/rpg_dvs_ros?
I can play back the rosbag which I recored like this:

But I can't play back the rosbag which downloads from your dataset like this :

I just learned this knowledge, any of your suggestions will be helpful to me, thank you

Are you running the dvs_renderer node? That generates the rendered event image that you're visualizing.

Yes, the above two pictures are running /dvs_rendering，but one has a image and the other has no image

Could you please tell me how to view your rosbag file? I would very appreciate any advice from you. Thanks!

Sorry for the late reply. I'm still a little unclear as to what you're running on your end. In order to generate visualizations under dvs_rendering, you need to run the dvs_renderer package from rpg_dvs_ros, and remap the event fields to match the ones in MVSEC (e.g. from events to /davis/left/events). Are you doing this already?

Thank you very much for your reply, I will modify it according to your suggestion, thanks again！

Thank you very much for your sincere help, I have been able to see your rosbag file like this :

But I have a question: is this what the event camera and lidar look like when fused? Why can't I see any display about lidar?
Thanks again for the great dataset that you supplied.

The rpg_dvs_ros package only visualizes event information. For visualizing the depths/event + depths, the other topics, such as /davis/left/blended_image_rect in the groundtruth file might be more useful.

Thank you for your prompt reply and patient guidance.


Failure while installing from master

Observed behavior
While installing from master branch the install fails after about 4-5 steps with error void-function.
Installing core packages
Importing package-keyring.gpg...
Importing package-keyring.gpg...done
Contacting host: elpa.gnu.org:443
Contacting host: elpa.gnu.org:443
Contacting host: melpa.org:443
Contacting host: melpa.org:443
Package refresh done
✓ Installed persistent-soft
✓ Installed use-package
✓ Installed quelpa
✓ Installed async
Installing core packages...done
/home/shopclues/.emacs.d/core/autoload/ui.el:0:0: error: void-function: (when-let*)
make: *** [.local/autoloads.el] Error 255

Here are the things that I've already tried:

Remove/clean and reinstall everything
update autoload file

Nothing seems to be working.
Expected behavior
Describe what you expected to happen.
Steps to reproduce

git clone https://github.com/hlissner/doom-emacs .emacs.d
cp init.example.el init.el
make install

System information

Click to expand
DOOM Doctor
Running Emacs v<IP_ADDRESS>, commit 5dacbb7cb1c6ac246a9ccd15e6c4290def67757c

shell: /usr/bin/zsh
Compiled with:
  XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GCONF GSETTINGS
  NOTIFY LIBSELINUX GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB
  TOOLKIT_SCROLL_BARS GTK3 X11 MODULES XWIDGETS
uname -a:
  Linux SC-LPT-102328 3.16.0-73-generic #95~14.04.1-Ubuntu SMP Thu Jun 9
  08:00:04 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

Attempt to load DOOM: failed
  (when-let*)

----

test-emacs
test-windows
test-fonts
test-gnutls
test-tls
Validated https://elpa.gnu.org
Validated https://melpa.org
Validated https://self-signed.badssl.com (this shouldn’t happen!)
Validated https://wrong.host.badssl.com/ (this shouldn’t happen!)
test-tar

----
There were issues!

Hopefully these can help you find problems. If not, run this doctor again with DEBUG=1:

  DEBUG=1 make doctor

And file a bug report with its output at https://github.com/hlissner/.emacs.d/issues

Please switch to develop. master is current broken, and has been for a while. Note that the install instructions have changed too (see the README on the develop branch).
Sorry for the confusion, I will be merging develop into master any day now.

@hlissner That worked. Thanks. Appreciate your effort on doom-emacs.


Typescript complains about undefined value in .where clause

I have a small problem when using this package on Typescript.
await MyModel.query()
    .skipUndefined()
    .where('col1', undefined) // Skips this WHERE query correctly, but Typescript complains
    .where('col2', null); // Type correct, but produces `WHERE "col2" IS NULL` (not skipped)

Here is the Typescript error with col1 by passing undefined to the second argument:
No overload matches this call.
  Overload 1 of 10, '(col: "col1" | "col2", expr: Expression<PrimitiveValue>): QueryBuilder<...>', gave the following error.
    Argument of type 'undefined' is not assignable to parameter of type 'Expression<PrimitiveValue>'.
      Type 'undefined' is not assignable to type 'Expression<PrimitiveValue>'.
  Overload 2 of 10, '(col: ColumnRef, expr: Expression<PrimitiveValue>): QueryBuilder<MyModel, MyModel[]>', gave the following error.
    Argument of type 'undefined' is not assignable to parameter of type 'Expression<PrimitiveValue>'.ts(2769)

This seems fixed by adding undefined to the union type PrimitiveValue. However I don't know what are the exact side effects of adding it and therefore unsure whether it will need further changes/adjustments or not. Does anyone mind checking this out?
Package versions:
objection: ^2.2.15
knex: ^0.21.21
pg: ^8.7.1
typescript: ^4.4.2

undefined is not allowed by default. It's only allowed when you use skipUndefined. To fix this, we would need have a different set of types IF skipUndefined is called. That's way too much work for such a small (and quite frankly a bad) feature. I'd recommend not using skipUndefined at all. Consider this query
Person.query().delete().where('id', someId).skipUndefined()

Now if you accidentally pass undefined in someId you end up deleting the whole table.
skipUndefined is a legacy method that should be removed from the API.


Dark mode for TUI

When using a dark theme in the terminal, the forced white background doesn't look great.

a4c61b687c813776ac2c18ce27cd9c28e5afa9cd
$ amdgpu_top --dark-mode

Thank you!


Fix:vLLM Model Supported check throwing circular dependency

What this PR does / why we need it:
After vLLM update to version 4.2
The load of Llama Arch was causing a circular dependency error.
This PR fixes that issue
Type of changes
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests you ran to verify your changes and the relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[x] Ran sanity checks on Llama, Mistral, and Gemma arch Models

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Re-running failed tests

/rerun-all - rerun all failed workflows.
/rerun-workflow <workflow name> - rerun a specific failed workflow. Only one workflow name can be specified. Multiple /rerun-workflow commands are allowed per comment.

/lgtm
/approve


Review 9: AddProfile.jsx

Overview
The focus for this code review will be centered around the Add Profile page.
Please pay attention too:

Javascript issues
React components

Review Branch
review-xxx
Files to review

AddProfile.jsx

Checklists

Architecture
Design
JS
React
Testing

Due date
Monday, April 1st 11:55pm
For more information
The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

AddProfile.jsx

ES-01: L31  ESLint: 'A space is required before '}'.(object-curly-spacing)'
JS-07: Avoid console.logs

AddProfile.jsx

ES-01: L31 ESLint: 'A space is required before '}'.(object-curly-spacing)'
JS-07: Avoid console.logs, this can cause flooding to the console and with enough even slow down performance

AddProfile.jsx
Consider adding check to avoid duplicate profile?
JS-07: Avoid console.logs, this can cause flooding to the console and with enough even slow down performance

AddProfile.jsx

UserProfileCollection does not add a profile for the user.
Ensure the formatting of the ProfileCardDisplay matches with the UserProfile.jsx

Ref Management: Convert the formRef initialization to use React's useRef hook for better ref handling in functional components.
User Data Access: Implement checks for null values when accessing Meteor.user().username to avoid runtime errors if the user is not loaded.
Role Default Value: Simplify management by keeping the default value for role in one place, preferably in the schema, to avoid inconsistency.
Prop Validation: Add prop validation to enhance component reliability and maintainability, especially if more props are to be used in the future.
Error Handling: Improve error handling in the Meteor method callback to manage UI state properly in case of errors.
Navigation Safety: Ensure that redirects after asynchronous operations check if the component is still mounted to prevent React state update warnings.


Contact Us page is pointing to old slack channel

README.md and http://vmware.github.io/docker-volume-vsphere/documentation/contactus.html are pointing to old slack channel which is old (https://vmwarecode.slack.com/archives/docker-volume-vsphere)
We have moved to #docker slack channel, README.md & Contact Us page needs to be updated.

[ ] http://vmware.github.io/docker-volume-vsphere/documentation/contactus.html
[ ] https://github.com/vmware/docker-volume-vsphere#public

/CC @tusharnt

We have to make correction at both places using https://vmwarecode.slack.com/messages/docker/

Fixed as part of PR #1790 and verified. Closing it.


AS3 Authenticate Name TLS_client

Environment
Application Services Version: 3.28.0
Hello,
Im trying to create TLS_client via AS3 and I would like to use "AuthenticateName", but I cant find how to declare it (define it). I tried to look for it on clouddocs f5 AS3, but there is no property for it.
Now Im using RESTAPI for it. Something like this:
curl -sSk -u xxx:xxx -H "Content-Type: application/json" -X POST https://x.x.x.x/mgmt/tm/ltm/profile/server-ssl -d '{"name":"foo","partition":"Common","defaultsFrom":"bar","authenticateName":"foobar"}'
Is there a way I cant find to define "AutheticateName" or is it not in AS3 yet or not posible?
Thank you
Kovi

Can you please provide an example of the tmsh configuration you would like to achieve?

Just set "serverName" in your TLS_Client object.  That corresponds to "authenticatename" in BIG-IP configuration (tmsh) for the serverssl profile.

Reopening this issue.

If you are still looking for this feature, please reach out to us at<EMAIL_ADDRESS>in order to prioritize.


Replace LogisticsDeliveryNoteById.deliveryLegs[].transport.units[].identifiers[{...}] by LogisticsDeliveryNoteById.deliveryLegs[].transport.units[].identifier{...} and add transportUnitIdentifierReference.value to LogisticsDeliveryNoteById.lineItems[].packages[]

We only need one identifier for a transport unit, and then it can be used as a reference in a package, so we can know in which transport unit a package has been placed.

Done, see commit 1ce3c5b.


README.md: Trim links a bit

The "source" link for centos-bootc changed; just drop it, as I'd like to have the documentation be canonical.
(Now of course hopefully we aren't going to go through a round
of moving things again any time in the near future, but still
I think users should click through to the docs site as
a general rule)

Are we able to keep the examples link? I find it very useful to see what other maintained examples are / could reference against.
I've seen other communities (ublue) reference them which was good affirmation.
Same for the "FROM part". It helps when copying and pasting to the Containerfile.

OK, I reworked this to just drop the source links.

LGTM! I agree on the source link. That shouldn't be shown. Thank you!


Update customer-lockbox-requests.md

Added text change in the workflow steps to indicate a new data point that will be available - when will the engineer start accessing the data. Also updating the screenshots. The names are same as the ones that need to be replaced.

Closing PR. Completed work on private repo.


Is this package compatible with other ggplot functions?

Thank you so much for this package -- it's been really helpful for visualisations!
This is not a big issue, but I'm trying to adjust the visualisations (colours, text size, etc) and not sure whether these functions are compatible with the usual ggplot functions?
For example, this works great:
pathway_pca(abundance = metacyc_abundance %>% column_to_rownames("pathway"),
metadata = metadata, group = "Plot")
But when I try to adjust with custom colours, nothing changes.
plot_colors <- c("#DD5129", "#FAB255", "#0F7BA2", "#43B284", "#4c2f61")
pca_picrust <- pathway_pca(abundance = metacyc_abundance %>% column_to_rownames("pathway"),
metadata = metadata, group = "Plot")
pca_picrust + scale_color_manual(values = plot_colors)

Hi @chung-wingko,
Thank you for your question and for using our package! I understand your desire to customize the visualizations using familiar ggplot2 syntax. While we would love to directly support ggplot2 functions, there are some limitations due to our use of the ggh4x package.
However, you can still achieve the customization you're looking for by accessing and modifying the individual layers of the plot. Here's how you can adjust the colors in your example:
pca_picrust <- pathway_pca(abundance = metacyc_abundance %>% column_to_rownames("pathway"),
                           metadata = metadata,
                           group = "Plot")

plot_colors <- c("#DD5129", "#FAB255", "#0F7BA2", "#43B284", "#4c2f61")

# Modify the color scale of the specific layer
pca_picrust$layers[[2]]$aes_params$colour <- NULL
pca_picrust$layers[[2]]$mapping$colour <- quote(Plot)

pca_picrust + scale_color_manual(values = plot_colors)

In this approach, we're first removing the hard-coded color mapping from the second layer (which typically contains the points), and then reassigning the color mapping to the 'Plot' variable. After these modifications, the scale_color_manual() function should work as expected.
For other ggplot2 customizations, you may need to use a similar approach of identifying and modifying specific layers. While it's not as straightforward as direct ggplot2 syntax, it does provide a way to achieve the customizations you're looking for.
We appreciate your feedback and will continue to explore ways to make our package more compatible with standard ggplot2 functions in future updates. If you have any more questions or need further clarification, please don't hesitate to ask!
Best,


⚠ ClamAV has degraded performance

In 6e235a4, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 205 ms

Resolved: ClamAV performance has improved in 189237d.


[AAP-27279] Fixes no call to launch end point on schedules

Addresses AAP-27279.
at the top level schedule list we were failing to make a call to the launch end point.

I identified another problem with schedules, Im not sure if it separate issue, but it is maybe good to include fix into this PR?
How to reproduce it:
Edit the schedules with Type: Management job
you will see fail:
TypeError
Cannot read properties of undefined (reading 'job_tags')
Call Stack
 setLaunchToWizardData
  frontend/awx/views/schedules/wizard/ScheduleSelectStep.tsx:131:9


Proper transaction support

Transactions were originally added to FrostDB because we used to split writes up and sort each row of a write into different granules. The scan layer excludes transactions that have a higher tx number than the read is being performed with to avoid reading a partial write.
However today we do not split up writes so reads excluding tx > read_tx doesn't make as much as sense because it's still a consistent read we're just now artificially limiting the amount of data we read.
I propose that we add proper database transactions to FrostDB such that we can atomically write to multiple tables at once. Potentially in the future support other database transactions as well such as queries, table creation/deletion etc.
API Proposal
type Table interface {
    InsertRecord(context.Context, arrow.Record) error
}

type Transaction interface {
    GetTable(string) (Table, error)
    Commit()
    Abort()
}

type DB interface {
    ...
    Begin() Transaction
}

Which would allow for something like

tx := db.Begin()
defer tx.Commit()
if err := tx.GetTable("table1").InsertRecord(ctx, r1); err != nil {
   tx.Abort()
   return
}
if err := tx.GetTable("table2").InsertRecord(ctx, r2); err != nil {
   tx.Abort()
   return
}

Multi-table transactions could be implemented quite easily by getting a write tx which already blocks reads from reading it as well as compactions from compacting it until it has been committed. In the event of an abort the transaction can walk back the writes and remove them from the L0 level of each table that they were written to before marking the transaction as complete. Thus still providing a consistent view.

I think multi-table transactions would only be beneficial in cases where we allow joining multiple tables. I'm very wary of adding joins to the DB (you introduce a world of complexity), so I'm simply not sure that explicit transactions would give us any benefits, but I might be missing something.

I think multi-table transactions would only be beneficial in cases where we allow joining multiple tables. I'm very wary of adding joins to the DB (you introduce a world of complexity), so I'm simply not sure that explicit transactions would give us any benefits, but I might be missing something.

What about the writes that are causally related? Like we have in production today where we write two tables and it has resulted in missing samples if one of the writes completes but the other does not?

True. That's what I was missing then.

One thing required for this change to to change snapshots to respect the tx. Right now they snapshot whatever is in the index.


Unsafe capability cast

here https://github.com/george8188625/Create-Diesel-Generators/blob/47672f012891e5faacb6ce6a8f8bc0aa5bef929f/src/main/java/com/jesz/createdieselgenerators/blocks/entity/HugeDieselEngineBlockEntity.java#L171

I'm aware of that


InvalidArgumentError (see above for traceback): Cannot assign a device to node 'save/RestoreV2_8'

distributed/cancer_classifier.py works in only one docker container.
It works in one container:
# both in <IP_ADDRESS>
python cancer_classifier.py --ps_hosts=<IP_ADDRESS>:8222 --worker_hosts=<IP_ADDRESS>:8223 --job_name=ps --task_index=0
python cancer_classifier.py --ps_hosts=<IP_ADDRESS>:8222 --worker_hosts=<IP_ADDRESS>:8223 --job_name=worker --task_index=0
But it not work in two containers:
# ps in <IP_ADDRESS>
python cancer_classifier.py --ps_hosts=<IP_ADDRESS>:8222 --worker_hosts=<IP_ADDRESS>:8223 --job_name=ps --task_index=0
# worker in <IP_ADDRESS>
python cancer_classifier.py --ps_hosts=<IP_ADDRESS>:8222 --worker_hosts=<IP_ADDRESS>:8223 --job_name=worker --task_index=0
the error msg I got in the worker:
I tensorflow/core/distributed_runtime/rpc/grpc_channel.cc:197] Initialize GrpcChannelCache for job ps -> {0 -> <IP_ADDRESS>:8222}
I tensorflow/core/distributed_runtime/rpc/grpc_channel.cc:197] Initialize GrpcChannelCache for job worker -> {0 -> localhost:8222}
I tensorflow/core/distributed_runtime/rpc/grpc_server_lib.cc:211] Started server with target: grpc://localhost:8222
WARNING:tensorflow:From /usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py:344 in init.: init (from tensorflow.python.training.summary_io) is deprecated and will be removed after 2016-11-30.
Instructions for updating:
Please switch to tf.summary.FileWriter. The interface and behavior is the same; this is just a rename.
I tensorflow/core/distributed_runtime/master_session.cc:993] Start master session 91acfc1008531f4d with config:
Traceback (most recent call last):
File "cancer_classifier_new.py", line 241, in
tf.app.run(main=main)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 43, in run
sys.exit(main(sys.argv[:1] + flags_passthrough))
File "cancer_classifier_new.py", line 209, in main
with sv.managed_session(server.target) as sess:
File "/usr/lib/python2.7/contextlib.py", line 17, in enter
return self.gen.next()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 974, in managed_session
self.stop(close_summary_writer=close_summary_writer)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 802, in stop
stop_grace_period_secs=self._stop_grace_secs)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/coordinator.py", line 386, in join
six.reraise(*self._exc_info_to_raise)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 963, in managed_session
start_standard_services=start_standard_services)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 720, in prepare_or_wait_for_session
init_feed_dict=self._init_feed_dict, init_fn=self._init_fn)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/session_manager.py", line 227, in prepare_session
config=config)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/session_manager.py", line 173, in _restore_checkpoint
saver.restore(sess, ckpt.model_checkpoint_path)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1388, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 766, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 964, in _run
feed_dict_string, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1014, in _do_run
target_list, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1034, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Cannot assign a device to node 'save/RestoreV2_8': Could not satisfy explicit device specification '/job:ps/task:0/device:CPU:0' because no devices matching that specification are registered in this process; available devices: /job:worker/replica:0/task:0/cpu:0
[[Node: save/RestoreV2_8 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:ps/task:0/device:CPU:0"](save/Const, save/RestoreV2_8/tensor_names, save/RestoreV2_8/shape_and_slices)]]
Caused by op u'save/RestoreV2_8', defined at:
File "cancer_classifier_new.py", line 241, in
tf.app.run(main=main)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 43, in run
sys.exit(main(sys.argv[:1] + flags_passthrough))
File "cancer_classifier_new.py", line 191, in main
saver = tf.train.Saver()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1000, in init
self.build()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1030, in build
restore_sequentially=self._restore_sequentially)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 624, in build
restore_sequentially, reshape)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 361, in _AddRestoreOps
tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 200, in restore_op
[spec.tensor.dtype])[0])
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 441, in restore_v2
dtypes=dtypes, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 759, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2240, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1128, in init
self._traceback = _extract_stack()
InvalidArgumentError (see above for traceback): Cannot assign a device to node 'save/RestoreV2_8': Could not satisfy explicit device specification '/job:ps/task:0/device:CPU:0' because no devices matching that specification are registered in this process; available devices: /job:worker/replica:0/task:0/cpu:0
[[Node: save/RestoreV2_8 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:ps/task:0/device:CPU:0"](save/Const, save/RestoreV2_8/tensor_names, save/RestoreV2_8/shape_and_slices)]]

I figure out it. The error caused by the code I updated.
After what I had finished what the WARNING told me to do, the error came out.
WARNING:tensorflow:From cancer_classifier.py:193 in main.: initialize_all_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Use `tf.global_variables_initializer` instead.
WARNING:tensorflow:From cancer_classifier.py:195 in main.: scalar_summary (from tensorflow.python.ops.logging_ops) is deprecated and will be removed after 2016-11-30.
Instructions for updating:
Please switch to tf.summary.scalar. Note that tf.summary.scalar uses the node name instead of the tag. This means that TensorFlow will automatically de-duplicate summary names based on the scope they are created in. Also, passing a tensor or list of tags to a scalar summary op is no longer supported.
WARNING:tensorflow:From cancer_classifier.py:196 in main.: scalar_summary (from tensorflow.python.ops.logging_ops) is deprecated and will be removed after 2016-11-30.
Instructions for updating:
Please switch to tf.summary.scalar. Note that tf.summary.scalar uses the node name instead of the tag. This means that TensorFlow will automatically de-duplicate summary names based on the scope they are created in. Also, passing a tensor or list of tags to a scalar summary op is no longer supported.
WARNING:tensorflow:From cancer_classifier.py:197 in main.: scalar_summary (from tensorflow.python.ops.logging_ops) is deprecated and will be removed after 2016-11-30.
Instructions for updating:
Please switch to tf.summary.scalar. Note that tf.summary.scalar uses the node name instead of the tag. This means that TensorFlow will automatically de-duplicate summary names based on the scope they are created in. Also, passing a tensor or list of tags to a scalar summary op is no longer supported.
WARNING:tensorflow:From cancer_classifier.py:199 in main.: merge_all_summaries (from tensorflow.python.ops.logging_ops) is deprecated and will be removed after 2016-11-30.
Instructions for updating:
Please switch to tf.summary.merge_all.
WARNING:tensorflow:From /usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/logging_ops.py:264 in merge_all_summaries.: merge_summary (from tensorflow.python.ops.logging_ops) is deprecated and will be removed after 2016-11-30.
Instructions for updating:
Please switch to tf.summary.merge.
WARNING:tensorflow:From /usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py:344 in __init__.: __init__ (from tensorflow.python.training.summary_io) is deprecated and will be removed after 2016-11-30.
Instructions for updating:
Please switch to tf.summary.FileWriter. The interface and behavior is the same; this is just a rename.

It may be the bug of the script for distributed training with latest TensorFlow.
I will refactor the code for distributed soon. If you want to run distributed TensorFlow application, please try tobegit3hub/distributed_tensorflow which is much better now.

@tobegit3hub Thank you!
I have tried tobegit3hub/distributed_tensorflow, it works!
But it still has the problem https://github.com/tensorflow/tensorflow/issues/5110

Yes, that's something we're working for now.


Fixes gh-2125

https://github.com/spring-cloud/spring-cloud-netflix/issues/2125
Updated okhttp3 from v3.6.0 to v3.8.1

Codecov Report

Merging #2170 into 1.3.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            1.3.x    #2170   +/-   ##
=======================================
  Coverage   76.25%   76.25%
=======================================
  Files         215      215
  Lines        6396     6396
  Branches      799      799
=======================================
  Hits         4877     4877
  Misses       1177     1177
  Partials      342      342

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 1d6b47c...c7260f2. Read the comment docs.

Committed another fix to PR branch accidentally. Will resubmit.


loss_per_head seems wrong

Hi, I think the calculation of loss_per_head is wrong.
def cross_entropy_loss(self, preds, targets): # n_heads, batch_size, logits
        preds = F.log_softmax(preds / self.hparams.temperature, dim=-1)
        return -torch.mean(torch.sum(targets * preds, dim=-1))

this code averages both 0 and 1 dims so the loss is not head-wise. When conducting self.loss_per_head += loss_cluster.clone().detach(), each head loss adds the same mean value.
 return -torch.mean(torch.sum(targets * preds, dim=-1), dim=-1) # revised

I think this would be good. If there's anything wrong with my understanding, please correct me. Thanks.

Yes, you are right. We made some changes to the repository recently and forgot to update this. I will fix it soon.

Thanks for the reply.


Wrong number in table 5

Hi,
thanks for sharing this odb information - really appreciated!
I think there is a small typo: in the first column of table 5, see https://leaf-obd.readthedocs.io/en/latest/pid/5b3.html#pid-5b3, it should be 5B3 instead of 3B3. Correct?

Thanks @Tobias-Georg you're correct. It should be 5B3 instead of 3B3.


fix: disable show all on mobile

closes #968
Prerequisites:

[x] Unless it is a hotfix it should be merged against the dev branch
[x] Branch is up-to-date with the branch to be merged with, i.e. dev
[x] Build is successful
[x] Code is cleaned up and formatted

Summary
hide the show all button on mobile

#968 is more a problem with Raw Data + Show All Dates than with Show All itself, no? If you turn off Raw, Show All works okay, it could maybe do with bigger vertical gridline spacing but the line is plenty visible.

#968 is more a problem with Raw Data + Show All Dates than with Show All itself, no? If you turn off Raw, Show All works okay, it could maybe do with bigger vertical gridline spacing but the line is plenty visible.

true I just implemented an XOR logic between "Show All" and "Raw Data"


Prune CI testing matrix

This PR addresses #615. I settled on (c) from that issue with an added Ubuntu/Python 3.9 base environment to run tests without the optional geopandas dependency.

Pruned matrix cuts test runtime to under 10 minutes. I think this is a good compromise @martinfleis. Thoughts?

I would suggest having the majority of tests based on stable libpysal and one env on master. So instead of installing dev version to every env, we would install it to one. We should try to replicate users' environments here and they will use the latest release in most cases, not master.
It means adding one, probably ubuntu-based env with DEV flag and keeping the condition for git installation as it is.

I would suggest having the majority of tests based on stable libpysal and one env on master. So instead of installing dev version to every env, we would install it to one. We should try to replicate users' environments here and they will use the latest release in most cases, not master.
It means adding one, probably ubuntu-based env with DEV flag and keeping the condition for git installation as it is.

This is a good idea. I'll update the PR.

One minor requests. GHA has an annoying tendency to trim the name of the test. If we call them all micromamba unittests (...) it is hard to see what they are apart from OS:

Can we use just ${{ matrix.os }}, ${{ matrix.environment-file }} as a name? I think it is clear these are tests.

One minor requests. GHA has an annoying tendency to trim the name of the test. If we call them all micromamba unittests (...) it is hard to see what they are apart from OS:

Can we use just ${{ matrix.os }}, ${{ matrix.environment-file }} as a name? I think it is clear these are tests.

Another great insight. I'll make that change and merge.


[Bug]: Map widget errors out on local community edition pull.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
On taking the latest release pull of Appsmith community edition, the maps widget is not functional.
Steps To Reproduce

Take a docker pull of appsmith-ce:release
Signup, Create an app, add the maps widget to the canvas.
It asks for api to be added.
Add the maps widget api in admin settings
Open the app. Note that there is a retry message. [This is addressed by another issue logged]
On refreshing the page, note that the map widget does not work.

https://github.com/appsmithorg/appsmith/assets/101863839/ae7ee756-bfda-4ebe-ae1f-4a0262b5864c
Note: Pulled master [appsmith-ce] locally and tried, it worked fine there.
Public Sample App
No response
Environment
Release
Issue video log
No response
Version
Self Hosted

Not an issue.


Limit customization file content to no more than 3000 lines(#2128)

Codecov Report

Merging #2141 into dev will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #2141      +/-   ##
============================================
- Coverage     29.86%   29.82%   -0.04%
- Complexity     1628     1632       +4
============================================
  Files           336      336
  Lines         17822    17835      +13
  Branches       2250     2249       -1
============================================
- Hits           5322     5320       -2
- Misses        11935    11950      +15
  Partials        565      565

Impacted Files
Coverage Δ
Complexity Δ

...duler/server/worker/task/ShellCommandExecutor.java
56.52% <0%> (-26.09%)
4% <0%> (-3%)

...nscheduler/server/worker/task/shell/ShellTask.java
70.83% <0%> (-6.25%)
5% <0%> (-2%)

...er/server/worker/task/AbstractCommandExecutor.java
36.74% <0%> (-4.66%)
14% <0%> (-1%)

...uler/common/utils/process/ProcessImplForWin32.java
0.66% <0%> (-0.39%)
1% <0%> (ø)

.../apache/dolphinscheduler/common/utils/OSUtils.java
48% <0%> (-0.35%)
23% <0%> (+1%)

.../dolphinscheduler/server/utils/FlinkArgsUtils.java
88.46% <0%> (-0.22%)
10% <0%> (-1%)

...duler/server/worker/runner/TaskScheduleThread.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
80% <0%> (ø)
15% <0%> (ø)
:arrow_down:

...pache/dolphinscheduler/common/utils/FileUtils.java
20% <0%> (+2.41%)
9% <0%> (+2%)
:arrow_up:

...he/dolphinscheduler/common/thread/ThreadUtils.java
77.96% <0%> (+6.77%)
13% <0%> (+1%)
:arrow_up:

... and 2 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 450a1f5...84bcdc7. Read the comment docs.


feat: Bump Golang to 1.17

Signed-off-by: Gábor Lipták<EMAIL_ADDRESS>

@gliptak can you run "go mod tidy" and push this please?

https://github.com/fidelity/kconnect/pull/358/checks?check_run_id=3545560305#step:3:10

https://github.com/fidelity/kconnect/pull/358/checks?check_run_id=3545560305#step:3:10

This is just an issue with this action, can ignore this


Update 13.5

Issue with newly updated 13.5 is breaking middle off fight and the alert delays when he reconnecting. Both alert and timer on raidboss

Unless this happened before the 5.18 update, this is most likely caused by missing network lines. Wait for the next parser update and then check again.

https://gist.github.com/ngld/e2217563bbbe1750c0917217f136687d#current-stuff


Unable to get data by calling GetSnapshotAsync

Hello Team,
The example is working fine in a console application but not working in a windows forms applicaton.
Code
    private static void Form1_Load(object sender, EventArgs e)
    {
        string filepath = @"C:\Birghtstar\RnD\DotNet\NotificationTest\SendNotificationTest\SendNotificationTest\Env\jdesimbamonitoring-65815de98a72.json";
        Environment.SetEnvironmentVariable("GOOGLE_APPLICATION_CREDENTIALS", filepath);

        FirestoreDb db = FirestoreDb.Create(project);
        // [START fs_get_all]

        RetrieveAllDocuments("jdesimbamonitoring").Wait();
    }

    private static async Task RetrieveAllDocuments(string project)
    {
        FirestoreDb db = FirestoreDb.Create(project);
        // [START fs_get_all]
        CollectionReference usersRef = db.Collection("Users");
        **QuerySnapshot snapshot = await usersRef.GetSnapshotAsync();**
        foreach (DocumentSnapshot document in snapshot.Documents)
        {
            Console.WriteLine("User: {0}", document.Id);
            Dictionary<string, object> documentDictionary = document.ToDictionary();
            Console.WriteLine("First: {0}", documentDictionary["First"]);
            if (documentDictionary.ContainsKey("Middle"))
            {
                Console.WriteLine("Middle: {0}", documentDictionary["Middle"]);
            }
            Console.WriteLine("Last: {0}", documentDictionary["Last"]);
            Console.WriteLine("Born: {0}", documentDictionary["Born"]);
            Console.WriteLine();
        }
        // [END fs_get_all]
    }

It stuck in the line QuerySnapshot snapshot = await usersRef.GetSnapshotAsync();
Can you please help me here.

This is the problem:
RetrieveAllDocuments("jdesimbamonitoring").Wait();

By calling Wait(), you're blocking the UI thread - it can't do anything else until the task returned by RetrieveAllDocuments has finished.
However, that task can't complete because it needs to come back in the UI thread here:
QuerySnapshot snapshot = await usersRef.GetSnapshotAsync();

So you've got a deadlock.
The fix is to avoid that Wait() call, probably by making your Form1_Load method asynchronous and awaiting the task instead. (There are other approaches, but that's probably the simplest:
private static async void Form1_Load(object sender, EventArgs e)
{
    string filepath = @"C:\Birghtstar\RnD\DotNet\NotificationTest\SendNotificationTest\SendNotificationTest\Env\jdesimbamonitoring-65815de98a72.json";
    Environment.SetEnvironmentVariable("GOOGLE_APPLICATION_CREDENTIALS", filepath);

    FirestoreDb db = FirestoreDb.Create(project);
    await RetrieveAllDocuments("jdesimbamonitoring");
}

That allows the UI thread to await asynchronously - it won't block while the documents are being fetched.
Note that this isn't a Firestore-specific issue - you'd see the same thing in any asynchronous code.

(I'll close this issue now as I'm very confident that's the problem. Please add a comment if this doesn't work for you and you need the issue to be reopened.)


Getting Error when trying to run feeder.js

Here is my error:

Line 198 of feeder.js error

just disable safeMoveTo in the config file, it doesn't work yet anyway.

I fixed it with removing the 2 {} brackets on line 198, working now but i cant connect to minimap server

which node version ?

Fixed* Just in your code David remove the two brackets {} on line 198 of feeder.js

which node version ?

tutorial will be available her: https://github.com/davidmann4/agario-feeder-bot/wiki

I have the same issue.
$ node -v
v0.12.9
I was using this version because we was running minimap server.
Which version you are using now?

I use node 0.12.9 as well. I just install the node version manager (nvm) and change it to 0.12.9. But it's fixed now if you read through this issue's comments.


wuffs-c requires clang-format-5.0

Is there a reason why version 5.0 is explicitly required? I only have clang-format installed (version 6.0), and creating a symlink in my $PATH keeps wuffs-c running normally.
I presume that the tool is used to format the C output from the generator. Why not make it configurable though? It could, for example, default to clang-format, where one could change it to clang-format-5.0 or any other desired version.

Why is an explicit version required? Why clang-format-5.0 and not just clang-format?
In the Wuffs repo (but not your repo), everything checked in under gen/c is clang-formatted. Different versions of clang-format will format the same C code differently. While the gen/c changes in https://github.com/google/wuffs/commit/abdf46045f1e7dd5d3a51ccdcc0f58ebbad8b96c are possibly a clang-format bug, it does illustrate my point. Without an explicit version, two people (or the same person working on two different systems) could cause spurious edits (and edit-revert wars) when checking in their changes.
Why version 5.0? Why not 4.0 or 6.0?
It's somewhat arbitrary, but 5.0 seemed a reasonable compromise of being not too old and not too new, as of today. A future change will undoubtedly bump 5.0 to something higher, whether 5.x or y.0.

Having said that, it'd be possible to make it configurable...


Add warnings for more scenarios

Previously: #33.

The WordPress Coding Standards project has a few sniffs that we could borrow from to improve the warnings created by the make-pot command.
These sniffs currently do:

Check for the presence of a translators comment if one of the text strings contains a placeholder.
Check for inconsistencies between single and plural arguments.
Check for multiple unordered placeholders.
Check that the translatable string itself is not empty

Source: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/blob/eccb1f721f95fdcf5c2e383340aa4b094c0317ad/WordPress/Sniffs/WP/I18nSniff.php#L36-L59
Let's determine whether it makes sense to port some of these over to WP-CLI.

Fixed by #64.


feature: Streaming mode in demo

Hey, really cool work. I've been working on a streaming mode demo that transcribes the stream straight from the mic.

What's the best way to present and contribute this work ? Do you have a process for accepting contributions ?

Hey! 👋 That's awesome! Feel free to submit a PR :)

Hey, really cool work. I've been working on a streaming mode demo that transcribes the stream straight from the mic.嘿，真的很酷的工作。我一直在做一個流媒體模式演示，直接從麥克風轉錄流媒體。

What's the best way to present and contribute this work ? Do you have a process for accepting contributions ?展示和貢獻這項工作的最佳方式是什麼？你們有接受捐款的流程嗎？

hey man,i using this project by main branch, but I didn't see record button.
Do you using webgpu-whisper branch?
thanks a lot

Hi @xenova, https://huggingface.co/spaces/Xenova/realtime-whisper-webgpu I noticed that this mode also exists on HuggingFace. Are there any plans to be bringing it to this library in the near future?
Thank you for all your hard work!


Does this tool support configuring a proxy?

I want to use this tool to au-create the issue in my repo. but my env is private, and we provide a proxy to access the internet. so I am not sure if I can configure a proxy with this tool to call my github repo. thanks.

You can use HTTP_PROXY and/or HTTPS_PROXY environment variable probably. Have a try!
$ HTTP_PROXY="..." HTTPS_PROXY="..." alertmanager-to-github start

    spec:
      containers:
      - name: alertmanager-to-github
        image: ghcr.io/pfnet-research/alertmanager-to-github:v0.1.0
        env:
        - name: HTTP_PROXY
          value: "..."
        - name: HTTPS_PROXY
          value: "..."
    ...


Reconsider the whole 60 day upvote process for feature requests

I completely understand that there are tons and tons of issues opened up due to the size of this project, but unfortunately it seems that the current process skews towards prioritizing certain types of features over others.  I haven't done any scientific analysis but think about it for a second... it would make sense that a feature that's a clone of some other program would be requested more frequently than an innovative one that may take more time to build up stream?  Also I'm sure there a timing/event based variables that play into the behavioral patterns... but regardless of the root cause, I see so many of certain types of features that are implemented based on one open issue that received the upvotes, while the sum of open issues from the multiple closed tickets vastly exceeds the interest level.
This also seems to be causing numerous duplicate issues to be created because the 60 day window expires and it forces another issue to be generated un-necessarily.
Any possibility of refining these parameters?  250 votes in 180days? 100 votes in 90 days?
(Maybe this doesn't apply to every issue but to certain ones that have significant support in the past or subjectively seem innovative and beneficial to give them chance to build up momentum/stir up a buzz?

Agree with this
Is there a reason why limits the voting period to 60 days, not indefinitely and start to implement after it reaches certain number of votes?
Or maybe move the feature requests from github issue to other place or even another repo like Laravel does with laravel/ideas?

Agree with this
Is there a reason why limits the voting period to 60 days, not indefinitely and start to implement after it reaches certain number of votes?
Or maybe move the feature requests from github issue to other place or even another repo like Laravel does with laravel/ideas?

And it seems like the policy is also to close and lock all future issues as duplicates. I went looking for a feature I really wanted and found it had already been requested before here and closed again here. So does this mean everyone is now permanently blocked from requesting or discussing this feature ever again?
There are thousands of issues like this


rest for 4 beat broken

Describe the bug
The rest for block does not work when 4 beat [sic] is selected. 2 beat is ok and I'm using that as a workaround.
This shows up in simulator.
To Reproduce
Steps to reproduce the behavior:

Make a rest for 4 beat block
Use it in some way that absence of the pause is evident

Expected behavior
I would expect it to pause the code for 4 beats, a time based on the global bpm.
Screenshots
See attached code at bottom.
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: 76.0.3809.100

Smartphone (please complete the following information):

n/a

Additional context
I'd suggest testing all of these in case any others are broken.
This is in the beta beta:
arcade version:  0.13.20
Microsoft MakeCode version:  5.19.9

Embedded code in png, this code does other stuff, you've got to run it and get to the final part where it says Polyphony test next:

The tree trunks are acting as a visual indication of where the code has got to across the threads. The if in the note3 block demonstrates the trivial workaround.
Flicking into JavaScript, I see the naming is inconsistent and wrong? 2 beats is BeatFraction.Double and 4 beats is BeatFraction.Breve. I would expect:

2 beats = minm
4 beats = semibreve

@pelikhan is this the right terminology?
Also, I cannot repro the 4 beat issue. Seems to work fine for me on hardware and in the simulator. Do you have a more minimal test case you could share that clearly demonstrates the issue?
Love the polyphonic melody you have going on there BTW!

Polyphony test will be silent in the non-beta version.
I just shut my browser down just in case there was some state issue. Firing it back up and going into incognito mode and dragging code in via magic png the bug is reproduced. If you see the trunk with 1 leaf circle appear at the same time as the 3 leaf circle one then it's buggy.

In addition to Chrome, I literally see (via the trunks on screen) the bug in both versions of MakeCode Arcade on Firefox 68.0.1. Both versions fail to output the polyphonic audio in Firefox, btw.

It's bpm related, below 120 it's borked.

This should be addressed. If not, please reopen.


Get StreamCat data by location

Function to allow user to pass lat / lon or x / y coordinates in known projection and retrieve corresponding COMIDs and StreamCat data

sc_get_comid function now accomplishes this


Expose interactive ui for manage world migrations

This task is to expose an interactive ui for managing a worlds migration.
Once we have resolved #161 and #162 and have a migration strategy for a world, the cli should provide an interactive way to update the world to reflect the changes.
Updating a remote world involves:
Validation:

Verify the caller has permission to update any exist components and systems that are registered
Verify that the caller has permission to extend write permission to any components that are being updated by systems

Execution:

Declaring any new classes (each declare requires a transaction iirc)
Register classes by calling register_system and register_component on the world
https://github.com/dojoengine/dojo/blob/c515e9e839c848e132506085be6eed67195cc758/lib/src/world.cairo#L72

2.1) For new classes, they can be registered permissionlessly
2.2) For existing classes, they can only be registered if the caller has permission (for now this will result in a transaction error, eventually we should expose a way for someone to propose an update that can be confirmed by someone with permission.

Provide write access for systems to any necessary components.

3.1) If the component is new, this can be done permissionlessly
3.2) If the component is existing and the caller has permission to grant write access, do so
3.3) If the component exists but the caller doesn't have permission to grant, request access (not supported right now)
One approach here could be to use TUI to interactive visualize and manage the tasks. Happy to explore other simpler approaches too.

If TUI or else, we should make this issue about creating a dojo-gui crate, the feature above should be the first element, but we should design it so it is easy to extend.

I thin the TUI approach is interesting, especially for the explorer. I'm not sold that it is the right for the migration runner, perhaps we do something simpler to begin with? Something more similar to kubectl which has similar concerns

Maybe this - https://github.com/mikaelmello/inquire ?

Oh that looks interesting. I'll leave it to @kariy to decide

Will start looking into this now.
I think a simple cli interface using inquire should be enough atm... I can try scaffolding a simple dojo-gui crate while waiting for the migration feature to be completed


Update Translations for 4.22

with lots of new text in the GUI,
Translations need to be updated

Please provide a link for easy signup.


How to show django errors, using {% bootstrap_field  %} ?

Hi, I have this code:

                                {% bootstrap_field form.bairro_entrega %}

                                {% bootstrap_field form.cidade_entrega %}

                                {% bootstrap_field form.cep_entrega %}

All 3 fields are required. When I click on submit, the browser go to the first empty field, ok.. its great! but doesnt show any message. How can I show a message to the user?
Cheers

Hi, doesn't work for me with fresh install of django-bootstrap3-develop, django 1.11, Python 3.4,


crypto:rand_bytes/1 is deprecated

wondering if the other branch which includes the change to crypto:strong_rand_bytes/1 will get merged soon?

I was doing some fixing, including the crypto:strong_rand_bytes/1 change.
BTW, I merged your PR.
Thanks for your contribution.


Vina force field

https://github.com/ccsb-scripps/AutoDock-Vina/blob/d13f9e93e253573b01ffbb28cfd1e2dd26ff9002/docs/source/docking_hydrated.rst?plain=1#L138
When I use Vina forcefield, I get this error: ERROR: Affinity map for atom type C_H is not present.
How to fix this problem?

I'm not entirely sure if C_H is a valid atom type for a PDBQT file, but, considering it is, you should include it when creating your .gpf file. In Section 3, prepare_gpf.py receives a ligand_types parameter, include this atom type in it so autogrid can generate a grid map file for C_H as well.
Something like:
pythonsh <script_directory>/prepare_gpf.py -l 1uw6_ligand.pdbqt -r 1uw6_receptor.pdbqt -y \
           -p ligand_types='A,NA,C,HD,N,OA,C_H' \

Keep in mind that it might be a bad naming in your .sdf file as well,

I'm not entirely sure if C_H is a valid atom type for a PDBQT file, but, considering it is, you should include it when creating your .gpf file. In Section 3, prepare_gpf.py receives a ligand_types parameter, include this atom type in it so autogrid can generate a grid map file for C_H as well.
Something like:
pythonsh <script_directory>/prepare_gpf.py -l 1uw6_ligand.pdbqt -r 1uw6_receptor.pdbqt -y \
           -p ligand_types='A,NA,C,HD,N,OA,C_H' \

Keep in mind that it might be a bad naming in your .sdf file as well,

I tried, but the next step gave me another error: autogrid4: ERROR:  unknown ligand atom type C_H

I think you want to use option "--scoring ad4" with vina.


Update sbt-pgp to 2.0.2

Updates com.jsuereth:sbt-pgp from 2.0.1 to 2.0.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.jsuereth", artifactId = "sbt-pgp" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #177 (0005c5c) into master (a7855ae) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #177   +/-   ##
=======================================
  Coverage   96.07%   96.07%
=======================================
  Files           4        4
  Lines         102      102
  Branches        4        4
=======================================
  Hits           98       98
  Misses          4        4

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 a7855ae...0005c5c. Read the comment docs.


Inverted "flipX()" and "flipY()"

svg-path-commander.js#L252
flipX() and flipY() should be inverted, flipX() is flipping the y coordinates and flipY()  the x coordinates ... Not a big deal, but worth mentioning.
Great library by the way, I'm using it to create a small SVG path editor so big thanks.

The logic is correct:
  flipX() {
    this.transform({ rotate: [180, 0, 0] }); // first value corresponds to X
    return this;
  }

  flipX() {
    this.transform({ rotate: [0, 180, 0] }); // second value corresponds to Y
    return this;
  }

Great library by the way, I'm using it to create a small SVG path editor so big thanks.

Link it in.

If by flipping you mean rotating, then yes the logic is correct, but most people think of flipping as mirroring and you can see this behavior in a lot of graphic libraries like fabric.js, konva.js, three.js ... so the logic would be more of a reverse scale.
 flipX() {
    this.transform({ scale: [-1, 1, 1] });
    return this;
 }

 flipY() {
    this.transform({ scale: [1, -1, 1] });
    return this;
 }

Link it in.

It's part of an ongoing project, the SVG editor part will look more or less like this demo here ... I'll keep you in touch when the project is done.

I just noticed that the transform method don't handle all scale axes so in this case just inverting the two methods work fine.
  flipX() {
    this.transform({ rotate: [0, 180, 0] }); // Rotate the Y axis, mirror the X axis
    return this;
  }
  flipY() {
    this.transform({ rotate: [180, 0, 0] }); // Rotate the X axis, mirror the Y axis
    return this;
  }

I just noticed that the transform method don't handle all scale axis so in this case just inverting the two methods works fine.

Yes it does. Perhaps we have a bug?

Yes it does and it works ... please don't mind my last post.

@Fennec-hub you are right about flipX / flipY, our original source in SVGPath was like that and I changed it at some point.
I remember I was working with font-icons -> SVG icons and it didn't make sense to me I couldn't flip icons upside-down because I was confused on why flipX did not return the expected outcome.
I just checked the native DOMMatrix interface and it does exactly the same as you expected, so...
The future version will fix that.

Glad to be of help and thanks again for assembling this great library

@Fennec-hub please have a look at the latest master. Have fun


3.9.0 BE - Quests that require dogtags only accept the original ones.

General information
Project issue is for:
3.9.0 BE
Expected result:
Queststo accept all dogtags, Normal / EOD / Unheard
Actual result:
Only Normal Dogtags are accepted
(Add a description of the issue here)
How to reproduce

Get some Dogtags
Check quest that require dogtags for handing in "Friend from the West part 1  the earlierst from Skier"

P.S: Due to quests being static, it is a worry that it might require editign all quests manually that have dogtags detection / handing in xD

Originally written by Stealthsuit

Need to be closed, solved.

Originally written by Stealthsuit


Some async_check_and_refresh_token

Please check Services, Known Bug / Issues and Troubleshooting over here first: https://github.com/fuatakgun/kia_uvo/blob/master/README.md
Region and Brand of car
UK Hyundai Tucson
Version of the integration
2.6.1 & 2.8.1
Describe the bug
Previously working integration now fails to initialize
Debug logs if an error occurred
This error originated from a custom integration.

Logger: homeassistant.config_entries
Source: custom_components/kia_uvo/coordinator.py:166
Integration: Kia Uvo / Hyundai Bluelink (documentation, issues)
First occurred: 20:33:46 (1 occurrences)
Last logged: 20:33:46

Error setting up entry Hyundai Europe<EMAIL_ADDRESS>for kia_uvo
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 383, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/kia_uvo/__init__.py", line 53, in async_setup_entry
    await coordinator.async_config_entry_first_refresh()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 215, in async_config_entry_first_refresh
    await self._async_refresh(
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 250, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/kia_uvo/coordinator.py", line 107, in _async_update_data
    await self.async_check_and_refresh_token()
  File "/config/custom_components/kia_uvo/coordinator.py", line 166, in async_check_and_refresh_token
    await self.hass.async_add_executor_job(
asyncio.exceptions.CancelledError

To Reproduce
Just run the integration
Expected behavior
It used to initialize
Screenshots
Additional context
Issue conincided with updating to Docker HA 2023.3.6

Well, apparently the trick is to comment out a section in configuration.yaml about nextcloud and restart. It's working perfectly now.


Convenience Actors Don't Replicate Correctly

Hi @getnamo. Thanks for this great plugin.
I'm trying use this plugin to give networked VR pawns their own Leap Motion controlled hands.
At the moment, I'm using UE4.13 Preview 2 and the Oculus CV1 (I've also tested with UE4.12 and had the same results).
I have modified the HMDLocomotion pawn to add a LeapEchoRiggedHandsActor as a Child Actor on the camera. When running a single instance of the game on one computer, this works great, I can see some rigged and animated hands thanks to the Leap Motion. When I package and run the project on two different computers, one hosting and one as a client, my network session blueprints work correctly - both players can see each other, and can see hands on the VR pawns, but the hand animations for all VR pawns are driven by the local player's leap motion device.
I want a way to configure the Leap Controller, or LeapEchoRiggedHandsActor class so that locally controlled rigged hands are correctly driven by the local leap controller, and remotely controlled rigged hands are correctly replicated by the server.
I am happy to help with finding a solution to this bug - however I don't know much about how animation blueprints work, so am stuck at the moment. Any suggestions?
NB: This question is a re-post from my UE4 AnswerHub question.

I'm not entirely sure where the network replication fails, but for some reason the network barrier I tried putting in 4.12 doesn't work. Ideally this should all happen in C++ but you can do it in blueprints if you follow https://www.unrealengine.com/blog/blueprint-networking-tutorials
This isn't going to be a focus for me for a bit of time, but you could also try to take any lessons from the vrexpansion plugin, which supports networked hmd and controllers: https://bitbucket.org/mordentral/vrexpansionplugin  (replicated camera header: https://bitbucket.org/mordentral/vrexpansionplugin/src/5b45f4f89518ed697e6ab4846ef65f34d260e5dd/Source/VRExpansionPlugin/Public/ReplicatedVRCameraComponent.h?at=default&fileviewer=file-view-default)


Fix disposal unit visuals and issue with sometimes not pressurizing

About the PR

In the case of Flush button already pressed before fully pressurized, the pressurization loop would exit early since the state was not updated after flushing. It returned while thinking it was still fully pressurized, so it exited the loop despite a pressure of zero, so adding an update to the state after TryFlush().
Also noticed the visuals were only being updated on initialize/flush/engage/disengage, so added updates upon fully pressurized and after flush animation completes.
Fixes #6332

(Also I thought I was going crazy since Disposal\units.yml already had the flushTime field under DisposalUnit type. I guess this is already similar to how they were doing it in other repos?)
Screenshots

FlushStart and FlushTime used to start the flashing light after flush animation:

Changelog

:cl:

fix: Fixed disposal units sometimes not pressurizing properly
fix: Fixed disposal unit visuals

@metalgearsloth
It looks like this is waiting on #8944
Would it be better for me to make a small PR just to fix the pressurizing issue in the meantime?

has this been superseded?

has this been superseded?

There were some changes here to the animation, but sloth found out it needs some more changes and opened #8944
Don't know if that means we should just close this or if it is okay for now until the animations are done properly.
shrug

Closing this since the bug was fixed elsewhere, and the visuals need more work to be done properly.


Allow to specify the chat_id for telegram driver in low level sendReq…

Allow to set a specific chat_id to sending messages/stickers/etc to a specific telegram user/channel. Useful to send stickers, messages, in channel managed by a bot.

Instead of using this, we should use array_replace_recursive for all methods that currently use array_merge_recursive. We don't need that if there :)

Codecov Report

Merging #312 into master will decrease coverage by 0.07%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #312      +/-   ##
============================================
- Coverage     85.11%   85.03%   -0.08%
- Complexity      801      802       +1
============================================
  Files            68       68
  Lines          2277     2279       +2
============================================
  Hits           1938     1938
- Misses          339      341       +2

Impacted Files
Coverage Δ
Complexity Δ

src/Mpociot/BotMan/Drivers/TelegramDriver.php
74.41% <0%> (-1.78%)
22 <0> (+1)

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 418a4a3...b1032c5. Read the comment docs.

The array_replace_recursive function not solve the problem:
$a =['chat_id'=>123];
print_r(array_replace_recursive($a, ['chat_id' =>456]));

The output is an array with chat_id equals to 456, so the value is replaced also in this case.

Isn't this the behavior you want, if you swap the parameter order?
https://3v4l.org/WO7QH#output

Yeah, I misunderstood your precedent comment sorry. I'll update the pull request.


Popup notification in android

display the pop-up notification in android

it would be better to use custom notifications instead of changing what the default is for however many users use this package, especially when majority of notifications aren't high priority

It's fine but I think you have to give the option to change notification
priority
On Mon, May 4, 2020 at 3:52 PM Ben Wildeman<EMAIL_ADDRESS>wrote:

it would be better to use custom notifications instead of changing what
the default is for however many users use this package, especially when
majority of notifications aren't high priority
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wix/react-native-notifications/pull/597#issuecomment-623381812,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANKRVNC5BO7IR5IZ5GHGJETRP2JNVANCNFSM4MTWQPWQ
.

Android pop-up when app is in background or foreground is still not showing, does this solve it?


New Enum Variants to CompletionItemKind

Hi, thanks so much for this project! I'm currently adding completions to an assembly lsp. Would it be possible to add Instruction and Register types to the CompletionItemKind enum in src/completion.rs? Would adding these variants require changes to any other pieces of the code? Happy to add the variants to the enum and open up a PR if that would make things easier.

The variants here are reflect those in the LSP spec and since they are encoded as plain integers there isn't a way to extend it with custom kinds. (Even if both your client and server would understand your custom variants, we can't add the variants in the library since any additions to the spec would likely conflict with those custom ones).
I'd recommend sticking with the existing ones, Instruction maps decently well to OPERATOR and Register to Variable for instance
https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#completionItemKind

That makes sense, thank you so much for the response and helpful recommendation! I'll go ahead and close this issue as resolved.


List examples on /

Right now it errors out. We should list examples from the readme there

I can take this one.... I've an idea.

https://puppeteeraas.com/
We can open another issue to manage how it looks.


Modbam_to_tabix with 0-based coordinate system

Hi Shian!
I have a few UCSC aligned BAM files. UCSC uses a 0 based coordinate system unlike the Ensembl genome builds.
I have my ModBamResult object and I am trying to use modbam_to_tabix to get it into tabix format. I noticed that with Ensembl-aligned bam files, everything works, however the bam files for my data are aligned to UCSC. Is there a way to make this function work for 0 based coordinate systems?
I get the following output:
methy <- modbam_to_tabix(mbr, "res.tsv.bgz")
`✔ Data converted: sample1_sorted.bam [8m 15.3s]
✔ Data converted: sample2_sorted.bam [4m 34s]
✔ Data converted: sample3_sorted.bam [9m 51.9s]
✔ Data converted: sample4_sorted.bam [13m 1.7s]
✔ Data converted: sample5_sorted.bam [10m 52s]
✔ Data converted: sample6_sorted.bam [8m 56.3s]
✔ Converting data to TSV [55m 31.6s]
✔ Sorting data [12m 21.8s]
...
[W::tbx_parse1] Coordinate <= 0 detected. Did you forget to use the -0 option?
[W::tbx_parse1] Coordinate <= 0 detected. Did you forget to use the -0 option?
[W::tbx_parse1] Coordinate <= 0 detected. Did you forget to use the -0 option?
[E::hts_idx_push] Unsorted positions on sequence #10: 248945815 followed by 1
Error in value[3L] : index build failed
file: res.tsv.bgz
✖ Compressing data [5m 1.2s]'
Thanks a lot for the help! :)

As far as I know, BAM should always be 0-based and SAM always 1-based. I think the issue occurred elsewhere, how large is the resultant TSV file and how much memory do you have? I think sort sometimes does funny things when there isn't enough memory and corrupts the file.

I think there may be an issue in my conversion function. I will let you know when I have updated with a fix.

Hello Shian,
The BAM files that I used had alignment and methylation calls done by Dorado v.7. When I do it with Dorado v.5, the modbam_to_tabix funciton works. I am currently re-basecalling my nanopore data with different options for Dorado v7. I suspect that the issue lies therein. I will keep you updated with how it goes.
Best,
Grigory


doc: added count using a getattr() function

Added count using getattr() for issue #165

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

Codecov Report

Merging #166 (7c2a803) into master (cad5273) will decrease coverage by 0.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
- Coverage   92.63%   92.06%   -0.57%
==========================================
  Files           6        6
  Lines        2675     2496     -179
==========================================
- Hits         2478     2298     -180
- Misses        197      198       +1

Impacted Files
Coverage Δ

pyodata/v2/model.py
92.69% <0.00%> (-0.72%)
:arrow_down:

pyodata/v2/service.py
90.18% <0.00%> (-0.30%)
:arrow_down:

Continue to review full report at Codecov.

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


Add option to not load neurons after downloading them

Fixes #12

If this looks good to you I will add tests for this new feature.


LinuxLiveDataReader.GetVersionInfo does not produce the correct answer

ClrMd 1.1.46104 (also reproduces on latest master), .NET Core 2.1 on Linux x64 (Ubuntu 18.04)
Unhandled Exception: System.ArgumentOutOfRangeException: Version's parameters must be greater than or equal to zero.
Parameter name: revision
   at System.Version..ctor(Int32 major, Int32 minor, Int32 build, Int32 revision)
   at Microsoft.Diagnostics.Runtime.Linux.LinuxLiveDataReader.ParseForVersion(String s)
   at Microsoft.Diagnostics.Runtime.Linux.LinuxLiveDataReader.GetVersionInfo(UInt64 addr, VersionInfo& version)
   at Microsoft.Diagnostics.Runtime.ModuleInfo.get_Version()
   at Microsoft.Diagnostics.Runtime.DataTargetImpl.InitVersions()
   at Microsoft.Diagnostics.Runtime.DataTargetImpl.get_ClrVersions()
   at (my code)

This happens when using a self-contained deployment and the directory name is a GUID, separated with dashes, like "909B593B-4E9B-4294-AC28-FD5A369FA776".
Looking at what's happening in LinuxLiveDataReader.GetVersionInfo, it's trying to parse the directory name containing .so files as a version (and apparently accepts any hexadecimal numbers, too!) but this obviously doesn't work for self-contained deployments: the directory name may be a valid version, but not the CLR version.

It seems to somehow work for most instances of my application (all of them running from directories named as GUIDs), but not this one. Maybe it has to do with the order of LinuxLiveDataReader._memoryMapEntries.

Yeah that's pretty sketchy code.  I should have reviewed that one closer when I merged it.
I'll replace that with just a Regex after I confirm there's not a better way to get the version of .Net Core.

It's a problem I've run into myself and I've been reading it from the .deps.json file for self-contained deployments. Here's the code I use to get the version of the running .NET Core app, which I think can be easily modified to work with arbitrary application paths:
        private const string FrameworkPackageName = "Microsoft.NETCore.App";

        private static Version GetNetCoreRuntimeVersion()
        {
            var version = Environment.Version;

            // In .NET Core earlier than than 3.0 Environment.Version returns this hardcoded version: https://github.com/dotnet/coreclr/pull/22664/files
            if (version != new Version(4, 0, 30319, 42000))
                return version;

            // ... so try to get the real version from the assembly path (yes, this is the only way: https://github.com/dotnet/corefx/issues/28132)
            // Of course, this does not work for self-contained deployments.

            var assembly = typeof(System.Runtime.GCSettings).Assembly;
            var assemblyPath = assembly.CodeBase.Split(new[] {Path.PathSeparator, Path.AltDirectorySeparatorChar}, StringSplitOptions.RemoveEmptyEntries);

            string versionString;

            int netCoreAppIndex = Array.IndexOf(assemblyPath, FrameworkPackageName);
            if (netCoreAppIndex >= 0 && netCoreAppIndex < assemblyPath.Length - 1)
            {
                versionString = assemblyPath[netCoreAppIndex + 1];
            }
            else
            {
                try
                {
                    versionString = GetNetCoreRuntimeVersionFromSelfContainedDeployment(); // Probably a self-contained deployment - try reading .deps.json
                }
                catch (Exception ex)
                {
                    Trace.WriteLine("Failed to get .NET Core version for self-contained deployment: " + ex);
                    return null;
                }
            }

            if (versionString == null)
                return null;

            Version.TryParse(versionString, out version); // version will be null if not parsed successfully
            return version;
        }

        private static string GetNetCoreRuntimeVersionFromSelfContainedDeployment()
        {
            // A huge hack based on https://stackoverflow.com/a/55720218/1536933
            // .NET Core apps published as "self-contained" have a .deps.json file like this
            // {
            // ...
            //  "targets": {
            //    ".NETCoreApp,Version=vX.Y/PLATFORM": {
            //      ...
            //      "Microsoft.NETCore.App/X.Y.Z"
            // where X.Y.Z is the .NET Core runtime version we're after!

            const string frameworkPropertyPrefix = FrameworkPackageName + "/";

            var entryAssembly = Assembly.GetEntryAssembly();
            if (entryAssembly == null)
                return null;

            var depsFilePath = Path.ChangeExtension(entryAssembly.Location, ".deps.json");
            if (!File.Exists(depsFilePath))
                return null;

            using (var fileReader = new StreamReader(depsFilePath))
            using (var jsonReader = new JsonTextReader(fileReader))
            {
                var depsJson = JObject.Load(jsonReader);

                var targets = depsJson.Property("targets")?.Value as JObject;
                if (targets == null)
                    return null;

                foreach (var target in targets.Properties())
                {
                    if (target.Value is JObject targetObject)
                    {
                        foreach (var targetProperty in targetObject.Properties())
                        {
                            if (targetProperty.Name.StartsWith(frameworkPropertyPrefix, StringComparison.Ordinal))
                                return targetProperty.Name.Substring(frameworkPropertyPrefix.Length);
                        }
                    }
                }

                return null;
            }
        }

I think we should just stop trying to parse the version on Linux, at least for now.  On Windows we embed this information near the beginning of the dll.  On linux this lives at a really deep offset.  (For example search libcoreclr.so for the ascii string "@(#)", which is where the version is.  In my local test environment the offset was 0x0079f480 into the binary.
I'm not sure of a faster way to look this information up other than searching the binary for it, but maybe there's a pointer I could look for.
I'm also not sure this string would be placed into a mini coredump if generated through dotnet-dump.  This would mean even if we fixed the live data target to search for and read it, we would get inconsistent results in a coredump case where libcoreclr.so wasn't available locally.
I will probably circle back to this in the future.

Isn't that going to break something else? Was the version not used for anything?

Inside the library, it was used to make windows symbol server requests for mscordaccore_[version/arch].dll but we don't use that archiving mechanism for Linux.  I double checked that we don't use that version for anything in .Net Core (but we do on desktop) and tested it.
I have two main thoughts here:

I'd rather report no answer (version of <IP_ADDRESS>) all of the time rather than report the correct answer some of the time.
I should have left this issue open to track fixing LinuxLiveDataReaderin the future.

I'm not sure of a faster way to look this information up other than searching the binary for it, but maybe there's a pointer I could look for.

Searching for the version string seems viable. One "@(#)Version" string is located in the "nativeStringResourceTable_mscorrc_debug" table with an offset of about 2k bytes. The string resource table is in the library exports.

@NextTurn Assuming that table goes into Mini dumps, yep that could be the fix here.
The underlying problem is consistent behavior.  If you do dotnet-dump -type Mini, does that string table make it into the dump file?  If it does, we are in business.  If it doesn't then we'll have inconsistent behavior between live targets, "Heap" coredumps, and "Mini" coredumps.  @mikem8361 might know the answer, but if not I can run a test of it.
Inconsistent behavior isn't the end of the world here as long as we don't report the wrong version, but I'd like to understand where we are before diving into this.  (Also I'm working on getting the codebase ready for v2, so this issue is taking a back seat for a bit.  I'm happy with returning <IP_ADDRESS> for now.)

Verified the string goes into "Heap"/"Mini" dumps from dotnet-dump.

dotnet-dump/SOSHosting has C# code that searches for the @(#)Version for the "eeversion" SOS command. This code is only used for libcoreclr.so.
https://github.com/dotnet/diagnostics/blob/7fef83c075965a10e25d3dec551ea6cca4ca8c83/src/SOS/SOS.Hosting/SOSHost.cs#L667
https://github.com/dotnet/diagnostics/blob/7fef83c075965a10e25d3dec551ea6cca4ca8c83/src/SOS/SOS.Hosting/SOSHost.cs#L764

@NextTurn Fixed this one in PR #349.  Thanks!

https://github.com/dotnet/diagnostics/blob/7fef83c075965a10e25d3dec551ea6cca4ca8c83/src/SOS/SOS.Hosting/SOSHost.cs#L768
@mikem8361 FYI the use of ModuleInfo.FileSize here is incorrect: the file size on disk is often smaller than the image span size in memory. It happens to work for libcoreclr.so.
Test results of LinuxLiveDataReader.GetVersionInfo using ModuleInfo.FileSize, corresponding to those in #349:
/usr/share/dotnet/shared/Microsoft.NETCore.App/3.0.1/System.Native.so
v<IP_ADDRESS>
/usr/share/dotnet/shared/Microsoft.NETCore.App/3.0.1/libclrjit.so
v4.700.19.47502
/usr/share/dotnet/shared/Microsoft.NETCore.App/3.0.1/libcoreclr.so
v4.700.19.47502
/usr/share/dotnet/shared/Microsoft.NETCore.App/3.0.1/libhostpolicy.so
v<IP_ADDRESS>
/usr/share/dotnet/host/fxr/3.0.1/libhostfxr.so
v<IP_ADDRESS>

@NextTurn but isn't the string you search for required to be present in the file, so you should be fine if you only search the regions which get initialized by the disk image?

Should I create a new issue about the new error?

Didn't #528 fix the new problem?

Just tested it now and yes, it did, thank you. (I wasn't following that PR.)


Run mypy on every version, exclude python version 3.5

aiohttp is not compatible with python versions below 3.6
This exposes another issue of the previous configuration: for some reason mypy did not check for the currently running python version only but for all of them. Now, each time mypy runs only for the currently installed version

@scop since you installed mypy in the first place, do you know why the installed checks ran for a python version below 3.6 (see this log)
As per the documentation, this should not be default behaviour

@scop since you installed mypy in the first place, do you know why the installed checks ran for a python version below 3.6 (see this log)

Because of this: https://github.com/nielstron/pysyncthru/blob/8655e20051b8d8029a4270f3e705f943e923f8da/setup.cfg#L2

aiohttp is not compatible with python versions below 3.6
This exposes another issue of the previous configuration: for some reason mypy did not check for the currently running python version only but for all of them.

Actually, I believe it just ran for 3.5, no matter what the runtime was. And I think that's actually as it should be.

Now, each time mypy runs only for the currently installed version

This I don't think is a good idea. Running mypy with python version (in mypy config/cmdline/detected/whatever) set to e.g. 3.8 may end up suggesting changes that break with earlier python versions.
It should be run only once (does not matter with which python version), with python_version in setup.cfg set to the oldest supported python version for the package, i.e. as it was previously. Running it with multiple versions doesn't hurt, but won't bring anything new to the table either. I suggest reverting almost all of this change (except removal of 3.5 from the matrix), and just changing python_version to 3.6 in mypy's setup.cfg section.

Thanks for the pointer! The setup of mypy therefore needs to be further adjusted.

It should be run only once (does not matter with which python version), with python_version in setup.cfg set to the oldest supported python version for the package, i.e. as it was previously.

What I don't like about the current setup is that the python versions are stored in a manifold of different places. Trying to reduce that to a minimum, I moved the python version tag to the travis setup. At last, all supported python versions need to be tested there and should therefore be listed in the file. Testing in each version explicitly does not bring much new to the table indeed but is also computationally cheap (as compared to running pre_commit in every version).

it could be minimized also by removing python_version from the mypy config (to make mypy use the runtime version) and moving the mypy run in travis to happen only with the oldest supported version, 3.6 at the moment. However that would increase likelihood of runs outside travis, e.g. local ones, ending up using something else (again the runtime), and thus cause possibly conflicting and breaking advice.


Configuration option to use web-crypto instead of node crypto for Cloudflare Workers

Hi
First of all, thanks for this amazing library. I'm using it in conjunction with SvelteKit, Prisma (hooked up to a neon db) and lucia-auth along side an upstash redis instance. The whole thing should be deployed to Cloudflare Workers at the end of the day.
However, using the SvelteKit Cloudlfare Adapter Plugin uses the Cloudflare version of Node, which is much closer to browsers than node, the build fails because they don't have the crypto package. Instead we'd have to use the web-crypto API.
I could spend some time on this and code it up, but my relationship with security is the same as it is with electricity.
I know enough about it that I don't know enough about it.

This could probably be done in a config-free manner, by detecting if the WebCrypto API is available, with a fallback to the Node.js sync APIs from node:crypto.
The main issue is that at the moment everything (encryption, decryption & hashing) assumes a sync/blocking API. The core functionality would have to be converted to an async function to support async crypto, which may be a bit of work.
As a side note on the performance side - which would have to be measured - I've got a feeling going async would bring some Promise allocation overhead per query that would only be compensated on very large encryption operations.

ah, understandable. Thanks for looking into it regardless!


Unable to create firewall rule if SqlLogin authentication was used (regression)

Azure Data Studio Version:

Steps to Reproduce:

Attempt to connect to Azure DB that doesn't have the correct firewall rule set (so ADS will prompt you to add one)
Add Azure account in the firewall rule prompt
Select OK (in firewall rule prompt)

not sure why the commit says 2 years ago, when I do a git log it gives me

This should be fixed in the new insiders. Can you test it again?

@aaomidi I'm seeing this same behavior in today's insiders - unable to create a firewall rule with this error message:
Firewall rule error: Error: Specifed tenant with ID 'undefined' not found.
Version info
Version: 1.21.0-insider (user setup)
Commit: eccf3cf5fe8f60d04d70096bc94c654cab60d81a
Date: 2020-07-24T09:33:09.435Z
VS Code: 1.48.0
Electron: 9.1.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041

This error occurs in both cases if the account is added to ADS from the firewall rule wizard or from the general accounts pane.

Thanks I found out what the issue is. If we use SqlLogin the tenantId isn't populated... We need to get the tenantId somehow. Maybe the account picker needs to also have a tenant picker.
@Charles-Gagnon any ideas?

Where is the tenant id used in this case?

https://github.com/microsoft/azuredatastudio/blob/587abd43c2d92d83af1214399172fe8734597915/src/sql/workbench/services/resourceProvider/browser/firewallRuleDialogController.ts#L64

I'm having this issue on:


adds a base64url format description

related #3167

@baywet I think you need to update this to remove the deprecated version info.
I'm also wondering if, now that we have definitively settled on byte rather than base64, if it makes sense to align with RFC 4648 terminology for this, or call it byteUrl or something.  Neither feels very satisfactory to me.

Can you expand on why do you think we should remove the deprecated aspect please? I might have missed part of the conversation here.
If it were only up to me, byte would be base64 and base64url would be base64url. byte only being an alias to byte64 for compatibility reasons. Those different names are simply adding confusion IMHO.

Edit to my previous comment: formats are kebab-case so byte-url, not byteUrl
Re: deprecated_note — see https://github.com/OAI/OpenAPI-Specification/pull/3167#issuecomment-1456845756
In #3174 I suggested having a base64 alias for byte but we discussed it in the last call and decided not to do that (as seen in #3182).  So we're stuck with byte.

@handrews so are we good with this PR as is or do you suggest we make other changes?

I'm still feeling meh about whether to use base64url for compatibility with OAS 3.1 vs byte-url for compatibility with OAS 3.0, the actual targeted version here.  As much as I dislike byte, it's what we're stuck with and this is a URL-safe version of it.  Although I suspect that very few tools will implement support for this so I'm not sure how much it matters.
@MikeRalphson, @darrelmiller, anyone else - any preference for base64url vs byte-url in the context of OpenAPI 3.0?

+1 for base64url, let's not be held back by our past missteps.

@MikeRalphson thanks - in that case this looks good to go.


drivers: led: Add LED GPIO driver

Change introduces LED GPIO driver.

There is similar PR, "Add generic gpio leds driver with sample" #21578

It does not build in CI, you need to add it to https://github.com/zephyrproject-rtos/zephyr/tree/master/tests/drivers/build_all

@jfischer-no Yes I created a similar PR #21578 sometimes ago with additional software blink feature but I didn't have time to finish it. We also decided there  that software blink should be done as generic module which can be used by any driver LED driver which doesn't support hardware blinking. If we are ok with basing on LED order in the board dts as the LED ids which are used for identify LED in the API than I am ok with this PR and I will close my one. If we cannot base on LED order, we could consider basing this driver on the led-controller binding: https://github.com/zephyrproject-rtos/zephyr/blob/master/dts/bindings/led/led-controller.yaml

@jfischer-no, I added requested test to build_all

@nashif, can we merge it ?


remove empty packages

In GitLab by @luitjansl on Aug 28, 2023, 11:27
when we split a model into submodels, sometimes we can end up with empty packages. For example, a river package may not have any rivers in a particular submodel. These empty packages should be removed from GroundwaterFlowModels, MF6 will give a runtime error otherwise

In GitLab by @JoerivanEngelen on Sep 7, 2023, 17:50
I had a function here to delete empty RIV/DRN/GHB packages (with a conductance) (empty meaning all nan, in this case), for what it's worth: Is for a structured case.
def _cut_dataset(ds, xmin, xmax, ymin, ymax):
    if "x" in ds.dims:
        ds = ds.sel(x=slice(xmin, xmax)).sel(y=slice(ymax, ymin))
        # Delete boundary conditions that are all nan occur
        if isinstance(ds, xr.Dataset) and "conductance" in ds.keys():
            if np.all(np.isnan(ds["conductance"])):
                return None
    return ds

Source: https://gitlab.com/deltares/imod/respighi/-/blob/modflow6/respighi/chunked.py?ref_type=heads#L28


如何加载多个terrainProvider

如何在cesium中加载多个terrainProvider?
可不可以不断添加   viewer.terrainProvider, 类似于viewer.imageryLayers.addImageryProvider(provider);

The question as I understand it asks if it's possible to add more than one terrain provider in Cesium. Currently this is not possible to do at runtime. You can upload your terrain files to Cesium ion (http://cesium.com/ion/) to combine them.  There is a PR that adds multiple globes to the scene:
https://github.com/AnalyticalGraphicsInc/cesium/pull/6450
that might help with that depending on your use case. I think the long term goal is to support time dynamic 3D Tiles, and then terrain becomes part of 3D Tiles, so that way you can dynamically change individual areas of terrain.


different behaviour in modelio when use aggregation vs composition

Really I don't know if this is and bug or it's my fault.
I'm using modelio 3.5.
Following the examples of documentation, the docs say: our parser system doesn’t care if you use an aggregation or a composition. You can use whatever you want. But I've found some differences.
If I use composition relationship between Employee and Job with OneToMany (unidirectional) the parser does not show the warning message that shows when I use the same relationship with aggregation, and write a @OneToOne relationship in the code, but using aggregation the code is @OneToMany.
Another case is using one-to-one aggregation relationship, the generated code is @ManyToOne.
So it appear that the parser always generates one-to-one when composition is used, and many-to-... or ...-to-many when aggregation is used.
This occurs with modelio, with umldesigner the result is not the same, but different.

I guess I should update the docs then... For a one-to-one, composition is to be used. Otherwise, an aggregation may always be needed... Thanks for taking the time to report this
Le 11 novembre 2016 14:25:39 GMT+01:00, David Palomar<EMAIL_ADDRESS>a écrit :

Really I don't know if this is and bug or it's my fault.
I'm using modelio 3.5.
Following the examples of documentation, the docs say: our parser
system doesn’t care if you use an aggregation or a composition. You can
use whatever you want. But I've found some differences.
If I use composition relationship between Employee and Job with
OneToMany (unidirectional) the parser does not show the warning message
that shows when I use the same relationship with aggregation, and write
a @OneToOne relationship in the code, but using aggregation the code
is @OneToMany.
Another case is using one-to-one aggregation relationship, the
generated code is @ManyToOne.
So it appear that the parser always generates one-to-one when
composition is used, and many-to-... or ...-to-many when aggregation is
used.
This occurs with modelio, with umldesigner the result is not the same,
but different.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jhipster/jhipster-uml/issues/234

--
Mathieu ABOU-AICHI

By the way, it is said that the parser doesn't care if the parser is the UMLDesigner one... not for Modelio. But I'm updating it anyway for best practice.

TL;DR
Yep it's true, it's under umldesigner section, but the complete paragraph mentions Modelio also (As mentioned with Modelio, our parser system doesn’t care ...) and in the Modelio section there is a similar phrase "You can use either of them, the parser will just see the two as associations anyway."
The following paragraph under modelio section explains in detail the purpose of both relationships, but I think that it's bypassed (my fault):
"Notice the left panel entitled ‘Class Model’. You only need the ‘Class’, ‘Attributes’, ‘Aggregation’, ‘Composition’ and ‘Data Type’ objects. You can already guess why you need the first 2. The ‘Aggregation’ object is used to defined aggregations: B is aggregated to A means that A has 0, 1 or more (n) instances of B. A doesn’t create (and destroy) B instances. Composition means that if A is composed of B (0, 1, or n instances), then it creates, manages and destroys B instances."
Perhaps it should be summarized like: "if you wants a one-to-one relationship use composition and aggregation in other case".  :-P
I'm sorry about that, but I've spent several days to find where the problem was.

Don't be. Thanks again for notifying me
Le 12 novembre 2016 00:12:28 GMT+01:00, David Palomar<EMAIL_ADDRESS>a écrit :

TL;DR
Yep it's true, it's under umldesigner section, but the complete
paragraph mentions Modelio also (As mentioned with Modelio, our parser
system doesn’t care ...) and in the Modelio section there is a similar
phrase "You can use either of them, the parser will just see the two
as associations anyway."
The following paragraph under modelio section explains in detail the
purpose of both relationships, but I think that it's bypassed (my
fault):
"Notice the left panel entitled ‘Class Model’. You only need the
‘Class’, ‘Attributes’, ‘Aggregation’, ‘Composition’ and ‘Data Type’
objects. You can already guess why you need the first 2. The
‘Aggregation’ object is used to defined aggregations: B is aggregated
to A means that A has 0, 1 or more (n) instances of B. A doesn’t create
(and destroy) B instances. Composition means that if A is composed of B
(0, 1, or n instances), then it creates, manages and destroys B
instances."
Perhaps it should be summarized like: "if you wants a one-to-one
relationship use composition and aggregation in other case".  :-P
I'm sorry about that, but I've spent several days to find where the
problem was.
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/jhipster/jhipster-uml/issues/234#issuecomment-260077148

--
Mathieu ABOU-AICHI


Ban the last monster

Set an example cookie like in the cluster setup docs.

Awesome title! Truly there are no monsters under my couch :joy:


[build-script] Add dependencies for all build-script products as a first approximation.

In most cases, I followed the ordering of dependencies defined already by
build-script's product classes. In a subsequent commit I am going to add an
option (disabled by default) that schedules this via a simple topological sort
based on proving our dep graph is a DAG and using RPOT numbers.

@swift-ci test