Problems used in the project

If I use it in my project, I find it impossible to customize the log output method, and I think newdownloadclient can be used as a public method to provider to external programs

@flymzero Thanks for creating a new issues!
However, I don't assume that this project is used as library from external project.
Sorry.


Modify sourcemap paths of imported files

Modify sourcemap paths of imported files relative to file.base instead of relative to file, if a different base is specified. Resolves #319

Possibly resolves #300 also.

Used path.sep to reference the platform specific separator.

What's the status of this? I came across this exact issue in our project, and after many hours I came up with a fix that looks a lot like this one. I can confirm that it will fix the issue in our project as well.
I think a more elegant solution is to use Array#map to fix the sources list, as it requires fewer variables. Something like this:
        sassFileSrc = file.relative;
        sassMap.sources = sassMap.sources.map(function(f){
          return f == sassMapFile
            // Replace the stdin with the original file name
            ? sassFileSrc
            // Get the path relative to base, not the original file path
            : path.join(path.dirname(sassFileSrc), f);
        });

The PR works. Cloud you please accept this PR and release the new version?
Thanks
Domenico

lgtm, @Snugug ?

Me to.

Great! Any chance you can push this to npm as v2.0.5?

:+1:


URL previews cluttering message view

For each link that I send via Zulip, a link preview is generated and attached to the message.
I did not find a way to disable this preview on a per message basis.
This becomes a problem when sending multiple links in one message, because suddenly a huge area is taken up only by link previews.
If I send another message afterwards it gets completely buried in the link previews.
It gets even worse when trying to reply to a message with a couple of links, because those previews get also attached to the quoted message.
Possible solutions imho:

disable previews for quotes (replies) - not fixing the whole issue though
allow to disable url previews on a per message basis (not ideal)
check if there is more than one link in a message and if that is the case, automatically disable link previews (would be my favorite)

as an example:

when quoting that message:

@lukasIO thanks for the report!  Issues like this are why the INLINE_URL_EMBED_PREVIEWS feature is default-off in Zulip.  When we do invest in making that something that we think is ready to have default-on, we'll look at solving this.

Makes sense, thanks!

@timabbott would it be possible to have there simple × icon next to the preview to remove it?
This way you can just discard those previews you don't want. Slack has it like that.

It would be fairly simple to add such an x that dismisses the preview for the current user for the duration of the current browser session, which may be good enough for v1.  Longer term I can imagine making it discard the preview for all users or something, but that's significantly more complicated (since we need to store it that state in the database; we could probably implement it as a message edit, indirectly).
@punchagan FYI, this is probably worth fixing while you're working on this.

Yep, I plan to fix this, once we wrap up the backend clean ups! :)

As I appreciate your effort, it doesn't seem to me this half solution is any good. If it is just for me, it still clutters the view for everyone else and I don't think others will want to close all previews again every time they start new session.
If you would be storing previews as attachments, easiest would be to just delete this attachment and be done with that, but I don't see that deep.

Thanks for your comments @vrozkovec. @timabbott is suggesting we do the simpler UI solution not to solve the problem that you mention, but as a first step to figure out what the visual UI and interaction should be like.
I think we definitely want to go the full distance and remove the previews for URLs, and also may be prompt the user if we want to add the domain to a blacklist where previews should never be shown. For instance, chat.zulip.org gets a lot of links of the Zulip documentation or GitHub issues. It may be worth turning off previews for those domains on czo.

@punchagan I see, thanks for explanation, makes sense. Blacklist is an interesting idea. Perhaps blacklist could be even user scoped? But that is maybe overkill and this setting should be left to organization admins only.
We would use it for docs.google.com domain for example, as we share a lot of links to those documents and previews clutter the discussion. Looking forward for the implementation!

@punchagan, @timabbott also I've noticed, that once preview of the link is generated, the header looks like a link - has the same color - but you cannot click on it. I think it would be good, if those headers were clickable, happened to me many times that I wanted to click the preview but had to search the message for the link.

@vrozkovec that would definitely be worth fixing! Thanks for the suggestion.

@punchagan please have a look at the screenshot below. What is also annoying is generating previews when someone quotes your text. Sometimes I have to search for what the other person actually said.
And also as you can see, previews just clutter the stream.

Thanks for looking into it!

Probably been said here, but I like how Discord allows you to put a link but not generate a preview if you put < and > around it, such as <https://www.google.com>.

Disabling the preview with <...> is a nice feature, but what syntax to use for [...](...) and [...][x] ... x: ... links? Maybe [...](<...>) and [...][x] ... x: <...>? Though it's debatable whether it's a good idea to introduce a custom Markdown syntax extension for this...
Also, this approach doesn't have great discoverability (especially since it's not standard Markdown), but that may be fine, anyone who's bothered enough by the link previews can probably be expected to google it :)

If it's any help, I'd prefer the option to control each individual preview, similar to how Slack does it:

Zulip could look like this, with similar button "X" to delete that specific preview on hover:

It would be fairly simple to add such an x that dismisses the preview for the current user for the duration of the current browser session, which is probably good enough for figuring out what the visual UI should be. Longer term I think we probably want to make it discard the preview for all users, but that's significantly more complicated (since we need to store it that state in the database; we could probably implement it as a message edit, indirectly).
@punchagan FYI, this is probably worth fixing while you're working on this.

Yeah +1 for removing the preview for a particular link for all users. Don't see much point in removing it for a single person (other than that it's easy to implement).
Another quite important feature (the lack thereof which leads me to now have to disable the link previews altogether) is to be able to disable previews for a specific domain. One particularly annoying one is linking to a Gitlab instance, where the link preview is the "Sign in" one. Completely useless noise, and these links are shared often in my organization.
UI-wise in Slack, if you're admin, when you dismiss a particular preview you also get an option to disable the previews altogether for that domain.
These two features (disable single preview for all users, disable all previews for particular domain) are quite important for making link previews usable IMO.

Question: would it be possible to have a per-user option to disable all previews? Ie, treat this as Personal Settings > Display Settings rather than as an org-wide setting?

I like that suggestion! Now that we give admins control over default user settings, it seems like making this a user setting would be more flexible, while still allowing admins to influence the feel of the organization. @timabbott what are your thoughts?


inputClass is not being set for password fields

While most of the input types are making use of the editableThemes.inputClass correctly, for password fields, I think this theme is not getting applied.

This was fixed by #425 and is in the latest release.


[Bug]: SMTP: Not getting complete error message if the host address is left blank and the datasource is tested

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Complete error message is not shown if the user leave the host address filed as blank.

Steps To Reproduce

Create an SMTP datasource
enter into all the fields except the host adress
click on test
observe - The error message which is coming from the backend is not fully shown

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
1.9.12

Hi Anand, this does not seem to be a bug, in case of multiple errors, only the first one would be shown in the toast IMO.

Hey @sumitsum
With the error-handling phase 1 multiple toast messages are shown on the datasource config page.
@ChandanBalajiBP Can you please confirm whether this applies to all the datasources?

Yes, this applies to all datasources.

But this issue was fixed with this PR


.fetch_one fails with JSONDecodeError("Expecting value", s, err.value) Python 3.5

raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
group = grp.fetch_one({'name': 'XXXXXXX'})
print(group)

shadycuz,
Did you fix above error?

No but I belive my problem was not having the correct service now API roles.

Or some other credential error.

I dont see as credential issue, as conn is successful.Might be API roles,not sure.
Will see, if anyone else confirms same
inc = ServiceNow.Incident(conn)
#inc = inc.fetch_one({'number': 'INC4578541'})
print (inc)
<servicenow.ServiceNow.Incident object at 0x000001E401FEA8D0>
inc = inc.fetch_one({'number': 'INC4578541'})
Only fetch_one issuing JSONDecodeError
C:\Program Files\Anaconda3\lib\json\decoder.py in raw_decode(self, s, idx)
355             obj, end = self.scan_once(s, idx)
356         except StopIteration as err:
--> 357             raise JSONDecodeError("Expecting value", s, err.value) from None
358         return obj, end
JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Which version of python are you using?
[]'s

I am getting the same error. Did anyone get a solution? i

It is not likely due to a connection as I'm getting the same error, and I'm opening it from my hard drive.  My file is JSON formatted, so while it fails to load, a similar json file in the same location loads fine.  I keep culling the file down to see if something in the file is causing a formatting error that makes it think there is a missing value, but no luck yet.  When I recently did a straight json.load with the other file that does load, I got a warning: "InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised."  Since my hard drive is shared with a OneDrive, I'm wondering if it's related to Office 365 and OneDrive since it has an external connection maybe a security patch just added to Office 365?

For me the error was that I'm using a Kingston instance, and Dublin+ instances should use the JSONv2 api as stated in the readme file.


refactor(dynamicCustomIdFinder): simplify and use regex

Changes

Updates the code to use regex instead of string searching and replacing
Simplifies the code by first looking for a match and after that extracting the values

Important

There is no prettier config present so my vscode config took over the formatting, has to be fixed but config is needed
(Minor issue): The lockfile is out-of-date, your pnpm lockfile version: 5.4 => updated version: 6.1

Thanks for the contribution and help. This is a much better solution.

There is no prettier config present so my vscode config took over the formatting, has to be fixed but config is needed

I have formatted the code to use my prettier settings and will create an official config soon.


Add "ParameterURL" metadata field

