Different behavior between CLI and pre-commit hook

This is my configuration .isort.cfg
[settings]
line_length=100
multi_line_output=3
include_trailing_comma=True

when I run pre-commit run --all-files
my modules look like this
import os

import pandas as pd
import mylib

no space is added from my local lib and third party ones.
If I run from command line isort -rc ., the file is changed to:
import os

import pandas as pd

import mylib

adding a space between third-party and local modules.
Any idea how to get a consistent behavior?

I fixed the problem using seed-isort-config before running isort in pre-commit:
  - repo: https://github.com/asottile/seed-isort-config
    rev: v2.2.0
    hooks:
    -   id: seed-isort-config

  - repo: https://github.com/timothycrosley/isort
    rev: 4.3.21
    hooks:
    - id: isort


Allow buck http cache verify http header

Right now buck http cache didn't do any auth check against http header. To ensure we are safe from others uploading buggy artifacts, this PR is adding the logic to verify the header.
This PR includes:

config change to introduce authorizationConfig, so that user can send the authorizedTokens over
introduce HttpHeaderAuthFilter  to do http header authentication, what it does is extract the Authorization from Http header, and verify if that contains in authorizedTokens from config
change BuckCacheApplication to register the HttpHeaderAuthFilter so that it could be used
change BuckCacheResource to execute the authentication for PUT request
update existing and new unit test to verify the change

To use that, it require:

add authorized token into http_write_headers in .buckconfig, e.g.

[cache]
  mode = http
  http_mode = readwrite
  http_url = http://localhost:6457
  http_write_headers = Authorization: ZAYHERO

add the token (i.e. ZAYHERO) into standalone.yml, e.g.

authorizationConfig:
  authorizedTokens:
    - ZAYHERO

@dhaval2025 @kageiit Please take a look

Can we change Authorization to be called Authentication instead, in the header?

Minor renames, but looks good otherwise. Please make the changes. I will approve and merge after the renames

@kageiit make further renames to s/authenticatedTokens/tokens , please take a look

LGTM, but would like @dhaval2025 to make a sanity check

@kageiit maybe check both headers then? Buck documentation uses Authorization in their examples: https://buckbuild.com/concept/buckconfig.html#cache.http_write_headers

looks good. Merging it in ..


Drop support for Node v0.12, begin v2.0

bump version to v2.0.0-0
drop Node v0.12 from travis and package.json

No actual breaking changes here, but get ready.
Fixes: https://github.com/IBM-Bluemix/gp-js-client/issues/55

Coverage remained the same at 89.938% when pulling 67b80896d6de398492cdc1a742d879532e517fb4 on srl295:drop-v12 into b21755cfd94214b58238a64a4a90171f3541c197 on IBM-Bluemix:master.


[IOTDB-2075] Accelerate the process of insertTablets by using thread pool

see https://issues.apache.org/jira/browse/IOTDB-2075
test result https://cwiki.apache.org/confluence/pages/resumedraft.action?draftId=195728781&draftShareId=fe6ddf54-ae5e-4ab6-a206-f5d27bde6535&
This not finally version ...

Coverage increased (+0.005%) to 67.327% when pulling 866170bd05583017442b4fbd9292192d5c0cef42 on jun0315:accelerate_insert_tablets into cbb09d0b99fa9317de06a277c4ee393a577029e9 on apache:master.

test result:
tablet count	use serial	use thread pool
100	169	166
1000	378	308
5000	1044	918
10000	1375	1497
50000	7648	7172
Because the speed of insertTable is very fast, about 0.15ms, there is no acceleration effect. In order to verify this conjecture,
insertTable sleep 10 ms
tablet count	use serial	use thread pool
100	1206	216
1000	12310	1167
5000	59562	5177
test device:
MacBook Pro (16-inch, 2019)
2.6 GHz 6-core processor Core i7
16 GB 2667 MHz DDR4

Test Result

Serialization
2 corePoolSize
4 corePoolSize
6 corePoolSize
8 corePoolSize
10 corePoolSize
12 corePoolSize

1000 Plans 48 rows  16 columns
1397
1292
1160
1040
1085
1089
1102

We can find that the efficiency is the highest when we take Runtime.getRuntime().availableProcessors()/2.
Next, we default to 1000 Plans 48 rows and change the different columns.

Serialization
6 corePoolSize

4 columns
901
1023

8 columns
958
1068

12 columns
1196
1259

14 columns
1278
1239

15 columns
1215
1001

16 columns
1364
1014

We can find that when there are 15 columns and 16 columns, the thread pool speed will exceed the serial speed, so we set the threshold to 15.
To sum up, we set the number of core threads in the thread pool to min(the number of different sg, availableProcessors()/2), and need columns >= 15
Test Device
MacBook Pro (16-inch, 2019)
2.6 GHz 6-core processor Core i7
16 GB 2667 MHz DDR4

Since the default maximum row is 10000, we redo the following experiment with row = 10000
Test Result

Serialization
2 corePoolSize
4 corePoolSize
6 corePoolSize
8 corePoolSize
10 corePoolSize
12 corePoolSize

1000 Plans 10000 row 16 column
1952
1906
1742
1608
1736
1670
1641

We can also find Runtime.getRuntime().availableProcessors() = 12 on known test machine
We can find that the efficiency is the highest when we take Runtime.getRuntime().availableProcessors()/2.

Serialization
6 corePoolSize

1000Plans 10000row 8 column
710
809

1000Plans 10000row 10 column
1018
960

1000Plans 10000row 12 column
1912
1717

We can find that when there are 10 columns , the thread pool speed will exceed the serial speed, so we set the threshold to 10.
Therefore, we set the number of core threads in the thread pool to min(the number of different sg, availableProcessors()/2), and need columns >= 10
It should be noted that in the latest test, we found that if the number of sg is large and exceeds twice the recommended number of CPU threads, it may lead to failure to allocate out of heap memory and NPE. Therefore, we will also turn off multithreading in this case.
Test Device
MacBook Pro (16-inch, 2019)
2.6 GHz 6-core processor Core i7
16 GB 2667 MHz DDR4


error in nvm install 10.12.0

Operating system and version: Windows 10

nvm debug output:

 nvm install 10.12.0

nvm ls output:

Downloading node.js version 10.12.0 (64-bit)...
Complete
Creating C:\Users\yoo\AppData\Roaming\nvm\temp

Downloading npm version 6.4.1... Download failed. Rolling Back.
Rollback failed. remove C:\Users\yoo\AppData\Roaming\nvm\temp\npm-v6.4.1.zip: The process cannot access the file because it is being used by another process.
Could not download npm for node v10.12.0.
Please visit https://github.com/npm/npm/releases/tag/v6.4.1 to download npm.
It should be extracted to C:\Users\yoo\AppData\Roaming\nvm\v10.12.0

How did you install nvm? (e.g. install script in readme, Homebrew):
Download exe file

What steps did you perform?

What happened?
install node v10.12.0,and npm v6.4.1 couldn't install to this enviroment,give the message Please visit https://github.com/npm/npm/releases/tag/v6.4.1 to download npm. but npm is already moved to https://github.com/npm/cli.This is the problem.

What did you expect to happen?
install node v10.12.0 and npm v6.4.1

Is there anything in any of your profile files (.bashrc, .bash_profile, .zshrc, etc) that modifies the PATH?
no

You're using windows here - nvm only supports windows when using WSL/BashOnWindows - otherwise, you might be using nvm-windows, a different project.


[FLINK-27421][python] Bundle test utility classes

What is the purpose of the change
Bundle test utility classes into the PyFlink package to make users write test cases easily.
Brief change log

*set pyflink.testing package into setup.py *

Verifying this change
Change build script, no test case.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (xno)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

@flinkbot  run azure


Fix the memory access pattern and change it to coalesced memory access.

Fix the memory access pattern and change it to coalesced memory access.

Tested on my laptop which has 8 CUs. The performance is improved a bit in general.


[FEATURE] Server middlewares should have fine-grained control like client middlewares

Describe the feature
In Nuxt, I have 2 middlewares in /middlewares dir, namely native and admin and I can use them using the definePageMeta() macro in the pages that I prefer. I have a refined control over the client middlewares and which pages I want them to be in.

It would be great to have this refined control for server middlewares as well. In /server/middlewares, I should be able to define middlewares and use them for specific server routes and NOT for all server routes, which is the default behavior. I should also be able to chain middlewares, much like in the client side.
Additional information

[ ] Would you be willing to help implement this feature?

### Tasks

Hi. Thanks for opening issue and explaining but Nuxt middleware are Powered by Vue router and completely different than Nitro/Server middleware. Feel free to open issue in nuxt repository.


[UI] Theme colour should be consistent in setting page in dark mode

Current Behavior
Theme color should be consistent in setting page in dark mode as it is in light mode
https://user-images.githubusercontent.com/71271069/219146017-747e6aff-b655-4b9d-be4c-20b43a215590.mov
Desired Behavior

Screenshots/Logs

Environment

Browser: Chrome Safari Firefox
Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker Kubernetes

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@Yashsharma1911 Is it just this page, or are there others that may need this as well?

@acald-creator, you are correct. I'm wondering the same thing. In addition, @Yashsharma1911 I'd want to work on this issue.

Hey @shruti1421 thanks for raising your hand just to let you know that I am already working on it however there may be some other related issues that you could help with


SetLogger accepts generic interface, backwards-compatible with stdlib

func SetLogger(l stdLogger)  where stdLogger is a logger interface compatible with both stdlib and some 3rd party loggers such as logrus.

I am super interested in this, I have had to pass in an implementation io writer that called glog into a new stdlib logger to get by this. It will be better if I didn't have to do the hack

not sure how many of those methods we actually use. but +1.


cmd/get: add commands for GetVertex, GetEdge, etc.

