Enable model serving metrics upstream

Closes: #946
Description

Follow up of #916
Enable model serving upstream
How Has This Been Tested?

Tested both in downstream clusters and upstream clusters.
For odh

Follow the instructions on https://github.com/opendatahub-io/odh-manifests/pull/732 to deploy latest model and parametrize the deployment
Add to the odhDashboardConfig the value of the namespace where the model is deployed in modelMetricsNamespace
Test that you can deploy a model and access the metrics
Now remove the value in modelMetricsNamespace
Check that you cannot access  the model visualization (changes need to propagate due to the watcher)

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

So i have a few takeaways:

I think that we should check somehow if the operator is installed @andrewballantyne i think you mentioned talking with landon that this was way trickier than just looking at the kdfef right)
If the monitoring operator is not installed, hide this feature behind a feature flag
If its upstream and it’s installed, get the namespace and pass it
If its downstream, we still have the issue with harcoded values, we don’t have a way to parametrize the redhat-ods-monitoring namespace, but I think that this is not a priority right now

Curious -- why not just another feature flag in the ODH DashboardConfig to hide the UI. The current draft PR look like it will help the API point the right direction but does nothing to enable/disable the feature.

LGTM -- waiting on @DaoDaoNoCode to review
/assign @DaoDaoNoCode


Update docs to show how to find available storageclass

Done.

[ ] Postgres
[ ] Elasticsearch
[ ] MongoDB
[ ] MySQL
[ ] Redis


⚠ Turnos 24x7 has degraded performance

In 13200b8, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 844 ms

Resolved: Turnos 24x7 performance has improved in 7b92c7e.


.net6 and vs2022 fixes as per #25341

Fixes #25341

@Rick-Anderson  - FYI

@codemillmatt much appreciated. It should be live in 20 minutes.


[Feature Request] - Option to Keep Emoji Toolbar Open After Choosing Emoji

Awesome plugin! It works much better than the Windows emoji picker. These are two ideas I believe would make your emoji workflow faster and thus make the plugin even better:

It would be great if you could add an option in the settings where it will not close the toolbar after you pick an emoji. (This is useful when picking multiple emojis at the same time.)
Another great idea is to have the option to save the last search result, so if you want to open it again, it is exactly where you closed it.

Thank you for the recommendations. I'll be sure to keep these in mind next time I iterate on the plugin.


[Volunteer] Add the PPMC members and committers in the contributor page

Add the members list using the markdown format to https://shenyu.apache.org/community/contributors
The list contains:
Mentors
id	githubUsername	public name
atri	atris	Atri Sharma
zhangduo	Apache9	Duo Zhang
jincheng	sunjincheng121	Jincheng Sun
jmclean	justinmclean, jmclean	Justin Mclean
djkevincr	djkevincr	Kevin Ratnasekera
ningjiang	WillemJiang	Willem Ning Jiang

PPMC (27)
(the listing below excludes mentors)

id	githubUsername	public name
	asxing	asxing	Asxing
	qicz	qicz	Congqi Zhu
	hex12fd16b	0x12FD16B	David Liu
	nuoyan	nuo-promise	JianMing Ding
	kimmking	kimmking	Kimm King
	jooks	JooKS-me	Kunshuai Zhu
	sabersola	SaberSola	Lei Zhang
	midnight2104	midnight2104	Liang Liu
	dengliming	dengliming	Liming Deng
	zjntgao		Meng Gao
	kevinclair	KevinClair	MingJie Song
	nicholasjiang	SteNicholas	Nicholas Jiang
	tydhot	tydhot	Tang Yudong
	fightingting	fightingting	TingTing Wang
	lw1243925457	lw1243925457	Wei Liu
	ccloomi	ccloomi	Xianjun Chen
	xiaoyu	yu199195	Xiao Yu
	huangxfchn	huangxfchn	XiaoFeng Huang
	yui	HessTina-YuI	Yi Liu
	zhangyonglun	tuohai666, zhangyonglun	Yonglun Zhang
	fengzhenbing	fengzhenbing	Zhenbing Feng

Committers (34)
(the listing excludes PPMC members above)

id	githubUsername	public name
	haiboduan	haibo-duan	Haibo Duan
	likeguo	li-keguo	Keguo Li
	impactcn	impactCn	SiYing Zheng
	dayu	daiwenyu	WenYu Dai
	luoxiaolong	lxl910128	XiaoLong Luo
	alextang	ttttangzhen	Zhen Tang
	zhangzicheng	dragon-zhang	ZiCheng Zhang

I am interested, I can try

OK. Feel free to ask me if you have any questions.


Small improvements to walkthrough and defaults

Description

Sorts lists
Improves walkthrough
Filters out invalid characters for the application name guess

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

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

Filters out invalid characters for the application name guess

love it!


Upgrade Django

After DRF was upgraded, we still need to update Django itself. Initial research was provided by Trevor in issue #28 and the remaining items from there include:

There are new TestCase enhancements to speed them up by using setUpTestData() instead of setUp(). Code changes needed but existing code will continue to work.
Check use of Model._meta in case there are deprecated calls.
GeoJSON native queries in database queries postgis extension is automatically created by migrate so a postgis template database is no longer needed.
Look into data migrations
manage.py tests --parallel
Added the json() method to test client responses to give access to the response body as JSON.
Update custom commands to use the new argparse system.

Things to review:

Use of check --deploy on production settings
UUIDField and DurationField for storing UUIDs
Investigate using setUpTestData for class wide data initialization. May be faster than setUp for TestCase level data initialization.
Use of data migrations instead of fixtures

Deprecation Compliance:

