EgressRules are namespaced, but applied globally

EgressRules are created in specific namespaces, rather than being namespace-less resources, but they currently apply globally. This is confusing to users who believe these rules are scoped to their namespace.
For example, one team creating an EgressRule. Other teams end up, unknowingly, using that egress, but don't notice because things seem to be working. If that first team decides they no longer need that rule, and they delete it, every other team is now unable to communicate to that external system.
Istio version: 0.5.1

Assuming per-namespace behavior is desirable, it looks like this can be added to Pilot with a little bit of work:

Expand node.Proxy to store the namespace (parse it from the Domain field, if neccessary)
Modify Pilot's buildEgressTCPListeners and buildEgressHTTPRoutes to skip over EgressRules not defined in the service's namespace.
The rest of xDS continues operating without knowing EgressRules were skipped.

I gave implementing this a shot as described above, filtering out egress rules from other namespaces before passing the config slice to the conflict rejection and it seems to work.
If this is a direction your interested in, I'm happy to start working towards upstreaming.

It looks like EgressRules are being phased out, and replaced with ExternalService + DestinationRule, which look to exhibit these same problems.

@rshriram can you please comment?

Service entries do not respect namespaces. Google.com is visible everywhere. Whether it’s accessible from a pod is a question of policy. By confining service entries to namespaces, you get into the problem of defining some default global namespace applicable everywhere, as well as resolving conflicts when same service entry is present in both namespaces but with different settings, etc.

I suggest using istio RBAC or mixer policies to restrict the namespaces that are allowed to talk to a service entry.

Should we update validation to deny ServiceEntry with non-default namespace?

If these apply across namespaces, why are the CRDs defined to be namespace scoped instead of cluster scoped.

k8s require any resource to be created in a namespace. CRD is like any type of k8s resource. The meaning and application for that resource is not enforced.

@andraxylia Resources can be cluster-scoped or namespace-scoped. EgressRules and the replacement networking APIs are namespace-scoped despite taking effect globally.
This leads users (like me as the OP here) assuming this is a bug in Istio, and these resources are supposed to only effect single namespaces, just like any other type of K8S resource.
@rshriram This might be true now, with the newer APIs and resources, but EgressRules were the sole way at the time this ticket was created. I expected, since they were namespace-scoped, to work similarly to Kubernetes's NetworkPolicies, only allowing egress to where EgressRules were defined in that namespace.
@kyessenov Thanks. I'm glad it's on the radar to fix.

@terinjokes you are right, thanks for correcting me. There is a "scope" field which can be set to Namespaced or Cluster. I did a quick search and all of our CRDs are Namespaced. This issue makes sense, we should change the scope to all APIs that we mean to be cluster scope.

Restricting an Egress (or later an ExternalService) to apply only to traffic out of a specific namespace makes sense in the use case where I have namespaces for my different environments (prod, stage, e2e, sandbox, etc). I don't want the pods in my e2e namespace to have any access at all to the external services backing my prod namespace. The different namespaces run against different environments of my third party dependencies.
It seems from the comments that istio won't implement the namespace restriction on egresses, so do I have to do this thru RBAC?


Backbone.history.navigate removes hash

I am using Backbone router and I'd like to load URLs without full page reload. I add handler that calls Backbone.history.naviagate() on links.
The issue here is that hash from url is stripped. I'd like to pass url with a hash as an anchor for some section on a page like it is done in Backbone documentation, but it doesn't work.
Here is an example where you can see it: http://jsbin.com/wevigoxuqa/1/edit?html,js,output
I expected that after click on link url will be as is: my-page#anchor, but I am getting my-page.
Related https://github.com/jashkenas/backbone/issues/2968

+1

Any update on this?
Also running into this with routes containing for example ...?filter["key"]=true

It is old, but I decided to put an answer here:
So, what happens is that you are using the getFragment, while the #anchor is a hash and the my-page is the fragment. To get what you need, you will do: const my link = Backbone.history.location.pathname + Backbone.history.location.hash


Added error message to failed transfer details.

Refer to the following commit https://github.com/Azure/azure-storage-azcopy/commit/b7993f7edaf49f2dd33d3faa1aa97e910b2fc0df from the PR https://github.com/Azure/azure-storage-azcopy/pull/1745

add error message to JobPartPlanTransfer and failed transfer details

change jobstatus of 403 errors from Cancelled to Failed

use atomic operations for errormessage length

check error message does not exceed max length

A new PR has been created in order to make the changes more modular, and easily reviewable.

Azcopy team anyway going to implement error message handling for each file. This PR is not much useful in that case. We dropping this change as its not space efficient.


Conflict with AnimateDiff

I could not get AnimateDiff to run it's final GIF preview after image generation.    It comes up with the a series of errors that ends with:
AttributeError: 'str' object has no attribute 'height'
After disabling this extension, all works fine.   I run about 18 extensions,  this the only one giving problems for AnimateDiff

thanks, it maybe conflicted when get the final image to saving. i need to install AnimateDiff to check it out.


[Question]: Font color

What happened?
Will it be possible to change also text color on the card?
Card Version
1.1.8
HomeAssistant Version
2022.2.2
Relevant log output
No response

I will think about it. In the meanwhile. if you have card-mod installed you can try style it with css.


Get rid of LOG.error based IDE errors for isBranchPresent calls

Closes #1298

Replaced substring based solution with usage of Vavr collections helper methods. Also, added a code comment to explain why we need this.


Unable to deploy in balena cloud

When I try to deploy latest release for my Syncrobit I get these errors.

