[Array + JSON handling] Query nested data from objects from Mongo

First actual queries running an access request on nested fields. Probably easier to do this on nested objects than  on arrays as a first step. Also, going to use Mongo as the base implementation for json and array types since they’re simpler to express in mongo. We’ll extend these to different SQL databases as their implementation allows.

Mongo nested value queries.
Generic interface for nested value access Queries

@pattisdr and I were discussing yesterday an ideal way to handle data categories within (potentially infinitely) nested objects. Initially there was concern around what to do when the data categories of a field within an object conflict with the data category of the object itself.
Some options we considered were:

fields within objects automatically inherit the object's data categories,
fields within objects must be manually tagged with data categories and anything without a tag is ignored

Ultimately we understood that for a seamless developer experience objects and nested objects should function no different to table joins within the mapping. For example:
{
    "user": {
        "id": 1,
        "username": "joe",
        "comments": [
            {
                "id": 2,
                "content": "hello i am joe"
            }
        ],
        "tags": [{
            "id": 3,
            "name": "person",
            "created_at": 12345
        }, {
            "id": 4,
            "name": "likes-to-comment",
            "created_at": 678910
        }]
    }
}

can be represented by
dataset:
  - fides_key: example
    name: Users
    collections:
      - name: user
        fields:
          - name: id
            data_categories: [system.operations]
            fidesops_meta:
              primary_key: True
          - name: username
            data_categories: [user.provided.identifiable]

      - name: tags
        fields:
          - name: id
            data_categories: [system.operations]
            fidesops_meta:
              primary_key: True
          - name: user_id
            data_categories: [system.operations]
            fidesops_meta:
              references:
                - dataset: example
                  field: user.id
                  direction: to
          - name: name
            data_categories: [user.derived]
          - name: created_at
            data_categories: [user.derived]

or
dataset:
  - fides_key: example
    name: Users
    collections:
      - name: user
        fields:
          - name: id
            data_categories: [system.operations]
            fidesops_meta:
              primary_key: True
          - name: username
            data_categories: [user.provided.identifiable]
          - name: tags
            # data_categories: [user.derived]
            fidesos_meta:
              data_type: object[]
            - fields:
              - name: id
                data_categories: [system.operations]
              - name: name
                data_categories: [user.derived]
              - name: created_at
                data_categories: [user.derived]

When you frame things like this the clear choice is to disallow specifying data categories on any object itself, and only allow data categories at field level.

Yes, that makes sense and is consistent with our general principle for fidesctl evaluations: fields do not "inherit" the declarations of their parent, they are just composed. So for example, if you have this object:
dataset:
- fides_key: eval_example
  collections:
  - name: collection_example
    data_categories: [system]
    fields:
    - name: field_one
       data_categories: [user.provided]
       fields:
       - name: field_two
          data_categories: [user.derived]

In this case, the evaluation logic has two potential options:

Treat the eval_example dataset as just being system data, since that's the top-level category
Treat the eval_example dataset as being composed of system, user.provided, and user.derived data, as there are nested elements with those declarations

We do the latter- individual fields must specify their data categories individually, they don't inherit anything from parents. So this looks good to me.
HOWEVER: there's nothing preventing us from stating that a nested object field might have a data category declaration, right? The language syntax supports it, so you'd need to be adding a net-new validation error in fidesops or something to prevent that

HOWEVER: there's nothing preventing us from stating that a nested object field might have a data category declaration, right?

You're right, good spot — we should add an extra validation error here that's raised when object types within datasets are given data categories.

@seanpreston, product question, are there preferences for how nested data might be returned to the user?
A) In its original form or B) With nested fields flattened?  I might expect A personally, but not strongly.
See example below:
Say a postgres database has a  users collection with one object field contact_information.  Say we want the alt_email field.
dataset:
  - fides_key: postgres_example
    name: Postgres DB
    collections:
      - name: users
        fields:
          - name: contact_information
             fields:
               - name: ssn
               - name: phone
               - name: alt_email

A) Would JSON data returned to the user be shaped in its original structure?
{
   "postgres_example: users": [
    {
      "contact_information": {
            "alt_email"<EMAIL_ADDRESS>      }
    }]
}

B) or would it be flattened?
{
   "postgres_example: users": [
<EMAIL_ADDRESS>  ]
}

In my opinion, I'd expect A as a user, but will defer to product on this one

thank you for weighing in @eastandwestwind!

I would expect the response to be returned in its original format (A).....but for my own education, is there any other advantage to receiving the response back in a flattened format? ie: easier to parse for receiving server or something?

I'm also in favour of A here, mainly because:

it's inconsistent with other parts of the returned data if some is flattened and some is not,
the end user (ultimate consumer) doesn't care whether the data is stored in a nested field within the DB, as far as they're concerned that's an implementation detail of whoever stored it, and the hierarchy of the data makes it more consumable.

Advantages to B could be for instance if identity propagation needed the data in that format for legacy reasons. It might also be less costly for us to deliver as the keys are stored that way during part of the traversal. Is there anything else?

Thank you! A it is!


Changes to Handling errors

There are some changes suggested in https://github.com/ballerina-platform/ballerina-spec/issues/489 and https://github.com/ballerina-platform/ballerina-spec/issues/350 issues.
This issue is created to track these changes.

Fixed in #27284, merged via #27652.

Fixed in #27284, merged via #27652.


Exception (wrong-type-argument number-or-marker-p  language)

I get this error in *elfeed-log*
[2018-07-01 07:51:10] [error]: https://blog.benjojo.co.uk/rss.xml: (wrong-type-argument number-or-marker-p  language)
Seems that this is the faulty entry:
     <item>
      <title>Lessons from APL, a "lost" language
</title>
      <link>https://blog.benjojo.co.uk/post/2014-05-26-APL-the-lost-language.md</link>
      <description>Lessons from APL, a "lost" language
===

A few weeks back the forum that I go on often held a obfuscation challenge, and people joked around about submitting a entry that was coded in APL.

APL (A Programming Language) is a programming language that w</description>
      <author>Ben Cox</author>
      <pubDate>Mon, 26 May 2014 19:03:53 +0000</pubDate>
    </item>

I'm unable to reproduce this issue for that feed. The sample you
provided looks fine, and the feed itself also seems to be fine. The
encoding of the carriage return and line feed (
) is strange,
but it's perfectly legal and shouldn't cause any issues.
What version of Elfeed, Emacs, and operating system are you using?

debian testing, emacs-snapshot (27.0.50), elfeed 3.0.0
the issue is reproducible every time I refresh the feed. Maybe something is corrupted in db?

Oh, I found the reason for that. Some time ago I have somehow got elfeed db corrupted. All unicode symbols were written escaped (like "\307\204\304\145..."). I have repaired db using simple python script and got wrong with the quotes. Sorry for bothering you. Elfeed rocks, btw!


python311Packages.spacy: Use Cython 0.x

Description of changes
spaCy is not yet compatible with Cython 3. This PR adds cython_0 as a nativeBuildInput. Before this change, it seemed to be using Cython 3 propagated from another dependency.
Things done

Built on platform(s)

[x] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 306905 run on aarch64-linux 1

12 packages built:

python311Packages.fastai
python311Packages.fastai.dist
python311Packages.spacy
python311Packages.spacy-lookups-data
python311Packages.spacy-lookups-data.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.spacy.dist
python311Packages.textacy
python311Packages.textacy.dist
python311Packages.textnets
python311Packages.textnets.dist

Result of nixpkgs-review pr 306905 run on x86_64-linux 1

12 packages built:

python311Packages.fastai
python311Packages.fastai.dist
python311Packages.spacy
python311Packages.spacy-lookups-data
python311Packages.spacy-lookups-data.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.spacy.dist
python311Packages.textacy
python311Packages.textacy.dist
python311Packages.textnets
python311Packages.textnets.dist

Result of nixpkgs-review pr 306905 run on x86_64-darwin 1

2 packages failed to build:

python311Packages.textnets
python311Packages.textnets.dist

10 packages built:

python311Packages.fastai
python311Packages.fastai.dist
python311Packages.spacy
python311Packages.spacy-lookups-data
python311Packages.spacy-lookups-data.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.spacy.dist
python311Packages.textacy
python311Packages.textacy.dist

Looking at nix-tree, the dependency propagating cython is pkgs/development/python-modules/cymem/default.nix. Seems potentially conflicting, yet another issue with the "build-time propagation"

Looking at nix-tree, the dependency propagating cython is pkgs/development/python-modules/cymem/default.nix. Seems potentially conflicting, yet another issue with the "build-time propagation"

Fixed in https://github.com/NixOS/nixpkgs/pull/306990


Obsoleted axios raises 3 high severity vulnerabilities

npm audit raises the following 3 high vulnerabilities:
axios  <=0.21.1
Severity: high
Incorrect Comparison in axios - https://github.com/advisories/GHSA-cph5-m8f7-6c5x
Server-Side Request Forgery in Axios - https://github.com/advisories/GHSA-4w2v-q235-vp99
Depends on vulnerable versions of follow-redirects
No fix available
node_modules/bitcoin-simple-rpc/node_modules/axios
  bitcoin-simple-rpc  *
  Depends on vulnerable versions of axios
  node_modules/bitcoin-simple-rpc

follow-redirects  <1.14.7
Severity: high
Exposure of sensitive information in follow-redirects - https://github.com/advisories/GHSA-74fj-2j2h-c42q
No fix available
node_modules/bitcoin-simple-rpc/node_modules/follow-redirects
  axios  <=0.21.1
  Depends on vulnerable versions of follow-redirects
  node_modules/bitcoin-simple-rpc/node_modules/axios
    bitcoin-simple-rpc  *
    Depends on vulnerable versions of axios
    node_modules/bitcoin-simple-rpc

