KeyError: 'request_id'

I am getting KeyError: 'request_id'
I don't know this is an issue or I am doing something wrong. If I am doing something wrong, please point me a correction.
Consumer:
class MessageConsumer(ListModelMixin, PatchModelMixin, UpdateModelMixin, DeleteModelMixin, GenericAsyncAPIConsumer):
    '''A Consumer for Notifications'''

    queryset = models.Message.objects.all()
    serializer_class = serializers.MessageSerializer

    def get_queryset(self):
        return models.Message.objects.filter(record__in=models.Record.objects.filter(user=self.request.user)).order_by('-message_date_created')

Routing:
websocket_urlpatterns = [
    path('message', consumers.MessageConsumer),
]

Serializer:
class MessageSerializer(serializers.ModelSerializer):
    '''Serializer for Notification'''
    message_date_created = serializers.DateTimeField(required = False, format="%Y-%m-%d", read_only=True)

    class Meta:
        model = models.Message
        fields = [
            'record',
            'message_title',
            'message_date_created',
            'message_status',
        ]

My request is, e.g.:
{"stream":"message","payload":{"action":"list","data":{},"request_id":"56def654-9087-45f8-9f12-7a7f6c78b04d"}}

I am receiving the following error:
WebSocket HANDSHAKING /message [<IP_ADDRESS>:56054]
WebSocket CONNECT /message [<IP_ADDRESS>:56054]
Exception inside application: 'request_id'
Traceback (most recent call last):
  File "/.../python3.8/site-packages/channels/consumer.py", line 58, in __call__
    await await_many_dispatch(
  File "/.../python3.8/site-packages/channels/utils.py", line 51, in await_many_dispatch
    await dispatch(result)
  File "/.../python3.8/site-packages/channels/consumer.py", line 73, in dispatch
    await handler(message)
  File "/.../python3.8/site-packages/channels/generic/websocket.py", line 196, in websocket_receive
    await self.receive(text_data=message["text"])
  File "/.../python3.8/site-packages/channels/generic/websocket.py", line 259, in receive
    await self.receive_json(await self.decode_json(text_data), **kwargs)
  File "/.../python3.8/site-packages/djangochannelsrestframework/consumers.py", line 158, in receive_json
    request_id = content.pop("request_id")
KeyError: 'request_id'
WebSocket DISCONNECT /message [<IP_ADDRESS>:56054]

Update:
I was using a serializer from drf json api, instead of drf. changing serializer to drf resolved the issue


Twitter API no longer gives read/write by default

As per https://developer.twitter.com/en/docs/apps/app-permissions and https://developer.twitter.com/en/docs/twitter-api/getting-started/about-twitter-api
Apps do not automatically get Elevated Access any more. You can still automatically get it by filling in a form, but it should be noted that the default keys will only have read permissions until you get Elevated Access AND set up authentication (as answered https://stackoverflow.com/questions/71439161/how-to-generate-twitter-api-access-token-and-secret-with-read-write-permission )
Thus the instructions need updating. Once that's done, the app seems to work fine though.

It would be possible to change it to OAuth2 authorization to delete tweets, but I don't know enough Go and just wanted to use it. With an elevated account it works. you have to pretend to write an app which needs OAuth1.1.


How to decode client getBlock(). txs. decodedTxRaw. signatures？

After parsing tsx, I also need to get the txHash of this transaction, but I don't see the txHash after parsing it. The signatures field is the txHash of the transaction, right? I still have a problem with my understanding, why has no one replied?

@webmaster128 please

The tx hash is simply sha256(txBytes), so you don't need to decode it at all in order to get the hash.

@chaim-g have you solved this ? i have also same question .


Make fmpq_poly.factor() return primitive factors

See https://github.com/flintlib/python-flint/pull/164#discussion_r1720795664

@GiacomoPope @Jake-Moss does either of you want to review this?
The main change here is that fmpq_factor returns primitive factors rather than monic factors e.g.:
In [7]: from flint import *

In [8]: x = fmpq_poly([0, 1])

In [9]: p = (x + fmpq(1,2))*(x - fmpq(1,4))/7

In [10]: p
Out[10]: 1/7*x^2 + 1/28*x + (-1/56)

In [11]: p.factor()
Out[11]: (1/56, [(4*x + (-1), 1), (2*x + 1, 1)])

This makes fmpq_poly consistent with fmpq_mpoly. Flint itself does not seem have an fmpq_poly_factor function so fmpq_poly.factor() uses fmpz_poly_factor and then on master it makes the factors monic. Here I've changed that to keep the factors primitive as returned by fmpz_poly_factor.
I also found some other issues:
fmpz_mod_poly.factor() crashes on the zero poly. This is documented for fmpz_mod_poly_factor:

Factorises a non-constant polynomial ...

There were inconsistent types in some places e.g. nmod_poly.leading_coefficient() returns int rather than nmod. I fixed that here.
The mpoly.factor() methods return fmpz as the multiplicity of a factor unlike other poly types that use int. I have fixed this for fmpz_mpoly and fmpq_mpoly but it might also need to be fixed for the new mpoly types in gh-164.
The mpoly types need to have a leading_coefficient method. I've added this but again it might be needed for new types in gh-164. It is also missing for some univariate types.
For fq_default and fq_default_poly there is no way to get the associated context from an instance e.g.:
In [6]: p = fq_default_ctx(163)

In [7]: x = p(1)

In [8]: x.ctx
---------------------------------------------------------------------------
AttributeError

I don't see any way to get the context, characteristic, etc from x. Same for fq_default_poly.
The modulus() has a different meaning for fq_default than for nmod and fmpz_mod. We probably need to add a .characteristic() method that could have the same meaning in all cases (at least the fmpz_mod context object could have this method). It also makes sense for fq_default_ctx to define .is_prime() just like fmpz_mod_ctx.
Also I have emphasised this a lot but let me just point out again that a carefully designed test like the test_factor_poly_mpoly test here can pick up on a lot of issues including ones that have been in the codebase for a long time. The test simultaneously tests all poly types and all mpoly types together and just covers a few simple and corner cases but it checks all types comprehensively and checks all outputs thoroughly. It also demonstrates clearly what the expected behaviour is supposed to be for each of the different types and where they are similar and where they are different. If we only write separate tests for different types or for univariate vs multivariate etc then we don't get the benefits of this.

There were inconsistent types in some places e.g. nmod_poly.leading_coefficient() returns int rather than nmod. I fixed that here.

Is there a particular reason for this? Does something depend on this behaviour? It seems like an unnecessary conversion to me, any use with another python-flint object will result in a conversation back to a fmpz or similar almost immediately.

The mpoly.factor() methods return fmpz as the multiplicity of a factor unlike other poly types that use int. I have fixed this for fmpz_mpoly and fmpq_mpoly but it might also need to be fixed for the new mpoly types in gh-164.

Yes they will, I'll add those pending the above

There were inconsistent types in some places e.g. nmod_poly.leading_coefficient() returns int rather than nmod. I fixed that here.

Is there a particular reason for this? Does something depend on this behaviour? It seems like an unnecessary conversion to me, any use with another python-flint object will result in a conversation back to a fmpz or similar almost immediately.

It definitely matters whether we return an int or an nmod because they have different behaviour. For example a common thing that you might want to do with a leading coefficient is to invert it:
In [1]: from flint import *

In [2]: lc_int = 3

In [3]: lc_nmod = nmod(3, 7)

In [4]: 1/lc_int
Out[4]: 0.3333333333333333

In [5]: 1/lc_nmod
Out[5]: 5

As for the unnecessary conversion there needs to be a conversion either way. The Flint routine returns a ulong and we need to turn that into a Python object whether that is PyLong or nmod. Cython handles it implicitly if you just return a ulong but it still needs to create an actual PyLong including allocating space on the heap for it.
One small difference actually is that CPython interns small integers (up to 255?) to avoid a heap allocation. We could do something similar with nmod if we had a context object to store the interned values for a given modulus. At least for small enough moduli it would make sense to do that when we can intern every possible value. It could be handled transparently by a context method like ctx.nmod_from_ulong().

It definitely matters whether we return an int or an nmod because they have different behaviour.

I just found a similar issue with nmod_mpoly and fmpz_mod_mpoly. The coefficients should be nmod and fmpz_mod but from gh-164 they are int and fmpz:
In [7]: ctx = fmpz_mod_mpoly_ctx(2, Ordering.lex, ["x", "y"], 11)

In [8]: x = ctx.gen(0)

In [9]: x.coeffs()
Out[9]: [1]

In [10]: type(x.coeffs()[0])
Out[10]: flint.types.fmpz.fmpz

In [11]: ctx = nmod_mpoly_ctx(2, Ordering.lex, ["x", "y"], 11)

In [12]: x = ctx.gen(0)

In [13]: type(x.coeffs()[0])
Out[13]: int

Likewise leading_coefficient needs to return nmod/fmpz_mod and also the constant term in the factor method.
Probably there are also inconsistencies in the factor_squarefree methods as well...

Sorry @Jake-Moss, you reviewed this and then I made it 3x bigger. If you are still up for reviewing then please do...
The changes here are:

The main change is that fmpq_poly returns primitive rather than monic factors which is consistent with fmpq_mpoly and SymPy and other things.
I have added tests to ensure compatibility of different types and that operations work with all types.
I have added sqrt to all scalar types so all scalar and polynomial types have sqrt.
I have added factor_squarefree to all poly types that did not have it so that all poly/mpoly types can be assumed to have it.
I have added lead_coefficient to all poly types and now all poly/mpoly types can be assumed to have it.
Various types are made more consistent and various exceptions raised are changed or made consistent. A bigger discussion needs to be has about exception types.
I've added docstrings for some methods that did not have them.

What I have not fixed here is the type pf the coefficients for nmod_poly and fmpz_mod_poly which are int and fmpz when they should be nmod and fmpz_mod. That is something that needs to be fixed before next release,

No worries, will do tonight

There were inconsistent types in some places e.g. nmod_poly.leading_coefficient() returns int rather than nmod. I fixed that here.

Is there a particular reason for this? Does something depend on this behaviour? It seems like an unnecessary conversion to me, any use with another python-flint object will result in a conversation back to a fmpz or similar almost immediately.

It definitely matters whether we return an int or an nmod because they have different behaviour. For example a common thing that you might want to do with a leading coefficient is to invert it:
In [1]: from flint import *

In [2]: lc_int = 3

In [3]: lc_nmod = nmod(3, 7)

In [4]: 1/lc_int
Out[4]: 0.3333333333333333

In [5]: 1/lc_nmod
Out[5]: 5

As for the unnecessary conversion there needs to be a conversion either way. The Flint routine returns a ulong and we need to turn that into a Python object whether that is PyLong or nmod. Cython handles it implicitly if you just return a ulong but it still needs to create an actual PyLong including allocating space on the heap for it.
One small difference actually is that CPython interns small integers (up to 255?) to avoid a heap allocation. We could do something similar with nmod if we had a context object to store the interned values for a given modulus. At least for small enough moduli it would make sense to do that when we can intern every possible value. It could be handled transparently by a context method like ctx.nmod_from_ulong().

I'm sorry I quoted the wrong section of your message. The above makes complete sense but I was look at this change where the exponent is created as an fmpz, then immediately converted to an int.

The mpoly.factor() methods return fmpz as the multiplicity of a factor unlike other poly types that use int. I have fixed this for fmpz_mpoly and fmpq_mpoly but it might also need to be fixed for the new mpoly types in https://github.com/flintlib/python-flint/pull/164.

https://github.com/flintlib/python-flint/pull/189/files#diff-b5a069a771eae54c8982ca6dae36c6a291b19e7bb79effc990091b0f2304fdc4R959-R963

I was look at this change where the exponent is created as an fmpz, then immediately converted to an int.

The mpoly.factor() methods return fmpz as the multiplicity of a factor unlike other poly types that use int.

It's important that the types are consistent and we don't get into a situation where we don't know what they are supposed to be any more. If it is supposed to be an int then it needs to be an int. I recently ran into a situation where having an fmpz in a place that would usually be an int caused a bug:
https://github.com/python/cpython/issues/120950
I'm a bit surprised that an fmpz is used at all for the multiplicity in the factor struct. It suggests great ambition about the sizes of polynomials that can be factorised.
The nmod_poly code just casts the exponent to an integer which in principle would be incorrect if the multiplicity is greater than 2^(FLINT_BITS-2):
https://github.com/flintlib/python-flint/blob/464a27650b3eb6df06e9d3bb660cdc494b9efcf5/src/flint/types/nmod_poly.pyx#L640-L641
On a 64 bit system it is inconceivable that an nmod_poly could have a factor of multiplicity greater than 2^62. On a 32 bit system could it have greater than 2^30? Maybe at a push for GF(p) with very small p...
I suppose you can have something like:
In [14]: from flint import *

In [15]: ctx = fmpz_mpoly_ctx(2, Ordering.lex, ["x", "y"])

In [16]: x, y = ctx.gens()

In [17]: ((x*y)**(2**64)).factor()
Out[17]: (1, [(x,<PHONE_NUMBER>3709551616), (y,<PHONE_NUMBER>3709551616)])

It's hard to imagine that a nontrivial polynomial could be factorised and have a multiplicity that does not fit slong/ulong.
For dense polynomials it is perhaps impossible:
In [18]: p = nmod_poly(0, 11)

In [19]: p[2**32] = 1
Flint exception (General error):
    Unable to allocate memory<PHONE_NUMBER>6).
Aborted (core dumped)

(Ideally we would have a way of preventing that core dump...)
In any case though we need the types to be consistent. The multiplicity is never going to be large enough to need to be anything other than an int and all other types already return int. Factorisation is an extremely expensive operation so I doubt very much that the cost of converting the exponents is significant.

In any case though we need the types to be consistent. The multiplicity is never going to be large enough to need to be anything other than an int and all other types already return int. Factorisation is an extremely expensive operation so I doubt very much that the cost of converting the exponents is significant.

Thank make sense, more than reasonable to have that.

I see that gh-190 is waiting on this so I'll merge once tests pass.


Fix deserialization of empty arrays

When a document has an empty array field, the REST api returns an empty object as the "arrayValue" for that field which causes the following error:
TypeError: Cannot read property 'map' of undefined

  37 |   }
  38 |   if ("arrayValue" in value) {
> 39 |     return value.arrayValue.values.map((value) => decode(value));
     |                                    ^
  40 |   }
  41 |   if ("mapValue" in value) {
  42 |     return deserialize(value.mapValue.fields);

  at decode (deserializer.js:39:36)
  at map (deserializer.js:4:20)
      at Array.map (<anonymous>)
  at deserialize (deserializer.js:3:28)
  at Object.<anonymous> (deserializer.test.js:68:7)

Thank you!
I noticed that an empty mapValue will cause a similar problem. Let me fix that as well and then publish to npm.

That was fast!
I'm happy to help.


[BUG] title

Describe the bug
The function gdk.PixbufNewFromFile sometimes fails to read JPG files with the error:
"Couldn’t recognize the image file format for file “fail.jpg”

The failed JPGs can be opened in an image viewer, like Pix on Linux, without a problem, and GitHub has no problem displaying it.
The images are downloaded with youtube-dl, and 95% of the images I download with youtube-dl works just fine. It seems like it is only thumbnails from certain channels that pretty much always fails. This leads me to think that there is some property set in the program that creates the thumbnails, that is set by these channels, that is not supported by the function gdk.PixbufNewFromFile.

To Reproduce
Steps to reproduce the behavior:
Code can be found here : https://github.com/Hultan/jpg-fail
Expected behavior
No fail
Error messages
"Couldn’t recognize the image file format for file “fail.jpg”
Environment:
gtk3 version: '3.24.23-1'
go version: '1.15.1-1'
os: 'Manjaro (Cinnamon)'

Hi,
WebP format is not handled by gtk3 (at this time...)

XViewer can't open it
Pix can do it
XnViewMp do it too

Just convert it to png or jpg.
Fichier
Nom de fichier	fail.jpg
Répertoire	~/jpg-fail-master
Taille de fichier	14.48 Kio (14 824)
Date/Heure de création	09/09/2020 - 18:50:11 CEST
Date/Heure de modification	09/09/2020 - 08:58:32 CEST
Date/heure d'accès	09/09/2020 - 18:50:19 CEST
Note	0
Libellé de couleur	0

Image
Format	WebP
Largeur	336
Hauteur	188
Nb bits	24
Modèle couleur	RGB
Taille d'impression	11.85x6.63 cm, 4.67x2.61 pouces
Compression	Aucune
Nb images/pages	1
Origine	Haut gauche

Oh, I thought that I had converted the WebP to a JPG, but apparently just by name I guess?

if you just change extension, its not a conversion ...
XnViewMp or Gimp could convert in multiple formats

if you just change extension, its not a conversion ...
I understand that, what I did was that I wrote a program to convert it myself, but obviously that is not working as expected. I might need to use convert-webp-to.jpginstead...I will look into that, since I don't know anything about image formats myself. I don't know why I thought that doing it myself was a good idea :-)
Thanks...

