text:
Add support for GeoIP2-Enterprise database

As written in comment to geoip.rb "The path to the GeoLite2 database file which Logstash should use. City and ASN databases are supported.".
Is there a plan to support GeoIP2-Enterprise database?
Thanks
Murali

Is there a plan to support GeoIP2-Enterprise database?

Hey @kvmuralidhar!
This plugin already supports MaxMind GeoIP2-Enterprise City and ISP databases. Here is a sample configuration:
geoip {
    source => "requestIPAddress"
    database => "/usr/share/logstash/geoip2/GeoIP2-City.mmdb"
    fields => [ "city_name", "country_code2", "region_name", "location" ]
}
geoip {
    source => "requestIPAddress"
    database => "/usr/share/logstash/geoip2/GeoIP2-ISP.mmdb"
    default_database_type => "ASN"
    fields => [ "isp", "organization", "autonomous_system_organization", "autonomous_system_number" ]
}

Docs reference: https://www.elastic.co/guide/en/logstash/current/plugins-filters-geoip.html#_supported_databases
Full list of fields: https://github.com/logstash-plugins/logstash-filter-geoip/blob/master/src/main/java/org/logstash/filters/Fields.java

@bcbee,
Thank you for your update.  I was looking for support to GeoIP2-Enterprise db since it has additional information for a given ip address.
'connection_type': 'Corporate',
'domain': 'googlebot.com',
'user_type': 'search_engine_spider'

authors: bcbee kvmuralidhar

repo: logstash-plugins/logstash-filter-geoip


text:
Sentinel configuration for redis plugin

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/
DONE:

CHG code refactor for rubocop style guide
ADD sentinel configuration with new sentinel parameters
Update jruby version

Why is it build the master branch, not mine?

source: gharchive/pull-request

repo: logstash-plugins/logstash-input-redis

license_type: permissive

license_source: github-api


text:
:bug: fixed safari page count position

fixed safari page count position bug

Codecov Report

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

@@           Coverage Diff           @@
##           master      #55   +/-   ##
=======================================
  Coverage   84.31%   84.31%
=======================================
  Files           1        1
  Lines          51       51
  Branches       14       14
=======================================
  Hits           43       43
  Misses          5        5
  Partials        3        3

Impacted Files
Coverage Δ

src/components/Paginate.vue
84.31% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

authors: codecov-io liaohainan

repo: lokyoung/vuejs-paginate

license: mit

license_source: bigquery


text:
Build fails with "func literal" error

Trying to build RPMs for testing latest updates and getting an error relating to Go context module during gox-build step of Makefile (see below). Apologies if this is not the correct process or designed for end-user usage. Steps to repeat

cd deploy/buildenv
./build.sh

Tried with the default 1.8.3 and 1.10.1 versions of Go.
Successfully tagged go-carbon-build:latest
+ cd /src/go-carbon/
+ make gox-build
rm -rf build
mkdir -p build
cd build && go build github.com/lomik/go-carbon
# github.com/lomik/go-carbon/receiver/pubsub
../_vendor/src/github.com/lomik/go-carbon/receiver/pubsub/pubsub.go:133: cannot use func literal (type func("context".Context, *"github.com/lomik/go-carbon/vendor/cloud.google.com/go/pubsub".Message)) as type func("github.com/lomik/go-carbon/vendor/golang.org/x/net/context".Context, *"github.com/lomik/go-carbon/vendor/cloud.google.com/go/pubsub".Message) in argument to rcv.subscription.Receive
Makefile:44: recipe for target 'gox-build' failed
make: *** [gox-build] Error 2

go 1.10.3 works fine. Dockerfile updated https://github.com/lomik/go-carbon/commit/57829e26737509cbe974cf450beb1b9f46331048

Hey thanks, that's sorted for me now.

source: gharchive/issue

authors: MikeKirkFT lomik

repo: lomik/go-carbon


text:
Themes for the ngx-markdown-editor

Greetings! I want to know how I can change the white theme of the editor. I know that it works with styles but I wanted to know if themes styles css exist for this package.

Sorry. There is no themes styles css exist

authors: Jeoxs lon-yang

repo: lon-yang/ngx-markdown-editor


text:
data_processing of opengenes

I wrote the code that extends sqlite with additional opengenes tables. However, it is not clear for me if I should git-ignore csv-es or not.

I think you should:

Make script universal, so it can get the path for sqlite and create or update new one. Maybe optional parameter about rewriting can be of use.
I do not think that you have to push those files and different sqlite database. Better push a script and updated sqlite (with new tables). We might want to reuse the script outside of longevity genes so it is better to have it in ./scripts or ./preprocessing folder instead of opengenes one

I have done it. I also wrote a CLI app to test prompts on the database

authors: Livia-Zaharia antonkulaga


text:
🛑 Less Plastic is down

