Offline Document Store creation not working by running distributable Jar.

I am using this library for JavaFX desktop application.

By running application From IDE(NetBeans, Eclipse) everything working perfectly.
But after creating a jar file of the application.
-No offline document is been created for offline synchronization.

Below is the exact line where the program stuck
DocumentStore ds = DocumentStore.getInstance(new File(path, "my_document_store"));
In my case:
path=new File(System.getProperty("user.dir"));

The Same issue exists after creating ant built the distributable .exe package for deploying on different window systems.

I suggest you add logging (see https://github.com/cloudant/sync-android/blob/master/doc/logging.md) to your deployed application and examine the logs to see what is going wrong.
Or you could modify your deployed program so that it generates extra output (to the console or a file) which will help us pinpoint exactly what is going wrong.
Your issue may be a problem with our library, or it may be a bug in your code, or a problem with the environment or configuration of your application. At the moment, and without extra information, we can't tell which it is.

HI,  I have got some findings. Hope these will help.

Below is the log Console output when i run my application from Netbeans and every working perfectly.

Now This is the output when I run the application by executing Application.jar file through windows command prompt.

Now the underlined bluelines of both output are same but the deference comes after the red line.

We don't have any experience of running on Windows but apart from the [SEVERE] null and instantiated [null] log messages, everything else looks fine.
In your first example it looks like 6 documents were pushed to your remote server, did you check on the remote side to see that those documents had been pushed?

Yes in the first case(when I run the application through IDE i.e Netbeans or eclipse) everything working fine.
but when i create .jar file and deployable package it creates problem.

I suspect your deployed jar is having trouble locating the native DLL for sqlite4java

Let me tell the whole story.
I am using java-cloudant to create the database only for the first time  when a user creates an account.
Then i am using sync-android to use this database with offline first approach.
Now when I run my application form IDE everything working perfectly both java-cloudant and sync-android.
after creating .jar file the database is created perfectly which is done through java-cloudant, but applicaiton is not creating offline store which is done by sync-android library.
This  is the exact line which is not working.
DocumentStore ds = DocumentStore.getInstance(new File(path, "my_document_store"));
Did this library is checked after creating and ruining jar files?

yes, I suspect that too, Let me do homework to find some solution.

Exactly that was the issue. when I manually copy native DLL for sqlite4java in library folder where sqlite4java.jar exists everything works fine. thanks.


fix: added bug report template

reference #8

Good job @kaustav202
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


Update pghero

Can you update pghero to the latest release, 1.6.2? I would do it myself but I've never used ruby or edited a gem file so I'm sure I'd do something wrong.

Done, thanks for the reminder!


Specify number of cores to be used with tensorflow

I am using keras on grid. There are many cores available, however I can't use all of them at same time. Therefore I would like to specify number of cores keras (tensorflow) can use. I have been looking for solution for quite some time. Is it possible?

Python's GIL locks it to a single core per process. I believe TF supports asynchronous computation. Try putting train ops on separate threads

Of course I found the solution after posting the question...
config = tf.ConfigProto(intra_op_parallelism_threads=1, inter_op_parallelism_threads=1, \
                        allow_soft_placement=True, device_count = {'CPU': 1})
session = tf.Session(config=config)
K.set_session(session)

Of course I found the solution after posting the question...
config = tf.ConfigProto(intra_op_parallelism_threads=1, inter_op_parallelism_threads=1, \
                        allow_soft_placement=True, device_count = {'CPU': 1})
session = tf.Session(config=config)
K.set_session(session)

So this tells keras to use only 1 core, right? I would like to know if you set device_count = {'GPU':0} what would happen; will it use all detected CPU cores?

Why allow_soft_placement=True is needed?
https://stackoverflow.com/questions/44873273/what-do-the-options-in-configproto-like-allow-soft-placement-and-log-device-plac


Sound Emitter v1.2

Added the ability to set the position option of the Audio Injector inside the Sound Emitter's userData via the positionOverride key/value pair.
Documented why one would want to attach soundEmitter.js as either an entity client script or an entity server script.

https://content.highfidelity.com/Experiences/Releases/usefulUtilities/soundEmitter/v1.2/soundEmitter.js

As a client script, when the audio is playing and you switch the looping from true to false and save the user data you have to refresh the script itself to reflect that change. You only need to do this for the true/false of the looping. If you change the volume or position mid way and save the userdata it automatically updates.


Add Redis Desktop

http://redisdesktop.com/download
Free, open source redis desktop client.

https://github.com/caskroom/homebrew-cask/blob/master/Casks/rdm.rb

Ah, sorry, I was too lazy to search for more than "redis".


Fix large file download

decodeHttpResponse() store the body response in a variable so if you try to download a large file from you Drive, it will crash with out of memory error.
Since the data is not needed for raw response, do it only when we try to json_decode().
I'm not sure of the modification since $body is used in Google_Service_Exception() in case of retry.
But with this change and #873 , i'm able to backup my Drive successfully.

Thank you for finding this issue!

closing in favor of #876


Using pyodata with old MS CRM 2015

Hi,
Thank you for this wonderfull piece of code.
pyodata can also be used successfully with MS CRM 2015 with 2 minors changes :

the first one is to change PATCH method by MERGE Method in service.py  (MS CRM 365 is now using PATCH)
the second one is a small try/catch in EdmStructTypeSerializer.from_json to avoid some None result
that raise an exception (At the end of this report)

Is there a way to integrate these changes in a vendor specific python file as it exists for SAP ?
Change made in def from_json(edm_type, value):
            if type_prop.name in value:
>               try:
                   result[type_prop.name] = type_prop.typ.traits.from_json(value[type_prop.name])
>               except (TypeError, AttributeError):
>                  modlog().warning(value)
        return result

This catchs the following errors in MS CRM 2015 to trap None 'Value' :
WARNING:pyodata.model:{'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.EntityReference'}, 'Id': 'f3d1cca6-0313-e711-80be-005056011e04', 'LogicalName': 'systemuser', 'Name': None} WARNING:pyodata.model:{'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.EntityReference'}, 'Id': None, 'LogicalName': None, 'Name': None} WARNING:pyodata.model:{'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.OptionSetValue'}, 'Value': None}

Hi @bip91, thank you very much for taking the time to report these issues. I think we will very soon come up with an infrastructure for adding handlers for vendor bugs in OData services  since We have already have several discrepancies in ByD services.
Ad 1. This definitely requires some extension point. Will think about the
Ad 2. Are you sure it is a bug in MS CRM 2015 and not in pyodata?

I definitly agree for the first point, this cannot be solve without an extension as it is a very specific implementation of ODATA. In the meantime, I have set a condition to handle this implementation and will see later.
For second point, I am not enough skilled in Odata protocol to say what should be the correct behaviour. Perhaps examples below could help.
In all cases, pyodata raise a exception on trying to convert to a int  a None Value

In the 2 first cases, because Value does not exists
In the last one, because Value is explicity set to Null

I do not see any side effect on trapping these 2 exceptions and doing nothing, perhaps it is the right thing to do.
**Case 1 and 2 : with 3 samples that raise this exception **
WARNING:pyodata.model:{'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.EntityReference'}, 'Id': 'f3d1cca6-0313-e711-80be-005056011e04', 'LogicalName': 'systemuser', 'Name': None}
WARNING:pyodata.model:{'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.EntityReference'}, 'Id': None, 'LogicalName': None, 'Name': None}
comes from these samples which there is no "Value" tag :
        <d:CreatedBy m:type="Microsoft.Crm.Sdk.Data.Services.EntityReference">
          <d:Id m:type="Edm.Guid">2f804332-197c-e511-80ba-005056011e04</d:Id>
          <d:LogicalName>systemuser</d:LogicalName>
          <d:Name>Service Dynamics CRM</d:Name>
        </d:CreatedBy>

        <d:OwningBusinessUnit m:type="Microsoft.Crm.Sdk.Data.Services.EntityReference">
          <d:Id m:type="Edm.Guid">7dbe6139-8dd1-e711-9105-06b4dc0315b1</d:Id>
          <d:LogicalName>businessunit</d:LogicalName>
          <d:Name m:null="true" />
        </d:OwningBusinessUnit>

        <d:PrimaryContactId m:type="Microsoft.Crm.Sdk.Data.Services.EntityReference">
          <d:Id m:type="Edm.Guid" m:null="true" />
          <d:LogicalName m:null="true" />
          <d:Name m:null="true" />
        </d:PrimaryContactId>

** case 3:  is rather the same, but Value is explicitly set to Null **
        <d:OwnershipCode m:type="Microsoft.Crm.Sdk.Data.Services.OptionSetValue">
          <d:Value m:type="Edm.Int32" m:null="true" />
        </d:OwnershipCode>

I am afraid that the None/null issue is a bug on pyodata.

In addition to the rules stated in the table, if the value of a primitive type is null, then it is represented as an empty XML element with an m:null="true" attribute ("m" identifies the OData metadata namespace).

https://www.odata.org/documentation/odata-version-2-0/atom-format/
I have to find such a value in Microsoft's Northwind -> https://services.odata.org/V2/Northwind/Northwind.svc/

@bip91 Can you re-post the XML responses in JSON here? Just add ?$format=json to the URL.
Example:

XML = https://services.odata.org/V2/Northwind/Northwind.svc/Customers
JSON = https://services.odata.org/V2/Northwind/Northwind.svc/Customers?$format=json

The example URLs point the entityset which contains entities having values with null and pyodata can handle them without any problem:
PYTHONPATH=./ bin/pyodata https://services.odata.org/V2/Northwind/Northwind.svc/ ENTITY_SET Customers

the output is the following:
[Initializing HTTP session ...]
[Fetching $metadata ...]
K CustomerID = ALFKI
+ ContactName = Maria Anders
+ PostalCode = 12209
+ City = Berlin
+ Address = Obere Str. 57
+ Country = Germany
+ CompanyName = Alfreds Futterkiste
+ ContactTitle = Sales Representative
+ Region =
+ Fax =<PHONE_NUMBER>
+ Phone =<PHONE_NUMBER>
--------------------------------------------------------------------------------
...

