state updated when date gets changed for iOS

when date in the porps gets changed for iOS, local state was not updated.

Hey @MJ0826, thank you for the PR 👍
Is it aimed at a performance improvement or does it also fix something else?

In case of iOS if i am using mode "time" and i render the DatePicker with the current date and with minimum date as current date, now if i change the date to some date in the future, i am not able to select the time before the current time for the future date as well, it was happening because the local state was still the date which was mentioned the first time it was rendered.

I think it might be an issue then, because currently the date prop in this component means:

Name
Type
Default
Description

date
obj
new Date()
Initial selected date/time

Which is not the same behaviour of DatePickerIOS, and it is made this way for achieving platform parity between Android an iOS (in the DatePickerAndroid you can only set the initial Date).
Merging this PR would mean:
a. Having a breaking change on the date prop behaviour (which is not a problem)
b. That the Android picker would not behave like the iOS picker.
Could you provide your code? I'd like to take a look at it 🤔

In customDatePickerIos(index.js)  i added this
 componentWillReceiveProps(nextProps) {     if(this.props.date !== nextProps.date) {       this.setState({         date: nextProps.date       });     }   }
in android its working fine, you are able to select previous time for the date in the future.

I'm still not getting one thing thought.
After your PR what would be the meaning of the Date prop?
Initial selected date/time
or
Selected date/time
If it is Selected date/time then I doubt it will work on Android this way, because changing it would not change the DatePickerAndroid selected Date 🤔

what i mean is if i render the date picker with a date, now i change the date by some other source, and i re-render the component with the change date(i.e the props get updated) , so when i give the updated date to date-picker, it was not updating the date from the nextProps. (Also, In case of android, you are not maintaining the local state for the date, you are directly taking it from the props, so if date in the props get updated, it automatically gets updated).

what i mean is if i render the date picker with a date, now i change the date by some other source, and i re-render the component with the change date(i.e the props get updated) , so when i give the updated date to date-picker, it was not updating the date from the nextProps

Yup, I get it, that's exactly what I was asking you in the previous post. It is a matter of initial value (like it is actually) vs controlled component (like the one you're proposing).  Accepting this PR would means introducing a breaking change (because the date prop would change its meaning), I just wanted to make it clear :)
One last thing, then I thing I could merge it.

(Also, In case of android, you are not maintaining the local state for the date, you are directly taking it from the props, so if date in the props get updated, it automatically gets updated).

That's true, but if you change in some way the Date prop while the modal is open it will not change the modal Date on Android.  Not sure about iOS, could you give it a try?
For example you can use a timer to update the Date prop while the modal is open.
This is a minor issue though, I'm not expecting many user doing something like this... but at least we can be clearer in the doc about this "issue" after the merge :)

i guess if a user updates the props, it will automatically update the modal contents accordingly, as it will re-render the component.

@MJ0826 I tested it, on iOS the date gets updated, on Android not (as I suspected, because it is not a controlled component)
I'm merging it for now, but it might needs to be documented before releasing a new version on npm.
Thanks anyway :)

Cool, Will look into it in more detail, what i can do in case of android.

Cross platform parity would be nice

@MJ0826 @temitope landed in release 4.0.0

@mmazzarolo thats great. thanks

Thanks so much for this fix! Worked like a charm.


Inside [] support full SQL functions and operators [moved]

This is Issue 513 moved from a Google Code project.
Added by 2011-08-19T11:45:28.000Z by<EMAIL_ADDRESS>Please review that bug for more context and additional comments, but update this bug.
Original labels: Type-Enhancement, Priority-Low, v2.1
Original description
Extend the "field part" of square brackets supporting full SQL functions and operators.

Now the supported syntax is: http://code.google.com/p/orient/w/edit/Document_Field_Part

Eagerly waiting for this feature. Thanks. Any plans on when 2.1-rc1 will be available?

2.1-rc1 was supposed to be released today, but we had a small regression with Lucene, so it will be released in the next 24h. This feature has been moved to 2.1-rc2. @luigidellaquila can you confirm it?

OK, thanks. Waiting for 2.1-rc2 will not be so bad since we know that it is close enough.

I'm afraid it's not trivial with current query executor, but I'll see what I can do

Thanks for the update. I know your hands are full with a lot of stuff. So this is just nice to have to write shorter and easier to grasp queries but as long as there other ways to achieve the same, I don't think it is a priority.

@lvca @luigidellaquila any news on this issue?

Fixed in 2.1.x and develop branches. The fix will be released with 2.1.12
Closing, please reopen if you see any unexpected behaviors
Thanks
Luigi

Documentation should be updated I guess:
https://orientdb.com/docs/last/Document-Field-Part.html

Hi @karolmie1
Updated, thanks! The docs will be re-generated in short.
Luigi


Don't send error emails from<EMAIL_ADDRESS>

Currently the default configuration is for error emails to be send from<EMAIL_ADDRESS>This assumes that any server is allowed to send from this address, meaning that more explicit SPF records can't be set without blocking everyone's email.
I think the default sender address needs to be changed, e.g. to send from the server's default address (something like www-data@server-name usually).
I'd like to see this fixed on all maintained versions, because at some point, we're going to want to lock down the SPF configuration and the current configuration will break.

If we leave it blank, does it default to the webserver configuration? That might be an alternative.

PR'd this. Opted for 3.x since changing this in a patch release seemed a bit cheeky.

As best I can see, this doesn't happen in SS4 since we removed the Zend-based handler.
@edlinklater I think that you first pointed me to this; do you have any reason to dispute that?

Fixed with https://github.com/silverstripe/silverstripe-framework/issues/5626


bug(responsive): settings not properly displaying

luis noticed the following issue

when resizing to this size, is not displaying properly
macOS

do you mean the notification with "new profile"?

this section

cc @Flemmli97

im not sure what is not displaying correctly? you mean it stretching all the way? it seems to be intended for me

@stavares843 i am still unsure at what exactly the bug is? how should it look like?


Update DG for add order command

In this PR

Updated the DG content for add order command
Included 3 split sequence diagrams for the multi-level nature of the command

Included object diagrams depicting state of the model at each level of the command

Codecov Report

Merging #141 (f84b6f4) into master (1b3887a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #141   +/-   ##
=========================================
  Coverage     63.47%   63.47%
  Complexity      630      630
=========================================
  Files           105      105
  Lines          2379     2379
  Branches        306      306
=========================================
  Hits           1510     1510
  Misses          746      746
  Partials        123      123

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


Only support XDG directory path configuration on Linux (3.x)

3.x version of https://github.com/godotengine/godot/pull/67038.
This also harmonizes the server code to be consistent with Linux desktop (with warnings printed for relative XDG paths).
This closes https://github.com/godotengine/godot/issues/66671.

Thanks!

Cherry-picked for 3.5.2.


k8s: -t for --template is deprecated since v1.1.1

This shouldn't affect backwards compatibility for the Kubernetes example.
-t for --template has been deprecated to make it --tty (https://github.com/kubernetes/kubernetes/pull/12813)
https://github.com/kubernetes/kubernetes/releases/tag/v1.1.1

@enisoc to look at this and merge, thanks!

@zoidbergwill Thanks for the fix! If you can add back the spaces that got eaten in sharded-vtworker.sh, it'll be ready to merge.

Thanks. Should I squash my commits, or isn't it an issue?

Squashing would be appreciated. Thanks!

Thanks. Done.


Tip: Add download ranking to wallets

Hello friends,
I've been looking at the download of wallets and I get an idea in my head that asks to promote competitiveness between wallet products.
I think that if the download screen is organized by some type of ranking or user votes it would generate more competition among the wallet creators, I think that the end users of the products would be happier and there would be more competition.
Also add a category where wallets can be organized by newly added

A filter function is a good idea. Maybe 'multicurrency' for wallets that support other assets. Newly added won't happen often tho and it's not a huge list.
But downloads would be hard to code I think. You'd need to poll google play, the apple store, and the chrome extension store. Which could break at any time if they change how it works.

Love your idea, I could implement something like Dogecoin Voting system using an Doge Address and an value for each wallet that could be easily tracked, but for now we will not add any type ratings on any wallet :)


Added releasing documentation

connects #976

Hi @peterdesmet,
I update the docs based on your ideas in #976.
BTW I don't think we need to go through the Working Group for releases because it's quite pure maintenance task and often it won't include any meanigful changes just being hot fixes. We ask Working Group to vote for any applicable change that is going to be merged into next anyway. So I don't think they need to vote twice.

Closing it for now to make a second attempt while working on https://github.com/frictionlessdata/datapackage/issues/976


 This is terribly distracting and frustrating.
VS Code version: Code 1.63.0 (7db1a2b88f7557e0a43fec75b6ba7e50b3e9f77e, 2021-12-07T06:26:56.179Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6850K CPU @ 3.60GHz (12 x 3598)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
63.91GB (42.13GB free)

Process Argv
--crash-reporter-id f825acfe-92fa-4526-a38a-f8815cf9f7ea

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-6850K CPU @ 3.60GHz (12 x 3597)

Memory (System)
50.14GB (45.27GB free)

VM
0%

Extensions (48)

Extension
Author (truncated)
Version

tsl-problem-matcher
amo
0.5.0

javascript-ejs-support
Dig
1.3.1

eslint-disable-snippets
drK
1.3.0

output-colorizer
IBM
0.1.2

jupyter-keymap
ms-
1.0.0

remote-containers
ms-
0.209.3

remote-ssh
ms-
0.68.0

remote-ssh-edit
ms-
0.68.0

remote-wsl
ms-
0.63.4

vscode-remote-extensionpack
ms-
0.21.0

vscode-icons
vsc
11.8.0

tsl-problem-matcher
amo
0.5.0

vscode-zipfs
arc
2.3.0

better-toml
bun
0.3.2

vscode-eslint
dba
2.2.2

javascript-ejs-support
Dig
1.3.1

xml
Dot
2.5.1

eslint-disable-snippets
drK
1.3.0

EditorConfig
Edi
0.16.4

prettier-vscode
esb
9.0.0

vs-code-xml-format
fab
0.1.5

generateuuid
Fir
0.0.5

shell-format
fox
7.2.2

vscode-graphql
Gra
0.3.41

yarn2-dependency-explorer
hah
0.0.1

terraform
has
2.17.0

vscode-test-explorer
hbe
2.21.1

vscode-esformatter
how
1.6.0

output-colorizer
IBM
0.1.2

innosetup
idl
1.6.1

vscode-random
jre
1.11.0

template-string-converter
meg
0.5.5

code-beautifier
mic
2.3.3

vscode-docker
ms-
1.18.0

vscode-kubernetes-tools
ms-
1.3.4

python
ms-
2021.11.1422169775

vscode-pylance
ms-
2021.12.0

jupyter
ms-
2021.11.1001550889

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.4

test-adapter-converter
ms-
0.1.4

vscode-js-profile-flame
ms-
0.0.21

vscode-paste-image
mus
1.0.4

monolit
Oli
0.0.33

vscode-yaml
red
1.2.2

vscode-icons
vsc
11.8.0

docker
vsc
1.0.0

yaml
vsc
1.0.0

(2 theme extensions excluded)

#134514

I'd also be fine with reading the release notes in Markdown

https://raw.githubusercontent.com/microsoft/vscode-docs/main/release-notes/v1_63.md

Your Release Notes are terrible

Terrible issue title ;) In the future, lay off the hyperbole and instead summarize the issue @oliversalzburg