In 2cef198, Less Plastic (https://lessplastic.jau.co.jp) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Less Plastic is back up in 60b33b1.

repo: longforme/longforme-uptime-monitor

license: MIT


text:
🛑 Less Plastic is down

In b22e743, Less Plastic (https://lessplastic.jau.co.jp) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Less Plastic is back up in 1b830ad.


text:
Dockerfile: update nfs-ganesha to v5_20240408

Which issue(s) this PR fixes:

Issue Longhorn/longhorn#8253
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context

@Mergify backport v1.6.x

repo: longhorn/longhorn-share-manager


text:
doc(manual test): update the setting name for drain node policy

Update the manual test plan since the setting has been changed

cc @longhorn/qa

authors: ChanYiLin innobead

repo: longhorn/longhorn-tests


text:
[BUG][infra] junit-report.xml failed to read

Describe the bug
Noticed currently the junit-report.xml report gets created from kubectl logs, and when py.test hits an error will break the junit-report.xml.
The error in the test result is not really helpful, we should probably consider using kubectl cp instead.
To Reproduce

Create an invalid test file in /manager/integration/tests and build the image.
Run regression with the image.

Expected behavior

Should create valid junit-report.xml.
Should see the result from the junit-report with some sort of error indicating where it's faulty. For example,

<?xml version="1.0" encoding="utf-8"?><testsuites><testsuite errors="1" failures="0" hostname="longhorn-test" name="pytest" skipped="0" tests="1" time="0.508" timestamp="2021-03-12T04:52:30.737996"><testcase classname="" file="tests/test_node_c3y1.py" name="tests.test_node_c3y1" time="0.000"><error message="collection failure">/usr/local/lib/python3.9/site-packages/_pytest/python.py:492: in _importtestmodule
    mod = self.fspath.pyimport(ensuresyspath=importmode)
/usr/local/lib/python3.9/site-packages/py/_path/local.py:704: in pyimport
    __import__(modname)
<frozen importlib._bootstrap>:1007: in _find_and_load
    ???
<frozen importlib._bootstrap>:986: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:680: in _load_unlocked
    ???
/usr/local/lib/python3.9/site-packages/_pytest/assertion/rewrite.py:134: in exec_module
    source_stat, co = _rewrite_test(fn, self.config)
/usr/local/lib/python3.9/site-packages/_pytest/assertion/rewrite.py:317: in _rewrite_test
    tree = ast.parse(source, filename=fn)
/usr/local/lib/python3.9/ast.py:50: in parse
    return compile(source, filename, mode, flags,
E     File "/integration/tests/test_node_c3y1.py", line 1
E       cleanup_node_disks, wait_for_disk_storage_available, \
E   IndentationError: unexpected indent</error></testcase></testsuite></testsuites>

Log
n/a
Environment:
n/a
Additional context
https://ci.longhorn.io/job/private/job/longhorn-tests-regression/43/#showFailuresLink

cc @meldafrawi

@meldafrawi please help with this. Thanks!

Also happens when timeout on regression deployment.

Failed to read test report file: https://ci.longhorn.io/job/public/job/ubuntu/job/amd64/job/longhorn-tests-ubuntu-amd64/301/

authors: c3y1huang innobead meldafrawi

repo: longhorn/longhorn


text:
[TASK] Release 1.2.2

What's the task? Please describe.
Action items for releasing v1.2.2
Describe the items of the task (DoD, definition of done) you'd like.

[ ] Release GA but not published w/ all images built, repo tags, chart PRs until all following items checked

[ ] Verify GA draft to ensure all artifacts ready
https://github.com/longhorn/charts/pull/30
[x] Run automatic e2e core tests regression - @meldafrawi
[x] Some mandatory manual regression: longhorn/chart PR install, upgrade) - @khushboo-rancher / @kaxing
[ ] Release (longhorn/longhorn, longhorn/chart, longhorn/website) and make sure doc links correctly in the release note
[ ] Merge the release branch to master (longhorn/longhorn, longhorn/website)
- For longhorn/longhorn, only minor release needs to merge back to the master (ex: 1.2.0, 1.3.0)
[ ] Create a new patch release branch for the following patch releases if the new release is minor release

manager/ui/tests/engine 1.2.x created
longhorn/website 1.2. created

[ ] Ensure rancher chart PRs ready and reviewed (cluster explorer 2.5, 2.6, and cluster manager) and image mirror - @shuo-wu review

[x] rancher/image-mirrors update
https://github.com/rancher/image-mirror/pull/170
[x] rancher/charts master branch for cluster manager
https://github.com/rancher/charts/pull/1536
[x] rancher/charts dev-* branches for cluster explorer
https://github.com/rancher/charts/pull/1537
https://github.com/rancher/charts/pull/1538
[x] Verify the chart being able to install & upgrade - @khushboo-rancher @meldafrawi

[ ] Update https://github.com/longhorn/upgrade-responder/blob/master/config/response.json

Additional context
N/A
cc @longhorn/qa

Validated: LGTM

Longhorn installation/upgrade rancher 2.5.9 cluster manager
Longhorn installation/upgrade rancher 2.5.9 cluster explorer
Longhorn installation/upgrade rancher 2.6

Validated the mandatory manual regression for v1.2.2 - LGTM

Basic operation RW
Attach/detach of volume
Rebuilding
snapshots creation deletion
Reverting to snapshot
Backup creation.
restore
Changes in settings

Thanks @meldafrawi for running core test.
Core test results -   https://ci.longhorn.io/job/private/job/longhorn-tests-regression/487  - All Passed

authors: innobead khushboo-rancher meldafrawi


text:
Changes to Guaranteed Replica Manager CPU should not cause cluster-wide volume restart

Currently, changes to Guaranteed Replica Manager CPU cause all volumes in the cluster to be disrupted, see docs:

After this setting is changed, all replica manager pods using this global setting on all the nodes will be automatically restarted. In other words, DO NOT CHANGE THIS SETTING WITH ATTACHED VOLUMES.
https://longhorn.io/docs/1.2.3/references/settings/#guaranteed-replica-manager-cpu

This request is to have the restart be localized to only a subset of the volumes so running applications can better tolerate the change. For instance, applying the change only to one node at a time would allow the node to be drained and cordoned before which would remove the attached volumes on that host.
An ideal UX:

User makes changes to the Guaranteed Replica Manager CPU setting in the longhorn UI/API
Longhorn records the change but does not apply to a replica manager until that manager no longer has attached pods.
The user can commence a drain/cordon maintenance operation on each node in a rolling fashion, which in turn triggers the replica manager pod to apply the change and restart

I think this is a good idea. Giving the user the ability to manage the change in setting via drain/cordon (Independent from the cordon/drain we only terminate pods once there is no longer a running instance)
The same change would need to be applied on the per node level configuration we have, but both should be able to utilize the same approach:
desired.request = X // spec
current.request = Y // status

if (safeToPerform && desired.request != current.request) {
    // update the resource requests and restart pod
}


text:
[ENHANCEMENT] Longhorn UI supports orphan CRs management

What's the task? Please describe
Longhorn UI supports orphan CRs

List
Deletion
Enable/disable global orphan-auto-deletion setting

Additional context
Longhorn #685

@smallteeths this needs to wait https://github.com/longhorn/longhorn/issues/685

@derekbit @smallteeths this is nice to have in 1.3.0, so please help cooperate on this. Thanks.

My idea is that we can add a orphaned data page.

Then, in the page, the layout is like

Name: resource.name
Node: resource.spec.nodeID
Disk: resource.spec.parameters["DiskPath"]
Path: resource.spec.parameters["DataName"]

We also need to introduce a global orphan-auto-deletion setting in Setting->General page. The default value is false
cc. @smallteeths

Setting will be automatically created w/o any UI change :slightly_smiling_face:

Setting will be automatically created w/o any UI change 🙂

Cool!

Yes I am working on this page, I am going to put the nav link under the drop down menu in setting.

And something else I'm not so sure about

Is batch deletion also required?
Do we need to add a filter by name component？

orphaned data might be better because there are other different orphaned data types in the future.
I'm not sure if putting orphaned data link in Setting is better. But it's fine for me with it.

Is batch deletion also required?
Yes, we need.

Do we need to add a filter by name component？
No, the name is UUID and meaningless.
But we need a filter by node and disk

Thank you @smallteeths

verified passed on v1.3.0-preview1 by
(1) checking there is Orphaned Data page and it has list and delete functionalities
(2) checking there is Orphan Auto-Deletion setting and it works
(3) running https://github.com/longhorn/longhorn-tests/pull/940 and checking the UI in the meantime

authors: derekbit innobead smallteeths yangchiu


text:
[DOC] Update the Longhorn document in Setting up Prometheus and Grafana

What's the task? Please describe
After #2406 was verified, we also need to update the Longhorn document in Setting up Prometheus and Grafana.
Describe the items of the task (DoD, definition of done) you'd like
I want to update the Longhorn document in Setting up Prometheus and Grafana.
After that, users could follow the Longhorn document in Setting up Prometheus and Grafana and import Longhorn Example v1.4.0 grafana dashboard
Additional context
N/A

Who created this, https://grafana.com/grafana/dashboards/17626-longhorn-example-v1-4-0/? What kind of info do you think we should add to https://longhorn.io/docs/1.4.0/monitoring/prometheus-and-grafana-setup/?
cc @PhanLe1010 @derekbit

Who created this, https://grafana.com/grafana/dashboards/17626-longhorn-example-v1-4-0/? It looks cool.

I created it and leveraged @PhanLe1010's dashboard. Just added some new metrics.

Thanks @derekbit and @roger-ryao :+1:

https://github.com/longhorn/website/pull/653 was merged

authors: derekbit innobead roger-ryao


text:
formatting on console.log of look logging

need space after moment().format() timestamp before colon and padEnd(30)
2017-06-27 23:53:21 : patrick                        !stat mantooth
2017-06-28 00:19:25: eddiejerky                     Logged 'Tterror'
2017-06-28 00:30:29 : normstorm                      !who Tterror

Believe already fixed in ParseEqLook, need to validate before closing
                      hands,shield,about,waist,pouch,rwrist,lwrist,primary,secondary,held,both,pSubmitter,null, (arg) => {
                      console.log(`${moment().format(MYSQL_DATETIME_FORMAT)} : ${pSubmitter.padEnd(30)} Logged '${arg}'`);
                    });```

addressed in 533c13f


text:
Long test setup when building objects referencing abstract classes

Adding @Build() annotation with a class, which references an abstract domain class results in very long test execution time.
How to reproduce:

Create abstract domain class X
Create regular domain class Y having field of type X
Create Spock Specification annotated with @Build(Y)
Running gradle check takes sometimes even more than 1 minute with empty test suite (no tests)

Exemplary source code reproducing issue can be found on: https://github.com/piotrgajow/Sandbox/tree/grails-BuildTestDataPlugin-AbstractIssue
Removing abstract keyword or @Build annotation reduces the time to a few seconds.
Environment:
Plugin: 3.0.1
OS: Windows 10
Gradle: 3.4.1
Grails: 3.2.8
Groovy: 2.4.7
JVM: 1.8.0_131

The class path scanning has been removed in version 3.3.0+ so this is no longer an issue going forward.

Hi @longwa, can you give an idea of when you think you'll be able to backport this to 3.0.x?
I have a project running on Grails 3.2.x that I can't upgrade to 3.3.x right now, and this issue is causing some of my CI builds to take up to 10 minutes longer than expected

Suggested fix: #112

authors: borgiani longwa piotrgajow

repo: longwa/build-test-data


text:
fix: dependabot fix

First of many

Dammit! I committed a file I did not mean to (fix for demo dashboards next). I was going to create a separate PR for it and will if you want me to. Let me know if you want me to revert the change from this PR

repo: looker-open-source/extension-examples


text:
Fix typo in class name

This PR:

[x] Fixes a typo in a class name in the readme
[ ] Provide ...
[ ] It breaks backward compatibility
[ ] Has unit tests (phpspec)
[ ] Has static analysis tests (psalm, phpstan)
[ ] Has documentation
[ ] Is an experimental thing

Follows #.
Related to #.
Fixes #.

Thanks!

authors: drupol jdreesen

repo: loophp/iterators


text:
REBORN: Double tap abilities don't seem to work with space modifier

Abilities such as NP's teleport don't behave as expected when holding down the space bar for normal cast.

That's intentional.
If you want those to work, you need to head into the _settings_game.cfg and edit this line
 //  Prevents you from double pressing to self cast (or makes it really fast so that it only happens with scripts)
 dota_ability_self_cast_timeout 0.01

Instead, you should hold Alt, and self-cast it, as it does the "double-tap" for self-cast or tp home for tp-scrolls and prophet's TP in one keystroke.

authors: JamiePaton Sembrani loopuleasa


text:
data-max-options ETA?

This directive is perfect for multiselect.. However, having a max number of options is a must-have. Is there an ETA or anything that can be done to hack it and make this work? Thanks!

Sorry, This feature is not in a high priority. But I promise it will be released in 2.1.x.(next release will be 2.1.0)

i am using v2.5.1.
How can i use this features.

authors: GaurangGhinaiya lordfriend sdawson26

repo: lordfriend/nya-bootstrap-select


text:
SOLVE: How do I enable the admin commands?

http://www.swgemu.com/forums/showthread.php?t=85624&p=1247644&viewfull=1#post1247644

Enabling Admin Commands and Debug Examine in Your Client
To enable these features in the client you will need to edit the user.cfg file in the c:\SWGEMU			folder to add these 4 liness:

[ClientGame]
0fd345d9 = true
[ClientUserInterface]
debugExamine=1

If the user.cfg file doesn’t exist, simply copy swgemu.cfg and rename it as user.cfg. Then, replace the contents of the new user.cfg with the contents of the table above.
You might also need to add a line in swgemu.cfg to tell the client to use the new user.cfg file:

.include "swgemu_login.cfg"
.include "swgemu_live.cfg"
.include "swgemu_preload.cfg"
.include "options.cfg"
.include "user.cfg"      <-- THIS LINE MIGHT BE MISSING or Commented out

Once the config files are done, totally shut down your client (if running) and restart it.

We can script this!

Really? This is the client in the windows host... we can do that from the vm guest?

Use the source luke.. If they have Git Bash we can script it on their side.  At least on the first machine they start from we can help them make sure that one is working.

Yoda documents this.

authors: Scurby karlbunch

repo: lordkator/ZonamaDev


text:
Peer to peer review

You did a great job so far! just a few suggestions to be addressed

We suggest you use a switch case instead of else if conditions to make your code shorter
We kindly suggest you use a camel case for all your variables to make it more understandable

Thank you for the feedback. Changes applied in commit 6b7095de58e4242e5d04c0f54389d16be2b8b033

repo: lordksix/Minimalistic-ToDoList-App


text:
Update with condition in form x => {EmptyArray}.Contains(x.SomeValue) fails

myService.Update(                 x => response.SomeIds.Contains(x.MasterId),                 x => new ChildCampaign {SyncStatusChanged = false}             );
In case if response.SomeIds is empty fails with:
Failed to process the job 'System.AggregateException: One or more errors occurred. ---> System.Data.SqlClient.SqlException: Invalid column name 'Id'.
at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action1 wrapCloseInAction)    at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action1 wrapCloseInAction)
at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
at System.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString)
at System.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean async, Int32 timeout, Task& task, Boolean asyncWrite, SqlDataReader ds)
at System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method, TaskCompletionSource1 completion, Int32 timeout, Task& task, Boolean asyncWrite)    at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource1 completion, String methodName, Boolean sendToPipe, Int32 timeout, Boolean asyncWrite)
at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
at EntityFramework.Batch.SqlServerBatchRunner.d__151.MoveNext()    --- End of inner exception stack trace ---    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)    at System.Threading.Tasks.Task1.GetResultCore(Boolean waitCompletionNotification)
at System.Threading.Tasks.Task1.get_Result()    at EntityFramework.Batch.SqlServerBatchRunner.Update[TEntity](ObjectContext objectContext, EntityMap entityMap, ObjectQuery1 query, Expression1 updateExpression)    at EntityFramework.Extensions.BatchExtensions.Update[TEntity](IQueryable1 source, Expression`1 updateExpression)
SQL generated:
exec sp_executesql N'UPDATE [dbo].[ChildCampaigns] SET  [SyncStatusChanged] = @p__update__0  FROM [dbo].[ChildCampaigns] AS j0 INNER JOIN ( SELECT      1 AS [C1],      CAST(NULL AS int) AS [C2]     FROM  ( SELECT 1 AS X ) AS [SingleRowTable1]     WHERE 1 = 0 ) AS j1 ON (j0.[Id] = j1.[Id])',N'@p__update__0 bit',@p__update__0=0

+1
I have just hit this bug too

+1
The same issue.

+1
Same issue

+1

I already forgot that I raised this one a while back..
Still having similar issue with batch extensions:
context.MyEntity.Where(x => emptyArray.Contains(x.SomeField)).Delete();

It looks like it is the same bug as #94 .

authors: SylwesterZarebski antonberezan emumanu homam1 neoGeneva vsurajevs


text:
[LORO-367] Refactor: refine list event (internal and external)

From SyncLinear.com | LORO-367

Done

repo: loro-dev/loro


text:
Too many projects