One aspect of the "model fit parameters" vs. "model-derived parameters" distinction is that sometimes an individual can propose taking the outcomes of some model fitting process and compute some novel derivative measure from those data, and the relevant reference for that derived parameter may be different to that of the model itself. Eg. The original 1994 DTI paper doesn't present FA.
With #92, metadata relating to "the model" is encapsulated within its own metadata dictionary, and there a URL can be provided for the model. With this PR I propose that in some instances it may be desired to provide a URL that describes just the specific parameter that is encoded in the corresponding data file.

Oh - interesting! Yes, I think that the 1995 is the correct reference here, and also yes: I think that paper has a math error in it (I see no other explanation). Funny.


[house rule] track strain prior to engagement and post it

I run a house rule where you can't spend advantage to gain strain above what you entered combat with. It'd be great if we posted the starting amount when combat began.


Add friend priority and add the ability to set friend type after initializing Friend

I added friend priority which can be used as a way for other mods (if they'd like) to add a priority to a friend this way when selecting which friend should come first in some sort of operation it can be more logical. This can work along with FriendType where regardless of priority BEST_FRIEND always has a higher priority than FRIEND, and the opposite with ENEMY and NEUTRAL. I also added the ability to change the friend type of a player because that's pretty necessary. Priority is completely optional and doesn't have to be used by a mod unless it takes advantage of it

Doesn't this kind of defeat the point of FriendType?

Not really, did you read my pull req description?

Not really, did you read my pull req description?

I did but I cant think of a situation where this would be useful over FriendType.

Not really, did you read my pull req description?

I did but I cant think of a situation where this would be useful over FriendType.

I'm sure it could be used, but it's completely optional either way.

It's not useful being combined, should kinda be either priority or friend type. Maybe change so priority 2 = BESTFRIEND and -1 = ENEMY and everything between

It's not useful being combined, should kinda be either priority or friend type. Maybe change so priority 2 = BESTFRIEND and -1 = ENEMY and everything between

You mean instead of an enum a int as the identifier?

Yeah, you could just have a priority int assigned with the enum value and just use priority as someEnumValue.getPriority(). If you dont understand im happy to show you what i mean in practice.

I mean the reason I made priority is so a user can create a unique priority for each of their friends at any number they want, providing a large amount of flexibility.

Yeah, you could just have a priority int assigned with the enum value and just use priority as someEnumValue.getPriority(). If you dont understand im happy to show you what i mean in practice.

No I know what you mean.

Yeah have an int value assigned to the enum and you can always put priority over 2 if need a "super special" friend

I mean the reason I made priority is so a user can create a unique priority for each of their friends at any number they want, providing a large amount of flexibility.

But what i fail to see is the reason for this flexibility. To me it seems useless to have together with FriendType.

I mean the reason I made priority is so a user can create a unique priority for each of their friends at any number they want, providing a large amount of flexibility.

But what i fail to see is the reason for this flexibility. To me it seems useless to have together with FriendType.

Alright I'll just add an int value to the enums instead.

I actually like the separate priority because it allows for easy reordering. Without it, -1, 0, 1, and 2 are not very expressive in terms of priority and using a float or double to express priority doesn't exactly sound feasible.

I actually like the separate priority because it allows for easy reordering. Without it, -1, 0, 1, and 2 are not very expressive in terms of priority and using a float or double to express priority doesn't exactly sound feasible.

I have similar thoughts here, but it does make storing those enum values a bit simpler.

So for example you can have 3 friends with the same priority and 1 enemy in your crystalling range, the CA should target the spot where your friends take less damage than your enemy.
If one friend has higher priority than another and yet another is a BEST_FRIEND, then the CA should target a spot that prioritizes your BEST_FRIEND's health, over your high priority friend's health, and over your lowest priority friend's health.

I'd say merge this and if there ends up being demand for something like what I added we can just add it.

I'll wait for toast.

😭


Code from the example does not work

I use the last DuckDB.NET.Bindings.Full version 0.8.1 and code from the example does not work because class DuckDBResult has only private properties

Thanks for spotting the issue. The internal API has changed since I put those examples. Do you want to send a PR?

Take a look https://github.com/Giorgi/DuckDB.NET/pull/122

Thanks!


[Bug] Drag unselected active tool

Prerequisites

[x] Which version are you using? (Is it latest?)
[x] Are you reporting to the correct repository?
[x] Did you search existing issues? (Were any related?)

Description
Method _dragHandler should not be executed on an unselected active tool when dragging another selected tool.
Steps to Reproduce the issue
Reproduce steps for OHIF viewer which uses csTools:

When I open the study: https://viewer.ohif.org/viewer/<IP_ADDRESS>.4.1.25403.345050719074.3824.20170125113417.1
Then I add Length tool measurement
Then I add Angle tool measurement (Angle tool is still active)
Then I hover under the Length tool measurement and drag it.
I see the error in console: "Uncaught TypeError: data.handles.middle is undefined"

Expected behavior: (What you expected to happen)
_dragHandler method is not executed on an unselected active tool. No exceptions are raised
Actual behavior: (What actually happened)
Exception is raised:
Uncaught TypeError: data.handles.middle is undefined
updateCachedStats https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:35460
_dragHandler https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:22401
triggerEvent https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:51153
onMouseMove https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:19981
mouseDown https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:20034
enable https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:20098
default https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:26101
dispatchEvent https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:53906
triggerEvent https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:58964
default https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:53481
_callee$ https://viewer.ohif.org/vendors~OHIFCornerstoneViewport.bundle.545b6e0833d68ab30b81.js:9129
tryCatch https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:179670
invoke https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:179896
method https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:179722
asyncGeneratorStep https://viewer.ohif.org/vendors~OHIFCornerstoneViewport.bundle.545b6e0833d68ab30b81.js:936
_next https://viewer.ohif.org/vendors~OHIFCornerstoneViewport.bundle.545b6e0833d68ab30b81.js:958
_asyncToGenerator https://viewer.ohif.org/vendors~OHIFCornerstoneViewport.bundle.545b6e0833d68ab30b81.js:965
_asyncToGenerator https://viewer.ohif.org/vendors~OHIFCornerstoneViewport.bundle.545b6e0833d68ab30b81.js:954
componentDidMount https://viewer.ohif.org/vendors~OHIFCornerstoneViewport.bundle.545b6e0833d68ab30b81.js:9187
ik https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189199
unstable_runWithPriority https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189275
fg https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189068
Yj https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189192
Lj https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189171
ig https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189069
unstable_runWithPriority https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189275
fg https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189068
ig https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189069
jg https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189068
Rg https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189162
enqueueSetState https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:189081
setState https://viewer.ohif.org/app.bundle.255df793a2cc1b5f17cd.js:188491
setStateFromProps https://viewer.ohif.org/OHIFCornerstoneViewport.bundle.70a492acfbde443762d7.js:394

@take5v Where you able to solve this?

@take5v Were you able to find a solution to this problem?

@ricardonava Not yet.

@take5v Oh well, I'm actively looking for a solution to this, let you know if I find something.


Jenkins Plugin Hanging

Trying to use the jenkins plugin to get deployment data.  I built from 2.0.5, but had to update the pom to use the 2.0.5 version of the Hygieia-core dependency.  After successfully building and installing/configuring in jenkins.  When I run a pipeline, it gets to the Publishing Deployment stage, and just hangs there forever.  When I run the test connection step, it seems to hit my API fine, but something in the plugin itself keeps getting stuck.
Here is my groovy script for the plugin. hygieiaDeployPublishStep applicationName: 'Hygieia', artifactDirectory: '/api/target', artifactGroup: 'com.capitalone.dashboard', artifactName: '*.jar', artifactVersion: '2.0.5', buildStatus: 'Success', environmentName: 'dev'
One thing to note is that using the groovy script generator, I was not able to select a build status, so I added that manually.  I also put some debug log statements in the plugin to see if I could track down exactly where it was hanging, and it looks like it is getting stuck on line 197 in the HygieiaDeployPublishStep.java class.  I can't get it to log anything inside the BuildBuilder constructor it is calling, or after this line.  Any help would be appreciated, I will continue looking at, but don't have much experience with jenkins plugins.

@rob-miller-777  - check this https://github.com/capitalone/Hygieia/issues/1195

line# 197 is: BuildBuilder buildBuilder = new BuildBuilder(run, hygieiaDesc.getHygieiaJenkinsName(), listener, BuildStatus.Success, true);
The next line is the call to publish: HygieiaResponse buildResponse = hygieiaService.publishBuildData(buildBuilder.getBuildData());
@rob-miller-777 do you see anything in the api log?

No, it never makes it that far.  Threw a logging message after that line, and got nothing.  Put a logging message inside that constructor and saw nothing.  That includes the jenkins logs and api logs.  I am currently under the assumption that our jenkins server isn't configured properly, so trying to run a jenkins server locally to test that out.

Yeah, I tried the same plugin on a locally running jenkins server, and it worked fine.  So I need to figure out whats wrong with the jenkins server.


Global credentials file for multiple pipelines

Description
I'm always frustrated when credentials for a pipeline changes, or I need to help a teammate with a specific pipeline but I don't have the right credentials handy, or the credentials rotate without me knowing, and then we spend minutes/hours getting it sorted out before I can even jump in to help...
Context
Our team builds many kedro pipelines that get deployed via Docker, but one issue we have is that each pipeline has its own credentials file... so each user maintains a project specific credentials.yml in a local directory ignored by gitignore, then when we build in AZDO we have to upload the right file and give permissions to the build pipeline for that file - when credentials rotate we then have to remember which pipelines used the now-expired credentials. For our group, it would be awesome to have a central place to store credentials where users can clone a repo securely, decrypt, and have all pipelines on their machines reference the one-safe-source credentials, and then our builds can all reference the same credentials file so updates roll out automagically.
Possible Implementation
I think it'd be great if a kedro project could just read GLOBAL_KEDRO_CREDENTIALS or something from an env variable, or be configured in a way to go outside the project context to get a file with the right credentials/keys in it.
Possible Alternatives
I'm a noob in the space in the first place so I don't have an alternative idea here

Hi @nicpayne713 this is absolutely something we want to improve out of the box. You can read our (now very long) issue #891 regarding how we want to improve the configuration experience more holistically.
If you want to introduce environment variables to your credentials workflow today, this is how you can do it yourself by editing hooks.py.

I'm pretty sure this should work as you describe.


Disable TestDateDiff on apache_spark

Error from TestDateDiff on the apache_spark cluster:
ERROR    configured_file:functions.py:232 15:49:39.369957 [error] [MainThread]:     org.apache.hive.service.cli.HiveSQLException: Error running query: org.apache.spark.sql.AnalysisException: org.apache.hadoop.hive.ql.metadata.HiveException: MetaException(message:Insert of object "org.apache.hadoop.hive.metastore.model.MTable@445045d2" using statement "INSERT INTO TBLS (TBL_ID,CREATE_TIME,DB_ID,LAST_ACCESS_TIME,OWNER,RETENTION,IS_REWRITE_ENABLED,SD_ID,TBL_NAME,TBL_TYPE,VIEW_EXPANDED_TEXT,VIEW_ORIGINAL_TEXT) VALUES (?,?,?,?,?,?,?,?,?,?,?,?)" failed : A truncation error was encountered trying to shrink LONG VARCHAR 'with data as (
ERROR    configured_std_out:functions.py:232 15:49:39      org.apache.hive.service.cli.HiveSQLException: Error running query: org.apache.spark.sql.AnalysisException: org.apache.hadoop.hive.ql.metadata.HiveException: MetaException(message:Insert of object "org.apache.hadoop.hive.metastore.model.MTable@445045d2" using statement "INSERT INTO TBLS (TBL_ID,CREATE_TIME,DB_ID,LAST_ACCESS_TIME,OWNER,RETENTION,IS_REWRITE_ENABLED,SD_ID,TBL_NAME,TBL_TYPE,VIEW_EXPANDED_TEXT,VIEW_ORIGINAL_TEXT) VALUES (?,?,?,?,?,?,?,?,?,?,?,?)" failed : A truncation error was encountered trying to shrink LONG VARCHAR 'with data as (

Basically, spark__datediff produces a TON of SQL, and TestDateDiff calls spark__datediff several times — so, several tons of SQL. That's too much for our measly little dockerized Spark clusters. A TODO is to look into minifying the amount of SQL produced by datediff.

Not sure why Snyk is failing :shrug:

Thanks for opening #379! I left a comment to the effect of: we are testing spark__datediff — just against Databricks, not our dockerized Apache Spark, which must have some different configs.

I forgot how you already told me that this is tested -- thank you for the reminder!


Format "None"

I have been trying to do the tableschema example (for writing to the database), but I consistently get the following error on table.save('articles', backend='sql', engine=db):
(lastchance) -bash-4.2$python create_lastchance_table.py
Traceback (most recent call last):
File "create_lastchance_table.py", line 14, in
table.save('articles', backend='sql', engine=db)
File "/scratch/miniconda3/envs/lastchance/lib/python3.7/site-packages/tableschema/table.py", line 174, in save
stream.save(target, **options)
File "/scratch/miniconda3/envs/lastchance/lib/python3.7/site-packages/tabulator/stream.py", line 417, in save
raise exceptions.FormatError(message)
tabulator.exceptions.FormatError: Format "None" is not supported
Writing to a file works fine (without errors), but we are really trying to use Frictionless Data as a standard for loading datafiles into databases.
Thanks!

@alw4ey
Hi, is it possible (if it's public) to share some data and code you use so I will be able to try reproducing the problem?

but we are really trying to use Frictionless Data as a standard for loading datafiles into databases.
:+1: cc @lwinfree

Sure, see attached; but this is just a reduced copy of the example.
Also, here is my pip requirements.txt listing, in case it offers some insight into the problem:
attrs==19.1.0
bitarray==0.9.3
cchardet==2.1.4
certifi==2019.6.16
chardet==3.0.4
Click==7.0
et-xmlfile==1.0.1
idna==2.8
ijson==2.4
isodate==0.6.0
jdcal==1.4.1
jsonlines==1.2.0
jsonschema==3.0.1
linear-tsv==1.1.0
openpyxl==2.4.11
psycopg2==2.8.3
pybloom-live==3.0.0
pyrsistent==0.15.3
python-dateutil==2.8.0
requests==2.22.0
rfc3986==1.3.2
six==1.12.0
SQLAlchemy==1.3.5
tableschema==1.6.0
tableschema-sql==0.11.0
tabulator==1.23.0
unicodecsv==0.14.1
urllib3==1.25.3
xlrd==1.2.0
create_lastchance_table.py.txt

@alw4ey
Sorry for the slow response. The example wasn't up-to-date. Could you please try:
# pip install sqlalchemy tableschema-sql
import sqlalchemy as sa
from tableschema import Table

# Create SQL database
db = sa.create_engine('sqlite://')

# Data from WEB, schema from MEMORY
SOURCE = 'https://raw.githubusercontent.com/frictionlessdata/tableschema-py/master/data/data_infer.csv'
SCHEMA = {'fields': [{'name': 'id', 'type': 'integer'}, {'name': 'age', 'type': 'integer'}, {'name': 'name', 'type': 'string'}] }

# Open from WEB save to SQL database
table = Table(SOURCE, schema=SCHEMA)
table.save('articles', storage='sql', engine=db)

# Open from SQL save to DRIVE
table = Table('articles', storage='sql', engine=db)
table.infer()
table.schema.save('tmp/articles.json')
table.save('tmp/articles.csv')

# Open from DRIVE print to CONSOLE
table = Table('tmp/articles.csv', schema='tmp/articles.json')
print(table.read(keyed=True))
# Will print
# [{'id': 1, 'age': 39, 'name': 'Paul'}, {'id': 2, 'age': 23, 'name': 'Jimmy'}, {'id': 3, 'age': 36, 'name': 'Jane'}, {'id': 4, 'age': 28, 'name': 'Judy'}]

Please let me know if the problem still exists

Nope, this took care of it, thanks!


add oid failded

[DEBUG   ]      agentx.agent - register       : Adding <IP_ADDRESS>.<IP_ADDRESS>.1 to list
[DEBUG   ]      agentx.agent - register       : Adding <IP_ADDRESS>.<IP_ADDRESS>.1.1 to list

This issue does not make clear what it is asking or suggesting. Please update it with a complete description of what you're trying to do, and attach a PR with code changes, if any, or close it.


Can I get RGB-D data with this repo?

Thank you very much for your hard work.
I want to get RGB-D data using HoloLens 2.
In the official repo, separately stored data is processed using Python and then data is created.
But I want to get RGB-D data from the device and store it frame by frame.
The most similar thing I found is
https://github.com/cyberj0g/HoloLensForCV
https://github.com/doughtmw/HoloLensForCV-Unity

What I need is to make the depth data displayed as a curved surface into a flat image, and find a region within it that matches the color image.

Mapping between RGB image and depth image is not implemented in this repo. The second repo you mentioned (https://github.com/doughtmw/HoloLensForCV-Unity) have the implementation you need on HoloLens 1. Probably you could refer to that and try to do that yourself :)


[WFCORE-4417] Upgrade WildFly Elytron to 1.9.0.CR3

https://issues.jboss.org/browse/WFCORE-4417
This pull request does introduce one new module, however this is just a new sub module within the WildFly Elytron project and not a new dependency.
Release Notes - WildFly Elytron - Version 1.9.0.CR3
** Bug
* [ELY-1761] - Unicode normalization in SASL client breaks some non-ASCII passwords
** Task
* [ELY-1764] - Get all Provider implementations moved to their own module.
* [ELY-1779] - Make 'org.wildfly.security.password.util' public API
* [ELY-1782] - Update the branch for new features and enhancements in the "Contributions" section in the README.md
** Release
* [ELY-1785] - Release WildFly Elytron 1.9.0.CR3

Core - Full Integration Build 8514 outcome was UNKNOWN using a merge of c22ca9bde644b5ed3dccc0e0e3169a77830f43b9
Summary: Canceled (Tests passed: 237, ignored: 18; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:04:41

Dependency Tree Analyzer Output:
New Dependencies:

org.wildfly.security:wildfly-elytron-provider-util:jar:1.9.0.CR3:compile

Thanks @bstansberry I have added a commit adding the license information.

Dependency Tree Analyzer Output:
New Dependencies:

org.wildfly.security:wildfly-elytron-provider-util:jar:1.9.0.CR3:compile


[Wallet] Fix SendAmount Activity Indicator bug

Description
App would crash because the Activity Indicator is not loadable inside of a Text element on Android. Using the built in showLoading property instead.
Other changes
N/A
Tested
On Android and iOS device
Related issues
https://sentry.io/organizations/celo/issues/1884452452/?environment=co.clabs.valora&project=1250733&referrer=alert_email
Backwards compatibility
Yes

Fixes #5010


Add DisableScientific

Issue: (part of) https://github.com/gavv/httpexpect/issues/190
Tasks:

[X] Update formatXXX to be methods of DefaultFormatter
[ ] Implement DisableScientific in DefaultFormatter

Coverage: 95.43%. Remained the same when pulling<PHONE_NUMBER>e6d65a48aa601efa7bf2ec6427e581 on neverbeenthisweeb:190-disable-scientific-fmt into fea393ee2257bb6e31c6f9c96839e5408201d426 on gavv:master.

I think it makes sense to split refactoring (making format funcs to be methods) and adding new flag into two prs. It would be easie to review them separately. If you don't mind this approach, let me know when the first part is ready for review.

I think it makes sense to split refactoring (making format funcs to be methods) and adding new flag into two prs. It would be easie to review them separately. If you don't mind this approach, let me know when the first part is ready for review.

Let's rename formatTemplate to something without format, to make it distinct from formatXxx methods. E.g. applyTemplate.

Let's rename formatTemplate to something without format, to make it distinct from formatXxx methods. E.g. applyTemplate.

got it, will do the change

@gavv ready for review

Thanks!


Create API metrics Dashboard

This issue is an outcome from SIT-2 review. We didn't have metrics Dashboard to demonstrate data transfer (SIT-2 document and section 5.3.3). Should we create API metrics Dashboard through CDK?

Maybe a SPIKE to discuss how we want to monitor the APIs?

This is what was required from SIT-2
Navigate to CloudWatch and click on "metrics", showing the API requests and data transfer amounts

After doing some research, this dashboard is already exist and provided by AWS. We don't need to create it.

Link to SIT-2


[Pickers] Migrate PickersMonth to emotion

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

It works on my local but argos fail, 🤔


error during video compression

I'm getting the following error when I use Tierpsy Tracker on the test data set:
Capture_Ch1_18062015_140908 Starting video compression.
ERROR:
ValueError: not enough values to unpack (expected 3, got 2)
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
Traceback (most recent call last):
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py", line 108, in <module>
    ProcessWorker(**args, cmd_original = subprocess.list2cmdline(sys.argv))
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py", line 38, in __init__
    self.execAllPoints()
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py", line 66, in execAllPoints
    cmd_original = self.cmd_original)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/trackProvenance.py", line 28, in execThisPoint
    func(**argkws)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/analysis/compress/processVideo.py", line 142, in processVideo
    compressVideo(video_file, masked_image_file, **compress_vid_param)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/analysis/compress/compressVideo.py", line 366, in compressVideo
    mask = getROIMask(img_reduce, **mask_param)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/analysis/compress/compressVideo.py", line 76, in getROIMask
    mask.copy(), cv2.RETR_TREE, cv2.CHAIN_APPROX_SIMPLE)
ValueError: not enough values to unpack (expected 3, got 2)
'/Users/tom/opt/anaconda3/envs/tierpsy/bin/python' '/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py' '/Users/tom/Desktop/example_data/multiworm/Capture_Ch1_18062015_140908.mov' --masks_dir '/Users/tom/Tmp/MaskedVideos/' --results_dir '/Users/tom/Tmp/Results/' --json_file '/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/extras/param_files/MULTIWORM_OPENWORM.json' --analysis_checkpoints 'COMPRESS' 'TRAJ_CREATE' 'TRAJ_JOIN' 'SKE_INIT' 'BLOB_FEATS' 'SKE_CREATE' 'SKE_FILT' 'SKE_ORIENT' 'INT_PROFILE' 'INT_SKE_ORIENT' 'FEAT_CREATE'
*************************************************
Tasks: 1 finished, 0 remaining. Total_time 0:00:48.
*************************************************

So I copied and pasted that last subprocess command to the command line, and got the following:
(tierpsy) Toms-MacBook-Pro:~ tom$ '/Users/tom/opt/anaconda3/envs/tierpsy/bin/python' '/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py' '/Users/tom/Desktop/example_data/multiworm/Capture_Ch1_18062015_140908.mov' --masks_dir '/Users/tom/Tmp/MaskedVideos/' --results_dir '/Users/tom/Tmp/Results/' --json_file '/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/extras/param_files/MULTIWORM_OPENWORM.json' --analysis_checkpoints 'COMPRESS' 'TRAJ_CREATE' 'TRAJ_JOIN' 'SKE_INIT' 'BLOB_FEATS' 'SKE_CREATE' 'SKE_FILT' 'SKE_ORIENT' 'INT_PROFILE' 'INT_SKE_ORIENT' 'FEAT_CREATE'
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning:numpy.ufunc size changed, may indicate binary incompatibility. Expected 192 from C header, got 216 from PyObject
Capture_Ch1_18062015_140908 Starting checkpoint: COMPRESS
COMPRESS
(1.0, 1.0, 'frames') (1.0, 'pixels') 1
Capture_Ch1_18062015_140908 Extracting video metadata...
Capture_Ch1_18062015_140908 Extracting video metadata. Total time = 0:00:14, fps = 37.0; Frame 500
Capture_Ch1_18062015_140908 Extracting video metadata. Total time = 0:00:29, fps = 32.4; Frame 1000
cannot read
Capture_Ch1_18062015_140908 Starting video compression.
Traceback (most recent call last):
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py", line 108, in <module>
    ProcessWorker(**args, cmd_original = subprocess.list2cmdline(sys.argv))
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py", line 38, in __init__
    self.execAllPoints()
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/ProcessWorker.py", line 66, in execAllPoints
    cmd_original = self.cmd_original)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/processing/trackProvenance.py", line 28, in execThisPoint
    func(**argkws)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/analysis/compress/processVideo.py", line 142, in processVideo
    compressVideo(video_file, masked_image_file, **compress_vid_param)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/analysis/compress/compressVideo.py", line 366, in compressVideo
    mask = getROIMask(img_reduce, **mask_param)
  File "/Users/tom/opt/anaconda3/envs/tierpsy/lib/python3.6/site-packages/tierpsy/analysis/compress/compressVideo.py", line 76, in getROIMask
    mask.copy(), cv2.RETR_TREE, cv2.CHAIN_APPROX_SIMPLE)
ValueError: not enough values to unpack (expected 3, got 2)

I saw in a closed post that some users had been having problems with openCV. I'm using the latest version, openCV version 4.2.0 on macOS Catalina 10.15.2. Could it be that Tierpsy is not compatible with the newest openCV? If so, can you please advise as to how this can be fixed? Thank you!

I got the exactly same error as "ValueError: not enough values to unpack (expected 3, got 2)" while compressing the video after installation followed the given instruction. Did you update it in the condo-forge version now? If not, could you recommend me how to fix this error? Thank you!


Allow usage of ArrayBuffer as request payload

If the request data is of type ArrayBuffer, convert it to a bas64 dataUrl. On recieving this dataUrl in the Message Listener and before sending the actual axios request, convert it back into an ArrayBuffer.

lgtm

lgtm


Setup server structure

Pull Request
Description
Added basic express server and setup server file structure. Also added some node packages needed to run express.
Fixes # (issue)
Type of change
Adding files, folders nd packages
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Absolutely not
Checklist:

[?] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

@JefZheng your PR should tell me what to test and how. What commands should I run? Am I testing things in Chrome or the terminal or Electron? What exactly should I be looking for??


Add RFS GPIO support

Depends on https://github.com/PelionIoT/mbed-edge/pull/81
@whygoyal  - we need a new edge-core release with the RFS functionality.
https://github.com/PelionIoT/meta-mbed-edge/blob/dev/recipes-connectivity/mbed-edge-core/mbed-edge-core.inc

Updated to correct author in commit. (Also happened to be rebased).


Implement player maze-building

Player should be able to build and deconstruct certain tiles in the tower zone, in order to build a maze of fences and towers on top of those mazes, which they should then be able to operate (only allow chickens to fire at mobs if they are currently manning any tower)

[ ] Figure out if the player can build only between waves or during them as well.
[ ] Prevent adding a building that would block the path from the start to the destination

The simplest would be to not allow the creation of walls during waves, only between. Towers, however, could be built whenever.
So the path would be calculated only once, as the wave starts.


Disable the switching of tabs during build

Whenever I'm building my solution VS2022 is always switching to the "Linux Debugger" output, which is very annoying, since I want to see the build output as it has all the details. I just want the Linux Debugger output when I'm actually deploying to the linux device.

@HakanL,
Thank you for the feature request, I can see how that can be non-favorable for those preferring the base build output instead.
If you haven't already, I can look into making this into an editable option for v2.0.

Editable option would be great!


[BUG]: SQLite - drizzle-kit pull fails in my project since kit >= 0.26.0 and orm >= 0.35.0

Report hasn't been filed before.

[X] I have verified that the bug I'm about to report hasn't been filed before.

What version of drizzle-orm are you using?
0.30.1
What version of drizzle-kit are you using?
0.38.2
Other packages
<EMAIL_ADDRESS>Describe the Bug
I have uploaded a repository to make it easier to reproduce and test the issue: https://github.com/rainst/drizzle-bug
what follows is the README of that repo:
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>are the most recent versions that still work fine on my database, when I run orm-kit pull.
If I update the packages to the next available minor versions<EMAIL_ADDRESS>and<EMAIL_ADDRESS>or the latest currently available I end up with this error:
Pulling from ['public'] list of schemas

[⣷] 0 tables fetching
[⣷] 0 columns fetching
[⣷] 0 indexes fetching
[⣷] 0 foreign keys fetching
[⣷] 0 policies fetching
[⣷] 0 check constraints fetching
[⣷] 0 views fetching
SqliteError: no such column: "%3.3f" - should this be a string literal in single-quotes?
    at Object.query (/Users/(...)/projects/drizzle-bug/node_modules/drizzle-kit/bin.cjs:79216:53)
    at fromDatabase3 (/Users/(...)/projects/drizzle-bug/node_modules/drizzle-kit/bin.cjs:38587:32)
    at introspectSqlite (/Users/(...)/projects/drizzle-bug/node_modules/drizzle-kit/bin.cjs:85132:9)
    at async Object.handler (/Users/(...)/projects/drizzle-bug/node_modules/drizzle-kit/bin.cjs:92395:9)
    at async run (/Users/(...)/projects/drizzle-bug/node_modules/drizzle-kit/bin.cjs:90501:7) {
  code: 'SQLITE_ERROR'
}

SCRIPTS FOR REPRODUCING AND TESTING

Run yarn pull to reproduce the error. ❌

Run yarn downgrade to install an older versions of the packages and then yarn pull, the database will be introspected without errors. ✅

Run yarn downgrade:old to install an older version of the packages that still use the old config based on driver rather than dialect and run yarn pull:old to introspect successfully. ✅

Run yarn check-latest to install the latest versions of drizzle orm and kit and run a pull to see if the bug has been fixed in the latest version available. 🔍

WHAT I AM TRYING TO DO
The database contains raw data extracted from Microsoft Flight Simulator 2020 (airports, navigation data and so on..), that I use to run this website: https://msfs.breincorporation.com/.
Currently I manually extract the Airport table and shave off the colums I don't need, I was trying to automate this step with the help of drizzle.

I have updated the app I use to export data from the game to sqlite, tried pulling again and I'm not getting an error anymore when using the latest versions of drizzle orm and kit.
I think this issue can be closed as it appears that it was caused by the sqlite driver used by that 3rd party app. I'm leaving it in the hands of the development team in case someone is still interested in getting to the bottom of what seems to be a very edge case (the bug is still reproducible when pulling my repo).
thank you !


Implement routing (for Authorization)

Pre-requisite  #14
Create routes and redirects:

Login -> app dashboard;
register -> login; and
logout -> login

now available with #79


Display class value in grid

Hello All
I have a class I encrypts and decryption values.  That being said how do display a calculated value on a grid.  I know how to do this many different ways but I would like input on how to do this the Serenity way.
I basically want my grid to look like this
ColumnA  ColumnB = Encrypted(ColumnB value)
I want  the column B value to be available to my form and grid pages so I need to do this on the entity if possible

Do it overriding OnReturn of your repository, or use a formatter

Didnt even think to use a formatter.  Thanks and you the man!!!!
Sent via the Samsung Galaxy S7 edge, an AT&T 4G LTE smartphone
-------- Original message --------From: Volkan Ceylan<EMAIL_ADDRESS>Date: 12/31/16  9:35 AM  (GMT-05:00) To: volkanceylan/Serenity<EMAIL_ADDRESS>Cc: mcb2323<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [volkanceylan/Serenity] Display class value in grid (#1715)
Do it overriding OnReturn of your repository, or use a formatter
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/volkanceylan/Serenity","title":"volkanceylan/Serenity","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/volkanceylan/Serenity"}},"updates":{"snippets":[{"icon":"PERSON","message":"@volkanceylan in #1715: Do it overriding OnReturn of your repository, or use a formatter"}],"action":{"name":"View Issue","url":"https://github.com/volkanceylan/Serenity/issues/1715#issuecomment-269867574"}}}

Can you tell me what Im missing
repository:
private class MyListHandler : ListRequestHandler
{
protected override void OnReturn()
{
base.OnReturn();
Row.EId = "Some String";
}
}
Entity
[DisplayName("EId"), NotMapped]
public String EId
{
get { return Fields.EId[this]; }
set { Fields.EId[this] = value; }
}

Row in list request handler is one entity used to load, your list handler can return multiple entities right?
Do it like
foreach (var e in Response.Entities)
    e.Eld = "Some String";


OAuth authentication

Anyone interested in OAuth authentication support for this package? I have working code using the OAuth NPM that I could contribute.

I'm interested @rcurrier666 and I'm on the support team at Disqus :). I'm considering using this package for a small project I'm working on right now. Anything I can help with?

@rcurrier666 would you be interested in becoming the maintainer of this package? I'm not planning on doing any more development, but if you're interested i would be happy to transfer rights of this repo to you.

@hay, I'm willing to take over the maintenance of the package if you no long want to deal with it. There seems to be an outstanding pull request already, but not a lot of suggestions or users so I should have the time. I'll have to learn how to be a package maintainer, but it can't be that hard (and I have 40+ years of software experience under my belt so I'm really good at picking up new concepts quickly. So if you want, go ahead and transfer it to me and I'll integrate the outstanding pull request as well as add in the OAuth support.

@dmatt, no offense but there are actually support people at Disqus? Is there any support for the API or just for integrating the service into a web site? The support@disqus email just returns an email saying it is not a support address and there doesn't seem to be any Disqus people responding on the Disqus Disqus board.
What I desperately need is a way to pass a small amount of data through the getAuthorizeUrl call back to my redirect_uri. The way it works for the other API I'm using is to simply add a query string to the end of the redirect_uri and that gets returned in the callback querystring. The Disqus API seems to require the redirect_uri to match exactly to the one on the admin page. The alternative approach some API take is to accept a "state" parameter to any endpoint that takes the redirect_uri. Normally its intended to be used as a datum to verify that the callback is actually in response to something my app did as opposed to someone attempting to DDOS my site. Plus it is a handy way to pass back small amounts of data (in my case the account ID of the user who is connecting their Disqus account to their account in my app). If this is something you can put forward to the API developers or if you can let me know the proper channel to ask for changes, I'm happy to provide more details of what I'm trying to do.
In the meantime, I hope to have simple OAuth support added to the NPM next week (at least in my branch). Full integration will likely take a bit longer as the existing code may need a bit of rework to make everything simple and transparent.

@rcurrier666 i made a transfer request. Good luck with this project!

@rcurrier666 thanks for the feedback, sorry that we haven't connected with you in the past. I'm reviewing all the details you shared and I can get in touch with you directly, and work with our engineers to help you out. Can you shoot me an email at dmatt @ disqus ? Thanks for contributing to this project.

Version 0.3.0


Hacktoberfest2022

🛠 Fixes Issue (Number)

👨💻 Changes proposed

✅ Check List (Check all the applicable boxes)

[x] My code doesn't break any part of the project (Zero Octave-Javascript-Projects).
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Profile-contactform click the name to change page theme, popup also available in the page

📷 Screenshots

Not properly followed file structure


Implement password based authentication

Right now we just have OAuth. This would be nice, but it requires a bit of care to implement because we're taking the full security burden on ourselves.

I have a service running meteor, and I'm investigating horizon. A couple of things to consider:

In meteor, the password is hashed on the client before being transmitted to the server (and bcrypted) - some details are listed here. It's nice to make the claim that password-equivalent data isn't stored in the database.

Integrating with a meta-authentication provider like auth0 could be really useful.

My business sells to many older, rural buyers.  While everyone in Silicon Valley may have a Twitter account, please consider the large swath of America that lives on the farm and not online.  Many of my customers don't subscribe to social media (or GitHub!).
I cannot deploy Horizon without some kind of password authentication.

@LawJolla that's a very good point from you and the reason why we're still holding on to use horizon.io. We develop internal backend applications (ERP, CRM, user management, etc) and we can't force the user to have a social account.
Also looking forward to this.

For FWIW -- we are in the same position.  We offer enterprise s/w that sits behind dozens of firewalls etc. (think SAP, PeopleSoft, etc) and obviously having to use a social OAuth mechanism is a total non-starter.  We need both the option to use the end customer's LDAP/AD system and/or self-contained user/pw management (and roles too of course) .

In response to @mattsoftware :

I would love to see some kind of token based authentication, i.e. exchanging username/password/two factor with a token (read JWT) signed with a public/private keypair. Although that would required the private key to be distributed between all the auth servers wanting to do the auth.

Here's a proposal that doesn't require private key to be shared:

During registration, the user's password is used to generate a public & private key pair on the client. This can be done by using the password as a seed for a PRNG.
The public key is sent to the server which stores it in DB.
The private key is not stored anywhere.
During authentication, the server sends some challenge text to the client.
The client regenerates the key pair based on the password, encrypts the challenge text using the private key, and sends it back to server.
Server verifies the authentication request by decrypting the encrypted challenge text using the public key from its database.

Another vote here.

Here is what the meteor guys have done:
Hackpad
accounts-password package

An optional 2FA would be great, there are some very good yet small JS libraries for generating the codes. Would would be a lovely alternative to 2FA is support for passwordless login approvals (e.g. mobile push "accept button").

I don't think its a good idea to implement authentication mechanisms as a part of horizon framework itself.
This should be a part of plugins. While its fine to have basic user information and logged-in status specified in the framework, how a person can authenticate should be extendible.
I am working on system where authentication can happen with passwords but also with SMS, RFID and nfc(MiFare) cards etc. Its better to have plugins for managing authentications and authentication states as there are usecases for that. To be honest meteor has a good grip on this, it wouldn't hurt to borrow from there.

If I'm not mistaken we can have password authentication in horizon right now using auth0 (#601 and #733).
It would be great to have it without having to use an external saas, but it's ok. One step at a time :)
By the way, congrats on the 2.0.0 release!

I also develop internal systems that can't use social accounts. Having this as part of a plugin/package would good.
I am looking to migrate off Meteor, but can't use Horizon without this. So I will have to explore other options, which is unfortunate because I like the simplicity that Horizon offers.

Perhaps it might be simpler just to use something like passport integrated into horizon? MEAN uses passport as well and passport supports over 300 pluggable authentication strategies including username/password and all others like github, facebook etc. that horizon currently supports.
http://passportjs.org/

Yeah this is our current plan. We'll replace our current homespun auth
methods with a passport plugin basically.
On Mon, Aug 29, 2016, 22:03 Shantanu Bhadoria<EMAIL_ADDRESS>wrote:

Perhaps it might be simpler just to use something like passport integrated
into horizon? MEAN uses passport as well and passport supports over 300
pluggable authentication strategies including username/password and all
others like github, facebook etc. that horizon currently supports.
http://passportjs.org/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rethinkdb/horizon/issues/176#issuecomment-243333829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAFVuRw1Np4AB8xrdRek0pOVaU9rDHhks5qk7mdgaJpZM4HpArb
.

@deontologician Awesome! +1 to that!

Oh that sounds like a good idea but a change in direction. What led to that? And of course, the $10k question, when might that hit the shelves?

May want to consider grant. Much smaller and cleaner
https://github.com/simov/grant

Well, the Horizon client can already use JWT's that have been generated by other means with the Horizon({authType: {token: ...}}) constructor. This means that you can use Passport (or whatever) to authenticate your users, generate the JWT token server-side and pass it to the client app through the login API.
I snatched the code that generates the JWT from the source code of the hz make-token CLI command to make it happen. Perhaps a utility function can be added to @horizon/server that setups a new RethinkDB user and generates the token? Something like horizonServer.getTokenForUser(username[, options]). It would then be used like so (hand-wavy):
import passport from 'passport'
import {Strategy as LocalStrategy} from 'passport-local'

...

const app = express();

let horizon = require('@horizon/server')(app.listen(8181), options);

....

passport.use(new LocalStrategy((username, password, done) => {
	if (validateLogin(username, password)) {
		horizon.getTokenForUser(username)
			.then(token => done(null, {token})).catch(done);
	} else {
		done(null, false, {
			message: 'Incorrect user name or password.'
		});
	}
}));

app.get('/api/login', (req, res) => {
	if (!req.user) {
		res.status(403);
	}
	res.json(req.user)
});

app.post('/api/login', passport.authenticate('local'),
	(req, res) => res.json(req.user)
);

The end result is that an authenticated client hitting GET /api/login will get the JWT token that can immediately be passed to the Horizon client constructor.
The getTokenForUser function checks if the user with the given id exists in the database, and creates it if not (sort of like allow_anonymous works). Then, it turns the user id into a JWT.
I think that is actually all that's necessary to have 3rd-party authentication in Horizon. You can even throw away the current OAuth authentication and rely on a recipe for connecting an authenticated user with a RethinkDB user.
Thoughts?

@tailsu good idea. Have you actually built this pattern and if so do you have a repo?

@seddonm1, here you go:
https://github.com/tailsu/horizon-custom-login

Thanks @stefan. You should post it in the #horizon slack channel.
On 14 Nov. 2016 20:50, "Stefan Dragnev"<EMAIL_ADDRESS>wrote:

@seddonm1 https://github.com/seddonm1, here you go:
https://github.com/tailsu/horizon-custom-login
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rethinkdb/horizon/issues/176#issuecomment-260291768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADTtUF_BjdAAG1yhQ3-nE_0RQd_SQf8zks5q-C7QgaJpZM4HpArb
.

Please post that for me, thanks!
Am 14.11.2016 10:56 schrieb "Mike Seddon"<EMAIL_ADDRESS>
Thanks @stefan. You should post it in the #horizon slack channel.
On 14 Nov. 2016 20:50, "Stefan Dragnev"<EMAIL_ADDRESS>wrote:

@seddonm1 https://github.com/seddonm1, here you go:
https://github.com/tailsu/horizon-custom-login
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/rethinkdb/horizon/issues/176#issuecomment-260291768
,
or mute the thread
<https://github.com/notifications/unsubscribe-
auth/ADTtUF_BjdAAG1yhQ3-nE_0RQd_SQf8zks5q-C7QgaJpZM4HpArb>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rethinkdb/horizon/issues/176#issuecomment-260293099,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAskEvE3q-rKQdd-2xWuS1fMVVBRbnjKks5q-DAzgaJpZM4HpArb
.

If Horizon's goal is to take the pain out of the initial scaffolding, this basic password authentication feature has to be built-in. There is no reason why we should be customizing this. Firebase, Loopback, FeathersJS, MeteorJS, ServerlessJS etc... They all offer this as an option.

Any update on this ?


Error on narrowing down non-null variables

Bug Report
🔎 Search Terms
typenarrowing, null
🕗 Version & Regression Information
This is the behavior in every version I tried, and I reviewed the FAQ for entries about null and narrowing
⏯ Playground Link
Playground link with relevant code
💻 Code
type A = {
    c: null
    d: string
} | {
    c: string
    d: null
} | {
    c: string
    d: string
}

const hello = (a: A): void => {
    console.log(a.c)
    console.log(a.d)
}

const main = (c: string | null, d: string | null): void => {
    if (!c && !d) {
        return;
    }

    if (!!c && !d) {
        return hello ({c, d})
    }

    if (!c && !!d) {
        return hello ({c, d})
    }

    return hello ({c, d})
}

🙁 Actual behavior
Typescript doesn't narrow down the variables to be non-null at the end

🙂 Expected behavior
Typescript should be able to narrow down the types to both be non-null

Narrowing is based on the parent control flow graph introducing a direct narrowing, rather than doing a sort of constraint-solving process as would be required to figure this one out. We don't intend to switch to the latter method since code that requires that sort of analysis is quite rare in practice.

https://github.com/microsoft/TypeScript/issues/52822#issuecomment-1434063410


Question: Problem after transmitting a signal

I'm using your great library to receive and transmit 433MHz codes (in python). I want to replace my "hardware" remote control with this.
Everything works now. But there's one thing I don't understand:
When I send a code/signal, the output pin seems to continue sending a signal. I call pi.wave_clear() after transmitting the signal.
My real hardware remote control doesn't work anymore after transmitting a signal with pigpio.
I found a workaroud: When I add pi.set_mode(self.gpio, pigpio.INPUT) at the end of my python script, my hardware remote control works again.
Any idea on that?
Regards, Maurus

You need to prepare a hopefully small example script which demonstrates the problem.  I will then be able to run the script and confirm or deny the problem.

Thanks for the fast reply. Here's the script (an adapted copy of your _433.py) and a json file containing the codes.
To run the script, type "python _433.py tx -f "test.json" -c a_on".
When I comment the line #302 (pi.set_mode(self.gpio, pigpio.INPUT)) my remote control doesn't work after running the script.
When I include the line #302 (as in the example), my remote control continues to work after running the script.
433_Analyze.zip

Before I look in detail could I just check it's not simply because the transmitter is left enabled.  What happens if you change #302 to pi.write(self.gpio, 0)?

Thanks for the hint, it works :)
But I still don't understand why I have to do this explicitely when I delete all waves with pi.wave_clear() before.

It is because you are potentially leaving the gpio pin in a logic 1 state, and thus leaving the transmitter active.  Deleting the waves won't change the state of the gpio, and just leave it in its last state.
You could add a final low going pulse at the end of your waveform if you end the loop with current_level as false (i.e. when the previous pulse when high).  You'd do this just before line 256.
It is probably easier to just include to the line suggested by @joan2937 above to always force it low.

This seems to have been resolved by ensuring the transmitter is off once the signal has been sent.
If you think I need to update any pigpio examples let me know.


New feature: filters

I am trying to migrate a very large (50K photos) photo library from Picasa to Lychee.  Lychee is awesome, but there are a few subtle features of Picasa that I have come to rely upon.  So I would like to contribute those features to Lychee if possible.
This is the first of those features -- a filtering mechanism.  I realize it's very similar to the search capability, but it's different in some key ways.
In picasa, when you are looking at your photos in their original folders, you can turn on filters that will let you see just the starred images, for example.  I file my photos in folders by date (either the date of an event, or the date I dump the photos from the camera).  This lets you look at your best photos, but still view them in an organized fashion.
The "Starred" smart album would be unwieldy with 2000+ starred photos like in my library.
So the filter feature lets you specify that you want starred photos or you can filter by tag.  You'll only see albums that contain photos matching your filter criteria, and when you open an album, the photos you see will be filtered.
As a newcomer here, I want to be respectful of the great work that's already been done.   If the contribution isn't quite ready to accept, please let me know what I can do to improve it.  UI changes, code style, etc.  I hope the feature makes sense, but if there are ways to better distinguish it from the existing search feature, I can make those changes, too.

This PR will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community.


pm2 not stable for production?

Good read, but you're stating pm2 is not ready for production at this time. Could you elaborate?

I'm closing this issue because I've added PM2 a while ago and lack of further discussion.


Hostile mobs no longer pay for Hunter

*Description of issue:
Hunter job not paying for kills of any hostile mobs (naturally spawned).

works fine with non-hostile/passive mobs.

Server Type (Spigot/Paperspigot/etc):
Paper
Server Version (using /ver):
[17:00:39 INFO]: This server is running Paper version git-Paper-183 (MC: 1.16.2) (Implementing API version 1.16.2-R0.1-SNAPSHOT)
[17:00:39 INFO]: Checking version, please wait...
[17:00:39 INFO]: Previous version: git-Paper-178 (MC: 1.16.2)
Relevant plugins (Delete if this isn't needed):
Jobs 4.16.1

Literally you not enabled pay-near-spawner option in generalConfig

Yes, I "literally" have not enabled that option, but these mobs were not spawned by a spawner.

We need more info. Send your current job from jobConfig

jobConfig.zip

Can't reproduce. Every hostile mobs works like a charm (naturally/non-naturally). You probably have some limitations or restrictions in some of jobs configuration, or something plugin which prevents from paying out.

I can find no option anywhere that disables pay for hostile mobs, and no other plugins of mine alters pay or exp with the job plugin.  If you're getting paid for unnaturally spawned mobs then you've enabled the option for players to be paid for killing mobs from spawners, however, and if disabling this disables pay for naturally spawned mobs too, then it isn't doing what it's supposed to be doing.

If you're getting paid for unnaturally spawned mobs then you've enabled the option for players to be paid for killing mobs from spawners, however, and if disabling this disables pay for naturally spawned mobs too, then it isn't doing what it's supposed to be doing.

Literally I got paid from spawners and naturally mobs not just from spawners.
You have to make sure that you aren't in a restrictedArea, you don't have income limit set in generalConfig, or your current world is in disabled worlds and its enabled, and many other things that can relates to income. In this case send your generalConfig.

"Literally I got paid from spawners" which means you set
"enable-pay-near-spawner:" to true.  I have this set to false.  Setting this to false should not disable pay for naturally spawned mobs: further verified by naturally spawned non-hostile mobs in the same area paying for this job.  The current world is NOT disabled, verified by every other job paying in the same world, same with restricted areas since I don't have a single one set.
generalConfig.zip

Nupe didn't work for me at all.  Saw the same trouble reverting... AND even non hostile mobs would not pay on my local copy so I began thinking it may just be something else interfering and narrowed it down to StackMob... which - once I updated the config file to the latest version - no longer seems to be an issue.


feat: add script to generate examples

The script assumes a valid + working FUSOR instance.

@jsstevenson @katiestahl ready for re-review


Switch tx from COMMIT to ASNYC

ref https://github.com/vegaprotocol/vega/issues/3462#issuecomment-839664072

Fixed as part of #46


Update license attribute

specifying the type and URL is deprecated:
https://docs.npmjs.com/files/package.json#license
http://npm1k.org/

The license was updated :-)


Make algebra after the List in supermodel template

Moves the creation of the algebra until after the List, since logically
the algebra informs how lists are handled. Closes #271

Coverage remained the same at 97.989% when pulling d387efe8e5f43116dcf1fd34f1ecd8d2bc608323 on alg-placement into 33cdf07aeacd6303abdd91572466d6effeca35ce on master.


Integration with dotCover

Hi all,
I am attempting to create a fully functional .Net Core build configuration in Teamcity. I got the Nunit integration up-and-running using the following packages:

dotnet-test-nunit | version 3.4.0-beta-3
dotnet-test-nunit-teamcity | version 3.4.0-beta-3

I can run the unit-tests from commandline (dotnet test) and the visual studio interface. Reporting in Teamcity is done using (dotnet test --teamcity). The thing I am trying to accomplish is getting code coverage information in the build configuration using dotCover.
The dotCover command expects a path to a testrunner executable, like documented here Is there a way to implement this using this testrunner?
I have tried the following:

Using the Nunit3 console runner. This results in error’s with not being able to load files or assembly’s from  Nunit. The command is run from the Nunit directory, so the probable cause here is that it’s not compatible with the .Net Core class library.
Using the NunitLite runner (in all the different builds). This results in error’s with not being able to load files or assembly’s. Probable cause here is that I have not restructured my test project like specified on the NuGet package page.
Supplying the dotnet test command to the dotCover. This results in dotCover indicating that it is not a valid executable.
Creating a simple cmd file with the dotnet test command as only statement. This results in the tests running, but it’s not picked up by dotcover properly (coverage session finished but no snapshots were created).

Relevant version information:
•	Teamcity 9.1.5 (build 37377)
•	JetBrains dotCover Console Runner 10.0.2
I am out of ideas basically. I hope that someone can point me in the right direction. I realize that this is more of a questions than an issue, but this does seem like the logical place to start.
Thank you in advance for your support
Oskar uit de Bos

I will leave this open as a question for a bit, but I don't think many people in the team use DotCover. I would recommend contacting JetBrains though, since this doesn't sound like an NUnit specific issue, but one with .NET Core and DotCover.

Thank you for keeping the question open. I will follow up on your recommendation and contact JetBrains. I will get back here with the results so other people may benefit.


Add speaker to support voice assist

Category

[x] Hardware / PCB
[ ] Software / ESPHome

Whats your idea?
Now that i2s microphones are supported (starting 2023.5) and the new assist functionality that comes with it, this project became even more awesome! But it ia missing one thing - the ability to hear the response. Can you add support for a speaker as well?
...

It would be interesting if the ESP-32 C3 is even able to drive all of these devices especially microphone and speakers need a lot of RAM according to the esp home website https://esphome.io/components/speaker/i2s_audio
There are also other alternatives MAX98357A Amazon.de MAX98357 Aliexpress in mono.
I mean stereo would be nice, but how good is the audio quality over a ESP I2S Bus ? Would it be usable for an audio system? (Streaming Music)
BTW here is the same board from adafruit, but cheaper https://www.aliexpress.com/item/1005005698015422.html


Create summary authentication capability table

User Story
As a ReportStream teammate, I want summarize all the information learned about authentication capabilities at ReportStream, so that I can share out materials and insights in an efficient and highly digestible manner.
Background and context
As part of the design brief to add additional third-party authentication providers and abstract our authentication layers, we will need to consider identity proofing as a separate functionality from Okta authentication. As more and more Senders are added to the Universal Pipeline in the near and far future, we will need to prove that these organizations 'are who they say they are', and are not bad actors that could send faulty or fraudulent data into ReportStream.
Open questions

What is the best way to portray this information?
If the information should be in a table, what goes in the axes?
What do scrum team(s) and program lead(s) expect as an output of this information?
What would help guide these stakeholders towards making decisions about this information?

Working links

Part of Authentication enhancements design brief
Authentication/verification information and process flows Mural

Acceptance criteria

[ ] Creation of a share-out artifact that portrays an easily digestible summary of authentication information learned
[ ] Shareout of this material to the broader team

Not in scope

[] Final decisions on RS's security model or system
[] Vetted recommendations on specific authentication functionalities
[] Deciding what authentication capability(s) ReportStream should have

Started a table in this Mural!


ASimNetworkSessionMicrosoftSysmon.yaml-42

Required items, please complete
Change(s):

Created a new file for WindowsEvent and added WindowsEvent parser in it. In Wvent file removed WindowsEvent parser and in ASimNetworkSession added WindowsEvent Parser name .

Reason for Change(s):

To Split the WindowsEvent and Event parser separately

Version Updated:

yes

Testing Completed:

yes

Checked that the validations are passing and have addressed any issues that are present:

yes

Hi @Alekhya0824,
KQL validation failures, please check for those. Thanks

Hi @Alekhya0824,
Validations failing please check. Thanks

Hi @Alekhya0824,
Please clear failed validations. Thanks

Hi @Alekhya0824,
I can see a few open comments from the reviewer. Please act on them.

And also, one suggestion: once you're done with the open comment change, please click on the Resolve Conversation button.

Hi @Alekhya0824,
Please check and respond on above comments. Thanks


no output when passing 0

the function returns nothing when i pass number 0 as a parameter.

@Mustafa744 thanks for reporting ❤ and I am going to fix this


Standardize on envoy node name format and use it as the primary identifier

At present the envoy --service-node name is generated by proxy agent using values of POD_NAME, POD_NAMESPACE passed using k8s downward API.
sidecar~<IP_ADDRESS>~myapp-84fcdcbcf9-dj7qn.default~default.svc.cluster.local
Mixer filter  sends source.workload.uid and destination.workload.uid to Mixer during every call.
These values are computed by pilot (mixer plugin) and placed in the mixer filter config. Since these uid values are different for every sidecard that is requesting config from pilot, every piece of config produced for http_connection_manager is different and not cacheable.
If Mixer filter uses envoy node_id as the uid, pilot does not need to configure it in per filter config.  The resulting config will be same in that case (except for source aware routing.)

Standardize on the new node id format   (remove ip address from it)
Mixer client uses node-id
Pilot also switches to using the presented workload ID.
Ensure that pilot / mixer client continue to accept the current node id format.

The fully qualified name of a workload is  pod-name . pod-namespace  . cluster
In multi cluster scenario cluster is important, in single cluster it is ignore.
Generally the fully name should contain name and registry <some name>@<some registry>

@louiscryan @costinm

Correction: UID is of the form kubernetes://redis-master-2353460263-1ecey.my-namespace

In the interest of time, we should push the logic into mixer to decode the node ID attribute. Envoy should switch from destination.uid and source.uid to using envoy.node.id attribute (which should be always sent), and Mixer can restore the attribute using the direction attribute. I also suggest to drop destination.namespace and source.namespace and restore that on Mixer. This restoration should happen prior to APAs since config resolution requires namespace.

For Cloud Foundry integration, we want endpoint identity to be authenticated.
We imagine this info being encoded in the client-cert which Envoy uses when connecting to Pilot/Mixer.  If client-cert is present, Pilot/Mixer would validate that the endpoint id presented in the XDS request matches what's present in the client cert.
Thoughts?
cc @shalako @sreetummidi

@rosenhouse Looking at the OP above, by endpoint id do you mean the proposed node id? This is the identity of the sidecar?
How would this work for gateways, which should also only receive a subset of config? Gateways are a little different from sidecars because, while all instances of a group of gateways should receive the same config, just as all sidecars of the same app should receive the same config, sidecars must be uniquely identified to prevent misrouting on ingress, and gateways are disposable. I suppose it can't hurt to have all gateways uniquely identified also.

Yes, I mean node id here, and yes this is the identity of the sidecar (and by extension, the pod / workload it is proxying for).  node here is not a K8s "node" but instead the network node in the mesh, effectively a unique workload id, like a "container id" in CF Diego.
You're right that there's less a risk for the edge gateways maliciously requesting incorrect config.  But there is an attack where a malicious workload with access to Pilot could impersonate an edge gateway by acting as an XDS client with a forged ID, and thereby discover information about routes and backend clusters belonging to other tenants.  Requiring all clients of Pilot to provide client-certs with metadata matching their claimed node id would prevent this.


Update Rainbow Six Siege

Game Title
Rainbow Six Siege
Status of the Anti-Cheat
😥 Broken
Tinkering Steps
The Ubisoft forums closed a few weeks ago. To show your support, you need to do so on their Discord server in a channel called #game-feedback and the thread called Linux/Steam Deck Support for Rainbow Six: Siege/Extraction.
Proof of Mention
https://forums.ubisoft.com/

They have a bug tracker website as well. The link might be able to be updated to the latest one: https://r6fix.ubi.com/projects/RAINBOW6-SIEGE-LIVE/issues/LIVE-49179


feat(preset-uno): add object rules

object-center missing rule: object-position: center;, we should add the variants (top, right...)

Can you resolve the conflicts? Thanks.

你能解决冲突吗？谢谢。

ok

I messed up the submission record. Does it matter? It affects me to rebuild a branch


Adjust CI workflows

I want to adjust CI workflows so that each crate has its own corresponding workflow, and there is also a workspace workflow, like this. I wonder if the maintainer can accept it?

Yes sounds good to me


cargo new can generate a better .gitignore file

running cargo new hello_world --bin --vcs git
generates a .gitignore file with
target

can we use something like https://github.com/github/gitignore/blob/master/Rust.gitignore ?
am using "cargo 0.9.0 (8fc3fd8 2016-02-29)" on OSX

This wouldn't be any different, all compiled files are in target?
Plus, it will add the .lock file if you leave off --bin.
On Wed, Apr 13, 2016 at 3:48 AM, Deepak Kannan<EMAIL_ADDRESS>wrote:

running cargo new hello_world --bin --vcs git
generates a .gitignore file with
target
can we use something like
https://github.com/github/gitignore/blob/master/Rust.gitignore ?
am using "cargo 0.9.0 (8fc3fd8
https://github.com/rust-lang/cargo/commit/8fc3fd8df3857f3e77454c992458cd7baeeb622b
2016-02-29)" on OSX
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/cargo/issues/2565