yes it's a good idea to learn how to do it, some time ago i did some research on webp and golang on github, like i said i found some tips but not enough time to do something that works properly, if you can find a solution, I would be happy to be notified. :)


Clicking on the "Data library" link on the create modal removes the number on the shared datasets

That's the number that comes from the visualization endpoint, so I guess this is a bug from the API. That means backend guys should take a look :S.
cc @juanignaciosl or @Kartones

I'll take a look, but right now...

A request to remote data, like:
http://theteam.localhost.lan:3000/u/cartofante/api/v1/viz.json?tag_name=&q=&page=1&type=remote
Returns:
{
  visualizations: […],
  total_entries: 66,
  total_user_entries: 62,
  total_likes: 0,
  total_shared: 0
}

Those last figures are the numbers those appear in the "new dashboard" subheader.

Ok, and what does this have to do do with the Data Library? due to being "remote tables" the counters dissapear?

I'm not sure if this is a backend issue. This is the relevant data of the first request ('datasets' one):
total_entries: 24
total_likes: 0
total_shared: 24
total_user_entries: 274

Clicking at Data library "tab" (no need to go to the modal) returns this:
total_entries: 8
total_likes: 0
total_shared: 24
total_user_entries: 274

As you see, in spite of having total_shared: 24 the number disappears.

Uhm, you are right...
Overloaded!

Wow, not that fast. Clicking either "Datasets" or "Data Library" trigger 2 requests each:
Click datasets:

https://team.cartodb.com/u/juanignaciosl/api/v1/viz/?tag_name=&q=&page=1&type=table&exclude_shared=true&per_page=20&tags=&only_shared=false&category=&locked=false&only_liked=false&order=updated_at
https://team.cartodb.com/u/juanignaciosl/api/v1/viz/?tag_name=&q=&page=1&type=table&exclude_shared=true&per_page=1&locked=true&only_shared=false&liked=false&tag=undefined

Click Data Library:
3. https://team.cartodb.com/u/juanignaciosl/api/v1/viz/?tag_name=&q=&page=1&type=remote&exclude_shared=true&per_page=20&tags=&only_shared=false&category=undefined&locked=false&only_liked=false&order=updated_at
4. https://team.cartodb.com/u/juanignaciosl/api/v1/viz/?tag_name=&q=&page=1&type=table&exclude_shared=true&per_page=1&locked=true&only_shared=false&liked=false&tag=undefined
Number 3 is the one that have 0 shared. I think first loads "unlocked" and second loads "locked". But the problem might be that category=undefined. Take a look. If you confirm request is ok and it's a backend issue feel free to assign it to me.

I will take a look in my next available gap, don't worry.

I've removed category parameter and seems like request is still returning 0, so sorry @juanignaciosl, seems like it is a backend bug, but if you find it is not, let me know please.


fix: erase bug and virtualised list err

The type of this PR is: Bugfix

This PR resolves SWA-324
Description
This PR fixes 2 things on the artist auto-suggest input in submission:

the error message that does not disappear
the virtualised list error

https://user-images.githubusercontent.com/42584148/160437269-c72c5acf-9941-4b69-ba75-b76f43df2fec.mp4
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes
The changes apply to all users.
iOS user-facing changes