3 high severity vulnerabilities

It is recommended to update it to v0.21.4

Latest version shipped with axios 0.24.0: https://www.npmjs.com/package/bitcoin-simple-rpc


Movement controls hint shown at the wrong place

They're shown somewhere a few metres up the middle of the map, instead of right at the start, is their position hard-coded? Maybe we can request an entity for these control hints on the map (also for the jump one) so that we don't need change them in the code when the map changes.

Fixed by @asymetrique in 139b775


Crash on Xamarin Forms ListView with CachedImage: 'Cannot access a disposed object'

I have a list of about 100 objects all with an image loaded asynchronously with the forms:CachedImage XAML property. The ListView has the CachingStrategy set to "RecycleElement". If it's set to RetainElement then there are no problems but performance drops.
Here's how to produce the problem: on Android if I scroll really fast down the list it'll eventually throw a "Cannot access a disposed object" exception.  This doesn't happen on UWP and I've yet to try it on iOS.
At this point I assume the CachedImage is sending out an HTTP request and having it come back to the cell when it's been disposed/recycled. I've tried running CachedImage.Cancel() just about everywhere I could (such as when the cell leaves the screen or reloads the data), but that doesn't seem to do anything.
I'm open to any suggestions or further bug testing if you need.

I'm getting the same error with 2.2.8 with the same setup. Images in a RecyclerView on Android -> scrolling very fast -> crash 'Cannot access a disposed object'.

@mattjennings @slarti-zak Does image load correctly afterwards? Or maybe application crashes?

The result is that the application crashes with the message 'Cannot access a disposed object'. I am currently trying to set up a test project to reproduce the bug.
While I was not yet able to reproduce this bug, I have produced another bug. Using a MvxImageLoadingView with an applied CircleTransformation in a MvxRecyclerView sometimes causes the transformation to not be applied. (See bugged images at the end.)
You may find the test project here. The bug occurs when scrolling through the images (up and down, querying the same image multiple times). It seems that a half finished version of the image (loading done, transformation not) is put into the cache when the load task of the image view gets cancelled because the image view gets recycled in the list.
The bug is easier to reproduce with a slower internet connection (not too slow, images still have to load). You can also see that sometimes the images are in the wrong order (incorrect image loaded).
There is no exception thrown in the log.
Images:

The "wrong order" issue may be related to this: https://github.com/luberda-molinet/FFImageLoading/issues/506. Thanks to @EGoverde - it's fixed now.
As to the rest I'll try to debug it soon.

The "wrong order" issue may be related to this: https://github.com/luberda-molinet/FFImageLoading/issues/506. Thanks to @EGoverde - it's fixed now.
As to the rest I'll try to debug it soon.

Glad to hear. I will try to get a crashing example, with regards to the issue topic, ready on the weekend. I have already experienced it within the example project but the current state seems to be working. Not sure yet what causes the crash.