@steveklabnik ah ok. was not aware
Do you think it makes sense to change the generated .gitignore file to ?
# Generated by Cargo
/target/

# Remove Cargo.lock from gitignore if creating an executable, leave it for libraries
# More information here http://doc.crates.io/guide.html#cargotoml-vs-cargolock
# Cargo.lock

I should have mentioned before,
that I am a rust noob. am trying out rust for the first time.
Generally I always look at the github/gitignore repo
for a "standard" .gitignore file
hence this issue

apparently github also ate the second half of my reply, due to mail
but it also will put the Cargo.lock in or not based on if you pass --bin or not.
I would rather submit a PR to the gitignore project to fix it to be more like ours, personally. @alexcrichton what do you think?

Yeah I agree with @steveklabnik that these entries are at least not necessary:
# Compiled files
*.o
*.so
*.rlib
*.dll

# Executables
*.exe

(as they're all in the target directory). I also don't think the leading / on target is always wanted because you may have sub-projects in Cargo which have their own target directories. It's relatively rare to name a module target so those projects can always tweak the gitignore specially.
As for Cargo.lock, I like having documentation in .gitignore, but @steveklabnik is also right that we generate it appropriately based on whether we're a library or binary, so for a "one true gitignore" it should be fine to leave in with docs + a link (as it is now), but through Cargo we already take care of it.
So I guess, yes, I agree that we should probably send a PR to gitignore to remove the build artifacts, but otherwise seems like a good gitignore to me!

Closing as we probably don't need to make changes locally for now, but thanks for the report!


Create the Confirmation Blocks API

Follow the Accounts API PR to develop a wrapper around the Confirmation Blocks API:

https://thenewboston.com/bank-api/confirmation-blocks
https://thenewboston.com/primary-validator-api/confirmation-blocks

I've started a bit of work on this issue if you'd like to assign me. Thanks!


Print statements appear with a delay in the console, making it useless to debug input event timing issues

Tested versions

Reproducible in 4.2.1-stable
Does not happen in 3.5.1 (I suppose all the 3.x branch)

System information
Godot v4.2.1.stable - Ubuntu 22.04.3 LTS 22.04 - X11 - Vulkan (Mobile) - integrated Intel(R) Iris(R) Plus Graphics (ICL GT2) () - Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 Threads)
Issue description
Print statements appear in the console with a slight delay when the project is run. I would rather have them appear in real time such as in Godot 3.5.1.
By "console" I mean the Output panel at the bottom of the main editor window, not the eventual OS-dependent terminal window.
This is very problematic when I debug issues related to input delays. In Godot 3.5.1, I can see the event appear in real time at the very moment the event happens in the window. This is not the case in Godot 4.2.1.
I am testing touchscreen events from a touch laptop. I would like to make sure Godot receives those in real time and this is not a problem with the OS instead that is likely to delay touches. For example, in browsers, there is a 300ms delay applied to disambiguate from scroll actions, and some desktop environments also preprocess those events to catch global gestures. Because of the delayed appearance of the output line in the console, I cannot sense at all if the event delivery delay is to blame on Godot or the operating system.
Steps to reproduce
Create a new project, make a 2d scene, add a script to the root node and add the following:
func _input(event):
    print(event)