I do think sometimes the gifs are a little "fast" which makes them more difficult to follow.  The creators could slow the sequences a little (just by pausing occasionally or using a higher frame capture rate) and they would be less "flashy".


[CRUD VIEW] thumbnail not enabled

When I enable ENABLE_REACT_CRUD_VIEWS but not enable THUMBNAILS, dashboard (or chart) list will look like this:

Expected results
For data security reasons, many companies may not want to show thumbnails for charts/dashboard. It doesn't make sense to show thumbnails when there is no thumbnails. The thumbnail view control should be associated with THUMBNAILS feature flag, and only show this control when THUMBNAILS feature is enabled.
Environment
master
Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

cc @nytai @junlincc

@benceorlai

@graceguo-supercat There is a feature flag that controls the default view for the listviews, LISTVIEWS_DEFAULT_CARD_VIEW. I think some users may still want the option to switch over to the card view, even if thumbnails are disabled.
@dpgaspar raised an issue regarding the frontend making a lot of api requests for the thumbnails that are 404ing (due to the feature flag not being enabled). We should definitely not be making the api requests if the feature flag is not enabled, but I'm not sure we should disable the view completely.

@graceguo-supercat what is the nature of the security concern about thumbnails?

for example, a chart's data is form a datasource that I do not have access. when i open chart list page, do i see the thumbnail/screenshot of the chart?

@graceguo-supercat There is a feature flag that controls the default view for the listviews, LISTVIEWS_DEFAULT_CARD_VIEW. I think some users may still want the option to switch over to the card view, even if thumbnails are disabled.
@dpgaspar raised an issue regarding the frontend making a lot of api requests for the thumbnails that are 404ing (due to the feature flag not being enabled). We should definitely not be making the api requests if the feature flag is not enabled, but I'm not sure we should disable the view completely.

That's what i experienced. I did not enable LISTVIEWS_DEFAULT_CARD_VIEW, i still see tons of requests to get thumbnails. No matter THUMBNAILS feature flag is enabled or not, I think it is necessary to have an option to disable card view.

@graceguo-supercat it seems to make sense to disable Thumbnails, however, I am not sure if it should be a systemwide setting (i.e. a flag) or a per-user setting or a permission. My concern with a flag base approach is that it turns off an important discoverability feature for users who otherwise would be able to see the charts. Any thoughts?

@benceorlai Thanks for the reply.
what is an important discoverability feature for users who otherwise would be able to see the charts. If you mean the chart image, that's probably the reason some company disable thumbnails feature, because we do not want users see chart image if they do not have permission. in cardview, it took 80% of page showing an identical placeholder, this also prevent users from discovering their own charts.

Carrying on the discussion from the meetup, let's try to explore a version of cards that's still useful even when thumbnails are not enabled. Disabling entire views because of a lack of thumbnails could lead to vastly different superset experiences which I  don't think is desirable.
We could start by removing the entire thumbnail section and just render the bottom portion as a start, when THUMBNAILS feature flag is disabled.

looping in @Steejay and @mihir174 for thoughts on a thumbnail free card

https://github.com/apache/superset/pull/13683

How to set the default listing style as Card view? Can anybody help


Add files via upload

Steve Winfield
Thank you for giving me the opportunity to apply for the role of Systems Engineer
Questions
Level 0 (optional) - Setup an Ubuntu VM
Prerequisites - Setup the environment
For the purpose of this task I decided to run all activities in the cloud
Specifically my AWS account.
I created an EC2 instance using an Ubuntu Server 16.04 LTS (HVM)
Connected to the instance
I typically use the Ubuntu terminal within windows 10 as the console to interface via SSH to my AWS EC2 Ubuntu instance
Info on the unbutu termain install for Windows can be found here
https://tutorials.ubuntu.com/tutorial/tutorial-ubuntu-on-windows#0
As seen here https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipPO-kFUTJx4yhK3rI8v0T2_SEjkdYO4a8NCM3_j
I then connected to the instance running in AWS
Level 1 - Collecting your Data
I did this using my current DataDog account that Id signed up for prior to speaking to datadog
Once the EC2 install was running i then connected via SSH and installed the agent
Using the following command
ubuntu@ip-172-31-14-234:~$ DD_API_KEY=12345678910 bash -c "$(curl -L https://raw.githubusercontent.com/DataDog/datadog-agent/master/cmd/agent/install_script.sh)"

Bonus question: In your own words, what is the Agent?

The Datadog agent is a python package that is installed on the client. Its purpose is to collect events and metrics from the client itself and then send them to the datadog platform. Once in the platform the end user can gain additional value from the monitoring and performance data that datadog has added value to, in terms of the display and presentation and representation of that data via the data dog web interface
Collecting Metrics

Add tags in the Agent config file and show us a screenshot of your host and its tags on the Host Map page in Datadog.

The tags on my instance are found in
/etc/dd-agent/datadog.yaml
My tags can be seen here
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipPVCYbWwrk78kDSnjuhb3mHDzCvWJnN_j86lmab
For my instance
And this is my environment that i was working with
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipOsQYUapDkWAavHjwuyXN_ReAoX-DqUmPS5W1n0

Install a database on your machine (MongoDB, MySQL, or PostgreSQL) and then install the respective Datadog integration for that database.

Answer:For the purpose of the execercise i decided to install mysql

ubuntu@ip-172-31-14-234:/etc/datadog-agent$ sudo apt-get install mysql-server
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
libaio1 libcgi-fast-perl libcgi-pm-perl libencode-locale-perl libevent-core-2.0-5 libfcgi-perl libhtml-parser-perl libhtml-tagset-perl libhtml-template-perl libhttp-date-perl libhttp-message-perl libio-html-perl liblwp-mediatypes-perl
libtimedate-perl liburi-perl mysql-client-5.7 mysql-client-core-5.7 mysql-common mysql-server-5.7 mysql-server-core-5.7
Suggested packages:
libdata-dump-perl libipc-sharedcache-perl libwww-perl mailx tinyca
The following NEW packages will be installed:
libaio1 libcgi-fast-perl libcgi-pm-perl libencode-locale-perl libevent-core-2.0-5 libfcgi-perl libhtml-parser-perl libhtml-tagset-perl libhtml-template-perl libhttp-date-perl libhttp-message-perl libio-html-perl liblwp-mediatypes-perl
libtimedate-perl liburi-perl mysql-client-5.7 mysql-client-core-5.7 mysql-common mysql-server mysql-server-5.7 mysql-server-core-5.7
0 upgraded, 21 newly installed, 0 to remove and 51 not upgraded.
Need to get 19.4 MB of archives.
After this operation, 162 MB of additional disk space will be used.
I then edited the conf.yaml added my username and password and a addtional tag
ubuntu@ip-172-31-14-234:/etc/datadog-agent/conf.d/mysql.d$ ls
conf.yaml  conf.yaml.example
ubuntu@ip-172-31-14-234:/etc/datadog-agent/conf.d/mysql.d$ sudo vi conf.yaml
ubuntu@ip-172-31-14-234:/etc/datadog-agent/conf.d/mysql.d$
init_config:
instances:
# NOTE: Even if the server name is "localhost", the agent will connect to MySQL using TCP/IP, unless you also
# provide a value for the sock key (below).

server: localhost
user: datadog
pass: datadog
port: 3306             # Optional
sock: /path/to/sock    # Connect via Unix Socket
defaults_file: my.cnf  # Alternate configuration mechanism
connect_timeout: None  # Optional integer seconds

tags:                  # Optional
- sawmysqlinstance
#   - optional_tag2
# options:               # Optional
#   replication: false
#   replication_channel: channel_1  # If using multiple sources, the channel name to monitor
#   replication_non_blocking_status: false  # grab slave count in non-blocking manner (req. performance_schema)
#   galera_cluster: false
#   extra_status_metrics: true
#   extra_innodb_metrics: true
#   extra_performance_metrics: true
#   schema_size_metrics: false
#   disable_innodb_metrics: false
#
#     NOTE: disable_innodb_metrics should only be used by users with older (unsupported) versions of
#           MySQL who do not run/have innodb engine support and may experiment issue otherwise.
#           Should this flag be enabled you will only receive a small subset of metrics.
#
#     NOTE: extra_performance_metrics will only be reported if performance_schema is enabled
#           in the MySQL instance and if the version for that instance is >= 5.6.0
#
#           extra_performance_metrics and schema_size_metrics will run two heavier queries
#           against your DB to compute the relevant metrics for all your existing schemas.
#           Due to the nature of these calls, if you have a high number of tables and/or schemas,
#           you may be subject to some negative impact in the performance of your DB.
#           Please bear that in mind when enabling them.
#           Metrics provided by the options:
#                     - mysql.info.schema.size (per schame)
#                     - mysql.performance.query_run_time.avg (per schema)
#                     - mysql.performance.digest_95th_percentile.avg_us
#
#           With the addition of new metrics to the MySQL catalog starting with agent >=5.7.0, because
#           we query additional schemas to get this full set of metrics. Some of these require the user
#           defined for the instance to have PROCESS and SELECT privileges. Please take a look at the
#           MySQL integration tile in the Datadog WebUI for further instructions.
#
# ssl:               # Optional
#   key: /path/to/my/key.file
#   cert: /path/to/my/cert.file
#   ca: /path/to/my/ca.file
# max_custom_queries: 20 # Optional
# queries:               # Optional
#  - # Sample Custom metric
#    query: SELECT TIMESTAMPDIFF(second,MAX(create_time),NOW()) as last_accessed FROM requests
#    metric: app.seconds_since_last_request
#    tags:               # Optional - only applied to this custom metric query, will not affect default mysql metrics
#        - custom_tag1
#        - custom_tag2
#    type: gauge
#    field: last_accessed
#  - # Sample Custom metric
#    query: SELECT TIMESTAMPDIFF(second,MAX(create_time),NOW()) as last_user FROM users
#    metric: app.seconds_since_new_user
#    tags:               # Optional - only applied to this custom metric query, will not affect default mysql metrics
#        - custom_tag1
#        - custom_tag2
#    type: gauge
#    field: last_user

Log section (Available for Agent >=6.0)
#logs:
#SAW comments - Via the Webinterface i then also installed/Enabled the mysql intergration services
It is then required to enable the mysql inegration as per this screen shot
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipPgcRPzS-fmlbjuyJ9k9IjI-6LHOzd0drIlYv7d
Level 2 - Visualizing your Data