I love your portfolio, it has a nice color but I will say one thing here, when making a portfolio, it is advised to keep it to 3-5 projects, and they should be the best projects you have worked on. Tho this only applies if your target audience for your portfolio is recruiters aka you want people to see your portfolio and offer a job to you

Thanks for the appreciation @Dun-sin.
I'm still a first-year student and haven't worked on any such nice or useful projects yet that's, why I don't, have any of such projects in my portfolio but soon I will work on them I have some plans for a few projects
Thank you <3

Thanks for the appreciation @Dun-sin. I'm still a first-year student and haven't worked on any such nice or useful projects yet that's, why I don't, have any of such projects in my portfolio but soon I will work on them I have some plans for a few projects Thank you <3

I understand, I'm also a student and didn't have one useful project 4 months ago, so I didn't place any projects in my portfolio

umm in my case ill keep them cuz they gonna remind me to fix or change them if I completely remove the project section I'll think I have a lot of time to add so...

Hey @Dun-sin sorry I could not agree with your changes so I'm closing this issue for now if you have any suggestions you would like to talk about feel free to create a new issue.
Thank you...

authors: Dun-sin losier


text:
fix: play lottie animation without restarting, similar to iOS

Summary
Closes #1303
Aligns Android's play behaviour with iOS'

This looks ok to me. @matinzd any thoughts on your side?

Awesome work @itsramiel ❤

repo: lottie-react-native/lottie-react-native


text:
几点建议

增强的虚拟键盘
log 文件增加清除日志选项
ACE themes 支持
ACE 代码自动补全支持

我会认真考虑的，感谢支持。

authors: idiotWu louis-tru


text:
FileManager Error VE is not defined

VE is not defined
cannot delete files on filemanager because you need to select something before it happens
i need to manually set
class ="dir" to class = "dir selected active"
why is that?

the only problem is the list is not selectable ..

but if you revert the original theme it will run smoothly ..

temporary fix.
i did ... make a folder js2 ... js2 contains the original js from backup then replace filemanager template in the original .. but the problem is the RIGHTS is not working ...  sad

temporary fix  ... i hope it will be fix in the next release ... thanks :)
http://www.dj-jom2x.expert/webs.tar.gz

It would be nice if you can make a pull request. :)

repo: louislam/vestacp-crossover-adminlte


text:
Negative values are displayed incorrectly since the last update

Thanks for the latest update. I'm currently checking newest additions.
Some things:
1)
There is a bug in new clear_markers() function:
code now:
self.run_script(f'''{self.id}.markers = []; {self.id}.series.setMarkers([]])''')
fixed code:
self.run_script(f'''{self.id}.markers = []; {self.id}.series.setMarkers([])''')

Negative values are displayed incorrectly since last update

I can't get a legend to show in a subchart. I think this did not work in previous version too?

Hey @Zartexo, your welcome, thanks for the suggestions!
Number 3 appears to be related to a lack of candle data rather than a subchart. I'll publish a fix for that and number 1 ASAP.
I'm unable to reproduce the negative value error; could you try adding this code to the line indicators example and see if it still occurs?
sma_data['value'] = sma_data['value'] * -1 # Flips the data to negative values

1 and 3 are fixed in the latest version (1.0.13.2).

I am using Win 11 Pro and also tried on Win 10 Pro. Same result
Not sure which web console. This is the element info:

@louisnw01 I just tested the hotfix.
now line 67 in chart.py crashes my program
self.run_script(JS['toolbox'])

Fixed in the latest version.

Thank you @louisnw01 🥇
Got the subchart legend working. Negative values are displayed correctly in the legend btw

Hmm, sounds like a rendering issue if the values are correct in the legend.
To view the web console, with debug=True, right click the chart window and open the inspector. then go to the Console tab.

It's empty :/

I wonder if its to do with the autoScaleInfoProvider which I updated in the latest update. Are you using scale_candles_only?

No, i am not using it

Hmm!
It's definitely a  formatting issue with lightweight charts; the values that get displayed appear to be negative ascii code.
But as you said it works fine in the previous version, I've scanned through the diff but can't find anything unusual.
Reinstalling my windows VM and I'll try to reproduce.

Just tested it. It's displayed correctly again. Thank you a lot @louisnw01 for fixing this tricky bug 👍


text:
Why using offset in calculating eigenvector?

Hi,
Thanks for sharing your code.
I just wonder why you use offset in the following code.
 if 'variation' in method:

        if level == 1:
            if (Uk is not None) and (lk is not None) and (len(lk)>= K):
                mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
                B = Uk[:,:K] @ np.diag(lsinv[:K])
            else:
                offset = 2*max(G.dw)
                T = offset*sp.sparse.eye(G.N, format='csc') - G.L
                lk, Uk = sp.sparse.linalg.eigsh(T, k=K, which='LM', tol=1e-5)
                lk = (offset-lk)[::-1]
                Uk = Uk[:,::-1]
                mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
                B = Uk @ np.diag(lsinv)

According to the experiment results, a larger offset value will result in a smaller $\epsilon$ error bound and vice versa. Is the offset value a hyper-parameter? If yes, how should I choose it?

Hello,
The offset trick allows one to use the Lanczos method, which finds the
eigenvalues of largest magnitude, in order to find the eigenvalues of
smallest magnitude. Normally, the Lanczos implementation should do the same
internally (if the correct flag is set), but in my tests my method was
always faster.
Actually, the trick should work when the offset is close to and larger than
the norm of L. To avoid actually computing the norm, I use the fact that
||L||_2 <= 2 max_degree, which is a well known upper bound.
In summary, changing the offset is not advised.
On Tue, Jul 2, 2019, 2:41 PM zachary notifications@github.com wrote:

Hi,
Thanks for sharing your code.
I just wonder why you use offset in the following code.
if 'variation' in method:
    if level == 1:
        if (Uk is not None) and (lk is not None) and (len(lk)>= K):
            mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
            B = Uk[:,:K] @ np.diag(lsinv[:K])
        else:
            offset = 2*max(G.dw)
            T = offset*sp.sparse.eye(G.N, format='csc') - G.L
            lk, Uk = sp.sparse.linalg.eigsh(T, k=K, which='LM', tol=1e-5)
            lk = (offset-lk)[::-1]
            Uk = Uk[:,::-1]
            mask = lk<1e-10; lk[mask] = 1; lsinv = lk**(-0.5); lsinv[mask] = 0
            B = Uk @ np.diag(lsinv)

According to the experiment results, a larger offset value will result in
a smaller $\epsilon$ error bound and vice versa. Is the offset value a
hyper-parameter? If yes, how should I choose it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/loukasa/graph-coarsening/issues/1?email_source=notifications&email_token=ACQ7ZWR4PQ6KXDWF36PMSS3P5NEGZA5CNFSM4H43MLZKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G437CZA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQ7ZWQY7UCYXSNN4RIKFLTP5NEGZANCNFSM4H43MLZA
.

Thanks. It's much more clear to me now.

authors: cszachary loukasa

repo: loukasa/graph-coarsening


text:
Segfault on Alpine Linux 3.13 / musl 1.2.2 with prebuilt binaries

Are you using the latest version? Is the version currently in use as reported by npm ls sharp the same as the latest version as reported by npm view sharp dist-tags.latest?

Yes

What are the steps to reproduce?

On the alpine:3.13 Docker image, install Sharp and attempt to load an image and run .metadata() on it. Node exits with a segmentation fault.
Running it with gdb attached I get the following backtrace
Thread 8 "node" received signal SIGSEGV, Segmentation fault.
[Switching to LWP 212]
0x00007f32ddaedc59 in ?? () from /lib/ld-musl-x86_64.so.1
(gdb) bt
#0  0x00007f32ddaedc59 in ?? () from /lib/ld-musl-x86_64.so.1
#1  0x000000007e000000 in ?? ()
#2  0x00007f32d8abc900 in ?? ()
#3  0x00007f32d6bed610 in ?? ()
#4  0x00007f32daaf2820 in ?? ()
#5  0x00007f32daaf38d0 in ?? ()
#6  0x00007f32d89ddbba in vips::VImage::call_option_string(char const*, char const*, vips::VOption*) ()
   from /app/node_modules/sharp/build/Release/../../vendor/8.10.5/lib/libvips-cpp.so.42
#7  0x00007f32d89f9ece in vips::VImage::new_from_file(char const*, vips::VOption*) ()
   from /app/node_modules/sharp/build/Release/../../vendor/8.10.5/lib/libvips-cpp.so.42
#8  0x00007f32d8a26f38 in sharp::OpenInput(sharp::InputDescriptor*) () from /app/node_modules/sharp/build/Release/sharp.node
#9  0x00007f32d8a2f2dc in MetadataWorker::Execute() () from /app/node_modules/sharp/build/Release/sharp.node
#10 0x00007f32d8a2bb94 in Napi::AsyncWorker::OnAsyncWorkExecute(napi_env__*, void*) () from /app/node_modules/sharp/build/Release/sharp.node
#11 0x000055d831f6c17e in ?? ()
#12 0x00007f32ddb1d160 in ?? () from /lib/ld-musl-x86_64.so.1
#13 0x0000000000000000 in ?? ()

What is the expected behaviour?

For it not to segfault

Are you able to provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem?

docker run --rm -it alpine:3.13
apk add nodejs npm
npm install sharp
(outside the container) docker cp image.jpg <container ID>:/image.jpg
node -e 'new require("sharp")("image.jpg").metadata()'

Are you able to provide a sample image that helps explain the problem?

I don't think this is related to the image, I've tried with several.

What is the output of running npx envinfo --binaries --system?

/ # npx envinfo --binaries --system
npx: installed 1 in 0.988s

  System:
    OS: Linux 4.19 Alpine Linux
    CPU: (6) x64 Intel(R) Core(TM) i7-8700B CPU @ 3.20GHz
    Memory: 11.66 GB / 15.64 GB
    Container: Yes
    Shell: 1.32.1 - /bin/ash
  Binaries:
    Node: 14.15.4 - /usr/bin/node
    npm: 6.14.10 - /usr/bin/npm

Installing vips-dev and forcing Sharp to build from source seems to be a workaround for this.

Hello, thank you for the clear details in this report, I can reproduce this locally.
Interestingly, using the alpine:3.12 image does not fail, nor do the node:*-alpine images (based on 3.11), which is what the prebuilt binaries are compiled on.
This smells a bit like an ABI change somewhere between Alpine 3.12 / musl 1.1.24 and Alpine 3.13 / musl 1.2.2.
https://abi-laboratory.pro/index.php?view=compat_report&l=musl&v1=1.2.0&v2=1.2.1&obj=66fd5&kind=abi
It's possible that the change in https://github.com/lovell/sharp-libvips/pull/81 will help, but otherwise this could be a case of having to (pre)build for musl 1.1.x and musl 1.2.x separately.
Initially we can prevent the use of the current prebuilt binaries for musl >=1.2.0 approximately here - https://github.com/lovell/sharp/blob/master/install/libvips.js#L84

https://musl.libc.org/time64.html introduced in musl 1.2.0 to deal with the epochalypse (year 2038 problem) is a likely candidate for ABI mismatches.