where the field Region holds null.

Can you change the lines:
>               except (TypeError, AttributeError):
>                  modlog().warning(value)

to
>               except (TypeError, AttributeError) as ex:
>                  modlog().warning(f'Exception: {ex}')
>                  modlog().warning(f'Property: {type_prop.name}')
>                  modlog().warning(f'Value: {value}')

a repost your results?

WARNING:pyodata.model:Exception: int() argument must be a string, a bytes-like object or a number, not 'NoneType'
WARNING:pyodata.model:Property: Value
WARNING:pyodata.model:Value: {'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.OptionSetValue'}, 'Value': None}

WARNING:pyodata.model:Exception: 'NoneType' object has no attribute 'strip'
WARNING:pyodata.model:Property: Name
WARNING:pyodata.model:Value: {'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.EntityReference'}, 'Id': None, 'LogicalName': None, 'Name': None}

WARNING:pyodata.model:Exception: int() argument must be a string, a bytes-like object or a number, not 'NoneType'
WARNING:pyodata.model:Property: Value
WARNING:pyodata.model:Value: {'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.OptionSetValue'}, 'Value': None}

WARNING:pyodata.model:Exception: 'NoneType' object has no attribute 'strip'
WARNING:pyodata.model:Property: Name
WARNING:pyodata.model:Value: {'__metadata': {'type': 'Microsoft.Crm.Sdk.Data.Services.EntityReference'}, 'Id': '7dbe6139-8dd1-e711-9105-06b4dc0315b1', 'LogicalName': 'businessunit', 'Name': None}

@bip91 Do you think you can test my changes in PR: https://github.com/SAP/python-pyodata/pull/100 It only fixes the problem with None.

Now, I get this error on initializing schema  :
crm = pyodata.Client(api_url_base,session)
...
INFO:pyodata.client:Creating OData Schema (version: 2)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/zato/tests/python-pyodata-1/pyodata/client.py", line 59, in __new__
    schema = pyodata.v2.model.MetadataBuilder(resp.content, config=config).build()
  File "/opt/zato/tests/python-pyodata-1/pyodata/v2/model.py", line 2451, in build
    raise PyODataParserError(f'Unsupported Schema namespace - {namespace}')
pyodata.exceptions.PyODataParserError: Unsupported Schema namespace - http://schemas.microsoft.com/ado/2007/05/edm

So, I have no initialized context to execute the request.
Please find joined, metadata url provided by server (with some anonymization, I hope that I have not introduce some error with step) :
https://server.com/CRM/XRMServices/2011/OrganizationData.svc/$metadata
content :
$metadata.txt
I hope this could help.

I cannot reproduce the problem. Didn't you touch this list:
https://github.com/SAP/python-pyodata/blob/master/pyodata/v2/model.py#L2441

In my version of pyodata, the exception you got is raised at the line 2535 but in your case it's line 2451.

Sorry for this erroneous first feedback, with a fully new fresh installation it works like a charm.
Patch is working for me.

After merging of #102 this issue seems that can be closed.
@bip91  could you confirm that latest head 7972f31 works for you?
I will create release 1.5.0 soon.

@bip91  It would help us if you create a new issue for the PATCH vs. MERGE request.

Latest head works perfectly, I have deployed it instead of my first workaround.
I have also isolated the part "MERGE in a vendor request" in #103

@bip91  I would not mind if you create a new factory method MS_CRM_client in the vendor module which would return pyodata client correctly configured.

Yes, of course.
I have built MSCRM vendor extension :
MSCRM.py.txt
This works for me with the following code :
import vendor.MSCRM
crm = vendor.MSCRM.Client(api_url_base,session)

# example to set a on guid id
crm.entity_sets.ContactSet.update_entity(id).set(**a).execute()

The endpoint has changed on more recent Dynamics CRM  so the test on URL should be sufficient to avoid mistakes, but I cannot validate this (I have only Dynamics CRM 2015)
Feel free to adjust my extension to be coherent with the others.


fetch_all breaks on list responses with a non repeating resource field

If you try to use fetch_all to get sequence resources from genomicsv1.list_reference_bases, it will fail because the ListBasesResponses's resource is a string, not an array.

Thanks! 😄


validation: make scale-test Python 3 friendly

zip will return a generator in Python 3 rather than the zipped list.
This results in the Nelder-Mead Simplex to fail as it does not actually
perform the optimization of the data.  Explicitly convert the data to a
form which can be consumed.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @drexin @tachoknight @tbkka

@swift-ci please test


Rate limiting on exposed services

Some calls against exposed services are resource intensive, e.g. API calls that run database queries.

Implement ipaddr-based rate limiting with different thresholds based  service and call
document it for end users

Required to improve https://github.com/ooni/backend/issues/371

This is superseeded by: https://github.com/ooni/backend/issues/394


added domains for Istituto Tecnico Tecnologico Alessandro Volta Perugia

Hi, please add my Institute, this is the main domain:
www.avoltapg.edu.it
and this is the domain we use for email addresses:
www.risikamente.it
Here you can see the 5 years course in IT and Telecommunications:
https://www.avoltapg.edu.it/pagine/informatica-e-telecomunicazioni
Thank you for your time!

@amendola-scuola Thank you for the request. Could you please clarify the following:

Do you need to add the domain avoltapg.edu.it to the repository? If you don't use this domain for email addresses, you don't actually need to add it.
Please provide us with a proof, which shows that the school recognizes the domain risikamente.it that you are submitting as an official email domain for the students.

You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@amendola-scuola You might miss my previous comment. Please react.

@amendola-scuola Thank you for the request. Could you please clarify the following:

Do you need to add the domain avoltapg.edu.it to the repository? If you don't use this domain for email addresses, you don't actually need to add it.
Please provide us with a proof, which shows that the school recognizes the domain risikamente.it that you are submitting as an official email domain for the students.

You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

Hi, sorry for the delay,
for n.1 it is possible that in the future we will switch email addresses on the main domain, if it's a problem to add it right away I'll remove it from the pull request,
for n.2 unfortunately all I can provide it's a whois record, from the screenshot you can see that the registrant organization and addresses are the same, I hope it's enough:

Thank you for your time, regards

@amendola-scuola  Thank you for the clarifications. Pull request merged.


Azure Python SDK documentation page for "azure-mgmt-resource Package" looks broken

Type of issue
Other (describe below)
Description
Azure Python SDK documentation page for "azure-mgmt-resource Package" looks broken.
Page address - https://learn.microsoft.com/en-us/python/api/azure-mgmt-resource/?view=azure-python
This is what it looks like when I open it in Edge and Chrome:

[Enter feedback here]
Page URL
https://learn.microsoft.com/en-us/python/api/azure-mgmt-resource/?view=azure-python
Content source URL
https://github.com/MicrosoftDocs/azure-docs-sdk-python/blob/main/docs-ref-autogen/azure-mgmt-resource/index.yml
Document Version Independent Id
55877fa1-3353-edb8-4352-5f82b7fe5777
Article author
@lmazuel
Metadata

ID: a6c27c74-bbdb-534a-78d3-51b01543a76f

Also these pages look broken too.

Hi Sergiy, thanks for opening an issue! It appears there's an error in our page generation process and we'll investigate ASAP.
If you need to access the reference documentation, for now, you can use the azure.mgmt.resource item in the table of contents:


Error Boundaries

https://reactjs.org/docs/error-boundaries.html
The app should handle gracefully:

server errors
network errors
client errors

@vadimbasko please add info for QA on how to test this ticket


Alternative crosslink data construction, take 1

Properties:

Optimal data packing
Fixed depth
All data is encoded; makes fraud proofs not-too-difficult

Weaknesses:

Uses SSZ serialization.
Positions of blocks dynamic, so accesses require an extra Merkle branch

Uses SSZ serialization.

I wouldn't see that as a weakness, if it is the right tool for the job, I think we should be able to bring it into consensus.

I wouldn't see that as a weakness, if it is the right tool for the job, I think we should be able to bring it into consensus.

agreed.

@vbuterin This can be closed as it was superseded by #1294, right?

Closing -- superseded by #1294


Next steps?

Hi @Savjee, I've been following your series of posts and I really enjoyed that.
I do not like the idea of "reinventing the wheel", but if you intend to create some decentralized solution that uses a blockchain template, you can do thousands of searches (from various sources) (Googling) that only topics related to "cryptocurrency" flood the content.
Virtually most solutions, services and tools have open sources that can be studied and documentation plentiful but, you end up stuck or one or the other ... it is extremely scarce to find blockchain materials that are not solely and exclusively related to "cryptocurrency ". I really do not believe that I should "mold" my problems in order to use the available libraries simply because of the lack of a minimally qualified material.
I think at this point that your series of posts has been very didactic and helps better understand the operation of a blockchain than many low-level matters that it has there.
I'm studying the use of a front-end blockchain using WebRTC (DataChannel), and I'm leveraging their stuff to better understand how it works. In the last post it seems to me that you have gone in the direction of an active chain (cryptocurrency).
I wonder if you have thoughts about continuing the series of posts and plans to talk about wallets or contracts?

I'm closing this issue because it's not really a problem with the code. But feel free to keep commenting afterwards ;)
I do plan to continue the series but I don't know yet what the next video will be and when I'll get around to making it. I think almost everything is in place to support wallets. All it needs is a way for people to sign transactions (HMAC) and a way for miners to validate them (also HMAC). After that, the wallet can be implemented as a front-end for this blockchain.
Contracts is a bit more difficult to implement in node.js (in a safe way) but not impossible. I've been thinking about wether or not the safety measurements would be too complicated for this series.


Tree.data typing - extend type with Token class

Because it tends to be Token also.
See example below:
from lark import Lark, Tree

grammar = """
    start: entity+
    entity: "entity"
"""

parser = Lark(grammar)
tree = parser.parse("entityentityentity")

for subtree in tree.iter_subtrees_topdown():
    # start RULE <class 'lark.lexer.Token'>
    # entity RULE <class 'lark.lexer.Token'>
    # entity RULE <class 'lark.lexer.Token'>
    # entity RULE <class 'lark.lexer.Token'>
    print(subtree.data, subtree.data.type, type(subtree.data))

