SQLAlchemy order_by with ivfflat not returning any results

This is my adapted code based on the README:
from pgvector.sqlalchemy import Vector
from sqlalchemy import Index, create_engine, func, select, text
from sqlalchemy.orm import DeclarativeBase, Mapped, Session, mapped_column

from database import DATABASE_URI

# Define the models
class Base(DeclarativeBase):
    pass

class Item(Base):
    __tablename__ = "items"
    id: Mapped[int] = mapped_column(primary_key=True)
    embedding = mapped_column(Vector(3))

# Connect to the database
engine = create_engine(DATABASE_URI, echo=False)

# Create tables in database
Base.metadata.drop_all(engine)
Base.metadata.create_all(engine)

# Insert data and issue queries
with Session(engine) as session:
    session.execute(text("CREATE EXTENSION IF NOT EXISTS vector"))

    index = Index(
        "my_index",
        Item.embedding,
        postgresql_using="ivfflat",
        postgresql_with={"lists": 100},
        postgresql_ops={"embedding": "vector_l2_ops"},
    )

    index.create(engine)

    session.add_all(
        [
            Item(embedding=[1, 2, 3]),
            Item(embedding=[-1, 1, 3]),
            Item(embedding=[0, -1, -2]),
        ]
    )

    # Print all vectors
    all = session.scalars(select(Item))
    for item in all:
        print(item.embedding)

    # Find 2 closest vectors to [3, 1, 2]
    closest = session.scalars(select(Item).order_by(Item.embedding.l2_distance([3, 1, 2])).limit(2))
    print(list(closest))

When I use hnsw index, it works. However, with ivfflat, the closest variable is an empty list.

Hi @pamelafox, it looks like the index is being created without any data. See the docs on IVFFlat and troubleshooting.


Switch from JS custom variables to HTML meta tags

Append page metadata as govuk- namespaced meta tags within the <head> tag, as an alternative to setting Google custom variables within a script tag. This removes Google specific implementation details (page level scope, custom variable name, custom variable slot), the GOVUK.Analytics namespace and an inline <script> tag.
Javascript within static will handle the setting of custom dimensions and variables for analytics, based on these meta tags. A WIP example of that can be found here: https://github.com/alphagov/static/commit/3c87c32cf379f34862b9a19c1c429f2cc5780f7c
Example output
<meta name="govuk-format" content="news">
<meta name="govuk-analytics-organisations" content="<D19><D20>">
<meta name="govuk-analytics-world-locations" content="<WL3>">

Prior discussion: https://github.com/alphagov/slimmer/pull/117

:+1:

:+1: for the proposed HTML. Not reviewed the code though.

Only commenting on the meta tags, as i've not had time to read the bulk of the changes.
The approach is :+1: but i'd recommend using different character for namespacing, so it's clear what is a many-word thing, vs a namespace. eg:
<meta name="govuk-analytics-world-locations" content="<WL3>">

Would be clearer as one of:
<meta name="govuk:analytics:world-locations" content="<WL3>">
<meta name="govuk.analytics.world-locations" content="<WL3>">

I don't have a strong preference on . vs :, Dublin Core uses ., which is pretty wide spread, but Open Graph uses :, which is probably ore recognisable.
I know it's a little pedantic, but I think the readability is worth it :)

Read the rest, the rest generally looks good :+1:
It's nice to be rid of the #ga-param hack, but there are still a few other references:

Slimmer test template can just be updated, I think
Smart Answer compare task, this will need updating when, slimmer is upgraded on SA, to ignore the meta tags instead. Nothing needs doing now, other than giving smart answer folks a heads up (and maybe creating an issue to track it/add visibility?)

@dsingleton I agree about the namespacing, I've switched to using : in 430b7ff

@bradleywright @edds do either of you have any objections to the chosen namespacing?

I'm happy with @fofr's name-spacing changes, but would like someone else to agree before merging.

All good here.


Hi @ChrisWu1997, could you please update the PKU Disk link again? Thank you!

Hi @ChrisWu1997, could you please update the PKU Disk link again? Thank you!


Update main code to work with current PRAW/API

Ahoy! I saw recently on reddit (I am komodo99) that the bot was down-- well, I knew it was down, but I didn't know it was written in a language I knew. ANYWAY, a couple hours later, and here is a 'fixed up' version that works with the current PRAW/reddit API. (DISCLAIMER, it works for me, I cannot promise it is exactly correct. It seems so, but the API is new to me.)
From https://www.reddit.com/prefs/apps/ : You will need to export two additional environmental variables, as seen on line 179: the reddit appID (the shorter value under 'personal use script'), and the reddit script secret, the longer value marked 'secret', in addition to the user/pw already being exported.
The requirements.txt has been updated, so don't forget to (re)run pip to pull in the new dependencies. I tested on python 3.5.2 on xenial 16.04, so I imagine it ought to work in general on python 3.5. I Don't know anything about heroku, unfortunately.
Otherwise, it ought to just go, I think! Thanks much!

Wow. Thank you very much, I'll test this and upload it to Heroku ASAP.

No problem, happy I can help! If it isn't working for whatever reason, ping me back and I will figure out why.


error connecting to mysql in docker containers

I have an application that works normally when I use a desktop environment. However, when I put in a docker environment the following error is returned:
BrokenFilesystemWarning)
<IP_ADDRESS> - - [31/May/2019 12:43:18] "GET /data/ HTTP/1.1" 500 -
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 2309, in __call__
return self.wsgi_app(environ, start_response)
File "/usr/local/lib/python3.6/dist-packages/flask_socketio/__init__.py", line 43, in __call__
start_response)
File "/usr/local/lib/python3.6/dist-packages/engineio/middleware.py", line 67, in __call__
return self.wsgi_app(environ, start_response)
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 2295, in wsgi_app
response = self.handle_exception(e)
File "/usr/local/lib/python3.6/dist-packages/flask_cors/extension.py", line 161, in wrapped_function
return cors_after_request(app.make_response(f(*args, **kwargs)))
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 1741, in handle_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python3.6/dist-packages/flask/_compat.py", line 35, in reraise
raise value
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 2292, in wsgi_app
response = self.full_dispatch_request()
 File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 1815, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python3.6/dist-packages/flask_cors/extension.py", line 161, in wrapped_function