Having commands like this makes development a lot easier, and allows users to quickly experiment.
@adamstruck I regret not going with   in funnel. What do you think? Should arachne stick with arachne vertex get or arachne get vertex

How about just arachne vertex?

Also a decent option, although it could be limiting if you want an additional vertex-related command in the future.

my vote would be for arachne vertex get.  This seems to be the pattern I see most often.

I am fine with scratching the original arachne vertex get like commands in favor of the more flexible  query interface.
It looks as though the javascript code ported from ophion is a bit stale and needs to be updated to match the python aql client.


Pay

A file has been shared using Link Sharing.
https://s.amsu.ng/B3qSg2oR8gDN
(Expires: Mar. 29, 2024)

A file has been shared using Link Sharing.
https://s.amsu.ng/B3qSg2oR8gDN
(Expires: Mar. 29, 2024)

https://github.com/coinbase/cbpay-js/issues/257#issuecomment-2041821037

A file has been shared using Link Sharing.
https://s.amsu.ng/B3qSg2oR8gDN
(Expires: Mar. 29, 2024)


[not bug] android blit type may make snapshot black

snapshot always show black
Steps to reproduce the behavior:

Set android blit type auto or none
Set graphic api to opengles3/2
build demo scene
See error

Expected behavior
normal blur bg
Environment (please complete the following information):

Version  1.0.0
Platform: Android
Unity version: 2019.4.0f1
Build options: blit type auto,opengles3

Additional context
Blit and ScreenCapture.CaptureScreenshotIntoRenderTexture neither can get correct color with that setting on android

has this bug fixed?

https://issuetracker.unity3d.com/issues/android-the-grabpass-does-not-work-when-the-blit-type-is-set-to-auto


Nested navigators in tabbars

I would love to understand how to use nested navigators in tabbars. I'm currently not sure how to do this and am wondering if that is because of a lack of documentation or lack of understanding from my side.
So basically, I would love to have tab1, tab2, tab3 and have scenes A, B, C that can be reachable from all tabs. If we do tab1 -> A -> B -> C -> B and then press tab2 -> A -> B ->C, pressing tab1 again should show the whole tab1 -> A -> B -> C -> B-stack. From there pressing tab2 will show the tab2 -> A -> B ->C-stack.
How should this be done?

For help topics its best to ask these sort of question on stackoverflow as there are allot more people there to help you out. But anyway, from the jist of your question you the views that you are wanting to be able accessed from all 3 tabs must still be inside the tablayout, ie meaning you can still see the tabbar if so the way i would personally do it would be having:
                     <Scene key={'tab1'} initial={true} component={Tab1}>                        <Scene key={'tab1A'} component={TabA}/>                        <Scene key={'tab1B'} component={TabB}/>                        <Scene key={'tab1C'} component={TabC}/>                     </Scene>                     <Scene key={'tab2'} component={Tab1}>                       <Scene key={'tab2A'} component={TabA}/>                        <Scene key={'tab2B'} component={TabB}/>                        <Scene key={'tab2C'} component={TabC}/>                     </Scene>                    <Scene key={'tab3'} component={Tab1}>                       <Scene key={'tab3A'} component={TabA}/>                        <Scene key={'tab3B'} component={TabB}/>                        <Scene key={'tab3C'} component={TabC}/>                     </Scene>
While you are having to duplicate the scene, in which there is probably a better way to do so. You reuse the components for A B and C.  But with different keys to call from each tab.
Like i said the way i would do it and there are probably better ways to do it.


Workspace.open() should support splitting up / down

The docs here...
https://atom.io/docs/api/v0.204.0/Workspace#instance-open
... says that Workspace.open supports the split option with arguments left or right.
In addition to this, I would like Workspace.open to support splitting up and down as well.
Needed by the Markdown preview pane for example:
https://github.com/atom/markdown-preview/issues/99
Regards /Johan

Here's the code that needs updating:
https://github.com/atom/atom/blob/ac1b7d4fae0443f5a423446c6bcb408a2da59636/src/workspace.coffee#L362

Agreed. Even just to have consistency.

This has been available since v1.2.0. Sorry for missing closing this :blush:


Client spring integration

Coverage increased (+0.5%) to 50.146% when pulling 61bb693637564ae4a3e64df900569c585d5cb558 on nobodyiam:client-spring-integration into 87129adc857c66414c11ba3a4b773a6a9740a3ac on ctripcorp:master.

Codecov Report

Merging #543 into master will increase coverage by 0.51%.
The diff coverage is 94.33%.

@@             Coverage Diff              @@
##             master     #543      +/-   ##
============================================
+ Coverage     46.32%   46.83%   +0.51%
- Complexity     1306     1340      +34
============================================
  Files           312      319       +7
  Lines          8469     8575     +106
  Branches        814      824      +10
============================================
+ Hits           3923     4016      +93
- Misses         4263     4276      +13
  Partials        283      283

Impacted Files
Coverage Δ
Complexity Δ

...ork/apollo/spring/config/ConfigPropertySource.java
100% <100%> (ø)
2 <2> (?)

.../spring/config/ConfigPropertySourcesProcessor.java
100% <100%> (ø)
2 <2> (?)

...pollo/spring/annotation/ApolloConfigRegistrar.java
100% <100%> (ø)
2 <2> (?)

...apollo/spring/config/PropertySourcesProcessor.java
100% <100%> (ø)
10 <10> (?)

...o/spring/annotation/ApolloAnnotationProcessor.java
100% <100%> (ø)
11 <11> (?)

...ework/apollo/spring/util/BeanRegistrationUtil.java
72.72% <72.72%> (ø)
3 <3> (?)

...amework/apollo/spring/config/NamespaceHandler.java
85% <85%> (ø)
3 <3> (?)

...mework/apollo/portal/component/PortalSettings.java
55.73% <ø> (-8.2%)
5% <ø> (ø)

...work/apollo/biz/message/DatabaseMessageSender.java
44.68% <ø> (-6.39%)
6% <ø> (ø)

... and 7 more

Continue to review full report at Codecov.

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

May I ask you a question?
I have a request:
When encapsulating the Apollo client SDK, it is necessary to determine whether the value corresponding to the key start with fields ‘>>>’ and end with fields '<<<' (encrypted markers) at getconfig time.
If there is an encryption token, the decrypted value is returned by calling the internal decryption SDK after load.
How to achieve this, which classes need to be rewritten.
可以问一个问题吗？
我有一个需求：
封装Apollo client SDK的时候需要在getconfig的时候判断key对应的value是否包含“>>>***<<<”字段（加密标记）
如果有加密标记，在load后通过调用内部解密SDK返回解密后的value。
这个如何实现，需要重写哪些class。

defaultConfig.getProperty(someKey, new Function<String, String>() {
            @Override
            public String apply(String s) {
                DecryptRequest request = new DecryptRequest();
                request.setEncryptedText(s);
                return HttpEncryptionWrapper.decrypt(request);
            }
        }, "defaultValue");

I am ready implement by AbstractConfig.getProperty(String key, Function<String, T> function, T defaultValue);
but do not know how to implement with @value Placeholder/XML Placeholder/Spring Annotation.
that devs just import xxx-company-apollo-client.jar and nothing need to modify.

Currently there is no hook to support this feature, I think apollo-client should provide such capability in the future.

Currently there is no hook to support this feature, I think apollo-client should provide such capability in the future.

I hook it with overwrite some classes.
DefaultInjector -> XXXInjector
DefaultConfigManager -> XXXConfigManager
DefaultConfigFactoryManager -> XXXConfigFactoryManager
DefaultConfigRegistry -> XXXConfigRegistry
DefaultConfigFactory -> XXXConfigFactory

overwrite file:
path:{rootMenu}/META-INF/services/com.ctrip.framework.apollo.internals.Injector
content(namespace of XXXInjector):com.{xxxcompany}.{xxxproductname}.{xxxapolloclient}.{XXXInjector}
Is that the right way to overwrite Default behavior？

How to overwrite apollo default behavior and add decode function(define customer apollo client)?

Solution 1:
step 1. overwrite class

{XXX}Injector implements Injector
{XXX}ConfigFactory implements ConfigFactory
{XXX}Config extends AbstractConfig implements RepositoryChangeListener

step 2. overwrite file:
path:{rootMenu}/META-INF/services/com.ctrip.framework.apollo.internals.Injector
content(namespace of XXXInjector):com.{xxxcompany}.{xxxproductname}.{xxxapolloclient}.{XXXInjector}
PS：app server nothing have to modified.

Solution 2:
step 1. define class

{XXX}PropertySourcesPlaceholderConfigurer extends PropertySourcesPlaceholderConfigurer

step 2. define class
{XXX}ApolloConfigRegistrar implements ImportBeanDefinitionRegistrar

step 3. define customer annotation：
@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.TYPE)
@Documented
@Import({XXX}ApolloConfigRegistrar.class)
public @interface Enable{XXX}ApolloConfig {
    String[] value() default {ConfigConsts.NAMESPACE_APPLICATION};
    int order() default Ordered.HIGHEST_PRECEDENCE;
}

step 4(PS). use @Enable{XXX}ApolloConfig instead of @EnableApolloConfig in app server.


Force-stop descriptions

Description
This PR...

Fixes: #6798

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[x] api doc

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[x] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

as far as I know, the vm will be destroyed (force-stopped) on the kvm host as well.
not sure how it works on vmware/xenserver.

as far as I know, the vm will be destroyed (force-stopped) on the kvm host as well. not sure how it works on vmware/xenserver.

It will be tried, but if it fails it will be marked destroyed anyway. That is the intention of the flag.

as far as I know, the vm will be destroyed (force-stopped) on the kvm host as well. not sure how it works on vmware/xenserver.

It will be tried, but if it fails it will be marked destroyed anyway. That is the intention of the flag.