Since your database integration is reporting now, clone your database integration dashboard and add additional database metrics to it as well as your test.support.random metric from the custom Agent check.

To add the metric I needed to create two files
Firsly the randomcheck.py file
This lives in  /etc/datadog-agent/checks.d/
The command i used was the follow
Sudo vi my_metric.py
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipO2JHyXiCgn-v06yxuiD9AL9YCLmFjwj5iLaFDl
I then added the following
import random
from checks import AgentCheck
class my_metric(AgentCheck):
def check(self, instance):
self.gauge('my_metric',random.randint(0,1000))
I then created the my_metric.yaml file
This lives in
etc/datadog-agent/conf.d/
Again
sudo vi randomcheck.yaml
And then i added the following to the file
init_config:
instances:
[{}]

Bonus question: Can you change the collection interval withoout modifying the Python Check file you created?
I dont believe this is possible - Unless the whole global check time via the agent is edited

#Visualizing Data
I then took a look at the Dashboards within the GUI
Selected Dashboard list
and searched for mysql
Seen here
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipMSqWZG_4nTgsSMF9rnIEqvIdQXI01UkYyNwdjc
I then added clicked "clone dashboard"
I then added a query value (dragged to the desktop view and search for my_metric)
As seen here
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipM0pQz6_uZyuFOexqakOa9hU8Cfay57j25t1vzh
For the anomaly function task
I choose the reads per second graph and added the anomaly query
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipPwrtP4t8d8pM5AN6cgLbFRtamEBnoWDvlPFZcn
#~Your custom metric with the rollup function applied to sum up all the points for the past hour into one bucket
I refered to this article to perform this task
https://docs.datadoghq.com/graphing/functions/rollup/
3600 seconds in an hour
So my JSON looked like the following
{
"viz": "query_value",
"requests": [
{
"q": "avg:my_metric{*}.rollup(sum, 3600)",
"type": null,
"style": {
"palette": "dog_classic",
"type": "solid",
"width": "normal"
},
"conditional_formats": [
{
"palette": "white_on_red",
"comparator": ">",
"value": null
},
{
"palette": "white_on_yellow",
"comparator": ">=",
"value": null
},
{
"palette": "white_on_green",
"comparator": "<",
"value": null
}
],
"aggregator": "sum"
}
],
"autoscale": true
}
Past 5 mintue view
https://photos.google.com/u/1/album/AF1QipN3zZonzki3Np1MVEp4QpataTmLI2L_tEsrpjqj/photo/AF1QipPpKPl0rqjWrb8dOQkKB-gMdGbj-8E5Edhit9PV
#API Script
from datadog import initialize, api
options = {
'api_key': 'e98ccb8ba11a832f9db9731085b67f9a',
'app_key': 'dca2fd353700ab97d7ff7e5f99323ef31afa09c5'
}
initialize(**options)
title = "SAWTimeboard created via API!"
description = "SAW informative timeboard."
graphs = [{
"definition": {
"events": [],
"requests": [
{
"q": "anomalies(avg:mysql.max_connections{host:i-0c397fcc6d561709f}.as_count(), 'basic', 1)",
"type": "line",
"style": {
"palette": "dog_classic",
"type": "solid",
"width": "normal"
},
"conditional_formats": [],
"aggregator": "avg"
}
],
"viz": "timeseries"
},
"title": "Average of postgresql.bgwriter.buffers_alloc on host precise64 * with anomaly function applied"
},
{
"definition": {
"events": [],
"requests": [
{
"q": "avg:my_metric{host:precise64}",
"type": "line",
"style": {
"palette": "dog_classic",
"type": "solid",
"width": "normal"
},
"conditional_formats": [],
"aggregator": "avg"
}
],
"viz": "timeseries"
},
"title": "SAW My custom metric scoped over my host"
},
{
"definition": {
"events": [],
"requests": [
{
"q": "avg:my_metric{*}.rollup(sum, 3600)",
"type": "line",
"style": {
"palette": "dog_classic",
"type": "solid",
"width": "normal"
},
"conditional_formats": [],
"aggregator": "avg"
}
],
"viz": "timeseries"
},
"title": "My custom metric with the rollup function applied to sum up all the points for the past hour into one bucket"
}
]
template_variables = [{
"name": "var-env",
"default": "dev"
}]
read_only = True
api.Timeboard.create(title=title,
description=description,
graphs=graphs,
template_variables=template_variables,
read_only=read_only)
#Take a snapshot of this graph and use the @ notation to send it to yourself.
#Bonus Question:What is the Anomaly graph displaying?
#---------------------------------------------------------------------#
SAW  - Task stopped at this stage
In the time board all graphs are view view a specifc time. IN a screen boad the time can be specfied you wanted to view a partiular value at. It is also customisable via drag and dropiing the widgets
reference

Take a snapshot of your test.support.random graph and draw a box around a section that shows it going above 0.90. Make sure this snapshot is sent to your email by using the @notification

Answer: click on the camera icon on the top right of the graph will get you a screenshot of the current graph, draw a box on the graph and type mesage @your_email, then the screenshot will be sent to your email address.

Level 3 - Alerting on your Data
#Monitoring Data
Since you've already caught your test metric going above 0.90 once, you don't want to have to continually watch this dashboard to be alerted when it goes above 0.90 again.  So let's make life easier by creating a monitor.

Set up a monitor on this metric that alerts you when it goes above 0.90 at least once during the last 5 minutes

Answer: click on the setting icon - create Monitor on the top right of the graph

1.Choose the detection method

2.Select the metric and scope you want to monitor

3.Select the alert grouping

reference

Bonus points:  Make it a multi-alert by host so that you won't have to recreate it if your infrastructure scales up.

Answer: select Multi Alert under 2-Define the metric, has been done in the previous step.
A simple alert aggregates over all reporting sources,You will get one alert when the aggregated value meets the conditions set below. This works best to monitor a metric from a single host. A multi alert applies the alert to each source, according to your group parameters.

Give it a descriptive monitor name and message (it might be worth it to include the link to your previously created dashboard in the message).  Make sure that the monitor will notify you via email.

Give the monitor a title. It is often useful to use a succinct explanation of the monitor so a notified team member can quickly understand what is going on.

Enter a message for the monitor. This field allows standard markdown formatting as well as @your_email_address.

This monitor should alert you within 15 minutes. So when it does, take a screenshot of the email that it sends you.

Bonus: Since this monitor is going to alert pretty often, you don't want to be alerted when you are out of the office. Set up a scheduled downtime for this monitor that silences it from 7pm to 9am daily. Make sure that your email is notified when you schedule the downtime and take a screenshot of that notification.

Answer: Monitors - Manage Downtime - Schedule Downtime

1.Choose what to silence.

2.Set a schedule.

(optional)3-4.Add an optional message to notify your team

Instructions
If you have a question, create an issue in this repository.
To submit your answers:

Fork this repo.
Answer the questions in answers.md
Commit as much code as you need to support your answers.
Submit a pull request.
Don't forget to include links to your dashboard(s), even better links and screenshots.  We recommend that you include your screenshots inline with your answers.

About Me
City of Manchester County Squash Player, Football Coach, Techie
My Linkedin


Consider deleting all io.flutter.facade classes

The io.flutter.facade classes autogenerated into .android seem to be an older attempt to create an embedding API that is mostly replaced. It adds maintenance cost since they need to keep building when the modules build and adds cognitive load since users need to realize what they are and subsequently ignore them.
Delete these from the template.
cc @blasten @matthew-carroll

Another component of this bug is that once we merge #39384, people on master would have their existing project break. Part of the migration would be to announce that if their add-to-app code is currently importing io.flutter.facade, flutter make-host-app-editable should be used to keep that code in their project so that it doesn't get deleted the next time flutter packages get is run.
Once a manual migration to io.flutter.embedding is done, the /android directory in the module can just be deleted to complete the migration.

Once the new Android embedding is turn on by default, we can close this issue since the new embedding doesn't use the facade class.

Discussed offline. Before we flip the switch https://github.com/flutter/flutter/issues/43638, the task is to send out an announcement email instructing people how to preserve and hold onto their facade classes if they want to keep them.

Can I please get a link to that manual you're referring to

it's at flutter.dev/go/android-project-migration


Update prepare-a-local-relay-chain.md

This PR aims to address issue #2091 thereby updating the git clone section to be consistent with the Polkadot version v1.0.0 used throughout the tutorial. However, this can be changed in the future if newer releases are adopted across all sections in the tutorial.


Vercel DB integration issue

Am seeing the following error after adding Vercel Postgresql DB integration:
[19:32:09.139] Cloning completed: 662.128ms
[19:32:18.678] Restored build cache
[19:32:18.797] Running "vercel build"
[19:32:19.961] Vercel CLI 37.1.2
[19:32:21.918] Installing dependencies...
[19:32:27.655]
[19:32:27.656] ><EMAIL_ADDRESS>prepare
[19:32:27.656] > husky
[19:32:27.657]
[19:32:27.734]
[19:32:27.735] removed 1 package in 6s
[19:32:27.735]
[19:32:27.735] 437 packages are looking for funding
[19:32:27.736]   run `npm fund` for details
[19:32:27.748] Detected Next.js version: 14.2.5
[19:32:27.767] Running "npm run build"
[19:32:28.018]
[19:32:28.018] ><EMAIL_ADDRESS>build
[19:32:28.019] > next build
[19:32:28.019]
[19:32:29.162]   ▲ Next.js 14.2.5
[19:32:29.162]   - Environments: .env.production, .env
[19:32:29.163]   - Experiments (use with caution):
[19:32:29.163]     · instrumentationHook
[19:32:29.163]
[19:32:29.232]    Creating an optimized production build ...
[19:32:29.862] [@sentry/nextjs - Node.js] Warning: No auth token provided. Will not create release. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/
[19:32:29.862] [@sentry/nextjs - Node.js] Warning: No auth token provided. Will not upload source maps. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/
[19:32:52.963] [@sentry/nextjs - Edge] Warning: No auth token provided. Will not create release. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/
[19:32:52.963] [@sentry/nextjs - Edge] Warning: No auth token provided. Will not upload source maps. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/
[19:32:58.260] [@sentry/nextjs - Client] Warning: No auth token provided. Will not create release. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/
[19:32:58.261] [@sentry/nextjs - Client] Warning: No auth token provided. Will not upload source maps. Please set the `authToken` option. You can find information on how to generate a Sentry auth token here: https://docs.sentry.io/api/auth/
[19:33:15.682]  ✓ Compiled successfully
[19:33:15.683]    Linting and checking validity of types ...
[19:33:37.283]    Collecting page data ...
[19:33:39.789] error: expected one dependency record for TOAST table, found 4
[19:33:39.790]     at Ke.parseErrorMessage (file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:12967)
[19:33:39.795]     at Ke.handlePacket (file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:9640)
[19:33:39.795]     at Ke.parse (file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:8285)
[19:33:39.796]     at file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:23863
[19:33:39.797]     at Array.forEach (<anonymous>)
[19:33:39.797]     at file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:23841
[19:33:39.797]     at file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:16185
[19:33:39.797]     at ge.<anonymous> (file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:14634)
[19:33:39.797]     at Generator.next (<anonymous>)
[19:33:39.797]     at l (file:///vercel/path0/node_modules/@electric-sql/pglite/dist/chunk-OVRU7FYL.js:1:14020) {
[19:33:39.798]   length: 121,
[19:33:39.798]   severity: 'ERROR',
[19:33:39.798]   code: 'XX000',
[19:33:39.799]   detail: undefined,
[19:33:39.799]   hint: undefined,
[19:33:39.799]   position: undefined,
[19:33:39.799]   internalPosition: undefined,
[19:33:39.799]   internalQuery: undefined,
[19:33:39.800]   where: undefined,
[19:33:39.800]   schema: undefined,
[19:33:39.801]   table: undefined,
[19:33:39.801]   column: undefined,
[19:33:39.801]   dataType: undefined,
[19:33:39.801]   constraint: undefined,
[19:33:39.801]   file: 'cluster.c',
[19:33:39.802]   line: '1253',
[19:33:39.802]   routine: 'swap_relation_files'
[19:33:39.802] }
[19:33:39.804]
[19:33:39.804] > Build error occurred
[19:33:39.808] Error: Failed to collect page data for /[locale]/api/submit-waitlist-email
[19:33:39.809]     at /vercel/path0/node_modules/next/dist/build/utils.js:1268:15 {
[19:33:39.809]   type: 'Error'
[19:33:39.809] }
[19:33:39.949] Error: Command "npm run build" exited with 1
[19:33:40.522]