Address URL patterns change first deprecated in 1.8 (https://docs.djangoproject.com/en/1.10/releases/1.8/#django-conf-urls-patterns)
Eliminate string reference to callable view in favor of actual callable
Update MIDDLEWARE_CLASSES to MIDDLEWARE in settings.py file. (https://docs.djangoproject.com/en/1.10/topics/http/middleware/#upgrading-middleware)
Update software dependencies to indicate PostGres 9.2 as a minimum supported version in line with Django 1.10 requirement.

Issues discovered:

'django.utils.log.NullHandler' was an alias, replaced with 'logging.NullHandler'
Updating to use JSONField is problematic as we as using the PostGres 9.3 JSON string field. If we use Django native PostGres support we will enforce a PostGres 9.4 upgrade and the JSONB field type. djorm-ext-pgjson does not support Django 1.10, so we will need to switch to a supported library (django-pgjson is one such that does not enforce an upgrade to JSONB type field)
Remove all use of url patterns in favor of simple url tuples


BufferedStore Configuration

I have tried to follow this example:  http://examples.sencha.com/extjs/6.2.0/examples/classic/grid/filtered-buffered-store.js
ExtJSComponent.js throws errors:
"Could not create Ext JS component with config"
&
"Uncaught TypeError: Cannot read property 'length' of undefined"
  this.store = Ext.create('Ext.data.BufferedStore', {
      id: 'store',
      storeId: 'store',
      model: 'extcomponents.model.mydata',
      pageSize: 21,
      leadingBufferZone: 300,
      proxy: {
          type: 'ajax',
          url: myurl,
          reader: {
              type: 'json',
              rootProperty: 'ROWS',
              totalProperty: 'TOTALCOUNT'
          },
          simpleSortMode: true
      },
      listeners: {
        load: function(){
            console.log('api returned')
        }
      },
      autoLoad: true
  })

If I changed BufferedStore to Store... works fine (though no buffer).

Seems to be working for me - what does the response from the server look like?

The error comes before the network call ever happens.  I'm not sure how to debug.    Any chance restarting your webpack watch task fixes it?  (npm start if you're using one of the boilerplates).  Failing that, if you share your project on GitHub I'd be happy to take a look.

I've got a bunch of stuff in here, so I'll first take a fresh copy of the boilerplate and work from it adding just what is need, and if I reproduce the error, I'll share it out.  thx

Ok, so I've narrowed it down to failing if my store is a BufferedStore (.Store works fine) and not using the classic toolkit (which I was not).  If I switch to modern, I get the above errors.
To be exact, if I comment out this line in the webpack.config.dev.js file and rebuild:
toolkit: 'classic',
or change it to:
toolkit: 'modern',

Apparently it is a bug:
https://www.sencha.com/forum/showthread.php?335348-Modern-toolkit-Grid-with-BufferedStore


[Feature]: Display a lesson on a subpage for a specific topic

As a user I want to see different lessons when I click on a specific subtopic (like "data type - string") and I want to see the lesson, when clicking on it.
Acceptance Criteria:

[ ] When the user clicks on the subtopic all the lessons under this subtopic are displayed on the subpage
[ ] When clicking on a lesson a concrete Lesson with a description (What to do in this Task Type), a question (additional information, optional), an interaction part (e.g. Multiple Choice), a field for the status text (Success or Failure) and a Submit-Button is shown
[ ] When clicking the Submit Button the Input from the User is validated and the status text shows the result

Structure of Lessonpage:

Technical Information:

[ ] There should be a class that stores the content for each lesson (eg. for a multiple choice lesson, the content includes: the question, the options, the right answers).
[ ] When the user clicks on the subtopic, lessons under this subtopic are created based on a content class and displayed on the subpage.


Test failed: test/integration/test_seq_copy_int.py

It failed since commit 5144d25 (Update to MXNet 1.3.0)
test/integration/test_seq_copy_int.py::test_seq_copy[--encoder rnn --decoder rnn --num-layers 3:2 --rnn-cell-type lstm --rnn-num-hidden 8 --rnn-residual-connections --num-embed 8 --rnn-attention-type coverage --rnn-attention-num-hidden 8 --weight-tying --rnn-attention-use-prev-word --rnn-context-gating --layer-normalization --batch-size 2 --loss cross-entropy --label-smoothing 0.1 --loss-normalization-type batch --optimized-metric perplexity --max-updates 2 --checkpoint-frequency 2 --optimizer adam --initial-learning-rate 0.01 --rnn-dropout-inputs 0.5:0.1 --rnn-dropout-states 0.5:0.1 --embed-dropout 0.1 --rnn-decoder-hidden-dropout 0.01 --rnn-decoder-state-init avg --rnn-encoder-reverse-input --rnn-dropout-recurrent 0.1:0.0 --rnn-h2h-init orthogonal_stacked --batch-type sentence --decode-and-evaluate 0 --learning-rate-decay-param-reset --weight-normalization --source-factors-num-embed 5---beam-size 2 --beam-search-stop first-False-True-True] FAILED

...
>               assert all(a == b for a, b in zip(lines, lines_equiv))
E               AssertionError

test/common.py:532: AssertionError

Thanks for reporting! Mhh that is strange. On Travis and locally the tests seem to be running fine, see. I wonder how we could reproduce this.

Right. I also don't know which information I can provide is useful.
I try to give as much configuration details as possible.
Sockeye is installed via virtualenv and with the following steps:
git clone https://github.com/awslabs/sockeye.git
cd sockeye
pip install -e . --no-deps
pip install -r requirements/requirements.gpu-cu90.txt
pip install -r requirements/requirements.dev.txt

I use Python3/3.6.4, cuda/9.0.176, cudnn/v7.2.1
The AssertionError came from producing an unexpected <unk>.

Thanks for looking into this! We do use --use-cpu during these tests, so they should indeed use the CPU. I will try see if I can reproduce this running on different hardware.

Thanks! BTW, it doesn't mater which type of GPUs are used. It failed on two gtxtitanx cards, or two gtxtitanxp cards, or two gtx1080ti cards, in our cluster.

Closing. Feel free to reopen if this issue surfaces again.


Update Frontend to work with the new RESTful Api

SInce things respond with statuses instead of error id and things. we need to update the front end to prepare itself for this!

this issue was created to stay current with #38

As stated in #38, I did this as part of that very pull.

@julianpoy When I tested, if I tried to make a duplicate user, I got the status code back, but the frontend was not ready for the status code, and gave no toast of the issue. Are you sure you adressed this?

I created #40 for this specific bug. All other routes have been updated. It seems I missed this one.

Okay cool thank you!


Fix start script problems - Issue #27

Here's the fix if it's decided that Issue #27 is something that needs doing.

Thank you @True14 , for finding and fixing this issue!


fix: form render mutating formData

resolves #368

:tada: This PR is included in version 3.0.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


False alarm warning message for invalid auth token

I am only using my Gitea account in my own computer. But I see this:
The login token is not valid anymore which may indicate a compromised account. Please check your account for unusual activities.

Unable to reproduce, no idea how it happened.


chore: support all client db connect to pgbouncer by default

pgbouncer configuration document: https://www.pgbouncer.org/config.html#section-databases

fix related issue: https://github.com/apecloud/kubeblocks/issues/7667

/cherry-pick release-0.9


Bug: HTML escaping in

 makes
.

Solved by https://github.com/cympfh/unidoc/commit/09785ea3be3d003fbb111f589e3c116000be8e7d


Multiple waypoint support

ShowRoute currently only supports one origin and one destination. It would be better if multiple waypoints were supported.

ShowRoute now takes an optional string array parameter containing waypoints.


Secret management in EKS Anywhere

Currently, EKS Anywhere uses secrets in many places. The Bare metal provider has BMC credentials in the hardware.csv file, vSphere provider has vSphere credentials in the cluster config yaml file, all providers can specify registry credentials in the cluster config yaml file, etc. All of these are stored in clear text. While some of these can be defined via environment variables, not all of them can. And while specifying secrets in environment variables is arguably better than storing them in files neither is an ideal security posture.
Important to note is that once these secrets are consumed by eksctl anywhere they are all stored as proper Kubernetes secrets. Again, not the most ideal security posture as Kubernetes secrets are just base64 encoded strings. Kubernetes itself recommends, among other things, to "Consider using external Secret store providers".
This issue is to request new functionality. Allow providing secrets to EKS Anywhere that aren't stored in config files in plain text. To be clear this is not a specific ask to make external secret store providers available per se. This ticket is to capture "what" is wanted, not "how" it is accomplished.
Last important note is that recent potential customers have made it know that Hashicorp Vault is available in their environments and would be ideal for use.

This could be an interesting option to consider. It could allow us to support many existing secret store providers.
https://github.com/external-secrets/external-secrets

Or we can implement https://kubernetes.io/docs/tasks/administer-cluster/encrypt-data/

At rest encryption solves part of the problem, but I would imagine customers want integration with their preferred secret management solution for compliance and centralized management, and features like rotation and ad-hoc credential creation that you don't get with Kubernetes secrets.
Having a list of secret management tools users/customers use and the features sets they leverage would be helpful.


Update DRC Healthsites

As a user I want to know that Healthsites for the DRC are up to date with health facilities present in OSM so that I can access up to date data
Here is an example of the data that needs to be pulled from OSM:
https://healthsites.io/map?place=Mbandaka

#856


Bug/fix testing

Minor bug fixes for UI Testing
:recycle: Current situation & Problem
UI Tests on the Consent stage of the Onboarding flow has been inconsistently failing. It seems to fail when the Signature Field is not in view when the program attempts to sign. UI Tests on HealthKit Authorization have also failed. This seems to happen when the Invitation Code Authorization is rejected, and may have to do with a typo in the build-and-test.yml file.
:gear: Release Notes

Fixed typo in build-and-test.yml file.
Added a step before the signature on the Consent View to tap the Signature Field, bringing it into view.

:books: Documentation
NA
:white_check_mark: Testing
Changes mostly pertain to OnboardingUITests.swift.
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[X] I agree to follow the Code of Conduct and Contributing Guidelines.

Will be resolved with #6


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/63835

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/95411


Girlscript Summer of Code Introduction/Tracking

👋 Hi @frankhart2018,
I am Prince,web developer Girlscript Summer of Code. I'll be collaborating with you on preparing your repo for GSSoC'24 Program
Why are these changes required?
After Analysing last year's contributions and feedback it would be great for students to have a couple of standard readme files on the repo to understand what the project is all about and some issues detailing what they can contribute. This will help you to see increased engagement in your repository as well.
As mentioned students get a chance to learn step by step, how to build this project as well as invite mentors to collaborate and add features to this repo. I will generate issues, which will provide guidance on how to prepare your repo for Girlscript summer of code 2024 on 10th May.
This issue will serve as a tracking issue to track all issues related to GSSoC'24. I recommend creating a new branch for every issue and opening a pull request to track changes so we can effectively collaborate with each other and merge changes when you and I feel like those changes are ready to be merged on your primary branch.
If you have any questions or concerns, please feel free to leave a comment on this issue or any of the other issues that are generated.
I look forward to working with you :octocat:

[x] #63
[x] #64
[x] #65
[ ] #66
[x] #67

Please take a look at the issue template and lmk if it looks good.

All done, closing.


Response view switcher

A multi switcher to switch the response between Table, JSON and Raw. It needs a server-side change to tell the client when a response could be viewed as a table

@parth-appsmith We can use the button group component here, also used on Body


Pretty printing for enums Syntax, k_normal::T, etc

Motivations
For debugging and general development, want pretty printed output of main enums/datatypes.
Solution
The top-level should optionally dump output via Debug as well as a pretty printed format.
Simple indented output to make the graph structure clear. Might want to optionally annotate Syntax elements with lexical position to help with debugging parsing/etc until source references are implemented (if ever).
Could also have a dot (Graphviz) pretty printer or similar.
The top level should also pretty print the extenv.

Graphviz/dot dropped. Can be added in separate issues if necesary.


Add speed option for actions

Proposal
This affects to delay between action parts (for example between mousedown and mouseup in click, or between each symbol typing in typeText), to the cursor speed. This option you can use if you need to emulate human-behavior (it can be useful for cases when your scripts work properly only if you do actions no too fast).
Now we have the following options: Mouse Action Options, Click Action Options, Typing Action Options.
I suppose it should be a new type of options:
Action Options: { speed: <number> }

Mouse, Click and Typing options extend it.
So, we add options argument to Select Text, Type Text and Press Key actions.
Multiply
If the test run speed options is set to some value1 and action option is set to the value2 - it will be multiplied and the action will be executing with speed value1 * value2

Note that factors should multiply, so if we have 0.5 set in test controller and 0.5 for action the resulting speed should be 0.25

Also I guess it should be between 0.1 and 1

I've updated the first post with feature proposal. What do you think about it?
\cc @inikulin @VasilyStrelyaev @DevExpress/testcafe

@AlexanderMoskovkin I don't think we need test controller option. There should be CLI and programmatical API option. Also, min value should be 0.01

Multiply

Imho, there should be  action speed option priority upon testController speed option and they should be independent. For example: we have a test where all actions should be executed quickly except one. Multiply in this case can be confusing:
test `my test` (async t => {
    t.speed = 5;
   // ... bunch of quick actions
   await t.click( { speed: 0.5 } )
   // ... bunch of quick actions
});

I don't really like idea to multiply factors. May be just use speed factor specified for action, if we don't have it - use factor specified for test controller.

By default it will be slow mode? speed === 1?

By default it will be slow mode? speed === 1
1 - fast, 0.01 - slow

Sure, sorry

I don't think we need test controller option. There should be CLI and programmatical API option.

Yes, I'm agree

I think the speed option for one action is almost useless, because it's hard to support and requires to much attention for separate actions. It's almost the same as random placed t.wait actions. In my opinion, the speed option is usefull, when i want to make slower test if it's requires more time between actions or to see what's going on at during test time.

I don't like idea to multiply factors too.

I think the speed option for one action is almost useless

For example some ASPx controls doesn't work properly when you type (or click, dblclick, drag) too fast like a robot (some script expect human-behavior with some delays between actions) . With this option you can run only several actions in slow mode but run the others fast (it will decrease tests duration)

Is possible a scenario where all actions are slow and one should be executed quickly ?

Can you give a real case when this may be useful. I can't

Ok, it was discussed in private, I'll update the top post a little later

I'll update the top post a little later

@AlexanderMoskovkin  Taking into account the discussion, the top post doesn't seem to be up-to-date yet. Have you already decided on all the things? Can you pls update it?

Have you already decided on all the things?

Not yet. I'll notify you

I've updated the top post. Here describes api changes. /cc @inikulin @VasilyStrelyaev

+1 to having ability to delay or speed up certain actions.


Notifications acting weird, causing slow downs.

http://pastebin.com/GNjRquzx
I was excited about the push notifications, so I stopped my bitcored, ran npm install bitcore-node, npm install insight-api, npm install bitcore-wallet-service from my bitcore node's folder.
Then I spun it up and the push notifications are not working.
I checked my nohup file, only to find that there were TONS of notifications queries coming from my phone every time I would open it.
(I have obfuscated my IPs with <IP_ADDRESS> and removed the wallet identifiers at the end)
I never registered my email, so I tried doing that. No notifications came.
It seems as though the notifications api calls were what was causing my slow performance every time I start up the app...
I can't figure out why this would happen... any guesses?

Push notifications go thru Google/Apple servers, are not related to the '/bws/api/v1/notifications'  api call.
/notifications just pulls updates (if available) from the wallet.
In the log I see queries every ~ 5 secs, that is normal. We removed socket.io dependency because websockets were not supported by cloudfare and socket.io pulling fail back system seems very buggy / traffic intensive. This  endpoint is very fast (~ 10/20ms), so it shouldn't be a problem.

So

No push notifications for private BWS. Only bws.bitpay.com?
No notification emails unless my BWS is on cloudfire?

Also, 1.4.0 has not remedied the lag issues I am experiencing. (I'm assuming the "sped up wallet balance lookup" is not released yet?...)

On Fri, Jan 29, 2016 at 10:59 AM, Dabura667<EMAIL_ADDRESS>wrote:

So

No push notifications for private BWS. Only bws.bitpay.com?

Unless you bundle you own app and distribute it on the app / play store...
is the way push notifications work, AFAIK.

No notification emails unless my BWS is on cloudfire?

emails notifications are not a problem in private server. You need to
configure emailservice with your proper SMTP.

—
Reply to this email directly or view it on GitHub
https://github.com/bitpay/bitcore-wallet-service/issues/454#issuecomment-176766033
.

--
Matías Alejo Garcia
@ematiu
Roads? Where we're going, we don't need roads!

Hi, are you still experiencing these issues?

Hi, are you still experiencing these issues?

Yes. I have tested:

Delete Cache
Rescan All addresses from wallet details

My BWS: 2-3 minutes
Bitpay BWS: 10-15 seconds
I am watching my log file using less F command while I make the requests, and I see that my BWS is receiving the two-step syncing request... it's just slower than BWS.
in my bitcore-node.json I have created a "bitcore-wallet-service" key with an object containing cluster: true, clusterinstances: 2 (my VPS is a 2 vCPU system) but that doesn't seem to help.
Another point of possible bottleneck is that I am routing through apache. I run SSL through apache, reverse proxy to http://localhost:3232/bws/api/ etc.
subsequent resync of wallet is less than 10 seconds even on my BWS. the GET requests for the addresses from insight is much faster, so I am assuming it was cached.
This tells me perhaps the speed issue is with insight.
I am using the latest version of bitcore, bitcoind, I reindexed with all the new necessary args in bitcoin.conf, running bitcore-wallet-service 1.9.0


Add and apply fourmolu code formatter.

Seems fine although I don't have a strong opinion on formatters (as long as there is one of them - currently that's stylish-haskell)

If we merge this we should delete the "fix-stylish-haskell" script


CodeCommit is no longer available for new customers, remove all references to it.

Describe the bug
CodeCommit is no longer available to new customers as of July 25, 2024. Remove all references to CodeCommit everywhere.
To Reproduce
Deploy the template in a new org with CodeCommit as the source, it will fail.
Expected behavior
Expect AWS services required to deploy this solution to be available.
Please complete the following information about the solution:

All versions, all regions in new accounts.

Screenshots
n/a
Additional context
See this discussion for a semi-official comment.

A simple workaround
<EMAIL_ADDRESS>


Error during this command: pip install flash-attn==2.0.4 --no-build-isolation

Describe the issue
Issue:
Command:
pip install flash-attn==2.0.4 --no-build-isolation

Log:
(llava) achandhok@achandhok-0-0:/sensei-fs/users/achandhok/LLaVA$ pip install flash-attn==2.0.4 --no-build-isolation
Collecting flash-attn==2.0.4
  Using cached flash_attn-2.0.4.tar.gz (4.2 MB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [8 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-v8_kacfm/flash-attn_d20678f39509460397d2b69d9bc4b015/setup.py", line 13, in <module>
          import torch
        File "/opt/conda/envs/llava/lib/python3.10/site-packages/torch/__init__.py", line 235, in <module>
          from torch._C import *  # noqa: F403
      ImportError: /opt/conda/envs/llava/lib/python3.10/site-packages/torch/lib/../../nvidia/cusparse/lib/libcusparse.so.12: undefined symbol: __nvJitLinkAddData_12_1, version libnvJitLink.so.12
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Used the v1.1.3 release and this issue wasn't there anymore


Cant we add new stuff to options ?

Hi,
I am new to react and i was using react select . I have a question. Cant we add to options if we dont find in the list of objects. For Eg. If am searching a city which is not there in the list, cant i add the the city to the list instead of showing no contents found ?
Please let me know
Thanks,
Vivin

Hi @vivinRajagopalan, I'm very sorry for ignoring this issue for so long. My Github "watch" setting was somehow deactivated without my realizing it and I didn't receive notifications for this project.
This sounds like something that your application logic should manage rather than react-virtualized-select. This project (and react-virtualized underneath) doesn't create any data. It just displays it. If the user enters a search string that doesn't match any results- you could insert a result and then it would appear in the filtered list. react-select has support for this.
I'm going to close this as a no-op for react-virtualized-select b'c I think the solution belongs in your application and/or react-select. But let me know if my response is confusing. :)


How to get personal access token?

Hi and thank you for taking the time to open this issue, and apologies for the delay in getting back. I hope that you were able to solve this issue as I understand it has been some time since it was first opened.
Due to the age of this issue I am going to close it for now, however if you are still having problems please either re-open the ticket, create a new issue, or contact us at<EMAIL_ADDRESS>where our support and engineering teams can help you out.
We will be updating our monitoring of issues being opened on repos to make sure we don’t let issues stagnate in future. We put a lot of time and effort into the quality of our products and this falls below the standards of support and care we offer elsewhere.


Unable to download your weight?

How can I download your weight from Japan?
Thanks

@AI-Hunter Sorry to reply you so late. I have released backbone weight files and yolo weight files. You can download them from the links in my README. Some yolo weight files will be upload as soon as possible.


Wrong CPU resource unit

In chapter Resource reservations, an example of CPU resource reservation is described, but the unit is incorrect.

CPU - 60ms

Should be core or millicore.
Ref: https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/#meaning-of-cpu

Resubmit the bug by Feedback tool.
/close

Duplicate of #18805


Fix namespace links

Added the option to generate links to other docs without broken links warning from Docusaurus
With the new config setting RewriteInterlinks links are generated like "../log" instead of "../log/log" in case Docusaurus uses a file with the same name as the folder as index
See #11

Sorry I haven't checked back on this project in a while. PR looks good and I'll write something in the README about using this when encountering issues. I guess this also means we can close #11 now?

Hey Sorry I haven't been on Github for a while. Yes the Issue can be closed (I already did it)


Workflow proposal

I don't know of a better place to write this so I'm doing it here but feel free to redirect me.
It will be nice if the groups are inherited from "default" so one can just add some augends for certain filetypes without having to duplicate the default group.
It will be also nice to make dial to use the group with the name of the filetype if it exists otherwise use 'default', that way instead of having several maps one could use the same and profit.
If you are ok with those, please, let me know. We could discuss the api and I could implement the requested changes.
By now I think both could be accomplished with this api:
 -- This one is a function I also think deal should have, probably with another name
local function words(vals)
     return augend.constant.new {
        elements = vals,
        word = true,
        cyclic = true,
        }
end

  require("dial.config").augends:register_group {
        default = {
          augend.date.alias["%Y-%m-%d"],
          ...
          },
          python = {
             words { "False", " True"  },
             extending="default" -- Or maybe true if it's from default, so less typing
            }
   }

   vim.keymap.set("n", "<C-A>", dm.inc_normal(filetype=true), { noremap = true })
   ...

Also, if you have a (semi)automated way of running the test, let me know how so I can run them myself.

Thanks for the proposal.
The automation of testing is an area that we have not worked on, and any PR would be welcome.
The topic about register_group() function is more of an API proposal than a workflow. It might be better to change the issue title or create a separate issue.

Yes, they will be probably 2 PRs for this, but I think they make more sense when used together, that's why a "workflow proposal". In any case, if there is a place to better discuss it before starting any implementation, then just let me know.

For now, the only place we have for discussion is here (issue page), so I hope we can discuss it here (or another issue).

We would like to continue to discuss changes to the API regarding configuration.
The current API for register_group is intentionally kept simple. In fact, simply inheriting multiple augends can still be accomplished with code like the following:
local augends_base = {
    augend1,
    augend2,
    ...
}
require("dial.config").augends:register_group {
    default = augend_base,
    python = list_concat { augend_base, {augend3} }
}

where list_concat is a function that concatenates multiple array-like tables.
Inherits is a convenient feature for the user, but there is also a concern about the complexity of API and inner implementation. When self-references or cross-references occur, dial.nvim must detect them and raise an error. Such errors would not be detected by the Lua language server.

True, then the inheritance should be achieved by just using vim.list_extend,    it makes sense not to complicate the api, but will be nice to add it to the doc as an example at least.
Regarding the file format thing, do you think it's better to extend the api as I said?

will be nice to add it to the doc as an example at least.

Agreed. I will consider adding it to the README and help.
For filetypes, I think the latter idea, i.e., add extra functions, is preferable, like:
require("dial.config").augends:on_filetype {  -- or more preferable function name
    python = list_concat {
        augend_base,
        {
            words {False, True}
        },
    markdown = list_concat {
        augend_base,
        {
            augend.misc.alias.markdown_header,
        }
    }
}

No, I mean if you are fine with dm.inc_normal(filetype=true)
Or you wanna have a different set of functions for filetypes.

My understanding is that by providing a function like augends:on_filetype(), API extensions like dm.inc_normal(filetype=true) should be unnecessary.

So you mean we should make inc_normal and similars trigger both kinds of groups? or that when calling them, the group of the filetype will act as "default"?
I'm not sure if that will make it more clear, check my PR, the implementation is  hacky (it looks like a sql injection :smile:  but the api seems clean to me (no needs of having two set of groups and no internal changes)

My idea is:

Provide augends:on_filetype() function in dial.config module, which can be used in the following ways:
require("dial.config").augends:on_filetype {  -- or more preferable function name
    python = concat_lists {
        augend_base,
        {
            words {False, True}
        },
    },
    markdown = concat_lists {
        augend_base,
        {
            augend.misc.alias.markdown_header,
        },
    },
}

require("dial.map").inc_normal() behaves like:

If the 'filetype' of the buffer is either "python" or "markdown", then the augends registered in on_filetype are used.
If the 'filetype' of the buffer is neither "python" nor "markdown", then the default augends are used.

When the argument is not omitted, like require("dial.map").inc_normal("group_name"), it behaves as it does now.

And now I'm checking the second PR (#53).
First of all, thank you for your very cool idea and preference to preserve the API of dial.nvim.
However, I'd like to provide a secure API, even if the internal implementation is a little more complicated.
To be honest, I was not aware that injections were possible. The risk of abuse may be small as it is just a plugin setting, but as a developer, I feel that we do not want to allow code given as a string to be executed.
I will consider appropriate ways to implement filetype support. I would appreciate your opinions, as there are many things I am not aware of on my own.

Yes, I'm aware of the hackiness of my PR, but I think it's better to keep a similar api (obviously, more secure) like making inc_normal(name: str| () -> str| nil ) , so instead of doing a "lua string injection" the user could simply do (keeping my PR naming):
vim.keymap.set("n", "<C-A>", dm.inc_normal(require("dial.extras")._resolve_ft), { noremap = true })

Adding on_filetype makes the intention less clear and at the same time less flexible IMHO. The current implementation of inc_normal (and similars) is returning a string but the mapping could instead return a regular lua closure.

I tried adding functionality based on the on_filetype idea (#54).
I do not intend to merge this PR immediately and will discuss which policy seems better before deciding to merge.
With my implementation, users will be able to set different augend rules for each filetype without changing the mapping settings at all.
I think it is clear and flexible enough, because users can still use augend:register_group() function if they desire a more complex customization.
How do you think?

Your PR is good enough for my usecase, as having per-filetype augends is what I really need. The idea of this pr was to make the thing more flexible, allowing other kinds of rules. But yes, maybe this is good enough and there is no need for that kind of flexibility.


Inline sourcemap not working

I'm using Gulp to run a task that processes my CSS through PostCSS with the cssnext plugin. Here's the task config:
gulp.task('styles', function () {
  var source = paths.src + 'styles/index.css';
  return gulp.src(source)
    .pipe(postcss([
      cssnext({
        browsers: 'last 2 versions',
        features: {
          rem: false
        },
        sourcemap: true,
        from: source
      })
    ]))
    .pipe(gulp.dest(paths.dest))
});

My index.css file has a bunch of @import lines, making use of the postcss-import feature included in cssnext.
Unfortunately, the resulting CSS file outputted by the Gulp task has no inline sourcemap at the end of it. Any idea why?

The thing is cssnext is not just a postcss plugin. You can use it directly and that the main goal of this tool. That why some options are just proxies to postcss options. (also, you can use gulp-cssnext)
In your case I think you should pass sourcemap & from to postcss directly.
Btw why are your using rem: false when your browser scope do not target old IEs ?

Thanks! That helped me figure out where to look to get sourcemaps working. The gulp-postcss package has sourcemaps disabled by default.
rem: false disables the px unit fallback, which I don't require for my project. The last 2 versions of IE can use REM except when used in the font shorthand property or in pseudo elements.


set hostname to domain value

To get proper certificate  we should set hostname value in openssl request

Hi @onur
I don't know if you are still maintaining this project. I made small change and it would be great if will find some time to review this :)

Just as an information, I lost my patience after I tried contacting with original author, hence I created new crate based on this one with your patch applied. It's here: https://crates.io/crates/ssl-expiration2

The change is added here https://github.com/VerKnowSys/ssl-expiration/commit/9f1c151097db79903e745ea9fcce2b7998947ca4 I'm closing the PR as owner is not responding for way too long


Only the first found plan is taken into account

The TFD solver is parameter in such a way that it give several plan in an increasing 'quality' order :
a - enable anytime search (otherwise finish on first plan found)
https://github.com/IntelligentRoboticsLabs/plansys2_tfd_plan_solver/blob/55d99752b0357d35d65d10c884092da34df9371c/src/plansys2_tfd_plan_solver/tfd_plan_solver.cpp#L92
But later, only the first  found plan is taken into account:
https://github.com/IntelligentRoboticsLabs/plansys2_tfd_plan_solver/blob/55d99752b0357d35d65d10c884092da34df9371c/src/plansys2_tfd_plan_solver/tfd_plan_solver.cpp#L97

Hi @teyssieuman
Yes. The idea is to have a plan as soon as possible to have a reactive system.
A nice, but complex improvement is to change the executing plan as soon as you find a better plan. Not only waiting for a better plan; new perceptions of the environment, translated to predicates, could produce better plans for achieving the final goal. But there are many open questions. For example: Should we abort executing actions? It depends if they are at the initial of the new plan.
Do you propose a better option than the current one?
Best

As you said, using a result as soon as they are available seem to be a good idea in the case of reactive system.
In my experimentations, it seems that:

Some better results can be available soon after the first one, sometime one or two seconds, which is not so long in relatively to the actions duration in most robotics applications.
It seems that the plans that are found after the first one, do not differs from the first one in many cases (I thing that this is due to the uses of heuristics to perform depth first search.
Considering those two points, a strategy can consist on:

Take the first plan as a result, and immediately execute it (we can add “wait” a parameter, in case a better result is found within a certain amount of time after the launch)
Let the planner run, until it found a new result, or it is asked for a new solution (or something change in the environment)
If a better plan is found AND, it is compatible with the current plan execution (the initial steps are the same, and the steps that differs are not executed yet, then we execute this better plan instead of the first one.
In case the better plan is not compatible with the current execution, then give the currently execution action end state as a new starting point. If a better path is found, then get back to the previous point.
I hope that those ideas do not look as a specification. This can be the case due to my professional activities, but it only aim to share ideas. Does it correspond to the ones you have?

Cordialmente …

Ok. I see your point and I agree with your view. One way to implement this could be:

Executor must deal with the existence of a new different/improved plan. The change of plan should be developed there, but first, we should solve the first part: continuously receiving a plan, even if the executor is already executing one.
Executor uses a service call to get the plan from the planner. Maybe Executor could be continuously asking for a new plan. The option is to use a publisher instead of a service.
If using service, and the executor continuously asks for a plan. The planner could return the last plan calculated, taking into account that it could be improved.

As soon as we solve this, we could work on the change of the plan in the executor. Are you planning on providing a solution for the above improvement?

I'm afraid to not be familiar enough with ros2 architectures to be pertinent with this choices. But isn't it possible to use a simple scalar integer topic that continuously stream the number of plan found for a given problem : 0 for no plan, 1 for the first ...
The Executor can then monitor this state and ark for a new plan when is is available.
Then, two service can be defined :

launch the problem resolution
get a plan (parameterized by the plan number)

I can try to help solving the above issue since I can have use of such a strategy.


prototype speech to text

Use https://developers.deepgram.com/docs/node-sdk-streaming-transcription

done with #6


Add weigth != 0 Check for Weighted Round Robin LoadBalancer

When adding item to edfSchduler, we do not check weight != 0, which may causes undefined behavior for MOSN

This issue proposes the adding of zero check for MOSN LB, not only for WRR
ref:

cc. @alpha-baby @jizhuozhi

i have another question:

When weightFunc return 0, do we need to remove the iterm from the edf heap?

Which condition would we get 0 weight, and if there is only one item, what will be happened if we remove it?

I think it's necessary to keep the size of items consistent @alpha-baby @antJack

what if rounding any weight<=0 to weight=1?
envoy's load_balancing_weight is uint32 and min value is 1.
ref: https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/endpoint/v3/endpoint_components.proto#config-endpoint-v3-lbendpoint

Maybe it's not a bug but a feature~~.
We could make it clear in the documentation that we don't accept negative numbers (or negative numbers have the highest priority), and 0 is a fallback, 0 items will not be used until all non-zero items are invalid

weight=0 will yield 1.0/weight = +Inf which causes undefined behavior for further code execution. And that's what we should prevent.
as for the meaning of weight<=0, still cannot imagine in which scenario that we should allow weight<=0. Currently I suggest keeping it simple, or there's any scenario?

MOSN has host weight const： in pkg/configmanager/parser.go

So there is only one problem: what should we do when weightFunc return 0 or less than 0
I think this value can be fixed using a consistent contract

maybe we should keep consistent: if weightFunc return <MinHostWeight then set to MinHostWeight, if > MaxHostWeight then set to MaxHostWeight


Add content/en/hypervisor

adding the term hypervisor to the glossary
Signed-off-by: Kiran Satya Raj<EMAIL_ADDRESS>Describe your changes
Related issue number or link (ex: resolves #issue-number)
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Thanks for suggesting the changes @NitishKumar06, they were relevant so I've committed them.

@iamNoah1 I'm afraid if the website contains some kind of checks where the build gets failed due to large number of commits raised. I have no idea till now if such check exists.  In order to keep on the safer side, It could be better if @KiranSatyaRaj squash all the commits. WDYT @iamNoah1 ?

Thanks @NitishKumar06
large number of commits does not incur build failures. But, since meaningless commits are not desired, squashing commits in a PR is desired as @NitishKumar06 suggested.
By the way, contributor doesn't have to keep squashing or amending commits for every PR updates. The squashing can be done by contributor just before this PR is ready to be merged.
Anyway, an approver will use squash and merge method to merge this PR.

@KiranSatyaRaj please let me know whenever you have incorporated or considered the suggested feedback and if we should took a look again :)

Sure, there were a few points I mentioned in the discussion regarding the term I just wanted to check whether they were neutral and straightforward, that's fine I'll work my way through it and get back to you with the applied changes as soon as possible.

Hey @iamNoah1 I made, can you please go through it and let me know what more can be done to make it better.


Support data viewer while debugging

The January 2021 release of the Python extension for VS Code supports a data viewer while debugging. Could this data viewer be ported for use with R objects while debugging? Since it is designed to work with Python objects, it may be compatible with {reticulate} Python objects.
Related: https://github.com/Ikuyadeu/vscode-R/issues/506

If the debug session source vscode-R ~/.vscode-R/init.R then in the debug console, View(obj) could open the data viewer too.

If the debug session source vscode-R ~/.vscode-R/init.R then in the debug console, View(obj) could open the data viewer too.

Running the following code
Sys.setenv(RSTUDIO="")
Sys.setenv(TERM_PROGRAM="vscode")

source("MY_PATH_TO_VSCODE_R\\vscode-R\\R\\init.R") # <-- replace with path to your init.R

my_function <- function(){
    df <- cars
    print('inside function') # <-- breakpoint here. Then run `View(df)` in debug console.
}

my_function()

with debug config
{
  "type": "R-Debugger",
  "name": "Debug R-File",
  "request": "launch",
  "debugMode": "file",
  "workingDirectory": "${workspaceFolder}",
  "file": "${file}"
}

Produces a similar result as using the data viewer in python. Is this roughly what you had in mind? (Without the manual setting of environment variables and manually running init.R of course).
Introducing a context menu command that opens this data viewer from within a debug session should be doable.
Python's data viewer looks quite nice. Porting it to work with R might be a way to improve the data viewer without too much work for ourselves, but that should probably be considered as part of https://github.com/Ikuyadeu/vscode-R/issues/506.

Running the following code
Sys.setenv(RSTUDIO="")
Sys.setenv(TERM_PROGRAM="vscode")

source("MY_PATH_TO_VSCODE_R\\vscode-R\\R\\init.R") # <-- replace with path to your init.R

my_function <- function(){
    df <- cars
    print('inside function') # <-- breakpoint here. Then run `View(df)` in debug console.
}

my_function()

with debug config
{
  "type": "R-Debugger",
  "name": "Debug R-File",
  "request": "launch",
  "debugMode": "file",
  "workingDirectory": "${workspaceFolder}",
  "file": "${file}"
}

Produces a similar result as using the data viewer in python. Is this roughly what you had in mind? (Without the manual setting of environment variables and manually running init.R of course).
Introducing a context menu command that opens this data viewer from within a debug session should be doable.
Python's data viewer looks quite nice. Porting it to work with R might be a way to improve the data viewer without too much work for ourselves, but that should probably be considered as part of https://github.com/Ikuyadeu/vscode-R/issues/506.

Thank you both. I was able to get the current data viewer to work using View() in the debug console, both with your code @ManuelHentschel and when using the Attach to R process configuration.
I agree that a context menu item would be helpful, as would a View icon on hover in the Run:Variables pane, similar to the one that is visible in the R:Workspace pane.
Bigger picture, porting over the Python extension's data viewer would be an excellent addition.

When I followed the above steps in Remote-Containers, I got the following error and the data viewer did not open.
Error in .External2(C_dataviewer, x, title) : unable to start data viewer
In addition: Warning message:
In utils::View(val, node$name) : unable to open display

If I run View(mtcars) after executing .vsc.attach() in the integrated terminal, the viewer will be displayed normally.
Could this be a problem specific to Remote-Containers?

@eitsupi If you ssh to any remote server and start an R session and View() any data frame, the same error will occur.

@renkun-ken Thanks for the reply.
I've since figured out on my own that it's probably related to #146, but I don't know what I need to add to the .Rprofile to make it work properly.


[macOS] Add native "Edit" menu.

Adds default "Edit" menu (and some extra functions for native menu to show it after "Scene" in the editor).
Adds handlers for native cut/copy/paste actions to the content view (redirected as key presses to Godot event handler).

Fixes https://github.com/godotengine/godot/issues/97437 (~I was not able to reproduce crash, but copy-paste now works in native file dialogs~, Edit: reproduced with updated info and added a fix).

Adding a default Edit menu with copy paste etc sounds good, especially for the editor. But how will this interact with setting up a NativeMenu Edit menu of your own? Will it be possible to add/modify callbacks or items in this Edit menu or opt out of it entirely if you want to make your own?

Like with the other default system menus, it's possible to add new items (by using get_system_menu(EDIT_MENU_ID) instead of create_menu() or attaching PopupMenu to a system menu), but not to modify remove default items. Default "Cut"/"Copy"/"Paste" simply send "ui_cut"/"ui_copy"/"ui_paste" actions to the active window, so it can be customized as well.

Ok, in my case I wouldn't like to remove the cut/copy/paste items. But I would like to change their callbacks to I can hook them into my own menu item object actions, and I would also like to enable/disable them as I do selection and clipboard content validation to check if the use can currently copy/cut/paste that disabled/enables the corresponding items.
If this is not possible it would be better if there was a project setting or way to just remove the default Edit menu, or at least hide it to be able to use my own.

Opened https://github.com/godotengine/godot/pull/97678 for the crash fix only.
Will probably update this one to move editor "Undo"/"Redo" to "Edit" (where it usually is), but this will require at least some default item customization capabilities.

For custom validation (when you can copy/paste/undo/redo) or custom undo action names in the menu items I guess you'd need to have some system similar to the delegate system in macOS where you supply reference to an object the engine can call to get this information when it needs it. But as far as I know Godot doesn't have a pattern like this to provide custom information suppliers, and this case might be a little specific to add one. But it's a very useful pattern in many GUI-related cases, so it could be worth considering how something like that could be introduced in Godot for other purposes as well. And if something like this was introduced it could also be used to handle the actual copy/paste/undo/redo actions if someone wants to override those instead of using the default implementations.

I guess you'd need to have some system similar to the delegate system

Since Godot is not using anything similar, I have added can_cut, cut, etc. callbacks to the NativeMenu.

Example
func _copy():
	print("!copy!")

func _can_copy():
	return $CheckBox.button_pressed

func _ready() -> void:
	NativeMenu.can_copy = _can_copy
	NativeMenu.copy = _copy

It might be worth implementing some sort of delegate system, but it out of the scope of this PR and probably deserve a separate proposal.

Since Godot is not using anything similar, I have added can_cut, cut, etc. callbacks to the NativeMenu.

Sounds like a good compromise, and I guess you'll add similar ones for undo/can_undo/undo_action_description etc? I know you like to keep things short, but it might still be worth adding "callback" to these (as in can_copy_callback instead of can_copy) to clarify that these are callbacks, and not confuse them with the current state of the menu item. Especially since this will be a bit of an unusual pattern with callbacks requesting return values.

I guess you'll add similar ones for undo/can_undo/undo_action_description etc?

No, with the way editor undo/redo is currently implemented, this is pretty much impossible to implement without breaking the editor.

I guess you'll add similar ones for undo/can_undo/undo_action_description etc?

Done. Currently, it's not fully integrated with the editor, "Edit" menu items work in editor (as generic content independent Undo/Redo), but there are duplicate context specific menu items in the local menus (same is true for Copy/Cut/Paste). Which is not perfect, but I'm not sure what's the best solution for this.

I guess you'll add similar ones for undo/can_undo/undo_action_description etc?

Done.

Nice. I see the undo_description and redo_description are now string properties, should these not be callbacks returning strings to be called on menu validation (before menu is opened) just like can_undo and can_redo to ensure they're updated at the correct time?

should these not be callbacks returning strings to be called on menu validation (before menu is opened) just like can_undo and can_redo to ensure they're updated at the correct time

Actually, this is probably unnecessary as well, and can be simplified to bool properties as well.

Actually, this is probably unnecessary as well, and can be simplified to bool properties as well.

As long as they can all be set during a popup_open_callback for the native Edit menu either way would be fine I guess, but I think it should be consistent. Either setting all of these overrides as properties or hooking up callbacks where you supply the values when they're needed. I think using callbacks would make it clearer that you're overriding the default implementation, and it would be clear the values you return from these callbacks would be used. With just bool and string properties I would be uncertain when these would/should be set and it would also invite expectations of getting the current state using these parameters and not just setting it to override default behavior.

Changed can_* properties to bool. Both can be set from popup_open_callback. In this case, callback seems like overkill.


range persian datepicker

when I want to use this calendar for Persian range, it makes an error and exactly not work, :(
please add this feature soon, thx

Hi @mafshark, this is so vague. What error does it throw? I just tried Persian range picker and it's working perfectly: CodeSandbox working demo

this is my code

`import React, { useState } from "react";
import "react-modern-calendar-datepicker/lib/DatePicker.css";
import DatePicker from "react-modern-calendar-datepicker";
import './style.scss'
const DP = () => {
const [selectedDay, setValue] = useState({ from: null, to: null });
return (

);
};
export default (DP);`

this is my code

`import React, { useState } from "react";
import "react-modern-calendar-datepicker/lib/DatePicker.css";
import DatePicker from "react-modern-calendar-datepicker";
const DP = () => {
const [selectedDay, setValue] = useState({ from: null, to: null });
return (

);
};
export default (DP);`

@MasoumeAfshar Did you set locale prop to 'fa' in the Datepicker component?
<DatePicker
   locale="fa"
   value={selectedDay}
   onChange={setValue}
/>

@MasoumeAfshar return is empty in this code 😑

import React, { useState } from "react";
import "react-modern-calendar-datepicker/lib/DatePicker.css";
import DatePicker from "react-modern-calendar-datepicker";
import './style.scss'
const DP = () => {
const [selectedDay, setValue] = useState({ from: null, to: null });
return (

);
};
export default (DP);

return (

);

<DatePicker
  locale="fa"
  value={selectedDay}
  onChange={setValue}
  shouldHighlightWeekends
/>

import React, { useState } from "react";
import "react-modern-calendar-datepicker/lib/DatePicker.css";
import DatePicker from "react-modern-calendar-datepicker";

const DP = () => {
  const [selectedDay, setValue] = useState({ from: null, to: null });
  return (
    <DatePicker
      locale="fa"
      value={selectedDay}
      onChange={setValue}
      shouldHighlightWeekends
    />
  );
};
export default (DP);

What is the version of the library you are using?
You can find it by running following command in the root of your project:
npm ls react-modern-calendar-datepicker

<EMAIL_ADDRESS>

Please run the following command in the root of your project:
npm update<EMAIL_ADDRESS>

@Kiarash-Z we should do something for the breaking change mentioned in #90. We can fix it or just let the developers know that there was a breaking change in README or documentation website.

@thebrodmann just released 2.0.0 with no changes because of this👍

@MasoumeAfshar please install the latest version and your issue should be fixed😇

I updated the version and it was fixed , thx :+1:


Bad connection issue (from hacf forum)

Hey @jcjames13009 @scoudibou,
I've been following your discussion on the hacf forum, but unfortunately I cannot send any replys there (blocked?). That said, to me it looks like the Logic-Level-Converter Board (blue) from jcjames is not soldered correctly: If I am not wrong, I don't see any soldered pins there.

Of course there are 12 pads, i just highlighted a few, can you check those?
Cheers

Hi @tspopp
Thanks for your message and taking care about my issue
I double checked the 12 pins and converter is soldered properly. But you're right it looks bad from the picture
I measured voltages when converter is installed on the pcb and have the following values. Whatever if Data wire (the white one) is connected or not to the pcb

When testing converter on a breadboard I have 3.2v on all LV outputs and 5v on all HV outputs
I'll receive soon a new converter. I'll test it and will keep you posted
Cheers
JC

Hey @jcjames13009,
not sure if your issue is really hardware related. It is unlikely but possible that the Thermor Aeromax Split 2 is using a different serial protocol which AquaMQTT is not able to understand at this point in time. I am very curios what happens if you set the AquaMQTT Board to Listener Mode (Passthrough Jumper connected) and install AquaDebug on the device. Afterwards you are able to record a trace of the raw serial data using the python script next to AquaDebug. Also have a look in the newly created Troubleshooting section....
Looking forward!

Hi @tspopp
I was also thinking that the AquaMQTT may not be compatible with my water heater and planned to test it in Listener mode
Your essage arrive at the right time and cool if you add AquaDebug feature. I'll do the test maybe this envenig or beginning of next week and will keep you posted
Cheers
JC

@jcjames13009 did anything come out of this?

I'll close this due to inactivity. We will reopen this in case we get feedback or results.


Allow users to change their username

backend already implemented, just post to the same endpoint

dupe of #25


want keras to work on GPU, but actually on CPU

System information:
OS Platform and Distribution: Linux Ubuntu 16.04:
TensorFlow installed from anaconda:
Keras installed from pip
TensorFlow version 1.4.1
Bazel version : None:
CUDA/cuDNN version : CUDA V7.5.17, CuDNN v6.0:
GPU: GeForce GeForce GTX 1050 Ti(3.94GB):
Code: mnist_mlp.py
When run the code, it shows the following log:

Using TensorFlow backend.
2018-01-25 17:31:16.572013: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA
2018-01-25 17:31:16.679268: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:892] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2018-01-25 17:31:16.679499: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1030] Found device 0 with properties:
name: GeForce GTX 1050 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.468
pciBusID: 0000:01:00.0
totalMemory: 3.94GiB freeMemory: 3.46GiB
2018-01-25 17:31:16.679513: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1120] Creating TensorFlow device (/device:GPU:0) -> (device: 0, name: GeForce GTX 1050 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)

Have no idea why it shows:

2018-01-25 17:31:16.679268: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:892] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero

And the code is actually running on the CPU, although the log shows the info of GPU
What have tried: stackoverflow :)

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


x='NaN'

<use id="strelka" xlink:href="#strelka" width="78.75" height="50" x="NaN" y="0">
<use id="test_stand" xlink:href="#test_stand" width="33.25" height="50" x="NaN" y="0">
<use id="tractor" xlink:href="#tractor" width="33.25" height="50" x="NaN" y="0">
<use id="tractor_car" xlink:href="#tractor_car" width="33.25" height="50" x="NaN" y="0">
<use id="train" xlink:href="#train" width="33.25" height="50" x="NaN" y="0">
"external-svg-sprite-loader": "^3.3.0"

@bensampaio in my case it reproduces when external-svg-sprite-loader try add to build svg font, and function:
https://github.com/karify/external-svg-sprite-loader/blob/8c57bd4cba28261fcfb7b299363b1aac462e5e13/lib/SvgSprite.js#L117
return width: NaN
maybe need add warning or error for easy debug it's case?

not, not have. But I think loader must throw error or ignore this file and show warning

@usebaz can you check if version 4.0.0-beta.6 fixes your problem? :)


Update README.md

This pr is under issue #19.

@harshsingh32 can you please add a logo of the GSSoC'23 batch to indicate that our project is part of GSSoC'23?

@harshsingh32 this is on priority


Bump to 0.27.1

Brings in  #822 which unblocks https://bugzilla.mozilla.org/show_bug.cgi?id=1366956,
and also #829 which should greatly reduce merge conflicts in checked in bindings.

@Manishearth you need to update the lock file:
diff --git a/Cargo.lock b/Cargo.lock
index 1615200..6ea1045 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1,6 +1,6 @@
 [root]
 name = "bindgen"
-version = "0.27.0"
+version = "0.27.1"
 dependencies = [
  "aster 0.41.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "cexpr 0.2.0 (registry+https://github.com/rust-lang/crates.io-index)",

r=me with that

Let's don't push on this. It seems the bindgen still doesn't work for stylo.

I'm aware, been investigating that.
See https://github.com/servo/rust-bindgen/pull/842#issuecomment-317161129

I think I'm hitting a different one likely regressed from #829.

:umbrella: The latest upstream changes (presumably #845) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-servo r=fitzgen

:pushpin: Commit d4da81a has been approved by fitzgen

:hourglass: Testing commit d4da81a5b6931905084a07b0644bdc17536b2999 with merge 23dbe487b8d2a142fb03d9aefc3b4dd29aa78009...

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing 23dbe487b8d2a142fb03d9aefc3b4dd29aa78009 to master...


Ripple Effect On Clicking Button

I want to add ripple effect on button so it gives better looks to this site and overall it enhance the interface of the site.

Can I work on this issue

I want to add ripple effect on button so it gives better looks to this site and overall it enhance the interface of the site.

@RakeshCharpe  Please go for it assigned  to you.


[Discussion] Tailwind CSS configurations and rules

Which configurations should we configure/adopt for tailwind CSS?
For now we only have the default Tailwind CSS configurations configured.
Colors

Customizing the default color palette for your project.

We should add (extend) custom colors (i.e: primary, secondary, primary-hover, success, warning) to stop using hardcoded or arbitrary tailwind values.
Spacing

Customizing the default spacing and sizing scale for your project.

By default, Tailwind includes a generous and comprehensive numeric spacing scale. The values are proportional, so 16 is twice as much spacing as 8 for example. One spacing unit is equal to 0.25rem, which translates to 4px by default in common browsers.

I think that the current scale is hard to reason (ex: p-2 give you 8px padding) but I understand that it can be more accessible by using rem units instead of pixels.
Screens

Customizing the default breakpoints for your project.

I think that for now we don't need to think much about this section.
Plugins

Extending Tailwind with reusable third-party plugins.

We already have the custom debugPlugin plugin that renders outline borders for elements (debug) and childs (debug-childs)
This is a good place to register utility classes for an eventual design system built using tailwind css rules in a CSS file with @apply (eg: [.btn-primary](While you can usually build the bulk of a well-crafted design using a constrained set of design tokens, once in a while you need to break out of those constraints to get things pixel-perfect.
), .btn-secondary).
Arbitrary values

When you find yourself really needing something like top: 117px to get a background image in just the right spot, use Tailwind’s square bracket notation to generate a class on the fly with any arbitrary value:

IMO we should avoid using this feature. Notable cases are for :content and to use dynamic variables.

@danvk @sehyod  please let me know if you have any thoughts about these items.


Add log_destinationless_tuples option

I added a new option: log_destinationless_tuples. A destinationless tuple is a kind of dropped tuples. It's dropped when a source or a stream doesn't have any destination. To enable destinationless tuple logging, both log_destinationless_tuples and log_dropped_tuples options need to be true. Only enabling log_dropped_tuples doesn't report destinationless tuples.
This is a breaking change but destinationless tuples will not be reported by default.
fixes #125

LGTM!


US_1.1: Consistent german language 🍪

As
user
I want to
have consistent german language in the GUI
in order to
have a great user experience.
5 🍪
This is an important issue for the already existing system. In the future we expect this to be considered throughout the new features. The responsibility is restricted to tabs that no other team actively modifies.

[ ] There is no mix of languages for names of input fields, error messages and the like

As @TimBasel pointed out, localizations should be implemented using i18n (as in pull request #64)

Adressed in more detail in #145 - #151.


Lokole fails to install with Python 3.9+ [Lokole requirements.txt file needs updating]

Any Python developers who can assist, please see:

iiab/iiab#3132

Thank you!

https://itsmycode.com/importerror-cannot-import-name-json-from-itsdangerous/

Should this issue be kept open or closed?
FYI this Internet-in-a-Box PR is now merged:

PR iiab/iiab#3149

Is this problem fixed? The Lokole can now install on IIAB now?
I have found a freelancer who wants to talk to me today to negotiate about working on this error.
@jvonau What can you tell me? By the way, I am running iiab full install now with  Raspios64bits and started 15 minutes ago.

Not fixed here, just have a workaround in place within IIAB.

@jvonau thanks for the clarifications. Should I put you in contact with whoever I would find to help correct the problem?
By the way, iiba full big install has just completed without problems. I have published the iiab-diagnostics of my install.

Just point out the above link to the fix in IIAB, your recruit should know what to do, or find somebody else. Other Ref: https://stackoverflow.com/questions/71618764/why-flask-cli-suddenly-throws-import-error-when-3-commit-before-worked-fine-no

Thanks @jvonau

Thanks again to @jvonau who provided this interim patch/workaround below (within IIAB) until Lokole fixes its own requirements.txt !

PR iiab/iiab#3149

@nzola I hope this is resolved quickly in coming days!
In any case. I've posted a 2-sentence public advisory to help newcomers — summarizing the situation in the "Known Issues" section at the bottom here:

https://github.com/iiab/iiab/wiki/IIAB-8.0-Release-Notes#known-issues

@mabuelhagag and @nzola can this ticket be closed now?
Context: About a year ago I was told that Lokole needed to update its own requirements.txt file, as this was obliging IIAB implementers to install Lokole twice as a temporary workaround — as detailed at https://github.com/iiab/iiab/issues/2833#issuecomment-931559600 and mentioned on PR iiab/iiab#3149.
Thanks for clarifying!

To my side, I confirm the Lokole thru IIAB install is smooth and I am satisfied. I recommand this issue be closed, unless otherwise.
Thanks

I'm closing this now. Thank you everyone for all your help and support ❤


Fixed print styles for page-alerts

staging: https://238-78596482-gh.circle-artifacts.com/0/tmp/circle-artifacts.SGdrH7S/packages/page-alerts/tests/site/index.html


Memory leak

The new release consumes more and more memory the longer it is left open.

@luciorubeens Something to look at, I noticed same thing in Windows 8.1 x64.

Yes, will investigate

@luciorubeens
There are quite a few comments on slack reporting these same issues.
@eugeneli /goldenpepe(slack) recently posted a screen reporting 'missing remote object'.
There are also Changer errors, which I've never NOT seen in the desktop-client.
There are several discussions around the web that may be relevant.
One possibly helpful discussion I found:
Electron: Missing remote object #8205
TL;DR
May be garbage collection issues.
Many commenting ditching remote in favor of ipcMain/ipcRenderer seems to work.
I'm not familiar with how it would be implemented,
but I hope it helps.

my analysis is this line is making memory consumption:
https://github.com/ArkEcosystem/ark-desktop/blob/master/client/app/src/accounts/account.controller.js#L769
The issue is that previousTxarray is hooked into an object and the whole array is never cleaned by the garbage collector of the javascript engine.
Since this is called every 8s here
https://github.com/ArkEcosystem/ark-desktop/blob/master/client/app/src/accounts/account.controller.js#L769
It gives you the idea how far it can goes.

Is this still an issue? If so, can someone walk me through the steps to reproduce it? I have been unsuccessful in identifying anything using the dev tools memory profiler. I'm not sure if it's because my accounts don't have transactions or if it has to do something with delegates etc.

@spresnal under number 2 its more details on this: https://blog.ark.io/new-ark-desktop-wallet-release-v1-4-0-2d4c286417f6 there is a temp solution atm if you are leaving it for longer periods of time you can switch to manual account refresh.


Reenable a couple notebook smoketests

And remove vscode-notebook-tests in favor of an .ipynb in vscode-smoketest-express
Fix #139672

Hey @TylerLeonhardt is this approach ok for the build?

Looks like that worked @roblourens? but you have a conflict now


Link from reference API to source code on GitHub

Tested for:

[x] param
[x] datashader
[ ] panel (❌)
[ ] holoviews (❌, same panel issue)
[ ] lumen (❌, same panel issue)

(no ref api docs)

hvPlot
colorcet
geoviews

https://github.com/holoviz-dev/nbsite/assets/6613202/633d0aae-675f-44ae-94b1-ff1e700fe4b8
https://github.com/holoviz-dev/nbsite/assets/6613202/e4900554-15a0-4169-84dc-9ed9b7075e3d

@droumis do you happen to remember what the issue was with Panel?

It's on my todo list to look back into this. I think I tried recently to reproduce and capture the issue better but ran into some other issue and then other priorities came up. I should have time in the next week to try again.

Panel: Seems to be working now for Panel on my local machine:
https://github.com/holoviz-dev/nbsite/assets/6613202/4ed109d0-3136-4216-ad26-ee6876b6fd69
Lumen: reference API doesn't have source links. We would need to change Lumen's special generate_reference.py code to get this working. I think we just skip Lumen for now.
I can try HoloViews later, but I've never tried to build these docs locally before. Is there a way to run the HoloViews docs build to the dev site using this specific nbsite branch? Maybe creating a fake PR and changing the github workflow in some way to point to this branch?

I can try HoloViews later, but I've never tried to build these docs locally before. Is there a way to run the HoloViews docs build to the dev site using this specific nbsite branch? Maybe creating a fake PR and changing the github workflow in some way to point to this branch?

Yes you should be able to do that:

create a PR on HoloViews that updates its docs.yaml workflow file, adding a step to re-install nbsite from this branch. I can never remember the pip install incantation required to install from a git repo, even less so from a branch 🙃
launch a dev build of HoloViews' docs from Github's UI picking the HoloViews PR.

In progress: https://github.com/holoviz/holoviews/pull/6046

Alright, it's working on the HoloViews dev site. I also tested on the Panel dev site.
I think we should come back to Lumen with another PR later.
Otherwise, I think this is ready to merge.


HIVE-23638: Spotbugs issues in hive-common

NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HIVE-XXXXX: Fix a typo in YYY)
For more details, please see https://cwiki.apache.org/confluence/display/Hive/HowToContribute

@belugabehr  thanks for the review!
Addressed your comments in the latest commit -- could you please take another look?


no output

Cheng,
I am running MetaDiff on some test data. It seems to go through the R file just fine but outputs an empty file. Here is the error message:
[04/21/2016 12:51:34]:  Running R script for metatest:
Rscript /proj/regeps/regep00/studies/aneurysms/analyses/rerdu/testrnaseq/run_metatest.R /proj/regeps/regep00/studies/aneurysms/analyses/rerdu/testrnaseq/fpkm.mat 0
[04/21/2016 13:03:48]:  /proj/regeps/regep00/studies/aneurysms/analyses/rerdu/testrnaseq/metadiff_results.tsv
[04/21/2016 13:03:48]:  Error generate fdr columns. The output from Rscript may not be formatted correctly. Check error messages in metadiff_r.log under the result directory.
I tried running the R script directly and the results matrix is generated (with some warnings) but it outputs to the console (file="" in write.table):
df_res <- ddply(data_mat, .(Feature), .fun = run_metatest)
write.table(df_res, file = "", row.names = F, quote=F, sep='\t')
I am using the latest version that was compiled in Nov 2015.
Thanks!
Rose

Hi Rose,
Can you provide a minimal working example to reproduce this error? It will help me to debug the issue. :smile:

Cheng

Cheng,
Here is the sample configuration file (4 samples, 1 covariate for experimental group, 1 covariate for patient. It is for matched paired samples.). The input files are from cufflinks.:
Sample	File_Name	C_group	C_patient
testA	/proj/testA/cufflinks_out/isoforms.fpkm_tracking	Aneu	1
testS 	/proj/testS/cufflinks_out/isoforms.fpkm_tracking	STA	1
testA2 	/proj/testA2/cufflinks_out/isoforms.fpkm_tracking	Aneu	2
testS2	/proj/testS2/cufflinks_out/isoforms.fpkm_tracking	STA	2
Here is the command I used to run MetaDiff.jar:
java -jar /udd/rerdu/local/MetaDiff.jar -input_file_list test_config_metadiff.txt -output_dir /proj/testrnaseq/ -method cufflinks
Here is the metadiff.log file:
[04/21/2016 13:11:45]:  Starting logging to file: /proj/testrnaseq//metadiff.log
[04/21/2016 13:11:45]:  processing file: /proj/testA2/cufflinks_out/isoforms.fpkm_tracking
[04/21/2016 13:11:48]:  processing file: /proj/testA/cufflinks_out/isoforms.fpkm_tracking
[04/21/2016 13:11:48]:  processing file: /proj/testS2/cufflinks_out/isoforms.fpkm_tracking
[04/21/2016 13:11:49]:  processing file: /proj/testS/cufflinks_out/isoforms.fpkm_tracking
[04/21/2016 13:11:51]:  Filtering features according to mean_fpkm_threshold = 1
[04/21/2016 13:11:51]:  22595 features remaining after filtering
[04/21/2016 13:11:51]:  Filtering features according to cv_threshold = 1
[04/21/2016 13:11:51]:  628 features remaining after filtering
[04/21/2016 13:11:51]:  Running R script for metatest:
Rscript /proj/testrnaseq/run_metatest.R /proj/testrnaseq/fpkm.mat 0
[04/21/2016 13:23:37]:  /proj/testrnaseq/metadiff_results.tsv
[04/21/2016 13:23:37]:  Error generate fdr columns. The output from Rscript may not be formatted correctly. Check error messages in metadiff_r.log under the result directory.
Here is  metadiff_r.log
There were 50 or more warnings (use warnings() to see the first 50)
While I was in R, I looked at warnings() and got:
In metareg(y, x, yvar, threshold = threshold, maxiter = maxiter,  ... :
Variance estimate has been set to zero
I assume this is because there are too few data points from the test data.
Thanks for your help!
Rose

Thanks Rose! Let me take a look and get back at you. :smile:

Hi Rose,
You are absolutely right. For meta-regression to work, you need to provide more than 1 data point for each category. Can you increase your sample size and see if it spits out the same error?
Cheng

I will add this issue to the READ.ME

I have reduced the test data to just one category so there are 2 data points for each group (control, experiment) but still have the same problems. The R script (run_metadiff.R, see below) that is generated seems to print the result out to the console rather than to a file. Is that correct? It seems like it should just write the results to a file. Also, would the FDR be computed from the pttest_factor(C_GROUP)?
Thanks!
Rose
library(metatest)
library(plyr)
args <- commandArgs(trailingOnly = TRUE)
file_in <- args[1]
data_mat <- read.table(file_in, header=T, sep = '\t')
run_metatest<-function(mat){
tryCatch({  meta_obj <- metatest(formula = y~factor(C_GROUP), variance = variance, data=mat)
res_vec <- c(meta_obj$convergence, meta_obj$tval, meta_obj$dfttest, meta_obj$pttest, meta_obj$bartLLR, meta_
obj$pBartlett)
cov_name <- dimnames(meta_obj$coefficients)[[1]]
names_vec <- c('Convergence', paste('tval_',cov_name,sep=''), 'dfttest', paste('pttest_',cov_name,sep=''), p
aste('bartLLR_',cov_name,sep=''), paste('pBartlett_',cov_name,sep=''))
names(res_vec)<-names_vec
return(res_vec)}, error = function(e) {
warning(paste('Metatest failed at Feature: ', mat[1,1],sep=''))
warning(e)
return(NULL)})}
df_res <- ddply(data_mat, .(Feature), .fun = run_metatest)
write.table(df_res, file = "", row.names = F, quote=F, sep='\t')

Hi, rose,
Unfortunately it's still not enough. You need at least 3 data points for each category. Also it's suppose to write to stdout because the java code read the output from stdout, so that's not a bug.
Cheng


perf: improve ApplyInstantID performance

InstantID is re-instantiated every time ApplyInstantID is called, but this is redundant from a parameter point of view. Moving the instantiation of InstantID in ApplyInstantID to InstantIDModelLoader can improve the performance of the ApplyInstantID node (and for InstantIDAttentionPatch).

let me talk to the instantid guys, this was done to support further development of instantID models, but as all technologies in this field that doesn't seem to be happening

In addition, does noise play a big role in Apply? If uncond_image_prompt_embeds could cached after first encoded, the performance improvement will be greater.

noise helps quite a bit yeah

GPU: Tesla T4 8G

Upon merging this PR, the ApplyInstantID now completes in approximately 200-300 ms, a significant reduction from the previous time 3300 ms+. I am contemplating open a PR for similar improvements in the IPAdapter repo as well.

instantid is simple because we have only one configuration, with ipadapter we have quite a few. it's doable but it needs more work


Support XML Output

We should support writing data out as .xml format. This will help us test the whole workflow

It sounds interesting, but the scope of the projects is reading pbf files.


bert model calls squeeze on non-dim 0 tensor

bert model calls squeeze on the last dimension of a 3D tensor:
torch.ops.aten.squeeze(tensor, -1) # tensor.shape = torch.Size([1, 256, 1])

However, ttnn.squeeze only supports dim 0 currently

No known workarounds. Might be fixed around end of July.


AppServiceEnvironments REST API: IpSecurityRestriction is not updated

Link to article:
https://docs.microsoft.com/en-us/rest/api/appservice/appserviceenvironments/listwebapps#ipsecurityrestriction
Problem:
What's wrong with the article?
Actual REST response contains different fields for ipsecurityrestriction then appear in docs. Actual response fields are: ipAddress, action, priority, name, description.
No subnetMask field any more. I think it is from a previous api-version.

@rloutlaw Can you please take a look at this issue?


Missing change log for VS Tools for Unity <IP_ADDRESS>

@conceptdev fyi

I've let the team know, thanks.

See https://github.com/MicrosoftDocs/visualstudio-docs-pr/pull/3004
cc @jbevain
Thanks.

See MicrosoftDocs/visualstudio-docs-pr#3004

404

@mabakay yes sorry the above URL requires special access rights.
To summarize here, <IP_ADDRESS> is a bugfix release, we found and fixed a deadlock in the library used to communicate with Unity’s debugger engine, making Visual Studio or Unity freeze, especially when hitting ‘Attach to Unity’ or restarting the game.
Thank you.

The updated build log has been approved and publishing is in progress.
Thanks for your feedback @mabakay.
#please-close


Generate aggregated coverage report containing all process definitions

As a developer I want to have a consolidated report (containing all process tests) being generated when I run the tests on my project.
Current behaviour (v1.0.3): When I run multiple test classes, each testing a different process, I only get a report for the first test class.
As a workaround we can use a dedicated instance of ProcessEngineCoverageExtension for each test so that we get one report.html for each.

Extracted the issue where test classes sharing an extension do not write the report correctly into #208


Problem compiling with ionic 3

Hi,
I'm trying to compile my app (ionic 3 app), but I have this problem:
[01:34:15]  ngc started ...  [01:34:17]  ionic-app-script task: "build"  [01:34:17]  Error: Unexpected value 'AutoCompleteItemComponent in              /Users/claudio/Progetti/xxx/node_modules/ionic2-auto-complete/src/autocompleteitem.component.ts'              declared by the module 'AutoCompleteModule in              /Users/claudio/Progetti/xxx/node_modules/ionic2-auto-complete/index.ts'. Please add a              @Pipe/@Directive/@Component annotation.  Error: Unexpected value 'AutoCompleteItemComponent in /Users/claudio/Progetti/xxx/node_modules/ionic2-auto-complete/src/autocompleteitem.component.ts' declared by the module 'AutoCompleteModule in /Users/claudio/Progetti/xxx/node_modules/ionic2-auto-complete/index.ts'. Please add a @Pipe/@Directive/@Component annotation.     at Error (native)     at syntaxError (/Users/claudio/Progetti/xxx/node_modules/@angular/compiler/bundles/compiler.umd.js:1550:34)     at /Users/claudio/Progetti/xxx/node_modules/@angular/compiler/bundles/compiler.umd.js:14552:40     at Array.forEach (native)     at CompileMetadataResolver.getNgModuleMetadata (/Users/claudio/Progetti/xxx/node_modules/@angular/compiler/bundles/compiler.umd.js:14534:54)     at addNgModule (/Users/claudio/Progetti/xxx/node_modules/@angular/compiler/bundles/compiler.umd.js:23050:58)     at /Users/claudio/Progetti/xxx/node_modules/@angular/compiler/bundles/compiler.umd.js:23061:14     at Array.forEach (native)     at _createNgModules (/Users/claudio/Progetti/xxx/node_modules/@angular/compiler/bundles/compiler.umd.js:23060:26)     at analyzeNgModules (/Users/claudio/Progetti/xxx/node_modules/@angular/compiler/bundles/compiler.umd.js:22935:14)
with ionic serve it works perfectly, but when I try to compile with ' ionic build android --prod --release ' I receive that error
this is my ionic info:
`Your system information:
Cordova CLI: 6.5.0
Ionic Framework Version: 3.2.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.3.7
ios-deploy version: 1.9.1
ios-sim version: 4.1.1
OS: macOS Sierra
Node Version: v5.8.0
Xcode version: Xcode 8.3.2 Build version 8E2002`
thank you in advice

Thanks, I will look into that ASAP

Fixed, see #20

ciao Mor,
thank you for this fix, but I'm sorry to inform you that there are other issues with ionic 3.
I made some tests, and I found this situation:

with @ionic/app-scripts >= 1.3.5 the build with --prod options fails with the following error:

Error: ./~/ionic2-auto-complete/index.ts Module build failed: TypeError: Cannot read property 'content' of undefined     at Object.optimizationLoader (/Users/claudio/Progetti/XXX-v2/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader-impl.js:14:24)     at Object.loader (/Users/claudio/Progetti/XXX-v2/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader.js:5:32)  @ ./src/app/app.module.ngfactory.js 33:0-55  @ ./src/app/main.ts

with @ionic/app-script < 1.3.5 (and > 1.3.1) the build goes ok, but when I start my app I see a white screen and, inside the logcat, I have this log line:

D SystemWebChromeClient: file:///android_asset/www/build/main.js: Line 42 : Uncaught Error: Module build failed: Error: ENOENT: no such file or directory, open '/Users/claudio/Progetti/XXX-v2/node_modules/ionic2-auto-complete/index.js' 05-15 22:25:05.196  3277  3277 I chromium: [INFO:CONSOLE(42)] "Uncaught Error: Module build failed: Error: ENOENT: no such file or directory, open '/Users/claudio/Progetti/XXX-v2/node_modules/ionic2-auto-complete/index.js'", source: file:///android_asset/www/build/main.js (42)
I attach you my package.json
package.json.txt
This is my updated ionic info:
`$ ionic info
global packages:
@ionic/cli-utils : 1.0.0
Cordova CLI      : 7.0.1
Ionic CLI        : 3.0.0

local packages:
@ionic/app-scripts              : 1.3.4
@ionic/cli-plugin-cordova       : 1.0.0
@ionic/cli-plugin-ionic-angular : 1.0.0
Ionic Framework                 : ionic-angular 3.2.1

System:
Node       : v7.10.0
OS         : macOS Sierra
Xcode      : Xcode 8.3.2 Build version 8E2002
ios-deploy : 1.9.1
ios-sim    : 4.1.1 `

Tell me if you need more informations or if I could help you
thank you

Thanks for reporting, I hope to take a look at it soon, do you have any ideas anyway?

A similar problem:
ionic cordova run android --prod --release
Running app-scripts build: --prod --address <IP_ADDRESS> --port 8100 --p 8100 --livereload-port 35729 --r 35729 --iscordovaserve --externalIpRequired --nobrowser
[22:23:23]  build prod started ...
[22:23:23]  clean started ...
[22:23:23]  clean finished in 2 ms
[22:23:23]  copy started ...
[22:23:23]  ngc started ...
[22:23:28]  ngc finished in 5.06 s
[22:23:28]  preprocess started ...
[22:23:28]  deeplinks started ...
[22:23:28]  deeplinks finished in 228 ms
[22:23:28]  optimization started ...
[22:23:28]  copy finished in 5.49 s
Error: ./~/ionic2-auto-complete/index.ts
Module build failed: TypeError: Cannot read property 'content' of undefined
at Object.optimizationLoader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader-impl.js:14:24)
at Object.loader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader.js:5:32)
@ ./src/app/app.module.ngfactory.js 30:0-55
@ ./src/app/main.ts,./~/ionic2-auto-complete/src/autocomplete.component.ts
Module build failed: TypeError: Cannot read property 'content' of undefined
at Object.optimizationLoader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader-impl.js:14:24)
at Object.loader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader.js:5:32)
@ ./src/pages/map/map.ngfactory.js 19:0-76
@ ./src/app/app.module.ngfactory.js
@ ./src/app/main.ts,./~/ionic2-auto-complete/src/autocompleteitem.component.ts
Module build failed: TypeError: Cannot read property 'content' of undefined
at Object.optimizationLoader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader-impl.js:14:24)
at Object.loader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader.js:5:32)
@ ./~/ionic2-auto-complete/src/autocomplete.component.ngfactory.js 9:0-79
@ ./src/pages/map/map.ngfactory.js
@ ./src/app/app.module.ngfactory.js
@ ./src/app/main.ts,./~/ionic2-auto-complete/src/boldprefix.pipe.ts
Module build failed: TypeError: Cannot read property 'content' of undefined
at Object.optimizationLoader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader-impl.js:14:24)
at Object.loader (/home/fixidea/projects/broker/node_modules/@ionic/app-scripts/dist/webpack/optimization-loader.js:5:32)
@ ./~/ionic2-auto-complete/src/autocompleteitem.component.ngfactory.js 8:0-68
@ ./~/ionic2-auto-complete/src/autocomplete.component.ngfactory.js
@ ./src/pages/map/map.ngfactory.js
@ ./src/app/app.module.ngfactory.js
@ ./src/app/main.ts
fixidea@zalman:~/projects/broker$ ionic info
global packages:
@ionic/cli-utils : 1.0.0
Cordova CLI      : 6.5.0
Ionic CLI        : 3.0.0