Hi @slarti-zak,
It took a while (I've been quite busy), but I made entirely new MvxCachedImageView which is far superior than old depreciated MvxImageLoadingView.
There are also samples for it: https://github.com/luberda-molinet/FFImageLoading/tree/master/samples/ImageLoading.MvvmCross.Sample
The issue you described there is fixed when using it too. It's available in the latest prerelease package.

Thanks you for your example, it helped a lot!

That's Great! I had yet to get back into this but will now try out (try to break) the new stuff.

We noticed this same issue last week when scrolling a rather large ListView. When is the next stable release of the package expected to be released?


Apex Custom Data Provider background colour not displayed

Who is the bug affecting?
Developer (me) currently
What is affected by this bug?
Custom background colours defined in Apex Custom Data Provider don't show up on doughnut chart
When does this occur?
On loading chart
Where on the platform does it happen?
Chart component from Managed Package
How do we replicate the issue?
Set a custom background colour property in the Custom Data Provider
Expected behavior (i.e. solution)
Custom background colours should be shown
Other Comments
I see this error in console:

aura_proddebug.js:61421  WARNING: SecureElement: [object HTMLStyleElement]{ key: {"namespace":"lwcc"} } does not allow setting the type attribute, ignoring!

possibly the method for injecting the style is no longer allowed?
Additionally I would like to use an array for this property but I'm not sure the syntax.
Here's my example code:
global inherited sharing class MyCustomChartDataProvider extends lwcc.ChartDataProvider {
  public override void init(Object initParameter) {}

  public override List<lwcc.ChartDataProvider.ChartData> getData() {
    List<lwcc.ChartDataProvider.ChartData> chartDataList = new List<lwcc.ChartDataProvider.ChartData>();
    lwcc.ChartDataProvider.ChartData data = new lwcc.ChartDataProvider.ChartData();
    data.detail = [95.0, 5.0];
    data.labels = ['Label 1','Label 2'];
    data.bgColor = '["rgb(100,100,100)","rgb(200,200,200)"]';
    chartDataList.add(data);
    return chartDataList;
  }
}

Hello, we do not continue to apply fixes to this project, but we only act as code reviewers and release managers.
You're welcome to create a Pull Request with your suggested changes, that we'll review. If approved, the Pull Request will be included in the open-source (repository) version of the package. However, new changes won't be part of the Managed Package.
Hope it's clear and you understand it. Happy to see your PRs coming in the future.
Best regards,
@scolladon & @victorgz


Show the commit message for a diff

When reviewing a PR, often I'd love to know what the commit message was for a particular diff section.

Would be amazing to have something like that.

I think you're talking about diffs in a PR. It's a rather complex feature given that we'd have to rebuild a blame-like view for all the files in a PR.

Too big. It's best to suggest it to GitHub or create a separate "Blame view for PRs" extension or ask for one here: https://github.com/sindresorhus/module-requests


feat: Reuse rule ID's 950011, 950012 and 950018 for consistent operation

We reused paranoia level skip rules 950010, 950011, and 950018.
These rule identifiers were in use in CRSv2, so we used different identifiers in CRSv3 to avoid conflicts (950020, 950021 and 950022). In CRSv4, for the sake of a consistent structure, we use these identifiers again, so that the skip rules of PL can be defined in the same way for all files (rule_id MOD 100 < 20).
Rule id 950130 has been kept - this ID also was introduced at first rule to avoid the collision.

Thank you

@dune73: Shouldn't we add this into the "new" changelog as an important change?

No. This only affects people attempting to run CRSv2 and CRSv4 in parallel.


allow compute status without signature

requires new provider.
New change:  added optional sign parameter to status function:
/**

Returns information about the status of all compute jobs, or a single compute job.
@param  {Account} consumerAccount The account of the consumer ordering the service.
@param  {string} did Decentralized identifier.
@param  {string} jobId The ID of the compute job to be stopped
@param  {boolean} sign If the provider request is going to be signed(default) (full status) or not (short status)
@return {Promise<ComputeJob[]>} Returns the status
*/
public async status(
consumerAccount: Account,
did?: string,
jobId?: string,
sign = true
): Promise<ComputeJob[]> {

Making high priority to prevent further slippage of quality.


Bump dev-master of branch-alias to 2.1

This is for add new release with tag 2.0 :)

Thanks for your contribution. Though, we haven't released version 2.0 yet and we don't need the 2.1 until we actually start adding something new after the 2.0 release.


Dimension subscripts for two dimensional array

[Enter feedback here]
Hi,
It would be helpful if you specify which one of the dimensions is the row and which one is the column, especially for 2 dimension arrays like int[,] array = new int[4, 2];
Based on the info in https://www.tutorialspoint.com/csharp/csharp_multi_dimensional_arrays.htm, it is int[row,column]
Also it would help if we know how the array data is saved in memory, by row or by column.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 76865f9f-cc54-3d13-278a-6d37fa57dc40
Version Independent ID: ee3fc35c-843a-8947-f202-56e9eeb3b0e3
Content: Multidimensional Arrays - C# Programming Guide
Content Source: docs/csharp/programming-guide/arrays/multidimensional-arrays.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for raising this @Soran-M
I've added this to our backlog to address.

Fixing as part of the work on collection expressions #36356


HelmRelease, Kustomization DependsOn

Currently HelmRelease only allowed to depends on HelmRelease, Kustomization only allowed to depends on Kustomization.
Is it a better solution if we can remove the type restriction ?

Is it a better solution if we can remove the type restriction ?

This implies a hard coupling between the two APIs (Kustomization and HelmRelease), I am not in favour of this.
You can add HelmReleases to the Kustomization heath checks, this way dependsOn can be used to create a dependency graph between HelmReleases and other Kubernetes objects.


alunochromos.txt

added alunochromos

@Meinards15 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@Meinards15 You might miss my previous comment. Please react.

@Meinards15 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Uneven surface profile preference sets slope preferences

Actually it's not being used for anything at the moment, but it's being saved to a variable called steepTerainOpts, which is misleading.  From discussion, the question should be used to set surface type preferences.


ant-design-pro 问题跟踪

基于本项目导出的关于 ant-design-pro 相关的问题跟踪记录。

issue: https://github.com/ant-design/ant-design-pro/issues/6190

issue: https://github.com/ant-design/pro-components/issues/2016
PR: https://github.com/ant-design/pro-components/pull/2115


No option to turn off logging

Expected
No bridge.log file created and updated by default.
Actual
The log file created with severity level Info (bridge.json option 'loggerLevel').
Solution
Create loggerLevel option No, use it by default.

Option value should be None.


Can't import pyiceberg.catalog

Apache Iceberg version
0.6.0 (latest release)
Please describe the bug 🐞
Note that the previous version 0.5.1 does NOT have this issue.
Running on Mac OS Monterey 12.6.7, Spyder IDE (Python 3.9.7 64-bit | Qt 5.9.7 | PyQt5 5.9.2 | Darwin 21.6.0 )
When importing:
import pyiceberg.catalog
the following exception occurs:
Traceback (most recent call last):
File ~/Python/data-calc-service/untitled0.py:9
import pyiceberg.catalog
File ~/opt/anaconda3/lib/python3.9/site-packages/pyiceberg/catalog/init.py:45
from pyiceberg.table import (
File ~/opt/anaconda3/lib/python3.9/site-packages/pyiceberg/table/init.py:87
from pyiceberg.table.name_mapping import (
File ~/opt/anaconda3/lib/python3.9/site-packages/pyiceberg/table/name_mapping.py:38
class MappedField(IcebergBaseModel):
File ~/opt/anaconda3/lib/python3.9/site-packages/pyiceberg/table/name_mapping.py:40 in MappedField
names: List[str] = conlist(str, min_length=1)
File ~/opt/anaconda3/lib/python3.9/site-packages/pydantic/types.py:869 in conlist
return Annotated[List[item_type], annotated_types.Len(min_length or 0, max_length)]
TypeError: Len() takes no arguments

seems like an issue related to pydantic library. What version of it do you have installed?
Here's my env setup using make install:
(venv) ➜  iceberg-python git:(main)  pip list | grep pydantic
pydantic                   2.6.1
pydantic_core              2.16.2

My python version is 3.9.7. My colleague told me that pyiceberg works with 3.9.11 and later. It didn't work with 3.9.7 in his env as well. I'm trying to upgrade Python and will update the ticket.

Addressed in #526


Addon for Lua

Extend export capabilities to export to Lua (Load Impact's current cloud generation language) scripts.

Done.


Support HTTP/2

We'd like CocoaSPDY to support HTTP/2, since it is the future and the use of SPDY will likely taper off over time. An implementation here would potentially come earlier than an official Apple version and could provide additional functionality, such as push requests and richer metadata. At present, there are no concrete plans for implementing HTTP/2 in CocoaSPDY, but if there is community interest please indicate it here.

It would be great to support HTTP/2, I sure, main browsers and services will be removing support SPDY in favor of HTTP/2 in the near future.

I'm interested in this. Apple has provided support for HTTP 2 in iOS 9, but as you know @kgoodier, I'm interested in server push which Apple does not seem to support (as far as I can find).
+1 to @griganskiy's comment. It looks like Chrome is removing SPDY early next year from chrome.
Does anyone have an idea of how much work this would take? I've heard the protocols are similar, but am finding it difficult to get a concrete list of differences.

You're right, Apple does not support server push with either their SPDY or HTTP/2 implementations. I wouldn't count on that happening either.
I haven't thoroughly costed this, but there are a lot of similarities. That's certainly very helpful, but I suspect the devil will be in the details. Headers and priorities are different and more complex, and there's the issue of getting HPACK in the code. As far as I know, everything else is basically the same or slightly different. It's still a big undertaking and difficult to justify with iOS 9 supporting HTTP/2.
I think the main argument for HTTP/2 in this project is if you don't control your servers, or have an aggressive SPDY deprecation plan to manage tech debt, or expect a large user base on iOS 8 and earlier, but still want push or require rich metadata.  If you control your servers and clients, SPDY is still a viable option and HTTP/2 offers only incremental technical improvements.


[Resteasy-1863]:SseEventSourceImpl should not close http client engine

@jimma actually closing the CloseableHttpResponse will have the same effect as setting the connection:close header right ?

The connection will be closed without blocking but not added back to the pool.
Same situation if the server-side close the connection.
the user connection pool will lost connections affecting the performance of other standard rest request.

What about using URLConnectionEngine in ApacheHttpClient4Engine just to invoke SSE request ? We will take adavanatge of JDK mechanism and not affect the user connection pool if a connection is being discarded because the stream cannot be read in hurry mode.

@jimma actually closing the CloseableHttpResponse will have the same effect as setting the connection:close header right ?

If you mean force to close the eventsouce , it is the same effect.  But It's a bit different when the event sink is closed in server and terminating chunk is received before close the eventsouce , the connection can be reused, it's keep alive.

The connection will be closed without blocking but not added back to the pool.

Yes. That's the not expensive to do this.

Same situation if the server-side close the connection.

Do you mean the even sink is closed before event source is closed ? This really depends the server side implementation. If http header is sent with "keep-alive" , server won't close the connection.

the user connection pool will lost connections affecting the performance of other standard rest request.

For the chunked inputstream it has to discard the connection. And it seems the efficient way to do this.

What about using URLConnectionEngine in ApacheHttpClient4Engine just to invoke SSE request ?
We will take adavanatge of JDK mechanism and not affect the user connection pool if a connection is being discarded because the stream cannot be read in hurry mode.

I only implement this for ApacheHttpClientEngine now and will have a look at URLConnectionEngine. What do you mean by not affect the user connection pool ? For jdk url connection,  connection is discarded(server socket close) and won't return back to pool.

But It's a bit different when the event sink is closed in server and terminating chunk is received before close the eventsouce , the connection can be reused, it's keep alive.

Actually  I dont think that closing the CloseableHttpResponse will allow to reuse the TCP connection event if the terminating chunked has been received. According to the doc and the code it seems that the connection is always discarded and not added back to the pool to be reused.
That's the issue encountered by the jersey guys here https://github.com/jersey/jersey/issues/3358
So If the connection is always discarded not added back to the pool, later a new TCP connection will have to be established and this is the expensive operation.

A simple code snipet as an example off what I was saying above:
public static void main(String[] args) {
	PoolingHttpClientConnectionManager poolingHttpClientConnectionManager = new PoolingHttpClientConnectionManager();
	poolingHttpClientConnectionManager.setMaxTotal(50);
	poolingHttpClientConnectionManager.setDefaultMaxPerRoute(50);
	RequestConfig.Builder requestConfigBuilder = RequestConfig.custom();
	requestConfigBuilder.setSocketTimeout(0);
	requestConfigBuilder.setConnectTimeout(0);
	requestConfigBuilder.setConnectionRequestTimeout(0);
	CloseableHttpClient closeableHttpClient = HttpClientBuilder.create()
			.setConnectionManager(poolingHttpClientConnectionManager)
			.setDefaultRequestConfig(requestConfigBuilder.build()).build();

	HttpGet httpGet = new HttpGet("http://localhost:9998/books/1");
	httpGet.addHeader("Connection", "keep-alive");
	httpGet.addHeader("Authorization", "Basic QURNSU46MTE=");
	httpGet.addHeader(HttpHeaders.ACCEPT, ContentType.APPLICATION_JSON.getMimeType());

	// No connection in the pool so a new TCP connection will be created and
	// it will be expensive (see 'execution time').
	// After closing the stream the connection will remain in the pool to
	// be reused (see 'pool stats').
	long start = System.currentTimeMillis();
	try {
		closeableHttpClient.execute(httpGet).getEntity().getContent().close();
		System.out.println("pool stats: " + poolingHttpClientConnectionManager.getTotalStats());
		System.out.println("execution time: " + (System.currentTimeMillis() - start));
	} catch (IOException e) {
		e.printStackTrace();
	}

	// One connection in the pool so it will be reused and since there is no
	// need to create any new TCP connection it will be fast (see 'execution
	// time').
	// After closing the stream the connection will remain in the pool to
	// be reused (see 'pool stats').
	start = System.currentTimeMillis();
	try {
		closeableHttpClient.execute(httpGet).getEntity().getContent().close();
		System.out.println("pool stats: " + poolingHttpClientConnectionManager.getTotalStats());
		System.out.println("execution time: " + (System.currentTimeMillis() - start));
	} catch (IOException e) {
		e.printStackTrace();
	}

	// One connection in the pool so it will be reused and since there is no
	// need to create any new TCP connection it will be fast (see 'execution
	// time').
	// After closing the stream the connection will not be added back to the
	// pool to
	// be reused (see 'pool stats').
	start = System.currentTimeMillis();
	try {
		closeableHttpClient.execute(httpGet).close();
		System.out.println("pool stats: " + poolingHttpClientConnectionManager.getTotalStats());
		System.out.println("execution time: " + (System.currentTimeMillis() - start));
	} catch (IOException e) {
		e.printStackTrace();
	}

	// No connection in the pool so a new TCP connection will be created and
	// it will be expensive (see 'execution time').
	// After closing the stream the connection will remain in the pool to
	// be reused (see 'pool stats').
	start = System.currentTimeMillis();
	try {
		closeableHttpClient.execute(httpGet).getEntity().getContent().close();
		System.out.println("pool stats: " + poolingHttpClientConnectionManager.getTotalStats());
		System.out.println("execution time: " + (System.currentTimeMillis() - start));
	} catch (IOException e) {
		e.printStackTrace();
	}
}

Hi @jimma
You're right it works  as you described it :+1:
What about merging the content of new closeHttpResponse() method into existing releaseConnection
instead of having both close() and  closeHttpResponse()?

What about merging the content of new closeHttpResponse() method into existing releaseConnection
instead of having both close() and closeHttpResponse()?

We have https://issues.jboss.org/browse/RESTEASY-889 to take care.  That's why I added another closeHttpResponse method.

Hi @jimma, @asoldano ,
This PR is OK for me.
Thanks you


My host machine supports KVM but after I run the docker image, the docker container does not support it?

Hi, on my CentOS machine, I run emulator-check, I got
accel:
0
KVM (version 12) is installed and usable.
accel

However, when I run emulator-check in the docker container, I got
accel:
8
/dev/kvm is not found: VT disabled in BIOS or KVM kernel module not loaded
accel

And I cannot run x86_64 emulator, can you help me with this?

Hi @Pezhin, it's exactly as the log said, you should try either of below:

Enable VT (Virtualization Technology) in BIOS settings
Load KVM module on the host: modprobe kvm_intel

Hi @thyrlian , I think on my host machine, the KVM module is already loaded but after I run the docker container, the kvm is not enabled in the docker container.
Any idea how I can fix this?

@thyrlian I re-read the doc and found that
docker run -it --privileged -v $(pwd)/sdk:/opt/android-sdk:ro thyrlian/android-sdk /bin/bash
the privileged is needed and also, we have to volume/dev/kvm.
I can run the emulator via this command
docker run -d --privileged -v /dev/kvm:/dev/kvm  -p 5901:5901 -p 2222:22 -p 5037:5037 -v /root/:/opt/android-sdk thyrlian/android-sdk-vnc

Congratulations, so everything is fine now?

@thyrlian yes. It runs very slowly tho but it runs. I will close the issue for u

Weird, the reason to use x86 emulator is because of its performance.    Can you maybe try with a different host machine?  Or try to increase the CPU share constraint and memory constraints?


python312Packages.opower: 0.4.3 -> 0.4.4

Diff: https://github.com/tronikos/opower/compare/refs/tags/v0.4.3...v0.4.4
Changelog: https://github.com/tronikos/opower/releases/tag/v0.4.4
Description of changes

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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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.

Result of nixpkgs-review pr 307943 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.opower
python311Packages.opower
python311Packages.opower.dist
python312Packages.opower
python312Packages.opower.dist


Add protection against chat ddos attacks

After witnessing a chat DDOS attack on Fritzsharp's stream recently, I think it would be advisable to be prepared for that in the future.
The devchatterbot could attempt to recognize such a pattern and put the chat on slow, maybe even temporarily mute the "new follower" soundbyte and gif which will otherwise drown out the streamer.

It would seem that I always had the email verification protection turned on, and mods could turn on the /r9kbeta as-needed. Don't want games with repeating text to mess with people. Like everyone typing !join for example.


Improve expected value error messages

It wasn't very helpful to see just "number <= ... expected", I wanted to see exactly what number I was giving it.

Pull Request Test Coverage Report for Build 17

4 of 5 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.861%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/t.lua
4
5
80.0%

Totals

Change from base Build 16:
0.0%

Covered Lines:
216

Relevant Lines:
223

💛  - Coveralls


Add space="medium" to Container in the widget/notepad templates.

I added missing space props to Container in the widget/notepad templates.

Thanks! Merged into next in https://github.com/yuanqing/create-figma-plugin/pull/115


Recommend improvements to the "Run a Validator" documentation

Follow the steps documented on https://docs.shardeum.org/node/run/validator
Note down any issues or differences in the documented experience
Update the docs to reflect and account for your noted experiences
Send a PR

Is this still an issue so that I take it on @kenny-io

Hi @kaksv yes this issue is not assigned yet and thank you for your interest. However you have another issue in progress here let's wrap that one up before starting a new one.

Thank you. Let me have a look at it
On Mon, May 20, 2024, 4:30 PM Ekene Eze @.***> wrote:

Hi @kaksv https://github.com/kaksv yes this issue is not assigned yet
and thank you for your interest. However you have another issue in progress
here https://github.com/shardeum/network-status/pull/5 let's wrap that
one up before starting a new one.
—
Reply to this email directly, view it on GitHub
https://github.com/shardeum/shardeum-docs/issues/314#issuecomment-2120467715,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIMXRENNJDKVKAZ2AYNVJC3ZDH3HHAVCNFSM6AAAAABHUVN4OSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRQGQ3DONZRGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


[docs] Create NextJS 14 example with Tailwind CSS

Adding an example on how to use NextJS 14 with Material UI and Tailwind CSS, which also fixes the FOUC when overriding Material UI styles with Tailwind on SSR and client.

[x] I have followed (at least) the PR section of the contributing guide.

@oliviertassinari As we discussed on Twitter. I'm just not sure how to add labels to the PR though.

Thanks for creating the example. Don't we need the ThemeRegistry for emotion to work as expected as in the other next.js examples?

Thanks for creating the example. Don't we need the ThemeRegistry for emotion to work as expected as in the other next.js examples?

Yes. It's all inside mui-provider.

I can't see anywhere this code: https://github.com/mui/material-ui/blob/master/examples/material-ui-nextjs-ts/src/components/ThemeRegistry/EmotionCache.tsx this is why I asked.

@mnajdova Added the emotion cache as asked. Sorry the delay.

This would close #40802. However, since this PR has been open for a while and we now point to next for PRs, changing the base branch will be complicated with many changes. @orafaelfragoso, if you want to continue working on this, please create a new PR. I'll close this one.


bbox loss always 0 during training, and acc 100%, but no detection results.

/root/mmdetection/mmdet/datasets/custom.py:154: UserWarning: CustomDataset does not support filtering empty gt images.
'CustomDataset does not support filtering empty gt images.')
2020-12-26 22:09:25,810 - mmdet - INFO - load model from: torchvision://resnet50
2020-12-26 22:09:26,112 - mmdet - WARNING - The model and loaded state dict do not match exactly
unexpected key in source state_dict: fc.weight, fc.bias
2020-12-26 22:09:32,945 - mmdet - INFO - Start running, host: root@ligaoyang_6, work_dir: /root/mmdetection/work_dirs
2020-12-26 22:09:32,945 - mmdet - INFO - workflow: [('train', 1)], max: 12 epochs
2020-12-26 22:10:08,314 - mmdet - INFO - Epoch [1][50/527]	lr: 2.500e-03, eta: 1:13:37, time: 0.704, data_time: 0.049, memory: 3787, loss_rpn_cls: 0.1513, loss_rpn_bbox: 0.0000, loss_cls: 0.2045, acc: 98.5449, loss_bbox: 0.0000, loss: 0.3559
2020-12-26 22:10:41,882 - mmdet - INFO - Epoch [1][100/527]	lr: 2.500e-03, eta: 1:11:18, time: 0.671, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.1802, acc: 100.0000, loss_bbox: 0.0000, loss: 0.1802
2020-12-26 22:11:15,701 - mmdet - INFO - Epoch [1][150/527]	lr: 2.500e-03, eta: 1:10:21, time: 0.676, data_time: 0.009, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0905, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0905
2020-12-26 22:11:49,085 - mmdet - INFO - Epoch [1][200/527]	lr: 2.500e-03, eta: 1:09:22, time: 0.667, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0497, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0497
2020-12-26 22:12:22,456 - mmdet - INFO - Epoch [1][250/527]	lr: 2.500e-03, eta: 1:08:33, time: 0.668, data_time: 0.009, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0307, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0307
2020-12-26 22:12:55,585 - mmdet - INFO - Epoch [1][300/527]	lr: 2.500e-03, eta: 1:07:45, time: 0.663, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0214, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0214
2020-12-26 22:13:29,316 - mmdet - INFO - Epoch [1][350/527]	lr: 2.500e-03, eta: 1:07:12, time: 0.675, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0160, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0160
2020-12-26 22:14:03,046 - mmdet - INFO - Epoch [1][400/527]	lr: 2.500e-03, eta: 1:06:37, time: 0.674, data_time: 0.007, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0124, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0124
2020-12-26 22:14:36,250 - mmdet - INFO - Epoch [1][450/527]	lr: 2.500e-03, eta: 1:05:56, time: 0.664, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0098, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0098
2020-12-26 22:15:10,211 - mmdet - INFO - Epoch [1][500/527]	lr: 2.500e-03, eta: 1:05:26, time: 0.679, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0082, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0082
2020-12-26 22:15:28,147 - mmdet - INFO - Saving checkpoint at 1 epochs
2020-12-26 22:16:05,196 - mmdet - INFO - Epoch [2][50/527]	lr: 2.500e-03, eta: 1:01:59, time: 0.727, data_time: 0.049, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0062, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0062
2020-12-26 22:16:38,036 - mmdet - INFO - Epoch [2][100/527]	lr: 2.500e-03, eta: 1:01:31, time: 0.657, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0055, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0055
2020-12-26 22:17:11,769 - mmdet - INFO - Epoch [2][150/527]	lr: 2.500e-03, eta: 1:01:10, time: 0.675, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0048, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0048
2020-12-26 22:17:45,323 - mmdet - INFO - Epoch [2][200/527]	lr: 2.500e-03, eta: 1:00:45, time: 0.671, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0043, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0043
2020-12-26 22:18:18,367 - mmdet - INFO - Epoch [2][250/527]	lr: 2.500e-03, eta: 1:00:16, time: 0.661, data_time: 0.009, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0039, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0039
2020-12-26 22:18:52,190 - mmdet - INFO - Epoch [2][300/527]	lr: 2.500e-03, eta: 0:59:52, time: 0.676, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0034, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0034
2020-12-26 22:19:25,987 - mmdet - INFO - Epoch [2][350/527]	lr: 2.500e-03, eta: 0:59:26, time: 0.675, data_time: 0.008, memory: 3787, loss_rpn_cls: 0.0000, loss_rpn_bbox: 0.0000, loss_cls: 0.0031, acc: 100.0000, loss_bbox: 0.0000, loss: 0.0031
my train goes wierd, the bbox loss is 0 from the beginning, acc becomes 100% from the second batch. And the detection result in inference is empty! Please help
I use faster_rcnn_r50_fpn_1x_coco.py as config file

Please use the issue template of reimplementation questions. We are not able to do any help here with your current information.

I also encounter your issue when I use COCO format dataset. But when I create a new dataset like KittiTinyDataset in tutorial, instead of reorganizing my dataset into COCO format,  everything seems OK.

I also encounter your issue when I use COCO format dataset. But when I create a new dataset like KittiTinyDataset in tutorial, instead of reorganizing my dataset into COCO format,  everything seems OK.

Did you find a solution to this? I am having the same problem.

Did you find a solution to this? I am having the same problem.

Did you find a solution to this? I am having the same problem

Did you find a solution to this? I am having the same problem

Since no more information is provided, this issue will be closed. Feel free to create a new issue using the error template to report your bug.


Discussion on model hyperparameter optimization (HPO) with PyPOTS and NNI

Issue description
We can share experiences and discuss the problems encountered during HPO in this issue.

Hi there 👋,
Thank you so much for your attention to PyPOTS! You can follow me on GitHub to receive the latest news of PyPOTS. If you find PyPOTS helpful to your work, please star⭐ this repository. Your star is your recognition, which can help more people notice PyPOTS and grow PyPOTS community. It matters and is definitely a kind of contribution to the community.
I have received your message and will respond ASAP. Thank you for your patience! 😃
Best,
Wenjie

A simple tutorial of Hyperparameter Optimization（HPO）in PyPOTS

Environment

• OS: Windows 10

• IDE: Pycharm 2023.2 community

• Python version: 3.9.13

• Torch version: 2.3.0+cu121

• NNI version: 3.0

• Training service: local

Based on the GitHub repo Awesome_Imputation, run the SAITS Model HPO Experiments with the air_quality dataset as a tutorial：
Step1. Data preparation（Downloads and preprocessing）
Run the python scripts to download and preprocess the datasets. In Awesome_Imputation, the path to the script is as follows: time_series_imputation_survey_code/data_processing/gene_air_quality.py
If the script runs successfully, you can find the data files in the directory of :
_time_series_imputation_survey_code/data_processing/data_processing/data

Step2. NNI Configration（Search space and Experiments）
• Search space configuration:
The search space configuration file is in the directory of time_series_imputation_survey_code/PyPOTS_tuning_configs/SAITS/SAITS_AQI_tuning_space.json. In this file, you need to configure the hyperparameters to be optimized and the corresponding search space. You can also refer to the NNI documentation to modify it according to your needs.
{
	"n_steps": {"_type":"choice","_value":[24]},
	"n_features":  {"_type":"choice","_value":[132]},
	"epochs":  {"_type":"choice","_value":[200]},
	"patience":  {"_type":"choice","_value":[10]},
	"n_layers":  {"_type":"choice","_value":[1,2,3]},
	"d_model":  {"_type":"choice","_value":[64]},
	"d_ffn":  {"_type":"choice","_value":[64, 128, 256, 512, 1024]},
	"n_heads":  {"_type":"choice","_value":[1,2,4,8]},
	"d_k":  {"_type":"choice","_value":[32, 64, 128, 256]},
	"d_v":  {"_type":"choice","_value":[32, 64, 128, 256]},
	"dropout":  {"_type":"choice","_value":[0,0.1,0.2,0.3,0.4,0.5]},
	"attn_dropout":  {"_type":"choice","_value":[0,0.1,0.2,0.3,0.4,0.5]},
	"lr":{"_type":"loguniform","_value":[0.00005,0.01]}
}

• Experiments configuration:
The experiments configuration file is in the directory of time_series_imputation_survey_code/PyPOTS_tuning_configs/SAITS/SAITS_searching_config.yml. In this file, you need to configure the experiments information and devices used in the experiments. You can also refer to the NNI documentation to modify it according to your needs.
experimentName: SAITS hyper-param searching
authorName: WenjieDu
trialConcurrency: 1
trainingServicePlatform: local
searchSpacePath: SAITS_AQI_tuning_space.json
multiThread: true
useAnnotation: false
tuner:
    builtinTunerName: Random

trial:
	command: set enable_tuning=1 && pypots-cli tuning --model pypots.imputation.SAITS --train_set ..\..\data\air_quality\train.h5 --val_set ..\..\data\air_quality\val.h5
    codeDir: .
    gpuNum: 1

localConfig:
    useActiveGpu: true
    maxTrialNumPerGpu: 1
	gpuIndices: 0

It's worth noting that, in the command line, we need to use “set enable_tuning=1 &&” in Windows OS to set the enable_tuning parameter.
Step3. Run Experiments（Run and Monitoring）
You need to change the path to the .yml file in the terminal and run the experiments with nni tools.
cd SAITS && nnictl create -c SAITS_searching_config.yml --port 8080

And then, you can click the URL and monitor the trials in the Web UI.

Some Tips.
1.How to view the status of trials and log files?
You can click on the trials in the Web UI to monitor the status of trials. If the status is ”Failed”，you could read the error message in the log file to debug.

2.How to kill ports in Windows OS?
If you have stopped an experiment using the command 'nni stop' and run another experiment with the same port, you may encounter the error message 'Port 8080 is not idle.' This means you may need to use a command in CMD to kill the port.
You can search for the PID of the port with the following command:
cmd input : netstat -aon|findstr "8080"

You can kill the corresponding PID with the following command:
taskkill /T /F /PID

Detailed information can be found at this link
3.How to stop and resume an experiment?
You can use the 'nni stop' command to stop an experiment, and its status will change to 'STOPPED' in the Web UI. Only experiments with a status of 'DONE' will have access to trial information through the Web UI after they have been stopped. Therefore, if you want to restart an experiment with a 'STOPPED' status, you can use the 'nni resume' command. Detailed information can be found at this link
We can exchange more experiences and questions to learn from each other!


Crashes in ST3

Not really sure how to use this. I've cloned the repo into my ST3 Packages folder.
I see Cinema4d (.res Description) and Cinema4d Python API, as available syntaxes, but the API crashes ST3. Any advice or instructions much appreciated ;) thx David

in the cinema4D_pythonAPI.YAML-tmLanguage file, I've truncated the variables lines into 4 groups. There was an issu with ST2, maybe it's the same problem. Could you give it a try ?


Fixed issue with base class and implementations

Codecov Report

Merging #88 (f10a839) into main (7a7df8a) will decrease coverage by 0.50%.
The diff coverage is 98.27%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
- Coverage   98.87%   98.37%   -0.50%
==========================================
  Files          30       30
  Lines         266      677     +411
==========================================
+ Hits          263      666     +403
- Misses          3       11       +8

Impacted Files
Coverage Δ

...CodeGen.Core/TemplateLoaders/FileTemplateLoader.cs
100.00% <ø> (ø)

...er.Commons.StringHelpers/CasingStringExtensions.cs
100.00% <ø> (ø)

...CodeGen.CSharp/Model/CSharpConstructorParameter.cs
83.33% <83.33%> (ø)

...r.Commons.CodeGen.Mapping.Avro/AvroSchemaParser.cs
90.12% <90.12%> (ø)

...ure.DataAttributes/AutoNSubstituteDataAttribute.cs
100.00% <100.00%> (ø)

...tler.Commons.CodeGen.CSharp/CSharpCodeGenerator.cs
100.00% <100.00%> (ø)

...ler.Commons.CodeGen.CSharp/CSharpTemplateLoader.cs
100.00% <100.00%> (ø)

...antler.Commons.CodeGen.CSharp/Model/CSharpClass.cs
100.00% <100.00%> (ø)

....Commons.CodeGen.CSharp/Model/CSharpCompilation.cs
100.00% <100.00%> (ø)

....Commons.CodeGen.CSharp/Model/CSharpConstructor.cs
100.00% <100.00%> (ø)

... and 19 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Unit tests for Operator Hub integration

We need more unit tests coverage for Operator Hub integration. At the time of opening this issue, the unit test coverage for pkg/kclient/operators.go is nil.
Originally posted by @girishramnani in https://github.com/openshift/odo/pull/3564/files#r455591391
/area service-operators
/area testing

As far as I understand, I need to come up with a simple Operator to write the unit tests. I've tried doing so using operator-sdk but it didn't work out. And due to shortage of time, I've not spent more time learning about it.
If there's another approach to do it, I'd be glad to be corrected.

Very relevant. More unit tests needed for overall service package. We should ideally be working on this along with #4242.
/lifecycle frozen

operator hub integration is going to be reworked in v3 together with testing
Pclo


Fix MetadataFilter formatting in chat info side bar

Reported in https://github.com/Quansight/ragna/pull/499#discussion_r1745589754.

The snippet below does not honor the indentation
https://github.com/Quansight/ragna/blob/9221bb8d67b5002abd42a6fc985a017b7aa0072f/ragna/deploy/_ui/central_view.py#L208-L210

form ragna.core import MetadataFilter

print(MetadataFilter.and_([MetadataFilter.in_("document_name", ["foo.md"])]))

AND(
  IN('document_name', ['foo.md']),
)

We can just wrap into <pre>...</pre> to keep our formatting.


Build Error: No matching function for call in batch_parameter_test.cc

Hi mjpc team,
While following the instructions in the readme file to install and test the project, I encountered a build issue.
Error log:
[build] /home/cxm/cpp_project/mujoco_mpc/mujoco_mpc/mjpc/test/estimator/batch_parameter_test.cc:109:21: error: no matching function for call to ‘mjpc::Batch::Optimize(mjpc::ThreadPool&)’
[build]   109 |   estimator.Optimize(pool);
[build]       |   ~~~~~~~~~~~~~~~~~~^~~~~~
[build] In file included from /home/cxm/cpp_project/mujoco_mpc/mujoco_mpc/mjpc/test/estimator/batch_parameter_test.cc:19:
[build] /home/cxm/cpp_project/mujoco_mpc/mujoco_mpc/mjpc/test/../../mjpc/estimators/batch.h:163:8: note: candidate: ‘void mjpc::Batch::Optimize()’
[build]   163 |   void Optimize();
[build]       |        ^~~~~~~~
[build] /home/cxm/cpp_project/mujoco_mpc/mujoco_mpc/mjpc/test/../../mjpc/estimators/batch.h:163:8: note:   candidate expects 0 arguments, 1 provided

As a temporary fix, I removed the 'pool' argument from the function call, and it resolved the issue. I'm curious to know if this is the right way to address the problem or if there are other implications I should be aware of.
Thank you for your guidance and assistance!

Thanks for pointing it out!  This was fixed on the deepmind branch, and will soon be merged to main: https://github.com/google-deepmind/mujoco_mpc/pull/199


error: You have not concluded your merge (MERGE_HEAD exists).

On a scoop update *, I stumble upon the following error:
$ scoop update *
Updating Scoop...
error: You have not concluded your merge (MERGE_HEAD exists).
hint: Please, commit your changes before merging.
fatal: Exiting because of unfinished merge.
Update failed.

I searched in my entire hard drive for a file named MERGE_HEAD, but no result.
$ fd MERGE_HEAD C:/

<empty return>

I've git merge --abort and git reset --hard origin/master every buckets in ~/scoop/buckets, but I've the same error.
I couldn't find any logs. Any thought?


Add GNewbury1/routeros

https://github.com/GNewbury1/terraform-provider-routeros
This redirects to:
https://github.com/terraform-routeros/terraform-provider-routeros
It's also published on the registry:
https://registry.terraform.io/providers/terraform-routeros/routeros/latest

Oups. Im closing this and open a new for the commit message.
Thx.


dark clear glass and dark fused quartz does not seem to be dark

What happens:
When i place it to optically isolate an area from light, the light levels beyond that wall of dark quartz/glass is not lowered
What you expected to happen:
It should block the light
Steps to reproduce:

build dark fused quartz
surround yourself with the fused quartz
light levels are uneffected

Affected Versions (Do not use "latest"):

EnderIO:  5.0.26
EnderCore: 0.5.30
Minecraft: 1.12-2
Forge: 2705

I've had something like this happen with EU2's Dark (Ineffable) Glass before, where light seemed to bleed in into a closed off room with dark glass on one side and solid walls (stone/cobble) on all others.
That might be the same issue, which only happened sporadically though (more frequently on servers).

Do you have a screenshot example of this problem?


Beyond todoist ?

Hi @hkgnp
First of all, thank you for your wonderful work on the plugins. I'm writing here because I know that there are some users (including me 😃) who are not developers but would be very interested in using the "sendToTodoist" feature for other software. I'm thinking for example of the very good GTD Everdo application which offers an API and a local storage of data.
https://everdo.net/
https://help.everdo.net/docs/advanced/api/
I have the impression that your agility to code would take you little time to offer a new plugin to the community 😃 of the same order for todoist 🤔. I'm unfortunately not able to do that myself 🥺. Besides I think it would be interesting to have a "SendToTerminal" type plugin to send blocks to other application via terminal.
Obviously I would totally understand that you wouldn't be interested at all if you don't use the tool... but your plugins create envy 😃.
That's it for my evening thought 😃

Hi!
Apologies, I don't use Everdo so I don't think I'll be making a plugin for it. Feel free to put up a request at https://discuss.logseq.com/c/feature-requests and maybe some other plugin developers who use it will be interested to make one for it!


Missing Port Requirement for LDAP over TLS

The ports requirements list here is missing an entry for LDAP over TLS (reference functionality here https://learn.microsoft.com/en-us/azure/azure-netapp-files/configure-ldap-over-tls).

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 219fc61e-07cc-2dd5-402c-2d8c5a9b7a3b
Version Independent ID: 220d7d4d-2900-2960-9c89-4714ea083fa4
Content: Understand guidelines for Active Directory Domain Services site design and planning
Content Source: articles/azure-netapp-files/understand-guidelines-active-directory-domain-service-site.md
Service: azure-netapp-files
GitHub Login: @b-ahibbard
Microsoft Alias: anfdocs

@matthansen0
Thanks for your feedback! We will investigate and update as appropriate.

@b-ahibbard
Can you please check and add your comments on this doc update request as applicable.

@matthansen0
I'm going to assign this to the document author so they can take a look at it accordingly

Thanks @matthansen0 . The table has been updated to include LDAP over TLS
#please-close

#please-close


feat: Fix readme grammar mistakes and typos

Small typo fixes

Wow very big contribute lol. Thanks.


Persistent error 13212

I'm on a N3DSXL and setup a connection in the DS wifi settings after deleting the existing ones and set only the primary DNS to the right one and still get the error

Where are you configuring DNS settings? Entralinked does not send this error code ever.

Where are you configuring DNS settings? Entralinked does not send this error code ever.

In the DS wifi settings

Which ones specifically? The 3DS has two menus where you can configure internet connections.
You have to configure one in the 3DS internet settings.

At first I only did the DS settings and when that didn't work I also did the same in the 3DS settings

Not sure if it's relevant info but I also have 2 other connections setup in the 3DS wifi settings

You have to delete those 2 other connections.

Okay

That fixed it 👍


Implement inline rename

This is early WIP of implementing Roslyn-powered Inline Rename.

Wow. Fantastic to see this in progress!

:kiss:

@forki :) But it's not a small task, as it's turned out...

@vasily-kirichenko I know. that's why you are doing it ;-)

I've stuck with this a bit. That's how it works at the moment:

What I think about it:

The (green) tags are calculated properly - they are shifted and increased in size as they should.
Tags and their content is not the same as real Document content because

Old code is not shifted (look at the + sign, for example)
Preview dialog says "No changes"

I think C#/VB editor modify AST directly, which then automatically reflected visually on documents.

If this is all true, I'm afraid we have to modify Documents manually, which raises other questions:

In what place we should do it? Candidates:

https://github.com/Microsoft/visualfsharp/pull/1932/files#diff-6ae7d2d36babc0c71812363409b9f718R142
https://github.com/Microsoft/visualfsharp/pull/1932/files#diff-6ae7d2d36babc0c71812363409b9f718R153
https://github.com/Microsoft/visualfsharp/pull/1932/files#diff-6ae7d2d36babc0c71812363409b9f718R196

Would it work for not opened Documents? If it does not, is it OK to open them, change, then close?

Tagging @jasonmalinowski and @dpoeschl for comments about how this works.

I'll try to take a look today

BIG thanks to @vladima for fixing this!
It works like this now:

What I have to finish:

add some validation, like we have in VFPT
add tests

@vladima I found an interesting bug:

@vasily-kirichenko Looks like there are quite a few unrelated changes in this branch?  Or do you need to merge with master? thx

@dsyme You are right. Done.
I will investigate the bug later.

~28 failures, e.g.
 1) Failed : Microsoft.VisualStudio.FSharp.Editor.Tests.Roslyn.ColorizationServiceTests.InactiveCode("Inactive Code1*)le","excluded code")

I've reverted the tryClassifyAtPosition fix, all should be green now.

The bug is a tricky one. If there are no text in the file after the renaming symbol, it pops up immediately, after deleting single character:

If there is text after it, then it appears later, depending on the length of trailing text:

I've compared our implementation to XAML one http://source.roslyn.io/#Microsoft.VisualStudio.LanguageServices.Xaml/Features/InlineRename/XamlEditorInlineRenameService.cs and found no differences. I've tested renaming in XAML editor, it works OK.
Any ideas?

@vasily-kirichenko This looks wrong (rename on a property e.Message replaces all text of the expression)

@dsyme yes, it's because FCS returns wrong ranges for qualified symbols. I had to work it around in Document Highlight service as well (it was copied from VFPT) and @vladima did something similar in this PR, too.
I think it would be really great to fix it at FSC level.

I think it would be really great to fix it at FSC level.

Agreed, since this is cropping up in more than one feature area

@dsyme could you comment on this? I mean is it by design:

System|.DateTime.Now => System
System.DateTime|.Now => DateTime
System.DateTime.Now| => System.DateTime.Now

?

@vasily-kirichenko Something to consider either for this PR or a future one is how to deal with conflicts and preventing bad renames.
For example:
let getFirstFromTwople (t: int*int) = t |> fst
let getFirstFromThreeple (t: int*int*int) = t |> fst
let getFirstFromFourple (t: int*int*int*int) = t |> fst

I can currently inline rename fst to FST, which would result in a compile error, assuming FST either doesn't exist or isn't an inlined function which works for tuples up to a certain size.
Another thing to consider is the Big Scary Warning Text for renaming something into something else in scope which has the same name, like in C#/VB:

As an aside, I've been playing with this today and showing it off to people in the office.  It's really, really great work.  Kudos 😄

@cartermp yes, I already thought about this. We have quite good non scope aware name validation in VFPT, but it requires porting significant amount of code, including our own higher level wrapper over the lever for nicer qualified names parsing (or maybe not and we can use classifyAtPosition). What about name conflicts in same scope, I don't know how to get all names for current one. Never done this.
However, I'd prefer to fix the bug (I have absolutely no idea how to do it) and make it solution wide (for all F# projects, making it work with other project types is a different unexplored area, however, the Xamarin guys have done it somehow, so we must be able to do it, too).

@vasily-kirichenko Makes sense.


feat(@vtmn/css): add rating component

Pull Request checklist
🚨 Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Don't request your main!
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] Check PR name, it must follow the following pattern : TYPE(@vtmn/PACKAGENAME): SHORT DESCRIPTION