Any idea how to solve this issue?

Fixed by going back to a working commit, and re-applying required DB integration related changes.

@chandasampath Awesome! Thank you for letting us know.


Can I change the Overlay position with animation?

Hi,
I have an overlay and I would like to move it to another position with a smooth movement. Is this possible? How?
Thanks in advance!

You can try to use CSS transition, something like:
.ol-overlay-container {
  transition-property: left, top;
  transition-duration: 0.25s;
  transition-timing-function: linear;
}


fix: removed validators side effects

Description

Reimplementation of #988 without the use of a shared context.
We decided that we don't want to have a context, even if it didn't tie any of the validators to using it or expecting anything to be there. This has some drawback: some code needs to be executed more times to retrieve the same values: one time to validate it and one time to put it into the resulting object.
Verification Steps

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

I don't really agree. We are duplicating database and AMS calls just to avoid having a context (which is not an anti-pattern and is not uncommon). I believe the previous approach (use it from validation context or get it yourself) didn't add any complexities at all since it didn't create any relation/dependency between the validators. On the other side, it allowed to avoid any duplication. But, as I said, I'm democratic and will follow the majority :D

Let's go ahead with this. It is a tradeoff matter. If we merge this and we notice an impact on performance then we can reevaluate the solution to take


Appium multi touch

Allows the passing of multiple tuples for single touch move and arrays of arrays of tuples for multiple touch moves at once.

I think we should discuss these more. If we want to support the touchmove thing with multiple, we could possible just do multiple arguments, rather than an array of arrays

It's not one array of arrays, it's multiple arguments that are arrays of arrays.
If you just pass in multiple single-dimensional arrays (tuples), those arguments will control where one finger moves on the screen.
If you pass in multiple arrays of arrays, each outer array represents one finger, with each inner array (tuple) being the moves that finger makes.
Is this too confusing?
One other approach I could see might be passing in multiple objects which each have an "actions" array.


radio group label spacing: ReportFormpage

from beta site testing "report" page
Anonymity question - first line of each option does not line up with corresponding radio button, and line spacing does not clearly separate the options.
EITHER
realign buttons with first line of each option
OR
Separate options with more space
This also falls in line with redoing utility components and converting them from Mui to custom components.
Fixing the spacing or text alignment on the labels may not be super simple since the radio groups are custom components built with multiple custom styled Mui components.


Select view with alias breaks

SELECT t.* FROM  myView AS t currently panics. This prevents DataGrip from opening view tables.
This blocks full resolution of https://github.com/liquidata-inc/dolt/issues/516

I actually already fixed this, but I'll add a passing one.

Thanks.
On Wed, Apr 22, 2020 at 1:31 PM Zach Musgrave<EMAIL_ADDRESS>wrote:

I actually already fixed this, but I'll add a passing one.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/liquidata-inc/dolt/issues/581#issuecomment-618023302,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJAR3CYGSYZWZOBCQNO233RN5HZLANCNFSM4MIXCBIA
.


Pattern #1000 is missing for guns

In L116 of FadeCalculator.ts you stop at pattern ID 999, but pattern ID 1000 can be obtained through trade-ups, so it is missing for all non-knife weapons.

Thanks for reporting this issue, seed 1000 is now available for all guns.


Option to add a custom vpn or geo provider

If done right, every vpn detection provider could be implemented purely in the config without the need of hard-coded implementations.

Added necessary config entries in c7972cf6b55e3628e9a65ea87975ecfdc30b726c.

Implemented in 771eb56ea479a70b92897b2c8ae2ce70aae4ee54.


Redo Documentation

Documentation needs to be redone according to the style guides laid out here

We should consider setting up Sphinx for documentation. It's pretty standard and will parse the google comment style listed above.
For setup and other documentation we should probably prioritize the GitHub wiki.

Can Sphinx read the google documentation formatting originally linked?

Yes, it supports that. I've used this on other projects recently.

Example documentation that's a little easier to read than their outline can be found here
https://github.com/nmpiazza/Hermes/blob/76c9c0e3efb9eef928b749a20a3806e1131ba738/app/mod_check/HTTP.py#L8-L24


Print correct path when unable to find GDExtension library

When loading an extension, the library_path isn't set till later when it actually attempts to load the file on line 767.
So its printing out a blank string when it should be printing the location it expects to find the extension.dll

Should be added to line 753 too probably

Thanks! And congrats for your first merged Godot contribution :tada:


[BUG]: Broken API (0.5.58 → 0.5.64) : ERROR TS2307 - Cannot find module 'xxx' or its corresponding type declarations

What happened?
When updating in my own plugin the dataview package to 0.5.59, TS can't found any declaration for Obsidian-dataview.

Translation:

Try npm i --save-dev @types/obsidian-dataview if it exists or add a new declaration (.d.ts) file containing `declare module 'obsidian-dataview';
ERROR TS7016 Could not find a declaration file for module 'obsidian-dataview" […]

Spotted that the two last version doesn't have the main.d.ts file:

For 0.5.59
For 0.5.58

Comparison to 0.5.56, where libs contains everything.
DQL
/
JS
No response
Dataview Version
0.5.58 - 0.5.56
Obsidian Version
Not relevant
OS
Windows

Updating to 0.5.60 should help, but now, any import lead to "any" and opening the index.d.ts & other files leads to that :

Error (TS2307)
error TS2307: Cannot find module 'xxxx'
My tsconfig:
{
  "compilerOptions": {
    "baseUrl": "./",
    "inlineSourceMap": true,
    "inlineSources": true,
    "module": "ESNext",
    "target": "ES6",
    "allowJs": true,
    "noImplicitAny": true,
    "moduleResolution": "node",
    "importHelpers": true,
    "isolatedModules": true,
    "resolveJsonModule": true,
    "strict":true,
    "lib": [
      "DOM",
      "ES5",
      "ES6",
      "ES7",
      "ES2021",
      "ES2022"
    ],
  },
  "include": [
    "**/*.ts"
  ]
}

Bug remains in 0.5.61

Remains in 0.5.64

I'm having the same issue. I did a fresh clone of the obsidian-sample-plugin repo and followed the instruction for adding the dataview api to the plugin and getting the same problems listed above.

As @Lisandra-dev pointed out this was broken after 0.5.56. From what I can see, in the following release, a dependency on ttypescript was removed and a misconfiguration in the rollup.config.js caused types to go missing (#2073) for few releases until it was fixed in 0.5.60 (#2074). However, from 0.5.60 and onwards the imports in the generated d.ts files are no longer relative paths, causing module resolution to fail resulting in importing those types as any as shown above.
Note the difference between the imports in index.d.ts (0.5.56) vs index.d.ts (0.5.60).
As a temporary workaround I manually edited the imports in the d.ts files in node_modules to use relative imports and that resolved the issue. Of course this will revert to the error when you wipe node_modules or do a clean install.
I don't know what a real fix to this would be. I'm assuming the removal of ttypescript had something to do with it but I'm not familiar with that package. Perhaps setting paths in tsconfig-lib.json?

My methods to fix this on my side:
https://github.com/ObsidianPublisher/obsidian-github-publisher/blob/e2e3738541d0188ae0f90397837395ee0c8f2a0b/tsconfig.json#L4-L10

I needed to make additional paths beyond the above but now I'm down to an error about LocalForage. I'm just trying to use in node, not an obsidian plugin.

Any update…?

I've got some workarounds
tsconfig.json
{
  "compilerOptions": {
    "paths": {
      "*": [
        "./node_modules/obsidian-dataview/lib/*"
      ]
    }
}

Regarding LocalForage issue, it's solved by adding somewhere in your code
import "localforage";


Remove flux store usages from utils

Summary
Remove flux store usages from utils. We should probably not have any references to the stores at all in the utils but that's a much larger task than what I'm trying to do here.
@lindy65 can you test to make sure MFA enforcement still works and that clicking on desktop notifications sends you to the right channel?
Ticket Link
Part of https://mattermost.atlassian.net/browse/MM-12564
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spintmint test server creation failed. See the logs for more information.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spintmint test server creation failed. See the logs for more information.

@jwilander - can you please take a look at why the spinmint won't build?

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0537aa63646762269.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0537aa63646762269

Thanks @jwilander
Tested:

MFA enforcement

works as expected for a user who has MFA turned on already as well as for a user who doesn't and has to set it up at next login.

clicking on desktop notifications sends you to the right channel

works as expected - no issues found

Spinmint test server destroyed

@jwilander I think this needs a rebase.


replace "component-spec" language bindings w/ OCM-Lib

Context / Motivation
component-cli was deprecated (superseded by ocm-cli and removed from gardener-cicd (pipeline template and all pipelines) as part of cleanup and consolidation activity.
component-cli as well as testmachinery uses golang bindings from component-spec repository for retrieving and deserialising OCM-Component-Descriptors. Said language-bindings were also superseded by a re-implementation (in golang) in ocm-repository.
The re-implemented ocm-lib supports handling of different versions of component-descriptors w/ a stable API (which is promised to be kept stable also for future spec-versions of OCM; there are already two spec-versions: v2 and v3 (gardener-cicd currently uses v2).
Adopting the re-implemented ocm-lib in testmachinery will allow us to archive the (currently unmaintained) "component-spec"-language-bindings and (hopefully) reduce related maintenance efforts in the future.
Implementation Proposal
Remove dependencies from testmachinery towards https://github.com/gardener/component-spec, and port it to using ocm-lib instead.

@dguendisch | @hendrikKahl : @fabianburth might help you w/ this task

Just wanted to mention that I did not forget about this but had a shift of priorities. I will try to get to this soon!


[Frontend] Buy page

Adding a very basic "Buy" section on the token page, allowing users to easily buy tokens using AVNU quotes. You should be able to specify a slippage, a token to spend (we'll only support quote tokens), and an amount to spend.
you can get AVNU quotes this way: useAvnu.ts

Please, before contributing, keep in mind that Unruggable prioritizes security and UX above everything else

Can I take this issue?

Feel free to assign it to me @0xChqrles

Hello I am a React and typescript software engineer, I have been for the past months exploring Starknet ecosystem and contributing to projects. I saw this project via Twitter and found It interesting, now that the ODhack allows to contribute, I would love to bring my help!

Hello @0xChqrles My name is Ugo, a full stack dev currently learning Cairo. i would love to contribute to this effort. Please can i be assigned this task? will get it done in no time

Hi @0xChqrles , i could work on this instead if KeneePatel is not available

I am working on it though. Will give update soon.


Restrict customProperties to kv pair of strings and warnings discouraging sensitive info storage

These changes will allow us to more easily implement compatibility with https://github.com/lmolkova/correlation/blob/master/http_protocol_proposal_v1.md#http-protocol-proposal

Adding @lmolkova for FYI


Fail to load pretrained models

Hello
I failed to load the pre-trained models in Intel Caffe 1.1.0 (commit commit a3d5b022fe026e9092fc7abc7654b1162ab9940d).
I get the following error message:
I0613 13:24:30.011922 22693 net.cpp:261] Creating Layer Convolution7
I0613 13:24:30.011926 22693 net.cpp:1498] Convolution7 <- Convolution6
I0613 13:24:30.011931 22693 net.cpp:1498] Convolution7 <- Convolution2_p
I0613 13:24:30.011937 22693 net.cpp:1472] Convolution7 -> Eltwise1
F0613 13:24:30.011960 22693 blob.cpp:260] Check failed: count_ == other.count() (25088 vs. 1568)
*** Check failure stack trace: ***
I am hoping to convert your models to Keras/Tensorflow to try them out on a smaller scale problem using transfer learning.

Hi Kenatse,
Could you please provide more information (e.g. how you are changing prototxts for transfer learning)? What specific SqNxt variant are you running?


Bug/table with plot validation

looks good. id question if the pasting logic is robust to deal with issues where properties are out of order (i.e. what if x is not the last property).

I'm nervous about that too, but I'm unsure how to address it right now. I think it has something to do with how we've set up the indexing of columns on the display as column attribute. even though we display X at index 0, we do not put X at index 0. This fix got us away from referencing sequential row directly, but still leaves some room for improvement.  I'm open to suggestions if you have ideas.


Add PACE BMS support

See syssi/esphome-pace-bms#1

I've moved the implementation into a new project: https://github.com/syssi/esphome-pace-bms/


Little fix with iframe ( sessionCheckIFrameName )

Hi @manfredsteyer,
for a while I had my angular material template broken with a scrollback I couldn't get rid of. I just noticed that the iframe "angular-oauth-oidc-check-session-iframe" was the source of this.
A simple fix :
iframe.style.visibility = 'hidden';
iframe.style.display = 'none'; // <--

With this line, the scrollbar goes away.
Thank you,
Julien

Hi @manfredsteyer, would you like a PR for this ?

thx for this. PR is merged. Will be released soon.


Always wipe the stuffer, even after rewrite

Description of changes:
s2n_stuffer_rewrite can cause the stuffer write cursor to go to zero, which causes the stuffer not to wipe the blob when s2n_stuffer_wipe() is called.

Fix this.
s2n_stuffer_free no longer needs to do a wipe, since https://github.com/awslabs/s2n/pull/1319 was merged
While I'm in there, free should reset all fields in the stuffer.

NOTE: I could optimize this by keeping a high_water_mark and memset up to there.  Zeroing the whole blob is less code, and hence less error prone.  If people think the high-water-mark approach is worth doing, I can do it, but it feels like premature optimization without reason to believe its needed.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1322 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #1322   +/-   ##
=======================================
  Coverage   89.21%   89.21%
=======================================
  Files         235      235
  Lines       20169    20169
=======================================
  Hits        17993    17993
  Misses       2176     2176


The file belongs to an unsupported type: mxml

It looks like you are using the AIR SDK and Compiler. That is AS3 only. I think that you meant to use the Flex SDK instead. It supports AS3 and MXML.

the problem has been resolved,thank you very much.


Handle challengedfile download for new Mavat

In the iplan process, we download some files to extract data from them. Recently, we changed the website we scrap data from to the new Mavat website, and since then, we're unable to download files to the server to extract data from.
In the new Mavat, the plan instruction file, and generally all files, are blocked to download. Tried with our challenge file download, and it didn't work.
One approach is to handle it using puppeteer- to open a browser that enters the file URL and download it to the wanted location.
Once finished, we should return the fetchInstructionFIle condition to work. as currently it's blocked.
This is the code scope that handles that logic, you may search for it in the code
mavat/index.js
if(false){
				pageInstructions = await getPlanInstructionsNewMavat([...get(data, 'rsPlanDocs', []), ...get(data, 'rsPlanDocsAdd', [])] );

				if(fetchPlanInstructions){
				pageInstructions = await getPlanInstructionsNewMavat([...get(data, 'rsPlanDocs', []), ...get(data, 'rsPlanDocsAdd', [])] );
			}
			}

@GalGend doesnt reproduce, everything downloads great. do i need to cng something in env?

@GalGend we need the cookie but dont always get the set-cookie back
this seems to work but i dont know if thats a good solution
https://github.com/meirim-org/meirim/pull/682/files


Kbit_device_map import error. DPO

from trl import get_kbit_device_map
ImportError                               Traceback (most recent call last)
 in <cell line: 1>()
----> 1 from trl import get_kbit_device_map
ImportError: cannot import name 'get_kbit_device_map' from 'trl' (/usr/local/lib/python3.10/dist-packages/trl/init.py)
Also Tried from trl.trainer.utils
Still Got:
ImportError
Traceback (most recent call last)
----> 1 from trl.trainer.utils import get_kbit_device_map
ImportError: cannot import name 'get_kbit_device_map' from 'trl.trainer.utils' (/usr/local/lib/python3.10/dist-packages/trl/trainer/utils.py)

hi @sankydesai
Can you update TRL? pip install -U trl and let me know if you still face the issue

Nope. The issue is very simple. I went to colab, did pip install -U trl and then imported  from trl import get_kbit_device_map. It fails. Please check.

Hi @sankydesai
Sorry for the confusion, as https://github.com/huggingface/trl/pull/1176 is not included in the pypi release, you need to install TRL from source
pip install -U git+https://github.com/huggingface/trl

closing as we made a recent release


feat: Linkable search results

#380

Turned into draft as currently does not work, so not reviewable.
(Using search redirects to https://deploy-preview-3398--prisma2-docs.netlify.app/docs/docs/?query=test&page=1 which is a white page for me)

@janpio The link you mentioned above has 2 /docs strings. Try this instead: https://deploy-preview-3398--prisma2-docs.netlify.app/docs/?query=test&page=1

@janpio Can you please review the search behaviour again? The results wont be available since we do not index on branch PRs

Searching now changes the URL to something that could be linked to 👍
Opening that URL in a new tab also starts the search again 🥳
Clsoing the search windows does not change the URL back though
I also see no search results at all (which I am not sure should be the case - last I tested this I just saw the same results as on the live site but search generally worked)

@janpio Can you test again and let me know if its good to go?

Search does not work for me at all on https://docs-git-linkable-search-prisma.vercel.app/ unfortunately.
The header also looks broken.


JBPM-4625 Generation of JPA enabled models

- i18n added for the Persistence Descriptor Editor

@wmedvede just out of curiosity, what is the reason to create a PR and then self-merge it few minutes later? I am seeing PRs as way for others to review things, which does not happen this way (at least not before the changes land in the branch).
I know that PR is more visible (comparing to direct push), so there is higher chance someone notices the changes and looks at them. Maybe this is worth creating the PR.


🖼 Set img height for social icons.

#96

@RVA2869 could you make the file changes in a separate PR, please? This would make it easier to understand what happened based on the title of it.

@Ainali I was messing around with webp - changes are reverted

@RVA2869 thank you! 🌟


remove lombok dependency in dao module

What is the purpose of the pull request
this pr is for remove lombok dependency

Codecov Report

Merging #1735 into dev will increase coverage by 0.44%.
The diff coverage is 1.04%.

@@            Coverage Diff            @@
##              dev   #1735      +/-   ##
=========================================
+ Coverage   17.15%   17.6%   +0.44%
=========================================
  Files         285     285
  Lines       13900   14029     +129
  Branches     2267    2361      +94
=========================================
+ Hits         2385    2470      +85
- Misses      11240   11268      +28
- Partials      275     291      +16

Impacted Files
Coverage Δ

.../dolphinscheduler/api/service/ExecutorService.java
0% <ø> (ø)
:arrow_up:

...pache/dolphinscheduler/common/utils/DateUtils.java
88.04% <ø> (-6.15%)
:arrow_down:

...phinscheduler/api/service/DataAnalysisService.java
0% <ø> (ø)
:arrow_up:

.../org/apache/dolphinscheduler/dao/entity/Alert.java
8% <ø> (ø)
:arrow_up:

...rg/apache/dolphinscheduler/dao/entity/Project.java
0% <ø> (ø)
:arrow_up:

...dolphinscheduler/api/service/SchedulerService.java
0% <ø> (ø)
:arrow_up:

...he/dolphinscheduler/api/service/LoggerService.java
0% <ø> (ø)
:arrow_up:

...lphinscheduler/api/controller/LoginController.java
0% <ø> (ø)
:arrow_up:

...che/dolphinscheduler/api/service/UsersService.java
73.57% <ø> (ø)
:arrow_up:

...olphinscheduler/api/service/AlertGroupService.java
88.76% <ø> (ø)
:arrow_up:

... and 29 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57c33be...9e0049d. Read the comment docs.


tests.nixpkgs-check-by-name: Fix for aliases to packages in pkgs/by-name and better testing

Context
In https://github.com/NixOS/nixpkgs/pull/275539 I made an oversight which shows itself when you run the tool on the current Nixpkgs:
❯ nix-build -A tests.nixpkgs-check-by-name
/nix/store/k7wwnxbrb2r05g5pw7hxw71gg1gls1dr-nixpkgs-check-by-name
❯ result/bin/nixpkgs-check-by-name --base . .
pkgs.NSPlist: This attribute is defined using `_internalCallByNamePackageFile`, which is an internal function not intended for manual use.
[...]
pkgs.uclibc: This attribute is defined using `_internalCallByNamePackageFile`, which is an internal function not intended for manual use.
pkgs.win-virtio: This attribute is defined using `_internalCallByNamePackageFile`, which is an internal function not intended for manual use.
Validation failed, see above errors

See https://github.com/NixOS/nixpkgs/pull/281374 to prevent the broken tooling from being used on master, that should be merged fairly soon before the nixos-unstable channel updates in like a day.
The problem
The problem is that there's some detection logic that makes sure the internal _internalCallByNamePackageFile attribute isn't used by packages.
If such a top-level attribute that's not in pkgs/by-name was detected to use this, an error is thrown. However, the detection logic can't distinguish between aliases and non-aliases.
So it gives errors even for aliases to packages defined in pkgs/by-name, e.g.
https://github.com/NixOS/nixpkgs/blob/5df0c94c5ed9c4b9664d63047f2f3a7f93c0bb43/pkgs/top-level/aliases.nix#L1099
https://github.com/NixOS/nixpkgs/blob/5df0c94c5ed9c4b9664d63047f2f3a7f93c0bb43/pkgs/top-level/all-packages.nix#L28854
The solution
It would be non-trivial to improve the detection logic, so instead this PR just changes the tool to not give an error in these cases.
Furthermore, a passthru.tests is added, addressing https://github.com/NixOS/nixpkgs/issues/256789 and preventing future such problems. It should be triggered automatically after ofborg finishes evaluation of a PR with a tests.nixpkgs-check-by-name: ... commit.

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

@ofborg build tests.nixpkgs-check-by-name

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/ci-will-soon-enforce-pkgs-by-name-for-new-packages/38098/2


Compatibility with M1 macbook (arm64)

Describe the bug
I cannot build the tests on an M1 macbook. It seems to work fine on intel machine.
The issue is that if I start with a clean project I get No such module CombineSchedulers in IDE where the import CombineSchedulers is and when building my tests - under Compile Swift Source Files (arm64) I also get the error No such module CombineSchedulers.
If I build my app target first, the errors in IDE disappears. But then I proceed to building my tests and I get No such module MYMODULE under Compile Swift Source Files (arm64).
Expected behavior
Test target should build and run tests.
Environment

Xcode 12.3
Swift 5.0

BUMP

We had a similar issue reported to swift-composable-architecture here: https://github.com/pointfreeco/swift-composable-architecture/issues/344
It turned out to not be a problem with the library but rather an issue with the project depending on it. Can you reproduce your issue in a brand new project with no other dependencies? If so, and if you have a fix, we'd gladly accept a PR!

Ha, now that I think of it, swift-composable-architecture depends on combine-schedulers, and that folks have been able to run the sample apps just fine, so I'm inclined to think that this isn't a library problem here, as well. Would love to have information about solutions to this issue. @mbrandonw and I haven't upgraded to M1s yet so we don't yet have a way of trying to reproduce the problem.

Thanks @stephencelis
I've reviewed pointfreeco/swift-composable-architecture#344 and it can be the issue.
I've to set "excluded architectures": arm64 because of using firebase messaging framework.
And then targets which has that excluded are not able to run this framework. Adding "clean" utils target fix the issue.

@micHar Given the discussion above I'm going to close this out, but if you think it's still a problem, please let us know!

I will try that, thanks for letting me know

Just to mention.
No Architectures excluded Utils target is not the fix at the end.
When you will try to include this Utils inside your Carthage base target (App) you will get same error from Xcode.
I've end-up removing completely Carthage from the project and use 100% XCFrameworks & SPM.
Then everything is ok (after removing excluded architectures)


Enable Flux python interface

There is currently a issue with starting flux inside the Github CI, I created a separate repository to demonstrate it https://github.com/jan-janssen/flux-github-ci


internal error cloudflare

I followed the guide to allow the certbot to cloudflare with the creation of API but it gives me an error
certbot: error: unrecognized arguments: --dns-cloudflare-credentials /etc/letsencrypt/credentials/credentials-9

I had the same issue (using Proxmox container helper scripts) and the issue was that the plugin for cloudflare was misconfigured https://github.com/tteck/Proxmox/issues/3929
The solution was to manually reinstall the plugin with the command python3 -m pip install --no-cache-dir certbot-dns-cloudflare in the container. After that, the challenge went trough.

As of today, I was able to replicate the issue.  Thanks to @dullin comment, I was able to reinstall certbot and retrieve a CERT.


Single Game Fanduel MVP constraint

Is there any way to constrain the pool of players for the MVP position? Essentially, there's generally only 4-6 players you'd want to consider for the MVP slot, but the optimizer considers the entire pool. Any way to limit the MVP slot to a select number of players?

You can try to remove some mvp players from optimization:
my_mvps = ['Player 1', 'Player 2', 'Player 3']
for player in optimizer.players:
    if player.is_mvp and player.full_name not in my_mvps:
        optimizer.remove_player(player)

Thanks Dima! The only other aspect I'd probably want to control for is exposure rates for MVP and the entire pool, but this has really helped.

You can set exposure for each player just set max_exposure attribute of player object to desired value


Updating documentation for required permissions when using GH App auth

In my testing I found that a gh: Resource not accessible by integration error was encountered when the GH App permissions are set according to the README's current state.
After setting the repo issues permission to read-write for the GH app issues were moved to the target project as expected.

Cheers 🎉 Thank you for your work on this project!

Cheers 🎉 Thank you for your work on this project!

Thank you very much, I appreciate it.


Copy priority on the dynamite tool

The drilldown automation tool should copy the buildings priority from the original when placing a new dynomite.

Duplicate to https://github.com/ihsoft/TimberbornMods/issues/35


Use client.request for unique calls (clangd & texlab)

This PR only addresses clangd and texlab
It uses client.request inplace of vim.lsp.buf_request to call methods
that are outside the lspSpec such as clangd's "textDocument/switchSourceHeader"
This is done to avoid conflicts when using multiple LSPs over the same
buffer.

Ok, I think we good now.

lint failing, you can remove the local lsp var in texlab

This pull request has been mentioned on Neovim Discourse. There might be relevant details there:
https://neovim.discourse.group/t/switch-to-h-cpp-via-lsp/813/11


Added get_rolling_hashes and Hashing class

Added Hashing class (O(n) preprocess then O(1) hash of any sublist)
Added get_rolling_hashes

Not sure if we need the class.
Not sure if we need inverses.


Implement ProjectHomeView

This is a subtask of #35. The intent is to stub out the ProjectHomeView and associated template. Here is Chris' wireframe mock up of the ProjectHomeView:

This issue was moved to NCEAS/metacatui#824


Update license, CLA and DCO descriptions

Update the website text describing the project's license and CLA, and clarify the DCO process.
Signed-off-by: Steve Winslow<EMAIL_ADDRESS>

This is in connection with the comment from @danrbailey at https://github.com/AcademySoftwareFoundation/openvdb-website/pull/18#issuecomment-564166265

Bumping up to see if that can be merged @jmlait @kmuseth @Idclip


LNBits 0.11.2 + TPoS v0.5.1 - tips are not separated from the payment and are not sent to a separate wallet if specified in the terminal settings

Hello,
tips are not separated from the payment and are not sent to a separate wallet if specified in the terminal settings.
I just created a terminal, specified a separate wallet for payment, another for tips, specified tips and paid 10% tip. As a result, one payment was made and the entire amount went to the main wallet. Tips should go separately to the wallet that was specified when creating the terminal.

If confirmed, this sounds like a regression.

Can't reproduce the error:

Payment goes in the TPoS wallet, and then it send the tip to the tip wallet! Maybe it's an issue with listeners, if it doesn't "catch" the payment, it doesn't process the tip. A restart fixes it!

Clean TpoS with and without tips!

The behaviour of the tips, if i recall and i could be wrong, always worked like that!


Add Logentrus to README.md (hook for Logentries)

Have tried to keep Logentrus clean and well documented. Let me know if you have any questions or recommendations.

LGTM
cc @sirupsen


Add CSV as a dependency of Zipkin exporter

Ruby 3.3 issues deprecation warnings when libraries require CSV, as it will not be included in  Ruby 3.4.
Let's declare the dependency to silence the warnings.

Fixed by #1560.

Closed by #1560


Clarification please

We decided to use this library because it is most recommended in the field of RFC 5545 pattern handling.
Our use case is a calendar with recurring events. We want to know exactly which events are due on a certain day.
I studied the examples out there and implemented a test code with one recurring event. The function GetOccurrences only returns unexpected results so far. But perhaps this is because of my nonproper use. To check that, I have the following questions:

The application must work independent of its location. That means, we use only UTC. Does ical.net work correctly with UTC when I do not provide any time zone?
When I add another event at the exact same time and recurrence as a first one but with a different UID, nothing in the GetOccurrences result changes. Can I not expect the library to see an event as different from another when only the UID is different?

Thanks for considering my request.

I doubt I can help much, but I was having the same issue. Then I set the description field and occurrences started appearing. I now just set the description field to a single space/white-space character (can not be empty or null) and things started working. Give it a go if you haven't already?
If that doesn't work then I'm not sure what else to suggest... I'm sure more information might be useful for others too (i.e. sample code etc) and they might be able to offer more help than I can...
Just to confirm I'm working exclusively with UTC times and every one of my unique identifier (UID) fields are different from each other.

@ruwentos in regards to 1. we are using this exclusively with UTC date/times and it has been working as expected and we rely on that quite a bit. So I would not be worried in that regards.


A couple of UI enhancement requests (showing Due Date on board, Description edit box)

Thanks for making this extension, it's been super helpful in my tools limited working environment (where we're allowed to use VS Code and that's really all)... :beers:
A couple of UI enhancements for your consideration:

In board view it would be more helpful to show Due Date especially if the Due Date is near (maybe make the date yellow) or has past (make the date red)
In card view, it would be helpful if the edit description box would expand to a size that it can hold all of it's contents during editing, scrolling inside an edit box is rough

Anyway, thanks again.

I appreciate the suggestions, though I will likely not focus on new visual features in the near future. I also want to avoid adding too many options. I'm considering if it would make sense to allow custom html / javascript to allow arbitrary layout / visual changes.
And while that's not exactly what you're asking for, you should be able to make the description box larger with custom css. For that follow the instructions in https://github.com/samgiz/vscode-kanbn/blob/main/docs/styles.md and use the kanbn-task-editor-field-textarea class.


Bug: NextJs dynamic routing not working

Describe the bug and the steps to reproduce it
the app/ folder in the repo drives the routing. nextjs documentation says that adding a file [slug].tsx with the brackets in the folder will add a dynamic route -- the equivalent of having a token/:address route in other kinds of routers. I currently have a basic page at file token/[slug].tsx
import { useRouter } from 'next/router'

export default function Page() {
  const router = useRouter()
  console.log('router', router);
  return (<p>{router.query.slug}</p>);
}

but when i go to the route /token/123, i get a 404, suggesting the routing isn't working. for reference, the route /token is displaying properly, via the index.tsx file in my token/ folder
What's the expected behavior?
the app is expected to route to token/123 with 123 being passed as a prop to the component
What version of the libraries are you using?
I'm not sure exactly where to find the BOAT version, but I cloned the repo about one week ago.
next 13.5.6
react ^18

@michaelcohen716 not a bug, you just need to put the [slug] part as a subfolder vs file. See example: https://github.com/coinbase/build-onchain-apps/compare/wes/slug-example

works, thank you


EIA923 Early Release!

[x] metadata
[x] runs through extract
[x] runs through transform
[x] runs through the etl w/ just eia (w/o PK constraints)
[x] add unmapped plant/utility ids (if any)
[ ] runs through the full etl
[ ] passes data validation tests

If we're going to integrate the Early Release data into the DB more generally (which seems great, if it's easy to integrate) would it make sense to keep the ER / not-ER flag column, and add it to the DB?  Would it be clear which records come from the ER data, or does that get too muddy in the normalization process? Could we just add a flag across the board for the monthly and annual data from the EIA-923 and EIA-860 ER year?

@zaneselvans  this is basically what i am suggestions in #1799

Well! the data runs through the eia-only etl just fine rn.
I ran the ./devtools/ferc1-eia-glue/find_unmapped_plants_utils.py script and the only unmapped anything was some plants that had NO INFO. I went back and checked... and these plants are very very inconsistent in the early release data across various tabs. So the harvesting process is doing a good job here nulling out  this data. I've temporarily added these plant id's into the pudl id mapping spreadsheet... but we should go back to these once we add the 860 early release.

Does it make sense to merge this PR without the EIA 860 Early Release data? We have the generators table from 2021 thanks to the eia860m, but we won't have any of the other 2021 data -- plants, utilities, new BGAs, etc.  Do you think the EIA-860ER will be similarly straightforward?

These are the exact questions I asked our team. I am already working on the eia860 er and i hope it'll be similarly straightforward. I don't think there is a real harm in integrating the new metadata and slight code updates. I would suggest not actually changing the datasource working partitions until they are both integrated, but merging this pr.

So use the new data archives / DOIs which include all the old data (however it's changed...) plus the 2021 eia923er data, but not actually allow that data to be processed until we get the eia860er integrated as well.
The new expected numbers of rows only apply if the 2021 eia923er data is processed though right?

(I merged in dev to fix the dependency issue that was causing the push to fail CI)

consolidated into #1834


DRILL-4530: Optimize partition pruning with metadata caching for the …

…single partition case.

Enhance PruneScanRule to detect single partitions based on referenced dirs in the filter.
Keep a new status of EXPANDED_PARTIAL for FileSelection.
Create separate .directories metadata file to prune directories first before files.
Introduce cacheFileRoot attribute to keep track of the parent directory of the cache file after partition pruning.

The metadata cache file changes look good. (Longer term, we need to clean this up and maybe split metadata into smaller files and/or consider providing alternative metadata cache implementations, but that is another issue.)

Other than prior comments, the pruning logic looks good to me.

@jinfengni ,  thanks for the review.  I have addressed your review comments.  Pls take another look.

+1
LGTM.


fix(balancer) ensure existing healthchecker is stopped when processing target event

Summary
Ensures that existing healthchecks are stopped when processing CRUD events on targets.
Full changelog

Ensure stop_healthchecker is called for existing balancer prior to create_balancer. Follows pattern from do_upstream_event

Notes:

Haven't added tests for this. Figured I would try copy the existing test from do_upstream_event but it doesn't look like this is tested at present? (Was looking at https://github.com/Kong/kong/blob/2.4.1/spec/01-unit/09-balancer_spec.lua so perhaps Iäm in the wrong place?)
Opened PR against master. But it seems like the branching strategy in this repo has changed somewhat since I last submitted a PR. Feel free to let me know if you want this rebased to a (release?) branch.

Issues resolved
Fix #7406
Fix #7407

Hi @onematchfox, thanks for your submission! This change is correct but needs a sister PR (Kong/lua-resty-healthcheck#74) to fix the issue. We'll merge this after releasing a new version of lua-resty-healthcheck and bumping it in Kong.

Hi @onematchfox, thanks for your submission! This change is correct but needs a sister PR (Kong/lua-resty-healthcheck#74) to fix the issue. We'll merge this after releasing a new version of lua-resty-healthcheck and bumping it in Kong.

:+1:
Was aware of the additional change that was required and had already opened a PR for this (https://github.com/Kong/lua-resty-healthcheck/pull/71). I hadn't removed worker_events.register_weak(self.ev_callback, self.EVENT_SOURCE) not sure I follow why you'd want to remove the registration of the callback as well but respect your expertise on the matter :smile:


Feat migrate to lilconfig

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

[x] Appropriate label or labels have been added
[x] ✅ Tests for the changes have been added (for type:fix / type:feat)
[x] 📝 Docs have been reviewed, added or updated if needed (for type:fix / type:feat)
[x] 🏁 Issue number has been linked if this PR closes one

Description
🏁 Closes: #249

fix(*): 🐛 wrap correctly when they are quotes in the message
feat(*): ✨ replace cosmiconfig w/ lilconfig & yaml to reduce deps

Please describe your pull request.
Breaking Change?

[ ] 💥 Yes
[x] No

❤ Thank you for contributing!

Codecov Report

Merging #253 (d8807e5) into master (d03d431) will increase coverage by 0.27%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #253      +/-   ##
==========================================
+ Coverage   96.93%   97.21%   +0.27%
==========================================
  Files          31       31
  Lines         392      395       +3
  Branches       66       66
==========================================
+ Hits          380      384       +4
+ Misses         10        9       -1
  Partials        2        2

Impacted Files
Coverage Δ

src/config/helpers/loadConfig.ts
90.00% <80.00%> (+18.57%)
:arrow_up:

src/utils/format-message/formatGitMessage.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d03d431...d8807e5. Read the comment docs.


(renamemaster) fix broken pin to start fixes #1473

Description
Package failed to install.
Motivation and Context
Package failed to install due to outdated extra optional steps.
Cleaned up broken steps. This fixes #1473
How Has this Been Tested?
Manually ran every line in POSH using chocolateypowershell package.
Screenshot (if appropriate, usually isn't needed):
Types of changes

[x] 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 change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[ ] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

I spun up a VM using chocolateypowershell for manual script testing. Would someone be so kind as to kick off building a patch version? Also the version in the moderation queue is wrongly marked as version 5.13 while the latest release is 3.15. I'm not sure what is going on here, but I didn't touch that. :sweat_smile: Can that version be retracted?


Create primechecker.py

What does the program do ?

In what programming language it is written?
python

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Package can't be use with StrongNamed project

Nuget package for Validar can't be used in StrongNamed Project because it must be signed.
Please add strong name in next version

Send me a PR

Sandra.SimpleValidator isn't StrongNamed, I have make PR for this, if it's accepted I finished PR here :)


Multiple filtering keywords in the Debug Console window

When we start a project in debug mode, we will have a corresponding Debug Console window. If I want to filter the output, I can enter some keywords in the filter box.

Now I want to have a different filtering with different keywords, of course, I have to change the filtering keywords.

However, I want to monitor the output with both sets of different keywords simultaneously. I don't want to switch back and forth the keywords.
My question now is:
1/ Can I have multiple debug console windows for the same debug session?
2/ Is there a way to have multiple filtering keywords? e.g "Registation|Staled", it does not work. Maybe I don't know the correct way.

As this seems to OR condition so is this possible to filter as above? looking for possible answers.

You can have multiple comma-separated patterns. I believe the placeholder text is trying to communicate this


Not working in emulator

Hi, great job on this homebrew! Reading online, I found it should work with 2 GBAs, so I loaded this up in mGBA (which is the only emulator I know of that supports GBA-to-GBA link) but when I replace the ROM and press A to proceed the homebrew always fails to read data. Is it not possible to get this working?

Hello! If you wanted to check the homebrew working between Gen 1/2 and 3 on emulators, then it won't work (I know of no emulator which can allow that). It should be fine for Gen 3 to Gen 3.
Though you are correct that mgba does not properly load save data. It's an issue in the emulator itself though, which I reported already. You can read more here: https://github.com/mgba-emu/mgba/issues/2864

Thanks for the quick reply! I thought mGBA would allow GBC-to-GBA, I must be mistaken :/ either way, since I can't get past the "Error reading the data!" error, in my case it won't work at all. I'll be waiting for mGBA to make a fix, although it's pointless if link to GB mode window doesn't work.

I mean... You are free to try it... Though it's something related to the emulators themselves, not to the homebrew. Sorry, and best of luck!


replaced hardcoded versions with attributes

Links in adoption guide with hardcoded versions: https://docs.google.com/document/d/1iSKulSxh_eUah6KoqtZzy5irqpQddho55atK35g8wpE/edit#heading=h.ey7ucr7zq350

recheck


Updated README.md

Updated README.md to add section for installing from source and from the Arch Linux AUR.

Thanks for adding it to the AUR!


Fix incorrect return value in LoadJobConfiguration

This corrects a minor issue in the BigQuery LoadJobConfiguration where useAvroLogicalTypes isn't adhering to the documented return type.

@Firehed, release can be found here: https://github.com/googleapis/google-cloud-php/releases/tag/v0.112.0. Thanks again for the PR.

That was fast! Thanks for the quick review :)


Error in eventalign for direct RNA sequecning

Hi,
I had tried it on my data from direct RNA sequencing.
The command for eventalign is working now.
However it is working when I am only using 1 gene in the reference sequence.
When I tried to use 2 or more genes in my reference sequence, this error occurs:
contig position reference_kmer read_index strand  event_index event_level_mean  event_stdv  event_length  model_kmer    model_mean model_stdv   standardized_level
[fai_fetch_seq] The sequence "Tetra" not found
nanopolish: src/alignment/nanopolish_eventalign.cpp:198: std::__cxx11::string get_reference_region_ts(const faidx_t*, const char*, int, int, int*): Assertion `cref_seq != __null' failed.
Aborted (core dumped)
I had tried mapping with graphmap and bwa, both resulted in the same error.