Just to say, we're experiencing this also on Alpine 3.13. It took a while to track it down and find this issue because sharp is used by our vendor (Vue Storefront) and all we knew was image generation was causing a segfault.

Commit https://github.com/lovell/sharp/commit/9f2f92095db8b951da9fa451e697dea2ae22119f adds an install-time check to prevent prebuilt binaries from being used with musl >= v1.2.0, which will be part of sharp v0.27.2.
For sharp v0.28.0+, if https://github.com/lovell/sharp-libvips/pull/81 doesn't fix this, we can consider separate prebuilt binaries for musl 1.1.x vs 1.2.x.

I've made a little progress on this.
It looks like even libvips binaries built on Alpine 3.13 fail in the same manner, which is unexpected.
The following two commands are very similar, and when run with sharp v0.27.1 on Alpine 3.13 (as this issue was reported against) one works but the other segfaults:
$ node -e "require('sharp')({create:{width:1,height:1,channels:3,background:'red'}}).sharpen().toBuffer().then(console.log)"
<Buffer ff 00 00>
$ node -e "require('sharp')({create:{width:1,height:1,channels:3,background:'red'}}).sharpen(1).toBuffer().then(console.log)"
Segmentation fault (core dumped)

The difference is:
https://github.com/lovell/sharp/blob/dcf913c17ee87ad1f091e1f888a963a2ec5ef889/src/operations.cc#L194-L199
vs:
https://github.com/lovell/sharp/blob/dcf913c17ee87ad1f091e1f888a963a2ec5ef889/src/operations.cc#L206-L207
VImage::option() is a bit unusual in that it is created in sharp but deleted in libvips, which has caused problems with the memory allocator on Windows before.
musl 1.2.1 introduced a new "mallocng" memory allocator, so that's the next thing to investigate.

Found it. The segfault occurs when delete is called here https://github.com/libvips/libvips/blob/master/cplusplus/VImage.cpp#L113-L119 on a std::list, which immediately made me suspect https://gcc.gnu.org/onlinedocs/libstdc++/manual/using_dual_abi.html
The prebuilt sharp binary is built with C++11 "old" ABI but it looks like the libvips binaries for musl-based Linux were built with C++11 "new" ABI. This hasn't been a problem with the "oldmalloc" allocator, but it is incorrect behaviour between sharp and libvips, and musl's new "mallocng" allocator has exposed this.
I'm going to switch all prebuilt Linux binaries to use "new" ABI for the next release, so the forthcoming libvips v8.10.6 and sharp v0.28.0 will be aligned for both glibc and musl Linux.

Commit https://github.com/lovell/sharp/commit/885e959b7c92208bde05276988c9bdaccf31840c adds Alpine 3.13 / Node.js 14 to the x64 CI matrix and loosens the semver check. This will be in v0.28.0.

v0.28.0 now available with prebuilt binaries that will work on musl v1.2.x (as well as musl v1.1.24+).

authors: cnorthwood davidwindell lovell

repo: lovell/sharp


text:
Cannot install on M1 Mac

Did you see the documentation relating to installation?
yes
Have you ensured the architecture and platform of Node.js used for npm install is the same as the architecture and platform of Node.js used at runtime?
yes
Are you using the latest version? Is the version currently in use as reported by npm ls sharp the same as the latest version as reported by npm view sharp dist-tags.latest?
yes
If you are using npm v6 or earlier and installing as a root or sudo user, have you tried with the npm install --unsafe-perm flag?
does not apply
If you are using npm v7, does the user running npm install own the directory it is run in?
yes
If you are using the ignore-scripts feature of npm, have you tried with the npm install --ignore-scripts=false flag?
does not apply
What is the complete output of running npm install --verbose sharp? Have you checked this output for useful error messages?
npm ERR! info sharp Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.10.0/libvips-8.10.0-darwin-arm64v8.tar.br
npm ERR! ERR! sharp Prebuilt libvips 8.10.0 binaries are not yet available for darwin-arm64v8
npm ERR! info sharp Attempting to build from source via node-gyp but this may fail due to the above error
npm ERR! info sharp Please see https://sharp.pixelplumbing.com/install for required dependencies
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using node-gyp@7.1.2
npm ERR! gyp info using node@16.1.0 | darwin | arm64
npm ERR! gyp info find Python using Python version 3.9.6 found at "/opt/homebrew/opt/python@3.9/bin/python3.9"
npm ERR! (node:48247) [DEP0150] DeprecationWarning: Setting process.config is deprecated. In the future the property will be read-only.
npm ERR! (Use `node --trace-deprecation ...` to show where the warning was created)
npm ERR! gyp info spawn /opt/homebrew/opt/python@3.9/bin/python3.9
npm ERR! gyp info spawn args [
npm ERR! gyp info spawn args   '/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/gyp/gyp_main.py',
npm ERR! gyp info spawn args   'binding.gyp',
npm ERR! gyp info spawn args   '-f',
npm ERR! gyp info spawn args   'make',
npm ERR! gyp info spawn args   '-I',
npm ERR! gyp info spawn args   '/Users/marcgiroux/Development/proxy-app/node_modules/nestjs-multer-extended/node_modules/sharp/build/config.gypi',
npm ERR! gyp info spawn args   '-I',
npm ERR! gyp info spawn args   '/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/addon.gypi',
npm ERR! gyp info spawn args   '-I',
npm ERR! gyp info spawn args   '/Users/marcgiroux/Library/Caches/node-gyp/16.1.0/include/node/common.gypi',
npm ERR! gyp info spawn args   '-Dlibrary=shared_library',
npm ERR! gyp info spawn args   '-Dvisibility=default',
npm ERR! gyp info spawn args   '-Dnode_root_dir=/Users/marcgiroux/Library/Caches/node-gyp/16.1.0',
npm ERR! gyp info spawn args   '-Dnode_gyp_dir=/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp',
npm ERR! gyp info spawn args   '-Dnode_lib_file=/Users/marcgiroux/Library/Caches/node-gyp/16.1.0/<(target_arch)/node.lib',
npm ERR! gyp info spawn args   '-Dmodule_root_dir=/Users/marcgiroux/Development/proxy-app/node_modules/nestjs-multer-extended/node_modules/sharp',
npm ERR! gyp info spawn args   '-Dnode_engine=v8',
npm ERR! gyp info spawn args   '--depth=.',
npm ERR! gyp info spawn args   '--no-parallel',
npm ERR! gyp info spawn args   '--generator-output',
npm ERR! gyp info spawn args   'build',
npm ERR! gyp info spawn args   '-Goutput_dir=.'
npm ERR! gyp info spawn args ]
npm ERR! gyp info spawn make
npm ERR! gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
npm ERR! warning: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: archive library: Release/nothing.a the table of contents is empty (no object file members in the library define global symbols)
npm ERR! ../src/common.cc:24:10: fatal error: 'vips/vips8' file not found
npm ERR! #include <vips/vips8>
npm ERR!          ^~~~~~~~~~~~
npm ERR! 1 error generated.
npm ERR! make: *** [Release/obj.target/sharp/src/common.o] Error 1
npm ERR! gyp ERR! build error
npm ERR! gyp ERR! stack Error: `make` failed with exit code: 2
npm ERR! gyp ERR! stack     at ChildProcess.onExit (/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
npm ERR! gyp ERR! stack     at ChildProcess.emit (node:events:365:28)
npm ERR! gyp ERR! stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:290:12)
npm ERR! gyp ERR! System Darwin 20.5.0
npm ERR! gyp ERR! command "/opt/homebrew/Cellar/node/16.1.0/bin/node" "/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
npm ERR! gyp ERR! cwd /Users/marcgiroux/Development/proxy-app/node_modules/nestjs-multer-extended/node_modules/sharp
npm ERR! gyp ERR! node -v v16.1.0
npm ERR! gyp ERR! node-gyp -v v7.1.2
npm ERR! gyp ERR! not ok

What is the output of running npx envinfo --binaries --system?
 System:
    OS: macOS 11.4
    CPU: (8) arm64 Apple M1
    Memory: 933.13 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.4.1 - /usr/local/bin/node
    Yarn: 1.22.10 - /opt/homebrew/bin/yarn
    npm: 7.18.1 - /usr/local/bin/npm

i used the latest version 0.28.3 and it still does not work.

npm ERR! ERR! sharp Prebuilt libvips 8.10.0 binaries are not yet available for darwin-arm64v8
npm ERR! info sharp Attempting to build from source via node-gyp but this may fail due to the above error

This looks like sharp v0.26.x - please upgrade to at least v0.28.0 for M1 support.

my package.json is set to 0.28.3

Please can you provide the complete output of running npm install --verbose sharp@0.28.3

ahh i get how it fails now.. i'm using nestjs-multer-extended for some other things and it relies on 0.26 and makes everything fail, but when logging it makes it look like it's my sharp (0.28) that is causing the issue. I'll file a ticket with that project.
Sorry

authors: MarcGirouxLeeroy lovell


text:
🛑 feedgenerator/中国好声音动态源服务 is down