Description

Add rating component

##Comment

To select a full star you need to clic a bit beyond the star icon, if someone has an idea to improve this behaviour it is welcomed :)

Does this introduce a breaking change?

No

❤ Thank you!

https://medium.com/free-code-camp/how-to-make-the-impossible-possible-in-css-with-a-little-creativity-bd96bb42b29d

We are waiting for refactoring it in design phase. We'll see this next week.


Description indicates ability to copy text "in edit-mode" however no documentation or shortcut explains entering edit-mode

Thx for pointing out this issue @imperialboy3. I've updated the section in the README that mentions this feature. Let me know if it is clearer now or how you would make it clearer!

Now I see it, thanks. The problem was that it didn't ever occur to me to scroll down. I ended up cloning it and adding a keypress to auto-copy lol.


Will this repository be closed in the near future?

Will the next commit allow me to set up an autoscaler in Spanner?
https://github.com/googleapis/googleapis/commit/d6467dbbb985d1777b6ab931ce09b8b3b1a7be08
In other words, will this project be closed?

There are no plans to deprecate the OSS Spanner Autoscaler (i.e. this repository) in the near term.
While there is a native Spanner autoscaling capability in development, we can’t comment on its progress, features, or the schedule for its potential release.


Rewrite CopiedContentIndicator using hooks

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against 79452ec18dde1739e3f92fbedcb0e8e60dfabf9a