Hi,
Can you provide the full commands you used to a) map the reads to the reference and b) run eventalign?
Jared

Hi,
I had first extracted my reads using nanopolish extract.
Followed by:
sed -i 's/U/T/g' try.fastq
graphmap align -r cp_nanopore.fa -d try.fastq -o try.sam --double-index
samtools view -bT cp_nanopore.fa try.sam > try.bam
samtools sort try.bam try.sort
samtools index try.sort.bam
nanopolish eventalign -r try.fastq -b try.sort.bam -g cp_nanopore.fa --print-read-names --summary=try.try > event
I had used the same commands for the 2 reference sequence, containing 1 gene and 2 or more genes.
Only the reference sequence with 2 or more genes had the error.
I would also like to check, is the reference_kmer or model_kmer corrected for direct RNA sequencing?
As RNA is sequenced from the 3' to 5', which is different from how DNA is being sequenced.

Can you provide the fasta headers of the cp_nanopore.fa file and one alignment record from the SAM file for the contig Tetra?
reference_kmer and model_kmer are both in the reference (5'->3') orientation, even for direct RNA. I think it is simpler to be consistent but I'm not convinced it is the right thing to do.

Hi,
These are the information you require:
fasta header:

Tetra
Control

fcf48413-5d17-4ff7-9344-9fd1a99403de_Basecall_1D_template:1D_000:template       0       Tetra   300     40      8M1I8M2I5M1I4M1D21M1I28M1D5M1I20M1D5M1I8M3I1M1S *       0       0       CCCCTTTTCAAGATAAACTGACG
In the direct RNA, it is sequenced from the 3' to 5'.
And in fast5 file, under Events, the sequences are flipped, when compare to basecalled sequence by albacore.
Did nanopore state that the orientation for the model_kmer is  5'->3'? or the order that it is being sequnced?
I would think that it would be better if we can just filp the model_kmer?
Example for the first few bases from the Events:
Kmer: "CGGTC", "GTCCT","TCCTA","CCTAC","TACCA","ACCAT"
Move:       1  ,          2     ,       1     ,     1      ,      2      ,      1
Form: CGGTCCTACCAT
From the fastq:
UACCAUCCUGGC
If you need, I can upload some fast5 files for the control RNA provided by nanopore.