local packages:
@ionic/app-scripts              : 1.3.7
@ionic/cli-plugin-cordova       : 1.0.0
@ionic/cli-plugin-ionic-angular : 1.0.0
Ionic Framework                 : ionic-angular 3.1.1

System:
Node       : v6.10.2
OS         : Linux 4.10
Xcode      : not installed
ios-deploy : not installed
ios-sim    : not installed

@claudiomoratti @FiXIDeA I have a fix! please stay patient a little more :)

great! don't worry, If you need tests feel free to write me

@claudiomoratti @FiXIDeA
Would you mind trying this one: npm install --save<EMAIL_ADDRESS>I haven't commited yet because I want to know if I'm on the right path.
I tested on my ubuntu and it seemed fine, can you please lete me know if it works for you too?

@kadoshms It works!
I have had to change the include line in app.scss to
@import "../../node_modules/ionic2-auto-complete/auto-complete";
'cause of new structure of sources.
I tested both with @ionic/app-scripts  1.3.7 and 1.3.4, with the same result!
thank you for your work

@claudiomoratti yeah I forgot to mention that sorry :)
Im closing the issue and will publish this fix soon.

@1.4.0-fix-to-38 did indeed fix the issue (+ changing the SSCC import)
When can we expect this to be committed into the master?
This branch was for 1.4.0, but the current version is 1.4.3

