transform bonkers

I might get to debug this myself, but not sure how much time I can spend today with the kids running around the house and the dishes piling up. For the record, just dumping it here:
I'm trying to get a minimal stand alone cps based server socket thing running. The snippet below will compile far enough to see the transformations with -d:cpsDebug, the whole thing is at http://ix.io/2shD
import "cps"

type Cont = ref object of RootObj
  fn*: proc(c: Cont): Cont {.nimcall.}

# CPS client hander

proc doClient(fdc: SocketHandle): Cont {.cps.} =

  while true:
    cps io(fdc, POLLIN)
    let s: string = sockRecv(fdc)
    if s.len > 0:
      echo "recv> ", s
    else:
      return

# CPS server handler

proc doServer(port: int): Cont {.cps.} =
  let fds: SocketHandle = sockBind(port)
  while true:
    cps io(fds, POLLIN)
    let fdc: SocketHandle = sockAccept(fds)
    # Create new client and add to work queue
    evq.work.addLast doClient(fdc)

Some things derail for me now:

The doClient loop gets transformed to this: note the double return

proc loop_14237113(locals_14237210: Cont): Cont =
  ## installing locals for env env
  let fdc: SocketHandle = env_14237084(locals_14237210).fdc
  if true:
    return io(env_14237084(fn: after_14237257, fdc: fdc).Cont, fdc, POLLIN)
    ## post-cps call; time to bail
    ## split at: after
    ## new tail call: after_14237257
    return env_14237084(fn: after_14237257, fdc: fdc).Cont
    ## creating a new proc: after_14237257

Comping the whole thing from the ix.io snippet gives me C compiler errors:

<EMAIL_ADDRESS>In function ‘after__9bgcr1CRd8IRKJAnbwRHzMA_4’:
/home/ico/.cache/nim/t_d/@mt.nim.c:611:78: error: ‘tyObject_envcolonObjectType___baahae0jnkMVren5WI9bsYw’ has no member named ‘fdc’
  611 |  fdc = (*((tyObject_envcolonObjectType___baahae0jnkMVren5WI9bsYw*) (locals))).fdc;
      |                                                                              ^
/home/ico/.cache/nim/t_d/@mt.nim.c:638:14: error: ‘tyObject_envcolonObjectType___HggcRF6w2HNjc8xK1EHtaA’ has no member named ‘s’
  638 |  T8_ = (*T7_).s; (*T7_).s = copyStringRC1(s);
      |              ^
/home/ico/.cache/nim/t_d/@mt.nim.c:638:24: error: ‘tyObject_envcolonObjectType___HggcRF6w2HNjc8xK1EHtaA’ has no member named ‘s’
  638 |  T8_ = (*T7_).s; (*T7_).s = copyStringRC1(s);
      |                        ^
compilation terminated due to -fmax-errors=3.

https://github.com/nim-lang/Nim/issues/15075

As far as I can tell this has not been solved by clyybbers recent forward declaration fix in the compiler. Workaround in 37f0063262cdc173f13a6b4871e410343a4dc775 is still needed to get this running.

https://github.com/nim-lang/Nim/issues/15118

Closing because it predates typed...

Closing because it predates typed...


⚠ Bofrost.de has degraded performance

In 24ea99d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 895 ms

Resolved: Bofrost.de performance has improved in be4415a after 9 minutes.


Implement a Pilatus object

Implement a Pilatus object for use in the flyer similar to the way an Eiger was implemented for the MX flyscan.

difference: no sequence ID for Pilatus

Detector distance units - see this code from lsdc.epics_det.det_setheader()

if (det_id=="PILATUS-6"):
set_det_pv("det_distance",dist)
else:
set_det_pv("det_distance",dist/1000.0)

Short code fragment from LSDC - see what all of these are doing and whether they are necessary for the flyer (presumably the template is, but not sure about others)

if (det_id == "PILATUS-6"):
set_det_pv('det_trigger_mode',0)        #internal for devel
det_set_autoinc_filenum(1)
det_set_file_template("cbf") # looks related to https://cars9.uchicago.edu/software/epics/pilatusDoc.html NDFileTemplate

#131

Done in https://github.com/NSLS-II-NYX/nyxtools/pull/6 - thus, closing!


Incremented version to 1.3.2.

Signed-off-by: Zelin Hao<EMAIL_ADDRESS>Description
Incremented version to 1.3.2.
Category
[Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation]
Why these changes are required?
What is the old behavior before changes and new behavior after changes?
Issues Resolved
part of https://github.com/opensearch-project/opensearch-build/issues/1882
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List

[x] New functionality includes testing
[x] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #976 (0edd918) into 1.3 (f797354) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              1.3     #976   +/-   ##
=======================================
  Coverage   71.98%   71.98%
=======================================
  Files          87       87
  Lines        1906     1906
  Branches      247      242    -5
=======================================
  Hits         1372     1372
  Misses        480      480
  Partials       54       54

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 f797354...0edd918. Read the comment docs.

Note; this change cannot be merged until there is a successful 13.2 distribution build with security for the integration tests to run against.

The https://ci.opensearch.org... link gets access denied.

@zelinh it looks like GitHub is requesting for the changes to be rebased before it can be merged.  Let me know if you'd like a hand with this.


Is there roadmap or development plan for the project?

I am interested in this low code solution and I am willing to add some feature or fix bugs, but I can't find enough docs or feature plans to decide what I can do later. So is there a developer group or something?

is there any update?

is there any update?