Can you delete cp_nanopore.fa.fai and try again? I think your fasta index for the reference is out of date.
The model kmers ONT provides are 3'->5' but I flipped them before loading into nanopolish and reverse the events. This was easier to implement rather than reversing the reference sequences.

On Oct 7, 2017, at 11:47 AM, awjga<EMAIL_ADDRESS>wrote:
Hi,
These are the information you require:
fasta header:
Tetra
Control
fcf48413-5d17-4ff7-9344-9fd1a99403de_Basecall_1D_template:1D_000:template 0 Tetra 300 40 8M1I8M2I5M1I4M1D21M1I28M1D5M1I20M1D5M1I8M3I1M1S * 0 0 CCCCTTTTCAAGATAAACTGACG
In the direct RNA, it is sequenced from the 3' to 5'.
And in fast5 file, under Events, the sequences are flipped, when compare to basecalled sequence by albacore.
Did nanopore state that the orientation for the model_kmer is 5'->3'? or the order that it is being sequnced?
I would think that it would be better if we can just filp the model_kmer?
Example for the first few bases from the Events:
Kmer: "CGGTC", "GTCCT","TCCTA","CCTAC","TACCA","ACCAT"
Move: 1 , 2 , 1 , 1 , 2 , 1
Form: CGGTCCTACCAT
From the fastq:
UACCAUCCUGGC
If you need, I can upload some fast5 files for the control RNA provided by nanopore.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I had deleted cp_nanopore.fa.fai, and remapped my reads, but I am still having the same error.
That's good to know, flipping of the model kmers and the event seems to be reasonable.
So I guess the event index is also flipped?