Android user-facing changes

Dev changes

🎉


Locale Sub-routing doesn't work

Hi,
I've got this configuration on my next js with next-translate:
Inside next.config.js
const nextTranslate = require('next-translate');

module.exports = nextTranslate({
   webpack: (config, { isServer, webpack }) => {
      return config;
   },
   target: 'serverless'
});

And inside i18n.json
{
   "locales": [
      "en",
      "gr"
   ],
   "defaultLocale": "gr",
   "pages": {
      "*": [
         "common"
      ],
      "/": [
         "home"
      ],
      "/about": [
         "about"
      ]
   }
}

The problem I'm facing is when switching the locale to en all blog posts are broken e.g
http://localhost:3000/blog/test <- Works
http://localhost:3000/en/blog/test <- 404 not found
Any ideas what I should do to make the sub-routing work correctly?
Thanks

Would be great to provide a reproducible repo in order to help you. With the provided info it should work well...

@aralroca I have seen the same thing happening but only on pages/[slug].tsx pages that also provide a getStaticPaths function. In my case, the getStaticPaths function was extremely simple (reading a list of allowed pages from a config with fallback: false to trigger 404 pages for everything else).
As soon as I removed the function, the pages worked again including translations. However you do loose static page generation then, if you don't provide the function.
I'm a bit in a rush to finish a project so I can't provide a full repository to demonstrate right now, maybe I'll come back to this in a few days.

Yeap confirming that's same issue as reported by @alexfrancavilla . In my case removing getStaticPaths breaks the application

Here's a reproducible repo with the issue:
Steps to reproduce

Clone repo and run npm i and npm run dev
Click on Change to English
Click on Markdown Primer 2020-01-02
Redirects to 404

Expected behaviour:

Redirect to the same md file without the /en inside the URL

@aralroca Any updates on this issue?