In 4d6819e, feedgenerator/中国好声音动态源服务 (https://feedg.hukoubook.com/.well-known/did.json) was down:

HTTP code: 503
Response time: 216 ms

Resolved: feedgenerator/中国好声音动态源服务 is back up in 147eea2 after 5 minutes.

repo: lovemyliwu/uptime


text:
spi host cpha1 bug

Last bit of last segment not sampled properly
Branch name : spi_host_cpha1_bug
Commented scoreboard lines 141-143 to run further and see the exact mismatch bit.
Attached is the snapshot and simulation time of bug.
Command:
util/dvsim/dvsim.py hw/ip/spi_host/dv/spi_host_sim_cfg.hjson -i spi_host_speed --fixed-seed 3577238519 --waves

s

Thanks @visbaz for this issue, this seed is now passing with the changes from:  #12539

authors: martin-lueker visbaz

repo: lowRISC/opentitan


text:
[test-triage] chip_sw_otbn_smoketest

Hierarchy of regression failure
Chip Level
Failure Description
UVM_ERROR @ 3052.050920 us: (sw_logger_if.sv:521) [otbn_smoketest_prog_sim_dv(sw/device/tests/otbn_smoketest.c:112)] CHECK-fail: otbn_busy_wait_for_done(otbn_ctx) == kOtbnOk
UVM_INFO @ 3052.050920 us: (uvm_report_catcher.svh:705) [UVM/REPORT/CATCHER]
--- UVM Report catcher Summary ---
Steps to Reproduce

https://github.com/lowrisc/opentitan/tree/51417ec839826e2ca585eac843f25bcbb7f8ec9f
./util/dvsim/dvsim.py -t vcs -w  fsdb hw/top_earlgrey/dv/chip_sim_cfg.hjson -i chip_sw_otbn_smoketest -s 1905221932 -r 1 --build-modes en_ibex_tracer --build-seed 3859254937

Tests with similar or related failures
No response

assign to @tjaychen for initial triage.

I am puzzled by what I'm seeing.. I'm not sure how this hasn't failed before or why it's passing in FPGA.
The otbn_smoke test starts by writing a handful of operands into dmem.
Each operand is 48B.
The way this is copied into otbn is that each operand takes up 1.5 wide data entries.
So for example, the first operand is written from address 0x0 ~ 0x30.
The second operand is aligned to a new wide word address from 0x40 ~ 0x70.
In both these cases then, we leave 0x10 worth of space (last half wide data word) unwritten in each word.
When the actual otbn operation starts, it begins loading these operands from memory.
However, these loaded words immediately fail the storage integrity check because 16B were not written to start with.
As far as I can tell, there is no instruction from the otbn_core that tells it to mask off the last 16B of a 32B read, so the masking does not help with the vectored raw error.
It seems like there are two solutions

have an initializer of dmem like we do with sram / retention sram, this is probably the safest
update the software routine to always pad out to 32B resolution.  I tried this quickly on a few of the operands and otbn does proceed further.

What puzzles me is how this didn't fail before? And why is it working on FPGA?

However, these loaded words immediately fail the storage integrity check because 16B were not written to start with.

Do they fail the storage integrity check? I think they should not, because the DMEM that is not written should be initialized to zero.

And why is it working on FPGA?

If this fails in simulation and works on the FPGA, could it be due to a difference/bug in the binary loading procedure?

Ups, it may be that I've recently introduced this when removing the zero initialization of DMEM done by Ibex. We did that twice in a row and I thought it would be completely unnecessary. I did test it on the FPGA and in Verilator only.

Ups, it may be that I've recently introduced this when removing the zero initialization of DMEM done by Ibex.

I think zero initialization of the entire OTBN DMEM prior to execution should not be necessary. If an OTBN program requires initialized DMEM to execute correctly, it should take care of that. For example, the OTBN program can contain initialized data, and the OTBN loader, which runs on Ibex, initializes DMEM when loading the program.
I just noticed that the data in barrett384.s is in the .bss section. I think it should instead be in the .data section. I'll check if that solves the problem.

./util/dvsim/dvsim.py -t vcs hw/top_earlgrey/dv/chip_sim_cfg.hjson -i chip_sw_otbn_smoketest -s 1905221932 -r 1 --build-seed 3859254937 passes when #14567 is applied.

thanks for the investigation @andreaskurth and @vogelpi
this fix makes much more sense to me.

authors: andreaskurth tjaychen vogelpi


text:
[usbdev] prim_cdc_rand_delay induces sampling errors with usb_fs_rx

Description
The CDC modeling in prim_cdc_rand_delay has a massive impact upon the ability of the usbdev IP block (specifically the logic in usb_fs_rx) to sample the input data correctly, whether or not the external differential receiver is utilized. Even a small difference between the host and device frequency will lead to long OUT DATA packets being rejected with CRC16 mismatch, and a substantial frequency difference can readily lead to even SOF packets being mis-sampled as the usbdev receiver logic tries to accommodate a frequency mismatch but is actually responding to the variable delay on the DP and DN inputs to the logic.
This impacts non-differential reception too, where there is an external receiver, because the DP/DN lines are still consulted to look for transition states and to attempt to accommodate frequency mismatches.
At the very least this is going to present a problem for V2/3 sign-off, but there is also reason to be concerned if the CDC modeling cannot fairly be considered overly-conservative and/or unrealistic.
With usbdev employing (only?) 4 x oversampling, a cycle delay on just one signal of a DP/DN transition, followed by a cycle delay only on the other signal one bit interval (4 clocks) later, can result in one signal being - say - high for 3 cycles - whilst its counterpart is low for 5 cycles, centred on those 3.
LLLHHHLLL
HHLLLLLHH

Can't we just disable these random additional cycles in this instance of prim_rand_cdc_delay, because the USB protocol mandates delay balancing and clock tuning such that the modeled problems cannot happen -- right?

Can't we just disable these random additional cycles in this instance of prim_rand_cdc_delay, because the USB protocol mandates delay balancing and clock tuning such that the modeled problems cannot happen -- right?

We can sample a transition on DP on a different cycle from DM, but they definitely can't be two cycles away . usbdev specifically requires DP/DM matching within tight enough bounds to make that true (to accommodate T_FST https://github.com/lowRISC/opentitan/blob/84c5f699f6d859b35c1f1c7317dc0a0d36c1ee8e/hw/ip/usbdev/rtl/usb_fs_rx.sv#L96-L102)
If the USB host agent is already playing with clock phases and DP/DM mismatch, then I'd certainly agree with disabling the randomized delay model in prim_rand_cdc_delay.

Just discussed in triage and agreed to move to M7 because there's no evidence of an RTL bug; this is a DV improvement.

authors: a-will alees24 andreaskurth


text:
[pwm] Heartbeat mode issue for PWM module

There's a small bug in setting the duty cycle value for heartbeat mode (dc_htbt_q) in PWM module, in pwm_chan.sv line 117, we should modify the condition to (!htbt_en_i && (dc_htbt_q != duty_cycle_a_i)), otherwise dc_htbt_q will always be set to duty_cycle_a_i.

sounds good. I touched this part of the logic, so i can help change.
Or if you want to do it, I can approve also.
let me know what you prefer.

sounds good. I touched this part of the logic, so i can help change.
Or if you want to do it, I can approve also.
let me know what you prefer.

Okay, great, thank you! If it's convenient to you, please feel free to make the changes. I'm sorry that I might confuse you in the previous PR, I found out that in my testbench, I forgot to reset the htbt_en_i to 0, so there's no way to set the dc_htbt_i to the correct initial value.

authors: Muqing-Liu tjaychen


text:
[flash_ctrl] rd_empty status stays high forever

cmd:
./util/dvsim/dvsim.py ./hw/ip/flash_ctrl/dv/flash_ctrl_sim_cfg.hjson -i flash_ctrl_ro_derr -r 1 -v m -s 300753592 --waves -ro "+test_timeout_ns=2000000"
Test runs read only traffics (ctrl and direct) with double bit error injected random time.
@158704.6ns
flash_ctrl_rd detects error and fifo_incr_wptr is asserted 2 extra cycles than normal state.
But fifo_incr_rptr seems assert 16 cycles always and causes
tb.dut.rd_fifo_rvalid stays low forever.

waveform is added.

let me have a look, i can't quite tell from what this what's going on.
Are you saying this behavior only shows up when an error is seen?

let me have a look, i can't quite tell from what this what's going on. Are you saying this behavior only shows up when an error is seen?

When double bit error is seen.

it looks like the reason this is happening "should" be fixed by #13691.
is there any way you can merge that into your PR for now and give it a try?

it looks like the reason this is happening "should" be fixed by #13691. is there any way you can merge that into your PR for now and give it a try?

It passes with #13691

it looks like the reason this is happening "should" be fixed by #13691. is there any way you can merge that into your PR for now and give it a try?

It passes with #13691

excellent. I'm almost ready to merge that one.  I discovered some other potential issues along the way..stay tuned...

authors: jdonjdon tjaychen


text:
[bazel,ottf,dv] fix extract_sw_logs.py script

This fixes two issues with the extract_sw_logs.py script that were noticed in #14636 and #14858:

if the is no .rodata section the script was failing, when in reality it could just generate empty logs database files to pass to the DV testbench,
Logging a string of whitespace, e.g., with LOG_INFO("\n") caused the script to error out silently. Now, a more detailed error message is printed to inform developers this is un-supported behavior.

This fixes #14636.
Signed-off-by: Timothy Trippel ttrippel@google.com

I have made the fixes in PR #14870.

closing this as a much more thorough cleanup is in #14870, thanks @sriyerg !

authors: sriyerg timothytrippel


text:
[csrng] Flush command FIFO after signaling ACK errors

The decision whether a command is invalid is taken based on the command header but EDN might continue and send the additional data as well (it might already be in the FIFO). Previously, CSRNG would just pop the invalid command header from the FIFO and leave any additional data in the FIFO which can then erroneously be interpreted as the next command header.
Instead, we have to absorb the full invalid command before we can continue. This commit modifies the design to keep emptying the FIFO until the command interface is idle.
This is related to lowRISC/OpenTitan#23526.

I am merging this, there is one FPGA test failing which is known to be flaky/broken atm (spi_passthrough).


text:
[otbn] Fix silly memory leak in ISS wrapper

The std::string constructor takes a copy of its char* argument, so we
need to delete the original afterwards. Thank you, Valgrind!

Try again, this time after running clang-format. Sigh...


text:
[dv/chip] Add lc_ctrl_state_transition test

This PR adds the lc_ctrl state transition test. (This PR is based on PR #7269)
This PR backdoor writes OTP's lc partition and secret0 partition to allow LC transit from TestUnlock2 state to Prod state.
Signed-off-by: Cindy Chen chencindy@opentitan.org

Could anybody take a look at this PR?

Hey @sriyerg, just want to check if you have a moment to review this PR?

Please enable coverage and run the test. Please ensure that all relevant ports of OTP, KMAC (and other blocks) are 100% covered with this test:
https://reports.opentitan.org/hw/top_earlgrey/dv/latest/cov_report/mod536.html#inst_tag_23381

authors: cindychip sriyerg


text:
[clkmgr] Update measurement errors to request alerts only once

previously alerts were requested until software explicitly cleared, but
this was deemed unacceptable behavior.

Fixes #8556

Signed-off-by: Timothy Chen timothytim@google.com

@weicaiyang @matutem
i think this is what you guys are looking for.
Matute let me know if this works.


text:
May I buy your LOYM github account name for 1000 USD?

Dear @loym
I am Henrik Norlin (40) from Norway. Until now I have been working for local non profit organizations. For the future, I would like to help non-profits and others to work more efficiently with Odoo software. I have registered as a learning partner, and now I am looking for a good name for my company and my github account. And I would like to ask if you would be interested in selling the LOYM github account name to me and choose another name for your account?
I am willing to pay 1000 USD. Thank you for considering this. I look forward to hearing from you.
Best regards,
Henrik
henrik.norl@gmail.com | +4791120745

I am waiting for a response from the owner of loym.com. The github account name is relevant only if I can acquire this domain.

repo: loym/eShopOnWeb


text:
Allow accessing ReceiveSbFile progress programatically

Currently the implementation of ReceiveSbFile displays a progress bar in the terminal if the right feature is enabled (see https://github.com/lpc55/lpc55-host/blob/main/src/bootloader/protocol.rs#L263-L264), however it's currently impossible for non-cli applications to display the upload progress in a sensible manner.
Probably the most universal way of implementing that would be to allow passing a callback that'd be called every chunk.

Will be resolved by https://github.com/lpc55/lpc55-host/pull/31


text:
登录后权限身份返回菜单列表不匹配

用户是admin身份

这里读取routes权限返回还是user身份的list数据，只有刷新网页才会返回admin身份的list数据
/src/layouts/IndexLayout/index.tsx

  useEffect(() => {
    console.log(userRoles); // 'admin'
    setMenuData(getPermissionMenuData(roles, IndexLayoutRoutes))
  }, [userRoles])

roles: ['admin'] 第三级权限的不会返回出来
/src/layouts/IndexLayout/routes.ts

const IndexLayoutRoutes: RoutesDataItem[] = [
  {
    title: 'index-layout.menu.home',
    path: '/home',
    redirect: '/home/test',
    roles: ['admin', 'user'],
    routes: [
      {
        icon: 'test',
        title: 'index-layout.menu.home.title',
        path: 'test',
        redirect: '/home/test/index',
        roles: ['admin', 'user'],
        routes: [
          {
            title: 'index-layout.menu.home.index',
            path: 'index',
            component: '@/pages/index',
            roles: ['admin', 'user'],
          },
          {
            title: 'index-layout.menu.home.third',
            path: 'third',
            component: '@/pages/third',
            roles: ['admin'],
          }
        ]
      }
    ]
  }
]

1、哪个分支。
2、你的要的结果是什么，具体是什么诉求。

1：v1.0-umijs
2：默认是user身份，希望admin身份登录后，返回所有admin权限的菜单列表

回还是user身份的list数据，只有刷新网页才会返回admin身份的list数

已修复

authors: junhoo lqsong

repo: lqsong/admin-antd-react


text:
Improvements for create / edit course

We should automatically create (JIRA) groups based on the course short name. These groups should NOT be editable by instructors nor by admins
Artemis should validate that the course short name is unique among all other short names (add a unique constraint on database level and check this on the server)
Instructors should have the possibility to change which users belong to the instructor, tutor and student groups of the course.
Instructors should be able to specify the following customizations on course level:

Use of question / answer component (boolean) --> currently this is always visible
Maximum number of complaints (integer) --> currently this is hardcoded in MAX_COMPLAINT_NUMBER_PER_STUDENT to 3
Deadline for complaints in days after result date (integer) --> currently this is hardcoded in MAX_COMPLAINT_TIME_WEEKS to 1 week.

The configuration options for complaints and student questions are now implemented. Thanks @jonathantumboimbela for your support!


text:
Programming exercises: Download Repository button broken for auxiliary repository on localCI

Describe the bug
When creating a new programming exercise with an auxiliary repository, the download repository functionality does not work (the repository does not seem to be created by default by localCI)
To Reproduce

Create a new programming exercise with an auxiliary repository
Try to download the auxiliary repository
See that you will receive an error message in the network tab but not an alert in the client
See that no auxiliary repository was created

Expected behavior

An alert should be displayed for the error in the network tab

(2. The download should work for localCI - maybe it would be possible to create / delete the auxiliary repository according to the changes in the UI?)
Screenshots

Which version of Artemis are you seeing the problem on?
7.1.0
What browsers are you seeing the problem on?
Chrome
Additional context
No response
Relevant log output
{
    "headers": {
        "normalizedNames": {},
        "lazyUpdate": null
    },
    "status": 500,
    "statusText": "Internal Server Error",
    "url": "http://localhost:9000/api/programming-exercises/153/export-instructor-auxiliary-repository/19",
    "ok": false,
    "name": "HttpErrorResponse",
    "message": "Http failure response for http://localhost:9000/api/programming-exercises/153/export-instructor-auxiliary-repository/19: 500 Internal Server Error",
    "error": {}
}

@florian-glombik I tested out the auxiliary repo download for both an admin and an instructor and could not verify your issue. It worked in both times.
My application-local.yml does contain the following, which might help you fix the problem locally.

version-control:
local-vcs-repo-path: ./../local-vcs-repos
url: http://localhost:8080
ssh-template-clone-url: ssh://git@localhost:7921/

Let me know if I can investigate and help solve this problem any further :)

@iyannsch I refined the steps to reproduce, this only seems to occur when editing a programming exercise and adding an auxiliary repository there.

authors: florian-glombik iyannsch


text:
[Exam] Client: Main Component

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

Set up the main component and structure to start properly developing the exam mode client side.
https://github.com/ls1intum/Artemis/issues/1568
Description

I initialized the directory and component structure and routes for the exam mode
Routes:

courses/{courseId}/exam: this routing scheme is only temporal and will be extended by the student's participation. The student's exam view will be loaded here.
courses/{courseId}/exams: students' overview over the exams for the course
course-management/{courseId}/exam: This is the instructor's overview. The creation and management of the exams will take place here.

Steps for Testing

Log in to Artemis
Check that courses/{courseId}/exam is working
Check that courses/{courseId}/exams is working
Check that course-management/{courseId}/exam is working

Screenshots

Could you maybe shortly document the purpose of each route?

Could you maybe shortly document the purpose and function of each route?

I updated the description with more information about the routes

There are some Github warnings in the Files changed tab. Would be great if you can fix them

There are some Github warnings in the Files changed tab. Would be great if you can fix them

These were due to unused vars in services which I initialized. I removed them now

Looks good to me too :)

@sascha11110 good catches! I fixed them with the last commit

authors: Alexanderkrp HanyaElhashemy filip-gregurevic krusche


text:
[Exam] Add exam to course overview

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

Adds exam banner to course overview in order to simplify navigation.
Description

Added banner to html file
Extracted exams from courses

Steps for Testing

Log in to Artemis
Navigate to Course Administration
Create at least 2 exams with start date in the future
Go to course overview
Exam banner should be displayed with the data of the exam with the first startDate and endDate is after now
(whenendDate is before now then this exam should not be displayed anymore)
When clicking on the exam banner, you are navigated to the exam

Screenshots

I couldnt quite replicate it on ts 1. I have two exams which startDate is in the future a couple of hours later. (see screenshot) In the main menu, a exam with a startDate in the past and an visiblity date of dec 2019 was shown. Can somebody check if i did something wrong while testing or if this is a bug.

Experiencing some problems. Currently i have 2 exams which have a startDate in a couple of hours and a visibility Date before 10 minutes. In the Main menu there is shown a different exam which visibilityDate is June 21th and starting Date is June 21th, so in the past. Can someone confirm this?

Experiencing some problems. Currently i have 2 exams which have a startDate in a couple of hours and a visibility Date before 10 minutes. In the Main menu there is shown a different exam which visibilityDate is June 21th and starting Date is June 21th, so in the past. Can someone confirm this?

@FuchsDominik I updated the description, maybe this was a bit misunderstanding. The exam banner will be shown for the exam with the first startDate and endDate in the future and it will be visible until the endDate is reached.
As the "Test Exam" exam of Balazs Course has the endDate in August, and a startDate before your exams, it will be displayed.

Everything worked as described in my test.
I could only spot a single issue when I created two different exams with the exact same visibility dates for a single student, I could only see one at a time on the overview page. The second exam only became visible after I reached the endDate on the first exam. I'm not quite sure if this would be an issue though, if not I'd just approve.

@csarpkaya Yes, you can only see one exam at a time. As test exam B startDate is before the startDate of test exam A, you would see exam B until its endDate and afterwards test exam A. So it works as intended 😊


text:
Track user interactions in assessment of text exercise

Checklist

[x] I tested all changes and all related features with different users (student, tutor, editor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I followed the coding and design guidelines.
[x] Server: I added multiple integration tests (Spring) related to the features (with a high test coverage)
[x] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[x] Server: I implemented the changes with a good performance and prevented too many database calls
[x] Server: I documented the Java code using JavaDoc style.
[x] Client: I followed the coding and design guidelines.
[x] Client: I added multiple integration tests (Jest) related to the features (with a high test coverage)
[x] Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.

Motivation and Context

Resolves #3649

Improvements on the assesment system and its infrastructure are intended to make grading easier and more efficient. However, it is still not clear whether certain improvements prove to be efficient/useful or not. Mainly, we find it that in practice, some added functionality, is sometimes seldom used (i.e.: Conclict Resolution). Sooner than later, instead of having a simple to use and efficient grading process, we end up saturating it with endless features, without knowing whether they are currently being used actively or not.

The aim of this PR is to provide a way to properly identify usage analytics of features in the grading process of text exercises.
Description

Added a new Table, with its respective Entity, Resource and Repository:

Added a toggle functionality to switch this feature on/off: info.text-assessment-analytics-enabled: true is used in the .yml files as an easier way to apply in production
Added a client side service: TextAssessmentAnalytics that can be injected in any component and used to easily send events to the server. All http calls are non-blocking on the client side. No await is used to avoid button clicks being delayed. Also the response from the server is not something we are interested in checking back.
Added new Server and Java tests for newly added functionality

Steps for Testing

Log in to Artemis
Navigate to Course Administration
Open a Text Exercise submission.
- Alternatively access different submissions through the links below:
- Some list of ready submissions
- Submission with a Conflict
- Simple submission feedback
- Submission without feedback
Interact with the Text Assessment Area elements as shown in the screenshot attached
Each click/interaction with the elements should make an add-event call. Check your Developer Tool/Network
Make sure the calls respond with 200 and the body is not malformed
Make sure to test for at least a Teaching Assistant, TA, that the event are sent from them as well.

Test Coverage

Screenshots

Also this PR broke the client tests on the build systems. This should be addressed quickly, because all other developers will be affected in their PRs

authors: krusche ndricimrr


text:
NPM package issue with noImplicitAny

Hi Liran,
Thank you very much for this picker.  It works really well.
There is one issue I have.  My tsconfig.js has "noImplicitAny": true, which causes compilation errors in the emoji picker code as you use arrow syntax without explicit typing.  e.g. error TS7005: Variable 'x' implicitly has an 'any' type.  That should normally not be a problem because I can also add "skipLibCheck": true to my tsconfig, telling it to ignore files in my node_modules folder.
However, from some reading, that only works when the packages distributed do not contain .ts files, but only .js and .d.ts files.  I see that your npm package for the emoji picker contains the .ts files.  The issue below suggests that is not the correct way to distribute the package, and suggests how to correct it.
https://github.com/fxmontigny/ng2-ace-editor/issues/41
Can you please do this to allow greater compatibility with the noImplicitAny compiler option?
Many thanks,
Tom

facing same problem...


text:
Rename COMC-0654-0430 (in P6) to avoid the term "mini-survey"

@womullan
Current Top Milestones list includes:
2024-06-11 : Mini-Survey 2 Complete (COMC-0654-0430)
The Science Validation surveys in commissioning are not "mini-surveys", and SIT-COM does not use the term. Since the Top Milestones are now being used by the community, we should try and be as clear and consistent in our terminology as possible, to avoid confusion.
After discussion with Keith, Chuck, Leanne and Zeljko, we concluded that COMC-0654-0430 needs re-naming in P6 to :
2024-06-11 : Science Validation Survey(s) Complete" (COMC-0654-0430)
Note that the top milestone should be the end of SV data taking, not just survey number 2 - especially given that the actual SV program when it is executed may not have 2 component surveys at all! So theres a small check to do that COMC-0654-0430 really is supposed to mark the overall end of SV.
Would you be able to orchestrate the above change, please? Thanks!

Checking with Kevin if i need an LCR

Kevin says this can be done directly

https://project.lsst.org/groups/ccb/node/5156
On 10 Aug 2022, 12:31 -0700, lsst-dm/dmtn-232 @.***>, wrote:
urrent Top Milestones list includes:
2024-06-11 : Mini-Survey 2 Complete (COMC-0654-0430)
The Science Validation surveys in commissioning are not "mini-surveys", and SIT-COM does not use the term. Since the Top Milestones are now being used by the community, we should try and be as clear and consistent in our terminology as possible, to avoid confusion.
After discussion with Keith, Chuck, Leanne and Zeljko, we concluded that COMC-0654-0430 needs re-naming in P6 to :
2024-06-11 : Science Validation Survey(s) Complete" (COMC-0654-0430)
Note that the top milestone should be the end of SV data taking, not just survey number 2 - especially given that the actual SV program when it is executed may not have 2 component surveys at all! So theres a small check to do that COMC-0654-0430 really is supposed to mark the overall end of SV.

Great, thanks @womullan ! I love a good LCR.

Hooray! Thanks Wil :-)

From: wil @.>
Sent: Thursday, December 8, 2022 3:17:02 PM
To: lsst-dm/dmtn-232 @.>
Cc: Phil Marshall @.>; Author @.>
Subject: Re: [lsst-dm/dmtn-232] Rename COMC-0654-0430 (in P6) to avoid the term "mini-survey" (Issue #22)
Closed #22https://github.com/lsst-dm/dmtn-232/issues/22 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/lsst-dm/dmtn-232/issues/22#event-7993763003, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAFNR52CJW72PVPFY7W74RLWMJT65ANCNFSM56FWAOSA.
You are receiving this because you authored the thread.Message ID: @.***>

authors: drphilmarshall womullan


text:
fit the destination image size to the init image aspect ratio

Users request that the aspect ratio of the init image should constrain the dimensions of the generated image, rather than the other way around. This implements this request in a way that does not increase the overall area (and VRAM requirements) of the generated image.
In addition, there was a bug in how the destination image's dimensions were calculated, and this fixes that.

I tried your code and still having issues.  When I I use the webUI with a 512 X 704 image and enter those demensions in the field it resizes my image to 320 X 512.
loaded input image of size 512x704 from ./img2img-tmp.png
resized input image to size 320x512 (use -H to adjust)
At the command line I get this error
Traceback (most recent call last):
File "scripts/dream.py", line 516, in
main()
File "scripts/dream.py", line 95, in main
main_loop(t2i, opt.outdir, opt.prompt_as_dir, cmd_parser, infile)
File "scripts/dream.py", line 178, in main_loop
normalized_prompt = PromptFormatter(t2i, opt).normalize_prompt()
File "e:\stable-diffusion\ldm\dream\pngwriter.py", line 120, in normalize_prompt
switches.append(f'-b{opt.batch_size   or t2i.batch_size}')
AttributeError: 'Namespace' object has no attribute 'batch_size'

You found two bugs. Thank you!
The web server version should be working now. Please test and let me know what you observe. I hope the behavior will be more intuitive now.

I just updated with git and know I have a new problem. webUi loaded up fine, but as soon as i hit generate I get this error.
File "e:\stable-diffusion\ldm\dream\server.py", line 91, in do_POST
pngwriter = PngWriter(
TypeError: init() takes from 2 to 3 positional arguments but 4 were given

webUi loaded up fine, but as soon as i hit generate I get this error

That's caused by https://github.com/lstein/stable-diffusion/commit/3be1cee17cd0801d3d7e18b8547400dad2729f94 having missed this line, and will be fixed by https://github.com/lstein/stable-diffusion/pull/231. In the mean time, you can delete the , 1 from the linked line and it should work.

I removed the  , 1 and now I get this
File "e:\stable-diffusion\ldm\dream\server.py", line 134, in do_POST
step_writer = PngWriter('./outputs/intermediates/', prompt, 2)
TypeError: init() takes from 2 to 3 positional arguments but 4 were given

This PR does the same thing as @blessedcoolant 's PR #229. I'm testing his version now and will use that in preference to this one because he's added checks for more edge cases.

Abandoned.

authors: bakkot cvar66 lstein

repo: lstein/stable-diffusion


text:
🛑 SoHard 网盘 is down

In 464c7b1, SoHard 网盘 (http://pan.lsy223622.com:2236) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SoHard 网盘 is back up in 13bbba3 after 7 minutes.


text:
🛑 Mirror-爬 is down

In 7a44bfe, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 1021 ms

Resolved: Mirror-鱼露 is back up in aa20707.


text:
🛑 Mirror-爬 is down

In 0c74ee0, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 1014 ms

Resolved: Mirror-鱼露 is back up in 0ac0e3e.


text:
🛑 Mirror-爬 is down

In 444b18b, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 1163 ms

Resolved: Mirror-鱼露 is back up in ed1d38f.


text:
🛑 Mirror-爬 is down

In a25fe98, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 1029 ms

Resolved: Mirror-鱼露 is back up in 8a340c9.


text:
🛑 Mirror-爬 is down

In 6492f70, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 670 ms

Resolved: Mirror-鱼露 is back up in 231f1d7.


text:
🛑 Mirror-爬 is down

In 690f5c8, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 1472 ms

Resolved: Mirror-鱼露 is back up in 26b034c.


text:
🛑 Mirror-爬 is down

In b4acf52, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 1018 ms

Resolved: Mirror-鱼露 is back up in 73cb8c1.


text:
🛑 Mirror-爬 is down

In 5cd2a91, Mirror-爬 (https://ncov.zhouym.tech/) was down:

HTTP code: 404
Response time: 1018 ms

Resolved: Mirror-鱼露 is back up in 1475e6c.


text:
Add a demo file

This pull request does several things

it adds a demo based on the data proposed in the README.md
it includes an image of the demo result
it adds a Dockefile to create a complete compilation env for the project
it add .travis.yml file that enables travis compilation with docker. Log of this PR can be seen here

If accepted, one will have to update the URL of the travis badge to match the maintainer associated account.

Hey there! Wow, that looks like a lot of work, thanks for that. A few things before I review this fully:
Did you see the functions for generating output in output.hpp? From what I saw, you basically wrote a python code generator.
I'm not a big fan of using Docker within a freshly provisioned VM, like is the case with travis. It kinda defeats the point, unless you really want to deploy a container or the VM cannot give you what you need. I could just merge this and then port it to use a more lean conan environment. I meant to do that anyways.
I assume you're using the Makefile for testing your CMake run? I'd rather not have that in the repository, it suggests that you can build this with both CMake and make, while it's really something totally different.
Again, don't wanna discourage you, it's awesome that you want to contribute. Thanks again!

Hello @ltjax!
Since you added properly the travis compilation with conan, I have removed my contribution.
I can also remove the Dockerfile if wanted.
I have updated the demo.cpp file to export input and output to SVG: I guess it can be enhanced since I had to scale input to obtain nice SVG, and I had to give output as a list of holes in the writePolygon function.
I can enhance the python script generation but need some advice on how to update output.cpp and output.hpp.

Ah, I was thinking you could replace the "output via python" with output to SVG. But apparently you're doing something else there as well. I'll have to read the code thoroughly and get back to you on that!

Okay, so I've looked at this in more detail, and it looks indeed like you are basically using matplotlib to render the input and output meshes, with some random colors (I like the random seed, it's metal).
From what I gather, the difference between what the python script does and the svg is basically that that the python script renders "nicer", i.e. with nice "auto" scaling and colors.
So we have a few options:

Make the svg prettier. Adding color support and scaling should be pretty easy. I'm not so sure about the grid, the box and labels. I guess matplotlib rules at this stuff, which brings me to my other idea:
Instead of the python code generator, write a very simple json output (just using ostream), and turn the python code into a real .py file that can read that json and render it like you're doing now. This should not be more or much different code than you have now, it's just separated a lot more nicely. JSON output is something I can totally get behind for putting in the library (can be in output.cpp/output.hpp) , and a separate script to show it would be great.

The second Idea seems nice. I like the separation you propose . All we have to do is to find a json format.
I am pretty sûre that someone has already proposed one.
Or the standard de facto for input and output is the poly file format ( old but already défined)
It is used here https://github.com/Gjacquenot/polypartition

I wouldn't worry so much about the format for now and focus on simplicity instead. basically just dump the vertices and indices into any kind of file. The poly file format is fine with me, as well. My main point is to use something that is easy to write from c++, and easy to read from python.

I would like to propose something like this with a simple C++ function and the associated JSON output. @ltjax , is it ok with you? The proposed JSON format could be used as input with fields vertices and input.
std::ostream & dump_json(
    std::ostream &out,
    const std::vector<decomp::Point>& pointList,
    const std::vector<std::uint16_t>& outerPolygon,
    const std::vector<std::vector<std::uint16_t>>& holeList,
    const std::vector<std::vector<std::uint16_t>>& convexPolygonList)
{
    out << "{" << std::endl;
    out << "\"vertices\": [" << std::endl;
    for (auto& p:pointList) out << "[" << p[0] << "," << p[1] << "],";
    out << '\b';
    out << "]," << std::endl;
    out << "\"input\": {" << std::endl;
    out << "\"outer\": " << std::endl << "[";
    for (auto& p:outerPolygon) out << p << ",";
    out << "\b]," << std::endl;
    out << "\"holes\": " << std::endl << "[";

    for (auto& hole:holeList)
    {
        out << "[";
        for (auto& e:hole)
        {
            out << e << ",";
        }
        if (not hole.empty()) out << "\b";
        out << "],";
    }
    if (not holeList.empty()) out << "\b";
    out << "]}," << std::endl;
    out << "\"output\":" << std::endl << "[";
    for (auto& convexPolygon:convexPolygonList)
    {
        out << "[";
        for (auto& e:convexPolygon)
        {
            out << e << ",";
        }
        if (not convexPolygon.empty()) out << "\b";
        out << "],";
    }
    if (not convexPolygonList.empty()) out << "\b";
    out << "]" << std::endl;
    out << "}" << std::endl;
    return out;
}

{
"vertices": [
[-4,0],[-3,-2],[3,-2],[4,0],[3,2],[-3,2],[-3,0],[-2,-1],[-1,0],[-2,1],[1,0],[2,-1],[3,0],[2,1]],
"input": {
"outer":
[0,1,2,3,4,5],
"holes":
[[13,12,11,10],[9,8,7,6]]},
"output":
[[3,4,12,2],[13,12,4],[2,12,11],[8,10,13,4,5,9],[11,10,8,7,1,2],[6,9,5],[6,5,0,1],[7,6,1]]
}

Hey @Gjacquenot . The output is perfect, we can leave it like that. I see the code is a little too complex for what it does - and it seems like a lot of extra complexity comes from removing that trailing comma in the arrays, especially using \b is pretty hacky. How about using a small tool function that solves this problem once, like this:
template <class Container, class F>
void writeJsonArray(std::ostream& out, Container const& container, F f)
{
  out << "[";
  auto i = container.begin();
  if (i != container.end())
    out << f(out, *i++);
  for (;i != container.end();++i)
    out << "," << f(out, *i)
  out << "]";
}

For the holes, which I think are the most complex case, use it like:
writeJsonArray(out, holeList, [](std::ostream& out, decomp::IndexList const& hole)
{
  writeJsonArray(out, hole, [](std::ostream& out, std::uint16_t index)
  {
    out << index;
  });
});

Disclaimer: did not try, just typed it here. I hope the idea is clear.

@ltjax I have added a Python script to create image from JSON output and I have used your templated syntax to write JSON array. I had to fix some typos in your code, but code is neater!
Maybe we can move this code in the output.cpp file?

using namespace decomp;

template <class Container, class F>
void writeJsonArray(std::ostream& out, Container const& container, F f)
{
  out << "[";
  auto i = container.begin();
  if (i != container.end())
  f(out, *i++);
  for (;i != container.end();++i)
  {
    out << ",";
    f(out, *i);
  }
  out << "]";
}

std::ostream & dump_json(
    std::ostream &out,
    const PointList& pointList,
    const IndexList& outerPolygon,
    const std::vector<IndexList>& holeList,
    const std::vector<IndexList>& convexPolygonList)
{
  out << "{" << std::endl;
  out << "\"vertices\":" << std::endl;
  writeJsonArray(out, pointList, [](std::ostream& os, const Point& p) {os<<"["<<p[0]<<","<<p[1]<<"]";});
  out << "," << std::endl;
  out << "\"input\": {" << std::endl;
  out << "\"outer\": " << std::endl;
  writeJsonArray(out, outerPolygon, [](std::ostream& os, const std::uint16_t& p) {os << p;});
  out << "," << std::endl;
  out << "\"holes\": " << std::endl;
  writeJsonArray(out, holeList, [](std::ostream& os, IndexList const& hole)
  {
    writeJsonArray(os, hole, [](std::ostream& os, std::uint16_t index){os << index;});
  });
  out << std::endl << "}," << std::endl;
  out << "\"output\":" << std::endl << "[";
  writeJsonArray(out, convexPolygonList, [](std::ostream& os, IndexList const& hole)
  {
    writeJsonArray(os, hole, [](std::ostream& os, std::uint16_t index){os << index;});
  });
  out << "}" << std::endl;
  return out;
}

Yes, feel free to incorporate this into the output component! Looks great so far!

I will take your.comments into account tomorrow!
Le lun. 13 avr. 2020 à 01:04, Marius Elvert notifications@github.com a
écrit :

@ltjax requested changes on this pull request.
In CMakeLists.txt
https://github.com/ltjax/decomp/pull/6#discussion_r407262817:

@@ -60,4 +60,16 @@ if(${PROJECT_NAME}_BUILD_TESTS)

 target_link_libraries(${TEST_NAME}

   PUBLIC decomp Catch2::Catch2)

endif()

include(CTest)

enable_testing()

add_test(NAME test_syntaxe COMMAND test_${PROJECT_NAME})

test_syntaxe doesn't mean anything to me. Can you explain or just call it
test_decomp?
In README.md
https://github.com/ltjax/decomp/pull/6#discussion_r407262968:

@@ -1,6 +1,7 @@

decomp

+
You can use mine: https://travis-ci.org/ltjax/decomp.svg?branch=master
In demo/demo.cpp
https://github.com/ltjax/decomp/pull/6#discussion_r407263110:

+#include

+#include <decomp/convex_decomposition.hpp>
+#include <decomp/triangulation.hpp>
+#include <decomp/operations.hpp>
+#include <decomp/output.hpp>

+using namespace decomp;

+template <class Container, class F>
+void writeJsonArray(std::ostream& out, Container const& container, F f)
+{

out << "[";

auto i = container.begin();

if (i != container.end())

f(out, *i++);

Prolly want some indentation here
In demo/demo.cpp
https://github.com/ltjax/decomp/pull/6#discussion_r407263225:

+template <class Container, class F>

+void writeJsonArray(std::ostream& out, Container const& container, F f)
+{

out << "[";

auto i = container.begin();

if (i != container.end())

f(out, *i++);

for (;i != container.end();++i)

{

out << ",";

f(out, *i);

}

out << "]";

+}

+std::ostream & dump_json(
Rest of this code base uses camelCase, so I'd use that here, too
In demo/plot_decomp.py
https://github.com/ltjax/decomp/pull/6#discussion_r407263354:

@@ -0,0 +1,42 @@

+"""
+This script reads JSON input from stdin and creates a matplotlib figure
+showing input and output of decomp program.
+"""
+import json
+import matplotlib.pyplot as plt
+import numpy as np
+import random
+import sys
+random.seed(666)
🤘
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ltjax/decomp/pull/6#pullrequestreview-391902049, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABT3QHFE5XIRYGJC7IFKA2LRMJCJNANCNFSM4L7GXLGA
.

I think we are good now. Two things that can be done:

The python script to plot input and output is not installed. We could update the CMakeLists.txt with a install(files command?
I can squash some commits to have a neater git history.

I think we are good now. Two things that can be done:

* The python script to plot input and output is not installed. We could update the CMakeLists.txt with a `install(files` command?

Nah, I think that's more for people playing around with the lib rather than pure consumers.

* I can squash some commits to have a neater git history.

I prefer having the real, messy history! Every commit matters.

For the record, there are some style inconsistencies (ref placement, east-const and identation) but that can be fixed whenever.

Thanks very much for the contribution!

@ltjax Thank you!

authors: Gjacquenot ltjax

repo: ltjax/decomp


text:
Why does CDDC output 53 channels?

Hi,
Thanks for sharing. What a nice work!
I have not been able to figure out why does CDDC block yield 53 channels?
Thanks,
LA Tran

Thanks for your interest.

As the figure shows, the number of orange square is 53, which are the considered search grids when building CDDC.


text:
🛑 Školský šport is down

In 661b705, Školský šport (https://skolskysport.sk/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Školský šport is back up in bd97d1a after 5 minutes.

repo: lubosm/minedusk-uptime


text:
🛑 DGC prod is down

In 61c2740, DGC prod (https://digichanges.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: DGC prod is back up in 73c1d3d.

repo: lucalop/monitoring


text:
add an integration test using different connection ID lengths

Depends on #1427.

Codecov Report

Merging #1430 into master will decrease coverage by 0.06%.
The diff coverage is 92.24%.

@@            Coverage Diff             @@
##           master    #1430      +/-   ##
==========================================
- Coverage   89.42%   89.35%   -0.06%
==========================================
  Files         122      121       -1
  Lines        8503     8480      -23
==========================================
- Hits         7603     7577      -26
- Misses        602      605       +3
  Partials      298      298

Impacted Files
Coverage Δ

internal/protocol/protocol.go
100% <ø> (ø)
:arrow_up:

packet_packer.go
91.44% <100%> (ø)
:arrow_up:

client_multiplexer.go
88% <60%> (-12%)
:arrow_down:

server_tls.go
75.97% <60%> (-0.85%)
:arrow_down:

internal/protocol/connection_id.go
86.21% <75%> (-5.1%)
:arrow_down:

client.go
88.13% <84.62%> (+0.26%)
:arrow_up:

internal/wire/header.go
93.8% <93.39%> (-6.2%)
:arrow_down:

server.go
91.1% <95%> (+0.31%)
:arrow_up:

internal/wire/header_parser.go
98.26% <98.26%> (ø)

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 3f9212b...3bdf683. Read the comment docs.

authors: codecov-io marten-seemann

repo: lucas-clemente/quic-go


text:
⚠ Punto de Fuga has degraded performance

In 8083216, Punto de Fuga (http://puntodefugapodcast.com) experienced degraded performance:

HTTP code: 200
Response time: 11521 ms

Resolved: Punto de Fuga performance has improved in 97369d7.

repo: lucasboh/upptime


text:
Specifying an output yields a deprecation warning

Hey,
I'm using the example from the readme with my paths adjusted as needed:
stylelint src/main/scss --custom-formatter stylelint-checkstyle-reporter -o target/stylelint-warnings.xml but that yields
`output` is deprecated. Use `report` or `code` instead.

Other tasks with -o don't yield such a warning, and therefore I'm unsure whether I need to update the arguments from the readme or not 😅
Any clue?
I'm using yarn 4.0.2 with
    "stylelint": "16.1.0",
    "stylelint-checkstyle-reporter": "0.2.0",

so they changed their API with stylelint 16 (which is their right, after all it is a major release). I am passing the second argument to the xml writer which accesses a property that triggers that warning. Good news is, it still works as planned. But I will have to make a new release of the package and basically change the usage. Stay tuned.

Thanks for the swift response! I noticed the deprecation note in our build logs and thought I might file an issue in case it stops working one day :)

So I just released version 1.0.0. The usage is mainly the same, you may want to switch to piping the stylelint output, but outside of one path change it basically stayed the same.

So I just released version 1.0.0. The usage is mainly the same, you may want to switch to piping the stylelint output, but outside of one path change it basically stayed the same.

Thanks!

authors: NotMyFault lucavb

repo: lucavb/stylelint-checkstyle-reporter


text:
NameError: name 'run_test' is not defined

python golem.py run test_project test --environment Local -i
Traceback (most recent call last):
File "golem.py", line 15, in
execute_from_command_line(os.getcwd())
File "/Users/daniel/Development/pythons/golem/lib/python3.6/site-packages/golem/main.py", line 27, in execute_from_command_line
commands.run(args.main_action, test_execution, args)
File "/Users/daniel/Development/pythons/golem/lib/python3.6/site-packages/golem/commands/init.py", line 51, in run
cmd.run(tex, args)
File "/Users/daniel/Development/pythons/golem/lib/python3.6/site-packages/golem/commands/base.py", line 99, in run
test=test_execution.test)
File "/Users/daniel/Development/pythons/golem/lib/python3.6/site-packages/golem/test_runner/start_execution.py", line 167, in run_test_or_suite
run_test(test_execution.root_path, test_execution.project,
NameError: name 'run_test' is not defined

fixed by #61

authors: danielmaddern lucianopuccio

repo: lucianopuccio/golem


text:
How far is this codebase?

Thanks a lot for starting this effort. I had a quick look at the code, and seems like though the GAN network structure is done, there is no training code or reference to data. Can you please explain? Many thanks.

if there is a wip flag, it is not done yet

authors: lucidrains stencilman

repo: lucidrains/gigagan-pytorch


text:
🛑 Synology is down

In fd875b1, Synology (https://synology.kerl.io) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Synology is back up in bd7bea4.

repo: lucienkerl/status


text:
🛑 Nextcloud is down

In 0311b35, Nextcloud (https://nextcloud.kerl.io) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nextcloud is back up in 01fecd5.


text:
Permissions missing

There are multiple permissions missing when you have more plugins, which means that if i e.g add a plugin, the permissions sometimes not even show up in the webeditor

You can add any permissions in the web editor, even when it doesn't show up. The permissions that show are merely suggestions based on what LP has seen since the server started. And it's quite common that not all are there.

Running a command like /help can help cache the permissions in LP and then they will appear in the permission list. Like Brainstone said, LP only suggests permissions that it knows about, it doesn't mean the permission isn't available (best to consult the documentation for your plugins or use Verbose mode).
Not a web editor issue, closing.

authors: BrainStone Redstonico Turbotailz

repo: lucko/LuckPermsWeb