problem report is sent when no exchange exists yet

When sending a credential offer in v1 and creating a bad request (in my case I made a type in one of the attribute names) ACA-Py will send a problem report to the other agent without having initiated the protocol.
Not sure if other endpoints / protocols are affected, but I find it an quite amusing issue :)

That's funny...  :-).

While funny, we should prevent this -- having a protocol initiator reporting a problem to another agent before the protocol has started. It would be good to have someone look into this.
First, we need to look at this in this specific case -- issue credential protocol (v1 and presumably v2). How do we prevent a problem report from the initiator going out before the first message of the protocol has been sent?
Second, what other protocols are susceptible to this issue? Is there a cross-cutting mechanism that can be applied to handle this, or does each protocol have to be updated?

Reply to @swcurran Re: Second, what other protocols are susceptible to this issue?
I see the issue in Credential Issue Exchange v1.0 (this issue) and v2.0 (Issue #2512). I don't see it in the Proof Exchange - there, instead of the report_problem call I see comments such as # other party does not care about our false protocol start.
@TimoGlastra This issue should be fixed by PR #2519. Can you please verify and close this issue if that is the case.


where could I get the source code of vanna-flask ?

where could I get the source code of vanna-flask ?         I want to revise the web page of the vanna-flask

https://github.com/vanna-ai/vanna-flask


Error in displaying sources - failed to obtain information

I was able to get it working using Ollama. I am getting the results, but the sources are not displayed.

Here's my .env
# Bing search key
BING_SEARCH_KEY=
# Google search key
GOOGLE_SEARCH_KEY=
GOOGLE_SEARCH_ID=
# aliyun key
ALIYUN_KEY=
# Yi Key
YI_KEY=
# google gemini
GOOGLE_KEY=
GOOGLE_PROXY_URL=
# baidu
BAIDU_KEY=
BAIDU_SECRET=
# tencent KEY:ID, SECRET:KEY
TENCENT_KEY=
TENCENT_SECRET=
# openai key
OPENAI_KEY=
# openai proxy
OPENAI_PROXY_URL=https://api.openai.com/v1
# moonshot
MOONSHOT_KEY=
# lepthon key
LEPTON_KEY=
# Local llm: Ollama host, default http://localhost:11434
OLLAMA_HOST=http://<IP_ADDRESS>:11434
# Free search engine: https://www.searx.space/
SEARXNG_HOSTNAME=http://localhost:8080

Hi, Are you using Docker for deployment?

please make sure SearXNG's API is working.
make sure this project can access SearXNG's API.
you may need to modify SEARXNG_HOSTNAME in .env if you are using docker to deploy this project. (please ensure that this Docker can access the host machine's network.

I am using docker for deployment and followed the process listed in the docs. What should the SEARXNG_HOSTNAME be? How do I test it

or...do I have to have to deploy searxng separately?

@arsaboo yes...  searxng-docker

@yokingma So, I am running searxng using the following:
version: "3.7"

services:
  redis:
    container_name: redis
    image: docker.io/library/redis:alpine
    command: redis-server --save 30 1 --loglevel warning
    networks:
      - searxng
    volumes:
      - redis-data:/data
    cap_drop:
      - ALL
    cap_add:
      - SETGID
      - SETUID
      - DAC_OVERRIDE

  searxng:
    container_name: searxng
    image: docker.io/searxng/searxng:latest
    networks:
      - searxng
    ports:
      - "8080:8080"
    volumes:
      - ./searxng:/etc/searxng:rw
    environment:
      - SEARXNG_BASE_URL=https://${SEARXNG_HOSTNAME:-localhost}/
    cap_drop:
      - ALL
    cap_add:
      - CHOWN
      - SETGID
      - SETUID
    logging:
      driver: "json-file"
      options:
        max-size: "1m"
        max-file: "1"

networks:
  searxng:
    ipam:
      driver: default

volumes:
  redis-data:

and I can access searxng using the IP address http://<IP_ADDRESS>:8080/
But I am still getting the same error. How do I change the SEARXNG_HOSTNAME?
It may be useful for you to create a complete docker compose file...to avoid such issues.

Same error even with:
version: "3.7"

services:
  redis:
    container_name: redis
    image: docker.io/library/redis:alpine
    command: redis-server --save 30 1 --loglevel warning
    volumes:
      - redis-data:/data
    cap_drop:
      - ALL
    cap_add:
      - SETGID
      - SETUID
      - DAC_OVERRIDE

  searxng:
    container_name: searxng
    image: docker.io/searxng/searxng:latest
    ports:
      - "8080:8080"
    volumes:
      - ./searxng:/etc/searxng:rw
    environment:
      - SEARXNG_BASE_URL=https://${SEARXNG_HOSTNAME:-localhost}/
    cap_drop:
      - ALL
    cap_add:
      - CHOWN
      - SETGID
      - SETUID
    logging:
      driver: "json-file"
      options:
        max-size: "1m"
        max-file: "1"

volumes:
  redis-data:

See here for an example of a combined docker-compose file.

@arsaboo Thanks for your suggestion, I have added docker-compose file.
docker compose up -d

Sources are working now... but for some reason Ollama isn't working with the same URL (I do see Ollama running on that URL). I'll check and let you know

@arsaboo Maybe the docker can't access the host's network?

# wondows or mac
OLLAMA_HOST=http://host.docker.internal:11434

Ok...so it worked if I run the docker on the same computer where Ollama is running. I was unable to get Ollama working from another host. But I will close this issue as the original issue is fixed. I will create a few more issues[feature requests] for you to consider.


Any chance of keeping the enforce-source

Hi, been used this plugins for a few days, works like a charm. Thanks for your work !
Now I understand that this project is set to be closed since poetry 1.5.0 is out, but I really like the enforce-source option, so even though the first problem this plugin tries to solved is solved with vanilla poetry, this plugin is still useful for me.
Any chance of this feature being in a new dedicated plugin maybe ?

Hi @ClementPinard , as we currently don't feel the need to have such option it's unlikely that we'll keep maintaining this plugin / developing a new plugin for such need. I suggest you to fork this repo and work out a poetry 1.5 compatible plugin ;)


Added error logging into the file queue.

@mpimenov Please check it out.

clang-format?

@mpimenov PTAL
clang-format will be used later, after it's config tunung, and in a separate request.

@mpimenov PTAL please.

LGTM


🌐 Update Chinese translation for docs/tutorial/body-multiple-params.md

also fixed code highlight

It seems this translation was updated at some point. Could you please check in a new PR if the changes are still needed? For now, I'll close this one, thanks! 🍰


Tree:clearSelection()

I could not find in docs neither in issues. How can I clear the selected nodes of the Tree?

Any Idea?

Just call selectItem on the item again.
So, if you select the item like this:
$('#myTree').tree('selectItem', $('#item1'));
Calling that exact same method again will function as a deselect. Admittedly not the greatest name for the method...

Thanks for the reply but for my need the deselectAll does the trick.


lxd/storage/ceph: Improve version parsing

Shorten the version string a bit.
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Testsuite passed


Added test cases

Pull Request Test Coverage Report for Build 51

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.282%

Totals

Change from base Build 49:
0.0%

Covered Lines:
3109

Relevant Lines:
3562

💛  - Coveralls


Save feature names in the C++ booster object

Currently, the feature_names field of the Python Booster object is lost after performing round-trip serialization (save and then load):
import xgboost as xgb
import numpy as np

X_train = np.random.rand(5,10)
y_train = np.random.rand(5,1)
dtrain = xgb.DMatrix(X_train, label=y_train)

params = {'objective': 'reg:squarederror'}
bst = xgb.train(params, dtrain, 2, evals=[(dtrain, 'train')])

bst.save_model('test.model')
print(bst.feature_names)  # prints ['f0', 'f1', 'f2', 'f3', 'f4', 'f5', 'f6', 'f7', 'f8', 'f9']

bst_loaded = xgb.Booster(model_file='test.model')
print(bst_loaded.feature_names)  # prints None

This leads to some confusing behavior, e.g. broken feature validation (#4854).
TODOs:

[ ] Save the feature names in the C++ booster object, so that the feature names are serialized along with the rest of the model. This way, save_model() will not lose the feature names.
[ ] Remove the generation of placeholder feature names like f0, f1 etc. We should only save the legitimate feature names from the data frame libraries such as Pandas and cuDF.

cc @trivialfis

For the sake of backward compatibility, I suggest saving the feature names as a Booster attribute, like what we currently do with other scikit-learn attributes:
https://github.com/dmlc/xgboost/blob/d6386e45e8ed19c238ee06544dabbcbf56e02bbc/python-package/xgboost/sklearn.py#L446
So we'd have
json_str = json.dumps(feature_names)
bst.set_attr(feature_names=json_str)

I can think of two alternatives.

Use Booster attributes

For the sake of backward compatibility, I suggest saving the feature names as a Booster attribute, like what we currently do with other scikit-learn attributes:
https://github.com/dmlc/xgboost/blob/d6386e45e8ed19c238ee06544dabbcbf56e02bbc/python-package/xgboost/sklearn.py#L446
So we'd have
json_str = json.dumps(feature_names)
bst.set_attr(feature_names=json_str)

Save feature names as part of the C++ Learner object.

This is cleaner than 1), but we may not be able to save feature names in the legacy binary serializer.

I vote for second option:

Save feature names as part of the C++ Learner object.

@trivialfis Are you okay with losing feature names when the binary serializer is used?

@hcho3 I'm fine with that.  Debugging the binary format is just too painful.

Here are some expectations:

Users should be able to set the feature names manually, including adding/removing/modifying feature names.
Users should be able to enable/disable validation.  When disabled, there should not be any overhead in prediction.
The feature name generation should be kept as an independent internal function used by features like plotting or score calculation.
XGBoost should not force the use of feature name in prediction.  For example if users train a model with dataframe, then supplying a ndarray for prediction should be fine.  We can still check the number of features between model and dataset.


Missing emoji

Hello,
Apple added a bunch more emoji in October (http://blog.emojipedia.org/ios-9-1-emoji-changelog/). I tried to test by outputting all of them, extracting the message from my phone with TouchCopy and then running the result through your parser. Pretty good overall, but I noticed a few that were missing.
Also, I noticed a few that don't get picked up but which seem to have symbols already in a few editors, like ☺☮✡. Can we get these picked up by your parser too? In my project, these files are being saved to ANSI Latin 1 and I would like replace all symbols with some textual representation.
EmojiTest.java.txt
Rob
:)

+1

Hi!
Thanks for raising the issue and sorry for not answering faster.
I started a new branch (https://github.com/vdurmont/emoji-java/tree/new-emojis-ios9.1) to add the new emojis.
I already added the "faces" and "hands" (see commit https://github.com/vdurmont/emoji-java/commit/202abaf19b72a0a7afab6bc66e1a7b21236d7a6d) and we still need to add the other ones to the database. It's a long work :)
Any help is welcome!
Regarding the david cross, the slightly smiling face and the peace symbol, they will be added on this branch so I think they're going to get picked up after this. Let me know if I misunderstood.

You understood perfectly! I would help, but I am umm.. washing my hair that night, yeah, sorry. Anyway, looking forward to the next iteration. :)

Hi Guys. Any updates on the new Smilies?
Can we simply replace the emojis.json file with a new file?

Hey sorry the holidays took all my time and I couldn't work on this.
I started adding the new emojis to the json file.... Hope to be able to finish this weekend.

I finally had the time to update the emoji database!
I started the release of v3.3.0 with all the new iOS 9.1 emojis, it should be available in maven central in a few hours. :rocket:

Thank you!


Missing fuel-android version 1.15.1

Cant find at https://jcenter.bintray.com/com/github/kittinunf/fuel/fuel-android/

Also livedata too.

It looks like there is something wrong with publishing script ... I will check what's wrong ...
In the meantime, @monkeymc can you use previous version?

I fixed it, you should be able to find them there
https://jcenter.bintray.com/com/github/kittinunf/fuel/fuel-android/1.15.1/
and
https://jcenter.bintray.com/com/github/kittinunf/fuel/fuel-livedata/1.15.1/


Refactor Kubernetes upgrade code

Is this a request for help?:
No
Is this an ISSUE or FEATURE REQUEST? (choose one):
FEATURE REQUEST
What version of acs-engine?:
master/HEAD

After the change to user OrchestratorVersionHint as user input, upgrade code still treat OrchestratorVersion as user's input. Need to refactor that
Also upgrade is focus on 1.5.3 -> 1.6.2, we should work on the scenario to upgrade  1.5 -> 1.6 regardless of the patch part.

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What you expected to happen:
More aligned with OrchestratorVersionHint change. #921

To help with the code here are the lines that need to be changed (there might be 1-2 more places). Upgrade uses the version specified in its API model to set the container service API model to generate the template.
Following lines need to be changed and tested:

https://github.com/Azure/acs-engine/blob/master/pkg/operations/kubernetesupgrade/v162upgrader.go#L42
https://github.com/Azure/acs-engine/blob/master/pkg/operations/kubernetesupgrade/v162upgrader.go#L27-L30
https://github.com/Azure/acs-engine/blob/master/pkg/operations/kubernetesupgrade/upgradecluster.go#L73-79


should Map and/or Seq and/or Set still extend Function1?

we discussed this on Gitter about a month ago, in response to @nafg's question: https://gitter.im/scala/collection-strawman?at=59f378ab51915d962afb2f5a
and @szeiger asked about it there just now: https://gitter.im/scala/collection-strawman?at=5a184823540c78242d43ebe2
but we should have a ticket of record. here it is

I put this on Gitter, but I'll record it here as well:

it's so easy to get a Function1 if you want one, with .apply _
I've seen code that was shorter because of this, but I think it comes at the expense of clarity
and the expense of difficult to interpret error messages
and the expense of incorrect code that accidentally compiles

and a month later:

I still would prefer that none of these things were Function1s, for my originally stated reasons. but I agree that Set is worst, Seq is not as bad, and Map is least bad
so it's a matter of degrees, but I still think they're all bad. I often see beginners (on IRC, Gitter, Stack Overflow...) running afoul of the Seq one in particular
it's sooooooo easy to get a Function1 if you want one, with .apply or .apply _
I reallllllly dislike that code like List(1,2,3).map("foo bar baz") actually compiles. beginner code that ran into this often is some form of this
I'm somewhat at a loss to make more sophisticated arguments besides just putting more l's in realllllly
the pro is that you can leave out .apply sometimes. it seems like a very small pro to me
the con is massive loss of typesafety because random, essentially nonsensical code still compiles
it seems like quite a large con to me
even if you aren't a beginner anymore, this List(1,2,3).map("foo bar baz") style seems like bad code to me. it's so much more clear with the explicit .apply
"Arrays aren't function values. They just aren't". Strings, same
plus having it apply to all of Seq, and not just IndexedSeq, means collections with O(n) apply like List get sucked in too
I see only two arguments in favor of Function1-ness that are at least somewhat convincing:

it's "weird" to have .apply but not be a Function1
tradition/backwards compat
as for 2, this is our big chance to change it and not be stuck with it for another 5-10-20 years. we have been putting off these chances since 2.8, holding them for exactly this time. this time is now
as for 1, what's so weird about it? seems fine to me. if you want a Function1, you can get one with .apply _
it doesn't seem nearly as weird to me, aesthetically, as allowing List(1,2,3).map("foo bar baz") to compile
and anyway, we shouldn't attach too much importance to subjective feelings about what feels weird to us. >I fail to see that the "weirdness" of has-apply-but-isn't-Function1 has bad following consequences for either beginners or experts
but I see actual beginner confusion, actual loss of meaningful typesafety, and actual obscurity resulting from the weirdness of extends Function1. it is a weirdness with real consequences

I see only two arguments in favor of Function1-ness that are at least somewhat convincing

hmm, but, there's also the matter of PartialFunction, which I didn't cover... the .apply _ trick gets you a Function1 but not a PartialFunction. started thinking about it after I wrote the above, and I see @joroKr21 brought it up on Gitter just now. he writes:

It seems useful to have get and getOrElse for Seq similar to Map since the index might be out of bounds. Currently in Scala you can do this with lift and applyOrElse from PartialFunction

not sure if I have a good response to that, I'll have to think on it a little more

Yep, PartialFunction is better justified. To me a lot of the weirdness actually comes from the fact that PartialFunction is a Function1 when Liskov points the other way around. I guess it's mostly there so we can destructure tuples with map { case (x, y) => ... } but with auto tupling in Dotty maybe it's time to reverse the relation. Also another thought - would it make sense for exhaustive patterns to produce a Function with the case syntax?
So I'm all for having Function1 <: PartialFunction and Seq, Map <: PartialFunction but not Seq, Map <: Function1. As it stands I lean towards not extending Function1 but instead adding get and getOrElse to Seq (or just to IndexedSeq) to be consistent with Map.

I think reversing Function/PartialFunction is out of scope for this ticket because it's almost certainly out of scope for Scala 2.13. (If anyone disagrees or wants to discuss it further, let's take it to https://contributors.scala-lang.org.)

As it stands I lean towards not extending Function1 but instead adding get and getOrElse to Seq (or just to IndexedSeq) to be consistent with Map

interesting, I'll think on that too.

But the collections design for 2.13 is meant to last through Dotty no? So
at what point is there an opportunity to reverse Function1 and
PartialFunction?
On Fri, Nov 24, 2017, 11:14 PM Seth Tisue<EMAIL_ADDRESS>wrote:

I think reversing Function/PartialFunction is out of scope for this
ticket because it's almost certainly out of scope for Scala 2.13. (If
anyone disagrees or wants to discuss it further, let's take it to
https://contributors.scala-lang.org.)
As it stands I lean towards not extending Function1 but instead adding get
and getOrElse to Seq (or just to IndexedSeq) to be consistent with Map
interesting, I'll think on that too.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scala/community-builds/issues/638#issuecomment-346917525,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGAUBe0u6k0MAfB_0P68DFFyLyaKNuhks5s55QQgaJpZM4QqO6Z
.

at what point is there an opportunity to reverse Function1 and
PartialFunction

let's keep this ticket on one topic, please — that's another discussion. (past discussions about it have gotten quite involved.)
one reason not to entangle the collections types with the function types is that it reduces our freedom to maneuver on both sides.

Good point. I guess that's an argument not to extend Function1 or
PartialFunction.
On Sat, Nov 25, 2017 at 8:49 PM Seth Tisue<EMAIL_ADDRESS>wrote:

at what point is there an opportunity to reverse Function1 and
PartialFunction
let's keep this ticket on one topic, please — that's another discussion.
(past discussions about it have gotten quite involved.)
one reason not to entangle the collections types with the function types
is that it reduces our freedom to maneuver on both sides.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scala/community-builds/issues/638#issuecomment-346978251,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGAUBaL8TSgdRv0XxsWLV_pZG-9o6MRks5s6MOjgaJpZM4QqO6Z
.

I don't think this will work. Seq's are perfectly good functions and so are maps. If you remove that you will break a lot of code for no apparent gain. And like I said on the gitter this would be a very serious change that will break lots of code in very obscure ways. We can consider to do this only if we are have proven rewrite tools that will work close to 100% of the time. So far we do not have them and I do not see them materializing real soon now. Even if we had the rewrite tools the question remains whether we want to impose such a change on our users.
This holds for Set as well as Seq and Map btw. I can see the technical argument for making sets not functions, but the migration problematic is just the same for them as for the others.
I believe the window to do changes to collections is closing rapidly. We should concentrate now on getting new collections up on par wrt performance and functionality with the old ones. Throwing more fundamental changes in the mix risks that the whole project will be not ready by the 2.13 deadline.

We can consider to do this only if we are have proven rewrite tools that will work close to 100% of the time.

Do Scalafix not deliver this?

new ticket of record, with a link to here: https://github.com/scala/collection-strawman/issues/306


glib: 2.66.3 -> 2.66.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for glib is: "C library of programming buildings blocks"
meta.homepage for glib is: "https://www.gtk.org/"
meta.changelog for glib is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.66.4 with grep in /nix/store/r0das144wzrpizv85fj03m09c8w8mmi1-glib-2.66.4-bin

directory tree listing: https://gist.github.com/3eca8ca0d1319599169f5c01c71d234e

du listing: https://gist.github.com/c14c7b0b8b089a075021426dcddc7789

Rebuild report (if merged into master) (click to expand)

35513 total rebuild path(s)

11887 package rebuild(s)

11752 x86_64-linux rebuild(s)
10101 i686-linux rebuild(s)
3741 x86_64-darwin rebuild(s)
9919 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
AusweisApp2
DisnixWebService
EmptyEpsilon
Literate
MIDIVisualizer
MMA
OSCAR
OVMF
OVMF-CSM
OVMF-secureBoot
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net
SDL2_ttf
SDL_Pango
SDL_gfx
SDL_gpu
SDL_image
SDL_mixer
SDL_net
SDL_sound
SDL_stretch
SDL_ttf
Sylk
Xaw3d
_1oom
_1password-gui
_20kly
_3mux
_90secondportraits
a2jmidid
abbaye-des-morts
abcde
abcl
abcm2ps
abiword
abootimg
accountsservice
acgtk
acoustidFingerprinter
acsccid
act
adapta-backgrounds
adapta-gtk-theme
adementary-theme
adlplug

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r0das144wzrpizv85fj03m09c8w8mmi1-glib-2.66.4-bin \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A glib https://github.com/r-ryantm/nixpkgs/archive/0b0cc7315123ebc12030c278e2d5d53e4302847e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r0das144wzrpizv85fj03m09c8w8mmi1-glib-2.66.4-bin
ls -la /nix/store/r0das144wzrpizv85fj03m09c8w8mmi1-glib-2.66.4-bin/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @lovek323 @7c6f434c @worldofpeace for testing.

Already on staging.

Already on staging.


Problem with L2_Loss on GPU

The following code works correctly when running on CPU but produces an error when run on GPU. This has been experienced with both a workstation with an Nvidia GTX 980 and a Nvidia Titan X. The error message indicates theres a problem on the CUDA Backend.

import tensorflow as tf
sess = tf.InteractiveSession()
input = tf.placeholder("float32", shape=[None, 2])
target = tf.placeholder("float32", shape=[None, 2])
w=tf.Variable(tf.zeros([2, 2]))
b=tf.Variable(tf.zeros([1]))
l=tf.nn.relu(tf.matmul(input, w) + b)
cost = tf.reduce_sum(tf.nn.l2_loss(l - target)) # this breaks
#cost = tf.reduce_sum( tf.pow(l - target, 2)/2 )
train_step = tf.train.GradientDescentOptimizer(0.01).minimize(cost)
sess.run(tf.initialize_all_variables())
dict={input: [[1,1],[1,1]], target: [[1,1],[1,1]]}
train_step.run(feed_dict=dict) # <---- this throws "F tensorflow/core/kernels/tile_ops.cc:116] TileOp: Invalid combination of Device, DT and NDIM: N5Eigen9GpuDeviceE, float, 0"

@kvamaraju, could you sync to the latest TensorFlow and retry? I think this problem should have been fixed by a recent CL.


Object files/libraries produced by Gradle contain absolute paths

The object files created via gcc contain an absolute path to the source file.  I suspect this is due to the way we call the compiler.
This can kill cacheability when a shared library is packaged into a jar when everything else is "the same".
Your Environment
Linux, gcc.

@lacasseio FYI, this is something we'll need to keep in mind for native caching.

You are completely right. This is indeed related to how we call the compiler and manage the file path to the source files. We use absolute path everywhere. This issue is true for all supported compilers (MSVC included). The only way I can think of right now to fix this is to relativize the path before calling the compiler.

The other option is to use identical file paths (and sync files to/from a working directory), but I think just doing relative paths is a good first step.

For the paths that are there due to debug symbols, there are other options:

Generate a separate symbol file and don't cache this. Or cache it and reuse it only when the source paths happen to be the same.
Ignore these when I don't need to be able to debug the binaries.
Copy the source files to the location expected by the binaries when I want to be able to debug them.

It also happens without debugging symbols because things like __FILE__ expand to the full file path (due to the way we pass the path to the compiler).  That was the case with the build I found this in.

One question would be what this actually breaks. We don't care that the object files aren't byte-wise equivalent to what they would be if compiled locally, we care only that they behave the same way for whatever purpose they're being used for.
That said, we have other options to deal with this too, such as transforming between preprocessor and the compiler, or detecting usages of __FILE__ during our dependency analysis and not caching these object files, or caching only when the source paths are the same. Or we might allow you to normalize this input away if you don't care.

FTR, tracking is what CCACHE does with macros like __FILE__ and __TIME__ etc.: https://ccache.samba.org/manual.html#_performance


Getting an error with 'filter' on line 63 of code

Says 'TypeError': Cannot read property 'filter' of undefined (line 63, file "supersql"). Any advice on how I can fix this? Came up during debugging because I still just get #NAME error when trying to use. Thanks for any help!

Make sure you are under new V8 runtime.


Orthogroup tree table  perf

This PR addresses some performance issues with the protein TreeTable on group pages.

I still need to address the issue where the page jumps when a filter is applied.

From ui-infra
Verbose help text should be replaced by one sentence and "read more..." toggle.
Replace "Note" with a warning icon :warning:

Run clustal seems greyed out

Don't forget the vertical placement of the tree w.r.t. table (it has gone out of sync a bit)

I had another play with useDeferredValue.
This approach seems to reduce the checkbox delay considerably (but not entirely)
  const [volatileCorePeripheralFilterValue, setCorePeripheralFilterValue] = useState<
    ('core' | 'peripheral')[]
  >([]);
  const corePeripheralFilterValue = useDeferredValue(volatileCorePeripheralFilterValue);

and then only use the "volatile" value in the SelectList.
The page/table moving around underneath the SelectList widget isn't ideal, as you already mentioned.

Maybe even this?
import { useState, useDeferredValue } from 'react';

export function useDeferredState<T>(initialValue: T): [T, T, React.Dispatch<React.SetStateAction<T>>] {
  const [volatileValue, setVolatileValue] = useState<T>(initialValue);
  const deferredValue = useDeferredValue(volatileValue);

  return [volatileValue, deferredValue, setVolatileValue];
}

Verbiage suggestion:
Currently

To see a phylogenetic tree please use a filter to display fewer than 1,000 sequences

Could be

To see a phylogenetic tree please use the filters to restrict to fewer than 1,000 sequences

@bobular thanks for the feedback and suggestions. I am going to merge this as it currently is. I think we can explore useDeferredValue and updating verbiage in a new PR.


Use LimitNOFILE for etcd service

Dumping etcd logs showed "too many open files" errors from etcd, which seems related to #1240 and https://github.com/coreos/coreos-overlay/commit/9d4e63bec512eb9ca97844e47afe93be8b2aea1a. This adapts the Azure user-data change from #3173 for the coreos/user-data file, which overrides the etcd service to increase file handles.

Code LGTM.

Considering that this is a stopgap until it makes its way into the stable channel, this LGTM.

This passed test-ec2 although it failed test-vagrant (but not with "too many open files"). I think it did fix that specific failure, and this change is coming soon from CoreOS upstream anyway, so I'm merging it after discussion with other maintainers.


Object doesn't support property or method '_releaseCamera' in IE 11 with Flash

I'm trying the latest version of the component in IE 11 (where it's used Flash) and I get this error: Object doesn't support property or method '_releaseCamera' in reset function:
if (this.userMedia !== true) {
// call for turn off camera in flash
this.getMovie()._releaseCamera();
}

Is it possible you may have updated the webcam.js script to the latest version, but not the webcam.swf file?  Or maybe the old SWF version is cached somewhere?  I ask because the _releaseCamera() method is very new, and was only just added to the Flash SWF, so you'll need the latest webcam.swf for it to work properly.

I updated both webcam.js and webcam.swf, I cleared the cache in IE 11 but I still get the error.
I binary compared webcam.sfw I'm using with the latest version available on GitHub and they are equal.
Is it possible that the version available on GitHub is not updated?

Weird.  I'm sure the version on GitHub is updated, as calling reset() works fine in other browsers when Flash is active.  Must be a weird IE 11 thing.  Unfortunately I don't have a PC with IE 11 right now, so I can't reproduce this.  I guess I can check if the object contains _releaseCamera() before calling it, but it's a blind fix.

Is there a way to check if my webcam.swf contains _releaseCamera function?
I think you can use free virtual machines from Microsoft that you can download from here https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/
I'm having the problema on IE 11 on Windows 10.

Any news about the error?
Were you able to reproduce the problem?

I'm very sorry, but I have not had time to reproduce this bug.  I do not have a working PC, and my VMWare refuses to recognize my webcam.  I haven't had time to debug this, but I applied a blind patch to check for the existence of _releaseCamera before calling it.  Hopefully this will work around the problem for now.  I just released v1.0.16 with this change:
https://github.com/jhuckaby/webcamjs/commit/70c20e8d6467e4b54d382b1bb666b7e6152e8f3c
Thanks for reporting this, and I am sorry for the delay.  I will leave this bug open, as I have no idea if this will even fix it.

Thank you for the fix. It worked.


Row/column header is grid dark mode does not have distict shading

We have distinct color in light mode, but not in dark

Properties window has slightly lighter shade in dark:

Which is fine


How to make Youtube video with autoplay and with sound on page load?

Dear All,
Today i got one requirement from client that we have an Youtube video in our page with help of iFrame we are rendering. But i'm unable to autoplay when page loads with sound.
Any one can help on this?
i. Able to do auto play with out sound i.e. muted.
ii. Able to enable the sound icon but not auto play.

This sounds like a general support question. Please, use one of the appropriate support channels for these types of questions.
GitHub issues are reserved for bug reports and feature requests.
Thx!


Add keybind event to socket2.

Description
Add a way for keybinds to trigger events in socket2.
Currently, hyprland-py achieves this by implementing our own socket.
Configuration:
bind=SUPER,E,custom_event,my_event

can't this be done with the plugin system?

can be but this is a fair core feat

I second this. I imagine a new dispatcher named event, which enables me to do this for example:
bind=SUPER,M,event,focusmain
bind=SUPER,1,event,focusother1
bind=SUPER,2,event,focusother2

so that pressing SUPER+M would for example trigger an event like this:
dispatchevent>>focusmain
And I could implement my custom focus logic in my own process. The alternative I guess is using exec and some custom IPC, but this would be much simpler and avoid the overhead of spawning a processes for each key press. Maybe you also should be able to set the event name? Maybe enable you to leave out the data to send, and then instead send some representation of the bind that triggered the event and you can parse it in your own process?

I can do it


Can't find the Export to Plain Text

After installed Pandoc with Ver 0.4.1 running in Windwos 11, Obsidian V1.5.12 . From the command Palette, I can see the Pandoc Plugin:Export as ePub, MediaWiki, etc but I can't find the option to export as Plain TXT and PDF .
Who to enable this ?

solution：https://github.com/OliverBalfour/obsidian-pandoc/issues/179


support for native classList

I've modified _hasClass(), _addClass() and _removeClass to use the native method classList in browsers that support it.
I've made a small testcase at http://jsperf.com/element-classlist-vs-element-classname/ that shows that this method works faster than using indexOfon the className.

Hi,
I had to rewrite the test: http://jsperf.com/element-classlist-vs-element-classname/4
On my setup (Firefox 9 beta), the indexOf method is the fastest. Could you run the test with your browser ?

I've run it again on latest Opera and will add more tests on Firefox.
Maybe a new feature request should be open for Firefox so they would improve ´classList´ performance.

Those hasOwnProperty tests could be done outside the function body as with _addEvent and _removeEvent, removing the need to feature test for classList support every time theses methods are called.

Yes, you're right.
Anyway I'm not merging this for now as it adds more code to be, yes, faster on Opera/Safari, but slower on Chrome/Firefox and the same in IE and some mobile versions.


check 'vhost' validity in 'http raw dvr api'

check 'vhost' validity in 'http raw dvr api', for https://github.com/ossrs/srs/issues/2015

Codecov Report

Merging #2435 (143c186) into develop (a594678) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #2435      +/-   ##
===========================================
- Coverage    42.77%   42.77%   -0.01%
===========================================
  Files          101      101
  Lines        35944    35946       +2
===========================================
  Hits         15376    15376
- Misses       20568    20570       +2

Impacted Files
Coverage Δ

trunk/src/app/srs_app_config.cpp
66.98% <ø> (ø)

trunk/src/app/srs_app_http_api.cpp
0.00% <0.00%> (ø)

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 a594678...143c186. Read the comment docs.


The admin should be able to de-activate an existing user in their organization so that when people leave they can be removed from seeing senstive financial data

Acceptance Criteria:

[x] Should have a button in the Admin panel in front of all users that de-activates them
[ ] Should not be able to de-activate themselves
[x] Should show a confirmation screen before de-activating the user to avoid accidental clicks
[ ] Should allow de-activated users to be re-activated
[x] Should not remove the user from the system and should merely soft-delete / mark them as inactive

Could you please assign this issue to me? Thanks!

@kitallis @dvenkatr I had added the soft-delete functionality. There's a trash-can icon that appears for each user which when clicked does the soft deletion for that user and removes them from the view. However, the All Users view, when loaded on its own displays this deleted user with a de-activated status.
We will either have to choose between:-
1> Only displaying active users. Removing the user on deletion would be consistent with this.
2> Displaying both active and de-activated users. In which case, we'll need to change the post-delete display show that the deleted user is displayed as de-activated and not removed.
I'm in favor of 1.
2 can be confusing for the admin. Either the admin will be interested in actions pertaining to active users or de-activated ones but not both together. Hence the UI should be task/status-specific and optimized for the more expected/usual case. Also 2 can lead to the admin mistakenly deleting an already deleted user. This won't cause problems but it's best we make the UI restricted to prevent it.
I suggest 1 with future modifications to toggle the All Users between active and de-activated with possible actions (Delete, Re-activate) appearing contextually.
1 would also address https://github.com/nilenso/kulu-backend/issues/13

@kitallis @dvenkatr Re-activation should perhaps be a separate issue and left out of scope from this?

Done with

https://github.com/nilenso/kulu-backend/pull/18
https://github.com/nilenso/kulu-website/pull/18

Also takes care of #13

@kitallis @dvenkatr I had added the soft-delete functionality. There's a trash-can icon that appears for each user which when clicked does the soft deletion for that user and removes them from the view. However, the All Users view, when loaded on its own displays this deleted user with a de-activated status.
We will either have to choose between:-
1> Only displaying active users. Removing the user on deletion would be consistent with this.
2> Displaying both active and de-activated users. In which case, we'll need to change the post-delete display show that the deleted user is displayed as de-activated and not removed.
I'm in favor of 1.
2 can be confusing for the admin. Either the admin will be interested in actions pertaining to active users or de-activated ones but not both together. Hence the UI should be task/status-specific and optimized for the more expected/usual case. Also 2 can lead to the admin mistakenly deleting an already deleted user. This won't cause problems but it's best we make the UI restricted to prevent it.
I suggest 1 with future modifications to toggle the All Users between active and de-activated with possible actions (Delete, Re-activate) appearing contextually.
1 would also address #13
If we go ahead with 1, we can remove the status from display as it becomes redundant then.

sounds like the best option to me. I also agree that re-activation can be tackled later (or not at all at the moment).

Should only admins be able to de-activate other users (and admins)? I think that should be the case. Thoughts?

@murtaza0xFF Yes, this ability should be available for only admins. Other users should not even be seeing this option.


chore(script): remove PUSH0 warnings for superchain testnets

Motivation
Per the new canyon hard fork, push0 is supported on all superchain testnet networks.
Solution
Remove the warning
(Eventually we should have a better way to add/remove these warnings rather than a static list, but we can think about this later)

I think Zora and PGN both have testnets that now would also support Shanghai for the same reason

@mds1 yep, but they aren't on alloy-rs/chains yet—let's add them and move these to the new repo in a follow up


[AutoPR resources/resource-manager] Added /whatIf operation to deployments at resource group and subscription scope

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7254

(message created by the CI based on PR content)

Hi @zikalino , is there any change to get this merged recently? We are working a new CLI cmdlet that needs to be done before Ignite, which is pending on this.

@zikalino  sorry for bugging again but is there anything we can do to re-open the PR? This one is important for our team and it would be greatly appreciated if we can get unblocked.


After filling in a form - set goals up / reminder for next time

Do we want to integrate some sort of SMTP email reminders?
Might be interesting to write our own from scratch using (this)[https://github.com/emersion/go-smtp]
Or just some out of the box service. I've used (Mailgun)[https://www.mailgun.com] loads personally and it's great and got a free plan.

Also thinking more about this we will need some sort of emails for authentication, confirm account reset password etc

Yeah email notifications are definitely on the cards but I'd push back on it for the moment. At the least, we want to set up user auth first - it can come a bit later down the line


Migrate to knative.dev/reconciler-test/pkg/resources/service

Migrating from knative.dev/eventing/test/rekt/resources/svc to knative.dev/reconciler-test/pkg/resources/service.

/assign @matzew @pierDipi


BUG: Assets do not update on switch on Swap screen

Describe the bug
When switching assets in from and to the amounts do not get swapped.
To Reproduce
Steps to reproduce the behavior:

Go to Swap screen
Select two assets and input amount
switch assets
See error

Expected behavior
Erase from amount when flipping tokens
Screenshots
https://github.com/user-attachments/assets/ce2e218c-3c89-4b0a-8557-fb14ecb79ed8
-v48

could not reproduce, issue contains android screen record

@flypaper0 is an android screen though that still also happens on iOS.
CAn we improve logic that if swap quote is found and switching the assets that the last amount of the quote also gets swapped?

This looks like an android issue , close it here


Matching results disappear when expending the term

Hi Oliver,
I am working with lunr since a couple of week (which I have to say is awesome in term of easy to use, and features). I am having a strange behavior who seems to be similar to something already described in this issue #68
I have entries some containing phone, others containing iphone. If I search for phon, both phone and iphone entries are returned. If I search for phone, only phone entries are returned.
I am using leading and trailing wildcard on term :
q.term(term, {
    boost: 100
})

q.term(term, {
     boost: 10,
     usePipeline: false,
     wildcard: lunr.Query.wildcard.LEADING | lunr.Query.wildcard.TRAILING
})

I tried by adding only with leading wildcard, but results are the same.
It can be easily reproduce with this code :
const lunr = require('lunr')
const data = [{name: 'phone'}, {name: 'iphone'}]
const index = lunr(function () {
  this.ref('name')
  this.field('name')
  data.forEach(item => this.add(item))
})

index.search('phone')
// [ { ref: 'phone', score: 1, matchData: { metadata: [Object] } } ]
index.search('iphone')
// [ { ref: 'iphone', score: 1, matchData: { metadata: [Object] } } ]
index.search('*phone')
// [ { ref: 'phone', score: 1, matchData: { metadata: [Object] } } ]
index.search('*phon')
// [ { ref: 'iphone', score: 1, matchData: { metadata: [Object] } } ]

Am I missing something, or is this a bug ? If it's one, how can I help to solve it ?
Thanks a lot.

I've created a fiddle with your reproduction, at the very least those results are not what I would expect, specifically the third one "*phone".
My guess is this is to do with stemming, though I'd have to do a bit more digging to be sure.

Thanks. I will try to dig in it too, but I discovered stemming just a few days ago....

Ok, I've taken a closer look at this, and it is due to stemming.
If you look at index.invertedIndex you will see what Lunr has indexed, it has two entries: iphon and phone. So, it is stemming "iphone" to "iphon", dropping the "e", but not doing so with "phone", this is weird, but algorithmic stemming is not perfect. I would certainly have expected those two terms to be stemmed identically.
Knowing that, its now easier to understand the results you are seeing: "*phone" can't match "iphone" because its actually indexed as "iphon". You're 4th search is close but doesn't have a wildcard at the end, making it *phon* would give you the results you expect.
You can remove the stemmer, though this will likely increase the index size, and reduce re-call for most searches:
lunr(function () {
  this.pipeline.remove(lunr.stemmer)
  // define the index
})

Alternatively you could put a pipeline function before the stemmer that indexes "iphone" as "iphone" and "phone".
Finally, you could try and figure out why the stemmer is doing what it is, though I warn you it is a lot of regex in the implementation, there is also a description of the rules it is following which might help.
If it was me, I'd go for option 2. Let me know how you get along with it.

Hi @olivernn
Thanks a lot for the time you spent on this. I understand that stemming is causing the trouble. Second solution sounds the best one, but I am just afraid there are others terms like iphone that I am not able to see...
That's why I am using leading and trailing wildcard, but if*phon* is working fine, *phone* is not working. I guess the trailing wildcard is understood as at least one character.
I will read the links you gave me and try to use a pipeline function.

I have the same issue with trailing wildcards and stemmer. For example if you try to index sophie using default index and try to search sophie* it will return empty results.
Here is fiddle with my example

I had a look into the stemmer code, and found out why he is doing this. There is this regex : var re_5 = /^(.+?)e$/ which is removing the trailing e of tokens. The same regex will apply for sophie @mike1808
So it's fine the search is going through the pipeline, but since using leading or trailing wildcard is disabling it, lunr is not able to solve *phone properly...
Is there a way to run the stemmer on the unwildcarded part of the query and then add the wildcards back ? Something like : '*term' -> '*' + stemmer('term') -> '*stemmedTerm'.

Just an update. I was able to do what I was saying with :
const results = index.query(function (q) {
  const stemmed = index.pipeline.runString('phone')

  stemmed.forEach(term => {
    q.term(term, {
      boost: 10,
      usePipeline: false,
      wildcard: lunr.Query.wildcard.LEADING
    })
  })
})

But.... If iphone is turned as iphon by the stemmer, phone is not turned as phon, because it does not match the replace condition of step 5 if (re.test(stem) || (re2.test(stem) && !(re3.test(stem)))).  I did think about re-implementing it, if nothing else then to understand the rules a bit more, but time is always in short supply!
@mike1808 regarding the behaviour of the wildcard, I think that is a bug, or at least incorrectly implemented. I took inspiration for the syntax from Lucene and its * matches 0 or more, where it looks like Lunr implements it as 1 or more. I'll take a look at what would be involved in fixing this.

@olivernn, I have the same issue as @mike1808 described. Thanks in advance for looking into it.

@mike1808 actually, scratch that, I just wrote a test and it does match 0 or more characters, you're right that it does need stemming to match.

@myalgo @mike1808 the test I just added.

@olivernn, why does * match 1 or more in Lunr? 0 or more sounds more appropriate.

seems moreover like + here

Is the proposed solution the best option right now?
lunr(function () {
  this.pipeline.remove(lunr.stemmer)
  // define the index
})

Or is there going to be a future change to * wildcard behavior?  Right now I do lose search results on many queries when I include the full query term and a wildcard. Like: Mike's sophie*  example.

@mdobeck that is the right way to remove the stemmer when building the index, there is also a search pipeline this.searchPipeline, that also includes stemming.
I'm going to close this now as I don't think there is anything left to do, feel free to comment and I'll re-open if required.


File formats for --dump-eqclasses

I'm trying to understand gene_eqclass.txt.gz and geqc_counts.mtx. I mostly get it from skimming quant.rs and the comments therein: gene_eqclass.txt.gz has a line giving the number of genes (or the number of USA targets), then a line giving the number of ECs. Then there is one line per EC, where the first n-1 entries are gene IDs, and the last entry is the EC idx (which is not the ordering in the file).
Then geqc_counts.mtx is cells x ECs, presumably with the row labels (cell barcodes being given by quants_mat_rows.txt. But what is the indexing for the columns, i.e. the ECs? Is that the EC idx (the last entry of each line in gene_eqclass.txt.gz) or the line number (-2) from gene_eqclass.txt.gz?
Thanks!

Hi @davidaknowles ,
Judging from this line and the way geqmap was populated here, I think it's the former, i.e., the indexing for the columns is the last entry of each line in gene_eqclass.txt.gz not the line number.
@rob-p can correct me if I am wrong.

Thanks Avi. @rob-p enjoy RECOMB, no rush.

Hi @davidaknowles,
Sorry for taking so long to get back here.  @k3yavi's interpretation is correct. What is written in the MatrixMarket file for each triplet is the row id (i.e. cell barcode), column id (equivalence class id), and UMI count.
The equivalence class id is the unique, distinct id associated with each pattern of gene occurrences (or gene + splicing-status occurrences).  For each equivalence class label in gene_eqclass.txt.gz, the equivalence class id is the last number of the line.
Best,
Rob

Perfect, thank you!


G3 station chosen (ASC)

we G3 choose Ascension Island, United Kingdom (ASC)

@LinxuanHu have you chosen monthly averages for the data frequency?

nice

good


Add support for rspec 3.4

https://github.com/rspec/rspec/releases/tag/v3.4.0

Any pointers to what doesn't work with 3.4? The gem spec seems to indicate that 3.4 is supported. I'm away from a computer at the moment, so I can't test right this second.

When I ran bundler, it says the dependencies didn't match up. I'll double
check tomorrow and post the message!
On Tue, Nov 24, 2015 at 3:59 PM Jake Goulding<EMAIL_ADDRESS>wrote:

Any pointers to what doesn't work with 3.4? The gem spec seems to indicate
that 3.4 is supported. I'm away from a computer at the moment, so I can't
test right this second.
—
Reply to this email directly or view it on GitHub
https://github.com/ci-reporter/ci_reporter_rspec/issues/16#issuecomment-159403439
.

Looks like the issue was on my end and how the gem decencies were being updated. Thanks for looking into it!


Fix bad header in 0.46 release note

Closes https://github.com/Qiskit/documentation/issues/751. As explained there, the problem is from Sphinx rather than our API generation script. It's using a <dl> in HTML where it should not be.
After, the list works properly:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 89.232%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
6
91.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
58847

Relevant Lines:
65948

💛  - Coveralls

This is ready for review and would be good to include in the 1.0 release. I'm not sure if it should be included directly in https://github.com/Qiskit/qiskit/pull/11760?

At this point, it's probably best just to do it with the rest of the release notes.  I've applied this patch to #11760 in 476c9451ae, so I'll close this PR - thanks for pointing it out.


Issue #86 Qing's submission

Well done, Qing.

Well done, Qing.

Thank you Professor Danil for this wonderful semester! I really learn a lot from these interesting exercises and topics!


Added Stanza website Clone

This PR fixes the issue Stanza Website UI #455. Please review sir

@Jaideep25-tech @AyushSingh22 Sorry for the inconvenience sir. Can you please upgrade the level label to 3? Only if you feel so.
Reasons:
I have made the website to work temporarily work by linking them to their original links.
Made the UI almost same as that of original one.
coded such that it can be used for further developments too.


Slightly Stricter Linting

Turned ESLint errors on (and fixed errors in code) for:

consistent-return
eqeqeq
no-useless-escape
quote-props

Still lots of things that could be tightened up in ESLint config, but mostly style related. Would be great to add Prettier before tackling those ;)

If we’re thinking of using Prettier then the style rules for ESLint can be removed. Current ESLint and Prettier versions work well together, and auto-fix of style errors would probably be best handled by Prettier.

I believe the way Prettier works is that it takes its cues from the ESLint config, so we will still want the style rules there in order to guide Prettier.

I believe the way Prettier works is that it takes its cues from the ESLint config...

It does not ;)
Because I just updated all my project build/dev toolsets, ESLint and Prettier configuration is fresh on my mind. So I've opened a new enhancement issue #1846 to continue this discussion, and determine how we want to implement code formatting tools in OpenSeadragon!


Update node-sass to 4.14.0 for compatibility with Node 14.x

Description
Bumped the node-sass dependency to be compatible with Node 14.x.
Types of changes

[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 to not work as expected)

Checklist:

[x] My code follows the code style of this project (run $ yarn lint).

@dpeukert Thanks for your pull request. Can you please provide additional info as to which issue this is addressing? Thank you! 🙏

@dpeukert Can you please provide additional info as to which issue this is addressing? Thank you! pray

Added.

@dpeukert Any luck you still have the error you faced and can still share it here? That would help for future search queries 🙏 Approved otherwise as this is a minor update and shouldn't contain breaking changes.

It was something along these lines IIRC (from https://github.com/sass/node-sass/blob/master/TROUBLESHOOTING.md#404s):
><EMAIL_ADDRESS>install /src/node_modules/node-sass
> node scripts/install.js

Downloading binary from https://github.com/sass/node-sass/releas…
Cannot download "https://github.com/sass/node-sass/releas…":

HTTP error 404 Not Found

@dpeukert Thank you for the clarification!


Try --depth 1 for Appveyor build

Well, this needs to be tested after ghc.haskell.org is upgraded.

Thanks @gracjan! I'll retry this PR once ghc.haskell.org is upgraded

Why not using the GHC on GitHub instead like Travis? I trie a bit but this https://ci.appveyor.com/project/izgzhen/hadrian/build/1.0.1 failed out of some weird reason

@izgzhen Yes, we can use the GihHub mirror. In fact we did use it the past, including the --depth 1 flag proposed in this PR, e.g. see this old snapshot:
https://github.com/snowleopard/hadrian/blob/3a04d3425a06ff9ad62eb71d20ddd7cce26ae0a0/appveyor.yml#L20-L25
However, at some point I decided to simplify the build script and have rewritten, trying to make it closer to the recommended way of building GHC on Windows (doc).
I don't really mind to revert back to cloning from GitHub as long as it brings us some time saving compared to the current simpler solution.

Ideally, I'd like us to test the build script in the doc via AppVeyor, so it doesn't get out of date.

I tried what this PR tried again, and failed again. What about closing this PR and restate the problem in an issue instead?

@izgzhen Good idea. Also, we need to quantify how much it actually helps. If this only shaves off 1% of the build time, then it may be not worth the effort -- the usual variability of build times is around 10% anyway.

@gracjan Thank you for the PR, but it looks like it may require a bit more digging to implement it.


README.md has incorrect example

I believe (currently line 148) of README.md
artifact.txt:application/vnd.acme.rocket.config.v1+json \ should be a layer and not a config, like: artifact.txt:application/vnd.acme.rocket.layer.v1+txt \
in the section: Push multiple files with different mediaTypes:

Resolved with recent PRs.


Add test cases for array type

I've created additional tests for this package during a bug finding mission.
Your package works absolutely fine. Finally, I've found the relevant bug in rc-field-form.
Nonetheless I'd like to give you back my test cases for providing this useful package.

Coverage remained the same at 90.182% when pulling 3300596db9f5b8aa43ae6678b77badb9a9e2cce2 on fknipp:master into 0d51d60086a127b21db76f44dff28ae18c165c47 on yiminghe:master.


Better names

Migrating to the names used by the documentation. Not sure if 'log' should be 'ledger'. "Log Entry" sounds more natural than "Ledger Entry". Also, might want to reserve 'ledger' to mean "log after consensus".

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #10   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines         140    140
=====================================
  Hits          140    140

Impacted Files
Coverage Δ

src/lib.rs
100% <ø> (ø)
:arrow_up:

src/log.rs
100% <100%> (ø)

src/historian.rs
100% <100%> (ø)
:arrow_up:

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 6ec0e58...ceb65c2. Read the comment docs.


[core] User-specified controller's telemetry namespace.

Telemetry namespace of controllers should not be hard-coded, but defined at construction or initialization. Uniqueness should be ensured by EngineMultiRobot.

I don't think this feature is useful after all.


Add underline support

Would be nice to add underline support.
To do it now you can use the following code:
import jira2markdown
from jira2markdown.elements import MarkupElements
from jira2markdown.markup.text_effects import QuotedElement, Underline

text = "This is a +test+"

class CustomUnderline(QuotedElement):
    TOKEN = "+"
    QUOTE_CHAR = "<u>"
    END_QUOTE_CHAR = "</u>"

elements = MarkupElements()
elements.replace(Underline, CustomUnderline)

print(jira2markdown.convert(text, elements=elements))

Hi @Bartemis, thank you for you proposal. Could you please test updated code with your test cases before I publish it? See https://github.com/catcombo/jira2markdown/pull/15

Hi @catcombo, sure I will add some test cases for you

@Bartemis great, thank you! I'll merge the PR and will publish a new version.


Fixes Issue #8

fixes issue #8 :
added mobile specific treatment
fixes some UI

This change is

LGTM


strainscan_build stuck after processing a cluster

Hi,
thanks for writing StrainScan, it's very useful!
However, I'm having some problems during the strainscan_build step.
I want to build the database on all the 358 NCBI RefSeq genomes of Lacticaseibacillus rhamnosus. I'm using ubuntu version 22.04 (installed as Windows Subsystem for Linux version 2 on a DELL mounting windows 11), with ~25.6 GB memory allocated.
Also, I installed strainscan using conda:
conda create -n strainscan -c bioconda strainscan=1.0.14 (latest version).
The command I'm trying to use for the building step is:
strainscan_build -i rhamnosus_genomes/ -o strainscan_rhamnosus_dir/ -k 31 -t 6 -u 100000 -e 1
The command starts running well, it detects 92 clusters, and then extracts k-mer from most of those clusters. However, after correctly processing one cluster (specifically, cluster named as C5) it stops and gets stuck. It does not start to process the next cluster, and running 'htop' I see that the processes are in S, no longer running.
I also left the computer run for one night, but the situation did not change: it does not start processing the next cluster.
I also tried several times running the same command, but it always gets stuck after processing cluster C5.
Any guesses on how to solve this?
Thank you!!

Hi, thanks for using StrainScan!
May I know the size of C5? If it's a large cluster (containing many strains), then this problem can be caused by a lack of memory.

Actually, k-mer extraction from cluster C5 always works well. By checking the folder Kmer_Sets_L2/Kmer_Sets I see that there are 7 empty folder (I think those clusters not yet processed), which are:

C26 (31 genomes),
C30 (15 genomes),
C43 (15 genomes),
C46 (49 genomes),
C55 (11 genomes),
C82 (21 genomes),
C86 (76 genomes).
Other folders with several genomes (such as C73, which has 18 genomes) was processed successfully.
I also attach the txt file with size of all the clusters.
hclsMap_95_recls.txt

However, the strange thing is that I don't have a "Killed" message (as it happens in other cases), but the process simply does not run without providing any warning.
What's your idea on how to solve this issue? Thanks

It looks like the C5 is a cluster only containing 4 strains. Then, it should not require a lot of memory.
It's a little hard to identify the reason for this problem without data or log. Thus, could you please provide the input reference genomes to us for testing? (via email -<EMAIL_ADDRESS>or any other ways you prefer). In this case, we can debug the program asap and let you know the possible solution. Thanks!

Thank you so much! I just sent you the genomes via mail

Hi, I have built the database successfully using our server (100G memory is given).
You can download the database via this link.
As suggested by the attached log file, the construction process requires about 29G memory, which is why your PC can not finish the construction.
strainscan.log.txt
If you have any other problems, please let me know. Thanks!

Yes, now it works perfectly. Many thanks!


Allow the selection of full and brief format from the WebSearch collection interface

Originally on 2010-07-07
(Originally submitted by Ferran Jorba on Savannah: https://savannah.cern.ch/task/?12167)
The tools to create formats (.bft and .py) are powerful and programmer oriented.
However, there is no easy way to assign those brief and full formats to a given collection. It would be nice to integrate those options to the WebSearch collection definition, like portalboxes, search options and other collection specific preferences. Maybe this could make .bfo files obsolete?
I understand that HB and HD should be the default, good enough for most collection and sites, and it should remain the valid ones unless a specific choice has been made.
Please note that this preference should be taken into account also when creating the first, cached page (http://cdsware.cern.ch/repo/?p=cds-invenio.git&a=search&h=HEAD&st=grep&s=def+create_latest_additions_info)

I believe this is an interesting need, indeed (we might also needed it INSPIRE for branding particular collections), but better do it on next...

Each collection can have it's own Jinja2 template (see https://github.com/inveniosoftware/invenio/blob/master/invenio/modules/collections/views/collections.py#L102-L105).

Awesome! Thanks


[CARBONDATA-3874] segment mismatch between maintable and SI table when load with concurrency

segment mismatch between maintable and SI table when load with concurrency
Why is this PR needed?
segment mismatch between maintable and SI table when load with concurrency
What changes were proposed in this PR?
segment mismatch between maintable and SI table when load with concurrency
Added control check for SI enable flag of SI tables in concurrent scenarios.
Does this PR introduce any user interface change?

No

Is any new testcase added?

No

https://issues.apache.org/jira/browse/CARBONDATA-3874

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1500/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3228/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1572/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3310/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3312/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1574/

LGTM

retest this please

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3315/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1577/

LGTM


docs: fix typos and grammar errors

add code tags where necessary
fix links

closes #89

Awesome ! will go in next release.


Restore global layout concept

App knows about global layout but doesn't care about the details
Tab can call App#onLayoutChange to let App know that layout has changed
App will only take care of persisting changes

Related to https://github.com/camunda/camunda-modeler/issues/910.

Added a couple of comments to the feature commit.


Upcoming breaking change regarding configGroupNesting

Edit: After #94 the format has changed again. The rest of comment has been updated to address this.
There are two new changes regarding the format of configurations for the file module. One is that all keys in configFile, dataFile and file require the value property, meaning, for example. what before was:
programs.plasma = {
  configFile = {
    "krunnerrc"."General"."FreeFloating" = true;
    "kwinrc"."Desktops"."Number" = 8;
    "kwinrc"."Desktops"."Rows" = 2;
  };
};

now must be:
programs.plasma = {
  configFile = {
    "krunnerrc"."General"."FreeFloating".value = true;
    "kwinrc"."Desktops"."Number".value = 8;
    "kwinrc"."Desktops"."Rows".value = 2;
  };
};

We also had to remove the recursive attribute config format (even though it was newly introduced), because it had some limitations with extensibility.
The other difference is that the new delimiter for nested groups is /, which was changed from .. This means that where you before had:
programs.plasma.configFile."group.subgroup1.subgroup2"."key" = "value";

you now will have to use:
programs.plasma.configFile."group/subgroup1/subgroup2"."key".value = "value";

This also means that if you don't have to escape periods anymore, they can just be left as is, but if you for some reason need / in your groups, you can escape this by part1\\/part2\\/part3, which will translate to a group named part1/part2/part3 (not that there should be any reason to have / in your groups for kde plasma config-files anyway).

Thanks to #144 by @quentinmit you now can skip the .value part of the key if you don't want to tweak immutability or persistence or other attributes. In other words
programs.plasma.configFile.group.key = "value";

will be equivalent to
programs.plasma.configFile.group.key.value = "value";

I also said in another issue that I would step aside from the project due to switching distros, but that probably won't happen anyway as I did find out:

An OK solution to my software needs (for development)
Other distros are hardly good options for my needs as well

In other words I'll continue to be involved in the project in the future :)
I'll close this now since this isn't all that relevant anymore with the latest addition.


Dll

Coverage remained the same at 100.0% when pulling 8cf5d6358b1e90a560e90482f2250ca4bec518b6 on dll into 536852b36dc46407d5199c53f52a6867f2e55662 on master.

Coverage remained the same at 100.0% when pulling 39046c7ed6134cea59c7f0ec65b8856cb83c7e06 on dll into 536852b36dc46407d5199c53f52a6867f2e55662 on master.


fix: some fixes and tests

Pull Request Test Coverage Report for Build 1031

21 of 59 (35.59%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-3.8%) to 72.165%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/Modal/index.js
2
14
14.29%

src/components/Modal/manage-tab.js
0
26
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/components/Modal/index.js
1
45.0%

Totals

Change from base Build 1026:
-3.8%

Covered Lines:
551

Relevant Lines:
727

💛  - Coveralls


Add postgres dump

Hi
I have added dump for postgresql
Tested on my zabbix 3.0.0 and postgresql 9.6.2
but I have not tested with mysql

Very nice @sergeygalkin - it's great to have an additional database covered.
One thing: people might not be happe if the script changes their config file. Could you change the script to put the config into a temp file PWFILE=$(mktemp) and set PGPASSFILE=$PWFILE. Does that work for you?

@maxhq make sense. I have added and tested.

I'd suggest a sqash to keep history clean.

@endersonmaia make sense

@sergeygalkin you could make a git rebase -i master from your functional branch and git push --forceto the remote branch you're using to this PR, but it seems that you don't have a public github fork for the project, is that right ?

@endersonmaia  I have created https://github.com/maxhq/zabbix-backup/pull/11 from my new   https://github.com/sergeygalkin/zabbix-backup.  I can't create single commit pull request in this pull request


Fix optional props

Fixed optional prop range in "RangeSlider".
Hoist required props in Usage and Props section.
Add missing props in Basic Usage section.
Fix "DatePicker" queryFormat value typo.

range has defaultProps, so it should be an optional prop in RangeSlider.
I'll fix the other components in reactivebase - propTypes with required props.

Updated PR and the description too.


License

This isn't fair. I think you should return Emil Kreutzman as an author of the original app.
link

You both did a greate job.

Yes, thanks, you're right. I will return the author with the release of the next version.


Not extend Junit Assert classes in Tests.

Expected behavior
Asserts from Junit must be used statically
Actual behavior
All the tests extend from Assert but do not override anything. This leads to tight coupling and fragile code. Moreover Jedis resources are not closed properly in tests, which could have been the reason for recent Travis failures. But for these changes I will create a new issue.
Steps to reproduce:
This is a test enhancement
Redis / Jedis Configuration
Jedis version:
Redis version:
Java version:

Resolved by #1331


game creator should automatically join / enter the lobby

There's a code for autojoin already but it's broken because it doesn't wait until the game is actually created on the server, i think.


[RC3] Re-opening F# Tutorial Script opens the wrong editor, making it think it's a XAML(?) app

I'm not sure what's happening.

Install RC3 + VSIX
Create Tutorial project.
Close the Tutorial.fsx file.
Re-open.

Expected: It opens, it's colorized, and it all "works".
Actual: Completely changes the editor into something with "Design", "Split", and "Source" tabs at the bottom.

@cartermp I don't know the underlying mechanism but it happens because Tutorial.fsx has Compile action in fsproj instead of None.

I suspect it also has something to do with System.Windows.Forms reference. Removing it restores normal behavior.

@majocha, I was able to 'fix' the issue by setting the action to 'None' like you suggested (vs. 'Compile').  Including or excluding System.Windows.Forms didn't seem to affect anything on my machine.

Interesting.  The Compile action is necessary to be able to run the Event-handling section of the tutorial as a WinForms app, so we definitely want to keep that.

HTML validation errors, DOCTYPE... Apparently it's some ancient HTML designer that kicks in and hangs. Didn't know Visual Studio still had one.

@brettfo my bad. I can't reproduce that one either. I guess I did more than one change at a time and it was confirmation bias on my part.

Just checked on another machine and this bug is also present on RC2 with OSS vsix built from recent master.

OK, I've noticed yesterday that this bug is not in "naked" RC3 i.e. with the old RC2 VisualFSharp.
I've reverted e110833748a18fb603ab22b99b8bd3fac83f7871, rebuilt VIsualFSharp and reinstalled the open source vsix and the bug is gone. This is tested on RC2 only, because I don't have RC3 at hand right now.

This was fixed by #2408


Mocking a method's response using SinonJS

I am very new to testing world. I have a following function:
exports.get = function (para1) {
    var response = querySys(para1);
    response.in = 1;
    return response;
}

I want to mock the result of querySys method.
I want it to return {in: 0, out: 1}.
So how do I do it with Sinon? Do I have to mock it or stub it?

You would stub it
http://sinonjs.org/docs/#stubs


Copy the allinone example and adapt it for Ironic

Replace the outdated "ironic" example with a fresh copy of the
"allinone" example, and begin updating it to properly deploy an allinone
host with Ironic.
Also adds an if-block to nova.conf, to adjust the virt driver if that
compute host should be using Ironic instead of libvirt.
Change-Id: I1ac20c10d64fe178220a88849d8f31b3b09e7aaa

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

This got updated in #2119 and merged there, so I"m closing this one.


Improve hacky manpage

note(1) is a mess. Improve it!

https://github.com/rumpelsepp/pynote/commit/92c654c87543109c017d66907e730022bda7cc76


Next, SSR, authExchange and cookie auth

Discussed in https://github.com/FormidableLabs/urql/discussions/1973

Originally posted by saevarb September 20, 2021
Hello.
I have built a nextjs application, and that same server hosts a GQL API which I am querying with urql. Authentication is done via an HTTP-only cookie, and I'm using next-urql.
I also want to use getServerSideProps to authenticate my pages, in case that is relevant.
My question is: How do I properly configure authExchange (and I guess any other relevant part of urql) so that urql can run during SSR when using cookie auth? It seems to me that on the server, I need to be able to access the next context's request and grab the cookie, so we can use it on the user's behalf. When on the client, nothing special needs to be done because the cookie is send automatically.
I thought I might be able to do this using the context parameter given by withUrqlClient, but it seems that this is always undefined on the server.

The documentation seems to state that we should be able to make use of the next context, but given that it is undefined every time on the server, this seems like a bug.

context is injected when using the ssr: true option which utilizes react-ssr-prepass and getInitialProps to aggregate data upfront, if you need the context in getServerSideProps you'll have to incorporate the passing of said context in the server-side function like this:
export async function getStaticProps(ctx) {
  const ssrCache = ssrExchange({ isClient: false });
  const client = initUrqlClient({
    url: 'your-url',
    // pass in the ctx from the upper function in here
    exchanges: [dedupExchange, cacheExchange, ssrCache, fetchExchange],
  });

  await client.query(TODOS_QUERY).toPromise();

  return {
    props: {
      urqlState: ssrCache.extractData(),
    },
    revalidate: 600,
  };
}


#Bug: Datatype Any return generator object.

@RamanjaneyuluIdavalapati

While sending a data type of class object through kwickapi argument I specified the type as Any in the api function for that reason i got generator object instead of class object.

Api function.

Making request.

Receiving request.

@Rajendra-R

The generator problem is solved.
And about your second issue, as of now kwikapi doesn't support sending class objects from client to server or vice versa.


Fix warnings related to the unsignedness of variables

When compiling the robotiq_ft_sensor_hardware package with colcon, e.g.
colcon build --packages-up-to robotiq_ft_sensor_hardware

I obtain the following warnings:
Starting >>> robotiq_ft_sensor_interfaces
Finished <<< robotiq_ft_sensor_interfaces [3.35s]
Starting >>> robotiq_ft_sensor_hardware
--- stderr: robotiq_ft_sensor_hardware
In file included from /usr/include/termios.h:25,
                 from /home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_com.cpp:49:
/usr/include/features.h:194:3: warning: #warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE" [-Wcpp]
  194 | # warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE"
      |   ^~~~~~~
/home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_state.cpp: In function ‘float rq_state_get_received_data(UINT_8)’:
/home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_state.cpp:242:9: warning: comparison is always true due to limited range of data type [-Wtype-limits]
  242 |   if (i >= 0 && i <= 5)
      |       ~~^~~~
/home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_com.cpp: In function ‘INT_8 rq_com_send_fc_16(INT_32, INT_32, const UINT_16*)’:
/home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_com.cpp:610:17: warning: comparison of integer expressions of different signedness: ‘UINT_32’ {aka ‘unsigned int’} and ‘INT_32’ {aka ‘int’} [-Wsign-compare]
  610 |   for (i = 0; i < n; i++)
      |               ~~^~~
/home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_com.cpp: In function ‘float rq_com_get_received_data(UINT_8)’:
/home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_com.cpp:1123:9: warning: comparison is always true due to limited range of data type [-Wtype-limits]
 1123 |   if (i >= 0 && i <= 5)
      |       ~~^~~~
---
Finished <<< robotiq_ft_sensor_hardware [10.2s]

Summary: 2 packages finished [13.8s]
  1 package had stderr output: robotiq_ft_sensor_hardware

This PR fixes the warnings related to the (un)signedness of variables.
With the changes introduced by this PR, the output of the same command is the following:
Starting >>> robotiq_ft_sensor_interfaces
Finished <<< robotiq_ft_sensor_interfaces [0.48s]
Starting >>> robotiq_ft_sensor_hardware
--- stderr: robotiq_ft_sensor_hardware
In file included from /usr/include/termios.h:25,
                 from /home/vpetrone/ros2_ws/src/rq_fts_ros2_driver/robotiq_ft_sensor_hardware/src/rq_sensor_com.cpp:49:
/usr/include/features.h:194:3: warning: #warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE" [-Wcpp]
  194 | # warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE"
      |   ^~~~~~~
---
Finished <<< robotiq_ft_sensor_hardware [0.81s]

Summary: 2 packages finished [1.54s]
  1 package had stderr output: robotiq_ft_sensor_hardware

@v8p1197 hey thanks for the pr, have you tested it on real hardware? otherwise i will merge it once i get the opportunity

@panagelak Hi there, I've fixed the other warning I was mentioning in my previous comment

warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE"

I've tested the updated version of my branch on the real hardware, and everything works.


Fix issue with manual ack not sent on parsing error

Problem
When there is an error while parsing a message and manual acknowledgment (ack) is set, an exception occurs, but the ack is not sent. The default RabbitMQ configuration will close this channel with a PRECONDITION_FAILED if no ack is received within the required period (default 30 minutes), see the documentation for more info. Fluentd will not recover from this state and will stop sending records to RabbitMQ in that case.
Solution
This PR catches parsing message exceptions, logs the error message, and continues processing. This ensures that the ack is always sent, preventing the channel from being closed by RabbitMQ.
Changes
Wrapped the parsing logic in a begin-rescue block to catch Fluent::Plugin::Parser::ParserError.
Logged the error message when a parsing exception occurs.
Ensured that the ack is sent in the ensure block.

Thank you!


Training: Monitoring used - directly and indirectly - in WM

Impact of the new feature
WM in general
Is your feature request related to a problem? Please describe.
Sharing knowledge and empowering WM/Ops to understand and debug the system. Sub-task of https://github.com/dmwm/WMCore/issues/11837
Describe the solution you'd like
Topic of this ticket is: Monitoring used - directly and indirectly - in WM
For the expected solution, please refer to the meta-issue for further details (in short: documentation + recorded presentation).
Describe alternatives you've considered
None
Additional context
None

@d-ylee @khurtado now that the training has been provided and training material updated in https://cms-wmcore.docs.cern.ch/training/monitoring/, shall we close this out?


make recv-side-twcc work

Description
make recv-side-twcc work
Reference issue
Fixes #117

will take a look tomorrow!


Discord link is not working

I would like to join the discord group but the link in the website is no longer working, likely expired.

Oh no! Let me check

Which link are you referring to? I checked the SlateDB.io link and the bandage on the readme. Both seem ok.

Ohh the link works when I use another browser to open it, might be some setting I've with my browser that causes this.


Could you please provide library for select multiple image from Gallery

Please

Not now. Maybe later.

I add my voice, this would be awesome

+1

Added this feature in 1.3.0 version

Hi @Frederikos . It does not look like this feature is available on the build i generated from the sample app. I am running the build on Marshmallow. Digging into the code sample, I see that requestMultipleImages() is never used. How do we use this feature?

Hi @yookore
This is not main feature and sample app not using it.
I write sample request in readme :
RxImagePicker.with(context).requestMultipleImages().subscribe(new Action1<List>() {
@Override
public void call(List uris) {
//Get images by uris.
}
});


Fix availability issue for mini

Pull request recommendations:
Fixes https://github.com/home-assistant/core/issues/93976 by porting https://github.com/vangorra/python_gogogate2_api/commit/e4b8e5ea65be6bcb38a3cc54e5a5b1bc2201aa28 from https://github.com/vangorra/python_gogogate2_api

@unDemian Tests are failing, please check

https://pypi.org/project/ismartgate/5.0.1/

You'll have to bump the lib in HA as well

Will do, thanks

https://github.com/home-assistant/core/pull/100636


Outdated example in documentation?

Hi, just flagging a very minor possible issue in the documentation.
On line 200 of /docs/image-configuration.md the example includes 'resize_srcset' => true - is this an outdated name for the generate_srcset_sizes option?

Hey @patrickcameron
I couldn’t find any setting named resize_srcset in earlier versions of Timmy 🙈. So I don’t know where that came from. I probably meant to use generate_srcset_sizes in the example.
Thanks for mentioning this 👍. Fixed in 695116850e8879c3fefd10f08953d5892783f78e.


Ensure the filesystem handle the tail '' correctly

Summary
During the ftw/nftw development, I found that procfs and romfs can't work handle the path with tail '', e.g.:
/proc/net/
/proc/0/group/
Let's fix them.
Impact
Testing

https://github.com/apache/incubator-nuttx/pull/1486


Don't fail if weightHeader is missing

Test on a sherpa sample, for example:
/GGJets_Pt-15_13TeV-sherpa/RunIISummer19UL17MiniAOD-106X_mc2017_realistic_v6-v2/MINIAODSIM

Addressed in https://github.com/kdlong/cmssw/commit/41c79e2f389e640ac0964398bc1440503c9f31b2


Use libgfortran on OS X

Follow-up on PR ( https://github.com/conda-forge/openblas-feedstock/pull/6 ). Uses libgfortran as a run-time dependency on OS X in addition to Linux. This seems to work from local testing, but will require the libgfortran package to exist on conda-forge before we can proceed. At this point it is merely a matter of waiting for the first Travis CI build to complete and upload a package. If this is failing on Travis CI at that time, please restart to get it to build.
cc @pelson @ocefpaf @msarahan

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Appears libgfortran is up and available. Thus this PR is now passing. Please feel free to merge if you are happy.

Thanks @jakirkham!


feat: add MusicBrainz IDs to song response

closes #14
apologies if this isn't an active repo anymore, i just really enjoy using this for a project i'm working on and having the musicbrainz id (weather or not last.fm even provides it 😅) would be super helpful for my usecase :-)
appreciate the work you put in to this!

Apologies I am not sure how I missed this PR before. LGTM, will release 👍


Can't get root component to work

Attempting to follow the root component guide but having some trouble. Everything else seems to be working. This is the main functionality that I find lacking in Nuxt and has me interested in Ream.
Is there an existing complete example set up with this functionality working?
Here's a work in progress https://glitch.com/edit/#!/lily-move

Just added an example: https://github.com/ream/ream/blob/master/examples/custom-root-component

Closing since it's solved.


Verify check on additional partners results in proper handling

I think we can do this.
registration.organisationDetails.grsRegistration is set in the saveRegistrationDetails by a get details call.
Which should mean the match is relevant to the speicific partner in question, rather than the partnership as a whole.

Either this Or https://github.com/hmrc/plastic-packaging-tax-registration-frontend/pull/489

Went for the other.


[New] add jsx-no-padded-children

Fixes #2045
The name of the rule mimics that of eslint rule padded-blocks. Don't know weather it is good or not.
I am concerned of the code duplication in functions checkPaddingTop and checkPaddingBottom, but cannot find a good way to solve it.

I am very unclear about how this rule should be designed.

Maybe it should be named jsx-padded-children?
Maybe it should be named  jsx-children-newline, and do what other *-newline rules do (function-parens-newline, array-bracket-newline, object-curly-newline). However since the concern of #2045 is about disallow empty line, this rule add an option to disallow empty lines, that is, only zero or one newline is allowed.

jsx-children-newline seems like a better name and model.

Closed this because I couldn't find a satisfactory solution.


Cannot get version number

python -m chatterbot --version
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/usr/local/lib/python2.7/dist-packages/chatterbot/__main__.py", line 8, in <module>
    print(chatterbot.__version__)
AttributeError: 'module' object has no attribute '__version__'

@yazdat
to get the version number try:
import chatterbot
chatterbot.__version__

This should possibly result in an outcome like:
'0.6.0'

I am not sure which Operating system are you using, Even after @gabru-md suggestion didn't worked, try to pip install chatterbot If it is Linux flavor try to sudo pip install chatterbot

Thanks for the response  gabru-md and  vkosuri.
import chatterbot
chatterbot.version
worked fine. I am running 0.6.1
I'm experimenting with chatterbot to use on a robot. I'm sure I will have more questions. I hope to contribute where I can.

even for me this python -m chatterbot --version at beginning then I used chatterbot.version to find my version ...


Google mobile ads not able to close in iphone 14

Plugin Version
2.0.1
Steps to Reproduce
When you show a full screen ad on the new apple 14 pro max, and maybe other versions as well, the ad cannot be closed anymore.

@reijerteunis Thanks for the report! This is a known issue being tracked at https://github.com/googleads/googleads-mobile-flutter/issues/191
Closing this issue as a duplicate, so please follow up on it for further updates.


Update new sensors to field_test_logger and other enhancements

Update new sensors to field_test_logger and other enhancements

New sensors added

Arguments handling refactored and new arguments added:
-u = suffix to the log file name (default: "")
-i = network interface used in logging (default: wlp1s0)
-b = batman interface used in logging (default: bat0)

error handling added, if max17260 not connected.  This flooded system log with HW without the max-chip.

@joenpera, what's the plan with this PR (to be pushed to get it merged or canceled)?

@joenpera, what's the plan with this PR (to be pushed to get it merged or canceled)?

Some delays with review.


Ako/ Add language switcher

This is to add language switcher component.
The desktop mode can be visible in storybook and the mobile is implemented as part of playground.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 60.905%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
218

Relevant Lines:
346

💛  - Coveralls


Misc: Add various pallets to pallets repo

murmur pallet
modified proxy pallet
modified scheduler pallet

@driemworks the pallets are there already, we only need to update them


fee and min amount warnings

warnings!

I think this screen should show the fee indicator too. For one, on unified they might not know if it was on chain or lightning that was paid. Plus for normal lightning receives without a channel there is a 1 sat fee too so that should be shown. Needs to be clear before and after that a fee was paid so there's no surprises.

This warning is appearing despite a channel already opened.

okay addressed the problem with the warning showing up and added more logic for whenever inbound is below request amount. also tweaked the language in the text below the qr code to warn the sender chooses in the unified case.
also added the explanation to the success screen

a 1 sat fee is taken out on the voltage side if a channel is already there. should we say that was a fee?


Reintroduce implicit_compile_deps

Why
Reverts #801
Why
ScalaPB features like extra imports and type mappers don't work with the current implementation of scala_proto_library.
This adds back implicit_compile_deps tha allows the user to add extra implicit dependencies added to all scala_proto_library targets.
How
Reverted some of the changes in #801. This should work with the latest versions of Bazel since toolchain transitions have been introduced.
This is not the ideal solution since changes to theses dependencies will trigger a rebuild of all the proto targets.
But it can be an escape hatch for people already using these features and are looking to migrate to Bazel.

There's a similar work related to proto toolchains and ability to specify custom deps via toolchain here #1104 . But it's blocked by lack of execution transitions support in older bazel versions.

Closing in favour of #1104


Ubuntu Email configuration

I successfully finished the configuration peatio application ubuntu machine and try to test the application have not successfully sending email or sms to Brazil. What I have to do to be able to test these features? Grateful.

use mailgun for smtp. mailgun.com
for sms you need a twilio account. use these settings in config.yml

Thank you ver much. What i have to do to configure Mailgun? What config files i need to update? And How?Tks again
Enviado do meu smartphone Samsung Galaxy.

-------- Mensagem original --------De: Hrishikesh Huilgolkar  Data: 10/03/2016  04:16  (GMT-03:00) Para: peatio/peatio  Cc: jcantalogo  Assunto: Re: [peatio] Ubuntu Email configuration (#503)

use mailgun for smtp. mailgun.com
for sms you need a twilio account. use these settings in config.yml

Reply to this email directly or view it on GitHub:
https://github.com/peatio/peatio/issues/503#issuecomment-194709842

I setup production and review my application.yml and email is working fine. Tks a lot.
BUT I am deploying this exchange in Brazil and Twillio does not have SMS service to brazilian numbers. There is any other service like that works with braziian SMS ?

I am using mailgun with the al the necessary settings but Peatio doesn't seem to be sending anything nor giving any errors. Can anyone point me to the code where the email verification code is and how i can see if the what the actionmail is doing?  Greatly appreciated.


Improve error when users use removed addon

After and before:
$ grun ./istioctl/cmd/istioctl manifest generate --set addonComponents.foo.enabled=true -d manifests
Error: component "foo" does not exist
$ ik manifest generate --set addonComponents.foo.enabled=true -d manifests
Error: stat manifests/charts/addons/foo: no such file or directory

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a4afcacf42a29ba9c1df506e6c365e63d0617e45
link
/test gencheck_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a4afcacf42a29ba9c1df506e6c365e63d0617e45
link
/test gencheck_istio

unit-tests_istio
a4afcacf42a29ba9c1df506e6c365e63d0617e45
link
/test unit-tests_istio

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

/retest

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
a4afcacf42a29ba9c1df506e6c365e63d0617e45
link
/test unit-tests_istio

gencheck_istio
a4afcacf42a29ba9c1df506e6c365e63d0617e45
link
/test gencheck_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a4afcacf42a29ba9c1df506e6c365e63d0617e45
link
/test gencheck_istio

unit-tests_istio
a4afcacf42a29ba9c1df506e6c365e63d0617e45
link
/test unit-tests_istio

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

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
24c9ab8874b2c75eac4b736004c289b39e9020e1
link
/test unit-tests_istio

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

/retest

In response to a cherrypick label: new pull request created: #28318


OCPBUGS-29110: Reconcile over ICSP/IDMS

During the normal reconciliation loop, this patch allow the Hypershift Operator to grab the new ICSP/IDMS and add them to the CPO, allowing the HCP pods to use the right internal registry image. The changes on the ICSP/IDMS will make the control-plane-operator pod to be restarted in order to update the OPENSHIFT_IMG_OVERRIDES env variable.
Which issue(s) this PR fixes:
Fixes #OCPBUGS-29110
Signed-off-by: Juan Manuel Parrilla Madrid<EMAIL_ADDRESS>

/test unit
/test verify

/jira refresh

/retest

/test e2e-aws

/retest

/test e2e-aws

/test e2e-aws

/lgtm

/cherry-pick release-4.16

/cherry-pick release-4.15


update pom.xml

add dependency of gitskarios-core

as the dependency has been add in the core's pom.xml, so we could delete many of this dependency