Having the same issue. If getStaticPaths is present dynamic routes (e.g. /de/post/[...slug]) do not work with fallback: false. They do work with fallback: true but there is a momentary flash of non-translated content (I think this may be #575) however the translations do load in after that.

@ykyp You need to add the locales to the getStaticPaths:
export async function getStaticPaths({ locales }) {
   const paths = getAllPostIds(locales);
   return {
      paths,
      fallback: false,
   };
}

and your function:
export function getAllPostIds(locales) {
   const fileNames = fs.readdirSync(postsDirectory);

   return fileNames.flatMap((fileName) => {
      const params = {
         id: fileName.replace(/\.md$/, ""),
      }

      return [
         { params },
         ...locales.map(locale => ({ params, locale })),
      ]
   });
}

@ykyp I close the issue, is re

Thanks a lot @aralroca confirming this has now fixed my issue 🙏


Support for HSMs

I'm just opening this issue for discussion of whether the current algorithms preclude support for signing by an HSM (hardware security module) or other types of key vaults.
Some questions that come to mind:

Is it reasonable to assume that the HSM will compute the required generator points, or would they need to be submitted by the caller?
Would there need to be a limit on the number of signed messages?
Should the messages be submitted as scalars in order to support different message encodings?
Do specific ciphersuites need to be explicitly supported (and perhaps included in the key metadata to prevent reuse across ciphersuites)?
What would support for blind signing look like?

And in general, are there examples of similar APIs for comparison?

The problem with HSMs is you can only multiply a point by the secret value and they only return the X coordinate which means you have to figure out which Y coordinate to use otherwise it breaks. Until they support returning the full point I’m not sure how the HSM can help. Software enclaves can do much better so perhaps that should be the focus first


New code:

New code that improves the usability of this module given that now you do not need to set the delegate and the items to play in the same declaration so that you avoid creating multiple instances of the player in case you have to add items to the player programatically.
Thanks!

Unfortunately this does not follow the (API design guidelines)[https://swift.org/documentation/api-design-guidelines/]


Have router make two separate attempts to reconnect to current partition.

Codecov Report

Merging #1276 into master will not impact coverage by -0.57%.

@@            Coverage Diff             @@
##           master    #1276      +/-   ##
==========================================
- Coverage    69.5%   68.94%   -0.57%
==========================================
  Files         169      169
  Lines       26691    26694       +3
  Branches     3346     3347       +1
==========================================
- Hits        18551    18403     -148
- Misses       6727     6880     +153
+ Partials     1413     1411       -2

Impacted Files
Coverage Δ

include/openthread-types.h
100% <ø> (ø)
:white_check_mark:

src/cli/cli.cpp
39.18% <ø> (ø)
:white_check_mark:

src/core/thread/mle_router.cpp
79.39% <100%> (-0.48%)
:x:

src/core/thread/mle.cpp
80.22% <100%> (-0.18%)
:x:

src/core/thread/key_manager.hpp
80% <ø> (-20%)
:x:

src/core/net/ip6.cpp
56.72% <ø> (-13.21%)
:x:

src/core/openthread.cpp
35.61% <ø> (-11.42%)
:x:

src/core/net/icmp6.cpp
84% <ø> (-4%)
:x:

src/core/net/netif.cpp
58.76% <ø> (-3.61%)
:x:

src/core/net/ip6_mpl.cpp
91.6% <ø> (-2.3%)
:x:

... and 11 more

Continue to review full report at Codecov.

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


Editor will crash when last project cannot be loaded

Describe the bug
Users can put the editor in an unusable state by removing the last project they worked on.
To Reproduce
Steps to reproduce the behavior:

Open the Editor
Create a new Project
Exit and save the new Project
Rename the Directory containing the project or delete it
The editor will keep crashing when the user tries to open it back up

This seems to apply to the current master only, the latest release version is not affected.


Memory leak in v3.1.0

node-fetch: 3.1.0
This code produces memory leak warning:
import fetch from "node-fetch";
import { Agent } from "http";

const agent = new Agent( {
    "maxSockets": 1,
} );

for ( let n = 0; n < 30; n++ ) {
    fetch( "http://example.com/", { agent } ).then( res => console.log( res.status ) );
}

200
200
(node:3152) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:465:17)
    at Socket.prependListener (node:events:501:14)
    at ClientRequest.<anonymous> (file:///d:/downloads/111/node_modules/node-fetch/src/index.js:319:10)
    at ClientRequest.emit (node:events:390:28)
    at tickOnSocket (node:_http_client:757:7)
    at onSocketNT (node:_http_client:820:5)
    at processTicksAndRejections (node:internal/process/task_queues:84:21)
200
(node:3152) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 data listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit
    at _addListener (node:events:465:17)
    at Socket.addListener (node:events:487:10)
    at Socket.Readable.on (node:internal/streams/readable:899:35)
    at ClientRequest.<anonymous> (file:///d:/downloads/111/node_modules/node-fetch/src/index.js:325:10)
    at ClientRequest.emit (node:events:390:28)
    at tickOnSocket (node:_http_client:757:7)
    at onSocketNT (node:_http_client:820:5)
    at processTicksAndRejections (node:internal/process/task_queues:84:21)
200
200
200
200
200
200
200

That doesn't really look like a memory leak. You're calling fetch multiple times without await so multiple fetch calls are running at the same time. This means it doesn't have a chance to close off the last connection before the next one opens.
The following uses await to ensure only one is called at a time.
import fetch from "node-fetch";
import { Agent } from "http";

const agent = new Agent( {
    "maxSockets": 1,
});

async function* asyncGenerator() {
  let i = 0;
  while (i < 30) {
    yield i++;
  }
}

for await (const i of asyncGenerator()) {
    await fetch( "http://example.com/", { agent } ).then( res => console.log( res.status ) );
}

Need to increase / decrease number of maxListeners before set event listener.
Or review logic and set listener only after request will be sent, not after it was queued.

@OmgImAlexis even in your example you are fetching multiple request simultaneously. getting the response headers can be very fast... but getting the actually body can be slower...
for await (const i of asyncGenerator()) {
-   await fetch( "http://example.com/", { agent } ).then( res => console.log( res.status ) );
+   const res = await fetch( "http://example.com/", { agent } )
+   console.log( res.status )
+   await res.arrayBuffer();
}

related to: https://github.com/node-fetch/node-fetch/issues/1446

Probably fixed by #1295.


Exception usage for one use case

The issue related to exception hierarchy:
ServiceResponseException tends to catch the exception without status code and the response format is not quite right.
DecodeException tends to catch exception with status code, get the right response format but cannot parse or understand by client.
One thing needs to discuss is what if I got the half of the response (e.g. network issue) with status code of 200. Which exception do I need to throw in this case?

Currently, we agreed to throw ServiceResponseException, as we prefer to let ServiceResponseException handle the case of partial response. I will go ahead close the issue. Please feel free to reopen this if you have additional questions.


use sd-notify to signal readyness

Provide a way to signal nsncd has successfully started up, and is ready to accept connections.
A more reliable way would be to only signal this once at least one worker has started up, but this is good enough (tm) for now.
This can be used in a systemd.service file with Type=notify.

@flokli can you tell us a bit more about why you want to add this? What can you do with this change that you can't without?
I know this isn't the change you're proposing, but just FYI if you're doing systemd things, we tried using socket activation once and hit a deadlock: systemd only activates one service at a time, so if the activation of another service tries to do a NSS lookup, that will make systemd enqueue the activation of nsncd, but won't actually try to start it until the other service completes its own startup (which then won't proceed since it's waiting for nsncd to respond to the lookup). #10

Actually, from #9:

I think it's also best to avoid readiness notification, because I don't totally understand whether the time between starting nsncd and getting the readiness notification counts as a running job and what that means for the job queue. We should just make nsncd a Type=simple service. This might cause race conditions with services that require nsncd (i.e., services with a foreign libc) that try to say Requires=nsncd.service, because they don't know when the NSCD socket has actually been created. If that causes a problem in practice for anyone, we should come back and add readiness notification and test it a bit more.

So...it sounds like @geofft considered that the change you're proposing might be good, but also maybe could cause similar job queue problems? I think we just never tested this out and if you want it it might be good but we should test carefully.
I'll wait to hear more about your use case first though.

So...it sounds like @geofft considered that the change you're proposing might be good, but also maybe could cause similar job queue problems? I think we just never tested this out and if you want it it might be good but we should test carefully.

I'll wait to hear more about your use case first though.

I plan using nsncd on NixOS, as a drop-in replacement for glibc-nscd (pending https://github.com/twosigma/nsncd/issues/37). In NixOS, as there's no common /usr/lib, glibc code paths in client applications don't know how to load any "non-glibc" nss modules listed in /etc/nsswitch.conf, so we absolutely need to do the lookups via the nscd socket - otherwise lookups provided by external NSS modules fail.
We put nscd.service with a RequiredBy=nss-lookup.target nss-user-lookup.target (and even think about pulling it in in early boot), so systemd waits until nscd is ready before starting services that have After= set to one of these targets.
glibc-nscd does fork when it's ready (Type=forking), but nsncd is currently using Type=simple, so systemd has no idea when the service is ready, starts it as soon as it started the nsncd process, while that one might not have created the unix socket, to accept connections. So downstream clients fail to connect.
This change lets nsncd signal readyness via https://docs.rs/sd-notify/latest/sd_notify/ once the nscd unix socket has been created, which is more precise.
I don't see how this could cause any deadlocking of service startup.
The issue @geofft mentioned was probably caused by the socket activation, and systemd only activating one service at a time (?) [^1]
[^1]: even though that smells like a bug that should be investigated, but let's leave it out of scope for here.

With https://github.com/twosigma/nsncd/pull/36 merged, I rebased this.
It now only adds the sd-notify to Cargo.lock, and all the upgrade noise is gone.

Codecov Report
Base: 30.18% // Head: 30.04% // Decreases project coverage by -0.14% :warning:

Coverage data is based on head (9fa26c5) compared to base (1d0bf2b).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
- Coverage   30.18%   30.04%   -0.15%
==========================================
  Files           5        5
  Lines         212      213       +1
==========================================
  Hits           64       64
- Misses        148      149       +1

Impacted Files
Coverage Δ

src/main.rs
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

I'm gonna merge this by itself and do an internal release to make sure nothing exciting happens. (The changes are de minimis so we don't need to bother with a CLA for this particular PR.)


Yard not working on Ruby 1.9.3

Steps to reproduce

Run the following command: yard -v on Ruby 1.9.3

Actual Output
jbowes@jbowes-mac ~/projects/nai-utils$ yard doc --no-private --debug
/Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/lib/yard/globals.rb:21:in log': /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/lib/yard/logging.rb:13: invalid multibyte char (US-ASCII) (SyntaxError) /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/lib/yard/logging.rb:13: invalid multibyte char (US-ASCII) /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/lib/yard/logging.rb:13: syntax error, unexpected $end, expecting keyword_end     PROGRESS_INDICATORS = %w(⣷ ⣯ ⣟ ⡿ ⢿ ⣻ ⣽ ⣾)                                ^ 	from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/lib/yard/cli/command_parser.rb:57:in initialize'
from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/lib/yard/cli/command_parser.rb:54:in new' 	from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/lib/yard/cli/command_parser.rb:54:in run'
from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/gems/yard-0.9.9/bin/yard:13:in <top (required)>' 	from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/bin/yard:23:in load'
from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/bin/yard:23:in <main>' 	from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/bin/ruby_executable_hooks:15:in eval'
from /Users/jbowes/.rvm/gems/ruby-1.9.3-p551@nai-utils/bin/ruby_executable_hooks:15:in `'
Expected Output
Expect yard to work
Environment details:

OS: CentOS 7 and MacOS 10.12.4
Ruby version (ruby -v): ruby 1.9.3p551 (2014-11-13 revision 48407) [x86_64-darwin16.0.0]
YARD version (yard -v): 0.9.9

Fix
In lib/yard/logging.rb, the encoding line must be the first line

0_0 Somebody still uses 1.9.3 in production?..

The contrib guide specifies "Ruby versions (1.8+)"

Yes, I understand your concern (I am not a contributor, unforturnately, just interested spectator). Of course, all explictily specified Ruby versions should be supported. I am just wondering whether there are still real cases for 1.9 support except for abandoned gems docs.

This is easy enough to fix.

You can go ahead and close Pull Request 1090 since you fixed it.

YARD's policy on 1.8 was defined in v0.9.6, by the way:

Removed official support for Ruby 1.x (1.8/1.9). YARD can still be installed in these versions, but support is not guaranteed. Simple bug fixes may still be considered via pull request only. Issues without code will be automatically closed.


Fix Unzip Failure Due to Already Existing

I am not a C# Developer
@esoccoli ran into an issue with @ethanf108 's weird clock. The boot would fail because it would try to unzip and couldn't because it already exists.

You seem to have included the changes from this pr in your other pr here. With that being the case this PR seems redundant and can likely be closed.


Search api : attributesToCrop , does not reduce data {feature? suggestion?]

I use cropping feature in elasticsearch or meili to reduce the content returned in a field. The intent of client-app is explicitly to crop field(s), and get only cropped data, without introducing new json structure _formatted. Client-app that is consuming the json can treat cropped-result same as non-cropped, also data transfer over network is reduced.
Search url: http://<IP_ADDRESS>:7700/indexes/movies/search?q=snowman&attributesToCrop=*&cropLength=100
Expected
{
    "id": "13382",
     //cropped non id fiends
},

Actual:
If the overview field is in MBs, incurs unnecessary network cost.
{
         "id": "13382",
         "title": "Rudolph the Red-Nosed Reindeer",
         "poster": "https://image.tmdb.org/t/p/w1280/r0vtyGOwTLZg4pgtbjlMGke7Qo1.jpg",
        //If the overview field is in MBs, incurs unnecessary network cost.
         "overview": "Sam the snowman tells us the story of a young red-nosed reindeer who, after being ousted from the reindeer games because of his glowing nose, teams up with Hermey, an elf who wants to be a dentist, and Yukon Cornelius, the prospector. They run into the Abominable Snowman and find a whole island of misfit toys. Rudoph vows to see if he can get Santa to help the toys, and he goes back to the North Pole on Christmas Eve. But Santa's sleigh is fogged in. But when Santa looks over Rudolph, he gets a very bright idea...",
         "release_date": -160012800,
         "_formatted":          {
            "id": "13382",
            "title": "Rudolph the Red-Nosed Reindeer",
            "poster": "https://image.tmdb.org/t/p/w1280/r0vtyGOwTLZg4pgtbjlMGke7Qo1.jpg",
            "overview": "Sam the snowman tells us the story of a young red-nosed reindeer who, after being ousted from the reindeer games",
            "release_date": -160012800
         }
},

Middle ground
Perhaps adding a flag to provide _original content, if flag present, include original content, if flag is not present, just return cropped content.
search?q=snowman&attributesToCrop=*&cropLength=100&includeOriginal
{
    "id": "13382",
    //cropped non id fiends
    ,
    "_original" : { //original **if explicitly requested**. This makes sure both user intents are satisfied.
            "id" :"13382",
            //un cropped fields
     }
},

Having json structure same for cropped and uncropped, ensures UI clients does not need code change,

Hello, you can already do it by mixing the attributes attributesToRetrieve and attributesToCrop.

attributesToRetrieve: Will filter the attributes you want to retrieve in original and formated attributes. By default the value is a wildcard for all attributes.
attributesToCrop: Will select which attributes you want to crop. If this attribute is not present on attributesToRetrieve, o the field will be added only on the formated response.

So what you can do is:
curl "http://localhost:7700/indexes/movies/search?q=c&limit=4&attributesToRetrieve=id&attributesToCrop=overview&cropLength=20"

and the response will be:
{
  "hits": [
    {
      "id": "100402",
      "_formatted": {
        "overview": "After the cataclysmic events in"
      }
    },
    {
      "id": "16281",
      "_formatted": {
        "overview": "Inspired by the E.C. comics of the 1950s"
      }
    },
    {
      "id": "9584",
      "_formatted": {
        "overview": "form a mile long 'convoy' in support of"
      }
    },
    {
      "id": "23730",
      "_formatted": {
        "overview": "murders in New York City seems to point"
      }
    }
  ],
  "offset": 0,
  "limit": 4,
  "nbHits": 16520,
  "exhaustiveNbHits": false,
  "processingTimeMs": 10,
  "query": "c"
}


[pt-br] Update navbar items for vanilla Docsy

Update navbar items in the Brazilian Portuguese localization so layout renders correctly using vanilla Docsy.
Changes in the same vein as #45869 and #46192 in order to allow #46409 to move forward.
/cc @edsoncelio @mboukhalfa @sftim
/area web-development

Thanks @stormqueen1990!
/approve


Set Host header automatically in reverse proxy mode

Follow-up to #1581 using a slightly different technique to override the header. I don't want to rely on addons for this, this should always happen.

I don't believe this is working as intended:
>>> mitmdump -d -R https://www.example.com/
>>> curl http://localhost:8080/

>> GET https://www.example.com/
   Host: localhost.com:8080
   ...

It looks like self.mode is actually evaluating as "transparent" at this point, though I'm not immediately sure why. Thus the new code block doesn't execute, and the old Host behavior persists.

Uh - Looks like I screwed up when testing this, thanks for catching that! We need to check self.config.options.mode here, the HTTP Layer mode is more restricted and can only be transparent/regular/upstream. I'll fix it ASAP.

I went ahead and made the PR. Looks like everything behaves as expected after that small change :)


[Bug] Application not ready with status code 404

Describe the bug
When attempting to run the demo (both Redis and RabbitMQ), Leek starts up successfully but once the agent starts, it continuously reports the following error:
app_1        | WARNING:leek.agent.consumer:(main) Application not ready with status code 404
app_1        | WARNING:leek.agent.consumer:(main) b'{"error": {"code": "404001", "message": "Application does not exist", "reason": "Index not yet created"}}\n'

To Reproduce
Steps to reproduce the behavior:

curl -sSL https://raw.githubusercontent.com/kodless/leek/master/demo/docker-compose-redis-no-auth.yml > docker-compose.yml
docker-compose up

Expected behavior
I think the agent should start successfully and not raise 404. When I tried using the app, it did not seem like it was functional; no workers, queues, or tasks were reported.
Logs
Attaching to es01, leek_mq_1, leek_worker_1, leek_publisher_1, leek_app_1
mq_1         | Configuring logger redirection
mq_1         |
mq_1         |   ##  ##      RabbitMQ 3.8.9
mq_1         |   ##  ##
mq_1         |   ##########  Copyright (c) 2007-2020 VMware, Inc. or its affiliates.
mq_1         |   ######  ##
mq_1         |   ##########  Licensed under the MPL 2.0. Website: https://rabbitmq.com
mq_1         |
mq_1         |   Doc guides: https://rabbitmq.com/documentation.html
mq_1         |   Support:    https://rabbitmq.com/contact.html
mq_1         |   Tutorials:  https://rabbitmq.com/getstarted.html
mq_1         |   Monitoring: https://rabbitmq.com/monitoring.html
mq_1         |
mq_1         |   Logs: <stdout>
mq_1         |
mq_1         |   Config file(s): /etc/rabbitmq/rabbitmq.conf
mq_1         |
mq_1         |   Starting broker... completed with 4 plugins.
app_1        | 2021-11-03 03:37:17,923 INFO Set uid to user 0 succeeded
app_1        | 2021-11-03 03:37:17,942 INFO RPC interface 'supervisor' initialized
app_1        | 2021-11-03 03:37:17,942 CRIT Server 'unix_http_server' running without any HTTP authentication checking
app_1        | 2021-11-03 03:37:17,945 INFO supervisord started with pid 1
app_1        | 2021-11-03 03:37:18,949 INFO spawned: 'bootstrap' with pid 9
app_1        | 2021-11-03 03:37:19,951 INFO success: bootstrap entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
app_1        |
app_1        | 8 8888         8<PHONE_NUMBER>   8<PHONE_NUMBER>   8 8888     ,88'
app_1        | 8 8888         8 8888         8 8888         8 8888    ,88'
app_1        | 8 8888         8 8888         8 8888         8 8888   ,88'
app_1        | 8 8888         8 8888         8 8888         8 8888  ,88'
app_1        | 8 8888         8<PHONE_NUMBER>88 8<PHONE_NUMBER>88 8 8888 ,88'
app_1        | 8 8888         8 8888         8 8888         8 8888 88'
app_1        | 8 8888         8 8888         8 8888         8 888888<
app_1        | 8 8888         8 8888         8 8888         8 8888 `Y8.
app_1        | 8 8888         8 8888         8 8888         8 8888   `Y8.
app_1        | 8<PHONE_NUMBER>88 8<PHONE_NUMBER>88 8<PHONE_NUMBER>88 8 8888     `Y8.
app_1        |
app_1        |
app_1        | |#|     Leek Celery Monitoring Tool                               |#|
app_1        | |#|     Versions: 0.5.2                                  |#|
app_1        | |#|     Release date: 2021/08/29 12:08:53                         |#|
app_1        | |#|     Codename: Fennec                                          |#|
app_1        | |#|     Repository: https://github.com/kodless/leek               |#|
app_1        | |#|     Homepage: https://tryleek.com                             |#|
app_1        | |#|     Documentation: https://tryleek.com/docs/introduction/leek |#|
app_1        |
app_1        | Author: Hamza Adami<EMAIL_ADDRESS>app_1        | Follow me on Github: https://github.com/kodless
app_1        | Buy me a coffee: https://buymeacoffee.com/fennec
app_1        |
app_1        |
app_1        | SERVICE     STATUS      URL
app_1        | =======     ------      ---
app_1        | - API       ENABLED    http://<IP_ADDRESS>:5000
app_1        | - WEB       ENABLED    http://<IP_ADDRESS>:8000
app_1        | - AGENT     ENABLED    -
app_1        |
app_1        | INFO:elasticsearch:PUT http://es01:9200/_scripts/task-merge [status:200 request:0.079s]
app_1        | INFO:elasticsearch:PUT http://es01:9200/_scripts/worker-merge [status:200 request:0.134s]
app_1        | 2021-11-03 03:37:25,858 INFO spawned: 'api' with pid 12
app_1        | [2021-11-03 03:37:26 +0000] [12] [INFO] Starting gunicorn 20.0.4
app_1        | [2021-11-03 03:37:26 +0000] [12] [INFO] Listening at: http://<IP_ADDRESS>:5000 (12)
app_1        | [2021-11-03 03:37:26 +0000] [12] [INFO] Using worker: gevent
app_1        | [2021-11-03 03:37:26 +0000] [23] [INFO] Booting worker with pid: 23
app_1        | [2021-11-03 03:37:26 +0000] [24] [INFO] Booting worker with pid: 24
app_1        | 2021-11-03 03:37:27,360 INFO success: api entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
app_1        | api: started
app_1        | Connected to elastic searchConnected to elastic search
app_1        |
app_1        | 2021-11-03 03:37:27,655 INFO spawned: 'agent' with pid 26
app_1        | 2021-11-03 03:37:27,655 INFO success: agent entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
app_1        | agent: started
app_1        | 2021-11-03 03:37:27,851 INFO spawned: 'web' with pid 38
app_1        | INFO:__main__:(main) Loading subscriptions...
app_1        | INFO:__main__:(main) Found 1 subscriptions!
app_1        | INFO:__main__:(main) Building consumers...
app_1        | INFO:leek.agent.consumer:(main) Building consumer [Subscription=leek-prod, Prefetch=1000, Pool=2] ...
app_1        | INFO:leek.agent.consumer:(main) Ensure connection to the broker amqp://admin:**@mq:5672//...
app_1        | INFO:leek.agent.consumer:(main) Broker is up!
app_1        | INFO:leek.agent.consumer:(main) Declaring Exchange/Queue and binding them...
app_1        | INFO:leek.agent.consumer:(main) Exchange/Queue declared and bound!
app_1        | INFO:leek.agent.consumer:(main) Ensure connection to the API http://<IP_ADDRESS>:5000...
app_1        | INFO:leek.agent.consumer:(main) API is up!
app_1        | INFO:__main__:(main) Consumers built...
app_1        | INFO:__main__:(main) Starting Leek Agent...
app_1        | 2021-11-03 03:37:29,184 INFO success: web entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
app_1        | web: started
app_1        | 2021-11-03 03:37:29,239 INFO exited: bootstrap (exit status 0; expected)
app_1        | WARNING:leek.agent.consumer:(main) Application not ready with status code 404
app_1        | WARNING:leek.agent.consumer:(main) b'{"error": {"code": "404001", "message": "Application does not exist", "reason": "Index not yet created"}}\n'
app_1        | WARNING:leek.agent.consumer:(main) Application not ready with status code 404
app_1        | WARNING:leek.agent.consumer:(main) b'{"error": {"code": "404001", "message": "Application does not exist", "reason": "Index not yet created"}}\n'
app_1        | WARNING:leek.agent.consumer:(main) Application not ready with status code 404
app_1        | WARNING:leek.agent.consumer:(main) b'{"error": {"code": "404001", "message": "Application does not exist", "reason": "Index not yet created"}}\n'
app_1        | WARNING:leek.agent.consumer:(main) Application not ready with status code 404
app_1        | WARNING:leek.agent.consumer:(main) b'{"error": {"code": "404001", "message": "Application does not exist", "reason": "Index not yet created"}}\n'
app_1        | WARNING:leek.agent.consumer:(main) Application not ready with status code 404
app_1        | WARNING:leek.agent.consumer:(main) b'{"error": {"code": "404001", "message": "Application does not exist", "reason": "Index not yet created"}}\n'
app_1        | WARNING:leek.agent.consumer:(main) Application not ready with status code 404

Desktop (please complete the following information):

OS: macOS
Version: 0.5.2

@chasleslr This is not a bug, this is a warning message indicating that the application specified in agent subscription is not yet created. once you create the application in http://<IP_ADDRESS>:8000 with the name leek, the warning will disappear and the agent will be able to index events for that application.
I agree that the warning is somehow misleading, will change it!

Ah indeed! I had created an app with another name, and it wasn't super clear that was causing the issue. I now see the app_name field in LEEK_AGENT_SUBSCRIPTIONS, and the line in the docs, but I probably would never have found it otherwise 😅 Thanks!


chore: use configData

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
use configData instead of config.
https://github.com/GreptimeTeam/helm-charts/pull/152
Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

@WenyXu You should use configData. If use configFile, you have to put the configuration in a standalone file.

@WenyXu You should use configData. If use configFile, you have to put the configuration in a standalone file.

Changed


segfault  when training a network

Following the recent cleanup in free_layer_custom(), we're now seeing a segfault while training a network.  Reported by CrazyBoris on discord:
darknet detector train ships.data yolov4-custom.cfg -map
Darknet v2.0-108-g4eaec0f7
...
[yolo] params: iou loss: ciou (4), iou_norm: 0.07, obj_norm: 1.00, cls_norm: 1.00, delta_norm: 1.00, scale_x_y: 1.05
nms_kind: greedynms (1), beta = 0.600000
Total BFLOPS 59.563
avg_outputs = 489778
Allocating workspace to transfer between CPU and GPU:  50.0 MiB

* * * * * * * * * * * * * * * * * * * * * * * * * * * * *
* Error location: /home/borys/Tools/darknet/src-cli/darknet.cpp, darknet_signal_handler(), line #431
* Error message:  signal handler invoked for signal #11 (Segmentation fault)
* Version v2.0-108-g4eaec0f7 built on Feb 17 2024 00:31:24
* * * * * * * * * * * * * * * * * * * * * * * * * * * * *
backtrace (11 entries):
1/11: darknet(_Z13log_backtracev+0x38) [0x5f0b4d1b6568]
2/11: darknet(darknet_fatal_error+0x208) [0x5f0b4d1b6818]
3/11: /lib/x86_64-linux-gnu/libc.so.6(+0x42520) [0x74ef5ce42520]
4/11: /lib/x86_64-linux-gnu/libc.so.6(free+0x1e) [0x74ef5cea53fe]
5/11: darknet(free_layer_custom+0x8ef) [0x5f0b4d17827f]
6/11: darknet(train_detector+0x3220) [0x5f0b4d1150c0]
7/11: darknet(_Z12run_detectoriPPc+0xb3d) [0x5f0b4d11829d]
8/11: darknet(main+0x44f) [0x5f0b4d09790f]
9/11: /lib/x86_64-linux-gnu/libc.so.6(+0x29d90) [0x74ef5ce29d90]
10/11: /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0x80) [0x74ef5ce29e40]
11/11: darknet(_start+0x25) [0x5f0b4d09a8a5]

Hi. This has been reported by me on discord. I can expand here a bit for more general info:

Pop!_OS 22.04 LTS x86_64; Kernel: 6.6.10-76060610-generic
GPU: RTX 4080 Super; CPU: AMD Ryzen 5 5600X (12) @ 4.200GHz; RAM: 32 GB

Darknet v2.0-108-g4eaec0f7
CUDA runtime version 12030 (v12.3), driver version 12030 (v12.3)
cuDNN version 12020 (v8.9.7), use of half-size floats is ENABLED
=> 0: NVIDIA Graphics Device [#8.9], 15.7 GiB
OpenCV v4.9.0
So running:
darknet detector train ships.data yolov4-custom.cfg

Works but adding -map at the end causes segmentation fault. This however does not happen with:

yolov4 tiny
yolov7x

Both work with -map and without.
I am using cfg from the repo adjusted to my dataset according to guidelines. My dataset consists roughly of 500 training set and 100 validation set. Later for inference yolov4-tiny and yolov7x work fine. Only somehow that yolov4 causes issues.
If I can help here or anything let me know. Thanks for looking into it

Reproduced the problem locally.  I believe this is now fixed with commit hash 6ccabd7.

Yes, I confirm that it has been fixed. Thanks!


Any interest in adding a DataLoader?

I have implemented a multi-process dataloader (similar to torch.utils.data.DataLoader) at teddykoker/tinyloader in < 100 lines of code. Would be happy to submit a PR here if desired.

Great work! The code is incredibly clean and readable! I think it would be a great addition to the extras. Defenitely will be adding this to our fork.

Great work! The code is incredibly clean and readable! I think it would be a great addition to the extras. Defenitely will be adding this to our fork.

Is rolling your own multi threaded queue better than using https://github.com/justheuristic/prefetch_generator/ or is the idea to have 0/minimal external dependencies?

Is rolling your own multi threaded queue better than using https://github.com/justheuristic/prefetch_generator/ or is the idea to have 0/minimal external dependencies?

@Meehai @teddykoker 's implementation is clean and similar to the Torch DataLoader API, so I think it plays well with the goal of having a) a readable source and b) a torch-like API.

@Meehai @teddykoker 's implementation is clean and similar to the Torch DataLoader API, so I think it plays well with the goal of having a) a readable source and b) a torch-like API.

See comment on data loader PR. I'd devote max 100 lines to this. Ideally merging it reduces lines.


Cooldown Bug

Cooldown Dosen't work
like ranking commands
any help

Can you show an error and screenshots, and ways to replicate the situation instead of just stating a fact without showing proof?

bro just spam !rank  or any rank command

bot send me cooldown message And Rank Message like no cooldown
and same problem with other commands

any help ?


pip-compile mishandling conditional requirements.

Currently pip-compile seems to be handling some conditional requirements incorrectly. In particular, the raven package depends on contextlib2, but only for python < 3.2. See here. However, pip-compile adds contextlib2 to the requirements even for newer versions of Python.
As an aside, some documentation about how pip-compile resolves dependencies would be really nice.
Environment Versions
OS Type: Debian Stretch
Python version: 3.5.3
pip version:  9.0.1
pip-tools version: 1.9.0
Steps to replicate

Create a requirements file with raven==6.1.0
pip-compile

Expected result
The requirements file should not include contextlib2.
Actual result
It does include contextlib2.

Ok, a little more investigation suggests that this is actually a bug in pip itself, or possibly raven. I'm going to close this and look into those packages.


Using another Geoserver

Hi,
I tried your framework, and it's really great.
Currently I'm working on a small UAV project, and I plan to use my own Geoserver.
Can I use my own Geoserver to serve map tile for your framework?
If not, could you please state on which class should I re-implement to be able to use my Geoserver instead of Google map?
Thanks

If it uses the same Javascript API as google maps it could work, otherwise a new implementation of the underlying GoogleMap object will need to be created in order to use the other type of Geoserver.


Fix typos (including "ExectueActivity" in a DEBUG log)

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

Coverage increased (+0.04%) to 67.536% when pulling faed6fd102544a7c8f8c3416c602bb125434a47c on syllant:master into fc651437ec06e1e1e4dd8c15487448389ac4fd9e on uber-go:master.


Leaked Secret in internal/credentials/gcp/testdata/test_gcp_key.json

Type: private_key
The issue was introduced on 2023-12-12T00:39:27.093+00:00 in:

the component: private_key
the specific location: internal/credentials/gcp/testdata/test_gcp_key.json

Private keys are the secret part in an asymmetric (public-private) keypair. They are used in different contexts and cryptographic applications.Most cryptography libraries use standard formats for private key exchange (like PEM )Public Keys and Public Key Certificates are public and have no sensitivity or confidentiality requirements (but do not accept a public key without trust on the link between the key and its owner).
Follow these http://docsdev.xygeni.io/xydocs/secrets/detectors/private_key.html

Hi, thanks for creating the issue.
This is a false positive as discussed here https://github.com/chainloop-dev/chainloop/pull/124#discussion_r1205739203
I am afraid that my original concerns about scanners getting confused are a reality. We'll look into alternative options.


Asynchronous theme switching

Hello ! I am building a music app, and thought it would be cool to use your library to change the app's WHOLE (this might be important) theme based on the currently playing music's art. I've noticed that the app freezes for ~3 seconds (this probably will vary depending on the device), I also have to have my custom way of getting the imageBitmap from the currently playing musics art (code below) which even if run on the IO thread, still freezes the app. I also saw that the Log "content" gets logged 3 times when switching theme, if it helps.
Here is my question : If not already, is there a way to implement asynchronous theme switching ?
I think that it's due to the fact that's it's my whole app that is switching theme, but a smooth switching would be nice !
Full code :
@Composable
fun ArtTheme(content: @Composable () -> Unit) {

    val seedColor = getImageBitmap()?.let {
        rememberThemeColor(
        image = it,
        fallback = MaterialTheme.colorScheme.background
        )
    }

    val state = rememberDynamicMaterialThemeState(
        seedColor = seedColor ?: MaterialTheme.colorScheme.background,
        isDark = true
    )

    DynamicMaterialTheme(
        animate = true,
        state = state
    ) {
        Log.d("content", "content wahoo")
        content()
    }

}

@Composable
fun getImageBitmap() : ImageBitmap? {
    val vm = koinViewModel<MusicViewModel>()
    var imageBitmap by remember { mutableStateOf<ImageBitmap?>(null) }
    val context = LocalContext.current
    LaunchedEffect(vm.currentArt) {
        withContext(Dispatchers.IO) {
            try {
                val image = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
                    context.contentResolver.loadThumbnail(
                        Uri.parse(vm.currentMusicUri) ?: Uri.EMPTY, Size(640, 480), null
                    )
                } else {
                    MediaStore.Images.Thumbnails.getThumbnail(
                        context.contentResolver,
                        1,
                        MediaStore.Images.Thumbnails.MINI_KIND,
                        null
                    )
                    // replace 1 with actual ID
                }
                imageBitmap = image.asImageBitmap()
            } catch (e: Exception) {
                e.printStackTrace()
            }
        }
    }
    return imageBitmap
}

The log in the content block will get called multiple times because the theme is animating from one to another.
As for the slowdown. It's most likely the call to rememberThemeColor. It is not very fast and in a future version I plan on removing the current API for it and modifying it.
It should really be called in a coroutine on a different thread.
I will also be investigating ways to improve its performance. But the current  implementation does come from the material-color-utilities.

After some testing it appears the size of the image has an impact, I went from 640 x 480 to 50, 50 and it works perfectly fine now


Weird error message

Hello
Looks like you got a copy/paste error all over the file, since you are not calling "bigtable.NewAdminClient" but "bigtable.NewClient".
https://github.com/GoogleCloudPlatform/golang-samples/blob/ba5aa117abd4ccd77864dc66e7d142e6e03098db/bigtable/reads/reads.go#L40

Thank you for the thoughtful feedback!


Ignore macOS failures on Travis CI

Since the Travis CI OS X infrastructure is incredibly unstable, we need to let the builds run on a best effort basis, but not report failure and not hang a result until it's finished.
With this setup, the Linux build dictates the success, completely ignoring OS X.

Hi @andschwa, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Andy Schwartzmeyer). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

:shipit:


Add helper methods to non-Keyboard classes

Currently, keyboard has a bunch of helper methods to easily create different effects (for example SetBreathing(Color, Color) instead of creating a Breathing struct and calling SetBreathing(Breathing).
This should be added to other devices as well.

Implemented in #85


Upgrade to pac4j v4

@millross I hope you're well in these hard times.
I just wanted to let you know that I have made a huge upgrade and refactoring of the vertx-pac4j library (v5) by upgrading to pac4j v4: https://github.com/pac4j/vertx-pac4j/commit/14c16f068252097eea1753ef40d02c8a4dae1e85
Sorry for the tests removal, but there were disabled and I wasn't able to maintain them.
I updated the demo as well and my tests are ok.
If you have any time for testing or feedbacks...

Hi Jérôme,
I’m doing fine, hope you are too. No need to apologise for maintaining the codebase as you see fit - its your code and if time doesn’t permit getting the tests working (not sure who disabled them, I certainly didn’t - I’m quite test-obsessed) then there’s not much else you can do.
I wish I had time for testing/feedback, but really don’t at the moment. Hope pac4 is ticking along well.
Regards,
Jez

On 15 Jun 2020, at 14:39, LELEU Jérôme<EMAIL_ADDRESS>wrote:
@millross https://github.com/millross I hope you're well in these hard times.
I just wanted to let you know that I have made a huge upgrade and refactoring of the vertx-pac4j library (v5) by upgrading to pac4j v4: 14c16f0 https://github.com/pac4j/vertx-pac4j/commit/14c16f068252097eea1753ef40d02c8a4dae1e85
Sorry for the tests removal, but there were disabled and I wasn't able to maintain them.
I updated the demo as well and my tests are ok.
If you have any time for testing or feedbacks...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/pac4j/vertx-pac4j/issues/139#issuecomment-644141393, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKB7TQGEPMJQY4UOZJFOQ3RWYQCDANCNFSM4MK3TOLA.


[AUTOPATCHER-CORE] Upgrade attr to 2.5.2 Package upgrade for Azure Linux 3.0

[AUTOPATCHER-CORE] Upgrade attr to 2.5.2 Package upgrade for Azure Linux 3.0
Upgrade pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=499602&view=results
buddy build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=499604&view=results

Re-running buddy build with test: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=499611&view=results


First cut at clipping

This is a significant amount of progress towards #36. Bounding boxes for clips are accumulated (in world coordinates, see that issue for more details) CPU-side and encoded in the BeginClip and EndClip elements. These are carried through the pipeline in a straightforward way, then there is a blend stack in the fine rasterizer.
Here are the cleanups not yet done:

The blend stack has finite size.
Optimize all-clear and all-solid clip tiles.

This PR also encodes transforms (they have always worked, just weren't encoded).

Confirmed that FillMask still works with this branch.

Good to know. I'll take that as a go-ahead to commit when I feel it's appropriate. (FYI, generally the strategy I use is that my local branch is one commit ahead. When I'm ready to PR from that, I merge the one behind it into master). I've started the scratch buffer (malloc-only) and do not think it will be very hard. I'll also comment on the dynamic allocation issue.


OrientDB Studio - Schema Manager - Sort Columns

This sorting feature I request, is really handy and useful for all of us working with the Schema Manager improving a lot on the visibility of schema attributes.
Thank you

+1

:+1:

@healiseu
i'm about to work on this
which columns do you think is needed to sort?
-className

Hi @maggiolo00,
I'd say at least: name, superclass,def cluster, records
Please let's keep in mind this enhancement when implementing https://github.com/orientechnologies/orientdb-studio/issues/21
Ideally we can implement https://github.com/orientechnologies/orientdb-studio/issues/21 and then add support for the sort feature. But let's keep this in mind while we work on that ER 21
Thanks,


Implemeted Static UI for displaying parsed csv and Implemented logic …

…to map parsed Csv and implemented order state(#22c8vf8)

Changes merged with https://github.com/hotwax/import/pull/8. Closing


Improve Alert message dialog created with IModelApp.notifications.outputMessage

Is your feature request related to a problem? Please describe.
I need to show some kind of dialog when an error happens. I found the following API from core-frontend, that seems to do what I want:
const message = new NotifyMessageDetails(OutputMessagePriority.Error, "TEST Brief message", "TEST Detailed message", OutputMessageType.Alert)
IModelApp.notifications.outputMessage(message);

It creates the following Dialog:

With more reasonable text values, it looks like this:

I'm no a UI/UX expert, but the component seems to have some issues:

briefMessage and detailedMessage blend in together and are laid out weirdly - I would expect brief message to be in a stronger font and maybe at the top and detailed message in a less intense font below.
I used OutputMessagePriority.Error so I would expect the title to be "Error" or instead use the briefMessage - the current"Alert"  message seems to vague
Error icon doesn't seem to be aligned with text (at least in its current form).

Describe the Solution you'd like
N/A - Would be nice to get some UX input here.
Describe alternatives you've considered
No response
Additional context
No response

Additionally, the notification can be displayed differently based on OutputMessageType.
As part of this item, let's add a storybook story to showcase different options and how those look currently.

@VeroniqueVezina

If the user can't open an iModel because it is already being used by another colleague, we should make that clear on the iModel tile so that there is no bad surprise on click.

Great idea, not cheap to have and implement (depends on how many iModels are in the iTwin), but I think we could do it - I'll create a backlog item for us.

"Contact your administrator for help".... users often don't know who the admin is.

Yeah that might be especially true for smaller companies. We'll need to put more effort into these messages as they are very critical to not get users frustrated. If you can't open an iModel, you can't work - we need good communication here.
One thing we haven't done or thought about - opening an iModel in readonly mode. We could avoid the frustration of not being even able to open the iModel by allowing to open it, but preventing any kind of modification to it, while clearly communicating that the open iModel is in such mode.

@karolis-zukauskas
This is sooooo true: If you can't open an iModel, you can't work - we need good communication here.
I like the idea of opening it in read-only mode if the user can still visualize and navigate properties. We could use the Alert component and show it as a banner at the top of the view to specify it's in read only mode. Maybe have: "This iModel is opened in read only mode because another user is editing it. Contact ______________ for help"
Even with the read-only mode, I'd tend to explore having something visual on the tile as well, but we can think about this later if you think it's too expensive

Added a couple of stories for the notifications APIs https://github.com/iTwin/appui/pull/854


A very minimalist commit, just to check signing...

Trying to discover if this commit is signed...

It isn't signed. If it were, you would see "verified" on the right side of the commit line in this conversation.

Hi, wondering why the heck there are so many steps at GH to do this... :-) Still trying...


docker container unable to attach to calico network (using mesos containerizer works fine)

Firstly, apologies if I am missing something obvious.
Running up tasks with calico networking under mesos using the mesos containerizer works fine.
But, running up tasks using the docker containerizer of directly in docker on a host with calico networking configured fails.  The error suggests that there is a problem connecting with the unix docker socket, but using curl to interrogate the socket indicates that traffic is hitting it, and the container is created but when it tries to attach to the calico network (by the look of it) it dies.
This is with calico/cni v1.10.0, calicoctl v1.6.1, calico/node docker image v2.6.2 (calico/cni v1.11.0 doesn't work with our mesos 1.2 setup as it doesn't manage to find/use calico-ipam plugin)
Current Behavior

create a docker network
docker network create --driver=calico --ipam-driver=calico-ipam calico-net-1-docker

as reflected by
curl --unix-socket /var/run/docker.sock http://localhost/events
{"Type":"network","Action":"create","Actor":{"ID":"1a1fbda4d066c7020c4e90c55899a498dce24a24f03708adfc675436dd91f7bd","Attributes":{"name":"calico-net-1-docker","type":"calico"}},"time":1509639826,"timeNano":1509639826180717680}

docker run --net calico-net-1-docker --name workload-A -tid busybox
f4a18aa1d69aa25511c00b658c2b7379394d1663e518b3409bce126bc9870d19
docker: Error response from daemon: failed to create endpoint workload-A on network calico-net-1-docker: NetworkDriver.CreateEndpoint: {"Err":"Network 1a1fbda4d066c7020c4e90c55899a498dce24a24f03708adfc675436dd91f7bd inspection error: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?"}
null.

docker socket matching events
{"status":"create","id":"f4a18aa1d69aa25511c00b658c2b7379394d1663e518b3409bce126bc9870d19","from":"busybox","Type":"container","Action":"create","Actor":{"ID":"f4a18aa1d69aa25511c00b658c2b7379394d1663e518b3409bce126bc9870d19","Attributes":{"image":"busybox","name":"workload-A"}},"time":1509640076,"timeNano":1509640076928211335}
and /var/log/calico/libnetwork/current:
time="2017-11-02T16:23:46Z" level=info msg=CreateNetwork JSON="{"NetworkID":"1a1fbda4d066c7020c4e90c55899a498dce24a24f03708adfc675436dd91f7bd","Options":{"com.docker.network.enable_ipv6":false,"com.docker.network.generic":{}},"IPv4Data":[{"AddressSpace":"CalicoGlobalAddressSpace","Pool":"<IP_ADDRESS>/0","Gateway":"<IP_ADDRESS>/0","AuxAddresses":null}],"IPv6Data":[]}"
time="2017-11-02T16:23:46Z" level=info msg="CreateNetwork response" JSON="{}"
time="2017-11-02T16:27:56Z" level=info msg=RequestAddress JSON="{"PoolID":"CalicoPoolIPv4","Address":"","Options":null}"
time="2017-11-02T16:27:56Z" level=info msg="Auto assigning IP from Calico pools"
time="2017-11-02T16:27:56Z" level=info msg="Auto-assign 1 ipv4, 0 ipv6 addrs for host 'calico-test-1-node-0'"
time="2017-11-02T16:27:56Z" level=info msg="Auto-assigned 1 out of 1 IPv4s: [<IP_ADDRESS>]"
time="2017-11-02T16:27:56Z" level=info msg="RequestAddress response" JSON="{"Address":"<IP_ADDRESS>/32","Data":null}"
time="2017-11-02T16:27:56Z" level=info msg=CreateEndpoint JSON="{"NetworkID":"1a1fbda4d066c7020c4e90c55899a498dce24a24f03708adfc675436dd91f7bd","EndpointID":"8972198a8ef0b973d56a65f38c3ed3dc8e6dd593adc5be5fb5e57d777e08d1be","Interface":{"Address":"<IP_ADDRESS>/32","AddressIPv6":"","MacAddress":""},"Options":{"com.docker.network.endpoint.exposedports":[],"com.docker.network.portmap":[]}}"
time="2017-11-02T16:27:56Z" level=error msg="Network 1a1fbda4d066c7020c4e90c55899a498dce24a24f03708adfc675436dd91f7bd inspection error: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?"
time="2017-11-02T16:27:56Z" level=info msg=ReleaseAddress JSON="{"PoolID":"CalicoPoolIPv4","Address":"<IP_ADDRESS>"}"
time="2017-11-02T16:27:56Z" level=info msg="Releasing IP addresses: [<IP_ADDRESS>]"
Environment

Calico version: calico/cni v1.10.0, calico/node v2.6.2, calicoctl v1.6.1
Orchestrator version (e.g. kubernetes, mesos, rkt): mesos 1.2.0
Operating System and version: Ubuntu 16.04

I'm wondering if this might be a profile/policy issue. Don't see any default profile or policy created for the docker networks.

@lanejlanej sorry so much time has passed. This slipped through my notifications and I lost track of it.
Calico's libnetwork plugin is run inside the calico/node container, and needs to be able to access the docker socket. As such, calico/node must be run as --privileged, and must get the following volume mounts: -v /run/docker/plugins:/run/docker/plugins -v /var/run/docker.sock:/var/run/docker.sock. Are you volume mounting the docker socket in?

@lanejlanej It's been some time since we heard from you, so I'm going to close this issue.
Again, apologies for not responding to your initial request sooner. Please reopen if you'd like to explore further.


ICEES KG - feature variables not binding properly in responses

The issue is to report that the variable AvgDailySO2Exposure_2 is not binding properly in that the same results are returned as distinct answers, with different CURIEs attached to them for sulfur dioxide. This issue may not be specific to this particular feature variable, but it is the one impacted in the responses to this query: biolink:ChemicalEntity related_to biolink:Disease (MONDO:0004979).

Note this issue probably should have been posted here: https://github.com/ExposuresProvider/icees-kg.


Update README.md

New extention contribution

Thanks for the PR! Possible to limit it to one section?

Thanks for the PR! Possible to limit it to one section?
Hii I have made changes.
You can add it in ### Extensions & Plugins section.

Thanks for the update.


Not able to set chrome user agent from spec

I have a test where i need to launch 2 browsers, 1 with android user-agent and other with windows-mobile user agent. I have set android user agent in conf file. For windows mobile, i am trying following:
driver1 = new protractor.Builder()
.usingServer('http://localhost:4444/wd/hub')
.withCapabilities(protractor.Capabilities.chrome()).setChromeOptions(new chrome.Options().addArguments("--user-agent=" + userAgent)).build();
On run, it says "Failed: chrome is not defined"

I am still looking forward for any suggestion or help.

It looks like chrome is not defined in the snippet new chrome.Options().addArguments("--user-agent=" + userAgent).  Maybe you forgot to import it?  Also, it looks like you're trying to user the underlying webdriver APIs directly, rather than doing this kind of thing through a Protractor Config File.


Feature request: Youtube iframe embed time control

Hi, thanks for creating this amazing plugin. I have some idea to imporve this plugin.
Youtube iframe have the API can control the embed video. I have already made some stuff for my self which can read all the timestamp in the obsidina note and open a html page in browser. In html page it chagne the timestamp to button which can go to specific time of the video. Cos I do not know how to do this in obsidian until I find your plugin. I hope you can integrate this function in your plugin.
Here is how the Youtube iframe API work:
js to control the Youtube
var player,
    time_update_interval = 0;

function onYouTubeIframeAPIReady() {
    player = new YT.Player('video-placeholder', {
        width: 1200,
        height: 800,
        videoId: 'pQN-pnXPaVg',
        playerVars: {'rel': 0},
        events: {
            onReady: initialize
        }
    });
}

function initialize(){
    clearInterval(time_update_interval);
}

function seek(x) {
    player.seekTo(x);
}

player.seek();

In html file
<button class="button" onclick="seek(260)">04:20</button>
Intro

Hope this can give you enough information to do it.

OK,I'll consider include this feature after the code review from obsidian officials is completed and pull request is merged

Done in v2.1.0


Review

Please look at the individual commits for more details. Especially look for FIXME additions.
Please also apply the changes from 61b1c8b601e910a8c03d7d0b1e2b18c87118451a and other commits that would apply to https://github.com/MLopezJ/coiote-to-LwM2M-asset-tracker-v2 before I do a review there.
Most notable changes are: 7f03ff3da6b6e4b4a92acd6497c8485d47684624 and 2cabf4d4de15b3a8832060f943baacef2018c3f4

Fully remove jest from the project

[ ] rename project https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/faf9f508a0c44c9e07b474fd664a915e033d0704
[ ] convert notes to ADR https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/e72ad32c5d75b02074b304683e8c4f534332d905
[ ] Remove references to personal repo https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/e72ad32c5d75b02074b304683e8c4f534332d905
[ ] release to NPM https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/987ca620eea26256c8277b71da8b60b36c105426
[ ] Update readme https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/987ca620eea26256c8277b71da8b60b36c105426
[ ] update package.json https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/f7b9eb8c5921cb025ed3eb657d9a086e2e989531
[ ] update package.json : keywords should be slugs https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/fde3ce6598711908c9b3241d7df67514cc3c9c99
[ ] use correct license: https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/61b1c8b601e910a8c03d7d0b1e2b18c87118451a
[ ] Analyze real use of end-to-end tests https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/4c70abe60625eedbecb9159378bdb9ecbcbabbf0
[ ] improve example https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/45df0fb16551b9f4d32dceccdcba76c1aae1cc2f
[ ] Return error: https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/a6f0ee2ef3fb8113227366a95d1a312b28119248
[ ] Update output format https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/1c3a0108492d9d5041f0138500625cc8e5a8566a
[ ] update comments https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/1c3a0108492d9d5041f0138500625cc8e5a8566a
[ ] remove functions https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/ae6be38a56d0a67a05960343f58821b654f5cf05
[ ] Analyze advantage of https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/2cabf4d4de15b3a8832060f943baacef2018c3f4
[ ] Identify missing values https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/7f03ff3da6b6e4b4a92acd6497c8485d47684624
[ ] use node:test https://github.com/MLopezJ/LwM2M-Asset-Tracker-V2-to-Asset-Tracker-web-app/pull/1/commits/de63b793fc79966d29d1bece99da33564d0de80a

Thanks for the review @coderbyheart . All the recommendations were analyzed and implemented.
In https://github.com/MLopezJ/asset-tracker-lwm2m-js/pull/5 is the current status of the project, with the open questions and pending tasks. I am working on that with Pisut.
I will let you know if I need your help there.


⚠ Bofrost.es has degraded performance

In bdb2462, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1397 ms

Resolved: Bofrost.es performance has improved in f2080bf after 42 minutes.


Use stage_proxy as variable for IQE test

Description
Try using the variable as stage_proxy to see the tests running
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I run go fmt ./... to check that my code is properly formatted
[ ] I run go vet ./... to check that my code is free of common Go style mistakes

Codecov Report

Merging #439 (c64bc7c) into main (7c36336) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #439      +/-   ##
==========================================
+ Coverage   30.97%   30.99%   +0.01%
==========================================
  Files          31       31
  Lines        3112     3110       -2
==========================================
  Hits          964      964
+ Misses       2066     2064       -2
  Partials       82       82

Impacted Files
Coverage Δ

pkg/services/updates.go
38.64% <0.00%> (+0.36%)
: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 7c36336...c64bc7c. Read the comment docs.


Compile error esp32/rom/queue.h:39:1:

I don't think this is your problem - but there is a compile error
Do you have a solution?

I have no time to investigate this issue. It's possible installation-related.
I suggest you post the issue on the Platformio Forum to ask for support as many other people has been successfully using this library with Platform.io.


Update regular emitter operation to log at DEBUG level

https://github.com/snowplow/snowplow-android-tracker/blob/master/snowplow-tracker/src/main/java/com/snowplowanalytics/snowplow/tracker/Emitter.java#L410

Update link to code: https://github.com/snowplow/snowplow-android-tracker/blob/50e104b49748fb02118fc5d6c71e8fffc39969f3/snowplow-tracker/src/main/java/com/snowplowanalytics/snowplow/internal/emitter/Emitter.java#L466
It is tracked as an error but it should be a debug log.


Port and integrate new UCSD brand/theme into horton

Descriptive summary
Please port, integrate, and 'skin' the horton application with the latest theme and branding from campus.
This ticket DOES NOT include design. Porting of existing homepage, customization of default layout, elements, etc. It is expected that a design phase will occur later to address these issues.
Any confusion on the distinction or scope of this work, let's discuss here.
Rationale
We will soon have a test environment which folks will be reviewing from a UI/UX standpoint. It would be very helpful to have that starting point include the campus theme and branding.
Related work

may be a new theme/tech stack coming soon from campus (via Jenn Dandle). So we'll potentially line up this work with that release.
dependent on a test environment to properly demo to staff. though we can get by with screenshots if needed.

getting this setup and assigned to David. It will go in to the next Sprint (16).

@dt-ucsd you still good to go for starting to dig into this in the Sprint starting today?

@dt-ucsd - can you please provide a summary/status update sometime today? Thanks 👍


Version 2.4.0 is still referenced as 2.3.0

Who is the active maintainer of this repo ? @povils ? somethone else

Just @povils currently

Thanks!


Navigating to <clusterlink>/logging in dashboard tries to launch the backup operator instead of logging

From @vincent99 when I was poking around all the re-directs from ember to vue.
Logging has the looking (https://github.com/rancher/dashboard/blob/master/pages/c/_cluster/logging/index.vue#L13) but refers to a CHART_NAME variable that doesn't exist (missing in https://github.com/rancher/dashboard/blob/master/config/product/logging.js )

Rancher version: master-head (09/22/2020) 67e24f2 k8s HA. With EKS downstream cluster.
I wasn't able to reproduce the issue reported here.
I had the backup operator and rancher logging apps installed in the same cluster and I navigated to each of them using the link directly in the browser address bar. Navigated back and forth from each app .
Used Firefox and Chrome based browsers.


Build failure

Expected behavior
Not to fail.
Actual behavior
Failed.
GO111MODULE=on GOBIN="$(pwd)/util" go get ./util/src/harvey/cmd/...
package golang.org/x/crypto/ssh/terminal
imports golang.org/x/sys/unix
imports math/bits: unrecognized import path "math/bits"
Steps to reproduce

QEMU output

```

```

It seems to work:
$ GO111MODULE=on GOBIN="$(pwd)/util" go get ./util/src/harvey/cmd/...
$

Also bootstrap.sh worked well with Go 1.13.6 ¿?
$ freebsd-version -kru
12.1-RELEASE-p1
12.1-RELEASE-p1
12.1-RELEASE-p1

What OS did you use?

Slackware Linux,
go v1.1.2

math/bits was added in Go 1.9. Upgrade it or skip util/src/harvey/cmd/telnet which in fact is old and useless since the console was back at kernel and properly set and depends on a listen1commented line in cpurc.
I encourage you upgrading Go better.

Thanks, that fixed that problem,  and have compiled all the binaries,.
I only installed go as your build system depended on it.
The next problem is ninep does not compile and I guess it is the same reason .
I have my own u9fs server for 9front, so I may persist and investigate further or I may just continue with 9front.
Overall This is too much effort for me, and I probably will wait for you to distribute an ISO file like other 9 variants.

Thanks for your time.