ok @DaanHoogland
can you copy the description in StopVMCmd.java to other java classes ?

as far as I know, the vm will be destroyed (force-stopped) on the kvm host as well. not sure how it works on vmware/xenserver.

It will be tried, but if it fails it will be marked destroyed anyway. That is the intention of the flag.

ok @DaanHoogland can you copy the description in StopVMCmd.java to other java classes ?

Yes, I have searched for the forced flag and alligned four classes here. Which others did I miss?

as far as I know, the vm will be destroyed (force-stopped) on the kvm host as well. not sure how it works on vmware/xenserver.

It will be tried, but if it fails it will be marked destroyed anyway. That is the intention of the flag.

ok @DaanHoogland can you copy the description in StopVMCmd.java to other java classes ?

Yes, I have searched for the forced flag and alligned four classes here. Which others did I miss?

@DaanHoogland no more classes I think.
can you add (vm is marked as Stopped even when command fails to be send to the backend, otherwise a force poweroff is attempted). to the descriptions ?

gha should be enough @weizhouapache ?

gha should be enough @weizhouapache ?

yes @DaanHoogland


support resource to be exported and imported

Ark should define standard rules  based on which resources would be exported and imported gracefully.

Add Header in Manifest: export-resources and import-resources  config with resource path, if there are multiple resources need to be exported, use ; to split.
import-resources  only work for Ark-plugin, for ark biz, all export-resources are automatically imported just as classes/packages.
getResource logic:

find resource on current classloader
If not find, check whether the resource is imported
If yes, then find export resource; If there are multiple, find the highest order resource (priority the same as ark plugin which export this resource)

getResources logic:

find resources on current classloader
If not find, check whether the resource is imported
if yes, find all resources being exported


crash

Fatal error: Could not check compatibility between iRainDrop\CPS\Commands\CPSCommand::getOwningPlugin(): iRainDrop\CPS\Commands\Plugin and pocketmine\plugin\PluginOwned::getOwningPlugin(): pocketmine\plugin\Plugin, because class iRainDrop\CPS\Commands\Plugin is not available in phar:///home/container/plugins/CPS (2).phar/src/iRainDrop/CPS/Commands/CPSCommand.php on line 18

What did you do right before this error happened?

I have the same error and we just restart the server and it crashes

i just install and start the server

What did you do right before this error happened?

It should be fixed in the latest release, sorry for the wait.

It should be fixed in the latest release, sorry for the wait.

np


How to use LIKE Operator in Realm at the time of query.

I am getting problem related to LIKE operator, at the time of Queries.
I have this query in sqlite3:
NSString *sqlQuery = [NSString stringWithFormat:@"SELECT * FROM ServiceAccountsTbl WHERE      serviceModes LIKE '%%%@%%'",serviceMode];
Query of above sqlQuery :
SELECT * FROM ServiceAccountsTbl WHERE serviceModes LIKE '%Email%'
Equivalent of above query, I have tried in Realm as follows:
RLMResults *allResults;
NSPredicate *pred = [NSPredicate predicateWithFormat:@"serviceModes LIKE[c] %@",serviceMode];
allResults = [ServiceAccountsModel objectsWithPredicate:pred];
But i am not getting success, please help.

LIKE is not supported yet as operator in NSPredicates by Realm, as seen in #1490. In general please refer to our predicate cheatsheet for an overview of supported operators.
For your specific query here, it seems like you could easily workaround this limitation by falling back to using just the CONTAINS operator like that:
[NSPredicate predicateWithFormat:@"serviceModes CONTAINS %@", serviceMode];


通过网络地址打开pdf ， 加载成功了显示不出来pdf

Hello,
thanks for using the library.
Could you translate the issue to English please? Provide as much details as possible, such as device manufacturer, model, Android version, and steps to reproduce the problem.

@voghDev his means that when use network,the pdf does not show.

Ok. The first thing I would try would be to replace the sample app included in this project with a link to your repository, and see if the RemotePdfViewPager is able to load it.
Other thing I'd try is to manage the download by myself (outside the library) using an AsyncTask or libraries like OkHttp or Retrofit, and when the Pdf is downloaded and copied to your SD card, try to open it with a normal PdfViewPager

closing this due to inactivity


Avoid retrieving pack PDSC file when removing a pack

Fix https://github.com/Open-CMSIS-Pack/cpackget/issues/82
It was taken as granted that if a pack PDSC isn't found in .Web/, cpackget would attempt to read the index.pidx file and download it.
So all public packs installed via cpackget will consequently have the PDSC file in .Web/. Nevertheless, there might be use cases where packs were installed elsewhere and the PDSC file isn't present. When removing such packs, cpackget would still attempt to retrieve the PDSC, which does not seem necessary.

Such a use case is that someone copies the content of a pack version manually into the corresponding folder without using cpackget. Also a non-public pack could have been installed and the corresponding pdsc file deleted from the .Local directory.


CMake Error at third_party/boringssl/CMakeLists.txt:334

I got below error when executing Cargo build in grps-rs
CMake Error at third_party/boringssl/CMakeLists.txt:334 (message):
  Unknown processor:[proxychains] DLL init

  [proxychains] DLL init

  x86_64

thread 'main' panicked at '
command did not execute successfully, got: exit code: 1

build script failed, must exit now', /home/lianfeng.lu/.cargo/registry/src/github.com-1ecc6299db9ec823/cmake-0.1.40/src/lib.rs:832:5
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

warning: build failed, waiting for other jobs to finish...
error: build failed

Seems it is caused by proxychains ,  which is the proxy setting of my development machine . How can i resolve it ?
By the way ,  how can i disable secure  feature(ssl/boringssl ) of grpc-rs ? it need go-lang , i do not want use it . Because it causes a lot of compilation errors .

I have no idea about proxychains, but you can follower the instructions here to disable the secure feature.

Ok， thank you. If i use ssl , how should i set the GOPATH ? I get a lot of compilation errors when compiling boringssl in grpc-rs/grpc-sys/grpc/third_party/boringssl

I resolved it . Thank you .
I got another error when compiling examples cargo build --example greeter_server
The error message is
error[E0599]: no function or associated item named `default_instance` found for type `dyn testing::ByteBufferParams_MESSAGE::_prost::Message` in the current scope
 --> /home/lianfeng.lu/grpc-rs/target/debug/build/grpcio-proto-57c7c982eabb2f91/out/wrapper_grpc.testing.rs:9:27
  |
9 |         ::prost::Message::default_instance()
  |         ------------------^^^^^^^^^^^^^^^^
  |         |
  |         function or associated item not found in `dyn testing::ByteBufferParams_MESSAGE::_prost::Message`

error[E0599]: no function or associated item named `default_instance` found for type `dyn testing::ByteBufferParams_MESSAGE::_prost::Message` in the current scope
   --> /home/lianfeng.lu/grpc-rs/target/debug/build/grpcio-proto-57c7c982eabb2f91/out/wrapper_grpc.testing.rs:108:27
    |
108 |         ::prost::Message::default_instance()
    |         ------------------^^^^^^^^^^^^^^^^
    |         |
    |         function or associated item not found in `dyn testing::ByteBufferParams_MESSAGE::_prost::Message`

error[E0599]: no function or associated item named `default_instance` found for type `dyn testing::ByteBufferParams_MESSAGE::_prost::Message` in the current scope
   --> /home/lianfeng.lu/grpc-rs/target/debug/build/grpcio-proto-57c7c982eabb2f91/out/wrapper_grpc.testing.rs:207:27
    |
207 |         ::prost::Message::default_instance()
    |         ------------------^^^^^^^^^^^^^^^^
    |         |
    |         function or associated item not found in `dyn testing::ByteBufferParams_MESSAGE::_prost::Message`

Did I do anything wrong ?

You did -- run cargo update can solve your problem.

Thank you!

Close issue if you have no more questions.


Enable Interface dispatch for WASM

@morganbr thanks for solving the conv, vtable base and init problems. Here's my changes for interface dispatch. I think it still needs null checks before callvirt is really done. It might be a good idea to make some kind of LLVM infrastructure for injecting simple exception throwing checks inline. This would also apply nicely to the IndexOutOfRangeException that needs to be thrown in the array access code.

The WIP check is wrong (WIP is in one of the commit messages, but this change is ready to go).

Thanks, @hippiehunter! This should be the last major codegen feature we need to run positive test cases!


Scif changes from prototype

Description

OpenDroneMap features are not part of this review
Converted from prototype to Alpha release
Types of changes

Now using existing transformer code (as opposed to rewriting all transformers)
Added a non-ODM (OpenDroneMap) workflow
Added Shapefile and BETYdb to GeoJSON conversion apps
Extended workflow apps to automatically perform Shapefile/BETYdb to GeoJson conversion
Better identification of output folder
End to end testing
Checklist:

[x] I have read the documentation on CONTRIBUTING
[x] My updates require a change to the documentation
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Time frame for completing Pull Request review (from creation date):

[x] As soon as possible
[ ] In the next two weeks
[ ] In the next month

Focus of Pull Request:
All comments are welcome regardless of which items are checked off.

[x] All of the below (no need to check off the other boxes)
[ ] There are no errors or inconsistencies
[ ] Everything appears clear and easy to understand
[ ] Suggest improvements
[ ] Identify areas where functionality and/or documentation is missing

Hints to reviewers:

Anything OpenDroneMap (ODM) is out of scope for this review
There are several issues outstanding for finishing up the SciF Docker image

re:

Recommend using "shellcheck" to check bash programs:

would an 'all' inclusive tool like super-linter https://github.com/github/super-linter be worth adopting?

@kyclark Some feedback on you comments

