Add a demo site to this project

I searched the website and README and could not find one example of how this PDF viewer looks, and I can interact with the component.
Adding a demo which shows its capabilities (or at least a screenshot) would help me to decide if I want to switch to this PDF viewer.

You can find examples in the documentation website: https://tato30.github.io/vue-pdf/

vue-pdf is not a full PDF viewer is only a tool/component to make one.


Redirect from www. to fasttrackml.io

There is currently a DNS error when trying to access www.fasttrackml.io.
The website is only accessible without the "www" at the beginning of the URL.
We need to set up a redirect from www.fasttrackml.io to fasttrackml.io in order to avoid this error.

hey @c-rindi, could you please setup a CNAME for www.fasttrackml.io that points to g-research.github.io.?

Redirect completed. 👍🏽

Done. 👍🏽

And www (wildcard) url redirect is also completed. 👍🏽

@c-rindi could it be that you did it as a redirect with the domain registrar instead of setting up a CNAME to g-research.github.io? I'm getting certificate errors when I try to go to https://www.fasttrackml.io/

CNAME Record has been corrected from "@" to "www" and now www.fasttrackml.io redirects to https://fasttrackml.io/.

DNS config should be :ok_hand: now.
We still have one issue: navigating to https://www.fasttrackml.io/ is causing an SSL certificate issue. We may need to remove and then re-add the custom domain to trigger the process of enabling HTTPS.
Could someone with enough permissions try this please, @suprjinx?

...After you update existing DNS settings, you may need to remove and re-add your custom domain to your site's repository to trigger the process of enabling HTTPS. For more information ...