I am aware of that, it was an innocent typo.
I am working on a much more stable version with some more features, stay tight :)

@kamok thanks


compose fails to startup

On main branch. does this require python or anything else not listed in the readme?
C:\projects\zDapr\demo2>docker-compose up docker-compose-dapr.yml
Traceback (most recent call last):
  File "docker\api\client.py", line 214, in _retrieve_server_version
  File "docker\api\daemon.py", line 181, in version
  File "docker\utils\decorators.py", line 46, in inner
  File "docker\api\client.py", line 237, in _get
  File "requests\sessions.py", line 543, in get
  File "requests\sessions.py", line 530, in request
  File "requests\sessions.py", line 643, in send
  File "requests\adapters.py", line 439, in send
  File "urllib3\connectionpool.py", line 670, in urlopen
  File "urllib3\connectionpool.py", line 392, in _make_request
  File "http\client.py", line 1255, in request
  File "http\client.py", line 1301, in _send_request
  File "http\client.py", line 1250, in endheaders
  File "http\client.py", line 1010, in _send_output
  File "http\client.py", line 950, in send
  File "docker\transport\npipeconn.py", line 32, in connect
  File "docker\transport\npipesocket.py", line 23, in wrapped
  File "docker\transport\npipesocket.py", line 72, in connect
  File "docker\transport\npipesocket.py", line 52, in connect