Per PEP8 (https://www.python.org/dev/peps/pep-0008), lines of code should be maximum 79 characters in length

There are other conventions (PEPs) that are being overridden aside from this one in our pylint.rc file and it's use described in the Organization-Info README.md file. I find this PEP strange given how wide modern screens are. If we should revisit the pylint.rc file overrides please arrange discussions for it, or let me know that you'd like to (re)visit it and I can set it up
For ArgParse section, first set of recommendations:

<1> The add_arguments() function mutates...

Is there a standard or widely used convention that indicates this is not to be done? Changing read/write parameters is quite standard in many languages.
For ArgParse section, second set of recommendations:

<4> Verify that args.betydb_url is something

I believe that one can write a customized validator for parameters and do away with the post-parse checking
Later...

If the betydb_url parameter is required...

If you look at the code you can see that it's not a required parameter; the value is fetched from the environment if it's not specified on the command line. This allows secret information to not be available as plain text on the command lines.

@dlebauer @kyclark

Recommend using "shellcheck" to check bash programs...

I'm adding an issue to investigate shell linters and have a recommendation: https://app.zenhub.com/workspaces/ua-ag-data-science-5a57a3198339f11ba1c85775/issues/agpipeline/issues-and-projects/178

Re: screen width
I code exclusively in terminal windows open to 80 cols no matter my screen width, but it's not just a matter of personal preference. Books, magazines, and newspapers regularly put text into narrow columns regardless of the width of the page. I don't think I'm alone in thinking that no line of text should exceed 80 characters (https://baymard.com/blog/line-length-readability, https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3734020/, https://www.freecodecamp.org/news/long-code-vs-short-code/, https://www.blog.theteamw.com/2009/10/26/100-things-you-should-know-about-people-2-line-length/, https://www.smashingmagazine.com/2014/09/balancing-line-length-font-size-responsive-web-design/), but I'm not willing to turn this into a battle. I just think all text -- whether code or comments -- should be restricted to some length.
Re: mutating values
You and I have wildly different coding styles. You prefer OOP and mutating objects, I prefer a functional style (FP) that aspires as much as possible to "pure" functions and immutable data structures. A pure function works only on the parameters explicitly passed (i.e., never uses globals or the environment or some hidden state in an object) and always returns the same value. Mutating a variable that is passed as an argument to a function definitely goes against most principles of FP. The "parser" object in question is not used beyond instantiating it and passing it as an object to be modified by the add_arguments() function, so why not entirely hide it within the function? You're right, however, in that this comment does get at a much bigger point. I would argue that functions should (generally) never mutate their arguments unless there's some good reason. Python's pass-by-reference/value mode is very implicit and could lead to subtle bugs for programmers who do not understand it.
Re: betydb_url
The betydb_url is ultimately required to be defined somewhere either as an argument or in the environment. I was just trying to show how the resolution of this value could be moved into the same scope as any other argument rather than validating it deeper in the program and using an exception to halt the program. By resolving it earlier and using "parser.error()" to halt the program, you get the usage for the user.

@kyclark Thanks for your feedback.

...but I'm not willing to turn this into a battle...

I quite agree that there's no need to turn this into a battle. There's quite a few opinions out there for various fixed column widths and any side can find something to justify their stance.
As with Agile and other processes, I feel we need to create an environment that fits our needs. Along those lines, I welcome a review of decisions made regarding departing from the PEP recommendations (as encompassed by the pylint.rc file). Any suggestions to help code acceptance and ease of development are welcome. I am not the final arbiter on what's done, although I do have an impact on decisions, as do others in the group. If you feel that something should be brought up to the group as a whole, you can do that, or prompt me to do that. My only request is to keep the suggestions as narrowly focused as possible.
I also agree with the following two statements and think that the first one should be implemented.

The "parser" object in question is not used beyond instantiating it and passing it as an object to be modified by the add_arguments() function, so why not entirely hide it within the function?

and

Python's pass-by-reference/value mode is very implicit and could lead to subtle bugs for programmers who do not understand it

I do feel that if someone is programming in Python, they need to be aware of what they're doing & in case they aren't, unit and integration (etc) testing should catch the problems
Finally:

Re: betydb_url

I missed that nuance as described in your comment, thanks for clarifying!

Interesting discussions above - for the scope of the PR I think many comments can be addressed, though some may warrant moving to new issues and/or a list of ideas for future directions/changes.
I am also wondering where the sample data can be found, and where the end-to-end testing mentioned in the PR description is found (e.g. so @kyclark could write an integration test?).

I am also wondering where the sample data can be found

The files mentioned in this section can be found at: https://drive.google.com/file/d/1U-P4J2OcrNOkaLi6xCUblXOFet7V6raf/view?usp=sharing


Translation script doesn't work for Portuguese (pt-pt)

I tried to use the yarn translation scripts to make the deepl translation to portuguese (pt-pt), but was unable, here is the output:
/mnt/d/git/nosgestesclimat$ DEEPL_API_KEY=<deepl_auth_key> yarn translate:rules
yarn run v1.22.19
warning ../package.json: No license field
$ node scripts/i18n/translate-rules.js
Parsing rules of 'data/**/*.yaml'
Getting missing rule for en-us...
Found no new entry to translate...
Getting missing rule for pt-pt...
Translating 2055 new entries to pt-pt...
pt-pt | 2053/2055 | ████████████████████████████████████████ | Translating alimentation . boisson . sucrées . facteur...

[SKIPPED] - actions:
Glossary not found, message: Not found
[SKIPPED] - actions pétrole:
Glossary not found, message: Not found
[SKIPPED] - alimentation . réduire viande . par deux:
Glossary not found, message: Not found
[SKIPPED] - alimentation . réduire viande . par deux:
Glossary not found, message: Not found
[SKIPPED] - alimentation . réduire viande . par deux . par semaine:
Glossary not found, message: Not found
...

This happens both in the nosgestesclimat, and the nosgestesclimat-site repos.
After further investigation, I found out that currently DeepL doesn't support the portuguese lang in the glossary.
Is it possible to skip the glossary and do the translation directly?

Thanks for the report ! Yes, I think the drawbacks of not using the glossary are not primordial, we can live without it if it enables a new language :)

Also related : https://github.com/datagir/nosgestesclimat-site/issues/583

For now, DeepL's glossary is limited in pair of languages supported. Consequently, I think we need to implement ourself a glossary step in our translation workflow.


Update Composer dependencies (2021-10-22-00-04)

Visual regression test passed!
View the visual regression test report


Serialisation fails on some operations.

I'm seeing some Unable to serialize transaction errors on certain operations. In my case I tested this with 'custom_json' for 'follows' and 'update_proposals'
Here is a gist with some test operation which has a succesful result for a 'vote'-operation but fails for those other 2 operations: https://gist.github.com/roelandp/1a1982d813363832e9ae792a25245c37
I cannot pull my finger around as to why they are failing.

It seems you are missing some operation parameters.
required_auths in custom_json and extensions in update_proposal_votes.
const op2 = [
  'custom_json',
  {
    id: 'follow',
    required_auths: []
    required_posting_auths: [hiveuser],
    json:
      '["follow",{"follower":' +
      hiveuser +
      ',"following":"jasonliberty","what":["blog"]}]'
  }
]

const op3 = [
  [
    'update_proposal_votes',
    {
      voter: hiveuser,
      proposal_ids: ['42'],
      approve: 'true',
      extensions: []
    }
  ]
]

you nailed it, again. Thx.
I was just fiddling with the serializer source to output the cause which indeed told me these missing keyvals.
thanks, closing!


add custom root path folder

Added opportunity set custom root folder path while creating component.

:+1:


Lands Compatibility Request