it seems like this project is dead :(

There is https://github.com/orgs/blocks/projects/1, but I don't know if it is updated 🤔


Replace setState() with dynamic getState()

Fixes #100
Activation of contract now is signaled through a boolean, which is changed only once after it's activated.
Moved state is not assumed to happen only though the call of UsernameRegistrar.moveRegistry(address), instead it looks up the ownership of the name. This allows the use of BaseRegistrarImplementation.reclaim(uint256,address) (owner of .eth ENS name) to operate migrations, where the ownership can be directly transferred to a new version of UsernameRegistrar, reducing the migration time and cost.
There will be a slight overhead in gas cost for every operation which reads getState(), which won't be noticed by users due it's minimal impact.
The old registrar MUST still use the old 'UsernameRegistrar.moveRegistry(address)' to mark the it as Moved, so it will operate as it was designed.

While I don't fully understand the context yet, I've commented on a potential bug. The name change is a bit confusing (removing a setter setState() to replace with a getter getState(); ideally we have both).

There is no setState because it was replaced by a dynamic lookup. If the contract don't owns the domain anymore, then it considers it moved.
This is also done by the ENS Domains base registrar, see here https://github.com/ensdomains/ethregistrar/blob/master/contracts/BaseRegistrarImplementation.sol#L31


fix: workers 0 is valid from the master

When scaling, if not all nodes will get users, the master will still
send a hatch request for 0 users. Without this fix the worker crashes
and the master ends up losing track of the number of active workers.

Codecov Report

Merging #94 into master will increase coverage by 0.15%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
+ Coverage   87.80%   87.95%   +0.15%
==========================================
  Files          10       10
  Lines         984      980       -4
==========================================
- Hits          864      862       -2
+ Misses         84       83       -1
+ Partials       36       35       -1

Impacted Files
Coverage Δ

runner.go
86.89% <50.00%> (+1.28%)
:arrow_up:

client_gomq.go
70.14% <0.00%> (-2.99%)
:arrow_down:

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 901d7fa...8ed3116. Read the comment docs.

Stop then ends up closing a nil channel. And the code becomes more natural
and "honest" without special casing 0
On Sat, 18 Jul 2020, 15:35 myzhan<EMAIL_ADDRESS>wrote:

@myzhan commented on this pull request.
In runner.go
https://github.com/myzhan/boomer/pull/94#discussion_r456795851:

if r.rateLimitEnabled {

	r.rateLimiter.Start()

}

r.startHatching(workers, hatchRate, r.hatchComplete)

Events.Publish("boomer:hatch", workers, hatchRate)

if r.rateLimitEnabled {

I think we can publish the "boomer:hatch" event and call hatchComplete(),
but skip rateLimiter.Start() and startHatching()?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/myzhan/boomer/pull/94#pullrequestreview-451060972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAASR42E6M7RNXDPRWBWZILR4GXMRANCNFSM4O4HEFAA
.

LGTM, thanks @tcolgate


Hotfix the XCM message queue to stop flagging messages as overweight

re: https://github.com/paritytech/polkadot-sdk/pull/6205

Work around would be to disable the idle processing by setting IdleMaxServiceWeight to None.


support prometheus metrics

Motivation
@binarycrayon  1461 provides a good start.
Since the latest code scheduling of sglang is integrated into schulder.py, so I implemented a version of prometheus metrics based on 1461 and the latest code.

Modifications

Checklist

[x] Format your code according to the Contributor Guide.
[ ] Add unit tests as outlined in the Contributor Guide.
[ ] Update documentation as needed, including docstrings or example tutorials.

@merrymercy @Ying1123 i can help reviewing and testing this PR

Thank you, I have been swamped by game release and didn't have time to finish the work. Happy to test too.
One thing would be useful for our autoscaling is to have indication of whether sglang is lagging out from heavy load of requests. Presumably by a combination of requests waiting to be processed and request process speed
@Ying1123 @merrymercy

if no one has get to this yet, I can add example configuration to your branch. Let me know. @Lzhang-hub @ByronHsu

@Lzhang-hub Can you fix the ci tests?

if no one has get to this yet, I can add example configuration to your branch. Let me know. @Lzhang-hub @ByronHsu

@binarycrayon  Sure, that would be great!

@Lzhang-hub Can you fix the ci tests?

OK

I will finish the review over the weekend

Added grafana dashboard example in a pr to this branch https://github.com/Lzhang-hub/sglang/pull/1

============ Serving Benchmark Result ============
Backend:                                 sglang
Traffic request rate:                    inf
Successful requests:                     1000
Benchmark duration (s):                  89.35
Total input tokens:                      237804
Total generated tokens:                  200655
Total generated tokens (retokenized):    172693
Request throughput (req/s):              11.19
Input token throughput (tok/s):          2661.60
Output token throughput (tok/s):         2245.81
----------------End-to-End Latency----------------
Mean E2E Latency (ms):                   37429.80
Median E2E Latency (ms):                 36665.81
---------------Time to First Token----------------
Mean TTFT (ms):                          15205.34
Median TTFT (ms):                        10752.00
P99 TTFT (ms):                           46274.60
-----Time per Output Token (excl. 1st token)------
Mean TPOT (ms):                          182.10
Median TPOT (ms):                        132.63
P99 TPOT (ms):                           1037.29
---------------Inter-token Latency----------------
Mean ITL (ms):                           112.13
Median ITL (ms):                         87.95
P99 ITL (ms):                            438.84
==================================================

@binarycrayon they look great. let's open another PR for doc and use this for the code change

@binarycrayon they look great. let's open another PR for doc and use this for the code change

then I'll wait for this one merged first.

looks good now. can you fix the failing test?

looks good now. can you fix the failing test? you can also run the test locally https://github.com/sgl-project/sglang/tree/main/test

Sorry, due to network reasons, in order to run the complete test, I need to change the all hf model path in the test, which seems a bit troublesome
I run one of test locally python3 -m unittest test_bench_serving.TestBenchServing.test_offline_throughput_without_radix_cache, it work normal

Bug fix: I change prometheus labels model name to serverd_model_name, because Modelconfig in main branch have changed.

@Lzhang-hub i am also on the cluster which does not have network access. usually i manually replace the model path with the local model path and only run one or two tests not all. the code looks good now. lets wait for the CI

Nice to have this feature! When we do benchmark, we can consider using this monitoring to observe the change curve over a period of time.

Coooool to see it has finally been merged!


All Stats gem for helmet gives insane armor, from 100 to 700 armor by 5% all stats gems, my friends went to 1000 armor

please allow us to disable the gems we want


AMBARI-24549. Move blueprint provisioning state property to host component level.

What changes were proposed in this pull request?
Blueprint provisioning state property was moved from cluster to host component level.
How was this patch tested?
Manual testing.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3831/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3833/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3834/
Test PASSed.

Hi, Can you please add @benyoka  as a reviewer for this change?
Thanks.

LGTM from blueprint perspective. Just to note: things will get more complicated later once we'll be in the multi-everything (multi-service and component) world.
cc @rnettleton @jayush


WIP: support migrate assignee

Fix https://github.com/go-gitea/gitea/issues/22553

This is outdated.


Disable free trial

Merge at 12am PST

📝 Changed routes:

…board/settings/billing
/pricing

(view full size)
(view full size)

Commit 3fcd6e936124b465fbede68cf6c04ce71a1d8ac2 (https://thirdweb-n6mknfecu.thirdweb-preview.com).


Fix readme inconsistencies

Fixes https://github.com/openshift/cluster-kube-descheduler-operator/issues/74

/lgtm
Feel free to push more changes if you find any. I will re-lgtm if needed.

Thanks @damemi! There is one more place that needs updated. I couldn't comment inline on it, but line 52 says "the operator expects config CR ...". That should be cluster.

@bergerhoffer thanks! Fixed

/lgtm


Error on SQL Execute

When using Execute SQL statement or Execute SQL statement in a new tab, I expect that the query executes for lines only within the scope of the delimiter. But I see that even if there is some incomplete statement elsewhere within the .sql, Dbeaver shows a error

When I do a Execute SQL statement placing my cursor in Line 7 -

If I just highlight line 7 to execute, it works fine.

DBeaver doesn't support PL/SQL in form of simple SQL.
You can use anonymous SQL blocks to execute PL/SQL:
[declare
...]
begin
  ...
end;

My purpose wasn't with writing PL/SQL. I just had the keyword there which I forgot to comment. This is just a example. I had this part of a bigger sql script and execute was always failing.
I wanted to point that just executing sql on line 7 should have worked as it was within the scope of the delimiter  ; ?

Ah, I see. Looks like you executing it as a script.
In order to execute single statement use CTRL+Enter (or Execute SQL Statement button in toolbar)

No Serge. I am not executing as script.
As mentioned in my post I am using Execute SQL statement or Execute SQL statement in a new tab. I am using oracle. I think it somehow doesn't like the LOOP keyword there. Not sure if this will occur for other keywords.

Then I just can't reproduce that.
Please post you SQL here (sometimes there is something special in particular code piece, e.g. hidden special characters) and I will try it on my machine.

I think there is some conflict with my settings. I am able to reproduce always in 1 machine but not in the other. I will try to figure out.


What is the relationship between IAO and EDAM?

The entire contents of EDAM (https://www.ebi.ac.uk/ols/ontologies/edam http://edamontology.org/page) seems in scope for IAO. However, key differences:

IAO is not actively developed and EDAM is (I assume)
EDAM is widely used for the tasks it was created for
EDAM makes more bioinformatician-friendly labels, e.g. format as opposed to data format specification -- IAO doesn't even have synonyms here
content: the actual relevant sections are fleshed out in EDAM, i.e. IAO doesn't actually have any formats under it, so EDAM is more pragmatic here. Same for operations.

Some of us are sometimes asked for an ontology recommendation for things like bioinformatics topics, data types, formats, operations, etc. EDAM is the best choice here, there are no alternatives.
This may be unsatisfying for some as EDAM is not integrated into OBO in any way. For many tasks - e.g. categorizing topics - this is fine as most use cases just require a terminology and no reasoning.
But I can see cases where it would be useful to have EDAM better integrated with OBO, e.g. for representing and reasoning over bioinformatics data types and operations.
One possibility is that EDAM top level classes could be made subclasses of relevant IAO classes (via a bridge file) and IAO cedes the content under here.
This may require some compromises on IAO's part, e.g. in ontological naming.
Some things in EDAM could be tidied up too. I don't really understand their hierarchy and whether their subClassOf actually means owl subClassOf. E.g. EDAM has JSON as a subClassOf YAML (I assume because every JSON document is expressible as a YAML document), similarly they have manchester syntax as a subClassof RDF which doesn't make much sense. I

After writing this I forgot that OBO includes SWO http://www.obofoundry.org/ontology/swo.html
It looks like SWO may actually be doing what I suggest! It seems to be a mix of IAO and EDAM....
However, SWO is trailing behind EDAM and has not been updated for a while. I requested a metadata update: https://github.com/OBOFoundry/OBOFoundry.github.io/issues/1040
I also just noticed that CC-BY-SA. That contraindicates any reuse within OBO :-(
One possibility here is to work with Wikidata, it is CC-0, and has many of the relevant concepts organized ontologically, e.g. multiple sequence alignment: https://www.wikidata.org/wiki/Q1377767
@rvita can we target

FYI - SWO does make use of IAO and EDAM. Jon Ison and I spend some time a few years back (in 2012, see https://softwareontology.wordpress.com/2012/07/25/swo-edam-merge-overview/) mapping SWO to EDAM. However, that work is now out of date and the latest update of SWO that I'm about to start working on may change the nature of that link.
However, there still will be associations among SWO - IAO - EDAM, so you may wish to take a look once I've got the next version of SWO out.
Thanks :) Allyson

Just to update this very old ticket, in the end I had to remove the majority of EDAM from SWO. Although Jon had agreed to import SWO in certain areas, as I agreed to import EDAM, the updates to EDAM never happened and as a consequence SWO was unbalanced. So, there are maybe 8-10 classes I have retained so that users don't wonder why they suddenly disappeared, but all hierarchies have been removed.
EDAM also chose to make their own format classes rather than import and extend SWO, so unfortunately I can't forsee any further collaboration unless things change.
There has been further discussion about the licencing (https://github.com/edamontology/edamontology/issues/724) where it seems it shouldn't be a problem, but I am not a lawyer ;-)
Apropos of nothing, I'm nearly finished updating the SWO build to use the ODK, hopefully making it easier for future contributors. It is a lovely ontology that just needs a bit of updating. I particularly think the licencing section is worthwhile, and would like to see someone who is familiar with https://spdx.dev/ investigate alignment with SWO.


Transferred pokemon are not deleted

Here is how this bug happen:

Use pogo-optimizer to check IVs as usual.
Disconnect from proxy
Re-login pokemon go and transfer some pokemons within normal network
Back to pogo-optimizer, previously transferred pokemon still on the page

Is there any command to clear the cache or something?

You just need to update the app.

On Jul 28, 2016, at 10:53 PM, qwer4790<EMAIL_ADDRESS>wrote:
Here is how this bug happen:

Use pogo-optimizer to check IVs as usual.
Disconnect from proxy
Re-login pokemon go and transfer some pokemons within normal network
Back to pogo-optimizer, previously transferred pokemon still on the page

Is there any command to clear the cache or something?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

As clarification, when running the latest version of pogo-optimizer if something like this happens at any point you can click the logout button and relaunch the app from your phone to get a fresh view, but this should never happen. The page should simply update with the current list and dump all of the old stuff. So in short, stop the proxy (ctrl+c) and run
git fetch origin master
git pull
npm install
npm start

This should check your version against the latest, update what needs updating, install the new dependencies, then start the proxy back up. You'll likely need to download the ca.crt or ca.pem again after doing this.
Note: This is assuming you are running this from a bash command line on a *nix based system. Docker is completely alien to me, maybe someone else can chime in on the update process for that if you need it.

Yes I use docker, so it might have some issue..I will check it again after update.

I've noticed that when a transfer is missed it's often updated after the next transfer.


ADSL E2E example needs touching up ahead of admiral 1.0

Example: switch out references to admiral.test for pharmaversesdtm, switch out references to deprecated derive_vars_merged_dtm.

@manciniedoardo we're in process of retiring this site, check out https://github.com/pharmaverse/examples from now instead and https://pharmaverse.github.io/examples/adam/ADSL.html. it uses quarto so easier to maintain and bring in new contributors

thanks @rossfarrugia


feat: edit hyperlinks

follows https://github.com/contentful/field-editors/pull/768
Allows editing hyperlinks, including changing a hyperlink to a different type of hyperlink (e.g. URL -> Entity and vice versa).
Plays well with tables

@cezarsmpio , I'm reassigning you since I've made quite a few small (but important) changes since you last saw this one and they have been unfortunately squashed into the last commit - hence I've annotated the changes for visibility.


ENH: Added error logging to DICOMDIR file parsing

Problem: When attempted to load DICOM files Slicer simply displayed that 0 patient was imported, without any hint of what could have been the problem.
Analysis: The issue was that study instance UID was missing from the DICOMDIR file but there was no error logged when this error occurred.
Solution: Added logging of DICOMDIR file parsing failures. Also added a bool return value to addDicomdir to allow the caller detect that error(s) occurred.

:+1 Thanks for the patches. Looks good to me.
Cc: @pieper

Yes, looks good to me - thanks for the contribution!


TCP Fast Open support

https://en.wikipedia.org/wiki/TCP_Fast_Open
Would be great to get this into OkHttp since it could improve performance in high latency networks.

Know any webservers that support it? Know if Android supports it?

Might be better to get this feature in AOSP first.
https://code.google.com/p/android/issues/detail?id=179517

AOSP does not support it today as far as I know, but chrome on Android does ostensibly because they have their own TCP stack. iOS 9 supports it via connectx.
We plan to support this in our internal webservers soon pending a kernel upgrade to 3.17.

Closing for now. If AOSP ever gets fast open, we’ll reopen.

@swankjesse do you know if this got AOSP? it greatly improve user experience.
Nougat should be on kernel 4, so definitely supports at kernel level. Do you think there's anything to be done internally for okhttp?

Unsure what we’ll need to do.

If Netty hasn't done this, not sure OkHttp should be leading this charge.  particuarly given moves to Https everywhere and Tls 1.3 and Quic in future.


!Remove old way of using knuu, only allow knuu as object

This will be closed once the #386 is closed

Done by #548


does not run

I managed to successfuly build mango (using move to 2.11 and move to spark2 scripts) but when I run bin/mango-submit.sh and other shell scripts, nothing happens (no output) even though I have SPARK_HOME in the Path. My OS is Linux Mint 18.2 (Ubuntu based)

Hi @antonkulaga ! What version is your SPARK_HOME pointing to? Are you just trying to run this locally?

Spark 2.20. I run move to 2.11 and move to spark.2 scripts before running Mango. I run spark locally. I wish I got at least some output from scripts (even with errors), by now I have zero

OOC, does this work with Spark 2.1.x? Spark 2.2.0 moves the Parquet dependency version, which causes classpath conflicts.  We're working on a fix to this, see https://github.com/bigdatagenomics/adam/pull/1518.

Have not checked for Spark 2.1.x but latest master of ADAM works pretty well with Spark 2.2.0 for me

Oh, interesting! It hadn't worked for me, but that was on one of the RCs. I'll retest today.

@antonkulaga  locally you should be able to run the following:
./scripts/move_to_scala_2.11.sh
./scripts/move_to_spark_2.sh
mvn clean package -DskipTests
./example-files/run-example.sh
And see the results at localhost:8080
This should be able to run with Spark 2.11 and 2.2.0 locally. If you are not getting any printout in the start script, can you change line 22 of the bin/mango-submit script from set -e to set -ex , then rerun and post the output here?


chore(bzlmod): set dev_dependency true for googletest

It's understandable that you think this is the right thing to do, but I believe it is incorrect. Abseil provides libraries for writing tests that depend on GoogleTest. dev_dependency means "If true, this dependency will be ignored if the current module is not the root module or --ignore_dev_dependency is enabled." If you want to use one of the test libraries, you need a dependency on GoogleTest. I actually don't think dev_dependency solves any real problem and should be removed from Bazel.


[Bug] Update esbuild version when compile with ./build.sh

What changes were proposed in this pull request
Issue Number: close #xxx

Brief change log
Transforming async generator functions requires es2018
Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

@wangsizhu0504 PTAL, thanks


feat(keystone,apigateway): return a readable message when the user is locked or disabled

这个 PR 实现什么功能/修复什么问题:

[x] 功能、bugfix描述
[x] 冒烟测试

是否需要 backport 到之前的 release 分支:

release/3.7

/area keystone apigateway
/cc @swordqiu

Codecov Report

Merging #9948 (c5223f9) into master (1ab8ce5) will increase coverage by 2.08%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #9948       +/-   ##
==========================================
+ Coverage    5.97%   8.05%    +2.08%
==========================================
  Files         849     107      -742
  Lines      132215   12991   -119224
==========================================
- Hits         7898    1047     -6851
+ Misses     123616   11824   -111792
+ Partials      701     120      -581

Flag
Coverage Δ

aFlag
8.05% <0.00%> (+2.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/hostman/hostinfo/hostinfo.go

pkg/mcclient/modules/monitor/datasource.go

pkg/multicloud/aws/eip.go

pkg/compute/models/routetables.go

pkg/baremetal/tasks/unmaintenance.go

pkg/multicloud/ucloud/disk.go

pkg/compute/guestdrivers/qcloud.go

pkg/cloudnet/utils/key.go

pkg/mcclient/options/k8s/tiller.go

... and 729 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #9948 (c5223f9) into master (1ab8ce5) will increase coverage by 2.08%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #9948       +/-   ##
==========================================
+ Coverage    5.97%   8.05%    +2.08%
==========================================
  Files         849     107      -742
  Lines      132215   12991   -119224
==========================================
- Hits         7898    1047     -6851
+ Misses     123616   11824   -111792
+ Partials      701     120      -581

Flag
Coverage Δ

aFlag
8.05% <0.00%> (+2.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/hostman/hostinfo/hostinfo.go

pkg/mcclient/modules/monitor/datasource.go

pkg/multicloud/aws/eip.go

pkg/compute/models/routetables.go

pkg/baremetal/tasks/unmaintenance.go

pkg/multicloud/ucloud/disk.go

pkg/compute/guestdrivers/qcloud.go

pkg/cloudnet/utils/key.go

pkg/mcclient/options/k8s/tiller.go

... and 729 more

Continue to review full report at Codecov.

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

/lgtm
/approve

/lgtm
/approve


Flask is not an optional dependency

0.4.28 moved Flask to an optional [server] dependency, but importing from moto (e.g. from moto import mock_s3) leads to moto.core.utils which tries to import from Flask:
.tox/py27/lib/python2.7/site-packages/moto/__init__.py:8: in <module>
   from .apigateway import mock_apigateway  # flake8: noqa
.tox/py27/lib/python2.7/site-packages/moto/apigateway/__init__.py:2: in <module>
   from .models import apigateway_backends
.tox/py27/lib/python2.7/site-packages/moto/apigateway/models.py:7: in <module>
   from moto.core import BaseBackend
.tox/py27/lib/python2.7/site-packages/moto/core/__init__.py:2: in <module>
   from .models import BaseBackend  # flake8: noqa
.tox/py27/lib/python2.7/site-packages/moto/core/models.py:8: in <module>
   from .responses import metadata_response
.tox/py27/lib/python2.7/site-packages/moto/core/responses.py:19: in <module>
   from moto.core.utils import camelcase_to_underscores, method_names_from_class
.tox/py27/lib/python2.7/site-packages/moto/core/utils.py:9: in <module>
   from flask import request
E   ImportError: No module named flask

Thanks, this is fixed in the most recent release.


AS3 does not assign APM connectivity profile

Environment

Application Services Version: 3.28.0
BIG-IP Version: 16.1.0

Summary
When using AS3 to configure a VS with a connectivity profile (APM), the declaration is accepted and returns a successful response, but the configured VS does not have the connectivity profile applied. Performing the same procedure with TMSH
Steps To Reproduce

Create a connectivity profile to use.
tmsh create apm profile connectivity custom_connect
tmsh save sys config

Submit the following declaration:

{
  "class": "ADC",
  "schemaVersion": "3.28.0",
  "Example_APM_profiles": {
    "class": "Tenant",
    "Application": {
      "class": "Application",
      "APMprofile_vs": {
        "class": "Service_UDP",
        "virtualPort": 443,
        "virtualAddresses": [
          "<IP_ADDRESS>"
        ],
        "profileConnectivity": {
          "bigip": "/Common/custom_connect"
        }
      }
    }
  }
}

Observe that the created configuration object does not have the connectivity profile assigned:
tmsh list ltm virtual /Example_APM_profiles/Application/APMprofile_vs

ltm virtual /Example_APM_profiles/Application/APMprofile_vs {
    creation-time 2022-01-27:15:14:03
    description Application
    destination /Example_APM_profiles/<IP_ADDRESS>:https
    ip-protocol udp
    last-modified-time 2022-01-27:15:14:03
    mask <IP_ADDRESS>
    partition Example_APM_profiles
    persist {
        source_addr {
            default yes
        }
    }
    profiles {
        udp { }
    }
    serverssl-use-sni disabled
    source <IP_ADDRESS>/0
    source-address-translation {
        type automap
    }
    translate-address enabled
    translate-port enabled
    vs-index 20
}

Expected Behavior
The Virtual Server should be configured with the correct profile applied. TMSH can already correctly configure the VS with the same data.
create ltm virtual /Example_APM_profiles/Application/APMprofile_vs_tmsh destination <IP_ADDRESS>:9999 ip-protocol udp mask <IP_ADDRESS> profiles add { custom_connect {context clientside } udp {}  }
ltm virtual APMprofile_vs_tmsh {
    creation-time 2022-01-27:15:19:45
    destination /Example_APM_profiles/<IP_ADDRESS>:distinct
    ip-protocol udp
    last-modified-time 2022-01-27:15:19:45
    mask <IP_ADDRESS>
    partition Example_APM_profiles
    profiles {
        /Common/custom_connect {
            context clientside
        }
        /Common/ppp { }
        /Common/udp { }
    }
    serverssl-use-sni disabled
    source <IP_ADDRESS>/0
    translate-address enabled
    translate-port enabled
    vs-index 23
}

Thank you for your feedback. I have confirmed this issue with the latest release, and I have added this to our internal product backlog as AUTOTOOL-2985.

It looks like connectivity profiles are not supported on UDP services. Are you able to use Service_HTTP or Service_HTTPS for your use case instead?

It looks like connectivity profiles are not supported on UDP services. Are you able to use Service_HTTP or Service_HTTPS for your use case instead?

Both Service_HTTP and Service_HTTPS respond with a 422 saying that the "profileAccess" property is required.
"profileAccess": {
    "bigip": "/Common/accessProfile"
}

Adding such a property lets the POST succeed, as well as create a properly configured Virtual Server.
ltm virtual APMprofile_vs {
    creation-time 2022-01-27:17:26:37
    description Application
    destination /Example_APM_profiles/<IP_ADDRESS>:https
    ip-protocol tcp
    last-modified-time 2022-01-27:17:26:37
    mask <IP_ADDRESS>
    partition Example_APM_profiles
    persist {
        /Common/cookie {
            default yes
        }
    }
    profiles {
        /Common/accessProfile { }
        /Common/custom_connect {
            context clientside
        }
        /Common/f5-tcp-progressive { }
        /Common/http { }
        /Common/ppp { }
        /Common/rba { }
        /Common/websso { }
    }
    serverssl-use-sni disabled
    source <IP_ADDRESS>/0
    source-address-translation {
        type automap
    }
    translate-address enabled
    translate-port enabled
    vs-index 26
}

It's not unreasonable that the original POST declaration wouldn't work if UDP services don't support connectivity profiles, but that does not explain why the TMSH command works instead.
tmsh create ltm virtual /Example_APM_profiles/Application/APMprofile_vs_tmsh destination <IP_ADDRESS>:9999 ip-protocol udp mask <IP_ADDRESS> profiles add { custom_connect {context clientside } udp {} }

This issue is in our product backlog as AUTOTOOL-2985.

Any info when this will be fixed for UDP services? We require this to get VPN services with DTLS running.


convert from using airtable APIs to mockResponse in kitchen sink

many examples are broken. One I ran into is this - https://studio.ensembleui.com/app/e24402cb-75e2-404c-866c-29e6c3dd7992/screen/fb8d28a2-834e-40a9-8419-155272fb0191
we should not be using the airtable for mocking data anymore

All airtable API calls replaced with mockResponse. Screens changed are as follows:

APIs: POST
APIs: code
Action: invokeAPI
ChartJs
Code: JSON path
Dropdown
Form
Map


Update README.md

small typo in the initializers/swagger_docs.rb file : the line for the parent_controller hash was missing a , before the the value :open_mouth:

Thanks Ian!


Prevent accidential feature type change

This is related to the Lake Michigan incident (#3681), where the user has changed Lake Michigan into a highway. In order to prevent such accidential changes of the feature type, we should do:

Make more clear what the user is doing. Changing the feature type of an existing object is only indicated by the hover text before clicking the inspector header box. After that click it doen't seem to be clear for new users what the feature type list means. Instead of just asking for a feature type iD should better show something like "Convert Lake (Lake Michigan)" and the Search box can show something like "Search replacement feature type".

Don't offer all feature types for existing objects from the database, but offer similar types only. It seems to be quite rare that an existing feature needs to be totally changed in type. ID might offer a button to indicate a total tagging error, which can enable all feature types. The availability of this button can be dependent on feature size or age. Advanced user are able to override the protection using the raw tag editor.

Make more clear what the user is doing. Changing the feature type of an existing object is only indicated by the hover text before clicking the inspector header box. After that click it doen't seem to be clear for new users what the feature type list means.

I don't know, I kind of disagree that it is confusing.  The hover text says "Change feature", and then they're presented with the familiar list of presets to pick from (We train them how to choose a preset several times in the walkthrough).  The user can back out of this selection any number of ways (pressing esc, clicking the forward button, or clicking literally anywhere else on the screen except for choosing another preset).  Even if they do change the preset.. The rendering of the feature will change, the preset text will change.  They can change it back via the same method, or undo.

Don't offer all feature types for existing objects from the database, but offer similar types only. It seems to be quite rare that an existing feature needs to be totally changed in type.

This is a harder problem than it seems, and building a bunch of knowledge about the idiosyncrasies of OSM tagging into iD itself would make things more confusing and we'd still never get things completely right.  For example, adding logic like "you should never change a lake to a highway" means that we have to code how lakes are tagged (coastline or lake?) how highways are tagged, what all this stuff means, and keep up with tag trends that change frequently.  The tags themselves are invented on the fly by non-experts.  And that's just one rule.  People would want thousands of these mini validations built into iD.  Instead, our flexible preset system covers 99% of cases pretty well.
An easy fix here would be to just say that the thing the user is trying to edit is too big and that they should use another editor (or some advanced iD mode) to edit it.

I don't know, I kind of disagree that it is confusing.
Clicking more or less accitentially on the header within the first few steps in iD I think it could be confusing.
Later it is likely ok.

This is a harder problem than it seems
It seems to be worth to think about it, because it can also simplify normal feature type retagging, and making the user aware of nearby feature type options can increase tagging quality.
Offering a button to show the full list in a second step would allow us to be somewhat incomplete.
My idea to do it:
Let the OSM community define feature groups in the Wiki. Source this into iD on build.
Show all feature types of all groups containing the current type, provided they are matching preset rules.

An easy fix here would be to just say that the thing the user is trying to edit is too big and that they should use another editor (or some advanced iD mode) to edit it.

Directing users to use editors which they are unfamilar with doesn't seem to be a good idea for the safety of the big features.
Requiring a special mode to edit large features seems to be acceptable, but having this mode on should not generate significant additional risk for other edits.
Doesn't a large feature edit warning stating size, name, old type, and new type provide sufficient protection?


GUI Bug

The issue
The buttons on the right side of the recipe table GUI are broken.

If you create a new world with Cooking For Blockhead (https://shor.cz/cfbh) instead of Delightfull Cuisine For Woodhead does this still happen?

Yes, it does. Sorry for the reply.
@xemnes

i think i know what the issue is, i had modified the texture of the gui to look a little better than vanilla. it seems i didnt include it properly in the build, i will fix this.

ok so i just looked into it, i dont see any problems with the build i uploaded to curseforge. also looking at your screenshot, i see that its using the old texture. the buttons should not look like that

are you using a resource pack that may change textures for cookingforblockheads? if so can you go into said resource pack and go to "assets\cookingforblockheads\textures\gui" and delete gui.png

Oh, yes, I use the Modernity resource pack.

ah yes, just checked the modernity resource pack and it does indeed change the gui texture, you will have to delete it from the resource pack in order for it to look correct.

Okay, thanks! That was simple!


Bulk edit improvements

Change the process to actually manually edit the entries, instead of loading a full instance, making a change and then saving. (basically use sql queries where possible, and rewrite that for yml/json)
Add: mass change a permission from x --> z
Add: mass remove a permission

This should make the process more useful, and more efficient for a large dataset.

Added in f6f9840eb7d74ff3402571e2a844d2e79a24c7d9


Use std::filesystem instead of manhandling paths as strings

We are noticing quite a few issues emerging from use of strings to manage paths. Here are few instances,

https://github.com/chipsalliance/Surelog/blob/094f1ad629c34d38c0c578f1a8c913eb8d7c1d82/src/SourceCompile/CompileSourceFile.cpp#L94
https://github.com/chipsalliance/Surelog/blob/094f1ad629c34d38c0c578f1a8c913eb8d7c1d82/src/SourceCompile/CompileSourceFile.cpp#L126

Also, Windows limit of 260 characters applies when using strings are arguments to classes like std::ofstream but it doesn't when using std::fs::path. We are running into the path limit on Windows with certain tests (most recently with BlackParrot) and using std::fs seems to resolve it. Unfortunately, the documented solution to fix the path limit doesn't seem to work for Surelog.

+1 for using fs::path to assemble file names.

The BlackParrot test path is not a Surelog specific issue, it is a BlackParrot test structure issue.

Yup, please update the minimum gcc, send a PR.

For sake of documentation - Found another reason justifying this change. Surelog calling external processes using system command and using backslashes on Msys2 causes failure. Specific test case failing as of today is DiffSimpleIncludeAndMacros where the input to the diff command isn't being found because the input output directory on the command-line was using backslashes.

Fixed by #2253


cannot use non loopback addresses

Im trying to install this on an AI server.
our ollama instance is running on a different machine, so problem 1. but i also cannot connect to this docker because the server is headless, i can't connect remotely to the loopback.
how do i make this listen on another address?
ports:
- "<IP_ADDRESS>:7878:7878"
in the docker yaml Seems to have no effect

Set the OLLAMA_HOST env variable


Remove angle-brackets from urls coming out of slack

Which version of matterbridge are you using?
v1.3.1
Would it be a sensible improvement to stop adding (or rather, to start stripping, if that's the case) the angle brackets from slack-slack bridge messages? It seems to prevent slack for hyperlinking and unfurling.

I suspect it's happening here, but please let me know if it might exist elsewhere:
https://github.com/42wim/matterbridge/blob/master/bridge/slack/slack.go#L413

Ah, seems that sometimes urls down get a "bar" |:
https://api.slack.com/docs/message-formatting#linking_to_urls
I think this regex would work for that, so long as it's after mentions get replaced, which already happens:
<(.*?)(\|.*?)?> (instead of <(.*?)\|.*?>)

Ok ready for review :)

Thanks!
LGTM


Roll Flutter Engine from 42ccd12a14c2 to e94b7c60311f (1 revision)

https://github.com/flutter/engine/compare/42ccd12a14c2...e94b7c60311f
2023-06-13<EMAIL_ADDRESS>Create v2 configuration for emulator tests. (flutter/engine#42652)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Prevent the browser from caching the chart data for more than one day

Fixes #1185
Changes Summary

Create a suffix in the form of YYYY-MM-DD
Append the suffix to the chart.json url in the ajax request

This pull request is ready for review.

This would work, but would also be fixed in #1133

Sorry for the duplicate. About this solution I think the main difference is that it will keep working even when/if using a different server than nginx, i.e. the code marks the intent, not the server configuration. But it is up to you to decide what you prefer.


Implement Benchmarking CI

🚀 Feature
It would be pretty interesting to implement benchmarking CI and CD for knowing how code changes affect to the performance of metacall. For example, since more safety was added to NodeJS loader, after the version 0.3.x, the performance has been damaged a lot. We should be able to control this in a better way.
There's a tool that already integrates that and it is compatible with CTest and Google Tests: https://github.com/benchmark-action/github-action-benchmark
Here's the C++ Google Test version: https://github.com/benchmark-action/github-action-benchmark/tree/master/examples/cpp
It produces nice outputs and notifies if performance is degraded: https://benchmark-action.github.io/github-action-benchmark/dev/bench/

Check this PR

https://github.com/metacall/core/pull/457

#457 Check this

This is implemented already.


Year / Series

For large collections going over several years it would be nice to have an option automatically put them into a year folder when imported corresponding with the year filter in the client. So some sort of an automatic drop destination

Already reported (a few times) WIP


About the bindingdb training set

Why did you only use a portion of the bindingdb dataset for training? As far as I know, this dataset is large and seems to have approximately 2 million molecules related to the target. Looking forward to your reply

Hello, I would like to know how you can obtain the label for the bindingdb dataset，

What are the values of threshold and y？thanks

The binary data sets were taken from MolTrans for comparison, please consult their report for more detailed information on data set generation.
The affinity data was taken from the Therapeutics Data Commons DTI DG benchmark


i want to draw my wav file linear method

hi
i want to draw my wav file In the form of bottom picture but drawn in the form of top picture

What changes should I make?

What changes should I make?

To what? Can you share that code?

no feedback, closing.


.guix-profile not available inside container

The .guix-profile directory is not available inside the container, when it should be. We need it to be able to access locales and tzdata and (possibly) other shared data.
Apparently guix environment and guix package now generate different profiles: The latter contains provenance information, the first one does not (hence, different input → different output). The best solution would be to start a container from the profile itself and not via guix environment.

https://issues.guix.gnu.org/issue/46291 adresses this issue (in combination with local changes).


Maintenance mode - smooth failover

I don't know if this is a linstor limitation or misunderstanding by myself ( https://github.com/LINBIT/linstor-server/issues/261), but I can't find a procedure to smoothly take a node off my cluster and do some maintenance.
I use drbd-reactor with linstor NFS gateway and actually if I do:
drbd-reactorctl disable

the DRBD resources on that node become Secondary (which is what I want) but NFS clients actually connected get stucked and NFS shares are not available for them. Whereas the failover seems to  have worked fine, only new NFS mount succeed (or remount / reboot previously connected NFS clients)

@chrboe still anything you can contribute to that? got that fixed?


Resource Scheduler

Hello,
It would be great to add a new view, called Resource.
This should display events for each calendar on each row.
CF https://fullcalendar.io/scheduler

Not planned yet. Thank you for suggestion.

Okay, anyway if you do it you'll be the first open source and free project to have it :)

Is there any updates on this?

Related requests (#200 #237 #299 #430 #438).
The latest update was 24th October 2019:

@mariusa
There was a constant demand for resource views.
It's also in our 2.0 plan.
Thanks for your attention.

Apologies for not searching first.

@tkav I appreciate for your attention. We are in progress. TOAST UI Calendar v2 is totally being rewritten. So it'll take a little long time than we think.

Any update on this?

Any update on the requested features

No. We don't have any progress on this.

Any update on this? @adhrinae


[SPARK-47751][PYTHON][CONNECT] Make pyspark.worker_utils compatible with pyspark-connect

What changes were proposed in this pull request?
This PR proposes to make pyspark.worker_utils compatible with pyspark-connect.
Why are the changes needed?
In order for pyspark-connect to work without classic PySpark packages and dependencies.
Spark Connect does not support Broadcast and Accumulator.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Yes, at https://github.com/apache/spark/pull/45870. Once CI is setup there, it will be tested there properly.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master.


Update DLP Duck Token (DUCK) Logo

Update the DLP Duck Token (DUCK) Logo
Contract Address: 0xC0bA369c8Db6eB3924965e5c4FD0b4C1B91e305F
Website: https://duckdao.io
Etherscan (DUCK): https://etherscan.io/token/0xc0ba369c8db6eb3924965e5c4fd0b4c1b91e305f
Twitter: https://twitter.com/dao_duck/
Medium: https://duckdao.medium.com/
Telegram (DUCK): https://t.me/duckliquiditypool
CoinGecko: https://www.coingecko.com/en/coins/dlp-duck-token

@xrn Let me know if there's any change required or requires additional information.

@xrn Let me know if there's any change required or requires additional information.


Update tensorrt llm to v0.9.0

Use pip install tensorrt_llm instead of building tensorrt_llm from scratch in Dockerfile

I tested the docker file and ran a freshly compiled large-v3 engine. This works nicely! :+1:
Thanks for the fix wrt to TensorRT 0.9.0, I actually needed this. It fixes the issue that TensorRT 0.7.1 excessively allocated VRAM (~3x the model). This is not the case anymore with 0.9.0.
One thing that I saw, but this is probably worth a separate PR: The VRAM usage is increasing by ~350mb for every new client connection. I hope this will not be the case if I use the single-model option (#223). I can try that tomorrow.

One thing that I saw, but this is probably worth a separate PR: The VRAM usage is increasing by ~350mb for every new client connection. I hope this will not be the case if I use the single-model option (#223). I can try that tomorrow.

I could verify that with the single model option, there is no memory increase. So this seems to be an issue with model cleanup and re-instantiation only.

@peldszus thanks for confirming here, will look into that when we instantiate new model for every new user.


Duplicating "Less Mess" crashes Rack v2

Running Rack v2.0.0 pro standalone or free  on Windows 11
I place one "Less Mess" module in an empty patch.
Duplicate it using ctrl-D or right-click->duplicate
Rack crashes
and this at the end of the logfile:
[37.959 fatal adapters/standalone.cpp:60 fatalSignalHandler] Fatal signal 11 SIG. Stack trace:
31:  0x0
30:  0x0
29: _C_specific_handler 0x7ffc1094acc0
28: _chkstk 0x7ffc114c81d0
27: RtlRestoreContext 0x7ffc11455160
26: KiUserExceptionDispatcher 0x7ffc114c72e0
25: ZN8LessMess12dataFromJsonEP6json_t 0x7ffbd3563700
24: ZN4rack3app12ModuleWidget11cloneActionEb 0x7ffba6912760
23: ZN4rack3app12ModuleWidget10onHoverKeyERKNS_6widget6Widget13HoverKeyEventE 0x7ffba6912d90
22: ZN4rack6widget6Widget10onHoverKeyERKNS1_13HoverKeyEventE 0x7ffba6dcc5c0
21: ZN4rack3app10RackWidget10onHoverKeyERKNS_6widget6Widget13HoverKeyEventE 0x7ffba691ff20
20: ZN4rack6widget10ZoomWidget10onHoverKeyERKNS0_6Widget13HoverKeyEventE 0x7ffba6dcac50
19: ZN4rack6widget6Widget10onHoverKeyERKNS1_13HoverKeyEventE 0x7ffba6dcc5c0
18: ZN4rack2ui12ScrollWidget10onHoverKeyERKNS_6widget6Widget13HoverKeyEventE 0x7ffba69440e0
17: ZN4rack3app5Scene10onHoverKeyERKNS_6widget6Widget13HoverKeyEventE 0x7ffba6929510
16: ZN4rack6widget10EventState9handleKeyENS_4math3VecEiiii 0x7ffba69488a0
15: ZN4rack6window3Svg4loadERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE 0x7ffba694ce90
14: glfwPlatformUnlockMutex 0x7ff

Dear Jens,
thank you for the bug report. I found a fix.
Are you compiling the plugin from sources or using the build version as a regular user?
In the first case I just uploaded a version that should fix your problem. In the second case I need the Rack team to compile a build and this will take a little more time. Let me know

I use it from the library. I will wait. Thank you for the modules.
best regards
Jens Peter
On Wed, 1 Dec 2021 at 18:07, Leonardo Gabrielli @.***>
wrote:

Dear Jens,
thank you for the bug report. I found a fix.
Are you compiling the plugin from sources or using the build version as a
regular user?
In the first case I just uploaded a version that should fix your problem.
In the second case I need the Rack team to compile a build and this will
take a little more time. Let me know
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LOGUNIVPM/LOGinstruments/issues/16#issuecomment-983847488,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGU7SNNYLART657T44EPWJ3UOZI5LANCNFSM5JC3TL4A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Try to update your plugin from the library. It should be fixed now.


data_root error while defining ExampleProvider

I have a gninatypes file. The path of this file is gninatype_dir
I have a sample.types file containing a single line. The contents of the file are as follows.
12.531857  1.317891e+01  3.390248e+01 5bqh_protein_nowat.gninatypes
My question is I get an error in batch =e_test_1.next_batch() part of a code block like below
ValueError: Could not read /content/5bqh_protein_nowat.gninatypes/5bqh_protein_nowat.gninatypes
The problem is that the path of the file changes from /content/5bqh_protein_nowat.gninatypes to /content/5bqh_protein_nowat.gninatypes/5bqh_protein_nowat.gninatypes and cannot find it.
Is there a mistake I made while creating the ExampleProvider?
gninatype_dir = '/content/5bqh_protein_nowat.gninatypes'
sample_types = '/content/sample.types'

e_test_1 = molgrid.ExampleProvider(data_root= gninatype_dir, stratify_min = 0, stratify_max = 5, stratify_step=1, labelpos=0, shuffle=False,stratify_receptor=False, balanced = False)
e_test_1.populate(sample_types)

for i in range(1):
        model_categorized.eval()
        batch =e_test_1.next_batch()
        ...

'/content/5bqh_protein_nowat.gninatypes' is not a directory.  You should have
gninatype_dir = '/content/'

How does it find the 5bqh_protein_nowat.gninatypes file in /content. Is there a search function running in core molgrid that searches for the protein gninatypes in sample.types to find in /content?

You told it where the data was when you set the data_root parameter.


[1.20.1 - Forge] Corails tombstone Incompatability

it instead drops them on the ground except for a few, like the ring of seven curses from engimatic legacy, silent gear elytra
however, nameless trinkets, artifacts, trinkets, rings of ascension, and majruszes accessories all dropped to the ground upon claiming a grave from death

@Corail31

Accessories seems to be a mod coming from Fabric and has no idea about the 'drop' event used in Forge. Fortunetely, it's not a mod used a lot and they prefer to claim to use some fabric grave mods instead of understanding the real problem ^^

I have resolved this issue within the 1.21.1 and above version of Accessories as I had gotten the support of YIGD (Which is multiplatform) and UniversalGraves (Fabric Only) which had the benefit of equipping directly.
I added the API event for adjusting accessories drops expecting such to be how many gravestone mods handled such but it was not true. I find the Event itself to at least be a failsafe to prevent this issue but I find it quite messy as it will just dupe it on the inventory or ground when retrieving the grave.
@Xplodin Recommend either turning on KeepInventory or use YIGD though connector for the time being. Up to you on how you want to handle this exactly as Idk when or if I will get to backport the change

@Corail31

Accessories seems to be a mod coming from Fabric and has no idea about the 'drop' event used in Forge. Fortunetely, it's not a mod used a lot and they prefer to claim to use some fabric grave mods instead of understanding the real problem ^^

I have resolved this issue within the 1.21.1 and above version of Accessories as I had gotten the support of YIGD (Which is multiplatform) and UniversalGraves (Fabric Only) which had the benefit of equipping directly.
I added the API event for adjusting accessories drops expecting such to be how many gravestone mods handled such but it was not true. I find the Event itself to at least be a failsafe to prevent this issue but I find it quite messy as it will just dupe it on the inventory or ground when retrieving the grave.
@Xplodin Recommend either turning on KeepInventory or use YIGD though connector for the time being. Up to you on how you want to handle this exactly as Idk when or if I will get to backport the change

Hi, I shall try YIGD, I just realized the items drop before I even get to the grave

Can confirm that Corails Tombstone works with 1.21 and above version of Accessories but will keep this open for 1.20.1 until maintenance for 1.20.1 ends

Can confirm that Corails Tombstone works with 1.21 and above version of Accessories but will keep this open for 1.20.1 until maintenance for 1.20.1 ends

do you happen to think itll get fixed by any chance for 1.20.1?
Id love to use corails but if im forced to YIGD, that is alright

when using YIGD, it seems to cause neruina to delete the grave when killed be a mob, leaving no death waypoint or even a grave

@Xplodin A backport of this issue with Corails Tombstone has been pushed that fixes the issue for 1.20.1


[Feature request] Order of class interfaces

Having the following input:
<?php

class Example implements CheckInterface, AuthInterface {}

a ordered_implements rule should order the import alphanumerically to achieve the following output:
<?php

class Example implements AuthInterface, CheckInterface {}

Other orders could be length and none (identical to ordered_imports).
This rule should not be risky, as interfaces cannot overwrite each other when being implemented.

In the case of ordered_imports sorting on none has at least the grouping.
Would a none option here not have the same effect as just not enabling the fixer?

Would a none option here not have the same effect as just not enabling the fixer?

Maybe to disable the fixer in an subsequent rule?
E.g. the Symfony ruleset enables the fixer but you don't want it, as such you set it explicitly to none.

In the case of ordered_imports sorting on none has at least the grouping.

which is a bug that will be fixed very soon by a PR of mine ;) @ see #2696 (comment)

I was talking about sort function, which in case of your comment is alpha, not none.

I created a PR for this: #4113 😄


I plan on porting this SDK to CUDA 7.0 from the ancient OpenCL implementation. Few quick questions.

Are there any gotchas in the original implementation? I plan, at least initially, to make my port as similar as possible to the original code just to get it functioning first (Eg: No dynamic parallelism, and minimal shared memory.) Anything I should look out for?
I'll probably do this sometime next week and (hopefully) create a pull request before the end of the month.
I'd port it to a newer version of OpenCL, but no reason bothering until Vulkan comes out :P

Actually, a few more questions. Whats OP_REC2 and OP_REC1? Are you doing a tree transversal with those, and they're showing which way it branches next? (Sorry if I create a plethora of questions, porting uncommented code often leads to confusion on all ends haha.)

I made the current OpenCL version, but actually it is slower than CPU even for GPUs with many ALUs. The reason is that I used a suboptimal strategy: I reimplemented the stack-based recursion mechanism in OpenCL, which means that all ALUs in a group would wait until the longest execution path has terminated. This is clearly not the good solution.
What is your parallelization strategy for the CUDO implementation?

Actually, a few more questions. Whats OP_REC2 and OP_REC1? Are you doing a tree transversal with those, and they're showing which way it branches next?

This code is almost 5 years old, so I have to re-interpret it almost as much as you ;-) The op variable in the /simulated recursion stack/ is actually a form of program counter: it tells, for a given level in the stack, where the function was at that level when it called the recursion. As Recursion can be called in two places only, there are three values for the op variable:

OP_BEGIN_FUNCTION: beginning of function
OP_REC1: first recursive call
OP_REC2: second recursive call

Well if it wasn't already I planned to change hour the trees themselves were stored before being copied to the gpu. I was going to make sure the gpu version was a linear stack of nodes.
The approach I was considering would limit depth to 22 in most cases, so I need to go back to the drawing board. I was going to recursively call the kernel every time I followed a branch. I think there is a way to alleviate this limit, but I can't test that since I recently bricked my titan and only have and 760 to play around with. The 760 only supports 22 depth at most.
I was also considering presorting search keys if libnabo doesn't do that already. That alone could create a 3x performance increase gpu side and perhaps an equally as big increase cpu side.
I'm not entirely sure. I need to do more research into exactly how the library works.

Oh just a heads up, I will primarily be testing the code that I write for this on Linux. I could losely insure that it'll work on Windows, but no solid promises.

Feel free to ask further questions if this answer is not clear. Also, if you make a better strategy in CUDA, it would be great to back-port it to OpenCL.

I would but I haven't done any work with OpenCL in years :( I'm very rusty, and I know I would do a far better job in CUDA.

Okay I've got a really good architecture in mind (Snr Software architect, I usually nerd out more about the architectures than I do the actual code haha)1. Thought I'd share it.
While constructing the KD tree:

Sort search keys (Really crucial)
Either we can dynamically shift the bucket size on a per node basis, or we can limit its size greatly to about 10 - 15. The larger this bucket size, the more it'll hurt the system. I have a few ideas on dynamically shifting the bucket size. Most of them would be done through boosting. For a lot of reference points, the build time would be awful. It should be fine for < 100K reference points though if done correctly.
When searching:
Order query points through some form of clustering mechanism to reduce warp divergence. Use ballot when the kernel is executing. Store similar paths in shared memory so that they can be used by other threads in the warp.
When a path proves to be too divergent, launch another kernel via dynamic parallelism and split the query cluster that the warp was executing over. This should take care of a decent bit of overhead caused by ballot. Although, we should hope that no more than 2200 paths become divergent.

I think (At least I hope) this would be roughly 9x faster than the CPU version. There is hopefully very little coalescing, most memory accesses would be shared. The memory overhead should be small since most of this would be copy once and forget.
There are a few more ways to expand this though mostly with how querying is done. I could implement a quick merge sort that runs over the query points before the search function executes. That would, for most low dimension cases, be fine (Eg: k < 20 or so). But beyond that, it would be awful and I would highly recommend that some form of clustering algorithm is used. (I would advise against SVMs. I always get awful performance no matter what the implementation is.)

I also have a few more very complex ideas and I'm not sure if they'll work but I want to try them (Eg: GPU based clustering and broadphase collision checking against the KD tree. Any node that doesn't collide with us is skipped. This could potentially greatly reduce the number of steps it takes to find where we are in the KD tree, but clustering like this may take a while

Unfortunately OpenCL does not support dynamic parallelism or ballots. Although I do believe that Vulkan does, so this is something that should be easily portable once Vulkan becomes available to the general public.

What was the bug?

I do not remember, it seems right while reading it now, but it is four years old.

So I have some news. In most cases it is significantly slower than the CPU, since I can only do so much to prevent divergence. But in some cases it does fully match. It hasn't fully exceeded CPU performance.
It also doesn't help that the CPU I'm running this on is one of the fastest single threaded CPUs available on the market, and the GPU is quite awful haha. I need to finish hooking it into libnabo, but I'll push after that.

One strategy that would certainly work well with GPU is to do multiple search in parallel without backtracking. That is the strategy used by FLANN. The bad thing is that it leads to non-deterministic and unbounded error, while the epsilon-based approximation allows a controlled and bounded error. But backtracking is really bad on GPU. I imagine that people have been researching other strategies. I am currently not working on that question any more, but it seems that a Google Scholar search for kd-tree GPU leads interesting results.

I tried countless approaches, and nothing seemed to create any form of benefit over a CPU implementation :/
It may be possible, it it would require more research than I am willing to dedicate right now. I'll push some sample CUDA code to the main repo if you want so people can tinker around with it.

@MatrixCompSci thanks for the status update And thanks for spending time on this. It would be great if you could push your state and a description of your experiences on a pull-request so we can archieve it.
Thanks again for spending time on this

@LouisCastricato Hope it is OK to re-surface this issue. Your experiments were very cool and had a strong potential to improve performance of the library (which is already quite good).
I have a few questions. Did you try or consider

Running linear search on all the points vs. spatial parttiioning through the Kd-tree?
Building the Kd-tree on the CPU and transferring it into the GPU?
Individual timing of the different K-NN steps?
Any performance analysis to understand whether you are memory or compute bound atm?

Thank you very much in advance.
Best,
Yoshua Nava

Funnily enough Im actually working on a LSH (locality sensitive hashing) variant of this with some folks at ETH Zurich (Seeing that libnabo is also from ETH, I'm collaborating with Bastian Rieck) and a security company.
So for kNN, this method is awful. The one I created here. The vote-ballot architecture does not scale well as you increase k since the cache coherency penalty is massive.
Linear searching through all the points is awful. The key to preserving cache and SIMD coherency is skipping branches. Thats why you can use LSH. Lets say that your data is N dimensional. Provably, you can construct an n << N dimensional coordinate space using LSH such that you can determine the first n < m < N branches that your data is under.
The dataset I am working with is in the range of 5PB - 10PB (Cant give specifics, sorry). If we were to construct a tree such that the GPU kernel thread blocks diverged early on, that would mean we would be heavily straining the bandwidth of the card as well as straining the network infrastructure and losing all hope of SIMD (given a nonbalanced tree). Said being, assuming we can predict the first m branches we can greatly reduce the strain on the GPU.
This is the main issue that I found when trying to port libnabo to CUDA. The strain was massive due to early on divergence. As such, the best way to do this would be to cluster on hashes and then execute kNN.
So, that answers (2). Don't use a kd-tree. Atleast, do not use exclusively kd-trees. You won't get anywhere. When trying to make this parallel your #1 priority is figuring out how to order the queries optimally.
(3) The later steps are by far the slowest since thats when all the threads are diverged. Increasing performance here requires decreasing divergence early on.
(4) The bound is like 50% memory bandwidth and 50% SIMD vs MIMD. Bastian and the fellow at the security company are discussing more abstract methods that I dont know if I am allowed to discuss to predict early branches.
Personally I am about to start a CS+Math PhD (wow have things changed since I posted this) so I might eventually come back to something like this. I doubt it would be libnabo in specific.

@LouisCastricato Thank you for your thorough and prompt reply.
I'm writing a few answers/questions below your comments:

So for kNN, this method is awful. The one I created here. The vote-ballot architecture does not scale well as you increase k since the cache coherency penalty is massive.

Are you trying to perform KNN on a similar test case as the one for which libnabo was designed? i.e. 3D point clouds where you want to find 1 match per point, and the dimension of points is at most 3. In the original paper that introduced libnabo, there was a really good comparison with FLANN, ANN and an octree-based strategy, as well as an explanation for many design aspects: https://hal.archives-ouvertes.fr/hal-01143458/document

Do you aim to run K-NN completely on the GPU? Do you think that's the best strategy?

Linear searching through all the points is awful. The key to preserving cache and SIMD coherency is skipping branches. Thats why you can use LSH. Lets say that your data is N dimensional. Provably, you can construct an n << N dimensional coordinate space using LSH such that you can determine the first n < m < N branches that your data is under.
The dataset I am working with is in the range of 5PB - 10PB (Cant give specifics, sorry). If we were to construct a tree such that the GPU kernel thread blocks diverged early on, that would mean we would be heavily straining the bandwidth of the card as well as straining the network infrastructure and losing all hope of SIMD (given a nonbalanced tree). Said being, assuming we can predict the first m branches we can greatly reduce the strain on the GPU.

So you are saying that the algorithm is bound by the front-end (branch prediction misses). Does this apply to both CPU and GPU?

This is the main issue that I found when trying to port libnabo to CUDA. The strain was massive due to early on divergence. As such, the best way to do this would be to cluster on hashes and then execute kNN.

How do you define divergence in this context? Is it an excess of branching in the branch-and-bound process?

Clustering is faster because it removes the memory bandwidth penalty for the first m steps, and it diminishes the cache coherency penalty after that.

You mean point clustering, as an alternative method to partition space, instead of the Kd-tree?

Sorry if some of the questions are too basic. I'm not a K-NN expert, so I'd like to make sure that I understand what you mean in full.
On a separate note, I've run libnabo on 3D point clouds through Intel VTune (hotspots, uarch, mem-access) and Advisor. Something quite interesting is that the footprint of this library is super small, as mentioned in the paper. It is really well designed. But it seems to have some back-end bounds (execution units parallelized on multiple cores through OMP taking longer than expected). This is only an initial exploration, I'll resume this as I also get more deeper knowledge on the algorithmic level, otherwise any low-level optimization might be low gain.
Best,
Yoshua Nava

Kind ping @LouisCastricato :)


Using Yajra datatables as service..but can't use Model accessor.

I have a custom accessor which is just collection of different string joined together to make a not so unique number but I can't use it with datatables.

protected $appends = ['unique_name'];
public function getUniqueNameAttribute($value)
 {

    return $this->posts->name . sprintf('%02d', $this->authors->id) . date('Ym') . sprintf('%02d', $this->id);

 }

I followed this link while making the datatable: https://datatables.yajrabox.com/service
btw I got a completely different code than shown there, maybe cause of updates.
but anyways it works 100% when i select some fields from the database but I had a question or rather an issue
How can I use my accessor for displaying it on datatable
I'm trying to use it like this which obviously doesn't work
public function query(Project $model)
{
    return $model->newQuery()->select('id', 'project_title', 'unique_name', 'due_date');
}

I know this is obviously not work but how can I use it?
System details

Operating System: Debian Stretch
PHP Version: 7.2.9
Laravel Version: Laravel Framework 5.7.7
Laravel-Datatables Version: "yajra/laravel-datatables": "^1.0"

You can use this undocumented PR https://github.com/yajra/laravel-datatables/pull/1416. Or manually add it using addColumn. Thanks!


min_epochs and EarlyStopping in conflict

Bug description
I have a problem where I use min_epochs because it can take a while before the training starts to converge.
EarlyStopping is triggered quite early, but I thought to set min_epochs appropriately to 'get over' that initial period.
However, even though training is converging by the time we reach min_epochs, early stopping will stop training immediately once we reached min_epochs, just because it was triggered very early on in training.
I think that EarlyStopping should pick itself back up if we improve upon the monitored metric before reaching min_epochs.
Example Trainer config:
trainer = L.Trainer(
        max_epochs=10000,
        callbacks=[
            EarlyStopping(monitor="val_loss", mode="min", patience=100),
        ]
        min_epochs=1000,
    )

Now imagine EarlyStopping triggering at epoch 100, but val_loss improving at 101 all the way until epoch 1000 - right now training will still stop.
What version are you seeing the problem on?
v2.2
How to reproduce the bug
No response
Error messages and logs
No response
Environment
No response
More info
No response

I also see this and think the implementation would be better suited if, after min_epochs is reached, EarlyStopping takes precedence. As it stands right now, it is as if EarlyStopping does not exist because training exits once min_epochs is reached no matter what.

I think the code in this issue would solve it: EarlyStopping with warmup.
class EarlyStoppingWithWarmup(EarlyStopping):
    """
    EarlyStopping, except don't watch the first `warmup` epochs.
    """

    def __init__(self, warmup=10, **kwargs):
        super().__init__(**kwargs)
        self.warmup = warmup

    def on_validation_end(self, trainer, pl_module):
        if (
            self._check_on_train_epoch_end
            or self._should_skip_check(trainer)
            or trainer.current_epoch < self.warmup
        ):
            return
        self._run_early_stopping_check(trainer)


SqliteError: attempt to write a readonly database

Hero can't start anymore, I've emptied ~/.cache/ulixee without success.
It was all working until it didn't.
Instances are started with userAgent: ~ chrome >= 125 && windows >= 10
SqliteError: attempt to write a readonly database
    at Database.exec (/home/user/hero/program/node_modules/better-sqlite3/lib/methods/wrappers.js:9:14)
    at new SqliteTable (/home/user/hero/program/node_modules/commons/lib/SqliteTable.ts:26:15)
    at new CertificatesTable (/home/user/hero/program/node_modules/core/models/CertificatesTable.ts:8:5)
    at new NetworkDb (/home/user/hero/program/node_modules/core/dbs/NetworkDb.ts:24:25)
    at new HeroCore (/home/user/hero/program/node_modules/core/index.ts:104:22)
    at new CloudNode (/home/user/hero/program/cloud/main/lib/CloudNode.ts:137:21)
    at GetHeroPlayground (/home/user/hero/program/browse.js:1739:17)
    at SpawnBrowser (/home/user/hero/program/browse.js:1776:30)
    at async ProcessMessage (/home/user/hero/program/browse.js:3485:26)
    at async MessagePort.<anonymous> (/home/user/hero/program/browse.js:3657:18) {
  code: 'UNKNOWN_SQLITE_ERROR_1544'
}

Solved, it was in /tmp/.ulixee


Unclear current_user_stripee function

See https://github.com/felixog/coursify/blob/master/coursify/app/controllers/pages_controller.rb#L114
What the purpose of the function?
As I understood it returns price which a user should pay upon registration.
Why it depend on amount of registered students?
Why it uses such uneffective code? It seems like a rails newbie written this.
you wrote this:
s = StudentUser.all
count = s.size

whereas a correct way would be to write it in such way:
count = StudentUser.count

This way it doesnt fetch hundreds of records from database but just calculates count and returns a result.

So I can increase the price after every order, without creating another table for orders just yet.
I know I have a lot of refactoring to do, I'll get on it once I get to a laptop, before I start adding new features.
—
Sent from Mailbox
On Mon, Apr 6, 2015 at 5:59 PM, Vitaly Dyatlov<EMAIL_ADDRESS>wrote:

See https://github.com/felixog/coursify/blob/master/coursify/app/controllers/pages_controller.rb#L114
What the purpose of the function?
As I understood it returns price which a user should pay upon registration.
Why it depend on amount of registered students?
Why it uses such uneffective code? It seems like a rails newbie written this.
you wrote this:
s = StudentUser.all
count = s.size

whereas correct a way would be to write it in such way:
count = StudentUser.count

This way it doesnt fetch hundreds of records from database but just calculates count and returns a result.
Reply to this email directly or view it on GitHub:
https://github.com/felixog/coursify/issues/3

Dyatlov that's not Rails code, that's straight up Ruby...
Calling people Rails newbies is not constructive.

@seanbolak what's wrong with word 'newbie'?
I see that the code is written in a bad style and in a bad way so from there come all my conclusions.
I wouldn't let such code go to any of my projects.

Thanks guys, that issue has been fixed now (see my last push).
The "stripee" function was a temporary solution (hence the bad name). I forgot to fix it.


Fix opensuse provision

Description
Fixes OpenSUSE OS detection and sets daemon options through docker's sysconfig file.
Related issue(s)

Maybe related to the first commit: https://github.com/docker/machine/pull/4511


Sandbox mode?

How do you make test payments?

I would like to know this too.

https://support.coinbase.com/customer/en/portal/articles/2648039-do-you-have-a-sandbox-for-testing-how-do-you-recommend-testing-my-integration-?b_id=13521
No sandbox according to Coinbase.

As I understand we have to test the ecomerce API with real transactions?

@baranovserhiy Correct. As far as I can tell there is no testnet option.

@joshuachinemezu @maksim-s @guacamoli @imeleshko What is your official policy on support/PRs here?
And particularly this question is important, and should be covered in your FAQ.

@sjelfull @baranovserhiy sorry for the massive delay, I'll be taking a look at this repo this week.

I’m closing this issue because it has been inactive for a few months (granted, maintainers for this repo were in flux for a while). Please feel free to reopen it if you still have something to report or issues around testing in production, thanks!

There is any note regards this?

Sandbox mode always off for the php api?

@oa-coinbase was there any update on the testnet for Coinbase Commerce?
I am pinging in this as you have mentioned there might be some news at the end of the previous year, 2020.

@oa-coinbase was there any update on the testnet for Coinbase Commerce?
I am pinging in this as you have mentioned there might be some news at the end of the previous year, 2020.

@oa-coinbase any update? its 2022

@oa-coinbase was there any update on the testnet for Coinbase Commerce?
I am pinging in this as you have mentioned there might be some news at the end of the previous year, 2020.

@oa-coinbase any update? its 2022

I'm sure 2050 will come and nothing has changed, remember, it's coinbase...

Really disappointing.

I can't test my code, moving to another service.

If you've built a custom integration using our API, there are a couple ways to test your integration. To test that your service responds to webhook notifications appropriately, you can send test webhooks by visiting the settings page, scrolling down to the Webhook subscriptions section, clicking on Details next to the relevant webhook, and clicking the Send test button. Additionally, we recommend making a small purchase from your store to ensure that the end-to-end integration is working correctly.
At this time, we don't offer any test or sandbox accounts, but it's something that we’re hoping to provide in the future.

List of webhook events
charge:confirmed
charge:created
charge:delayed
charge:failed
charge:pending
charge:resolved
invoice:created
invoice:paid
invoice:payment_pending
invoice:unresolved
invoice:viewed
invoice:voided
I hope it helps. It helped me. Invoice API docs are incomplete.


[Feature request] Enable nullable reference types

1. Description
Nullable reference types (NRT) is a C# 8.0 language feature that makes it easier to prevent and debug NullReferenceExceptions.
2. Exception
Due to the lack of NRT, you currently don't get a compiler warning when using something like HtmlNode.SelectNodes(). Then when you try to use it in a foreach without a null check, you get a NullReferenceException.
var nodes = htmlNode.SelectNodes("");
foreach (var node in nodes) // exception here if no node matched
{
    // do something
}

Hello @Drake53 ,
It's already possible to fix this issue by turning off the BackwardCompatibility option such as
var doc = new HtmlDocument() {BackwardCompatibility = false};

I will discuss with my developer if we should release a version 2.0 of the library with only this issue fixed. This one is so big and will break so many codes and applications that we never feel comfortable to directly fix it.
Best Regards,
Jon

Thank you for the response @JonathanMagnan, I was not aware of the BackwardCompatibility field. However I think there may be a misunderstanding, since this does not address my issue which prompted me to create this feature request.
I tested with the following code, but nodes is still null:
var doc = new HtmlAgilityPack.HtmlDocument();
doc.BackwardCompatibility = false;
doc.LoadHtml("
Hello world
");
var nodes = doc.DocumentNode.SelectNodes("img");

NRT is a compile-time feature which helps developers prevent NullReferenceExceptions, but for it to work, libraries such as this need to annotate their code properly. In the above example, the method currently looks like this:
https://github.com/zzzprojects/html-agility-pack/blob/master/src/HtmlAgilityPack.Shared/HtmlNode.Xpath.cs#L40
public HtmlNodeCollection SelectNodes(string xpath)

However, if no node matches, null is returned: https://github.com/zzzprojects/html-agility-pack/blob/master/src/HtmlAgilityPack.Shared/HtmlNode.Xpath.cs#L54
IMO it makes more sense to return an empty list, but that's a separate issue.
With NRT, the method should be updated to indicate that the return value may be null:
public HtmlNodeCollection? SelectNodes(string xpath)

The compiler will then warn me if I try to use the result directly (for example in a foreach loop), prompting me to add a null check first.

Hello @Drake53 ,
Indeed I misunderstood the issue. I'm a little bit too used to the issue of not returning an empty collection.
After some discussion, we choose not to implement it.
The best solution would be for the library to return an empty list. We are still discussing if we really want to release a v2.x with this fix.
The second best solution would be yours. Implementing NRT is a very good suggestion and will surely give at least some hint to many people, but unfortunately, this kind of change for a library as much used as Html Agility Pack can also cause some issues for other developers. From our history with this library, whenever we do this kind of change, that always fires back to us, and we need to roll back as we broke someone else code.
I will keep this issue open until we make our decision.

Yeah I can imagine some people getting upset if the compiler suddenly gives them a NRT warning on in this example SelectNodes, even though they have set OptionEmptyCollection to true so it never returns null.

@JonathanMagnan
TL;DR: Regarding the concern expressed in your comment, the "second best solution" is in reality a far better solution than the "best solution".
i do not agree with your argument. Here is why:
As a foundation i will take your concern of "From our history with this library, whenever we do this kind of change, that always fires back to us, and we need to roll back as we broke someone else code." This is a sensible concern to have and which i wholeheartedly agree with.
However, the rest of your argument, the choice of "best" and "second best" solution do completely go against that concern.

"Best solution": Returning an empty collection instead of null.
This changes the behavior of this API and therefore poses the exact risk you are concerned about: breaking someone's code that relies on the fact the method returning null when no matching nodes are found.

"Second best solution": Using nullabilty annotations for reference types.
Nullability annotations for reference types do not change the behavior nor the real signature of this API. (They are merely metadata giving hints/information for static code analysis done by the compiler.) Therefore, there is little risk breaking someone's code, simply because neither the signature nor the behavior of the method changes.

It is quite puzzling to me how you could express a concern for risking to break existing code using HAP, while at the very same time expressing to favour a "best solution" that exhibits this risk to a much higher degree than the "second best solution" would.

Thank you @elgonzo for your insight,
I will surely think about it and reconsider my position over the weekend.

Vote +1.

@JonathanMagnan what I don't understand is that you talk about breaking code or breaking changes.
NRT the only thing it does is that if I use this library and in my csproj I have set <Nullable>enable</Nullable> I will start to get warnings in cases where I do not control the references that may come null, nothing else:
https://learn.microsoft.com/en-us/dotnet/csharp/nullable-references
If you are willing to implement it I offer to create a PR and implement it.

Hello @bdovaz ,

Nowadays it is common to target .NET Standard 2.0 and .NET 6+.

I agree, but we try to be backward compatible. Why do I agree that .NET Standard 1.3 and 1.6 are pretty much obsolete, I don't think dropping .NET Framework 4.0 will be a good thing. A lot of people still use HAP in their legacy projects, and it normally just takes us a few more minutes to deploy, so it's not a big deal either. But yeah, we have too many target hehe

If you are willing to implement it I offer to create a PR and implement it.

Sure go ahead. I cannot guarantee it will be merged but we will sure really consider it.
From what I understand, @elgonzo, you would also HAP go ahead with this solution, right?
Best Regards,
Jon

I don't think dropping .NET Framework 4.0 will be a good thing. A lot of people still use HAP in their legacy projects [...]
From what I understand, @elgonzo, you would also HAP go ahead with this solution, right?

I tend to agree with going forward with this, for the 2 cents my opinion is worth...
However, i have no experience with compiling source code featuring nullable reference types (which is a C# 8 language feature) for very old build targets like .NET Framework 4.0, and cannot really make a blind yet reasonable assumption about the expected quirks and edge cases such a build configuration might exhibit.
In general, if support for rather old framework targets (such as those not yet supporting .NET Standard, like for example the mentioned .NET Framework 4.0) prevents you from including useful features in HAP, then it is in my opinion time to ponder whether support for these old framework targets actually still outweighs useful features. I am not discussing here about the costs/efforts required for introducing new features/stuff to the library code. Costs vs benefits is an important consideration for the HAP authors/maintainers, but it's not what my ramblings here are about.
.NET Framework 4.0 went out of support more than seven years ago. In my opinion, devs that still need to maintain legacy projects that require a framework/runtime environment that is out-of-support for many years already should be more than content in also using some old-ish legacy version of HAP (or other libraries) for these old legacy projects. Personally, an argument like a project that is being stuck on .NET Framework 4.0 (or older) and that is not able to migrate to for example 4.7.2 (the earliest .NET framework version fully supporting .NET Standard 2.0 without issues) but at the same time somehow apparently requiring the newest and shiniest version of HAP from the year 2023 doesn't make much sense to me. But remember, this is just my opinion here, so feel free take from it whatever you want... or not... :-)

Even then, targeting .net 4 and .netstandard 2.0 while using nullable reference types at same time might still work. As NRTs are design time sugar.


Unitialized String causes Segmentation Fault

class Foo
  getter x, y
  def initialize
    @x = uninitialized Int32
    @y = uninitialized String
  end
end

puts Foo.new.x
puts Foo.new.y

When trying to call Foo.new.y
Using compiled compiler at .build/crystal
0
Program exited because of a segmentation fault (11)

I don't know what else you expect.

@sdogruyol Right now the equivalent is this. As it's said there, using this is unsafe.
The 0 is because right now when you do Foo.new the compiler will allocate memory for Foo and initialize it to zero, but this might change in the future (we do this because Nil always has a type id of zero, so nilable instance variables always get the right type with this trick).

Okay, now i get this.
But shouldn't this be a compile time error when trying to call something uninitialized like Foo.new.y?

@sdogruyol No, the idea of an uninitialized variable is "Compiler, don't bother me if I don't properly initialize this variable, just consider it initialized". Later, if you don't initialize it, it's your fault, and bad things can happen (including segfaults).

This feels really unsafe to me.
Seems like i got used to safety of Rust thinking that this should be a compiler error or at least warning.
FYI you can close this issue at will.

This is really unsafe. This is intentionally really unsafe, it's the reason d'être for this feature. Don't use it until you absolutely need it.

Yes, it's unsafe on purpose. In rust you have unsafe { ... } which basically is "Compiler, don't check the type safety of this, I know what I'm doing, because I need to do some things here in a way that, by your rules, would be slow". In Crystal we don't have unsafe { ... }, but there are just 4 unsafe things, so it's basically the same.
Oh, I just searched how to do the same thing in Rust, and what a coincidence!
https://doc.rust-lang.org/std/mem/fn.uninitialized.html
It's even named "uninitialized"! I swear we didn't know that before choosing that name. And there's even a big paragraph with the title "Undefined behaviour", and it says in bold "This is incredibly dangerous, and should not be done lightly. Deeply consider initializing your memory with a default value instead." :-)

@asterite i didn't know about that. Thanks a lot :smiley:


Misguidedto

[Enter feedback here]
Run a failover- step 10 “Site Recovery doesn't clean up the source VM after failover. You need to do that manually.”
If cx would like to re-protect the VM, the source server could not be deleted then will cause the re-protect failed.
Please add the description that if the VM doesn't reportect anymore, then it could be cleanup.

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

ID: 9814de5d-d6e2-463d-f4b9-dffa577daec3
Version Independent ID: 577fe16b-ce8a-8a73-b040-a9ba9abfe35b
Content: Tutorial to fail over Azure VMs to a secondary region for disaster recovery with Azure Site Recovery. - Azure Site Recovery
Content Source: articles/site-recovery/azure-to-azure-tutorial-failover-failback.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.
@rayne-wiselman  - fyi.

#reassign:v-harisudhan

As advised by my manager @jyothisuri closing the issues that are older than a year.

#please-close


Added hoogle plugin

Can you modify this to use the library approach instead, now that Hoogle 5.0.11 is out which provides defaultDatabaseLocation?
You will probably have to add it as an extra dependency at the end of  stack.yaml


Expose image channels

This change comes from the need to save the texture as an image file
In order to save the image from raw data, the width, height and channels information should be provided. This change exposes the channels property to do just that.

Nothing wrong with that. Thanks for contributing!


[BUG] Chart Editor Song Options Menu has selection reticle for Player 2

Is there an existing issue for this?

[x] I have searched the existing issues and none matched.

Operating System
Windows 10 x64
CPU
No response
GPU
No response
Storage
No response
Game Version
4.15.0
Game Mode
dance
Theme
Default, but happened too on this port of CSLA
Describe the problem
When going into the song options in the chart editor, a selection reticle is displayed for player 2, preventing me from making changes to xmod and playback speed without moving player 2's selection to the exit button. One can still exit the menu with Escape, but then changes will not be applied (though the UI state remains the same when opening the menu again).

Describe what should happen
By my reckoning, chart editing is not multiplayer. Only player 1 should be able to select things and be able to confirm the selection by himself.
Relevant Log output
info.txt
log.txt
timelog.txt
userlog.txt

Duplicate of #467.

Closing since I failed to think of "Edit/Practice" in addition to "Chart Editor" when searching for existing issue.

That is understandable, internally the editor/practice are the same thing, just with different flags to determine their functionality.