pywintypes.error: (2, 'CreateFile', 'The system cannot find the file specified.')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker-compose", line 3, in <module>
  File "compose\cli\main.py", line 81, in main
  File "compose\cli\main.py", line 200, in perform_command
  File "compose\cli\command.py", line 60, in project_from_options
  File "compose\cli\command.py", line 152, in get_project
  File "compose\cli\docker_client.py", line 41, in get_client
  File "compose\cli\docker_client.py", line 170, in docker_client
  File "docker\api\client.py", line 197, in __init__
  File "docker\api\client.py", line 221, in _retrieve_server_version
docker.errors.DockerException: Error while fetching server API version: (2, 'CreateFile', 'The system cannot find the file specified.')
[34868] Failed to execute script docker-compose

additional context:
C:\projects\zDapr>docker-compose --version
docker-compose version 1.29.2, build 5becea4c

C:\projects\zDapr>docker --version
Docker version 20.10.14, build a224086

C:\projects\zDapr>dotnet --list-sdks
3.1.120 [C:\Program Files\dotnet\sdk]
3.1.421 [C:\Program Files\dotnet\sdk]
5.0.403 [C:\Program Files\dotnet\sdk]
5.0.408 [C:\Program Files\dotnet\sdk]
6.0.203 [C:\Program Files\dotnet\sdk]

this appears to happen when you have docker installed, but not running