The output of command /version on my server is: >.... This server is running Paper version git-Paper-372 (MC: 1.15.2) (Implementing API version 1.15.2-R0.1-SNAPSHOT
The output of command /version citizens on my server is: >.... Citizens version 2.0.27-SNAPSHOT (build 2067)
The output of command /version sentinel on my server is: >.... Sentinel version 2.1.5-SNAPSHOT (build 360)
A pastebin.com link to my (full!) Sentinel/config.yml file is:  https://pastebin.com/n4HqPv0y
Can we can get compatibility for Lands like we have with Massivecore Factions so that sentinel NPCs can target members of certain Lands and ignore members of certain Lands?

Thaaaat is a premium-only plugin. Sentinel cannot internally support plugins that cannot be publicly redistributed alongside Sentinel.
The developer of that plugin could add integration into Sentinel from their end if they're willing to.


Ability to export the local files of a single scenario

What would that feature address
This feature would enable the user to export all the files needed for a scenario without exporting all the .data. Sometimes, just one or a small group of scenarios need to be exported.
Description of the ideal solution
A Python line of code and/or an API Rest command line should be able to give all the files of the local system concerning a scenario
Proposal
Having a Python command to export a scenario : taipy.export("folder_name", "scenario_id")
The result would be the following folder hierarchy:
folder_name
   - cycles
        - cycle_id.json
   - datanodes
        - datanode_id1.json
        - datanode_id2.json
        ...
   - jobs
        - job_id1.json
        - job_id2.json
        - job_id3.json
        ...
   - pipelines
        - pipeline_id1.json
        - pipeline_id2.json
   - scenarios
        scenario_id.json
   - tasks
        - task_id1.json
        - task_id2.json

When the folder_name already exists, the call of taipy.export() should delete the folder before recreating it.

@FlorianJacta Do we want to export pickle files as well ? I would say no! Do you agree?

If a MongoRepository or a SQLRepository is used, we still want to export the scenario as JSON files.


Root object pose visualization

First of all, very appreciate your time and effort!! This work is pretty solid.
Just have one quick question: How can I visualize the root object poses during optimization (epoch 0~N) as in BANMo repo?

Hi, there is a script to do so in the next commit in a few days.

Thanks! It works.
Just so you know that pyrender is out of the original environment.yml.


Create live script to run OpticalCritical example

[ ] For speed of execution, could reduce the number of incidence angles to e.g. 5. This would need to be presented differently rather than the existing image plot
[ ] Add live script controls for user to change lambda (stripe spacing), and depth (h)

@zan8simone
Could you please clarify how lambda is represented in the code? I think maybe the terminology is different to the paper you showed? It is f.a from fig 1.a of the manual?

Hi, what is "lambda" in the paper, is named "a" in the manual and "a" in the code.

The original 55 incidence angles takes around 42 seconds to run on my machine, whereas 5 angles takes ~3s. I've plotted these 5 angles using the original image plot, and again as a tiled layout. What are your thoughts as to the best way to display the results?

Perhaps there could be a threshold for plotting < 10(?) angles as a tiled layout, and more than that as an image plot?
Either way I think the plotting code might be best separated out of the optical_critical function


cleaner.go:90:41:1 other occurrence(s) of "Proper cluster ID" found in: cleaner.go:136:38

cleaner.go:90:41:1 other occurrence(s) of "Proper cluster ID" found in: cleaner.go:136:38

Fixed in https://github.com/RedHatInsights/insights-results-aggregator-cleaner/pull/47


[chore] Fix svc type of meshery-operator

Signed-off-by: Pranav Singh<EMAIL_ADDRESS>Description
I had missed few files. Now it should be good.
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

I think this is a false positive - https://github.com/meshery/meshery/runs/7696833437?check_suite_focus=true#step:6:65

I wonder why there are two different versions here:
rewrite charts/meshery-0.5.0.tgz (99%)
[62](https://github.com/meshery/meshery/runs/7699039029?check_suite_focus=true#step:6:63)
 rewrite charts/meshery-operator-0.6.0.tgz (99%)


Add Dates to Compendia & Jobs

For internal housekeeping and also reference for users it would be nice to have meaningful timestamps/dates for Compendia & Jobs.
E.g.: Uploaded, Last run, Execution start, Execution end.
This would also aid in properly sorting the results of job?compendium_id=…, which right now is just returned in the order that they have been created (which leads to unnecessary pagination travel because the most recent job is the last item on the last page).

Discussed with @nuest : Add start and end times to each execution step (unpack bagtainer, build image, execute container, etc.)

This is being worked on in #12 .

solved with #12.


Alternative conditioning mechanism?

Great lean transformer LDM!
I'm wondering if you can make the conditioning mechanism more versatile. For example, allowing conditioning on tensors that are encoded before entering a cross-attention layer in the diffusion backbone? I think this would make the code even useful for wider applications.

Hi @metatl not sure I understand your question - you can condition on any tensor you want to here - https://github.com/apapiu/transformer_latent_diffusion/blob/2352f0051c40340bacd67920a56c7af91bfab93d/tld/transformer_blocks.py#L137 - is this what you would want?

I think you are right. I just need to encode the conditioning image into a tensor and pass it to the model.


corect request type enums

please check I've got the ErrorDetail.RequestType enums correct now.
Also weigh in on whether we should make this a breaking change (i.e. remove the deprecated ones). No one is using it atm but I don't really want to make a whole new major version just to correct some enums no one is using.


Measured parameters for DSWx-HLS

Description

Added code for the Measured parameters for DSWx-HLS

Affected Issues

N/A

Testing

All pytest completed successfully

The !Not found! check passed unit tests because I accidentally used !Not Found! in the augment function (capital F not lowercase)
Will refactor to use a constant instead of literal strings at all checks


Delete template file

Delete fiamma-testnet-1/bitvm2-staker-validators/vaildator-template.json

Delete fiamma-testnet-1/bitvm2-staker-validators/vaildator-template.json

You should delete your original PR's modifications to the template files instead of submitting a new PR.


/otg tp command failing to find biomes that it should find

Something not quite right with /otg tp as it's not finding biomes that are sometimes less than 100 blocks away.
The below is a biome with no spaces in the name and the biome is >500 blocks across, the command failed to find the biome even though it was within view distance and I found it manually shortly afterwards.

Needs to be checked if it's an issue with biome config names, biomes too small to find, or an actual bug.

I already stated in the bug report above that it isn't a biome name issue and that biome is >500 blocks across.
I tried to narrow this down recently and it may only be something that impacts developers (need to verify), as the last time I noticed it the world had the biome layout changed from initial generation (generated world, changed a biome rarity so changed biome layout and /otg tp failed to find a biome that was 100 blocks away).

WIll need a seed, coords and biome name to repro this plz.

Will do next time it happens.


Too many connections - Mobile

Hello,
When I use the map on my phone, after a few secondes, the map stop working. If I reload, I have this message : "Too many connections ! The tunnel session '.....' has violated the rate-limit of 20 connections per minute. (...)"
Someone know how I can solve this issue ?
Thanks
Dj

Im using chrome mobile and had the same thing. Worked fine for 12 hours then all of a sudden got this message as well.

I have 50 going at home but only one mobile uplink

I'm using multi accounts to scan several areas. So when I put it on my phone, I can see these multi areas but after a few secondes this error appears. That's maybe the reason?
I have to use only one account if i want to put it on my phone without this "Too many connections" error ?

lol 50 connections? tell me you don't mean that you are running 50 instances of the App. I may rest easier knowing there aren't people out there blatantly hammering their servers as hard as their home PC can manage it

I was running 4 instances of the app at home with one mobile link when I got this error. So I thought the "Too many  connections" error on my phone was caused by that. So I tried with only one instance but still have the same problem... I dont know what I have to do.

I have the same issue today - I'm running one instance of the app at home and am using the mobile version on Safari.

Using safari or chrome on 6s+ from my win10 rig gives me this problem. But so far not from my win7 rig, having a different issue there (works at first, then I get "waiting"). Works on both computers for a few minutes before needing to log out and back in. Just one instance. Separate account. Version 0.2.1.
I guess it should be noted that on the win10 rig I've installed jsnodes and python 2.7 for some other PoGoMap examples.
And on the win7 rig I did nothing but run the exe.
Not sure if any of that is usful.


You forgot to turn off debug (console.log) on latest beta27

I think this is a little bit annoying for debugging

Console logging also causes huge performance problems when lots of items are added, anyway this can be made a config when doing Meteor.connect?

Sorry, fixed in 1.0.0-beta28.

@Mokto thanks for the quick fix. is there a way to enable it? I think it's useful for when you need to see the data, just having the ability to turn it On/Off is optimal.

You can do
Meteor.ddp.on('added', message=>{

});

;)

Perfect thank you
On Mon, Apr 18, 2016 at 11:16 AM Mokto<EMAIL_ADDRESS>wrote:

You can do
Meteor.ddp.on('added', message=>{
});
;)
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/inProgress-team/react-native-meteor/issues/52#issuecomment-211425685


S3 naming convention

I confirm these files are made available under CC0 1.0 Universal (https://creativecommons.org/publicdomain/zero/1.0/legalcode)
Issue #, if available:
#206
Description of changes:

@rafihm Please review the files and let me know for any further update.

@rafihm Updated the files.Please have a look and let me know for any further changes.

Thanks @5u5hma :)
@jongogogo Please review the PR.

@jongogogo Please review and let me know for changes.


Ability to transfer channel membership between homeservers

I'm considering this distinct to https://github.com/matrix-org/GSoC/blob/master/IDEAS.md#decentralised-accounts because it is not as complex to implement.
The general idea is that I might want to move from @caleb:HomeserverA to @cjd:HomeserverB and while I accept that my handle will change, I would like to swap my new user into all of the channels and private messages which my old user was in and when I switch, I would like to keep my backscroll.
To bound the scope of this proposal:

Name change is expected, this is not about making a global p2p nickname namespace
Both involved homeservers are willing to perform the transfer

How the workflow might work:

On the old homeserver, In your user settings, next to "disable account" there is "transfer account", which also shows a warning because it is also permanent, after you click this button, you enter the new id to transfer to (which should already be registered with the new homeserver).
It produces for you a block of base64 which contains the new name and a signature authorising the shift, the user copies this.
User logs into the new homeserver and goes to settings and clicks "import account" and pastes the base64 code.
After verifying, the new homeserver broadcasts a transfer message to the network, this message contains the authorization to transfer from the old homeserver and so it is signed by both old and new.
Receiving a transfer message, all homeservers find-and-replace the old user with the new one in the db.
New homeserver begins to synchronize history of all of the channels joined from the old one. OR downloads them from the old homeserver through some special api...

Notes:

base64 blob and new homeserver name constitute 2 factor auth for transfer, this helps mitigate the risk that someone accidentally transfers to the wrong account (not theirs).
Transfers must be accepted, otherwise they can be used to "invite" people to zillions of channels
Transferring username and image are a "nice to have" but definitely not required.

somewhat related to https://github.com/matrix-org/synapse/issues/1209


fix: checking navigator type before to determine isMacOs

This PR contains:

[ ] bugfix
[ ] feature
[ ] refactor
[ ] tests
[ ] documentation
[x] others

Breaking Changes?

[ ] yes
[x] no

Checklist

[x] commit messages follow the Conventional Commits pattern

A feature commit message is prefixed "feat:"
A bugfix commit message is prefixed "fix:"
A document related commit is prefixed "docs:"

[ ] docs have been added / updated (for bug fixes / features)

Describe Your Changes

Adds checking for /Mac/.test(navigator.platform) to avoid the error 'navigator is not defined' with SSR.
Does this PR affects any existing issues?

[ ] yes
[x] no

This will fix the crash, but it will always use the other shortcut in MacOs. That is fine?

i don't understand why it makes the problem.
(sorry, am not familiar with Angular, other web side neither)
but if it does, using Angular core library also is a good way to detect platform?

You should be able to detect platform from the request body. You can read more information regarding that here https://angular.io/guide/universal.
I will merge this.

thank you!