Update dependencies

Pull Request Test Coverage Report for Build 62

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

Totals

Change from base Build 57:
0.0%

Covered Lines:
173

Relevant Lines:
176

💛  - Coveralls

closed in favour of https://github.com/escaped/django-exiffield/pull/13


Fix InvalidCastException on startup on non-English regional settings

Exception System.InvalidCastException with a message of
Conversion from string "8.2" to type 'Decimal' is not valid. was
raised when running the application on non-English regional settings
(when character different than . (dot) is a numerical separator).
Explicit usage of InvariantCulture when converting string to decimal
is not based on the regional separator, so it's always working.

Oh and I see this PR also fixes #1 :)


Multi cluster loadbalancing

Hi,
i'm looking for an on premise solution for managing services of type load balancer in multiple kubernetes cluster. This is currently not supported and is maybe out of scope of this project. I recognized that this is possible by configuring kube-vip to use wireguard.
Would it be an acceptable feature if the kube-vip-cloud-provider could create services in an upstream cluster and mapping node ports of downstream cluster nodes to the upstream cluster service ?
Are there big disadvantages compared to the wireguard solution with kube-vip?
Or should it be a separate provider for that because the logic would be completely decoupled from the current kube-vip-cloud-provider implementation?
I put already some time into an attempt to implement it in general. You can take a look on it at my fork. I've tested it successfully with an ingress-nginx-controller but not with tls enabled until now.
There are some limitations until now. Endpoints for Upstream Services are limited to one subset and sessionAffinity is hardcoded. I've added an example configuration to examples directory.