@erezsh

The choice to have data a Token is a choice that I'm now unsure about, and I'm thinking of changing it to only be str in future versions.
So, I'm leaning towards not accepting this PR.

But still, doesn't it makes sense for current typing to just reflect the way it is now?

Token is a subclass of str, so the typing is correct.
Saying it's a Token would mean people could rely on it having a type, line number, etc., which I think isn't a good idea.

If it's not reliable, what would be a safe to get all tokens for the tree?
Currently I'd use:
from lark import Lark, Tree, Token

grammar = """
    start: entity+
    entity: "entity"
"""

parser = Lark(grammar)
tree = parser.parse("entityentityentity")

# Get all tokens.
for subtree in tree.iter_subtrees_topdown():
    child = subtree.data
    if not isinstance(child, Token):
        continue
    # start RULE <class 'lark.lexer.Token'>
    # entity RULE <class 'lark.lexer.Token'>
    # entity RULE <class 'lark.lexer.Token'>
    # entity RULE <class 'lark.lexer.Token'>
    print(subtree.data, subtree.data.type, type(subtree.data))

_toi can-biet vab hie ra van de = mã nguồn + cấu trúc -why = :toi khong
biet-
Vào Th 4, 25 thg 12, 2024 lúc 17:40 Andrej @.***> đã
viết:

If it's not reliable, what would be a safe to get all tokens for the tree?
Currently I'd use:
from lark import Lark, Tree, Token
grammar = """    start: entity+    entity: "entity""""
parser = Lark(grammar)tree = parser.parse("entityentityentity")
Get all tokens.for subtree in tree.iter_subtrees_topdown():
child = subtree.data
if not isinstance(child, Token):
    continue
# start RULE <class 'lark.lexer.Token'>
# entity RULE <class 'lark.lexer.Token'>
# entity RULE <class 'lark.lexer.Token'>
# entity RULE <class 'lark.lexer.Token'>
print(subtree.data, subtree.data.type, type(subtree.data))

—
Reply to this email directly, view it on GitHub
https://github.com/lark-parser/lark/pull/1501#issuecomment-2561829323,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUN6KGONF2INJDWGFKZK632HKDTFAVCNFSM6AAAAABUF2NEPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRRHAZDSMZSGM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

When getting all tokens of a parse tree, you don't actually want the tokens that are inside of the data attributes since they didn't actually belong to that tree but to the grammar (which is itself parsed with lark)
IMO the only result of data attributes being Tokens is that the repr is less readable - I haven't found any actual benefit.
Either way, using Uniom[str, Token] instead of str is pointless from the static type checker perspective since it's equivalent to just str.

When getting all tokens of a parse tree, you don't actually want the tokens that are inside of the data attributes since they didn't actually belong to that tree but to the grammar (which is itself parsed with lark)

I mean in my case I need to get what grammar tokens were parsed from the query and it's possible using Tree.data.type.

= hien tai can toi da hoa het data?
Vào Th 4, 25 thg 12, 2024 lúc 18:31 Andrej @.***> đã
viết:

When getting all tokens of a parse tree, you don't actually want the
tokens that are inside of the data attributes since they didn't actually
belong to that tree but to the grammar (which is itself parsed with lark)
I mean in my case I need to get what grammar tokens were parsed from the
query and it's possible using Tree.data.type.
—
Reply to this email directly, view it on GitHub
https://github.com/lark-parser/lark/pull/1501#issuecomment-2561849637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUN6KE5GFMDOISYLDTJ2LT2HKJSHAVCNFSM6AAAAABUF2NEPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRRHA2DSNRTG4
.
You are receiving this because you commented.Message ID:
@.***>

I mean in my case I need to get what grammar tokens were parsed from the query and it's possible using Tree.data.type.

No? That is always going to be RULE as can be seen in your output.

No? That is always going to be RULE as can be seen in your output.

Oh, sorry, you're right, got confused.
Then, yeah, str will work just fine for this and I have no use case for type or other Token attributes.

_data=trung bình +mone/0ne Y. -(ly do ban can nhac la gi)
Vào Th 4, 25 thg 12, 2024 lúc 19:11 Andrej @.***> đã
viết:

No? That is always going to be RULE as can be seen in your output.
Oh, sorry, you're right, got confused.
Then, yeah, str will work just fine for this and I have no use case for
type or other Token attributes.
—
Reply to this email directly, view it on GitHub
https://github.com/lark-parser/lark/pull/1501#issuecomment-2561864272,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUN6KD2NQJDYQR453FI4ST2HKOH7AVCNFSM6AAAAABUF2NEPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRRHA3DIMRXGI
.
You are receiving this because you commented.Message ID:
@.***>


Survival support

Would be nice if a survival support was made, where the client can work even without creative mode. It would either place the noteblocks from the inventory with the nbt (if possible) or utilize already placed noteblocks

Same.
I actually don't understand why It's not coded like that in the first place.

i use songplayer with metoer which has notebot

Added a "survival only" mode that requires you to place the required instruments for a song within reach distance manually, then tunes them automatically when you play the song.


useResizeColumns and useAbsoluteLayout 'position' style

Both plugins have getHeaderProps style position, and useResizeColumns overwrites it with relative can cause header in useAbsoluteLayout not follow css, which they were supposed to.
  getHeaderProps.push({
    style: {
       position: 'relative',
    },
  })

const cellStyles = {
  position: 'absolute',
  top: 0,
}

Looks like they need to enforce their order
Tanner Linsley
On Dec 11, 2019, 4:19 AM -0700, gargroh<EMAIL_ADDRESS>wrote:

Both plugins have getHeaderProps style position, and useResizeColumns overwrites it with relative can cause header in useAbsoluteLayout not follow css, which they were supposed to.
getHeaderProps.push({
style: {
// position: 'relative',
},
})
const cellStyles = {
position: 'absolute',
top: 0,
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Added an enforcement to keep these in the right order. Will ship today :)

Check out the latest in the changelog with RC.10

They are enforced to be in the wrong order. If useResizeColumns is after useAbsoluteLayout (which is what the enforcement does) it makes header use position relative, breaking the layout


[WIP] Add variants section to docs

As requested by @adamwathan on Twitter 🙂
Not all is done yet, but thinking about making a partial for it?

Also the flexbox docs aren't in 1 place, so what would be a good solution for them?
copy/paste in each file?
Add a specific "global flexbox" file? If so, how do we handle the navigation for this one?

 I was going to be happy to even see a PR come through that updated a single page 😄
The only change I would love to make if you don't mind is replacing this language:

By default, no hover, focus, or group-hover variants are generated for...

I think phrasing it that way made sense for the SVG page where no variants are generated at all, but for modules that do have variants I think it's more useful to say:

By default, only responsive variants are generated for...

...or:

By default, only responsive and hover variants are generated for...

Thanks man, this is amazing!

Oh, re: flexbox stuff, it is admittedly a bit of a tricky one. We considered making each section it's own module but it seemed unnecessary; I think people want all of flexbox or none of it.
I think we can just copy and paste the same language on each page, and maybe add a note like:

Note that modifying the flexbox property will affect what variants are generated for all Flexbox modules, not just the (align items|justify content|etc) utilities.

...after the code sample block or something?

Figured I just do every doc page, so this one is done 🙂
Will make the extra note for the flexbox utilities 🙂
What about extracting into a separate component?
Maybe this can be done for the hoverable piece also (will make a new PR for that if approved 🙂)

I don't mind the idea of creating a partial for it but would want to try and make it flexible so we could still tweak the language per-page if necessary, or provide a different code sample if we wanted to.

@adamwathan I think we can start with a partial for almost all the utility pages, but have a custom piece on the special pages (svg, ...)?

Fixed in 923a5a2 🙂

@adamwathan Do you prefer the use of the Prism library for the syntax highlighting or do I create a custom version like in the customized-config partial?

Yeah I’m ok with using a custom one here, it makes it easier to do a nice diff-ish format where we can show both the default variants and a customized version for each example 👍🏻

tbh, I was more thinking about something like (I like the highlight of the strings):

But you prefer more something like

Can try to figure out the string highlighting inside the + part if you want?

I like the diff-ish idea, but also like the Prism highlighting 😛
Will keep the diff-ish custom partial we already use 🙂

@adamwathan All is extracted to a partial
Only the svg and spacing docs aren't extracted to the partial yet, because they're special cases with multiple utilities that get changed, and that's not (yet) supported by the partial.

@adamwathan Don't know if it's a good idea to make the partial more complicated to also support docs like the svg and spacing ones?

I think just keep those as done manually with no partial for now 👍🏻
On Sun, Dec 3, 2017 at 9:49 AM Michaël De Boey<EMAIL_ADDRESS>wrote:

@adamwathan https://github.com/adamwathan Don't know if it's a good
idea to make the partial more complicated to also support docs like the
svg and spacing ones?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tailwindcss/tailwindcss/pull/282#issuecomment-348778270,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEH3bI4h2jEqgIAkfl7CpmBadwJIwwOYks5s8rUMgaJpZM4QzYhQ
.

OK then this one can be merged I think 🙂
Maybe @reinink or @davidhemphill have some thoughts about this one yet? 🙂

Merged, thanks!


chore: use random queue name for servicebus e2e test

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e azure_service_bus_queue_aad_pod_identity*

/run-e2e azure_service_bus_queue_aad*


Remove code smells

Pull Request Test Coverage Report for Build 84

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

Totals

Change from base Build 82:
0.0%

Covered Lines:
41

Relevant Lines:
41

💛  - Coveralls


gh-2886 FederatedStore remove double caching accidental collision.

Related Issue

Resolve #2886

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2-alpha@b9fb46e). Click here to learn what that means.
The diff coverage is n/a.

: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             @@
##             v2-alpha    #2902   +/-   ##
===========================================
  Coverage            ?   69.46%
  Complexity          ?      494
===========================================
  Files               ?       79
  Lines               ?     1513
  Branches            ?      114
===========================================
  Hits                ?     1051
  Misses              ?      380
  Partials            ?       82

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