oh. you deployed it.. appreciate


Drone Build container + Custom Networks

This might be related to: https://github.com/drone/drone/issues/1492 but am not sure.
I recently updated my docker-compose to the v2 syntax and put both my drone + gitlab containers in a network called ci.
Everything works and drone correctly sees when I perform a push to gitlab and attempts to start the build. The problem is the build execution container environment can’t see gitlab from the ci network so times out when I start to do a git fetch.
I thought making the repo trusted and setting net: ci would be enough but seems I am missing something? I confirmed that when drone launches the build execution environment it is only in the docker bridge network while all of my other containers (drone, gitlab, postgres, etc) are on my ci network.

[X] I have searched for existing issues
[X] I have discussed the issue with the community at https://gitter.im/drone/drone
[ ] I have provided a sample .drone.yml file to help the team reproduce
[ ] I have provided details from the build logs
[ ] I have provided details from the server logs by running docker logs drone
[X] I am not using the issue tracker to ask why my build failed

drone does not use docker compose v2 syntax for composed containers and does not yet support custom networks. Subscribe to https://github.com/drone/drone/issues/1268 for updates

Thanks for the heads up! :)


Add CI step to check Unicode symbols used for bra and ket

As pointed in #4, there are two symbols that can be used for each of the bra and ket notations.
The preferred symbols to use are U+27E9 MATHEMATICAL RIGHT ANGLE BRACKET and U+27E8 MATHEMATICAL LEFT ANGLE BRACKET.
This change adds a check to prevent U+232A RIGHT-POINTING ANGLE BRACKET and U+2329 LEFT-POINTING ANGLE BRACKET from creeping back in the code. It also fixes several occurrences of these symbols in the katas.

An example of this check failing can be found at https://dev.azure.com/michaylova/Quantum Katas CI/_build/results?buildId=60.


added ip-anonymisation-tag to analytics