Yeah, I'd love to chat about what your plans are!

Thank you for your reply. In general i thought that this project could do what is actually done by cloud providers for load balancers (kubernetes -> kubernetes instead of kubernetes -> cloud provider). Credentials are a kubeconfig which only needs permissions for objects in one namespace which can be limited.
This would be helpful if you want to run a managed kubernetes platform in an on-permise environment and you don't have a simple builtin solution for external load balancing.
In the meantime the platform solution we prefer is using metallb for that. The advantage of doing it in the proposed way would be that there is no software needed for allocating ips in provisioned clusters.
The ip-range configuration per namespace by a configmap is also nice for restricting allocation per cluster or a group of clusters.
If you have any other questions please tell me your prefered way for communication. I was not sure whether you want to discuss it in this issue or outside of it.


Generating insane file sizes

I have a geojson that is 43MB, but I'm needing to get to at least zoom level 20, and so obviously the file it creates is just massive.
When running: tippecanoe -o test.mbtiles -z20 --coalesce-densest-as-needed test.geojson, I get a filesize of 108GB(!).
What would be the best option to get this smaller? I'm not super concerned about the sharpness of the polygon, I just need a reasonable small filesize.
Lowering the detail or changing the simplification (both?)?
Also is there a way to convert my geojson file so I can do parallel processing on it? It can take 2+ days to create the file on my server.