`

You probably need to contact Balena about this. Not something we can help with.
The release works fine. Their must be an error with your fleet


Vector Tiles

Adds vector tile support (polygons, polylines, points, meshes, boxes, cylinders, ellipsoids and spheres for classifying 3D Tiles and terrain).
Future work:

[ ] Instancing
[ ] Polygon z-order

@bagnell please update https://github.com/AnalyticalGraphicsInc/3d-tiles/pull/124 to keep the implementation and spec in sync.

I think I mentioned this in another PR, but it would be useful to have tests for the new @private batch primitives.

Just those comments.

Also, update CHANGES.md with any new classes: https://github.com/AnalyticalGraphicsInc/cesium/blob/3d-tiles/CHANGES.md

Added

Update GroundPrimitiveBatch to use a dynamic bottom cap like #4751

to the tasklist.

@pjcozzi This is ready for another look.
I still need to update CHANGES.md and add better Sandcastle examples. The 3D Tiles Classification Sandcastle example just shows a test tile.

Amazing!
@lilleyse can you please do a first review?  Good luck:

😃

@pjcozzi @lilleyse This is ready for another look.

@bagnell can you merge in #6010 / master and use the new experimental doc tag?

Future work:

Instancing
Polygon z-order

Updated #3241

@bagnell @lilleyse let's talk this afternoon on vector vs. classification tile naming.
Also, let's not merge this under after the release on Friday - no need to rush it into the release.

Let me know if this isn't the right place to put this, but from the forum: https://groups.google.com/forum/#!topic/cesium-dev/eXH90obYoSs

We use the vector-tiles branch in map.geo.admin.ch to display 3d tilesets.
One of them is a vector 3d tileset displaying labels.
Problem is the labels aren't display anymore with the last version of this branch.

The incriminated line is :
https://github.com/AnalyticalGraphicsInc/cesium/commit/ce68124e68fdedf8c9b80f280ed0f9edc34bbb8e#diff-bcb4739f7b54493e9b77acc1393eb8acL472
The exception never appears in the console and block the points tileset to be displayed.
Setting the rectangle property like before fixes my problem.

@ggetz That is intended. RECTANGLE is required by the spec. They'll need to regenerate the tileset.

Throughout any code related to pointColor needs to be just color, as reflected in the spec.

A couple new files have slightly low coverage:


Fix minor typo in Depth

Executation => Execution

@spencer-lunarg Thank you for your kind maintenance of this great guide.


Fix comments issue

Comments are attached to BytePos, so you should not replace the span of the node with DUMMY_SP afterwards

@kdy1 thank you so much

It really works now with the fixture test!!


Added translation for the german language based on the english template.

Added translation for the german language based on the english template.

Thanks ! :)


Compound primary keys

RethinkDB doesn't support compound primary keys but it does support arrays as primary keys. The downside is that this makes a mess of marshaling and unmarshaling. I'm wondering whether it would be worth adding some magic tag or whether I'm missing some trick. Basically, suppose I have:
type User struct {
  Company string `gorethink:"company"`
  Name    string `gorethink:"name"`
  Age     int    `gorethink:"age"`
}

and I want a compound primary key on company&name then I need to transform this into
type User struct {
  ID   []string `gorethink:"id"` // really: []string[company, name]
  Age int       `gorethink:"age"`
}

I'm wondering whether it would be doable to introduce something like:
type User struct {
  Company string `gorethink:"id[0]"`
  Name    string `gorethink:"id[1]"`
  Age     int    `gorethink:"age"`
}

Thoughts? Obviously it would be nicest if the DB supported compound primary keys, but I'm not seeing it happen soonish: https://github.com/rethinkdb/rethinkdb/issues/3875

Hey @tve, thanks for the suggestion, I really like the idea and think it will definitely make it easier for some people to use the driver (which is something I am trying to work on at the moment).
I will try to have a think about your solution and see if there are any edge cases that it will not work for and try to think about how it might be implemented but I hope that I will have a working solution to this issue in a couple of weeks.

Hey @tve, I have just created a PR for this issue, if you have the time (and are comfortable with reflection) would you be able to review the changes? Thanks!

Oh cool, yes, I'll review.

Closing as this has now been merged to develop and should be release shortly.


Accept SecurityContext when SSL Enabled.

Security context will enable custom certificates to be accepted, such as if a server is using a self-signed certificate, without the need to pass it through a BadCertificate handler.
Also enables client to send a client certificate for TLS/SSL handshake

Matt - this is awesome - thanks for the PR.
I will push a new version to pub ASAP

2.2 has been uploaded to pub


Is TemplateUI on Nuget?

I really want use TemplateUI in my ongoing Project, but I don't know how to go about it, is it on Nuget?

i hope these controls will be add on Xamarin Community Toolkit 👀

@mytahir Currently TemplateUI is not yet available in NuGet. But... why?. There are a couple more controls and some fixes almost ready. After merge and polish details, TemplateUI will be in NuGet.
@mhrastegary77 That is the idea!. However, there are some controls based on Shapes, etc. Shapes was added in Xamarin.Forms 4.7 and the Community Toolkit is using actually 4.5. We need to wait to move some controls, but the idea is to bring as many controls as possible to the Community  Toolkit.

@jsuarezruiz thanks :D that's amazing, can't wait to use these awesome controls in my projects!!!

Thank you @jsuarezruiz for the reply, you are really doing great on Xamarin. This will help reduce time consuming and some complexities in Xamarin which are the very things People complaining about in Xamarin(including me) but to my Perspective XamarinComunityToolkit and TemplateUI could go independently without merging them, because this will reduce some of TemplateUI Features which are awesome and amazing


Added support for min/max y/x on GridItem

Like min/max h/w, this can clamp the X and Y coordinates of GridItems to the given min and max.

Please remove the build files from this PR, thanks.

Closing this because I recreated the PR from a branch that doesn't include the build files. See new PR: #444


Add driven plugin

Please add the relevant code to allow maestro to communicate with the Driven Server.
Sample code for Driven Plugin
https://gist.github.com/gavinwhyte/a28cbd3e1f3567bab1a6
https://gist.github.com/gavinwhyte/8fe13ab450410d474494
Installation guide for plugin.
http://docs.cascading.io/driven/1.0/getting-started/

I am not sure this would go into maestro. Would it not go into etl. et al?

Good point, I will discuss with Vineeth, as per conversation with Stephan, he preferred it if I posted the sample code and config as a maestro github issue.

I agree, probably it will go into the individual etl jobs, perhaps via some tiny method/function in util.etl.

We don't have a nice way to raise an issue against an etl projects.
On 23/06/2015 4:22 PM, "rowandavies"<EMAIL_ADDRESS>wrote:

I agree, probably it will go into the individual etl jobs, perhaps via
some tiny method/function in util.etl.
—
Reply to this email directly or view it on GitHub
https://github.com/CommBank/maestro/issues/416#issuecomment-114375849.

I forgot that util.etl is on github now, so this could go there.
On 23/06/2015 7:02 PM, "Vineeth Varghese"<EMAIL_ADDRESS>wrote:

We don't have a nice way to raise an issue against an etl projects.
On 23/06/2015 4:22 PM, "rowandavies"<EMAIL_ADDRESS>wrote:

I agree, probably it will go into the individual etl jobs, perhaps via
some tiny method/function in util.etl.
—
Reply to this email directly or view it on GitHub
https://github.com/CommBank/maestro/issues/416#issuecomment-114375849.

Raised on etl-util as https://github.com/CommBank/util.etl/issues/15


IOS EasyAuth.signOut() does not sign out.

Love the design of your library, though I'm having a few issues with EasyAuth.
I've been attempting to use easy-firebase for a basic email authentication IOS app, and it seems there there are some issues in the IOS example that are also happening when I implement the library with the base instructions provided.
Reproducible by compiling the latest IOS example, signing in with email, and attempting to sign out.
IOS 16.2
Iphone 14 Pro
easy-firebase 1.4.3
Firestore rules:
rules_version = '2';
service cloud.firestore {
  match /databases/{database}/documents {
    match /{document=**} {
      allow read, write: if request.auth != null;
    }
  }
}

After setting the state of my app to "signed out", which brings my app back to it's original Sign In form, and running "EasyAuth.signOut()", I receive the following logged errors:

This is not a problem if I change my firestore rules to just "allow read, write;" (which is certainly not a solution). Is there a listener that could still be attached when the sign out function succeeds and my auth header is no longer valid?

It looks like stopping the listener before the guard check at onUserUpdate is fixing this issue in my case (swapping lines) 144 / 145:


Fixed 27397 Querying with an integer larger than SQLite supports crashes with OverflowError

let me know if there's something I miss cc: @timgraham

Yes, the fix isn't going to work for Ticket27397.objects.exclude(id=11111111111111111111111111111111111111111111111) (no results will be returned rather than all objects).

Sorry still I don't understand your means about this ? I have test exclude nothing overflowing,

Add this assertion to your new test:
self.assertEqual(
    len(Order.objects.exclude(pk=11111111111111111111111111111111111111111111111)),
    1
)

It doesn't pass.

len(Order.objects.exclude(pk=11111111111111111111111111111111111111111111111)) is 0 Zero is not equal

The test should pass. The Order that exists doesn't have that pk.

@timgraham
>>> from django.contrib.auth.models import User
>>> User.objects.filter(id=1222222222222222222222222222222222222222222222222)
<QuerySet []>
>>> User.objects.filter(id=1222222222222222222222222222222222222222222222222)
<QuerySet []>
>>> User.objects.exclude(id=1222222222222222222222222222222222222222222222222)
<QuerySet []>
>>> len(User.objects.exclude(id=1222222222222222222222222222222222222222222222222))
0
>>>

this test with except when i added and without except is OverflowError , when i use except return 0,
i'm so sorry @timgraham  Where is my mistake?

If there are users in the database, User.objects.exclude(id=1222222222222222222222222222222222222222222222222)) should return all the users, not zero users.

Thanks ,but i not sure should return all the users because before Overflow occurred from SQLlite this means any select all of database side, do you have solution?

>>> from django.contrib.auth.models import User
>>> User.objects.filter(id=1)
<QuerySet [<User: raminfp>]>
>>> User.objects.filter(id=1222222222222222222222222222222222222222222222222)
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 232, in __repr__
    data = list(self[:REPR_OUTPUT_SIZE + 1])
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 256, in __iter__
    self._fetch_all()
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 1087, in _fetch_all
    self._result_cache = list(self.iterator())
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 54, in __iter__
    results = compiler.execute_sql()
  File "C:\Python27\lib\site-packages\django\db\models\sql\compiler.py", line 835, in execute_sql
    cursor.execute(sql, params)
  File "C:\Python27\lib\site-packages\django\db\backends\utils.py", line 83, in execute
    sql = self.db.ops.last_executed_query(self.cursor, sql, params)
  File "C:\Python27\lib\site-packages\django\db\backends\sqlite3\operations.py", line 142, in last_executed_query
    params = self._quote_params_for_last_executed_query(params)
  File "C:\Python27\lib\site-packages\django\db\backends\sqlite3\operations.py", line 129, in _quote_params_for_last_executed_query
    return cursor.execute(sql, params).fetchone()
OverflowError: Python int too large to convert to SQLite INTEGER
>>> User.objects.exclude(id=1222222222222222222222222222222222222222222222222)
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 232, in __repr__
    data = list(self[:REPR_OUTPUT_SIZE + 1])
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 256, in __iter__
    self._fetch_all()
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 1087, in _fetch_all
    self._result_cache = list(self.iterator())
  File "C:\Python27\lib\site-packages\django\db\models\query.py", line 54, in __iter__
    results = compiler.execute_sql()
  File "C:\Python27\lib\site-packages\django\db\models\sql\compiler.py", line 835, in execute_sql
    cursor.execute(sql, params)
  File "C:\Python27\lib\site-packages\django\db\backends\utils.py", line 83, in execute
    sql = self.db.ops.last_executed_query(self.cursor, sql, params)
  File "C:\Python27\lib\site-packages\django\db\backends\sqlite3\operations.py", line 142, in last_executed_query
    params = self._quote_params_for_last_executed_query(params)
  File "C:\Python27\lib\site-packages\django\db\backends\sqlite3\operations.py", line 129, in _quote_params_for_last_executed_query
    return cursor.execute(sql, params).fetchone()
OverflowError: Python int too large to convert to SQLite INTEGER
>>>

Sorry, I don't have any ideas about a proper fix.

i talk about signed integer https://code.djangoproject.com/ticket/27397#comment:24
also C API : https://github.com/python/cpython/blob/master/Modules/_sqlite/util.c#L129&L153
but still i not sure for open  ticket python community about this,

@timgraham  please answer this question : https://code.djangoproject.com/ticket/27397#comment:25
also look :  http://www.sqlite.org/capi3ref.html#sqlite3_int64

If you're proposing to change the default primary key type to BigIntegerField, no we can't do that due to backwards-compatibility.

You are right ,Oops, i think this fix so far enough,because with use exclude with large integer raise OverflowError

@timgraham i think we should write on docs about behavior OverflowError from SQLlite,What do you think?

We usually try to fix bugs rather than document them.

I wonder if this could be solved by raising an EmptyResultSet somewhere.

Hi @charettes
Do you have idea?  What do you means  EmptyResultSet ?

Hi @timgraham
my idea Filter values,
diff --git a/django/db/backends/sqlite3/base.py b/django/db/backends/sqlite3/base.py
index e9c8e11..b476748 100644
--- a/django/db/backends/sqlite3/base.py
+++ b/django/db/backends/sqlite3/base.py
@@ -324,6 +324,16 @@ class SQLiteCursorWrapper(Database.Cursor):
     def execute(self, query, params=None):
         if params is None:
             return Database.Cursor.execute(self, query)
+        if isinstance(params, (tuple)):
+            if params:
+                if isinstance(params[0], (int, long)):
+                    if params[0].bit_length() > 63:
+                        lst = list(params)
+                        lst[0] = 1234
+                        params = tuple(lst)
+                        query = self.convert_query(query)
+                        return Database.Cursor.execute(self, query, params)
+
         query = self.convert_query(query)
         try:
             return Database.Cursor.execute(self, query, params)

output :
./manage.py shell
>>> User.objects.exclude(id=1)
<QuerySet []>
>>> User.objects.exclude(id=12)
<QuerySet [<User: raminfp>]>
>>> User.objects.exclude()
<QuerySet [<User: raminfp>]>
>>> User.objects.exclude(id=11333333333333333333333333333333)
<QuerySet [<User: raminfp>]>
>>> User.objects.exclude(id=113333333333333333333333333333335555555555555)
<QuerySet [<User: raminfp>]>
>>> User.objects.exclude(id=113333333333333333333333333333335555555555555555555555555555)
<QuerySet [<User: raminfp>]>
>>> User.objects.exclude(id=113333333333333333333333333333335555555555555555555555555555888888888888)
<QuerySet [<User: raminfp>]>
>>> User.objects.filter(id=113333333333333333333333333333335555555555555555555555555555888888888888)
<QuerySet []>
>>> User.objects.filter(id=113333333333333333333333333333335555555555555555555555555555888888888888)
<QuerySet []>
>>>
>>>
>>> User.objects.exclude(id=1133333333333333333333333333333355555555555555555555555555551212321312312321)
<QuerySet [<User: raminfp>, <User: test>]>

browser in admin panel :

@RaminFP this approach won't fly as the sentinel integer used to replace overflowing values can match an existing row.

@charettes this is not final fix,so i need flag for discover , execute User.objects.exclude() return all objects,
>>> User.objects.exclude()
<QuerySet [<User: raminfp>, <User: test>]>
>>>

@charettes lst[0] = 0 is good?

It has the same caveats for a model with a an IntegerField instead of an
AutoField. I don't think this is where this should be handled.
2016-11-10 11:35 GMT-05:00 Ramin Farajpour Cami<EMAIL_ADDRESS>
@charettes https://github.com/charettes lst[0] = 0 is good?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/7529#issuecomment-259738763, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAkTYXpxAI15bOcz8B48kvf-cTXOmohks5q80e9gaJpZM4Kr85w
.

@timgraham @charettes sorry for tag,
Python doesn't distinguish between integer sizes .We'll need to explicitly check for 2^63 and -2^63 or sys.maxint

Simon said, this fix doesn't look elegant.

your means i write bad code, or not this is a bad idea?

@RaminFP we believe this is not the right approach as it could break in subtle ways. Using '' as a sentinel value could still break when using raw():
Author.objects.create(name='')
assert not Author.objects.raw('SELECT * FROM author WHERE name = %s', [2^64])

OK @charettes ,please give me suggestion rather then ''?
what do I do?

@RaminFP I haven't investigated if this is possible but I think this would require all lookups available for numeric fields to deal with this case on SQLite. For example the Exact and GreaterThan lookups could raise an EmptyResultSet like the In lookup does.
That's the only way I can of but I'm starting to think this might not be worth the required efforts. It seems odd to me SQLite doesn't handle this case like other RDMS do.

@charettes Thanks , i need your help and Tim for fix it,this is a special case,

As Simon suggested, you could check if this could be fixed in SQLite.

@timgraham I am not familiar with lookups, and how lookup avoid OverflowError
i see here : https://docs.djangoproject.com/en/1.10/howto/custom-lookups/#custom-lookups
this example : Author.objects.filter(id__oveflow=11111111111111111111111111111111111111111)
how to filer large number on lookup?
please give me example lookup for this if this is possible,

I'm sorry but I can't provide any further guidance about a proper solution.

@charettes i need you ,I like fix this issue,please give me diff patch of lookups

i going to close PR for new PR


'NoneType' object has no attribute 'strip'

from langchain.document_loaders.csv_loader import CSVLoader
loader = CSVLoader(file_path='docs/whats-new-latest.csv', csv_args={
'fieldnames': ['Line of Business',
'Short Description'
]
})
data = loader.load()
print(data)
/.pyenv/versions/3.9.2/envs/s4-hana-chatbot/lib/python3.9/site-packages/langchain/document_loaders/csv_loader.py", line 53, in
content = "\n".join(f"{k.strip()}: {v.strip()}" for k, v in row.items())
AttributeError: 'NoneType' object has no attribute 'strip'
Can anyone assist how to solve this?

Same here, did you get it sorted out?

Hi, @LeiSanity how did you resolve this?

Hey, same issue

Also having the same issue

Same issue!

@othyn I don't know if this is an option for you but I just switched over to using the Pandas Dataframe Loader https://python.langchain.com/en/latest/modules/indexes/document_loaders/examples/pandas_dataframe.html
and it works fine :)

@MogwaiMomo Thank you!
I was using it as part of privateGPT - so don't really have a choice on libraries unfortunately. Although interestingly it does have pandas support, but thats one for the maintainers.
Although I think the original problem was caused by something in the data formatting of the ingesting CSV.
Instead of debugging the application, I looked to debugging the data instead and that seemed to fix things. Althouh, as I just wiped out all the columns I wasn't using, so I'm not entirely sure of the fix to any future readers.
Althought for the maintainers of the repo, it would be nice to have some more type checks on perhaps on if the expected row data isn't in alignment with the expected type? Although I'm not sure what this will look like in Python as my skills there are a bit rusty.

For me, this issue was a empty line at the top of my csv file

To avoid getting that NoneType object has no attribute 'strip' error, I had to make sure that ALL the field names in the CSV were listed. (This defeated the purpose of this option so ultimately I just removed the fieldnames option and kept the delimiter and quotechar options.
So if you have 3 columns in your CSV then you need to mention ALL three columns in the fieldnames option ['col1','col2','col3']
I originally thought that the fieldnames option would allow you to specify the columns you want from your CSV and all other columns would go into a separate spot (based on the csv module docs). It does not.

Thanks for the insight @steven-levey-sp.
The method is a bit deceiving. I also thought it should function the same way you mentioned. I'll have to update my CSV before loading it in to my script.


Date format on chat messages in MessageThread component does not reflect browser's local settings for Edge browser

Describe the bug; what happened?
Date format displayed on chat messages in the MessageThread component does not reflect the browser's locale settings for Microsoft Edge (works fine for Chrome/Safari).
When Edge language is set to English (United States), date is still formatted as dd/mm/yyyy.
What are the steps to reproduce the issue?
Consume MessageThread component with a chat message in a React app and view in Microsoft Edge.
What behavior did you expect?
The date format to be in line with the browser's locale settings e.g. English (United States) to be mm/dd/yyyy.
If applicable, provide screenshots:

@azure/communication-react npm package version (if applicable): 1.10.0

@azure/communication-chat npm package version (if applicable): 1.3.2

OS & Device: Windows on PC

Browser: Microsoft Edge

Browser Version: 119.0.2151.72

Hi @Raveen-Maldeni , thanks for raising this issue. Please allow me to check with our team and get back to you. Thanks

@Raveen-Maldeni Can you please send us sample of your code?

Here's a code snippet:
`import React from "react";
import { MessageThread, SendBox, DEFAULT_COMPONENT_ICONS, ChatMessage, FluentThemeProvider, MessageProps, MessageRenderer } from "@azure/communication-react";
import { initializeIcons, registerIcons, Persona, PersonaSize } from "@fluentui/react";
function mapToChatMessage(message, allUsers: User[], currentUser: User): ChatMessage {
return {
senderDisplayName: getUserName(message.createdByUserId, allUsers ?? []),
senderId: message.createdByUserId,
editedOn: message.createdOn.valueOf() === message.lastModified.valueOf() ? undefined : message.lastModified,
content: message.text,
createdOn: message.createdOn,
contentType: "richtext/html",
messageType: "chat",
messageId: message.id,
mine: message.createdByUserId === currentUser.id,
status: "delivered" //This can be set to delivered if the message has been saved to cosmosdb successfully if we want to utilise the retry behaviour of the chat component.
};
}
function updateMessageToRender(messageProps: MessageProps, defaultMessageRenderer: MessageRenderer): JSX.Element {
const MessageElement = () => {
// eslint-disable-next-line @typescript-eslint/consistent-type-assertions
const chatMessage = messageProps.message as ChatMessage;
return (

{chatMessage.mine ? (
//Current User's icon
<Persona
text={chatMessage.senderDisplayName}
size={PersonaSize.size32}
initialsColor={...}
initialsTextColor={...}
onRenderPrimaryText={() => <></>}
/>
) : (
//Other User's icon
<Persona
text={chatMessage.senderDisplayName}
size={PersonaSize.size32}
initialsColor={...}
initialsTextColor={...}
onRenderPrimaryText={() => <></>}
/>
)}
{defaultMessageRenderer({ ...messageProps })}

);
};
return MessageElement();
}
export function CommentsContainer(props) {
...
return (

        <FluentThemeProvider>

                    {
                        <MessageThread
                            userId={currentUser.id}
                            messages={hitComments?.map((message) => mapToChatMessage(message, allUsers, currentUser)) ?? []}
                            onDeleteMessage={async (messageId: string) => {
                                dispatch(
                                    ...
                                );
                            }}
                            onUpdateMessage={async (messageId: string, content: string) => {
                                dispatch(
                                    ...
                                );
                            }}
                            showMessageDate={true}
                            onRenderMessage={(props: MessageProps, defaultOnRender) => {
                                if (!defaultOnRender) {
                                    throw new Error("defaultOnRender is undefined");
                                }
                                return updateMessageToRender(props, defaultOnRender);
                            }}
                        />
                    }

        </FluentThemeProvider>

);

}
`

I'll have an update for you on this today.

We have a feature currently in the beta version PublicPreview 1.11.0-beta.1 of the library that does support custom timestamps.
Either through the ChatComposite here
Or through components using the MessageThread here

Yes the ideal behavior works in the beta but we didn't want to release to our production environment with the beta package. Is there any known timeline for when that will reach a regular release?

We are using Javascript's toLocaleDateString() method for formatting the date time string. I have linked our code below.
https://github.com/Azure/communication-ui-library/blob/fb1a6d4e87548215f9d5654f5b60a0a114912c0d/packages/react-components/src/components/utils/Datetime.ts#L17
After some initial investigation Javascript's toLocaleDateString() does seem to have differing behaviour depending on the system and browser.
I am going further look into this and will update you with what I find.

What is the locale and timezone settings for your system? do they differ from what you have set in the browser?

Yes they do. My system is set to Auckland, NZ datetime but my browser is set to English US. Was wanting to confirm this was a browser related issue and not due to the implementation. I've noticed on Chrome with the same system and browser settings it respects the browser settings for formatting the date.
Now that you've clarified its using toLocaleDateString I think we can close this since it seems to be due to Edge rather than the package.
Thanks for looking into this.


[fix 1446807] Order by join date

When we will want to move on about it the code is ready.

@akatsoulas r?


[WIP] Added some math functions and helpers

Switched the order of upper and lower error for datapoints because it seems more reasonable to use it in this order (lower,upper).
Added some default constructors and type-specific ones which are needed when calling some functions with DataPoint types as an argument.
Need an opinion by @fzierler on how we treat the systematic uncertainties for things like relative error and region of datapoints.

Need an opinion by @fzierler on how we treat the systematic uncertainties for things like relative error and region of datapoints.

I don't understand what you mean by that.

Need an opinion by @fzierler on how we treat the systematic uncertainties for things like relative error and region of datapoints.

I don't understand what you mean by that.

I mean if we want to have something like relerror(D::DataPoint) how is the correct way to handle systematic uncertainties?
Should we then have one tuple with (stat_err .+ sys_err)./val or return two tuples for the errors independently?
I am also not sure if we even should keep the systematic error after all.

I am also not sure if we even should keep the systematic error after all.

I agree.


Slownes when adding many components

When you add a new component to the grid, the existing components are first removed from the div before the new and existing components are added. This is done using the following code:
GridLayout.java line 354:
div.removeAll();
div.add(components.toArray(new Component[0]));
However, this approach becomes inefficient when dealing with a large number of components (100+), as it requires frequent updates to the DOM. A more efficient solution is to assemble all the components first and then add them to the div in a single operation.
Suggestion:
Add this to GridLayout.java:
public void addComponentToComponentList(Component component)
public void addComponentToComponentList(Component component, int column1, int row1)
public void addComponentToComponentList(Component component, int column1, int row1, int column2, int row2)
public void addComponentsToDiv()

Issue is now resolved.


More tests inner parse + docs

added more unit tests. Moved innerlayout tests from whenTests to simpleLayoutParserTests
Side note:
I tried allowing balanced brackets (e.g. {} doesn't need an escape), but this cannot be done without being fully backwardscompatible

Current coverage is 70.68%

Merging #981 into master will not affect coverage as of 7bc235e

@@            master    #981   diff @@
======================================
  Files          262     262
  Stmts        14786   14786
  Branches      1613    1613
  Methods          0       0
======================================
  Hit          10451   10451
  Partial        401     401
  Missed        3934    3934

Review entire Coverage Diff as of 7bc235e

Uncovered Suggestions

+0.08% via ...oredConsoleTarget.cs#232...244
+0.08% via ...c/NLog/LogFactory.cs#180...191
+0.07% via ...anceCounterTarget.cs#308...318
See 7 more...

Powered by Codecov. Updated on successful CI builds.


Add BuildProgressPlugin

Reincarnation of #950. Please see it for details.

@gaearon

require('webpack').ProgressPlugin should be fine. Peer dependency was only removed because now that we declare compat with Webpack 2, it will be annoying to bump it every few weeks. If npm provided a way to declare compat with all beta versions, I would've kept it.

But we are assuming specific plugins in react-dev-utils rely on Webpack. It's fine to import anything from it if these things are supported in 1.x and 2.x.

Are you saying that I should remove the peer dependency of webpack and trust that it is there at runtime? As of now, I used the semver matching from 4cd9fd909ad32cbf09e8a6f9d9be5f1a27a4dcfe.

Are you saying that I should remove the peer dependency of webpack and trust that it is there at runtime?

Yes. (It's not as bad as it sounds, this plugin is useless without webpack anyway.)

(It's really nice btw, great work)

Removed the peer dep, is this ok behavior though for when someone installs this for their own use?

is this ok behavior though for when someone installs this for their own use?

Yes it’s fine. Peer deps are often more trouble than they’re worth, and until Webpack 2 is out of beta, they are definitely a pain since semver authors refuse to have a way to refer to non-stable versions via a range.

Ping, would you like to make the requested changes so we can get it in?

Sure

@gaearon pending CI, should be good -- though I'm not sure if I'm happy with the quality of progress the ProgressPlugin returns.
Also, it makes the CI log very dirty. Use your discretion or maybe we should explore alternatives to both outlets (progress plugin and progress package) (or submit PRs to webpack to make progress % better).
Maybe we should shut off the progress bar when not in an interactive terminal?

Maybe we should shut off the progress bar when not in an interactive terminal?

👍

@gaearon turns out travis is tty. I ended up checking for process.env.CI.

Let's hold off on this until 0.10.0 and hope webpack's progress reporting got better. If not we can adjust it to just show current step / steps remaining.

@Timer any updates on this one? Would be really nice locally but I agree, not for CI.


Please actually update the docs

As I spent hours trying to get this to compile, I realized that the documentation differs so much from the actual thing that I need to do just to make it work.

No-one stops you from updating the docs with your corrections and issuing a pull request.


Chore: look into LGTM alerts/CodeQL

Context https://github.com/netlify/cli/pull/1053#issuecomment-674388096
and https://lgtm.com/projects/g/netlify/cli/?mode=list

Closing as stale. Please re-open if needed


Fix AlignVariablesSection using fixed separator

Fixes #605

Codecov Report
Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (0372450) 96.88% compared to head (d7032f7) 90.10%.

Files
Patch %
Lines

robotidy/transformers/AlignVariablesSection.py
9.09%
10 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #609      +/-   ##
==========================================
- Coverage   96.88%   90.10%   -6.79%
==========================================
  Files          50       50
  Lines        4917     4920       +3
==========================================
- Hits         4764     4433     -331
- Misses        153      487     +334

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Finish Building Out Full  Component

Summary
Finish building the full <VideoSeries /> component by combining a <SeriesHero /> (see #16 ) component and a three-column grid of <VideoThumbnail /> (see #15 ) components populated by data from the mock api responses added in #11 . In the attached screenshot, you can see what I'm m talking about. Your job is to create a very similar user interface using only components from our melos component library: https://github.com/lifechurch/melos
You won't be able to recreate the exact look and feel that you see on the Bible.com videos page, but that's OK! 😄 What you see on Bible.com represents our old design style, but melos represents our newest design specs. I am sure you'll have questions, so feel free to ping me. We'll get one of our YouVersion designers to give you the final 👍  when you're done. So, you can expect there to be some back and forth work with them getting it just right.
Mock API Data
After #11 has been completed, there should be some mock data available to use while building your component. Instead of hard-coding the strings and images, you want to grab those from the mock api data. If you have questions about this, just make a comment here and we'll do our best to answer quickly.
This is what you're building

I’ll take this one if that’s how this is going down 👍🏼

@MrChristofferson how's is this going? Need anything?

@michael-martin-al Not yet, just noticed the other components moved to 'Done', I'll start integrating :)


Missing Dependency/ Python Package from environment.yaml

Hi there, I was trying to run the example from the readme and got the message

python examples/random_env.py --env_name PassWater
Traceback (most recent call last):
File "examples/random_env.py", line 5, in
from softgym.registered_env import env_arg_dict, SOFTGYM_ENVS
File "/h/ashokdha/projects/softgym/softgym/registered_env.py", line 1, in
from softgym.envs.pour_water import PourWaterPosControlEnv
File "/h/ashokdha/projects/softgym/softgym/envs/pour_water.py", line 8, in
from shapely.geometry import Polygon
ModuleNotFoundError: No module named 'shapely'

I installed shapely into my conda environment and this went away. I think there may be a missing dependency for this in the environments.yaml file?
Thank You,
Dhananjay

Thanks for posting this issue! I have updated environments.yaml


Inconsistency with actual lua (stack issue)

This repository hasn't been touched in ages but it is still useful so I would like to point something out to those who use it.
The program:
local functions = {}
for i = 1, 10 do
    functions[i] = function()
        print(i)
    end
end
functions[2]()
functions[9]()

real lua output:
2
9

lbi output:
10
10

I'm having trouble describing the problem with words so I hope the above example was enough for you to understand.
I would create a pull request if I knew how to fix this but the easiest work around for me was to use https://github.com/Rerumu/FiOne instead.

Thanks for the bug report.
I think it might be this code lol
		[35] = function(instruction)	-- CLOSE
			io.stderr:write("NYI: CLOSE")
			io.stderr:flush()
		end,

But yea, looks like a pretty big problem with closures. I will try to fix this sometime in the next few days.


Q: Is it possible to add a class to the current sorted column?

Is there a way to add a css class to the current sorted column? I've been trying to do this but so far, no success.

Assuming issue raiser has resolved their issue, as no comments for nearly 3 weeks.

Ok, so I came across this old thing but I wanted to update with one small change to @PaulL1 's solution. His answer got me on the right track but the if condition needs to change slightly. My col.sort is always evaluated as true it's an object. Rather you want to do a conditional on whether or not priority is defined.
`gridoptions.columnDefs[0].headerCellClass = $scope.highlightHeader;
$scope.highlightHeader: function( row, rowRenderIndex, col, colRenderIndex ) {
if ( typeof(col.sort.priority) !== 'undefined'  ){
return 'header-sorted';
}
};`


SILOptimizer: Break circular dependency with SIL library by moving extendStoreBorrow()

Fixes a cycle introduced by https://github.com/apple/swift/pull/60467.

@swift-ci please smoke test


easy/pipeline/server

It appears that FramedIo is being deprecated (according to doc) and minihttp code is not up to date with the latest of the other tokio repos. This piece of code doesn't work any longer and when changed to Framed the new method went away etc...
So, what is the best way to handle something like the easy/pipeline/server new with the update libraries? Example of minihttp code:
NOTE: Tried socket.framed(HttpCodec);  // Merged Decoder/Encoder to support Codec changes.
// Create the transport
let transport = EasyFramed::new(socket,
request::Decoder,
response::Encoder);
    // Return the pipeline server task
    let server = pipeline::EasyServer::new(service, transport);
    handle.spawn(server.map_err(|e| {
        println!("error: {}", e);
    }));

Maybe a better question. What would be the recommend way to implement an HTTPs like server similar to minihttp?

So this won't exactly answer your question, but have you looked at https://github.com/popravich/minihttp? Apparently that's the new hip 'minihttp' people are talkin' about, so if you were looking at 'tokio-minihttp'... that's so last month.
At this moment however 'popravich/minihttp' is not updated for tokio-core's most recent version, only tokio-proto, but based on where you asked this question, that should be oki.
Hope this helped.

Sorry, the update to this lib is lagging a bit. The PR is in progress here https://github.com/tokio-rs/tokio-proto/pull/75

Thanks for the update. So, it appears the code (similar) to minihttp will be moved here with PR for an http protocol?

PR is merged! I'll update minihttp shortly.


Design Continued.

Add field.* functionality to nested function in SELECT clause.
@GumpacG
Paused

(design) Planning for JOINs development
@acarbonetto
In Progress
Same-table JOIN #1441
Putting together a github proposal for same-table design

Update anonymizer to now engine
@matthewryanwells
In Progress

Begun setting up tests for new data anonymizer
Compare new anonymizer results to legacy anonymization

Update documentation for pagination
@Yury-Fridlyand
In Review
On upstream review #1592

Update docs according to the team review

WHERE and HAVING in pagination
@Yury-Fridlyand
In Review
On upstream review #1500

ORDER BY and LIMIT in pagination
@Yury-Fridlyand
On hold
On upstream review #1599, depends on #1500

Merge index scans for pagination
@MaxKsyunz
In Review
On upstream review #1600

Test JDBC with V2 pagination
@Yury-Fridlyand
On hold
#254, depends on #1599 and #1500

Support reading different date formats
@GabeFernandez310@GumpacG
In Progress

Removed support for time only or date only formats
Address PR comments and modify IT to only include supported formats

Investigate ODBC issue with AWS_SIGv4 authentication
@Yury-Fridlyand
In Review
Raised issue: sql-odbc#39Most recent fixes are not in the v1.4 driver (also, it is not yet signed)Will need to release a new driver on completion
Try to reproduce customer environment with VPC
Continue experiments

Status Update | May 18, 2023

Topic
Dev
Status
Comments
What have you done yesterday?
What are you going to do today?
Any impediments?
ETA

Add nested support in WHERE clause.
@forestmvey
In Progress

Continued implementation of WHERE clause nested function in predicate expression.
Add test coverage.

Nested support in SELECT clause design documentation.
@forestmvey
Upstream Review

Nested WHERE clause documentation proposal.
@forestmvey
Internal Review

Revise based on comments.

Add field.* functionality to nested function in SELECT clause.
@GumpacG
Paused

(design) Planning for JOINs development
@acarbonetto
In Progress
Same-table JOIN #1441
Design up for review

Update anonymizer to now engine
@matthewryanwells
In Progress

Submitted PR
Address comments

Update documentation for pagination
@Yury-Fridlyand
In Review
On upstream review #1592

Update docs according to the team review

WHERE and HAVING in pagination
@Yury-Fridlyand
In Review
On upstream review #1500
Demo

ORDER BY and LIMIT in pagination
@Yury-Fridlyand
On hold
On upstream review #1599, depends on #1500

Merge index scans for pagination
@MaxKsyunz
In Review
On upstream review #1600

Test JDBC with V2 pagination
@Yury-Fridlyand
On hold
#254, depends on #1599 and #1500

Support reading different date formats
@GabeFernandez310@GumpacG@acarbonetto
In Progress

Addressed PR comments and modified IT to only include supported formats
Continue to address PR comments

Investigate ODBC issue with AWS_SIGv4 authentication
@Yury-Fridlyand
In Progress
Raised issue: odbc#39, odbc#41Fix PoC in odbc#42
Continue testing
Continue testing

Status Update | May 26, 2023

Topic
Dev
Status
Comments
What have you done yesterday?
What are you going to do today?
Any impediments?
ETA

Add nested support in WHERE clause.
@forestmvey
In Review

Make upstream PR.
Awaiting PR feedback.

Add nested all fields support.
@forestmvey
In Progress

Begin implementing.
Development continued.

Nested support in WHERE clause design documentation.
@forestmvey
In Review
On upstream review #1640

Update anonymizer to new engine
@matthewryanwells
In Review
In review #226
Addressed comments
Continue addressing comments

WHERE and HAVING in pagination
@Yury-Fridlyand
In Review
On upstream review #1500

ORDER BY and LIMIT in pagination
@Yury-Fridlyand
On hold
On upstream review #1599, depends on #1500

Merge index scans for pagination
@MaxKsyunz
In Review
On upstream review #1600

WIP

Pagination docs update
@MaxKsyunz
In Review
On upstream review #1644

Remove getTotalHits feature
@Yury-Fridlyand
In Review
On upstream review #1649

Test JDBC with V2 pagination
@Yury-Fridlyand
On hold
#254, depends on #1599 and #1500

Support reading different date formats
@GumpacG@acarbonetto
In Review
Internal review #268
Completed unit test coverage, checkstyle, and added more integration tests.
Address PR comments

Investigate ODBC issue with AWS_SIGv4 authentication
@Yury-Fridlyand
In Review
FIX: odbc#42

Verify the fix

Close cursor API in V2
@Yury-Fridlyand
In Progress
#1660
WIP
WIP

Fixing prometheus start issue when last run was crashed
@MitchellGale
In Progress
Fixing issue: #1083
Attempted at changing dependencies in build.gradle for doctest and integ-test

May 30th, 2023

Topic
Dev
Status
Comments
What have you done yesterday?
What are you going to do today?
Any impediments?
ETA

Add nested support in WHERE clause.
@forestmvey
In Review
On upstream review #1657

Awaiting PR feedback.

Add nested all fields support.
@forestmvey
In Progress

Development continued.
Adding code coverage and cleaning up implementation.

Nested support in WHERE clause design documentation.
@forestmvey
In Review
On upstream review #1640

Update anonymizer to new engine
@matthewryanwells
In Review
On upstream review #1665
Passed tests
Get merged

WHERE and HAVING in pagination
@Yury-Fridlyand
In Review
On upstream review #1500

Rebase and resolve conflicts

ORDER BY and LIMIT in pagination
@Yury-Fridlyand
On hold
On upstream review #1599, depends on #1500

Merge index scans for pagination
@MaxKsyunz
Complete
On upstream review #1600
Done

Pagination docs update
@MaxKsyunz
Complete
On upstream review #1644
Done

Remove getTotalHits feature
@Yury-Fridlyand
In Review
On upstream review #1649

Rebase and resolve conflicts

Test JDBC with V2 pagination
@Yury-Fridlyand
On hold
#254, depends on #1599 and #1500

Support reading different date formats
@GumpacG@acarbonetto
In Review
On upstream review #1664

Investigate ODBC issue with AWS_SIGv4 authentication
@Yury-Fridlyand
In Review
FIX: odbc#42

Close cursor API in V2
@Yury-Fridlyand
Done
On upstream review #1660

Rebase and resolve conflicts

Fixing prometheus start issue when last run was crashed
@MitchellGale
In Review
Fixing issue: #1083
Finished as forked PR Made changes to build.gradle for doctest and integ-test to make stop Prometheus by MitchellGale · Pull Request #270 · Bit-Quill/opensearch-project-sql

Fix sql-cli folder delete issuie
@MitchellGale
In Progress
1675

Working on fix for [BUG] delete of sql-cli is not run if build does not complete · Issue #1675 · opensearch-project/sql

Status Update | June 2nd

Topic
Dev
Status
Comments
What have you done yesterday?
What are you going to do today?
Any impediments?
ETA

Support inner_hits ExprValue construction.
@forestmvey
In Progress

Beginning ticket.

Add nested all fields support.
@forestmvey
On Hold

Website Documentation for nested in 2.8 release.
@forestmvey
In Progress

Documenting changes for release.

WHERE and HAVING in pagination
@Yury-Fridlyand
In Review
On upstream review #1500

Rebase and resolve conflicts

ORDER BY and LIMIT in pagination
@Yury-Fridlyand
On hold
On upstream review #1599, depends on #1500

Test JDBC with V2 pagination
@Yury-Fridlyand
On hold
#254, depends on #1599 and #1500

Fixing prometheus start issue when last run was crashed
@MitchellGale
In Review
Fixing issue: #1083

Fix sql-cli folder delete issue
@MitchellGale
Paused
1675

Fix ODBC installer
@Yury-Fridlyand
In Review
odbc#44
WIP, verification
Completed

ODBC driver release preparations
@Yury-Fridlyand
In Review
odbc#45odbc#46odbc#47odbc#48odbc#49

Completed

Fix JDBC driver for VPC + SSH tunnel case
@Yury-Fridlyand
In Progress
jdbc#83
WIP
WIP

Update OpenSearch Improving website link
@matthewryanwells
In Review
1639
Address Comments
Wait to be merged

Fix byte divide/modulus bug
@matthewryanwells
In Review
272
Address Comments
Address Comments

Update OpenSearch website arithmetic documentation
@matthewryanwells
In Review
4232
Addressed Comments
Post upstream PR

Spike on supporting user-defined datetime formats
@GumpacG
In Review
Possible solution in internal review #273
Implemented a possible solution
Discuss solution

Spike on adding new type for text
@GumpacG
In Progress

Ramp-up on the issue at hand

OpenTelemetry Exporter for OpenSearch
@MaxKsyunz
In Progress

Setup public fork, transferred code, issues, and project to it
Submit first PR to opentelemetryUpdate exporter code to latest

Status Update | June 5th, 2023

Topic
Dev
Status
Comments
What have you done yesterday?
What are you going to do today?
Any impediments?
ETA

Support inner_hits ExprValue construction.
@forestmvey
In Progress

Beginning ticket.
Finding way to support for nested and non-nested data response.

Add nested all fields support.
@forestmvey
On hold

Website Documentation for nested in 2.8 release.
@forestmvey @Andrew Carbonetto
In Progress

Adding documentation for score and metafields

WHERE and HAVING in pagination
@Yury-Fridlyand
In Review
On upstream review #1500

Rebase and resolve conflicts

ORDER BY and LIMIT in pagination
@Yury-Fridlyand
On hold
On upstream review #1599, depends on #1500

Test JDBC with V2 pagination
@Yury-Fridlyand
On hold
#254, depends on #1599 and #1500

Fixing prometheus start issue when last run was crashed
@MitchellGale
In Review
Fixing issue: #1083

Continue bug fixing issues in GHA after PR review comments were addressed

Fix sql-cli folder delete issue
@MitchellGale
Paused
1675

Fix ODBC installer
@Yury-Fridlyand
In Review
odbc#44
WIP, verification
Completed

ODBC driver release preparations
@Yury-Fridlyand
In Review
odbc#45odbc#46odbc#47odbc#48odbc#49

Completed

Fix JDBC driver for VPC + SSH tunnel case
@Yury-Fridlyand
In Progress
jdbc#83
WIP
WIP

Update OpenSearch Improving website link
@matthewryanwells
In Review
1639
Waiting to be merged
Waiting to be merged

Fix byte divide/modulus bug
@matthewryanwells
In Review
272
Address Comments
Address Comments

Update OpenSearch website arithmetic documentation
@matthewryanwells
In Review
4232
Waiting to be merged
Waiting to be merged

Dev Sum fix
@matthewryanwells
In Progress

Familiarize self with code and start work
Fork doesn’t have imports working

Spike on supporting user-defined datetime formats
@GumpacG
In Review
Possible solution in internal review #273
Implemented a possible solution
Discuss solution

Spike on adding new type for text
@GumpacG
In Progress

Ramp-up on the issue at hand

OpenTelemetry Exporter for OpenSearch
@MaxKsyunz
In Progress

Setup public fork, transferred code, issues, and project to it
Submit first PR to opentelemetryUpdate exporter code to latest

2.8 Release checklist
@Andrew Carbonetto
In Review

All testing and sanity checks are complete. Checklist is in final stages.
Setup post-release items (retrospective) with team and complete checklist items (documentation)

Status Update - June 9, 2023

Topic
Dev
Status
Comments
What have you done yesterday?
What are you going to do today?
Any impediments?
ETA

Support inner_hits ExprValue construction.
@forestmvey
In Progress

Add IT tests, cleanup code, make internal PR.
Revise based on PR feedback.

Add nested all fields support.
@forestmvey
On hold

Website Documentation for nested in 2.8 release.
@forestmvey@acarbonetto
In Progress

Adding documentation for score and metafields
Continue documentation for nested.

WHERE and HAVING in pagination
@Yury-Fridlyand
In Review
On upstream review #1500

ORDER BY and LIMIT in pagination
@Yury-Fridlyand
On hold
On upstream review #1599, depends on #1500

Rebase and resolve conflicts

Test JDBC with V2 pagination
@Yury-Fridlyand
On hold
#254, depends on #1599 and #1500

Fixing prometheus start issue when last run was crashed
@MitchellGale
In Review
Fixing issue: #1083

Continue bug fixing issues in GHA after PR review comments were addressed

Fix sql-cli folder delete issue
@MitchellGale
Paused
1675

Fix ODBC installer
@Yury-Fridlyand
In Review
odbc#44

ODBC driver release preparations
@Yury-Fridlyand
In Review
odbc#45odbc#46odbc#47odbc#48odbc#49odbc#50odbc#51

Fix JDBC driver for VPC + SSH tunnel case
@Yury-Fridlyand
In Review
jdbc#83

JDBC driver release preparations
@Yury-Fridlyand
In Review
jdbc#85jdbc#86jdbc#87jdbc#88jdbc#89

Fix byte divide/modulus bug
@matthewryanwells
In Review
1716
Upstream PR
Waiting to be merged

Update OpenSearch website arithmetic documentation
@matthewryanwells
In Review
4232
Waiting to be merged
Address Comments

Dev Sum fix
@matthewryanwells
In Progress
6984
Set up optional parameter on the sum function
Fix bug using new optional parameter

Spike on supporting user-defined datetime formats
@GumpacG
In Review
Possible solution in internal review #273
Implemented a possible solution
Discuss solution

Make Tableau connector submission
@GumpacG
In Progress

Conducted testing
Set up docker with multiple nodes to attempt to make test results more consistent

OpenTelemetry Exporter for OpenSearch
@MaxKsyunz
In Progress

Submit first PR to opentelemetryUpdate exporter code to latest
Add opensearchexporter to otelcolcontribRecord demo

2.8 Release checklist
@acarbonetto
In Review

All testing and sanity checks are complete. Checklist is in final stages.
Setup post-release items (retrospective) with team and complete checklist items (documentation)


[request] "limited" mode for functionality on older terminals

On some very limited terminals, it is not possible to open this because they lack certain features.
For instance, I was attempting to use a project that uses tcell on plan 9, and when I succeeded in making it build I could not make it work in 9term.
2020/10/28 11:29:00 Error launching View (terminal not cursor addressable).
Would it be possible to have a sort of limited mode that did not use the mouse so that it would run in these terminals?

plan 9 is going to be more or less impossible for reasons unrelated to the terminal.  The issue with plan 9 is that I have no knowledge of how to do the equivalent of stty to put the terminal in raw mode, without that you're doomed.
The second problem is the terminal definition.  We need a way to be able to place the cursor at a precise location on the screen so that we can draw at a specific coordinate.  I would imagine that the terminal emulator (9term?) should be able to do this, but we may lack the details of how to do that.  Under traditional UNIX terminals you send an escape code to put the cursor at specific X,Y locations.  That escape code is defined by the "cup" ability in terminfo typically.  Apparently we don't have that.
If you are a plan 9 user, and are willing to help contribute the code to solve these problems then I'd be happy to see tcell work on this platform -- unfortunately I lack sufficient knowledge and couldn't locate suitable information that would tell me enough about how the plan 9 terminal system works to code it up.

Ok, don't worry about it. Thanks
On 10/28/20, gdamore<EMAIL_ADDRESS>wrote:

plan 9 is going to be more or less impossible for reasons unrelated to the
terminal.  The issue with plan 9 is that I have no knowledge of how to do
the equivalent of stty to put the terminal in raw mode, without that you're
doomed.
The second problem is the terminal definition.  We need a way to be able to
place the cursor at a precise location on the screen so that we can draw at
a specific coordinate.  I would imagine that the terminal emulator (9term?)
should be able to do this, but we may lack the details of how to do that.
Under traditional UNIX terminals you send an escape code to put the cursor
at specific X,Y locations.  That escape code is defined by the "cup" ability
in terminfo typically.  Apparently we don't have that.
If you are a plan 9 user, and are willing to help contribute the code to
solve these problems then I'd be happy to see tcell work on this platform --
unfortunately I lack sufficient knowledge and couldn't locate suitable
information that would tell me enough about how the plan 9 terminal system
works to code it up.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/gdamore/tcell/issues/402#issuecomment-718016862

Closing this for now...
(I did look a bit more in depth and it is possible to address cursors, but plan9 does not use escape sequences at all, nor does it use ttys.  It appears you need to interact directly with libdraw and with the events subsystem.  This is probably very similar to the Windows console API in level of effort.  It will need someone else to champion it, ideally with sufficient knowledge to contribute code.  Plan 9 is really bizarre in these particulars.)

Plan 9 has "vt" which emulates a normal terminal. Things ssh + irssi
work with it.
It also has a bash port somewhere.
On 10/29/20, gdamore<EMAIL_ADDRESS>wrote:

Closed #402.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/gdamore/tcell/issues/402#event-3938924979

Good day @gdamore
Thank you for the great library!
Earlier in this thread on Plan9 you mentioned

This is probably very similar to the Windows console API in level of effort. It will need someone else to champion it, ideally with sufficient knowledge to contribute code

Could you describe where would this scope of work begin? For instance - what functionality (files, functions, etc) to replicate for Plan9. How would a testing look like?

You would probably want to look at the source code for the vt program. Then you would implement the equivalent functions to what is in console_windows.go.  If you're familiar enough with plan 9 it's probably not too bad.  But I'm not and I don't have a system to test it on.

for references:

VT command is available at:
http://code.9front.org/hg/plan9front/file/d414d0d5c557/sys/src/cmd/vt
While the console_win.go is at:
https://github.com/gdamore/tcell/blob/master/console_win.go

for references:

VT command is available at:
http://code.9front.org/hg/plan9front/file/d414d0d5c557/sys/src/cmd/vt
While the console_win.go is at:
https://github.com/gdamore/tcell/blob/master/console_win.go

Yes, I've seen those.  I don't know if we could somehow utilize vt to do the necessary things for us or not.  The entire plan9 console and I/O subsystem is quite different from either UNIX or Windows, and I haven't had enough time to wrap my head fully around it.  There are a tiny number of people running it as their windowing system, so I've not been super incentivized to work on this.  I think it's something someone familiar with plan9 could do though -- and I'm happy to provide pointers or advice.  That console_win.go is a good start.

Yes, I've seen those.  I don't know if we could somehow utilize vt to do the necessary things for us or not.  The entire plan9 console and I/O subsystem is quite different from either UNIX or Windows, and I haven't had enough time to wrap my head fully around it.  There are a tiny number of people running it as their windowing system, so I've not been super incentivized to work on this.  I think it's something someone familiar with plan9 could do though -- and I'm happy to provide pointers or advice.  That console_win.go is a good start.


Updated AnimeSaturn's extension links

AnimeSaturn recently changed their domain, so I modified the links for their extension accordingly.

Perfect, this issue can be closed then:
https://github.com/jmir1/aniyomi-extensions/issues/617


Check pnn and command exist in SiteConfig fallback.

This patch avoids a KeyError Exception being thrown when 'phedex-node'
and 'command' keys are not present in the fallback stage out definitions in
the site config. It explicitly checks for their absence and raises an
appropriate StageOutInitError.
modified:   src/python/WMCore/Storage/StageOutMgr.py

I think precise error message is better. So we can see where it fails, although currently it seems stageout error is not send back to agent.
How about clean up a little.

msg = "Unable to retrieve fallback %s\n"
msg += "From site config file.\n"
msg += "Unable to perform StageOut operation"

if pnn is None:
    raise StageOutInitError(msg % "phedex")
if command is None:
    raise StageOutInitError(msg % "commend")

And for the same as other code which has the same message.

Precise error message is better I think.

@ticoann I suggest we merge this PR as it is. If needed, I can update this code later on, I have some changes on my private branch already.


Mail service should keep email on cache if mail service is temporarily down and sent again when mail service is up. And once failed, could retry serval times

Queuing like that could be done by mailer daemons like postfix, I don't think every application needs to re-implement it, or at least, it should probably be disabled by default.

Queuing like that could be done by mailer daemons like postfix, I don't think every application needs to re-implement it, or at least, it should probably be disabled by default.

Yes, this could be done by writing emails to the postfix pickup/maildrop directory and then letting postfix handle sending them.

I also gree that it is MTA's task, not Gitea's.


Improve merged PR cleanup

after merging and manually deleting this PR's branch, the "delete PR branch" step was successfully skipped: https://github.com/bioimage-io/spec-bioimage-io/runs/4409156816?check_suite_focus=true


layout parser does not work well when try with diverse layout of PDFs

I'm working on a set of PDFs that have different page layouts i.e. multi-column/single column+images+figures+tables. It's giving the below average results though I have tried different provided models for it.

Can anyone suggest me how to achieve better results with such a diverse document pages?
Also,the most important, I need the reading order in the return Layout variable of results. How can I get this? Example given below:

Layout(_blocks=[TextBlock(block=Rectangle(x_1=201.03945922851562, y_1=413.36480712890625, x_2=1500.326904296875, y_2=1290.304931640625), text=None, id=None, type=Figure, parent=None, next=None, score=0.9502648115158081), TextBlock(block=Rectangle(x_1=174.8553466796875, y_1=270.81329345703125, x_2=1229.29443359375, y_2=416.44305419921875), text=None, id=None, type=Title, parent=None, next=None, score=0.9470152258872986), TextBlock(block=Rectangle(x_1=200.6973419189453, y_1=489.5100402832031, x_2=560.0352172851562, y_2=518.6799926757812), text=None, id=None, type=Text, parent=None, next=None, score=0.8652349710464478), TextBlock(block=Rectangle(x_1=260.79986572265625, y_1=1346.680419921875, x_2=1495.7305908203125, y_2=1452.4434814453125), text=None, id=None, type=Text, parent=None, next=None, score=0.8538650870323181)], page_data={})
Let me know if anyone has any suggestion/solution to improve it. Thanks a lot.
Environment

Linux
layout parser version: 0.3.4
I followed the official documentation.

Screenshots

Hi there,
I don't know how to fix this but did you already tried the cloud vision api of google?
I think it's maybe an interesting solution for you.


cURL installation

Hi,
It would be very helpful to get an exampl of the installation of cURL for Windows. I have found some but they require the use of a different than Pushbullet platform. ?Do you know of any example?
Thanks in advance.
Cheers,
Felipe

Hi Felipe,
I unfortunately don't have access to a Windows machine at the moment and thus can't provide much guidance about cURL installation on that OS beyond what's available via Google. That being said, I should note that statapush is designed to work with a number of different push notification platforms, including Pushover and IFTTT. If you're having trouble getting Pushbullet to work, perhaps it's worth trying a different provider?
Hope this helps!
Best,
Will


python312Packages.boto3-stubs: 1.35.82 -> 1.35.84, python312Packages.botocore-stubs: 1.35.82 -> 1.35.84.post1

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 366540

x86_64-linux

:white_check_mark: 14 packages built:

aws-sam-cli
aws-sam-cli.dist
python311Packages.boto3-stubs
python311Packages.boto3-stubs.dist
python311Packages.botocore-stubs
python311Packages.botocore-stubs.dist
python311Packages.types-aiobotocore
python311Packages.types-aiobotocore.dist
python312Packages.boto3-stubs
python312Packages.boto3-stubs.dist
python312Packages.botocore-stubs
python312Packages.botocore-stubs.dist
python312Packages.types-aiobotocore
python312Packages.types-aiobotocore.dist


Update dependency phpunit/php-code-coverage to v11

This PR contains the following updates:

Package
Type
Update
Change

phpunit/php-code-coverage
require-dev
major
10.1.11 -> 11.0.0

Release Notes

sebastianbergmann/php-code-coverage (phpunit/php-code-coverage)
v11.0.0
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: composer.lock
Command failed: composer update phpunit/php-code-coverage:11.0.0 --with-dependencies --ignore-platform-req='ext-*' --ignore-platform-req='lib-*' --no-ansi --no-interaction --no-scripts --no-autoloader --no-plugins
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires phpunit/php-code-coverage 11.0.0 (exact version match: 11.0.0 or <IP_ADDRESS>), found phpunit/php-code-coverage[11.0.0] but these were not loaded, likely because it conflicts with another require.
  Problem 2
    - brianium/paratest is locked to version v7.4.0 and an update of this package was not requested.
    - brianium/paratest v7.4.0 requires phpunit/php-code-coverage ^10.1.11 || ^11.0.0 -> found phpunit/php-code-coverage[10.1.11, 11.0.0] but these were not loaded, likely because it conflicts with another require.
  Problem 3
    - phpunit/phpunit is locked to version 10.5.10 and an update of this package was not requested.
    - phpunit/phpunit 10.5.10 requires phpunit/php-code-coverage ^10.1.5 -> found phpunit/php-code-coverage[10.1.5, ..., 10.1.11] but it conflicts with your root composer.json require (11.0.0).

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.


Why WebSocketSubjects multiplex does not return a Subject?

Hi.
Let me just start by saying that there is a possibility I misunderstood multiplex operator completely. If that is so, sorry for spamming (and I would be super thankful for some resources on the subject).
As far as I understand, multiplex simulates having websocket connections to multpile endpoints, while it really has only single connection to a "gateway" endpoint that can forward messages incoming from endpoints we pretend we have connection to. For example we pretend we have a connection per conversation in a chat app, while in fact there is only one connection to a server that marks incoming messages with flag that forwards a message to proper Observable.
If I get it right, then the question is, why we have a possibility to simulate such a connection only in one way (server -> client messages only)? Since multiplex returns an Observable, we do not have a possibility to next message directly to fake socket that represents single conversation in my chat app. I have to use origin WebSocketSubject and while calling next on it, every time manually mark my message with a flag that forwards my message to proper conversation.
In fact, we could take it much further and make multiplex return WebSocketSubject, so that it could be itself multiplexed recursively again and again. In complex chat app with tree-like threads, conversations and user structure, I could represent such structure with my WebSocketSubjects and always forward messages directly where I want, without any boilerplate.
Also quite interestingly if done right, it would not be a breaking change at all.
Thoughts?

+1

I agree with you 100% @mpodlasin and I find myself in a similar scenario. How did you resolve this? Can you post code?

Closing this, as we've updated the documentation. Perhaps at some point we can add a feature as outlined, but there hasn't been much demand for it, TMK.

Hey everyone!
I currently have some problems regarding websockets + rxjs.
My initial problem is that I want to control the network errors, outages, etc separately from business logic, preferably in the service itself. Optionally, I would have somewhere a function wich would return an observable that emits 'OK' or 'Err' just in case I want to have some icon somewhere that the connection is lost.
It was not easy and intuitive, and I don't even think it's the proper way to do this, but given the tools with RxJS + websocket, I could not do it any better.
My problem was that it's not easy to find a way to implement the reconnect functionality.
This is basically my code:
import { webSocket } from 'rxjs/webSocket';
import './style.css';

import { BehaviorSubject, retry, Observable, shareReplay } from 'rxjs';

const RECONNECT_DELAY = 5000;

const errors$ = new BehaviorSubject<Error | undefined>(undefined);
const subject$ = webSocket({
  url: 'wss://example.com/ws',
  closeObserver: {
    next: (event) => {
      console.log(event);
      errors$.next(new Error('Connection closed'));
    },
  },
  openObserver: {
    next: (event) => {
      errors$.next(undefined);
    },
  },
});

const channel1: Observable<unknown> = subject$
  .multiplex(
    () => ({
      join: 'JOIN',
      channel_name: 'channel_1',
    }),
    () => {
      return {
        action: 'LEAVE',
        channel_name: 'channel_1',
      };
    },
    (msg: any) => msg.channel_name === 'channel_1'
  )
  .pipe(
    retry({ delay: RECONNECT_DELAY }),
    shareReplay({ bufferSize: 1, refCount: true })
  );

channel1.subscribe((x) => console.log('channel_1', x));

const channel2: Observable<unknown> = subject$
  .multiplex(
    () => ({
      join: 'JOIN',
      channel_name: 'channel_2',
    }),
    () => {
      return {
        action: 'LEAVE',
        channel_name: 'channel_2',
      };
    },
    (msg: any) => msg.channel_name === 'channel_2'
  )
  .pipe(
    retry({ delay: RECONNECT_DELAY }),
    shareReplay({ bufferSize: 1, refCount: true })
  );

channel2.subscribe((x) => console.log('channel_2', x));

This somehow works, and if there is a need for reconnect, this code does the job and reconnects. After a successful reconnect, the channel subscriptions are also sent, and we are happy.
I am only not happy about one thing: because writing to a channel is done like this:
subject$.next({channel_name: 'channel_1', message: 'hello'});

It can happen that in case of an accidental disconnect, messages could be queued. When a reconnect then happens in the future, these messages are sent before the subscription messages are sent.
What do you think? Am I missing something, or is there anything I can do better?
If multiplex would return a Subject, these cases I think could be fixable at the subject side, because it could already know the channel, and could possibly link with the messages.


Manually modify the configuration file automatically restored after restarting the service

My steps:

change port of outbonds through vim "/Users/xxx/Library/Application Support/v2raya/config.json"
brew services restart v2raya

But the configuration file automatically restored after restarting the service, manually modifying the configuration file does not take effect.

@d1ngx write a Launchd service by yourself and run it with launchctl


Implements VLAN tagging on network adapters

Hi,
this is currently missing the Pester test, I could need some advice on this...

Updates xHyper-V to version 3.18 (current from PSGallery)
Implements the ability to define a network adapter configuration for MSFT_xVMNetworkAdapter
Updates documentation and examples accordingly

This solves issue #188
In the node data you can now define an array of hashtables that hold configuration for adapter, that is connected to the defined switch.
NetAdapterConfiguration = @(
    @{
        VlanId = 1
        SwitchName = 'Default Switch'
        MACAddress = '00-15-5d-ff-ff-01'
    },
    @{
        VlanId = 1
        SwitchName = 'Internal'
        MACAddress = '00-15-5d-ff-ff-02'
    }
)

It seems dependencies in DSC are severly broken....
Currently I am struggling with the issue in DscResource.Common: https://github.com/dsccommunity/DscResource.Common/issues/37


x11 always starts with 1024x768 resolution

Description
Although bootloader and virtual terminal will detect the correct screen size and switch to the correct resolution, once you start x11 it will switch to a 1024x768 framebuffer, regardless of monitor size.
it is possible to switch to a correct resolution with xrandr, but proper behaviour would be x11 choosing the best resolution automatically (like it does in raspbian with the same configuration).
I compared configuration files with raspbian (which works correctly) and could not find any difference in config files.
I tested on yocto-3.1/meta-raspberry:dunfell on a RPi4 - i only have a 4 model here, so i cannot crosstest with an earlier chipset.
tested on different monitors with different sizes/aspect ratios... always 1024x768
X11 detects all possible resolutions according to Xorg.0.log.
Steps to reproduce the issue:

bitbake core-image-x11
boot system on rpi4
watch boot

Describe the results you received:
system boots, console comes up with correct resolution
once the system starts x11, resolution gets switched to 1024x768
Describe the results you expected:
x11 comes up with correct resolution
Additional information you deem important (e.g. issue happens only occasionally):
Build Configuration:
BB_VERSION           = "1.46.0"
BUILD_SYS            = "x86_64-linux"
NATIVELSBSTRING      = "universal"
TARGET_SYS           = "arm-poky-linux-gnueabi"
MACHINE              = "raspberrypi4"
DISTRO               = "poky"
DISTRO_VERSION       = "3.1"
TUNE_FEATURES        = "arm vfp cortexa7 neon vfpv4 thumb callconvention-hard"
TARGET_FPU           = "hard"
meta
meta-poky
meta-yocto-bsp       = "my-yocto-3.1:5d47cdf448b6cff5bb7cc5b0ba0426b8235ec478"
meta-raspberrypi     = "dunfell:ac717473e89b835da2eb418b0507644d71aa61c2"
no changes to local.conf except MACHINE.
Additional details (revisions used, host distro, etc.):
Xorg.0.log
dmesg.log

The same thing happened to me,
I solved it by changing the HDMI port of the rpi4,
from HDMI-port1 to HDMI-port0 (closest to the USB-C power connector).

Just tested - no didn't help - still 1024x768.
To be fair - there is a workaround - you can add
xrandr --output HDMI-1 --preferred

to your xinit script and the resolution will switch to the correct one.
I'd still prefer x11 to actually behave the same as on Raspbian and come up in the correct resolution, but I'm unable to find out why it doesn't.

Do you use a micro-HDMI to HDMI adapter? or do you use a micro-HDMI to HDMI cable?
In my case, when I use the adapter, the resolution remains at 1024 x 768, both in yocto and in Raspi-OS.
I had to buy the cable for it to work properly.
You can try indicating the resolution you want in the config.txt file.
https://www.raspberrypi.org/documentation/configuration/config-txt/video.md
For example, to obtain a resolution of 1080p (1920x1080) 60 Hz, add the following lines:
# Hdmi 0
hdmi_group:0=1
hdmi_mode:0=16
# Hdmi 1
hdmi_group:1=1
hdmi_mode:1=16

i'm using the official micro-HDMI to HDMI cable from RaspBi (with the logo on it).
I don't want to hardcode the resolution, because in the end I don't know what the device is going to be connected to.
and I know the resolution is available and detected, because xrandr --output HDMI-1 --preferred will switch to the correct one.
Just for some reason, x11 chooses to not use the preferred resolution right from the start.

@horihel any progress on this?

unfortunately no - I ended up putting the xrandr workaround into the startup script - that solved the problem for me.
Eventually the project I was working on got discontinued - so I can't tell if later updates fixed this.
Sorry.

That's alright. I'll close this for now and we can pick it later if needed. Thanks for the update, @horihel.


ACM-2664

Fixed faulty hcp version check which caused duplicate versions to show up sometimes
Before:

After:

Signed-off-by: Omar Farag<EMAIL_ADDRESS>

/cc @fxiang1


Ensure no mistaken speculation

Test from https://github.com/EnzymeAD/Enzyme.jl/issues/554

Codecov Report
Base: 75.16% // Head: 72.59% // Decreases project coverage by -2.57% :warning:

Coverage data is based on head (5d39423) compared to base (8721c50).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #567      +/-   ##
==========================================
- Coverage   75.16%   72.59%   -2.58%
==========================================
  Files          17       17
  Lines        5734     5436     -298
==========================================
- Hits         4310     3946     -364
- Misses       1424     1490      +66

Impacted Files
Coverage Δ

src/compiler/interpreter.jl
26.53% <0.00%> (-65.63%)
:arrow_down:

src/compiler/orcv2.jl
0.00% <0.00%> (-61.91%)
:arrow_down:

src/compiler/utils.jl
58.57% <0.00%> (-37.32%)
:arrow_down:

src/logic.jl
75.00% <0.00%> (-5.00%)
:arrow_down:

src/api.jl
58.91% <0.00%> (-2.78%)
:arrow_down:

src/compiler/validation.jl
59.59% <0.00%> (-2.48%)
:arrow_down:

src/compiler/pmap.jl
75.27% <0.00%> (-2.27%)
:arrow_down:

src/pmap.jl
83.33% <0.00%> (-1.29%)
:arrow_down:

src/typeanalysis.jl
90.00% <0.00%> (-0.91%)
:arrow_down:

src/Enzyme.jl
85.26% <0.00%> (-0.83%)
:arrow_down:

... and 4 more

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.


package.json: change scripts/peg.js to run in prepublish not postinstall

To workaround an npm problem in which it sometimes refuses to run the
postinstall script, change the package so that it generates the parser
in the prepublish phase and not postinstall.
This also required explicitly enumerating the files to be put into the
package since by default it omits anything that's in .gitignore, but we
want it to include the generated files in the package.

lgtm. and you ran through the steps to show it works.

there should also be a pr against 0.4.x for an 0.4.2 juttle release.

Actually this was meant to be off of 0.4.x. Will reopen appropriately.


Update linters configuration

Edit pre-commit configuration for correct operation of the linters;
Integrate yamlint into pre-commit configuration to ensure the YAML files adhere to specified guidelines;
Fix warning raised by yamllint in the CI workflow configuration

It'd be helpful to have something specific in the title and description.

pydocstyle -> flake8-docstrings


Performance issue when there are too many mesh peers

Describe the bug
Right now mesh peer param Dhigh=12, there could be too many mesh peers per subnet on a test mainnet node

Gossip Block process time

Gossip Block Received Delay

If we configure network.gossipsubDHigh=9 so that there is no subnet has more than 9 mesh peers

we got better metric

Expected behavior
Lodestar should able to handle high number of mesh peers per spec (with Dhigh=12)

gossipsubDHigh was set back to 12, closing the issue


Auto center the selected day in the listview

is it possible to center the selected day in the listview?

@Matt0550  did you find any solution? I want to center the current day.

@Matt0550 , @tariqzarif the package supports now the auto center


3.2.1 and 3.2.2

Coverage remained the same at 76.69% when pulling d7ea72f155b522aa2dfe76c4c56b965f0f90995c on asottile:32bugfixes into 0206aca17461ece5279b9eaf715e52f7b6b8e6e5 on dahlia:python.


feat(pointcloud_preprocessor): use unordered_map in ring_outlier_filter

Related Issue(required)

Description(required)

If the index of the ring is greater than 127, the node will die due to out_of_rage of the vector, so use unordered_map.
This is for

sensors with more than 128 rings in the future.
sensors with ring index of 1-128 (I'm not sure if it exists)

Review Procedure(required)

Related PR(optional)

https://github.com/tier4/autoware.iv/pull/2419
Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Read commit-guidelines
[x] Assign PR to reviewer

If you are adding new package following items are required:

[ ] Documentation with description of the package is available
[ ] A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[x] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR / build-and-test-pr: Required to pass before the merge.
Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

LGTM

@yukkysaito Please approve again.


mean: Tweak boot waiting

Should fix CI

Sigh, the ping wait thing is fine for now. I think we need to look into ways to ditch it though.  I think I'll work on that next.


Impossible to add turning restictions

I'm trying to add turning restrictions on this node
http://www.openstreetmap.org/node/3102914057
but looks like it's impossible. Arriving from Corso Regina, heading west, you can't turn left (to south).
There is no apparent way to indicate it.

Possibly related: clicking on the intersecting oneway road doesn't bring up the turn restrictions
intersecting is http://www.openstreetmap.org/way/154889905, which is circular. If I manually cut the circular way at some  other point, everything works

This bug happens when the intersection vertex is also the start and end node on the (circular) intersecting way. Fixing this now..

Hi,
I re-tried adding the turn-left restriction when reaching the node from Corso Regina Margherita heading west, but still find out how...
Massi

The fix hasn't yet been released yet to the main OpenStreetMap site.  I will probably do a release soon.
But for now, can you try to use the latest iD in the link below and let me know if it allows the turn restriction?
https://openstreetmap.us/iD/master/

Hi,
looks like working :)
Massi


Dev and Staging centrals are out of sync with the Ballerina central

Description:
Some packages are missing in the dev or staging central
This is mainly due to the workflow gets timeouted when pushing the package
https://github.com/ballerina-platform/module-ballerina-http/actions/runs/11357865904/job/31608354345#step:10:7582
Describe your task(s)
Sync central with dev and staging

Seems like we need to increase the timeout and add a retryConfig to the central client in Ballerina CLI. @azinneera FYA

Any update on this? We are seeing the timeout occurring today as well.

Getting the same issue when releasing patches for update 5 and 6:
https://github.com/ballerina-platform/module-ballerina-http/actions/runs/12193309976/job/34015356182#step:10:10223
https://github.com/ballerina-platform/module-ballerina-http/actions/runs/12193311104/job/34015358969#step:10:10225
But the packages are actually pushed to the central. Due to this fact, we have to release the github packages separately


Handling unexpected constant types

As discussed here https://github.com/vimeo/psalm/pull/2612
This PR tries to handle unexpected constant types.
When I looked at the codebase, it appears they don't use the assert() function so I tried to handle it differently.

For DocBlock.php
There is a string type param(with strict types), which means it failed if $constant_value was not a string. However, the function seems to fallback on $message if there was no constant. I figured I would do the same when constant() return anything but a string
For Configuration.php
The constant value is used to push to ini_set, which will fail if given anything but a string. The fetching of the constant value is also optional. I checked if the constant value was anything that could be casted to string(scalar or null). If not, I consider the constant as not defined

Those two fixes are subjective, not sure I fixed them the best way possible

Codecov Report

Merging #1 into master will decrease coverage by 0.02%.
The diff coverage is 42.85%.

@@             Coverage Diff              @@
##             master       #1      +/-   ##
============================================
- Coverage     85.46%   85.43%   -0.03%
- Complexity     3850     3853       +3
============================================
  Files           156      156
  Lines          9720     9725       +5
============================================
+ Hits           8307     8309       +2
- Misses         1413     1416       +3

Impacted Files
Coverage Δ
Complexity Δ

src/Util/Annotation/DocBlock.php
92.38% <0%> (-1.34%)
66 <0> (+1)

src/Util/Configuration.php
96.79% <100%> (+0.01%)
188 <0> (+2)
: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 5e523bd...d1e6fa0. Read the comment docs.

Seems like the test coverage decreased slightly. I checked but I have no idea how to reach Docblock::parseAnnotationContent as it's private and not explicitly called.
Either there's some kind of magic or it's not used...


Image size

I'm wrestling with my Docker image file size, and noticed that Huginn continues to expand. Virtual size - now 2.5G - went up 1G since I last checked it a couple of months ago.
CONTAINER ID   IMAGE          PORTS                                      NAMES  SIZE
6cc7feb88104   huginn/huginn  <IP_ADDRESS>:3000->3000/tcp, :::3000->3000/tcp  huginn 20MB (virtual 2.53GB)

Playing around, I found this dive utility, and ran it this way:
docker run --rm -it -v /var/run/docker.sock:/var/run/docker.sock  -v $(which docker):/bin/docker wagoodman/dive:latest 8c8139adb5fe
and it reports:
Image name: 8c8139adb5fe
Total Image size: 2.5 GB
Potential wasted space: 1.7 GB
Image efficiency score: 66 %

Any chance this might be some low-hanging-fruit savings?

Came here looking for reports of anyone not being able to run the latest images, and when I checked I could see that it too was huge at 2.5G - I don't have any other images that are anywhere close to that big. The latest image just won't stay running for me , it just starts and stops repeatedly. This was logged repeatedly:
/scripts/setup_env: line 82: /tmp/.env.example: Permission denied
I had to go back to an image from 16 days ago to get back stability without that error and checking its size it's still quite big

Portainer shows the image layers and sizes.
The largest 4 are as below.
I have no idea what they are!
8 | 403.8 MB | RUN /scripts/prepare
10 | 388.8 MB | RUN /scripts/standalone-packages
20 | 803.3 MB | \|1 UID=1001 RUN umask 002 && git init && bundle config set --local path vendor/bundle && bundle config set --local without 'test development' && APP_SECRET_TOKEN=secret DATABASE_ADAPTER=mysql2 ON_HEROKU=true bundle install -j 4
24 | 843.9 MB | \|1 UID=1001 RUN umask 002 && APP_SECRET_TOKEN=secret DATABASE_ADAPTER=mysql2 ON_HEROKU=true bundle exec rake assets:clean assets:precompile && chmod g=u /app/.env.example /app/Gemfile.lock /app/config/ /app/tmp/ && chown -R "$UID" /app