1. Why is this change necessary?
When integrating google analytics in Europe it is mandatory to integrate ip-anonymisation. It is not explicitly enabled in the current codebase and the default is (of course - it's google) false.
There is some kind of a discussion in the forum with a bid to send in a PR -> https://forum.shopware.com/discussion/68503/google-analytics-anonymizeip-in-cookie-consent-manager
2. What does this change do, exactly?
It enables ip-anonymisation in google analytics and adds a new fancy twig-block around the tag-configuration to better handle twig-overrides of the affected file.
3. Describe each step to reproduce the issue or behaviour.
see above
4. Please link to the relevant issues (if any).
none found
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[(x)] I have read the contribution requirements and fulfil them. -> the link in your pr-template leading to your contribution-guideline is dead

why don't you make the anonymisation configurable per sales channel via admin with default true?

why don't you make the anonymisation configurable per sales channel via admin with default true?

i am going to have a look at it

Hi @Carrvin,
thanks for the hint and your contribution.
We discussed this internally and we came to the conclusion that a settings item inside the sales channel analytics tab would be good - like @uehler already mentioned. The default should be "anonymize_ip".
You said that you are interested in this but the PR is now closed. If you don't have the time right now feel free to file an issue at https://issues.shopware.com/
Best regards
Tobias


templateUrl should just be a template

In some projects, all js files are compiled into a giant app.js file, then templateUrl, "angular-quill.html" will not be accessible, because the app will only look for "app.html".

+1 :(


A guide or tool to fetch all historical photometry of a given target

Description
Beyond Kepler/K2, many projects have collected and/or continue to collect time series photometry of large swaths of the sky over various timespans.  Many of these projects are publicly available, for example:

ASASSN
SuperWASP
Integral-OMG
ASAS
AAVSO

and many more.  It would be useful to have, most simply, a list of these data sources to assist researchers interested in combining all available photometry of a target through time.  Beyond the list, it would be useful to have a system for combining the raw data (in various formats), into "flat files" (date, time, time standard, instrument, band, flux, flux error).  Many researchers must have built tooling to do this, but there is no clear public code.
Benefits

Makes historical data more visible and accessible
Simplifies access to disparate surveys
Brings all available information to bear on a problem of interest
Puts K2 data in context of longer timescale variability
Option for serendipitous discoveries from archival data

Costs

Not all photometric systems are standardized, so inter-comparisons may be misleading
Not all objects appear in all surveys, so need some resilience to missing data
Many surveys have large pixels, causing contamination that might not be present in K2 data
Survey sensitivities are limited, so most interesting comparisons will be for high-amplitude variables

Great idea. Checked the MAST portal for HAT-P-& (KIC 10666592) and although Kepler, Galex, and a few others show up, none of the photometry surveys like HATNet, WASP, etc. show up.
I wonder if it's worth talking to MAST about this to see if they are planning on incorporating these into MAST anytime soon? If so seems to be a more ideal solution for the community if they are.

Follow-up. We talked to MAST this morning. They might have some of these data sets in their Virtual Observator (VO) but they weren't sure which ones. They said it was more likely NExScI had the data as that was NExScI's role (to archive these data). Even then it might be sparse though as not all these groups delivered all data completely or consistenetly. MAST was very encouraging of anyone asking these groups to deliver their data to the MAST VO which would then make them easily retreivable.


docs: Mention bear for indexing Make-based projects

Maybe you can just add a link to compilation database format description? It mentions CMake, Bazel and Bear, it may list more (because there are more) alternatives in the future.

There is already a link to that reference earlier in the docs. The reason for mentioning Bear here is that it reduces the number of indirections (you don't need to go the Clang docs, then identify that Bear might work for you, then figure out how to use Bear).

There is already a link to that reference earlier in the docs.

It is mentioned in the context of what compilation database is, but the link may be helpful is the context of compilation database generation as well.

The reason for mentioning Bear here is that it reduces the number of indirections (you don't need to go the Clang docs, then identify that Bear might work for you, then figure out how to use Bear).

I understand the intentions. The problem is that Bear might not work, which leaves user in a dead end with no alternative solutions in sight.

Correct.

I suppose that's unlikely, but there's no harm in addiing a second mention. I've opened https://github.com/sourcegraph/scip-clang/pull/385 for that.

I thought it was unlikely too, but then I ended up working on a project which defeated Bear and ended up writing my own generator :rofl:

Would you like me to mention your tool in the README, or would you rather not have the extra attention (e.g. someone may file an issue etc.)?

but then I ended up working on a project which defeated Bear

What build system was this using? (If you can share?)

I've opened another follow-up PR (https://github.com/sourcegraph/scip-clang/pull/387), thanks for the various links in your project README.


Reinstalling 2 times and it doesnt work for me.

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8
What version did you experience this issue on?
3.1.1
What happened?
Please enter 1-10, Q: [1] 1
Starting the InvokeAI browser-based UI..
Traceback (most recent call last):
File "C:\Users\tesmo\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in run_module_as_main
return run_code(code, main_globals, None,
File "C:\Users\tesmo\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "D:\Ai\InvokeAI.venv\Scripts\invokeai-web.exe_main.py", line 4, in
from invokeai.app.api_app import invoke_api
File "D:\Ai\InvokeAI.venv\lib\site-packages\invokeai\app\api_app.py", line 19, in
from ..backend.util.logging import InvokeAILogger
File "D:\Ai\InvokeAI.venv\lib\site-packages\invokeai\backend_init.py", line 4, in
from .model_management import ModelManager, ModelCache, BaseModelType, ModelType, SubModelType, ModelInfo  # noqa: F401
File "D:\Ai\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management_init.py", line 4, in
from .model_manager import ModelManager, ModelInfo, AddModelResult, SchedulerPredictionType  # noqa: F401
File "D:\Ai\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management\model_manager.py", line 249, in
from .model_cache import ModelCache, ModelLocker
File "D:\Ai\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management\model_cache.py", line 31, in
from .models import BaseModelType, ModelType, SubModelType, ModelBase
File "D:\Ai\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management\models_init_.py", line 5, in
from .base import (  # noqa: F401
File "D:\Ai\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management\models\base.py", line 17, in
from diffusers import DiffusionPipeline, ConfigMixin
ModuleNotFoundError: No module named 'diffusers'
Screenshots

Additional context
No response
Contact Details
No response

In your venv, try pip install diffusers to install the diffusers package

In your venv, try pip install diffusers to install the diffusers package

Oh thanks! For some reason this information is not in the instructions.

No, it didn't work. Now I understand why this is not in the instructions)

The installer has been redone to prevent issues like this (certain packages not being installed). Please create a new issue if the problem persisted with the latest release (v4.1.0)


Fix to use cookbook name in local mode or chef-zero

Description
Cookbook name and version from metadata are used in chef_fs_data_store instead of directory name
Issues Resolved
closes #4816
Check List

[x] New functionality includes tests
[x] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

should be careful and make sure travis passes as much as is possible before we whack merge here.

I believe travis check is hanging since October and needs to be reset.

Looks great, thanks!


Docker

Could this work from inside a docker container?

Not sure about Java version. But the first revisions of the project does. You can see that earlier project's backend was on php. I will test this on my server, but now i don;t have such time. Sorry.

I was just about to create a Dockerfile but since there's no .jar file when cloning the repository or no information within docs on how I should compile it into a .jar file, then I can't get anywhere. I'm very unused to having to do with java in most possible ways...
So if someone can tell me how I provide the .jar then I make the Dockerfile and push a pull request!

@jimpsson You'd have to install Maven inside the Dockerfile, copy project files and run mvn clean package to generate the jar. However, since this is a Spring Boot app, you can run this to start the application right away mvn spring-boot:run.
@Rudolf-Barbu maybe it's time to add some building instructions, maybe a contributing guide, too?

I am looking at the code now. Wouldn't running the software in docker just display the resources of the container in the dashboard?
We need a way to pass the resource info from the host to the docker container running Ward


Modify CodeSandbox example to run without server sandboxes

See: https://codesandbox.io/s/github/FormidableLabs/urql/tree/example/move-sw-using-defer/packages/react-urql/examples/1-getting-started
Summary
This converts our first CodeSandbox examples (before we convert more over) to be a CRA sandbox instead of a full Node CodeSandbox. This will improve the speed at which users can run and try out our examples.
We originally intended to do this, but ran into problems with the implementation. This PR adds workarounds for the problems we found, namely missing worker bundling and broken service worker registration.
The only drawback of this is that offline support may be broken in the sandboxes we create (for now)
Set of changes

Convert from Parcel to Create React App.
Move index.html and service worker sw.js to public/
Move service worker registration to index.html with forcing code
Add BroadcastChannel to let the app handle GraphQL requests
Add missing private: true field to package.json
Remove template files

@andyrichardson I think we should minimise the dotfiles as much as possible? Not sure if they were needed. Do we want to restore/keep the .gitignore though?

This is currently on-hold to see whether we can make the CodeSandbox workaround for registering a service worker more seamless via letting them know (?)
Other than that it may also be confusing to have this approach in the codebase. Comments do hint at what it does, but optimally we’d want a more boxed up approach maybe.
We’d definitely want this to work without a server-sandbox imho, since that adds a lot of friction for quick tests and checks. That being said this PR in its current form simply adds “a different kind of friction”


Drawio integration with gitlab

Preflight Checklist

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

Describe the bug
Drawio integration with gitlab
To Reproduce
Steps to reproduce the behavior:

Go to 'https://x.x.x.x/drawio'
Click on 'gitlab'
Scroll down to '....'
See error

Expected behavior
It should redirect to gitlab url and list the repos
Screenshots
We are using jgraph/drawio:20.1.4 along with Gitlab 15.1 version. We are trying to integrate the both.
In gitlab we have created an application if with the below setting

and k8s yaml file we have given the gitlab url , gitlab secret and gitlab application id and when we try to open drawio with gitlab we get the below error.

draw.io version [20.1.4]

Additional context

Hi can you please let me know if its an issue from gitlab or drawio side so that we can proceed accordingly

I have the same problem

I have the same problem

Hi,
I'm currently working on this issue. Can you please update me with details on how you setup both gitlab and draw.io (self-hosted, app setting, ...)
I've tried the integration using gitlab.com and self-hosted draw.io instance and it works.
I've used OAuth app (https://docs.gitlab.com/ee/integration/oauth_provider.html) with redirect url as https://drawio.example.com/gitlab (not it's not gitlab.html)
Note: http support will be added in the next release

I want to know whether the api "this.redirectUri + '?getState=1'" should be implemented by ourselves. There is nowhere to say that this is to be implemented by ourselves; I didn't find any relevant instructions in the blog. I think it's just too bad up.
If we need to implement it ourselves, what should this api return?

The redirectUri is implemented via GitlabAuthServlet
In the super class AbsAuth '?getState=1' is implemented, it's main purpose is to implement the state check. In non-GAE environments, the state check is disabled as it requires key-value store at the server.
TLDR, it is implemented in Java and implementation is provided along with a ready to use docker image.


Documentation: 'fallbackAsPlaceholder' atribute is not documented

Thanks for this package - it solved all my problems with image caching and display!
'img-loader' has an input attribute 'fallbackAsPlaceholder' -- which is implemented, but not yet documented. See issue #34

is it work ?


Selenium WebDriver issue

I am on a Macbook Pro M2, OS version Sonoma 14.0
While trying to use the BrowsingAgent, I am consistently getting this error:
Error initializing WebDriver: Message: session not created: Chrome failed to start: exited normally.
  (session not created: DevToolsActivePort file doesn't exist)
  (The process started from chrome location /Applications/Google Chrome.app/Contents/MacOS/Google Chrome is no longer running, so ChromeDriver is assuming that Chrome has crashed.)
Stacktrace:
0   chromedriver                        0x00000001026c04dc chromedriver + 4162780
1   chromedriver                        0x00000001026b8664 chromedriver + 4130404
2   chromedriver                        0x000000010230fbc0 chromedriver + 293824
3   chromedriver                        0x000000010234175c chromedriver + 497500
4   chromedriver                        0x000000010233e128 chromedriver + 483624
5   chromedriver                        0x0000000102384c40 chromedriver + 773184
6   chromedriver                        0x00000001023496bc chromedriver + 530108
7   chromedriver                        0x000000010234a930 chromedriver + 534832
8   chromedriver                        0x0000000102685e08 chromedriver + 3923464
9   chromedriver                        0x000000010268a3dc chromedriver + 3941340
10  chromedriver                        0x000000010266e038 chromedriver + 3825720
11  chromedriver                        0x000000010268af3c chromedriver + 3944252
12  chromedriver                        0x00000001026606f4 chromedriver + 3770100
13  chromedriver                        0x00000001026a7980 chromedriver + 4061568
14  chromedriver                        0x00000001026a7af8 chromedriver + 4061944
15  chromedriver                        0x00000001026b82e4 chromedriver + 4129508
16  libsystem_pthread.dylib             0x000000018a00f034 _pthread_start + 136
17  libsystem_pthread.dylib             0x000000018a009e3c thread_start + 8

Most of the solutions that I saw on SO point towards certain arguments in chrome_options which I see have already been added in the selenium.py file. Not sure how to solve this.

Try to quit chrome before running the browsing agent, @pythiantech, or install canary and specify chrome_profile_path in selenium_config to a profile in canary. Let me know if this helps.

Installed canary and specified chrome_profile_path too. No joy:(
The only way it works is if chrome_options.add_argument('--headless') is specified. With this of course you cannot see what's going on.

@pythiantech according to this it seems like you need to run browsing agent as a regular user, instead of as a root. Let me know if this helps


Updated Romanian translations

As requested in #9.

Looks good, thank you!
For future reference, is there a particular way you'd like me to contact you when translations need to be updated? It won't be super frequent, but odds are a couple of strings will generally change each update.

Thank you! You can find me on discord by username NotImpLife. I'll gladly assist with translations whenever you need.


[ request ] Allow to filter related models based on subscription status.

If you look at my post here - https://laracasts.com/discuss/channels/eloquent/global-query-scope-and-related-model-method
I am trying to only show Jobs that belongTo User where the User is subscribed().
Is there a way we can add a special trait to related models that have a foreign key to user to only show records from users who subscribed() returns true for?

I don't think you would need a foreign key.
$jobs = Job::get();

$jobs = $jobs->filter(function(Job $item) {
       return $item->user->subscribed() == true;
});

@fetch404 but then you loose the ease of pagination with the database query. - I ended up using a global query scope - here's how

I imagine this can be done using the query builder, or maybe join statements.


Implements add_dynamic_template_data/2

Per the v3 API Documentation, it seems that the "dynamic_template_data" field must exist on the personalizations key.
Adds dynamic_template_data to the whitelist of keys that are mapped to personalizations.

Thanks for the PR.
After looking at the SendGrid Docs, it looks like the dynamic_template_data key must exist under a personalization object which isn't currently part of the lib. #27 adds support for it. Once that is in, you'll need to add it to the personalization API.

I appreciate the response!  I've made the requested changes.  Let me know if anything else is needed.

👍 🎉


⚠ Turnos 24x7 has degraded performance

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

HTTP code: 200
Response time: 928 ms

Resolved: Turnos 24x7 performance has improved in 5606e6b.


tradingview.com

Hello!
Found more recent false-positives in the list. This domain has quite a few subdomains listed, not all of them should be whitelisted, as most of the tradingview.com entries belong on the list.
These domains should be whitelisted as they provide market graphs for stocks:
widgetdata.tradingview.com
s.tradingview.com
s3.tradingview.com
You can see this for example on Investopedia.com
Cheers

Thanks for reporting, I've added it to the allowlist, and I've reported it upstream to  blocklistproject/Lists


Improve scrolling + History bookmarks

Describe the feature
Two requests:

Scrolling is very slow when you scoll backward to see precioys commands and output or when you scroll back to continue using the shell. It would be nice if it were much faster.

“Bookmarks” for specific points in the window would be very nice so you could jump to a specific point in the window would also be super.

Screenshots / Draws

Hello @mbierman 👋
You're right since three / four releases the scrolling is not user friendly (because of upgrading the rendering engine).
I definitely need to improve it.

Yes, perhaps you can use the user scrolling speed to change your scroll speed (how many tabs and scrolls in a time).
On slow scrolling (for any rows scroll with only one tab) now it's ok ;)
But if i scroll up any pages, this could be a little faster ;)

Thanks @svenh0 for your feedback 😉
You’re right I will implement a different scrolling based on user velocity

Is this on the horizon yet? I know, you have been busy! (And doing great work.)

Thanks for your support @mbierman Michael 😇
This issue requires an important upgrade of the rendering engine (hterm) and this upgrade must come with a major (15.x) release.
This issue is important also on macOS because on this platform the scroll is so bad with the current engine release 😭
I know that this issue was opened months ago and I will try to make my best efforts to fix it.

Statement from @AInteriorB :
 Also I expect to be „back at the bottom“ as soon as I type any key. This would accelerate a usual workflow, when I have to look what happened above and then continue my work 😉

Scrolling is very slow when you scoll backward to see precioys commands and output or when you scroll back to continue using the shell. It would be nice if it were much faster.

I absolutely agree like written in dublicate #492

Hello 👋
 Also I expect to be „back at the bottom“ as soon as I type any key. This would accelerate a usual workflow, when I have to look what happened above and then continue my work 😉
This behavior will be available in a few hours now.

This works, but

No improvements in scrolling up
Some colors are more matte
Copy-Paste is no longer possible on iOS, as mentioned in #456
Keyboard is not enabled when opening a new session

Keyboard is not enabled when opening a new session

Even on 852 build?

No improvements in scrolling up

You're right I will improve the momentum

Some colors are more matte

I'm using default Xterm.js colors. I will change them. Do you have a wanted palette?

Keyboard is not enabled when opening a new session

Even on 852 build?

Fixed in 852 💪😀

@AInteriorB I will implement a feature to import these color schemes : https://github.com/mbadolato/iTerm2-Color-Schemes

I'm now working on a scrollbar (iOS / macOS / iPadOS) : #599

First attempt of a custom scrollbar available on the latest beta release

Custom scrollbar now available inside the latest beta release


Add Blueprint compat package

Actually fixes #567

Doesn't seem to be enough any interest in this so abandoning the effort.


Use natlas' libnmap instead of the unmaintained libnmap

Drafting to start to show it works, but we shouldn't merge this until natlas/python-libnmap gets detached to a standalone repository. Once we do that, I'll change the repo name and setup deployment to pypi so that we can pip install it like normal, probably under the name natlas-libnmap.
When this lands it will close #299

@ajacques Hey look the underlying library has tests and even has an explicit test for the payload that resulted in the CVE
https://github.com/natlas/natlas-libnmap/blob/master/libnmap/test/test_new_parser.py#L7-L16
The library isn't setup in CI yet, but that shouldn't freeze this PR imo.


Data Type Side Nav- 508 issue

Required Fix

[x] RF-1b
[x]  RF-7

Context
See EPA CAMD Accessibility Report  for how to fix and additional details

Issue #
Issue Description

RF-1b
None of the visual headings have html tags, they all appear to be divs. For example:  Data Type

RF-7
There is at least one form input field that does not have a label tag properly associated with it. This issue is a violation of Section 508 and of WCAG 2.0 Success Criterion 1.3.1

Testing
Tester: @aprematta

Pull Request
RF-1b - Header Tags

Gave "CAMPD: Clean Air Markets Division" an h1 tag

removed href

Updated usa-footer__logo-heading to a h2 tag
Updated .panel-header in global-overrides.scss to include header tags
Gave h2 tags to the following:

Data Type (side panel header)
Filters (side panel header)
Custom Data Download (data preview header)

Gave h3 tags to the following

Time Period (filter panel header)
Program (filter panel header)
State/Territory (filter panel header)
Facility (filter panel header)
Unit Type (filter panel header)
Unit Fuel Type (filter panel header)
Control Technology (filter panel header)
Data Preview (data preview header)

Program filter sub headings:

h4 tag to Active and Retired
h5 tag to Annual and Ozone

RF-7 - Data Type Labels

Created a Label for the Data Type dropdown
Created a Label for the Data Sub Type dropdown

Both issues pass testing, closing this ticket


BackgroundColor

Hello, good day everyone. I've faced an issue on Flutter 3.10.6 that the backgroundColor is not changing.
To fix this issue I've sent this pull request: https://github.com/splashbyte/action_slider/pull/5
By replacing DecoratedBox to Container

Thanks for opening the issue and the MR :)
I will have a look at it tomorrow.


DXT5 reads RGB but can do YCgCo conversion and create straight alpha

New pull request that replaces
https://github.com/SteamDatabase/ValveResourceFormat/pull/36
UncompressDXT5 takes two extra bools that enable YCgCo conversion and straight alpha.

You didn't rebase on master correctly, thus your diffs include unnecessary changes.
What's the need in straight alpha option?

I could be reading everything completely wrong here, but how do I gain access to the Flags in the instance of Texture from the static DDSImage class?

@SinZ163 is saying he found a YCoCg image that had no vtex flags, so CLAMPU fails for this purpose. Looks like we will need to look at REDI strings after all.
As for the straight alpha option, could you explain the need in doing that?

Basically to look at REDI you need to do something similar to this: https://github.com/SteamDatabase/ValveResourceFormat/blob/master/ValveResourceFormat/Resource/Resource.cs#L132-L140
but instead of looking at first item in the list, iterate all of them, and check if one of them matches the exact YCoCg compiler string.

Oh, I missread https://github.com/SteamDatabase/ValveResourceFormat/pull/36#issuecomment-166028138 I thought you meant that controlled straight alpha.
Anyway. The reason the alpha option is needed in some way or another is basically that PNG only support straight alpha (where a pixel is proportionally blended with the background based on alpha value) and not premultiplied alpha (where the alpha is cut from the background each pixel added on top).
If you use a PNG with wrong alpha you get a dark halo around everything. But the crappy part is that the process of making the alpha straight is lossy. I guess that is why Bitmap.Save method don't do it even for premultiplied pixel formats.
The DXT5 specification says the format is straight alpha, but all the textures I've looked at from Dota 2 looks to be premultiplied (DXT4).
So, basically the choice is between making the PNGs usable or make them as lossless/conforming as possible. I don't think a command line flag for that is such a bad idea.

I haven't really had the time to properly look into REDI-reading yet, but from the first quick looks I don't understand how to access other blocks from inside the Texture class. Next week I'll have more time to sit down and understand it all, but a push in the right direction would be good.
I understand how to do it anywhere I have access to the resource-object where I have Blocks to loop thru, but not inside Texture.

I did that for you in 1220a94b20a70ca4ad3d4ce925d383003f3eae78

Just tried to mess with this, looks like some textures that have blue background are turned into alpha in png. That's how it's supposed to work, right?
@SinZ163 what do you think?


Move to docc documentation

Resolves https://github.com/apple/swift-metrics-extras/issues/18

The nightly failure is somethign we must fix independently: https://github.com/apple/swift-metrics-extras/issues/17

Added CI for Swift 5.6 and 5.7.
@swift-server-bot test this please

Okey I chimed in on https://github.com/apple/swift-statsd-client/pull/36
THanks for review!


re2 not compiling

Hey, I'm not longer able to compile cqerl. Looks like one if it dependencies (https://github.com/matehat/re2) is the root cause of the problem. I tried to replace it with the latest version from here https://github.com/tuncer/re2 and it worked fine.
...
===> Compiling setup
===> Compiling idna
===> Compiling bear
===> Package <<"pc">> not found. Fetching registry updates and trying again...
===> Updating package registry...
===> Writing registry to /root/.cache/rebar3/hex/default/registry
===> Generating package index...
===> [cloudi_service_oauth1:1.5.1] Only existing version of cloudi_service_db_riak is 1.3.3 which does not match constraint ~> 1.5.1. Using anyway, but it is not guaranteed to work.
===> Writing index to /root/.cache/rebar3/hex/default/packages.idx
===> Fetching pc ({pkg,<<"pc">>,<<"1.4.0">>})
===> Downloaded package, caching at /root/.cache/rebar3/hex/default/packages/pc-1.4.0.tar
===> Compiling pc
===> Compiling re2
===> Compiling /code/deps/re2/c_src/re2_nif.cpp
===> /code/deps/re2/c_src/re2_nif.cpp:8:21: fatal error: re2/re2.h: No such file or directory
 #include <re2/re2.h>
                     ^
compilation terminated.

** (Mix) Could not compile dependency :re2, "/root/.mix/rebar3 bare compile --paths "/code/_build/prod/lib/*/ebin"" command failed. You can recompile this dependency with "mix deps.compile re2", update it with "mix deps.update re2" or clean it with "mix deps.clean re2"

Can you make a PR with the change?

Sure -> https://github.com/matehat/cqerl/pull/127


Bad resource file upload test (#130)

Bad resource file upload notification test
Closes #130

@alexeyk @gvidon tests fail. See travis output.

I saw, will get to this by Monday.
-----Исходное сообщение-----
От: "Paul Walsh"<EMAIL_ADDRESS>Отправлено: 29.08.2015 11:50
Кому: "okfn/datapackagist"<EMAIL_ADDRESS>Копия: "Gvidon Malyarov"<EMAIL_ADDRESS>Тема: Re: [datapackagist] Bad resource file upload test (#130) (#137)
@alexeyk @gvidon tests fail. See travis output.
—
Reply to this email directly or view it on GitHub.

Tests passed after reloading them manually on travis.

Ready


After update to latest version Error: "list indices must be integers or slices, not str"

Describe the bug
After updating to the latest version there is an error: "list indices must be integers or slices, not str" I tried to reinstall it but no luck.
Expected behavior
Before the update, there was no error. If I revert it to the previous version everything is working as expected.
Screenshots

Diagnostic File
list indices must be integers or slices, not str
Debug logs
No response
Home Assistant Version
Core: 2024.5.0 Frontend: 20240501.0
solaredge-modbus-multi Version
latest
Installation Type
Container
Read the Instructions

[X] I have read https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki
[X] My configuration follows https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Configuration
[x] This issue is not about a template error (template errors should not be reported as an issue, read https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Template-Design-Notes)

Additional Context
No response

Please try Release v2.4.15-pre.1
Upload your diagnostic file if this pre-release works.

The full release for Release v2.4.15 is now available.
With your Synergy inverter this release will add new devices that show the status of the modules.


feat: Enable adding files

Description
Enable adding files (refactored from documents). Specifically, this PR:

Defines data models for File objects
Supports storing files in SQL databases
Adds an API route to retrieve all the files associated with a source
Add pagination to API route

Also, adds some items to the .gitignore.
I ran alembic via the instructions:
poetry run alembic upgrade head
poetry run alembic revision --autogenerate -m "Add FileMetadata table and drop Documents table"

Testing
Added a test that loads the memgpt paper, retrieves the documents via the new API route, and confirms that:

the correct number of files were made
the metadata contains the right file name
the sources are set correctly on the files

cc @4shub for correctness - I noticed it didn't mention dropping the Documents table or adding a new FileMetadata table - is this expected?

This is correct, alembic only does any change on database deletions or modifications. You did not remove any database you just removed a scheme