Zoom level 20 is extremely deep. Each of those tiles, even if it is entirely covered by a polygon, still has to spell out the gzip header, the square geometry, and whatever attributes your feature has. Nothing you do with detail or simplification is going to change this very much.
So my main suggestion is to use -zg instead of -z20 and only tile to whatever depth the inherent detail of your geometry actually requires. Vector tiles are meant to be overzoomable, so they can be viewed at deeper zoom levels than the tileset goes by magnifying the deepest available tile.
If you can't change that, the best you're going to be able to do is to lower the detail and the buffer size so there is as little geometry as possible in each tile. But it's only going to make a tiny difference.
If you want to be able to parse the GeoJSON file in parallel with -P, the requirement is just that no feature can be split across multiple lines. If you can't generate that directly, you can use tippecanoe-json-tool to preprocess your GeoJSON into newline-delimited json:
tippecanoe-json-tool source.json > out.nd.json

The tiling stage of Tippecanoe will always use as many CPUs as are available, no matter how the GeoJSON was formatted.

Vector tiles are meant to be overzoomable, so they can be viewed at deeper zoom levels than the tileset goes by magnifying the deepest available tile.

Maybe that's what I'm doing wrong. I'm using leaflet to display everything, but after a certain zoom level it just disappears. If that's the case I should be able to configure something on that then?

I didn't even know Leaflet supported vector tiles, but it yes, it probably has an overzooming setting somewhere.