return cors_after_request(app.make_response(f(*args, **kwargs)))
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 1718, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python3.6/dist-packages/flask/_compat.py", line 35, in reraise
 raise value
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 1813, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.6/dist-packages/flask/app.py", line 1799, in dispatch_request
return self.view_functions[rule.endpoint](**req.view_args)
File "/usr/local/lib/python3.6/dist-packages/flask_cors/decorator.py", line 128, in wrapped_function
resp = make_response(f(*args, **kwargs))
File "/tmp/app/views/main.py", line 63, in data
bd_data = get_db_data()
File "/tmp/app/views/main.py", line 81, in get_db_data
conn = MySQLdb.connect(user="root", passwd="root", db="test")
File "/usr/local/lib/python3.6/dist-packages/MySQLdb/__init__.py", line 85, in Connect
return Connection(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/MySQLdb/connections.py", line 204, in __init__
super(Connection, self).__init__(*args, **kwargs2)
_mysql_exceptions.OperationalError: (2002, "Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2)")

Here is the file for docker compose:
version: '3'
services:
    db:
        image: mysql
        container_name: container-db
        environment:
            - MYSQL_ROOT_PASSWORD=root
            - MYSQL_DATABASE=test
        ports:
            - "3306:3306"
        volumes:
            - ./mysql:/var/lib/mysql
        networks:
            - app-tier
        restart: always
    service:
        image: service-image
        container_name: container-service
        ports:
            - "8080:8080"
        entrypoint: ["./wait-for-it.sh","db:3306","--timeout=0","--strict","--","python","run.py"]
        depends_on:
            - db
        networks:
            - app-tier
        restart: always
        environment:
            DB_HOST: db:3306
            YOLO_VERSION: v2
    app:
        image: view-image
        container_name: container-view-app
        ports:
            - "80:80"
        depends_on:
            - service
        networks:
            - app-tier
        restart: always
networks:
  app-tier:
    driver: bridge

and the variables that I pass to SQLAlchemy:
DB_USERNAME = 'root'
DB_PASSWORD = 'root'
DB_NAME = 'test'

DB_HOST = str(os.environ.get('DB_HOST'))
YOLO_VERSION = str(os.environ.get('YOLO_VERSION'))

SQLALCHEMY_DATABASE_URI = 'mysql://' + DB_USERNAME + ':' + DB_PASSWORD + '@'+DB_HOST+'/' + DB_NAME

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.

solution please :p


fixed end date issue and issues scheduling pipelines in Firefox

Closes: RHOAIENG-1197
Description

The issue at hand was that in Firefox, when trying to schedule a pipeline run, returned incoherent values for the end date. Upon closer investigation, I found that this was because of the usage of the Date() constructor, which is inconsistent at runtime if the parameter isn't an ISO 8601 formatted string. Here is the documentation if you are interested.
Unfortunately, it seemed that I could not schedule any pipelines in Firefox, due to this runtime issue, which is a larger issue. Therefore, I need guidance on whether or not the testing I have done is adequate, or if it needs more.
How Has This Been Tested?

Run npm run test.
Test Impact

I added a test for my newly created function in frontend/src/utilities/__tests__/time.spec.ts
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

Need guidance on whether more testing is necessary or what I have is adequate.

After setting a value for the end date, if I then toggle the switch off and then on again I get the following error:

^ Code is ready for review

/lgtm
/approve


need super-mixins flag in Fasta to alter warnings/errors around super calls

When compiling Flutter we get a slew of errors like this:
compiler message: file:///usr/local/google/home/vegorov/src/flutter-clean/flutter/packages/flutter/lib/src/rendering/binding.dart:295:11: Error: Superclass has no method named 'hitTest'.
compiler message:     super.hitTest(result, position); // ignore: abstract_super_member_reference
compiler message:

The error comes from this code:
/// An object that can hit-test pointers.
abstract class HitTestable { // ignore: one_member_abstracts
  // This class is intended to be used as an interface with the implements
  // keyword, and should not be extended directly.
  factory HitTestable._() => null;

  /// Check whether the given position hits this object.
  ///
  /// If this given position hits this object, consider adding a [HitTestEntry]
  /// to the given hit test result.
  void hitTest(HitTestResult result, Offset position);
}

/// The glue between the render tree and the Flutter engine.
abstract class RendererBinding extends BindingBase with ServicesBinding, SchedulerBinding, HitTestable {
  // This class is intended to be used as a mixin, and should not be
  // extended directly.
  factory RendererBinding._() => null;

  @override
  void hitTest(HitTestResult result, Offset position) {
    // ...
    // This super call is safe since it will be bound to a mixed-in declaration.
    super.hitTest(result, position); // ignore: abstract_super_member_reference
  }

}

as you can see the error is valid because Dart language conflates the notion of a class and a mixin together. You can also see that the error is suppressed in the analyzer.
To make Flutter builds error free we need a way to perform a similar suppression in CFE, while retaining correctness.
/cc @kmillikin @peter-ahe-google @leafpetersen @lrhn

This is already supported:

Don't use CompilerOptions.onError and CompilerOptions.reportMessages.
Instead use CompilerOptions.onProblem

Keeping this open to track the implementation


Why didn't I find the file "Run cat. exe"?

I looked for all the folders, but I couldn't find them.

现在我找到它了，但是它似乎没有反应


[Question]: Download remote settings for multiple functions in a workspace

Hi,
I had a question in regards to downloading remote settings for a function app.
My workspace contains multiple function apps.
I am able to deploy, test it, it all seems to work fine.
But when I try to download remote settings, it will prompt if local.settings.json doesn't exist for my first function app.
But if I try to select another function app and download it, I no longer have the prompt to select the target folder/app to download it to. It always overwrites the first one.
How do I force it to ask me every time what app to download the remote settings for?
Hope you can help, thanks!
Here is a screenshot of what I mean:

Hi @simplecommerce
Thanks for reporting this issue. This is a limitation of download remote settings right now. Rather than prompting the user for which workspace folder or using the default app setting, it is automatically selecting the first workspace.
I think the only workaround would be to open the workspace in its own VS Code editor. Thanks for bringing this to our attention; we'll fix this so that it prompts the user if there is more than one workspace.

@nturinski I think I spoke too soon, is it possible that it currently also doesn't support multiple local.settings.json for a workspace containing multiple functions?
See attached screenshots:
In this image, I have 3 functions and the local.settings.json is in the one I want to test.

When I select that function and try to run it, I get this error.

It works if I copy/paste the local.settings.json in the first one.
I assume this is the same issue but with the test/run feature.
If you confirm, should I create a separate issue for this?

I believe it's related. We currently only grab the local.settings.json from the first workspace folder. I think the solution would be the same. I think we can just fix it all in this one issue, but thanks for bringing it to our attention 😊

@simplecommerce Great find, working on a fix for this!
In response to your comment regarding the debugging scenario - does this workflow solve your issue?

On the left sidebar, select the Run and Debug tab

Use the header dropdown to change attachment

@simplecommerce Great find, working on a fix for this!
In response to your comment regarding the debugging scenario - does this workflow solve your issue?

On the left sidebar, select the Run and Debug tab

Use the header dropdown to change attachment

In my workspace I have 3 function apps, I am able to select the one I want to run and debug no issues.
The issue is only with the local.settings.json, it only seems to look for/pick the local.settings.json file of the first function app in the list.
If I delete the file and try to run any function app, it gives me that error. But if I download the settings remotely it will create the file in the very first function and then allow me to test any of the selected functions within my workspace with that file.
Sorry if I am not clear.

The issue is only with the local.settings.json, it only seems to look for/pick the local.settings.json file of the first function app in the list.

Oh I think I see what's happening, thank you for clarifying for me 😊

We've merged in a fix for this issue.  It should be available in the next release.  If you'd like to test it out before then, here is a link to download the vsix:  https://dev.azure.com/ms-azuretools/AzCode/_build/results?buildId=56394&view=artifacts&pathAsName=false&type=publishedArtifacts
Thank you for helping us improve our product!


#339: FileModels cannot handle path formats from different OS

Fixes #339

The 5 new code smells will not be fixed:
Three of them are in test code: will not fix this due to readability.
Two of them are in source code: will not fix this due to architecture.


[Storage] Improvements to Docs/Samples/Code Snippets

[ ] Explain the hierarchy of clients in the readme properly

[ ] We can add a brief intro to the clients(BlobServiceClient, ContainerClient, BlobClient, BlockBlobClient), the hierarchy, and types of blobs, service and what users can do with the service.

[ ] byPage() - Can add more description on how the maxPageSize, continuationToken are optional and how to use them - logged #6450
[ ] for await did not work - with the (default)babel project during the UX Study. - wont't fix. I believe this will gradually be picked up same as other new language constructs.

@jeremymeng suggested providing a link to the new syntax

[ ] Add more samples for general use cases - this is definitely must to have. Logged #6452

[ ] example downloading - download() vs downloadToFile()
[ ] Add examples and snippets for SAS credential generation

[x] Drive the samples and code snippets with the DefaultAzureCred rather than passing on the account name, key and sharedKeyCred - logged  #6447

[ ] or connection string?

[ ] Clients in samples have too many layers - create a blobServiceClient, blobServiceClient.getContainerClient() - logged #6448

[ ] Add samples for new ContainerClient()

[ ] Samples link is hidden in the initial set of 6 things(API Ref. Documentation, npm LINK, etc) - should be more discoverable - @HarshaNalluru has a PR to update guidelines in https://github.com/Azure/azure-sdk/pull/754. Once that's approved, we need to update all samples readme. logged #6449

Consider Examples to get started instead of Samples

[x] Code snippets in the documentation(readme) - snippets are now standalone and tested for Storage. @willmtemple should have issue tracking updating other libs.

[ ] Azure.SharedKeyCredential vs SharedKeyCredential from "@azure/storage-blob" in the snippets
[ ] some have let declarations and some don't

i is not defined in the examples(in JS Doc)

[ ] We can optimize the examples better so that the users can just Copy/paste the snippet for their use case

[ ] Add more description in the JS Docs explaining the cases where a function might fail/result in an error.  - logged #6451

[ ] Example - creating a container. Write about exceptions that the function might throw which can be caught by the users with try-catch
[ ] Descriptions for errors for each and every function - For example, uploading a blob to a non-existing container

[ ] private methods like listSegments() are showing up with the intellisense(VS Code) - nothing easy here to remove them from intellisense.  Maybe mention that they are internal/private in the JS Doc?

[ ] Segment methods should be private, Only happens in JS, not in TS, needs some investigation

[ ] library doesn't report error upon passing folder path to downloadToFile() - There's a bug tracking it already
[ ] Azure.github.io docs - can we have functions at the top and classes later on? - a common problem for api references doc. You have to list everything and it's hard for the doc system to know which ones are more important than others

Hard to find global functions like generateBlobSASQueryParams
https://azure.github.io/azure-sdk-for-js/storage-blob/globals.html

[ ] For a constructor with two or more overloads, users don't look at the other constructors and just end up using only the first one they see. - not something we own. There are issues logged against typedoc.

[ ] Azure.github.io docs - improve the way overloads are shown

Issue - Even though the first overload(selected by default) is in colour grey, users couldn't recognize that it's one of the overloads. Maybe, pick a different colour if possible

[ ] maybe mention "this is one of the two overloads of the constructor" in the JSDoc description?

[ ] getLengthOfBlob - contentLength attribute

[x] Description can be improved (Done)
[ ] Differences in description w.r.t the rest api docs.microsoft.com vs our JS DOC should be studied - could do as part of bug bash/doc bash

[ ] should call out the links to REST API - logged #6445

[ ] Mention that it is being used internally in the SDK

cc - @ramya-rao-a @joheredi @XiaoningLiu @jiacfan

Going to close this one.  I logged issues for actionable items.  Please review if there's any missing.


Go SplittableDoFn testing

This is a catch-all jira for all tasks needed to fully test SplittableDoFns in Go. For progress on SplittableDoFns themselves, see BEAM-3301
Imported from Jira BEAM-9882. Original Jira may contain additional context.
Reported by: danoliveira.

The following subtask(s) are associated with this issue: #20213


Create Gui for easy use

Create gui (based on tkinter?) to get easy access to all of the important parameters.
First steps should include:

Set interval for ping test
Set ping target
Define own path (with explorer) for storing the results
Start the network test with defined parameters.

the network_test_class.py offers a class which can be configured by using the new gui.

Hi,
I'll take this enhancement up.

I can also have this issue, if you wish.

@DrgnAI please collab. with @rishabhrkaushik to work together on that issue. If needed I can also create an own GUI-Branch. (Your work from #32 could be included)
If you need further details or Input - let me know!

@2Max4
I want to understand the purpose of the following condition in the code. The line is part of network_test_class.py at line number 182  currently.
Why is speed_test only performed when max_rtt > 10?
if test_result["max"].max() > 10:
    self.df_my_speed = self.df_my_speed.append(self.get_speed_results_as_df(), ignore_index=True, sort=False)

You've also referenced the condition in #7.
I've removed this condition in modular copy of my GUI pull request.
Also can you check my pull request and let me know your thoughts. I'll plan some actions based on those for tomorrow.

Hey @rishabhrkaushik ,
The main idea is that constantly running network speed tests on your network will slow your network down. This is why I only want to run a speed test if the max. Ping time of the ping test is greater then 10ms. Meaning: if you have a high latency the chance is high that also your network speed might suffer - so I do the Speedtest.
Because a ping is less traffic then the Speedtest I let the ping test run in constant intervals (which you can set via. --interval as an argument.
I will have a look at the gui and the class asap. Yet I have got quite few time - towards weekend I should finde a space to check it out!
Please give me feedback if my comment helped you!
Btw: it would be great to change the static 10 with a variable that could be loaded from the file where all default parameters are stored.

Hi @2Max4
Got it. I'll add the condition again and add 10 in defaults.json file. I have some further updates in mind, I'll start working on those till then.

Hi @2max4,
I've added the conditions to run speed test only when ping test latency is high back.
I've also added condition to autosave test data every n seconds. I've changed it's implementation though from initial implementation.
Latency and save interval I've added in config.ini.
Let's keep default values of test in seperate file itself. This file maybe frequently updated and merging with config.ini may result in config file getting corrupted by user.

Great idea! I will merge your branch asap.

Hi together,
huge thumbs up for the GUI in the current state I am really a fan of this now.
However for myself there is a problem while running the self(to_csv) in NetworkTest.py:

'NetworkTest' object has no attribute 'df_my_pingto_csv'
Traceback (most recent call last):
File "M:\Repos\Network_Analysis\GUI\modules\NetworkTest.py", line 222, in run_network_test
self.df_my_pingto_csv(self.ping_file_path)
AttributeError: 'NetworkTest' object has no attribute 'df_my_pingto_csv'

Hi @Skupino
There is a typing bug. Just change line 222 of NetworkTest.py. I've submitted pull request for  the same.
self.df_my_pingto_csv(self.ping_file_path)
to
self.df_my_ping.to_csv(self.ping_file_path)

Hi @Skupino
There is a typing bug. There is a missing . in the file. Just change line 222 of NetworkTest.py. I've submitted pull request for the same.
self.df_my_pingto_csv(self.ping_file_path)
to
self.df_my_ping.to_csv(self.ping_file_path).
Let me know if you face any other bug.

Thanks for the quick fix, I will continue testing and inform you if I find any further issues.
But anyway, nice work so far. I really like it.

Hi,
I found another issue, but it is not explicitly fault of the GUI, as long as you activate the Checkbox (Clear results) every saving attempt of the new .csv will automatically be cleared afterwards. This leads to missing data as every data moves in the archive.
We should add a one-time check for the box or a one-time execution for this function.

Hi @Skupino,
It is by design. This happens only at the start of a new test. The old files are not actually deleted but are moved to Archive folder with current timestamp. Also the default value of clear field is false. Hence I think the method works.

Hi @Skupino,
It is by design. This happens only at the start of a new test. The old files are not actually deleted but are moved to Archive folder with current timestamp. Also the default value of clear field is false. Hence I think the method works.

Hi @rishabhrkaushik
Yes that is correct but It will continue moving the files to archive for the new files created after the inital (old) already have been moved in the archive as long as the checkbox keeps being in the activated state. It is "clearing the old + the new results."

Hi @Skupino,
I reviewed the code. It is working for me. Every time you click "Start Test" button, it is regarded as new test and the older files will be moved to archive if clear result is enabled. If user has to append it to old file, they have to deselect the checkbox.
Adding a button doesn't feel like a good idea because someone might want to enable this clearing option by default so that every time they run test, new file is created. In button, this won't be possible.

Hi @rishabhrkaushik,
I am sorry if I was unclear in the problem description. It should work exactly the way you describe it, however it is not working like that in every aspect. What you described should be the case for every „new“ test via the „Start Test“-Button, but the („old“) files will be moved to the archive permanently, and with old files I mean the files before the test starts as well as the file the running test should save in the data directory.
So at the moment the clear test function kills(archives) all data, however it is originally designed to only move the existing/old data.
If the clear-checkbox is selected no data is saved in the data directory, everything gets moved in the archive.
I hope I could describe the problem properly now. If I am forgetting something or seeing something the wrong way, feel free to correct me. Greets Skupino

Hi @Skupino ,
I think I understood issue faced. Since the GUI and test is running on separate threads, the files will only be saved after current test have been completed. On my machine it takes around 30 seconds if the ping tests are failing.
The code seems to be working in my machine fine. If clear content box is selected before the start of the test, the old files are moved to archive folder and when you stop the test, new files are being created in Data folder with new test data.
Can you run the following test in your machine to help me dig deeper in the issue.

Clear the archive folder and data folder.
Run the test with clear checkbox unchecked for 10 min. Note the time of start and end of the test. Let's call them t0 and t1.
End the test and wait for sometime. This should create test result files in data folder. Check if they are created.
Compare the beginning and end of the test time and compare with t0 and t1. Let's see if they match.
Wait for 5 min. Check the clear test checkbox and run the test for 5 min. The test starts at t2 and ends at t3.
Stop the test and wait for sometime for results to be saved.

The following is the expected behavior after complete test.

Archive folder should contain test result with name test_type_t2.
The beginning of these files should be starting at t0 and ending at t1.
Data folder should contain test results with name test_type.
These files should begin at t2 and end at t3.

Let me know if you observe the same result. Let's compare only the ping_test for moment since speed test is executed only if ping test gives high latency.

Hey @rishabhrkaushik
Thanks for your work! Just for me getting it right: this means that you do not auto-save the files (while running the test) in the Data-Dir but only when the test is stopped.
My initial idea was to constantly "update" the files in the Data Dir (say every 10 minutes). This way also every 10 Minutes new graphs can be generated from those files which then (in a later state) could be provided via. a webinterface.
Since currently I got lots of work to be done for my degrees - @Skupino will be in lead for this issue!
Thank you both for the great work!
Greetings Max

Hi @2max4,
I've added the condition to save the files every n seconds (configurable) in the last commit.  In that case as well, the data should be updated in data directory. The files will be moved to archive only when starting new test.

@rishabhrkaushik
perfect!

Hi @rishabhrkaushik ,
for documentation puproses I added a testconcept-directory where I will put a testconcept for this case following your recommendations. I will run a test on the "old" status from yesterday for this too, that we have our changes documented.

Hi @rishabhrkaushik,
I have done the testing as you described it (also check the test-results in the testconcept directory) and I was not able to recreate the issues that I thought I have seen. Sorry for the inconvenience.
At least we are 100% sure now that everything works properly.
I also noticed one interesting fact: When network connection fails the current state gets safed anyway.
So again, sorry for confusing you :)

@Skupino No problem.


[BUG] Gizmos only rendering 3D Icons, not rendering Lines or other Gizmo Geometry

Anything rendered via lines like colliders, meshes via gizmos disappear as soon as Imgui is rendered.
Completely fresh project, using the shipped demo script.
Unity Version
2022+ (Tested in 2022.3 and 2023.1) URP 14 & URP 16

I updated to a new version of ImGuizmo. Would mind checking if the main is working?

Updating the package causes a editor crash on entering playmode.
Editor Log attached below.
EditorCrashImguizmo.log


FSI fixes and improvements

I'm sorry that we haven't split that changes to separate commits, but it was a bit difficult after we merge the master branch.
So, the major changes we did here are:

The main change we have done is splitting Arrow FSI to ArrowCsvForeignStorage, ArrowForeignStorage, the first one is for CREATE DATAFRAME query, and the second one is for our embedded engine, to allow users passing arrow tables directly to omnisci engine.
Refactor Arrow FSI a bit, so fragmenting logic and updating metadata is done only in one place now.
Refactor threading a bit, to get rid of task groups, so we have a parallel for a loop by columns.
Add initial implementation for converting arrow's dictionaries to omnisci dictionaries.

Formatting issues seem to be out of scope of that pr.


Removed uses of deprecated @warn_unused_result attr

The @warn_unused_result attribute is no longer needed as its behavior is now the default. A future change to the compiler (https://github.com/apple/swift/pull/2760) will make its use an error. This PR removes the uses of this attribute.

@phausler did this in a later commit (9cd298bd57a39129209183a92946ede85a4f1118)


Shared Component Properties in Experience Fragments

Required Information

[x] AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM SDK
[x] ACS AEM Commons Version: 5.1.12
[x] Reproducible on Latest? yes

Expected Behavior
When components with Shared and Global Properties are used in experience fragments, I should be able to author the shared and global properties in the experience fragments as well.
Actual Behavior
Shared Component Properties do not work in Experience Fragments.
Steps to Reproduce

Enable the Shared Component Properties OSGi service.
Specify a “page root path” pattern via OSGi config for com.adobe.acs.commons.wcm.impl.PageRootProviderConfig
Add SharedComponentPropertiesPageInfoProvider as an info provider for your xf page component
Add a component with either dialogshared or dialogglobal to an experience fragment.
You will notice the Shared Properties are not available to be edited.

Should this be closed since https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2715 got merged?

Yes.


Create page and component folder

Create folders to put pages in and components in for the frontend

Finished in #48


Help with kubectl deploy and add tutorial to support older devices

HI @fmdlc, I am currently trying to run this on my raspberry version 1, I only own this version.
The support of kubectl goes up to 1.5.7 (+1.6 drop support for older raspberries)
Here is where I found out: https://github.com/kubernetes/kubeadm/issues/253
But still I am facing some issues with the deployment:
Running: kubectl apply -f https://raw.githubusercontent.com/fmdlc/ISP-Checker/master/kubernetes/ISP-Checker-deploy.yaml

Response: The connection to the server localhost:8080 was refused - did you specify the right host or port?

If you want to add this to your tutorial:
Older raspberry devices may need version 1.5.7 of kubectl, since they dropped support 1.6 and above. If you try to install an unsupported message, you might receive this:
Illegal instruction
Remove kubectl and install kubectl 1.5.7 version:
sudo apt-get install  kubectl=1.5.7-00
Known devices that only support 1.5.7:

Pi Zero
Pi 1 version (both A and B)

UPDATE: Avoid using kubectl in older raspberry devices, I recommend to install only docker and docker compose and run make install
Still should be nice to have a tip on the tutorial telling users to avoid using kubectl on older raspberry devices.

UPDATE: Avoid using kubectl in older raspberry devices, I recommend to install only docker and docker compose and run make install
Still should be nice to have a tip on the tutorial telling users to avoid using kubectl on older raspberry devices.


Running Experiments on Custom Datasets

Hi,
Could you provide guidance on conducting experiments with custom datasets not included in TensorFlow's dataset collection? Your assistance is greatly appreciated.
Thanks,

Hi! Thank you for your interest.
Maybe this guide https://www.tensorflow.org/datasets/add_dataset?hl=en can be useful to you to convert your custom dataset to tensorflow format.
Then the integration in our repository should be easy.


Update testset_generation.md

Explicitly added distributions to dataset generation, otherwise it won't work

thank you so much for the fix ❤

also would love to hear your thoughts on the new testset generator too, what do you think? is it working for you as expected? what would you like to see here?


Build every platform into single tag

I created a simple action runner which builds everything and puts it into one single tag so people can release for all platforms instantly

Just wanted to let you know that I have a version of this functionality from another project I'm looking to backport in to this, probably next week. I'll try to pick the best of both.  :)

I just redid the way versions work to build on top of GitHub Action run ids instead of timestamps. I think this is closer to what everyone wants:

Same version number for Windows, macOS, and Linux builds
If one build fails all the builds fail
Version numbers are now clear simple incrementing from 1 values
Should fix the Windows timestamp-related issues

I used the matrix build option to allow for all three builds to run in parallel.
I very much appreciate the PR and it really did highlight that I needed to redo how the installer runner[s] should work. I had actually already started down this path implementing this matrix build version before you sent the PR but hadn't pushed it yet as I wanted to do some cleanup. Take a look and LMK what you think!


Unexpected type inference difference with dynamic and Object.

The following / attached bug1b6n.dart.txt program
produces this output (VM and dart2js):
CR<int>
CR<int>
Object
Object
Object
Object
[CS([]), CS([]), CS([]), CS([]), CS([]), CS([])]
[CR<int>, CR<int>, CR<int>, CR<int>, CR<int>, CR<int>]

The first six types printed are from the statement print(R);.
The type argument passed to AR.run.R is inferred differently depending on whether the result of run is passed to a castXX method with a dynamic or Object parameter type.
This seems like it must be a bug to treat two 'top' types differently.

enabling nnbd and using Object? has the same discrepancy
adding an explicit cast to the dynamic case (i.e. T cast1o<T>(dynamic any) => any as T;) makes no difference.

/cc @leafpetersen @johnniwinther
bug1b6m.dart:
import 'dart:async';

@pragma('dart2js:tryInline')
T cast1i<T>(dynamic any) => any;

@pragma('dart2js:noInline')
T cast1o<T>(dynamic any) => any;

@pragma('dart2js:as:trust')
@pragma('dart2js:tryInline')
T cast2i<T>(Object any) => any as T;

@pragma('dart2js:as:trust')
@pragma('dart2js:noInline')
T cast2o<T>(Object any) => any as T;

@pragma('dart2js:as:check')
@pragma('dart2js:tryInline')
T cast3i<T>(Object any) => any as T;

@pragma('dart2js:as:check')
@pragma('dart2js:noInline')
T cast3o<T>(Object any) => any as T;

class CR<T> {
  List<T> _a;
  CR(this._a);
  toString() => 'CS($_a)';
}

class AR {
  @pragma('dart2js:noInline')
  FutureOr<R> run<R>(FutureOr<R> Function() callback) {
    print(R);
    return callback();
  }

  CR<T> boot1i<T>(List<T> a) {
    return cast1i(run(() {
      return CR(a);
    }));
  }

  CR<T> boot1o<T>(List<T> a) {
    return cast1o(run(() {
      return CR(a);
    }));
  }

  CR<T> boot2i<T>(List<T> a) {
    return cast2i(run(() {
      return CR(a);
    }));
  }

  CR<T> boot2o<T>(List<T> a) {
    return cast2o(run(() {
      return CR(a);
    }));
  }

  CR<T> boot3i<T>(List<T> a) {
    return cast3i(run(() {
      return CR(a);
    }));
  }

  CR<T> boot3o<T>(List<T> a) {
    return cast3o(run(() {
      return CR(a);
    }));
  }
}

main() {
  var ar = AR();
  var c1i = ar.boot1i(<int>[]);
  var c1o = ar.boot1o(<int>[]);
  var c2i = ar.boot2i(<int>[]);
  var c2o = ar.boot2o(<int>[]);
  var c3i = ar.boot3i(<int>[]);
  var c3o = ar.boot3o(<int>[]);
  var L = [c1i, c1o, c2i, c2o, c3i, c3o];
  print(L);
  print(L.map((x) => x.runtimeType).toList());
}

Yes, this is expected.  Simpler example:
R run<R>(R Function() callback) {
  print(R);
  return callback();
}
main() {
  dynamic x = run(() => 3);
  Object y = run(() => 3);
}

Downwards inference in general takes preference over upwards inference because of unsound covariance.  (Arguably we should not do so for generic functions since they are soundly variant, and I've been considering changing it in a future language release, but currently it does.)  However, a downwards context of dynamic is not treated as a constraint, but rather as a lack of constraints.  So for the first line, the call to run is inferred with no downwards constraints, which means that upwards inference is used to choose R and int is the result.  For the second line, the call to run is inferred with a downwards constraint of Object, which constrains R to be Object, and upwards inference is not allowed to further refine the  result.

Thanks for the explanation.
To confirm we understand, a T cast<T>(...)... method that is intended to replace an as expression should be written with a dynamic parameter type to prevent the addition of a constraint that could modify the operation of type inference.

Yes using dynamic as the parameter type should do what you want.

Closing this issue as there seems to be no action item.


Show connected wallet info

Goal
When a user connects to the dApp, let's display 2 things:

Is the connected address a member of the batch? (is the in the allowList)
Is the connected address already checked in?

Feel free to place that info whenever you want. It could be just some icons that show next to the wallet for members.
For more info about the contract: #9

Selam Eda
I would like to work on this.
Can you assign to me?
Thanks

sure, thanks! @kocaemre

Have a nice weekend!
I think issue has been solved. @edakturk14


load/use my own components ?

Hello,
I'm tuning login page to be able to 2FA and more.
Can you give me an hand about how can I load/use my own components inside this nova .vue ?
Regards

Here is an complete example:
https://github.com/Muetze42/laravel-nova-example-projects/tree/feature/custom-login
I have also developed a custom F2A login for a customer.
For this I created a customized login mask based on the Nova login.


fix isDataEqual dates comparison

See https://github.com/tannerlinsley/react-query/discussions/433

Anything left here?

Oops, forgot to mark PR as ready :D

@tannerlinsley this is ready to merge


Fixes codewords in the roundend summary

Media

[ x I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

fix: Traitor codewords now get properly displayed in the round end summary

ok but i did it like that since its the same way tatoralert shows it on separate line


disable certificate validation

Could you please indicate whether or not there is a way to [disable certificate validation]?
Thanks in advance

Using the default FTPS code will validate any certificate given by the server.

Have you guys developed any property that switches it to disable? If not -
are you planning so, or I may contribute to this dev?
Tx,
DNS
On Jul 11, 2017 3:32 AM, "Harsh Gupta"<EMAIL_ADDRESS>wrote:

Using the default FTPS code
https://github.com/hgupta9/FluentFTP#faq_ftpswill validate any
certificate given by the server.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hgupta9/FluentFTP/issues/135#issuecomment-314359860,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABmuxogPcfP8--75teAlNliVv5XUhK_1ks5sMyUMgaJpZM4OTMCz
.

I don't know what you mean. Fork it and change whatever you want. Submit a PR, if it looks good, I will accept.


Normal map extraction error

I caught mismatch dimension error during normal map extraction
            normal = (normal - normal.min()) / ((normal.max() - normal.min()))  # normal = 1, 1, H, W, C
            normal_list.append(normal)
            ...
            divided = torch.max(divided, dim=3, keepdim=True)[0].repeat(1, 1, 1, 3)  # divided= 4, H, W, C
            divided_list.append(divided)

        normal_out = torch.cat(normal_list, dim=0)   # normal_out = 1, 1, H, W, C
        divided_out = torch.cat(divided_list, dim=0)

For able to save and preview results I suggest:
            normal = (normal - normal.min()) / ((normal.max() - normal.min()))
            normal_list.append(normal.squeeze())  # squeeze all extra dimensions
            ...
            divided = torch.max(divided, dim=3, keepdim=True)[0].repeat(1, 1, 1, 3)
            divided_list.append(divided)

        normal_out = torch.stack(normal_list, dim=0)  # replace cat to stack
        divided_out = torch.cat(divided_list, dim=0)

Workflow attached

You may have an old version of the ComfyUI-Video-Matting -nodes, there was a bug in the BRIAAI node giving wrong mask shape, which would also add extra dimension to the normal map.


include source and country dropdowns in proposal

fix for the mobile layout, it works now for categories stripping long strings, but we need it to work with two more dropdowns

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Version 7 issue with css (it seems)

Hi All, having an issue since upgrade from version 6.8.4 to version 7.x.x (having issues since 7)
Apologies, it is my first time posting an issue - hence I may have missed out on some important information - please shout and I will add what's required.
Problem overview:
Previously, the slider would appear as follows:

However after version 7 has been released, the same page started appearing as follows:

I've tried keeping version 6.8.4 on just the CSS files, and it seems like most of the design stays as it should be, the only issue is pagination is missing! Please see the screenshot:

I tried understanding whether any substantial CSS changes have been made from version 6.8.4 to version 7, but couldn't work it out.
I've decided to stay on version 6.8.4 for the time being, but wondering if anyone else had any issues with CSS since the update?

https://swiperjs.com/migration-guide#html-layout


Prettify error messages for exec.Commands in Fluid

Ⅰ. Describe what this PR does
Fluid relies on exec.Command to execute some operations and interacts with K8s API-Server(e.g. The helm for install/check/delete engine-related resources). exec.Command will always returns a meaningless err that would confuses users about the cause of their problems.
For example, when Fluid CSI Plugin fails to bind mount a FUSE mount point onto a pod, an event is issued on the pod showing messages like:
MountVolume.Setup failed for volume "default-jfsdemo": rpc error: code = InvalidArgument desc = exit status 1

This PR prettifies such error messages with a clearer explanation about the cause. For example, the event message mentioned above would be changed to:
MountVolume.Setup failed for volume "default-jfsdemo": rpc error: code = Internal desc = timeout waiting for FUSE mount point to be ready

which indicates that it's highly possible the FUSE pod is still not ready on that node
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/test fluid-e2e

/lgtm


Jesse: Watch<PHONE_NUMBER>(?#IS NorAm)

Jesse requests the watch of the watch_number<PHONE_NUMBER>(?#IS NorAm). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Rejected by Makyen in Charcoal HQ. Reason: 'Went with a blacklist for this number instead, due to being part of a spam wave.'


add "Format->Diagram->Animation" Checked  Button to hide or show animation

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
I use "Flow Animation" connection. This is a great feature but consumes large amount of CPU performance -> battery power. It leads to excessive heat and rapid discharge of my notebook.
Describe the solution you'd like
It would be great to be able to Hide or Show animation graphics when needed without changing actual property "Flow Animation".
Describe alternatives you've considered
Manualy turn off "Flow Animation" for all connection in the diagram. Which is VERY inconvenient.
Additional context

24.6.0 adds config option:

{
  "enableAnimations": false
}

thanks!

Result:

Still would be even better to have interactive button to switch between fast:


fix: do not save colors without a color scheme

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
The color mapping was saved every time whether or not a user chose a color scheme for their dashboard. We should not save a color mapping unless the user specifies a color scheme.
TEST PLAN

Tested manually
REVIEWERS
@betodealmeida @datability-io @xtinec

👍

Codecov Report

Merging #7347 into lyftga will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           lyftga    #7347   +/-   ##
=======================================
  Coverage   64.68%   64.68%
=======================================
  Files         426      426
  Lines       20897    20897
  Branches     2302     2304    +2
=======================================
  Hits        13517    13517
  Misses       7256     7256
  Partials      124      124

Impacted Files
Coverage Δ

...rset/assets/src/dashboard/components/SaveModal.jsx
43.9% <0%> (ø)
:arrow_up:

...uperset/assets/src/dashboard/components/Header.jsx
42.2% <0%> (ø)
: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 8fd7d4b...14e71a0. Read the comment docs.


feat: multiple content scripts with index files

Details
This PR implements #613, allowing for content scripts to be defined via index.ts files within the contents directory - for example:
src
└── contents
    ├── another-valid-script
    │   ├── utils.ts
    │   └── index.tsx
    └── valid-script.ts

See the original issue for more context / how this mirrors existing functionality.
Implementation
My current approach is fairly straightforward - in addition to shallowly scanning for files within contents, addContentScriptsNestedDirectories now handles also scanning shallow directories for any index-like files within them. From what I can tell, simply calling toggleContentScript on these nested files behaves as expected and registers the scripts 🎉
A few other things I tested (though please double check me!):

dev file watching works as expected, including for non-entrypoint modules like utils.ts in the above tree example)
there doesn't seem to be any collision issues between files (say, contents/foo.ts) and directories (contents/foo/index.ts) - these appear to be registered as independent scripts. I'd assume that's desired behavior, but let me know if we want some other conflict resolution strategy
Content Scripts UI files also work as expected
browser-specific entrypoints (i.e. contents/foo/index.chrome.ts) are handled with the same logic as shallow files

Opening this as a draft PR for now, since I'm not entirely settled on this implementation approach - just wanted to get something up to start iterating.
Naming
I've been referring to this case as "nested directories", though I think that's a bit misleading since it's really only one level deep. If there's a better name we should use here let me know and I can update code accordingly!
Example
I've been using a standalone example project for my local testing, though I'm happy to create a more formal example if that's easier for review / demonstrating! Let me know what works best.
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I agree to license this contribution under the MIT LICENSE
[x] I checked the current PR for duplication.

Contacts

(OPTIONAL) Discord ID: aeroencychris

If your PR is accepted, we will award you with the Contributor role on Discord server.
To join the server, visit: https://www.plasmo.com/s/d

I've been using a standalone example project for my local testing, though I'm happy to create a more formal example if that's easier for review / demonstrating! Let me know what works best.

We can prob add it into the /with-many-content-scripts example?

I've been referring to this case as "nested directories", though I think that's a bit misleading since it's really only one level deep. If there's a better name we should use here let me know and I can update code accordingly!

Maybe content script index file?

We can prob add it into the /with-many-content-scripts example?

Sure, want me to add that after merging in a second PR (since it looks like examples are a separate repo)?

Maybe content script index file?

Sure, renamed to addContentScriptsIndexFiles

Yeet for the example PR, we can merge it in later in a chore merge as well.
On Mon, Jul 3, 2023, 11:40 Chris Santamaria @.***>
wrote:

We can prob add it into the /with-many-content-scripts example?
Sure, want me to add that after merging in a second PR (since it looks
like examples are a separate repo)?
Maybe content script index file?
Sure, renamed to addContentScriptsIndexFiles
—
Reply to this email directly, view it on GitHub
https://github.com/PlasmoHQ/plasmo/pull/657#issuecomment-1618993032, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTJP5UPQPFVYFZXQ4BV6STXOMGY7ANCNFSM6AAAAAAZ27KDYA
.
You are receiving this because you commented.Message ID:
@.***>


[BEAM-1349] Merge python-sdk to master

R: @davorbonaci

Coverage decreased (-0.01%) to 69.62% when pulling be0e32e36313390ed04106d57f4c9dfeabb91b4d on python-sdk into 847e4e9f0b84efa4726692cc8b7c9a0610703888 on master.

Comments:

Files in runners/ are probably faulty merges. Investigate and probably revert?
Move sdks/python/.gitignore to top-level .gitignore?
Remove sdks/python/LICENSE?

Didn't look carefully inside sdks/python, but I suspect there should be some updates to Jenkins configuration and elsewhere too. I'd be fine moving forward and fixing it in a follow-up.


[Feature request] Include source in published package and set declarationMap to true

When command/CTRL-clicking something imported from SST in Visual Studio code, I always end up at the type definition file, which isn't very useful. I can right-click the editor tab and then select Reveal in Explorer View, which takes me to the compiled JavaScript, but I have to flick back and forth to see the types associated with the implementation. What I ideally want is to navigate to the source and, fortunately, TypeScript has a nice feature to enable it.
Simply add "declarationMap": true to tsconfig.json's compilerOptions and include SST's source in the published package. We'll now be able to navigate directly to the implementation source in Visual Studio Code. This will greatly improve the developer experience by easily being able to see what SST is doing under the hood.

Oh interesting. Let me share this internally.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


fix: RFC JSON formatting

A couple of RFCs' JSON examples don't display properly in deployed site, so I ran a formatter over it to resolve the issues (the ```json= is the primary problem). The issue can be observed here: https://hyperledger.github.io/aries-rfcs/latest/features/0685-pickup-v2/#reference
Also updates RFC 0496 to be in ADOPTED category in mkdocs/index (#834)

I manually verified that the DCO is correct, despite the hung check.  @ryjones — should we plan on going ahead with merges while the DCO check is stuck based on manual checking?  What is the impact if we mistakenly think the DCO is correct and it is not?

@swcurran you should add this file in all of your repos

@JamesKEbert could you push some other commit to trigger the running of the DCO check? It should go then...

Done :+1:


Monitor proposed changes to RPC layer

There is a proposal to replace gRPC with Connect: https://github.com/scionproto/scion/issues/4434.
So far, I haven't found a Rust library for Connect. There might be one by the time this change is performed, otherwise we would have to implement (part of) the protocol ourselves.

Tracking connect discussion about rust support.


Context - work with context state management

Content was added to deal with user authentication. Later on other context will be added based on components needs.


tests: Fix flaky testTwoSubStrOnSameColumn

Fixes the following failure:
io.crate.integrationtests.TransportSQLActionTest > testTwoSubStrOnSameColumn FAILED
    java.lang.AssertionError:
    Expected: is "e_s0"
        but: was "e"

Also removes the useless groupBySetup call.

@aelesbao please review

lgtm


updates for dashboard tests

How to verify that that the test works:

yarn up
yarn cy:debug
Choose test file(s) in the cypress test runner
Test(s) should pass

Extra check (runs headless):

yarn cy:e2e -s <test file(s)>
Result should be "✔  All specs passed!"

Make sure your .env contains:
REACT_APP_LOGIN_ROOT='https://login.linode.com'
REACT_APP_API_ROOT='https://api.linode.com/v4'
REACT_APP_APP_ROOT='http://localhost:3000'
REACT_APP_LAUNCH_DARKLY_ID='************************' (CMR flag)
REACT_APP_CLIENT_ID='********************' (https://cloud.linode.com/profile/clients, copy ID)
MANAGER_OAUTH='**************************************************************************'
(create in https://cloud.linode.com/profile/tokens > "Add a Person Access Token", Save the token that appears in the modal, you will only see it once)

I think we can go ahead and test and merge this for now and I will create separate PR's for the next changes.

@WilkinsKa1 yeah that's just a weird state that the app will get into at the very end of the last test since it's simulating an error.

Most are working consistently, there's one that will fail after you merge with develop (changes from demo prep) and one that's failing now for reasons I can't work out.

@Jskobos would you mind checking again? I've made some changes since this.


Document ingress controllers and DNS for Helm

Closed via https://github.com/xmtp/xmtpd-infrastructure/pull/15


wishlist from umdaschgroup

Description 📝

BIG IFC models to upload and view it as fast as possible
Import and Export IFC2x3 CV2.0 and IFC4.3 (*it would be nice to show the structural analytical view, including .ifczip format
What we really need is to display the geometrical and non-geometrical information accurately in web IFC viewer.

a.	Autodesk BIM360 or ACC viewer fails to open the geometry of IFC (see. Attached images- model geometry error), we would like to avoid this problem

b.	When the origin of model is far over 100 km away, we found that Autodesk BIM360 or ACC viewer fails to display the object accurate.. (see. Attached images- model geometry error),

c.	Orthographic & perspective view mode, as well as overriding shaded colour options
d.	IfcPresentationLayerAssignment information in the model tree
e.	3D viewCube and zoom options
f.	Selected item graphic display (Highlight selected item and have the option to show unselected items in transparent or wireframed view)
g.	Overlay PDF drawings

Create Parameter and PropertySet
a.	If we can create the addtional IfcParameter and PropertySet would be great. For example, we need Quanity parameters and values, however, we get sometimes without the Quantity parameter and values from the clients. We use simplebim or IfcOpenShell to add the Quantity parameters and values at the moment… It would be great if Tooeen can have the same features.
Federating the Ifc Models.
a.	We would like to keep the different IfcBuilding within one IfcSite when we federate the models (see. Attached example of simplebim and solibri/ IfcModelStructure)

b.	When we federate the models, we would like to have a good federated building storey structure
c.	Custom model tree – to show both federated Ifc Model structure and individual Ifc model structure
6)	Merging model
a.	We have to merge different Ifc files and export as a one model from time to time. At the moment, we use simplebim for it. Can we have this feature?
7)	CDE feature
a.	I like the idea to connect with SharePoint for example. Then we could really use Tooeen as CDE (Common Data Environment) to upload not only IFC models

Modifying Element Property
a.	Re-assign IfcBuildingStorey
b.	If we can automize the reassigning IfcBuildingStorey based on the element elevation would be great.
c.	Re-assign IfcClass. For example, we get many IfcBuildingElementProxy from our clients. It would be nice if we can re-assign the Ifc Entity and Type according to the buildingSMART classificaiton
d.	Modifying the existing parameter values

Clear Documentation to find out the information of the codes.

Suggested solution 💡
No response
Alternative ⛕
No response
Additional context ☝
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that requests the same feature to avoid creating a duplicate.

Working on it @ThatMetaverseArchitect! Do we have access to all these models?

Big IFCs can now be loaded fast with streaming.
All versions of IFC are now supported. For IFCzip, you can easily unzip it using a library like this and then opening it with our library.
Here are all the responses:

3.a. I think most (if not all) the geometry issues for your models have been already solved. Otherwise, you can open specific issues in the core library and we'll address the remaining ones.
3.b. We can also open all your models, even the ones that are far away from the origin.
3.c. We now support orthographic and perspective view, as well as a shaded option.
3.d. The model tree provided by the library can be customized, or you can build your own with the existing API.
3.e. We now have a view cube in the UI library.
3.f. The highlighter allows for customization, and the model graphics can be customized as you see fit.
3.g. To overlay PDF drawings you can just use specific libraries to load them in the Three.js scene. You can see this as an example.

We can already edit the existing properties and create new ones.
Here are all the responses:

5.a. Federation of multiple models is already supported.
5.b. and 5.c. The current API allows the structure can be customized to your needs.

Our current API already supports this!
This is also already supported.
IFC property edits are already supported by our API without limits, so all these points are possible with our libraries!
Done! You can check it here.


Moving jsbridge code to the sdk

Status
READY
JIRA Ticket
UMC-24
Description
I'm not ready to merge this in yet, I just wanted to get a diff out there for people to look at. Most of the translation is in SCJavascriptBridgeAPI.java. I tried to make it match the iOS counterpart as much as possible.
In the future, we should look at doing some general standardization and cleanup, for both maintainability and efficiency reasons. A code formatter would go a long way towards making the sdk easier to update. Using one mode of JSON de/serializing would speed up and improve code quality.
Todos

[ ] Tests

@boundlessgeo/spatial-connect

👍


Release plan for 0.10.0

We have just completed the release of 0.9.X, and in reflection of the time it took to release 0.9.0, I think it is worth discussing the release of 0.10.0 now.
Note that since 0.9, prqlc binary releases have made it easier to try out past versions.

Yes great! I think the main things we're focusing on:

We have some big talks coming up, I know folks have been spending a lot of time on these.
Some bugs, would really like to find a way of making these easier to fix. I'll try and spend some time on one of the priority ones.
Modules?
What else?

V good idea. I added a comment to that bug.
(I didn't want to gate the release on anything like that, because otherwise we can drag on releases. If we want to make that change, we can make a 0.10.1 release immediately...)


Support FontFace Parameter & APIs

Roblox have announced cloud-based fonts like Montserrat, Raleway, Work Sans, etc, and GuiObjects with the ability to render text no longer uses GuiObject.Font to change what font should the text be rendered in, instead it uses the new FontFace parameter.
The new fonts are only available by setting FontFace, this means that NumberSpinner will definitely not work with newer fonts. In this PR, I have updated the code to support FontFace, this also requires a change in how text size is calculated and retrieved in Digit.lua. It now uses GetTextBoundsAsync instead, an await method.
This meant the calculation of the text size will require to be called in a separate thread as Digit.lua uses a proxy metatable, async/await methods are not allowed in metamethods otherwise will cause a C stack overflow error.
Tested already, using Montserrat.
Issues related: #2

Thanks for the PR! This old repo needed a bunch of cleaning up, so I took care of that and included some of your changes as well. The new version is available on wally and supports fontface.


update ISSUE_TEMPLATE folder files

What is the change?

add bug_report.md to bug-report.yml to enable Github's form based issue template

https://youtu.be/qQE1BUkf2-s?t=23

add config.yml file to help direct users to the helpful pages
add feature_request.yml

I consolidated the improve-and-existing-component.md and propose-a-new-component.md into this file. I mentioned inside this file that you can suggest improvement to an existing component or propose a new one

Motivation

encourage's bug reporter's to put more care into their bug report before submission
this may help maintainer's receive more detailed & higher quality bug report's
adds helpful tips for user's during the process of creating a bug/issue report

Demo of Change

bug_report.yml file
feature-request.yml file
config.yml file

Types of changes

[X] Documentation Only

@jrgarciadev
Feel free to make edit to my Github branch, github now has Codespaces enabled for nearly all repos on Github.com 🎉
So you don't even need to run my code on your local computer since Codespaces allows you to edit in the browser now
My fork of nextui: https://github.com/cliffordfajardo/nextui

Hi @Dhananjay-JSR - thank your for the valuable PR feedback. I learned quite a bit of the new features thanks to your suggesstions.
Changes made:

added Code of Conduct
fixed typos
added field for selecting browser from dropdown list

Follow Up Question from your original pull request feedback
I'm a bit reluctant to make the hyper links shorter for the some of the links inside the description labels since Github renders those very small & its easy to miss those links

I believe one benefit of having the long URL (in places that is easier to miss) is that its much harder for people to miss the useful links in this context. Typically a user might be feeling rushed or frustrated when submitting a bug report or pull request and they might glance over the details.
The long explicit urls helps reduce friction for maintainers and make resources/tips for contributors a bit more easily accessible (hard to miss)
@jrgarciadev - any thoughts?
Not that this matters too much (view it more as a data point), other repos like solidJS and react-query also have the longer explicit URLs

react-query bug_report.yml file

solidjs bug_report.yml file

Hey, @cliffordfajardo this looks amazing, thank you so much, just one thing, could you please remove the line that says Code of conduct?

@jrgarciadev -
1.are you saying remove the code of conduct from the feature_request.yml and bug_report.yml?
2.can you point to the specific line or file?


Animate Same Scene Transition

Is there a way to animate same scene transition?
Let's say I have two scenes, Home and Product. Home being the initial scene.
When I call:
Actions.Product() <-- coming from Home, it animates the transition
Actions.Product() <-- coming from Product, it doesn't animate.
How can I animate that second transition?

Using:
react-native-router-flux v3.37.0

Actions.Product({animation : 'fade'})

@Dani93380 Thank you for your answer, but this works only coming from a different route. If I am already at a Product scene, no animation happens.


animations example: unknown type for floem::style::BorderRadius

I got a panic when trying to cargo run the animations example.
To reproduce:
git clone https://github.com/lapce/floem
cd floem/examples/animations
RUST_BACKTRACE=1 cargo run

$ RUST_BACKTRACE=1 cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.11s
     Running `/home/homestar/tmp/floem/target/debug/animations`
thread 'main' panicked at src/animate/prop.rs:148:17:
unknown type for floem::style::BorderRadius
stack backtrace:
   0: rust_begin_unwind
             at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/std/src/panicking.rs:645:5
   1: core::panicking::panic_fmt
             at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/core/src/panicking.rs:72:14
   2: floem::animate::prop::AnimatedProp::animate
             at /home/homestar/tmp/floem/src/animate/prop.rs:148:17
   3: floem::animate::animation::Animation::animate_prop
             at /home/homestar/tmp/floem/src/animate/animation.rs:339:13
   4: floem::view_data::ViewState::compute_style
             at /home/homestar/tmp/floem/src/view_data.rs:221:25
   5: floem::context::AppState::compute_style
             at /home/homestar/tmp/floem/src/context.rs:276:9
   6: floem::context::StyleCx::style_view
             at /home/homestar/tmp/floem/src/context.rs:1085:29
   7: floem::view::View::style::{{closure}}
             at /home/homestar/tmp/floem/src/view.rs:183:13
   8: <floem::views::stack::Stack as floem::view::View>::for_each_child_mut
             at /home/homestar/tmp/floem/src/views/stack.rs:105:16
   9: floem::view::View::style
             at /home/homestar/tmp/floem/src/view.rs:182:9
  10: <alloc::boxed::Box<dyn floem::view::View> as floem::view::View>::style
             at /home/homestar/tmp/floem/src/view.rs:680:9
  11: <alloc::boxed::Box<dyn floem::view::View> as floem::view::View>::style
             at /home/homestar/tmp/floem/src/view.rs:680:9
  12: floem::context::StyleCx::style_view
             at /home/homestar/tmp/floem/src/context.rs:1132:9
  13: floem::view::View::style::{{closure}}
             at /home/homestar/tmp/floem/src/view.rs:183:13
  14: <floem::views::container_box::ContainerBox as floem::view::View>::for_each_child_mut
             at /home/homestar/tmp/floem/src/views/container_box.rs:68:9
  15: floem::view::View::style
             at /home/homestar/tmp/floem/src/view.rs:182:9
  16: <alloc::boxed::Box<dyn floem::view::View> as floem::view::View>::style
             at /home/homestar/tmp/floem/src/view.rs:680:9
  17: floem::context::StyleCx::style_view
             at /home/homestar/tmp/floem/src/context.rs:1132:9
  18: floem::view::View::style::{{closure}}
             at /home/homestar/tmp/floem/src/view.rs:183:13
  19: <floem::views::stack::Stack as floem::view::View>::for_each_child_mut
             at /home/homestar/tmp/floem/src/views/stack.rs:105:16
  20: floem::view::View::style
             at /home/homestar/tmp/floem/src/view.rs:182:9
  21: <alloc::boxed::Box<dyn floem::view::View> as floem::view::View>::style
             at /home/homestar/tmp/floem/src/view.rs:680:9
  22: floem::context::StyleCx::style_view
             at /home/homestar/tmp/floem/src/context.rs:1132:9
  23: floem::view::View::style::{{closure}}
             at /home/homestar/tmp/floem/src/view.rs:183:13
  24: <floem::window_handle::WindowView as floem::view::View>::for_each_child_mut
             at /home/homestar/tmp/floem/src/window_handle.rs:1596:9
  25: floem::view::View::style
             at /home/homestar/tmp/floem/src/view.rs:182:9
  26: floem::context::StyleCx::style_view
             at /home/homestar/tmp/floem/src/context.rs:1132:9
  27: floem::window_handle::WindowHandle::style
             at /home/homestar/tmp/floem/src/window_handle.rs:503:9
  28: floem::window_handle::WindowHandle::process_update_no_paint
             at /home/homestar/tmp/floem/src/window_handle.rs:673:17
  29: floem::window_handle::WindowHandle::process_update
             at /home/homestar/tmp/floem/src/window_handle.rs:651:12
  30: floem::app_handle::ApplicationHandle::idle
             at /home/homestar/tmp/floem/src/app_handle.rs:330:13
  31: floem::app::Application::run
             at /home/homestar/tmp/floem/src/app.rs:132:9
  32: floem::app::launch
             at /home/homestar/tmp/floem/src/app.rs:29:5
  33: animations::main
             at ./src/main.rs:87:5
  34: core::ops::function::FnOnce::call_once
             at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Same problem on my Windows 11 box.


armv71 WARN

Installing Wire-Pod on Pi 3 B+ and I get the following when trying to run the setup  via sudo STT=vosk ./setup.sh
Debian-based Linux confirmed.
armv7l WARN: The Coqui and VOSK bindings are broken for this platform at the moment, so please choose Picovoice when the script asks.

I've separately installed VOSK using pip3 install vosk but it makes no difference, still the same error when I run sudo STT=vosk ./setup.sh
I'm happy to go with Picovoice if needed, but have no idea how...

Run it like sudo ./setup.sh, without the STT=vosk, then you will be given the option to choose Picovoice. Though, I recommend doing a reinstall of Raspbian with the 64-bit version rather than 32-bit instead.

Thank you, I will try that shortly when the current process has finished.
TBH I have never considered the 64 v 32, the Pi has been running for quite some time with various things I have on it, so ideally don't want to do a fresh install.

@kercre123 same issue with sudo ./setup.sh where no options are presented, just:
Debian-based Linux confirmed.
armv7l WARN: The Coqui and VOSK bindings are broken for this platform at the moment, so please choose Picovoice when the script asks.

Just pushed a fix, try git pull then try sudo ./setup.sh again.


data-hidefor not able to overwrite default "medium"

data-hidefor/data-hide-for of Responsive Toggle is not able to overwrite the default (medium):

Confirmed.


feat(conform-react,conform-zod): better type inference

Made some minor adjustment on the Submission type and improved type inference capability on the useForm hook.

Huge quality of life improvement!


Added Foreman as a dependency.

@FearMediocrity Most devs have foreman installed globally. Any reason we should add to the gemspec? for contributors?
I could see maybe adding this to what's generated. Maybe it's already there.

I added it because if you follow your instructions I had to manually. ( I was using a sandboxed app to try it out).
If a dev already uses it then it will be an already satisfied dependency, if not it will save a minute of head scratching for the beginner coming from React to Rails.

Thanks.


[sw] Private CI is broken, since merging PR #9316

I suspect it's just a matter of two conflicting PRs going in: CI was green for the PR itself.
@timothytrippel: Would you mind taking a look? If you don't get a chance, no problem: we can either do some debugging or back out the change tomorrow morning UK time.

Looks like I should have got to the bottom of my inbox before filing this! Possible back-out PR at #9381.

Yes, it was the case of some conflicting PRs going in. @sriyerg has found there was some X's coming out of SRAM and is working on a more permanent solution such that we can keep the optimization originally introduced in #9316, but I have reverted #9316 in #9381 to get things working again in the meantime. Sorry about that!

I believe we can close this as the root cause was addressed in #9382.


Need Help with Admin/Dev API Reference

Describe the issue
Hi, I'm currently looking for ways to automate the build & publish process for a Business Central extension and I'm having trouble finding the admin/dev API reference. What I have found online points me to the admin web API that I do not know how to access.
More context, we are using on-prem instances of Business Central which we access using our internal IP addresses. They are mostly for development, it looks something like this: <IP_ADDRESS>:8080/BCInstance1 to access the UI and we have a dev port for it as well which we use to download symbols/packages and publish the extension through the VS Code extension.
I managed to get around what the extension is doing up to publishing the extension by calling the dev(?) web API. To get the packages I perform get requests to <IP_ADDRESS>:12345/BCInstance1/dev/packages?publisher=Microsoft&appName=System&VersionText=<IP_ADDRESS>&tenant=default. It works, it gets me the packages then I check the manifest for dependencies and eventually get everything I need.
I managed to replicate what AppHandling/Publish-NavContainerApp.ps1 is doing and publish the app to <IP_ADDRESS>:12345/BCInstance1/dev/app using a post request with a multi-part payload.
The issue I keep running into is that there's hardly any documentation or reference for these endpoints and I was wondering if I'm simply missing it or there is a way I can get some documentation on the dev web API.
The reason I ended up calling the endpoints directly is because of authentication, we use user/password and windows authentication depending on the instance and I got the requests to work for user/password using basic authentication when calling the endpoint. For windows authentication or Entra ID is probably similar granted I get a token and set the authentication headers properly.
What I'm having trouble with now is getting the runtime version of an extension, unpublishing it and deleting it. I can't find and web API endpoints that I can call to do this, is it even possible?
We want to perform CI/CD in the end, to build and publish extensions to on-prem and cloud instances of business central from a pipeline.
My issues boil down to the following.

What web API can I use to download symbols, publish an extension/app, get the runtime package for it, unpublish and delete a package? Please provide examples
Are there PowerShell scripts or anything similar that I can use or look at to get this information from?

A lot of questions, which isn't really related to Containers and BcContainerHelper, but I can answer one
The dev endpoint is not documented. Why not use the PowerShell cmdlets to install your extension?
You might want to look at https://www.alops.be/ - I strongly recommend people to use a managed DevOps solution instead of building their own.

Thanks for answering this, I'm just a bit lost with getting this to work after searching and trying a lot of things.
I had a quick look at https://www.alops.be/, it might work if we switch to Azure DevOps.
You mentioned PowerShell cmdlets, can you please give me a link to them? I've tried a few and couldn't get them to work, while others come with the BC installation (I think).


Not all previously entered data is visible when viewing already saved forms

I filled in the Admission form and saved it successfully. I then viewed the admission form and saw that a number of fields were missing data:
Select fields:
Eye opening
Verbal response
Motor response
Airway
Input fields:
Glasgow coma scale
HCO3
Duration of antibiotics

Cool. I will check that out tonight. It might be related to the stuff Jake is looking at to validate the schema and form field match.
Sent from my iPhone

On Aug 7, 2016, at 11:19, Charlie Harris<EMAIL_ADDRESS>wrote:
I filled in the Admission form and saved it successfully. I then viewed the admission form and saw that a number of fields were missing data:
Select fields:
Eye opening
Verbal response
Motor response
Airway
Input fields:
Glasgow coma scale
HCO3
Duration of antibiotics
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm off out now @igiveadamn but might pop back online later on this evening

Thanks, man. I'll check it out a little later.
On Sun, Aug 7, 2016 at 12:21 PM, Charlie Harris<EMAIL_ADDRESS>wrote:

I'm off out now @igiveadamn https://github.com/igiveadamn but might pop
back online later on this evening
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/igiveadamn/ccdb/issues/24#issuecomment-238074623, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAT5YFCpYEDzIy-Z40Qm5jCRZBtnhbWvks5qdbGmgaJpZM4Jedg0
.


May be a bug about the mask type "c"

The mask type "c" corresponds to the situation that the current pixel can be predicted by all the surrounding ones within the kernel, as described in detailed in the paper.
/compressai/layers/layers.py;
class MultistageMaskedConv2d：
        elif mask_type == 'C':
            self.mask[:, :, 0::2, 1::2] = 1
            self.mask[:, :, 1::2, :] = 1

The kernel of mask_type C will be:
tensor([[0., 1., 0.],
          [1., 1., 1.],
          [0., 1., 0.]])

However, it should be:
        elif mask_type == 'C':
            self.mask[:, :, 0::2, 1::2] = 1
            self.mask[:, :, :, 0::2] = 1

which returns kernel:
tensor([[1., 1., 1.],
        [1., 0., 1.],
        [1., 1., 1.]])

Hi, thank you for your comment. It should be correct as you said when the kernel size was 3x3. However, in this revised version, we changed the kernel size of type "B" and "C" from 3x3 to 5x5 for better performance. Thus the code could be correct for 5x5. Unfortunately, the results did not show improvement by enlarging the kernel size. Maybe 3x3 is sufficient.

Oh, I just noticed that the kernel size has changed. Thanks for your quickly rely. Have a nice day!


More documentation around running Prometheus

Once you've instrumented your code with autometrics, you still need to configure and run Prometheus in order for it to scrape the metrics.
We should document how to do this + point it at your app.
Suggested by @axiomatic-aardvark

Added this to https://github.com/autometrics-dev#5-configuring-prometheus


Could you please fix Tran_Opamp.py example

I get this error message:
Traceback (most recent call last):
  File "/Users/rudi/Desktop/YalRF/yalrf/Tran_Opamp.py", line 19, in <module>
    y.run('AC1')
  File "/Users/rudi/Desktop/YalRF/yalrf/YalRF.py", line 152, in run
    sol = self.analyses[name].run(self, x0, nodeset)
  File "/Users/rudi/Desktop/YalRF/yalrf/Analyses/AC.py", line 75, in run
    dev.calc_oppoint(self.xdc, False)
TypeError: Opamp.calc_oppoint() takes 2 positional arguments but 3 were given

Thank you very much for taking a look.

See if it works now

It seems to work now, although I don't remember what the TB does exactly. There are many TBs that might be outdated as I've changed the interfaces many times. The HarmonicBalance class is not working anymore, only MultiToneHarmonicBalance, which can be used with a single input tone

Now all is perfectly fine. Thank you so much, Victor.

Am 02.09.2022 um 14:08 schrieb Victor Preuss @.***>:
It seems to work now, although I don't remember what the TB does exactly. There are many TBs that might be outdated as I've changed the interfaces many times. The HarmonicBalance class is not working anymore, only MultiToneHarmonicBalance, which can be used with a single input tone
—
Reply to this email directly, view it on GitHub https://github.com/victorpreuss/YalRF/issues/2#issuecomment-1235423550, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXB6CNKYRMAMFA74RXJ4TTDV4HU5XANCNFSM6AAAAAAQDAJKCA.
You are receiving this because you authored the thread.

no worries, please let me know if you find any more issues (there are probably a good amount)


SNS topic can't be created for bucket with period in name

I have asset buckets related to static sites, with the domain name in the bucket name; e.g., www.example.com-uploads.
Domovai fails when creating an SNS topic with the bucket name in the SNS topic name, as SNS topic are not allows to have periods in the name; e.g., trying to create an SNS topic with name domovoi-s3-bucket-events-www.example.com-upload fails, Topic name contains invalid characters. Must contain only alphanumeric characters, hyphens (-), or underscores (_).
For example:
@app.s3_event_handler(bucket='www.example.com-upload', events=['s3:ObjectCreated:*'])
def resize_image(event, context):
     return True

andy@shoggoth:dosizer$ domovoi deploy
Regen deployment package.
:
Found existing permission grant statement domovoi-17ccb624, skipping
Subscribing {'prefix': None, 'suffix': None, 'events': ['s3:ObjectCreated:*'], 'func': <function resize_image at 0x7efc4c523aa0>, 'use_sns': True} to events in S3 bucket www.example.com-upload
Traceback (most recent call last):
  File "/home/andy/.local/bin/domovoi", line 168, in <module>
    sns_topic = sns.create_topic(Name="domovoi-s3-bucket-events-{}".format(s3_bucket))
  File "/home/andy/.local/lib/python2.7/site-packages/boto3/resources/factory.py", line 520, in do_action
    response = action(self, *args, **kwargs)
  File "/home/andy/.local/lib/python2.7/site-packages/boto3/resources/action.py", line 83, in __call__
    response = getattr(parent.meta.client, operation_name)(**params)
  File "/home/andy/.local/lib/python2.7/site-packages/botocore/client.py", line 317, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/home/andy/.local/lib/python2.7/site-packages/botocore/client.py", line 615, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.InvalidParameterException: An error occurred (InvalidParameter) when calling the CreateTopic operation: Invalid parameter: Topic Name

Thanks for reporting! Will fix this shortly. Just need to pick a way to
escape the periods.
On Fri, Feb 2, 2018 at 13:25 Andy Cowell<EMAIL_ADDRESS>wrote:

I have asset buckets related to static sites, with the domain name in the
bucket name; e.g., www.example.com-uploads.
Domovai fails when creating an SNS topic with the bucket name in the SNS
topic name, as SNS topic are not allows to have periods in the name; e.g.,
trying to create an SNS topic with name
domovoi-s3-bucket-events-www.example.com-upload fails, Topic name
contains invalid characters. Must contain only alphanumeric characters,
hyphens (-), or underscores (_).
For example:
@app.s3_event_handler(bucket='www.example.com-upload', events=['s3:ObjectCreated:*'])
def resize_image(event, context):
return True
andy@shoggoth:dosizer$ domovoi deploy
Regen deployment package.
:
Found existing permission grant statement domovoi-17ccb624, skipping
Subscribing {'prefix': None, 'suffix': None, 'events': ['s3:ObjectCreated:*'], 'func': <function resize_image at 0x7efc4c523aa0>, 'use_sns': True} to events in S3 bucket www.example.com-upload
Traceback (most recent call last):
File "/home/andy/.local/bin/domovoi", line 168, in
sns_topic = sns.create_topic(Name="domovoi-s3-bucket-events-{}".format(s3_bucket))
File "/home/andy/.local/lib/python2.7/site-packages/boto3/resources/factory.py", line 520, in do_action
response = action(self, *args, **kwargs)
File "/home/andy/.local/lib/python2.7/site-packages/boto3/resources/action.py", line 83, in call
response = getattr(parent.meta.client, operation_name)(**params)
File "/home/andy/.local/lib/python2.7/site-packages/botocore/client.py", line 317, in _api_call
return self._make_api_call(operation_name, kwargs)
File "/home/andy/.local/lib/python2.7/site-packages/botocore/client.py", line 615, in _make_api_call
raise error_class(parsed_response, operation_name)
botocore.errorfactory.InvalidParameterException: An error occurred (InvalidParameter) when calling the CreateTopic operation: Invalid parameter: Topic Name
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kislyuk/domovoi/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0nPVleVA6EAL2ww3HHTdnufDvR9NdLks5tQ30xgaJpZM4R3zGe
.

S3 bucket names cannot contain underscores, while SNS topic names cannot contain periods. So I'll use that replacement as a reversible name transform.

Released in v1.5.2.


Curl PHP

Hello, with the following code I call the API v4 to print the values of a spreadsheet. It is going very well.
$curl = curl_init();
curl_setopt($curl, CURLOPT_URL,"https://sheets.googleapis.com/v4/spreadsheets/
spreadsheetID/values/PHP?valueRenderOption=UNFORMATTED_VALUE&key=yourKeyId");
$result = curl_exec ($curl);
curl_close ($curl);
What I can't achieve is to do an insert or update on it, I can't manage to put the PHP curl together for this. Some help? Please try as below but it doesn't work.
$curl = curl_init();
curl_setopt($curl, CURLOPT_URL,"https://sheets.googleapis.com/v4/spreadsheets/
spreadsheetID/append/PHP?InsertDataOption=INSERT_ROWS&key=yourKeyId");
$jvalues=array(array("5869","35685987","Ema","2020-05-29","no brinda","3563430423"));
$jsonData = array(
'values' => $jvalues
);
$jsonDataEncoded = json_encode($jsonData);
curl_setopt($ch, CURLOPT_POST, 1);
curl_setopt($ch, CURLOPT_POSTFIELDS, $jsonDataEncoded);
curl_setopt($ch, CURLOPT_HTTPHEADER, array('Content-Type: application/json'));
$result = curl_exec ($curl);
curl_close ($curl);

Some help, I found no examples anywhere.
Thanks.

Hi @JavierVassallo,
This forum is for issues related to the Google API Client library. We can't really help debug issues unrelated to that. You could try Stackoverflow or Reddit though.


MINOR: update new version in additional places

Note: This goes only to trunk. 0.10.0 branch will need a separate PR with different versions.

@ewencp @junrao when you have a sec :)

LGTM

LGTM


Create CPP_Factorial_of_large_number.cpp

Issue --> https://github.com/HacktoberfestSIT/Problem-Solving/issues/54
MY solution is 100%correct  written in CPP  ✔✔✔✔✔
This code is able to calculate the factorial of more than 100 correctly
I checked it
Sir/Madam , Please check it .....
@OctoplusNinja

Sir /Madam , Please review it.
@OctoplusNinja

hey @iamprofessor1 , can you add the files in proper folders, eg: array/Factorial_of_large_number/CPP/{your code}

Okay Sir /Madam @OctoplusNinja
Please Check / review it

@iamprofessor1 , you can delete the previous created file which is directly in the array folder.

Also, my pronouns are he/him/his. :)

@OctoplusNinja  Deleted

Great, Merged 🚀!


symlinks under src/ not followed/included for BOSH packaging

Just ran into a situation where files pointed at by a symlink under the src/ dir were not added to the compilation VM, even though we specified the files in the package's spec file.
For example:
$ pwd

/Users/pivotal/workspace/garden-runc-release # bosh release directory

$ cat packages/runc/spec

---
name: runc

dependencies:
  - libseccomp
  - golang_1.6.1

files:
  - github.com/opencontainers/runc/**/* # include everything under src/github.com/opencontainers/runc/

$ ls -lah src/github.com/opencontainers/runc/Godeps/_workspace/src/github.com/opencontainers/

total 8
drwxr-xr-x   4 pivotal  staff   136B May 11 16:52 .
drwxr-xr-x  13 pivotal  staff   442B Jul  4 09:55 ..
lrwxr-xr-x   1 pivotal  staff    79B May 11 16:52 runc -> /Users/pivotal/workspace/garden-runc-release/src/github.com/opencontainers/runc
drwxr-xr-x   4 pivotal  staff   136B Apr 22 12:05 runtime-spec

The symlinked files under src/github.com/opencontainers/runc/Godeps/_workspace/src/github.com/opencontainers/runc do not make it to the compilation VM.
Is this expected behaviour or is this a bug?
Cheers,
Ed

@teddyking this is expected -- for packaging symlink files themselves.

Closing - sounds expected, but symlinking methods have changed slightly in the new CLI. Feel free to give that a try and start providing feedback in that repo. Ruby CLI code is frozen.


Bug: Ninja CleanDead tool remove non-dead dynamic outputs

When running ninja -t cleandead, ninja will remove file which are dynamic outputs of the build (files registered in dyndep files as implicit outputs), but those dynamic outputs should not be considered dead. For instance in my example below, input1.o and input2.o are removed by ninja -t cleandead.
I have looked at CleanDead implementation and I have noticed it was never calling LoadDyndeps() during its initialization.

Below the experience I made the reproduce the issue:
rule my_tool_dyndep
    command = python my_tool.py --dyndep $out --output $output $in
rule my_tool
    command = python my_tool.py --output $out $in

build output.txt.dyndep : my_tool_dyndep list.txt
    output = output.txt
build output.txt : my_tool list.txt || output.txt.dyndep
    dyndep = build/test2/outputB.txt.dd

my_tool.py read list.txt, which contains a list of file to concatenate. The concatenation written in output.txt. To to experiment dynamic outputs, it also output one .o for every .txt contain in list.txt (in my example list.txt contains input1.txt input2.txt).  When run with --dyndep the script only generate the dyndep corresponding to the expected action.
output.txt.dyndep
ninja_dyndep_version = 1
build output.txt | input1.o input2.o: dyndep | input1.txt input2.txt

Related: https://gitlab.kitware.com/cmake/cmake/-/issues/21406

I failed to flag this with #2406, but I'm pretty sure that this should be closed as fixed.


It occurs "java.lang.Exception: Exception in :insert-token" error when I request token using MySQL database.

It occurs java.lang.Exception: Exception in :insert-token.......Caused by: java.sql.BatchUpdateException: Incorrect string value: '\xAC\xED\x00\x05sr...' for column 'scope' at row 1 error when I request token using MySQL database.
Scope is a set, maybe a set can't be stored in MySQL database.
I have a solution, I will make a pull request to you for this issue.

yup, confirmed. thanks for a fix!

having a bit of free time, I sorted out again the problem with scopes. I wanted to ensure that stores do not need to convert scopes to string and then back to set (on database read). instead, they should always work on plain string representation and the conversion should happen outside. otherwise each store (sql, redis or some custom defined) would need to deal with the same problem separately.

I recommend that scopes be stored in MySQL database as a set if there is no bug. Up to now, I find two bugs about that. The first is Incorrect string value: '\xAC\xED\x00\x05sr...' for column 'scope' at row which I commented above. Another is 'scope' is too long to insert.
Let's fix the bugs and store scopes as set.

Well done!


Get back IP address of Client

Question
From Pode Server, there is any way to get back the IPaddress of the client which request API ?

Hey @ClissonFlorian,
There is a way to get it, using the event object Pode supplies you can get the IPAddress from the Request property:
$e.Request.RemoteEndPoint.Address.IPAddressToString

Ie, in a Route:
Add-PodeRoute -Method Get -Path '/ip' -ScriptBlock {
    param($e)

    # this will be the ip of the client
    $e.Request.RemoteEndPoint.Address.IPAddressToString
}

You can do the same in middleware as well 😃

It's perfect :), thank you very much.

You're very welcome! 😄

Hi @Badgerati
Sorry for reopening this. Inspired by the above, I'm trying this and getting InputObject is null
    Add-PodeRoute -Method Get -Path '/ip' -ScriptBlock {
        param($e)

        # this will be the ip of the client
        $e.Request.RemoteEndPoint.Address.IPAddressToString
    }

The above shows nothing.
$e | get-member doesnt show anything
and
    Add-PodeRoute -Method Get -Path '/ip' -ScriptBlock {
        param($e)

        if ($null -eq $e)
        {
            Write-Host "NULL"
        }
        else
        {
            Write-Host "Not NULL"
        }
    }

Also outputs null. Any helps appreciated :)

Hey @TechnicallyJoe,
Aaah, are you using Pode 2.0? There were changes in there that removed the $e.
If you are, then this should do it:
Add-PodeRoute -Method Get -Path '/ip' -ScriptBlock {
    # this will be the ip of the client
    $WebEvent.Request.RemoteEndPoint.Address.IPAddressToString
}

If you don't mean to use Pode 2.0, you can force the module version to 1.X via Import-Module Pode -MaximumVersion 1.99.99 -Force in your main server script :)

Thanks @Badgerati
$WebEvent solved my issue. Thanks for the great work with Pode! 👍


Update KIS to 1.0.2 - CKAN file supplied.

Here you go.
no clue if it's possible, else it might be wise to use "http://kerbal.curseforge.com/ksp-mods/228561-kerbal-inventory-system-kis/files/latest" for the auto-update checker.

Line 6 (download) is missing a closing quotation mark (") before the comma.

Wow, that was sloppy, thanks! Just fixed it.


Data caching

Cache data so that it's not constantly getting/setting from/to the database

Added as of https://github.com/srnyx/annoying-api/commit/f68b2372ef68f5e93d2f28df777384c429695662


Add a regression test for memrchr errors with clang

See https://github.com/msys2/MINGW-packages/pull/13125

It's very unlikely to regress the same way again but other regression on other symbols is more likely 😞

It's very unlikely to regress the same way again but other regression on other symbols is more likely 😞

Yeah... just trying to make a habit of adding tests..


[api-docs] Remove dm_permission param from Edit Guild Application Command endpoint (#4861)

A new commit was made into the api-docs repo: https://github.com/discord/discord-api-docs/commit/ceeb0faf8c25aeb26f69c6edbcac95417d9e99b2

Remove duplicate phrase

Remove dm_permission parameter from editing guild commands

Fix table formatting

Thanks @advaith1
Co-authored-by: advaith<EMAIL_ADDRESS>Co-authored-by: advaith<EMAIL_ADDRESS>This is a bot created issue.

Can we close this? Discord just fix typo and remove dm_permission from edit guild application command
Right now we have createApplicationCommand and upsertApplicationCommand with an optional guildId which determine whether it's a guild or global slash command. I think the core lib shouldnt validate that if it's a guild command but there's dm_permission (only global slash command can have dm_permission, guild slash command cannot have dm_permission) then throw new Error , maybe it should be in validation plugin