Can you run cat cp_nanopore.fa.fai and paste the results?

Here is the results:

Tetra
GGCTCTCTAAATAGCAATATTTACCTTTGGAGGGAAAAGTTATCAGGCATGCACCTGGTAGCTAGTCTTTAAACCAATAGATTGCATCGGTTTAAAAGGCAAGACCGTCAAATTGCGGGAAAGGGGTCAACAGCCGTTCAGTACCAAGTCTCAGGGGAAACTTTGAGATGGCCTTGCAAAGGGTATGGTAATAAGCTGACGGACATGGTCCTAACCACGCAGCCAAGTCCTAAGTCAACAGATCTTCTGTTGATATGGATGCAGTTCACAGACTAAATGTCGGTCGGGGAAGATGTATTCTTCTCATAAGATATAGTCGGACCTCTCCTTAATGGGAGCTAGCGGATGAAGTGATGCAACACTGGAGCCGCTGGGAACTAATTTGTATGCGAAAGTATATTGATTAGTTTTGGAGTACTCG
Control
ATGGCTGTCTCTAAAGTTTACGCTAGATCCGTCTACGACTCCCGTGGTAACCCAACCGTCGAAGTCGAATTAACCACCGAAAAGGGTGTTTTCAGATCCATTGTTCCATCTGGTGCCTCCACCGGTGTCCACGAAGCTTTGGAAATGAGAGATGAAGACAAATCCAAGTGGATGGGTAAGGGTGTTATGAACGCTGTCAACAACGTCAACAACGTCATTGCTGCTGCTTTCGTCAAGGCCAACCTAGATGTTAAGGACCAAAAGGCCGTCGATGACTTCTTGTTGTCTTTGGATGGTACCGCCAACAAGTCCAAGTTGGGTGCTAACGCTATCTTGGGTGTCTCCATGGCCGCTGCTAGAGCCGCTGCTGCTGAAAAGAACGTCCCATTGTACCAACATTTGGCTGACTTGTCTAAGTCCAAGACCTCTCCATACGTTTTGCCAGTTCCATTCTTGAACGTTTTGAACGGTGGTTCCCACGCTGGTGGTGCTTTGGCTTTGCAAGAATTCATGATTGCTCCAACTGGTGCTAAGACCTTCGCTGAAGCCATGAGAATTGGTTCCGAAGTTTACCACAACTTGAAGTCTTTGACCAAGAAGAGATACGGTGCTTCTGCCGGTAACGTCGGTGACGAAGGTGGTGTTGCTCCAAACATTCAAACCGCTGAAGAAGCTTTGGACTTGATTGTTGACGCTATCAAGGCTGCTGGTCACGACGGTAAGGTCAAGATCGGTTTGGACTGTGCTTCCTCTGAATTCTTCAAGGACGGTAAGTACGACTTGGACTTCAAGAACCCAGAATCTGACAAATCCAAGTGGTTGACTGGTGTCGAATTAGCTGACATGTACCACTCCTTGATGAAGAGATACCCAATTGTCTCCATCGAAGATCCATTTGCTGAAGATGACTGGGAAGCTTGGTCTCACTTCTTCAAGACCGCTGGTATCCAAATTGTTGCTGATGACTTGACTGTCACCAACCCAGCTAGAATTGCTACCGCCATCGAAAAGAAGGCTGCTGACGCTTTGTTGTTGAAGGTTAACCAAATCGGTACCTTGTCTGAATCCATCAAGGCTGCTCAAGACTCTTTCGCTGCCAACTGGGGTGTTATGGTTTCCCACAGATCTGGTGAAACTGAAGACACTTTCATTGCTGACTTGGTTGTCGGTTTGAGAACTGGTCAAATCAAGACTGGTGCTCCAGCTAGATCCGAAAGATTGGCTAAGTTGAACCAATTGTTGAGAATCGAAGAAGAATTGGGTGACAAGGCTGTCTACGCCGGTGAAAACTTCCACCACGGTGACAAGTTGTAA

Hi,
Thanks! you had found the problem!
Not sure why cp_nanopore.fa.fa is empty.
I think that is causing the issues.

Great


Azure Advisor recommendations / Configure low usage VM recommendation / new lookback period and CPU trigger

Advisor's recommendation for Low Usage VMs has evolved a lot. It now has a new trigger option of 100% utilization. When the trigger is set to 100% it will work as multiple rules and Advisor will present recommendations of VMs with less than 5%, 10%, 15% and 20% of CPU utilization.
It also supports different lookback periods of up to 90 days.
None of those changes are on this doc yet. The images on this doc are quite old as well.
I'm happy to make those changes and create a PR.

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

ID: 38f3c39c-4a81-87cd-2741-5f8631b46a4d
Version Independent ID: 46fe35e1-dd95-ef4a-0983-a14d6b94db39
Content: Get started with Azure Advisor - Azure Advisor
Content Source: articles/advisor/advisor-get-started.md
Service: advisor
GitHub Login: @ikhapova
Microsoft Alias: ikhapova

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

I have added this to ADO as an item to work on (https://msazure.visualstudio.com/One/_workitems/edit/25840978). Please let me know if I should start. This seems like a pretty straight-forward update.

#reassign:jenniferlee-ms

#reassign:jenniferlee-ms

Thanks for your dedication to our documentation. This issue has been addressed in the documentation. #please-close


Change default prefetch to Worker-Fetch

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests

Description
Link prefetch has been shown to only work on Chrome, other browsers ignore it. Worker-fetch is a better default as it works across all browsers.
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

While i agree there is a problem with the current default, this mode has problem too, and shows worse performance than the native link... I wonder if we can create something better that uses the best of chrome while supported in other browsers..
I dont like the idea of choosing the lowest common denominator approach. If Safari/firefox is bad, it should not affect chrome users.

I have no doubt there's a better solution here.
For the time being, I think using the worker fetch implementation is the lesser of two evils. I'm just thinking of the Firefox/Safari user who deploys a starter and gets no prefetching at all.

Closed with https://github.com/BuilderIO/qwik/pull/935

I wish there was a polyfill-like solution that helps other browsers actually prefetch when give the standard, declarative hints.


安装依赖出错

安装依赖...
安装依赖需要时间，请耐心等待!
sh: line 40: 当前node版本(如果没有node，请自行安装): : command not found
v18.15.0
sh: line 44: 当前npm版本(如果没有npm，请自行安装): : command not found
9.5.0
sh: line 48: cd: /ql: No such file or directory
Nothing to stop. No server is running for the store at /root/.local/share/pnpm/s                                                                                tore/v3
 ERR_PNPM_NO_GLOBAL_BIN_DIR  Unable to find the global bin directory
Run "pnpm setup" to create it automatically, or set the global-bin-dir setting,                                                                                or the PNPM_HOME env variable. The global bin directory should be in the PATH.
 ERR_PNPM_NO_PKG_MANIFEST  No package.json found in /root/.local/share/pnpm/glob                                                                                al/5
npm ERR! code EBADENGINE
npm ERR! engine Unsupported engine
npm ERR! engine Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Required: {"node":"^18.17.0 || >=20.5.0"}
npm ERR! notsup Actual:   {"npm":"9.5.0","node":"v18.15.0"}
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2023-09-26T11_18_37_771Z-debug-0.log
 ERR_PNPM_NO_GLOBAL_BIN_DIR  Unable to find the global bin directory
Run "pnpm setup" to create it automatically, or set the global-bin-dir setting, or the PNPM_HOME env variable. The global bin directory should be in the PATH.
sh: line 57: pip3: command not found
sh: line 59: cd: /ql/scripts/: No such file or directory
sh: line 60: cd: /ql: No such file or directory
sh: line 61: apk: command not found
sh: line 62: cd: /ql: No such file or directory
sh: line 63: apk: command not found
依赖安装完毕...建议重启 Docker
有任何问题，请在此仓库提交Issue： https://github.com/FlechazoPh/QLDependency

我也是这个问题