Yeah, there is a plugin for it (https://github.com/Leaflet/Leaflet.VectorGrid)
And I was able to figure out overzooming on it pretty easily. Thanks for the help! I'm going to go back to all my layers and just do -zg to get the rest of them to normal sizes too.

@m0ngr31 how to you use the overzooming function in vectorgrid, I can't find the setting? Thanks


[feat] titlebar work's like browser tab.

Describe the problem
I want develop a editor, that has multiple window. Like visual studio.
For now I have a library That I can popout window by click a button.
But If Drag a tab out of window can be support, That would be perfect.
Describe the solution you'd like
While drag a html element. I can change the cursor.
For now, While dragging in browser, the cursor can't be change. I don't know If tauri can support it?
Alternatives considered
Support a special props like data-tauri-drag-region, So that drag cursor can be customize.
Additional context
IDE has this behavior：visual studio，unreal engine4.
My library online example : https://codesandbox.io/s/grape-layout-example-irh1p5

I don't quite understand what you want actually. Do you want to drag a tab from one window to another window? Do you want to change the cursor icon while dragging?

I think he wants to create a new window while dragging a tab (like the browser does) and move that tab contents to that window.

I don't quite understand what you want actually. Do you want to drag a tab from one window to another window? Do you want to change the cursor icon while dragging?

I have use browser drag drop API implementation drag window from one to another.
But while Drag a tab out of a window, the cursor will be disable drop icon.

I think he wants to create a new window while dragging a tab (like the browser does) and move that tab contents to that window.

I wang drag a tab like a browser.

I haven't adapted tauri window yet.

Then please open issues only if you're using tauri.

I want find a solution of change drag cursor first.

We can provide a window.set_cursor_icon method but you should probably look into css cursors instead.

I haven't adapted tauri window yet.

Then please open issues only if you're using tauri.

I want find a solution of change drag cursor first.

We can provide a window.set_cursor_icon method but you should probably look into css cursors instead.

sadlly, the cursor can't be change while dragging in browser.
I am find a platform that can change the cursor while dragging.
And I found tauri can add a flag data-tauri-drag-region to titlebar.
Can I change cursor with add a flag data-tauri-drag-region while dragging some thing.

I haven't adapted tauri window yet.

Then please open issues only if you're using tauri.

I am using tauri. Just not adapted tauri's webWindow

Hello, how to solve this problem finally?

Hello, how to solve this problem finally?

You can try this solution

We can provide a window.set_cursor_icon method but you should probably look into css cursors instead.

Hello, how to solve this problem finally?

You can try this solution

We can provide a window.set_cursor_icon method but you should probably look into css cursors instead.

Thank you. I also develop multiple windows, but I don't know how to manage each window through tabs. It looks like Google Chrome


Could you make a release?

I have a merged patch since Aug 16, 2016 and still no release. Could you do it?

Releases are specially helpful for distribution packagers, as they usually don't take code from master branches, but from actual releases.


[Bug] Named color in variable declaration

In my project, I have been using Color Name to name the colors.
Basically, if the variable name is a named color of the extension, it shows the color preview. In case of variable declaration, it should not have the preview. Since the color from the actual data is slightly different from the name. (See dodgerblue as example from this screenshot)

My proposed solution would be to add a setting enabled by default to ignore DocumentSymbols, which would not provide a color picker on any variable/function/class (etc) declaration name, does it sound good to solve your case?

There I fixed it in #80. Now there is this new setting "ignoreVariableName" (enabled by default) which tells the extension to ignore variable names.
Please test it and let me know of any bugs.
Thanks!


Where to find settings.json

The settings.json file is no longer stored in the Skill root directory

Excellent update, thanks heaps David!


Way to use CollapsingToolbar/CollapsingToolbarScaffold with Compose Scaffold?

Hi there!
Great library, thanks so much for building this out. I'm currently trying to implement this in my application, and can't seem to figure out how to get the CollapsingToolbar to work with a regular Compose Scaffold.
I have a TopAppBar, BottomAppBar, and FloatingActionButton, so using the Scaffold included by default in Compose is somewhat necessary. Is there a way to wrap the TopAppBar in a CollapsingToolbar and get it to scroll correctly? I can't seem to get it to work.
Here's what I have so far:
val scaffoldState = rememberScaffoldState(
    rememberDrawerState(initialValue = DrawerValue.Closed)
)

val collapsingToolbarState = rememberCollapsingToolbarState()

Scaffold(
    scaffoldState = scaffoldState,
    topBar = {

        CollapsingToolbar(
            collapsingToolbarState = collapsingToolbarState
        ) {
            TopAppBar(
                backgroundColor = MaterialTheme.colors.background,
                title = {
                    Text(text = "TopAppBar")
                }
            )

            Box(
                modifier = Modifier
                    .fillMaxWidth()
                    .height(150.dp)
                    .parallax(0.5f)
                    .graphicsLayer {
                        alpha = collapsingToolbarState.progress
                    }
            ) {
                Image(
                    painter = painterResource(id = R.drawable.ic_launcher_foreground),
                    contentDescription = null
                )
            }
        }
    },
    floatingActionButtonPosition = FabPosition.End,
    floatingActionButton = {
        FloatingActionButton(onClick = { /*TODO*/ }) {
            Text("X")
        }
    },
    drawerContent = {
        Text(text = "drawerContent")
    },
    bottomBar = {
        BottomNavigation(
            backgroundColor = MaterialTheme.colors.background
        ) {
             .....
        }
    },
) {
    LazyColumn(
        modifier = Modifier.fillMaxWidth()
    ) {
        items(100) {
            Text(
                text = "Item $it",
                modifier = Modifier.padding(8.dp)
            )
        }
    }
}

However, the expanding/collapsing doesn't seem to work. Here's a video:
https://user-images.githubusercontent.com/1452739/155588999-d923f535-e7a9-44b4-8d48-774df0b5e352.mov
Thanks in advance for the help!

Is there a problem if you leave topBar empty and wrap Scaffold with CollapsingToolbarScaffold?

Hey @onebone , sorry for the delay in response. This kind of fixes the issue, but now the bottom nav bar and fab don't stay anchored to the bottom, they scroll with the collapsing toolbar. I'll attach another video (sorry, from another example so things look a bit different).
Here's the code:
val state = rememberCollapsingToolbarScaffoldState()

var enabled by remember { mutableStateOf(true) }

CollapsingToolbarScaffold(
    modifier = Modifier.fillMaxSize(),
    state = state,
    scrollStrategy = ScrollStrategy.ExitUntilCollapsed,
    enabled = enabled,
    toolbar = {
        // In between 30 and 18 (30 expanded, 18 collapsed)
        val textSize = (18 + (30 - 18) * state.toolbarState.progress).sp

        Box(
            modifier = Modifier
                .background(MaterialTheme.colors.primary)
                .fillMaxWidth()
                .height(150.dp)
                .parallax(0.5f)
                .graphicsLayer {
                    alpha = state.toolbarState.progress
                }
        ) {
            Image(
                modifier = Modifier.padding(16.dp),
                painter = painterResource(id = R.drawable.ic_launcher_foreground),
                contentDescription = null
            )
        }

        Text(
            text = "Title",
            modifier = Modifier
                .padding(60.dp, 16.dp, 16.dp, 16.dp)
                .graphicsLayer {
                    alpha = 1 - state.toolbarState.progress
                },
            color = Color.White,
            fontSize = 18.sp
        )
    }
) {
    Scaffold(
        scaffoldState = rememberScaffoldState(),
        topBar = { },
        floatingActionButtonPosition = FabPosition.End,
        floatingActionButton = {
            FloatingActionButton(onClick = { /*TODO*/ }) {
                Text("X")
            }
        },
        drawerContent = {
            Text(text = "drawerContent")
        },
        bottomBar = {
            BottomNavigation(
                backgroundColor = MaterialTheme.colors.background
            ) {
                // FIXME: Make sure selected/unselected colors are the same
                BottomNavigationItem(
                    icon = {
                        Icon(
                            painter = painterResource(id = R.drawable.ic_launcher_foreground),
                            contentDescription = null
                        )
                    },
                    label = {
                        Text(text = "One")
                    },
                    selected = true,
                    onClick = { /*TODO*/ }
                )
                BottomNavigationItem(
                    icon = {
                        Icon(
                            painter = painterResource(id = R.drawable.ic_launcher_foreground),
                            contentDescription = null
                        )
                    },
                    label = {
                        Text(text = "Two")
                    },
                    selected = false,
                    onClick = { /*TODO*/ }
                )
            }
        },
    ) {
        Log.e("TAG", "Material background color: ${MaterialTheme.colors.background}")

        LazyColumn(
            modifier = Modifier
                .fillMaxWidth()
        ) {
            items(100) {
                Text(
                    text = "Item $it",
                    modifier = Modifier.padding(8.dp)
                )
            }
        }
    }
}

https://user-images.githubusercontent.com/1452739/157969383-086e5df7-1ecd-472c-985b-3122733a9354.mov

I'm having the same issue. So far I've been unable to workaround this, so I'm removing the library from the project for the time being.


added folders to packages

I forgot to add the folders to the packages list.

Codecov Report

Merging #191 into master will decrease coverage by 0.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
- Coverage   74.69%   73.83%   -0.87%
==========================================
  Files         126      126
  Lines       13199    13199
==========================================
- Hits         9859     9745     -114
- Misses       3340     3454     +114

@giacomov I can merge this... small bug fix in setup.

Go ahead!
Sent from my Windows 10 phone
From: J. Michael Burgess
Sent: Tuesday, May 16, 2017 9:04
To: giacomov/3ML
Cc: Giacomo Vianello; Mention
Subject: Re: [giacomov/3ML] added folders to packages (#191)
@giacomov I can merge this... small bug fix in setup.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Added support for resetting properties of lpar, partition, nic

See commit message.
TODO: Test

Coverage increased (+0.04%) to 36.716% when pulling bdaf02661a0655fb0332184603aeedc080467b63 on andy/set-null into af9c1285bda861a18d52e5108f45077c4fd8b8fb on master.


Add the support for optional dependencies

Would it be possible to have a feature as described in the title?
To be more clear, let's take as an example the model of a robot when working in simulation and in the real robot:

on simulation (maybe) I want to have all the dependencies, including the gazebo plugin for the sensors,
on the real robot I just want to have the minimal model of the robot description

What do you think?

You could either have one set of recipes for simulation and one set for the real robot. But I don't know if this approach would scale very well, so I really like your suggestion. We already support args for compilation flags we could add support also for conditional dependencies.

Actually, @alaurenzi already implemented this: you need to use the depends_if field
You can see a recipe example here (xbot2).
Xenomai is a dependency only if the option -m xeno is passed to forest grow
forest grow xbot2 -m xeno

FYI @EdoardoRomiti


Translate components

Translates components files. i refactored take-photo translations to match new style
convention discussed here
https://github.com/HospitalRun/hospitalrun-frontend/issues/557

@jkleinsc updated


Update hot spares RFE

At the last minute we decided to change where ClaimedTimestamp lived.
Update the RFE doc for posterity.
HIVE-1576

/assign @joelddiaz


SP5: need line styles

need line styles and also a way for them to make their own paints

9430a8fd6aacab151cdfcb5be67e23bd42037a46


Using parameters in ADD COLUMN statement doesn't work

Describe the bug
this code doesn't work:
query = "ALTER TABLE {table_name:Identifier} ADD COLUMN {field_name:Identifier} Int64" self._get_client().query(query=query, parameters={"table_name": table_name, "field_name": field_name})
This results in an error:
"Code: 62. DB::Exception: Syntax error: failed at position 48 ('{'): {field_name:Identifier}. Expected one of: IF NOT EXISTS, column declaration, compound identifier, list of elements, identifier. (SYNTAX_ERROR) (version <IP_ADDRESS>00 (official build))\n"
It's important for me to use parameters, since the inputs can come from the user.
Is this supposed to work?
Configuration
Environment

clickhouse-connect version: 0.7.0
Python version: 3.9
Operating system: mac/linux

ClickHouse server

ClickHouse Server version: clickhouse cloud
ClickHouse Server non-default settings, if any:
CREATE TABLE statements for tables involved:
Sample data for these tables, use clickhouse-obfuscator if necessary

This is a known issue in ClickHouse.  Currently server side parameters are only guaranteed to work for SELECT statements.  I'll update the clickhouse-connect documents to mention that limitation.

You can also try client side binding.  Note that even though the input is supplied by the user, this doesn't really provide much of a SQL Injection attack vector.  Multi statements are not allowed, so all the user can really do is execute an `ALTER TABLE'
statement on a table where they have permissions, and even in that case as a best practice you should sanitize your inputs to allow only valid table and column names.

Docs added.  There's nothing more than can be done on the client side.


Change 'Handler' to 'HandlerClient' on Docs

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


Let newer Jenkins versions override PNG icons

This PR adds the ability to allow modern Jenkins versions to override PNGs with SVGs where applicable, in favor of core getting a rid of PNG icons in a foreseeable future: jenkinsci/jenkins#5778
Before:

After:

[X] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[X] Ensure that the pull request title represents the desired changelog entry
[X] Please describe what you did
[X] Ensure you have provided tests - that demonstrates feature works or fixes the issue

@olamy Mentioning you, accordingly you are the right one to pull PRs 👀

@olamy can you release this please?

@olamy could you release please?