[Troubleshooting custom domains and GitHub Pages](https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/troubleshooting-custom-domains-and-github-pages#https-errors)

I've updated the settings and everything is 👌 now — thanks @c-rindi and @naskio!


Byte to hex string conversion optimizations

Per #698 I found the performance of the plugin to be a problem when moving data through a notification characteristic as fast as possible.
This PR is focused on that use-case; high-throughput data transfer from peripheral to the central. There may be additional performance gains to squeeze from the reverse path, however I haven't tested that case yet and suspect it is not as significant of an issue.
I would appreciate extra scrutiny on the optimized Swift conversion function, as I could only test using the windows swift toolchain on my PC. The TypeScript side passes the existing conversion spec tests and the Kotlin side has been tested running on my Pixel 8 Pro passing data from an nRF devkit (the nature of this specific app requires the egressed data to contain a SHA-256 hash which is checked by the central so I am very confident data integrity is being maintained there).
I ran a profiling session moving 512KB of data from this implementation and recorded a wall time of only 124.61ms to perform the byte->hexstring conversion:

Appreciative of any feedback or suggested improvements necessary to land this. 👍

Nice work @NaterGator
I'll do some testing on this later this week hopefully.
It looks like the lint target has failed... are you able to fix this up?

@peitschie updated to fix the linting error and failing tests. Unfortunately I can't run verify:ios or lint:ios on win32, so I'll need to wait for the CI build to see if the fallback iOS<14 path passes.

I just pushed some demo projects that might help with evaluating these changes.
There's a Zephyr project to pump a BLE notification characteristic with as much data as possible available here: https://github.com/NaterGator/zephyr-ble-notif-throughput I've included hex builds for the Nordic nRF52832 and nRF52840 Development Kits.
There's a Vue 3 capacitor app here: https://github.com/NaterGator/capacitor-ble-indication-throughput
When running together you get something like this which makes for a pretty easy profiling target...

I've tested this on Android and iOS and can confirm the conversion all works as expected on both platforms.
Thanks for the careful work here @NaterGator ... your harnesses and associate embedded code helped speed things up a lot!
@pwespi is there anything further you'd like to see here before we squash and merge?

I've tested this on Android and iOS and can confirm the conversion all works as expected on both platforms. Thanks for the careful work here @NaterGator ... your harnesses and associate embedded code helped speed things up a lot!
@pwespi is there anything further you'd like to see here before we squash and merge?

Looks good to me. Thank you for the great contribution @NaterGator!


thumbnail/attachment upload uses inconsistent directories for raw and derived datasets

Hi. During the time I was working on #248, I found an inconsistency in the URL formatting. This may be due to the old-ish version of SciCat we're running, but just in case it isn't addressed yet:
When uploading the thumbnail for the raw datafiles, we need to add "RawDatasets" to the URL. However, when uploading the thumbnail for the derived datafiles, we do not add the expected "DerivedDatasets" to the URL, but instead must use "datasets". This is confusing and should be one or the other.

Is this resolved for you Brian?

Is this resolved for you Brian?

Have yet to check, we've had the new version installed on a new server, but due to the Better with Scattering workshop, I haven't had the chance to test it. (also, there's an HTTPS warning due to an unsigned certificate that I need to find a workaround for...). Will report back as soon as I have.

I'll assume this is resolved. @toqduj reopen, if needed.

Will do, thanks!
On Tue, Nov 3, 2020 at 2:09 PM Tobias Richter<EMAIL_ADDRESS>wrote:

I'll assume this is resolved. @toqduj https://github.com/toqduj reopen,
if needed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SciCatProject/catamel/issues/249#issuecomment-721105805,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJSRSIERHB2PDEJFCEXMYLSN76I5ANCNFSM4JVY5LFA
.


[fix][test] Fix PrometheusMetricsTest.testBrokerConnection

Fixes #18535
Motivation
in #17905, we enable transaction coordinator in the broker, this will lead to opening a connection in broker, so connection_count should recalculate.
Modifications
Enable transaction coordinator in broker only in testDuplicateMetricTypeDefinitions.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/coderzc/pulsar/pull/33

Codecov Report

Merging #18539 (4362c4b) into master (aeb4503) will increase coverage by 5.36%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master   #18539      +/-   ##
============================================
+ Coverage     31.39%   36.75%   +5.36%
+ Complexity     6651     1956    -4695
============================================
  Files           697      208     -489
  Lines         68015    14430   -53585
  Branches       7285     1582    -5703
============================================
- Hits          21353     5304   -16049
+ Misses        43667     8550   -35117
+ Partials       2995      576    -2419

Flag
Coverage Δ

unittests
36.75% <33.33%> (+5.36%)
:arrow_up:

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

Impacted Files
Coverage Δ

...va/org/apache/pulsar/client/impl/ConsumerImpl.java
14.84% <0.00%> (ø)

...he/pulsar/client/impl/MultiTopicsConsumerImpl.java
22.26% <0.00%> (ø)

...lsar/client/impl/conf/ClientConfigurationData.java
96.66% <100.00%> (+0.11%)
:arrow_up:

...apache/pulsar/client/impl/AutoClusterFailover.java
70.00% <0.00%> (-6.12%)
:arrow_down:

...che/bookkeeper/mledger/impl/ManagedLedgerImpl.java

...pulsar/broker/admin/impl/PersistentTopicsBase.java

...sistent/PersistentDispatcherMultipleConsumers.java

...ersistentStickyKeyDispatcherMultipleConsumers.java

...ersistentStreamingDispatcherMultipleConsumers.java

...apache/pulsar/broker/admin/v1/SchemasResource.java

... and 484 more

/pulsarbot run-failure-checks


Fix and rephrase documentation

Signed-off-by: chenz4027<EMAIL_ADDRESS>https://github.com/open-cluster-management-io/open-cluster-management-io.github.io/issues/114

/hold

/unhold


Duplicate Set-Cookie headers

In some cases the cookie saver might produce duplicate Set-Cookie headers. Investigate and fix.
Found by Stephen at https://katanaproject.codeplex.com/workitem/367.

Hi Anders, I still have this problem, some times, come cookies get duplicated

@samueltorresff Sorry to hear that. The solution is a bit of a hacky fix that tries to repair things after the fact. In #14 I've suggested a reworked version that would work much better.


[ko] Enhance kubeadm/control-plane-flags.md

Related issue: #28409
Change line format based on the original file and some words to common likes 사용 예 -> 예시.

@ClaudiaJKang 감사합니다!
/lgtm

/approve


NodePort is not accessible on dsr mode

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
My Kubernetes cluster has two nodes and I replaced kube-proxy with the functionality of cilium.
cilium works in DSR mode，cilium works in DSR mode, version 1.14.1。
kubernetes cluster deploys the following services.

I found that on <IP_ADDRESS> executing curl http://<IP_ADDRESS>:37778 will fail, while executing curl http://<IP_ADDRESS>:37777 on <IP_ADDRESS> will fail.
these curl commands should work correctly. This problem was discovered after upgrading from version 1.12.x to 1.14.1. I think it is a bug caused by the introduction of new features.
Cilium Version
Client: 1.14.1 c191ef6f 2023-08-10T18:54:57+02:00 go version go1.20.7 linux/amd64
Daemon: 1.14.1 c191ef6f 2023-08-10T18:54:57+02:00 go version go1.20.7 linux/amd64
Kernel Version
Linux 5b4c2dd3-ca11-5f1c-ba44-0034bc5ffca5 5.10.16-1.el7.bclinux.x86_64 #1 SMP Sun Feb 14 06:58:31 CST 2021 x86_64 x86_64 x86_64 GNU/Linux
Kubernetes Version
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.17", GitCommit:"8206683d1b72be8592759ce3d595efa3c7a568ec", GitTreeState:"clean", BuildDate:"2023-05-24T06:04:26Z", GoVersion:"go1.19.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.17", GitCommit:"8206683d1b72be8592759ce3d595efa3c7a568ec", GitTreeState:"clean", BuildDate:"2023-05-24T06:03:22Z", GoVersion:"go1.19.6", Compiler:"gc", Platform:"linux/amd64"}
Sysdump
cilium-sysdump-20230914-195830.zip
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

cilium-sysdump-20230914-195830.zip

My Kubernetes cluster has two nodes and I replaced kube-proxy with the functionality of cilium.
cilium works in DSR mode，cilium works in DSR mode, version 1.14.1。

:wave: thank you for reporting this! Could you clarify the text above - I take it that on 1.14.1, DSR mode doesn't work for you? Have you started to debug what's causing the disruption? Using hubble or cilium monitor to check for unexpected packet drops or flows might be a good start here.
You probably also want to re-test when v1.14.2 is released (bringing https://github.com/cilium/cilium/pull/27381).

My Kubernetes cluster has two nodes and I replaced kube-proxy with the functionality of cilium.
cilium works in DSR mode，cilium works in DSR mode, version 1.14.1。

:wave: thank you for reporting this! Could you clarify the text above - I take it that on 1.14.1, DSR mode doesn't work for you? Have you started to debug what's causing the disruption? Using hubble or cilium monitor to check for unexpected packet drops or flows might be a good start here.
You probably also want to re-test when v1.14.2 is released (bringing https://github.com/cilium/cilium/pull/27381).

My cluster has two nodes, A and B. I ran a pod in the cluster and created a nodeport type svc for it.
I found the following issues:
If the pod is on node A, I cannot access the container on node A using the 'NodeB_IP: Nodeport '

What I observed through cilium monitor is as follows:

I tested version 1.14.2 and the problem persists.
I tested multiple versions and found that versions after 1.13.2 have the same problem, and
1.12.x, 1.13.0~1.13.2 are all normal.

Ah that's curious. It looks like the backend selection only happens on the remote node. Even though you have the SocketLB enabled - so it should happen on the client node at connect() time.
Can you grab some SocketLB logs while trying the test?

I'm not quite sure how to collect socket log.
I executed hubble observe --pod sp-network-check-9c112653-00a3-5886-88a4-e1d5634e8ec7 on node A and got the following output:

I also tried executing cilium monitor -v -t trace-sock synchronously while testing, but from the output, there didn’t seem to be anything of value.


Export function by ordinal

. . .
The question is related to:

DllExport - latest:

Is it possible to export a function by ordinal ?,
Would love to have control over that😁

Not sure what do you mean for "to export a function by ordinal". Can you illustrate the expected behavior?
Maybe you meant the same ordinals in export table using the order of the definitions in src? But I remember this is already so

In c or cpp you can create a definition file which will state your exports, you can specify the function name and right after it the ordinal it is exported to, I would be nice if in the dllexport attribute you will be able to specify a ordinal or via a different settings file, the main usage here is for having more control over the exported functions other then the order they appear in your code,

Oh, I see. Why not.
.def will be a bit weird in dotnet environment, so either attributes or some option in our configuration.
Also, never had to avoid names in my practice for C++ but why also not to allow optionally skip the names. By the way, are you trying to reduce the size or some obfuscation, or?

Today it's not possible. But this feature will be considered later. Thanks for the suggestion. Thanks for using!


Add support for GHC 8.8

Thanks! While we're at it, can you also add 8.10.1 to travis?

Yes, I extended it to run ghc-8.10.1 as well.

FYI, .travis.yml should be generated from .cabal file (the relevant bit is tested-with key) with haskell-ci, not edited manually.
You can also update older GHC versions in tested-with to use the newest minor version.

Cool didn't known about haskell-ci. Will do.

@arybczak do you think this can be merged and pushed to the hackage?

Yes. There needs to be a lower bound on base in the cabal file (>= 4.9), but other than that looks good to merge :+1:

Can we merge this PR? https://github.com/scrive/kontrakcja/pull/1491 will be using this so it is best to merge this to master.


[test] Improve the solution

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Address https://github.com/callemall/material-ui/pull/5269#issuecomment-250579140.

As raised by @pradel, we still have an issue with the regression test suite:

<EMAIL_ADDRESS>test:regressions /home/travis/build/callemall/material-ui
node test regressions -l
Building webpack bundle
Booting HTTP server
Server listening on port 8080
Connected to https://c26b9d58.ngrok.io. Now testing...
[Test] Test Suite
=====================
Running:  Avatar/ImageAvatar
✔ Element <[data-reactroot]> was visible after 90 milliseconds.
Error: ENOENT: no such file or directory, open '/home/travis/build/callemall/material-ui/test/regressions/screenshots/baseline/Avatar/ImageAvatar/chrome-53.0.2785.101-linux.png'
at Error (native)
closed! 0
if git log ${TRAVIS_COMMIT_RANGE} | grep -Ei '[codemod]'; then
cd packages/material-ui-codemod && npm install && npm test
fi


Update websocketpp to e036a5au to fix compilation under Arch and Ubuntu 17.10

Under Arch Linux (and Ubuntu 17.10 according to #196), compilation fails with an error in websocketpp: error: no matching function for call to ‘make_shared(std::reference_wrapper<boost::asio::io_context>)’.
To resolve this, update websocketpp to zaphoyd/websocketpp@e036a5aeab11b0066e5d06ea6d693431507adb5a which is the first revision after the current zaphoyd/websocketpp@ac5d7ea5 which compiles without warnings.

Looks like there is an issue with the travis build at the moment which isn't related to this change...

I have this issue in openSUSE Tumbleweed as well. Also solved by
cd socket.io-client-cpp/lib/websocketpp
git pull origin master

Thanks to @syncu2!  It is strange why git submodule reference to websocketpp still not updated to latest commit in master which works fine?

Included in https://github.com/socketio/socket.io-client-cpp/commit/af68bf3067ab45dc6a53261284e0da9afd21b636.

Included in https://github.com/socketio/socket.io-client-cpp/commit/af68bf3067ab45dc6a53261284e0da9afd21b636.


feat(smith): add DocumentBuilder::input_exhausted

This indicates we might have run into https://github.com/rust-fuzz/arbitrary/issues/139

My current attempt at working around https://github.com/rust-fuzz/arbitrary/issues/139 is to double the size of the input until it’s large enough to not be exhausted, but currently this causes DocumentBuilder::operation_definition to overflow the stack, I suspect through generating very deeply nested selection sets.

By the way, at first this PR seems unnecessary because DocumentBuilder only borrows &mut Unstructured<'_> so after it is dropped the called should be able to call Unstructured::is_empty themselves. However because DocumentBuilder only has one lifetime parameter and borrows &'a mut Unstructured<'a> with the same 'a in two places. Because something something variance, Unstructured is still considered borrowed when we try to use it. https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=193bdb3f0ffae931f060751c39ef9087 is a reduced test case.
The "proper" fix would be:
struct DocumentBuilder<'a, 'b> {
    u: &'a mut Unstructured<'b>,

… but that changes the signature of the type which would need to be updated at every use, including each impl


I can help for FR translations and MP Dedi-server testing

Hi,
Thank you for your mod.
If needed, I can help you with :

FR Translations
MP dedi-server testing (I owned my self one)

Hi Lactic68,
Thank you for offering your help!

French translations are (very) welcome.
I think multiplayer won't work at the moment. I never tried and, to be honest, is not very high on my priority list to fix that, since I only play "singleplayer" myself.  As mentioned in the Readme, pull-requests for multiplayer support are welcome.

Thanks for the pull-request!

I created a new, separate issue for multiplayer support and will close this issue, since french translations are merged. Thanks for your contribution @Lactic68


where is the UI?

I'm going through the code and I can't figure out how to run the UI.
I saw that the demo is down and that you are not able to bring it back up (issue #18 ). So I tried to do it myself but I only see the code to segment the regions. In the video it seems that there is some sort of UI to navigate the book and the annotated regions. How can I start that program/server?

Hi, thanks for the interest in the project! Yes, as mentioned in #18, we don't have the resources to reset the demo online... The codebase for the demo is not public (although we may consider releasing it in the case you are interesed), this repository only corresponds to the method described in the ICFHR paper


Problem using cancancan in engine

I want to use cancancan in an engine. So I added it to my gemspec file like this:
s.add_dependency "cancancan"

In the engines dummy app and in a test app i load my engine and I'm getting always errors about undefined methods for every cancancan method.
undefined local variable or method `load_and_authorize_resource'
undefined local variable or method `can?'
[...]

This is my ability.rbfile in the main app:
class Ability
  include CanCan::Ability

  def initialize(user)
    user ||= User.new # guest user (not logged in)
    can :manage, :all
  end
end

I tried several solutions for using cancancan in engines but I found no solution. Here are some examples:

http://mx.kelsin.net/2011/08/10/using-cancan-in-a-engine-and-your-app/
https://github.com/CanCanCommunity/cancancan/wiki/Authorization-for-Namespaced-Controllers
https://github.com/CanCanCommunity/cancancan/wiki/Admin-Namespace

Where is my fault? Is there a good tutorial to use cancancan in an engine?

I'm using cancancan in an engine: https://github.com/swobspace/wobauth. My abilities are namespaces like this:
module Wobauth
  class AdminAbility
    include CanCan::Ability

    def initialize(user)
      ...
    end
  end
end

Hi
does anybody know if this is still an issue as I have having the same problems in a Rails 4 api engine?

 I forgot to add gem 'cancan' to lib/yourenginename.rb.
For abilities, you'd store the models under app/models/yourenginename/ability.rb for example and the file would need to contain:
module Yourenginename
  class Ability
    include CanCan::Ability

    def initialize(user)
        # your init code
    end
  end
end

I had this issue, I installed cancancan and I created Ability class using generator and I was very confused why it's happening, but it solved once I restarted rails server!!


Update the dependency to core-java 0.10.65-SNAPSHOT.

Auto-generated PR to update core library dependency to the latest version.

Version already updated in https://github.com/SpineEventEngine/jdbc-storage/pull/85.


deprecation: electron-prebuilt has been renamed to electron

When running npm i after generating project using template:
deprecated<EMAIL_ADDRESS>electron-prebuilt has been renamed to electron. For more details, see http://electron.atom.io/blog/2016/08/16/npm-install-electron

As of version 1.3.1, we have begun publishing electron and electron-prebuilt packages to npm in tandem. The two packages are identical. We chose to continue publishing the package under both names for a while so as not to inconvenience the thousands of developers who are currently using electron-prebuilt in their projects. We recommend updating your package.json files to use the new electron dependency, but we will continue releasing new versions of electron-prebuilt until the end of 2016.

Thanks, updated.


Cannot Map X52 Z Rotation to Yaw

Hi,
Just testing this out for the first time. I cannot map the Rotation Axis's to any value. Only the XYZ (Z in this case being the throttle will map)
The slider also won't map.
The main issue is I cannot twist the stick which is the z rotation to the yaw but being able to use other rotation axis would be good too.
Also it would be really good if I could control camera position using a POV hat.
Thanks.

Hello,
I have mostly been using my Dualshock 4 gamepad and some old joystick I had laying around for testing and the only problem I encountered so far was that the POV hat would only register an input in the up direction regardless of which way I actually moved it.
These controllers I used for testing did not have a rotation axis so thats likely why I havent run into this issue while testing.
I'm currently looking for a solution for the problem after having reproduced it with a Raspberry Pi Pico imitating a custom HID.
When I have something ready I'll comment on this issue.

Hey @roldershaw, could you please try assigning your Z Rotation to the "Throttle Axis"?
The problem seems to be that for most devices, Unity's Inputsystem treats the x and y axes differently than z, rx, ry and rz. The latter ones are only assignable to an actual axis action (e.g. Throttle Axis). The x and y axes are usually combined into a stick control which contains button controls named up, down, left, right. This is what makes them assignable to the composite bindings.
I am looking at different options to make axes assignable too. (Maybe adding an extra dialog where you can choose wherther you want to assign a control as an axis or a button. Or adding an additional binding specifically for axis controls.)
As for the D-Pad: I have tried my old joystick and the inputsystem seems generally confused with the D-Pad. It is working without issues on the DS4 controller. I have yet to test using the Raspberry Pi Pico.

I'll give this a test tomorrow :)

Ah yes you are right, RZ can map to the throttle axis and works as expected.
In KSP1 AFBW can map the RZ, but it does just show everything as an Axis, maybe it remaps them somehow?

I will implement adding another binding to which you should be able to bind x, y, z, rx, ry and rz. I will update the experimental release on github when I am done so you can test it.

In KSP1 AFBW can map the RZ, but it does just show everything as an Axis, maybe it remaps them somehow?

AFBW more or less directly uses the lower level APIs (SDL, DirectInput, XInput). I chose not to use them because KSP 2 already comes with Unity's Inputsystem bundled and I figured it would be quicker to implement something. Also the games uses the Inputsystem for its controls so it is easy to modify them.

I have the implementation ready: https://github.com/Codenade/ksp2-inputbinder/releases/tag/experimental
There is now an extra binding for axes on actions where it makes sense:

If you have any suggestions let me know!

I was having the same issue as Rolder. Is it possible to install the experimental branch via CKAN?

Thanks very much! I'll give both of those a go!

I have updated the experimental version to reflect the latest changes in 17-cannot-map-x52-z-rotation-to-yaw. This should also close #21.
Please check if the issues still persist!


Consolidating platform targets

I came across this post about how to make a multi-platform, single-scheme library: http://promisekit.org/news/2016/08/Multiplatform-Single-Scheme-Xcode-Projects/
What do you all think about this? I can put in the work if it's something you may be interested in merging (I'm probably going to do it in my fork anyway). Just want to gauge interest and hear your thoughts about the idea. I'll also try it out in another small library, so I may have more to report after that's done.

@samsonjs At first, brief, glance this setup looks very cool! If you are already planning on doing it for your fork it would be awesome if you could suggest the changes back.
If this doesn't break Carthage/CocoaPods support, it looks like a superior setup compare to the current one IMO.

I really like the idea of smaller failure targets too. I'm interested. And we don't have any dependencies to worry about.  :-D

@samsonjs any progress on this one? Sounds cool!

@DivineDominion I haven't tried to tackle this one yet. Took a detour at work in the fall and haven't been working with ReSwift for a while, but I will be back on my new app soon and still hope to give this a shot.


fix(preference): rerender Select component if localized string changed

Types

[x] 🐛 Bug Fixes

Background or solution
#2880 的后续修复
Changelog
Preference Panel: rerender Select component if localized string changed

/backport to v2.24

/backport to v2.24

/backport to v2.24


Implement DRAW action

The DRAW action draws cards from the respective player's deck and inserts them into his hand.
Steps to implement this action:

Add action creator in src/actions.js. draw should take a playerId and amount of cards to be drawn.
Add a test to test/reducers/actions.js to ensure the action creator works.
Add DRAW case to players reducer and implement proper behavior
Add test to player reducer to ensure it handles the DRAW action.

Bear in mind the different possibilities there are for drawing cards in HS:

If the player has more cards in their deck than are being drawn remove the cards from their deck and add them to the players hand like normal.
if the player has 10 cards in their hand then the drawn cards must be moved to the graveyard instead.
if the player has no cards in their deck then the player's hero must take fatigue damage. For this case we need to keep track of how much fatigue damage a player has taken, so add a fatigue property to the player reducer. fatigue starts at 0 and increments by 1 every time the player overdraws

Let's do this one last. I'm unsure of what the best way to implement the fatigue mechanic is.


How to fill in the parameter when using vercel deployment?

Is your feature request related to a problem?
How to fill in the parameters for deploying services on vercel
Describe the solution you'd like
I don't have any except the VITE_OPENAI_API_KEY parameter

Additional context
No response

You only need to provide the VITE_OPENAI_API_KEY to deploy the service (or provide neither).
OpenAI Host: to access directly if your network environment cannot access api.openai.com due to the government's firewall.
Azure related keys: To use Azure TTS and speech recognition services

To deploy the service on Vercel, you only need to supply the VITE_OPENAI_API_KEY (or provide neither).
The OpenAI Host is for direct access if your network environment is unable to connect to api.openai.com due to government-imposed firewalls.
Azure-related keys are required for utilizing Azure TTS and speech recognition services, while AWS-related keys are necessary for using the Amazon Polly TTS service.

To deploy the service on Vercel, you only need to supply the VITE_OPENAI_API_KEY (or provide none of these when deploying and fill it in after deploying in website settings.). The OpenAI Host is for direct access if your network environment is unable to connect to api.openai.com due to government-imposed firewalls. Azure-related keys are required for utilizing Azure TTS and speech recognition services, while AWS-related keys are necessary for using the Amazon Polly TTS service.

thank you for your answer.
The deploy button cannot be clicked if nothing is filled in during deployment. The message "missing" will be displayed

And then I filled in some random parameters, then deployed them, and didn't automatically reply to me. I wonder why

Please try this
Deploying with Vercel

Please try this
Deploying with Vercel

In this case you only need to provide the OpenAI API key when deploying

@erbanku thank you very much! this vercel deploy is good.
Why is this project's vercel deploy faulty? (https://github.com/hahahumble/speechgpt )

Actually, the Vercel Deploy button is a link like the ones below.
Project's Vercel deploy button:
https://vercel.com/new/clone?repository-url=https%3A%2F%2Fgithub.com%2Fhahahumble%2Fspeechgpt&env=VITE_OPENAI_API_KEY,VITE_OPENAI_HOST,VITE_AWS_REGION,VITE_AWS_ACCESS_KEY_ID,VITE_AWS_ACCESS_KEY,VITE_AZURE_REGION,VITE_AZURE_KEY&envDescription=If%20you%20do%20not%20want%20to%20provide%20a%20value%2C%20use%20REPLACE_WITH_YOUR_OWN.&project-name=speechgpt&repository-name=speechgpt

Modified Vercel deploy button:
https://vercel.com/new/clone?repository-url=https%3A%2F%2Fgithub.com%2Fhahahumble%2Fspeechgpt&env=VITE_OPENAI_API_KEY&envDescription=If%20you%20do%20not%20want%20to%20provide%20a%20value%2C%20use%20REPLACE_WITH_YOUR_OWN.&project-name=speechgpt&repository-name=speechgpt

Ass you can see the original one requires other environment variables, and it's must, not optional, you can bypass this by deleting the other required environment variables in the link.

@erbanku ok. I get it. Thank you again!


Fix live channel

I decided to fix the live channel functionality instead of dropping support for it. This has not been tested enough and closes #43 when it has been tested (and possibly fixed)

Has been tested enough, works like a charm


add service test case - [closed]

In GitLab by @sokril on Feb 6, 2018, 07:05
Merges test -> dev

In GitLab by @wknet123 on Nov 13, 2019, 10:30
the coverage is  12.8%   check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:30
closed


F18 information outdated

It seems that the F18 front-end branch has been accepted as the brand new flang and is part of the official LLVM project. The former flang compiler still exists here.
I can take a shot at updating the page in the upcoming days, if no one else does it first.

Absolutely, let's fix it!

Yes, I think it should be just called Flang.

@ivan-pi I believe this was fixed with #16 and can be closed?

Sure. Great work!


A way to get notified when watch build fails?

Is there a way to be notified when a watch build fails?
I have watch monitoring our code base to run jshint, etc at all times.  When any of the tools fail, I would like to use grunt-play to play a sound that let's me know something is broken.

I was trying to do something along this line: http://stackoverflow.com/questions/17351717/grunt-watch-detecting-success-and-failure-of-tasks but with grunt watch and TypeScript. I have the same issue, there doesn't seem to be an easy way to get the feedback from a task when it's done.


Use IEC standard abbreviations (GiB, TiB, etc)

resolves #477
see also https://github.com/dbt-labs/dbt-core/pull/6742
Description
When using multiples of 1024, the IEC standard abbreviations are "GiB", "TiB", "PiB" instead of "GB", "TB", "PB" (which are for multiples of 1000).
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Hey @dbeatty10 do we expect any side effects here? Or is it "just" about the log format?

Hey @dbeatty10 do we expect any side effects here? Or is it "just" about the log format?

I'm not aware of any side effects -- I think its "just" about the log format! My understanding is that these units are for human consumption rather than machine-readable values.
Here's an example screenshot from the original implementation in https://github.com/dbt-labs/dbt-core/pull/2530#issuecomment-642902939


Ror4

Create Observation:	Project: and "Species List:" should be bolded like the "Upload Images:" and other section labels.	Done
All:	Too much white space on right and left	Done
All:	Box Around logo to tiight	Done
Image partial:	Fix Image Rating blue bar	Done
Home:	Filter checkboxes too much space	Done
Image partial:	move expand icon to bottom left	Done
All:	Action links on right	Done
Show Observation:	Remove grey box in species list	Done
Show Observation:	proposed name titles redundant	Done
Contributors Page:	Verticle Images Cut off	Done
Contributors Page:	All text on left, image on irght	Done
Ask Webmaster:	Text is too long	Done
Ask Webmaster:	Make TextArea larger	Done
Publications:	Link Broken on nav	Done
How To Page:	Headings are too small	Done
Introduction Page:	Headings are too small text doesn't wrap	Done
All:	Links - fix the color	Done

Coverage decreased (-0.29%) to 83.62% when pulling 4d26a40dcbf7048fdd6dd46b155f56a46e1ea9e9 on raysuelzer:ror4 into 774cff3852a3cf8cfae255b74d57eb78ec20d2b7 on MushroomObserver:ror4.


Changing organisation should navigate user back to home page of organisation

Steps To Reproduce:

log in to argo
Navigate to org -> Repository
Change the org
User still able to see the same repo only the owner name is changing.

Pushed the fix from this!!!


focal adhesion cellular component and process graphs do not align

Hi
I am concerned that these two graphs do not align.
Either focal adhesion CC terms should include a cell-cell junction parent term.
Or the biological process GO:0048041 focal adhesion assembly should not have the parent term GO:0045216    cell-cell junction organization
see screen shots
Thanks
Ruth

Looks like the problem is with the parent of adherens junction assembly.


Add injector middleware

Uses Status as error response format
Use go-playground/form for parsing url.Values
Use go-playground/validator to validate structs
Removed support for multipart/file

Signed-off-by: Tamal Saha<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


RTSP stream crash

Hi, I tried to stream cctv feed using rtsp but everytime I do ue4  will stop responding. I tried to stream the same rtsp link using VLC player and it's working fine.
Other link I tried rtsp://<IP_ADDRESS>/vod/mp4:BigBuckBunny_175k.mov > this link also cause ue4 to stop responding while working fine in VLC player
Additional info:

Windows 10
Unreal Engine 4.17 from Epic Launcher
VLCMedia plugin with tag 4.17 copied to project folder
I couldn't provide the exact link as it is internal use only

Found the solution in issue #36 by deleting libscale_plugin.dll


Rewrite nft parsing using json

Rewrite the full nft parser to use json output of nft
Add more logs

This is done this way to avoid some inconsistency of the human readable nft output that sometimes causes parsing issues.
This might have cause some port forward deletion to fail because zos can't find the rule handle anymore.
Probably fixes #77
The new parser has been covered by unittests, but we need to test actual nodes that were affected by this bug.

Codecov Report

Merging #83 into development will increase coverage by 0.71%.
The diff coverage is 60.37%.

@@               Coverage Diff               @@
##           development      #83      +/-   ##
===============================================
+ Coverage        25.34%   26.06%   +0.71%
===============================================
  Files               54       54
  Lines             5157     5337     +180
===============================================
+ Hits              1307     1391      +84
- Misses            3741     3812      +71
- Partials           109      134      +25

Impacted Files
Coverage Δ

base/nft/nft.go
33.33% <0%> (-21.51%)
:arrow_down:

apps/core0/subsys/containers/network.go
0% <0%> (ø)
:arrow_up:

apps/core0/helper/socat/socat.go
21.17% <0%> (-0.26%)
:arrow_down:

base/nft/parser.go
69.18% <71.11%> (-12.63%)
:arrow_down:

apps/core0/subsys/containers/flist.go
39.44% <0%> (+6.56%)
:arrow_up:

Continue to review full report at Codecov.

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


Remove webhook conversion logic

In preparation for https://github.com/cert-manager/cert-manager/pull/6455.
This PR removes all the CRD webhook conversion logic.
We currently only support the v1 webhook version, so this logic is not used.
Kind
/kind cleanup
Release Note
NONE

/approve


Configure with gradle

Gives the option of using Gradle instead of Maven.
Adds the Gradle Wrapper to the repo, this means a user would not need to have Maven or Gradle installed its all now inside the repo.
mvn clean compile exec:java
is equivalent to
gradlew clean build run
Works exactly the same way, updated the readme to reflect this.
build.gradle is the main change.
gradlew & gradlew.bat are autogenerated files.

LGTM :shipit:

bumping a year later :-)

after one year it still looks solid 💃

@zhangzhx will you or one of the admins be able to review this? Do you need some form of legal confirmation from @blundell as per PR #12 on the README?

@blundell
Thanks for the contribution and sorry for the late reply. This repo is lack of maintenance and outdated. The latest AWS Java SDK version is 1.11.157, and we have changed the recommended coding style for using AWS Java SDK quite a bit. But updating the SDK version and coding style could be in a separate PR.
Curiously, how are you using this sample repo? We are willing to add more samples other than S3 to the repo and would like to know what is the use case from your perspective of view.
@Mecharyry yes. As this is quite a big change adding support of Gradle, we do need a verbal agreement to the license.
@blundell This PR LGTM, are you submitting this change under Apache 2.0 License?

Apache 2.0, yes!

I have merged this PR. Please let us know if you have any comments or feedback to this repo.


monaco-languageclient config improvement, wrapper+languageclientwrapper improvements

Changes:

MonacoLanguageClient:

The MessageTransports are configured directly. This makes handling them easier in an application context or with LanguageClientWrapper
Extend ConnetionConfigOptions
Improve Type names

MonacoEditorLanguageClientWrapper

Allow to configure and use multiple language clients (implements #677)
UserConfig is now WrapperConfig and is one level less deep.
LanguageClientWrapper:

You can now pass an initialized web socket to LanguageClientWrapper. Internal handling of the message transports has been improved.
Restart options are now integrated directly.
Adjust language client configuration structure and naming

TODO:

[x] Further improve type names
[ ] More Unit Tests?
[x] Finish wrapper advanced example. Rename and allow to switch between python/json

@CGNonofr some TODOs left, but generally the changes look good, some awkward behaviour of vitest browser mode: https://github.com/TypeFox/monaco-languageclient/blob/wrapper-improvements/packages/wrapper/test/languageClientWrapper.test.ts#L71-L79
but the WA works, it never finishes once I put an await there. No problem when doing it manually in the exmaples. 🤷♂

@CGNonofr I may add more unit test, but feel free to review the changes.

@CGNonofr any thoughts? I was not very active the last two weeks. This will change again next week.

I admit I'm getting a bit lost in those libraries 😕

Fell being me, when looking at vast changes in monaco-vscode-api. 😆

This is ready from my side. I am already working on further improvements. WIP PR will come later and probably some questions, too.

And another PR with update to mva 9.0.x


Breakpoint gets a new child bound breakpoint every time its When Hit action changes

User-reported as follows:
I set a regular breakpoint, on line 732 say:

Now I right click on it in the gutter and add an action. It turned into four:

This makes things pretty confusing, if I click on the gutter to remove the breakpoint, it only removes one of them.

Basically, every time a When Hit action is added or removed for a given breakpoint (this is esp. easy to do inadvertently in VS 2015 due to the new non-modal UI for this), a new bound breakpoint is created and not being destroyed.
The root cause is potentially a debugger bug triggered by the fact that we don't report function names in IDebugCodeContext for breakpoints (because we don't know it). When "When Hit" changes, the debugger actually destroys & recreates the breakpoint, and as part of that codepath it needs to match the newly created breakpoint vs the old one; it always bails out early refusing to even compare them if function name is null or empty, and so the old breakpoint is never removed.


'organization_id' parameter is ignored in listConnections()

see https://github.com/workos/workos-php/pull/97

Hey @whitewhidow - gonna go ahead and close this one now that we merged/released. Lmk if anything further needs to be done on this one.


[BUG] Fix reference documentation issues in 'Microsoft.Azure.CognitiveServices.AnomalyDetector'

Describe the bug
Enabling ref documentation for the library gives errors on build.
Expected behavior
We should be able to generated ref documentation with build.
To Reproduce

Enable generation of ref docs by removing the exclusion of the library from 'eng\Directory.Build.Common.props'
Build the SDK directory.

@@.> to help check and CC @@.>.

From: msftbot[bot] @.>
Sent: Thursday, September 9, 2021 05:20
To: Azure/azure-sdk-for-net @.>
Cc: Bowen Gong @.>; Mention @.>
Subject: Re: [Azure/azure-sdk-for-net] [BUG] Fix reference documentation issues in 'Microsoft.Azure.CognitiveServices.AnomalyDetector' (#23883)
Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @yingqunpkuhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fyingqunpku&data=04|01|bowgong%40microsoft.com|507741d2e95449a036b808d9730e6295|72f988bf86f141af91ab2d7cd011db47|1|0|637667327930163871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KFJIJ02PM9GPmqQkntUP3GQYsvlXWFKzfav%2Fl84ZMa8%3D&reserved=0, @bowgonghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbowgong&data=04|01|bowgong%40microsoft.com|507741d2e95449a036b808d9730e6295|72f988bf86f141af91ab2d7cd011db47|1|0|637667327930163871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=HLbesPMBR%2B2lUvPh3gR0XJ6xKIdf4XB4uKcX%2FyEeAAE%3D&reserved=0.
Issue Details

Describe the bug
Enabling ref documentation for the library gives errors on build.
Expected behavior
We should be able to generated ref documentation with build.
To Reproduce

Enable generation of ref docs by removing the exclusion of the library from 'eng\Directory.Build.Common.props'
Build the SDK directory.
Author:
Mohit-Chakraborty
Assignees:

Labels:
Service Attention, Client, needs-triage, Docs, Cognitive - Anomaly Detector
Milestone:

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-net%2Fissues%2F23883%23issuecomment-915579754&data=04|01|bowgong%40microsoft.com|507741d2e95449a036b808d9730e6295|72f988bf86f141af91ab2d7cd011db47|1|0|637667327930173827|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=vfyQn61RR7KKdqyzJ5u9SY7GZnyZz%2FY5hE4LjAbBZcI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANOGEPABJ3B5FRTJMTV2QBTUA7HPFANCNFSM5DVLNDZA&data=04|01|bowgong%40microsoft.com|507741d2e95449a036b808d9730e6295|72f988bf86f141af91ab2d7cd011db47|1|0|637667327930173827|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=G7r3VXhR%2BizREvtTqbXth6X8xnjPK74k1UxzW%2FH%2FTU0%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|bowgong%40microsoft.com|507741d2e95449a036b808d9730e6295|72f988bf86f141af91ab2d7cd011db47|1|0|637667327930183782|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EmIZNoOPN9O8LKRj9jT1y5opgsJi1p4eEqdyFoQ%2BN6w%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|bowgong%40microsoft.com|507741d2e95449a036b808d9730e6295|72f988bf86f141af91ab2d7cd011db47|1|0|637667327930183782|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ijBNrrbhnRopJ8t5sM24FEh3c0ZX2c5GQJHQIQNVRxY%3D&reserved=0.

There's something misleading, for anomaly detector, we have two SDK folders, one is
azure-sdk-for-net/sdk/anomalydetector at main * Azure/azure-sdk-for-net (github.com)https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/anomalydetector which should be the latest one.
The other is
azure-sdk-for-net/sdk/cognitiveservices/AnomalyDetector at main * Azure/azure-sdk-for-net (github.com)https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/cognitiveservices/AnomalyDetector.  This one is where Mohit report the bug. But actually doesn't include the latest changes.
Shall we just remove the second one?
Adding @Laurent @.> and @@.> to help here
From: Bowen Gong @.>
Sent: Thursday, September 9, 2021 12:10 PM
To: Azure/azure-sdk-for-net @.>; Azure/azure-sdk-for-net @.>; Congrui Huang @.>
Cc: Mention @.>; Tony Xing @.>
Subject: RE: [Azure/azure-sdk-for-net] [BUG] Fix reference documentation issues in 'Microsoft.Azure.CognitiveServices.AnomalyDetector' (#23883)

@@.> to help check and CC @@.>.

From: msftbot[bot]<EMAIL_ADDRESS>Sent: Thursday, September 9, 2021 05:20
To: Azure/azure-sdk-for-net<EMAIL_ADDRESS>Cc: Bowen Gong<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-net] [BUG] Fix reference documentation issues in 'Microsoft.Azure.CognitiveServices.AnomalyDetector' (#23883)
Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @yingqunpkuhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fyingqunpku&data=04|01|conhua%40microsoft.com|232fac3dee05419db37208d97347a810|72f988bf86f141af91ab2d7cd011db47|1|0|637667573865090533|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=03HRnOSnR8v9sCboPSdclowcYmxU0%2BKRu92WWm7CvJs%3D&reserved=0, @bowgonghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbowgong&data=04|01|conhua%40microsoft.com|232fac3dee05419db37208d97347a810|72f988bf86f141af91ab2d7cd011db47|1|0|637667573865100490|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=bkFgHJOSsxyQv3PNAYYRVF5sQYk8OtGi151y8j3BLQs%3D&reserved=0.
Issue Details

Describe the bug
Enabling ref documentation for the library gives errors on build.
Expected behavior
We should be able to generated ref documentation with build.
To Reproduce

Enable generation of ref docs by removing the exclusion of the library from 'eng\Directory.Build.Common.props'
Build the SDK directory.
Author:
Mohit-Chakraborty
Assignees:

Labels:
Service Attention, Client, needs-triage, Docs, Cognitive - Anomaly Detector
Milestone:

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-net%2Fissues%2F23883%23issuecomment-915579754&data=04|01|conhua%40microsoft.com|232fac3dee05419db37208d97347a810|72f988bf86f141af91ab2d7cd011db47|1|0|637667573865100490|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=cOaIx4mW8jLOu8xyct3HHX2X8QYJnCL2qEuhIaXeL2A%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANOGEPABJ3B5FRTJMTV2QBTUA7HPFANCNFSM5DVLNDZA&data=04|01|conhua%40microsoft.com|232fac3dee05419db37208d97347a810|72f988bf86f141af91ab2d7cd011db47|1|0|637667573865110448|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=WBqu6AbKY3FCQPB5l6uU5N0xEj75J8VESHs9ft8vKMs%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|conhua%40microsoft.com|232fac3dee05419db37208d97347a810|72f988bf86f141af91ab2d7cd011db47|1|0|637667573865120403|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=B8HrJa5utaDMyiAz4MupTfqQTkktSOzsDg82NcERVIs%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|conhua%40microsoft.com|232fac3dee05419db37208d97347a810|72f988bf86f141af91ab2d7cd011db47|1|0|637667573865120403|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KogvwdPrNZjU7HJHbOJIr%2BnMOHGkEO45LOqKBJJOfAc%3D&reserved=0.

Hi @Mohit-Chakraborty, we're working with SDK team to see why we have two SDK folders for Anomaly Detector. Actually this one https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/anomalydetector is the latest one.

The issue is for the track1 SDK.
We are successfully generating ref docs for the track2 SDK that has the latest changes.


[TB] We should have fonts in a dropdown list (the list of top 100 google fonts) and then only allow user to select the available font weights

Problem/Concern

link to problem content

Proposed Solution

This is completed except for the font weight portion.
@lwnoble - when you get a moment could you please remind me which of font weights 0-4 use the primary or secondary fonts?

Fixed in latest code per @lwnoble


GOSSI filter wavelengths are just eyeballed from a plot

They should really be taken from some more official source, like the calibration pipeline.

Except there is no calibration pipeline... I think this is the best we will ever do.


Clarify Usage of module.* in Advanced Example

Thank you for these helpful examples! The advanced example uses the module values app and env in the service template but it isn't clear how these values are populated. A comment in the readme or the template would help to clarify that the values come from the file structure per the AppEnvCluster.

Whenever I'm unsure where a definition comes from, I use Go To Definition in my IDE to find it. It's also not documented where services comes from. Do you think the module style reference is so much harder to find / counter-intuitive that it warrants documenting, whereas a direct reference (to services) is not?

I was thinking that a comment clarifying this could save someone else the 10 minutes I lost on this in the future. I don't even get anything like "Go To Definition" when I am looking at this in VS Code, so that wouldn't have helped me either:

However, I did just find this in the readme:

This template mandates a three-level application/environment/cluster directory structure whose leaf directories contain the manifest files for a particular combination of application, environment, and cluster.

I missed that last night.


Obfuscating Breaks when using controlFlowFlattening on optional callback functions.

Current Behavior
When using controlFlowFlattening on an optional function parameter, the execution breaks instead of running a ?.() callback. A TypeError: _0x4f6992 is not a function error. This means it's calling the function even though the execution is optional.
Steps to Reproduce

Download and try code below, you should get a type error.

import * as esbuild from "esbuild";
import * as JavaScriptObfuscator from "javascript-obfuscator";

const code = `
function executionBreaksOfUndefinedOptionalImport(
  optionalCallback?: () => void
) {
  console.log('I work');
  optionalCallback?.();
  console.log('I work too');
}

console.log('Testing with no optional callback');
executionBreaksOfUndefinedOptionalImport();

console.log('Testing with optional callback');
executionBreaksOfUndefinedOptionalImport(() => {
  console.log('I am optional');
});
`;

(async () => {
  const jsResult = await esbuild.transform(code, {
    loader: "ts",
    sourcemap: false,
    minify: true,
    legalComments: "none",
  });

  console.log(`Pre Obfuscated Code: ${jsResult.code}`);
  eval(jsResult.code);

  const obfuscatedCode = JavaScriptObfuscator.obfuscate(jsResult.code, {
    controlFlowFlattening: true,
    controlFlowFlatteningThreshold: 1,
  }).getObfuscatedCode();

  console.log(`Obfuscated Code: ${obfuscatedCode}`);
  eval(obfuscatedCode);
})();

Disable controlFlowFlattening, and watch how it works fine.

Your Environment

Obfuscator version used: 4.1.1
Node version used: 22.12.0
Esbuild Version Used: 0.15.18

Stack trace
TypeError: _0x4f6992 is not a function
    at Object.bldMI (eval at <anonymous> (test.ts:39:3), <anonymous>:1:1188)
    at executionBreaksOfUndefinedOptionalImport (eval at <anonymous> (test.ts:39:3), <anonymous>:1:1303)
    at eval (eval at <anonymous> (test.ts:39:3), <anonymous>:1:1623)
    at test.ts:39:3
    at processTicksAndRejections (node:internal/process/task_queues:105:5)

The issue is not a failure of esbuild, I even tried it with normal javascript, but I used esbuild in this example to show a use case of typescript.

duplicate: https://github.com/javascript-obfuscator/javascript-obfuscator/issues/1277


Commented method to select dates

Hi i found in CalendarPickerView at line 290 that you commented method to select dates. Is this any reasons for that or just misstake? Becouse of that methods like withSelectedDates doesn't work as expected.

ya sorry for that, looks like I commented it for no reason
you can try that method uncommented and let me know if you runs into any error.
👍

I checked that and it looks ok, givens dates are selected properly ;)

@Gralls  Maybe send a PR :)


[Starter Project] Implement kubectl drain command

kunectl drain is a client side functionality that can be ported to this client from: https://github.com/kubernetes/kubernetes/blob/ca89308227abdf4f2c895118528c354c32aa3046/pkg/kubectl/cmd/drain.go#L239
This is a great starter project as the sequence of commands in the kubectl code are calling simple APIs. The best place for it is a new util package in /kubernetes.
Original post:

hi,I have achieved schedule and unschedule using  patch_node function.But which function can drain node in preparation for maintenance?

You may find your answer here: https://kubernetes.io/docs/api-reference/v1.6/

I think we have nothing to do with this issue, do we? Can we close it?

It would be great if somebody would confirm this.
It seems to me like kubectl cordon and kubectl drain functions is not implemented in api level. They are client functionality.
drain implemented here:
https://github.com/kubernetes/kubernetes/blob/ca89308227abdf4f2c895118528c354c32aa3046/pkg/kubectl/cmd/drain.go#L239
Seems like drain makes node unshedulable=true and delete all pods.
In short, this client does not have drain functionality. If I can find another client that has I will post it here.

It look like it is a client side functionality. I am not sure if they plan to move it to server but looking at the code, it should not be hard to convert that code to python using this client. Would you mind creating a feature request with this information and mark it as help needed so somebody look at it? This is a great starter project for anybody wants to start developing in this repo.

@mbohlool  I will need that functionality soon and it seemed simple enough to implement it on top of this library. If it is not implemented when I do it, I can contribute it to the library. I have a question about it though. I thought this library was completely autogenerated from swagger source. So, if I were to add a functionality like this how would I attach my functionality to auto generated code. Can you direct me to an example in codebase? I could not find any examples.

All files in the kubernetes/base (that some of them are sym-linked back in) are manually coded. You can add a utility package or something like that is the base repo and update the submodule here to use them. examples are config/* and watch/* files. This document explains the structure of our clients in more detail.

/assign


feat: move style injection from render to useInsetionEffect

Atm this is just a quick experiment if we can utilize useInsertionEffect instead of injecting styles in render which isn't the best for performance. E.g. following the guidelines described in https://github.com/reactwg/react-18/discussions/110
From my limited and quick testing this seems to work fine, is there anything I'm missing and does this sound like a good idea to you? If that is the case I'll clean up the PR a little more, e.g. renaming generateAndInjectStyles to something more fitting

I'll have to look at this one later. I switched us to use the new effect type for global styles but I'm not sure about doing it for component styles since they need to be injected as fast as possible on first render

One possible solution could be to inject styles in useInsertionEffect if React 18, otherwise fallback to doing it in render like before

@probablyup

they need to be injected as fast as possible on first render

I haven't looked at the code in React to verify, but the doc claims "it fires synchronously before all DOM mutations."
That seems to satisfy "as fast as possible" but are there situations where this still isn't enough?
Otherwise this LGTM, pending whatever cleanup @alexandernanberg wanted to do.


ValueError: could not broadcast input array from shape (2048,1448) into shape (31,200)

I am running this in an Anaconda env (conda 4.7.12) in Windows 10 Enterprise.
I am getting this stacktrace when running pipeline.recognize on one of my images:
2020-05-17 09:43:17.709956: W tensorflow/core/framework/cpu_allocator_impl.cc:81] Allocation of 759169024 exceeds 10% of system memory.
2020-05-17 09:43:18.125906: W tensorflow/core/framework/cpu_allocator_impl.cc:81] Allocation of 759169024 exceeds 10% of system memory.
Traceback (most recent call last):
  File "c:\Users\my_user\work\my_project\processing\run_example.py", line 109, in <module>
    main()
  File "c:\Users\my_user\work\my_project\processing\run_example.py", line 49, in main
    detection_kwargs=detection_kwargs
  File "c:\Users\my_user\work\my_project\processing\data_extraction.py", line 518, in extract_bbs_text_dir_files
    df, prediction_group = get_df_from_labeled_img(path_img, path_json, detection_kwargs=detection_kwargs)
  File "c:\Users\my_user\work\my_project\processing\data_extraction.py", line 215, in get_df_from_labeled_img
    prediction_groups = pipeline.recognize(images, detection_kwargs=detection_kwargs)
  File "C:\Users\my_user\Anaconda3\envs\my_env\lib\site-packages\keras_ocr\pipeline.py", line 58, in
recognize
    **recognition_kwargs)
  File "C:\Users\my_user\Anaconda3\envs\my_env\lib\site-packages\keras_ocr\recognition.py", line 431, in recognize_from_boxes
    target_width=self.model.input_shape[2]))
  File "C:\Users\my_user\Anaconda3\envs\my_env\lib\site-packages\keras_ocr\tools.py", line 97, in warpBox
    full[:crop.shape[0], :crop.shape[1]] = crop

ValueError: could not broadcast input array from shape (2048,1448) into shape (31,200)

I have processed more than 100 images, but this is happening in this image.
Unfortunately, I cannot share the image as it has personal data, but I can help to debug the problem, but first I would like to know if this error is from my end or it could be a issue of keras-ocr.
Thanks in advance

Something is going wrong with the box warping process. Please provide a sample image to reproduce the issue. My apologies for the inconvnience.

Exactly, I had error on this line, too. I ended up wrapping warpBox call into a try: catch block so that execution does not break.

Hi!
I have found that if the box width/height is smaller than the supplied margin, this error occurs.

I'm using size 640x640 as the example of this repo with the margin 50 and having the same issue, if I try a smaller margin I also get the same problem. Also sometimes there is another problem when an array called slots becomes zero sized (This one has something to do with the contours, but I don't remember well and the error of this issue is happens more often)


 Please report to GitHub!)!

BDWC (Installer and Python module) currently works in Linux with GNOME-based DE.
Thus, I have no plans to make "Installer for macOS". (Also we need to find a way to change wallpapers on macOS.)
Sorry :-)


KmerID dependencies in conda

Dear developers,
I'm trying to setup all the python dependencies needed to install kmerID in a conda environment (due to server policies in my institution) but actually I do not manage to pip sys, subprocess, os, operator, glob and tempfile. In detail, I currently have Python 2.7.15 (Anaconda, Inc.) and I get the following error:

Could not find a version that satisfies the requirement tempfile (from versions: )
No matching distribution found for X

with X equal to the listed libraries. Do you know how to circumvent this so that I can try to install KmerID effectively?
Thank you for your support,
Nora

Hi Nora, all the libraries you list are part of the Python standard library (https://docs.python.org/2/library/). So there should be no need to install them. They should be always already included in every Python distribution. Ulf

Dear Ulf,
thank you for your quick response. My question derived from not finding the listed modules in the pip freeze resulting list. If you confirm me that I can proceed nonetheless, I'll try to install KmerID hope everything works fine.
Best,
Nora


add pydantic for validation and serialization

while the interface should be backwards compatible, i.e. most tests and examples do not need any changes, this change wants a minor version bump
examples should be updated in a following change to use pydantic models where appropriate

See https://github.com/ollama/ollama-python/pull/311 for pydantic type class fixes and serialization


[Slack] Worker terminated due to reaching memory limit

Extension
https://www.raycast.com/mommertf/slack
Raycast Version
1.77.1
macOS Version
14.6
Description
I tried creating a Raycast app with a app manifest directly in Slack and also using the pre-made Slack app (believe that was newly added). With both no functionality works.
The error log shows this:
Error: Worker terminated due to reaching memory limit: JS heap out of memory

NodeError:errors:406:5
[kOnExit]:worker:313:26
Worker.<computed>.onexit:worker:229:20

I haven't done too much troubleshooting, but I've developed Slack integrations before and as I've been using Slack for a long time I wonder if i have too many messages/chats so it can't handle it?
Also using Enterprise Grid.
Steps To Reproduce

Create Slack app with manifest
Install to workspace
Copy xoxp- key into the extension
Try to search or read unread messages - nothing works

Current Behaviour
No response
Expected Behaviour
No response

Could you share a recording? It'll make it easier to see what's happening.

@thomaslombart don't have my Mac with me at the moment, probably won't be able to send anything until tomorrow.
But literally nothing happens. I did a quick GitHub search and it seems like this memory issue is quite widespread across multiple extensions wherever it is processing a large amount of data.

Very likely an issue with large workspace : #4284
When pulling data from the Slack API, the extension doesn't stop pulling until it gets every users, channels and messages.

The optimal solution would be to replace pagination like these:
https://github.com/raycast/extensions/blob/5c077f5a72c66ae6caab0a5958f4d89fb30c86df/extensions/slack/src/shared/client/SlackClient.ts#L84-L94
with:

(recommended) just retrieve the first page and support typeahead search. We will need to explore if all the APIs support a search pattern to restrict the results. For example, can't find it for https://api.slack.com/methods/users.list
raycast native pagination and a typeahead search. But unless we figure out an API that supports typeahead search pattern, I think users of large workspaces will need to keep scrolling down to retrieve all items and search.


Wrong bech32 format with a newly  generated wallet

Summary of Bug
so i have a 2nd user now with wrong bech32 format error, which in the first case i thought its because a user tried to use his old wallet address, but it seems its not the case, see the info:
root@vmi561079:~/celestia-app# celestia-appd keys add secondary_wallet
root@vmi561079:~/celestia-app# celestia-appd version
0.6.0
root@vmi561079:~/celestia-app# celestia-appd tx staking delegate celestiavaloper1q3v5cugc8cdpud87u4zwy0a74uxkk6u43cv6hd 500000utia  --from=celestia1zx6ctp0lk4u3r6r0yxnmcyrcwyapfr4gp3la5k --chain-id=mamaki --fees 200utia --keyring-backend test
**Error: decoding bech32 failed: invalid checksum (expected q4gx4p got 3cv6hd)**
root@vmi561079:~/celestia-app#

note that the exact same message appeared in the other case too, and i saw one other in russian channel: (expected q4gx4p got 3cv6hd)
For the first case i run into, asking the user to import wallet with seed seems to have worked, but that was when i thought he wasnt using a new address, so not sure how this can really happen.
Version
0.6.0
Steps to Reproduce

install 0.6.0
create wallet
try to delegate

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

I can't find celestiavaloper1q3v5cugc8cdpud87u4zwy0a74uxkk6u43cv6hd on https://celestia.explorers.guru/ so I don't think it is a valid validator address.
I did find celestiavaloper1q3v5cugc8cdpud87u4zwy0a74uxkk6u4q4gx4p here: https://celestia.explorers.guru/validator/celestiavaloper1q3v5cugc8cdpud87u4zwy0a74uxkk6u4q4gx4p
Given the prefix of the address is the same, I expect this is a copy + paste error and the user intended on using the latter address.

Based on Discord I think the user intended on delegating to Nodes.Guru. Based on https://celestia.explorers.guru/validator/celestiavaloper1kg5njpnn494gnek8p7ug6juqrzwq3c3gxrdj5s I think they can try delegating to the address celestiavaloper1kg5njpnn494gnek8p7ug6juqrzwq3c3gxrdj5s.

Resolved over Discord, this issue was user error.


[lecture12-2][0612] Small typo in the lecture slide

There's a really really minor typo in the lecture slide p27.

As you can see, above slide is incorrectly written as (B, BN), even though it should calculate the IoU of B1 and BN.
Thank you.

@goals4292 Oh, Thank you for the comment, you are correct:)
I will revise it in the final revision.


chore(l10n): Improve zh-tw localization quality

Description

Change some words to the common ones in Taiwan , e.g. 默认->預設, 文件->檔案, etc.
c.c. @SiderealArt
Just curious about if you are open to have a Traditional Chinese version for the website? There are only English and Simplified Chinese currently.
Related Issues

Checklist:

[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you linted your code locally prior to submission?
[x] Have you successfully ran app with your changes locally?

The website does not appear to be open-source though.


Added ibm-watson-iot org

Added ibm-watson-iot org for our open source projects to be aggregated and displayed under IBM.io site.

Made the changes @stevemart .

thanks!


TranspileLayout page in the IBM QISKIT Documentation (section Transpilation, subsection qiskit.transpiler)

URL to the relevant documentation
https://github.com/Qiskit/documentation/blob/949404627ff59ea8356aba2675bb32dcd44557b9/docs/api/qiskit/qiskit.transpiler.TranspileLayout.mdx?plain=1#L18
Please describe the UI problem.
Documentation seems incomplete
In this documentation page the information is incomplete. It is an important reference for the IBM Qiskit Challenge 2024. The layout and routing scheme for the transpilation documentation needs to have a clear reasoning about how the init_layout is transformed to the final_layout by mapping the input circuit to the backend and then adding the swap gates to make it compatible. That explanation is incomplete as the sentence cuts off with a colon and never actually finishes off the example. Please correct it.

Hi @ronitd2002, good catch. The source code for this API reference documentation lives in qiskit/qiskit, so I will have this issue transferred there. In the meantime, I see in the source file that the code block is in the file but not rendering properly, so in the meantime I will paste the missing code here, which I hope helps for the Challenge:
       from qiskit.circuit import QuantumCircuit, QuantumRegister

       qr = QuantumRegister(3, name="MyReg")
       qc = QuantumCircuit(qr)
       qc.h(0)
       qc.cx(0, 1)
       qc.cx(0, 2)
       qc.draw("mpl")```


Text is converted to shapes

Hi,
this is a general question regarding the handling of text elements; I have implemented SVG rendering using your library and I am very pleased with the results, especially since you are abstracting the output device which is exactly what I need in my project. However, I realized that you are converting all text elements into shapes which is nice for rendering but in my case, I need the text to be rendered as such. Is there a way to intercept the rendering process to accomplish this?
Apart from that, this is a great little library to handle SVGs. Thank you very much.

Could you please clarify what you mean by

[...] but in my case, I need the text to be rendered as such

Rendering text inherently requires it to be converted into shapes.

Sure. I need to generate a custom output format that does not have the typical Graphics2D support and is quite limited. Therefore is was happy to see that your abstraction Output is fairly similar to my generic output interface, so the mapping was extremely simple. However, I need to output text as text elements so that they can be treated as such in the receiving application (e.g. for searching).

Rendering text inherently requires it to be converted into shapes.
I do not agree with that. it's true if you go end render in a bitmap but definitely not if you want to output a PDF or a similar format.

Actually I already spent some time going through the code and I do understand why you prefer the shape driven approach but as explained, it does not work for me. I was considering a secondary interface only for text support that could be treated as a marker interface and could be used instead of the shape output if implemented. That would be something I would try to handle in a fork If I decide to go down that road.
BTW: i am located in Frankfurt.

Sure. I need to generate a custom output format that does not have the typical Graphics2D support and is quite limited. Therefore is was happy to see that your abstraction Output is fairly similar to my generic output interface, so the mapping was extremely simple. However, I need to output text as text elements so that they can be treated as such in the receiving application (e.g. for searching).

I see, thank you for the clarification. My statement regarding rendering was with bitmap formats in mind, which of course is not true for non-pixel based formats.

Actually I already spent some time going through the code and I do understand why you prefer the shape driven approach but as explained, it does not work for me. I was considering a secondary interface only for text support that could be treated as a marker interface and could be used instead of the shape output if implemented. That would be something I would try to handle in a fork If I decide to go down that road.

I have some ideas how to handle this. Will do some experiments and report bag :)

The newest snapshot adds a TextRenderer textRenderer() method to Output. As it is foreseeable that text rendering might need some reworking in the future TextRenderer has not much exposed to it. However I have also added a TextExtractingTextRenderer abstract implementation of TextRenderer, which can be used to consume all text content inside a <text> element. Let me know if this suffices in your scenario.

I have pulled the code and right now I am trying to wrap my head around the rendering process. A few questions:

How do I get the actual font settings for the rendered text? So far this seems all to be located in the TextContainer, right?
What is the purpose of the contextFontSize() function?
The types TextContainer, StringTextSegment, and TextSegment are package visible so I cannot use them to build out  the rendering logic.

Thank you for your work on this. I guess with the information above, I can implement a rendering for text.

How do I get the actual font settings for the rendered text? So far this seems all to be located in the TextContainer, right?

Font information is available through the RenderContext. I can make that available in the TextExtractingTextRenderer implementation. Do you need access to any other attributes e.g. x, y positions etc.?

What is the purpose of the contextFontSize() function?

The default font size for SVGs is user-agent dependent and usually inherits the font size of "surrounding elements" (e.g. if it is embedded in html content). In this case this is used to pass the font size of the current Graphics2D context being rendered to (if the Output is graphics based). If you return an empty optional, then the default fallback size of 10 will be used.

The types TextContainer, StringTextSegment, and TextSegment are package visible so I cannot use them to build out  the rendering logic.

This is on purpose as I expect text rendering might need some refactoring in the future to allow for more flexibility. I can't promise any API stability of these types hence they aren't exposed. I am aware that this greatly limits implementors of TextRenderer. However I am sure we can work out a suitable interface that works for you.

How do I get the actual font settings for the rendered text? So far this seems all to be located in the TextContainer, right?

Font information is available through the RenderContext. I can make that available in the TextExtractingTextRenderer implementation. Do you need access to any other attributes e.g. x, y positions etc.?

OK, if I receive all relevant information for text rendering in public abstract void processText(@NotNull String text) then I can take over on my side. I have seen that the RenderContext basically provides most of the information to setup the rendering target and amend the text rendering (baseline, anchor, and such things). If the text is not supposed to be rendered at (0|0), I would need of course the coordinates. I have seen that the rotation is already setup in the transform so that seems to be handled elsewhere?

What is the purpose of the contextFontSize() function?

The default font size for SVGs is user-agent dependent and usually inherits the font size of "surrounding elements" (e.g. if it is embedded in html content). In this case this is used to pass the font size of the current Graphics2D context being rendered to (if the Output is graphics based). If you return an empty optional, then the default fallback size of 10 will be used. Note that it will only be called once at the start of rendering.

Got it. Thank you.

The types TextContainer, StringTextSegment, and TextSegment are package visible so I cannot use them to build out  the rendering logic.

This is on purpose as I expect text rendering might need some refactoring in the future to allow for more flexibility. I can't promise any API stability of these types hence they aren't exposed. I am aware that this greatly limits implementors of TextRenderer. However I am sure we can work out a suitable interface that works for you.

I think if I can handle the rendering as discussed above, I am fine. Well, the utter complexity of the typographic model of SVG might not be covered by that approach (yet) but it makes implementation easier for now.

OK, if I receive all relevant information for text rendering in public abstract void processText(@NotNull String text) then I can take over on my side. I have seen that the RenderContext basically provides most of the information to setup the rendering target and amend the text rendering (baseline, anchor, and such things). If the text is not supposed to be rendered at (0|0), I would need of course the coordinates. I have seen that the rotation is already setup in the transform so that seems to be handled elsewhere?

Is your goal to truthfully replicate the visuals of the resulting SVG with in addition knowing at what location each letter is positioned? If so would it be workable for you to use the shape converted rendering for the visuals while receiving additionally the letter positions? This way you wouldn't have to worry about replicating the position computation for each letter.

Of course, the rendition should be 'as close as possible' to what you are rendering as shapes. But on the other hand, I would like to render as much glyphs as possible in one drawing as this would push rendering optimizations to the underlying graphics engine (like kerning and such things). So if it is a <text>, I would like to get the position, styling, and the text itself.

Well, the limitation of my 'Output Device' is that the resolution is extremely limited (it's a very special custom graphics format) and thus any vector graphics is limited to discrete 16 bit integer coordinates. While this is sufficient for the general graphics elements, it becomes a problem with glyphs as these are very narrow and the bezier control points typically detoriate the depiction when snapped.  On the other hand does the format text rendering through the underlying display engine so I would prefer rendering "Hello World" in one phrase, eventually bold, italic, 12pt at position 123|456 rather than having each character individually for the above reasons. While having the character itself is already an enhancement over the shape approach, having the whole string would be the best.
And maybe for your understanding; I am not looking for a pixel-perfect rendition. I can live with these restrictions as long as I can render the majority of the symbols that I receive as SVGs.
I hope this gives you some insight into the limitations that I am facing. I had created my own SVG rendering engine but it was extremely limited and could not render all SVG files. Especially it had issues with different namespaces and styles so I prefer using your engine as it has much better support for these features.
I have just pulled the latest version and will see, if I can accomplish what I need with the latest changes. Thank you for your support so far.

One quick observation; I have pulled the latest code and suddenly no characters are rendered anymore. I have found out that in GlyphRenderer.java in line 78, the shape of the glyph contains no coords. Digging into the issue a bit further, I found out that none of the TextSpan nodes contained the actual text that was in the SVG file. I believe it is related to an issue with the new CodepointsCharacterIterator that fails to return the characters. I changed the method first() to reset the field index to zero and then the characters get rendered. I am sure that this is not the fix but it should lead you into the right direction to fix this.

Thank you for the additional information. Maybe I can find a nice way to make consecutive characters (i.e those without any explicit placement using x, y or rotate) available to the TextOutput. For everything else the only sane way is to provide single characters. In this case you would have to decide yourself based on the transform whether you want to join them together. Note that space characters are also passed to the TextOutput, so you could always record the first transform, join all the characters and render the resulting string at that first transform in TextOutput#endText.

I believe it is related to an issue with the new CodepointsCharacterIterator that fails to return the characters. I changed the method first() to reset the field index to zero and then the characters get rendered. I am sure that this is not the fix but it should lead you into the right direction to fix this.

Thank you for digging into this. Seems like the first() and last() methods of CharacterIterator are meant to mutate the iterator, which I missed. Somehow this didn't create any issues with the JDK I am using.

I have added TextOutput#glyphRunBreak() which signals that the end of consecutively laid out code points has been reached (in the sense that none of the glyphs specify a custom x,y or rotation). I have also added an GlyphRunTextOutput implementation which handles collecting the code points into strings of glyph runs.

I have figured out how to use that. I believe I can implement the majority of the renditions using my awkward output logic. The glyphRunBreak is very helpful, however you might consider to have beginText() and endText() final with delegate methods so that the housekeeping is not accidentially overwritten? I use these to block any shape rendering that might come from jsvg as I am taking over.

Ah good catch. Yes they should be final indeed.


Fixing the rendering of the separator widget

before:

after:

@ascott

Coverage decreased (-0.09%) to 81.148% when pulling f64436fe687aa5a35410b9b2051c66b5f827395b on mistercrunch:fix_separator into e243a14c64c97f81e205037bb11bcd9170c17163 on airbnb:master.


Getting wrong spec type from spec constructor

I would expect (and need) AssociativeSpec there.
I don't know how this is happening, looks like some satisfies? oddity. Re-ordering the cond from in octet.spec/spec would surely solve it. I don't have any explanation for this behavior, there's nothing special in my project.
NOTE: this is ClojureScript

I'm not clearly inderstand your code, but I'll try to explain that I see wrong in it with an analogous example:
(satisfies? cljs.core/IVector [])
;; => true
(satisfies? cljs.core/IVector :foo)
;; => false

Then:
(every? #(satisfies? cljs.core/IVector %) [[] []])
;; => true
(every? #(satisfies? cljs.core/IVector %) [:foo []])
;; => false

And then:
cljs.user=> (defn foobar [& args] (every? #(satisfies? cljs.core/IVector %) args))
#'cljs.user/foobar
(foobar [] [])
;; => true
(foobar :foo [])
;; => false

The problem is not in spec, the problem is that you are expecting that :foo will satisfy ISpec and it does not.

And then, I found that it works as expected:
cljs.user=> (buf/spec :field1 buf/int32)
#object[octet.spec.AssociativeSpec]
cljs.user=> (buf/spec buf/int32)
#object[octet.spec.IndexedSpec]

@niwinz It's the other way around, somehow  octet.spec/spec thinks keywords satisfies ISpec and it gives me IndexedSpec instead of AssociativeSpec.

@skrat Hmm, I think that this behavior is caused because you are calling (buf/int32) instead just passing it as value.

I see the source of octet.spec/spec and there's nothing wrong with it. I don't know how to solve this, it keeps giving me IndexedSpec. I tried both calling buf/uint32 and (buf/uint32), there's no difference.

Very very strange. In any case forget my first response, I was completely confused and didn't see the real problem.
Can you try it in an isolated environment? just download octet repository and execute ./scripts/repl (having nodejs installed) and try to execute:
(require '[octet.core :as buf])
(buf/spec buf/uint32)
;; => #object[octet.spec.IndexedSpec]
(buf/spec :foo buf/uint32)
;; => #object[octet.spec.AssociativeSpec]

@niwinz Yes you're right, it works in isolated environment. I'm not sure where to look now but I guess this can be closed.


Where to download the Tusimple dataset

The author provide a script to generate training tfrecords from the dataset. But where can I download the dataset?

@ArtlyStyles The dataset seems not avaliable now:)


vscode has no code hints for bursted assets

There is no code hint when writing assets test, is there any good way?
For example, when I input assert.a, I hope vscode can prompt me the words are, are_not

We do not run busted in vscode nor was it designed to support code completion in vscode. Pull requests are welcome!

Hi, @DorianGray
I wrote a simple vscode code snippet, which can quickly generate busted unit test related code, if you need it, I can push a Pull request
{
	"assert.is_true": {
		"prefix": "assert.is_true",
		"body": [
			"assert.is_true($1);"
		],
		"description": "busted assert"
	},
	"assert.is_not_true": {
		"prefix": "assert.is_not_true",
		"body": [
			"assert.is_not_true($1);"
		],
		"description": "busted assert"
	},
	"assert.is.equal": {
		"prefix": "assert.is.equal",
		"body": [
			"assert.is.equal($1);"
		],
		"description": "busted assert"
	},
	"assert.is.truthy": {
		"prefix": "assert.is.truthy",
		"body": [
			"assert.is.truthy($1);"
		],
		"description": "busted assert"
	},
	"assert.is.falsy": {
		"prefix": "assert.is.falsy",
		"body": [
			"assert.is.falsy($1);"
		],
		"description": "busted assert"
	},
	"assert.is_not.equal": {
		"prefix": "assert.is_not.equal",
		"body": [
			"assert.is_not.equal($1);"
		],
		"description": "busted assert"
	},
	"assert.is.errors": {
		"prefix": "assert.is.errors",
		"body": [
			"assert.is.errors(",
			"\tfunction()",
			"\t\t$1",
			"\tend",
			")"
		],
		"description": "busted assert"
	},
	"assert.is_not.errors": {
		"prefix": "assert.is_not.errors",
		"body": [
			"assert.is_not.errors(",
			"\tfunction()",
			"\t\t$1",
			"\tend",
			")"
		],
		"description": "busted assert"
	},
	"test": {
		"prefix": "test",
		"body": [
			"test(",
			"\t\"$1\",",
			"\tfunction()",
			"\t\t",
			"\tend",
			")"
		],
		"description": "busted assert"
	},
	"describe": {
		"prefix": "describe",
		"body": [
			"describe(",
			"\t\"$1\",",
			"\tfunction()",
			"\t\t",
			"\tend",
			")"
		],
		"description": "busted assert"
	}
}

These functions actually come from the luassert project. I think the correct way to get completion in your editor is to tel your code scanner to look for function definitions in that project's code as well as busted & your own local project.
I do not think code libraries should be responsible for creating completion routines for any editor, VScode or otherwise. The list would never end, and some have many different formats they would want the data in depending on the plugin you used. This should be generated by the editor/plugin somewhere feeding it the code libraries being used.

OK, I will try it later


Add command to list current location in source

When stopped in a debug session, have a command to show source code around the current location this (or any arbitrary) thread is stopped at. Should have a default (5 lines above/below or similar) but also accept an argument to display more source code context.

I am working on this.   Kerry Lynn<EMAIL_ADDRESS>

@kerlyn-bsd3 any progress on this? Let me know if you run into any issues or have any questions.

Sorry, been swamped at work.  Should have a PR tonight or tomorrow.

Isn't this almost working already? Stopping appears to print the source code so there's just missing a "list" command for manual source print?

@quarnster correct. The part this is not implemented yet, however, is the ability to print out sections of different source files outside of your current position.


Correct display of endpoints

Over-long line added in eacc7ac58de6701416cbe5d44a4d3dfa4ba679a2
prevents GitHub display of table.

Travis fails for an unrelated reason.

Thanks @andrewgaul!