Run the project. Do key strokes, mouse clicks, touch the screen if you have one, and notice that the corresponding output appears with a delay in the console.
Minimal reproduction project (MRP)
N/A

May be related:

#84481
#79482

Can you reproduce this after enabling Interface > Editor > Update Continuously in the Editor Settings?

It is much better, back to the realtime console output like in Godot 3!
Now, the "Update continuously" wording seems to have no relation to the console output. I would not have been able to discover it.
I guess, this can be related to the window redraw cycles capping that affects the console too, which is disabled when checking that box. If the window refresh rate is reduced to a lower value when unfocused, I can understand why there is so much perceptible delay in the logs. Is that correct?

The reason this occurs is that editor redrawing in 4.x is limited to 10 FPS when the editor is unfocused by default. This also occurs in 3.x, but it's limited to 20 FPS instead, which means resource usage while unfocused is higher. You can change this by adjusting Unfocused Low Processor Mode Sleep Usec in the Editor Settings (lower values result in higher FPS).

An idea would be a priority system for rendering parts of the editor.
Instead of enforcing a framerate cap on the entire thing, we can think about a way to pass down a rendering policy to the different nodes. Could be preset flags such as "normal / non-priority" or the specific framerate cap if it still makes sense. Depending on the value, each individual node would be responsible to limit its resource usage if applicable.
While most "dumb" widgets such as labels, tab containers, TreeViews do not consume that many cycles and update if there is actual interaction only, those who do constant rendering based on time or do heavy processing would slow down in consequence. For example, the animation player can skip frames; the particles node can do the same or just pause momentarily. The behaviour of each resource-intensive node can be adapted individually.
In addition, we can differentiate between the unfocused mode where the resources must be freed for the running game, and the unfocused situation where the user is just interacting with a different program on its operating system, maybe doing side-by-side Godot and another program on two monitors or split screen windows and doing some kind of visual comparison.

An idea would be a priority system for rendering parts of the editor.

This is very complex to implement in practice, which is why the entire editor is always redrawn when something requests an update.

That's right; there is no requirement of any partial redraw ability in my idea. It is only about letting each node know that there is a need to save resources. In turn, nodes that are the most performance-hungry would have their code adjusted so that they are aware of this information and request updates less often. The "light" nodes would nearly require no change to their behaviour.