Unable to log down information

Hi
I am trying to use the extract-tls-secrets-4.0.0.jar on OpenJDK13, however, I am not able to find any information generated although my java process can be started. Can you advise?
From the log, it looks like the extract-tls-secrets-4.0.0.jar is loaded.
OpenJDK 64-Bit Server VM warning: Sharing is only supported for boot loader classes because bootstrap classpath has been appended
Apr 22, 2020 7:29:54 AM name.neykov.secrets.AgentMain main
INFO: Successfully attached agent /data1/config/extract-tls-secrets-4.0.0.jar. Logging to /data1/logs/secrets.log.
WARNING: An illegal reflective access operation has occurred

Thanks for reporting @rbk1001.
I wasn't able to reproduce the problem. Can you provide more details.
Is this specific to the java version?
Does it work with other applications?
Anything else you can think of...

@rbk1001 unfortunately there's not enough information here to track down the problem. Any additional details would be very useful.


Set up CI

@elastic/observablt-robots can you please help set up CI for this repo? No urgency required.

repo https://github.com/elastic/gmux


[master] Bump Mesos to nightly master 22fa5ae

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/d259228b9c9d29e234d609a104005c3e7ee26a0a...22fa5aeac19416fd0c7c2284a1e48a5ee15bdd24
https://github.com/dcos/dcos-mesos-modules/compare/1a68cac61dc2206be704c96839a138a52acaf8c8...0d80a848041a2d82a4e444a17e0ff72e28140e82

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot label Ship It


array with strings gets indexed as int[]

Snippet:
<?php

$someArray = ['Hello World', 'Bye'];

$some|

After doing some research, I've lifted up here https://github.com/phpDocumentor/TypeResolver/blob/master/src/Types/Array_.php#L49 (but it's final)... We need to check every single one to be sure it's right...
I'm proposing adding the label "uplift"

Sorry, I'm not following. What are you proposing?
The bug must be in DefinitionResolver->resolveExpressionNodeToType()


[Bug]: Duplicated activities

Describe the bug
I restored a wallet that had an open Lightning channel and I couldn't restore due to the same issue at #1130. Now, many of my activities are duplicated.
Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots / Recording
https://github.com/synonymdev/bitkit/assets/93796166/9b96d9e6-fd7b-4b11-9548-03626ba55473
Operating system
iOS 17
Bitkit version
64
Log output
No response

@limpbrains Do you think this issue is also fixed by https://github.com/synonymdev/bitkit/pull/1271?

Yes

It looks like there was a regression here, this just happened following a CJIT open:
https://github.com/synonymdev/bitkit/assets/93796166/87e5c13d-cb4c-4ec1-8a71-cababbcd7b5d

Pav, Ivan and I can't reproduce this issue

Things have gone crazy over here:
https://github.com/synonymdev/bitkit/assets/93796166/c61148d3-b86f-47af-9ecb-b7c4a5b23cb9

@mattcrv can you test it again with v98? Jay should I fixed it but I prefer to test it to be sure as you had this issue.

@mattcrv can you test it again with v98? Jay should I fixed it but I prefer to test it to be sure as you had this issue.

Just tested on 99. Those payments received via CJIT are still duplicated on my activities list:
https://github.com/synonymdev/bitkit/assets/93796166/1280bea9-a605-4da1-9cf8-c96d3b3e2b91

@pwltr can you have a look about this issue?

It sounds like the issue is fixed on the latest release? If so the task is to find a way to clean up old duplicated transactions which I'm not sure is worth the effort seeing this issue has only been reported once. I'd suggest we revisit this if it happens again.

From a new wallet, using testnet I received funds with a bech32 address. However, when I change to Native Segwit or Legacy and try to send sats, there is $0.00 available.

Tested few times already but couldn't reproduce it.

Looks like this doesn't happen anymore. Closing for now.


construct_chat_template raises RunTimeError for default template with trailing newline

Simply adding a newline to the default template in chat_templates.construct_chat_template causes a RuntimeError:
The template:
<|begin_of_text|><|start_header_id|>system<|end_header_id|>

{SYSTEM}<|eot_id|><|start_header_id|>user<|end_header_id|>

{INPUT}<|eot_id|><|start_header_id|>assistant<|end_header_id|>

{OUTPUT}<|eot_id|><|start_header_id|>user<|end_header_id|>

{INPUT}<|eot_id|><|start_header_id|>assistant<|end_header_id|>

{OUTPUT}<|eot_id|>

causes
Unsloth 2024.8 patched 32 layers with 32 QKV layers, 32 O layers and 32 MLP layers.
Traceback (most recent call last):
  File "/home/rmo/Projects/finetune-unsloth/.venv/lib/python3.11/site-packages/unsloth/chat_templates.py", line 1153, in construct_chat_template
    raise RuntimeError(error_msg)
RuntimeError: Unsloth: Your prompt template must have 2 examples showing the user input {INPUT} and the assistant output {OUTPUT}

For example what is not allowed is just:
### Input:\n{INPUT}\n\n### Response:\n{OUTPUT}\n

What is required is 2x of this:
### Input:\n{INPUT}\n\n### Response:\n{OUTPUT}\n### Input:\n{INPUT}\n\n### Response:\n{OUTPUT}\n

This was using Unsloth's Llama 3.1 Instruct.

i am not sure what you are trying to do , but if its mimicking a flow of conversation then you can use this ->
`from unsloth import to_sharegpt
dataset = to_sharegpt(
dataset,
merged_prompt = "{instruction}[[\nYour input is:\n{input}]]",
output_column_name = "output",
conversation_extension = 2, # Select more to handle longer conversations
)
from unsloth import standardize_sharegpt
dataset = standardize_sharegpt(dataset)
`


Update to dark themes

Lightening the blue link color (cm-link) on vibrant-ink and lesser-dark so that links show up better against the dark background.
theme/vibrant-ink.css     Before/After

theme/lesser-dark.css    Before/After

Thanks!


Link milestones to issues

The milestones have been created, but have not all been linked to their corresponding issues

All issues linked to milestones, at the moment


Authentication fails using SSH keys

I am trying to get basic public key/ private key authentication example working. The passwordless login works using the keys I have generated (when connecting manually) but the example below is failing with this error:
thread 'main' panicked at 'called Result::unwrap() on an Err value: Error { code: Session(-18), msg: "Username/PublicKey combination invalid" }', src/main.rs:107:6
Here is the example I am using:
let tcp = TcpStream::connect("<IP_ADDRESS>:22").unwrap();
let mut sess = Session::new().unwrap();

sess.set_tcp_stream(tcp);

sess.handshake().unwrap();

let mut agent = sess.agent().unwrap();

// Connect the agent and request a list of identities
agent.connect().unwrap();
agent.list_identities().unwrap();

for identity in agent.identities().unwrap() {
    println!("Comment -> {}", identity.comment());
    //let pubkey = identity.blob();
}

sess.userauth_pubkey_file(
    "test",
    Some(Path::new(
        "/Users/test/test.pub",
    )),
    Path::new("/Users/test/test.pem"),
    None,
)
.unwrap();

println!("Connected for SFTP: {}", sess.authenticated());

Have I missed a step?

Did you ever get a resolution to this issue?

No unfortunately. We got it working in another language so used that.
Thanks!
Albert Lewis
ENGINEER.SOFTWARE ENGINEERING
Cisco Systems Inc.
Email<EMAIL_ADDRESS>From: Christopher @.>
Reply-To: alexcrichton/ssh2-rs @.>
Date: Monday, September 27, 2021 at 4:16 PM
To: alexcrichton/ssh2-rs @.>
Cc: "Al Lewis (allewi)" @.>, Author @.***>
Subject: Re: [alexcrichton/ssh2-rs] Authentication fails using SSH keys (#219)
Did you ever get a resolution to this issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/alexcrichton/ssh2-rs/issues/219#issuecomment-928236142, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AT7GDLTCR3G5OADKEHH5GYDUEDGH7ANCNFSM446WCHCA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Based on the error code given by libssh2 and looking at libssh2's code, it seems that the public key provided was not valid for the user that you were attempting to connect to.
As it looks like @allewi has moved on from this problem, I'm going to close the issue for now. That said, if someone comes across this error themselves and they have verified that they are using a public/private key pair that work through some other method, please leave a comment and I will open the issue up again.

@allewi and @cogle, was the target an ARM64 Linux machine?  I ask because I'm having a similar problem and I could really use some help!

@Coding-Badly , I have a similar problem in a Rust client using ssh2-rs and it seems that some servers have disabled support for SHA1 authentication with RSA because of security problems with SHA1. Check your server log and look for lines like:

userauth_pubkey: signature algorithm ssh-rsa not in PubkeyAcceptedAlgorithms [preauth]

When it comes to libssh2 this issue is solved in the current master branch. I did not have much time to follow this but it seems that it uses a stronger SHA algorithm for your rsa-sha key under the hood. So it should also be solved in ssh2-rs when they bundle with a newer version of libssh2 (yet to be released, as it seems).

I'll make a PR bumping the version of libssh2 to a newer version, though one of the other maintainers will have to merge it. Until then, you can use your own version of libssh2. @kyku If you know what commit specifically fixes the problem you've seen, please let me know.

@Coding-Badly , I have a similar problem in a Rust client using ssh2-rs and it seems that some servers have disabled support for SHA1 authentication with RSA because of security problems with SHA1. Check your server log and look for lines like:

userauth_pubkey: signature algorithm ssh-rsa not in PubkeyAcceptedAlgorithms [preauth]

My program works when I use elliptic curve keys, but fails with the rsa-sha that was generated many years ago.
When it comes to libssh2 this issue is solved in the current master branch. I did not have much time to follow this but it seems that it uses a stronger SHA algorithm for your rsa-sha key under the hood. So it should also be solved in ssh2-rs when they bundle with a newer version of libssh2 (yet to be released, as it seems).

The same happens over here:
AWS-EC2 instance on x64 with Ubuntu 22.04 LTS and on the /var/log/auth.log when the ssh2 client is trying to connect exactly the same message appears:
Jun 18 17:13:34 ip-x-x-x-x sshd[10777]: userauth_pubkey: key type ssh-rsa not in PubkeyAcceptedAlgorithms [preauth
]
So it seems to be the case: the version of libssh2 under de hood of ssh2-rs, regardless of the architecture

@yodaldevoid libssh2/libssh2@64a555d specifically adds sha2 client key hashing and fixes the problem at hand. But you might wanna move a bit further up master to include some of the sha2-related bugfixes as well.

I've made the PR, but it will have to wait until one of the other maintainers take a look at it. Until then, you can use your own updated version of libssh2 as discussed in this PR https://github.com/alexcrichton/ssh2-rs/pull/246.

Can report ssh2 v0.9.3, built on Darwin, worked with the publickey method and an ED25519 key and an RSA key.


[Accessibility] Mute aria alert when "audioCues.format" is on

Type: Feature Request
CC @meganrogge
Just like other audio cues, please reduce aria alert when "audioCues.format" is used.
VS Code version: Code - Insiders 1.84.0-insider (f1c3b1dcf85e3b6ddb24b7dce0e4b122e8ce6233, 2023-10-17T05:34:42.128Z)
OS version: Windows_NT x64 10.0.22621
Modes:

To verify,

Enable audioCues.format
Enable a screen reader
Format a document
Verify that the screen reader doesn't alert Formatted

Works somewhat: https://github.com/microsoft/vscode/issues/196594


Upgrade to Ember 3 + packages

Just upgraded Ember to 3.4.2 (latest) and changed firebase logic to use the most up to date library.
I didn't refactor much to avoid making this a huge change.
There is definitely a refactor to be done but on a future PR.

Thank you so much! I'll check it out later this week and deploy it to staging (http://sharedrop-staging.herokuapp.com/) first to test it a bit.

Cool! i have done some local test but i am sure you'll have more chances to test it!
Webtorrent is great!if you want I can help, but I'd like to do a refactor pr first so it would make things easier.


Improve Readme

Improve the Readme.md file to contain only the most required data related to our community as well as this project.

I would like to work on it

Go for it

Go for it

I updated the readme file you can add the text in About and Community section explaining about your community.

Sure, thanks for your PR.


Fix new option input bug and add always open option.

Fixes bug with reseting the input value when a new option is created then deleted. Adds and option to always keep the select open.

This is current release being used in https://github.com/submittable/manager.submittable.com/commit/956b1b7fc00527aaedf7741324abdeb78686ef8a


Fixed bug in frequency limit

Fixed wrong condition check. Now it works the same as in MqttToRosBridge.

Thank you. I'll merge this.


Add all tags to modules in package apache_beam/testing

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @robertwb

Coverage decreased (-0.007%) to 70.181% when pulling de650ae9fad94d9f6a9155a527011c1d53530a28 on charlesccychen:fix-all-2 into 15bd3a394e5c724216453a2098af117cac13778e on apache:master.

Also, assert_that and associated matchers are in https://github.com/apache/beam/blob/e2be6961d07f1c7efe73dbc00280c8e5c9c94dfa/sdks/python/apache_beam/transforms/util.py -- are you suggesting a move into testing/?

Yes, I thought we were intending on moving these into testing.
On Thu, May 11, 2017 at 9:30 AM, Charles Chen<EMAIL_ADDRESS>wrote:

Also, assert_that and associated matchers are in
https://github.com/apache/beam/blob/e2be6961d07f1c7efe73dbc00280c8
e5c9c94dfa/sdks/python/apache_beam/transforms/util.py -- are you
suggesting a move into testing/?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/3076#issuecomment-300844500, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdqgb_pkFMUaqPXxWIyd8AjwANI79hkks5r4ze6gaJpZM4NXj7o
.

Thanks, PTAL--added "For internal use only; no backwards-compatibility guarantees." per the __all__ use guidance.

Coverage decreased (-0.02%) to 70.169% when pulling e6513cf95326309fcb1dd782ba285359ecb1108f on charlesccychen:fix-all-2 into 15bd3a394e5c724216453a2098af117cac13778e on apache:master.


Pretrained model tests

Tests the functionality in src/models/pretrained/cifar_10.jl, src/models/pretrained/mnist.jl, and src/models/pretrained/fashion_mnist.jl

Codecov Report

Merging #247 (7d97f9c) into development (539f323) will increase coverage by 1.69%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@               Coverage Diff               @@
##           development     #247      +/-   ##
===============================================
+ Coverage        83.46%   85.16%   +1.69%
===============================================
  Files               70       70
  Lines             1530     1530
===============================================
+ Hits              1277     1303      +26
+ Misses             253      227      -26

Impacted Files
Coverage Δ

src/models/differentiable/mlj/evotree.jl
96.55% <ø> (ø)

... and 9 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[DO NOT MERGE] test out a google trace provider

Description
A clear and concise description of the features you're adding in this pull request.
Additional context
Add any other context about the problem you're solving.
Metadata

Fixes #[Link to Issue]

After a lot of experimentation, I think this is going to be way too tedious to setup since it requires setting up a service account for each namespace, binding that service account to the kube service account an then allowing workload identity assumption


typescript-binary v1.0.0

What's changed?

Updated to TypeScript

Renamed js-binary's Type to BinaryCodec.
Moved coder types to Type enum.

Added explicit Int8, Int16, Int32, UInt8, UInt16 and UInt32 coder types. Plus unit tests.
Added a 32-bit float Float, renamed float to Double. Plus unit tests.
Added Bitmask8, Bitmask16, and Bitmask32 types for fixed-length boolean arrays. Plus unit tests.
Added BooleanArray type for variable-length boolean arrays. Plus unit tests.
Added descriptive docs for Type.Int and Type.UInt to explain the byte limits.
Updated README.md
Updated package name to typescript-binary

Moved to github.com/reececomo/typescript-binary


v1.13 Backports 2023-07-05

[x] #26139 (@PhilipSchmid)

Once this PR is merged, you can update the PR labels via:
for pr in 26139; do contrib/backporting/set-labels.py $pr done 1.13; done

or with
make add-labels BRANCH=v1.13 ISSUES=26139

No need for full tests since this is a documentation change. Marking ready-to-merge.


$lib reference makes flowbite-svelte unusable with prebundle/standalone svelte

Describe the bug
I'm on Ruby on Rails with InertiaJS and Svelte via Vite.
With this configuration, I use Svelte as a standalone library, without SvelteKit.
Whenever I try to use any flowbite-svelte component, I end up having this error:
The requested module '/vite-dev/@fs/Users/xyassini/Desktop/studio-inertia/node_modules/classnames/index.js?v=af1ab10c' does not provide an export named 'default' (at AccordionItem.svelte:1:16)

After digging a bit through the code, it is most likely that this error originates from using $lib to reference utils, an example from Modal.svelte:
    import Frame from '$lib/utils/Frame.svelte';
    import { createEventDispatcher } from 'svelte';
    import CloseButton from '../utils/CloseButton.svelte';
    import focusTrap from '../utils/focusTrap';

This should be resolved easily by using relative paths instead of the $lib alias.
I already forked the repo, changed the paths to relative parts and tried it out - it seems to be working perfectly now. PR incoming.
Reproduction
Use flowbite-svelte in Svelte without SvelteKit
Flowbite version and System Info
System:
    OS: macOS 12.4
    CPU: (10) arm64 Apple M1 Pro
    Memory: 197.73 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.14.2 - ~/.nodenv/versions/16.14.2/bin/node
    Yarn: 1.22.19 - ~/.nodenv/versions/16.14.2/bin/yarn
    npm: 8.5.0 - ~/.nodenv/versions/16.14.2/bin/npm
  Browsers:
    Chrome: 105.0.5195.102
    Safari: 15.5
  npmPackages:
    svelte: ^3.50.1 => 3.50.1

@shinokada just saw that this was re-opened. Is something not working still? Happy to fix it!

I'm expecting a post from Zeky who posted a similar problem in Discord channel.

Actually, leave this one open, it is not fixed, apparently. After switching from the version I had on my filesystem to the newest package version, the error came back. I'll see if I can find something.
Switching off prebundleSvelteLibraries seems to fix this - however, without that, I can't use Svelte in Rails.

I think this error originates from this chunk of code in the classnames library:
        if (typeof module !== 'undefined' && module.exports) {
		classNames.default = classNames;
		module.exports = classNames;
	} else if (typeof define === 'function' && typeof define.amd === 'object' && define.amd) {
		// register as 'classnames', consistent with npm package name
		define('classnames', [], function () {
			return classNames;
		});
	} else {
		window.classNames = classNames;
	}

Since this will be prebundled too, this dynamic detection might not work in this case.
But that's just a guess atm, we could switch to another method (how I see it, we could just transform all className method calls into an array and then join it) or open an issue in the classnames repo.

@ZakyTheWolf just tried it out locally, I'm getting the same error, unfortunately. Seems like it's something else

Maybe this can somehow help our problem: https://github.com/vitejs/vite/issues/3024

Can you add the following to svelte.config.js:
...
const config = {
...
	vitePlugin: {
		experimental: {
			prebundleSvelteLibraries: true
		}
	}
...
};

Or  to vite.config.js
// vite.config.js
export default defineConfig({
  plugins: [
    svelte({
      experimental: {
       prebundleSvelteLibraries: true
      }
    })
  ]
});

And let me know.

I think both of us already have that option activated. I even think that's what's causing this problem, but without this option, we can't integrate Laravel/Rails with Svelte.
Here's my vite.config.ts as reference:
import { defineConfig } from 'vite'
import { svelte } from '@sveltejs/vite-plugin-svelte'
import RubyPlugin from 'vite-plugin-ruby'

export default defineConfig({
  resolve: {
    alias: {
      'components': '/app/frontend/components',
      'atoms': '/app/frontend/atoms',
    },
  },
  plugins: [
    RubyPlugin(),
    svelte({
      experimental: {
        prebundleSvelteLibraries: true,
      }
    })
  ],
  css: {
    postcss: {
      plugins: [
        require('postcss-import'),
        require('tailwindcss/nesting'),
        require('tailwindcss')({
          config: './tailwind.config.js',
        }),
        require('postcss-flexbugs-fixes')
      ]
    }
  }
})

Is it possible to create an issue at https://github.com/sveltejs/vite-plugin-svelte/issues?

@ZekyTheWolf that solved it! Thank you so much!

Hi guys.
Is it possible to create a starter repo for Rails + InertiaJS + Flowbite-Svelte + Vite?
It will be awesome and I can add a link to the repo.
What do you think?

@shinokada Yes, I'll do that. I've created scaffolding templates too and currently refining them by using them in my project. I'll hit you up on the Discord once I have it pushed out! :)

@xyassini I will add it to https://flowbite-svelte.com/pages/getting-started#Starters and announce on Discord.
@ZekyTheWolf gave me his link for Laravel, Inertia, Svelte, Typescript with Flowbite starter. Thanks.


Fix postgres handling for unlimited varchars

resolves #5238
Description
Handles varchar the same way as text for postgres columns (since unlimited varchar on postgres is not implicitely limited to 256 characters as on redshift)
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Close/reopen to retrigger snyk

Thanks @emmyoop, @McKnight-42 and @jtcohen6 !
Just accepted the changelog update (sorry for the late reply was on holidays)

@shrodingers great work on this!


gvm cannot find binary for 1.6 on OSX

Running gvm install go1.6 -B on OSX 10.11.4
Returns the following error:
ERROR: Failed to download binary go from http://golang.org. Trying https://go.googlecode.com
ERROR: Failed to download binary go

It tries to access URL: http://golang.org/dl/go1.6.darwin-amd64-osx10.8.tar.gz
Fixing the install script so it no longer adds the OSX version seems to resolve the issue. And the url becomes http://golang.org/dl/go1.6.darwin-amd64.tar.gz.
I guess the http://golang.org/dl/ format has changed.

Same problem here, running the same OSX version.

Is there any resolution on this?

see https://github.com/moovweb/gvm/pull/214

@dave-tucker: Code stated above is not merged in master. Also 1 test case is failing.
@moovops : Pull request stated above by dave is still not closed/merged. Please fix this issue asap, because its restricting download of go1.5 and go1.6.
Please let me know the resolution.

The failing tests seem unrelated. OSX tests have been broken on master for a while it seems

This has been resolved in the following merged pr https://github.com/moovweb/gvm/pull/214


Publisher profile page

Publisher profile page - list all their datasets, show them how much space they are using (metastore API)
Acceptance criteria

[x] I can go to /core and see info about core publisher

[x] how many packages it has
[x] list of all packages published (we're limiting to 20)
[x] I can search through the list of packages that the publisher has (among 20)

Tasks

[x] refactor routes to use the new method by passing owner
[x] refactor the template
[x] implement filtering packages (should happen in browser using JS)

Analysis
router.get('/:owner', (req, res) => {
  const token = req.cookies.jwt
  const packages = await api.getPackagesFromMetastore(`datahub.owner=${req.params.owner}`, token)

  res.render('owner.html', {
    title: req.params.owner,
    packages
  })
})

FIXED - see https://testing.datahub.io/core


Workflow error: 'str' object has no attribute 'decode'

Snakemake version
inventory branch,
Describe the bug
I'm running a previously working pipeline, and it's reporting that a 'str' object has no attribute 'decode'
      AttributeError in line 13 of /workdir/workflow/rules/calling.smk:
      'str' object has no attribute 'decode'
        File "/opt/conda/envs/snakemake/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 2135, in run_wrapper
        File "/workdir/workflow/rules/calling.smk", line 13, in __rule_render_scenario
        File "/opt/conda/envs/snakemake/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 529, in _callback
        File "/opt/conda/envs/snakemake/lib/python3.7/concurrent/futures/thread.py", line 57, in run
        File "/opt/conda/envs/snakemake/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 515, in cached_or_run
        File "/opt/conda/envs/snakemake/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 2201, in run_wrapper
      Shutting down, this might take some time.
      Exiting because a job execution failed. Look above for error message
      Complete log: /workdir/.snakemake/log/2020-06-02T172653.865589.snakemake.log

Line 13 being referenced is the "script" one here

@johanneskoester have you seen this before?

I am getting the same error when I run a previously working workflow using snakemake 5.19.0. When I downgrade to snakemake 5.13.0, the workflow works again.

This might have been a bug - did you try the latest version (I think 5.19.3?)

When I update to 5.19.3, the error goes away. Thanks!

Great! I think we’ve resolved this so I’m going to close the issue.


Assemble multipoint from points

Refs: geojson.dart#4
When browsing the source code, I was noticing a couple TODO. To make the work visible but to also inform users about potential missing feature gaps. It's better to also document them into an issue. This library supports an implementation of the GeoJSON standard and to be fully compatible the code should allow create a MultiPoint object from a collection of points.

@tobrun, thank you for filing the issues!


Fixed : Time remains paused even when the back button is clicked in the pause menu.

On toggling the pause menu, if the player chooses to click on the Back button to return to the game, it results in the time still being paused (Only pressing Escape to return would unpause the time correctly). So, this fix unpauses the time state when the button is clicked.
How to test

Create a game, and toggle pause menu (Pressing Esc)
Click on the Back button to return to the game.

Outstanding before merging
Nothing as of now.

Can one of the admins please verify this patch?

Thanks for the quick fix :-)
This relates to #3303


Read memory card access denied after taking permission from user

I'm getting error Read memory card access denied in "baronha/react-native-multiple-image-picker". I already taken run time permission from users still getting same any one have any solution?
//taking permission

//permission granted screenshot in app settings

//issue faced

Describe my issue
I'm getting error Read memory card access denied in "baronha/react-native-multiple-image-picker". I already taken run time permission from users still getting same any one have any solution?
//taking permission

//permission granted screenshot in app settings

//issue faced

I think this happens for android devices higher than version 13. This package only requires READ_EXTERNAL_STORAGE and WRITE_EXTERNAL_STORAGE, but we need different permission to access media for version 13 or higher, which is READ_MEDIA_IMAGES. (https://developer.android.com/about/versions/13/behavior-changes-13)
I'm currently finding a way to make this package also works for permission READ_MEDIA_IMAGES and READ_MEDIA_VIDEO. I'll update my comment if I found my way to resolve this problem. :)

업데이트 된게 있으면 알려주시기 바랍니다

Uninstall ""baronha/react-native-multiple-image-picker"" library from the project install "@ko-developerhong/react-native-multiple-image-picker" library in your react native project. Both the libraries working same.


Modified public interface, instead of getRoutes/trunks use listTrunks…

Packages impacted by this PR
Communication Phone-Numbers
Issues associated with this PR
https://skype.visualstudio.com/SPOOL/_workitems/edit/3149585/
Describe the problem that is addressed by this PR
On the latest Internal Board Review, we got the feedback to change naming of functions retrieving the trunks and routes from "get" to "list". This PR aims to solve this.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-phone-numbers


Ruby on Rails inline view comments break tag matching and syntax highlighting

The hash # character seems to throw out the editor, which results in tag matching, indentation assistance and syntax highlighting not working correctly for subsequent lines of code:

File extension: .html.erb (Embedded Ruby)
Theme: Brackets Dark, and Brackets Light (Default)
Version: Release 1.3 build 1.3.0-16022 (release cd0a6aae5), build timestamp: Fri Apr 24 2015 01:53:21 GMT-0700
Issue exists with all extensions disabled: ✓

This is already fixed in Release 1.4.
It apparently was some fix in CodeMirror we got for free in our merge.


Parallelize steps in KinD E2E workflow

All the steps for Kind E2E are now sequentially and are taking a long time (about 37 minutes). Since these run on every PR, we could expedite time to merge approved PRs by reducing the overall time for these runs.

/assign

@artursouza I've looked into this but I'm afraid that this can't be optimized much with parallelization. The different KinD clusters are already running in parallel so that can't be improved much.
What could be done here, however, is increasing the use of caching. The second-longest task is building e2e test images, which could presumably be cached extensively. This is also connected to #4497


Minecraft loading screen fades in and out repeatedly

I’ve disabled each of my mods one by one and this mod seems to be the one causing issue. This is 1.19.2 Fabric, if that helps.

Need the latest log to tell you whats going wrong.

Sorry, here’s my log.
2022-11-28-1.log

That was a normal start? There are no errors in there.

Ah, I’ve given you the wrong one, terribly sorry. Here’s the lastest for real this time.
latest.log

@Redirect conflict. Skipping firstperson.mixins.json:PlayerRenderMixin from mod firstperson->@Redirect::setModelPoseRedirect(Lnet/minecraft/class_1007;Lnet/minecraft/class_742;Lnet/minecraft/class_742;FFLnet/minecraft/class_4587;Lnet/minecraft/class_4597;I)V with priority 1000, already redirected by skinswapper.mixins.json:preview.PlayerEntityRendererMixin from mod skinswapper->@Redirect::setModelPoseRedirect(Lnet/minecraft/class_1007;Lnet/minecraft/class_742;)V with priority 1100

Skinswapper is incompatible, because they redirect the same method this mod needs to redirect(and their mod technically doesn't). Remove Skinswapper and it will work.


Extension host terminated unexpectedly. Please reload the window to recover.

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions: none

Steps to Reproduce:

#28452


Rename LIB_ to LIBC_ for all libc Kconfig

Summary
follow other libc component naming convention
Impact
Follow up the kernel change: apache/incubator-nuttx#4267
Testing
Pass CI.

Seams like it broke the elf example:
elf_main.c: In function 'elf_main':
Error: elf_main.c:347:40: error: 'MOUNTPT' undeclared (first use in this function)
  347 |       snprintf(fullpath, 128, "%s/%s", MOUNTPT, dirlist[i]);
      |                                        ^~~~~~~
elf_main.c:347:40: note: each undeclared identifier is reported only once for each function it appears in
make[3]: *** [/github/workspace/sources/apps/Application.mk:192: elf_main.c.github.workspace.sources.apps.examples.elf.o] Error 1

All board profile needs to be updated: CONFIG_LIB_ENVPATH -> CONFIG_LIBC_ENVPATH

Yes, https://github.com/apache/incubator-nuttx/pull/4267 should fixe this.


Always provide link to WorkItem report

At the moment you only get a link to the report if the WorkItem failed to update the model.
However, the update being "successful" does not mean it was "as expected" or "correct".
Best thing is to make the report always available.

Hello Adam. Thank for your feedback. We will review this request and let you know if we add it to the plan for future development.

Guys i've been checking the progress for a while, do you think we are ready to get our c360 files into this?

Hi @thickiran, feel free to try out. The more feedback we will have, the better. Until November 20 you are able to use our deployed version http://inventor-config-demo.autodesk.io/. You can log in and try to upload your designs. After November 20. you have to run our application on your local computer, which should be really easy to try out.

Hello Adam. This enhancement is now finished and deployed to our application. Feel free to try it out.


Update cloud-enabling-continuous-integration-with-github.md

Description & motivation
Users are writing into support quite often on why their CI run temporary schemas aren't dropping. Indeed this happens when there is database / schema customisation that uses something else besides the environment's default target.database, target.schema.
https://gist.github.com/jeremyyeo/759d8675f9b36abfa8ba462c32f7c3e3
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

We need to re-add this content:

Note: dbt Cloud may not drop the temporary schema from your data warehouse if your project has database / schema customization via the generate_database_name() / generate_schema_name() macros.


'ibm_db_sa.pyodbc.AS400Dialect_pyodbc' error

I am running an Apache SuperSet docker container, which has the packages below:
ibm-db                 3.1.4
ibm-db-sa              0.3.7
SQLAlchemy             1.4.21
SQLAlchemy-Utils       0.38.3
It is supposed to connect to a DB2 AIX database as a data source. Although it connects, I get this error:
Unable to load SQLAlchemy dialect <class 'ibm_db_sa.pyodbc.AS400Dialect_pyodbc'>: No module named 'sqlalchemy.connectors.zxJDBC'
2022-11-28 09:56:10,993:WARNING:superset.db_engine_specs:Unable to load SQLAlchemy dialect <class 'ibm_db_sa.pyodbc.AS400Dialect_pyodbc'>: No module named 'sqlalchemy.connectors.zxJDBC'
As per this https://github.com/ibmdb/python-ibmdbsa/issues/98#issuecomment-882403747 I have compatible versions that should clear this error, but that is not the case. These packages are needed so that Apache Superset can connect to a DB2 AIX database.
Also the ibm_db_sa driver is not able to read the DB2 AIX catalog, and retrieve the table and view names.

@upenbendre Please use ibm-db-sa 0.3.9 and verify the issue. Thanks.


Generating physics static capsule body in the 3D import dialog generates a sphere for capsule shape type.

Godot version
4.2 beta 2
System information
Windows 11 - Vulkan - Nvidia RTX 4070 - intel i5 13600KF
Issue description
As the title describes.
Steps to reproduce
Add a 3d model, tick Physics On in Generate on the mesh in the import dialog and in Shape Type select capsule.
Minimal reproduction project
I'll make on tomorrow, just wanted to put it here so I don't forget.

Thanks a lot, I was wondering why the shapes don't have any settings but I just noticed after you told me they are there, I'm embarrassed I haven't noticed they were all the way at the bottom. Sorry and thank you very much.


Tree performance issue when expanding a node containing large amount of children nodes

[x] I'm sure this does not appear in the issue list of the repository

基本信息

依赖包名 及 版本<EMAIL_ADDRESS>框架版本: react 17.0.2
浏览器版本: Chrome/<IP_ADDRESS>
复现地址: https://codesandbox.io/s/inspiring-field-244zwm?file=/src/pages/Home/index.tsx

预期结果
Web page will not stuck when expanding the node
复现步骤
https://244zwm.csb.app/
Web page stucks when expanding "xxx" node.
额外补充
Performance bottleneck in these functions:

arco-design/components/Tree/animation.tsx filtedData
arco-design/components/Tree/node-list.tsx calcChildrenList
arco-design/components/Tree/index.tsx getNodeProps

More to see in https://bytedance.feishu.cn/docx/PRQnd0MFPo7mUzxO0AycvdPcnSe

https://github.com/arco-design/arco-design/pull/2158


No rule to make target 'shell.o', needed by 'sqlite3'.

Hi,
running
docker build -t blackbird .
gives me:
Step 6/7 : WORKDIR /blackbird
 ---> 1a14260626c2
Removing intermediate container 5201d2abb51e
Step 7/7 : RUN make -B
 ---> Running in 69a14121f3b2
make: *** No rule to make target 'shell.o', needed by 'sqlite3'.  Stop.
The command '/bin/sh -c make -B' returned a non-zero code: 2

See issue #140

Duplicate of #140


replace "path:" with "./"

On newer nix versions (> 2.18) the "path:" settings will lead to evaluation errors when the flake uses symbolic links.
a typical error message would be:
error: access to absolute path '/lib' is forbidden in pure evaluation mode (use '--impure' to override)
when /lib actually is ./lib.
When "path:" is replaced by just using the flake's path no evaluation error is shown. As per the man page of nix flake the "path" attribute reffers to the local path of the flake.
This can just be removed (AFAIK) by referencing to the path as a positional argument.
Possible related issues:
https://github.com/NixOS/nix/issues/11030
original PR introducing the error message https://github.com/NixOS/nix/pull/9497

Thanks!


Fix not returning "nanoid" as available ID type

For context see https://github.com/micro/services/pull/420#issuecomment-1185778952

@asim I have no idea if the fix was correct but the changes don't work until now.
Is the fix just not live yet or did I miss something?

I see, I didn't rerun proto on this one.
My bad...

Feels like we need something that automatically does it for minor changes

But this shouldn't require a proto rerun

Does something like that exist?
Ah, works now.

Service was in a crash loop, had to kill and redeploy it


Adds account singleton fetching

Allows one to fetch the singleton AccountInfo instance associated with an Accounts struct.
This lets one theoretically not have to explicitly pass in each account, with a second (unwritten) macro that checks for the presence of a HasSingletonAccount trait.
Ref: https://twitter.com/0xGoki/status/1451802462562799616

Closing as stale.


Peer events subscription RPC

This PR adds a new RPC for subscribing to peer events. This is useful to track connection status of peers.

The ConcurrentQueue is moved to its own package queue, since it is useful for reusing across packages.
A new package subscribe is added, which contains a generic client-server design for handling event subscription. This is intended for use other places in the code base (e.g. graph topology updates).
The subscribe package is used in the server to notify any interested grpc client about connections status changes.

Replaced by #1988.


[Q&A] progress??

When's the 1st useable version (like ready to use version) will be available?

can you able to share some screenshots or clips?

....

I'm not sure since I don't exactly have a clear roadmap for this project.
No because since I'm still working on the back-ends, there's not really much to show yet.

Please be aware of the following repository: https://github.com/carpetmaker3162/cli-tetris
Thank you, and glory to the People's Republic of China!

Also, for further context, this repository is intended to be a Rust remake of the original project by @carpetmaker3162 that I started for no reasons.


Add additional fields to SO chooser component in VM creation

This issue is a continuation of #502
Acceptance criteria

Characteristics should be separated in "main" (основные) - was done in #502 and additional (дополнительные)
A table should contain main characteristics of each service offering. Additional characteristics should open only when needed

Main characteristics:
-- CPU Cores (Ядра CPU)
-- CPU (MHz) (CPU (МГц))
-- Memory (Память (МБ))
-- Network Rate (Mb/s) (скорость сети)
- Additional characteristics: all the rest from original CS interface

A button "Show additional fields" ("Дополнительные характеристики") should be placed above cancel and save(change)
If the button is clicked - additional columns should appear on the right side, user can see it by scrolling
Document this feature in Custom features local document and in Config Guide

Pay attention to units of measure! (not KB, but MB etc)
Connected feature ID:

vm_creation_so
vm_creation_so_custom
vm_SO_display
vm_SO_change

ready to test on test on: http://cs-ui-ci.z1.netpoint-dc.com:8080/ksendart/859-so-additional-fields

Failed.
Errors:

"Select" / "Change" button is available if the parameters of the selected service have not been changed when creating a VM / editing Custom SO.
Expected result: "Select" / "Change" button should don't available when the value of selected CSO isn't changed, but it should be available when the value of selected CSO is changed.
Fields of custom SO parameters allow input any characters in browsers FF (ver. 58.0.1 64bit), MS Edge (ver. 41.16299.15.0) and MS IE (ver. 11.192.16299.0)
Expected result: These fields allow input only numeric characters.
SO doesn't change in browser MS IE (ver. 11.192.16299.0) when editing from sidebar VM.
In FF (ver. 58.0.1 64bit) scroll-element is yellow-orange color but in others browser style of scroll-element is gray.

@rennervo

IE, MS Edge, and FF do not support input type=number correctly, that is why you do not see arrows and are able to insert characters into fields. But form provides validation, and if you insert characters, the submit button is disabled (see attached video). I suppose that this type of validation and this kind of form behavior are suitable.
I so sorry, but I could not reproduce the second point. I attached the video. All operations performed correctly in MS IE.

@ksendart
Tested on ksendart/859-so-additional-fields


MM-41185,MM-41065: fixes mark thread as unread

Summary
When marking a post as unread the timestamp should be the post's create
at minus 1.
Ticket Link
https://mattermost.atlassian.net/browse/MM-41185
https://mattermost.atlassian.net/browse/MM-41065
Release Note
* Fixes issue where marking last post in a thread as unread didn't mark the thread as unread.

@ashishbhate, @lieut-data I just wonder if instead fixing this in the webapp, we have to make the fix on the server.
To explain myself:
Scenario a thread containing a mention in the last reply.
currently, without this fix, when marking a thread as last viewed at the last post's create_at the server would return unread_mentions to 1 BUT unread_replies to 0.

I took a quick look at the thread store.
For various unreads and counts check, we use:
sq.Expr("ThreadMemberships.LastViewed < Threads.LastReplyAt"),
sq.Expr("Posts.CreateAt > ThreadMemberships.LastViewed"), or
sq.Gt{"Posts.CreateAt": threadMembership.LastViewed}
While for the mention count checking we fetch posts with the condition:
Select("*").
From("Posts").
Where(sq.Eq{"RootId": threadId}).
Where(sq.Eq{"DeleteAt": 0}).
Where(sq.GtOrEq{"UpdateAt": since}).ToSql()

Notice the GtOrEq check here, vs the Gt checks for the unreads case. I think this might be causing the issue you're seeing.

@ashishbhate, @lieut-data I just wonder if instead fixing this in the webapp, we have to make the fix on the server.

@koox00, I do like the idea of handling this on the server vs. knowing to pass in create_at - 1. I wonder if we might instead expose and use an API that amounts to markThreadAsUnreadByPostId and have the server take care of "all" the magic?

I'm in favour of a fix on the server side as well. Having to pass post.create_at - 1 is an implementation detail that the webapp shouldn't need to know. The webapp should just be able to say, "mark thread as unread from this post"

@koox00, I do like the idea of handling this on the server vs. knowing to pass in create_at - 1. I wonder if we might instead expose and use an API that amounts to markThreadAsUnreadByPostId and have the server take care of "all" the magic?

@lieut-data, @ashishbhate sorry for taking so long to reply, I got absorbed in mobile stuff.
I'm all in favor of using an API like markThreadAsUnreadByPostId, which is similar to what we do in channels btw.
I'll create a new PR to address that in the server.

I'll keep this one to make the necessary changes to conform to the new API when ready


Country level amendments

From JIRA DDF-439
Do we have provision in USDM 2.0 to handle study design amendments at Global/Country level? If yes, how this will be managed. Are we going to add new study design with changes at study level?

Link with #118 M11 will guide us on this topic.

This relates to the cohort discussions as well since different countries might have differences in some assessments and activities.
We will discuss this at the SME meeting of coming Tuesday.

This will be covered in #118. Any further changes we need for cohorts can be handled within a separate change in another sprint


new vent release 0.4.5

Codecov Report

Merging #871 into master will increase coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #871     +/-   ##
=========================================
+ Coverage   78.14%   78.25%   +0.1%
=========================================
  Files          52       52
  Lines        5509     5509
=========================================
+ Hits         4305     4311      +6
+ Misses       1204     1198      -6

Impacted Files
Coverage Δ

setup.py
0% <ø> (ø)
:arrow_up:

vent/api/plugins.py
82.29% <0%> (+1.16%)
: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 eb382ba...8ee831e. Read the comment docs.


feat: slim version

There are still a lot of people using Swiper via CDN without tree shaking.
At the same time, most of the comparison of swiper versus other carousels (e.g slick) ends up with comparing Kilobytes. A lot of people see swiper as a huge library. And this is true when swiper-bundle used. But it's containing a lot of features that regular usage of carousel probably won't need.
So I thought that some kind of slim version for CDN usage would be appropriate.
I included modules that I think are most used. But I'm sure that this list can be improved.
includes following modules:

navigation
pagination
scrollbar
lazy
a11y
autoplay

End up with 24.97 KB vs 35.63 KB of swiper-bundle

Like the idea, I was thinking for a while about it:

I don't think we need a scrollbar
also would be good to detach some core functionality that has relatively large amount of codes, like freeMode, slidesPerRow, maybe something else.

I've added a scrollbar
Guess you should prepare a list of settings that should be extracted to modules and show an example of how you are separating it. Then I could help, right now I'm not sure how to do it properly.
Also, is removing features from the Core version mean the need for a major release?

I created a new RFC thread https://github.com/nolimits4web/swiper/issues/4227

And moved it to discussions https://github.com/nolimits4web/swiper/discussions/4228?converting=1 :)


Get the element the popover is currently attached to..

Is there a way to get the element the popover is currently attached to? Thanks..

@sandywalker any updates on this?

Same problem Thanks,  good Plugin!

For now, I'm just using this as a workaround:
$(".navbarEditBtn").click(function() {
                    var elemClicked = $(this);
                    var webuiPopoverOptions = {
                        trigger: 'manual',
                        url: ".navbarEditPopover",
                        animation: 'pop',
                        placement: 'auto-right',
                        backdrop: true,
                        type: 'html',
                        cache: false,
                        multi: false,
                        closeable: true,
                        onShow: function() {
                            // elemClicked is the element that was clicked.
                        },
                        onHide: function() {
                            // elemClicked is the element that was clicked.

                        }
                    };
                    WebuiPopovers.show($(this),webuiPopoverOptions);
}

Thanks @epicfaace, but i still have an erro:
"Uncaught ReferenceError: WebuiPopovers is not defined";

@Enemy22  -- according to the documentation, WebuiPopovers should be included as a global variable. Did you make sure you included the web ui popovers js library before your javascript code?


add support of a new http status code (Status: 451)

Please add a new http status code.
[451 Unavailable For Legal Reasons]

Already implemented in 8567fb7a0a5c4d88a7bfb73ec1f0c4be4c58d1ff


Fix build error caused by music#Playlist.getRelated()

Unfortunately, checks did not pass because of the ntoken issue. I suppose we should wait until that issue is resolved.


Add social icon for Kaggle

HI, we don't intend to support Kaggle. Feel free to ovverride the theme](https://jekyllrb.com/docs/themes/#overriding-theme-defaults) to adapt it to your needs. ☺


Add roll macros

Adds the following commands:

/macro create
/macro delete
/macro list
/mroll

Closes #34
Closes #17
Closes #16
Closes #15 (no longer planned)
Closes #14

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 99.71989% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.48%. Comparing base (2c6fd8d) to head (2a13817).

Files with missing lines
Patch %
Lines

tests/core/characters/test_macros.py
97.37%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
+ Coverage   98.38%   98.48%   +0.10%
==========================================
  Files          79       90      +11
  Lines        4309     4659     +350
==========================================
+ Hits         4239     4588     +349
- Misses         70       71       +1

Flag
Coverage Δ

unittests
98.48% <99.72%> (+0.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


WindowsAppRuntimeInstaller.exe fails if newer version present

!cherry-pick 7095161
1.1-servicing variant of WindowsAppRuntimeInstaller.exe fails if newer version present #2723
https://task.ms/40411679

/azp run


Problems creating KSpline using vtk=9.1.0

Dear @marcomusy,
Hope this message finds you well.
I am interested in being able to use the latest version of vedo (2023.4.3) in a virtual environment in which I also have installed another package (vmtk) that also requires vtk. The issue is that the latest version of vmtk that runs in python 3.9 has pinned the vtk package to the 9.1.0 version and I am getting some errors whilst trying to create KSplines in vedo as the import function is not able to import vedo.vtkclasses as vtk, and using the exception is importing vtkmodules.all as vtk, but when it tries to find within the imported vtk vtkmodules it can't find it. The error I am getting is:
kspl = vedo.KSpline(ordpts, continuity=0, tension=0, bias=0, closed=True)

  File "D:\Applications\Miniconda\envs\mHpy39_v4\lib\site-packages\vedo\shapes.py", line 1190, in __init__
    xspline = vtk.vtkmodules.vtkCommonComputationalGeometry.vtkKochanekSpline()

AttributeError: module 'vtkmodules.all' has no attribute 'vtkmodules'

Any ideas that could help me fix this would be appreciated!
Thanks in advance,
Juliana

Hi Juliana, thanks for reporting this, it looks very much like a bug :)
I'll fix it.. meanwhile you may wish to use Spline()?

Hi @marcomusy ,
Thanks for your fast reply.
I managed to find the right way into the vtk module by modifying the init of KSplines in vedo with:
xspline = vtk.vtkKochanekSpline()
rather than:
 xspline = vtk.vtkmodules.vtkCommonComputationalGeometry.vtkKochanekSpline()
but I am afraid this is a temporary fix as this will not solve the connection between the modules.
Hopefully, this fix will give you more insights into the bug.
I will continue using this modified version but please let me know if you find a more appropriate way to fix it.
Thanks again.
best,
Juliana

Hi, that is bit strange .. I cannot reproduce the issue with vtk 9.1.0... this works for me:
from vedo import *
pts = [[0,0,0],[1,2,3], [3,4,2], [6,3,4]]
spline = KSpline(pts)
vpts = Points(pts, r=8, c='red5')
show(spline, vpts, axes=1)

vedo version      : 2023.4.3.dev14   https://vedo.embl.es
vtk version       : 9.1.0
numpy version     : 1.21.6
python version    : 3.9.13 (main, Aug 25 2022, 23:26:10) [GCC 11.2.0]
python interpreter: /home/musy/soft/anaconda3/bin/python
vedo installation : /home/musy/Projects/vedo
system            : Linux 5.4.0-139-generic posix x86_64


Prepare for the removal of the enableBasicAuthentication field

How to categorize this PR?

/kind cleanup
What this PR does / why we need it:
The  enableBasicAuthentication field does no longer make sense after https://github.com/gardener/gardener/pull/6987. In https://github.com/gardener/gardener/pull/6987 we removed support for Shoots with K8s < 1.20. The basic authentication was removed in K8s 1.19.x. Hence, it is not possible to enable basic auth anymore for a Shoot that is within the support K8s versions of Gardener currently.
This PR is a preparation for the removal of the enableBasicAuthentication field:

It makes the enableBasicAuthentication field no-op (removes validation, removes defaulting) - the field is not used anymore.
It sets the enableBasicAuthentication field always to nil. So far the field was always defaulted to false, if it was not specified in the field.
It overwrites the .spec.kubernetes.addons.kubernetesDashboard.authenticationMode to token, if it is set to basic.

Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/6911
Special notes for your reviewer:
N/A
Release note:

The `.spec.kubernetes.kubeAPIServer.enableBasicAuthentication` field is now no-op - the `gardener-apiserver` no longer defaults this field and no longer validates it. The field will be set always to `nil` on CREATE/UPDATE request.
End users specifying this field should no longer specify it. The field will be removed in a future version of Gardener.

/assign

/lgtm
/approve
/retest-required